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

Fix table references to SATOPTS in KRNUM and IMBNUM #336

Merged
merged 4 commits into from
Jul 24, 2024

Conversation

gdfldm
Copy link
Collaborator

@gdfldm gdfldm commented Jul 18, 2024

Change refs to tables to refs to keyword

Table refs have now been fixed:

I have now wrapped the Table number 'Table 5.40' in SATOPTS in a reference and tried to give it a unique ref-name. This ref-name is then cross-referenced from IMBNUM and KRNUM. I have successfully tested the links in main.fodt.

@blattms
Copy link
Member

blattms commented Jul 23, 2024

Why is that better?
Do the ref-names of tables get changed automatically? If not we could use a more unique scheme and rename them. See links in #54

@gdfldm gdfldm changed the title Modify references to SATOPTS in KRNUM and IMBNUM Fix table references to SATOPTS in KRNUM and IMBNUM Jul 24, 2024
@blattms blattms merged commit 0b05954 into OPM:main Jul 24, 2024
@gdfldm gdfldm deleted the xref-krnum branch July 24, 2024 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants