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

Implementation of SR2.17 and some backwards specifications #109

Merged
merged 33 commits into from
Dec 11, 2023
Merged

Conversation

Rickedb
Copy link
Owner

@Rickedb Rickedb commented Dec 11, 2023

Enhancements

  • Project now targets the latest LTS version .NET 8
  • Added some string optimizations using StringBuilder instead of += concatenation
  • Few syntax sugar improvements for using Enum instead of converting it to an int every time when accessing datafields
  • Some syntax "sugaring" for creating datafields list
  • MessageTemplates are now registered as lazy fields, so we only access them when discovering the responsible message template/mid
  • Quick lookup mid X message template dictionary changed to SortedDictionary for better performance

New Features

  • Added StandarizedRevision which handles when revision is set to zero, returning as rev 1

New Mids/New Revisions

  • Communication
    • Mid0004
      • Rev 2
  • Parameter Set
    • Mid0019
      • Rev 2
    • Mid2506 🆕
  • Job
    • Mid0033
      • Rev 5
    • Mid0140
      • Rev 4
  • Tightening
    • Mid0061
      • Rev 9
      • Rev 10
      • Rev 11
    • Mid0064
      • Rev 10
    • Mid0065
      • Rev 9
      • Rev 10
      • Rev 11
    • Mid0066
      • Rev 2
    • Mid0902 🆕
  • Multi Spindle
    • Mid0101
      • Rev 6
    • Mid0104 🆕
  • Result
    • Mid1201
      • Rev 2
      • Rev 3
    • Mid1202
      • Rev 2
  • Tool
    • Mid0704 🆕

Related issues

@Rickedb Rickedb self-assigned this Dec 11, 2023
@Rickedb Rickedb merged commit bd898b4 into master Dec 11, 2023
6 checks passed
@Rickedb Rickedb deleted the sr2.17 branch December 11, 2023 01:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant