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

SPEC-ACTIONS in hsp2 #90

Open
21 of 26 tasks
rburghol opened this issue Jun 8, 2022 · 4 comments
Open
21 of 26 tasks

SPEC-ACTIONS in hsp2 #90

rburghol opened this issue Jun 8, 2022 · 4 comments

Comments

@rburghol
Copy link
Contributor

rburghol commented Jun 8, 2022

Overview

This issue is set up to plan and track development for a proposed "SPEC-ACTIONS" module in hsp2. Development has 2 main components:

  1. Provide support for pre-hsp2 uci files that use SPEC-ACTIONS
  2. Provide an enhanced "next generation" type implementation for SPEC-ACTIONS

Tasks

Reference

@PaulDudaRESPEC

@PaulDudaRESPEC
Copy link
Member

@rburghol , this looks like a great outline for getting started! We look forward to collaborating!

@aufdenkampe
Copy link
Collaborator

@rburghol, I second @PaulDudaRESPEC's appreciation for creating this issue!

It's great to see your interest for implementing these Special Action capabilities! Do you and your team have plans to dive in, get started, and/or issue a pull request?

The RESPEC/LimnoTech collaborative team just got another round of funding to continue developing HSP2, but we won't be getting started with that work until September. Addressing SpecialActions is in our scope of work, but we had been thinking of first prioritizing additional code refactoring to move all existing modules to object-oriented class structures (similar to what we did in late 2021 for RQUAL modules). So it would be helpful to better understand your needs to evaluate whether we should tweak priorities.

cc: @ptomasula & @tredder75

@rburghol
Copy link
Contributor Author

rburghol commented Jun 8, 2022

@aufdenkampe thanks! My ambition is to get started asap -- but be smart about the approach to minimize refactoring all over again as your work on SA evolves.

My groups main need is for advanced water supply operational code, mostly on RCHRES. If RCHRES execution is already objectified that will be great. That said, I'll be writing objects, so I should be able to easily adapt to whatever your evolving framework looks like.

Tentative Schedule:

  • I am going to go heavy on outlines/checklists for the next week, as well as assembling a dataset for testing. Comments will be appreciated.
  • I will post some proposed base classes for water supply system/reservoir related actions based on direct translations of what we currently use for comment by the end of June. If no one has time to comment, no worries, changes is constant and it's open source, I'll be writing code that can adapt to the infrastructure.
  • By early July I hope to be prototyping on our local branch/fork.
  • I will develop on my groups fork, with a goal for a pull request whenever you all solidify the framework and I can tie things in nicely. Our dev branch: https://github.com/HARPgroup/HSPsquared

I will look at your RQUAL stuff for an inkling about design goals, any links that you can share to point me in the right direction would be awesome. Also, forward on any test datasets that you may have if you have some in mind.

@rburghol
Copy link
Contributor Author

rburghol commented Sep 2, 2022

Hey @aufdenkampe and @PaulDudaRESPEC -- wanted to let you know I have begun working in earnest on the special actions. Started with the parser for the most basic SA, and tracking that progress here #99.

  • there is a downloadable test dataset
  • a link to my groups repo if anyone wanted to examine the code
  • I have identified all the places in the code that need to get work to support parsing
  • I have put together a basic parser support for standard actions though the data parsing info is off so I am not getting good info yet.

I won't do a pull request till something is actually working though I am making a good bit of progress, but if anyone wants to take a look that would be awesome since I assume I am making a silly mistake of some sort in the parser CSV def or code.

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

3 participants