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.

Sign up for GitHub

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

Jump to bottom

Update TransformationRules.md #99

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update TransformationRules.md #99

wants to merge 1 commit into from

Conversation

jsaligoe
Copy link

@jsaligoe jsaligoe commented Oct 12, 2019 •
edited
Loading

  1. For consideration: updates to MT rule titles for clarity/consistency
  2. Minor edits for grammar and consistency, e.g., “Alternative” vs “Alternate”
  3. Updates to MT property list to be consistent with MT Rule documents
  4. Added proposed rules MT00X that will be added under "Issues" for discussion and consideration

@jsaligoe
Update TransformationRules.md
6201e25 
1. For consideration: updates to MT titles for clarity/consistency
2. Minor edits for grammar and consistency, e.g., “Alternative” vs “Alternate”
3. Updates to MT property list to be consistent with MT Rule documents
4. Added proposed rules MT00X that will be added for discussion and consideration (file names TBD)
jsaligoe added a commit to jsaligoe/2017.2 that referenced this pull request Oct 13, 2019
@jsaligoe
Update GeneralFlattening.md
bf3296b 
Proposes modification to address that this rule can be combined with other rules to avoid information loss. 
1. For consideration: update to title for consistency/clarity as it applies specifically to nested structures; updates names of other MT rules as proposed in [pull request 99](INSPIRE-MIF#99)
2. Minor edits for grammar, clarity, and consistency
3. Included language referring to MT00X (PROPOSED) “Retain Nested Child Elements” and MT00X (PROPOSED) “Abbreviate Prefix of Compound Property Names,” should they be accepted.
@jsaligoe jsaligoe mentioned this pull request Oct 13, 2019
Open
heidivanparys added a commit to heidivanparys/model-transformation-rules that referenced this pull request Jan 22, 2023
@heidivanparys
Update the overview of rules, make rule descriptions consistent
977396f 
Changes for consistency:
- add identifier to every rule description
- make the titles in the description of the rules consistent with the
overview that describes all properties of the rules
- use "simplification rule" and "substition rule" as categories
- editorial changes

Some of these changes were suggested in
INSPIRE-MIF/2017.2#99
@heidivanparys heidivanparys mentioned this pull request Jan 22, 2023
Merged
heidivanparys added a commit to INSPIRE-MIF/model-transformation-rules that referenced this pull request Jan 22, 2023
@heidivanparys
Update the overview of rules, make rule descriptions consistent
62f5700 
Changes for consistency:
- add identifier to every rule description
- make the titles in the description of the rules consistent with the
overview that describes all properties of the rules
- use "simplification rule" and "substition rule" as categories
- editorial changes

Some of these changes were suggested in
INSPIRE-MIF/2017.2#99
@heidivanparys
Copy link
Contributor

The changes proposed have been implemented in the new repository dedicated to model transformation rules, see also INSPIRE-MIF/model-transformation-rules@62f5700. Therefore, this pull request can be closed.

Note that the proposed rules are not added in the overview yet, they should be approved first.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jsaligoe @heidivanparys