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

DCEG & FC: FairwayAggregation requires two Fairway features #148

Open
DavidGrant-NIWC opened this issue Apr 8, 2024 · 2 comments
Open
Labels
DCEG Issues/Proposals for changes to the S-101 DCEG. Post-Edition 2.0.0 S-101 Associations Issue with modelling or implementation of S-101 Assocuiations

Comments

@DavidGrant-NIWC
Copy link

Multiplicity of the binding to Fairway should be 2:∞ rather than 1:∞.

image

image

image

@JeffWootton
Copy link
Collaborator

Am concerned that, where a Fairway System and associated Fairway Aggregation is split at ENC dataset boundaries such that only one instance of a Fairway feature is included in the dataset, the dataset will fail a related validation check checking for at least 2 instances of Fairway to be included in a Fairway Aggregation association.

@DavidGrant-NIWC
Copy link
Author

Am concerned that, where a Fairway System and associated Fairway Aggregation is split at ENC dataset boundaries such that only one instance of a Fairway feature is included in the dataset, the dataset will fail a related validation check checking for at least 2 instances of Fairway to be included in a Fairway Aggregation association.

  • Good point and something I hadn't considered. In that case, 1,* makes sense, but you may want to note the reasoning somewhere.
  • There is still a problem because the current DCEG requires 0,1 and the FC requires 0,*.
  • Also, the reference to the aggregation should be to clause 25.7 instead of 25.13

image

image

@JeffWootton JeffWootton added DCEG Issues/Proposals for changes to the S-101 DCEG. Post-Edition 2.0.0 S-101 Associations Issue with modelling or implementation of S-101 Assocuiations labels Aug 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DCEG Issues/Proposals for changes to the S-101 DCEG. Post-Edition 2.0.0 S-101 Associations Issue with modelling or implementation of S-101 Assocuiations
Projects
None yet
Development

No branches or pull requests

3 participants
@DavidGrant-NIWC @JeffWootton and others