Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implemented antenna calibration API with backwards compatibility #281

Merged
merged 6 commits into from
Jun 12, 2019
Merged

Implemented antenna calibration API with backwards compatibility #281

merged 6 commits into from
Jun 12, 2019

Conversation

Anacron-sec
Copy link
Contributor

Q A
Bug fix? no
New feature? yes
Doc update? no
BC breaks? no
Deprecations? no
Fixed tickets #...

This is a simple improvement based on pull request #244 but doesn't break backwards compatibility.

@Rotzbua
Copy link
Collaborator

Rotzbua commented Aug 30, 2018

@an4cr0n So #244 is obsolete with this pull request?

@Rotzbua Rotzbua self-requested a review August 30, 2018 21:18
@Sonic0
Copy link
Contributor

Sonic0 commented Aug 30, 2018

In my opinion, yes. We wait for @an4cr0n

@Anacron-sec
Copy link
Contributor Author

Anacron-sec commented Aug 31, 2018

@Rotzbua depends if you want to include the python scripts in the other one, but i think they don't belong here.
P.S. Don't merge this as we recently improved the code getting rid of _antennaCalibrated variable, i'll update this ASAP.

@Rotzbua Rotzbua merged commit cb2e64d into thotro:master Jun 12, 2019
@Rotzbua
Copy link
Collaborator

Rotzbua commented Jun 12, 2019

@Sonic0 @an4cr0n thanks for your pull request 👍 , i merged them before i basically abandon this lib, i hope your code helps others

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.

3 participants