search

LDMX-Software / ldmx-sw

The Light Dark Matter eXperiment simulation and reconstruction framework.
https://ldmx-software.github.io
GNU General Public License v3.0
21 stars 19 forks source link

Define printing method for UserAction python class #662

Closed tomeichlersmith closed 4 years ago

tomeichlersmith commented 4 years ago

This will help when the process method printMe is called for people to see what parameters they are passing to the process and to the UserAction method.

omar-moreno commented 4 years ago

That should already exists.

On Thu, Mar 26, 2020, 10:25 Tom Eichlersmith notifications@github.com wrote:

This will help when the process method printMe is called for people to see what parameters they are passing to the process and to the UserAction method.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/LDMX-Software/ldmx-sw/issues/662, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA4JMXAYDPDLFSIYEBNCQ3TRJOFXVANCNFSM4LUM5IAQ .

tomeichlersmith commented 4 years ago

I just want to link it to the python print command if possible so that when print is called on the parameter values, python calls the printMe function for the UserAction.

What I'm thinking of doing is shown on StackOverflow.

omar-moreno commented 4 years ago

I guess, I would get rid of the PrintMe function and just add str to the Producer and Analyzer classes instead. In the end, PrintMe is just trying to replicate what strnatively does.

Then you can just call print UserActionObject or print ProcessorObject.

On Thu, Mar 26, 2020 at 10:41 AM Tom Eichlersmith notifications@github.com wrote:

I just want to link it to the python print command if possible so that when print is called on the parameter values, python calls the printMe function for the UserAction.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/LDMX-Software/ldmx-sw/issues/662#issuecomment-604575096, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA4JMXCHHHNYO7E4PFNFHU3RJOHVDANCNFSM4LUM5IAQ .

tomeichlersmith commented 4 years ago

Sounds good, I will plan on doing that.

omar-moreno commented 4 years ago

Awesome!

On Thu, Mar 26, 2020, 10:48 Tom Eichlersmith notifications@github.com wrote:

Sounds good, I will plan on doing that.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/LDMX-Software/ldmx-sw/issues/662#issuecomment-604578689, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA4JMXCOOH6ZQ2YLF65IBI3RJOINZANCNFSM4LUM5IAQ .