Skip to content

Update latest changes from dev#14

Merged
fbaptista merged 23 commits intomainfrom
dev
Oct 6, 2021
Merged

Update latest changes from dev#14
fbaptista merged 23 commits intomainfrom
dev

Conversation

@fbaptista
Copy link
Copy Markdown

@fbaptista fbaptista commented Sep 20, 2021

@fbaptista fbaptista changed the title WIP: Update latest changes from dev Update latest changes from dev Oct 5, 2021
Comment thread test/test.py Outdated
Co-authored-by: franpandol <pandol.francisco@gmail.com>
@franpandol
Copy link
Copy Markdown

franpandol commented Oct 6, 2021

Solo vi dos cosas un toque raras, primero que hay variables en mayúscula, no es un error de sintaxis pero es raro en python

Kperf = KperfData(rpc)

y lo otro que es justo en el mismo lugar, es que esa variable se declara después de un método interno que la usa. Creo que en runtime no debería dar problemas, me pareció un toque extraño pero no se si está hecho asi a propósito.

@fbaptista fbaptista merged commit 738e1cf into main Oct 6, 2021
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.

5 participants