-
Notifications
You must be signed in to change notification settings - Fork 3
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
Variant Import #284
Variant Import #284
Conversation
@elpiankova, I sold my soul to devil trying to understand why I had no sessions
Temporary countermeasure against #256
Fixed #262 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It works now. We can make new issue for quicklooks. @landswellsong, I'm ready to press "Approve"
Ugh so press it :D |
# TODO: check that directory exists properly | ||
# TODO: any more elegant way? re-yield or smth | ||
for v in ftp.check(): | ||
yield v |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@landswellsong Объяснишь, что ты имел в виду про "элегантный вид"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@landswellsong Леша, если можешь объясни нам этот TODO
перевірила CesiumMap - в point[2] щось приходить |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Вперед к другим задачам:) Один TODO про ftp я отметила, чтоб спросить у Леши. Если что тоже откроем новое issue
relevant last years commits minus hotfixes and FFT