Skip to content

Conversation

@HyperCodec
Copy link
Owner

Add a tracing feature for better debugging.

@HyperCodec HyperCodec added the enhancement New feature or request label Apr 4, 2025
@HyperCodec HyperCodec self-assigned this Apr 4, 2025
@HyperCodec HyperCodec linked an issue Apr 4, 2025 that may be closed by this pull request
@HyperCodec
Copy link
Owner Author

Once genetic-rs is updated, I can finish this PR.

@HyperCodec
Copy link
Owner Author

Might want to add a trait that requires std::fmt::Debug + AsRef<NeuronLocation> so I can use the instrument macro on functions that take those args as input.

@HyperCodec
Copy link
Owner Author

Some of these spans are probably unnecessary, might remove them later. For now I need to add some debug printing.

@HyperCodec
Copy link
Owner Author

I think I might just add a lot of this debug printing while I'm actually debugging, since at least I know then it'll be practical and not just a bunch of spam.

@HyperCodec HyperCodec marked this pull request as ready for review April 8, 2025 14:59
@HyperCodec
Copy link
Owner Author

might want to add a way to debug the current activation registry, right now it's pub(crate) so viewing it for debug could be difficult.

@HyperCodec
Copy link
Owner Author

also some of these instruments/spans are probably unnecessary because no events are being fired in them, but I don't really know if I want to remove them because they could be useful for debugging in the future.

@HyperCodec
Copy link
Owner Author

I think for now I'll just merge this and deal with that stuff later.

@HyperCodec HyperCodec merged commit e54f1d8 into new-readme Apr 8, 2025
3 checks passed
@HyperCodec HyperCodec deleted the tracing branch April 8, 2025 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Tracing feature

2 participants