Michael C Ryan

Results 18 comments of Michael C Ryan

Thanks, that’s great! Let me know if there's anything else I need to do. Just a heads-up—this PR is part of a series of similar checkers for other models. I...

I wanted to provide some context: I decided to merge a few of my branches since the code was relevant enough. I just added more sonar_model checkers and tests.

I agree with your perspective. My intention was not to remove the sonar model parameter, (it only helps echopype users to provide more information) but rather to explore the possibility...

@leewujung Acknowledged. Ill begin putting in these changes.

@leewujung My team has asked me to get this PR merged because the sonar-checking code is essential for something they’re working on. I suggested moving the code to their library,...

@leewujung I think there may have been some miscommunication, and I take full responsibility for that. Here’s my understanding: Originally, this PR aimed to enhance the functionality of the echodata...

Upon reviewing this issue, it seems the focus is more on finding solutions for enhancing maintenance and upgrading during the development phase rather than on operational improvements (though these are...

Hello, I’ve prepared some code on my fork and re-based it with the latest upstream , but I haven’t submitted the PR yet as my test environment isn’t fully ready....