-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Pre-trained models? #1
Comments
@zuru No, there are no pre-trained models for this implementation. I cannot reproduce the paper's performance and am still figuring out what bug is responsible for the performance gap. Thank you for pointing out about the incompatibility of the weight decay. |
Setting weight decay to
indicate something is not there yet. The resulting corner maps though are noisy and corner detection does not behave very well. Please let me know if you manage to identify the issue and reproduce the results. |
@zuru Hi zuru, I have converted the pretrained weights from the original CFL to PyTorch and also implemented the model that can reproduce the results in the paper during inference. (there are a few differences in the metrics, they are close) The models have TFCFL in their names. I'll also put the link to the pretrained weights in README. Now, I still have not figured the exact training strategy used by Clara et.al so the training performance still does not come close to the inference performance. |
Hello, are there any pre-trained models available for the PyTorch implementation?
If not, has this training implementation been verified to (closely?) reproduce the paper's results with the default hyperparameters (i.e. train for 100 epochs with a batch size of 4)?
A known issue is the incompatibility between PyTorch's and TensorFlow's weight decay which I suspect will influence results compatibility.
The text was updated successfully, but these errors were encountered: