search

tomography / XRFtomo

a repository for xray fluorescence data processing
Other
3 stars 6 forks source link

Created widget and main application classes. The gui app displays but… #2

Closed aglowacki closed 6 years ago

aglowacki commented 6 years ago

… the buttons are not connected to any actions.

decarlof commented 6 years ago

@aglowacki shall we rename src as xfluo? this was we can build it as a python library (import xfluo), if so I will also move the read.py (now src/IO/aps_dataset_io.py) main() into a doc/demo folder and keep in the xfluo/io just the library functions (restricted_float, find_index, read_elements, read_projection). Let me know if you are OK with this and I will start to build the readthedoc part

aglowacki commented 6 years ago

That sounds good to me, I just added src to have some directory structure.

On Jul 31, 2018, at 11:35 PM, Francesco De Carlo notifications@github.com wrote:

@aglowacki https://github.com/aglowacki shall we rename src as xfluo? this was we can build it as a python library (import xfluo), if so I will also move the read.py (now src/IO/aps_dataset_io.py) main() into a doc/demo folder and keep in the xfluo/io just the library functions (restricted_float, find_index, read_elements, read_projection). Let me know if you are OK with this and I will start to build the readthedoc part

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/tomography/xfluo/pull/2#issuecomment-409446608, or mute the thread https://github.com/notifications/unsubscribe-auth/AGQFzFGtXLDI6PwOE0K96beoYPhgeDYxks5uMTAXgaJpZM4VpKij.

aglowacki commented 6 years ago

I changed src to xfluo in the second commit for this pull request.

decarlof commented 6 years ago

excellent. I will merge this and start to add the docs.