Restore patch program alt shortcuts

I am used to pressing alt + e + p + a to patch bytes and alt + e + p + p to apply patches only using the keyboard so I added the alt shortcuts

I like this. Seems like a no brainer. Thanks!


However, RESTful naming conventions recommend the plural form for resource collections.
Excerpt:

A collection resource is a server-managed directory of resources. Clients may propose new resources to be added to a collection. However, it is up to the collection to choose to create a new resource, or not. A collection resource chooses what it wants to contain and also decides the URIs of each contained resource.
Use “plural” name to denote collection resource archetype.

Consider using the following plural forms instead:

poems
sessions
queues

This might warrant a separate issue in itself, but on the topic of path names, I suggest categorizing "comments" as a sub-collection of poems.
For example, to request an index of comments for a given poem, a RESTful client might expect the form:
GET /api/poems/{id}/comments

I agree on the plural resource names.

I suggest categorizing "comments" as a sub-collection of poems.

However, I don't think this particular change makes sense for our use cases. We really need to flesh it out some more, but a poem could have different sets of comments on it in different contexts. For example, a poem could exist in the global database and have comments on it in a non-session context. It could also be part of multiple clubs' sessions and have comments on it in the context of each of those sessions.


ente-auth: init at 3.1.3

Description of changes

Closes #336291
I'm aware there already is an open PR at #333514 but my PR inits ente-auth as a from-source build.
The appimage wrapped in #333514 has issues detecting OpenGL libraries at least on my system, while the from-source build works without issues on my setup.
I added @schnow265 as maintainer, assuming he's willing to help maintain a from-source flutter build as well. If not, please notify me if I should remove you from the list.
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

Of course I want to help.

Added updateScript and tests.version.

I attempted to package ente-auth a few weeks before, but got stuck: one of the flutter dependencies called sqlite3_flutter_libs tries to download the native sqlite3, even though sqlite is specified in the build input. I'm surprised you don't have this in pubspec.lock.json, how did you generate this file?
Also I'd be happy to be a maintainer too :)

Thanks for all the feedback, I'll address it shortly!

Running it on x86_64-linux in an x11 wm results in a black window, it prints the following: [ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: MissingPlatformDirectoryException(Unable to get application documents directory) I made sure ~/Documents exists, I'll troubleshoot this later.
Apart from the update script, code lgtm.
Also 4.0.2 got released recently.

I had the same issue installing ente-auth.
I figured out that you have to install the xdg-user-dirs package to fix this issue.
Unforunatly I can't rember were I read this.

I had the same issue installing ente-auth.
I figured out that you have to install the xdg-user-dirs package to fix this issue.
Unforunatly I can't rember were I read this.

Upstream issue on arch: https://github.com/ente-io/ente/issues/2046
Going into a nix shell with xdg-user-dirs fixes this error, but then there's a new error:
** (ente_auth:42549): WARNING **: 17:51:53.995: libsecret_error: Failed to unlock the keyring [ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: PlatformException(Lib secret error, Failed to unlock the keyring, null, null)
On arch this is fixed by installing gnome-keyring, on NixOS going into a nix-shell with gnome-keyring doesn't fix it.
A weird thing is putting xdg-user-dirs into buildInputs doesn't fix the first error.
I'll check what keyring related NixOS options are later, thanks everyone for the help!

I had the same issue installing ente-auth.
I figured out that you have to install the xdg-user-dirs package to fix this issue.
Unforunatly I can't rember were I read this.

Upstream issue on arch: ente-io/ente#2046 Going into a nix shell with xdg-user-dirs fixes this error, but then there's a new error: ** (ente_auth:42549): WARNING **: 17:51:53.995: libsecret_error: Failed to unlock the keyring [ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: PlatformException(Lib secret error, Failed to unlock the keyring, null, null)  On arch this is fixed by installing gnome-keyring, on NixOS going into a nix-shell with gnome-keyring doesn't fix it. A weird thing is putting xdg-user-dirs into buildInputs doesn't fix the first error.
I'll check what keyring related NixOS options are later, thanks everyone for the help!

Oh ya, that issue is where I found that fix.
I'm using the kde wallet and it works fine.

The libsecret error can be fixed by using the gnome-keyring (or I assume kwallet) module, not just the package.
I think we should make a module with programs.ente-auth.enable option that does environment.systemPackages = [ ente-auth xdg-user-dirs ]; and services.gnome.gnome-keyring.enable = true;.

I think we should make a module with programs.ente-auth.enable option that does environment.systemPackages = [ ente-auth xdg-user-dirs ]; and services.gnome.gnome-keyring.enable = true;.

Not sure if that's a good idea since it'll restrict it to one particular provider, which is also the most intrusive (requires PAM integration). I think that we just need upstream to emit a clearer error message here.

Not sure if that's a good idea since it'll restrict it to one particular provider, which is also the most intrusive (requires PAM integration). I think that we just need upstream to emit a clearer error message here.

Yeah, a module for this can be intrusive and overkill, what about mentioning it in meta.longDescription or somewhere else?

I implemented a custom update script that takes care of updating the pubspec lock JSON as well as the simple-icons rev and hash. If there's anything else, let me know!

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/1993


Fix typo in services.yaml

s/varaible/variable/

Hi @mika,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

Please sign the CLA and switch target branch to dev.


Feature: update text size and color for the subscribe card

What type of PR is this? (check all applicable)

[x] 🍕 Feature
[ ] 🐛 Bug Fix
[ ] 📝 Documentation Update
[ ] 🎨 Style
[ ] 🧑💻 Code Refactor
[ ] 🔥 Performance Improvements
[ ] ✅ Test
[ ] 🤖 Build
[ ] 🔁 CI
[ ] ⏩ Revert

Description

I removed the former input and button on the subscriber card and added a new one with a better design
I also updated text size and color for the subscribe card
Related Tickets & Documents

Fixed #24
Mobile & Desktop Screenshots/Recordings

Added tests?

[ ] 👍 yes
[x] 🙅 no, because they aren't needed
[ ] 🙋 no, because I need help

Added to documentation?

[ ] 📜 README.md
[ ] 📕 storybook
[x] 🙅 no documentation needed

📸 Credits
This pull request template was created by Open Sauced 🍕

Hey @dev-phantom, Thanks for your contribution.
Since there is an issue for the footer and the input + button located in the subscribe section, I would suggest to revert those changes and only include the one mentioned in the issue #24

@TAKANOME-DEV fixed

Great, just resolve the conflicts in this PR and it will be ready to be merged

Conflict Resolved


On the sign up page, users should know why the button is not active after they input their data.

Description
The number of characters a user might want to use as password can not be predicted and so might be less than 5(what it takes to get the 'register' button active). Telling them what is wrong will improve user experience ''password should be more than 4 characters". I will like to work on this.
Screenshots

Additional information
No response
🥦 Browser
Opera
👀 Have you checked if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Contributing Guidelines?

[X] I have read the Contributing Guidelines

Are you willing to work on this issue ?
Yes I am willing to submit a PR!

A good spot - go for it !

Hi my merge was successful, it's working perfectly now.  I noticed some changes in the codebase though.

Hi my merge was successful, it's working perfectly now. I noticed some changes in the codebase though.

Which merge are you talking about ?

That one , I thought it had already merged automatically.  Thank you for allowing me to contribute,  I enjoyed it.


Interface expansion

This adds the tag_get_data function to pymoab and handles most of the tag type issues we'll run into (see #8).

Mostly some style comments, but some algorithmic ones too. Thanks!

Just finished addressing @scopatz's quite valid comments/concerns.  :)

Looks great! Thanks!


--exclude-path doesn't behave correctly with absolute paths

Here are steps to duplicate the problem:
cat <<EOF > ignorefile
*.js
**/*.js
EOF
echo "not really javascript" >> foo.js

This works as expected:
jshint --exclude-path ignorefile .

This doesn't:
jshint --exclude-path ignorefile $(pwd)

producing
/tmp/bar/foo.js: line 1, col 1, Expected an assignment or function call and instead saw an expression.
/tmp/bar/foo.js: line 1, col 4, Missing semicolon.
/tmp/bar/foo.js: line 1, col 5, Expected an assignment or function call and instead saw an expression.
/tmp/bar/foo.js: line 1, col 11, Missing semicolon.
/tmp/bar/foo.js: line 1, col 12, Expected an assignment or function call and instead saw an expression.
/tmp/bar/foo.js: line 1, col 22, Missing semicolon.

6 errors

I know this is old but I still see it happening. It makes jshint ignore quite confusing.
Can we get this fixed?


Opening cards doesnt work

since commit where i reorganized whoops

@isaaclo123 is this solved by your fixing commit?

^ iti is


Updated README to fix inconsistencies

Fixes #99

Codecov Report

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

@@           Coverage Diff           @@
##           master     #101   +/-   ##
=======================================
  Coverage   85.73%   85.73%
=======================================
  Files          43       43
  Lines        3546     3546
=======================================
  Hits         3040     3040
  Misses        506      506

Flag
Coverage Δ

#SwiftKuery
85.73% <ø> (ø)
:arrow_up:

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 c31d99d...263c2da. Read the comment docs.

Yep, you're right. Fixed.


archive: call file.Sync() after tar stream copy

Flush file-data from tarfile into disk.
Signed-off-by: Wei Fu<EMAIL_ADDRESS>

Codecov Report

Merging #3652 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master   #3652      +/-   ##
=========================================
+ Coverage   42.39%   42.4%   +0.01%
=========================================
  Files         127     127
  Lines       14048   14051       +3
=========================================
+ Hits         5955    5958       +3
  Misses       7193    7193
  Partials      900     900

Flag
Coverage Δ

#linux
45.91% <100%> (ø)
:arrow_up:

#windows
37.29% <100%> (+0.01%)
:arrow_up:

Impacted Files
Coverage Δ

archive/tar.go
48.49% <100%> (+0.31%)
:arrow_up:

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 9339104...27dfe3d. Read the comment docs.

Background: I meet the issue that the commit snapshot action is ok but the data disk has problem. Actually, the containerd already does the digest compare during applying content into snapshot. But we have no idea when the data has been flushed into disk. Maybe the machine is crash for some reason. So I add the fsync call for regular file .

other flaky case:
--- FAIL: TestImagesCreateUpdateDelete (0.10s)
    --- FAIL: TestImagesCreateUpdateDelete/ReplaceLabelsAnnotationsFieldPath (0.01s)
        images_test.go:531: timestamp for updatedat not after createdat: 2019-09-16 15:21:08.9089364 +0000 UTC <= 2019-09-16 15:21:08.9089364 +0000 UTC
FAIL
coverage: 60.1% of statements

I used vm.dirty_background_ratio and vm.dirty_ratio to increase the page cache for file data.

REF: https://www.kernel.org/doc/Documentation/sysctl/vm.txt

With more dirty page cache, the data will not be flushed into disk after open/write/close operations. Based on this, we can use echo b > /proc/sysrq-trigger to reboot system immediately without flushing dirty page cache. In this case, the committed snapshotter will lose data.
In order to simulate this case, I built one layer image named by localhost:5000/fsync:1.
root@ubuntu-xenial ~/g/s/g/f/fsync-issue# cat Dockerfile
FROM scratch

COPY . .

There are regular files, symlink, hardlink and sub folder in the layer.
root@ubuntu-xenial ~/g/s/g/f/fsync-issue# ls -alh
total 8.1M
drwxr-xr-x  3 root root 4.0K Sep 26 17:18 .
drwxr-xr-x 12 root root 4.0K Sep 26 13:11 ..
-rw-r--r--  1 root root 4.1M Sep 26 13:46 data
-rw-r--r--  1 root root   23 Sep 26 17:18 Dockerfile
-rw-r--r--  2 root root 4.1M Sep 26 16:49 hardlink-level_data
drwxr-xr-x  2 root root 4.0K Sep 26 16:49 level
lrwxrwxrwx  1 root root   10 Sep 26 16:50 symlink-level_data -> level/data
-rw-r--r--  1 root root  695 Sep 26 16:33 testing.sh
root@ubuntu-xenial ~/g/s/g/f/fsync-issue# tree
.
├── data
├── Dockerfile
├── hardlink-level_data
├── level
│   └── data
├── symlink-level_data -> level/data
└── testing.sh

1 directory, 6 files

I run containerd with master branch and fetch image data without unpacking.
My virtualbox has the default vm config as the following description.
root@ubuntu-xenial ~/g/s/g/f/fsync-issue# sysctl -a | grep dirty
vm.dirty_background_bytes = 0
vm.dirty_background_ratio = 10
vm.dirty_bytes = 0
vm.dirty_expire_centisecs = 3000
vm.dirty_ratio = 20
vm.dirty_writeback_centisecs = 500
vm.dirtytime_expire_seconds = 43200

Run testing.sh for missing data.
root@ubuntu-xenial ~/g/s/g/f/fsync-issue# bash testing.sh
there is image without unpack
ctr image ls
REF                    TYPE                                                 DIGEST                                                                  SIZE    PLATFORMS   LABELS
localhost:5000/fsync:1 application/vnd.docker.distribution.manifest.v2+json sha256:fe2adef4589a3462fc8dd2a648244e94444f4f122a3c9203638fe3aa541c3802 6.0 MiB linux/amd64 -

ctr snapshots ls
KEY PARENT KIND

make sure there is no folder in snapshots
ls -al /var/lib/containerd/io.containerd.snapshotter.v1.overlayfs/snapshots
total 8
drwx------ 2 root root 4096 Sep 26 17:19 .
drwx------ 3 root root 4096 Sep 26 14:04 ..

make more dirty pagecache
vm.dirty_background_ratio = 50
vm.dirty_ratio = 80

unpack image and use sysrq-trigger to let box reboot without filesystem sync
ctr image pull localhost:5000/fsync:1 && echo b > /proc/sysrq-trigger
localhost:5000/fsync:1:                                                           resolved       |++++++++++++++++++++++++++++++++++++++|
manifest-sha256:fe2adef4589a3462fc8dd2a648244e94444f4f122a3c9203638fe3aa541c3802: done           |++++++++++++++++++++++++++++++++++++++|
layer-sha256:947a6cea40d99c485f15cfa932250436628eedae85d5d5e83c9d71a71c0c9c31:    done           |++++++++++++++++++++++++++++++++++++++|
config-sha256:ba3bf94c90bd3095c5dabab51054ce78326097e441e83f02de63bdf2600b23a1:   done           |++++++++++++++++++++++++++++++++++++++|
elapsed: 0.1 s                                                                    total:   0.0 B (0.0 B/s)
unpacking linux/amd64 sha256:fe2adef4589a3462fc8dd2a648244e94444f4f122a3c9203638fe3aa541c3802...
done
Connection to <IP_ADDRESS> closed by remote host.
Connection to <IP_ADDRESS> closed.

After reboot, we got nothing but the snapshotter has been committed. The boltdb always calls the fdatasync for each page.
root@ubuntu-xenial /h/vagrant# ls -al /var/lib/containerd/io.containerd.snapshotter.v1.overlayfs/snapshots
total 8
drwx------ 2 root root 4096 Sep 26 17:19 .
drwx------ 3 root root 4096 Sep 26 14:04 ..
root@ubuntu-xenial /h/vagrant# ctr snapshot ls
KEY                                                                     PARENT KIND
sha256:a6e30ddc136c26c1213a97eb304817db537d98101a2125dc067d68f0645ac4f4        Committed

We meet this case in production and I was thinking that only sync dir for each layer. But the linux manpage said:

Calling fsync() does not necessarily ensure that the entry in the directory containing the file has also reached disk.  For that an explicit fsync() on a file descriptor for the directory is also needed.

The dir fsync only syncs the metadata, like subfile, into disk. Please check the following result. We can tell that we need to sync regular file after writing.
root@ubuntu-xenial /v/l/c/i/s/1/fs# ls -al /var/lib/containerd/io.containerd.snapshotter.v1.overlayfs/snapshots/14/fs
total 12
drwxr-xr-x 3 root root 4096 Sep 26 17:29 .
drwx------ 4 root root 4096 Sep 26 17:29 ..
-rw-r--r-- 1 root root    0 Sep 26 13:46 data
-rw-r--r-- 1 root root    0 Sep 26 17:18 Dockerfile
-rw-r--r-- 2 root root    0 Sep 26 16:49 hardlink-level_data
drwxr-xr-x 2 root root 4096 Sep 26 16:49 level
lrwxrwxrwx 1 root root   10 Sep 26 16:50 symlink-level_data -> level/data
-rw-r--r-- 1 root root    0 Sep 26 16:33 testing.sh

root@ubuntu-xenial /h/vagrant# tree /var/lib/containerd/io.containerd.snapshotter.v1.overlayfs/snapshots
/var/lib/containerd/io.containerd.snapshotter.v1.overlayfs/snapshots
└── 14
    ├── fs
    │   ├── data
    │   ├── Dockerfile
    │   ├── hardlink-level_data
    │   ├── level
    │   │   └── data
    │   ├── symlink-level_data -> level/data
    │   └── testing.sh
    └── work

To be honest, the change will impact performance, especially much more small files, like node_modules. It is trade-off and we also need subcommand to check each file is correct in one committed snapshotter.
@dmcgowan

I guess this is using syncfs under the hood for the dir fd?
https://linux.die.net/man/2/syncfs

I guess this is using syncfs under the hood for the dir fd?
https://linux.die.net/man/2/syncfs

@crosbymichael it seems it will sync the filesystem which containing the dir fd :joy:. might impact performance more?

ping @crosbymichael @dmcgowan

LGTM

any update?

@crosbymichael @dmcgowan I need to rethink about this patch. It brings performance issue, especially too many files in layer, like nodeJS, python application. Basically, ext4 filesystem(common use case) does buffer flushing 5 sec by default. The dirty page does rely on total memory in host. Generally, it is ok without file.Sync().
According to performance issue, I remove the 1.4 milestone and close it. Reopen to discuss if it becomes critical issue. Also I will open issue to track how to detect layer data in snapshotter is correct.

@fuweid We can adopt a verification approach to mitigate the IO overhead associated with using fs.sync to ensure file persistence on disk. During the file decompression process, we can add a setfattr to each file, indicating that the file has passed verification. If feasible, we would then directly employ the chattr +i command.
When calling ImageStatus, we can further inspect snapshotter.Stat. Within the implementation of snapshotter.Stat, we can traverse all files under the verification path, checking whether they carry an attr that denotes they've been verified.
If every file carries this attr, we can affirm the completeness of the entire filesystem. Subsequently, we can add a label within snapshots.Info to signify that this snapshot has been verified, and update the store to ensure that each snapshot is only verified once.
If a file lacks this attr, it might signify file corruption. In such a case, we can delegate to the higher-level ImageStatus to delete the snapshot and re-extract it.
As we're merely adding an attr to each file, and each snapshot is only verified once, the overhead of this approach should be minimal. I believe this strategy is feasible and merits a PR submission for validation.

I wrote a small program to test the total time taken to write 3000 files, each of 10K size:
Time taken with fsync for each file:
root@DESKTOP-C1QTOS8:/opt/hello/test# ./main
cost: 8518ms
root@DESKTOP-C1QTOS8:/opt/hello/test# ./main
cost: 8817ms
root@DESKTOP-C1QTOS8:/opt/hello/test# ./main
cost: 8448ms

Time taken without forcibly using fsync:
root@DESKTOP-C1QTOS8:/opt/hello/test# ./main
cost: 162ms
root@DESKTOP-C1QTOS8:/opt/hello/test# ./main
cost: 163ms
root@DESKTOP-C1QTOS8:/opt/hello/test# ./main
cost: 153ms

Code:
package main

import (
	"fmt"
	"math/rand"
	"os"
	"time"
)

func main() {
	rand.Seed(time.Now().UnixNano()) // Set the random seed

	content := make([]byte, 1024) // File content, size of 10k

	// Randomly fill the file content
	for j := 0; j < len(content); j++ {
		content[j] = byte(rand.Intn(256))
	}

	start := time.Now()

	for i := 1; i <= 3000; i++ {
		filename := fmt.Sprintf("file%d", i) // Filename

		// Write file
		file, err := openFile(filename, os.O_WRONLY|os.O_CREATE|os.O_TRUNC, 0644)
		if err != nil {
			fmt.Printf("open file %s failed: %v", filename, err)
			return
		}

		_, ew := file.Write(content)
		if ew != nil {
			fmt.Printf("write file %s failed: %v", filename, err)
			return
		}

		for _, op := range []func() error{file.Close} { // Asynchronous
		// for _, op := range []func() error{file.Sync, file.Close} { // Synchronous
			if opErr := op(); err == nil {
				err = opErr
			}
		}

		if err != nil {
			fmt.Printf("flush disk error: %v", err)
		}
	}

	since := time.Since(start)

	fmt.Printf("cost : %dms \n", since/1e6)
}

func openFile(name string, flag int, perm os.FileMode) (*os.File, error) {
	f, err := os.OpenFile(name, flag, perm)
	if err != nil {
		return nil, err
	}
	// Call chmod to avoid permission mask
	if err := os.Chmod(name, perm); err != nil {
		f.Close()
		return nil, err
	}
	return f, err
}


Execution failed for task ':app:validateSigningDebug'. > java.util.concurrent.ExecutionException: com.android.builder.utils.SynchronizedFile$ActionExecutionException: java.io.IOException: com.android.ide.common.signing.KeytoolException: Failed to create keystore.

i have same problem

A description would be helpful.

How was this error encountered?
Do you have steps to recreate it?
What does your environment look like?

Based on com.android.builder.utils package, my guess is that you're trying to build or run Synthea on Android????

I'm closing this issue. No explanation or details. Impossible to recreate.


Debugging web assembly on visual studio unable to inspect any value

[EDIT by guardrex to add the metadata. Please use the This page feedback button+form at the bottom of the topic to open new issues.]
After a lot of effort i managed to make the break points to hit debugging a blazor web assembly application . I did what ever the documentation said but the debugger was unable to bind the breakpoints. It seems that what is needed is the chrome debugger window to open immediately after the chrome window when I start to debug from visual studio.
But now after I managed to stop at the breakpoint it is not useful at all because i can't inspect any values no matter what their type is (int string objects arrays).
I have installed visual studio 16.6.0 preview 4.0.
Is there something I could do?

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

ID: ec2f61c7-9f51-6735-e328-3c7b495d70c3
Version Independent ID: 4d06e565-282d-6387-aa90-cb2c2986d35a
Content: Debug ASP.NET Core Blazor
Content Source: aspnetcore/blazor/debug.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

@kostasxagoraris ... We've had the 3.2 release now. If you're still having this problem, open an issue with the product unit for investigation at ...
https://github.com/dotnet/aspnetcore/issues


NULL value for local_size is ignored

If gvki encounters a NULL value for the local_size parameter, it assumes the local size to be equal to the global_size. This is incorrect.
We could change the local size to be equal to * in each dimension. It should be easy to have GPUVerify JSON interface support this, as the related command line arguments already do.

local_work_size can also be a NULL value in which case the OpenCL implementation will
determine how to be break the global work-items into appropriate work-group instances.

Looks like I made the wrong choice. Would assuming that each work item is in its own work group be sound? That means no thread can synchronise with any other thread which would probably be the worst case in terms of "racey-ness". Although I suppose  if the kernel reads get_local_id() or get_local_size() then we'd miss potentially different behaviour based on the work group size.
Perhaps just emitting that the local size is unconstrained is the best thing to do.

Would assuming that each work item is in its own work group be sound?

That would be less sound than what you're doing now, as this means no races on __local arrays can occur.

This has been implemented by

8c8e674a5317aebcab5225e564e907413b9d17b4
89b1b50167c277071a21b08b9886235efabf62d7

If the local_size is unconstrained GVKI will emit an array for local_size where each value in array is a string "*". The length of the local_size array will match the size of the global_size array.
Example:
"global_size": [ 2, 2, 2]
"local_size": ["*","*","*"];

I don't have to emit "*" however. It simplifies GPUVerify's json parser I could emit null instead.

Let me figure out what is easiest on the side of GPUVerify.

@jeroenk . You can find the current reference output here.

Yeah, let's just omit local_size. That'll result in a cleaner interface.

Except things could go horribly wrong if there was ever a bug in GVKI (or another tool) that meant that local_size was not emitted. Currently local_size must be specified but this change would make it optional.
If you think it is worth accepting the risk for an improved interface then I'll make the change to GVKI to not emit local_size when it is unconstrained.

The only think that can go horribly wrong is that this might result in false positives or time outs. I think that is acceptable.

Okay. I've implemented this in ae270ec0ee297889ce1498445d5a89e1f9a7a1d7

Thanks. I've pushed the related change to GPUVerify.

Thanks for this.
A
On 02/03/2015 16:07, Jeroen Ketema wrote:

Thanks. I've pushed the related change to GPUVerify.
—
Reply to this email directly or view it on GitHub
https://github.com/mc-imperial/gvki/issues/4#issuecomment-76739917.


Add Activity Timeline Component

Implement the Activity Timeline component according to spec: https://www.lightningdesignsystem.com/components/activity-timeline/

Removing from Google Summer of Code 2019 proposal timeline. Will follow up with additional time after Expression and Tree Grid.


Strange behaviour of cyrillic symbol using ngPattern

Hi all. In my directive I use ngPattern and regExp
/^[A-Za-z0-9._%+-]+@[A-Za-z0-9.-]+\.[A-Za-z]{2,6}$/ for some input.
When I try to tap cyrillic symbol in that input for the first time that change current email with some strange behaviour. It seems like angular tries to encode my symbol. For example:
Was<EMAIL_ADDRESS>Became<EMAIL_ADDRESS>xn--...-hdd - was added to the email.
If I try to continue tapping there is no problem and cyrillic symbols appear in the input.

@sschegolev, could you post a live demo (e.g. using CodePen, Plnkr etc) ?

I have created a Plinkr and can't see the Issue
https://plnkr.co/edit/vQjAaGV3dNMyqQDcwMY5?p=preview
everything work normally

Unfortunately, there is not much we can do if we can't reproduce the problem.
I can't think of anything that might cause what you described. ngPattern does not change the input value; it merely validates it and reports if it's valid or not.
Closing for now, but feel free to reopen when you have a minimal reproduction.


识别结果有误

葛亚,13515842218,浙江省 宁波市 镇海区 蛟川街道临江小区30幢306室,315200
{
    address: ""
    city: "宁波市"
    cityCode: "3302"
    county: "镇海区"
    countyCode: "330211"
    name: "蛟川街道临江小区30幢306室"
    phone: "13515842218"
    province: "浙江省"
    provinceCode: "33"
    zipCode: "315200"
}

反而旧的项目能正确识别https://wzc570738205.github.io/smart_parse/

新版不支持地址分开输入


Update dependencies

Update dependencies including protobuf.
Regenerate stubs.

Fixes #122

Would you be so kind as to tag a new release?

(I think only a release is going to trigger an upload to PyPi from https://github.com/KapJI/ghome-foyer-api/blob/main/.github/workflows/publish.yml )

Also see https://github.com/leikoilja/ha-google-home/issues/918#issuecomment-2455721981

@satmandu v1.1.0 is published.

many thanks!


MOB-3607: Reset payment nonce and payment view

JIRA
MOB-3607
DESCRIPTION
Reset payment nonce if user cancels the 3DS
Types of changes
What types of changes does your code introduce to the project?
Put an x in the boxes that apply

[x] Bugfix
[ ] New feature
[ ] Tech Debt
[ ] Breaking change
[ ] Documentation Update
[ ] Other: define what type of change it is

CHECKLIST
Put an x in the boxes that apply. You can also fill these out after creating the PR. This is simply a reminder of what we are going to look for before merging code.
Please review the guidelines for contributing to this repository.

[ ] All work relating to the ticket is complete (Code complete/Acceptance Criteria met)
[ ] Lint and unit tests pass locally (if appropriate)
[ ] Added tests that prove the fix is effective or that feature works (if appropriate)
[ ] Documentation (if appropriate)

SCREENSHOTS
[Add screenshots if any for changes you have made to the App/UI]

I think you need to add resetPaymentNonce in tests as well


gc.setScale()

Screen.canvas is not being resized according to gc.setScale(), although the window size is.

It has been some time, But the scale issue has been fixed.

great, i'll try it again later :)
2016-11-19 11:44 GMT-02:00 Sven Arends<EMAIL_ADDRESS>
It has been some time, But the scale issue has been fixed.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/winspeednl/LibZ/issues/1#issuecomment-261714677, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAZ7cf-Hcqi3bn56JdQxIJWX8EUGQxYkks5q_v0hgaJpZM4KgRD3
.


Nil pointer Exception if target contains a space

If the target as a name with space like "Super Record"  iOS replace the space with an "_" so the correct name for the model become "Super_Record.momd"
I have solved temporarily changing:
let infoDictionary = NSBundle.mainBundle().infoDictionary as NSDictionary?
let stackName = (infoDictionary!["CFBundleName"] as! String)
let storeName = stackName + ".sqlite"

with
let infoDictionary = NSBundle.mainBundle().infoDictionary as NSDictionary?
let stackName = (infoDictionary!["CFBundleName"] as! String).stringByReplacingOccurrencesOfString(" ", withString: "_")
let storeName = stackName + ".sqlite"

@michaelarmstrong you should considering it if you are writing the new stack.

Tackling this as part of the Stack refactor.


Bot Management reference architecture diagram

Summary

Screenshots (optional)

Documentation checklist

[ ] The documentation style guide has been adhered to.
[ ] If a larger change - such as adding a new page- an issue has been opened in relation to any incorrect or out of date information that this PR fixes.
[ ] Files which have changed name or location have been allocated redirects.

A few styling updates + content edits

Ok, all changes have been reviewed and commited.
The PR still has Claire as a reviewer and she's out till later in the month. Any chance we can approve this before she gets back?


fix(MorphTarget): mesh must be found multiple

Fixes morphing bug
re #1
Deprecation
VRMManager.findMesh has been deprecated. Use findMeshes instead.

:tada: This PR is included in version 1.4.1 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Updated Assignment 2

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
Adding code.
What did you learn from the changes you have made?
Data inspection, Data-visualization, Model fit and RMSPE.
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
When calculating the predictor variables, besides using the approach " num_predictor_variables = len (predictor_variables.columns) ", I can also use the approach " num_predictor_variables = predictor_variable.shape[1] ".
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
When I ran the code for the last questions, there was an error.  Attended the work period and fixed the error.
How were these changes tested?
Tested in Python.
A reference to a related issue in your repository (if applicable)
Checklist

[x] I can confirm that my changes are working as intended

A2: Incomplete
Q2(i) - Please use the plots from the output to categorize which variables show positive, negative and no association
Q3 - Your lm (linear model) is missing a few parameters, thus not allowing you to capture the coefficients and intercept of each respective one
Q4 - Wrong RMSE Value calculated. consider using different inputs in your parentheses () for mean_squared_error in your code

A2: Complete
Great work making the changes noted above. Everything looks perfect now!


Fixed bug in {} monad looping

Since the program loops through a subarray in this function the indexing gets offset and needs to be fixed before it gets returned.

Hmmm... sorry off by one error


plugin removes keybinding

Hello!
I have the following keymapping in my keymap.toml:
[manager]
keymap = [
     { on = [
    "d",
    "d",
  ], run = [
    "remove",
    "escape --visual --select",
  ], desc = "Move the files to the trash" },
]

prepend_keymap = [
  { on = [
    "m",
  ], run = "plugin bookmarks --args=save", desc = "Save current position as a bookmark" },
  { on = [
    "'",
  ], run = "plugin bookmarks --args=jump", desc = "Jump to a bookmark" },
  { on = [
    "b",
    "d",
  ], run = "plugin bookmarks --args=delete", desc = "Delete a bookmark" },
  { on = [
    "b",
    "d",
  ], run = "plugin bookmarks --args=delete", desc = "Delete a bookmark" },
]

but I'm not able to use dd in yazi. After commenting out
{ on = [
    "b",
    "d",
  ], run = "plugin bookmarks --args=delete", desc = "Delete a bookmark" },

I'm able to use dd again.

Hey,
I copied your keymap.toml (with the keymaps for the bookmark uncommented) and I couldn't replicate the issue.
What version of Yazi are you using? Can you put here the output of yazi --debug?

BTW you could also add the keymaps directly to the manager keymaps, without using prepend_keymap.
The prepend_keymap is more useful when you want to add new keymaps without changing the defaults, but seeing that you are already defining the full keymap you can just add this plugins keymaps there as well.
For more information you can check it here https://yazi-rs.github.io/docs/configuration/overview/#mixing.

Hey,
I copied your keymap.toml (with the keymaps for the bookmark uncommented) and I couldn't replicate the issue.
What version of Yazi are you using? Can you put here the output of yazi --debug?

Hm... that's weird. I'm using Yazi 0.3.2 (69e7c8f 2024-08-30).
yazi --debug:

Yazi
    Version: 0.3.2 (69e7c8f 2024-08-30)
    Debug  : false
    OS     : linux-x86_64 (unix)

Ya
    Version: 0.3.2 (69e7c8f 2024-08-30)

Emulator
    Emulator.via_env: ("alacritty", "")
    Emulator.via_csi: Ok(Unknown([]))
    Emulator.detect : Unknown([])

Adapter
    Adapter.matches: X11

Desktop
    XDG_SESSION_TYPE           : Some("x11")
    WAYLAND_DISPLAY            : None
    DISPLAY                    : Some(":0")
    SWAYSOCK                   : None
    HYPRLAND_INSTANCE_SIGNATURE: None
    WAYFIRE_SOCKET             : None

SSH
    shared.in_ssh_connection: false

WSL
    /proc/sys/fs/binfmt_misc/WSLInterop: false

Variables
    SHELL              : Some("/run/current-system/sw/bin/fish")
    EDITOR             : Some("hx")
    VISUAL             : None
    YAZI_FILE_ONE      : None
    YAZI_CONFIG_HOME   : None

Text Opener
    default: Some(Opener { run: "$EDITOR $@", block: true, orphan: false, desc: "$EDITOR", for_: None, spread: true })
    block  : Some(Opener { run: "$EDITOR $@", block: true, orphan: false, desc: "$EDITOR", for_: None, spread: true })

Multiplexers
    TMUX               : false
    tmux version       : No such file or directory (os error 2)
    ZELLIJ_SESSION_NAME: None
    Zellij version     : 0.40.1

Dependencies
    file             : 5.45
    ueberzugpp       : No such file or directory (os error 2)
    ffmpegthumbnailer: 2.2.2
    magick           : 7.1.1-35
    fzf              : 0.54.2
    fd               : 10.1.0
    rg               : 14.1.0
    chafa            : 1.14.2
    zoxide           : 0.9.4
    7z               : No such file or directory (os error 2)
    7zz              : 24.07
    jq               : 1.7.1

--------------------------------------------------
When reporting a bug, please also upload the `yazi.log` log file - only upload the most recent content by time.
You can find it in the "/home/tornax/.local/state/yazi" directory.

BTW you could also add the keymaps directly to the manager keymaps, without using prepend_keymap.
The prepend_keymap is more useful when you want to add new keymaps without changing the defaults, but seeing that you are already defining the full keymap you can just add this plugins keymaps there as well.
For more information you can check it here https://yazi-rs.github.io/docs/configuration/overview/#mixing.

Oh, that's good to know. Thank you!
I'll close this issue then for the time being. Maybe I'll find a way to reproduce this.


Add CI

This adds CI to the system. All three of the checks will be done in a single job in this context through multiple steps.

We need to make sure this doesn't run on merge to main yet. (That will be handled by other processes TBD later)

Heads up, similar changes to the lint-spec workflow are being merged in #2. This includes the formatting and preventing it from running in the central repo.

Roger. I'll pull them from mine and repush.

Made the changes necessary to harmonize our CI stages with the CC pipeline (namely the formatting check.)

Rebasing to include the new CI and see if it plays nicer.


Support creation of vmware hosts

Added config for creating vmware hosts through either v1 machine or v2 host.
Not done many PR's so let me know if anything has to be changed.

@golvteppe What is the intent of this PR? Meaning, where do you intend to use the generated structs? They are not directly needed by any existing Rancher code. VMWare hosts can be added as long as there is a valid docker machine driver configured in Rancher

To explain why I ask: we auto-generate all the code in this repo based off the the schemas present in the API when rancher/server is started. Since vmware is not enabled by default, this would just get deleted the next time we generate the code

@cjellick Ah, I have misunderstood where the code is coming from then :) I use the struct (just v2 version now / in host struct) in Terraform code and we create VMware hosts through Terraform. We are only using VMware as machine driver at the moment through Terraform, so kind of dependent on this struct in go-rancher. Would it be possible to activate this and add it? I think it is added in the rancher/go-rancher v2 branch already, but then again that seems outdated.
If there are other ways to trigger creation of VMware hosts through go, I can use them instead.

If there isn't already there really needs to be a generic way to use drivers without involving generating every possible driver you might want to use, because users can upload their own arbitrary drivers.
HostTemplates in 2.0 might mostly solve this already since the publicValues and secretValues are arbitrary JSON fields.

So if I understand correctly go-rancher will not be supporting creation of VMware hosts? Since the rancher provider in terraform soon will use "github.com/rancher/go-rancher/v2" as client, I am kinda stuck with that option if I want to use the offical release (I assume). I can try removing the struct and see what happens when the data is passed on to go-rancher from terraform.
It is also possible to create VMware hosts through the rancher cli (atleast v0.5.1), so I just assumed that VMware would be something that would be included in go-rancher aswell.

@vincent99 A generic way to use drivers in this specific go-rancher repo, or just with the API in general?
Because this repo isnt really intended to support that at this point. I'm not sure if this has an impact on how rancher-compose adds hosts, but I imagine it might.

Very stale. Closing. Thanks! Please open a new PR if there is still a relevant change.


UserInfo Endpoint not called

Correct me if I am wrong but it would seem that in your set up the UserInfo endpoint is not called. You have loadUserInfo: true in your Oidc.UserManager options in SecruityService.js but this has no effect because it is the UserManager instance in callback.html which deals with the signIn response. Should the loadUserInfo: true go in the callback.html UserManager instead?

you're right
var mgr = new Oidc.UserManager({userStore: new Oidc.WebStorageStateStore(), loadUserInfo: true});
Thanks

Do you have one more suggestion?

Not at the moment. It’s a very helpful project, thank you 👍

Thanks for suggestio!


Upsert creating custom functions even when a postgres unique constraint is in place

I did my best to follow the README on this one, but I'm still missing something. Running on master
From my structure.sql:
CREATE TABLE public.items (
    id bigint NOT NULL,
    jmdict_id bigint
    -- etc…

I was curious about this and it definitely looks like the gem will call this method to create the custom functions even if use_pg_native? will return true. Just emptied both methods that were creating PG functions and the gem still worked fine from the look of it.
This is important to me because my scripts don't have permission to create functions in prod so these just keep failing/rescuing out countless times and fill my log quota

@searls I'm out of town for the next week but once I'm back I'll take a look at making sure it doesn't create the function if it's not necessary.


Export doesn't work anymore

Hello 👋
Describe the bug

Commands disappeared, only "export website using previsously selected files and settings appears in command panel".
This command doesn't work, showing the following error in the console

Operating System
Windows 10
To Reproduce
Steps to reproduce the behavior:

Export a note with "export website using previsously selected files and settings appears in command panel"
Note doesn't export and the export windows "Generating HTML" doesn't load.

Screenshots

Thanks for your work. I'm looking forward to use your plugin again 🙏

Hi, thank you. This is a duplicate of #226, and will be fixed in the next update. I will close this in favor of the other issue


Leaderboard beautification

Also with christian, we downloaded this version and tested our current location, if an enemy appears and then if a score increments once the player defeats an enemy


Spiking Java fleet manager template as reactive with executor pool for Operator SDK usages

To build the Java factorized fleet manager, we are considering going for a modern Quarkus approach with default reactive stack. One of the reason for the reactive stack is for me to be confident that adding fleet managers will be low resource intensive. We already aim to deploy 3 instances at minimal (one per AZ). With the idea of deploying 10s of services off this technology, we should aim at resource reduction.
Now today the fleet manager is not an operator, but with Kubernetes Control Plane being a likely target API server in the future, it is very likely that some interaction with the Kube API will be part of the fleet manager responsibility.
Spiking a model where the fleet manager is reactive but use the executor pool for operator SDK tasks and see how simple / complicated that model is

CC @metacosm @danielezonca

One thing that I'm not clear about is how you're doing the scaling because operators are not scalable by default… We're thinking about options there but the typical way of doing things is more about high availability than scaling because basically only one operator can process events at a time (if you're not doing anything specific, which is what we're considering, but that's not, by any means, a solved issue, it's not even solved for the Go side of things).
Your operator can be deployed multiple times but there's only one leader that processes the events, the other ones are just tracking what's going on to more or less keep their local caches warm but are not processing the events so there isn't any scaling going on, just state replication so that whenever the leader goes down, another one can be elected and be ready to take over quickly instead of having to replay lots of events. At least, that's my current understanding.

Ah interesting. Though I did not get you jump from let's fix the horizontal scaling first before addressing the vertical scaling.
My understanding from your point is that for an operator deployed on multiple AZs, then only one will be active. But it's then pretty important that it scales to the demands more easily as a single instance, hence reactive FTW. Makes sense?

Well, to truly scale, I guess we need both, and indeed, I wasn't quite clear in my explanation…
The thing I'm not clear about, though, is how are clusters partitioned across AZs (I assume that refers to Availability Zones)? If there's an effective separation of events across AZs, i.e. no chance that a cluster event in one AZ will ever target a resource in another AZ (because of namespaces or just because clusters are completely independent to start with) then, indeed, you can run one operator per AZ.
The current pattern is that for a set of events that might affect all or part of a cluster, there should ever be only one controller dealing with these events. If you can partition your cluster and configure your operator to watch events on that "closed-world" partition then it's fine to run several operators, each configured to deal with such isolated partitions. For example, it's perfectly fine to deploy an operator instance per namespace when that operator will only ever deal with resources in that particular namespace.
That said, I'd argue that if your controller is written in such a way that it does a reconcile-the-world each time it reconciles thing then it should be possible to have several instances work concurrently. It's just a lot more complex to reason about because you have to assume that things your controller observes might be changed by another instance at the same time. It should work OK when only dealing with Kubernetes resources (because after all, your controller should already assume that kubernetes resources might change from underneath it) but it's quite tricky when provisioning external resources who don't often work with the same philosophy…

AZ stands for Availability Zones and are different data centers close to one another (in latency). For Kubernetes, it's just a cluster than happens to run on these 3 data centers as one (with some anti affinity to spread it). So the events won't be sharded by AZ. The goal is to be able to lose one or two AZ but still remain operational.

In that case, right now, only the leader election mechanism would be recommended, i.e. only one operator instance can be actively processing the events if your operator is listening cluster-wide. It really depends on what the operator is supposed to be doing. Like I said, if one instance can have total control over a namespace and doesn't need to modify cluster-wide shared resources, then you can deploy one operator instance per namespace without issue.

@emmanuelbernard @metacosm thanks for the discussion. I've been following it but I am having some difficulties understanding:  Why are we talking about operators for a fleet-manager? Are fleet managers ("an API handling fleet management logic") eventually going to be operators too?
Or it is more like there are some specific concepts in the Java Operator SDK we'd lke to re-use (e.g reconciliation)

Spiking a model where the fleet manager is reactive but use the executor pool for operator SDK tasks and see how simple / complicated that model is

+1

In that case, right now, only the leader election mechanism would be recommended

On the reconciliation part, if it weren't for https://github.com/quarkusio/quarkus/issues/10716, we could have re-used the quartz extension and have leader election for free via clustered jobs. The Quartz extensions needs a blocking database connection and it is not possible to run Hibernate ORM (i.e. Blocking) and Hibernate Reactive side by side.

That's actually what I don't understand either since the current implementation is not an operator as far as I'm aware but I don't know much about the current architecture or why a switch to using JOSDK is considered… I just provided some info about the constraints as they are for scaling operators.

I'll ask a more basic question - have we decided to switch from go to java for the factorized fleet manager?

I suspect yes when Kubernetes Control Plane comes to fruition.

Hmm, that's another thing I don't understand… There's already a control plane in Kubernetes so I'm sure you're referring to something different but, lacking context, this whole conversation is quite confusing to me. Are you referring to https://github.com/kcp-dev/kcp by any chance?

@metacosm
Fleet manager is handling a fleet of Kafka instances which are deployed on a fleet of OpenShift Dedicated (OSD) clusters. It does things like OSD terraforming, dynamic scaling, pushing the custom resources to the specific OSD cluster that will receive a Kafka instance etc. Fleet Manager is also exposing a API for a user or a UI to request creations of Kafka instances.
In the future, we want to converge on the notion of a Kubernetes Control Plane which is a  Kubernetes server managing a fleet of other Kubernetes and deploying custom resources accordingly. In that future, the fleet manager will do less work but will receive user requests as a logical custom resource from KCP and do some transformation as well sa a bunch of business validations.
In that context, you can see how the future fleet manager looks a lot more like an Operator that interacts with the KCP API and  the KCP API is the kubernetes server API with the concept of custom resource and all (it's a Kube without pods).
I hope it make more sense.

Thanks @pmuir for the title improvement.

Discussing the subject with @maxandersen and @n1hility, they were arguing against too much of a premature optimisation. We can use RESTEasy reactive and have the route bee implemented in classic blocking fashion and use Hibernate ORM classic. Even that model brings good memory and resource efficiency. We can see the non blocking Hibernate ORM if the load turns out to be a problem.
That resolves the problem the spike was trying to address. I’m thinking that we can then close that issue and make the decision to use the architecture approach mentioned here for the Java template. How does that work for you @danielezonca and @r00ta ?

How does that work for you @danielezonca and @r00ta ?

Fine for me, this is already how we implemented the fleet manager in OpenBridge.

Fine for me too. We probably need to wait KCP to revisit/reconsider it

What's the timeframe for KCP?

Some projects using by by mid year.

How does that work for you @danielezonca and @r00ta ?

Fine for me, this is already how we implemented the fleet manager in OpenBridge.

@r00ta that's cool. Can I get access to the repo?

Currently the code is this single repo, folder manager.
Note: this component is not covering yet AMS and other aspects too

Thank you @danielezonca, will have a look.

Related to https://github.com/bf2fc6cc711aee1a0c2a/ffm-project/issues/5


Error: Cannot find module 'jsdom/level2/core'

When I attempt to require zombie in a simple script, I get the error below.  I've included a listing of the current installed npm packages.  Any idea what is causing this?
Error: Cannot find module 'jsdom/level2/core'
at resolveModuleFilename (node.js:280:15)
at loadModule (node.js:242:22)
at require (node.js:306:16)
at EventEmitter.HTML5Parser (/usr/local/lib/node/.npm/html5/0.2.12/package/lib/html5/parser.js:31:12)
at [object Object].appendHtmlToElement (/usr/local/lib/node/.npm/jsdom/0.1.20/package/lib/jsdom/browser/htmltodom.js:86:15)
at Object.innerHTML (/usr/local/lib/node/.npm/jsdom/0.1.20/package/lib/jsdom/browser/index.js:341:27)
at Object.jsdom (/usr/local/lib/node/.npm/jsdom/0.1.20/package/lib/jsdom.js:25:17)
at History. (/usr/local/lib/node/.npm/zombie/0.8.8/package/lib/zombie/history.js:63:24)
at /usr/local/lib/node/.npm/zombie/0.8.8/package/lib/zombie/history.js:2:61
at History. (/usr/local/lib/node/.npm/zombie/0.8.8/package/lib/zombie/history.js:30:16)
1 awt@DEV 2 ~/projects/md2> npm ls installed
npm info it worked if it ends with ok
npm info using<EMAIL_ADDRESS>npm info using<EMAIL_ADDRESS><EMAIL_ADDRESS>         A C++ module for node-js that does base64 encoding and decoding.    =pkrumins active installed latest remote base conversion base64 base64 encode base64 d
<EMAIL_ADDRESS>  Unfancy JavaScript    =jashkenas active installed latest remote stable javascript language coffeescript compiler
<EMAIL_ADDRESS>      Markup as CoffeeScript.    =mauricemach active installed latest remote template view coffeescript
<EMAIL_ADDRESS>        High performance middleware framework    =creationix =tjholowaychuk active installed remote
<EMAIL_ADDRESS>          CSS Object Model implementation and CSS parser    =nv active installed latest remote CSS CSSOM parser styleSheet
<EMAIL_ADDRESS>        Sinatra inspired web development framework    =tjholowaychuk active installed latest remote framework sinatra web rest restful
<EMAIL_ADDRESS>          HTML5 HTML parser, including support for SVG and MathML foreign content    =aredridel installed remote
<EMAIL_ADDRESS>         HTML5 HTML parser, including support for SVG and MathML foreign content    =aredridel active installed latest remote
<EMAIL_ADDRESS>     Forgiving HTML/XML/RSS Parser in JS for both Node and Browsers    =tautologistics active installed latest remote
<EMAIL_ADDRESS>           Jade template engine    =tjholowaychuk active installed latest remote
<EMAIL_ADDRESS>         jQuery: The Write Less, Do More, JavaScript Library    =coolaj86 active installed latest remote util dom jquery
<EMAIL_ADDRESS>         CommonJS implementation of the DOM intended to be platform independent and as minimal/light as possible while completely adhering to the w3c DOM specifica
<EMAIL_ADDRESS>         CommonJS implementation of the DOM intended to be platform independent and as minimal/light as possible while completely adhering to the w3c DOM specifica
<EMAIL_ADDRESS>           A super simple utility library for dealing with mime-types    =broofa active installed latest remote util mime
<EMAIL_ADDRESS> Command line mjsunit runner which provides an easy way to hook into mjsunit and start running tests immediately..    =tmpvar active installed latest remot
<EMAIL_ADDRESS>       Easy unit testing for node.js and the browser.    =caolan active installed latest remote
<EMAIL_ADDRESS>         A package manager for node    =isaacs active installed remote package manager modules install package.json
<EMAIL_ADDRESS>       Simplified HTTP request method.    =mikeal active installed latest remote
<EMAIL_ADDRESS>           Syntactically Awesome Stylesheets (compiles to css)    =tjholowaychuk active installed latest remote sass template css view
<EMAIL_ADDRESS>      The cross-browser WebSocket    =rauchg =Tim-Smart active installed remote
<EMAIL_ADDRESS>          Web development, cut-the-crap style.    =mauricemach active installed latest remote framework websockets coffeescript
<EMAIL_ADDRESS>         Insanely fast, full-stack, headless browser testing using Node.js    =assaf active installed latest remote test spec headless full-stack
npm ok

Reopen if it's still a problem! jsdom's API has changed in the mean time, and we're using that now.


[Event Request] Codeunit 802 "Online Map Management" - OnSetupDefaultBeforeInsertParam

Hi, would it be possible to get an event OnSetupDefaultBeforeInsertParam in Codeunit 802 (Online Map Management), like this:
procedure SetupDefault()
    var
        OnlineMapSetup: Record "Online Map Setup";
        OnlineMapParameterSetup: Record "Online Map Parameter Setup";
        IsHandled: Boolean;
    begin
        OnlineMapSetup.DeleteAll();
        OnlineMapParameterSetup.DeleteAll();
        OnSetupDefaultBeforeInsertParam(IsHandled);
        if not IsHandled then
          InsertParam(
            'BING',
            Text015,
            'http://bing.com/maps/default.aspx?where1={1}+{2}+{6}&v=2&mkt={7}',
            'http://bing.com/maps/default.aspx?rtp=adr.{1}+{2}+{6}~adr.{1}+{2}+{6}&v=2&mkt={7}&rtop={9}~0~0',
            'http://bing.com/maps/default.aspx?rtp=pos.{10}_{11}~adr.{1}+{2}+{6}&v=2&mkt={7}&rtop={9}~0~0',
            false, '', '0,1',
            'http://go.microsoft.com/fwlink/?LinkId=519372');
        OnlineMapSetup."Map Parameter Setup Code" := 'BING';
        OnlineMapSetup.Insert();
    end;

[IntegrationEvent(false, false)]
local procedure OnSetupDefaultBeforeInsertParam(var IsHandled: Boolean)
begin
end;

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


feat: Refine mutex tree definition, add methods

Another fairly easy PR to review. The bulk is coming next...


Better handle icons and checkboxes with separators in PopupMenu

Currently, icons and checkboxes are handled very weirdly when in a separator item. This PR fixes the two issues with it:

The checkbox state is ignored in separator mode, as they can't be selected, therefore pointless.
Icons are now drawn together with the text.

Before:
After:

Thanks!


ds4windows

Is there any possibility that that ds4windows is also available for mac users?

This Version from Jays2Kings will no longer be updated.
Please use this Version: https://github.com/Ryochan7/DS4Windows
Maybe they can help you there


Cross-platform support

Resolved #3
Tested all available images of windows & ubuntu. As discussed in the issue macOS cannot be supported at the moment until Google Chrome is added to the image.
Updated README.md to reflect supported platforms
Tested on separate branch. See results here

Thanks again for your work! Merging


chore: Update CODEOWNERS to use committers team

:tada: This PR is included in version 3.3.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Error: Resource not accessible by integration

In https://github.com/www-learn-study/saraswati.learn.study/actions/runs/6727753248/job/18286039868 I have run into this:
Run marocchino/sticky-pull-request-comment@v2
  with:
    header: pr-preview
    message: [PR Preview Action](https://github.com/rossjrw/pr-preview-action) v1.4.4
  :---:
  :rocket: Deployed preview to https://www-learn-study.github.io/previews/pr-preview/pr-43/
  on branch [`gh-pages`](https://github.com/www-learn-study/saraswati.learn.study/tree/gh-pages) at 2023-11-02 03:00 UTC
    append: false
    recreate: false
    delete: false
    only_create: false
    only_update: false
    hide_details: false
    hide: false
    hide_and_recreate: false
    hide_classify: OUTDATED
    ignore_empty: false
    skip_unchanged: false
    follow_symbolic_links: false
    GITHUB_TOKEN: ***
  env:
    action: deploy
    targetdir: pr-preview/pr-43
    pr: 43
    pagesurl: www-learn-study.github.io/previews
    emptydir: /tmp/tmp.NX01XUKSxB
    datetime: [202](https://github.com/www-learn-study/saraswati.learn.study/actions/runs/6727753248/job/18286039868#step:4:208)3-11-02 03:00 UTC
    actionref: v1
    actionrepo: rossjrw/pr-preview-action
    deployrepo: www-learn-study/previews
    token: PREVIEW_TOKEN
    action_version: v1.4.4
    deployment_status: success
Error: Resource not accessible by integration

I've figured out how to fix it (permissions: ...), and will post a PR with a proposed docs clarification.

I thought I fixed this, but am seeing this again, see https://github.com/www-learn-study/saraswati.learn.study/issues/50#issuecomment-1790218847.


DBZ-8208 Use the Loggings utility class

https://issues.redhat.com/browse/DBZ-8208

@Naros Applied, thanks


Fix issue 1319 - return correct code on bad / invalid requests

Description
Instead of returning Forbidden. returning the correct code
Related issues
Addresses [issue #1319].

/azp run

/azp run

/azp run

/azp run

@eladiw, the problem is that the spec says we should return 403 in these cases. I agree it shouldn't be this way though.

A HTTP status code of 403 signifies that the server refused to perform the search, while other 4xx and 5xx codes signify that some sort of error has occurred

@brendankowitz, what do you think?

@eladiw, the problem is that the spec says we should return 403 in these cases. I agree it shouldn't be this way though.

A HTTP status code of 403 signifies that the server refused to perform the search, while other 4xx and 5xx codes signify that some sort of error has occurred

@brendankowitz, what do you think?

If we also don't return the correct status code, we should also fix this, but it doesn't have to be in this PR since its just changing the issue types.

I'll merge this as a partial fix for #1319. I've updating but bug with additional information about changing the status code.


How to close browser tab?

It looks that there is no way to close the browser tab. I think this is required to implement the OAuth flow.

I think we need to use dismissViewController method. See http://strawberrycode.com/blog/sfsafariviewcontroller-and-oauth-the-instagram-example/

This isn't something you can reliably do across platforms and browsers. Dismissing the tab isn't required for OAuth, all that matters is that the response is ultimately delivered to the requesting app.

Dismissing the tab isn't required for OAuth, all that matters is that the response is ultimately delivered to the requesting app.

True but consider scenario:

Bob opens Cordova mobile app on his iPhone and taps 'Sign in with Google' button. Browser tab opens.
Bob enters his Gmail email and password. He approves requested access to his profile.
Browser tab redirects to custom URL schema. The app receives this request. It cannot respond with HTML (because it is Cordova app). So to Bob it looks like nothing has happened.

Bob is frustrated: to him it looks like the app is not working, even though if he taps 'Done' button he would sign in to the app.

I'm looking for a way to achieve this on Android.
My situation is as follows:
Android behaves differently than iOS when redirecting to a custom URL schema. iOS handles this redirect in the same webview as the one initiating the OAuth flow. Android, however, opens a new webview within the same app to deal with the redirect.
The solution for this seems simple at first, just close the old webview using navigator.app.exitApp();. But when one does that directly after opening a Browser Tab, the entire app exits. You need to wait for the new webview to be loaded before you can safely exit the old webview. As there is no way of directly communicating between these webviews, I'm currently abusing localStorage to achieve this.
There is one problem with this approach: the opened Browser Tab will stay open in the background with no way to close it.
Any ideas?

@mjwwit I believe you need to add  to your config.xml

@gregor-srdic Ok, do you know how to configure this? I've been reading https://cordova.apache.org/docs/en/latest/config_ref/ but I can't find anything that looks like what I need.

@mjwwit Google achieves closing the extra browser WebView with Firebase OAuth as documented here: https://firebase.google.com/docs/auth/web/cordova
I'm trying to figure out how to do it the same way. However, it's in a different project. https://github.com/firebase/firebase-js-sdk/issues/102  I will post my findings here once it's found.

@gregor-srdic <preference name="AndroidLaunchMode" value="singleInstance" /> doesn't stop a BrowserTab from opening outside of the app... it only prevents another instance of the app being created when a deep link is entered.

@gregor-srdic & @mjwwit , we were very close.  <preference name="AndroidLaunchMode" value="singleTask" /> stops the new window from opening. And should automatically close (at least it does with custom URL scheme).

Looks like @washowasho found the solution. The singleTask launch mode solves the problem of the browser tab staying open on Android. And since the iOS implementation already has a working close() method, I think this whole issue can be closed.

@washowasho Do you have any tips on how to close the browser tab on iOS? Feel like I've tried everything at this point:

I redirect the tab to a page I have control over. On that page I've tried doing:

window.location = 'my.scheme://'
window.location.replace('my.scheme://')

Instead of redirecting to a page, I've tried redirecting to the scheme (in Express): response.redirect('my.scheme://')

None of the above closes the tab on iOS. Pretty much all of them closes the tab on Android.
Hope you have a pointer or two.

Try this. It works for me using custom URL scheme.
<!DOCTYPE html>

Thanks, that did something. But the user is presented with a small popup asking if he wants to open in our app, which, in an OAuth flow, is less than desired.
Anyway thank you for your time, I'll go to plan B, which is use this for Android, and cordova-plugin-safariviewcontroller for iOS.

Hi! Any new about this?

Try this. It works for me using custom URL scheme.
<!DOCTYPE html>

Washowasho's solution works for me. Here is my working code for ASP.Net Core 3.1. It redirects your request with query string to the same page immediately after the page loads. It means the window.close() works because the location is navigated by javascript.
`if(Request.Query.ContainsKey("reload"))
{
if(Request.Query["reload"]=="false")
{
return Content("Hello my little ponies <button onclick="window.close();">Close this tab", "text/html");
}
}
var queryString = Request.QueryString.Value.Replace("?", "");
return Content("<!DOCTYPE html>", "text/html");`


Illumina Dragen module test data

Adding test data for a tumor/normal variant calling pipeline, versions with deduplication enabled and disabled. More types of pipelines to come.

Thanks!


Fonts missing in build

I'm using RicherTextLabel for a heading:

But when I build for windows the font is missing

The other fonts used in the project are working, and the only difference is these are set as the "default font" in the projects main theme file.
Am I missing some step to ensure the font is properly bundled in the build?

I've also had this issue, windows build resulted in many instances to have default font face

I've just swapped the font to the one that works in other normal rich text labels and it's also not working in the build, as far as I can tell custom fonts don't work at all in the RicherTextLabel when building for windows. I've fiddled with some settings but nothing is helping.

Are any errors getting thrown out in the debug console?

No errors that I can see
On Thu, 5 Dec 2024, 6:41 pm Rhys Newell, @.***> wrote:

Are any errors getting thrown out in the debug console?
—
Reply to this email directly, view it on GitHub
https://github.com/chairfull/GodotRichTextLabel2/issues/18#issuecomment-2519631857,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD3YOMWCZEHIIPJV2CJAHGD2EAGVDAVCNFSM6AAAAABTBWATJSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMJZGYZTCOBVG4
.
You are receiving this because you authored the thread.Message ID:
@.***>

This code here:
https://github.com/chairfull/GodotRichTextLabel2/blob/main/addons/richtext2/font_helper.gd#L108
Seems to get an empty string for normal_font_path in the build, but in the editor when I run it it has the expected path res://assets/fonts/Kaph-Regular.ttf. I will keep digging a bit.

It's the strangest thing, I've tried clearing the font cache with helped with issues in editor/run mode, but the build still refuses to use this font. What's weirder now is the top label in this screenshot doesn't even reference the font it's using. This is an older font, nothing else uses it, it's not a fallback or anything.

I'm at a loss, I may have to go back to standard RichTextLabels

So from what I can tell this is related to font_auto_setup being true, even if I uncheck the box, which causes the FontHelper.set_fonts method to run always, even in the build.
https://github.com/chairfull/GodotRichTextLabel2/blob/5e88a9e8be63334a4df0911280017a6dbc6e69aa/addons/richtext2/richer_text_label.gd#L406-L408
So I think it's a two part problem:

font_auto_setup isn't respecting the setting in the UI and always uses the default (true)
FontHelper.set_fonts fails to find the fonts in the build version for some reason

I don't know godot well enough to offer a fix via a PR, my solution is to set font_auto_setup to false in the code and leave it off. Hopefully I've been helpful!


interface error (computing)

There are some smart contract functions that do not have a return value and defining the interface will result in an error. There are also cases where the same function name has a different return value.

I am not following @SolExpDB. Do you think the solidity compiler error is wrong or that the vscode extension has autocompleted or quickfixed in a way that generates a compiler error?

@kanej I think this is a bug in the vscode extension, some functions with the same name and arguments but different return values are not buggy. But there are some functions that give an error

Solidity error 1686 is coming from solc, this issue is part of the Solidity language.


Fix wrong case in documentation of -CRLfile option

The "-CRLfile" option of verify(1) was incorrectly spelled as "-crlfile", but the options are case-sensitive. This PR updates the errant documentation.

Pushed, thanks.


[REQUEST] Button To Grab WordList From Clipboard

title ^

No sorry, it takes 5 seconds to create a file, paste what you want and drag&drop it inside OB.


Responsive layout resize - always triggers layoutUpdated

Browser: Chrome
Vue version: 2.6.12
vue-grid-layout version: 2.3.12
@layout-updated event is triggered at every resize event, not only on resizeend, when responsive layout is enabled.
Shouldn't there be a check to stop this event being triggered at every resize? And make sure is triggered only at resizeend?
Or maybe separate the events resizeWidgets and resizeWindows?

Related? https://github.com/jbaysolutions/vue-grid-layout/issues/21


Add Code Coverage Script

It would be good to add a code coverage report script that can be run locally and in CI. With things ever changing before v1, it would be helpful to ensuring a) we have necessary tests in place and b) there is no lingering dead code.

Uh... done. 😅


[DR] Near-axis magnetic field calculation from efit files

This change concerns the magnetic field calculation on axis when reading efit files. Before this change, we had issues because we were not treating the axis extra specially. The field equations are as follows.
$$B_R = \frac{1}{R} \frac{\partial \psi}{\partial Z} = \frac{\partial}{\partial Z} \left( \frac{\psi}{R} \right),$$
$$B_Z = - \frac{1}{R} \frac{\partial \psi}{\partial R} = - \frac{\partial}{\partial R} \left( \frac{\psi}{R} \right) - \frac{\psi}{R^2}.$$
The last equality is what Gkeyll does under the hood. The workflow goes that from the efit file, psi is read in. This is defined everywhere, including at $R=0$. Then the quantities are computed of $\psi/R$ and $\psi/R^2$. These make taking the derivatives easier since the part of the code that calculates the magnetic field does not know $R$. This breaks down at $R=0$ because we have a divide-by-zero error. Instead, we should take the limit of $B$ as $R \to 0$ and use L’Hopital’s rule to evaluate the limits.
$$B_R = \lim_{R \to 0} \frac{1}{R} \frac{\partial \psi}{\partial Z}  = \lim_{R \to 0} \frac{\partial^2 \psi}{\partial Z \partial R},$$
$$B_Z = - \lim_{R \to 0} \frac{1}{R} \frac{\partial \psi}{\partial R} = - \lim_{R \to 0} \frac{\partial^2 \psi}{\partial R^2}.$$
The modification activates special kernels to compute these second derivatives instead for when the values of B are returned as NaN due to the divide by zero error in calculating $\psi / R$ from the efit file. Using these near-axis kernels for the nearest index to R=0 is not necessarily enough because the computational grid does not know R directly, mainly when calculating the magnetic field. We will treat the cells extra specially using the near-axis kernels only after the far-axis kernels fail, which always results in a NaN in the B field. These NaNs in the B field occur due to NaNs in the stencil of $\psi/R$, but there are so many components that go into this stencil that it is difficult to check. The NaNs accumulate in the magnetic field's magnitude, which can be checked for NaNs in a single line of code.

You can't check for NaNs in Gkeyll at all. This is impossible. Also, it is a terrible idea to check for NaNs (even if you can) to activate "special kernels". Please rethink this issue properly and figure out a robust way of computing the geometry as we get close to the axis.

We do not have a value of R when the magnetic field is computed. The code pre-computes $\psi/R$ and $\psi/R^2$ when the geometry is read and then these quantities are used in the magnetic field calculation. Because these quantities are projected onto a DG basis at the computational nodes, we loose the information about the efit file's grid. We need to know if we are in the cell closest to $R=0$ on the efit file grid. Perhaps we can store another array to keep track of this.

We don't need to check for Nans or anything like that. When we call the kernel, we know the value of R at the cell boundaries. We can call the special kernel when we are at the axis.

Right. We do know the value of R, but we don't know which cell from the efit file this is in. We can store the efit file index for r in a gkyl array and check if this index is less than 1 anywhere to compute the special kernels

I am very confused why you need the index. You just need to know the value of R. You know the value of R inside the kernel. You only call this kernel once. Just check inside the kernel if R is at the desired tolerance for switching to a different means of computing B (just like how in the kernel I sent you, I know the value of the density and I just tell it to use Lax fluxes if the density is small).

I get what you mean. We can define a tolerance elsewhere for R at the boundary. One issue is that this tolerance is not that small and it does depend on what efit file you are using

We will need a different tolerance for each different efit file depending on their resolution. Each will go near the axis, but to different extents. We can define something universal, but not efit files are the same. Since this is really just a mirror thing, I don't mind this. After some testing, I see how it would work and it's straightforward, but it's not universal therefore not robust

This should not be dependent on efit file but delta R only. So you can choose a tolerance to be a small fraction of delta R. But, in any case, the problem here is still not solved. How will you ensure that Bz at R=0 is continuous?

What is delta R? Is this the cell size in the efit file? Then this is dependent on efit file resolution. Is there some other issue with continuity other than ensuring that
$$\frac{\partial B_Z}{\partial R} \bigg\rvert_{R=0} = 0$$ ?

I did some testing and it seems like the efit cell size is already in this part of the code through the physical RZ grid. We can read the cell sizes there and determine if the special kernels need to be called from this. I checked and this is the correct logic for high resolution grids and seems to be placed along the correct boundary.
https://github.com/ammarhakim/gkylzero/blob/d83f7a0e25ce04e506e2ff30203fefb1eac01fca/zero/calc_bmag.c#L180


React-native-web support

Hello,
First of all thank you for this package, it's really clever and helpful.
I was wondering if it would be hard to add react-native-web support in it ?
For a quick context on why it would be awesome, we are using react-native and react-native-web and allowing react-native-web inside would allow us for example to reuse all of our already made components inside the webview

That's sounds interesting.
If rnrb.preact is true, this library converts react imports to preact imports here:
https://github.com/inokawa/react-native-react-bridge/blob/4d9a19b2ca119c42ab97a1ebbaba9ae468f33eb0/src/plugin/transformer.js#L16C20-L34
I'm not sure what level of support is the best, but adding rnrb.reactNativeWeb like option and converting react-native imports under web entry to react-native-web may work as you expected.

Will try it at my work and if it works will make a pull request for it, thanks for the headups ! 👍

That sounds interesting.
If rnrb.preact is true, this library converts react imports to preact imports here: https://github.com/inokawa/react-native-react-bridge/blob/4d9a19b2ca119c42ab97a1ebbaba9ae468f33eb0/src/plugin/transformer.js#L16C20-L34
I'm not sure what level of support is the best, but adding rnrb.reactNativeWeb like option and converting react-native imports under web entry to react-native-web may work as you expected.

Any Update on this i am trying to get react-native-web to work via the above suggestion but no luck so far

Just got it to work by using import "react-native-web" directly from the webapp.js file


Add chunkGroups option to stats config docs

This option is picked up here:
https://github.com/webpack/webpack/blob/4de3ce05/lib/Stats.js#L145-L148
And used here:
https://github.com/webpack/webpack/blob/4de3ce05/lib/Stats.js#L474-L476
This controls the addition of namedChunkGroups in the stats output.

Thanks!


Fast forward release-4.12 to main

What this PR does / why we need it:
Update release-4.12 branch with the latest from main
Reverts https://github.com/openshift/hypershift/commit/9eb0b946990bc84a341962741d3fadfafc4e4bfa since that's a problem only in 4.13.

/retest-required

Looks like the e2e test timed out. Hopefully it's a temporary infrastructure issue.

/test e2e-aws

/lgtm


⚠ Bofrost.es has degraded performance

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

HTTP code: 200
Response time: 1840 ms

Resolved: Bofrost.es performance has improved in 4440fbf after 7 minutes.


Markdown Not Resolving

Markdown is not resolving properly in a few places on this page.

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

ID: ea85ad74-5055-ec55-412e-187405524050
Version Independent ID: ea85ad74-5055-ec55-412e-187405524050
Content: Unity Catalog privileges and securable objects - Azure Databricks
Content Source: databricks/data-governance/unity-catalog/manage-privileges/privileges.md
Service: azure-databricks
GitHub Login: @mssaperla
Microsoft Alias: saperla

@jack-toussaint
Thanks for your feedback! We will investigate and update as appropriate.

Thanks for providing feedback that helps improve our documentation. We've created an internal work item (DOC-13579) to address your feedback. The timeline for resolution varies based on resourcing.
#please-close


Add analytics

Status :
Ready

Description :
Add google analytics.

Codecov Report

Merging #270 into development will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff              @@
##           development     #270   +/-   ##
============================================
  Coverage        93.09%   93.09%
============================================
  Files               26       26
  Lines              855      855
============================================
  Hits               796      796
  Misses              59       59

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 e335178...e5ff078. Read the comment docs.


Building instructions for Windows invalid

Hello! Thank you! I try to build the windows exe but nothing work! once the ddns-updater.exe is build and the folder data is create also ... Nothing appear!
Do you have a tuto step by step please?
On the main page of the project you provide this command: go build -o ddns-updater.exe cmd/main.go but into cmd we don't have main.go file but 2 folders def and svc and into this both folder there is an main.go file
Originally posted by @maxihoster in https://github.com/Plaenkler/DDNS-Updater/issues/15#issuecomment-2021879386

The README.md does not yet contain updated instructions for using the DDNS-Client under Windows. Building instructions for the normal executable and the Windows service should be written.


How to allow users to start using the Company Communicator bot and all of its features?

Hi Team,
Can specific groups of users be defined as authors to make them Power Users or Admins of the Company Communicator?
While the rest of my users in the company should be able to access the Company Communicator App after allowing the Enterprise Application like below?
Enterprise Application | Properties
Enabled for users to sign-in? YES
Assignment required? NO
Visible to users? YES
Enterprise Application | Users and groups
User1
User2
...
UserX
Can you confirm if this is the correct method to use Company Communicator?
Thank you.

@gsv022 ,
Where can I see or get into that page to enter the UPN for the users that will be the author of this application ?

@albert-widjaja ,
Here is the place where UPN can be updated.

Go to resource group-->Click on app service/web app-->Goto configuration section(old)/environment variables(new)-->add/remove 'n' number of UPN within AuthorizedCreatorUPN's.

Separate each UPN with semicolon(;)

@gsv022 ,
Thank you, can I use the security group as the input rather than the user name in the above screenshot?

Security group won't work here


Timeouts on mac iOS devicelab tests

These two tests closed the framework tree by timing out:

https://ci.chromium.org/ui/p/flutter/builders/prod/Mac_ios complex_layout_ios__start_up/9174/overview
https://ci.chromium.org/ui/p/flutter/builders/prod/Mac_ios flutter_gallery__transition_perf_e2e_impeller_ios/2664/overview

Subsequent re-runs passed.

Marking as dup of https://github.com/flutter/flutter/issues/121231, https://github.com/flutter/flutter/issues/120808


Session Pool growing out of bound

My messaging application is a stand-alone spring boot application. I am maintaining an apache commons pool of smpp sessions to be kept alive and used when needed. But the pool starts to grow beyond its max limit. I have explained my application design below. Any help/suggestion would be greatly appreciated.
Current Design
I have an object pool of type ZdSmppSession (Apache Commons Pool), with pool size set to 40
Each ZdSmppSession has the following
1. A sendMessage method
2. A private variable of type com.cloudhopper.smpp.SmppSession
3. A PostConstruct method
The Post Construct method does the following
1. initiates a handshake and initialize the private variable com.cloudhopper.smpp.SmppSession
2. schedules an enquire link to be sent every 60 secs
I have about 120 worker threads which does the following
1. Read message from rmq
2. Get a ZdSmppSession object from pool
3. Calls sendMessage on the object
4. Release it back to pool
So when a ZdSmppSession object is created in the pool it is ready to send messages
When ZdSmppSession is released back to pool it keeps the connection alive by sending enquire link every 60 secs
Issue
The issue I am facing is that the pool grows beyond its maximum limit, even though the max limit is set to 40.
An observation I have made is that this issue doesnt happen if I remove the actual message sending line : com.cloudhopper.smpp.SmppSession.submit(message, SMPP_TIMEOUT) inside ZdSmppSession.sendMessage() and replace it with Thread.sleep(50)

While I'd like to help -- it seems like this is more of a question with Apache commons pool -- why it isn't limiting your max pool size.
What are you doing with async events that are triggered (e.g. session dies)?  Are you removing removing those from the pool?

Soon after constructing the session object I add a scheduler to push an enquire_link every second. This ensures that the session is kept alive when its unused in the pool. In addition to that, before sending a message the session first checks if its state is BIND, and if it is not then I first trigger a bind request on that session. In short, once a session object is created it is never removed.

I still think this is a question for Apache commons pool.  If its not honoring the pool size max -- you'll need to figure out under what cases that library would grow beyond what you'd expect.  I'm going to close this issue for now.


tests: net: socket: misc: move to new ztest API

Move net/socket/misc tests to use new ztest API
TEST=twister -T tests/net/socket/misc
Signed-off-by: Hu Zhenyu<EMAIL_ADDRESS>

The old one is merged. Will provide a new PR to change the "_test_suite" problem


Fixing image pull

This PR is a sequel of #676. It closes #671 and closes #581
When azk was created, the process of downloading images by Docker didn't support parallel download of layers. To solve this, the lib https://github.com/azukiapp/docker-registry-downloader was implemented.
However, this lib became unneeded and out-of-date over time. Currently, with the oncoming deprecation of Docker Hub API V1, azk has shown several issues on downloading fresh images.
Removing docker-registry-download completely from azk, besides solving the problem, has added support for images from other repositories other than Docker Hub official repo.

LGTM!

LGTM

LGTM


Automatically suspend all applications out of focus

Just an idea – suspend all applications which are not active, but been in focus before (so only the ones with "windows"). This way you don't need to type anything manually and just use this script on startup (or maybe only on battery power). It suspends all PIDs, including children, which is important for Chrome and Electron like apps.
Good job anyway, I didn't know about AppKit and I used to use killall -STOP "Google Chrome Helper" manually a lot.
This one works fine for suspending Chrome and Atom for me.

This is just an idea, which probably is going to live in my dotfiles. It's supposed to be ran with system python, so the shebang is pointing directly on it (I have homebrewed python is my system path) and it's python 2 only.

At some point most major OSes are going to move to Python 3. Ubuntu is already on that path, and I don't see why Apple wouldn't follow suit. It would be nice to have the script be version-agnostic so that you don't get weird errors once OSX moves to Python 3. :smiley:


[Request] Recurring Events

I haven't found anything that mention the ability to create recurring events is this feature available in the current version?
If not how difficult it would be to implement? Maybe I could try looking over your code and doing myself(and send a PR later).
Or even if you could give some idea of how to implement this behavior in angular controller or service would be nice too.
Cheers.

@bruno-reis I did consider supporting the recurring event, for example, add extra fields to indicate the recurring frequency and on which datetime slot.
But it will become complicated when you modify that event as a single event, for example, you want to delete the event on a specific day while not affecting other days.
So my second thought is the user should be responsible for generating the recurring events. They can easily manipulate their event source array to repeat one event multiple times.
If you have better solution, feel free to let me know. Thanks!


Add: search and filter contributors

Changes are in accordance with #15
Refer this document to see the changes made in UI (Not able to upload screenshots here due to some error)
Note: Select role filter is not functional because roles and affiliations are merged in a single positions array which make it difficult to distinguish between them. Template code is written for that feature.

Also @tanya17-05, it would be better if you can change the UI/UX according to the existing one:

Use the Colors already in the use.
Make the button border rounded & button size similar to the input field size.
Also It would be better in UX, if you can rezise the search and maybe filter itself when you click on the filter (as there is much of the free space avaliable) & in the next line in the mobile view (I assume you have not implemented it yet).
Thanks again for the contribution. :)

Ok @uniqueg @git-anurag-hub , I'll push these changes soon 👍

@git-anurag-hub @uniqueg, I've made changes, please check if there is anything else to add.

@git-anurag-hub I got the point wrong 😅. I thought you asked to make inputs to align vertically in the model itself for small devices. So basically I've to remove the modal and make the input fields to appear in the page itself, right below the search bar, when user clicks filter button (or can I just remove that button and make that filter form appear and never hiding it, what's your opinion?)

I was thinking maybe we can do something like this: https://elastic.github.io/eui/#/forms/combo-box#pill-colors as this will fit our use case both on desktop & mobile. We can show all the positions, "Past contributors" & "Project leads" in the same dropdown which can then they can add up as the user clicks on them.

I think the best option will be to hide the filter initially & then show it under the search field when someone clicks on it (both on desktop & mobile).
Also, I was thinking maybe we can do something like this for Roles & Affiliation: https://elastic.github.io/eui/#/forms/combo-box#pill-colors as this will fit our use case both on desktop & mobile. We can show all the options in the same dropdown (with some sections) which can then they can add up as the user clicks on them.
Also in the requirements set by @uniqueg https://github.com/elixir-cloud-aai/landing-page/issues/15#issuecomment-1295837286, I think the 1st point was to show all the active contributors, not the past contributors. So maybe you can also see into this.

@git-anurag-hub, got it 👍

No @uniqueg there are some changes @tanya17-05 is working on it!

Also @uniqueg, I think the build fails as the NOTION_TOKEN in not set in the environment which tanya pushes.
The solution for this will be to add the NOTION_TOKEN to the docker build environment rather than making it dependent on the client's environment.

@tanya17-05 did you have a chance to push this to a branch from within the organization to check if the CI pipeline passes? I would like to merge this, but would feel better with the CI checks passing

@uniqueg I was trying to create a new branch (filter_contributors) and push this code from it but there is no option for me to do this. I tried to search about this issue, but I guess, I do not have access to create branch in this repository. Can you and @git-anurag-hub please suggest what I can do?

Oh, sorry, @tanya17-05, my bad. I created a new team, gave write permissions for all the front-end projects and added you (you might need to accept via email, or perhaps not, I'm not sure) and @git-anurag-hub. So I'm pretty sure with those permissions you should be able to push branches and create pull requests.

@uniqueg no problem, just got the permissions 😊

Since, I've made another PR by different branch. I guess this PR can be safely closed without merging.


Bugfix - Trimming leading/trailing spaces when saving or editing AB entries

What it solves
This PR closes #2417
How this PR fixes it
Any leading/trailing spaces are trimmed before saving it into the AB.
How to test it

Open/Load a safe
Go to Address book
Create or edit an AB entry adding some leading/trailing spaces to the owner name.
Name should be saved without those spaces.

I guess the same needs to be done when creating/editing owners and safes. Maybe it should be in the reducer?

@katspaugh done!

Works fine
Tested:
Loading a safe with names with blank spaces at the beginning and the end
Creating a safe
Adding names to the AB entry
Editing names in the owners settings
Editing the name of the safe in the settings
Importing a .csv file into the AB
I Made sure the names also don't have blank spaces when Exporting the .csv
Note: Not an issue, but for the record
If there are many spaces between 2 strings of characters, for example: "Owner      Name", it will show as "Owner name" in the AB, but for the .csv and the edition forms it will still show as "Owner      Name". I don't consider this an issue since is rare somebody will save it like this, and also if the person does it, it is intended for him to be saved that way.


Move equals to SootNode class

Add equals and hashcode methods to SootNode, which means better encapsulation.
Two SootNodes are considered equal iff:

Their respective Unit classes are the same
The values of the operation are the same, meaning

The amount of values are identical
The types of the values are identical at their respective positions

Hashcodes are generated using the sum of the hashcodes of the respective types, times the index.
During the writing of this I discovered that there is a possible bug in the current implementation of the equals. Namely, the getValues does not recursively traverse statements, meaning that nested statements are taken as-is and not decomposed. I'm not sure if this is intended behaviour but I thought that I'd mention it.

@JSAbrahams With regards to the potential bug you mentioned: It is not possible for statements to contain other statements. It is, however, possible for a statement to contain an expression which in turn contains another expression. You are correct that these types should actually be checked recursively. I will look into adding that functionality to the equals method after this PR has been merged, but for now we can ignore it.


Docs fix: Initial state isn't a string, it's undefined

@prewk but the typeof operator returns a string :wink:

Wow, I'm tired. Sorry.


Added Altimeter Python interface

Signed-off-by: ahcorde<EMAIL_ADDRESS>🎉 New feature
This PR is part of this mete ticket https://github.com/ignitionrobotics/sdformat/issues/931
Require

https://github.com/ignitionrobotics/sdformat/pull/969

Summary
Added Altimeter Python interface
Test it
from sdformat import Altimeter

altimeter = Altimeter()

Checklist

[x] Signed all commits for DCO
[x] Added tests
[ ] Added example and/or tutorial
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[x] codecheck passed (See contributing)
[x] All tests passed (See test coverage)
[x] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

Codecov Report

Merging #970 (43d44f8) into main (7baa3ee) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #970   +/-   ##
=======================================
  Coverage   66.66%   66.66%
=======================================
  Files           2        2
  Lines          27       27
=======================================
  Hits           18       18
  Misses          9        9

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 7baa3ee...43d44f8. Read the comment docs.