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

#1825 define and subclass Illumina sequencer serieses #1828

Merged

Conversation

turbomam
Copy link
Contributor

@turbomam turbomam commented Oct 3, 2024

closes

@jamesaoverton some multi-quotes got messed up in this PR too. If you can show me how to fix one, then I can do the rest.

cc @aclum

@turbomam turbomam linked an issue Oct 3, 2024 that may be closed by this pull request
@turbomam
Copy link
Contributor Author

turbomam commented Oct 3, 2024

and

java -jar build/robot.jar --prefix "REO: http://purl.obolibrary.org/obo/REO_" merge \
--input src/ontology/obi-edit.owl \
template \
--template src/ontology/templates/devices.tsv \
annotate \
--ontology-iri "http://purl.obolibrary.org/obo/obi/dev/modules/devices.owl" \
--output src/ontology/modules/devices.owl
MANCHESTER PARSE ERROR the expression ('has part' some container) at row 3, column 17 in table "src/ontology/templates/devices.tsv" cannot be parsed: encountered 'container', but expected one of:
	Class name
	( ... )
	{ ... }
	Self

@jamesaoverton
Copy link
Contributor

All of the dark green quotation marks in the diff have been added by your editor, and are changing the semantics of the table. There's no point in debugging the ROBOT command until they're removed.

image

On the last PR, I used a text editor to manually remove them. People usually use Excel for editing these tables. I like https://www.visidata.org/. I haven't used LibreOffice in many years, so I don't know how it behaves.

@turbomam
Copy link
Contributor Author

turbomam commented Oct 4, 2024

thanks. I get the sense that the ROBOT template error might be resolved when I fix the quotations marks

@turbomam
Copy link
Contributor Author

turbomam commented Oct 7, 2024

Thanks @jamesaoverton @mgiglio99 @sebastianduesing

I think I have implemented all of the suggestions from today's meeting

@turbomam turbomam requested a review from jamesaoverton October 8, 2024 15:00
@jamesaoverton
Copy link
Contributor

jamesaoverton commented Oct 10, 2024

OBI is strict about lowercase labels, so I changed "Series Instrument" to "series instrument" throughout. Is that ok with you @turbomam?

If so, I'll update the devices.owl and then merge this.

@turbomam
Copy link
Contributor Author

OBI is strict about lowercase labels, so I changed "Series Instrument" to "series instrument" throughout. Is that ok with you @turbomam?

If so, I'll update the devices.owl and then merge this.

Yes! Thank you.

@jamesaoverton
Copy link
Contributor

Discussed on the OBI call 2024-10-21. We will release #1827, then merge this and make a new release candidate.

@jamesaoverton jamesaoverton merged commit 26c29ce into master Oct 22, 2024
1 check passed
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.

Add grouping classes for series of DNA sequencers
2 participants