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

Feedback ontology loading #482

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

Feedback ontology loading #482

wants to merge 11 commits into from

Conversation

dariober
Copy link
Contributor

No description provided.

garrettjstevens and others added 9 commits November 12, 2024 03:48
In `packages/apollo-mst/src/AnnotationFeatureModel.ts` we set the
session onject to type `any` and ignore the warnings and ignore warning

Add test file so_types.gff3 with unconventional types.
* The UI shows the job as still in progress even after the importing is done.
  This is because (?) we don't have a `jobsManager.done(job)` in packages/jbrowse-plugin-apollo/src/session/ClientDataStore.ts

* We should show how far we are from completing the file import.
@garrettjstevens
Copy link
Contributor

@dariober I think I've fixed the issue where the ontology was always loaded in the test. The meta section of the pre-loaded indexeddb databases didn't match what was expected, so it always tried to re-load it. I added the missing meta.storeOptions.prefixes to the json files and fixed the import of those files so it transformed it from an object to a map. That made it match what Apollo was expecting, so now it doesn't try to re-load the ontologies every time.

@dariober
Copy link
Contributor Author

dariober commented Jan 7, 2025

@shashankbrgowda This PR should notify the user that an ontology (sequence or gene ontology) is being loaded. It should show a progress bar in the "Running job" window (see screenshot below). If the ontology is already loaded then nothing should appear.

image

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.

2 participants