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

Doxygen format #64

Open
tdealtry opened this issue Jun 20, 2024 · 3 comments
Open

Doxygen format #64

tdealtry opened this issue Jun 20, 2024 · 3 comments

Comments

@tdealtry
Copy link
Collaborator

tdealtry commented Jun 20, 2024

We've now got our doxygen auto-building here 🎉

But I've noticed that the format of the doxygen is to be a copy of the code e.g. here

Collectively, I think we need to decide - is this what we want the doxygen to be?

Personally, I would suggest that instead of a copy of the code, it should be a description of what each method does / data member represents.

Thoughts in emoticon poll format

  • 👍 for old code copy
  • 👎 for new written description

Related to #47

@tdealtry
Copy link
Collaborator Author

In terms of implementing new written description (if this is chosen)

  • Standard convention is, I believe, to add to doxygen to header file
  • We may need to modify the comments in the source file around method declariations
 // *************************************
covarianceOsc::~covarianceOsc() {
// *************************************

@KSkwarczynski
Copy link
Member

Can't we have both?
I agree we should expand description whnever we can

But I find code copy to be kinda usefull. It isn't just a copy but have all hyperlinks it make navigation much easier

@tdealtry
Copy link
Collaborator Author

Both is something I hadn't considered. Yes, that could be nice!

I suppose we can keep this open until we see whether the doxygen for something that is commented in both header & source file formats as we like

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

No branches or pull requests

2 participants