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

Treating several alternative TS definitions #133

Open
olejandro opened this issue Oct 12, 2023 · 5 comments
Open

Treating several alternative TS definitions #133

olejandro opened this issue Oct 12, 2023 · 5 comments

Comments

@olejandro
Copy link
Member

Currently the tool fails if several alternative ts definitions are present.

@olejandro
Copy link
Member Author

@siddharth-krishna would it be a good idea to stop execution and ask the user to provide a single ts definition instead?

@Antti-L
Copy link

Antti-L commented Oct 12, 2023

@olejandro Does VEDA support several alternative ts definitions? I didn't know that; Is there an example of using such somewhere? I know scenario-specific ~Timeslices tables, but I think only one table should be included in any model run? So, maybe you mean just that: selecting scenarios to be included in a model run?

@olejandro
Copy link
Member Author

olejandro commented Oct 12, 2023

Thanks @Antti-L. Yes, I believe we are writing about the same issue. Tbh, I don't know what Veda would do if several scenario files each with it's own ts definition were included in a scenario group...
The tool requires only one such definition to be present in a dataset; it fails when there is more than one.
The tool already accepts passing a list of files to be processed (as an alternative to processing all the files in a folder). So maybe the tool should advise the user to use this functionality to include a single ts definition?

@Antti-L
Copy link

Antti-L commented Oct 12, 2023

Yes, agree, I think it should advice the user about that funcionality if such an error occurs.

@olejandro
Copy link
Member Author

On the other hand, it really depends on the use case. What I write above probably makes sense, if the user intends to run the model. It doesn't make sense if the intention of the user is to use the data in another UI, e.g. TIMES MIRO App that has scenario management capabilities. 🤔

@olejandro olejandro changed the title Treating several alternative TS definition Treating several alternative TS definitions Oct 12, 2023
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

No branches or pull requests

2 participants