fix(stepfunction): add two extra methods to track

Add two extra methods:

stopExecution
sendTaskFailure

Class: AWS.StepFunctions
https://docs.aws.amazon.com/AWSJavaScriptSDK/latest/AWS/StepFunctions.html

:tada: This PR is included in version 1.106.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Allow configuring server-side TLS using CA only

As stated in the configuration instructions in the README, TLS requires both CERT and KEY path to be defined, for mTLS.
Correct me if I'm wrong, but it is currently not possible to configure server-side TLS, by specifying the CA path only.
In our setup, temporal-server is running as an ECS task, with an associated target group (configured as gRPC). An AWS ELB routes requests to this target group and does TLS termination. Our temporal-web must reach temporal-server through the ELB, therefore we must configure it for server-side TLS.
The changes suggested in this PR allow creating credentials using CA only for server-side TLS, and add a note regarding that to the README.
Let me know how I can improve it, happy to contribute.
PS: I realize I must sign the Temporal Contributor License Agreement before acceptance, is a signed PDF alright? Let me know where I should send it.

personally have no issues with this as it seems light enough but will defer to @feedmeapples

personally have no issues with this as it seems light enough but will defer to @feedmeapples

thx for the PR,
left one comment

thx for the PR,
left one comment


TabController / NestedScrollView / SliverAppbar -> InnerBox size -> Tiny content scrolls out / under appbar

Hi,
Topic is that I have the widget structure mentioned in subject, and on the first tab content that is not a list but a fixed box with a 4 label matrix. My problem is that InnerBox height while collapsing bar is increasing and content is going under appbar, what I want is in this case to avoid having those labels covered by the appbar so:

Having inbberbox maxsize screenHeight - collapsedAppbarHeight so I cannot scroll out content of the screen if it is not a listview which has really lots of items.
In the below example then when reaching tottally collapsed state it would bounce but I cannot scroll out

Currently I can see innerbox maximum size  probably is screenHeight, this works for me when I have lists as a content in tabs but even with lists with only 2-3 items have the same problem that I can scroll them under the appbar so I end up with seemingly empty tab.
I'm having the structure:
DefaultTabController
  NestedScrollView
    headerSliverBuilder:
      SliverAppBar
      SliverPersistentHeader
    body: TabBarView

Visible Tab content is a simple structure with 4 texts in
Column
  Row
  Row

Probably I'm just misusing the API, could somebody give me some hints?
Attaching 3 screen so you can imagine what I'm talking about :)
Thanks a lot in advance

Please add the output of flutter doctor -v.
Can you provide a minimal reproduction as executable code (main.dart)?

Hey zoechi,
Sorry for the late response I had some private issues:
flutter doctor -v output for me is:
[✓] Flutter (Channel beta, v0.6.0, on Mac OS X 10.13.6 17G65, locale en-HU)
    • Flutter version 0.6.0 at /Users/zoltanrak/flutter
    • Framework revision 9299c02cf7 (5 weeks ago), 2018-08-16 00:35:12 +0200
    • Engine revision e3687f70c7
    • Dart version 2.1.0-dev.0.0.flutter-be6309690f

[✓] Android toolchain - develop for Android devices (Android SDK 27.0.3)
    • Android SDK at /Users/zoltanrak/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-27, build-tools 27.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b01)
    • All Android licenses accepted.

[✓] iOS toolchain - develop for iOS devices (Xcode 9.4.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 9.4.1, Build version 9F2000
    • ios-deploy 1.9.2
    • CocoaPods version 1.5.2

[✓] Android Studio (version 3.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 27.1.1
    • Dart plugin version 173.4700
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b01)

[✓] Connected devices (1 available)
    • iPhone X • EF1AD6B0-047B-4B44-AECC-C8D5590A2E34 • ios • iOS 11.4 (simulator)

• No issues found!

Code snippet - where the point is that there is a simple text and if you scroll all the way up it will go under the app bar and I don't want it to go under -> here probably the nested scrollview size is the problem or?- is:
import 'package:flutter/material.dart';

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

class MyApp extends StatelessWidget {
  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return new MaterialApp(
      title: 'Flutter Demo',
      theme: new ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: new MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  MyHomePage({Key key, this.title}) : super(key: key);

  final String title;

  @override
  _MyHomePageState createState() => new _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> with SingleTickerProviderStateMixin {

  final List<Widget> tabPages = <Widget>[
    Text('first'),
    Text('second'),
    Text('third')
  ];

  TabController _tabController;

  @override
  void initState() {
    super.initState();
    _tabController = TabController(
        vsync: this,
        length: 3
    );
  }

  @override
  void dispose() {
    _tabController.dispose();
    super.dispose();
  }

  @override
  Widget build(BuildContext context) {
    return new Scaffold(
      body: DefaultTabController(
          length: 3,
          child: NestedScrollView(
              headerSliverBuilder: (BuildContext context, bool innerBoxIsScrolled) {
            return <Widget>[
              SliverAppBar(
                  primary:  true,
                  floating: false,
                  pinned:   true,
                  forceElevated: false,
                  expandedHeight: 300.0,
                  flexibleSpace: FlexibleSpaceBar(
                    title: Stack(
                        alignment: Alignment.bottomCenter,
                        children: <Widget>[
                          Padding(
                            padding: const EdgeInsets.only(bottom: 96.0),
                            child: Text('Large text',
                                softWrap: true,
                                textAlign: TextAlign.center,
                                style: new TextStyle(
                                    fontWeight: FontWeight.bold,
                                    fontSize: 16.0,
                                    color: Colors.white)),
                          ),
                          Padding(
                            padding: const EdgeInsets.only(bottom: 70.0),
                            child: Text('Small text',
                                softWrap: true,
                                textAlign: TextAlign.center,
                                style: new TextStyle(
                                    fontWeight: FontWeight.bold,
                                    fontSize: 22.0,
                                    color: Colors.white)),
                          ),
                          Padding(
                            padding: const EdgeInsets.only(bottom: 13.0),
                            child: Text('''Even smaller text''',
                                softWrap: true,
                                textAlign: TextAlign.center,
                                style: new TextStyle(
                                    fontWeight: FontWeight.normal,
                                    fontSize: 11.0,
                                    color: Colors.white)),
                          ),
                        ]),
                  )),
              SliverPersistentHeader(
                floating: false,
                pinned: true,
                delegate: _SliverAppBarDelegate(
                  TabBar(
                    controller: _tabController,
                    indicatorWeight: 2.0,
                    labelStyle: TextStyle(
                        fontWeight: FontWeight.bold, fontSize: 13.0),
                    indicatorColor: Colors.red,
                    labelColor: Colors.red,
                    unselectedLabelColor: Colors.grey,
                    tabs: [
                      Tab(text: 'first'),
                      Tab(text: 'second'),
                      Tab(text: 'third')
                    ],
                  ),
                ),
              )
            ];
          }, body: TabBarView(
            controller: _tabController,
            children: tabPages.map((Widget widget) {
              return widget;
            }).toList(),
          ))
      )
    );
  }
}

class _SliverAppBarDelegate extends SliverPersistentHeaderDelegate {
  _SliverAppBarDelegate(this._tabBar);

  final TabBar _tabBar;

  @override
  double get minExtent => _tabBar.preferredSize.height;

  @override
  double get maxExtent => _tabBar.preferredSize.height;

  @override
  Widget build(BuildContext context, double shrinkOffset,
      bool overlapsContent) {
    return new Container(
      color: Colors.white,
      child: _tabBar,
    );
  }

  @override
  bool shouldRebuild(_SliverAppBarDelegate oldDelegate) {
    return false;
  }
}

best,
sodjas

Update:
I solved it, this is practically a no issue, main problem was, that I followed some old examples and combined them together.
Solution is that you simply follow the sample code in - nested_scroll_view.dart -> key is to use for tab content all the facilities provided specifically to slivers for example SliverOverlapAbsorber etc... then sliverlist or slivergird -> with this my texts won't scroll out the collapsed sliverappbar.
Anyway thank for the quick support here!

Hi, sorry I had some sh***y weeks and was not able to react. Here is my simple build function which works for me. Hope it helps:
@override
  Widget build(BuildContext context) {
    // This method is rerun every time setState is called, for instance as done
    // by the _incrementCounter method above.
    //
    // The Flutter framework has been optimized to make rerunning build methods
    // fast, so that you can just rebuild anything that needs updating rather
    // than having to individually change instances of widgets.
    return Scaffold(
      body: NestedScrollView(
          physics: BouncingScrollPhysics(),
          headerSliverBuilder: (BuildContext context, bool innerBoxIsScrolled) {
            //debugPrint('Inner box scrolled ${innerBoxIsScrolled}');
            return <Widget>[
              new SliverOverlapAbsorber(
                  handle:
                      NestedScrollView.sliverOverlapAbsorberHandleFor(context),
                  child: SliverAppBar(
                    floating: false,
                    pinned: true,
                    forceElevated: false,
                    centerTitle: true,
                    backgroundColor: Colors.blue,
                    expandedHeight: 400.0,
                    flexibleSpace: FlexibleSpaceBar(
                      centerTitle: true,
                      title: Text('FlexibleSpace'),
                    ),
                  ))
            ];
          },
          body: Builder(
            builder: (BuildContext context) {
              return new CustomScrollView(
                primary: false,
                physics: BouncingScrollPhysics(),
                slivers: <Widget>[
                  new SliverOverlapInjector(
                    // This is the flip side of the SliverOverlapAbsorber above.
                    handle: NestedScrollView.sliverOverlapAbsorberHandleFor(
                        context),
                  ),
                  SliverGrid.count(
                    crossAxisCount: 2,
                    children: <Widget>[
                      Text(
                        'first',
                        textAlign: TextAlign.center,
                      ),
                      Text(
                        'second',
                        textAlign: TextAlign.center,
                      ),
                      Text(
                        'third',
                        textAlign: TextAlign.center,
                      ),
                      Text(
                        'fourth',
                        textAlign: TextAlign.center,
                      )
                    ],
                  )
                ],
              );
            },
          )),
      floatingActionButton: FloatingActionButton(
        onPressed: _incrementCounter,
        tooltip: 'Increment',
        child: Icon(Icons.add),
      ), // This trailing comma makes auto-formatting nicer for build methods.
    );
  }

The solution is using SliverOverlapAbsorber as the wrapping widget for the SliverAppBar.  Without SliverOverlapAbsorber, the body contents scrolls under the SliverAppBar out of view unnecessarily, even if the body contents is not filling the entire screen height.


How do you distribute traffic to a SQL DB

As far I as know you cannot distribute traffic to database unless it is an Oracle RAC server.

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

ID: 7eaaf7c7-c1fe-1afc-e7f8-bc0e90044fb8
Version Independent ID: 6461936a-3f5d-c2fe-cb37-e40fc9dc929e
Content: What is Azure Load Balancer?
Content Source: articles/load-balancer/load-balancer-overview.md
Service: load-balancer
GitHub Login: @asudbring
Microsoft Alias: allensu

@NormCooke
Thanks for the feedback! We are currently investigating and will update you shortly.

@NormCooke As is discussed Here, Load Balancer backends can only be Virtual Machines. Azure SQL DB cannot be a backend for an Azure Load Balancer.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

You missed my point. This is an illustration in the article, how is this possible?
<EMAIL_ADDRESS>Norm Cooke
From: TravisCragg-MSFT<EMAIL_ADDRESS>Sent: Monday, November 11, 2019 5:25 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Norm Cooke<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] How do you distribute traffic to a SQL DB (#42507)
@NormCookehttps://github.com/NormCooke As is discussed Herehttps://docs.microsoft.com/en-us/azure/load-balancer/load-balancer-standard-overview#why-use-standard-load-balancer, Load Balancer backends can only be Virtual Machines. Azure SQL DBhttps://docs.microsoft.com/en-us/azure/sql-database/sql-database-technical-overview cannot be a backend for an Azure Load Balancer.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/42507?email_source=notifications&email_token=ADLOLZEVIHWYSDTY3T3FSWDQTHLU3A5CNFSM4JLPI662YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDYLBMQ#issuecomment-552644786, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADLOLZAIFET3MMIP4Z5OFRLQTHLU3ANCNFSM4JLPI66Q.

@NormCooke Azure Load Balancers can distribute to SQL Servers (or any other database) that are installed on VMs in Azure. Because Azure SQL DBs do not exist inside your VNET and are not on VMs, you cannot use them for backend pools.
I take it you are referring to the concept that databases that do not natively sync cannot be load balanced?

Yes the illustration is wrong, I teach Azure classes and this illustration is all over the courseware. I have to explain to the students that illustration is wrong. Please fix this illustration.
Norm Cooke
From: TravisCragg-MSFT<EMAIL_ADDRESS>Sent: Monday, November 11, 2019 7:51 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Norm Cooke<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] How do you distribute traffic to a SQL DB (#42507)
@NormCookehttps://github.com/NormCooke Azure Load Balancers can distribute to SQL Servers (or any other database) that are installed on VMs in Azure. Because Azure SQL DBs do not exist inside your VNET and are not on VMs, you cannot use them for backend pools.
I take it you are referring to the concept that databases that do not natively sync cannot be load balanced?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/42507?email_source=notifications&email_token=ADLOLZGEQAPRKSHFVUAKX23QTH4Y5A5CNFSM4JLPI662YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDYURHI#issuecomment-552683677, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADLOLZCH4IFGF4C5PU7YDWDQTH4Y5ANCNFSM4JLPI66Q.

@NormCooke I am re-opening the issue and assigning to the author to update to a realistic example.

@NormCooke can you post a picture of the illustration that is incorrect?  It's now showing above and i'm not following which one needs to be edited.  Thanks!

<EMAIL_ADDRESS>Norm Cooke
From: Allen Sudbring<EMAIL_ADDRESS>Sent: Tuesday, November 12, 2019 12:23 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Norm Cooke<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] How do you distribute traffic to a SQL DB (#42507)
@NormCookehttps://github.com/NormCooke can you post a picture of the illustration that is incorrect? It's now showing above and i'm not following which one needs to be edited. Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/42507?email_source=notifications&email_token=ADLOLZDNL56D24VY5PXGFI3QTLRALA5CNFSM4JLPI662YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOED3BCBQ#issuecomment-552997126, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADLOLZDERMOTPIDYLS4XLF3QTLRALANCNFSM4JLPI66Q.

@NormCooke I'm still not able to see that image. Is it this image you are referring to:
https://github.com/MicrosoftDocs/azure-docs/blob/master/articles/load-balancer/media/load-balancer-overview/IC744147.png
Thanks!

yes
Norm Cooke
From: Allen Sudbring<EMAIL_ADDRESS>Sent: Tuesday, November 12, 2019 12:54 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Norm Cooke<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] How do you distribute traffic to a SQL DB (#42507)
@NormCookehttps://github.com/NormCooke I'm still not able to see that image. Is it this image you are referring to:
https://github.com/MicrosoftDocs/azure-docs/blob/master/articles/load-balancer/media/load-balancer-overview/IC744147.png
Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/42507?email_source=notifications&email_token=ADLOLZF5RQVFXIPGIAIKI7TQTLUSZA5CNFSM4JLPI662YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOED3EWJY#issuecomment-553012007, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADLOLZBU6IAJLM3DGWAEP7TQTLUSZANCNFSM4JLPI66Q.

That illustration says SQL and not Azure SQL, our other illustrations will specifically call out if it's azure sql and mention it in the description. Its is assumed in a tiered infrastructure that the backend is regular SQL databases.  Are you wanting those sql databases to be designated as on VM's?

You are missing the point. You cannot load balance database traffic whether it is Azure SQL, onpremise database, or sql server on virtual machine.
You cannot load balance database traffic, full stop, period.
Norm Cooke
From: Allen Sudbring<EMAIL_ADDRESS>Sent: Tuesday, November 12, 2019 12:57 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Norm Cooke<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] How do you distribute traffic to a SQL DB (#42507)
That illustration says SQL and not Azure SQL, our other illustrations will specifically call out if it's azure sql and mention it in the description. Its is assumed in a tiered infrastructure that the backend is regular SQL databases. Are you wanting those sql databases to be designated as on VM's?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/42507?email_source=notifications&email_token=ADLOLZHLURIVC542TXLKHA3QTLVALA5CNFSM4JLPI662YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOED3FFAQ#issuecomment-553013890, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADLOLZGQEVW2IKY6VVJATB3QTLVALANCNFSM4JLPI66Q.

I will verify with PM, but I believe this is referring to SQL Always-on Cluster and the load balancing to the availability group listener, which is load balanced.  I can change the illustration to be more clear that it's not referring to single sql databases but a SQL Always On cluster running in azure.

You can only distribute read intent traffic to availability group listener, you cannot make updates so you cannot load balance database traffic.
Norm Cooke
From: Allen Sudbring<EMAIL_ADDRESS>Sent: Tuesday, November 12, 2019 1:08 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Norm Cooke<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] How do you distribute traffic to a SQL DB (#42507)
I will verify with PM, but I believe this is referring to SQL Always-on Cluster and the load balancing to the availability group listener, which is load balanced. I can change the illustration to be more clear that it's not referring to single sql databases but a SQL Always On cluster running in azure.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/42507?email_source=notifications&email_token=ADLOLZALA6QFOH3DKQAT2WDQTLWIDA5CNFSM4JLPI662YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOED3GVCY#issuecomment-553020043, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADLOLZAWU4NOVE57WLURU23QTLWIDANCNFSM4JLPI66Q.

You should change the illustration to refer to business tier not database tier.
Norm Cooke
From: Allen Sudbring<EMAIL_ADDRESS>Sent: Tuesday, November 12, 2019 1:08 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Norm Cooke<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] How do you distribute traffic to a SQL DB (#42507)
I will verify with PM, but I believe this is referring to SQL Always-on Cluster and the load balancing to the availability group listener, which is load balanced. I can change the illustration to be more clear that it's not referring to single sql databases but a SQL Always On cluster running in azure.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/42507?email_source=notifications&email_token=ADLOLZALA6QFOH3DKQAT2WDQTLWIDA5CNFSM4JLPI662YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOED3GVCY#issuecomment-553020043, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADLOLZAWU4NOVE57WLURU23QTLWIDANCNFSM4JLPI66Q.

The illustration just needs to be clearer that's its a tiered infrastructure with a web app to a data tier with SQL AO, similar to this page, also put a blurb in that's its a SQL always on cluster in the description:
https://docs.microsoft.com/en-us/azure/architecture/guide/architecture-styles/n-tier
I will create a work item to make it clearer.
#please-close

This is not load balanced. You cannot load balance database read write access. Your illustration is wrong you need to refer to a middle tier as the link you provided illustrates. The link you provided does not load balance the database access. If you think it does you are mis interpreting the purpose of the article.
Norm Cooke
From: Allen Sudbring<EMAIL_ADDRESS>Sent: Tuesday, November 12, 2019 2:08 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Norm Cooke<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] How do you distribute traffic to a SQL DB (#42507)
The illustration just needs to be clearer that's its a tiered infrastructure with a web app to a data tier with SQL AO, similar to this page, also put a blurb in that's its a SQL always on cluster in the description:
https://docs.microsoft.com/en-us/azure/architecture/guide/architecture-styles/n-tier
I will create a work item to make it clearer.
#please-close
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/42507?email_source=notifications&email_token=ADLOLZHOJT7336FGA7OTY3TQTL5KBA5CNFSM4JLPI662YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOED3SDSQ#issuecomment-553066954, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADLOLZAF72KCFSBSUBRXQMLQTL5KBANCNFSM4JLPI66Q.


Hovering a button should change the cursor to pointer unless disabled

This is one of the QA items in https://github.com/artsy/collector-experience/issues/228

Needs a rebase but looks good!

rebased!


Perhaps provide some more gradual examples on the https://bastion.rs/ site?

Hello!
I've looked at https://bastion.rs/, and I must say that, as a newcomer, I find the examples a bit intimidating.
I have a basic understanding of how actors work, but when I look at the site, as a newcomer, what I see:
use bastion::prelude::*;

#[fort::root]
async fn main(_: BastionContext) -> Result<(), ()> {
    println!("Running in Bastion runtime!");
    Ok(())
}

My reaction: "Ok... I can print things inside of a 'runtime', whatever that is... Looks simple enough."
And then:
use bastion::prelude::*;
use std::sync::atomic::{AtomicBool, Ordering};
use std::sync::Arc;

fn main() {
    Bastion::init();

    let started = AtomicBool::new(false);
    let started = Arc::new(started);

    Bastion::children(|children| {
        children.with_exec(move |ctx: BastionContext| {
            let started = started.clone();
            async move {
                println!("Started!");

                if started.swap(true, Ordering::SeqCst) {
                    println!("Already started once. Stopping...");

                    // This will ask the system to stop itself...
                    Bastion::stop();
                    // ...and this will stop this child immediately...
                    return Ok(());
                    // Note that if Err(()) was returned, the child would have been
                    // restarted (and if the system wasn't stopping).
                }

                // This will return None.
                let try_recv = ctx.try_recv().await;
// Lots more lines with supervisors, children, child groups, etc...

Bastion certainly looks interesting, and I'd love to learn more about it, but... Wow, that's a lot going on. What's the difference between a supervisor? A child group? A child? How does Fort come into that? What's a context? A runtime? Help!
As a basic example, it's a bit intimidating, there's kinda way too many concepts being introduced at once. It's probably going to scare away some beginners.

Hey and thanks a lot for your feedback! It does indeed look quite overwhelming, we might need to refer to the examples instead!


[bug]: Getting Error Messages Installing

Is there an existing issue for this?

[X] I have searched the existing issues

OS
Windows
GPU
cuda
VRAM
8GB
What happened?
I have the latest versions of Python and the Visual C+ Libraries installed and ran the registry update. I start the install.bat file and I repeatedly get these errors.
*** Installing InvokeAI Requirements ***
Activating environment
updating pip with "C:\Users\roger\invokeai.venv\Scripts\python"
Looking in links: c:\Users\roger\AppData\Local\Temp\tmpke9x_4a0
Requirement already satisfied: setuptools in c:\users\roger\invokeai.venv\lib\site-packages (63.2.0)
Requirement already satisfied: pip in c:\users\roger\invokeai.venv\lib\site-packages (22.2.1)
ERROR: Could not open requirements file: [Errno 2] No such file or directory: 'environments-and-requirements/requirements-base.txt'
[notice] A new release of pip available: 22.2.1 -> 23.0
[notice] To update, run: python.exe -m pip install --upgrade pip
Requirements installation failed. See above for errors and check https://invoke-ai.github.io/InvokeAI/installation/INSTALL_AUTOMATED/#troubleshooting for potential solutions.
The installer will exit now.
I already have PIP 23.0 installed. Lstien had kindly recommended in another post how to install InvokeAI manually - https://github.com/invoke-ai/InvokeAI/blob/v2.3.0-rc2/docs/installation/020_INSTALL_MANUAL.md.
However, I replied that I went to the installation procedures link that he kindly provided. As soon as I read the first sentence, "!!! warning "This is for advanced Users" and then the second sentence. "python experience is mandatory" I stopped. I'm by means an Advanced User and have no python experience at this point (I am planning on taking a course in the future). Will the easier Installation Instructions and files found at https://invoke-ai.github.io/InvokeAI/ be updated to allow a complete novice like me to setup and install InvokeAI on my Windows 11 PC eventually? Please let me know, thanks.
Screenshots

Additional context
No response
Contact Details
<EMAIL_ADDRESS>

This is very weird. Is the requirements-base.txt file not there at all?
There is a new version about to come out. You can try the new installer: https://github.com/invoke-ai/InvokeAI/files/10671597/InvokeAI-installer-v2.3.0-rc6.zip

Thanks so much for your reply and the new installer. I upgraded my 3 year old Gaming PC to Windows 11 last week and have had nothing but problems. Yesterday, I did a clean fresh install of Windows 10 from a USB drive. I'll try running the installer tomorrow and let you know if I have any problems getting it setup. Thanks again.


Plugin could not be activated because it triggered a fatal error.

Parse error: syntax error, unexpected ':', expecting '{' in /home/XXXXXX/public_html/wp-content/plugins/network-media-library-master/network-media-library.php on line 63

Thanks for the message.
As per the plugin requirements in the readme file, you need to be running PHP 7 or later to use this plugin.


Added sector number into the ID CRC calculation (which was not in it mistakenly)

Thanks!  I take it.


Wrong dll name for netcore2.0

All dlls in the nuget have the name JenkinsWebApi.dll.
Only in the netcoreapp2.0 folder, the dll is named JenkinsWebApiCore20.dll
This causes the following exception:
ReflectionTypeLoadException:
'Unable to load one or more of the requested types.
Could not load file or assembly 'JenkinsWebApi, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'. Das System kann die angegebene Datei nicht finden.'

maybe it's also an issue, because I call jenkins from a netstandard2.0 project and host it in a netcoreapp2.1
putting in a reference to %userprofile%\.nuget\packages\jenkinswebapi\2.2.0\lib\netstandard2.0\JenkinsWebApi.dll in both projects makes it run successfully.

in JenkinsWebApiCore20.csproj
the following must be added:
 <TargetFramework>netcoreapp2.0</TargetFramework>
 <AssemblyName>JenkinsWebApi</AssemblyName>    <!--<-here-->
 <RootNamespace>JenkinsWebApi</RootNamespace>  <!--<-here-->

Fixed in V2.3


Can't find child even though child was rendered

this one is really odd.  Here is a screencast of it: https://youtu.be/WyfVGls69ws
but basically I find that the  is indeed returned in the container's render yet the mount find isn't finding it per my css feature marker.
    it.only('renders a helmet tag for seo', () => {
      const liveScreen = mount(
        <Provider store={createStore(reducers, { live: { panels, seo: { title: "Test Title" } } })}>
          <Live
            breakpoint="site-web"
            setOverlay={() => ''}
            overlay=""
            geoLocationDeclined
            renderInfoInDetail={() => {}}
            getUseUtagGlobal={() => {}}
            pageViewEvent={() => {}}
            AccessEnablerUrl=""
          />
        </Provider>);

        const tag = liveScreen.find('.ft-seo-tag-title');

      expect(tag).to.have.length(1);
    });

the container is wrapped in a ReactTimeout HOC but I don't think that matters
const TimeoutLive = ReactTimeout(connect(mapStateToProps, {
  fetchLive,
  fetchLivePanel,
  fetchLiveVideo,
  clearLive,
  saveLiveRestart,
})(Live));

I end up with tag having length 0.
When I do a .debug() on the container, I don't see it so I guess it's not being render, which is weird, as you see in the vid it was.  I will investigate unless anyone has thoughts on it...
I think what might be happening is the helmet lib is swallowing that render and rendering it in its own HOC form, because here is what I see from debug.  Notice the title of Test Title is there:
<HelmetWrapper defer={true} encodeSpecialCharacters={true}>
                <SideEffect(NullComponent) defer={true} encodeSpecialCharacters={true} title="Test Title" titleAttributes={{...}}>
                  <NullComponent defer={true} encodeSpecialCharacters={true} title="Test Title" titleAttributes={{...}} />
                </SideEffect(NullComponent)>
              </HelmetWrapper>

Update #1
well this is weird, I put a marker on the helmet tag and it still can't find it:
const title = liveScreen.find('.ft-seo-tag')
<HelmetWrapper className="ft-seo-tag" defer={true} encodeSpecialCharacters={true}>
                <SideEffect(NullComponent) className="ft-seo-tag" defer={true} encodeSpecialCharacters={true} title="Test Title" titleAttributes={{...}}>
                  <NullComponent className="ft-seo-tag" defer={true} encodeSpecialCharacters={true} title="Test Title" titleAttributes={{...}} />
                </SideEffect(NullComponent)>
              </HelmetWrapper>

ends up in length of zero even though clearly the className is there when looking at the enzyme output!
Update #2
Ok I don't like doing finds on Component names because that couples my tests to implementation details but for the sake of debugging this, I did:
const title = liveScreen.find(Helmet)

What's interesting or weird is that it finds it when going the find by Component route.
But then that still begs the question of my original problem, if that's the case why didn't my find on the marker work if you can clearly see it as a prop below when I find it by Component Name?

So this is mounted, I'd figure that it should be able to find anywhere in the react tree, through any HOC levels in order to find my marker, without having to do anything other than mount()...is that correct?:

when I try this, it's not finding it and I don't know why:

Update #3
Just fiddling around with stuff.  This is weird too.  So if I try this route just for the hell of it, I'm able to get at the child of the HOC that has what I'm actually trying to test:

When we just look at props you see:

So I said fine, it's weird how helmut's structure is, not intuitive but thought ok, lets see if I can get the title by doing const title = tag.props().children[0].props().children since you see there that children is a property and contains the title I'm testing:

hmm why would that be an error?
If I take one step back, it's saying I can't call props() on that child when it does have props?

resolved but was kinda nasty.  @ljharb any ideas to improve on this?
solution: expect(title.children[0][0].props.children).to.equal(data.live.seo.title);
screencast:  https://youtu.be/tnPWcb-55b4

title.childAt(0).prop('children') doesn't work?
You should never need bracket access on an enzyme wrapper.

well usually I bypass having to do any traversing like that by adding css feature markers but yea.  I'll try that although doing children[0].prop('children') didn't work.  I had to access it via multidimensional array due to the fact that for some reason I couldn't find by feature marker this time.

https://github.com/airbnb/enzyme/issues/1233#issuecomment-335547400


Permissions Error When Hosted on Azure

Hello, I'm trying to wrap my head around using Microsoft Graph for an app that will need to be hosted on Azure. I took the sample app and published it to an Azure app I own. I will be using Azure AD so I set up an AD app as well (putting the app id and secret in my web app's web.config using your instructions). I can launch the app fine. I click on "Get email address" and I get sent to a login page. When I log in via my (Microsoft) account I am presented with the following:
You do not have permission to view this directory or page.
Any idea what I might be doing wrong?
My end goal is to simply check to see if a user is a member of a specific group and if not, kick them out. I'm thinking I can use GraphServiceClient.Me.CheckMemberGroups. Am I right here? Any help you can offer would be wonderful.

Hello. I apologize for the lateness of this reply. I don't know why your Microsoft account would behave that way, unless there is something special about the way you registered your client. Did you use the app registration portal to register it  (https://apps.dev.microsoft.com/) I ask because you mention that you're using Azure AD, and that suggests to me that you used portal.azure.com instead. Is there a reason why you can't use the app registration portal?
As for your plan, I think that looking at the user's member groups before allowing any further action should work.

Closing for now. Please reopen if it continues to be a problem.

Hi Jim. I ended up scrapping what I had done and started over from scratch. In doing that I got things to work as expected from the instructions (i.e. by using a converged application created at apps.dev.microsoft.com). I am still curious if this will work with an Azure AD application. These can be provisioned via apps.dev but ultimately are managed at portal.azure.com as you mentioned. I had initially gone with an Azure AD because the app I'm creating will be hosted in Azure and you can easily link an AAD to your web app in the Authentication blade.

I don't believe that you can use an Azure AD application with the v2.0 endpoint. This article discusses the differences between the v1 and v2 endpoints:
https://docs.microsoft.com/en-us/azure/active-directory/develop/active-directory-v2-limitations


Why do media cards allow multiple media URL's?

This document suggests that media cards (animation, audio, video) play multiple files one after the other. The documentation in the Swagger file suggests that the multiple files are meant to be the same content in different formats: https://github.com/microsoft/botframework-sdk/blob/master/specs/botframework-protocol/botframework-channel.json
This should be investigated to see what really happens.

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

ID: c36ddb50-9e44-96e3-2df3-174166efa75d
Version Independent ID: 68ac4b64-6616-2216-3519-e0b33d3a53e4
Content: API reference - Bot Service
Content Source: articles/rest-api/bot-framework-rest-connector-api-reference.md
Service: bot-service
GitHub Login: @ivorb
Microsoft Alias: kamrani

See bot-docs-pr PR 1698 for more context.

Work in progress.
See PR: https://github.com/MicrosoftDocs/bot-docs-pr/pull/1712

You can find the staged document at this location: https://review.docs.microsoft.com/en-us/azure/bot-service/rest-api/bot-framework-rest-connector-api-reference?view=azure-bot-service-3.0&branch=pr-en-us-1712

Issue resolved. See also spec definition: https://github.com/Microsoft/botframework-sdk/blob/master/specs/botframework-activity/botframework-cards.md#media-card-media.
The updated article will be available in the next doc build.


Added colorspace and color_range to torchcodec

Without this change, with ffmpeg7 we get warning messages like the following:
Changing video frame properties on the fly is not supported by all filters
For more info see this:
https://trac.ffmpeg.org/ticket/10823

I also experimented with a similar change, but I believe it's not supported by all versions of FFmpeg. If we do something like this, we'll need to guard it appropriately.


Filter empty PluginData folder in DistantObject-default (fixes #6065)

DistantObject-default currently installs an empty GameData/DistantObject/PluginData folder, which causes conflicts with DistantObject at uninstallation (see #6065). This pull request removes the empty folder.

Wauw, that this issue wasn't noticed for so long. Merging


Update docs with mention of how @with_cmp affects .hash()

I have run into this use case a few times now: I want all of characteristic's goodness for the entire set of attributes, but it only makes sense for .__hash__() to reference some subset of of them, because the rest are not meaningful in that context. Characteristic already has a facility to make this distinction for compares (with_cmp/exclude_from_cmp) and I'd like to submit a PR that does it for hashes as well in an analogous manner.
Any objections @hynek ? :+1:

... and it becomes clear from reading the code that @with_cmp makes .__hash__() behave in the exact way needed to satisfy this use case. Perhaps a mention of this in the docs might be useful?

I'm traveling right now but for now is like to mention characteristic's fork hynek/attrs that does hash separately because I ran into frequent problems with mutable members.
(I'll look at your PR once I'm back)

Thanks for the pointer, I'll be using attrs from now on. It's got a number of small improvements, but there are so many that it adds up to a big difference in usability and elegance. As far as mutable members interacting with hashes goes, that's expected. There are several ways of dealing with that edge case, but none are general enough imo to make them foolproof, so I'd say leaving the burden with the user is acceptable there.
The pull request is just a doc change, no hurry there.

Heh yeah, attrs is born out of my own frustrations with characteristic; hence that effect. :)


Test against PHP 7.2

Q
A

Bug fix?
no

New feature?
yes

BC breaks?
no

Deprecations?
no

Fixed tickets
-

Related issues/PRs
-

License
MIT

With the release of PHP 7.2, would be interesting to test it.

It won't work, because of the countable thing. :/
Could you instead let it try to fail?

Actually, the tests passed. Do you want to allow failures?


Allow other devices to register scans

Allow another device (like a laptop) to register scans
Not sure how it should work yet.

Proposal:

Cards become devices with a unique identifier (for cards this is the serial, for other devices chosen (or generated (or mac)))
Users get a key that they can use together with the identifier to register a scan (but only for themselves)
Things like the scanner get key that can register for all users
Scans store which device made them

Will need a way for the user to rotate the key, or the ability to generate new ones (and delete or disable old ones)

actually I like that people have to use the scanner (that we worked quite hard on) and that checking in isnt automatic. Might revisit in the future


Integration Test - Release - 2019/08/01 15:28:14

Auto create for integration test. AnotherAuthorUpdatedArticleFlow

@lamebrain : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Updated Arch installation instructions

Came across Arch install instructions and noticed it mentions pacman, which won't install AUR packages.
I included Arch build instructions for AUR helper of choice and also building with the PKGBUILD.
Looking forward to digging into more of this, this project has huge promise for unprivileged Docker in Docker builds and I have already suggested to my DevOps evangelists to follow its development. :+1:

you are awesome thanks so much!

Ignore my inability to click "View details" and instead click "Revert". :whale:


Analysis services Track2 Rename Namespace to Analysis

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.AnalysisServices


Update cdn-pop-locations.md

Adding new POP locations for Standard CDN from Microsoft

@mdgattuso : Thanks for your contribution! The author, @magattus, has been notified to review your proposed change.

@mdgattuso - Microsoft authors of articles need to make updates through the private repository so the updates can be staged and validated by the current validation rules. Read about article authors working in the public repo for more information. Thank you.


Expanding type system for generic types

PR for #10

Oh, just realized that I haven't updated the README.md yet.  I'll try to get a check-in for that in the next few days.
Let me know if you have any comments or concerns about this PR in the meantime.

Ok, that should finish up my check-ins for this PR.

After doing some more testing, I found an issue with fields in generic types.
I'm writing more unit tests to make sure the solution I have is right.

Ok, I think that'll fix everything up.
Let me know if you see any more issues.

awesome work. i will ship it now


fix: use sf.fullStack if it exists

What does this PR do?
Uses SfError.fullStack as the displayed error when it exists so that the original error stack can be seen/used.
What issues does this PR fix or reference?
@W-0@

QA Notes:

when SFDX_ENV=development
I modified @salesforce/packaging to access an undefined property on an undefined variable
  if (options['abc'].length) {
    return;
  }

and then linked through plugin-packaging
BEFORE:
ERROR running force:package:beta:install:  Cannot read properties of undefined (reading 'abc')
*** Internal Diagnostic ***

Type: Cannot read properties of undefined (reading 'abc')
    at Function.wrap (/Users/william.ruemmele/projects/oss/plugin-packaging/node_modules/@salesforce/core/lib/sfError.js:61:25)
    at Install.catch (/Users/william.ruemmele/projects/oss/plugin-packaging/node_modules/@salesforce/command/lib/sfdxCommand.js:283:38)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async Install._run (/Users/william.ruemmele/projects/oss/plugin-packaging/node_modules/@salesforce/command/lib/sfdxCommand.js:106:13)
    at async Config.runCommand (/Users/william.ruemmele/projects/oss/plugin-packaging/node_modules/@oclif/core/lib/config/config.js:272:25)
    at async Object.run (/Users/william.ruemmele/projects/oss/plugin-packaging/node_modules/@oclif/core/lib/main.js:76:5)
******

and AFTER linking this PR
 ➜  SFDX_ENV=development ../../oss/plugin-packaging/bin/dev force:package:beta:install --package
ERROR running force:package:beta:install:  Cannot read properties of undefined (reading 'abc')
*** Internal Diagnostic ***

Type: Cannot read properties of undefined (reading 'abc')
    at Function.wrap (/Users/william.ruemmele/projects/oss/command/node_modules/@salesforce/core/lib/sfError.js:61:25)
    at Install.catch (/Users/william.ruemmele/projects/oss/command/src/sfdxCommand.ts:404:27)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async Install._run (/Users/william.ruemmele/projects/oss/command/src/sfdxCommand.ts:201:7)
    at async Config.runCommand (/Users/william.ruemmele/projects/oss/plugin-packaging/node_modules/@oclif/core/lib/config/config.js:272:25)
    at async Object.run (/Users/william.ruemmele/projects/oss/plugin-packaging/node_modules/@oclif/core/lib/main.js:76:5)
Caused by: Type: Cannot read properties of undefined (reading 'abc')
    at installPackage (/Users/william.ruemmele/projects/oss/packaging/src/package/packageInstall.ts:41:14)
    at Package.install (/Users/william.ruemmele/projects/oss/packaging/src/package/package.ts:83:26)
    at Install.run (/Users/william.ruemmele/projects/oss/plugin-packaging/src/commands/force/package/beta/install.ts:146:46)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async Install._run (/Users/william.ruemmele/projects/oss/command/src/sfdxCommand.ts:198:34)
    at async Config.runCommand (/Users/william.ruemmele/projects/oss/plugin-packaging/node_modules/@oclif/core/lib/config/config.js:272:25)
    at async Object.run (/Users/william.ruemmele/projects/oss/plugin-packaging/node_modules/@oclif/core/lib/main.js:76:5)
******


Geo-redundant storage should be enabled for Storage Accounts

Benefit/Result/Outcome
To be compliance clients should use geo-redundancy to create highly available applications. Currently MLZ uses storage accounts for diagnostic logging so this is primarily to document this control for future storage use. A separate issue will be created for central diagnostic logging. Note: This is a commercial policy but should be across clouds.
Description
NIST SP 800-53 Rev. 4 CP-6 - The organization: a. Establishes an alternate storage site including necessary agreements to permit the storage and retrieval of information system backup information; and b. Ensures that the alternate storage site provides information security safeguards equivalent to that of the primary site.
Acceptance Criteria

Document this as a control requirement and setting this via TF as well as out of band scripting.
Decide whether to set diagnostic storage as redundant.

Closing - already implemented. Logs default to the Standard_GRS sku.


cache initial-style jaxpr transformations

These changes reduce the tracing time on the shoyer version of the code in #3847 from ~50s to ~2.5s.
~I'm a little wary to add more caching until we land #9188 to fix #9187.~ But otherwise this is straightforward!
I think we can say this fixes #3847, though there's a lot more improvement we could make in this area.

This was merged, but not marked as merged...


misc/cgo/testcshared: broken on linux-arm64-packet builder

The linux-arm64-packet builder is failing as of CL 198544 (#24715) (https://build.golang.org/log/bb6d9d6b204064abed1ed6589f858f8b2946dda8):
##### ../misc/cgo/testshared
shared_test.go:72: executing go install -installsuffix=5577006791947779410 -buildmode=shared runtime sync/atomic failed exit status 2:
# /workdir/tmp/go-build310193300/b009/libruntime,sync-atomic.so
runtime.emptyfunc·f: missing section for relocation target runtime.emptyfunc
runtime.emptyfunc·f: reloc 1 (R_ADDR) to non-elf symbol runtime.emptyfunc (outer=runtime.emptyfunc) 0 (Sxxx)
FAIL	misc/cgo/testshared	7.164s

The CL needs to be fixed (if the fix is trivial) or rolled back ASAP.
CC @tklauser @bradfitz @ianlancetaylor

Sorry about that. I'm working on a fix.


CodeCop warns about using parenthesis but gives an error when I do

Using property Page of a page part, I get the warning from CodeCop to use parenthesis:

But when I do that, I get an error message that I use the property Page as a function:

In general, I'm not happy with the many, many places that are treated by CodeCop as a function while it is not really a function. For example RecRef.Number. Is that a function? Not to me, but CodeCop advises to use parenthesis.

Hi @ajkauffmann, the error with Page seen as a method has already been reported in #1861 and has been fixed in our code base. Concerning RecRef.Number, even if it can look like the concept of property in some other languages, this is actually a procedure in AL, so it should be used with parenthesis if no arguments are specified.


Automate tw

We have to remove Tw as a free parameter and adjust the whole method to the fact, that L is not a meaningful absolute measure, but is well suited for being a relative measure in between reconstruction-cycles.

The biggest change is, that we have to compute L’s for many different Tw’s until this gap I was talking about shrinks. Codewise it means that in local_l_statistics I am not calling uzal_cost anymore Of course I could simply loop over all Tw’s and call uzal_cost with this changing Tw every single time. But uzal_cost Is averaging over T = 1:Tw anyway! So my idea in uzal_cost_pecuzal was to compute L for each Tw step by step for the actual embedding (L1) AND the candidate embedding (L2) and compute L2 - L1 in each step. Whenever this distance increases, the loop breaks (assumed that this distance is negative). This is all done in the function uzal_cost_pecuzal. Of course this slows down the whole thing. I encountered slowing factors of up to 6 so far, which is bad and maybe you can help me to improve this.
I tried with parametrizing all the auxiliary functions, but this had no effect at all.
The bottleneck is clearly the re-computation of all nearest neighbors in every itearation of T = 2:Tw in the main function uzal_cost_pecuzal. First I thought it would be sufficient to compute this only once at the beginning and use the same information about the nearest neighbors in each iteration of T, but it turned out that this will bias the results heavily. The problem is, that vtree is taking the trajectory Y[1:end-Tw]. If we would only perform the tree search once at the beginning than this is based on Y[1:end-Tw], but it should be based on Y[1:end-T], T = 2:Tw. Meaning, when I set the maximum Tw, which coincides with the maximum encountered delay values (this is a free parameter), to be large, say, Tw=200 then the results will change compared to the case where, say, Tw=100, even though the statistic the algorithm works on will only be based on, say, T = 4. So despite the fact that it will never actually get to Tw=100 or even Tw=200, it will give different results dependent on what Tw you set in the first place. That is why vtree needs to be computed all over again. That is the problem and maybe - hopefully - you have a solution for this? I know, that for large datasets that will literally have no effect, but even on intermediate sizes it has!

Hi @hkraemer I see that this is already merged. Is there something you'd still like me to do, in general for here or for the paper? If so please let me know.

Hi @hkraemer I see that this is already merged. Is there something you'd still like me to do, in general for here or for the paper? If so please let me know.


Nothing happens

So. I've installed Howdy AUR from software center. Everything was smooth.
Configured everything. All was smooth aswell.
Tested. Added face. Smooth.
Added those recommended lines to /pam.d/
And I do not see a single blink of an emitter when I am asked for password. What can be wrong?
Just nothing happens.
Garuda linux / xfce de

Did you configure Howdy with the sudo howdy config command?

Yep. And all was good> Did you configure Howdy with the sudo howdy config command?

Did you check if something is recorded in journalctl at the time when you try to use Howdy? Additionally, you can set Howdy (with the config file) to debug mode, so it will print out into a log file what's the problem.

Did you check if something is recorded in journalctl at the time when you try to use Howdy? Additionally, you can set Howdy (with the config file) to debug mode, so it will print out into a log file what's the problem.

What do you mean by term 'using'? Configure is done and has no issues, and this is all i can do with Howdy, because nothing else happens!

for example is you run sudo howdy test, does the window open and shows the picture of the cam? What does journalctl records when you run that command?

for example is you run sudo howdy test, does the window open and shows the picture of the cam? What does journalctl records when you run that command?

window does open and camera works just fine, I have added faces.
command:
sudo howdy test
Opening a window with a test feed
Press ctrl+C in this terminal to quit
Click on the image to enable or disable slow mode
QStandardPaths: XDG_RUNTIME_DIR not set, defaulting to '/tmp/runtime-root'
QStandardPaths: XDG_RUNTIME_DIR not set, defaulting to '/tmp/runtime-root'
^C
Closing window
I'm not so good with linux, and am not familiar with journalctl - what exactly i must command?

Great troubleshooting by @peterroth so far!
Where are you trying to activate Howdy? Have you tried using it with sudo?

I want Howdy to scan my face when password prompted. But Howdy does not activate, even though i added faces etc. Howdy works only to and add/test. That's all where it is active

Right, and if you run sudo -i you only get asked for a password and no other output

Right, and if you run sudo -i you only get asked for a password and no other output

exactly!
sudoing, logging in, updating app, etc etc - Howdy does not start

I have same issue

In a new Ubuntu 23.04 I also facing the same. Everything set up, but no launching on sudo or login. Probably something related to PAM

This worked for me: https://github.com/boltgolt/howdy/wiki/Howdy-on-Debian-Testing---KDE-Plasma#fix-the-script-for-debian-testing

This fixed the issue for me: https://github.com/boltgolt/howdy/wiki/Howdy-on-Debian-Testing---KDE-Plasma#fix-the-script-for-debian-testing

This worked for me. Thanks @NickFlamel

@NickFlamel  thanks mate that worked for me too on Kali Linux!

@NickFlamel Thank you SO much! Worked on Debian 12 LTS!


Avoid reloads in libxsmm_spmdm_compute_fp32_thread.

Because of aliasing, the compiler cannot prove that handle->n and
handle->m do not change. It has to reload them at every loop
iteration, wasting a significant time.
We have measured an improvement of 0-5% (median 1.6%) on our internal tensorflow
sparse matrix multiplication benchmarks.

Thank you very much for your contribution!

As you seem to be still interested in LIBXSMM's functionality (at least SPMDM), have you noticed this desaster: https://github.com/tensorflow/tensorflow/pull/24706?
I was urged to help adjusting things for an API change in TF (allocator stuff), then nobody looked at the PR for a long time, and finally I was asked to reformat the files. Of course, I would write new code adhering to any required (format-)policy. However, this was completely unrelated work.
Anyhow, the PR cotained the following:

Necessary fix to deal with TF's change to the allocation API. Mostly some cleanup as the change was on Xsmm's side to accomodate the revised TF API.
Updated code for integrating Xsmm's convolutions to at least work (again).
Enabled Xsmm's BFloat support for SpMDM.

Regarding the latter (3), Bfloat for SpMDM was always present since the first integration between TF and XSMM. However, at that time (TF v1.1) TensorFlow had no (public) support for Bf16 (likely because of no foundation in Eigen, etc.). My recommendatation would be to look at enabling Xsmm/Bf16 in TF.
Regarding (2), I think integration of convolution is outdated and can be removed. Main reason is that today, one would likey integrate Xsmm into Eigen and then leverage this work in TF. See also a placeholder about such work: https://github.com/hfp/libxsmm/issues/281. This is also true because both TF and Xsmm progressed for instance wrt code fusion.
Our next release of Xsmm (1.14) will contain your PR. Our upcoming v2.0 merge a completely revised DL implementation based on a universal batch-reduce kernel which comes with extensive support for low precision, future instruction set extensions, and combined layers as well as entirely new layer kinds.

Hi Hans,
I'm working on global performance, not TensorFlow specifically, so I do not know the exactly how the TF team handles pull requests. That being said I had a quick look at your pull request, and there did not seem to be any pending issue, so I'm wondering why it has not been merged. I'll ping  the TF team.

Will this change be in a release? That makes it easier to pull in for
TensorFlow.
-- Jeremiah Willcock
On Sun, Oct 20, 2019 at 11:44 PM Clement Courbet<EMAIL_ADDRESS>wrote:

Hi Hans,
I'm working on global performance, not TensorFlow specifically, so I do
not know the exactly how the TF team handles pull requests. That being said
I had a quick look at your pull request, and there did not seem to be any
pending issue, so I'm wondering why it has not been merged. I'll ping the
TF team.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/hfp/libxsmm/pull/355?email_source=notifications&email_token=AAX34KGWYR6XO62OCXJQSRLQPVFVFA5CNFSM4JCI3KJKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBZHO6I#issuecomment-544372601,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAX34KFC5FTMFW4MBEHVFFDQPVFVFANCNFSM4JCI3KJA
.

@jewillco : according to @hfp's comment, it will be in 1.14.

Our next release of Xsmm (1.14) will contain your PR.

This PR #355 is part LIBXSMM v1.14!
In addition, I have validated v1.14 against TF/master. For latter, I have locally updated https://github.com/tensorflow/tensorflow/pull/24706 to TF/master (as of 25th Oct. 2019) and compiled with LIBXSMM (--define tensorflow_xsmm_convolutions=1 --define tensorflow_xsmm_backward_convolutions=1 --define tensorflow_xsmm=1). I have tested in particular SpMDM:
bazel-bin/tensorflow/core/kernels/sparse_matmul_op_test --benchmarks=all
bazel-bin/tensorflow/core/kernels/sparse_matmul_op_test

Updating https://github.com/tensorflow/tensorflow/pull/24706 to current TF/master was straight-forward.


Robot flags aren't very visible from above

Let's just make the whole robot body their corner colour.

This was fixed in https://github.com/srobo/competition-simulator/pull/124


added halo profiles to ccl note

Description of halo profiles in CCL note

Pull Request Test Coverage Report for Build 4052

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

Totals

Change from base Build 4020:
0.0%

Covered Lines:
1790

Relevant Lines:
1865

💛  - Coveralls

Compiles locally for me. I think the only thing missing is the 2D NFW description. It seems we also have that capability from the docs:
https://ccl.readthedocs.io/en/v2.0.0/api/pyccl.haloprofile.html
@damonge are there any updates needed based on the new halo modelling capabilities?

OK, I'm gonna add information about the latest version of the halo profiles that just got merged into master and we'll take it from there.

OK, I've added to @KuanWang-Astro 's text and described the new halo profile functionality. @elisachisari if you're OK with it I'm happy to merge this.

Humm, the docs don't seem to be up to date. Let me have a look at rtd.

OK, here we go @elisachisari . The new profiles are documented here: https://ccl.readthedocs.io/en/master/api/pyccl.halos.profiles.html

I could add a line on the old profile docs clarifying that these are deprecated (people can use them, but we'll remove them eventually)

Yes, I think this is a good idea

OK, done! (should appear in RTD after merging)


Negative values after RELU when loading a VGG model and poping layers

Hi all,
I have experienced a problem while using this implementation of VGG16 :https://gist.github.com/baraldilorenzo/07d7802847aaad0a35d3
The network is not the problem, I have some great results with it when I try to classify ImageNet.
I needed to load the model with the pre-trained weights which can be found there :
https://drive.google.com/file/d/0Bz7KyqmuGsilT0J5dmRCM0ROVHc/view
Then, I wanted to use the output values of intermediate layers.
But when I look at an output coming after a "relu", it happens that some values are negative. When I compute the np.min() of the output, I get -9.59634780884e-06
Here is a script reproducing the error.
https://gist.github.com/leonardblier/ddb0cc75ad95ac8f644cf02c83cf82e7
I have tried to do the same thing, but without loading the weights, and the min of the output is 0.
Do you know where this problem could come from ? Am I doing a mistake while poping the layers ?
Bests !

The magic of floating-point math makes it possible that something like that might happen.

Yes, I know that this can be the source of problems. But here, what is weird is that the last operation is the "relu". Moreover, the values are all positive if I don't load the weights before.

@NasenSpray I also come across the same problem, and I'm sure it is not because of the float-point math. I did some test. The error of the previous one is relatively much larger.And If I don't use Relu in keras, but use theano directly, there is no such problem.
I also found someone reported strange things about Relu.
Please see #1948 for more details.

It seems to me that this issue has been solved in the Keras 1 version. I close this issue.

This seems to still be an issue when loading the pre-trained VGG19 model.  A convolutional layer in VGG19 that has a ReLU activation setting outputs small values around 0 rather than outputting 0.
I was able to get the expected ReLU outputs by essentially re-creating the VGG19 model from its pre-trained weights and breaking the ReLU functions into separate layers.  See code below for replicating the issue:
from vgg19 import VGG19
from keras.preprocessing import image
from imagenet_utils import preprocess_input
from keras.models import Model
from keras.models import Sequential
from keras.layers import Convolution2D
from keras.layers import MaxPooling2D
from keras.layers import Activation
import numpy as np
import urllib, cStringIO
from PIL import Image
from glob import glob
# Load image from the internet
URL = 'http://www.citypass.com/img-60fb651/products/seattle/attractions/spaceneedle/gallery/spaceneedle-gallery-city_sunset_light.jpg'
file = cStringIO.StringIO(urllib.urlopen(URL).read())
img = Image.open(file)
img = img.resize((224,224))
# Set-up model
base_model = VGG19(weights='imagenet')
# Original - relu in convolution layer
conv1 = Model(input=base_model.input, output=base_model.get_layer('block1_conv1').output)
# Separated
# Convolution
model = Sequential()
conv1_layer  = base_model.layers[1]
model.add(Convolution2D(conv1_layer.nb_filter, conv1_layer.nb_row, conv1_layer.nb_col, border_mode='same', input_shape=conv1_layer.input_shape[1:4]))
model.layers[0].set_weights(base_model.layers[1].get_weights())
# ReLU Activation
act_model = Sequential()
act_model.add(Activation(activation='relu', input_shape=base_model.layers[1].output_shape[1:4]))
# Make predictions
x = image.img_to_array(img)
x = np.expand_dims(x, axis=0)
x = preprocess_input(x)
base_acts = conv1.predict(x)
conv_out = model.predict(x)
separated_acts = act_model.predict(conv_out)
# Compare activations
# The number of 0 valued outputs is much different for the first convolutional layer
# is much different than if you break it up into a convolutional step and then a ReLU step.
(base_acts == 0).sum()
(separated_acts == 0).sum()
# These values will be the same, showing that the outputs are very close to 0, just not exactly 0.
(np.absolute(base_acts) < 0.0001).sum()
(np.absolute(separated_acts) < 0.0001).sum()


JS sample for batch upload

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: v-geberr sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Documentation removed

It appears in the latest periodic update that a large number of documentation files were removed: https://github.com/awsdocs/iam-user-guide/commit/2b1fa3805ebb57dfd1d9188ab1f5da7ee0a5b76c
For example the reference_policies_elements* files have gone, so the documentation for this page: https://docs.aws.amazon.com/IAM/latest/UserGuide/reference_policies_elements_condition.html appears to be missing.

Thanks! We're investigating what happened so it doesn't happen again. The docs will be updated shortly.


Putting Video in slides causes same videos to play

Putting video or other video components, causes the same videos to repeat. The first video plays on the 3rd and that repeats,

Did you check closed issue #102? There is example with adding videos to slider from repo owner. If you view it, launch video on first slide and go to the next one. You will see that it needs 1 more step to remove it from buffer by itself (tl;dr on slide with number 8 you will still hear video playing).
Now if you overwrite example with code from snippet 1
(I've added some samples and put the same video from first slide to third one)

snippet 1
import React from "react";
import AwesomeSlider from "react-awesome-slider";
import "react-awesome-slider/dist/styles.css";
import "./styles.css";

export default function App() {
  return (

React Awesome Slider

      <AwesomeSlider className="awesome-slider">

          <video
            style={{
              width: "100%",
              height: "100%",
              position: "absolute",
              top: 0,
              left: 0,
              objectFit: "cover"
            }}
            controls
            src="https://archive.org/download/Popeye_forPresident/Popeye_forPresident_512kb.mp4"
          />

8

          <video
            style={{
              width: "100%",
              height: "100%",
              position: "absolute",
              top: 0,
              left: 0,
              objectFit: "cover"
            }}
            controls
            src="https://archive.org/download/Popeye_forPresident/Popeye_forPresident_512kb.mp4"
          />

          <video
            style={{
              width: "100%",
              height: "100%",
              position: "absolute",
              top: 0,
              left: 0,
              objectFit: "cover"
            }}
            controls
            src="http://commondatastorage.googleapis.com/gtv-videos-bucket/sample/ForBiggerEscapes.mp4"
          />

          <video
            style={{
              width: "100%",
              height: "100%",
              position: "absolute",
              top: 0,
              left: 0,
              objectFit: "cover"
            }}
            controls
            src="http://commondatastorage.googleapis.com/gtv-videos-bucket/sample/ForBiggerBlazes.mp4"
          />

42

      </AwesomeSlider>

  );
}

behaviour works as expected if you consider the same videos. It needs 1 more step to pause the video from the first slide but third one contains the same video. Therefore it will still play when moving to it. And if you continue on changing slides via right arrow, you will note the same rule. If you move from slide(3) to slide(5) video will stop playing.


supabase-ssr: Failed to execute 'postMessage' on 'BroadcastChannel'

Bug report

[x] I confirm this is a bug with Supabase, not with my own application.
[x] I confirm I have searched the Docs, GitHub Discussions, and Discord.

Describe the bug
Using the supabase-ssr library with Nuxt 3, I'm getting the following error:
DOMException: Failed to execute 'postMessage' on 'BroadcastChannel': #<Object> could not be cloned.
    at SupabaseAuthClient._notifyAllSubscribers (http://localhost:3000/_nuxt/node_modules/.cache/vite/client/deps/chunk-A274CJAX.js?v=7251ccc1:2156:31)
    at SupabaseAuthClient._recoverAndRefresh (http://localhost:3000/_nuxt/node_modules/.cache/vite/client/deps/chunk-A274CJAX.js?v=7251ccc1:2101:20)
    at async http://localhost:3000/_nuxt/node_modules/.cache/vite/client/deps/chunk-A274CJAX.js?v=7251ccc1:2351:11

I'm trying to pin down where the error comes from, it could be related to serializing cookies. I hope to get help, determine if this error has an impact on auth, and ideally fix it.
To Reproduce
Steps to reproduce the behavior, please provide code snippets or a repository:
I'll try to provide a reproduction, but since this error is auth related, I can't provide the actual code base.
Expected behavior
No postMessage / broadcast error to occur.
Screenshots

System information
Operating System: Linux
Node Version: v18.18.0
Nuxt Version: 3.9.3
CLI Version: 3.10.0
Nitro Version: 2.8.1
Package Manager<EMAIL_ADDRESS>Additional context
Add any other context about the problem here.

I have the exact problem.. any solutions?


GitHub username

Please fix failing tests.

Coverage decreased (-0.04%) to 89.556% when pulling 024a2b89461853f5742304ba69f665e67bbabb50 on akatsoulas:github-username into 86fc079cca1daf3c3f6728ea26bb8b83769484db on mozilla:master.


refact: use temporaryFolder() rule

instead of manual folders, use the TemporaryFolder() which is supposed to be removed after the test finishes.
fix issue with setting the config in one test.

Coverage decreased (-0.07%) to 75.873% when pulling 0d2c3137100a3f1ddcec5617085cd38d126140cf on test/updateTempFolder into 37c8790cc03368d308687457b15909cc03325cae on master.

Apologies @bmeike, i will make changes soon. Currently Android Studio is showing some error when building it. I will debug and fix that soon.

Why is this not merged?  @jayahariv, can you please get this in?


Order of intervals can break striplog

The problem in the to_log() function, which is consistently getting the indices (start_ix, stop_ix, top_index, base_index) backwards in this dataset.
I think the problem is that we need to reverse indices when order is 'elevation'.
I think this affects to_log() and fill() at least.
See https://swung.slack.com/archives/C01FC2V9NVD/p1616330076013100

@kwinkunks this is resolved as of https://github.com/agile-geoscience/striplog/commit/527fe88aa96062f804668cfabdff768e8dd889d9, yes? Looking to see if I can close this issue.

Let's say this is fixed.


can't stop pyspider

Yesterday I crawl a website, it crawl too much data, today I restart pyspider, and it still crawl  yesterday's website, I set the status 'Stop' and 'Debug', but don't work for me

Do you mean recrawl the website from beginning? there is no way to do that, recreate a project.

Yes, it always recrawl, but I found a way that I just deleted the 'data' directory and so the whole project was cleaned away, so I can stop crawling


Add is_palindrome variations

So, I was going through our repo and found that we only have one variation of is_palindrome. So I decided to add a couple more. BTW I opened a PR request from my repo's master but @goswami-rahul  told me to do it from a separate branch so I have closed that request. Plus, I also narrowed the commits to just a single one.
Also, the algorithms can be found on geeksforgeeks.com. I thought it'd be nice to have different versions for the readers.

I have some requests. It is palindrome not palidrome. Please change the commit name and title PR too.
Using the stack to solve is also good idea too. I mean putting the half of string (from middle to end)to stack, and then poping each element to compare each element of string from start to middle. Could you add it when you have a chance?

@danghai That is what I am doing. I deleted my old PR because it was from my local master. This is from a different branch. I like the idea of stack but IMHO that is unnecessary. You gotta create a stack class for that which would make the solution awfully long. But, you're right with Stack.
I think the name of the file is correct. @goswami-rahul  was the one who created this file so I don't need to change the name. As far commit is concerned, whoever is merging it can just use the squash feature of the merge pull request button.

@SaadBenn  for stack, I mean declare stack = [] , and then use stack.append(), and stack.pop. We do not need to write class for stack.

@SaadBenn It is good

Alright I'll merge it then, thanks for the review.  @danghai


Really weird issue when svgo is executed in a subshell from Python

I'm on Windows 7, Node.js v0.10.38, svgo v0.5.1, I installed svgo globally with: npm install -g svgo
Here is my svgo command:
svgo C:\_projects\_test\car.svg C:\_projects\_test\car.min.svg
Here is what I experience:

When I execute the above command via Windows command prompt (CMD.exe) it works fine.

When I execute it from my Python script like this:
subprocess.call(['svgo', 'C:\_projects\_test\car.svg', 'C:\_projects\_test\car.min.svg'], shell=True)
then it still works fine.

But I want to capture the STDERR output (if any) so I try to execute it like this from my Python script:
p = subprocess.Popen(['svgo', 'C:\_projects\_test\car.svg', 'C:\_projects\_test\car.min.svg'], stderr=subprocess.PIPE, shell=True)
err = p.communicate()[1]
result = p.returncode
then it does not work! svgo starts to run that's for sure but the created output file is 0 bytes long!

I have no idea why svgo fails this way (creates a 0 bytes long output file) when its STDERR output is redirected (PIPE'd) to my Python program. I do run other Node.js CLI programs the very same way such as uglifyjs, cleancss, html-minifier but those other programs work fine even when I launch them from Python via the subprocess.Popen method. Only svgo fails this way.
Note: The same issue is present with svgo 0.5.0, I did not test it with older versions.

I have no idea, svgo uses standard console.error() method, so if it doens't go to STDERR, I guess it should be addressed to node.js.

Yeah, but the main problem is that the resulting minified(!) file is 0 bytes long. (Not talking about the STDERR output here.)
So I'm guessing here, maybe because svgo somehow detects that STDERR is being PIPEd, it is maybe trying to read the input (original svg data) from STDIN too?? This would explain the 0 bytes long minified file output.
Again, the other minifier CLI tools (e.g. uglifyijs, etc.) are also written in node.js and they work just fine when I run them from Python's subprocess.Popen method. Only svgo fails, it does not minify the specified input file, it creates a 0 bytes long minified file instead. Which I would say is a major problem.

So I'm guessing here, maybe svgo somehow detects that STDERR is being PIPEd then in turn it may try to get the input (original svg data) from STDIN as well?

No, svgo does nothing of that.
Btw, have you tried node 0.12? Or io.js. node.js may have some issues.

Not, it is not realated to node version.
I've made really good progress on this issue, here are my findings:
By entering the following command into a Windows command prompt (CMD.exe) :
svgo car.svg car.min.svg 2 > null
I get the zero bytes long minified file as a result and I can see an error message on the screen which is coming from svgo I assume:
Unknown argument: 2
But if I run this:
svgo car.svg car.min.svg 2> null
Notice the difference: there is no space between 2 and >.
So maybe on Windows, when shell=True provided for Python's pOpen function then Python places a space there between the 2 and the > characters? I'm not sure about this, but this would be a reasonable explanation.
Another thing which I found out is this:
If I do redirect both STDOUT and STDERR like this:
p = subprocess.Popen(['svgo', 'C:\_projects\_test\car.svg', 'C:\_projects\_test\car.min.svg'], stdout=subprocess.PIPE, stderr=subprocess.STDOUT, shell=True)
then I do not encounter the problem!
I do not exactly know why, but at least it gives me a workaround solution.

Well, it looks like Windows' peculiarities.

Because I found a workaround, I close this.


bugfix：error in node 0.12.7

for issues TypeError: Object.keys called on non-object #88
modify the order of condition.

Coverage remained the same at 93.592% when pulling 17a55be98fcc3d07f306aad58cfb8a7a55c8a473 on 81735595:master into 6971e27a577d165cde360ebed86a59dfc18ac55b on iarna:master.

Coverage remained the same at 93.592% when pulling 17a55be98fcc3d07f306aad58cfb8a7a55c8a473 on 81735595:master into 6971e27a577d165cde360ebed86a59dfc18ac55b on iarna:master.

Coverage remained the same at 93.592% when pulling 17a55be98fcc3d07f306aad58cfb8a7a55c8a473 on 81735595:master into 6971e27a577d165cde360ebed86a59dfc18ac55b on iarna:master.

Coverage remained the same at 93.592% when pulling 17a55be98fcc3d07f306aad58cfb8a7a55c8a473 on 81735595:master into 6971e27a577d165cde360ebed86a59dfc18ac55b on iarna:master.

An error in all versions of node, I expect. Thank you!


Implement dynamic meta tags

Description
This Pull Request implements dynamic meta tags for all plays using react-helmet.
Fixes #76
Type of change

[x] New feature (non-breaking change which adds functionality)

How Has This Been Tested?
Inspecting the  section in the document inspector or copying and pasting the current url into a platform that supports embeds should let anyone test the new functionality.
On a local server though, you will have to use an external browser extension to verify that the meta tags are working as expected.
I didn't write any tests or make any changes to the documentation, as I didn't think it was necessary.
Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

Notes:
I should mention that while I was testing the meta tags on localhost, I had to provide full paths for the meta tag (ex: "www.reactplay.io/static/exampleimage12345.png"), and I removed the domain from url before the final commit. If this causes the meta image to not work it should be super easy to fix, but I just thought I should note it down here.
I hope the new changes are good, let me know if I need to anything else !!

Hello @erayalkis
We have upgraded to React v18 as part of this issue: https://github.com/atapas/react-play/issues/87
We have tested the existing plays. All good. Please pull this change to your branch and keep it up-to-date. You have to do sanity of things to make sure everything works for your changes in your branch.
Happy Coding!

I'm on it! Will ensure that everything works and push the updated version ASAP

All done, everything works as intended!

The new components introduce no UI or logic changes, so I had to test it manually using an external app, it's why the links look different on the screenshots. All of the pages look the same:

And when you post a link of a play on a platform that supports embeds (I personally used discord here), it looks like this:

If there's no cover image available, it just uses the default one instead.

@all-contributors please add @erayalkis  for Code

Will do! :)


Run in existing shell OR create interactive child shell

It would be useful to run a command within the existing shell and/or to create an interactive child shell.
Specifically, I'm trying to create the following recipe for Python, which switches my environment into a specific virtual environment:
activate:
  source venv/bin/activate

However, because Just is running commands in their own shell instance, this executes the activation in that shell instance, then exists that shell.
If there were a way to tell Just to execute the command in the existing shell, this would operate the same as a normal alias, which is the ideal. If that is not possible, it would be nice if the new shell instance used to run the command could be made into an interactive child shell instance, which would achieve nearly the same effect.

It looks like the follow works:
activate:
  bash -i <<< 'source venv/bin/activate; exec </dev/tty'

However, I can only get it to work for bash, not zsh. If there's ever a way to build the functionality in, that would be ideal. But for now this is a good enough workaround.

This is a longstanding request, and unfortunately, I don't think there's a good way to support this, since doing so would probably require different commands for each shell, and just tries to be shell-agnostic.

It looks like the follow works:
activate:
  bash -i <<< 'source venv/bin/activate; exec </dev/tty'

However, I can only get it to work for bash, not zsh. If there's ever a way to build the functionality in, that would be ideal. But for now this is a good enough workaround.

@fastfedora found a way to get this to work:
activate-venv:
  zsh -is <<< 'source .venv/bin/activate; exec </dev/tty'

I also found this oh-my-zsh plugin: https://github.com/ohmyzsh/ohmyzsh/tree/master/plugins/python#virtual-environments

@fastfedora thanks for this, found a way to get this to work in zsh:
activate-venv:
  zsh -is <<< 'source .venv/bin/activate; exec </dev/tty'

Thanks so much. That works for me too.

I also found this oh-my-zsh plugin: https://github.com/ohmyzsh/ohmyzsh/tree/master/plugins/python#virtual-environments

I'll check this out. Thanks!


kubectl ws ~ fails

Describe the bug
kubectl ws ~ is supposed to switch to my home workspace (which does exist and is accessible via a different command).  Instead it gives an error message and makes no change.
(base) mspreitz@mjs12 kcp % kubectl ws use root
Current workspace is "root".

(base) mspreitz@mjs12 kcp % kubectl ws ~
Error: invalid resource name "/Users/mspreitz": [may not contain '/']

(base) mspreitz@mjs12 kcp % kubectl ws current
Current workspace is "root".

(base) mspreitz@mjs12 kcp % kubectl ws
Current workspace is "root:users:zu:yc:kcp-admin".
Note: 'kubectl ws' now matches 'cd' semantics: go to home workspace. 'kubectl ws -' to go back. 'kubectl ws .' to print current workspace.

To Reproduce
See above
Expected behavior
I expected kubectl ws ~ to do the same thing as kubectl ws.
Additional context
kcp commit 8a36261dd4a653961bd9029eb29f7dd01172147b .
MacOS 12.4 on Intel.
go version 1.18.4.

That's probably due to the fact that the ~ character is expanded by the system.
On Linux and Mac, using quotes to block expansion works : kubectl ws '~'
That's not nice but in this case the simple kubectl ws is there.

D'Oh!
(base) mspreitz@mjs12 kcp % kubectl ws '~'
Current workspace is "root:users:zu:yc:kcp-admin".


Report Settings should be accessible at all time

Now that open surveys are editable, we should be able to access the report settings while the engagement is open.
Superusers only (TBC by Josh)
Add "Edit Settings" in the action drop-down for Open Surveys.
Change the button on the survey builder from "Save & Continue" to "Report Settings".
Add a "Back" button to the report Settings page to return to the survey builder.


fix: update pnpm version to 8

Thanks for the PR - unfortunately I didn't notice it until after applying the same fix.


Add support for K for state elimination

Add support for K for state elimination. See:
https://www.sciencedirect.com/science/article/pii/S0005109812001069

Issue resolve by pull requests 23, 24, and 25


https://http3check.net/ is not happy

Hey,
In the context of investigating https://github.com/traefik/traefik/issues/8255 , I ran a trivial HTTP/3 server, such as:
package main

import (
	"net/http"

	"github.com/lucas-clemente/quic-go/http3"
)

type myhandler struct{}

func (mh myhandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
	w.Write([]byte("HELLO"))
}

func main() {
	http.Handle("/", myhandler{})
	http3.ListenAndServe(":443", "./cert.pem", "./key.pem", nil)
}

and it turns out that https://http3check.net/ is not happy about it, and it displays a result like:

IMHO, this is a pretty bad report as they don't explain what the problem(s) actually are, but here we are.
My naive suspicion is maybe they expect higher draft versions (h3-30, h3-31, ... , h3-34) in the Alt-Svc, but who knows...
What do you think?
Relatedly, I'm puzzled by this part of your README:
It has support for HTTP/3 (RFC 9114), including QPACK (RFC 9204). In addition the RFCs listed above, it currently implements the IETF QUIC draft-29.
I would think that when one has fully implemented RFC 9114, it supersedes the implementation h3-29, no? How can one say that "only" h3-29 is implemented (i.e. none of the later drafts), and at the same time that RFC 9114 is implemented?
Is it that I'm too literal, and would it be correct to say that "RFC 9114 is partially implemented, as none of h3-30, ... h3-34 are implemented" ?
Then again, it's completely possible that I misunderstood something, and I would be very happy to be enlightened on the subject. :-)
Thanks

How can one say that "only" h3-29 is implemented (i.e. none of the later drafts), and at the same time that RFC 9114 is implemented?

Nobody is saying that only h3-29 is implemented. quic-go implement h3-29, QUIC v1 and QUIC v2 at the same time.
The HTTP header looks correct, it shows QUIC v1 support (h3=...). This is probably a bug in http3check, not in quic-go. Might be worth raising an issue with them.

I shared link to this issue on their slack

How can one say that "only" h3-29 is implemented (i.e. none of the later drafts), and at the same time that RFC 9114 is implemented?

Nobody is saying that only h3-29 is implemented. quic-go implement h3-29, QUIC v1 and QUIC v2 at the same time.

Sorry, it looks like I haven't been able to make myself clear, so please allow me to reformulate. What's the difference in saying that:

RFC 9114 is implemented
VS
IETF QUIC draft-29 is implemented
VS
IETF QUIC draft-34 is implemented
?

To me they are all different versions of the same thing, no?

The HTTP header looks correct, it shows QUIC v1 support (h3=...). This is probably a bug in http3check, not in quic-go. Might be worth raising an issue with them.

Good to know, thanks.

To me they are all different versions of the same thing, no?

There's minor differences between these versions. A QUIC server is free to implement as many versions as it wants.

To me they are all different versions of the same thing, no?

There's minor differences between these versions. A QUIC server is free to implement as many versions as it wants.

Ah ok, so let me see if I finally fully understand: IETF QUIC draft-29 and RFC 9114 are indeed both specifications for HTTP/3, slightly different from one another (with RFC 9114 being the most recent one), and quic-go implements both of them? And it advertises it in the Alt-Svc, so that it can comply with a client that speaks either of them?
If yes, then I agree that it is no cause for http3check.net to "complain". I could see it reporting an error if quic-go implemented only draft-29, but since it also implements RFC 9114...

Ah ok, so let me see if I finally fully understand: IETF QUIC draft-29 and RFC 9114 are indeed both specifications for HTTP/3, slightly different from one another (with RFC 9114 being the most recent one), and quic-go implements both of them? And it advertises it in the Alt-Svc, so that it can comply with a client that speaks either of them?

Correct.

@mpl Have you raised an issue with http3check? Is there anything left to do in quic-go?

No I haven't, and I won't be able to until tomorrow.
But I leave it up to your judgement if you want to close the issue right
away.
On Mon, Aug 22, 2022, 10:45 Marten Seemann @.***> wrote:

@mpl https://github.com/mpl Have you raised an issue with http3check?
Is there anything left to do in quic-go?
—
Reply to this email directly, view it on GitHub
https://github.com/lucas-clemente/quic-go/issues/3509#issuecomment-1222043712,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEY3F3NJVJTWF2V52KZWR3V2M4ZHANCNFSM565ZVP4Q
.
You are receiving this because you were mentioned.Message ID:
@.***>

I'll close the issue, but it would be cool if you post the link to the http3check issue here once you've opened it!

@marten-seemann welp, they don't seem to advertise any public repo or bug tracker, and @mradalbert already talked to them on slack apparently, so I don't think there's much more I can do for now. If they don't want open and public feedback, so be it...

(In my previous experiences it seems that http3check.net only has the interest of matching whatever LiteSpeed currently supports, which makes sense since they're owned by LiteSpeed Technologies.)

LiteSpeed seems to have corrected its test page. Both tests are now successful for my website with caddy v2.6.4:


auth: unit-test for authStore.AuthDisable()

This will cover unit-test for AuthDisable in store.go
/cc @mitake @xiang90

LGTM. Defer to @mitake

lgtm, thanks @Rushit
@xiang90 BTW, can I ignore jenkins-proxy-ci? I couldn't see something is going under this PR (the link of Detailed returned 404)

can I ignore jenkins-proxy-ci?

ignore it for now.


Azure Core linter rules for unions

There is plenty of union cases that are not nicely representable in many languages, I don't believe we have any linter rules today guiding here.
union A {
  string,
  {name: string}
}

Agreed to the following linting rules:

Discourage fixed enums and unions
Discourage anonymous types in variants


Replying to a reply does not reply. Instead it does create a normal tweet

Any idea, why this will not work?
public async Task<bool> ReplyAsync(long tweetId, string replyMessage)
{
    var tweetsToReplyTo = await _twitterClient.Tweets.GetTweetsAsync(new[] {tweetId}).ConfigureAwait(false);
    var tweetToReplyTo = tweetsToReplyTo?.FirstOrDefault();
    if (tweetToReplyTo == null)
    {
        return false;
    }

    await _twitterClient.Tweets.PublishTweetAsync(new PublishTweetParameters(replyMessage)
    {
        InReplyToTweet = tweetToReplyTo
    }).ConfigureAwait(false);
    return true;
}

Instead of replying to existing tweets it just creates a normal tweets. Do I lack some info?
The tweetId passed is fine, it does give me a tweet and it does not return false..

Hello there,
So the problem comes from the fact that you need to have an @<username> at the start of a reply now. This has been a change on the Twitter API side.
Here is how a code could look like :
var tweet = await userClient.Tweets.GetTweetAsync(1315345368717066240);
var reply = await userClient.Tweets.PublishTweetAsync(new<EMAIL_ADDRESS>Reply to<PHONE_NUMBER>717066240")
{
    InReplyToTweet = tweet
});

Twitter no longer authorises to create a reply without the @username so you will need to get the tweet before being able to send a reply.
I will update the documentation to reflect this.

Documentation updated.


Hamburger Menu on Navbar should automatically hide when user click anywhere on the screen

Description of the bug. ✍🏻
Hamburger Menu on Navbar should automatically hide when the user clicks anywhere on the screen...I want to use useRef hook for this to fix this issue so kindly assign this issue to me as a GSSOC23 contributor..By the way Fun fact: The website looks cool to me I can't find any issue here other than this issue..
screencast-www.linkedin.com-2023.06.27-23_24_42.webm
Current Behaviour
....
Expected Behaviour
.....
Reproduction Steps
No response
Screenshots
No response
Additional information
No response
Contributor terms

[X] I agree to follow the Contributing Instructions

Please assign this to me

@Utkarsh7368 Assigned.


Could you please update to version 2.0

Thank you!

You can find a forked version of this project that builds newer versions of scrcpy here


Support terms with object values in context

such as
@context: {
...
hydra:property: { @type: @vocab }
}

Would this allow for something like
{
   "@context":
   {
      "image": {
         "@id": "http://schema.org/image",
         "@type": "@id"
      }
   },
   "image": "http://manu.sporny.org/images/manu.png"
}

or is that already possible?
Thanks for your work.


ERROR: /etc/wireguard/wstunnel.sh: line 119: dig: command not found

Describe the bug
[#] source /etc/wireguard/wstunnel.sh && pre_up myvpn
/etc/wireguard/wstunnel.sh: line 119: dig: command not found
[#] ip link delete dev myvpn
Cannot find device "myvpn"

To Reproduce

Set up Wireguard on vps and client
Set up wstunnel on vps and client

I get the above error when I run $ sudo wg-quick up myvpn on my archlinux machine. Any ideas what's going wrong?
Desktop (please complete the following information):

OS:  Arch Linux x86_64
Browser. Brave
Version 6.4.12-arch1-1

Additional context
Wireguard works fine without wstunnel.

Hello,
Your issue is not related to wstunnel.
Don't know were you got your script, but you are missing the dig command line tool. (look into your distro how to install it)


xSccmSqlSetup: Added parameter to support TempDB files

Pull Request (PR) description
Added parameter to specify the TempDB file count, size, and growth (for sql server 2016 and above) during an sql server installation.
This Pull Request (PR) fixes the following issues
Task list

[X] Added an entry to the change log under the Unreleased section of the
file CHANGELOG.md. Entry should say what was changed and how that
affects users (if applicable), and reference the issue being resolved
(if applicable).
[X] Resource documentation added/updated in README.md.
[X] Resource parameter descriptions added/updated in README.md, schema.mof
and comment-based help.
[X] Comment-based help added/updated.
[ ] Localization strings added/updated in all localization files as appropriate.
[ ] Examples appropriately added/updated.
[ ] Unit tests added/updated. See DSC Community Testing Guidelines.
[ ] Integration tests added/updated (where possible). See DSC Community Testing Guidelines.
[ ] New/changed code adheres to DSC Community Style Guidelines.

This change is

Hey @CCWT! I was looking at this earlier and wanted to verify whether your intent was to close this PR?

Hey @CCWT! I was looking at this earlier and wanted to verify whether your intent was to close this PR?


Add quality step with sonarcloud

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Unset commit template

There is no way to unset the commit template, as the input dialog is a required field you are not allowed to submit an empty string.

Users can clear the commit template:
shortcut: cmd-shift-alt-c / ctrl-shift-alt-c
command palette: Atom Commit Templates: Clear Commit Template