Skip to content
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.

Sign up for GitHub

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

Jump to bottom

add meaningful __str__ and __repr__ to classes #35

Open
nowheremanx opened this issue May 2, 2022 · 3 comments
Open

add meaningful __str__ and __repr__ to classes #35

nowheremanx opened this issue May 2, 2022 · 3 comments
Assignees
@machallboyd
Labels
enhancement New feature or request

Comments

@nowheremanx
Copy link
Contributor

nowheremanx commented May 2, 2022 •
edited
Loading

for easier debugging. __str__ is also important for jupyter notebook users to print out summaries
@bksettler bksettler assigned bksettler and unassigned bksettler May 2, 2022
@zenon18 zenon18 assigned zenon18 and unassigned zenon18 May 3, 2022
@machallboyd machallboyd self-assigned this May 11, 2022
machallboyd added a commit that referenced this issue May 11, 2022
@machallboyd
[#35] Quick and dirty str and repr
ef21784
machallboyd added a commit that referenced this issue May 11, 2022
@machallboyd
[#35] Clean up NXMap string functions
734ed13
machallboyd added a commit that referenced this issue May 11, 2022
@machallboyd
[#35] Formatting, clean up lcss representation
b6ed565
@machallboyd machallboyd mentioned this issue May 11, 2022
Closed
machallboyd added a commit that referenced this issue May 11, 2022
@machallboyd
[#35] Make flake8 happier
9514811
machallboyd added a commit that referenced this issue May 13, 2022
@machallboyd
[#35] coordina str function
e9d2e2d
machallboyd added a commit that referenced this issue May 13, 2022
@machallboyd
[#35] black on trace
c94ecf3
machallboyd added a commit that referenced this issue May 13, 2022
@machallboyd
[#35] Quick and dirty str and repr
575da18
machallboyd added a commit that referenced this issue May 13, 2022
@machallboyd
[#35] Clean up NXMap string functions
0715337
machallboyd added a commit that referenced this issue May 13, 2022
@machallboyd
[#35] Formatting, clean up lcss representation
1e92ead
machallboyd added a commit that referenced this issue May 13, 2022
@machallboyd
[#35] Make flake8 happier
af2f933
machallboyd added a commit that referenced this issue May 13, 2022
@machallboyd
[#35] coordina str function
e3ead35
machallboyd added a commit that referenced this issue May 13, 2022
@machallboyd
[#35] black on trace
efb6b4a
machallboyd added a commit that referenced this issue May 31, 2022
@machallboyd
[#35] Add str and repr functions to Trace
f929a1a
machallboyd added a commit that referenced this issue May 31, 2022
@machallboyd
[#35] Add str and repr functions to NxMap
e4dd8f2
machallboyd added a commit that referenced this issue Jun 23, 2022
@machallboyd
[#35] Add str and repr functions to Trace
1d8699b
machallboyd added a commit that referenced this issue Jun 23, 2022
@machallboyd
[#35] Add str and repr functions to NxMap
cad702b
machallboyd added a commit that referenced this issue Jun 23, 2022
@machallboyd
[#35] Add str and repr functions to Trace and NXMap
68eb738
machallboyd added a commit that referenced this issue Jun 23, 2022
@machallboyd
[#35] Explode on missing _frame if printing
5895949
machallboyd added a commit that referenced this issue Jun 23, 2022
@machallboyd
[#35] Remove testing relic
4bbd8cc
@Rowlando13
Copy link
Collaborator

@machallboyd Is this done? I thought I remember a PR for it but can't find it.

@machallboyd
Copy link
Collaborator

I added some to two classes, but there's probably more to be done.

@Rowlando13 Rowlando13 added the enhancement New feature or request label Jul 24, 2022
@Rowlando13
Copy link
Collaborator

Got it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@machallboyd machallboyd

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@machallboyd @nowheremanx @zenon18 @Rowlando13 @bksettler