Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ensure allocated path is long enough for pssolar #8261

Merged
merged 1 commit into from
Jan 7, 2024
Merged

Conversation

PaulWessel
Copy link
Member

See GenericMappingTools/pygmt#2959 for background. Ensure we add at least 1 point when going to pole. For whatever reason this is not needed on Windows and macOS but cost us just another allocation point.

See GenericMappingTools/pygmt#2959 for background.  Ensure we add at least 1 point when going to pole.
@PaulWessel PaulWessel added the bug Something isn't working label Jan 7, 2024
@PaulWessel PaulWessel added this to the 6.5.0 milestone Jan 7, 2024
@PaulWessel PaulWessel self-assigned this Jan 7, 2024
@PaulWessel PaulWessel merged commit 6da8ee2 into master Jan 7, 2024
6 checks passed
@PaulWessel PaulWessel deleted the path-safety branch January 7, 2024 09:32
@PaulWessel
Copy link
Member Author

I am rebuilding the tar balls but no need for new Windows or macOS installers since they were never affected by this strange bug.

@joa-quim
Copy link
Member

joa-quim commented Jan 7, 2024

Don't know why it was said that it didn't affect Windows. It did.

@PaulWessel
Copy link
Member Author

OK, I just echoed what Yvonne said. So you got the commit included in your new installers then?

@joa-quim
Copy link
Member

joa-quim commented Jan 7, 2024

No. Saw only now.

@seisman
Copy link
Member

seisman commented Jan 7, 2024

Please check out the 6.5.0 tag and build the win64 installers again, then Paul should upload the new installers to the FTP and I will replace the old ones on the release page.

@joa-quim
Copy link
Member

joa-quim commented Jan 7, 2024

That will probably take me 2 to 3 hours.
I keep saying that we should stop doing these last second changes. Past have shown that they often do more harm than good.

@PaulWessel
Copy link
Member Author

Well, true, but we need to move to a system where this is all done by CI so we dont rely on people being alive or on travel or eating or whatever. All this manual activity (from me to) lead to issues.

@joa-quim
Copy link
Member

joa-quim commented Jan 7, 2024

@yvonnefroehlich
Copy link
Member

joa-quim: Don't know why it was said that it didn't affect Windows. It did.

PaulWessel: OK, I just echoed what Yvonne said.

I am very sorry! I did not expect that this fix will be included in GMT 6.5, and I was busy with other stuff today.
Based on what I tried on Saturday, I was not able to reproduce this issue in PyGMT on Windows and only faced it on Linux. But if @joa-quim found this issue also on Windows, I probably overlooked something 🙁. Thanks @joa-quim for looking at this again and also re-building the installers!

@PaulWessel
Copy link
Member Author

@seisman I have placed @joa-quim's latest installers in the GMT ftp directory with correct permissions. See the SHAs.txt file in his post for the SHAs.

@joa-quim
Copy link
Member

joa-quim commented Jan 7, 2024

Yvonne, if you didn't see it is very likely because it wasn't there, but you probably used the VCPKG build, that I find often strangely different from my builds (c.f. the different test failures from what I see)

@seisman
Copy link
Member

seisman commented Jan 8, 2024

@joa-quim Are you sure the sha256sum values are correct?

I have:

f8b7133ab8199117fc316cf73bd8390b1d5b3a3aa78c6451bb9e63ee6476c585  gmt-6.5.0-win64.exe
6a638ce91da637fe17ce77d9bd8163150128fbe37439537f826dfc2359e5a4dc  gmt-6.5.0-win64.zip

@joa-quim
Copy link
Member

joa-quim commented Jan 8, 2024

@seisman The sha's you posted above are correct and correspond to what I have in my local SHAs.txt file. Either I forgot to update it (which I think I did) or it copied a cache file. Update the SHAs.txt file at site.

@seisman
Copy link
Member

seisman commented Jan 8, 2024

OK, I've uploaded the new installers to the release page and also updated the shasum file.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants