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

[SD] Enhance "Tempo e Sincronização" page #998

Merged

Conversation

nunogoncalves03
Copy link
Member

No description provided.

content/sd/0003-tempo-e-sincronizacao.md Outdated Show resolved Hide resolved
content/sd/0003-tempo-e-sincronizacao.md Outdated Show resolved Hide resolved
content/sd/0003-tempo-e-sincronizacao.md Outdated Show resolved Hide resolved
@fmata97
Copy link
Contributor

fmata97 commented Feb 23, 2024

Are you sure this is right? It doesn't make much sense to me, I'm pretty sure it was correct before. Since RTT can be the same as RTT_{min} (i.e., when it is the minimum possible value), we can get a negative precision.

You're right, the previous expression was correct, I've changed it back to the old one.

Nit: try to keep lines wrapped to 80-100 characters max. Ideally, start every sentence in a different line.

Done

Copy link
Member

@diogotcorreia diogotcorreia left a comment

Choose a reason for hiding this comment

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

You forgot to change the result of the exercise as well 🙃
I've also noticed some more stuff related to the tables that I missed the first time.

Thanks for all the work!

content/sd/0003-tempo-e-sincronizacao.md Show resolved Hide resolved
content/sd/0003-tempo-e-sincronizacao.md Outdated Show resolved Hide resolved
content/sd/0003-tempo-e-sincronizacao.md Outdated Show resolved Hide resolved
@diogotcorreia diogotcorreia merged commit 628ad1a into leic-pt:master Feb 23, 2024
3 checks passed
@nunogoncalves03 nunogoncalves03 deleted the sd/tempo-e-sincronizacao branch February 24, 2024 08:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sd Sistemas Distribuídos
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants