Scrollable widget has an opaque hitTestBehavior and won't give widgets under it a chance to catch hits

Steps to reproduce

Run the sample app
Tap on a red widget
Tap on a green widget

Expected results
Both red and green widget should catch the tap and you should get these logs in console:
flutter: onTap 1
flutter: onTap 2

Actual results
Only green widgets receive the tap since they're caught by the topmost Scrollable:
flutter: onTap 2

Code sample
Code sample
import 'package:flutter/material.dart';

void main() {
  runApp(const MaterialApp(home: Home()));
}

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

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Stack(
        children: [
          ListView.builder(
            itemBuilder: (c, i) {
              final child = Container(
                height: 100,
                color: i.isEven ? Colors.red : null,
                child: Text('$i'),
              );
              if (i.isEven) {
                return GestureDetector(
                  onTap: () => print('onTap 1'),
                  child: child,
                );
              }
              return child;
            },
          ),
          ListView.builder(
            itemBuilder: (c, i) {
              final child = Container(
                height: 100,
                color: i.isOdd ? Colors.green : null,
                child: Text('$i'),
              );
              if (i.isOdd) {
                return GestureDetector(
                  onTap: () => print('onTap 2'),
                  child: child,
                );
              }
              return child;
            },
          ),
        ],
      ),
    );
  }
}

Screenshots or Video
No response
Logs
No response
Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.19.3, on macOS 14.4.1 23E224 darwin-arm64, locale
    en-US)
    • Flutter version 3.19.3 on channel stable at /Users/amirpanahandeh/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision ba39319843 (4 weeks ago), 2024-03-07 15:22:21 -0600
    • Engine revision 2e4ba9c6fb
    • Dart version 3.3.1
    • DevTools version 2.31.1

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/amirpanahandeh/SDK
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/amirpanahandeh/SDK
    • Java binary at: /Applications/Android
      Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      17.0.7+0-17.0.7b1000.6-10550314)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15E204a
    • CocoaPods version 1.14.3

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

[✓] Android Studio (version 2023.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      17.0.7+0-17.0.7b1000.6-10550314)

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

[✓] Connected device (4 available)
    • Amir’s iPhone (mobile)     • 00008101-00052C14362A001E            • ios
      • iOS 17.4.1 21E236
    • iPhone 15 Pro Max (mobile) • 152C5F7B-B91F-4787-A67F-54A919478013 • ios
      • com.apple.CoreSimulator.SimRuntime.iOS-17-4 (simulator)
    • macOS (desktop)            • macos                                •
      darwin-arm64   • macOS 14.4.1 23E224 darwin-arm64
    • Chrome (web)               • chrome                               •
      web-javascript • Google Chrome 119.0.6045.123

[✓] Network resources
    • All expected network resources are available.

• No issues found!

Looks like changing RawGestureDetector's behavior from HitTestBehavior.opaque to HitTestBehavior.translucent in ScrollableState solves the issue. I'm not fully sure about the side effects tho.

Thanks for the report and potential fix. I was able to replicate the reported behavior.

Looks like changing RawGestureDetector's behavior from HitTestBehavior.opaque to HitTestBehavior.translucent in ScrollableState solves the issue.

https://github.com/flutter/flutter/blob/4cfa5e266436d317da58d121691627979e8aa83a/packages/flutter/lib/src/widgets/scrollable.dart#L974

 stable, master flutter doctor -v
[!] Flutter (Channel stable, 3.19.5, on macOS 12.2.1 21D62 darwin-x64, locale
    en-GB)
    • Flutter version 3.19.5 on channel stable at
      /Users/dhs/documents/fluttersdk/flutter
    ! Warning: `flutter` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/flutter, which is not inside
      your current Flutter SDK checkout at
      /Users/dhs/documents/fluttersdk/flutter. Consider adding
      /Users/dhs/documents/fluttersdk/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/dhs/documents/fluttersdk/flutter.
      Consider adding /Users/dhs/documents/fluttersdk/flutter/bin to the front
      of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 300451adae (4 days ago), 2024-03-27 21:54:07 -0500
    • Engine revision e76c956498
    • Dart version 3.3.3
    • DevTools version 2.31.1
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

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

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

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

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

! Doctor found issues in 1 category.

[!] Flutter (Channel master, 3.22.0-5.0.pre.55, on macOS 12.2.1 21D62
    darwin-x64, locale en-GB)
    • Flutter version 3.22.0-5.0.pre.55 on channel master at
      /Users/dhs/documents/fluttersdk/flutter
    ! Warning: `flutter` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/flutter, which is not inside
      your current Flutter SDK checkout at
      /Users/dhs/documents/fluttersdk/flutter. Consider adding
      /Users/dhs/documents/fluttersdk/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/dhs/documents/fluttersdk/flutter.
      Consider adding /Users/dhs/documents/fluttersdk/flutter/bin to the front
      of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision ddf79b755d (6 hours ago), 2024-04-07 18:34:24 -0400
    • Engine revision c10e950c66
    • Dart version 3.5.0 (build 3.5.0-18.0.dev)
    • DevTools version 2.34.1
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/dhs/Library/Android/sdk
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for
      more details.

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

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

[✓] IntelliJ IDEA Ultimate Edition (version 2021.3.2)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 65.1.4
    • Dart plugin version 213.7228

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

[✓] Connected device (3 available)
    • Darshan's iphone (mobile) • 21150b119064aecc249dfcfe05e259197461ce23 • ios
      • iOS 15.3.1 19D52
    • macOS (desktop)           • macos                                    •
      darwin-x64     • macOS 12.2.1 21D62 darwin-x64
    • Chrome (web)              • chrome                                   •
      web-javascript • Google Chrome 109.0.5414.119

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

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

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

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

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

! Doctor found issues in 1 category.

I think this issue as well as

https://github.com/flutter/flutter/issues/92119
and https://github.com/flutter/flutter/issues/145330

are the same thing.
There are currently two different proposals out to resolve them:

https://github.com/flutter/flutter/pull/145848
https://github.com/flutter/flutter/pull/146403

Why is the expected result to engage with both layers when this is proper behavior? Imagine stacking 2 pieces of paper on top of each other (stack) you stack 2 lists on top of each other, both extend to full screen size. You have colored and transparent containers but the containers are still there. They even have numbers. Layer with red containers is not accessible because it's under a full screen of a different widget.
If touching a transparent container should be omitted then why is it on the screen in the first place?

Why is the expected result to engage with both layers when this is proper behavior? Imagine stacking 2 pieces of paper on top of each other (stack) you stack 2 lists on top of each other, both extend to full screen size. You only touch the first piece of paper at all times. The pieces of paper do not have holes (as provided in example)
You have colored and transparent containers but the containers are still there. They even have numbers. Layer with red containers is not accessible because it's under a full screen of a different widget.

The example code provided here is just a minimal reproducible code for the problem. In the original project I encountered the issue, it's more complicated than that and in fact, it's a two-dimensional scrollable with gaps between items that underlying scrollable going to fill and needs to be hit testable. @RelappsStudio

Could you please provide the actual example, or as close as you can get to it? I'm actually curious how 2 layers are being intertwined with each other on separate scrollers. Also would greatly help in finding a proper solution @Amir-P

Could you please provide the actual example, or as close as you can get to it? I'm actually curious how 2 layers are being intertwined with each other on separate scrollers. Also would greatly help in finding a proper solution @Amir-P

I'll try to post a code close to my real case if I find the time. @RelappsStudio
Regarding the solution, I think having a configurable 'hitTestBehavior' for 'Scrollable's is not a bad idea. It doesn't change the current behavior amd provides more control for special use cases like the one I have.


[TC-115] make sure ats config files are owned by ats.ats

make sure ats config files are owned by ats.ats, which makes the config files be more friendly to ATS. This is because if the owner of a config file is not ats.ats, ATS sometimes does not reload the config while receiving a "traffic_line -x".
Issue at https://issues.apache.org/jira/browse/TC-115

@dg4prez Thanks for the comments, revised accordingly.

Can one of the admins verify this patch?

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/43/

Can one of the admins verify this patch?

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR-trafficops-test/32/

GH issue #1015

It looks like you're only filtering out cron files.  That's a little specific - can you filter out anything not in the /opt/trafficserver directory?

@dg4prez revised accordingly, please help to check, thanks.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/561/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR-trafficops-test/517/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/828/
Test PASSed.

@smalenfant revised accordingly.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/829/
Test PASSed.

I believe this has already been implemented in traffic_ops_ort.pl - though the difference seems to be that this PR would make all files that don't fall under /opt/trafficserver owned by root whereas currently only those files with /cron/ in the pathname will be made owned by root and everything else is owned by ATS.
If the current implementation is acceptable, please close this PR.


Leaftlet providers and plugins

Hi !
I'd like to know how to use other providers and some plugins. For example I'd like to use the following :
https://github.com/leaflet-extras/leaflet-providers (to use HERE for example).
And bouncing from the following plugin :
https://github.com/hosuaby/Leaflet.SmoothMarkerBouncing
Thanks a lot !!

@paparent all the plugins that we are aware of are in the docs section if they are missing feel free to create one and make a PR to add them to our docs!

@DonNicoJs Hi
Yeah sure I can develop one ! But those two seemed a bit more complex (maybe, because I've never played with leaflet and vue-leaflet). So wanted input from other how easy is to implement that!

@paparent the best thing to do is to keep in consideration those points:

leaflet exposes utility that you can leverage findRealParent and propsBinder
everything should be initialised in the mounted lifecycle hook
remember to set leaflet as a peerDependecy
rollup is the best library bundler so you can leverage ESM ( you can check how we do it here in the main repo )
if you want you can add me to the repo so I can keep an eye on the code, but I can't promise to be super fast because I am quite busy as it is :)
check this repo or one of the other plugins for inspiration ( for example leaflet-markercluster )

For the rest is easy to implement, a bit less easy to do it smoothly :) but I am here to help

Labelling as plugin-request and closing, feel free to re-open if you have more questions

any tutorial on creating a plugin?

@gelinger777 not that I am aware of, take a look at other plugins and take in consideration what I listed above

Any Ideas how to use this one?
https://github.com/mapshakers/leaflet-mapkey-icon  ?
As I understood  would need to write a wrapper, for that, am I right?

@gelinger777 you don't need any plugin for this one, just import css and js and assign the icon to LMarker


why do you use envoy

Hi, i am reading your project and i love it, can i ask u a question, why do you use Envoy in your project. Can you tell me the purpose, the pros and cons of this. thank u so much

Hi,
First of all, it is just a project for demonstration or POC purposes of the GRPC capabilities/limitations.
Regarding your question, since at the moments, browsers does not support GRPC, we must use the Envoy proxy for that. There is no other way to call with javascript  using grpc from the browser to server. What envoy does is translating the REST call to grpc. In the future when this capability will be implemented, the java script API code won't have to be changed, just to get recompiled, all the calls to it will remain the same.


null text in notification

Lets switch out the the name for the handle of the user, and remove the channel name from the message.

Lets switch out the the name for the handle of the user, and remove the channel name from the message.


Requires-Python >=3.10,<4.0

What about python 3.11 3.12 3.13?
Thanks

Python 3.7 was EOL June last year (2023). So you'll increasingly find it hard to get support for libraries now. Especially anything based on python typing.
The 3.10 limit is primarily for type hinting using pep 604 unions. However this library heavily depends on PEP 593 Annotated so I couldn't make it support < 3.9.

FYI unit tests suggest this is working for 3.11 and 3.12.
3.13 is scheduled to be released in October 2024. I can look to enable testing for that pretty close to python release date.
If you find bugs with 3.13 support before that you can report here and I'd investigate.


Reduce usage of direct dependencies

Summary

Change Type

Check List Before Merging

[ ] This PR passes all pre-commit hook validations.
[ ] This PR is fully tested and regression tests are included.
[ ] Relevant documentation has been updated.

Additional Information

talked to josh via side channel: he approves so I "self approve" here


Update leaderboard results (Streaming)

Per conversation in #286, separate streaming leaderboard update from the original ScaNN PR.

I just finished re-running every single streaming algorithm with the corrected recall computation, and my results agree with yours:
cufe,"diskann(('R32_L50', {'Ls': 70, 'T': 16}))",msturing-30M-clustered(final_runbook.yaml),10,0.0,1.1743693351745605,4886392.0,0.0,0.0,streaming,0.584525296875
cufe,"diskann(('R32_L70', {'Ls': 70, 'T': 16}))",msturing-30M-clustered(final_runbook.yaml),10,0.0,1.3363685607910156,4885996.0,0.0,0.0,streaming,0.6280998906249999
cufe,"diskann(('R50_L50', {'Ls': 70, 'T': 16}))",msturing-30M-clustered(final_runbook.yaml),10,0.0,1.5138330459594727,4885584.0,0.0,0.0,streaming,0.6481847187499999
diskann,"diskann(('R32_L50', {'Ls': 70, 'T': 16}))",msturing-30M-clustered(final_runbook.yaml),10,0.0,1.0090949535369873,4888048.0,0.0,0.0,streaming,0.664196796875
diskann,"diskann(('R32_L70', {'Ls': 70, 'T': 16}))",msturing-30M-clustered(final_runbook.yaml),10,0.0,1.0452346801757812,4887084.0,0.0,0.0,streaming,0.7042683281250001
diskann,"diskann(('R50_L50', {'Ls': 70, 'T': 16}))",msturing-30M-clustered(final_runbook.yaml),10,0.0,1.058459997177124,4885896.0,0.0,0.0,streaming,0.72150353125
hwtl_sdu_anns_stream,"HWTL_SDU_ANNS_stream(('R65_L70', {'Ls': 100, 'T': 16}))",msturing-30M-clustered(final_runbook.yaml),10,0.0,1.0412211418151855,4887040.0,0.0,0.0,streaming,0.769358078125
pinecone,"pinecone(('R32_L100', {'Ls': 300, 'k_1': 30, 'T': 8}))",msturing-30M-clustered(final_runbook.yaml),10,0.0,0.44964075088500977,2010136.0,0.0,0.0,streaming,0.8792984374999999
pinecone,"pinecone(('R32_L100', {'Ls': 400, 'k_1': 30, 'T': 8}))",msturing-30M-clustered(final_runbook.yaml),10,0.0,0.5977978706359863,2010196.0,0.0,0.0,streaming,0.8977126875
pinecone,"pinecone(('R32_L100', {'Ls': 500, 'k_1': 30, 'T': 8}))",msturing-30M-clustered(final_runbook.yaml),10,0.0,0.4760725498199463,2010188.0,0.0,0.0,streaming,0.9104433437499999
pinecone,"pinecone(('R32_L100', {'Ls': 520, 'k_1': 30, 'T': 8}))",msturing-30M-clustered(final_runbook.yaml),10,0.0,0.4755425453186035,2012272.0,0.0,0.0,streaming,0.9120122343750001
puck,"Puck('C200_F200_FN8_Flat_filter_topk1200', {'radius_rate': 1.0, 'search_coarse_count': 200})",msturing-30M-clustered(final_runbook.yaml),10,0.0,0.0420689582824707,162020.0,0.0,0.0,streaming,0.08013546875000001
puck,"Puck('C200_F200_FN8_Flat_filter_topk1800', {'radius_rate': 1.0, 'search_coarse_count': 200})",msturing-30M-clustered(final_runbook.yaml),10,0.0,0.04240989685058594,162484.0,0.0,0.0,streaming,0.080622875
puck,"Puck('C200_F200_FN8_Flat_filter_topk1500', {'radius_rate': 1.0, 'search_coarse_count': 200})",msturing-30M-clustered(final_runbook.yaml),10,0.0,0.0414884090423584,162152.0,0.0,0.0,streaming,0.08365264062500001
puck,"Puck('C200_F200_FN8_Flat_filter_topk1900', {'radius_rate': 1.0, 'search_coarse_count': 200})",msturing-30M-clustered(final_runbook.yaml),10,0.0,0.042151689529418945,162104.0,0.0,0.0,streaming,0.08497718750000001
puck,"Puck('C200_F200_FN8_Flat_filter_topk2300', {'radius_rate': 1.0, 'search_coarse_count': 200})",msturing-30M-clustered(final_runbook.yaml),10,0.0,0.04342079162597656,162112.0,0.0,0.0,streaming,0.08687278125
puck,"Puck('C200_F200_FN8_Flat_filter_topk2100', {'radius_rate': 1.0, 'search_coarse_count': 200})",msturing-30M-clustered(final_runbook.yaml),10,0.0,0.042818546295166016,162408.0,0.0,0.0,streaming,0.0902274375
puck,"Puck('C200_F200_FN8_Flat_filter_topk2200', {'radius_rate': 1.0, 'search_coarse_count': 200})",msturing-30M-clustered(final_runbook.yaml),10,0.0,0.046465158462524414,162116.0,0.0,0.0,streaming,0.09184689062499998
puck,"Puck('C200_F200_FN8_Flat_filter_topk2000', {'radius_rate': 1.0, 'search_coarse_count': 200})",msturing-30M-clustered(final_runbook.yaml),10,0.0,0.0447077751159668,162460.0,0.0,0.0,streaming,0.09214789062499999
pyanns,"pyanns(('R32_L100', {'Ls': 300, 'T': 8}))",msturing-30M-clustered(final_runbook.yaml),10,0.0,0.5063211917877197,1994584.0,0.0,0.0,streaming,0.8696431718750002
pyanns,"pyanns(('R32_L100', {'Ls': 400, 'T': 8}))",msturing-30M-clustered(final_runbook.yaml),10,0.0,0.5399785041809082,1992652.0,0.0,0.0,streaming,0.8865686875
scann,"ScaNN,tree=700/5000,AH2,reorder=317",msturing-30M-clustered(final_runbook.yaml),10,0.0,0.0023641586303710938,1024.0,0.0,0.0,streaming,0.9924002968749999
zilliz,"zilliz(('R32_L110', {'Ls': 400, 'T': 8}))",msturing-30M-clustered(final_runbook.yaml),10,0.0,0.4923396110534668,1996968.0,0.0,0.0,streaming,0.905939296875
zilliz,"zilliz(('R32_L110', {'Ls': 450, 'T': 8}))",msturing-30M-clustered(final_runbook.yaml),10,0.0,0.490079402923584,1997400.0,0.0,0.0,streaming,0.91261615625
zilliz,"zilliz(('R32_L110', {'Ls': 500, 'T': 8}))",msturing-30M-clustered(final_runbook.yaml),10,0.0,0.48936939239501953,1997708.0,0.0,0.0,streaming,0.917919640625
zilliz,"zilliz(('R32_L110', {'Ls': 550, 'T': 8}))",msturing-30M-clustered(final_runbook.yaml),10,0.0,0.4806337356567383,1996332.0,0.0,0.0,streaming,0.9222139687499998
If you are fine with this I will ask Harsha to publish the website updates.

Thanks for confirming the results :)
It will be great if Harsha can publish the website update.

Thanks a lot! Let me know how it goes with the OOD track.

@nk2014yj @ZiaddAhmedd @KhylonWong @veaaaab @ingberam @hhy3 please note the updated leaderboard. With PR #280 we realized that the streaming recall was being computed incorrectly due to a caching bug. For the NeurIPS23 competition entrants, we will publish both the re-calculated leaderboard and the original leaderboard. We apologize for the error.


Disallow video push_to_hub

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Add cryptographically secure pseudo-random generator to dart:io.

Either fix or document this.

There is no longer a dart:crypto library.  It has been moved to package:crypto.
A secure RNG should be provided by the embedder, so it should be made available in
dart:io for the standalone Dart executable, and in dart:html for the browser.
Dart:html already supports it in Crypto.getRandomValues, so this is now just a dart:io issue.
Labelling this issue as dart:io, and merging duplicate issues with it.
Duplicate issues, or blocked issues, are #15993 and #15064

cc @crelier

Discussion I had with @sgjesse and @lrhn was that the VM team will take ownership of this one and expose it as "sec = new Random.secure();" or similar.
The general idea being that you start relying on the embedder (e.g. standalone VM, browser) to provide a secure random number source.

Greatness. Thanks @crelier !

Actually, @crelier would you might submitting a quick patch to CHANGELOG.md indicating this new API landed? Thanks!


mxnet compile failed

System configuration:Ubuntu14.04,cuda8,cudnn5
Error info when compiling mxnet:
/usr/include/c++/4.8/bits/stl_vector.h:919:7: note:   no known conversion for argument 1 from ‘nnvm::dim_t* {aka long int*}’ to ‘unsigned int*&&’
make: *** [build/src/operator/custom/custom.o] Error 1
Does anyone know how to fix it?

Please use gcc4.9 or higher. I suspect gcc4.8 is too old.


Bump version and get rid off .travis.yml file

Signed-off-by: Petr "Stone" Hracek<EMAIL_ADDRESS>

Let's test


Upgrade to v1.14.1

I am just trying to build according to https://github.com/cortexproject/cortex/blob/v1.14.1/go.mod#L216-L241
I will try your suggestions, let me double check them

yeah, no

vendor/github.com/weaveworks/common/httpgrpc/server/server.go:137:8: undefined: grpc.WithBalancerName
that is because of the grpc upgrade. I will revert


[Bug]: Google.EarthPro Installed package type is not compatible with exe

Brief description of your issue
Trying to update Google.EarthPro but it reports as wrong InstallerType.
WinGetCreate cannot detect the InstallerType
InstallerTypes:
Exe: Installed package type is not compatible with exe WinGet-Google.EarthPro.<IP_ADDRESS>48-2022-01-07-12-55-02.800.log
Burn: Installed package type is not compatible with burn
Wix: Installer failed with exit code: 1620
MSI: Installer failed with exit code: 1620
Inno: Installed package type is not compatible with inno
Nullsoft: Installed package type is not compatible with nullsoft
Steps to reproduce
❯ winget install Google.EarthPro --version <IP_ADDRESS>86
Found Google Earth Pro [Google.EarthPro] Version <IP_ADDRESS>86
This application is licensed to you by its owner.
Microsoft is not responsible for, nor does it grant any licenses to, third-party packages.
Downloading https://dl.google.com/dl/earth/client/advanced/current/googleearthprowin-7.3.3-x64.exe
  ██████████████████████████████  65.4 MB / 65.4 MB
Successfully verified installer hash
Starting package install...
Successfully installed

❯ winget upgrade Google.EarthPro
A newer version was found, but the install technology is different from the current version installed. Please uninstall the package and install the newer version.

Expected behavior
To update Google Earth
Actual behavior
See Steps to reproduce
Environment
❯ winget --info
Windows Package Manager (Preview) v1.2.3411-preview
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.22000.318
Package: Microsoft.DesktopAppInstaller v1.17.3411.0

and

winget --info
Windows Package Manager v1.1.12653
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.22000.318
Package: Microsoft.DesktopAppInstaller v1.16.12653.0

I think it may be a case of an MSI installer packaged inside an EXE.
If you convert this to schema 1.1.0 then you can possibly use @jedieaston's solution from https://github.com/microsoft/winget-cli/pull/1796#issuecomment-993548211

One step ahead of you, however me and his script do not like eachother https://github.com/microsoft/winget-pkgs/discussions/38882#discussioncomment-1924220

Edit: however i do not know why i didn’t add it manually
Me on most days of the week

Oh i was one step ahead of myself too, i was going to use jedieaston's script to find the InstallerType, but i didn't add the AppsAndFeaturesEntries because they don't do anything in winget v1.2.3411-preview

I can't detect inner InstallerType with that script (yet), you'll have to add that manually :(


fix: add py.typed to fix mypy cannot find module error

Fix mypy error: Cannot find module named 'dicomweb_client'

+1 for this. Without this file, code using this library will not use the type hints within it for type checking: see https://mypy.readthedocs.io/en/latest/installed_packages.html


Doubts about the app

Hello ekke
This are not real issues, just some doubts:
1 - I've noticed that you use 3 languages en fr and german. My question is: How does the user selects the language he wants? Or a language is selected in the code and the app is released with that language?
2- About the update of the app. Let's suppose the conference program changes during the event. How does (if so) the app deals with that. The user needs to update the application in google play?
3 - How do you manage the json files with the conference content (speakers, schedule, ...) ? You create an excel (or some database) and then converte it to json, or you type directly the json?
Sorry for the inconvenience of so much questions, but this may also help others I think.
Danke im Voraus
Ricardo
Lisbon, Portugal

Hi Ricardo,
will come back to your questions on the weekend - just deep in customer project

thanks a lot. no worries
On Tue, Nov 12, 2019 at 10:32 AM ekke<EMAIL_ADDRESS>wrote:

Hi Ricardo,
will come back to your questions on the weekend - just deep in customer
project
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ekke/c2gQtWS_x/issues/6?email_source=notifications&email_token=AHIREUDQBOU7RSC6NG2CENLQTKA5LA5CNFSM4JMBIZN2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDZZLLY#issuecomment-552834479,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHIREUF3UIHFAWVZSFP4UFTQTKA5LANCNFSM4JMBIZNQ
.

--
Ricardo Santos

Hi Ricardo,
for the languages:
app uses the language from device, if not found: uses english
update conference schedule:
there's a Timer - see main.qml line #570 - checks every hour with the server if version changed and if yes pops up a dialog to update the data
conference content: I'm getting this as JSON from server api (see dataserver.cpp), then cache this as JSON files

thank you ekke
I will look into it
regards
On Fri, Nov 15, 2019 at 1:13 PM ekke<EMAIL_ADDRESS>wrote:

Hi Ricardo,
for the languages:
app uses the language from device, if not found: uses english
update conference schedule:
there's a Timer - see main.qml line #570 - checks every hour with the
server if version changed and if yes pops up a dialog to update the data
conference content: I'm getting this as JSON from server api (see
dataserver.cpp), then cache this as JSON files
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ekke/c2gQtWS_x/issues/6?email_source=notifications&email_token=AHIREUGQW6H37QNJADKLQ2TQT2N7FA5CNFSM4JMBIZN2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEEFMI5Y#issuecomment-554353783,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHIREUC62MEHIW2NOLJ7DC3QT2N7FANCNFSM4JMBIZNQ
.

--
Ricardo Santos

2 more basic questions.
In most pages, pressing the back key in the android phone (<) always open
the left nav bar, which is odd to me. How can I make it to go to the home
page instead?
And How can I quit the app, while being in homepage, pressing the back key
2 times.
regards
On Fri, Nov 15, 2019 at 2:14 PM Ricardo Santos<EMAIL_ADDRESS>wrote:

thank you ekke
I will look into it
regards
On Fri, Nov 15, 2019 at 1:13 PM ekke<EMAIL_ADDRESS>wrote:

Hi Ricardo,
for the languages:
app uses the language from device, if not found: uses english
update conference schedule:
there's a Timer - see main.qml line #570 - checks every hour with the
server if version changed and if yes pops up a dialog to update the data
conference content: I'm getting this as JSON from server api (see
dataserver.cpp), then cache this as JSON files
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ekke/c2gQtWS_x/issues/6?email_source=notifications&email_token=AHIREUGQW6H37QNJADKLQ2TQT2N7FA5CNFSM4JMBIZN2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEEFMI5Y#issuecomment-554353783,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHIREUC62MEHIW2NOLJ7DC3QT2N7FANCNFSM4JMBIZNQ
.

--
Ricardo Santos

--
Ricardo Santos

@Ricardosgeral in main.qml
Keys.onBackPressed:
simply remove
if(initDone && navigationBar.position === 0) {
                openNavigationBar()
                return
            }

that always go to the exit page. I wanted to go to the home page and only
go to exit page when in the home page.
tried this but did not work
        if(initDone && HomePage.position === 0) {

            openHomePage()

            return

        }

and
function openHomePage() {
    HomePage.open()

}

On Fri, Nov 15, 2019 at 2:45 PM ekke<EMAIL_ADDRESS>wrote:

@Ricardosgeral https://github.com/Ricardosgeral in main.qml
Keys.onBackPressed:
simply remove
if(initDone && navigationBar.position === 0) {
openNavigationBar()
return
}
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ekke/c2gQtWS_x/issues/6?email_source=notifications&email_token=AHIREUDUGJGNNGTHYBZED33QT2Y2BA5CNFSM4JMBIZN2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEEFUHZQ#issuecomment-554386406,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHIREUABYRXT4OGG4UMDM2LQT2Y2BANCNFSM4JMBIZNQ
.

--
Ricardo Santos

@Ricardosgeral
(untested) something like this should do the trick:
if (navigationIndex !== homeNavigationIndex) {
	navigationIndex = homeNavigationIndex
	return
}

but sorry - I have to close this issue.
If you need consulting about app architecture and details of App development you can book me ;-)

yes it worked. Sorry I'm no developer. I know some things. Just a Civil
Engineering trying to makean app for a conference :D
I understand your point ;)
Do you have a way for me to pay you for your valuable work so far. You
should consider to put a donate button in your wordpress site.
regards
On Fri, Nov 15, 2019 at 4:03 PM ekke<EMAIL_ADDRESS>wrote:

Closed #6 https://github.com/ekke/c2gQtWS_x/issues/6.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ekke/c2gQtWS_x/issues/6?email_source=notifications&email_token=AHIREUFJWGFXUFLTESQE5OLQT3B5XA5CNFSM4JMBIZN2YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOU4KMWLQ#event-2803157806,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHIREUD3DBZ45NGSPIRXX2TQT3B5XANCNFSM4JMBIZNQ
.

--
Ricardo Santos

no problems @Ricardosgeral


pip dependency conflicts: conda-repo-cli 1.0.20 requires nbformat==5.4.0, but you have nbformat 5.7.3 which is incompatible.

I was running this code when I got the dependency error
!pip install clyent==1.2.1
!pip install nbformat==5.4.0
!pip install pycoingecko
!pip install plotly
!pip install mplfinance
!pip install --upgrade nbformat

The error was:
Collecting clyent==1.2.1
  Downloading clyent-1.2.1.tar.gz (20 kB)
  Preparing metadata (setup.py): started
  Preparing metadata (setup.py): finished with status 'done'
Building wheels for collected packages: clyent
  Building wheel for clyent (setup.py): started
  Building wheel for clyent (setup.py): finished with status 'done'
  Created wheel for clyent: filename=clyent-1.2.1-py3-none-any.whl size=9192 sha256=5bb63874dd4f9cfb3c94bc14d08cb24a84efcc95dd0d3433c399d50670801757
  Stored in directory: c:\users\anni.sanskrati jain\appdata\local\pip\cache\wheels\36\00\a6\496d95012b21c0d7b1c980147f150246fc122768aaa222491f
Successfully built clyent
Installing collected packages: clyent
  Attempting uninstall: clyent
    Found existing installation: clyent 1.2.2
    Uninstalling clyent-1.2.2:
      Successfully uninstalled clyent-1.2.2
Successfully installed clyent-1.2.1
ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behavior is the source of the following dependency conflicts.
conda-repo-cli 1.0.20 requires nbformat==5.4.0, but you have nbformat 5.7.3 which is incompatible.
Collecting nbformat==5.4.0
  Downloading nbformat-5.4.0-py3-none-any.whl (73 kB)
     ---------------------------------------- 73.3/73.3 kB 1.3 MB/s eta 0:00:00
Requirement already satisfied: jsonschema>=2.6 in c:\users\anni.sanskrati jain\anaconda3\lib\site-packages (from nbformat==5.4.0) (4.16.0)
Requirement already satisfied: traitlets>=5.1 in c:\users\anni.sanskrati jain\anaconda3\lib\site-packages (from nbformat==5.4.0) (5.1.1)
Requirement already satisfied: fastjsonschema in c:\users\anni.sanskrati jain\anaconda3\lib\site-packages (from nbformat==5.4.0) (2.16.2)
Requirement already satisfied: jupyter-core in c:\users\anni.sanskrati jain\anaconda3\lib\site-packages (from nbformat==5.4.0) (4.11.1)
Requirement already satisfied: attrs>=17.4.0 in c:\users\anni.sanskrati jain\anaconda3\lib\site-packages (from jsonschema>=2.6->nbformat==5.4.0) (21.4.0)
Requirement already satisfied: pyrsistent!=0.17.0,!=0.17.1,!=0.17.2,>=0.14.0 in c:\users\anni.sanskrati jain\anaconda3\lib\site-packages (from jsonschema>=2.6->nbformat==5.4.0) (0.18.0)
Requirement already satisfied: pywin32>=1.0 in c:\users\anni.sanskrati jain\anaconda3\lib\site-packages (from jupyter-core->nbformat==5.4.0) (302)
Installing collected packages: nbformat
  Attempting uninstall: nbformat
    Found existing installation: nbformat 5.7.3
    Uninstalling nbformat-5.7.3:
      Successfully uninstalled nbformat-5.7.3
Successfully installed nbformat-5.4.0
Requirement already satisfied: pycoingecko in c:\users\anni.sanskrati jain\anaconda3\lib\site-packages (3.1.0)
Requirement already satisfied: requests in c:\users\anni.sanskrati jain\anaconda3\lib\site-packages (from pycoingecko) (2.28.1)
Requirement already satisfied: charset-normalizer<3,>=2 in c:\users\anni.sanskrati jain\anaconda3\lib\site-packages (from requests->pycoingecko) (2.0.4)
Requirement already satisfied: urllib3<1.27,>=1.21.1 in c:\users\anni.sanskrati jain\anaconda3\lib\site-packages (from requests->pycoingecko) (1.26.11)
Requirement already satisfied: certifi>=2017.4.17 in c:\users\anni.sanskrati jain\anaconda3\lib\site-packages (from requests->pycoingecko) (2022.9.14)
Requirement already satisfied: idna<4,>=2.5 in c:\users\anni.sanskrati jain\anaconda3\lib\site-packages (from requests->pycoingecko) (3.3)
Requirement already satisfied: plotly in c:\users\anni.sanskrati jain\anaconda3\lib\site-packages (5.9.0)
Requirement already satisfied: tenacity>=6.2.0 in c:\users\anni.sanskrati jain\anaconda3\lib\site-packages (from plotly) (8.0.1)
Requirement already satisfied: mplfinance in c:\users\anni.sanskrati jain\anaconda3\lib\site-packages (0.12.9b7)
Requirement already satisfied: matplotlib in c:\users\anni.sanskrati jain\anaconda3\lib\site-packages (from mplfinance) (3.5.2)
Requirement already satisfied: pandas in c:\users\anni.sanskrati jain\anaconda3\lib\site-packages (from mplfinance) (1.4.4)
Requirement already satisfied: python-dateutil>=2.7 in c:\users\anni.sanskrati jain\anaconda3\lib\site-packages (from matplotlib->mplfinance) (2.8.2)
Requirement already satisfied: pyparsing>=2.2.1 in c:\users\anni.sanskrati jain\anaconda3\lib\site-packages (from matplotlib->mplfinance) (3.0.9)
Requirement already satisfied: pillow>=6.2.0 in c:\users\anni.sanskrati jain\anaconda3\lib\site-packages (from matplotlib->mplfinance) (9.2.0)
Requirement already satisfied: kiwisolver>=1.0.1 in c:\users\anni.sanskrati jain\anaconda3\lib\site-packages (from matplotlib->mplfinance) (1.4.2)
Requirement already satisfied: cycler>=0.10 in c:\users\anni.sanskrati jain\anaconda3\lib\site-packages (from matplotlib->mplfinance) (0.11.0)
Requirement already satisfied: numpy>=1.17 in c:\users\anni.sanskrati jain\anaconda3\lib\site-packages (from matplotlib->mplfinance) (1.21.5)
Requirement already satisfied: fonttools>=4.22.0 in c:\users\anni.sanskrati jain\anaconda3\lib\site-packages (from matplotlib->mplfinance) (4.25.0)
Requirement already satisfied: packaging>=20.0 in c:\users\anni.sanskrati jain\anaconda3\lib\site-packages (from matplotlib->mplfinance) (21.3)
Requirement already satisfied: pytz>=2020.1 in c:\users\anni.sanskrati jain\anaconda3\lib\site-packages (from pandas->mplfinance) (2022.1)
Requirement already satisfied: six>=1.5 in c:\users\anni.sanskrati jain\anaconda3\lib\site-packages (from python-dateutil>=2.7->matplotlib->mplfinance) (1.16.0)
Requirement already satisfied: nbformat in c:\users\anni.sanskrati jain\anaconda3\lib\site-packages (5.4.0)
Collecting nbformat
  Using cached nbformat-5.7.3-py3-none-any.whl (78 kB)
Requirement already satisfied: jupyter-core in c:\users\anni.sanskrati jain\anaconda3\lib\site-packages (from nbformat) (4.11.1)
Requirement already satisfied: traitlets>=5.1 in c:\users\anni.sanskrati jain\anaconda3\lib\site-packages (from nbformat) (5.1.1)
Requirement already satisfied: jsonschema>=2.6 in c:\users\anni.sanskrati jain\anaconda3\lib\site-packages (from nbformat) (4.16.0)
Requirement already satisfied: fastjsonschema in c:\users\anni.sanskrati jain\anaconda3\lib\site-packages (from nbformat) (2.16.2)
Requirement already satisfied: attrs>=17.4.0 in c:\users\anni.sanskrati jain\anaconda3\lib\site-packages (from jsonschema>=2.6->nbformat) (21.4.0)
Requirement already satisfied: pyrsistent!=0.17.0,!=0.17.1,!=0.17.2,>=0.14.0 in c:\users\anni.sanskrati jain\anaconda3\lib\site-packages (from jsonschema>=2.6->nbformat) (0.18.0)
Requirement already satisfied: pywin32>=1.0 in c:\users\anni.sanskrati jain\anaconda3\lib\site-packages (from jupyter-core->nbformat) (302)
Installing collected packages: nbformat
  Attempting uninstall: nbformat
    Found existing installation: nbformat 5.4.0
    Uninstalling nbformat-5.4.0:
      Successfully uninstalled nbformat-5.4.0
Successfully installed nbformat-5.7.3
ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
conda-repo-cli 1.0.20 requires nbformat==5.4.0, but you have nbformat 5.7.3 which is incompatible.

I was using

Python3
Jupyter Notebook
Window 11

In my experience, errors like this can usually be ignored. The installation of TuriCreate is probably fine. Try using TuriCreate in this environment.


Question about decision...

Hi, your script is amazing but i don't find anything about decisions in the form, like change the question based on the previous anwser... so did you can help on that?
Thank's for your time!

Hi Allan, I'm glad that you have found this but the documentation is still in draft.
For your question in changing the question based on previous answer, do you mean the option previousResponsePattern?
(1) If yes, it allows you to insert user's previous answer into the next question. Use case:
<input type="text" name ="name" placeholder="Full Name">
<input type="text" fm-questions="Hi {{previousResponse}}. Let me know your gender!">

The default pattern is {{previousResponse}} but you can change it by setting previousResponsePattern option to new value.
(2) If no, you may mean skip/changing to different question within a form after an answer. This feature is not supported at the moment. The questions are asked based on pre-defined html form. However, you can switch form easily after an input from user. For example, you can ask user to switch to forget-password form if they insert wrong credential.
Hope this will help you.

My case is the case 2 ...
But i will try to implement that!
thank's !


RequestError: RequestError(400, 'search_phase_execution_exception', 'Failed to parse query [tender/awardCriteri...

Sentry Issue: STANDARD-SEARCH-15
RequestError: RequestError(400, 'search_phase_execution_exception', 'Failed to parse query [tender/awardCriteria]')
(4 additional frame(s) were not displayed)
...
  File "elasticsearch/client/utils.py", line 84, in _wrapped
    return func(*args, params=params, **kwargs)
  File "elasticsearch/client/__init__.py", line 852, in search
    "GET", _make_path(index, doc_type, "_search"), params=params, body=body
  File "elasticsearch/transport.py", line 358, in perform_request
    timeout=timeout,
  File "elasticsearch/connection/http_urllib3.py", line 231, in perform_request
    self._raise_error(response.status, raw_data)
  File "elasticsearch/connection/base.py", line 230, in _raise_error
    status_code, error_message, additional_info

RequestError(400, 'search_phase_execution_exception', 'Failed to parse query [Buyer / Vendor Unique Hash]')

q : Buyer / Vendor Unique Hash


Consider adding useful extension module

While the aspiration of this project is to have bindings for anything available in the browser, it might be interesting consider having an additional auxiliary module.
 Like https://developer.mozilla.org/en-US/docs/Web/API/Document/createElement. You pass it any string and get an element back. Maybe you want to create something more specific and get a better type back like document.createElement("input") and get HTMLInputElement back.
So having something like:
type document

type htmlInputElement

@scope("document")
external createInputElement: (unit, @as("input") _) => htmlInputElement =
  "createElement"

let i = createInputElement()

The input string can't be misspelled and the return type is better.
Perhaps each current API module can have an APIExt companion module?
Let me know what you thoughts.

I think this type of thing should go in the module directly. I'd say that's an idiomatic binding.

There will be other examples:
type document

type element = {tag: string}

type htmlInputElement

@scope("document")
external querySelector: string => element = "querySelector"

external unsafeCast: 'i => 'o = "%identity"

let querySelectorInputElement = (selector: string): option<
  htmlInputElement,
> => {
  let i = querySelector(selector)
  i.tag === "input" ? Some(unsafeCast(i)) : None
}

This is not strictly a binding but does provide a safe API.

Well, I think we definitely should ship "improved" APIs/helpers. Maybe others have differing opinions on this, but I think so at least. It'll increase the usefulness a lot for the lib.
We need to figure out where to draw the line obviously.
If we go for it, then I think we should find a way to pull completions from the "helper"/extension modules too.
Here's an example of something similar that might fit here - helpers for working efficiently with FormData: https://github.com/zth/res-x/blob/main/src/FormDataHelpers.res


Make ClimaDiagnostics compatible with ClimaLand

Purpose

Make ClimaDiagnostics compatible with ClimaLand
To-do

Start date in p
Content

[x] I have read and checked the items on the review checklist.

I think this PR is no longer needed. Can we close it? @AlexisRenchon


Not working when adding flag --v

When adding on the customFlags parameters the option  "--v", the script doesn't seems to work properly and timeout

Hi @momala454  can you explain what --v is made for? Can you provide code sample of how you are using it please?

i was having random problems with and wanted to debug it, so i added the --v parameter to chrome
just add --v to the customflags and any script will be broken

so you want to enable chrome logging? Can you describe the use case that requires this flag to be passed?

i had random (and rare) timeout of chrome, even if i wait 1 minute, chrome is not responding to my request
i didn't get rid of the issues, i just catch the timeout and restart from the beginning the process

this kind of errors
HeadlessChromium\Exception\OperationTimedOut: Operation timed out (45sec)
at HeadlessChromium.Utils.tryWithTimeout(Utils.php:65)
at HeadlessChromium.Communication.ResponseReader.waitForResponse(ResponseReader.php:103)
at HeadlessChromium.PageUtils.PageEvaluation.waitForResponse(PageEvaluation.php:69)
at HeadlessChromium.PageUtils.PageEvaluation.getReturnValue(PageEvaluation.php:101)

@momala454 do you think you would be able to provide a minimal reproducible test case?

Sorry it looks like it's not crashing anymore, maybe it was a bug with chrome (now fixed)

I'm glad to hear it, feel free to re-open if it occurs again


Do not pin Cargo.lock for fuzzing

Cargo.lock pinned an outdated version of ron and its dependencies
* [ ] I've included my change in CHANGELOG.md

@5225225 was there a particular reason for including the lock file?

IIRC this is the generated gitignore that cargo-fuzz makes, unless I messed it up at some point in making it. Maybe the generated gitignore should ignore the lockfile? It is basically a binary, and in general the guidance is to commit lockfiles for binaries. Though that's for reproducable builds which matters less here.
in general I prefer to keep lockfiles so you can always check out even old commits and know they will build, but for a fuzzer I suppose it doesn't matter too much.
But it shouldn't be actually using an outdated version of ron because it should be a path dependency, so the version there is always up to date with what the code in-repo is.
I'm not too bothered either way, though.

IIRC this is the generated gitignore that cargo-fuzz makes, unless I messed it up at some point in making it. Maybe the generated gitignore should ignore the lockfile? It is basically a binary, and in general the guidance is to commit lockfiles for binaries. Though that's for reproducable builds which matters less here.
in general I prefer to keep lockfiles so you can always check out even old commits and know they will build, but for a fuzzer I suppose it doesn't matter too much.
But it shouldn't be actually using an outdated version of ron because it should be a path dependency, so the version there is always up to date with what the code in-repo is.
I'm not too bothered either way, though.

Thanks for sharing your insights @5225225! We could keep the lock file around but perhaps update it every so often? I only noticed this as I was running cargo fuzz and suddenly had a dirty file on my hands :)


Componetize all the things

Massive changes to the structure of the app, my friends.  I basically just went through and did the tedious work of shoving markup into functional stateless components.  A bit of redundancy was reduced, but the most important thing is that the app is easily testable now.  I also moved the list of "Pairing Techs" into App.js as an array, so adding or removing a tech from the list is as easy as editing the array.
There are a couple of problems.  The biggest one is that the "Other" text field in the Add modal doesn't work anymore.  It doesn't seem to have any state passed to it.  Easy to fix, but I wanted input on that.  The second problem is that I couldn't test how the users look when listed as I didn't have the backend running.  Hope this helps!

The biggest one is that the "Other" text field in the Add modal doesn't work anymore.

This is an outstanding issue which we've been meaning to get around to.
I think the dev environment is still wired up to the live db, so you should be able to see the changes on users for real if your refresh your local version - but maybe we already locked that down...I've lost track :)
I'll run it from my end with everything to check it.


data: use Int comparison operators in favor of K

Simple change from when I was working on CSF presentation.

@daejunpark yeah still open at https://github.com/kframework/evm-semantics/pull/225

Ah, you are correct, I thought that got merged, should have double-checked.
Will push on that PR instead.


⚠ Bofrost.be has degraded performance

In 4f64729, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1202 ms

Resolved: Bofrost.be performance has improved in dddc5e4 after 8 minutes.


possible unsafety issue

The function cranelift_simplejit::SimpleJITBuilder::symbol (as it's close related friend symbols, and maybe other funcions) take a *const u8 as parameter.
From what I can see, there is no check whatsoever on the value provided before it gets used here.
This function should probably either be marked as unsafe, or take something less permissive than a *const u8 (maybe a NewType whose builder is marked unsafe?). As of now it is possible to pass it a null pointer or a dangling pointer (dropped Vec, pointer to data from an old stack-frame...), and writing to any of those is definitely Undefined Behavior

Is this issue still ongoing ? I don't see SimpleJITBuilder anywhere. I had assume it would have been in here https://github.com/bytecodealliance/wasmtime/tree/main/cranelift/jit


Use module's composer.json file when available.

When a module has a composer.json file, like Coder the composer.json that is provided by the module should be merged into the one on Drupal packagist.

@generalredneck reported this a few days ago. Please follow drupal-composer/drupal-parse-composer/issues/33


Allows custom parser declarations

Adjusts the TypeScript typings to allow custom parsers to be specified.

Pull Request Test Coverage Report for Build 328

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 93.732%

Totals

Change from base Build 325:
0.0%

Covered Lines:
668

Relevant Lines:
702

💛  - Coveralls

Thanks!  Very much appreciated


Support Analyzer 4

This package has a transitive dependency on Analyzer through i18n_extension preventing Analyzer 4 from being supported this is holding back a lot of packages from being updated.

I updated dependency in i18n package and the pull request is accepted. I will update the i18n version here when i18n is updated on pub.dev


Errors on Eigen and other files

Hello! I am interested in your wonderful event tracking algorithm, However, I did everything needed for compiling,including "git submodule update --init --recursive",  It has the following errors, which come from Eigen. Could you specify the version of Eigen you used?
haste/thirdparty/eigen/Eigen/src/Core/util/Meta.h:504:25: error: ‘invoke_result’ in namespace ‘std’ does not name a template type
typedef typename std::invoke_result<F, ArgTypes...>::type type1;
^
haste/thirdparty/eigen/Eigen/src/Core/util/Meta.h:504:38: error: expected unqualified-id before ‘<’ token
typedef typename std::invoke_result<F, ArgTypes...>::type type1;
I also changed a different version of Eigen,The problem seems solved. But another compiling error is:
In file included from /HASTE/haste/include/haste/app/tracking.hpp:8:0,
from /HASTE/haste/include/haste/app/command_parser.hpp:7,
from /HASTE/haste/include/haste/app_gui.hpp:5,
from /ASTE/haste/src/tracking_app_gui.cpp:4:
/home/haste/include/haste/aux/aux.hpp: In function ‘std::vector<std::__cxx11::basic_string > haste::splitString(const string&, char)’:
/home/lizeyu/HASTE/haste/include/haste/aux/aux.hpp:18:63: error: invalid use of ‘void’
while (std::getline(iss, item, delim)) { ret.emplace_back() = item; }
At your convenience , could you give me some hints about the problem?
Thank you!

Hi, thanks for your interest in our algorithms.
The required Eigen version is specified in the README.md (version >= 3.4):
https://github.com/ialzugaray/haste/blob/955c80acf6256d1d4fead22e6b11b8e351b34f53/README.md#L70
Since this one is not originally distributed with Ubuntu 18.04, we also ship it in the thirdparty/ folder. The path seems to be correct from the error you report, so please make sure you are not linking against symbols from the thirdparty/ folder using an outdated system-wide installed Eigen library, which is the only reasonable explanation I can think of.
Regarding the error you report on the method emplace_back, this method returns a reference since C++17 which is enforced in the CMakeList: https://github.com/ialzugaray/haste/blob/955c80acf6256d1d4fead22e6b11b8e351b34f53/CMakeLists.txt#L6
Please make sure you are compiling the code with C++17/C++20 standard instead of C++11 or older.


add branch protection for release-1.0 branch for fulcio

Summary

add branch protection for release-1.0 branch for fulcio

Thanks!


Change Density test to avoid pulling image be the main factor

cc @fgrzadkowski

If you're going to flip to pause, flip to the gcr.io image of it in the process. (On phone, but it's something like gcr.io/google-containers/pause cc @jlowdermilk)
I'm also not enamored with the 100 -> 30 swap, because @rrati was definitely hitting issues at 100

Regarding 100 -> 30 swap, I did it consciously, because in v1.0 we are NOT targeting 100 pods per node. What we would like to support is 30 pods per node, thus I think we shouldn't try to optimize for this case now. (But I agree we can get back to it after v1.0).

As for flipping the image, kubernetes/pause:go is the image that is used by Pod Infra Container. Thus it is already on the machine - that's why it will never be pulled for the "user container" itself.

I don't think we should drop the pod count here.  The density test I wrote can easily be modified to be multiple tests that scales up as desired, but we shouldn't drop from 100 imo.  100 pods/node may not be a target for 1.0, but that degree of density has shaken out a number of 1.0 blocking issues.  I don't feel 1.0 will be served better by dropping the count in this test.
iirc, 1.0 is targeting 30-50 pods/node.  This test should become multiple at various density levels imo.  30, 50, 100, etc.  That's easy enough to do with a for loop outside the It condition.

I think that running 100 pods per node is exposing many more issues than those we currently care for v1.0. And if all of them will be a result of running this Density test, it would be difficult to say whether it's a problem also at the level of 30 pods or does it appear only with say more than 80 pods per node.
Basically, what I would like to do is to have a "regression test" that will be part of e2e suite.
And I think that "should allow starting 30 pods per node" with some expectations added to it would be a good starting point for it.

Maybe a good solution for it would be:

leave a 100 pod per node test as skipped by default
add a 30 pod per node test as a test that is run as part of e2e suite.
What do you think?

Density isn't run as part of default e2e right now anyways. Would this
change allow a really low time budget density test? If so, I'm supportive
of 30-in-e2e.. But the 100.. Do we need parametrization?
On Mar 27, 2015 6:55 AM, "Wojciech Tyczynski"<EMAIL_ADDRESS>wrote:

Maybe a good solution for it would be:

leave a 100 pod per node test as skipped by default
add a 30 pod per node test as a test that is run as part of e2e
suite. What do you think?

—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/pull/6051#issuecomment-86946057
.

Yes - I know currently it's not running. And the whole e2e test still takes too long to be part of e2e suite. However, the ""should allow starting 30 pods per node" scenario (after the image change) is taking less than a minute (I ran it 3 times and all of them were running 40-50 seconds) so I think that is reasonable. The 100 pods per node can run at least 4 minutes.

To be clear - I'm happy to change all of it in this PR, but I'm waiting for some confirmation that you guys agree with that.

At less than a minute, I have no problem with it just being an e2e (which
requires some other gymnastics to get it back out of the skip list :/)
On Mar 27, 2015 7:12 AM, "Wojciech Tyczynski"<EMAIL_ADDRESS>wrote:

To be clear - I'm happy to change all of it in this PR, but I'm waiting
for some confirmation that you guys agree with that.
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/pull/6051#issuecomment-86952066
.

Thanks - I will try to change it.

I'm fine with 2 separate tests.  I think keeping the 100 pod/node even if it's not run as part of e2e is important.  We use it a lot.
We should be able to enable the density suite and exclude the 100 pod/node test during normal e2e runs.

Thanks @rrati - SGTM. I will work on it and let you know once I update this PR.

I've updated the PR - PTAL
also I'm running e2e tests - the new PerfTest already passed:
• [SLOW TEST:43.356 seconds]
PerfTest
/go/src/github.com/GoogleCloudPlatform/kubernetes/_output/dockerized/go/src/github.com/GoogleCloudPlatform/kubernetes/test/e2e/density.go:209
should allow starting 30 pods per node

Is there a need to create another test suite?  This is an awful lot of duplicated code.  Why not just add another It clause, make the suite non-pending, and exclude running the 100 pod/node test at e2e invocation?

This test isn't pending - it's just excluded as the whole when running e2e suite. I agree it's not beautiful, so if you know how to do it better, can you please be more precise?

Any comments/suggestions?

My thought was something like:
for cnt := range []int{30, 50 100} {
It("...%d pods per node", cnt) {
...
}
}
Not sure if ginkgo allows that type of definition, but other test frameworks have in other languages.  The above makes it easy to add another level of density to the test since the only difference between the tests is the number of pods/node.  The It clauses will be unique so that can still be filtered on.

I agree that would be fine as long as the whole test is disabled. But what I would like to achieve is to enable the 30-pod-per-node test in our e2e runs. And that will not run because the whole Density test is disabled:
https://github.com/GoogleCloudPlatform/kubernetes/blob/master/test/e2e/driver.go#L73
That's why I created the separate test. IIUC, this will not work with what you're suggesting.

We could rework that exclude logic in the driver to call out the 100 pod/node test explicitly.  Are there other tests in the density suite we would not want to run?
Would it make sense to move to an explicit blacklist of tests we want to avoid in an e2e run?  It seems like the default behavior should be to run all e2e tests so it makes sense that we should require some effort to prevent a test from running (ie changing the blacklist)

OK - I've refactored the test to avoid code duplication and only 30-pod-per node is enabled in e2e test suite.
PTAL

LGTM

Zach - since you already commented on it, I'm assigning it to you. Feel free to reassign if you think someone else should also take a look into it.

LGTM


Gateway does not forward packets to the application after manually setting the device address

Using the HTTP Application Manager API, we can create a device with arbitrary device address, e.g. "DE AD BE EF". The device is correctly registered in the application. The gateway receives packets from the device with the specified ID, but the packets are not forwarded to the application. Changing the first two bytes of the address to "26 01" did not solve this issue.
Again using the HTTP Application Manager API, we can create a device without a device address, i.e. the parameter "dev_addr" is completely missing in the request. Again, the device is correctly registered in the application, but the device address is not generated automatically and it is not possible to set it after registering.
This is the request sent to the application using the Application Manager API with Insomnia:

{
	"app_id": "fw-test",
	"dev_id": "abcdefghijkl",
	"lorawan_device": {
		"app_eui": "70B3D57ED000A7FF",
		"app_id": "fw-test",
		"dev_addr": "DEADBEEF",
		"dev_eui": "B76C00FC74F78B20",
		"dev_id": "abcdefghijkl",
		"app_s_key": "01020304050607080102030405060708",
		"nwk_s_key": "01020304050607080102030405060708",
		"activation_constraints": "abp"
	}
}

Can you try with a DevAddr from the "testing" range 00000000-03FFFFFF?

Using a device address from the testing range (01234567) does work as expected, i.e. the application receives the packages sent by the device.

Great. Testing addresses can be assigned freely by the developer. Network addresses (starting with 26 or 27) need to be assigned by the network.

Thank you.


Feature/enable head

fix https://github.com/ipa320/cob4/issues/349 (for cob4-5 and cob4-7)

used @ipa-nhg's test branch from cob4-5
rebased to latest indigo_dev config structure
little cob4-5 cleanup and harmonization
copied files for cob4-7

first tests (see https://github.com/ipa320/cob_robots/pull/660#issuecomment-297307109) ok
needs more testing

knoepkes :+1:
twist control :+1:
longterm :-1:

I also get an error during first initialization of head
fxm@cob4-7-b1:~/git/care-o-bot/src/cob_robots/cob_default_robot_config/robots (feature/enable_head)$ rosservice call /head/driver/init
success: False
message: /u/fxm/git/care-o-bot/src/ros_canopen/canopen_master/src/sdo.cpp(429): Throw in function void canopen::SDOClient::transmitAndWait(const canopen::ObjectDict::Entry&, const canopen::String&, canopen::String*)
Dynamic exception type: boost::exception_detail::clone_impl<boost::exception_detail::error_info_injector<canopen::TimeoutException> >
std::exception::what: SDO
[canopen::tag_objectdict_key*] = 60c2sub1

fxm@cob4-7-b1:~/git/care-o-bot/src/cob_robots/cob_default_robot_config/robots (feature/enable_head)$ rosservice call /head/driver/init
success: False
message: CAN is not ready
fxm@cob4-7-b1:~/git/care-o-bot/src/cob_robots/cob_default_robot_config/robots (feature/enable_head)$ rosservice call /head/driver/init
success: False
message: could not reset node '70'
fxm@cob4-7-b1:~/git/care-o-bot/src/cob_robots/cob_default_robot_config/robots (feature/enable_head)$ rosservice call /head/driver/init
success: True
message: ''

after that I can move the head using rqt_trajectory_controller

I also get an error during first initialization of head

Every time?

Doing more tests now - on a fresh roslaunch cob_bringup robot.launch:
fxm@cob4-7-b1:~$ rosservice call /head/driver/init
success: False
message: /u/fxm/git/care-o-bot/src/ros_canopen/canopen_master/src/sdo.cpp(429): Throw in function void canopen::SDOClient::transmitAndWait(const canopen::ObjectDict::Entry&, const canopen::String&, canopen::String*)
Dynamic exception type: boost::exception_detail::clone_impl<boost::exception_detail::error_info_injector<canopen::TimeoutException> >
std::exception::what: SDO
[canopen::tag_objectdict_key*] = 60c2sub1

fxm@cob4-7-b1:~$ rosservice call /head/driver/init
success: True
message: ''
fxm@cob4-7-b1:~$ rosservice call /torso/driver/init
success: False
message: Transition timeout; Could not enable motor; Transition timeout

The /head/driver/init error seems to occur every first try and works on the second init.
However, for the torso, I'm not able to initialize at all anymore...

tested head with knoeppkes:

head_joint_configurations and command_gui_buttons are ok :+1:
but sometimes the head "bails out" with error

Full Name: /Actuators/Head/head driver: head_3_joint
Component: head driver: head_3_joint
Hardware ID: none
Level: ERROR
Message: Motor has fault; Node has emergency error

error_register: 33
errors: 8611#0

ToDo
ControllerInterface for torso should (most likely) also be set to
controller_interface: cob_twist_controller/ControllerInterfacePosition

this config is fine for now and can be merged, I'll cross-link the bug 8611#0 to ipa320/cob4#349

DO NOT MERGE (yet). We should merge after the head_joint is proven to be robust. Please reopen this PR against ipa320/enable_head

I rebased to latest ipa320/indigo_dev and also pushed to https://github.com/ipa320/cob_robots/tree/feature/enable_head

errors: 8611#0

Means position tracking error..

@ipt-tif
made some mechanical adjustments to the head (will report separately)
after that 8611 occurred less often moving to dashboard configurations
however, when moving together with torso - against torso momentum - 8611 still occured a few times
according to @ipa-tif a re-dimensioning of the 2nd head motor might help
also. sometimes init/recover was not successful on first try...but a retry mostly solved the issues...

also. sometimes init/recover was not successful on first try...but a retry mostly solved the issues...

A candump might help.
I guess that the previous driver instance was not shutdown properly, but I am not really sure.


ENH: update package version

Hi! This is the friendly automated conda-forge-webservice.
I've started a version update as instructed in #12.
I'm currently searching for new versions and will update this PR shortly if I find one! Thank you for waiting!
Fixes #12

This can be closed now


How to work with Additional Identities using the REST API

Describe your use-case which is not covered by existing documentation.
Currently I am stuck how to view additional Identities using the REST API. When calling
https://jenkins/user/userName/api/json?pretty=true
the section with the additional identities is always empty as shown in the screen shot below.

Is there any documentation available how to make this information available?
Reference any relevant documentation, other materials or issues/pull requests that can be used for inspiration.
No response
Are you interested in contributing to the documentation?
No response

nothing available. it probably needs some code changes to be available on rest API


Added Error implementation for RunError

This makes RunError implement the Error trait, for easier interoperation with other libraries, if needed. I also upgraded dependencies and cleaned up Cargo.toml a bit.

@Razican, Is it needed? tarpaulin is not a library and currently error trait is not used.

@Razican, Is it needed? tarpaulin is not a library and currently error trait is not used.

As you wish, I thought it would make sense to implement it in case tarpaulin wants to show a backtrace of the error too :)

hmm I think I'll hold back on this, currently the display trait is enough and the most useful backtraces are probably from the actual test binaries in which case setting the RUST_BACKTRACE variable is really the only way to see them.


Prerelease premint api helper

Remove preminter code from 1155 contracts
Add a new SDK for preminting to packages
Cleanup dependencies for 1155 contracts

Current dependencies on/for this PR:

main

PR #276   👈

PR #304

PR #305

This comment was auto-generated by Graphite.

Merge activity

Oct 30, 6:36 PM: @iainnash started a stack merge that includes this pull request via Graphite.


way to run and compare all benchmarks in a crate

right now we have to do
cargo bench -q -p daft-minhash --bench=minhash -- compare target/benchmarks/minhash
cargo bench -q -p daft-minhash --bench=windowed -- compare target/benchmarks/windowed

It would be nice if we could instead do
cargo bench -q -p daft-minhash -- compare target/benchmarks

Right now my workflow combines tango-bench and cargo export

This is neat idea I would really love to implement, because I have very similar use case when I ran several benchmarks (from different crates) using shell-script.
Unfortunately, it will not work because cargo bench actually run not only benches but also unit tests. When you will pass any argument that standard harness is not supports (like -t) it will fail the whole cargo bench process. You can check it yourself, run following command in the context of a project that has at least 1 unit test. It should fail.
$ cargo bench -- compare -t 0.1
    Finished `bench` profile [optimized] target(s) in 0.04s
     Running unittests src/lib.rs (target/release/deps/pmem-dbbad5b7c1126fc4)
error: Unrecognized option: 't'
error: bench failed, to rerun pass `-p pmem --lib`

One workaround I can think of is to implement separate runner cargo tango that will run only benches.

yea having cargo tango would be really nice. I want to make a CI runner for this and being able to only run tango benches is a requirement.

here's an example of where I was using tango... but it is a bit limited
https://github.com/hyperion-mc/hyperion/pull/201

another example... I just re-integrated it. Right now I am using a matrix
https://github.com/hyperion-mc/hyperion/pull/781


chore: ec addition for non-zero points

Description
Problem*
This PR adds an ec addition for the embedded curve dedicated to non-zero inputs.
Summary*
By enabling the constant inputs for ec_add and asserting the inputs are not zero, we avoid the is_infinite overhead in bb.
Additional Context
I am not sure whether we want to document this or not?
Documentation*
Check one:

[X] No documentation needed.
[ ] Documentation included in this PR.
[ ] [For Experimental Features] Documentation to be submitted in a separate PR.

PR Checklist*

[X] I have tested the changes locally.
[X] I have formatted the changes with Prettier and/or cargo fmt on default settings.

We should hold this PR until we see if it is still relevant after switching to the alternative ec_add.

I'm surprised that this isn't resulting in an increase in the number of gates.

I'm surprised that this isn't resulting in an increase in the number of gates.

I guess it is due to the activation of constant folding for the ec add opcode.


Disable validation during initialization of swagger mddleware (restify)

Using swagger-restify-mw, it appears that initializing the middleware always hooks the validators.
How do I enable swagger-restify w/o validation - i.e. if I want to use our own one?
Or even better - how do I intercept a validation errors "after" swager-validator throws? It appears that in case of errors, the "after" event is not sent on the server.

Hi, when you initialize swagger-node, you likely have something that looks a bit like this:
SwaggerExpress.create(config, function(err, swaggerExpress) {
  if (err) { throw err; }

  // install middleware
  swaggerExpress.register(app);

the act of installing the middleware will install the full middleware tools, including the validation logic.  You'll want to instead register just the ones that you want:
// install middleware
app.use(swaggerExpress.cors());
app.use(swaggerExpress.jsonErrorHandler());
app.use(swaggerExpress.metadata());
app.use(swaggerExpress.security());
// app.use(swaggerExpress.validator());
app.use(swaggerExpress.expressCompatibilityMW());
app.use(swaggerExpress.router());
app.use(swaggerExpress.swaggerDoc());

Note that by not installing the validator, it will not cause input/output to be validated.


Ztunnel hairpinning to waypoint - wrong identity initiated

Bug Description
Scenario:
when pod not in the mesh connects to a pod in the mesh and the mesh pod  has a waypoint.
What happens:

Destination ztunnel hairpins and connects to the waypoint
ztunnel tries to connect to the waypoint using the source pod identity.
istiod refuses to sign the cert, as the source pod is not in the destination ztunnel's node

As the istiod behaviour is correct, we shouldn't have the destination ztunnel try to initiate connections on behave of the source pod's identity.
It should probably use the destination pod's service account, as the destination ztunnel represents the server, not the client. The problem with that, is that the waypoint will see the client's identity as destination's pod identity, where it should address this as coming from an unknown identity (as the source pod is not in the mesh).
Perhaps when hair-pinning we should do a plain tcp proxy to the waypoint (instead of mTLS)?
Version
1.18.0-alpha.0

Additional Information
No response
Affected product area

[X] Ambient
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
[ ] Upgrade
[ ] Multi Cluster
[ ] Virtual Machine
[ ] Control Plane Revisions

Is this still occurring? This seems tricky because the current documentation on the waypoint says that ALL connections to the waypoint are under mTLS. I'm wary of the ztunnel presenting untrusted traffic as trusted...should L7 policies just only apply for tunneled traffic?

Steven is writing a short doc on this, plan to present soon

https://docs.google.com/document/d/1uM1c3zzoehiijh1ZpZuJ1-SzuVVupenv8r5yuCaFshs/

Consensus from 09/06 Ambient meeting: Hairpinning will be done via Plain H2 TLS on outbound leg (no srcid), HBONE on inbound leg (zt dstID) -> effectively Option 3 from the ztunnel hairpinning doc

Hi @yuval-k @stevenctl @keithmattix @louiscryan @howardjohn saw this issue in drive ambient to beta doc - is this issue still relevant with network policy enforcing traffic going to waypoint and decline unknown identity?

This will be fixed by removing hairpinning as part of https://docs.google.com/document/d/1DAM_6SajCyC2hD8wBUraaZWZJqUVYPyemzw0-xpHmdY/edit#heading=h.er7ajjjlw2f3 I think?

https://github.com/istio/ztunnel/pull/913 removed hairpin


refactor: add new UncheckedSquare type for data validation

Add an unchecked square type to simplify using the core board functions when we know that a square is valid.

Thanks so much for this!
Can you please rebase this into more digestable commits? I like to do reviews by commit since that helps me make sense of things.

closing to resubmit on sourcehut


Board never closes

Using this library with johnny-five. However, when I finish and go to shutdown the connection to the raspberry pi I get a message in the console Board Closing but this just hangs forever.
I've been just manually going in and running a kill -9 on the process. This isn't ideal but I couldn't find other ways to more gracefully close the connection.
Any suggestions?

Can you tell me how you're attempting to close the app?
Normally, I use process.exit() when I want the app to exit.

When I ran in repl mode, I used process.exit(), but it still just hung.
Now I'm running out of repl mode and I'm just sending it a termination signal (CTRL-C).
Any other suggestions for closing it?

That's strange, ctrl-c works for me, as does process.exit(). Can you tell me the following?

Node version
Node architecture (e.g. armv6 or armv7)
npm version
Raspberry Pi version
Which OS and version (e.g. Raspbian 2016-03-18)
Wiring Pi version, from gpio -v

Node version 4.2.1
npm 2.14.7
Pi 2 B v1.1
Raspian (should be latest version, I ran sudo apt-get upgrade)
GPIO 2.32
I have a few other boards of the same type lying around so I can test it on another one

One thing I'm realizing is a confounding variable is I'm also listening for events on a HID device (a USB RFID reader). So maybe somehow that's the thing that's still lingering and preventing the process from closing?
I'll run a few tests. The only reason I suspected the raspio is because it hangs at the "Board Closing" line with no other info.

Can you figure out which kernel you have? uname -r IIRC. Running sudo apt-get upgrade doesn't necessarily upgrade to the latest release, just like how running it won't upgrade Ubuntu 14.04 to 15.10.
I'm not sure that really matters anymore though, I bet it's the HID device. Raspi IO doesn't do any sort of cleanup code, and doesn't make use of the beforeExit event. Raspi IO doesn't make a connection to anything, unlike say the Arduino, so there's nothing to close or cleanup.

Ah Ok, so potentially I need to figure out how handle device.close() during shutdown. Sounds like that's my issue. I'll go ahead and close this issue since it's likely not your issue.
You're saying that johnny-five though has a beforeExit that I could potentially work with?

You're saying that johnny-five though has a beforeExit that I could potentially work with?

It's built in to Node.js itself: https://nodejs.org/dist/latest-v5.x/docs/api/process.html#process_event_beforeexit
Best of luck!


SIGINT runs once. Example version freedom. Unblock Svelte postinstall.

Current dependencies on/for this PR:

main

PR #51   👈

This comment was auto-generated by Graphite.


jack/detail-view

Looks good. The ScrollView wasn't working because the scrollView.contentSize was missing and I also had to remove the bottom constraint of the contentView.

Looks good to me.


chore(merge-back): 2.21.0

See CHANGELOG

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: f2b8ec6bd0c8754dbcc428adda8297205738c998
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Update for CUDA 12.2.2

xref: https://github.com/conda-forge/cuda-feedstock/issues/13

@conda-forge-admin , please re-render

@conda-forge-admin , please re-render

Think we need to add this to conda-forge.yml
provider:
  linux_aarch64: default

@conda-forge-admin , please re-render

Thanks Alex! 🙏


Check failed: adm() when running without VS Code

I followed tutorial (https://microsoft.github.io/MixedReality-WebRTC/manual/cs/helloworld-cs-core3.html) and made simple console application with .NET Core 3.1. When I run it within VS Code it works correctly. But when I run just compiled .exe it shows error after a few seconds after initializing peer connection:
Fatal error in: ../../media/engine/webrtcvoiceengine.cc, line 253
last system error: 0
Check failed: adm()

Why can't my application work without VS Code? Am I doing something wrong?

This is a problem with your sound setup. Likely you don't have a microphone available. webrtcvoiceengine.cc is the code capturing the microphone; if it fails this means there's a problem during initialization.

I have a microphone and I allowed it to be used in apps on Windows settings. When I run my app within VS Code, this error doesn't appear

I'm not sure what "within VS code" vs. "compile the exe" means. They should run the same executable, just a different parent process. Only things that can differ are the environment / start path, which is unlikely to affect audio, and maybe the microphone being already in use so failing to initialize?

When i run app in VS Code it uses the following command:
& 'c:\Users\Me\.vscode\extensions\ms-dotnettools.csharp-1.23.14\.debugger\vsdbg.exe' '--interpreter=vscode' '--connection=4c1b17a5a29f44c9be65b565e6c510e3'

And all works!
When I try to run app:
.\bin\Debug\netcoreapp3.1\WebRTC.exe

It crashes with error which I described above
No, microphone is not in use already

I understand that these two ways of launching uses the same exe to run, but the results is different -_-


Add a type guard to the metadata tutorial

The tutorial looked like this before:

Now it looks like this:

The reason is that Epochs.metadata may be None, in which case all the Pandas methods won't work; hence, the type checker complains.

I'm wondering if I couldn't be even more explicit and state: look, epochs.metadata may be None on some cases;
so it's good practice to always check if that's the case first

Another option would be to avoid type instability, i.e., using None for the case when no metadata is available, but use an empty pd.DataFrame instead. Then no such gymnastics would be required at all.

Another option would be to avoid type instability, i.e., using None for the case when no metadata is available, but use an empty pd.DataFrame instead. Then no such gymnastics would be required at all.

Yes I would like that but it would require a deprecation cycle

Yes I would like that but it would require a deprecation cycle

It's also a bit more complicated. pandas is not a hard requirement of MNE-Python, but we wanted people without pandas to be able to load epochs with metadata and -- though they couldn't do metadata-based operations -- they could still do some other stuff like average or epochs["simple_condition"] based on event ids. So right now if you load metadata-enabled epochs without pandas, IIRC metadata is there but could be a dict for example.
So to keep things simple here I think the assert statement is good enough, thanks @hoechenberger !


script: update grafana add a "failed query" panel

Those query errors would return to client, when we find them in the grafana panel, we should check the log to see details.
And we'll should also add alert rules for them.

@coocood @shenli

@tiancaiamao
I think use log2 axis is better because the different error may have much different count.

LGTM

@jomenxiao PTAL

@LinuxGit PTAL


move resume.py from fwtwirl

[x] move resume.py from fwtwirl
[x] write a test for resume.py

done


betteruptime_aws_cloudwatch_integration does not support the "paused" attribute

As the title states , the betteruptime_aws_cloudwatch_integration does not support to be paused using terraform.
I get the following message when applying configuration.
betteruptime_aws_cloudwatch_integration.backups: Modifying... [id=37769]
╷
│ Error: PATCH https://betteruptime.com/api/v2/aws-cloudwatch-integrations/37769 returned 422: {"errors":"Sorry, you misspe
lled some attributes","invalid_attributes":["paused"]}
│
│   with betteruptime_aws_cloudwatch_integration.backups,
│   on uptime_integration.tf line 1, in resource "betteruptime_aws_cloudwatch_integration" "backups":
│    1: resource "betteruptime_aws_cloudwatch_integration" "backups" {
│
╵

Hi @Gael101,
Thank you for raising this.
I'm passing this on to the team to get it fixed. We'll keep you updated here on GitHub.
Thank you for your patience. I appreciate it.

Hi @Gael101,
Thank you for reporting this.
We have released a fix here: https://github.com/BetterStackHQ/uptime/pull/4656
Let me know if the issue persists for you. 🙏
Thanks again for raising this. I appreciate it.


add global loading state and use it in the UI components

For example, when switching wallet provider, while the new is loading, it would be useful to prevent the user from further interactions until the current one is resolved.

This is mostly implemented. It turns out we can rely on the walletApi to a high degree. Whenever we change provider, account etc, it will be undefined until loaded. Keeping this issue open for a little while longer.

There is nothing really actionable here, since it's not entirely clear what kind of a loading state this issues is for. The implementation now does look to walletApi being defined, and locks UI components accordingly. Will close this issue for now.


Loading gtk module fails

Hi,
I'm on Debian Testing using baresip 0.5.0. I'd suppose baresip module could be loaded like baresip -m gtk. But this fails. Even specifying the full path: baresip -m /usr/lib/baresip/modules/gtk.so fails:
Local network address:  IPv4=...
pre-loading modules: 1
dl: mod: .//usr/lib/baresip/modules/gtk.so (.//usr/lib/baresip/modules/gtk.so: cannot open shared object file: No such file or directory)
module /usr/lib/baresip/modules/gtk.so: No such file or directory
could not pre-load module '/usr/lib/baresip/modules/gtk.so' (No such file or directory)

After cd / the module file is found but still cannot be loaded:
Local network address:  IPv4=...
pre-loading modules: 1
aufilt: gtk_vumeter
gtk: message_init failed (Invalid argument)
module /usr/lib/baresip/modules/gtk.so: Invalid argument
could not pre-load module '/usr/lib/baresip/modules/gtk.so' (Invalid argument)

Could you please comment on these 2 issues: only relative path to the module file allowed and gtk module loading failure?
Thanks,
Andrey

the GTK module has to be loaded via config-file using the module_app directive:
module_app             gtk.so

absolute paths are not allowed.

After uncommenting the line
module_app              gtk.so

in ~/.baresip/config, gtk module has been loaded successfully. Thanks!

absolute paths are not allowed.

Is this anywhere documented? And also what modules cannot be loaded via command line at all?

No one answered Andrey about it being documented…


Warning: React.createElement: type is invalid -- expected a string (for built-in components) or a class/function (for composite components) but got: undefined. You likely forgot to export your component from the file it's defined in

Current Behavior
I am working with react-navigation and it has been working fine for me before,
but now when I have to include it in an app, it gives me this error,
Warning: React.createElement: type is invalid -- expected a string (for built-in components) or a class/function (for composite components) but got: undefined. You likely forgot to export your component from the file it's defined in
the project works fine without the navigation feature,
I tried changing this line
import {StackNavigator} from 'react-navigation';
to
import StackNavigator from 'react-navigation';

but then navigation doesn't work, giving the error
(0 , _reactNavigation2.default) is not a function
Any help is highly appreciated
Thanks

software
version

react-navigation
^1.0.0-beta.11

react-native
0.46.4

node
8.1.2

npm or yarn
npm 5.0.4

I think the issue you're running into is from index.android.js/index.ios.js. You need to make sure you're using import { SimpleApp } from 'react-navigation'; and then registering the component.

That worked!
Can't believe I spent a whole day.
I just added the curly braces and that helped.


Add E2E tests, runAsUser, add comments for OpenShift UID issue.

Reverts #16 and add some comments for OpenShift.
Extends #17 by changing container used for initContainers.
Bringing E2E to k8s-fah!
Things tested

multi-arch compatibility on docker.
multi-arch e2e tests.

Travis-CI and CircleCI are used to run ARM64 e2e tests without emulation. Running in GitHub Actions was too slow.
Pick your poison, only need one.
and badges! (may require account signups.)
k8s-fah
Run folding@home on Kubernetes.
Signed-off-by: Tiger Kaovilai<EMAIL_ADDRESS>

Looks like I am hitting an issue with https://github.com/richstokes/k8s-fah/pull/18

Circle CI | Travis


Change appearance property.

Add property to change font size, info view text size and dot appearance.

Run sample project to check changes.


File.open succeeds when "opening" a directory (Linux).

In the files package, File.open succeeds when "opening" a directory (on Linux).  Since the File class is used primarily for reading/writing files, I don't think this makes much sense.
Calling size on the resulting File object for me is returning the integer: 0x7fffffffffffffff.
Trying to read from the resulting File object for me is returning no bytes.
I propose that File.open should return FileError when the given FilePath points to a directory (or other non-file).
If we agree, I could implement this on Linux but I don't have any way to test with Windows.

I think you're right. Also, nice catch.
@andymcn can test on Windows. He actually voluntarily develops on Windows. Which, you know, no accounting for taste.

I started implementing this then thought to check it against other languages.
On OS X 10.11.1...

Ruby 2.3.0's File::open() will open directories, and gives the same size values as current Pony
Python 2.7.11's open() won't open directories. It is careful to chase down symlinks to see if there's a directory on the end.

Which behaviour would be best for Pony? Happy to survey other languages if this makes you unsure...

Well, it's a bit obvious what I think from the fact that I opened this ticket to propose the change, but I think Python's behavior is more correct (and I'm mainly a Ruby programmer in my day job :wink:).
It's also worth nothing that Ruby won't File.read a directory:
File.read("/home/jemc")
Errno::EISDIR: Is a directory @ io_fread - /home/jemc
from (pry):1:in `read'

In particular, I think the size behavior is harmful, as it will cause "naive" (read: intuitive) Pony code to allocate a huge buffer for no reason.  Ruby doesn't have this problem because naive/intuitive code will just use File.read, which only works for files.

Er, yes. Good point ;-)
Currently thinking this is going to need a new C function to call stat rather than lstat...

This problem is with File.open only, and not File.create, which I hadn't realised.
I agree that the Ruby behaviour (and current Pony behaviour) is wrong :)
As @georgemarrows pointed out, the current stat implementation is subtly unhelpful. I've changed it so that the "file type" fields in FileInfo, ie file, directory, pipe, and size, now report the values for the symlink target.
That lets us use FileInfo(from).directory in File.open to decide not to try to open a directory.
That means File.open requires the FileStat capability as well as FileRead. I think that's ok, but if we think it's a problem, we could create a private FileInfo constructor that doesn't check the capability, but I lean towards not doing that.
I've checked in the changes to do this, along with a unit test.
Sorry to steal away an implementation! I realised when looking at @georgemarrows 's point about stat that it needed fixing, and how to fix it, so I jumped in.

@sylvanc - no problem. Your prerogative!
One thing while cleaning up this area - do you want to be able to File.open a pipe? I would have guessed not, so perhaps...
diff --git a/packages/files/file.pony b/packages/files/file.pony
index 989fef2..d8d9b30 100644
--- a/packages/files/file.pony
+++ b/packages/files/file.pony
@@ -104,7 +104,12 @@ class File
 if
   not from.caps(FileRead) or

 try FileInfo(from).directory else true end

 try

   let finfo = FileInfo(from)

   finfo.directory or finfo.pipe

 else

   true

 end

then
_errno = FileError
else

Good point, @georgemarrows, I pushed this (but with just one FileInfo constructed).


How to override ngx translate pipe as I want to change my keys into uppercase before translating it by pipe.Just to make translation case insensitive..

Form https://github.com/ngx-translate/core#5-use-the-service-the-pipe-or-the-directive

You can also use the output of the built-in pipes uppercase and lowercase in order to guarantee that your dynamically generated translation keys are either all uppercase or all lowercase. For example:
{{ 'ROLES.' + role | uppercase | translate }}
role = 'admin';
will match the following translation:
{
"ROLES": {
"ADMIN": "Administrator"
}
}


Fix error when setting Default Retention Policy

Fixes issue #35
Also fixes the name of the dialog when altering Retention Policy

For those who are searching a compiled version with this bugfix, you can download it from my fork.


how to get the groundtruth in handheld dataset?

I download the handheld.bag and run the code. I get the trajectory generated by lvi-sam,but how to get the groungtruth file?

Have you solved the problem? I have the problem like this.I get the trajectory without ground truth  and the terminal inform "QObject::connect: Cannot queue arguments of type ‘QVector’ (Make sure ‘QVector’ is registered using qRegisterMetaType().)"

rtk

I download the handheld.bag and run the code. I get the trajectory generated by lvi-sam,but how to get the groungtruth file?

rtk
Is it in the bag？ How can I get it.


Update Kafka loading docs to use the streaming data loader

This PR updates the docs ahead of 0.16.0 release to incorporate the new screenshots and the data loader flow.
Also:

Fixes format of redirect file
Removes totally useless page and adds a redirect
Fixes some general formatting issues.

@vogievetsky I would also update the main data loading docs beyond just the tutorial to use the data loader.

rebased and force pushed to resolve a git merge mess-up


Decide where compute should happen for Entityset.add_last_time_indexes with dask dataframes

@frances-h knows how to resolved. will close once finished


Remove boilerplate code

from line 245 :
 if (clock == null) {
                throw new IllegalArgumentException("Clock must not be null");
       }
       this.clock=clock;

replace with
   `this.clock = Assert.requireNonNull(clock,"Clock must not be null");`

I am beginner to contribute opensource.

Thank you for your contribution. That's merged and backported now.