Skip to content

use doxygen-awesome css theme #3351

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

Merged
merged 9 commits into from
Jan 24, 2023
Merged

Conversation

syurkevi
Copy link
Contributor

Change documentation theme to use doxygen-awesome

Hoping to improve readability and provide the foundation for a number of inbound documentation improvements 🤞

This change bumps the doxygen.mk to 1.9.5. May still need some tweaks with the searchbar, need to test on arrayfire.org.

Sidebar, lightmode/darkmode, and function preview below:
image
image

@9prady9
Copy link
Member

9prady9 commented Dec 20, 2022

Awesome theme change! could we switch to dark mode too ?

@syurkevi
Copy link
Contributor Author

Awesome theme change! could we switch to dark mode too ?

User selectable)

@9prady9
Copy link
Member

9prady9 commented Dec 24, 2022

LGTM @umar456 ?

@9prady9
Copy link
Member

9prady9 commented Dec 24, 2022

@syurkevi Since you are at improving docs, can also please remove the following old doxygen configurations.

warning: Tag 'CLANG_ASSISTED_PARSING' at line 1207 of file '/home/pradeep/gitroot/ArrayFireWorkspace/arrayfire/build/docs/doxygen.mk.out' belongs to an option that was not enabled at compile time.
         To avoid this warning please remove this line from your configuration file or upgrade it using "doxygen -u", or recompile doxygen with this feature enabled.
warning: Tag 'CLANG_ADD_INC_PATHS' at line 1215 of file '/home/pradeep/gitroot/ArrayFireWorkspace/arrayfire/build/docs/doxygen.mk.out' belongs to an option that was not enabled at compile time.
         To avoid this warning please remove this line from your configuration file or upgrade it using "doxygen -u", or recompile doxygen with this feature enabled.
warning: Tag 'CLANG_OPTIONS' at line 1223 of file '/home/pradeep/gitroot/ArrayFireWorkspace/arrayfire/build/docs/doxygen.mk.out' belongs to an option that was not enabled at compile time.
         To avoid this warning please remove this line from your configuration file or upgrade it using "doxygen -u", or recompile doxygen with this feature enabled.
warning: Tag 'CLANG_DATABASE_PATH' at line 1236 of file '/home/pradeep/gitroot/ArrayFireWorkspace/arrayfire/build/docs/doxygen.mk.out' belongs to an option that was not enabled at compile time.
         To avoid this warning please remove this line from your configuration file or upgrade it using "doxygen -u", or recompile doxygen with this feature enabled.

Also, the results of the search typing are coming up in white color with white background here. The screenshot doesn't do justice but do try it and see.
image

\ingroup arith_mat

Addition of two inputs.
Add two arrays.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We have several operator+ overloads can accept both arrays and value types(int, float, etc.). This change makes it seem like we can only accept array types.

Additional information can include casting behavior. See me for details when you get to it.


Absolute value
\snippet test/math.cpp ex_arith_func_abs
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This sample doesn't exist. You need to add the test/math.cpp tests to this PR.

\ingroup numeric_mat

Round to integer less than equal to current value
Round to the integer less than or equal to the magnitude of the input value.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This explanation sounds confusing for floor. Can't we just say round down to the nearest integer?

\ingroup numeric_mat

Round to integer greater than equal to current value
Round to the integer greater than or equal to the magnitude of the input value.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This explanation sounds confusing. Can't we just say round up to the nearest integer?

\ingroup numeric_mat

Hypotenuse of the two inputs
Find the length of the hypotenuse of two inputs.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perhaps add a LaTeX formula for this?

\return \ref AF_SUCCESS if the execution completes properly

\ingroup arith_func_exp
\note Computes `1/(1+e^-x)`.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This shouldn't be a note

\param[in] d3 is size of fourth dimension
\param[in] seq_dim is dimesion along which [0, dim[seq_dim] - 1] is generated
\param[in] ty is the type of array to generate
* C++ Interface for creating an array with `[0, n-1]` values along the `seq_dim` dimension and tiled across other dimensions described by dimension parameters.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be a brief description. I think the second part should be part of the body of this function if not put in the dox file associated with this function.

// 1, 3, 5]

// Create b by summing across the first dimension
array b = sum(a); // sum across the first dimension, same as sum(a, 0)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should show the result of the computation in the example

array c = sum(a, 1); // sum across the second dimension

// Create d by summing across the third dimension
array d = sum(a, 2); // sum across the third dimension
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not super useful for this shape of an array. It will just return the same value as the input.

@umar456 umar456 merged commit 33935ab into arrayfire:master Jan 24, 2023
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

Successfully merging this pull request may close these issues.

4 participants