new feature: allow inject a customized signer for aws s3

Feature Description
background: https://github.com/apache/iceberg-rust/issues/506

It seems we need to implement this signer in iceberg-rust and pass it as a customized signer to opendal.

Problem and Solution
but currently the signer is a struct AwsV4Signer.
we need make the signer in S3Core a trait before allowing it customizable.
maybe we can defined a trait called S3Signer in opendal, and make AwsV4Signer as it's implementation by default.
Additional Context
No response
Are you willing to contribute to the development of this feature?

[X] Yes, I am willing to contribute to the development of this feature.

to make the Signer customizable, i've got some difficulties to solely add it in the opendal part.
adding a trait called S3Signer in opendal requires a signature to be object safe, but a signature like fn sign<T>(&self, req: &mut Request<T>, cred: &AwsCredential) -> Result<()> is
not object safe because it contains a generic parameter inside it.
so i think there might have two ways to make it possible:

add a GenericSigner trait with fn sign(&self, req: &mut dyn SignableRequest, cred: &AwsCredential) -> Result<()> in reqsign.

but as Signer accepts &mut impl SignableRequest, we may have to change the api of Signer to allow it accept trait objects.

change Signer into an enum like:

pub enum Signer {
  V4(V4Signer),
  Boxed(Box<dyn CustomSigner>)
}

this approach can make the api of Signer unchanged, but would introduce a small dispatch cost on every sign.
@Xuanwo any suggestion about this?

Hi, S3::customized_credential_load should allow you to implement your own logic.

Hi, S3::customized_credential_load should allow you to implement your own logic.

imho customized_credential_load is to customize the credential logic, but here we need customize is the signing logic.
the Signer currently is not a trait like dyn AwsCredentialLoad yet.

Thank you for the clarification. I now understand what the problem is.
My current ideas:

Add an Sign trait for entire reqsign
use http;

#[async_trait]
trait Sign: 'static {
	type Credential;

    async fn sign(&self, req: &mut http::request::Parts, cred: &Self::Credential) -> Result<()>
}

Expose a Signer from the reqsign
reqsign will expose a type AwsSigner = Arc<dyn Sign<Credential=AwsCredential>>.

OpenDAL can accept any impl Sign<Credential=AwsCredential> with AwsV4Signer as default impl.

@Xuanwo, I'm slightly confused about the the state of reqsign with respect to configurable signers.
The new Sign trait was introduced in https://github.com/Xuanwo/reqsign/pull/459 and then replaced in https://github.com/Xuanwo/reqsign/pull/482 with the Load and Build traits which should achieve the same.
I'm confused about how reqsign is published (new to Rust). The 0.16.1 crate docs have an AwsV4Signer which I don't find in the code. And the 0.16.1 crate docs don't have Load or Build traits. (But I also can't find the reqsign_core and the reqsign_aws_v4 crates published so I think I'm missing something.)

@rshkv The code in v0.16.x hasn't been merged into main. Maybe that's the issue.

The new Sign trait was introduced in Xuanwo/reqsign#459 and then replaced in Xuanwo/reqsign#482 with the Load and Build traits which should achieve the same.

Hi, that will be included in the 0.17 release.

Ah, gotcha. Thank you for the quick reply.


Y,Cb,Cr channels of the image?

Hi @RjDuan
Thank you for your work.
While creating components in following code, RGB channels of the images are being treated as YCbCr channels. If this observation is correct, could you please let me the reason behind re-naming RGB channels to YCbCr channels. Or someone should actually convert an image from RGB to YCbCr color space.
https://github.com/RjDuan/AdvDrop/blob/35ceeb0e1c4b258bf3995302fbce9f13b39860f0/infod_sample.py#L72
Thank you.

Hi Asif,
Thanks for your interest.
Converting RGB to YCbCr is part of JPEG compression. It claims human eyes
are more sensitive to Green color, thus compression in YCbCr space better
reserves information in green color (when you enlarge epsilon, you can
observe more green stuff reserves)
Hope it helps.
Ranjie Duan
Asif Hanif @.***> 于2022年11月26日周六 03:20写道：

Hi @RjDuan https://github.com/RjDuan
Thank you for your work.
While creating components in following code, RGB channels of the images
are being treated as YCbCr channels. If this observation is correct, could
you please let me the reason behind re-naming RGB channels to YCbCr
channels. Or someone should actually convert an image from RGB to YCbCr
color space.
https://github.com/RjDuan/AdvDrop/blob/35ceeb0e1c4b258bf3995302fbce9f13b39860f0/infod_sample.py#L72
Thank you.
—
Reply to this email directly, view it on GitHub
https://github.com/RjDuan/AdvDrop/issues/18, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKEPNZZGZVTGXAZR7U2MXCLWKEGQHANCNFSM6AAAAAASLSD5QY
.
You are receiving this because you were mentioned.Message ID:
@.***>

Thank you for the response. I get your point about significance of YCbCr color space.
My question is about the lines of code that are converting RGB to YCbCr color space. As far as I can see, RGB channels of the input images are not being converted to YCbCr in the code. Rather RGB channels are simply being considered YCbCr channels. Could you let me know the part of the code in the repo which is actually converting image from RGB to YCbCr color space.
Thank you.

Oh, sorry, I forgot I had commented out this part of code = = .
The code is in compression.py (def rgb_to_ycbcr(image)).
As with increasing of epsilon, I found the green blocks left are more
perceptible, so I commented out the transfer between rgb to YCbCr.
When the epsilon is small, there is no big difference.
You can try it by feeding images into def rgb_to_ycbcr() and then
performing the attack.
This part has no influence on the success rate, but impacts the
stealthiness of the generated adv. examples.
Hope it helps.
Asif Hanif @.***> 于2022年11月26日周六 18:57写道：

Thank you for the response. I get your point about significance of YCbCr
color space.
My question is about the lines of code that are converting RGB to YCbCr
color space. As far as I can see, RGB channels of the input images are not
being converted to YCbCr in the code. Rather RGB channels are simply being
considered YCbCr channels. Could you let me know the part of the code in
the repo which is actually converting image from RGB to YCbCr color space.
Thank you.
—
Reply to this email directly, view it on GitHub
https://github.com/RjDuan/AdvDrop/issues/18#issuecomment-1328024579, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AKEPNZ4J4ZX7NYIOTWHNSZLWKHUJPANCNFSM6AAAAAASLSD5QY
.
You are receiving this because you were mentioned.Message ID:
@.***>

Thanks @RjDuan for clarification.
My query has been addressed, so I am closing the issue.


Ask Password claim value not honoured when creating user with SCIM2

Describe the issue:
Ask Password claim value not honored when creating a user with SCIM2
How to reproduce:
Setup Identity server for user creation with ask password option[1].
Create a user with the following curl with askPassword set to false as shown below.
curl -v -k --user admin:admin --data '{"schemas":[],"name":{"familyName":"Tharaka","givenName":"Wijekoon"},"userName":"Tharaka","password":"password","emails":[{"primary":true,"value":"tharakawijekoon2@gmail.com"}],"EnterpriseUser":{askPassword:"false"}}' --header "Content-Type:application/json" https://localhost:9443/scim2/Users

An email will be sent to the user even though ask password is set to false.
Expected behavior:
If askPassword is set to false email should not be sent to the user. In the code, we only check whether the askPassword claim URI is present not the value[2].
Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: [IS 5.7.0]

[1]https://docs.wso2.com/display/IS570/Creating+Users+Using+the+Ask+Password+Option
[2]https://github.com/wso2-extensions/identity-governance/blob/master/components/org.wso2.carbon.identity.recovery/src/main/java/org/wso2/carbon/identity/recovery/handler/UserEmailVerificationHandler.java#L160

This issue is a duplicate of: https://github.com/wso2/product-is/issues/9458


Different Behavior of tf.raw_ops.Acos+tf.raw_ops.exp with jit_compile=True

Issue type
Bug
Have you reproduced the bug with TensorFlow Nightly?
No
Source
source
TensorFlow version
2.14.0
Custom code
Yes
OS platform and distribution
Ubuntu 22.04.3 LTS (x86_64)
Mobile device
No response
Python version
No response
Bazel version
No response
GCC/compiler version
No response
CUDA/cuDNN version
11.8
GPU model and memory
GPU 0: NVIDIA GeForce RTX 2070 GPU 1: NVIDIA GeForce RTX 2070 GPU 2: NVIDIA GeForce RTX 2070 GPU 3: NVIDIA GeForce RTX 2070
Current behavior?
When the tf.raw_ops.Acos+tf.raw_ops.Exp operation is invoked within a tf.function with JIT compilation enabled (jit_compile=True), it produces different results compared to the same operation called without JIT compilation. This inconsistency is observed when the code is executed on a CPU device.
Standalone code to reproduce the issue
import tensorflow as tf
import traceback

class Network(tf.Module):
    def __init__(self):
        super().__init__()

    @tf.function(jit_compile=True)
    def __call__(self, x):

      x = tf.raw_ops.Acos(x=x, )
      x = tf.raw_ops.Exp(x=x, )
      return x

m = Network()
dic = {'ele': (-731778.6211090556-59304.1731637927j), 'size': [], 'dtype': tf.complex128}
inp = {
    "x": tf.constant(dic['ele'], dtype=tf.as_dtype(dic['dtype'])),
}

with tf.device('/CPU:0'):
    tf.config.run_functions_eagerly(True)
    no_op_res = m(**inp)
    tf.config.run_functions_eagerly(False)
    with tf.device('/CPU:0'):
        op_res = m(**inp)

    tf.debugging.assert_near(tf.cast(no_op_res, tf.float64), tf.cast(op_res, tf.float64), atol=0.001, rtol=0.001)

Relevant log output
File "/home/guihuan/LLM/results/tf-2/2023-10-22-20-21/test.py", line 28, in <module>
    tf.debugging.assert_near(tf.cast(no_op_res, tf.float64), tf.cast(op_res, tf.float64), atol=0.001, rtol=0.001)
  File "/home/guihuan/.conda/envs/night/lib/python3.9/site-packages/tensorflow/python/util/traceback_utils.py", line 153, in error_handler
    raise e.with_traceback(filtered_tb) from None
  File "/home/guihuan/.conda/envs/night/lib/python3.9/site-packages/tensorflow/python/ops/control_flow_assert.py", line 102, in Assert
    raise errors.InvalidArgumentError(
tensorflow.python.framework.errors_impl.InvalidArgumentError: Expected 'tf.Tensor(False, shape=(), dtype=bool)' to be true. Summarized data: b''
b'x and y not equal to tolerance rtol = tf.Tensor(0.001, shape=(), dtype=float64), atol = tf.Tensor(0.001, shape=(), dtype=float64)'
b'x (shape=() dtype=float64) = '
-1.3327908001899897
b'y (shape=() dtype=float64) = '
-1.3287233642820495

Hi @zoux1a ,
The difference is only of precisional difference which is expected due to various fusion operations inside XLA and also it does float32 conversions internally causing some precisional changes. Please refer developer comment on similar behaviour.
The assertion is success if I modify rtol to 0.01 from 0.001. Please refer attached gist.


Challenge lab writeup missing

I would like to add the writeup. @itssingh please assign this to me.


Unexpected scatter behavior

To Reproduce
auto a = zeros({3, 4});
auto indices = {array({1}), array({2})};
auto axes = {0, 1};
auto updates = reshape(array({1, 2, 3, 4}, float32), {1, 2, 2});
auto out = scatter(a, indices, updates, axes);
std::cout << out << "\n";

Expected behavior
I would expect a 2x2 matrix pasted at the (1, 2) coordinate of the a matrix, ie:
array([[0, 0, 0, 0],
       [0, 0, 1, 2],
       [0, 0, 3, 4]], dtype=float32)

instead the update wraps over the next row:
array([[0, 0, 0, 0],
       [0, 0, 1, 2],
       [3, 4, 0, 0]], dtype=float32)

Desktop (please complete the following information):

OS Version: [e.g. MacOS 14.5]
Version [e.g. 0.15.0]

This is definitely a bug. Running the same computation on the CPU gives the expected result:
  auto a = zeros({3, 4});
  auto indices = {array({1}), array({2})};
  auto axes = {0, 1};
  auto updates = reshape(array({1, 2, 3, 4}, float32), {1, 2, 2});
  auto out = scatter(a, indices, updates, axes, Device::cpu);
  std::cout << out << std::endl;

array([[0, 0, 0, 0],
       [0, 0, 1, 2],
       [0, 0, 3, 4]], dtype=float32)

Do you want me to have a look at the GPU kernel?

I already started, I think its a bug in the 1d specialization. It should be a quick fix.


koji_upload: include koji identifiers in buildroot

Add some koji identification data to buildroot.extra.osbs.
Signed-off-by: Mark Langsdorf<EMAIL_ADDRESS>

Documentation added.

Added to the release notes under Improvements:

koji builds now display the koji name-nvr and digests with versions in the metadata under extra.osbs.koji


Change format of listcogs

Change the format of list cogs to look like the format in the branch dynamic-cogs which I'm not working on right now.

 Try upgrading locally and see if that matches.


Prevent close of remodal through event

It should be possible to cancel a close event triggered by close, cancel and confirmation buttons based on an event triggered. Suppose I have a modal and until I execute an specific action, I do not want it to be closed.
Remodal should trigger a before-close event that can prevent default behavior of closing.

:+1: very important and simple feature, miss this badly..

Actually it should behave like the Angular-UI Bootstrap modal. There is also a closing event.
They have this concept:
Finally, a modal.closing event is broadcast to the modal scope before the modal closes.
If the listener calls preventDefault on the event, then the modal will remain open.
The $close and $dismiss methods return true if the event was allowed.
The event itself includes a parameter for the result/reason and a boolean
parameter that indicates whether the modal is being closed (true) or dismissed.

I think you should do the same here

+1 to this. This is an important feature. Any chance of getting this included?

+1 to this as well, I could really use this on my current project. I came to the issues tab to make an issue about it.

One temporary solution:
$remodal-confirm.on('click', function (e) {
  e.preventDefault();
  e.stopPropagation();
});

I could really use this feature as well. One idea I have (yet to be proven) is using a transparent element that takes up the entire window, with a z-index greater than that of the modal while I want to disable 'clicking off' the modal. In my situation, the user doesn't need to interact with any other elements while I'm temporarily disabling the ability to dismiss the modal. This work around doesn't address the 'esc' key, however.

Could have really done with this feature on a project of mine! Needed to stop window from closing if AJAX from within modal failed and display a nice little error.


polkit: Apply unreleased patch to fix pkexec without a graphical agent

Merged upstream PR https://www.github.com/polkit-org/polkit/pull/423
Fixes https://github.com/NixOS/nixpkgs/issues/18012
Redo of https://github.com/NixOS/nixpkgs/pull/144797 now that the upstream PR is merged
Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

Just for context, in https://gitlab.freedesktop.org/polkit/polkit/-/merge_requests/59#note_2100970 you said that the previous iteration of that PR couldn't be reviewed and is security-critical; what changed?

Upstream has merged the PR

https://github.com/NixOS/nixpkgs/pull/322981

I still don't understand how the choice of forge would influence the decision to merge changes that cannot easily be reviewed?

I still don't understand how the choice of forge would influence the decision to merge changes that cannot easily be reviewed?

The quote I had in https://gitlab.freedesktop.org/polkit/polkit/-/merge_requests/59#note_2100970 was from @jtojnar https://github.com/NixOS/nixpkgs/pull/144797#issuecomment-962442463 (Who is not an upstream developer)
Now https://gitlab.freedesktop.org/polkit/polkit/-/merge_requests/59 has been recreated by an upstream developer https://github.com/polkit-org/polkit/pull/423 and reviewed&merged by the upstream developer


Fix various issues on the Dashboard

Pull Request Description

Fix React DevTools not working in Firefox
Fix selection of asset names when editing them, not working at all in Firefox
Convert tick/cross buttons when editing assets, and the "plus" and "reload" buttons on the "shared with" column, "labels" column, and keyboard shortcuts table, to match more with the rest of the design
Update clip path when the container resizes, so that the icons for hidden columns never overlap the actual table header
Fix #10184

Fix renames being committed even when cancelling

Fix duplicate name detection - previously, all asset types only checked folders with the same name, not assets with the same name

I'm not 100% sure this is the correct behavior still

Stop using kbd (aria.Keyboard) to display keyboard shortcuts, since they should not be displayed in a monospace font.
Fix "plus" and "reload" buttons going past the right side of their parent table cell
Limit length of PermissionDisplay - if the username of a user with permission is too long, it uses a tooltip instead
Update the username dynamically for all permissions owned by self, when changing username in the settings.

This avoids having to fully invalidate the directory tree every time the username changes, given that nothing changes about the assets' metadata themselves.

Cache children in the Drive tree

This avoids loading spinners when closing a folder and immediately reopening it.
Note that children are still re-fetched on reopen to ensure freshness

Important Notes

This MAY be split into multiple smaller PRs. However, I think it's better to QA as a single PR, to avoid duplicating work checking behavior that may be changed by a sibling PR (assuming the PR was split into multiple).

Testing Instructions

Mostly test that the above bugs have been fixed
But also test the rest of the app in general, because quite a few things are being changed here

Checklist
Please ensure that the following checklist has been satisfied before submitting the PR:

[x] The documentation has been updated, if necessary.
[ ] Screenshots/screencasts have been attached, if there are any visual changes. For interactive or animated visual changes, a screencast is preferred.
[x] All code follows the
Scala,
Java,
TypeScript,
and
Rust
style guides. In case you are using a language not listed above, follow the Rust style guide.
[x] Unit tests have been written where possible.

| Limit length of PermissionDisplay - if the username of a user with permission is too long, it uses a tooltip instead

How long is too long?

When dropping a label on directory PATCH request is being send twice (at least on Firefox)

Reload on firefox works selectively (sometimes I need to reload 2 or 3 times or hard reload for dashboard to apear)

Removing labels is not working on Firefox

and 5. addressed, will address the others soon

(@PabloBuchu i guess this PR is ready for re-QA?)

We should trigger rename by clicking anywhere on the name field, not only on the text

this should already be the case i think

When I quickly double-click on the name field, renaming doesn't show up

double clicking opens the asset (projects are run, directories are toggled).
the mouse shortcut for rename is supposed to be single click + single click - apparently this is the behavior of finder on macos?
will definitely address the other issues though

addressed
addressed
can't repro (after temporarily changing linux modifier keys to icons) - it may be easier for you to fix it?
WIP i guess
N/A (see above)
N/A (see above)
addressed

CRs should be addressed as well

also addressed

QA ✅


[BUG]

Describe the bug
libcamera-apps build fails to build on Raspberry Pi Zero 2 W (Linux raspberrypi 5.10.92-v8+ #1514 SMP PREEMPT Mon Jan 17 17:39:38 GMT 2022 aarch64 GNU/Linux) with  /usr/local/src/libcamera-apps/preview/egl_preview.cpp:18:10: fatal error: libdrm/drm_fourcc.h: No such file or directory
18 | #include <libdrm/drm_fourcc.h>
My operating system (downloaded pre-built image from Q Engineering similar to their new GitHub site -- thank you, folks at Q Engineering!):
pi@raspberrypi:/usr/local/src/libcamera-apps $ cat /etc/os-release
PRETTY_NAME="Debian GNU/Linux 11 (bullseye)"
NAME="Debian GNU/Linux"
VERSION_ID="11"
VERSION="11 (bullseye)"
VERSION_CODENAME=bullseye
ID=debian
HOME_URL="https://www.debian.org/"
SUPPORT_URL="https://www.debian.org/support"
BUG_REPORT_URL="https://bugs.debian.org/"
pi@raspberrypi:/usr/local/src/libcamera-apps $

Should pre-compile tests look for library/header files libdrm?
For the benefit of other would-be compilers on the Pi Zero 2 running Bullseye: before I could commence the build, I had installed :
 exif
 libexif-dev
 libodb-boost-2.4
 libboost-system1.74.0
 libboost-thread-dev
 libboost-program-options-dev

Bug report
bug-report_20220225_0821.txt
Add any other context about the problem here.
Build commenced around 7:53 p.m. and error-out at 10:58 p.m., hence 3 hours, with 44% completion
Here's the console:
pi@raspberrypi:/usr/local/src/libcamera-apps $ make
Scanning dependencies of target images
[  2%] Building CXX object image/CMakeFiles/images.dir/bmp.cpp.o
[  4%] Building CXX object image/CMakeFiles/images.dir/yuv.cpp.o
[  6%] Building CXX object image/CMakeFiles/images.dir/jpeg.cpp.o
[  8%] Building CXX object image/CMakeFiles/images.dir/png.cpp.o
[ 10%] Building CXX object image/CMakeFiles/images.dir/dng.cpp.o
[ 12%] Linking CXX shared library libimages.so
[ 12%] Built target images
Scanning dependencies of target post_processing_stages
[ 14%] Building CXX object post_processing_stages/CMakeFiles/post_processing_stages.dir/post_processing_stage.cpp              .o
[ 17%] Building CXX object post_processing_stages/CMakeFiles/post_processing_stages.dir/negate_stage.cpp.o
[ 19%] Building CXX object post_processing_stages/CMakeFiles/post_processing_stages.dir/hdr_stage.cpp.o
date
[ 21%] Building CXX object post_processing_stages/CMakeFiles/post_processing_stages.dir/pwl.cpp.o
[ 23%] Building CXX object post_processing_stages/CMakeFiles/post_processing_stages.dir/histogram.cpp.o
[ 25%] Building CXX object post_processing_stages/CMakeFiles/post_processing_stages.dir/motion_detect_stage.cpp.o
[ 27%] Building CXX object post_processing_stages/CMakeFiles/post_processing_stages.dir/sobel_cv_stage.cpp.o
[ 29%] Building CXX object post_processing_stages/CMakeFiles/post_processing_stages.dir/face_detect_cv_stage.cpp.              o
[ 31%] Building CXX object post_processing_stages/CMakeFiles/post_processing_stages.dir/annotate_cv_stage.cpp.o
[ 34%] Building CXX object post_processing_stages/CMakeFiles/post_processing_stages.dir/plot_pose_cv_stage.cpp.o
[ 36%] Building CXX object post_processing_stages/CMakeFiles/post_processing_stages.dir/object_detect_draw_cv_sta              ge.cpp.o
[ 38%] Linking CXX shared library libpost_processing_stages.so
[ 38%] Built target post_processing_stages
Scanning dependencies of target VersionCpp
Generating version string: 56721d65cfe1-intree 24-02-2022 (20:56:09)
[ 38%] Built target VersionCpp
Scanning dependencies of target preview
[ 40%] Building CXX object preview/CMakeFiles/preview.dir/null_preview.cpp.o
[ 42%] Building CXX object preview/CMakeFiles/preview.dir/preview.cpp.o
[ 44%] Building CXX object preview/CMakeFiles/preview.dir/egl_preview.cpp.o
/usr/local/src/libcamera-apps/preview/egl_preview.cpp:18:10: fatal error: libdrm/drm_fourcc.h: No such file or di              rectory
   18 | #include <libdrm/drm_fourcc.h>
      |          ^~~~~~~~~~~~~~~~~~~~~
compilation terminated.
make[2]: *** [preview/CMakeFiles/preview.dir/build.make:108: preview/CMakeFiles/preview.dir/egl_preview.cpp.o] Er              ror 1
make[1]: *** [CMakeFiles/Makefile2:407: preview/CMakeFiles/preview.dir/all] Error 2
make: *** [Makefile:171: all] Error 2
pi@raspberrypi:/usr/local/src/libcamera-apps $ date
Thu 24 Feb 20:58:21 PST 2022
pi@raspberrypi:/usr/local/src/libcamera-apps $  git status
On branch main
Your branch is up to date with 'origin/main'.

Untracked files:
  (use "git add <file>..." to include in what will be committed)
   CMakeCache.txt
   CMakeFiles/
   CPackConfig.cmake
   CPackSourceConfig.cmake
   Makefile
   apps/CMakeFiles/
   apps/Makefile
   apps/cmake_install.cmake
   cmake_install.cmake
   compile_commands.json
   core/CMakeFiles/
   core/Makefile
   core/cmake_install.cmake
   core/version.cpp
   encoder/CMakeFiles/
   encoder/Makefile
   encoder/cmake_install.cmake
   image/CMakeFiles/
   image/Makefile
   image/cmake_install.cmake
   image/libimages.so
   output/CMakeFiles/
   output/Makefile
   output/cmake_install.cmake
   post_processing_stages/CMakeFiles/
   post_processing_stages/Makefile
   post_processing_stages/cmake_install.cmake
   post_processing_stages/libpost_processing_stages.so
   preview/CMakeFiles/
   preview/Makefile
   preview/cmake_install.cmake
   utils/CMakeFiles/
   utils/Makefile
   utils/cmake_install.cmake

nothing added to commit but untracked files present (use "git add" to track)
pi@raspberrypi:/usr/local/src/libcamera-apps $ date
Fri 25 Feb 08:15:37 PST 2022
pi@raspberrypi:/usr/local/src/libcamera-apps $ pi@raspberrypi:/usr/local/src/libcamera-apps $ git log
commit 56721d65cfe157056bb239a960c6fbd09f7cff0d (HEAD -> main, origin/main, origin/HEAD)
Author: Devendra Tewari<EMAIL_ADDRESS>Date:   Mon Feb 21 19:42:08 2022 -0300

    Include sstream header

    Fix for fatal error: implicit instantiation of undefined
    template 'std::basic_stringstream<char>'

Sounds like maybe no one installed libdrm-dev? (as per https://www.raspberrypi.com/documentation/accessories/camera.html#building-libcamera-apps)
Alternatively you could use -DENABLE_DRM=0 when you run cmake.

Oops.  I was unaware of the Build instructions as David cited in the comment preceding this.  I missed the link under "here" in the README.txt.


feat: add to cart api added

Related Issue
Closes: #708
Changes made 👷🏻♂
Add to cart api is added when user add the product it will add by unique ID

Screenshots  📸

@tamalCodes  review it

@tamalCodes

@tamalCodes please reply fast so that i can able to work today ASAP

https://www.loom.com/share/91155c3b3e3945d396f01f868b42502b

@tamalCodes see!

Thankyou Sir It is Great Experience With You @tamalCodes


search -g not return anything?

rustsym search -g . main

returns

[]

while

rustsym search -l src/main.rs main

returns

[{"path":"src/main.rs","name":"main","container":"","kind":"function","line":8}]

Hmm, it might be erroring on one of the files in the src directory. What is the directory structure of .? Are there any more files other than main.rs?

for your information:

$ rustsym search . main -g
[]
$ ls
Cargo.lock  Cargo.toml  src  target
$ ls src/
main.rs

Strange, its working fine here with that same structure. What OS are you running?

fyi:

Distributor ID: Debian
Description:    Debian GNU/Linux 8.5 (jessie)
Release:        8.5
Codename:       jessie

Can confirm this is happening on Linux. Seems to be a sporadic failure.. sometimes I'm getting a panic. I'll get this fixed and deploy a new version.


CI: Display LightHouse stats in nicer way

Fixes #582
This PR:

Formats a comment displaying the LightHouse stats in a visual and tabular manner
Uses<EMAIL_ADDRESS>to put up the comment on the PR
Adds a check to run the LightHouse Statistic Comment only over a GitHub PR

Here are the LightHouse Stats:

Category
Score

🟢 Performance
96

🟢 Accessibility
97

🟢 Best practices
100

🟢 SEO
98

🔴 PWA
40

Currently, the comment is not being put up, since the GitHub Token is not accessible by integration (since the token is not available on forked repositories).

Thank you, this looks like a good approach! As mentioned in code review, I would suggest to use gh which is pre-installed on GitHub runners to reduce third-pary dependencies.
Also, the following permissions may be needed:
permissions:
  pull-requests: write

My guess is that with actions being quite dynamic ecosystem we should reconsider the specific actions proposed. This PR is valuable but if it staying in unfinished state discourages other contributors let's close it. Also for now the priority would be fixing ci in the first place


Fix data resync test and add more tests to l3 validator

Coverage remained the same at 75.804% when pulling d5cf2c013de8607d52756d67a6595fcee552ea32 on jmedved:master into 1aaadf4ed84a7750b9e508694e58220e3460b00e on contiv:master.


Cannot build cquery

Waf: Entering directory `/home/agauniyal/projects/cquery/build'
CLANG_INCLUDE_DIR: /home/agauniyal/projects/cquery/build/clang+llvm-4.0.0-x86_64-linux-gnu-ubuntu-14.04/include
CLANG_LIB_DIR:     /home/agauniyal/projects/cquery/build/clang+llvm-4.0.0-x86_64-linux-gnu-ubuntu-14.04/lib
[ 1/33] Compiling src/cache.cc
[ 2/33] Compiling src/clang_complete.cc
[ 3/33] Compiling src/clang_cursor.cc
[ 4/33] Compiling src/clang_index.cc
[ 5/33] Compiling src/clang_translation_unit.cc
[ 6/33] Compiling src/clang_utils.cc
[ 7/33] Compiling src/command_line.cc
[ 8/33] Compiling src/file_consumer.cc
[ 9/33] Compiling src/include_complete.cc
[10/33] Compiling src/indexer.cc
[11/33] Compiling src/ipc.cc
[12/33] Compiling src/ipc_manager.cc
[13/33] Compiling src/language_server_api.cc
../src/command_line.cc:2007:16: error: lambda capture 'working_files' is not used [-Werror,-Wunused-lambda-capture]
              [working_files, global_code_complete_cache,
               ^
../src/command_line.cc:3014:37: error: lambda capture 'config' is not used [-Werror,-Wunused-lambda-capture]
  WorkThread::StartThread("stdin", [config, request_times]() {
                                    ^
2 errors generated.

Waf: Leaving directory `/home/agauniyal/projects/cquery/build'
Build failed
 -> task in 'app' failed with exit status 1 (run with -v to display more information)

After fixing that -
Waf: Entering directory `/home/agauniyal/projects/cquery/build'
CLANG_INCLUDE_DIR: /home/agauniyal/projects/cquery/build/clang+llvm-4.0.0-x86_64-linux-gnu-ubuntu-14.04/include
CLANG_LIB_DIR:     /home/agauniyal/projects/cquery/build/clang+llvm-4.0.0-x86_64-linux-gnu-ubuntu-14.04/lib
[ 7/33] Compiling src/command_line.cc
[14/33] Compiling src/lex_utils.cc
[15/33] Compiling src/match.cc
[16/33] Compiling src/options.cc
[17/33] Compiling src/platform.cc
[18/33] Compiling src/platform_linux.cc
[19/33] Compiling src/platform_win.cc
[20/33] Compiling src/position.cc
[21/33] Compiling src/project.cc
[22/33] Compiling src/query.cc
[23/33] Compiling src/query_utils.cc
[24/33] Compiling src/serializer.cc
[25/33] Compiling src/standard_includes.cc
[26/33] Compiling src/task.cc
[27/33] Compiling src/test.cc
[28/33] Compiling src/third_party_impl.cc
[29/33] Compiling src/timer.cc
[30/33] Compiling src/utils.cc
[31/33] Compiling src/work_thread.cc
[32/33] Compiling src/working_files.cc
[33/33] Linking build/app
/usr/bin/ld: warning: libtinfo.so.5, needed by /home/agauniyal/projects/cquery/build/clang+llvm-4.0.0-x86_64-linux-gnu-ubuntu-14.04/lib/libclang.so, not found (try using -rpath or -rpath-link)
/home/agauniyal/projects/cquery/build/clang+llvm-4.0.0-x86_64-linux-gnu-ubuntu-14.04/lib/libclang.so: undefined reference to `setupterm'
/home/agauniyal/projects/cquery/build/clang+llvm-4.0.0-x86_64-linux-gnu-ubuntu-14.04/lib/libclang.so: undefined reference to `tigetnum'
/home/agauniyal/projects/cquery/build/clang+llvm-4.0.0-x86_64-linux-gnu-ubuntu-14.04/lib/libclang.so: undefined reference to `set_curterm'
/home/agauniyal/projects/cquery/build/clang+llvm-4.0.0-x86_64-linux-gnu-ubuntu-14.04/lib/libclang.so: undefined reference to `del_curterm'
clang-5.0: error: linker command failed with exit code 1 (use -v to see invocation)

Waf: Leaving directory `/home/agauniyal/projects/cquery/build'
Build failed
 -> task in 'app' failed with exit status 1 (run with -v to display more information)

Hmm, this worked - https://github.com/commercialhaskell/stack/issues/1012#issuecomment-285555664

Would it be possible to use system installed clang instead of downloading clang 4. I've more recent one (clang 5) and would like to use that instead 😛

Everything works perfectly! Finally, a good cpp extension with which I'm satisfied with. Autocompletion is much better than other extensions, goTo support is excellent and overall integration with vscode is nice. Thank you for your work 😁

Would it be possible to use system installed clang instead of downloading clang 4. I've more recent one (clang 5) and would like to use that instead 😛

clang5 is actually broken atm - it is on my todo list to investigate why. clang4 seems to work a bit better.

Everything works perfectly! Finally, a good cpp extension with which I'm satisfied with. Autocompletion is much better than other extensions, goTo support is excellent and overall integration with vscode is nice. Thank you for your work 😁

Thanks! Feel free to submit any feature requests/etc.


commit

@rpreslar4765 thank you for the PR. This seems to be a mistake PR with many merge conflicts. I'll close for now


File Display Names Different Than File Names

As a repository manager,
I would like to be able to set the display name of files attached to an object,
So that I don't need to format file names for display.
Ann P.

Use the same filename when downloading.

Is this really a ticket about file-level descriptive metadata? Is that the same thing as #63?

No. This is about metadata on the file object itself.
TODO:

edit form for the file object
add one field for display name
the file name should remain the same as it was uploaded and changing the display name will not change how the file is downloaded

Since the implementation of file sets, which metadata, is this user story applicable anymore?

I don't think it's applicable anymore. The FileSet can have a title, which would serve as the display name. We could add a new field to the File object, which could be editable, but I think this is overkill. For now, the File will be named the same as it was uploaded and the FileSet could be anything, but will default to the name of the preservation file, i.e. the file that was uploaded or the file with the _preservation designation.


NextJS 7+  /_next/static/images to /_next/static/{buildid}/images

How does this plugin work with NextJS 7+? in next.js 7+ the build output was changed from from /_next/static/images to /_next/static/{buildid}/images

Hi @msreekm
This plugin actually doesn't use the buildid folder on purpose. The reason is that the images already have a hash of their content included in the filename (/_next/static/images/avatar-1e8c19705d29f56c9eb7156443769215.png). If an image gets changed/overwritten, the hash will also change and so gets a new filename and the new image gets required anywhere (also on a cdn etc).
If it would be within the build folder, the image would always have a new url on a new build, even if it wasn't changed (and images usually don't change that often), and so it can't be requested from browser cache and also needs to be cached separately on a cdn. So, it was a performance decision to not use this buildid folder.
It still puts all images within /_next/static/images which isn't a problem at all, they still get served correctly.
Did you encounter a problem with this approach or would you still like to put them into the buildid folder, regardless of the performance argument?
We actually already provide a setting to overwrite the target folder (https://github.com/cyrilwanner/next-optimized-images#imagesfolder), if you want, I can easily extend this setting to also allow a function which gets the buildid as an argument, so you could put them into this folder (e.g. (buildId) => `${buildId}/images` ).


Fix file contents retrieval

Greetings!
First, thank you for this adapter, it saved me a lot of work already. I am using version 2.x of google/apiclient and noticed that the file contents are not retrieved correctly (instead of the contents, I got a Response object.
I fixed this in my project by extending the Adapter, but as I just found your 2.x branch, I would like to propose the change I made here as well :).
:octocat:

This would be a good moment to remove the RC requirement, wouldn't it?

Indeed, it would :)

Thanks, taking a look at this now.
It looks like they made a change between RC and V2 which broke this.

We've released v3.0.0 which uses the new google/pubsub package, instead of the standard php api client.
This should be resolved with an update.
The instantiation of the adapter has however changed (slightly).


[Discussion] Python Version

when installing it says.

Specifications:

spleeter -> python[version='>=3.6,<3.7.0a0|>=3.7,<3.8.0a0']

Your python: python=3.8

what does this mean?

=3.6,<3.7.0a0|>=3.7,<3.8.0a0

i mean, to which version i have to switch?

Hi @muZa11
How are you trying to install spleeter ?
Spleeter is supposed to be compatible with python 3.8 by now, so I'd like to be able to reproduce your error.

Hi @muZa11
How are you trying to install spleeter ?
Spleeter is supposed to be compatible with python 3.8 by now, so I'd like to be able to reproduce your error.

steps:

sudo -i
apt install libgl1-mesa-glx libegl1-mesa libxrandr2 libxrandr2 libxss1 libxcursor1 libxcomposite1 libasound2 libxi6 libxtst6
downloaded anaconda for linux x84 64bit
bash ~/anacodaXXX.sh
conda install -c conda-forge spleeter

i have also tried pip method but couldn't installed libsndfiles so rollback to anaconda method.
I am in Kali Linux.

steps:

sudo -i
apt install libgl1-mesa-glx libegl1-mesa libxrandr2 libxrandr2 libxss1 libxcursor1 libxcomposite1 libasound2 libxi6 libxtst6
downloaded anaconda for linux x84 64bit
bash ~/anacodaXXX.sh
conda install -c conda-forge spleeter

i have also tried pip method but couldn't installed libsndfiles so rollback to anaconda method.
I am in Kali Linux.

see this screenshot. just tried again.
https://drive.google.com/file/d/1vqynF89ERXW1AKa4CnPcQu35M8oEmoTI/view?usp=sharing

see this screenshot. just tried again.
https://drive.google.com/file/d/1vqynF89ERXW1AKa4CnPcQu35M8oEmoTI/view?usp=sharing

Hi @muZa11
We've just revamped the conda package and it has a new location. Can you try conda install -c deezer-research spleeter ?

Hi @muZa11
We've just revamped the conda package and it has a new location. Can you try conda install -c deezer-research spleeter ?

Is there any stem separation improvements over the original 1.43 that I have been using fine on my Windows 10 computer as reading all these emails, it seems like a lot of trouble to upgrade it so have been leaving mine with the original version?
Rog
 On Thursday, January 21, 2021, 07:17:33 AM CST, Moussallam<EMAIL_ADDRESS>wrote:

Hi @muZa11
We've just revamped the conda package and it has a new location. Can you try conda install -c deezer-research spleeter ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Is there any stem separation improvements over the original 1.43 that I have been using fine on my Windows 10 computer as reading all these emails, it seems like a lot of trouble to upgrade it so have been leaving mine with the original version?
Rog
 On Thursday, January 21, 2021, 07:17:33 AM CST, Moussallam<EMAIL_ADDRESS>wrote:

Hi @muZa11
We've just revamped the conda package and it has a new location. Can you try conda install -c deezer-research spleeter ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

the version problem is gone but
https://drive.google.com/file/d/18dtIeBIPe9ujw3niFuKEuk1KJtQD7e4C/view?usp=sharing

the version problem is gone but
https://drive.google.com/file/d/18dtIeBIPe9ujw3niFuKEuk1KJtQD7e4C/view?usp=sharing

There is indeed an issue with conda install. We'll have a look into it.

There is indeed an issue with conda install. We'll have a look into it.

Hi @muZa11
run the command conda config --add channels conda-forge before the spleeter install one and it should be good. I'll update the documentation accordingly.
Let me know if that does not solve your issue

Hi @muZa11
run the command conda config --add channels conda-forge before the spleeter install one and it should be good. I'll update the documentation accordingly.
Let me know if that does not solve your issue


Added MultiFunctionCorrelationModel and supporting classes.

Added SPDCorrelationFunction base class and three derived classes :  FourParameterCorrelationFunction, LinearDecayCorrelationFunction, and DampedCosineCorrelationFunction.
Modified FourParameterCorrelationModel and LinearDecayCorrelationModel classes to use new correlation functions for consistency.
Added new MultiFunctionCorrelationModel class which can utilize multiple (different) derived instances of SPDCorrelationFunction .

I thought this already existed (that 4param and lineardecay inherited from a base SPD class/interface), but I guess what you're doing here is making up for the fact that the SPD base didn't exist, it was just 4param/LD out there as two unrelated classes?
Why are there .o files included in the commit?

requirement_brief_v2.pdf
This is the briefing which is scheduled to be presented on 1/12/22

I updated the files to provide a bit more accessibility to the parameters / parameter names.   I also added the ConstantCorrelationFunction to allow a constant correlation to be represented, since this is explicitly disallowed by the defined parameter ranges for the FourParameter Model.   (commit # 6)   Kevin Lam (BAE) found a bug in the updated LinearDecayCorrelationFunction class which I also fixed ( commit # 7)

Irwin Krinsky and Kevin Lam (MSP team) pointed out some inefficiencies with the design, stemming partially from populating the parameter names for each function...  Unfortunately, the names can't just be defined statically in all cases, since some, like the like the Linear Decay function, have a variable number of parameters.   I reworked the functions so that they parameter names / values aren't populated until asked for.  While this does still take time to do, it removes any of the associated computational inefficences, but it is doubtful someone would query for them often...   For most scenarios, all the user really needs is the computed correlation coefficient, which doesn't require the parameter names/values to be externally available.


Google Play Books can not be read

#18 my submitted pull request adds a whitelist feature.
@octatone has not responded to any messages or emails about this pull request.


MINOR: Fixed broken links in the documentation

@hachikuji More doc fixes, this time on broken links. Thanks for taking a look.

@vahidhashemian Thanks for the fixes. We had a separate issue for the varnish link which I merged separately (since it had been open for quite a long time).

@hachikuji Yes, just saw it this morning. I rebased the PR. The Jenkins error should be unrelated.

LGTM. Thanks again for looking closely at the docs!


adding exect/execvP for FreeBSD/DragonflyBSD

@bors r+

:pushpin: Commit cefa53b7bbab47586f76faeb190cd3240f16a8fc has been approved by JohnTitor
It is now in the queue for this repository.

:hourglass: Testing commit cefa53b7bbab47586f76faeb190cd3240f16a8fc with merge f0a9334cc1ffebd2948e0215b557af15f8a6f0d9...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13, checks-cirrus-freebsd-14
Approved by: JohnTitor
Pushing f0a9334cc1ffebd2948e0215b557af15f8a6f0d9 to main...


[Bug][Gitlab] GitLab pipeline jobs collection takes a long time

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
In version v0.16.0, when collecting job data of GitLab pipeline, each execution will repeatedly collect all job data within the specified range, resulting in a long collection time.
What do you expect to happen
If the job status is completed and has not been updated, the job information will not be collected again.
How to reproduce
Run GitLab pipeline collection twice.
Anything else
No response
Version
v0.16.0
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

@kyleliu1008 Hi, thanks for the report. However, I am unable to reproduce the problem. The incremental collection seems to be working fine on v0.16.0

Please provide more detailed information

The above screenshots are logs from two executions, both of which show full data collection was performed.

@kyleliu1008 Interesting, I believe the problem is caused by the status=created, collector would re-fetch those unfinished jobs by design and yes, it is totally normal for "duplicated" records in the raw table.
Are they all with pending status? Neither success nor failed? Is that normal for your jobs to be in the queue for days?

Indeed, there are many jobs in status=created among the projects I have collected, and this should be a problem with the CI settings. However, there are already a large number of these types of jobs, and I hope not to repeat collecting them. Is there a better solution to identify whether a job needs to be updated, such as the ”last update time“?

@klesh What will happen if we do not re-fetch the 'created' status GitLab job?

Indeed, there are many jobs in status=created among the projects I have collected, and this should be a problem with the CI settings. However, there are already a large number of these types of jobs, and I hope not to repeat collecting them. Is there a better solution to identify whether a job needs to be updated, such as the ”last update time“?

No, not really since the API doesn't support such a filtering option.

@klesh What will happen if we do not re-fetch the 'created' status GitLab job?

That is a good idea, I think it makes sense to fetch only the finished job only.

In fact, there is another phenomenon related to the data in the "_raw_gitlab_api_job" table: when the pipeline is in a non-completed state, devlake will repeatedly collect all jobs under the pipeline, causing the jobs data corresponding to the pipeline in the "_raw_gitlab_api_job" table to double each time it is collected, leading to the increasing size of the table. This problem becomes more prominent when there are a large number of blocked pipelines or when pipelines have long lifecycles (of course, these issues should be improved in CI practices).
Of course, the raw table is meant to record the originally collected data, but the data volume of the devlake table I deployed quickly reached millions. This is related to the fact that our project generates a lot of invalid pipelines (which we are also working on resolving). If devlake could recognize that the pipeline and its corresponding job have not changed and not collect them again, it would be perfect.

In fact, there is another phenomenon related to the data in the "_raw_gitlab_api_job" table: when the pipeline is in a non-completed state, devlake will repeatedly collect all jobs under the pipeline, causing the jobs data corresponding to the pipeline in the "_raw_gitlab_api_job" table to double each time it is collected, leading to the increasing size of the table. This problem becomes more prominent when there are a large number of blocked pipelines or when pipelines have long lifecycles (of course, these issues should be improved in CI practices).
Of course, the raw table is meant to record the originally collected data, but the data volume of the devlake table I deployed quickly reached millions. This is related to the fact that our project generates a lot of invalid pipelines (which we are also working on resolving). If devlake could recognize that the pipeline and its corresponding job have not changed and not collect them again, it would be perfect.

@kyleliu1008 Yes, I understand your point perfectly. However our hands are tied when Gitlab API doesn't support filtering records by updated field, the idea of "devlake could recognize that the pipeline and its corresponding job have not changed and not collect them again" is logically impossible, we are simply unable to predict whether or not a pipeline/job has been updated without fetching them from the API.  In case you may wonder if we could compare the fetched record against the database before writing to the _raw_table, the answer is NO since it would create a cyclic dependency between subtasks which would create problems for #5411
Anyway, it won't be a problem if we collect only those FINISHED records, which are supported by the Gitlab API by the scope parameter. Do you think it is ok for your case?

This would be great as it can prevent repeatedly collecting Jobs that have not been completed (with the status of "created", "manual", "skipped"). Will it also have no impact on metrics?
I found that archived jobs are also being collected.

The details returned by the archived job include the artifacts_file information.

interesting, what does "archived" mean? why shouldn't they be collected?
cc @Startrekzky


ion refresher running continiosly and get this error

TypeError: Cannot read property 'nativeElement' of null
at LoadingWpPopIn.init (http://localhost:8100/build/main.js:118046:46)
at OverlayPortal.NavControllerBase._trnsStart (http://localhost:8100/build/main.js:32021:20)
at LoadingWpPopIn._trnsStart (http://localhost:8100/build/main.js:31987:19)
at LoadingWpPopIn.Transition.start (http://localhost:8100/build/main.js:15904:33)
at OverlayPortal.NavControllerBase._transition (http://localhost:8100/build/main.js:32004:24)
at OverlayPortal.NavControllerBase._postViewInit (http://localhost:8100/build/main.js:31866:18)
at OverlayPortal.NavControllerBase._viewTest (http://localhost:8100/build/main.js:31953:25)
at OverlayPortal.NavControllerBase._nextTrns (http://localhost:8100/build/main.js:31725:25)
at OverlayPortal.NavControllerBase._queueTrns (http://localhost:8100/build/main.js:31689:14)
at OverlayPortal.NavControllerBase.insertPages
myionic info:-

@abmnukmr  I can't reproduce this problem, can you provide some source code that triggers this crash?

Hello! As it seems it has been a while since there was any activity on this issue i will be closing it for now. Feel free to comment if you are still running into this issue. Thanks for using Ionic!


Determine supported browser versions and whether we need to set up additional polyfills for older browsers

create-react-app documentation is a bit confusing on this point. It says

The browserslist configuration controls the outputted JavaScript so that the emitted code will be compatible with the browsers specified

but it also says

Note that this does not include polyfills automatically for you. You will still need to polyfill language features (see above) as needed based on the browsers you are supporting.

... which sounds contradictory to me. One thing is clear, IE 11 and below require polyfills if we want to support that. This may come with the tradeoff of a bigger binary/slower code.
Also, the default config in package.json is
"production": [
    ">0.2%",
    "not dead",
    "not op_mini all"
],

I think the first line means browser versions that have >0.2% usage. Not sure what the other two lines are or if these lines are treated as a union or intersection.
AIs here:

Investigate how create-react-app builds work to clarify what browsers it supports out of the box without additional polyfills
Determine what browsers we're currently supporting with the default config, and whether we want to change the config to support older browsers (this likely requires product input).
Determine whether we need any polyfills based on 1 and 2.

This is not time sensitive as it doesn't affect development. However we should avoid using experimental/non-standard features.

Larry is going to do an inventory to see how much of this, if any, we need to do before launch


fix(14232): pasting from ms word

Details

This PR fixes the issue with copying text from MS word. Prior to the fix, any text pasted into the app from MS Word resulted in an image representation of the text instead of html.
Fixed Issues

$ https://github.com/Expensify/App/issues/14232
PROPOSAL: https://github.com/Expensify/App/issues/14232#issuecomment-1407475708
Tests

Open the app
Open any report
Open Microsoft Word and write anything in it
Copy and paste it in report chat and send

[x] Verify that text gets pasted instead of image
[x] Verify that no errors appear in the JS console

Offline tests

QA Steps

Open the app
Open any report
Open Microsoft Word and write anything in it
Copy and paste it in report chat and send

[x] Verify that text gets pasted instead of image
[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is correct English and approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web
https://user-images.githubusercontent.com/30054992/215921891-16ed3906-1516-4aac-b3af-211da17ea9a1.mov
https://user-images.githubusercontent.com/30054992/216148171-743697ed-02cf-48c5-a7bf-3c759e39186e.mp4

Desktop
https://user-images.githubusercontent.com/30054992/215921810-d5dfd423-9ec1-40a7-9625-621b99001fc9.mov
https://user-images.githubusercontent.com/30054992/216148435-f225f134-edb3-460f-933d-983f598067fd.mp4

There are some cases I suggest to test:

Copy text only, image only (both from selection and from context menu), mixed (both text and image) from these resources:

Slack
Mac Word
Windows Word
any public app or website where image link is publicly accessible

On Windows 10, take screenshot using Start+Shift+S -> this will automatically copy image to clipboard
On Mac, Take screenshot using Shift+Command+4 and copy from context menu or Command+C

Copy single file or multiple files from explorer (windows) or finder (mac)

@0xmiroslav  Do I need to test these?

Seems to work. I wasn’t able to test on Windows though as I’m on mac.
On Thu, 2 Feb 2023 at 9:42 PM, Miroslav Stevanovic @.***>
wrote:

@allroundexperts https://github.com/allroundexperts what is your test
results per each case I shared?
—
Reply to this email directly, view it on GitHub
https://github.com/Expensify/App/pull/14713#issuecomment-1414041336, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AHFJUUEKG2KILLN4QJ7QRMTWVPPY3ANCNFSM6AAAAAAUM6BUCQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Seems to work. I wasn’t able to test on Windows though as I’m on mac.

@allroundexperts are you sure you tested against last commit you did? _.get not working. I suggested to use lodashGet. There's no usage of _.get across the app.

@0xmiroslav I can assure you that I tested it pretty thoroughly. I even set a breakpoint to check if _.get was working fine. But now when I tested it again, I see some weird behaviour with _.get. I have attached images as well. When I run it in console, it works fine but upon execution in run time, it gives wrong value. Do you have a clue why this is happening?

Seems to work. I wasn’t able to test on Windows though as I’m on mac.

@allroundexperts are you sure you tested against last commit you did? _.get not working. I suggested to use lodashGet. There's no usage of _.get across the app.

@0xmiroslav I can assure you that I tested it pretty thoroughly. I even set a breakpoint to check if _.get was working fine. But now when I tested it again, I see some weird behaviour with _.get. I have attached images as well. When I run it in console, it works fine but upon execution in run time, it gives wrong value. Do you have a clue why this is happening?

@0xmiroslav I see the problem now. In console, writing _ resolves to lodash. It runs lodash.get when I run it in console. That is why my tests were passing. I'll just use lodash.

Closing, see issue


PolicyRecordingBot - Save Logs

I'm trying to get all logs and save into a file or send to Logic Apps.
Which method should I intercept those logs?
`public void OnNext(LogEvent logEvent)
{
// Do nothing for metrics for now.
if (logEvent.EventType == LogEventType.Metric)
{
return;
}
     // Log only http traces if enabled.
     if (logEvent.EventType != LogEventType.HttpTrace)
     {
        // Unless we have an error/warning to log.
        if (logEvent.Level != TraceLevel.Error && logEvent.Level != TraceLevel.Warning)
        {
           return;
        }
     }

     var logString = this.formatter.Format(logEvent);
     Trace.TraceInformation(logString);
     lock (this.lockLogs)
     {
        this.logs.AddFirst(logString);
        if (this.logs.Count > MaxLogCount)
        {
           this.logs.RemoveLast();
        }
     }
  }`

I tried to get them inside this method (OnNext) but all thing I tried (Trace.TraceInformation, send to Logic Apps, Write a file locally...), nothing worked!

Solved!


Switch to clang-format verison 9 to track Drake master

This change is

And CI passed.  Merging now!

@jamiesnape I think we need to PR the llvm tap change from https://github.com/RobotLocomotion/drake/pull/13323 into drake-external-examples as well?


Add form that allows users to add meeting notes

Proposed changes
Form allows users to add meeting notes.
Checklist

[x] Are the issues being addressed linked to this PR?
[x] Do all commit messages start with the issue number?
[x] Are all code changes sufficiently tested?
[x] Are there screenshots for UI changes?

Just looking at the screenshot:

Why isn't there any margin between the meeting notes and the form?
Why is the form in a card, even though the sidebar doesn't have very much space for the extra padding?
Why doesn't the text input span the full width of its container?

Just looking at the screenshot:

Why isn't there any margin between the meeting notes and the form?
Why is the form in a card, even though the sidebar doesn't have very much space for the extra padding?
Why doesn't the text input span the full width of its container?
Why is there so much space between the text input and the button?

These issues have been addressed by reviewing the product spec and following those style guidelines.


Link in README.md for connector is broken

The link configured on the text "Apps Script" is broken, it returns a 404.
Link to directory of the file with the error
https://github.com/GoogleCloudPlatform/fourkeys/tree/main/connector

Thank you!


Write a blog on Next.js

Write a blog on Next.js

I would Like to contribute to this issue.

I would Like to contribute to this issue. @debrajrout

I would Like to contribute to this issue. @debraj
Follow and give a stare in order to merge

I would Like to contribute to this issue. @debrajrout

Follow and give a stare in order to merge

can you suggest where i write this blog please !

can you suggest where i write this blog please !

Create a readme file and write there

Hey, @debrajrout can you please assign this task to me? Currently, I'm working on Next.js projects so I've better things to contribute.

Hey, @debrajrout can you please assign this task to me? Currently, I'm working on Next.js projects so I've better things to contribute.

Yes you can but you have to maintain the proper folder structure and give a star

Okay got your points.


Ship remove auth steps

Ships #3244
See https://sourcegraph.slack.com/archives/C06K09TB8UR/p1711491797667499?thread_ts=1710459115.678939&cid=C06K09TB8UR for results
⚠ We want to conclude the test tomorrow so let's wait for the final call by @chenkc805 before shipping this.
Test plan
CI

👏🏼

Shipping: https://sourcegraph.slack.com/archives/C06K09TB8UR/p1711645172072679?thread_ts=1710459115.678939&cid=C06K09TB8UR


modal.dispose() does not fully clean up body element mutations

Prerequisites

[X] I have searched for duplicate or closed issues
[X] I have validated any HTML to avoid common problems
[X] I have read the contributing guidelines

Describe the issue
Calling dispose() on a CLOSED modal will fully remove all events and data of the modal. This works as expected.
However, should you attempt to dispose() a modal while its OPENED, there are lingering bits of dom mutation left over. Glancing the dom and source code quickly it looks like this is mainly just a modal-open class on the body, as well as inline styling on the body.
I'm not super confident in my assumption that dispose() should be responsible for this, but given it cleans up its backdrop and focustrap on dispose, which are only ever active when its OPENED, it makes me think this is a bug that it ignores the body element. I believe dispose() should also remove the mutations it performs on the body.
I can submit a fix for this if its agreed to be a bug - let me know
Reduced test cases
https://jsfiddle.net/tnLuvz8d/
What operating system(s) are you seeing the problem on?
Windows, macOS, Android, iOS, Linux
What browser(s) are you seeing the problem on?
Chrome, Safari, Firefox, Microsoft Edge, Opera
What version of Bootstrap are you using?
5.1.3 is used in the demo - haven't checked how far back this is the case

I am having the same issue.

I don't know if I have the exact same issue, but it feels similar. I'm using Vue to toggle the "closable" state of the modal to prevent users from closing the modal while some async operation is ongoing.
I originally tried to recreate the modal every time the state changes (with some workaround to prevent the animation). That caused the modal to not fully close due to some object references gone wrong I assume. The _backdrop._element was no longer on the DOM while an equal object was (but BS had no reference to it).
I fixed my "toggle closable" using this:
this.bsModal._backdrop.isVisible = Boolean(this.closable ? true : 'static')
this.bsModal._config.keyboard = this.keyboard && this.closable

Maybe this will help some of you


Fixed broken links to https://yld.io. Fixed broken yld.io logos.

I just noticed the links to yld.io were broken, so I fixed them. On pages other than home, the yld.io logo was using a broken image uri so I changed to match the homepage.

thanks @xinUmbralis +1


Proposal: API spec for paginated version of getAlertGroups and getAlerts

Background
Without pagination of the APIs, the getAlertGroups and getAlerts can cause a significant performance impact on the server
After discussed in proposal, we get an agreement on create a brand new API to return only alert group information.
This proposal will discuss the API spec of the paginated version of getAlertGroups and getAlerts
getAlertGroupsInfo

URL query parameters:

maxResults Integer. The maximum number of alert groups to return in one getAlertGroupsInfo operation. The range is 1–1000. Optional
nextToken String. The token for the next set of items to return. (You received this token from a previous call.). Optional
filter An array of strings. A list of matchers to filter alert group by. Optional
receiver String.  Optional

Sample response

      groups: [
      {
          "labels": {
              "group": "hello1"
          },
          "receiver": {
              "name": "example-sns"
          },
          "route": "{ROUTE_UUID}"
      },
      {
          "labels": {
              "group": "hello2"
          },
          "receiver": {
              "name": "example-sns"
          },
          "fingerprint": "{group_UUID}"
      }
      ],
        nextToken:"123"

getAlerts

URL query parameters:

maxResults Integer. The maximum number of alerts to return in one getAlerts operation. The range is 1–1000. Optional
nextToken String. The token for the next set of items to return. (You received this token from a previous call.). Optional
active Boolean. If true, the returned list includes active alerts. The default is true. Optional
silenced Boolean. If true, the returned list includes silenced alerts. The default is true. Optional
inhibited Boolean. If true, the returned list includes inhibited alerts. The default is true. Optional
unprocessed  Boolean. If true, the returned list includes unprocessed alerts. The default is true. Optional
filter An array of strings. A list of matchers to filter alerts by. Optional
receiver String.  Optional
groupId String. The group id to filter alerts by. Optional

Sample response

{
  "alerts": [
    {
      "annotations": {
        "summary": "this is a test alert used for demo purposes"
      },
      "endsAt": "2021-10-21T22:07:31.501Z",
      "fingerprint": "375eab7b59892505",
      "receivers": [
        {
          "name": "sns-0"
        }
      ],
      "startsAt": "2021-10-21T22:02:31.501Z",
      "status": {
        "inhibitedBy": [],
        "silencedBy": [],
        "state": "active"
      },
      "updatedAt": "2021-10-21T22:02:31.501Z",
      "labels": {
        "alertname": "test-alert"
      }
    }
  ],
  "nextToken": "123"
}

other consideration
Add UUID for each route
For current data structure, one route can not be unique identified by receiver name.
However in order to search the alerts by alert group we need a fingerprint to identify the referenced alert group. The finger print to unique identify the alerting group should be group labels + route info.
We proposed to generate a unique identifier to each route as it is created
Add group information for each alert in the response of getAlerts API
It will be a better user experience if we can link the alerts back to the alert groups.
We proposed to add the groups info for each alert to the response of getAlerts API

I would also love a spec for paginated version get silences.

Hi, I was wondering if there's an update on when pagination will be implemented for the getAlerts endpoint? Thanks!

Hi, I was wondering if there's an update on when pagination will be implemented for the getAlerts endpoint? Thanks!

This is implemented here: https://github.com/prometheus/alertmanager/pull/3521


Georgian text overlaps

Windows Terminal version
1.19.10573.0
Windows build number
10.0.19045.4239
Other Software
No response
Steps to reproduce
Type text in Georgian
Expected Behavior
Text shouldn't overlap
Actual Behavior
Georgian text overlaps and isn't readable

When it comes to Georgian I have very very bad news...
Unicode's UAX #11: East Asian Width specifies how many columns a given character is supposed to take up, even for characters that aren't used in East Asia.
"ლ" for instance is U+10DA in Unicode and for whatever reason its width is explicitly specified as "Narrow". 🤦
However, the good news is that there's exactly 1 (one) modern font in the entire world which supports monospace Georgian: DejaVu Sans Mono.
On top of that, there are two more up to date clones/forks of that font:

"Menlo" by Apple is a font you can't use because Apple's licensing terms only allow it to be used on their devices. Since I don't want you to violate those terms I urge you to definitely avoid visiting this page: https://github.com/ueaner/fonts
It's recently been replaced by "SF Mono" which doesn't support Georgian.
"Hack" is another clone, but it's open source and is the default font for KDE. You can find it here: https://sourcefoundry.org/hack

Out of all of these, I recommend using Hack since it's the most modern one. If you download Hack, I recommend against using their .exe installer. Just download the .zip instead, extract it, select all 4 files, right click and select to "Install for all users":

Installing fonts for only your own account has been broken for a long time on Windows.
There may be other monospace fonts, but I bet that if it's not a derivative of DejaVu Sans Mono, it won't support Georgian. The only exception is "GNU Unifont" but that's not a font you'll want to use. 😅


ERROR:Invalid UI version in the JSON-RPC response

What version of Gateway-ST are you using (gateway version)?

$ gateway version
Development build

I tried the lasted version and  v1.77.3 , and get the same results.
Thanks a lot!!!
Does this issue reproduce with the latest release?
What operating system and processor architecture are you using (uname -mrsv)?
uname -mrsv Output
$ uname -mrsv
Linux 5.19.0-41-generic #42~22.04.1-Ubuntu SMP PREEMPT_DYNAMIC Tue Apr 18 17:40:00 UTC 2 x86_64

What did you do?
I have followed all steps of Test network · storj/storj Wiki · GitHub 2 but at the time of login through http://localhost:11000/minio/login I got the Error : “Invalid UI version in the JSON-RPC response”.

and I try the follow command and get the same result.
gateway run \
--access "$(storj-sim network env GATEWAY_0_ACCESS)" \
--minio.access-key "$(storj-sim network env GATEWAY_0_ACCESS_KEY)" \
--minio.secret-key "$(storj-sim network env GATEWAY_0_SECRET_KEY)" \
--server.address <IP_ADDRESS>:7777

What did you expect to see?
What did you see instead?

Thanks for the report. While we don't have a fix for this behavior yet, please take a look at #65, where an immediate fix is discussed.

Closing this as we released v1.9.0 containing a fix for this issue.


A. 📺 Universal Browser Connectivity

eta: 2023Q2
Why: A huge part of “the Web” is happening inside the browser. As a universal p2p networking stack, libp2p needs to be able to offer solutions for browser users.
Goal: js-libp2p supports both WebTransport and (libp2p-) WebRTC protocols, enabled by default. This allows connections between browsers and public nodes, and eventually between browsers and non-public nodes and in between browser nodes.

[x] https://github.com/libp2p/js-libp2p/issues/1478
[x] https://github.com/libp2p/js-libp2p/issues/1462
[ ] WebTransport

@achingbrain what's the status on the WebTransport subtask?

This is complete 🎉


Compact approach to output

Hello,
I saw your project on wowhead today and followed it bit, as you noted you are not well with CSS, I wanted to step forward and help you on the html output. You can look at my past from my profile. I`m still learning the RUST at this stage (very basics of it compile - run on web server etc...).
What I`m offering (suggesting) is to make more compact view so it wont take ages to scroll down. Also I can make it responsive so it will be mobile ready.
This is my very first draft (direct HTML edit on your page so bars are just went all the way down due to flex view).

Let me know if this version is fine enough, thanks for the handy web-app.

I think a two column (responsive) view is definitely the way to go. The main stickler is the 'grid' format and making sure the bars are aligned elegantly.
I think that the bars are an important visualization to quickly impart visually the relative values of the options in a way that numbers alone don't convey.
As you've probably seen from looking at the code, things are divided into two columns: The left contains the icons and the names, the right contains the values and the bars. I did it this way because it made it easy to make the left column a consistent width for all items. To make sure the bars line up with them is a bit of a hack: I made sure the bars were the same height as the icons and I inserted lines of whitespace for things like the mats or crafting/trading notes.
Unfortunately out of the box flexbox doesn't lend itself immediately to this kind of grid approach, although I've seen some approaches that might work.
The easiest thing might be to enclose each section (i.e. Blood and Sargerite) in its own div and arrange those in a flex container that tries to place them in a row.


ci: migrate to sage

Codecov Report

Merging #47 (4fb3f87) into master (fc4f0cc) will increase coverage by 0.05%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master      #47      +/-   ##
==========================================
+ Coverage   13.00%   13.06%   +0.05%
==========================================
  Files          13       13
  Lines         715      712       -3
==========================================
  Hits           93       93
+ Misses        607      604       -3
  Partials       15       15

Impacted Files
Coverage Δ

routingv7/calculatematrix.go
0.00% <0.00%> (ø)

routingv7/calculateroute.go
0.00% <0.00%> (ø)

routingv7/client.go
0.00% <ø> (ø)

routingv7/types.go
0.00% <ø> (ø)

routingv8/client.go
50.70% <0.00%> (+0.70%)
:arrow_up:


No ‘version’ key in the manifest file for custom integration ‘midea_ac’.

No ‘version’ key in the manifest file for custom integration ‘midea_ac’. As of Home Assistant 2021.6, this integration will no longer be loaded. Please report this to the maintainer of ‘midea_ac’
I believe this needs an update to work with 2021.6
I have an unavailable status showing in HA.

Sorry, I see the commit that fixed this. Closed.


feat: add reveal from 3rd party guide

Description

Adds guide to reveal from a 3rd party.

Testing required outside of automated testing?

[x] Not Applicable

Screenshots (if appropriate):

[x] Not Applicable

Rollback / Rollforward Procedure

[x] Roll Forward
[ ] Roll Back

Reviewer Checklist

[ ] Description of Change
[ ] Description of outside testing if applicable.
[ ] Description of Roll Forward / Backward Procedure
[ ] Documentation updated for Change

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


Facing issue with date field.

Hi i am facing same issue as below mentioned in stackoverflow. anyone have suggestions for the same??
http://stackoverflow.com/questions/19819870/date-query-with-isodate-in-mongodb-doesnt-seem-to-work

I'm dealing with this issue as well.  Is this a PHP issue?  When passing a Carbon date into a where query as in where('starts_at', '<=', Carbon::now()), the resulting Mongo query is {"starts_at":{"$lte":{"sec":1446007699,"usec":78000}}}.  It should be {"starts_at": {"$lte": ISODate("2015-10-28T04:57:19.000Z")}}`.  The Carbon date is clearly being converted into a MongoDate (in PHP), but the MongoDate isn't being converted into proper Mongo query language.
I thought this would work: where('started_at', '<=', \DB::raw('IsoDate(' . Carbon\Carbon::now()->toIso8601String() . ')')), but no such luck, it gives the dreaded "zero-length keys are not allowed" error.  Not sure where to go from here, short of doing the query manually.


Can I use SentencepieceTokenizer in C#?

hi!
I have an LLM in Onnx format and a sentencepiece.model, and I used HuggingFace and SententPiece together in Python. Now I plan to do inference in C# + OnnxRuntime, but I haven't found a suitable version of the SententPiece library in C#. I saw that there is a SentencepieceTokenizer here. Can I use SentencepieceTokenizer in C#?
My files were downloaded from here: https://huggingface.co/K024/ChatGLM-6b-onnx-u8s8/tree/main/chatglm-6b-int8-onnx-merged. Thank you."

Yes, the Nuget package could be found here: https://www.nuget.org/packages/Microsoft.ML.OnnxRuntime.Extensions/0.8.0

I only saw the C# demo code for registering the extension.
In Python, I need to use the tokenizer to get the Ids of the user input text first when using the LLM model for inference, and then do the subsequent processing.
But I didn’t find a class that wraps this tokenizer as a C# version in this extension library, did I misunderstand it?
Can you give some examples?
For example: I defined it like this in python:
from sentencepiece import SentencePieceProcessor  sp_model = SentencePieceProcessor(model_file=model_path)  ids = sp_model.encode(s)

@sayanshaw24 , can you add SPM tokenizer into our C# example?

Perhaps this issue can be closed? I was looking for something similar on the decoder end and was able to develop what I think @tylike wanted.
I should note beforehand that this was done using v0.8.0; the function build_my_graph was renamed at some point to build_graph.
Here's a working solution I came up with:
##building the model
from onnxruntime_extensions._ortapi2 import make_onnx_model
from onnxruntime_extensions._cuops import SingleOpGraph
import onnx

kwargs={'model':open(path/to/model, 'rb').read()}
model = make_onnx_model(graph)
onnx.save_model(model,/outputpath/model.onnx)

##inference
import onnxruntime as _ort
from onnxruntime_extensions import get_library_path as _lib_path
import numpy as np

so = _ort.SessionOptions()
so.register_custom_ops_library(_lib_path())
sess = _ort.InferenceSession(/outputpath/model.onnx, so)

alpha = 0
nbest_size = 0
flags = 0

inp_dict = {'inputs': np.array(['your text here']),
            'nbest_size': np.array([nbest_size],dtype=np.int64),
            'alpha':np.array([alpha],np.float32),
            'add_bos':np.array([flags & 1], dtype=np.bool_),
            'add_eos':np.array([flags & 2], dtype=np.bool_),
            'reverse':np.array([flags & 4], dtype=np.bool_)}

outs = sess.run(None,input_feed=inp_dict)
token_array = outs[0]

While this isn't C#, hopefully it illustrates how to perform inference using the ONNX tokenizer; it should be relatively straightforward to implement from the python code.
Just make sure to load the extensions library when performing inference in C#:
SessionOptions options = new SessionOptions();
options.RegisterOrtExtensions();
session = new InferenceSession(model, options);
``

This test helped me quite a bit: https://github.com/microsoft/onnxruntime-extensions/blob/696e7502f16de207bad1f13d236af9455c6e26d6/test/test_sentencepiece_ops.py#L463

@wenbingl
Where are the folder for c# Examples?
I see Java folder under the root but no trace of CSharp folder?


getx upload file  is very slowly.  like this bug #940.

ATTENTION: DO NOT USE THIS FIELD TO ASK SUPPORT QUESTIONS. USE THE PLATFORM CHANNELS FOR THIS.
Fill in the template. Issues that do not respect the model will be closed.
Describe the bug
getx upload file  is very slowly.  like this bug #940.
when i use dio. it's fast.
**Reproduction code
example:
To Reproduce
Screenshots
Flutter Version:
Flutter 2.2.2 • channel stable
Dart 2.13.3
Getx Version:
get: ^4.1.4
Describe on which device you found the bug:
android 11
Minimal reproduce code
final contentType = lookupMimeType(path);
final formdata = FormData({"file":MultipartFile(path,filename: path.split('/').last,contentType: contentType!=null?contentType:"application/octet-stream"),"folder":folder} );
Result result = await cRequest('/admin/sys-file/upload/app',method: 'POST',data: formdata,uploadProgress: (val) async {
if(uploadProgress == null) EasyLoading.showProgress(val/100,status: '${val.toStringAsFixed(0)}%');
else uploadProgress(val);
});

You have to use multithreading for async/await post methods that have large multipart files attached to them.
This is to isolate the UI memory from post request process.
See the official video:
https://www.youtube.com/watch?v=qrFTt1NZed8

I meet same issue, upload a image file 1.9 MB, UI freezing, and then upload failed.

Hey there, it sounds like your UI is freezing while using the post method for uploading files. Just a moment ago, I was facing the same issue. Then I removed the "uploadProgress" parameter from the post method. And now the UI is not freezing anymore. uploadProgress param has a glitch.

I appreciate your helpful answer. Is there a way to use the uploadProgress parameter without any issues?

您必须对附加有大型多部分文件的 async/await post 方法使用多线程。
这是为了将 UI 内存与 post 请求过程隔离。
观看官方视频： https://www.youtube.com/watch?v =qrFTt1NZed8

This operation should be encapsulated inside getConnect

您必须对附加有大型多部分文件的 async/await post 方法使用多线程。
这是为了将 UI 内存与发布请求过程隔离。
观看官方视频： https://www.youtube.com/watch?v =qrFTt1NZed8

The microtask queue is blocked

您必须对附加有大型多部分文件的 async/await post 方法使用多线程。
这是为了将 UI 内存与 post 请求过程隔离。
观看官方视频：https://www.youtube.com/watch?v =qrFTt1NZed8

这个操作应该封装在getConnect里面

微任务队列被阻塞

I meet same issue, upload a image file 1.9 MB, UI freezing, and then upload failed.

Hey there, it sounds like your UI is freezing while using the post method for uploading files. Just a moment ago, I was facing the same issue. Then I removed the "uploadProgress" parameter from the post method. And now the UI is not freezing anymore. uploadProgress param has a glitch.

use dio 😭

Just used Dio and it works async


Fix assign shard logic

Search         // TODO: Temporary hack to assign shards to pipelines.

closing as duplicate of #4184


Upgrade to MixItUp 3: Divs with content not showing

Here's a JSFiddle with the issue: https://jsfiddle.net/ashraj98/af5L8tuL/2/
I got these same divs and CSS styles to work with the old version of MixItUp, but I can't seem to get this to work on updating to MixItUp 3.

Hi @ashraj98,
With v3 you no longer need to set display: none on your targets. Here's you're fiddle updated and working:
https://jsfiddle.net/af5L8tuL/3/


Resource

Hi, I am using Resource Owner type, I can authenticate but I it's not returning all the user claims for
Minimal working example
Created client like this:
        var resourceOwnerClient = new Client
        {
            ClientId = "ro.client",
            AllowedGrantTypes = GrantTypes.ResourceOwnerPassword,

            ClientSecrets =
            {
                new Secret { Value = "SuperSecretPassword".Sha256()}
            },
            AllowedScopes = { "weatherapi.read", "openid", "profile" }
        };

Startup.cs:
                .AddAspNetIdentity<IdentityUser>()
                .AddConfigurationStore(options =>
                {
                    options.ConfigureDbContext = b => b.UseSqlServer(Configuration.GetConnectionString("IdentityServerConfiguration")
                        //,sql => sql.MigrationsAssembly(migrationsAssembly)
                        );
                })
                // this adds the operational data from DB (codes, tokens, consents)
                .AddOperationalStore(options =>
                {
                    options.ConfigureDbContext = b => b.UseSqlServer(Configuration.GetConnectionString("IdentityServerPersistedGrant")
                        //,sql => sql.MigrationsAssembly(migrationsAssembly)
                    );

                    // this enables automatic token cleanup. this is optional.
                    options.EnableTokenCleanup = true;
                })
                .AddResourceOwnerValidator<ResourceOwnerPasswordValidator<IdentityUser>>()
                .AddDeveloperSigningCredential();

How come it is not returning all the claims associate with profile Identity Resource?

User definitely has values for some of them:

I access it with postman with these parameters:

This is the decoded JWT:

thank you!

It's the job of your profile service to collect and return the claims. typically based on the requested scopes. Did you implement that?

Thank you for the reply.  Yes I implemented a profile service but I pretty much used same implementation as one that comes with IS4.  Anyway, I added then to the ApiResourceClaims table and it is sending it back now...


How to make all this work with flow ?

It would be nice if this generator would support flow for static type checking.

@rogergl: I dont think this is a feature for the core generator. We would like to keep it as simple and slim as possible.
It should be easily implementable in your setup. Have a look into https://www.npmjs.com/package/flowcheck-loader.


feat(TDI-43395):update dom4j to 2.1.1

What is the current behavior? (You can also link to an open issue here)
for jira: https://jira.talendforge.org/browse/TDI-43395
What is the new behavior?
update dom4j to 2.1.1
Please check if the PR fulfills these requirements

[x] The commit message follows Talend standard
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features) ?
[ ] The code coverage on new code >75%
[ ] The new code does not introduce new technical issues (sonar / eslint)

What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build / CI related changes
[ ] Other... Please describe:

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Pull request test result: failure :x:

Pull request test result: failure :x:

Pull request test result: failure :x:

Pull request test result: success :star:
Sonar results


Use interface for cache instead of using FS directly

@xepozz this issue is resolved?

I don't think so because we have our own implementation for file cache.
It looks superfluous.

I want to abandon FastRouteCache in favor of specialized packages that implement PSR interface.
If anyone wants to use file cache he can do composer req yiisoft/cache-file and add some lines to config.

I want to abandon FastRouteCache in favor of specialized packages that implement PSR interface.
If anyone wants to use file cache he can do composer req yiisoft/cache-file and add some lines to config.

It will be removed. This was a temoprary solution.


support natsio persistence

Describe the Bug
The cluster always in progress when terminating cluster
For UI issues please also add a screenshot that shows the issue.
Versions Used
Kubernetes: v1.1.0
Kubeclipper: v1.1.0
Environment
How many nodes and their hardware configuration:
CentOS 7.7.1908
masters: 4cpu/8g
How To Reproduce
Steps to reproduce the behavior:
Go to 'Cluster Manager - Cluster'
Select a cluster and click on 'Delete'
Jump to 'Cluster Manager - Cluster'
The Cluster is always in progress
** Screenshot**

/assgin @qinyer

/assign @qinyer

/kind need-verify
I think it caused by poor network environment
@qinyer @sacredartr

/milestone v1.1.0

maybe we can support natsio persistence to avoid lost message between server and agent
/remove-kind need-verify
/milestone v1.2.0
/kind feature

/retitle support natsio persistence

ref:

https://docs.nats.io/reference/faq
https://docs.nats.io/nats-concepts/jetstream

support external natsio now.
/close


Java Support Yet?

I saw a issue below explaining how there isn't support for java yet with this extension. Has support for java since then been added, or is there still not support for Java?

No there is a pull request open for it but its stalled out at the moment https://github.com/daveleroy/sublime_debugger/pull/106

Thank you. Can you leave a comment or tag me if Java support is ever added?


Add Pills Component and Styling

Please can you have a look over this thanks - can't tag until contributor

If all ok now could this get merged please cheers @Ehesp

Want that max-props-per-line rule removed from linting prior to merging.

Just noticed are these acting like tabs? If so I should have been more precise in the description that they are just intended to be used for things like tags like the pills on the Bootstrap framework.

@rbrtsmith - Have removed the multiline prop rule - do you want to raise another story for any new changes for the pills or shall I reacreate another set not tied to tabs? Awaiting this to get merged but if it's not what you want then will leave. Cheers

I would suggest removing the tab based stuff in this branch before merging
then we don't end up with possible redundant code lingering around.
Thanks!
On 23 Aug 2017 10:54 am, "Kay Wood"<EMAIL_ADDRESS>wrote:

@rbrtsmith https://github.com/rbrtsmith - Have removed the multiline
prop rule - do you want to raise another story for any new changes for the
pills or shall I reacreate another set not tied to tabs? Awaiting this to
get merged but if it's not what you want then will leave. Cheers
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/rbrtsmith/nebula-components/pull/17#issuecomment-324280764,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEwE8fb3wtXMequCGbqdul6eoa59um2Sks5sa_bMgaJpZM4O9eC0
.


uptime

Mac OSX (Darwin) enum_os() uptime is not working correctly. It keeps coming up as '0'.

issue resolved. using subprocess.check_output() instead of subprocess.call()


[metricbeat] Add priorityClassName config

This PR add a priorityClassName config to metricbeat helm chart.

[x] Chart version not bumped (the versions are all bumped and released at the same time)
[x] README.md updated with any new values or changes
[x] Updated template tests in ${CHART}/tests/*.py
[x] Updated integration tests in ${CHART}/examples/*/test/goss.yaml

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

Hi @gadiener, we have found your signature in our records, but it seems like you have signed with a different e-mail than the one used in your Git commit. Can you please add both of these e-mails into your Github profile (they can be hidden), so we can match your e-mails to your Github profile?

nice!


Tolerate timestamps w/ truncated nanos.

Closes #1783.

I don't have the bandwidth right now to follow this properly, but have you guys looked at the proto3 JSON mapping spec for Timestamp and the Timestamp class in google.protobuf. Could we use that library's support for JSON serialization and deserialization of the well-known type Timestamp?

Possibly, the phrase

Uses RFC 3339, where generated output will always be Z-normalized and uses 3, 6 or 9 fractional digits.

may actually shed some light on what we've seen empirically.

@dhermes Seems like you didn't find a counter-example to the "truncate/floor to micros" bit.

Yeah LGTM


chore: import kintone/dts-gen

DO NOT SQUASH MERGE, USE MERGE COMMIT

Why

We'd like to transfer kintone/dts-gen into kintone/js-sdk.
What

[x] package.json
[x] README in root
[x] README for dts-gen
[x] build settings

How to test

Checklist

[x] Read CONTRIBUTING.md
[ ] Updated documentation if it is required.
[ ] Added tests if it is required.
[ ] Passed yarn lint and yarn test on the root directory.

Thanks you for your PR!
I confirmed that this PR can generate a record field definition.


Stale bot is not able to close pull requests

We installed stale bot to multiple repositories, and it can successfully put comment on the pull requests

However, even after the specified daysUntilClose after the stale labelled, the issues remains open and not closed by stale bot, can someone help on this issue? or is there any logs we can look and try to debug the issue?
our config is:
# Number of days of inactivity before an issue becomes stale
daysUntilStale: 14
# Number of days of inactivity before a stale issue is closed
daysUntilClose: 7
# Issues with these labels will never be considered stale
exemptLabels:
  - pinned
  - security
# Label to use when marking an issue as stale
staleLabel: stale
# Comment to post when marking an issue as stale. Set to `false` to disable
markComment: >
  This issue has been automatically marked as stale because it has not had
  recent activity. It will be closed if no further activity occurs. Thank you
  for your contributions. Please see `.github/stale.yml` for stale settings.
# Comment to post when closing a stale issue. Set to `false` to disable
closeComment: >
  This issue has been automatically closed due to no further activity after
  the previous `stale` comment. Please see `.github/stale.yml` for stale settings.

Thanks

yeah, I am actually facing the similar issue for my org (it looks like it stops working recently)

I'm having the same issue with a similar config to above.


Dynamic Library Support for Windows

What language and solver does this apply to?
C++, any solver
Describe the problem you are trying to solve.
I have an executable built in C++ that is distributed with a dynamic library. During patching of the executable, the dynamic library is replaced with a new version. We support Linux, Mac and Windows. For Unix, the dynamic library build of or-tools works great. Is there going to be support for a Windows dynamic library build?
Describe the solution you'd like
Support for dynamic library build for Windows
Describe alternatives you've considered
Adding or-tools as a shared library to the dynamic library we distribute

really not in the plan. ASAIK, It means tagging all methods with the dllexport flags.

I would having a separate library that contains a thin solving API and OR-Tools beneath.
You can make that a DLL.

I'm trying to enable linking of ortools as a static library into a separate DLL. The problem I'm running into is for Debug vs Release builds for Windows Visual Studio. Using find_package to retrieve ortools::ortools doesn't seem to be able to handle Debug vs Release.
When I build with cmake --build build_release --config Release --target install or cmake --build build_release --config Debug --target install, the targets have the same name and end up overwriting each other. So then I tried to have different install directories for the Debug and Release build. However, I don't know how to use find_package to direct to these different install directories for Visual Studio's Debug vs Release configs.
Any suggestions here?
Also, is there perhaps any update on adding dynamic library support?