Recordings not getting saved

Hi,
New to Viseron, starting with default configuration file in GUI. Testing with NVIDIA CUDA image and 2 cameras. Object detection seems to be working but recordings are not getting saved. Went through docs but still couldnt figure out. Getting below error -
viseron  | [2023-02-04 18:46:13] [INFO    ] [viseron.components.ffmpeg.recorder.camera_1] - Starting recorder
viseron  | [2023-02-04 18:46:14] [INFO    ] [viseron.components.nvr.nvr.camera_1] - Stopping recording in: 9
viseron  | [2023-02-04 18:46:15] [INFO    ] [viseron.components.nvr.nvr.camera_1] - Stopping recording in: 8
viseron  | [2023-02-04 18:46:16] [INFO    ] [viseron.components.nvr.nvr.camera_1] - Stopping recording in: 7
viseron  | [2023-02-04 18:46:17] [INFO    ] [viseron.components.nvr.nvr.camera_1] - Stopping recording in: 6
viseron  | [2023-02-04 18:46:18] [INFO    ] [viseron.components.nvr.nvr.camera_1] - Stopping recording in: 5
viseron  | [2023-02-04 18:46:19] [INFO    ] [viseron.components.nvr.nvr.camera_1] - Stopping recording in: 4
viseron  | [2023-02-04 18:46:20] [INFO    ] [viseron.components.nvr.nvr.camera_1] - Stopping recording in: 3
viseron  | [2023-02-04 18:46:21] [INFO    ] [viseron.components.nvr.nvr.camera_1] - Stopping recording in: 2
viseron  | [2023-02-04 18:46:22] [INFO    ] [viseron.components.nvr.nvr.camera_1] - Stopping recording in: 1
viseron  | [2023-02-04 18:46:23] [INFO    ] [viseron.components.nvr.nvr.camera_1] - Stopping recording in: 0
viseron  | [2023-02-04 18:46:23] [INFO    ] [viseron.components.ffmpeg.recorder.camera_1] - Stopping recorder
viseron  | /recordings/camera_1/2023-02-04/18:46:13.mp4: Invalid argument
viseron  | [2023-02-04 18:46:28] [ERROR   ] [viseron.components.ffmpeg.recorder.camera_1] - Failed to concatenate segments: Command '['ffmpeg', '-hide_banner', '-loglevel', 'error', '-y', '-protocol_whitelist', 'file,pipe', '-f', 'concat', '-safe', '0', '-i', '-', '-c:v', 'h264', '-c:a', 'copy', '-movflags', '+faststart', '/recordings/camera_1/2023-02-04/18:46:13.mp4']' returned non-zero exit status 1.

Docker Compose -
version: "2.4"

services:
  viseron:
    image: roflcoopter/amd64-cuda-viseron:latest
    container_name: viseron
    restart: unless-stopped
    privileged: true
    volumes:
      - /media/data/viseronrec:/recordings
      - ~/viseron/config.yml:/config
      - /etc/localtime:/etc/localtime:ro
    ports:
      - 8888:8888
    runtime: nvidia

Config.yml -
ffmpeg:
  camera:
    camera_1:
      name: Front_View
      host: 192.168.X.X
      port: 554
      path: /Streaming/Channels/102
      username: XXXX
      password: XXXX
      stream_format: rtsp
      fps: 6
      recorder:
        codec: h264

    camera_2:
      name: Back_View
      host: 192.168.X.X
      port: 554
      path: /Streaming/Channels/102
      username: XXXX
      password: XXXX
      stream_format: rtsp
      fps: 6
      recorder:
        codec: h264

darknet:
  object_detector:
    cameras:
      camera_1:
        fps: 1
        scan_on_motion_only: false
        labels:
          - label: person
            confidence: 0.75
            trigger_recorder: true

      camera_2:
        fps: 1
        labels:
          - label: person
            confidence: 0.75
            trigger_recorder: true

mog2:
  motion_detector:
    cameras:
      camera_2:
        fps: 1

nvr:
  camera_1:
  camera_2:

Am i making any mistake?

I added filename_pattern under recorder and recordings have started working!!
recorder:
    filename_pattern: 'FrontView_%Y-%m-%d-%H%M%S'


Simfphys type vehicles are completely broken and undriveable [Garry's Mod]

Most likely this is caused by the lack of support for raycast vehicles, if i had to guess. They will have very strange suspension behavior upon being spawned, and attempting to drive one will cause it to flip all over the place uncontrollably upon making any movement inputs.

Duplicate of #107

How to fix that?

You can feel free to try and do it if you know C++ since its open source....

Actually you can't even build for GMod unless you have the newer (closed source) class layouts, so you'd need to create a test suite in the 2013 SDK making the barrier to entry extremely high (if you've ever tried compiling the SDK with a modern toolchain you know what I mean) vs just building and testing in GMod directly.
But even then you wont be able to test for 64 bit without also making the SDK compatible (removing all the inline asm for instance).
ofc you can just write blindly and not test anything
From https://github.com/Joshua-Ashton/VPhysics-Jolt#how-to-build

Unfortunately we cannot redistribute the additional code/headers needed to build Volt for games such as Garry's Mod which uses a different VPhysics interface (CS:GO's) to what is found in the Valve-provided public SDKs.


entity search bug fix v2.0.1

tags for indicators and sightings renamed from Phantom to Splunk SOAR
fixed issue when no entities found via "search by entity id" action
fixed issue when no entities found via "search entity" action with observable value or entity title
new package added in builds folder

@mikhva-eiq can you pull the main branch for new structure of the app then please make your commits in this structure (under source).
Also please include a new build in builds following the existing naming convention

Dupe of #6


ClassCastException instead of ComparisonFailure when comparing Double to Integer using usingRecursiveComparison()

Summary
In my case I have two complex objects which have a "value" member variable of type "Object". When I compare two of those objects via "usingRecursiveComparison()" and one of the objects has a double value and the other an integer value, I get a ClassCastException instead of a  ComparisonFailure as expected.
Example
This behavior can also be reproduced via a simplified comparison:
assertThat(Double.MAX_VALUE).usingRecursiveComparison().isEqualTo(Integer.MAX_VALUE) // ClassCastException instead of expected  ComparisonFailure

Is this an expected/intended beahvior? Is there a workaround for more complex objects where the same field of type "Object" could have such incompatible types?
Kind regards,
Thomas

Hi @Sandared,
I have tried your example and it gave an introspection error:
org.assertj.core.util.introspection.IntrospectionError:
Can't find any field or property with name 'value'.
Error when introspecting properties was :
- No getter for property 'value' in java.lang.Double
Error when introspecting fields was :
- Unable to obtain the value of the field <'value'> from <1.7976931348623157E308>
	at org.assertj.core.util.introspection.PropertyOrFieldSupport.getSimpleValue(PropertyOrFieldSupport.java:91)
	at org.assertj.core.api.recursive.comparison.RecursiveComparisonConfiguration.dualValueForField(RecursiveComparisonConfiguration.java:631)

Could you give us a reproducible test case?

If I wrap the values in a class like:
  @Test
  void test() {
    MyObject actual = new MyObject(Double.MAX_VALUE);
    MyObject expected = new MyObject(Integer.MAX_VALUE);
    assertThat(actual).usingRecursiveComparison().isEqualTo(expected);
  }

  static class MyObject {
    Object value;
    MyObject(Object value) {
      this.value = value;
    }
  }

I get a ClassCastException:
java.lang.ClassCastException: class java.lang.Integer cannot be cast to class java.lang.Double (java.lang.Integer and java.lang.Double are in module java.base of loader 'bootstrap')
	at org.assertj.core.util.DoubleComparator.compareNonNull(DoubleComparator.java:17)
	at org.assertj.core.util.NullSafeComparator.compare(NullSafeComparator.java:25)
	at org.assertj.core.util.DoubleComparator.compare(DoubleComparator.java:17)
	at org.assertj.core.api.recursive.comparison.RecursiveComparisonDifferenceCalculator.propertyOrFieldValuesAreEqual(RecursiveComparisonDifferenceCalculator.java:688)

Since you are trying to compare doubles with integer, I think that behavior makes sense as a default behavior.
Now what you could is register your own way of comparing numbers for specific fields, this test works:
MyObject actual = new MyObject(Double.valueOf(Integer.MAX_VALUE));
MyObject expected = new MyObject(Integer.MAX_VALUE);
BiPredicate<Number, Number> closeEnough = (n1, n2) -> Math.abs(n1.doubleValue() - n2.doubleValue()) <= 0.5;

assertThat(actual).usingRecursiveComparison()
                  .withEqualsForFields(closeEnough, "value")
                  .isEqualTo(expected);

Note that at the moment, using  withEqualsForType(closeEnough, Number.class) instead of withEqualsForFields(closeEnough, "value") does not work as AssertJ selects its default Double comparator over closeEnough, whether it is the desired behavior is debatable, on one hand the Double comparator is an exact match for the Double value to compare but OTOH user own comparator / BiPredicate should have precedence. We still need to think what the best approach is here.

Hi @joel-costigliola
thanks for digging into this issue :)
I just verified that I got the ClassCastException also with the minimal example and I can reproduce it on GitPod where I use the following JDK:
openjdk 11.0.13 2021-10-19 LTS
OpenJDK Runtime Environment Zulu11.52+13-CA (build 11.0.13+8-LTS)
OpenJDK 64-Bit Server VM Zulu11.52+13-CA (build 11.0.13+8-LTS, mixed mode)

I created a repository for that here: https://github.com/Sandared/assertj-bug-1
If you want to try it for yourself you can use this link https://gitpod.io/#https://github.com/Sandared/assertj-bug-1
I also added a testcase that more or less resembles the usecase I have in production. It is very similar to what you did with the MyObject. We have a "PropertyDTO" datatype that can hold virtually any value in form of an object.
Wouldn't a better number comparison be something like this?
BiPredicate<Number, Number> exactly = (n1, n2) -> new BigDecimal(n1.toString()).compareTo(new BigDecimal(n2.toString())) == 0;

If I understand your workaround (and the current limitations regarding withEqualsForType(closeEnough, Number.class)) corrctly that would mean for my case that I have to do the comparison logic for that field completely customized, as the value field in my case could not only hold Numbers. Do I understand that correctly?
Kind regards,
Thomas

I just thought about your workaround: the example in the documentation of withEqualsForFields() implies that I have to know the fields exactly up front, i.e., I cannot use regexes like .*?properties for a properties field that might exist on different levels of a recusive object hierarchy.
E.g., if I have a class like this:
public class HierarchicalNode {
  List<HierarchicalNode> children;
  List<Property> properties;
}

I would have to explicitly define every nested properties field up front, e.g., node.properties, node.children.properties, node.children.children.properties etc. or am I getting this wrong?

Sorry for spamming :/
Regarding this test case of yours:
  @Test
  void test() {
    MyObject actual = new MyObject(Double.MAX_VALUE);
    MyObject expected = new MyObject(Integer.MAX_VALUE);
    assertThat(actual).usingRecursiveComparison().isEqualTo(expected);
  }

  static class MyObject {
    Object value;
    MyObject(Object value) {
      this.value = value;
    }
  }

I really think, that a ClassCastException is not an expected behavior here. Actually I would just expect the assertion to fail, as those two objects are not equal. So I would expect a failing test and not a test in error.
Given the case you have a more complex object, e.g.,
@Test
  void test() {
    MyObject a = new MyObject(Double.MAX_VALUE);
    MyObject b = new MyObject(Integer.MAX_VALUE);
    MyComplexObject actual = new MyComplexObject(a, b);
    MyComplexObject expected= new MyComplexObject(b, a);
    assertThat(actual).usingRecursiveComparison().ignoringCollectionOrderInFields("values").isEqualTo(expected);
  }

  public static class MyComplexObject {
    private List<MyObject> values;
    public MyComplexObject(MyObject a, MyObject b) {
      this.values = List.of(a,b);
    }
  }

Wouldn't that then sometimes lead to failing the test, although the lists are actually equal?
At least this is what I observe in my case.
Kind regards,
Thomas

Thanks for the detailed case, you have a point on the MyComplexObject example which should have worked, ClassCastException should probably be reported as comparison failures instead of blowing the comparison (still need to think about it more but that looks reasonable).

Hi Joel,
thanks, glad I could clarify that case :)


BONText’s description should include typical description information

In addition to printing a character-by-character summary of the string, It should start with a typical description with a character count or other useful information:
<BONText: 0x7fbc78e13490, characters: 23:
[character 0]
[character 1]
...
[character 22]
> // ← end of BONText description

That // ← end of BONText description would actually be in the output, so as not to be confusing (“does my string have a > character at the end?”). Could also leave out the > and just go with [end of BONText description]

Ended up also doing a lot of cleanup and testing on descriptions.


History does not work with vertices

Describe the bug
Hey,
When using the History Plugin, it realised the vertices event are received by the beforeAddCommand but they are not actually handled by the plugin. Maybe it's a limitation of the plugin which would be sad, but maybe it's just a mistake.
Currently when we add/remove vertices they are ignored by the undo/redo feature.
PS: the function beforeAddCommand sometimes receive one argument and sometimes two, but not the same order, which makes it harder to use and I didn't understand why.
Your Example Website or App
https://codesandbox.io/s/new-sky-n37gp1?file=/src/app.tsx
Steps to Reproduce the Bug or Issue

Add a vertice
Click undo
The step before the vertice is undo but not the vertice

Expected behavior
The vertice add/remove should be part of the stack.
Screenshots or Videos
https://user-images.githubusercontent.com/1637651/222148276-39ffdfb1-3c0e-4710-98d3-f566c442c4d0.mov
Platform

OS: macOS
Browser: Chrome
Version: 110

Additional context
No response

Please upgrade to<EMAIL_ADDRESS>


Add clockworksms action

This doesn't pass linting, please check and resolve

Nevermind, the linting is resolved now. Thanks!


[preset] Add missing libicu dependencies to package presets

We were accidentally using the system libicu instead of building it,
causing version mismatches.

@swift-ci please test

Build failed
Swift Test OS X Platform
Git Sha - ccb41f044ac7bfb61e76460b7df59bfca6bbf286

Build failed
Swift Test Linux Platform
Git Sha - ccb41f044ac7bfb61e76460b7df59bfca6bbf286


Clone Capybara Session?

Hi!
As poltergeist is not thread-safe and it's necessary to instance a new Capybara::Session for each thread, is there any way to clone a session so that we don't need to log in more than once?
Thanks!

There's no built-in support for cloning a session, but if you're using cookies for your session storage you can read the relevant cookies from an existing session and then set them on new sessions you create using page.driver.cookies and page.driver.set_cookie


Clarification

Hi,
Firstly, I think this is great, thanks for the initiative!
I was wondering - could you confirm if for config changes, I would need to supply the entire modified config in the payload? Lets say I want to modify my aliases, would I need to make the modification to the json file in its entirety, and then submit it?
Thanks!
James

Yes, you need to provide the full modified payload, this is because it's not a "real" API that hooks into each respective control-layer of the pfSense web application, it is indeed a fauxAPI that works as a configuration manipulation interface to the pfSense config.xml file.
If you take a closer look at the Python client-lib it does provide a way to submit configuration sections, however you notice that it simply does a config_get to obtain the existing config file first and then overwrites with the user supplied (new) configuration section.
There is no escaping this, fauxAPI is what it is because pfSense does not have a well defined or standardised application control layer to interface with - to this end, the OPNSense fork appears to have adopted a well defined application architecture with a control layer API and will likely become the preferred dev-ops solution that fauxAPI set out to gap-fill for.


FIX: Allow any dtype-specifier for ndimage output

Reference issue
Fixes bug introduced in gh-18841.
What does this implement/fix?
As of #18841, calling ndimage.map_coordinates(..., output='float32') fails with:
File "/opt/conda/envs/fmriprep/lib/python3.11/site-packages/sdcflows/transform.py", line 105, in _sdc_unwarp
	resampled = ndi.map_coordinates(
	            ^^^^^^^^^^^^^^^^^^^^
File "/opt/conda/envs/fmriprep/lib/python3.11/site-packages/scipy/ndimage/_interpolation.py", line 455, in map_coordinates
	output = _ni_support._get_output(output, input, shape=output_shape,
	         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/opt/conda/envs/fmriprep/lib/python3.11/site-packages/scipy/ndimage/_ni_support.py", line 96, in _get_output
	output = f_dict[output]
	         ~~~~~~^^^^^^^^
KeyError: 'float32'

This replaced output = np.sctypeDict[output], which would normalize scalar types to their numpy types. Looking at the codes in that dict, it is basically equivalent to:
def __getitem__(self, key)
    dtype = np.dtype(key)
    if len(dtype) != 0:
        raise KeyError(key)
    return dtype.type

Which will fail for record types, but not much else.
The proposed solution just uses np.dtype() directly and moves on. If desired, I could add more checks, such as that len(output) must be 0 or output.type must not be void or must be numeric, or whatever the actual intent of this section is.
Additional information
xref nipreps/sdcflows#433

It seems reasonable to allow output="float32" etc indeed.
From poking around a bit with scipy 1.9, it seems that ndimage emits a reasonable error if asked a non-numeric dtype:
In [37]: from scipy import ndimage
    ...: >>> import numpy as np
    ...: >>> a = np.arange(12.).reshape((4, 3))
    ...: >>> a
    ...: >>> ndimage.map_coordinates(a, [[0.5, 2], [0.5, 1]], order=1, output=np.dtype([('a', int), ('b', 'float')]))
....
<snip>
...
--> 456     _nd_image.geometric_transform(filtered, None, coordinates, None, None,
    457                                   output, order, mode, cval, npad, None, None)
    458     return output
RuntimeError: data type not supported

I still think it'd be useful to filter out non-numeric types earlier, via something like
dt = np.dtype(output)
if not issubclass(dt.type, np.number): raise ...

np.number comes from https://numpy.org/doc/stable/reference/arrays.scalars.html, under an assumption that map_coordinates expects numeric inputs and can handle all of them.

Okay, I added a check. Do you want some tests fully exercising _get_output()? Or just that it handles sensible dtypes and errors on inappropriate ones?

The closer you can move this Achilles to the turtle, the better :-)

Added tests. Having a bit of trouble getting an editable install up and running, so testing locally is very slow. Hopefully the linter caught all the bugs in the tests.

Let's see what the CI says.
Locally it's easiest to use $ python dev.py, does it not work for you?

Ah, thanks for the pointer. I tried python dev.py for something but possibly had a messed up virtual environment, so just switched to pip install -e .[test], thinking that should work. Figured out dev.py and fixed up the tests. This should be good to go.

Thanks for the reality check @rgommers. Hopefully this is a more reasonable number of tests, written more clearly.

thanks for your first contribution to SciPy @effigies !


Thematize Icon component

Description
Yamada UI components are tied to a theme. However, the Icon component does not have a theme.
Problem Statement/Justification
Themes have defaultProps, which can be utilized to share props throughout the project. However, the Icon component does not have a theme, so it cannot take advantage of them.
Proposed Solution or API
No response
Alternatives
No response
Are you willing to participate in implementing this feature and create a pull request include the implementation?
None
Additional Information
No response

I'll try it.


kairos-agent 2.4.3 can not build rpi4 image

When building with the latest kairos-agent for rpi4 (2.4.3 via osbuilder:latest (v0.10.2), the MBR can't be written.
Reverting osbuilder to v0.10.1 and therefore using kairos-agent 2.4.1 at least no error is shown.
At the moment the rpi4 is still not able to boot and reports "Firmware not found" at the pre efi screen (grub is not reached).
Any ideas?

It's the version we are using on master:
https://github.com/kairos-io/kairos/blob/master/Earthfile#L24-L25
How are you running it?

via :latest
docker run -v $PWD:/HERE -v /var/run/docker.sock:/var/run/docker.sock --privileged -i --rm --entrypoint=/build-arm-image.sh quay.io/kairos/osbuilder-tools:latest \
 --model rpi4 \
 --state-partition-size 6200 \
 --recovery-partition-size 4200 \
 --size 15200 \
 --images-size 2000 \
 --local \
 --config /HERE/cloud-config.yaml \
 --efi-dir /HERE/boot \
 --docker-image kairos-rpi-ogi /HERE/build/out.img

this is at the moment the only way how I can build an image on the mac...

now I'm using v0.10.1 instead of :latest but this is the result:

further I noticed:

and fixed it via:

BUT it doesn't help
Looks like beside the the problem that the disk is not properly formatted there is another problem with MBR for rpi or so....

the first partition (EFI) is empty...

OK if just tested with the prebuild rpi4 image
export IMAGE=quay.io/kairos/opensuse:leap-15.5-standard-arm64-rpi4-v2.4.3-k3sv1.28.2-k3s1-img
docker run -ti --rm -v $PWD:/image quay.io/luet/base util unpack "$IMAGE" /image

The partition table is also screwed up (check with fdisk -l) and has to be fixed via parted /dev/sda unit s print on a linux machine BUT the first partition is properly populated and therefore rpi can boot.
So there are 2 problems:

there is a difference on how the image is build in CI and how the custom image is build via build-arm-image.sh
there is a common problem on how the partition table is written

Hope this helps to narrow it down.

I'm trying the command @Ognian is using with some latest image:
docker run -v $PWD:/work -v /var/run/docker.sock:/var/run/docker.sock --privileged -i --rm --entrypoint=/build-arm-image.sh quay.io/kairos/osbuilder-tools:latest  --model rpi4  --state-partition-size 6200  --recovery-partition-size 4200  --size 15200  --images-size 2000    --config /work/cloud-config.yaml  --efi-dir /work/boot  --docker-image quay.io/kairos/ubuntu:23.10-arm64-rpi4-master  /work/build/out.img

and I'm getting this error:
...
+ cp -rfv /tmp/arm-builder.t7U3Muskbc/etc/kairos/branding/grubmenu.cfg /tmp/grubmeny.cfg.5gvXju
cp: cannot stat '/tmp/arm-builder.t7U3Muskbc/etc/kairos/branding/grubmenu.cfg': No such file or directory
...

originating here: https://github.com/kairos-io/osbuilder/blob/e4482ddc088e8141a705c9c24f86775abef0a6db/tools-image/build-arm-image.sh#L304

I'm stupid, that's a "generic" image I pulled :facepalm: . Trying again...

ok this one succeeds:
docker run -v $PWD:/work -v /var/run/docker.sock:/var/run/docker.sock --privileged -i --rm --entrypoint=/build-arm-image.sh quay.io/kairos/osbuilder-tools:latest  --model rpi4  --state-partition-size 6200  --recovery-partition-size 4200  --size 15200  --images-size 2000    --config /work/config.yaml    --docker-image quay.io/kairos/ubuntu:23.04-standard-arm64-rpi4-v2.4.3-k3sv1.27.6-k3s1  /work/build/out.img

I had to remove --efi-dir because I don't know what @Ognian has in  --efi-dir /HERE/boot. I'll try to find what this directory is supposed to have.

I used the contents of this zip file: https://github.com/pftf/RPi4/releases/tag/v1.35 as my --efi-dir and I built an image with this command:
docker run -v $PWD:/work -v /var/run/docker.sock:/var/run/docker.sock --privileged -i --rm --entrypoint=/build-arm-image.sh quay.io/kairos/osbuilder-tools:latest  --model rpi4  --state-partition-size 6200  --recovery-partition-size 4200  --size 15200  --images-size 2000    --config /work/config.yaml  --efi-dir /work/boot  --docker-image quay.io/kairos/ubuntu:23.04-standard-arm64-rpi4-v2.4.3-k3sv1.27.6-k3s1  /work/build/out.img

Then I flashed the image with:
cat build/out.img | sudo dd of=/dev/sda oflag=sync status=progress bs=10MB

and booted a raspberry pi 4 with the card. I got /sys/firmware/efi populated (means I'm booted with efi).
I don't get any warnings when I do fdisk -l /dev/mmcblk0 .
@Ognian are you not booting from an sdcard? Why does yours appear as /dev/sda ? Also, what are the contents of your --efi-dir ?

@jimmykarily yes I'm booting from an external USB3 disk (SSD); the content of the --efi-dir is only the file extraconfig.txt which extends the rpi hardware config.txt by setting additional HW parameters, overclocking etc... In previous releases the efi dir was populated with it's proper content, so I had to copy only the "additional" things I needed, therefore I'm using only the extraconfig.txt there.
I'm using the opensuse:leap standard image but I also get the grub copy error.

I tried the same command (see above) with this image: quay.io/kairos/opensuse:leap-15.5-standard-arm64-rpi4-v2.4.3-k3sv1.27.6-k3s1 and it still works with no errors.
Our differences are:

the contents of the --efi-dir (to be honest, I'm not even sure the files I put there are what is expected)
the image itself (You are using a custom build? I'm using the released one)
the SSD (I'm using the sdcard and I don't have a disk to try)

Maybe this helps narrow it down.

Looking again, I see you had the fdisk -l problem also with the released image. Then the problem must be on the external disk somehow, because the rest of the setup is similar to mine.

I've ordered an nvme disk and a USB enclosure, it should be here next week. Unless someone else beats me to it, I will try again as soon as its here.

@jimmykarily if I follow your process, I can reproduce the issue when trying to boot from an external SSD

I had no issues booting the resulting images built with earthly for the 2.5.0 via SSD. I wonder if the issue is not related to those efi artifacts?
@jimmykarily @Ognian


spire-agent: 1.4.0 -> 1.4.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/spiffe/spire/releases.
meta.description for spire-agent is: "The SPIFFE Runtime Environment"
meta.homepage for spire-agent is: "https://github.com/spiffe/spire"
meta.changelog for spire-agent is: "https://github.com/spiffe/spire/releases/tag/v1.4.1"
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.4.1 with grep in /nix/store/0p66wkhy5xafcs219cv2vypsfjxwqyf7-source

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath
spire
spire-agent
spire-server

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/0p66wkhy5xafcs219cv2vypsfjxwqyf7-source \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A spire-agent https://github.com/r-ryantm/nixpkgs/archive/6bc734f2f5565c24c4e9280c10d344a6690c5c2e.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/0p66wkhy5xafcs219cv2vypsfjxwqyf7-source
ls -la /nix/store/0p66wkhy5xafcs219cv2vypsfjxwqyf7-source/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

3 packages built:

spire
spire-agent
spire-server

Maintainer pings
cc @jonringer @fkautz for testing.

duplicate #190179


BUG: nodemon is not a dev dependency

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
Nodemon need to be added as dev dependency for it to work. Server doesn't start currently and throws error when npm run dev is used
Before

After adding as dev dependency

I would like to work on this
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC'24 contributor
[X] I want to work on this issue

@hereisSwapnil
Can you add GSSoC labels on this?


Ex Nihilo

text: Ruling #1 has been reversed.


Cloudflare worker for backend calls

Need a worker to be the middle layer between the dashboard and backend to pass in authentication items and tokens needed to authenticate with BQ. Tokens can be handled there.

what backend calls are we expecting? aren't we talking to hlx RUM only?


I just wanted to say

This package is so good. You guys have done a really really good job. It worked for my template in about an hour on nearly the first try. Thank you.

Thanks, much appreciated! 👍


HTTP 431 error when using the "edit in playground" button for later stages in a task

I'm getting an HTTP 431 error when using the "edit in playground" button for later stages in a task, where the context is long. Apparently 431 is "Request Header Fields Too Large" so seems like the state is just too long to be put in the URL?

Yes, that's probably the cause. Thanks for pointing this out!

probably switching to putting the state in a POST request or something would work?

Maybe. In this case, the flow is:

User clicks the "edit in playground" button
Runs page UI code constructs a URL pointing to the playground, containing the generation request as a query parameter, and navigates to that URL
Playground UI code parses the query parameter and updates the page's state, then removes the query parameter from the current URL

We could have the "edit in playground" button act like a form submission, making a POST request to the backend and having the backend return a pre-rendered playground page. This is difficult to do with our current backend infrastructure. The UI is a bunch of static files, served by Caddy. We haven't done any server-side rendering before.
It would be more convenient to put the generation request in localstorage and read it from there.

And in fact the playground page already caches the last executed generation request in localstorage, so this could be pretty easy to do.

Maybe window.postMessage?


Fix: JS: Improve error handling of batch addition

Crash if there is an error in batch addition.
Example:
const usearch = require('usearch');
const index = new usearch.Index({ metric: 'l2sq', connectivity: 16, dimensions: 3 });
index.add(42n, new Float32Array([0.2, 0.6, 0.4]));
index.add(
  [41n, 42n, 43n],
  [[0.1, 0.6, 0.4], [0.2, 0.6, 0.4], [0.3, 0.6, 0.4]]
)

42n is duplicated, which causes an error, but then it crash.
It seems that it is not better to throw an exception during the process, so we changed it to throw it after it is completed.

This patch fixes the crash, but is there a better way?
I am drafting this because I would like to change it if there is a best way to fix it.


Illuminate\Http\Request fullUrl() does not preserve the original URL's order

Laravel Version: 5.2.*
PHP Version: 5.6.14
Database Driver & Version:

Description:
Using Request's fullUrl() with an array in the parameter does not preserve the original URL. Looks like alphabetical ordering is taking place.
Steps To Reproduce:
Add to routes.php:
Route::get('test', function(Illuminate\Http\Request $request) {
	return $request->fullUrl();
});

Visit a URL:
http://localhost:8000/test?array[b][value1]=value1&array[b][value2]=val2&array[a][value3]=value3&array[a][value4]=value4

Array visualized (b is BEFORE a):
[
  b: [ value1: "value1", value2: "value2" ],
  a: [ value3: "value3", value4: "value4"]
]

Output in browser (note that array[a] came first):
http://localhost:8000/test?array%5Ba%5D%5Bvalue3%5D=value3&array%5Ba%5D%5Bvalue4%5D=value4&array%5Bb%5D%5Bvalue1%5D=value1&array%5Bb%5D%5Bvalue2%5D=val2

That looks like the correct behaviour, yes.


0.62.0

Shai hi,
Thanks for the PR. Merged and published :)
https://wealth.forcrowd.org/_system/content/contributors
If you would like to update your info later on, feel free to send me a new request.
Hope it was easy to setup and instructions were clear btw? Let me know if you had any issues.
Also, if you would like to work on our project, you are more than welcome, again just let me know.

Thanks @coni2k ! I would be happy to contribute to this project.
Do you have any road map of features you plan to be working on? Are you open for any suggestions for improvments?

Yes, I am definitely open for suggestions, please send me if you have any ideas.
There is no proper to do list at the moment but I'm planning to work on it.
Today and tomorrow I will be on the way, so I can answer bit late. And afterwards we can talk about what we can do.

Shai hello again,
Sorry for my late reply, lately it was pretty busy here.
Here is a quick list of ideas for the project.

angular2 / asp.net core: Last update was about typescript. Now I would like to study these two technologies a little bit. If it's easy to switch, I'll try to upgrade the application.
New UI / content: Current UI is not good as I want. It should be easier to use and understand. Probably I will move current extensive content into medium articles and try to have admin dashboard kind of UI for the application.
Embeddable: What I would like to create is some sort of survey application and I would like to experiment with making it embeddable to other websites. Since I'm planning to use medium for the content part, it should be embeddable to medium for instance. So, inside of your article, you can ask some questions and make your article bit more interactive.
Custom formulas: Current application has some field types and based on these types, it creates cells with a fixed formula. On top of this, I'd like to introduce custom formulas, at least basic operations (addition, subtraction, sum, average etc.) Make the application more excel-like, so you can create your own field with custom formulas.

And of course there are lot of small items to work on as well.
If you find any of it interesting, just let me know and we can look how we can work together. As you can see, I'm learning through this process as well. So, again, if you have any ideas, please feel free to share.


Sample code of sending Opencv Frame via WebGear, please?

Question

The amount of effort and quality code you've put into this project is quite impressive, thank you for your hard work!
I've taken a look into WebGear and am wondering if you can provide sample code of how it would be possible to use Opencv Frames as WebGear source (as opposed to a file or rtsp stream).
Thank you!
Acknowledgment

[x] A brief but descriptive Title of your issue.
[x] I have searched the issues for my issue and found nothing related or helpful.
[ ] I have read the FAQ.
[ ] I have read the Wiki.

Context

Your Environment

VidGear version:
Branch:
Python version:
pip version:
Operating System and version:

Optional

@cesarandreslopez

how it would be possible to use Opencv Frames as WebGear source (as opposed to a file or rtsp stream).

There's no direct way of doing this yet but it's not difficult. Here's the trick to do it:
Step-1: First run any WebGear example to trigger the auto-generation process. The default generated will be saved at $HOME/.vidgear directory of your machine.
Step-2: Build your own Frame Producer Route: create script code which must look something like this:
#import necessary libs
import uvicorn, asyncio, cv2
from starlette.routing import Route
from vidgear.gears import WebGear
from vidgear.gears.helper import reducer
from starlette.responses import StreamingResponse

#initialize WebGear app with a valid source you want to use
web = WebGear(source = "/home/abhishek/stmarc_video.avi", logging = True) #also enable `logging` for debugging

#create your own frame producer
async def my_frame_producer():
	# loop over frames
	while True:
		#read frame from
		frame = web.stream.read()
		#break if NoneType
		if frame is None: break

		#do something with frame here

		#reducer frames size if you want more performance otherwise comment this line
		frame = reducer(frame, percentage = 50) #reduce frame by 50%
		#handle JPEG encoding
		encodedImage = cv2.imencode('.jpg', frame)[1].tobytes()
		#yield frame in byte format
		yield  (b'--frame\r\nContent-Type:image/jpeg\r\n\r\n'+encodedImage+b'\r\n')
		await asyncio.sleep(0.01)

#now create your own streaming response server
async def video_server(scope):
	assert scope['type'] == 'http'
	return StreamingResponse(my_frame_producer(), media_type='multipart/x-mixed-replace; boundary=frame') #add your frame producer

#append new route to point your own streaming response server created above
web.routes.append(Route('/my_frames', endpoint=video_server)) #new route for your frames producer will be `{address}/my_frames`

#run this app on Uvicorn server at address http://<IP_ADDRESS>:8000/
uvicorn.run(web(), host='<IP_ADDRESS>', port=8000)

#close app safely
web.shutdown()

Step-3: Change Webpage address in your own HTML file: Go to $HOME/.vidgear/templates directory of your machine and change the line - 25 on index.html file:
From:

To:

Final Step: Run the above script and see output at address http://<IP_ADDRESS>:8000/ on your browser.
It's a dirty fix but will do the job intended :smiley:

hey @abhiTronix    is there anyway that i can use WebGear with react


Colab tutorial notebook errors out

Bug Description
Colab Notebook (fl_asr_tutorial_inference_ctc): https://colab.research.google.com/github/flashlight/flashlight/blob/master/flashlight/app/asr/tutorial/notebooks/InferenceAndAlignmentCTC.ipynb
Reproduction Steps
I'm using the GPU runtime (CUDA backend) on the Colab.
Running the second cell with the cmake .. -DCMAKE_BUILD_... command produces the following error:
env: MKLROOT=/opt/intel/mkl
env: ArrayFire_DIR=/opt/arrayfire/share/ArrayFire/cmake
env: DNNL_DIR=/opt/dnnl/dnnl_lnx_2.0.0_cpu_iomp/lib/cmake/dnnl
-- -rdynamic supported.
-- CUDA found (library: /usr/local/cuda/lib64/libcudart_static.a;-lpthread;dl;/usr/lib/x86_64-linux-gnu/librt.so include: /usr/local/cuda/include)
-- CUDA architecture flags: -gencodearch=compute_30,code=sm_30-gencodearch=compute_35,code=sm_35-gencodearch=compute_50,code=sm_50-gencodearch=compute_52,code=sm_52-gencodearch=compute_60,code=sm_60-gencodearch=compute_61,code=sm_61-gencodearch=compute_70,code=sm_70-gencodearch=compute_75,code=sm_75-gencodearch=compute_75,code=compute_75
CMake Error at cmake/InternalUtils.cmake:126 (message):
  FL_BUILD_CORE Required to build FL_BUILD_APP_ASR
Call Stack (most recent call first):
  flashlight/app/CMakeLists.txt:11 (fl_dependent_option)
  CMakeLists.txt:247 (include)

Platform and Hardware
Colab with GPU.
Additional Context
[Add any additional information here]

Hi @vwrj, I have been facing the same bug, it occurs on all the machines I tested (EC2 DL VM and local). Have you been able to resolve this bug?

@pranavpawar3 using the previous flashlight release works -- thank you!


Added two columns to table

Made report for email address policies more useful (returns templates)

Some updates to your very nice ERAC

Some updates to your very nice ERAC

Thanks Brian - will take a look
From: Brian Reid @.>
Sent: 26 April 2023 17:53
To: spgoodman/ExchangeRecipientAdmin @.>
Cc: Subscribed @.***>
Subject: Re: [spgoodman/ExchangeRecipientAdmin] Added two columns to table (PR #8)
@brianreidc7 commented on this pull request.
Some updates to your very nice ERAC

Reply to this email directly, view it on GitHubhttps://github.com/spgoodman/ExchangeRecipientAdmin/pull/8#pullrequestreview-1402427876, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADUACNOGZPSJTMBKKSDD6ALXDFHFJANCNFSM6AAAAAAXMWOVFY.
You are receiving this because you are subscribed to this thread.Message ID<EMAIL_ADDRESS>


Add lily_push_unset and document calling keyopt functions from C.

define my_keyopt(:a a: *Integer = 0, :b b: *Integer = 0, :c c: *Integer = 0) {}

This function can't be called from C in a way that only a and c are set. The interpreter needs to add lily_push_unset so that C can invoke this kind of function. This addition should include an example somewhere on the site that explains how to use it.

Closing this as part of the gitlab migration: https://gitlab.com/FascinatedBox/lily/issues/356


Tag a stable version

# composer req profiler

  [InvalidArgumentException]
  Could not find package symfony/profiler-pack at any version for your minimu
  m-stability (stable). Check the package spelling or your minimum-stability

require [--dev] [--prefer-source] [--prefer-dist] [--no-progress] [--no-suggest] [--no-update] [--no-scripts] [--update-no-dev] [--update-with-dependencies] [--ignore-platform-reqs] [--prefer-stable] [--prefer-lowest] [--sort-packages] [-o|--optimize-autoloader] [-a|--classmap-authoritative] [--apcu-autoloader] [--] [<packages>]...

Done

Should it be 1.0 or 3.3? Because it's 3.3 in symfony/recipes: https://github.com/symfony/recipes/tree/master/symfony/profiler-pack/3.3


AK: Factorize implementation of Optional's operator==

Ah, I only tried with clang-20 and gcc-14 on my machine, looks like it can't compile with gcc-13.

GCC 13 was confused during overload resolution because of the redefinition of the operator in explicit specialization of Optional like Optional<String>, Optional<FlyString> and Optional<JS::Value>. And I'm glad it did because we can actually delete these overload as well!

(macOS CI seems unrelated)

I don't understand the commit message here.

factorize (verb): to express a number in terms of its factors

As I understand this change, it's about making operator== for Optional<T> be a free function instead of defining it for multiple specializations of Optional?

Factorize is used in programming to say merging common implementation, probably not as much as I thought, though. The most convincing usage that I found is here, and I wouldn't use it as a source :sweat_smile:.
I was quite sure it has already been used in the repo.
$ git log --grep=Factorize --shortstat | grep Author | sort -u
Author: Lucas CHOLLET<EMAIL_ADDRESS>
... it's definitely not used as much as I thought :eyes:.

Making it a free function was just a side effect, but as far as I remember it doesn't change anything on the usage of the operator, right?


[GHSA-653v-rqx9-j85p] deep-object-diff vulnerable to Prototype Pollution

Updates

Affected products
Description

Comments
The fix listed above is pointing to the wrong version of v1.1.19 which appears to be a typo the correct fixed version is v1.1.9. This is causing issues in pnpm/npm audit as the latest version of v1.1.9 is less than v1.1.19 (doesn't exist) so we just need to change this to point to the correct version.

This contribution has been closed because it is a duplicate of #828.

Thanks for updating with the dupe @CallmeMari. Any chance we can get #828 merged?

I apologize for the delay, we are having technical difficulties with getting #828  but rest assured that the correction has been made yesterday. We appreciate your patience with this issue.


Simplifying code

The code in history_tree_node.rs and append_only_zks.rs can be further simplified. A few suggestions:

[ ] Remove any remaining instances of unwrap().
[ ] Look for repeated code among conditionals and abstract it out into a separate function.

I believe this is almost done, except that we also need to get rid of all sources of panic, such as assert

I looked for assert statements and it seems now we only have asserts in test cases. We could likely close this issue.


Evolve the schema of the Connection resource status

Part 1 of 2 to address #123
Summary of Changes
Change the status object in Connection resources to able to represent the statuses of the different external clusters/resources that connections can define.
Any additional details or context that should be provided?
The ConnectionStatus OpenAPI spec schema now looks like:
    ConnectionStatus:
      required:
      - authentication
      type: object
      properties:
        ccloud:
          $ref: "#/components/schemas/CCloudStatus"
        kafka_cluster:
          $ref: "#/components/schemas/KafkaClusterStatus"
        schema_registry:
          $ref: "#/components/schemas/SchemaRegistryStatus"
        authentication:
          $ref: "#/components/schemas/Authentication"

The status.authentication object is unchanged, but is being deprecated and replaced with the status.ccloud object. The status.ccloud, status.kafka_clsuter and status.schema_registry nested objects all use the ConnectedState enum:
    ConnectedState:
      enum:
      - NONE
      - ATTEMPTING
      - SUCCESS
      - EXPIRED
      - FAILED
      type: string

and there is a direct mapping from the older Status enumeration to this new enum.
The status.authentication object is being generated from the status.ccloud object, and the sidecar code has already transitioned to only using status.ccloud; the status.kafka_cluster and status.schema_registry objects will be set in a subsequent PR.
We will keep the status.authentication object for a time being, so the extension has time to migrate to the status.ccloud, status.kafka_cluster and status.schema_registry objects. Once that migration has been performed, we will remote the status.authentication object altogether, in a future PR.
Pull request checklist
Please check if your PR fulfills the following (if applicable):

Tests:

[ ] Added new
[ ] Updated existing
[ ] Deleted existing

[x] Have you validated this change locally against a running instance of the Quarkus dev server?make quarkus-dev

[x] Have you validated this change against a locally running native executable?make mvn-package-native && ./target/ide-sidecar-*-runner

Huh, I'd have expected CI to catch that indentation issue. I guess Checkstyle doesn't care about it.


[TB] Dropdown molecule example not reflecting the changes being made in the settings

Problem/Concern
If I set the border radius for the dropdown molecule to be 0px, I would expect the dropdown to be rectangular with 90 degree corners.  If I change the selection to be full item instead of "left border only", I would expect the dropdown to show selected menu items where the item background color changes when selected or hovered, but I am not seeing that.
Proposed Solution

Border radius is being removed.

Given Bryce's comment above, and my screenshot of no border radius setting present:

This issue is complete.


feat: add a pkg mgmt onboarding issue template

Fixes issue #2

Thanks, @rchincha
I copied the output to a test repo and filled it out to see what it would look like for a contributor:
There's a lot of great information captured, but also wondering if it's steep for engaging package manager devs wanting to participate. And, how would contributors iterate on the contents of the issue? The owner of the issue could keep editing...
I'm also wondering what we'd do with the issue as issues tend to be relatively short-lived tracking items. Do we want to track all the package managers as issues, or should we capture these as markdown files within the repo for longevity and reference?
Another approach might be to create a markdown template package-managers/package-tempalte.md and ask folks to copy the template and make iterative PRs to that description. This way there's a low bar to get the package managers added, and we can iterate on the contents with several folks contributing.
We likely still want to track various package managers to submit the markdown template, but could the contents of the template be a bit lighter?
title: "[PkgMgmt]: <name>"
labels: ["pkgmgmt"]
body:
  - type: textarea
    attributes:
      label: "Describe the package management system"
      description: "Try to describe your package management system as well as you can, as that will help with speeding up onboarding"
  - type: textarea
    attributes:
      label: "Package Manager Tracking Issue?"
      description: "Links to requests to participate in the package manager ecosystem"

Thank you for tackling this @rchincha.

Thanks, @rchincha I copied the output to a test repo and filled it out to see what it would look like for a contributor: There's a lot of great information captured, but also wondering if it's steep for engaging package manager devs wanting to participate. And, how would contributors iterate on the contents of the issue? The owner of the issue could keep editing... I'm also wondering what we'd do with the issue as issues tend to be relatively short-lived tracking items. Do we want to track all the package managers as issues, or should we capture these as markdown files within the repo for longevity and reference? Another approach might be to create a markdown template package-managers/package-tempalte.md and ask folks to copy the template and make iterative PRs to that description. This way there's a low bar to get the package managers added, and we can iterate on the contents with several folks contributing.
We likely still want to track various package managers to submit the markdown template, but could the contents of the template be a bit lighter?

I'm on the fence. I like the proposed template, but also understand @SteveLasker's apprehension with the seemingly steep requirements for submission and the short-lived nature of GH Issues. Can we find a compromise here, where package manager maintainers submit issues, have a discussion about the specifics with the working group in that thread, and then close the issue with a PR that commits the discovered information to the repo?

@afflom @SteveLasker
If we already know where those PRs would write to (pkgmgr/* perhaps) we can just make it a generic markdown template so folks can submit a PR.

@rchincha, absolutely, 👍

Working on porting this into a template in the repo itself.

Fixed, please take a look.


BiDirectional dataflow

please ! add LiveLink data to MoBU input  !! please

Thanks for the comment.
Could you describe what you would like exactly? Which data should be sent back to Mobu? Who takes ownership and how?


Pagination underflow protection

the current Offset() method in the pagination package has a risk to underflow when Page is set to 0. For example:
if Page is set to 0, and PerPage is set to 10, the currently calculated offset would be<PHONE_NUMBER>3709551606, which is of course incorrect.
Adding a guard check here prevents this from happening.

What's the status of this?

@ladydascalie Let's get a decision on this.


Fix "invalid port in upstream" on nginx controller

If you don't have the ingress port mapped same as the service, you got the error "invalid port in upstream" on nginx. To fix this, we should check if the targetPort is valid (>0).
Fix #53

This change is

Coverage decreased (-0.02%) to 39.753% when pulling bda66469053ee38278c20587f8ce167b616c7748 on gianrubio:branch/invalid-port into 0af8cccc7f195d8d4cceb6af074be381c04e797a on kubernetes:master.

Please don't merge this. I'll write a warning msg rather hide the error.

/lgtm

@gianrubio sorry for the delay in the review


[AutoDiff] Fix wrong diagnostic for indirectness and diagnose unreachable.

The diagnostic emission method added for indirectness was implemented very wrong. It should use the recursive emission logic instead of forcing an error on the top-level invoker. This PR fixes that.

Reject functions that need to be differentiated but do not have a return. Resolves TF-7.

@swift-ci please test tensorflow


Implementations, documentation and changes related with the calibration

Contains a new structure of the calib module in agreement with the different calibration modules in the WBS. Incluides README files for the code developers.
Implementation code to calibrate MC simtel array files. Python implementation of the current functions in reconstruction.c (hessioxxx) and corresponding test functions.
Example macro of how to calibrate the MC files (tested only in Prod2).
First draft on the documentation about the different calibration modules.

Note: Test functions does not seems to work well due to some issues with the camera geometry.

That's strange, round is a python builtin:
https://docs.python.org/3.5/library/functions.html#round


Use getMorphClass instead of the full quantified class for target_type

Hi, it`s a polymorphic relationship, it would be better using the name as set on the morphMap instead

As per this issue, I don't think this will help:
https://github.com/thomasjohnkane/snooze/issues/40
Feel free to let me know if i'm wrong :)

Ok yeah makes sense I haven't thought about that


About the InputLayerAdj

Hi,
I have trouble running the code. It raises TypeError: Input layers to a 'Model' must be 'InputLayer' objects. Does it mean that custom the input layer is not allowed?
I'm wondering whether this is due to the keras version, or you have modified the Keras source code.

I changed the keras version and solved the problem:)


Orphaned Application resources

Following application related resources seem to be left around after deleting the application.

Application's docker container image
Application's task definition in ECS

Steps:

cld env create
cld app deploy
cld app redeploy
cld app delete

This is no longer an issue so closing it.


I can’t log into my MyAnimeList account

I’m trying to log into MyAnimeList using Edge, but I keep getting an error

Thank you for bringing this issue to my attention, unfortunately I haven't had a chance to investigate this sooner or catch it myself.
Apparently, the extension ID on Chromium-based browsers has changed. Not sure how that happen, but I've updated the oAuth configuration on MAL's side and you should be able to log into your account. No update is necessary.
Please, reach out if the issue persists.


Bug#148 -

Fix NullPointerException for BuilderDependencyHelper as the use of Files.listFiles call in downloadDependenciesAndGetPath method inside a for loop was making it unpredictable and throwing NPE if temp dir is empty , incorrect or gets cleaned up .

I have read the CLA Document and I hereby sign the CLA


Is rsmq support for Redis5+ redis stream

Is rsmq support for Redis5+ redis stream?

RSMQ works differently than Redis Streams. Although some parts can be compared to the XACK etc. functionality. RSMQ works with a Redis LUA script as its main function.

RSMQ works differently than Redis Streams. Although some parts can be compared to the XACK etc. functionality. RSMQ works with a Redis LUA script as its main function.


Add opt-in support for System.Text.Json.Serialization

While it is quite straightforward to add System.Text.Json.Serialization support to IJsonValue<T> through a generic JsonConverter<T>, this is not usually the right choice.
The optimal code path is to manage the lifetime of a JsonDocument (Dispose()-ing it when you are done with your call path) and construct your JSON values from its JsonElements, as this avoids a managed Clone() of the underlying memory.
While serialization support avoids any additional allocation, deserialization means we are not in control of the JsonDocument, and so it always incurs the cost of a Clone().
However, without a custom JsonConverter, deserialization of our extended JSON types causes a stack overflow error as it tries to use the standard object converter (which recurses into the AsAny property which returns itself which has an AsAny property...etc...
Our proposed solution is to implement such a converter, with a static EnableInefficientSerializationSupport property. This is false by default which causes an explanatory exception to be thrown on both Read and Write.
A considered but discarded scenario is to allow Write() but not Read() as write is efficient. However, I feel that the lack of symmetry would cause confusion. We could enable Write() later without this being a breaking change.

Actually - we didn't discard the asymmetry. We embraced it.
I have renamed the enabling property EnableInefficientDeserializationSupport


[BUG] Cannonballs does not trigger

Describe the bug
Edgeville furnace full inventory of steel bars and the action bar does not appear whether I use space or I click with the mouse.
I disabled every single other plugin and it still does not trigger. When I string magic shortbows the action bar does appear.
I tried both enabling and disabling the smithing cannonballs option in the settings but it simply does not work.
To Reproduce
Steps to reproduce the behavior:

Go to Edgeville
Click on furnace with steel bars and ammo mould
Make cannonballs and see no progress bar

Expected behavior
Normally the action bar is supposed to appear.
Screenshots
NA

What type of mould are you using? Works well for me with the double ammo one using either the spacebar or clicking on the option at the Edgeville ufrnace

Maybe the issue here is the small ammo mould. I observe the same issue as @alotOSRS using a normal ammo mould. I do not have the double ammo mould to confirm tho.

Happening to me at edgevile with a plain ammo mould

Found the issue. It was specific to the plain ammo mould. Made a fix which is waiting for review on Runelite. Should be live in few days.


[FLOC-3840] Add --process-dependency-links to the release pip install step

To install our forked version of testtools, we need to specify --process-dependency-links when performing pip install. This flag was missing from the script which sets up the release environment.

LGTM. Thanks!


"aesctr" test failing on ESP32-C3

While testing the esp32c3-devkit:crypto configuration I found out that the aesctr is the only test that is failing.
This issue was also confirmed by @acassis. Does anyone have any idea of why this is happening?
Thanks in advance!
Log:
nsh> aescbc
aescbc test ok
nsh> aesxts
OK encrypt test vector 0
OK decrypt test vector 0
OK encrypt test vector 1
OK decrypt test vector 1
OK encrypt test vector 2
OK decrypt test vector 2
OK encrypt test vector 3
OK decrypt test vector 3
OK encrypt test vector 4
OK decrypt test vector 4
OK encrypt test vector 5
OK decrypt test vector 5
OK encrypt test vector 6
OK decrypt test vector 6
OK encrypt test vector 7
OK decrypt test vector 7
OK encrypt test vector 8
OK decrypt test vector 8
OK encrypt test vector 9
OK decrypt test vector 9
OK encrypt test vector 10
OK decrypt test vector 10
OK encrypt test vector 11
OK decrypt test vector 11
OK encrypt test vector 12
OK decrypt test vector 12
OK encrypt test vector 13
OK decrypt test vector 13
nsh> des3cbc
ok, encrypt with /dev/crypto, decrypt with /dev/crypto
nsh> hmac
hmac md5 success
hmac md5 success
hmac md5 success
hmac sha1 success
hmac sha1 success
hmac sha1 success
hmac sha256 success
hmac sha256 success
hmac sha256 success
nsh> aesctr
10: CIOCCRYPT: Invalid argument
10: crypt with /dev/crypto failed
10: CIOCCRYPT: Invalid argument
10: crypt with /dev/crypto failed
10: CIOCCRYPT: Invalid argument
10: crypt with /dev/crypto failed
10: CIOCGSESSION: Invalid argument
10: crypt with /dev/crypto failed
10: CIOCGSESSION: Invalid argument
10: crypt with /dev/crypto failed
10: CIOCGSESSION: Invalid argument
10: crypt with /dev/crypto failed
10: CIOCCRYPT: Invalid argument
10: crypt with /dev/crypto failed
10: CIOCCRYPT: Invalid argument
10: crypt with /dev/crypto failed
10: CIOCCRYPT: Invalid argument
10: crypt with /dev/crypto failed

@anjiahao1 please take a look.


It is possible to send data through the Chat Control (iframe)?

I'm investigating the possible use of a certain Snrio:
Suppose my client  click on link that leads to my  Bot site Contains the Chat Control.
the link also contains user Id (POST).
so it is possible to send the bot this user id  as soon as the customer enters the site? so the Bot can perform a specific process Against this User?
Thanks

Sounds like really valid scenario, @danmarshall is it possible for someone to pass data to chat control iframe? I am not sure though how would the interoperability will work with this.

in addition, i know that i can use Direct Line API but It means I'll have to implement full client side (for example if the API (from bot side) send Cards with buttons to the client the user see only text and if i want it to show like in the Chat control or like in the  emulator i should build this.. ) so if i can use the Built chat control and still Send through information It can be very effective.

Hi @Lipa9963 yes you can pass these additional URL parameters to your webchat iframe:

userid
username

This needs to be added to the documentation here and in the dev portal.

@danmarshall Works. Thanks!

@Lipa9963  how do you achieved this? Like this?
<iframe src="https://webchat.botframework.com/embed/smilesbot?s=<secret_code>&userid=1234&username=Praveen
" style="height: 502px; max-height: 502px;"></iframe>

On Wed, Jan 11, 2017 at 1:27 PM, Praveen R<EMAIL_ADDRESS>wrote:

@Lipa9963 https://github.com/Lipa9963 how do you achieved this? Like
this?

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/BotBuilder/issues/1987#issuecomment-271845153,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AWFzXVKR9d1YciB4lerNvMwV4IZ6ZxXwks5rRLydgaJpZM4LdkCZ
.

Sounds like this issue has been resolved. Closing now. If I'm in error, please reopen!

Hi guys,
how can I read such parameters from the bot side?
Thank you very much

@insanediv is your bot in Node or C# ?

Hi @danmarshall it is a node bot

@insanediv see https://github.com/Microsoft/BotBuilder-Samples/tree/master/Node/core-GetConversationMembers

@danmarshall is it possible to use this mechanism to pass arbitrary parameters to the bot via the web chat embed code?

@insanediv no. For arbitrary values, you will need to host your own custom webchat page.
See the "backchannel" in the WebChat repo readme and example.

@danmarshall Thank you very much.
I managed to pass custom parameters to my bot, but have another question:
How is it possible to store such parameters to the session.userData object?
I show you my code:
Client side
 botConnection.postActivity({type: "event",
                             value: "customarg",
                             from: {id: "me" },
                             name: "customEvent"})
                            .subscribe(id => console.log("success"));

Server side
bot.on("event", function (event) {
    var msg = new builder.Message().address(event.address);
    if (event.name === "customEvent") {
        // HOW CAN I STORE event.value IN session.userData ?
        console.log(event.value);
    }
});

Thank you again

I finally managed to solve this. For anyone that needs please visit
my github page

For those who would like to read the user ID and user name from a C# bot, they are available in the turn context, which in my bot is available in methods
OnMessageActivityAsync(ITurnContext<IMessageActivity> turnContext, CancellationToken cancellationToken) in class DialogBot.cs, and in OnMembersAddedAsync(IList<ChannelAccount> membersAdded, ITurnContext<IConversationUpdateActivity> turnContext, CancellationToken cancellationToken) in class QnABot.cs.
In these methods, turnContext.Activity.From.Id contains the passed-in user ID, and turnContext.Activity.From.Name contains the passed-in user name.

@insanediv see https://github.com/Microsoft/BotBuilder-Samples/tree/master/Node/core-GetConversationMembers

I couldn't open this link,
any idea how I could access the data passes in the iframe src in the bot?
I am using nodejs
thanks


Fix build_custom example.

Add required kind.

 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.


Remove extra new lines from the converted file

After a file has been converted, a new line is add after every line of content.
This behavior makes annoying the CSV usability without removing the additional lines.
Thank's in advance.

Did you find a fix for this? I am having this issue as well


Added connector-id flag to kubectl-gs login

What does this PR do?
Documented a new connector-id flag supported by kubectl-gs login command and adjusted the documentatoin of the authentication process.
What does it look like?
There is a new entry in the list of kubectl-gs login flags describing the new flag
Any background context you can provide?
Kubectl-gs login command was extended to support multiple identity providers. During the login process it prompts the user to select an identity provider.
A new flag was also added to skip the prompt by providing an identifier of a specific identity provider.
This PR ensures that the above changes are reflected in the documentation
What is needed from the reviewers?
Reviewers can check whether the new entry in the kubectl-gs login documentation makes sense to them
Have you maintained the front matter?

[x] (Please bump the last_review_date in case this qualifies as a review for an entire page. Provide user_questions which should be answered in the page. Provide a meaningful description.)

Here is an update of the schema image: https://docs.google.com/drawings/d/1UD1Pw9UxzCEO7WMtp57MOizkVqU6Oq5IiUdTbhU0VWo/edit
You can download the PNG from there.

Thanks. Schema image updated.


:robot: Triggering CI on branch 'release-next' after synching to upstream/main

/test 47-e2e-aws-ocp-47

/lgtm
/approve

/retest

/retest

/close


Add support for receiving control characters

This helps receiving special control characters that are ignored by GLFW, such as backspace ('\b') or newline ('\n').
Mainly to fix Vietnamese typing through Unikey in raygui (which requires certain changes to raygui).

@nthung-2k5 Sorry, I can't merge it, changes in GLFW must be issued in GLFW repository.


#77 Close fullscreen using the control fix

Inspired by #78 fixing bug #77 but with added tests.

Inspired by https://github.com/mderrick/react-html5video/pull/78 fixing bug #77 but with added tests.

I have 2.0.0 but the fullscreen button still only enters fullscreen. Going out doesn't work.

@Bram-Zijp I hadn't deployed it as I was going to bundle in a another fix before deploying. I just released v2.0.1. It's being published as we speak. Let me know if it works.
https://travis-ci.org/mderrick/react-html5video/builds/209022178

Works like a charm, thanks. Side note; I had to add these styles to get the default player playbar icon to display correctly (they were to small).
    ._1CiqE svg {
            width: 100%;
            height: 100%;
        }

I wouldn't rely on those classnames as they change per build. For now, I'd add a custom className to the Video and target svgs from there.
Could you please create an issue with the the exact browser version and I will look on fixing it. Thank you!


Remove unused fixes

Fix #568.

May I push to that branch? The Flato13 recipe does currently not process MIROC-ESM because of that issue, with this PR it should be possible.

You can also merge this branch into your one.

That one doesn't exist anymore, but I will open a new one once this PR is merged.

If it's just about adding 1 line to the flato13 recipe you can do it here.

Is he around?

If not, I would recommend waiting till he is back from holidays.


Show Subtitles

I have enabled to display subtitles on my youtube settings for every video, but I don't get them in AVPlayer. Is there a setting in XCDYoutubeKit to enable them?

Unfortunately, there isn’t.

are there alternatives?

@ogezue You will most likely have to do download the video and then process it by adding the subtitles manually. If you want to show then while streaming (which is what you seem that you want) then you will have to build additional custom work.

I don't have any idea how to achieve this. Can you point me into the right direction?
Why is it working in Browser?


Adds specific vsphere templates for kube versions in ci

Issue #, if available:
Description of changes:
This builds on the work @g-gaston added to support e2e testing against previous builds of eks-a.  Currently we do not have a great way to handle the upgrade from last minor tests once the new branch/release is using newer versions of eks-d.  In the ci vsphere cluster we have always shared template which will only have the latest eks-d tag on it.  This will break the previous tests due missing the previous eks-d version tag.
I have imported the last two eks-d versions for 0.11 and main/release-0.12 into the ci cluster with their own unique names.  This sets all the new vars for the e2e tests to use, I think automatically?
Testing (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/unhold
/approve

/cherrypick release-0.12


Deprecating SDKs for Microsoft.StorSimple and Microsoft.HybridData

Hello @patelkunal , it looks like you've specified a deprecation date for many of the packages that has already passed. As per the Azure Retirement Policy, the date needs to be at least 12 months out for Track 1 packages and 36 months out for Track 2 packages.
From the guidance:
For Track 1 SDK libraries, product teams may announce "end of support in 1 year" after a replacement Track 2 library is available. Support during the 1 year may be limited to just critical security fixes. Track 2 libraries must follow the standard 3-year notification.


Problem with workflow = [('train', 1),('val',1)]

I'm wanting to see the validation loss, so looking at previous issues, I added the workflow = [('train', 1),('val',1)] to this, but I'm getting the following error: AttributeError: 'ConfigDict' object has no attribute 'dataset'
What could be causing this?
Thanks

Maybe you can check the config format for symbol errors.

I had the same issue, was able to trace it back to this line here: https://github.com/open-mmlab/mmdetection/blob/master/tools/train.py#L223
Here the validation set pipeline is set up as a copy of the train set pipeline.
val_dataset.pipeline = cfg.data.train.get(             'pipeline', cfg.data.train.dataset.get('pipeline'))
The code tries to access cfg.data.train.dataset, which doesn't exist and thus throws this error. To fix it, I changed L225 to
datasets.append(build_dataset(cfg.data.val))
and deleted the previous lines that copy the train set config into the validation set config. Not sure why this was happening in the script anyways, but maybe I am missing something here?


Automated cherry pick of #530: fix guest resume on migrate failed, fix snapshot action log

Cherry pick of #530 on release/2.8.0.
#530: fix guest resume on migrate failed, fix snapshot action log

/lgtm
/approve

/lgtm


only set host when server addr is IP (close #447)

Codecov Report

Merging #451 (554bbd5) into main (93759ff) will decrease coverage by 0.28%.
The diff coverage is 73.91%.

@@            Coverage Diff             @@
##             main     #451      +/-   ##
==========================================
- Coverage   80.54%   80.26%   -0.29%
==========================================
  Files          70       70
  Lines        8154     8175      +21
==========================================
- Hits         6568     6562       -6
- Misses       1227     1252      +25
- Partials      359      361       +2

Impacted Files
Coverage Δ

pkg/filter/proxy/server.go
70.37% <68.42%> (-0.22%)
:arrow_down:

pkg/filter/proxy/request.go
95.45% <100.00%> (+0.14%)
:arrow_up:

pkg/object/mqttproxy/session.go
74.23% <0.00%> (-7.98%)
:arrow_down:

pkg/cluster/cluster.go
49.65% <0.00%> (-1.03%)
:arrow_down:

pkg/object/mqttproxy/client.go
77.67% <0.00%> (-0.94%)
:arrow_down:

Continue to review full report at Codecov.

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


Minor UI issue if we Skip Backup.

In the case of skip backup, when we go to set the password, there is a dot of background image which overlaps the "got it".

https://github.com/bithyve/hexa/commit/94edafcc58058ab33adbb86c4e498f9789e26c49

Verified this issue on dev app v2.0(351)


feat: update opentrons fixed trash svg in LL

Overview
The Opentrons fixed trash image looks really wonky, so we need a new svg to replace it.

Acceptance criteria

[ ] The fixed trash image is an accurate representation of what the trash bin looks like
[ ] The new fixed trash image shows up in the online LL and in the App

New JIRA ticket: https://opentrons.atlassian.net/browse/RAUT-204


pkg/workerpool(cdc): fix race between handling error and unregistering handler

What problem does this PR solve?
Issue Number: close #4447
What is changed and how it works?

Fixed race between handling error and unregistering handler in pkg/workerpool. The root cause is that, if an error has already caused a handler to be automatically unregistered, Unregister() will return immediately, regardless of whether the error handler has returned. This has caused race in unified sorter.

Check List
Tests

Unit test
Integration test

Side effects

Increased code complexity

Related changes

Need to cherry-pick to the release branch

Release note
Fix stability problem in workerpool, which is used by Unified Sorter.

/run-leak-tests
/run-integration-tests

/run-leak-tests

/run-verify

/run-check-issue-triage-complete

/check-issue-triage-complete

/run-verify

/run-all-tests

/run-integration-tests

/run-integration-tests

/run-integration-tests

/merge

/run-integration-tests

/run-integration-test /tidb=pr/32434

/run-integration-test /tidb=pr/32434

/run-integration-test /tidb=pr/32434


Once Its restarted the processing of message goes smoothlyin Pubsub Sink Connector
May i know why i have this issue?
Even though i get the DEADLINE_EXCEEDED exception the message is getting posted to Pubsub.
THe only problem is the offsets are not getting updated so i keep publishing the same message to the Pubsub topic and everytime i publish the message i get the DEADLINE_EXCEEDED exception
Please help

Please delete this issue. I will track it under kafka connectors


Error when building in vite app

First things firts, great module!
I'm encountering a weird error. When building my app with vite build, the app crashes due to not managing to import this module since it is compiled to commonjs and not es-module.

Thanks for the feedback.
I was able to reproduce the issue and I will look into it.

I opened PR #104 that should address this.

THANKS!

Thanks @bell-steven for contributing!
I hope to find some time to look at it tonight.

Released in version 3.1.3


docs; update docs on website

The docs for setting up luxe on the website is outdated compared to the docs in the docs-src branch.

Can you elaborate on what part specifically? I updated the website so the site is current, the docs-src is old.

Oh yeah, I thought it was the other way around!
However, the website docs are still outdated -- I was referring specifically to the guide saying haxelib install hxcpp 3.2.205 rather than the most recent one. There's also other clearly outdated instructions such as
Install the latest stable Haxe release, 3.2.1
luxe requires Haxe 3.2.0+.
Haxe 3.3.x and 3.4.x are not stable releases yet.

Right yea, this was an intentional concession while I/we test 3.4.
While it works great so far there for a majority, there are still several pain points with hxcpp on Windows, Linux, and Android right out of the box being a dead end. I would prefer that new users aren't thrown directly into a mess until those things are made clearer, so I haven't updated this just yet.
I will soon though.

Roger that!


[ci] fix R CMD CHECK note about example timings (fixes #4049)

This PR attempts to fix #4049. See https://github.com/microsoft/LightGBM/issues/4049#issuecomment-793412254 for a more detailed explanation of this proposal.

Haha we cannot win with CI recently! Now some builds here are failing because of an outage at GitLab (where Eigen is hosted).
https://status.gitlab.com/

https://github.com/microsoft/LightGBM/pull/4055/checks?check_run_id=2063377218
Fetching submodules
  /usr/local/bin/git submodule sync
  /usr/local/bin/git -c protocol.version=2 submodule update --init --force --depth=5
  Submodule 'include/boost/compute' (https://github.com/boostorg/compute) registered for path 'external_libs/compute'
  Submodule 'eigen' (https://gitlab.com/libeigen/eigen.git) registered for path 'external_libs/eigen'
  Submodule 'external_libs/fast_double_parser' (https://github.com/lemire/fast_double_parser.git) registered for path 'external_libs/fast_double_parser'
  Submodule 'external_libs/fmt' (https://github.com/fmtlib/fmt.git) registered for path 'external_libs/fmt'
  Cloning into '/Users/runner/work/LightGBM/LightGBM/external_libs/compute'...
  Cloning into '/Users/runner/work/LightGBM/LightGBM/external_libs/eigen'...
  Error: fatal: unable to access 'https://gitlab.com/libeigen/eigen.git/': The requested URL returned error: 502
  Error: fatal: clone of 'https://gitlab.com/libeigen/eigen.git' into submodule path '/Users/runner/work/LightGBM/LightGBM/external_libs/eigen' failed
  Failed to clone 'external_libs/eigen'. Retry scheduled
  Cloning into '/Users/runner/work/LightGBM/LightGBM/external_libs/fast_double_parser'...
  Cloning into '/Users/runner/work/LightGBM/LightGBM/external_libs/fmt'...
  Cloning into '/Users/runner/work/LightGBM/LightGBM/external_libs/eigen'...
  Error: error: RPC failed; HTTP 502 curl 22 The requested URL returned error: 502
  Error: fatal: error reading section header 'shallow-info'
  Error: fatal: clone of 'https://gitlab.com/libeigen/eigen.git' into submodule path '/Users/runner/work/LightGBM/LightGBM/external_libs/eigen' failed
  Failed to clone 'external_libs/eigen' a second time, aborting
  Error: The process '/usr/local/bin/git' failed with exit code 1


Ricejasonf/map

Note that this is branched off of my original PR #226.

Sorry I'm going to close this. It was a lot simpler to make new, appropriately names branch off of benchmarks/map and cherry-pick the commit from this. I can do a new PR when make check is complete.
I would like to know if I even came close to what you guys were talking about regarding the type-level hash, which was the intent of this PR.

Sorry I'm going to close this
Why close this?

I would like to know if I even came close to what you guys were talking about regarding the type-level hash, which was the intent of this PR.

Yes, this is very close, except that if two hashes collide then we'll get the wrong results. Each bucket has to contain a list of elements whose hash is that of the bucket, and then we'll do a linear search inside the bucket. Or you could use basically any hashing scheme (linear probing or something more exotic), if you wish.

I was thinking that the point of normalizing the integral constants was to make
m[size_c<5>] == m[int_c<5>]
If the buckets contain a list of elements that are linearly searched using Comparable then it seems pointless to put int_c<5> and size_c<5> in the same bucket. Is there ever a case where two IntegralConstants will be equal but have different types?
I would like to add some tests for the above when you get back to me on the desired behaviour. I would also add tests to check that map ultimately uses Comparable, but I argue that there is no realistic use that justifies it.

I was thinking that the point of normalizing the integral constants was to make
m[size_c<5>] == m[int_c<5>]

Correct. More specifically, the requirement is that if hana::equal(x, y), then hana::equal(hash(x), hash(y)). This ensures that we fall into the right bucket all the time, since the hash of a value determines the bucket in which it will end up in the map.

If the buckets contain a list of elements that are linearly searched using Comparable then it seems pointless to put int_c<5> and size_c<5> in the same bucket.

Looking up in such a map is a two-step process. First, we compute the hash of the key, which gives us the bucket. Now, many different keys can have the same hash, and hence fall into the same bucket, since the hash is not expected to be perfect (collision free). Once we've found the bucket, we then do a linear search within this bucket to find the key that we're interested in. If hash(int_c<5>) != hash(long_c<5>), then they will fall into different buckets, and we'll never find int_c<5> if we search for long_c<5> (and vice versa).

Is there ever a case where two IntegralConstants will be equal but have different types?

Yes, for example int_c<5> == long_c<5>, but decltype(int_c<5>) != decltype(long_c<5>).

ef2f06f I made the buckets contain indices of map's storage a tuple.  With regard to benchmark.at_key.lookup, there is a greater upfront cost, but lookups appear to scale better (4s vs 21s @n=200). find and erase_key use the key based lookup so I expect they will perform better as well. The other functions should be unchanged from master as the map's storage is once again a tuple.

BTW, detail::hash_table could easily be made the back-end for hana::set as well.

For my particular use case I have a better way than trying to use sets as keys. Since sets or other containers as keys would probably always result in a linear search, then perhaps it is better to delete the default implementation for hash so if people try to use something as a key they know what they are getting into.

I tried using all_of to check if all of the keys were "fast", meaning they could never have a hash collision. It shaves about 10% off of the at_key.index_of_lookup benchmark, but loses the ability to check for duplicates at creation time. I won't bother pushing, but I thought it was worth mentioning.

I just pushed 295f5fc to fix hash for type_tag.

Superseded by #247, where latest changes on the Hashable have been made. #247 also shown a small compile-time speedup over this PR, due to different techniques being used.


FEATURE REQUEST: Adding support for WEBP images

Current version of Django Filer doesn't support Webp images. WebP lossless images are 26% smaller in size compared to PNGs and WebP lossy images are 25-34% smaller than comparable JPEG images at equivalent quality. It's probably going to become a web standard at some stage, therefore adding support for this format would be a great improvement.

Before adding Webp support to django-filer, we need that in easy-thumbnails. So please ask there first. If Pillow supports webp, then creating a pull request on easy-thumbnails will not be that hard.

Thanks for your quick answer @jrief, I have added a Feature request in easy-thumbnails repository. I will update this thread when I get news from them.

Using WebP with django-filer actually is quite easy. Here is a short instruction.
That said, django-filer does not support WebP natively. This means that the source (or canonical image) mus be in a format supported by ~100% of all browsers, currently ~80%. Whenever we reach that point, let's discuss this again.

If you want to generate webp thumbnails, you can use these settings (thanks @benzkji):
THUMBNAIL_EXTENSION = 'webp'
THUMBNAIL_TRANSPARENCY_EXTENSION = 'webp'

That means you don't have to follow this tutorial anymore.

@corentinbettiol not completly true. If you need compatibilty, you might still need to, as the settings above just output webp for everything ,-). Though there also is a shortcut, when providing the original image (unthumbnailed) as a fallback for non webp capable browsers, you'll not need any signals and the like. As most people still upload jpeg anyways, this can work very well. Only drawback is download size, as these jpges will be originals, and probably quite heavy ;)

Also, as of version 3.0 filer will accept the WebP format for uploaded images, too (the source format).
This might be interesting if you want to manually control the format images are delivered: With THUMBNAIL_PRESERVE_EXTENSIONS = ('webp',) all WebP uploads would be delivered as WebP.


Add queso-ui v9

What's this PR do?
See title
Why are we doing this? How does it help us?
Keeps us current with latest style iterations
How should this be manually tested?
make
npm install
make restart
Click around the portal and make sure you don't see any broken styles. Nothing drastic has changed for this one so it's pretty unlikely
How should this change be communicated to end users?
N/A
Are there any smells or added technical debt to note?
Nah
What are the relevant tickets?
An outcome of https://3.basecamp.com/3098728/buckets/736178/todos/2480540270
Have you done the following, if applicable:
(optional: add explanation between parentheses)

[ ] Added automated tests? ( )
[ ] Tested manually on mobile? ( )
[x] Checked BrowserStack? ( )
[x] Checked for performance implications? ( )
[x] Checked accessibility? ( )
[ ] Checked for security implications? ( )
[ ] Updated the documentation/wiki? ( )

TODOs / next steps:

[ ] your TODO here

Just noticed this one includes a deprecated SCSS file thanks for heroku deploys 🎉 . Working on a fix

@AndrewGibson27 if you want to give this one a quick look, I fixed that issue referenced above. That deprecated file was base-v2.scss, which I deleted during the legacy cleanup.


Survey/Question: Add a lightweight, optional validation module?

Version 1.0.0.pre.7 removed the built-in #valid? and #errors methods, directing you to validation libraries like ActiveModel::Validations instead. A question to active followers/users: Does anybody find this to be a difficult approach to work with?
ActiveModel provides a rich set of validations, and re-implementing them seems to be wasteful. Some will also enjoy keeping the library lightweight, by not always requiring ActiveModel if not needed for your use case. However, I do see a potential middle ground - does anybody need some validation functionality, but can't or won't use an outside library?
I want to hear about those use cases, if they exist. And if they do, I'm open to ideas for what you would need in an optional module that provides validation hooks.

I feel like the #valid? interface provides a nice, lightweight insertion point for custom validation. Data validation is a fairly common thing, Dynamo is a pretty flexible datastore, so why not leave that option open?
As an aside, it looks like the existing remnant of valid? is a class method. This doesn't appear to allow any access to the row data -- I'd imagine it would be used like so:
def valid?
  some_validation(@data)
end

Does this make sense? Or am I misunderstanding how it's intended to be used?


Fix warnings errors

Supercedes #644
Bumps all dependencies to latest versions, and fixes the broken stacker cfg caught by the well-known cfgs lint.

Thanks! Feel free to merge.

How do you want to handle the MSRV stuff? It's test only, I can skip the upgrade for now if you'd like.

Incidentally, I've just resolved this by removing test checks for MSRV


Issues with time in queries when creating alarms from SPY

Sentinl and Kibana version
Kibana: 6.2.4
Sentinl: 6.2.4 (tag <IP_ADDRESS>)

Issue description
Creating alarms with SPY on visualizations ends with passing the specific timestamps and not the dynamic time fields. This removes the dynamic aspect of a query.

Reproducing the issue step-by-step
Creating a line visualization.
Time field, upper right corner on kibana:

Index: mos-*
Y-axis: count
X-axis: Date Range

From: now-1w/w
To: now/w

Expected results vs. actual results
Actual.txt
Expected.txt

TLDR:
Actual: {"gte":1528869894415,"lte":1528873494415,"format":"epoch_millis"}
Expected: {"gte":"now-1h/h","lte":"now/h"}

Sample of data
Script to generate data: mos.sh
Additional details

The time is the same, it is in milliseconds since the epoch.
1528873494415 -<PHONE_NUMBER>415 = 3600000 ms (1 hour)

I am aware of that, but this makes the query generated from the visualization only valid for one specific hour.  If the timestamps are passed on to a watcher the queries will not be dynamic. The watcher relies upon someone manually making the queries dynamic.

@havlan I agree with this proposal, it'll make the wizard smarter.
Does the elasticsearch API allow for dates formats such as now-180m/m?
If so we can use minutes as base measure and use something as follows?
mock
function diff_minutes(dt2, dt1)
 {
  var diff =(dt2.getTime() - dt1.getTime()) / 1000;
  diff /= 60;
  return Math.abs(Math.round(diff));
 }

var input = {"gte":1528869894415,"lte":1528873494415,"format":"epoch_millis"}
var diff = diff_minutes( new Date(inpute.gte), new Date(inpute.lte)) + 'm';
var timerange = {"gte":"now-"+diff,"lte":"now/h"}

{
  gte: "now-60m/m",
  lte: "now/m"
}

@lmangani  I think so (manually tested). I got a suggestion using seconds coming up in a sec.

@lmangani #457

https://github.com/havlan/sentinl/tree/454_dynamic_time
It's a simple solution, it solves the "general" case of @timestamp usage. But I don't know if it breaks anything else. Are there more time fields that should be included? Or is this an ok default?

@havlan do you mean you want to add a possibility to specify a custom time field?

@sergibondarenko I was hoping to add a solution that does not assume default '@timestamp'.
if (indexPattern && indexPattern.getTimeField()) {
        if (!alarm._source.input.search.request.body.query.bool.must) {
          alarm._source.input.search.request.body.query.bool.must = [];
        }

        alarm._source.input.search.request.body.query.bool.must.push({
          range: {
            [indexPattern.timeFieldName]: {
              from: watcherRange
            }
          }
        });
      }

I think if this if would ever pass, I don't need to specify which time field to replace, some code is still required though. Would make code cleaner and easier to eventually add more timefields.
if(must[key].range['@timestamp'])

switch(indexPattern.timeFieldName)

I would still need to manually create edits for each common timestamp field, I guess?

@havlan an override option would always be great but i 100% agree with your approach here

@havlan feel free to override anything specific in your PR towards the goal, we'll review and test the overall and potential side-effects together afterwards, how's that?

@lmangani my thought is that in order to achieve this I need the indexPattern to somehow be set or passed along. It is not being set right now.


[GSSOC'21]

💥 Proposal
(A clear and concise description of what the proposal is.)
Level Order traversal of a binary tree
Have you read the [Contributing Guidelines on Pull Requests]
YES
(https://github.com/TesseractCoding/NeoAlgo/blob/master/CONTRIBUTING.md#reporting-new-issues)?
(Write your answer here.)

/assign


Accessing only unchanged values of a snapshot would prevent re-renders

Normally extracting full snapshot would broadcast all changes, however if only unchanged values are being accessed it will prevent re-rendering.
Try to uncomment line 14 to see the desired behaviour
https://codesandbox.io/s/valtio-444r4?file=/src/App.tsx:0-509
import { proxy, useSnapshot } from "valtio";

class State {
  count = 0;
  value = 0;
}

const store = proxy(new State());

export default function App() {
  const snapshot = useSnapshot(store);

  //Comment this line out to enable rerender
  const value = snapshot.value;

  return (
    <>
      {/* Try adding values */}
      <button onClick={() => (store.count += 1)}>add</button>
      {/* Will show stale value */}
      <button onClick={() => alert(snapshot.count)}>show value</button>
    </>
  );
}

Hi, thanks for trying out.
snapshot is for using only in render. Because onClick handler is outside render, it's recommended to use store instead of snapshot.
<button onClick={() => alert(store.count)}>show value</button>

We have eslint-plugin-valtio to detect this misusage.

The reason commenting out changes the behavior is a different story. If no property access happens during render, the snapshot is assumed to be used as a whole.
Hope it helps clarifying.

@dai-shi thank you so much for your quick reply and helpful response! I would definitely use the es-plugin from now on.
I was under the impression that const snapshot = useSnapshot(store) would always trigger re-render. Perhaps would be nice to add some kind of gotchas section to readme also a link to es-plugin would be useful. Let me know if you would like a pull request.

I was planning to add eslint-plugin-valtio in README, once https://github.com/pmndrs/eslint-plugin-valtio/issues/4 is resolved. (@Aslemammad are you available?)
Meanwhile, let me explain why it's a gotcha, but not wrong. (and when you get comfortable, a PR would be great.)
Suppose we have this state (or store).
const state = proxy({
  obj: {
    count: 0,
    text: 'hello',
  },
})

If using the snapshot with accessing count,
const snap = useSnapshot(state)
snap.obj.count

it will re-render only if count changes.
If the property access is obj,
const snap = useSnapshot(state)
snap.obj

then, it will re-render if obj changes. This includes count changes and text changes.
Now, we can subscribe to the portion of the state.
const snapObj = useSnapshot(state.obj)
snapObj

This is technically same as the previous one. It doesn't touch the property of snapObj, so it will re-render if obj changes.
In summary, if a snapshot object (nested or not) is not accessed with any properties, it assumes the entire object is accessed, so any change inside the object will trigger re-render.

@dai-shi brilliant explanation, thank you!
I think your comment as it is, would be a great addition to the documentation.
Btw the very first thing I did before started using Valtio in my project was testing how would re-render work with these two cases
const snap = useSnapshot(state)

const {count} = useSnapshot(state)

and in my head it sounded like: "Great! I can subscribe to all changes or only those that I need"
But now I realized that re-render optimization is even more advance, which is a great selling point!
Referencing store in handlers actually makes a lot of sense, maybe worth pointing out that using "snapshots" in handlers is an anti-pattern.
Just want to say that you did great work with Valtio! It's refreshingly simple and powerful.

One thing to note is we prefer compact README. So, either to use <details> or a dedicated wiki page would be good. I don't mean to hide gotchas, and tbh I would like to describe such technical details. But, for the first look at README shouldn't take too long.

But now I realized that re-render optimization is even more advance, which is a great selling point!

Yeah, currently it's not selling the point very well. btw, I've been working on this render optimization for years with react-tracked, and valtio is a variant in this sense.
Also, wrote some blog posts like this: https://blog.axlight.com/posts/what-is-state-usage-tracking-a-novel-approach-to-intuitive-and-performant-api-with-react-hooks-and-proxy/

https://github.com/pmndrs/valtio/wiki/Some-gotchas is created.
Maybe, we should split pages once we have many items in the page.


Can't install Version 3.11

I can successfully install V3.7 on my cloud server but when I install 3.11 I get the following error:
Verifying archive integrity... All good.
Uncompressing traccar  100%
++++++++++ YAJSW SET ENV ++++++++++
java version "1.8.0_121"
Java(TM) SE Runtime Environment (build 1.8.0_121-b13)
Java HotSpot(TM) 64-Bit Server VM (build 25.121-b13, mixed mode)
wrapper home : /opt/traccar
configuration: /opt/traccar/conf/wrapper.conf
java version "1.8.0_121"
Java(TM) SE Runtime Environment (build 1.8.0_121-b13)
Java HotSpot(TM) 64-Bit Server VM (build 25.121-b13, mixed mode)
---------- YAJSW SET ENV ----------
++++++++++ YAJSW SET ENV ++++++++++
/opt/traccar/bin/setenv.sh: line 44: java: command not found
wrapper home : /opt/traccar
configuration: /opt/traccar/conf/wrapper.conf
/opt/traccar/bin/setenv.sh: line 56: java: command not found
---------- YAJSW SET ENV ----------
/opt/traccar/bin/wrapper.sh: line 7: java: command not found

Another question I can't find the 64 bit version download for 3.8 to 3.11, anyone can help please. Thanks a lot.

Looks like your Java is not configured properly.
As for 64 bit, there is no need for it anymore. The package is universal.

Thank you Tananaev, but it can install V3.7 without problem in the same environment, what configuration do I need to change for the V3.11?

You need to configure Java to be available in the path for all users. It should be the case if you install Java from standard package manager.


possible wrong use of MPI_INTEGER?

#272 is failing the macos travis build - root cause I think is that the conda openmpi doesn't include fortran support...but it also fails with:
MPI_ERR_TYPE: invalid datatype
in some of the c tests....now according to documentation MPI_INTEGER is a fortran datatype, but is used in the C workers...can it be that this is not available if fortran support is not enabled?
(I have totally not checked that this is indeed what triggers the error...so this is speculative)
Hence it would also lead to bugs if for some reason MPI_INT and MPI_INTEGER have different size?
so do all MPI_INTEGER need to be changed to MPI_INT in the worker template code?

low priority


Upgrade xctool on Travis instead of installing HEAD version

xctool 0.2.7 was finally released, so this updates the .travis.yml to upgrade xctool to the latest release, rather than installing from HEAD.

That was fast. Thanks :balloon:

np, another one down for Hacktoberfest :smile_cat:

:sparkling_heart:


Code clean up not working

finalCleanUp does not remove if (rootData === undefined) rootData = data; for some reason

in 5.0.1-beta.3