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.

Sign up for GitHub

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

Jump to bottom

0.1.0 update #12

Merged
merged 59 commits into from
Aug 3, 2022
Merged

0.1.0 update #12

merged 59 commits into from
Aug 3, 2022

Conversation

JPenuchot
Copy link
Owner

  • Added a plotter that allows comparison by composable keys
  • Added CMakePresets.json
  • Added a common alias type for JSON objects using Boost flat containers
  • Added a save_plot function for multi-format saving
  • Added error management & pretty-print across the whole codebase
  • Improved safe JSON field access
  • Adapted to Sciplot & Nlohmann JSON API changes

JPenuchot added 30 commits April 15, 2022 14:55
@JPenuchot
Added internal JSON alias type
39c9174
@JPenuchot
Updated code to use grapher::json_t alias type instead oh nlohmann::json
97defd4
@JPenuchot
Using boost::container::flat_map instead of std::map for JSON objects…
c7f763d 
… yielding a ~10% performance improvement
@JPenuchot
Added reference to CPPP 2021 presentation
d5d0614
@JPenuchot
Patch version bump
039942b
@JPenuchot
Fix readme link to example configs
dc7e548
@JPenuchot
Added error management code
055ed66
@JPenuchot
Linking against fmt
ebb0e2a
@JPenuchot
Error management cleanup thanks to new check() and warn() functions
720f6bf
@JPenuchot
Slight refactoring
e4bf5fa
@JPenuchot
Improved json_value: reference support, better type handling (uses gr…
30eb023 
…apher::json_t::... for compile-time type comparisons)
@JPenuchot
Removed unnecessary copies
c799417
@JPenuchot
Added new core types for maps and multimaps, based on boost containers
6c6997b
@JPenuchot
JSON value access improvements: refactoring, added support for more t…
007a3b0 
…ypes, type checking conditions are more consistent.
@JPenuchot
Replaced JSON value and reference access with json_at and json_at_ref…
5decd88 
… calls for more consistent and safer JSON reading
@JPenuchot
Bump to v0.1.0
98c3aab
@JPenuchot
Added partial implementation for compare_all
0eae4cb
@JPenuchot
Replaced .get_ref with json_at_ref
aabe4a4
@JPenuchot
Made json_at an inline function
de213d1
@JPenuchot
compare_all progress: data wrangling and curve vector building now fi…
3f31cee 
…nished, the only things that remain now are plot building and saving
@JPenuchot
Added plot drawing, to be improved by adding legends and so on
d128fb2
@JPenuchot
Added compare_all to exposed plotters, disabled grouped_histogram
6459ce3
@JPenuchot
Fixed json_pointer constructions and split curve vector declarations …
fdbbaae 
…for better readability
@JPenuchot
Added fallback return statement in to_string
c021c0e
@JPenuchot
Added CMakePresets.json file, three presets are available including o…
99c9f62 
…ne that is specifically made for development as it provides compile_commands.json and has clang-tidy enabled
@JPenuchot
Added git exception for builds where non-dev builds are compiled
8959d76
@JPenuchot
Generating one plot per feature instead of one plot per feature+bench…
616c78c 
…mark
@JPenuchot
Added exception for CMakeUserPresets.json
e8f51b4
@JPenuchot
Error management fix: warning no longer terminates execution. Conditi…
0930d22 
…on value is also passed through
@JPenuchot
compare_all renamed to compare_by, error reporting improved
5ac2d47
JPenuchot added 27 commits May 12, 2022 13:42
@JPenuchot
JSON dump format is now a single line for more compact info reports
0998187
@JPenuchot
Error message simplification
87bc748
@JPenuchot
clang-format
7d84267
@JPenuchot
Added plot saving helper function to manage multiple extensions more …
1fe7e7d 
…easily
@JPenuchot
Using new save_plot function across all plotters
dacac45
@JPenuchot
Changed x/ylabel to x_/y_label for parameter name style consistency, …
962eca2 
…added a shameful fix for filenames being too long until a better solution is found
@JPenuchot
get_help() return type is now std::string instead of std::string_view
1ad344a
@JPenuchot
Changed return type of get_help to std::string
1e76b0a
@JPenuchot
Changed return type of get_help to std::string
2b76d9f
@JPenuchot
Max filename limit made a tiny bit cleaner (just a tiny bit)
0c91fca
@JPenuchot
Refactoring, added C++ symbol name demangling, more comments, more he…
027a880 
…lp...
@JPenuchot
Added getter for default plot options explainer
90458d5
@JPenuchot
Small refactoring, better explainer
b306cc4
@JPenuchot
Removed llvm-timescopes documentation as the file will likely remain …
b78198a 
…unmaintained
@JPenuchot
ENABLE_DOCS is now a cache variable
16888da
@JPenuchot
Removed get_help, plotter documentation will be moved to doxygen
a689173
@JPenuchot
Removed help command from grapher-utils
6403b29
@JPenuchot
Removed get_help declarations from plotter declarations
862ea43
@JPenuchot
Replaced get_help functions with doxygen paragraphs
93acc98
@JPenuchot
Renamed plotter_i to plotter_base_t
1f1ea64
@JPenuchot
Fused utils/config and utils/plot into utils/json, improved documenta…
5857044 
…tion by adding a page for JSON configs
@JPenuchot
Improved debug messages for json ref getter
c40a05e
@JPenuchot
Removed useless header
8cdcb1b
@JPenuchot
Fix ref type for value computation
3864960
@JPenuchot
Fixed y_label value getting for plot config
e0e8224
@JPenuchot
Adapted to new Sciplot API
a671230
@JPenuchot
Adapted to other Sciplot and Nlohmann JSON API changes
f13fe7c
@JPenuchot JPenuchot marked this pull request as ready for review August 3, 2022 15:40
@JPenuchot JPenuchot merged commit db83ff6 into main Aug 3, 2022
@JPenuchot JPenuchot deleted the dev branch August 3, 2022 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@JPenuchot