Preliminary hidden bwp monitor #11
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
nrscope/src/libs/task_scheduler.cc
Outdated
| * dci size with CRC == 1 to serve as a target and maybe use dynamic programming | ||
| * to tune the dci-size parameters towards the target size | ||
| */ | ||
| std::ifstream f("/home/xyc/hidden_bwp_40/NG-Scope-5G/nrscope/hidden_bwp_db/369.txt"); |
There was a problem hiding this comment.
let me change this as a config file (not config file but copy in CMAKE term)...
| @@ -31,6 +31,9 @@ | |||
| #include "asn1_utils.h" | |||
| #include <cstdio> | |||
| #include <stdarg.h> | |||
| #include <nlohmann/json.hpp> | |||
There was a problem hiding this comment.
Let me add to README how to install this (installed on our server though)
| @@ -1149,6 +1150,11 @@ int srsran_pdcch_nr_decode_with_rnti_nrscope_dciloop(srsran_pdcch_nr_t* q, | |||
|
|
|||
| // Copy DCI message | |||
| srsran_vec_u8_copy(dci_msg->payload, c, dci_msg->nof_bits); | |||
| // printf("[hidden bwp dci bits]: "); | |||
There was a problem hiding this comment.
Keep those comments for later use thanks
| @@ -48625,6 +48629,624 @@ SRSASN_CODE cell_group_cfg_s::unpack(cbit_ref& bref) | |||
| } | |||
| return SRSASN_SUCCESS; | |||
| } | |||
|
|
|||
| SRSASN_CODE cell_group_cfg_s::from_json(json js_in) { | |||
There was a problem hiding this comment.
Didn't follow the 80-character rule here; match what they have in srsRAN libraries and this make those long indexing looks easier.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
As title. Write a JSON parser that takes JSON-format (as discussed) RRC Reconfiguration and monitor on hidden bwp. If ASN format given, we can just use the ASN decoder (even better).
Test on MOSOLAB 40MHz multi bwp. Works well; correctly deduce the DCI size from configs and monitor the hidden bwp DCIs.
Next steps:
Interface with the actual DB.
Frequency start detection (hidden bwp might start on a different RB, multiple of 6 though).
CRC check for different DCI size for cross validation.
^ more can be found in comments as well.
The JSON parser will be improved based on more configuration scenarios we see (too many optional fields and hence existence check handling).
a2.csv
a1.csv