-
Notifications
You must be signed in to change notification settings - Fork 5
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
Provide a small python tool to generate filter #35
Comments
In case I'm not mistaken about the whereabouts of this issue, you'd may be interested in http://pybind11.readthedocs.io/en/master/ This is probably the most advanced solution to generate Python bindings from Modern C++11 & more code. |
Thanks @LucHermitte for your feedback but it seems more related to #31 |
The file https://github.com/CS-SI/SIRIUS/blob/master/doc/sphinx/source/code_py/simple_signal_proc.py should be a good start. |
@rkanavath those really are different issues that I will try to explain here: #31 is merely just about providing python API to what Sirius does which is resampling. Now, Sirius uses filters for resampling purpose. Filters that have to be created somehow. Hence, trying to summarize all that, this current issue is about creating a tool that will help users shape rather simple filters. Then users could give such a filter to Sirius. For that they will launch it as an executable or using its C++ or Python API (the last one being the subject of #31). Please tell me if I did not answer your question. |
okay. IIUC you have to plug some python function into sirius processing. is that correct ? |
Well not entirely. I guess I was not clear enough. Maybe it comes from the word "filter". Just to make it clear, here filter is not an OTB filter. Nothing like a function or a method. In Sirius context a filter is a collection of samples (weights) that one will apply to an image or a signal as part of a convolving process. Now, metaphorically, say Sirius is a car that is really fast. Everyone can use it in a standard fashion which actually covers some of the functionality one could do with a car. But, if one wishes to take the most out of Sirius, it needs special gasoline (a special filter in our real context). So now we raise this issue which aims at filling this void by giving users who borrowed Sirius what they need to easily fashioned they own special gasoline. I believe we could have given this "filters creator" his own repository and name. I am quite unsure I made this clearer :) |
okay. filter has many faces and currently there is a "filter" in sirius, one in OTB (don't know why this come up in discussion) and another in image processing terms. I understand your reference to filter is some function (mathematical) applied to an image. so in a way you are applying a filter ( written in form of a function perhaps) to a sirius process. I will have to revisit this once I settle on python bindings. |
@tfeuvrier-cs this tool must be external. It takes some parameters (which describe the filter) as input and generate a file with the filter inside. This file is provided to sirius as input. |
@savmickael : this issue could be close. ; this tool is available from gen-sirius-filter project (https://gitlab.cloud-espace.si.c-s.fr/msavinau/gen-sirius-filter) |
No description provided.
The text was updated successfully, but these errors were encountered: