fix: :bug: fix the number of pages cutPDF keep

PR Details
fix the number of pages cutPDF keep
Description
fix the number of pages cutPDF keep
Related Issue

Motivation and Context

How Has This Been Tested

Types of changes

[ ] Docs change / refactoring / dependency upgrade
[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[X] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[X] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] I have added type hints (flow) to cover my changes.
[X] All new and existing tests passed.

Please squash this PR 🤦♂

@fharper see #87


error

hi im getting error all

i m fist in python can u plz help with this? how to set path?

try to attach the error instead of the code. That would be more helpful.


utilize phoenix 6 cancoder faults

throwing some code together to give an indication when a cancoder has been enabled right after bootup

wow y'all really buried that CANcoder class in there
project for tmrw night I'm gonna sleep


Improve image uploading

Some requests (can consider breaking up into multiple issues):

[x] Fix actually uploading to the S3 bucket
[x] Fix MIME type for images uploaded to S3
[ ] Avoid abandoned images (i.e., images uploaded before the robot is ready)
[ ] Allow uploading JPEG files as well, not just PNGs
[ ] Show a frontend UI to crop the image before uploading it
[ ] Increase image upload size to something like 25 MB and crop later on

Also, it seems that uploading the image doesn't work in prod. Maybe there is an issue with the S3 configuration? Here is what I see:

Avoid abandoned images (i.e., images uploaded before the robot is ready)

Is the idea to upload the images to server (i.e. call the appropriate API route) upon submission of whole robot rather than upload of the image? I think this is probably the correct answer, fwiw.
Also it may be beneficial to also support uploading links for images. Now, this does open the website up to external images disappearing on us, but I'm not sure this is that big a deal.

Oh we should add some drag-and-drop image thing, that would be cool

@codekansas let me add drag n drop for image upload

@codekansas can you please explain the meaning of abandoned images?
should we upload the image when the form is submitted?

can you please explain the meaning of abandoned images?

basically the user uploads an image, but it is not linked to a robot, so the image sits around in s3

can you please explain the meaning of abandoned images?

basically the user uploads an image, but it is not linked to a robot, so the image sits around in s3

If you're concerned about this: what about when a user edits an image? Or deletes it from the robot? Ought we remove it from s3?

@codekansas do you mean deleting images uploaded as part of creating a robot or a part?

What if that image is still associated with the robot or part in the image management part of the user?
We can choose to delete the image in the Robot or Part page when user changes the image or cancel creating the Robot or Part?

Let me know what you think?

mostly addressed with refactors


[𝕎𝚊𝚝𝚌𝚑✓VIDEO𝕊.!!] El Siri Viral ᴠɪᴅᴇᴏ ʟᴇᴀᴋᴇᴅ Update On Social Media X Telegram

18 sec ago El Siri Viral ʟᴇᴀᴋᴇᴅ ᴠɪᴅᴇᴏ Viral On Social Media. [𝕎𝚊𝚝𝚌𝚑✓VIDEO𝕊.!!] El Siri Viral ᴠɪᴅᴇᴏ ʟᴇᴀᴋᴇᴅ Update On Social Media X Telegram
𝑾𝒂𝒕𝒄𝒉 🟢 ➤ ➤ ➤ 🌐 𝖢𝗅𝗂𝖼𝗄 𝖧𝖾𝗋𝖾 𝖳𝗈 𝗅𝗂𝗇𝗄 (𝖥𝗎𝗅 𝗅 𝖵𝗂𝗋𝖺𝗅 𝖵𝗂𝖽𝖾𝗈 𝖫𝗂𝗇𝗄)
🔴 ➤► 𝖣𝖮𝖶𝖭𝖫𝖮𝖠𝖣👉👉 (𝖥𝗎𝗅𝗅 𝖵𝗂𝗋𝖺𝗅 𝖵𝗂𝖽𝖾𝗈 𝖫𝗂𝗇 𝗄) 𝖶𝖺𝗍𝖼𝗁
🟢 ➤ ➤ ➤ 🌐 𝖢𝗅𝗂𝖼𝗄 𝖧𝖾𝗋𝖾 𝖳𝗈 𝗅𝗂𝗇𝗄 (𝖥𝗎𝗅𝗅 𝖵𝗂𝗋𝖺 𝗅 𝖵𝗂𝖽𝖾𝗈 𝖫𝗂𝗇𝗄)
🔴 ➤► 𝖣𝖮𝖶𝖭𝖫𝖮𝖠𝖣👉👉 (𝖥𝗎𝗅𝗅 𝖵𝗂𝗋𝖺𝗅 𝖵𝗂𝖽𝖾𝗈 𝖫𝗂𝗇 𝗄)
El Siri Viral's latest video has taken the internet by storm, sparking lively discussions across various social media platforms. The rising digital influencer, known for her captivating content—especially her recent Spiderman-themed video—has experienced a massive surge in popularity. Fans are eagerly awaiting the full version, while audiences flock to Instagram, X (formerly Twitter), and Discord to engage with her latest creation.
As excitement continues to grow, fans are actively sharing their thoughts online, highlighting the crucial role social media plays in propelling creators like El Siri Viral into the spotlight. With her unique style and creative approach, she has made a significant impact on the digital scene, leaving fans excited for what’s to come next.  The rising digital influencer, known for her captivating content—especially her recent Spiderman-themed video—has experienced a massive surge in popularity. Fans are eagerly awaiting the full version, while audiences flock to Instagram, X (formerly Twitter), and Discord to engage with her latest creation.
As excitement continues to grow, fans are actively sharing their thoughts online, highlighting the crucial role social media plays in propelling creators like El Siri Viral into the spotlight. With her unique style and creative approach, she has made a significant impact on the digital scene, leaving fans excited for what’s to come next.


Report #3 of same issue- Starting view hits API limits

Description
Oftentimes when selecting a task from Map Roulette the map starts really zoomed out.
Instead of the screen saying "Zoom in to Edit", it downloads everything in view until I hit my API limit.
This makes using Rapid impossible
Screenshots

Version
2.3.1
What browser are you seeing the problem on? What version are you running?
Chrome v126.0
The OS you're using
mac
Steps to reproduce

Go to Map Roulette
Open a task

The browser URL at the time you encountered the bug
https://rapideditor.org/edit#map=2.00/0.0/30.2&background=Bing&datasets=fbRoads,msBuildings&disable_features=boundaries
The auto-detected useragent string for your browser (leave blank if you're manually filling this form out)
Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36

If it's the same issue can you link to the other ones?  It helps us keep the issue list organized.

@sicknacco has filed #1453 and #1391 on this in the past. This problem isn't going to get any better with the OSM Api limiting being even tighter these days.

Closing and tracking in #557. Some more aggressive filtering at low zoom levels would be a huge performance win.


Init noita-entangled-worlds 0.32.5

Things done
Tested and working great on NixOS.

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@NixOS/nixpkgs-merge-bot merge

Why not build from source?

great question.
I had trouble implementing building from source and found it easier to just grab the release.
I can try again later tonight and see if I can get it working. I am pretty new at building a nix pkg.

Looks good, but there are two stray commits in here. Clean those out and you've got my approval.

Looks good, but there are two stray commits in here. Clean those out and you've got my approval.

must have been a rebase issue. Removed

ofborg doesn't build the package since steam-unwrapped is an unfreeRedistributable

Why not build from source?

great question. I had trouble implementing building from source and found it easier to just grab the release. I can try again later tonight and see if I can get it working. I am pretty new at building a nix pkg.

If you do want to build it from source, here is my version of the package that builds it from source:
https://github.com/NixOS/nixpkgs/pull/368989/files
Not quite sure how to suggest these changes on the pull directly so I will just leave this link here.


license question

Just wondering what license this library falls under, as there's none declared (seems like that's missing from your other repos as well) – could you please clarify? If you don't care how it's being used, you could pick and declare a lax one like Unlicense or WTFPL :wink:

Added MIT

Thanks a lot!


Add tagsetutils review

Here are some changes that I would like to suggest for your PR.

Merged locally using cherry picking.


Define Q1 goals

Let me write a few lines on what was achieved in Q1 from the goals above.
O1: Successfully kick-off bounties

A few bounties were identified mainly on improving documentation. We also had one bounty completed.

More info below:
https://github.com/maplibre/maplibre-native/issues?q=is%3Aissue+label%3A"💰+bounty+S"+
O2: Maintain/support MapLibre native on both Android and iOS

repository splitting plan finally took shape thanks to @ntadej initiative. He started with Qt platform. More details here: https://github.com/maplibre/maplibre-native-qt
Reorg/modularisation project is ongoing. More details here: https://github.com/orgs/maplibre/projects/8/views/3
PR's were usually triaged/reviewed in a two days timeframe

O3: Improve code contribution process

Some good steps were done in CI area in Q1. This will be a continuous work, especially with the reorg.metal project being in progress.
Guidelines update for PR's was more like a false positive issue. We think that the way things are now are manageable. We may revisit this later in the year if needed.


is_private set to true for channel messages.

Channel messages are flagged as if they are direct messages.
is_private is set to 1
type is set to 1
Test code:
expected behavior: bot will die on a direct message.
actual behavor: bot will die on any channel message
<?php
/*!
 * \file
 * \brief is_private test code.
 * \date 2016-09-13
 * \version 0.1
 */

include __DIR__.'/vendor/autoload.php';

$discord = new \Discord\Discord([
    'token' => TOKEN_HERE,
]);

$discord->on('ready', function ($discord) {

    echo "Bot is ready.", PHP_EOL;

    // Listen for events here
    $discord->on('message', function ($message) {
# Accept private messages only
        if ($message->channel->is_private === TRUE) {
                echo "Recieved a private message from {$message->author->username}: {$message->content}", PHP_EOL;
                print_r($message);
                print_r($message->channel);
                print_r($message->channel->recipients);

                die();
        }
    });
});

$discord->run();

ran it again without the === TRUE, same result.

A valid work-around is to cycle through the guilds/channels to see if there's a matching channel id as the incoming message.
                global $discord;
                $guildRepository = $discord->guilds;
                $gi = $guildRepository->getIterator();
                while ($gi->valid()) {
                        $go = $gi->current();
                        $channelRepository = $go->channels;
                        $ci = $channelRepository->getIterator();
                        while ($ci->valid()) {
                                print_r($ci->current());
                                if ($ci->current()['id'] == $message->channel->id) {
                                        die('This is a message from a channel.');
                                }
                                $ci->next();
                        }
                        $gi->next();
                }

Some more weirdness. I'm running the code in two different environments, one has this issue, the other doesn't.
No problems: PHP 5.6.24-0+deb8u1 on Debian.
Problems: PHP 7 (whatever current version) on Ubuntu.

I have found the issue, it lies inside of a function in Discord\Parts\Channel\Message.php, that function is `public function getChannelAttribute()
{
foreach ($this->discord->guilds as $guild) {
if ($guild->channels->has($this->channel_id)) {
return $guild->channels->get('id', $this->channel_id);
}
}
    if ($this->cache->has("pm_channels.{$this->channel_id}")) {
        return $this->cache->get("pm_channels.{$this->channel_id}");
    }

    return $this->factory->create(Channel::class, [
        'id'   => $this->channel_id,
        'type' => Channel::TYPE_DM,
    ], true);
}`

Inside the function, the if statement:
if ($guild->channels->has($this->channel_id)) {                 return $guild->channels->get('id', $this->channel_id);             }
always results in false for some reason, so changing it to:
if (!empty($guild->channels->get('id', $this->channel_id)) {                 return $guild->channels->get('id', $this->channel_id);             }
fixed the issue entirely, allowing $message->channel->is_private to finally have false inside of it, along with the entire channel elements inside of it and the guild. No more workarounds-ish.

Running the following code on the develop branch works for me:
$discord->on('ready', function ($discord) {
    echo 'Ready';

    $discord->on('message', function ($message) {
        if ($message->channel->is_private == true) {
            echo "Received a private message from {$message->author->username}: {$message->content}", PHP_EOL;
        }
    });
});

Closing due to age.


PHP 7.1.8 or higher requirement for nesbot/carbon

Should fix Scrutinizer not setting up tests

A new inspection was created.

No it does not.

The inspection completed: 3 new issues


App Store is rejecting App with Widget Extension

After adding the iOS widget extension, during iOS distribution I received an Invalid Binary response with the following error.
ITMS-90853: Invalid font - There was an error validating the font named 'Noto Sans KR Bold' at 'Runner.app/Frameworks/App.framework/flutter_assets/assets/fonts/NotoSansKR/NotoSansKR-Bold-Hestia.ttf'. ‘name’ table structure.

ITMS-90853: Invalid font - There was an error validating the font named 'Noto Sans KR Regular' at 'Runner.app/Frameworks/App.framework/flutter_assets/assets/fonts/NotoSansKR/NotoSansKR-Regular-Hestia.ttf'. ‘name’ table usability.

It was confirmed that the issues of similar cases were closed.
The flutter/bin/internal/material_fonts.version file is written as below.
flutter_infra_release/flutter/fonts/bd151aa3c2f7231344411a01dba4ef61b3cd56b2/fonts.zip

I did not change it, and after deleting flutter>bin>cache>artifacts>material_fonts directory, I ran flutter clean && flutter pub get and redeployed it, but I got the same error response.
I'm not sure what to do. I need help.

flutter doctor -v

[✓] Flutter (Channel unknown, 2.2.3, on macOS 11.3.1 20E241 darwin-x64, locale ko-KR)
    • Flutter version 2.2.3 at /Users/nero/Project/FlutterContribution/flutter
    • Framework revision f4abaa0735 (5 months ago), 2021-07-01 12:46:11 -0700
    • Engine revision 241c87ad80
    • Dart version 2.13.4

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/nero/Library/Android/sdk
    • Platform android-30, build-tools 31.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 13.1, Build version 13A1030d
    • CocoaPods version 1.10.1

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

[✓] Android Studio (version 3.6)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 44.0.2
    • Dart plugin version 192.7761
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)

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

[✓] Connected device (1 available)
    • Chrome (web) • chrome • web-javascript • Google Chrome 96.0.4664.55

• No issues found!

@nero-angela
Can you try to upgrade to latest stable or master and see if you still get same error ?
Also, are there any more instances of flutter_infra/flutter/fonts/ that you can search for ?
Furthermore, show us the fonts you are using Noto Sans KR Bold in pubspec.yaml. Could it be due to the spaces in the names ?

@darshankawar
1. try with latest stable version
I tested with flutter 2.5.3 but It also has same problem.
flutter  docter -v
[✓] Flutter (Channel unknown, 2.5.3, on macOS 11.3.1 20E241 darwin-x64, locale ko-KR)
    • Flutter version 2.5.3 at /Users/nero/Project/FlutterContribution/flutter
    • Upstream repository unknown
    • Framework revision 18116933e7 (7 weeks ago), 2021-10-15 10:46:35 -0700
    • Engine revision d3ea636dc5
    • Dart version 2.14.4

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/nero/Library/Android/sdk
    • Platform android-30, build-tools 31.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 13.1, Build version 13A1030d
    • CocoaPods version 1.10.1

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

[✓] Android Studio (version 3.6)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 44.0.2
    • Dart plugin version 192.7761
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)

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

[✓] Connected device (2 available)
    • iPhone 8 (mobile) • 1A3A1310-C03B-4D9E-A220-9138A6810F00 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-15-0 (simulator)
    • Chrome (web)      • chrome                               • web-javascript • Google Chrome 96.0.4664.55

• No issues found!

iOS response
TMS-90853: Invalid font - There was an error validating the font named 'Noto Sans KR Bold' at 'Runner.app/Frameworks/App.framework/flutter_assets/assets/fonts/NotoSansKR/NotoSansKR-Bold-Hestia.ttf'. ‘name’ table usability.

ITMS-90853: Invalid font - There was an error validating the font named 'Noto Sans KR Regular' at 'Runner.app/Frameworks/App.framework/flutter_assets/assets/fonts/NotoSansKR/NotoSansKR-Regular-Hestia.ttf'. ‘name’ table structure.

ITMS-90853: Invalid font - There was an error validating the font named 'Noto Sans KR Bold' at 'Runner.app/Frameworks/App.framework/flutter_assets/assets/fonts/NotoSansKR/NotoSansKR-Bold-Hestia.ttf'. ‘name’ table structure.

ITMS-90853: Invalid font - There was an error validating the font named 'Noto Sans KR Medium' at 'Runner.app/Frameworks/App.framework/flutter_assets/assets/fonts/NotoSansKR/NotoSansKR-Medium-Hestia.ttf'. ‘name’ table structure.

ITMS-90853: Invalid font - There was an error validating the font named 'Noto Sans KR Regular' at 'Runner.app/Frameworks/App.framework/flutter_assets/assets/fonts/NotoSansKR/NotoSansKR-Regular-Hestia.ttf'. ‘name’ table usability.

ITMS-90853: Invalid font - There was an error validating the font named 'Noto Sans KR Medium' at 'Runner.app/Frameworks/App.framework/flutter_assets/assets/fonts/NotoSansKR/NotoSansKR-Medium-Hestia.ttf'. ‘name’ table usability.

2. are there any more instances of flutter_infra/flutter/fonts/
I'm not sure what an instance of `flutter_infra/flutter/fonts/' is and where to find it.
3. pubspec.yaml fonts
The current font is being used without any problem in the previously distributed app, and this problem occurs only after adding the ios widget extension.
flutter:
  uses-material-design: true
  fonts:
    - family: NotoSansKR
      fonts:
        - asset: assets/fonts/NotoSansKR/NotoSansKR-Bold-Hestia.ttf
          weight: 700
        - asset: assets/fonts/NotoSansKR/NotoSansKR-Medium-Hestia.ttf
          weight: 500
        - asset: assets/fonts/NotoSansKR/NotoSansKR-Regular-Hestia.ttf
          weight: 300
    - family: Parata
      fonts:
        - asset: assets/fonts/Prata-Regular.ttf
    - family: Arita
      fonts:
        - asset: assets/fonts/Arita-buriSB.ttf

Thanks for the update. Keeping it open for further insights from the team.
Similar issue : https://github.com/flutter/flutter/issues/69510
/cc @willlarche

This is an issue with the font and not the framework. I've had success in the past reproducing App Store font validation errors using macOS' built-in FontBook app (File  → Validate File and Validate Font). I suggest re-downloading the third-party font or using the official supported version from fonts.google.com.

This is an issue with the font and not the framework. I've had success in the past reproducing App Store font validation errors using macOS' built-in FontBook app (File → Validate File and Validate Font). I suggest re-downloading the third-party font or using the official supported version from fonts.google.com.

When I changed to the NotoSansKR font of fonts.google, the distribution was completed safely.
thank you for saving my time. :)


Application keeps randomly hanging after adding SynEdit with syntax highlighting

Ever since implementing SynEdit with syntax highlighting, the application randomly hangs for seconds to minutes at a time, sometimes never recovering.

Sometimes, it doesn't even show "Not Responding" in the title. And the actions leading up to it are random - from typing a couple lines of code, to maximizing the window.

Went hours without the problem. Now the problem is constant. So much as adding a new line break freezes the app.

Didn't realize adding the word "fix" automatically closes the issue.
I've ATTEMPTED a fix, and needs testing. I suspect that having two different TSynEdit controls on the same form is the culprit.


Error message is still not in bottom of textfield.

I have installed using the pod. Have seen the example too. But the Error line message still seems at the top right & not at bottom of the text field.
In the description area, I read, what's new? the section, where it's mentioned that error message, can be shown at the bottom.
Can anyone help me with this?

Please checkout master branch directly and drag the file it may be in master branch code don’t remember exactly but if its in swift code you can also use that in objective c using bridge. Thanks
Sent from my iPhone

On 04-Dec-2020, at 5:12 PM, khushabu29<EMAIL_ADDRESS>wrote:

I have installed using the pod. Have seen the example too. But the Error line message still seems at the top right & not at bottom of the text field.
In the description area, I read, what's new? the section, where it's mentioned that error message, can be shown at the bottom.
Can anyone help me with this?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Actually, I have checkout the master branch only.. attaching a screenshot of code in the master branch for the error label..


Updates needed to recover from my phone and domain being hacked and my business accounts and credit cards  is not working I think they were transferred to someone else

Please do not report security vulnerabilities here. The Responsible Disclosure Program details the procedure for disclosing security issues
Thank you in advance for helping us to improve this library! Please read through the template below and answer all relevant questions. Your additional work here is greatly appreciates and will help us respond ASAP quickly as possible. For general support or usage questions, use the [Authorization tCommunity (https://community.auth0.com/) or Auth0 _Support. Finally, to avoid duplicates, please search existing Issues before submitting one here.
By submitting an Issue to this repository, you agree to the terms within the Auth0 Code of Conduct.
Description

Provide a clear and concise description of the issue, including what you expected to happen.

Reproduction

Detail the steps taken to reproduce this error, what was expected, and whether this issue can be reproduced consistently or if it is intermittent.
Where applicable, please include:

Code sample to reproduce the issue

Log files (redact/remove sensitive - [ ] information)~~~~~~~~
Application settings (redact/reove # sensitive information) ℹ~~~~

[ ] > - Screenshots
[x] devices are online~~![Screenshot_20240312_000314_One UI Home.jpg](https://github.com/auth0/auth0-log-schemas/assets/154641628/c19efcf3-2c16- # - [ ] -418d-a02d-019d0c241a04)
@patrickkang
~~

Environment

Please provide the following:

Version of this library used:
Version of the platform or framework used, if applicable:
Other relevant versions (language, server software, OS, browser):
Other modules/plugins/libraries that might be involved:
devices
<EMAIL_ADDRESS><EMAIL_ADDRESS>Jjlmedia@lzychowski
Jjlmedia@patrickkang
jjlmedia@stacybird
jjlmedia@beckitrue

Send information to me at<EMAIL_ADDRESS>


Add customer logging support

The change add customer logging support for cri container in windows platform. it extend the existing log_path property of the container definition. and make it support binary protocal (containerd have similar impementaion in linux plat).
for example now you can define in your contianerdef.json file
"log_path": "binary:d:\gosrc\bin\logger.exe"
at the time the container get create, the logger is launched automatically. today we use a fixed protocal for the logger, it will require following parameters
[stdout-pipename] [stderr-pipename] [liftime-signal-filename] [containerid] [labels]
the two named pipe is require to be host to receive the log pull out from hcsshim. and lifetime-signal] file is required to be create by the logger, and the logger then take responsible to watch the file, once the containerd shutdownt he container, it will delete the file, and logger take responsible to shutdown at that time. containerd will wait at most 10s to wait for logger shutdown, if not, it will kill the logger.
the other two parmeter is used to pass information into the customer logger.

@ambarve @katiewasnothere @dcantah FYI
Overall I'm supportive of us exposing logging binary support using the CRI log path fields. I think that is a fine approach to take given the forked nature of our code base from upstream CRI.
However, I've been reading through the original pluggable logging PR in containerd (containerd/containerd#3085), and I think we will need to make some changes to align with the approach taken there.

The shim process should invoke the logging binary, rather than cri. This will require changes to Microsoft/hcsshim to support logging URIs and specifically logging binary support.
The logging binary support package (github.com/containerd/containerd/runtime/v2/logging) does not currently support Windows. It will need some changes to accommodate this, such as a different way to pass files to the logging binary process, and different signal handling.

i think we need to calrify in wihch layer we want to expose such ability. from ctr layer, there is some customer logger tech implemented, but this change implement this in cri layer.  if we want to make cri layer directly depend on ctr layer then we need to implement windows part for ctr. however, the problem is the protocal for linux /windows can't be same unlucky.
another problem is ctr is the parent of the logger which mean if we want to keep ctr support customer logger for windows, we will have to leave ctr be the owner of logger, but the hcsshim or runner

let's have async offline to clarify this. i think we need to define a goal here. basically, with this change, it introduce the customer logging for CRI layer. so it looks like similar for the customer logging in CTR layer but it's different. we can have two different approach.

Keep the cri layer customer logging be indepdent with the CTR layer. in this case,  as all the change I have now is in CRI layer so it's there. I can move the shim become the parent of logger but then it mean it can't keep consistant with linux implementation
Keep the cri layer depend on ctr layer customer logging. this we will need to implement the ctr customer logging in containerd first and then we can use it. in addtional, we will have to make the containerd be the parent of the logger.


[Scope Change]  IT Localization - Codeunit 12152 SubcontractingManagement - procedure DelLocationLinkedComponents

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


How to make grPC Client to Establish connection based on IP address?

Hi @Tradias:
I'm attempting to execute the client from a remote machine. However, despite setting the Server IP in the provided code (below) and ensuring it's configured with the 'InsecureChannelCredentials' option, the connection isn't established. I also observed that your "helloworld" client is configured exclusively with "localhost".
    const auto host = "<IP>:" + cfg.getGrpcPort();
    grpc::Status status;
    helloworld::Greeter::Stub stub{grpc::CreateChannel(host, grpc::InsecureChannelCredentials())};

Could you kindly direct me to a sample client code that connects to a server without SSL enabled or with minimal settings regarding SSL, specifically utilizing the ChannelCredentials option? This is one of my limitations in testing the framework in a real production use case...
Thanks for your help!
Thanks,
Tharma

Hi, I personally only ever used hostname based address, for example in docker-compose or k8s deployments. I did however have issues with IP address clients on Windows (e.g. using <IP_ADDRESS>) in the past but not anymore (not sure when it changed but gRPC 1.51.1+ seems to work). Again this does not look like an asio-grpc question :), if you continue to have issues you might want to open an issue on gRPC.
You can also check the state of the channel before starting an RPC and asking it to establish a connection to the server by using:
    const auto channel = grpc::CreateChannel(host, grpc::InsecureChannelCredentials());
    const auto state = channel->GetState(true);
    while (GRPC_CHANNEL_READY != state && GRPC_CHANNEL_TRANSIENT_FAILURE != state) {
    const auto deadline = std::chrono::system_clock::now() + std::chrono::seconds(5);
    bool has_state_changed =
        co_await agrpc::notify_on_state_change(grpc_context, *channel, state, deadline, asio::use_awaitable);
    }

Thank you for the quick response, @Tradias. This is helpful. Sure, will check with the gRPC team.
Regards,
Tharma


SLF4J Exception Handling

Ensure that the stack trace is logged when the last argument to log.error is Throwable.
The current example from SLF4J does not print a stack trace when logging an exception unless there are no parameterized arguments.
http://www.slf4j.org/faq.html#paramException
Fails:
String s = "Hello world";
try {
  Integer i = Integer.valueOf(s);
} catch (NumberFormatException e) {
  logger.error("Failed to format {}", s, e);
}

Works
String s = "Hello world";
try {
  Integer i = Integer.valueOf(s);
} catch (NumberFormatException e) {
  logger.error("Failed to format", e);
}

Thank you! Looks great :) Could you declare getLastElementIfThrowable() as private method and move to the end of the file? If you like, you can merge it to branch v1.1 too.

I made the changes you requested in this PR, as for merging into branch v1.1 should I create go ahead and replay this commit in that branch and create a separate pull request for it?


Rebuild with updated yggdrasil-extras

yggdrasil-network/yggdrasil-extras@b91e022 contains fixes present in yggdrasil-develop-0.3.15-0087 so that IPv6 internet traffic that is erroneously sent to the dummy adapter will generate an ICMPv6 Destination Unreachable response. This will hopefully mean that running Yggdrasil will no longer cause some applications to incorrectly hang for some time trying IPv6 internet connectivity when there is none.
Please update the Android build accordingly by generating a new .aar:
cd /path/to/yggdrasil-go
git checkout develop
git pull origin develop
go get github.com/yggdrasil-network/yggdrasil-extras@b91e022
./build -a

Done. Check v 1.6 (14) from https://t.me/yggdrasil_android .

This works. MeWe doesn't have anymore problems.

This works. MeWe doesn't have anymore problems.


Cant get to_connection to work properly

arch = [
    to_head( '..' ),
    to_cor(),
    to_begin(),
    to_Conv("conv1", 320, 8, offset="(0,0,0)", to="(0,0,0)", height=64, depth=64, width=8),
    to_Pool("pool1", offset="(0,0,0)", to="(conv1-east)", height=60, depth=60, width=1),

    to_Conv("conv2", 160, 16, offset="(1,0,0)", to="(pool1-east)", height=32, depth=32, width=16),
    to_Pool("pool2", offset="(0,0,0)", to="(conv2-east)", height=30, depth=30, width=1),

    to_Conv("conv3", 80, 32, offset="(0,0,0)", to="(pool2-east)", height=16, depth=16, width=32),
    to_Pool("pool3", offset="(0,0,0)", to="(conv3-east)", height=1, depth=1, width=32, caption='GlobalAveragePooling'),

    to_SoftMax("soft1", 8 ,"(0,0,0)", "(pool3-east)", caption="fc1"),
    to_connection( "soft2", "soft1"),
    to_SoftMax("soft2", 1 ,"(1,0,0)", "(soft1-east)", caption="fc2", height=1, width=1, depth=1),
    to_end()
    ]

The above makes a green arrow through the whole network instead of the last 2 layers.  Why?

My bad.  You have to add the to_connection()'s after the to_X are already defined.


Add Lion optimizer

Proposed changes
Implement the Lion optimizer.
Checklist
Put an x in the boxes that apply.

[x] I have read the CONTRIBUTING document
[x] I have run pre-commit run --all-files to format my code / installed pre-commit prior to committing changes
[ ] I have added tests that prove my fix is effective or that my feature works
[x] I have updated the necessary documentation (if needed)

Hello,
Besides the tests in python/tests/test_optimizers.py, should I add something ? I have tried it on the mnist MLP example and compared with Torch and Optax implementations.

Thank you for the feedback! I addressed the comments, I hope it is better now.

Hi @deschena. I'm just dropping in to say hello 😄  and also thanks for the contributions!
Also I wanted to add your name to our  acknowledgments section with all the ops you added early on. If you are ok with that, would you include it in this PR?

Hi @awni, thanks for dropping in 😃 To be fair, I do not remember which ops I worked on 🥲. I am so glad to see so many people contributing to MLX already, congratulations to all!

I looked at the history, these ones:
Sine, Cosine, arange, randint, truncated normal, bernoulli, linear and logistic regression python examples

a few python bindings and some bug fixes!

@deschena pinging you to edit the acknowledgements file so we can merge this :-)

Apologies for the delays, how does that look now ? Thanks for reviewing the changes :)


The footer component on the website is not routed

The footer component throws an error of 404
Means it's either not routed or the router cannot find the default component

Hello. I will like to be assigned to this issue.
I will fix the routing and create a new footer design


Keywords to references not shown in the pdf version of the reference list

In the HTML version of the Conventions document the "keywords" are shown as expected in the reference list, but not in the PDF version. See this example:
HTML:

▪ [NetCDF] NetCDF Software Package . UNIDATA Program Center of the University Corporation for Atmospheric Research

PDF:

▪ [] NetCDF Software Package . UNIDATA Program Center of the University Corporation for Atmospheric Research .

Dear @larsbarring
Thanks for pointing this out. I am going to repost it in David's open discuss issue on the release just made. I hope that's OK.
Jonathan


3039-replace old partner text to new partner text

Resolves #3039
Description
Change old partner text to new partner text
Type of change

Breaking change (fix or feature that would cause existing functionality to not work as expected)

How Has This Been Tested?

Go to partner page ( http://localhost:3000/diaper_bank/partners/2 )
check labels
edit partner Information

Screenshot after change

Left one last comment about the change. After we confirm that the change is safe and works locally, we can move onto merging.

Hi @edwinthinks
I checked all pages and forms.
Everything is ok 🚀

@italomatos nice! I see a few conflicts in the merge. Could you address them and then we can get this merged in :) 🚀

@italomatos nice! I see a few conflicts in the merge. Could you address them and then we can get this merged in :) 🚀

rebase done 🚀


Add NX Generator with simple default configuration

[ ] Create simple default config (based on Logging react test app config)
[ ] Search through all console.log|warn|trace|error|info statements in .ts files (which are considered source) and replace with default logger call

Wont be implemented


fix: update cosign chart and remove extra logs

Description
What this PR does / why we need it:

removes extra logging statement added during development
updates cosign verifier helm chart template to specify a well-known path for key field. The cosign verifier expects an absolute file path in the key field and not the contents directly. This is because the value cosign.key passed in via helm chart is translated to a secret that is then mounted to a well known path in the Ratify container. Cosign verifier loads this key directly via a file path provided in the key field. This behavior is the legacy one but for backwards compatibility we must support it.

Which issue(s) this PR fixes (optional, using fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when the PR gets merged):
Fixes #
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Helm Chart Change (any edit/addition/update that is necessary for changes merged to the main branch)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Please also list any relevant details for your test configuration

locally installed v1.1.1 using helmfile.yaml with extra fields such as cosign.key value. Then performed an upgrade to latest dev branch bits with same helm values and made sure verification of cosign sig continues to succeed.

Checklist:

[ ] Does the affected code have corresponding tests?
[ ] Are the changes documented, not just with inline documentation, but also with conceptual documentation such as an overview of a new feature, or task-based documentation like a tutorial? Consider if this change should be announced on your project blog.
[ ] Does this introduce breaking changes that would require an announcement or bumping the major version?
[ ] Do all new files have appropriate license header?

Post Merge Requirements

[ ] MAINTAINERS: manually trigger the "Publish Package" workflow after merging any PR that indicates Helm Chart Change

looks like a cosign test is failing.

looks like a cosign test is failing.

updated the chart to fix cosign failing test


chore: troubleshooting PR 1414

Troubleshooting the pull request #1414. The tests commented out are the ones that fail in normal executions.

The same error in https://source.cloud.google.com/results/invocations/c2749c3d-ceec-439b-a14e-d26ae7789631
[1/7] Initializing...                                                                                (19.7s @ 0.54GB)
 Version info: 'GraalVM 22.3.3 Java 11 CE'
 Java version info: '11.0.20+8-jvmci-22.3-b22'
 C compiler: gcc (redhat, x86_64, 4.8.5)
 Garbage collector: Serial GC
 5 user-specific feature(s)
 - com.google.api.gax.grpc.nativeimage.GrpcNettyFeature
 - com.google.api.gax.nativeimage.GoogleJsonClientFeature
 - com.google.api.gax.nativeimage.OpenCensusFeature
 - com.oracle.svm.thirdparty.gson.GsonFeature
 - org.graalvm.junit.platform.JUnitPlatformFeature
Jun 11, 2024 2:37:11 AM com.google.api.gax.nativeimage.NativeImageUtils registerClassForReflection
WARNING: Failed to find io.grpc.netty.shaded.io.netty.channel.ProtocolNegotiators on the classpath for reflection.
[junit-platform-native] Running in 'test listener' mode using files matching pattern [junit-platform-unique-ids*] found in folder [/tmpfs/src/github/java-storage-nio/google-cloud-nio/target/test-ids] and its subfolders.
[2/7] Performing analysis...  []                                                                        (59.6s @ 0.74GB)
   8,083 (79.45%) of 10,174 classes reachable
  12,374 (67.14%) of 18,430 fields reachable
  37,326 (39.83%) of 93,703 methods reachable
     899 classes, 2,091 fields, and 28,354 methods registered for reflection

Error: Classes that should be initialized at run time got initialized during image building:
 com.google.api.client.util.Base64 was unintentionally initialized at build time. To see why com.google.api.client.util.Base64 got initialized use --trace-class-initialization=com.google.api.client.util.Base64
To see how the classes got initialized, use --trace-class-initialization=com.google.api.client.util.Base64
Error: Use -H:+ReportExceptionStackTraces to print stacktrace of underlying exception

@suztomo I am going to close this. please reopen if needed.


Unify 'Hash' and 'Key' response fields

Some commands return a hash of a created object as Key, some return it as Hash. This is pretty obnoxious. We should pick one and use it everywhere.

100% agreed. Thanks for opening this issue, will make a PR to close it.


Configure Select Interaction with featureOverlay layer

Overview
Adds the ability to configure a Select interaction with a featureOverlay layer. This layer is expected to already exist in the map. This is useful for when the user wants the selected items to be added to a pre-existing layer, rather than an unmanaged internal layer. This change is completely backwards compatible, and does not affect existing functionality at all.
Also adds documentation and an example for the new functionality.
Fixes #8064
Demo

Notes
This is my very first PR to openlayers. I'm excited, but also humble. Please let me know if there are any stylistic / philosophical issues with the code, and if I should change things to move in a different direction.
I'm not sure why, but the "Feature Overlay" demo feels a hair slower than the default "Click" demo.

Thank you so much for taking a look! I've dropped feature overlay getter commit, and fixed the typo:
diff --git a/src/ol/interaction/Select.js b/src/ol/interaction/Select.js
index 29aeb1a49..72b7217a4 100644
--- a/src/ol/interaction/Select.js
+++ b/src/ol/interaction/Select.js
@@ -154,7 +154,7 @@ class SelectEvent extends Event {
  * further feature filter using the `filter` option.
  *
  * Selected features are added to the `featureOverlay` layer if specified,
- * otherwise in an internal unmanaged layer.
+ * otherwise to an internal unmanaged layer.
  *
  * @fires SelectEvent
  * @api
@@ -302,15 +302,6 @@ class Select extends Interaction {
     return this.hitTolerance_;
   }

-  /**
-   * Returns the overlay layer that stores selected features.
-   * @return {module:ol/layer/Vector} Layer.
-   * @api
-   */
-  getFeatureOverlay() {
-    return this.featureOverlay_;
-  }
-
   /**
    * Returns the associated {@link module:ol/layer/Vector~Vector vectorlayer} of
    * the (last) selected feature. Note that this will not work with any

This is ready for another look.

Thanks for the detailed feedback! I'll have a chance to address them this evening.

The code looked decent, @rajadain, looking forward to future contributions.


IGAPP-328: foreground push notification custom duration

As i mentioned in the other PR. I think for foreground push notifications the SHOW_DURATION is too short, so here my proposal

@steffenkleinle ok perfect. this was more or les just a short draft. thx for your proposal


"subnet_mapping": only one of subnet_mapping,subnets can be specified, but subnet_mapping,subnets were specified.

Description
AWS provider since v5.31.0, validates that exactly one of either subnets or subnet_mapping is configured.
Currently, the terraform-aws-alb module uses [] as the default of subnets. It means module users who want to use subnet_mapping have to set subnets to null in order to avoid triggering validation error:
"subnet_mapping": only one of `subnet_mapping,subnets` can be specified, but `subnet_mapping,subnets` were specified.

[x] ✋ I have searched the open/closed issues and my issue is not listed.

Versions

Module version [Required]:
9.4.0
Terraform version:
v1.5.7
Provider version(s):
provider registry.terraform.io/hashicorp/aws v5.31.0

Reproduction Code [Required]
terraform {
  required_version = "~> 1.0"
  required_providers {
    aws = {
      source  = "hashicorp/aws"
      version = "= 5.31.0"
    }
  }
}

data "aws_availability_zones" "zones" {}

module "vpc" {
  source  = "terraform-aws-modules/vpc/aws"
  version = "5.4.0"

  name = "alb-test"
  cidr = "<IP_ADDRESS>/16"

  azs             = data.aws_availability_zones.zones.names
  public_subnets  = ["<IP_ADDRESS>/24", "<IP_ADDRESS>/24", "<IP_ADDRESS>/24"]
  private_subnets = ["<IP_ADDRESS>/24", "<IP_ADDRESS>/24", "<IP_ADDRESS>/24"]

  enable_nat_gateway = true
  single_nat_gateway = true
}

resource "aws_eip" "lb" {
  for_each = toset(data.aws_availability_zones.zones.names)

  domain = "vpc"
  tags = {
    Name = "alb-test-${each.key}"
  }
}

module "aws-load-balancer" {
  source  = "terraform-aws-modules/alb/aws"
  version = "9.4.0"

  name = "alb-test"

  load_balancer_type               = "network"
  enable_cross_zone_load_balancing = true
  vpc_id                           = module.vpc.vpc_id

  subnet_mapping = [
    for i, az in data.aws_availability_zones.zones.names:
    {
      subnet_id     = module.vpc.public_subnets[i]
      allocation_id = aws_eip.lb[az].id
    }
  ]

  enable_deletion_protection = false
}

Expected behavior
terraform apply successfully creates the load balancer.
Actual behavior
Planning failed. Terraform encountered an error while generating this plan.

╷
│ Error: Invalid combination of arguments
│
│   with module.aws-load-balancer.aws_lb.this[0],
│   on .terraform/modules/aws-load-balancer/main.tf line 12, in resource "aws_lb" "this":
│   12: resource "aws_lb" "this" {
│
│ "subnet_mapping": only one of `subnet_mapping,subnets` can be specified, but `subnet_mapping,subnets` were specified.
╵
╷
│ Error: Invalid combination of arguments
│
│   with module.aws-load-balancer.aws_lb.this[0],
│   on .terraform/modules/aws-load-balancer/main.tf line 65, in resource "aws_lb" "this":
│   65:   subnets                    = var.subnets
│
│ "subnets": only one of `subnet_mapping,subnets` can be specified, but `subnet_mapping,subnets` were specified.

Additional context
A workaround is to add subnets = null when using the module. Alternatively, use AWS provider < 5.31 (with terraform-aws-alb <= 9.2.0, because the later versions depends on AWS provider >= 5.31).

This issue has been resolved in version 9.4.1 :tada:


Add nix-formatter-pack

I build a nice little tool to easily setup multiple formatters (we currently use nixpkgs-fmt and statix, this adds deadnix). The benefit is that running nix fmt fixes all issues found by all tools and nix flake check checks for all tools.
This is very opinionated so feel free to close it, if you don't like it :)
Requires https://github.com/t184256/nix-on-droid/pull/204.

rebased on master. Thanks for the review!


Extend the scope of submariner to cluster-to-non-cluster

Currently, scope of submariner is to achieve connectivity of cluster-to-cluster. This issue is to extend its scope to cluster-to-non-cluster.
Background of this issue is that k8s doesn't provide a fixed egress source IP from a pod when a pod access to outside k8s cluster, in a common way (Please see discussion in https://github.com/kubernetes/enhancements/pull/1105.) So, this could be a problem to achieve the connectivity of cluster-to-non-cluster.
I'm not sure that submariner is the right component to handle this, but as I commented in https://github.com/kubernetes/enhancements/pull/1105#issuecomment-571694606, submariner will help achieve this goal. Therefore, I would like to get submariner community's attentions to it. (At least to consider whether this could also be a submariner's scope or this should be handled in a different project that submariner can collaborate with, etc ..)

I will explain with these slides k8s-to-non-k8s.pdf .

Ping @Oats87 this makes sense to me for connecting legacy cloud workloads to submariner / k8s workloads. I propose we model it in terms of submariner.

@mangelajo @Oats87
Sorry for late update.
I've implemented operator version of this feature as a separated component from submariner itself, here.
It is completely rewritten with go and it now has unit tests and e2e tests, so it should be ready for starting discussion on moving it to subproject of submariner.
Could you check it?
I would like to resume discussions on what will be the next step.

This looks great @mkimuram and @mangelajo there is definitely merit here for this use case, especially given a Submariner-enabled cluster is really just an IPsec (or wireguard) endpoint

@mkimuram following the upstream conversation, please send a kep-like, updated for submariner into https://groups.google.com/forum/#!topic/submariner-dev/EEyaG5iDg6M so we can discuss and see how to make this possible in submariner.

@mkimuram do you want to dedicate a big part of next meeting to this? I think we need to help you model things in a way they are more compatible with the submariner design but still you achieve the same functionality.

@mkimuram we can also setup a one time meeting on a different time if it works better for you

@mangelajo

do you want to dedicate a big part of next meeting to this? I think we need to help you model things in a way they are more compatible with the submariner design but still you achieve the same functionality.

Thank you for taking care of this issue.
Actually, the enhancement proposal below is still a draft, but I would appreciate it if you could take time to give me a feedback in the next meeting.
https://github.com/mkimuram/enhancements-1/blob/non-cluster/20200602-cluster-to-noncluster-connection.md

Still discussing online.


Link to icons in example workflow is corrupted

Link to icons in example workflow is corrupted:

Fixed! Pull/download the latest commit and it fixes the file paths.


Add max error retry config option to glue client

Updated the hive.rst documentation as well

merged, thanks!


Make 'AD' field in VCF header Number=R

In the script that converts the VarDict output to a VCF, it adds a header for the AD field that has Number=.:
https://github.com/AstraZeneca-NGS/VarDict/blob/12e24ad1fcd6e18c93a55a0bcbc61fe1a9d31ca2/var2vcf_valid.pl#L94
For the reasons explained here, this field should have Number=R, since it has one value for each allele (including reference). This change will assist downstream tools especially.
I'm using VarDictJava, but I gather that it uses the same script so everyone would benefit from this fix.

I'm 👍 on this. Was about to submit a PR this evening!

I pinged the PR with docs support. I'm with you.

Thanks again for this, it's now merged and included and hopefully makes everyone's life easier dealing with VarDict outputs downstream. Much appreciated.

Thanks! We appreciate you too!


Ford long v2

Add support for openpilot longitudinal control (alpha) to Ford

https://github.com/commaai/panda/pull/1359

Looks like CmbbOvrrd_B_RqDrv is if driver is overriding, but only more than the request, we should set that all the time (but first figure out what it changes)

Tested a few different scenarios with AEB:

Blocked ACCDATA to the car: dash alert saying FCW and AEB unavailable which was true
Set these signals in ACCDATA incorrectly/oppositely, yet AEB still worked as expected:

Cmbb_B_Enbl to 1
AccBrkTot_A_Rq and AccPrpl_A_Rq to 2 m/s^2
AccBrkPrchg_B_Rq and AccBrkDecel_B_Rq to 0
CmbbOvrrd_B_RqDrv to 1
CmbbEngTqMn_B_Rq to 0

Set CmbbDeny_B_Actl to 1, FCW still worked, but AEB seemed either less sensitive or non-existent.


Master train 195

Includes:
#1747
#1752

@mesosphere-mergebot bump-ee

@mellenburg what's the situation on the integration-test/vagrant-bash tests?

Failed integration-test/vagrant-bash runs:

https://teamcity.mesosphere.io/viewLog.html?buildId=717001
https://teamcity.mesosphere.io/viewLog.html?buildId=716919
https://teamcity.mesosphere.io/viewLog.html?buildId=716743

Restarted, still failing:
https://teamcity.mesosphere.io/viewLog.html?buildId=717100
test_applications.test_octarine |
test_networking.test_ip_per_container |
test_service_discovery.test_service_discovery_docker_bridge |
test_service_discovery.test_service_discovery_docker_bridged_network |
test_service_discovery.test_service_discovery_docker_overlay_port_mapping |

Created https://jira.mesosphere.com/browse/DCOS_OSS-1467 for tracking the task of debugging the failed integration-test/vagrant-bash tests.

@vespian @branden per some analysis of @mellenburg it looks like the combination of #1747 and #1752 is introducing some new failure modes with our integration tests, but independently they are ok. So we are going to pull one off the train and proceed... is one of these more important to land first? Also, if you have any feedback or thoughts on what might be causing the issue here with test_applications.test_octarin please let us know.

Ran a bisect on the two PRs that were included in this train.
train-195a: https://github.com/dcos/dcos/pull/1769
train-195b: https://github.com/dcos/dcos/pull/1770
train-195a, includes the changes from PR #1747 and that PR is the pr appears to be the one that results in the failure of integration-test/vagrant-bash that we're seeing on this train.
I'm going to close this train, bump #1747 back to WIP and ask that that PR be investigated before it merges.


userdata: fix append scenarios

Description

Fixes case of appending userdata when both template and vm data are either shellscript or cloudconfig
Fixes error when appending gzip userdata
Fixes case when userdata manual text from VM is not getting decoded-encoded correctly.
Fixes case of appending multipart data when both template and vm data contain same format types.
Refactor - moved validateUserData method to UserDataManager class

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[ ] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[ ] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

@blueorangutan package

@shwstppr a [SF] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 6469

@blueorangutan test

@shwstppr a [SF] Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[SF] Trillian test result (tid-7062)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 43226 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr7741-t7062-kvm-centos7.zip
Smoke tests completed. 111 look OK, 2 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File

test_01_migrate_VM_and_root_volume
Error
79.91
test_vm_life_cycle.py

test_02_migrate_VM_with_two_data_disks
Error
58.69
test_vm_life_cycle.py

test_hostha_enable_ha_when_host_in_maintenance
Error
303.94
test_hostha_kvm.py

test_hostha_kvm_host_recovering
Error
9.29
test_hostha_kvm.py

@harikrishna-patnala based it on main as https://github.com/apache/cloudstack/pull/7643 was merged on main

@blueorangutan package

@shwstppr a [SF] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 6478

@blueorangutan package

@shwstppr a [SF] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 6479

@blueorangutan test

@shwstppr a [SF] Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[SF] Trillian test result (tid-7096)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 42800 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr7741-t7096-kvm-centos7.zip
Smoke tests completed. 110 look OK, 3 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File

test_08_arping_in_ssvm
Failure
5.18
test_diagnostics.py

test_09_arping_in_cpvm
Failure
5.20
test_diagnostics.py

test_deploy_vm_with_registered_userdata_with_override_policy_append
Failure
109.77
test_register_userdata.py

test_01_migrate_VM_and_root_volume
Error
85.09
test_vm_life_cycle.py

test_02_migrate_VM_with_two_data_disks
Error
51.49
test_vm_life_cycle.py

@blueorangutan package

@shwstppr a [SF] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 6497

@blueorangutan test

@kiranchavala a [SF] Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[SF] Trillian test result (tid-7102)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 41302 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr7741-t7102-kvm-centos7.zip
Smoke tests completed. 112 look OK, 1 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File

test_01_migrate_VM_and_root_volume
Error
76.56
test_vm_life_cycle.py

test_02_migrate_VM_with_two_data_disks
Error
53.33
test_vm_life_cycle.py


can't seem to get the client connected

client was activated, server shows no errors, it is running on the same docker server in the same compose file, it does not work with "unison_server" as servername, so I tried with the ip address but it has the same error, client starts and immediately exits error 255
http/api and ssh ports are working (see the telnet commands for that), I restarted the server to test that too.
any idea ?

Hi there,
Are you testing ports from the container or from your host?
Ports should be reached from the client for SSH to work, so it really depends on how you expose ports on docker-compose and the docker network you're using.
If you can share your docker-compose it's easier to help.
If you check example here server and client are on the same docker network so client can resolve server hostname using an alias.
    networks:
      unison:
        aliases:
          - unicloud_server

for SSH port if you are on the same network (like in the compose example) you can use internal port  SERVER_PORT=22, if you are on another machine you need to use exposed port.
If you clone the repo and run docker-compose up -d everything should works just fine.  I tested several times on different platforms :-) If you use the docker-compose on the root of the repo you'll find a local_test folder with all the files generated by the app, one for server and another for the client.
Just to clarify what's the registration process.
When you register a client, its ssh public key is added to server authorized key.
You should have on server data volume a file like this.
local_tests/server/.ssh/unicloud_authorized_keys
Check if the client pub key is there but it should be if you completed the registration process.
Please share your docker-compose, I'll be happy to have a look.
Start with the docker-compose example provided, if you want to start fresh just cleaning all possible leftover files enter local_test folder and run cleanup.sh script.
Hope this was useful.
Thanks
A.

Hi,
The only difference with my compose files is that I am using macvlan for networking, so I don't need to define any ports, both containers have their own ip. I can confirm that the public key from the client is in the authorized file on the server, I can also confirm that when I replace the entrypoint for the client with "tail -f /dev/null" and when I enter the container via bash, I can ping the server on its hostname, I can also telnet on the server ports 22 and 80, so networking everything is ok. Some evidence in attach, still the unison_client doesn't want to connect. I attached the compose file too.
I started clean for the third time, on 3 different servers already (first try was without macvlan and where server and client were on different hosts in different regional zones - client in US and server in Europe)
Thanks
V

Thanks for sharing compose and other evidence.
Everything from network perspective seems just fine.
When client start checks ssh connection, if not able to connect will exit.
It's possible that when the client start the server (or at least port 22) is still not up.
This is why I added this on code example:
    restart: on-failure

this will make a loop restart until server is ready.
Can you try to add it? I tried also depends but for local testing restart seems more efficient.
You can comment UNISON_PARAMS env from a client, it's a leftover (my bad) for a test with a windows machine to change ssh algorithm but unless you are on windows you don't really need it.
If you can enter client docker container with ssh this is what the client does to initiate sync (make sure server is up and running).
unison unicloud
Since sync works trough ssh if you run it you should be able to test ssh connection.
A

I removed the PARAMS (I am running the containers on Ubuntu 20.04) and restarted the client, same issue. Server is still running (couple of days) so it can't be that the SSH is not up yet. I changed the container to start a shell and did the "unison unicloud" but that does not work, there is no prf file

Client Profile is on unison user $HOME
Should be /data/.unison/unicloud.prf
Run the same command as unison user.

I did some other tests with ubuntu 20.04 and everything seems fine with the docker-compose.yml in the repo.
Something is broken server side, maybe ssh port is up but it's in a loop restart.
You can check ssh status on server using supervisor (as root)
supervisorctl status
still on server container check also /data/logs/supervisor* for possible errors.
I can't think of anything else that could explain the error.
Test fresh with the default docker-compose.

The server is fine

I really think the client is the issue, there are no files in /data/.unison

You should have unicloud.prf inside /data/.unison folder.
Unison profile is generated during client startup process so if you changed the docker entrypoint it's normal you don't have it.
Something like this.
root=ssh://unison@unicloud_server:22//shares/share1

root=/data/share
clientHostName=testing-client1
batch = true
auto = true
prefer = newer
log = false
#place additional params with UNISON_PARAMS env
ignore = Name .git*
ignore = Name .idea
ignore = Name .unison
ignore = Name .DS_Store

You might try to register the client again.
Stop the client container, delete it from client management page /clients/mgt
Remove client data folder (in your case seems /opt/docker/unison/client/data) make sure you delete all hidden files as well.
Remove custom docker entrypoint and try registration process again.

I tried everything, I abandoned my macvlan config, just copied your entire compose file, just the same error, I even changed the my compose version from 3.5 to 3.3 too.
Since the client restarts in 10ms it probably does not create the .prf file, in the bind mount outside the container, the .unison folder is empty (see last screenshot)

Thanks for your patience :-)
I'm really trying to replicate the issue but I really can't.
I'm testing with ubuntu 20.04 and on my macos but always the same behavior.
I register the client, I create the share and after a couple of restarts the client start the sync normally.
The other thing that comes to my mind could be related to permissions reading ssh pvt key when testing connection.
Is default userid 1000 a valid user on your machine?
On startup process unison profile is created after successfully tested the ssh connection,  that's the reason why you don't have it.
code
That's the command the client run to test ssh connecton.

yes user with 1000 exists on my host, but this has nothing to do with the permissions on the filesystem within the container, I don't see why that should matter, it's just a number in the container

If client ssh key has wrong permission ssh auth won't work.
If you want to enable ssh logs (server side) check this comment on a previous issue:
https://github.com/agarbato/unicloud/issues/16#issuecomment-1024980007
At least you'll know if the client can reach the server and connection is refused server side.
Meanwhile I'll keep trying to replicate on a fresh ubuntu server and add some debug logs when ssh connection is not working.
Sorry it's not working for you.

I just pushed a new image with ssh connection test debug enable.
When connection is not working you should see on compose logs something like this.
test_unicloud_client_1  | SSH Connection KO, exit code 255, output unison@unicloud_server: Permission denied (publickey,password,keyboard-interactive).

Pushed another image update with server ssh debug log enabled when SERVER_DEBUG=True (like in compose example).
On server /data/log/sshd.log you'll find ssh connection logs.

server

client before activate

client after activate

server log and ssh info

# more sshd.log

Server listening on <IP_ADDRESS> port 22.
Server listening on :: port 22.
Connection from <IP_ADDRESS> port 43394 on <IP_ADDRESS> port 22 rdomain ""
Failed publickey for unison from <IP_ADDRESS> port 43394 ssh2: RSA SHA256:1jE/Av9SUtGF7EAcxB0UqjpBxNqHvCzI/7fk/iZkCDM
Connection closed by authenticating user unison <IP_ADDRESS> port 43394 [preauth]
Connection from <IP_ADDRESS> port 43402 on <IP_ADDRESS> port 22 rdomain ""
Failed publickey for unison from <IP_ADDRESS> port 43402 ssh2: RSA SHA256:1jE/Av9SUtGF7EAcxB0UqjpBxNqHvCzI/7fk/iZkCDM
Failed none for unison from <IP_ADDRESS> port 43402 ssh2
Failed password for unison from <IP_ADDRESS> port 43402 ssh2
Failed password for unison from <IP_ADDRESS> port 43402 ssh2
Connection closed by authenticating user unison <IP_ADDRESS> port 43402 [preauth]

Connection from <IP_ADDRESS> port 43448 on <IP_ADDRESS> port 22 rdomain ""
Authentication refused: bad ownership or modes for directory /data
Failed publickey for unison from <IP_ADDRESS> port 43448 ssh2: RSA SHA256:1jE/Av9SUtGF7EAcxB0UqjpBxNqHvCzI/7fk/iZkCDM
Connection closed by authenticating user unison <IP_ADDRESS> port 43448 [preauth]

thanks for helping, but I have lost too much time with this, it surely has something to do with file/directory permissions, but this should be handled by the containers when starting up, and not by me constantly changing permissions just to get it started, will look into something else.

It's definitely a permission issue.  It's well written in bold on README.md and I tried to point you in the right direction.  Thanks for testing anyway, all feedback are well appreciated.   The best I can do is check permission on startup and exit with a clear message.  SSH must have proper permission to work.


Black screen.

I'm trying to make a modification to the game, but when playing it from the editor, it shows a black screen.
I can still hear the audio.
Please can someone tell me how to fix it because I really don't know how to debug potential visual elements that could be obscuring the view.

Actually, nevermind I resolved the issue : I was using the wrong renderer (went from GLES3 to GLES2).

sweet! (;


Pin to official NPM ember-osf instead of a specific git commit

Purpose
Make the demo app install ember-osf from NPM rather than a specific pinned git commit. This is more realistic for apps going forward.
Summary of changes
This pin will allow patch releases (per semver spec), to best support planned evaluation of greenkeeper on this repo.
Testing notes
I was having trouble getting the demo app to recognize my staging account this AM; not sure why.

I was getting weird API errors on staging, but this worked on prod. Let me know if problems occur.


problems during training

Hello, I'm very impressed with your work. I also want to ask a question, does the training run train.py? I'm curious if there will be static obstacles in the training scene?

Yes, please refer to the steps shown in the Readme.md to conduct training in the simulation. In this work, static obstacles are not included in the training and are only handled during the deployment of the SARL* algorithm. But feel free to modify the simulation environment to add them.


How can I install vega plugin in Kibana 5.6.1

HI Guys,
I am new in Kibana. I want to know that how can I configure vega in Kibana 5.6.1 in ubuntu 16.04.
There is not an option display for vega graphs under Other in Visulization tab in Kibana 5.6.1
Kindly help me to install vega plugin in kibana 5.6.1
I am using elastic search 5.6.1
Kibana 5.6.1
regards,
Adil kadiyawala

You need to follow instructions on the main page with this older release. Let me know if you run into problems.


torchdynamo+fx2trt hangs torchbench opacus_cifar10

Command to reproduce: python run.py opacus_cifar10 -d cuda -t eval --torchdynamo fx2trt
The process never halts (waited for 1 hour)

It is a tracer issue. Need more efforts to dive in.

Closing the issue as this seems stale. Feel free to reopen.


New Flutter photo picker requires several steps to get to Google Photos

Steps to reproduce
The new Flutter photo picker requires the following steps to allow a user to select photos from Google Photos, on the latest released version of Android (on Pixel 7 Pro):

A screen opens with two buttons, "Photos" and "Albums", and a three-dot menu at the top right.

It used to be that Google Photos popped right up, and it does on Android 33 on the emulator.
The "Photos" and "Albums" buttons only show you photos locally stored on the phone.

The user has to tap the three-dot menu, which has only one option, "Browse".

Why is there even a menu button here, if there is only one option? It would be better to have a Browse button, because that would be much more discoverable.

"Browse" takes you to a screen that says "Search this phone" at the top, and has a list of apps: "Browse files in other apps".

Google Photos is not even shown on the first page of these icons (I see three Drive icons, then "Media picker").
There is no visual indication anywhere that this list of icons can be scrolled, so that you can see more icons on the right. This is totally un-discoverable UI.

The user has to scroll the icons list to the left (to see more icons on the right), then can tap Photos.
Finally, the list of Google Photos albums pops up.

This is a ridiculously complicated way to get to your photos, when your primary means of interacting with the photos you have taken in the past is through Google Photos. (I don't have even a fraction of all my photos actually physically cached on my phone...)
cc @stuartmorgan
Expected results
The buttons for Google Photos or other album software should appear on the very first modal bottom sheet that pops up, and if the icons don't all fit on the screen, there should be a way to scroll this, just how the Share modal bottom sheet works right now, displaying all apps that can handle the Share intent.
Actual results
It takes a lot of hunting effort and some creativity to even find the Google Photos albums, and that leads to a very negative user experience, because users think they can't select photos from their albums anymore.
Code sample
N/A
Screenshots or Video
N/A
Logs
N/A
Flutter Doctor output
Doctor output
flutter doctor -v
[✓] Flutter (Channel beta, 3.19.0-0.1.pre, on Fedora Linux 39 (Workstation Edition) 6.6.9-200.fc39.x86_64, locale en_US.utf8)
    • Flutter version 3.19.0-0.1.pre on channel beta at /opt/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 1a9a60d044 (3 weeks ago), 2024-01-10 13:38:09 -0800
    • Engine revision bbebee1465
    • Dart version 3.3.0 (build 3.3.0-279.0.dev)
    • DevTools version 2.31.0

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.1)
    • Android SDK at /home/luke/Android/Sdk
    • Platform android-34, build-tools 33.0.1
    • Java binary at: /opt/android-studio/jbr/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at google-chrome

[✓] Linux toolchain - develop for Linux desktop
    • clang version 17.0.6 (Fedora 17.0.6-1.fc39)
    • cmake version 3.27.7
    • ninja version 1.11.1
    • pkg-config version 1.9.5

[✓] Android Studio (version 2022.2)
    • Android Studio at /opt/android-studio
    • Flutter plugin version 73.0.2
    • Dart plugin version 222.4582
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)

[✓] VS Code (version 1.85.2)
    • VS Code at /usr/share/code
    • Flutter extension version 3.81.20231227

[✓] VS Code (version 1.86.0-insider)
    • VS Code at /usr/share/code-insiders
    • Flutter extension version 3.79.20231130

[✓] Connected device (4 available)
    • Pixel 7 Pro (mobile)       • 29161FDH300BLP • android-arm64  • Android 14 (API 34)
    • sdk gphone x86 64 (mobile) • emulator-5554  • android-x64    • Android 13 (API 33) (emulator)
    • Linux (desktop)            • linux          • linux-x64      • Fedora Linux 39 (Workstation Edition) 6.6.9-200.fc39.x86_64
    • Chrome (web)               • chrome         • web-javascript • Google Chrome 120.0.6099.216

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

• No issues found!

That popup is not implemented by Flutter, it just calls out to Android with the following activity:
usePhotoPickerAndroid -> ActivityResultContracts.PickVisualMedia
!usePhotoPickerAndroid  -> Intent(Intent.ACTION_GET_CONTENT)
https://developer.android.com/training/data-storage/shared/photopicker

Why not use Intent.ACTION_PICK? I think that brings up a list of all apps that can be used to pick photos.

I think Intent.ACTION_PICK is not well supported and deprecated, also requires base URI to start picking from. But anyway, I think google photos still works with ACTION_GET_CONTENT, no?

Possibly it does, I haven't tested this directly (I am working in Flutter). I se to rembe Ron another picker issue that I filed that it's not going to be possible to use anything but the new Android photo picker in future, which seems ridiculous to me, because it is such a bad experience. Can you at least make it possible in perpetuity for apps to set usePhotoPickerAndroid = false?

I don't expect usePhotoPickerAndroid = false will be removed as long as it has this advantage over the new picker and is still supported by Android.

Closing since, as explained above (and as I have explained previously), none of the UI you are providing feedback on is part of the plugin. Feedback about the new Android photo picker experience itself would need to be directed to the Android team.

cc @stuartmorgan

Please don't just CC arbitrary people when filing issues. We have a triage process that routes bugs.

@moffatman I was able to test this with usePhotoPickerAndroid = false and usePhotoPickerAndroid = true.
First issue: this setting is not even exposed in image_picker, you have to add a direct dependency upon image_picker_platform_interface and image_picker_android to get to this setting, and then run the following code:
  if (Platform.isAndroid) {
    ImagePickerAndroid.registerWith();
    (ImagePickerPlatform.instance as ImagePickerAndroid).useAndroidPhotoPicker = true;
  }

Second issue: at least on Pixel 7 Pro, on Android, all that setting useAndroidPhotoPicker = true does is remove the three-dot menu at the top right of the picker sheet, which has the single menu option "Browse".
On Pixel 7 Pro, Android 14, with useAndroidPhotoPicker = false:

On Pixel 7 Pro, Android 14, with useAndroidPhotoPicker = true:

Whereas, on the Android emulator, API level 33, with useAndroidPhotoPicker = false, I get the following (this is what I want to happen on Pixel 7 / Android 14, also):

vs. on the Android emulator, API level 33, with useAndroidPhotoPicker = true, I get the same very limited Android photo picker with no "Browse" menu option:

What the problems are:
Despite the fact that these issues I am filing keep getting closed, there is still a major issue here.
(1) For whatever reason, on Pixel 7 Pro / Android 14, ACTION_GET_CONTENT just uses the same really limited Android photo picker, albeit adding a Browse menu item. I assume that the default Android photo picker is intercepting this intent, or something. There is no way to ask for image_picker to try using a different intent. Please add a config option to select photos with a different intent. This is a plugin issue, NOT an Android issue, which my issues keep getting closed for.
(2) I want to be able to set useAndroidPhotoPicker = false from the image_picker API, and that is not exposed right now. I don't want the default Android photo picker to run, ever. This is also a plugin issue, NOT an Android issue.

I want to be able to set useAndroidPhotoPicker = false from the image_picker API

The current approach is our standard model for settings that are fundamentally linked to a single platform. We will not be re-exposing this from the app-facing package.

There is no way to ask for image_picker to try using a different intent. Please add a config option to select photos with a different intent.

We will not be adding yet another mode to the Android implementation of image_picker. You are welcome to make your own unendorsed Android implementation that has an alternate approach, or to use a different plugin that allows sending arbitrary intents.


[BugFix] Own short_key_encode result by LogicalSplitMorselQueue

Signed-off-by: ZiheLiu<EMAIL_ADDRESS>What type of PR is this：

[ ] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Problem Summary(Required) ：

TupleKey::short_key_encode returns a std::string, but is assigned to Slice, which only references to the std::string not owns it.
=================================================================
==4160==ERROR: AddressSanitizer: stack-use-after-scope on address 0x7f20b16c3180 at pc 0x00000a7f31d9 bp 0x7f20b16c2f80 sp 0x7f20b16c2730
READ of size 2 at 0x7f20b16c3180 thread T358 (pip_wg_executor)
    #0 0xa7f31d8 in MemcmpInterceptorCommon(void*, int (*)(void const*, void const*, unsigned long), void const*, void const*, unsigned long) ../../.././libsanitizer/sanitizer_common/sanitizer_common_interceptors.inc:839
    #1 0xa7f37e8 in __interceptor_memcmp ../../.././libsanitizer/sanitizer_common/sanitizer_common_interceptors.inc:871
    #2 0xa7f37e8 in __interceptor_memcmp ../../.././libsanitizer/sanitizer_common/sanitizer_common_interceptors.inc:866
    #3 0xac16495 in starrocks::memcompare(char const*, unsigned long, char const*, unsigned long) /root/starrocks/be/src/util/memcmp.h:154
    #4 0xac165e3 in starrocks::Slice::compare(starrocks::Slice const&) const /root/starrocks/be/src/util/slice.h:247
    #5 0xbb314ae in starrocks::pipeline::LogicalSplitMorselQueue::_valid_range(std::unique_ptr<starrocks::ShortKeyOption, std::default_delete<starrocks::ShortKeyOption> > const&, std::unique_ptr<starrocks::ShortKeyOption, std::default_delete<starrocks::ShortKeyOption> > const&) const /root/starrocks/be/src/exec/pipeline/scan/morsel.cpp:475
    #6 0xbb300ab in starrocks::pipeline::LogicalSplitMorselQueue::try_get() /root/starrocks/be/src/exec/pipeline/scan/morsel.cpp:419
    #7 0xae131b4 in starrocks::pipeline::ScanOperator::_pickup_morsel(starrocks::RuntimeState*, int) /root/starrocks/be/src/exec/pipeline/scan/scan_operator.cpp:413
    #8 0xae1034f in starrocks::pipeline::ScanOperator::_try_to_trigger_next_scan(starrocks::RuntimeState*) /root/starrocks/be/src/exec/pipeline/scan/scan_operator.cpp:274
    #9 0xae0f63c in starrocks::pipeline::ScanOperator::pull_chunk(starrocks::RuntimeState*) /root/starrocks/be/src/exec/pipeline/scan/scan_operator.cpp:220
    #10 0xacf6355 in starrocks::pipeline::PipelineDriver::process(starrocks::RuntimeState*, int) /root/starrocks/be/src/exec/pipeline/pipeline_driver.cpp:250
    #11 0x12aa4e76 in starrocks::pipeline::GlobalDriverExecutor::_worker_thread() /root/starrocks/be/src/exec/pipeline/pipeline_driver_executor.cpp:144
    #12 0x12aa37ad in operator() /root/starrocks/be/src/exec/pipeline/pipeline_driver_executor.cpp:69
    #13 0x12aac8b1 in __invoke_impl<void, starrocks::pipeline::GlobalDriverExecutor::initialize(int)::<lambda()>&> /usr/include/c++/10.3.0/bits/invoke.h:60
    #14 0x12aabdb6 in __invoke_r<void, starrocks::pipeline::GlobalDriverExecutor::initialize(int)::<lambda()>&> /usr/include/c++/10.3.0/bits/invoke.h:110
    #15 0x12aab0c0 in _M_invoke /usr/include/c++/10.3.0/bits/std_function.h:291
    #16 0xa99b00b in std::function<void ()>::operator()() const /usr/include/c++/10.3.0/bits/std_function.h:622
    #17 0x11a2f11f in starrocks::FunctionRunnable::run() /root/starrocks/be/src/util/threadpool.cpp:57
    #18 0x11a2bf21 in starrocks::ThreadPool::dispatch_thread() /root/starrocks/be/src/util/threadpool.cpp:549

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto backported to target branch

[x] 2.5
[x] 2.4
[ ] 2.3
[ ] 2.2

run starrocks_fe_unittest

run starrocks_admit_test

run starrocks_admit_test

@Mergifyio backport branch-2.5

@Mergifyio backport branch-2.4


Add multiple pool test

Signed-off-by: Timothy Asir Jeyasingh<EMAIL_ADDRESS>

@bipuladh , @vbnrh Please review


Linux Guests will not persist.

grub-bhyve will sit and wait for a console to connect before it will boot the operating system.
Ran into similar issue when getting linux guests to start thus the ' > /dev/null '
https://github.com/pr1ntf/iohyve/blob/master/iohyve#L316
and
https://github.com/pr1ntf/iohyve/blob/master/iohyve#L321

All grub support is now forked off from master.
If you are interested in working on this, please pull against the grub-support branch.

Re-opening for work beginning on grub2-bhyve support again.

This has been fixed with latest grub2-bhyve.


removing unused illumina test data

fixes #510

Coverage remained the same at 71.06% when pulling<PHONE_NUMBER>30e876deb9a4c5dc86ca8ea7f883fa on lb_delete_illumina_test_data into 42ee88989fedbd767ea1b9add6a9770c65999074 on master.


Update ClusterRole permissions in documentation

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

Project coverage is 51.91%. Comparing base (f88413a) to head (f34a58e).

:exclamation:  There is a different number of reports uploaded between BASE (f88413a) and HEAD (f34a58e). Click for more details.
HEAD has 3 uploads less than BASE

Flag
BASE (f88413a)
HEAD (f34a58e)

oats-test
1
0

k8s-integration-test
1
0

integration-test
1
0

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1079       +/-   ##
===========================================
- Coverage   81.90%   51.91%   -29.99%
===========================================
  Files         139      139
  Lines       11329    11329
===========================================
- Hits         9279     5882     -3397
- Misses       1534     5050     +3516
+ Partials      516      397      -119

Flag
Coverage Δ

integration-test
?

k8s-integration-test
?

oats-test
?

unittests
51.91% <ø> (-0.02%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Implement categories better

Inputting 'cds', 'CDS', 'coding-sequence' and 'coding_sequence' all generate different results. Currently only 'CDS' will show the correct sbol symbol in D3.

Bug with imparse
Sent pull request at https://github.com/lapets/imparse/pull/1


Allow ::slotted(selectors)

Shadow DOM V1 needs ::slotted(selectors) to be able to style slotted content inside the shadow root.
I patched this for GWT, and will submit a pull request here with the necessary change.

https://github.com/google/closure-stylesheets/pull/113


[DevPortal] Cannot remove keys

Description
When trying to click remove keys after generating a client id/secret pair, the operation fails with a console error.

Steps to Reproduce

Create an application
Generate client id/secret pair
Click on the "Remove Keys" button

Affected Component
APIM
Version
4.3.0
Environment Details (with versions)
No response
Relevant Log Output
No response
Related Issues
No response
Suggested Labels
No response

This is not reproducible in the latest pack


Poor performance with Gif

Thanks, I like the idea of this project.
This can be really convenient for some debugging tools and to advertise new features.
I did some test with a small project (unity2d)

But the performance of Gif recording is poor, I tried to set the resolution to 480p and the framerate is horrible and there are lot of Jittering.
The same king of setting with Mp4 Recorder is smooth at 60 fps

maybe it's counterintuitive, but gif recording is far far slower than mp4 and it is unlikely to be improved.


feat: some general additions

Wooooo, this is awesome @rishabkumar7 + @madebygps !!
I really like the timeline behind it, makes it very explicit and clear for a beginner.
And I like how each phase has the same / similar structure in the docs, makes it easy to follow.
Just a start, I'll come back and iteratively add to it over time...

thanks Lou! all this looks great.


Use images from pattern site

I suggest overwriting the images in this README with the architecture image from https://developer.ibm.com/patterns/visual-recognition-with-tensorflow-and-openwhisk/
I would do it, but this has two images in the README and i'm not sure if one or both should be removed. I'm happy to perform the PR if someone advises

cc @loafyloaf


CSS animation

You must tell something about css animations in your readme like
An animation lets an element gradually change from one style to another.
You can change as many CSS properties you want, as many times as you want.
To use CSS animation, you must first specify some keyframes for the animation.
Keyframes hold what styles the element will have at certain times.

You write something

Please assign this issue this to me @ankushrai999


Introducing Bahasa Indonesia Localization: Translation of Vue I18n English Language Code

Summary: Translated Vue I18n English language code to Bahasa Indonesia.
Description for the changelog: This pull request encompasses a comprehensive translation effort aimed at providing support for Bahasa Indonesia within the Vue I18n English language codebase. By leveraging the principles of internationalization and localization, the codebase has been extended to cater to Indonesian-speaking users, thereby enhancing the accessibility and usability of the application for a wider audience.
Other info:

The translation process involved meticulous attention to detail to ensure the accuracy and fidelity of the Bahasa Indonesia equivalents to the original English strings. Every effort was made to maintain consistency and coherence throughout the translation process.
Careful consideration was given to cultural nuances and linguistic conventions specific to Bahasa Indonesia, ensuring that the translated content aligns seamlessly with the language's idiosyncrasies and user expectations.
This initiative underscores our commitment to inclusivity and user-centric design, as we strive to accommodate diverse linguistic preferences and facilitate a more immersive and personalized user experience.
Contributors are encouraged to review the changes and provide feedback or suggestions for further refinement, as we continually strive to optimize and enrich the language support within our application.

Hi @jgadsden
I was added Bahasa Indonesia localization to your explanation files, i hope you will check later.
Feel free if you need more code modification from me.
Thanks

perfect, many thanks @teguh02 , I will go ahead and merge


Remove the guidata option

Should potentially close #410

Bot please update snapshots!

I've not looked too deeply into the implementation yet, but it looks like this PR is breaking the some logic upon the application of a boolean operator.
When creating two initial shapes, then applying a cut operator on them, the two initial shapes are not set hidden anymore.
Also when trying to hide them manually from the left panel seems to not work reliably.

I noticed this after your comment, i'm looking into why this is happening. Thanks for pointing it out.

This error here is related?

This error here is related?

Might be related to #397, have to look into it. Thanks a lot for letting me know

Bot please update snapshots!

This error here is related?

Might be related to #397, have to look into it. Thanks a lot for letting me know

Hi @trungleduc, I tried reproducing this on my end locally but it seems like it works fine for me. However I'd want to confirm a couple of things -

Did you select any object before clicking intersection? (As it takes the color of that object)
Do you also get commands.js:262 Uncaught (in promise) TypeError: Cannot read properties of undefined (reading 'value') error when you try to do intersection with no obect or a single object? (This should be a valid error but maybe we could handle it better.)

https://github.com/user-attachments/assets/5149bc12-ca48-44b9-aced-3a7978d6f9c3

Visibility toggling works fine
Only result is visible after operation

yeah, I can not reproduce it anymore, thanks for double-checking it!

we should update https://github.com/jupytercad/JupyterCAD-FreeCAD since it also uses the guidata section

we should update https://github.com/jupytercad/JupyterCAD-FreeCAD since it also uses the guidata section

Yes, I'll be on that next. Thank You :)

Thanks! Let's make an alpha release of this so we can start updating jupytercad-freecad

Sounds good, should I wait for the release to open a PR on jupytercad-freecad?

That will be easier for you that way 👍🏽

It's out https://github.com/jupytercad/JupyterCAD/releases/tag/v3.0.0a0 You can install it with pip: https://pypi.org/project/jupytercad/3.0.0a0/


Refreshing a route while logged in redirects to index.html

I'm running into an issue with redirects when I deploy this app. The issue is that when I am logged in and reload the page, I get redirected to index.html which then redirects me to the home page. The expected behavior is that reloading the page should maintain the current route.
Steps to reproduce:

Log in
go to /app/home
Reload the page
App redirects user to / route which forces the user to have to navigate back to the /app/home to get where they were.

I made a video which documents this behavior:
https://www.youtube.com/watch?v=3rhpsWgAX3s
I played around with this and the behavior can be "fixed" by changing the 404 redirect to a 404 rewrite, but it has the negative consequence that none of the meta tags load and it shows up to facebook link debugger as a 404 error (in contrast to the default behavior where it all loads correctly). So that's to say that the 404 rewrite change probably isn't a long term solution.
Any ideas how you'd go about fixing this? Thanks so much for contributing this demo repo Nader, it's really great!

I've got a similar problem although with a slight difference:

Log in
go to /app/home
Reload the page
App redirects user to / route then redirec to /app/home

I followed this documentation: https://docs.aws.amazon.com/amplify/latest/userguide/redirects.html#redirects-for-single-page-web-apps-spa
Were you able to handle this issue @CaptainChemist ?

@devnantes44 No, I wasn't able to solve it unfortunately.

@devnantes44 @CaptainChemist have you both had a chance to go through the above link and if so, is the issue still happening?

@mtliendo I just tried having the following be my only redirect and I can confirm that this did not fix my problem and in fact made it worse.
source address: </^[^.]+$|.(?!(css|gif|ico|jpg|js|png|txt|svg|woff|ttf|map|json)$)([^.]+$)/>
target address: /index.html
type: 200
Previously, I could directly navigate to pages that was not a protected route and I would not be navigated to the home page, it was only protected pages that would get navigated back to the home page regardless of whether the person was logged in or not.
source address: /<*>
target address: /index.html
type: 404 (Redirect)