Skip to content

Conversation

niemela
Copy link
Member

@niemela niemela commented Sep 1, 2025

Fixes #462.

I don't love the structure of the text... but is this the semantics we want? I think so.

Copy link
Collaborator

@Matistjati Matistjati left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Not strictly related, but it's IMO not unambiguous enough whether a nested folder structure is allowed or not. I would really like to see this clarified in the text.

@niemela
Copy link
Member Author

niemela commented Sep 3, 2025

Looks good. Not strictly related, but it's IMO not unambiguous enough whether a nested folder structure is allowed or not. I would really like to see this clarified in the text.

Agreed. Create an issue for it? @Matistjati

@niemela niemela merged commit e264111 into master Sep 3, 2025
@niemela niemela deleted the clarify-data-structure branch September 3, 2025 08:57
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.

Clarify requirement on structure under /data/invalid_output and /data/valid_output
2 participants