Experiment different javascript runtimes

Problem description
So far we only run lodestar with NodejS. The goal is to experiment other javascript runtimes like Bun and Deno to see if lodestar performs better there. I had different benchmark for them:

Memory: as tested [here](url), Bun is very efficient to store a short Uint8Array of 32 bytes which is critical for our merkle tree. This could help unblock #5857

Number of bytes used to store an Uint8Array of 32 bytes

-
NodeJS 22.4.1
Bun 1.1.34
Deno 2.0.5

Uint8Array(32)
241.74
47.35
200.65

Performance - deserialize a BeaconBlock:

deserialize a BeaconBlock of 200 transactions, 500 bytes each: https://github.com/ChainSafe/ssz/blob/d7417022d15755a70b481f6f815ae08a4e685cec/packages/ssz/deserialize_beacon_block_perf_test.ts#L6
modify 100k validators of a 2M-validator merkle tree: https://github.com/ChainSafe/ssz/blob/d7417022d15755a70b481f6f815ae08a4e685cec/packages/persistent-merkle-tree/nodejs_bun_deno.ts#L5

Here's my result on a Mac M1:

-
NodeJS 22.4.1
Bun 1.1.34
Deno 2.0.5

Deserialize a BeaconBlock
0.25ms
0.11ms
0.719ms

Modify 100k nodes of 2M merkle tree
23.86ms
4.727ms
15.889ms

to run the NodeJS, I used the tsx and  it's worth to note by @nazarhussain that  Typescript overhead is almost negligible in case of Bun, ~10% in case of Deno and 100% in case of TSX/TS-Node
Solution description
Based on the test results I'd like to discuss how to go from there

Bun is potentially to be a candidate javascript runtime that help boost lodestar's performance, do we need other tests for it?
One blocker for Bun is the lack of napi support, how to deal with that? https://github.com/oven-sh/bun/issues/158

Additional context
No response

also note the result run by @nazarhussain  here:
For such comparisons we should avoid typescript complexity. I run it over the build JS files and got different results. But yet Bun is the winner.

Node: 9.1ms
Deno: 8.2
Bun: 3.71

One other observation is that Typescript overhead is almost negligible in case of Bun, ~10% in case of Deno and 100% in case of TSX/TS-Node

Looks like Bun's performance trumps Deno's by a substantial amount. Is there any advantage of Deno over Bun that we should consider?

Looks like Bun's performance trumps Deno's by a substantial amount. Is there any advantage of Deno over Bun that we should consider?

@ensi321  I don't see any, as discussed in the previous meeting we want to explore Bun because its JavaScriptCore engine is way different from NodeJS + Deno. One obstacle for Bun is it does not support Node API, we should try Bun's FFI instead and compare the performance with Node API

According to https://bun.sh/docs/api/node-api (and verified doing some basic tests), bun supports most of NAPI


All bucket support

I tried to deployed this stack, it give the bucket thumb feature for one bucket.
But I have many bucket, should I deploy this stack repeatly?
I try deploy it for the second bucket, but shows below

I have resolved this problem. Thanks a lot.

@sbstnplcn Hi, you can fork my repo MediaX.
You need to create a branch for each bucket that executes thumbnails, and modify the name of the stack and bucket in each branch Then deploy each branch.
I hope it can help you.

@timandy I came into the same issue and I came up with this solution
new ImageOptimizationStack(app, '1MediaBucketResize', {

}, "bucket-1");

new ImageOptimizationStack(app, '2MediaBucketResize', {

}, "bucket-2");

new ImageOptimizationStack(app, '3MediaBucketResize', {

}, "bucket-3");

But then I ran into an issue about the response headers policy already existing with this name.
So I had to change
        responseHeadersPolicyName: "ImageResponsePolicy",

into
        responseHeadersPolicyName: `ImageResponsePolicy${this.node.addr}`,

On line 232 of image-optimization-stack.ts
Your fork seems to have a lot of QoL improvements, I just wish the comments in the codebase and readme would be in English


What are craps-test.py and run_simulations.py in the Python directory for?

It looks like craps-test.py might just be some pre unit testing smoke tests, so I would think that this could be removed or refactored and moved to the test directory.
run_simulations.py looks like convenience methods to run simulations without having to set up the tables and players etc. I think that these would be good to have somewhere so maybe they can be refined and moved into crapssim?

craps-test.py is exactly as you say.  It can be removed with the new testing
I'm not sure what to do with the helper functions in run_simulations.py.  I'm not sure if they add value to the package or confuse it.  Ideally it would be nice to have a place for specific run methods for certain simulations, but there's so many ways to do it that it might be counterproductive to add to crapssim.  Thoughts?

For now I added a Pull Request to dev (#9) to just remove that directory.

How about Craps.Rproj? Can that be removed as well?

Craps.Rproj can be removed (ideally through the .gitignore file).  It dates back to when this directory contained some R tools for visualization, but it makes more sense for them to be separate.
I really like the idea of some simulation methods that are consistent and well documented.  Since this is a larger issue, I've added it separately in issue #14


how to output the raw output?

it's rendered by glow even if -f is dropped

you can use mods --raw


Risc v

This PR adapts latest RISCV vector spec changes

Added prefix (_riscv) for all riscv intrinsics
Update some intrinsics' parameter, like vfredxxxx, vmerge

Is the vector intrinsic API freeze?

Is the vector intrinsic API freeze?
I fixed the intrinsics API issue at least two times.

Yes, it is painful, since the standard changed, have to live with it.


No extend-expect type definitions

react-testing-library version: 1.9.3
node version: 9.2.1

What you did:
Created fresh project from CRA + typescript scripts and then wrote a simple test
test('renders App without crashing', () => {

  const { queryByTestId } = render(<App/>);

  expect(queryByTestId('app-hoc')).toBeInTheDOM();
});

What happened:
The test is passing but the project does not build. Throwing this error:

Property 'toBeInTheDOM' does not exist on type 'Matchers'.

Problem description:
Typescript won't allow building project when we are using property which does not exist on some particular type (Matchers in this case)
Suggested solution:
Extend jest expect typings with a new types definition for methods from extend-expect.js

Hello, did you try this : https://github.com/kentcdodds/react-testing-library/blob/master/README.md#custom-jest-matchers-typescript

Hi, I've just noticed that in the README file...
It works thanks. I'll close the issue.


BEIT3 models not available in TIMM and cannot create a model objecft

Describe
Model I am using BEIT3:
I am trying to finetune the model, but I am not able to create a model object as BEIT3 is not available in TIMM model library
model = create_model(
        model_config,
        pretrained=False,
        drop_path_rate=args.drop_path,
        vocab_size=args.vocab_size,
        checkpoint_activations=args.checkpoint_activations,
    )

@rahcode7 You could post a feature request at https://github.com/huggingface/pytorch-image-models/issues

Is it possible to run it currently without timm. I have downloaded pretrained weights from this repo and changed the above code to the model
model = torch.load("/mypath/beit3_base_patch16_224.zip")
But, I am now getting 'dict' object has no attribute 'state_dict' error.
Can you please help on how to run it without the timm dependecies ?


Updated Spring Data Cosmosdb dev guide to reflect v3 support

Internal Supportability workitem
Related Public Doc update PR.

@kushagraThapar please review this
Internal Supportability workitem
Related changes Public Doc update PR.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

No changes are needed. Already has links to the actual info.


Show completed date/time for tasks in the done column

Currently a tasks that's marked as complete looks like this

It should be updated to look like our About us. So, the timing shown in a completed task should be the task's completed time, not the actual start/end time in the task.

Resolved #179


Foundation press and plugins "revolution slider"

hello,
I use foundationpress. I buy the plugin "revolution slider" and when I instaled I have an error message:
Warning: An unexpected error has occurred. Something seems to not work with WordPress.org or configuration of the server. If you continue to have problems, please try the support forums. (WordPress does not establish a secure connection to WordPress.org. Please contact your server administrator.) In /home/www/wordpress/public_html/wp-includes/update.php on line 295
I wished to know if revolution slider was consistent with "foundationPress."
thank you for your answer

Hello. First of all. Please do not use header-styling when describing your problem. It makes it harder to read. Secondly, I do not offer support for third party plugins. This is a starter-theme for web developers who want a good starting point for new sites. The internet is packed with sites where you can download polished themes that are packed with plugins and whanot.
I don't mean to be harsh, but if a gallery plugin is the biggest problem, it might not be a good idea going ahead with node, grunt, sass etc.

http://foundation.zurb.com/docs/components/orbit.html
https://wordpress.org/plugins/wp-orbit-slider/

I've also found that using the capabilities of Foundation's Orbit in conjunction with something like the Advanced Custom Fields plugin allows for very customizable sliders, which much less chance for conflicts from a dedicated slider plugin.

Before diving into Orbit, be aware that it has been deprecated and the development and support stopped as of May last year.

Humm really... Its hard to keep a track of eveything would recommend this one I have use a few times http://owlgraphic.com/owlcarousel/

thank you for your answer ...
I will use the plugin "Orbit".
@ +

I think this slider is more than capable - it works well. https://wordpress.org/plugins/fluid-responsive-slideshow/


Bug: Review no-member error for ProtoBuf

Running Pylint on master generates the following error:
memphis/memphis.py:450:23: E1101: Module 'google.protobuf.descriptor_pb2' has no 'FileDescriptorSet' member (no-member)

We should fix this or disable it on a per-line basis using:
# pylint: disable-next=no-member
print("A no-member error will be suppressed for this line

Thanks @Adarsh-jaiss !  I'm not sure that disabling the linting error is the right move here without first checking that the code works.  In the previous instance, it made sense to disable the error because we explicitly check that the member exists and have an alternative code path if not.  Here, we absolutely depend on the file attribute existing.  This one will require more work on your end to debug.  In particular, it seems like you would need to:

Set up Memphis locally using Docker
Create a station with a protocol buffer-based schemaverse attachment
Modify the memphis.py code to add some print statements or something so you can verify that this code block is run, the file field exists, etc.
Install the SDK into a local virtual environment
Write a simple producer that produces messages in protocol buffer format.

I think this is going to be a fair amount of work.  If you don't want to take that on quite yet, I think there is other work that needs to be done that would be much easier.  For example, it would be nice to add examples to the function documentation like those for Consumer.fetch() so that we can eventually generate useful API docs.  What do you think?
Either way, we value your contributions and thank you!

In which issue, is that function reffered to?
I would love to work on that issue


Remove gRPC + protobufs from API

Summary
The API's gRPC support is not often used and not worth the maintenance according to various maintainers. Some currently used gRPC libraries are also deprecated need upgrading, like grpc-gateway v1: https://github.com/argoproj/argo-workflows/issues/12565#issuecomment-2026688627.
The HTTP support is also based off the gRPC support (via grpc-gateway) and this can cause some inconsistencies as well, such as #12721
The protobufs only exist for the purpose of gRPC and their OpenAPI definitions can sometimes differ from the k8s CRD due to tooling differences as well, such as with #13530 and #12631. Some protobuf libraries are also deprecated and need updating, like #11053
The build complexity to support protobufs and gRPC is also non-trivial. A decent number of tools are installed in the Makefile solely for protobuf + gRPC codegen.
Details
Per https://github.com/argoproj/argo-workflows/issues/11053#issuecomment-2185664503, https://github.com/argoproj/argo-workflows/issues/13530#issuecomment-2322026951, https://github.com/argoproj/argo-workflows/issues/12565#issuecomment-2026688627, gRPC is only used in the API and is a source of many maintenance headaches as well as build complexity. Given that many of the libraries are deprecated and in need of non-trivial breaking upgrades, it may be wise to just remove gRPC support entirely.
gRPC also predates Alex C, who also thought it and protobufs were too much of a maintenance hassle.

Message from the maintainers:
Love this feature request? Give it a 👍. We prioritise the proposals with the most 👍.

In this proposal how would we continue to support HTTP?

I've seen folks suggest plain mux before, which might work. Alex C was deferential toward using Gin.
I don't have a particularly strong opinion here and I'm not a Go expert to have a concrete preference either.


Remove unused $elms variable in selection-buttons.js

Previously: https://github.com/alphagov/govuk_frontend_toolkit/pull/295
Turns out the this.$elms write isn't necessary since it wasn't being done before anyway. It was being written to $elms but not being read from at any later point.
cc @gemmaleigh

LGTM 👍


Deploy to AWS via Terraform

WIP: Testing out Terraform for automated cloud deploy.

Nice work @ABeltramo !
Maybe is better to have a dedicated .env for AWS (or maybe AWS-nvidia, AWS-AMD, etc). This way these changes might be merged without affecting the defaults.

Replaced with https://github.com/PierreBeucher/cloudypad


How to use the colour system

There is a bunch here about why we have foreground colours and background colours for that we know will pass accessibility.
https://medium.com/eightshapes-llc/light-dark-9f8ea42c9081
Story
"We stress tested our color system with Dropboxes new crazy brand"
Some ideas from Treb
Accessibility out of the box was a big priority so we've created a system where it's easier to understand which colour is design to be paired with another colour.

Brain dump of things I would like explained:

What variables we recommend changing $AU-color-background, $AU-color-foreground-action, $AU-color-foreground-focus, $AU-color-foreground-text
How do I know my custom colours meet WCAG AA ( what the a11ycolor function does )
How they work together

Background and text color is mixed together to make border and muted ( why we do this )
Backgrounds generate shades from the original background color

Foreground on background, why this relationship exists ( An explanation of the grid on the test page )
Why the focus colour is purple by default
When colours don’t do all the work we use other methods to convey meaning ( page-alerts )

Currently work in progress:
https://gist.github.com/Xochi/066cc160c2d540198b76fb7881eae238

Currently work in progress:
https://gist.github.com/Xochi/066cc160c2d540198b76fb7881eae238

Thanks @Xochi here is some feedback, really good work overall mate!

I really like the intro and it reads really well, we might want to turn this into two articles, this currently reads like version one.

A blog post explaining the issues with colour and how the design system hopes to solve them
A refined version for developers who need to accomplish a task e.g. change the colours

I really like the way the first two paragraphs work together, however the first paragraph talks about accessibility but there isn't really a relationship to colour until the second paragraph, maybe we can flip it and talk about colour and how it relates to accessibility?

I think we can probably replace "nascent" with a more plain english word.

When I read the text below I really want to know what the solutions are, can we tease them here by adding some small explanation.

We came up with some interesting solutions for the design system that can help designers bypass the often tedious work to get the right palette.

There is a bit of repetition in the second paragraph:

Title: colour palette in seconds  Para:colour palette set up in seconds
overlooked aspect of accessibility is colour about getting accessibility right accessibility first, with the intro I felt like I knew it was accessible

I would like a little bit of an explanation somewhere about how to change the variables and apply them throughout the system

I'm not convinced about splitting the light and dark theme up is the correct approach. Looking at it logically it makes sense here however I think there might be a better solution 🤔

The links for the components are broken?

Great work @Xochi excited to see this get published!


Script loading by ajax after first initialization works but I don't want it

Hie. I have a problem.
After loading scripts here
utilsLoaded: function() {
    // if autoFormat is enabled and there's an initial value in the input, then format it
    if (this.options.autoFormat && this.telInput.val()) {
        this._updateVal(this.telInput.val());
    }
    this._updatePlaceholder();
}

one of a set of new functions cut phone number. It is normal behavior but I want to change it. Why do you load scripts by ajax and don't include all required functions? May be I want to change behavior..
May be it shuold be like this
utilsLoaded: function() {
    // if autoFormat is enabled and there's an initial value in the input, then format it
    if (this.options.autoFormat && this.telInput.val() && !this.options.smartFormat) {
        this._updateVal(this.telInput.val());
    }
    this._updatePlaceholder();
}

where a smartFormat flag can cansel updating value by default

Closed due to inactivity.


EI-39 - analytics pageview data

This PR has been duplicated into the web project: das-employer-incentives-web


Encourage creation of log drains during FTUX

We could also look at an app's failed operations and make a suggestion there.

This could probably be solved by whatever solution we come up with for #394.


[Bug] Temperature is always 0.0

The temperature for answer generations is always 0.0, since the category of all questions is arena-hard-v0.1. This implicitly means that the temperature will be defaulted to 0.0. Wanted to flag since I think this might be undesirable behavior.

Hi, this is not a bug but an intended parameter for Arena Hard.
Since most of the prompts are difficult problem solving oriented questions (many of them being very technical), we use temperature 0 so models can produce more accurate answer. Further, we want to limit the variance of the win rate as well. Thanks.


PHP Fatal error: Allowed memory size of exhausted - close connection?

Im running ratchet 0.3 and over time, this error always pops up.  All my users are on mobile devices and Im wondering if the connections are not closing properly.  I saw other users with this issue and it sounds like maybe going to 0.4 could fix that - any ideas? Do I need to do anything so these users are really gone (ping/pong -> if so how - I couldnt find any info about this)?
Thanks,
JB

Updating to 0.4 is certainly a good idea with the number of improvements to it.
Running out of memory could be a number of things. If you have a lot of connections and your memory limit for PHP is really small you may need to increase the limit.
Another problem could be a memory leak. React and Ratchet have been thoroughly tested so if it's a memory leak the most likely culprit would be your application built on top of it. If you're keeping lookups in your app class during the lifespan of a connection make sure you're cleaning up resources in your onClose method so garbage collection can take affect.

I have the same problem. If there are only push clients, the memory usage of server is stable. But when I run one client, every message increase memory usage about 4576B. To simulate:

run server: php server.php
run push client: php client-push.php
run client (it can be also web browser connected to ws://localhost:8080): php client.php

In the server output you will see something like:
1 / 1917120 / 4194304 / 4576
 /  /  /
After while the server will finished with message: Fatal error: Allowed memory size of 4194304 bytes exhausted (tried to allocate 4096 bytes) in ...vendor\react\stream\src\WritableResourceStream.php on line 55
I am running it on Windows 7 Enterprise, PHP Version 7.2.5.
client.txt
client-push.txt
server.txt

There has been a new release of RFC6455 (https://github.com/ratchetphp/RFC6455/releases/tag/0.2.4) that includes @kusnir's fix. (Thanks @kusnir!)
@theborland - if there are further problems please open a new issue.


enable linter

this linter will fail the build if rule sets are not met

Hey @ljubon . I see this in CI.yml https://github.com/ljubon/ParquetSharp.DataFrame/blob/main/.github/workflows/ci.yml#L27C1-L28C1. How do your changes work with this ci job?


Get the default device IP address

rq_worker now returns the default gateway
removed netifaces because it was crazy dependent on other packages
ip of default device found using good old subprocess library

Codecov Report

Merging #783 into master will decrease coverage by 0.07%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #783      +/-   ##
==========================================
- Coverage   77.53%   77.45%   -0.08%
==========================================
  Files          51       51
  Lines        4740     4741       +1
==========================================
- Hits         3675     3672       -3
- Misses       1065     1069       +4

Impacted Files
Coverage Δ

vent/helpers/meta.py
72.16% <ø> (ø)
:arrow_up:

vent/api/plugin_helpers.py
72.38% <0%> (-0.94%)
:arrow_down:

vent/core/rq_worker/watch.py
43.93% <0%> (-0.05%)
: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 6d7693f...ec122dc. Read the comment docs.


feat(linter): remove deprecated eslint v9 rules no-return-await and no-mixed-operators

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#3188  👈
#3187
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Boshen and the rest of your teammates on  Graphite

Merge activity

May 7, 5:11 AM EDT: Graphite rebased this pull request after merging its parent, because this pull request is set to merge when ready.


Add JVM Support

I only tested this on a Windows PC connecting with an Android device. Video/Screensharing and Audio works with the sample apps. DataChannel was only tested via Junit.
Since this relies on webrtc-java for native interop, it should theoretically work on any platform it supports as well.

@aschulz90 great job. Let me take some time to review.

I've tried to run the JVM sample app on Mac. Unfortunately, there is exception getting user media:
java.lang.Error: Unhandled Exception
        at dev.onvoid.webrtc.media.video.VideoDeviceSource.start(Native Method)
        at com.shepeliev.webrtckmp.LocalVideoStreamTrack.<init>(LocalVideoStreamTrack.kt:13)
        at com.shepeliev.webrtckmp.MediaDevicesImpl.getUserMedia(MediaDevices.kt:89)
        at com.shepeliev.webrtckmp.MediaDevices$Companion.getUserMedia(MediaDevices.kt)
        at com.shepeliev.webrtckmp.sample.shared.RoomComponent$ViewModel$openUserMedia$2.invokeSuspend(RoomComponent.kt:86)
        at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33)
        at kotlinx.coroutines.DispatchedTask.run(DispatchedTask.kt:108)
        at kotlinx.coroutines.scheduling.CoroutineScheduler.runSafely(CoroutineScheduler.kt:584)
        at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.executeTask(CoroutineScheduler.kt:793)
        at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.runWorker(CoroutineScheduler.kt:697)
        at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.run(CoroutineScheduler.kt:684)

Also, JVM tests fail on Mac with java.lang.UnsatisfiedLinkError yet. I'll try to figure out what is the reason as soon as be able.
It's really great to add JVM platform, however I think we should make it work properly on any OS (windows, osx, linux).

@shepeliev @aschulz90  Any updates on this?

@tamimattafi
I don't have any Mac (neither Intel nor M1) or Linux device with a webcam and microphone to test on. Maybe you could help by checking the tests and demo on them, if you have any of those device types.

@aschulz90 Hello! I have both Mac on Intel and M1, I can help on Saturday if it's suitable for you

@aschulz90 We tried on both M1 and Intel macs, here's a report:
1. Sample run from main():

Crash when the PC on Intel chip is missing a camera or microphone:

java.util.NoSuchElementException: List is empty.
        at kotlin.collections.CollectionsKt___CollectionsKt.first(_Collections.kt:214)
        at com.shepeliev.webrtckmp.sample.shared.RoomComponent$ViewModel.createPeerConnection(RoomComponent.kt:180)
        at com.shepeliev.webrtckmp.sample.shared.RoomComponent$ViewModel.createRoom(RoomComponent.kt:122)

Selecting Speakers on M1 MacBook throws an exception:

java.lang.Error: Set playout device failed
	at dev.onvoid.webrtc.media.audio.AudioDeviceModule.setPlayoutDevice(Native Method)
	at com.shepeliev.webrtckmp.WebRtc.setAudioOutputDevice(WebRtc.kt:78)
	at com.shepeliev.webrtckmp.SelectMicrophoneCameraScreenKt$SelectMicrophoneCameraScreen$4$1$3$1$1.invoke(SelectMicrophoneCameraScreen.kt:166)
	at com.shepeliev.webrtckmp.SelectMicrophoneCameraScreenKt$SelectMicrophoneCameraScreen$4$1$3$1$1.invoke(SelectMicrophoneCameraScreen.kt:163)
	at com.shepeliev.webrtckmp.SelectMicrophoneCameraScreenKt$DeviceSelector$1$4$2$1.invoke(SelectMicrophoneCameraScreen.kt:235)
	at com.shepeliev.webrtckmp.SelectMicrophoneCameraScreenKt$DeviceSelector$1$4$2$1.invoke(SelectMicrophoneCameraScreen.kt:234)

The confirm button on M1 MacBook doesn't react to clicks, even tho all devices were selected, seems like it's missing a permission
Share desktop asks for permission, denying the permission, keeps showing the screen sharing, and by clicking on create (room) it throws this exception:

java.util.NoSuchElementException: List is empty.
	at kotlin.collections.CollectionsKt___CollectionsKt.first(_Collections.kt:214)
	at com.shepeliev.webrtckmp.sample.shared.RoomComponent$ViewModel.createPeerConnection(RoomComponent.kt:179)
	at com.shepeliev.webrtckmp.sample.shared.RoomComponent$ViewModel.createRoom(RoomComponent.kt:122)
	at com.shepeliev.webrtckmp.sample.shared.RoomComponent.createRoom(RoomComponent.kt)
	at com.shepeliev.webrtckmp.VideoScreenKt$VideoScreen$2$1$2$2$1$1.invoke(VideoScreen.kt:122)
	at com.shepeliev.webrtckmp.VideoScreenKt$VideoScreen$2$1$2$2$1$1.invoke(VideoScreen.kt:122)
	at androidx.compose.foundation.ClickablePointerInputNode$pointerInput$3.invoke-k-4lQ0M(Clickable.kt:898)

Because of these issues, I couldn't really test the actual connection. Not sure how to give this permission manually since I don't see the app there, probably I need to build a .dmg and install the app, or something is missing in the manifest / plist for macOS.

Tests in commonTest run on jvm:
IceServerTest fails both on M1 and Intel:

java.lang.UnsatisfiedLinkError: 'void dev.onvoid.webrtc.media.audio.AudioDeviceModule.initialize(dev.onvoid.webrtc.media.audio.AudioLayer)'
	at dev.onvoid.webrtc.media.audio.AudioDeviceModule.initialize(Native Method)
	at dev.onvoid.webrtc.media.audio.AudioDeviceModule.<init>(AudioDeviceModule.java:36)
	at com.shepeliev.webrtckmp.WebRtc.initializePeerConnectionFactory(WebRtc.kt:42)
	at com.shepeliev.webrtckmp.WebRtc.initialize(WebRtc.kt:37)
	at com.shepeliev.webrtckmp.WebRtc.getPeerConnectionFactory$webrtc_kmp(WebRtc.kt:16)
	at com.shepeliev.webrtckmp.PeerConnection$native$2.invoke(PeerConnection.kt:40)
	at com.shepeliev.webrtckmp.PeerConnection$native$2.invoke(PeerConnection.kt:39)
	....

PeerConnectionTest fails both on M1 and Intel:

java.lang.UnsatisfiedLinkError: 'void dev.onvoid.webrtc.media.audio.AudioDeviceModule.initialize(dev.onvoid.webrtc.media.audio.AudioLayer)'
	at dev.onvoid.webrtc.media.audio.AudioDeviceModule.initialize(Native Method)
	at dev.onvoid.webrtc.media.audio.AudioDeviceModule.<init>(AudioDeviceModule.java:36)
	at com.shepeliev.webrtckmp.WebRtc.initializePeerConnectionFactory(WebRtc.kt:42)
	at com.shepeliev.webrtckmp.WebRtc.initialize(WebRtc.kt:37)
	at com.shepeliev.webrtckmp.WebRtc.getPeerConnectionFactory$webrtc_kmp(WebRtc.kt:16)
	at com.shepeliev.webrtckmp.PeerConnection$native$2.invoke(PeerConnection.kt:40)
	at com.shepeliev.webrtckmp.PeerConnection$native$2.invoke(PeerConnection.kt:39)
	...

@shepeliev @aschulz90 I would love to help with any further steps to accelerate the integration on JVM

@tamimattafi thanks for the tests. Unfortunately, I don't have any quick answer for these issues on mac. It definitely requires more researching. I'll back to this as soon as be able.

Are you using OpenJDK by any chance?  And perhaps running this from the IDE and not a built package?
Check this: https://bugs.openjdk.org/browse/JDK-8272639
They've used  the --resource-dir option when building the package and pointed it to a custom info.plist containing the proper key/val
<key>NSMicrophoneUsageDescription</key>
<string>The application is requesting access to the microphone.</string>

@tricknology Yes indeed, I'm using OpenJDK 20 on my machine, and JetBrains Runtime version 17 on the IDE. And indeed I run the sample from main() method from the IDE as stated in the test report.
My guess is, as you stated that it is better to build a package and install it properly for proper testing with permissions.

@tamimattafi I feel like there should be a way to pass options to the compiler in run config.
You might also specify a makefile to do the same.. unfortunately I'm not exactly sure on the step-by-step.

Selecting Speakers on M1 MacBook throws an exception:

This should be fixed now.

@its-Chiedu There is a way to pass every detail including info.plist
https://github.com/JetBrains/compose-multiplatform/blob/master/tutorials/Native_distributions_and_local_execution/README.md#customizing-infoplist-on-macos
However, I'm having some issues building a .dmg using java 17 and java 20, I will look through the issue when I find some free time.

@shepeliev I have merged the latest changes from main and updated the sample app for jvm support. Can you please review that part again?
Also, there are currently two JS tests failing:
com.shepeliev.webrtckmp.RtcCertificatePemTests.generateEcdsaPem[js, browser, ChromeHeadless123.0.6312.124, MacOS10.15.7] FAILED
    AssertionError at /Users/runner/work/webrtc-kmp/webrtc-kmp/build/compileSync/js/test/testDevelopmentExecutable/kotlin/jsMainSources/kotlin/kotlin/test/JsImpl.kt:23

com.shepeliev.webrtckmp.RtcCertificatePemTests.generateRsaPem[js, browaser, ChromeHeadless123.0.6312.124, MacOS10.15.7] FAILED
    Error at /Users/runner/work/webrtc-kmp/webrtc-kmp/webrtc-kmp/src/jsMain/kotlin/com/shepeliev/webrtckmp/RtcCertificatePem.kt:28

But I don't know how these tests failing could relate to this PR.

Unfortunately, still have an exception on both of Intel and M2 macs:
java.lang.Error: Unhandled Exception
        at dev.onvoid.webrtc.media.video.VideoDeviceSource.start(Native Method)
        at com.shepeliev.webrtckmp.LocalVideoStreamTrack.<init>(LocalVideoStreamTrack.kt:13)
        at com.shepeliev.webrtckmp.MediaDevicesImpl.getUserMedia(MediaDevices.kt:89)
        at com.shepeliev.webrtckmp.MediaDevices$Companion.getUserMedia(MediaDevices.kt)
        at com.shepeliev.webrtckmp.sample.shared.RoomComponent$ViewModel$openUserMedia$2.invokeSuspend(RoomComponent.kt:86)
        at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33)
        at kotlinx.coroutines.DispatchedTask.run(DispatchedTask.kt:108)
        at kotlinx.coroutines.scheduling.CoroutineScheduler.runSafely(CoroutineScheduler.kt:584)
        at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.executeTask(CoroutineScheduler.kt:793)
        at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.runWorker(CoroutineScheduler.kt:697)
        at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.run(CoroutineScheduler.kt:684)

I guess some problem is in native WebRTC SDK that embeded with WebRTC java lib.
There are two options for now:

make basic sample Java app which reproduces the problem and raise an issue at https://github.com/devopvoid/webrtc-java
build webrtc-java from the sources adding some logs into low-level C code and try to figure out by myself

I think we could start with the first option for now. Java background devs help is appreciated :)

Any updates? 🤔

Probably still blocked by requiring a way to request camera/microphone permissions on macOS JVM.

@aschulz90, good work, do you have a working sample for JVM?

The sample app has JVM desktop support and can be run like described in the Readme.

oh my bad, this may be of discussion but, it runs locally if you actually try in a real scenario, say with a turn server, and you try sending or/and receiving an offer it won't work

Signaling is out of scope of this libary and it shouldn't matter what medium you choose for it: memory, like the sample; HTTP/WebSocket; or avian carrier.
At the time of the PR's creation the sample used Firestore for signaling and it worked fine then, as seen in the screenshots.
TURN works fine in an app I created, but I can't show the source.

I’m not very familiar with this topic, which is why I asked the question. I’m currently learning and noticed that I couldn’t find Firebase.firestore anywhere in the project. Could you please show me how it’s being used?
I have a small sample using Turn signaling, but I can’t figure out why it’s not working. Could you please take a look and help me understand the issue?


Simulated Flow from Carla

Thanks for opensource this work. After reading your arXiv paper, I found that the Flow360 dataset you proposed was obtained using Carla simulator. I have a question about using Carla to simulate optical flow.
I was confused by the visualization of simulated optical flow in some cases. As mentioned in the following two issues, there are obvious stratification or quantification problems. Have you encountered this problem when simluating the proposed dataset?
https://github.com/carla-simulator/carla/issues/4795
https://github.com/carla-simulator/carla/issues/5514

Hi! Thanks for attention.
Regarding the optical flow of CARLA, we also found that the ground truth was unreliable at the beginning of the simulation, so we rewrote the function to obtain the optical flow (via imu sensor and 3D coordinates), and verified the collected optical flow based on the forward-backward check and the Mean wArped pixel Error (MAE). Specifically, MAE refers to the pixel-by-pixel difference between two frames of images after warping the image using optical flow. We tested based on the demo collected images of the dataset, and the results are shown in the figure below. It can be seen that the MAE in all views is significantly reduced after loading the optical flow distortion, except for the down camera, which may be due to the fact that the ground is almost unchanged in the two frames before and after the motion, but an error is introduced due to the warped extra black area:

For reference, the MAE of the training split of the Sintel dataset is about 3.8 pixels, and we have not performed statistics on the entire Flow360 for now (it may be necessary to remove sky pixels for statistics).
Unfortunately, this part of the optical flow generation function is the property of my internship company, and I have no right to decide to open source this part of the code. If you need more information, such as business or academic collaborations, please contact me further<EMAIL_ADDRESS>

Hi! Thanks for attention.
Regarding the optical flow of CARLA, we also found that the ground truth was unreliable at the beginning of the simulation, so we rewrote the function to obtain the optical flow (via imu sensor and 3D coordinates), and verified the collected optical flow based on the forward-backward check and the Mean wArped pixel Error (MAE). Specifically, MAE refers to the pixel-by-pixel difference between two frames of images after warping the image using optical flow. We tested based on the demo collected images of the dataset, and the results are shown in the figure below. It can be seen that the MAE in all views is significantly reduced after loading the optical flow distortion, except for the down camera, which may be due to the fact that the ground is almost unchanged in the two frames before and after the motion, but an error is introduced due to the warped extra black area:

For reference, the MAE of the training split of the Sintel dataset is about 3.8 pixels, and we have not performed statistics on the entire Flow360 for now (it may be necessary to remove sky pixels for statistics).
Unfortunately, this part of the optical flow generation function is the property of my internship company, and I have no right to decide to open source this part of the code. If you need more information, such as business or academic collaborations, please contact me further<EMAIL_ADDRESS>

Hi! Thanks for attention.
Regarding the optical flow of CARLA, we also found that the ground truth was unreliable at the beginning of the simulation, so we rewrote the function to obtain the optical flow (via imu sensor and 3D coordinates), and verified the collected optical flow based on the forward-backward check and the Mean wArped pixel Error (MAE). Specifically, MAE refers to the pixel-by-pixel difference between two frames of images after warping the image using optical flow. We tested based on the demo collected images of the dataset, and the results are shown in the figure below. It can be seen that the MAE in all views is significantly reduced after loading the optical flow distortion, except for the down camera, which may be due to the fact that the ground is almost unchanged in the two frames before and after the motion, but an error is introduced due to the warped extra black area:
For reference, the MAE of the training split of the Sintel dataset is about 3.8 pixels, and we have not performed statistics on the entire Flow360 for now (it may be necessary to remove sky pixels for statistics).
Unfortunately, this part of the optical flow generation function is the property of my internship company, and I have no right to decide to open source this part of the code. If you need more information, such as business or academic collaborations, please contact me further<EMAIL_ADDRESS>
We supplemented the warped point error statistics on the entire Flow360 test split (including the sky), and it can be seen that the warped error of the gt flow is about 3.07 pixel, which is comparable to the level of the Sintel dataset.

Thank you for providing a verification and selection option in your reply. This result shows that the simulated flow in Flow360 is reliable.
I found that the unreliable stratification or quantification problems not only occurs at the beginning of the simulation, but also later. I haven't found its regularity yet.
Because your dataset is synthesized into the 360 view, I haven't downloaded it to visualize whether this problem also exists.

Hi, we suggest you can refer to this link. Also, we are considering open source the original 6 pinhole camera dataset with optical flow and semantics together, hope this may help you.

Thanks for your answer. I have found out the (possible) orginal implementation of Carla flow, but I still can't explain the reason for this abnormal simulation output. I listed the resources I found here.
https://github.com/carla-simulator/carla/blob/master/Unreal/CarlaUE4/Plugins/Carla/Content/PostProcessingMaterials/VelocityMaterial.uasset
https://github.com/carla-simulator/carla/blob/master/Unreal/CarlaUE4/Plugins/Carla/Source/Carla/Sensor/OpticalFlowCamera.cpp
https://github.com/BrainGardenAI/OptiFlow
https://github.com/ProfFan/UnrealOpticalFlowDemo

Hello @MasterHow
When you say "we rewrote the function to obtain the optical flow (via imu sensor and 3D coordinates)", do you mean with Unreal engine or with python ?
Best regards.


multimap-tests.eo:180-182: This test was temporary...

The puzzle 15-96820566 from #15 has to be resolved:
https://github.com/objectionary/eo-collections/blob/4dbdfd5a31f26cdd6fefef22053e8b7f22cf6cb3/src/test/eo/org/eolang/collections/multimap-tests.eo#L180-L182
The puzzle was created by @rultor on 08-Nov-22.
Estimate: 30 minutes,  role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@EugeneDar Can we solve it by increasing the stack size?

@Graur if I remember correctly, we already did it here.

Correct me, if I was wrong

@EugeneDar Can you check it, please?


bump coco to 0.3

If stjepang/coco#20 gets merged and published, we won't have to worry about bumping rayon-core's minimum rustc for this.

@cuviper Done! Merged and published.

@stjepang https://crates.io/crates/coco still doesn't list 0.3.2 ;)

Oops, sorry! Now it does. :)

Here are some cargo-chrono plots before and after:

Mostly wins, but two big regressions:

fibonacci_split_recursive: this is a kind of weird one, that I'm willing to write off.  FWIW, the related fibonacci_join benchmarks (not shown here) are about twice as fast now!
parallel_find_last: the really weird part about this is that it's now performing even worse than parallel_find_missing, which itself stayed about the same.  Parallel finding the last element should be able to short-circuit and return sooner than with a missing element that has to check everything.  I'm really baffled by this one!

On a whim, I just tried compiling rayon-demo with all test modules except find commented out, and now parallel_find_last does perform better than parallel_find_missing as expected, and about the same as with the old coco.  But parallel_find_middle got worse.
My best guess is that these are just sensitive to changes in overall code generation and/or layout. Meh.
bors r+


More integration tests on ISO Date time format

Raised following PR:
https://github.com/9tigerio/db2rest/pull/825

@souravroy you don't have to manually type the link... instead use the GitHub short keywords syntax in a PR description or commit message (the title or comment) using one of the following formats:
fixes #823
closes #823
etc.
See details here: https://docs.github.com/en/issues/tracking-your-work-with-issues/using-issues/linking-a-pull-request-to-an-issue
You can go to your PR and edit the title or the first comment and it will autolink the PR to this Issue.

@souravroy you don't have to manually type the link... instead use the GitHub short keywords syntax in a PR description or commit message (the title or comment) using one of the following formats: fixes #823 closes #823 etc.
See details here: https://docs.github.com/en/issues/tracking-your-work-with-issues/using-issues/linking-a-pull-request-to-an-issue You can go to your PR and edit the title or the first comment and it will autolink the PR to this Issue.

Thank you @thadguidry . This's really useful, I'll follow the instructions in my next PR

closed #825


Copy site tree QueeMessages error

When we try to copy a language site tree to another language. We get the following error:
Call to a member function getLocale() on null
This problem occurs because we have a few file rescan records in the QueueMessages table with a timeout value.

The following code can be found in sitemap_copy_all.php
$q = Queue::get('copy_page');
$q->receive(Config::get('concrete.limits.copy_pages'));

In concrete/src/Foundation/Queue/DatabaseQueueAdapter.php the queId is bind to the query
$statement = $this->db->prepare("
                    select *
                    from QueueMessages
                    where queue_id = ? and handle is null or timeout + {$timeout} < {$microtimeInt}
                    limit {$maxMessages}
                    for update
                ");
$statement->bindValue(1, $queueId);

The problem seems to be related to the or timeout + {$timeout} < {$microtimeInt} limit {$maxMessages} for update part in the query.
Because of this the rescan files records are also include while they have queue_id of 1. The result is the error since the cID can't be found.

This should be fixed in the latest release candidate – there is a parentheses around the relevant query expression.


Feat: (low prio) Logs are double reporting timestamps

Here's some output of kubectl:
uc807dda342eb5f:~/run/eksa$ k logs --since=30m -n eksa-packages eks-anywhere-packages-5b47488b49-k5d9z
1.6608493109306107e+09	INFO	PackageBundle.PackageBundle	Add/ProcessBundle:	{"bundle": "eksa-packages/v1-23-60"}
1.6608493531004937e+09	INFO	PackageBundle.PackageBundle	Add/ProcessBundle:	{"bundle": "eksa-packages/v1-23-60"}
uc807dda342eb5f:~/run/eksa$ k logs --timestamps=true --since=30m -n eksa-packages eks-anywhere-packages-5b47488b49-k5d9z
2022-08-18T19:01:50.930723793Z 1.6608493109306107e+09	INFO	PackageBundle.PackageBundle	Add/ProcessBundle:	{"bundle": "eksa-packages/v1-23-60"}
2022-08-18T19:02:33.100605706Z 1.6608493531004937e+09	INFO	PackageBundle.PackageBundle	Add/ProcessBundle:	{"bundle": "eksa-packages/v1-23-60"}

There are, for whatever reason, two timestamps in our logging output, There's really no need for this, it's just wasted space. Plus 1.6608493109306107e+09 is not a useful way to display a timestamp, but fwiw:
$ date -d @$(printf %f "1.6608493531004937e+09")

There's probably an easier way, but that was what jumped to mind.

Also note that kubectl logs provides a --timestamp switch so even printing the timestamp once might not be relevant. We should consider using structured logs.


Fix batching in fairseq SentencepieceTokenizer

This fixes #639 by moving the fairseq id patching out of the loop so it's not applied multiple times to sequences earlier in the batch.
I've added the check to ensure it's not applied when add_rev is true mirroring its position in the for loop, but I'm unsure if that's required (presumably there are no HF tokenizers which emit a reversed string and have this fairseq hack?).
I tested it against the example in #639 and both batches and single sequences work correctly now.

/azp run onnxruntime-extensions.CI

/azp run onnxruntime-extensions.CI

I ran the Python tests locally on macOS ARM64 and everything passed. The access violation in the Windows tests happens either in test_cliptok or test_cv2 both of which passed fine on my machine and don't use the code I changed, and they also passed fine in Linux and macOS in the CI. Is there anything more I can do to run the failure down?

@sayanshaw24 , can you review this PR?


nth-prime: Replace [] tests with individual tests (#298)

See #298.

Great work again. Thanks a ton! 🎉


Windows Terminal

How to use the new windows terminal as administrator?

You can get there by right clicking on the taskbar entry:

By using the start menu:

Or by using the run dialog to launch wt.exe as you would launch any other process elevated.


Fix Google API rate limit exception

This PR fixes #1090 by

introducing timeout in Google API HTTP calls. Sometimes Google API would hang for no good reason and adding timeout can reduce hanging time. Currently, the timeout is set to be 10 seconds for each HTTP request against Google API.

adding retry logic when exception happens in Google API calls, for example, exceeding the rate limit. Will do blocking backoff sequential attempts, until success or reached attempts count.

increasing the batch size from 100 to 10000 rows by each content append to Google Sheet. With bigger batches, it can reduce the number of API requests needed for uploading a file, and increases the response time of the API call. Need to have a balance between the batch size and the timeout value.

 Created from JetBrains using CodeStream

Also did we limit the amount of data that gets uploaded to google sheets any where? I think it's better to limit the number of rows somewhere. I also think google sheets have a limit. So with the completion of file sink operator, we can add a limit of for example 10,000 rows?

For the current test runs, I find it can handle a much bigger data size. I think this functionality can handle millions of data without issue. Of course, we do not want users to abuse it. We have a limit on Google Drive volume that we are hosting those files. I suggest we can keep it with a large limit or no limit for now as it mainly used internally only anyway. later we have two possible solutions:

when the file sike operator is ready, we can tighten the limit on the google sheet export.
when the Oauth system is integrated, we can export to the user's Google Drive, which would push the concern to the user.


Fix pkg-config on Windows

If building on Windows as shared library GLFW adds the suffix "dll" to the library name. This suffix is missing in the pkg-config file.

Many thanks for this PR. I'll update the changelog and credits in a bit.

Note: I added you to the credits as moritz-h as I could not find your full name. Let me know if you want that changed.

Note: I added you to the credits as moritz-h as I could not find your full name. Let me know if you want that changed.

@dougbinks Thanks. My full name is Moritz Heinemann. Seems most people in that file are listed by real name, so maybe fits better :smiley:

Thanks, I've updated the contributers with your full name.


Source Request : kakaowebtoon TW

Source name
kakao webtoon TW
Source link
https://tw.kakaowebtoon.com/
Source language
chinese
Other details
18+/NSFW=yes
Even if it's just WebView.
Acknowledgements

[X] I have checked that the extension does not already exist by searching the extension listing and verified it does not appear there.
[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a meaningful title with the source name.
[X] I will fill out all of the requested information in this form.

Up

This site is shutting down with over a year left, can you add this expansion, would love to download all the pieces I bought.


Looking for the implementation of "Getting a GlUE"

Hi @JavierAntoran ,
my name is Hao, I'm searching for a valid method for the Uncertainty Explanations, like to decompose the Uncertainty of the prediction back to the features, until I found your paper "Getting a GLUE: A Method for Explaining Uncertainty Estimates", and find it fantastic, do you have implementations of the "CLUE" in this repo or where can I find it, because I see the reference of this github in the paper? I'm researching on using ICU data to do Sepsis Prediction, really appreciate it if you can help.
Thanks in Advance
Hao

Hey @SuperHao-Wu,
The code is now available at: https://github.com/cambridge-mlg/CLUE
Best,
Javier


microsoft/aspnet:4.7.2-windowsservercore-1803 timezone correct but time is incorrect

Using Image microsoft/aspnet:4.7.2-windowsservercore-1803
Run: docker exec container powershell get-date... returns incorrect time.
Expected: The Current (Now) time returned
Possible issue:
w32Time Service is not running? Don't seem to be able to make it run, always exits with access denied.
Thanks

Please direct this issue to the Windows Containers forum at https://social.msdn.microsoft.com/forums/en-us/home?forum=windowscontainers. Your examples make use of mcr.microsoft.com/dotnet/framework/runtime:4.7.2-windowsservercore-1803. There's nothing special about that image from a .NET standpoint. It's essentially just a thin wrapper around the mcr.microsoft.com/windows/servercore:1803 image which already comes with 4.7.2 installed. One thing you may want to try is using one of the main Windows image at https://hub.docker.com/_/microsoft-windows.


Implement new logo

Paging superhero @NPDebs, make serve-gitpod worked!!!

Thank you for calling my attention back to it. 🤲🏽

Nope Debs! I no longer fly blind 😫.
Thanks boss, Berenice already pointed that out so I fixed it.
Would go ahead and draft this pr then 🤝

Nope Debs! I no longer fly blind 😫.

I'm happy for you. 😂

Berenice already pointed that out...

Always hundred steps ahead. 🤍

Would go ahead and draft this pr then 🤝

I'll try to reply sooner this time.


Home Section Improvement

Hey @SurajPratap10 I tried to Improve home section with logo of IMAGINE-AI and some animation, please assign me this issue under GSSOC.
Result
https://github.com/SurajPratap10/Imagine_AI/assets/99237795/69d248c2-10d3-4010-8369-62ef7144b725

assigned @9520prashant , You raise a PR and I will address the changes


[Enhancement] consider adding tolerations/nodeaffinity configuration

First off, thanks a ton for creating this. Great job! Hopefully the synology folks pick it up and run with it.
Many helm charts provide the ability to set tolerations and node affinity of the components they create. For example, if I wanted to make sure the synology-csi controller runs on my control plane nodes and not worker nodes I'd add the following values (particular values are specific to my infrastructure, obviously):
# infra nodes have "CriticalAddonsOnly" taint to ensure application workloads arent scheduled there
tolerations:
- key: "CriticalAddonsOnly"
  operator: "Equal"
  value: "true"
  effect: "NoExecute"

# try to schedule on control-plane nodes first
affinity:
  nodeAffinity:
    preferredDuringSchedulingIgnoredDuringExecution:
      nodeSelectorTerms:
      - matchExpressions:
        - key: node-role.kubernetes.io/control-plane
          operator: In
          values:
          - 'true'

This may even be considered a non-starter for any but the most trivial kubernetes clusters. I'm not sure if some of the synology-csi components are meant to be running on all nodes on a cluster. It seems to be working fine on my two node cluster even though everything is deployed to the worker node due to taints on the master, but im still getting things up and running.
Here is an example in oath2-proxy.

First of all, please accept my apologies for the very late response.
Well, I'm not sure anybody would really need iSCSI volumes on control-plane nodes because none of the predefined workloads (e.g. Core DNS) requires them in the first place, but... you are right that this should be supported anyway so that it could be used in similar use cases. Would you mind sending a PR?

I ended up switching to democratic-csi and simple nfs volumes due to synology's rather severe LUN limitations and some other usability issues i encountered trying to use iscsi.

In release 0.6.0, support for affinity, nodeSelector and tolerations was added.


[BEAM-14475] add test cases to GcsUtil

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

R: @pabloem
R: @chamikaramj

Run Java Precommit

retest this please

Run Java Precommit

Retest this please

run java precommit

run java precommit

Run Java PreCommit

these are great. thanks @johnjcasey !


How to listen on clipboard data changed?

Hello.
I can't find any documentation about how to register a listener on the clipboard changes. So when i put something in the clipboard via cmd+c i'm hope that some event could be fired.
So is it lacking of the documentation on API, or this feature hasn't been added yet?
Thanks.
(For example, QT has the signal http://doc.qt.io/qt-5/qclipboard.html#changed)

It hasn't been implemented. There is a issue #2280 tracking it already. So I merge this to #2280.


Reformat: Show empty days in between

How to zoom or scroll vertically? With long timespans, single bars get very small.


request: jfrog rt ls

I would love being able to list files or directories from a artifactory path, ignoring subfolder.
Example:
$ jfrog rt ls repo/path/to/foder

and it would list the content of subfolder.
rt search
only displays files, and does not display folder.
Currenytly, it seems it is only feasible through spec file
Thanks

@gsemet -
@yahavi created https://github.com/jfrog/jfrog-cli/pull/413 to support this functionality,

@gsemet
In addition to the search command, I'd like to inform you that this functionality is also available by the rt-fs JFrog CLI plugin. The rt-fs plugin allows you to run "file-system" like commands with Artiactory servers (Supported by JFrog CLI 1.41.0 and above).
You can install the plugin by running:
jfrog plugin install rt-fs

And then:
jfrog rt-fs ls repo/path/to/folder

Adding to @yahavi's message, the source code of the rt-fs plugin is available here.
Big LIKE for this plugin @yahavi!
Since it looks like this requirement has been implemented by both #413 and rt-fs, I guess this issue can be closed.


Error 500 - Running AutoTrain

now get the following error with the new UI:

see here for all the parameters
{
"lr": 0.00003,
"epochs": 3,
"batch_size": 2,
"warmup_ratio": 0.1,
"gradient_accumulation": 1,
"optimizer": "adamw_torch",
"scheduler": "linear",
"weight_decay": 0,
"max_grad_norm": 1,
"seed": 42,
"block_size": 1024,
"use_peft": true,
"lora_r": 16,
"lora_alpha": 32,
"lora_dropout": 0.05,
"logging_steps": -1,
"evaluation_strategy": "epoch",
"save_total_limit": 1,
"save_strategy": "epoch",
"auto_find_batch_size": false,
"fp16": true,
"use_int8": false,
"model_max_length": 2048,
"use_int4": true,
"target_modules": "",
"merge_adapter": false,
"use_flash_attention_2": false,
"disable_gradient_checkpointing": false
}:

what do the logs say?

you also need special access to A100, if you dont have access, choose A10G Large

it seems that it is working now:

thanks!

💥 closing the issue :)


Accessibility issue in content switcher component

Accessibility issue in content switcher component
For ibm-content-switcher, the accessibility checker gives  the error
"The ARIA attribute 'aria-selected' is not valid for the element  with ARIA role 'button'"

Hi @bikashpattnayak could you tell us what version of carbon-components-angular you are using?

hi @maicongodinho, I am currently using "carbon-components-angular": "^4.40.0" in my project.

Hi @maicongodinho , I did the upgrade to the exact version 4.59.7. Still see the same issue.

Can you share the template with us? (how you are using the ibm-content-switcher)

Strange @bikashpattnayak, I created a component with just that component and ran the IBM Accessibility Checker tool. No errors were found as reported.

You are sure that error is from this component?

@bikashpattnayak Can you verify if your accessibility assessment plugin is up to date? I cannot reproduce this error as well via storybook or creating a demo app.

The role attribute of the button that the ibmContentOption directive is assigned to is set to button instead of tab. ibmContentOption directive automatically assigns the role tab, so by removing the role on your part (Assuming you have overriden it) will resolve this issue.


OCPBUGS-6859: Consolidate manifest directory location and clean up RPM spec file

MicroShift attempts to load manifest files from /etc/microshift/manifests and /usr/etc/microshift/manifests directories
The /etc/microshift/manifests empty directory is created when installing RPMs
Clean up  unused directories

/var/hpvolumes
/var/run/kubelet
/var/lib/kubelet/pods
/var/run/secrets/kubernetes.io/serviceaccount

Closes OCPBUGS-6859

pkg/config/config.go still points to /usr/lib/microshift/manifests/ and not /usr/etc/microshift/manifests/. This should be changed as well, right?

/lgtm

@pmtk , I missed a few places to replace the manifests directory. More files added now - can you look at it again?

/retest

This is going to trigger documentation updates, too.
What is the motivation for changing /usr/lib/ to /usr/etc/? How are those directories expected to be used on an ostree-based system?
/hold

What is the motivation for changing /usr/lib/ to /usr/etc/? How are those directories expected to be used on an ostree-based system?

The reason is due to symmetry with config.yaml file, which is located in the /etc or /usr/etc pair.
So, the situation I was trying to fix is the following

/etc/microshift/config.yaml.default -> /usr/etc/
/etc/microshift/mamifests -> /usr/lib/microshift/manifests
We had 2 configuration directories under /usr.

@dhellmann, I reverted the manifest directory change, as discussed.
/unhold


⚠ Bofrost.es has degraded performance

In be0c50e, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1576 ms

Resolved: Bofrost.es performance has improved in b23a2af after 6 minutes.


[Static Web Apps] Fix Node.js and Core Tools version mismatch

This PR brings a few changes:

It updates the base image to mcr.microsoft.com/azure-functions/python:4-python3.9-core-tools, updating Python/Core Tools versions and allowing support for M1 macs
It changes the default Node.js version to 16 instead of LTS to avoid unwanted upgrades and mismatch with core tools version
It adds a CORE_TOOLS_VERSION param to allow switching to a different core tools version that the default one, for example if you wish to use a Node 14 version
It fixes the mismatch between the previous default Node version (16) and Core Tools (v3) installed in the previous image

Fixes #1306

LGTM! @anthonychu @TsuyoshiUshio Does this look good to you?

Looks good. Keep in mind that Node.js 16 isn’t available in Static Web Apps functions yet. But hopefully by the time this change is released to stable it will!

Sounds good. Merging!


YUBICO_SERVER can no longer be an empty string, causes restart loops

This vaultwarden commit, done as part of addressing vaultwarden issue #3003, appears to disallow setting YUBICO_SERVER to an empty string, such that it now considers it a configuration error rather than quietly(?) failing and continuing.  The default .env.template, and any .env that left those lines in, now produces a restart loop and Error 503 for any users.
For users: workaround is to comment out YUBICO_* in .env, and re-run docker-compose up. Presumably .env.template should be changed so those lines are commented out by default as well.
Is there any mechanism to help existing deployments here other than asking to patch vaultwarden to consider empty strings to be unset?
[edit] I just noticed that the Vaultwarden issue has someone expecting an empty string to be some other behavior, so the idea of considering empty strings to be unset might not even be accepted. [edit2] Their reasoning here.

Ran into this myself this AM, as my watchtower schedule was weekly and they updated the image just after my last watchtower check-in.
Thanks for flagging, I came to the same conclusion and commented the vars out. It seems like they will keep this "empty string is a config error" approach, which I agree with.

+1, thanks for flagging. The workaround worked for me as well.

Thanks @AySz88 for finding this. I updated .env.template and put a notice at the top of the project readme, but beyond that I'm not sure there's much else we can do. If you have any other ideas on how to make this as painless as possible for everyone who will be running into this, let's discuss it here.


[BUG] : Template path is not directing to path defined in navigation

Bug Report
Description
When we click on Template in the navbar, it gives a 404 error, not directing us to the correct path.
Steps to Reproduce

Go to the website's  navbar
Click on Template
It will give a 404 error

Expected Behavior
It should redirect to the '/collections' path and will give the correct template code.
Actual Behavior
It's going to the '/template' path and giving a 404 error
Screenshots

Environment

Operating System: [e.g., Windows, macOS, Linux]
Browser: [e.g., Chrome, Firefox, Safari]
Version: [e.g., 22]

Additional Information
I am a participant in GSSoC 2024, please assign this issue to me

@richab246
It should redirect to '/template' as of now. The template page hasn't been created yet

@AkashSharma001  Can I create template page


Implementation Question

Hi Click community,
i am working on a decorator library myself, and I wonder how do you avoid that during debugging the debugger steps into your library code?
What I try to say is that in my library the decorator modifies a function, including a wrapper that is executed before the user function. During debugging, when I step into the user function, i am first in the library code and only than the function is called.
thx,
Manuel

As far as I know we don't. PyCharm's debugger has an option like "only step into my code", but that's not related to Click.
This question may be better suited on the mailing list (flask@python.org), or on a discussion forum. This issue tracker is for issues with Click itself.


This pr is to add Copyright

Ⅰ. This pr is to add Copyright
Ⅱ. Does this pull request fix one issue?

fixes #XXXX
Ⅲ. List the added test cases (unit test/integration test) if any, please explain if no tests are needed.
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Something wrong


pull out diffie-hellman

Will review. ETA 24 - 48 hrs.

Will finish https://github.com/crypto-browserify/diffie-hellman/pull/11 then I'll be good to ACK.

rebased

This looks like it's good to go now.

1efee1790e4f66894c3476c4b00d2300dc4eadf6 3.9.11


App crash after build: FATAL EXCEPTION: main - java.lang.RuntimeException: Unable to instantiate activity - java.lang.ClassNotFoundException: Didn't find class "com.NAME.APPNAME.MainActivity" on path: DexPathList

I took me way to long to fix this error, so I hope this helps someone else:
java.lang.RuntimeException: Unable to instantiate activity ComponentInfo{com.NAME.APPNAME.dev/com.NAME.APPNAME.MainActivity}: java.lang.ClassNotFoundException: Didn't find class "com.NAME.APPNAME.MainActivity" on path: DexPathList[[zip file "/data/app/com.NAME.APPNAME.dev-XzP2FDlcF7ok_QeihJlBQQ==/base.apk"],nativeLibraryDirectories=[/data/app/com.NAME.APPNAME.dev-XzP2FDlcF7ok_QeihJlBQQ==/lib/x86, /data/app/com.NAME.APPNAME.dev-XzP2FDlcF7ok_QeihJlBQQ==/base.apk!/lib/x86, /system/lib]]
Maybe this error came because I renamed the bundle itentifier in the AndroidManifest files and everywhere else I could find it. This step wasn´t explicitly explained in the setup of the project.
My solution in the end was to also rename the "android/app/src/main/kotlin/com/venturearkstudio/noteapp" folder structure into my bundle identifier ".../com/NAME/APPNAME". And then also in the MainActivity.kt file the "package com.venturearkstudio.noteapp" to "package com.NAME/APPNAME".
Not sure if these steps should be added into the setup instructions or if I made something wrong and then ran into this issue.

Hello @rainerlonau ,
Thanks for the heads up!
Sorry, was away for few short day. Yes, that is 1 way to fix the MainActivity issue. But, so far this is the first time I saw issue as this. Let me do a update on the latest Flutter build on this project and try.

Maybe this error came because I renamed the bundle itentifier in the AndroidManifest files and everywhere else I could find it. This step wasn´t explicitly explained in the setup of the project.

I guess it´s because of what I mentioned. In your setup you have a few spots where you explain to change the bundle identifier. But I changed it also in other places and I guess that´s when MainActivity could be found anymore because the folders were not renamed correctly (at first)

Ok, great. After I complete a new project that used this package, I will update the README on this. Thanks again.:)


Triple "a" apps?

cross-platforma apps.

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

ID: 4cc8f83c-a8f5-ab4e-76f5-b617b65fe1ed
Version Independent ID: 43dfad72-eea2-7607-9bee-b43f1cbab255
Content: Sharing Code Options - Xamarin
Content Source: docs/cross-platform/app-fundamentals/code-sharing.md
Service: unspecified
GitHub Login: @asb3993
Microsoft Alias: amburns

Thanks, @brokenjavagit! I went ahead and fixed this typo. Fix should be published soon.
Appreciate the help!


chore: api-backend to 0.0.16

chore: Promote api-backend to version 0.0.16

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Test

Hi,
We are cleaning up the MERL Center repository in order to make it easier for new members to navigate and avoid sending reminder messages for issues that you no longer want to work on.
Please check the box(es) below and let us know if you are still interested in working on this issue and your role in the work product.
I'm @Rabia2219 , here's my status on this issue

[ ] I'm interested in being the Lead Author
[ ] I'm interested in being a Contributing Author
[ ] I'm interested in being an Editor
[ ] I no longer have time to work on this
[ ] I don't have time to work on this now, but contact me when other people are actively working on it and I might be able to join then

@ExperimentsInHonesty are you getting this?

Response from GitHub Support
Hi Rabia,
Thank you for following up! Taking a look at those specific events, I can confirm that our system did send notifications consistent with the mentioned user's account settings. For example, users with only web notifications enabled received a web notification, while any with email set as their preferred delivery destination received an email. I have cross-referenced this with our email server logs and the email messages are indeed being sent successfully from our end. Your account will not have received any notification for the @mention in the first comment, as we don't send a notification for self-mentions from the same account.
I hope this information helps! Please let me know if you have any further questions at all.
Cheers,
Alex

This issue is complete.  It turns out they are just ignoring the messages they receive in email, if we email by @ mention.


Update powershell-beginners-guide.md

clear is also an alias for clear-host.
PR Summary

PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[ ] Summarized changes
[x] Change is not breaking
[ ] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: to the beginning of the title and remove the prefix when the PR is ready.

User-facing changes

[ ] Not Applicable
OR
[x] User-facing Documentation needed

[ ] Issue filed - Issue link:

Testing - New and feature

[x] Not Applicable or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

[ ] Add [feature] if the change is significant or affects feature tests

@iSazonov please take a look again.

@aavdberg Thanks for your contribution!


Can't hit a plugin entity

I tried the plugin it worked just fine for me but I noticed that I can't hit the user entity from users-permissions plugin and it's just made for api content types so is that a bug or it's just how this plugin work?

Hello @Louai99k ,
If I got you correctly you want to fetch all comments per user. Am I right? If yes, plugin is not strictly connected with User entity and such functionality might be useful separated endpoint like api/comments/my. Should be easy to do.

No I actually want to post a comment direct to the user content type from the users-permissions plugin like
POST http://localhost:1337/api/comments/plugin::users-permissions.user:<USER_ID>

As for now it's possible to post comments only against api:: like collections which by nature are "content like". Without such rule also 'roles' entities are going to be available what is a bit tricky.

Can I try to added then make a PR?

I'm not seeing working with built in collections as a feature on our roadmap. I would suggest you to make an extension to the plugin service using Strapi Extensions concept.

Comments are working against collections exposed and configured via Settings page so its indeed as you described. Plugin works for what is set via dedicated page, and it contains as available only api:: like collections.


[firestore-bigquery-export] Using Firestore export for importing documents not helpful

This feature request is for extension: _ (firestore-bigquery-export)
What feature would you like to see?
The docs suggest that one should use Firestore exports for large collections, but that is not very helpful as an BigQuery import from a Firestore export has a very different schema than the schema of the extension.
A better approach would be to make use of the Firestore getPartitions() functionality on collection groups and from there spin up various instances (maybe Cloud Run, maybe Compute Engine), run these queries all in parallel.
And each query would then stream the data to a Cloud Storage file that after the query partition is finished gets loaded into BigQuery using a batch (rather than a streaming) import.
How would you use it?
This would work perfectly for collection with 100k or more rows.
Questions about concrete implementation
Or is my premise wrong and I am missing something that makes the "import of the Firestore export"-approach compatible?
Is this something that makes sense to you from a architectural approach?
Would it make sense to use Cloud Run if ensuring that the computation time per partition is < 1h. (Probably easier to setup/deploy)
Or would it make sense to go directly with Compute Engine?

Hey @IchordeDionysos, there is an import script that batches Firestore records and inserts them into the BigQuery table created by the extension. Documentation here. Does this fulfil your needs?

Hey @IchordeDionysos, there is an import script that batches Firestore records and inserts them into the BigQuery table created by the extension. Documentation here. Does this fulfil your needs?

@russellwheatley yes, I am aware of, but for larger collections this is not really an option as the import would take much longer and it honestly a bit flaky for that, due to Firestore backend errors in large batch jobs.
The proposed alternative was to use a Firestore export, but as described above this does not really fulfill the need, as the schema is different.

@russellwheatley yes, I am aware of, but for larger collections this is not really an option as the import would take much longer and it honestly a bit flaky for that, due to Firestore backend errors in large batch jobs.
The proposed alternative was to use a Firestore export, but as described above this does not really fulfill the need, as the schema is different.

That's fair enough. I don't have experience backfilling such a vast amount of documents into an existing BigQuery table created by the extension.
Hey @fredzqm, do you have any thoughts on a solution to such a problem?

That's fair enough. I don't have experience backfilling such a vast amount of documents into an existing BigQuery table created by the extension.
Hey @fredzqm, do you have any thoughts on a solution to such a problem?

There exists a way to export Datastore/Firestore and import them into BigQuery.
https://cloud.google.com/bigquery/docs/loading-data-cloud-datastore
The schema, however, isn't compatible. I would recommend using UNION those tables in SQL if desired.
The pro for the managed export pipelines: parallelized, all data exported. It is a good fit if u want a periodically backup and write volume is high. All documents are billed one read for one export.
The pro for the BigQuery export extension: delta based, realtime. It is a good if u want to track all history changes and write volume is low. Cloud Function is billed.

There exists a way to export Datastore/Firestore and import them into BigQuery.
https://cloud.google.com/bigquery/docs/loading-data-cloud-datastore
The schema, however, isn't compatible. I would recommend using UNION those tables in SQL if desired.
The pro for the managed export pipelines: parallelized, all data exported. It is a good fit if u want a periodically backup and write volume is high. All documents are billed one read for one export.
The pro for the BigQuery export extension: delta based, realtime. It is a good if u want to track all history changes and write volume is low. Cloud Function is billed.

@russellwheatley let's try to work out and document some way to do a big initial import. Whether it's making the schemas mesh between Firestore exports to BQ and our import script, or another suggestion on how to do a big import. Right now our docs are a little handwavey:

The import script can take up to O(collection size) time to finish. If your collection is large, you might want to consider loading data from a Cloud Firestore export into BigQuery.

@russellwheatley let's try to work out and document some way to do a big initial import. Whether it's making the schemas mesh between Firestore exports to BQ and our import script, or another suggestion on how to do a big import. Right now our docs are a little handwavey:

The import script can take up to O(collection size) time to finish. If your collection is large, you might want to consider loading data from a Cloud Firestore export into BigQuery.


build: make peerDependencies into normal dependencies

Because this package was using peerDependendencies when it should be using regular
dependencies it is incompatible with projects that use non-compliant versions of those
packages.
The project that I'm working with uses another major version of uuid. Since this package
makes uuid a peer dependency, it will try to use the version I have installed, even though
it's incompatible. I don't see any downside to using normal dependencies rather than peer
dependencies besides some packages being duplicated, which is not much of a concern
for me at least. From what I understand, peer dependencies are mostly used for creating
plugins, which this package is not doing.

Ping @onebytegone


Option to send multiple response headers as comma separated single header

RFC 2616 specifies that response headers with multiple values sent as comma separated single header or as multiple headers to be the treated the same. Current implementation of Slim 3.x sends as multiple headers only.
Although this may not be an issue with modern browsers, I ran into a problem with a legacy client application that cannot handle duplicate headers. (No problem handling comma-separated values).
Adding a new container settings - singleHeader gives the flexibility to send response header in either format.

Reading up on this, it seems to be preferred to send one header only with multiple values as comma separated.
Should be just change our implementation to do that?

Reading up on this, it seems to be preferred to send one header only with multiple values as comma separated.
Should be just change our implementation to do that?

Ah… Set-Cookie must be sent as multiple separate headers

Ah… Set-Cookie must be sent as multiple separate headers

I'd think that dealing with a client that is unable to properly parse HTTP does not warrant implementing a configuration settings for ALL Slim applications to change behavior for ALL clients.

I agree with SvenRtbg.

I have to disagree. It is not my intention to affect ALL Slim applications. That is why I specified this as an optional settings.
The option is also not arbitrary. It is part of the standard spec. & I wanted the flexibility.
As I stated earlier, if I could change the client, I would have.
You know that not all clients of Slim server are browsers.
A simple mistake by a fellow programmer without foresight affects us all.
Thanks for looking into this.

My preference is to send one header with multiple values where possible, as my reading of RFC2616 is that this is preferred over sending multiple headers with the same name.
However, as I commented on 1st Jan, this cannot be a blanket thing as there are headers where this doesn't apply, such as Set-Cookie and may be others? Similarly, they may be headers where there can only be one value?
I don't want a setting; I want Slim to do the right thing.

One could always write a middleware which rewrites the headers to format which the legacy client can understand.
$app = new Slim\App

$app->add(function($request, $response, $next) {
    $response = $next($request, $response);
    $something = $response->getHeaderLine("X-Something");
    return $response = $response->withHeader("X-Something", $something);
});

$app->get("/header", function($request, $response, $arguments) {
    return $response
        ->withAddedHeader("X-Something", "foo")
        ->withAddedHeader("X-Something", "bar");
});

$app->run();

Yields
$ curl --include httlhost:8080/header
HTTP/1.1 200 OK
X-Something: foo,bar

@akrabat Respect your decision. Thanks.
@tuupola Thanks. Will try it out.


Rambox can't find GSettings schemas

Describe the bug
A clear and concise description of what the bug is.
Rambox apps cause rambox to crash when attempting to upload a file.
It gives the message:
(rambox:15067): GLib-GIO-ERROR **: 10:09:18.390: No GSettings schemas are installed on the system

To Reproduce
Steps to reproduce the behavior:

Install rambox
Run rambox in terminal
Add any app that has file upload functionality (I tested Google Messages, Element, and Slack)
Attempt to upload file

Expected behavior
Rambox apps should be able to upload files from filesystem
Additional context
I've seen from browsing other issues that this has been solved before for riot-desktop at first by adding gsettings schema to the package wrapper but currently fixed by using wrapGAppsHook in electron. However, for some reason this did not fix the issue in Rambox.
Notify maintainers

@gnidorah @Ma27
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
these paths will be fetched (0.05 MiB download, 0.28 MiB unpacked):
/nix/store/d43v6bx7r6fcaq3fbbfd5mwh1f5s7rmg-bash-interactive-4.4-p23-dev
copying path '/nix/store/d43v6bx7r6fcaq3fbbfd5mwh1f5s7rmg-bash-interactive-4.4-p23-dev' from 'https://cache.nixos.org'...

system: "x86_64-linux"
host os: Linux 5.4.49, NixOS, 20.03.2533.76adfc180c0 (Markhor)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.3.6
channels(root): "nixos-20.03.2533.76adfc180c0, nixpkgs"
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute:
# a list of nixos modules affected by the problem
module:

Add wrapGAppsHook to https://github.com/NixOS/nixpkgs/blob/df81d9a41c94792e02527dc9fa105153155f2e7c/pkgs/applications/networking/instant-messengers/rambox/default.nix#L21


Cherry pick #321 to v0.6: Update the detection method for docker overlay2 issue

This PR is to cherry pick #321 into the v0.6 branch.
/assign @wangzhen127

/lgtm
/approve


problems

this paper just evently attribute revenue to all touchpoint in click path , it is naive

@cysjtu Yeah, I did notice that. It does a unique on the click path and does the attribution from there i.e. A -> B -> A -> C becomes attributed equally to {A, B, C}. Let me look for better papers.


Error: connect ECONNREFUSED

I've been banging my head over this for about a week now and I can't seem to find a solution.
I'm on OSX Sierra with phantomjs-prebuilt in my node_modules and I get the following error:
Error: connect ECONNREFUSED <IP_ADDRESS>:49837
     at Object.exports._errnoException (util.js:1023:11)
     at exports._exceptionWithHostPort (util.js:1046:20)
     at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1090:14)

And this is the code:
let html = pug.renderFile(Utils.root('src/templates/test.pug'), { data: data });
convertToExcel(html, (err, stream) => {
  if (err) {
    console.error(err);
    throw err;
  }
  res.setHeader('Content-Type', 'application/octet-stream');
  res.setHeader('Content-Disposition', 'attachment; filename=report.xlsx');
  stream.pipe(res);
}

I've tried several different configurations with no success so far.
Can you provide any insight?

You should try to create the conversion function with these options
var conversion = require("html-to-xlsx")({
  strategy: 'dedicated-process',
  phantomJSPath: require('phantomjs-prebuilt').path
});

However, maybe you will need to create for me  a minimal repository where I can replicate this.

Thanks for the quick reply.
Your suggestion works but I'm wondering why I can't use 'phantom-server' strategy.
Using 'dedicated-process' seems to be really slow. I also use your other library (phantom-html-to-pdf) and I'm creating pdfs faster than I'm creating xlsx, which seems a little odd.

Hm, it uses the same underlying library - https://github.com/pofider/phantom-workers
So it should work the same for both.
Did you try to pass only phantomJSPath and keep default strategy?

Yes I did and it gives me the same error as before. Could it be something with OSX Sierra and phantomjs? If so, I can live with it since production deployment will be on a linux machine where it will - hopefully - work.

Could be, but the phantomjs from the require('phantomjs-prebuilt').path should be fine.
This package has outdated phantom-workers, could you try to update it and try again?

Nope. Doesn't matter what I do, whenever I use the default strategy, which is phantom-server, it throws the same error.

Hmm, I guess you will have to add some console.logs to find out if the phantomjs is crashing right after the start or it is not accepting the communication.
I've seen problems opening http connection to phantomjs previously on some corporate environments with proxy. However the problem can be somewhere completely else.

Can you try the same code also on windows or linux?

I'll try it and let you know. Thanks for the help.

So, I tried it on a Linux machine (with the same phantomjs version) and everything works as expected  so I guess it's a problem specific to OSX Sierra.

Thank you for info. So I guess this is for you solved, you can use dedicated process in development and production is linux anyway for you.

Yes, that's right. You can close the issue and thanks again for the help.


gzrtp module needs updating

Looks like gzrtp module needs updating:
CXX [m] build-x86_64/modules/gzrtp/gzrtp.o
modules/gzrtp/gzrtp.cpp:172:1: error: invalid conversion from ‘int (*)(menc_media**, menc_sess*, rtp_sock*, int, void*, void*, sdp_media*)’ to ‘int (*)(menc_media**, menc_sess*, rtp_sock*, udp_sock*, udp_sock*, const sa*, const sa*, sdp_media*)’ [-fpermissive]
 };
 ^

Its media_alloc function does not match current version of menc_media_h.  Or is it so that the module is not anymore used by anyone?

thanks for the report, I just fixed it in commit
f9d1e0879c214c03a2f2408031b69233f707d4a7
I am not using the gzrtp module personally, but I think we can keep it here for now.

Compiles now.


ContactPicker InvalidOperationException

Describe the bug
I am using ProjectReunion 0.8.1 on a Desktop app. When using the ContactPicker, I am getting an InvalidOperationException. This is for a UWP app that we are testing to port to WinUI Desktop. Any help or guidance is appreciated. Thanks.
Steps to reproduce the bug
ContactStore contactStore = await ContactManager.RequestStoreAsync(ContactStoreAccessType.AllContactsReadOnly);
Windows.ApplicationModel.Contacts.ContactPicker page = new Windows.ApplicationModel.Contacts.ContactPicker();
page.SelectionMode = ContactSelectionMode.Contacts;
var contacts = await page.PickContactsAsync();
Expected behavior
ContactPicker to launch
Error
System.InvalidOperationException
HResult=0x80131509
Message=Exception of type 'System.InvalidOperationException' was thrown.
Source=WinRT.Runtime
StackTrace:
at WinRT.ExceptionHelpers.ThrowExceptionForHR(Int32 hr)
at ABI.Windows.ApplicationModel.Contacts.IContactPicker2.global::Windows.ApplicationModel.Contacts.IContactPicker2.PickContactsAsync()
at Windows.ApplicationModel.Contacts.ContactPicker.PickContactsAsync()
at App4.MainWindow.<myButton_Click>d__5.MoveNext() in C:\Users\james\source\repos\App4\App4\App4\MainWindow.xaml.cs:line 64
onecoreuap\shell\windows.system.launcher\lib\windows.system.launcher.cpp(1659)\Windows.System.Launcher.dll!18620262: (caller: 7A608038) Exception(1) tid(3ff4) 8000000E A method was called at an unexpected time.
Exception thrown at 0x760EB4B2 in App4.exe: Microsoft C++ exception: wil::ResultException at memory location 0x02F7AEE0.
Exception thrown at 0x760EB4B2 in App4.exe: Microsoft C++ exception: [rethrow] at memory location 0x00000000.
onecoreuap\shell\windows.system.launcher\lib\windows.system.launcher.cpp(1676)\Windows.System.Launcher.dll!18620230: (caller: 7A608038) ReturnHr(1) tid(3ff4) 8000000E A method was called at an unexpected time.
Msg:[onecoreuap\shell\windows.system.launcher\lib\windows.system.launcher.cpp(1659)\Windows.System.Launcher.dll!18620262: (caller: 7A608038) Exception(1) tid(3ff4) 8000000E A method was called at an unexpected time.
]
Exception thrown: 'System.InvalidOperationException' in WinRT.Runtime.dll
Version Info
ProjectReunion 0.8.1

Windows 10 version
Saw the problem?

May 2020 Update (19041)
Yes

You may need to call IInitializeWithWindow::Initialize on the object before invoking PickContactsAsync.

I tried the InitializeWitWindow method as with a FolderPicker etc but is getting a InvalidCastException. Am I missing anything?
ContactStore contactStore = await ContactManager.RequestStoreAsync(ContactStoreAccessType.AllContactsReadOnly);
Windows.ApplicationModel.Contacts.ContactPicker page = new Windows.ApplicationModel.Contacts.ContactPicker();
var hwnd = this.As().WindowHandle;
var initializeWithWindow = page.As(); //InvalidCastException
initializeWithWindow.Initialize(hwnd);
            page.SelectionMode = ContactSelectionMode.Contacts;
            var contacts = await page.PickContactsAsync();

System.InvalidCastException: Specified cast is not valid.
at System.Runtime.InteropServices.Marshal.ThrowExceptionForHR(Int32 errorCode)
at WinRT.IObjectReference.AsInterfaceTInterface
at WinRT.CastExtensions.As[TInterface](Object value)
at App4.MainWindow.myButton_Click(Object sender, RoutedEventArgs e)

HI,
May I know if there is any update on this? I retested the issue on Windows 11 (10.0.22000 Build 22000) with WindowsAppSDK (1.0.0-preview3) and is getting the same InvalidOperation error on PickContactsAsync:
ContactStore contactStore = await ContactManager.RequestStoreAsync(ContactStoreAccessType.AllContactsReadOnly);
Windows.ApplicationModel.Contacts.ContactPicker page = new Windows.ApplicationModel.Contacts.ContactPicker();
var contacts = await page.PickContactsAsync();

onecoreuap\shell\windows.system.launcher\lib\windows.system.launcher.cpp(1628)\Windows.System.Launcher.dll!00007FFA894C04F7: (caller: 00007FFA5B2C7D53) ReturnHr(1) tid(626c) 8000000E A method was called at an unexpected time.
Msg:[onecoreuap\shell\windows.system.launcher\lib\windows.system.launcher.cpp(1611)\Windows.System.Launcher.dll!00007FFA894C0405: (caller: 00007FFA5B2C7D53) Exception(1) tid(626c) 8000000E A method was called at an unexpected time.
]
Exception thrown: 'System.InvalidOperationException' in WinRT.Runtime.dll
If I add the following:
IntPtr m_hwnd = WinRT.Interop.WindowNative.GetWindowHandle(currentWindow);
WinRT.Interop.InitializeWithWindow.Initialize(page, m_hwnd);
I am getting the following in the second line:
Exception thrown: 'System.InvalidCastException' in System.Private.CoreLib.dll

Just tested some code now and I get the InvalidCastException when trying to show the picker. I did have to initialize the picker with the nice WinRT.Interop.InitializeWithWindow.Initialize API.


This code doesn't funct in devices of less than 1000px

Bug
Please include the following information with your bug report:

What MDL Version are you using? latest version, I'd downloaded it today

What browser(s) is this bug affecting (including version)?
I'm using Mozzila Firefox, last build. But I've tested it on Chrome too

What OS (and version) are you using?
Windows 10 Pro, last build

What are the steps to reproduce the bug? Can you create a plunker/codepen/jsfiddle which reproduces it?
https://pastebin.com/S0pJcn47

What is the expected behavior?
The menu is usable for all devices.

What is the actual behavior?
Into 1000px or less, it just disappears.

Any other information you believe would be useful?
here you've got a screenshot:

Image is bad, invert text to the other site


[MWPW-155627] Merch Callout feature: Missing font cascade

Added fallback to "Adobe Clean" font for callout text.
Callout text font before Fix:

Callout text font after Fix:

Resolves: MWPW-155627
Test URLs:
Before: https://main--milo--adobecom.hlx.page/drafts/rosahu/rosahu
After: https://mwpw-155627--milo--rohitsahu.hlx.page/drafts/rosahu/rosahu

I have completed testing on the Milo pull request from the forked branch rohitsahu:MWPW-155627.
Result: The 'Adobe Clean' font is now correctly displayed for callout text. The issue has been resolved. I have added the label 'ready for Stage' to the Milo PR.

@rohitsahu - are the fonts getting applied to radio buttons as well?

It is only for the callout text


Database Compression

Files Added

compressed_tile.cpp
compressed_tile.h

Other Existing Files Modified
Some additional functions have been added to support database compression. Here are the following files:

tile.h
tile.cpp
tile_group.h
tile_group.cpp
data_table.cpp
transaction_level_gc_manager.cpp

Slides

Click Here

Strategy for Compressing

Once a tile gets full (all slots occupied by tuples), the CompressTile can be created using the CompressTable function
We scan each column and sort it.
We compute the median, min and max, this gives us the minoffset(min-median) and maxoffset(max-median).
If these offsets can be represented in a smaller data type than the original data type, we compress the entire column.
The median is chosen as the base, and essentially only the offsets are stored in the column.
Currently we can compress SMALLINT and INTEGER and BIGINT. TINYINT is already 1 byte so we do not compress it.

Modifications done to GC

While adding a tuple to the recycle map,the garbage collector ensures that the tuple does not belong to a compressed tile group.
If it belongs to a compressed tile group, it skips this tuple to ensure that it is never returned when the ReturnFreeSlot function is called.

Tile Compaction

Tile Compaction can be implemented in the GC, once the GetActiveTupleCount function is fixed. The strategy would be to reset the tiles inside the tile group. The notion is that the executor would only examine a tile group if it has non zero active tuple count.

Testing

Compression Correctness Test
Compression Size Test
VARCHAR Deduplication Test

Compression Correctness Test:

This test inserts 2500 tuples (Unit Tests for Integers and Decimals are included)
The first 2 tile groups are full (since the max capacity is 1000 tuples per tile group) and get compressed.
The third tile group has 500 slots vacant and is still not full and is uncompressed.
Thus we now have compressed and uncompressed data.
We now perform a SELECT * on this and expect to correctly recieve the true value of the compressed data and uncompressed data.

Compression Size Test:

This test inserts 100 tuples
Each tuple is of the form [Integer, Integer, Decimal, Varchar]
Thus the tuple length is (4+ 4+ 8+ 8) = 24 bytes.
We currently support the compression of integers
The integers get compressed to TINYINT making the tuple sizes now (1+ 1+ 2+ 8) = 12 bytes
The test checks this decrease in size.

VARCHAR Reduplication Tests

Unit tests to check reduplication by creating 5 distinct VARCHAR [foo, bar, hello, world] and duplicating entries for each of them
The CompressTable function de-duplicates the data and stores the value as SMALLEST.

Coverage decreased (-0.8%) to 70.684% when pulling e0e676371a275d73e4073d36d664f44f02009cda on rohit-cmu:master into 8e87b9f885ac49df67b2d120134980f79819d251 on cmu-db:master.

Coverage decreased (-0.8%) to 70.675% when pulling f8210e5f3d15d581813da174e7c50245ab49b7e7 on rohit-cmu:master into 8e87b9f885ac49df67b2d120134980f79819d251 on cmu-db:master.

Coverage decreased (-0.8%) to 70.67% when pulling 552f5d9f7e1a812beb83bc83a5eec6b986c7c37e on rohit-cmu:master into 8e87b9f885ac49df67b2d120134980f79819d251 on cmu-db:master.

Coverage decreased (-0.8%) to 70.651% when pulling 552f5d9f7e1a812beb83bc83a5eec6b986c7c37e on rohit-cmu:master into 8e87b9f885ac49df67b2d120134980f79819d251 on cmu-db:master.

We have addressed all the comments from Code Review 2.

We should have a discussion about whether we want to merge this.
Nice job guys.

We decided that it is going to be too much work to merge this PR. We may use it as a reference in the future. Let's talk about plans for next semester when you get back. Thanks for all your work!


Temporarily disable Tizen build.

Also includes a few Tizen build improvements:

Building cross filesystem has changed.
Synchronize build arguments for submodules with official builds.

@dotnet-bot test ci please