robotpy/robotpy-wpilib

Tim Winters

@Twinters007

if we added it just to ctre_talon when would that raise an error? sim or only on the roborio?

Dustin Spicuzza

@virtuald

you would only want it to be in cantalon_sim

not in the user class

Tim Winters

@Twinters007

oh ok

Dustin Spicuzza

@virtuald

because the C++ thing is already doing typechecking

Tim Winters

@Twinters007

would that be enough then? why do we need to check the headers?

Dustin Spicuzza

@virtuald

well

what if you put in 'int' and the function takes a 'string'

there are a lot of functions

Tim Winters

@Twinters007

like if I put the wrong type check in?

Dustin Spicuzza

@virtuald

yes

Tim Winters

@Twinters007

that'd be bad

Dustin Spicuzza

@virtuald

yes

Tim Winters

@Twinters007

so what would checking the headers do? Check my types vs the header, then say 'you're good, check what the user put?'

Dustin Spicuzza

@virtuald

it's something I would add to the unit tests

right now we have a thing in wpilib/tests/test_specification.py

and it calls wpilib/tests/spec_scanners/hal_scanner.py

obviously we wouldn't want to do these checks at runtime, but it makes for a great unit test

it's one of the reasons nobody ever reports segfaults for robotpy -- because we have automation that ensures we don't mess up things like that :+1:

Tim Winters

@Twinters007

have you used enforce?

Dustin Spicuzza

@virtuald

never

Tim Winters

@Twinters007

Dustin Spicuzza

@virtuald

just found it on the googles

Tim Winters

@Twinters007

I'm trying to make pydev ignore the colon

Dustin Spicuzza

@virtuald

there might be better alternatives

oh, maybe update your pydev then

the new one has support for new grammars

Tim Winters

@Twinters007

updating

I have the latest one..

Dustin Spicuzza

@virtuald

maybe set your project grammar to 3.6?

Tim Winters

@Twinters007

https://gist.github.com/avidal/1252523

hmm

lemme check that

yeah it was set to 2.7

Dustin Spicuzza

@virtuald

that would do it

David

@auscompgeek

it does

this isn't javascript

Tim Winters

@Twinters007

yeah idk what I was thinking

Dustin Spicuzza

@virtuald

@/all RobotPy 2017.0.3 released with timing fix for IterativeRobot

David

@auscompgeek

why does gitter keep sending "virtuald pushed 0 commit(s)"

Dustin Spicuzza

@virtuald

doesn't say it here

Dustin Spicuzza

@virtuald

I really like the unified documentation site

anyways, heading to bed. nite.

Tim Winters

@Twinters007

A can you make it the stable version?

Dustin Spicuzza

@virtuald

FYI: I don't expect much RobotPy activity out of myself Friday/Saturday... but we'll see what happens

Tim Winters

@Twinters007

You have to go to the 'latest' version to get the uniformity

R Shupe

@shipshupe

o/, also liking the unified doc site. We'll be working friday/sat, maybe give a magicbot refactor a try :)

Dustin Spicuzza

@virtuald

the stable site has the uniformity too

nite all

People

Repo info

Activity