-
Notifications
You must be signed in to change notification settings - Fork 74
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
MAXIM Denoising not working as intended? #1289
Comments
I think there's a few things to try:
That'll help start narrowing down where the issue is. |
Hey there,
|
Appreciate the testing, and sorry you're running into this. Let's try a different tack. Try cloning this repo (https://github.com/thekevinscott/upscaling-noise-bug), running The model is the same on browser and node, so given that it's working in the browser, I would expect it to be identical in Node, which would imply some sort of issue in your code. However, I don't see any problems in the code you originally shared. If the linked repo works, then we can figure out what's different between your code and the linked repo; if it doesn't, well then, I've no idea! (In that case, please share your Node version, hardware configuration, etc. as maybe that'll provide a clue.) |
Unfortunately, this didn't work. After running
I've tried about anything (reinstalling, updating node, updating npm, rebuilding), pretty much anything in similar topics like this, or this, or this one. Node version: V20.9.0 P.S If you're wondering why the path in the error is "upscaling-test" rather than "upscaling-noise-bug", according to one of the possible solutions, creating it in a new folder, and rebuilding the entire project in there could help. |
Just to clarify, if you do something like:
It gives you the above issue? (I'm unclear whether the error originates in TFJS or in Upscaler). If it's an issue with TFJS, it's probably above my pay grade and better asked on the TFJS repo. FWIW I also ran into a bunch of problems getting the latest TFJS Node to build, but I'm on a Mac and it was related to Rosetta not playing nicely with the arm build. Doesn't sound like what you're seeing. You could try going back to the original repo you posted with, use the installed |
Hey,
So I've recently started using additional models (aside from the upscaling).
Specifically, the MAXIM Denoising model for node.js
For me however, the only noise it removes is like 0.5% (not even joking.
For reference;
This is obviously the example shown, with the Input/Ouputand here's my Output after utilising the code shown below;
Any idea why, and more importantly, how to avoid/fix this?
The text was updated successfully, but these errors were encountered: