Opened 16 years ago
Closed 15 years ago
#341 closed enhancement (fixed)
Better project/setup_model interface for users
Reported by: | rwilson | Owned by: | James Hudson |
---|---|---|---|
Priority: | normal | Milestone: | |
Component: | Architecture and API | Version: | |
Severity: | minor | Keywords: | |
Cc: |
Description (last modified by )
We need to think about refactoring the code users change to run ANUGA. This is files like project.py, setup_model.py, etc.
Currently the code to put ASC elevations into ANUGA looks like:
convert_dem_from_ascii2netcdf(project.demname, ...)
dem2pts(project.demname, ...)
domain = create_domain_...(...)
domain.set_quantity('elevation', project.demname+'.pts', ...)
Hidden away under all that is the assumption that the filename passed to dem2pts() is just a root and '.dem' is *assumed* as the extension. This leads to confusion as:
- project.demname is used all over as a root filename, no real names used,
- sometimes extensions are supplied to a function with the filename, sometimes not.
We should always pass full filenames into functions. If there is an output file, the name for that should be supplied also. This will remove some confusion for beginners as to what is happening.
Additionally, it may now be possible to get ANUGA code to *automatically* convert files. If we say that a file is an "input elevation" file, then ANUGA can convert an ASC file to a PTS file just by looking at the extension (not a real good idea, but simple) or by some 'file'-like magic (do 'man file').
Similarly, a consistent function naming convention would be nice. In runcairns.py we see this to convert an ASC file to a DEM file:
convert_dem_from_ascii2netcdf(project.demname, ...)
dem2pts(project.demname, ...)
This could look like:
asc2dem(project.demname, ...)
dem2pts(project.demname, ...)
or
convert_asc_to_dem(project.demname, ...)
convert_dem_to_pts(project.demname, ...)
I general, a rethink on the 'user' code more from the aspect of 'a user just trying to run ANUGA' instead of an 'ANUGA developer' is called for.
Change History (7)
comment:1 Changed 16 years ago by
Description: | modified (diff) |
---|
comment:2 Changed 16 years ago by
Description: | modified (diff) |
---|
comment:3 Changed 16 years ago by
Description: | modified (diff) |
---|
comment:4 Changed 16 years ago by
Description: | modified (diff) |
---|
comment:5 Changed 16 years ago by
Description: | modified (diff) |
---|
comment:6 Changed 15 years ago by
Owner: | changed from ole to James Hudson |
---|
comment:7 Changed 15 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
Fixed in rev:7814. File extensions are enforced, and naming conventions for functions is consistent.