-
Notifications
You must be signed in to change notification settings - Fork 0
Basiav/sampleset times extraction #10
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
Open
basiav
wants to merge
38
commits into
main
Choose a base branch
from
basiav/sampleset_times_extraction
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
38 commits
Select commit
Hold shift + click to select a range
01518e2
Times extraction 1
basiav 19637c2
updates:
basiav e1f2200
Working
basiav 6e24d59
Second
basiav b0e69c2
Added
basiav 2beea10
Works
basiav 105a8c7
Done
basiav c11bdc3
All
basiav dce3d15
Changes in sampleset_metadata
basiav f78dbeb
Comm
basiav c027409
update
basiav 6ce6ea2
Qommnunity demo notebook update
kacper3615 cd6a224
No default value for AdvantageSampler
kacper3615 6a16a14
Updated requirements
kacper3615 e89de6d
Added
basiav 20b8297
Updates
basiav f21b3a5
Updates
basiav 1f73a2f
First version to push
basiav ba4b2d9
Updates
basiav f27e5b4
Added time units, find_embedding time capturing for heuristic embedding
basiav f8ea3aa
Delete Qommunity/results_analyzer.py
basiav da47995
Fixed hierarchical_searcher error
kacper3615 51faac7
Full modularity matrix will calculate only once
kacper3615 334096d
PR suggestions applied
basiav c1a846b
Merge branch 'main' of https://github.com/kacper3615/Qommunity into b…
basiav 059957c
Fix with return_metadata in HierarchicalSearcher
basiav db80fde
Update in the demo/sampleset_times_extraction/iterative_searcher_samp…
basiav 7d4f4a9
Applied Black formatter
kacper3615 007e5c6
Progres
basiav 04aadef
Updates:
basiav a7e9485
Saving plots
basiav ff42031
Updates
basiav 6e2b8bc
Updates
basiav 47e544f
Udpate
basiav de065ef
All
basiav ed6d0d3
Safety copy
basiav 2eae861
Sampleset info gathering prototype working
basiav 3f56f08
Rename powerlaw_169.ipynb to erdos_renyi_169.ipynb
basiav File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
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
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.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it a temporary solution?
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe, it may be expanded later with embeddings returning/nicer results interface, so you kind of sensed it right that it might be changed. But a list object for now. Is it ok? I was rather explaining why a list not np.empty like the rest.