[firebase_auth]: Android physical device not receiving sms

Is there an existing issue for this?

[X] I have searched the existing issues.

Which plugins are affected?
Auth
Which platforms are affected?
Android
Description
The verifyPhoneNumber api is not sending otp codes when using debug phone numbers that are in the fire console. It just hangs and about 10mins later it lets me to the next screen to input the verification code. The same thing when testing real numbers. And this is only happening on android, ios is working perfectly.
I have been using been using firebsae_auth for a little over a year now. I have set up my release, debug, and google signed keys into the firebase console.
i have upgraded to the new sms multi factor authentication (only because sms messages stopped sending to android)
I also have set up debug keys in the firebase console for my apps.
Reproducing the issue
// initial setup
final app = await Firebase.initializeApp(
options: DefaultFirebaseOptions.currentPlatform);
FirebaseAuth.instanceFor(app: app);
await FirebaseAppCheck.instance.activate(
androidProvider: AndroidProvider.debug,

appleProvider: AppleProvider.debug,

);
then when using the verifyPhone api try to send an otp to a debug number or real number
i am testing on a real device
Firebase Core version
2.27.0
Flutter Version
3.19.4
Relevant Log Output
//this is the output from start to finish after trying to send an otp code

E/zzb     (16322): Failed to initialize reCAPTCHA config: No Recaptcha Enterprise siteKey configured for tenant/project *
W/System  (16322): Ignoring header X-Firebase-Locale because its value was null.
D/TrafficStats(16322): tagSocket(195) with statsTag=0xffffffff, statsUid=-1
D/TrafficStats(16322): tagSocket(166) with statsTag=0xffffffff, statsUid=-1
D/TrafficStats(16322): tagSocket(118) with statsTag=0xffffffff, statsUid=-1
D/TrafficStats(16322): tagSocket(196) with statsTag=0xffffffff, statsUid=-1
D/TrafficStats(16322): tagSocket(195) with statsTag=0xffffffff, statsUid=-1
I/PlayCore(16322): UID: [10649]  PID: [16322] IntegrityService : requestIntegrityToken(IntegrityTokenRequest{nonce=C_-YGH46p_J4QyWGlTO_MrQjHhiSSGwn7LqWiNGAkV8, cloudProjectNumber=551503664846, network=null})
I/PlayCore(16322): UID: [10649]  PID: [16322] IntegrityService : Initiate binding to the service.
I/PlayCore(16322): UID: [10649]  PID: [16322] IntegrityService : ServiceConnectionImpl.onServiceConnected(ComponentInfo{com.android.vending/com.google.android.finsky.integrityservice.IntegrityService})
I/PlayCore(16322): UID: [10649]  PID: [16322] IntegrityService : linkToDeath
I/PlayCore(16322): UID: [10649]  PID: [16322] OnRequestIntegrityTokenCallback : onRequestIntegrityToken
I/PlayCore(16322): UID: [10649]  PID: [16322] IntegrityService : Unbind from service.
W/System  (16322): Ignoring header X-Firebase-Locale because its value was null.

Flutter dependencies

Expand Flutter dependencies snippet

  app_settings: ^5.1.1
  cached_network_image: ^3.3.1
  collection: ^1.18.0
  dio: ^5.4.2+1
  event_bus: ^2.0.0
  firebase_analytics: ^10.9.0
  firebase_auth: ^4.18.0
  firebase_core: ^2.27.2
  firebase_crashlytics: ^3.4.20
  firebase_messaging: ^14.7.21
  flex_color_scheme: ^7.3.1
  flutter:
    sdk: flutter
  flutter_cache_manager: ^3.3.1
  flutter_datetime_picker_bdaya: ^3.0.2
  flutter_debouncer: ^2.0.0
  flutter_hooks: ^0.20.5
  flutter_keyboard_visibility: ^6.0.0
  flutter_local_notifications: ^17.0.0
  flutter_rating_bar: ^4.0.1
  flutter_riverpod: ^2.5.1
  flutter_svg: ^2.0.10+1
  flutter_timezone: ^1.0.8
  font_awesome_flutter: ^10.7.0
  geolocator: ^12.0.0
  get_storage: ^2.1.1
  go_router: ^14.0.1
  google_fonts: ^6.2.1
  hooks_riverpod: ^2.5.1
  image_cropper: ^6.0.0
  image_picker: ^1.0.7
  intl: ^0.19.0
  jose: ^0.3.4
  json_annotation: ^4.9.0
  profanity_filter: ^2.0.0
  riverpod_annotation: ^2.3.5
  scrollable_positioned_list: ^0.3.8
  share_plus: ^9.0.0
  sizer: ^2.0.15
  skeletonizer: ^1.1.1
  timezone: ^0.9.2
  upgrader: ^10.3.0
  url_launcher: ^6.2.5

Additional context and comments
I have searched through similar issues and all of them were that i need to upload my keys, which are already there. I can see in the firebase console that requests from my android are getting through, but just seems the recaptcha doesnt want to open on the phone.

do you need any more info on this?

@engrpatel21 Thank you for creating this issue, Can you please try to create a new project with just this functionality and see if the issue also exists?

Did you also try different devices, and did you find this issue occurring on all of them?

Did you also try different devices, and did you find this issue occurring on all of them?

yes i have tried on other devices for android, pixel6, s9, s20. i always get the failed to initialize recaptcha.  some times the sms takes up to 20 mins to get sent waiting for the api to return a valid response.

@engrpatel21 Thank you for creating this issue, Can you please try to create a new project with just this functionality and see if the issue also exists?

i will try to get this done asap let you know results, its pretty involved with all the setups to get everything going and test on a brand new project.

According to the 2nd comment in this stack overflow link https://stackoverflow.com/questions/74739659/firebase-sms-authentication-is-delayed-or-sometimes-not-received. It could possibly be an issue with the mobile service provider. It might be worth exploring this possibility. Could you let us know where you are testing (i.e., which country)? Additionally, it might be beneficial to get in touch with Firebase support to inquire about the coverage in your area.

im in the usa, ive has this system in production for a year without any issues. its also a factor of the api call return time. there is a huge latency to when i get a response back. but when i look at fire base console under app check and authentication, i can see requests that just came in.

i am currently working trying to repoduce the problem in a seperate project. i will let you know the outcome and relevant details

ok so i have made a new project. this is the nature. the signing keys debug keys as my production project.
this is the code

import 'package:firebase_auth/firebase_auth.dart';
import 'package:firebase_core/firebase_core.dart';
import 'package:flutter/material.dart';
import 'package:smstest/firebase_options.dart';

void main() async {
  WidgetsFlutterBinding.ensureInitialized();

  final app = await Firebase.initializeApp(
      options: DefaultFirebaseOptions.currentPlatform);
  FirebaseAuth.instanceFor(app: app);
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(

        colorScheme: ColorScheme.fromSeed(seedColor: Colors.deepPurple),
        useMaterial3: true,
      ),
      home: const MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({super.key, required this.title});

  final String title;

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  int _counter = 0;
  bool isLoading = false;

  void _incrementCounter() {
    setState(() {

      _counter++;
    });
  }

  final phoneNumber = TextEditingController();

  void submitPhoneNumber() async {
    setState(() {
      isLoading = true;
    });
    await FirebaseAuth.instance.verifyPhoneNumber(
      phoneNumber: '+1${phoneNumber.text}',
      timeout: const Duration(seconds: 20),
      verificationCompleted: (PhoneAuthCredential credential) {},
      verificationFailed: (FirebaseAuthException e) {
        print(e.code);
        print(e.message);
        print(e.credential);
        if (e.code == 'invalid-phone-number') {}
      },
      codeSent: (String verificationId, int? resendToken) {
        setState(() {
          isLoading = false;
        });
      },
      codeAutoRetrievalTimeout: (String verificationId) {},
    );
  }

  @override
  Widget build(BuildContext context) {

    return Scaffold(
      appBar: AppBar(

        backgroundColor: Theme.of(context).colorScheme.inversePrimary,

        title: Text(widget.title),
      ),
      body: Center(

        child: Column(
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
            const Text(
              'You have pushed the button this many times:',
            ),
            TextField(
              controller: phoneNumber,
            ),
            FilledButton(
                onPressed: isLoading ? null : submitPhoneNumber, child: const Text('Submit')),
          ],
        ),
      ),
      floatingActionButton: FloatingActionButton(
        onPressed: _incrementCounter,
        tooltip: 'Increment',
        child: const Icon(Icons.add),
      ),
    );
  }
}

this is my app/build.gradle

again this is the error message
I/zzb     (31079): ForceRecaptchaV2Flow from phoneAuthOptions = false, ForceRecaptchav2Flow from firebaseSettings = false
W/System  (31079): Ignoring header X-Firebase-Locale because its value was null.
D/TrafficStats(31079): tagSocket(6) with statsTag=0xffffffff, statsUid=-1

plugins {
    id "com.android.application"
    id "kotlin-android"
    id "dev.flutter.flutter-gradle-plugin"
    id "com.google.gms.google-services"
}

def localProperties = new Properties()
def localPropertiesFile = rootProject.file('local.properties')
if (localPropertiesFile.exists()) {
    localPropertiesFile.withReader('UTF-8') { reader ->
        localProperties.load(reader)
    }
}

def flutterVersionCode = localProperties.getProperty('flutter.versionCode')
if (flutterVersionCode == null) {
    flutterVersionCode = '1'
}

def flutterVersionName = localProperties.getProperty('flutter.versionName')
if (flutterVersionName == null) {
    flutterVersionName = '1.0'
}

def keystoreProperties = new Properties()
def keystorePropertiesFile = rootProject.file('key.properties')
// def googleApiProperties = new Properties()
// def googleApiPropertiesFile = rootProject.file('google.api.properties')

if (keystorePropertiesFile.exists()) {
    keystoreProperties.load(new FileInputStream(keystorePropertiesFile))

}

android {
    namespace "io.zyka"
    // compileSdk flutter.compileSdkVersion
    compileSdk 34
    // ndkVersion flutter.ndkVersion
     ndkVersion "25.1.8937393"

    compileOptions {
        sourceCompatibility JavaVersion.VERSION_1_8
        targetCompatibility JavaVersion.VERSION_1_8
        coreLibraryDesugaringEnabled true

    }

    kotlinOptions {
        jvmTarget = '1.8'
    }

    sourceSets {
        main.java.srcDirs += 'src/main/kotlin'
    }

    defaultConfig {
        // TODO: Specify your own unique Application ID (https://developer.android.com/studio/build/application-id.html).
        applicationId "io.zyka"
        // You can update the following values to match your application needs.
        // For more information, see: https://docs.flutter.dev/deployment/android#reviewing-the-gradle-build-configuration.
        minSdkVersion 21
        targetSdkVersion 34

        // targetSdkVersion flutter.targetSdkVersion
        versionCode flutterVersionCode.toInteger()
        versionName flutterVersionName
        multiDexEnabled true
    }
    signingConfigs {
       release {
           keyAlias keystoreProperties['keyAlias']
           keyPassword keystoreProperties['keyPassword']
           storeFile keystoreProperties['storeFile'] ? file(keystoreProperties['storeFile']) : null
           storePassword keystoreProperties['storePassword']
       }
   }

    buildTypes {
         debug {
            minifyEnabled true
            signingConfig signingConfigs.release
        }
         release {
            signingConfig signingConfigs.release
            minifyEnabled true
            proguardFiles getDefaultProguardFile('proguard-android-optimize.txt'), 'proguard-rules.pro'
            ndk {
              debugSymbolLevel 'FULL'
            }
       }
    }
}

flutter {
    source '../..'
}

dependencies {
    implementation platform('com.google.firebase:firebase-bom:32.6.0')
    implementation 'com.google.firebase:firebase-analytics'
    implementation("com.google.firebase:firebase-auth:22.3.1")
    // implementation 'androidx.appcompat:appcompat:1.3.1'
    implementation "androidx.window:window:1.0.0"

    // For Java-friendly APIs to register and unregister callbacks
    implementation "androidx.window:window-java:1.0.0"

    coreLibraryDesugaring 'com.android.tools:desugar_jdk_libs:2.0.3'
}

this is my root/gradle
allprojects {
    repositories {
        google()
        mavenCentral()
    }
}

rootProject.buildDir = '../build'
subprojects {
    project.buildDir = "${rootProject.buildDir}/${project.name}"
}
subprojects {
    project.evaluationDependsOn(':app')
}

tasks.register("clean", Delete) {
    delete rootProject.buildDir
}

this is my gradle properties

distributionBase=GRADLE_USER_HOME
distributionPath=wrapper/dists
zipStoreBase=GRADLE_USER_HOME
zipStorePath=wrapper/dists
distributionUrl=https\://services.gradle.org/distributions/gradle-8.4-all.zip

this is the error message which is the same one i have been getting
I/zzb     (31079): ForceRecaptchaV2Flow from phoneAuthOptions = false, ForceRecaptchav2Flow from firebaseSettings = false
W/System  (31079): Ignoring header X-Firebase-Locale because its value was null.
5
D/TrafficStats(31079): tagSocket(6) with statsTag=0xffffffff, statsUid=-1
E/zzb     (31079): Failed to initialize reCAPTCHA config: No Recaptcha Enterprise siteKey configured for tenant/project *
W/System  (31079): Ignoring header X-Firebase-Locale because its value was null.
D/TrafficStats(31079): tagSocket(176) with statsTag=0xffffffff, statsUid=-1

I 've noticed that you are using mismatched versions of firebase auth and firebase core, you can use flutterfire install latest to install latest versions.
Can you please try that and see if also the same issue still present ?

there wasn't a command called flutterfire install latest, but there was a command called flutterfire update but it ended up updating to the versions i already have shown you.
name: smstest
description: "A new Flutter project."

publish_to: "none"

version: 1.0.0+1

environment:
  sdk: ">=3.3.2 <4.0.0"

dependencies:
  flutter:
    sdk: flutter

  cupertino_icons: ^1.0.6
  firebase_auth: ^4.19.6
  firebase_core: ^2.27.2

dev_dependencies:
  flutter_test:
    sdk: flutter

  flutter_lints: ^3.0.0

flutter:
  uses-material-design: true

this is the output in the console.
➜  smstest flutterfire install latest
Unhandled exception:
Could not find a command named "install".

Usage: flutterfire <command> [arguments]

Global options:
-h, --help       Print this usage information.
    --verbose    Enable verbose logging.
-v, --version    Print the current CLI version.

Available commands:
  configure   Configure Firebase for your Flutter app. This
              command will fetch Firebase configuration for you
              and generate a Dart file with prefilled
              FirebaseOptions you can use.
  update      Update the version of firebase plugins in your
              pubspec to the latest version and clean your
              workspace to ensure that everything works
              properly.

Run "flutterfire help <command>" for more information about a
command.
#0      CommandRunner.usageException (package:args/command_runner.dart:105:7)
#1      CommandRunner.runCommand (package:args/command_runner.dart:178:13)
#2      CommandRunner.run.<anonymous closure> (package:args/command_runner.dart:122:25)
#3      new Future.sync (dart:async/future.dart:305:31)
#4      CommandRunner.run (package:args/command_runner.dart:122:14)
#5      main (file:///Users/zyka/.pub-cache/hosted/pub.dev/flutterfire_cli-0.2.7/bin/flutterfire.dart:57:48)
<asynchronous suspension>
➜  smstest flutterfire update
Cleaning up current workspace ...
Upgrading all firebase plugins to the latest version ...
Running 'flutter pub get'...
Ready to use the latest version of FlutterFire! 🚀

im also wondering if my versions are write on the android side
dependencies {
    implementation platform('com.google.firebase:firebase-bom:32.6.0')
    implementation 'com.google.firebase:firebase-analytics'
    implementation("com.google.firebase:firebase-auth:22.3.1")
    // implementation 'androidx.appcompat:appcompat:1.3.1'
    implementation "androidx.window:window:1.0.0"

    // For Java-friendly APIs to register and unregister callbacks
    implementation "androidx.window:window-java:1.0.0"

    coreLibraryDesugaring 'com.android.tools:desugar_jdk_libs:2.0.3'
}

an update, i have manually updated firebase_core and firebase_auth manually, then i went in and updated the firebase bom in app/build.gradle along with adding app check and debug tokens on the app check console in firebase. here is the latest code
import 'package:firebase_app_check/firebase_app_check.dart';
import 'package:firebase_auth/firebase_auth.dart';
import 'package:firebase_core/firebase_core.dart';
import 'package:flutter/material.dart';
import 'package:smstest/firebase_options.dart';

void main() async {
  WidgetsFlutterBinding.ensureInitialized();

  final app = await Firebase.initializeApp(
      options: DefaultFirebaseOptions.currentPlatform);
  FirebaseAuth.instanceFor(app: app);
  await FirebaseAppCheck.instance.activate(
    androidProvider: AndroidProvider.debug,
    appleProvider: AppleProvider.debug,
  );
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        colorScheme: ColorScheme.fromSeed(seedColor: Colors.deepPurple),
        useMaterial3: true,
      ),
      home: const MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({super.key, required this.title});

  final String title;

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  int _counter = 0;
  bool isLoading = false;

  void _incrementCounter() {
    setState(() {
      _counter++;
    });
  }

  final phoneNumber = TextEditingController();

  void submitPhoneNumber() async {
    setState(() {
      isLoading = true;
    });
    await FirebaseAuth.instance.verifyPhoneNumber(
      phoneNumber: '+1${phoneNumber.text}',
      timeout: const Duration(seconds: 20),
      verificationCompleted: (PhoneAuthCredential credential) {},
      verificationFailed: (FirebaseAuthException e) {
        print(e.code);
        print(e.message);
        print(e.credential);
        if (e.code == 'invalid-phone-number') {}
      },
      codeSent: (String verificationId, int? resendToken) {
        setState(() {
          isLoading = false;
        });
      },
      codeAutoRetrievalTimeout: (String verificationId) {},
    );
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        backgroundColor: Theme.of(context).colorScheme.inversePrimary,
        title: Text(widget.title),
      ),
      body: Center(
        child: Column(
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
            const Text(
              'You have pushed the button this many times:',
            ),
            TextField(
              controller: phoneNumber,
            ),
            FilledButton(
                onPressed: isLoading ? null : submitPhoneNumber,
                child: const Text('Submit')),
          ],
        ),
      ),
      floatingActionButton: FloatingActionButton(
        onPressed: _incrementCounter,
        tooltip: 'Increment',
        child: const Icon(Icons.add),
      ),
    );
  }
}

name: smstest
description: "A new Flutter project."

publish_to: "none"

version: 1.0.0+1

environment:
  sdk: ">=3.3.2 <4.0.0"

dependencies:
  flutter:
    sdk: flutter

  cupertino_icons: ^1.0.6
  firebase_auth: ^4.20.0
  firebase_core: ^2.32.0
  firebase_app_check: ^0.2.2+7

dev_dependencies:
  flutter_test:
    sdk: flutter

  flutter_lints: ^4.0.0

flutter:
  uses-material-design: true

plugins {
    id "com.android.application"
    id "kotlin-android"
    id "dev.flutter.flutter-gradle-plugin"
    id "com.google.gms.google-services"
}

def localProperties = new Properties()
def localPropertiesFile = rootProject.file('local.properties')
if (localPropertiesFile.exists()) {
    localPropertiesFile.withReader('UTF-8') { reader ->
        localProperties.load(reader)
    }
}

def flutterVersionCode = localProperties.getProperty('flutter.versionCode')
if (flutterVersionCode == null) {
    flutterVersionCode = '1'
}

def flutterVersionName = localProperties.getProperty('flutter.versionName')
if (flutterVersionName == null) {
    flutterVersionName = '1.0'
}

def keystoreProperties = new Properties()
def keystorePropertiesFile = rootProject.file('key.properties')
// def googleApiProperties = new Properties()
// def googleApiPropertiesFile = rootProject.file('google.api.properties')

if (keystorePropertiesFile.exists()) {
    keystoreProperties.load(new FileInputStream(keystorePropertiesFile))

}

android {
    namespace "io.zyka"
    // compileSdk flutter.compileSdkVersion
    compileSdk 34
    // ndkVersion flutter.ndkVersion
     ndkVersion "25.1.8937393"

    compileOptions {
        sourceCompatibility JavaVersion.VERSION_1_8
        targetCompatibility JavaVersion.VERSION_1_8
        coreLibraryDesugaringEnabled true

    }

    kotlinOptions {
        jvmTarget = '1.8'
    }

    sourceSets {
        main.java.srcDirs += 'src/main/kotlin'
    }

    defaultConfig {
        // TODO: Specify your own unique Application ID (https://developer.android.com/studio/build/application-id.html).
        applicationId "io.zyka"
        // You can update the following values to match your application needs.
        // For more information, see: https://docs.flutter.dev/deployment/android#reviewing-the-gradle-build-configuration.
        minSdkVersion 23
        targetSdkVersion 34

        // targetSdkVersion flutter.targetSdkVersion
        versionCode flutterVersionCode.toInteger()
        versionName flutterVersionName
        multiDexEnabled true
    }
    signingConfigs {
       release {
           keyAlias keystoreProperties['keyAlias']
           keyPassword keystoreProperties['keyPassword']
           storeFile keystoreProperties['storeFile'] ? file(keystoreProperties['storeFile']) : null
           storePassword keystoreProperties['storePassword']
       }
   }

    buildTypes {
         debug {
            minifyEnabled true
            signingConfig signingConfigs.release
        }
         release {
            signingConfig signingConfigs.release
            minifyEnabled true
            proguardFiles getDefaultProguardFile('proguard-android-optimize.txt'), 'proguard-rules.pro'
            ndk {
              debugSymbolLevel 'FULL'
            }
       }
    }
}

flutter {
    source '../..'
}

dependencies {
    implementation platform('com.google.firebase:firebase-bom:33.0.0')
    implementation 'com.google.firebase:firebase-analytics'
    implementation("com.google.firebase:firebase-auth")
    implementation("com.google.firebase:firebase-appcheck")
    implementation("com.google.firebase:firebase-appcheck-playintegrity")
    // implementation 'androidx.appcompat:appcompat:1.3.1'
    implementation "androidx.window:window:1.0.0"

    // For Java-friendly APIs to register and unregister callbacks
    implementation "androidx.window:window-java:1.0.0"

    coreLibraryDesugaring 'com.android.tools:desugar_jdk_libs:2.0.3'
}

here is the log output
I/zza     (22069): ForceRecaptchaFlow from phoneAuthOptions = false, ForceRecaptchaFlow from firebaseSettings = false
W/System  (22069): Ignoring header X-Firebase-Locale because its value was null.
D/TrafficStats(22069): tagSocket(91) with statsTag=0xffffffff, statsUid=-1
4
D/TrafficStats(22069): tagSocket(6) with statsTag=0xffffffff, statsUid=-1
I/PlayCore(22069): UID: [10660]  PID: [22069] IntegrityService : requestIntegrityToken(IntegrityTokenRequest{nonce=C_-YGH46p_J4QyWGlTO_MrQjHhiSSGwn7LqWiNGAkV8, cloudProjectNumber=551503664846, network=null})
I/PlayCore(22069): UID: [10660]  PID: [22069] IntegrityService : Initiate binding to the service.
I/PlayCore(22069): UID: [10660]  PID: [22069] IntegrityService : ServiceConnectionImpl.onServiceConnected(ComponentInfo{com.android.vending/com.google.android.finsky.integrityservice.IntegrityService})
I/PlayCore(22069): UID: [10660]  PID: [22069] IntegrityService : linkToDeath
I/PlayCore(22069): UID: [10660]  PID: [22069] OnRequestIntegrityTokenCallback : onRequestIntegrityToken
I/PlayCore(22069): UID: [10660]  PID: [22069] IntegrityService : Unbind from service.
W/io.zyka (22069): Accessing hidden field Ljava/util/Collections$SynchronizedCollection;->mutex:Ljava/lang/Object; (max-target-o, reflection, denied)
W/io.zyka (22069): Accessing hidden field Ljava/util/Collections$SynchronizedCollection;->c:Ljava/util/Collection; (unsupported, reflection, allowed)
W/io.zyka (22069): Accessing hidden method Ljava/util/Collections$SynchronizedSet;-><init>(Ljava/util/Set;Ljava/lang/Object;)V (max-target-o, reflection, denied)
W/io.zyka (22069): Accessing hidden method Ljava/util/Collections$SynchronizedCollection;-><init>(Ljava/util/Collection;Ljava/lang/Object;)V (max-target-o, reflection, denied)
W/System  (22069): Ignoring header X-Firebase-Locale because its value was null.
W/FirebaseAuth(22069): [SmsRetrieverHelper] Timed out waiting for SMS.

if i wait long enough, i can see the request come in to app check panel as an authenticated request

i also put flutter on the newest version along with the newest versions of these libraries

also, on apps that have app check enabled with play integrity, isn't recaptcha not supposed to show?

do you need any more info from me?

so am i just going to be ignored?

any update for this issue?

Meet the same issue sometimes, any updates on this?

Having the same issue, but in my case, users in USA are receiving the messages normally, but on other countries, the sms is delaying hours, or even not arriving at all. Most of the issues are happening with Brazilian and Colombian phones.


Nginx metrics registering as gauges

Additional environment details (Operating System, Cloud provider, etc):
Ubuntu 14.04
Steps to reproduce the issue:

Setup nginx integration with localhost endpoint and a tag
All new metrics come up in datadog as gauge instead of proper type.

Describe the results you received:
I see in the code, that the type seems to be mentioned:
('nginx.net.conn_opened_per_s', conn, tags, 'rate')

However after opening metrics explorer, this metric had type gauge set.
Describe the results you expected:
..._per_s metrics should be registered as rates
Additional information you deem important (e.g. issue happens only occasionally):

Hi @viraptor thanks for reporting.
As confusing as it sounds, the metric type you see is actually correct - rates are computed on the client side and sent as gauges to the backend.
You can see https://docs.datadoghq.com/metrictypes/ for more details
Closing for now, feel free to reach out to support@ if you're experiencing any issue with the nginx integration.


Update streaming connector wiki page

Problem description
The following wiki page has stale information:
https://github.com/pravega/flink-connectors/wiki/Streaming-Connector
Specifically, I noticed that:

It refers to some timeout parameters that no longer exist;
It refers to FlinkPravegaWriter and FlinkExactlyOncePravegaWriter separately, and they have been merged in PR #30 .

Problem location
Wiki documentation.
Suggestions for an improvement
Update the documentation.

We also need to update the batch connector (transaction config) and table connector (TBD).


Adjust the NSG validation logic for GA

Which issue this PR addresses:
Fixes: ARO-2590
What this PR does / why we need it:
To adjust the validation logic of NSG validation for GA.  Previously in private preview, we allow no nsgs to be attached to the subnets which the FP then assumes that the feature shouldn't be used.  In GA, we make it explicit so that when the preconfigured nsg flag is enabled and no subnets are attached, the FP will error out instead.
Test plan for issue:
Unit tests.
Is there any documentation that needs to be updated for this PR?
no

/azp e2e

/azp run e2e

/azp run "E2E - PullRequest"

/azp run e2e

/azp run e2e


feat: remove duplicate example apps

Proposed changes
Removes the duplicate examples left over from the V2 node SDK. Both live and prerecorded examples now use V3 JS SDK

Codecov Report
All modified lines are covered by tests :white_check_mark:

:exclamation: No coverage uploaded for pull request base (alpha@4dc4125). Click here to learn what that means.

Additional details and impacted files
@@           Coverage Diff            @@
##             alpha     #172   +/-   ##
========================================
  Coverage         ?   69.42%
========================================
  Files            ?       16
  Lines            ?      520
  Branches         ?      103
========================================
  Hits             ?      361
  Misses           ?      132
  Partials         ?       27

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Incorrect parsing of JS code within declarations

This is my code:
export const Circle = styled.div<{ selected: boolean }>`
  background-color: ${(props) => (props.selected ? colorPalette.purple : colorPalette.black)};
`;

I've activated the rule 'scale-unlimited/declaration-strict-value' from https://github.com/AndyOGo/stylelint-declaration-strict-value.
Stylelint complains: "Expected variable, function or keyword for "=>" of "background-color" (scale-unlimited/declaration-strict-value)".
I will also open an issue with the rule author, but mistaking JS code for a CSS property value seems to be a parsing issue.
Using stylelint 15.6.0<EMAIL_ADDRESS>stylelint-declaration-strict-value 1.9.2.

Ok, thanks, so this is a plugin issue if I understand correctly?

Yep, they don't support CSS-in-JS.


Esri VectorTiledLayer refresh problem

myVectorLayer = AGSArcGISVectorTiledLayer(url: url!)
if let rc = AGSRequestConfiguration.global().copy() as? AGSRequestConfiguration {
    rc.requestCachePolicy = .reloadIgnoringLocalAndRemoteCacheData
    rc.shouldCacheResponse = false
    myVectorLayer?.requestConfiguration = rc
}
self.mapView.map?.operationalLayers.add(myVectorLayer!)

Hi
There is such a solution on the ios side
Our equivalent to RequestConfiguration but I couldn't find the process of canceling cache

Hi @teomanyaman - This looks like swift and I think you're therefore best asking the question the ArcGIS Runtime iOS GeoNet forum found here: https://community.esri.com/community/developers/native-app-developers/arcgis-runtime-sdk-for-ios

Hi TADraeseke,
What is the equivalent of this code in the Android SDK?

Hi @ikbalyasar. Are you serving these tiles from ArcGIS Enterprise? If you have access to the server to configure it, you could add an outgoing header.
The Android SDK honors Cache-Control: no-cache or Cache-Control: max-age=<seconds> which could ensure that tiles are not cached and are always requested.


ERROR: llama_cpp_python-0.2.19-cp311-cp311-macosx_12_0_x86_64.whl is not a supported wheel on this platform.

getting error when running the below command
pip install -r requirements_apple_intel.txt
Could your please help me how can I fix this error.

What version of Python are you using?

@jllllll I have some users unable to install on MacOS with M1. Can't figure out why 😂

This is the error we get:
Building wheels for collected packages: llama-cpp-python
  Building wheel for llama-cpp-python (pyproject.toml): started
  Building wheel for llama-cpp-python (pyproject.toml): finished with status 'error'
  error: subprocess-exited-with-error

  Ã— Building wheel for llama-cpp-python (pyproject.toml) did not run successfully.
  â”‚ exit code: 1
  â•°â”€> [24 lines of output]
      *** scikit-build-core 0.6.1 using CMake 3.27.7 (wheel)
      *** Configuring CMake...
      loading initial cache file /tmp/tmp9tc9476m/build/CMakeInit.txt
      -- The C compiler identification is unknown
      -- The CXX compiler identification is unknown
      CMake Error at CMakeLists.txt:3 (project):
        No CMAKE_C_COMPILER could be found.

        Tell CMake where to find the compiler by setting either the environment
        variable "CC" or the CMake cache entry CMAKE_C_COMPILER to the full path to
        the compiler, or to the compiler name if it is in the PATH.

      CMake Error at CMakeLists.txt:3 (project):
        No CMAKE_CXX_COMPILER could be found.

        Tell CMake where to find the compiler by setting either the environment
        variable "CXX" or the CMake cache entry CMAKE_CXX_COMPILER to the full path
        to the compiler, or to the compiler name if it is in the PATH.

      -- Configuring incomplete, errors occurred!

      *** CMake configuration failed
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
  ERROR: Failed building wheel for llama-cpp-python
Failed to build llama-cpp-python
ERROR: Could not build wheels for llama-cpp-python, which is required to install pyproject.toml-based projects

[notice] A new release of pip is available: 23.2.1 -> 23.3.1
[notice] To update, run: pip install --upgrade pip
Failed to install llama-cpp-python
+ echo 'Failed to install llama-cpp-python'

@jllllll Is using this the right way for Metal cpu's?
UNAME_M=arm64 python -m pip install llama-cpp-python==0.2.19 --prefer-binary --extra-index-url=https://jllllll.github.io/llama-cpp-python-cuBLAS-wheels/basic/cpu

@gaby Try this command:
UNAME_M=arm64 python -m pip install llama-cpp-python==0.2.19 --only-binary llama-cpp-python --extra-index-url=https://jllllll.github.io/llama-cpp-python-cuBLAS-wheels/basic/cpu

@jllllll I had to use --only-binary=:all: to get it to work with amd64. I have to find someone with an M1 to test the new command. Will report back :-) Thanks!

@jllllll Still didn't work, this is with py3.11 on a M2 Pro
+ls_cpu 'Architecture:                    aarch64
CPU op-mode(s):                  64-bit
Byte Order:                      Little Endian
CPU(s):                          6
On-line CPU(s) list:             0-5
Vendor ID:                       0x00
Model name:                      -
Model:                           0
Thread(s) per core:              1
Core(s) per cluster:             6
Socket(s):                       -
Cluster(s):                      1
Stepping:                        0x0
BogoMIPS:                        48.00
Flags:                           fp asimd evtstrm aes pmull sha1 sha2 crc32 atomics fphp asimdhp cpuid asimdrdm jscvt fcma lrcpc dcpop sha3 asimddp sha512 asimdfhm dit uscat ilrcpc flagm ssbs sb paca pacg dcpodp flagm2 frint
Vulnerability Itlb multihit:     Not affected
Vulnerability L1tf:              Not affected
Vulnerability Mds:               Not affected
Vulnerability Meltdown:          Not affected
Vulnerability Mmio stale data:   Not affected
Vulnerability Spec store bypass: Vulnerable
Vulnerability Spectre v1:        Mitigation; __user pointer sanitization
Vulnerability Spectre v2:        Not affected
Vulnerability Srbds:             Not affected
Vulnerability Tsx async abort:   Not affected'

++ dpkg --print-architecture
Recommended install command for llama-cpp-python: UNAME_M=arm64 python -m pip install llama-cpp-python==0.2.19 --only-binary=:all: --extra-index-url=https://jllllll.github.io/llama-cpp-python-cuBLAS-wheels/basic/cpu

+ pip_command='UNAME_M=arm64 python -m pip install llama-cpp-python==0.2.19 --only-binary=:all: --extra-index-url=https://jllllll.github.io/llama-cpp-python-cuBLAS-wheels/basic/cpu'

+ echo 'Recommended install command for llama-cpp-python: UNAME_M=arm64 python -m pip install llama-cpp-python==0.2.19 --only-binary=:all: --extra-index-url=https://jllllll.github.io/llama-cpp-python-cuBLAS-wheels/basic/cpu'

+ eval 'UNAME_M=arm64 python -m pip install llama-cpp-python==0.2.19 --only-binary=:all: --extra-index-url=https://jllllll.github.io/llama-cpp-python-cuBLAS-wheels/basic/cpu'

++ UNAME_M=arm64
++ python -m pip install llama-cpp-python==0.2.19 --only-binary=:all: --extra-index-url=https://jllllll.github.io/llama-cpp-python-cuBLAS-wheels/basic/cpu
Looking in indexes: https://pypi.org/simple, https://jllllll.github.io/llama-cpp-python-cuBLAS-wheels/basic/cpu
ERROR: Could not find a version that satisfies the requirement llama-cpp-python==0.2.19 (from versions: none)
ERROR: No matching distribution found for llama-cpp-python==0.2.19

[notice] A new release of pip is available: 23.2.1 -> 23.3.1
[notice] To update, run: pip install --upgrade pip
+ echo 'Failed to install llama-cpp-python'
+ exit 1
Failed to install llama-cpp-python

Seems it works fine with MacOS x86, I was able to create a test CI to prove it: https://github.com/gaby/testbench/actions/runs/7013936843/job/19080863382
Seems to be related to arm64/m1/m2

@gaby  Your Mac is recognizing itself as aarch64. I only have wheels for arm64 at the moment. They are both essentially the same thing. I'll upload some aarch64 wheels when I have the time. Can't find any other solution for that.

@jllllll According to Google aarch64 is another way of saying arm64.

"aarch64" and "arm64" are the same thing. AArch64 is the official name for the 64-bit ARM architecture, but some people prefer to call it "ARM64" as a continuation of 32-bit ARM.

Several users suggested building manylinux wheels, which would solve this issue.
https://github.com/pypa/cibuildwheel
https://cibuildwheel.readthedocs.io/en/stable/faq/#how-to-cross-compile

Not really sure how manylinux wheels are supposed to fix an issue on MacOS.
The true problem is that Python/pip pointlessly differentiates between aarch64 and arm64 despite them being the same and seemingly provides no option to override this.
Uploading new wheels is a trivial problem to fix as I just need to write a simple script to download all the MacOS wheels and upload a version that is renamed to use aarch64. Shouldn't need to rebuild anything.

Not really sure how manylinux wheels are supposed to fix an issue on MacOS.
The true problem is that Python/pip pointlessly differentiates between aarch64 and arm64 despite them being the same and seemingly provides no option to override this.
Uploading new wheels is a trivial problem to fix as I just need to write a simple script to download all the MacOS wheels and upload a version that is renamed to use aarch64. Shouldn't need to rebuild anything.

I see what you mean, cool! The main diff I read about manylinux, is that those wheels work dor both x86 and arm64.
I played with cibuildwheel a bit and even though it builds the wheels for arm64 it fails trying to link the llama libs. Not sure how to fix that yet, lol. The CI is way more simple though:
name: CI

on:
  push:
    branches: [ "main" ]
  pull_request:
    branches: [ "main" ]

jobs:
  build_wheels_macos:
    name: Build wheels for macos
    runs-on: macos-latest
    steps:
      - uses: actions/checkout@v4
        with:
          repository: 'abetlen/llama-cpp-python'
          ref: 'v0.2.20'
          submodules: 'recursive'
      - name: Build wheels
        uses<EMAIL_ADDRESS>        env:
          CIBW_ARCHS_MACOS: x86_64 universal2
          CIBW_PROJECT_REQUIRES_PYTHON: ">=3.10"
          CIBW_BEFORE_ALL: 'export CMAKE_ARGS="-DLLAMA_NATIVE=off -DLLAMA_METAL=on"'
          CIBW_BEFORE_BUILD: 'pip install build wheel cmake'
      - name: List wheels
        run: ls -la /wheelhouse/*.whl

@jllllll Thanks for your help, turns out my issue is related to running in Docker with MacOS. When running in Docker the platform is linux/arm64 not macos. Thus why it can't find any compatible wheels 😂


What's the value of the CSS?

We should either update the CSS to be more meaningful or delete it.

I'm going to add default HTML and CSS to get the UI set-up.


Wordpress Coding Standards & Unit Testing Setup via Travis CI

I was using this for a project that needed to follow standards, so I figured I would go ahead and see if you wanted it integrated into the master branch on the main repo. The unit test I have been using to check everything is based on the Underscores theme by Automattic, which can be found here: https://github.com/Automattic/_s
Essentially it checks for WordPress Coding Standards, best practices and syntax errors. It fits nicely into a boilerplate theme because if you want to use it, it's there, if not, it has no effect on your project. But for me it's always a pain to set up unit testing. This may get a couple people off to a good start and encourage the spread of WP Coding Standards :)
At this point, with a few adjustments, I got the FoundationPress build to pass. Honestly didn't take too much. Major kudos to the whole community for contributing solid code. The WP Coding Standards can be fairly strict.

Hello. Thank you for this! Right now I'm on vacation. Will definitely see properly through this when I'm back. It's brilliant that you share your expertise. Will do code review and take your PR soon as I return from travel.
Cheers,
Ole Fredrik

Great work!

Thanks guy's, at this point it would definitely be comforting to have a couple more eyes go over the code that was changed. The biggest things are function names that had to be tweaked from 'FoundationPress_' to 'foundationpress_' to eliminate camel casing and conform to WP Coding Standards, and making sure that my search and replace found all of the instances of those functions.
Also, I am definitely not a complete newbie, but also not an expert at setting up the .travis.yml file. But the one from the underscores repo is so well commented, I think everything in there is correct.

I haven't tested the code myself and I'm not familiar with Travis so I can't comment on that part. But I took a quick look at the diffs and it looks nice so far when it comes to adjusting to WordPress coding standards. I've been thinking more than once that I should do that, I'm glad you did!
I found one issue, the function FoundationPress_entry_meta() is renamed to foundationpress_entry_meta() in this pull request, but both content.php and single.php still contains a call to the function by the old name:
<?php FoundationPress_entry_meta(); ?>
And when we are talking about coding standards, I'd like to discuss some parts that could be changed as well.
For example the action hooks looks like this, with camelCase:
```do_action( 'foundationPress_post_before_comments' );````
The WordPress Naming Conventions say:

Use lowercase letters in variable, action, and function names (never camelCase).

…so I think the action hooks should be changed as well, if we want to go all-in on coding standard, right? That would break backward compatibility, though.
Class naming is also not following standard, it looks like this right now:
class FoundationPress_comments extends Walker_Comment{
The Naming Conventions say:

Class names should use capitalized words separated by underscores.

Awesome input! I definitely agree in terms of going all in. I have found that the Coding Standards "sniffs" throw flags at certain things sometimes and don't others. I thought it was just a bug on my local build, but it there must be something else going on.
Thanks for letting me know of those functions also. I was definitely a little concerned about stuff like that.
In terms of making sure that it's backwards compatible, do you think there are very many people coming back to download partial bits of code? I really don't know, I guess I have done it once or twice. What would be a good solution for that?

Great!
I really have no idea if backward compatibility is a big issue. Personally it's not. FoundationPress, AFAIK, is meant to be a starter theme that you modify rather than update but I've seen a lot of issues here where people want to use it as a traditional theme (like all those issues about child theme support) so I guess some do update. On the other hand, FoundationPress has already had big changes many times over. I would guess that these action hooks are not frequently used. Anyway, that's for Ole to decide.

Hi. I do not see backward compatibility as a major issue in this case. The changes proposed are good and will make the theme a lot better with regard to follow wordpress standard conventions for coding. I have looked through diffs (from mobile safari, since I'm on vacation) and it looks very good!  @Josh-Rathke, could you update your pull request with the input from @Ateles? If both of you have gone through the code and see that everything is running fine, I'm happy to merge the PR from my phone. Well done, guys!

Hey @olefredrik, just to make sure, you want all of the Class Names and Action Hooks to be changed as well? I think those are the last remaining instances of code that need to be brought up to par with WordPress standards.
I am travelling the next few days, so not sure when I'll be able to get it done, @Aetles if you can get to it first, I'll merge the PR in with my Repo so that it gets pulled into this PR. Otherwise, I'll get to it when I can.

Hi there. Just came back from vacation. I've reviewed your code and tested that everything runs fine, so I merged your pull request. The class names and action hooks should be changed as well, but we can make a separate PR for that later. I'm a little busy the next two weeks, going to New York in a few days and have loads which must be arranged prior to it. Again, thank you for your contribution so far :)


Clarify if a package is needed to use the AddProblemDetails method

The documentation section on ProblemDetals mentions using a method called AddProblemDetails but that method does not exist in the default packages of a web API. The documentation needs to clarify what package needs to be installed to be able to use this method.
At the bottom of the page, the NuGet package Hellang.Middleware.ProblemDetails is mentioned as an alternative to the code in the rest of the page. However, I found this article that says the aforementioned package has to be installed to use the AddProblemDetails method.
This documentation page also references the AddProblemDetails method without specifying what is needed to use it.

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 0e03fdf4-b19b-17c3-04f4-336ab6d2ee06
Version Independent ID: 38515dfb-91a5-b395-db9d-084bbaf095c8
Content: Handle errors in ASP.NET Core
Content Source: aspnetcore/fundamentals/error-handling.md
Product: aspnet-core
Technology: aspnetcore-fundamentals
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

The AddProblemDetails method and all with it is new and included by default in ASP.NET Core 7.
I just created a new web app to confirm.
Both docs pages you referenced are for version 7, thus it is included by default and therefore any mention of package installation is not necessary. The blog article you linked is from 2020, so before .NET 7 was released.

My bad.
Wasn't aware that there was such a big difference from version 6 to version 7.
Apologies for any inconvenience caused.

New problem details service in 7.0


Add a check for original as well as modified

When a deletion comes through from trogdir and it is just a string, then
the changeset does not have the key in the modified hash. This checks
for the key in either the modified hash or original hash.

:+1:


ARROW-10220 [Javascript] Javascript toArray() method ignores nulls on some types.

8385

 Quality gate: Passed
 Everything looks good
Details


i can't install the platformio-ide-terminal, i previously can but when i uninstalled atom the problem came into account. Not even this all the packages can't be installed.

[x] I have reset Atom to defaults prior to submitting report.
[ ] I have not reset Atom to defaults prior to submitting report.

Description

Steps to reproduce

Expected behavior:
Actual behavior:
Reproduces how often:
Versions

> atom --version

> apm --version

OS name and version:
Platformio-ide-terminal version:
Additional Information

Thank you for reaching out to use with this issue report.
Please do not ignore the issue template fields in future, this sort of disregard isnt helpful in any case, in this instance the issue is well know, and, many duplicates to #700 have been filed including yours.
There is a workaround, its been posted in this issue https://github.com/platformio/platformio-atom-ide-terminal/issues/707#issuecomment-514634895 < click that.
There are other workarounds in thread, so reading and typing more than the minimum is necessary.
Thanks for your understanding.


LUIS datetime issue after daylight saving ended.

Bot Info

SDK Platform: Node.js
Active Channels:  MS Teams, Cortana
Deployment Environment: Azure Bot Service

Issue Description
Until Friday 9th March my LUIS model was correctly returning dateTime value with type builtin.datetimeV2.datetime. Ex: If I say 'NOW' it will actually give time and date value for current time in (CST). On Sat. March 10 when daylight saving ended clocks moved 1 hr ahead. So now if I run same query with LUIS it still give timeDate as if daylight saving was not ended. Ex: for now it gives 1 HR lesser than current hour. I am not sure where I can fix this. Is there any param I need to have ?
Code Example
LUIS entities data for the query "find a meeting room now" where now is an entity type builtin.datetimeV2.datetime
  "entities": [
    {
      "entity": "now",
      "type": "builtin.datetimeV2.datetime",
      "startIndex": 12,
      "endIndex": 14,
      "resolution": {
        "values": [
          {
            "timex": "PRESENT_REF",
            "type": "datetime",
            "value": "2018-03-12 10:56:45"
          }
        ]

Expected Behavior
Above value should be  "value": "2018-03-12 11:56:45"
Can I expect LUIS to return updated time as per daylight savings updates ?
Actual Results
builtin.datetimeV2.datetime still returns CST time as per daylight savings. It is still running 1hr behind. Ex: if CST now is 11pm luis is still returning 10pm for NOW. This would have been true untill friday 9th March.

@rjgmail88 in which region is the app published and what's the selected 'Timezone' setting for it when publishing to production?

LUIS app is published in North America Region. The Timezone setting is defaulting to (GMT) Western Europe Time London even though I select Central Time US/Canada and publish.

By default LUIS resolves datetimes using UTC time. So “now” at this moment returns something like
{
    "timex": "PRESENT_REF",
    "type": "datetime",
    "value": "2018-03-15 03:49:46"
}

If you need resolution in a specific timezone, you can use a query parameter named timezoneOffset, that takes a time offset in minutes to be used to compute resolution.
The default offset value is 0. You test it by adding the parameter to a query? For example: “&timezoneOffset=480&q=I%20want%20to%20know%20the%20time%20now”.
Let me know if that works.

Yes, I did add timezoneOffset=-360 to my url. But still response is as follows which is 1 hr lesser than local CST. Following shows time 2018-03-16 14:22:50 for NOW it actually 15:22:50.
As mentioned in title of this thread all this used to work before daylight saving ended. I haven't modified anything.
{
      "entity": "now",
      "type": "builtin.datetimeV2.datetime",
      "startIndex": 12,
      "endIndex": 14,
      "resolution": {
        "values": [
          {
            "timex": "PRESENT_REF",
            "type": "datetime",
            "value": "2018-03-16 14:22:50"
          }
        ]
      }

https://westus.api.cognitive.microsoft.com/luis/v2.0/apps/{key}?subscription-key={key}&timezoneOffset=-360&verbose=true&q=find a room now

The timezoneOffset is always relative to UTC DateTime. The issue is that after the daylight saving, the actual offset between CST & UTC became 5 hours instead of 6, while the user’s code still uses a 6-hour offset (-360 minutes as you communicated below).
This is not a bug on LUIS, we ask our customers to give the offset relative to UTC DateTime.
A good solution could be to dynamically calculate the offset, for example:

Get the datetime of your target zone
Get UTC DateTime
Subtract them and pass that difference to LUIS in minutes

Example C# code snippet that we will be adding to our documentation:
TimeZoneInfo targetZone = TimeZoneInfo.FindSystemTimeZoneById("Central Standard Time");
DateTime utcDatetime = DateTime.UtcNow;
DateTime cstDatetime = TimeZoneInfo.ConvertTimeFromUtc(utcDatetime, targetZone);
int offset = (int)((cstDatetime - utcDatetime).TotalMinutes);

Thank you.


Enabled strictNullChecks. Note: has breaking changes

As described in #17 , I'm opening this PR just in case you want to check out the solution. But I'm neutral towards merging it; it provides compile-time check for something we're kind-of sure of, but, unfortunately, it breaks backward compatibility.
Sorry for opening it as PR, but I want to delete my fork and want to keep the project's knowledge in one place: your repo, if you don't mind.
Mostly expecting the PR to be closed/rejected.
Thanks again!

Hi @hossameldeen, have you checked the comments?

Yup, long time ago, and replied to them too :-)
In case you can't see them, here's the first reply, and the the other one. Please, tell me if there are any comments I've missed.

My bad, the comment has been pending to send for weeks, here it is.
Thank you @hossameldeen.


Support for '%' and 'n' presentation types in mpf.__format__

I've been working on these presentation types, to complete the set.
'%' is straightforward, just multiply the mpf by 100 and add a % sign at the end.
'n' is a lot trickier, since some locales may be difficult to program.
For instance,
>>> import locale
>>> locale.setlocale(locale.LC_ALL, 'en_IN.utf8')
>>> print('{:.10n}'.format(100000.25))
1,00,000.25

Would a partial implementation of 'n' (i.e., only care about thousands and decimal separators) be useful or should we not bother at all with it?

'n' is a lot trickier

Python as a language has much wider audience than mpmath and a lot of features, related to different locales, e.g.:
>>> float('٣.٣')
3.3

But
>>> mpmath.mpf('٣.٣')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/sk/src/mpmath/mpmath/ctx_mp_python.py", line 71, in __new__
    val = from_str(val, prec, rounding, base)
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/sk/src/mpmath/mpmath/libmp/libmpf.py", line 1348, in from_str
    man, exp = str_to_man_exp(x, base)
               ^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/sk/src/mpmath/mpmath/libmp/libmpf.py", line 1312, in str_to_man_exp
    x = MPZ(x, base)
        ^^^^^^^^^^^^
ValueError: string contains non-ASCII characters
>>> gmpy2.mpfr('٣.٣')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ValueError: string contains non-ASCII characters

should we not bother at all with it?

I think we could skip this, unless someone will came with real use cases.

I think we could skip this, unless someone will came with real use cases.

OK, so no 'n' presentation type.
I'll make a PR with '%' mode and then another one with the full documentation for __format__, if that's OK with you @skirpichev.

pr is ready for review: https://github.com/mpmath/mpmath/pull/847


EVP_Digest* tests

Checklist

[x] tests are added or updated

This adds tests for EVP_Digest* functions, both streaming and oneshot versions.

reconfirm


ServerListUpdateInProgress change should be atomic

Using an atomic compareAndSet, instead of separate get and set operations.

ribbon-pull-requests #227 FAILURE
Looks like there's a problem with this pull request

NetflixOSS » ribbon » ribbon-pull-requests #1 SUCCESS
This pull request looks good

ribbon-pull-requests #228 SUCCESS
This pull request looks good


Support parallel minor versions

Follow-up to https://github.com/microsoft/types-publisher/pull/723.
This was causing CI on DT to fail for https://github.com/DefinitelyTyped/DefinitelyTyped/pull/43179.
I couldn't figure out how to write a test to exercise this code-path. If it’s easy to point out, I’d gladly add test coverage.

Currently these are allowed:
react-native
react-native/v42

Whereas this is disallowed:
react-native/v0.42

The above should all be allowed now, but not those with patch components:
react-native/v0.42.1

I published 3.4.1 just now, so Definitely Typed's CI should get the new version next time it runs.

Thanks! 🙏


possible to use without Arduino?

Hi,
I am hoping to add usb midi to my esp32s2 project.
I am working in esp-idf, and don't have Arduino installed
Is there any way that this lib can work in esp-idf?
thank you!

Nope, sorry.
You have to use plain tinyusb.
https://github.com/hathach/tinyusb/tree/master/examples/device/midi_test


Add Receiver::new_sender and Sender::new_receiver

I've added some basic tests, let me know if it need more extensive tests. I'm also not sure the changes I've made are sufficient or if they also need changes inside the various flavours.

The CI failure seems to be unrelated.

Could you say more about what the use case for these methods is?

I have a use case where I'm holding a Receiver and a Sender in the same structure, where the Sender is only used to be cloned and shared. Since both are a pointer to the same structure it just wastes memory. I'm also using thousands of these structures so even small reductions add up.

Are there are reviewers available for this? I really need this feature.

Seems this needs to clarify what happens when called after the channel is disconnected.

This has taken too long and I've moved on to using a different crate.


Deploy kind clusters in parallel

This greatly reduces e2e deployment time.
Signed-off-by: Stephen Kitt<EMAIL_ADDRESS>

Parallelising subsequent steps involves handling multiple K8s contexts in parallel, which isn’t quite as straightforward.

This greatly reduces e2e deployment time.

OK, only 30s less on the Travis job, but on systems with the images already available the change is quite significant.


Timed Transition fire may fail if it gets disabled when some thread is sleeping

Background:

Given Transitions t0 and t1 share an input place p0

And t0 is timed [a,b]
And t0 is enabled at time ti
And timespan is set to [ti+a,ti+b]
And no thread is waiting for t0's timespan

When Thread th0 tries to fire t0 before timespan

And th0 goes to sleep on its own
And th1 fires t1

Then t0 gets disabled

And th0 wakes up at ti+a
And th0 tries to fire t0

Actual:

And th0 never verifies whether t0 was actually fired

Expected:

And t0 checks if t0 was fired
And if t0 was not fired, th0 sleeps in t0's condition queue

An improvement here would be th0 sleeping with highest priority in t0's queue when this occurs

Ready to verify


Uncaught TypeError: Cannot read properties of null (reading 'useReducer')     at Object.useReducer (react.development.js:1626:1)

I just followed next js setup tutorial and got a lot of errors:
client.js:1 Warning: An error occurred during hydration. The server HTML was replaced with client content in .
client.js:1 Warning: Container: Error boundaries should implement getDerivedStateFromError(). In that method, return a state update to display an error message or fallback UI.
Uncaught Error: There was an error while hydrating. Because the error happened outside of a Suspense boundary, the entire root will switch to client rendering.
Uncaught (in promise) Error: Cancel rendering route
Uncaught TypeError: Cannot read properties of null (reading 'useReducer')
client.js:1 The above error occurred in the  component

Hey @delbaoliveira, the problem was that I was using the last versions of the react, react-dom and next packages. After installing the dependencies below, everything worked fine although new features of next.js and react were not available because those packages are outdated:
{
  "dependencies": {
      "next": "^12.1.0",
      "react": "^17.0.2",
      "react-dom": "^17.0.2"
  }
}

Having the same issues.

I am not able to reproduce this. Please open a new issue if you are still having trouble here. Thank you!


I tried it...

Same versions and everything, I made a comment earlier.
Tried newest version and the BoP issues are still present. Volcanoes and Umbral woods are the ones I care about most, but there are a couple other biomes totally absent from the world after travelling hundreds of thousands of blocks in random directions and searching in a 10k radius each time.
Most other things seem to be fixed, and I was going to suggest an option to make a large type biome setting for servers (Imagine a desert being actually dangerous because of no shelter, food, or wood for days of travel, Gobi and Sahara type huge) but I haven't checked the in game settings. I'll rehash once I figure that stuff out, because that idea sounds awesome to me.
Thanks

TF doesn't guarantee that 100% of other mods' biomes will be used because it's not possible to 100% accurately identify and categorise all of them. You can use /terra locate biome & /terra locate terrain to see if any exist in the world from your location.
If you want a better chance of finding most biomes then reduce the climate/biome sizes. If you want giant deserts increase the biome/climate sizes


VOTSTS dataset

Hi, thank you for the work. You showed the results on the VOTSTS 2022 but VOT dataset is not placed under the datasets folder, and dataloader has no implementation for VOTSTS. Do you have a plan to add it?

Since the evaluation of VOT requires using the official toolkit, it's hard to integrate the VOT dataset into our aot-benchmark.
But, if you are interested in how to infer AOT-based models on VOT, you can refer to our MS-AOT tracker, which is the winner in the VOT 2022 challenge.

@z-x-yang Is the MS-AOT tracker code available? I believe the link you sent is a dead link now.


drafts: Drafts UI re-design

Can you post screenshots on the changes?

The changes thus far look great; merged, thanks @vinitS101!


Compatibility with tools that use RF instead of durability [old: Cyclic Automatic Fishing Net and Redstone Arsenal]

Right. So. I put a Flux-Infused Fishing Rod into the Automatic Fishing Net and it eats the rod, literally. It catches a fish then the rod vanishes completely. O.o
This is Cyclic 1.10.2 - 1.10.22 ...

Wat mod is this item from?
Most fishing rods go down one durability when you use them.  What happens when you use this one outside the machine? I assume it is unbreakable or somehow consumes some other resource?
The Automatic Fishing Net currently doesnt understand RF power if thats what it is charged with, it would be a nice upgrade though.

As I said in the title, Redstone Arsenal, and yeah, it's a RF powered fishing rod. Which is why I was using it, was going to set up a pull / charge / put back setup like Direwolf20 is doing for his beheader sword.

The tools from the Redstone Arsenal in then versions for MC1.10.2 has no metadata for damage. They use NBT data instead of metadata. Different form IC2 Electiric tools.

Thanks for information everyone.  In current 1.10 builds in my testing, it does not eat the rod anymore at least, which is a good thing.
I am changing this from a bug to a feature request: that feature being "working with RF tooks and drain RF instead of durability".
This feature could apply also to other tools using RF such as auto user, controlled miner, etc.
I am no longer adding new features to 1.10, but all the Thermal mods are in 1.12 now.

It looks like this bug still hasn't been fixed.  I'm playing All the Mods 3 for Minecraft 1.12 and just experienced the same "bug" as the OP.  I made the RF Fishing Rod, put it in the fisher, watched it catch a fish and then it just disappeared.  Is there a fix underway for this since this is 1.12 and you're making it for 1.12?

Added this to RF. thanks for being patient, took me a while @JaRyCu  . latest version tested and working with flux infused rod https://minecraft.curseforge.com/projects/cyclic/files/2501283


Error on inline math

There is an exception when processing markdown file containing inline math via Sphinx. The same markdown file previously worked.
Here is a Travis log.
The immediate error message is
Exception occurred:
  File "/home/travis/virtualenv/python3.6.3/lib/python3.6/site-packages/docutils/parsers/rst/roles.py", line 356, in math_role
    text = rawtext.split('`')[1]
IndexError: list index out of range

I can also reproduce on my machine by adding an inline math expression
 `$ a $`

to tests/sphinx_custom_md/index.markdown and running python3 -m unittest -q tests.test_sphinx.CustomExtensionTests. Here is the full output
Running Sphinx v1.7.9
making output directory...
loading pickled environment... not yet created
building [mo]: all of 0 po files
building [html]: all source files
updating environment: 1 added, 0 changed, 0 removed
reading sources... [100%] index
/usr/local/lib/python3.5/dist-packages/docutils/io.py:245: DeprecationWarning: 'U' mode is deprecated
  self.source = open(source_path, mode, **kwargs)
======================================================================
ERROR: test_integration (tests.test_sphinx.CustomExtensionTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/code/recommonmark/tests/test_sphinx.py", line 66, in test_integration
    ['</table>']
  File "/code/recommonmark/tests/test_sphinx.py", line 23, in _run_test
    app.build(force_all=True)
  File "/usr/local/lib/python3.5/dist-packages/sphinx/application.py", line 325, in build
    self.builder.build_all()
  File "/usr/local/lib/python3.5/dist-packages/sphinx/builders/__init__.py", line 299, in build_all
    self.build(None, summary='all source files', method='all')
  File "/usr/local/lib/python3.5/dist-packages/sphinx/builders/__init__.py", line 355, in build
    updated_docnames = set(self.env.update(self.config, self.srcdir, self.doctreedir))
  File "/usr/local/lib/python3.5/dist-packages/sphinx/environment/__init__.py", line 568, in update
    self._read_serial(docnames, self.app)
  File "/usr/local/lib/python3.5/dist-packages/sphinx/environment/__init__.py", line 587, in _read_serial
    self.read_doc(docname, app)
  File "/usr/local/lib/python3.5/dist-packages/sphinx/environment/__init__.py", line 662, in read_doc
    doctree = read_doc(self.app, self, self.doc2path(docname))
  File "/usr/local/lib/python3.5/dist-packages/sphinx/io.py", line 302, in read_doc
    pub.publish()
  File "/usr/local/lib/python3.5/dist-packages/docutils/core.py", line 218, in publish
    self.apply_transforms()
  File "/usr/local/lib/python3.5/dist-packages/docutils/core.py", line 199, in apply_transforms
    self.document.transformer.apply_transforms()
  File "/usr/local/lib/python3.5/dist-packages/sphinx/transforms/__init__.py", line 96, in apply_transforms
    Transformer.apply_transforms(self)
  File "/usr/local/lib/python3.5/dist-packages/docutils/transforms/__init__.py", line 171, in apply_transforms
    transform.apply(**kwargs)
  File "/code/recommonmark/recommonmark/transform.py", line 343, in apply
    self.traverse(self.document)
  File "/code/recommonmark/recommonmark/transform.py", line 313, in traverse
    self.traverse(child)
  File "/code/recommonmark/recommonmark/transform.py", line 313, in traverse
    self.traverse(child)
  File "/code/recommonmark/recommonmark/transform.py", line 303, in traverse
    newnode = self.find_replace(c)
  File "/code/recommonmark/recommonmark/transform.py", line 287, in find_replace
    newnode = self.auto_inline_code(node)
  File "/code/recommonmark/recommonmark/transform.py", line 214, in auto_inline_code
    return self.state_machine.run_role('math', content=content)
  File "/code/recommonmark/recommonmark/states.py", line 134, in run_role
    content=content)
  File "/usr/local/lib/python3.5/dist-packages/docutils/parsers/rst/roles.py", line 356, in math_role
    text = rawtext.split('`')[1]
IndexError: list index out of range

Here is a Travis log from a few weeks ago where the same file is processed successfully. Unfortunately I can't tell what the relevant difference is. (Some version numbers have changed, but installing the older Sphinx on my own machine, for example, does not remove the problem.)

This caught me out too. Sphinx needs to have a math extension enabled.
I think this might actually be a minor bug in docutils as if you work around it there is a useful warning message.

WARNING: using "math" markup without a Sphinx math extension active, please use one of the math extensions descri bed at http://sphinx-doc.org/en/master/ext/math.html

Any more details on how to fix it? I did a PR that hacked it, but it would be useful if it's just a conf.py change. I see it's a Sphinx 1.8 issue.

I'm still getting the error on Sphinx 1.8.
In sphinx 1.7 it uses Sphinx's math role:
ipdb> inspect.getfile(role_fn)
'/Users/eric/.pyenv/versions/3.6.6/lib/python3.6/site-packages/sphinx/ext/mathbase.py'

In Sphinx 1.8 it uses the docutils one, which is where this error comes from:
ipdb> inspect.getfile(role_fn)
'/Users/eric/.pyenv/versions/3.6.6/lib/python3.6/site-packages/docutils/parsers/rst/roles.py'

Thanks @Anntoin, that explains why my modified unit test failed with all versions. However, the repo whose Travis log I linked already has sphinx.ext.mathjax.
Thanks @ericholscher, and sorry if I opened this in the wrong project. I now see the same thing as you: Sphinx 1.7.9: no error. Sphinx 1.8.0: error.

Can confirm the workaround to use Sphinx 1.7:
pip install sphinx==1.7

how do I force 1.7 on readthedocs?
the problem is that it runs:
conda install --yes --quiet --name latest mock pillow sphinx sphinx_rtd_theme
after installing the custom user environment and that forces sphinx to upgrade.
I even tried to install mock pillow sphinx=1.7 sphinx_rtd_theme in my user environment but still the command above upgrades to 1.8.

Encounter the same problem. sphinx version 2.0.1.
But OK with 1.7.9.

I'm also using 1.7.9,  along with these dependencies (inside a Flask Application)
root@ac0b60dc58c3:/src/app# pip freeze | grep -iwE '(sphinx|recommonmark|docutils|websupport)'
docutils==0.14
recommonmark==0.5.0
Sphinx==1.7.9
sphinx-rtd-theme==0.4.3
sphinxcontrib-websupport==1.1.2 # adds WebSupport

And I'm running into this error
root@4e00e44b18a6:/src/app# python build_docs.py
Running Sphinx v1.7.9
loading translations [es]... done
loading pickled environment... not yet created
building [mo]: targets for 0 po files that are out of date
building [websupport]: targets for 9 source files that are out of date
updating environment: 9 added, 0 changed, 0 removed
/src/venv/lib/python3.6/site-packages/recommonmark/parser.py:65: UserWarning: Container node skipped: type=document
  warn("Container node skipped: type={0}".format(mdnode.t))
Traceback (most recent call last):rs/ise
  File "build_docs.py", line 27, in <module>
    build_docs()
  File "build_docs.py", line 23, in build_docs
    support.build()
  File "/src/venv/lib/python3.6/site-packages/sphinxcontrib/websupport/core.py", line 146, in build
    app.build()
  File "/src/venv/lib/python3.6/site-packages/sphinx/application.py", line 331, in build
    self.builder.build_update()
  File "/src/venv/lib/python3.6/site-packages/sphinx/builders/__init__.py", line 342, in build_update
    'out of date' % len(to_build))
  File "/src/venv/lib/python3.6/site-packages/sphinx/builders/__init__.py", line 355, in build
    updated_docnames = set(self.env.update(self.config, self.srcdir, self.doctreedir))
  File "/src/venv/lib/python3.6/site-packages/sphinx/environment/__init__.py", line 568, in update
    self._read_serial(docnames, self.app)
  File "/src/venv/lib/python3.6/site-packages/sphinx/environment/__init__.py", line 587, in _read_serial
    self.read_doc(docname, app)
  File "/src/venv/lib/python3.6/site-packages/sphinx/environment/__init__.py", line 662, in read_doc
    doctree = read_doc(self.app, self, self.doc2path(docname))
  File "/src/venv/lib/python3.6/site-packages/sphinx/io.py", line 302, in read_doc
    pub.publish()
  File "/src/venv/lib/python3.6/site-packages/docutils/core.py", line 218, in publish
    self.apply_transforms()
  File "/src/venv/lib/python3.6/site-packages/docutils/core.py", line 199, in apply_transforms
    self.document.transformer.apply_transforms()
  File "/src/venv/lib/python3.6/site-packages/sphinx/transforms/__init__.py", line 96, in apply_transforms
    Transformer.apply_transforms(self)
  File "/src/venv/lib/python3.6/site-packages/docutils/transforms/__init__.py", line 171, in apply_transforms
    transform.apply(**kwargs)
  File "/src/venv/lib/python3.6/site-packages/recommonmark/transform.py", line 328, in apply
    self.traverse(self.document)
  File "/src/venv/lib/python3.6/site-packages/recommonmark/transform.py", line 308, in traverse
    self.traverse(child)
  File "/src/venv/lib/python3.6/site-packages/recommonmark/transform.py", line 308, in traverse
    self.traverse(child)
  File "/src/venv/lib/python3.6/site-packages/recommonmark/transform.py", line 308, in traverse
    self.traverse(child)
  File "/src/venv/lib/python3.6/site-packages/recommonmark/transform.py", line 298, in traverse
    newnode = self.find_replace(c)
  File "/src/venv/lib/python3.6/site-packages/recommonmark/transform.py", line 282, in find_replace
    newnode = self.auto_inline_code(node)
  File "/src/venv/lib/python3.6/site-packages/recommonmark/transform.py", line 211, in auto_inline_code
    return self.state_machine.run_role('math', content=content)
  File "/src/venv/lib/python3.6/site-packages/recommonmark/states.py", line 133, in run_role
    content=content)
  File "/src/venv/lib/python3.6/site-packages/docutils/parsers/rst/roles.py", line 356, in math_role
    text = rawtext.split('`')[1]
IndexError: list index out of range

While trying to debug this, I found out that if I downgrade sphinxcontribwebsupport to 1.1.0 the error dissapears.
Hope this helps someone.

It seems that there are multiple pull requests for this fix but none of them is merged.
There are simple workaround $` abc `$

Encounter the same problem. sphinx version 2.0.1.

Encounter the same problem. sphinx version 2.0.1.
But OK with 1.7.9.

try sphinx 1.7.9 or 1.6.9, failed

I encounter this issue as well with sphinx 3.2.1


Build for Python 3.8

We're looking to move our dev environment to 3.8, but currently this is our blocker
It looks like there's a Python 3.8 build on conda-forge that seems to work just fine on a manual install. Can I make a PR that enables the build for 3.8 or do you have blockers? (I couldn't find anything)

Sorry for the delay—I'll work this into the PR I'm currently working on

Keep in mind, however, that the right thing to do is built a noarch package and then test it on a matrix of Pythons.

We're doing this now in current CI. Can't speak for conda-forge but I'm reasonably sure they are as well.


LEGO: Pull request from lego/hb_946_20170804211736446 to master

LEGO: Pull request from lego/hb_946_20170804211736446 to master with localized lcls

@csigs,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Latitude/longitude option 'don't summarize'

Dear weiweicui,
I really like the Flowmap! I'm experiencing 1 problem.
If I'd like to plot for example a point in Mâcon (France / picture 1) the location ends up in the United States. So I would like to solve this with the latitude/longitude option, only the 'count' option (picture 2) makes this unpossible for me.
Visual 'ArcGIS Maps for power BI' (picture 3) gives the opportunity to 'don't summarize' the latitude and longitude. Could you make this option available for your flowmap?
Thanks.

That is strange. It looks like your AdrToLongitude field is recognized as a string field. Not sure why PBI does that, maybe your field contains other things than decimals... Once the field is accepted as numeric, you should see the "Average" option and please use it.

Hi, I have the same problem with latitude and longitude. Has there been any progress on this issue?

Hi, I have the same problem with latitude and longitude. Has there been any progress on this issue?

Hi.  I found a workaround - change the summarization for the lat/long data fields before you drag it into the FlowMap visual (change it to Average or Min/Max).  After inserting it into FlowMap, you still cant change the summarization, but that's ok.  You already set it :)  Good luck.

I have this same problem. Field setting are decimal numbers. Table setting are Geographic and Don't summarise. But still when I drag the filed into Flow Map is sets it to Count, with no option to change it, and the plots don't work. With Place names it is great.

I have this same problem. Field setting are decimal numbers. Table setting are Geographic and Don't summarise. But still when I drag the filed into Flow Map is sets it to Count, with no option to change it, and the plots don't work. With Place names it is great.

In the newest versions of PBI, this issue persists. In my case, a solution was to use the fields Origin and Destination with concatenated coordinates: Latitude, Longitude.


Fallback to get_common_ancestor_commit_hash for manual GL pipelines

Workaround/fallback for undocumented GitLab behaviour: if a pipeline is
manually run against any branch (i.e. it is not triggered by a commit or
by a MR or PR), CI_COMMIT_BEFORE_SHA is set to a string of 40 zeros. In
such case, to avoid that commit range check fails with error, we need to
fallback to get_common_ancestor_commit_hash
Solves #92

some more tests required, I'm closing this PR and I will open another one


Fetch the latest Hex package version

https://github.com/nimblehq/elixir-templates/projects/2#card-45281172
What happened
Fetch the latest hex package version
Insight
Instead of hard code the version we could fetch the latest version when the template is trigger.
This PR solving 2 problems:

Reduce the maintenance effort as we no need to increase the version manually, open a PR, merge, release,...
Always fetch the latest version when applying the template

I don't use the Application environment for this piece of code because that isn't the best practice for a library
      defp hex_package(:test), do: Nimble.Phx.Gen.Template.Hex.PackageMock
      defp hex_package(_env), do: Nimble.Phx.Gen.Template.Hex.Package

Proof Of Work

The unit test suite should be passed (thanks to Mox)
The integration test also passed

Looks good. I would still prefer to remove the setup block as discussed in #26. It would remove the coupling in all tests.

It doesn't remove actually, it makes the setup as hidden by moving to the AddonCase, so the test will look magical IMO.

Looks good. I would still prefer to remove the setup block as discussed in #26. It would remove the coupling in all tests.

I removed the setup block and adding the @describetag mock_latest_package_versions: [{:xx, "xx"}] tag already ;)


Update bluesky-stomp and prevent duplication of BasicAuthentication

Fixes #636
Current parsing of config passes a blueapi BasicAuthentication with fields username, passcode as an instance of bluesky-stomp BasicAuthentication with fields username, password. I am not certain that we would be able to pass the former as an instance of the latter even if their arguments agreed, as they are different Pydantic models. We can reuse bluesky-stomp's BasicAuthentication as part of the config of blueapi and not duplicate the logic.

dev-requiremnts need an update


fix the app crashing when user is not signed in; prompt user by displ…

I fixed the app crash when user wallet is not connected when using the index tokens

I cherrypicked these changes and moved them in. Bunch of merge conflicts (my fault) and it just ended being easier.


For discussion: improve isConsequences

It seems to be working as expected now. Given
a.
b.
c.
{d}.
e :- not d.

#project a.
#project e.

#script (python)

def on_model(m):
    cons = []
    for atom in m.context.symbolic_atoms:
        if m.is_consequence(atom.literal):
            cons.append(atom.symbol)
    print(f"consequences: {' '.join(str(x) for x in cons)}")

def main(ctl):
    prj = set()
    ctl.ground()
    ctl.solve(on_model=on_model)

#end.

the output is
$ clingo project.lp --project 0
clingo version 5.7.2 (3d2f2b6e)
Reading from project.lp
Solving...
consequences: e a
Answer: 1
a b c e
consequences: a
Answer: 2
a b c d
SATISFIABLE

Models       : 2
Calls        : 1
Time         : 0.048s (Solving: 0.00s 1st Model: 0.00s Unsat: 0.00s)
CPU Time     : 0.048s

Should we just merge?

Should we just merge?

Fine with me 👍


Owner Change Implementation

Finding discovered during Red4Sec security audit.
Description
As it has been detected in the code, currently the smart contract does not contain any functionality to change the owner of the contract. This functionality may be very useful in the future, so we highly recommend using an OpenZeppelin's library that allows you to change the owner.
This library is https://github.com/OpenZeppelin/openzeppelin-contracts-ethereum-package/blob/master/contracts/ownership/Ownable.sol and it would be enough if the contract CentralizedBlockRelay.sol:26-28 and WitnetRequestBoard.sol:129 inherited Ownable.sol contract from OpenZeppelin to be able to modify the owner.

Impact
We consider that including this feature would be a good option in some extreme cases such as;

Transfer of the Token and its Infrastructure to third-parties.
Cyberattacks.
Human Failures.
Exposure of the owner's private key.
Internal attacks of disgruntled employees or anyone who at some point have had access to the private key.

By implementing this new feature, the team could face these possible situations in the future.

Code Reference
https://github.com/witnet/witnet-ethereum-bridge/blob/cf4ffe9434be79ec29079a3fcaf885adfb213cb8/contracts/WitnetRequestsBoard.sol#L129
https://github.com/witnet/witnet-ethereum-block-relay/blob/a3bf55cc58dd8295d3ac4daed45de48a708823fc/contracts/CentralizedBlockRelay.sol#L27-L28

Mitigation

Implement mechanisms for modifying the contract owner.
Inherit contracts from OpenZeppelin since these have been previously audited and can facilitate this implementation.

I may be wrong on this, but to my best knowledge this feature is already enabled implicitly by its proxy.
Given that the proxy allows updating the address of the WRB "controller" as long as the conditions set by the current instance are met, my impression is that replacing the current instance for a different one that sets a new owner offers a somehow equivalent degree of functionality in terms of dealing with the extreme situations you explained.
Of course replacing the WRB could have a bigger cost impact (as a new instance needs to be deployed) and there could be long term consequences of a lengthier controller history. The consequences of replacing the WRB for a bridge node operator are not trivial either, but that's easy to if the bridges use the proxy to resolve the address of the latest controller.
So, taking all of that into account, and also considering that CentralizedBlockRelay.sol is a provisional contract that should only live on mainnet for as little time as possible and should be superseded by ABSBlockRelay.sol as soon as possible, is still your recommendation in force?
Thanks in advance for your feedback!


export with Python show Unsupported command [dragAndDropToObject

ERROR: Caught exception [ERROR: Unsupported command [dragAndDropToObject | //div[@id='crowdPickMainArea']/div/div/div/div[2]/div/div | //div[@id='crowdPickConditionArea']/div/div]]
But I need the python scripts with the drag_and_drop method,I want  to know how katalon-recorder do this

Hey @03128crz, I see that you closed the issue that you created within 10 days.
Can you please explain how did you manage to fix this issue?
I have a similar one:
# ERROR: Caught exception [ERROR: Unsupported command [selectFrame | index=0 | ]]

and it's blocking my Python script!

Hey @03128crz, I see that you closed the issue that you created within 10 days.
Can you please explain how did you manage to fix this issue?
I have a similar one:
# ERROR: Caught exception [ERROR: Unsupported command [selectFrame | index=0 | ]]

and it's blocking my Python script!


balena-intel: rename connectivity packagegroup

Rename packagegroup-resin-connectivity to packagegroup-balena-connectivity and update all references.
This PR needs to be merged when balena-os/meta-balena#2187 has been merged.
Change-type: patch
Changelog-entry: balena-intel: rename connectivity packagegroup
Signed-off-by: Mark Corbin<EMAIL_ADDRESS>

Can one of the admins verify this patch?

@resin-jenkins add to whitelist

@balena-ci I self-certify!


Bump analyzer to v5

Previously we just add deprecated_member_use: ignore to the analysis_options.yaml file in the monorepo root. Now analyzer has gone to v5 so the usage of deprecated APIs must be dealt with now.
The hardest parts are the last three, since they change the class returned or require another variable, so likely we'll have to work around them.

[x] Replace getType with getClass for LibraryCompilationUnitElement and LibraryElement.
[ ] Replace name with getDisplayString for DartType. Note: need to think about whether nullability
[x] Replace element with element2. Note that the return type changes from ClassElement to InterfaceElement, so it requires a bit more work. Most times we should be able to just cast it to ClassElement since ClassElement implements InterfaceElement.
[x] Replace enclosingElement with enclosingElement3.
[x] Somehow replace resolveToBound from DartType with TypeSystem's instance method resolveToBound.
[ ] Replace parts (CompilationUnitElement) with parts2 (PartsElement) from LibraryElement.
[ ] Replace import (ImportElement) with imports2 (LibraryInputElement) from LibraryElement.

Replace element with element2. Note that the return type changes from ClassElement to InterfaceElement, so it requires a bit more work. Most times we should be able to just cast it to ClassElement since ClassElement implements InterfaceElement.

FYI in case you want to migrate to the current version of analyzer and plan to have no lints about deprecated members: with analyzer 5.2 element2 is deprecated and element is the new one (The old element is removed in 5.0).

Yeah, so my process will be:

Migrate deprecated API's in analyzer v4
Upgrade analyzer to v5, ensure everything is still working
Migrate deprecated API's again
Then I'll probably have to repeat these steps for analyzer v6

As I might've said previously, unfortunately the analyzer is probably one of the most tiresome packages to keep up-to-date with.

Any progress? I'm now doing the same, fixing all static analysis warnings and upgrading to the new analyzer.

Sorry, not really. It's the busiest time of the year for me and I've been having a bad cold/flu. I am blocked on the resolveToBound issue, particularly with the case when dartType is FunctionType (see where resolveToBound is defined).

@ykmnkmi Also, I'm absolutely okay if you want to use the existing changes in your PR. You don't need to let me know.

Fixed by #37.


[CT-2850] [Feature] Improve the error message for dbt clone when source = destination

Is this your first time submitting a feature request?

[X] I have read the expectations for open source contributors
[X] I have searched the existing issues, and I could not find an existing issue for this feature
[X] I am requesting a straightforward extension of existing dbt functionality, rather than a Big Idea better suited to a discussion

Describe the feature
Mentioned by @aranke here and here:

Something else to note here is that you cannot use the same target directory for both the source and target

You get a very weird error that source and target states are the same, and so it's a no-op.
I ran into this during local development, so think this warning needs to be surfaced loudly.

This happens because

dbt now (over)writes manifest.json during parsing.
[…]
during the subsequent run (step 4) in the reproduction case, it overwrites target/manifest.json, then reads it back in to detect changes, and finds no changes.

from @jtcohen6 in https://github.com/dbt-labs/dbt-core/issues/7790#issuecomment-1578738124.
Describe alternatives you've considered
Simple solution that will work:

Write output to tmp directory during dbt run
Copy tmp directory to the output directory at the end of the dbt run

Who will this benefit?
Anyone using state in any form (including deferral, clone) reusing the same artifact directory.
Are you interested in contributing this feature?
Yes
Anything else?
There are 3 possible solution here, from easiest to hardest:

Raise a warning when the input and output artifact directories are the same
Write to tmp directory as mentioned above
Investigate root cause and fix underlying issue

Lots of places we could try to catch this. The most obvious to me (but it's Jinja) is inside the clone materialization, as soon as we know that target_relation == defer_relation:
https://github.com/dbt-labs/dbt-core/blob/8350dfead3cf3a9257d1d4e7f76d0a7e1f04d9d3/core/dbt/include/global_project/macros/materializations/models/clone/clone.sql#L34-L37


README.md: Use IPv4 notation for localhost

On machines with IPv4 and IPv6, running the python SMTP server on localhost
may only listen on the IPv6 loopback, and the unit tests will then fail to
connect.

https://github.com/lettre/lettre/pull/769#issuecomment-1140418584 🙄


Add test/Project.toml

Uses the new approach to test environments available in Julia v1.2 and beyond.

It is strange that the test with a *.txt image is failing on MacOS. I understood that ReferenceTests.jl would simply compare the data as Julia arrays without image encodings so seeing a different that causes the test to fail is interesting. Any thoughts on this @johnnychen94 ?

I will merge this given that the error is unrelated.

It is strange that the test with a *.txt image is failing on MacOS. I understood that ReferenceTests.jl would simply compare the data as Julia arrays without image encodings so seeing a different that causes the test to fail is interesting.

Sorry for the delay. I was experiencing the same issue when I tried to add ReferenceTests to this package. It might be a ReferenceTests issue but I failed to figure it out.


Add zmerlynn to review rotation for examples/

@mikedanese @zmerlynn

Looks like your copy of contrib is way out of date. This needs a rebase.


bump guest RAM size to 768 MiB

With the current 256 MiB guest RAM size, kernel 4.18.0-161.el8.x86_64
fails to boot:

EFI stub: UEFI Secure Boot is enabled.
EFI stub: ERROR: Failed to allocate usable memory for kernel.
efi_relocate_kernel() failed!
efi_main() failed!

at which point the guest kernel hangs voluntarily, so we never complete
the varstore template verification.
According to
https://access.redhat.com/documentation/en-us/red_hat_enterprise_linux/8/html/performing_a_standard_rhel_installation/system-requirements-reference_installing-rhel#check-disk-and-memory-requirements_system-requirements-reference,
"Table B.1. Minimum RAM requirements", a RAM size of 768 MiB suffices for
a "Local media installation" of RHEL8. Given that our use case does not
involve any userspace payload (such as Anaconda etc), 768 MiB is good
enough for us too. Bump the "-m" option's argument accordingly.
Signed-off-by: Laszlo Ersek<EMAIL_ADDRESS>

Thank you all!
Laszlo


Boost and ZLib vulnerabilty CVE-2018-25032

Hi Boost community,
I detected (using FNCI reports) that boost rely on ZLib 1.2.11 in libs/beast/test and tools/boost_install/test. Recently ZLib was updated to version 1.2.12 to fix a vulnerability (CVE-2018-25032), more informations in this article. There is any plan to use this new version ?
Thanks,
Etienne

Beast already updated that https://github.com/boostorg/beast/pull/2489, but it wasn't merged in time for the 1.80 release. I cannot speak for tools/boost_install (or whether that detection represents an actual dependency).
Next release will have the change.

If you can grab the develop branch of beast for now, it has the fix.

merged? can this be closed?

Verify that the fix is in, that the CHANGELOG has it, and that the in-progress release notes for Boost has it, and then close it.

Fixed in https://github.com/boostorg/beast/commit/ca824f607ccab6682b955eb91ed49753e6433069 and https://github.com/boostorg/beast/commit/925043e1e8df7474165f23c9a60223b0ab886be4.
Mentioned in the CHANGELOG: https://github.com/boostorg/beast/blob/d0dd9c50694bcbb8836775e780eb046f66b3e1d7/CHANGELOG.md?plain=1#L157


Change AndroidTest target SDK to version 19

The AndroidTest application should be targeting an version of android which runs on a phone, rather than only on wearables. This PR lowers the target SDK version to API level 19 (Android 4.4).
Reviewer @mikerhodes
Reviewer @indisoluble

Sounds good, +1.

+1
API level 20 is equal to 19 plus the wearable extension. No point in having that constraint.


Wordpress plugin

Hello Guys,
First of all, thank you for creating and maintaining this great tool !
I have a feature request: can you provide us a Wordpress plugin to share source code inside blogs post.
Thank you !

@nebrass this is a great idea! This is not something the Carbon core team is going to create or support, but if someone from the community creates it, we would be happy to add it to our Community section on our README

Closing since our team isn't going to create this — happy to add it to our README is someone creates this!


bind <C-[> will effect <ESC> in visual mode

        move = {
            enable = true,
            goto_next_start = {
                ["<C-]>"] =<EMAIL_ADDRESS>            },
            goto_previous_start = {
                ["<C-[>"] =<EMAIL_ADDRESS>            },
        },

bind <C-[> will cause <ESC> can not work in visual mode, I guess is because treesitter-obj move-map is default remap = true ?

This is because Ctrl+[ is equivalent key to ESC. This is a vim's default behaviour, so don't map this key.


update build tools version & fix travis error

Add before_script for travis-ci about emulator settings.  status
update gradle wrapper version to 2.3
update demo build tools version to 22.0.1

Great, the get passed from travis-ci. Please do your first merge. :-D

:)


Adding Android Environment Variables

Changes

Adding Environment Vars for Android Build Tools/Java to unity-editor wrapper and to .bashrc

Checklist

[x] Read the contribution guide and accept the code of conduct
[x] Readme (updated or not needed)

Super, thank you!!


[EAK-366] Implemented Autocomplete component

Generally very nice, thank you 👍 . Please name either the pull-request and the commits according to the requirements: [EAK-365] Bla-bla https://github.com/exadel-inc/etoolbox-authoring-kit/blob/master/CONTRIBUTING.md#procedural-pull-request-questions Please test/make sure that the new component works a) in multifields b) with DependsOn actions, such as "visible", "disabled", and especially "set" and "set-if-blank"

Tested in multifields, with DependsOn actions, such as "visible", "disabled", and especially "set" and "set-if-blank"


On runtime code-highlighting.

Hey,
I am creating an IDE and I want to underline wrong variables that don't exit with a red line.
So is there a way I can access the coloring list and modify it ?
I hope you understand me, If not tell me.

Ok.. Just noticed that my question is answered in the readme... Thanks anyways :P
I suggest you better move that explanations to the wiki tab... Because the wiki is the first thing I checked and never noticed the explanation in the read me.

Yes, the README.md has grow so large that it's starting to lose effectiveness. I have plans to migrate it to the Wiki.


"Run in current namespace" missing in Spyder 5.0.0a1

The Spyder Run options are missing from the latest version of the software.
In particular, it is not possible to set the option for running scripts in "console's current namespace instead of an empty one", as described here: https://github.com/spyder-ide/spyder/pull/10032
Is there a workaround for setting "current_namespace=True" to all runfile calls?
I need to define variables on-the-fly and use them while running script files.
Thank you for your help.

@goanpeca, please take a look at this one.

Is there a workaround for setting "current_namespace=True" to all runfile calls?

Sorry, there isn't.

I need to define variables on-the-fly and use them while running script files.

Please go back to 4.1.4. 5.0.0a1 is still very alpha and has several issues we need to fix.


fix(bytecode): make bytecode builds functional

This fixes two typos that were blocking bytecode builds from working:
reasion_skia_rect_set_byte => reason_skia_rect_set_byte
reason_skia_matrix_set_color_byte => reason_skia_paint_set_color_byte
These are hard to catch if we don't enable bytecode builds in CI. That's maybe something we should consider 😃

After some tracing and a ton of help from @bryphe (Thanks!), I got bytecode builds working for the project. They work in both the SDL example here and for the Revery example app!
I also changed the scope of this PR from fixing just those two small typos to fixing the bytecode builds in general.
This is a huge step on the way to getting hot-reload to work in Revery!


Implement terraform interpolations

Currently, some of the values cannot be parsed because there are some terraform interpolations there.  In some cases, it might be impossible to retrieve the data ( e.g. using remote module outputs ).
At least there must be a method for general use for terraform interpolations. Then we can just iterate and add more support on iterations throughout the terraform lifecycle.

Shouldn't this be done at terraform_validate level?

The plan is to remove terraform_validate dependency in near future. It has been added too many things within the terraform-compliance that terraform_validate doesn't support. Might be much easier without having the dependency.

I don't like waiting for dependencies, else it will be a blockchain application 😋

All interpolations are supported with 1.0.0 release.