Skip to content
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

error while reproducing #3

Closed
insanemate033-gif opened this issue Oct 21, 2024 · 21 comments
Closed

error while reproducing #3

insanemate033-gif opened this issue Oct 21, 2024 · 21 comments
Assignees

Comments

@insanemate033-gif
Copy link

module not found error for cvdm
Screenshot 2024-10-19 at 11 30 30 PM

@nanoxas
Copy link
Collaborator

nanoxas commented Oct 21, 2024

Hi, yes we are aware of this error. The eval script currently does not work as we are updating the code to improve usability.

@nanoxas nanoxas self-assigned this Oct 21, 2024
@insanemate033-gif
Copy link
Author

hi, can you give update on this, i just want to run for super resolution.

@nanoxas
Copy link
Collaborator

nanoxas commented Oct 25, 2024

I'm going to work on this specific issue next week. I will close the issue whenever the problem is fixed.

@insanemate033-gif
Copy link
Author

insanemate033-gif commented Oct 25, 2024 via email

@nanoxas
Copy link
Collaborator

nanoxas commented Oct 25, 2024

I'm assuming you downloaded BioSR. What needs to be done is first fix the data loader (you can change this in the train.py script and load the data there, skipping the data loader). For this experiment, the parameter diff_inp needs to be set to true. And for the evaluation (the eval script) needs to be experiment-specific as it is currently set up for an experiment that is not related to super-resolution. The module issue is a pathing issue that might be fixed by changing the imports.

Again, I will fix this next week, as I don't have the time for it today. Please be patient.

@nanoxas
Copy link
Collaborator

nanoxas commented Oct 28, 2024

The issue should be fixed now. Please read the updated readme and prepare the environment accordingly. Download the BioSR data first, update the config file, train the model using the train script, and use the eval script over the test set to obtain the results.

@nanoxas nanoxas closed this as completed Oct 28, 2024
@insanemate033-gif
Copy link
Author

insanemate033-gif commented Oct 29, 2024 via email

@nanoxas
Copy link
Collaborator

nanoxas commented Oct 29, 2024

Then, you are missing the tensorflow module. Please see the updated instructions.

@insanemate033-gif
Copy link
Author

insanemate033-gif commented Oct 29, 2024 via email

@insanemate033-gif
Copy link
Author

insanemate033-gif commented Nov 1, 2024 via email

@nanoxas
Copy link
Collaborator

nanoxas commented Nov 1, 2024

Hi, it was just a dependency issue. I updated the requirements file. It should be fixed now.

@insanemate033-gif
Copy link
Author

insanemate033-gif commented Nov 4, 2024 via email

@insanemate033-gif
Copy link
Author

insanemate033-gif commented Nov 4, 2024 via email

@nanoxas
Copy link
Collaborator

nanoxas commented Nov 4, 2024

I don't know what could be wrong I installed the environment from scratch and it worked for me :(

@nanoxas nanoxas reopened this Nov 4, 2024
@nanoxas nanoxas closed this as completed Nov 4, 2024
@insanemate033-gif
Copy link
Author

insanemate033-gif commented Dec 12, 2024 via email

@insanemate033-gif
Copy link
Author

insanemate033-gif commented Dec 12, 2024 via email

@insanemate033-gif
Copy link
Author

insanemate033-gif commented Dec 19, 2024 via email

@nanoxas
Copy link
Collaborator

nanoxas commented Dec 19, 2024

Sorry, I can't help you here because what you are having is a pathing issue. Check if you updated the code to the last version. If you did the class that python is missing should be in that folder, what you need to do is fix the pathing of the project such that python finds the missing class.

@insanemate033-gif
Copy link
Author

insanemate033-gif commented Dec 22, 2024 via email

@ayakimovich
Copy link
Collaborator

Actually I m only focusing on the application part I want the results from my datasets, that is why I want to reproduce this, i am not able to update the code as there is no folder biosr_dataloader. Can you just look it on my system through any desk so it would work

On Thu, Dec 19, 2024, 5:21 PM Gabriel della Maggiora < @.> wrote: Sorry, I can't help you here because what you are having is a pathing issue. Check if you updated the code to the last version. If you did the class that python is missing should be in that folder, what you need to do is fix the pathing of the project such that python finds the missing class. — Reply to this email directly, view it on GitHub <#3 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/BBQXTAM4MEO4H22DDRUZWHL2GKXKVAVCNFSM6AAAAABQKR5MN2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNJTGUZDENJXGI . You are receiving this because you were mentioned.Message ID: @.>

Hi @insanemate033-gif we thank you for your interest, however what you are requesting goes beyond the scope of the open source code license. Furthermore, I think you are misusing the issues functionality here. What you are raising is not an "issue" as you are not providing any steps to reproduce it.

Unless you can provide a concrete description and steps to reproduce an issue, this thread should be closed. Requests to adopt the codebase to your data cannot constitute an issue. Unless you follow the policy I'm going to report your activity.

@nanoxas
Copy link
Collaborator

nanoxas commented Dec 22, 2024

Hi @insanemate033-gif. I provided the dataloader and added the pathing for you. I tested the code in a new environment, and it trained as expected (with the biosr.npz). Please see if this fixes your issue. If you want to train with your own data, you will have to create the dataloader yourself.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants