[FancyZones] Fix #25353 setting page bouncing

Summary of the Pull Request

PR Checklist

[x] Closes: #25353
[ ] Communication: I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected
[ ] Tests: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Dev docs: Added/updated
[ ] New binaries: Added on the required places

[ ] JSON for signing for new binaries
[ ] WXS for installer for new binaries and localization folder
[ ] YML for CI pipeline for new test projects
[ ] YML for signed pipeline

[ ] Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments

Fix FancyZones setting page bouncing when scroll to bottom (issue because combobox in this page use StackPanel make it bouncing when scroll to bottom > Use normal combobox)

Validation Steps Performed

Open FancyZones setting page > Scroll to bottom > Verify no bouncing anymore

Using a trackpad, i can still get the bounce to happen.  Via a mouse scroll wheel, i cannot get this to repro in any state.
Observation:
Zones sections, having both Zone behavior and appearance expanded, i can get it to stutter, but if either is collapsed, i cannot.   If i change to "Custom colors", It does not repro.
100% repro via trackpad with these two fully expanded.
That makes me believe this isn't the correct solution, the problem is somewhere in the "Zones" area and maybe something isn't being collapsed correctly.

shifting to a draft PR due to it still occurs.

Also, @quyenvsp, thank you very much for taking the time for creating a PR here, i do want to stress that

@crutkas Oh I am using desktop so don't know Touchpad has problem, I will check again. My fix is for Mouse scroll wheel in special case Have 2 Textbox in page expanded then 100% mouse scroll will bouncing
You can repro by temp code in main branch
Duplicate exclude textbox at src/settings-ui/Settings.UI/SettingsXAML/Views/FancyZonesPage.xaml:244~253
Then use mouse wheel to see it bouncing, then do same with this MR it will not happen.

@niels9001 any idea?


Ignored redundantPlugins still produces advice in output

Plugin version

0.74.0
Gradle version

6.9
(Optional) Android Gradle Plugin (AGP) version

4.1.2
Describe the bug

Can't suppress redundantPlugins check:
dependencyAnalysis {
    issues {
        all {
            onRedundantPlugins {
                severity("ignore")
            }
        }
    }
}

./gradlew :project:projectHealth

Still see the advice:
Plugin advice:
- java-library: this project has both java-library and org.jetbrains.kotlin.jvm applied, which is redundant. You can remove java-library

project/build/reports/dependency-analysis/advice-all-variants-pretty.json:
  "pluginAdvice": [
    {
      "redundantPlugin": "java-library",
      "reason": "this project has both java-library and org.jetbrains.kotlin.jvm applied, which is redundant. You can remove java-library"
    }
  ]

It makes it harder to read the buildHealth report with many unsuppressed messages.
To Reproduce
Steps to reproduce the behavior:

Checkout https://github.com/avito-tech/avito-android/tree/dependency-analysis-reproducer-ignore-redundantPlugins
./gradlew -p subprojects :gradle:build-properties:projectHealth

Expected behavior

No plugin advice at all.
Additional context

Let me know if you need any advice on how to get started with this.

This fix just landed on main.


Demo Documentation for Tabs broken

Describe the bug
Some demos in Documentation > Tabs
To Reproduce
Steps to reproduce the behavior:

Go to https://flowbite.com/docs/components/tabs/
Click on the button/tab you see in the image below...

Expected behavior
Switch tabs content
Screenshots

Desktop (please complete the following information):

OS: Windows
Browser: MS Edge
Version: the latest version

Smartphone (please complete the following information):
Did not test on smartphone.
Additional context
No.

Hi~These are just designs, so they are not actually broken.
You need to add attributes to the elements to make them switchable.
Check https://flowbite.com/docs/components/tabs/#interactive-tabs

@Lexachoc is right, the first few examples are static. We might update them though in the future.


Upload the Maven Central for Release Builds

Description
Once the Git flow process is tidied up, we need to add support in the build process for uploading to Maven Central, to make the client available for other projects in an easier fashion.
Steps to reproduce

The Maven site https://maven.apache.org/repository/guide-central-repository-upload.html has some notes. I haven't looked into it too much just yet.

Expected Behavior

Developers should be able to include the typesense-java client in their Maven/Gradle builds with appropriate version numbers and tags.

Actual Behavior

Not currently supported.

Metadata
Typsense Version:
OS:

@kishorenc Are there any plans for this?

This is done!
<dependency>
    <groupId>org.typesense</groupId>
    <artifactId>typesense-java</artifactId>
    <version>0.0.3</version>
</dependency>

hi @kishorenc any idea about this error?
Duplicate class com.fasterxml.jackson.core.Base64Variant found in modules jetified-jackson-core-2.9.6 (com.fasterxml.jackson.core:jackson-core:2.9.6) and jetified-typesense-java-0.0.3 (typesense-java-0.0.3.jar)


Invalid plugin specification aba_payment.

Hi b, Whenever I run flutter pub get, there is an error:
Invalid plugin specification aba_payment.
Invalid "plugin" specification.
How can I fix it? Or is it the package issue?

Hi b, Whenever I run flutter pub get, there is an error: Invalid plugin specification aba_payment. Invalid "plugin" specification.
How can I fix it? Or is it the package issue?

Can show me your pubspec.yaml?

Hi b, Whenever I run flutter pub get, there is an error: Invalid plugin specification aba_payment. Invalid "plugin" specification.
How can I fix it? Or is it the package issue?

Can show me your pubspec.yaml?

`name: payment_test
description: A new Flutter project.
The following line prevents the package from being accidentally published to
pub.dev using flutter pub publish. This is preferred for private packages.
publish_to: 'none' # Remove this line if you wish to publish to pub.dev
The following defines the version and build number for your application.
A version number is three numbers separated by dots, like 1.2.43
followed by an optional build number separated by a +.
Both the version and the builder number may be overridden in flutter
build by specifying --build-name and --build-number, respectively.
In Android, build-name is used as versionName while build-number used as versionCode.
Read more about Android versioning at https://developer.android.com/studio/publish/versioning
In iOS, build-name is used as CFBundleShortVersionString while build-number used as CFBundleVersion.
Read more about iOS versioning at
https://developer.apple.com/library/archive/documentation/General/Reference/InfoPlistKeyReference/Articles/CoreFoundationKeys.html
version: 1.0.0+1
environment:
sdk: '>=2.10.0 <3.0.0'
Dependencies specify other packages that your package needs in order to work.
To automatically upgrade your package dependencies to the latest versions
consider running flutter pub upgrade --major-versions. Alternatively,
dependencies can be manually updated by changing the version numbers below to
the latest version available on pub.dev. To see which dependencies have newer
versions available, run flutter pub outdated.
dependencies:
flutter:
sdk: flutter
aba_payment: ^0.0.1
The following adds the Cupertino Icons font to your application.
Use with the CupertinoIcons class for iOS style icons.
cupertino_icons: ^1.0.2
dev_dependencies:
flutter_test:
sdk: flutter
The "flutter_lints" package below contains a set of recommended lints to
encourage good coding practices. The lint set provided by the package is
activated in the analysis_options.yaml file located at the root of your
package. See that file for information about deactivating specific lint
rules and activating additional ones.
flutter_lints: ^1.0.0
For information on the generic Dart part of this file, see the
following page: https://dart.dev/tools/pub/pubspec
The following section is specific to Flutter.
flutter:
The following line ensures that the Material Icons font is
included with your application, so that you can use the icons in
the material Icons class.
uses-material-design: true
To add assets to your application, add an assets section, like this:
assets:
- images/a_dot_burr.jpeg
- images/a_dot_ham.jpeg
An image asset can refer to one or more resolution-specific "variants", see
https://flutter.dev/assets-and-images/#resolution-aware.
For details regarding adding assets from package dependencies, see
https://flutter.dev/assets-and-images/#from-packages
To add custom fonts to your application, add a fonts section here,
in this "flutter" section. Each entry in this list should have a
"family" key with the font family name, and a "fonts" key with a
list giving the asset and other descriptors for the font. For
example:
fonts:
- family: Schyler
fonts:
- asset: fonts/Schyler-Regular.ttf
- asset: fonts/Schyler-Italic.ttf
style: italic
- family: Trajan Pro
fonts:
- asset: fonts/TrajanPro.ttf
- asset: fonts/TrajanPro_Bold.ttf
weight: 700

For details regarding fonts from package dependencies,
see https://flutter.dev/custom-fonts/#from-packages
`

Hi b, Whenever I run flutter pub get, there is an error: Invalid plugin specification aba_payment. Invalid "plugin" specification.
How can I fix it? Or is it the package issue?

Can show me your pubspec.yaml?

`name: payment_test
description: A new Flutter project.
The following line prevents the package from being accidentally published to
pub.dev using flutter pub publish. This is preferred for private packages.
publish_to: 'none' # Remove this line if you wish to publish to pub.dev
The following defines the version and build number for your application.
A version number is three numbers separated by dots, like 1.2.43
followed by an optional build number separated by a +.
Both the version and the builder number may be overridden in flutter
build by specifying --build-name and --build-number, respectively.
In Android, build-name is used as versionName while build-number used as versionCode.
Read more about Android versioning at https://developer.android.com/studio/publish/versioning
In iOS, build-name is used as CFBundleShortVersionString while build-number used as CFBundleVersion.
Read more about iOS versioning at
https://developer.apple.com/library/archive/documentation/General/Reference/InfoPlistKeyReference/Articles/CoreFoundationKeys.html
version: 1.0.0+1
environment:
sdk: '>=2.10.0 <3.0.0'
Dependencies specify other packages that your package needs in order to work.
To automatically upgrade your package dependencies to the latest versions
consider running flutter pub upgrade --major-versions. Alternatively,
dependencies can be manually updated by changing the version numbers below to
the latest version available on pub.dev. To see which dependencies have newer
versions available, run flutter pub outdated.
dependencies:
flutter:
sdk: flutter
aba_payment: ^0.0.1
The following adds the Cupertino Icons font to your application.
Use with the CupertinoIcons class for iOS style icons.
cupertino_icons: ^1.0.2
dev_dependencies:
flutter_test:
sdk: flutter
The "flutter_lints" package below contains a set of recommended lints to
encourage good coding practices. The lint set provided by the package is
activated in the analysis_options.yaml file located at the root of your
package. See that file for information about deactivating specific lint
rules and activating additional ones.
flutter_lints: ^1.0.0
For information on the generic Dart part of this file, see the
following page: https://dart.dev/tools/pub/pubspec
The following section is specific to Flutter.
flutter:
The following line ensures that the Material Icons font is
included with your application, so that you can use the icons in
the material Icons class.
uses-material-design: true
To add assets to your application, add an assets section, like this:
assets:
- images/a_dot_burr.jpeg
- images/a_dot_ham.jpeg
An image asset can refer to one or more resolution-specific "variants", see
https://flutter.dev/assets-and-images/#resolution-aware.
For details regarding adding assets from package dependencies, see
https://flutter.dev/assets-and-images/#from-packages
To add custom fonts to your application, add a fonts section here,
in this "flutter" section. Each entry in this list should have a
"family" key with the font family name, and a "fonts" key with a
list giving the asset and other descriptors for the font. For
example:
fonts:
- family: Schyler
fonts:
- asset: fonts/Schyler-Regular.ttf
- asset: fonts/Schyler-Italic.ttf
style: italic
- family: Trajan Pro
fonts:
- asset: fonts/TrajanPro.ttf
- asset: fonts/TrajanPro_Bold.ttf
weight: 700

For details regarding fonts from package dependencies,
see https://flutter.dev/custom-fonts/#from-packages
`

Hi b, Whenever I run flutter pub get, there is an error: Invalid plugin specification aba_payment. Invalid "plugin" specification.
How can I fix it? Or is it the package issue?

Can show me your pubspec.yaml?

description: A new Flutter project.

# The following line prevents the package from being accidentally published to
# pub.dev using `flutter pub publish`. This is preferred for private packages.
publish_to: 'none' # Remove this line if you wish to publish to pub.dev

# The following defines the version and build number for your application.
# A version number is three numbers separated by dots, like 1.2.43
# followed by an optional build number separated by a +.
# Both the version and the builder number may be overridden in flutter
# build by specifying --build-name and --build-number, respectively.
# In Android, build-name is used as versionName while build-number used as versionCode.
# Read more about Android versioning at https://developer.android.com/studio/publish/versioning
# In iOS, build-name is used as CFBundleShortVersionString while build-number used as CFBundleVersion.
# Read more about iOS versioning at
# https://developer.apple.com/library/archive/documentation/General/Reference/InfoPlistKeyReference/Articles/CoreFoundationKeys.html
version: 1.0.0+1

environment:
  sdk: '>=2.10.0 <3.0.0'

# Dependencies specify other packages that your package needs in order to work.
# To automatically upgrade your package dependencies to the latest versions
# consider running `flutter pub upgrade --major-versions`. Alternatively,
# dependencies can be manually updated by changing the version numbers below to
# the latest version available on pub.dev. To see which dependencies have newer
# versions available, run `flutter pub outdated`.
dependencies:
  flutter:
    sdk: flutter
  aba_payment: ^0.0.1

  # The following adds the Cupertino Icons font to your application.
  # Use with the CupertinoIcons class for iOS style icons.
  cupertino_icons: ^1.0.2

dev_dependencies:
  flutter_test:
    sdk: flutter

  # The "flutter_lints" package below contains a set of recommended lints to
  # encourage good coding practices. The lint set provided by the package is
  # activated in the `analysis_options.yaml` file located at the root of your
  # package. See that file for information about deactivating specific lint
  # rules and activating additional ones.
  flutter_lints: ^1.0.0

# For information on the generic Dart part of this file, see the
# following page: https://dart.dev/tools/pub/pubspec

# The following section is specific to Flutter.
flutter:

  # The following line ensures that the Material Icons font is
  # included with your application, so that you can use the icons in
  # the material Icons class.
  uses-material-design: true

  # To add assets to your application, add an assets section, like this:
  # assets:
  #   - images/a_dot_burr.jpeg
  #   - images/a_dot_ham.jpeg

  # An image asset can refer to one or more resolution-specific "variants", see
  # https://flutter.dev/assets-and-images/#resolution-aware.

  # For details regarding adding assets from package dependencies, see
  # https://flutter.dev/assets-and-images/#from-packages

  # To add custom fonts to your application, add a fonts section here,
  # in this "flutter" section. Each entry in this list should have a
  # "family" key with the font family name, and a "fonts" key with a
  # list giving the asset and other descriptors for the font. For
  # example:
  # fonts:
  #   - family: Schyler
  #     fonts:
  #       - asset: fonts/Schyler-Regular.ttf
  #       - asset: fonts/Schyler-Italic.ttf
  #         style: italic
  #   - family: Trajan Pro
  #     fonts:
  #       - asset: fonts/TrajanPro.ttf
  #       - asset: fonts/TrajanPro_Bold.ttf
  #         weight: 700
  #
  # For details regarding fonts from package dependencies,
  # see https://flutter.dev/custom-fonts/#from-packages

Hi b, Whenever I run flutter pub get, there is an error: Invalid plugin specification aba_payment. Invalid "plugin" specification.
How can I fix it? Or is it the package issue?

Can show me your pubspec.yaml?

description: A new Flutter project.

# The following line prevents the package from being accidentally published to
# pub.dev using `flutter pub publish`. This is preferred for private packages.
publish_to: 'none' # Remove this line if you wish to publish to pub.dev

# The following defines the version and build number for your application.
# A version number is three numbers separated by dots, like 1.2.43
# followed by an optional build number separated by a +.
# Both the version and the builder number may be overridden in flutter
# build by specifying --build-name and --build-number, respectively.
# In Android, build-name is used as versionName while build-number used as versionCode.
# Read more about Android versioning at https://developer.android.com/studio/publish/versioning
# In iOS, build-name is used as CFBundleShortVersionString while build-number used as CFBundleVersion.
# Read more about iOS versioning at
# https://developer.apple.com/library/archive/documentation/General/Reference/InfoPlistKeyReference/Articles/CoreFoundationKeys.html
version: 1.0.0+1

environment:
  sdk: '>=2.10.0 <3.0.0'

# Dependencies specify other packages that your package needs in order to work.
# To automatically upgrade your package dependencies to the latest versions
# consider running `flutter pub upgrade --major-versions`. Alternatively,
# dependencies can be manually updated by changing the version numbers below to
# the latest version available on pub.dev. To see which dependencies have newer
# versions available, run `flutter pub outdated`.
dependencies:
  flutter:
    sdk: flutter
  aba_payment: ^0.0.1

  # The following adds the Cupertino Icons font to your application.
  # Use with the CupertinoIcons class for iOS style icons.
  cupertino_icons: ^1.0.2

dev_dependencies:
  flutter_test:
    sdk: flutter

  # The "flutter_lints" package below contains a set of recommended lints to
  # encourage good coding practices. The lint set provided by the package is
  # activated in the `analysis_options.yaml` file located at the root of your
  # package. See that file for information about deactivating specific lint
  # rules and activating additional ones.
  flutter_lints: ^1.0.0

# For information on the generic Dart part of this file, see the
# following page: https://dart.dev/tools/pub/pubspec

# The following section is specific to Flutter.
flutter:

  # The following line ensures that the Material Icons font is
  # included with your application, so that you can use the icons in
  # the material Icons class.
  uses-material-design: true

  # To add assets to your application, add an assets section, like this:
  # assets:
  #   - images/a_dot_burr.jpeg
  #   - images/a_dot_ham.jpeg

  # An image asset can refer to one or more resolution-specific "variants", see
  # https://flutter.dev/assets-and-images/#resolution-aware.

  # For details regarding adding assets from package dependencies, see
  # https://flutter.dev/assets-and-images/#from-packages

  # To add custom fonts to your application, add a fonts section here,
  # in this "flutter" section. Each entry in this list should have a
  # "family" key with the font family name, and a "fonts" key with a
  # list giving the asset and other descriptors for the font. For
  # example:
  # fonts:
  #   - family: Schyler
  #     fonts:
  #       - asset: fonts/Schyler-Regular.ttf
  #       - asset: fonts/Schyler-Italic.ttf
  #         style: italic
  #   - family: Trajan Pro
  #     fonts:
  #       - asset: fonts/TrajanPro.ttf
  #       - asset: fonts/TrajanPro_Bold.ttf
  #         weight: 700
  #
  # For details regarding fonts from package dependencies,
  # see https://flutter.dev/custom-fonts/#from-packages

Okay bro, I will recheck this library soon. With latest version of flutter 2.5, I didn't try it yet, so you might using the latest flutter now that push to error like this.

null safety was merged


Ensure service is enabled for SystemD

Ensure that the service is properly enabled for SystemD.

Closed by #7 and #8


Node v1.18.2 throws this error

Even though the documentation clearly mentions that the newly supported version for Node.js is v1.18, I get the following error when running gulp serve
Error: Your dev environment is running NodeJS version v18.20.3 which does not meet the requirements for running this tool. This tool requires a version of NodeJS that matches >=12.13.0 <13.0.0 || >=14.15.0 <15.0.0 || >=16.13.0 <17.0.0
Am I missing something?

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

ID: 5bee09a8-022e-1b5a-4aab-d6e7c6216808
Version Independent ID: 49edce1e-f2be-e989-917f-c6b389363e2f
Content: Set up your SharePoint Framework development environment
Content Source: docs/spfx/set-up-your-development-environment.md
Service: sharepoint-online
Sub-service: sharepoint-framework
GitHub Login: @VesaJuvonen
Microsoft Alias: vesaj

Please clarify... there's no such Nodejs version v1.18... are you mixing SPFx & Node versions?
Share some context on your environment, like:

output of node -v
output of npm list -g --depth=0
contents of your SPFx project's package.json file

Oh, sorry that was a mistake. I meant Node.js v18. When I run gulp server I get the error as I mentioned above. Here is my node version output:
Run: node -v
v18.20.3
Run: npm list -g --depth=0
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>package.json:
{
"name": "welcome-howi",
"version": "0.0.1",
"private": true,
"engines": {
"node": ">=16.13.0 <17.0.0"
},
"main": "lib/index.js",
"scripts": {
"build": "gulp bundle",
"clean": "gulp clean",
"test": "gulp test"
},
"dependencies": {
"@microsoft/sp-component-base": "1.17.3",
"@microsoft/sp-core-library": "1.17.3",
"@microsoft/sp-lodash-subset": "1.17.3",
"@microsoft/sp-office-ui-fabric-core": "1.17.3",
"@microsoft/sp-property-pane": "1.17.3",
"@microsoft/sp-webpart-base": "1.17.3",
"tslib": "2.3.1"
},
"devDependencies": {
"@fluentui/react": "^7.199.1",
"@microsoft/eslint-config-spfx": "1.17.3",
"@microsoft/eslint-plugin-spfx": "1.17.3",
<EMAIL_ADDRESS>"0.5.0",
"@microsoft/sp-build-web": "1.17.3",
"@microsoft/sp-module-interfaces": "1.17.3",
"@rushstack/eslint-config": "2.5.1",
"@types/webpack-env": "~1.15.2",
"ajv": "^6.12.5",
"eslint": "8.7.0",
"gulp": "4.0.2",
"typescript": "4.5.5"
}
}

The project is using SPFx v1.17.3. That version only supports Node.js v16, which is what this line in your package.json is saying:
{
  ...
  "engines": {
    "node": ">=16.13.0 <17.0.0"
  }
  ...
}

So, your options are to:

Downgrade your Node.js install > Node.js v16
Upgrade your SPFx project to a version that supports Node.js v18. That's SPFx v1.18.0+
Use NVM to have multiple Node.js installs

FWIW, I'd go with the last option above. Learn more: https://www.voitanos.io/blog/better-node-js-install-management-with-node-version-manager/

If it helps, using PNPM instead of NPM allow to specify the node's version inside .npmrc.
Details : https://pnpm.io/npmrc#use-node-version.
I use the settings for some years now, and I moved out of any node version manager.
Just install latest node LTS, and specify in each project the required node version.
That said, I strongly advocate using pnpm instead of npm. Not only because of the speed and disk usage improvements but also because PNPM requires you to have clean dependencies (see https://rushjs.io/pages/advanced/phantom_deps/ and https://rushjs.io/pages/advanced/npm_doppelgangers/).


Pass request to getRequestConfig()

Is your feature request related to a problem? Please describe.
To fetch the user's locale from the database and apply it to the next-intl config, I need to read values from the incoming request. However, there is no way to access the request from getRequestConfig().
Describe the solution you'd like
Pass the request as a parameter to getRequestConfig().
Describe alternatives you've considered
Apply the getRequestConfig() middleware later so that my own middleware was able to run and inject necessary headers into the request.

i18n.ts (with its getRequestConfig) is called during the RSC render (see docs). You can read from headers() or cookies() there, but the raw request is not available for Server Components (only in the middleware).

Thank you so much for your quick reply and the hint. I must've missed this in the docs so please excuse the feature request.

I think I'll add a hint about this to make it a bit more explicit. Good that you've brought this up @llamadeus!


internal/goglfw: separate windows specific struct from common structs

What issue is this addressing?
Updates #2546
What type of issue is this addressing?
feature
What this PR does | solves
Many of the structs found in internal.h (internal_windows.go) contain defines such as GLFW_WIN32_WINDOW_STATE which gets replaced with a struct defined in win32platform_windows.go (win32_platform.h).
Originally, these structs where directly placed inside of internal_windows.go. However, to make it easier to add macOS and Linux these cannot be in this file.
This commit separates the windows specific structs into the respective windows file and updates the field to be named state.

@hajimehoshi this test failure doesn't look related to this commit?

@hajimehoshi this test failure doesn't look related to this commit?

I believe so.


[FEATURE] Migrate plugin away from using enzymejs for its unit tests.

Is your feature request related to a problem?
Bumping the version of OpenSearch-Dashboards used by the alerting-dashboards-plugin to run unit tests from 1.3 to 2.0 caused the following 4 unit tests to start failing:

DelayedLoader.test.js should set Timer for 1 seconds if initial loading is true
DelayedLoader.test.js should clear Timer on componentWillUnmount if exists
DelayedLoader.test.js should not show loader if data fetching is finished before threshold
WhereExpression.test.js calls closeExpression when closing popover: This test examines the behavior of closing an EuiPopover component. OSD 2.0 bumped the version of EUI used by the plugins from 29.3.2 to 34.6.0. The changelog mentions that changes were made to the functionality of EuiPopover, and this PR describes changes that relate to handling escaping from a popover which impacted this test. This change seems to impact the way this component behaves when testing using enzymejs.

What solution would you like?
The alerting-dashboards-plugin should likely migrate away from using enzymejs to align with the other frontend plugins.

Test 4 mentioned above has been set to skip for now.

Refactoring the testing architecture in this way will be a backlog enhancement.


fix(browser): fix reference query param

Fixes: #370

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 88.34%. Comparing base (6ab80d0) to head (985b343).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #371   +/-   ##
=======================================
  Coverage   88.34%   88.34%
=======================================
  Files          10       10
  Lines        3699     3699
=======================================
  Hits         3268     3268
  Misses        431      431

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


When the operator is deleted, they are not removed from status

/kind bug
What steps did you take and what happened:
[A clear and concise description of what the bug is.]
When I set an operator to the absent state, after it is deleted, I found it's still in the member list of the status.
apiVersion: operator.ibm.com/v1alpha1
kind: OperandRequest
metadata:
  annotations:
    kubectl.kubernetes.io/last-applied-configuration: >
      {"apiVersion":"operator.ibm.com/v1alpha1","kind":"OperandRequest","metadata":{"annotations":{},"name":"common-service","namespace":"ibm-common-services"},"spec":{"services":[{"name":"ibm-cert-manager-operator","state":"present"},{"name":"ibm-mongodb-operator","state":"present"},{"name":"ibm-iam-operator","state":"present"},{"name":"ibm-management-ingress-operator","state":"absent"},{"name":"ibm-ingress-nginx-operator","state":"absent"},{"name":"ibm-metering-operator","state":"absent"},{"name":"ibm-licensing-operator","state":"absent"},{"name":"ibm-commonui-operator","state":"absent"},{"name":"ibm-healthcheck-operator","state":"present"},{"name":"ibm-auditlogging-operator","state":"absent"},{"name":"ibm-catalog-operator","state":"absent"},{"name":"ibm-platform-api-operator","state":"absent"}]}}
  selfLink: >-
    /apis/operator.ibm.com/v1alpha1/namespaces/ibm-common-services/operandrequests/common-service
  resourceVersion: '9623622'
  name: common-service
  uid: 5cfb4833-5a8a-11ea-8270-0aa228443300
  creationTimestamp: '2020-02-29T00:28:10Z'
  generation: 3
  namespace: ibm-common-services
  finalizers:
    - finalizer.request.ibm.com
spec:
  services:
    - name: ibm-cert-manager-operator
      state: absent
    - name: ibm-mongodb-operator
      state: present
    - name: ibm-iam-operator
      state: absent
    - name: ibm-management-ingress-operator
      state: absent
    - name: ibm-ingress-nginx-operator
      state: absent
    - name: ibm-metering-operator
      state: absent
    - name: ibm-licensing-operator
      state: absent
    - name: ibm-commonui-operator
      state: absent
    - name: ibm-healthcheck-operator
      state: present
    - name: ibm-auditlogging-operator
      state: absent
    - name: ibm-catalog-operator
      state: absent
    - name: ibm-platform-api-operator
      state: absent
status:
  conditions:
    - lastTransitionTime: '2020-02-28T19:31:04-05:00'
      lastUpdateTime: '2020-02-28T19:31:04-05:00'
      message: Creating subscription ibm-cert-manager-operator
      reason: Creating subscription
      status: 'True'
      type: Creating
    - lastTransitionTime: '2020-02-28T19:31:04-05:00'
      lastUpdateTime: '2020-02-28T19:31:04-05:00'
      message: Creating subscription ibm-healthcheck-operator
      reason: Creating subscription
      status: 'True'
      type: Creating
    - lastTransitionTime: '2020-02-28T19:31:04-05:00'
      lastUpdateTime: '2020-02-28T19:31:04-05:00'
      message: Creating subscription ibm-mongodb-operator
      reason: Creating subscription
      status: 'True'
      type: Creating
    - lastTransitionTime: '2020-02-28T19:31:05-05:00'
      lastUpdateTime: '2020-02-28T19:31:05-05:00'
      message: Creating subscription ibm-iam-operator
      reason: Creating subscription
      status: 'True'
      type: Creating
    - lastTransitionTime: '2020-02-28T19:37:49-05:00'
      lastUpdateTime: '2020-02-28T19:37:49-05:00'
      message: Deleting subscription ibm-cert-manager-operator
      reason: Deleting subscription
      status: 'True'
      type: Deleting
    - lastTransitionTime: '2020-02-28T19:37:50-05:00'
      lastUpdateTime: '2020-02-28T19:37:50-05:00'
      message: Deleting csv ibm-cert-manager-operator.v3.7.0
      reason: Deleting csv
      status: 'True'
      type: Deleting
    - lastTransitionTime: '2020-02-28T19:40:49-05:00'
      lastUpdateTime: '2020-02-28T19:40:49-05:00'
      message: Deleting subscription ibm-iam-operator
      reason: Deleting subscription
      status: 'True'
      type: Deleting
    - lastTransitionTime: '2020-02-28T19:40:50-05:00'
      lastUpdateTime: '2020-02-28T19:40:50-05:00'
      message: Deleting csv ibm-iam-operator.v3.5.0
      reason: Deleting csv
      status: 'True'
      type: Deleting
  members:
    - name: ibm-cert-manager-operator
      phase:
        operatorPhase: Succeeded
    - name: ibm-healthcheck-operator
      phase:
        operatorPhase: Succeeded
    - name: ibm-iam-operator
      phase:
        operatorPhase: Installing
    - name: ibm-mongodb-operator
      phase:
        operatorPhase: Succeeded
  phase: Creating

@DanielXLee
What did you expect to happen:
The deleted operator can be removed from the status.members  and status.phase can be updated.
Anything else you would like to add:
[Miscellaneous information that will assist in solving the issue.]
Environment:

ODLM version: 0.0.1
Minikube/KIND/OCP version: OCP 4.2
Kubernetes version: (use kubectl version):
OS (e.g. from /etc/os-release):

yes, this is a member status update issue, I will fix this issue in enhance https://github.com/IBM/operand-deployment-lifecycle-manager/issues/147


Adding annotation line

@fsdavid I will merge this first into dev_annotation, since there are some changes which will benefit the dev_annotation branch.
feel free to continue review the code, and add your comments either here or in dev_annotation, and ping me


Add API Release Guidelines to GitHub - extract of the API Release Pro…

…cess on CAMARA Confluence
Some links to other GitHub resources may need to be updated
What type of PR is this?
Add one of the following kinds:

documentation

What this PR does / why we need it:
Include the file with an extract of the API release process for the completeness of GitHub documentation.
Which issue(s) this PR fixes:

Fixes #30
Special notes for reviewers:
Changelog input
 Add the API release guidelines (extract from CAMARA Confluence)

Additional documentation
This section can be blank.

Reading over the document my thought is that we need to make the point that release numbering and the API versions within a release are independent as clear as possible. That should be a disclaimer on the top of the document. And there should be a link to the API versioning in this disclaimer.


Feature designv4

:tada: This PR is included in version 2.7.1-rc.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.7.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


fix: cloudaccount try get info of empty project

这个 PR 实现什么功能/修复什么问题:
修正：cloudaccount的projectId可能为空，导致详情中获取空的项目详情
是否需要 backport 到之前的 release 分支:

release/2.13

/area region
/cc @yousong

Codecov Report

Merging #4354 into master will increase coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master   #4354      +/-   ##
=========================================
+ Coverage    6.87%   6.87%   +<.01%
=========================================
  Files         521     521
  Lines       88094   88095       +1
=========================================
+ Hits         6055    6059       +4
+ Misses      81533   81531       -2
+ Partials      506     505       -1

Flag
Coverage Δ

#aFlag
6.87% <0%> (ø)
:arrow_up:

Impacted Files
Coverage Δ

pkg/compute/models/cloudaccounts.go
0.86% <0%> (-0.01%)
:arrow_down:

pkg/cloudcommon/db/lockman/inmemory.go
67.07% <0%> (+2.43%)
:arrow_up:

pkg/appsrv/response.go
69.23% <0%> (+3.07%)
:arrow_up:

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 d4ed948...57ce102. Read the comment docs.

/lgtm
/approve


Using build.rollupOptions.input with --outDir fails to build

Describe the bug
When using build.rollupOptions.input (e.g.: like this), this works:
npm exec vite build

This does not:
npm exec vite build --outDir path/to/dist

Nor does this:
npm exec vite build --outDir dist

I've removed unrelated lines from my vite.config.js below to a minimal set that reproduces this.
Reproduction
https://paste.sr.ht/~whynothugo/8da81e5c2e30a4f6f07317d7f8d8c5d4d0f7d6b4
Steps to reproduce

Run npm exec vite build. It works.
Run npm exec vite build --outDir dist. It will fail with the following output:

vite v5.1.4 building for production...
✓ 0 modules transformed.
x Build failed in 4ms
error during build:
RollupError: Could not resolve entry module "dist/index.html".
    at getRollupEror (file:///app/node_modules/vite/node_modules/rollup/dist/es/shared/parseAst.js:375:41)
    at error (file:///app/node_modules/vite/node_modules/rollup/dist/es/shared/parseAst.js:372:42)
    at ModuleLoader.loadEntryModule (file:///app/node_modules/vite/node_modules/rollup/dist/es/shared/node-entry.js:18928:20)
    at async Promise.all (index 0)

Note the reference to index.html; building this project (without --outDir) generates no such file.
System Info
System:
    OS: Linux 6.7 Alpine Linux
    CPU: (24) x64 13th Gen Intel(R) Core(TM) i7-13700K
    Memory: 9.50 GB / 62.57 GB
    Container: Yes
    Shell: 1.36.1 - /bin/ash
  Binaries:
    Node: 18.18.0 - /usr/local/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 9.8.1 - /usr/local/bin/npm
  npmPackages:
    @vitejs/plugin-vue: ^5.0.4 => 5.0.4

Used Package Manager
npm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Here's a full reproduction example:
git clone https://git.sr.ht/~whynothugo/vite-16256
cd vite-16256
docker run --rm -it -v "$(pwd):/src" -w /src node:20-alpine sh
npm i

Running npm exec vite build --outDir path/to/dist fails, running npm exec vite build works.
I kept package-lock.json for reference only; you can delete it before running npm i and the (current) latest versions also reproduce the issue.
I originally reproduced this with node:18-alpine, but node:20-alpine also reproduces this.

Oh, it turns out that npm exec is silently swallowing the --outDir flag.
This fixes the issue:
@@ -1 +1 @@
-npm exec vite build --outDir dist
+npm exec -- vite build --outDir dist

Sorry for the noise!


Cupertinocontextmenu add previewBuilder

Use case
Cupertinocontextmenu add previewBuilder
In older version of Flutter has the Beautiful Feature in the cupertinocontexmenu there is option called previewbuilder. In Later version that has been removed from the flutter I don't
I request anyone can implement the feature again in the cupertino Libary .
The previewBuilder was very use when you  want show any other in the preview.
Proposal
For ref Flutter 3.19.0

Hi @sudhar08, it isn't exactly clear what you're reporting in this issue. previewBuilder was replaced with builder. Is there something previewBuilder did that builder doesn't do? It would help if you could provide a code sample that showcases what you want to achieve and what you have tried along with screenshots.
Thank you


Feature Request: --environment-resource-group argument for specifying the resource group where the container app environment lives

Is your feature request related to a problem? Please describe.
The az containerapp create command always assumes that the container app environment is in the same resource group as the container app you're trying to create. When using the az containerapp create command, I cannot specify a different resource group for the container app environment. I can specify a different resource group when using the portal, or doing a YAML deploy. The only place where I have the ability to specify a resource group is the -g or --resource-group flag, and that flag is to specify where the container app should be deployed.
Example:

Create resource-group-1
Create resource-group-2
Deploy container-environment in resource-group-1
Deploy container-app in resource-group-2

When creating, use container-environment as the environment for container-app (not possible via command line because there is no option unless you're using YAML deploy)

If you try to specify -g resource-group-2 and --environment container-environment, it always assumes that the environment is in the same resource group as the container app you're trying to create, and gives you the error:
ERROR: The environment '/subscriptions/asdfasdf-asdfasdf-asdfasdf-asdfasdfasdf/resourceGroups/resource-group-2/providers/Microsoft.App/managedEnvironments/container-environment' does not exist. Specify a valid environment
Describe the solution you'd like.
There should be an additional parameter to specify which resource group the environment is in, something like --environment-resource-group. See below for more details on why, but this is currently already possible in 2 different ways. It feels like a very intuitive feature to add to the az containerapp create command.
Describe alternatives you've considered.
This is currently already possible in the portal (see screenshot below. I blanked out part of the RG names for security purposes, but you can see that they are different resource groups).

It is also possible to do this as a part of the az containerapp create --yaml command by specifying the fully-qualified resource name in the managedEnvironmentId: parameter https://learn.microsoft.com/en-us/azure/container-apps/azure-resource-manager-api-spec?tabs=yaml#container-app-examples:~:text=managedEnvironmentId ...
Additional context.
Thank you for your time!

@sanfordn I tried this today and it worked for me,
the --environment parameter not only accepts the name, but also the id,
"Name or resource ID of the container app's environment."
so I did,
az containerapp create -n thenameofmyapp -g somerg --image someacr.azurecr.io/somename:12345 --environment '/subscriptions/the-guid-of-the-sub-where-the-env-lives/resourceGroups/the-rg-of-the-env/providers/Microsoft.App/managedEnvironments/the-name-of-the-env' --registry-server someacr.azurecr.io --registry-username someacr --registry-password theacrpassword
and it worked fine

Haven't checked back on this in a while, and I'm seeing that this would be a valid solution. thanks for the response here.


Alphabetize "Colleges and Main Units" and "Campus Partners" filters list

@caadler and @Ruthie2018 is this for all filters lists?

Test document: https://drive.google.com/open?id=1-RIzczHoLbFt7DcIy-kbFvv04Tfdvvb1QlomM4pBHmk


replace process.kill with process.exit

Closes #352

#352 is very annoying, and this patch fixes it. @tststs any possibility to merge that ?


Q about another langauge

Hello, I train model from scratch ( audio feature select with Kor wec2vec) with kor .mp4 .
but result show just head , not bg or torso  (even i give --torso , --bg_img )
Do you have any idea?  thx for reading

To include the torso in the output video you need to use the .pth file which is in the trial_videoname_torso/checkpoints folder.

Hello, I train model from scratch ( audio feature select with Kor wec2vec) with kor .mp4 . but result show just head , not bg or torso (even i give --torso , --bg_img )
Do you have any idea? thx for reading

To include the torso in the output video you need to use the .pth file which is in the trial_videoname_torso/checkpoints folder.


Godot mono 3.5 beta5 fails to export

Godot version
v3.5.beta5.mono.official [815f7fe63]
System information
Win10.0.19043, Ryzen 4800HS, GTX 1660Ti
Issue description
Exporting project fails without error message in beta5.
No Data_XYZ is created. For larger (not the included example) project the *.pck becomes only a fraction of its expected size (1.5MB instead of +90MB).
Tried turning off rcedit & signtool, but no change.
Only tested on Win10.
Loading same project into v3.5.beta4.mono.official [b6968ab06] exports fine.
Steps to reproduce
Attached project can be exported in 3.5b4, but in 3.5b5 the "data_XYZ" directory is not created.
The project work when run in the editor, but as export the mono-code seems to not be executed.
E.g. the button do not work in the exported version.
Minimal reproduction project
3.5_B5_monotest.zip

This was confirmed by @bruvzg, who fixed it in #60752. I'll keep the issue open and pinned for a couple of days as other users might run into it in 3.5 beta 5.

Fixed by https://github.com/godotengine/godot/pull/60752.


openblas win64 int64 visual stodio2015 0x000007b error

My computer is x64 based, and I compiled the program with x64 compiler. When I run my program, it gives 0x000007b error code.

From a quick google for the code, this error could mean that your program finds a 32bit dll at runtime instead of the required 64bit version.

Please use win64-int32 dll

Thank you! Blas can run well now. But there is another problem. When I include lapacke.h, the program can't even be complied.
It suggests that loss ';' in front of  the name "lapack_make_complex_float"
Is there something I forgot to do?

Oh, this maybe a bug at lapacke.h


Plant Sower lack of automation until "filter" get's full of seeds.

Industrial Foregoing Version: any
TeslaCoreLib Version:  any
First I would like to thank you for your hard work and very useful mod.
This is not an direct issue/bug but more like a QoL/feature request.
The Plant Sower doesn't have an inventory and so you have to have a minimum of 144 of the seeds you want to be able to automate it else it's going to be a temporary run back and forth to manually plant your seeds in all the available slots so you get more seeds back.
Please let me know if there is another way of doing this and thanks if you are going to consider adding a feature like MFR's old planter.

Use Item Splitter

Ugh, ok got me some time to figure that out, I think you should explain this better in the manual.
Thank you.


Relax constraint on derived type overrides

fix #1538

I believe it was a more simple case to allow it for discriminator purpose. I think the problem with allowing this generic system is it really not compatible with csharp inheritance logic for example.


Bot Multi-turn prompt Feature issue QnaMaker

Github issues should be used for bugs and feature requests. Use Stack Overflow for general "how-to" questions.
Version
What package version of the SDK are you using: Bot Framework V4
Describe the bug
I have a direct line bot connected to QnaMaker and testing the new prompt feature on how to respond to the user, below sample Qna from QnaMaker
1)
q) How to get New York Printer details(QnAID: 13)
a) details are below NY73889-B0

2)
q) How to get Dallas printer details (QnAID: 14)
a) details are below DL093883-B1

3)
q) printer location
a) please chose from the 2 options below
   Dallas(QnAID: 13)  New York(QnAID: 14)    (Prompts)
4)
q)Dallas
a) Dallas is a beautiful city

5)
q)New York
a) City of lights

The issue: the user asks a question "printer location" and gets the response with a prompt New York and Dallas. Then when the user select one of the location say Dallas, ideally it should respond with the answer

q) How to get Dallas printer details (QnAID: 14)
a) details are below DL093883-B1  since we have the prompt connected to the question 4 and we use the old state which has the selected option QnaID for dallas.. but instead it gives the answer below since it only matches with the dallas prompt text responded by user without considering the old State, any help is appreciated

q)Dallas
a) Dallas is a beautiful city
protected override async Task<(object newState, IEnumerable output, object result)> ProcessAsync(object oldState, Activity inputActivity)
{
Activity outputActivity = null;
QnABotState newState = null;
var query = inputActivity.Text; ;
//Step 4) We Build the json based on the value selected user so we make a call to the Qna Maker using the response from the user on the button click and the previous state
//here we check if there was a prompt  from the previous question and update the state to send to the bot for response based on the option selected
if (((QnAPrompting.Models.QnABotState)oldState) != null)
{
    for (int i = 0; i < ((QnABotState)oldState).PreviousPromptValues.Length; i++)
    {
        if (((QnABotState)oldState).PreviousPromptValues[i].DisplayText == query)
        {
            ((QnABotState)oldState).qnaId = ((QnABotState)oldState).PreviousPromptValues[i].QnaId;
            ((QnABotState)oldState).question = query;
        }
    }
} //Step5). when we get the response from the user after the choosing an option from the prompt, it queries the qnamaker and filters based on the old state as well
//Step1). we get the prompts based on the question asked for the first time the old state is empty
var qnaResult = await _qnaService.QueryQnAServiceAsync(query, (QnABotState)oldState);
var qnaAnswer = qnaResult[0].Answer;
var prompts = qnaResult[0].Context?.Prompts;

if (prompts == null || prompts.Length < 1)
{

    if (((QnAPrompting.Models.QnABotState)oldState) != null)
    {
        if (((QnAPrompting.Models.QnABotState)oldState).IsPreviousPrompt)
        {
            string method = "/knowledgebases/{0}/{1}/qna/";
            var method_with_id = String.Format(method, "KBID", "prod");
            var uri = "https://westus.api.cognitive.microsoft.com" + "/qnamaker/v4.0" + method_with_id;
            Console.WriteLine("Calling " + uri + ".");
            var response = await Get(uri);
        }
    }
    outputActivity = MessageFactory.Text(qnaAnswer);
}
else //Step 2.)prompt parameters are stored in the state
{

    ContextValues ctx = new ContextValues();
    ctx.PreviousQnaId = qnaResult[0].Id;
    ctx.PreviousUserQuery = query;
    //store here in azure table
    newState = new QnABotState
    {
        question = string.Empty,
        top=10,
        userId = "Default",
        context = ctx,
        PreviousPromptValues = prompts,
    };

    outputActivity = CardHelper.GetHeroCard(qnaAnswer, prompts);
}
//Step 3) Prompt is displayed to the user
return (newState, new Activity[] { outputActivity }, null);

}

To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
Give a clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.
[bug]

The Microsoft Bot Framework team prefers that how to questions be submitted on Stack Overflow. The official Bot Framework Github repos are the preferred platform for submitting bug fixes and feature requests.
I'm closing this post as a duplicate of your stack overflow question here.


Migrate JUnit tests to Spek

Closes #1170

Just my 2 cents on this: I believe that the contribute documentation should include info on how someone could run Spek tests and debug Spek tests. For people that do not use a plugin or have their first interaction with Spek, it feels a bit difficult to get around on testing their contribution (speaking from personal experience 😂 )

Do we also need the Runner so that the tests work when executing ./gradlew test?

Majority of tests are already run with Spek test engine, and they execute fine with ./gradlew test. This PR just migrates the few tests that were using the JUnit engine to Spek.

@pavlospt Do you want to create a small issue to point out the improvements that could be made to the Documentation around Spek? Otherwise it might get lost in this PR thread.


Use PosixPath for path handling and create parent folders if they not exist

This should fix an error if the parents of the CACHE_FILENAME do not exist in some environments.
See: https://github.com/DurgNomis-drol/mytoyota/issues/270

You should now have maintainer access @GitOldGrumpy 😊 Can you pls review this MR?

Looks good to me.

Strange, merging ist still blocked :-/
Have you received and accepted the Maintainer invitation? At the moment it looks like you are still not registered as a repo Maintainer.

Looks good to me.

Strange, merging ist still blocked :-/ Can you please approve these changes @GitOldGrumpy?
Have you received and accepted the Maintainer invitation E-Mail? You are still shown here as Contributor instead of Caollaborator.

Yes I had seen the email, didn't realise I had to accept! I've done that now and it approved. Know cant merge because of open conversations.


AttributeError: 'PokemonGoBot' object has no attribute 'hunter_locked_target'

Expected Behavior
Bot running ;)
Actual Behavior
Bot not running ;)
Your FULL config.json (remove your username, password, gmapkey and any other private info)
Pretty much default config.json (derived from config.json.example), except for some threshold levels.
Output when issue occurred
2017-02-08 19:32:24,920 [       cli] [INFO] PokemonGO Bot v1.0
2017-02-08 19:32:24,934 [       cli] [INFO] commit: fda3ab8d
2017-02-08 19:32:24,938 [       cli] [INFO] Configuration initialized
...
[2017-02-08 19:32:33] [PokemonGoBot] [INFO] #175 Togepi
[2017-02-08 19:32:33] [PokemonGoBot] [INFO] #239 Elekid
[2017-02-08 19:32:33] [PokemonGoBot] [INFO]
[2017-02-08 19:32:33] [RandomAlivePause] [INFO] Next random alive pause at 19:39:42, for a duration of 0:02:45
[2017-02-08 19:32:33] [RandomPause] [INFO] Next random pause at 19:43:46, for a duration of 0:04:03
[2017-02-08 19:32:33] [RecycleItems] [INFO] Next forced item recycle at 19:34:45
[2017-02-08 19:32:34] [pokemongo_bot.health_record.bot_event] [INFO] Health check is enabled. For more information:
[2017-02-08 19:32:34] [pokemongo_bot.health_record.bot_event] [INFO] https://github.com/PokemonGoF/PokemonGo-Bot/tree/dev#analytics
[2017-02-08 19:32:34] [PokemonGoBot] [INFO] Starting bot...
[2017-02-08 19:32:35] [CollectLevelUpReward] [INFO] Received level up reward:
...
[2017-02-08 19:32:35] [       cli] [INFO]
[2017-02-08 19:32:35] [       cli] [INFO] Ran for 0:00:10
[2017-02-08 19:32:35] [       cli] [INFO] Total XP Earned: 0  Average: 0.00/h
[2017-02-08 19:32:35] [       cli] [INFO] Travelled 0.00km
[2017-02-08 19:32:35] [       cli] [INFO] Visited 0 stops
[2017-02-08 19:32:35] [       cli] [INFO] Encountered 0 pokemon, 0 caught, 0 released, 0 evolved, 0 never seen before ()
[2017-02-08 19:32:35] [       cli] [INFO] Threw 0 pokeballs
[2017-02-08 19:32:35] [       cli] [INFO] Earned 0 Stardust
[2017-02-08 19:32:35] [       cli] [INFO] Hatched eggs 0
[2017-02-08 19:32:35] [       cli] [INFO] Next egg hatches in 5.00 km
[2017-02-08 19:32:35] [       cli] [INFO]
[2017-02-08 19:32:35] [       cli] [INFO] Highest CP Pokemon:
[2017-02-08 19:32:35] [       cli] [INFO] Most Perfect Pokemon:
Traceback (most recent call last):
File "pokecli.py", line 865, in
main()
File "pokecli.py", line 206, in main
bot.tick()
File "/home/dj/PokemonGo-Bot/pokemongo_bot/init.py", line 778, in tick
if worker.work() == WorkerResult.RUNNING:
File "/home/dj/PokemonGo-Bot/pokemongo_bot/cell_workers/catch_pokemon.py", line 55, in work
if self.bot.hunter_locked_target != None:
AttributeError: 'PokemonGoBot' object has no attribute 'hunter_locked_target'
[2017-02-08 19:32:35] [sentry.errors] [ERROR] Sentry responded with an error: 'ascii' codec can't decode byte 0x9c in position 1: ordinal not in range(128) (url: https://app.getsentry.com/api/90254/store/)
Traceback (most recent call last):
File "/home/dj/PokemonGo-Bot/local/lib/python2.7/site-packages/raven/transport/threaded.py", line 174, in send_sync
super(ThreadedHTTPTransport, self).send(data, headers)
File "/home/dj/PokemonGo-Bot/local/lib/python2.7/site-packages/raven/transport/http.py", line 47, in send
ca_certs=self.ca_certs,
File "/home/dj/PokemonGo-Bot/local/lib/python2.7/site-packages/raven/utils/http.py", line 66, in urlopen
return opener.open(url, data, timeout)
File "/home/dj/PokemonGo-Bot/local/lib/python2.7/site-packages/future/backports/urllib/request.py", line 494, in open
response = self._open(req, data)
File "/home/dj/PokemonGo-Bot/local/lib/python2.7/site-packages/future/backports/urllib/request.py", line 512, in _open
'_open', req)
File "/home/dj/PokemonGo-Bot/local/lib/python2.7/site-packages/future/backports/urllib/request.py", line 466, in _call_chain
result = func(*args)
File "/home/dj/PokemonGo-Bot/local/lib/python2.7/site-packages/raven/utils/http.py", line 46, in https_open
return self.do_open(ValidHTTPSConnection, req)
File "/home/dj/PokemonGo-Bot/local/lib/python2.7/site-packages/future/backports/urllib/request.py", line 1284, in do_open
h.request(req.get_method(), req.selector, req.data, headers)
File "/usr/lib/python2.7/httplib.py", line 1057, in request
self._send_request(method, url, body, headers)
File "/usr/lib/python2.7/httplib.py", line 1097, in _send_request
self.endheaders(body)
File "/usr/lib/python2.7/httplib.py", line 1053, in endheaders
self._send_output(message_body)
File "/usr/lib/python2.7/httplib.py", line 895, in _send_output
msg += message_body
UnicodeDecodeError: 'ascii' codec can't decode byte 0x9c in position 1: ordinal not in range(128)
[2017-02-08 19:32:35] [sentry.errors.uncaught] [ERROR] [u"AttributeError: 'PokemonGoBot' object has no attribute 'hunter_locked_target'", u'  File "pokecli.py", line 865, in ', u'  File "pokecli.py", line 206, in main', u'  File "pokemongo_bot/init.py", line 778, in tick', u'  File "pokemongo_bot/cell_workers/catch_pokemon.py", line 55, in work']
Mi 8. Feb 19:32:35 CET 2017 Pokebot configs/config.json configs/auth.json Stopped.
Steps to Reproduce
~/PokemonGo-Bot$ ./run.sh configs/config.json configs/auth.json
Other Information
OS:
Ubuntu 16.04
Branch:
master
Git Commit:
fda3ab8d
Python Version:
Python 2.7.12
Occured right after updating to last commit.
Any other relevant files/configs (eg: path files)

Nevermind, modifying catch_pokemon.py according to #5904 solves it.


Decide and Provide the Basic Implementation of App Architecture

There are a number of architectural patterns and state management techniques that can be implemented with Flutter. It is for the community to discuss and come up with an actionable approach for this project.
I propose that we go with one of the following options for state management:

Scoped Models
Bloc Pattern

Any edits, suggestions, and all proposals are welcome and appreciated. For more information about the project, kindly refer Readme.md

I suggest provider now?


Video Trimming: Remove panel highlight when trimming from quick actions

Bug Description
originally reported by @samwhale
The Video Settings panel should not highlight when the quick action is clicked. The quick action could highlight the slider if we wanted any highlighting to happen?
Expected Behaviour

Steps to Reproduce

Screenshots

Additional Context

Plugin Version:
WordPress Version:
Operating System:
Browser:

Verified in QA


Fields RO/AO and PRO/PAO have same Description, different Type

In the VCF output, I notice that RO/AO and PRO/PAO have identical Description but different Type (Integer/Float).
##INFO=<ID=RO,Number=1,Type=Integer,Description="Reference allele observation count, with partial observations recorded fractionally">
##INFO=<ID=AO,Number=A,Type=Integer,Description="Alternate allele observations, with partial observations recorded fractionally">
##INFO=<ID=PRO,Number=1,Type=Float,Description="Reference allele observation count, with partial observations recorded fractionally">
##INFO=<ID=PAO,Number=A,Type=Float,Description="Alternate allele observations, with partial observations recorded fractionally">

Is the Description wrong for AO/RO in that partials aren't recorded fractionally?
Or is AO/RO just rounded versions of PAO/PRO ?
And is it because RO is a single number, whereas PRO is a list (per allele) ?

I would also appreciate clarity on this.

There is a mistake. The RO and Al fields should say
##INFO=<ID=RO,Number=1,Type=Integer,Description="Count of full observations of the reference haplotype.">
<ID=AO,Number=A,Type=Integer,Description="Count of full observations of this alternate haplotype.">

Make that:
##INFO=<ID=RO,Number=1,Type=Integer,Description="Count of full observations of the reference haplotype.">
##INFO=<ID=AO,Number=A,Type=Integer,Description="Count of full observations of this alternate haplotype.">


Browser connection is missing when trying to close tab using closeTab()

I execute one test case, but it will always failed when close current tab.
Then error info is as below:
[DEPRECATED] gauge.screenshotFn will be removed soon, use gauge.customScreenshotWriter instead.
Error: Connection to browser lost. This probably isn't a problem with Taiko, inspect logs for possible causes.
at validate (/Users/windyliu/Workstream/workstream-auto/node_modules/taiko/lib/taiko.js:3455:11)
at module.exports.screenshot (/Users/windyliu/Workstream/workstream-auto/node_modules/taiko/lib/taiko.js:2134:3)
at module.exports.(anonymous function) (/Users/windyliu/Workstream/workstream-auto/node_modules/taiko/lib/taiko.js:3585:36)
at Object.gauge.screenshotFn (/Users/windyliu/Workstream/workstream-auto/tests/step_helper.js:131:18)
at Promise (/Users/windyliu/.gauge/plugins/js/2.3.10/src/screenshot.js:44:34)
at new Promise (<anonymous>)
at /Users/windyliu/.gauge/plugins/js/2.3.10/src/screenshot.js:42:14
at Object.capture (/Users/windyliu/.gauge/plugins/js/2.3.10/src/screenshot.js:73:10)
at /Users/windyliu/.gauge/plugins/js/2.3.10/src/executor.js:70:22
at process._tickCallback (internal/process/next_tick.js:68:7)
        Error Message: rpc error: code = Unavailable desc = all SubConns are in TransientFailure, latest connection error: connection error: desc = "transport: Error while dialing dial tcp <IP_ADDRESS>:50495: connect: connection refused"
        Stacktrace:
        Failed Step: Answer quiz question with attempt as "fail"
        Specification: specs/01_positions.spec:164
        Error Message: rpc error: code = Unavailable desc = transport is closing
        Stacktrace:
    Error Message: rpc error: code = Unavailable desc = all SubConns are in TransientFailure, latest connection error: connection error: desc = "transport: Error while dialing dial tcp <IP_ADDRESS>:50495: connect: connection refused"
    Stacktrace:
  Error Message: rpc error: code = Unavailable desc = all SubConns are in TransientFailure, latest connection error: connection error: desc = "transport: Error while dialing dial tcp <IP_ADDRESS>:50495: connect: connection refused"
  Stacktrace:
  Error Message: rpc error: code = Unavailable desc = all SubConns are in TransientFailure, latest connection error: connection error: desc = "transport: Error while dialing dial tcp <IP_ADDRESS>:50495: connect: connection refused"
  Stacktrace:

My env is as below:
node: 10.16.3
Taiko: Version: 1.0.9 RELEASE
Gauge version: 1.0.7
Commit Hash: ed7b4fd6
Plugins
-------
html-report (4.0.9)
js (2.3.7)
screenshot (0.0.1)
xml-report (0.2.3)

code is as below:
step("Answer quiz question with attempt as <status>", async(status) => {
    let quiz_url = await getApplicantURL("take a short online quiz");
    await closeApplicantPanel();
    await waitFor(2000);
    await openTab(quiz_url);
    // answer quiz question
    await waitFor(async() => await button("Send").exists());
    let radio_buttons = await $(".question-mcq-control").elements();
    console.log(await radio_buttons[1].text());
    await click(radio_buttons[1]);
    await click(radio_buttons[4]);
    await click(button("Send"));
    await waitFor(async() => await text("Unfortunately you didn't answer enough questions correctly to move forward, but don't worry, you have 1 more attempts remaining.").exists());
    await click("Try again");
    let radio_buttons2 = await $(".question-mcq-control").elements();
    if (status == "fail") {
        await click(radio_buttons2[0]);
        await click(radio_buttons2[5]);
        await click(button("Send"));
        await waitFor(async() => await text("Thank you for your response!").exists());
    } else {
        await click(radio_buttons2[0]);
        await click(radio_buttons2[3]);
        await click(button("Send"));
        await waitFor(async() => button("Next step").exists());
    }
    await closeTab();
    await closeTab();
    await reload(currentURL());
});

I have 4 statement closeTab() in my test case, the 1st 2 of them works fine and the tab is closed as expected. But when trying to close the current Tab in the 3rd statement, the browser connection closed. It weird.
for the API closeTab() , is there some kind of limit count to use?
in my case, I have more than 3 tabs opened, and I closed 1 tab, and then the error occurs and the browser connection is missing, and then the test failed. And actually the browser is not closed, only the browser connection with taiko is missing.
I think there is something wrong with the API closeTab()

I was having a similar issue while executing one of my codes. But then handled it as follows
//Open and close a new Tab
await goto("https://the-internet.herokuapp.com/windows");
await click("Click Here");
await closeTab();

//Highlight different Shadow DOM
await goto("https://the-internet.herokuapp.com/shadowdom");
await highlight(`Let's`);
await highlight(listItem(`Let's`));

So from opening a tab, closing it and then navigating to another URL, this worked.
But I did get the error initially during execution. It seems Taiko doesn't know which window to focus. This code sequence gave the error you got
> goto("https://the-internet.herokuapp.com/windows");
[PASS] Navigated to URL https://the-internet.herokuapp.com/windows
>  value: {
  url: 'https://the-internet.herokuapp.com/windows',
  status: { code: 200, text: 'OK' }
}
[PASS] Navigated to URL https://the-internet.herokuapp.com/windows
> click("Click Here")
[PASS] Clicked element matching text "Click Here" 1 times
> switchTo("The Internet")
[PASS] Switched to tab matching The Internet
> closeTab("New Window")
[PASS] Closed tab(s) matching New Window
> goto("https://the-internet.herokuapp.com");
"[FAIL] Error: Connection to browser lost. This probably isn't a problem with Taiko, inspect logs for possible causes., run
`.trace` for more info."

Thanks for the sample to replicate @amitsarkar007 I could see the error too


Target Orientation Issue

Hello everyone,
I studied navigation on husky with 'pposgd_fuse' and only 'depth-sensor' values. The agent could able to reach target point. After, it reached target point total reward began decrease due to orientation couldn't obtain. How can I handle this problem without removing 'angle_cost' from rewards array?
Thanks in advance.

Hi again,
In my opinion, after the agent reached to target point, the different process must be applied.
Sincerely.


Winds-pleasure-attacked detail view not opening

The card for winds-pleasure-attacked opens just fine, but the detailed view isn't opening for me.
https://pr100.roshar-map.paleo.dev/#/en-US/events/winds-pleasure-attacked

Happening with this one too:
https://pr100.roshar-map.paleo.dev/#/en-US/events/shallan-kills-tyn

May have mentioned it previously but venli-is-claimed-by-odium is a case of one that won't even show a card. Has a marker on the timeline, but that's it.

Okay, so I went through all of the events looking for various things, got all of my changes in, and now I'm looking at the latest instance of pr100...
Now the chard for Adolin-is-born won't come up and I'm pretty sure that was there before. Seems like you mentioned the error with all of this is happening when it builds the site, which I guess fits with that symptom? Anyways, just wanted to give that data point in case it's helpful.

Fixed in 4170158e82ff5daed8dd852bd66e5ff2771b6569


Don't wait forever when dialing the server on Windows

Had this backwards, thought we wanted to wait forever for some reason (even looked for the convo in Slack, couldn't find it)
@maxtaco @patrickxb

What does the error look like when it fails to connect?

c:\work\src\github.com\keybase\client\go\keybase>keybase ctl stop

WARNING Running in devel mode
ERROR Timed out waiting for pipe '\.\pipe\kbservice\Users\Steve\AppData\Roaming\KeybaseDevel\keybased.sock' to come available

Ok great.

Merging without travis because this is windows only


Promote PendingAction for general use.

PendingAction is a class used in HttpSM to safely tracking pending actions. This is generally useful. In particular there is a race condition in PluginVC which should be fixed by using this class.
In addition to promoting this out of "HttpSM.h" for general use, the implementation has been adjusted for greater thread safety. Multiple threads can safely assign to the instance, with the guarantee that all actions except the one stored in the instance have been canceled.

In addition to promoting this out of "HttpSM.h" for general use, the implementation has been adjusted for greater thread safety. Multiple threads can safely assign to the instance, with the guarantee that all actions except the one stored in the instance have been canceled.

I don't like that this is done at the same time with the promotion. It's not even a separate commit. It's not reviewer friendly because you can't simply see the additional changes, and we can't revert it separately if it has problems.

#8846 adds #pragma once for this new header, which will be needed as this is used more. This is currently only marked for 10.x, but if we wind up bringing this back to 9.x we'll want to bring #8846 with it.

#8846 adds #pragma once for this new header, which will be needed as this is used more. This is currently only marked for 10.x, but if we wind up bringing this back to 9.x we'll want to bring #8846 with it.

Actually, I'd like to get this into 9.2.x since #8847 uses PendingAction. Marking for 9.2.x.

Cherry-picked to v9.2.x


'DPRO' is not recognized as an internal or external command, operable program or batch file.

Hello!
Thank you for the great work!
I have cloned this repo and did a npm i. But I am getting below error when i try to run any tests.
'DPRO' is not recognized as an internal or external command,
operable program or batch file.
What am i missing here?

me too facing this issue ?Can anyone Please help

@ankititp any luck with the issue?

No , still facing same issue :(

@archlichking Could you kindly help?

are you on windows? did you follow https://github.com/TestArmada/boilerplate-nightwatch#run ?


[FIX] Remove broken issueCommentTrigger

To re-enable builds. We can plan longer term action to fix the comment trigger. But at least we will have a working build...

java.lang.NoSuchMethodError: No such DSL method 'cleanWs' found among steps

What kind of upgrade they did honestly on Jenkins :o this is gonna be painful...

https://issues.apache.org/jira/projects/INFRA/issues/INFRA-24340

Regarding cleanWs I remember it's something I did after the 3.7 release... (because maybe the builds were not cleaned well... but we were never sure about it)
If you look at the backport, the build is running fine. Should we revert it for now? That could help us getting back on track quicker

I did put back deleteDir() instead... let's see the result

Creating placeholder flownodes because failed loading originals.
ERROR: Cannot resume build because FlowNode 84 for FlowHead 1 could not be loaded. This is expected to happen when using the PERFORMANCE_OPTIMIZED durability setting and Jenkins is not shut down cleanly. Consider investigating to understand if Jenkins was not shut down cleanly or switching to the MAX_SURVIVABILITY durability setting which should prevent this issue in most cases.

Ouch... :(
I think the build seems okish like that, should we merge this?

I do not see a :green_apple: build on this...


AshUUID causes compilation deadlock between mutually related resources

Given two resources that belong_to each other, such as the below, compilation of the project fails in a deadlock:
defmodule Pineapple do
  use Ash.Resource, data_layer: AshPostgres.DataLayer, extensions: [AshUUID]

  code_interface do
    define_for Area
  end

  postgres do
    table "pineapples"
    repo MyApp.Repo
  end

  attributes do
    uuid_attribute :id, prefix: "pnp"
    create_timestamp :inserted_at
  end

  relationships do
    belongs_to :ham, Ham
  end

  actions do
    defaults [:create, :read, :update]
  end
end

defmodule Ham do
  use Ash.Resource, data_layer: AshPostgres.DataLayer, extensions: [AshUUID]

  code_interface do
    define_for Area
  end

  postgres do
    table "hams"
    repo MyApp.Repo
  end

  attributes do
    uuid_attribute :id, prefix: "ham"
    create_timestamp :inserted_at
  end

  relationships do
    belongs_to :pineapple, Pineapple
  end

  actions do
    defaults [:create, :read, :update]
  end
end

The error is:
== Compilation error in file lib/my_app/my_domain/resources/pineapple.ex ==
** (CompileError) deadlocked waiting on module Ham
    :erlang.garbage_collect/1
    (ash_uuid 1.0.0) lib/ash_uuid/transformers/belongs_to_attribute.ex:79: anonymous fn/2 in AshUUID.Transformers.BelongsToAttribute.reject_yet_existent/1
    (elixir 1.17.1) lib/enum.ex:4429: Enum.reject_list/2
    (elixir 1.17.1) lib/enum.ex:4432: Enum.reject_list/2
    (ash_uuid 1.0.0) lib/ash_uuid/transformers/belongs_to_attribute.ex:15: AshUUID.Transformers.BelongsToAttribute.transform/1
    (spark 2.2.3) lib/spark/dsl/extension.ex:590: anonymous fn/4 in Spark.Dsl.Extension.run_transformers/4

== Compilation error in file lib/my_app/my_domain/resources/ham.ex ==
** (CompileError) deadlocked waiting on module Pineapple
    (ash_uuid 1.0.0) lib/ash_uuid/transformers/belongs_to_attribute.ex:79: anonymous fn/2 in AshUUID.Transformers.BelongsToAttribute.reject_yet_existent/1
    (elixir 1.17.1) lib/enum.ex:4429: Enum.reject_list/2
    (ash_uuid 1.0.0) lib/ash_uuid/transformers/belongs_to_attribute.ex:15: AshUUID.Transformers.BelongsToAttribute.transform/1
    (spark 2.2.3) lib/spark/dsl/extension.ex:590: anonymous fn/4 in Spark.Dsl.Extension.run_transformers/4
    (elixir 1.17.1) lib/enum.ex:4858: Enumerable.List.reduce/3
    (elixir 1.17.1) lib/enum.ex:2585: Enum.reduce_while/3

Compilation failed because of a deadlock between files.
The following files depended on the following modules:

    lib/my_app/my_domain/resources/pineapple.ex => Pineapple
  lib/my_app/my_domain/resources/ham.ex => Ham

Ensure there are no compile-time dependencies between those files and that the modules they reference exist and are correctly named

Though they refer to each other this did not cause a compilation error until AshUUID was introduced.

Interesting, thank you for reporting this issue!
I've been able to reproduce that but I actually not know well how we can solve this, maybe @zachdaniel has had similar problems before and can give an hint here?

So I'm not sure what belongs_to_attribute.ex is meant to do, but it is unfortunately not possible to use information from a related resource to modify the current resource, for this exact reason. I don't really have any advice fro you aside from "you'll have to figure something else out" 😦 . This typically means that whatever you were trying to derive from the destination must instead be explicitly configured by users.

Specifically, this call:
 relationship.destination.spark_dsl_config()

will always cause deadlocks between mutually related resources (and slow down compile times in general)

Thanks, @zachdaniel , I suspected that this wasn't quite right.
Since AshUUID is configurable for each of its uses, within belongs_to_attribute the specific settings used on the foreign_key target of the relationship are retrieved to automatically replicate them on the attribute generated by the belongs_to macro.
It has always worked well, but in this specific case, it breaks.
A workaround is to have the user disable the auto-generation of the xxx_id field via the belongs_to macro so that they can manually define the field with the same settings as the target.
This is one of the reasons why I was already considering making a much simpler and non-configurable version of this module, replaced by the idea we discussed today of directly incorporating uuidv7 into the core of Ash.
Tomorrow I will try to comment here with a temporary solution.

Hmm...but a belongs_to attribute should never be auto generated, right? Can we just omit the autogeneration constraints for the belongs_to attribute?

A simpler mode or package would be nice, it's not clear from the docs what the prefixing, encoding and other settings are for, nor whether AshUUID will cause uuid_primary_key to give v7 ids, or if I have to change uuid_primary_key :id to uuid_attribute :id for every resource. The compilation deadlock means I haven't got that far in my experimentation.
I'd guess most users just want the uuids to be v7 either globally or per reource/attribute, no other changes or options, and maybe a function for timestamp extraction.

Yeah, if we add it in core it would probably look like this uuid_v7_primary_key :id

Okay, I found a solution to the problem and I will try to address everything.
Regarding this issue reported by @spacebat, as I imagined, the only solution I found is to manually define the _id attributes when there are cross-relations between 2 resources. I have added tests for the suggested workaround and these are the interesting parts:
Ham resource:
defmodule AshUUID.Test.Ham do
  @moduledoc false

  use Ash.Resource, domain: AshUUID.Test, data_layer: AshPostgres.DataLayer, extensions: [AshUUID]

  postgres do
    table "hams"
    repo AshUUID.Test.Repo
  end

  attributes do
    uuid_attribute :id, prefix: "ham"

    uuid_attribute :burger_id,
      prefix: "bur",
      version: 4,
      primary_key?: false,
      allow_nil?: true,
      default: nil,
      writable?: true,
      public?: true

    create_timestamp :inserted_at
  end

  relationships do
    belongs_to :burger, AshUUID.Test.Burger, define_attribute?: false
  end

  actions do
    defaults [:read, create: :*, update: :*]
    default_accept :*
  end
end

Burger resource:
defmodule AshUUID.Test.Burger do
  @moduledoc false

  use Ash.Resource, domain: AshUUID.Test, data_layer: AshPostgres.DataLayer, extensions: [AshUUID]

  postgres do
    table "burgers"
    repo AshUUID.Test.Repo
  end

  attributes do
    uuid_attribute :id, prefix: "bur", version: 4

    uuid_attribute :ham_id,
      prefix: "ham",
      primary_key?: false,
      default: nil,
      writable?: true,
      public?: true

    create_timestamp :inserted_at
  end

  relationships do
    belongs_to :ham, AshUUID.Test.Ham, define_attribute?: false
  end

  actions do
    defaults [:read, create: :*, update: :*]
    default_accept :*
  end
end

Usage test:
test "testing hams and burgers" do
  ham =
    AshUUID.Test.Ham
    |> Ash.Changeset.for_create(:create, %{})
    |> Ash.create!()

  assert %AshUUID.Test.Ham{} = ham
  assert :prefixed = AshUUID.identify_format(ham.id)
  assert ["ham", b62_string_uuid] = String.split(ham.id, "_")
  assert {:ok, string_uuid} = AshUUID.Encoder.decode(b62_string_uuid)
  assert {:ok, %{version: 7}} = Uniq.UUID.info(string_uuid)

  burger =
    AshUUID.Test.Burger
    |> Ash.Changeset.for_create(:create, %{ham_id: ham.id})
    |> Ash.create!()

  assert %AshUUID.Test.Burger{} = burger
  assert :prefixed = AshUUID.identify_format(burger.id)
  assert ["bur", b62_string_uuid] = String.split(burger.id, "_")
  assert {:ok, string_uuid} = AshUUID.Encoder.decode(b62_string_uuid)
  assert {:ok, %{version: 4}} = Uniq.UUID.info(string_uuid)
  assert :prefixed = AshUUID.identify_format(burger.ham_id)
  assert ["ham", b62_string_uuid] = String.split(burger.ham_id, "_")
  assert {:ok, string_uuid} = AshUUID.Encoder.decode(b62_string_uuid)
  assert {:ok, %{version: 7}} = Uniq.UUID.info(string_uuid)

  ham =
    ham
    |> Ash.Changeset.for_update(:update, %{burger_id: burger.id})
    |> Ash.update!()

  assert :prefixed = AshUUID.identify_format(ham.burger_id)
  assert ["bur", b62_string_uuid] = String.split(ham.burger_id, "_")
  assert {:ok, string_uuid} = AshUUID.Encoder.decode(b62_string_uuid)
  assert {:ok, %{version: 4}} = Uniq.UUID.info(string_uuid)

  burger = Ash.load!(burger, :ham)
  ham_id = ham.id

  assert %AshUUID.Test.Ham{id: ^ham_id} = loaded_ham = burger.ham
  assert :prefixed = AshUUID.identify_format(loaded_ham.id)

  assert burger.id == ham.burger_id
end

In response to @zachdaniel, the need for the belongs_to_attribute transformer arises from the fact that AshUUID allows for a configurable id definition among various versions of UUID, in encoded or non-encoded mode, with a configurable prefix or without, and when a belongs_to relationship is defined, the generated _id attribute should match to the same configurations for everything to work well.
What might not be clear regarding this extension is that its goal was not only to allow the use of UUID version 7 within Ash but also to enable replication of what is well described in this article.
In any case, with the upcoming addition of support for UUID version 7 into the core of Ash, those who are only interested in this part will be able to do it completely without the AshUUID extension, and alternative solutions like the one proposed here will no longer be necessary.
But I ask @zachdaniel if he has any suggestions on how to achieve in Ash what is described in the article I mentioned regarding base62 encoded ids and/or prefixed when they end up in routes or APIs: do you think it can be done automatically with an extension or is it something that will be better managed manually project by project?
An affirmative answer to this question could help me and make AshUUID definitively and completely obsolete.

I think for those trying to replicate that kind of id behavior, it would make sense to have an extension like this one. At minimum, a custom type like the one provided here makes sense, because you're separating the application-representation from the data-layer representation, and custom types are the way to go there.

We can potentially enrich our UUID types in core to support these kinds of options, but we'd need to do that one at a time, and see how we go. Something like constraints: [encode?: :base64, prefix: "some-text"]

Perfect @zachdaniel , thank you!
That's what I was hoping for and I agree: let's start by adding uuid v7 to Ash and then later I will try to prepare other PRs to gradually add the rest like encoding or prefixing and we'll see how it goes 🤓


Genes expressed in peroxisome but found in GPR of beta-oxidation reactions in mitochondria

Current behavior:

Inspired by #746, the following peroxisome genes are found by this code through following steps:

find reactions that are in mitochondria compartment from Fatty acid oxidation subsystem
go through these reactions and check each gene's compartment based on the annotation in genes.tsv
report the ones that suppose to be expressed only in peroxisome compartment

Reaction
Peroxisome gene
Equation
GPR
GPR in gene name

MAR02028
ENSG00000115425
NADP+ + palmitoyl-CoA <=> (2E)-hexadecenoyl-CoA + H+ + NADPH
ENSG00000115425 or ENSG00000116353
PECR or MECR

MAR02148
ENSG00000115425
NADP+ + octanoyl-CoA <=> (2E)-octenoyl-CoA + H+ + NADPH
ENSG00000115425 or ENSG00000116353
PECR or MECR

MAR02149
ENSG00000115425
lauroyl-CoA + NADP+ <=> (2E)-dodecenoyl-CoA + H+ + NADPH
ENSG00000115425 or ENSG00000116353
PECR or MECR

MAR02183
ENSG00000115425
myristoyl-CoA + NADP+ <=> (2E)-tetradecenoyl-CoA + H+ + NADPH
ENSG00000115425 or ENSG00000116353
PECR or MECR

MAR02187
ENSG00000115425
decanoyl-CoA + NADP+ <=> (2E)-decenoyl-CoA + H+ + NADPH
ENSG00000115425 or ENSG00000116353
PECR or MECR

MAR02207
ENSG00000115425
(2E)-hexenoyl-CoA + H+ + NADPH <=> hexanoyl-CoA + NADP+
ENSG00000115425 or ENSG00000116353
PECR or MECR

MAR02381
ENSG00000060971
3-oxooleoyl-CoA + CoA --> acetyl-CoA + palmitoleoyl-CoA
(ENSG00000138029 and ENSG00000167315) or (ENSG00000060971 and ENSG00000138029)
(HADHB and ACAA2) or (ACAA1 and HADHB)

MAR03269
ENSG00000133835
NAD+ + 3(S)-3-Hydroxydodecen-(5Z)-Oyl Coenzyme A <=> H+ + NADH + 3-Oxolaur-Cis-5-Enoyl Coenzyme A
ENSG00000072506 or ENSG00000084754 or ENSG00000113790 or ENSG00000133835 or ENSG00000138029 or ENSG00000138796
HSD17B10 or HADHA or EHHADH or HSD17B4 or HADHB or HADH

MAR03269
ENSG00000113790
NAD+ + 3(S)-3-Hydroxydodecen-(5Z)-Oyl Coenzyme A <=> H+ + NADH + 3-Oxolaur-Cis-5-Enoyl Coenzyme A
ENSG00000072506 or ENSG00000084754 or ENSG00000113790 or ENSG00000133835 or ENSG00000138029 or ENSG00000138796
HSD17B10 or HADHA or EHHADH or HSD17B4 or HADHB or HADH

MAR03297
ENSG00000113790
H2O + Trans,Cis-Dodeca-2,5-Dienoyl Coenzyme A <=> 3(S)-3-Hydroxydodecen-(5Z)-Oyl Coenzyme A
ENSG00000084754 or ENSG00000113790 or ENSG00000127884 or ENSG00000138029
HADHA or EHHADH or ECHS1 or HADHB

MAR03374
ENSG00000113790
trans-2-cis,cis-5,8-tetradecatrienoyl-CoA <=> (3E,5Z,8Z)-Tetradecatrienoyl Coenzyme A
ENSG00000113790 or ENSG00000167969 or ENSG00000198721
EHHADH or ECI1 or ECI2

MAR03378
ENSG00000113790
Trans-2-Cis-8-Tetradecadienoyl Coenzyme A <=> Trans-3-Cis-8-Tetradecadienoyl Coenzyme A
ENSG00000113790 or ENSG00000167969 or ENSG00000198721
EHHADH or ECI1 or ECI2

MAR03379
ENSG00000113790
H2O + Trans-2-Cis-8-Tetradecadienoyl Coenzyme A <=> (3S)-3-Hydroxy-Cis-8-Tetradecenoyl Coenzyme A
ENSG00000084754 or ENSG00000113790 or ENSG00000127884 or ENSG00000138029
HADHA or EHHADH or ECHS1 or HADHB

MAR03380
ENSG00000113790
NAD+ + (3S)-3-Hydroxy-Cis-8-Tetradecenoyl Coenzyme A <=> H+ + NADH + 3-Oxo-Cis-8-Tetradecenoyl Coenzyme A
ENSG00000072506 or ENSG00000113790 or ENSG00000133835 or ENSG00000138796
HSD17B10 or EHHADH or HSD17B4 or HADH

MAR03380
ENSG00000133835
NAD+ + (3S)-3-Hydroxy-Cis-8-Tetradecenoyl Coenzyme A <=> H+ + NADH + 3-Oxo-Cis-8-Tetradecenoyl Coenzyme A
ENSG00000072506 or ENSG00000113790 or ENSG00000133835 or ENSG00000138796
HSD17B10 or EHHADH or HSD17B4 or HADH

Expected feature/value/output:

Remove Peroxisome genes in above table from listed mitochondrial beta-oxidation reactions.

after merging #760, GPRs of MAR03269 and MAR03380 had been resolved and thus excluded from this issue
the changes proposed here to MAR02381 overlap with #759, could thus be implemented together

oh yea that would work; go ahead

oh yea that would work; go ahead

now the proposed change in #759 has been implemented in #770

fixed by #770


BUG: Enchanted works locally but not through ngrok/mobile.

I am using ngrok static domain and had everything working. Then today the app doesn't work on iphone - I have tested on two iphones in different geographies. Then I ran Enchanted mac app on the computer running ngrok/ollama and it works fine. Same server url, I don't think I've changed anything but can't figure out why it won't work
Ollama running on 11434:

Ngrok running with addr = 11434:

What could cause this // how to debug? Thank you!

OK I dived deeper in and found out that its because I've used up all the bandwidth on my ngrok plan:

I only actually have sent probably sent less than 50 prompts from enchanted mobile through the ngrok tunnel as I mostly use it locally on my machine, which the above evidences skips ngrok completely, so the Enchanted app is constantly sending https requests. Is this intentional? Could this be throttled down to not consume all this bandwidth? Or is this number of requests normal?

+1 here, duplicated by #98 also. I'll leave commentary on #73.

Fixed in https://github.com/AugustDev/enchanted/issues/73, version 1.6.6


Refactor all uses of FlutterError.

Description
Make FlutterError objects more structured so they can be displayed better in debugging tools such as Dart DevTools.
Related Issues
https://github.com/flutter/flutter/issues/27327
Tests
Most tests pass as is as the text output does not change as the refactor does not break existing text based error displays.
New test cases are added in flutter_test to verify the structure is maintained and new tests are added to diagnostics_test.dart to verify that the additional text formatting logic works as expected.
Other test cases are updated for cases where the output changed in way that is minor and either completely neutral or positive.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] I signed the [CLA].
[x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require Flutter developers to manually update their apps to accommodate your change?

[x] Yes, this is a breaking change (Please read [Handling breaking changes]). https://groups.google.com/forum/#!topic/flutter-announce/hp1RNIgej38
[ ] No, this is not a breaking change.

This looks fantastic.
I only really have one overall comment. Currently FlutterError has a single constructor, which takes a list of diagnostics. I propose that we have two constructors, as follows (also shown here are the changes to diagnostics and message documentation):
  /// Create an error message from a string.
  ///
  /// The message may have newlines in it. The first line should be a terse
  /// description of the error, e.g. "Incorrect GlobalKey usage" or "setState()
  /// or markNeedsBuild() called during build". Subsequent lines should contain
  /// substantial additional information, ideally sufficient to develop a
  /// correct solution to the problem.
  ///
  /// In some cases, when a FlutterError is reported to the user, only the first
  /// line is included. For example, Flutter will typically only fully report
  /// the first exception at runtime, displaying only the first line of
  /// subsequent errors.
  ///
  /// All sentences in the error should be correctly punctuated (i.e.,
  /// do end the error message with a period).
  ///
  /// This constructor defers to the [new FlutterError.fromParts] constructor.
  /// The first line is wrapped in an implied [ErrorSummary], and subsequent
  /// lines are wrapped in implied [ErrorDescription]s. Consider using the
  /// [new FlutterError.fromParts] constructor to provide more detail, e.g. using
  /// [ErrorHint]s or other [DiagnosticsNode]s.
  factory FlutterError(String message) {
    final List<String> lines = message.split('\n');
    final List<DiagnosticsNode> parts = <DiagnosticsNode>[];
    parts.add(ErrorSummary(lines.first));
    if (lines.length > 1)
      parts.addAll(lines.skip(1).map<DiagnosticsNode>((String line) => ErrorDescription(line));
    return FlutterError.fromParts(parts);
  }

  /// Create an error message from a list of [DiagnosticsNode]s.
  ///
  /// By convention, there should be exactly one [FlutterSummary] in the list, and
  /// it should be the first entry. (This is not enforced.)
  ///
  /// Other entries are typically [ErrorDescription]s (for material that is always applicable for
  /// this error) and [ErrorHint]s (for material that may be sometimes useful, but may not always
  /// apply). Other [DiagnosticsNode] subclasses, such as [DiagnosticsStackTrace], may
  /// also be used.
  FlutterError.fromParts(this.diagnostics) : super(null);

  /// The information associated with this error, in structured form.
  ///
  /// The first node is typically an [ErrorSummary] giving a short description of the problem,
  /// suitable for an index of errors, a log, etc.
  ///
  /// Subsequent nodes should give information specific to this error. Typically these will be
  /// [ErrorDescription]s or [ErrorHint]s, but they could be other objects also. For instance,
  /// an error relating to a timer could include a stack trace of when the timer was scheduled
  /// using the [DiagnosticsStackTrace] class.
  final List<DiagnosticsNode> diagnostics;

  /// The message associated with this error.
  ///
  /// This is generated by serializing the [diagnostics].
  String get message => toString();

I think if we do this we can actually avoid many of the changes that are made in this patch.

FWIW, I did a little looking at how these proposed changes would impact folks externally.  I'm still working on the impact for extenders but feel pretty good about my greps for the FlutterError and FlutterErrorDetails cases (the latter took some added visual inspection to check only for calls that used the optional context arg).
In a nutshell, I found that of the 2138 Flutter packages inspected (of 6169 fetched from pub)

39 packages throw FlutterErrors for a total of 69 calls
25 packages create FlutterErrorDetails with the context param, a total of 33 times

I can take a look at implementers/extenders of FlutterError that override toString() but that'd take longer since I'll need to write proper AST visitors for that...  Let me know how valuable that'd be.

I'm happy with this approach; will defer to @goderbauer for detailed review.

I've pushed a version reflecting the final state from the code review discussion that happened here.
Now that the core FlutterError change is less breaking I'm now splitting the CL into 3 sub CLs. A core CL with the breaking changes and internal fixes, a secondary CL with nice to have changes (mainly adding hints), and a third CL with optional changes that I'll audit further to figure out which ones still make sense with the final design.

Published https://github.com/flutter/flutter/pull/30983 which is just the core of this CL with 39 files instead of 90+.
Consider this CL closed. I will close it officially in a bit once I handle all @pq's comments in the followup cl.

FWIW, I did a little looking at how these proposed changes would impact folks externally. I'm still working on the impact for extenders but feel pretty good about my greps for the FlutterError and FlutterErrorDetails cases (the latter took some added visual inspection to check only for calls that used the optional context arg).
In a nutshell, I found that of the 2138 Flutter packages inspected (of 6169 fetched from pub)

39 packages throw FlutterErrors for a total of 69 calls
25 packages create FlutterErrorDetails with the context param, a total of 33 times

I can take a look at implementers/extenders of FlutterError that override toString() but that'd take longer since I'll need to write proper AST visitors for that... Let me know how valuable that'd be.

Good news is those calls to throw FlutterEerror in 39 packages are no longer broken. The 25 packages passing the context parameter will need to be updated. Optionally we could make the context parameter type Object and accept either a String or DiagnosticsNode as the value. It would be ideal if Dart supported union types for this case. Even better if we could describe a union type where one option in the union type was deprecated.

Addressed all comments. Closing this pull request now as I've split the pull request into 3 smaller chunks. Looking forward to the continued discussion on the smaller CLs.


RM-14-fix-my-sql-integration-test

Okay, have to log off but given this:
https://app.circleci.com/pipelines/github/bluelabsio/records-mover/2310/workflows/94e631e2-8b74-4390-93f2-69523a4fb514
Fairly sure these tests will pass.
If they do, ready for your review @Brunope !


API Review cleanup #1

Remove constructor defaults
Make service properties internal
Add Logging/HttpContextAccessor services which are required by
identity

Fixes https://github.com/aspnet/Identity/issues/358
Fixes https://github.com/aspnet/Identity/issues/357
cc @divega @suhasj

:shipit:


Meshery Community Page Implementation

Updates for the Meshery Community Page.

We are currently trying to come up with contents for the Meshery community page as it is currently empty.
Proposed Content for Meshery Community Page

Implement this Calendar Section for the Meshery Community Page!
Be sure to check out the mockup of the full Figma Design for more context.
Feel free to take up another issue (section) of this page once your initial attempt has been successfully reviewed. Thank you!
Section Screenshot.

Contributor Guides and Resources

🎨 Wireframes and designs for Meshery Site in Figma
🖥 Contributing to Meshery Website
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

Duplicate Issue