Skip to content

Remove couenne#1547

Draft
classiqdor wants to merge 2 commits intomainfrom
remove_couenne
Draft

Remove couenne#1547
classiqdor wants to merge 2 commits intomainfrom
remove_couenne

Conversation

@classiqdor
Copy link
Copy Markdown
Contributor

No description provided.

@review-notebook-app
Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@review-notebook-app
Copy link
Copy Markdown

review-notebook-app Bot commented Apr 9, 2026

View / edit / reply to this conversation on ReviewNB

TomerGoldfriend commented on 2026-04-09T14:15:05Z
----------------------------------------------------------------

I think recently we had to clear this output, since it was too heavy for documentation :-/ .


@@ -11,9 +11,19 @@
]
Copy link
Copy Markdown
Member

@TomerGoldfriend TomerGoldfriend Apr 9, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Different solution but OK, same cost.


Reply via ReviewNB

@@ -9,9 +9,19 @@
]
Copy link
Copy Markdown
Member

@TomerGoldfriend TomerGoldfriend Apr 9, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Different solution but OK, same cost.


Reply via ReviewNB

@@ -9,9 +9,19 @@
]
Copy link
Copy Markdown
Member

@TomerGoldfriend TomerGoldfriend Apr 9, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Still need to verify the difference here.


Reply via ReviewNB

@@ -10,9 +10,19 @@
]
Copy link
Copy Markdown
Member

@TomerGoldfriend TomerGoldfriend Apr 9, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Different solution but OK, same cost.


Reply via ReviewNB

@@ -9,9 +9,19 @@
]
Copy link
Copy Markdown
Member

@TomerGoldfriend TomerGoldfriend Apr 12, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Different solution but looks OK


Reply via ReviewNB

@classiqdor classiqdor marked this pull request as draft April 16, 2026 11:01
@classiqdor
Copy link
Copy Markdown
Contributor Author

Converting to draft for now

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants