Skip to content

Conversation

jdcourcol
Copy link
Contributor

The immediate need is to store things like:
grafik

We want to store the result of the execution of the validation and the generated assets for display in the GUI.

The PR does not address:

  • the definition of the validation itself (code, what is the validation for, description...), which should be a separate entity.
  • the provenance (when was the validation executed, who executed it ...), which should follow standard model with activity, contributors, agents...
  • other complications like "score"
  • change over time: newer execution results may show up, invalidating the previous ones.

To be added later:

  • the schema migration, REST api, import and tests

@AurelienJaquier
Copy link
Contributor

I don't see any entry for the generated figure in your ValidationResult model. Would the figure be stored in the ValidationResult, or would it be stored elsewhere?

@jdcourcol
Copy link
Contributor Author

The figure is stored as an asset. Entities can have assets (usually a file).

@AurelienJaquier
Copy link
Contributor

Ok, I see. I have another question. I see that the ValidationResult contains a link to the validated entity. Would the validation result show in the validated entity, like the annotations? Or do we have to manually add an annotation into the validated entity linking to the ValidationResult?

@jdcourcol
Copy link
Contributor Author

no, there is no annotation to add. The UI can fetch all the ValidationResult pointing to the displayed entity.

@chr-pok
Copy link
Contributor

chr-pok commented May 16, 2025

Overall looks good to me!

Just one additional item would be required that is outlined in General_schema.docx, namely some kind of logging information for the user. The idea is to store additional details on the validation run, e.g., if the validation was skipped, if there were any warnings, why the validation failed etc. Otherwise, it will be difficult for a user to know how to fix the artifact in case the validation failed.

I imagine this could be either another asset containing a log file, or better an optional string with (human-readable) details on the validation run. We called it notes in the docx but perhaps validation_details would be better.

@jdcourcol
Copy link
Contributor Author

yes, that would be another asset indeed.

@AurelienJaquier
Copy link
Contributor

I have modified the General_schema.docx to rename notes into validation_details, and make it an asset

@dkeller9
Copy link
Contributor

@jdcourcol some validation results will be in the format of key: value, where the value is of either boolean or a number. Should we make these results as an asset or is it better to have a dedicated structure in entitycore?

@jdcourcol jdcourcol requested a review from dkeller9 May 19, 2025 09:06
@dkeller9
Copy link
Contributor

I am worried that if we add this in now without handling version and provenance it will be hard to migrate in the future. I would suggest to have a meeting on Tuesday to discuss this.

@jdcourcol
Copy link
Contributor Author

yes, but we can add that when we face it. A good reference would be https://sciunit.readthedocs.io/en/latest/basics.html#score

@jdcourcol
Copy link
Contributor Author

I am worried that if we add this in now without handling version and provenance it will be hard to migrate in the future.

Can you elaborate why ?

@dkeller9
Copy link
Contributor

If version and provenance information is not present then what would be migrated? Do we just assume that what's in there is the base version and that all validations have been run on it? If so that's fine but we need to be sure to rerun the validations any time things change.

@jdcourcol
Copy link
Contributor Author

For the time being, we are going to import the existing validation results stored in the knowledge graph.

@jdcourcol jdcourcol merged commit adcae06 into main May 21, 2025
1 check passed
@jdcourcol jdcourcol deleted the validation_result branch May 21, 2025 13:41
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.

5 participants