fix(menuconfig): allow choice override menuconfig

Hi :wave:
I have encountered this issue when spending far too much time with kconfig and RIOT...
Handle special cases when having a choice override with a menuconfig.
This is a bit more complete solution to #94
The failure can be checked with:

config PRE
    bool "Pre prompt"

choice OVERRIDE_ME

menuconfig PRE_MENUCONFIG
    bool "PRE_MENUCONFIG prompt"

config PRE_CONFIG
    bool "PRE_CONFIG prompt"
    depends on PRE_MENUCONFIG

endchoice

menuconfig FOO
    bool "FOO prompt"

if FOO

config FOOPRE
    bool "FOOPRE prompt"

choice OVERRIDE_ME
    bool "OVERRIDE_ME prompt"

menuconfig DOES_NOT_CRASH_MENUCONFIG
    bool "DOES_NOT_CRASH_MENUCONFIG prompt"

config DOES_NOT_CRASH_CONFIG
    bool "DOES_NOT_CRASH_CONFIG prompt"
    depends on DOES_NOT_CRASH_MENUCONFIG

endchoice

config FOOPOST
    bool "FOOPOST prompt"

endif

choice OVERRIDE_ME

menuconfig POST_MENUCONFIG
    bool "POST_MENUCONFIG prompt"

config POST_CONFIG
    bool "POST_CONFIG prompt"
    depends on POST_MENUCONFIG

endchoice

config POST
    bool "POST prompt"

Here is the failure and solution in action...

ping @leandrolanzieri just to do a quick check...

Note that this case only trigger if there is a failure, if there is a missing corner-case then the selected node will just be messed up, maybe we could have set _sel_node_i to 0 instead of pass but we will see if anything pops up.

Thanks @leandrolanzieri and @kaspar030


Use patterns/signatures to grab data from memory

This PR adds pattern scanning functionality and removes hardcoded addresses/offsets which was limiting it to SA-MP 0.3.7 R1 only, now it should theoretically work on all of them, tested it with R4 and it was working
Both AddChatMessage call and SendMessage hook are now using pattern scanning to be found on memory.
NOTE: I haven't tested it in your plugin yet, but it's working in my sample, I can't do testing myself right now as I don't have access to my windows machine directly, I did this PR by connecting to my laptop at home using AnyDesk, so it's a pain to do it...

Looks good, works on R1, dunno for other versions. I am bad at using gits, still learning..
I will merge this and make some changes, good work


Getting 404 page not found.

https://github.com/openebs/openebs/tree/master/k8s/demo has a hyperlink of README file, for setting up openebs in hyperconverged mode, that is throwing this error.
URL:
https://github.com/openebs/openebs/blob/master/k8s/hyperconverged/README.md

Removed the broken link.


rust: Use f-string formatting syntax

Or "variable capture" as it's called around here. Since of Rust 1.67, our God-Emperor Clippy started demanding this form of formatting.
error: variables can be used directly in the `format!` string
    --> src/wrappers/themis/rust/libthemis-sys/build.rs:49:13
    |
49 |             eprintln!("{}", error);
    |             ^^^^^^^^^^^^^^^^^^^^^^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
    = note: `-D clippy::uninlined-format-args` implied by `-D warnings`
help: change this to
    |
49 -             eprintln!("{}", error);
49 +             eprintln!("{error}");
    |

Given that we're running WITH_FATAL_WARNINGS, we must either cave in, or suppress the lint. Suppressing it on Rust 1.56 requires suppressing another lint about unknown lint being suppressed.
So, given that we're unlikely to have users which require Rust 1.56 specifically, I think it's okay to cave in, update MSRV to 1.58, and update all affect format strings so that Clippy stays happy.

An alternative is to apply the changes demanded by Clippy and update the minimum required version from 1.56 to 1.58. Since a) the current stable is 1.67, b) RustThemis does not really have that many users that should not be intrusive in practice.

just to clarify, after that change all code written 1-2 years ago with rust <1.58 version wouldn't be compiled with the current version of themis?

just to clarify, after that change all code written 1-2 years ago with rust <1.58 version wouldn't be compiled with the current version of themis?

Correct. This will cause syntax errors with Rust 1.56 and earlier, breaking the builds.
I think this is acceptable as long as we do not backport this change to 0.14.x.
Applications that depend like this
[dependencies]
themis = "0.14"

will still work.
When they upgrade to "0.15" they will need a newer compiler (at least 1.58, the next stable version after 1.56).

Okay, understood. Is it a common approach to update rust compiler version on every release or how often usually it happens in rust projects? I looked on firefox and they upgrade compiler's version after 2+ releases (usually after 2 new versions) and currently uses 1.65 for stable builds. Now 1.67 is the latest version. But FF is a product, not library. So I looked at several popular rust libraries and what I found:

tokio has been supporting versions published at least 6 months ago. And now, it's supporting 1.49
serde doesn't have any policy of rust version support and declares 1.13 in cargo.toml
toml declares 1.60 in cargo.toml
slog supports at least 15 last versions of rust, and now it is 1.31.

In our docs we have been published 1.31 version or later. I think we should choose a policy for ourselves how old rust compiler we want to support...
@vixentael , what do you think?

This becomes less of a problem with Rust 1.67.1, but only temporarily.
In 1.67.1 they have downgraded this lint to "pendatic" (ignored by default, unless explicitly enabled). However, this is only a temporary measure, until support for auto-fixing code to avoid this lint is there. They are still going to have it warn-by-default in the future, so this PR is still valid.

A common support policy that I've seen in Rust libraries:

Hard commitment to support N latest stable versions, where N is often 3 or 6 (which is about 3–6 months).
That is, library must work with those versions without any breaking changes, period.
Say, we take N = 3, then we must support at least 1.65.0 released on 2022-11-03.
This is “hard” minimum version. We cannot use features added in later versions under any circumstances.

Minimum supported Rust version stays pinned at a particular version unless it is necessary to upgrade.
This is “soft” minimum version. Currently on master, this is 1.56.
We test against it, but it can be bumped if necessary. However, no frivolous bumps without a good reason (improved maintainability, security, whatever). It also cannot be bumped higher than the hard minimum, obviously.

No minimum supported version bumps in bugfix patch releases. Always release it with a minor version bump of the library, so that people can pin a minor version to stay on a particular old compiler until they are ready.

However, no frivolous bumps without a good reason (improved maintainability, security, whatever).

Do we really need to bump the version now? Can we temporarily suppress such warnings for some time? until we decide to bump the version for some serious reason instead of syntax sugar?

I see no reason why we shouldn't use 1.58 as minimum instead of 1.56. The Rust compiler is always updating, the more versions were behind, the more technical debt we have.
I'd bump to 1.58 min + update product docs to mention the min version.

@vixentael:

update product docs to mention the min version

https://github.com/cossacklabs/product-docs/pull/300

Unit tests (1.58)
  Downloaded csv v1.2.0
error: package `csv v1.2.0` cannot be built because it requires rustc 1.60 or newer, while the currently active rustc version is 1.58.1

🤦

Aight. So I've pinned that one dev-dependency that didn't work with 1.58, now it's all green again ☀


is it possible to have prefix for URL

For example I want nipap URL to be http://myserver/nipap
Thank you

I'm not sure if this is actually supported. You could try changing your Apache config to serve it off /nipap instead but I'm afraid that we might not be using relative URLs so it might break :/

I'll be closing this issue. I don't see that it's important to support custom paths as it's so easy to setup virtual hosts.


Set Default Layout mode

What's the Problem?
I would love to have the Column layout mode to be the default.
Solution/Idea
Add an option to set a layout mode to be the default for all folders you open.
Alternatives
The alternative would be for the app to not crash and return to whatever mode it feels like returning after the crash.
Priorities

Capability
Priority

This proposal will allow developers to accomplish W
Must

This proposal will allow end users to accomplish X
Should

This proposal will allow developers to accomplish Y
Could

This proposal will allow end users to accomplish Z
Won't

Files Version
No response
Windows Version
No response
Comments
No response

You can by going to Files settings > preferences > file and folder > turn off individual preferences. Then select collum layout in any folder.


Could not import PIL.Image. The use of array_to_img requires PIL

System information
Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No
OS Platform and Distribution : ubuntu 18.04 LTS
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: Jetson Nano
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): tensorflow==1.15.2+nv20.6
Python version: 3.6
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version: CUDA 10.2 / cuDNN 8.00
GPU model and memory: NVIDIA Tegra X1 (nvgpu)/integrated and 4GB memory
Keras: 2.0.5
Describe the problem
While executing an application, i got following exception

After that, I installed pillow-7.2.0. But i am getting same error message ImportError: Could not import PIL.Image. The use of array_to_img requires PIL.
Could you please help me to resolve the issue?

@suresh-s
Provide the exact sequence of commands / steps that you executed before running into the problem.Thanks!


DASH Stream (webm) not working

Description
Hello, for a help application that is based on the Chromium Embedded Framework (CEF) I want to use videojs. By default the CEF is not playing h.264 encoded video. So I want to use webm encoded videos. Playing single webm files with videojs is no problem but I can not play dashed webm streams.
Please help me to get this dash stream working with videojs. The dash files are created via ffmpeg.
Steps to reproduce
I have created a fiddle without any additional dash scripts: https://jsfiddle.net/agmorpheus/pdegsch3/3/
Videojs is displaying an error: Playback cannot continue. No available working or supported playlists. (Playing HLS streams works perfectly)
I also created a fiddle with dash.js and videojs-dash.js file: https://jsfiddle.net/agmorpheus/pdegsch3/4/
Video is playing for a second and then stops.
A lot of buffer errors are shown in the console
Results
Expected
I expect that videojs plays the dash stream.
When I use the dash.js script without videojs the dash stream works.
What I also expect, if the dash stream is not working or supported by videojs, then if should play a fallback source. The fiddle example has a fallback source and it is not played.
Actual
see Steps to reproduce
Error output
see fiddle examples
Additional Information
versions
videojs
latest
browsers
Chrome
OSes
Win10
plugins
no plugins

After some more testing I found out, that the webm Dash Stream starts to work, when the source is added via javascript and not the source tag.
Working Example: https://jsfiddle.net/agmorpheus/pdegsch3/6/
Why is this not working,
<video
          id="player_1"
          class="video-js vjs-fluid"
          controls
          preload
          playsinline
          poster="https://dd2p284ir9f52.cloudfront.net/textsample/thumb.jpg"
        >
            <source
            src="https://dd2p284ir9f52.cloudfront.net/textsample/dash/manifest.mpd"
            type="application/dash+xml"
          />
</video>

but this works?
player.src([
{
src: "https://dd2p284ir9f52.cloudfront.net/textsample/dash/manifest.mpd",
type: "application/dash+xml",
}
]);
Thanks


Pass autofillHints value to EditableText

Hi,
Currently, autofillHint property of TextFormField is not working because its value is not propagated down the widget tree.
This PR passes the autofillHints value to the EditableText within the RepaintBoundary widget.
This solves the #95522
Pre-launch Checklist

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[ ] I signed the [CLA].
[x] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is [test-exempt].
[ ] All existing and new tests are passing.

@Kurczaak  You don't need to pass autofillHints to EditableText. They was removed intentional. Instead you need to change EditableText.
From
bool get _needsAutofill => widget.autofillHints?.isNotEmpty ?? true;

To
bool get _needsAutofill =>
      widget.autofillClient != null ||
      (widget.autofillHints?.isNotEmpty ?? true);

@Kurczaak could you sign the CLA when you get a chance? Otherwise we won't be able to review the pull request.


Don't spend build time optimizing build-time-only crates

Crates used exclusively at build time, such as proc-macro crates and
their dependencies, don't benefit substantially from optimization; they
take far longer to optimize than time saved when running them during the
build. No machine code from these crates will appear in the final
compiled binary.
Use the new profile-overrides mechanism in Rust 1.41
(https://doc.rust-lang.org/cargo/reference/profiles.html#overrides) to
build such crates with opt-level 0.
Before:
Finished release [optimized] target(s) in 1m 22s
After:
Finished release [optimized] target(s) in 27.16s

The time savings will mainly be at clean builds? Is this commonly used among lib crates? Haven't seen it before.

Yes, this will speed up any build that needs to build syn, proc_macro2, and other proc macro crates.
A binary crate using nbd-async would need to do the same thing to get the build performance benefit. This change helps when rebuilding nbd-async itself, which I was doing repeatedly when testing other changes.


Lost transferred data

I swapped sd chips in my phone to another and It kept the app but my data was not transferred... It did not even transfer the introduction to writerplus doc. Anyway to retrieve it? I'm just glad I keep rough hand copies.

Same problem. Did you ever get an answer?


Get ItemBox selected item

Hello all,
I'm working on a toast notification where I have to get the item of a list box selected by the user.
Currently, my notification looks well but I don't know how to retrieve the user choice ...
Is it on the "Arguments" property of the button that I have to play?
My code:
`$Content = $null
$Select1 = New-BTSelectionBoxItem -Id 'Item1' -Content 'Choice 1'
$Select2 = New-BTSelectionBoxItem -Id 'Item2' -Content 'Choice 2'
$Select3 = New-BTSelectionBoxItem -Id 'Item3' -Content 'The good choice'
$list = New-BTInput -Id 'Selection001' -Items $Select1, $Select2, $Select3 -Title "Liste of choices" -DefaultSelectionBoxItemId "Item3"
$text = New-BTText -Text "hello"
$Binding = New-BTBinding -Children $text
$Visual = New-BTVisual -BindingGeneric $Binding
$Audio = New-BTAudio -Source ms-winsoundevent:Notification.Default
$button = New-BTButton -Content "Button" -Arguments "???"
$action = New-BTAction -Inputs $list -Buttons $button
$Content = New-BTContent -Visual $Visual -Actions $action
Submit-BTNotification -Content $Content
`
If someone knows how to make it work it will be very appreciated :)
Thank you

Unfortunately, I think this will be one of the "missing links" we run into because of not being able to subscribe to WinRT event in PowerShell. i.e. We can make these selection boxes... but can't really do anything with them.
I'll certainly look into though, it's possible something changed in the Fall Creators Update that will help bridge this gap.

As far as I understand, this and receiving text input will not be possible, as we would need to handle activation or background activation. Therefore in an native UWP app, we would need to create an override for OnActivated/OnBackgroundActivated and need to be able to configure the BackgroundExecutionManager. As the AppId of PowerShell is used, this cannot be done.
As an alternative for Win32 apps, it would be necessary to use the compat library from the Windows 10 SDK, Register a COM activator and to extend the NotificationActivator class with an override for the OnActivated method.
Additionally there is code that needs to be added to register AUMID and COM server.
If anyone has an idea, how to implement this in PowerShell, I would be happy to support.
Here is the docs link for the Win32 implementation: https://docs.microsoft.com/en-us/windows/uwp/design/shell/tiles-and-notifications/send-local-toast-desktop

This is now possible! See here for an example: https://toastit.dev/2020/08/08/actionable/
Better documentation is coming, but feel free to open a discussion if you're trying to crack a specific use case!


i3status-rust: 0.10.0 -> 0.11.0

Motivation for this change
Updated to 0.11.0. Changes: https://github.com/greshake/i3status-rust/releases/tag/v0.11.0
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[X] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

Notify maintainers
cc @

@GrahamcOfBorg build i3status-rust

@GrahamcOfBorg build i3status-rust

@GrahamcOfBorg eval

Thanks!


Make this package implementation-agnostic

Rather than always loading Node Sass, this now requires users to pass in
either Dart Sass or Node Sass as an option to the plugin.
Closes #26

Thanks @nex3, that all looks good to me 👍 However I think it would be prudent to add a CHANGELOG that makes it clear to anyone using this directly that the require usage has changed in v3.

Thanks @nex3. I'm guessing this should be released as a new major version.

Published as v4.0.0 👍 Thanks :) I'll try to be quicker with the subsequent PRs :)


Check if properties belong to the same entity but different relationships.

If a Flight entity is related to a departure and  arrival Airport, finding flights by both destination and arrival airports generates an incorrect Cypher query.
Here are the entities used:
@NodeEntity
public class Airport {
    String code;  // e.g. "LAX"
    String name;  // e.g. "Los Angeles"
}

@NodeEntity
public class Flight {
    String name;
    @Relationship(type = "DEPARTS")
    Airport departure;
    @Relationship(type = "ARRIVES")
    Airport arrival;
}

The query is genereated via a method on an SDN Repository:
public interface FlightRepository extends Neo4jRepository<Flight, Long> {
    public List<Flight> findAllByDepartureCodeAndArrivalCode(String departureCode, String arrivalCode);
}

Expected Behavior
Using the example scenario above, I would expect a list of Flights to be returned that exist between the two named Airports.
Current Behavior
The query fails to return expected instances. When logging is turned on, the following query is produced (lines split to aid readbility:
MATCH (n:`Flight`)
MATCH (m0:`Airport`) WHERE m0.`code` = $`code_0` AND m0.`code` = $`code_1`
MATCH (n)-[:`DEPARTS`]->(m0) MATCH (n)-[:`ARRIVES`]->(m1)
WITH DISTINCT n RETURN n,[ [ (n)-[r_a1:`ARRIVES`]->(a1:`Airport`) |
[ r_a1, a1 ] ], [ (n)-[r_d1:`DEPARTS`]->(a1:`Airport`) |
[ r_d1, a1 ] ] ], ID(n) with params {code_0=LHR, code_1=LAX}

The second MATCH clause doesn't differentiate between the two Airport instances, instead trying to match both arrival and departure names against the same Airport. As this is never going to work (apart from round-trip flights, I guess), no flights are returned.
Context

Your Environment

Java version: OpenJDK 11
Neo4j version (The version of the database): 4.1 (also tried 3.4.9)
OGM Version (The version of this software): 3.2.16 (included via Spring Boot 2.3.4.RELEASE)
OGM Transport used (One of Bolt, HTTP or embedded): Bolt
Neo4j Java Driver version (in case of Bolt transport): 4.0.2
Link to your project:

POM file:
<?xml version="1.0" encoding="UTF-8"?>
<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
	xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 https://maven.apache.org/xsd/maven-4.0.0.xsd">
	<modelVersion>4.0.0</modelVersion>
	<parent>
		<groupId>org.springframework.boot</groupId>
		<artifactId>spring-boot-starter-parent</artifactId>
		<version>2.3.4.RELEASE</version>
		<relativePath/> <!-- lookup parent from repository -->
	</parent>
	<groupId>com.example</groupId>
	<artifactId>flights</artifactId>
	<version>0.0.1-SNAPSHOT</version>
	<name>flights</name>
	<description>Neo4j Example</description>

	<properties>
		<java.version>11</java.version>
	</properties>

	<dependencies>
		<dependency>
			<groupId>org.springframework.boot</groupId>
			<artifactId>spring-boot-starter-data-neo4j</artifactId>
		</dependency>

		<dependency>
			<groupId>org.projectlombok</groupId>
			<artifactId>lombok</artifactId>
			<optional>true</optional>
		</dependency>
		<dependency>
			<groupId>org.springframework.boot</groupId>
			<artifactId>spring-boot-starter-test</artifactId>
			<scope>test</scope>
			<exclusions>
				<exclusion>
					<groupId>org.junit.vintage</groupId>
					<artifactId>junit-vintage-engine</artifactId>
				</exclusion>
			</exclusions>
		</dependency>
	</dependencies>

	<build>
		<plugins>
			<plugin>
				<groupId>org.springframework.boot</groupId>
				<artifactId>spring-boot-maven-plugin</artifactId>
			</plugin>
		</plugins>
	</build>

</project>

Steps to Reproduce

Using the above model and repository, the sample Spring Boot application looks as follows:
@SpringBootApplication
public class FlightsApplication {

	public static void main(String[] args) {
		SpringApplication.run(FlightsApplication.class, args);
	}

	@Bean
        CommandLineRunner demo(AirportRepository airportRepository, FlightRepository flightRepository) {
        return args -> {
	    // Clean up from previous runs.
	    flightRepository.deleteAll();
	    airportRepository.deleteAll();

	    // Create a few airports
	    Airport lhr = airportRepository.save(new Airport("LHR", "London Heathrow"));
	    Airport lax = airportRepository.save(new Airport("LAX", "Los Angeles"));
	    Airport cdg = airportRepository.save(new Airport("CDG", "Paris Charles de Gaulle"));

	    // Create some flights
	    Flight lhrToLax = new Flight("FL 001", lhr, lax);
	    Flight lhrToCdg = new Flight("FL 002", lhr, cdg);
	    Flight laxToLhr = new Flight("FL 003", lax, lhr);

	    // Save the flights.
	    flightRepository.saveAll(Arrays.asList(lhrToLax, lhrToCdg, laxToLhr));

	    // Find a flight from London to Los Angeles
	    List<Flight> results = flightRepository.findAllByDepartureCodeAndArrivalCode("LHR", "LAX");

            // Expect Flight "FL 001" to be returned, but result size is 0.
	    if (results.size() == 0) {
	        System.err.println("Sorry! No flights found!");
	    } else {
	        results.forEach(f -> System.out.println("Flight: " + f.toString()));
	    }
        };
    }
}

Could you use
@org.springframework.data.neo4j.annotation.Query(
			"MATCH (n:`Flight`)\n"
			+ "MATCH (m0:`Airport`) WHERE m0.`code` = $`code_0`\n"
			+ "MATCH (m1:`Airport`) WHERE m1.`code` = $`code_1`\n"
			+ "MATCH (n)-[:`DEPARTS`]->(m0) \n"
			+ "MATCH (n)-[:`ARRIVES`]->(m1) \n"
			+ "WITH DISTINCT n RETURN n,[ [ (n)-[r_a1:`ARRIVES`]->(a1:`Airport`) | \n"
			+ "[ r_a1, a1 ] ], [ (n)-[r_d1:`DEPARTS`]->(a1:`Airport`) | \n"
			+ "[ r_d1, a1 ] ] ], ID(n)"
	)

on the method as workaround until we find a way to fix this in OGM, please?

Related: https://jira.spring.io/browse/DATAGRAPH-1424

This is fixed and will be part of 3.2.22. Thanks for your report and your patience.


Create a new release

Hey @Favna, can you create a new release to include the latest merged fixes ?
Cheers!

Published v1.3.0 just now


KeyError(f'invalid depth {depth} for resnet') when import build_detector

Traceback (most recent call last):
File "demo_prune.py", line 15, in
from mmdet.models import build_detector
File "/home/yantianyi/mmdet-3.5/mmdet/models/init.py", line 2, in
from .backbones import *  # noqa: F401,F403
File "/home/yantianyi/mmdet-3.5/mmdet/models/backbones/init.py", line 3, in
from .resnext import ResNeXt
File "/home/yantianyi/mmdet-3.5/mmdet/models/backbones/resnext.py", line 7, in
from .resnet_2 import Bottleneck as _Bottleneck
File "/home/yantianyi/mmdet-3.5/mmdet/models/backbones/resnet_2.py", line 386
raise KeyError(f'invalid depth {depth} for resnet')
^
SyntaxError: invalid syntax
I have installed the mmnet, but cant import the modules.

Here is my packges:
python=3.5.6
pytorch=1.1.0 cuda9.0
mmdet=1.1.0
mmcv=0.3.2
so how can i fix it?
thank u very much!

Hi @yanty123 , a invalid depth for resnet backbone will raise the above KeyError. The valid depth value for resnet is only in [18,34,50,101,152]

Hi @yanty123 , a invalid depth for resnet backbone will raise the above KeyError. The valid depth value for resnet is only in [18,34,50,101,152]

Thank you sir, I know what you mean. However I haven't define the model yet, it's just the import line causes the error.
Just  'from mmdet.models import build_detector' this import line trigger the error.
So where is the problem?
Thank you!

Hi, I guess the import error might be caused by your incorrectly installation? Did you install mmdetection following the official guideline?

Hi, I guess the import error might be caused by your incorrectly installation? Did you install mmdetection following the official guideline?

I guess so.I did follow the ReadMe and there isn't any problem, so i dont know where is the problem.
I run it well in the py3.6 env, so it's quiet strange. .

Hi, does manually adding your mmdetection absolute path to PYTHONPATH help?

Hi, does manually adding your mmdetection absolute path to PYTHONPATH help?

oh it really helps! Thank you!
However, there is a new similiar problem when using the code
' from mmdet.apis import inference_detector, init_detector_pruned, init_detector, show_result_pyplot '
File "/home/yantianyi/video/tools/mmdetection/demo_prune.py", line 20, in
from mmdet.apis import inference_detector, init_detector_pruned, init_detector, show_result_pyplot
File "/home/yantianyi/mmdet-3.5/mmdet/apis/init.py", line 1, in
from .inference import (async_inference_detector, inference_detector,
File "/home/yantianyi/mmdet-3.5/mmdet/apis/inference.py", line 21
f'but got {type(config)}')
^
SyntaxError: invalid syntax
So how can I fix this one?

It seems like the imported init-module runs, but i just import it.


Prepared repository for transition to Hiero

I prepared the README.md file for transition to Hiero by renaming all necessary instances of Hedera to Hiero.

cc.
@Ndacyayisenga-droid

Thanks for working on this @Lemeri123!
I guess there a couple of things that need though that need to first get changed for this to get merged. I think we can keep this work in progress

Alright then. Sure thing


How to create a relationship between a new post type like products and categories table as you did with posts ?

Laravel Version: 5.3
Voyager Version: 0.9.34
PHP Version: 5.6
Database Driver & Version: MySQL

Description:
How to create a relationship between a new post type like products and categories table as you did with posts ?
I'm trying use your GREAT package to create a simple e-commerce application, so I'm wondering how to link the categories with the "products" table&BREAD I created  as you did with the posts ?
THANKS.

Just released v0.10 with this feature.

GREAT!!
I was waiting for the v0.10 release!
THANKS


Implement new NEAR RPC errors decoding

This PR still needs to be tested with various error scenarios from the RPC, e.g transaction timeouts etc.

This PR still needs to be tested with various error scenarios from the RPC, e.g transaction timeouts etc.

When can the test be completed

I am unable to get RPC nodes to return errors I want to (tricky on testnet, practically impossible on localnet imo). If anyone manages to give me JSON responses of errors from RPC then that would help a lot already.

I am unable to get RPC nodes to return errors I want to (tricky on testnet, practically impossible on localnet imo). If anyone manages to give me JSON responses of errors from RPC then that would help a lot already.

I don't think timeout needs special handling since it's unpredictable and only needs to handle explicit error types

I mean rather about testing overall, not handling specially errors (yet). I wouldn't merge this PR in blindly.


The localization dropdown incorrectly refers to "Taiwanese"

"Taiwanese" is a different language, the term used in the dropdown is more commonly and accurately referred to as "Traditional Chinese".

Seems an easy enough change. Based on Django's master setup it seems they use "Traditional Chinese" and "Simplified Chinese".


[Cookbook] Simple Function calling example with Mixtral-8x7B

[Cookbook] Simple Function calling example with Mixtral-8x7B
Along with some fixes to function calling using OpenAI and Anyscale modelparsers

Stack created with Sapling. Best reviewed with ReviewStack.

-> #748
#746
#745
#743
#730

Test Plan:
New function calling notebook:
https://github.com/lastmile-ai/aiconfig/assets/25641935/959f3e24-abb6-44b9-94ee-5cee2eb07ef8


Release Google.Cloud.Logging.NLog version 3.2.0

Changes in this release:

Commit a0f7730: docs: Move NLog configuration into docs directory and expand it
Commit 932a8d1: feat: Adds EnableJsonLayout option to use custom jsonLayout

Fixes #5634.

Created release for Google.Cloud.Logging.NLog-3.2.0


PROBLEM IN INSTALLING PYCAIRO

Ref Youtube video: Manim Tutorial Installation on Windows by Theorem of Beethoven
https://www.youtube.com/watch?v=ZltiKHFWmv8
Check 8:46 of the video
I was going through this video about installing manim and was trying to do the same on my PC. I was stuck at 8:46 where pycairo is being installed.

PLEASE HELP

Try with version of 32 bits:

Or the another:

I did. Same issue

Send the ScreenShots of every one.

I just restarted my PC and again tried to install the 32 bit version. Voila!

Perfect


Generalize ItemLevelDownloader for download as well as upload. Adding azure blob storage uploader.

@omeshp , please review this


selectTranslate not available for lazy loaded translations if transloco pipe is not used

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[X] Bug report
[ ] Performance issue
[ ] Feature request
[ ] Documentation issue or request
[ ] Support request
[ ] Other... Please describe:

Current behavior
It is not possible to use selectTranslate in conjunction with lazy loaded translations without using the transloco pipe. The scope was specified in the providers, but for some reason it says that the translation key is missing. Also, if you do use the transloco pipe in the HTML, selectTranslate will still not (always) be available when you use it in ngOnInit hook. In the stackblitz below you can see the behavior if you go to /lazy-typescript-only page.
https://stackblitz.com/edit/ngneat-transloco-issue
Expected behavior
selectTranslate should be available even if you don't use the transloco pipe. Sometimes you need to translate strings within typescript without using the transloco pipe. Also selectTranslate returns an observable which should emit the translation once it's available (perhaps wait for the file to be resolved before selectTranslate observable emits the translation). In the current situation the observables emits only one attempt which means that if the translation is not available at the time of subscribing, it will show a missing key warning.
Minimal reproduction of the problem with instructions
If you open the stackblitz:
https://stackblitz.com/edit/ngneat-transloco-issue
You can go to /lazy-typescript-only and if you look at the console you see the missing key warning while the file was specified in the module. If you uncomment the html snippet in the lazy-typescript-only component, you will see that the translation is available, but ONLY when you press the button.
What is the motivation / use case for changing the behavior?
To make this library better and bug-free
Environment

Angular version: 7

Browser:
- [X] Chrome (desktop) version XX
- [ ] Chrome (Android) version XX
- [ ] Chrome (iOS) version XX
- [ ] Firefox version XX
- [ ] Safari (desktop) version XX
- [ ] Safari (iOS) version XX
- [ ] IE version XX
- [ ] Edge version XX

For Tooling issues:
- Node version: XX
- Platform:

Others:

You are using the wrong syntax. In typescript, it should be selectTranslate('title', {}, 'lazy-page')

@ArielGueta Ahh I see, my bad then. The third param name was called lang. I thought you were only able to pass a language code as parameter then such as 'en' or 'es'. Thanks, you saved me!

@ArielGueta I just tried it in my own projects, but that doesn't seem to work like you say. I can only get the translations after a while (after DOM render). I tested this on a function that is attached to a click event:
    this._translocoService.selectTranslateObject('PhoneNumberInput.Countries').subscribe(x => console.log(x));
    this._translocoService.selectTranslateObject('Countries', {}, 'features/phone-number-input').subscribe(x => console.log(x));
  }```

The first selectTranslateObject works in a click event, but not in ngOnInit or the constructor.
The second selectTranslateObject never works, not even after the file was loaded in.

It seems like it doesn't re-emit when the translation is available. It just spits out undefined once. Could it be that the file size (about 225 lines) of the json file is the cause of the network lag?

can you update the stackblitz?

@ArielGueta I just updated the stackblitz. You can find it in lazy-typescript-only. I can tell it has something to do with network latency, because you rarely get a value back on ngOnInit if you refresh a couple of times.

Why do you keep using     this._translocoService.selectTranslateObject('PhoneNumberInput.Countries').subscribe(x => console.log(x));?
It's not the correct syntax.

@ArielGueta I used the syntax you gave me and that didn't work at all for me. The one you just mentioned only works in a click event (meaning that the json file was ready at the time of clicking)

You guys are both right 🙂
@mohamedaarab1994 The syntax that @ArielGueta said is the correct one, but there was a bug related to loading scopes using the selectTranslateObject (which is now fixed in v2.12.2).
Thanks for the report and the help, Merry Christmas 🎄 and Happy Hanukkah! 🕎

@ArielGueta @shaharkazaz It seems this is not fixed, I am experiencing this in version 2.14.
selectTranslate() does not emit during component initialisation

Ok, my apologies, we are using scoped files (not one huge file) for translations, so if I provide the scope in the call
this.translocoService
  .selectTranslate(key, {}, 'testimonials')
  .subscribe(result => {
    console.log(result);
	})

then it works fine.

@rmcsharry I am really grateful for your previous answer, since it saved us a lot of work hours. The documentation regarding this function is...well...it should be revised and completed :(

@kmihaly you are always welcome to open a pr and change the things you find missing :)

Hi all,
I keep having the same problem, the scoped translations are not loaded when calling the selectTranslate.
this.transloco
.selectTranslate('COMMON.LANGUAGE', {}, { scope: 'app' })
.subscribe((res) => {
  console.log(res);
});

I'm receiving  Missing translation for 'app.COMMON.LANGUAGE'
If I use the transloco pipe everything works.
Angular 16.2.0 and transloco 6.0.4
Thank you

@bitforcesrl First, I encourage you to update to the latest version, ' 7.4.1under the@jsverse` scope.
I've tested the selectTranslate scope functionality in the playgrounds and didn't encounter what you are experiencing.
Please open a new issue with a reproduction link so I can address the issue.


recipes: Switch to using UNPACKDIR instead of WORKDIR

UNPACKDIR is new contruct for do_unpack things in latest master we should be using that instead of WORKDIR for referencing those files

Please note that this change would not work on scarthgap so we need to branch scarthgap before applying this patch

You might have missed one: https://github.com/agherzan/meta-raspberrypi/pull/1329

Note that I've also created "scarthgap".


Docs skip the fact that kubeconfig is required

http://docs.projectcalico.org/v1.5/getting-started/kubernetes/installation/

mkdir -p /etc/cni/net.d
cat >/etc/cni/net.d/10-calico.conf <<EOF
{
    "name": "calico-k8s-network",
    "type": "calico",
    "etcd_endpoints": "http://<ETCD_IP>:<ETCD_PORT>",
    "log_level": "info",
    "ipam": {
        "type": "calico-ipam"
    },
    "policy": {
        "type": "k8s"
    }
}
EOF

This fails because there is no kubernetes key here, error coming out of calico cni:
{
    "code": 100,
    "msg": "invalid configuration: no configuration has been provided"
}

calico/cni repo mentions the key:

https://github.com/projectcalico/calico-cni/blob/master/configuration.md#kubernetes-specific

It's also not very clear from the error what kind of config is required.

I think this changed in v1.4, as there are no longer default values for k8s_api_root, etc.

@caseydavenport Thanks, if the CNI plugin v1.4.0+, CNI Configuration must add k8s_api_root option, otherwise kubelet error:
E1108 15:46:09.966758    8232 cni.go:255] Error adding network: invalid configuration: no configuration has been provided
E1108 15:46:09.966796    8232 cni.go:209] Error while adding to cni network: invalid configuration: no configuration has been provided

cni configuration like:
{
    "name": "calico-k8s-network",
    "type": "calico",
    "etcd_endpoints": "http://<IP_ADDRESS>:4001,http://<IP_ADDRESS>:4001,http://<IP_ADDRESS>:4001",
    "log_level": "info",
    "kubernetes": {
        "k8s_api_root": "http://<IP_ADDRESS>:8080"
    },
    "ipam": {
        "type": "calico-ipam"
    }
}

This doc indicates the need to specify k8s API access details: http://docs.projectcalico.org/v2.0/reference/cni-plugin/configuration#kubernetes-specific


Add log-collector to e2e tests

Log-collector creates

fluentd-master deployment on one of the master nodes

fluentd-worker daemonset to pushes all the logs to fluentd-master

fluentd-master service for workers to talk to the master

fluentd-master writes all container logs and docker, kubelet service logs to disk at /var/log/log-collector

At the end of the tests log-collector uses sftp to download logs from remote:/var/log/log-collector based on filter.
--log-ouput-dir flag in main_test.go can be used to specify destination of all the logs
Note:
I think Jenkins job can archive the --log-ouput-dir when tests have completed?
/cc: @yifan-gu @aaronlevy @pbx0

rktbot run e2e

Also cc @xiang90

coreosbot run e2e

coreosbot run e2e

coreosbot run e2e

@abhinavdahiya LGTM except one comment.

coreosbot run e2e

coreosbot run e2e

coreosbot run e2e

coreosbot run e2e-selfetcd


Python 3.10 Match case indentation not working

Tree sitter indent works with yati for if else expression, however for Python3.10 match case indentations don't work yet.

Fixed in 870ac3e7290a85169515bd13609d0c0be5bc8c93


Question: [No EventManager Class]

I want to use event in my custom control, but there is no EventManager Class, I'm certain I have already using System.Windows.

EventManager is a WPF concept that does not exist in UWP.  @MikeHillberg fyi


Unable to install on Python 3.3

Downloading/unpacking oauth2 (from -r /home/christian/documents/projects/rh/requirements.txt (line 49))
  Downloading oauth2-1.5.211.tar.gz
  Running setup.py egg_info for package oauth2
    Traceback (most recent call last):
      File "<string>", line 16, in <module>
      File "/tmp/pip_build_christian/oauth2/setup.py", line 18
        print "unable to find version in %s" % (VERSIONFILE,)
                                           ^
    SyntaxError: invalid syntax
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):

  File "<string>", line 16, in <module>

  File "/tmp/pip_build_christian/oauth2/setup.py", line 18

    print "unable to find version in %s" % (VERSIONFILE,)

                                       ^

SyntaxError: invalid syntax

----------------------------------------
Cleaning up...
Command python setup.py egg_info failed with error code 1 in /tmp/pip_build_christian/oauth2
Storing complete log in /home/christian/.pip/pip.log

@jensenbox do you mind closing this issue? This repo is dead.

See #158. @jaitaiwan is working on PT3K support. Also, praise be to Gaia for this repo has risen! :innocent:


Changes Gauge for Counter

From Prometheus documentation: A gauge is a metric that represents a single numerical value that can arbitrarily go up and down
In these cases the correct choice should be Counter instead.

copy-pasting, same as ever 😅


Question/Request Login through SSO

We are looking into using these hooks, but we have our own login system.
How will this work in combination with the SSO?

I'm in the same boat. These hooks appear to assume you are BC for login. Hopefully someone from the team responds to this one soon!

I'm in the same boat. These hooks appear to assume you are BC for login. Hopefully someone from the team responds to this one soon!

Hi all, @rodeyseijkens & @avisra can you give more detail please around what the problem is that you are running into / what the specific need is that you have around the SSO? It sounds like you are looking to abstract the SSO from data-hooks so that you can include your own login, but I don't want to make assumptions. Thanks!

Hi all, @rodeyseijkens & @avisra can you give more detail please around what the problem is that you are running into / what the specific need is that you have around the SSO? It sounds like you are looking to abstract the SSO from data-hooks so that you can include your own login, but I don't want to make assumptions. Thanks!

@amckemie Sure thing. My assumption is that these hooks reference the current customer via the useCustomer hook - which I believe is set via the login process (which logs in a user to a BC customer account). I have my own identity provider and auth server that I'd like to use for logging into the app (planning on using NextAuth) - but I still want to set the customer behind the scenes so the cart/checkout/etc components can reference the customer data. My identity provider already has the BC customer ID stored for each of my users.
In my current setup, I am using the BC SSO api to forcefully login users using their customer ID: https://developer.bigcommerce.com/api-reference/storefront/customer-login-sso
So I think that I'm looking for... is a way to forcefully set the BC customer that useCustomer relies on - from my own custom login process.

@amckemie Sure thing. My assumption is that these hooks reference the current customer via the useCustomer hook - which I believe is set via the login process (which logs in a user to a BC customer account). I have my own identity provider and auth server that I'd like to use for logging into the app (planning on using NextAuth) - but I still want to set the customer behind the scenes so the cart/checkout/etc components can reference the customer data. My identity provider already has the BC customer ID stored for each of my users.
In my current setup, I am using the BC SSO api to forcefully login users using their customer ID: https://developer.bigcommerce.com/api-reference/storefront/customer-login-sso
So I think that I'm looking for... is a way to forcefully set the BC customer that useCustomer relies on - from my own custom login process.

Any progress on this?
I am hitting the SSO endpoint to create a sign the users into the bigcommerce system, so that when they click on the checkout link, they are still logged in... however, if they then click edit cart, the login session is lost, forcing the user to log back in again

Any progress on this?
I am hitting the SSO endpoint to create a sign the users into the bigcommerce system, so that when they click on the checkout link, they are still logged in... however, if they then click edit cart, the login session is lost, forcing the user to log back in again

Does anyone have a good workaround for this in the meantime?

I am working on it, probably today I will be able to create a PR!
We expect to have it published next week.

This is definitely a step in the right direction. Is it possible though to retrieve this shop-token completely server-side? My understanding is their Customer Login SSO actually requires a redirection to their token endpoint b(from the customer's browser).
This is an example flow I would expect:

Customer comes to my NextJS site and uses NextAuth.JS to login via our auth providers. Our database already has the BC customer ID on file for all of our users. And if there is not one on file, we use the BC apis to automatically generate one and store the customer ID.
On authentication callback from NextAuth.JS we would initiate a server-side call to retrieve (from BC using the customer ID) and set the SHOP-TOKEN cookie.
This would automatically be available to the storefront data hooks and all the APIs would function as expected.

Does this solve the issue where a user logins in (with sso), navigates to the checkout page, then hits edit cart and is sent back to the site in a logged out state?

This is definitely a step in the right direction. Is it possible though to retrieve this shop-token completely server-side? My understanding is their Customer Login SSO actually requires a redirection to their token endpoint b(from the customer's browser).
This is an example flow I would expect:

Customer comes to my NextJS site and uses NextAuth.JS to login via our auth providers. Our database already has the BC customer ID on file for all of our users. And if there is not one on file, we use the BC apis to automatically generate one and store the customer ID.
On authentication callback from NextAuth.JS we would initiate a server-side call to retrieve (from BC using the customer ID) and set the SHOP-TOKEN cookie.
This would automatically be available to the storefront data hooks and all the APIs would function as expected.

Reviewing your example, I am having a hard time understanding the use case of the useLogin hook in the event we already have our own login/authentication mechanism (e.g. we are using NextAuth.JS and IdentityServer4).
I will tinker with this some more.

Yes, the Custom Login SSO requires a redirect but you can do it in the server as well:

Generate the Login SSO URL
Fetch that URL (with `redirect: "manual") in the server to obtain the tokens
Find and return the SHOP_TOKEN

Example
The benefit of using the useLogin is that it forces to revalidate the useCustomer.
In your case, I think you have these options:

As you said, set the SHOP-TOKEN in the authentication callback from NextAuth (is this possible?) and manually revalidate the useCustomer hook. In this case is not necessary the useLogin hook.
Don't use the signIn method from NextAuth Create and create a new API route to handle all the authentication. In that new route you can POST /api/auth/signin/:provider  and if everything is OK get and return the SHOP_TOKEN. Use this new API route as the url option of the useLogin.

Makes sense? @avisra

Does this solve the issue where a user logins in (with sso), navigates to the checkout page, then hits edit cart and is sent back to the site in a logged out state?

Yes, it should. In the end, login (with or without sso) means that we store the SHOP_TOKEN cookie. Do you have any examples you can show of the problem you have?

example here:
https://oomphmade.com/pricing/cards
user adds item to cart
logins in
goes to checkout
presses edit cart (on bc)
goes back to cart, the user is now logged out of the next frontend

@avisra

Our database already has the BC customer ID on file for all of our users.

Can I ask how you've achieved this?

example here:
https://oomphmade.com/pricing/cards
user adds item to cart
logins in
goes to checkout
presses edit cart (on bc)
goes back to cart, the user is now logged out of the next frontend

This is probably because the checkout page is not part of your Next App, you are redirected to your "main" store. From https://oomphmade.com/cart to https://oomph-made.mybigcommerce.com/checkout.

This is true... However, the login session is usually saved. if i login, then close the browser, then reopen, I am still logged in.
has there been any progress on supporting the embedded checkout?
I had a play with the method:
https://github.com/bigcommerce/storefront-data-hooks/issues/6
But was always getting an error saying the cart is empty, even though i had stuff in my cart.

* As you said, set the SHOP-TOKEN in the authentication callback from NextAuth (is this possible?) and manually revalidate the `useCustomer` hook. In this case is not necessary the `useLogin` hook.

Yes I believe this makes sense. I have the code setup on my side - and have been able to set the SHOP_TOKEN cookie on my NextAuth callback. But how can I revalidate the useCustomer hook like you mention in the first option?

* As you said, set the SHOP-TOKEN in the authentication callback from NextAuth (is this possible?) and manually revalidate the `useCustomer` hook. In this case is not necessary the `useLogin` hook.

Yes I believe this makes sense. I have the code setup on my side - and have been able to set the SHOP_TOKEN cookie on my NextAuth callback. But how can I revalidate the useCustomer hook like you mention in the first option?

Cool, how did you manage to set the SHOP_TOKEN cookie? Is similar to this gist?
To revalidate the useCustomer you have to call the revalidate method. E.g:
https://github.com/bigcommerce/storefront-data-hooks/blob/034be1e4c7fa363f4b1362fd1a6d50e25b2632ea/src/use-login.tsx#L36-L42
This will force the customer data to be updated.

Awesome. Thanks a bunch for this. I have the SHOP_TOKEN being retrieved and added on login via NextAuth. And revalidating the customer using revalidate() is showing the customer information coming back properly, which is awesome. However, I noticed two issues that I may need assistance with:

After logging in (and revalidating the customer) - going to checkout (redirects to bigcommerce checkout page - e.g. same as what Vercel commerce is doing) the user is not logged in - it is displaying guest checkout and has the option for user to login there. I would expect that the BC checkout page would know who the user is.

I noticed that SHOP_TOKEN does not always come back from the fetch call. If I do it too quickly in succession, I will get SHOP_SESSION_TOKEN and a few others - but not SHOP_TOKEN.

I am seeing the same as @avisra

What are you seeing @magicspon ? 1 or 2?

I have created a new ticket for 1 (#36).
About 2, can you provide more info in a new ticket? @avisra
I close this ticket in favor of the new ones as a solution for the Login SSO will be included in the next release.

@jorgemasta 1 and 2.
A logged in user goes to the checkout and is logged out.
Also, seeing SHOP_SESSION_TOKEN rather than SHOP_TOKEN

@magicspon, @jorgemasta, did anyone got it working. As of now when I am clicking at checkout from nextjs commerce store, it goes to BC store checkout as a guest user. I want that my customer don't have to login again there.

I did, but it was a doozie.
I have two apps:
shop.mysite.com (BigCommerce)
www.mysite.com (my headless shop)
First, it is important that your headless shop create the shop_token/shop_session_token with the TLD mysite.com instead of www.mysite.com (otherwise the shop will not be able to use it).
Second - and this one is unfortunate - but BigCommerce has a weird quirk where it will attempt to create a new shop_session_token even though one exists already at the TLD. The first time it does this, it will be the correct value - but on the shop.mysite.com domain. The second time it does this, it will be a different value and it won't work (use will be guest).
To solve this problem.. I used a Cloudflare worker. Both subdomains are hosted through my CloudFlare account. So, I added a worker that replaces the "SHOP_SESSION_TOKEN" domain in the Set-Cookie header. This worked flawlessly - after much tinkering.

I'm having some issues using the updated useLogin hook:
const login = useLogin({
	url: "/api/auth/sso",
})

const onSubmit = async (formData) => {
	login({
		email: formData.email,
		password: formData.password,
	})
}

if the wrong email/password is provided, an error is Thrown.

And the api/auth/sso endpoint is never called, even with the correct email/password
Any ideas

@avisra
are you using the new useLogin hook?

@avisra,
It looks like
1- You are using your own auth while I am using build-in (That comes with nextJs boilerplate).
2- Then with the help of domain/subdomain settings, you are setting cookies on the parent domain & utilizing them on the child domain.
Did I get both points right? If yes, Then will it be possible to use your flow with bigcommerce built-in auth?
@magicspon which scenario you are using? build-in or your own auth provider?

I'm using my own auth provider (via craftcms).
I successfully managed to login the user with SSO, using my own hook. However, i'd rather use the hook provided with my own auth.

I believe there is an error in the docs:
this works:
	const login = useLogin({
		options: {
			url: "/api/auth/sso",
		},
	})

I'm getting a 200 response status from calling my sso endpoint.
Then i'm immediately getting a 500 internal server error from the /api/bigcommerce/customers endpoint

@jorgemasta I'm using your SSO example (modified slightly)
https://gist.github.com/magicspon/f3617e8d12db7cb46199283d2b043272

Sorted...
Created a new API Account.  now it's working as expected
@jorgemasta thanks for your work on this, greatly appreciated


Adds proxy support for API calls to BrowserStack

Bumps browserstack module to latest version
Passes proxy settings to the browserstack module for making API requests using the configured proxy

@dignifiedquire, have you found a chance to review this? : )

Thanks, sorry for the delay :octocat:

any chance of a release soon? :)


Multiple sounds playing on one press

So, say I have a sound bound to F5 and another one bound to Ctrl+F5. If I press Ctrl+F5, then the intended sound will play, but if F5 is held too long (which can be a split second), the sound underneath (just F5) will play as well. I've made sure and this isn't just a result of me pressing F5 before Ctrl, nor is it a result of me releasing Ctrl before F5. This is also a little difficult since if I press and release a key too quickly, it won't register at all and nothing will play.

Registering the hotkeys would solve that problem.


Excuse , I have a question,  what does the para --cls_list means ?

Excuse , I have a question,  what does the para --cls_list means ?

and in solver_makeup.py. in func “de_norm”， what is the means of out = (x + 1) / 2, thanks


Storage containers losing all items when ship is mounted

Using movingworld 6.350 and davincisvessles 6.353 with forge <IP_ADDRESS>55 (Issue persists with forge 2682)
When a ship is mounted, all storage containers spill out their containing items.

Confirming, it's happening on my machine, too. Same versions.

FYI - this bug not present in minecraft 1.12.2, forge <IP_ADDRESS>39, Davinici-1.12-6.340

Here too. :/
Minecraft 1.12.2
Forge <IP_ADDRESS>57
davincisvessels-1.12-6.353-full.jar
movingworld-1.12-6.350-full.jar

any workaround or progress yet? My ship is still in the hangar and started vibrating, because it want to go on a trip ;)

I use twitch, and it only offers davinci 340 and moving world 342, even when I choose exactly 2757.
Also GitHub only offers 340... And I just tested that with the built in inventory, an engine, a chest, a trapped chest, can and an ender chest and lost nothing.
Where are you getting these 350 versions from, and why aren't you using the 340 release that does not have this issue?

This newer version was given on a seperate issue, as it has a fix for crashing the game when attempting to use blocks from Chisels & Bits.


Copy mp4 files from source to destination bucket

To save money, we want to copy ONLY the mp4 files from the source bucket to the destination bucket, how can it be done?

Hi there,
Trying to get a little clarification on what you're trying to do.
Are you saying you don't want to encode your source MP4 file but merely copied to the destination bucket where the encoded video files are being written to?
If so, you would have to modify the MediaConvert template you are using for encoding and remove the MP4 File output group so that your source is not being re-encoded.
To copy the source file to the destination bucket, you could modify the Lambda function that does the encoding to execute an S3 copy of your input file to the destination bucket perhaps right after you run the encode function.


Remove perturbation seeds [HPPC-103]

(Update). I've been thinking for more hours... the heuristic below is broken and the whole idea of linear slot lookup is prone to problems with hash-conflict avalanching... keep thinking what to do about it.
I've been thinking about it for about 4 hours. If we want to remove local perturbation we absolutely have to deal with conflict avalanches when merging two containers that use the same hash key order.
Our hash tables are essentially bucket-sorting keys by their hash value (masked). So when we do a putAll we're essentially adding keys that have nearly-sorted (except for the wrap-up case) hashes. For example (this is from real life).
                                        cont.mask:    11111111 11111111 11111111
 slt | key value                           | hash
   0 | 11100010 00110010 10011111 11011111 | 01110011 11111111 11111111 11111101  <- wrapup from the end, doesn't matter
   1 | 00000000 00000000 00000000 00000000 | 00000000 00000000 00000000 00000000
   2 | 01000010 11011010 11111100 11100111 | 00111101 00000000 00000000 00000010
   4 | 11000010 10001110 01100111 01000000 | 10100001 00000000 00000000 00000100
   8 | 10110101 01000001 00101100 01000100 | 00110010 00000000 00000000 00001000
  12 | 01110000 11001100 00110000 00111001 | 00110111 00000000 00000000 00001100
  13 | 10110111 01011111 11110001 00001011 | 00010101 00000000 00000000 00001101
  14 | 11011100 00001100 00100000 00111001 | 10001111 00000000 00000000 00001110
  15 | 10000111 01010000 10011010 11111001 | 10010110 00000000 00000000 00001110
  16 | 00111101 00101110 11011011 01011000 | 10010100 00000000 00000000 00001101
  18 | 10100100 10001111 01010100 01000000 | 01100111 00000000 00000000 00010010
  19 | 11001100 11111101 10010100 10100011 | 00110010 00000000 00000000 00010010
  20 | 11100010 11000100 11000000 10010001 | 10101101 00000000 00000000 00010100
  21 | 11011000 00010001 11001010 10111110 | 01001011 00000000 00000000 00010010

The exponential time growth results because we're essentially filling up the front of the hash table with lots of colliding keys before we can expand, causing the cost of slot lookup to increase from O(1) to essentially a linear scan through, on average, half the number of elements stored in the table... Too bad.
Koloboke and fastutil workaround the problem by pre-sizing the target hash container to the estimated "worst" capacity. Essentially:
ensureCapacity(target.size() + source.size());

It is pretty darn good as it dodges the problem. But looking at fastutil's code made me think if it's the best possible estimate. Assuming the problematic case only happens with two containers that have the same underlying implementation (hash-ordered keys); if we know this for sure then the heuristic to determine min. required capacity to prevent the problem of avalanching conflicts can be actually more conservative. Here it is:
//
// GIVEN: target.putAll(source)
//
if (target.slots >= source.slots) {
  // no need to expand; even if source key hashes are sorted they will either
  // already exist in target's table slots or they will cause it to expand
  // even further (but this will happen gracefully).
} else {
  // target has fewer slots than we do. This may result in collisions. But we only need to expand
  // up to source.slots capacity because then the condition above holds.
  target.ensureSlots(source.slots);
}

Issue: HPPC-103 (migrated from JIRA), created by Dawid Weiss (@dweiss), resolved Mar 09 2015
Linked issues:

HPPC-112
HPPC-104
HPPC-186
HPPC-115

Comment by Dawid Weiss (@dweiss) (migrated from JIRA)
This can be done in a number of ways – what Vincent already explored https://github.com/vsonnier/hppc/issues/18 or what Sebastiano came up with (proper sizing of arrays).

Comment by Dawid Weiss (@dweiss) (migrated from JIRA)
I've decided to leave perturbation seed in. While there is an inherent performance penalty associated with an additional memory read and xor, this penalty is a small sacrifice for a potentially exponential time of merging (putAll, custom code) of hash containers. I can think of many real-life scenarios in which this may be actually a common case and I don't think the risk should be present in the out-of-the-box library.
If needed, one can override computePerturbationValue or invoke the newInstanceWithoutPerturbations static instantiation method.
In fact, I think we will strengten the guard against hash clustering conflicts by making each hash container compute a perturbation value based on the current content on resize. This would result in different key distributions even for containers storing disjoint set of keys at the same capacity (and thus would make putAll and all associated operations safe without the need for any trickery).


coreutils:  Missing dependency "crypto"

random.ts has a "require" for "crypto", but package.json has no reference to that package:
https://github.com/phosphorjs/phosphor/blob/51a653011dae5533dcc3097e193b70b32ff9fe24/packages/coreutils/src/random.ts#L42
https://github.com/phosphorjs/phosphor/blob/39d54e8a5863f4e96b80dc05404e07367af5e638/packages/coreutils/package.json#L14

This causes issues with rollup.js
This is apparently causing issues downstream in Angular 6 (newer WebPack?).

Edit:  After a bit of digging I see that "crypto" is now a core Node module and that while you do have runtime checking for the current environment (Node or Browser), it hasn't been isolated for downstream bundlers in general.
FWIW in scenarios like this I typically create two "index.ts" files:

index.client.ts (no reference to require("crypto"))
index.server.ts (has reference to require("crypto"))

Then update package.json with:
  "main": "dist/index.server.js",
  "browser": "dist/index.client.js",

I can issue a PR for the above if you like (but want to make sure you are happy with the proposed solution before I do the work).

Hi @GordonSmith, see discussion in https://github.com/phosphorjs/phosphor/pull/349.

@blink1073 That PR works 100% for me (and all browser users) - but if there is a genuine need to fallback to the Node crypto library for some reason (the coreutils might be used in other scenarios like unit tests etc.), then my proposed solution might be more acceptable to @sccolbert

Does webpack honor the browser field for the entry point?

Yes (99% sure) - webpack defaults to browser = true as well (which shouldn't be a surprise).

@gordonsmith my method was a hacky workaround to be sure. We should close my pull request and replace it with yours.


chore: use process.env instead of import.meta

This pull request intends to change import.meta and use process.env instead, mainly because Playwright don't play well with import.meta and in the future, we will use Playwright instead of Cypress.

What is the mean issue with meta.env on playwright to use inside the test files?

#168


Write Excel in Python

Formatting and structure checks

[x] Have you used the correct folder and file structure? - see Contribution Guidelines

[x] Is your article properly formatted in Markdown? - see Github Markdown guide

[x] Have you used our preferred heading structure? - H3 (###) and up

[x] Does your article follow any of the suggested structure formats? - see example formats

[x] Is your article over 750 words? - Tool: Wordcounter.net

[x] Is the article you are submitting an in-depth and unique article? Does it go beyond what is in the official docs and what is covered in other blog sites. See these articles as examples.

[x] Does your article provide enough value and detail about your topic? - Articles should be clear, accurate, and fully explained.

[x] Can your article be understood by beginners? - Assume the audience is smart but has no prior exposure to the common terminology in your article.

[x] Have you included a hero image that is 600x400 pixels and have the copyright to use it? - Tools: Pixlr Image Editor for resizing and Unsplash for Creative Commons images

Grammar & plagiarism checks

[x] Have you spellchecked and grammar checked your article? - Tools: Write&Improve - ProWritingAid - Grammarly
[x] Please place your article through a 3rd party plagiarism checker. We suggest using Quetext, this tool is free and has a daily limit. We typically accept articles with 10% or less.
[ ] Have you checked your article for readability? - Tool: Hemmingway
[x] Have you added sources for quotes and images that aren't yours?

Technical checks

[x] Are your code snippets properly formatted for syntax highlighting - see Syntax guide
[x] Have you checked your code runs correctly and you've highlighted all necessary dependencies for installation?
[x] Are the software programs and packages you're highlighting in your article up to date, using current versions and not deprecated?

Topic suggestion this closes

Remove the backticks and add the issue number below to link and close your Topic Suggestion (issue) when your article has been published (PR has been merged). See this video for more details.

This closes #4191

Finally, delete the article checklist notes in blockquotes and submit your PR. We look forward to reviewing your article.

This fixes #4191

Hi @srishilesh
Thanks.  I just added the Author info along with a headshot image under /content/authors directory.

Plagiarism check done ✔

@srishilesh
Thanks for the feedback and appreciation. I would through the minor edit made.

@Bennykillua
Thank you for your patience as our team has been working diligently to work through the recent increased volume of articles coming in.
We are going through are ready articles in the order they were received and working on improving our review time as well.
🚀

@Bennykillua Thank you for your patience as our team has been working diligently to work through the recent increased volume of articles coming in.
We are going through are ready articles in the order they were received and working on improving our review time as well.
🚀

@section-engineering-education/content-moderator-team I will be reviewing this one 👍


vuejs doesn't install with using nuget PackageReference

Version
2.5.16
Reproduction link
https://github.com/BobbyCannon/VueNugetPackageReference
Steps to reproduce
Create a new web csproj but use the new PackageReference instead of Packages.config nuget format.
What is expected?
the script should be added to the website /scripts
What is actually happening?
the package is referenced but no files are actually installed

2.5.16

The vue package on NuGet is not published by the Vue.js team (we have no official NuGet support right now). You can reach out to the publisher to report the issue.

I've reached out to the owner of the nuget package. Thanks.

I`m the owner of vue NuGet package.
NuGet is not recommended for <PackageReference> import front-end packages.
For <PackageReference>, normally use Bower as the package manager for refer .js/.css and other static contents.
By the way, in Visual Studio 2017 15.8, there will have a totally new package manager for ASP.NET static content reference named "LibMan". please review  What Happened to Bower? for more details.

@BobbyCannon did you get any response from them?

The reason I'm asking - is I know how to fix this with just a couple of lines in .nuspec file, but I don't want to pollute nuget.org with another package.
Is there a repo I can submit a PR to?


This page is not c++

The name of the page is called C++. Its misleading since it looka like som C winapi.

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

ID: ac828919-7780-c626-d071-724e0cbc5d56
Version Independent ID: 47d85824-56f3-9fa1-4348-30c864bc756b
Content: C++ ODBC app accessing a SQL database - ODBC Driver for SQL Server
Content Source: docs/connect/odbc/cpp-code-example-app-connect-access-sql-db.md
Product: sql
Technology: connectivity
GitHub Login: @David-Engel
Microsoft Alias: v-davidengel

@MartinKosicky -- thank you for your feedback.  Please clarify your concern.  The code appears to be C++ to me.
cc @David-Engel for awareness

Hmm, well you are using stdio.h instead of iostream. Using goto instead if RAII objects. Code doesnt have c++ wrappers at all. You are using macros instead of lambdas. Struct defines look like in C.   The SQL code is pure C style API. I suggeat you either provide c++ wrappers and write it as C++ code or rename the documentation to C

@MartinKosicky -- thank you for clarifying.  Is the code as-is functioning for you?
@David-Engel -- please look into this issue.

yes it is functioning but,  it's not C++. You can also write it in python and it will work but it wont be C++

It compiles as C++, which is sufficient to label it as such.
Regards,
David
#please-close

The article has been renamed to "C / C++".
Regards,
David


Catalog Controller Metrics should only be available over https

Today, if it's enabled, its non-secured over http.

/close


XSS when embeding youtube video

When we try to embed youtube video using below iframe syntax, javascript code gets executed.

Steps to reproduce

Click on youtube toolbar button
Paste  in embed code box
Click ok
Observe alert

Actual result
Alert is shown with domain name
Expected result
Pasted code should be sanitized and js should not be exeucted to display alert.

Hi Team,
Any ETA on fix for this issue?

@agabhane At the moment, I do not have the time to work on this issue. Can you open a PR?

Hi @fonini , is there any ETA for this fix ?

Hi @sushruts, unfortunately, I have no time to look into this right now.

Hi @fonini , I see there are no updates around this plugin. Do you have a any ETA for this issue?

Hi @fonini , I see there are no updates around this plugin. Do you have a any ETA for this issue?


Time parse error in master

this works in 0.24.2 and not work in master:
p Time.parse("2011-08-15 12:00", "%Y-%m-%d %H:%M")

Unhandled exception: Time format did not include time zone and no default location provided (Time::Format::Error)
  from src/time/format/parser.cr:362:9 in 'raise'
  from src/time/format/parser.cr:30:9 in 'time'
  from src/time/format.cr:77:5 in 'parse'
  from src/time/format.cr:74:3 in 'parse'
  from src/time.cr:542:5 in 'parse'
  from src/time.cr:541:3 in 'parse'
  from 1.cr:3:3 in '__crystal_main'
  from src/crystal/main.cr:104:5 in 'main_user_code'
  from src/crystal/main.cr:93:7 in 'main'
  from src/crystal/main.cr:133:3 in 'main'

i find that now need to set location:
Time.parse("2011-08-15 12:00", "%Y-%m-%d %H:%M", Time::Location.local)

why not set it by default?

Because the correct default massively depends on usecase.

Well the local default seems easier for most cases.
Also doing Time.new just gives local time in this case.
Example how it is now:
puts Time.parse("2012-11-01 22:08:12", "%F %T", Time::Location.local)
puts Time.parse("Fri Oct 31 23:00:24 2014", "%c", Time::Location.local)
puts Time.parse("Fri Oct 31 23:00:24 2014", "%c", Time::Location::UTC)
puts Time.parse("20150624", "%Y%m%d", Time::Location.local)
puts Time.new

with output:
2012-11-01 22:08:12 +01:00
2014-10-31 23:00:24 +01:00
2014-10-31 23:00:24 UTC
2015-06-24 00:00:00 +02:00
2018-08-22 21:00:05 +02:00

With the default suggested the code would be:
puts Time.parse("2012-11-01 22:08:12", "%F %T")
puts Time.parse("Fri Oct 31 23:00:24 2014", "%c")
puts Time.parse("Fri Oct 31 23:00:24 2014", "%c", Time::Location::UTC)
puts Time.parse("20150624", "%Y%m%d")
puts Time.new

So for me also +1 for the defaulting to local time. If you want UTC or something else you can always give it.

There's also Time.parse! accepting %z

@w-A-L-L-e Seems you've been looking at outdated docs from 0.20.0 judging by the links you posted. The most recent API documentation is here: https://crystal-lang.org/api/latest/Time.html
There is now Time#parse_local which behaves like you described, see #6369 for details on this topic.

ah yes parse_local and parse_utc work now... great ;).  anyway it does break some older examples I found around the net, they could have also made parse work like parse_local if you omit the parameter. But I guess crystal is still young enough to do this kind of api changing ;)


Update pom.xml

Purpose
This PR updates pom.xml

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Where will Site Recovery replicated data be saved?

There are inconsistencies in the description of the Azure Site Recovery cache storage account in the current documentation. Please help to confirm and adjust the documentation, thank you.

The following section mentions that it is copied directly to managed disk, and only the copy log is written to the cache storage account.

Link: https://docs.microsoft.com/en-us/azure/site-recovery/vmware-azure-enable-replication#enable-replication

Link: https://docs.microsoft.com/en-us/azure/site-recovery/site-recovery-faq#if-i-replicate-to-azure--what-kind-of-storage-account-or-managed-disk-do-i-need-

Link: https://docs.microsoft.com/en-us/azure/site-recovery/vmware-azure-common-questions#where-does-site-recovery-replicate-data-to

2. However, the following documentation mentions that the replicated data is written to the cache storage account.

Link: https://docs.microsoft.com/en-us/azure/site-recovery/vmware-azure-tutorial#set-up-the-source-environment
I am confused whether the data copied by Site Recovery is written to the cache storage account or copied directly to the managed disk? Please help to confirm or adjust the documents.

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

ID: 101f1927-9428-20c5-06d0-ccc48bad912c
Version Independent ID: 6c23efba-9fab-8161-7885-dbddb7e737b2
Content: Enable VMware VMs for disaster recovery using Azure Site Recovery - Azure Site Recovery
Content Source: articles/site-recovery/vmware-azure-enable-replication.md
Service: site-recovery
GitHub Login: @Sharmistha-Rai
Microsoft Alias: sharrai

@AdamGuanhh Thanks for your comment. We will review the issue and get back to you shortly.

Assigning to content author @Sharmistha-Rai for document enhancement.

The documentation is correct.
The process server writes to cache storage account. The data resides temporarily in the cache storage account, and finally ASR sends it to the target managed disk and uses it to create the latest snapshot.

May I ask if the data you mentioned in "The data resides temporarily in the cache storage account, and finally ASR sends it to the target managed disk and uses it to create the latest snapshot" are you referring to disk data? The official documentation describes that the replicated items will be copied directly to the managed disk, do these two mean the same thing?

Docs link: Enable VMware VMs for disaster recovery using Azure Site Recovery - Azure Site Recovery | Microsoft Docs


Add AlembicImporter unit test

I don't understand why the namespace are not recognized :
/home/travis/build/poparteu/openMVG/src/openMVG/dataio/AlembicImporter_test.cpp:7:26: error: ‘sfm’ is not a namespace-name

Perhaps there is a missing include and so the namespace is unknown...

I think you'd better include sfm_data.hpp directly and not rely on another include to get access to it, i think it's a good practice. That being said, I don't know why it does not recognize it...


Regression in ExportSVG for Python 3.10

PartCAD started throwing an exception for build123d parts at some point last week
(always tries to use the latest version of build123d, for now).
I was able to reproduce the same exception with the following script.
It throws an exception after an update in 3.10 but not in 3.11:
import build123d as b3d
exporter = b3d.ExportSVG()

$ python test.py
Traceback (most recent call last):
  File "/tmp/1/test.py", line 3, in <module>
    exporter = b3d.ExportSVG()
  File "/home/clairbee/projects/robotics/partcad/.conda/lib/python3.10/site-packages/build123d/exporters.py", line 944, in __init__
    self.add_layer(
  File "/home/clairbee/projects/robotics/partcad/.conda/lib/python3.10/site-packages/build123d/exporters.py", line 991, in add_layer
    layer = ExportSVG._Layer(
  File "/home/clairbee/projects/robotics/partcad/.conda/lib/python3.10/site-packages/build123d/exporters.py", line 907, in __init__
    self.fill_color = convert_color(fill_color)
  File "/home/clairbee/projects/robotics/partcad/.conda/lib/python3.10/site-packages/build123d/exporters.py", line 902, in convert_color
    if isinstance(c, RGB):
  File "/home/clairbee/projects/robotics/partcad/.conda/lib/python3.10/typing.py", line 994, in __instancecheck__
    return self.__subclasscheck__(type(obj))
  File "/home/clairbee/projects/robotics/partcad/.conda/lib/python3.10/typing.py", line 997, in __subclasscheck__
    raise TypeError("Subscripted generics cannot be used with"
TypeError: Subscripted generics cannot be used with class and instance checks

$ python --version
Python 3.10.13

Ubuntu
bookworm/sid
Linux hana 6.1.0-1035-oem #35-Ubuntu SMP PREEMPT_DYNAMIC Wed Feb 21 10:58:23 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

I am not able to reproduce on python 3.10. Can you provide your installed versions of build123d, ocpsvg, ezdxf? I believe there was a discussion on discord that gumyr recommended upgrading the version of ezdxf. I am not sure what versions of ezdxf are affected but we may need to update the requirements in pyproject.toml.
@gumyr by the way, I noticed the notes for apple silicon installs are incomplete since they do not install ezdxf. https://build123d.readthedocs.io/en/latest/installation.html#special-notes-on-apple-silicon-installs
I can fix that but I would like to know more about the version requirements for ezdxf.

I noticed the ezdxf problem on my machine but not on the build servers so I thought it might be a version thing.  After upgrading it went away but I unfortunately didn't record the appropriate version numbers.  I have 1.1.4 now so maybe that should be the minimum?

ezdxf==1.0.3
ocpsvg==0.2.0
build123d==0.5.0

Upgrading ezdxf to 1.1.0 made the problem go away. Thanks!

Thank you very much for the information, I will fix the two issues mentioned above.

closed with merged PR https://github.com/gumyr/build123d/pull/638


[WOR-1346] Wording changes related to workspace access (auth domains, Azure group-constraints)

Jira Ticket: https://broadworkbench.atlassian.net/browse/WOR-1346
Introduces the following changes related to when a user does not have access to a workspace.

If the user attempts to directly access the Workspace Dashboard (through the URL), the text of the no access message has been generalized. We do not know in this case whether the workspace even exists, so we cannot differentiate between Azure and GCP.

Before:

After:

In the Workspaces list, if the user does not have access due to an auth domain (GCP) or group policy (Azure) the workspace will be in the list, but clicking it will show a dialog instead of going to the dashboard.

Before:

After:

The dialog that is opened will remain unchanged for GCP workspace, but Azure workspaces will show this one instead:

The link goes to: https://support.terra.bio/hc/en-us/articles/19124069598235
For reference, this is the GCP version of the dialog:

@cahrens for #1, should we also include the troubleshooting tip "or your linked identity may have expired"?

@cahrens for #1, should we also include the troubleshooting tip "or your linked identity may have expired"?

@bethsheets I thought it intentionally wasn't in No. 1 because that is supposed to be generic for GCP and Azure. The article seemed to be very specific about NIH-access, which comes into play for Anvil… I was assuming it was more aimed at Azure.
What was the intention behind the "linked identity may have expired", and what would you like?

Integration test failures are unrelated and have been reported to various teams.


Added support for HTTP HEAD method for akka-http and http4s

Also added minimal test cases to show that the "Unknown method" error no longer occurs.

I don't know why you're getting these test failures, they look unrelated. I'll take a look


engine client: fix env propagation to docker subcommand

Fixes https://github.com/dagger/dagger/issues/6989, confirmed with a (particularly gnarly) integ test

The integ test is passing locally but failing in CI due to docker: Error response from daemon: failed to create shim task: OCI runtime create failed: runc create failed: unable to start container process: error during container init: error setting cgroup config for procHooks process: failed to write "a *:* rwm": write /sys/fs/cgroup/devices/docker/b175032dc2eda803243528897adb744ba9957ed50bca77896bd78a68030cf52a/devices.allow: operation not permitted: unknown 🤮
It's probably fixable with the right magic configuration to disable cgroups somewhere in the dockerd service. Will push a fix and merge after.

Uh okay, the cgroup thing is a real nightmare. Unfortunately I'm just going to fall back to rm'ing the test for now. I know the issue is fixed and there's at least a comment on the modified code to reduce the chances of the fix getting lost someday.
The failure here only happens in the test workflow (not locally and not in the testdev workflow). My best guess is that:

Something in the oci runtime config created for our engine in K8s (the one that gets hit in the test workflow) is not granting full permissions to all devices in its cgroup

could also be a kernel config or a seccomp/apparmor/etc. thing

That results in dockerd's attempt to configure rwm for all devices on the privileged dagger engine container to fail because you can't increase permissions beyond your parent cgroup

I really went down a path of true insanity, to the point of configuring the dockerd service with a custom oci runtime implemented as a shell script that rewrote the oci runtime config of containers to remove that device cgroup permission,  but even that didn't work for some reason?
So this isn't worth blocking on. Unfortunately I don't really see any paths to unit testing this that wouldn't just devolve into their own forms of insanity, so I think merging without the test is the least worst option atm.


chore(podfile): update lock

Description
When running setup steps on a new install, noticed that the Podfile.lock was out of date.

Ahh this looks like the old churn we used to get because cocoa pods changed the url for the main spec repo. I think you will want to do something along these lines:
https://www.willowtreeapps.com/craft/how-to-fix-unexpected-podfile-lock-diffs-after-pod-install
tl;dr
pod repo list
# if you have a repo called 'master' or similar
pod repo remove master

Yup that fixed it @brainbicycle 👍


Lazy load optional modules to speedup pip package import

This PR lazy loads optional external Python modules to speedup initial import times of the TF pip package.
These changes speedup initial import by roughly 24%:

modules
percentage of TF import

kubernetes
14.5 %

pandas
6.9 %

scipy
2.6 %

This partially addresses #37729, but import times are still quite slow. A large chunk is spent importing the tf.compat which I don't think is used a lot by the average user, so it would be great if this could be lazy loaded as well in the future.
/cc @mihaimaruseac

Good catch, I fixed it in 818acdd9416ccbde08d2330dbe3f4070c2b92ef9

@frankchn Thanks for approving. I fixed the pylint error in the latest commit, sorry about that.

This PR was rolled back in 9f95b52fe19eb4738043482ddd24f0ddee334065.
@gbaned @frankchn Could you give some context of why this was the case?

Hi @lgeiger -- an internal test failed causing an auto-rollback. I am investigating this issue now.

Hi @lgeiger, we have resolved the internal issue and the change has been re-committed as 5a737ab. Thanks!

Awesome, thanks for taking care of re-commiting.


Fix hardcoded zpool name in scrubbed_at function

I ran into problems using this plugin due to what appears to be a hardcoded zpool name resulting in the error 'cannot open 'tank': no such pool'.
This should fix it up.

Looks good! thanks a lot 👍

I have push the fix to rubygems.

Thanks!


NDRS-955: Make tests include unbonding and dropping eras.

This changes the configuration used in the reactor tests so that the 5-era tests cause eras to become unbonded and dropped in the era validator.
This would have made the tests catch the bug fixed in https://github.com/CasperLabs/casper-node/pull/1038.
https://casperlabs.atlassian.net/browse/NDRS-955

bors r+

bors r-

bors r+


Inquiry: Multiple Examples for RequestWithBody

Is there a way to have a multiple examples in the request body?
I have a trigger that will react depending on the input of the request body for example isAutomated: true or isAutomated:false.
I already tried adding multiple examples but I think only the first one is being used. TIA

I am seeing the same behavior. Only the first example is being used.
    [OpenApiRequestBody(contentType: "application/json", bodyType: typeof(ProvisioningCreationModel), Description = "Provisioning Creation Model", Example = typeof(ProvisioningCreationModelExample))]
`    Examples.Add(
OpenApiExampleResolver.Resolve(
"ProvisioningCreationModelStudentExample",
"Provisioning Creation Model for student type",
"Provisioning Creation Model for student type",
new ProvisioningCreationModel()
{
LdsAccountId = "1234567890123456",
FirstName = "John",
LastName = "Doe",
CESUUID = Guid.NewGuid(),
UserType = "Student"
},
namingStrategy
));
Examples.Add(
OpenApiExampleResolver.Resolve(
"ProvisioningCreationModelExmployeeExample",
"Provisioning Creation Model for employee type",
"Provisioning Creation Model for employee type",
new ProvisioningCreationModel()
{
LdsAccountId = "1234567890123456",
FirstName = "John",
LastName = "Doe",
CESUUID = Guid.NewGuid(),
UserType = "Employee",
StudentAnthologyProvisioningOptionalSettings = null
},
namingStrategy
));
return this;

}`


Remove rbx-types in favor of @rbxts/types

Pull Request Test Coverage Report for Build 62

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 99.608%

Totals

Change from base Build 61:
0.0%

Covered Lines:
254

Relevant Lines:
255

💛  - Coveralls


Serf status 113: image rewrites end up in 404 errors.

Hi,
I´ve got the problem, that pagespeed brokes the image rewrites (www.currywursttest.com), which causes in 404 errors at my page. This happens after page speed runs a few days. I don´t know if the serf status 113 is related to this problems, but it seems to.
How can I debug this problem further?
nginx version: nginx/1.10.1 (Debian 8)
My configuration is:
pagespeed on;
pagespeed FileCachePath /var/ngx_pagespeed_cache;
pagespeed FileCacheSizeKb            102400;
pagespeed FileCacheCleanIntervalMs   3600000;
pagespeed FileCacheInodeLimit        500000;
pagespeed MessageBufferSize 100000;
pagespeed LowercaseHtmlNames on;
pagespeed PreserveUrlRelativity on;
pagespeed RewriteLevel CoreFilters;
pagespeed InPlaceResourceOptimization on;
pagespeed DisableRewriteOnNoTransform off;
pagespeed RespectVary on;
pagespeed FetchHttps enable;
pagespeed HttpCacheCompressionLevel 9;
pagespeed CustomFetchHeader Accept-Encoding gzip;
pagespeed Domain cdn.podlove.org;
pagespeed CssInlineMaxBytes 8000;
pagespeed EnableFilters prioritize_critical_css,inline_preview_images,lazyload_images,insert_dns_prefetch,inline_google_font_css,lazyload_images;
pagespeed EnableCachePurge on;
pagespeed ShardDomain https://www.currywursttest.com https://static.currywursttest.com,https://static2.currywursttest.com;

pagespeed Statistics on;
pagespeed StatisticsLogging on;
pagespeed LogDir /var/log/pagespeed;
pagespeed StatisticsLoggingIntervalMs 60000;
pagespeed StatisticsLoggingMaxFileSizeKb 1024;

location ~ "\.pagespeed\.([a-z]\.)?[a-z]{2}\.[^.]{10}\.[^.]+" {
  add_header "" "";
}
location ~ "^/pagespeed_static/" { }
location ~ "^/ngx_pagespeed_beacon$" { }

pagespeed StatisticsPath /ngx_pagespeed_statistics;
pagespeed GlobalStatisticsPath /ngx_pagespeed_global_statistics;
pagespeed MessagesPath /ngx_pagespeed_message;
pagespeed ConsolePath /pagespeed_console;
pagespeed AdminPath /pagespeed_admin;
pagespeed GlobalAdminPath /pagespeed_global_admin;

I could reproduce the problem, but then I requested https://www.currywursttest.com/?PageSpeedFilters=rewrite_images and the 404's went away.

Thank you. Is it critical that PageSpeedFilters could be executed by everyone?
Where do I find, what serf status 113 means? Google doesn´t tell my anything about it.

I was looking into the same, figuring out Serf's status codes doesn't seem to be straightforward.
@jeffkaufman looked into resolving Serf's error codes earlier, see https://www.jefftk.com/p/serf-error-codes. But that regrettably does not mention status code 113.
At the moment I don't understand yet how such a low number could be logged -- but a guess is that if APR_OS_START_USERERR equals zero in your case, 113 would equal SERF_ERROR_WAIT_CONN
#define SERF_ERROR_RANGE 100
#define SERF_ERROR_START (APR_OS_START_USERERR + SERF_ERROR_RANGE)

....

/* This code is for when the connection is blocked - we can not proceed
 * until something happens - generally due to SSL negotiation-like behavior
 * where a write() is blocked until a read() is processed.
 */
#define SERF_ERROR_WAIT_CONN (SERF_ERROR_START + 3)

Whoops, please ignore my previous comment, I got confused while following all the places where error definitions occur. That does not add up right at all.

It seems to happen right after reloading nginx. I could reproduce it by now again.

Serf status code 113 translates to No route to host. I've seen no other reports mentioning this error code, so I think this was a network issue. Closing this.


chore: Clear warnings

Proposed changes
Removed static from base path functions to clear warnings.  If this function is in a header it is public and should not be static.
Types of changes

[X] Other...

Windows didn't seem to care (and it should have cared, lol).
I'm assuming this function should be public to the user?  If so then I'll make the extern changes.

😂
Yes, the base endpoint should be public 👍
Thank you, @ciband !


data-rule-number broken

Subject of the issue
data-rule-number doesn't work in 1.15.0 (although it the rule does work if you pass rules: {foo: {number: true}} in the options instead of using the data-rule-* attribute approach).
Your environment

1.15.0 (served via https://cdn.jsdelivr.net/jquery.validation/1.15.0/jquery.validate.min.js)
which browser and its version

Steps to reproduce
https://jsfiddle.net/jinglesthula/amaxhbyq/
Enter abcd in the field and try to submit.  It accepts the non-numeric value.   The person who reported the bug to me mentioned that some of the other rules also weren't working using data-rule-* attributes, but I have a hunch they all stem from how numeric values are handled.
(Note: you can verify that the plugin itself is running by leaving it blank to see the required rule apply correctly.)
Also, I discovered that it was working in 1.13.1: https://jsfiddle.net/jinglesthula/swtb3Lf5/
Expected behaviour
It should show the Please enter a valid number message and prevent form submission.
Actual behaviour
Doesn't appear to be applying the rule

This commit fa5f69c529614fcf7d32e20c6ba41f8ce84f647e (PR #1391) is the culprit.

For now, you have to use
data-rule-number="true"

Unittest + PR welcome.

This appears to affect pretty much any custom rule as well that doesn't take a parameter. normalizeAttributeRule just never decides that the rule is worth adding.

@Arkni @staabm @AlexGodofsky Well confirm any data-rule-* needs ="true" using version 1.15.0


Set DPDK core requirement to 8

Set DPDK core count requirement to 8 (the recommended amount).

It's ok to raise the requirements. But for key performance testing, it needs to run in exclusive VM. It means the VM is dedicated on a host, so there is no side effect from usage of other VMs.


Remove icsDeploy.sh script and use global one instead

This change is

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from the review on Reviewable.io


Dozer partially case sensitive in 5.5.1

Field mappings are apparently now partially case sensitive in Dozer 5.5.1. There appears to be no option to switch back to the 5.4.0 case-insensitive mappings.
Example from discussion at https://groups.google.com/forum/#!topic/dozer-mapper/k6i_Bf6ZIMc.
import org.dozer.DozerBeanMapper;
public class App {
public static void main(String[] args) {
DozerBeanMapper mapper = new DozerBeanMapper();
UpperCaseBean src = new UpperCaseBean();
src.setMYSTRING("This is from the upper case bean");
LowerCaseBean dst = new LowerCaseBean();
mapper.map(src, dst);
System.out.println(dst.getmystring());
}
public static class UpperCaseBean {
private String MYSTRING;
public String getMYSTRING() {
return MYSTRING;
}
public void setMYSTRING(String MYSTRING) {
this.MYSTRING = MYSTRING;
}
}
public static class LowerCaseBean {
private String mystring;
public void setmystring(String mystring) {
this.mystring = mystring;
}
public String getmystring() {
return this.mystring;
}
}
}
With the following mapping, the "mystring" field will not be mapped:

UpperCaseBean
LowerCaseBean

However, with the following mapping, the field "mystring" will be mapped:

UpperCaseBean
LowerCaseBean

mystring
mystring

This seems like very counter-intuitive behavior. It would be nice to simply add a  configuration option to control this behavior.

This seems like kind of a HUGE problem, since it totally breaks automatic mapping for any properties with different cases in the getters or setters.

I am also facing the same problem with version 5.5.1 .please share the fix for this case sensitive issue.

Does anyone know if this has been fixed? The only workaround we've found is to manually list out every single property name in the mapping files, which is obviously a really undesirable option.

+1

I'm also facing the same issue

I getting the impression that Dozer development is stopped, unfortunately. I see there's only been 4 commits in the last year. :-(  It might be fork time...

@brian428 ; I am trying to get Dozer active again. Happy for any pull requests / help.

Closing due to inactivity.

I'd like to maintain backwards compatibility by optionally supporting case insensitive mapping. I can think of two ways to implement this:

add a configuration flag "wildcard-case-insensitive" to global and per-mapping configurations.
use a wildcard mapping strategy interface that takes the lists of unmapped fields, src and dest classes and determines pairs of src/dest fields to map. The implementation could be configured at global or class-pair-mapping level. A case sensitive strategy would be provided as default and the case insensitive one as an alternative.

@garethahealy Which would you prefer? Also, I'm unsure what should happen if a class defines two parameters that are equal except for their case. This affects both the filter step for already mapped fields as well as the field alignment.

Configuration options sounds best.


Upgrade Astro version to 0.13.0

Changes
Upgrades snowpack.dev to use Astro 0.13.0
Testing
N/A
Docs
Upgrade to the doc website only.

Handling in #3497 along with fixing hardcoded version