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

Last day changes #2

Merged
merged 28 commits into from
Aug 31, 2024
Merged

Last day changes #2

merged 28 commits into from
Aug 31, 2024

Conversation

Iainmon
Copy link
Owner

@Iainmon Iainmon commented Aug 31, 2024

Cleaning up code

Iainmon added 25 commits August 25, 2024 00:12
@Iainmon
Remove new Remote implementation debug script.
d7d9a13
@Iainmon
Remove .my_chpleconfig
d504c44
@Iainmon
Remove .vscode and .timedata
7d7f8dd
@Iainmon
Remove .vscode and .timedata
67db160
@Iainmon
Move experiments into their place.
f279b24
@Iainmon
Move examples into examples folder.
f86593a
@Iainmon
Add scripts symblink.
aa492f5
@Iainmon
Move help into learning folder.
d912e6b
@Iainmon
Remove .cls-commands.json
4a91fd9
@Iainmon
Improved simple domain implementaition.
d3ed19a
@Iainmon
Add many construction functions to SimpleDomain.chpl.
e3aca6c
@Iainmon
Add functions to simple domain implementation.
fd6eee8
@Iainmon
Improve simple domain iteration.
7fbda3c
@Iainmon
Cleaned up SimpleDomain quite a bit.
d3f116e
@Iainmon
Rebuild docs.
3a897f9
@Iainmon
Fix path in example.
5d3a68b
@Iainmon
Fix path in example.
b926296
@Iainmon
Add tensors for examples.
2862582
@Iainmon
Add example file.
3dbbf18
@Iainmon
Convert library to use simple domains. much faster.
a5eff71
@Iainmon
Remove compilation dump files.
23c0c81
@Iainmon
Add files for final deep dive.
495f14f
@Iainmon
Add first stage of removing SimpleDomain from my code. Spent quite a …
1c01dd8 
…bit of time diagnosing all of the insertions and additions.
@Iainmon
Add SimpleDomain back in for merge to main.
4d8f272
@Iainmon
Remove all usages of SimpleDomain.
6aefbcf
@Iainmon Iainmon changed the title Clean up code Last day changes Aug 31, 2024
@Iainmon
Copy link
Owner Author

Iainmon commented Aug 31, 2024 •
edited
Loading

I have cleaned up the code, somewhat. There is a difference in the performance between main at affbc30 (snapshot: https://github.com/Iainmon/ChAI/tree/intern-presentation-snapshot) and the current branch. Specifically, ./MNISTNet --imageCount=10000 takes 11 seconds to run on main, but almost 14 seconds on this new PR. I can't figure out what the problem is in the time that I have left, but I wanted to document it.

@Iainmon Iainmon merged commit 39184ff into main Aug 31, 2024
3 checks passed
DanilaFe pushed a commit that referenced this pull request Oct 4, 2024
@jeremiah-corrado
Merge pull request #2 from jeremiah-corrado/updates-for-2.2
9727617 
Minor updates for compatibility with chapel 2.2
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
@Iainmon