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

server/datagramReceived: fix undefined variable port #96

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

iblislin
Copy link

We tried to export the data from Phillips FM20 (Fetal Monitor), but ran into an error about the undefined variable.


Sponsored by: CMUH (China Medical University Hospital)

@iblislin
Copy link
Author

Hi @doismellburning ,

could you review this patch?

@doismellburning
Copy link
Member

My apologies for the error and for the delay! As you may have noticed, this isn't exactly being proactively maintained, but I may try to find some cycles to bring it a little more up to date (though it will be trickier now I don't own the relevant hardware)

But yes, well spotted, I broke things in #55, thanks for the fix!

@doismellburning
Copy link
Member

I'm going to need to sort out the build configuration here before I can merge this - it was using the free Travis CI tier for testing, which has now been switched off, but I'll see what I can do!

@iblislin
Copy link
Author

iblislin commented May 15, 2024

As you may have noticed, this isn't exactly being proactively maintained, but I may try to find some cycles to bring it a little more up to date

We are planning to build a new monitoring system for our delivery rooms, where are equipped with lots of FM20.
This repo is a good starting point for us. We appreciate this works. :)

BTW, are there any license info for this project?

Sponsored by: CMUH (China Medical University Hospital)
@doismellburning
Copy link
Member

Good question regarding license. Personally I'm happy with anything appropriately Free, but ultimately I think this is a question for @JMathiszig-Lee as Mr Dr Somno!

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.

2 participants