weecology
[WIP] Distribute config file from primary location
We had two copies of deepforest_config.yml. The first was the main copy in the root directory of the repo. The second was a copy that got distributed with the package. This is error prone and unnecessary since we can package directly from the root file.
Unless I'm misunderstanding #531 I think this should address it.
This is currently broken because we're not actually testing against the installed package, we're testing against the source code, which doesn't have the data included. This is part of the same problem as #562. I'm going to open an issue to discuss the general solution
This relates to a broader conversation that I would love to move to hydra, add argparse support for config. This is a 2.0 kinda thing, there may be a more short term patch here for the immediate need.
Actually there's probably some cleanup in here that is still worth including so reopening to check if it still needs to be added
@bw4sz & @henrykironde - this should finally complete our move to src/deepforest by reducing us to a single deepforest_config.yml. I did end up having to move it out of root, so I moved it to src/deepforest/deepforest_config.yml which I think makes sense. All other changes are in service of this move. I'd like you both to sign off on this one in case there's something weird about that file move that I failed to consider.
And @henrykironde - once this is merged lets go ahead and roll a new release.
I've rebased this again. @henrykironde can we please get this merged once the tests pass and roll a new release.