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

Importing v3 #482

Merged
merged 91 commits into from
Jul 6, 2024
Merged

Importing v3 #482

merged 91 commits into from
Jul 6, 2024

Conversation

northdpole
Copy link
Collaborator

@northdpole northdpole commented Jan 14, 2024

closes #479

TODO:

  • make spreadsheet_parsers.parse_export_format() return the same format as parse_hierarchical_export_format()
  • add parse_export_format test case to test_parse_standards_from_spreadsheeet
  • make base "importer" class that handles gap analysis and embeddings generation with a callback that tells us how to import specific element
  • create an importer method in main that loops through the available importer implementations calls their callbacks and then proceeds to call the generor methods
  • make main.parse_standards_from_spreadsheeet able to monitor job completeness and report progress
  • make main.parse_standards_from_spreadsheeet able to return list of standards that have been imported
  • add tests for parsers()
  • add tests for base parser

@northdpole northdpole self-assigned this Apr 17, 2024
@northdpole northdpole force-pushed the importing-v3 branch 2 times, most recently from 193b441 to ee3fe29 Compare May 31, 2024 08:56
@northdpole northdpole force-pushed the importing-v3 branch 9 times, most recently from 32425a2 to c3ff6bb Compare June 9, 2024 15:52
@northdpole northdpole force-pushed the importing-v3 branch 7 times, most recently from 53b5fb2 to 695f026 Compare June 19, 2024 22:17
@northdpole northdpole force-pushed the importing-v3 branch 7 times, most recently from ac39fa5 to 8041a45 Compare June 26, 2024 09:25
@northdpole northdpole force-pushed the importing-v3 branch 11 times, most recently from 4e9205a to a576808 Compare July 6, 2024 12:42
@northdpole northdpole merged commit a56acf1 into main Jul 6, 2024
4 of 6 checks passed
@northdpole northdpole deleted the importing-v3 branch July 6, 2024 14:25
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.

Importing is currently not user-friendly
2 participants