okay, this only reads the histogram, and lives in the pipeline thread. maybe document somewhere in the header where the module struct is defined that commit_params is thread safe wrt gui thread and pipes, and this one isn't?