sandialabs / pyttb

Python Tensor Toolbox
https://pyttb.readthedocs.io
BSD 2-Clause "Simplified" License
25 stars 13 forks source link

Feedback fixes #229

Closed ntjohnson1 closed 1 year ago

ntjohnson1 commented 1 year ago

This is a combination of a variety of pieces. If there is anything that needs more discussion I can separate into a different branch/PR and we can land everything else.

Resolves:

Some other niceties:


:books: Documentation preview :books:: https://pyttb--229.org.readthedocs.build/en/229/

ntjohnson1 commented 1 year ago

FYI @dmdunla I wasn't able to add @jeremy-myers as a reviewer so it might make sense to check his permissions. Otherwise I tagged him here as a heads up to see if there is anything I misinterpreted of forgot to resolve for the issues raised.

jeremy-myers commented 1 year ago

@ntjohnson1 I just accepted @dmdunla 's invitation; try adding me for review again.

dmdunla commented 1 year ago

@jeremy-myers when you do get a chance to review, please just re use and do not merge. I’ll merge once the reviews are complete.


From: Jeremy Myers @.> Sent: Sunday, September 10, 2023 12:00:04 PM To: sandialabs/pyttb @.> Cc: Dunlavy, Danny @.>; Mention @.> Subject: [EXTERNAL] Re: [sandialabs/pyttb] Feedback fixes (PR #229)

@ntjohnson1https://github.com/ntjohnson1 I just accepted @dmdunlahttps://github.com/dmdunla 's invitation; try adding me for review again.

— Reply to this email directly, view it on GitHubhttps://github.com/sandialabs/pyttb/pull/229#issuecomment-1712898138, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AHIY2IVNXMQMZEOMT5BW3G3XZX52JANCNFSM6AAAAAA4QMZLJQ. You are receiving this because you were mentioned.Message ID: @.***>