-
Notifications
You must be signed in to change notification settings - Fork 4
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
Change smrf.data class naming conventions #171
Comments
I think this follows the line of #163 as the class names are not all that useful. For example, air temperature is in |
We can combine these two tickets. Only logged this separately as it looks like there could be some bigger restructuring of the load data logic once you open that can of worms. I usually follow the pattern to put class names inside the |
As SMRF is growing with more and more subpackages, having class names in the |
The files under smrf.data should be updated to match their classes they hold. Also rename the
grid
class, so we don't end up with references ofgrid.GRID
The text was updated successfully, but these errors were encountered: