Hyekang

[ivanzvonkov]

@joos2010kj Please only add the files which should change for the forecaster training to work. As far as I am aware all the .dvc, .gitignore, data/*, scripts/data files are not necessary for this PR.

[ivanzvonkov]

I think it's good practice with PRs to add a description when possible

[ivanzvonkov]

@joos2010kj I still see all the .dvc files in here

[joos2010kj]

@joos2010kj I still see all the .dvc files in here

@ivanzvonkov Sorry, can you point out where you see it? I don't see any here: https://github.com/nasaharvest/crop-mask/tree/hyekang

[ivanzvonkov]

@joos2010kj I still see all the .dvc files in here

@ivanzvonkov Sorry, can you point out where you see it? I don't see any here: https://github.com/nasaharvest/crop-mask/tree/hyekang

@joos2010kj you've removed them. See https://github.com/nasaharvest/crop-mask/pull/59/files

[ivanzvonkov]

@joos2010kj What works within this branch currently?

[joos2010kj]

@joos2010kj I still see all the .dvc files in here

@ivanzvonkov Sorry, can you point out where you see it? I don't see any here: https://github.com/nasaharvest/crop-mask/tree/hyekang

@joos2010kj you've removed them. See https://github.com/nasaharvest/crop-mask/pull/59/files

Yeah that's what I thought. Then what did you mean by "I still see all the .dvc files in here"?

[joos2010kj]

@joos2010kj What works within this branch currently?

Sorry, could you elaborate on this question? What do you mean by what works?

[ivanzvonkov]

@joos2010kj What works within this branch currently?

Sorry, could you elaborate on this question? What do you mean by what works?

I was wondering about what part the code worked till when you ran the train script, but you answered during the call. All good.