Allow VectorSource to take an N-vector
Nobody has claimed this yet.
- Dominant language
- C++
- Stars
- 0
- Forks
- 0
- PR merge metrics
- No merged PRs in 30d
Description
When we added in the VectorSource (#502), it could only output a 1-vector. However, it should be more generic than that and we should be able to make it an N-vector. A relatively straightforward way to do this is:
1. Make the VectorSource constructor take a map of string -> vector indices that the VectorSource holds onto.
2. Make the Set() method take a string and a value; it looks up the correct index of the vector to set based on the string, and sets it.
3. When setting the data into the context, it iterates over all things in the map and sets them all on the output port.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating the VectorSource constructor, Set() method, and the code that writes data into the context output port. Update those entry points to use string-to-vector-index mappings and verify that every mapped value is written to the output vector.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- autonomous-driving
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100