added maintainer to new bitbucket-kubernetes-credentials-plugin

Description

This PR is for adding permissions so that I can release the new bitbucket-kubernetes-credentials-plugin. I am the sole maintainer (@robbert229 on github, johnrowl on jira)
Transfered Repository: https://github.com/jenkinsci/bitbucket-kubernetes-credentials-plugin
Original Repository: https://github.com/robbert229/bitbucket-kubernetes-credentials-plugin
Jira Hosting Issue: https://issues.jenkins-ci.org/browse/HOSTING-956
Submitter checklist for changing permissions

Always

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

https://github.com/jenkinsci/bitbucket-kubernetes-credentials-plugin
For a newly hosted plugin only

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

https://issues.jenkins-ci.org/browse/HOSTING-956
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)

[x] Make sure to @mention an existing maintainer to confirm the permissions request, if applicable

Not applicable since its a new plugin.

[x] Use the Jenkins community (LDAP) account name in the YAML file, not the GitHub account name

[x] Make sure to @mention the users being added so their GitHub account names are known if they require GitHub merge access (see below).

Not applicable since I am filing the request for myself for a new plugin.

[x] All newly added users have logged in to Artifactory at least once

Reviewer checklist (not for requesters!)

[ ] Check this if newly added person also needs to be given merge permission to the GitHub repo (please @ the people/person with their GitHub username in this issue as well). If needed, it can be done using an IRC Bot command
[ ] Check that the $pluginId Developers team has Admin permissions while granting the access.
[ ] In the case of plugin adoption, ensure that the Jenkins Jira default assignee is either removed or changed to the new maintainer.
[ ] If security contacts are changed (this includes add/remove), ping the security officer (currently @daniel-beck) in this pull request. If an email contact is changed, wait for approval from the security officer.

There are IRC Bot commands for it

Sorry, two more changes, please use the plugin- prefix instead of the component- prefix for the file. component is used for libraries and so forth.  Also, you can remove the -plugin suffix from the file and just have plugin-<artifaceId>.yml

@slide I think I grok this now. Also, if I understand correctly then I should use name: "bitbucket-kubernetes-credentials" instead of name: "bitbucket-kubernetes-credentials-plugin" since name is the artifactId

Good catch, yes!


Education

Is this available for Education? ie M365 A3/A5?

@slrattan The issue does not include the details or reference of the document to be modified. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Document Decal interactions with transparent materials

This behavior may not be obvious at first, so it's important to mention it in the class reference.

Thanks!


Broken link

On the english books line 252 the following link is broken:
http://www.flashbulbinteraction.com/WTS.html

Fixed in https://github.com/vhf/free-programming-books/issues/2234


Fix poorly formatted metadata

This was caused by a bad edit in https://github.com/apache/beam/commit/12f107249b7617d84c0552df4977de4f3801e557#, right now this notebook is invalid
✨ RENDERED ✨

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @rezarokni @AnandInguva


html5shiv should be added back to the generator

It's a top-level thing that every HTML project needs, unless it's not supporting IE8. We should default to including it and let projects that don't support IE8 remove it.

When was it removed? I'm coming up empty outside of the printshiv.

https://github.com/contolini/generator-cf/commit/b0585ab2f00df0ce739c4bf6e2d1c736ad42d962
Part of PR #1, so I guess we can't really say it was "removed", but.

Oh sure -- blame @contolini -- I see how it is.


Add Docker Support

Using Docker in WSL does not seem to be correctly identified.
I tried to add Docker support, but I'm not sure if it's the best solution :)

Done, but not sure if the test is relevant though.

Sorry for the delay, I forgot to-rereview your PR.
I agree It's a bit hard to make really useful tests on this library :sweat_smile: We may try to improve them in the future.
Let's merge your PR. Thanks for your work :sparkles:


[SPARK-13203] Add scalastyle rule banning use of mutable.SynchronizedBuffer

@andrewor14
Please take a look

Jenkins, test this please

Test build #50972 has started for PR 11134 at commit 1c80127.

Test build #50973 has started for PR 11134 at commit 1c80127.

Test build #50972 has finished for PR 11134 at commit 1c80127.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/50972/
Test PASSed.

Test build #50973 has finished for PR 11134 at commit 1c80127.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/50973/
Test PASSed.

@srowen

Yes that's fine if it passes tests and it does

LGTM :)

@andrewor14
Is there anything I need to do for this ?

@tedyu just have some patience. Sometimes we're just giving others a chance to look before merging. Merged to master


mqtt with raspbery pi - install issues

Report from user:
I have the interface working on a Raspberry Pi but had some issues following the instructions and using the supplied install.sh
Default kdb+ build from kx comes named q/l32arm . I needed to rename to q/l32
Otherwise error will be seen:
q)\l mqtt.q
'/home/pi/q/l32/mqtt.so: cannot open shared object file: No such file or directory
[2]  /home/pi/q/mqtt.q:2: connX :mqtt 2:(connX;3);
^
[0]  ()
Install.sh fails
Detected System

OS: linux-gnueabihf
TYPE: arm
MACHINE TYPE: arm-unknown-linux-gnueabihf
ERROR: OSTYPE linux-gnueabihf not currently supported by this script
Please view README.md for installation instructions
Issue 2 can be bypassed with: OSTYPE="linux-gnu" ./install.sh but that shows another error:
ERROR: Directory 'lib' does not exist. Please run from release package
The .so is located in the cmake folder

Instead of using install.sh I just did copies manually:
cp q/mqtt.q $QHOME/
cp cmake/mqtt.so $QHOME/l32/

Steps
#Broker and command line tools install
sudo apt-get install mosquitto mosquitto-clients

#q lib folder comes as l32arm from Kx make sure to rename to l32
#.bashrc entries:
export PATH="$PATH:/home/pi/q/l32"
export QHOME="/home/pi/q"

sudo apt-get install libssl-dev git cmake

wget https://github.com/eclipse/paho.mqtt.c/archive/v1.3.2.tar.gz
tar -xzf v1.3.2.tar.gz
cd paho.mqtt.c-1.3.2
make
sudo make install

export BUILD_HOME=/usr/local/lib

cd ..
wget -O mqtt.tar.gz https://github.com/KxSystems/mqtt/archive/1.0.0-rc.1.tar.gz
tar -xzf mqtt.tar.gz
cd mqtt-1.0.0-rc.1
mkdir cmake && cd cmake
cmake ..
make
make install
cd ..
cp q/mqtt.q $QHOME/
cp cmake/mqtt.so $QHOME/l32/

Ref: https://cmake.org/cmake/help/v3.7/variable/CMAKE_HOST_SYSTEM_PROCESSOR.html#variable:CMAKE_HOST_SYSTEM_PROCESSOR


FileUploadInputElement does not always work in Safari

Steps to Reproduce

Create flutter web project.
Add a button that triggers a file upload:

  _uploadImage() {
    InputElement uploadInput = FileUploadInputElement();
    uploadInput.accept = '.png,.jpg,.jpeg';
    uploadInput.click();

    uploadInput.onChange.listen((e) {
      // read file content as dataURL
      final files = uploadInput.files;
      if (files.length == 1) {
        final file = files[0];
        FileReader reader = FileReader();

        reader.onLoadEnd.listen((e) {
          _bloc.uploadImage(reader.result, file.type);
        });

        reader.onError.listen((fileEvent) {
          _log.e('Error uploading content image: $fileEvent');
        });

        reader.readAsArrayBuffer(file);
      }
    });
  }

Run the website in Safari (I experience the bug in Safari 14.0.2)
Click and select a file to upload

Expected results:
After selecting a file, the onChange listener is triggered.
Actual results:
The onChange listener is not always triggered.

Logs

[✓] Flutter (Channel beta, 1.25.0-8.3.pre, on macOS 11.1 20C69 darwin-x64, locale es-ES)
    • Flutter version 1.25.0-8.3.pre at /Users/svprdga/opt/flutter
    • Framework revision 5d36f2e7f5 (11 days ago), 2021-01-14 15:57:49 -0800
    • Engine revision 7a8f8ca02c
    • Dart version 2.12.0 (build 2.12.0-133.7.beta)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/svprdga/Library/Android/sdk
    • Platform android-30, build-tools 30.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.3, Build version 12C33
    • CocoaPods version 1.10.0

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[✓] VS Code (version 1.52.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.18.1

[✓] Connected device (1 available)
    • Chrome (web) • chrome • web-javascript • Google Chrome 88.0.4324.96

• No issues found!

Hi @svprdga
Thanks for filing the issue, just created a minimal code sample, running on the same Safari version. it works every time.

code sample
import 'dart:html';

import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Material App',
      theme: ThemeData.dark(),
      home: Home(),
    );
  }
}

class Home extends StatefulWidget {
  @override
  _HomeState createState() => _HomeState();
}

class _HomeState extends State<Home> {
  String fileType;

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text('Material App Bar'),
      ),
      body: Center(
        child: Container(
          child: Text(fileType ?? ""),
        ),
      ),
      floatingActionButton: FloatingActionButton(
        child: Icon(Icons.add),
        onPressed: () {
          InputElement uploadInput = FileUploadInputElement();
          uploadInput.accept = '.png,.jpg,.jpeg';
          uploadInput.click();

          uploadInput.onChange.listen((e) {
            // read file content as dataURL
            final files = uploadInput.files;
            if (files.length == 1) {
              final file = files[0];
              FileReader reader = FileReader();

              reader.onLoadEnd.listen((event) {
                print(event.timeStamp);
                setState(() {
                  fileType = event.toString();
                });
              });

              reader.onError.listen((fileEvent) {
                print(fileEvent.type);
                setState(() {
                  // fileType = fileEvent.type;
                });
              });

              reader.readAsArrayBuffer(file);
            }
          });
        },
      ),
    );
  }
}

flutter doctor -v
[✓] Flutter (Channel beta, 1.25.0-8.3.pre, on macOS 11.1 20C69 darwin-x64, locale en-GB)
    • Flutter version 1.25.0-8.3.pre at /Users/tahatesser/Code/flutter_beta
    • Framework revision 5d36f2e7f5 (11 days ago), 2021-01-14 15:57:49 -0800
    • Engine revision 7a8f8ca02c
    • Dart version 2.12.0 (build 2.12.0-133.7.beta)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Volumes/Extreme/SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Volumes/Extreme/SDK
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Volumes/Extreme/Xcode.app/Contents/Developer
    • Xcode 12.3, Build version 12C33
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[✓] VS Code (version 1.52.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.18.1

[✓] Connected device (1 available)
    • Chrome (web) • chrome • web-javascript • Google Chrome 88.0.4324.96

• No issues found!

[✓] Flutter (Channel master, 1.26.0-13.0.pre.178, on macOS 11.1 20C69 darwin-x64, locale en-GB)
    • Flutter version 1.26.0-13.0.pre.178 at /Users/tahatesser/Code/flutter_master
    • Framework revision 22f1700427 (21 minutes ago), 2021-01-26 14:34:03 +0800
    • Engine revision 6e391c5942
    • Dart version 2.12.0 (build 2.12.0-257.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Volumes/Extreme/SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Volumes/Extreme/SDK
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Volumes/Extreme/Xcode.app/Contents/Developer
    • Xcode 12.3, Build version 12C33
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[✓] VS Code (version 1.52.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.18.1

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 11.1 20C69 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 88.0.4324.96

• No issues found!

Thank you

Hi @svprdga
Thanks for filing the issue, just created a minimal code sample, running on the same Safari version. it works every time.

code sample
import 'dart:html';

import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Material App',
      theme: ThemeData.dark(),
      home: Home(),
    );
  }
}

class Home extends StatefulWidget {
  @override
  _HomeState createState() => _HomeState();
}

class _HomeState extends State<Home> {
  String fileType;

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text('Material App Bar'),
      ),
      body: Center(
        child: Container(
          child: Text(fileType ?? ""),
        ),
      ),
      floatingActionButton: FloatingActionButton(
        child: Icon(Icons.add),
        onPressed: () {
          InputElement uploadInput = FileUploadInputElement();
          uploadInput.accept = '.png,.jpg,.jpeg';
          uploadInput.click();

          uploadInput.onChange.listen((e) {
            // read file content as dataURL
            final files = uploadInput.files;
            if (files.length == 1) {
              final file = files[0];
              FileReader reader = FileReader();

              reader.onLoadEnd.listen((event) {
                print(event.timeStamp);
                setState(() {
                  fileType = event.toString();
                });
              });

              reader.onError.listen((fileEvent) {
                print(fileEvent.type);
                setState(() {
                  // fileType = fileEvent.type;
                });
              });

              reader.readAsArrayBuffer(file);
            }
          });
        },
      ),
    );
  }
}

flutter doctor -v
[✓] Flutter (Channel beta, 1.25.0-8.3.pre, on macOS 11.1 20C69 darwin-x64, locale en-GB)
    • Flutter version 1.25.0-8.3.pre at /Users/tahatesser/Code/flutter_beta
    • Framework revision 5d36f2e7f5 (11 days ago), 2021-01-14 15:57:49 -0800
    • Engine revision 7a8f8ca02c
    • Dart version 2.12.0 (build 2.12.0-133.7.beta)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Volumes/Extreme/SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Volumes/Extreme/SDK
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Volumes/Extreme/Xcode.app/Contents/Developer
    • Xcode 12.3, Build version 12C33
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[✓] VS Code (version 1.52.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.18.1

[✓] Connected device (1 available)
    • Chrome (web) • chrome • web-javascript • Google Chrome 88.0.4324.96

• No issues found!

[✓] Flutter (Channel master, 1.26.0-13.0.pre.178, on macOS 11.1 20C69 darwin-x64, locale en-GB)
    • Flutter version 1.26.0-13.0.pre.178 at /Users/tahatesser/Code/flutter_master
    • Framework revision 22f1700427 (21 minutes ago), 2021-01-26 14:34:03 +0800
    • Engine revision 6e391c5942
    • Dart version 2.12.0 (build 2.12.0-257.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Volumes/Extreme/SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Volumes/Extreme/SDK
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Volumes/Extreme/Xcode.app/Contents/Developer
    • Xcode 12.3, Build version 12C33
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[✓] VS Code (version 1.52.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.18.1

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 11.1 20C69 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 88.0.4324.96

• No issues found!

Thank you

Thanks for the response @TahaTesser, I don't have right now a mac but as soon as I get one I will try your sample and post back the result. Please await for my response before closing this issue.
Thanks.

Thanks for the response @TahaTesser, I don't have right now a mac but as soon as I get one I will try your sample and post back the result. Please await for my response before closing this issue.
Thanks.

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.

Hi @svprdga while I'm not sure of the exact cause, I can paste some internal code that has worked for us:
import 'dart:async';
import 'dart:html';

import 'package:turtle_app/file_manager/file_picker/file_picker.dart';
import 'package:turtle_app/util/platform_file/platform_file.dart';

FilePicker getFilePicker() => HtmlFilePicker();

class HtmlFilePicker implements FilePicker {
  @override
  Future<PlatformFile?> getFile(FilePickerSource source) {
    if (source == FilePickerSource.file) {
      final completer = Completer<PlatformFile>();

      final uploadInput = FileUploadInputElement();
      document.body!.append(uploadInput);

      void onUploadInputChange(e) {
        void dispose() {
          uploadInput.removeEventListener('change', onUploadInputChange);
          uploadInput.remove();
        }

        final files = uploadInput.files;
        if (files == null) return completer.complete(null);
        completer.complete(PlatformFile.fromHandleSync(files.first));

        // Add an arbitrary delay to clean up to avoid premature
        // disposal of the element
        Future.delayed(const Duration(seconds: 1)).then((value) => dispose());
      }

      uploadInput.addEventListener('change', onUploadInputChange);
      uploadInput.click();

      return completer.future;
    }

    throw UnsupportedError('${source} not supported on this platform');
  }
}

I'm not sure of the exact cause, but here are some key differences I'm noticing:

In Safari, it appears input elements may need to be part of the dom to have an effect (https://stackoverflow.com/questions/47664777/javascript-file-input-onchange-not-working-ios-safari-only)
I use addEventListener which is synchronous. In some browsers, async handlers (such as the onChange stream) may not work
I add the listener before calling onClick to ensure proper order of operations
The dialog can only open in response to a click event due to security reasons

It also looks like this fails only on versions some Safari browsers which make it difficult to reproduce. I'm on 14.0.3 (146<IP_ADDRESS>.7).
The code I'm sharing is coupled to our app, but you can scan through it to see how you might modify your code to match to see if it resolves your issues.


Feature Request: Show interval range in graphical debugger

Hello,
I use clpBNR a lot these days and I also use the graphical debugger of swi prolog a lot.
However, the graphical debugger does not show the interval range of clpBNR constrained variables.
It would simplify the debugging of constraints a lot if we could see them in the graphical debugger like normal constraints.
I noticed the other day that constraints from clpfd are correctly shown in the graphical debugger, so I suppose clpBNR constraints could be shown similarly.
Thank you a lot for your work on this library.

I hadn't observed this as I don't use the graphical debugger (XWindows on MacOS isn't particularly stellar).
clpBNR supports a number of hooks pertaining to attributes (attr_portray_hook, project_attributes, attribute_goals, user:portray) but I guess the graphical debugger must use something different or an existing hook in a different way. Some investigation required.

As a quick fix try:
?- set_prolog_flag(clpBNR_verbose, true).
true.

then try the graphical debugger again.

I've just tried your quick fix and it works.
Thank you very much for your help.

Fixed in 0.11.1.
Intervals displayed in graphical debugger in short or long form dependent on setting of flag clpBNR_verbose


Option to override HttpLoggingPolicy.MAX_BODY_LOG_SIZE

Is your feature request related to a problem? Please describe.
I would like to log AzureResourceManager REST API response bodies  for debug purposes.
I can easily define log options via code
        HttpLogOptions logOptions = new HttpLogOptions();
        logOptions.setLogLevel(HttpLogDetailLevel.BODY_AND_HEADERS);
        logOptions.setPrettyPrintBody(true);
        AzureResourceManager azure = AzureResourceManager.configure()
                .withLogOptions(logOptions)

which is great, except the fact that HttpLoggingPolicy.DefaultHttpResponseLogger will not log response bodies with size > 16Kb.
This is controlled by HttpLoggingPolicy.MAX_BODY_LOG_SIZE
Describe the solution you'd like
I would like to override MAX_BODY_LOG_SIZE value with optional parameter via additional field in HttpLogOptions.
E.g. int maxBodyLogSize.
Describe alternatives you've considered
I can create a new class that implements HttpResponseLogger interface but it will be almost identical to HttpLoggingPolicy.DefaultHttpResponseLogger
Additional context
N/A
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[X ] Description Added
[X ] Expected solution specified

@weidongxu-microsoft could you please take a look?

It is azure-core behavior.
@alzimmermsft Do we have easier way to override MAX_BODY_LOG_SIZE? In mgmt, in many cases, 16k is way too small.
Me too can only think of another response logger as workaround.

The body log size for the default HttpResponseLogger isn't configurable but you can create a custom HttpResponseLogger where you copy the default and change this value.
I can get this added to the backlog to allow this to be configured.


change harness tests to wait for initialization

Current dependencies on/for this PR:

main

PR #87

PR #88   👈

This comment was auto-generated by Graphite.


server: Check Schema - download schema on build #265

Closes #265

Is it possible to make the download optional?
I think it would be better to still have a default swagger-schema and json-schema definitions locally and update them if developer wants to (doesn't happen really often, right?)
I'm a long distance commuter and work sometimes in the train (no internet connection). I have all the needed dependencies in the local cache but the download of the schemas will fail.

Then let's keep these files in git. It will be more safe for offline usage and more predictable. I've added only a task for updating them.

@akryvko sounds good. Could you please also update README.md and let's merge it!

👍

:+1:

:+1:

👍


Added OS-specific overrides for hooks.

Adds OS-specific overrides for hooks.
Addresses Issue #792.

Thanks for the PR. I did notice this targets 0.20.x of trunk. However, it should be based on main, for the 0.21.x release.

Is it possible to merge this so current users have it, and I can create another PR for 0.21.x?

Is it possible to merge this so current users have it, and I can create another PR for 0.21.x?

I really don't want to push out new features to 0.20.x, I'd prefer to focus on the next release.

Closed in favor of PR #877.


Record failing rows for tests into an auditable table

Feature
Feature description
dbt should record rows that fail schema or data tests into tables, to be used for auditing. There are two general ways this could work:

insert into an errors table with a schema like:

run started at
invocation id
test name
failing row primary key / json

Create a table for each test failure in a scratch schema, matching the schema of the test query that failed

The baked-in schema tests will need to be rewritten to provide debuggable information in these tables.
This should probably be an opt-in feature for tests. It could be configured either in schema.yml, though it would also be good to configure whole swaths of tests at once.
Who will this benefit?
dbt users could use this table to quickly determine why their tests failed. At present, it's surprisingly difficult to debug a failed test.

@drewbanin - I totally get that we wouldn't want to bring back all the rows by default, what would be awesome, would be some kind of executable SQL on fail. So the workflow would look like:

Run test
Evaluate Pass/Fail
[optionally] On fail run SQL and spit out the result to logs.

The reason I think this would work nicely is that our analysts are used to looking at the logs of a build to see if something has failed, and having the info right there in the logs would be a massive productivity gain - albeit if there's a config overhead to write in explicitly what should happen on failing for each test.

I support and would use the use case of reporting on failed tests. I am more interested in using them for auditing than for debugging — I would like a nice easy table to query via Looker for a conditional "Go Fix ETL" alert.
When I went looking to do this feature I naturally reached for hooks, as I'm using them already for audit purposes much as described in the documentation.
I'm logging the individual model run:
on-run-start:
    - "create table if not exists {{ target.schema }}.audit_dbt_runs (model text, state text, time timestamp)"

models:
  pre-hook:
    - "insert into {{ target.schema }}.audit_dbt_runs (model, state, time) values ('{{this.name}}', 'start', getdate())"
  post-hook:
    - "insert into {{ target.schema }}.audit_dbt_runs (model, state, time) values ('{{this.name}}', 'end', getdate())"

As well as the overall results:
on-run-start:
    - "create table if not exists {{ target.schema }}.audit_dbt_results (node text, status text, execution_time decimal(10,2), time timestamp)"
on-run-end:
    - "insert into {{ target.schema }}.audit_dbt_results (node, status, execution_time, time) values {{ results_values(results) }}"

(I have a simple macro results_values to turn the results into insertable values.)
In this case I would generally like the exact same thing for tests. Something exactly analogous to the on-run-start/on-run-end with the end run context results would suit my needs:
on-test-start:
    - "create table if not exists {{ target.schema }}.audit_dbt_test_results (node text, status text, execution_time decimal(10,2), time timestamp)"
on-test-end:
    - "insert into {{ target.schema }}. audit_dbt_test_results (node, status, execution_time, time) values {{ results_values(results) }}"

that looks like a great start to a 'dbt_results_mart', where you could have models as a dim, dbt_runs as a dim and fact and errors as an optional fact

@ianterrell This is a great idea. Are you able & willing to share your results_values macro?

i'd be interested in getting a look at that macro was well.

Here's an easy macro for this.
{% macro unpack_results(results) %}

  {% for res in results -%}
-- This is useful for debugging
--     {{ log(res, info=True) }}
        ('{{ invocation_id }}', '{{run_started_at}}', NOW(), '{{ res.node.unique_id }}', '{{ res.status }}', '{{ res.error if res.error != None else ""}}', '{{ res.skip if res.skip != None else "" }}', '{{ res.fail if res.fail != None else "" }}', {{ res.execution_time }} ){{ "," if not loop.last }}
  {% endfor %}

{% endmacro %}

Hi @Aylr and @gordonhwong!
In case it's still useful for anyone, the following macro is what I use:
{% macro results_values(results) %}
  {% for res in results -%}
    {% if loop.index > 1 %},{% endif %}
    ('{{ res.node.alias }}', '{{ res.status }}', {{ res.execution_time }}, getdate())
  {% endfor %}
{% endmacro %}

This pairs with:
on-run-start:
    - "create table if not exists {{ target.schema }}.audit_dbt_results (node text, status text, execution_time decimal(10,2), time timestamp)"
on-run-end:
    - "insert into {{ target.schema }}.audit_dbt_results (node, status, execution_time, time) values {{ results_values(results) }}"

that looks like a great start to a 'dbt_results_mart', where you could have models as a dim, dbt_runs as a dim and fact and errors as an optional fact

This is exactly what I am looking for!
IMO a  run / test / error mart is a prerequisite for any mature data pipeline. Especially to track data run duration and data quality over time. Such a model can then be used as a source for firing alerts when data quality starts to deviate from an average , a run deviates from an average etc....
Of course one could do this by bringing in other tools that help with this (like https://docs.greatexpectations.io, a tool I have seen being suggested a few times).
But imo this should be part of the CORE of dbt; for me it falls under the altering and logging features displayed here: https://www.getdbt.com/product/

Hey @bashyroger, thanks for bringing this topic back to the fore. I just opened a new issue to sketch out our latest thinking about how dbt should store tests in the database: https://github.com/fishtown-analytics/dbt/issues/2593

Hi @Aylr and @gordonhwong!
In case it's still useful for anyone, the following macro is what I use:
{% macro results_values(results) %}
  {% for res in results -%}
    {% if loop.index > 1 %},{% endif %}
    ('{{ res.node.alias }}', '{{ res.status }}', {{ res.execution_time }}, getdate())
  {% endfor %}
{% endmacro %}

This pairs with:
on-run-start:
    - "create table if not exists {{ target.schema }}.audit_dbt_results (node text, status text, execution_time decimal(10,2), time timestamp)"
on-run-end:
    - "insert into {{ target.schema }}.audit_dbt_results (node, status, execution_time, time) values {{ results_values(results) }}"

I noticed that i need to add a commit statement also in the 'on-run-end' to persist the record. Am I doing something wrong?

Hi @Aylr and @gordonhwong!
In case it's still useful for anyone, the following macro is what I use:
{% macro results_values(results) %}
  {% for res in results -%}
    {% if loop.index > 1 %},{% endif %}
    ('{{ res.node.alias }}', '{{ res.status }}', {{ res.execution_time }}, getdate())
  {% endfor %}
{% endmacro %}

This pairs with:
on-run-start:
    - "create table if not exists {{ target.schema }}.audit_dbt_results (node text, status text, execution_time decimal(10,2), time timestamp)"
on-run-end:
    - "insert into {{ target.schema }}.audit_dbt_results (node, status, execution_time, time) values {{ results_values(results) }}"

Hi @ianterrell ,
Thanks for sharing the code. I need to enhance this macro.  Is there a way to execute this macro by itself? I can use this macro using on-run-end but am unable to execute this micro using dbt run-operation.
Thanks,
Pons

Hi @Aylr and @gordonhwong!
In case it's still useful for anyone, the following macro is what I use:
{% macro results_values(results) %}
  {% for res in results -%}
    {% if loop.index > 1 %},{% endif %}
    ('{{ res.node.alias }}', '{{ res.status }}', {{ res.execution_time }}, getdate())
  {% endfor %}
{% endmacro %}

This pairs with:
on-run-start:
    - "create table if not exists {{ target.schema }}.audit_dbt_results (node text, status text, execution_time decimal(10,2), time timestamp)"
on-run-end:
    - "insert into {{ target.schema }}.audit_dbt_results (node, status, execution_time, time) values {{ results_values(results) }}"

Hi @ianterrell ,
Thanks for sharing the code. I need to enhance this macro. Is there a way to execute this macro by itself? I can use this macro using on-run-end but am unable to execute this micro using dbt run-operation.
Thanks,
Pons

Looks like, I can only execute using an on-run-end. Below is what I got from DBT documentation
"The results variable contains a list of Result objects with one element per resource that executed in the dbt job. The Result object provides access within the Jinja on-run-end context to the information that will populate the run results JSON artifact."

@jtcohen6 How would you feel about re-opening this issue? I greatly appreciate the work you did in #3316, but I don't think it satisfies the request in this ticket - namely, an audit log of test results.
In particular, the limitations of #3316 are:

Not immediately apparent whether a test was recently run (have to look at table creation timestamps), and tables for tests that have been removed from the project continue to persist.
Not immediately apparent whether a test passed on its most recent run (have to count rows in the table).
Not immediately apparent which tests correspond to which tables since tests with long names are truncated with a hash that is not easy to map back to the original test name.
Tables are overwritten every time a test is run, so there is no historical record of test results.
The compiled test queries themselves are not saved.
No record of which user ran the test.

Options that I see to generate an audit log:

Enable on-run-start and on-run-end hooks for dbt test.
Create on-test-start and on-test-end hooks analogous to on-run-start and on-run-end, and have results object available in on-test-end context.
Enable pre-hooks and post-hooks for tests.
Enable inclusion of local files in macros (via {% include %}) so that run_results.json could be parsed and written to the database in a run-operation command.
Snapshot all tables in the dbt_test__audit schema immediately after dbt test. Does not solve limitations 1 and 3 above.

Option 3 is @ianterrell's idea of using pre and post hooks to generate the test audit log. We do something similar to create a run audit log for some of our clients who use dbt, and it has worked very well.
Aside from the work to do the implementation, are there blockers/reasons not to add pre and post hooks to tests? I would be happy to attempt to do the implementation with some (likely substantial) guidance.


change optional match syntax in query

Query supports an optional match operator, however the syntax is subtle (a trailing ? on the predicate) and the part of the query that is optional is determined by that query lines subject & object variables. This gets more confusing if there are multiple optional matches. This should be moved to Sparqls optional match format where its much clearer both that there is an optional match, and what the optional match is made up of.

The current syntax is a standard SPARQL 1.1 property path. OPTIONAL {} is still useful for optional blocks of query lines.


Allow Google Analytics ID to be overridden

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/1933
Originally created by @DouweM on 2020-04-01 18:27:51

View 5 previous comments from the original issue on GitLab


Convolution over sequence

Hi,
I'm trying to convolve a filter over a sequence block type and I'm not sure what the cannonical methodology is.
Thanks

I didn't really understand your problem. Do you want to do a convolution on each element of the sequence separately OR on the sequence itself as a whole? What have you tried? (code snippets are always helpful)

Apologies: I am trying to do the following. Given a variable length sequence of vectors [v_1, \dots, v_n] where n varies in the batch and v_i is d-dimensional, I would like to treat the sequence itself as an image with dimensions n x d.
Then I would like to apply a convolutional layer to this image.  I'm implementing the TCNN model of http://www.aclweb.org/anthology/P14-1062

Just for clarification, lets assume that you can do that (I'm not sure how yet, it is beyond my current abilities - maybe concatenating the sequence to generate a td.Tensor((n,d)) then convolution as I did in issue #14 ).
Then, for each example, the number of outputs of the convolution would be different (because they depend on the size (n,d) of the input image). How would you  feed this into the next layer, in your model?

That solution won't work for William, because he wants n to vary by
input.  However, the set of operations and type-shapes supported by loom
(the dynamic graph implementation) is set at graph-construction time, so
this won't work.  To put it another way, you wouldn't have an n to pass
in when building the fold blocks.
However, there is a workaround.  Let's say you wanted to do a 3x3
convolution on your n by d matrix, but this matrix is represented by a fold
block that emits a sequence of d-vectors.  You could take your sequence,
and pass it to td.NGrams(3) in order to get all consecutive triples of
rows.   Then you stack the triples into 3xd matrices and run your
convolution.  I'm guessing it would look something like this:
def _convolve_three(x, y, z):
return *convolution_goes_here(*tf.stack([x, y, z], axis=2))
sequence_block_goes_here >> td.NGrams(3) >>
td.Map(td.Function(_convolve_three))
Warning: this workaround will create extra copies of your data.  This could
be problematic if your implementation is memory bound.
On Thu, Feb 23, 2017 at 11:03 PM, Fábio Dias<EMAIL_ADDRESS>wrote:

Just for clarification, lets assume that you can do that (I'm not sure how
yet, it is beyond my current abilities - maybe concatenating the sequence
to generate a td.Tensor((n,d)) then convolution as I did in issue #14
https://github.com/tensorflow/fold/issues/14 ).
Then, for each example, the number of outputs of the convolution would be
different (because they depend on the size (n,d) of the input image). How
would you feed this into the next layer, in your model?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/fold/issues/15#issuecomment-282220017, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AABepNnGtYK2PoKIlK8Jyy2V_jAMDytaks5rfoC6gaJpZM4MK4Ui
.


Made RemoteWebElement extendable

newElement uses the actual class' constructor to create new objects.
An alternative would have been to make the method protected (it is now private).
This change enables users to extend the RemoteWebElement API by subclassing it.

Sounds good.  Thanks!


add txs status

 const receipt = await alchemy.core.getTransactionReceipt(txHash);
use the above to get the receipt of each txs


Replace Redis URL & URN mapping store with Azure table

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


feat(tier4_autoware_utils, obstacle_cruise_planner): add function to create deleted marker

Description
added a function to delete wall marker in tier4_autoware_utils, and applied it to obstacle_cruise_planner.
In the following video, with this PR, the stop red wall disappeared quickly once the object was removed (and the trajectory's color becomes green).

without this PR
https://user-images.githubusercontent.com/20228327/182741991-fd301c65-3890-4fbb-854f-844c07d8c281.mp4
with this PR
https://user-images.githubusercontent.com/20228327/182741970-481211de-2189-4040-8501-e68fc0527888.mp4
If you want to confirm if this works,

merge this commit to this branch first,
set "obstacle_cruise_planner" in this "cruise_planner" argument

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

#1116

@h-ohta
Could you close #1116 if there is no any more problems?

No. This PR is the part of fix for #1116.


Merge into aio-cli-plugin-cloudmanager

In order to reduce code duplication and to ease dealing with all CloudManager APIs in a single client this project should be merged into https://github.com/adobe/aio-cli-plugin-cloudmanager

this is nothing we want to do for internal reasons.


Discard error report in silent_emitter

Closes #3435.

rustfmt used to write every parsing errors to the internal buffer (https://github.com/rust-lang/rustfmt/pull/3466/files#diff-2c385b8cef9df0e34298e9406e403210L675), and this could grow quite large when the file contains a lot of macros.

ah ok! Thanks @topecongiro


#703: Re-add health check components

Closes #703

Re-add health check monitor and queue, as well as reconciler to policy controller. These components were removed accidentally as part of the separation of controllers
Verification steps
Run the local setup and include a health check section in the DNSPolicy
Verify that a DNSHealthCheckProbe is created and the status is updated

@sergioifg94 Do we not have any tests that check this? Maybe an e2e test that uses health checks in some way would be good to add?

@mikenairn We only have integration tests and unfortunately they didn't catch this because the health check components didn't get deleted from the test suite. I agree we could use some e2e tests for it. I'll create an issue


Update maintainers list

Updating the list of maintainers

Once this is change is approved the following updates are made to complete the maintainer process

CNCF Dapr Maintainers list updated here https://github.com/cncf/foundation/blob/main/project-maintainers.csv#L979
Email addresses are update in https://lists.cncf.io/g/cncf-dapr-maintainers/directory by emailing<EMAIL_ADDRESS>with the changes.
The maintainer teams are updated in the Dapr org https://github.com/orgs/dapr/teams?query=maintainer


Finish fleshing out the OpenAPI case request/response object in data service

 Design & implement 'Case' API entity


React personal contacts: Upgraded to 1.16.1

Category

[ ] Bug Fix
[x] New Feature
Related issues: fixes #X, partially #Y, mentioned in #Z

What's in this Pull Request?
Upgraded to SPFx 1.16.1.

merged to v3. Thank you for the contribution


Automatically deploy the catalogue pipeline if there are no internal model changes

For https://github.com/wellcomecollection/platform/issues/5271
How it works:

At the end of a build on main, Buildkite triggers a "catalogue pipeline deploy" pipeline: https://buildkite.com/wellcomecollection/catalogue-pipeline-deploy-prod

The script that pipeline runs gathers a bunch of info:

What's the live pipeline/API?
What versions of internal model does the associated index know about?
What changed between the current commit and the latest internal model in the index?

If nothing in internal_model/src/main has changed in the interim, it kicks off an automatic deploy into the prod pipeline. Otherwise, it fails the build and waits for a human to deploy locally.

As a future enhancement, I wouldn't mind making that a button you can click in Buildkite, but this is enough for a first pass. I've tested it in the branch and seen it get as far as running weco-deploy, but because I've not deployed any images from this branch, it's hard to know it's working – I'll test it properly when I merge https://github.com/wellcomecollection/catalogue-pipeline/pull/1825
I've also pulled all the deployment-y bits out into this new pipeline, so they can succeed/fail together.

Is there anyway to have the python script be a condition to a deploy step in the buildkite config to keep all deployment stuff there?

I'm not sure I understand what you're suggesting? Do you mean using the Python script to gate whether we run the weco-deploy command, but having that command declared in pipeline.yml?
Where this is a little tricky is that unlike the other services, the pipeline doesn't have a static environment ID – we have to look up the pipeline date, then pass that to the weco-deployment command. The pipeline doesn't deploy like the other services. I'd need to come up a way to pass state from the script to the weco-deploy command. That's probably possible, but it's a bit of work and I want to check that's what you're suggesting.

This would then have the same visibility as the rest of the deployment (fails etc).

The parenthetical is a bit confusing – we still have the same failure behaviours as before. If the weco-deploy command fails, the Python script will exit(1) and bubble up a failure to weco-deploy

That's what I meant. I see the issue. Why are we deploying to prod in previous steps if prod doesn't exists?

Why are we deploying to prod in previous steps if prod doesn't exists?

We're deploying to prod for the adapter services (CALM adapter, Sierra adapter, etc.), not the pipeline services (transformers, ID minter, ingestor, etc.).

Right, let's get this into main and see if it works!


Feature/write user params

Adding additional cvParams and userParams to metadata output

Squashed and pulled into the development branch for version 1.4.3


Azure VM deployment: add install command delay

Adds a 20 second delay to the install command, as from tests it seems to prevent the apt install command from failing.
It's not a pretty solution, but I don't know how else to fix it.
install success rates (successful / total):

main application (test 1): 2/10
main application (test 2): 4/10
unzip only: 4/10
sleep, then unzip: 10/10
sleep, then main application: 10/10

Sicko Fix
Chorus:
Yeah, uh-huh, delay gang, yeah, yeah
Twenty-second wait, got the fix in sight, whoa
Installation stallin’, but we gonna make it right, yeah
Not pretty, but it’s lit, feel the vibe, hold tight
Verse 1:
Apt install actin' up, we can't get no love,
Throw a twenty on it, like we sippin' from the cup,
Success rates, we climbin', kinda guessin' though,
It seems to work, we vibe with the flow.
Bridge:
Seemed like a fix, yeah, but it's kinda vague,
Who needs data, when you got the swag?
Install success, it’s the new wave,
Travis style fix, no root cause to save.
Chorus:
Yeah, uh-huh, delay gang, yeah, yeah
Twenty-second wait, got the fix in sight, whoa
Installation stallin’, but we gonna make it right, yeah
Not pretty, but it’s lit, feel the vibe, hold tight
Verse 2:
It’s not a pretty sight, like a glitch in the night,
But the dev said, “I don’t know,” keep the problem outta sight,
Band-Aid on the wound, quick fix for the hype,
Software strugglin’, but we delay and ignite.
Outro:
Delay gang forever, yeah, we takin' it slow,
Twenty seconds to zen, let the good vibes flow,
Might not be pretty, but it's all that we know,
Sicko fix in the code, yeah, we ready to go.
Final Thoughts:
This pull request, man, it’s like Travis at the Astroworld stage, bringing the heat but maybe lacking the finesse. We feel the energy, but let’s dive deeper next time, fix it for real, not just for the vibe. Let’s keep the beat, but also keep the integrity of the code. It's lit, but let’s make it smarter, ya feel?
#SickoFix #DelayGang #TravisStyle #LitButNotLegit


Cannot use multiple EfficientNetB7 backbones in the same model

I am trying to use 3 'EfficientNetB7' backbones with different input size in the same model but I get the following error.
Is there a way to change the name of the model name?
I tried model._name to set a new name but didn't work.
Please help
ValueError: The name "efficientnet-b7" is used 2 times in the model. All layer names should be unique.

You might want to use solution on here
Altough.... this is not documented and subject to change over time 🤔

https://github.com/keras-team/keras/issues/6194#issuecomment-833002993


pypi package

We probably ought to offer installation via pip.  Iirc openff-toolkit doesn't play nice with pypi, but we can probably make that optional and use rdkit as the fundamental toolkit (which now has a pypi install route).

Related to #350.


ClassAttributesSeparationFixer - Fix trait_import spacing with conflict or visibility resolution

Fixes #5843
Depends on #5868

Coverage increased (+0.002%) to 92.307% when pulling f5b48cf61cd09f79d6b3efd8a6176fbb1f66dca7 on paulbalandan:fix-trait-import-with-resolution into 1b656b57d4ab35c80d5c5c3b60b1e360ba713018 on FriendsOfPHP:3.0.


Update @gear-api examples

https://github.com/gear-tech/wiki/pull/112

The Rust part is ok to me.

@shamilsan  Hi Shamil, seems you need to resolve those conversions.

@shamilsan Hi Shamil, seems you need to resolve those conversions.

Done.


Fix MixedReality ObjectAnchors Conversion Sample issue

Fix issue https://github.com/Azure/azure-sdk-for-net/issues/21459
@scottaddie , @AlexGhiondea and @jongio for notification

@scottaddie , @AlexGhiondea, @JeffreyRichter  Could you please review this PR?

This is a duplicate PR https://github.com/Azure/azure-sdk-for-net/pull/22460, so we will close it.


Add empty states

Add an empty state graphic to the following pages:

category page (filtered results)
your project page
volunteered project page

Note: the graphic should be all the same; messages are different.
Figma design link:
https://www.figma.com/file/7PSJtPUtwrdPRtXbtQFRFZ/helpwithcovid.com?node-id=1025%3A7626
(As mentioned in admin-product - @ryanhefner @adrianthedev would any of you want to take on this task?)

@tinnei really like the design :)


Why are all the actions removing the source and then re-downloading it?

https://github.com/aws-samples/aws-iam-identity-center-pipeline/blob/f3d0b43641ba8b5cdb161652d48106af99093c5c/iam-identitycenter-pipeline.yaml#L243

Good catch, remove this unnecessary step. Thanks for contributing.


Added webpack 4 support

Thanks! Btw, there's some problems with missing webpack.version property in webpack 3.5, i'll patch it.


Fix 3

Add uppercase support

Codecov Report

Merging #4 into master will not change coverage.
The diff coverage is 50%.

@@            Coverage Diff            @@
##             master       #4   +/-   ##
=========================================
  Coverage     70.68%   70.68%
  Complexity       24       24
=========================================
  Files             1        1
  Lines            58       58
=========================================
  Hits             41       41
  Misses           17       17

Impacted Files
Coverage Δ
Complexity Δ

src/Util.php
70.68% <50%> (ø)
24 <0> (ø)
:arrow_down:

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 8736ef0...ed00c4d. Read the comment docs.


fix(lazy/export): Fix lazy + 'export { foo } from "bar"'

Description
When using lazy dependency loading, along with export { foo } from "bar", the generated code for the export currently uses a identifier property, not a function call (_bar.foo, instead of _bar().foo).
The first commit in this PR shows the problem.
The second commit fixes it.
This changes things slightly where previously export { foo } from "bar" was denied lazy-loading for "bar". Was this for performance reasons? If it was only for performance, and if it's something we'd like to preserve, I'll introduce the necessary commit.
Discussion
The original design of CommonJS + Lazy was different than the direction in which we're currently taking it, and I want to make sure that everyone understands, and that we're all aligned on the new design.
Originally, identifiers were denied lazy-loading on-the-fly, which led to many bugs, mostly due to the order of operations. It has since changed to use a preliminary pass using LazyIdentifierVisitor. The goal was to change the failure mode of Lazy. Before, it was able to produce faulty code. Now, the failure mode should be valid but unoptimized code (not always the case yet, as shown in this PR).
The way I see it, if we run CommonJS without LazyIdentifierVisitor, it should still produce valid code. LazyIdentifierVisitor should only be thought of as a performance optimization.
Therefore, CommonJS should not be making decisions as to what can and cannot be lazy loaded.

Can you stop the merge? Without the few lines removed it causes swc to fail immediately on my codebase. I have no idea why

@kdy1 can you give this another glance please? I rebased, changed nothing else

@kdy1 can you give this another glance please? I rebased, changed nothing else


Add custom build script

Coverage remained the same at 37.014% when pulling 0e7852bdf1aa995b897db75f658ad988daf212d6 on qingweibinary:qingwei/plain_build into cd48e8450bf0a23046be1192ec8880101a613319 on binary-com:master.

Do you want to achieve a 'reac-less' and 'react-included' builds?

Exactly, it is requested from Raunak.

Bad idea ...


As an editor I want different background colors for the top of the news article to vary the expression

Acceptance criteria

Could change background colors blue | white, blue is default

In order to ensure good quality we should (Keep the ones we need for this US)

[ ] Perform integration testing
[ ] Perform functional testing
[ ] Extra focus on usability
[ ] Test for portability
[ ] Test for accessibility
[ ] Create automated acceptance test/smoke test
[ ] Perform performance and load test
[ ] Perform user testing
[ ] Perform security testing

Needs to be discussed with designer. Why do we need this? How should it work?

Update: Nick and Birte no longer want the option to change color for the top. They want the blue color on every news article if it's not too hard to create an inverted version of the top bar. Hopefully we can solve that when we make the top bar.


Adds geojson of building g (campus III)

Indoor map for building /  campus III. Closes the second task of #190.

As you have assigned us as reviewers, can we assume that this PR is ready and you are expecting no more changes? The related issue is still in Develop (Ongoing).

Just a quick heads-up from team TF: In #227, we made some changes to the GeoJSON code to also use it for the buildings.

There is now a separate folder geojsons inside of assets, because we felt mixing images and GeoJSONs in one folder would become confusing very quickly.
The GeoJSON filenames for the indoor maps are now declared in the list geojsonLayerFiles in leafletMap.js.

Could you maybe adapt your code to these small changes? Thank you! ^^

While you are at this:
Please rename your both files for the "Hörsaalgebäude".
Right now the are called HS_0.geojson and HG_-1.geojson which is inconsistent!
I would recomment HS as prefix

Just a quick heads-up from team TF: In #227, we made some changes to the GeoJSON code to also use it for the buildings.

There is now a separate folder geojsons inside of assets, because we felt mixing images and GeoJSONs in one folder would become confusing very quickly.
The GeoJSON filenames for the indoor maps are now declared in the list geojsonLayerFiles in leafletMap.js.

Could you maybe adapt your code to these small changes? Thank you! ^^

I just did this and made some fixes with the naming of the files.
Now they are consistent with each other.

As you have assigned us as reviewers, can we assume that this PR is ready and you are expecting no more changes? The related issue is still in Develop (Ongoing).

It is ready to be merged. I just have not been sure because the issue is about campus II + III. However we decided to split it.


Unification-based pointer analysis

@yihozhang is this ready to merge? If so, can you make it not a draft?

@mwillsey Yes this is a self-contained example, although a bit different from one in the paper


[User Confusion]  create-subst silently creates new setters for not existing pattern values

When running 'kpt cfg create-subst SUB_NAME SUB_PARSING_VALUE --pattern SUB_PATTERN --value PATTERN_VAL' to create substitution. The "value" flag is "required" to refer to a setter. If the setter doesn't exist, the create-subst will create a setter for users.
This silently creating new setters could cause user confusions and be more error-prone.
Expected behavior
if the setter doesn't exist, block users from creating the substitutions until they create the setter first.
Reason
User may have a typo in the value field. Current create-subst behavior will pass the command and creates a typo setter quietly. Users have no clue that they actually make a mistake, and it could take them longer time to find out the typo and clean up the bad substitution

Blocking might not be a good user experience. Users want the tool to understand user intent and work in most of the cases rather than throwing errors. To avoid confusion, we should add a warning message that the setter is being created which is addressed in this PR. https://github.com/kubernetes-sigs/kustomize/pull/2419

@phanimarupaka Just a quick check, do we still accept setter and substitution using the same name? Guess a warning for dup names is what user's looking for.

@yuwenma We do not anymore. https://github.com/kubernetes-sigs/kustomize/pull/2437


Add mysql extension for sql scripts

#3410

The Travis build failure is not related to this PR, #3411 should fix it.
Could you recall the search queries for in-the-wild usage on github.com in the original post, for other reviewers?

@pchaigno
sure.
https://github.com/search?p=6&q=extension%3Amysql+NOT+nothack&ref=searchresults&type=Code&utf8=✓
https://github.com/search?utf8=✓&q=filename%3Ayml.mysql+extension%3Amysql&type=Code&ref=searchresults
https://github.com/search?p=1&q=filename%3AREADME.mysql&ref=searchresults&type=Code&utf8=✓
https://github.com/search?utf8=✓&q=filename%3AINSTALL.mysql


Change behavior when number of samples of either train or validation is too small.

Suggested modifications:
Before training:

assert length of train and val samples in hdf5 is equal or larger than "batch size"
if val samples fails assert, expected behavior would be to continue to test loop. Not purpose in training with no val loop.
elif train samples fails assert: warn user and continue (see below)

During training, if not enough train_samples to perform train loop, expected behavior of gdl is to:

skip train loop and go on to val loop only once (for example, num_epochs could be overwritten to 0 after val loop)

Currently, loss is set None and raises an error.
NB: This bugfix could also be seen as a feature: useful for running only validation loop (if num_trn_samples is manually set to 0).

While I took a look at how to implement the modifications it became apparent that the train_py and configyaml structure may need to be redefined to make such functionalities more explicit. A simple hotfix has been proposed in the linked pull request. Afterward, we can discuss with the broad team on how to implement the suggested modifications in the best possible way.


lastsize is inconsistent

When an incoming order is filled from several standing orders, sometimes the lastsize field (in a later quote) seems to be the sum of the fills, and sometimes not.

sell
price = 2
qty = 5

sell
price = 3
qty = 100

BUY
price = 4
qty = 50

This results in a lastsize of 50, even though the fills were of size 45 and 5.
But on other occasions this does not happen:

sell
price = 1
qty = 5

sell
price = 1
qty = 5

BUY
price = 1
qty = 10

This results in a lastsize of 5.
I'm telling you this because it's the only way in which our 2 servers diverge in results now. I fired off 2000 randomly-generated orders at both of them, and the final results were otherwise identical. :)

Meh I'm diverging from the official server myself, so I can't even say your version is wrong.

I can now confirm we both diverge from official, as well as from each other, solely in lastsize. Not sure why.

Hmm I think when multiple fills are generated on official at the same time, it might be arbitrary which one ends up as the last price / size.


Fails to load due to sudo giving up errors about time of the day

Everything works until the main executable is called with sudo on last line of entrypoint script
sudo -H -u '#'"${AMULE_UID}" sh -c "amuled -c ${AMULE_HOME} -o"
that gives this error:
sudo: unable to get time of day: Operation not permitted
sudo: error initializing audit plugin sudoers_audit

Are you using the image from DockerHub or building the image yourself?
What is your OS and processor architecture?

I'm using the image from DockerHub, i'm running on a Raspbian GNU/Linux 10 (buster), Raspberry Pi 4 si it's ARM

I'm running the same Docker image in a Raspberry Pi 4 (with Raspbian 11). It must be some bad configuration.
Could you try to run the docker run command with few parameters and see if it starts?
docker run ngosang/amule
https://github.com/ngosang/docker-amule#docker-cli

Closing due to no activity.

Sorry for the long delay, I've been unable to work on this issue for some time.
I've figured out that I've got some serious problems with ownership and permission of NAS folders, I've screwed up with some NFS mounts around.
Now I've fixed everything I'm able to start amule running the correct users, but I'm having issues with amule connection.
From the logs, everything seems in order (there is a small error about wxWidgets, don't know if could be relevant)

Creating group amule with GID 1000 ...
Creating user amule with UID 1000 ...
/home/amule/.aMule/amule.conf file found. Using existing configuration.
/home/amule/.aMule/remote.conf file found. Using existing configuration.
15580-03-18 06:39:12: Logging to stdout enabled
9455-08-13 00:10:56: Initialising aMuleD 2.3.3 compiled with wxBase(GTK2) v3.0.4 and Boost 1.77
8230-09-12 00:10:56: Checking if there is an instance already running...
8230-09-12 00:10:56: No other instances are running.
16941-03-25 06:39:12: Creditfile loaded, 0 clients are known
6461-05-14 00:10:56:  - Credits expired for 6 clients!
8775-02-08 00:10:56: *** TCP socket (ECServer) listening on <IP_ADDRESS>:34712
9183-05-31 00:10:56: Created Server UDP-Socket at port 34665
8230-09-12 00:10:56: ListenSocket: Ok.
8775-02-08 00:10:56: Created Client UDP-Socket at port 34672
17757-11-04 06:39:12: Loading server.met file: /home/amule/.aMule/server.met
!15580-03-18 06:39:12: 19 servers in server.met found
!17757-11-04 06:39:12: No server list address entry in 'addresses.dat' found. Please paste a valid server list address into this file in order to auto-update your server list
8230-09-12 00:10:56: Loading temp files from /temp.
8775-02-08 00:10:56: Loading PartFile 1 of 12
8775-02-08 00:10:56: Loading PartFile 2 of 12
8775-02-08 00:10:56: Loading PartFile 3 of 12
8775-02-08 00:10:56: Loading PartFile 4 of 12
8775-02-08 00:10:56: Loading PartFile 5 of 12
8775-02-08 00:10:56: Loading PartFile 6 of 12
8775-02-08 00:10:56: Loading PartFile 7 of 12
8775-02-08 00:10:56: Loading PartFile 8 of 12
8775-02-08 00:10:56: Loading PartFile 9 of 12
8775-02-08 00:10:56: Loading PartFile 10 of 12
8775-02-08 00:10:56: Loading PartFile 11 of 12
8775-02-08 00:10:56: Loading PartFile 12 of 12
8230-09-12 00:10:56: All PartFiles Loaded.
17621-09-28 06:39:12: Found 12 part files
17621-09-28 06:39:12: Adding file /temp/013.part.met to shares
17621-09-28 06:39:12: Adding file /temp/014.part.met to shares
17621-09-28 06:39:12: Adding file /temp/015.part.met to shares
17621-09-28 06:39:12: Adding file /temp/016.part.met to shares
17621-09-28 06:39:12: Adding file /temp/017.part.met to shares
17621-09-28 06:39:12: Adding file /temp/018.part.met to shares
17621-09-28 06:39:12: Adding file /temp/019.part.met to shares
17621-09-28 06:39:12: Adding file /temp/020.part.met to shares
17621-09-28 06:39:12: Adding file /temp/021.part.met to shares
17621-09-28 06:39:12: Adding file /temp/022.part.met to shares
17621-09-28 06:39:12: Adding file /temp/023.part.met to shares
17621-09-28 06:39:12: Adding file /temp/024.part.met to shares
15988-07-07 06:39:12: No shareable files found in directory: /incoming
17621-09-28 06:39:12: Found 12 known shared files
17485-08-21 06:39:12: amuled: OnInit - starting timer
15:49:36: Debug: In file /home/buildozer/aports/community/wxgtk/src/wxWidgets-3.0.4/src/unix/threadpsx.cpp at line 507: 'pthread_cond_timedwait()' failed with error 0x00000016 (Invalid argument).
15580-03-18 06:39:12: Asio thread 1 started
15580-03-18 06:39:12: Asio thread 2 started
15580-03-18 06:39:12: Asio thread 3 started
15580-03-18 06:39:12: Asio thread 4 started
8230-09-12 00:10:56: Loading IP filters 'ipfilter.dat' and 'ipfilter_static.dat'.
8230-09-12 00:10:56: Loaded 0 IP-ranges from '/home/amule/.aMule/ipfilter.dat'.
8230-09-12 00:10:56: Loaded 0 IP-ranges from '/home/amule/.aMule/ipfilter_static.dat'.
18438-05-09 06:39:12: IP filter is ready
!15988-07-07 06:39:12: Connecting
8230-09-12 00:10:56: Connecting to eMule Security (<IP_ADDRESS> - <IP_ADDRESS>:6584) using protocol obfuscation.
17621-09-28 06:39:12: Read 153 Kad contacts
!8230-09-12 00:10:56: Kad started.
7550-03-10 00:10:56: Connected to eMule Security (<IP_ADDRESS>:6584)
15580-03-18 06:39:12: New external connection accepted
8639-01-02 00:10:56: Connecting client: amule-remote 0x0001
8230-09-12 00:10:56: Access granted.

But despite the log saying that it's connected, amule seems to be stuck with amber and yellow status, search is not working.
I've reviewed the ports and all is forwarded correctly, I even tried another docker with a less recent version of amule and given the same amule.conf it works, but since it's an old version it crashes too frequently, yours is way more stable.
Do you have any suggestions about where to look for this issue?

Try to remove amule.conf before starting the container. It will create a new file from scratch.
Use the WebUI instead of the desktop client.
Make sure the desktop client version matches aMule Docker version (2.3.3). I have tested 2.3.2 and it's working too.

Thanks for the suggestions. I've tried to, but it seems to no avail.
Had the conf recreated, didn't differ so much from mine, I've just customized port numbers and password hash for web and GUI clients. Connection with desktop client is still successful but with the same odd behavior as before.
With the WebUI, I can connect once but as soon as I enter the correct password I get a connection refused, log complain about amuleweb command exited with status code -6


Increase clarity around the maximum number of participants

Our docs state that the maximum length of the participants array is 255, but we use the uint256 type for the nParticipants (derived from the length of the participants array).
We already have a runtime check that the length of the participants array is less than 255. It should check that the length is less than or equal to 255.
Given that check, we can cast nParticipants as a uint8 in internal functions.
This PR fixes the check, fixes the docs, and makes nParticipants a uint8 everywhere.

Closing as using uint8 instead of uint256 is actually more gas expensive.


How do I make it work on mobile phones without Google service?

It's my build.gradle
implementation 'com.google.android.gms:play-services-base:15.0.1'
implementation 'com.google.android.gms:play-services-maps:15.0.1'
implementation 'com.google.android.gms:play-services-analytics:16.0.0'
implementation 'com.google.firebase:firebase-core:16.0.0'
implementation 'com.google.firebase:firebase-analytics:16.0.0'
implementation 'com.google.firebase:firebase-messaging:17.0.0'
implementation 'com.google.firebase:firebase-crash:16.0.0'
implementation 'com.google.firebase:firebase-perf:16.0.0'

and my android os verdion is android 8.0.0 (huawei), without google service
targetsdkversion  26
and then:
java.lang.SecurityException: Failed to find provider com.google.android.gsf.gservices for user 0; expected to find a valid ContentProvider for this authority
at android.os.Parcel.readException(Parcel.java:1945)
at android.os.Parcel.readException(Parcel.java:1891)
at android.content.IContentService$Stub$Proxy.registerContentObserver(IContentService.java:768)
at android.content.ContentResolver.registerContentObserver(ContentResolver.java:1964)
at android.content.ContentResolver.registerContentObserver(ContentResolver.java:1953)
at com.google.android.gms.internal.firebase-perf.zza.zza(Unknown Source:38)
at com.google.android.gms.internal.firebase-perf.zzq.(Unknown Source:145)
at com.google.android.gms.internal.firebase-perf.zzg.zzp(Unknown Source:127)
at com.google.android.gms.internal.firebase-perf.zzg.zzb(Unknown Source:0)
at com.google.android.gms.internal.firebase-perf.zzh.run(Unknown Source:2)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1162)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:636)
at java.lang.Thread.run(Thread.java:784)

I think I've solved the problem.
In my build.gradle
implementation 'com.google.firebase:firebase-perf:16.0.0'
If I remove the perf，my phone can work!!!
So,the perf sdk is a problem

There should be a way to use every firebase library without crashing on phones without Play Services. The libraries should just do nothing instead of making the app crash

@MMarco94 the app does not crash, that SecurityException is not a fatal app exception.
We would like to support all devices, but that's just not possible for some of our APIs right now.  Google Play services can help us do things we would not otherwise be able to do (or if we could do them, they would not meet our performance / security / size standards)

I understand that some things need Play Services to work, but crashing is not the solution.
A user of our app is reporting that the app is crashing with the following stacktrace:
06-20 22:17:36.233 21005 21107 E AndroidRuntime: FATAL EXCEPTION: pool-6-thread-1
06-20 22:17:36.233 21005 21107 E AndroidRuntime: Process: xxxx.yyyy, PID: 21005
06-20 22:17:36.233 21005 21107 E AndroidRuntime: java.lang.SecurityException: Failed to find provider com.google.android.gsf.gservices for user 0; expected to find a valid ContentProvider for this authority
06-20 22:17:36.233 21005 21107 E AndroidRuntime: 	at android.os.Parcel.readException(Parcel.java:2004)
06-20 22:17:36.233 21005 21107 E AndroidRuntime: 	at android.os.Parcel.readException(Parcel.java:1950)
06-20 22:17:36.233 21005 21107 E AndroidRuntime: 	at android.content.IContentService$Stub$Proxy.registerContentObserver(IContentService.java:768)
06-20 22:17:36.233 21005 21107 E AndroidRuntime: 	at android.content.ContentResolver.registerContentObserver(ContentResolver.java:1924)
06-20 22:17:36.233 21005 21107 E AndroidRuntime: 	at android.content.ContentResolver.registerContentObserver(ContentResolver.java:1913)
06-20 22:17:36.233 21005 21107 E AndroidRuntime: 	at com.google.android.gms.internal.firebase-perf.zza.zza(Unknown Source:38)
06-20 22:17:36.233 21005 21107 E AndroidRuntime: 	at com.google.android.gms.internal.firebase-perf.zzq.<init>(Unknown Source:143)
06-20 22:17:36.233 21005 21107 E AndroidRuntime: 	at com.google.android.gms.internal.firebase-perf.zzg.zzp(Unknown Source:126)
06-20 22:17:36.233 21005 21107 E AndroidRuntime: 	at com.google.android.gms.internal.firebase-perf.zzg.zzb(Unknown Source:0)
06-20 22:17:36.233 21005 21107 E AndroidRuntime: 	at com.google.android.gms.internal.firebase-perf.zzh.run(Unknown Source:2)
06-20 22:17:36.233 21005 21107 E AndroidRuntime: 	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1162)
06-20 22:17:36.233 21005 21107 E AndroidRuntime: 	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:636)
06-20 22:17:36.233 21005 21107 E AndroidRuntime: 	at java.lang.Thread.run(Thread.java:764)

For what I can tell, a SecurityException is not different from any other exception, and it makes the app crash.

@MMarco94 I tried the sample in this repository on an emulator without Play services and could not observe a crash.  Are you able to reproduce?

implementation 'com.google.firebase:firebase-perf:16.0.0'
You should add this, and you can reappear

@MMarco94 @LiuDongCai unless we can reproduce this there is no way to proceed.  Obviously we don't want our APIs to crash on phones without Google services, and in my testing they do not crash.

@samtstern Please  use HTC u11 ，android 8.0

@seventhmoon thank you!  With your help I was able to reproduce it.

We've found the cause and a fix will come in a future version of the firebase-perf SDK.  Thanks all!

@samtstern
crash immediately，when no google play service
FATAL EXCEPTION: pool-4-thread-1
Process: com.addcn.car8891, PID: 2474
java.lang.SecurityException: Failed to find provider com.google.android.gsf.gservices for user 0; expected to find a valid ContentProvider for this authority
at android.os.Parcel.readException(Parcel.java:1942)
at android.os.Parcel.readException(Parcel.java:1888)
at android.content.IContentService$Stub$Proxy.registerContentObserver(IContentService.java:768)
at android.content.ContentResolver.registerContentObserver(ContentResolver.java:1924)
at android.content.ContentResolver.registerContentObserver(ContentResolver.java:1913)
at com.google.android.gms.internal.zzdmf.zza(Unknown Source:35)
at com.google.android.gms.internal.zzdmf.zza(Unknown Source:3)
at com.google.android.gms.internal.zzewh.zzan(Unknown Source:126)
at com.google.android.gms.internal.zzewh.(Unknown Source:19)
at com.google.android.gms.internal.zzevz.zzcke(Unknown Source:151)
at com.google.android.gms.internal.zzevz.zzb(Unknown Source:0)
at com.google.android.gms.internal.zzewa.run(Unknown Source:2)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1162)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:636)
at java.lang.Thread.run(Thread.java:764)

so, right now.,  fixed?

@xiyouMc  upgrade firebase perf sdk to 1.6.2 should have the fix


Freature request: Add batched transaction support

This would be greatly appreciated. Thank you.
See: https://cloud.google.com/firestore/docs/manage-data/transactions#transactions

More Feature Request for server apps:

bulkWriter
verifyIdToken
createSessionCookie
verifySessionCookie
bundle (pass rest format to client)

J
https://code.build/p/sveltekit-todo-app-with-firebase-admin-tqdc5j

@Moe03 Are you open to contribution? Please let me know, I can submit a PR for batch write support.


Add 'commit messages' for things like emp run, emp scale, emp set

It'd be good to have some context when these things happen.
Maybe a flag to make it required or not, and a -m option to add it automatically from the command line, or else you get prompted?
I think this is a good set of commands to start with:

emp run
emp scale
emp set

@sanjayprabhu

Discussed a little bit with @phobologic
Rough decisions for v1:

empire already supports events, the things we want to track commit messages for are a subset of those events.
instead of worrying about storage for v1, add support for adding a commit message to those events

For v2, we can add an alternative event backend that also publishes events to some persistent storage (S3 or RDS etc)

v1:

[ ] emp cli should support -m  for a subset of commands (similar to -a )
[ ] emp api needs to be able to handle the commit message for those events

v1.1:

[ ] should have some flag that allows you to require commit messages for a set of actions for the empire server
[ ] if the server requires a commit message, it should return an error saying commit message is required

v1.2:

[ ] the client should handle the error from the server and have some UI for seeing if the user wants to enter the required commit message

We could store the commit message on the description column on releases, but maybe it'd be better to store it in a separate column.

Yeah, the weird part with storing this is the fact that we are looking for commits in other events that don't create releases, like scaling events.

Ah yeah, good call. Although honestly, maybe those should create new releases? The only reason we didn't was to have the same behavior as Heroku.

+1 to scaling events creating releases. Would love to be able to see them
in emp releases.
On Wed, Apr 27, 2016, 15:46 Eric Holmes<EMAIL_ADDRESS>wrote:

Ah yeah, good call. Although honestly, maybe those should create new
releases? The only reason we didn't was to have the same behavior as Heroku.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/remind101/empire/issues/767#issuecomment-215254193

--
Sanjay

I'm waffling on the idea.  It feels weird in the 12factor world.  I'd almost be more prone to support an 'Events' table and an emp events command that let you look at all the old events, whether for a single app or all of an environment.

Why is it weird? I think it's nice to have the entire application described
by a release. As a contrived example, if you shrink CPU and memory
constraints after a release and then rollback the application won't
actually work as well as before.
On Wed, Apr 27, 2016, 16:50 Michael Barrett<EMAIL_ADDRESS>wrote:

I'm waffling on the idea. It feels weird in the 12factor world. I'd almost
be more prone to support an 'Events' table and an emp events command that
let you look at all the old events, whether for a single app or all of an
environment.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/remind101/empire/issues/767#issuecomment-215266611

--
Sanjay

Yeah, the more I think about this, the more I think scaling should be a new release. Releases should be immutable, but scaling is the only operation that modifies a release.
But, to play devils advocate, there is a performance disadvantage, because creating a new release requires that we submit it to the scheduler, which means all new task definitions across all processes. It may also get weird if ECS ever adds autoscaling, since that wouldn't be reflected in the release.

Just the way we've defined a release in the past - a release is the combination of code & config.  Scaling didn't change either of those - you could deploy a release across any sort of scale.  They feel like very different things - and I could see us wanting to limit what someone can do in the future.  IE: let people scale, but no change config or code.

emp will now support -m for any action that publishes an event.
additionally, you can either provide EMPIRE_MESSAGES_REQUIRED or run the server with --messages.required to require messages for all of these events.
emp will support asking for a message if one is required.


[SPARK-43125][CONNECT] Fix Connect Server Can't Handle Exception With Null Message

What changes were proposed in this pull request?
Fix the bug when Connect Server throw Exception without message.

Why are the changes needed?
Fix bug

Does this PR introduce any user-facing change?
No

How was this patch tested?
Unnecessary

@martin-g @HyukjinKwon PTAL. Thanks

Could you add a test case for this scenario?

hmm.. there is no [SERVER] tag, we should remove if from pr title

Merged to master and branch-3.4.

@Hisoka-X @HyukjinKwon Any reason we drop the abbreviate?

@Hisoka-X @HyukjinKwon Any reason we drop the abbreviate?

Sorry, I don't get it. What do you mean about drop the abbreviate? This PR just do null check.

@Hisoka-X @HyukjinKwon Any reason we drop the abbreviate?

Sorry, I don't get it. What do you mean about drop the abbreviate? This PR just do null check.

Ah sorry! I didn't see you did the abbreviate in the top, please ignore this!

@Hisoka-X @HyukjinKwon Any reason we drop the abbreviate?

Sorry, I don't get it. What do you mean about drop the abbreviate? This PR just do null check.

Ah sorry! I didn't see you did the abbreviate in the top, please ignore this!

@Hisoka-X I think we also change this line?
https://github.com/apache/spark/pull/40780/files#diff-d21a96890e660df398527183a191b0d0c73521aada856e2491c33243f269fdceR128
I could add it to my ongoing PR https://github.com/apache/spark/pull/40785

@Hisoka-X I think we also change this line? https://github.com/apache/spark/pull/40780/files#diff-d21a96890e660df398527183a191b0d0c73521aada856e2491c33243f269fdceR128
I could add it to my ongoing PR #40785

OK for me, but I just want to remind, add null check just want to avoid throw NPE in setMessage, not want to change null to  ""

@Hisoka-X I think we also change this line? https://github.com/apache/spark/pull/40780/files#diff-d21a96890e660df398527183a191b0d0c73521aada856e2491c33243f269fdceR128
I could add it to my ongoing PR #40785

OK for me, but I just want to remind, add null check just want to avoid throw NPE in setMessage, not want to change null to ""

I see. With additional look I think here it's better to keep what it is. as Status's description is nullable:
    this.code = checkNotNull(code, "code");
    this.description = description;
    this.cause = cause;
  }

public Status withDescription(String description) {
    if (Objects.equal(this.description, description)) {
      return this;
    }
    return new Status(this.code, description, this.cause);
  }


WSL should support loading from arrays

Migrated from https://bugs.webkit.org/show_bug.cgi?id=176207:
At 2017-09-01T02:28:53Z<EMAIL_ADDRESS>wrote:
Patch forthcoming.

At 2017-09-01T02:36:43Z<EMAIL_ADDRESS>wrote:
Created attachment 319565
the patch

At 2017-09-01T02:42:54Z<EMAIL_ADDRESS>wrote:
Comment on attachment 319565
the patch
View in context: https://bugs.webkit.org/attachment.cgi?id=319565&action=review

Tools/WebGPUShadingLanguageRI/Parse.js:182

       if (uintVersion + "u" != token.text)

I'd use !==

At 2017-09-01T02:52:39Z<EMAIL_ADDRESS>wrote:
Landed in https://trac.webkit.org/changeset/221468/webkit

At 2017-09-27T19:52:02Z<EMAIL_ADDRESS>wrote:
rdar://problem/34694170

At 2017-09-01T02:51:56Z<EMAIL_ADDRESS>wrote:
(In reply to Saam Barati from comment #2)

Comment on attachment 319565 [details]
the patch
View in context:
https://bugs.webkit.org/attachment.cgi?id=319565&action=review

Tools/WebGPUShadingLanguageRI/Parse.js:182

       if (uintVersion + "u" != token.text)

I'd use !==

Fixed.


Update Readme to link to the repository CONTRIBUTING.md

Description
Small PR to update the link, it took me few minutes to realize it wasn't the CONTRIBUTING.md I was expecting.
Checklist

[x] I have signed the CLA
[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[x] I have updated the CHANGELOG.md and added information about my change to the "dbt-bigquery next" section.

Feel free to merge as I don't have the rights 👍


[C5][UI] Prevent setting unused cookies in browser

When user logged to an UI application(i:e Publisher, Store), in the response to the login request login/token/publisher we set following cookies

Set-Cookie:WSO2_AM_TOKEN_MSF4J_Env=-9280-9b4aa38deed2; path=/api/am/publisher; HttpOnly; Secure
Set-Cookie:WSO2_AM_TOKEN_1_Env=8cb260d2-d880-370a; path=/publisher; Secure
Set-Cookie:WSO2_AM_TOKEN_2_Env=-9280-9b4aa38deed2; path=/publisher; HttpOnly; Secure

But out of those cookies only  WSO2_AM_TOKEN_MSF4J_Env currently being used.
To make the things clear, it's better to remove other unused set-cookie headers.

@Rajith90 WDYT ?

Hi Kasun,
I removed WSO2_AM_TOKEN_1, and change the context of the WSO2_AM_TOKEN_2 to "login/logout/{appName}" because in order to revoke the token when log out request comes.
MSF4j token is untouched

@Rajith90 ,
Can't we use one cookie name for both purposes, MSF4J token for both paths? It seems it is possible to set one cookie for two different paths[1].
i:e

Set-cookie: WSO2_AM_TOKEN_MSF4J=sfa3423rejdasda;Path=/am/api/publisher;http=true
Set-cookie: WSO2_AM_TOKEN_MSF4J=sfa3423rejdasda;Path=/login/token/publisher

[1] : https://tools.ietf.org/html/rfc6265#section-4.2.2
[2] : https://stackoverflow.com/questions/4056306/how-to-handle-multiple-cookies-with-the-same-name

Fixed with: https://github.com/wso2/carbon-apimgt/pull/4745 hence closing the issue


Fixed a typo [testEquansIgnoreCase() --> testEqualsIgnoreCase()]

Fixed a typo in the test name.

Thanks! Cherry-picked int 4.1 and master.


interest-is-interesting: docstring update

Update docstrings to reflect actual functions.

FYI I added the same fix for the exemplar file as well. If you post more of those PRs, please look in both places. Usually they have the same errors.


[K3s] Update docs to mention channel server for installs and upgrades

Updates the docs accordingly for installs and upgrades
Closes https://github.com/rancher/k3s/issues/1744

Note to self: Update the rancher/channelserver README.MD


Delete Firebase Functions associated with this project

The only remaining firebase function automatically created a member
object upon REQUEST_INVITE operation. Raha-API now handles this with PR
#9 on the raha-api project.

🔥


Add margin at the end of main view

Where's my 30th title!?

Thank you for submitting these issues! I've reviewed each one and appreciate the detail you've provided.
I look forward to resolve these as efficiently as possible.


fix(plugin): handle errors during the status command execution

This patch ensures that we continue the status command execution even if we encounter an error while trying to fetch the status of the instances.
Closes #4847

/ok-to-merge

/ok-to-merge


chore: add .DS_Store File to generated .gitignore

Description
As a MacOS User, it would help to get the .DS_Store File ignored by default in a new dart_frog project.
Requirements

[ ] Add .DS_Store to gitignore

@jxstxn1 would you like to be assigned to this issue?

Yeah, for sure :)