Skip to content

[TIME] Add ASIL related requirements and rework existing - #3020

Merged
pawelrutkaq merged 13 commits into
eclipse-score:mainfrom
etas-contrib:bjoernatbosch_improve_time_requirements
Sep 4, 2026
Merged

[TIME] Add ASIL related requirements and rework existing#3020
pawelrutkaq merged 13 commits into
eclipse-score:mainfrom
etas-contrib:bjoernatbosch_improve_time_requirements

Conversation

@BjoernAtBosch

@BjoernAtBosch BjoernAtBosch commented Jun 23, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Requirements for Time feature:

  • Define safety-related stakeholder and feature requirements
    For that I mainly focused on two areas:
    • Requesting FFI for Time API users
    • Requesting the support of a reliable/safe time qualification (this requires support by the involved components/ECUs of the vehicle)
  • Fix some requirement types
  • Improve some time related stakeholder and feature requirements

This is a draft PR for now as it requires some discussion
Closes eclipse-score/time#33

BjoernAtBosch
BjoernAtBosch commented Jun 23, 2026
View reviewed changes
Comment thread docs/requirements/stakeholder/index.rst Outdated
@github-actions

github-actions Bot commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

The created documentation from the pull request is available at: docu-html

@BjoernAtBosch
BjoernAtBosch marked this pull request as ready for review June 26, 2026 13:08
lavrovvalera
lavrovvalera suggested changes Jul 9, 2026
View reviewed changes
Comment thread docs/features/time/docs/requirements/index.rst Outdated
Comment thread docs/features/time/docs/requirements/index.rst Outdated
Comment thread docs/features/time/docs/requirements/index.rst Outdated
Comment thread docs/requirements/stakeholder/index.rst Outdated
Comment thread docs/requirements/stakeholder/index.rst Outdated
pawelrutkaq
pawelrutkaq reviewed Jul 9, 2026
View reviewed changes
Comment thread docs/features/time/docs/requirements/index.rst Outdated
pawelrutkaq
pawelrutkaq reviewed Jul 9, 2026
View reviewed changes
Comment thread docs/features/time/requirements/index.rst
pawelrutkaq
pawelrutkaq reviewed Jul 9, 2026
View reviewed changes
Comment thread docs/requirements/stakeholder/index.rst
pawelrutkaq
pawelrutkaq reviewed Jul 9, 2026
View reviewed changes
Comment thread docs/requirements/stakeholder/index.rst Outdated
pawelrutkaq
pawelrutkaq reviewed Jul 9, 2026
View reviewed changes
Comment thread docs/requirements/stakeholder/index.rst
ryan-steel
ryan-steel reviewed Jul 10, 2026
View reviewed changes
Comment thread docs/features/time/docs/requirements/index.rst Outdated
BjoernAtBosch and others added 2 commits July 10, 2026 11:25
@BjoernAtBosch
Rephrase notes to no mention Autosar
9028ded
@BjoernAtBosch @ryan-steel
Apply suggestion from @ryan-steel
fd50b35 
Co-authored-by: Ryan Steel <str1yok@bosch.com>
Signed-off-by: Björn Hornburg <Bjoern.Hornburg@de.bosch.com>
@github-project-automation github-project-automation Bot moved this to In Progress in TIM - Time FT Jul 29, 2026
@anmittag

anmittag commented Jul 31, 2026

Copy link
Copy Markdown
Member

@BjoernAtBosch any update on the PR?

@anmittag

anmittag commented Aug 20, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

TLC 2026-08-20: @BjoernAtBosch a) branch is outdated b) review findings are not closed

Please adapt accordingly!

@BjoernAtBosch
Merge remote-tracking branch 'origin/main' into bjoernatbosch_improve…
93a5ef3 
…_time_requirements

# Conflicts:
#	docs/features/time/index.rst
#	docs/features/time/requirements/aou_req.rst
@github-actions

github-actions Bot commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Documentation preview for this pull request is available at:
pr-3020: https://eclipse-score.github.io/score/pr-3020/

@BjoernAtBosch

BjoernAtBosch commented Aug 21, 2026

Copy link
Copy Markdown
Contributor Author

Hi @lavrovvalera, @pawelrutkaq, @FScholPer, and @NEOatNHNG:
I've revised this PR due to your comments. Please re-review thoroughly.

There is an open discussion if an ASIL_B rating of an API requirement implies:

For the current proposal in review, I assumed the latter one which is the reason to:

  • withdraw feat_req__time__veh_time_reliable_accur_qual again (which then would be requested implicitly by the ASIL rating),
  • not marking the AbsTimeAPI as ASIL (because I don't want it to guarantee integrity for all its elements).

pawelrutkaq
pawelrutkaq reviewed Aug 28, 2026
View reviewed changes
Comment thread docs/requirements/stakeholder/index.rst
pawelrutkaq
pawelrutkaq reviewed Aug 28, 2026
View reviewed changes

@pawelrutkaq pawelrutkaq left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

for me fine

@pawelrutkaq
pawelrutkaq merged commit 4e05874 into eclipse-score:main Sep 4, 2026
4 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in TIM - Time FT Sep 4, 2026
@BjoernAtBosch
BjoernAtBosch deleted the bjoernatbosch_improve_time_requirements branch September 4, 2026 15:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pawelrutkaq pawelrutkaq pawelrutkaq left review comments
@anmittag anmittag anmittag approved these changes
@NEOatNHNG NEOatNHNG Awaiting requested review from NEOatNHNG
@antonkri antonkri Awaiting requested review from antonkri antonkri is a code owner
@FScholPer FScholPer Awaiting requested review from FScholPer FScholPer is a code owner
@qor-lb qor-lb Awaiting requested review from qor-lb qor-lb is a code owner
@johannes-esr johannes-esr Awaiting requested review from johannes-esr johannes-esr is a code owner
@arsibo arsibo Awaiting requested review from arsibo arsibo is a code owner
@ramceb ramceb Awaiting requested review from ramceb ramceb is a code owner
@4og 4og Awaiting requested review from 4og 4og is a code owner
@LittleHuba LittleHuba Awaiting requested review from LittleHuba LittleHuba is a code owner
@michaelsaborov michaelsaborov Awaiting requested review from michaelsaborov michaelsaborov is a code owner
@rmaddikery rmaddikery Awaiting requested review from rmaddikery rmaddikery is a code owner
@umaucher umaucher Awaiting requested review from umaucher umaucher is a code owner
+2 more reviewers
@ryan-steel ryan-steel ryan-steel left review comments
@lavrovvalera lavrovvalera lavrovvalera approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

@BjoernAtBosch BjoernAtBosch

Labels

None yet

Projects

TIM - Time FT
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Improvement: Clarify the ASIL requirements level

6 participants

@BjoernAtBosch @anmittag @lavrovvalera @FScholPer @pawelrutkaq @ryan-steel