Added new features with some improvments #5

Slayingripper · 2022-02-11T02:05:10Z

Added the following 👍

Integration with BMP085 sensors
Ability to use as a WX station
Profile for WX
Profile for Static station location
Changes APRS packet variables in WX to make it more readable
Added variables ready to be used with more advanced sensors (rain, wind etc) just uncomment.

The whole project would benefit from some refactoring hopefully I will get to it at some point

dj1an

Thanks for this PR!

I think some small Changes should be made to address issues for Users who do not have a BMP Sensor connected.

I haven't tried yet, but is the cubecell even initializing without a Sensor?

Some Sanitation or checks against the User provided fixed Position should be made in regard of many different GPS Dates and formatting.

Will try to optimize your code later and test with real Hardware.

dj1an · 2022-02-11T13:49:17Z

src/LoRa_APRS_Tracker.cpp

  delay(500);
  Serial.println("CubeCell LoRa APRS Tracker, DJ1AN");
  setup_display();
  VextON(); // activate RGB Pixel
  show_display("DJ1AN", "CubeCell", "LoRa APRS Tracker", 500);
-
+  bmp.begin();


should we check for WEATHER_DATA before starting BMP?

dj1an · 2022-02-11T13:49:37Z

src/LoRa_APRS_Tracker.cpp

+void telemetry() {
+  pinMode(Vext, OUTPUT);
+  digitalWrite(Vext, LOW);
+  Serial.begin(115200);


Serial was already started in Setup

Was having some issues before with the hardware, might have left that in by mistake when I was testing.

dj1an · 2022-02-11T13:50:11Z

src/LoRa_APRS_Tracker.cpp

+  pinMode(Vext, OUTPUT);
+  digitalWrite(Vext, LOW);
+  Serial.begin(115200);
+  if (!bmp.begin()) {


If no BMP connected we would spam the serial console.
Check for WEATHER_DATA?

Yes, that would seem more appropriate

dj1an · 2022-02-11T13:53:06Z

src/LoRa_APRS_Tracker.cpp

+    if (WEATHER_DATA == true) {
+
+      Serial.print("Weather data enabled");
+      aprsmsg = "!" + lat + DBEACON_OVERLAY + lng + DBEACON_SYMBOL + WIND_DIRECTION + "/" + WIND_SPEED + "g" + RAINX + "t" + TEMPERATURE + "h" + HUMIDITY + "b" + PRESSURE;


Maybe send out Weather as real APRS Telemetry messages in a future Version.

Can you elaborate what you mean real APRS Telemetry ? In this case it will show up on APRS.FI which only the values specified. When sending out "..." it will just ignore them, hence why I left them there so if you connect some more sophisticated sensor you could uncomment the top portion.

APRS has a Spec for Telemetry packets, not just adding text to packet comment.
This draws nice graphs on aprs.fi in the Telemetry section.

I have done so in my solar powered digipeater, see
https://github.com/dj1an/LoRa-APRS-Digipeater/blob/main/src/main.cpp#L401

Would be nice2have in future Versions, but for now we could leave it as it is

edit: ok i see. this is aprs wx format. Great!

For Weather Data there's the specific packet format that covers all this data.
Starting on Page 62: http://www.aprs.org/doc/APRS101.PDF

aprs.fi renders nice graphs specific to the weather and generates summary reports based on local stations.
https://aprs.fi/weather/a/EW8865

Slayingripper · 2022-02-11T14:27:13Z

Not sure how to implement sanitation checks since the user has to manually edit the code anyway. APRS will just give out an error anyway if LAT/LONG is not in the correct format.

Slayingripper · 2022-02-11T15:05:43Z

I would like to add the ability to use more sensors, like the BMP280 or something else, but I don't have the hardware to test it with. I might get my hands on some at some point and would try to add it in the form of a switch statement or something.

dj1an · 2022-02-11T15:32:27Z

Not sure how to implement sanitation checks since the user has to manually edit the code anyway. APRS will just give out an error anyway if LAT/LONG is not in the correct format.

yeah, will add an example Position in APRS lat/long format

I would like to add the ability to use more sensors, like the BMP280 or something else, but I don't have the hardware to test it with. I might get my hands on some at some point and would try to add it in the form of a switch statement or something.

great Idea.

dj1an · 2022-02-11T19:38:25Z

Please checkout the new wx Branch
I merged your Code and did some minor Changes.
As i have no BME Sensor wired at the Moment, i cannot Test the Temperature readout.

Can you provide a wiring diagram for Documentation?

There will be a bug with fixed Beaconing.
At the Moment it needs GPS Rx, even if we are in "Fixed" Mode with user defined Coordinates.

Slayingripper · 2022-02-11T20:34:11Z

I was searching for a fritzing model for the board but could not find one. I'll see if I can make one.
I'm powering the BMP180 sensor off the 3.3v pin
VCC -> VDD
GND -> GND
SCL -> SCL (located under the display ribbon)
SDA -> SDA (located under the display ribbon)

I will add support for the BMP280 later on when I get my hands on it.

Wx

…Tracker into dj1an-master

dj1an · 2024-08-01T16:46:57Z

I just got a BMP Sensor in my Hands.
The Code seems to work. Will merge it, Thank you! @Slayingripper .
73 DJ1AN

This reverts commit 4417811, reversing changes made to 6189f1e.

Slayingripper force-pushed the master branch from b1ed147 to 848e62a Compare February 11, 2022 09:40

Slayingripper added 2 commits February 11, 2022 10:10

Run clang formating

d3f00e7

Added WX functionality, fixed position profile

d032936

Slayingripper force-pushed the master branch from 848e62a to d032936 Compare February 11, 2022 10:16

dj1an reviewed Feb 11, 2022

View reviewed changes

dj1an added 2 commits February 11, 2022 20:25

changes for merging

54152c9

Merge branch 'Slayingripper-master' into wx

3363b4e

dj1an linked an issue Feb 12, 2022 that may be closed by this pull request

New Profile with fixed Position #4

Closed

Slayingripper added 9 commits February 13, 2022 20:39

Sensor Diagram and Clean Up

6c5fd66

Merge pull request #1 from dj1an/wx

42c874f

Wx

Merge branch 'master' of https://github.com/dj1an/Cubecell_LoRa_APRS_…

78330eb

…Tracker into dj1an-master

Made adding callsign easier

3b994ae

minor format fixes

be73a21

removed some redundunt comments

89df341

Merge branch 'dj1an-master'

4475a34

new fixes

433b2ab

fix

17b877d

dj1an merged commit 4417811 into dj1an:master Aug 1, 2024

dj1an added a commit that referenced this pull request Nov 11, 2024

Revert "Merge pull request #5 from Slayingripper/master"

79b0aec

This reverts commit 4417811, reversing changes made to 6189f1e.

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Added new features with some improvments #5

Added new features with some improvments #5

Slayingripper commented Feb 11, 2022

dj1an left a comment

dj1an Feb 11, 2022

dj1an Feb 11, 2022

Slayingripper Feb 11, 2022

dj1an Feb 11, 2022

Slayingripper Feb 11, 2022

dj1an Feb 11, 2022

Slayingripper Feb 11, 2022

dj1an Feb 11, 2022 •

edited

Loading

dkozel Feb 11, 2022

Slayingripper commented Feb 11, 2022

Slayingripper commented Feb 11, 2022

dj1an commented Feb 11, 2022

dj1an commented Feb 11, 2022

Slayingripper commented Feb 11, 2022

dj1an commented Aug 1, 2024

Added new features with some improvments #5

Added new features with some improvments #5

Conversation

Slayingripper commented Feb 11, 2022

dj1an left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

dj1an Feb 11, 2022 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Slayingripper commented Feb 11, 2022

Slayingripper commented Feb 11, 2022

dj1an commented Feb 11, 2022

dj1an commented Feb 11, 2022

Slayingripper commented Feb 11, 2022

dj1an commented Aug 1, 2024

dj1an Feb 11, 2022 •

edited

Loading