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

Add preliminary extension exclusions. #285

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

balhoff
Copy link
Member

@balhoff balhoff commented Oct 27, 2022

@tmushayahama can you check that you can update your python script to find these tags in order to avoid creating entries in the JSON file for these lines?

@tmushayahama
Copy link

Hi @balhoff I was trying to parse the shex using the PyShex but the exclude form extensions is not in annotations or exprs. Do you need to define it.

@tmushayahama
Copy link

@balhoff yes this is working @vanaukenk @ukemi you can add these extensions exclusions . However, lets make a dev branch and play with shex there. What do you think? is main branch shex editing okay?

tagging @kltm

@kltm
Copy link
Member

kltm commented Nov 18, 2022

At some point, we'll want to have a development branch (or branches, to try new things and work with new data) and a production branch that is meant to be stable for working with release data. As we're not doing much with ShEx and production data at the moment, we can be flexible, but we'll likely eventually want to make a distinction.

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

Successfully merging this pull request may close these issues.

3 participants