[GestureRecognizer] Pinch to Zoom has slight delay before gesture recognition

Reopening this issue because it is still happening.
Original Issue:
https://github.com/flutter/flutter/issues/58636
There is significant detection lag when using pinch to zoom. See video. It goes away if you remove the onTap gesture.
https://user-images.githubusercontent.com/1863934/155457159-1138a72c-d79d-45af-9ce2-d231e71c0d62.mov
Example Dart code:

code sample
import 'package:flutter/cupertino.dart';
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return const CupertinoApp(
      title: 'ZoomTest',
      theme: CupertinoThemeData(primaryColor: Colors.white),
      home: ZoomApp(),
    );
  }
}

class ZoomApp extends StatefulWidget {
  const ZoomApp({Key? key}) : super(key: key);

  @override
  State<ZoomApp> createState() => _ZoomAppState();
}

class _ZoomAppState extends State<ZoomApp> {
  double _sizeNow = 150;
  double _sizeStart = 150;

  @override
  Widget build(BuildContext context) {
    return Center(
        child: GestureDetector(
            onTap: () {
              setState(() {
                _sizeNow = 300;
              });
            },
            onScaleStart: (ScaleStartDetails details) {
              setState(() {
                _sizeStart = _sizeNow;
              });
            },
            onScaleUpdate: (details) {
              setState(() {
                _sizeNow = _sizeStart * details.scale;
              });
            },
            child: Container(
                width: _sizeNow, height: _sizeNow, color: Colors.orange)));
  }
}

Hi @chipweinberger, Thanks for filing the issue. I ran your code sample and I was able to zoom pretty smoothly on both ios and android and could not see any pinch gesture detection lag.
Please share your output of flutter doctor -v.
https://user-images.githubusercontent.com/31410839/155704283-ea1502c1-d99f-4175-bbca-113a20bac2c5.mov

flutter doctor -v (mac)
[✓] Flutter (Channel stable, 2.10.2, on macOS 12.1 21C52 darwin-arm, locale en-GB)
    • Flutter version 2.10.2 at /Users/mahesh/Documents/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 097d3313d8 (2 days ago), 2022-02-18 19:33:08 -0600
    • Engine revision a83ed0e5e3
    • Dart version 2.16.1
    • DevTools version 2.9.2

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/mahesh/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • ANDROID_HOME = /Users/mahesh/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.10.2

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

[✓] Android Studio (version 4.2)
    • 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 11.0.8+10-b944.6916264)

[✓] IntelliJ IDEA Community Edition (version 2021.2.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 61.2.4
    • Dart plugin version 212.5080.8

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

[✓] Connected device (3 available)
    • Redmi K20 Pro (mobile) • <IP_ADDRESS>:5555 • android-arm64  • Android 11 (API 30)
    • macOS (desktop)        • macos            • darwin-arm64   • macOS 12.1 21C52 darwin-arm
    • Chrome (web)           • chrome           • web-javascript • Google Chrome 98.0.4758.102

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

flutter doctor -v

[✓] Flutter (Channel stable, 2.10.1, on macOS 12.1 21C52 darwin-arm, locale en-US)
• Flutter version 2.10.1 at /Volumes/User/MBP-Google-Drive/jamcorder/app/flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision db747aa133 (3 weeks ago), 2022-02-09 13:57:35 -0600
• Engine revision ab46186b24
• Dart version 2.16.1
• DevTools version 2.9.2
[✗] Android toolchain - develop for Android devices
✗ Unable to locate Android SDK.
Install Android Studio from: https://developer.android.com/studio/index.html
On first launch it will assist you in installing the Android SDK components.
(or visit https://flutter.dev/docs/get-started/install/macos#android-setup for detailed instructions).
If the Android SDK has been installed to a custom location, please use
flutter config --android-sdk to update to that location.
[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
• Xcode at /Applications/Xcode.app/Contents/Developer
• CocoaPods version 1.11.2
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[!] Android Studio (not installed)
• Android Studio not found; download from https://developer.android.com/studio/index.html
(or visit https://flutter.dev/docs/get-started/install/macos#android-setup for detailed instructions).
[✓] VS Code (version 1.64.2)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.34.0
[✓] Connected device (2 available)
• iPhone 13 (mobile) • FF7C845F-340A-4611-BF36-E93C8E838C71 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-15-2
(simulator)
• Chrome (web)       • chrome                               • web-javascript • Google Chrome 98.0.4758.109
[✓] HTTP Host Availability
• All required HTTP hosts are available

It looks like you are repro'ing the issue. In your vidoe, from 0:03 to 0:04 the mouse is moving but the square it not expansing. Thats the lag i'm referring to.

Thanks for the clarification I can see a slight delay between when pinch gesture starts and when the container starts responding to the gesture. This is reproducible on both the stable and the master channel. The delay is similar on android too.
https://user-images.githubusercontent.com/31410839/156115006-67a0f200-177a-4543-b9c2-69bb8929d3d1.mov

flutter doctor -v (mac)
[✓] Flutter (Channel stable, 2.10.2, on macOS 12.1 21C52 darwin-arm, locale en-GB)
    • Flutter version 2.10.2 at /Users/mahesh/Documents/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 097d3313d8 (2 days ago), 2022-02-18 19:33:08 -0600
    • Engine revision a83ed0e5e3
    • Dart version 2.16.1
    • DevTools version 2.9.2

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/mahesh/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • ANDROID_HOME = /Users/mahesh/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.10.2

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

[✓] Android Studio (version 4.2)
    • 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 11.0.8+10-b944.6916264)

[✓] IntelliJ IDEA Community Edition (version 2021.2.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 61.2.4
    • Dart plugin version 212.5080.8

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

[✓] Connected device (3 available)
    • Redmi K20 Pro (mobile) • <IP_ADDRESS>:5555 • android-arm64  • Android 11 (API 30)
    • macOS (desktop)        • macos            • darwin-arm64   • macOS 12.1 21C52 darwin-arm
    • Chrome (web)           • chrome           • web-javascript • Google Chrome 98.0.4758.102

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

[✓] Flutter (Channel master, 2.11.0-0.0.pre.681, on macOS 12.1 21C52 darwin-arm, locale en-GB)
    • Flutter version 2.11.0-0.0.pre.681 at /Users/mahesh/Documents/flutter_master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision a288bd5e3f (2 hours ago), 2022-02-24 23:06:19 -0600
    • Engine revision 1012ac4b5a
    • Dart version 2.17.0 (build 2.17.0-141.0.dev)
    • DevTools version 2.11.0

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/mahesh/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • ANDROID_HOME = /Users/mahesh/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    • All Android licenses accepted.

[!] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! CocoaPods 1.10.2 out of date (1.11.0 is recommended).
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin code that responds to your
        plugin usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To upgrade see https://guides.cocoapods.org/using/getting-started.html#installation for instructions.

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

[✓] Android Studio (version 4.2)
    • 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 11.0.8+10-b944.6916264)

[✓] IntelliJ IDEA Community Edition (version 2021.2.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 61.2.4
    • Dart plugin version 212.5080.8

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

[✓] Connected device (3 available)
    • iPhone 12 Pro (mobile) • 14F40EC6-3B73-469A-A1C0-8CF0B152290F • ios            •
      com.apple.CoreSimulator.SimRuntime.iOS-14-5 (simulator)
    • macOS (desktop)        • macos                                • darwin-arm64   • macOS 12.1 21C52
      darwin-arm
    • Chrome (web)           • chrome                               • web-javascript • Google Chrome
      98.0.4758.109

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

It goes away if you remove the onTap gesture.

The scale and the tap recognizer are competing in the GestureArena [1]. The scale recognizer will only start processing the gesture once it is certain that your gesture is not going to be a tap. That's why you see the slight delay.
In other words: This is working as intended.
[1] https://docs.flutter.dev/development/ui/advanced/gestures#gesture-disambiguation

By the time we have 2 fingers on the screen we know it’s a drag, not a tap.
On Wed, Mar 2, 2022 at 3:20 PM Michael Goderbauer @.***>
wrote:

It goes away if you remove the onTap gesture.
The scale and the tap recognizer are competing in the GestureArena [1].
The scale recognizer will only start processing the gesture once it is
certain that your gesture is not going to be a tap. That's why you see the
slight delay.
In other words: This is working as intended.
[1]
https://docs.flutter.dev/development/ui/advanced/gestures#gesture-disambiguation
—
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/99046#issuecomment-1057489053,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAOHB7V2YD5AHU3A6AMG74TU57ZVDANCNFSM5PGIROPQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Also, this might be a separate bug, but I hit this lag even without a tap gesture. I can't figure out how to repro it in a MVP though.
  @override
  Widget build(BuildContext context) {
    if (widget.midiMetas == null || widget.midiMetas!.isEmpty) {
      return const Text('Loading...');
    }
    return GestureDetector(
      dragStartBehavior: DragStartBehavior.start,
      onScaleStart: (ScaleStartDetails details) {
        setState(() {
          _pixelsPerSecondStart = _pixelsPerSecond;
        });
      },
      onScaleUpdate: (details) {
        setState(() {
          _pixelsPerSecond = _pixelsPerSecondStart * pow(details.scale, 3);
        });
        widget.setPixelPerSecondCallback(_pixelsPerSecond);
      },
      child: ListView.builder(
        key: const PageStorageKey<String>("MidiFileListKey"),
        itemCount: widget.midiMetas!.length + 1,
        itemBuilder: (context, index) => buildItem(context, index),
        controller: widget.scrollController
      )
    );
  }

By the time we have 2 fingers on the screen we know it’s a drag, not a tap.

Unfortunately that's not true. Multi-touch devices support parallel taps with multiple fingers.

Unfortunately that's not true. Multi-touch devices support parallel taps with multiple fingers.

That seems like it should be considered a separate gesture, a 'onTwoFingerTap' gesture.


Create LICENSE

cgfc

hi

aim bit crazy

Hey this is a craze aim bot that will improve your game

db2416d1d01ceca0085b9783b6e4509599aa1c85

?


[SMALLFIX] Put the recompute log in the configured logs directory

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

Merged build finished. Test PASSed.

@aaudiber Is the default value mentioned in any documentation?

Yes, updated now

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

LGTM


Module and base image update.

Description
Update Go modules and UBI 9 micro base image. Update actions to use Go 1.21. Fix linter errors and warnings.
GitHub Issues
List the GitHub issues impacted by this PR:

GitHub Issue #

https://github.com/dell/csm/issues/952

Checklist:

[X] I have performed a self-review of my own code to ensure there are no formatting, vetting, linting, or security issues
[X] I have verified that new and existing unit tests pass locally with my changes
[X] I have not allowed coverage numbers to degenerate
[X] I have maintained at least 90% code coverage
[ ] I have inspected the Grafana dashboards to verify the data is displayed properly
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[X] I have maintained backward compatibility

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Please also list any relevant details for your test configuration

[X] Unit tests

Manual inspection of the GUI
I have verified that the dashboards show the data properly while generating I/O and storage resources

[ ] Yes
[X] No

run e2e test

e2e pass


Wait parameter is not documented (or working)

When running VSInstaller.exe --help, it mentions the existence of a --wait parameter. In fact, this might rightly be undocumented on this page, because it doesn't do anything. However, if the feature has been removed, it should be removed from the --help documentation. (Or, ideally, if it's simply broken, a bug should be filed against the installer, and it should also be added to this page.)

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

ID: 5e97b834-fa82-6aa0-5913-4d5a8ed23366
Version Independent ID: e988e3ea-8565-ca88-6ade-ab67425a51ae
Content: Use command-line parameters to install Visual Studio
Content Source: docs/install/use-command-line-parameters-to-install-visual-studio.md
Product: visual-studio-windows
Technology: vs-installation
GitHub Login: @anandmeg
Microsoft Alias: meghaanand

@LadyCailin -- Lady Cailin, thank you for your well-considered feedback.  If you feel this is a product issue, please add it here:

Report product issues to us via the Report a Problem tool that appears both in the Visual Studio Installer and in the Visual Studio IDE.
Use the new Suggest a feature for Visual Studio IDE menu item.
Suggest a feature, track product issues, and find answers in the Visual Studio Developer Community.

@anandmeg -- Megha, please look into this potential documentation issue.

@LadyCailin, thanks your feedback. Can you please share some more info on the scenario where --wait failed for you?
Here's some help info on --wait usage:

--wait is documented in the Layout command and command-line parameters table on  the Use command-line parameters to install, update, and manage Visual Studio page.
Check out an example usage of --wait command.

If you're running into a case where --wait is failing for you, please report your issue using the Report a Problem tool.
Thanks!

Ah! I missed it, I'm seeing it now with Ctrl-F. Anyways, it's not working, so I will report a problem, but it sounds like you're saying that it definitely should work, in which case it is for sure a bug, not a documentation problem. Thanks!


Open API 3.0 issue with duplicate parameters

I did check out this closed ticket #130.
In my case I'm trying to have a different parameter name in my route than what is in the parameters block property in swagger.json. If I disable autogen for the particular route eg. swagger.auto = false it removes the duplicate parameter block. (leaves only $ref line as expected)
In my swagger.json file  under components I have this:
"parameters": {
  "ContextIdParameter": {
    "name": "id",
    "in": "path",
    "description": "context uuid",
    "schema": {
      "type": "string"
    },
    "required": true
  },
},

The idea being each route would have its own same-name parameter eg. /route-1/{id}, /route-2/{id}, etc...
/some-url/:id (cleaner)  I believe that would normally be /some-url/:ContextIdParameter
#swagger.parameters["$ref"] = ["#/components/parameters/ContextIdParameter"]
In my route parameters  block of swagger.json I get the duplicate:
"parameters": [
    {
      "name": "id",
      "in": "path",
      "required": true,
      "schema": {
        "type": "string"
      }
    }, // the duplicate block
    {
      "$ref": "#/components/parameters/ContextIdParameter"
    }
  ],

update
Wanted to point out, query parameters don't have the double output issue even using auto gen

Hi @jdc-cunningham . Thank you for opening the issue.
I'll try to fix this as soon as possible.


Feature request: (print-size) to list the size of all functions

When I'm writing egglog, ensuring no functions have a big size is a good indication of efficient code.
I think adding (print-size) command to list all function's sizes will be good.

We already have (print-size <function-name>) so I think making a variant with no arguments that does all of them is a good idea! I would accept a PR.


CENTOS STREAM 8 dotnet-runtime-deps-7.0 dont provides

Description
I tried to update dotnet version in Centos Stream 8. When update it fails with this error:
Error:
 Problem 1: cannot install the best update candidate for package dotnet-runtime-7.0-7.0.10-1.x86_64
  - nothing provides dotnet-runtime-deps-7.0 >= 7.0.11 needed by dotnet-runtime-7.0-7.0.11-1.x86_64 from packages-microsoft-com-prod
 Problem 2: package aspnetcore-runtime-7.0-7.0.11-1.x86_64 from packages-microsoft-com-prod requires dotnet-runtime-7.0 >= 7.0.11, but none of the providers can be installed
  - cannot install the best update candidate for package aspnetcore-runtime-7.0-7.0.10-1.x86_64
  - nothing provides dotnet-runtime-deps-7.0 >= 7.0.11 needed by dotnet-runtime-7.0-7.0.11-1.x86_64 from packages-microsoft-com-prod

Configuration
Centos Stream 8 x64
Repo Package:
[packages-microsoft-com-prod]
name=packages-microsoft-com-prod
baseurl=https://packages.microsoft.com/centos/8/prod
enabled=1
gpgcheck=1
gpgkey=https://packages.microsoft.com/keys/microsoft.asc
sslverify=1
priority=10

cc @leecow
Looks like dotnet-runtime-deps-7.0-7.0.11-1.x86_64.rpm wasn't published to https://packages.microsoft.com/centos/8/prod/Packages/d/.

Yes, I think the same.

Sorry for the delay. CentOS 8 is now updated.


Port iv to Qt5

switch the cmake search from Qt4 and Qt5, making use of the components for the Qt5 libraries
adopt the automoc handling provided by cmake, so there is no more need to manually run & add moc sources
make use of the cmake imported targets for the Qt5 libraries, so there is no need to manually care about variables for libraries and include directories

Description
This is the swtch of the iv tool from Qt4 to Qt5, as Qt4 is EOL for a couple of years already.
Luckly Qt5 is generally compatible with Qt4, so there are no code changes required.
TBH I am not sure whether this is considered "small change" or not, although almost all the changes were:

replace Qt4 with Qt5 in the build system
use the Qt5 cmake components
enable the cmake automoc, and drop the own code
use the Qt5:: imported targets
which would be done almost in the same way by anybody else skilled in Qt5/cmake...

Tests

build OIIO with Qt5 headers installed, Qt5 is found, and iv is built
ran the newly built iv, and do some basic operations

Checklist:

[x] I have read the contribution guidelines.
[ ] If this is more extensive than a small change to existing code, I
have previously submitted a Contributor License Agreement
(individual, and if there is any way my
employers might think my programming belongs to them, then also
corporate).
[x] I have updated the documentation, if applicable.
[x] I have ensured that the change is tested somewhere in the testsuite
(adding new test cases if necessary).
[x] My code follows the prevailing code style of this project.

Wow. This actually works? It turned out only require build script changes, and no edits to the iv code itself was necessary to port from Qt4 to Qt5?

Wow. This actually works? It turned out only require build script changes, and no edits to the iv code itself was necessary to port from Qt4 to Qt5?

Commit a0d42b4a39670a0d4f25e81b19a5112d916508ce already fixed the Qt includes, so there is nothing more to just build.  Of course iv needs thorough testing, but builds and runs fine, so it can be done step by step.

FindQt5.cmake doesn't appear to be a standard part of the CMake installation. Is this a file you intended to include but forgot to find? Where do I find the .cmake module to allow a findpackage(Qt5)?

Qt5 ships cmake configuration modules (Qt5CoreConfig.cmake, etc -- see the various development packages of qtbase/etc), so find_package(Qt5 COMPONENTS Core):

gives a cmake warning if Qt5CoreConfig.cmake is not available
works if it installed

Yes, works for me! Merging. Thanks for this fix!
We are now upgraded to Qt 5.6 minimum in master (per 2017, and even 2016 VFXPlatform guidelines).
No more Qt4.


Make the example paths prettier

I also removed the prepare script. Doesn't make sense as we are going to use a github that publishes to NPM.... right? 😅
This PR makes the paths prettier with a nice alias. On vite and ts so everyone is happy!

Beautiful magical code!


Downloading unmapped reads contained in salmontools-results.tar.gz from ~all RNA seq samples on refine.bio

I would love to download ~all of the unmapped reads for RNA-seq samples processed by refine.bio, with the following exceptions:

I would be fine ignoring GTEX samples
I would like to ignore bacterial and archaeal samples

I trial'd out downloading the data using this notebook:
https://github.com/taylorreiter/2022-refinebio-unmapped/blob/main/notebooks/try_pyrefinebio.ipynb
The meat of the code is reproduced below:
processor_ids = [processor.id for processor in pyrb.Processor.search(name="SALMONTOOLS")]
samples = pyrb.Sample.search(technology="RNA-SEQ")

filepaths = []
accessions = []
computed_filenames = []
downloadurls = []

for sample in samples[0:99]:
    for result in sample.results:
        if result.processor.id in processor_ids:
            computed_files = pyrb.ComputedFile.search(result__id=result.id)

            for computed_file in computed_files:
                if computed_file.is_qc:
                    filepaths.append(sample.accession_code + "-" + computed_file.filename)
                    accessions.append(sample.accession_code)
                    computed_filenames.append(computed_file.filename)
                    downloadurls.append(computed_file.download_url)

Some challenges I encountered:

The loop is very slow. Is there a better way to query refine.bio to retrieve download links? I don't think this approach will scale to all RNA seq data in refinebio
Is it possible to only get salmontools-result.tar.gz and not also -multiqc* files?
After running through ~27 of iterations, I received ServerError: The server encountered an issue. Do you know what might of caused this, if there is a way to avoid this, or the best way to keep the code running if this is going to happen frequently?

Some questions I have:

Are the S3 download links persistent? If not, how long do they last for?
I was planning on producing the download links, saving them to a spreadsheet, and then automating their download from S3, but I wasn't sure if this is ill-advised.
Does this information already exist is a database or json somewhere? I'm curious if it would be more efficient to work with the database directly considering i'm hoping to download so many files.

Any insight you have would be greatly appreciated!

Hello @taylorreiter !

The loop is very slow. Is there a better way to query refine.bio to retrieve download links? I don't think this approach will scale to all RNA seq data in refinebio
Is it possible to only get salmontools-result.tar.gz and not also -multiqc* files?

This should be possible. I will add filename to the available API filters so you can filter explicitly for that.

After running through ~27 of iterations, I received ServerError: The server encountered an issue. Do you know what might of caused this, if there is a way to avoid this, or the best way to keep the code running if this is going to happen frequently?

I think there are a couple ways to improve that. The current default page size is 25 but you can get ~100 samples from the API relatively quickly. I will update here with a modified script that collects what you are hoping for.

Are the S3 download links persistent? If not, how long do they last for?
I was planning on producing the download links, saving them to a spreadsheet, and then automating their download from S3, but I wasn't sure if this is ill-advised.

The download_url link expires after 7 days (defined as seconds from creation time)

Does this information already exist is a database or json somewhere? I'm curious if it would be more efficient to work with the database directly considering i'm hoping to download so many files.

The information exists in the database and I would be able to provide a csv of everything. However generating signed URLs would be harder to generate on your end.
So once I am able to add filename to the available filters for ComputedFile at that point you will be able to:
 computed_files = pyrb.ComputedFile.search(filename="salmontools-results.tar.gz", is_qc=True, limit=1000)

 for cf in computed_files:
     my_custom_handler(cf)

It may be possible that the above code might get throttled if you are doing > 100 iterations. If so you could pause for one second between each of the API calls (fetching another 1000 computed files).
 computed_files = pyrb.ComputedFile.search(filename="salmontools-results.tar.gz", is_qc=True, limit=1000)

 for cf_page in computed_files.pages:
     for cf in cf_page:
       my_custom_handler(cf)
     time.sleep(1)

I will let you know here as soon as this is ready for you to try.

Also one more thing worth mentioning. The Processor search method does not accept filtering at this time and the name that is used is Salmontools.
processor_ids = [
    processor.id
    for processor in pyrb.Processor.search(limit=600)
    if processor.name == "Salmontools"
]


SPI configuration example

Hello,
I'm looking for some example of SPI configuration where it is possible to select CS line and spiFlags as it is in pigpio:
int spiOpen(unsigned spiChan, unsigned baud, unsigned spiFlags)

Current SPI implementation available as example does not allowed to set even CS line:
        var config = Spi
                .newConfigBuilder(pi4j)
                    .id("my-spi-device")
                    .name("My SPI Device")
                    .address(SPI_CHANNEL)
                    .mode(SpiMode.MODE_0)
                    .baud(Spi.DEFAULT_BAUD)
                    .build();

I've found that SPI implementation I'm looking for is available in PiGpioNativeImpl. This is how I'm going to use it:
piGpio = PiGpioNativeImpl.newInstance();
piGpio.initialize();
spiHandler = piGpio.spiOpen(1, 15600000, 0x0140);    //0x140 -> SPI AUX with BCM17 as CS

byte[] data = new byte[2];
data[0] = (byte)0x70;
data[1] = (byte)0x3C;
piGpio.spiWrite(spiHandler, data, 2);
piGpio.spiClose(spiHandler);
piGpio.shutdown();

I'm not sure how to incorporate above to Context. Should I create context (var pi4j = Pi4J.newAutoContext()) before using SPI?
Thanks in advance for your support.
BTW: please consider to release pi4j v2 and let people to play with it. Thanks to this you will have users feedback and probably more pull-requests and maybe few more contributors in development.

Above example with PiGpioNativeImpl is working without running pi4j context. Of course program has to be run as admin.
I'm still not sure if context is needed or not. Could you please give me some advice on this issue?

@tatery

BTW: please consider to release pi4j v2 and let people to play with it. Thanks to this you will have users feedback and probably more pull-requests and maybe few more contributors in development.

We are working towards this goal.  We have just completed releasing Pi4J v1.3 and Pi4J v1.4 recently and now all efforts are focused on Pi4J v2.0.  However before an official release we need to complete the minimum viable feature-set and minimum architectural design goals.   Pi4J v2.0 compiled SNAPSHOTS are now available in the OSS Maven Repository and available here: https://github.com/pi4j/download/

Above example with PiGpioNativeImpl is working without running pi4j context. Of course program has to be run as admin.
I'm still not sure if context is needed or not. Could you please give me some advice on this issue?

If you are using the pi4j-library-pigpio directly, then there is no need for Context.  The pi4j-library-pigpio library is the RAW JNI wrapper library for the PiGPIO native library (http://abyz.me.uk/rpi/pigpio/).  Using the pi4j-library-pigpio directly is perfectly fine, but its not really using any of the Pi4J architecture or APIs.
The ultimate goal is for Pi4J to provide encapsulated APIs that are independent of the implementation layer.  Even allowing you to change implementation providers without significantly modifying your codebase.

'm looking for some example of SPI configuration where it is possible to select CS line and spiFlags as it is in pigpio:
int spiOpen(unsigned spiChan, unsigned baud, unsigned spiFlags)
Current SPI implementation available as example does not allowed to set even CS line:

    var config = Spi
            .newConfigBuilder(pi4j)
                .id("my-spi-device")
                .name("My SPI Device")
                .address(SPI_CHANNEL)
                .mode(SpiMode.MODE_0)
                .baud(Spi.DEFAULT_BAUD)
                .build();

It does not look like the Pi4J v2.0 APIs support direct configuration of spiFlags with the exception of SPI MODE.  What flag are you needing to modify?  We just have to make sure that the API we provide can be supported via multiple SPI provider implementations and no unique to just PIGPIO.
Here are some additional details on spiOpen(...): http://abyz.me.uk/rpi/pigpio/pdif2.html#spi_open

where it is possible to select CS line

Do you mean you want to change to using the auxiliary SPI chip-select.  Are you needing to change active-low / active-high states for the chip select?
I'm definitely not an expert in SPI .. so feel free to help educate me if I'm missing something.
Thanks, Robert

@eitch ,
Adding you to this thread since you are working on the LinuxFS implementation of I2C/SPI providers.

@savageautomate

What flag are you needing to modify?

I need to use auxiliary SPI with BCM17 as CS but current pi4j SPI API doesn't allow to select CS line (SPI0 has 2 CS lines, SPI1 has 3 CS lines). So to select CS line for SPI0/SPI1 Ux flags (3-bits) from pigpio should be available (in my opinion) in pi4j AP

Are you needing to change active-low / active-high states for the chip select?

I do not need this. I have never seen chip that use CS active high but maybe there are some - I don't know.

@savageautomate please have a look what is specified on raspberry pi website regarding SPI: https://www.raspberrypi.org/documentation/hardware/raspberrypi/spi/README.md.

There is an SPI example in the CrowPi project, see https://pi4j.com/documentation/io-examples/spi/

That's only half of the truth. The configuration part is missing.

I do like the idea of adding a bus (SpiBus) configuration parameter.   I was thinking (perhaps I misunderstood) that you needed more than just the bus configuration.   I had do do some digging to figure out why SpiBus was missing in the APIs and I think it's just a legacy issue that when porting the code to V2 was missed.  I don't think the V1 code ever supported alternate SPI busses probably due to original lack of support via WiringPi and later no alternate SPIDEV overlays ... way back when.
I may leave the flags configuration parameter in place for someones need of advanced configuration bits.
In looking into the potential API  for a LinuxFS-based SPI implementation provider (using spidevB.D interfaces), it looks like the native ioctl API also supports a request parameter which are essentially flags as well.
int ioctl(int fd, unsigned long request, ...);

The second argument is a device-dependent request code.

REF:

https://man7.org/linux/man-pages/man2/ioctl.2.html
https://www.kernel.org/doc/Documentation/spi/spidev

I'll update PR https://github.com/Pi4J/pi4j-v2/pull/147 to add  SpiBus support.

@tatery ,
OK, please take a look at the lasted changes in PR https://github.com/Pi4J/pi4j-v2/pull/147.
I added the bus config parameter as well as channel and chipSelect (channel and chipSelect are both aliases of address added for greater clarity/easy code readability.)
 // create SPI config
 var config  = Spi.newConfigBuilder(pi4j)
     .id("my-spi")
     .name("My SPI")
     .bus(SpiBus.BUS_1)                    <----- CONFIGURE SPI BUS
     .chipSelect(SpiChipSelect.CS_0)       <----- CONFIGURE SPI CS/ADDRESS/CHANNEL
     .mode(SpiMode.MODE_3)
     .build();

 // use try-with-resources to auto-close SPI when complete
 try (var spi = pi4j.spi().create(config)) {

     // write data using output stream
     spi.out().write(sample);
 }

Also, please check the flags implementation code (linked below).  I lifted most of it from your example, but please double check the bitwise operations.  My bitwise-fu is severely out of practice.   The goal was to accept the configured flags value then apply mode and bus bits to override/update the flags value.  (just like your code did.)
SEE:

https://github.com/Pi4J/pi4j-v2/pull/147/commits/beea5a556e069c324939635afff72b2e7f263c07#diff-680c211fdf3afe067275438d15d85243b1b28347f7d8d690d54877799c63ea8b

Thanks, Robert


Primary GPU not detected?

What you were trying to do
Setting the latest dev build up and play to see what's been up.
What actually happened
I have a dedicated GPU (Nvidia Geforce) and an integrated Intel HD 530. But when I run the game, the logs return these following lines (Complete logs attached below):
...
13:39:05.764 [main] INFO  o.t.rendering.ShaderManagerLwjgl - Loading Terasology shader manager...
13:39:05.764 [main] INFO  o.t.rendering.ShaderManagerLwjgl - LWJGL: 2.9.3 / windows
13:39:05.764 [main] INFO  o.t.rendering.ShaderManagerLwjgl - GL_VENDOR: Intel
13:39:05.764 [main] INFO  o.t.rendering.ShaderManagerLwjgl - GL_RENDERER: Intel(R) HD Graphics 530
13:39:05.764 [main] INFO  o.t.rendering.ShaderManagerLwjgl - GL_VERSION: 4.4.0 - Build <IP_ADDRESS>50
13:39:05.764 [main] INFO  o.t.rendering.ShaderManagerLwjgl - SHADING_LANGUAGE VERSION: 4.40 - Build <IP_ADDRESS>50
...
On checking my Task Manager, the game was in fact running on my Integrated GPU (which btw isn't too powerful, hence, bad performance (<30fps))
Since I used a machine with an iGPU till last year, I don't really know if the problem is an old one.
I even tried installing the stable release and forcing the executable to run using the Nvidia processor, but it didn't change anything at all.
How to reproduce
Just ran the game
Log details and game version
Latest source on the develop branch
2017-12-08_13-39-02.zip
Computer details
RAM: 8GB
GPU: Nvidia GeForce GTX 960M
CPU: Intel Core i7-6700HQ
Java Version: 1.8.0_65

UPDATE:
Read up a bit on how Nvidia has a middleware called Optimus that decides which applications use the GPU and which don't, to manage power. It started using the GPU when I manually selected the High Performance profile for Java from Nvidia's Control Panel.
It all boils down to programatically making Java use the High Performance profile by default at the application startup phase.

Hi @zeokav, and welcome back. We are aware of this problem and the solution you found (forcing the use of the discrete GPU through the manufacturer's software) is the only solution I am aware of: no programmatic solution so far, i.e. via LWJGL, which is the library we use to open the opengl context.
If you do find a programmatic solution, please let us know.

@emanuele3d heya! :D
So I did a bit of hunting around and it ends up with the same response on pretty much all forums -

Export the global variable _declspec(dllexport) DWORD NvOptimusEnablement = 0x00000001;. This is generally the best solution, however, depending on which language you're using, you may not be able to export global variables.

That's what's mentioned on the optimus docs as well. Setting environment variables for java applications at runtime is apparently not that straightforward. Need to do it before processes are launched, from what I can gather. Will try to find something though.
Who would've ever expected C to make things simpler? 😕

It's not the responsibility of an OpenGL application to choose the processor it runs on. There are programmatic workarounds that are both platform and vendor specific, but these are non-ideal for multi-card desktops, or systems with more capable iGPUs, and are primarily designed for optimizing workstations. It's an unfortunately clunky side effect of having a system with an iGPU that only makes use of a card as a graphics co-processor, but using the manufacturer's software is the correct approach to this issue.

Closing as solved


Not setting wallpaper after removing a bunch of haskell packages

Setting colors for the terminal emulator works fine, it just won't set the wallpaper.
No error message is given.
I tried reinstalling python-pywal with pacman on arch linux (bspwm), to no avail.
feh --bg-scale $file works fine.
What exactly from haskell may I be missing? Is this related to haskell at all?
May or may not be related to #579

Ok, I think I found the problem, but I don't know how to fix it properly. Commenting lines 103 to 106 on  wallpaper.py fix wallpaper setting. It's getting the wrong desktop for some reason. I did install a gnome application yesterday not sure if it had anything to do with it

Ok, I think I found the problem, but I don't know how to fix it properly. Commenting lines 103 to 106 on  wallpaper.py fix wallpaper setting. It's getting the wrong desktop for some reason. I did install a gnome application yesterday not sure if it had anything to do with it

I do however noticed that my environment variable $XDG_CURRENT_DESKTOP is GNOME. Maybe that's the root cause of the problem?

I do however noticed that my environment variable $XDG_CURRENT_DESKTOP is GNOME. Maybe that's the root cause of the problem?

This was not a problem with pywal. I was somehow running gnome DE over bspwm, I didn't even know that was possible. I also use gdm and I just noticed I had my DE set to gnome, even though it ran along with bspwm somehow. I think when I installed the gnome it set itself as the default DE.
Remember kids stay in school and don't do gnomes

This was not a problem with pywal. I was somehow running gnome DE over bspwm, I didn't even know that was possible. I also use gdm and I just noticed I had my DE set to gnome, even though it ran along with bspwm somehow. I think when I installed the gnome it set itself as the default DE.
Remember kids stay in school and don't do gnomes


add check upper case code in javascript

Why: add code to check upper case

Closes #1134
What's being changed: file

Pretty sure I was assigned #1134

@jaypavasiya
This issue was assigned. Please look for other issues that are not assigned yet. Also, make sure to type !assign in the issue.
Thank you!


Cloud Logging: does not work for Java Cloud Functions

Environment details

OS type and version: Cloud Logging Java11 environment
Java version: Java11

Steps to reproduce
Try to write any LogEntry with Java11 Cloud Function with appropriately set service account.
It will fail with: io.grpc.StatusRuntimeException: DEADLINE_EXCEEDED: deadline exceeded after 49.998533332s. (stacktrace below).
The very same code will work correctly with Cloud Run and the same service account.
Code example
val entry = {any-logging-entry}
val logging = LoggingOptions
        .getDefaultInstance()
        .getService()
logging.write(Collections.singleton(entry))

Stack trace
https://gist.github.com/knyttl/280b465c7ab4135a41f12e77eb07efee

I found the problem - Cloudfunction MUST NOT log asynchronously, because if they finish earlier, the log will not be published correctly.
logger.synchronicity = SYNC

This should be better documented.

Right now Cloud Functions isn't officially supported by this library, but we're working hard on adding better support over the next couple quarters.

The provided minimal example in the prev. comment does not build. Use the following instead:
The code:
package com.example;

import com.google.cloud.functions.HttpFunction;
import com.google.cloud.functions.HttpRequest;
import com.google.cloud.functions.HttpResponse;
import com.google.cloud.logging.Logging;
import com.google.cloud.logging.LoggingOptions;
import com.google.cloud.logging.LogEntry;
import com.google.cloud.logging.Payload.StringPayload;
import com.google.cloud.logging.Severity;
import java.util.Collections;

public class Example implements HttpFunction {
    @Override
    public void service(HttpRequest request, HttpResponse response) throws Exception {
        Logging logging = LoggingOptions.getDefaultInstance().getService();
        LogEntry logEntry = LogEntry
            .newBuilder(StringPayload.of("test log message"))
            .setLogName("cloud-func-test-logging-logger")
            .setSeverity(Severity.INFO)
            .build();
        logging.write(Collections.singleton(logEntry));
    }
}

and add the following into pom.xml file under <dependencies> element:
  <dependency>
    <groupId>com.google.cloud</groupId>
    <artifactId>google-cloud-logging</artifactId>
    <version>3.1.1</version>
  </dependency>

Like @knyttl described in the comment logging library in JAVA sends logs asynchronously by default. It is done to allow batching multiple write calls into one and to minimize API call quota consumption. The sentence about it will be added to README of the repository.
The work will be tracked via issue #665


Objects are not valid as a React child (found: object with keys {item}). If you meant to render a collection of children, use an array instead.

          I'm using version 1.2.0. Issue above is not fix :(

Originally posted by @dungpv in https://github.com/Mindinventory/react-native-stagger-view/issues/2#issuecomment-1323052026

Cc @dungpv


ProtoDataStream to work as cursor

We have the following scenario:
-Service and Client on the same box and using net.pipe binding to talk to each other
-service reads a huge DataTable that could be ~4Gb or even more
-consumer needs to receive this DataTable
-the communication happens using WCF contract like Stream GetData()
-we are using ProtoBuf.Data.ProtoDataStream for table serialization/deserialization
The problem:

both client and server are on one box so there are times when the RAM used by DataTable is doubled.

If there a way to read from the service and delete read data on a service side ?

Hi @alex-parfenyuk, yes, don't use DataTable because it will buffer the entire result set. Use IDataReader instead.

All of the APIs (DataSerializer, IDataSerializerEngine, IProtoDataWriter, ProtoDataReader) support IDataReader. You do not need a DataTable anywhere to use protobuf-net-data.


Connection not close if the transaction returns null

In jpoRx the code of a transaction can return null; in that case a NullPointerException is thrown and the connection remains open.

fixed by eb50150


Error on event beforeSubmit from activeform component

What steps will reproduce the problem?

Create a page with an active form on it
Add several beforeSubmit events
Submit

What is the expected result?
Evaluate all the events attached, if there is someone that its false, we should stop the submit process
What do you get instead?
Even if there are events that return false, when the last one returns true, the form is submitted
Additional info
$("#form").on('beforeSubmit', function(){
console.log('C1'); // we actually should stop here
return false;
});
$("#form").on('beforeSubmit', function(){
console.log('C2'); // but im being called
return true;
});
$("#form").on('beforeSubmit', function(){
console.log('C3'); // and even if C1 is false we should stop here
return false;
});
$("#form").on('beforeSubmit', function(){
console.log('C1 - TRUE'); // that its why im being called
return true;
});

Q
A

Yii version
2.0.37

PHP version
-

Operating system
-

I change only the result property only for this events, to keep all the results as an array and after that we validate the final array, if there is some === false, the event will return false

Won't this work?
If the previous result wasn't false it can continue
$("#form").on('beforeSubmit', e => {
  if (e.result !== false) {
    if (we can submit) {
      return true;
    }
  }
  return false;
});

$("#form").on('beforeSubmit', e => {
  if (e.result !== false) {
    if (we can submit) {
      return true;
    }
  }
  return false;
});

$("#form").on('beforeSubmit', e => {
  if (e.result !== false) {
    if (we can submit) {
      return true;
    }
  }
  return false;
});


Using forms inside of a column

Hey,
I want to use a form (for example - deletion form) inside of a column.
There is a problem that the whole table wrapped by a form so it won't work...
What we can do about it?
Thanks

Right now I'm using an ugly hack
new RenderFunc( function() {
return '';
} ) ,

Hi!
Without whole table wrapped by a form alot of features will not work, so I recommend you to use main form or send your requests via ajax or use hidden form before/after grid that will be submitted via js.


Check membership for user using new API

API checkMembership is changed to checkMembershipForUser,
modify the code as per this change.
Signed-off-by: Abhijeet Kasurde<EMAIL_ADDRESS>

I am following this https://octokit.github.io/rest.js/v18#orgs

Note that prow-github-actions is on an older version of @actions/github which has a dependency on ~v16 of @octokit/rest
This is why the tests failed with
src/utils/auth.ts(24,24): error TS2339: Property 'checkMembershipForUser' does not exist

This would require a much greater overhaul of the project to bring @actions/github up to it's current version. Also note that v17.0.0 of @actions/github had a large number of breaking changes, so this is no small task. I've captured that here: https://github.com/jpmcb/prow-github-actions/issues/38
Using this API also shouldn't cause the /lgtm labeler to fail. I'll look into what's happening in the linked issue and see if I can provide any feedback. Closing for now - feel free to comment with questions!


I found out in Fetch Env, the action space is the linear velocity of grip, instead of grip position described in the paper

Today I found out in Fetch Env, the action space is the linear velocity of grip, instead of grip position described in the paper.
Because when I whether set the action always to [1,0,0,0] or [0.001,0,0,0], the grip always go to the same furthest place in the X axis positive direction. But with different velocity to get to there.
So I have a confusion about this, what exactly is the action space in the code, does it correspond to the paper Plappert et al. (2018) and Hindsight Experience Replay?

The action space of the Fetch env does stand for position commands (pos of gripper + gripper angle).  But it is the relative position w.r.t. to the last time step. That is the reason why you giving [1, 0, 0, 0]  and the arm keeps moving to the target where is 1 meters (actually 0.05m) away from the current pos in the X direction.

@huiwenzhang
Thanks for your elaborate explanation, it makes sense.
But I get confused about your last sentence. What do you mean "1 meters (actually 0.05m) away from the current pos in the X direction" ?  Do you mean the unit of the [x,y,z,grip_state] is 0.05m? Or what's the unit of the action space? Same as the one in observation or not?
Thanks again for replying.

The unit is meter. If you look at the source code of the fetch env, you will know that the action is scaled down by a factor 0.05 before feeding into the controller. That is where the 0.05 comes from.

@huiwenzhang Thanks a lot.

@huiwenzhang
Hello!
Recently I tried to run the multiple thread environment, but it kept failing. And I proposed this issue in  here. Could you please take a look? Any feedback is really appreciated!


Number of stride in EfficientNet 5th MBconv block layer

Hi, I've found that # of stride in 5th MBconv block layer of the backbone (efficient net) is set to 2 ('r3_k5_s22_e6_i80_o112_se0.25'), which are different from the official Tensorflow implementation. ('r3_k5_s11_e6_i80_o112_se0.25') This might affect halving the resolution of the output of that layer, but seems should be kept to the same (paper, Table 1). Is it intended behavior?

along with the last (7th) block. Should it be "r1_k3_s11_e6_i192_o320_se0.25"?


06/19/2018 00:39 : Create Pull Request for branch master -> prTesting1_2018061900

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestRef1.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Error when using cwe_data function of docker-admin.sh

Describe the bug
I get the following error message when using cwe_data function of docker-admin.sh:
cwe_data: ERROR 3948 (42000) at line 1: Loading local data is disabled; this must be enabled on both the client and server sides
To Reproduce

Navigate to vulncode-db folder
Execute setup.sh
Write the command: ./docker/docker-admin.sh cwe_data
See Error (Check screenshot)

Desktop

OS: macOS Catalina

Additional Context
The error looks to be related to MySQL but since everything is containerised, I am not sure how to debug.

Issue 6bea981ff87ab0fe840b57fffb764e8e85e421bf adds --local-infile=1 to the mysql client. I have to verify if this is sufficient to import data from a csv file though.


Generated Role.yaml and RoleBinding.yaml don't work without cluster-admin privileges for namespace local deployments of the Ansible Operator (and probably other's as well)

Bug Report
What did you do?
Trying to follow the use-case for developers (and not cluster-admins) to deploy operators into a development namespace. Developers can create their own projects where they are (local project) admins.
oc new-project ansible-operator
oc get rolebinding
NAME                    ROLE                    USERS                 GROUPS                           SERVICE ACCOUNTS   SUBJECTS
admin                   /admin                  wkulhane-redhat.com

Naturally creating the CRD has to be done as a cluster-admin. But I expect to be able to create (project local) Roles and Role Bindings as a local project admin. This does not work at the moment.
What did you expect to see?
oc create -f deploy/role.yaml
Role memcached-operator created
oc create -f deploy/rolebinding.yaml
RoleBinding memcached-operator created

What did you see instead? Under which circumstances?
oc create -f deploy/role.yaml
Error from server (Forbidden): error when creating "deploy/role.yaml": roles.rbac.authorization.k8s.io "memcached-operator" is forbidden: attempt to grant extra privileges: [{[*] [] [pods] [] []} {[*] [] [services] [] []} {[*] [] [endpoints] [] []} {[*] [] [persistentvolumeclaims] [] []} {[*] [] [events] [] []} {[*] [] [configmaps] [] []} {[*] [] [secrets] [] []} {[*] [apps] [deployments] [] []} {[*] [apps] [daemonsets] [] []} {[*] [apps] [replicasets] [] []} {[*] [apps] [statefulsets] [] []} {[get] [monitoring.coreos.com] [servicemonitors] [] []} {[create] [monitoring.coreos.com] [servicemonitors] [] []} {[*] [cache.example.com] [*] [] []}] user=&{wkulhane-redhat.com 3371479c-db8f-11e8-a57f-06322a6db0d8 [system:authenticated:oauth system:authenticated] map[scopes.authorization.openshift.io:[user:full]]} ownerrules=[{[get] [ user.openshift.io] [users] [~] []} {[list] [ project.openshift.io] [projectrequests] [] []} {[get list] [ authorization.openshift.io] [clusterroles] [] []} {[get list watch] [rbac.authorization.k8s.io] [clusterroles] [] []} {[get list] [storage.k8s.io] [storageclasses] [] []} {[list watch] [ project.openshift.io] [projects] [] []} {[create] [ authorization.openshift.io] [selfsubjectrulesreviews] [] []} {[create] [authorization.k8s.io] [selfsubjectaccessreviews] [] []} {[get] [] [] [] [/healthz /healthz/*]} {[get] [] [] [] [/version /version/* /api /api/* /apis /apis/* /oapi /oapi/* /openapi/v2 /swaggerapi /swaggerapi/* /swagger.json /swagger-2.0.0.pb-v1 /osapi /osapi/ /.well-known /.well-known/* /]} {[create] [ authorization.openshift.io] [selfsubjectrulesreviews] [] []} {[create] [authorization.k8s.io] [selfsubjectaccessreviews] [] []} {[create] [ project.openshift.io] [projectrequests] [] []} {[list watch get] [servicecatalog.k8s.io] [clusterserviceclasses clusterserviceplans] [] []} {[create] [authorization.k8s.io] [selfsubjectaccessreviews selfsubjectrulesreviews] [] []} {[create] [ build.openshift.io] [builds/docker builds/optimizeddocker] [] []} {[create] [ build.openshift.io] [builds/jenkinspipeline] [] []} {[create] [ build.openshift.io] [builds/source] [] []} {[get] [] [] [] [/api /api/* /apis /apis/* /healthz /openapi /openapi/* /swagger-2.0.0.pb-v1 /swagger.json /swaggerapi /swaggerapi/* /version /version/]} {[delete] [ oauth.openshift.io] [oauthaccesstokens oauthauthorizetokens] [] []} {[get] [] [] [] [/version /version/* /api /api/* /apis /apis/* /oapi /oapi/* /openapi/v2 /swaggerapi /swaggerapi/* /swagger.json /swagger-2.0.0.pb-v1 /osapi /osapi/ /.well-known /.well-known/* /]} {[impersonate] [authentication.k8s.io] [userextras/scopes.authorization.openshift.io] [] []} {[create get] [ build.openshift.io] [buildconfigs/webhooks] [] []} {[get list watch create update patch delete] [operators.coreos.com] [clusterserviceversions catalogsources installplans subscriptions] [] []} {[create] [automationbroker.io] [access] [] []} {[create delete deletecollection get list patch update watch] [] [pods pods/attach pods/exec pods/portforward pods/proxy] [] []} {[create delete deletecollection get list patch update watch] [] [configmaps endpoints persistentvolumeclaims replicationcontrollers replicationcontrollers/scale secrets serviceaccounts services services/proxy] [] []} {[get list watch] [] [bindings events limitranges namespaces/status pods/log pods/status replicationcontrollers/status resourcequotas resourcequotas/status] [] []} {[get list watch] [] [namespaces] [] []} {[impersonate] [] [serviceaccounts] [] []} {[create delete deletecollection get list patch update watch] [apps] [daemonsets deployments deployments/rollback deployments/scale replicasets replicasets/scale statefulsets statefulsets/scale] [] []} {[create delete deletecollection get list patch update watch] [autoscaling] [horizontalpodautoscalers] [] []} {[create delete deletecollection get list patch update watch] [batch] [cronjobs jobs] [] []} {[create delete deletecollection get list patch update watch] [extensions] [daemonsets deployments deployments/rollback deployments/scale ingresses networkpolicies replicasets replicasets/scale replicationcontrollers/scale] [] []} {[create delete deletecollection get list patch update watch] [policy] [poddisruptionbudgets] [] []} {[create delete deletecollection get list patch update watch] [networking.k8s.io] [networkpolicies] [] []} {[create] [authorization.k8s.io] [localsubjectaccessreviews] [] []} {[create delete deletecollection get list patch update watch] [rbac.authorization.k8s.io] [rolebindings roles] [] []} {[create delete deletecollection get list patch update watch] [ authorization.openshift.io] [rolebindings roles] [] []} {[create] [ authorization.openshift.io] [localresourceaccessreviews localsubjectaccessreviews subjectrulesreviews] [] []} {[create] [ security.openshift.io] [podsecuritypolicyreviews podsecuritypolicyselfsubjectreviews podsecuritypolicysubjectreviews] [] []} {[get list watch] [ authorization.openshift.io] [rolebindingrestrictions] [] []} {[create delete deletecollection get list patch update watch] [ build.openshift.io] [buildconfigs buildconfigs/webhooks builds] [] []} {[get list watch] [ build.openshift.io] [builds/log] [] []} {[create] [ build.openshift.io] [buildconfigs/instantiate buildconfigs/instantiatebinary builds/clone] [] []} {[update] [ build.openshift.io] [builds/details] [] []} {[admin edit view] [build.openshift.io] [jenkins] [] []} {[create delete deletecollection get list patch update watch] [ apps.openshift.io] [deploymentconfigs deploymentconfigs/scale] [] []} {[create] [ apps.openshift.io] [deploymentconfigrollbacks deploymentconfigs/instantiate deploymentconfigs/rollback] [] []} {[get list watch] [ apps.openshift.io] [deploymentconfigs/log deploymentconfigs/status] [] []} {[create delete deletecollection get list patch update watch] [ image.openshift.io] [imagestreamimages imagestreammappings imagestreams imagestreams/secrets imagestreamtags] [] []} {[get list watch] [ image.openshift.io] [imagestreams/status] [] []} {[get update] [ image.openshift.io] [imagestreams/layers] [] []} {[create] [ image.openshift.io] [imagestreamimports] [] []} {[delete get patch update] [ project.openshift.io] [projects] [] []} {[get list watch] [ quota.openshift.io] [appliedclusterresourcequotas] [] []} {[create delete deletecollection get list patch update watch] [ route.openshift.io] [routes] [] []} {[create] [ route.openshift.io] [routes/custom-host] [] []} {[get list watch] [ route.openshift.io] [routes/status] [] []} {[update] [ route.openshift.io] [routes/status] [] []} {[create delete deletecollection get list patch update watch] [ template.openshift.io] [processedtemplates templateconfigs templateinstances templates] [] []} {[create delete deletecollection get list patch update watch] [extensions networking.k8s.io] [networkpolicies] [] []} {[create delete deletecollection get list patch update watch] [ build.openshift.io] [buildlogs] [] []} {[get list watch] [] [resourcequotausages] [] []} {[create] [ authorization.openshift.io] [resourceaccessreviews subjectaccessreviews] [] []} {[create update delete get list watch patch] [servicecatalog.k8s.io] [servicebrokers serviceclasses serviceplans serviceinstances servicebindings] [] []} {[create update delete get list watch] [settings.k8s.io] [podpresets] [] []}] ruleResolutionErrors=[]

Same for rolebinding.yaml
Environment

operator-sdk version:
master as of 10/30/2018
Insert operator-sdk release or Git SHA here. If you have paste the Gopkg.lock operator-sdk information here.

Kubernetes version information:
OpenShift Container Platform 3.11.16
Insert output of kubectl version here
Client Version: version.Info{Major:"1", Minor:"11+", GitVersion:"v1.11.0+d4cacc0", GitCommit:"d4cacc0", GitTreeState:"clean", BuildDate:"2018-10-10T16:38:01Z", GoVersion:"go1.10.3", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"11+", GitVersion:"v1.11.0+d4cacc0", GitCommit:"d4cacc0", GitTreeState:"clean", BuildDate:"2018-09-26T12:30:09Z", GoVersion:"go1.10.2", Compiler:"gc", Platform:"linux/amd64"}

oc version:
oc v3.11.0+0cbc58b
kubernetes v1.11.0+d4cacc0
features: Basic-Auth GSSAPI Kerberos SPNEGO
Server https://master.wk311.example.opentlc.com:443
openshift v3.11.16
kubernetes v1.11.0+d4cacc0

Kubernetes cluster kind: OpenShift Container Platform 3.11.16

Are you writing your operator in ansible or go?
Ansible

@shawn-hurley As requested. :-) Let me know if you need a live cluster to test.

@wkulhanek I was able to test anbd reproduce on my cluster. I see the same output thanks for the bug!
I think this is because openshift's admin role does not use the * selector in its rules definition for the verbs, instead it lists them. I believe we may need to list our verbs in the scaffold. I was also able to reproduce with a golang operator.
Note: this is only a problem for a user with admin rights in their namespace, a cluster admin does not have this problem.

/remove-lifecycle stale

Thanks for finding and reporting this @wkulhanek and keeping it alive @shawn-hurley! Just ran into this same issue with my first operator and was glad to see that it wasn't unnoticed by this project. Happy to help test workarounds or new scaffolding.

Hi @shawn-hurley,
So, the issue is with the "" in the definition of the role. Which means that it could be solved by replace the '' for what is required access.
So could we not solve it by justing addressing in the docs and maybe as a comment as well in the scaffold project the k8s doc over RBCA? It shows that could sort many of issues raised here as well over permissions. WDYT?
PS.: Removing the ansible-operator label since it is valid for all types.


Dart -> Engine roll is failing on failure in const_finder_test

Starting with the roll here: https://github.com/flutter/engine/pull/25008
But not occurring on the roll here: https://github.com/flutter/engine/pull/25003 which failed for other reasons
Running command "/b/s/w/ir/cache/builder/src/out/host_debug_unopt/dart-sdk/bin/dart /b/s/w/ir/cache/builder/src/flutter/tools/const_finder/test/const_finder_test.dart /b/s/w/ir/cache/builder/src/out/host_debug_unopt/gen/frontend_server.dart.snapshot /b/s/w/ir/cache/builder/src/out/host_debug_unopt/flutter_patched_sdk"
Generating kernel fixtures...
/b/s/w/ir/cache/builder/src/flutter/tools/const_finder/test/fixtures/lib/consts.dart
Checking recursive calls.
Checking for expected constants.
Checking for non-constant instances.
Expected: {"constantInstances":[{"stringValue":"1","intValue":1,"targetValue":null},{"stringValue":"10","intValue":10,"targetValue":null},{"stringValue":"6","intValue":6,"targetValue":null},{"stringValue":"7","intValue":7,"targetValue":null},{"stringValue":"8","intValue":8,"targetValue":null},{"stringValue":"9","intValue":9}],"nonConstantLocations":[{"file":"file:///b/s/w/ir/cache/builder/src/flutter/tools/const_finder/test/fixtures/lib/consts_and_non.dart","line":14,"column":26},{"file":"file:///b/s/w/ir/cache/builder/src/flutter/tools/const_finder/test/fixtures/lib/consts_and_non.dart","line":17,"column":41},{"file":"file:///b/s/w/ir/cache/builder/src/flutter/tools/const_finder/test/fixtures/lib/consts_and_non.dart","line":17,"column":26},{"file":"file:///b/s/w/ir/cache/builder/src/flutter/tools/const_finder/test/fixtures/lib/consts_and_non.dart","line":19,"column":26},{"file":"file:///b/s/w/ir/cache/builder/src/flutter/tools/const_finder/test/fixtures/pkg/package.dart","line":14,"column":25}]}
Actual:   {"constantInstances":[{"stringValue":"1","intValue":1,"targetValue":null},{"stringValue":"10","intValue":10,"targetValue":null},{"stringValue":"6","intValue":6,"targetValue":null},{"stringValue":"7","intValue":7,"targetValue":null},{"stringValue":"8","intValue":8,"targetValue":null},{"stringValue":"9","intValue":9}],"nonConstantLocations":[{"file":"file:///b/s/w/ir/cache/builder/src/flutter/tools/const_finder/test/fixtures/lib/consts_and_non.dart","line":14,"column":26},{"file":"file:///b/s/w/ir/cache/builder/src/flutter/tools/const_finder/test/fixtures/lib/consts_and_non.dart","line":17,"column":26},{"file":"file:///b/s/w/ir/cache/builder/src/flutter/tools/const_finder/test/fixtures/lib/consts_and_non.dart","line":19,"column":26},{"file":"file:///b/s/w/ir/cache/builder/src/flutter/tools/const_finder/test/fixtures/pkg/package.dart","line":14,"column":25}]}
Tests failed - exit code: -1

/cc @dnfield @a-siva

@alexmarkov https://dart.googlesource.com/sdk/+/daae24b488023bbcf2f5a353fa29b084ba16b6c7 seems to be the culprit for failing const_finder_test.

Should we revert that CL or do you have a forward fix?

I don't think that cl can be reverted as it fixed different regression.

Original regression was https://github.com/dart-lang/sdk/commit/2d63f26ef5a4170ff1d6035d5388824788fd30d8, it could not have been cleanly reverted, so two follow-up fixes landed yesterday: https://dart.googlesource.com/sdk/+/daae24b488023bbcf2f5a353fa29b084ba16b6c7 and https://dart.googlesource.com/sdk/+/e8f57b66058287fb5243928341f1dde517dc249a. And the first follow-up fix caused new regression.

@aam Maybe this just needs an update to the expectation file?

@mkustermann the expectations looks correct.
New actual results seems to be missing one non-const at {"file":"file:///usr/local/google/home/aam/p/f/flutter/engine/src/flutter/tools/const_finder/test/fixtures/lib/consts_and_non.dart","line":17,"column":41}
line 17:   final Target target3 = Target('3', 3, Target('5', 5, null)); // should be tree shaken out.

The line
  final Target target3 = Target('3', 3, Target('5', 5, null)); // should be tree shaken out.

has 2 constructor invocations.
When const_finder visits ConstructorInvocation nodes in kernel AST, it doesn't go deeper if class matches and non-constant instance is found:
https://github.com/flutter/engine/blob/2ab18532859ca9ba7f3fa13479dd1bb03f3d9435/tools/const_finder/lib/const_finder.dart#L66-L77
It means that the 2nd ConstructorInvocation at the same line will not be visited if it is not hosted out of the 1st ConstructorInvocation. Tree shaker might or might not do that hoisting depending on various factors. So, I think it is correct to update the expectations for this test.
@dnfield @zanderso Longer term I think we should fix const_finder to visit children nodes even if the matching ConstructorInvocation node is found, otherwise we cannot guarantee that it finds all constant and non-constant nodes.

@aam found that the issue is not fully fixed with https://github.com/flutter/engine/pull/25020: now all expectations appear in the test, but in a different order.
We still need to update expectations. Also, it would be useful to sort nonConstantLocations in the test to make it less fragile.

https://github.com/flutter/engine/pull/25022 manual roll landed with updated expectations.


Add night dancer by imase

Song Name
night dancer
Artist Name
imase
Source
Youtube
Youtube Link
https://youtu.be/kagoEGKHZvU?si=kq9m88gUYKVBwbPG
Direct File Link
No response
Song ID
20228300
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: 92398039

reject
same here: song id doesnt exist


refactor(raw_vehicle_cmd_converter): remove duplicates and add unit test

Description

refactor change

100% tests passed, 0 tests failed out of 5

Label Time Summary:
copyright     =   2.17 sec*proc (1 test)
cppcheck      =   0.70 sec*proc (1 test)
gtest         =   0.15 sec*proc (1 test)
lint_cmake    =   0.40 sec*proc (1 test)
linter        =   4.92 sec*proc (4 tests)
xmllint       =   1.64 sec*proc (1 test)

Total Test time (real) =   5.08 sec
Finished <<< raw_vehicle_cmd_converter [5.44s]

Summary: 1 package finished [6.10s]

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.

TODO
rtemove ff_map_initialized_ and terminate when there is no ff map


No Block Source Available

Version: v<IP_ADDRESS>-bb4b082 (64-bit)
Current Numbers of Blocks: 195043
Need help. addnode doesn't work.

Check your debug.log for any peering issues. DNS seed works fine so you shouldn't have an issue there apart when you have a firewall issue on your side.

Here are a couple of issues:
ERROR: AcceptToMemoryPool: non-final
P2P peers available. Skipped DNS seeding (idk if this is an error)

Latter pretty much says you actually have peers. Odd. You can try deleting peers.dat so the wallet can grab some fresh peers from the DNS seeds.

That didn't work.
Should I try uninstalling it and reinstalling it?

If that didn't work your network might have an issue. Attach a clean debug.log please. Don't think uninstalling will change anything.

debug.log.zip
the log is in this zip.

Weird. It seems like it synced just fine but then stopped for no reason at all. You actually do get connections to peers just fine according to the log but then receive no blocks. What you can try now is removing your blockchain data and then trying to use the bootstrap file hosted here: https://bootstrap.chain.so (instructions are there too).

So far it is working, I will let you know if it works perfectly.

This solution is working perfectly. Thanks for helping me!

Glad to hear, closing here then 👍


Update specs2-core to 4.12.2

Updates org.specs2:specs2-core from 4.12.1 to 4.12.2.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.specs2", artifactId = "specs2-core" } ]

labels: test-library-update, semver-patch

Codecov Report

Merging #22 (1fe5f88) into master (eef9037) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #22   +/-   ##
=======================================
  Coverage   31.78%   31.78%
=======================================
  Files          17       17
  Lines         258      258
  Branches        6        6
=======================================
  Hits           82       82
  Misses        176      176

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 eef9037...1fe5f88. Read the comment docs.


nlog.dll.nlog is not correctly resolved if hosted on UNC path

GetCandidateFileNames returns AbsolutPath instead of LocalPath in Ln 671.

That causes nlog.dll.nlog hosted on UNC path not to be resolved.

It retruns:
/Share/nlog.dll.nlog

instead of:
\\Server\Share\nlog.dll.nlog

Moved from http://nlog.codeplex.com/workitem/7043

not sure where to fix this

This seems to be solved before, but broken again in 4.0.1
See
https://github.com/NLog/NLog/commit/9032367cdf88d66a345d49c8f3fdcf00b820233e#commitcomment-11797254
and https://github.com/Mike120Git/NLog/pull/1

Raising to must have as the solution is almost clear.

needs unit tests, if possible

PR merged


[opt](routine load) support routine load perceived schema change

At present, if the table structure changes, the routine load cannot perceive it. As a long-running load, it should be able to perceive the changes in the table structure.

run buildall

run buildall

run buildall

run buildall


1:1 tools

[x] Chat
[x] View students screen
[x] Student and teacher audio
[ ] Nudge
[ ] Share my screen (teacher's screen)
[ ] Student / teacher video (lower pri)
[ ] Full screen (show the student's screen in full screen, browser full screen as well).
[ ] Pointer (a pointer that allows the teacher to move around on the students screen).

@matthewsaxby thoughts on a pointer and full screen mode?


nixos/codimd: add option environmentFile for injecting secrets

Motivation for this change
Currently the upstream module is missing a way to pass secrets to the service without adding them to the Nix store.
Things done
Secrets are injected from the environment into the rendered
configuration before each startup using envsubst.
The test now makes use of this feature for the db password.

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg test codimd


Update Puppet module metadata and add link to module source

What type of PR is this?
/kind bug
Any specific area of the project related to this PR?
/area integrations
What this PR does / why we need it:
This PR updates Puppet module metadata and adds a link to the module's source. The link added to the readme is for the benefit of people viewing https://forge.puppet.com/modules/sysdig/falco
Which issue(s) this PR fixes:

Relates to #142
Special notes for your reviewer:
A release to the Puppet Forge will be needed to finish fixing the linked issue.

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

bb47871 Update metadata and add link to module source

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: genebean
To complete the pull request process, please assign fntlnz after the PR has been reviewed.
You can assign the PR to them by writing /assign @fntlnz in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Welcome @genebean! It looks like this is your first PR to falcosecurity/evolution 🎉

Closing as I have rewored this code into a standalone module. The current module cannot be installed via git because it is not in the root of a repo / a repo by itself.

Ok, thanks for the update @genebean

@maxgio92 FWIW, https://github.com/falcosecurity/falco/issues/2005 has details and https://forge.puppetlabs.com/modules/puppet/falco has been released

Thanks @genebean! I left some comments on Falco #2005


Add log/wait/fault-stream support for asynchronous event sinks

Purpose

Support a way to retrieve the sink options and type at sink mapper. Fixes #1469
Support error handling (log/wait/fault-stream) when event sinks publish data asynchronously. Fixes #1465
Fix time management in not pattern cases.

Release note

#1469 Support a way to retrieve the sink options and type at sink mapper.
#1465 Support error handling (log/wait/fault-stream) when event sinks publish data asynchronously.

Codecov Report

Merging #1473 into master will increase coverage by 0.24%.
The diff coverage is 56.41%.

@@             Coverage Diff             @@
##             master   #1473      +/-   ##
===========================================
+ Coverage     67.86%   68.1%   +0.24%
- Complexity       41      42       +1
===========================================
  Files           643     643
  Lines         26915   26963      +48
  Branches       4248    4256       +8
===========================================
+ Hits          18265   18363      +98
+ Misses         6869    6811      -58
- Partials       1781    1789       +8

Impacted Files
Coverage Δ
Complexity Δ

...ore/util/parser/helper/DefinitionParserHelper.java
74.89% <ø> (ø)
0 <0> (ø)
:arrow_down:

.../output/sink/distributed/DistributedTransport.java
85.71% <ø> (ø)
0 <0> (ø)
:arrow_down:

...t/stream/state/AbsentLogicalPreStateProcessor.java
81.86% <100%> (+0.09%)
0 <0> (ø)
:arrow_down:

.../io/siddhi/core/util/transport/InMemoryBroker.java
88.37% <100%> (ø)
0 <0> (ø)
:arrow_down:

...o/siddhi/core/stream/output/sink/InMemorySink.java
100% <100%> (ø)
0 <0> (ø)
:arrow_down:

.../siddhi/core/util/event/handler/StreamHandler.java
74% <100%> (+12%)
0 <0> (ø)
:arrow_down:

.../io/siddhi/core/stream/output/sink/SinkMapper.java
44.06% <100%> (+1.96%)
0 <0> (ø)
:arrow_down:

...a/io/siddhi/core/util/SiddhiAppRuntimeBuilder.java
81.1% <100%> (+0.14%)
0 <0> (ø)
:arrow_down:

...ain/java/io/siddhi/core/stream/StreamJunction.java
67.8% <36%> (+7.95%)
0 <0> (ø)
:arrow_down:

...ry/input/stream/state/StreamPreStateProcessor.java
94.08% <66.66%> (-0.84%)
0 <0> (ø)

... and 19 more

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 95db2a2...2229e3f. Read the comment docs.


Support for newer versions of zod

I'd like to use a newer version of zod but am running into peer dependency issues.
$ npm ci
npm ERR! code ERESOLVE
npm ERR! ERESOLVE could not resolve
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/zod
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/@asteasolutions/zod-to-openapi
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Conflicting peer dependency<EMAIL_ADDRESS>npm ERR! node_modules/zod
npm ERR!   peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!   node_modules/@asteasolutions/zod-to-openapi
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR!
npm ERR! See /home/<redacted>/.npm/eresolve-report.txt for a full report.

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/<redacted>/.npm/_logs/2022-06-28T20_05_29_528Z-debug-0.log

@blade2005 we release v1.2.3. In it we made the peerDependency of zod be  ^3.14.0. Could you please verify that updating to the latest version resolves your issue.

Yes that fixes it perfectly @AGalabov thanks for the quick turn around


Multiple Provider Support + OIDC Support

This is a major refactor that allows us to support multiple providers.
Also included is a complete OIDC provider + base class for easily implementing further providers supported by https://golang.org/x/oauth2
Work still needed:

[ ] Tidy up pre-existing provider tests
[ ] Live OIDC testing (ensure unit tests are valid)

Tested with Okta; it is also working fine here.

This is ready to go, I just need to finish the docs (almost done :)

Hi @thomseddon so if I pull the :latest tag from docker for ARM/ Raspberry Pi will these changes be included? so for ARM also it is 2.1 Prod?  thanks!

Thanks, I tested it with MS Azure and it is working flawlessly

can you provide some guidance how you got it to work please? maybe share the docker-compose  ?

Thanks, I tested it with MS Azure and it is working flawlessly

can you provide some guidance how you got it to work please? maybe share the docker-compose ?

Sure, here it is:
  traefik:
    image: traefik
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock
      - ${CONFIG_FOLDER}/traefik:/data
    ports:
      - 80:80
      - 443:443
    extra_hosts:
      - host.docker.internal:<IP_ADDRESS> # Temporary fix until docker fix this mess
    restart: always
    command:
      - --api
      - --providers.file.directory=/data
      - --providers.docker=true
      - --providers.docker.exposedbydefault=false
      - --providers.docker.defaultrule=Host(`{{ normalize .Name | replace "-${COMPOSE_PROJECT_NAME}" ".${DOMAIN}"}}`)
      - --entrypoints.web.address=:80
      - --entrypoints.websecure.address=:443
      - --certificatesresolvers.default.acme.httpchallenge=true
      - --certificatesresolvers.default.acme.httpchallenge.entrypoint=web
      - --certificatesresolvers.default.acme.storage=/data/acme.json
      - --entrypoints.websecure.http.tls.certresolver=default
      - --entrypoints.websecure.http.middlewares=sts
      - --entrypoints.web.http.redirections.entrypoint.to=websecure
    labels:
      - traefik.enable=true
      - traefik.http.routers.traefik.middlewares=auth
      - traefik.http.routers.traefik.entrypoints=websecure
      - traefik.http.routers.traefik.service=api@internal
      # Global settings
      - traefik.http.middlewares.sts.headers.stspreload=true
      - traefik.http.middlewares.sts.headers.stsseconds=31536000

  traefik-forward-auth:
    image: thomseddon/traefik-forward-auth
    environment:
      - DEFAULT_PROVIDER=oidc
      - PROVIDERS_OIDC_ISSUER_URL=${OIDC_ISSUER}
      - PROVIDERS_OIDC_CLIENT_ID=${OIDC_CLIENT_ID}
      - PROVIDERS_OIDC_CLIENT_SECRET=${OIDC_CLIENT_SECRET}
      - SECRET=${TFA_SECRET}
      - COOKIE_DOMAIN=${DOMAIN}
      - AUTH_HOST=traefik-forward-auth.${DOMAIN}
      - WHITELIST=${TFA_WHITELIST}
    labels:
      - traefik.enable=true
      - traefik.http.routers.traefik-forward-auth.middlewares=auth
      - traefik.http.routers.traefik-forward-auth.entrypoints=websecure
      - traefik.http.services.traefik-forward-auth.loadbalancer.server.port=4181
      - traefik.http.middlewares.auth.forwardauth.address=http://traefik-forward-auth:4181
      - traefik.http.middlewares.auth.forwardauth.authresponseheaders=X-Forwarded-User
    restart: always

  portainer:
    image: portainer/portainer
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock
      - portainer:/data
    command: --no-auth
    restart: always
    labels:
      - traefik.enable=true
      - traefik.http.routers.portainer.middlewares=auth
      - traefik.http.routers.portainer.entrypoints=websecure
      - traefik.http.services.portainer.loadbalancer.server.port=9000

$OIDC_ISSUER=OIDC_ISSUER=https://login.microsoftonline.com/9188040d-6c67-4c5b-b112-36a304b66dad/v2.0 - with 9188040d-6c67-4c5b-b112-36a304b66dad being the ID of the tenant (in my case  this is the common one (Live, XBOX & co).
You can use https://docs.microsoft.com/en-us/azure/active-directory/develop/v2-protocols-oidc for reference.

Thank you for sharing it. I am trying to use my office 365 tenant ID but i keep seeing this redirect error in the traefik-forward-Auth log
level=fatal msg="oidc: issuer did not match the issuer returned by provider, expected "https://login.microsoftonline.com/tenant Id " got "https://sts.windows.net/tenant id
Any idea why?


Map colour

Hi, I have a question about the colours on the maps. In the legend, the colour code #FEFFB1 is given for the incidence from 15 to 25, but today the code #D5DDD0 came out for Berlin on the States Map. This caused confusion for one of my users because he could not assign the colour to the legend. However, I have already noticed this problem several times, that colours appear on the map that are not stored in the legend. Is this due to the size of the image, that the colours run in such small districts / federal states or is it due to something else?

Oh yeah, you are right

on my own server with the same quality setting for the png it seems right

what the damn is this?
the only thing what is different is that i am running a newer Version of lib vips............
will give it a try ................

i just debug the original code from here and .............. the result is rigth !? i'm confused now
i#m in investigation

I made a mistake, I have a different quality setting, so it was correct on my server!
Quality Setting of 25 solves the problem! (i am running 50, but to hold the resulting image smal this should be as smal as possible)
Write a PR


ACL examples

Example for getting ACL of bucket for specific user
Example for getting ACL of blob for specific user

@mziccard FYI

@jerjou , you will want to cc me from this point forward.

Coverage increased (+0.09%) to 84.337% when pulling 3f83c1bd5d68133132b121c5198a75b29317bf81 on jerjou:even-more-examples into a187b11f72d91df0b053de9c219c1b5f12211f0f on GoogleCloudPlatform:master.


Dynamic Sections shouldn't show itself if either API doesn't respond or Empty result

On the original site i have seen that block it self said that No Records Found , so I will add question in tracker to replicate this behaviour or remove the block completely.

as per last meeting we can go with "No Records Found" message.


Any expenses visible to you (i.e. SmartScanning) on Android don’t reflect on Chrome Staging

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number:
Reproducible in staging?: Needs reproduction
Reproducible in production?: Needs reproduction
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @jliexpensify
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1720478723447669
Action Performed:

SmartScan expense on Android
See that it shows as visible to you
Open Chrome (staging) and login with same account

Additional steps:

Remove an expense with an error
SmartScan same expense
Verify that it doesn’t show on Chrome (staging) with the visible to you whisper

Expected Result:
The expense would reflect in Chrome as well
Actual Result:
The expenses don’t exist in chrome.
Logged out of the Android app and back in and only 2 of the 5 receipts from the last week got added. The other 3 are missing, and the 2 that got added were added to an Old.Dot Workspace.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[X] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

View all open jobs on GitHub

@jliexpensify I haven't been able to reproduce this issue on my side - the expenses created on Android are showing in Chrome. Can you please give me the specific steps you are taking to create the expense on your Android?

@VictoriaExpensify hmm so this happened at the start of last week across multiple days, but testing the app this week, it's a lot better. I haven't come across this behaviour again since an update (I'm on 9.0.5-13) so lets close this one for now.


Submission failed

Hello,Over the past few days, the progress bar has been stuck at 0% when submitting the result archive file. Could you please advise on how to resolve this issue?
Thank you.

@Lijingze666 We are using AWS for storing submissions. You may need to check whether you have connection issues with AWS the server by checking the Developer Console when clicking the Submit button.


fixed stub/added solution/+link to open terminal

I have provided the solution to the exercise of the curriculum.
For instructions on how to open up the Js console in our web browsers I have provided a link to the FCC's github site (#34251) and I am awaiting the pull request to be accepted . Even if it doesn't get accepted and the link remains a stub leave the link here (even if it is referring to a stub) as in time it will get completed too 👍
Certifications/JavaScript Algorithms and Data Structures/Debugging/Use the JavaScript Console to Check the Value of a Variable

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Really sorry I shall try and see the bug in the weekend+ remake a new pr :)
Get Outlook for Androidhttps://aka.ms/ghei36

From: Aditya<EMAIL_ADDRESS>Sent: Wednesday, March 6, 2019 9:45:27 PM
To: freeCodeCamp/freeCodeCamp
Cc: AlexandreLeFou; Mention
Subject: Re: [freeCodeCamp/freeCodeCamp] fixed stub/added solution/+link to open terminal (#34252)
@ezioda004 commented on this pull request.
Hi @AlexandreLeFouhttps://github.com/AlexandreLeFou,
I'm closing this PR as its gone stale, please feel free to make a new PR in the future.
Happy contributing!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/freeCodeCamp/freeCodeCamp/pull/34252#pullrequestreview-211433805, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Akez6Tr6TAcizbccS6qrtBANPaKmm5D3ks5vUBrXgaJpZM4YTlXs.


[CDS-101042] Fixed Agro rollouts image mix-up

Thanks for contributing to the Harness Developer Hub! Our code owners will review your submission.
Description

Please describe your changes: __________________________________
Jira/GitHub Issue numbers (if any): ______________________________
Preview links/images (Internal contributors only): __________________

PR lifecycle
We aim to merge PRs within one week or less, but delays happen sometimes.
If your PR is open longer than two weeks without any human activity, please tag a code owner in a comment.
PRs must meet these requirements to be merged:

[ ] Successful preview build.
[ ] Code owner review.
[ ] No merge conflicts.
[ ] Release notes/new features docs: Feature/version released to at least one prod environment.

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66eacd1b83a2990a34250749--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66ebc301f8bce62ece49a62f--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66ebc406d113202ba98e0544--harness-developer.netlify.app


Cannot clear path using .attr in Chrome

When trying to change the route of a path, it is tempting to do so as follows.
$myPath.attr({ d: "" });
However, this does not work in Chrome 42.0.2311. It does however work in Firefox 37.0.2
The workaround in this situation is
$myPath.node.setAttribute("d", "");
`

Attempted to create isolated test case but could not reproduce:
http://jsfiddle.net/zq2q80g6/
I tried this on the same version of chrome. Tried Snap v0.3.0 and v0.4.1.
Can you see anything else in your use case that might be affecting this?


Irrelevant "Sending version" message on metadata saving

Describe the bug
When metadata is failing to be saved for a new subset, the "Sending version data to the server" message still shown.
App version
v0.6.*
To Reproduce
Steps to reproduce the behavior:

Create a new subset.
Go to the "Publish" form step.
Switch of the internet on the mashine.
Click the "Save button".
See the error "Failed to fetch" for metadata and "Sending version data to the server" message.

Expected behavior
Either do not show the "Sending version data to the server" message.
Or make the app show failed version saving attempt.
Suggested solution
To not show the "Sending version data to the server" message is ore natural. Probably Message, in that case, has to be "Cannot save the version before the metadata is successfully saved".
FYI: If the metadata saving process filed, the application does not attempt to send the version to the server.

Hello I was wondering if this issue is open for contribution? if so I would love to work on this.

Hello I was wondering if this issue is open for contribution? if so I would love to work on this.


Dynamic and Asynchronous data not work

the Route like  <Route name="UserLocator" component={User} staticName={true} getDisplayName={() => store.getStore().something.foo || '...'} />, but when Asynchronous data is done  the breadcrumb still display
'...', seems that breadcrumbs not re-render

well, the property foo should be connected in the Component so that it will re-render while change in store
: )


bump sqlservernotebook version (#15025)

This PR fixes #15007

Pull Request Test Coverage Report for Build 727498256

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 43.526%

Totals

Change from base Build 727248062:
0.0%

Covered Lines:
26120

Relevant Lines:
54839

💛  - Coveralls


fix babel preset typescript

 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.:x: emersonlaurentino sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Rework Settings Page UI

Checklist

[x] the contributor license agreement is signed
[x] commit message follows commit guidelines
[x] tests are included
[x] screenshots are included showing significant UI changes

Description of change

#1337
Based upon meetings in the Edalex team, this has been redesigned visually. The SettingsList and SettingsListControl components were also made for reuse in the future.
Jest tests as well as TestNG tests (SearchSettingsTest) should still work with these changes.

Hi @edalex-ian, the two builds are green now.  Should I merge on ?

Yup, @edalex-tom said he's happy to proceed, and you seem happy, and it's holding up other work. So, merge on. :+1:


Fix issue with parsing details in PYSEC-2022-42969

Fixes #106
Proposed Changes

Remove offending link/colon from details for PYSEC-2022-42969.

Since it is using the official description from https://nvd.nist.gov/vuln/detail/CVE-2022-42969 and Github is linked as a reference already I removed the Taken from ... from details as it contains the offending colon.

@oliverchang It also looks like this is might be a contested advisory/CVE spam? See https://github.com/pypa/advisory-database/issues/104

@oliverchang It also looks like this is might be a contested advisory/CVE spam? See #104

Unclear if this is definitely spam. My read of the upstream issue sounds like: this is a legitimate issue, but it's a ReDoS so the impact for most people is pretty low and unlikely to be fixed in what is now a maintenance mode project.
We don't really have a precedence for excluding ReDoS in general. Do you see these issues as significantly impacting the usefulness of vuln scanners?

Merging for now, we can continue discussion here or in the issue you linked!

We don't really have a precedence for excluding ReDoS in general. Do you see these issues as significantly impacting the usefulness of vuln scanners?

IMHO, no.

Merging for now, we can continue discussion here or in the issue you linked!

Thank you :)