-
Notifications
You must be signed in to change notification settings - Fork 182
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
File based logging and DL/UL mode #48
Comments
@cellular777 Thanks for your great effort and contribution to LTESniffer.
For discussion and code sharing, I think we have 2 options:
For convenience, could you add my Twitter account: @hdtuanss. We can use DM on Twitter for fast sharing and contacting. Thanks, |
@cellular777 |
oof, my formatting got weird (fixed above). Sorry for the delay. And the subsequent delay.
The code is in a local repo. I got permission to share back out, so I will need to set up a repo with this account and then will do one of the above. @hdtuanss do you prefer the pull request or adding to a project (whatever is best for you). @hdtuanss ha, I dont use twitter but I will make one so we can chat more directly. @hdtuanss @Paloma-96 I unfortunatey was pressed for time and am out the following week. Come 2/5/2024 I will focus on getting the above sorted. Sorry again for the delay... On other news, I do have a couple fixes for the UL rntimanager errors on the other issue. #37 I will post under there in general what I did. But those changes will likely just be included in process above. |
Hi Stevens, Thanks for your update. I have also seen your update on another issue #37. For both issues, I think a pull request would be better for us. Could you please set up your repo and make a pull request for me? It would be great if you could include the fix for issue #37 together on your pull request. Also, on point 1 we are discussing, since your new mode "2" can run independently from original modes "0" and "1", I think it will not have any problem if we add that mode to LTESniffer. So, let's make a separate new mode first; then, I will compare and see if I can combine your mode "2" and mode "1" into one. Once again, I appreciate your contribution a lot. If you dont have Twitter account, please give me your any social account so I can add yours. Thank you. |
@hdtuanss
I have been adding some capabilities to a local version of the repo.
The above required a lot of changes, are you interested in the above 1 or 2? If so, I will see if I can get permission to share.
Thanks again for the great work on this project!
The text was updated successfully, but these errors were encountered: