Please remove database configurations

and move them to bob.bio.db.

Working on it

Hi,

I'm moving the config files from bob.bio.(face, spear, vein) to bob.bio.db. What is the best course of action with the entry points definition? Shall I move them too to bob.bio.db??

I'm asking this because, bob.bio.base has the script resources.py that "prints a detailed list of all resources that are registered" for a particular bob.bio.* package. This script returns a nice print of the supported databases.

I have two possible solutions for this one:

1. Keep the entry point definition (not the configs) in the target packages (bob.bio.*). This will keep this nice feature as it is today.
2. Move them to bob.bio.db. For this we have to think in another hierarchy to define the database entry points if we want to keep this feature.

I recommend the first option. Works nice.

What do you think?

Thanks

What is the best course of action with the entry points definition? Shall I move them too to bob.bio.db?

Yes. Please move them to bob.bio.db too. This makes maintaining them a lot easier.

We want to patch resources.py to list only the available ones. So if an entry point fails to load, we issue a warning when --details is provided.

Ok,

We should open a new issue to keep track of this. If I merge the code right now, we will lose the feature.

Are you sure this is the best way to approach this task? In my opinion it is better to list only the database resources supported by a particular package. The effort of maintenance will be minimum (only the setup.py of each bob.bio.* package).

I just don't want to lose this feature which was recently created (bob.bio.base#25 (closed)):

./bin/resources.py --types d --package bob.bio.face

The way that I implemented the output is the following.

List of registered databases:

- bob.bio.face 2.0.6b0 @ /remote/idiap.svm/user.active/tpereira/gitlab/bob/bob.bio.face: 
  + arface             --> bob.bio.db.default_configs.arface: database
  + banca              --> bob.bio.db.default_configs.banca_english: database
  + caspeal            --> bob.bio.db.default_configs.caspeal: database
  + frgc               --> bob.bio.db.default_configs.frgc: database
  + gbu                --> bob.bio.db.default_configs.gbu: database
  + lfw-restricted     --> bob.bio.db.default_configs.lfw_restricted: database
  + lfw-unrestricted   --> bob.bio.db.default_configs.lfw_unrestricted: database
  + mobio-female       --> bob.bio.db.default_configs.mobio_female: database
  + mobio-image        --> bob.bio.db.default_configs.mobio_image: database
  + mobio-male         --> bob.bio.db.default_configs.mobio_male: database
  + multipie           --> bob.bio.db.default_configs.multipie: database
  + multipie-pose      --> bob.bio.db.default_configs.multipie_pose: database
  + scface             --> bob.bio.db.default_configs.scface: database
  + xm2vts             --> bob.bio.db.default_configs.xm2vts: database

As you can see, everything is pointing to bob.bio.db.

I think what you are suggesting could work too. Would you be willing to do this for the rest of the databases that are already ported too?

Sure, no problem

Putting the entry points in one place had this advantage of having unique names for databases. Before we had a couple of entry points with the same name in different packages.

mentioned in commit c988740d

This issue is outdated. Will close it

Status changed to closed