search

pelahi / VELOCIraptor-STF

Galaxy/(sub)Halo finder for N-body simulations
MIT License
20 stars 26 forks source link

Interloper variables #3

Closed DCunnama closed 7 years ago

DCunnama commented 7 years ago

io.cxx: In function ‘void WriteProperties(Options&, Int_t, PropData*)’: io.cxx:1261: error: ‘struct PropData’ has no member named ‘n_interloper’ io.cxx:1265: error: ‘struct PropData’ has no member named ‘M_interloper’

It appears the latest change introduced a bug.

If ZOOMSIM is not defined, HIGHRES is not defined, however that doesn't stop io.cxx from attemping to access the interloper members.

Not sure what the desired behaviour is.

pelahi commented 7 years ago

For checking contamination will fix flag

On 20 Jan 2017 17:55, "DCunnama" notifications@github.com wrote:

io.cxx: In function ‘void WriteProperties(Options&, Int_t, PropData*)’: io.cxx:1261: error: ‘struct PropData’ has no member named ‘n_interloper’ io.cxx:1265: error: ‘struct PropData’ has no member named ‘M_interloper’

It appears the latest change introduced a bug.

If ZOOMSIM is not defined, HIGHRES is not defined, however that doesn't stop io.cxx from attemping to access the interloper members.

Not sure what the desired behaviour is.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/pelahi/VELOCIraptor-STF/issues/3, or mute the thread https://github.com/notifications/unsubscribe-auth/ALeQS-MGm1m3Plyo-1L1NzuYsWq_nOeaks5rUISMgaJpZM4LpH8I .

pelahi commented 7 years ago

should be fixed, as I'm running it now with latest version.

On 20 January 2017 at 17:55, DCunnama notifications@github.com wrote:

io.cxx: In function ‘void WriteProperties(Options&, Int_t, PropData*)’: io.cxx:1261: error: ‘struct PropData’ has no member named ‘n_interloper’ io.cxx:1265: error: ‘struct PropData’ has no member named ‘M_interloper’

It appears the latest change introduced a bug.

If ZOOMSIM is not defined, HIGHRES is not defined, however that doesn't stop io.cxx from attemping to access the interloper members.

Not sure what the desired behaviour is.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/pelahi/VELOCIraptor-STF/issues/3, or mute the thread https://github.com/notifications/unsubscribe-auth/ALeQS-MGm1m3Plyo-1L1NzuYsWq_nOeaks5rUISMgaJpZM4LpH8I .

DCunnama commented 7 years ago

Not for me, have you tried to compile with ZOOMSIM off?

pelahi commented 7 years ago

arg, yes, didn't push to repository before I left for soccer. Sorry Dan.

On 20 January 2017 at 18:24, DCunnama notifications@github.com wrote:

Not for me, have you tried to compile with ZOOMSIM off?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/pelahi/VELOCIraptor-STF/issues/3#issuecomment-274036096, or mute the thread https://github.com/notifications/unsubscribe-auth/ALeQSxLlb2TjqKqo8cL6E5laRoyVCpkSks5rUItlgaJpZM4LpH8I .

DCunnama commented 7 years ago

Ha ha, ok fair enough. No problem. Enjoy the football!

On Fri, Jan 20, 2017 at 12:28 PM, Pascal Jahan Elahi < notifications@github.com> wrote:

arg, yes, didn't push to repository before I left for soccer. Sorry Dan.

On 20 January 2017 at 18:24, DCunnama notifications@github.com wrote:

Not for me, have you tried to compile with ZOOMSIM off?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/pelahi/VELOCIraptor-STF/issues/3# issuecomment-274036096, or mute the thread https://github.com/notifications/unsubscribe-auth/ ALeQSxLlb2TjqKqo8cL6E5laRoyVCpkSks5rUItlgaJpZM4LpH8I

.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/pelahi/VELOCIraptor-STF/issues/3#issuecomment-274037106, or mute the thread https://github.com/notifications/unsubscribe-auth/AMagN3gawTTX4nOrf1WuVTJLr76nThdBks5rUIxNgaJpZM4LpH8I .