use coincidenceEventPolicy for WeightedEventTimeBlocks?

Open
  • Is something wrong?
#237 0 comments 0 reactions 0 assignees View on GitHub

Maintainers usually reply within 1 day

Nobody has claimed this yet.

Assessment

Difficulty
3/5
Estimated time
1-2 days
Newbie friendliness
48/100
Issue type
Feature
Clarity
Mostly clear
Activity status
Active
Tech stack
python, yaml
Domain
backend

Research direction

Read the referenced model/Protocol.yml line 16 and find where coincidenceEventPolicy and WeightedEventTimeBlocks are defined. Check how the policy is represented and whether it can include WeightedEventTimeBlocks; the issue asks whether this should be added but gives no acceptance criteria, so clarify the intended behavior before making a change.

Written by the indexing model from the issue text.

Description

question

At the moment, we have
https://github.com/ETSInitiative/PETSIRD/blob/8f2e763e9c26d6143e096bb1d3a72f2bf4c2189e/model/Protocol.yml#L16
It might be better to add this to the policy.

Dominant language
Python
Stars
15
Forks
18
Avg merge
1d 11h
Merged PRs (30d)
12

Getting set up

Open in Codespaces

Starts the project's dev container in your browser, under your own GitHub account.

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from ETSInitiative/PETSIRD

All issues in ETSInitiative/PETSIRD

Similar issues

More Python issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.