Skip to content
This repository was archived by the owner on Dec 15, 2023. It is now read-only.

[DEM-943] Feature UHFLI integration #573

Merged
merged 40 commits into from
Jun 26, 2019

Conversation

CumulonimbusCalvus
Copy link
Contributor

@CumulonimbusCalvus CumulonimbusCalvus commented May 23, 2019

⛈ 🍝

@CumulonimbusCalvus CumulonimbusCalvus force-pushed the feature/DEM-943/UHFLI_integration branch 3 times, most recently from d4bc1df to 6b8d994 Compare May 24, 2019 15:12
@CumulonimbusCalvus CumulonimbusCalvus force-pushed the feature/DEM-943/UHFLI_integration branch from 6b8d994 to 75e5a93 Compare May 27, 2019 15:33
@peendebak peendebak force-pushed the feature/DEM-943/UHFLI_integration branch from 75e5a93 to 3d1d053 Compare May 29, 2019 13:10
@CumulonimbusCalvus CumulonimbusCalvus force-pushed the feature/DEM-943/UHFLI_integration branch 2 times, most recently from ea8f53a to f8ef796 Compare June 13, 2019 05:55
Copy link
Contributor

@qSaevar qSaevar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Almost all small comments about form and style

@CumulonimbusCalvus CumulonimbusCalvus force-pushed the feature/DEM-943/UHFLI_integration branch 3 times, most recently from 78845ea to 90d1b37 Compare June 21, 2019 12:57
@CumulonimbusCalvus CumulonimbusCalvus force-pushed the feature/DEM-943/UHFLI_integration branch from 50e4ebb to 5ce0c56 Compare June 25, 2019 12:15
Copy link
Contributor

@qSaevar qSaevar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some suggestions, you can take them or not.

width = [upwards_edge_pixels_x/pixels_x, upwards_edge_pixels_y/pixels_y]
resolution = [pixels_x, pixels_y]


def create_dummy_data_array(width: float, sawteeth_count: int, channel_index: int = 1, trace_number: int = 1):
def create_dummy_data_array(width: float, processing: str, sawteeth_count: int, channel_index: int = 1, trace_number: int = 1) -> DataArray:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This line is to long, and a missing blank line above

CumulonimbusCalvus and others added 23 commits June 26, 2019 10:37
@CumulonimbusCalvus CumulonimbusCalvus merged commit beaedca into dev Jun 26, 2019
@CumulonimbusCalvus CumulonimbusCalvus deleted the feature/DEM-943/UHFLI_integration branch August 16, 2019 15:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants