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

Release workflow: simplify upper level parsing and release #516

Open
jannahastings opened this issue Oct 26, 2023 · 2 comments
Open

Release workflow: simplify upper level parsing and release #516

jannahastings opened this issue Oct 26, 2023 · 2 comments
Assignees

Comments

@jannahastings
Copy link
Contributor

Currently parsing and releasing the BCIO upper level consists of several steps, including that we parse the BCIO Upper Defs spreadsheet and merge the content with the spreadsheet of extracted relations from LucidChart to create the BCIO Upper Level Merged spreadsheet. However, there is a bug in this process in that the merged file does not contain the correct status flags as are specified the original Upper Defs spreadsheet. In addition, the LucidChart upper levle visualisation is not editable by all team members. Thus, to simplify the release workflow, suggest that the Upper Level Merged file be corrected and then the Upper Defs file be removed. Thereafter, users can edit the Upper Level Merged file directly including maintaining status and relationships.

@b-gehrke
Copy link
Contributor

Currently, the fields included in the BCIO Upper level Merged spreadsheet are hardcoded to BCIO_ID, Name, Parent, Definition, Synonyms, Examples - plus relations. Therefore, all of the following fields defined in BCIO_Upper_Defs.xlsx are not included:

Logical definition, Definition source, Curator note, Comment, Curation status

@jannahastings Are the others expected to be missing or should they be included as well?

@jannahastings
Copy link
Contributor Author

Thanks for spotting this! The others are not expected to be missing. We should include them as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants