Skip to content

Fix for GMT-6.3 issue adding -0 suffix to file name in the output text#291

Merged
sungeunbae merged 2 commits intomasterfrom
gmt-6.3-fix
Jul 3, 2023
Merged

Fix for GMT-6.3 issue adding -0 suffix to file name in the output text#291
sungeunbae merged 2 commits intomasterfrom
gmt-6.3-fix

Conversation

@sungeunbae
Copy link
Member

@sungeunbae sungeunbae commented Jul 2, 2023 •
edited
Loading

The output of this command contains file name ending with -0, which should be without the -0 suffix.

 gmt spatial -Ie -D /home/seb56/Documents/CS_faults_coverage/nhm_to_real/_tmp_Hossack_sh3c_aeo/srf.path /home/seb56/Pre-processing/VM/../SrfGen/NHM/res/rough_land.txt /home/seb56/Documents/CS_faults_coverage/nhm_to_real/_tmp_Hossack_sh3c_aeo/tempEXT.tmp
176.909264461	-39.4315739635	55.3311625775	234.106867229	/home/seb56/Pre-processing/VM/../SrfGen/NHM/res/rough_land.txt-0	/home/seb56/Documents/CS_faults_coverage/nhm_to_real/_tmp_Hossack_sh3c_aeo/tempEXT.tmp-0
173.768737845	-39.3656360293	92.7109810386	329.843700793	/home/seb56/Pre-processing/VM/../SrfGen/NHM/res/rough_land.txt-0	/home/seb56/Documents/CS_faults_coverage/nhm_to_real/_tmp_Hossack_sh3c_aeo/tempEXT.tmp-0

This causes a string match to fail and the function subsequently returns invalid output.

@sungeunbae sungeunbae changed the title Fix for GMT-6.3 issue adding -01 suffix to file name in the output text Fix for GMT-6.3 issue adding -0 suffix to file name in the output text Jul 3, 2023
@sungeunbae sungeunbae merged commit 4bed043 into master Jul 3, 2023
@sungeunbae sungeunbae deleted the gmt-6.3-fix branch July 3, 2023 22:28
sungeunbae added a commit that referenced this pull request Jul 6, 2023
@sungeunbae
Fixed a bug introduced by GMT-6.3 related fix (#291)
fb3b49d
@sungeunbae sungeunbae mentioned this pull request Jul 6, 2023
Merged
sungeunbae added a commit that referenced this pull request Jul 6, 2023
@sungeunbae
Merge pull request #292 from ucgmsim/gmt-6.3-fix
9c82db2 
Fixed a bug introduced by GMT-6.3 related fix (#291)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@joelridden joelridden joelridden approved these changes

@claudio525 claudio525 claudio525 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@sungeunbae @joelridden @claudio525