climix issueshttps://git.smhi.se/groups/climix/-/issues2020-02-14T09:04:12Zhttps://git.smhi.se/climix/climix/-/issues/135better integration of the master table2020-02-14T09:04:12ZLars Bärringbetter integration of the master tableNow when much of the metadata is auto-generated it is useful to bring the "master table" into closer coordination with the climix git.Now when much of the metadata is auto-generated it is useful to bring the "master table" into closer coordination with the climix git.0.11.0Klaus ZimmermannKlaus Zimmermannhttps://git.smhi.se/climix/climix/-/issues/134Storing of parameters is broken2019-11-20T15:08:17ZKlaus ZimmermannStoring of parameters is brokenThe storing of parameters works by converting the parameter to the output units, but
this only is appropriate if by chance the units are compatible. In other cases it crashes
climix.The storing of parameters works by converting the parameter to the output units, but
this only is appropriate if by chance the units are compatible. In other cases it crashes
climix.0.10.0Klaus ZimmermannKlaus Zimmermannhttps://git.smhi.se/climix/climix/-/issues/133remove etc/metadata.yml ix.defs that are in the autogenerated metadata2019-11-20T10:09:50ZLars Bärringremove etc/metadata.yml ix.defs that are in the autogenerated metadata0.10.0Lars BärringLars Bärringhttps://git.smhi.se/climix/climix/-/issues/132Correct handling of units (as per discussion)2023-01-31T16:49:02ZLars BärringCorrect handling of units (as per discussion)Currently the output units is drawn from the user unit column[of the master table]. This is usually, but not not always correct, and irrespective of which it is an *ad hoc* solution that should have a more permanent and correct solution....Currently the output units is drawn from the user unit column[of the master table]. This is usually, but not not always correct, and irrespective of which it is an *ad hoc* solution that should have a more permanent and correct solution. For example the temperature_sum unit should be `Kelvin seconds` (in canonical units) but this currently cannot infer this from the input units, which is `Kelvin`. As discussed, it seems like that intensive quantities and extensive quantities have to be handled differently. Or, more precisely, if input is an intensive quantity and output is an extensive quantity (as is the case for temperature_sum and some of the indices related to precipitation)
We could use something like $`[I]/[O]=c[C]`$, where $`[I]`$ and $`[O]`$ is respectively **input** and **output** units, $`c`$ is a numeric constant having unit $`[C]`$. The input unit is given by the input data file, and the output unit is given by the metadata defining the index.
Now, if $`[C]=1/s`$ the aggregation involves a transformation of an intensive quantity to an extensive quantity, and if $`[C]=s`$ it is the other way around.
[cf #60, #81]https://git.smhi.se/climix/climix/-/issues/131Add a changelog2019-11-13T16:38:00ZKlaus ZimmermannAdd a changelogWe should keep a changelog to keep on top of the developments and to use in the release process.We should keep a changelog to keep on top of the developments and to use in the release process.0.10.0Klaus ZimmermannKlaus Zimmermannhttps://git.smhi.se/climix/climix/-/issues/130Error reporting in the editor component2019-11-11T14:44:09ZKlaus ZimmermannError reporting in the editor componentGenerally, errors in the table are silently ignored by `climix-editor`. This can lead to, eg, missing parameters.
It would be good to add error detection and reporting to the editor to make it a bit more robust.Generally, errors in the table are silently ignored by `climix-editor`. This can lead to, eg, missing parameters.
It would be good to add error detection and reporting to the editor to make it a bit more robust.0.10.0Klaus ZimmermannKlaus Zimmermannhttps://git.smhi.se/climix/climix/-/issues/129Better implementation of spell_length (possibly other spell based indices)2020-03-04T12:52:27ZKlaus ZimmermannBetter implementation of spell_length (possibly other spell based indices)The current implementation of spell_length relies on `da.apply_along_axis`. This is far from optimal and outright problematic when the chunking in data files is badly aligned for this use case.The current implementation of spell_length relies on `da.apply_along_axis`. This is far from optimal and outright problematic when the chunking in data files is badly aligned for this use case.0.13Klaus ZimmermannKlaus Zimmermannhttps://git.smhi.se/climix/climix/-/issues/128Improve reporting for broken indices2019-11-11T09:38:45ZKlaus ZimmermannImprove reporting for broken indicesWith support for multiple metadata files and the increasing number of indices, it would be nice to have better reporting about problems with broken indices.With support for multiple metadata files and the increasing number of indices, it would be nice to have better reporting about problems with broken indices.0.10.0Klaus ZimmermannKlaus Zimmermannhttps://git.smhi.se/climix/climix/-/issues/127Deal with missing long_name in parameters2019-11-11T09:35:05ZKlaus ZimmermannDeal with missing long_name in parametersQuantity parameters are not required to have a long_name. However, at the moment a missing long_name in the index definitions raises an exception. Climix should be able to handle this without a problem.Quantity parameters are not required to have a long_name. However, at the moment a missing long_name in the index definitions raises an exception. Climix should be able to handle this without a problem.0.10.0Klaus ZimmermannKlaus Zimmermannhttps://git.smhi.se/climix/climix/-/issues/126Improve metadata handling2019-11-11T09:30:22ZKlaus ZimmermannImprove metadata handlingCleanup dataclasses based metadata handling.Cleanup dataclasses based metadata handling.https://git.smhi.se/climix/climix/-/issues/125Add single threaded dask setup2019-11-08T13:28:25ZKlaus ZimmermannAdd single threaded dask setupFor debugging purposes, it is useful to have a single-threaded way of setting up dask.For debugging purposes, it is useful to have a single-threaded way of setting up dask.0.10.0Klaus ZimmermannKlaus Zimmermannhttps://git.smhi.se/climix/climix/-/issues/124Fix dask setup to *not* prescribe local directory2019-11-04T15:31:53ZKlaus ZimmermannFix dask setup to *not* prescribe local directoryThe dask setup for local cluster at the moment prescribes a fixed location on a ram disk.
While the use of a ram disk might be beneficial and certainly not all (HPC) file systems are equally suited for the task, the prescription of one p...The dask setup for local cluster at the moment prescribes a fixed location on a ram disk.
While the use of a ram disk might be beneficial and certainly not all (HPC) file systems are equally suited for the task, the prescription of one particular directory is to restrictive and creates problems on shared environments.
For now, let's revert to dask choosing the directory, possibly giving a configuration option later on.0.9.0Klaus ZimmermannKlaus Zimmermannhttps://git.smhi.se/climix/climix/-/issues/123Adapt editor to new table format2019-11-04T15:10:35ZKlaus ZimmermannAdapt editor to new table formatThe index function column in the table no longer contains the `FUN_` prefix and the editor has to be adapted to this.The index function column in the table no longer contains the `FUN_` prefix and the editor has to be adapted to this.0.9.0Klaus ZimmermannKlaus Zimmermannhttps://git.smhi.se/climix/climix/-/issues/122Fix "ready" detection in editor2019-11-04T15:08:06ZKlaus ZimmermannFix "ready" detection in editorSome rows in the table deliver the `ready` column as text instead of number.
These are not detected as ready, even if the text is "1". While this should be fixed in the table,
it would be nice if the editor could deal with it gracefully ...Some rows in the table deliver the `ready` column as text instead of number.
These are not detected as ready, even if the text is "1". While this should be fixed in the table,
it would be nice if the editor could deal with it gracefully as well.0.9.0Klaus ZimmermannKlaus Zimmermannhttps://git.smhi.se/climix/climix/-/issues/121Keep table ordering in editor2019-11-04T14:45:43ZLars BärringKeep table ordering in editor0.9.0Klaus ZimmermannKlaus Zimmermannhttps://git.smhi.se/climix/climix/-/issues/120Improve code style2019-11-04T09:28:58ZKlaus ZimmermannImprove code styleUp to now we have not used a code style checker consistently. This has led to a small amount of code smells.
It would be good to introduce an automatic code style checker and to improve the code accordingly.Up to now we have not used a code style checker consistently. This has led to a small amount of code smells.
It would be good to introduce an automatic code style checker and to improve the code accordingly.Klaus ZimmermannKlaus Zimmermannhttps://git.smhi.se/climix/climix/-/issues/119Add option to deactivate sentry2019-11-04T09:22:28ZKlaus ZimmermannAdd option to deactivate sentryDuring development and debugging it can be useful to avoid sending sentry notes.
For this purpose it would be nice to have an option to deactivate sentry.During development and debugging it can be useful to avoid sending sentry notes.
For this purpose it would be nice to have an option to deactivate sentry.Klaus ZimmermannKlaus Zimmermannhttps://git.smhi.se/climix/climix/-/issues/118Multiple thresholds2023-05-03T13:00:27ZLars BärringMultiple thresholdsMany indices are based on a threshold. For many of these indices it is relevant to have several thresholds E.g. for Summer Days the threshold is tasmax > 25 degC. It is often requested to have alternative thresholds e.g. 21, 23, 25, 27, ...Many indices are based on a threshold. For many of these indices it is relevant to have several thresholds E.g. for Summer Days the threshold is tasmax > 25 degC. It is often requested to have alternative thresholds e.g. 21, 23, 25, 27, 29 degC.
The index name should then be txgtn (TBC).
Cf. #73https://git.smhi.se/climix/climix/-/issues/117Deal with negative parameters in index template names2019-11-08T13:24:45ZKlaus ZimmermannDeal with negative parameters in index template namesBy convention, index names can contain the letter `m` to mark negative parameter as in `tngtm10` to mean `tngt` with `-10` as a parameter. This could probably be handled by matching for both `['tngt', 'm10']` and `['tngtm', '10']`. If th...By convention, index names can contain the letter `m` to mark negative parameter as in `tngtm10` to mean `tngt` with `-10` as a parameter. This could probably be handled by matching for both `['tngt', 'm10']` and `['tngtm', '10']`. If this leads to ambiguity we throw an error.0.10.0Klaus ZimmermannKlaus Zimmermannhttps://git.smhi.se/climix/climix/-/issues/116Possible bug in cdd/cwd (spell_length)2019-10-08T15:15:19ZLars BärringPossible bug in cdd/cwd (spell_length)It seems that the climix results sometimes are is one day shorter than the corresponding NGCD counterparts (altcdd/altcwd), see #102 for details.It seems that the climix results sometimes are is one day shorter than the corresponding NGCD counterparts (altcdd/altcwd), see #102 for details.