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

TlmDefで一時変数を使えるようにしたい #140

Closed
meltingrabbit opened this issue Dec 27, 2021 · 4 comments
Closed

TlmDefで一時変数を使えるようにしたい #140

meltingrabbit opened this issue Dec 27, 2021 · 4 comments
Assignees
Labels
enhancement New feature or request priority::high priorityg high tools WINGS

Comments

@meltingrabbit
Copy link
Collaborator

概要

TlmDefで一時変数を使えるようにしたい

詳細

  • static int Tlm_MOBC_(unsigned char* contents, int max_len)
    {
    if (244 > max_len) return TF_TOO_SHORT_LEN;
    #ifndef BUILD_SETTINGS_FAST_BUILD
    TF_copy_u32(&contents[0], (uint32_t)(TMGR_get_master_clock().mode_cycle));
    TF_copy_u8(&contents[4], (uint8_t)(mode_manager->stat));
    に一時変数をいれたい.
    • 一時変数を使えたほうが,テレメがシンプルになるし,高速化も期待できる
  • DBはこんな感じになる?

image

close条件

できたら

備考

もしかしたらWINGS側の対応も必要かも?

@meltingrabbit meltingrabbit added enhancement New feature or request priority::high priorityg high tools WINGS labels Dec 27, 2021
@meltingrabbit meltingrabbit self-assigned this Dec 27, 2021
@meltingrabbit
Copy link
Collaborator Author

みやすさ的にこっちかな?

image

@meltingrabbit
Copy link
Collaborator Author

これしないと,

TF_copy_u8(&contents[14], (uint8_t)event_logger->tlogs[event_logger->tlm_info.tlog.err_level].events[EL_TLOG_TLM_PAGE_SIZE*event_logger->tlm_info.tlog.page_no+0].group);
TF_copy_u32(&contents[15], event_logger->tlogs[event_logger->tlm_info.tlog.err_level].events[EL_TLOG_TLM_PAGE_SIZE*event_logger->tlm_info.tlog.page_no+0].local);
TF_copy_u32(&contents[19], event_logger->tlogs[event_logger->tlm_info.tlog.err_level].events[EL_TLOG_TLM_PAGE_SIZE*event_logger->tlm_info.tlog.page_no+0].time.total_cycle);

とかが破滅してきたんですよね...

@yngyu @yanagida-kanta どう思います?

@meltingrabbit
Copy link
Collaborator Author

@yngyu と話して,まあとりあえずは問題ないだろうとなったので,すすめる.

@meltingrabbit
Copy link
Collaborator Author

C2A,ツール,DBすべてマージされたので,close

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request priority::high priorityg high tools WINGS
Projects
None yet
Development

No branches or pull requests

1 participant