erpc *-bus compatible idl function generation

Hi,
i have implemented the use of erpc via can bus in my stm32 project.
As far as I can tell, the erpc IDL (erpcgen) tooling will generate fixed ID's for differentiating / managing the different rpc function calls. (id 0, 1, 2, 3 ... N).
When utilizing a (1:*)-Bus, the slaves on that bus can all receive the rpc data. Normally, bus arbitration is done via various addressing schemes (can, i2c, ethernet etc).
In some use case, it is not possible/desired to rely on bus arbitration and instead broadcast packages (erpc calls)  (canId 0x00, ethernet x.x.x.255). The current implementation of erpc does not tolerate that, because the function calls on the server/client are only separated by whatever ID was generated via the IDL. If a client would send a rpc call with id 0, that function could travel to 3 different servers and they would all execute whatever 0 means for them (probably different functions altogether).
Is it feasible to change the ID utilized by the erpcgen IDL tool to be a semi-random hash (32+bits) to differentiate between message function types instead? The erpcgen tool could also work recursively through a project (containing .erpc files for all servers and clients in the system) to make sure, that each and every individual erpc function has a different id (hash) so that no hickups occur.
This would allow (or is the first step to a) multi server/client approach in transport agnostic systems.

H i am not sure if semi random hash would solve somthing. Instead of that maybe we can have just offset... Or program id. So you would have program id, interface id and function id.... Program id sounds good to me. So far you can set manualy id for each function using annotations in current solution. But if you agree with program ID we can add this one. Or offset... Sounds easier to implement and should work.

An offset is fine, but a manual one is kind of weird. The hash suggestion mainly comes from other bigger idl genertion framworks like the one used inside of ROS (for example). The non trivial part is letting the IDL automagically differentiate ALL needed erpc message definitions in a way that

lets erpc functions be unique at all times
lets erpc functions be the same after regenerating the message (if nothing changed)

Only a hash (which takes into account message contents, return values, parameters and it's name) can properly encode the message types future proof

I understand now. Well sounds good to me. So simple solution could be take function definition as string and encode it (hash,...).

would probably work. I mean ... if the message already exists, it's name / arguments / retun values are essentially  the same. Which means that duplicate message types should not be a problem (because the two different functions already behave identical).

I would apply this principle also for interface ID. So combination interface and function should make it more rare. The is still same risk as you described but much smaller than now.

@Hadatko  i have a proof of concept working which uses an md5 hash from the function prototype string

<FunctionType> <FunctionName>(<FunctionParams>)

I have edited the code structure in such a way, that the uint32_t methodId is now replaced with a 129 byte (char[33]) data structure (md5 + \0 string) and it seems to work fine.
It should theoretically allow multiple clients and servers on the same bus, each of them sending and receiving rpc information which should be identified properly, because servers can now identify functions based on a truly unique Identifier.
Although i Have to say that the underlying write functiosn for header and such are messy (even without me interfering) ... :p
I will clean this up and update the IDL as well as the erpc internals and create a PR at some point (after im relatively satisfied).
Does someone need this?


Cosmos Kit Upgrade type fix

What is the purpose of the change
This fixes a bug while using the new iframe Cosmos Kit feature. The iframe feature works by mimicking a wallet, but we shouldn't add the iframe wallet to the WalletRegistry (which is where the features attribute is set) because it shouldn't be selectable by users. Thus, we just need to use an optional in case the iframe wallet is selected automatically.
Brief Changelog

Type error fix

Testing and Verifying
This change has been tested locally by rebuilding the website.

Thank you for the initiative! I really appreciate the help 🙏
After some QA, I could identify these issues:

This error occurs while doing any transaction. I think it's related to some breaking change within the account store

When trying to open the profile. The app crashes. I believe it's because of the wallet images. Next does not recognize the Custom DAO wallet. I suspect we'll have to change the Next Image component for img in profile and wallet select.

Thank you for the initiative! I really appreciate the help 🙏
After some QA, I could identify these issues:

This error occurs while doing any transaction. I think it's related to some breaking change from Cosmos-Kit within the account store signAndBroadcast

When trying to open the profile. The app crashes. I believe it's because of the wallet images. Next does not recognize the Custom DAO wallet. I suspect we'll have to change the Next Image component for img in profile and wallet select.

Perfect. I'll get right on those.

@JoseRFelix, both issues have been fixed 👍


Cleanup

I performed a cleanup of the code base

typos
async issues
renames that went wrong
unused methods
properties that can be auto properties
missing curly braces for clarity
switch statements that are more readable as switch expressions
inconsistent line breaks (crlf vs lf)

@stefanloerwald Gods work, thanks. See comment I made on the commit.
By the way, I definitely think we should remove all commented out code from the repo. My eyes are bleeding when I see it, we have version control for a reason ;- ) But that could maybe be saved for a rainy day.

@stefanloerwald Nice job, thanks for this!


[Feature Request] Possibility of being able to differentiate personal and professional applications / bookmarks

Normally, as users we have different applications / bookmarks in the professional and private / personal spheres.
I would like to be able to differentiate them clearly, if possible grouped in some way.
I can think of several options:

Have the possibility to create two super categories (Professional, personal) and post the categories of those super categories. And then differentiate them in some way
You have two tabs on the page itself, in such a way that one tab contains personal applications / bookmarks and the other professional ones. So they would be clearly differentiated.

I think it can be a very interesting feature and make the application more functional.
In any case, thank you for your dedicated effort. For me it is the best application there is to save links to applications and bookmarks.

Why don't you spin a second instance and have two separate Flame desktops?
Don't get why people love to overcomplicate things.
On Thu, Dec 2, 2021 at 9:00 AM tomandocubatas @.***>
wrote:

Normally, as users we have different applications / bookmarks in the
professional and private / personal spheres.
I would like to be able to differentiate them clearly, if possible grouped
in some way.
I can think of several options:

Have the possibility to create two super categories (Professional,
personal) and post the categories of those super categories. And then
differentiate them in some way
You have two tabs on the page itself, in such a way that one tab
contains personal applications / bookmarks and the other professional ones.
So they would be clearly differentiated.

I think it can be a very interesting feature and make the application more
functional.
In any case, thank you for your dedicated effort. For me it is the best
application there is to save links to applications and bookmarks.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pawelmalak/flame/issues/240, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIBTXK5O3NJCU4UNBPKQEZLUO6C2LANCNFSM5JHK2D7Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Thats exactly what i did. work and home flames

Why don't you spin a second instance and have two separate Flame desktops? Don't get why people love to overcomplicate things.
…
On Thu, Dec 2, 2021 at 9:00 AM tomandocubatas @.***> wrote: Normally, as users we have different applications / bookmarks in the professional and private / personal spheres. I would like to be able to differentiate them clearly, if possible grouped in some way. I can think of several options: - Have the possibility to create two super categories (Professional, personal) and post the categories of those super categories. And then differentiate them in some way - You have two tabs on the page itself, in such a way that one tab contains personal applications / bookmarks and the other professional ones. So they would be clearly differentiated. I think it can be a very interesting feature and make the application more functional. In any case, thank you for your dedicated effort. For me it is the best application there is to save links to applications and bookmarks. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#240>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AIBTXK5O3NJCU4UNBPKQEZLUO6C2LANCNFSM5JHK2D7Q . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Hello,
Indeed, one option is to have two instances, but I don't think the truth is the most optimal or simple. Have two points where to modify links, etc, two instances to maintain, etc, etc.
Of course it is my opinion and you have yours and I respect it as it cannot be otherwise.
I believe that what I propose is a good alternative, I believe that it covers a need that many of us have and in an optimal and simple way. But obviously I am not the one to decide if it is something that makes sense within the philosophy that the owner of this application wants to give it.

What about a button somewhere to switch to a whole new page like alternative links but still using the 1/main url?
Like different tabs for the page. I was using Homer before that has this, and I just made a page for work and a page for home all on the same main URL


T cannot be implicitly cast to T & { optional: U? }

--!strict
type Base = {
	x: number,
	y: number,
}

type Extra = Base & {
	z: number?,
}

local function takeExtra(_: Extra) end

local function takeBase(base: Base)
	takeExtra(base)
end

Fails with:
Type Error: (14,12) Type 'Base' could not be converted into 'Base & {| z: number? |}'
caused by:
  Not all intersection parts are compatible.
Table type 'Base' not compatible with type '{| z: number? |}' because the former is missing field 'z'

Intentional but I wish this had an error message explaining it


add doc for operator.

ref: https://github.com/stolostron/backlog/issues/24609
Signed-off-by: morvencao<EMAIL_ADDRESS>

/lgtm


Updating The Nuage VSP Plugin Documentation for ACS 4.10 release

These changes in the documentation are in accordance with our merged feature support (including bug fix) PRs. Here are some of our merged feature support PRs:
https://github.com/apache/cloudstack/pull/1578
https://github.com/apache/cloudstack/pull/1579
https://github.com/apache/cloudstack/pull/1580
https://github.com/apache/cloudstack/pull/1452
Moreover, we have restructured and extended the documentation with some screen-shots.

@rhtyd @karuturi Please do review this cloudstack-docs PR, which includes changes to the Nuage VSP Plugin documentation for ACS 4.10 release.

LGTM thanks. /cc @karuturi we can merge this.

@rhtyd @karuturi @krissterckx Additionally, added a section on how to run The Nuage VSP Plugin specific Marvin tests.
You can merge this PR as it is complete now.

@rhtyd @karuturi Can we please merge this ?

@rhtyd @karuturi Can we please merge this now that 4.10 is out  ?

LGTM.  Maybe we wait until the official announcement of 4.10 goes out, but I think that is just semantics.

LGTM, merged since <IP_ADDRESS> is voted and 4.10 branch is cut.


(maint) Upgrade to latest copy of ssl.sh

This version adds 5 curl retries with a delay of 2 seconds to work
around transient issues
Even though the waiter scripts seem to be properly waiting for
containers to initially become available at their designated
names, there are still transient failures that occur when performing
various curl commands while handling certs

Test FAILed


Typo in overview

There's a typo at the top of this page:
"This quickstart shows how to send messages to a Service Bus topic and receive messages a subscription to that topic by using..."

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

ID: 443c9051-c55e-1202-7b75-ce86f6977017
Version Independent ID: 677b6375-196f-fed8-7300-b9823c1031d5
Content: Get started with Azure Service Bus topics and subscriptions - Azure Service Bus
Content Source: articles/service-bus-messaging/service-bus-dotnet-how-to-use-topics-subscriptions.md
Service: service-bus-messaging
GitHub Login: @spelluru
Microsoft Alias: spelluru

Thanks for the feedback! We will review and get back to you shortly.

I've sent a PR request to the docs team. It should go live once the review process is completed.


MD WIDGETS: The image widget should allow users to adjust the display size

Description
The current image widget can only display images at their original size. A parameter needs to be added to allow users to customise the display size of the image on the page.


urllib3.exceptions.MaxRetryError: HTTPConnectionPool(host='<IP_ADDRESS>', port=7860): Max retries exceeded with url: /run/textgen (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7fdc8637a9b0>: Failed to establish a new connection: [Errno 111] Connection refused'))

Connection refused when trying to use Agent-LLM with oobabooga
env file important parts:
=========================
AI PROVIDER CONFIG
=========================
AI_PROVIDER=oobabooga
AI_MODEL=vicuna
AI_TEMPERATURE=0.2
MAX_TOKENS=2000
AI PROVIDER: CUSTOM (e.g., Oobabooga, Fastchat, etc.)
=========================
AI_PROVIDER_URI=http://<IP_ADDRESS>:7860
Command to start Oobabooga:
call python server.py --model MiniGPT-4-LLaMA-13b-4bit-128g --listen --wbits 4 --groupsize 128 --auto-devices --no-stream

Are you running Agent-LLM in Docker or local?


AutovalidateMode.always on Form not working as expected

Steps to reproduce
Display the following widget:
Form(
  autovalidateMode: AutovalidateMode.always,
  child: TextFormField(
    validator: (value) => value?.isEmpty == true ? 'Empty' : null,
  ),
),

Expected results
We would expect the error message "Empty", to be displayed under the TextFormField, because the Form has AutovalidateMode.always and the value of the field is empty.
Actual results
The error message is not displayed when the Form is rendered. Only when a change happens on the TextFormField, for example adding and removing text, the error is displayed.
Workaround
When the TextFormField has AutovalidateMode.always, the validation does happen immediately and the error message is displayed.
Looking into the source code, Form runs the _validate function in the beginning of its build method, before its _fields have registered. The _validate method goes over an empty Set of fields.
The FormField on the other hand sets its own _errorText before calling the builder is called, causing the actual builder of TextFormField to have the correct errorText available at the first time it builds.
Code sample
Code sample
import 'package:flutter/material.dart';

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

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

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      home: Scaffold(
        body: Center(
          child: Form(
            autovalidateMode: AutovalidateMode.always,
            child: TextFormField(
              validator: (value) => value?.isEmpty == true ? 'Empty' : null,
            ),
          ),
        ),
      ),
    );
  }
}

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.16.9, on macOS 14.2.1 23C71 darwin-arm64, locale en-BE)
    • Flutter version 3.16.9 on channel stable at /Users/fre/fvm/versions/stable
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 41456452f2 (7 days ago), 2024-01-25 10:06:23 -0800
    • Engine revision f40e976bed
    • Dart version 3.2.6
    • DevTools version 2.28.5

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

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

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

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

[✓] IntelliJ IDEA Community Edition (version 2023.3.2)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 77.0.1
    • Dart plugin version 233.13135.65

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

[✓] Connected device (3 available)
    • iPhone SE (mobile) • 00008110-000E7901349B801E • ios            • iOS 17.2.1 21C66
    • macOS (desktop)    • macos                     • darwin-arm64   • macOS 14.2.1 23C71 darwin-arm64
    • Chrome (web)       • chrome                    • web-javascript • Google Chrome 121.0.6167.139

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

• No issues found!

I found there's also an invalid test that forces another rebuild to get a positive result:
https://github.com/flutter/flutter/blob/e02e2079bea7d04151e8d4240d70002e4b54186d/packages/flutter/test/widgets/form_test.dart#L889

It should be
TextFormField(
validator: (value) => (value == null || value?.isEmpty == true) ? 'Empty' : null,
)
since the initial value will be null.

Thanks for the report @dumazy.
I can reproduce this on the latest master and stable channels.
From docs,

Used to auto-validate [Form] and [FormField] even without user interaction.

On opening of app, the error isn't displayed since TextField is empty.
stable, master flutter doctor -v

[!] Flutter (Channel stable, 3.16.9, on macOS 14.2.1 23C71 darwin-arm64, locale
    en-NG)
    • Flutter version 3.16.9 on channel stable at
      /Users/damilolaalimi/sdks/flutter
    ! Warning: `dart` on your path resolves to
      /opt/homebrew/Cellar/dart/3.1.5/libexec/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/damilolaalimi/sdks/flutter.
      Consider adding /Users/damilolaalimi/sdks/flutter/bin to the front of your
      path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 41456452f2 (7 days ago), 2024-01-25 10:06:23 -0800
    • Engine revision f40e976bed
    • Dart version 3.2.6
    • DevTools version 2.28.5
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/damilolaalimi/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/damilolaalimi/Library/Android/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

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

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

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

[!] Android Studio (version unknown)
    • Android Studio at /Users/damilolaalimi/Downloads/Android Studio
      Preview.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to determine Android Studio version.
    • Java version OpenJDK Runtime Environment (build
      17.0.7+0-17.0.7b1000.6-10550314)

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

[✓] VS Code (version 1.83.1)
    • VS Code at /Users/damilolaalimi/Downloads/Visual Studio Code.app/Contents
    • Flutter extension version 3.78.0

[✓] Connected device (4 available)
    • Damilola’s iPhone (mobile) • 00008110-001964480AE1801E            • ios
      • iOS 17.1.1 21B91
    • iPhone 15 Pro (mobile)     • 79E44FCE-BEA4-4C4B-9C1C-EFBD9964E172 • ios
      • com.apple.CoreSimulator.SimRuntime.iOS-17-2 (simulator)
    • macOS (desktop)            • macos                                •
      darwin-arm64   • macOS 14.2.1 23C71 darwin-arm64
    • Chrome (web)               • chrome                               •
      web-javascript • Google Chrome 121.0.6167.85

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

! Doctor found issues in 2 categories.

[!] Flutter (Channel master, 3.19.0-15.0.pre.38, on macOS 14.2.1 23C71 darwin-arm64, locale en-NG)
    • Flutter version 3.19.0-15.0.pre.38 on channel master at /Users/damilolaalimi/fvm/versions/master
    ! Warning: `dart` on your path resolves to /opt/homebrew/Cellar/dart/3.1.5/libexec/bin/dart, which is not inside your current Flutter SDK checkout at /Users/damilolaalimi/fvm/versions/master. Consider adding /Users/damilolaalimi/fvm/versions/master/bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision e02e2079be (4 hours ago), 2024-02-01 02:00:32 -0500
    • Engine revision 68943afd62
    • Dart version 3.4.0 (build 3.4.0-97.0.dev)
    • DevTools version 2.31.0
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/damilolaalimi/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/damilolaalimi/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

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

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

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

[!] Android Studio (version unknown)
    • Android Studio at /Users/damilolaalimi/Downloads/Android Studio Preview.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to determine Android Studio version.
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-17.0.7b1000.6-10550314)

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

[✓] VS Code (version 1.83.1)
    • VS Code at /Users/damilolaalimi/Downloads/Visual Studio Code.app/Contents
    • Flutter extension version 3.78.0

[✓] Connected device (5 available)
    • Damilola’s iPhone (mobile)      • 00008110-001964480AE1801E            • ios            • iOS 17.1.1 21B91
    • iPhone 15 Pro (mobile)          • 79E44FCE-BEA4-4C4B-9C1C-EFBD9964E172 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-17-2 (simulator)
    • macOS (desktop)                 • macos                                • darwin-arm64   • macOS 14.2.1 23C71 darwin-arm64
    • Mac Designed for iPad (desktop) • mac-designed-for-ipad                • darwin         • macOS 14.2.1 23C71 darwin-arm64
    • Chrome (web)                    • chrome                               • web-javascript • Google Chrome 121.0.6167.85

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

! Doctor found issues in 2 categories.

I'm using Flutter 3.19.6. I know this issue has 6 months old.
If is needed, I will open a new issue, but, the validator for each field works, marking them in red and displaying the error validation as in the picture:

But as you can see in the video, using autovalidatemode in always, the keyboard shows/hides and is impossible to type anything:

Using autovalidatemode in onUserInteraction doesn't have such performance, but changing the focus of each field, the validation won't work:


electron (desktop) error: GL_INVALID_FRAMEBUFFER_OPERATION ?

Hello,
after updating from IndigoEngine 0.7.1 to 0.9.2 and along with it from Scala 2.13.1 to Scala 3.1.0, ScalaJS 1.7.1.
I get the following errors:
<EMAIL_ADDRESS>start target/indigoRun
> electron .

[21781:1105/121117.365903:ERROR:gles2_cmd_decoder.cc(4968)] [.RendererMainThread-0x7f95c5810000]GL ERROR :GL_INVALID_FRAMEBUFFER_OPERATION : glDrawArrays: framebuffer incomplete
oder.cc(4968)] [.RendererMainThread-0x7f95c5810000]GL ERROR :GL_INVALID_FRAMEBUFFER_OPERATION : glDrawArrays: framebuffer incomplete
[21781:1105/121117.625685:ERROR:gles2_cmd_decoder.cc(4968)] [.RendererMainThread-0x7f95c5810000]GL ERROR :GL_INVALID_FRAMEBUFFER_OPERATION : glDrawArrays: framebuffer incomplete
(... more of the same with every frame)

In the electron window everything seems rendered fine though. Not sure if this is a concern, but I haven't seen this before.

Interesting - thanks for reporting. These are the kinds of issues that are the toughest to debug unfortunately.
When you say "desktop" I assume you mean with Electron (i.e. Chrome)?
Couple of things:

I'm confident this isn't the issue, but 0.9.2 is built against Scala 3.0.2, next release will be the 3.1.0 version - probably isn't that but you could down grade and see what happens.
The next version has had a lot of work done on the rendering pipeline, so this error might just go away.
To get a clue what's causing the issue, the easiest place to start is just to start removing things from your view until the error goes away - at which point - the error is either will the thing you removed or the thing before it. You can often use that to build a minimal example that exhibits the behaviour.

I'm hoping to push the next version out this weekend, but if you're feeling adventurous you can always check out the project and (having read the README for software requirements) do a local publish bash build.sh, then upgrade your project to 0.9.3-SNAPSHOT and see if you still get the error? 😀

Hello,
I could isolate the cause to this line (the only item that appears across al the scenes):
.withSubSystems(Set(FPSCounter(Point(5,5), 60)))
So even though the FPS counter appears, it seems to cause mentioned rendering issues.
I also have it in 0.9.3-SNAPSHOT.

Excellent work, but we're not quite done yet if you'll indulge me. :-)
You've added the FPSCounter but you haven't specified a layer it should render to, therefore it's just rendered last (if memory serves).
Couple of questions:

Do you use TextBox's and Shapes anywhere else? Or alternatively, do you have any scenes where it breaks and you definitely don't have any more of those?
Can you give me an example of something that is being draw last i.e. just under the FPS counter?
What browser + platform are you using?

The incomplete frame buffer error happens when there is something drawn underneath and also when nothing is being drawn underneath.
If something is being drawn (underneath or not):
it is always:
Graphic(Rectangle(...), ..., Material.Bitmap(...))
There is also a scene that displays text using a font, the one like in the Snake example.
This scene also has the error. The text is not underneath the FPS Counter.

Hmmm... I can't seem to replicate that. Wonder if it's a system difference.
I'm on a Mac running Electron v14.0.1, are you a Windows person by any chance? I only ask because we had a bug in the past where Windows was being stricter about graphics card limits and behaviours than OSX and Linux were..

Hello,
Running Electron v10.1.1 on macOs 10.4.6
Some additional settings which I activated.
  val advancedGameConfig: AdvancedGameConfig =
    AdvancedGameConfig(WebGL2WithFallback, true, 256, false, false, true)

  val config: GameConfig =
    GameConfig.default
      .withViewport(width, height)
      .withFrameRate(6)
      .withMagnification(magnification)
      .withAdvancedSettings(advancedGameConfig)

Ah ha! It's your electron version (which will be using an older Chrome version).
To replicate the problem:

I ran fastOptJS then indigoRun as normal (or runGame if you've got the aliases set up), which ran the game on my machine with no errors.
Then I navigated to the target/indigoRun directory containing package.json.
yarn add electron (can also use npm) to add a local version of electron
I edited the package.json file to have the same electron version as you("electron": "^10.1.1").
yarn install which installs the specified version of electron.
yarn start which runs the game - everything appears fine, but the terminal is flooded with:

[45439:1115/210530.270483:ERROR:gles2_cmd_decoder.cc(4968)] [.RendererMainThread-0x7f9cb200da00]GL ERROR :GL_INVALID_FRAMEBUFFER_OPERATION : glClear: framebuffer incomplete

So the fix is to upgrade electron, npm update -g electron I imagine. I did also test it with the latest electron, and that seemed to work just fine.

Hello,
I confirm indeed this resolves the issue.


feat: add secret provider aws to license model

Issue
https://gravitee.atlassian.net/browse/AM-3308
Description
Add secret provider aws to license model
Additional context

Gravitee.io Automatic Deployment
🚀 A prerelease version of this package has been published on Gravitee's private artifactory, you can:

use it directly by updating your project with version: 5.18.0-feat-secretprovider-aws-license-SNAPSHOT
download it from Artifactory here

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

5.18.0
GitHub release

Your semantic-release bot :package::rocket:


seperate contact page created

new contact page created and linked to navbar
FIxes: #63

Looks Nice ,Good Work @paulkrishanu4

@HimanshuNarware kindly label this PR otherwise it will not be counted in leaderboard

@PSS2134 label this PR


Rubrics

Design rubrics, considering:

type size, weight, color
underline weight, color
spacing
make sure they work on their own + within sidebars

On their own, in PyData default:

In sidebar, in PyData default:

Here's a link to the new rubric design in Figma:
https://www.figma.com/file/ahjEJjeaFL6oTjeQaueGDQ/napari-Sphinx-Theme?node-id=42%3A106

Some notes about the new rubric design:

the text is the same size as H3
the line should go above it (15px above on wide screens, see redlines screenshot above), and span the width that the text is allowed to fill (so this width will change if the rubric is in the main body text of the page versus inside a sidebar or other element
the line is black, 1px

P.S. More info about what these rubric things are, if helpful:
The "rubric" directive inserts a "rubric" element into the document tree. A rubric is like an informal heading that doesn't correspond to the document's structure.
https://docutils.sourceforge.io/docs/ref/rst/directives.html#rubric

Looks good to me!


Fix parsing for PrivateKeyDer

This following key is supposed to be pkcs8 key:
-----BEGIN PRIVATE KEY-----
MFMCAQEwBQYDK2VwBCIEIC2pHJYjFHhK8V7mj6BnHWUVMS4CRolUlDdRXKCtguDuoSMDIQDrvH/x8Nx9untsuc6ET+ce3w7PSuLY8BLWcHdXDGvkQA==
-----END PRIVATE KEY-----
it is included in one of rcgen tests for ring, so it is indeed probably pkcs8. The problem is that the version is set as 1, so the previous parsing considered it Sec1.
I am opening this PR so I can test it with rcgen to check again if it work correctly. I am not sure if this fix the issue. The tests in rcgen works now with this patch.
This may be relevant.

The problem is that the version is set as 1, so the previous parsing considered it Sec1.

Indeed, its version is encoded as 02 01 01 (INTEGER of len 1 with value 1), but RFC 5208 Appendix A defines the Version field as:
Version ::= INTEGER {v1(0)} (v1,...)
I think this is a similar error as I've seen elsewhere (x509 cert version for e.g.): version one is encoded as value zero, not value one. The right encoding for PKCS8 Version v1 is 02 01 00.

it is included in one of rcgen tests for ring, so it is indeed probably pkcs8

Aha, specifically ED25519_TEST_KEY_PAIR_PEM_V2, which has a comment explaining its provenance as coming from a debug print in a unit test, presumably using ring :thinking:
I'm curious how the version was encoded incorrectly here, but in either case your fix to relax the heuristic seems OK to me in the presence of real world keys like this.
Thanks for digging in!

This may be relevant.
...
I'm curious how the version was encoded incorrectly here,

Oh! I missed this link entirely. The response there does seem like a relevant explanation.

I'm curious how the version was encoded incorrectly here,

Oh! I missed this link entirely. The response there does seem like a relevant explanation.

So should we not encode that as Pkcs8, but either yield an error or maybe even add a fourth variant?

So should we not encode that as Pkcs8, but either yield an error or maybe even add a fourth variant?

I'm not sure yet.
RFC 5958 says:

If any items tagged as version
-- 2 are used, the version must be v2, else the version should be
-- v1.  When v1, PrivateKeyInfo is the same as it was in [RFC5208]

So it sounds like you could emit a RFC5958 PrivateKeyInfo that was compatible with PKCS8 by avoiding the new fields.
In this case, the PrivateKeyInfo's first and only member OneAsymmetricKey is using a --2 tagged field; it includes the optional explicitly tagged [1] PublicKey at the end. So should and does use version 2 and it isn't strictly speaking PKCS8.
That said: do the downstream APIs we feed these bytes to care? or are they all just treating this as PKCS8 anyway? :thinking:

Yeah, I guess for our purposes ring and aws-lc-rs are the authorities on what counts as PKCS#8.

So it should, and does, use version 2 and it isn't strictly speaking PKCS8.

I take this back, I think since 5958 obsoletes 5208 it would be fair to still consider it PKCS8.
Ring is specifically citing 5958 for it's definition of PKCS8 so I think in the end this is semantics and we shouldn't have over-fit our heuristic on the RFC 5208 version number.
I think we should ship the fix we have and not split the enum further.

Ring is specifically citing 5958 for it's definition of PKCS8 so I think in the end this is semantics and we shouldn't have over-fit our heuristic on the RFC 5208 version number.
I think we should ship the fix we have and not split the enum further.

SGTM.

For completeness, it looks like aws-lc-rs is citing the older RFC, but also recognizes V2 and the fn for decoding is described as supporting v1 and v2 through the same entrypoint.


Bug: No validation rules for the placeholder

PHP Version
8.2
CodeIgniter4 Version
4.4.4
CodeIgniter4 Installation Method
Manual (zip or tar.gz)
Which operating systems have you tested for this bug?
Linux
Which server did you use?
apache
Database
MySql 8
What happened?
When providing an array to the Validation run() function, if a field isn't present and you also have a placeholder field in the same rule set (having been set with setRules() (such as required|is_unique[users.Email,UserID,{UserID}]) then $this->retrievePlaceholders($row, $data); is incorrectly returning results for fields that aren't in its scope of validation. As such the error gets trigggered No validation rules for the placeholder when the field isn't actually present in the ruleset.
Steps to Reproduce
For example, the following is passed to setRules()
<pre>Array
(
    [CompanyID] => Array
	(
	    [rules] => required
	)

    [FirstName] => Array
	(
	    [rules] => required
	)

    [LastName] => Array
	(
	    [rules] => required
	)

    [Email] => Array
	(
	    [rules] => required|is_unique[users.Email,UserID,{UserID}]
	)

)
</pre>

But the data provided to run() is:
Array
(
    [UserID] =>
    [UserType] => 2
    [CompanyID] => 1
    [FirstName] => New
    [LastName] => user
    [Email] =><EMAIL_ADDRESS>    [UserLevel] => user
    [ResetPass] => 0
)

The error is generated "No validation rules for the placeholder: UserID" but clearly this isn't in the rule set
Expected Output
No error regarding placeholders to be reported.
Anything else?
The solution is to clearly remove the data prior to submitting it to run() however run should only be validating data it needs to and not every field given.

That is a bug in your app, not in CI4.
The error message may be not good, but what the message is trying to tell you is that you must validate the placeholder field. How do you think we can improve the error messages?

however run should only be validating data it needs to and not every field given.

No. You must validate all user input. Because attackers can send any data to your server.
If you don't validate all user input, it is just a bad practice.

Duplicate of #7605

I sent a PR to improve the error message: #8639


Option to Make Annotations Relative to Screen

Add an annotation that is locked to a place in the view rather than a location on the globe

Consider how this behaves when exported for Google Earth. Annotations were designed to represent KML BalloonStyle elements, which are always attached to a map location. We do step outside of the spec to allow dragging/resizing annotations, but this would be a further departure from the spec. This is more like a ScreenOverlay, except those contain an image instead of text.


Click is disabled when enabled_hotarea=0

When hotarea are enabled, I can click on a window to set the focus on it and then the overview ends.
When enabled_hotarea=0, if I click on a window, it get the focus but the overview does not ends.

Currently hot zone and mouse click jump are enabled with the same option, I can separate them, but wait until the next version, because some of the update code in the next version will conflict with the current one.

done


Remove jpinkney from codeowners

Signed-off-by: Josh Pinkney<EMAIL_ADDRESS>What does this PR do?
What issues does this PR fix or reference?
Is it tested? How?

PR Checklist

[ ] E2E tests pass (when PR is ready, comment /test v8-devworkspace-operator-e2e, v8-che-happy-path to trigger)

[ ] v8-devworkspace-operator-e2e: DevWorkspace e2e test
[ ] v8-che-happy-path: Happy path for verification integration with Che

/test v8-devworkspace-operator-e2

/test v8-devworkspace-operator-e2e


[CARBONDATA-2899] Add MV module class to assembly JAR

When compile the project with profile -Pmv, MV module class should be added into assembly JAR

[X] Any interfaces changed?
No

[X] Any backward compatibility impacted?
No

[X] Document update required?

[X] Testing done
Please provide details on
- Whether new unit test cases have been added or why no new tests are required?
- How it is tested? Please attach test report.
- Is it a performance related change? Please attach the performance test report.
- Any additional information to help reviewers in testing this change.
rerun all testcase

[X] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.
NA

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/6452/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/8139/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/68/

LGTM


updated folder structure for the repo

The files are now sorted, and a new src folder is added for the html css and js files, also the links in the code are updated.

Also I wanted to ask if the hactoberfest and the hacktoberfest are the same?
If not can you add the hacktoberfest and hacktoberfest-accepted  lable to this PR It will be helpful for me.

@Gyanthakur Also I wanted to ask if the hactoberfest and the hacktoberfest are the same? If not can you add the hacktoberfest and hacktoberfest-accepted lable to this PR It will be helpful for me.

you use previous
create new clone and solve issue
hosted link: https://e-learning-website-nu.vercel.app/login1.html


Remove files from the disk

In the previous release (0.13),  there used to be 2 "Remove" options. One for removing the task only, another for removing the files from the disk as well. This is particularly useful when the user finds out that he's downloading some wrong torrents.
It's gone in the new release, why?

#583 fixes this. I'll release it later tonight 😄 Sorry!


Migrate to 0.11.0

Added in https://github.com/nabakin/drop/commit/93eec1a518581f7ae7c63f26dc09aa4c6a54455d


GitLab in Docker starts to throw HTTP 500 after a while

I want to share this issue here first before I report it to GitLab to learn if any of you encountered the same thing.
I have been having problems with running tests on local GitLab in a Docker container for a while now.
Regardless of the GitLab version (I tried older 13.7.1-ee.0 as well as current latest 13.11.1-ee.0) after a while GitLab starts throwing HTTP 500.
The reason for that seem to be file ownership problem, as Postgres starts to report that it cannot access some of its files. Excerpts from its log:
2021-04-27_16:33:10.16215 WARNING:  could not open statistics file "pg_stat_tmp/global.stat": Operation not permitted

(TODO: add more error messages)
When I opened bash in the container and checked those file it turned out that they are owned by root with mode 600 so that's why Postgres gitlab-psql user cannot access them:
root@gitlab:/# ls -la /var/opt/gitlab/postgresql/data/pg_stat_tmp/
total 32
drwx------  6 gitlab-psql gitlab-psql   192 Apr 27 14:31 .
drwx------ 29 gitlab-psql gitlab-psql   928 Apr 27 14:25 ..
-rw-------  1 root        root         1526 Apr 27 14:31 db_0.stat
-rw-------  1 gitlab-psql gitlab-psql  5751 Apr 27 14:23 db_1.stat
-rw-------  1 root        root        13525 Apr 27 14:31 db_16386.stat
-rw-------  1 gitlab-psql gitlab-psql   639 Apr 27 14:31 global.stat

I tried following https://docs.gitlab.com/omnibus/docker/README.html#permission-problems (although it does not seem right as I had this problem on a very fresh GitLab instance, not upgraded from a previous version) but some time after container restart those files started to be owned by root again.
I checked that Postgres processes are run by gitlab-psql user in the container, so I see no obvious reason for all of these to happen...
I am using macOS Big Sur (11.2.3 (20D91)) and Docker for Mac 3.3.1 (63152) (Docker Enginer: 20.10.5) but I notices some 500s in the GitHub Actions test runs, f.e. https://github.com/egnyte/gitlabform/actions/runs/790297758 , but haven't yet verified if these have the same root cause.
@ss7548 @YuraBeznos @mcjffld @poffey21 @Pigueiras as authors of recent PRs: have you encountered this?

Sorry I can't help with this, I was executing the integration tests against a real GitLab dev instance because I couldn't manage to run GitLab in my laptop.

I also run tests in my dev instance and have not encountered this issue.  I looked at my prod instance and all permissions are correct.  I am running 13.9.2

I think I got it fixed in v2 RC3.


[Suggestion] Reddit posts in private subs

It is currently not possible to rip a private subreddit, or a posts that a user has posted to a private subreddit. Would it be at all possible to eventually rip this content?

You would need to be logged in to an account which can access said subreddit. We could use cookies for this. See #354. Reddit's security would prevent someone (the public) without permission from accessing the content by any means.


some clippy fxing

Please help keep the CHANGELOG up to date by providing a one sentence summary of your change:
#1402

Thanks! 😄


DBZ-3083 Make CI builds resilient against disconnects on GH Actions infrastructure

https://issues.redhat.com/browse/DBZ-3083

CI passed, so I'll merge it. Thx!


Sema: Allow protocol typealiases to witness associated type requirements

When resolving a type witness by lookup, also consider protocol members.
Fixes rdar://problem/30442622.

@swift-ci Please smoke test


support for "sinon": "^4.0.1" missing

npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
I have
"sinon": "^4.0.1",
"chai": "^4.1.2",

I was about to open a PR for this... is there any reason why sinon and sinon-chai have such strict versioning?  It doesn't seem to be doing too much with these packages, particularly sinon.  I'm new to this repo, but would it be possible to allow all versions of sinon/chai?
Looks like I don't have permissions to create a pull request, but package.json could look something like this:
{
  "name": "karma-sinon-chai",
  "description": "Sinon and Chai for Karma",
  "keywords": [
    "karma",
    "chai",
    "sinon",
    "mocha",
    "should"
  ],
  "version": "1.3.1",
  "author": "Kevin Mees<EMAIL_ADDRESS>  "main": "./index.js",
  "license": "MIT",
  "homepage": "https://github.com/kmees/karma-sinon-chai",
  "repository": {
    "type": "git",
    "url": "https://github.com/kmees/karma-sinon-chai.git"
  },
  "bugs": {
    "url": "https://github.com/kmees/karma-sinon-chai/issues"
  },
  "contributors": [
    "Greg Thornton<EMAIL_ADDRESS>    "Tobias Kopelke<EMAIL_ADDRESS>  ],
  "peerDependencies": {
    "chai": "*",
    "sinon": "*",
    "sinon-chai": "*"
  },
  "dependencies": {
    "lolex": "^1.6.0"
  },
  "devDependencies": {}
}

Is there any reason why we

Is there any reason why we wouldn't want to do this?
I wouldnt see any reason not to.

I just removed the upper limit for peer dependencies with 2.0.1.


Removing auto-generated 'Device' tab

I'm trying to get rid of the 'Device' tab in the metadata.
I'm trying this, with no luck :
bugsnag.Config.BeforeNotify(error =>
{
  error.Metadata.RemoveTab("DEVICE");
  return true;
});

How can I get rid of this tab ?

Hi @jeanlaurent, I believe the tab name is actually lowercase in the payload. Did you try error.Metadata.RemoveTab("device")?

Closing due to inactivity, device should fix the problem.

The problems still persists...I  cannot remove "Device" Tab using error.Metadata.RemoveTab("device")

The error.Metadata.RemoveTab() call won't work for device information as this isn't stored as metadata but captured and appended to the notification before being sent (see https://github.com/bugsnag/bugsnag-dotnet/blob/v1.4/src/Bugsnag/NotificationFactory.cs#L79)
Reopening this issue for further investigation. Note that this only applies to v1.4.

Yeah this is fixed in v2 looking at https://docs.bugsnag.com/platforms/dotnet/asp-net-core/automatically-captured-data/#device-information

Closing this as in the new version of the notifier this is possible to do in a callback, as linked to above.


Add resources to Mathematics, Haskell, Linux, and JavaScript

Hacktoberfest notes:

due to volume of submissions, we may not be able to review PRs that do not pass tests and do not have informative titles.
please read our contributing guidelines
be sure to check the output of Travis-CI for linter errors
if this is your first open source contribution, make sure it's not your last!

What does this PR do?
Add resources to Mathematics, Haskell, Linux, and JavaScript
For resources
Description
Added in 3 free programming books & 1 mathematics book:
Haskell Notes for Professionals (PDF)
Linux commands Notes for Professionals (PDF)
JavaScript Notes for Professionals (PDF)
The Joy of Cryptography (2020) (PDF) I added the other link and put the 'in process' notation.
Why is this valuable (or not)?
They are interesting subject areas.
How do we know it's really free?
You don't need an email to get the PDF's
For book lists, is it a book? For course lists, is it a course? etc.
Books
Checklist:

[x] Not a duplicate
[x] Included author(s) if appropriate
[x] Lists are in alphabetical order
[x] Needed indications added (PDF, access notes, under construction)

Thank you for your review!

better to add a commit to a PR than to restart (losing comments)

Sorry, my mistake. I'm new to this, so I was thinking the 'resubmit PR' meant to make a new one.


Error after I just git clone and upload the extension

I just git clone the project and upload the extension to the chrome in developing mode, and it shows that.

Same issue as #1


Added new versions of Tarantool to CI

I see you've met the same problem with the error format in Tarantool 2.8. I've solved it this way: https://github.com/tarantool/ddl/pull/61/files#diff-46bb4ca7dd938769f45b0014c0b70e7d47ce3c190419b67d445be821e719e0c5


silx/test folder

There is a silx/test/ directory that used to be an unused placeholder for the test suite and that now contains code, possibly put there by accident. Should we delete this directory?
https://github.com/silx-kit/silx/commits/master/test

I think it is a script to do interactive test.

File to be deleted.


Support recording request headers in record mode

The title says it all... :)

Maybe for the beginning just add a switch to record all of them or none (as of now). Personally, I feel removing superfluous headers is much easier and quicker than adding the necessary ones in
this option seems sensible
@tomakehurst is it possible to record all request headers as an option?
E.x."RecordAllHeaders" : "true",


Firefox: Stack trace breaks?

Describe the bug
I try to run my dev server, and get this error:
[get stackTraceLimit@https://github-8qeupk.w-corp.staticblitz.com/blitz.f2b430413f5697306be55fc5c653053509a24007.js:6:291483
getErrorWithoutStack@https://github-8qeupk.w-corp.staticblitz.com/blitz.f2b430413f5697306be55fc5c653053509a24007.js:6:1073932
generateUnhandledRejectionError@https://github-8qeupk.w-corp.staticblitz.com/blitz.f2b430413f5697306be55fc5c653053509a24007.js:6:1074166
processPromiseRejections@https://github-8qeupk.w-corp.staticblitz.com/blitz.f2b430413f5697306be55fc5c653053509a24007.js:6:1074928
processTicksAndRejections@https://github-8qeupk.w-corp.staticblitz.com/blitz.f2b430413f5697306be55fc5c653053509a24007.js:6:797087
_0x48295c@https://github-8qeupk.w-corp.staticblitz.com/blitz.f2b430413f5697306be55fc5c653053509a24007.js:8:227321
]

Link to the blitz that caused the error
https://stackblitz.com/edit/github-8qeupk?file=package.json
Steps to reproduce
visit the blitz, let the automation try to start the dev server
Expected behavior
dev server boots. no error.
Screenshots
No response
Platform

OS: Ubuntu
Browser: FireFox
Version: 99.0.1

Additional context
No response

Hey 👋 Thanks for reporting this issue. I have briefly looked at it and it seems there's still an issue with the esm package aside from the weird error you're seeing in Firefox. For both we need to take a closer look and I created a ticket internally for this.


Playground sample README update

Add stuff to the playground sample README, re: #4421.

Awesome!!

Thanks @ganicke


Use UITextView tint color for coloring the undeline and floating labe…

One might want to customize the colors used by the floating label and underline.
On Android, this can already be done by customizing the theme or TextInputLayout in XML.
According to this on Android, the colors cannot be changed without a hack, creating access to private color members.
So on iOS, the behavior should probably be similar: Use the theme color.
Description
Changes Proposed in this pull request:

Use TintColor for FloatingLabelActiveTextColor
Use UITextView.Appearance.TintColor for _editingUnderlineColor

This is a:

[ ] Bug Fix
[ ] Feature Request
[X] New Feature

This is a good stop-gap for the issue of changing colors.
Thank you.


Chinese version of this document missing?

There used to be a Chinese version of this document. But it is missing recently. How to find or get the Chinese version of this document?
https://docs.microsoft.com/zh-cn/azure/devops/git/what-is-version-control
[在此处输入反馈]

文档详细信息
⚠ 请勿编辑此部分。 docs.microsoft.com ➟ GitHub 问题链接必须具有此部分。

ID: 64f1b21d-ba58-7b3f-126f-d62c0ff931b3
Version Independent ID: c82e367a-1080-8a43-0386-b9e1eecc42c7
Content: What is version control? - Azure DevOps
Content Source: https://mseng.visualstudio.com/DefaultCollection/TechnicalContent/_git/devops-learn-pr?path=%2fdocs%2fgit%2fwhat-is-version-control.md&version=GBlive&_a=contents
Service: devops
GitHub Login: @samgu
Microsoft Alias: routlaw

@moyanming These documents were just migrated over to docs.microsoft.com and I think the translated versions are coming shortly. I'll see if I can get an update on the timeline.

@steved0x
Hello, do you have a repository in Chinese?
Or build one?
I translated several articles, but found that he did not have a Chinese repository ?
How do I deal with and cooperate?

Azure DevOps documentation is not currently localized.


0.13 upgrade

The repo here wasn't up-to-date with the latest version uploaded on Nexus. Fortunately, I could extract the scripts for that mod, so I've added them here.
The main change was splitting PostProcessingBehaviour into PostProcessLayer and PostProcessVolume. I did some Googling and looked at DFU as a reference, I think it should behave the same as before.
I ran my build through some DFU players, everything seems good.
Tell me if you want a compiled .dfmod to upload on Nexus. If you haven't kept up with DFU modding, you need Unity 2019.4.28f1 to build for the latest versions

I can upload the newest dfmod if you send it to me, thank you so much!


[Scenes] Add Chip-tool support for scenes commands

Preliminary test shows that if we try to run the add-scene command, chip-tool issue the CHIP_ERROR_INVALID_ARGUMENT error code since the number of arguments seems to be more than the average command

Once a destination device has been commissioned and paired with the nodeId=25119 and the GroupId=0x4141, the following formatting of the add-scenes command will result in the General error: 0xc3 (UNSUPPORTED_CLUSTER) from that end device:
./chip-tool scenes add-scene 0x4141 1 1 "Test" '[{"clusterId": 5, "attributeValueList":[{"attributeId": 1, "attributeValue": [1]}]}]' 25119 1
Chip-tool support for scene commands thus seems functional.


Space issue fixed  #150

Fix the alignment of items on the Navbar. Increase Spaces #150

Host your repo on github pages and share the link below..

https://github.com/harshita214/Aec-Library-Website


configuration/http_filters/buffer_filter.md

原文：https://github.com/servicemesher/envoy/tree/master/configuration/http_filters/buffer_filter.md

认领


cohorts: Use IntervalIndex to reduce search space.

This was a great improvement for the county grouping but a regression for periodic time groupings where the bounding boxes are quite large and do not reduce the search space at all. In #306 I switched to using the bitmask to reduce the search space. This is even better for the county grouping and generally better for the periodic grouping. More optimizations are possible there.


Plan to update the base OS version?

Thank you for maintaining the Ruby runtime images!  Please allow me to ask a question.  Is there a plan or ongoing project to update the base OS version?
All images are currently based on Ubuntu 16.04 LTS.  It is stable and still works great, but it has been almost five years since it was released and it's becoming outdated [^1], meaning you can use fewer and older packages for your application.  There are already times where you need to manually build and install recent versions of software from the source and its dependencies instead of simply installing prebuilt packages on the fly using apt.
Looking at Circle CI and GitHub Actions, their officially provided docker images are mostly based on Ubuntu 18.04/Debian Buster (default) or Ubuntu 20.04 (experimental).  So, in order to test your application on the same OS as the running environment, you need to prepare and maintain Ubuntu 16.04 based custom images for CI.  This is becoming a burden in development.
I'd much appreciate if Ubuntu 20.04 based images could be provided before 16.04 is considered completely obsolete.  I'm eager to test and submit feedback when beta versions are ready.
[^1]: For the record, it will keep receiving security updates for the next three years until April 2024 through the Extended Security Maintenance period. (Less than 6 months to Ubuntu 16.04 ESM: 6 things to prepare)

I have a task to update the base image to 20.04. We'll get that done soon, probably not before the end of April, but this quarter for sure.

Sorry this got pushed down the stack again, but we're finally working on it. See #221

Thank you so much for the work and your continuous effort!  Unfortunately, the GAE-based Rails app I worked hard on ceased to exist last month due to service/company shutdown, and it's really sad I couldn't get to see it running on the new base image.  I hope other users will benefit a lot from this.  Cheers!


Role configurator bug

There is a problem in the B/E when building the permission object when we have permission groups from the same parent with different different levels of depth

fixed with a373333


Battle mechs.

About the PR

This PR adds Nanotrasen and Syndicate battle mechs to the game.
(re-opening an abandoned Pull Request - #19263 )
Why / Balance

Technical details

The MechGunSystem has been added, as well as the SharedGunSystem has been slightly changed.
Media

The mechs in the first two lines can be researched in R&D

Required items to create new mechs

Syndicate mechs in the nuke ops uplink

Researched mech weapons in the security techfab

Requirements

[X] I have read and am following the Pull Request and Changelog Guidelines.
[X] I have added media to this PR or it does not require an ingame showcase.

Breaking changes

Changelog

:cl: ThunderBear2006, NULL882

tweak: H.O.N.K. received an airtight cabin for honk operations.
add: Added new Nanotrasen mech - Ripley MK-II (Has a pressurized cabin for space operations, but the added weight has slowed it down.)
add: Added new Nanotrasen mech - Clarke (A fast-moving mech for space travel. It has built-in trusts.)
add: Added new Nanotranse mech - Gygax (While lightly armored, the Gygax has incredible mobility thanks to its ability that lets it smash through walls at high speeds.)
add: Added new Nanotrasen mech - Durand (A slow but beefy combat exosuit that is extra scary in confined spaces due to its punches.)
add: Added new CentCom mech - Marauder (A specialized mech issued to the ERT.)
add: Added new DeathSquad mech - Seraph (A limited edition exosuit that comes with advanced equipment.)
add: Cybersan began producing combat mechs to supply nuclear operatives with them. (Drak Gygax - 100 TC and Mauler - 150 TC)
add: In the exosuit fabricator, you can now create researched mechs such as: Ripley MK-2, Clark, Gygax, Durand.

It's happened!!!

I love this

about sprites, they probably should be resprited, black outlines are very bad

Oops, I forgor to add the ability to print a banana and a mousetrap mortar.

Did a quick glance at this +4258 lines monstrosity..
uplink-bundele-mech-heavy-desc = Contains a set of heavy armored Cybersan mech with integrated chainsword, Ultra AC-2, LBX AC 10 "Scattershot", BRM-6 Missile Rack and P-X Tesla Cannon
Bundele should be Bundle. "A set of heavy armored Cybersan mech" (and many more similar examples) sounds off. It shouldn't have "a set of".

Like that?
uplink-bundele-mech-heavy-desc = Contains heavy armored Cybersan mech with integrated chainsword, Ultra AC-2, LBX AC 10 "Scattershot", BRM-6 Missile Rack and P-X Tesla Cannon.

Compared to the others, Ripley's textures look too outdated

This PR is being finalized.

What about the paddy? There's a texture for the claw and the upgrade kit but I don't see it mentioned anywhere else

What about the paddy? There's a texture for the claw and the upgrade kit but I don't see it mentioned anywhere else

Later.

I thought there was a feature freeze on new vehicles until vehicle was refactored ?

I thought there was a feature freeze on new vehicles until vehicle was refactored ?

Mech code isn't vehicles which is another crime vehicles did.

YES!!! I LOVE PUTTING GIANT GUNS ON MECHS!!!!!!

Another thing, in the old pr it seems like the mechs will be able to do combat spins and stuff, but iirc in 13 you had to be facing the direction you wanted to shoot to actually fire the gun. I feel like that slightly balances them out a bit and simulates the clunkyness that piloting a mech would cause, unless you improved that over the old pr. Good luck though, cool mechs are one of my favorite things from 13.

As a roboticist main, damn

sentient titanfall mech

cybersan is just a typo, presumably because of russian sounds

https://github.com/space-wizards/space-station-14/pull/23948#issuecomment-1891045447 maybe that energy barrier can be atached with Durand or it can be like a module of Mech

this is based, but the uplink descriptions need more flavor and less detail. for example:
Assault Mech Teleporter
A lightly armored mech with a chainsaw sword, AC 10 "Scattershot" assault module, light missile rack, and Tesla cannon. Packs a lot of punch and takes a lot of punch.
Cybersun's greatest hit. Chainsaw sword, AC 10 "Scattershot" assault module, powerful missile rack, and P-X Tesla cannon.
I mighta done too much flavor but I want the idea of buying these to be exciting and mysterious

so hot take but I don't think mechs really NEED their tank controls from SS13. It would be nice.. but it would also be clunky and not in a good way. I think mechs being super slow is already a huge downside in the movement based combat we have.

these mechs in this PR still hurt the pilot whenever they get hit by ANYTHING, not just if that hit actually penetrates, so yeah it'd be a bit crappy because these mechs don't properly protect you

these mechs in this PR still hurt the pilot whenever they get hit by ANYTHING, not just if that hit actually penetrates, so yeah it'd be a bit crappy because these mechs don't properly protect you banging on the mech with a baton will eventually crit the pilot inside and kill them even though the word mech defines a external vehicle that isn't just power armor

this is a value that can be changed, too. mechToPilotDamageMultiplier: is the bit that should be changed, and for combat mechs it should ideally be 0.1 or even lower.

This PR is being finalized.

Are you working on the requested changes?

I don't really have time to work on the changes, so I think the best solution would be to close this PR. Perhaps someone will want to reopen it.

WE WERE SO CLOSE......
NOOOOOOOOOOOOOOOOO


Data_Structures => Tree => Binary_Tree => Subtree_check.cpp

Assigned Issue Number ?
#1286
Language Opted ?
C++
Number of files added or updated ?
1
Name of these files ?
Data_Structures => Tree => Binary_Tree => subtree_check.cpp
example :
Algorithm => Search_Algorithm => Linear_Search => Linear_Search.py
Checklist

[x] I've read the contribution guidelines.
[x] I've referred the correct issue number.
[x] I've fill up this entire template correctly.

@ahampriyanshu could you add hacktoberfest-accepted tag to my PR? I was under the impression that this repository was accepting hacktoberfest PRs.


Distribution scripts fix #162 did not build latest codebase

I have pulled the latest angular-timer with bower using:
 "dependencies": {
    "angular": "~1.3.13",
    "angular-timer": "git@github.com:siddii/angular-timer.git"
}

The files under the distribution folder in bower_components/angular-timer/dist/are still showing v1.3.0 - 2015-03-03 5:32 PM but the latest build should be v1.3.0 - 2015-03-06 10:12 AM

@fman7 I had this issue yesterday & it prevented me from making any progress because the old build was breaking with mine.
In case you are in the same boat as me, I got it to work by removing the angular-timer directory that I previously bower installed and doing a git clone https://github.com/siddii/angular-timer.git
this grabs the right build and includes the fix (#164)


Create dependabot.yml

Description
Add Dependabot configuration to keep dependencies up-to-date.
It has opened PRs on my fork: https://github.com/VachaShah/opensearch-py/pulls
Issues Resolved
As part of https://github.com/opensearch-project/opensearch-clients/issues/11
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Closing this, will open a new one from the correct branch.


Comments Not Appearing on cUSD and cEUR Payments

Current behavior
Comments Fail to appear on cUSD and cEUR transactions.
Desired behavior
Comments should appear on cUSD and cEUR transactions in the transaction feed.
Steps to Reproduce

Send a payment to an account with an attached comment
Observe the comment in the transaction feed.

Versions
v1.30.x
https://user-images.githubusercontent.com/26950305/163634774-06451464-b0c1-4416-9de6-938ba9f32fbb.mov
Video

Hey @MuckT, We have verified the above issue on Latest Android Internal Release Build V1.30.1 ,Test flight Release Build V ,Android Alfajores Build V1.30.1 & Test Flight Alfajores Build V1.30.1 and observed that the issue is not reproducible.
Observation:

User is able to add description and also able to see description in Payment sent activity while sending cUSD and cEUR

Devices: Samsung Galaxy A9 pro (8.0), Google Pixel XL (10.0), iPhone 13 mini(15.1.1), iPhone 12(14.7.1)
Attachments: Mainnet.mp4
Alfajores.mp4
Thanks..!!

I disagree that this is a P0.
This is a long term existing issue. It happens because when we send a transaction we show a temporary thing in the feed that we create locally until we get the actual transfers from blockscout. These temporary (or "standby" as we call it in the code) entry has the bug above, but like I said it's an existing issue and should not block the release. It only happens because of blockscout issues.

Closed in favor of #1639.


Documentation Microsite

Get the documentation microsite up and operational including examples.
Sections for the core operations and then pages for each of the modules.

@ChristopherDavenport
Quick Start

To use log4cats in an existing SBT project with Scala 2.11 or a later version, add the following dependencies to your build.sbt depending on your needs:

I think you might have had a mis-type here

This is what you get when you copy-paste. Yes, I'll fix that.

Fixed by #35


Iterate Lessons

Focus on minimal lesson require for development and testing.

Iterate succeeding lessons as project progresses.

Document lessons location and how to create, and edit them.

I added an api documentation here here


Secret properties in application.conf cannot be set via environment variables

Not all secret properties in the application.conf files can currently be set via environment variables, even though the application.conf files give the impression that this would be possible. The reason is that the hard-coded secrets paths in some cases do not match the paths of the configuration properties.
For example, the database secrets can be configured via environment variables because the properties in application.conf match the paths in the getSecret() calls (they are database.username and database.password in both cases):
https://github.com/eclipse-apoapsis/ort-server/blob/4a4a6d13a01bbb8282dd968195067d5011211645/core/src/main/resources/application.conf#L52-L64
https://github.com/eclipse-apoapsis/ort-server/blob/4a4a6d13a01bbb8282dd968195067d5011211645/dao/src/main/kotlin/DatabaseConfig.kt#L120-L121
For the RabbitMQ configuration they do not match. For example, the RabbitMQ password in the analyzer worker configuration is at the paths analyzer.receiver.password and orchestrator.sender.password, but getSecret() is called with the path rabbitMqUser:
https://github.com/eclipse-apoapsis/ort-server/blob/4a4a6d13a01bbb8282dd968195067d5011211645/workers/analyzer/src/main/resources/application.conf#L66-L77
https://github.com/eclipse-apoapsis/ort-server/blob/4a4a6d13a01bbb8282dd968195067d5011211645/workers/analyzer/src/main/resources/application.conf#L84-L95
https://github.com/eclipse-apoapsis/ort-server/blob/4a4a6d13a01bbb8282dd968195067d5011211645/transport/rabbitmq/src/main/kotlin/RabbitMqConfig.kt#L56-L57
https://github.com/eclipse-apoapsis/ort-server/blob/4a4a6d13a01bbb8282dd968195067d5011211645/transport/rabbitmq/src/main/kotlin/RabbitMqConfig.kt#L70
As a result, it is currently not possible to configure the secrets for RabbitMQ via environment variables and the secret-file provider has to be used for them. This makes the configuration unnecessarily complex if environment variables would be the preferred solution, like in the Docker Compose setup.
Note that I did not check which other secrets are affected by this problem.

The current implementation has the benefit that the RabbitMQ secrets have to be configured only once if the secret-file provider is used, as they are always requested from the same path, no matter which property in application.conf they refer to.
One option to fix this issue while keeping this behavior would be, that if a secret provider is configured, the values of the properties reference the name of the secret instead of the value.
For example, if no provider is configured, analyzer.receiver.password would contain the RabbitMQ password. If a provider is configured, the property would contain the path of the secret instead, for example "rabbitMqPassword". The provider would then use this value to look up the actual secret value.

That the configuration still contains properties for passwords is wrong IMHO; this is a remainder from an early stage before the introduction of secret providers. The mode that allows reading secrets from the configuration was just a quick work-around, but it was never fully functional due to the mismatches between secret names and configuration property names.
Wouldn't be the correct way to solve this issue to implement a secret provider that reads secrets from environment variables? (And if it was actually needed to have multiple sources for secrets, there could be a composite secret provider that queries a configurable set of child providers.)

I've understood and imagined the secret management to be like this:

secrets required for deploying ORT Server are given to the containers via environment variables,
secrets required for ORT steps to run within ORT Server, like VCS credentials for the analyzed projects are managed with the secret providers, so stored in Vault for example.

Is this incorrect?

@mmurto

* secrets required for deploying ORT Server are given to the containers via environment variables,

Incorret, based on @oheger-bosch's response. As described in this issue it "accidentally" works for some secrets where the secret paths match those of the properties in the config files, but in general they should be loaded from the config secret provider.

* secrets required for ORT steps to run within ORT Server, like VCS credentials for the analyzed projects are managed with the secret providers, so stored in Vault for example.

Correct.
@oheger-bosch

Wouldn't be the correct way to solve this issue to implement a secret provider that reads secrets from environment variables?

That would allow to use environment variables, but there would still be an inconsistency between config properties and secrets:
You can configure different transports of the same type but not set separate secrets, because the secret paths are hardcoded. For example, when using RabbitMQ you could set different values for ADVISOR_RECEIVER_TRANSPORT_SERVER_URI and ANALYZER_RECEIVER_TRANSPORT_SERVER_URI but you cannot set different credentials because they are always read from rabbitMqUser and rabbitMqPassword (or probably RABBIT_MQ_USER and RABBIT_MQ_PASSWORD as env variables).
It would be more consistent if the URI would always be read from RABBIT_MQ_URI.

(And if it was actually needed to have multiple sources for secrets, there could be a composite secret provider that queries a configurable set of child providers.)

I think that's not required.

@mmurto

* secrets required for deploying ORT Server are given to the containers via environment variables,

Incorret, based on @oheger-bosch's response. As described in this issue it "accidentally" works for some secrets where the secret paths match those of the properties in the config files, but in general they should be loaded from the config secret provider.

* secrets required for ORT steps to run within ORT Server, like VCS credentials for the analyzed projects are managed with the secret providers, so stored in Vault for example.

Correct.

Thanks for the clarifications.

(And if it was actually needed to have multiple sources for secrets, there could be a composite secret provider that queries a configurable set of child providers.)

I think that's not required.

For me it would be clearer to have multiple sources of secrets, or somehow have the ability to provide secrets required for the ORT Server to function via environment variables, and secrets for ORT runs via Vault/other secret provider. IMO these are distinct in nature: the former ones are declared once for the deployment, using whatever the correct way to do so for the deployment platform is. As we're deploying to K8s, we use External Secrets Operator to fetch the secrets from the applicable secret store and inject them to the containers via environment variables. I'm not super deep in the security hardening though - is there some advantage for adding another layer (secret provider) on top of this?
The secrets required by ORT runs on the other hand can change at run time, for example when a new organization allows access to their private repos. so for them a dynamically accessed secret provider seems natural to me.

* secrets required for deploying ORT Server are given to the containers via environment variables,

Incorret, based on @oheger-bosch's response. As described in this issue it "accidentally" works for some secrets where the secret paths match those of the properties in the config files, but in general they should be loaded from the config secret provider.

And thinking this through more, it probably can't work that way for all secrets, as some secret providers like Vault and Scaleway Secrets require secrets like API keys or tokens to access them. They obviously can't be fetched from the provider so they will anyway need to be provided via some other means.

There is a distinction between secrets required during ORT runs to access repositories and secrets required for accessing infrastructure of the ORT Server deployment. The former are accessed via the secrets abstraction; the latter via the config abstraction.
The config abstraction is more low-level and allows only read access to secrets. Reading secrets from environment variables would be one possible implementation. But there are other valid approaches as well. For instance, in Azure there is a special integration between Azure Key Vaults and Azure Kubernetes clusters. Here, the secrets are mounted as files into pods.

There is a distinction between secrets required during ORT runs to access repositories and secrets required for accessing infrastructure of the ORT Server deployment. The former are accessed via the secrets abstraction; the latter via the config abstraction.
The config abstraction is more low-level and allows only read access to secrets. Reading secrets from environment variables would be one possible implementation. But there are other valid approaches as well. For instance, in Azure there is a special integration between Azure Key Vaults and Azure Kubernetes clusters. Here, the secrets are mounted as files into pods.

Thanks, this makes it clearer for me!

So now (I think :D) I get it. There's one ConfigManager per ORT Server instance, and the ConfigManager is constructed with one instance of ConfigFileProvider and one instance of ConfigSecretProvider.
Of these, ConfigFileProvider provides the ORT configuration files, such as license classifications etc. and ConfigSecretProvider provides the secrets required to run ORT Server (which can also be files, but they're secret files.
If this understanding is correct, would it be clearer to separate the ConfigFileProvider from ConfigManager? I think configuration for where to get license classifications etc. could make sense to provide at run time, and I think it could also be valuable to be able to configure different ConfigFileProvider per organization, so IMO ConfigFileProvider may be closer to the secrets abstraction than the ConfigSecretProvider.

Your understanding is correct.
However, I do not see the need for having multiple ConfigFileProvider instances. The file provider merely abstracts the way how configuration files are accessed, and I assume this is always the same for a given ORT Server installation. It may be the case that the configuration files for different organizations are located at different paths. But this then needs to be addressed when calling the provider.


Error : failed to extract organization name from organizationURL

Report
I am setting the azure pipeline scaler for first time , I had installed keda version 2.7.1 and created the scaleobject and TriggerAuthentication using below yaml file , but I am getting error
apiVersion: v1
kind: Secret
metadata:
  name: pipeline-auth
data:
  personalAccessToken: <PAT_TOKEN>
---
apiVersion: keda.sh/v1alpha1
kind: TriggerAuthentication
metadata:
  name: pipeline-trigger-auth
  namespace: devops
spec:
  secretTargetRef:
    - parameter: personalAccessToken
      name: pipeline-auth # pipeline-auth
      key: personalAccessToken # personalAccessToken
---
apiVersion: keda.sh/v1alpha1
kind: ScaledObject
metadata:
  name: azure-pipelines-scaledobject
  namespace: devops
spec:
  scaleTargetRef:
    name: azdevops-deployment-keda
  minReplicaCount: 1
  maxReplicaCount: 5
  cooldownPeriod: 3
  triggers:
  - type: azure-pipelines
    metadata:
      poolID: "1"
      organizationURLFromEnv: "AZP_URL"
    authenticationRef:
     name: pipeline-trigger-auth

Expected Behavior
scaledObjects should be not failing
Actual Behavior
I can see error in the keda operator
2022-12-07T19:25:31Z	ERROR	controller.scaledobject	Reconciler error	{"reconciler group": "keda.sh", "reconciler kind": "ScaledObject", "name": "azure-pipelines-scaledobject", "namespace": "devops", "error": "error parsing azure Pipelines metadata: failed to extract organization name from organizationURL"}
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem
/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.11.2/pkg/internal/controller/controller.go:266
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2.2
/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.11.2/pkg/internal/controller/controller.go:227
Steps to Reproduce the Problem

installed keda with helm chart
created scaledobject and TriggerAuthentication

Logs from KEDA operator
example

2022-12-07T19:25:31Z	ERROR	controller.scaledobject	Reconciler error	{"reconciler group": "keda.sh", "reconciler kind": "ScaledObject", "name": "azure-pipelines-scaledobject", "namespace": "devops", "error": "error parsing azure Pipelines metadata: failed to extract organization name from organizationURL"}
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem
/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.11.2/pkg/internal/controller/controller.go:266
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2.2
/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.11.2/pkg/internal/controller/controller.go:227
KEDA Version
2.7.1
Kubernetes Version
1.23
Platform
Microsoft Azure
Scaler Details
azure pipeline
Anything else?
No response

Hi,
Could you share the value of the env AZP_URL?

I have tried with dev.azure.com/ABC and it works but with dev.azure.com/ABC/ it doesn't work. Do you have a slash at the end?

yes i have slash at the end , but I am curious , do I have to put organizationURLFromEnv as mentioned below or should it have value like organizationURLFromEnv: "https://dev.azure.com/ABC"
organizationURLFromEnv: "AZP_URL"

For this propose, the protocol isn't relevant because it parses the orgName based on slashes (that's why if fails with the extra slash), but I guess it should be with the protocol

yeah, it should be with the protocol and without the end slash because the code does this:
url := fmt.Sprintf("%s/_apis/distributedtask/pools?poolID=%s", metadata.organizationURL, poolID)

thank there were two issues , by doing below steps I was able to get it fixed and thank you for your help.

I was using different secret for deployment of agent and different for TriggerAuthentication , which was actually a problem , moved to one single secret
second using the AZP_URL without / like you suggested

Nice to hear that you solved it ❤
Can we close this issue as it's already fixed?

yes it closed , please close the issue thank you for your help


feat(core): support 'read' option for ngIvy queries

This PR makes it possible to specify the read option for queries. The exact type to be red can be specified using a newly introduced QueryReadType. Most of the PRs code boils down to tests plus simple logic to return appropriate value (based on QueryReadType). Additionally there is a bug where container nodes were not considered by queries - this is fixed in this PR as well.

Looks great. Please take care of the minor comments, and than I will merge it in.

@mhevery I've addressed all the comments, please have another look / merge.


Structured configuration

@Alex-Izquierdo does this require changes to the container build?

@benthomasson no, all required changes are included here.


Why does pip-accel make an sdist (?) archive when I pip-accel install -e .?

I want to use pip-accel to install and cache binary archives for the dependencies listed in ./setup.py, but I don't want or need to build or cache an archive for . -- I'm only installing it to get the dependencies, and to add . to the Python path. There are a lot of files in . (e.g. 1GB temporary SQL files in my working copy), and pip-accel seems to want to include these in the archive, too. It hangs for a long long time (longer than most people can be bothered to wait, assuming it has hung completely), on Obtaining file://...

Hi Tai Lee, thanks for the feedback and sorry to hear that pip-accel is giving you trouble!
Disclaimer: Let me preface all of this by explaining that when I created pip-accel I didn't intend for it to support editable installations at all (it simply wasn't on my radar). Users requested proper support for this feature and adding it was rather trivial based on integration with older versions of pip. That situation has since changed unfortunately, but I've tried to preserve compatibility with older versions of pip-accel which is why there is now "suboptimal" support for editable installations (I say "suboptimal" because it works fine for some use cases but rather badly for other use cases, as you found out in #71).
Analysis: I'm aware of this behavior and the problems it causes (as you describe) and would love to change it but don't really know how to, given the way that pip and pip-accel integrate. I actually (fairly frequently) hit the same annoyance that you describe in your last sentence when I install an editable checkout of one of my projects and pip-accel causes pip to process a multi-hundred-megabyte .tox directory that's located inside the checkout. In other words, I feel your frustration and have previously investigated whether it's possible to fix this rather annoying consequence of how pip and pip-accel integrate (unfortunately I didn't come very far, or I would have fixed it already :-)).
Your previous bug report (#71) and this one share the same root cause, however I understand why you reported this as a separate issue; users of pip-accel shouldn't need to know about the internals of the tool at all. I intend to investigate if the behavior described in #71 can be improved and that will likely touch the same root cause and so possibly fix this issue as well.
Short term workarounds: In the short term all I can do is suggest a couple of temporary workarounds that may or may not be satisfactory for you, depending on your use case (which I am not familiar with):
Assuming you have access to a requirements file that lists the dependencies of the package/project that you're working on, you can pip-accel install --requirement $REQ_FILE and then pip install --editable $CHECKOUT to cache your dependencies while sidestepping the issues described here and in #71. Of course this requires that 1) the project provides a requirements file and 2) you have access to the requirements file.
If the project doesn't provide a requirements file but you have write access to or even control over the project, you may consider adding a requirements file and modifying the setup.py script to fill in the install_requires argument based on the requirements file (to avoid duplication).
Of course I completely understand if these workarounds are going rather far and you may as well consider abandoning your efforts to utilize pip-accel; I'm just trying to think of short term workarounds, I'm not trying to convince you to apply them.

I am actually already installing a requirements.txt file first with pip-accel, then doing pip install -e ., just as you describe. However, the problem is that the requirements.txt file is not always complete (sometimes even empty) or up to date, and in those cases (e.g. when starting a new project from a template), all dependencies or all outdated dependencies will be installed by pip, slowly.
Usually, this won't be a big deal. One person will create a project and install unpinned dependencies once, then pin them. But it means we have to make sure that everyone on the team who ever makes a project from the template remembers to pin dependencies sooner than later, and it otherwise wouldn't normally happen until the project was closer to an initial release.
But pip-accel is still definitely useful when the requirements.txt file is up to date.

Thanks for the reply and good to hear about your use case. Your use of pip-accel can still be beneficial if the majority of the requirements are already part of the requirements file, but I can see how this is not exactly ideal during development :-).

The root cause of all this (how pip and pip-accel integrate)
I was writing a follow up with more details when I received your reply, so I'll just post it here now:
In issue #71 and this issue I keep mentioning "the way that pip and pip-accel integrate" and how this makes it hard to fix these issues related to editable installations. I thought it might be useful for you and others reading along to get an impression of what it is that actually causes these issues:

pip-accel uses pip to transform the command line arguments into a requirement set of unpacked source and/or wheel distributions, preferably without connecting to the internet (using the --no-index option) but if needed (when some dependencies need to be downloaded) falling back to a rerun without the --no-index option. I don't see how I could move away from this approach without (badly) re-implementing features like the parsing of requirements files, fetching of distribution archives from PyPI, resolving conflicts in the requirement set, etc. That way lies madness :-).

pip-accel utilizes the pip install --download (in pip 7.x) and pip download (in pip 8.x, refer to the not yet released pip-8.1-upgrade branch) commands to implement both aspects of step one (unpacking downloaded distributions and downloading missing distributions).

When you think about it, the command pip install --download=$LOCAL_INDEX --editable $CHECKOUT doesn't really make sense at all. Nevertheless pip attempts to do something useful, which in this case means to copy $CHECKOUT to a temporary directory (this is part of what's so slow, it's copying your SQL files and my .tox directory trees) and pack that into a ZIP archive.

Once pip-accel receives the requirement set with unpacked distribution archives from pip, it realizes that one of the requirements concerns an editable installation and so it side steps the binary distribution caching mechanism for that requirement and uses pip to perform an editable install of that distribution.

At step four pip has already cached a source distribution archive for the project that was to be installed in editable mode so even though pip-accel doesn't use that cached source distribution, the time to create it was still wasted :-(.

Brainstorming about possible solutions
I'm tempted to document my brainstorming about how to improve this situation :-). The command line arguments are passed to pip by pip-accel which means pip-accel can know whether an editable installation was requested before that request is passed to pip. If the editable installation comes from a local directory (e.g. an existing checkout) then I could modify pip-accel to do something like this:

Find a way to get the install_requires value(s) from the setup.py script without asking pip to "download" the local directory (i.e. create a useless source distribution archive).

Replace the --editable ... argument in the command line with the install_requires value(s) obtained by introspecting the setup.py script.

Continue as usual until all requirements have been installed.

Use pip to install the editable requirement with the --editable and --no-deps options.

This approach would work when the $ARGUMENT in pip-accel install --editable $ARGUMENT refers to a local directory, but of course none of this is going to work if $ARGUMENT refers to e.g. a URL on GitHub. What's worse is that requirement files can also contain lines of the form --editable ... and pip-accel will never see these until pip has already processed the request to "download" these editable requirements and it's too late to avoid the "bad interactions". This is what happens every time I try to come up with workarounds for this: I undermine my conceptual workarounds by finding new corner cases and realizing I might never be able to make this Just Work (TM) for users of pip-accel...


Double quotes from Discord aren't escaped

Exactly what it says in the title. Any message from Discord with double quotes fails because "text"text"text" is not a valid string in json.

Can you test with the fix merged recently?

Yup, seems to be fixed by #51!


Support for closures

Here are a few tests that we expect to fail, because we do not currently implement closures like ECMAScript does:
src/afw/tests/test262/statements/try.as:

scope-catch-block-lex-close
scope-catch-block-var-none
scope-catch-param-lex-close

Adaptive Script does not hoist 'let' and 'const' variables to the top of a block like ECMAScript does. Although TypeScript also supports hoisting, it will give you an error if a variable is accessed before it's declared, just like Adaptive Script.
If we decided to support hoisting, it will require an Adaptive Compiler rewrite and will involve switching from a 1 pass compile to a 2 pass.

src/afw/tests/test262/identifier-resolution.as S10.2.2_A1_T4 fails because the 'let x = 1' is in source after referenced in f2(). This can be fixed by putting 'let x;' before the function f2() line.

let x = 0;

function f1(){
  let x;
  function f2(){
    return x;
  };

  x = 1;
  return f2();
}

if(!(f1() === 1)){
  throw "#1: Scope chain disturbed";
}

src/afw/tests/test262/statements/switch.as scope-lex-close-case expects to fail, but with closure in place it should expect to succeed..

src/afw/tests/test262/statements/try.as scope-catch-block-lex-close expects to fail, but with closure in place it should expect to succeed..

src/afw/tests/test262/statements/switch.as scope-lex-close-dflt is failing because the assert() has 3 parameters.

@JeremyGrieshop I changed the 4 tests mentioned above so they will pass. Please check them to make sure they are what you want. I'm getting ready to do a pull request and merge the 'scope' branch into 'develop'.


[ENH] Split validation and conversion in utils

Reference Issues/PRs
dependent on #1129
Splits the functionality of validation/checking and conversion into two separate packages in utils, tidies up file naming, moves test example generation to data_gen. Imports could possibly be tidier, still some possibly unnecessary private imports.

_convert_collection and collection in validation moved to conversion, validation functionality moved out into _check_collection, rest merged into one function in _convert_collection.

This is just a refactor and the functionality is the same right? Bit hard to tell from the diff if anything is new or removed.

yes its just creating a validation and checking package and moving functionality between.


bug: possible miscompilation of switch in 0.8.4

Example code:
 import Prim "mo:⛔";

 actor {
   func f() {};

   func g(name : Text) : () -> () {
     switch (name) {
       case "some" f;
       case _ Prim.trap("Not found");
     };
   };

   public func init(name : Text) {
     g(name)();
   };
 };

Using a return keyword fixes it:
      case ("some") return f;

There is a difference between 'case ("some") f;' and 'case ("some") return f; '  (for 0.8.4).
The error message is "Canister v32cj-3iaaa-aaaaa-aaa2a-cai violated contract: function invocation does not match its signature". Does that mean that without the return keyword it tries to invoke f instead of returning f?
Reported by @timohanke  fyi @nomeata

I can reproduce the behaviour
https://m7sm4-2iaaa-aaaab-qabra-cai.ic0.app/?tag=1621584345

Happens with if analogously:
if (name == "some") f else Prim.trap("Not found");

This might be confusion in the backend if a const thing meets a None thing. Let me see.

Can reproduce in drun. Very weird, stuff like existence of unrelated public method make a difference. Maybe some confusion with table indices.

Here is a reproducer using wasmtime:
import Prim "mo:⛔";
func f() {};
func g() : () -> () {
  if (true) f else Prim.trap("Not found");
};
func go1() { g()(); };
go1();

Incidentially, the problem goes away on master, due to b0230e41f2a0c8de568a262ed7d83bee12581235 in particular.
So we could consider this fixed if we want :-)

That's great, I'll prepare a regression test from this.

Here is the effect of that commit, as per
./compare-wat.sh -f b0230e41f2a0c8de568a262ed7d83bee12581235^ -t b0230e41f2a0c8de568a262ed7d83bee12581235 run/issue3887.mo

(a useful tool!)
   (func $g (type 14) (param $clos i32) (param $b i32) (result i32)
     local.get $b
-    if  ;; label = @1
+    if (result i32)  ;; label = @1
+      i32.const 2097287
     else
       i32.const 2097267
       call $blob_of_text
       call $as_ptr_size
       global.get 17
       global.get 18
       call 31
       unreachable
-    end
-    i32.const 0)
+    end)

I should stare at the code if my change really fixes the problem, or just made it disappear in this case.

Found it. The following code reproduces it on master:
import Prim "mo:⛔";
func f() : () {};
(if (true) f else Prim.trap("Not found"))();

And this is the fix:
@@ -9504,7 +9511,7 @@ and compile_prim_invocation (env : E.t) ae p es at =
     IC.performance_counter env

   | OtherPrim "trap", [e] ->
-    SR.unit,
+    SR.Unreachable,
     compile_exp_vanilla env ae e ^^
     IC.trap_text env

PR coming up.


[Gateway] - Use standardized token-lists

Issue Type

[ ] bug report
[x] feature request

Current Behavior
We use a non-standardized version of tokenlists on the gateway
Expected Behavior
We could switch to use the tokenlist standard followed by tokenlists.org/Uniswap, which is already being maintained here: bobanetwork/token-list
For cases, where the gateway would need extra information on each tokens - the tokenlist can be extended with custom 'extensions'
Steps to Reproduce
/
Suggested Fix
/

We have this token list for the gateway. https://github.com/bobanetwork/boba/blob/develop/packages/boba/register/addresses/tokenInfo.json
Do we want to switch to bobanetwork/token-list?


helm 3 support

The current dockerfile use helm version 2 .... would be gr8 to have an image with helm 3 .

The Dockerfile has been updated.


status: fix warning for unused doc comment

rustdoc does not generate documentation for expression fields

Thanks.  In this github org you'll need to sign off on commits.