beat.core merge requestshttps://gitlab.idiap.ch/beat/beat.core/-/merge_requests2020-06-09T08:09:09Zhttps://gitlab.idiap.ch/beat/beat.core/-/merge_requests/120Move to new images2020-06-09T08:09:09ZSamuel GAISTMove to new imagesThis merge request moves all the test images to the new version created through beat/beat.env.builder>
The use of these images means that the old image enumeration becomes effectively a backup solution.
The test should now take less ti...This merge request moves all the test images to the new version created through beat/beat.env.builder>
The use of these images means that the old image enumeration becomes effectively a backup solution.
The test should now take less time as the label based search is faster.
Using these new images also means that we can archive all the various beat.env.XXX repositories and only use beat/beat.env.builder>.Soft loopsAndré AnjosAndré Anjoshttps://gitlab.idiap.ch/beat/beat.core/-/merge_requests/121Fix database prototype2020-06-18T13:58:35ZSamuel GAISTFix database prototypeThe database prototype setup method has a wrong signature and is used in beat/beat.editor> when creating a new Database asset.
This merge request fixes it as well as add the call to the default implementation which is mandatory for ever...The database prototype setup method has a wrong signature and is used in beat/beat.editor> when creating a new Database asset.
This merge request fixes it as well as add the call to the default implementation which is mandatory for everything to be setup properly.
The setup method has been added to two Views in the test prefix so that its use is also tested.
Fixes #97Soft loopshttps://gitlab.idiap.ch/beat/beat.core/-/merge_requests/122Add optionnal environment entry to database2020-06-26T10:11:18ZSamuel GAISTAdd optionnal environment entry to databaseThis merge request add support for defining the database environment to use in the database declaration.
This allows the BEAT infrastructure to no depend on list of database supported by a given environment and allows for more flexibili...This merge request add support for defining the database environment to use in the database declaration.
This allows the BEAT infrastructure to no depend on list of database supported by a given environment and allows for more flexibility to test new databases locally especially when the use of Docker is required when running for example C++ algorithms.
Part of #64Soft loopshttps://gitlab.idiap.ch/beat/beat.core/-/merge_requests/123Ensure that writing to a non existing output shows an error2020-07-08T15:29:37ZSamuel GAISTEnsure that writing to a non existing output shows an errorThe test added by this merge request ensure that writing to an output that does not exists is reported.
The error will be that "NoneType" as no property "write" which is expected following how outputs work.
Part of #99The test added by this merge request ensure that writing to an output that does not exists is reported.
The error will be that "NoneType" as no property "write" which is expected following how outputs work.
Part of #99Soft loopsSamuel GAISTSamuel GAISThttps://gitlab.idiap.ch/beat/beat.core/-/merge_requests/124Add test for setup in cxx algorithm2020-07-21T11:07:52ZSamuel GAISTAdd test for setup in cxx algorithmCurrently only the processing part during an experiment run was tested.
This new test ensures that the setup feature works properly.Currently only the processing part during an experiment run was tested.
This new test ensures that the setup feature works properly.André AnjosAndré Anjoshttps://gitlab.idiap.ch/beat/beat.core/-/merge_requests/125Add minimal rw tmpfs for /tmp and /run2020-07-29T09:18:52ZSamuel GAISTAdd minimal rw tmpfs for /tmp and /runAt least in production, some issues happened that required
to have /tmp writable. To preserve the read-only state
of the container, use a small tmpfs that is enough for
the container to run. /run is another that might be needed
so make i...At least in production, some issues happened that required
to have /tmp writable. To preserve the read-only state
of the container, use a small tmpfs that is enough for
the container to run. /run is another that might be needed
so make it part of the defaults used.
Fixes #101Soft loopsFlavio TARSETTIFlavio TARSETTIhttps://gitlab.idiap.ch/beat/beat.core/-/merge_requests/126Pre commit cleanup2020-08-06T08:14:38ZSamuel GAISTPre commit cleanupThis merge request updates the pre-commit configuration and applies a full source cleanup.This merge request updates the pre-commit configuration and applies a full source cleanup.https://gitlab.idiap.ch/beat/beat.core/-/merge_requests/127Update docker related information2020-09-02T11:18:10ZSamuel GAISTUpdate docker related informationFollowing the release of BEAT runtime images on Docker hub and beat/beat.env.examples>, update the documentation related to the use of Docker.Following the release of BEAT runtime images on Docker hub and beat/beat.env.examples>, update the documentation related to the use of Docker.André AnjosAndré Anjoshttps://gitlab.idiap.ch/beat/beat.core/-/merge_requests/128Algorithm database sharing2020-11-10T10:34:52ZSamuel GAISTAlgorithm database sharingImplement a way to mount the database folder on the algorithms containers.
Fixes #103Implement a way to mount the database folder on the algorithms containers.
Fixes #103André AnjosAndré Anjoshttps://gitlab.idiap.ch/beat/beat.core/-/merge_requests/129Sphinx 3 autodoc no longer accepts autodoc_default_flags and autodoc_default_...2020-11-10T13:07:53ZAmir MOHAMMADISphinx 3 autodoc no longer accepts autodoc_default_flags and autodoc_default_options should be usedhttps://gitlab.idiap.ch/beat/beat.core/-/merge_requests/130Add missing global parameters to shared_datasets experiment2020-11-10T13:59:01ZSamuel GAISTAdd missing global parameters to shared_datasets experimentWhile missing parameters in globals will not affect the execution, it will be detected by beat/beat.editor>.While missing parameters in globals will not affect the execution, it will be detected by beat/beat.editor>.Samuel GAISTSamuel GAISThttps://gitlab.idiap.ch/beat/beat.core/-/merge_requests/131Fix algorithm name pattern in experiment globals2020-11-12T12:20:40ZSamuel GAISTFix algorithm name pattern in experiment globalsIt's missing support for - in the author part of
algorithm author name.
Fixes #104It's missing support for - in the author part of
algorithm author name.
Fixes #104Flavio TARSETTIFlavio TARSETTIhttps://gitlab.idiap.ch/beat/beat.core/-/merge_requests/132Refactor database sharing2020-11-16T16:48:00ZSamuel GAISTRefactor database sharingThis refactoring moves the database sharing responsibility to the database declaration.
The author can choose whether the dataset shall be made available directly to the algorithm container.
As a consequence, the implementation done in...This refactoring moves the database sharing responsibility to the database declaration.
The author can choose whether the dataset shall be made available directly to the algorithm container.
As a consequence, the implementation done in beat/beat.cmdline!103 will not work anymore.
Fixes #105André AnjosAndré Anjoshttps://gitlab.idiap.ch/beat/beat.core/-/merge_requests/133Custom tmpfs through label2020-11-30T08:50:01ZSamuel GAISTCustom tmpfs through labelThis merge request implements a new feature handling additional tmpfs mount points provided through image label.
The goal here is to allow images with special needs to provide them through a well defined format.
Fixes #106This merge request implements a new feature handling additional tmpfs mount points provided through image label.
The goal here is to allow images with special needs to provide them through a well defined format.
Fixes #106André AnjosAndré Anjoshttps://gitlab.idiap.ch/beat/beat.core/-/merge_requests/134Fix protocole template creation when migrating a database2021-01-29T16:55:46ZSamuel GAISTFix protocole template creation when migrating a databaseThe wrong name was used to generate the template.
Also improve handling of protocol template double creation.
If the content is different warn the user and ask
whether it should continue with the database creation.The wrong name was used to generate the template.
Also improve handling of protocol template double creation.
If the content is different warn the user and ask
whether it should continue with the database creation.André AnjosAndré Anjoshttps://gitlab.idiap.ch/beat/beat.core/-/merge_requests/135Fix generated protocol templates2021-01-29T17:38:21ZSamuel GAISTFix generated protocol templatesThe original work done for the protocol template was working however it has been discovered that the helper script did not correctly generate the protocol templates.
The script has been fixed in !134 however the test data were not.
Thi...The original work done for the protocol template was working however it has been discovered that the helper script did not correctly generate the protocol templates.
The script has been fixed in !134 however the test data were not.
This merge request fixes that. As a byproduct of this cleanup, it has been found that the large database was in fact wrongly written and thus has been fixed as well.André AnjosAndré Anjoshttps://gitlab.idiap.ch/beat/beat.core/-/merge_requests/136Cleanup empty fields in v2 databases2021-02-03T08:30:02ZSamuel GAISTCleanup empty fields in v2 databasesWhile they are optional, beat/beat.editor> ensures that
you do not get useless empty fields that may look
like something is missing hence remove them from
here to ensure testing goes well.While they are optional, beat/beat.editor> ensures that
you do not get useless empty fields that may look
like something is missing hence remove them from
here to ensure testing goes well.https://gitlab.idiap.ch/beat/beat.core/-/merge_requests/137Add pyproject.toml2021-04-13T07:54:35ZSamuel GAISTAdd pyproject.tomlThis follows bob packages new standardThis follows bob packages new standardFlavio TARSETTIFlavio TARSETTIhttps://gitlab.idiap.ch/beat/beat.core/-/merge_requests/138Gracefully handle docker client creation error2021-04-13T12:14:31ZSamuel GAISTGracefully handle docker client creation errorUp until now, the client creation did not fail when Docker daemon was not running but the behaviour seems to have changed on macOS.
In any case, ensuring that the client creation failure does follow the rest of the logic is the right ...Up until now, the client creation did not fail when Docker daemon was not running but the behaviour seems to have changed on macOS.
In any case, ensuring that the client creation failure does follow the rest of the logic is the right thing to do.
Fixes #107
Fixes beat/beat.editor#282Flavio TARSETTIFlavio TARSETTIhttps://gitlab.idiap.ch/beat/beat.core/-/merge_requests/139Ignore version.txt for end of file fixing2021-04-15T13:06:24ZSamuel GAISTIgnore version.txt for end of file fixingThis file is modified automatically and thus will fail each time after a new release.This file is modified automatically and thus will fail each time after a new release.Samuel GAISTSamuel GAIST