ambrop72 / aprinter

3D printer firmware written in C++
Other
143 stars 42 forks source link

PT100 support #29

Open mentaluproar opened 7 years ago

mentaluproar commented 7 years ago

Are there any plans to add support for the E3D PT100 kit?

ambrop72 commented 7 years ago

Hi, No support right now but it's easy, let me see if I can get it done in a few days.

mentaluproar commented 7 years ago

Wow. Thank you.

On Jun 6, 2017, 3:54 AM -0400, Ambroz Bizjak notifications@github.com, wrote:

Hi, No support right now but it's easy, let me see if I can get it done in a few days. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

ambrop72 commented 7 years ago

I implemented support for https://wiki.e3d-online.com/wiki/E3D_PT100_Amplifier_Documentation (I hope I got that right). I updated the online service (www.aprinter.eu) in case you were using that. To configure it, go to Configurations --> your configuration --> Heaters --> heater and set "Temperature Conversion" to "E3D PT100 Amplifier".

Implementation is based on a lookup table with linear interpolation and supports easily adding additional lookup table for other devices (I did not yet have any conversion based on a lookup table). Currently the lookup table has floats so it uses a few hundred bytes of program memory, but that should not be a problem in most cases.

mentaluproar commented 7 years ago

The PT100 amplifier is connected to an analog pin on the RAMPS board, but not the T pins. In my case, it’s on pin A3.

On Jun 7, 2017, 3:50 PM -0400, Ambroz Bizjak notifications@github.com, wrote:

I implemented support for https://wiki.e3d-online.com/wiki/E3D_PT100_Amplifier_Documentation (I hope I got that right). I updated the online service (www.aprinter.eu) in case you were using that. To configure it, go to Configurations --> your configuration --> Heaters --> heater and set "Temperature Conversion" to "E3D PT100 Amplifier". Implementation is based on a lookup table with linear interpolation and supports easily adding additional lookup table for other devices (I did not yet have any conversion based on a lookup table). Currently the lookup table has floats so it uses a few hundred bytes of program memory, but that should not be a problem in most cases. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

ambrop72 commented 7 years ago

Should not be a problem. The B/T/etc are just definitions in the corresponding Board configuration (Board --> your_board --> Analog Inputs). You can change an existing Analog Input definition (unless it's used for something else already) or define a new one, with the correct pin.

On Wed, Jun 7, 2017 at 11:48 PM, mentaluproar notifications@github.com wrote:

The PT100 amplifier is connected to an analog pin on the RAMPS board, but not the T pins. In my case, it’s on pin A3.

On Jun 7, 2017, 3:50 PM -0400, Ambroz Bizjak notifications@github.com, wrote:

I implemented support for https://wiki.e3d-online.com/ wiki/E3D_PT100_Amplifier_Documentation (I hope I got that right). I updated the online service (www.aprinter.eu) in case you were using that. To configure it, go to Configurations --> your configuration --> Heaters --> heater and set "Temperature Conversion" to "E3D PT100 Amplifier". Implementation is based on a lookup table with linear interpolation and supports easily adding additional lookup table for other devices (I did not yet have any conversion based on a lookup table). Currently the lookup table has floats so it uses a few hundred bytes of program memory, but that should not be a problem in most cases. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/ambrop72/aprinter/issues/29#issuecomment-306935112, or mute the thread https://github.com/notifications/unsubscribe-auth/ACgTsaG9h6oJYOfyfWSAS-9n3NNl-q1oks5sBxqRgaJpZM4NwwS5 .

mentaluproar commented 7 years ago

Thanks.  I tried it out but have been unable to get it to compile.  Did I make a mistake with the attached profile?

On Jun 7, 2017, 8:03 PM -0400, Ambroz Bizjak notifications@github.com, wrote:

Should not be a problem. The B/T/etc are just definitions in the corresponding Board configuration (Board --> your_board --> Analog Inputs). You can change an existing Analog Input definition (unless it's used for something else already) or define a new one, with the correct pin.

On Wed, Jun 7, 2017 at 11:48 PM, mentaluproar notifications@github.com wrote:

The PT100 amplifier is connected to an analog pin on the RAMPS board, but not the T pins. In my case, it’s on pin A3.

On Jun 7, 2017, 3:50 PM -0400, Ambroz Bizjak notifications@github.com, wrote:

I implemented support for https://wiki.e3d-online.com/ wiki/E3D_PT100_Amplifier_Documentation (I hope I got that right). I updated the online service (www.aprinter.eu) in case you were using that. To configure it, go to Configurations --> your configuration --> Heaters --> heater and set "Temperature Conversion" to "E3D PT100 Amplifier". Implementation is based on a lookup table with linear interpolation and supports easily adding additional lookup table for other devices (I did not yet have any conversion based on a lookup table). Currently the lookup table has floats so it uses a few hundred bytes of program memory, but that should not be a problem in most cases. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/ambrop72/aprinter/issues/29#issuecomment-306935112, or mute the thread https://github.com/notifications/unsubscribe-auth/ACgTsaG9h6oJYOfyfWSAS-9n3NNl-q1oks5sBxqRgaJpZM4NwwS5 .

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

ambrop72 commented 7 years ago

What is the error? Can you provide the exported configuration file?

mentaluproar commented 7 years ago

The config file is attached to the previous message.  I just got a big long list of errors.

On Jun 8, 2017, 3:28 AM -0400, Ambroz Bizjak notifications@github.com, wrote:

What is the error? Can you provide the exported configuration file? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

ambrop72 commented 7 years ago

@mentaluproar There is no attachment that I can see.