Skip to content

Conversation

@carsten-kuebler
Copy link
Contributor

Basic definition for use of US sensors. New detections and additional information for detected objects

Basic definition for use of US sensors. New detections and additional information for detected objects
@carsten-kuebler carsten-kuebler added the FeatureRequest Proposals which enhance the interface or add additional features. label Dec 9, 2018
@carsten-kuebler carsten-kuebler added this to the v3.1.0 milestone Dec 9, 2018
@carsten-kuebler carsten-kuebler requested a review from a user December 9, 2018 18:10
@carsten-kuebler
Copy link
Contributor Author

@CarloVanDriestenBMW Figures will be updated to .svg figures.

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Needs rework!


// Beginning blockage, sensor specific for front radar
//
EXTENDED_QUALIFIER_BEBINNING_BLOCKAGE = 12;
Copy link

Choose a reason for hiding this comment

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

typo?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok

optional Identifier sensor_id = 7;

// The extended qualifier describes the reason (not the effect) why the
// event data qualifier \c #data_qualifier is reduced or not available.
Copy link

Choose a reason for hiding this comment

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

[...] event data, indicated by the qualifier \c # data_qualifier, is reduced or not available.

// The extended qualifier describes the reason (not the effect) why the
// event data qualifier \c #data_qualifier is reduced or not available.
//
// \note required for USS
Copy link

Choose a reason for hiding this comment

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

Capital letter at the beginning and dot at the end? Please check all comments.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I removed the note.


//
// Extended qualifier describes the reason (not the effect) why the event
// data qualifier \c #data_qualifier is reduced or not available.
Copy link

Choose a reason for hiding this comment

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

see above

//
EXTENDED_QUALIFIER_OTHER = 1;

// Normal operation mode
Copy link

Choose a reason for hiding this comment

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

dots missing? check all comments

//
optional TrilaterationStatus trilateration_status = 3;

// Describes the general moving direction of the feature.
Copy link

Choose a reason for hiding this comment

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

of the feature or detection?

//
optional Trend trend = 4;

// Ultrasonic signalway. Snder to receiver.
Copy link

Choose a reason for hiding this comment

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

typo

// UltrasonicDetection. \c UltrasonicDetection refer to DetectedXXX by \c
// UltrasonicDetection::object_id.
//
repeated Signalway singalway = 5;
Copy link

Choose a reason for hiding this comment

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

typo

//
TRILATERATION_STATUS_OTHER = 1;

// No trilateration used
Copy link

Choose a reason for hiding this comment

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

dots missing

//
TREND_CONSTANT_APPROACHING = 2;

// \note There is no definition
Copy link

Choose a reason for hiding this comment

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

really? Please get in contact with the one who requested this property to get a definition.

Carsten Kuebler and others added 6 commits December 14, 2018 12:50
Remove Typos.
Improve Documentation
Vector - images may be to large :-)
Small figures for documentation
Insert .svg s and add some Documentation
Update Trend Definiton
@carsten-kuebler
Copy link
Contributor Author

@CarloVanDriestenBMW Update Trend definiton

@ghost ghost merged commit c9417ee into master Dec 14, 2018
@ghost ghost deleted the feature/ultrasonicsensor branch December 14, 2018 15:12
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

FeatureRequest Proposals which enhance the interface or add additional features.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants