Skip to content

prevent using python v314#200

Open
g-pechorin wants to merge 2 commits into
mainfrom
not-v314
Open

prevent using python v314#200
g-pechorin wants to merge 2 commits into
mainfrom
not-v314

Conversation

@g-pechorin

@g-pechorin g-pechorin commented Mar 27, 2026

Copy link
Copy Markdown
Contributor
🦋 Bug Fix

PR Description

There is some issue using the pyscog libraries with python v3.14 (and later?)
This change sets an upper limit on the python version to ensure we're using [3.10, 3.14)

Related Issues or other material

Related #
Closes #

Screenshots, example outputs/behaviour etc.

✅ Added/updated tests?

  • [] This PR contains relevant tests / Or doesn't need to per the below explanation

[optional] What gif best describes this PR or how it makes you feel?

alt_text

@g-pechorin g-pechorin marked this pull request as ready for review March 27, 2026 10:23
@github-actions

Copy link
Copy Markdown

Coverage

Coverage Report
FileStmtsMissCoverMissing
carrottransform
   __init__.py201525%16–17, 20, 22, 25, 28–30, 33–37, 39, 41
   _version.py5260%5–6
carrottransform/cli
   __init__.py00100% 
carrottransform/cli/subcommands
   __init__.py00100% 
   run.py1591789%92–93, 105–107, 132, 165–167, 291, 299, 334–337, 383, 393
   run_v2.py20480%55, 57–59
carrottransform/tools
   __init__.py40100% 
   args.py1462880%96–97, 106–109, 133, 150, 154, 163, 168, 178, 181, 186, 197–199, 206, 212, 214, 222, 230, 260, 273, 283, 288, 331, 335
   at_path.py80100% 
   concept_helpers.py21195%27
   core.py69297%118, 126
   date_helpers.py45588%24–25, 28, 65, 70
   db.py210100% 
   file_helpers.py532749%18–20, 35, 37–41, 50–52, 57–61, 76, 78–80, 83–85, 87, 91, 96
   logger.py120100% 
   mapping_types.py200100% 
   mappingrules.py238597%120, 377, 380–382
   metrics.py1394666%129–134, 136, 143–156, 159, 168, 174–179, 262, 264–270, 286–288, 291, 298, 300, 303, 349–350
   omopcdm.py1531391%45–47, 131, 155, 160, 172–175, 181, 199, 205
   orchestrator.py1751094%87–88, 98, 101, 118–120, 205, 370, 372
   outputs.py1766364%50, 73, 125, 151–152, 169–171, 174–177, 180–183, 186, 189–190, 192–196, 198, 201, 204, 207, 211–212, 215, 217, 219, 221–222, 225–227, 230–232, 241–242, 251–253, 267, 281, 283, 290, 292–295, 297, 309–310, 315–316, 318–320, 331
   person_helpers.py872373%17–18, 20–21, 23–24, 85, 98, 102, 133–134, 147, 160–163, 166–168, 170–171, 174–175
   record_builder.py1822287%26, 56, 71, 76, 79, 88, 100, 108–109, 183, 210, 259, 300–301, 315, 323, 327, 341, 348, 365, 392–393
   sources.py1555266%27, 47, 50–51, 54, 64, 114–115, 184–188, 190–191, 193–194, 196, 199–200, 203–205, 207–212, 214, 218–220, 222, 228, 230–231, 233–234, 236, 239–240, 243–245, 247–252, 254
   stream_helpers.py340100% 
   types.py640100% 
   validation.py30196%51
TOTAL203633683% 

Tests Skipped Failures Errors Time
87 1 💤 0 ❌ 0 🔥 10.596s ⏱️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant