Skip to content

Set var arg type before calling viewpointAdaptConstructor#267

Closed
aosen-xiong wants to merge 1 commit into
opprop:masterfrom
aosen-xiong:vararg-type
Closed

Set var arg type before calling viewpointAdaptConstructor#267
aosen-xiong wants to merge 1 commit into
opprop:masterfrom
aosen-xiong:vararg-type

Conversation

@aosen-xiong

Copy link
Copy Markdown

No description provided.

Co-authored-by: Werner Dietl <wdietl@gmail.com>
@wmdietl

wmdietl commented Jun 27, 2024

Copy link
Copy Markdown
Member

Wouldn't it be easier to pull in all changes in eisop instead of cherry-picking a few?

@aosen-xiong

Copy link
Copy Markdown
Author

Wouldn't it be easier to pull in all changes in eisop instead of cherry-picking a few?

@wmdietl Should we do it right now? I thought we are supposed pull in the change by release.

@wmdietl

wmdietl commented Jun 28, 2024

Copy link
Copy Markdown
Member

@wmdietl Should we do it right now? I thought we are supposed pull in the change by release.

Pulling in all changes up to a certain date seems less error prone than just pulling in certain commits.
With this approach, how do you know whether this change depended on some earlier changes or not?
Usually, just waiting for the next release should be enough, with the exception of maybe #268, as the updates to plume-scripts broke this repository.

@aosen-xiong

Copy link
Copy Markdown
Author

@wmdietl Should we do it right now? I thought we are supposed pull in the change by release.

Pulling in all changes up to a certain date seems less error prone than just pulling in certain commits. With this approach, how do you know whether this change depended on some earlier changes or not? Usually, just waiting for the next release should be enough, with the exception of maybe #268, as the updates to plume-scripts broke this repository.

Okay, I will wait until changes related to viewpointadpatation PRs have been merged. In that case, we can have less PICO crashed .

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