Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@bska
Copy link
Member

@bska bska commented Aug 21, 2025

In particular this means that

GSATPROD
  SAT  12345.67 89101.1 121314.15 /
/

will create the group SAT if it does not already exist at the point when the GSATPROD keyword is entered. Groups created in this way will be parented directly to the FIELD group.

@bska bska added the manual:enhancement This is an enhancement/improvent that needs to be documented in the manual label Aug 21, 2025
@bska
Copy link
Member Author

bska commented Aug 21, 2025

I'm creating this PR in draft mode for two reasons

  1. It depends on, and contains, the earlier PR Polish IOrderSet Implementation #4687.
  2. I need to fully verify that it solves the issue that prompted PR Check that group exists before accessing #4688.

I will keep the PR in a draft state until such time as it is ready for review and merging.

@bska bska force-pushed the gsatprod-creates-new-group-if-needed branch 2 times, most recently from c478c29 to 5dee650 Compare August 22, 2025 07:33
In particular this means that

    GSATPROD
      SAT  12345.67 89101.1 121314.15 /
    /

will create the group 'SAT' if it does not already exist at the
point when the GSATPROD keyword is entered.  Groups created in this
way will be parented directly to the FIELD group.
@bska bska force-pushed the gsatprod-creates-new-group-if-needed branch from 5dee650 to f2ba587 Compare August 22, 2025 08:25
@bska
Copy link
Member Author

bska commented Aug 22, 2025

The earlier PR #4687 was merged into the master branch and I'm now satisfied that this solves the problem highlighted by #4688. I'm marking the PR as "ready for review" and I'm running a build check.

@bska bska marked this pull request as ready for review August 22, 2025 09:46
@bska
Copy link
Member Author

bska commented Aug 22, 2025

jenkins build this please

Copy link
Member

@akva2 akva2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good and seems to do what was announced

@bska
Copy link
Member Author

bska commented Aug 22, 2025

PR approved and build check is green. I'll merge into master.

@bska bska merged commit dca3571 into OPM:master Aug 22, 2025
2 checks passed
@bska bska deleted the gsatprod-creates-new-group-if-needed branch August 22, 2025 10:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

manual:enhancement This is an enhancement/improvent that needs to be documented in the manual

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants