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

Samples for data logging #84

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

Conversation

microbit-matt-hillsdon
Copy link
Collaborator

DataLoggingNextGenTest is suitable for testing MICROBIT_LOG_MODE 2

Depends on lancaster-university/codal-microbit-v2#399

DataLoggingNextGenTest is suitable for testing MICROBIT_LOG_MODE 2
@microbit-matt-hillsdon
Copy link
Collaborator Author

@JohnVidler I don't really have a strong view re these sample changes being merged. They were useful for my testing at least.

@JohnVidler JohnVidler self-assigned this Jan 29, 2024
@JohnVidler JohnVidler added documentation Improvements or additions to documentation enhancement New feature or request labels Jan 29, 2024
@JohnVidler JohnVidler added this to the v0.2.60 milestone Jan 29, 2024
@JohnVidler
Copy link
Collaborator

@JohnVidler I don't really have a strong view re these sample changes being merged. They were useful for my testing at least.

Examples are generally good 👍 and I don't see any reason not to include these, especially if someone down the line wants to test additional logging options.

That said, I think both of these could be the same samples file, with some additional comments on what they do. As these are minor changes I'll roll them into this PR and get this merged shortly.

@microbit-matt-hillsdon microbit-matt-hillsdon marked this pull request as ready for review January 29, 2024 21:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants