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

First bunch of opencl 46 suggestions #598

Merged
merged 2 commits into from
Dec 12, 2023

Conversation

jenshannoschwalm
Copy link
Contributor

  • preferences
  • setting up

@elstoc had messed up my authentication stuff ...

@jenshannoschwalm
Copy link
Contributor Author

BTW - have just seen that you included lens vignetting and the backthumbs crawler stuff already. Do you have any questions about that or other things i left undocumented?

@elstoc
Copy link
Contributor

elstoc commented Dec 6, 2023

If you could look over the stuff I've done and let me know if it needs anything more that'd be great. If there's anything more that's undocumented it's because I haven't started looking at it yet, so I don't yet know if I have questions.

@jenshannoschwalm
Copy link
Contributor Author

No, I checked my to-do list for updated dtdocs and there is only opencl stuff left out.

@elstoc
Copy link
Contributor

elstoc commented Dec 6, 2023

Are you expecting to push some changes to the performance tuning section as well, to cover these alterations and those in darktable-org/darktable#15166?

@jenshannoschwalm
Copy link
Contributor Author

Are you expecting to push some changes to the performance tuning section as well

Absolutely!

@jenshannoschwalm
Copy link
Contributor Author

The second commit is about the device tuning ... checked for correctness, added some hints / suggestions.

  1. Feel free to change style whatever you think is best
  2. Let me know about things you are not sure about ...

@elstoc
Copy link
Contributor

elstoc commented Dec 7, 2023

Nice, thanks. I'll take a look at it this weekend

- preferences
- setting up

Fix OpenCL tuning parameters docs

- removed unvalid stuff residuals from 4.2/4.4
- updatad docs for shared mem fraction
- some usage / setting hints

sfdvdaf
@jenshannoschwalm
Copy link
Contributor Author

More fixes, squashed and force-push a single commit for easier reviewing.

I think the only opencl topic missing in this pr would be sheduling profiles ... That is certainly just partly wrong since some dt versions and needs some caretaking. As i don't use multiple devices myself it will need some testing and code reading, will likely be able to do that on the weekend but in another pr.

@elstoc
Copy link
Contributor

elstoc commented Dec 12, 2023

LGTM thanks!

@elstoc elstoc merged commit a0256fc into darktable-org:master Dec 12, 2023
1 check passed
@jenshannoschwalm jenshannoschwalm deleted the opencl_46_1 branch December 20, 2023 19:08
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.

2 participants