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

Provide VTD index where possible during import #371

Closed
twagoo opened this issue Sep 27, 2023 · 1 comment
Closed

Provide VTD index where possible during import #371

twagoo opened this issue Sep 27, 2023 · 1 comment
Labels
Milestone

Comments

@twagoo
Copy link
Member

twagoo commented Sep 27, 2023

Where field value is determined in logic, pass the vtd index where applicable. This is helpful for curation.

@twagoo twagoo added this to the VLO 4.12 milestone Sep 27, 2023
@twagoo twagoo modified the milestones: VLO 4.13, VLO 4.12 Feb 27, 2024
@twagoo
Copy link
Member Author

twagoo commented Jul 4, 2024

adding VTD context via an extra parameter to the addDocField* methods of CMDIData cannot be implemented in a trivial way, since the importer first collects all values per facet (FacetProcessor.processFacets()) and only then consolidates these into a field. The values can be derived from any number of paths in the source document so there is no straightforward relation between value and origin at the moment of storage.

@twagoo twagoo added the wontfix label Jul 4, 2024
@twagoo twagoo closed this as completed Jul 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant