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.

Sign up for GitHub

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

Jump to bottom

[dataquery] dataquery module should include all data from CouchDB_Import_Instruments #9372

Open
4 of 6 tasks
driusan opened this issue Sep 30, 2024 · 0 comments
Open
4 of 6 tasks

[dataquery] dataquery module should include all data from CouchDB_Import_Instruments #9372

driusan opened this issue Sep 30, 2024 · 0 comments
Assignees
@driusan
Labels
Area: Instruments PR or issue related instruments Module: conflict_resolver PR or issue related to conflict_resolver module Module: dataquery PR or issue related to (new) dataquery module Module: instruments PR or issue related to instruments module Priority: High PR or issue should be prioritised over others for review and testing Proposal PR or issue suggesting an improvement that can be accepted, rejected or altered
Milestone
27.0.0

Comments

@driusan
Copy link
Collaborator

driusan commented Sep 30, 2024 •
edited
Loading

This will probably need to be split up into different tickets and closed, but for now I'm making one for auditing the fields from the CouchDB Import Instruments script and comparing that the fields are all in the new dataquery tool.

The fields from the script that need to be verified as present are:

  • Administration
  • Data Entry
  • Validity
  • Conflicts Exist
  • DDE Complete
  • (Each field from each instrument)
@driusan driusan added the Proposal PR or issue suggesting an improvement that can be accepted, rejected or altered label Sep 30, 2024
@driusan driusan added the Priority: High PR or issue should be prioritised over others for review and testing label Dec 12, 2024
@driusan driusan mentioned this issue Jan 9, 2025
Merged
@driusan driusan added Module: dataquery PR or issue related to (new) dataquery module Area: Instruments PR or issue related instruments Module: conflict_resolver PR or issue related to conflict_resolver module Module: instruments PR or issue related to instruments module labels Jan 21, 2025
driusan added a commit that referenced this issue Jan 24, 2025
@driusan
[dataquery] Add instrument flags to instruments query engine (#9529)
d908264 
This adds instrument flags to the data that is bulk loaded by
bulkLoadInstanceData and uses that to address the low hanging fruit of
#9372 of including administration flags in the data query tool.

It does not address the DDE and conflict related fields that are in the
CouchDB import script because we should investigate if those make sense
in the instruments query engine or if it's possible to have those
provided by the conflict resolver module rather than the instruments
module, as they are more related to that module.
@driusan driusan self-assigned this Jan 24, 2025
@driusan driusan moved this to Assigned in LORIS RoadMap Feb 4, 2025
@driusan driusan added this to the 27.0.0 milestone Feb 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@driusan driusan

Labels
Area: Instruments PR or issue related instruments Module: conflict_resolver PR or issue related to conflict_resolver module Module: dataquery PR or issue related to (new) dataquery module Module: instruments PR or issue related to instruments module Priority: High PR or issue should be prioritised over others for review and testing Proposal PR or issue suggesting an improvement that can be accepted, rejected or altered
Projects
LORIS RoadMap
Status: Assigned
Milestone
27.0.0
Development

No branches or pull requests
1 participant
@driusan