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

Dataset learning #9

Open
wants to merge 31 commits into
base: dev
Choose a base branch
from
Open

Dataset learning #9

wants to merge 31 commits into from

Conversation

skourta
Copy link
Collaborator

@skourta skourta commented Feb 6, 2023

Loading and learning from a dataset.

@skourta
data loading alpha version
de5c010
@skourta
removed berakpoints and fixed code style to match previous code
50658fc
@skourta
added support for the legality check of skewing
6985d62
@skourta
removed breakpoint
8bdd2c2
@skourta
working on resuming learning
c84c891
@skourta
refactored code from later commit
1643fc2
@skourta
fixed loading exec time from dataset and cleaned some code
6c8b39b
@skourta
refactored checking the hostname in function for future changes
617b692
@skourta
fixed invalidating exec time on init time
00ff9b6
@skourta
load new version of dataset and save it
3422a62
@skourta
dataset learning with one functiona at a time
0300f24
@skourta
removed import bug
0455369
@skourta
data saviung multiple formats
d5efeca
@skourta
data saviung multiple formats
5e1f766
@skourta
added solver resutls to the dataset, fixed circular imports and remov…
c4c162f 
…e import *
@skourta
changed saving frequency to increase performance
6b78258
@skourta skourta self-assigned this Feb 6, 2023
hbenyamina
hbenyamina approved these changes Mar 2, 2023
View reviewed changes
Copy link
Collaborator

@hbenyamina hbenyamina left a comment

Choose a reason for hiding this comment

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

Results proven experimentally. Though I still think synchronization is important, if the model can reach the same performance without sync, that proves it is unnecessary.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hbenyamina hbenyamina hbenyamina approved these changes

Assignees

@skourta skourta

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@skourta @hbenyamina