-
Notifications
You must be signed in to change notification settings - Fork 415
Interactive Path Analysis Feature Integration #2526
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
Changes from all commits
Commits
Show all changes
145 commits
Select commit
Hold shift + click to select a range
59c712c
add Interactive Path Analysis (iteration 2, with multiselection featu…
w0lek db4ff57
return draw_crit_path_elements to be used in IPA, change the entry po…
w0lek 6b5ef3e
minor changes
w0lek ec044ec
remove timing report modification, adding report metadata string to t…
w0lek d7e28d4
minimize diff in tatum/TimingReporter.cpp
w0lek f941c6a
fix spellmistake, rename from OPTION_IS_FLOAT_ROUTING to OPTION_IS_FL…
w0lek b0fbefd
correct spell mistake, renaming OPTION_HIGHTLIGHT_MODE to OPTION_HIGH…
w0lek 8dfc1ae
move gtkcomboboxhelper into draw subfolder
w0lek 94aaf71
spell correction
w0lek df14178
typo in comment
w0lek 94ddbba
few minor fixes
w0lek c871dd5
typo in comment
w0lek 927bc3a
few more minor fixes in telegramoptions.h
w0lek 9f636d6
typo in var name fix
w0lek 5002740
few more minor fixes
w0lek 8bb4a9d
add default constructor for TelegramBuffer, requested in PR
w0lek 51525cf
using stream.eof() in tryConvertToInt
w0lek 6f6c7fd
constructor TelegramBuffer(int) is explicit
w0lek ff4be46
TelegramOptions::isDataTypeSupported is const
w0lek d3c5180
tryConvertToInt uses vtr::atoi
w0lek 6cab9c4
gtkcomboboxhelper.cpp and .h, update documentation, replace anonymous…
w0lek 62cc12a
GateIO add deletion constructors
w0lek 07d4fa3
add doc for public methods of gateio, improve GateIO::takeReceivedTas…
w0lek b5d3994
add comment to gateio.h about tasksMutex
w0lek ab867d3
GateIO information about usage from main thread is added
w0lek bed8f4b
move TelegramOptions definitions to a .cpp file
w0lek 7d21166
replace splitString with vtr::split
w0lek a9a9085
replace anonymous namespaces with static functions
w0lek 7cf6131
pathhelper.cpp formatting to make some lines shorter
w0lek 0c42d0b
add doc to telegrambuffer.h
w0lek df28932
add comments for TelegramBuffer::takeTelegramFrames
w0lek faf9a07
add IPA unittests
w0lek f9ffb52
add test_server_telegramoptions_get_wrong_keys TEST_CASE
w0lek e02e0c5
include std::string in telegramoptions.h
w0lek c752064
move include gateio.h and serverupdate.h under condition #ifndef NO_G…
w0lek e1372bc
fix build errors with NO_GRAPHICS flag defined
w0lek 6586c12
remove anonymous namespace from test_server_*.cpp
w0lek e6c136f
fix post #endif comment formatting
w0lek ebff401
Merge branch 'master' into ipa_it2_clean
w0lek abcc69a
fix warning [cast from type ‘const char*’ to type ‘Bytef*’] in zlibtu…
w0lek 087fb13
remove thirdparty/sockpp/conanfile.py since it's not used during buil…
w0lek 44a95ca
move sockpp to lib/EXTERNAL as a submodule
w0lek 3b9728c
move --server and --port into separate argument group "server options…
w0lek 50d79c3
pass argument in ServerContext::set_crit_path_elements by ref, add Se…
w0lek 585b5ab
cleanup: remove cmake THIRDPARTY_INCLUDE_DIRS as not needed
w0lek f2f47b1
Some sockpp headers generate warnings, so treat them as system header…
w0lek 08e754b
implement server functionality as a module which could be disabled du…
w0lek e2e9ad0
fix contour_color formatting
w0lek 61b1b1b
fix formatting in draw_crit_path_elements
w0lek bbe8a9b
fix code formatting
w0lek a1f08c4
use reference to tatum::TimingPathElem while iterating in draw_crit_p…
w0lek b9b33d8
use elementCounter for element color instead of i
w0lek 0959238
use snake_case for variables in draw_crit_path_elements
w0lek 200dbbd
formatting fix in gtkcomboboxhelper.cpp
w0lek 1150189
replace NULL with nullptr in gtkcomboboxhelper.cpp
w0lek bf67582
typo fix
w0lek b2bd272
move std::move semantic on top of TaskResolver interface. rename ::ad…
w0lek b582978
taskresolver uses VTR_LOG_ERROR instead of std::cerr
w0lek ab8971d
task resolver, improve the case with unhandled path details level str…
w0lek 65d5900
taskresolver format fix
w0lek 4d1e882
remove useless shortcut const Netlist<>&
w0lek a1b02c0
using VTR_LOG instead of std::cout in gateio
w0lek f34fa57
typo in comment
w0lek 46d2f20
gateio, initilize receivedMessage once before loop
w0lek be03fc2
typo fixes
w0lek 3e0a784
refactor GateIO::startListening by splitting it's huge body to serial…
w0lek f5d6d1e
remove redundant std::move inside TelegramParser::tryExtractJsonValueStr
w0lek 49dd7ab
telegramparser: return std::optional<std::string> instead of passing…
w0lek 968f7b6
fix unused variable warning in gateio
w0lek 437d21d
remove CRITICAL_PATH_ITEMS_SELECTION_NONE as unused, add const for EC…
w0lek 7a84ac6
commconstants.h uses std::string_view instead of char*
w0lek f7dfd50
fix warning unused variable in gateio
w0lek 4748ac2
pathhelper uses KEY_SETUP_PATH_LIST and KEY_HOLD_PATH_LIST
w0lek aadee42
get ref to string instead of copy in TelegramOptions constructor
w0lek a3aef6d
optimize Option instance creation
w0lek 5f1a732
remove redundant std::move in getTruncatedMiddleStr
w0lek 236ed56
code formatting
w0lek 81e9b2d
code formatting fix
w0lek f5fd142
code formatting fix
w0lek 7395c36
add space after each comma in test_server_taskresolver.cpp
w0lek c53f704
typo fix
w0lek 9f3f281
code formatting fix
w0lek ecf6ede
indentation in task.cpp
w0lek 1506997
remove redundant std::move in tasks.cpp
w0lek 6c4c873
code formatting
w0lek e7d6ec2
code formatting
w0lek a38b28a
code formatting
w0lek 892944d
resize receivedMessage buffer outside the loop
w0lek e58efd0
add comment for ClientAliveTracker class
w0lek 6cf5db3
allocate resultBuffer on the heap in zlibutils
w0lek 5b440ec
rework commconstants to use extern std::string instead of std::string…
w0lek 1de3eff
delete[] resultBuffer before return, which fix potential memory leak …
w0lek d5168df
tryCompress and tryDecompress returns std::optional<std::string> inst…
w0lek edb8fd4
add documentation for tryCompress and tryDecompress methods in zlibutils
w0lek 0a7580b
add doc for TelegramHeader and TelegramFrame structures
w0lek c4e7357
optimization related with std::string allocation in TelegramParser::t…
w0lek 038440e
unite generate_setup_timing_report and generate_hold_timing_report in…
w0lek cf3c11e
store SetupHoldTimingInfo and PostClusterDelayCalculator inside Serve…
w0lek 6649088
snakecase for zlibutils
w0lek 83b5fe8
snakecase in zlibutils.cpp
w0lek 1ac572b
snakecase for telegramparser
w0lek ef942f5
snakecase for telegramoptions
w0lek 3aed8fc
snakecase in telegramheader
w0lek c019984
snakecase for telegrambuffer
w0lek a59e33c
snakecase for taskresolver
w0lek 87c4fd4
snakecase for server::Task
w0lek e62a990
snakecase in serverupdate
w0lek 482f04b
snakecase for pathhelper
w0lek adaa43d
snakecase in gateio
w0lek 111b698
snakecase for convertutils
w0lek 05fdd4b
wrap more server related files with NO_SERVER macro
w0lek 723db0b
snake case for bytearray
w0lek 1afb1c8
minimize diff caused by formatting
w0lek 42abe49
server specific unit tests wrapped with NO_SERVER macro
w0lek 4ada45c
update sockpp submodule to use fork which has fixes of compilation wa…
w0lek ff1e438
Merge branch 'master' into ipa_it2_clean
w0lek 761c1be
CI: add test case when graphic is enabled but server mode is disabled
w0lek 0a0d009
Merge branch 'master' into ipa_it2_clean
w0lek 5efd5c1
make draw_crit_path_elements more generic, now it doesn't depend on S…
w0lek 974eed0
sanity checks for port number in CheckSetup() is added
w0lek 5b10a33
code format
w0lek 5886d61
ByteArray::find_sequence returns std::pair<bool, std::size_t> instead…
w0lek 09a75de
constexpr instead of static for get_truncated_middle_str::middle_plac…
w0lek 55b2aa8
constexpr instead of static for get_truncated_middle_str::minimal_str…
w0lek 0803dfb
get_pretty_str_from_float replaced with get_pretty_str_from_double to…
w0lek 565adea
reimplement TelegramOptions::is_data_type_supported
w0lek b620135
replace unneccessary "std::string(name)" with just "name"
w0lek 563eb54
align << operator in Task::info()
w0lek 333dd5c
optimization in Task::bake_response() to avoid unnecessary data coping
w0lek 8d639fc
avoid TelegramFrame copy constructor in TelegramBuffer::take_telegram…
w0lek e931555
remove obsolete members from ServerContext
w0lek 1193fca
fix build after cleanup
w0lek 32c0730
simplify bool draw_current_element logic
w0lek a608ba1
add comment of default value for skip_draw_delays in draw_flyline_tim…
w0lek 531f485
code formatting
w0lek 86eeedd
cleanup subtree_config.xml from sockpp
w0lek a4c90dd
typo fix
w0lek e1b578a
constexpr instead of local static in TelegramParser::try_extract_json…
w0lek f35de4e
variable BYTES_NUM_IN_32KB for 32768
w0lek 6e0a0dc
remove commconstants.cpp, use inline for declaring const std::string …
w0lek 6d0b10e
fix ;;
w0lek 1e4f109
ByteArray inherits std::vector<char> instead of std::vector<uint8_t>
w0lek 0f6c83c
implicit conversion ByteArray to std::string_view, remove template fo…
w0lek 453f417
std::move optimizations for crit paths result and report string
w0lek 2b74463
Merge branch 'master' into ipa_it2_clean
w0lek File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,3 +1,7 @@ | ||
[submodule "libs/EXTERNAL/libcatch2"] | ||
path = libs/EXTERNAL/libcatch2 | ||
url = https://github.com/catchorg/Catch2.git | ||
[submodule "libs/EXTERNAL/sockpp"] | ||
path = libs/EXTERNAL/sockpp | ||
#url = [email protected]:fpagliughi/sockpp.git | ||
url = [email protected]:w0lek/sockpp.git # fork where in branch v1.0.0_no_complication_warnings there are compilation warnings fixes for upstream tag v1.0.0 of sockpp |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.