Comments (4)
The addition of these options are a great improvement and I know the PR #137 is merged already, but how are we supposed to easily know the name of the json dump to use it as input for
i.sentinel.mask
? Do we need to browse in the mapset files or does it get printed wheni.sentinel.import
finishes?
I agree that this needs to be documented.
The underlying idea is that the history written by r.support is just textual (even quite limited in length and troublesome with the linebreaks) and cannot be easily machine-parsed. Hence additional JSON support was added to store the values in a better way.
Effectively, with -j
a JSON file is written out for each band into LOCATION/MAPSET/json/BANDRASTERNAME.json
, so nothing to fiddle with manually.
The advantage is that the user doesn't have to carry around the MTD_MSIL2A.xml
from the zip'ed Sentinel-2 SAFE file as the relevant values are simply stored in the GRASS GIS data structure under json/
.
This can then be fed to e.g. i.sentinel.mask
as metadata
parameter:
OK, now we need to make it a text snippet to be added to the manual... :)
from grass-addons.
Effectively, with
-j
a JSON file is written out for each band intoLOCATION/MAPSET/json/BANDRASTERNAME.json
, so nothing to fiddle with manually.
@neteler @AnikaBettge from the source code it seems that json file are saved by default to LOCATION/MAPSET/cell_misc/MAP_NAME/description.json
from grass-addons.
@lucadelu yes you are right. I changed the description in #339
from grass-addons.
The underlying idea is that the history written by r.support is just textual (even quite limited in length and troublesome with the linebreaks) and cannot be easily machine-parsed. Hence additional JSON support was added to store the values in a better way.
This makes sense. LOCATION/MAPSET/cell_misc/MAP_NAME/description.json
stated in a subsequent comment seems like a good choice.
Effectively, with
-j
a JSON file is written out for each band intoLOCATION/MAPSET/json/BANDRASTERNAME.json
, so nothing to fiddle with manually.
So why not to create that always and get rid of the -j
flag and the trouble of documenting that (esp. since it is a very not standard behavior in GRASS).
As long a the path/name of that JSON file is good, the rest is relatively easy to make it backward/forward compatible: Just add a version
to the top level. Then you can just say it is a new, perhaps experimental, part of the db/loc/mapset structure and threat that as anything else. Possibly add more modules or functionality which deal with that directly such as r.support. If you decide the contents/format is wrong, you just add checks for the version and change what is needed. If you are lucky, many changes in JSON will be just backward compatible automatically and if the filename contains extension (.json
) then you could even change the format, although that seems unlikely here and now.
If you want to discuss this on mailing list, feel free to copy-paste what I just wrote.
from grass-addons.
Related Issues (20)
- [Bug] i.fusion.hpf can longer be installed with g.extension HOT 5
- v.db.pyupdate warning during installation
- [Feat] i.sentinel.import: handle processing baselines HOT 3
- [Bug] Add concurrency groups/cancel in progress for skipping outdated CI runs
- Add tests to i.modis, i.landsat, i.sentinel download HOT 23
- Addons for plotting: Standardize parameters and flags HOT 3
- [Bug] r.accumulate segfaults without specifying accumulation_type
- Dependency Dashboard
- [Bug] Installing v.kriging fails HOT 2
- [Bug] r.fuzzy.set rounds shape/converts parameter decimal value to 0 HOT 6
- [Bug] v.surf.nnbathy: NameError: `StandardError` is not defined HOT 8
- Add per-test timeout for grass.gunittest tests
- Require the tests to pass in CI
- v.rast.move: empty keywords break addon overview script HOT 2
- [Bug] i.sentinel.download: give warning instead of errors if single files are corrupt
- v.surf.mass: Input should be vector "area" map? HOT 1
- [Bug] testsuite: do not run tests for deprecated module HOT 4
- [Bug] m.tnm.download and m.cdo.download fail to build on Windows server due to using external Python library requests
- r.stream.distance elevations with value of -1 [Bug]
- [Bug] Remove r.sim.water.mp from addons
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from grass-addons.