Sorting by active monthly desc. #6

Merged
nutomic merged 1 commit from sort_active_monthly into main 2021-04-27 13:10:50 +00:00
Owner
No description provided.
dessalines added 1 commit 2021-03-24 23:46:21 +00:00
Owner

I had sorting implemented, but dont think that it should be handled by this tool, because there are a lot of different possibilities for sorting. I think it makes more sense to handle that in the program which uses the data.

I had sorting implemented, but dont think that it should be handled by this tool, because there are a lot of different possibilities for sorting. I think it makes more sense to handle that in the program which uses the data.
Author
Owner

I def think sorting should be part of this tool, because otherwise the output has an unpredictable order, and the main point of this is to create an output file that can be placed into a git repo, with predictable changes. Having a predictable sort is a sane default.

I def think sorting should be part of this tool, because otherwise the output has an unpredictable order, and the main point of this is to create an output file that can be placed into a git repo, with predictable changes. Having a predictable sort is a sane default.
Owner

I'm not sure if thats the main point of this tool. And in that case, it seems more reliable to sort by instance name, no?

I'm not sure if thats the main point of this tool. And in that case, it seems more reliable to sort by instance name, no?
Author
Owner

Ideally you could pass in a sort as a command line param. Names could work, but I think in the case of a crawler, where you're potentially bound to get data that could be unused or irrelevant, popularity sorts make the best sense.

Ideally you could pass in a sort as a command line param. Names could work, but I think in the case of a crawler, where you're potentially bound to get data that could be unused or irrelevant, popularity sorts make the best sense.
nutomic merged commit fefd708777 into main 2021-04-27 13:10:50 +00:00
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: LemmyNet/lemmy-stats-crawler#6
No description provided.