Java 13.2: Finding Packages and CLASSPATH

Description
Add audio, documentation, and video regarding the Finding Packages and CLASSPATH.
Location of File to be added
The files should be added inside the Finding Packages and CLASSPATH folder which is inside the Packages folder present in the Java main folder. The documentation file can be in the .md/.py/.ipynb format. Audio should be in .mp3 format and to be submitted through a Google drive link. Video can be in any format and should be submitted through a Google Drive link.
Note:

You are required to comment with \assign CONTENT_TYPE, so that other content types can be created at the same time.
Failing to comment in the above manner will lead to the removal of assignment from the issue even if the bot assigns.
You are required to do a PR within 7 days otherwise you will be unassigned and others will be assigned.
Issues will be assigned on a first come first serve basis.

Domain
Java_Domain
Type of Contribution
Audio, Video, Documentation
Code of Conduct

[x]   I follow Contributing Guidelines & Code of conduct of this project.

\assign Document

\assign Documentation

\assign Document

\assign Document

@shoebxsiddiqui go with documentatio

@shoebxsiddiqui go with documentation

Yes sir

#3360

#3360
@garvitraj done with the PR sir

#3525 @garvitraj


Feature: Send env info message on safe apps sdk initialization

This PR:

Is required for https://github.com/gnosis/safe-apps-sdk/issues/29 to work, add ENV_INFO message sent on a handshake between the interface and an app
fix onUserConfirm handler not being called if the transaction is signed with an off-chain signature
Pass safeTxHash to offchain signer

Travis automatic deployment:
https://pr1349--safereact.review.gnosisdev.com/app

Travis automatic deployment:
https://pr1349--safereact.review.gnosisdev.com/app

Travis automatic deployment:
https://pr1349--safereact.review.gnosisdev.com/app

Travis automatic deployment:
https://pr1349--safereact.review.gnosisdev.com/app

Travis automatic deployment:
https://pr1349--safereact.review.gnosisdev.com/app

@rmeissner what is a failure of signature generation? The modal thing you wrote in slack?

Yes, the comment in slack was related to the other PR but applies here too :) basically right now if the user rejects the signature it will not notify the safe apps sdk (as far as I understand the code it would throw https://github.com/gnosis/safe-react/pull/1349/files#diff-622f1ade90af2b81fb05085d0f4345bbR34 and then it would not notify the safe app ... but I might be wrong :P)

Travis automatic deployment:
https://pr1349--safereact.review.gnosisdev.com/app

Travis automatic deployment:
https://pr1349--safereact.review.gnosisdev.com/app

Travis automatic deployment:
https://pr1349--safereact.review.gnosisdev.com/app


test duration won't be applied if same state is set twice

Repro steps
If we call vscode.TestRun.failed() twice, for example, at
https://github.com/microsoft/vscode-extension-samples/blob/e02887bb3198b674e138ddbeaf8e2001fb16f176/test-provider-sample/src/testTree.ts#L112
change it to:
options.failed(item, message, undefined);
options.failed(item, message, duration);

The duration information won't display in the test explorer. Seems that this is because the second call of .failed is ignored according to:
https://github.com/microsoft/vscode/blob/973e50f544dbfa6cd06dba557eb4d069d8e4d914/src/vs/workbench/contrib/testing/common/testResult.ts#L495-L497
Proposal
I think maybe we should either:

Document this detail in the API definitions.
Or, allow the extension call this API multiple times with the same status to append additional information.

Personally I prefer option 2, which allows extension to append more information to the test item as a test running session goes on.
Environment
Version: 1.64.0-insider (user setup)
Commit: 00f1ca47ada2fda8ee1437e17b27af5b4002cecb
Date: 2022-01-11T09:15:49.397Z
Electron: 13.5.2
Chromium: 91.0.4472.164
Node.js: 14.16.0
V8: 9.1.269.39-electron.0
OS: Windows_NT x64 10.0.19044

/verified


virtualbox: 5.2.8 -> 5.2.12 (18.03)

Motivation for this change
Backports VirtualBox 5.2.12 through cherry-picks since it is required for using under kernel 4.17.
Furthermore, having up-to-date software like VirtualBox seems like a good idea.
Things done
Dogfooding it here!
Tested it works both under 4.14 and 4.17.

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

✔ NixOS
🆖 macOS
🆖 other Linux distributions

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

 $ nix path-info -S $(nix-build -A virtualbox --no-out-link)
/nix/store/z8l8zpnv502n6yiqhpnyk2a3p9j4w7x9-virtualbox-5.2.12     758515504

@GrahamcOfBorg  build virtualbox linuxPackages.virtualbox

No attempt on aarch64-linux (full log)
The following builds were skipped because they don't evaluate on aarch64-linux: virtualbox, linuxPackages.virtualbox
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowBroken = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowBroken = true; }
to ~/.config/nixpkgs/config.nix.

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: virtualbox, linuxPackages.virtualbox
Partial log (click to expand)
while evaluating 'makeOverridable' at /private/var/lib/ofborg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/lnl7-mac/lib/customisation.nix:72:24, called from /private/var/lib/ofborg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/lnl7-mac/lib/customisation.nix:117:8:
while evaluating anonymous function at /private/var/lib/ofborg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/lnl7-mac/pkgs/os-specific/linux/virtualbox/default.nix:1:1, called from /private/var/lib/ofborg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/lnl7-mac/lib/customisation.nix:74:12:
while evaluating the attribute 'linux_4_14' at /private/var/lib/ofborg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/lnl7-mac/pkgs/top-level/all-packages.nix:13304:3:
while evaluating 'callPackageWith' at /private/var/lib/ofborg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/lnl7-mac/lib/customisation.nix:113:35, called from /private/var/lib/ofborg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/lnl7-mac/pkgs/top-level/all-packages.nix:13304:16:
while evaluating 'makeOverridable' at /private/var/lib/ofborg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/lnl7-mac/lib/customisation.nix:72:24, called from /private/var/lib/ofborg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/lnl7-mac/lib/customisation.nix:117:8:
while evaluating anonymous function at /private/var/lib/ofborg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/lnl7-mac/pkgs/os-specific/linux/kernel/linux-4.14.nix:1:1, called from /private/var/lib/ofborg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/lnl7-mac/lib/customisation.nix:74:12:
while evaluating 'makeOverridable' at /private/var/lib/ofborg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/lnl7-mac/lib/customisation.nix:72:24, called from /private/var/lib/ofborg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/lnl7-mac/pkgs/os-specific/linux/kernel/linux-4.14.nix:5:1:
while evaluating anonymous function at /private/var/lib/ofborg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/lnl7-mac/pkgs/os-specific/linux/kernel/generic.nix:9:1, called from /private/var/lib/ofborg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/lnl7-mac/lib/customisation.nix:74:12:
assertion failed at /private/var/lib/ofborg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/lnl7-mac/pkgs/os-specific/linux/kernel/generic.nix:41:1

Success on x86_64-linux (full log)
Attempted: virtualbox, linuxPackages.virtualbox
Partial log (click to expand)
  DEPMOD  4.14.54
make: Leaving directory '/nix/store/khs5al82iv9bn53hv0dbn6yl44yb1wc6-linux-4.14.54-dev/lib/modules/4.14.54/build'
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/al95rv6hf1xlgbaxrybi5al08zppq8rz-virtualbox-modules-5.2.12-4.14.54
strip is /nix/store/b0zlxla7dmy1iwc3g459rjznx59797xy-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/al95rv6hf1xlgbaxrybi5al08zppq8rz-virtualbox-modules-5.2.12-4.14.54/lib
patching script interpreter paths in /nix/store/al95rv6hf1xlgbaxrybi5al08zppq8rz-virtualbox-modules-5.2.12-4.14.54
checking for references to /build in /nix/store/al95rv6hf1xlgbaxrybi5al08zppq8rz-virtualbox-modules-5.2.12-4.14.54...
/nix/store/z8l8zpnv502n6yiqhpnyk2a3p9j4w7x9-virtualbox-5.2.12
/nix/store/al95rv6hf1xlgbaxrybi5al08zppq8rz-virtualbox-modules-5.2.12-4.14.54

@GrahamcOfBorg test virtualbox.simple-gui

No attempt on aarch64-linux (full log)
The following builds were skipped because they don't evaluate on aarch64-linux: tests.virtualbox.simple-gui
Partial log (click to expand)
Cannot nix-instantiate `tests.virtualbox.simple-gui' because:
error: attribute 'simple-gui' in selection path 'tests.virtualbox.simple-gui' not found

No attempt on x86_64-linux (full log)
The following builds were skipped because they don't evaluate on x86_64-linux: tests.virtualbox.simple-gui
Partial log (click to expand)
while evaluating ‘hydraJob’ at /var/lib/gc-of-borg/.nix-test-rs/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/rbvermaa-spot/lib/customisation.nix:162:14, called from /var/lib/gc-of-borg/.nix-test-rs/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/rbvermaa-spot/nixos/release.nix:29:28:
while evaluating the attribute ‘drvPath’ at /var/lib/gc-of-borg/.nix-test-rs/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/rbvermaa-spot/lib/customisation.nix:179:13:
while evaluating the attribute ‘drvPath’ at /var/lib/gc-of-borg/.nix-test-rs/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/rbvermaa-spot/lib/customisation.nix:146:13:
while evaluating the attribute ‘buildCommand’ of the derivation ‘vm-test-run-virtualbox-simple-gui’ at /var/lib/gc-of-borg/.nix-test-rs/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/rbvermaa-spot/pkgs/stdenv/generic/make-derivation.nix:148:11:
while evaluating the attribute ‘buildCommand’ of the derivation ‘nixos-test-driver-virtualbox-simple-gui’ at /var/lib/gc-of-borg/.nix-test-rs/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/rbvermaa-spot/pkgs/stdenv/generic/make-derivation.nix:148:11:
while evaluating the attribute ‘buildCommand’ of the derivation ‘nixos-vm’ at /var/lib/gc-of-borg/.nix-test-rs/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/rbvermaa-spot/pkgs/stdenv/generic/make-derivation.nix:148:11:
while evaluating the attribute ‘text’ of the derivation ‘run-nixos-vm’ at /var/lib/gc-of-borg/.nix-test-rs/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/rbvermaa-spot/pkgs/stdenv/generic/make-derivation.nix:148:11:
while evaluating anonymous function at /var/lib/gc-of-borg/.nix-test-rs/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/rbvermaa-spot/pkgs/build-support/closure-info.nix:9:1, called from /var/lib/gc-of-borg/.nix-test-rs/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/rbvermaa-spot/nixos/modules/virtualisation/qemu-vm.nix:105:13:
assertion failed at /var/lib/gc-of-borg/.nix-test-rs/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/rbvermaa-spot/pkgs/build-support/closure-info.nix:11:1

ofborg eval error for x86_64-linux looks unrelated.
tests.virtualbox.simple-gui and tests.virtualbox.headless succeed on my local machine.

Looks like it. Strange.


add haxe lint

fix #216

Pushed a fixup.
Please squash-merge.

can you restart the circleci builds? they seem to be stuck on github

The config changed, close+reopen should work usually to retrigger a build.

Rebased/squashed now.


Add density cutoff option to fix FDBM for all-electron densities.

Fixes #276

[ ] Add an example of a full-density-based model using AIMS electron density.

Adds an option to cut off high-density values in FDBM Pauli integral. This enables using the FDBM simulation with all-electron densities where high-density values near nuclei cause artifacts in the resulting images.

I was recently using different soft_clamp function which is simpler (no need for exp) I think, but perhaps as long as it works it does not matter.
I was trying to compare it with yours, but not sure if I use yours correctly.
https://colab.research.google.com/drive/1zM1QtqKDIx7ULy8geUq7FPVWitqfwK7R?usp=sharing
import numpy as np
import matplotlib.pyplot as plt

def soft_clamp(y, dy, y1, y2):
    """
    Applies a soft clamp to y, smoothly transitioning values above y1 towards y2.
    Also computes the derivative dy accordingly using the chain rule.

    Parameters:
    - y: np.ndarray, input values to clamp
    - dy: np.ndarray, derivatives of y with respect to some variable x
    - y1: float, lower threshold for clamping
    - y2: float, upper threshold for clamping

    Returns:
    - y_new: np.ndarray, clamped y values
    - dy_new: np.ndarray, updated derivatives
    """
    y_new  = y.copy()
    dy_new = dy.copy()
    mask   = y_new > y1
    y12    = y2 - y1
    invdy  = 1.0 / y12
    z = (y_new[mask] - y1) * invdy
    y_new[mask]   = y1 + y12 * (1 - 1 / (1 + z))
    dy_new[mask] *= 1.0 / (1.0 + z)**2
    return y_new, dy_new

def soft_clamp_exp(array_in, min_value, max_value, width=10.0):
    """
    Soft clamp function using NumPy masks, smoothly clamping values.

    Parameters:
    - array_in: np.ndarray, input array
    - min_value: float, minimum clamp value
    - max_value: float, maximum clamp value
    - width: float, width of the transition region

    Returns:
    - array_out: np.ndarray, output array after applying the soft clamp
    """
    array_out = array_in.copy()

    # Mask for values greater than max_value
    if max_value < np.inf:
        mask_max = array_out > max_value
        array_out[mask_max] = (
            (array_out[mask_max] - max_value) /
            (1 + np.exp((array_out[mask_max] - max_value) / width)) + max_value
        )

    # Mask for values less than min_value
    if min_value > -np.inf:
        mask_min = array_out < min_value
        array_out[mask_min] = (
            (array_out[mask_min] - min_value) /
            (1 + np.exp((array_out[mask_min] - min_value) / -width)) + min_value
        )

    return array_out

# Example usage:
# y(x) = 1 / (x - 2)^6, derivative dy/dx = -6 / (x - 2)^7

# Define the domain, avoiding x = 2 to prevent division by zero
x  = np.linspace(0.0, 4.0, 1000)  # Start slightly above 2
y  = 1. / (x - 2.)**6
dy = -6. / (x - 2.)**7

y1 = 1.0
y2 = 3.0

# Apply the soft clamp
y_clamped, dy_clamped = soft_clamp(y, dy, y1, y2 )

y_clamped_exp = soft_clamp_exp(y, -y1, y1, width=(y2-y1) )

# Plotting the results
plt.figure(figsize=(12, 6))

# Plot y and its clamped version
plt.subplot(1, 2, 1)
plt.plot(x, y, label=r"$y(x) = \frac{1}{(x-2)^6}$", color='blue', alpha=0.5)
plt.plot(x, y_clamped, label="Clamped y", color='red')
plt.plot(x, y_clamped_exp, label="Clamped_exp  y", color='g')
plt.axhline(y1, ls='--', c='k', label="y1 and y2")
plt.axhline(y2, ls='--', c='k')
plt.title("Soft Clamping Function")
plt.xlabel("x")
plt.ylabel("y")
plt.legend()
plt.grid(True)
plt.ylim(0.0,5.0)

# Plot dy and its clamped derivative
plt.subplot(1, 2, 2)
plt.plot(x, dy, label=r"$dy/dx = -\frac{6}{(x-2)^7}$", color='blue', alpha=0.5)
plt.plot(x, dy_clamped, label="Clamped_exp dy/dx", color='red')
#plt.plot(x, dy_clamped_exp, label="Clamped_exp dy/dx", color='g')
plt.axhline(0, ls='--', c='k')  # Reference line for zero derivative
plt.title("Derivative of Soft Clamped Function")
plt.xlabel("x")
plt.ylabel("dy/dx")
plt.legend()
plt.grid(True)
plt.ylim(-15.0,15.0)

plt.tight_layout()
plt.show()

We talked in the meeting that I should add the AIMS density to the pyridine example to demonstrate the problem and the fix. However, the artifacting that I observed in my tests does not really manifest itself noticeably with pyridine since the heaviest element there is nitrogen which does not have such high core density.
In my test I observed the effect at least with Cl-atoms, so some molecule with that or a heavier element would work better. Any suggestions which molecule we want? If not, then I will just pick something random.
@ondrejkrejci @mondracek @ProkopHapala

that or a heavier element would work better. Any suggestions which molecule we want? If not, then I will just pick something random.

What was quite nice you were testing GUI with some benzene substituted by -Cl, -Br groups if I remember. That could be quite nice. Maybe even more nice would be pyridine substituted by these groups halogen groups.
But in general pick whatever you you find convenient.

What was quite nice you were testing GUI with some benzene substituted by -Cl, -Br groups if I remember.

Yes, 1-bromo-3,5-dichlorobenzene. We actually already have it in the examples as an xyz file: https://github.com/Probe-Particle/ppafm/tree/main/examples/benzeneBrCl2.

Maybe even more nice would be pyridine substituted by these groups halogen groups.

@ProkopHapala Something like this: https://pubchem.ncbi.nlm.nih.gov/compound/606256?

Yes

@NikoOinonen did you test this PR in the CLI version? Did it work? I tried to run FDBI for some FHI-AIMS-generated XSF files and I keep hitting into the #312 issue. (I had to do some manual editing of the XSF files to overcome the problem that our code does not really understand these XSF from FHI-AIMS in their original form, but that does not seem to be the origin of the issue.)

@mondracek I was just now trying to get it to work in the CLI, but I run into the exact same issue. @yakutovicha Do you think you can finish #316 soon?

I added the example, but only for the GPU version until the CLI version issue gets fixed.
@ProkopHapala Can you add this file to the Zenodo record for the examples when you create it: https://www.dropbox.com/scl/fi/ilx7oe6iax375tqdleuax/BrClPyridine_hartree_density.tar.gz?rlkey=5j5xyc9n36zgw66xsb5mq8rvd&st=wyenjh4n&dl=0

@mondracek Regarding using densities from aims, I just used the cube output and then converted it to xsf.
I used the following script for the conversion (not sure if the origin is handled correctly, so better to keep it at (0, 0, 0)):

from pathlib import Path
from ase.io.cube import read_cube
from ppafm.io import saveXSF, primcoords2Xsf, Hartree2eV
import numpy as np

def cube_to_xsf(cube_path, xsf_path, scale=1.0):

    with open(cube_path, "r") as f:
        cube = read_cube(f, read_data=True)
        data = cube['data'].T * scale
        atoms = cube['atoms']
        origin = cube['origin']

    lattice = atoms.cell
    xyzs = atoms.get_positions() - origin
    Zs = atoms.get_atomic_numbers()
    lvec = np.concatenate([origin[None], lattice])

    xsf_path.parent.mkdir(exist_ok=True)
    atomstring = primcoords2Xsf(Zs, xyzs.T, lvec)
    saveXSF(xsf_path, data=data, lvec=lvec, head=atomstring)

if __name__ == "__main__":

    input_dir = Path(".")
    output_dir = Path(".")

    cube_to_xsf(input_dir / 'cube_001_hartree_potential.cube', output_dir / 'hartree.xsf', scale=Hartree2eV)
    cube_to_xsf(input_dir / 'cube_002_total_density.cube', output_dir / 'density.xsf', scale=1.0)

@mondracek Regarding using densities from aims, I just used the cube output and then converted it to xsf.
I used the following script for the conversion (not sure if the origin is handled correctly, so better to keep it at (0, 0, 0)):
from pathlib import Path
from ase.io.cube import read_cube
from ppafm.io import saveXSF, primcoords2Xsf, Hartree2eV
import numpy as np

def cube_to_xsf(cube_path, xsf_path, scale=1.0):

    with open(cube_path, "r") as f:
        cube = read_cube(f, read_data=True)
        data = cube['data'].T * scale
        atoms = cube['atoms']
        origin = cube['origin']

    lattice = atoms.cell
    xyzs = atoms.get_positions() - origin
    Zs = atoms.get_atomic_numbers()
    lvec = np.concatenate([origin[None], lattice])

    xsf_path.parent.mkdir(exist_ok=True)
    atomstring = primcoords2Xsf(Zs, xyzs.T, lvec)
    saveXSF(xsf_path, data=data, lvec=lvec, head=atomstring)

if __name__ == "__main__":

    input_dir = Path(".")
    output_dir = Path(".")

    cube_to_xsf(input_dir / 'cube_001_hartree_potential.cube', output_dir / 'hartree.xsf', scale=Hartree2eV)
    cube_to_xsf(input_dir / 'cube_002_total_density.cube', output_dir / 'density.xsf', scale=1.0)

This script should be in our utilities.

This script should be in our utilities.

I'm not sure about that. It's quite specific to this case when you are computing exactly the hartree potential and the density using aims. And again the origin is handled somehow weirdly: I had a cube with non-zero origin and after conversion it was displaying right in VESTA but was not working in ppafm. I think the real solution is to fix #190 so we don't have to make any conversions.

Well,
Just an idea - the origin is always wrong from AIMS, there is no way, so it would start from a real 0,0,0 - I am wondering, why it is not working with ppafm, but then it does not make sense.
Regards,
Ondrej
From: NikoOinonen @.>
Sent: Tuesday, November 12, 2024 5:01 PM
To: Probe-Particle/ppafm @.>
Cc: Krejci Ondrej @.>; Mention @.>
Subject: Re: [Probe-Particle/ppafm] Add density cutoff option to fix FDBM for all-electron densities. (PR #313)
This script should be in our utilitieshttps://github.com/Probe-Particle/ppafm/tree/main/ppafm/cli/utilities.
I'm not sure about that. It's quite specific to this case when you are computing exactly the hartree potential and the density using aims. And again the origin is handled somehow weirdly: I had a cube with non-zero origin and after conversion it was displaying right in VESTA but was not working in ppafm. I think the real solution is to fix #190https://github.com/Probe-Particle/ppafm/issues/190 so we don't have to make any conversions.

Reply to this email directly, view it on GitHubhttps://github.com/Probe-Particle/ppafm/pull/313#issuecomment-2470767579, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADFZAFK3BMHIA3XRRD6YM4L2AIJZ3AVCNFSM6AAAAABQFXBNZ6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINZQG43DONJXHE.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

@mondracek I added the CLI run script, so you can try it.
It's a little bit different from the GPU Python script, because it uses the dz2 tip for electrostatics instead of the delta density which had the wrong grid dimensions. I tried the same method for subtracting the valence electrons as in the pyridine example, but for some reason it was giving me really severe artefacts from the electrostatics so I did it this way. It's not ideal, but it demonstrates the effect of the cutoff.


Added the difficulty tests. Changed fixtures submodule commit in order to pass transaction tests.

Changes Unknown when pulling 4e2eba3368011c2b5f4417206d76ea009c9140b7 on RomanZacharia:develop into ** on ethereum:develop**.

Looks good.


Update Meta “tags”

Automatically generated by Netlify CMS

👷 Deploy Preview for monetaboot processing.

Name
Link

🔨 Latest commit
948f42ac045f92e4fd30ef48d8b2729501ee4b31

🔍 Latest deploy log
https://app.netlify.com/sites/monetaboot/deploys/6393f5bda90ede0008ee4416


[dv/top] connect alert_agent to top-level testbench

Connect alert_esc_agent to monitor alert_rx/tx.
In top-level, alert_esc_agent is not active. The driver is disabled,
it only needs to check if alert handshakes are completed.

Auto generate LIST_OF_ALERTS and NUM_ALERTS parameters from top-level
hjson file.

Signed-off-by: Cindy Chen<EMAIL_ADDRESS>

The latest force push add automation for alert_i hier path as Sri recommended. Could you take another look and see if there are any other suggestions? @weicaiyang @sriyerg


fix: Add cockroachdb backend

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


Gnome Tracker: no miners

Issue description
Installed NixOS 18.09 on several laptops, with Gnome. All Tracker dependant functions won't work (e.g. nothing to show in both Gnome Documents and Gnome Music).
Tracker is installed, but there are no miners available:
tracker daemon --list-miners-available
0 collecteur est installé
System installed in French, the message means that 0 miners are installed. Still, there IS a message from Tracker, which means that Tracker itself is installed.
Steps to reproduce
Install NixOS 18.09 with:

services.xserver.displayManager.gnome3.enable = true;
services.gnome3.tracker.enable = true;
services.gnome3.tracker-miners.enable = true;

Technical details
Please run nix-shell -p nix-info --run "nix-info -m" and paste the
results.

system: "x86_64-linux"
host os: Linux 4.14.88, NixOS, 18.09.1676.7e88992a8c7 (Jellyfish)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.1.3
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

I took look (though on unstable with gnome 3.30).
tracker daemon --list-miners-available lists zero miners here as well, but the services are running and eg. gnome-photos and gnome-music is working. So I suspect this isn't actually a problem.
It's worth double checking that you've turned on tracking for the relevant directories, either through Settings -> Search -> gear at the bottom right. Or doing something like this dconf write /org/freedesktop/tracker/miner/files/index-recursive-directories ['&PICTURES', '&DOCUMENTS', '&MUSIC'].
Now, gnome-documents isn't working here either. But it looks like this is the problem

des. 31 10:15:28 x1 .gnome-document[18107]: Failed to load shared library '/nix/store/ph28v47rmjs6mvnddxmskj1cgakjf67l-gnome-documents-3.30.0/lib/gnome-documents/libgdprivate-1.0.so' referenced by the typelib: libgd.so: cannot open shared object file: No such file or directory

This seems to result from broken linking:
cd $(nix eval nixpkgs.gnome3.gnome-documents --raw)/lib/gnome-documents
❯ ldd libgdprivate-1.0.so | rg libgd
	libgd.so => not found
        ...

@jtojnar have filed an upstream issue for this: https://gitlab.gnome.org/GNOME/gnome-documents/issues/5 We should probably apply his mentioned fix.

IIRC, the miners are picked up from XDG_DATA_DIRS, and we cannot add them to tracker because that would cause a dependency cycle. Maybe we could add the miners to pathsToLink in the tracker module.

It's worth double checking that you've turned on tracking for the relevant directories, either through Settings -> Search -> gear at the bottom right.

All switches are turned on. The gear at the bottom right is here, but disabled.

Or doing something like this dconf write /org/freedesktop/tracker/miner/files/index-recursive-directories ['&PICTURES', '&DOCUMENTS', '&MUSIC']

error: 1:expected value

Usage:
  dconf write KEY VALUE

Write a new value to a key

Arguments:
  KEY         A key path (starting, but not ending with '/')
  VALUE       The value to write (in GVariant format)

This is still confusing. tracker daemon --list-miners-available says none available and Settings -> Search -> Search Locations is greyed out. But there's a tracker process running.
 - system: `"x86_64-linux"`
 - host os: `Linux 5.9.1, NixOS, 21.03pre248170.007126eef72 (Okapi)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.3.7`
 - channels(root): `"nixos-21.03pre248170.007126eef72, home-manager"`
 - channels(austin): `"home-manager"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

I also have this problem but I only have a tracker3 command (gnome-music still shows my music) and I cannot change the search locations in gnome settings either.
(Maybe relevant is that I changed my XDG_DIRECTORIES using home-manager which creates .config/user-dirs.confwith enabled=False to not overwrite my custom directories on every login)

system: "x86_64-linux"
host os: Linux 5.10.13, NixOS, 21.03.20210208.8c87313 (Okapi)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.4pre20201205_a5d85d0
nixpkgs: /nix/store/milbcj3ck45mipsi0xmgx29a42qx6h09-source

It works for me now:
$ result/libexec/tracker3/daemon --list-miners-available
Found 1 miner installed:
  org.freedesktop.Tracker3.Miner.Files

The command is not available by default, tracked by #153378.


About failing to highlight syntax correctly

Type: Bug
If I need to use this C# Dev kit, I must first upgrade the C# extension to the preview version.
But this creates several problems.

My syntax highlighting is invalid. I am developing a Blazor Wasm Host project, but the project is developed from .Net 6.0, and the current version is .Net 7.0.
The most important thing is that I reopened a project. Syntax highlighting does not seem to be a big problem at present, but as long as I change the Extension to the release version, the problem will be solved.

Related commands provided by the C# Dev kit cannot be used in Blazor, including .Net Restrt, which will cause errors in some commands
Extension version: 1.25.9
VS Code version: Code 1.79.2 (695af097c7bd098fbf017ce3ac85e09bbc5dda06, 2023-06-14T08:57:04.379Z)
OS version: Windows_NT x64 10.0.22621
Modes:

System Info

Item
Value

CPUs
13th Gen Intel(R) Core(TM) i7-13700F (24 x 2112)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
undefined

Memory (System)
63.84GB (41.78GB free)

Process Argv
--crash-reporter-id ea7c6aa3-c78c-47b1-96da-a5973dfa85be

Screen Reader
no

VM
0%

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263cf:30335440
vscoreces:30445986
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593cf:30376535
pythonvs932:30410667
vsclangdf:30486550
c4g48928:30535728
dsvsc012cf:30540253
pynewext54:30695312
azure-dev_surveyone:30548225
2e4cg342:30602488
pyind779:30671433
f6dab269:30613381
pythonsymbol12:30671437
a9j8j154:30646983
showlangstatbar:30737416
vsctsb:30748421
pythonms35:30701012
pythonfmttext:30731395
pythoncmv:30756943
fixhidewlkth:30730051
showindicator:30766890
pythongtdpath:30769146
i26e3531:30769768
dh2dc718:30763024

I am also experiencing this issue, fresh install of vs code

This issue is quite stale and we've made a number of fixes in these areas - if you're still having issues on the latest version of the extension, please re-open and include the C# logs + general reproduction steps.


Additional maintainers?..

Hi! I'm a maintainer of knex and we use your package. There were complaints of security vulnerabilities that transitively come from liftoff. Would you be open to accepting additional maintainers into your project in order to address the vulnerabilities?

@phated

This project is going through a complete rewrite and then will be properly migrated to the @gulpjs organization.

Thank you for the heads-up!

@phated Would you be open to releasing a hotfix version if we provide a PR that would resolve current security warnings about liftoff? Our users keep complaining about security warnings they are getting, and we can't remove liftoff dependency without a semver major.


Manage consents / My data

Show used (and unused) services on My data -> Services tab.

Codecov Report

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

@@           Coverage Diff           @@
##           master      #97   +/-   ##
=======================================
  Coverage   62.96%   62.96%
=======================================
  Files          20       20
  Lines         243      243
=======================================
  Hits          153      153
  Misses         90       90

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 7093559...d0587ba. Read the comment docs.


Config-editor-ui: regenerate lock

can you please create one commit PR with all fixes?

Maybe better will be creating a new PR with one commit? Why is a high number of deletions?


TypeError: 'undefined' is not an object (evaluating 'window.plugins.socialsharing')

Hey,
I am trying to use this in a sample iOS project.
I have added the files as you mentioned, but on click I am getting this error -
TypeError: 'undefined' is not an object (evaluating 'window.plugins.socialsharing').
I am using Cordova 3.4.0.

Hello mates, i made everything being said here so i can have this plugin works on either ios and android but still nothing yet. When i console.log($cordovaSocialSharing), it gives me an empty object. I can actually see the plugin files in the plugins folder and the js file that i put in the js folder. I link the socialsharing.js in the index.html, but still nothing works. Any help please..!

I forgot to add the line of error thrown to me
Error: 'undefined' is not an object (evaluating 'n.plugins.socialsharing.canShareVia')

The custom plug-in cordova is wrong， undefined is not an object (evaluating 'window.plugins.oaCommonOperate.finish')！How did you solve it？


Duplicated chatbot message box issue

Jira Issue: https://issues.redhat.com/browse/AAP-34074

Description

The styles applied to the chatbot message box are set by using inline styles.  When the chatbot is loaded from the STAGE server, the following warning message shows up in the development console:

index-CXRSxDSp.js:55 Refused to apply inline style because it violates the following Content Security Policy directive: "default-src 'self' data:". Either the 'unsafe-inline' keyword, a hash ('sha256-lmsyJ3MLn/9zJPUSTRtlKyXRD2upigFYpVAUZs6+KS8='), or a nonce ('nonce-...') is required to enable inline execution. Note also that 'style-src' was not explicitly set, so 'default-src' is used as a fallback.

Actually the same message shows up when the chatbot is loaded from a local server. However, when it was loaded from a local server, the styles are applied and no issues are found on UI.
This warning message can be suppressed by specifying the following setting for Django CSP:
CSP_STYLE_SRC = ("'self'", "'unsafe-inline'")

The string 'unsafe-inline' looks scary, but according to the reply on this Stackoverflow page, it should be fine for CSP_STYLE_SRC.  It is the change contained in this PR.
Testing

Steps to test

Pull down the PR
Run the AI Connect service
Login through Red Hat SSO and open the chatbot page
Open Developer Console and make sure no warning message is found

Scenarios tested

See above
Production deployment

[X] This code change is ready for production on its own
[ ] This code change requires the following considerations before going to production:

This will be addressed by the PatternFly virtual assistant team with https://github.com/patternfly/virtual-assistant/issues/263
I am closing this without merging


"item" and "object" are not precise names for concept

In KiiGame:
Item = something you can pick up to the inventory and use it on objects or items by dragging.
Object = is a stationary thing in a room, that is clickable and you can use items on it.
Especially "object" is too widely used, especially in coding. Perhaps "furniture" would be a good name.

Room objects are now "furniture". "Item" stays for now.


Broken submission for .xlsx

When I drag and drop an excel sheet (.xlsx) I get: "Sample sheet upload failed: Unknown error."
However, if I export the same sheet as a .csv and submit it via the webinterface I get no errors.
Should we fix this or just restrict to .csv files (or mentioned it in the documentation)?
Microsoft Office Professional Plus 2019
Excel version 1808

Problem was solved by closing the sample sheet in excel


pushState only works for main view's navigation

Any plan to make it works for all views?

No, there will be new/additional type of router later which will support routing for anything


"tap" on keyboard does not result in "nudge"

G2 appears not to respond to immediate feedhold, ie feedhold issued during rampup.  As a result, a simple "tap" on the manual mode (a jog followed by an immediate stopJog) does not result in the expected traversal.  Instead, the 500ms planned motion for a jog must time-out (instead of the immediate stop) which results in a much larger move than anticipated for a nudge.

Closing this as not relevant.  Keyboard mode is changing in upcoming versions of the engine.


TURF: Add center operation

Follow these instructions for adding a new turf function.
Turf docs: http://turfjs.org/static/docs/module-turf_center.html

When running center on two selected polygons, Drop-n-Chop outputs 3 points.  The naming convention of these outputs implies:

center of the first poly
center of the second poly
center of both polys

But, the one that should be the center of both polys is actually the center of the last polygon selected.
I think this function is supposed to give the geographic center of all features
My understanding from the turf docs, is that it is expecting a feature collection, not individual features.  So, this might be the problem.


set FlowSpec attrs via metaclass, add pytest to CI

(factored out of #612)

set core FlowSpec attrs (file, name, path_spec, _graph, _steps) at definition, using FlowSpecMeta metaclass
add proof-of-concept pytest test (and associated testing utilities); run pytest in CI
use click's standalone_mode to control expectations around SystemExit on flow completion

Not immediately obvious to me what happened in "R tests on macos-latest":
Loading required package: data.table
Loading required package: Matrix
Loading required package: glmnet
Loaded glmnet 4.1-2
Loading required package: caret
Loading required package: lattice
Loading required package: ggplot2
Metaflow 2.3.0 executing BasicArtifactsTestFlow for user:runner
Creating local datastore in current directory (/Users/runner/work/metaflow/metaflow/R/tests/.metaflow)
Validating your flow...
    The graph looks good!
2021-08-24 16:47:29.143 Workflow starting (run-id<PHONE_NUMBER>050969):
2021-08-24 16:47:29.167 [1629823649050969/start/1 (pid 9336)] Task is starting.
2021-08-24 16:47:33.403 [1629823649050969/start/1 (pid 9336)] Error in py_call_impl(callable, dots$args, dots$keywords) : SystemExit: 0
2021-08-24 16:47:33.420 [1629823649050969/start/1 (pid 9336)]
2021-08-24 16:47:33.420 [1629823649050969/start/1 (pid 9336)] Detailed traceback:
2021-08-24 16:47:33.421 [1629823649050969/start/1 (pid 9336)] File "/Users/runner/.virtualenvs/r-metaflow/lib/python3.7/site-packages/metaflow/R.py", line 102, in run
2021-08-24 16:47:33.421 [1629823649050969/start/1 (pid 9336)] entrypoint=full_cmdline[:-len(metaflow_args)])
2021-08-24 16:47:33.421 [1629823649050969/start/1 (pid 9336)] File "/Users/runner/.virtualenvs/r-metaflow/lib/python3.7/site-packages/metaflow/cli.py", line 1010, in main
2021-08-24 16:47:33.421 [1629823649050969/start/1 (pid 9336)] start(**start_kwargs)
2021-08-24 16:47:33.421 [1629823649050969/start/1 (pid 9336)] File "/Users/runner/.virtualenvs/r-metaflow/lib/python3.7/site-packages/click/core.py", line 1137, in __call__
2021-08-24 16:47:33.422 [1629823649050969/start/1 (pid 9336)] return self.main(*args, **kwargs)
2021-08-24 16:47:33.422 [1629823649050969/start/1 (pid 9336)] File "/Users/runner/.virtualenvs/r-metaflow/lib/python3.7/site-packages/click/core.py", line 1090, in main
2021-08-24 16:47:33.422 [1629823649050969/start/1 (pid 9336)] sys.exit(e.exit_code)
2021-08-24 16:47:33.422 [1629823649050969/start/1 (pid 9336)] Calls: <Anonymous> -> py_call_impl
2021-08-24 16:47:33.422 [1629823649050969/start/1 (pid 9336)] Execution halted
2021-08-24 16:47:33.429 [1629823649050969/start/1 (pid 9336)] Task failed.
2021-08-24 16:47:33.441 Workflow failed.
2021-08-24 16:47:33.442 Terminating 0 active tasks...
2021-08-24 16:47:33.442 Flushing logs...
    Step failure:
    Step start (task-id 1) failed.

Error: Error in run_tests(context) : status_code == 0 is not TRUE
Calls: source ... eval -> run_tests_all_contexts -> run_tests -> stopifnot
Execution halted
Error: Process completed with exit code 1.

"R tests on macos-latest" still failing, seems like a real issue, I'll try to reproduce locally

Alright, figured out the R tests issue, this was the fix, tests all passing now! 🎉

This is ready to review, I think!


query cancellation with signal?

Heya. Thanks for this library, it's very handy!
Just to be sure, I found no way to pass signal to my fetcher. See here

signal?: AbortSignal
AbortSignal instance provided by TanStack Query
Can be used for Query Cancellation

Am I missing something? 🤔
Thanks!

Ah yeah this could be handy. So would you like to be able to abort singel requests within a batch?
For example if I do:
batcher.fetch(1)
batcher.fetch(2)
batcher.fetch(3)

It will fetch [1,2,3] in a singel request(depending on your implementation). It creates a "list" of requests and then merges them into one request. But would you like to be able to abort one of them within the batching window?
Like:
batcher.fetch(1)
let req = batcher.fetch(2)
batcher.fetch(3)
req.abort()
// only 1,3 will be fetched?

Or do you want to abort a whole batch in a given go? Would exposing the current queued batch and some controlls to manipulate it be sufficient?
Like
const batcher = batshit.create({..})

batcher.fetch(1)
batcher.fetch(2)
batcher.fetch(3)

batcher.queue.clear()

// no requests will be made since queue was cleared before the scheduled batch was fired.

?


format doesn't work

$('http://github.com/SoapBox/linkifyjs/search/?q=this+is+a+really+long+query+string').linkify({
	format: function (value, type) {
		console.log(value);
		return 'link';
	}
}).html();

Result:
"http://github.com/SoapBox/linkifyjs/search/?q=this+is+a+really+long+query+string"

The message in console also does not appear

Which version of Linkify are you using? I tried this in Chrome and got this
"link"

As well as the console message.

Installed using bower
jQuery-linkify#1.1.7 (latest is 2.0.0-beta.5)
└── jquery#2.1.4

The format option is only available in 2.0.
Try this instead:
bower install linkifyjs#v2.0.0-beta.5


fix typo

Fix typo
CC @RichardLitt @jbenet to make sure this correction gets published to the ipfs blog

THanks!


Optimize env_prepend/append_path functions

Replace grep test with remove + add logic to prevent duplicates.
Because this approach does not need any external functions it's about 4
to 5 times faster then the original implementation.
Also added some error logging in case the provided path is not a valid
directory.

LGTM 🚀


learn for my thesis

hello, actually I'm still learning about informer but I will use it for my thesis in the future. I am gonna ask cause u separated the file encoder, decoder, and main informer, is it necessary to separate them? (I'm not really good at machine learning actually but I'm happy to learn about it and still confused about that) thank you so much

thank you so much for your answer!


Text and link colors for breadcrumb

Ensure that breadcrumb links and text (commas) are white for mastheads with images, and black (#333) for mastheads with no images.

:shipit:


feature/40_add_compass_widget

Added Compass Button to Widgets - used to reset orientation.

This task got mis-aligned via automation, due to a mis-named feature branch. Will move manually until complete.


Windows support for magic

In Windows, magic must be initialized with the kwarg magic_file that gives the path of the .magic file. This PR:

Allows for Windows support by allowing the .magic file to be specified by the MAGIC_FILE_PATH setting in settings.py.
Further documentation in the README to describe how to set this up on Windows.

See #12 .

Thanks! I haven't tested it on Windows, it's good to see it works with this patch.


Token Summary

As a user I should see the name, amount, address and blockie for a token attached to a wallet.

Closing this issue in rimble-ui. Will re-open a related issue in appropriate repo.


imagemagick7: 7.0.11-5 -> 7.0.11-6

Let's not base imagemagick bumps on staging from now on, as the rebuild count isn't that high.

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

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

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

Result of nixpkgs-review pr 118938 at a19afca4 run on aarch64-linux 1

15 packages marked as broken and skipped:

asymptote
autotrace
devede
gnome3.gnome-documents
gopro
invoice2data
photoflow
scribus
split2flac
spring
...

1 package failed to build:

natron

89 packages skipped due to time constraints:

abiword
adapta-gtk-theme
arc-theme
arx-libertatis
asciidoc-full
asciidoc-full-with-plugins
awesome (awesome-4-0)
beets
bibata-cursors
bibata-cursors-translucent
...

87 packages built successfully:

almanah
asc-key-to-qr-code-gif
beetsExternalPlugins.alternatives
beetsExternalPlugins.copyartifacts
beetsExternalPlugins.extrafiles
betterlockscreen
blockhash
brasero
brasero-original
cd-hit
chafa
cuneiform
dmtx-utils
docbook2odf
dvd-slideshow
dvdauthor
dvdbackup
evolution
fbida
fgallery
fontpreview
foxtrotgps
fped
freedroid
frescobaldi
gnome3.gdm
gnome3.gnome-initial-setup
gnome3.gnome-panel
goattracker
goattracker-stereo
hello-wayland
i3lock-fancy
i3lock-pixeled
imagemagick
imagemagickBig
imagemagick_light
imgproxy
libpst
libwmf
lilypond
mate.caja-extensions
mediainfo-gui
megapixels
multilockscreen
netbeans
noteshrink
optar
paperwork
pdf-redact-tools
pdf2odt
pdfsandwich
perl530Packages.Gtk3ImageView
perl530Packages.PerlMagick
perl532Packages.Gtk3ImageView
perl532Packages.PerlMagick
portmod
pqiv
protonvpn-gui
python38Packages.SQLAlchemy-ImageAttach
python38Packages.Wand
python38Packages.paperwork-backend
python38Packages.paperwork-shell
python38Packages.pyocr
python38Packages.pyvips
pywal (python38Packages.pywal)
python39Packages.SQLAlchemy-ImageAttach
python39Packages.Wand
python39Packages.paperwork-backend
python39Packages.paperwork-shell
python39Packages.pyocr
python39Packages.pyvips
python39Packages.pywal
recoverjpeg
shutter
solfege
sound-juicer
stm32cubemx
swaylock-fancy
tango-icon-theme
timelapse-deflicker
ttygif
variety
vips
wpgtk
wv
xfce.xfce4icontheme (xfce.xfce4-icon-theme)
zbar

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement.

Backported in ad7604ddbd9b299701fb5e15bc39cff80deffce2.


release the training data？

is it possible to release part of training data or construction code？

Our training data is publicly available and we will provide the download link soon.

thanks for your contribution，i wonder if there is any updates？

thanks for your contribution，i wonder if there is any updates？

Our SFT data encompasses:

LLaVA-665k: https://github.com/haotian-liu/LLaVA
DocVQA: https://www.docvqa.org/docvqa
ChartQA: https://github.com/vis-nlp/ChartQA
InfographicVQA: https://www.docvqa.org/datasets/infographicvqa
AI2D: https://allenai.org/data/diagrams
ST-VQA: https://rrc.cvc.uab.es/?ch=11&com=downloads
TextVQA: https://dl.fbaipublicfiles.com/textvqa/images/train_val_images.zip
SynthDoG-EN: https://huggingface.co/OpenGVLab/InternVL/resolve/main/synthdog-en-images.zip
Geometry3K: https://github.com/lupantech/InterGPS
PGPS9K: https://github.com/mingliangzhang2018/PGPS
Geo170K: https://huggingface.co/datasets/Luckyjhg/Geo170K
LLaVA-Med: https://github.com/microsoft/LLaVA-Med
VQA-RAD: https://huggingface.co/datasets/flaviagiammarino/vqa-rad
SLAKE: https://huggingface.co/datasets/BoKelvin/SLAKE
RefCOCO grounding (REC) part of shikra:  https://github.com/shikras/shikra/blob/main/docs/data.md
ShareGPT4V: https://huggingface.co/datasets/Lin-Chen/ShareGPT4V
A subset of ALLaVA-4V-Caption: https://huggingface.co/datasets/FreedomIntelligence/ALLaVA-4V
LAION GPT4V: https://huggingface.co/datasets/laion/gpt4v-dataset
TextOCR GPT4V: https://huggingface.co/datasets/jimmycarter/ textocr-gpt4v

We only include the training split of each dataset.


window/showMessageRequest: fix invalid action item serialization

I'm not sure these lines were needed.
At least it broke the Scala Metals LSP integration for me.
Reverting those lines resolved the issue.
[package]
name = "kak-lsp-reproduce-toml-bug"
version = "1.0.0"

[dependencies]
lsp-types = { version = "0.95.0", features = ["proposed"] }
serde = "1.0.0"
toml = { version = "0.8.8", features = ["display"] }

use serde::ser::Serialize;
fn main() {
    let item = lsp_types::MessageActionItem {
        title: "some title".to_string(),
        properties: std::collections::HashMap::new(),
    };
    let mut s = String::new();
    item.serialize(toml::ser::ValueSerializer::new(&mut s)).unwrap();
    println!("New style: {:?}", s);
    s = toml::to_string(&item).unwrap();
    println!("Old style: {:?}", s);
}

Prints:
New style: "{ title = \"some title\" }"
Old style: "title = \"some title\"\n"

The new style was panicking here for some reason:
https://github.com/kak-lsp/kak-lsp/blob/6ac4d3c585000270d837bfa1941c71218f32ee75/src/session.rs#L77.

Thanks, sorry I didn't have a way to reproduce and I didn't check this code path


Refactor rmw_connext into rmw_connext and rmw_dds

Feature request
Feature description
For IDL to C++ based DDS implementations there is no need for making a complete rmw_ library to integrate one vendor. By using a generic rmw_dds library vendors like connext, splice, opendds can be integrated with much less code duplication. We have done this for connext and opendds as part of AXCIOMA, it took us several iterations but at the end the core differences are minimal between DDS vendors
Using some templates or helper classes the minimal changes between vendors can be handled, these changes are in the area of:

Retrieval of the domain participant factory
QoS XML usage

Implementation considerations

@jwillemsen, did you mean to call it rm_connect in the title? I'm assuming you meant rmw_connext. It probably won't make a difference but might be helpful for searching, etc to change it.

While this would be a good goal for a refactoring there are difference in the API of each vendor which will likely be a problem.
Also since nobody is planning to work on this and it had been about a year since this ticket was filled I will go ahead and close the ticket. Please feel free to continue commenting. Any pull request towards this goal would be appreciated.


Keyboard shortcuts

Hello,
Thanks for this great library!
A beginner's question: what's the easiest way to add keyboard shortcuts? I'd like to execute a closure upon receiving a specific keyboard combination, ex. Cmd-R.
Thanks!

Hi, Apple has a wealth of information about SwiftUI readily available:
https://developer.apple.com/videos/
An example from one of my own apps, HMScriptEditor:
      Button(action: run) {
        Label("Run", systemImage: "play.fill")
      }
      .help("Run the HomeMatic script on the base station")
      .keyboardShortcut("r", modifiers: .command)

oh, this is much easier than I thought. Thank you so much!


How to control the header of a black row always attached automatically when copying a table from Excel and pasting it into the editor

Version

v1.4.1

Test Environment

PC, Windows 10, Chrome

Current Behavior

엑셀에서 테이블을 복사하여 에디터에 붙여넣기 할 때, 항상 검은 행의 해더가 자동으로 붙는것을 제어하는 방법을 알고 싶습니다. 서비스를 사용하는데 검은 행의 해더가 필요 없고 지적사항으로 나왔네요... 제어 방법을 찾아도 안보는데, 있는 건지? 있다면 가이드 부탁 드리겠습니다.
en: When copying a table from Excel and pasting it into the editor, I want to know how to control the black row headers to be automatically attached.

Expected Behavior

옵션 처리로 자동 헤더 사용 여부 지정
en: Specify whether to use automatic headers with option processing

@socketbear
It is the editor's specification that the table contains a header. The editor is a conversion editor between Markdown and WYSIWYG, according to GFM, requires that the table include a header as follows: In other words, a table cannot be created without a header.
Please check it!

I understood what it was. When pasting, I want to set the first line as the header. Is it possible?

@socketbear
That feature is not provided either. But we're considering handling the function you said. When the specification is determined and features are added. I'll tell you again.

Thank you. Wait for the moment of glory.

I will echo what @socketbear said here. That would be an excellent feature. Specifically right now when you are pasting something from an Excel document the first line (the header) comes as gray and empty. I would have expected that the gray line contains the first line of the pasted document.
This image is from the docs. It seems very weird that the header is not populated.


Cannot get aggregate device  to work with sounddevice/rtmixer signal processing.py example

I am new to sound device and rtmixer, I have the following aggregate device (2 USB mic +Apple EarPods  built-in mic) in MacOSX:
Available Input Devices:
{'name': 'Aggregate Device', 'index': 6, 'hostapi': 0, 'max_input_channels': 4, 'max_output_channels': 0, ....,default_samplerate': 48000.0}
Available Output Devices:
{'name': 'Built-in Output', 'index': 1, 'hostapi': 0, 'max_input_channels': 0, 'max_output_channels': 2, ..., 'default_samplerate': 48000.0}
I can get the live plot mic.py to work (despite the fact that the built-in microphone really have one channel instead of two) but not the signal processing example to work.  My declaration for the class is as follow:
stream = rtmixer.MixerAndRecorder(
device=[6,1], channels=[4,2], blocksize=0, latency=latency,
samplerate=samplerate)
I am wondering if my mistakes is on the queue or the stream class is not declared properly or the device/channels not declared correctly ?  Thank you for any input and help!

What exactly is going wrong? Are you getting error/warning messages?
I don't have a lot of experience with aggregate devices, so I don't know if that will work, but why don't you add your output to the aggregate device as well?
And if all else fails, you can try to use separate Mixer and Recorder objects. You could then call play_ringbuffer() on one and record_ringbuffer() on the other. I guess the rest of the signal processing example could stay the same.

the built-in microphone really have one channel instead of two

This is interesting, but it is not under my control. Either the aggregate device creates one more channel than necessary or the PortAudio library does that.


Update background.js

Added extra substitutions based on http://xkcd.com/1625/

This is a duplicate of https://github.com/Posnet/xkcd-substitutions/pull/34/files.


Add e2e test case for default scaling strategy

Add e2e test case for default scaling strategy
Checklist

[X] When introducing a new scaler, I agree with the scaling governance policy
[X] I have verified that my change is according to the deprecations & breaking changes policy
[X] Tests have been added
[X] Changelog has been updated and is aligned with our changelog requirements
[X] A PR is opened to update our Helm chart (repo) (if applicable, ie. when deployment manifests are modified)
[X] A PR is opened to update the documentation on (repo) (if applicable)
[X] Commits are signed with Developer Certificate of Origin (DCO - learn more)

Fixes #6416
Relates to #

Please run the e2e test task! Thanks!

Please run the e2e test task! Thanks!

@junekhan I see you've committed the test I sent you from this conversation
https://github.com/kedacore/keda/issues/6416#issuecomment-2538375158
can you not run the test locally?
I can send you the output if that saves having to create an entire PR just to run a test

@junekhan I see you've committed the test I sent you from this conversation
https://github.com/kedacore/keda/issues/6416#issuecomment-2538375158
can you not run the test locally?
I can send you the output if that saves having to create an entire PR just to run a test

Sorry, I don't have an environment to run this test. And I think it's nontrivial to add the test case, otherwise, there won't be any controversy. It can serve as an evidence to people holding the same idea as you.

@junekhan I see you've committed the test I sent you from this conversation
#6416 (comment)
can you not run the test locally?
I can send you the output if that saves having to create an entire PR just to run a test

Sorry, I don't have an environment to run this test. And I think it's nontrivial to add the test case, otherwise, there wouldn't have been any controversy. It can serve as evidence for people holding the same idea as you.

I created an environment to run the tests. Regardless, here is the output if you want to see.
I don't disagree that adding a test would be useful for the project, though I did not intend this one to cover all necessary test cases, just demonstrate your example. I suppose something is better than nothing. I am a little surprised you would commit someone else's code without checking, even if that code is just a reworking of existing code, that is what most of the tests are. But not a point I want to belabour.
Anyway, I assume you have not been able to run the tests I sent for eager either, so I've uploaded the output here.

I created an environment to run the tests. Regardless, here is the output if you want to see.
I don't disagree that adding a test would be useful for the project, though I did not intend this one to cover all necessary test cases, just demonstrate your example. I suppose something is better than nothing. I am a little surprised you would commit someone else's code without checking, even if that code is just a reworking of existing code, that is what most of the tests are. But not a point I want to belabour.
Anyway, I assume you have not been able to run the tests I sent for eager either, so I've uploaded the output here.

@chinery  Apologies for not asking for your permission of the test snippet. If you agree it's much simpler for people to follow the test case and the outcome than reading lengthy passages, you can re-auther and create another pull request. I will immediately withdraw this PR.

/run-e2e internal*

Merry Christmas all.
@JorTurFer I just saw you approved this PR, but I'm not sure the code is intended for merging. This is a test I wrote in issue #6416 showing that the default scaling strategy implements the behaviour that eager is documented as implementing (so showing that the documentation is wrong). @junekhan submitted this (presumably expecting the test to fail).
Perhaps you have some input to add on the issue thread itself. I realise it's wordy but there is a tl;dr at the top and you can skip to the latest messages.
To be explicit: this PR does not fix #6416 (if anything it validates it).


Is there a way to clone a partial part of a git project?

I know we can do a sparse checkout to clone just a folder or a file but I can't find it on the docs. Is there a way?

Sparse checkouts are not supported. Since this is an upstream issue, closing as a duplicate of https://github.com/libgit2/libgit2/issues/2263.


an issue that I'm not doing fine.

Hi,
It was heard from Zhixing any PR/Issues are welcome here, that's why I am filing this issue. hoping someone can help to make me happier.
Thanks,
Meow~

well I am feeling better now. meow~~


Tlul adapter host

This will fail CI due to #1870. I'm putting together a PR that can fix that on the software side.

PR #1880 provides a fix which allows CI to pass on this

@eunchan can you re-review this PR? It would be good to get it fixed to unbreak FPGA debugging.


TLS cert generated by Chirpstack Gateway has no IP SANs

While adding a Dragino LPS8N gateway configured with Chirpstack Gateway Bridge to a RAK7248 Chirpstack Gateway/Server, the MQTT connection fails because the web-interface generated TLS cert does not contain an IP SANs.
LPS8N log:
Wed Jul 26 15:42:55 2023 user.info chirpstack-gateway-bridge[23988]: time="2023-07-26T23:42:55.270738971Z" level=info msg="starting ChirpStack Gateway Bridge" docs="https://www.chirpstack.io/gateway-bridge/" version=4.0.10
Wed Jul 26 15:42:55 2023 user.info chirpstack-gateway-bridge[23988]: time="2023-07-26T23:42:55.281460734Z" level=info msg="backend/semtechudp: starting gateway udp listener" addr="<IP_ADDRESS>:1700"
Wed Jul 26 15:42:55 2023 user.err chirpstack-gateway-bridge[23988]: time="2023-07-26T23:42:55.767280004Z" level=error msg="[client]   x509: cannot validate certificate for <IP_ADDRESS> because it doesn't contain any IP SANs" module=mqtt
Wed Jul 26 15:42:55 2023 user.warn chirpstack-gateway-bridge[23988]: time="2023-07-26T23:42:55.775287815Z" level=warning msg="[client]   failed to connect to broker, trying next" module=mqtt

I don't think this is a ChirpStack bug. I believe the error is about the client unable to validate the server certificate. ChirpStack does not generate server certificates, only client certificates for the gateways. If I'm not mistaking the ChirpStack Gateway Bridge tries to connect to <IP_ADDRESS>.
Here you find scripts to generate certificates: https://github.com/chirpstack/chirpstack-certificates.


Add bindings for all word confidences

As a side note what's with the seperate error types instead of one error enum? Or wrapping a tesseract errno if there is one.
I've followed the rest of the code and made a new error type but it does not seem ideal.

As a side note what's with the seperate error types instead of one error enum? Or wrapping a tesseract errno if there is one.

My thinking was that each function would have an error type specific to it. That way, the calling function wouldn't have to handle any error variants that could never happen.
Do you think that's sensible? No one else has reviewed this library before.

Thank you - I've released it as version 0.6.0

I think it's way too painful to every be worth it. Even in c libraries tend to have an enum with all the error variants. That's actually more useful as you can see why a function failed instead of just what high level function failed.
Looks like tesseract doesn't have an errno and just returns null. If you just make an enum of the error variants that would be somewhat useful and erganomic.

Also there's no need to release a new version on every PR. I may end up making a few more depending on my needs.


Word wrap needed in display of aggregation metadata text

I have a raster aggregation in a resource with method metadata that does not word wrap.  It should
https://www.hydroshare.org/resource/877bf9ed9e66468cadddb229838a9ced/

This was caused by #2063, the word-break is no necessary to get the wrapping for the coordinate string and causes the word break in the method described here.

@sblack-usu - if this is just a display issue do you want to reassign this one to Mauriel?

This one is done and part of 1.32 which is going out next week.


Experimental: Potential Logging Fix

Unable to reproduce #737 on my end but refactoring filenames to avoid global module names is a good best practice anyway.
@kkunal1408 can you please test this branch on your setup and see if it resolves #737?

This did not solve my problem, even after installing from scratch. I will try to dig deep and update.

@stevenmburns I think we should merge this in regardless just to avoid confusion between the global logging module and align.utils.logging. Ready for review.


Reset local kills whole class (while have to kill just local test include)

Created unit test class (here: for ZCL_ABAPGIT_REPO), commited it to a new branch
switched back to master, pulled
The test include remained (as a new file to commit, although se80 does not see it) - this might be a bug by itself (maybe a separate issue?)
Go to advanced -> Reset local -> note the text in the screenshot - it mentions the whole class

Tick it and press OK...
... And yes, the class is deleted, AG dies in dump ...

P.S. to restore: created a one-file ag and made another Reset local there - it re-created the class from remote master.

duplicate of https://github.com/larshp/abapGit/issues/1066 ?

yes, looks like. Closing then.


aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa

e

i

e
On Thu, Aug 12, 2021, 08:56 Connor Brewster @.***>
wrote:

i
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/cbrewster/aaaaaaaaa/issues/1#issuecomment-897756624,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFOJ3UHAFACOXCHCDFRLDT3T4PVKXANCNFSM5CAGW3DA
.

`


feat: upgrade esbuild to 0.9

startService stopService are removed (no longer needed)
metafile is directly returned by esbuild

Nice, I was planning to do this today 😅


Error when trying to pass name of role rather than ID

Expectation is that I would be able to do something like this:
{
    "active": true,
    "username":"newuser"
    "password":"newpass"
    "email"<EMAIL_ADDRESS>    "roles": [ "admin", "RANDOM_CA_owner" ]
}

But seeing the following error:
AttributeError: 'unicode' object has no attribute 'items'

I will take a look at this. Currently roles are expected to be objects and only support ID lookups.
e.g.
"roles": [{"name": "admin", "id": 123}]

I could see how this could be annoying depending on your use case.
I will try to at least support:
"roles": [{"name": "admin"}]

Which would save the client a lookup. I will update the documentation regardless.

That would be great, thanks!
One other thing that the docs could use (are they public? if so happy to send a PR) would be an example of passing a role in:
POST /users HTTP/1.1
Host: example.com
Accept: application/json, text/javascript

{
   "username": "user3",
   "email"<EMAIL_ADDRESS>   "active": true,
   "roles": []
}

Err, is the docs/ folder what gets sent to lemur.readthedocs.io?

Yep everything under /docs eventually makes its way to lemur.readthedocs.io.


How to use "strict" search with enableSearch setting?

How can I use "strict" option with the search setting. I am looking for something similar to "Name Only" example at Angular Filter page.
Another example of what I want is, If you search for "ak" in the example here, result set should not have "Alaska" because it doesn't have the "ak" combination. It is in the result set because of its "Id" which "AK". Please suggest asap.

Hi,
I've implemented something that lets you do this at my fork which I'm actively maintaining and you can find here. You can install it using bower install myforce-angularjs-dropdown-multiselect, if you have an issue you may also create it at my fork.
You can specify the searchField as a setting, if you set it to 'label' the search will be applied on label only.
This is included from version 1.4.0.
Documentation for my fork can be found here

+1


eject tries to use /bin/umount

Issue description
I tried to eject a mounted USB storage volume...
eject: unable to exec /bin/umount of `/mnt'
eject: unmount of `/mnt' failed

Steps to reproduce

Plug in USB stick
Mount USB stick manually
eject /dev/sdX

Technical details

System: 17.03beta617.f3a9ccc (Gorilla)
Nix version: nix-env (Nix) 1.11.8
Nixpkgs version: "17.03beta617.f3a9ccc"

Don't forget to close this!


Added error codes to errors.

Description
In https://github.com/adobe/jwt-auth/pull/11, I described how error codes could be added to errors so consumers could programmatically figure out why jwt-auth failed and react accordingly. I think there was a miscommunication though because the codes were added to the error message itself rather than a property directly on the error. While consumer code could search for a particular error code within the error message, programmatically reacting to text within the message is usually frowned upon as the message is considered subject to change and isn't intended for programmatic decisioning.
In this PR, I'm demonstrating how we could add a code property to the error objects. The possible error codes are the ones found at the top of index.js and those in the Responses section of the endpoint documentation.
I haven't added tests or updated the readme yet. I wanted to see if you would be amenable to this change first.

Related Issue
https://github.com/adobe/jwt-auth/issues/9

Motivation and Context
Programmatically determine what caused an error to occur.

How Has This Been Tested?
So far, only manual testing.

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Tests are failing (they were not updated)

@shazron (or anyone else maintaining this library), is this PR something you would consider? If so, we can update tests, but I didn't want to update tests until I got an idea of whether something like this would be accepted.

Can we please get a response on this PR? This blocks https://github.com/adobe/reactor-uploader/issues/28 for us.

Yes, we'd be good with the PR. Just need to update those tests and we'll release a new major as not to break folks who depend on the old behaviour.

Resolved by #52 and #53


Initialize root resources

Replaces #815, depends on #823, closes #803.
As mentioned in https://github.com/solid/community-server/pull/815#issuecomment-881220183 we lose some flexibility, which might become an issue once we want to have the option for both an acl or acp authorizer. But we can look into that when we get there.
Moved favicon to images folder since it's not a resource in the root container.
Made a new PR so reviews are possible and due to the dependency. Also note my comment on the index.html itself in https://github.com/solid/community-server/pull/815#discussion_r672165390

@joachimvh Good to go, updated index.html to remove the statement about configuring root pods.

@joachimvh Not sure how my rebase got us to test failures.
FAIL test/integration/SparqlStorage.test.ts
  ● A server with a SPARQL endpoint as storage › can add a Turtle file to the store.

    InternalServerError: No handler supports the given input: [, Content-Type is required for data conversion., Cannot convert from unknown to internal/quads, Missing Content-Type header.]

      104 |     // Find the error range (4xx or 5xx)
      105 |     if (errors.some((error): boolean => error.statusCode >= 500)) {
    > 106 |       throw new InternalServerError(message);
          |             ^
      107 |     }
      108 |     throw new BadRequestHttpError(message);
      109 |   }

      at WaterfallHandler.findHandler (src/util/handlers/WaterfallHandler.ts:106:13)

Ah it wasn't my rebase. The CI wasn't running before because this was a non-main PR.
And SPARQL doesn't run locally by default.

Right, the sparql store only supports quads, we can't put HTML in there.
That's one of those things we wanted to use the content routing store for at some point but that one had issues.
Not sure what the best solution is here. We could update the sparql config to use a different root template folder that does not contain html. That would cause some duplication in the template folder though.
We could also simply not write resources that produce errors but that would hide potential issues.
Or we could create an AnyToQuadConverter that converts non-RDF data to a single triple with the object being a binary dump of the data 😅 .
None of these is ideal though.

Not sure what the best solution is here.

My suggestion: a try/catch around every created resource, print a warning when it could not be created.
Perhaps move favicon back to where it was.
The message Cannot convert from unknown to internal/quads confused me; perhaps fix that it is no longer unknown?

We could update the sparql config to use a different root template folder that does not contain html. That would cause some duplication in the template folder though.

There is a future for symlinks (which currently seem to be ignored).

Perhaps move favicon back to where it was.

Favicon was moved away from the folder because it is currently being served through the StaticAssetHandler, it's not actually in the pod (although we can totally put it there so users have control over the favicon by updating the root container).

The message Cannot convert from unknown to internal/quads confused me; perhaps fix that it is no longer unknown?

Will have to investigate that because I'm not sure why the content-type is missing there.

Favicon was moved away from the folder because it is currently being served through the StaticAssetHandler, it's not actually in the pod

Perfect.

The message Cannot convert from unknown to internal/quads confused me; perhaps fix that it is no longer unknown?

Will have to investigate that because I'm not sure why the content-type is missing there.

Found out why this happens and it's actually a bug that always happens when generating resources that I'm surprised our pod creation works as it is. The issue is that the resources generator uses the file mapper for determining the content type of the resources that it generates. But folders have no actual content-type, so the mapper sets that to undefined (which is correct). The fix is to set the content-type in the resources generator to text/turtle (or any other RDF type, doesn't really matter which one). So fortunately easy to fix.
The reason this never errored is because we check containers here: https://github.com/solid/community-server/blob/1ad03871b20f49afbf033cb94ceaec51ed82866e/src/storage/DataAccessorBasedStore.ts#L363-L364
and everything still works if we pass undefined as format there.
The interesting part is that the initializer now actually succeeds in also writing the index.html resource to the sparql store. My guess is that our RDF to quad converter actually accepts text/html since it also supports RDFa. The result is an empty list though (since there is no RDFa in the index.html), so we just write an empty resource to the backend.


[inetstack] Support Ipv4 Options

Description
We should support Ipv4 options.

Changing this to an enhancement that won't be fixed for now, because it concerns an improvement that would be nice to have, but it is not mandatory.


Update MD file on new branch 2017-11-14_08-24-35

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestRef1.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


fetch current pod ip from socket mark

https://github.com/merbridge/merbridge/issues/117  part 1.

@dddddai PTAL.


Added Frame Range Option to Convert Tool

Frame range option allows to only retrieve relevant data and reduce waiting time, especially useful for  bigger traces
Change-Id: I3b7298a20c41d2c67a59069ddf31961f37a3b12d

CI gfxreconstruct build queued with queue ID 56820.

CI gfxreconstruct build queued with queue ID 56850.

CI gfxreconstruct build # 3347 passed.

CI gfxreconstruct build queued with queue ID 69624.

CI gfxreconstruct build # 3415 running.

CI gfxreconstruct build # 3415 failed.

CI gfxreconstruct build queued with queue ID 100998.

CI gfxreconstruct build # 3625 running.

CI gfxreconstruct build # 3625 passed.

CI gfxreconstruct build queued with queue ID 181553.

CI gfxreconstruct build # 4125 running.

CI gfxreconstruct build # 4125 failed.


Container Image runs fine with AWS Lambda Runtime Emulator Interface, but 502 on zappa deploy

Context

I followed the steps here to create a container image, upload it to ECR, and run zappa deploy -d. All routes work perfectly when I test out the image with the AWS Runtime Interface Emulator, i.e, curl -XPOST "http://localhost:9000/2015-03-31/functions/function/invocations" -d '{"path": "/", "httpMethod": "GET", "requestContext": {}, "body": null}', but zappa deploy -d throws me a 502 error. I've run zappa tail, but all I see there is [1659730496257] 'NoneType' object is not callable.
Unfortunately I'm not cleared by my company to release more than the settings.json file here, but happy to share more details privately.
I have a few questions:

How is it possible that the image works when being tested on the runtime interface emulator, but fails during deployment? I've rewritten the settings file and made sure that slim_handler is set to false as well.
Are there any suggestions on where to embed print statements in the handler.py to make zappa tail more informative during debugging?
Given that I'm deploying this as a docker image, I am not using a venv, and invoking zappa from my project's conda environment. Could this be the root of any issues?

Expected Behavior
zappa deploy -d executes successfully and returns the API endpoint.
Actual Behavior

NoneType object is not callable. Same message replicated in zappa tail. But the same image works when tested with the AWS local Runtime Interface Emulator

Your Environment

Zappa version used: 0.55.0
Operating System and Python version: macOS 12.4 Monterrey, Python 3.9
The output of pip freeze: Running everything in docker, so shouldn't be an issue
Link to your project (optional):
Your zappa_settings.json:
{
"lambda_docker_flask": {
"app_function": "server.app",
"project_name": "app",
"lambda_description": "Zappa + Docker + Flask"
}
}

502 often results from a problem in the code.  Make sure that the code can be run before deployment

It definitely can - the code is running locally via a standard python3 script.py call, and even using AWS' runtime interface emulator via a curl -XPOST "http://localhost:9000/2015-03-31/functions/function/invocations" -d '{"path": "/", "httpMethod": "GET", "requestContext": {}, "body": null}'.
Zappa deployment is the only time it looks to be failing.

Just to be sure I would test the code outside of the lambda environment.  It's possible that the lambda integration isn't hitting the code as expected.

Thanks! Tried that too; ran the code in a different venv and also on a different machine entirely - no bugs there.

Update for anyone facing similar issues: FIXED
I try/except'ed every import statement in my app and printed out all failures. Turns out all my issues were caused because of module import errors. After resolving those import paths, my app loads successfully. Not sure why this had to be done since Dockerizing the app shouldn't have allowed import errors to exist (or am I mistaken?).
I'm now facing 504 timeout errors, but that's a story for another day.

@padmanabhankrishnamurthy did you figure out the issue you had with the 504? From the CloudWatch logs I can tell my Lambda is starting up correctly, but for some reason it runs all the way to timeout at 30 seconds instead of returning my "hello world"

Good to know!
For anyone browsing this thread looking for ideas about the 504 error  - in my case, my app.py script  was importing a ton of heavy libraries. Just importing the libraries and instantiating a few global objects was taking ~60s, by which time a timeout error was thrown. Moving the imports and instantiations to the route instead of keeping them global, and also using the task async decorator solved the issue for me.


Uncomment travis.yml lines regarding snapshot and cache

Hmm, still getting AAPT2 aapt2-3.3.1-5013011-linux Daemon #0: shutdown

Again, adding the caches causes the build to timeout :/ I'll remove the cache lines for now as I won't have time to fully debug this


[BUG] grafana dashboard for kubernetes pods workloads shows 'no data to show' message

Rancher Server Setup

Rancher version: 2.7-head commit ID 8062f1e
Installation option (Docker install/Helm Chart): Helm

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc): RKE1

Proxy/Cert Details:

Information about the Cluster

Kubernetes version: 1.25.5 & 1.24.8
Cluster Type (Local/Downstream):Downstream aws ec2 instance

If downstream, what type of cluster? (Custom/Imported or specify provider for Hosted/Infrastructure Provider):

User Information

What is the role of the user logged in? (Admin/Cluster Owner/Cluster Member/Project Owner/Project Member/Custom)

If custom, define the set of permissions: admin

Describe the bug
testing rancher-monitoring 101.1.0+up40.1.2  in k8s v1.25.5,  grafana loads and shows data, however, i created a workload in the default namespace, and when I check compute resources/Workload, there's no data - but it does find the workload itself (test-nginx)
this also occurs in v1.24.8, and i'm seeing the same behavior in rancher-monitoring:101.0.0+up19.0.3
To Reproduce

install rancher-monitoring:101.1.0+up40.1.2
create workload in default namespace > deployment > nginx:latest 3 replicas
navigate to cluster > monitoring > grafana (link)
select General / Kubernetes / Compute Resources / Workload

Result
The deployment created in step 2 is listed, but there is no data to show in the graphs
Expected Result
Data for selected deployment displayed in grafana graphs
Screenshots

Additional context

/backport 2023-Q1-v2.6x

https://etcd.io/docs/v3.5/op-guide/maintenance/#defragmentation


ROCANA-4398: SIGAR collects Windows system info

This return Windows for the SystemInfo.Sysname and the SystemDistribution.Description. The rest of the fields remain blank for the time being.

LGTM


Some confusion

Hi,
Q1: Why are the experimental results of the v2 and v1 versions of the paper different? Is there any change?
Q2: how long are you cost for fine-tuning? on 10000 samples?
Thanks!

A1: In the first version, when calculating the metrics, any correct result from either the first or second decompilation was counted as correct, and only one sample result was counted, which may have introduced some randomness and resulted in higher scores. In the second version, we took the last checkpoint and fixed this error, and the metrics were calculated using the average of 5 samples to reduce random error.
A2: About 9 hours for 3 epochs.

ok, i got it. thanks for your reply~

The lora model and dataset has been made public.

Great!
In addition, i found that the exebench dataset has three optimization options (O0, O3, Os). If you only use these for fine tuning, but you are evaluating four options (O0-O3), does this have an impact?

We use the code, but recompile the code to these four options (O0-O3) for training.

how can i obtain the source code of exebench?

i download the exebench fron huggingface

ok, you mean the 'func_def' keyword is the function source code?

yes

but i failed to compile them. because they all have undeclared identifiers.

have  you follow my script ?
maybe you need concat synth_deps and func_def to compile.

ok, i c


Included line-height in the base font mixin

Checklist

[x] I have read and understand the contribution guide

[ ] A CHANGELOG entry is included

[ ] At least one test case is included for this feature or bug fix

[x] Documentation was added or is not needed

[?] This is an API breaking change

Issue Resolved / Feature Added
Characters are vertically clipped when the font doesn't specify proper metrics.
Resolution
Unilaterally assign basic, low-level, easy to override, line-height measurements based on our existing definitions.

VideoPlayer does actually have a non-Latin line height definition a few lines below, so it should not be a problem.
https://github.com/enactjs/enact/blob/c0f978cc66e442146670e33e0e254d2a5fd6d86b/packages/moonstone/VideoPlayer/VideoPlayer.less#L110
The other two shouldn't be a problem, but still worth verifying that things aren't terribly broken.

That would be great. Should we remove those line-height rules in this PR since they have no effect and would only confuse future us?

That is a negative, Red Leader. non-Latin header line-height is 1.5em while our standard non-Latin is 1.7em (to account for things like Thai tall characters). Header line-height we can more tightly assign than generalized body text because we're (the Framework is) responsible for the layout and possible region overlap of our Header component.

:+1: I was only thinking of non-latin but the line-height rules would still take effect in latin locales.


Add Chinese language?

I briefly translated the Chinese language myself. if it is possible. You can add it. This mod is very good.
(However, I cannot Chineseize the creature name and status. I hope you can improve it in the future.)

It is a different mod, not related to playerAnimator.


HTML and CSS beautify functions are not included

Hi @sedouard !
I have looked into your code today and I have a proposition to use also HTML and CSS beautify functions depends on file extension. If I manage some time yesterday I will try to make a pull request with changes releated to the improvement.

Hey there! I'm not sure if I follow... JSBeautifier only formats javascript. Can you explain a bit more?

That's not exactly true. As far as I dig into documentation https://github.com/beautify-web/js-beautify#css--html, you can see there's such an option :)
Your code only needs to determine extension of file (it can be easily done with node path module) and then use particular beautifying function.

😆 Well will you look at that! Didn't realize JSBeautifier did that as well. Nice 👌🏽
Actually now that I think about this I will probably deprecate this package as grunt-jsbeautifier seems like the popular choice. I may have written this package before I knew it existed? Or maybe before hand. I'd recommend contributing to that module and not this one.


Build infrastructure and static configuration fixes.

This is needed before the C++ dialect deprecation.

Clean up cub include dir spec in CMake.
Move THRUST_DEPRECATED logic out of compiler.h and into new header.
Fix CPP dialect detection on newer MSVC.
Remove raw __cplusplus checks from source code.
Move the namespace macros/docs out of version.h.
Remove raw __cplusplus checks.
Use _Pragma instead of #pragma in macro.
Remove THRUST_BEGIN/END_NS macros.

These were used inconsistently, rendering them non-functional. Removing
to prevent people from trying to use them.

CI in shelve 28211130.

Renamed the macro and made the same change in cub as part of thrust/cub#21.

New shelve 28219812

New shelve 28221412

Updated with a ton of changes that get this to build on MSVC2015.
A commit with just the new changes can be found here: https://github.com/allisonvacanti/thrust/commit/1c9440ea98c45b1530363cfe254a9cf8e4cd1aa8
Relaunched CI under shelve 28231909.

Merged with the -Werror and deprecation PRs to check for incompatible CI builds. New shelve is 28232673.

 It doesn't have extended constexpr, but _MSVC_LANG does report C++14.

https://gcc.godbolt.org/z/Lc23fF
https://docs.microsoft.com/en-us/cpp/overview/visual-cpp-language-conformance?view=vs-2019
https://docs.microsoft.com/en-us/cpp/preprocessor/predefined-macros?view=vs-2019

So what do we want to do about this?
/polls 'Use "C++14 minus extended constexpr" as our base' 'Deprecate MSVC 2015 as well and fix THRUST_CPP_DIALECT to report MSVC as C++11 instead of C++14'

I vote for the second option (and am currently testing MSVC 2017 and 2019).

This last push updates the deprecation checks to give an #error for C++ < 14, MSVC < 2015, Clang < 6.0, GCC < 5.0. These can all be ignored by defining THRUST_IGNORE_DEPRECATED_CPP_DIALECT and CUB_IGNORE_DEPRECATED_CPP_DIALECT.
This patch also gets everything building on MSVC 2017 and 2019 using C++14 with the async algorithms enabled, and all tests pass (with the exception of #1098, that still needs to be tracked down).
New shelve: 28243163

Make sure to put the "Reviewed-by:" line in this commit (one for each reviewer).


Console Error When Adding User on Permissions Tab

Bug Report
When adding a user in the permissions tab, there is an error message in the JavaScript console. The operation actually works, and if you didn't have developer tools open, you wouldn't notice the issue. But it's still not good.
Steps to reproduce the behaviour

SAM authorizer is enabled in Dockstore, although this probably will happen with the InMemory authorizer as well.

Add a user as a writer or reader (should be one that works, e.g., if using SAM, then the user you are adding should be registered with SAM).
User gets added, but you see this in console:

PermissionsComponent.html:35 ERROR Error: ExpressionChangedAfterItHasBeenCheckedError: Expression has changed after it was checked. Previous value: 'mat-form-field-should-float: true'. Current value: 'mat-form-field-should-float: false'.
    at viewDebugError (core.js:8445)
    at expressionChangedAfterItHasBeenCheckedError (core.js:8433)
    at checkBindingNoChanges (core.js:8535)
    at checkNoChangesNodeDynamic (core.js:11423)
    at checkNoChangesNode (core.js:11393)
    at debugCheckNoChangesNode (core.js:11997)
    at debugCheckRenderNodeFn (core.js:11951)
    at Object.eval [as updateRenderer] (PermissionsComponent.html:35)
    at Object.debugUpdateRenderer [as updateRenderer] (core.js:11940)
    at checkNoChangesView (core.js:11291)

┆Issue is synchronized with this JIRA Story
┆Sprint: Lighthouse Sprint 31 Eblis
┆Issue Number: DOCK-51
┆Epic: SAM for Authorization

Looks like: https://github.com/angular/material2/issues/10914

I've seen these ExpressionChangedAfterItHasBeenCheckedError a lot, it's typically an indication that we didn't do something quite right rather than the library.

Seems fixed for me  on staging
DockstoreAPI - 1.5.0-alpha.8
DockstoreUI - 2.2.0-alpha.6


Potentially misleading log message for upsert-only policy

Using the upsert-only policy produces misleading log messages with many DNS records which should be excluded regardless of the upsert-only policy (because of TXT ownership):
time="2017-04-28T10:41:40Z" level=debug msg="Records to be excluded from the delete list due to upsert-only policy: [_amazonses.test.example.org -> "tdjBfskkZTxJBJra/123/55UM=" (type "TXT") atun43v34uio4s7sdfvwnkwmxrpl4._domainkey.test.example.org -> atun123kwmxrpl4.dkim.amazonses.com (type "CNAME")
....
 kube-1.test.example.org -> kube-1-145102334.eu-central-1.elb.amazonaws.com (type "ALIAS")
....]"

Expected: only log DNS records which would not have been excluded if we use the default policy.

@hjacobs yes, @linki already pointed out this, step of applying policy comes before the step where ownership is detected. We will think of the best way to improve this misleading log messages

@ideahitme yes, I assumed this already --- the logging just makes users (like me) believe (without looking in the code) that it would be dangerous to switch from upsert-only to the default policy.

As a quick fix I intend to just drop the offending log line. IMO, it doesn't make to much sense there anyways.
Sounds weird, but it avoids confusion. When you switch to --policy=sync you can always run it with --dry-run first to see that it does behave.
The better solution will be tracked here: https://github.com/kubernetes-incubator/external-dns/issues/181
@ideahitme @hjacobs agreed?

👍

https://github.com/kubernetes-incubator/external-dns/pull/193


Fix tutorials

This PR intends to improve tutorials (#4239) and should fix #6927 with updated WMS tutorial.

Small update to CONTRIBUTING.md as I had some issues with the steps to install jekyll

Update to the Mobile tutorial to avoid JS error when device orientation is not supported

WMS tutorial revamp:
The current WMS demo server is no longer available. I found a free WMS service from Mundialis . If necessary I can contact them to make sure this is ok to use it.
For the most part, I kept the same original tutorial structure and I added a brief explanation on WMTS at the beginning. However, I could not find a service for the WTS part, so I removed the example and kept the theoritical explanations (I wonder if WTS is still relevant).

I checked the other tutorials and didn't find more issues.
This is my first PR on an open source project ;)

This is great, thank you so much!

You're welcome, I am a fan of this lib :)


Add NoArm64 testRequires and skip Schema1RegistrySuite on arm64

- What I did
- How I did it
- How to verify it
- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)
schema1 manifests is not working on ARM64, we should skip integration-cli
tests for schema1 manifests on ARM64.
https://github.com/docker/docker/blob/master/Dockerfile.aarch64#L108
Signed-off-by: Lei Jitang<EMAIL_ADDRESS>

close because I'll push this branch to upstream by accident.

I usually set the push URL for upstream to some fake URL (e.g. no-pushing) to prevent accidentally pushing there http://stackoverflow.com/questions/10260311/git-how-to-disable-push#10260389

@thaJeztah  Thank you for your tips. I had done that on my common server. This PR was worked on a arm64 server and that was my first time to switch to a arm64 server. I just clone the repo and then edit, then test and push, forget the origin is till https://github.com/docker/docker.  So the mistakes happened.   my bad :sweat:

@coolljt0725 no worries, no harm done :smile: