bob issueshttps://gitlab.idiap.ch/groups/bob/-/issues2022-05-12T13:36:56Zhttps://gitlab.idiap.ch/bob/bob.measure/-/issues/67rocch and calibration implementations are missing2022-05-12T13:36:56ZAmir MOHAMMADIrocch and calibration implementations are missingsince we removed bob.math in https://gitlab.idiap.ch/bob/bob.measure/-/merge_requests/106since we removed bob.math in https://gitlab.idiap.ch/bob/bob.measure/-/merge_requests/106The Great Deprecationhttps://gitlab.idiap.ch/bob/bob.bio.face/-/issues/80Cannot specify original directory and extension for most of the databases any...2022-05-25T09:16:16ZManuel Günthersiebenkopf@googlemail.comCannot specify original directory and extension for most of the databases anymoreWhile in previous database implementations, it was relatively straightforward to utilize the database interface in order to load pre-extracted features, in the new database interfaces this is no longer possible, for two reasons:
1. Ther...While in previous database implementations, it was relatively straightforward to utilize the database interface in order to load pre-extracted features, in the new database interfaces this is no longer possible, for two reasons:
1. There is no simple way of providing the database interface with the directory where to read the data from. For example, in an old interface (LFW), we can still set:https://gitlab.idiap.ch/bob/bob.bio.face/-/blob/c7ee7213f83f62b1e36685290e1defd10fea2c20/bob/bio/face/database/lfw.py#L90, while this option does no longer exist in newer interfaces: https://gitlab.idiap.ch/bob/bob.bio.face/-/blob/c7ee7213f83f62b1e36685290e1defd10fea2c20/bob/bio/face/database/scface.py#L30, although it should be relatively straightforward to implement that since the default value is used just a few lines below: https://gitlab.idiap.ch/bob/bob.bio.face/-/blob/c7ee7213f83f62b1e36685290e1defd10fea2c20/bob/bio/face/database/scface.py#L47. It should be simple to re-expose this option to the user.
2. The filename extension is by default empty: https://gitlab.idiap.ch/bob/bob.bio.face/-/blob/c7ee7213f83f62b1e36685290e1defd10fea2c20/bob/bio/face/database/scface.py#L50, and there is no possibility to change that on the constructor. But even if we would expose the extension similarly to the directory name, we would still be in trouble since the default loader just **appends** the extension rather than ** replacing** it: https://gitlab.idiap.ch/bob/bob.bio.base/-/blob/406f2da1faadacd4d4fe4e36e5a0010d78557513/bob/bio/base/database/csv_dataset.py#L145
The main issue with 2. is that someone has decided to ignore our old behavior to store keys without filename extension, and just added the original extension to the key. For example, running:
```
import bob.bio.face
db = bob.bio.face.database.SCFaceDatabase(protocol="far")
samples = db.all_samples()
print(samples[0].key)
```
will print `filename.JPG` instead of `filename` (without extension).
So, what I would propose is (the least amount of changes, anything else would require to re-create all the CSV-based databases):
- [ ] Expose the `original_directory` and `original_extension` parameters for all databases to the user, keeping their default values as they currently are.
- [ ] Change the line in https://gitlab.idiap.ch/bob/bob.bio.base/-/blob/406f2da1faadacd4d4fe4e36e5a0010d78557513/bob/bio/base/database/csv_dataset.py#L145 to replace the extension with the given one (if one is given) rather than appending it to the filename.
Any objection?https://gitlab.idiap.ch/bob/bob.bio.base/-/issues/180It is not possible to run experiments without dask wrapper2022-04-26T12:06:41ZAmir MOHAMMADIIt is not possible to run experiments without dask wrapperNot using dask is useful especially when you want to debug your code. With the current implementation, this is not possible.Not using dask is useful especially when you want to debug your code. With the current implementation, this is not possible.Amir MOHAMMADIAmir MOHAMMADIhttps://gitlab.idiap.ch/bob/nightlies/-/issues/61bob.ip.binseg Tests hang, Job Failed #2655822022-05-12T13:37:25ZAmir MOHAMMADIbob.ip.binseg Tests hang, Job Failed #265582Job [#265582](https://gitlab.idiap.ch/bob/nightlies/-/jobs/265582) failed for e03c39bce11c74e60373f99d6098d3bf7f75c4b6:
Looking at macOS builds, most probably the hanging test is:
```
ip/binseg/test/test_cli.py::test_experiment_stare_wi...Job [#265582](https://gitlab.idiap.ch/bob/nightlies/-/jobs/265582) failed for e03c39bce11c74e60373f99d6098d3bf7f75c4b6:
Looking at macOS builds, most probably the hanging test is:
```
ip/binseg/test/test_cli.py::test_experiment_stare_with_multiprocessing PASSED [ 58%]
```André AnjosAndré Anjoshttps://gitlab.idiap.ch/bob/bob.pad.base/-/issues/43The documentation of the file list database is outdated2022-05-04T13:26:04ZTiago de Freitas PereiraThe documentation of the file list database is outdatedDuring the workshop, we spotted the documentation of the file list database interface (https://www.idiap.ch/software/bob/docs/bob/bob.pad.base/master/vanilla_pad_features.html) is outdated.
It doesn't match with what is currently implem...During the workshop, we spotted the documentation of the file list database interface (https://www.idiap.ch/software/bob/docs/bob/bob.pad.base/master/vanilla_pad_features.html) is outdated.
It doesn't match with what is currently implemented.
ping @ydayer @amohammadiYannick DAYERYannick DAYERhttps://gitlab.idiap.ch/bob/bob.devtools/-/issues/96Issue with dask distributed and click >=8.1.02022-06-27T14:26:59ZTiago de Freitas PereiraIssue with dask distributed and click >=8.1.0We need to bump our dask, distributed versions because of this issue.
https://github.com/dask/distributed/pull/6018
ping @hotroshiWe need to bump our dask, distributed versions because of this issue.
https://github.com/dask/distributed/pull/6018
ping @hotroshihttps://gitlab.idiap.ch/bob/bob.io.base/-/issues/23Drop ALL C++ code from this code.2022-04-25T15:23:49ZTiago de Freitas PereiraDrop ALL C++ code from this code.This package (**in practical terms, which means the features we use the most**) has three base functionalities.
1. With the support of `bob.io.image` it reads and writes images (bob.io.base.load and bob.io.base.save)
2. Reads and writes...This package (**in practical terms, which means the features we use the most**) has three base functionalities.
1. With the support of `bob.io.image` it reads and writes images (bob.io.base.load and bob.io.base.save)
2. Reads and writes HDF5 files with matrices with the command `bob.io.base.load` and `bob.io.base.save`.
3. Support to read and write complex HD5 files
I'll rewire the bob.io.base API to
1. use imageio for reading and writing image formats (solving 1);
2. use h5py for reading and writing hdf5 files (solving 2);
3. rewrite pieces of code that use the complex HD5 files using h5py (solving issue 3).
Sounds reasonable enough?
ping @amohammadi @ydayer @flavio.tarsetti @mguenther @lcolboisThe Great Deprecationhttps://gitlab.idiap.ch/bob/bob.fusion.base/-/issues/11Can we archive this package?2022-04-28T05:50:20ZTiago de Freitas PereiraCan we archive this package?The last activity here was in July 2020 (https://gitlab.idiap.ch/bob/bob.fusion.base/-/commit/1e3a80315285c13a1570615b919398d412e51edb).
All the other commits on top of this one were maintenance commits.
Is there any project benefiting ...The last activity here was in July 2020 (https://gitlab.idiap.ch/bob/bob.fusion.base/-/commit/1e3a80315285c13a1570615b919398d412e51edb).
All the other commits on top of this one were maintenance commits.
Is there any project benefiting from this?
Thankshttps://gitlab.idiap.ch/bob/bob.bio.base/-/issues/179Pass post_process_samples, biometric_references, and probe_samples together2022-05-25T07:04:24ZYu LinghuPass post_process_samples, biometric_references, and probe_samples togetherIt would be better to have a function that let the `post_processor` choose what samples it needs, i.e. pass `post_process_samples`, `biometric_references`, and `probe_samples` into the function together regardless of Z-norm or T-norm, in...It would be better to have a function that let the `post_processor` choose what samples it needs, i.e. pass `post_process_samples`, `biometric_references`, and `probe_samples` into the function together regardless of Z-norm or T-norm, instead of pass specific samples according to the `post_processor`.
![Screenshot_2022-04-13_095243](/uploads/c0723480507a0f31767228cd878014e5/Screenshot_2022-04-13_095243.png)Yu LinghuYu Linghuhttps://gitlab.idiap.ch/bob/bob.fusion.base/-/issues/10NIghtlies failing because of this one2022-04-14T14:23:53ZTiago de Freitas PereiraNIghtlies failing because of this onehttps://gitlab.idiap.ch/bob/nightlies/-/jobs/264456
Does anyone uses this package?https://gitlab.idiap.ch/bob/nightlies/-/jobs/264456
Does anyone uses this package?https://gitlab.idiap.ch/bob/bob.pad.face/-/issues/44Nightlies failing because of this one2022-04-12T18:10:12ZTiago de Freitas PereiraNightlies failing because of this one....... so much work....... so much workhttps://gitlab.idiap.ch/bob/nightlies/-/issues/60Nightlies failing because of this one2022-04-12T16:27:56ZTiago de Freitas PereiraNightlies failing because of this one....... so much work....... so much workhttps://gitlab.idiap.ch/bob/bob.bio.face/-/issues/79background samples in LFW2022-04-13T15:08:39ZHatef OTROSHIbackground samples in LFWHi,
As there is no background sample in LFW, it seems that the `background_model_samples` needs to be an empty list, but it is `[0]` now in [bob/bio/face/database/lfw.py#L339](https://gitlab.idiap.ch/bob/bob.bio.face/-/blob/master/bob/...Hi,
As there is no background sample in LFW, it seems that the `background_model_samples` needs to be an empty list, but it is `[0]` now in [bob/bio/face/database/lfw.py#L339](https://gitlab.idiap.ch/bob/bob.bio.face/-/blob/master/bob/bio/face/database/lfw.py#L339)
ping @tiago.pereiraTiago de Freitas PereiraTiago de Freitas Pereirahttps://gitlab.idiap.ch/bob/bob.pad.base/-/issues/42NIghtlies failing because of this one2022-04-12T07:39:58ZTiago de Freitas PereiraNIghtlies failing because of this onehttps://gitlab.idiap.ch/bob/nightlies/-/jobs/264177
It seems a sphinx error.https://gitlab.idiap.ch/bob/nightlies/-/jobs/264177
It seems a sphinx error.https://gitlab.idiap.ch/bob/bob.devtools/-/issues/95Issue with Twine + python 3.92022-04-10T15:33:16ZTiago de Freitas PereiraIssue with Twine + python 3.9The command `bdt ci readme ....` is consistently failing on linux with python 3.9
https://gitlab.idiap.ch/bob/bob.extension/-/jobs/264169
https://gitlab.idiap.ch/bob/bob.bio.video/-/jobs/264064
https://gitlab.idiap.ch/bob/bob.bio.face/-...The command `bdt ci readme ....` is consistently failing on linux with python 3.9
https://gitlab.idiap.ch/bob/bob.extension/-/jobs/264169
https://gitlab.idiap.ch/bob/bob.bio.video/-/jobs/264064
https://gitlab.idiap.ch/bob/bob.bio.face/-/jobs/264052
This command basically does a sanity check on the `long_description` field of the `setup.py`.
In our case, this field is filled with the content of README.rst.
For this check, the command uses a library called `readme_renderer`, which calls a function from `bleach`
Well, in short, there is an API issue `readme_renderer` and this breaks **all our builds**
```
File "/scratch/builds/bob/bob.bio.video/miniconda/lib/python3.9/site-packages/twine/commands/check.py", line 121, in check
warnings, is_ok = _check_file(filename, render_warning_stream)
File "/scratch/builds/bob/bob.bio.video/miniconda/lib/python3.9/site-packages/twine/commands/check.py", line 92, in _check_file
rendering_result = renderer.render(
File "/scratch/builds/bob/bob.bio.video/miniconda/lib/python3.9/site-packages/readme_renderer/rst.py", line 114, in render
return clean(rendered)
File "/scratch/builds/bob/bob.bio.video/miniconda/lib/python3.9/site-packages/readme_renderer/clean.py", line 72, in clean
cleaner = bleach.sanitizer.Cleaner(
TypeError: __init__() got an unexpected keyword argument 'styles'
```
There is wrong pinning of bleach on `readme_renderer`.
This can be solved by updating the `readme_renderer` version from `conda-forge`, which currently is 27.0, but they already have a tagged the version 34.0 (https://github.com/pypa/readme_renderer/releases/tag/34.0)https://gitlab.idiap.ch/bob/bob.bio.video/-/issues/23Nightlies failing because of this one2022-04-10T10:41:06ZTiago de Freitas PereiraNightlies failing because of this onehttps://gitlab.idiap.ch/bob/nightlies/-/jobs/263945https://gitlab.idiap.ch/bob/nightlies/-/jobs/263945Tiago de Freitas PereiraTiago de Freitas Pereirahttps://gitlab.idiap.ch/bob/bob.db.base/-/issues/29Moving this function `read_annotation_file` to bob.bio.base....2022-04-19T08:59:07ZTiago de Freitas PereiraMoving this function `read_annotation_file` to bob.bio.base....any objection?
https://gitlab.idiap.ch/bob/bob.db.base/-/blob/master/bob/db/base/annotations.py#L37
ping @amohammadi @ydayer
FYI: @mguenther
Another step to deprecate this package.any objection?
https://gitlab.idiap.ch/bob/bob.db.base/-/blob/master/bob/db/base/annotations.py#L37
ping @amohammadi @ydayer
FYI: @mguenther
Another step to deprecate this package.https://gitlab.idiap.ch/bob/bob.bio.spear/-/issues/39Follow-up from "Port GMM Voxforge baseline to dask pipelines"2022-04-05T09:39:20ZAmir MOHAMMADIFollow-up from "Port GMM Voxforge baseline to dask pipelines"The following discussion from !50 should be addressed:
- [ ] @amohammadi started a [discussion](https://gitlab.idiap.ch/bob/bob.bio.spear/-/merge_requests/50#note_72609):
> we need to write down why we cache audio files here. I do...The following discussion from !50 should be addressed:
- [ ] @amohammadi started a [discussion](https://gitlab.idiap.ch/bob/bob.bio.spear/-/merge_requests/50#note_72609):
> we need to write down why we cache audio files here. I don't remember anymore.Yannick DAYERYannick DAYERhttps://gitlab.idiap.ch/bob/bob.bio.spear/-/issues/38Follow-up from "Port GMM Voxforge baseline to dask pipelines"2022-04-05T09:39:20ZAmir MOHAMMADIFollow-up from "Port GMM Voxforge baseline to dask pipelines"The following discussion from !50 should be addressed:
- [ ] @amohammadi started a [discussion](https://gitlab.idiap.ch/bob/bob.bio.spear/-/merge_requests/50#note_72597):
> bob.db.atnt is deprecatedThe following discussion from !50 should be addressed:
- [ ] @amohammadi started a [discussion](https://gitlab.idiap.ch/bob/bob.bio.spear/-/merge_requests/50#note_72597):
> bob.db.atnt is deprecatedYannick DAYERYannick DAYERhttps://gitlab.idiap.ch/bob/bob.ip.base/-/issues/15Deprecation2022-04-26T11:49:18ZTiago de Freitas PereiraDeprecationHi guys,
As you may know, we are on a deprecation wave of C++ code, and this image processing library is implemented mostly in this language.
Follow bellow a list of functionalities this package has.
Does anyone **uses and wants to kee...Hi guys,
As you may know, we are on a deprecation wave of C++ code, and this image processing library is implemented mostly in this language.
Follow bellow a list of functionalities this package has.
Does anyone **uses and wants to keep** some of its features?
Please let me know, by marking with a cross, as soon as possible.
@amohammadi, @ydayer, @flavio.tarsetti, @lcolbois, @sebastien.marcel, @mguenther, @andre.anjos, @ageorge
# Classes
- [x] bob.ip.base.GeomNorm ---> I have a pure python counterpart
- [x] bob.ip.base.FaceEyesNorm ---> I have a pure python counterpart
- [x] bob.ip.base.LBP
- [ ] bob.ip.base.LBPTop
- [ ] bob.ip.base.DCTFeatures
- [ ] bob.ip.base.TanTriggs
- [ ] bob.ip.base.Gaussian
- [ ] bob.ip.base.WeightedGaussian
- [ ] bob.ip.base.GradientMagnitude
- [ ] bob.ip.base.BlockNorm
- [ ] bob.ip.base.HOG
- [ ] bob.ip.base.GLCMProperty
- [ ] bob.ip.base.GLCM
# Functions
- [ ] bob.ip.base.flip
- [ ] bob.ip.base.flop
- [ ] bob.ip.base.crop
- [ ] bob.ip.base.shift
- [ ] bob.ip.base.scale
- [ ] bob.ip.base.scaled_output_shape
- [ ] bob.ip.base.rotate
- [ ] bob.ip.base.rotated_output_shape
- [ ] bob.ip.base.angle_to_horizontal
- [ ] bob.ip.base.block
- [ ] bob.ip.base.block_output_shape
- [ ] bob.ip.base.block_generator
- [ ] bob.ip.base.crop
- [ ] bob.ip.base.shift
- [ ] bob.ip.base.extrapolate_mask
- [ ] bob.ip.base.max_rect_in_mask
- [ ] bob.ip.base.angle_to_horizontal
- [ ] bob.ip.base.histogram
- [ ] bob.ip.base.lbphs
- [ ] bob.ip.base.lbphs_output_shape
- [ ] bob.ip.base.histogram_equalization
- [ ] bob.ip.base.gamma_correction
- [ ] bob.ip.base.integral
- [ ] bob.ip.base.zigzag
- [ ] bob.ip.base.median
- [ ] bob.ip.base.sobel