Skip to content

Document how to speed up your bot in a seperate optimizing page #247

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

Merged
merged 2 commits into from
May 17, 2023

Conversation

TAG-Epic
Copy link
Member

@TAG-Epic TAG-Epic commented May 2, 2023

Currently missing

  • HTTP JSON optimizing
  • Changing the global rate limit (needs to be properly supported first)

@TAG-Epic TAG-Epic added the type/docs Changes to the documentation label May 2, 2023
Copy link
Contributor

@MaskDuck MaskDuck left a comment

Choose a reason for hiding this comment

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

summary:

  • you need 's since the ' makes a lot of difference (e.g. its and it's)
  • NTP needs capitalization

@@ -0,0 +1,82 @@
Optimizing
==========
Heres a few tips to make your bot run a tiny bit faster, from most impact to least. The accuracy of the ordering is more of a guess, and depends on your usage.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Heres a few tips to make your bot run a tiny bit faster, from most impact to least. The accuracy of the ordering is more of a guess, and depends on your usage.
Here's a few tips to make your bot run a tiny bit faster, from most impact to least. The accuracy of the orderings is more of a guess, and depends on your usage.


If it is synchronized, it will show "System clock synchronized: yes" and "NTP service: running"

If the system clock is not synchronized but the ntp service is running you will have to wait a few minutes for it to sync.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
If the system clock is not synchronized but the ntp service is running you will have to wait a few minutes for it to sync.
If the system clock is not synchronized but the NTP service is running you will have to wait a few minutes for it to sync.


If the system clock is not synchronized but the ntp service is running you will have to wait a few minutes for it to sync.

To enable the ntp service run the following command:
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
To enable the ntp service run the following command:
To enable the NTP service run the following command:


If it is synchronized, it will show "System clock synchronized: yes" and "NTP service: running"

If the system clock is not synchronized but the ntp service is running you will have to wait a few minutes for it to sync.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
If the system clock is not synchronized but the ntp service is running you will have to wait a few minutes for it to sync.
If the system clock is not synchronized but the NTP service is running you will have to wait a few minutes for it to sync.


If the system clock is not synchronized but the ntp service is running you will have to wait a few minutes for it to sync.

To enable the ntp service run the following command:
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
To enable the ntp service run the following command:
To enable the NTP service run the following command:

Co-authored-by: MaskDuck <it-is@too-obvious-that.maskduck.is-a.dev>
@TAG-Epic TAG-Epic added this pull request to the merge queue May 17, 2023
@TAG-Epic TAG-Epic merged commit c3332de into master May 17, 2023
@TAG-Epic TAG-Epic deleted the tag-epic/docs/document-json-speedups branch September 2, 2023 11:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/docs Changes to the documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants