Transitioning a TabBarView manually doesn't honor the duration and curve

Steps to Reproduce
Calling TabController.animateTo with a duration and curve doesn't affect the behavior of the TabBarView animation. Although it's correctly affecting the behavior of the TabPageSelector and the TabBar animations.
Sample: https://gist.github.com/osama-lionheart/bab886d774f4e87b77104835edffbc49
Video: https://youtu.be/FzRBgflZrR4
Flutter Doctor
[✓] Flutter (Channel master, v0.3.3-pre.10, on Mac OS X 10.13.4 17E199, locale en-US)
[✓] Android toolchain - develop for Android devices (Android SDK 27.0.3)
[✓] iOS toolchain - develop for iOS devices (Xcode 9.3)
[✓] Android Studio
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
[✓] Android Studio (version 3.1)
[✓] VS Code (version 1.22.2)
[✓] Connected devices (1 available)

• No issues found!

Sorry for the late response.
Is this still an issue?

Yes. I just updated flutter to the latest master channel and I'm still observing the same issue.
Here's my latest flutter doctor output:
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel master, v0.5.8-pre.61, on Mac OS X 10.12.6 16G1408, locale en-US)
[✓] Android toolchain - develop for Android devices (Android SDK 28.0.1)
[✓] iOS toolchain - develop for iOS devices (Xcode 9.2)
[✓] Android Studio (version 3.1)
[✓] VS Code (version 1.25.1)
[✓] Connected devices (1 available)

• No issues found!

I have the same issue.

I'm facing the same issue. Are there any updates on this?

Code Sample
import 'dart:async';

import 'package:flutter/material.dart';

void main() => runApp(MaterialApp(home: MyTabbedPage()));

class MyTabbedPage extends StatefulWidget {
  const MyTabbedPage({Key key}) : super(key: key);
  @override
  _MyTabbedPageState createState() => _MyTabbedPageState();
}

class _MyTabbedPageState extends State<MyTabbedPage>
    with SingleTickerProviderStateMixin {
  final List<Color> _colors = const [
    Colors.red,
    Colors.green,
    Colors.blue,
  ];

  TabController _tabController;
  Timer _timer;

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

    _timer = Timer.periodic(Duration(seconds: 5), (timer) {
      var index = (_tabController.index + 1) % _tabController.length;
      _tabController.animateTo(
        index,
        duration: Duration(seconds: 2),
        curve: Curves.easeInOut,
      );
    });
  }

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

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Column(
        crossAxisAlignment: CrossAxisAlignment.center,
        children: <Widget>[
          Expanded(
            child: TabBarView(
              controller: _tabController,
              children:
                  _colors.map((color) => Container(color: color)).toList(),
            ),
          ),
          Flexible(
            flex: 0,
            child: Padding(
              padding: EdgeInsets.symmetric(vertical: 32.0),
              child: TabPageSelector(
                controller: _tabController,
                color: Colors.grey.shade300,
              ),
            ),
          ),
        ],
      ),
    );
  }
}

flutter doctor -v
[✓] Flutter (Channel dev, v1.18.0-6.0.pre, on Mac OS X 10.15.4 19E287, locale
    en-GB)
    • Flutter version 1.18.0-6.0.pre at /Users/taha/Code/flutter_dev
    • Framework revision 84c84fb249 (31 hours ago), 2020-04-20 21:35:01 -0400
    • Engine revision 2bde4f0ae4
    • Dart version 2.9.0 (build 2.9.0-1.0.dev a12c36dd97)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at /Users/taha/Code/sdk
    • Platform android-29, build-tools 29.0.3
    • ANDROID_HOME = /Users/taha/Code/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_212-release-1586-b4-5784211)
    • All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 11.4.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.4.1, Build version 11E503a
    • CocoaPods version 1.9.1
[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 3.6)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 45.1.1
    • Dart plugin version 192.7761
    • Java version OpenJDK Runtime Environment (build
      1.8.0_212-release-1586-b4-5784211)
[✓] VS Code (version 1.44.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.9.1
[✓] Connected device (4 available)
    • SM M305F   • 32003c30dc19668f • android-arm64  • Android 10 (API 29)
    • macOS      • macOS            • darwin-x64     • Mac OS X 10.15.4 19E287
    • Chrome     • chrome           • web-javascript • Google Chrome
      81.0.4044.113
    • Web Server • web-server       • web-javascript • Flutter Tools
• No issues found!

Issue replicable on latest stable and master versions.

stable and master flutter doctor -v
[✓] Flutter (Channel stable, 1.22.5, on Mac OS X 10.15.4 19E2269 darwin-x64,
    locale en-IN)
    • Flutter version 1.22.5 at /Users/dhs/documents/Fluttersdk/flutter
    • Framework revision<PHONE_NUMBER> (7 days ago), 2020-12-10 11:54:40 -0800
    • Engine revision ae90085a84
    • Dart version 2.10.4

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

[✓] Xcode - develop for iOS and macOS (Xcode 12.0.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.0.1, Build version 12A7300
    • CocoaPods version 1.9.3

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 46.0.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)

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

[✓] Connected device (1 available)
    • SM G975F (mobile) • RZ8M802WY0X • android-arm64 • Android 10 (API 29)

• No issues found!

[✓] Flutter (Channel master, 1.26.0-2.0.pre.64, on Mac OS X 10.15.4 19E2269
    darwin-x64, locale en-IN)
    • Flutter version 1.26.0-2.0.pre.64 at
      /Users/dhs/documents/Fluttersdk/flutter
    • Framework revision 9ff4326e1f (26 hours ago), 2020-12-17 06:19:05 +0000
    • Engine revision 6edb402ee4
    • Dart version 2.12.0 (build 2.12.0-157.0.dev)

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

[✓] Xcode - develop for iOS and macOS (Xcode 12.0.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.0.1, Build version 12A7300
    • CocoaPods version 1.9.3

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

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 46.0.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)

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

[✓] Connected device (3 available)
    • sdk gphone x86 arm (mobile) • emulator-5554 • android-x86    • Android 11
      (API 30) (emulator)
    • macOS (desktop)             • macos         • darwin-x64     • Mac OS X
      10.15.4 19E2269 darwin-x64
    • Chrome (web)                • chrome        • web-javascript • Google
      Chrome 87.0.4280.88

• No issues found!

Same issue here with Flutter 2.0.3

Still reproducible on stable 2.8 and master 2.9.

code sample (null safe)
import 'dart:async';

import 'package:flutter/material.dart';

void main() => runApp(MaterialApp(home: MyTabbedPage()));

class MyTabbedPage extends StatefulWidget {
  const MyTabbedPage({Key? key}) : super(key: key);
  @override
  _MyTabbedPageState createState() => _MyTabbedPageState();
}

class _MyTabbedPageState extends State<MyTabbedPage>
    with SingleTickerProviderStateMixin {
  final List<Color> _colors = const [
    Colors.red,
    Colors.green,
    Colors.blue,
  ];

  late TabController _tabController;
  late Timer _timer;

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

    _timer = Timer.periodic(Duration(seconds: 3), (timer) {
      var index = (_tabController.index + 1) % _tabController.length;
      _tabController.animateTo(
        index,
        duration: const Duration(seconds: 2),
        curve: Curves.easeInOut,
      );
    });
  }

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

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Column(
        crossAxisAlignment: CrossAxisAlignment.center,
        children: <Widget>[
          Expanded(
            child: TabBarView(
              controller: _tabController,
              children:
                  _colors.map((color) => Container(color: color)).toList(),
            ),
          ),
          Flexible(
            flex: 0,
            child: Padding(
              padding: EdgeInsets.symmetric(vertical: 32.0),
              child: TabPageSelector(
                controller: _tabController,
                color: Colors.grey.shade300,
              ),
            ),
          ),
        ],
      ),
    );
  }
}

flutter doctor -v
[✓] Flutter (Channel stable, 2.8.1, on macOS 12.1 21C52 darwin-arm, locale en-GB)
    • Flutter version 2.8.1 at /Users/mahesh/Documents/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 77d935af4d (6 days ago), 2021-12-16 08:37:33 -0800
    • Engine revision 890a5fca2e
    • Dart version 2.15.1

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

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

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

[✓] Android Studio (version 4.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

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

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

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-arm64   • macOS 12.1 21C52 darwin-arm
    • Chrome (web)    • chrome • web-javascript • Google Chrome 96.0.4664.110

• No issues found!

[✓] Flutter (Channel master, 2.9.0-1.0.pre.93, on macOS 12.1 21C52 darwin-arm, locale en-GB)
    • Flutter version 2.9.0-1.0.pre.93 at /Users/mahesh/Documents/flutter_master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 1be7f1ea56 (7 days ago), 2021-12-14 22:14:07 -0500
    • Engine revision e444009bf4
    • Dart version 2.16.0 (build 2.16.0-109.0.dev)
    • DevTools version 2.9.1

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

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

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

[✓] Android Studio (version 4.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

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

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

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-arm64   • macOS 12.1 21C52 darwin-arm
    • Chrome (web)    • chrome • web-javascript • Google Chrome 96.0.4664.110

• No issues found!

I think in the end the problem is that switching tabs/pages is finally being done in Tabs._warpToCurrentIndex() which doesn't use the Duration provided in animateTo() but just uses the duration field defined in TabController:
https://github.com/flutter/flutter/blob/master/packages/flutter/lib/src/material/tabs.dart#L1638
So whatever Duration is used in TabController.jumpTo() only leads to TabController whenCompleteOrCancel() to be called after that duration. For the final (graphical) animation TabController.duration is always used.

@rubenaster wrote a PR to solve this issue (see link below) but unfortunately it was not completed so the PR has been closed. If anyone would like to start from that PR and apply the review comments and drive this PR to completion, you are definitely welcome to do so!
PR: https://github.com/flutter/flutter/pull/122279

So is this fixed? How are we supposed to customize the TabBarView animations then if the duration and curve params don't actually do anything?

@rubenaster @Hixie  Instead of a list, how about creating a simple variable for the duration and curve parameters of the animateTo method in TabController, and then referencing them in the TabBarViewState? I don't think there's a need to track the previous duration and curve by storing them in a list. #122279
So i just add a simple variables and tests for this. #135832


consider switching from codemirror to ace

http://ace.c9.io/#nav=about
seems to have better mode support (eg scala works).
also supports emacs/vim bindings.

settled on cm a long time ago.


ENH: special: Just a few small changes (np can be omitted in functi…

…on names)

Reference issue

What does this implement/fix?

np.function_name(...) can be written as function_name(...) directly because these functions "sqrt", "arange", "sin" and "cos" is already included in scipy/special/orthogonal.py (see line 79-80).
Additional information

I think in general we prefer to keep the np. these days. I think the explicit imports of the functions probably largely date from the very early days (this is an old file) when some of the authors preferred the explicit imports.
If we did want to fix up this file, it would be to add the np. prefix to the function calls that don't have them, but it's not a significant need to make this consistent in this file. It's fine to leave it as-is.

Okay, think it over and agree with you. Thanks.


Clarify that compares only AND in the mask if vd == v0

This PR clarifies that masked compares AND in the mask only if the destination register is v0, as discussed in #900

While we're here, I think "when mask-undisturbed" warrants a further clarification. Mask-producing instructions like these ignore vtype.vma and always have mask agnostic behavior. I think we should say something more explicit, like "on implementations that implement mask-agnostic as mask-undisturbed". Otherwise the following example is wrong (consider a == b and VL = 1).

Is that really true? I know it’s the case for mask-logical instructions (which correspondingly aren’t allowed to be masked) but I don’t recall this being the case for comparisons.

My reading of
https://github.com/riscv/riscv-v-spec/blob/master/v-spec.adoc#343-vector-tail-agnostic-and-vector-mask-agnostic-vta-and-vma
is that inactive destination (mask) bits can be asserted.

What clause?

Could you use some quotation marks? All I see that’s potentially relevant (but isn’t actually) is “Mask destination tail elements are always treated as tail-agnostic, regardless of the setting of vta.”

A little further down:

When a set is marked agnostic, the corresponding set of destination elements in any vector destination operand can either retain the value they previously held, or are overwritten with 1s. Within a single vector instruction, each destination element can be either left undisturbed or overwritten with 1s, in any combination, and the pattern of undisturbed or overwritten with 1s is not required to be deterministic when the instruction is executed with the same inputs.

Maybe I'm making a logical leap that I shouldn't?

What I am missing is what marks this particular set agnostic. I agree that writing 1 is legal for an agnostic mask bit, but what mandates the result of comparisons be agnostic?

Mask-producing instructions like these ignore vtype.vma and always have mask agnostic behavior.

No. Mask-producing instructions are always vta but not vma.

Sorry. Please disregard my comments.


Songify stopped updating Slack status

Not sure when this started, but when I have both apps open and I have song playing, my Slack status isn't updating. I've tried removing/re-adding Songify but no change. I've also restarted Slack.

The thing is that Spotify has no websocket API to let me know if someone skips/starts/stops a song an therefore I need to poll the API by calling it multiple times. The API Limit is super strict and they my app extremly quick as the number of users has raised to +100. I try to find a better solution for this but atm don't know how to do it better. Sorry!


missing manpages

4 executables still do not have manpages:
/usr/sbin/getscom
/usr/sbin/getsram
/usr/sbin/pflash
/usr/sbin/putscom

Merged as of b7546b534d95112e44d84c3b9d510b2711746534


Updates to our issue status icons and colors

Summary
Our current issue icon colors are a source of constant user feedback, citing confusion with errors, confusion between why Open is green and Closed is red, accessibility concerns and the general scariness of seeing red across the issues index page when a bunch of closed issues is usually a good thing.
Furthermore, customers also need a way to quickly differentiate issues that are closed because they are not going to be worked on vs. those that are closed after they are resolved (either by a PR being merged, or some other means). However we also have over 500M closed issues on GitHub that we cannot retroactively capture the intent as they were resolved.
In an effort to address the above feedback we are planning the following updates:

An issue will have three core statuses (Draft, Open, Closed), with each having a distinct icon. The draft status is already being introduced in the new GitHub Issues within projects and in the future will be possible to create them across the platform.
The issue closed icon will be updated from red to purple, addressing accessibility concerns, aligning with other icons in GitHub and addressing confusion we see today.
We will provide the capability to mark the resolution of an issue to differentiate between closing because it is not going to be worked on or closing because it is done. This will be rolled out later this quarter as we start a conversation with GitHub integrators and apps to see how they can best support this new functionality.

The following image shows the end state outlined above.

🚢 This has shipped: https://github.blog/changelog/2022-05-19-the-new-github-issues-may-19th-update/


Cleanup wrong HTTP-Status codes in API-Docs

Wrong HTTP-Status codes in many API-Docs, needs review
Describe the results you received:
grep "201 OK" docs/reference/api/*
docs/reference/api/docker_remote_api_v1.14.md:        HTTP/1.1 201 OK
docs/reference/api/docker_remote_api_v1.15.md:        HTTP/1.1 201 OK
docs/reference/api/docker_remote_api_v1.15.md:        HTTP/1.1 201 OK
docs/reference/api/docker_remote_api_v1.15.md:        HTTP/1.1 201 OK
docs/reference/api/docker_remote_api_v1.15.md:        HTTP/1.1 201 OK
docs/reference/api/docker_remote_api_v1.16.md:        HTTP/1.1 201 OK
docs/reference/api/docker_remote_api_v1.16.md:        HTTP/1.1 201 OK
docs/reference/api/docker_remote_api_v1.16.md:        HTTP/1.1 201 OK
docs/reference/api/docker_remote_api_v1.16.md:        HTTP/1.1 201 OK
docs/reference/api/docker_remote_api_v1.17.md:        HTTP/1.1 201 OK
docs/reference/api/docker_remote_api_v1.17.md:        HTTP/1.1 201 OK
docs/reference/api/docker_remote_api_v1.17.md:        HTTP/1.1 201 OK
docs/reference/api/docker_remote_api_v1.17.md:        HTTP/1.1 201 OK
docs/reference/api/docker_remote_api_v1.18.md:        HTTP/1.1 201 OK
docs/reference/api/docker_remote_api_v1.18.md:        HTTP/1.1 201 OK
docs/reference/api/docker_remote_api_v1.18.md:        HTTP/1.1 201 OK
docs/reference/api/docker_remote_api_v1.18.md:        HTTP/1.1 201 OK
docs/reference/api/docker_remote_api_v1.19.md:    HTTP/1.1 201 OK
docs/reference/api/docker_remote_api_v1.19.md:    HTTP/1.1 201 OK
docs/reference/api/docker_remote_api_v1.19.md:    HTTP/1.1 201 OK
docs/reference/api/docker_remote_api_v1.19.md:    HTTP/1.1 201 OK
docs/reference/api/docker_remote_api_v1.20.md:    HTTP/1.1 201 OK
docs/reference/api/docker_remote_api_v1.20.md:    HTTP/1.1 201 OK
docs/reference/api/docker_remote_api_v1.20.md:    HTTP/1.1 201 OK
docs/reference/api/docker_remote_api_v1.20.md:    HTTP/1.1 201 OK
docs/reference/api/docker_remote_api_v1.21.md:    HTTP/1.1 201 OK
docs/reference/api/docker_remote_api_v1.21.md:    HTTP/1.1 201 OK
docs/reference/api/docker_remote_api_v1.21.md:    HTTP/1.1 201 OK
docs/reference/api/docker_remote_api_v1.21.md:    HTTP/1.1 201 OK
docs/reference/api/docker_remote_api_v1.22.md:    HTTP/1.1 201 OK
docs/reference/api/docker_remote_api_v1.22.md:    HTTP/1.1 201 OK
docs/reference/api/docker_remote_api_v1.23.md:    HTTP/1.1 201 OK
docs/reference/api/docker_remote_api_v1.23.md:    HTTP/1.1 201 OK
docs/reference/api/docker_remote_api_v1.24.md:    HTTP/1.1 201 OK
docs/reference/api/docker_remote_api_v1.24.md:    HTTP/1.1 201 OK

Describe the results you expected:
Expected "HTTP/1.1 201 Created" as per RFC2616
Additional information you deem important (e.g. issue happens only occasionally):
Sometimes wrong responsetype ("OK" instead of "Created"), sometimes wrong HTTP-Status Code number "201" instead of "200" documented)
There may be other statuscode deviations in the api docs

If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
For more information about reporting issues, see CONTRIBUTING.md.
You don't have to include this information if this is a feature request
(This is an automated, informational response)

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

Are you interested in opening a pull request to fix those?

try to revise the whole api documentation throughout the day #dibs


If remove the foreign key, the functions of quartz will be affected?

If remove the foreign key, it may affect the functions of quartz?
If so, how to solve these problems?

Not sure why you'd want to remove the foreign keys, but Quartz does not rely upon them so long as the columns exist it should work.


app.pageTransition Refreshing page scrolls will to the top

Environment
mac m1
Reproduction

stackblitz

On the right window, there is a refresh button on the left of the URL
Describe the bug

nuxt.config.js

If pageTransition is set. Refreshing page scrolls will to the top
export default defineNuxtConfig({
  app: {
    pageTransition: { name: 'page', mode: 'out-in' },
  },
});

But disable pageTransition. the scroller will be at the position before refreshing
Additional context
No response
Logs
No response

I want to work on this issue, please assign me.

++

@abhishek-kumar-91 are you still intending to look at this one?  I could look I am looking for a nice first issue to get going with


Reset Column OID after each test

In order to keep OID-s deterministic per test without having to start up a new cluster per test.
Optimizes DDLIntegrationTest which currently uses new cluster for all tests and prepares a ground for deterministic assertions of select _raw once we have OID-s written to the source.

Jenkins assigning a runner too long: Still waiting to schedule task
retest this please

Something is wrong here.
There is somme flakiness and failure always looks like
AssertionFailedError: unexpected exceptionorg.elasticsearch.cluster.coordination.FailedToCommitClusterStateException: publishing failed
        at __randomizedtesting.SeedInfo.seed([F14815597DDF68AE]:0)
        at junit.framework.Assert.fail(Assert.java:57)
        at junit.framework.TestCase.fail(TestCase.java:223)
        at org.elasticsearch.test.ClusterServiceUtils$2.onFailure(ClusterServiceUtils.java:92)
        at org.elasticsearch.cluster.service.MasterService$SafeClusterStateTaskListener.onFailure(MasterService.java:513)
        at org.elasticsearch.cluster.service.MasterService$TaskOutputs.lambda$publishingFailed$0(MasterService.java:411)
        at java.base/java.util.ArrayList.forEach(ArrayList.java:1511)
        at org.elasticsearch.cluster.service.MasterService$TaskOutputs.publishingFailed(MasterService.java:411)
        at org.elasticsearch.cluster.service.MasterService.onPublicationFailed(MasterService.java:295)
        at org.elasticsearch.cluster.service.MasterService.publish(MasterService.java:269)

very likely that it's related


Enhanced signature validation

Handle non-ascii characters and several facebook specific escape approaches ala https://developers.facebook.com/docs/messenger-platform/webhook-reference#security

Merged this. Thank you @ljoseppi for your contribution!

Thank YOU for finishing up my work. Unfortunately, I've been really busy at my job and didn't have enough time to make some of the tools required to generate the tests work. Thanks! I appreciate that.


About Test & Evaluation (Unanswered Question Issue)

I have trained your model using GeForce GTX 1080 with reduced batch_size=55. After 60000 steps, I used python config.py --mode test  and got the following result:
Exact Match: 69.69964664310955, F1: 78.46608330209344
Then I tried to remap the answer_dict so that the id in the answer_dict can stay consistent with dev-v1.1.json. I stored the remapped answer_dict to the json file, then used the official evaluation script to test the result.
the command I used:
python evaluate-v1.1.py dev-v1.1.json predictions-remapped.json
the result I got:
Exact Match: 67.18070009460737, F1: 75.63031756686154
I noticed that there were many unanswered questions. The output in the console looked like this:
...
Unanswered question 5733fe73d058e614000b6740 will receive score 0.
Unanswered question 57373d0cc3c5551400e51e85 will receive score 0.
Unanswered question 57377ec7c3c5551400e51f07 will receive score 0.
Unanswered question 57377ec7c3c5551400e51f09 will receive score 0.
Unanswered question 5737a0acc3c5551400e51f4b will receive score 0.
...

I have also checked the answer_dict (without remapping the id), the ids(=tot, tot+=1) should be series of continuous integers, however it seems that some of the numbers are missing.
Is this a desired behavior? Or does it have something to do with the preprocessing/testing procedure?
Thank you very much, looking forward to your reply.

In preprocessing, the questions are firstly assigned an id then filtered by context length, so if you map all question id to uuid, many questions will be missing.

Thanks a lot for your swift reply.
I have checked the maximum length of context/question tokens in the dev set:
max_len(context_tokens)=706
max_len(ques_tokens)=34

Then I increased para_limit and ques_limit to 5000 and 500 respectively to ensure that the question and answer pair in dev set will not be filtered by filter_func.
However, the Unanswered Question Issue remains (though the total number of the unanswered question reduced) and the performance increased to Exact_match: 67.92809839167455, F1: 76.59515371470633
Does it mean that I need to retrain my model using the new filter constraint?

I also test the model with the official script and the F1 is only 0.1% lower than log. There may be some errors in your implementation. The number of unanswered questions should be less than the batch size (10 in my test).
The code now supports testing with the official script, and I'll release it today.

After lowing down the batch_size for testing, the problem was solved, thank you very much!


adjust summary message to ARC accounting

This PR adjusts the summary message to have the exact same content as the message sent by ARC

Codecov Report

Merging #595 (21a9d23) into main (1f99f74) will increase coverage by 0.27%.
Report is 19 commits behind head on main.
The diff coverage is n/a.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #595      +/-   ##
==========================================
+ Coverage   63.99%   64.27%   +0.27%
==========================================
  Files          45       45
  Lines        4497     5161     +664
  Branches       58        0      -58
==========================================
+ Hits         2878     3317     +439
- Misses       1619     1844     +225

Components
Coverage Δ

apel_plugin
∅ <ø> (∅)

auditor
86.01% <ø> (+3.89%)
:arrow_up:

slurm_collector
65.98% <ø> (-0.44%)
:arrow_down:

slurm_epilog_collector
0.00% <ø> (ø)

htcondor_collector
∅ <ø> (∅)

priority_plugin
36.00% <ø> (-1.93%)
:arrow_down:


Bugfix: Create DAO failed transaction stays on Pending page

When a user attempts to deploy a dao via a blockchain transaction, If a transaction fails or is rejected, the pending page gets stuck.
Acceptance Criteria:

When a transaction is created - the pending page should show.
If a transaction is completed - should navigate to the specific dao page
If a transaction fails - the pending page should go away and should be redirected back to the create dao page


feat(text): add forest green color scheme

earl? zeloopers? MIKE? beautiful.

can you add the screesnhot to THEMES.md aswell

On it sir @harbassan, right after I finish this english project

@darkthemer Just letting you know this exists as I don't believe i'm able to assign the PR to you


fix: move notebooks and samples

remove sagemaker folder in notebooks and examples to fix marketplace links

Is it not possible to change the marketplace links? I think the repo is easier to understand when sagemaker & bedrock snippets are clearly separated

Is it not possible to change the marketplace links? I think the repo is easier to understand when sagemaker & bedrock snippets are clearly separated

we have to update all the listings for that, so for now we want to fix links and then we will update them one by one with model updates

Sad, but I guess we need to fix the links. Is there a linear issue to track that?


use 5.0.0 GA cannot not collect cpu & gc

Please answer these questions before submitting your issue.

Why do you submit this issue?
[ ] Question or discussion
[x] Bug
[ ] Requirement
[ ] Feature or performance improvement

Question

What do you want to know?

Bug

Which version of SkyWalking, OS and JRE?

Which company or project?

What happen?
If possible, provide a way for reproducing the error. e.g. demo application, component version.

Requirement or improvement

Please describe about your requirements or improvement suggestions.

It is not about the version. It is about which JVM you are using? See JVMService#run, that is how it collects all JVM info. If your JVM is not supported, we are welcome you to contribute your JVM metric collection implementation.

Closing. No update in 6 days.


table height makes the table unresponsive

When I set my table's height to be 300 the table becomes unresponsive, but if I set it to 700 the table becomes responsive.
<BootstrapTable
data = { data }
height = { 300 }
search = { true }
pagination = { true }
striped hover
className="table-padding">
Is there a fix for this issue?

it's a known issue for a long time, it's hard to explain. Actually, we fix some unalign column issue but make table unreponsive when you config the height.


GitHub bot conflicts for the same repo during dev/testing

Currently, the production github bot is acting on the same repos during local testing of all services
It is always acting on repos in this org: https://github.com/brn-test-assessment
We need a way of randomly creating repos which are distinct whenever the bot is acting on repos in that org (but not any others).
One way to accomplish this would be to introduce a random string into the repo names, but only if the ghbot is acting on that org. This may introduce other complications and will need significant testing.

Another idea would be to make sure the bot only interacts with repos that are valid in the database it has access to.
If combined with the idea of introducing random characters/numbers into the repo name, this should prevent conflicts from cases where dev env and prod env respond to payloads from the same repo.
This would mean the bot needs read access to the DB, so we should also develop a custom db user for the bot which only has read access.

The steps will now be:

[ ] Modify the CRUD function to add 6 random numbers/letters to the repo name (needs to be stored in assessment tracker entry)
[ ] Modify bot to check the db upon receiving a new payload to ensure payload comes from a db it is supposed to respond to.


[PYIC-2843] Process async CRI Credentials lambda (WIP)

Proposed changes
What changed
New Lambda to process Async CRI VC responses off the F2F queue
Why did it change
Issue tracking

PYIC-2843

Checklists
Environment variables or secrets

No environment variables or secrets were added or changed

Other considerations

PR superseded: closing


October updates

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Problem/Feature
Forgot to add these in October
Description of Changes:

Added is_featured to /mock/clips
/mock/users/follows now returns 410 Gone
Removed channel.follow v1
Added 410 Gone responses to POST /eventsub/subscriptions on EventSub WebSocket server for removed events

Checklist

[X] My code follows the Contribution Guide
[X] I have self-reviewed the changes being requested
[X] I have made comments on pieces of code that may be difficult to understand for other editors
[X] I have updated the documentation (if applicable)

I'm dumb and forgot to actually remove /users/follows and also add /channel/followers but I'll do that tomorrow or something


CalendarExtender does not show disable date on Internet Explorer

SamplePage.txt

The AJAX Control Toolkit v16.1
The installation method

[X] Installer

I installed this version on Visaul Studio 2015. And added a calendar to a blank web page. After that, I set it's EndDate property to today's date with
CalendarExtender1.EndDate = DateTime.Today;
line. It perfectly works (can't select future dates and they shown as 28 for example) on Google Chrome and Firefox. But in Internet Explorer, future dates shown as 28 (normal) they can be selected.
Actual result screenshot belongs on IE and Expected result screenshot belongs on Firefox and Chrome.
Actual result

Expected result

Browser(s) used
Google Chrome 53.0.2785.116
Firefox 48.0.2
Internet Explorer 11.0.9600.18230
A site deployment method

[ ] VS development web-server,
[X] IIS

Hi @MikhailTymchukDX
Here is my full web form and .CS file code:
Sample.docx

It's hard to recreate a working example from this code, so let's base our discussion on the sample project I attached: CalendarTest.zip
The AJAX Control Toolkit NuGet package v.16.1.0 is used here.
Code-behind that sets EndDate matches your code.
Please run this project in different browsers and inform us of your results.

Interesting @MikhailTymchukDX , your code does not procude the result what I get.
I see a few different things between your code and mine:

I downloaded relevant .exe from https://www.devexpress.com/Products/AJAX-Control-Toolkit/ page.
I used body and form html elements, not you.
You used Content, I don't.
You used ajaxToolkit:CalendarExtender but mine is<asp:CalendarExtender which I specify it on TagPrefix="asp".

Do you think these things might generate the difference?

Interesting @MikhailTymchukDX , your code does not procude the result what I get.

Yes, I forgot to mention that my sample does not produce the result you got.
The goal is to understand the difference to find the cause of the issue.

Do you think these things might generate the difference?

No, these differences are not relevant to our situation.
So, please specify the following:

Your system locale
Errors that you see in a browser's console (F12), if any

Your system locale

I'm using Win7 Enterprise SP1 x64. My CurrentCulture is tr-TR.

Errors that you see in a browser's console (F12), if any

I don't see any error in developers tools/console.

@sonergonul Thank you so much for the screenshot!
It indicates that your document mode is set to emulate IE 7, which does not meet the minimum requirements.
Would you please set the Document mode to Edge and check your result?

@MarinaRukavitsyna
Oh, that's wonderful to know. Thank you very much.
It works on 9, 10 and Edge as you mentioned. Now I searched how to provide a functionality to my webform that works always on Edge document mode and I found that can be solved using meta  html tag as:
<meta http-equiv="X-UA-Compatible" content="IE=edge"/>
By the way, maybe you might wanna add these Document Mode compabilities to your wiki  page also?


Feature: Added tooltips to Git columns in the details layout

Resolved / Related Issues

[X] Were these changes approved in an issue or discussion with the project maintainers? In order to prevent extra work, feature requests and changes to the codebase must be approved before the pull request will be reviewed. This prevents extra work for the contributors and maintainers.
Closes #12675

Validation
How did you test these changes?

[X] Did you build the app and test your changes?
[ ] Did you check for accessibility? You can use Accessibility Insights for this.
[ ] Did you remove any strings from the en-us resource file?

[ ] Did you search the solution to see if the string is still being used?

[ ] Did you implement any design changes to an existing feature?

[ ] Was this change approved?

[ ] Are there any other steps that were used to validate these changes?

/azp run


Use dist folder to improve performance

Fixes #60

Never mind, I see the real reason is due to shellac making a shell call to npx to invoke wrangler. This prevents wrangler from being cached by GitHub Actions.


Get all likes of "username" and download only new likes that arent in folder "foldername" downloaded already

Hey thank you very much this software works great!
How can I accomplish my subject?
I want to trigger a cronjob everyday to sync my soundcloud likes to my plex library

Thank you for a feature proposing!
I thought about it some time and I have some idea, how I would implement it.
I'd like to add new command called nehm sync. This command will:

Get all likes of user
Get all tracks in folder
Compare them
Download missing tracks to folder

What do you think of it?

point 3: skip track if it has been converted to a 30seconds preview mostly these tracks are only 400kb big in size.
point 4.5: command: nehm sync --no-preview do not download likes that are previews of songs which you need soundcloud go to listen to the full track
thank you very very much!

Cool, but I have one question:

point 3: skip track if it has been converted to a 30seconds preview aka. has been gone to soundcloud g o only, mostly these tracks are only 400kb big in size.

Do you mean, what nehm sync should skip track if this track in folder is a 30 second preview?

uhm sometimes a full track is getting disabled and only a 30 seconds preview is available, which means you have to buy soundcloud go to listen to the full version again. Lets say we were lucky and did a nehm sync before it went into a soundcloud go only track, so nehm will just skip this 30seconds preview, basically 3 and 4.5 is the same :D

Some information about --no-preview flag: SoundCloud API doesn't send track if it is SoundCloud Go preview, so user always becomes no preview, normal tracks.

@Snake4life Please test!


A way to pass through DeviceAllow strings

I'm trying to get FreeSWITCH to function inside a rkt container, using Digium (DAHDI) PRI cards. I've passed the DADHI deices through on the rkt run line with --volume dahdi,kind=host,readOnly=false,source=/dev/dahdi --mount volume=dahdi,target=/dev/dahdi When I try to load mod_freetdm, I get a permission denied trying to open /dev/dahdi/ctl. It turns out that rkt set DevicePolicy=closed, and didn't put a DeviceAllow for the DAHDI devices.
Searching, it appears that if I did each device individually with --volume/--mount, rkt should allow them—but there is one device per channel, in addition to the control ones. It's a quad-PRI card, there are 100 of them—and really there can be several quad-PRI cards per machine, each card adding another 96.
systemd's DeviceAllow has a solution (at least according to the documentation): to allow based on the device class—DeviceAllow: char-dahdi rw, DeviceAllow: char-dahdi_channels rw. But it appears impossible to do that with rkt.
(Of course, --insecure-options=path lets it work, but I think it preferable not to do that.)
 All your remarks are correct, it should be possible at systemd level but we currently don't have a way to rely this information to stage2. It will probably require a bit of design/UX planning and hopping through appc to expose this setting, so it is currently not on the radar.


Computing accuracy on close- and open-ended questions

Hi, thank you for providing the code for finetuning the model.
To be able to reproduce your results in the paper, I would like to know how you computed the accuracy on close- and open-ended questions in VQA-RAD and Slake.
Can you confirm that for close-ended questions, you get the set of all answers from the "CLOSE" type questions in both test and train sets of each dataset and call find_most_similar_index() as in the test.py script?
And for open-ended questions, you get the set of all answers from "OPEN" type questions in both test and train sets of each dataset and call find_most_similar_index() as in the test.py script?
Thank you

I also have this confusion.

Much curious about that, too.


[Bug] Footer is broken only in deployment

Current behavior and impact:
The footers css seems to be broken in the deployment.
Steps to reproduce:

Visit any currently active deployment
Scroll to the bottom

Expected behavior:
Footer has correct design in actual deployment.
Acceptance Criteria:

[x] Footer design fixed

check live deployment

PR is approved, issue will be closed once changes are merged


How to build from source ?

I need to make some changes.
How do I build this from source ?
So I install nodejs, and then npm install, which installs all packages in package.json.
Then I run
npm run build
And then I get

<EMAIL_ADDRESS>build:post D:\src\flatpickr-master\flatpickr-master
cp src/typings.d.ts dist/typings.d.ts

The command "cp" is written incorrectly or
it could not be found.
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>build:post: cp src/typings.d.ts dist/typings.d.ts
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build:post script.

Is the build running only on Linux ?

Ah, figured it out.
Needed to add the cp command from the gnu win32 packages to the user path environment variables.


Update shared-query-azure-cli.md

query Count VMs by OS ---> query Summarize resources by location
The name of the query is different from the created one in this article.

I went through the steps and "Summarize resources by location" appeared on the screen. Not "Count VMs by OS"

@daphnemamsft
Can you review the proposed changes?
IMPORTANT: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


Attempt to parse environment variables as UTF-8

What?
This PR makes the Unix get_environment() implementation attempt to decode values as UTF-8, and if that fails, return the value as-is, as has been the previous behavior. (Since the UNIX specification does not specify an encoding for environment variables, this sounds like the safe bet.)
This matches what the Windows get_environment() implementation does in spirit; it uses GetEnvironmentVariableW() and String::utf16.
Why?
I have a project which renders texts in short movie sequences based on inputs read from environment variables (as that seemed the easiest way to get values into the Godot process), and there was an audible groan when I saw the all-too-familiar UTF-8 mojibake for ä being interpreted as Latin-1:
With vanilla 4.2.1.stable.official.b09f793f5 on macOS:

With this PR:

Since this applies to all these platforms we need to confirm this also works on all these platforms
The POSIX standard specifies the portable character set for values, for portability, unsure if this is a strict standard

The POSIX standard specifies the portable character set for values, for portability, unsure if this is a strict standard

Portable character set is ASCII without control chars, so it's compatible with UTF-8.

Unsure what happens if it's encoded differently but that'll just be differently garbled, so not a serious issue

So... how would I get this merged? I'm currently stuck forward-porting this patch locally :)

Thanks!


[DS-2921] Item registration ignores bundles

https://jira.duraspace.org/browse/DS-2921
Briefly tested atop DS-79 (which I was testing when I tripped over this).

+1


Fix Tox using different Python than expected on macOS (#688)

Coverage remained the same at 93.691% when pulling 51a46e2aca5607841963a1d448eb4453af0300f9 on matthew16550:fix-688 into 55530c8c6de4865fa97b0b7bfd29c2985bb19451 on jakubroztocil:master.

Thanks, Matthew!


ImplicitDefaultLocale does not contain kotlin 1.5 functions

Expected Behavior

We expect ImplicitDefaultLocale to have consistent behavior for functions added in kotlin 1.5.
Observed Behavior

It only works for toUpperCase and toLowerCase.
Steps to Reproduce

"foo".lowercase()

Context

https://kotlinlang.org/docs/whatsnew15.html#stable-locale-agnostic-api-for-upper-lowercasing-text
However, kotlin 1.5 suggests String.replaceFirstChar { it.uppercase() } instead of capitalize().
The existing rule will report this, but in this case, I am concerned that it may be a false positive.
Your Environment

Version of detekt used: 1.17.1
Version of Gradle used (if applicable): 7.1
Gradle scan link (add --scan option when running the gradle task):
Operating System and version:
Link to your project (if it's a public repository):

It only works for toUpperCase and toLowerCase.

I believe that's actually the correct behavior.
Using .uppercase() and .lowercase() without an explicit Locale is fine as those functions will use the invariant locale. The former toUpperCase and toLowerCase would use instead Locale.getDefault() on the JVM (that's what the rule is trying to prevent).

I completely misunderstood Locale.ROOT and Locale.getDefault() and I agree that the behavior is correct now. 😞
Thanks for sharing your insight. 👍


Map File Handling

Issue Summary
Map files that are handled by the browser return as 404.
To Reproduce

Load a minified file
<script src="{{asset 'theme/bootstrap/js/bootstrap.min.js'}}"></script>
The browser will try and load it directly without the cache version
ghost_1    | [2019-10-01 00:20:49] INFO "GET /assets/theme/bootstrap/js/bootstrap.min.js.map/" 404 81ms

The file technically exists however when you access files directly without a cache version it fails. This means its incompatible with browser source map files. Obviously I could disable the .map ability within the minified file, but that's not solving a problem.
Is it possible to redirect the file to the cached version? E.g, /assets/theme/bootstrap/js/bootstrap.min.js.map/
redirects to
 /assets/theme/bootstrap/js/bootstrap.min.js.map?v=7c9a8c9e17

I'm wondering if this could be handled in the redirects.json so that it's not taking away control from the theme. There would need to be a shortcode however to get the current cache version.

I didn't ask for help or support. I pointed out an issue.


Edit workflows to include development_314 branch

Description
Edit workflows to include development_314 branch
Related Issues
https://github.com/threefoldtech/tfgrid-sdk-ts/issues/2196
Checklist

[ ] Tests included
[ ] Build pass
[ ] Documentation
[ ] Code format and docstrings
[ ] Screenshots/Video attached (needed for UI changes)

great! but we need the playground and new_stats builds too

The full clients run the build workflows for all packages.


[Hub Generated] Review request for Microsoft.DataProtection to add version preview/2022-11-01-preview

This is a PR generated at OpenAPI Hub. You can view your work branch via this link.
ARM API Information (Control Plane)
Azure 1st Party Service can try out the Shift Left experience to initiate API design review from ADO code repo.  If you are interested, may request engineering support by filling in with the form https://aka.ms/ShiftLeftSupportForm.
Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[x] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
By default, Azure SDKs of all languages (.NET/Python/Java/JavaScript for both management-plane SDK and data-plane SDK, Go for management-plane SDK only ) MUST be refreshed with/after swagger of new version is published. If you prefer NOT to refresh any specific SDK language upon swagger updates in the current PR, please leave details with justification here.

Contribution checklist (MS Employees Only):

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[x] Check this box if any of the following apply to the PR so that the label "ARMReview" and "WaitForARMFeedback" will be added by bot to kick off ARM API Review.  Missing to check this box in the following scenario may result in delays to the ARM manifest review and deployment.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits. You can use OpenAPIHub to initialize the PR for adding a new version. For more details refer to the wiki.

[x] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[x] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If you have any breaking changes as defined in the Breaking Change Policy, request approval from the Breaking Change Review Board.
Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Additional details on the process and office hours are on the Breaking Change Wiki.
NOTE: To update API(s) in public preview for over 1 year (refer to Retirement of Previews)
Please follow the link to find more details on PR review process.

"schemes": [

There are a bunch of breaking changes reported on this PR. Please get a breakingchanges approval and ARM can review the changes afterwards. Once you have the approval please remove the "ARMChangesRequested" label from the PR to make the PR visible to the ARM reviewers

Refers to: specification/dataprotection/resource-manager/Microsoft.DataProtection/preview/2022-11-01-preview/dataprotection.json:11 in df7de47. [](commit_id = df7de47795158e4280586e2d8cd44b1c37af3da0, deletion_comment = False)

      "readOnly": true

This looks like a breaking change - writable to readonly. But I'm imagining its probably actually an authoring oversight in the previous swagger version?

Refers to: specification/dataprotection/resource-manager/Microsoft.DataProtection/preview/2022-11-01-preview/dataprotection.json:4868 in e6f6e6c. [](commit_id = e6f6e6cd491bcbef6f88b44633323ed3d679a8de, deletion_comment = False)

      "description": "Flag to specify if the path is relative to Backup Item or full path",

nit, don't capitalize - try to use plain English in docs

Refers to: specification/dataprotection/resource-manager/Microsoft.DataProtection/preview/2022-11-01-preview/dataprotection.json:6025 in e6f6e6c. [](commit_id = e6f6e6cd491bcbef6f88b44633323ed3d679a8de, deletion_comment = False)

      "description": "list of prefixes to be used relative to the ItemPath for Restore criteria",

nit, don't capitalize - e.g. just call it 'the item path for restore criteria' or refer to properties by a name with quotes if you really want to 'itemPath'
'for restore criteria' may be redundant in this context?

Refers to: specification/dataprotection/resource-manager/Microsoft.DataProtection/preview/2022-11-01-preview/dataprotection.json:6029 in e6f6e6c. [](commit_id = e6f6e6cd491bcbef6f88b44633323ed3d679a8de, deletion_comment = False)

    "isPathRelativeToBackupItem": {

booleans aren't extensible
Replace boolean/switch properties with better enum
A Boolean will forever have two valid values (true or false). A string enum type is always preferred. Also, properties should always provide better values just than True and False. For example two switches "isTypeA" and "isTypeB" should be replaced with one enum "type": [A, B, DefaultType]. Even if you still believe [True, False] are the correct values for a property, you should use a string enum with values [True, False] instead of boolean. Enums are always a more flexible and future proof option because they allow additional values to be added in the future in a non-breaking way, e.g. [True, False, Unknown].

Refers to: specification/dataprotection/resource-manager/Microsoft.DataProtection/preview/2022-11-01-preview/dataprotection.json:6024 in e6f6e6c. [](commit_id = e6f6e6cd491bcbef6f88b44633323ed3d679a8de, deletion_comment = False)

  "description": "Parameters for Backup",

capitalization. overly brief?

Refers to: specification/dataprotection/resource-manager/Microsoft.DataProtection/preview/2022-11-01-preview/dataprotection.json:4560 in e6f6e6c. [](commit_id = e6f6e6cd491bcbef6f88b44633323ed3d679a8de, deletion_comment = False)

"schemes": [

There are a bunch of breaking changes reported on this PR. Please get a breakingchanges approval and ARM can review the changes afterwards. Once you have the approval please remove the "ARMChangesRequested" label from the PR to make the PR visible to the ARM reviewers
Refers to: specification/dataprotection/resource-manager/Microsoft.DataProtection/preview/2022-11-01-preview/dataprotection.json:11 in df7de47. [](commit_id = df7de47795158e4280586e2d8cd44b1c37af3da0, deletion_comment = False)

Breaking change approval is added to the PR now.

      "readOnly": true

This looks like a breaking change - writable to readonly. But I'm imagining its probably actually an authoring oversight in the previous swagger version?
Refers to: specification/dataprotection/resource-manager/Microsoft.DataProtection/preview/2022-11-01-preview/dataprotection.json:4868 in e6f6e6c. [](commit_id = e6f6e6c, deletion_comment = False)

Yes, this was an oversight. This was fixed in previous Swagger version as well.
https://github.com/Azure/azure-rest-api-specs/pull/21452/files

@anjorsh :  I see completeness issues  Swagger KPI Report - Power BI for API version 2022-11-01-preview
could you please check and resolving before merging this PR

/azp run unifiedPipeline

🕐

ARM signoff was obtained.

.Net SDK generation fail, @fengzhou-msft to help take a look

        "PrepareTimedout",

should be PrepareTimedOut

Refers to: specification/dataprotection/resource-manager/Microsoft.DataProtection/preview/2022-11-01-preview/dataprotection.json:4835 in d0a0048. [](commit_id = d0a00482813dba2dc561daf76e20be63101b4003, deletion_comment = False)

        "CommitTimedout",

should be CommitTimedOut

Refers to: specification/dataprotection/resource-manager/Microsoft.DataProtection/preview/2022-11-01-preview/dataprotection.json:4836 in d0a0048. [](commit_id = d0a00482813dba2dc561daf76e20be63101b4003, deletion_comment = False)

        "CommitTimedout",

If you've already shipped a non-preview probably this is too breaking.

In reply to:<PHONE_NUMBER>

Refers to: specification/dataprotection/resource-manager/Microsoft.DataProtection/preview/2022-11-01-preview/dataprotection.json:4836 in 47531fc. [](commit_id = 47531fcea25ebe8b3f822b7155a89b68fa48fc17, deletion_comment = False)

        "PrepareTimedout",

If you've already shipped a non-preview probably this is too breaking.

In reply to:<PHONE_NUMBER>

Refers to: specification/dataprotection/resource-manager/Microsoft.DataProtection/preview/2022-11-01-preview/dataprotection.json:4835 in 47531fc. [](commit_id = 47531fcea25ebe8b3f822b7155a89b68fa48fc17, deletion_comment = False)

@anjorsh Yup, LGTM


add % symbol to occ type editor for deterministic value ratio

This is completed in branch stage_damage_compute_bug

Perfect thank you


Removed 'Project Sakura' duplicated records

Project Sakura had duplicated records.. about 6 duplications. I've removed them leaving one record.

@salil-naik


mgmt: mcumgr: Supress false error log output

img_mgmt_state_read try to open mcuboot_primary_1 (ram_flash partition) without checking whether it exists or not in application.
We don't enable CONFIG_FLASH_SIMULATOR in application in order to save ram usage.
This commit introduces error log output.
nrfconnect/sdk-zephyr@df0fa6d
Generally it would failed when opening slot2 partition which makes CI failed when test DFU.
This error message could be founded on thingy53 and nrf5340 when https://developer.nordicsemi.com/nRF_Connect_SDK/doc/latest/nrf/working_with_nrf/nrf53/nrf5340.html#simultaneous-multi-image-dfu feature is used.

We have two options.

Skipped opening slot 2 image when CONFIG_FLASH_SIMULATOR is not enabled.
Change img_mgmt_erased_val error to warning message.

I am asked to review this PR, but is it in a reviewable state now @pirun ?

This PR is not acceptable, my NACK will remain

This PR should be closed since it's a workaround not a fix.
There is a ticket https://nordicsemi.atlassian.net/browse/NCSDK-21790 to fix the issue.


Can't create Full Scope PAT

My org is cracking down on fullscoped pats where they're not allowed to be created anymore. Can you add in the functionality to create partial scope pat?
getAuthenticationInfoAsync failed
java.lang.Error: java.io.IOException: HTTP request failed with code 400: {"$id":"1","innerException":null,"message":"FullScopePatPolicyViolation","typeName":"Microsoft.VisualStudio.Services.DelegatedAuthorization.SessionTokenCreateException, Microsoft.VisualStudio.Services.WebApi, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a","typeKey":"SessionTokenCreateException","errorCode":0,"eventId":3000}

Hello, I was wondering if there are any changes on this. I think it might be a good idea to allow the users to set whatever token they want to use.

I made the change. Can anyone  give me access to create a branch?

Any updates? I'm experiencing the same. Looks like the plugin hasn't been updated in one year.

@JerrickLewis7190 Maybe you can fork this repo and create a PR back?

Hoping for a fix soon here too!

@JerrickLewis7190 did you ever create a PR?

This is a major blocker.

This seems to be the issue with the most up-votes.
I also hit this exception using Rider. I saw FullScopePatPolicyViolation mentioned, which may relate to my organization's rules around generating "full scope" PATs.
As @JerrickLewis7190 suggested, providing an option (config file, reading an environment variable, UI implementation) to enter a given PAT (the token itself) would solve this. The documentation can detail what permissions that PAT should have (I doubt it needs full; I expect it to need things to pull/push to repos, comment on PRs, and similar).

Is there a recommendation from the plugin owners on how we can contribute this fix? It relates to security, so I want to understand if any additional contributing requirements outside of those documented in the README.

Note: I went through related issues and redirected them to this one given, at time of writing, this had the most up votes. Let's focus the discussion here.

@ForNeVeR - Do you see any concerns with the community submitting a PR to allow user-provided token instead of whatever existing auth flow exists? I prefer to get maintainer blessing for security-related things before diving into solutions.

I do not have any concerns about such a contribution. Just make sure there's some setting/checkbox in a user-visible place, and use IntelliJ ways of storing the passwords (PasswordSafe).

@connorjs Thanks for taking this, please do check this PR: https://github.com/microsoft/azure-devops-intellij/pull/574
I believe there were testing concerns there. Perhaps we could have a "setting/checkbox" to enable requesting PATs with only the required scopes?


Would it be possible to add a @hover event ?

I would like to display additional info in a little box next to the tree search as people are hovering items... is it possible to do this currently? If not, could we have a @hover event please?
Thanks!

Have you tried to Customize Option Label or Customize Value Label ?


Review EOS TUS implementation and understand interplay with tusd lib interfaces

@butonic @labkode the way in which we were handling tus uploads was inefficient. Like you mentioned here, https://github.com/owncloud/ocis/issues/997#issuecomment-736781877, even in the tus workflow, we were assembling all the chunks locally on the storageprovider machine and then copying the assembled file to EOS. This is also the way we handle v1 chunking which isn't ideal either.
EOS now has support for handling HTTP requests directly and the logic for combining the chunks, which I'm working on connecting now. Once we have that, we could directly forward the requests, making it a much better approach. So we could enable TUS again once that is done.


support custom event css class name

Plausible now supports tracking custom events by just adding a CSS class name to an element https://plausible.io/docs/custom-event-goals#2-add-a-css-class-name-to-the-element-you-want-to-track-on-your-site.
Hope this library can support this feature.

Published as<EMAIL_ADDRESS>Use like this:
<PlausibleProvider taggedEvents ....>
Thanks!


Steps are displayed in the "on this page" menu, even though they are not titles or navigation items

Description
Steps are used mainly in organizing a procedure in documentation.  They should be excluded from the page navigation, as they don't represent any sort of navigation through the page.
Steps To Reproduce
I created a procedure with the Steps shortcode.
Expected Behavior
The steps should display in body of text only.
Actual Behavior
The pretty procedure layout is displayed but also Step 1, step 2, etc. is displayed in the "On this page" navigation.
Screenshots

I tried to use bold fencing or an ordered list instead, but I lose the pretty layout intended.
Thank you.
V.

Hi, thanks for this issue. It is by design that the Steps shortcode uses h3 instead of ordered/unordered lists.
You can override this behavior by:

create layouts/shortcodes/steps.html file to override the default steps shortcode template:

<div class="steps ml-4 mb-12 border-l border-gray-200 pl-6 dark:border-neutral-800 [counter-reset:step]">
    {{ .Inner | markdownify }}
</div>

use the steps shortcode with <> angle brackets:

{{< steps >}}

### Step 1

This is the first step.

### Step 2

This is the second step.

{{< /steps >}}

References:

https://gohugo.io/content-management/shortcodes/#shortcodes-without-markdown


cmorize_obs.py error with import utilities

The cmorizer main script produces an error at the very beginning when trying to load the read_cmor_config  function from the local utilities .py file.
If we remove the . in the following line
https://github.com/ESMValGroup/ESMValTool/blob/cabb681938e36908ee032b9f208924902fd4a630/esmvaltool/cmorizers/obs/cmorize_obs.py#L26
the cmorizer goes back to work.

this is bonkers since the relative path import method is correct with the .

what's the error you are seeing? :beer:

@valeriupredoi here you go ...
Traceback (most recent call last):
  File "cmorize_obs.py", line 26, in <module>
    from .utilities import read_cmor_config
ModuleNotFoundError: No module named '__main__.utilities'; '__main__' is not a package

While working on cmorizers few months ago I had the exact same problem using the relative path import on our HPC machine, so I suggested to avoid it in  cmorize_obs.py
Looking a bit around I found the following thread relative-python-imports giving a reasonable explanation on this error.
comments?

Have you tried to reinstall?
There were some changes in the location of the cmorizers (see #1163).

note that relative imports in Python 3_ are slightly different from Python 2+ (that stackoverflow doc you are pointing to is for 2+): https://stackoverflow.com/questions/16981921/relative-imports-in-python-3
Pls reinstall and make sure you are using Python 3+ (that should be the case as norm since they Python version from the esmvaltool environment is 3+)

@valeriupredoi installation is fine, but the error came from a development script that I made to run multiple cmorizers that was calling directly the python script as python cmorize_obs.py instead of using the compiled tool! sorry my bad ...

no problemo, dev s**t happens :grin:


[BACKLOG-25742] Exception occurs when viewing transformation detail o…

…n Carte server status page

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -f 'pom.xml' -pl 'plugins/ms-access/impl,plugins/ms-access/ui,plugins/gp-bulk-loader/core,ui,plugins/lucid-db-streaming-loader/core' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'plugins/ms-access/impl,plugins/ms-access/ui,plugins/gp-bulk-loader/core,ui,plugins/lucid-db-streaming-loader/core' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  plugins/gp-bulk-loader/core/src/main/java/org/pentaho/di/trans/steps/gpbulkloader/GPBulkLoaderMeta.java
  plugins/lucid-db-streaming-loader/core/src/main/java/org/pentaho/di/trans/steps/luciddbstreamingloader/LucidDBStreamingLoaderMeta.java
  plugins/ms-access/impl/src/main/java/org/pentaho/di/job/entries/msaccessbulkload/JobEntryMSAccessBulkLoad.java
  plugins/ms-access/ui/src/main/java/org/pentaho/di/ui/job/entries/msaccessbulkload/JobEntryMSAccessBulkLoadDialog.java
  ui/src/main/java/org/pentaho/di/ui/util/SwtSvgImageUtil.java

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.

:warning: Coverage Changes (click to expand)
org.pentaho.di.trans.steps.salesforceupsert.SalesforceUpsertMeta

Instruction Change: -0.1% :small_red_triangle_down:

@bmorrise


Moving Camel sources to top level dir

Fixes #465
This moves camel sources to the camel/source dir.
I'm keeping the name "source" for 0.7, as per: https://github.com/knative/eventing/blob/master/docs/decisions/sources-to-importers.md

/ok-to-test
/lgtm
/approve


Intel Aero flashing problem

Hello,
I compiled the latest version from master for Intel Aero RTF.
I'm used this command:
make intel_aerofc-v1_default
After flashing the flight controller is not running and when
I tried to flash stable version the bootloader enters in infinite loop
when it tries different baudrates.
My compiler version is 7.2.1.
Is somebody experienced with this issue ?
Best regards
Evgeny

@altshuler sorry for the late response. Has this been resolved in the meantime?

The issue is solved.
Thanks !


Add documentation section on contributing new methods

Summary

One thing NiMARE is meant to be is a central location for meta-analytic algorithms. Folks who produce new methods (e.g., in graduate school) should be able to add those methods to NiMARE, but we of course have requirements regarding both code and method quality. We should come up with some instructions for contributing new methods and add them to the documentation.
I see this as a specific section within the contributing guidelines.
Additional details

Hopefully this will convince independent methodologists to add their methods to NiMARE.
Next steps

Determine what we want the guidelines to be.

Maybe we could use a model like scikit-learn's extension model?

Add the guidelines to the contributing documentation.

Scikit-learn has a template for extensions, scikit-learn-contrib/project-template. The approach seems to involve creating estimators, for which there is both documentation and a check_estimator test. I didn't see any information on moving from a contrib project to scikit-learn though.

I think we probably need a checklist of requirements for algorithms that should be merged, in addition to requirements for mergeable code.
Some possible requirements for mergeable algorithms:

Associated publication, preprint, or conference presentation.
Some statement regarding long-term support for the algorithm. I don't think we should require the original developer to support the implementation in NiMARE, but we should at least have a solid idea of whether they will or not.
Available test data for validating the NiMARE/Python implementation.


Upgrading to 3.2.1 causes out of memory error in watch mode

This compiled successfully using tsc --watch in 3.1.6, but broke in 3.2.1. I traced it down further to 3.2.0-dev.20181113 and most specifically this PR: https://github.com/Microsoft/TypeScript/pull/28490.

TypeScript Version:  3.4.0-dev.20190403

Search Terms:
OOM 3.2.1
Code
// Type-safe deep updates
// The original object is returned if the deep field's new value is equal to its current value per Object.is.
// These are use-once; don't make two separate updates using part of the same chain.
// Example usage:
// const o = { a: { b: 0, c: 1 }, d: 2 };
// update(o)('a')('c').map(c => c + 2); // Evaluates to { a: { b: 0, c: 3 }, d: 2 }
export const updateIfChanged = <T>(t: T) => {
  const reduce = <U>(u: U, update: (u: U) => T) => {
    const set = (newU: U) => Object.is(u, newU) ? t : update(newU);
    type Key = U extends ReadonlyArray<any> ? number | keyof U : keyof U;
    type Value<K extends Key> = K extends keyof U ? U[K] : U extends ReadonlyArray<infer V> ? V : never;
    return Object.assign(
      <K extends Key>(key: K) =>
        reduce<Value<K>>(u[key as keyof U] as Value<K>, (v: Value<K>) => {
          return update(Object.assign(Array.isArray(u) ? [] : { }, u, { [key]: v }));
        }),
        { map: (updater: (u: U) => U) => set(updater(u)), set });
  };
  return reduce<T>(t, (t: T) => t);
};

Expected behavior:
The code compiles and watches when running tsc --watch.
Actual behavior:
When running tsc --watch it outputs:
<--- Last few GCs --->

[42844:00000191A5C7B950]    49850 ms: Scavenge 1388.4 (1424.0) -> 1387.5 (1424.5) MB, 2.9 / 0.0 ms  (average mu = 0.206, current mu
 = 0.179) allocation failure
[42844:00000191A5C7B950]    49855 ms: Scavenge 1388.4 (1424.5) -> 1387.6 (1425.0) MB, 3.1 / 0.0 ms  (average mu = 0.206, current mu
 = 0.179) allocation failure
[42844:00000191A5C7B950]    49863 ms: Scavenge 1388.5 (1425.0) -> 1387.7 (1425.5) MB, 3.5 / 0.0 ms  (average mu = 0.206, current mu
 = 0.179) allocation failure

<--- JS stacktrace --->

==== JS stack trace =========================================

    0: ExitFrame [pc: 000003B19215C5C1]
    1: StubFrame [pc: 000003B19210D64B]
    2: ConstructFrame [pc: 000003B19210D1E3]
Security context: 0x020661c9e6e1 <JSObject>
    3: createKeywordTypeNode [000001B8101A26D1] [C:\Users\Nathan\Documents\TypeScript\built\local\tsc.js:~
51212] [pc=000003B1924CE3F4](this=0x0296c36051a9 <Object map = 00000283A02FA809>,kind=132)
    4: typeToTypeNodeHelper(aka typeToTypeNodeHelpe...

FATAL ERROR: Ineffective mark-compacts near heap limit Allocation failed - JavaScript heap out of memory
 1: 00007FF7A3D1121A v8::internal::GCIdleTimeHandler::GCIdleTimeHandler+4810
 2: 00007FF7A3CEA5B6 node::MakeCallback+4518
 3: 00007FF7A3CEAFA0 node_module_register+2160
 4: 00007FF7A3F7B3EE v8::internal::FatalProcessOutOfMemory+846
 5: 00007FF7A3F7B31F v8::internal::FatalProcessOutOfMemory+639
 6: 00007FF7A44B9304 v8::internal::Heap::MaxHeapGrowingFactor+11476
 7: 00007FF7A44AFA67 v8::internal::ScavengeJob::operator=+25543
 8: 00007FF7A44ADFDC v8::internal::ScavengeJob::operator=+18748
 9: 00007FF7A44B6F57 v8::internal::Heap::MaxHeapGrowingFactor+2343
10: 00007FF7A44B6FD6 v8::internal::Heap::MaxHeapGrowingFactor+2470
11: 00007FF7A4059DD7 v8::internal::Factory::NewFillerObject+55
12: 00007FF7A40F1ABA v8::internal::WasmJs::Install+29530
13: 000003B19215C5C1

Playground Link:
Here's a repo with the source code and tsconfig.json: https://github.com/Methuselah96/typescript-oom.
Related Issues:
None.

This does indeed appear to be caused by #28490. The OOM happes when the --declaration flag is specified and createAnonymousTypeNode takes off doing enormous amounts of work.
@weswigham You added the code in #28490, you may want to take a look.

There's a depth > 10 check that can just be lowered - we didn't have any examples that exploded nearly as quickly as this one.

It's not quite that simple. Any check except depth !== 0 ends up producing an invalid declaration file. And going back to depth !== 0 is effectively the same as undoing #28490.
I'm going to assign this one back to you.

ends up producing an invalid declaration file

Ohho? Sounds like we can't actually print declarations for this type structure and either need to reduce accuracy or issue a declaration error. I'll look into it.

@weswigham Any update on this?

Any check except depth !== 0 ends up producing an invalid declaration file.

Two things going on here:

Turns out we just had a bug where since we were using the declaration to cache generated names for type parameters, the generated names for differing instances of the same symbol were the same (since they share a declaration), so they conflicted with one another. This is pretty easy to fix, and makes the simpler example:

export type Key<U> = keyof U;
export type Value<K extends Key<U>, U> = U[K];
export const updateIfChanged = <T>(t: T) => {
    const reduce = <U>(u: U, update: (u: U) => T) => {
        const set = (newU: U) => Object.is(u, newU) ? t : update(newU);
        return Object.assign(
            <K extends Key<U>>(key: K) =>
                reduce<Value<K, U>>(u[key as keyof U] as Value<K, U>, (v: Value<K, U>) => {
                    return update(Object.assign(Array.isArray(u) ? [] : {}, u, { [key]: v }));
                }),
            { map: (updater: (u: U) => U) => set(updater(u)), set });
    };
    return reduce<T>(t, (t: T) => t);
};

emit without producing code with any errors.
2. When we print the declarations, we expand up to depth levels of conditionals if their names aren't reachable. This is usually fine.... except these nesting conditionals have infer types! And they essentially are prone to the problem this code exhibits:
function f<X>(arg: X) {
    type Cond1 = X extends [infer A] ? A : never;
    type Cond2 = X extends [infer A] ? A : never;

    let x: Cond1 = null as any;
    let y: Cond2 = null as any;
    x = y; // is err, should be ok
    y = x; // is err, should be ok
}

(in the example in the OP, we inline the alias into the constraint and the argument, meaning we have two conditionals with differing identities, as in the above, and the keyof one isn't allowed to index the other!) Fixing this is more involved - our logic for relating conditional extends clauses with infer type parameters needs to do something similar to what we do for signatures - we need to "instantiate the source in the context of the target" to unify the type parameters with differing identities.
@ahejlsberg should I fix these separately? The two parts seem different enough that they warrant separate review.

This is fixed in TS 4.5 (and probably much earlier).


Wrong results shown in Q examples

Ref: https://docs.perl6.org/language/quoting#Literal_strings:_Q
It looks like the top parts of the example (the code) were changed without
updating the bottom part accordingly (the strings produced by the code).
For example Q (this is fine, because of space after Q) does not produce  "A literal string"

In fact, the two first don't produce anything at all.


Update sbt-ci-release to 1.5.9

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

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.geirsson", artifactId = "sbt-ci-release" } ]

labels: sbt-plugin-update, semver-patch

Codecov Report

Merging #45 (47967f6) into master (aa80d96) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 47967f6 differs from pull request most recent head d3ba084. Consider uploading reports for the commit d3ba084 to get more accurate results

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

Continue to review full report at Codecov.

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


Skip preserving file extension for generated path

Babel would emit the files with js extension irrespective of the extension of source files, which would cause problem to test individual files without using webpack or gulp,  since required modules path would never be resolved, becuase generated files would have an extension of js and the generated webpack alias path would be referring to jsx(i.e original extension)
I've shown an example to support the use case
.babelrc
{
  "presets": ["es2015", "react","stage-2"],
  "plugins" : [
  ["babel-plugin-webpack-alias", { "config": "./webpack/server/server.js" }]
  ],
}

webpack config
resolve : {
    extensions: ['','.js','.jsx'],
    modules : [srcPath,path.join(basePath,'./node_modules')],
    alias : {
      template: path.join(srcPath,'./template')
    },
  },

es6 code
import Layout from 'template/Layout';

Generating es5 file post babel transformation
src/server/template/Layout.jsx -> lib/server/template/Layout.js

Post babel transformation
var _Layout = require('./Layout.jsx'); //should be './Layout.js'
var _Layout2 = _interopRequireDefault(_Layout);

Running in node
Error: Cannot find module './Layout.jsx'

What I can do is adding an option noOutputExtension which, when set to true:
{
  "presets": ["es2015", "react","stage-2"],
  "plugins" : [
  ["babel-plugin-webpack-alias", { "noOutputExtension": true, "config": "./webpack/server/server.js" }]
  ],
}

would output all require statements without extensions:
var _Layout = require('./Layout');
var _Layout2 = _interopRequireDefault(_Layout);

Would that solve your issue?

Yes, it would
On Thu, Sep 8, 2016, 2:51 PM Adrien Antoine<EMAIL_ADDRESS>wrote:

What I can do is adding an option noOutputExtension which, when set to
true:
{
"presets": ["es2015", "react","stage-2"],
"plugins" : [
["babel-plugin-webpack-alias", { "noOutputExtension": true, "config": "./webpack/server/server.js" }]
],
}
would output all require statements without extensions:
var _Layout = require('./Layout');var _Layout2 = _interopRequireDefault(_Layout);
Would that solve your issue?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/trayio/babel-plugin-webpack-alias/issues/25#issuecomment-245541794,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAnUA5rmE-ahtDAs0_UKwYcGcSUNEyfMks5qn9OlgaJpZM4J3thn
.

Ok, I'm working on it now

Thanks for such a quick response, and the plugin is minimal but very
helpful.
On Thu, Sep 8, 2016, 2:56 PM Adrien Antoine<EMAIL_ADDRESS>wrote:

Ok, I'm working on it now
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/trayio/babel-plugin-webpack-alias/issues/25#issuecomment-245542977,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAnUA0uThdts8rHsxJzk9O8d6d5xl1Ktks5qn9S-gaJpZM4J3thn
.

Thanks! @devarsh

The change is made, I'll publish a new version

@devarsh It's just been released in v2.1.0!

@adriantoine thanks alot, i'll check and revert back to you


react-router depends on babel to install

Just tried to install RR rc3 during a pair programming session. Seems like React Router depends on babel being available globally or throws on install.

The postinstall script calls npm run build which uses our build script to create the lib directory when installing from a git repo. However, this shouldn't cause problems since babel is part of our devDependencies and you should get it with a normal npm install.
Did you use the npm install --production by any chance?

I failed to mention it explicitly, but we're not relying on a global babel. Running our build script using npm means it'll use ./node_modules/.bin/babel, which you should get on a normal npm install.

@mjackson I'd be a little nervous doing that because if something specifies react-router as a dependency, the devDeps won't be installed (they are only installed for the top-level and only when not in production mode, e.g. NODE_ENV or --production set).

@phated
If something installing react-router as a dependency, then it should be pulling react-router from NPM, and the built files will already be available, so the build step of postinstall just won't run.
This step only does something if you're installing from the GH repo.

@taion that would be ideal and expected but people can still depend on libraries pointing to the github repo by specifying it as "rackt/react-router" in their package.json

@phated
I don't see a better way to support that short of actually checking in the built files, which IMO is the wrong thing to do. We should only source control the actual source.
As it stands, you can do npm i rackt/react-router so long as you have Babel installed globally. It's not perfect, but it's slightly more convenient than e.g. not supporting installing from GH at all. For example, you just can't install React with npm i facebook/react.

I'm afraid we can't support people depending on us using a repo URL w/out checking in our pre-built files, which we're not going to do. The postinstall hook is for people who want to install stuff from master to play around with it. But that's all I want to support.

Running in dev fixed the issue. Thanks @mjackson


try/except sleep unlink out/err files in tests.py

Liu, Ben
Another feature request.  On Windows systems, the deletion of the out/err files can sometimes fail because the file has not been finalized yet.  Previously we added a try/except on the unlink to get around this (this is at the end of tests.py).
Would it be possible to sleep(1) and try again, then only give the log message if that also failed?

This may fit in with a another proposal to use a standard set of decorators to do handle these cases.


Stopping when long break is active

No way to check if on long break or not, or any storage of the number. The number should increase and decrease in conjunction with the break time if it is stopped.

Was this solved? I think it was...


Cannot create a Listener. "info" command not working as expected.

Empire Version
Latest, Downloaded today. Currently 276 Modules.
OS Information (Linux flavor, Python version)
Kali Linux Rolling 2017, updated and upgraded.
Also tried inside virtualbox with another Kali Linux 2017.
Expected behavior and description of the error, including any actions taken immediately prior to the error. The more detail the better.
After "git clone https://github.com/EmpireProject/Empire",
And after it's proper installation without errors ("./setup/install.sh"),
And after "./empire", or "sudo ./empire",
when the menu opens and I type "listeners", then I should be able to create my listeners by issuing "info" and getting the relative parameters, setting them up properly, and then executing, as explained in every tutorial and at https://powershellempire.com/?page_id=110 .
Screenshot of error, embedded text output, or Pastebin link to the error
The problem is that issuing "info" does not work:
Here are the logs. What follow after "//" are of course my personal comments..
(Empire) > // once the main menu is loaded (it loads correctly as expected)
(Empire) > listeners // I type in "listeners" to jump to the listeners page
[!] No listeners currently active // I guess this is expected since I've just installed Empire.
(Empire: listeners) > info // I type in "info" expecting it to print out the beautiful options page, stating everything I can and cannot set about the listener.
[!] Invalid listener name
(Empire: listeners) >
That's it. I've tried every option in the menus (using "help"), there's no way around it.
Any additional information
It keeps saying "Invalid listener name" when I issue "info" inside of "(Empire: listeners) > ", HOW AM I SUPPOSED TO SPECIFY A LISTENER's NAME  IF I CAN'T FIRST CREATE ONE?
I can't do anything without firstly setting up the Listener, so please help me out! I can't see why it's not working...

uselistener [tab][tab] to choose your listener type

Worked Wonderfully. A very deep Thank you.
Solved, can be Closed.


[test] fix runtime-tests cmake settings

A little oversight from my end before landing #288, make runtime-tests was not working for newly created build folders.

63 tests [fail 49] on Travis. Based on the messages I think we can't access PMC events on Travis:
interval_short_name ERROR
	Expected: @syscalls\:\s[0-9]*
	Found: Error creating map: '@syscalls'
Attaching 2 probes...
Error loading program: interval:s:1 (try -v)
software ERROR
	Expected: @\[.*\]\:\s[0-9]*
	Found: Error creating map: '@'
Attaching 1 probe...
Error loading program: software:faults:1 (try -v)
hardware ERROR
	Expected: @\[.*\]\:\s[0-9]*
	Found: Error creating map: '@'
Attaching 1 probe...
Error loading program: hardware:cache-misses:10 (try -v)

Some kprobes are also not available:
func ERROR
	Expected: SUCCESS func .*
	Found: Attaching 1 probe...
Error loading program: kprobe:do_nanosleep (try -v)
sum ERROR
	Expected: @.*\[.*\]\:\s[0-9]*
	Found: Error creating map: '@bytes'
Attaching 1 probe...
Error loading program: kprobe:vfs_read (try -v)

Interestingly enough, some of the successful tests are using kprobe:do_nanosleep and interval:s:1. Not sure if those tests should be failing or if all tests are just flaky at this point.

Anyway, this can land and we can fix Tavis later since as it is right now runtime tests are not running for developers too.

thanks!


Fix transitionTimeout and going back to homeSlide after interval

Finally figured it out. It seems that you accidentally called moduleTransition instead of manualTransition.
Fixing the declaration of the transitionTimeoutCallback function and making the function call this.manualTransition instead of this.moduleTransition fixes the issues.
Resolves: https://github.com/shbatm/MMM-Carousel/issues/64

Many thanks for the PR! :smiley:


Functions from the avrt.h header?

I wanted to experiment with raising the priority of my audio threads, so I wanted to call the AvSetMmThreadCharacteristicsA. But that doesn't seem to exist in the windows crate. Am I missing something?
This is the function I wanted to try:
https://docs.microsoft.com/en-us/windows/win32/api/avrt/nf-avrt-avsetmmthreadcharacteristicsa

You're not missing anything; it appears to be missing in win32metadata. Transferring issue there. (Heads up, we're in a holiday week so you won't see much Microsoft activity until next week.)


[10.0.x] kie-issues#1613: Add CI :: Check license headers GitHub Actions workflow

This is partial fix of the issue https://github.com/apache/incubator-kie-issues/issues/1613 as we open one PR per repository.
Please make sure your PR meets the following requirements:

[ ] You have read the contributors guide
[ ] Pull Request title is properly formatted: [Issue XXXX] Subject
[ ] Pull Request contains link to the JIRA issue
[ ] Pull Request contains description of the issue
[ ] Pull Request does not include fixes for issues other than the main ticket
[ ] Your feature/bug fix has a testcase that verifies it
[ ] You've tested the new feature/bug fix in an actual OpenShift cluster
[ ] You've added a RELEASE_NOTES.md entry regarding this change

How to retest this PR or trigger a specific build:

(Re)run Jenkins tests
Please add comment: Jenkins [test|retest] this

How to backport a pull request to a different branch?

In order to automatically create a backporting pull request please add one or more labels having the following format backport-<branch-name>, where <branch-name> is the name of the branch where the pull request must be backported to (e.g., backport-7.67.x to backport the original PR to the 7.67.x branch).

NOTE: backporting is an action aiming to move a change (usually a commit) from a branch (usually the main one) to another one, which is generally referring to a still maintained release branch. Keeping it simple: it is about to move a specific change or a set of them from one branch to another.

Once the original pull request is successfully merged, the automated action will create one backporting pull request per each label (with the previous format) that has been added.
If something goes wrong, the author will be notified and at this point a manual backporting is needed.

NOTE: this automated backporting is triggered whenever a pull request on main branch is labeled or closed, but both conditions must be satisfied to get the new PR created.

PR job #14 was: ABORTED
Possible explanation: Most probably a timeout, please review
Please look here: https://ci-builds.apache.org/job/KIE/job/kogito/job/10.0.x/job/pullrequest/job/kogito-images.build-and-test/14/display/redirect
See console log:

Console Logs
Build KIE » kogito » 10.0.x » pullrequest » kogito-images.build-image #83 completed: FAILURE[Pipeline] }[Pipeline] // stage[Pipeline] }Failed in branch kogito-data-index-postgresqlBuild KIE » kogito » 10.0.x » pullrequest » kogito-images.build-image #81 completed: FAILUREBuild KIE » kogito » 10.0.x » pullrequest » kogito-images.build-image #82 completed: FAILURE[Pipeline] }Build KIE » kogito » 10.0.x » pullrequest » kogito-images.build-image #80 completed: FAILURE[Pipeline] }[Pipeline] // stage[Pipeline] // stage[Pipeline] }Failed in branch kogito-jobs-service-ephemeral[Pipeline] }Failed in branch kogito-jobs-service-postgresql[Pipeline] }[Pipeline] // stage[Pipeline] }Failed in branch kogito-jit-runnerBuild KIE » kogito » 10.0.x » pullrequest » kogito-images.build-image #78 completed: FAILURE[Pipeline] }[Pipeline] // stage[Pipeline] }Failed in branch kogito-base-builderBuild KIE » kogito » 10.0.x » pullrequest » kogito-images.build-image #84 completed: ABORTED[Pipeline] }[Pipeline] // stage[Pipeline] }Failed in branch kogito-jobs-service-allinoneClick here to forcibly terminate running steps[Pipeline] }[Pipeline] // stage[Pipeline] }Failed in branch kogito-data-index-ephemeral[Pipeline] // parallel[Pipeline] }[Pipeline] // dir[Pipeline] }[Pipeline] // script[Pipeline] }[Pipeline] // stage[Pipeline] stage[Pipeline] { (Declarative: Post Actions)[Pipeline] script[Pipeline] {[Pipeline] sh+ wget --no-check-certificate -qO - 'https://ci-builds.apache.org/job/KIE/job/kogito/job/10.0.x/job/pullrequest/job/kogito-images.build-and-test/14/api/json?depth=0'[Pipeline] readJSON[Pipeline] sh

we don't need this, as this repo is not active on main anymore.
and for 10.0.0. release the individual repos content like .rat-exclude, DISCLAIMER, LEGAL, etc... will be removed for source anyway.


feat: implement plugin execution order

[x] test
[ ] doc

If there is nothing wrong with the code and design, I can try to add documentation
What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[x] Feature
[ ] Code style update
[ ] Refactor
[ ] Docs
[ ] Underlying tools
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

By default, the plugins are executed in the same order as before I believe
Other information:
related to #5106, #6251
Design
interface
interface Plugin {
  (): any
  after?: string|Array<string>
}

Example
vue-cli-plugin-foo/generator/index.js
module.exports = api => {
  api.render('./template')
}

vue-cli-plugin-bar/generator/index.js
module.exports = api => {
  api.render('./template')
}
module.exports.after = 'vue-cli-plugin-bar'

the 'after' can be used to fine-tune the execution order
Implement
Topological Sort Algorithm sorts the plugins by 'after'. see leetcode 210

I like this idea! after will be very useful.
But I'm not so sure about stage. To actually use it we need to export quite a lot of stage constants and then the internal workflow of Vue CLI becomes somehow public. I'm not sure it's maintainable.

stage  is just to ensure that the generator of  @vue/cli-service is  invoked firstly, and it's a substitution  of sortObject.  Even without 'stage', 'after' can work with the current code.  Or we can just mention 'after' in the documentation?

Yeah, I think it can be kept an implementation detail until we figure out a better way to use it.
In the current state, I feel it's like z-index in CSS, and z-index is very hard to manage unless there's a design system to put some constraint on it…

hey，wonderful realization！
但是我似乎没有办法让A排在B前面


update nncf test quant dataset

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