Skip to content

Conversation

@rburghol
Copy link
Contributor

@rburghol rburghol commented Jan 2, 2026

This is a prototype to allow an update of settings ONLY from a UCI into an hdf5 that has already been created with hsp2 import_uci ....

  • Copies parameters over, but omits timeseries data like WDM, mutsin ...
  • This results in a huge time savings when testing, and I can only imagine that if I were developing a model that was UCI based, I would be very grateful for this feaure.
  • I consider this ready to merge as it DOES work initially, but caveats follow.
  • This works well the first time, but subsequent attempts fail when trying to run due to duplicate column names that occur due to renaming in the function readUCI()
  • See this issue for details on the problems with this function after the 2nd update attempt: Bug: readUCI() renames columns before setting defaults, but does not flush to hdf5, which creates duplicate columns #201

print("Note: this will NOT update external data such as WDM, mutsin etc.")
print("To update all data, use the command 'hsp2 import_uci ...' ")
readUCI(ucifile, h5file, False)
with open(ucifile) as fp:
Copy link
Contributor

Choose a reason for hiding this comment

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

Since you aren't reading any time-series from the WDM files you don't need to read the UCI file to get the WDM file names. Just need "readUCI(ucifile, h5file, False)"

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.

2 participants