Init weights

[acerioni]

No description provided.

[GwenaelleSa]

A few WIP comments left.
The new parameter in the config files should be described in the readme of the repo.

[cleherny]

Do you want me to make the change here? Or do you?
Anyway, I'd like to modify the post-processing script for the quarry example to make it consistent with the proj-dqry. So I can make the change directly in this branch if it's easier.

[acerioni]

I've suggested an updated. If accepted, edits to the configuration files can be discarded.

[acerioni]

I suggest to update as follows:

INIT_MODEL_WEIGHTS = cfg['model_weights'].get('init_model_weights', False)

which would set INIT_MODEL_WEIGHTS to False by default, should the parameter not be provided by the config file. This would allow us to avoid updating all the existing configuration files.

[GwenaelleSa]

It was getting messy, so I added to the mess by doing the modifications I asked for and approving the PR.
@cleherny Thank you for your proposition, but I think we can merge this PR much faster than the bigger one on the DQRY example, so it's worth to keep this separated.
@acerioni You comment brought a good idea. I implemented it the same way we did for other optional arguments of the OD, meaning with a if-else statement rather than the get method. The result should be the same if I understood you correctly.

I don't merge because it's not very catholic knowing I was the last to modify this branch, but if @acerioni or @cleherny you agree with the modification, feel free to do it.

[stdl-s-sonarqube]

Analysis Details

1 Issue

• 0 Bugs
• 0 Vulnerabilities
• 1 Code Smell

Coverage and Duplications

• No coverage information (0.00% Estimated after merge)
• 0.00% Duplicated Code (0.00% Estimated after merge)

Project ID: swiss-territorial-data-lab_object-detector_AYZ4zWIzr7JdaaSXwexc

View in SonarQube

[acerioni]

Thanks @GwenaelleSa for your latest edits 👍