v3.1.4 - Fixed leap year bug, various other minor fixes

.github/workflows/main.yml

@@ -3,20 +3,20 @@ name: Look4Sat CI
 on:
   push:
     tags:
-      - '*'
+      - v**
 
 jobs:
   deploy:
     runs-on: ubuntu-latest
     steps:
-      - name: Checkout latest commit
-        uses: actions/checkout@v3
+      - name: Checkout Latest Commit
+        uses: actions/checkout@v4
 
       - name: Setup java environment
-        uses: actions/setup-java@v3
+        uses: actions/setup-java@v4
         with:
           distribution: 'temurin'
-          java-version: '11'
+          java-version: '17'
 
       - name: Assemble APK and Bundle
         run: |
@@ -34,7 +34,7 @@ jobs:
           alias: ${{ secrets.KEY_ALIAS }}
           keyPassword: ${{ secrets.KEY_PASSWORD }}
         env:
-          BUILD_TOOLS_VERSION: "33.0.0"
+          BUILD_TOOLS_VERSION: "34.0.0"
 
       - name: Sign Bundle
         uses: r0adkll/sign-android-release@v1
@@ -46,25 +46,25 @@ jobs:
           alias: ${{ secrets.KEY_ALIAS }}
           keyPassword: ${{ secrets.KEY_PASSWORD }}
         env:
-          BUILD_TOOLS_VERSION: "33.0.0"
+          BUILD_TOOLS_VERSION: "34.0.0"
 
-      - name: Rename APK
+      - name: Rename APK and Bundle
         run: |
           mv ${{steps.sign_apk.outputs.signedReleaseFile}} app/build/outputs/apk/release/look4sat.apk
+          mv ${{steps.sign_bundle.outputs.signedReleaseFile}} app/build/outputs/apk/release/look4sat.aab
 
       - name: Deploy Bundle to Play Store
-        uses: r0adkll/upload-google-play@v1.0.17
+        uses: r0adkll/upload-google-play@v1
         with:
           serviceAccountJsonPlainText: ${{secrets.SERVICE_ACCOUNT_JSON}}
           packageName: com.rtbishop.look4sat
+          releaseFiles: app/build/outputs/apk/release/look4sat.aab
           track: production
-          mappingFile: app/build/outputs/mapping/release/mapping.txt
           whatsNewDirectory: fastlane/metadata/android/en-US/whatsnew
-          releaseFiles: ${{steps.sign_bundle.outputs.signedReleaseFile}}
 
       - name: Create release
         uses: ncipollo/release-action@v1
         with:
-          artifacts: app/build/outputs/apk/release/look4sat.apk
+          artifacts: "app/build/outputs/apk/release/look4sat.apk,app/build/outputs/apk/release/look4sat.aab"
           bodyFile: fastlane/metadata/android/en-US/whatsnew/whatsnew-en-US
           token: ${{ secrets.RELEASE_TOKEN }}

app/build.gradle

@@ -7,16 +7,14 @@ plugins {
 }
 
 android {
-    compileSdkVersion 33
-    buildToolsVersion '33.0.0'
     namespace 'com.rtbishop.look4sat'
-
+    compileSdk 34
     defaultConfig {
         applicationId "com.rtbishop.look4sat"
         minSdk 21
-        targetSdk 33
-        versionCode 313
-        versionName '3.1.3'
+        targetSdk 34
+        versionCode 314
+        versionName '3.1.4'
 
         testInstrumentationRunner "androidx.test.runner.AndroidJUnitRunner"
 
@@ -43,11 +41,11 @@ android {
         kotlinCompilerExtensionVersion compose_compiler_version
     }
     compileOptions {
-        sourceCompatibility JavaVersion.VERSION_11
-        targetCompatibility JavaVersion.VERSION_11
+        sourceCompatibility JavaVersion.VERSION_17
+        targetCompatibility JavaVersion.VERSION_17
     }
     kotlinOptions {
-        jvmTarget = JavaVersion.VERSION_11.toString()
+        jvmTarget = JavaVersion.VERSION_17.toString()
     }
     testOptions {
         animationsDisabled true
@@ -56,7 +54,7 @@ android {
 }
 
 kotlin {
-    jvmToolchain(11)
+    jvmToolchain(17)
 }
 
 dependencies {

app/proguard-rules.pro

@@ -0,0 +1,11 @@
+# Please add these rules to your existing keep rules in order to suppress warnings.
+# This is generated automatically by the Android Gradle plugin.
+-dontwarn org.bouncycastle.jsse.BCSSLParameters
+-dontwarn org.bouncycastle.jsse.BCSSLSocket
+-dontwarn org.bouncycastle.jsse.provider.BouncyCastleJsseProvider
+-dontwarn org.conscrypt.Conscrypt$Version
+-dontwarn org.conscrypt.Conscrypt
+-dontwarn org.conscrypt.ConscryptHostnameVerifier
+-dontwarn org.openjsse.javax.net.ssl.SSLParameters
+-dontwarn org.openjsse.javax.net.ssl.SSLSocket
+-dontwarn org.openjsse.net.ssl.OpenJSSE

app/src/main/java/com/rtbishop/look4sat/framework/data/LocalRadioSource.kt

@@ -17,10 +17,10 @@
  */
 package com.rtbishop.look4sat.framework.data
 
-import com.rtbishop.look4sat.domain.data.ILocalRadioSource
-import com.rtbishop.look4sat.framework.data.dao.RadiosDao
 import com.rtbishop.look4sat.domain.model.SatRadio as DomainRadio
 import com.rtbishop.look4sat.framework.model.SatRadio as FrameworkRadio
+import com.rtbishop.look4sat.domain.data.ILocalRadioSource
+import com.rtbishop.look4sat.framework.data.dao.RadiosDao
 
 class LocalRadioSource(private val radiosDao: RadiosDao) : ILocalRadioSource {
 
@@ -31,6 +31,7 @@ class LocalRadioSource(private val radiosDao: RadiosDao) : ILocalRadioSource {
     }
 
     override suspend fun insertRadios(radios: List<DomainRadio>) {
+        radiosDao.deleteRadios()
         radiosDao.insertRadios(radios.toFrameworkRadios())
     }
 

app/src/main/java/com/rtbishop/look4sat/injection/BaseModule.kt

@@ -28,7 +28,12 @@ import com.rtbishop.look4sat.domain.data.DataRepository
 import com.rtbishop.look4sat.domain.predict.SatelliteManager
 import com.rtbishop.look4sat.framework.LocationManager
 import com.rtbishop.look4sat.framework.SettingsManager
-import com.rtbishop.look4sat.framework.data.*
+import com.rtbishop.look4sat.framework.data.FileDataSource
+import com.rtbishop.look4sat.framework.data.LocalDatabase
+import com.rtbishop.look4sat.framework.data.LocalEntrySource
+import com.rtbishop.look4sat.framework.data.LocalRadioSource
+import com.rtbishop.look4sat.framework.data.MIGRATION_1_2
+import com.rtbishop.look4sat.framework.data.RemoteDataSource
 import com.rtbishop.look4sat.presentation.radarScreen.BTReporter
 import com.rtbishop.look4sat.utility.DataParser
 import com.rtbishop.look4sat.utility.DataReporter
@@ -37,11 +42,11 @@ import dagger.Provides
 import dagger.hilt.InstallIn
 import dagger.hilt.android.qualifiers.ApplicationContext
 import dagger.hilt.components.SingletonComponent
+import javax.inject.Singleton
 import kotlinx.coroutines.CoroutineScope
 import kotlinx.coroutines.Dispatchers
 import kotlinx.coroutines.SupervisorJob
 import okhttp3.OkHttpClient
-import javax.inject.Singleton
 
 @Module
 @InstallIn(SingletonComponent::class)

app/src/main/java/com/rtbishop/look4sat/presentation/radarScreen/RadarViewModel.kt

@@ -19,7 +19,11 @@ package com.rtbishop.look4sat.presentation.radarScreen
 
 import android.hardware.GeomagneticField
 import android.util.Log
-import androidx.lifecycle.*
+import androidx.lifecycle.LiveData
+import androidx.lifecycle.MutableLiveData
+import androidx.lifecycle.ViewModel
+import androidx.lifecycle.liveData
+import androidx.lifecycle.viewModelScope
 import com.rtbishop.look4sat.domain.IDataRepository
 import com.rtbishop.look4sat.domain.ISatelliteManager
 import com.rtbishop.look4sat.domain.ISettingsManager
@@ -33,10 +37,10 @@ import com.rtbishop.look4sat.utility.DataReporter
 import com.rtbishop.look4sat.utility.round
 import com.rtbishop.look4sat.utility.toDegrees
 import dagger.hilt.android.lifecycle.HiltViewModel
+import javax.inject.Inject
 import kotlinx.coroutines.delay
 import kotlinx.coroutines.isActive
 import kotlinx.coroutines.launch
-import javax.inject.Inject
 
 @HiltViewModel
 class RadarViewModel @Inject constructor(
@@ -107,8 +111,8 @@ class RadarViewModel @Inject constructor(
             if (settings.getRotatorEnabled()) {
                 val server = settings.getRotatorServer()
                 val port = settings.getRotatorPort().toInt()
-                val azimuth = satPos.azimuth.toDegrees().round(1)
-                val elevation = satPos.elevation.toDegrees().round(1)
+                val azimuth = satPos.azimuth.toDegrees().round(2)
+                val elevation = satPos.elevation.toDegrees().round(2)
                 reporter.reportRotation(server, port, azimuth, elevation)
             }
             _passData.postValue(RadarData(satPos, track))

app/src/main/res/values/strings.xml

@@ -69,12 +69,12 @@
     <string name="pass_satName">%s</string>
     <string name="pass_satId">Id:%d</string>
     <string name="pass_aosAz">AOS - %.1f°</string>
-    <string name="pass_elevation">Elevation: %.1f°</string>
+    <string name="pass_elevation">ELEV: %.1f°</string>
     <string name="pass_los">%.1f° - LOS</string>
-    <string name="pass_altitude">Altitude: %.0f km</string>
+    <string name="pass_altitude">ALT: %.0f km</string>
     <string name="pass_placeholder"> - - : - - </string>
-    <string name="pass_startTime">HH:mm:ss - EEE</string>
-    <string name="pass_endTime">EEE - HH:mm:ss</string>
+    <string name="pass_startTime">HH:mm:ss - MMM dd</string>
+    <string name="pass_endTime">dd MMM - HH:mm:ss</string>
 
     <string name="radar_az_text">Azimuth</string>
     <string name="radar_az_value">%.1f°</string>

base/build.gradle

@@ -3,7 +3,7 @@ plugins {
 }
 
 kotlin {
-    jvmToolchain(11)
+    jvmToolchain(17)
 }
 
 dependencies {

base/src/main/java/com/rtbishop/look4sat/domain/data/IRemoteDataSource.kt

@@ -21,7 +21,7 @@ import java.io.InputStream
 
 interface IRemoteDataSource {
 
-    val radioApi: String get() = "https://db.satnogs.org/api/transmitters/?format=json"
+    val radioApi: String get() = "https://db.satnogs.org/api/transmitters/?format=json&status=active"
 
     suspend fun getDataStream(url: String): InputStream?
 }

base/src/main/java/com/rtbishop/look4sat/utility/DataParser.kt

@@ -19,48 +19,46 @@ package com.rtbishop.look4sat.utility
 
 import com.rtbishop.look4sat.domain.model.SatRadio
 import com.rtbishop.look4sat.domain.predict.OrbitalData
+import java.io.InputStream
+import kotlin.math.pow
 import kotlinx.coroutines.CoroutineDispatcher
 import kotlinx.coroutines.withContext
 import org.json.JSONArray
 import org.json.JSONObject
-import java.io.InputStream
-import kotlin.math.pow
 
 class DataParser(private val defaultDispatcher: CoroutineDispatcher) {
 
-    suspend fun parseCSVStream(csvStream: InputStream): List<OrbitalData> =
-        withContext(defaultDispatcher) {
-            val parsedItems = mutableListOf<OrbitalData>()
-            csvStream.bufferedReader().useLines { lines ->
-                lines.forEachIndexed { index, line ->
-                    if (index != 0) {
-                        val values = line.split(",")
-                        parseCSV(values)?.let { tle -> parsedItems.add(tle) }
-                    }
+    suspend fun parseCSVStream(csvStream: InputStream): List<OrbitalData> = withContext(defaultDispatcher) {
+        val parsedItems = mutableListOf<OrbitalData>()
+        csvStream.bufferedReader().useLines { lines ->
+            lines.forEachIndexed { index, line ->
+                if (index != 0) {
+                    val values = line.split(",")
+                    parseCSV(values)?.let { tle -> parsedItems.add(tle) }
                 }
             }
-            return@withContext parsedItems
         }
+        return@withContext parsedItems
+    }
 
-    suspend fun parseTLEStream(tleStream: InputStream): List<OrbitalData> =
-        withContext(defaultDispatcher) {
-            val tleStrings = mutableListOf(String(), String(), String())
-            val parsedItems = mutableListOf<OrbitalData>()
-            var lineIndex = 0
-            tleStream.bufferedReader().forEachLine { line ->
-                tleStrings[lineIndex] = line
-                if (lineIndex < 2) {
-                    lineIndex++
-                } else {
-                    val isLineOneValid = tleStrings[1].substring(0, 1) == "1"
-                    val isLineTwoValid = tleStrings[2].substring(0, 1) == "2"
-                    if (!isLineOneValid && !isLineTwoValid) return@forEachLine
-                    parseTLE(tleStrings)?.let { tle -> parsedItems.add(tle) }
-                    lineIndex = 0
-                }
+    suspend fun parseTLEStream(tleStream: InputStream): List<OrbitalData> = withContext(defaultDispatcher) {
+        val tleStrings = mutableListOf(String(), String(), String())
+        val parsedItems = mutableListOf<OrbitalData>()
+        var lineIndex = 0
+        tleStream.bufferedReader().forEachLine { line ->
+            tleStrings[lineIndex] = line
+            if (lineIndex < 2) {
+                lineIndex++
+            } else {
+                val isLineOneValid = tleStrings[1].substring(0, 1) == "1"
+                val isLineTwoValid = tleStrings[2].substring(0, 1) == "2"
+                if (!isLineOneValid && !isLineTwoValid) return@forEachLine
+                parseTLE(tleStrings)?.let { tle -> parsedItems.add(tle) }
+                lineIndex = 0
             }
-            return@withContext parsedItems
         }
+        return@withContext parsedItems
+    }
 
     suspend fun parseJSONStream(jsonStream: InputStream): List<SatRadio> {
         return withContext(defaultDispatcher) {
@@ -78,6 +76,10 @@ class DataParser(private val defaultDispatcher: CoroutineDispatcher) {
         }
     }
 
+    fun isLeapYear(year: Int): Boolean {
+        return ((year % 4 == 0) && (year % 100 != 0)) || (year % 400 == 0)
+    }
+
     private fun parseCSV(values: List<String>): OrbitalData? {
         try {
             val name = values[0]
@@ -120,8 +122,8 @@ class DataParser(private val defaultDispatcher: CoroutineDispatcher) {
             val argper: Double = tle[2].substring(34, 42).toDouble()
             val meanan: Double = tle[2].substring(43, 51).toDouble()
             val catnum: Int = tle[1].substring(2, 7).trim().toInt()
-            val bstar: Double = 1.0e-5 * tle[1].substring(53, 59).toDouble() /
-                    10.0.pow(tle[1].substring(60, 61).toDouble())
+            val bstar: Double =
+                1.0e-5 * tle[1].substring(53, 59).toDouble() / 10.0.pow(tle[1].substring(60, 61).toDouble())
             return OrbitalData(name, epoch, meanmo, eccn, incl, raan, argper, meanan, catnum, bstar)
         } catch (exception: Exception) {
             return null
@@ -153,8 +155,10 @@ class DataParser(private val defaultDispatcher: CoroutineDispatcher) {
         val daysArray = arrayOf(31, 28, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31)
         var dayOfYear = dayOfMonth
         // If leap year increment Feb days
-        if (((year / 4 == 0) && (year / 100 != 0)) || (year / 400 == 0)) daysArray[1]++
-        for (i in 0 until month - 1) { dayOfYear += daysArray[i] }
+        if (isLeapYear(year)) daysArray[1]++
+        for (i in 0 until month - 1) {
+            dayOfYear += daysArray[i]
+        }
         return dayOfYear
     }
 }

base/src/main/java/com/rtbishop/look4sat/utility/DataReporter.kt

@@ -17,13 +17,13 @@
  */
 package com.rtbishop.look4sat.utility
 
+import java.net.InetSocketAddress
+import java.nio.ByteBuffer
+import java.nio.channels.SocketChannel
 import kotlinx.coroutines.CoroutineScope
 import kotlinx.coroutines.Job
 import kotlinx.coroutines.cancelAndJoin
 import kotlinx.coroutines.launch
-import java.net.InetSocketAddress
-import java.nio.ByteBuffer
-import java.nio.channels.SocketChannel
 
 class DataReporter(private val reporterScope: CoroutineScope) {
 
@@ -56,7 +56,7 @@ class DataReporter(private val reporterScope: CoroutineScope) {
                 if (rotationSocketChannel == null) {
                     rotationSocketChannel = SocketChannel.open(InetSocketAddress(server, port))
                 } else {
-                    val buffer = ByteBuffer.wrap("\\set_pos $azimuth $elevation\n".toByteArray())
+                    val buffer = ByteBuffer.wrap("\\P $azimuth $elevation\n".toByteArray())
                     rotationSocketChannel?.write(buffer)
                 }
             }.onFailure { error ->