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

UL/DL Mode, statistic logs, error protection, subtree srsRAN #58

Open
wants to merge 37 commits into
base: main
Choose a base branch
from

Conversation

bstevens777
Copy link

Dear @hdtuanss,
Sorry, this took so long to get set up.

This is @cellular777 I had to do this on a work account (bstevens777).

There are some things you likely will not want. I subtree'd LTESniffer, srsRAN etc.
This led me to need to rebase off v2.0.0 which is where I made these changes.

Let me know what you may need help with. I will reach out on twitter shortly.

cellular777/bstevens777

Brian Stevens and others added 30 commits March 4, 2024 17:42
This reverts commit 3489e47b15f7ba640bc15608d731668633a5f1fd.
@hdtuanss
Copy link
Collaborator

hdtuanss commented Mar 8, 2024

Hi @bstevens777,
No problem, the wait is worth it. I appreciate your significant contributions to improving LTESniffer.
I have reviewed your pull request and will work on it this weekend, as I am currently occupied with other tasks. I will contact you soon.
Thank you

@bstevens777
Copy link
Author

@hdtuanss no worries. I also reached out to you on X so let me know if you need something there if it is easier.

I saw you are doing Ph.D studies. I just finished my doctorate a few months ago. I wish you the best of luck. It is a grind.

hdtuanss

This comment was marked as resolved.

@hdtuanss hdtuanss self-requested a review March 17, 2024 01:09
@hdtuanss hdtuanss dismissed their stale review March 17, 2024 01:15

review again

Copy link
Collaborator

@hdtuanss hdtuanss left a comment

Choose a reason for hiding this comment

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

merge to develop branch

hdtuanss added a commit that referenced this pull request Mar 31, 2024
@alphafox02
Copy link

Builds fine on 22.04 with stock gcc11, but with gcc12 there’s a few errors preventing it from building. Confused me for a second till I switched over to another Pc with gcc11 and then I realized what was going on. I plan to test the develop build with an x310 asap.

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.

None yet

4 participants