High memory consumption during linking when using ModuleSplitStyle.SmallModulesFor(...)

This is based on a discord discussion.
We have a webapp using  ModuleSplitStyle.SmallModulesFor(...).
Scala 3.2.2, Scalajs 1.13
fullLinkJS (and fastLinkJS) produces a 7GB HashMap full of 50 million linker.frontend.modulesplitter.Tagger$DynamicPaths.
Which makes the compilation painfully slow.
No issues with the default split style ModuleSplitStyle.FewesModules, and also no issue with ModuleSplitStyle.SmallestModules
A tree that has the issue (not minimized at all) is available here:
https://github.com/reform-org/reform/commits/7ac6e83a4253d28919938b057883bd8ce39dd630
sbt fullLinkJS is enough to trigger the issue. I tested the split style by changing line 36 build.sbt.
@gzm0 tagging you as you requested this issue as a reminder (and I don’t think I can assign people) :-)

Thanks for the report.
I'm not able to checkout that commit from that repo:
$ git checkout 7ac6e83a4253d28919938b057883bd8ce39dd630
fatal: reference is not a tree: 7ac6e83a4253d28919938b057883bd8ce39dd630

Could you make sure that there is a branch that points to that commit?

FWIW:

I can reproduce this on the repo locally (but have not even attempted to minimize so far).
I suspect this line to be the culprit (just a gut feeling):
https://github.com/scala-js/scala-js/blob/44014be9ef5904f3b19052ac5c9861b1020567c4/linker/shared/src/main/scala/org/scalajs/linker/frontend/modulesplitter/Tagger.scala#L106-L107

I can confirm locally that the following path fixes the problem (on reform-scalajs-linker-memory-issue):
modified   linker/shared/src/main/scala/org/scalajs/linker/frontend/modulesplitter/Tagger.scala
@@ -90,10 +90,19 @@ private class Tagger(infos: ModuleAnalyzer.DependencyInfo,
       .put(pathRoot, pathSteps)

     if (updated) {
+      val excluded = excludedClasses.contains(className)
       val classInfo = infos.classDependencies(className)
-      classInfo
-        .staticDependencies
-        .foreach(staticEdge(_, pathRoot, pathSteps))
+
+
+      if (excluded) {
+        classInfo
+          .staticDependencies
+          .foreach(staticExcludedEdge(_, pathRoot, pathSteps))
+      } else {
+        classInfo
+          .staticDependencies
+          .foreach(staticEdge(_, pathRoot, pathSteps))
+      }

       classInfo
         .dynamicDependencies
@@ -109,6 +118,14 @@ private class Tagger(infos: ModuleAnalyzer.DependencyInfo,
       tag(className, pathRoot, pathSteps)
   }

+  private def staticExcludedEdge(className: ClassName, pathRoot: ModuleID, pathSteps: List[ClassName]): Unit = {
+    if (excludedClasses.contains(className))
+      tag(className, pathRoot, pathSteps)
+    else
+      // Force a "dynamic edge" from the external module.
+      dynamicEdge(className, pathRoot, pathSteps)
+  }
+
   private def dynamicEdge(className: ClassName, pathRoot: ModuleID, pathSteps: List[ClassName]): Unit =
     tag(className, pathRoot, pathSteps :+ className)

What it does is preventing forced dynamic edges inside the small split parts of the module.
I suspect this is splitting too much. Naively, we could do fine and coarse grained analysis completely independently, but that can lead to circular module dependencies in the following case (app is fine, lib is coarse):
graph TD;

subgraph entrypoint
main
end

subgraph coarse
lib.A
lib.C
end

subgraph fine
app.B
end

main --> lib.A --> app.B --> lib.C


[NSE-823] Use SPARK_VERSION_SHORT instead os SPARK_VERSION to find SparkShims

What changes were proposed in this pull request?
We should use SPARK_VERSION_SHORT to find SparkShim, not the SPARK_ERSION. As in use environment, SPARK_VERSION may be changed, the SPARK_VERSION_SHORT won't change.
How was this patch tested?
Origin GA

@PHILO-HE

@PHILO-HE

This change is useful for supporting a spark snapshot version in the future. E.g., for 3.3.0-SNAPSHOT, the short version is 3.3.0.
The patch looks good to me.


when excute the plan, alert:Unable to execute plan. Details: Error: 404: Not Found

Thanks @zhongerxin - this appears to be a regression from a recent change, I'll have a fix shortly!

Thanks @zhongerxin - this appears to be a regression from a recent change, I'll have a fix shortly!
Thank you，I was wondering why there's no 'processPlan' in the backend interface.


🐛 [firebase_core] Execution failed for task ':firebase_core:compileDebugJavaWithJavac'

Execution failed for task ':firebase_core:compileDebugJavaWithJavac'
Describe the bug
failed to run flutter project after adding firebase dependancies to build.gradle files
Steps to reproduce
Steps to reproduce the behavior:

add firebase to project level build.gradle file like the following:
  buildscript {
     ext.kotlin_version = '1.6.10'
     repositories {
         google()
         mavenCentral()
         google()
         jcenter()
     }

     dependencies {
         classpath 'com.android.tools.build:gradle:7.2.1'
         classpath "org.jetbrains.kotlin:kotlin-gradle-plugin:$kotlin_version"
         classpath 'com.google.gms:google-services:4.3.13'
     }
 }
 allprojects {
     repositories {
         google()
         mavenCentral()
         jcenter()
     }
 }
 rootProject.buildDir = '../build'
 subprojects {
     project.buildDir = "${rootProject.buildDir}/${project.name}"
 }
 subprojects {
     project.evaluationDependsOn(':app')
 }

 task clean(type: Delete) {
     delete rootProject.buildDir
 }

add firebase to app level build.gradle file to the following:
 def localProperties = new Properties()
 def localPropertiesFile = rootProject.file('local.properties')
 if (localPropertiesFile.exists()) {
     localPropertiesFile.withReader('UTF-8') { reader ->
         localProperties.load(reader)
     }
 }

 def flutterRoot = localProperties.getProperty('flutter.sdk')
 if (flutterRoot == null) {
     throw new GradleException("Flutter SDK not found. Define location with flutter.sdk in the local.properties file.")
 }

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

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

 apply plugin: 'com.android.application'
 apply plugin: 'kotlin-android'
 apply from: "$flutterRoot/packages/flutter_tools/gradle/flutter.gradle"
 apply plugin: 'com.google.gms.google-services'

 android {
     compileSdkVersion flutter.compileSdkVersion
     ndkVersion flutter.ndkVersion

     compileOptions {
         sourceCompatibility JavaVersion.VERSION_1_8
         targetCompatibility JavaVersion.VERSION_1_8
     }

     kotlinOptions {
         jvmTarget = '1.8'
     }

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

     defaultConfig {
         applicationId "com.example.flutter_course_15_flash_chat"
         // You can update the following values to match your application needs.
         // For more information, see: https://docs.flutter.dev/deployment/android#reviewing-the-build-configuration.
         minSdkVersion 19
         targetSdkVersion flutter.targetSdkVersion
         versionCode flutterVersionCode.toInteger()
         versionName flutterVersionName
     }

     buildTypes {
         release {
             // TODO: Add your own signing config for the release build.
             // Signing with the debug keys for now, so `flutter run --release` works.
             signingConfig signingConfigs.debug
         }
     }
 }

 flutter {
     source '../..'
 }

 dependencies {
     implementation "org.jetbrains.kotlin:kotlin-stdlib-jdk7:$kotlin_version"
     implementation platform('com.google.firebase:firebase-bom:25.3.0')
     implementation 'com.google.firebase:firebase-analytics-ktx'
 }

I add firebase-auth to my project using:
flutter pub add firebase_auth

and this is my pubspec.yaml file:
name: flutter_course_15_flash_chat
    description: A new Flutter project.
    publish_to: 'none'
    version: 1.0.0+1

environment:
  sdk: ">=2.17.5 <3.0.5"

dependencies:
  flutter:
    sdk: flutter

  cupertino_icons: ^1.0.2
  animated_text_kit: ^4.2.2
  firebase_auth: ^3.4.2

dev_dependencies:
  flutter_test:
    sdk: flutter
  flutter_lints: ^2.0.0

flutter:
  uses-material-design: true
  assets:
    - images/

when I try to run my app I got this error:
Launching lib\main.dart on AOSP on IA Emulator in debug mode...
Running Gradle task 'assembleDebug'...

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':firebase_core:compileDebugJavaWithJavac'.
> Could not resolve all files for configuration ':firebase_core:debugCompileClasspath'.
   > Failed to transform firebase-common-20.1.1.jar (com.google.firebase:firebase-common:20.1.1) to match attributes {artifactType=android-classes-jar, org.gradle.category=library, org.gradle.libraryelements=jar, org.gradle.status=release, org.gradle.usage=java-api}.
      > Could not find firebase-common-20.1.1.jar (com.google.firebase:firebase-common:20.1.1).
        Searched in the following locations:
            https://dl.google.com/dl/android/maven2/com/google/firebase/firebase-common/20.1.1/firebase-common-20.1.1.jar
   > Failed to transform firebase-annotations-16.0.0.jar (com.google.firebase:firebase-annotations:16.0.0) to match attributes {artifactType=android-classes-jar, org.gradle.category=library, org.gradle.libraryelements=jar, org.gradle.status=release, org.gradle.usage=java-api}.
      > Could not find firebase-annotations-16.0.0.jar (com.google.firebase:firebase-annotations:16.0.0).
        Searched in the following locations:
            https://dl.google.com/dl/android/maven2/com/google/firebase/firebase-annotations/16.0.0/firebase-annotations-16.0.0.jar

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 17s
Exception: Gradle task assembleDebug failed with exit code 1

Expected behavior
running the app successfully
Sample project
this is my flutter repo containing this bug

Additional context
Add any other context about the problem here.

Flutter doctor
Run flutter doctor and paste the output below:
Click To Expand
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 3.0.5, on Microsoft Windows [Version 10.0.19044.1826], locale en-US)
Checking Android licenses is taking an unexpectedly long time...[√] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
[√] Chrome - develop for the web
[X] Visual Studio - develop for Windows
    X Visual Studio not installed; this is necessary for Windows development.
      Download at https://visualstudio.microsoft.com/downloads/.
      Please install the "Desktop development with C++" workload, including all of its default components
[√] Android Studio (version 2021.2)
[√] VS Code (version 1.69.2)
[√] Connected device (4 available)
[√] HTTP Host Availability

! Doctor found issues in 1 category.

Flutter dependencies
Run flutter pub deps -- --style=compact and paste the output below:
Click To Expand
Dart SDK 2.17.6
Flutter SDK 3.0.5
flutter_course_15_flash_chat 1.0.0+1

dependencies:
- animated_text_kit 4.2.2 [flutter characters]
- cupertino_icons 1.0.5
- firebase_auth 3.4.2 [firebase_auth_platform_interface firebase_auth_web firebase_core firebase_core_platform_interface flutter meta]
- flutter 0.0.0 [characters collection material_color_utilities meta vector_math sky_engine]

dev dependencies:
- flutter_lints 2.0.1 [lints]
- flutter_test 0.0.0 [flutter test_api path fake_async clock stack_trace vector_math async boolean_selector characters charcode collection matcher material_color_u
tilities meta source_span stream_channel string_scanner term_glyph]

transitive dependencies:
- async 2.8.2 [collection meta]
- boolean_selector 2.1.0 [source_span string_scanner]
- characters 1.2.0
- charcode 1.3.1
- clock 1.1.0
- collection 1.16.0
- fake_async 1.3.0 [clock collection]
- firebase_auth_platform_interface 6.3.2 [firebase_core flutter meta plugin_platform_interface]
- firebase_auth_web 4.0.0 [firebase_auth_platform_interface firebase_core firebase_core_web flutter flutter_web_plugins http_parser intl js meta]
- firebase_core 1.19.2 [firebase_core_platform_interface firebase_core_web flutter meta]
- firebase_core_platform_interface 4.4.3 [collection flutter flutter_test meta plugin_platform_interface]
- firebase_core_web 1.7.0 [firebase_core_platform_interface flutter flutter_web_plugins js meta]
- flutter_web_plugins 0.0.0 [flutter js characters collection material_color_utilities meta vector_math]
- http_parser 4.0.1 [collection source_span string_scanner typed_data]
- intl 0.17.0 [clock path]
- js 0.6.4
- lints 2.0.0
- matcher 0.12.11 [stack_trace]
- material_color_utilities 0.1.4
- meta 1.7.0
- path 1.8.1
- plugin_platform_interface 2.1.2 [meta]
- sky_engine 0.0.99
- source_span 1.8.2 [collection path term_glyph]
- stack_trace 1.10.0 [path]
- stream_channel 2.1.0 [async]
- string_scanner 1.1.0 [charcode source_span]
- term_glyph 1.2.0
- test_api 0.4.9 [async boolean_selector collection meta source_span stack_trace stream_channel string_scanner term_glyph matcher]
- typed_data 1.3.1 [collection]
- vector_math 2.1.2

@siba36
Can you try gradle cache clean and see if it helps ?

I solved this problem by deleting .gradle folder in my user directory and then I upgraded gradle to the latest version which required my to update my java to a version that is higher than version 8 which I was using and I picked the latest version which is 18 and then I updated some kotlin packages.
This is the app level build.gradle file that worked for me:
def localProperties = new Properties()
def localPropertiesFile = rootProject.file('local.properties')
if (localPropertiesFile.exists()) {
    localPropertiesFile.withReader('UTF-8') { reader ->
        localProperties.load(reader)
    }
}

def flutterRoot = localProperties.getProperty('flutter.sdk')
if (flutterRoot == null) {
    throw new GradleException("Flutter SDK not found. Define location with flutter.sdk in the local.properties file.")
}

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

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

apply plugin: 'com.android.application'
apply plugin: 'kotlin-android'
apply from: "$flutterRoot/packages/flutter_tools/gradle/flutter.gradle"
apply plugin: 'com.google.gms.google-services'

android {
    compileSdkVersion flutter.compileSdkVersion
    ndkVersion flutter.ndkVersion

    compileOptions {
        sourceCompatibility JavaVersion.VERSION_1_8
        targetCompatibility JavaVersion.VERSION_1_8
    }

    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 "com.example.flutter_course_15_flash_chat"
        // You can update the following values to match your application needs.
        // For more information, see: https://docs.flutter.dev/deployment/android#reviewing-the-build-configuration.
        minSdkVersion 19//flutter.minSdkVersion
        targetSdkVersion flutter.targetSdkVersion
        versionCode flutterVersionCode.toInteger()
        versionName flutterVersionName
    }

    buildTypes {
        release {
            // TODO: Add your own signing config for the release build.
            // Signing with the debug keys for now, so `flutter run --release` works.
            signingConfig signingConfigs.debug
        }
    }
}

flutter {
    source '../..'
}

dependencies {
    implementation "org.jetbrains.kotlin:kotlin-stdlib-jdk7:$kotlin_version"
    implementation platform('com.google.firebase:firebase-bom:30.3.1')
    implementation 'com.google.firebase:firebase-analytics-ktx'
}

and then I installed firebase_auth using:
 flutter pub add firebase_auth

which added this line to my dependancies in pubspec.yaml file:
firebase_auth: ^3.5.1


Fancy Zone with windows 10 size by size..

Description of the new feature / enhancement
auto assign all the app side by side just like windows 10 in each zones.   i
Scenario when this would be used?
media players/.. showing all videos at once
Supporting information
No response

can you provide more details?  I don't want to jump to conclusions for what you're talking about.  Doing a step by step would be helpful for intended output.
/needinfo


Allow Rate Limiting within WebSockets

This PR adds the capability to do rate limiting within websocket requests.
For example (see examples/main.py):
@app.websocket("/ws")
async def websocket_endpoint(websocket: WebSocket):
     await websocket.accept()
     ratelimit = WebSocketRateLimiter(times=1, seconds=5)
     while True:
         try:
             data = await websocket.receive_text()
             await ratelimit(websocket, context_key=data) # NB: context_key is optional
             await websocket.send_text(f"Hello, world")
         except WebSocketRateLimitException:
             await websocket.send_text(f"Hello again")

Context
I was attempting to rate limit graphql requests. As graphql requests can come over a request or over a websocket, the conventional method of dependencies doesn't work. But the majority of the things we need to rate limit are already there - we have FastAPI with a redis client. All we have to do is construct the redis cache key a little differently.
In order to support multiple rate strategies for a single route, currently in depends.py we iterate through the dependencies of the current route and store the index of the dependency which is then used in the rate limiting key. (See https://github.com/peterbraden/fastapi-limiter/commit/dd385d624c8d1000d9f3d2b335186cb6f638f1ab)
In https://github.com/peterbraden/fastapi-limiter/blob/4e6c6fa6d8339bad459d801814c61735b658ae25/fastapi_limiter/depends.py#L32 the code assumes that the route has a dependencies list.
In fact dependencies is optional in APIRoute and doesn't appear at all in APIWebSocketRoute.
In a websocket none of this makes sense, we likely want to ratelimit more often than the lifetime of the websocket connection, and additionally we don't have connection scoped dependencies.
Instead of trying to shoehorn this into the existing method, I've simply added a class WebSocketRateLimiter that derives from RateLimiter, and can be called directly within a websocket connection.

Thanks! Please update README

I added a section to the readme.
The CI check seems to be failing on the install of poetry which I think is unrelated to this PR. The tests pass locally for me in any case.
Thanks,

Thanks!


FixFix backend_active_tasks only scan one be

Proposed changes
Fix select * from backend_active_tasks but only return one random be info.

run buildall

run buildall

run buildall


[DCSFP - PZ69 - AH64D] Add functions to AH-64D preprog pz69

By community suggestion, change functionalities on the pre-prog AH64D preprog pz69

Remove of VHF volume increase / decrease to avoid confusion
ACT/STBY : Pilot EUFD Swap button
Large Upper Freq rotary knob : Pilot EUFD IDM Inc/Dec
Small Upper Freq rotary knob : (No use)
Large Lower Freq rotary knob : Pilot EUFD WCA Inc/Dec
Small Lower Freq rotary knob : Pilot EUFD RTS Inc/Dec

I'll release.


The cpu and memory fields should be numbers instead of text

In the aws_ecs_task table, I'd like to see cpu and memory as number columns. The results from the aws cli come back as strings, but they are numbers. For example:
$ aws ecs describe-tasks --cluster <redacted cluster name> --tasks <redacted task arn>

<snip>
            "cpu": "512",
            "createdAt":<PHONE_NUMBER>.393,
            "desiredStatus": "RUNNING",
            "enableExecuteCommand": false,
            "healthStatus": "UNKNOWN",
            "lastStatus": "RUNNING",
            "launchType": "FARGATE",
            "memory": "1024",
<snip>

@mildebrandt thanks for so quick check and sharing us with immediate input appreciate it. We are looking into this feasibility, will make appropriate actions.


Fix some links

Remove unused link in doc. Follow up d1d0837.
Update the reference link in a message. Follow up dc81356.

r? @ehuss
(rust_highfive has picked a reviewer for you, use r? to override)

@bors: r+
Thanks!

:pushpin: Commit f37b9bd77aef63f47153be74e3a31a02ae9da6a6 has been approved by alexcrichton

:hourglass: Testing commit f37b9bd77aef63f47153be74e3a31a02ae9da6a6 with merge 25eb5cd153e4e9879d643ce4c9a04cc5d6998547...

:sunny: Test successful - checks-azure
Approved by: alexcrichton
Pushing 25eb5cd153e4e9879d643ce4c9a04cc5d6998547 to master...


Mobile Navigation Page Unclickable

The navigation burger button [three horizontal lines] is  unclickable  / disabled on mobile.

What branch is this issue happening on? Also what browser are you using? the sandbox page works fine on my phone and my browsers


Fix a corner case issue when daemon panics

There is an extreme corner case where when the daemon
panics(for some othe reason) at the same time as a container is stopping
and cleaning up the sandbox and the sandbox may have been
left with an inconsistent state. This libnetwork vendoring
fixes that case.
Vendoring in libnetwork @ 5305ea570b85d61dd0fd261cd7e1680da1884678
Signed-off-by: Jana Radhakrishnan<EMAIL_ADDRESS>

traceback of the panic that this PR solves is mentioned here.
It will help the search results
WARN[0001] Failed detaching sandbox eac39d3aed8fa2f621873ef3f2753b3d7105b9a291f20789481177acc65ed5fb from endpoint 5348bb4689762e77b93f8eb68a026753c308ddbd6f5aa0a8803ecdd88fe522bd: failed to get network from store during leave: network 6416ecdb5f6621d36f0c431f42b680be6ccad2712aa3f5b0ee6c4ddb127ba63f not found

WARN[0001] Failed deleting endpoint 5348bb4689762e77b93f8eb68a026753c308ddbd6f5aa0a8803ecdd88fe522bd: failed to get network during Delete: network 6416ecdb5f6621d36f0c431f42b680be6ccad2712aa3f5b0ee6c4ddb127ba63f not found

WARN[0001] Failed detaching sandbox eac39d3aed8fa2f621873ef3f2753b3d7105b9a291f20789481177acc65ed5fb from endpoint 1c0916d84c5ef46c16664c9fbca54a5cb7bb65e3b915b33881f04fa64781fcbe: failed to get network from store during leave: network e947d811abb99124c31f252c3a1c9cdc877f7984a4f9258b0e94bc689a30d5bd not found

WARN[0001] Failed deleting endpoint 1c0916d84c5ef46c16664c9fbca54a5cb7bb65e3b915b33881f04fa64781fcbe: failed to get network during Delete: network e947d811abb99124c31f252c3a1c9cdc877f7984a4f9258b0e94bc689a30d5bd not found

ERRO[0001] failed to cleanup up stale network sandbox for container 630eff52ed4d554620494bb1e75771831022577a266345197aaa6e22afbb5863
DEBU[0001] Assigning addresses for endpoint t6cont's interface on network bridge
DEBU[0001] RequestAddress(LocalDefault/<IP_ADDRESS>/16, <nil>, map[])
WARN[0001] Failed detaching sandbox eac39d3aed8fa2f621873ef3f2753b3d7105b9a291f20789481177acc65ed5fb from endpoint 5348bb4689762e77b93f8eb68a026753c308ddbd6f5aa0a8803ecdd88fe522bd: failed to get network from store during leave: network 6416ecdb5f6621d36f0c431f42b680be6ccad2712aa3f5b0ee6c4ddb127ba63f not found

WARN[0001] Failed deleting endpoint 5348bb4689762e77b93f8eb68a026753c308ddbd6f5aa0a8803ecdd88fe522bd: failed to get network during Delete: network 6416ecdb5f6621d36f0c431f42b680be6ccad2712aa3f5b0ee6c4ddb127ba63f not found

WARN[0001] Failed detaching sandbox eac39d3aed8fa2f621873ef3f2753b3d7105b9a291f20789481177acc65ed5fb from endpoint 1c0916d84c5ef46c16664c9fbca54a5cb7bb65e3b915b33881f04fa64781fcbe: failed to get network from store during leave: network e947d811abb99124c31f252c3a1c9cdc877f7984a4f9258b0e94bc689a30d5bd not found

WARN[0001] Failed deleting endpoint 1c0916d84c5ef46c16664c9fbca54a5cb7bb65e3b915b33881f04fa64781fcbe: failed to get network during Delete: network e947d811abb99124c31f252c3a1c9cdc877f7984a4f9258b0e94bc689a30d5bd not found

WARN[0001] Failed detaching sandbox eac39d3aed8fa2f621873ef3f2753b3d7105b9a291f20789481177acc65ed5fb from endpoint 7eef40a13894d014b0196668bb57ab543f6bc5c7e851e85f8840de269645dbbf: container 630eff52ed4d554620494bb1e75771831022577a266345197aaa6e22afbb5863: endpoint create on GW Network failed: service endpoint with name gateway_630eff52ed4d already exists

DEBU[0001] Releasing addresses for endpoint t6cont's interface on network bridge
DEBU[0001] ReleaseAddress(LocalDefault/<IP_ADDRESS>/16, <IP_ADDRESS>)
ERRO[0001] Error deleting sandbox id eac39d3aed8fa2f621873ef3f2753b3d7105b9a291f20789481177acc65ed5fb for container 630eff52ed4d554620494bb1e75771831022577a266345197aaa6e22afbb5863: could not cleanup all the endpoints in container 630eff52ed4d554620494bb1e75771831022577a266345197aaa6e22afbb5863 / sandbox eac39d3aed8fa2f621873ef3f2753b3d7105b9a291f20789481177acc65ed5fb
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xb code=0x1 addr=0x30 pc=0x880281]

goroutine 64 [running]:
github.com/docker/libnetwork.(*sandbox).populateNetworkResources(0xc208590680, 0xc208600a20, 0x0, 0x0)
	/go/src/github.com/docker/docker/vendor/src/github.com/docker/libnetwork/sandbox.go:458 +0x141
github.com/docker/libnetwork.(*sandbox).SetKey(0xc208590680, 0xc20881f1e0, 0x12, 0x0, 0x0)
	/go/src/github.com/docker/docker/vendor/src/github.com/docker/libnetwork/sandbox.go:394 +0x364
github.com/docker/docker/daemon.(*Container).setNetworkNamespaceKey(0xc2086e0a00, 0x6775, 0x0, 0x0)
	/go/src/github.com/docker/docker/daemon/container_unix.go:1092 +0x374
github.com/docker/docker/daemon.func·029(0xc2087d8058, 0x6775, 0xc20884a8a0, 0x0, 0x0)
	/go/src/github.com/docker/docker/daemon/daemon.go:960 +0x46
github.com/docker/docker/daemon/execdriver/native.func·001(0x0, 0x0, 0xc2086b5740, 0x40, 0x6775, 0xc208732e00, 0x68, 0x0, 0x0)
	/go/src/github.com/docker/docker/daemon/execdriver/native/create.go:145 +0x100
github.com/opencontainers/runc/libcontainer/configs.FuncHook.Run(0xc2087a5fc0, 0x0, 0x0, 0xc2086b5740, 0x40, 0x6775, 0xc208732e00, 0x68, 0x0, 0x0)
	/go/src/github.com/docker/docker/vendor/src/github.com/opencontainers/runc/libcontainer/configs/config.go:215 +0x50
github.com/opencontainers/runc/libcontainer.(*initProcess).start(0xc2087d49b0, 0x0, 0x0)
	/go/src/github.com/docker/docker/vendor/src/github.com/opencontainers/runc/libcontainer/process_linux.go:213 +0x568
github.com/opencontainers/runc/libcontainer.(*linuxContainer).Start(0xc2087d6d80, 0xc2087aa680, 0x0, 0x0)
	/go/src/github.com/docker/docker/vendor/src/github.com/opencontainers/runc/libcontainer/container_linux.go:107 +0x1b9
github.com/docker/docker/daemon/execdriver/native.(*Driver).Run(0xc2086b38b0, 0xc2087d8000, 0xc2087ad770, 0xc2087c2320, 0x1, 0x1, 0xc208795760, 0x0, 0x0, 0x0, ...)
	/go/src/github.com/docker/docker/daemon/execdriver/native/driver.go:167 +0x6de
github.com/docker/docker/daemon.(*Daemon).run(0xc20805f040, 0xc2086e0a00, 0xc2087ad770, 0xc208795760, 0x0, 0x0, 0x0, 0x0)
	/go/src/github.com/docker/docker/daemon/daemon.go:962 +0x246
github.com/docker/docker/daemon.(*containerMonitor).Start(0xc2087ced20, 0x0, 0x0)
	/go/src/github.com/docker/docker/daemon/monitor.go:145 +0x484
github.com/docker/docker/daemon.*containerMonitor.Start·fm(0x0, 0x0)
	/go/src/github.com/docker/docker/daemon/container.go:769 +0x39
github.com/docker/docker/pkg/promise.func·001()
	/go/src/github.com/docker/docker/pkg/promise/promise.go:8 +0x2f
created by github.com/docker/docker/pkg/promise.Go
	/go/src/github.com/docker/docker/pkg/promise/promise.go:9 +0xfb

LGTM

LGTM


Added vector store

Closing this until we have a specific need.

Too many changed for one PR
No description of changes
New tools not integrated into the bot


ShowBuildNumberOnDesktop doesn't update build number after update/upgrade.

I tried disabling and re-enabling the option to no avail, is this build information being cached or pulled from a static registry key or something?

It's taken from HKLM\SOFTWARE\Microsoft\Windows NT\CurrentVersion, value BuildLab. And yes, even on clean installation of 1909 it still reads 18362.19h1_release.190318-1202. Feel free to report it to Microsoft via Feedback Hub. They don't care anyway. :)


More consistent maps

Reduces the amount of shader chunks needed for all maps;
Supports multiplying texture * color * vertexColor (needed for GLTF)
New property for each map: VertexColorChannel (e.g. diffuseVertexColorChannel)

Renamed (for all maps):
diffuseMapVertexColor -> diffuseVertexColor
diffuseMapVertChannel -> diffuseVertexColorChannel
diffuseMapTint -> diffuseTint
Also updated docs

This is a breaking change right?

With all the renaming it is. Before the renaming it wasn't.
... can add get/setters for old property names

OK, it's not longer breaking.

OK, all looks good to me. Merge?

Why not?
I tested various stuff with it, but you might try testing some other stuff too + make sure it does work with GLTF the way it should.

Checked against the glTF sample model test suite. Works fine! Merging...


codec() causes mypy internal error when type has a path

Using codec() of xju 2.0.3 with the code snippet:
import mypytest_import
c = codec(mypytest_import.C2)

where the mypytest_import file content is
from dataclasses import dataclass
@dataclass
class C2:
    i: int

results in

mypytest.py:4: error: INTERNAL ERROR -- Please try using mypy master on GitHub:
https://mypy.readthedocs.io/en/stable/common_issues.html#using-a-development-mypy-build
If this issue continues with mypy master, please report a bug at https://github.com/python/mypy/issues
version: 1.7.1
mypytest.py:4: : note: please use --show-traceback to print a traceback when reporting a bug

There is no issue when using the Codec class instead of the codec function.

xju 3.0.1 should fix this

... nah, still broken...


Moving Google Maps Elements

Is it possible to move the default Google maps elements within the map? Specifically, I am trying to move the "Google" logo from the bottom left corner and the user location button from the top right corner.
Thanks

That's odd, mine is showing on the right bottom side

Any progress on this?

Did you guys figure this out already?

I moved over to Mapbox.

We've been considering that. How did it take you to set up mapbox? Cheers!

Overall a little better than I thought, I think I was running into some weird problems with changing what the map markers looked like, but I believe that's resolved. It's a little better to customize, so I'd reccommned looking into it.

Thanks for the quick response. Overall, the designs are way better than Google Maps. I think we'll switch to that in one of the next releases of our app.

Yeah, I also think that there is a dependency on the Google apps being installed on the device running it, so users of custom ROMS may run into problems. Haven't looked into it much though.

That's a GoogleMaps SDK limitation. Wontfix :(


Tidy up dev/device form processing and docs

r+wc

https://github.com/mozilla/zamboni/commit/d823f37


Error with logging during Translation |

TRANSLATION ERROR

We trained the model using 5000 train steps (we changed the config file number of steps for train and validation and learning rate decay) since it was taking too long to train on more steps. Although our model trained successfully (with 5000 train steps), we got the following error on running the translate command (python translate.py --config translate.cfg):

FileNotFoundError: [Errno 2] No such file or directory: '/content/drive/My Drive/data-to-text-hierarchical-master/experiments/exp-3/translation-log.txt
Thus, we were wondering when/where this file gets created and how to avoid this error (does it get created after 30000 train steps? because we saw that you manually stopped training after 30000 steps). If you could let us know how to avoid this error, that would be great!
PREPROCESSING WITH OUR DATA
2. Further, we are trying to train the data on our own dataset, that we have created in exactly the same format as your data (but we have not used the make_dataset.py file to do so). We have at most 100 records per entity for the data input and output files (train, test and validation). But the preprocessing step is taking an extremely long time. So we changed to 24 records (just as you had) and successfully works. Is it the case that we have to have at most 24 records per entity for the preprocessing step to work? (I looked over the preprocess.py code and it does not seem like that we need to have a specific limit for the number of records in the input data).
PS: Thank you for getting back to us on our previous issue, we really appreciated the quick response

Hi,

It's actually hard to tell exactly without seeing the error, but my guess would be that the issue comes from an invalid folder rather than the actual filepath. Is /content/drive/My Drive/data-to-text-hierarchical-master/experiments/exp-3/ an existing folder? Did you use logging while training ? If  yes, then the issue does not come from the logging module, rather something about that specific path. If you cannot find out why, you can disable logging by setting log_file: "" in the config file. One thing is for sure: you can train for any number of steps and this won't affect logging or anything in the translation script (other than model quality).

There is no specific limit for the number of records in an entity. 100 can be ok, depending on how many entities you have. In rotowire, there are ~28 entities, for a total of 672 records. There are two constraints when using larger input objects: preprocessing will take longer (as you saw), but will work eventually; training might be impossible because trying to load model weights + input + intermediate representations on GPU will cause a memory error. This depends on the actual number or records (i.e. number of entities times number of records per entity) and your hardware.

Thank you for getting back to us on our previous issue, we really appreciated the quick response

No problem, with pleasure. Let me know if there's anything else, I'll let you close this issue if not!
Also, note that for things that are not issues, but rather discussions (e.g. your questions regarding your own data), I have opened a  discussion space for this repo. Feel free to post there as well, so that discussions are not "lost" when we close issues!

Thank you so much for the prompt response and all your guidance.
The problem is that even by setting log_file: "", we get the following error
Traceback (most recent call last):
File "translate.py", line 6, in
main()
File "/content/drive/My Drive/data-to-text-hierarchical-master/onmt/bin/translate.py", line 48, in main
translate(opt)
File "/content/drive/My Drive/data-to-text-hierarchical-master/onmt/bin/translate.py", line 18, in translate
translator = build_translator(opt, report_score=True)
File "/content/drive/My Drive/data-to-text-hierarchical-master/onmt/translate/translator.py", line 24, in build_translator
out_file = codecs.open(opt.output, 'w+', 'utf-8')
File "/usr/local/lib/python3.7/codecs.py", line 898, in open
file = builtins.open(filename, mode, buffering)
FileNotFoundError: [Errno 2] No such file or directory: 'experiments/exp-3/gens/test/predictions_30K.txt'
I think this is probably because of the following lines in the translate.cfg file:
output: experiments/exp-3/gens/test/predictions_30K.txt
model: experiments/exp-3/models/model_step_30000.pt
This seems to indicate that 'experiments/exp-3/gens/test/predictions_30K.txt' gets creates after 30000 steps but we only run for 1000 steps. So how do we fix this error if we want to only run for 1000 steps.
We tried changing the lines to the following:
output: experiments/exp-3/gens/test/predictions_1K.txt
model: experiments/exp-3/models/model_step_1000.pt
But got a similar error that claims that the file ' experiments/exp-3/gens/test/predictions_1K.txt' is not found.
I was wondering if you could let us know all the lines we might have to change in the translate.cfg file in order for our model to work with a lower number of train steps (such as 1000 or 5000). Thanks so much again (and will post to the discussion forum as well with issues about our dataset, if they come up)

Hi,
In this case, the issue is definitely from the folder, and is not code related.
When you try to open a file in writing mode (let's say folder1/folder2/filename.txt) but the folder doesn't exist, then python will throw a FileNotFoundError rather than saying clearly that it's a folder.
Can you check and be extra sure thaht experiments/exp-3 exists ?

Hi Clement,
Yes, completely understand this. It definitely doesn’t exist, I thought the
folder gets created somewhere down the line (due to the code). So what
should I do if the folder exp-3 doesn’t exist?
Thank you,
Warm regards,
Shyamoli
On Fri, Jul 30, 2021 at 4:44 PM Clément Rebuffel @.***>
wrote:

Hi,
In this case, the issue is definitely from the folder, and is not code
related.
When you try to open a file in writing mode (let's say
folder1/folder2/filename.txt) but the folder doesn't exist, then python
will throw a FileNotFoundError rather than saying clearly that it's a
folder.
Can you check and be extra sure thaht experiments/exp-3 exists ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/KaijuML/data-to-text-hierarchical/issues/24#issuecomment-890143870,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AU3R45VRDXIZVUB4YDD2DHDT2MFI7ANCNFSM5BGV6CSQ
.

--
CONFIDENTIALITY NOTICE: This E-Mail is intended only for the individual or
entity to which it is addressed and may contain information that is
confidential. If you have received this communication in error, do not
distribute. Please delete the original message and immediately notify the
sender.

Hi Shyamoli,
Ah ok, great! Now we can solve the issue!
Code will run fine once we create the experiment folder. I have written some instructions in the README. Basically, you can use the following command to create the experiment:
python create-experiment.py --name exp-3
(if there are no experiments folder, it will create it, and then create a specific exp-3 folder, with gens and models.
 Normally, I would suggest to run the translation script in the same experiment folder as the one you used for training.
Let me know,

Hi Clement,
I did create the experiment folder before preprocessing the data, but used
the following line of code:
python create-experiment.py --name exp-1
This was mentioned on the GitHub page. So I have an experiments folder but
no exp-3 sub-folder, which is where the issue is arising.
So should I now run the create-experiment line with experiment 3 (even
after the experiment folder already exists)?
Thank you,
Warm regards,
Shyamoli
On Fri, Jul 30, 2021 at 4:59 PM Clément Rebuffel @.***>
wrote:

Hi Shyamoli,
Ah ok, great! Now we can solve the issue!
Code will run fine once we create the experiment folder. I have written
some instructions
https://github.com/KaijuML/data-to-text-hierarchical#experiments in the
README. Basically, you can use the following command to create the
experiment:
python create-experiment.py --name exp-3
(if there are no experiments folder, it will create it, and then create a
specific exp-3 folder, with gens and models.
 Normally, I would suggest to run the translation
script in the same experiment folder as the one you used for training.
Let me know,
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/KaijuML/data-to-text-hierarchical/issues/24#issuecomment-890150470,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AU3R45S6OYHM3QELRP3TH2TT2MHCLANCNFSM5BGV6CSQ
.

--
CONFIDENTIALITY NOTICE: This E-Mail is intended only for the individual or
entity to which it is addressed and may contain information that is
confidential. If you have received this communication in error, do not
distribute. Please delete the original message and immediately notify the
sender.

This is my mistake, I didn't realise the translate.cfg file used exp-3. I have modified it to avoid this issue in the future.
What you can do: replace all exp-3 by exp-1 in your file, and everything should be ok. You do not need to create a second experiment folder for translation. You should use the same folder you used for training.

Hi Clement,
Oh great! Thank you so much, that is extremely helpful. Will do that and
test.
Best wishes,
Shyamoli
On Fri, Jul 30, 2021 at 5:10 PM Clément Rebuffel @.***>
wrote:

This is my mistake, I didn't realise the translate.cfg file used exp-3. I
have modified it to avoid this issue in the future.
What you can do: replace all exp-3 by exp-1 in your file, and everything
should be ok. You do not need to create a second experiment folder for
translation. You should use the same folder you used for training.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/KaijuML/data-to-text-hierarchical/issues/24#issuecomment-890155571,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AU3R45THCQ66MTYPG5CIZNDT2MINJANCNFSM5BGV6CSQ
.

--
CONFIDENTIALITY NOTICE: This E-Mail is intended only for the individual or
entity to which it is addressed and may contain information that is
confidential. If you have received this communication in error, do not
distribute. Please delete the original message and immediately notify the
sender.

Dear Clement,
Sorry to bother you but would it be possible to get on a call sometime soon about some other errors we are getting while trying to test the model on our data? This would be extremely helpful to us. If this is inconvenient for you, no worries.
Thank you,
Warm regards,
Shyamoli

Sure, send me an email!
I'm closing this issue, we'll reopen if needed.

Dear Clement,
Great, thank you so much, we would love to meet whenever you are available.
Does tomorrow anytime after 1pm Eastern Standard Time work for you?
Warm regards,
Shyamoli
On Tue, Aug 3, 2021 at 4:10 PM Clément Rebuffel @.***>
wrote:

Sure, send me an email!
I'm closing this issue, we'll reopen if needed.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/KaijuML/data-to-text-hierarchical/issues/24#issuecomment-892132072,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AU3R45UXXDGQXVEX65F7WXLT3BEJ5ANCNFSM5BGV6CSQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

--
CONFIDENTIALITY NOTICE: This E-Mail is intended only for the individual or
entity to which it is addressed and may contain information that is
confidential. If you have received this communication in error, do not
distribute. Please delete the original message and immediately notify the
sender.


Why is CPM the default normalization?

https://github.com/openproblems-bio/openproblems/blob/3d8964a6c02496c0c604f0b1ddadc40589ca43a8/openproblems/tools/normalize.py#L44-L49
It's much more standard to use CP10k, or counts per median lib size. CPM might distort 0s vs non0s heavily.

Agreed that 10k would be better. I recall a recent paper (from Lior?) arguing something like this as well. Either way, I'd prefer scran, but that's slow... Do you want to open a PR?

I think this happened because for some reason it was included in the dimensionality reduction task and then transferred to everything when we made these generic functions. Always seemed weird to me as well so happy to have it replaced with something more standard.

I'm fine with this change. Worth noting that the PR would have to change many text references to CPM in method names, function names and the like.


Brianem/1.19 samples update

Purpose

...

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

Hi Brian,
I am new to Github.  I have been using Cognitive Services for translation and TTS for a year now.  I recently started playing with transcription.  The files I am trying to transcribe have 1 - 2 second pauses between sentences and the recognizer engine stops and returns the first sentence and never gets to the rest of the contents of the wav file.  Do you know a way around this?


Can't find libAirMaps.a in Build phases

I am following  this tutorial to integrate google maps in react native link: https://codeburst.io/react-native-google-map-with-react-native-maps-572e3d3eee14
After dragging AirGoogleMaps folder into project. I am expecting

But in my project libAirMaps.h is missing:

Environment:
react-native-cli: 2.0.1
react-native: 0.55.4
Target Platform: Xcode Version 9.3.1 (9E501) / I Phone 6 simulator

@himaanshi Did you find a solution?

Use Xcode to open project
Find Libraries in Project Navigator and click right to choose Add Files to "[Your Project]" ...
Find AirMaps.xcodeproj in node_modules/react-native-maps/lib/ios and add it
Find Link Binary With Libraries in Build Phases and add libAirMaps.a

Or you can see this: https://medium.com/@juanli_dev/install-react-native-maps-bccae45b6083

Very old issue, closing. Please re-open if you still have it.


[doc] Add a note about the version of checkstyle

Proposed changes
Issue Number: close #9784
Problem Summary:
Please refer to #9784
Checklist(Required)

Does it affect the original behavior: No
Has unit tests been added: No Need
Has document been added or modified: Yes
Does it need to update dependencies: No
Are there any changes that cannot be rolled back: No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

CC @morrySnow


Contribution question

Hi,
I'm Balaji from Germany
I went through the project and would love to contribute to new features and issues towards the project. How could I start?  Could someone please help me with a walkthrough?
Ciao,
Balaji Vengatesh Murugesan

__

acc

Hi @balajimurugesan2016,
You can check out this doc for the first steps in setting up the project in a dev environment. Have in mid that the code base was very recently ported to java 11 and chances are that this was not reflected there.

In general, you need maven, an IDE ( both InteliJ and Eclipse will do ), and a CF environment with a postgresql service to run the backend.
All repos have a decent unit test coverage. After merge or on PR we can run an integration test suite on our own infrastructure ( the integration tests setup & code are too coupled with the infrastructure to OSS and to exect anyone to set up on their own ). Load & Stress tests are again internally executed before release on the main dev branch (usually bi-weekly).
The legal aspects for contribution are covered by the CF foundation's contributor licence agreement. Upon PR, a CLA bot will pitch in and guide through the process.

If you have questions, you can find me and a few developers from the team in this slack channel.
Do you have any particular features in mind? Could you share details about the stories? We may have similar, envisioned in the team backlog.

Hi Dido ,
As of now, I don't have any feature requests. But I'm an avid developer/user for cloud foundry services. We have a lot of requests coming up in the future. So naturally, I'm more interested to know the inner workings and contribute towards the project.
Best Regards,
Balaji Vengatesh Murugesan


[Feature] Vela cli for printing generating manifests

Is your feature request related to a problem? Please describe.

For use with existing CI/CD solutions (e.g., ArgoCD), Vela CLI requires the ability to return the configuration manifest to stdout or file.
Because vela up applies built manifests directly to the cluster, it is difficult to use with argocd or kubectl apply -f - or something else.
Describe the solution you'd like

vela export or vela build or vela show or vela up --dry-run?
I'm not sure exactly which subcommand name is appropriate. :)
Describe alternatives you've considered

Additional context

Thanks for describing the feature. Could you please help give an example to help us better understand it?
BTW, do you think it's better to vela directly comparing to argocd or kubectl? Thanks.

This sounds a good enhancement. In general, vela cli already emits deploy.yaml during up lifecycle. What we should do is probably output the manifests available into stdout.
I think vela export --format yaml/json would be the best choice. The other three doesn't really mean printing the config manifest.

@anarcher
I have generally make vela export work as an Argo plugin with only one yaml generated. But with multiple yamls (AppConfig, Component, Scopes) it would fail. See more details in issue https://github.com/argoproj/argo-cd/issues/4958. Please help with that.


.travis.yml: add 1.8.x builds to travis

Codecov Report

Merging #691 into master will increase coverage by 0.16%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #691      +/-   ##
==========================================
+ Coverage   52.97%   53.13%   +0.16%
==========================================
  Files          42       40       -2
  Lines        3160     3045     -115
==========================================
- Hits         1674     1618      -56
+ Misses       1342     1249      -93
- Partials      144      178      +34

Impacted Files
Coverage Δ

build.go
75% <ø> (-9.91%)
:x:

cmd/gb/build.go

internal/depfile/depfile.go

depfile.go
32.06% <ø> (+24.06%)
:white_check_mark:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 137520c...138daef. Read the comment docs.

Test flake caused by gopkg.in. Updates #690


Add Scroll Pre-Alpha

Partially solves #88

[x] Scroll Pre-Alpha

@rmeissner anything still missing for the pre-alpha one? we might wanna split these into separate PRs if that speeds things up

Thanks for merging, @rmeissner ! Does your team wish to deploy the contracts to our network 534354 or shall we do it ourselves (using the singleton now)?

Feel free to do it yourself. We should probably push a new version of the singleton factory (or you patch it, both are fine).

Feel free to do it yourself. We should probably push a new version of the singleton factory (or you patch it, both are fine).

Sure, happy to re-deploy myself. If you could push the singleton factory to npmjs this week, that'd be appreciated - or lmk if not and I'll patch.

Sorry for the delay
<EMAIL_ADDRESS>


Duplicate word correction

I loved this tutorial, but found a typo error. Also, I am students still trying to wrap my head around git, so, not sure if this is how best to do this. Feel free to reject this commit, as it appears prettier jacked your white space up.
There was a "the the" on about line 613.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Stephen Bondor seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Corner case where commits have been made to local branch

I hit a corner case where I already comitted the changes to my branch. And would like to post the change as a patch. Something like:
git show > /path/to/diff.patch
A blank review was created due to absense of diff.
-------------------------------
Error Log:
-------------------------------
yash@yash incubator-lens (lens-551) $python ../rbt-jira/rbt-jira.py post-review
/usr/local/lib/python2.7/dist-packages/requests/packages/urllib3/util/ssl_.py:90: InsecurePlatformWarning: A true SSLContext object is not available. This prevents urllib3 from configuring SSL appropriately and may cause certain SSL connections to fail. For more information, see https://urllib3.readthedocs.org/en/latest/security.html#insecureplatformwarning.
  InsecurePlatformWarning
/usr/local/lib/python2.7/dist-packages/requests/packages/urllib3/util/ssl_.py:90: InsecurePlatformWarning: A true SSLContext object is not available. This prevents urllib3 from configuring SSL appropriately and may cause certain SSL connections to fail. For more information, see https://urllib3.readthedocs.org/en/latest/security.html#insecureplatformwarning.
  InsecurePlatformWarning
Fetching origin
Already up-to-date.

This message is displayed because 'merge.tool' is not configured.
See 'git mergetool --tool-help' or 'git help config' for more details.
'git mergetool' will now attempt to use one of the following tools:
meld opendiff kdiff3 tkdiff xxdiff tortoisemerge gvimdiff diffuse diffmerge ecmerge p4merge araxis bc3 codecompare emerge vimdiff
No files need merging
On branch lens-551
nothing to commit, working directory clean

You should have apache's git added as a remote(by the name, say apache). Then you can do rbt-jira.py post-review -b apache/master. And you'll be able to generate diff against apache/master, instead of your own master.

And please configure a merge tool for your git. It might be useful in the future.

@yssharma can we close this?

Thanks @prongs


How to add xmake.nvim?

Version confirmation

[X] Confirm

Following prerequisites

[X] Confirm

Neovim version
NVIM v0.9.1
Operating system/version
macos 14
Terminal name/version
iterm 2
$TERM environment variable
No response
Branch info
main (Default/Latest)
Fetch Preferences
SSH (use_ssh = true)
How to reproduce the issue
I filled in the following information in configs/custom/xmake.lua
return function() -- This file MUST return a function accepting no parameter and has no return value
	vim.g.compile_commands_dir = require(".vscode")
end

and filled in the following information in `plugins/custom.lua'
custom["Mythos-404/xmake.nvim"] = {
	lazy = true,
	event = "BufReadPost xmake.lua",
	config = true,
	dependencies = { "MunifTanjim/nui.nvim", "nvim-lua/plenary.nvim" },
}

when i type :w xmake doesn't work, and it displayed Error code 255

Expected behavior
no error code, xmake.lua can work
Actual behavior
No response
Additional information
No response

You set config = true so no need to add an xmake.lua as its config file. Remove the config/custom/xmake.lua.
Switch to your work directory and nvim the real xmake.lua used to config how to build the project.

I have removed xmake.lua, but still got the error

did you install the xmake itself?

of course, The above file was generated with$ xmake create -l c -P ./hello

It works after adding these lines to lua/modules/plugins/custom.lua:
local custom = {}

custom["Mythos-404/xmake.nvim"] = {
	lazy = true,
	event = "BufReadPost xmake.lua",
	config = true,
	dependencies = { "MunifTanjim/nui.nvim", "nvim-lua/plenary.nvim" },
}

return custom

I found that the problem seems to be that the execution directory of neovim does not match the directory I opened when I use nvim xmake.lua
I asked ChatGPT and it replied as follows, where should I add this code


Does go-carbon have any notion of stalling?

We have this situation where relay's metricsQueued climbs to almost a million. And it looks like go-carbon is not even pressured at all.
Does go-carbon have a meter that tells carbon-c-relay to back off after a certain size?
We think that our input-buffer is too conservative (51200), but looking at the code, it looks like input-buffer is used to create channel capacity, so we are not sure if increasing it to 1 million would help. Are we on the right path?

Answering my own question,input-buffer definitely act as back-off throttler.
The smaller the number, the less carbon-relay is buffering. So, increasing it to 1 million actually did the opposite effect from what we desire.


Create plugin-github-pr-comment-build

(This PR template only applies to permission changes -- ignore for changes to the tool)
Description
Add permissions for new github-pr-comment-build plugin.
Git repo: https://github.com/jenkinsci/github-pr-comment-build-plugin
Hosting issue: https://issues.jenkins-ci.org/browse/HOSTING-248
Permissions pull request checklist
Always

[x] Add link to plugin/component Git repository in description above

For a newly hosted plugin only

[x] Add link to resolved HOSTING issue in description above

For a new permissions file only

[x] Make sure the file is created in permissions/ directory
[x] artifactId (pom.xml) is used for name (permissions YAML file).
[x] groupId / artifactId (pom.xml) are correctly represented in path (permissions YAML file)
[x] Check that the file is named plugin-${artifactId}.yml for plugins

When adding new uploaders (this includes newly created permissions files)

[n/a] Make sure to @mention an existing maintainer to confirm the permissions request, if applicable
[x] Use the Jenkins community (LDAP) account name in the YAML file, not the GitHub account name
[x] All newly added users have logged in to Artifactory at least once

That's what I get for trying to do it in the github UI.  Sorry for the trouble, the filename is now fixed.


Timestamp parse error makes entire id3 tag unreadable

Reproducer
use std::path::Path;

use lofty::config::{ParseOptions, ParsingMode};
use lofty::file::TaggedFileExt;
use lofty::probe::Probe;
use lofty::tag::Accessor;

fn main() {
    // the error seems to occur with strict or relaxed parsing mode
    let parsing_options = ParseOptions::new().parsing_mode(ParsingMode::Relaxed);
    let path = Path::new("sample.mp3");
    let file = Probe::open(&path)
        .expect("can't probe")
        .options(parsing_options)
        .read()
        .expect("can't read");
    let tag = file.primary_tag().expect("no tag");
    println!("{:?}", tag.artist());
}

Summary
Trying to parse the attached file gives this error:
can't read: BadTimestamp("Expected a separator")

Expected behavior
If the timestamp is invalid could it be skipped in relaxed parsing mode?
ffprobe output:
Input #0, mp3, from '/Users/miles/tmp/sample.mp3':
  Metadata:
    major_brand     : dash
    minor_version   : 0
    compatible_brands: iso6mp41
    creation_time   : 2015-07-08 16:53:04
    encoder         : Lavf54.20.4
    title           : Freedom
    artist          : Pharrell Williams
    album           : Freedom - Single
    title-sort      : Freedom
    album-sort      : Freedom - Single
    artist-sort     : Pharrell Williams
  Duration: 00:02:44.49, start: 0.000000, bitrate: 130 kb/s
  Stream #0:0: Audio: mp3 (mp3float), 44100 Hz, stereo, fltp, 128 kb/s
  Stream #0:1: Video: mjpeg (Baseline), yuvj444p(pc, bt470bg/unknown/unknown), 600x600 [SAR 72:72 DAR 1:1], 90k tbr, 90k tbn (attached pic)
      Metadata:
        comment         : Other

Assets
sample.mp3.zip

If the timestamp is invalid could it be skipped in relaxed parsing mode?

That was what I originally intended to happen, and completely forgot to implement it. :smile:

2015-07-08 16:53:04

I've never seen a timestamp like this before, though (should be "2015-07-08T16:53:04"). Do you know where the file came from? The parser can already handle multiple common spec violations, and this may be one that should be supported if it comes from some popular software/platform.
Besides that though, yeah I do need to fix errors for BestAttempt/Relaxed parsing.

I’ll try to find out the origin. A user sent it to me.

Unfortunately I don’t know the origin of this file. I’m not sure if it’s likely to be a common occurrence.

In that case, I don't know if I should include an exception for that mistake. I'll have it just return what it can parse. So 2015-07-08 16:53:04 will just return a timestamp with year, month, day and discard the rest (when not using ParsingMode::Strict).

That sounds like a perfectly reasonable best effort result. Thanks for taking a look at it!

i also just ran into this (BadTimestamp("Expected a separator")), using the default BestAttempt mode and also trying Relaxed, the value in question (to my knowledge, the only one in the file ffmpeg shows):
TYER            : 20180923
this value comes from a old version of yt-dl (like 2021 or before)
(mp3 file, only concerning the id3v2 frame, not id3v1)
the git version (as of 4a2bcf5d0f93e16c210c9515e043dab3bd590f45) does not error out, but also does not parse the timestamp

I guess I could ignore it for BestAttempt/Relaxed, never seen this before.

on newer versions of yt-dl(p) it is replaced with the TDRC tag, a reproduction can be done with:
# yt-dlp version 2024.08.06
yt-dlp -x --audio-format mp3 --embed-metadata -o "out.mp3" https://www.youtube.com/watch\?v\=XCZcbUcaxoM

ffprobe out.mp3

hexdump -C out.mp3 | less

00000040  20 55 6e 69 74 79 00 54  44 52 43 00 00 00 0a 00  | Unity.TDRC.....|
00000050  00 03 32 30 31 38 30 37  32 36 00 54 58 58 58 00  |..20180726.TXXX.|

(PS: one of my uploaded videos)
this also currently triggers the lofty error.
i dont actually care that much about that tag, only really came here because of the whole probe failing, but wanted to provide more information.

20180726

Dates without separators should be supported since I fixed #452 (not published yet).
I haven't had much time to work on Lofty lately, I'd like to get #218 in before getting 0.22.0 released, since it'll probably be my last major update for a bit.
Related to that, if you could help test that PR at all it'd be much appreciated. See https://github.com/Serial-ATA/lofty-rs/pull/218#issuecomment-2413873070

Dates without separators should be supported since I fixed https://github.com/Serial-ATA/lofty-rs/issues/452 (not published yet).

sorry, didnt re-test with the git version; can confirm that the TDRC is correctly parsed there, though the TYER is still not present (i guess to be expected as its invalid - at least in the id3v2 section).


Broadcasting breaks scripts and tests

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.0.2 (dc4b81f 2024-05-21T15:26:16.686968000Z)
What command(s) is the bug in?
forge test
Operating System
None
Describe the bug
If you run vm.startBroadcast(); in a script, forge tests will break.
Example

Setup a new project

forge init

Change the following files:

Counter.s.sol
// SPDX-License-Identifier: UNLICENSED
pragma solidity ^0.8.13;

import {Script, console} from "forge-std/Script.sol";
import {Counter} from "src/Counter.sol";

contract CounterScript is Script {
    function setUp() public {}

    function run() public returns (Counter) {
        vm.startBroadcast();
        Counter counter = new Counter();
        vm.stopBroadcast();
        return counter;
    }
}

Counter.t.sol
// SPDX-License-Identifier: UNLICENSED
pragma solidity ^0.8.13;

import {Test, console} from "forge-std/Test.sol";
import {Counter} from "../src/Counter.sol";
import {CounterScript} from "script/Counter.s.sol";

contract CounterTest is Test {
    Counter public counter;
    CounterScript public script;

    function setUp() public {
        script = new CounterScript();
        counter = script.run();
    }

    function test_Increment() public {
        counter.increment();
        assertEq(counter.number(), 1);
    }
}

Run a test

forge test --mt test_Increment --zksync -vvvv

You'll get an EvmError: Revert during the setup. You'll notice if you do this with vanilla foundry after running foundryup it does not fail.
foundryup
forge test --mt test_Increment -vvvv

Hey @PatrickAlphaC, the team has discussed the issue at hand.
Your issue fundamentally clashes against how we approach handling EraVM and EVM together during test execution.
In particular, your script's run method would be executed inside the EraVM, where cheatcodes are not enabled, which leads to the failure you described, as the vm address isn't handled, nor any of the cheatcodes.
I understand your intent in avoiding code duplication, but you'll need to restructure it as adding support for this functionality would mean almost redoing our EraVM integration in its entirety.
TL;DR
Running cheatcodes inside EraVM (any contract call inside the test) is not supported.
PS: the broadcast-family of cheatcodes is rather useless in tests as they don't accomplish anything in that context.

Why would this change "almost redoing our EraVM integration in its entirety"?
Why not just have a flag that says "if local tests, ignore that cheatcode"?
It seems developers (or perhaps, me at least) are running into more and more issues where the devex on zkSync is quite different than other EVM chains. This should be avoided if we would like people to deploy to zkSync. Otherwise, you're asking people to have separate environments and test suites for zkSync, which seems to defeat the purpose of having solidity compatibility.


Cannot add laravel-dev-tools

Unable to add laravel-dev-tools. im using lazy.nvim
local lsp_configurations = require("lspconfig.configs")

if not lsp_configurations.blade then
	lsp_configurations.blade = {
		default_config = {
			name = "blade",
			cmd = { "blade" },
			filetypes = { "blade" },
			root_dir = require("lspconfig.util").root_pattern("~/.local/share/nvim/laravel-dev-tools"),
		},
	}
end
require("lspconfig").my_new_lsp.setup({})

When i reopen my init.lua i get:

You have to replace my_new_lsp with blade in the setup function.
Replace this
require("lspconfig").my_new_lsp.setup({})

With this
require("lspconfig").blade.setup({})

root_dir is not the path to the executable. The root_pattern function expects a list of files lspconfig is going to look for. You probably want to use something like composer.json or some other laravel specific file.
root_dir = require("lspconfig.util").root_pattern("composer.json"),

And according to the link you have there, the command to start the language server is this
cmd = { "laravel-dev-generators", "lsp" },

i genuinely apologize for wasting your time. i didn't know what i was doing after hours frustrating tries. im really sorry. i noticed that youve updated the doc, i appreciate it too. thanks and sorry.


NotFound()werkzeug.exceptions.NotFound: 404 Not Found: The requested URL was not found on the server. If you entered the URL manually please check your spelling and try again.

i use "pip install --no-cache -r requirements/local.txt", and write a run.py to start on localhost.
run.py as follows：
import imp
import os
from superset import create_app
os.environ["superset_dev"] = os.path.join(os.getcwd(), 'superset_dev/Scripts')
if __name__ == '__main__':
    superset_app = create_app()
    superset_app.run(host="localhost", port = "3000",debug=True )

when i run it， something is wrong, like this, what should i do? Please help me,thanks!

1516, in full_dispatch_request
rv = self.dispatch_request()
File "d:\xuelangyun\superset.venv\lib\site-packages\flask\app.py", line 1492, in dispatch_request
self.raise_routing_exception(req)
File "d:\xuelangyun\superset.venv\lib\site-packages\flask\app.py", line 1474, in raise_routing_exception
raise request.routing_exception  # type: ignore
File "d:\xuelangyun\superset.venv\lib\site-packages\flask\ctx.py", line 376, in match_request
result = self.url_adapter.match(return_rule=True)  # type: ignore
File "d:\xuelangyun\superset.venv\lib\site-packages\werkzeug\routing.py", line 2042, in match    raise NotFound()werkzeug.exceptions.NotFound: 404 Not Found: The requested URL was not found on the server. If you entered the URL manually please check your spelling and try again.

Closing this as stale since it's been silent for so long, and we're trying to steer toward a more actionable Issues backlog. If people are still encountering this in current versions (currently 3.x) please open a new Issue or a PR to address the problem.


Update java-function-invoker version to next release version v0.1.0

Expecting the next  java-function-invoker version v0.1.0 to be released soon

It's now v0.1.1 after re-enabling the docker builds


fix the command output

fix the command output

This change is

May I know what is being fixed by this patch?

@chenhuan12 Please rebase your branch so that it picks up 99fbc2b. That will fix the deploy/netlify failure. Thanks.

cc @zhangxiaoyu-zidif

@chenhuan12
Hi， could you tell us your kubernetes version

kubectl version
Client Version: version.Info{Major:"1", Minor:"6", GitVersion:"v1.6.9+a3d1dfa6f4335", GitCommit:"eb26ffbbf057d8e42362444ae2b91d74ac1febb7", GitTreeState:"dirty", BuildDate:"2017-09-12T20:36:51Z", OpenPaasKubernetesVersion:"v1.03.03", GoVersion:"go1.7.5", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"6", GitVersion:"v1.6.9+a3d1dfa6f4335", GitCommit:"eb26ffbbf057d8e42362444ae2b91d74ac1febb7", GitTreeState:"dirty", BuildDate:"2017-09-12T20:36:51Z", OpenPaasKubernetesVersion:"v1.03.03", GoVersion:"go1.7.5", Compiler:"gc", Platform:"linux/amd64"}

which version do you test it? @chenhuan12
Because you want this PR to merge into master branch, I recommand you to use recent verion. Thanks.

@zhangxiaoyu-zidif @markthink @chenhuan12 Please comment so we can either move forward or close this.
Same for #5516

I recommend to close it currently. If this output from the recent version, that would be reasonable, but if it is from a older version, that would be unnecessary. Same as #5516


[Feature] Customized plugin configuration to save build time and improve Developer experience

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
This feature can help developers and users of DevLake who primarily use local setup to enhance the experience and save build time. An ideal user has a single or very niche plugin use-case. Eg: Only want to use GitHub and not the rest or otherwise. A simple configuration file or an option to run a specific set of plugins that are useful would be a great add-on.
Use case
Suppose if a user only needs to use GitHub but Docker-compose builds for the entire files (Jira, Feishu, GitLab, etc. are not used all at once in most cases).
For low end devices (~ 8GB of RAM), it takes a lot of time for the docker to build. I had to wait for a few hours but still faced the challenge. In my scenario, I had to delete all the unused plugins from the plugin folder (except GitHub, gitextractor, dbt, core, helper) to make it smooth, quick, and easy.
A simple configuration file can be a good idea to reduce build time. If this or the solution already exists, it would be great to have it highlighted in our official docs.
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Adding further insights to this as clarified by @hezyin, It was my mistake and it won't be encountered in Manual mode.


Screenshare stops working

Hyprland Version
Hyprland, built from branch  at commit 1b48642fd15c433c53876f1b933dcd46265caf8f dirty (). Tag: v0.30.0
Bug or Regression?
Bug
Description
During my last boot screensharing stopped working on my browser (chrome).
After investigation I found that there is something wrong with the xdg-desktop-portal-hyprland service:
❯ systemctl --user status xdg-desktop-portal-hyprland
× xdg-desktop-portal-hyprland.service - Portal service (Hyprland implementation)
     Loaded: loaded (/usr/lib/systemd/user/xdg-desktop-portal-hyprland.service; static)
     Active: failed (Result: exit-code) since Tue 2023-10-24 16:42:57 CEST; 2s ago
    Process: 41970 ExecStart=/usr/lib/xdg-desktop-portal-hyprland (code=exited, status=127)
   Main PID: 41970 (code=exited, status=127)
        CPU: 3ms

Oct 24 16:42:57 arch systemd[1064]: xdg-desktop-portal-hyprland.service: Scheduled restart job, restart counter is at 5.
Oct 24 16:42:57 arch systemd[1064]: xdg-desktop-portal-hyprland.service: Start request repeated too quickly.
Oct 24 16:42:57 arch systemd[1064]: xdg-desktop-portal-hyprland.service: Failed with result 'exit-code'.
Oct 24 16:42:57 arch systemd[1064]: Failed to start Portal service (Hyprland implementation).

output of:
journalctl --user -xeu xdg-desktop-portal-hyprland.service

Oct 24 16:41:04 arch xdg-desktop-portal-hyprland[41655]: /usr/lib/xdg-desktop-portal-hyprland: error while loading shared libraries: libsdbus-c++.so.1: cannot open shared object file: No such file or directory
Oct 24 16:41:04 arch systemd[1064]: xdg-desktop-portal-hyprland.service: Main process exited, code=exited, status=127/n/a

How to reproduce
I'm not sure how to reproduce, it simply stopped working. I'm updated and running xdg-desktop-portal-hyprland-git arch AUR.
Crash reports, logs, images, videos
No response

downgrading from git version worked:
pacman -S xdg-desktop-portal-hyprland

systemctl --user stop xdg-desktop-portal-hyprland
systemctl --user start xdg-desktop-portal-hyprland


jsDelivr CDN

Better network & is official:
https://www.jsdelivr.com/projects/foundation
Should auto-update for 6.2.x

https://www.foundationcdn.com/


Out-of-dictionary results returned

I have locally confirmed getting findAll results that are not in the dictionary provided, using the Algorithm.TRANSPOSITION algorithm.
I think I could add an assertion for this case in TranspositionTransducerTest.java, where currently it seems that this condition is not being asserted against. Or is this already tested for in some other part of the test suite? Then try to reproduce on the existing test dictionary...
For now unable to get existing tests working though..

Seems to equally apply to the Algorithm.STANDARD.
Additional tidbit: in so far testing only a single out-of-dictionary find is included in each result of findAll.

What do you mean by, findAll results?  Also, please provide a test case that demonstrates the issue.

Sorry about that, it was the name of my Scala wrapper around tranduce. It will take a huge amount of creativity being able to demonstrate this without providing the proprietary lexicon... but it should be demonstrated in a reproducible way, I know. Lets put it on hold until I get to that.

Okay.  In the meantime, please test your wrapper against version 2.2.0, as the issue #6 was a pretty bad bug and may be the root cause of this.

On first examination, looks like the bug I reported here consistently reproduces with 2.1.2 but not with 2.2.0. I will check again tomorrow. Any idea how the fix for #6 may have solved this?

Yes, I do.  Depending on the terms provided, some of the non-final nodes were overwriting the final ones, and vice-versa.  The way the final nodes were stored missed a few edge cases.  The tiny list I was using for testing happened to not have such terms, which is how the bug made it to production.
While debugging #6, I encountered similar behavior to what this issue describes, which is why I recognized it as the likely root cause.


[tests] Various cleanups via Eclipse with extra hand holding

isolated updates to test packages only.

Coverage: 87.51%. Remained the same when pulling 9214c1d19f0aa823ac277cb9c79368a0b60f76c8 on hazendaz:copyright into 7c5df279a7404e310e6f792fdb0ec0c22f090693 on mybatis:master.


Many errors when running on ZFS

What kind of request is this (question/bug/enhancement/feature request):
bug
Steps to reproduce (least amount of steps as possible):
Launch Rancher standalone via docker /var/lib/rancher on a ZFS volume.
Result:
Rancher seems to start (and works) but logs a lot of stuff like:
E0226 17:04:06.233306      29 pod_workers.go:191] Error syncing pod 5c5045fc-c7db-4e68-bbfa-b7a17e536290 ("coredns-7944c66d8d-vbjqr_kube-system(5c5045fc-c7db-4e68-bbfa-b7a17e536290)"), skipping: failed to "CreatePodSandbox" for "coredns-7944c66d8d-vbjqr_kube-system(5c5045fc-c7db-4e68-bbfa-b7a17e536290)" with CreatePodSandboxError: "CreatePodSandbox for pod \"coredns-7944c66d8d-vbjqr_kube-system(5c5045fc-c7db-4e68-bbfa-b7a17e536290)\" failed: rpc error: code = Unknown desc = failed to create containerd task: failed to mount rootfs component &{overlay overlay [workdir=/var/lib/rancher/k3s/agent/containerd/io.containerd.snapshotter.v1.overlayfs/snapshots/7031790/work upperdir=/var/lib/rancher/k3s/agent/containerd/io.containerd.snapshotter.v1.overlayfs/snapshots/7031790/fs lowerdir=/var/lib/rancher/k3s/agent/containerd/io.containerd.snapshotter.v1.overlayfs/snapshots/1/fs]}: invalid argument: unknown"

On the "local" cluster, many "helm-operation" tasks are hanging and core-dns is not starting.
I assume that it's sonehow related to https://github.com/k3s-io/k3s/issues/66 but I have no deeper technical understanding of that issue.
 That seems to fix the issue and also my data seems to be there. Is it safe to have this on a different (non-backuped) volume?


Chrome Canvas.clip() is slow, more general and fast  workaround is required

Chrome browser demonstrates poor performance for Canvas.clip() operation. FF and Safari are faster but extensive usage of clip operation also has noticeable influence to rendering performance. The test case is grid component where every cell is clipped before its content rendering. Having ~ 10x50 visible cells can slow down rendering what especially important when a user scrolls up/down grid content.
Partially the problem has been solved by minimizing number of clip operations where it is possible, but more deeper investigation is required.  The questions are the following:
-- Can minimizing clip over clip (when we clip over prev. cllp) speed up rendering ?
-- Can offscreen buffer usage solve the problem ?
-- What is about applying global composition instead of clip ?

@barmalei Could you share some info on how this problem was fixed?


Provide wheels for Rosetta (macOS, non-avx)

While JAX provides native ARM support for M1 other libraries are not yet there and have to be run under Rosetta. x86 wheels don't work on M1 because Rosetta doesn't emulate AVX. Would it be possible to provide non-AVX x86 wheels for this use case?

We could. You can build jaxlib from source without AVX enabled easily enough following the usual instructions.
I'm a bit reluctant to do this because AVX is a CPU feature that was released 8 years ago, and at some point we just want to assume a minimum level of vectorization support. AVX, and fused multiply-add (FMA) which arrived at the same time (Intel Haswell) are pretty important for machine learning workloads.
Can you say more about which other packages you're blocked on? How long would we need to support the non-AVX wheels? I'm trying to gauge how popular this request is likely to be; if it's a niche thing, then I'd suggest just building your own wheels.

I fully agree that AVX and FMA should be assumed as a minimum. Unfortunately, we still have some old legacy code in tensorflow 1.x that will probably stay around for a year or so. There are also other packages like https://web.casadi.org and probably some others but tensorflow is the main blocker :(
Yes, I guess that makes sense 👍  If there is no further interest I'll build my own wheels.

I'm new to Jax, have a MacOS ARM and the initial setup wasn't that smooth.
pip install -U pip
pip install jax
pip install jaxlib

And then I get
>>> import jax
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/amir.fayazi/monorepo/.venv/lib/python3.8/site-packages/jax/__init__.py", line 35, in <module>
    from jax import config as _config_module
  File "/Users/amir.fayazi/monorepo/.venv/lib/python3.8/site-packages/jax/config.py", line 17, in <module>
    from jax._src.config import config  # noqa: F401
  File "/Users/amir.fayazi/monorepo/.venv/lib/python3.8/site-packages/jax/_src/config.py", line 28, in <module>
    from jax._src import lib
  File "/Users/amir.fayazi/monorepo/.venv/lib/python3.8/site-packages/jax/_src/lib/__init__.py", line 85, in <module>
    cpu_feature_guard.check_cpu_features()
RuntimeError: This version of jaxlib was built using AVX instructions, which your CPU and/or operating system do not support. You may be able work around this issue by building jaxlib from source.

I thought the support was there. Is the recommendation to build from source? I want to run it locally on the laptop for initial prototyping.


[Improvement][Readme] update DolphinScheduler slack invitation url

the slack invitation url has been expired, please update the url address in README.md and README_zh_CN.md
Join the slack channel by [this invitation link](https://join.slack.com/t/asf-dolphinscheduler/shared_invite/zt-mzqu52gi-rCggPkSHQ0DZYkwbTxO1Gw ).

please use this new slack invitation url: https://join.slack.com/t/asf-dolphinscheduler/shared_invite/zt-omtdhuio-_JISsxYhiVsltmC5h38yfw
this invitation url never expires

I will fix it.


Cross version upgrade

I am currently using version 1.09. If I upgrade directly to version 1.11, the following error will be reported
“Failed to upgrade Dapr: error validating "": error validating data: [ValidationError(Configuration.spec.mtls): unknown field "controlPlaneTrustDomain" in io.dapr.v1alpha1.Configuration.spec.mtls, ValidationError(Configuration.spec.mtls): unknown field "sentryAddress" in io.dapr.v1alpha1.Configuration.spec.mtls]”
But if I upgrade to 1.10 first and then upgrade to 1.11, it will be successful

I have see issue, upgrade 1.11 to 1.12.4

I have same issue, upgrade 1.11 to 1.12.4


GPT2 Support

Wanted to share support for the HF OpenAI GPT-2 models (gpt2, gpt2-medium, gpt2-large, and gpt2-xl) in case there is any interest for it to be merged. The existing gpt_bigcode support was utilized; however, model sanitization with the HF OpenAI GPT-2 models is different enough because of the unneeded attention "bias" layers & needing to transpose the Conv1D weights as they are stored along with some other gymnastics. There is potential to merge into 1 model implementation with gpt_bigcode, but IMO that would unnecessarily complicate both model implementations.

Could you point me to a model that should work? I tried python -m mlx_lm.generate --model openai-community/gpt2 --prompt "Write a story about Einstein" but got a bad response

Could you point me to a model that should work? I tried this model: https://huggingface.co/openai-community/gpt2 withpython -m mlx_lm.generate --model openai-community/gpt2 --prompt "Write a story about Einstein" but got a bad response

openai-community/gpt2-xl works much better. It is shocking how much smaller models have progressed when you compare to the gpt2 models.


Pass current parameters to display name function

This should be able to fix #64 by using the #71 solution (but give more infos as parameters are needed)

In fast already handle by the prettifyParam function nevermind ^^


parser, ast: add explain for ALTER TABLE stmts

What problem does this PR solve?
As part of change column type (but also other operations) it is difficult to know what happens when a DDL operation is performed. MySQL supports explain assertions, but these are a little limited.
In chatting to @bb7133 we discussed adding an EXPLAIN for DDL.  The TiDB issue is https://github.com/pingcap/tidb/issues/8429
This is an extension to MySQL syntax.
What is changed and how it works?
Alter table statements are now explainable.
Check List
Tests

Unit test

Code changes

Has exported variable/fields change

Side effects

Increased code complexity

Related changes

None

/lgtm

/lgtm

/merge


change processing model for bugfixing loop

There are couple of source conditions defined as UNHACK_LOOP. Right now the compiler treats the loop controllers ('[', ']', ':' and '/') as predefined operators. They are wrong. They must be defined as a mere macro for each.
Currently UNHACK_LOOP cannot be enabled because of the current processing model of MmlOperationUses.
When resolving macros into flatten stream of MIDI events, they are converted from within the list of MmlOperationUse which are either from the track MML or macro definitions.
When those loop controllers are being expanded as macro, the content is nothing and no content of the loop will be iterated.
To fix this issue, the MmlOperationUses must be stored first, and then they have to be expanded.
There is another source condition labeled as LOOP_BY_RESULT, which actually does somewhat similar to the desired model explained above. But it expands the loop contents as the result of first iteration of MML processing. It will break operators that depend on loop, e.g. spectra. So it is actually no-go.
There is another difficulty in the processing model switch.
A simple desired processing model would be that Loop class structure stores a list of source MmlOperationUses while processing them, and at the end of the loop it expands the stored ones just like the compiler does right now (but remember that current list does not work across macros i.e. macro ']' will break that).
It seems simple but doesn't work... When expanding macros we need to resolve variables and macro arguments (call it "context" here), but this context information is lost after "flattening" macros and those operators cannot be re-processed afterwards. To make it doable, we need to preserve the context changes (which I'm afraid that it's going to be too much to process).
Or somehow change the way how context list of MmlOperationUses are resolved. That might be easier.

Also, we cannot simply "#define" those operators, especially ':', because ':' is also used to define the type of a variable (e.g. "gate : length").

After a couple of attempts to fix this, I'm going to mark this as by design. There is almost no practical benefit to fix this, while the impact of the changes are big and they tend to be inefficient solution.


Network: Persisted Requests Queue - clear persisted requests after the response

Details

Additional functionality was added to Network.js and NetworkRequestQueue.js to better handle persisted requests

clear requests from persistent storage only after they are completed
retry persisted request

Added unit tests covering persisted request handling
Fixed Issues

$ https://github.com/Expensify/App/issues/5987
Tests

Launch the app
Go offline
Write a few messages
Close the app
Be online
Open the app
Observe the messages sent while being offline are correctly sent to the backend

QA Steps

Follow the steps from the bug report

Make sure you are offline.
Send some messages to any user.
Refresh the page on the web. Close & open the app on mobile.
Now you can still see the messages.
Now enable internet
Shortly (a second after resuming the connection) after refresh or close the app
The messages should be posted after the refresh or after you open the app again

note: the order of the messages might be incorrect, this PR is not addressing the order

This comment has additional details on steps and timing: https://github.com/Expensify/App/issues/5987#issuecomment-962959408
Tested On

[x] Web
[x] Mobile Web
[x] Desktop
[x] iOS
[x] Android

Screenshots

Web

https://user-images.githubusercontent.com/12156624/144221544-5308bdb4-902b-4f5f-acb1-9564ac637777.mp4
Mobile Web

https://user-images.githubusercontent.com/12156624/144224565-d756ee4b-d2d4-4c50-b35e-fb5b1ebce8c9.mp4
Desktop

https://user-images.githubusercontent.com/12156624/144230141-1c63ea16-429f-4544-b949-4cbd1a7c968f.mov
iOS

https://user-images.githubusercontent.com/12156624/144236936-5b52e27b-cb66-4cde-8272-17073c9fa35d.mov
Android

https://user-images.githubusercontent.com/12156624/144220867-0adf287f-c63d-4220-8a94-0a2fa5ea89c5.mp4

@roryabraham do you mind co-reviewing this with me?

This is ready for review
I've made some small changes to address Rory's concerns.

@johnmlee101 @roryabraham can you review this PR? @kidroca 's been waiting three weeks

Sorry, this accidentally got closed: https://github.com/Expensify/App/pull/4908#issuecomment-1013444052

@johnmlee101 @roryabraham
Can you post an update
What do I need to do to move this forward?

Oh, I see there are conflicts
I'll be able to merge and test again tomorrow

@johnmlee101 @roryabraham @mallenexpensify
Before I merge main, fix the conflicts and retest everything can someone at least confirm that's the only thing left to do and there are no requested changed

I think the changes right now are good, and I'll be ready to approve once the conflicts are resolved.

Yeah, I looked this over again and I don't think I have any further comments.

Synced with main and retested - everything working as intended

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by @roryabraham in version: 1.1.33-4 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by @sketchydroide in version: 1.1.34-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Installation by Bower.

Have you thought about the availability of the installation by Bower?

I was able to install it with bower by using:
bower install https://github.com/rewish/jquery-bgswitcher.git

Is that what you're looking for?


Request for Open Source Training Code

I would like to test your model on other datasets, so I would like to request you to open source the training code along with the corresponding documentation.Thank you!

this is on our roadmap -- will give an update here when it's in place!


max simplified

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

62 of 64 (96.88%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-1.3%) to 98.742%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

objwalker.go
62
64
96.88%

Totals

Change from base Build<PHONE_NUMBER>:
-1.3%

Covered Lines:
157

Relevant Lines:
159

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

68 of 68 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
163

Relevant Lines:
163

💛  - Coveralls


Allow containers/common to use internal TF

TODO:

[ ] Write new tests or update the old ones to cover new functionality.
[ ] Update doc-strings where appropriate.
[ ] Update or write new documentation in packit/packit.dev.
[ ] ‹fill in›

Fixes
Related to
Merge before/after

RELEASE NOTES BEGIN
Packit now supports automatic ordering of ☕ after all checks pass.
RELEASE NOTES END

Service has to be redeployed after the PR is merged, I'm going to do it ASAP.


Line weights in charts

#3
An "I know we've talked about this, but we may not have written it down anywhere" issue. Default line weights in charts? Macro uses 2.0 pt/px for web and print styles (also round caps and round joins, where style guide says square caps and miter joins).

One thing to keep in mind regarding line widths is accessibility: thicker lines are more visually accessible than thinner ones.


[DENTnet] Add in-app staking support for DENTX

Staking doc: https://docs.dentnet.io/dentnet-documentation/overview/core-concepts/proof-of-stake/staking
Dashboard: https://main.dentnet.io/dashboard/#/overview

5GngHp6nfGB7J4SLgXZENBrVLA6Dop4bVVjUUHdCp6dFMLGt


[DOCFIX] Add mkdir step to Alluxio building guide #9251

For a user new to Alluxio and follows the Building-Alluxio-From-Source guide for the first time, the cloned repository doesn't have directory ./underFSStorage. bin/alluxio validateEnv local will complain this directory doesn't exist and fail.
I think it makes sense to add this step in the guide.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4869/
Test FAILed.

Merged build finished. Test FAILed.

Jenkins, test this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4870/
Test PASSed.

alluxio-bot, merge this please

alluxio-bot, cherry-pick this to branch-2.0 please


help(Mat-Menu): Can I open nested menu on click event instead of hover

StackOverflow: https://stackoverflow.com/questions/tagged/angular-material2
stackblitz: https://stackblitz.com/edit/angular-sjy3em-ermrc7?file=app%2Fmenu-overview-example.ts
Google Groups: https://groups.google.com/g/angular-material2/c/Rr0AhgLoFuU/m/g0DczGA2AgAJ
What are you trying to do?
I am using mat-menu for mobile device to create hamburger menu.
I am trying to open nested mat-menu on click instead of hover
What troubleshooting steps have you tried?
created sample code in below link
https://stackblitz.com/edit/angular-sjy3em-ermrc7?file=app%2Fmenu-overview-example.ts
What are you seeing that does not match your expectations?
only main menu icon click is working and for sub levels only hover is working not click
Reproduction
https://stackblitz.com/edit/angular-sjy3em-ermrc7?file=app%2Fmenu-overview-example.ts
We can only help if we can reproduce the problem ourselves.
Environment

Angular:
CDK/Material:
Browser(s):
Operating System (e.g. Windows, macOS, Ubuntu):

Our menu component is the pop-up kind of menu shown in https://material.angular.io/components/menu/examples. For a navigation "menu" in a sidenav, you would want to use something like mat-list.


Add Glyphicon Globe with other side of Globe

It will be nice if there will be one more Globe glyphicon with more usual for Europian/Asia users side (Eurasia/Africa/Australia)

The Glyphicons are an external project that we just package/bundle.
Please direct icon suggestions to the Glyphicons' author. Thanks!
See also https://github.com/FortAwesome/Font-Awesome/issues/281

Actually, this is already part of the full for-pay Glyphicons (as glyphicons-globe-af), but Bootstrap only includes the "Halflings" set of Glyphicons.


Rename toIgnore to pathToIgnore to avoid naming conflict.

Thanks for the PR!
To accept your contribution we'll need you to sign our contributor license agreement here: https://cla.developers.google.com/clas

@rictic Already signed, I'm contributing on behalf of Google/Chromium.

Huh, I wonder why go/clasigners was unhappy. Looks like clabot is happy though. Thanks for the contribution!


Auto-increment next development iteration. (#1816)

Auto-increment next development iteration.

Signed-off-by: dblock<EMAIL_ADDRESS>
Make bwc increments on X.Y and main branches.

Signed-off-by: dblock<EMAIL_ADDRESS>Description
Manually backporting #1816 due to DCO failure
Issues Resolved
[List any issues this PR will resolve]
Check List

[X] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

I'll cleanup my own mess in https://github.com/opensearch-project/OpenSearch/pull/2164! Thanks for your help.


Fix: enable explicit keep-alives in the sidecar-to-sidecar gRPC connection

Sets explicit keep-alives in the gRPC server and client, to make sure that they are enabled.
Also sets some other keep-alive parameters more explicitly and documents some choices.

Updates the gRPC library as there are some fixes included that are related to keep-alives

Note for maintainers we're awaiting user feedback on this.

Closing this, it doesn't seem to be the right thing to do. Switching to #7329 instead


[Bug] Does not start anymore - ENOENT - Win10

Insomnia Version: From the error message I figure it's 7.0.5
Operating System: Windows 10

Worked fine yesterday. Now when I execute Insomnia a receive an exception:

Uncaught Exception:
Error: ENOENT: no such file or directory, open
'\?\C:\Users\me\AppData\Local\insomnia\app-7.0.5\resources\app...\package.json'
at Object.fs.openSync (fs.js:577:3)
...
at process.binding.internalModuleReadJSON (ELECTRON_ASAR.js:665:19)
...

Reinstalled Insomnia. Closed for me.

I get this too, and a reinstall did work. But something keeps breaking after a few uses of the app for the last few versions, and I don't want to have to keep reinstalling Insomnia each time.

@glyphCezilleSantillan shall I reopen it for you? Can you provide more details?

Please create a new issue with details if you are experiencing issues @glyphCezilleSantillan 🙂

Also experiencing this

Try opening it with admin rights. It worked for me.


MagPhase vocoder v2.0 released (April 2018)

New in Version 2.0 (April 2018):

Constant frame-rate support.
Improved sound quality.
Two types of post-filter available.
Selectable number of coefficients for phase features (real and imag).
Selectable number of coefficients for the magnitude feature (mag).

Nice.i see you updated the merlin SLT demo "s2" experiment. Perhaps I can free up a couple hours and try it.
Is there anything we should be aware of with the new version?

Hi, yes, the "s2" experiment was completely reformatted, now everything is centralised in one main script. Also, you can vary several parameters as wanted.
The function signatures have changed, be aware of that. Other than that, everything should work similarly.
In case you try this version, please, let me know if you have any suggestion or bug you may find.

I've noticed that it's really clean now, great.
Ran the full demo and the feature extraction is also extremely fast.
Also ran the world demo ("s1") again.
Here are the results, do they sound correct to you?
https://drive.google.com/drive/folders/13BhNi0vCA0ufhTeOfIoWggRZPQZGL4aG?usp=sharing
(the magphase sample is from the magphase ph folder)

Thanks! I perceive that it sounds OK, but a bit "wobbly". I think there is something going on with the F0. I am going to check that as soon as I have time. What's your opinion?

I realised that the full data demo was using a very simple architecture. So, I just changed it to match what is used in s1. he wobbly effect should be gone now.
Also, did you try with 48kHz data?

I have some results of magphase vocoder with lossless and lowdim demo.
https://drive.google.com/file/d/1LONGI3xb4hYV8fMqdS8CDDNCbdFxbtkY/view?usp=drivesdk
The  voice quality of lowdim still v
Very bad. I don't know the version which intergrated into merlin is good or bad with my language but it have same algorithm for compress acoustic feature with lowdim demo right?