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

modify ISP pipeline for better image quality #15

Closed
wants to merge 1 commit into from

Conversation

sjbgg
Copy link

@sjbgg sjbgg commented Apr 17, 2023

  1. move the nr2d module before local contrast and sharpen(currently empty), so that final images can get better details
  2. modified requirements.txt, removed specific version, tested fine

1. move the nr2d module before local contrast and sharpen, so that final images can get better details
2. modified requirements.txt, removed specific version, tested fine
@xx-isp
Copy link
Collaborator

xx-isp commented Apr 26, 2023

Thanks @sjbgg. Will test the quality of the new pipeline on a few more images before accepting the PR.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sjbgg the requirements.txt file is generated automatically using the gen_requirements.py file without any manual tweaking. It gives a good reference of the library versions on which the pipeline will certainly run. A user can use this requirements.txt to install the exact versions of libraries on which the pipeline will run using the how-to-run guide.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see. I ran into some problems while installing so I deleted all the version numbers. And it automatically installed the latest version from the server.
I agree the version numbers are good for reference though.

@xx-isp xx-isp linked an issue Apr 26, 2023 that may be closed by this pull request
@bakhtawar-10xe
Copy link
Contributor

PR is outdated now after pipeline Version 1.1 Upgrade

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

Successfully merging this pull request may close these issues.

Pipeline corrction
3 participants