Skip to content

Implemented data logger plugin #7937

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

touchtouch105
Copy link

Implemented plugin to log a ton of player data and events to a local .csv file.
What config looks like:
image

example csv output:
image

@runelite-github-app
Copy link

@runelite-github-app
Copy link

This plugin requires a review from a Plugin Hub maintainer. The reviewer will request any additional changes if needed.


Internal use only: Reviewer details Maintainer details

@raiyni
Copy link
Member

raiyni commented May 2, 2025

There exists a plugin that logs CSV data csvexport

@touchtouch105
Copy link
Author

There exists a plugin that logs CSV data csvexport
That plugin logs every data point that can reasonably be queried every tick, not really equivalent

@tylerwgrass
Copy link
Member

If you wanted a smaller subset you could PR/suggest that the existing plugin has config options to toggle different info from being saved.
To me this seems more useful than a separate plugin with the random subset of fields you have picked.

@touchtouch105
Copy link
Author

touchtouch105 commented May 2, 2025

If you wanted a smaller subset you could PR/suggest that the existing plugin has config options to toggle different info from being saved. To me this seems more useful than a separate plugin with the random subset of fields you have picked.

The existing plugin is both broken and not currently being supported as per the issue tickets that have been open for 5 months
That plugin is also not event driven so the amount of data it generates is not suitable for long term logging.

@tylerwgrass
Copy link
Member

I would not assume that it is broken or unsupported just from the issues, granted I haven't used the plugin.
It is still preferred to contribute to existing plugins instead of creating new ones that practically do the same thing as multiple identical plugins.
If it is unmaintained, that is the purpose of the Plugin Takeover Policy.

@touchtouch105
Copy link
Author

I would not assume that it is broken or unsupported just from the issues, granted I haven't used the plugin. It is still preferred to contribute to existing plugins instead of creating new ones that practically do the same thing as multiple identical plugins. If it is unmaintained, that is the purpose of the Plugin Takeover Policy.

Was basing it off i downloaded it and one of the two features isnt populatable, which has had an outstanding issue against it for 5 months.
image

These really are two separate plugins though, completely different use cases, the only thing they have in common is writing data to csv files which a lot of plugins do. I can go through the motions of a plugin takeover request if that is the desired action.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants