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

Create an Error Message for Incorrect Usage of 'set_parm!' #222

Open
carolmcs opened this issue Jul 15, 2024 · 0 comments
Open

Create an Error Message for Incorrect Usage of 'set_parm!' #222

carolmcs opened this issue Jul 15, 2024 · 0 comments

Comments

@carolmcs
Copy link
Contributor

When using the function set_parm!(db, collection, attribute, element_label, value) providing an attribute that is actually a (sparse) time series, it throws the error

LoadError: Something went wrong. Unknown composite type: PSRClassesInterface.PSRDatabaseSQLite.TimeSeries{Float64}

image

It would be helpful to receive a more instructive error message, for example guiding the user to use the correct method (update_time_series!).

I noticed from the stack trace that there may be missing handling for TimeSeries in the function _string_for_composite_types.

image

In addition, I would appreciate having a method where I can pass the collection and the attribute, and it tells me whether it is a scalar attribute or a time series This would help me write better methods for updating elements.

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

1 participant