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

Reduce measure errors #15

Closed
wants to merge 2 commits into from
Closed

Reduce measure errors #15

wants to merge 2 commits into from

Conversation

JanHBade
Copy link

@JanHBade JanHBade commented Nov 3, 2024

@JanHBade
Copy link
Author

@Martinsos hello?

@Martinsos
Copy link
Owner

Hey @JanHBade ! I am pretty slammed recently so don't have much time to review the PRs here, sorry for the delay.

Url got modified to point to your repo -> that should be reverted.
There is some issue with indentation also.
We should also add a coment that explains why this was added, since it is not obvious.

Finally, I would love to be fully convinced this is the right change. I saw that other PR and mentioning of certain source file but I don't know where to find it -> any way you can point me to it? Also, how confident are you in this change, anythign that could go wrong with it? How mcuh do you understand about it?

@JanHBade
Copy link
Author

JanHBade commented Dec 5, 2024

made an new one #16

@JanHBade JanHBade closed this Dec 5, 2024
@Martinsos
Copy link
Owner

@JanHBade why new one? And you didn't asnwer my questions.

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