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.

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

\Delta_{CT} is replaced by \Delta_{FB} #115

Closed
wants to merge 3 commits into from
Closed

Conversation

aziziph
Copy link
Contributor

@aziziph aziziph commented Oct 3, 2023

No description provided.

@aziziph aziziph requested a review from gmatteo October 3, 2023 09:13
@Panadestein
Copy link
Member

Is this change necessary now? This could slow down again the publication of the paper.

@aziziph
Copy link
Contributor Author

aziziph commented Oct 3, 2023

Is this change necessary now? This could slow down again the publication of the paper.

I think so. I am not sure if it has any effect on the procedure. Are you sure that it will cause any effect? If yes, then we may ignore and live with what we have. There is also a minor problem with the number of equation 3, the parenthesizes are mixed. I can't manage to solve it. If we want to make this pull request to be merged we may need also to take care of this issue.
@dgolze Do you have any comment on this?

@dgolze
Copy link
Contributor

dgolze commented Oct 3, 2023

Hi Maryam, since this is a rather cosmetic change, I would leave it because we enter otherwise the "recommend-accept" procedure again, which took weeks for no apparent reason.

@dgolze dgolze closed this Oct 3, 2023
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.

4 participants