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.

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

Update Django, Ray, Ubuntu #123

Merged
merged 27 commits into from
Nov 7, 2024

Conversation

jhazentia
Copy link
Member

This includes the following. Thanks to Doug Boulware for Ray compatibility updates.

  • Update contact info
  • Update Django to 4.2 or newer (less than 5), ray to 2.10 or newer (tested using 2.37)
  • Changes to support Ubuntu 22.04 (get_current_cpu_clock_speed() updated)

Tested performance of new ray version by measuring run time of CBRS measure action using perf_counter immediately before and after action in scheduler. Repeated the action 5 times. Average run time with old ray was 103.16 seconds, new version average run time was 102.438 seconds.

Copy link
Member

@aromanielloNTIA aromanielloNTIA left a comment

Choose a reason for hiding this comment

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

I will test this next week, but I think I noticed that the PVT data is not added to the sigmf-data file after these changes

@jhazentia jhazentia marked this pull request as draft October 31, 2024 20:46
@jhazentia jhazentia marked this pull request as ready for review November 4, 2024 21:55
Copy link
Member

@aromanielloNTIA aromanielloNTIA left a comment

Choose a reason for hiding this comment

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

Changes look good, tested and checked data output on seadog01. Let's hold off on creating a new release until #124 and #125 are merged as well.

@aromanielloNTIA aromanielloNTIA merged commit 177d6a1 into master Nov 7, 2024
4 checks passed
@aromanielloNTIA aromanielloNTIA deleted the SEA-234_ubuntu22.04_python3.10_ray2.10 branch November 7, 2024 21:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants