.claspignore is not ignoring anything in it

Expected Behavior
I expected my .claspignore to ignore the files in it.
Actual Behavior
Nothing in .claspignore is being ignored. It also looks like it's not ignoring whatever starts with a period.
Steps to Reproduce the Problem

Create a .claspignore file
Put in every version possible of what I'm trying to ignore:
/node_modules
node_modules/
node_modules/**
/node_modules/
node_modules//.*
node_modules//./**
\node_modules
node_modules
node_modules*
\node_modules*
node_modules*.
node_modules*.*
node_modules
clasp status shows me way too much.
(I am a newbie and I could be missing something, but still.)

Specifications

Node version (node -v): 18.16.0
Version (clasp -v): 2.4.2
OS (Mac/Linux/Windows): Windows

It also looks like it's not ignoring whatever starts with a period.

Same here, plus #983.

Brand new here with a big .venv/ and .git/ in my project. The clasp status output is a mess. And super inefficient! #694
It continues to recursively scan ignored folders. And spam the output with a long list of ignored files.  So one might miss the short list of of included ones.
Workaround: clasp status |sed '/^Ignored/q'
Not ignored files:
└─ /foo/appsscript.json

Ignored files:

I struggled with this one too and figured it out.
Maybe this information will help someone else. The multimatch package is used under the hood to ignore files. See:
https://github.com/google/clasp/blob/d633ca9f8544c94cd245006623de3b8d68181700/src/files.ts#L86-L87
If you look at the multimatch docs, this package uses globbing patterns to match files. Now look at the mentioned code. If some of the provided patterns match the relative path, then a file is marked as ignored.
If you want to ignore directories, you should do:
.cache/**
.git/**
node-modules/**

If you want to ignore files, the file name is enough:
.gitignore
Makefile
README.md
package-lock.json
package.json

Notice that if you add a comment like # directories, it will be passed to multimatch function too.
I also noticed very poor performance when ignoring the node_modules/ directory, as all files inside are iterated.
I hope this helps!


LDAP Alternate for Edit Tracking

Hi. This is a great widget and helped us a lot on using web app builder. First of all I would like to thank you for this. And would like to ask you a question. I'm not sure if this is the right place thus I can post it somewhere else if you want me to.
So here is the question. Is it possible to use edit tracking instead of LDAP? We would like to edit tracking with our own system without LDAP. Users will be recorded in a database. Do you have any suggestions about this scenario?
Thanks in advance.

At a basic level, editor tracking with the locallayer widget is powered by the index page 'getting' a username for the current user and populating the _llwUser variable with that value.  However you populate that username is up to you; The example given in Widget.js uses asp code:

But this could just as easily be a php script as well.  I hope this is enough to get started


gopass-summon-provider: 1.15.14 -> 1.15.15

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/gopasspw/gopass-summon-provider/releases.
meta.description for gopass-summon-provider is: Gopass Summon Provider
meta.homepage for gopass-summon-provider is: https://github.com/gopasspw/gopass-summon-provider
meta.changelog for gopass-summon-provider is: https://github.com/gopasspw/gopass-summon-provider/blob/v1.15.15/CHANGELOG.md
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.15.15 with grep in /nix/store/s6ffk2771q64lwvwd6d09hbnzqy7z2rf-gopass-summon-provider-1.15.15
found 1.15.15 in filename of file in /nix/store/s6ffk2771q64lwvwd6d09hbnzqy7z2rf-gopass-summon-provider-1.15.15

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

gopass-summon-provider

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/0213321fbde03a1d83668ac2ebc6e12716617dd6#gopass-summon-provider

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/s6ffk2771q64lwvwd6d09hbnzqy7z2rf-gopass-summon-provider-1.15.15
ls -la /nix/store/s6ffk2771q64lwvwd6d09hbnzqy7z2rf-gopass-summon-provider-1.15.15/bin

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

x86_64-linux

:white_check_mark: 1 package built:

gopass-summon-provider

Maintainer pings
cc @sikmir for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

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

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 364346

x86_64-darwin

:white_check_mark: 1 package built:

gopass-summon-provider


vJunos Switch not showing interfaces

Hello - I've deployed the latest build of vrnetlab and used it to build a vJunos Switch for use in containerlab however I can't see any of the ge-* interfaces within Junos itself.
Looking at the deployment steps here https://www.juniper.net/documentation/us/en/software/vjunos/vjunos-switch-kvm/topics/deploy-and-manage-vjunos-switch-onkvm.html it shows an XML file being used to tell the vSwitch how many interfaces to build and what bridges to connect them to.
I've been through all the files in the vjunosswitch folder and can't see this XML being referenced anywhere.. are we missing a step from the build process in vrnetlab ?
Or have I missed a config/setup step somewhere?
Any help much appreciated!

without pasting the topology you tried and logs from the container it is impossible to say anythign


fc.lc broken

Domain
https://fc.lc/
Link
https://fc.lc/
Version
0.0.0
What browsers are you seeing the problem on?
Chrome
What OS are you seeing the problem on?
Windows
Anything else?
No response

No example link were provided.


GH-84 handle unknown min severity

Fixes #84
Add patch code for min_severity due to Xray API bug in <3.60.2. Include new test that only executes when Xray version is prior to 3.60.2.
This breaks from our convention that the provider only support latest version of Xray. Since the API bug has since been fixed in theory this patch is unnecessary. However, I think this patch will be useful for customers who aren't yet ready/won't upgrade Xray in immediate future.

Yeah, we came across this bug as well, as we're implementing the Xray provider and noticed the issue when importing existing policies. We ended up upgrading Xray to resolve.


Error while running "sudo howdy config" without nano editor in Fedora 31

When running sudo howdy config command, it comes up with the error listed below:
Opening config.ini in the default editor
Traceback (most recent call last):
  File "/usr/bin/howdy", line 93, in <module>
    import cli.config
  File "/usr/lib64/security/howdy/cli/config.py", line 20, in <module>
    subprocess.call([editor, os.path.dirname(os.path.realpath(__file__)) + "/../config.ini"])
  File "/usr/lib64/python3.7/subprocess.py", line 339, in call
    with Popen(*popenargs, **kwargs) as p:
  File "/usr/lib64/python3.7/subprocess.py", line 800, in __init__
    restore_signals, start_new_session)
  File "/usr/lib64/python3.7/subprocess.py", line 1551, in _execute_child
    raise child_exception_type(errno_num, err_msg, err_filename)
FileNotFoundError: [Errno 2] No such file or directory: '/bin/nano': '/bin/nano'

In /usr/lib64/security/howdy/cli/config.py:
#Open the config file in an editor

# Import required modules
import os
import subprocess

# Let the user know what we're doing
print("Opening config.ini in the default editor")

# Default to the nano editor
editor = "/bin/nano"

# Use the user preferred editor if available
if os.path.isfile("/etc/alternatives/editor"):
        editor = "/etc/alternatives/editor"
elif "EDITOR" in os.environ:
        editor = os.environ["EDITOR"]

# Open the editor as a subprocess and fork it
subprocess.call([editor, os.path.dirname(os.path.realpath(__file__)) + "/../config.ini"])

Seems it did not handle the case that there is no nano editor in the system.

I've searched for similar issues already, and my issue has not been reported yet.
Linux distribution (if applicable):
Fedora 31
Howdy version:
2.5.1

Howdy already tries to get a preferred editor from both /etc/alternatives and the environment, and only if both fail will it fallback to nano. I don't think there's much more to do without writing way more code than required for 99% of users

For Fedora 40 users, of which a fresh install doesn't have an editor entry.
sudo update-alternatives --install /usr/bin/editor editor $(which vim) 100


feat: Add tasks for debugging neovim on linux

This should close https://github.com/cursorless-dev/cursorless/issues/2366, as @pokey added OS X tests already. We can't assume what terminal exists on the linux development machine so I added a way to invoke a bunch of the common ones. The caveat here is that you might end up having multiple terminals installed and it could run a terminal that you prefer isn't the default, but I don't think that's a huge deal atm.
There's obviously lots of other terminals that I didn't add but I think this is good enough for now and we can add more in the unlikely event other developers crop up that run into others.
I haven't got as far as being able to actually run the tests yet, but that's due to a bug I need to hunt down that's separate from these tasks. These both at least run and allow me to debug them on attach.
Checklist

[ ] I have added tests
[ ] I have updated the docs and cheatsheet
[ ] I have not broken the cheatsheet

This now adds other changes I needed to make the tests actually run in my nix dev machine, but with these changes it mostly works. I still run into https://github.com/cursorless-dev/cursorless/issues/2446 and https://github.com/cursorless-dev/cursorless/issues/2445 though.
I add the flake.nix for now, with changes that build upon https://github.com/cursorless-dev/cursorless/pull/1908. Atm cursorless-neovim requires the latest neovim node-client, which isn't in nixpkgs yet. I have an open PR for that, but in the meantime I have to do a manual override from my PR branch, thus the big overlay, which will eventually go away.
I also explicitly call command nvim to avoid any aliases being used, because that caused me massive debug pain at first. I also install a bare version of neovim and prefix PATH so that that version is picked up by vscode first, otherwise also a big pain because of how neovim packaging often works on nix.

Sounds good to me. A few minor feedbacks and rebasing to latest nvim-talon should fix the test on CI failure and we can merge.
@fidgetingbits  https://github.com/cursorless-dev/cursorless/pull/1908 has been merged in main (hence in nvim-talon branch) so I am guessing all the nix stuff can be removed from this PR and it will avoid the merge conflicts too?

Sounds good to me. A few minor feedbacks and rebasing to latest nvim-talon should fix the test on CI failure and we can merge.
@fidgetingbits  https://github.com/cursorless-dev/cursorless/pull/1908 has been merged in main (hence in nvim-talon branch) so I am guessing all the nix stuff can be removed from this PR and it will avoid the merge conflicts too?

will rebase later today hopefully and see ya. not quite all the Nix stuff can be removed. my PR to update the neovim node plugin in nixpkgs still hasn't landed, so the overlay will still be needed, but the diff will be simpler after rebase for sure.

Just finished locally testing this after rebase and is all good. Ran into a weird issue related to one of pokey's changes where it couldn't rm -f cursorless-neovim because it was a folder not a symlink, but it was kind of hidden because that folder was also .gitignore'd at some point. Anyway, afaict seems like it's just a remnant of my old test environment and not an actual bug, so should be ok.
Just have to wait for the windows CI test to pass, but it seems like the other ones are passing now so hopefully is good.

Ready to be merged, cc @pokey


Update scroll lock method

I'm submitting a:

[ ] bug report
[x] feature request

What OS are you using?
Windows 10
What browser(s) is this affecting:
Chrome, Firefox, Opera, Edge, IE
Current behaviour:
When opening a <mwc-dialog>, the current scroll lock method (adding overflow: hidden; to <body>) hides the body scroll bar, causing page content to shift.
Example of content shifting:

Expected behaviour:
Page content to not shift when locking body scroll.
Example of content not shifting:

This is caused by MDC's Foundation layer, and we don't want to deviate too heavily from their styling.
Having overflow: hidden to stop scrolling is a standard practice, and while it is unfortunate that it reflows layout with the scrollbar, we believe other solutions would be prohibitive to implement.


Add link check action

Test the new link check action.

Test succeeded, will add this action in #6


npm guide

This adds a really barebones guide to publishing code on npm.
Review

@wpears
@contolini

I like substack on the general module writing process. One of node's greatest strengths is its vibrant ecosystem, so checking around for the module you're about to write is always a good first step.

Good call @contolini.
@wpears I'm going to add a link to Substack's guide as I agree it's worth a read (and that searching for existing modules is a good first step). Feel free to add or tweak the wording of this document from there.

Updated ^^

Looks good; merging.


Mission Issues

When attempting to do the second mission of GTA V at Simeon's Dealership with a friend using both the last stable version and the latest nightly build, we encountered an issue where the cars inside the building would duplicate and clip into each other leading to some explosions and retendering the mission unable to be attempted. I'm sorry if this is already a known issue, but I couldn't find it in any of the other issues

Did you both start the mission? Only one player need to start the missions otherwise entities will be duplicated

Neither of us were able to start the mission, as every time we got close it duplicated everything and the game told us the area was disrupted and to come back later

Would it be possible to make synced vehicles "ghost vehicles" as a toggleable option? This might solve that issue.

Another possibility I thought about, could have a mission resource that enables traffic for the "host" (as for time/weather sync) and disables for others (it should also "force remove" some NPCs which aren't removed by normal "disable traffic", like people who have their car/bike/money stolen and ask for help, my guess is that they are mission entities and as such bypass the natives which disable traffic). If possible it could also sync the cutscenes.

I have encountered the same issues as OP. I set up my own server using the ZeroTier integration.  We eventually got the scene to work by having the guest player be far enough away so when the host player got to Simeon the guest player's version of the cars would not load.
However, the mission was still softlocked because later you drive to an area, get out of the car and jump a fence, then there is supposed to be an NPC that interacts with your partner but there is no one there so nothing happens.
Also, it seems the latest GTA Online update (Los Santos Drug Wars) is causing a fatal error with Script Hook V, so I cannot test the issue any further at the moment.

I just discovered a different issue. During the very first mission, when you have to swap characters, the character you swap to automatically dies and any attempt to skip that section fails with the message "Mission Failed T. Died."

I just discovered a different issue. During the very first mission, when you have to swap characters, the character you swap to automatically dies and any attempt to skip that section fails with the message "Mission Failed T. Died."

Same problem.It happens not only when switching characters, but also during other tasks, that characters or NPCs die inexplicably.


Missing details in 'Application Files dialog box'

I'm looking at the 'Application Files dialog box' section of this page.
Could this include more information about how that list of files is generated?
Currently it says:

It contains a grid that lists the project files that are not excluded by default, or that have a download group.

What does 'excluded by default' mean? Is the list generated by analysing the project files, looking at the build output, or something else? As far as I can see, the answer is 'something else', since the list of files does not match either of those other sources. Also, the contents of the list are different after cleaning my build output directories compared to after a successful build.

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

ID: ab856c02-f379-f49e-e802-87bf64aaaf64
Version Independent ID: cef1c93e-b929-502d-a553-158aea470c1c
Content: Publishing ClickOnce Applications - Visual Studio (Windows)
Content Source: docs/deployment/publishing-clickonce-applications.md
Service: visual-studio-windows
Sub-service: deployment
GitHub Login: @Mikejo5000
Microsoft Alias: mikejo

@HughWarrington Did you already read this article? https://learn.microsoft.com/en-us/visualstudio/deployment/how-to-specify-which-files-are-published-by-clickonce
"...all non-code files in the project are deployed along with the application." And the article has information such as this: "Files with the Build Action property set to Content are designated as application files and will be marked as included by default." along with exceptions to this.
What are you seeing that does not conform to expected behavior?

@HughWarrington Let me know if you have any more details and we can re-open this.

Hi @Mikejo5000 thank you for your reply.
Yes I read that other page too. It has helpful information, although as it happens I find that first sentence  "...all non-code files in the project are deployed along with the application" to be vague (1), technically incorrect (2), and contradicted by the rest of the page (3).
(1) What are "non-code files in the project"? Is a dll file code or non-code?
(2) Looking in my current project for example, here's a file which is surely 'non-code' which is not deployed along with the the application: an .ico file with build action 'None'.
(3) Even within the same paragraph, we learn that files can selectively be excluded -- so not 'all' files are deployed necessarily.
The linked page does tell you one way to add files to the Application Files dialog box (set Build Action to Content) but that's clearly not the only way, since my built .dlls are included in the dialog box already and they don't appear in Solution Explorer. Furthermore as I wrote above "the contents of the list are different after cleaning my build output directories compared to after a successful build" so that's another part of the answer.
My question remains 'how exactly is the contents of the Application Files dialog box arrived at?'

For what it's worth, despite my ranting above I don't think this is a 'badly written documentation' issue, more an 'undocumented behaviour' issue.

@HughWarrington We will be tracking this issue internally as I look into the details, so I'm closing it here.


compileAsync does not loadSchema a schema that is $ref’d by a oneOf with a discriminator

What version of Ajv are you using? Does the issue happen if you use the latest version?
latest (8.13.0)
Ajv options object

const options = {
  discriminator: true,
  async loadSchema(url) {
    console.log('loadSchema', url)
    const name = url.substring("scheme://".length)
    return schemas[name]
  },
};

JSON Schema

const schemas = {
  main: {
    type: "object",
    discriminator: {propertyName: "foo"},
    required: ["foo"],
    oneOf: [
      {
        $ref: "scheme://schema1",
      },
      {
        $ref: "scheme://schema2",
      },
    ],
  },
  schema1: {
    type: "object",
    properties: {
      foo: {const: "x"},
    },
  },
  schema2: {
    type: "object",
    properties: {
      foo: {enum: ["y", "z"]},
    },
  },
};

Sample data

{"foo": "x"}

Your code

const Ajv = require("ajv");
const ajv = new Ajv(options);
// If you uncomment this, then there is no error
// Object.entries(schemas).forEach(([id, schema]) => {
//   ajv.addSchema(schema, 'scheme://' + id)
// })
async function main() {
  const validate = await ajv.compileAsync(schemas.main)
  validate(data)
}
main().catch((err) => console.log(err))

Validation result, data AFTER validation, error messages
Error: discriminator: oneOf subschemas (or referenced schemas) must have "properties/foo"

What results did you expect?
No error
Are you going to resolve the issue?
Maybe if it is easy
Notes:

If you call ajv.addSchema (i.e., uncomment the commented code), then the error goes away. Therefore, it is the resolving of the $ref that is the issue

When discriminator: true, then loadSchema is not called.

But if you set discriminator: false and remove the discriminator from schema.main, then loadSchema is called.
loadSchema scheme://schema1
loadSchema scheme://schema2

My guess is that lib/vocabularies/discriminator/index.ts’s getMapping() needs to throw new MissingRefError so that _compileAsync can catch the error and load the missing schemas.

Hi @yonran thanks for raising this, it does look like it could be a bug. Since you responded "Maybe if it is easy" I'll ask if you would like to have a go at submitting a PR? Either way I will look into this and consider it a bug for now.


Using Deepmatcher 0.1.2 in Kaggle - torchtext.legacy issue

Using Deepmatcher 0.1.2 in Kaggle - torchtext.legacy issue
I have tried to use Deepmatcher 0.1.2 in Kaggle.com. I have tried both
!pip install -qqq deepmatcher==0.1.2
and
!pip install -qqq deepmatcher
Once the installation is done, when I try to import the package with
import deepmatcher as dm
the following error appears:

ModuleNotFoundError                       Traceback (most recent call last)
 in
----> 1 import deepmatcher as dm
/opt/conda/lib/python3.7/site-packages/deepmatcher/init.py in
8 import sys
9
---> 10 from .data import process as data_process
11 from .models import modules
12 from .models.core import (MatchingModel, AttrSummarizer, WordContextualizer,
/opt/conda/lib/python3.7/site-packages/deepmatcher/data/init.py in
----> 1 from .field import MatchingField, reset_vector_cache
2 from .dataset import MatchingDataset
3 from .iterator import MatchingIterator
4 from .process import process, process_unlabeled
5 from .dataset import split
/opt/conda/lib/python3.7/site-packages/deepmatcher/data/field.py in
11 import fasttext
12 import torch
---> 13 from torchtext.legacy import data, vocab
14 from torchtext.utils import download_from_url
15 from urllib.request import urlretrieve
ModuleNotFoundError: No module named 'torchtext.legacy'
By default, torchtext version 0.8.0a0+cd6902d is installed in Kaggle.
This version of torchtext has the following sub packages:
dir(tochtext)
['all',
'builtins',
'cached',
'doc',
'file',
'loader',
'name',
'package',
'path',
'spec',
'version',
'data',
'datasets',
'git_version',
'nn',
'utils',
'version',
'vocab']
So there is no torchtext.legacy. I also could not find any version of torchtext where both data and vocab were in torchtext.legacy. So there may be a bug in deepmatcher 0.1.2 related to torchtext.legacy.
When I install deepmatcher 0.1.1 with
!pip install -qqq deepmatcher==0.1.1
everything is working fine.
I have tested some examples both on CPU and GPU and there are no issues.

This happens when your installed version of torchtext is out of date - deepmatcher requires torchtext 0.9 due to certain backwards incompatible changes made in 0.9. I'll update the pip required version in setup.py.

@sidharthms also experiencing this issue. Checked the torchtext release notes and it's been removed as of v.0.12.0.

I also have this issue, and I have torchtext v0.12.0. I saw on stackoverflow that it was supposed to work with torchtext v0.10.0 but this version isn't available on pip anymore.
ERROR: Could not find a version that satisfies the requirement torchtext==0.10.0 (from versions: 0.1.1, 0.2.0, 0.2.1, 0.2.3, 0.3.1, 0.4.0, 0.5.0, 0.6.0, 0.12.0)
ERROR: No matching distribution found for torchtext==0.10.0

We'll look at migrating into the new torchtext over the next 1-2 months,
but in the meantime you can get it working on most Linux platforms I
believe (you can try running on Colab https://colab.research.google.com/
online) - torchtext 0.10 is still available for those
https://pypi.org/project/torchtext/0.10.1/#files. 0.10 is not available
for the newer Mac versions apparently.
On Fri, Jun 10, 2022 at 4:09 PM rafaelbo @.***> wrote:

I also have this issue, and I have torchtext v0.12.0. I saw on
stackoverflow that it was supposed to work with torchtext v0.10.0 but this
version isn't available on pip anymore.
ERROR: Could not find a version that satisfies the requirement
torchtext==0.10.0 (from versions: 0.1.1, 0.2.0, 0.2.1, 0.2.3, 0.3.1, 0.4.0,
0.5.0, 0.6.0, 0.12.0)
ERROR: No matching distribution found for torchtext==0.10.0
—
Reply to this email directly, view it on GitHub
https://github.com/anhaidgroup/deepmatcher/issues/85#issuecomment-1152792779,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABWE5ZLJSBADAAMMZNUUA3TVOPDLHANCNFSM46S5MEGA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Groupby dictionary aggregation works incorrect in cases when 'by' and columns to aggregate has intersection

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):

Ubuntu 20.04.3 LTS

Modin version (modin.__version__):

0.10.2+2.g253ecd98.dirty

Python version:

Python 3.8.6

Code we can use to reproduce:

import modin.pandas as pd

df = pd.DataFrame({
    "col0": [ 1,  2,  1,  2],
    "col1": [11, 12, 13, 14],
    "col2": [ 4,  4,  4,  4],
    "col3": [ 1,  1,  1,  1]
})

print(df.shape)
gb = df.groupby("col0")
result = gb.agg({"col0": "count"})
string = "%s" % repr(result)
print(string)

Describe the problem

Code above works on Pandas but produces a ValueError: No objects to concatenate exception on Modin on Python and Ray engines (those that I tried). I traced the problem down to these lines
        if drop and len(df.columns.intersection(by_part)) > 0:
            df.drop(columns=by_part, errors="ignore", inplace=True)

that are located inside of reduce function. As a result of these lines df that contains only one column col0 gets empty because this column is dropped. I am not sure whether it is intentional, it doesn't look right for me. @dchigarev please comment.
Source code / logs

One more code example with inconsistent behavior between Modin and Pandas:
df = pd.DataFrame({"a": [1, 2, 3], "b": [4, 5, 6], "c": [7, 8, 9], "d": [10, 11, 12]})
result = df.groupby("a")["a", "b", "c"].agg({"b": "count", "c": "sum", "a": "sum"})
print(result)


Hugo extended not available as snap

Hugo extended isn't in the snap version. Is there any reason this isn't included by default?
If it's important that they are separate app, then maybe hugo-extended can be installed alongside hugo within the snap package?

Not a bug. Use --channel=extended to install extended. To switch from already installed Hugo to Hugo Extended run snap refresh hugo --channel=extended.

My apologies, didn't realize you had it on a different channel!


Need to test DataFrame.product with small DataFrames

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):

Ubuntu 19.10

Modin version (modin.__version__):

0.7.3+42.gc78828e.dirty

Python version:

Python 3.7.5

Code we can use to reproduce:

diff --git a/modin/pandas/test/utils.py b/modin/pandas/test/utils.py
index 1faaf38..d4d257c 100644
--- a/modin/pandas/test/utils.py
+++ b/modin/pandas/test/utils.py
@@ -63,6 +63,11 @@ test_data = {
         ]
         for i in range(NCOLS)
     },
+    "small": {
+        "col0": [1, 2, 3, 4],
+        "col1": [8.0, 9.4, 10.1, 11.3],
+        "col2": [4, 5, 6, 7],
+    },
     # "int_float_object_data": {
     #     "col3": [1, 2, 3, 4],
     #     "col4": [4, 5, 6, 7],

Describe the problem

When dataframe size is small and is smaller than min_count parameter value for product it causes a lot of tests to fail. Pandas behaves quite strangely and Modin works differently.
Source code / logs

I've found out that pandas have a strange output for product and sum only if numeric_only parameter specified or skipna=False, so I've changed modin behavior to it mirrors pandas in #1827 and also added test cases that were suggested by @gshimansky above


Outside enemies spawning inside

Recently, when using this mod, I've noticed that the game spawns Earth Leviathans inside the facility.
Since the mod list I was using was quite expansive, I wasn't sure if it was this mod specifically or something else interfering, but after putting together a mod list of just this mod, lethal days, and better saves, it still happened. r2modman profile code: 018d5252-97bb-43a6-bae6-6468291ca18d (I added some mods afterwards to make finding the enemies easier)
My method of testing was: 1. start the game, 2. land on experimentation, 3. go inside, 4. look at the console until it said that an earth leviathan got added, 5. if the power limit was reached before that, quit and go to 1, otherwise, 6. try to find the earth leviathan.
I have also seen it add eyeless dogs but didn't manage to find them.

Thanks for this report! It seems like when I was recoding the configuration for the enemies, the daytime and outside enemies got added to the inside enemies list.
Instead of having to wait around for leviathans to spawn inside, you can change the spawn amount curve to something like 15, and use the UnityExplorer mod to enter freecam and visually confirm the position of any enemies.


Haskell-level re-exports of qualified modules are mis-reported

For
module Lib (module Data.Functor.Identity, module Data.Maybe) where

import qualified Data.Functor.Identity
import qualified Data.Maybe

Haddock generates

But the exports, as reported by -ddump-hi, are only
exports:
  Data.Maybe.maybe
  GHC.Base.Maybe{GHC.Base.Just GHC.Base.Nothing}

The Haskell Report explains this:

The form “module M” names the set of all entities that are in scope with both an unqualified name “e” and a qualified name “M.e”. This set may be empty.

I think you can find the relevant code here:
https://github.com/haskell/haddock/blob/bea565ec5a029b8c19965aa22f34c23a112c0a7f/haddock-api/src/Haddock/Interface/Create.hs#L263-L271

Thanks for the pointer!
Reliably detecting whether we can use the compact "module X" representation requires more info than unrestrictedModuleImports currently has. As an example, consider
module M (module Data.Tuple) where

import qualified Data.Tuple

While the qualified import may suggest that some exports of Data.Tuple are not re-exported, in fact Prelude already makes the entirety of Data.Tuple available without qualification, and therefore M, too, re-exports all of Data.Tuple.
But maybe we can overlook edge cases like this and simply use the expanded representation. At least we wouldn't mis-report un-exported entities.


Distinguish between stored documents and temporary ones

It is not as smart as I hoped it would be, but if I didn't overlook anything, it seems to work:
This PR adds a Type StoredDocument<D> and it's counterpart TemporaryDocument<D> (which is just used as return-type for cloning). Sincerely, I could not find a way without modifying Document, but it should not affect the original documents.
It could be used to type things like game.user or canvas.scene.

Don't we also need to make the change for the toObject and toJSON methods for DocumentData then...?

I knew that this could not be so easy :-D

But not as hard as I thought :-)


Bugfix for select method

Added simple extra parameter

Dear @bgultekin, could you give a look at this PR? I'm depending on this fix for a Laravel 5.5 project.

Your changes is come from the old code base, so, please re-fork this project then you can see your PR has been there, in the latest release.
Regards

@nthachus Maybe I'm missing something, but in the latest release I can see that the select method has only 5 parameters. It should have 6 parameters.
Please look at https://github.com/bgultekin/laravalid/blob/898834e741ae9ba0f13b39598dffb5b01d9c28f0/src/Bllim/Laravalid/FormBuilder.php#L125 and https://github.com/LaravelCollective/html/blob/41cd9291a69bd24f2184e504be041348a87308a8/src/FormBuilder.php#L582

Yes, you're right, so, please make a PR based on the latest code for that
change.
Regards
On Sun, Feb 25, 2018 at 3:58 AM, yapsr<EMAIL_ADDRESS>wrote:

@nthachus https://github.com/nthachus Maybe I'm missing something, but
in the latest release I can see that the select method has only 5
parameters. It should have 6 parameters.
Please look at https://github.com/bgultekin/laravalid/blob/
898834e741ae9ba0f13b39598dffb5b01d9c28f0/src/Bllim/
Laravalid/FormBuilder.php#L125 and https://github.com/
LaravelCollective/html/blob/41cd9291a69bd24f2184e504be0413
48a87308a8/src/FormBuilder.php#L582
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bgultekin/laravalid/pull/53#issuecomment-368259745,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AERJ-qlJ-vaYUmbyLJOqmvpM5SuunXRGks5tYHfrgaJpZM4SRuUf
.

@nthachus I have created a new PR: https://github.com/bgultekin/laravalid/pull/57


NLU-2697 - Fetch upstream

Connor use rebase and merge or create merge commit while merging
@andrew-gradinari  correct me on this one.

@Conor-Keaney use rebase and merge or create merge commit while merging @andrew-gradinari correct me on this one.

You are right, I believe rebasing is preferable since it preserves the commit history.
By the way, I have documented the entire process here - https://inindca.atlassian.net/browse/NLU-2591?focusedCommentId=1774484


"main" in manifest

I know it works fine on npm without "main" field in package.json, but it may fail in other enviroments as did in NativeScript (https://github.com/NativeScript/NativeScript/issues/466). I understand what that is a NativeScript bug, but is it a problem to add "main" fields to the package.json?

It's npm-wise valid and as you mentioned yourself a NativeScript bug. None of our business, thus :) Just updating one package.json isn't a big deal indeed, but updating hunderds of modules would be.

@arthurvr that is true and I agree, but problem is that NativeScript does not makes releases too often, so it's easier to fix concrete package.


fix HBS.toJavaMap

see https://github.com/bizreach/play2-handlebars/commit/fe42c1768 and https://tech.bizreach.co.jp/posts/32/play2-handlebars/

Thank you for your contribution again.
As you say, this is just a missing part of #14 changes.


Enabling "PublishSingleFile" may causes "ResourceManager.Current" to throw an error

Describe the bug
When I tried to enable "PublishSingleFile" and attempted to publish it as a single executable file, an error occurred when accessing ResourceManager.Current:
Unable to find the specified file.
   at WinRT.ExceptionHelpers.<ThrowExceptionForHR>g__Throw|39_0(Int32)
   at WinRT.ExceptionHelpers.ThrowExceptionForHR(Int32)
   at ABI.Windows.ApplicationModel.Resources.Core.IResourceManagerStaticsMethods.get_Current(IObjectReference)
   at Windows.ApplicationModel.Resources.Core.ResourceManager.get_Current()
   at Demo.MainWindow..ctor() in E:\Projects\Demo\MainWindow.xaml.cs:line 17

Steps to reproduce the bug

Enable the PublishSingleFile option in win-x64.pubxml
In the constructor of MainWindow, write the following code:

public partial class MainWindow
{
    public MainWindow()
    {
        string filePath = @"D:\demo.txt";

        try
        {
            var curr = ResourceManager.Current;
        } catch(Exception e)
        {
            File.WriteAllLines(filePath, [e.Message, e.StackTrace]);
        }
    }
}

Open this window on OnLaunched:

    protected override void OnLaunched(LaunchActivatedEventArgs args)
    {
        MWindow = new MainWindow();
        MWindow.ExtendsContentIntoTitleBar = true;
        MWindow.Activate();
    }

Publish, then run the exe and check the output errors in the root directory of the D drive:

Unable to find the specified file.
   at WinRT.ExceptionHelpers.<ThrowExceptionForHR>g__Throw|39_0(Int32)
   at WinRT.ExceptionHelpers.ThrowExceptionForHR(Int32)
   at ABI.Windows.ApplicationModel.Resources.Core.IResourceManagerStaticsMethods.get_Current(IObjectReference)
   at Windows.ApplicationModel.Resources.Core.ResourceManager.get_Current()
   at Demo.MainWindow..ctor() in E:\Projects\Demo\MainWindow.xaml.cs:line 17

Expected behavior
No response
Screenshots
No response
NuGet package version
Windows App SDK 1.5.5: 1.5.240627000
Packaging type
Unpackaged
Windows version
Windows 11 version 22H2 (22621, 2022 Update)
IDE
Visual Studio 2022
Additional context
This is the code mentioned above:
Demo.zip

I think the ResourceManager from the Microsoft.Windows.ApplicationModel.Resources namespace will work with single file publishing.

Thank you very much!
After replacing it with the ResourceManager you mentioned, it worked. I spent several months looking for a solution. 😂

@NXY666 Thanks for reporting this, and @lhak thanks for the solution.  Closing issue as resolved.


Feature: Update layout button icon as layout changes

What feature or improvement do you think would benefit Files?
At the moment, the layout button's icon does not match the current chosen layout from the flyout.

Requirements

This proposal will allow the user to see what the current layout is being used, and provides a better hint at what this button does, which is unlabelled in the UI

Files Version
2.5.13 Preview
Windows Version
22621.1928
Comments
No response

Thank you for your feedback.

When using adaptive layout should we use a specific icon or the one corresponding to the current layout (details or grid)?

Adaptive mode was turned into a toggle from based on the  Work in progress discord post


Packet parsing timeout

My Ruby 1.8.7 based project includes a queue subscriber and it works perfectly on my workstation with ActiveMQ running on the same host. However, as I deploy to the production environment, one message gets popped from the queue and then I get the timeout exception:
Packet parsing timeout (Stomp::Error::PacketParsingTimeout)

This error seems to have reared its head again. :disappointed: Not sure what is happening, but with the current version of the stomp gem and JRuby 1.7.19 it seems to be hanging with this error no matter the parse_timeout or reliable setting.
Do the other timeouts interact with this in any way?

So, ..... "this problem" means ???? Is it that you are getting a:
Stomp::Error::PacketParsingTimeout

???
Or a "hang" ??
What broker are you running?  And what is in the broker logs when this occurs?


Add a new icon

Duplicates ❌

[X] I have searched the existing icons and requests.

Icon name
Tools
Icon description
Used o represent the repair category into sustainable development

Icon keywords
tool, tools, repair
Icon author
No response
Icon SVG code
<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 24 24" width="24" height="24"><path fill="none" d="M0 0h24v24H0z"/><path d="M5.33 3.271a3.5 3.5 0 0 1 4.254 4.963l10.709 10.71-1.414 1.414-10.71-10.71a3.502 3.502 0 0 1-4.962-4.255L5.444 7.63a1.5 1.5 0 1 0 2.121-2.121L5.329 3.27zm10.367 1.884l3.182-1.768 1.414 1.414-1.768 3.182-1.768.354-2.12 2.121-1.415-1.414 2.121-2.121.354-1.768zm-6.718 8.132l1.414 1.414-5.303 5.303a1 1 0 0 1-1.492-1.327l.078-.087 5.303-5.303z"/></svg>

Are there any other important details?
No response

Thanks @felixbusadkt we add it :)

Thanks @felixbusadkt & @Sabrinavigil
Here are the links for the development follow-up:

https://github.com/Decathlon/vitamin-web/issues/1301
https://github.com/Decathlon/vitamin-android/issues/142
https://github.com/Decathlon/vitamin-ios/issues/96


[Bug] Prescale should be enabled by default

Describe the Bug
As suggested in #832 by @sedghi, enabling prescaling fixed window level issues when loading a stack of images through wadouri. Their comment also mentions that it would make sense to have this enabled by default. I am not sure what the consequences of this would be so I'm hesitant to create the PR myself, but didn't want this to be forgotten, hence this issue.
Steps to Reproduce
A reproduction is available here: https://sjorsgielen.nl/cs3d.zip. It's pretty large since it contains a 90MB volume, so Github doesn't allow me to upload it directly. For future record, here is a zip file containing everything except the volume:
cs3d.zip
First, disable SharedArrayBuffer in the variables at the top of src/main.ts because of #886.
Run yarn install and yarn dev to start the webserver, then browse to the indicated port.
The current behavior
In the default window level, the image is rendered incorrectly. The window level must be set exceedingly high for the image to render normally.
By setting preScale: { enabled: true } in the call to cornerstoneDICOMImageLoader.wadouri.loadImage, the image is rendered normally if no explicit window level is set.
The expected behavior
No specific flags need to be passed for the image to render normally.
OS
macOS 14.0
Node version
20.8.1
Browser
Safari 17.0

would you mind creating a PR for this?

I think this has been fixed in the main branch, we have preScale to true always by default in the wadouri as well, can you check please

I think this has been fixed in the main branch, we have preScale to true always by default in the wadouri as well, can you check please

I would like to try - but since #990 unfortunately loading images through wadouri does not seem to be possible anymore. I can try on the last release before #990 for reference. But if I need to run on main (or latest release), I'm afraid I need either a new way to load a cornerstone Image from an ArrayBuffer or the loader option to wadouri should be restored.

@sedghi I have just tried on 1.44.2 but the issue remains there if preScale.enabled: true is not explicitly passed to cornerstoneImageLoader.wadouri.loadImage(). As mentioned, on 1.44.3 / main I cannot test it, because I need the loader option to load my images, and it has been removed.

@sedghi I can look into this one and #886 further now that I'm almost on the latest release again. I could use some additional guidance, though, in particular on two topics:

could you give me a small overview of how scaling is performed throughout the various components (and why there), perhaps with references to source code?
could you share your current method of quickly testing local changes? I've tried various ways to build my test application against local clones of cornerstone3d (e.g. yarn link), and the best way I found is to remove cornerstone3d from my test application's node_modules and symlink the cornerstone3d source directory into it manually. Then, every time I make a source code change to cornerstone3d I rebuild its dist directory, remove my test application's vite cache and restart vite, then reload the page. This works, but is somewhat cumbersome. Any suggestions from your experience for more rapid testing would be welcome here.

This is now enabled automatically on main, and I can confirm this issue is resolved in Cornerstone3D 2.0.


Build Failed with ReactJS.framework

If I put the library ReactJS.framework to the embedded Binary, I can't build properly. Whenever it builds, it  can't proceed. Instead, It seemed to hang for 2 minutes displaying "Running 2 of 2 two custom shell scripts" on the top of xcode. And it gave final message, "build failed" without error message.  So If I get rid of it in the embedded Binary, it works.

Can you please provide more information? Please include:

the command you used to run infer
the full output (please use https://gist.github.com/ or equivalent)
the contents of toplevel.log (also on https://gist.github.com/)

please re-open with more details


Split zebrad lockfile build into a separate workflow

Motivation

We might want to run the lockfile job on each new tag.
Its run conditions are not the same as the ci.yml job.

Suggested Run Conditions
    paths:
      # code (and tests, but this job doesn't build them)
      - '**/*.rs'
      # hard-coded checkpoints (and proptest regressions, but this job doesn't run them)
      - '**/*.txt'
      # dependencies
      - '**/Cargo.toml'
      - '**/Cargo.lock'

      # changes:
      #
      # workflow definitions
      # TODO: change to the new workflow name
      - '.github/workflows/ci.yml'

      # removals:
      #
      # no test data snapshots - this isn't a test
      # - '**/*.snap'
      # we don't run `cargo deny` in this check
      # - '**/deny.toml'

Related Work
Originally posted by @teor2345 in https://github.com/ZcashFoundation/zebra/issues/3998#issuecomment-1082608428

We can re-open this if it's ever needed.


Propose stream-based Replication of Log Entries

Stream-based Replication of Log Entries
Rationale
The existing request-response style API offers limited parallelism and pipelining capabilities and does not allow for dynamic adjustment of request sizes.
By introducing a Stream as an argument to RaftNetwork::append_entries(), we can empower the method to replicate log entries in a more flexible and potentially optimized way.
Accordingly, the storage layer should also expose a stream-based API. We propose adding RaftLogReader::try_get_log_entries() -> impl Stream<Item=Entry>, which will enable the streaming of log entries as needed.
Proposed new API:
trait RaftLogReader<C: RaftTypeConfig> {
    async fn get_log_entries(&mut self, since: u64)
               -> impl Stream<Item=C::Entry>;
}

trait RaftNetwork<C: RaftTypeConfig> {
    async fn entries(&mut self,
                   prev: Option<LogId>,
                   entries: impl Stream<Item=C::Entry>)
              -> Result<
                       impl Stream<Item=Result<Response, RPCError>,
                       RPCError<_,_,RaftError<>>
                    >;`
}

RaftNetwork::entries() now takes a stream of log entries for transmission and returns a stream of replication responses. Should a replication attempt result in an Err, such as the absence of prev on the remote peer or the detection of a higher term vote, the caller should immediately discard the stream.

Nice!


4 600 / 3 650 as office staff

That proposition has been proposed on general chat but never really discussed, it's was clear that 2 points was essential, first, more space for starter and second, not ending 10x 1 260 ...
That proposition seems to respect that. Also, more importantly, it's introducing an exponential factor on staff as office construction IS exponential itseft.
Actually :
1 - 140 / 60
2 - 220 / 110
3 - 300 / 180
4 - 380 / 240
5 - 460 / 300
6 - 540 / 360
7 - 620 / 420
8 - 700 / 480
9 - 780 / 540
10 - 860 / 600
11 - 940 / 660
12 - 1 020 / 720
13 - 1 100 / 780
14 - 1 180 / 840
15 - 1 260 / 900
Proposed in this commit :
1 - 220 / 66
2 - 280 / 114
3 - 380 / 194
4 - 520 / 306
5 - 700 / 450
6 - 920 / 626
7 - 1 180 / 834
8 - 1 480 / 1 074
9 - 1 820 / 1 346
10 - 2 200 / 1 650
11 - 2 620 / 1 986
12 - 3 080 / 2 354
13 - 3 580 / 2 754
14 - 4 120 / 3 186
15 - 4 700 / 3 650

@spencerm
It's updated for better reflect people request.
1 - 550 / 298
2 - 607 / 359
3 - 667 / 428
4 - 729 / 501
5 - 793 / 578
6 - 859 / 658
7 - 925 / 740
8 - 992 / 826
9 - 1 060 / 913
10 - 1 129 / 1 002
11 - 1 199 / 1 094
12 - 1 269 / 1 186
13 - 1 340 / 1 281
14 - 1 411 / 1 377
15 - 1 483 / 1 474

Updated to New version.

As discussed, definitely open to exploring a "mega HQ" option and/or making levels higher than 12 more affordable but as-is don't want to give folks more staff as otherwise large airlines will overpower smaller ones


Testing

https://github.com/user-attachments/assets/976483eb-8973-45fc-93a3-264c6a9a6183

https://tekeye.uk/html/html5-video-test-page


Place valence and bevy specific stuff in valence_anvil behind feature flags

Objective

Closes #504

Solution

Place valence and bevy specific stuff in valence_anvil behind feature flags.
Support for entity and POI chunks can easily be added to DimensionFolder in the future.
Support for writing chunks is not included in this PR.

Looks like check-depgraph is failing to post a comment

Hmm yeah, I'll try fixing that separately.


BSOD on Windows 10 with Hyper-V Enabled

Problem description

Trying to folow the instructions found at "Set Up Your Swarm" causes Windows 10 to fatally crash with a BSOD (Blue Screen of Death). This is a long-time, known Virtualbox bug, which Oracle (and I) assign blame to Microsoft.
My issue is that the documentation should be updated to reflect this critical error. (It's very unfortunate, because no one on windows 10 can complete the getting started guide.)
I have separately filed a Docker Issue for the technical problem.
Problem location

I saw a problem on the following URL: https://docs.docker.com/get-started/part4/#set-up-your-swarm

Project version(s) affected

docker-machine.exe version 0.10.0, build 76ed2a6
Suggestions for a fix

Suggest some warning that attempting to run VirtualBox with Hyper-V enabled will cause a BSOD. Ironically, Docker requires Hyper-V enabled for itself to run.

@NonCreature0714 What command are you running that triggers the BSOD? If it's using driver of virtualbox instead of Hyper-V, that is not supported, which is why there are tabs and callouts on the "Set up your swarm" section.

@johndmulhausen I'm working to clarify. Frustratingly, following the recommended instructions caused BSOD three times yesterday - but today everything is running perfectly, so I'm struggling to reproduce. I did run into something else: using docker-machine with Powershell requires Powershell run as Administrator.
I'm nervous to say much more if I can't reproduce the bug; I'll close this if I can't reproduce within 24 hours.

@johndmulhausen I should also mention that several people at my place of work also encountered this bug, but now can't reproduce.

@NonCreature0714 Keep me posted. At a high level: Does following the Hyper-V instructions work?

@johndmulhausen the Hyper-V instructions work.
The best I can tell right now is that there was some conflict with running Vagrant and docker-machine in proximity on the same machine. Vagrant absolutely causes BSOD because of a bug in VirtualBox. And it's possible that when I ran docker-machine afterwards there was still some unresolved issue. I'm currently working to reproduce. Either way, this looks very much like an edge case.
I'm strongly advocating for Docker to internal operations/dev/IT clients, while at the same time as learning SaltStack (which uses Vagrant for tutorials). So I'm realizing now this isn't as clear cut as I thought it was. This was particularly frustrating, because I wasn't able to complete a Docker demo to my internal clients because of crashes.

@johndmulhausen I'm closing this ticket as un-reproducible. It appears it has something more to do with Vagrant.

Sorry to revive this, but I have the same issue. Docker with Hyper-v disabled used to work fine before I updated to the latest version. Now it requires Hyper-v but causes a BSOD.
Version: 17.09.0-ce-win33 (13620) running on windows 10
Any ideas?

I also have this issue! Description: https://www.virtualbox.org/ticket/16801

@librisius & I are having the same problem: https://www.virtualbox.org/ticket/16801

Sorry to reopen but i'm  having the sam issue. I'm using windows insider preview and docker desktop. Whenver i use docker desktop and then close docker desktop and then reboot as soon my pc reboots it goes into a BSOD HYPERVISOR_ERROR. I'm 100% sure it is docker causing the issue. please help

same thing after increasing ram to 5 gb and giving 4 processor threads in the docker settings


fix(ui): add default coherence mode to generation slice migration

Summary

The valid values for this parameter changed when inpainting changed to gradient denoise. The generation slice's redux migration wasn't updated, resulting in a generation error until you change the setting or reset web UI.
Related Issues / Discussions
Multiple discord reports:

https://discord.com/channels/1020123559063990373/1149506274971631688/1224769870135033937
https://discord.com/channels/1020123559063990373/1149506274971631688/1224828774504468561

QA Instructions
You can test it by booting up 3.7.0 and setting the coherence mode, then upgrade to 4.0.0. We expect an error when you inpaint on canvas, until you reset web UI or change the setting once.
This PR fixes it so the default value is correct and you don't have to reset or change the setting
Merge Plan
n/a
Checklist

[x] The PR has a short but descriptive title, suitable for a changelog
[ ] Tests added / updated (if applicable) n/a
[ ] Documentation added / updated (if applicable) n/a

@blessedcoolant I added data migration capabilities to our redux slices, so if you need to change the valid values for something you can also change it in the migration to prevent UI errors when users update. No big deal just letting you know this is possible now


[flutterfire_ui] add support for email link sign in on web

Any plans to expand email link sign-in to web (currently only iOS and Android)?
Originally posted by @wolframm in https://github.com/FirebaseExtended/flutterfire/discussions/6978#discussioncomment-1804939

I hope for web support.
By the way, to support web, two things are necessary: the redirect URL inside DynamicLink and the entered email address. The URL can be obtained with GoRouter or similar, and for retrieving the entered email address, I've added a callback to EmailLinkSignInScreen.
I consider this a temporary solution, so I haven't created a pull request for it, but please refer to it if you're looking for a simple implementation.
https://github.com/ywake/FirebaseUI-Flutter/tree/email-link-web
diff

I didn't realize this won't work for webApps at the moment. Will this be taken up?


Fedora support

This enables booting Fedora in the virtual cluster and installing arbitrary Kubernetes versions on it. The CI gets switched over to Fedora to get rid of the hack with testing on old, obsolete Clear Linux versions.

I pushed the source branch for this PR to the main repo to get it tested completely by the CI:
https://cloudnative-k8sci.southcentralus.cloudapp.azure.com/blue/organizations/jenkins/pmem-csi/detail/fedora/1/pipeline

Looks good so far - 1.14 on Fedora passed, 1.13 is came up.

Two random failures in "direct mode":
https://cloudnative-k8sci.southcentralus.cloudapp.azure.com/blue/rest/organizations/jenkins/pipelines/pmem-csi/branches/PR-425/runs/2/nodes/82/steps/91/log/?start=0
PMEM Volumes [Driver: pmem-csi] [Testpattern: Dynamic PV (default fs)] provisioning [BeforeEach] should allow concurrent writes on the same node - failed while waiting for the namespace to be created. The test itself would have been skipped. From the logs one can see that around 20:11:36 the PMEM driver on worker-3 reconnected to master, so it looks like something wasn't properly working in the cluster.
Same with the failed "ValidateVolumeCapabilities should fail when no volume id is provided" (https://cloudnative-k8sci.southcentralus.cloudapp.azure.com/blue/rest/organizations/jenkins/pipelines/pmem-csi/branches/PR-425/runs/2/nodes/82/steps/87/log/?start=0): at 19:40:23, we can see an "etcdserver: request timed out" error.

Same failure in next attempt.

More memory seems to have helped.

@okartau I think we can merge this now if you agree.


Applied modified version of libocpp patch

Describe your changes
As per the comment here, this PR applies the libocpp.patch file to the charin-e2e-demo-with-revisions branch.
Issue ticket number and link
Relates to this issue.
Checklist before requesting a review

[X] I have performed a self-review of my code
[X] I have made corresponding changes to the documentation
[ ] If OCPP 2.0.1: I have updated the OCPP 2.0.1 status document
[X] I read the contribution documentation and made sure that my changes meet its requirements

@louisg1337 can you fix the formatting?


Fix up swift stepping in the new mangling

I got the swift stepping test running with an intermediate version of the new mangling, but something changed on the swift side and broke these tests.  This gets them running again.
The stepping out also ran into some additional difficulties because we have debug info for the swift standard library, which the stepping tests also have to successfully get out of.  I fixed a few places in the step out from here generation to make this more robust.

Explanation: lldb relies on the demangled forms of various inter-call thunks to figure out how to follow them into the actual function being called.  The logic for doing this was change for Swift 4.0.  lldb was changed to accommodate those changes but the swift side hadn't settled down, and subsequent changes broke the stepping test cases we had.  This PR gets those tests back to passing.
Scope: This change has two parts.  One was to accommodate the new mangling.  That code just makes correct the decoding of the thunks & figuring out what to do with them.  The other part is to make the procedure used to walk out of places we decide we don't want to stop in simpler when possible, so that it would be more robust.
Radar (and possibly SR Issue): rdar://problem/31515444
Risk: This is a complex area of lldb, so touching it is never going to be low risk, but these changes are pretty straightforward changes, and there are a lot of tests for stepping in the lldb testsuite.  So this is a low-medium risk change.
Testing: The failing tests now pass, and none of the other lldb stepping tests were affected.

@swift-ci please test


Draft: export thumbnails

Summary
PALS would like to contribute back the following work:

The ability to export thumbnails, similar to the way files get exported.

Screenshots
From the export form, if a user checks to include thumbnails the csv will get exported with a "thumbnail_file" column header and filenames as its values. The actual files/thumbnails will be included in the files directory of the exported zipped file.

TODO: add specs


Added Copy button to the codeblock

Added Copy button to the codeblock for better accessibility.
Completed issue no. #5

Good work @Savio629


After installing patch to activate full link, only mirror link shows up not carplay or android auto

I installed the patch to unlock full link, which it did, but it is only showing mirror link as an option not android auto or apple carplay.
This is on a Seat leon 2016 with the 6.5inch MIB2, software version 0245.
Thank you for any assistance!

Set AppleDIO=on and GAL=on in adaptations of block 5F.


Visualize unread notifications and mark them as read with fzf

Description
feature

Add a little colored dot next to notifications that are unread
add an hotkey to mark them as read with fzf

note

[ ] builds upon #19

GIF

Draft reason
Marked as draft because I'm not sure the line at the end is a reasonable idea. (Probably not)
gh notify "$Args"

I want fzf to stay open when marking notifications as read.
fzf offers a reload action and I managed to get it to work with the function below.
# list your starred repos, allow to unstar with fzf
function ghst {
	FZF_DEFAULT_COMMAND='gh api user/starred --template "
	{{- tablerow (\"REPO\" | color \"yellow+b\") \"⭐\" (\"LAST PUSH\" | color \"yellow+b\") (\"DESCRIPTION\" | color \"yellow+b\") -}}
	{{- range . -}}
		{{- tablerow (.full_name | color \"cyan+h\") (.stargazers_count | color \"yellow+h\") (timeago .pushed_at | color \"gray+h\") .description -}}
	{{- end -}}"' \
		GH_FORCE_TTY=100% fzf --ansi --no-multi \
		--header $'CTRL+B - Browser View | CTRL+F - Toggle README\nCTRL+U - Unstar       | CTRL+R - Toggle Release' \
		--header-lines=1 --preview "gh repo view {1}" --preview-window hidden \
		--bind 'enter:accept+execute(clear)' \
		--bind "ctrl-b:execute-silent(gh repo view {1} --web)" \
		--bind 'ctrl-f:toggle-preview+change-preview(gh repo view {1})' \
		--bind 'ctrl-r:toggle-preview+change-preview([[ $(gh release list -R {1}) ]] && gh release view -R {1})' \
		--bind 'ctrl-u:execute-silent(gh api -X DELETE /user/starred/{1})+reload(eval "$FZF_DEFAULT_COMMAND")'
}

I have not cracked the solution for an implementation with gh notify.
The line would look similar to the following.
	--bind 'ctrl-m:execute-silent(gh api -X PUT notifications -F read=true --silent)+reload(...)'

see #21


Unhandled: read ECONNRESET on windows target

Environment

Server - Azure Pipelines
Private Project, Hosted VS2017

Issue Description
The Copy Files Over SSH Task fails in my release pipe with the error you can see below.
All the files are transferred without problems and after that the task still fails.
I've set it to continue on error but now every release finishes with a warning and I don't get an error if something actually fails.
I copy to a windows machine running OpenSSH.
When I select the option to clear the target folder before copying I also get the error that the "rm" command cannot be found. (See second log excerpt below, partially in German, sorry)
Error logs
2018-12-18T16:44:27.9381300Z Completed copying 5 files to the remote machine.
2018-12-18T16:44:27.9926546Z ##[error]Unhandled: read ECONNRESET
2018-12-18T16:44:27.9978712Z ##[section]Finishing: Securely copy files to the remote machine
2018-12-18T18:34:21.9916863Z ##[error]Der Befehl "rm" ist entweder falsch geschrieben oder
2018-12-18T18:34:21.9925004Z konnte nicht gefunden werden.
2018-12-18T18:34:21.9925069Z
2018-12-18T18:34:21.9957154Z ##[error]Failed to clean the target folder on the remote machine. Command rm -rf "C:\web\www\fhwebshop"/* exited with code 1.
2018-12-18T18:34:22.0336089Z ##[error]Unhandled: read ECONNRESET
2018-12-18T18:34:22.0475422Z ##[section]Finishing: Securely copy files to the remote machine

This issue is also tracked here:
https://developercommunity.visualstudio.com/content/problem/380609/release-pipeline-copy-files-over-ssh-task-fails-ev.html

In our case the Use NuGet version task fails on the build (log output below).  Is there any fix for this yet?
##[section]Starting: Use NuGet 5.3.1
Task         : NuGet tool installer
Description  : Acquires a specific version of NuGet from the internet or the tools cache and adds it to the PATH. Use this task to change the version of NuGet used in the NuGet tasks.
Version      : 0.158.0
Author       : Microsoft Corporation
Help         : https://docs.microsoft.com/azure/devops/pipelines/tasks/package/prev-versions/nuget-installer-0
ERR:read ECONNRESET
##[section]Finishing: Use NuGet 5.3.1


APP-207: Add support of new API for Tenant and User management

Can one of the admins verify this patch?

@kaa-leeroy add to whitelist

LGTM

@kaa-leeroy retest this please


Interactive push test app & instructions

See https://github.com/ably/ably-java/issues/486 and https://github.com/ably/AndroidPushExample.
Given push tests need to be tested interactively with an actual device, we should make sure we have an app with the same functionality and instructions on how to test the app, along with details in the README of this repo.  This may exist in https://github.com/ably/push-test-ios.
@paddybyers can you please make sure @ricardopereira knows what is needed to ensure we have consistent interactive push testing.

@Srushtika is handled this in https://github.com/ably/iOSPushExample.


OSE ARGOS ENT

Hi,
Could you add  the OSE ARGOS Ent in bordeaux academic?
I connect myself on the following https://ent2d.ac-bordeaux.fr/argos/pr/index/index#
And after that i click on the Pronote link tht connect me.
If you need any thinks as data, or if you want that i try something i'm available!
Regards,

Hi You can use the function ent2d_bordeaux . Please look at the documentation before creating an issue!

It does not work as parent.
It should be fixed with PR #183


Add new Task to run a UEFI application (if compute is running in UEFI)

depends on : https://github.com/RackHD/on-http/pull/201
depends on : https://github.com/RackHD/on-taskgraph/pull/81

@RackHD/corecommitters @tannoa2 @uppalk1 @zyoung51

Addressed @benbp  Comments

*** BUILD #461 ***

:

:+1: after my two questions are either by clarified or fixed.

+1

+1 after the file encoding issues are fixed:
file -i lib/jobs/uefi-job.js
lib/jobs/uefi-job.js: text/x-c++; charset=utf-8
Everything else in the repo is us-ascii encoded.

@zyoung51  all set!!

:+1:

:+1:


Improving performance of generic map copy

The GenericMap's Copy method is run in diverse stages of the pipeline, for each processed flow.
Preallocating the map size to the expected resulting size can improve Copy CPU performance and memory allocations by 40%-50%.
Benchmark results with 20-entries maps (before vs after):
BenchmarkGenericMap_Copy-12       658083              1671 ns/op            1852 B/op          2 allocs/op
BenchmarkGenericMap_Copy-12       978686              1082 ns/op            1199 B/op          1 allocs/op

Benchmark results with 30-entries maps (before vs after):
BenchmarkGenericMap_Copy-12       354102              3280 ns/op            4337 B/op          4 allocs/op
BenchmarkGenericMap_Copy-12       633978              1758 ns/op            2324 B/op          1 allocs/op

@jpinsonneau if you know the approximate length of the resulting map, yes (same for slices).
If you don't know the length, doing m := map[string]string{} is commonly preferred.


checkov: 2.0.659 -> 2.0.660

Motivation for this change
https://github.com/bridgecrewio/checkov/releases/tag/2.0.660
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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/)
22.05 Release Notes (or backporting 21.11 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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 150831 run on x86_64-linux 1

1 package built:

checkov


[SPARK-21519][SQL] Add an option to the JDBC data source to initialize the target DB environment

Add an option to the JDBC data source to initialize the environment of the remote database session
What changes were proposed in this pull request?
This proposes an option to the JDBC datasource, tentatively called " sessionInitStatement" to implement the functionality of session initialization present for example in the Sqoop connector for Oracle (see https://sqoop.apache.org/docs/1.4.6/SqoopUserGuide.html#_oraoop_oracle_session_initialization_statements ) . After each database session is opened to the remote DB, and before starting to read data, this option executes a custom SQL statement (or a PL/SQL block in the case of Oracle).
See also https://issues.apache.org/jira/browse/SPARK-21519
How was this patch tested?
Manually tested using Spark SQL data source and Oracle JDBC

Can one of the admins verify this patch?

test this please

Could you add a test case to JDBCSuite?

Test build #80065 has started for PR 18724 at commit 92d082a.

Test build #80065 has finished for PR 18724 at commit 92d082a.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80065/
Test FAILed.

Merged build finished. Test FAILed.

ping @LucaCanali

Thank you very much @gatorsmile for the review. I plan to provide the required changes and add a test case, however it is probably going to take one more week before I can do that.

Thanks for your time!

ok to test

Test build #80505 has started for PR 18724 at commit 0a0ff0a.

Test build #80507 has started for PR 18724 at commit 55e63a3.

Test build #80505 has finished for PR 18724 at commit 0a0ff0a.

This patch fails SparkR unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80505/
Test FAILed.

Test build #80507 has finished for PR 18724 at commit 55e63a3.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80507/
Test PASSed.

Test build #80539 has started for PR 18724 at commit 5792fd6.

LGTM

Test build #80539 has finished for PR 18724 at commit 5792fd6.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80539/
Test PASSed.

Thanks! Merging to master.


Case sensitive issues

we noticed the original kube-oidc project was case sensitive with reference to user names..
i.e A user is authenticated via the provider and the flow goes to authorization and oidc is reading the user ID lets say<EMAIL_ADDRESS>from provider and if user logs in with<EMAIL_ADDRESS>authorization of kube api fails
Any help on this is highly appreciated

Can you please provide a specific example of what the issue is?  Pleas eprovide specific API calls, JWT claims, and corresponding RBAC rules with your expectations and actual results.

closing due to lack of response


lastStatedTtl Handling

Hi!
Is it possible to disable the lastStatedTtl handling? In our setup, all records get a TTL if one record has a TTL set.
Thanks!

@ddimension - Thanks for your issue, I will have a look into it shortly.

Could you please provide an example of the desired functionality?


Don't always codegen "unknown" visitors

In some cases, "unknown" is not necessary and annoying to have to deal with. Some examples:

When implementing a server, you shouldn't have to deal with unknown. Only clients need to handle unknown. (Assuming server is always ahead of client).
When using model mode. e.g. for db-api.
When the type is only used in requests. Since they're always constructed client side, the client never will receive a maybe-unknown value.
When you just know the list won't grow, e.g. for State (AL, AK, AZ, ...). Ofc if we ever did add a 51st state, you'd just have to make a StateV2.

(1) and (2) can't be in the IR because it depends on which generator is being run.
(3) the generator could do but it should probably live in the IR to avoid repeated work in the generators.
(4) would require YAML + IR changes.

this is now fixed


Switch node-uuid to uuid

node-uuid has recently been merged into the uuid module and the node-uuid module is no longer maintained. I suggest switching over to node-uuid (I could prepare a pull request if necessary). Any objections?

Please send a PR!


Design database schemas and developer database adapter

Discussed in https://github.com/ubiquity/bounty-bot/discussions/1

Originally posted by pavlovcik September 10, 2022
We're gonna plan to use supabase which is open-source firebase alternative. This is originally a postgres database.
The tables we have to basically have are

[ ] user table
[ ] issue table
[ ] payment table

The database adapter is sort of a high level interface for CURL operations.

kicking off this issue right now.

Please share your wallet address here @0xcodercrane

@pavlovcik This is my wallet address: 0x00868BB3BA2B36316c2fc42E4aFB6D4246b77E46

@pavlovcik This is my wallet address: 0x00868BB3BA2B36316c2fc42E4aFB6D4246b77E46

https://etherscan.io/tx/0x74dd4fc9c9bc64c83d556a68166967d39feb519b18b54b810131a1c765296156


31 expand the search for steam accounts by adding cosplay steamid

[x] updated the algorithm search
[x] added auto run with admin privileges


build: fix build failures on manpages/bash-completion target due to missing GOPATH

Pass expected variables to build commands and prefer them over hardcoded ones. Also, git ignore dist directory with build artifacts.

Sorted .gitignore while at it.

:+1: thanks!
LGTM if it works


update the readme file

update the readme file with proper contribution guidelines which is an important part of it.

I want to work on this issue under JWOC @Gaurav-Verma07

Resolved with PR #23


Remove custom options

I've wasted time thinking preserveComments was an option for uglify when all along it was a specific thing added to gulp-uglify all because the options are all dumped together without clean headers or separation between what is what :/
The time I've wasted will not return but at least I might save somebody's else time by opening a ticket about this.

Was trying to use the same options for gulp-uglify and for rollup-plugin-uglify and couldn't understand why Gulp crashes all the time. I dug and dug until I've found that rollup-plugin-uglify doesn't know what preserveComments means.

In version 3.0.0, I'll be removing custom options in favor of one defined upstream with any remaining options specifically grouped in a key for gulp-uglify.

@yairEO I've removed all the custom options in my "three" branch, which will be the next major version.
I'm closing this issue since this work is now done.


Climate entity incorrectly showing 'Cooling to Target'

First of all: thanks for the great work on this integration!
There is a small issue with the climate entities showing 'cooling to target' when cooling is enabled for that room. The controller API response is misinterpreted because "cooling": true and "coolingEnabled": true merely indicate that the room is allowed to use cooling. It does not mean that cooling is currently active for the room. It also does not mean that if "cooling" is false that the room is heating to the set target.
To solve this, I'd suggest updating the logic in climate.py
    def hvac_action(self) -> HVACAction | None:
        if not self.thermostat.cooling_enabled:
            return None

        if self.thermostat.cooling:
            return HVACAction.COOLING

        return HVACAction.HEATING

This whole logic could probably be removed as it provides no value. The default HVAC action of 'Auto' is correct: it simply shows the current and target temperature, the controller and gateway have no information if cooling or heating is currently active to achieve the target temperature.

Thanks @arnebret !
And thank you very much in clarifying the behaviour of the API. Interpreting an API without documentation is very challenging.
I removed the hvac_action function in a new pull request. Can you have a look to verify?

@arjenbos Looks good, all climate entities now show 'Auto'. Don't see any errors in the logs while testing and all works as supposed to.
PS: I mentioned in my original post that the controller and gateway have no information on the cooling/heating status, but that's not strictly true. The information is there (you can see it in the 'xpert only' panel on the controller), but it's not available on any of the endpoints I can find or that your integration is using. Would be a 'nice to have' to add if we ever find an endpoint that has this status available.

Thanks for testing @arnebret ! I will release a 1.0.1 version with the bug fix.
I will have a look at the expert panel. The API endpoint that's used in the expert panel is a new adventure. I need to "reverse engineer" the used javascript to generate the requests (without damaging heat pumps).


ldtk: 1.3.3 -> 1.3.4

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for ldtk is: Modern, lightweight and efficient 2D level editor
meta.homepage for ldtk is: https://ldtk.io/
meta.changelog for ldtk is: https://github.com/deepnight/ldtk/releases/tag/v1.3.4
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.3.4 with grep in /nix/store/1j4k4g9h0drr87nl9h8f822z1zcl93fz-ldtk-1.3.4
found 1.3.4 in filename of file in /nix/store/1j4k4g9h0drr87nl9h8f822z1zcl93fz-ldtk-1.3.4

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
ldtk

Instructions to test this update (click to expand)

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

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

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

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

1 package built:

ldtk

Maintainer pings
cc @felschr for testing.

Result of nixpkgs-review pr 252479 run on x86_64-linux 1

1 package built:

ldtk


Add query semaphore

This adds a semaphore to limit the concurrent execution of graphql queries and subscriptions. This semaphore is implemented in the LoadManager and is waited on with the query_permit method. Statistics were added so that we may track this as we currently track wait times for DB connections.
Limiting the cuncurrent queries prevents an increase in resource usage when the DB is contended. The request will wait for a permit before spawing blocking tasks and requesting DB connections, so waiting requests will consume less resources.
I experimented with different locations for the permit. It turns out that spawning less blocking tasks not only reduces the memory consumption but also reduced tokio contention in local tests. I'm not sure why that is the case, since blocking tasks run on a separate thread pool, but good news is good.
This also does some general refactoring for better use of async.

There is however an undesirable aspect to this which is that the permit is most effective before the herd cache, making the herd cache potentially less effective. Ideally we'd have things be async all the way to the herd cache, I'll see how close we can get to that.

I've revamped this to put the permit after the caches have been checked, which is the ideal. This required making execute_root_selection_set an async fn, which took a bit of refactoring.
@That3Percent the herd cache is now async, which is much easier to implement since Shared futures do the hard part for us. Could you review that part?

@lutter Thanks for the review, I've addressed it.

@lutter I've done one necessary change which is to also check the semaphore wait time in the overload check.

@lutter thanks for the review and for doing the rebase!


build: test against PHP 8.4

PHP 8.4 was released yesterday. We can test it.

@grogy Thank you for this PR! Can you perhaps also update the other configuration files (coding standard and static analysis) in this PR? Then we have a complete set.

@stelgenhof I updated the rest of files. But still is issue with waiting MR from https://github.com/vimeo/psalm/pull/10928


Using mod as a dependency causes issues with runData in IDE

I'm working on a mod that uses this mod as a dependency, but when I try using runData in my IDE the instance will crash due to a ResourceLocationException. I've tested it with runClient and runServer as well but I don't get the error then which leads me to believe it's only an issue with using runData.
latest.log

Try removing '--all' from data run args

Make sure your datagen run args has '--mod', mod_id

Yeah, I have that but it still doesn't work. I'll upload my build.gradle to paste bin for you to check. I also tried using the official maven repo as well thinking cursemaven was screwing something up but that didn't fix anything either.
https://pastebin.com/y7QRLfxf


feat: Add CollectionExtensions

Hey,
what do you think about this approach to gain some performance improvements? We are using Linq in some cases where it's not the best option regarding performance. With this little trick, we get a performance improvement as these Any methods will be used instead of the default Linq method. I think there are some more methods that we could add. We don't have to change existing code to make use of this.

Method
Mean
Error
StdDev
Allocated

Array_Any
1,165.1 ns
351.8 ns
19.28 ns
32 B

Array_Exists
508.8 ns
249.2 ns
13.66 ns
-

List_Any
1,020.8 ns
342.4 ns
18.77 ns
40 B

List_FindIndex
518.8 ns
201.7 ns
11.06 ns
-

Type of Changes

[x] New feature (non-breaking change which adds functionality)

Checklist

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.
[ ] I've added relevant tests.

@henon @ScarletKuro What's your opinion on that?

Fyi performance tag is probably more suitable than enhancement so it shows up differently in the changelog

With this little trick, we get a performance improvement as these Any methods will be used instead of the default Linq method

I don't see any usage of Any(predicate) where the type is array or list in the core.

Fyi performance tag is probably more suitable than enhancement so it shows up differently in the changelog

Thanks. I didn't know that such a label existed. I'll try to use the right labels in the future 👍

With this little trick, we get a performance improvement as these Any methods will be used instead of the default Linq method

I don't see any usage of Any(predicate) where the type is array or list in the core.

There are not many usages, but there are some:

I also saw that a private field is declared as IList instead of List.  In such a case, the new extension method is not available.

There is a SonarCloud C# rule for this, that's where I had the idea from.

Why are they internal and not public ?


Create a scrapping script to add all the yml files

Description
The jupyter notebook Tutorial.ipynb provides the method to create a yml file from the json description of a FHIR resources like Patient. The index of the resource is available here.
It would be very valuable to automatise this process.
I've already fixed the pb we had connectif to the hl7 website.
Fixes #2
Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)
[x] This change requires a documentation update

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation if any
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective, if unittests are enabled
[ ] New and existing unit tests pass locally with my changes, if any

Hey that's really good!
I have a question: is it inscriptis.get_text which remove the line breaks in the .json files? We really need to keep them to be able to clean the file later.


set frontView backgroundColor to transparent?

In some of my apps I'm using a  to apply background color to the "main content" but this style(https://github.com/Kureev/react-native-side-menu/blob/master/styles.js#L26) overrides that tag.
When I set the backgroundColor to transparent on frontView I'm able to use the LinearGradient correctly.

Fixed in gh repo, next release will contain that it.


How should I merge the wslexe into wslgit

Hi!
Days ago I forked this project and did some changes to it.
First I changed the code to use file stem name as the command rather than git only.(python.exe for python, php.exe for php, etc)
Second , as different command may use different environment as interactive or not , I separated it into two binary files.
Third, more to the second reason, I made the command to source an environment script before executing the real command. This is useful for programs like python in virtualenv.
As the wslgit is only made for git, should I submit a PR to merge my code ?
my project: jswh/wslexe

I have no plans to support any other tools besides git. So please do not file a PR, as I will not merge it. However, I guess it could be a useful tool for many, so you may want to continue working on it in your own repository.


Use square brackets to look up group.

This change makes it so that square brackets, rather
than dot notation, is used to look up groups. It is
common for group names to have hyphens in them, which
breaks when using dot notation.

This repository has chosen to disable pull requests.

@wtcross please send pull requests to https://github.com/ceph/ceph-ansible


refine malachite metrics to unified units

What type of PR is this?
bug fixes
What this PR does / why we need it:
unify malachite metrics to a standard

CPU usageRatio represents the usage ratio (number of cores). What is the meaning of dividing it by 100,

which one?


Use container metrics enum more places

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.

@kkraune or @gjoranv or @olaaun Please review and merge


[FLINK-36425][refactor] Replace jobStatus.state string with Flink JobStatus enum

What is the purpose of the change
Use / enforce the Flink JobStatus enum directly in job status.state this how we already use it and it would simplify the code in a lot of places to avoid string juggling .
Verifying this change
Tests have been updated to use/expect the enum and e2es should further cover existing behaviour.
The change have been manually verified with multiple envs and Flink versions.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changes to the CustomResourceDescriptors: yes
Core observer or reconciler logic that is regularly executed: yes

The change looks trivial and good. What I can't justify is why RestApiMetricsCollectorTest failed. Is that flaky?

Yes that test is flaky and not related/dependent on these changes.


Class cannot be read... using a custom type

I'm currently working with Joda-Time and any fields declared as DateTime appear to be stored as strings in the database. I subsequently cannot retrieve the value on read. On closer examination, the table structure shows a blank value in the type column using SQLite Manager.

I'm working towards a workaround where I ignore the datetime field and sync it with a string field that gets persisted, but I'm wondering if it is feasible to add a directive to 'save as' to a given type (say @AsString) in the SugarRecord declaration.

This is a great idea for an annotation. Marking this as a future future release. 1.4.0 and 1.5.0 look pretty loaded at the moment.


Average calculation in legend

First at all, thank you very much for this plugin @mtanda, it is very useful.
I noted a problem regarding average calculation in the legend. In my case I should have a voltage value close to 230 V, but Grafana shows me out-of-range and non-sense values (greater than 3000 V). Instead maximum and minimum work fine.

I think to have found the problem. The average calculation is bound to the bucket size parameter. If I change it, also the average changes.

Thanks for report. I found a bug. I'll fix it.

I fix the bug, I'll register new version to grafana.net.


Minor spacing fix in explain-machine-learning-model-permutation-feature-importance-ml-net.md

Adds a space:

by Breiman's Random Forests paper(see section 10)

by Breiman's Random Forests paper (see section 10)

Thank you @justinormont !


Server does not support Redis TLS Connection

Describe the bug
The server is unable to connect to a redis instance that requires TLS. Digital Ocean managed redis requires TLS and livekit gives the following error when trying to start: unable to connect to redis: EOF. DO has a post about this
Server

Version: [v0.15.4]
Environment: [DO Kubernetes, DO managed Redis]

To Reproduce
Steps to reproduce the behavior:

Deploy a livekit-server instance to DO Kubernetes
Create a DO managed redis instance
Get the pod logs for the livekit-server pod and observe the EOF error.

Expected behavior
The server should connect to redis over TLS if it is specified in the configuration.
Additional context
I have tested the connection to redis with redis-cli and the database is fine. When I omit the --tls flag from the redis-cli command ping fails in the same way as in livekit. Something like this can be implemented.

Thanks for the detailed note. Sounds like the work-around is to accept TLS without validation?
That sounds fine to me. Feel free to make a PR!


feat(editor): list for read-only article

Codecov Report

Merging #283 (5d13f1d) into dev (02a934f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev     #283   +/-   ##
=======================================
  Coverage   87.12%   87.12%
=======================================
  Files         112      112
  Lines        1359     1359
=======================================
  Hits         1184     1184
  Misses        175      175

Impacted Files
Coverage Δ

lib/helper/utils.ex
80.00% <ø> (ø)

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 02a934f...5d13f1d. Read the comment docs.


various refactorings

Made all noise sampling functions "const"
Added getters for all the setters
Fix typos in the comments
turn m_cellularJitter from "float" to "FN_DECIMAL"
Made everything that was "protected" become "private"
fixed double->float narrowing conversion warnings in VC++ for the LUTs
fixed other miscellaneous double->float narrowing warnings
use std::min/max instead of fmin/fmax for cellular distances
fix function name typo: SetCellularDistance2Indicies -> SetCellularDistance2Indices

Thanks for cleaning it all up


Unable to scroll or select in

Here is my sample component:
import React, {Component} from 'react'
import faker from 'faker'

const bs = () =>
  `${faker.commerce.productName()} is incredible ${faker.company.bs()}`

export default class App extends Component {
  render() {
    const items = Array(50).fill().map(bs)
    return (
      <element>
        <list
          label="List"
          class={{
            border: { type: 'line'},
            style: { border: { fg: 'blue' } },
          }}
          width="50%"
          height="50%"
          left="25%"
          top="25%"
          style={{
            item: { fg: 'black' },
            selected: { fg: 'white', bg: 'black' },
          }}
          items={ items }
          onSelect={ () => console.error('selected') }
        />
      </element>
    );
  }
}

And the result, as expected:

The only drawback is that there is no interaction, I tried with arrows, mousewheel, vi shortcuts, with or without vi property… Nothing moves :( If I add clickable={ true } onClick={ () => console.log('clicked') } the "clicked" messages appear normally, but still nothing about scrolling or selecting item.
Is there some extra configuration I missed?

I found the solution : add mouse={ true } and keys={ true } to enable all the mess :)
That may desserve a line in README ;)

Yup. blessed is somewhat cryptic at times and its documentation does not help to spot those usual issues.

That may desserve a line in README ;)

Well I will gladly merge a PR ;)

Close ticket?


Add new line on json formatter

Add new line after each log on json formatter (GrapeLogging::Formatters::Json)
Before:
{"date":"2018-08-24T19:56:17.285+09:00","severity":"INFO","data":{"status":200,"time":{"total":0.59,"db":0,"view":0.59},"method":"GET","path":"/ping","params":{},"host":"localhost"}}{"date":"2018-08-24T19:56:20.920+09:00","severity":"INFO","data":{"status":200,"time":{"total":0.48,"db":0,"view":0.48},"method":"GET","path":"/ping","params":{},"host":"localhost"}}{"date":"2018-08-24T19:57:41.173+09:00","severity":"INFO","data":{"status":200,"time":{"total":0.37,"db":0,"view":0.37},"method":"GET","path":"/ping","params":{},"host":"localhost"}}
...

After:
{"date":"2018-08-24T19:56:17.285+09:00","severity":"INFO","data":{"status":200,"time":{"total":0.59,"db":0,"view":0.59},"method":"GET","path":"/ping","params":{},"host":"localhost"}}
{"date":"2018-08-24T19:56:20.920+09:00","severity":"INFO","data":{"status":200,"time":{"total":0.48,"db":0,"view":0.48},"method":"GET","path":"/ping","params":{},"host":"localhost"}}
{"date":"2018-08-24T19:57:41.173+09:00","severity":"INFO","data":{"status":200,"time":{"total":0.37,"db":0,"view":0.37},"method":"GET","path":"/ping","params":{},"host":"localhost"}}
...

@aserafin
I wanna use this PR.
Plz merge it !


feat: add sonar support for ruby

Fixes
Closes : DII-751,
setting up sonar monorepo for ruby
Checklist

[x] I acknowledge that all my contributions will be made under the project's license
[ ] Run make test-docker
[ ] Verify affected language:

[ ] Generate twilio-go from our OpenAPI specification using the build_twilio_go.py using python examples/build_twilio_go.py path/to/twilio-oai/spec/yaml path/to/twilio-go and inspect the diff
[ ] Run make test in twilio-go
[ ] Create a pull request in twilio-go
[ ] Provide a link below to the pull request

[ ] I have made a material change to the repo (functionality, testing, spelling, grammar)
[ ] I have read the Contribution Guidelines and my PR follows them
[ ] I have titled the PR appropriately
[ ] I have updated my branch with the main branch
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added the necessary documentation about the functionality in the appropriate .md file
[ ] I have added inline documentation to the code I modified

If you have questions, please create a GitHub Issue in this repository.

You may want to add sonar.ruby.coverage.reportPaths to sonar.properties

done

List, Page, Context & Instance operations are implemented, is it okay to merge this now ?


Fix todo example

I encountered an issue in a project I'm contributing to: gitStream would request changes saying my PR has a todo in it, even though I didn't add any todos. I asked another team member if they know what the issue is, and they figured it out: our todo regex was copied from the example in the docs, and that example is incorrect. The current example:
/^[+].*(TODO)|(todo)/

It matches:

either a line starting with a plus sign, followed by any amount of any characters, followed by TODO (uppercase)
or it matches any line that has todo (lowercase) in it, ignoring the ^[+].* part

Because of the second part, it reacted to a todo comment I had deleted. This PR fixes this issue by placing both variants (TODO and todo) inside the same group.
Demo on regex101: before the change, after the change

Thanks for the fix @Yusyuriv This is super helpful!

Thank you Yan!