The system cannot find the file specified after switching to Flutter master channel

Steps to reproduce
1: Install or upgrade Flutter to the latest stable version (3.27.1):
flutter upgrade
2: Switch to the master channel by running:
flutter channel master
3: Attempt to run any Flutter command, such as:
flutter doctor
Expected results
Flutter Doctor Output:
[✓] Flutter (Channel stable, 3.27.1, on Microsoft Windows [Version 10.0.19045.3570], locale en-US)
    • Flutter version 3.27.1 on channel stable at C:\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 463d0668f1 (6 days ago), 2024-12-14 15:15:17 +0100
    • Engine revision 92d8b65f12
    • Dart version 3.5.0
    • DevTools version 2.24.0

[✓] Windows Version (Installed)
    • Windows 10, version 22H2

[!] Android toolchain - develop for Android devices (Android SDK version 33.0.2)
    • Android SDK at C:\Users\User\AppData\Local\Android\Sdk
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"` to install it.
    • Android Studio not found; unable to find bundled Java version.
    • Try updating or reinstalling Android Studio.
    • Try re-installing or updating Android SDK tools.

[✓] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[✓] Visual Studio - develop for Windows (Visual Studio Community 2022 17.7.5)
    • Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
    • Visual Studio Community 2022 version 17.7.5
    • Windows 10 SDK version 10.0.19041.0

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

[✓] Network resources
    • All connected resources are available.

! Doctor found issues in 1 category.

Actual results
flutter  Doctor:
The system cannot find the file specified.
cb4b5fff73850b2e42bd4de7cb9a4310a78ac40d was unexpected at this time.
Code sample
Code sample
[Paste your code here]

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[Paste your output here]

Hi @ari-0, Thanks for filing the issue. Looks like the master channel switch didn't completely download the flutter sdk,

The system cannot find the file specified.
cb4b5fff73850b2e42bd4de7cb9a4310a78ac40d was unexpected at this time.

Do you have the complete logs of the error, this doesn't really give much info?
For me the master channel switch went through

flutter channel 'master'
 mahesh@MacBook-Pro-2 platform_stack_test % flutter upgrade
flutterFlutter is already up to date on channel stable
Flutter 3.27.1 • channel stable • https://github.com/flutter/flutter.git
Framework • revision 17025dd882 (4 days ago) • 2024-12-17 03:23:09 +0900
Engine • revision cb4b5fff73
Tools • Dart 3.6.0 • DevTools 2.40.2
 %
mahesh@MacBook-Pro-2 platform_stack_test % flutter channel master
Switching to flutter channel 'master'...

Upgrading engine...
Downloading Darwin arm64 Dart SDK from Flutter engine 33d7f8a1b307dc46fe969facf0e1feba6f40aeb2...
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed

  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
  5  229M    5 11.7M    0     0  14.4M      0  0:00:15 --:--:--  0:00:15 14.4M
 17  229M   17 40.8M    0     0  22.4M      0  0:00:10  0:00:01  0:00:09 22.4M
 31  229M   31 72.1M    0     0  24.6M      0  0:00:09  0:00:02  0:00:07 24.6M
 52  229M   52  120M    0     0  30.8M      0  0:00:07  0:00:03  0:00:04 30.8M
 65  229M   65  151M    0     0  31.7M      0  0:00:07  0:00:04  0:00:03 31.7M
 81  229M   81  188M    0     0  32.4M      0  0:00:07  0:00:05  0:00:02 35.3M
 95  229M   95  218M    0     0  32.3M      0  0:00:07  0:00:06  0:00:01 36.0M
100  229M  100  229M    0     0  31.8M      0  0:00:07  0:00:07 --:--:-- 36.7M
Building flutter tool...
Downloading android-arm-profile/darwin-x64 tools...                752ms
Downloading android-arm-release/darwin-x64 tools...                252ms
Downloading android-arm64-profile/darwin-x64 tools...              296ms
Downloading android-arm64-release/darwin-x64 tools...              279ms
Downloading android-x64-profile/darwin-x64 tools...                352ms
Downloading android-x64-release/darwin-x64 tools...                299ms
Downloading android-x86 tools...                                   641ms
Downloading android-x64 tools...                                   688ms
Downloading android-arm tools...                                   791ms
Downloading android-arm-profile tools...                           482ms
Downloading android-arm-release tools...                           474ms
Downloading android-arm64 tools...                                 881ms
Downloading android-arm64-profile tools...                         740ms
Downloading android-arm64-release tools...                         489ms
Downloading android-x64-profile tools...                           668ms
Downloading android-x64-release tools...                           493ms
Downloading android-x86-jit-release tools...                       691ms
Downloading ios tools...                                         2,415ms
Downloading ios-profile tools...                                 1,322ms
Downloading ios-release tools...                                 1,324ms
Downloading Web SDK...                                           2,474ms
Downloading CanvasKit...                                           704ms
Downloading package sky_engine...                                  237ms
Downloading flutter_patched_sdk tools...                           407ms
Downloading flutter_patched_sdk_product tools...                   450ms
Downloading darwin-arm64 tools...                                1,021ms
Downloading darwin-x64/FlutterMacOS.framework tools...           1,382ms
Downloading darwin-x64/gen_snapshot tools...                       420ms
Downloading darwin-x64-profile/FlutterMacOS.framework tools...        740ms
Downloading darwin-x64-profile tools...                            282ms
Downloading darwin-x64-profile/gen_snapshot tools...               397ms
Downloading darwin-x64-release/FlutterMacOS.framework tools...        560ms
Downloading darwin-x64-release tools...                            266ms
Downloading darwin-x64-release/gen_snapshot tools...               430ms
Downloading darwin-arm64/font-subset tools...                      225ms
Successfully switched to flutter channel 'master'.
To ensure that you're on the latest build from this channel, run 'flutter upgrade'
mahesh@MacBook-Pro-2 platform_stack_test % flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
Could not load custom device from config index 0: Expected enabled to be a boolean.
[✓] Flutter (Channel master, 3.7.0-15.0.pre.16, on macOS 15.1.1 24B2091 darwin-arm64, locale en-US)
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 16.0)
[✓] Chrome - develop for the web
[!] Android Studio (version 2024.1)
    ✗ Unable to find bundled Java version.
[✓] IntelliJ IDEA Community Edition (version 2021.2.1)
[✓] VS Code (version 1.94.0)
[✓] Connected device (2 available)
[✓] HTTP Host Availability

! Doctor found issues in 1 category.

Thanks for your response! After switching to the master channel, none of the Flutter commands work, including flutter doctor. Every command produces the same error:This makes it challenging to gather additional logs or diagnostics.
I’ve tried:
Clearing the bin/cache directory.
Verifying my network connection.
Attempting to switch channels again manually.
Unfortunately, the issue persists. Do you have any suggestions for resolving this or alternative ways to debug?
and here is new message that shows,
`flutter channel master
Switching to flutter channel 'master'...
Upgrading engine...
Checking Dart SDK version...
Downloading Dart SDK from Flutter engine 7df127b1ee7d52c4cb0d42eedd00190e2c4ade3d...
Expanding downloaded archive with PowerShell...
Building flutter tool...
Running pub upgrade...
Resolving dependencies...
Downloading packages...
Got dependencies.
Downloading android-arm-profile/windows-x64 tools...             1,463ms
Downloading android-arm-release/windows-x64 tools...             1,269ms
Downloading android-arm64-profile/windows-x64 tools...           1,387ms
Downloading android-arm64-release/windows-x64 tools...           1,362ms
Downloading android-x64-profile/windows-x64 tools...             1,298ms
Downloading android-x64-release/windows-x64 tools...             1,241ms
Downloading android-x86 tools...                                 2,672ms
Downloading android-x64 tools...                                 2,677ms
Downloading android-arm tools...                                    4.1s
Downloading android-arm-profile tools...                            3.7s
Downloading android-arm-release tools...                         1,708ms
Downloading android-arm64 tools...                                  3.0s
Downloading android-arm64-profile tools...                       1,822ms
Downloading android-arm64-release tools...                       1,777ms
Downloading android-x64-profile tools...                         1,828ms
Downloading android-x64-release tools...                         1,705ms
Downloading Web SDK...                                              9.4s
Downloading package sky_engine...                                1,124ms
Downloading package flutter_gpu...                                 327ms
Downloading flutter_patched_sdk tools...                         1,369ms
Downloading flutter_patched_sdk_product tools...                   227ms
Downloading flutter_patched_sdk_product tools...                   363ms
Failed to download https://storage.googleapis.com/flutter_infra_release/flutter/7df127b1ee7d52c4cb0d42eedd00190e2c4ade3d/flutter_patched_sdk_product.zip. Ensure you have network connectivity and then try again.
Exception: 404
C:\Users\hdps>flutter doctor
Downloading package sky_engine...                                  873ms
Downloading package flutter_gpu...                                 103ms
Downloading flutter_patched_sdk tools...                           496ms
Downloading flutter_patched_sdk_product tools...                   649ms
Downloading flutter_patched_sdk_product tools...                   404ms
Failed to download
https://storage.googleapis.com/flutter_infra_release/flutter/7df127b1ee7d52c4cb0d42eedd00190e2c4ade3d/flutter_patched_sd
k_product.zip. Ensure you have network connectivity and then try again.
Exception: 404
C:\Users\hdps>flutter doctor
Downloading package sky_engine...                                  632ms
Downloading package flutter_gpu...                                 353ms
Downloading flutter_patched_sdk tools...                           493ms
Downloading flutter_patched_sdk_product tools...                   243ms
Downloading flutter_patched_sdk_product tools...                   419ms
Failed to download
https://storage.googleapis.com/flutter_infra_release/flutter/7df127b1ee7d52c4cb0d42eedd00190e2c4ade3d/flutter_patched_sd
k_product.zip. Ensure you have network connectivity and then try again.
Exception: 404
C:\Users\hdps>curl -I https://storage.googleapis.com/flutter_infra_release/flutter/7df127b1ee7d52c4cb0d42eedd00190e2c4ade3d/flutter_patched_sdk_product.zip
HTTP/1.1 404 Not Found
X-GUploader-UploadID: AFiumC7SyTz3dto73oi7NTEAFsz3f3agRWYmM50exPsABrIsB846j81l2oMAYsN5G9vAXbw5
Content-Type: application/xml; charset=UTF-8
Content-Length: 264
Access-Control-Allow-Origin: *
Access-Control-Expose-Headers: Content-Type
Date: Sat, 21 Dec 2024 07:31:29 GMT
Expires: Sat, 21 Dec 2024 07:31:29 GMT
Cache-Control: private, max-age=0
Server: UploadServer
Alt-Svc: h3=":443"; ma=2592000,h3-29=":443"; ma=2592000
C:\Users\hdps>flutter doctor
Downloading package sky_engine...                                  926ms
Downloading package flutter_gpu...                                 106ms
Downloading flutter_patched_sdk tools...                           776ms
Downloading flutter_patched_sdk_product tools...                   519ms
Downloading flutter_patched_sdk_product tools...                   509ms
Failed to download
https://storage.googleapis.com/flutter_infra_release/flutter/7df127b1ee7d52c4cb0d42eedd00190e2c4ade3d/flutter_patched_sd
k_product.zip. Ensure you have network connectivity and then try again.
Exception: 404`

I see the same issue, I'm unable to do anything on current master.
I suspect this is the same issue in https://github.com/flutter/flutter/issues/160704 ?

I get this exception:
flutter upgrade
Flutter is already up to date on channel main
Flutter 3.28.0-2.0.pre.38568 • channel main •
https://github.com/flutter/flutter.git
Framework • revision 7df127b1ee (6 hours ago) • 2024-12-20 21:23:04 -0800
Engine • revision 7df127b1ee
Tools • Dart 3.7.0 (build 3.7.0-267.0.dev) • DevTools 2.41.0
sma@MacBook-Pro-von-Stefan tellorium_poc % flutter doctor -v
Downloading package sky_engine...                                  594ms
Downloading package flutter_gpu...                                  38ms
Downloading flutter_patched_sdk tools...                         1.199ms
Downloading flutter_patched_sdk_product tools...                   134ms
Downloading flutter_patched_sdk_product tools...                   205ms
Failed to download
https://storage.googleapis.com/flutter_infra_release/flutter/7df127b1ee7d52c4cb0
d42eedd00190e2c4ade3d/flutter_patched_sdk_product.zip. Ensure you have network
connectivity and then try again.
Exception: 404

Looking at https://storage.googleapis.com/flutter_infra_release/, I see a long XML where the latest flutter_patched_sdk_product.zip is
<Contents>
<Key>flutter/0014e0353aa9b2947ff38cc9c9aad8741e5999f7/flutter_patched_sdk_product.zip</Key>
<Generation>1714597161998195</Generation>
<MetaGeneration>1</MetaGeneration>
<LastModified>2024-05-01T20:59:22.105Z</LastModified>
<ETag>"1bfbe3ef0e0cba8d47041bce53e0422e"</ETag>
<Size>3747802</Size>
</Contents>

which is obviously the wrong one.
I think, a critical upload is missing, breaking the flutter installation!

Duplicate of #160704


Fix PyPI README

Expected Behavior

The look of https://pypi.org/project/pybotics/0.2.7/

Actual Behavior

Broke RST format: https://pypi.python.org/pypi/pybotics/0.3.0

Still broken

Relative links have the potential to break the whole doc: https://stackoverflow.com/questions/16367770/my-rst-readme-is-not-formatted-on-pypi-python-org

Create a python script to convert relative links to absolute?


Add option to set minimum TLS version

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue, please leave a comment

Describe the Feature

While CRT provides a way to set the minimum TLS version, this is not currently exposed by the Swift SDK.
Is your Feature Request related to a problem?

Without the ability to enforce a minimum TLS version, apps with heightened or mandatory minimum security requirements become more complicated to deploy, especially if the app can't customize the execution environment.
Proposed Solution

Describe alternatives you've considered

Additional Context

[ ] I may be able to implement this feature request

Next steps

does CRT bindings have TLS version exposed
Create config for TLS version in HTTP configs


Update active-directory-certificate-based-authentication-ios.md

Remove reference to iOS in Android page

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

Thanks for the contribution, @gaubert-ms
#sign-off


TCVP-2239 Staff portal fix email resend verification

Description
This PR includes the following proposed change(s):

TCVP-2239
Fixed the not being able to resend email verification from staff portal issue by publishing the correct message.
Removed the unnecessary EmailVerificationSend message.

Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Refactoring / Documentation
[ ] Version change

if your change is a breaking change, please add breaking change label to this PR
How Has This Been Tested?
Ran the apps through docker and clicked on the resend email verification button in staff portal and confirmed the email verification email has been received.
Does the change impact or break the Docker build?

[ ] Yes
[x] No

If Yes: Has Docker been updated accordingly?

[ ] Yes
[ ] No

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have made corresponding changes to the documentation
[x] New and existing unit tests pass locally with my changes

please test more these changes do not appear to result in an email being sent

I tested this couple of times on my local and worked perfectly for me. Can you please specify your test steps for your failure case? There might be something out of sync on your local dev environment.


Consume page: 'Add to solution' should behave like hyperlink

System Information (please complete the following information):

Model Builder Version: <IP_ADDRESS>5801
Visual Studio Version

Describe the bug

On which step of the process did you run into an issue: Consume page
Clear description of the problem: The 'Add to solution' hyperlinks on the sample project boxes should behave like any other hyperlink. Currently, it is showing a blue background box around it on-hover.

To Reproduce
Steps to reproduce the behavior:

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

Expected behavior
"Add to solution" should have the same hyperlink properties as in other places in VS.
Screenshots
If applicable, add screenshots to help explain your problem.

Additional context
Add any other context about the problem here.

This issue can't be repro on main branch: https://privategallery.blob.core.windows.net/gallery/refs/heads/main/atom.xml

Windows 10 Enterprise, Version 20H2
ML.Net Model Builder (Preview): <IP_ADDRESS>2301
Microsoft Visual Studio Enterprise 2019: 16.9.4
.Net: 5.0.202

Steps：
1). Download data set from: https://raw.githubusercontent.com/dotnet/machinelearning/main/test/data/taxi-fare-train.csv
2). Create new C# console app with .Net 5.0;
3). Add model builder by right click on the project;
4). Click "Value prediction" on Scenario page to navigate to Environment page;
5). Select Local (CPU) on Environment page;
6). Select File and click "Browse..." to input the downloaded dataset, set "fare_amount" to "Column to predict (Label)" on Data page;
7). Click "Start training" with default 600s train time on Train page, and wait Training complete;
8). Click "Predict" button on Evaluate page;
9). Navigate to Consume page, find that the two buttons of 'Add to solution' have behaved like hyperlink.

@zewditu
Hyperlinks on hover should not have an underline. Please refer to VS UI Guidelines here.
https://devdivdesignguide.azurewebsites.net/windows/patterns/buttons-and-hyperlinks/#visual-style-2


Update pki docs

This branch partially answers the need for documenting external id claims described in issue https://github.com/openziti/ziti-doc/issues/411

Thank you for the thoughtful review. I'll close this in favor of a new attempt to document what the administrator needs to know to use external ID claims successfully.


[#5355] export Paimon jdbc user and password to properites

What changes were proposed in this pull request?
export paimon jdbc user and password to properites
Why are the changes needed?
Fix: #5355
Does this PR introduce any user-facing change?
no
How was this patch tested?
existing IT

@mchades please help to review, thanks

should we mask the jdbc user and password on the gravitino ui?

should we mask the jdbc user and password on the gravitino ui?

I think need. @LauraXia123 cc

should we mask the jdbc user and password on the gravitino ui?

I think need. @LauraXia123 cc

 the password is masked on ui


Simulations with Multiple Wave Directions

Hello,
Is it possible to run simulations with multiple wave directions and direction dependent elevation spectra? Ideally I might be able to define this data in a text file.
I'm aware there are some great updates coming to the HydroDyn module, perhaps this will be or is already a feature.
I've scoured the source code but can't really make sense of it. I see references to WaveMod==7 here and there but a bit lost.
Best,
Sam

Dear @Sam-Ramsahoye,
OpenFAST currently supports multidirectional sea states, but not directionally dependent wave spectra.  Basically, you can specify both a wave spectra (JONSWAP) and a directional spreading function (COS2S), but not a joint wave-direction spectra.
The upcoming release of OpenFAST v4.0 includes many hydrodynamic improvements (based on the following PR: https://github.com/OpenFAST/openfast/pull/1008), but directionally dependent wave spectra will still not directly be supported.  You mention the new WaveMod = 7 option which will be part of OpenFAST v4.0.  This feature will enable users to fully prescribe the sea state in the frequency domain by requiring the user to specify the height, direction, and phase of each wave component at each wave frequency.
Best regards,


Attaching listeners for standard DOM events

Is there a way to attach listeners for standard DOM events like copy, for example?
As far as I can see, this won't work as it's not a famous:event or a $dispatcher event:
events: {
  '#el': {
    'copy': function($dispatcher) {
      console.log(e);
   }
}

There doesn't seem to be a way to access the DOM node directly from with event. $dispatcher exposes a virtual DOM node for the host component but even event listeners attached to them do not show up once the DOM tree is created. $DOMElement doesn't expose a node so it's a no go either.
Any ideas?

Have you seen the previous issue? I'm not sure if it works for a copy event, but it works for an input event (and supposed to work for others).
https://github.com/Famous/framework/issues/36#issuecomment-117275706

input is a native UI event from famous:events
https://github.com/Famous/framework-guides/blob/develop/%24event.md#start-of-content

Oh right, I see.

@djgrant We're looking into this, checking with the Engine team to see the best way to do this in Engine.


Fix UVCal select method to handle total_quality_array

In the select method of UVCal, there is no downselect of the total_quality_array if present in the object. This results in an unchanged total_quality_array, which then fails the object self-check following the select (since the total_quality_array will have different dimensions than the main data array). If present, the total_quality_array should be changed along with the main data, similar to how input_flag_array is handled.

This was addressed in PR #213.


Ensure message order of broadcasts

Description
This changeset modifies the obcpbft broadcast.go to always send messages in the order in which they were received.
This is a companion to PR#1928 and incorporates some feedback from @corecode.
Motivation and Context
The previous implementation of the broadcaster would spawn a goroutine per broadcast request, which could wake up in an arbitrary order, delivering messages to recipients in that arbitrary order.  This caused problems for sections of the code which expected and depended on messages arriving in order.
One manifestation of this was request duplication as a request might be prepared and committed before the broadcast of its original contents was received, causing the network to view the request as new and invoke it twice.
The other problem was that these spawned go routines would stack and stack until the underlying gRPC stack would timeout and cause these calls to unwind and be garbage collected.
How Has This Been Tested?
The existing unit tests and CI, plus the busywork stress2b tool.
Checklist:

[X] I have added a Signed-off-by
[X] Either no new documentation is required by this change, OR I added new documentation
[X] Either no new tests are required by this change, OR I added new tests
[X] I have run goimports, go vet, and golint. I have cleaned up all valid errors and warnings in code I have added or modified. These tools may generate false positives.  The goal is clean, consistent, and readable code.

Signed-off-by: Jason Yellick<EMAIL_ADDRESS>

@corecode Commit 0051f52 should address your feedback.

@corecode please leave a LGTM comment if @jyellick has resolved all of your feedback. Thanks.

looks good

Thanks @corecode


pallas-primitives: fix PostAlonzoAuxiliaryData

I noticed a small difference between the cddl and the rust definition for PostAlonzoAuziliaryData.
I wasn't sure if it was intentional or a mistake but I made the adjustment and all tests still pass.
If this is intentional feel free to close this PR. It was a simple adjustment so no worries.

I don't think we should change this in Alonzo primitives. The plutus_v1 vs v2 was introduced in Babbage. The babbage primitives already contemplate that CDDL change: https://github.com/txpipe/pallas/blob/2b5f22def2a76341337417533b7116e13ac9880c/pallas-primitives/src/babbage/model.rs#L503

Yep, my bad. I must have gotten confused late at night. Closing now.


MixedFileUpload has inconsistent ReferenceCounted behavior when migrating from memory to disk

I can work on a patch for this. My proposed solution is to make MixedFileUpload use its own ref counting, like MemoryFileUpload already does. However this would be a big change to MixedFileUpload so I'm not sure about its compat implications.

Expected behavior
MixedFileUpload should behave like a normal ReferenceCounted object: Every retain should have a corresponding release that should succeed.
Actual behavior
Migration of MixedFileUpload from memory to disk, when the memory limit is reached, essentially resets the refCnt to 1, so if there were more retain calls before, their release calls will fail.
Steps to reproduce
Run the test case below. The test case passes if migration is not triggered, e.g. by removing the addContent line, or by changing the limitSize parameter to a higher value. If migration is triggered, the test case fails with:
io.netty.util.IllegalReferenceCountException: refCnt: 1, decrement: 2
	at app//io.netty.util.internal.ReferenceCountUpdater.retryRelease0(ReferenceCountUpdater.java:183)
	at app//io.netty.util.internal.ReferenceCountUpdater.nonFinalRelease0(ReferenceCountUpdater.java:167)
	at app//io.netty.util.internal.ReferenceCountUpdater.release(ReferenceCountUpdater.java:154)
	at app//io.netty.util.AbstractReferenceCounted.release(AbstractReferenceCounted.java:81)
	at app//io.netty.handler.codec.http.multipart.MixedFileUpload.release(MixedFileUpload.java:370)
	at app//com.example.MixedFileUploadTest.multipleRetain(MixedFileUploadTest.java:48)
...

Minimal yet complete reproducer code (or URL to code)
package com.example;

import io.netty.buffer.Unpooled;
import io.netty.handler.codec.http.multipart.MixedFileUpload;
import org.junit.jupiter.api.AfterEach;
import org.junit.jupiter.api.BeforeEach;
import org.junit.jupiter.api.Test;

import java.io.IOException;
import java.io.UncheckedIOException;
import java.nio.charset.StandardCharsets;
import java.nio.file.Files;
import java.nio.file.Path;

public class MixedFileUploadTest {
    private Path tmpDir;

    @BeforeEach
    public void setUp() throws IOException {
        tmpDir = Files.createTempDirectory("MixedFileUploadTest");
    }

    @AfterEach
    public void tearDown() throws IOException {
        Files.list(tmpDir).forEach(path -> {
            try {
                Files.delete(path);
            } catch (IOException e) {
                throw new UncheckedIOException(e);
            }
        });
        Files.delete(tmpDir);
    }

    @Test
    public void multipleRetain() throws IOException {
        MixedFileUpload mixedFileUpload = new MixedFileUpload(
                "foo", "foo", "foo", "foo",
                StandardCharsets.UTF_8,
                -1, 2, tmpDir.toString(), false
        );

        mixedFileUpload.retain();

        // trigger moving to disk
        mixedFileUpload.addContent(Unpooled.wrappedBuffer(new byte[5]), true);

        mixedFileUpload.release(2);
    }
}

Netty version
4.1.77.Final
JVM version (e.g. java -version)
openjdk version "17.0.1" 2021-10-19
OpenJDK Runtime Environment Temurin-17.0.1+12 (build 17.0.1+12)
OpenJDK 64-Bit Server VM Temurin-17.0.1+12 (build 17.0.1+12, mixed mode, sharing)

OS version (e.g. uname -a)
Linux yawkat-oracle 5.15.0-41-generic #44-Ubuntu SMP Wed Jun 22 14:20:53 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

MixedAttribute probably has the same issue

Oof! Yes, they should maintain their own independent reference counts, and in turn the references they have to their internal resources should only count as one reference.
I'll be happy to review a PR that fixes this!


run a flash page, a cmd window show NOT SANDBOXED

OS: win8 + 64
Electron: 0.35.4
app.commandLine.appendSwitch('ppapi-flash-path', __dirname + '/pepflashplayer.dll');
app.commandLine.appendSwitch('ppapi-flash-version', '<IP_ADDRESS>');

when open a page which have flash object. a cmd window show 'NOT SANDBOXED' than quickly closed auto.

how can i configure to stop show the cmd window

This is a duplicate of #1779.


R package has version <IP_ADDRESS> on release_0.90 branch

Should the version number be updated in DESCRIPTION?
https://github.com/dmlc/xgboost/blob/release_0.90/R-package/DESCRIPTION#L4

Oops, I forgot to update it, sorry about that. Let me fit it now.


Added roles_accepted on utilities and utility. Also added has_roles a…

…dmin and roles.

This is the result if you only have bronze role:

This is the result if you have admin as well as bronze role:

maybe we should add a call to add_update_bronze_data in check_bronze_kyc_level

maybe we should add a call to add_update_bronze_data in check_bronze_kyc_level
this is already updating without the add_update_bronze_data in the check_bronze_kyc_level.

This is in check_bronze_auth function.

which is removed with this PR

placed back the following:
                if not add_update_bronze_data(user):
                    if flash_it:
                        flash('Unable to update user KYC data', 'danger')
                    return False

And added:
add_update_bronze_data(current_user)

inside the check_bronze_kyc_level function.


DOC, RFC: better visibility for "algorithms" pages

#6767 yielded a few documentation pages that are live, but are not very findable; they are located in doc/overview/algorithms. Most are very good; a couple of them may need some cleanup / fleshing out. Find a way to make them more discoverable. Possibilities:

an algorithms TOC page that is linked from docs/overview/index.rst
combine the algorithms pages into one page that is linked from docs/overview/index.rst
merge the algorithms with the "implementation details" content
other ideas ???

cc @jasmainak

I don't quite like "implementation details" because it gives the impression that they are not important. Something I've realized after discussions with @mshamalainen is that we miss the mathematical details in MNE-Python.
Scikit-learn does a good job in their "user guide" and so does the MNE-C manual. But MNE-Python really skims over it. I shouldn't have to go read 5 papers to figure out what is the one equation of beamformers.

looks like in #6767 we also lost the io page, @agramfort might want to restore it?

Yeah we should probably restore it for release

There was a lot of overlap between the old pages but I didn't mean to remove the content. So yes restore plus deduplicate is the way to go. @drammock feel free to go for it on Monday, we should be able to release Tues or Wed hopefully


change copyright to personal

Can one of the admins verify this patch?

Pull Request Test Coverage Report for Build 1844

0 of 0   changed or added relevant lines in 0 files are covered.
9 unchanged lines in 4 files lost coverage.
Overall coverage increased (+0.02%) to 63.101%

Files with Coverage Reduction
New Missed Lines
%

core/config/sr_config.go
2
0.0%

eventlistener/loadbalance_event_listener.go
2
66.67%

core/server/options.go
2
50.0%

core/handler/bizkeeper_provider_handler.go
3
73.91%

Totals

Change from base Build 1839:
0.02%

Covered Lines:
5457

Relevant Lines:
8648

💛  - Coveralls


User/elormcoch/wsl extension fix

Adapts the launch and debug commands based on whether the extension is run in a remote environment.

@microsoft-github-policy-service agree company="Microsoft"

/azp run

@microsoft-github-policy-service rerun


Add read support for CST output files

We need this to be able to read in the output from many beam simulators.

This also needs to document the types of supported CST files (there are apparently many variations in type) and document any settings in CST output required to generate supported files.

I think we can call this done. We still need documentation on how to produce the right sort of files  from CST, but I'll add that as a new issue.


Add: one-line installer

That's the first iteration for CFSA-168. It works under assumption that https://api.opensuse.org/package/show/devel:sap:trento/trento repo is on the zypper list. When the trento is in the official GA of Update repository, it will work out of box, otherwise you would manually need to add the repo e.g. https://download.opensuse.org/repositories/devel:/sap:/trento/15.3/devel:sap:trento.repo
And here arises a question. Should adding the repository be included in the script, or we may assume the repository is always there?
The description of CFSA-168:
Create a script (or binary) so that the user can install trento in one line.
E.g, a curl with a bash script
We want to reduce to the bare minimum the parameters that the user will need to pass to the script, maybe by using an interactive approach but keeping the option to provide arguments via command line for an unattended installation, which can be also leveraged by the CI.
Acceptance Criteria

Make Agent installation a simply one liner
All content needs to be in the installation archive for offline use (air-gap support)

@aleksei-burlakov you can safely assume that https://download.opensuse.org/repositories/devel:/sap:/trento/ is the canonical download location.
For the time being, what the script should do is to add that repo and leverage zypper in to install the latest tagged version of the Trento Agent.
We definitely should not compile anything here, because that's done already either in the Makefile or in the rpm package itself.

@stefanotorresi @arbulu89 thank you for reviewing. I have amended the commit.

It adds the repository (now only for 15.3)
It installs the consul.
It doesn't compile anything.

The script shouldn't only install the option, it should most probably provide the options to start the tool (and this will need many different steps, like cleaning up current folders, creating others, etc)..

@stefanotorresi should the script also start the agent and/or web?

@stefanotorresi should the script also start the agent and/or web?

Definitely not. The agent will be handled by systemd. I'm not sure yet about the server part.

One more thing I forgot is that we clearly need to indicate somehow that this is covering just the Agent, and the Server installer will be implemented separately.

How-to try this script.
Scenario 1: there are two nodes Server (<IP_ADDRESS>) and Agent (<IP_ADDRESS>) with disabled firewalls.
On node S, install the consul and run it
$ wget https://releases.hashicorp.com/consul/1.9.5/consul_1.9.5_linux_amd64.zip
$ unzip consul_1.9.5_linux_amd64.zip
$ ./consul agent -server -bootstrap-expect=1 -data-dir=consul-data -ui -client <IP_ADDRESS> -bind <IP_ADDRESS>

On node C, run the script
$ curl -s https://raw.githubusercontent.com/trento-project/trento/23e5aea217296981453b30706f865d50d163f26f/install.sh | sudo bash -s -- --bind-ip <IP_ADDRESS> --server-ip <IP_ADDRESS>
(The url will change in future. If so, you can call the script as
$ sudo ./install.sh --bind-ip <IP_ADDRESS> --server-ip <IP_ADDRESS>
)
When the script has finished, one can also start the trento webserver
$ CONSUL_HTTP_ADDR=http://<IP_ADDRESS>:8500 trento web serve
then open <IP_ADDRESS>:8080 in the browser.

It's important to mention, that after the latest changes the script can't be used in a docker container because it uses the systemctl (docker containers don't support systemctl). This however should not be a problem, because the agent is not supposed to be used in containers.

@fabriziosestito  As agreed, in https://github.com/trento-project/trento/compare/4cce8ad4b9403b05a8d2981098195b9898a23591..a3882671b6b409ceec4d94a72557b609060163b2
are the changes:

Decrease the consul mentioning
Remove repeating installing of curl and which
Don't appent the -agent to the node name
Disable the services

The PR is continued in #226


fix: Left panel not scrollable on Admin settings page when the list is too long

Description

Correcting props in datasource auth component. Also, updating styles for left panel on Admin settings page to make it scrollable.

Fixes #18851
Type of change

Bug fix

How Has This Been Tested?

Correct props are getting passed due to this PR change. Also, left panel on Admin settings page is now scrollable.

Manual

Checklist:
Dev activity

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Test plan has been approved by relevant developers
[ ] Test plan has been peer reviewed by QA
[ ] Cypress test cases have been added and approved by either SDET or manual QA
[ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
[ ] Added Test Plan Approved label after reveiwing all Cypress test

/ok-to-test sha=22c16bd

/ok-to-test sha=45587e6

/ok-to-test sha=45587e6


HW4 - Group 1

LDA, GMM, SVM, and a little extra.

Beautiful! Amazing improvement for NN from 2 layers to 6 layers.


bug fix: stale rev still updates

this PR fixes #58
stale updates are still allowed - not treated as failure; but the latest modified and generation version intact (not to be affected), so that getting the latest value would always return the newest rev.
it comes with a unit test to ensure such behavior.

Are the changes to handle concurrent putting?
The comment is confusing since it removes the previous panic without adding errors. Therefore, users don't know if the index tree is updated or not.

for now this will be checked in. However, if in the future this stale-update-allowed approach lead to bugs in application or other server components, we will take other approach to err the stale update.


Make is_commutative method const and override it for some ops

Qualify is_commutative method with const
Override it for Equal, Minumum, NotEqual ops
Ensure that all overrides of the method are public

Migrated to #3278


Unable to load addon script

Hi!
Trying to import the addon (downloading it as a ZIP since there is no official releases) and I'm getting this error when trying to enable it:

I`m using Godot 4.0 RC2

Sorry, I forgot to mention that plugin is for Godot 3.5.1. I working on porting to version 4.

Oh just saw this.
Created this pull request with some updates. It's not complete yet but maybe it can help you a bit: https://github.com/eskandrej/Minstance/pull/6


Align help tags on the right. Put some tags on their own line.

The lines have 81 characters. These are should be less than or equal to 80.
Additionally, I think aligning tag to the right improve the readability but decrease maintainability.

@lambdalisue I only modified lines having *...* tags, and among them I can't find lines exceeding 80 characters. Could you please tell me what are the lines having 81 characters or more? The only ones I found are lines 341 and 404, but i didn't modify them as they don't have *...* tags.
Indeed, the purpose is to improve readability. Once this formatting is done. It should not be change in the future.

@gbcreation Please use vim-helplint.
https://github.com/machakann/vim-vimhelplint

Thanks for the linter. I fixed my last changes to make them in compliance with it.

LGTM

@gbcreation I will revert this change.  Please use tab characters instead of spaces.
Please see Vim help documents.

It must be left align instead of right.
I should check it properly though...

I have reverted it by hand :-(


Vimeo: Chokes on --add-metadata

youtube-dl  --ignore-config --verbose  --download-archive ~/.ytdlarchive --no-overwrites --call-home --continue --write-info-json --write-description --write-thumbnail --merge-output-format mkv--all-subs --sub-format srt --convert-subs srt --write-sub --add-metadata https://vimeo.com/70668043 https://vimeo.com/70666333

[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'--ignore-config', u'--verbose', u'--download-archive', u'/home/vxbinaca/.ytdlarchive', u'--no-overwrites', u'--call-home', u'--continue', u'--write-info-json', u'--write-description', u'--write-thumbnail', u'--merge-output-format', u'mkv--all-subs', u'--sub-format', u'srt', u'--convert-subs', u'srt', u'--write-sub', u'--add-metadata', u'https://vimeo.com/70668043', u'https://vimeo.com/70666333']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2016.01.27
[debug] Python version 2.7.10 - Linux-4.2.0-25-generic-x86_64-with-Ubuntu-15.10-wily
[debug] exe versions: ffmpeg 2.7.5-0ubuntu<IP_ADDRESS>, ffprobe 2.7.5-0ubuntu<IP_ADDRESS>, rtmpdump 2.4
[debug] Proxy map: {}
[debug] Public IP address: <IP_ADDRESS>
[vimeo] 70668043: Downloading webpage
[vimeo] 70668043: Extracting information
[vimeo] 70668043: Downloading webpage
[vimeo] 70668043: Downloading JSON metadata
[vimeo] 70668043: Downloading m3u8 information
[info] Video description is already present
[info] Video description metadata is already present
[vimeo] 70668043: Thumbnail is already present
[debug] Invoking downloader on u'https://10-lvl3-hls.vimeocdn.com/1453990861-28223b02a7d6053983227f4b64333f85d0240957/01/4133/2/70668043/178317076.mp4.m3u8'
[download] Ask Ash No. 1-70668043.mp4 has already been downloaded
[download] 100% of 9.83MiB
[ffmpeg] Adding metadata to 'Ask Ash No. 1-70668043.mp4'
[debug] ffmpeg command line: ffmpeg -y -i 'file:Ask Ash No. 1-70668043.mp4' -c copy -metadata 'comment=More | junnnktank.com/thenakedissue/faq
f. Ash twitter.com/ashvandeesch
This is Ash. She'"'"'s from Holland.  You ask her questions, she makes a video and answers them {while looking pretty damn cute}.
Ask Ash |<EMAIL_ADDRESS>-metadata 'description=More | junnnktank.com/thenakedissue/faq
f. Ash twitter.com/ashvandeesch
This is Ash. She'"'"'s from Holland.  You ask her questions, she makes a video and answers them {while looking pretty damn cute}.
Ask Ash |<EMAIL_ADDRESS>-metadata artist=JUNNNKTANK -metadata 'title=Ask Ash No. 1' -metadata date=20130719 -metadata purl=https://vimeo.com/70668043 'file:Ask Ash No. 1-70668043.temp.mp4'
ERROR: Conversion failed!
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/YoutubeDL.py", line 1737, in post_process
    files_to_delete, info = pp.run(info)
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/postprocessor/ffmpeg.py", line 395, in run
    self.run_ffmpeg(filename, temp_filename, options)
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/postprocessor/ffmpeg.py", line 159, in run_ffmpeg
    self.run_ffmpeg_multiple_files([path], out_path, opts)
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/postprocessor/ffmpeg.py", line 155, in run_ffmpeg_multiple_files
    raise FFmpegPostProcessorError(msg)
FFmpegPostProcessorError

Looks like it's failing on metadata embed, it's choking on --add-metadata

Here's the output of the ffmpeg command:
$ ffmpeg -y -i 'file:Ask Ash No. 1-70668043.mp4' -c copy -metadata date=20130719 -metadata 'description=More | junnnktank.com/thenakedissue/faq
f. Ash twitter.com/ashvandeesch
This is Ash. She'"'"'s from Holland.  You ask her questions, she makes a video and answers them {while looking pretty damn cute}.
Ask Ash |<EMAIL_ADDRESS>-metadata 'title=Ask Ash No. 1' -metadata artist=JUNNNKTANK -metadata purl=https://vimeo.com/70668043 -metadata 'comment=More | junnnktank.com/thenakedissue/faq
f. Ash twitter.com/ashvandeesch
This is Ash. She'"'"'s from Holland.  You ask her questions, she makes a video and answers them {while looking pretty damn cute}.
Ask Ash |<EMAIL_ADDRESS>'file:Ask Ash No. 1-70668043.temp.mp4'
ffmpeg version 2.8.5 Copyright (c) 2000-2016 the FFmpeg developers
  built with gcc 5.3.0 (GCC)
  configuration: --prefix=/usr --disable-debug --disable-static --disable-stripping --enable-avisynth --enable-avresample --enable-fontconfig --enable-gnutls --enable-gpl --enable-ladspa --enable-libass --enable-libbluray --enable-libdcadec --enable-libfreetype --enable-libfribidi --enable-libgsm --enable-libmodplug --enable-libmp3lame --enable-libopencore_amrnb --enable-libopencore_amrwb --enable-libopenjpeg --enable-libopus --enable-libpulse --enable-libschroedinger --enable-libsoxr --enable-libspeex --enable-libssh --enable-libtheora --enable-libv4l2 --enable-libvidstab --enable-libvorbis --enable-libvpx --enable-libwebp --enable-libx264 --enable-libx265 --enable-libxvid --enable-shared --enable-version3 --enable-x11grab
  libavutil      54. 31.100 / 54. 31.100
  libavcodec     56. 60.100 / 56. 60.100
  libavformat    56. 40.101 / 56. 40.101
  libavdevice    56.  4.100 / 56.  4.100
  libavfilter     5. 40.101 /  5. 40.101
  libavresample   2.  1.  0 /  2.  1.  0
  libswscale      3.  1.101 /  3.  1.101
  libswresample   1.  2.101 /  1.  2.101
  libpostproc    53.  3.100 / 53.  3.100
Input #0, mpegts, from 'file:Ask Ash No. 1-70668043.mp4':
  Duration: 00:01:05.50, start: 0.700000, bitrate: 1259 kb/s
  Program 1
    Stream #0:0[0x101]: Video: h264 (High) ([27][0][0][0] / 0x001B), yuv420p, 1280x720, 24 fps, 24 tbr, 90k tbn, 48 tbc
    Stream #0:1[0x102]: Audio: aac (LC) ([15][0][0][0] / 0x000F), 44100 Hz, stereo, fltp, 144 kb/s
[mp4 @ 0x559d96690260] Codec for stream 0 does not use global headers but container format requires global headers
[mp4 @ 0x559d96690260] Codec for stream 1 does not use global headers but container format requires global headers
Output #0, mp4, to 'file:Ask Ash No. 1-70668043.temp.mp4':
  Metadata:
    date            : 20130719
    description     : More | junnnktank.com/thenakedissue/faq
                    : f. Ash twitter.com/ashvandeesch
                    : This is Ash. She's from Holland.  You ask her questions, she makes a video and answers them {while looking pretty damn cute}.
                    : Ask Ash |<EMAIL_ADDRESS>    title           : Ask Ash No. 1
    artist          : JUNNNKTANK
    purl            : https://vimeo.com/70668043
    comment         : More | junnnktank.com/thenakedissue/faq
                    : f. Ash twitter.com/ashvandeesch
                    : This is Ash. She's from Holland.  You ask her questions, she makes a video and answers them {while looking pretty damn cute}.
                    : Ask Ash |<EMAIL_ADDRESS>    encoder         : Lavf56.40.101
    Stream #0:0: Video: h264 ([33][0][0][0] / 0x0021), yuv420p, 1280x720, q=2-31, 24 fps, 24 tbr, 90k tbn, 90k tbc
    Stream #0:1: Audio: aac ([64][0][0][0] / 0x0040), 44100 Hz, stereo, 144 kb/s
Stream mapping:
  Stream #0:0 -> #0:0 (copy)
  Stream #0:1 -> #0:1 (copy)
Press [q] to stop, [?] for help
[mp4 @ 0x559d96690260] Malformed AAC bitstream detected: use the audio bitstream filter 'aac_adtstoasc' to fix it ('-bsf:a aac_adtstoasc' option with ffmpeg)
av_interleaved_write_frame(): Operation not permitted
[mp4 @ 0x559d96690260] Malformed AAC bitstream detected: use the audio bitstream filter 'aac_adtstoasc' to fix it ('-bsf:a aac_adtstoasc' option with ffmpeg)
frame=    2 fps=0.0 q=-1.0 Lsize=       3kB time=00:00:00.13 bitrate= 151.7kbits/s
video:1kB audio:3kB subtitle:0kB other streams:0kB global headers:0kB muxing overhead: unknown
Conversion failed!

These lines are relevant:
[mp4 @ 0x559d96690260] Malformed AAC bitstream detected: use the audio bitstream filter 'aac_adtstoasc' to fix it ('-bsf:a aac_adtstoasc' option with ffmpeg)
av_interleaved_write_frame(): Operation not permitted
[mp4 @ 0x559d96690260] Malformed AAC bitstream detected: use the audio bitstream filter 'aac_adtstoasc' to fix it ('-bsf:a aac_adtstoasc' option with ffmpeg)

With NativeHlsFD, -bsf:a aac_adtstoasc is not applied in downloading. FFMpeg can't handle the downloaded file. @remitamine I see 'm3u8_native' is added in your commit e5c209a1bcea206bee684914599c84acf886487c. Do you have any reason for that change? I've tested a few Vimeo videos HlsFD using ffmpeg works.

If it helps, I downloaded that entire channel and encountered what I think may be a similar error but I just re-ran the rip (this is easy since I have it not clobber or retry already downloaded videos) and it passed over the problem video but this particular one wouldn't do it without dropping the --add-metadata flag.
92/94 videos on that channel completed without error.

@vxbinaca The problematic line is this one: https://github.com/rg3/youtube-dl/blob/ed7cd1e859cf97e975a28a5e8c58a1d1aca819fe/youtube_dl/extractor/vimeo.py#L459
Change the following lines in youtube_dl/extractor/vimeo.py
            formats.extend(self._extract_m3u8_formats(
                m3u8_url, video_id, 'mp4', 'm3u8_native', m3u8_id='hls', fatal=False))

to:
            formats.extend(self._extract_m3u8_formats(
                m3u8_url, video_id, 'mp4', m3u8_id='hls', fatal=False))

Works for me. However, the parameter 'm3u8_native' is added by another developer, so I have to ask his/her original intention before making changes.

Works for me. However, the parameter 'm3u8_native' is added by another developer, so I have to ask his/her original intention before making changes.

https://github.com/rg3/youtube-dl/pull/7126#discussion_r41700477

So let's call @dstftw: using NativeHlsFD by default is causing problems on ffmpeg like this. (OK with avconv v12_dev0-2305-g77a44f5). Is using ffmpeg to download HLS streams causing a different problem?

So do you suggest switching all m3u8_native to m3u8 in all (~45) extractors?

The problem is: if m3u8_native is specified by the extractor, there's no simple way to force HlsFD. Maybe adding an option --hls-prefer-ffmpeg solves all the problems? If we keep current framework, replacing all m3u8_native occurrences is necessary for this case.

Presence of --hls-prefer-ffmpeg does not really solve the problem for average user who use defaults, but it would be useful.

adding -bsf:a aac_adtstoasc to the command that add metadata if the protocol is m3u8_native will fix the problem.

Do you have any reason for that change?

for me hls native is better for two reasons(at least):

can continue the download
progress bar

I'll wait for the PR. I get my script from pip anyway and check twice daily for new copies.


How to create map with regulatoryElementLayer

Hi,
I am struggling to create a lanelet2 map with a regulatory element layer (mostly for priority on intersections).
Based on this I have created a unit test, but I do not know what is the correct way to create a fully-functional map.
I have tried to use it but haven't found in any of them way to create egulatoryElementLayer

JOSM
TIER4 vector map builder
TIER4 scenario editor

Could provide some information and tips on how to create a map?
Best,
Adam

Okay,
I have solved this and writing for future it's pretty simple and goes as follows:
  <relation id='3459134591' visible='true' version='1'>
  <tag k='type' v='regulatory_element' />
  <tag k='subtype' v='right_of_way' />
  <!-- Lanelets participating -->
  <member type ='relation' ref='34591' role='yield' />
</relation>


Breaks fabricjs selection control

Version
3.0.0
Reproduction link
https://jsfiddle.net/byd46gfp/1/
Steps to reproduce

Click the blue rectangle to reveal the selection controls. Notice those controls can't be clicked
Click button "Add" to add another rectangle; drag on canvas with mouse to form a region that contains both rectangles to select both rectangles; click empty space in canvas to deselect those rectangles; click any rectangle to select it. Notice the selection controls work this time, you can resize/rotate the rectangle with them.

What is expected?
Selection controls would always work, after step 1.
What is actually happening?
It doesn't work after step 1.

Note, it works with vue 2.x:  https://jsfiddle.net/byd46gfp/2/

I found that using markRaw can make it work correctly:
    this.canvas = Vue.markRaw(new fabric.Canvas(c, {
      width: c.clientWidth,
      height: c.clientHeight
    }));

Thanks, it works with markRaw.


World normals

Add worldNormal() access for both fragment and vertex shader. To easily detect up-pointing faces like roofs on tilted view.
Following of an issue related in https://github.com/tangrams/eraser-map/pull/56.
Once merged, update eraser-map stylesheet.

[ ] revert stylesheet once reviewed

What speaks against passing the untransformed vertex normal  to the polygon fragment shader?
The trade-off seems to be spending one uniform + a vector-matrix multiplication per fragment vs spending one varying (which is likely to be optimized out when not used)

I don't know, for me both solutions are fine, especially since one reduce the matrix mult, but this varying optimization is unclear since it seems to be driver specific, even if a warning is generated it's really unsure whether it'd be thrown away.

@hjanetzek either works, but @blair1618 and I were thinking that this feature is not used so often that pre-allocating an entire varying is worth it. If someone wanted to, they could create a varying themselves? If we find it's needed commonly, then we could decide to give up a varying for it, but since we may start to bump up against device limits for those, we wanted to be cautious.

Many mobile GPUs only have 8 interpolators and right now our shaders use 6 of those with built-in varying vectors. The cost of passing the un-transformed vertex normal into the fragment shader as a varying is that it further reduces the varyings available to a user. I agree that we don't know the exact trade-off of this yet, but since the user only sees the worldNormal() function, we're free to swap out the matrix multiplication for a varying in the future.

ok, the per fragment multiplication is probably not the expensive part of shaders using that feature.
Just had an optimization thought for our use: If one could set colors for the wall and flat top separately then the fragment shader could just check the vertex color :)

@hjanetzek, we've seen that used a few times, so may be worth putting in options for different vertex color. Some cases (including current Eraser Map style) actually still need the normal because they do more complex changes like change texture/material properties... which wouldn't be possible within the bounds of the current single-VBO architecture (could be considered in the future though).

@karimnaaji This looks good to go after that test style code is reverted.


Enable docusaurus website build by default

Change-type: patch
Signed-off-by: Kyle Harding<EMAIL_ADDRESS>

@balena-ci I self-certify!

I don't know if we do want any builds running until they have cloudflare_website input set

I don't know if we do want any builds running until they have cloudflare_website input set

cloudflare_website is still required as per
https://github.com/product-os/flowzone/blob/5db785a6485d6f163f813dc5767179eb9b6eced6/flowzone.yml#L1617
But I missed this on your original PR, we want to default to docusaurus, not custom build script. Your PR actually failed the e2e test because this repo is configured for docusaurus but we accidentally defaulted to false.

The docusaurus_website being true and false is an implementation detail only valid for one repository atm. Can you refer to what does default: true does in GitHub actions

If you don't provide that input when calling Flowzone, it will default to
true. This is what we want because before your PR that input didn't exist
and docusaurus was assumed true in all cases.
Now we have that input so repos with custom docs builds can manually set
docusaurus_website to false.
On Mon, Dec 19, 2022, 11:06 a.m. Vipul Gupta @.***>
wrote:

The docusaurus_website being true and false is an implementation detail
only valid for one repository atm. Can you refer to what does default:
true does in GitHub actions
—
Reply to this email directly, view it on GitHub
https://github.com/product-os/flowzone/pull/401#issuecomment-1357893218,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE4CWIBMWOSJY4IIZSGJRLLWOCBXXANCNFSM6AAAAAATBVRXPE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Yeah, I edited the message a bit once I looked at the commit more closely. I will open a followup PR to fix the README


New relic infrastructure alerts

Adding New Relic Infrastructure alert condition CRUD.
Added argument on update alerts conditions to take in the condition id.

Hi @mlvazqu2
Thank you so much for your contributions, awesome!
Thanks!
Chan Gao


Can't seem to get SPM working

As per screenshot, integration with Xcode SPM not working. Other packages working as expected.

Works if I specify the main branch rather than the current version 1.0.0

Hey @mightyleader, the issue was somehow because Chat used master branch of its dependency library. I switched to fixed version and now it works. Please check out version 1.0.3. Have a wonderful day!


(v4) Tooltip destruction incomplete

After destroying a tooltip, mousing over the element that used to host it triggers an error on line 1596.

This should be fixed now in rc32, thank you.

This does not appear to be solved yet. Still getting the error here: https://github.com/iamceege/tooltipster/blob/v4/js/jquery.tooltipster.js#L1607
"this.$emitter is null"

Got it, it was the mouseenter listener, fixed for real this time ;) Thank you.

Yep, that's got it.Thanks.


add mips64 reg

Checklist

[ ] Language changes are updated in docs/reference_guide.md
[ ] User-visible and non-trivial changes updated in CHANGELOG.md
[ ] The new behaviour is covered by tests

Could you update the CHANGELOG.md?

Also, bpftrace has architecture-specific tests for register access: https://github.com/iovisor/bpftrace/blob/master/tests/runtime/intptrcast . It would be great to add MIPS tests as well.

Yeah, some basic tests would be nice

@George-Hu  can you add in a changelog entry?
Tests would be nice too

#1708 for the changelog


fix: add cluster-admins to defualt argocd rbac policy

Signed-off-by: Yi Cai<EMAIL_ADDRESS>What type of PR is this?
/kind enhancement
What does this PR do / why we need it:
Have you updated the necessary documentation?

[ ] Documentation update is required by this PR.
[ ] Documentation has been updated.

Which issue(s) this PR fixes:
Report from a user:

Currently the following policy.csv is set to "g, system:cluster-admins, role:admin". This means when a cluster admin logs into openshift gitops via Dex/SSO they can't really do anything useful, and the errors when they do something like a Sync aren't obvious. A less astute openshift user wouldn't even realize that system:cluster-admin and cluster-admin are different groups. We should include cluster-admin as well as system:cluster-admin.
This is achievable with the following patch
oc patch argocd openshift-gitops -n openshift-gitops --type=merge -p='{"spec":{"rbac":
{"policy":"g, system:cluster-admins, role:admin\ng, cluster-admins, role:admin\n"}
}}'

Fixes GITOPS-1541
Test acceptance criteria:

[x] Unit Test
[ ] E2E Test

How to test changes / Special notes to the reviewer:

Have a cluster ready
Create a user using HTPasswd as instructions in this link
Follow the steps in this link to add the user  to cluster-admins group for Argo CD.
Install Openshift-Gitops operator and login to the default Argo CD instance with the new user created in step 2. Observe now the user is able to add, sync etc. apps as users in system: cluster-admins group.

/test v4.7-e2e

/test v4.7-e2e

/lgtm
/approve


Consider an expressJs reverse proxy

Consider an expressJs reverse proxy for wifi-connect (/wifi-conenct -> :9000). Or relay the request through the ExpressJS backend, to Wi-Fi connect. This may be better in the long run as WiFi-Connect can then eventually be behind a login page when it is implemented to express in the future.

I think it makes sense to do the proxying for wifi connect via express on a specific route, especially since it's only HTTP requests we need to worry about. The alternative to this would be to roll an nginx reverse proxy, but thats senseless burden running another container just to manage those calls.


New fixture - ETC-ColorSource-Spot

http://www.qlcplus.org/forum/viewtopic.php?f=6&t=10009
Physical and photometric data is from the ETC website (I have used maximum values).

Thanks


unable to run Make install

when i cloned the project  , i read the README , it said we need to run "make install" under the panther-analysis directory .but it is prompting error .
here is the error :
pipenv sync --dev
make: pipenv: No such file or directory
make: *** [Makefile:39: install] Error 127

@Danish-khalid it's been a while since you opened the issue, can you try again?


Integrate main to next branch

wip

@sonalivdeshpande Can I safely close this PR?

@sonalivdeshpande Can I safely close this PR?

yep


Create manual-setup.mdx

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

da
manual-setup.mdx
Localization added, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

Closing the PR following https://github.com/withastro/starlight/pull/2247#discussion_r1734259681


Cannot seem to install on MacOS

Hi on trying to run cargo install cargo-web  on macos I get the following error:
Compiling app_dirs v1.1.1
error[E0619]: the type of this value must be known in this context
--> /Users/rebo/.cargo/registry/src/github.com-1ecc6299db9ec823/app_dirs-1.1.1/src/imp/platform/macos.rs:10:40
|
10 |         Ok(Component::RootDir.as_ref().into())
|                                        ^^^^
error: aborting due to previous error
Any ideas?

Since it doesn't look like app_dirs crate will be fixed anytime soon I've forked it and pushed new cargo-web with a fix. So, closing!


Capitalize the H in GitHub

Why:
Because this is our brand name. I didn't change references to other people their projects if they spelled it with a lowercase h (mostly in the libraries.md file).
I didn't find any other forms of misspellings. It's possible there's a bunch of all lowercase "github" out there, since that's hard to search for with the URLs all over the place.
What's being changed:
Github -> GitHub.
Check off the following:

[ ] I have reviewed my changes in staging. (look for the deploy-to-heroku link in your pull request, then click View deployment)
[ ] For content changes, I have reviewed the localization checklist
[ ] For content changes, I have reviewed the Content style guide for GitHub Docs.

@anywherepilot Thanks so much for opening a PR! I'll get this triaged for review ⚡

@anywherepilot Thanks so much for opening a PR! I'll get this triaged for review ⚡

Thanks for your patience! Our small team is working our way through reviewing all of the amazing contributions ✨

Thanks for your patience! Our small team is working our way through reviewing all of the amazing contributions ✨


[Admin/Команда] A deleted team member can be edited

Environment: Windows 11,
Google Chrome Version 111.0.5563.147 (64-bit)
Reproducible: Always
Build found: 8490f607507e781d172f910b7e6859e64e055861
Preconditions

The site is open.
Logged as admin.
At least 1 team member has been added.

Steps to reproduce

Click on the 'Delete' button
Click 'OK' in the 'Confirmation' window.
Click on the 'Edit' button on the deleted team member.
Edit some fields and click the 'Зберегти' button.

Actual result

The member hasn't disappeared from the member's list.
There isn't a 'DELETE' request in the DevTools/an 'UPDATE' request crashed with 500 status code.

[ ] There are many identical requests - the cause of the overload needs to be investigated.

https://github.com/ita-social-projects/StreetCode/assets/135108144/a772ec6a-47f6-4581-85f8-540fd90806b5
https://github.com/ita-social-projects/StreetCode/assets/135108144/5036db1d-0684-4416-b6d0-43f4fa0f84a1
https://github.com/ita-social-projects/StreetCode/assets/135108144/220a43b2-fb14-4b2b-b5bd-549f9a10a788
Expected result
The member has disappeared from the member's list immediately after the delete option.
Same problem with objects on pages:

[ ] Partners
[ ] Для фанатів

User story and test case links
User story #834 #833 #499
[Test case]

The issue no longer occurs.

The issue no longer occurs


InvalidSignatureException calling lambda with function_arn

I'm successfully creating a lambda, but as soon as I tried to invoke it I was getting an error:
DEBUG 2018-04-17T23:20:49Z: hyper::client::pool: pooling idle connection for "https://lambda.eu-west-1.amazonaws.com"
DEBUG 2018-04-17T23:20:49Z: tokio_core::reactor: scheduling direction for: 1
DEBUG 2018-04-17T23:20:49Z: tokio_core::reactor: blocking
Err(Unknown("{\"message\":\"The request signature we calculated does not match the signature you provided. Check your AWS Secret Access Key and signing method. Consult the service documentation for details.\"}"))
DEBUG 2018-04-17T23:20:49Z: tokio_core::reactor: scheduling direction for: 1
DEBUG 2018-04-17T23:20:49Z: tokio_core::reactor: blocking
DEBUG 2018-04-17T23:20:49Z: tokio_core::reactor: loop process - 1 events, Duration { secs: 0, nanos: 4066964 }
DEBUG 2018-04-17T23:20:49Z: tokio_core::reactor: loop poll - Duration { secs: 0, nanos: 33803 }
DEBUG 2018-04-17T23:20:49Z: tokio_core::reactor: loop time - Instant { tv_sec: 746752, tv_nsec: 335893326 }

inspired by #997 I changed the function_arn I received from create_function to just the function name I provided and it I can invoke my lambda that way.
This work as a workaround, but I think there's a bug in the signature calculation.

Hmm, interesting! That does look like it's similar to #997.

In order to reproduce this, do you think you could share the code that produces this error?

@srijs You can reliably reproduce this by calling either get_function or create_function, then take the function_arn from the response (eg. FunctionConfiguration):
let x = self.create_function(&CreateFunctionRequest {
    code: FunctionCode {
        zip_file: Some(zip),
        ..Default::default()
    },
    function_name: function_name.into(),
    handler: "client.run".to_string(),
    role: role.into(),
    runtime: "python3.6".to_string(),
    ..Default::default()
})?;

let function_arn = x.function_arn.unwrap();

and then try to invoke it:
let r = self.invoke(&InvocationRequest {
    function_name: function_arn,
    payload: Some(payload.into_bytes()),
    ..Default::default()
})?;

I've uploaded my full (working) implementation to github if that helps (it's quite long, sorry. The readme on the lambda branch contains instructions how to run it): https://github.com/kpcyrd/boxxy-rs/blob/8b756ee23666c74bab2c96de3360d5556e18e7c4/examples/lambdash.rs
This patch changes my invoke call from the user supplied function name to the full function_arn returned by either create_function or get_function. This reliably triggers error: {"message":"The request signature we calculated does not match the signature you provided. Check your AWS Secret Access Key and signing method. Consult the service documentation for details."} .
diff --git a/examples/lambdash.rs b/examples/lambdash.rs
index d6b5a58..cf31f8d 100644
--- a/examples/lambdash.rs
+++ b/examples/lambdash.rs
@@ -365,7 +365,7 @@ fn run() -> Result<()> {
                     },
                 };

-                match lambda.invoke_boxxy(args.function_name.as_str(), &cmd)? {
+                match lambda.invoke_boxxy(function_arn.as_str(), &cmd)? {
                     ResponseResult::Ok(resp) => {
                         stdout.write_all(&resp.stdout)?;
                         stdout.flush()?;

Thanks for the info, @kpcyrd!
Digging into this issue, it seems like it has to do with the way that we compute the canonical URI during the signing process.
When I generate a signature using the Node.js SDK for an ARN (arn:aws:lambda:us-east-1:XXX:function:ExampleFunction), it double-escapes the URI to /2015-03-31/functions/arn%253Aaws%253Alambda%253Aus-east-1%253AXXX%253Afunction%253AExampleFunction%2522, while rusoto "only" single-escapes to /2015-03-31/functions/arn%3Aaws%3Alambda%3Aus-east-1%3AXXX%3Afunction%3AExampleFunction.
I found an issue on the Node.js SDK (https://github.com/aws/aws-sdk-js/issues/853) which seems to indicate that URI escaping is handled slightly differently for S3 vs. other services, which explains why we have an integration test for S3 that successfully checks whether we correctly handle unicode in path names, but it doesn't work for Lambda here (and likely other services).
Albeit related, itt also doesn't look like a proper duplicate of #997, so I'm going to keep these two issues separate for now.

Currently getting this with cognito as well (pretty much identical error).

@kitsuneninetails which cognito crate/function call specifically? It seems the way to work around this is avoiding some special characters that are not encoded correctly. That may or may not work in your usecase (depends on if you need full ARNs).

Okay, this is going to seem whacky, but it now works.  No code changes on my part (although I was fiddling with a lot of debug printing, etc., but even after removing that, it now works).  This makes me wonder if it's a timing thing, or if I'm logged into the console, maybe it won't work, etc.

Should be done! 💯


control.layers doesn't redraw layer without changing zoom (1.0.0-rc.1; Chrome 51.0.2704.79 and FF 46.0.1)

Thank you all for maintaining and expanding this wonderful product. I will admit that I am using ESRI's Leaflet plugin to generate my layers so this might be an upstream error.
I have a map with several large layers. They are all controlled via control.layers and are not added to the map automatically. If I turn a layer on it shows up like it supposed to. However, when I turn that layer off and then on again, it does not show up. The only way I can get the layer to redraw is to zoom out which triggers a redraw of the map.
If this isn't a vanilla leaflet issue, would it be possible to get a map.redraw() method that just redraws everything currently on the map at the same zoom and center point? Something like that would fix my issue and I can see it being useful for others.
thanks!

Can you create an example of the problem? I tried in the playground below, but the layer shows up every time.
http://playground-leaflet.rhcloud.com/rof/2/edit?html,console,output

Also see map.invalidateSize() to redraw the map.

@jieter I will see what I can do.The data that I am using can't be used on the internet due to NDAs and such. I will see if I can draw upon a dummy dataset. It could just have to do with using too much memory.
@nathancahill I will look into that. Thanks!

@Guerillero What kind of data are you trying to display? Are you using some dynamic layers from the esri plugin maybe?

@IvanSanchez Yes, I am using several dynamic layers using the ESRI plugin and my organization's ArcsServer.

ESRI Leaflet issue. Thanks for the help


Remove sRGB option

Fixes #506

SonarQube analysis reported 1 issue
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 AncientFungus:LGEPR: Cppcheck cannot find all the include files (use --check-config for details)


How to apply "common" fine-grained access control to API (AppSync)

Hi all.
I am currently working on a project which requires a use-case that is similar to a lot of other applications I have been considering to migrate to Amplify. However, the only showstopper right now, is that there does not seem to be a "clean" way of implementing fine-grained access control without pipeline resolvers and additional lookup tables. While I understand a lookup table will be necessary, I do have a feeling that the AppSync API itself will get quite "messy".
To get a better feeling of what I am trying to do, I will provide a GraphQL example with Amplify transformers.
type Computer
  @model
  @auth(rules: [
    { allow: owner, ownerField: "owner" },
    { allow: owner, ownerField: "editors", operations: [read, update]}
    { allow: owner, ownerField: "viewers", operations: [read]}
  ]) {
  id: ID!
  owner: String
  editors: [String]
  viewers: [String]
  name: ID!
  computerConfigurations: [ComputerConfiguration] @hasMany
}

type ComputerConfiguration
  @model
  @auth(rules: [
    { allow: owner, ownerField: "owner" },
    { allow: owner, ownerField: "editors", operations: [read, update, create]}
    { allow: owner, ownerField: "viewers", operations: [read]}
  ]) {
  id: ID!
  owner: String
  editors: [String]
  viewers: [String]
  cpu: Int
  memory: Int
  hdd: Int
}

So this example above illustrates two models, Computer and ComputerConfiguration. Computer has a @hasMany relation to ComputerConfiguration.
When a user creates a Computer entry, he/she can then add another user as either an editor or viewer to that entry. This seems to be straightforward. However problems quickly creep up because there is a relationsship between Computer and ComputerConfiguration where the same access control does not co-exist, and it also seems to be non-trivial to implement this replicated access-control entries to the same editors and viewers without adding a lambda function of some sort.
Is there something I am missing here in my considerations and am I trying to do something which is becoming quite cumbersome for AppSync to handle in general compared to a bare metal API implementation?
Thanks in advance.
Which Category is your question related to?
GraphQL Access Control
Amplify CLI Version
7.4.2
What AWS Services are you utilizing?

AppSync
Cognito
DynamoDB

Provide additional details e.g. code snippets

Hello, the behavior you are describing is supported by using the @auth and @hasMany directives. Using the secondary owner fields to grant access to records seems to solve what you are describing because it scopes access to what has been specified by the record (ie. to editors and viewers), regardless of relation to other records.
Can you be more specific in exactly what you are asking? Thank you.

Hello, the behavior you are describing is supported by using the @auth and @hasMany directives. Using the secondary owner fields to grant access to records seems to solve what you are describing because it scopes access to what has been specified by the record (ie. to editors and viewers), regardless of relation to other records.
Can you be more specific in exactly what you are asking? Thank you.

So what I was looking for here was applying the same permissions from Computer on to ComputerConfiguration, I do understand that I can access the ComputerConfiguration records from the Computer and that way utilize the permissions from the parent.
But instead  lets assumes that someone in  editors would want to list all ComputerConfiguration he/she has access, how can this be done?

Hi again @danielleadams. I actually made a typo in the GraphQL schema, which has been corrected. editors in ComputerConfiguration have create.

@maziarzamani I think what you can do is use a listComputerConfiguration API call from the editor's client, which you will get from that read operation, and if you want to even include the associated Computer, you can add a @belongsTo directive on the ComputerConfiguration type. Is that the behavior you are looking for?

@maziarzamani I think what you can do is use a listComputerConfiguration API call from the editor's client, which you will get from that read operation, and if you want to even include the associated Computer, you can add a @belongsTo directive on the ComputerConfiguration type. Is that the behavior you are looking for?

Hi again @danielleadams. This is where problems start to creep up. Because the editor entries on ComputerConfiguration do not really exist, unless I somehow replicate these entries in a pipeline resolver manually from the Company object. When a Computer object has been created, I basically need the same editors and viewers to exist in the @hasMany relations. You could call it a form of inheritance of the permissions. Does that make sense?

Hey @maziarzamani :wave: apologies for the delay! I've been taking a further look at this one and wanted to clarify whether we are looking to duplicate the editors and viewers from the newly created Computer object onto a new ComputerConfiguration when it is created?

Hey @maziarzamani 👋 apologies for the delay! I've been taking a further look at this one and wanted to clarify whether we are looking to duplicate the editors and viewers from the newly created Computer object onto a new ComputerConfiguration when it is created?

Hey @josefaidt! Yes that is correct, and also reflect the changes on those two permission arrays on updates, which is my biggest concern as it is a 1-to-many relationsship, so several ComputerConfiguration entries may been updating.

Hey @maziarzamani thank you for clarifying. Unfortunately I do not believe this is currently supported but seems like a good feature request to enhance the @default directive to inherit values from the parent. Say we have the following schema:
type Post
  @model
  @auth(
    rules: [
      { allow: owner, ownerField: "owner", identityClaim: "email" }
      {
        allow: owner
        ownerField: "editors"
        identityClaim: "email"
        operations: [read, update, delete]
      }
      {
        allow: owner
        ownerField: "viewers"
        identityClaim: "email"
        operations: [read]
      }
    ]
  ) {
  id: ID!
  editors: [String]
  viewers: [String]
  title: String!
  comments: [Comment] @hasMany(indexName: "byPost", fields: ["id"])
}

type Comment
  @model
  @auth(
    rules: [
      { allow: owner, ownerField: "owner", identityClaim: "email" }
      {
        allow: owner
        ownerField: "editors"
        identityClaim: "email"
        operations: [read, update, delete]
      }
      {
        allow: owner
        ownerField: "viewers"
        identityClaim: "email"
        operations: [read]
      }
    ]
  ) {
  id: ID!
  postID: ID! @index(name: "byPost", sortKeyFields: ["content"])
  post: Post @belongsTo(fields: ["postID"])
  editors: [String]
  viewers: [String]
  content: String!
}

Given the current functionality, we can either rely on our frontend to appropriately set values from the parent, or set a pipeline resolver or Lambda to set these values to enforce the inheritance from our backend.
// frontend comment creation
async function handleOnCommentSubmit(event, post) {
    event.preventDefault()
    const content = event.target.content.value
    const id = await createComment({
      content,
      postID: post.id,
      editors: post.editors,
      viewers: post.viewers,
    })
  }

Marking this as a feature request 🙂 to enforce this inheritance from the schema

Thanks @josefaidt!

Is there a workaround I can utilize while waiting for the spaceship to arrive? I can replicate the secondaryOwner entries from Computer down to ComputerConfiguration manually when creating the entries, but what do I do when a secondaryOwner is removed? I can have thousands of entries that need to be manually updated, and the same goes for new secondaryOwners which needs to be able to see entries that existed before they were added? I am a bit surprised that this type of ACL is not supported out of the box.


Update action.yml

Adds cleanup mechanic to the artifact-name. This replaces unsupported characters with an underscore.

Tested on https://github.com/SkylineCommunications/BOOST-Test-SDK-Automation/actions/runs/10937380694  and on
https://github.com/SkylineCommunications/BOOST-Test-SDK-Automation/actions/runs/10937456411
both linux & windows. Using github.repository and without.


fix: banner shouldn't be injected to asset

Summary
rspack's wrong behavior of injecting banner into asset is because rspack doesn't distinguish between auxiliaryFiles and files.

Test Plan

need distinguish auxiliaryFiles and files.
https://github.com/webpack/webpack/blob/8241da7f1e75c5581ba535d127fa66aeb9eb2ac8/lib/Compilation.js#L4643

!canary

!bench


Suboptimal overflow of author list (xml2rfc-3.24.0)

Describe the issue
Chaode Yu's entry in the coverpage authors list is somewhat awkwardly wedged to the side in the PDF form::

(Since the datatracker inexplicably does not generate the PDF publication form, I created it today with xml2rfc 3.24.0; this is not the date of the .xml.)
This document is nearly done, so the RFC editor will see it soon.
Code of Conduct

[x] I agree to follow the IETF's Code of Conduct

This seems the same as https://github.com/ietf-tools/xml2rfc/issues/702


feat: A new parameter model called "native_tool_call" is added.

(squashed commit for easier review)
A new parameter model called "native_tool_call" is added.
If false (the default), the behaviour when there are tools enabled doesn't change: a prompt is built which asks the model to generate a function call compatible with the offered tools. If it's valid the ouput is the added to the context.
If true, the native mechanism that the API has is used instead. That means that the responses have to be inspected to detect if a tool_call has to be handled and act accordingly. This allows more complex  behaviours where the model can try several strategies and call tools knowing the output of others.
Ollama tool calling API only supports non streaming requests. In case both streaming and native_tool call are present, native tool calling is silently ignored.
Testing
The feature has been tested through the ui and using a script which calls these endpoints using
/api/chat/completions
/ollama/api/chat
With the OpenAI API, these models have been tested, with varying success

OK
gpt-4o-mini
qwen/qwen-2.5-72b-instruct (OpenRouter endpoint)
anthropic/claude-3-haiku  (OpenRouter endpoint)
deepseek/deepseek-chat  (OpenRouter endpoint)
google/gemini-pro  (OpenRouter endpoint)
Partial support
cohere/command-r-plus (OpenRouter endpoint)

-Fails
meta-llama/llama-3.1-70b-instruct  (OpenRouter endpoint)
With the Ollama API, only  llama3.2:3b has been tested.
These are  prompts used with two functions provided:

get_current_time
get_sensor_data with a single argument which just returns the sensorid multiplied by two.

("what is the first letter of the latin alphabet",  "no tools needed"),
("what is the date of today (use the get_current_time function)", " 1 tool no args"),
("what is the value of sensor 1", " 1 tool with args"),
("what are the value of sensors 1 and 4", " 2 tools (the may be run in parallel if the API supports it )"),
("if the value of sensors 1 is less that 5 m/s report the value of sensor 4. Otherwise report sensor 3", " 2 tools (in s
equence if the model does it )"),
("choose a integer between 1 an 10 write it here. If it's bigger that 5 report sensor 4. Otherwise report sensor 1", " tool call after having generated some text"),
("what is the value of sensor 'HELLO' ",  "tool call raises an exception")

The full output is attached to the PR.
Before submitting, make sure you've checked the following:

[X] Target branch: Please verify that the pull request targets the dev branch.
[X] Description: Provide a concise description of the changes made in this pull request.
[X] Changelog: Ensure a changelog entry following the format of Keep a Changelog is added at the bottom of the PR description.
[-] Documentation: Have you updated relevant documentation Open WebUI Docs, or other documentation sources?
If the PR gets accepted, I will update the docs accordingly.
[X] Dependencies: Are there any new dependencies? Have you updated the dependency versions in the docume
ntation?
[X] Testing: Have you written and run sufficient tests for validating the changes?
[X] Code review: Have you performed a self-review of your code, addressing any coding standard issues and ensuring adherence to the project's coding standards?
[X] Prefix: To cleary categorize this pull request, prefix the pull request title, using one of the following:

BREAKING CHANGE: Significant changes that may affect compatibility
build: Changes that affect the build system or external dependencies
ci: Changes to our continuous integration processes or workflows
chore: Refactor, cleanup, or other non-functional code changes
docs: Documentation update or addition
feat: Introduces a new feature or enhancement to the codebase
fix: Bug fix or error correction
i18n: Internationalization or localization changes
perf: Performance improvement
refactor: Code restructuring for better maintainability, readability, or scalability
style: Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc.)
test: Adding missing tests or correcting existing tests
WIP: Work in progress, a temporary label for incomplete or ongoing work

Changelog Entry
Description
A new parameter model called "native_tool_call" is added. If set to true, the tool calls are performed using the API and not the prompt based approach used until now.
output.txt

If this does what it says on the tin, it's likely the best PR ever.

Closing due to conflicts, feel free to reopen!

Closing due to conflicts, feel free to reopen!

Excuse me @tjbck , but I would need some guidance about what's the problem and how could I fix it. Should I try to merge the changes against a newer version of the dev branch? Thank you

Yes.  Dev says merge conflict. Translation: "we changed some stuff and your code no longer fits cleanly. Please reapply your changes to the latest codebase.".

Thanks Adam. I'll try to fix it and make a new PR.


HEAD request failed with return View (version > 0.21)

HEAD request works fine with return "string" or something
but 500 error code when I return View["name"] from module.
Get["/"] = parameters =>
{
//return "meme"; //it works fine
return View["index"]; // 500 error
};

Nancy.Hosting.Aspnet
and here is nancy razor view

NancyApplication1
<style type="text/css">
    body {
        text-align: center;
    }
</style>

    blah blah

OK, I think you need to give us more than that. There are hundreds running Nancy on the ASP.NET host returning views like that without getting 500 errors. Do you have a stack trace? Anything?

Unfortunately not, but I can attach the demo project here

https://www.dropbox.com/s/7bdassxi4b14k1z/NancyApplication1.zip?dl=0

any lucky with my sample project above?

Haven't had time to test it yet. It would be super helpful if you could get a hold of a stack trace. You could try turning on request tracing and see if that gives anything.

at the moment, I can only get trace log info from xml file

{00000000-0000-0000-5E00-0080000000FB}
ManagedPipelineHandler
128
500
Internal Server Error
0
0

This sounds more like an IIS issue.  Can you issue other non-GET/POST requests?

Yes, the problem is I can do HEAD request with version 0.21. and with version >0.21 no problem with HEAD request if I return string or json from Module. but when I return View[], I could not do HEAD request.

It also happens on my server, not just IIS express that I did testing

I also tried the sample project on different pc/mac. the problem was still persisted

This is detailed error
Event code: 3005
Event message: An unhandled exception has occurred.
Exception information:
Exception type: NullReferenceException
Exception message: Object reference not set to an instance of an object.
at Nancy.Responses.MaterialisingResponse.<.ctor>b__0(Stream stream)
at Nancy.HeadResponse.CheckAndSetContentLength(Response response)
at Nancy.HeadResponse..ctor(Response response)
at Nancy.Routing.DefaultRequestDispatcher.<>c__DisplayClass2.b__1(Task1 completedRouteTask)    at Nancy.Helpers.TaskHelpers.WhenCompleted[T](Task1 task, Action1 onComplete, Action1 onFaulted, Boolean execSync)
at Nancy.Routing.DefaultRequestDispatcher.<>c__DisplayClass2.b__0(Task1 completedTask)    at Nancy.Helpers.TaskHelpers.WhenCompleted[T](Task1 task, Action1 onComplete, Action1 onFaulted, Boolean execSync)
at Nancy.Routing.DefaultRequestDispatcher.Dispatch(NancyContext context, CancellationToken cancellationToken)
at Nancy.NancyEngine.<>c__DisplayClass7.b__5(Task1 t)    at Nancy.Helpers.TaskHelpers.WhenCompleted[T](Task1 task, Action1 onComplete, Action1 onFaulted, Boolean execSync)
at Nancy.NancyEngine.InvokeRequestLifeCycle(NancyContext context, CancellationToken cancellationToken, IPipelines pipelines)
at Nancy.NancyEngine.HandleRequest(Request request, Func2 preRequest, CancellationToken cancellationToken)    at Nancy.NancyEngineExtensions.HandleRequest(INancyEngine nancyEngine, Request request, Func2 preRequest, Action1 onComplete, Action1 onError, CancellationToken cancellationToken)
at Nancy.Hosting.Aspnet.NancyHandler.ProcessRequest(HttpContextBase context, AsyncCallback cb, Object state)
at Nancy.Hosting.Aspnet.NancyHttpRequestHandler.BeginProcessRequest(HttpContext context, AsyncCallback cb, Object state)
at System.Web.HttpApplication.CallHandlerExecutionStep.System.Web.HttpApplication.IExecutionStep.Execute()
at System.Web.HttpApplication.ExecuteStep(IExecutionStep step, Boolean& completedSynchronously)
Request information:
Request URL: http://localhost:12116/
Request path: /
User host address: ::1
User:
Is authenticated: False
Authentication Type:
Thread account name: dev-PC\dev
Thread information:
Thread ID: 6
Thread account name: dev-PC\dev
Is impersonating: False
Stack trace:    at Nancy.Responses.MaterialisingResponse.<.ctor>b__0(Stream stream)
at Nancy.HeadResponse.CheckAndSetContentLength(Response response)
at Nancy.HeadResponse..ctor(Response response)
at Nancy.Routing.DefaultRequestDispatcher.<>c__DisplayClass2.b__1(Task1 completedRouteTask)    at Nancy.Helpers.TaskHelpers.WhenCompleted[T](Task1 task, Action1 onComplete, Action1 onFaulted, Boolean execSync)
at Nancy.Routing.DefaultRequestDispatcher.<>c__DisplayClass2.b__0(Task1 completedTask)    at Nancy.Helpers.TaskHelpers.WhenCompleted[T](Task1 task, Action1 onComplete, Action1 onFaulted, Boolean execSync)
at Nancy.Routing.DefaultRequestDispatcher.Dispatch(NancyContext context, CancellationToken cancellationToken)
at Nancy.NancyEngine.<>c__DisplayClass7.b__5(Task1 t)    at Nancy.Helpers.TaskHelpers.WhenCompleted[T](Task1 task, Action1 onComplete, Action1 onFaulted, Boolean execSync)
at Nancy.NancyEngine.InvokeRequestLifeCycle(NancyContext context, CancellationToken cancellationToken, IPipelines pipelines)
at Nancy.NancyEngine.HandleRequest(Request request, Func2 preRequest, CancellationToken cancellationToken)    at Nancy.NancyEngineExtensions.HandleRequest(INancyEngine nancyEngine, Request request, Func2 preRequest, Action1 onComplete, Action1 onError, CancellationToken cancellationToken)
at Nancy.Hosting.Aspnet.NancyHandler.ProcessRequest(HttpContextBase context, AsyncCallback cb, Object state)
at Nancy.Hosting.Aspnet.NancyHttpRequestHandler.BeginProcessRequest(HttpContext context, AsyncCallback cb, Object state)
at System.Web.HttpApplication.CallHandlerExecutionStep.System.Web.HttpApplication.IExecutionStep.Execute()
at System.Web.HttpApplication.ExecuteStep(IExecutionStep step, Boolean& completedSynchronously)
Custom event details:

Apparently PreExecute does not get called on the MaterialisingResponse before it is used by the whatever HEAD response there is

I have a repro for this. Will try track down the problem.


Fixes broken travis badge, adds codecov one

Fixes the styleCI badge for master, adds the codecov one.

Codecov Report

Merging #11 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #11   +/-   ##
=========================================
  Coverage     67.46%   67.46%
  Complexity       37       37
=========================================
  Files             9        9
  Lines           126      126
=========================================
  Hits             85       85
  Misses           41       41

Continue to review full report at Codecov.

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


Adjustments made for 1.1.4

To validate this pull request against other sbt modules, please visit this link.


Navigator Chevron Cursor Fix

Signed-off-by: Sahil Suman<EMAIL_ADDRESS>Description
Added cursor property for changing cursor to a pointer
This PR fixes #4659
Notes for Reviewers
Signed commits

[ ] Yes, I signed my commits.

@warunicorn19 Please check it and merge it

Codecov Report

Merging #4713 (a26377a) into master (3205aea) will decrease coverage by 6.97%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4713      +/-   ##
==========================================
- Coverage   32.83%   25.86%   -6.98%
==========================================
  Files          61       61
  Lines        5107     5107
==========================================
- Hits         1677     1321     -356
- Misses       2991     3457     +466
+ Partials      439      329     -110

Flag
Coverage Δ

gointegrationtests
?

unittests
25.86% <ø> (ø)

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

Impacted Files
Coverage Δ

mesheryctl/internal/cli/root/system/start.go
1.11% <0.00%> (-58.52%)
:arrow_down:

mesheryctl/internal/cli/root/system/restart.go
2.50% <0.00%> (-47.50%)
:arrow_down:

mesheryctl/internal/cli/root/system/status.go
0.00% <0.00%> (-47.44%)
:arrow_down:

mesheryctl/internal/cli/root/system/update.go
1.05% <0.00%> (-34.74%)
:arrow_down:

mesheryctl/internal/cli/root/system/stop.go
1.78% <0.00%> (-23.22%)
:arrow_down:

mesheryctl/internal/cli/root/system/check.go
1.39% <0.00%> (-22.38%)
:arrow_down:

mesheryctl/internal/cli/root/system/logs.go
3.29% <0.00%> (-20.88%)
:arrow_down:

Continue to review full report at Codecov.

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


Switch to react-router

Summary
Switching from rails routes to using react-router.
https://github.com/cerner/terra-ui.com/issues/22
https://github.com/cerner/terra-ui.com/issues/21

Spoke offline with @bjankord. Going to switch from react-router hash history to browser history.


Compat.stderr != STDERR

I've been seeing some weird buffering in IJulia recently (notably with ProgressMeter.jl, which uses Compat.stderr), and I suspect it's because Compat.stderr == STDERR is false in IJulia on 0.6 (but true at the REPL). Likewise for STDOUT/STDIN.

It seems there are a lot of problems with redirection interacting with code that copies a reference to stderr.  See also e.g. JuliaLang/julia#26798.
I wonder if a more general solution would be to make stdout and stderr in Julia a mutable container along the lines of:
mutable type IOContainer
    io::IO
end
const stdout = IOContainer(...real stdout object...)

with redirect_stdout mutating stdout.io = ...new IO....   That way existing references to stdout would be updated.

But in the meantime IJulia probably needs to rework how it redirects I/O.

@cstjean, have you already checked whether this was fixed on IJulia master by #641?

Yes, that's the version I'm on.

Looks like the problem is that #641 hard-coded redirect_stdout to directly change STDOUT.

Might be unrelated, but this looks wrong:

I don't know how Compat.stdout manages to track STDOUT at the REPL inside of redirect_stdout, but whatever it's doing is broken in IJulia.

AFAICT, this was fixed by #671 / not relevant on 1.0


OpenTelemetry-Swift -> v1.1.0

updated deprecated GRPC library secure api -> usingPlatformAppropriateTLS

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2021-11-08T22:20:52.615+0000

Duration: 13 min 16 sec

Commit: 003cf1c4038abc64435c528b6667cdf2ed8f575a

:robot: GitHub comments
To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.


Localadmin user name for Windows systems is not configurable

In the azure driver config, it seems to be mandatory to specify 'localadmin' as the user name and still have winrm bootstrap work. This username should be configurable, just as the password is. In the knife-azure gem, the username is configurable: https://github.com/chef/knife-azure/blob/1.4.0/lib/azure/role.rb#L278

@adamedx Looks like a couple of things need fixing here... we select the default_ssh_username if no :bootstrap_options[:vm_user] is set https://github.com/chef/chef-provisioning-azure/blob/master/lib/chef/provisioning/azure_driver/driver.rb#L106 - this would create a machine using the username 'ubuntu' and then attempt to winrm bootstrap it using 'localadmin'
So the workaround is to always set :bootstrap_options[:vm_user] for Windows VM roles until fixed.

I'll give that workaround a try @stuartpreston -- thank you!

Closing this as no further code changes are planned before the ARM driver.