install all include files

NiftyPET / NIPET

High-throughput PET image reconstruction with high quantitative accuracy and precision

Apache License 2.0

29 stars 7 forks source link

install all include files #53

Open KrisThielemans opened 9 months ago

KrisThielemans commented 9 months ago

Fixes #51

However, I'm not sure if this is a good idea. The names of these include files could potentially overlap with names from other packages. It would be far better to put include files in a subdir e.g. nipet/scanner_0.h, such that such conflicts can be avoided.

Up to you if you want to merge this. I'm stuck anyway (#52)

casperdcl commented 9 months ago

hmm I think headers in niftypet/nipet/{lm,prj,sct}/src/*.h need to be moved to niftypet/nipet/include/ if they need to be exposed publicly.

@pjmark wdyt?