Derive SerializeAs/DeserializeAs for remote impls?

The examples for using serde_as with remote impls look like a major ergonomic improvement, but avoiding the SerializeAs/DeserializeAs boilerplate would help even more.

Hi, this sounds like a reasonable extension. Could you please explain how you imagine it should work from the user perspective, and maybe also to what code it should expand to? A tiny code snippet of the macro invocation would be great. This would help me figure out how it should work. A set of macro_rules macros is likely the easiest. It needs to handle cases where you want either SerializeAs or DeserializeAs, but not both. That can be done with additional macros.

Taking an example from the docs as a starting point:
// Pretend that this is somebody else's crate, not a module.
mod other_crate {
    // Neither Serde nor the other crate provides Serialize and Deserialize
    // impls for this struct.
    pub struct Duration {
        pub secs: i64,
        pub nanos: i32,
    }
}

////////////////////////////////////////////////////////////////////////////////

use other_crate::Duration;

// Serde calls this the definition of the remote type. It is just a copy of the
// remote data structure. The `remote` attribute gives the path to the actual
// type we intend to derive code for.
#[derive(serde::Serialize, serde::Deserialize)]
#[serde(remote = "Duration")]
struct DurationDef {
    secs: i64,
    nanos: i32,
}

impl SerializeAs<Duration> for DurationDef {
    fn serialize_as<S>(value: &Duration, serializer: S) -> Result<S::Ok, S::Error>
    where
        S: serde::Serializer,
    {
        DurationDef::serialize(value, serializer)
    }
}

#[serde_as]
#[derive(serde::Serialize)]
struct Data {
    #[serde_as(as = "Vec<DurationDef>")]
    vec: Vec<Duration>,
}

It feels like it should be possible to just have:
use other_crate::Duration;

// Serde calls this the definition of the remote type. It is just a copy of the
// remote data structure. The `remote` attribute gives the path to the actual
// type we intend to derive code for.
#[derive(serde::Serialize, serde::Deserialize, SerializeAs)]
#[serde(remote = "Duration")]
struct DurationDef {
    secs: i64,
    nanos: i32,
}

#[serde_as]
#[derive(serde::Serialize)]
struct Data {
    #[serde_as(as = "Vec<DurationDef>")]
    vec: Vec<Duration>,
}

The type to serialize as could just be taken from the #[serde(remote)] attribute, but it might make sense to have a separate attribute for more niche usecases where the serialize/deserialize functions don't come from serde's derives.

Thanks for the explanation. A derive macro seems easy and flexible to use. If you only need one of the two traits, you can do that.

Initially, I thought of something like this. It has the downside of duplicating the type names.
serde_with::serde_as_impls!(DurationDef Duration);
// i.e., localtype remotetype

But it could be extended to support the module form, which you can use with serde(with = "..."):
mod bytes_as_hex {
    fn serialize() {}
    fn deserialize() {}
}

struct BytesAsHex;
serde_with::serde_as_impls!(BytesAsHex Vec<u8> bytes_as_hex);
// i.e., localtype remotetype convertion_module

Another downside is that it now becomes awkward to only have one of the two traits, likely requiring serde_as_impls_de and serde_as_impls_ser variants to solve.

Since a local type is always necessary for serde_as, maybe the module support can also be added to the derive macro.
mod bytes_as_hex {
    fn serialize() {}
    fn deserialize() {}
}

#[derive(SerializeAs)]
#[serialize_as(type = Vec<u8>, module = bytes_as_hex)]
struct BytesAsHex;

I tried to implement the derives but failed to find a solution to find the correct trait bounds. When using the remote derive, the serialize and deserialize functions are inherent functions on the local type, but they will have further constraints for generics. Unfortunately, serde has many rules how to build the trait bounds, so I feel reimplementing them is too much for this crate. A reimplementation might also drift, so I don't think this is a good solution. For example, you have bound on the container or field and various with attributes which need to be understood.
Since the two functions are inherent to the type, they cannot be used in  where-bounds. This thus prevents me from directly relying on them for the heavy lifting.
I don't see it as helpful to only try to support types without generics. They might be a little bit easier, but a half-working implementation which just stops working after doing reasonable changes to your type feels worse than no derive. But given that bound can be applied to the container, this might not even solve the problem in the first place.
For now, I am going to close this issue since I see no way to properly implement it. Maybe part of the serde derives are better reusable in the future, then this might be solvable.


feat: revm crate diff

Current dependencies on/for this PR:

master

PR #8   👈

This comment was auto-generated by Graphite.


Hello,
Please use method ‘BulkInsert’
https://github.com/ilyabreev/ClickHouse.Net/blob/755d87ea65fa294033b9208e33633a21125cd519/src/ClickHouse.Net/ClickHouseDatabase.cs#L266
чт, 10 окт. 2019 г. в 11:56, Hacker<EMAIL_ADDRESS>
sir,how to bulk insert a list?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ilyabreev/ClickHouse.Net/issues/4?email_source=notifications&email_token=AAH7HF2NCH54VIJBAEAKTFLQN3U3ZA5CNFSM4I7J4D32YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HQ3WAVQ,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH7HFZMMMUKXABSITRT6VLQN3U3ZANCNFSM4I7J4D3Q
.

--
Отправлено с iPad


support instagram scraping with cookie in REST API

needs URL design for accepting cookie as a query param.
inspired by @aaronpk: https://chat.indieweb.org/dev/2017-11-29#t1512020937736000

done!


Added hover on instagram and linkedin

I recently implemented a hover effect on Instagram and Linked in for my open-source project. This enhancement, documented under issue number #203 , allows users to experience an interactive and dynamic visual response when hovering over certain element.

Hello @mohdshahzil brother someone has already added hower effect pull request and i merged it so sorry , you can make other pull request than this .


Codemods with dependencies are not working

Seems like dependencies are no longer being resolved
Examples command: npx @codeshift/cli --packages<EMAIL_ADDRESS>../foo/bar/src
Output:

Cause:
This is likely due to the bundling, or with the resolution of dependencies via live-package-manager

Fixed


Add scripts folder and add scripts  to index.html

I add my code from Json Place Holder to create a table to show output data from API

Good job


[Bug]: Install Error : Trying to access array offset on value of type null

What happened?
I tried several install of TastyIgniter following the install instruction (https://tastyigniter.com/docs/installation), all goes well up to the wizard install. However, after choosing a simple install or a full install (with templates), the process halts with the following error: Trying to access array offset on value of type null
Restating the install process does not change anything.
What did you expect to happen?
Complete the install process successfully.
Version
Other (please detail)
What browser are you seeing the problem on?
Chrome
Relevant log output
[2023/03/31 11:14:07] => .============================ POST REQUEST ==========================.
[2023/03/31 11:14:07] => Postback payload: Array
(
    [disableLog] => 1
)

[2023/03/31 11:14:07] => Foundation setup: finishInstall
[2023/03/31 12:14:07] => Handler error (onInstall): Trying to access array offset on value of type null
[2023/03/31 12:14:07] => Trace log:
#0 /var/www/tasty/setup/classes/SetupController.php(585): Illuminate\Foundation\Bootstrap\HandleExceptions->handleError()
#1 /var/www/tasty/setup/classes/SetupController.php(470): SetupController->setSeederProperties()
#2 /var/www/tasty/setup/classes/SetupController.php(253): SetupController->completeSetup()
#3 /var/www/tasty/setup/classes/SetupController.php(281): SetupController->onInstall()
#4 /var/www/tasty/setup/classes/SetupController.php(46): SetupController->execPostHandler()
#5 /var/www/tasty/setup/bootstrap.php(62): SetupController->__construct()
#6 /var/www/tasty/setup.php(1): include_once('...')
#7 {main}

This issue will be closed and archived in 3 days, as there has been no activity in the last 60 days. If this issue is still relevant or you would like to see it actioned, please respond and we will re-open this issue. If this issue is critical to your business, consider the Dedicated Support Service where a Service Level Agreement is offered.


Added SAP VM size reco

@alkohli

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

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

@MicrosoftDocs/public-repo-pr-review-team
If this PR needs approval, or if the commits need to be pulled into the private repo, contact @dansisson. Alpa Kohli has approved this arrangement. Thank you.

@LouisBerner - checking with our team.
Meanwhile @alkohli - can you confirm?
@dansisson - can you review?
#assign: @dansisson

@LouisBerner - @dansisson @alkohli - if you haven't yet done so, can you send email to<EMAIL_ADDRESS>to explain the arrangement? thanks


ENH Avoid copying tokens in tokenizers in Python

Currently, tokenizers return Vec<String>  where tokens are be slices of the input string. Moving to Vec<&str> would remove one memory copy and is likely to help with run time.
This should be possible with PyO3 0.7.0 (not yet released) that will allow using lifetime specifiers in pymethods.

Closing as resolved some time ago.


Build(deps-dev): Bump @storybook/addon-actions from 6.5.12 to 6.5.13 (#4270)

Pull request opened by the merge tool on behalf of #4270

Merged in d7518ccf50e1e8f4da2a5ac6c8b243b16995f035


Adding finally the functional widget

We have widget now! Also, fixed gradlew permissions!

What do you mean with gradlew permissions?

gradlew couldn't be executed


add: auth.go test

[x] Added test for GetValidPasswords in src/internal/pkg/auth/auth.go

Hello! Thank you for the feedback. Fixed according to the review notes.


Fixed a ValueError thrown when using widget apps

Fixes #29

Coverage remained the same at 100.0% when pulling 18253b9c7071c6075aedef2b76255a2c8e71acee on unicode-fix into a33c595f9cf5a492eaad703fb715181b78b0e763 on master.


ui: current sound being viewed should be highlighted in packs SoundList

Expectation
If sound x is the current sound and sound x is in the pack SoundList, it should be accented
Example

We can also have it so that if the current sound is in the similar <SoundList /> component, we can highlight it there, as well.


Remove promise shim (+update docs)

closes #28
@meaku code you please to a tiny review?

tests failing because there is no Promise in node 0.10

We should set a min Node Version and update the docs. It's fine for me then. Can you take care of it @flootr?

 The docs are updated. Sorry... did't see it. I'll take care of the engine.


Integrating an online compiler to run the code in c , c ++ , java and various languages

As I am selected as a mentee in Gssoc'21 so please assign this task to me.

after a complete walkthrough of the website and source code, I found that there is an error in the compilation of code so  I would love to Integrate the online compiler to run the code, so please assign this issue to me.

Hi @rohitsigar, First of all, thank you so much for taking out the time to create this issue 🚀
I think you have not set up the environment variables for the project, that why it's showing an error.
It is working on the deployed version - https://syntaxmeets.vercel.app/
You can set up your online judge API key
Create a new .env file and add the following variables
REACT_APP_SYNTAXMEETS_BACKEND_API = 'http://localhost:4000/'
REACT_APP_ONLINE_JUDGE_API = YOUR API KEY

Happy Contributing! 👍


Initial concept of adding the studio adoption requirement to the lifecycle

Signed-off-by: John Mertic<EMAIL_ADDRESS>

@kdt3rd - would it make sense to review at the next TAC meeting?

Closing as I've merged this into #417


cni: alternative to repair controller

prototype of https://github.com/istio/istio/issues/40303

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

/test all

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
c0ab5de73e86e04c97fa96be166b2b2910fc3d31
link
true
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
c0ab5de73e86e04c97fa96be166b2b2910fc3d31
link
true
/test release-notes_istio

unit-tests-arm64_istio
c0ab5de73e86e04c97fa96be166b2b2910fc3d31
link
true
/test unit-tests-arm64_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test all

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
c0ab5de73e86e04c97fa96be166b2b2910fc3d31
link
true
/test unit-tests-arm64_istio

release-notes_istio
139add850c1d796cb00a111549d9475f0aa80f76
link
true
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
139add850c1d796cb00a111549d9475f0aa80f76
link
true
/test release-notes_istio

unit-tests-arm64_istio
139add850c1d796cb00a111549d9475f0aa80f76
link
true
/test unit-tests-arm64_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
139add850c1d796cb00a111549d9475f0aa80f76
link
true
/test release-notes_istio

unit-tests-arm64_istio
139add850c1d796cb00a111549d9475f0aa80f76
link
true
/test unit-tests-arm64_istio

lint_istio
139add850c1d796cb00a111549d9475f0aa80f76
link
true
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
139add850c1d796cb00a111549d9475f0aa80f76
link
true
/test release-notes_istio

unit-tests-arm64_istio
139add850c1d796cb00a111549d9475f0aa80f76
link
true
/test unit-tests-arm64_istio

lint_istio
139add850c1d796cb00a111549d9475f0aa80f76
link
true
/test lint_istio

gencheck_istio
139add850c1d796cb00a111549d9475f0aa80f76
link
true
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
139add850c1d796cb00a111549d9475f0aa80f76
link
true
/test release-notes_istio

unit-tests-arm64_istio
139add850c1d796cb00a111549d9475f0aa80f76
link
true
/test unit-tests-arm64_istio

lint_istio
139add850c1d796cb00a111549d9475f0aa80f76
link
true
/test lint_istio

gencheck_istio
139add850c1d796cb00a111549d9475f0aa80f76
link
true
/test gencheck_istio

unit-tests_istio
139add850c1d796cb00a111549d9475f0aa80f76
link
true
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
139add850c1d796cb00a111549d9475f0aa80f76
link
true
/test release-notes_istio

unit-tests-arm64_istio
139add850c1d796cb00a111549d9475f0aa80f76
link
true
/test unit-tests-arm64_istio

lint_istio
139add850c1d796cb00a111549d9475f0aa80f76
link
true
/test lint_istio

gencheck_istio
139add850c1d796cb00a111549d9475f0aa80f76
link
true
/test gencheck_istio

unit-tests_istio
139add850c1d796cb00a111549d9475f0aa80f76
link
true
/test unit-tests_istio

integ-cni_istio
139add850c1d796cb00a111549d9475f0aa80f76
link
true
/test integ-cni_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test ?

@howardjohn: The following commands are available to trigger required jobs:

/test analyze-tests_istio
/test gencheck_istio
/test integ-basic-arm64_istio
/test integ-cni_istio
/test integ-distroless_istio
/test integ-helm_istio
/test integ-ipv6_istio
/test integ-operator-controller_istio
/test integ-pilot-istiodremote-mc_istio
/test integ-pilot-istiodremote_istio
/test integ-pilot-multicluster_istio
/test integ-pilot_istio
/test integ-security-istiodremote_istio
/test integ-security-multicluster_istio
/test integ-security_istio
/test integ-telemetry-istiodremote_istio
/test integ-telemetry-mc_istio
/test integ-telemetry_istio
/test lint_istio
/test release-notes_istio
/test release-test_istio
/test unit-tests-arm64_istio
/test unit-tests_istio

The following commands are available to trigger optional jobs:

/test benchmark_istio
/test integ-assertion_istio

Use /test all to run the following jobs that were automatically triggered:

analyze-tests_istio
gencheck_istio
integ-basic-arm64_istio
integ-cni_istio
integ-distroless_istio
integ-helm_istio
integ-ipv6_istio
integ-operator-controller_istio
integ-pilot-istiodremote-mc_istio
integ-pilot-istiodremote_istio
integ-pilot-multicluster_istio
integ-pilot_istio
integ-security-istiodremote_istio
integ-security-multicluster_istio
integ-security_istio
integ-telemetry-istiodremote_istio
integ-telemetry-mc_istio
integ-telemetry_istio
lint_istio
release-notes_istio
release-test_istio
unit-tests-arm64_istio
unit-tests_istio

In response to this:

/test ?

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/test integ-cni_istio

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
139add850c1d796cb00a111549d9475f0aa80f76
link
true
/test release-notes_istio

unit-tests-arm64_istio
139add850c1d796cb00a111549d9475f0aa80f76
link
true
/test unit-tests-arm64_istio

lint_istio
139add850c1d796cb00a111549d9475f0aa80f76
link
true
/test lint_istio

gencheck_istio
139add850c1d796cb00a111549d9475f0aa80f76
link
true
/test gencheck_istio

unit-tests_istio
139add850c1d796cb00a111549d9475f0aa80f76
link
true
/test unit-tests_istio

integ-cni_istio
139add850c1d796cb00a111549d9475f0aa80f76
link
true
/test integ-cni_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


HADOOP-9613. [JDK8] Update jersey version to latest 1.x release.

This PR is for a review.

Would you close this pull request?


The existing link pointed to wrong Vnet article

@macolso : Thanks for your contribution! The author(s) have been notified to review your proposed change.


cv2/cv2.abi3.so, 2): Library not loaded: /usr/local/opt/libunistring/lib/libunistring.2.dylib   Referenced from: /usr/local/opt/gnutls/lib/libgnutls.30.dylib

Expected behaviour
I am using yolov8 pip install ultralytics then running
Actual behaviour
Traceback (most recent call last):   File "/Users/cs/project/yolov8-cs/1.py", line 1, in <module>     import cv2   File "/Users/cs/.pyenv/versions/yolo_venv/lib/python3.10/site-packages/cv2/__init__.py", line 181, in <module>     bootstrap()   File "/Users/cs/.pyenv/versions/yolo_venv/lib/python3.10/site-packages/cv2/__init__.py", line 153, in bootstrap     native_module = importlib.import_module("cv2")   File "/Users/cs/.pyenv/versions/3.10.4/lib/python3.10/importlib/__init__.py", line 126, in import_module     return _bootstrap._gcd_import(name[level:], package, level) ImportError: dlopen(/Users/cs/.pyenv/versions/3.10.4/envs/yolo_venv/lib/python3.10/site-packages/cv2/cv2.abi3.so, 2): Library not loaded: /usr/local/opt/libunistring/lib/libunistring.2.dylib   Referenced from: /usr/local/opt/gnutls/lib/libgnutls.30.dylib   Reason: image not found
Steps to reproduce

example code：import cv2
operating system  macos 10.15.7 (19H1615)
architecture (x64)
opencv-python version  ：Probably up to date, as it uses pip install ultralytics directly

Issue submission checklist

[ ] This is not a generic OpenCV usage question (looking for help for coding, other usage questions, homework etc.)

[ ] I have read the README of this repository and understand that this repository provides only an automated build toolchain for OpenCV Python packages (there is no actual OpenCV code here)

[ ] The issue is related to the build scripts in this repository, to the pre-built binaries or is a feature request (such as "please enable this additional dependency")

[ ] I'm using the latest version of opencv-python

https://github.com/AUTOMATIC1111/stable-diffusion-webui/issues/8168#issuecomment-1495988226
I used the above link to solve the problem because the version is too high

ultralytics
opencv-python == <IP_ADDRESS>
2.pip install --upgrade "protobuf<=3.20.2"


openhcl: Support AK cert renewal

This is a backport of #371. The code is unchanged from that PR other than diffs required to apply the change cleanly.

pending Hyper-V tests from mingwei

Validated the PR with hyper v


[CP Staging] Fix: navigate after clearing contact method

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/48447
$ https://github.com/Expensify/App/issues/48450
PROPOSAL:
Tests

Go to Account settings.
Go to Profile > Contact methods.
Click New contact method.
Enter an existing email > Next.
Enter magic code > Next.
Click on the added existing email.
Click X button.
Verify you are navigated to contacts list page
Tap New contact method.
Enter a new contact method > Next.
Go offline.
Verify that the offline indicator is shown right above the soft keyboard or at bottom of the page when the soft keyboard isn't open

[ ] Verify that no errors appear in the JS console

Offline tests

QA Steps

[ ] 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: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb 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 the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[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 grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[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 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(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[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 the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[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] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

https://github.com/user-attachments/assets/50ef5b3b-66f7-4b40-b440-c0f74ed22dd8

Android: mWeb Chrome

https://github.com/user-attachments/assets/af68aafb-d86e-41a6-8a65-3cd575aaecda

iOS: Native

https://github.com/user-attachments/assets/ad832d0c-d5c0-46cd-9088-ac3e5f957a73

iOS: mWeb Safari

https://github.com/user-attachments/assets/8e32943a-176a-410a-a90b-c06051d8a97c
https://github.com/user-attachments/assets/0118f03d-d7a9-4746-bc4f-b65439a856ab

MacOS: Chrome / Safari

https://github.com/user-attachments/assets/fecb972e-36d9-4b4a-a115-e0764eefe43b

MacOS: Desktop

https://github.com/user-attachments/assets/b2916e96-fbb9-4b8b-9c40-442ebf65ddbd

Reviewer Checklist

[ ] I have verified the author checklist is complete (all boxes are checked off).
[ ] I verified the correct issue is linked in the ### Fixed Issues section above
[ ] I verified testing steps are clear and they cover the changes made in this PR

[ ] I verified the steps for local testing are in the Tests section
[ ] I verified the steps for Staging and/or Production testing are in the QA steps section
[ ] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] 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)

[ ] I checked that screenshots or videos are included for tests on all platforms
[ ] I included screenshots or videos for tests on all platforms
[ ] I verified tests pass on all platforms & I tested again on:

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

[ ] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[ ] I verified proper code patterns were followed (see Reviewing the code)

[ ] 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).
[ ] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[ ] I verified that comments were added to code that is not self explanatory
[ ] 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.
[ ] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[ ] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[ ] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[ ] 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.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I verified that this PR follows the guidelines as stated in the Review Guidelines
[ ] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] 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
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] 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)
[ ] 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)
[ ] All JSX used for rendering exists in the render method
[ ] 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

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

[ ] 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

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

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

[ ] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[ ] 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)
[ ] 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.
[ ] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[ ] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[ ] I verified that all the inputs inside a form are aligned with each other.
[ ] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[ ] 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.
[ ] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[ ] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Cherry-picked to staging by https://github.com/mountiny in version: 9.0.28-1 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by https://github.com/roryabraham in version: 9.0.28-3 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


[Feature Request]: Switch to new Control Flow syntax & update Prettier to support it

Relevant Scope
angular application
Description
We're already on Angular v17 so that we can switch to the new control flow syntax.
Prettier added support for the syntax in v3.1.0, so that would need to updated as well.
Describe the solution you'd like
Run ng generate @angular/core:control-flow
Check the existing TODO comments for showing the empty state, and add a simple default status message until we get inspiration for some design.
Describe alternatives you've considered
No real impact without this.
Do you want to contribute with a pull request?
Yes

Hi @ajitzero,
I'm waiting for the next stable version of Intellij IDEA with proper support before accepting such a change to make sure everyone have a good developer experience with it (including me).
The new version should be released next week: https://youtrack.jetbrains.com/issue/WEB-63164

Hi @ajitzero,
The support landed in Jetbrains IDEs. You can work on it now!


Label Pods of Managed Deployments

We're currently adding just a couple labels to Pods specified by customer Deployments. https://github.com/UffizziCloud/uffizzi_controller/blob/3dec0f245f6e43a69b3c842cf75ecea79a35b2ae/internal/clients/kuber/requests.go#L48-L51
We'd like to add some additional labels that can be used to later do more useful reporting.
Some suggestions and example values:

app.uffizzi.com/project-id: "3255"
app.uffizzi.com/project-name: "Example Project"
app.uffizzi.com/account-id: "3306"
app.uffizzi.com/account-name: "Example Account"
app.uffizzi.com/user-id: "3465" (value of deployments.deployed_by_id)
app.uffizzi.com/deployment-id: "25505"
app.uffizzi.com/compose-file-id: "38128"
app.uffizzi.com/deployment-metadata: '{"labels": {"gitlab": {"repo": "example/apps/example", "merge_request": {"number": "2666"}}}}'

Note that all values are strings, not numbers!
This info should probably be accepted by the existing ApplyContainers endpoint https://github.com/UffizziCloud/uffizzi_controller/blob/3dec0f245f6e43a69b3c842cf75ecea79a35b2ae/internal/http/containers.go#L66
But I think it should be optional. If this new information is not passed to the HTTP API endpoint, or if any values are "", then we should not add the labels to the Pod template.
I'd love to see some GitHub and GitLab metadata too but that's less important and I'm not sure what the best schema would be. If we include this, we should probably copy this schema from flux, argocd, or someone else who's similarly automating.
This will enable new reporting use cases like:

Replacing my clunky Python script and its many queries for customer usage with a single BigQuery query.
Probably excluding container logs for specific projects?
Other reporting without referencing our Postgres database at all, only querying the k8s Cluster or our cloud infrastructure provider.

If getting or applying any of those values is difficult, just skip them! Literally anything will be better than what we have now.

Note that in addition to the label keys, values also have a restricted character set, so we'll probably need to modify them before use.  https://kubernetes.io/docs/concepts/overview/working-with-objects/labels/#syntax-and-character-set

Valid label value:

must be 63 characters or less (can be empty),
unless empty, must begin and end with an alphanumeric character ([a-z0-9A-Z]),
could contain dashes (-), underscores (_), dots (.), and alphanumerics between.


Build fails: CARGO_CFG_TARGET_FEATURE env-var is not defined or is not UTF-8: NotPresent (architecture: armv7)

See log.
Project: https://qdrant.tech
rust-1.75.0
FreeBSD 14.0

I faced this issue when building on ppc64le machine (IBM Power9).
error: failed to run custom build command for `quantization v0.1.0 (https://github.com/qdrant/quantization.git#0caf67d9)`
thread 'main' panicked at /home/kastanday/.cargo/git/checkouts/quantization-e4ab6fc511580078/0caf67d/quantization/build.rs:16:10:
CARGO_CFG_TARGET_FEATURE env-var is not defined or is not UTF-8: NotPresent

Are you sure your Rust/cargo installation is up-to-date?
It seems to break on a missing CARGO_CFG_TARGET_FEATURE variable. But if compiling with cargo, it should always be there according to this.


Updated Nebula plugin to 3.2.0 and Gradle to 2.10

NetflixOSS » Hystrix » Hystrix-pull-requests #342 FAILURE
Looks like there's a problem with this pull request

NetflixOSS » Hystrix » Hystrix-pull-requests #344 SUCCESS
This pull request looks good


The log info is :  D/DebugDB: Open http://<IP_ADDRESS>:8080 in your browser

what's worry with me, i can not open the database…

r u trying this with emulator ?

@amitshekhariitbhu  yes

@lhlhlh111000  I need to fix the issue happening with emulator.

@lhlhlh111000 @amitshekhariitbhu I don't think we can control emulator IP address by code. but my simple solution for this is
In terminal
adb forward tcp:8080 tcp:8080
Open browser with http://localhost:8080

@vinayagasundar thanks for the info. Will try it soon

@vinayagasundar It works!

I am getting same problem. @amitshekhariitbhu

@vinayagasundar workaround works for me too, but please @amitshekhariitbhu try to make it work with emulators too. Using the default Android Studio emulators is the default option while developing, and it's incredibly faster than using real devices. Plus, after API 23 is not possible to use the Android Device Monitor to pull the DB files from the emulator device, so your debugger is practically the only available tool we have to inspect our data!

@lorenzos sure sure... we will fix it in new release.

not open database in andriod device

I am getting it on Real device too.
Android version: 6.0
Android device model: XT1092

@Aexyn Your device was connected to wifi when you first started the app?

@amitshekhariitbhu No.. It is connected to desktop the whole time

Also it was unable to show the my database as I have a custom path where I store them

+1

I work at android studio 3.0 canary 1
Same error:
05-24 12:44:14.270 14660-14660/...D/DebugDB: Open http://<IP_ADDRESS>:8080 in your browser
Can you help please.

@guyulmaz  Checking the issue. @vinayagasundar any idea?

@amitshekhariitbhu @guyulmaz I'll check this and let you know.

@guyulmaz are you sure your system & device connected to the same Wifi Network
if you're emulator or using USB please refer  here
@amitshekhariitbhu

@vinayagasundar
I'm using the emulator of android studio, the emulator is Nexus 5 api 24 (Android 7)

@guyulmaz okay please check the work around  here

@vinayagasundar
\AppData\Local\Android\sdk\platform-tools>adb forward tcp:8080 tcp:8080
then
Open browser with http://localhost:8080
it works thanks

D/DebugDB: Open http://<IP_ADDRESS>:8081 in your browser
it is my Emulator ip,not my laptop ip, i could not open in my browser

I had the same issue. When switched back to WiFi network from my data connection. Everything seems be working like charm. Good work.

I had the same issue. When switched back to WiFi network from my data connection everything worked. Good work.

It seems that this happens when the device is using a 3G/4G mobile internet, instead of WiFi. Suggestion: Perhaps the app could detect "<IP_ADDRESS>" as a probable error condition and change the message -- something like this:
D/DebugDB: Open http://<IP_ADDRESS>:8080 in your browser. Warning: This address may not work. Ensure your Android device is on the same WiFi network as your computer, then Force Quit and restart the app.

How about a small gradle plugin which would run adb forward tcp:8080 tcp:8080 automatically when running an app with the library in the classpath? I guess you could hook in to the install task or something.

@Kisty This is good suggestion.

i have an issue that error in my room database when use your library. This is the error error: "TransactionDatabase_Impl is not abstract and does not override abstract method clearAllTables() in RoomDatabase". Can you give the solution?

Hey. I can not execute a query like this:
SELECT max (day) as max_day FROM statistics
does not show anything at all!

@lhlhlh111000 @amitshekhariitbhu I don't think we can control emulator IP address by code. but my simple solution for this is
In terminal
adb forward tcp:8080 tcp:8080
Open browser with http://localhost:8080

This tip works for me, just reminding that:
For real smartphone, http://<IP_ADDRESS>:8080 works on Mac OS, while http://localhost:8080 works on Win OS.
Thanks :)

I had the same issue. But looking up the tablets IP and trying with that still worked.

Turn on your Wifi


1.99 breaks glm.vec2() constructor when using numpy with explicit dtype

Hi,
Upgrading from pyGLM from 1.2.0 to 1.99 has broken initialization of vec2 objects when using numpy values of 32 bit precision.
This code snippet:
import glm
import numpy as np
k = np.array([1.,2.],dtype=np.float32)
v=glm.vec2(k[0],k[1])

Produces the error:
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: invalid argument type(s) for vec()

Hi there,
thank you for letting me know about this issue.
The reason why this occurs is because np.array([1.,2.],dtype=np.float32)[0] returns an element of type numpy.float32 - which is not a standard float type.
Previously I had been using PyNumber_Check to determine wether or not a given type can be converted to a a number or not - though that is not what this function is actually intended for. It only checks if the type supports numeric arithmetics or conversion of any sort, thus, glm types were also interpreted as numbers.
Hence I removed that functionality.
I might have an idea to get it to work correctly though, so don't worry.
By the way, you should be able to simply do the following to achieve the same goal:
import glm
import numpy as np
k = np.array([1.,2.],dtype=np.float32)
v = glm.vec2(k) # <--

Cheers
--Zuzu_Typ--

Thanks for the quick response! That's good to know.
Indeed that solution appears that it would work. I'll make the necessary changes.

Dang. I should have tested the numpy buffer protocol before committing this.
Well due to the way NumPy's arrays are generalized, they all support being converted to float or int using float(arr) or int(arr) respectively, though that only works on arrays with only one single element.
This requires me to check, wether or not I can actually get a number from such a type or if it raises an exception.
Thus decreasing performance for numpy array to PyGLM type conversions.


ERROR: (11-22 19:29:53.238) no such package 'modules/perception/tool/offline_visualizer_tool': BUILD file not found on package path.

We appreciate you go through Apollo documentations and search previous issues before creating an new one. If neither of the sources helped you with your issues, please report the issue using the following form. Please note missing info can delay the response time.
System information

OS Platform and Distribution (e.g., Linux Ubuntu 14.04):
Apollo installed from (source or binary):
Apollo version (1.0, 1.5, 2.0, 2.5, 3.0):

Steps to reproduce the issue:

Please use bullet points and include as much details as possible:

Supporting materials (screenshots, command lines, code/script snippets):
I want to use the offline visualizatin tools.
But after runing the following command:
bazel build -c opt //modules/perception/tool/offline_visualizer_tool:offline_lidar_visualizer_tool
terminal shows me:
ERROR: (11-22 19:29:53.238) no such package 'modules/perception/tool/offline_visualizer_tool': BUILD file not found on package path.

There is no '/modules/perception/tool/offline_visualizer_tool' in the master branch.
You need to use the following instead:
bazel build -c opt --cxxopt=-DUSE_GPU //modules/perception/lidar/tools:offline_lidar_obstacle_perception

Thank you.
I tried. But got the error:
daisylab@daisylab-Precision-Tower-7910:~/newdisk4/av/apollo-5.0$ bazel build -c opt --cxxopt=-DUSE_GPU //modules/perception/lidar/tools:offline_lidar_obstacle_perception
WARNING: The following rc files are no longer being read, please transfer their contents or import their path into one of the standard rc files:
/home/daisylab/newdisk4/av/apollo-5.0/tools/bazel.rc
ERROR: /home/daisylab/newdisk4/av/apollo-5.0/WORKSPACE:4:1: name 'new_http_archive' is not defined
ERROR: /home/daisylab/newdisk4/av/apollo-5.0/WORKSPACE:32:1: name 'new_http_archive' is not defined
ERROR: /home/daisylab/newdisk4/av/apollo-5.0/WORKSPACE:48:1: name 'new_git_repository' is not defined
ERROR: /home/daisylab/newdisk4/av/apollo-5.0/WORKSPACE:57:1: name 'new_http_archive' is not defined
ERROR: /home/daisylab/newdisk4/av/apollo-5.0/WORKSPACE:66:1: name 'new_http_archive' is not defined
ERROR: /home/daisylab/newdisk4/av/apollo-5.0/WORKSPACE:75:1: name 'new_http_archive' is not defined
ERROR: /home/daisylab/newdisk4/av/apollo-5.0/WORKSPACE:137:1: name 'new_http_archive' is not defined
ERROR: /home/daisylab/newdisk4/av/apollo-5.0/WORKSPACE:144:1: name 'new_http_archive' is not defined
ERROR: /home/daisylab/newdisk4/av/apollo-5.0/WORKSPACE:209:1: name 'new_http_archive' is not defined
ERROR: /home/daisylab/newdisk4/av/apollo-5.0/WORKSPACE:217:1: name 'new_http_archive' is not defined
ERROR: /home/daisylab/newdisk4/av/apollo-5.0/WORKSPACE:225:1: name 'http_archive' is not defined
ERROR: /home/daisylab/newdisk4/av/apollo-5.0/WORKSPACE:253:1: name 'new_http_archive' is not defined
ERROR: /home/daisylab/newdisk4/av/apollo-5.0/WORKSPACE:274:1: name 'http_archive' is not defined
ERROR: error loading package '': Encountered error while reading extension file 'bazel/grpc_deps.bzl': no such package '@com_github_grpc_grpc//bazel': error loading package 'external': Could not load //external package
ERROR: error loading package '': Encountered error while reading extension file 'bazel/grpc_deps.bzl': no such package '@com_github_grpc_grpc//bazel': error loading package 'external': Could not load //external package
INFO: Elapsed time: 0.294s
INFO: 0 processes.

Should have been fixed in master HEAD.


i followed the steps but there is a problem here : this.APP_ASSETS

this.APP_ASSETS = [
...this.APP_ASSETS,
// {src: ${this.APP_SRC}/your-path-to-lib/libs/jquery-ui.js, inject: true, vendor: false}
// {src: ${this.CSS_SRC}/path-to-lib/test-lib.css, inject: true, vendor: false},
{ src: ${this.APP_SRC}/libs/js/scoll-to-top.js, inject: true, vendor: false},
// { src: ${this.APP_SRC}/libs/masterslider/jquery.easing.min.js, inject: true, vendor: false },
// { src: ${this.APP_SRC}/libs/masterslider/masterslider.min.js, inject: true, vendor: false },
// { src: ${this.APP_SRC}../../assets/js/script.js, inject: true, vendor: false},
//  { src: ${this.APP_SRC}/libs/masterslider/style/masterslider.css, inject: true, vendor: false },
// { src: ${this.APP_SRC}/libs/masterslider/ms-lightbox.css, inject: true, vendor: false },
// { src: ${this.APP_SRC}/libs/masterslider/skins/default/style.css, inject: true, vendor: false },
// { src: ${this.APP_SRC}/libs/masterslider/style.css, inject: true, vendor: false }
];
this function didn't call to the Library local as it should do .
my folder libs where should i put it .
scoll-to-top.js : the same problem when i called to the function did not work but when i put an alert test outside the function it works
could you help me to solve this problems thank you .

I can't comprehend your issue. I see that there is no problem with APP_ASSETS loading your scroll-to-top.js. Could you rephrase your issue and perhaps provide a fork that highlights the issue with angular-seed?

when i called  scroll-to-top.js it didn't work it appear the page bt the fuction on it did't work .i other side master slide  in the local it work but when i integrated it on the angular seed the same issue didn't work the problem in the js pages all the css work normally

I recommend you provide a code example in the form of a fiddle or repository.

this is the arborescence of my work including the libs files

and you found bellow the file witch contain the project.config and the html.could you tell me where is the problem please
angular.zip

That's not a working code sample. Create a fiddle or a repository with your specific issue.
However I believe that your issue is not a bug in angular-seed but rather the incorrect use of the third party libraries. I suggest creating a small page on your own using that library using pure HTML/JS - this is ideal in a fiddle. Make sure all your dependencies and required packages are loaded and use your browser's built-in debugger.
I can't help you any further at this point.


Fixes #16, Added environment variables

Fixes #16

Added environment variable: DB_SERVER_CONTAINER_NAME and PGDATA

The most recent commit (ba152c5) involved incorporating documentation regarding both the new and pre-existing environment variables. Initially, I opted to provide a brief description for each variable within the DEVELOPER.md file. However, it might be more practical to create a dedicated document named ENVIRONMENT-VARIABLES.md. This would facilitate more comprehensive descriptions, especially if we anticipate the introduction of numerous new variables in the future. Additionally, we can consider including this information within the configuration section of the READ.me file in the \bin directory for better accessibility.
I'm awaiting a decision on which option would be the most suitable for the project.

I'll require confirmation on the variable descriptions as well, as I'm uncertain about their precise usage within the project.


EQ3 thermostats broken after update to 0.96

Home Assistant release with the issue:
0.96.0
Last working Home Assistant release (if known):
0.95.4
Operating environment (Hass.io/Docker/Windows/etc.):
Hass.io
Component/platform:
EQ3 Bluetooth Smart Thermostats
Description of problem:
Error while setting up platform eq3btsmart
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 126, in _async_setup_platform
    SLOW_SETUP_MAX_WAIT)
  File "/usr/local/lib/python3.7/asyncio/tasks.py", line 442, in wait_for
    return fut.result()
  File "/usr/local/lib/python3.7/concurrent/futures/thread.py", line 57, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/usr/src/homeassistant/homeassistant/components/eq3btsmart/climate.py", line 69, in setup_platform
    devices.append(EQ3BTSmartThermostat(mac, name), True)
TypeError: append() takes exactly one argument (2 given)

Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
climate:
  - platform: eq3btsmart
    devices:
      livingroom:
        mac: !secret
      bedroom:
        mac: !secret
      kitchen:
        mac: !secret

Fixed in #25238 will land in 0.96.1.

I updated HASS to 0.96.1 but eq3btsmart still does not work for me.
My config:

arch | armv7l
-- | --
dev | false
docker | false
hassio | false
os_name | Linux
python_version | 3.7.3
timezone | Europe/Warsaw
version | 0.96.1
virtualenv | true

climate.yaml:
- platform: eq3btsmart
  devices:
    salon_valve:
      mac: '00:1A:22:xx:xx:x1'
    small_room_valve:
      mac: '00:1A:22:xx:xx:x3'

Other ble devs like xiaomi temperature and humidity sensor works but eq3 not.
hcitool finds these devices:
sudo hcitool lescan
LE Scan ...
00:1A:22:xx:xx:x1 CC-RT-BLE
4C:65:A8:xx:xx:xx MJ_HT_V1
00:1A:22:xx:xx:x3 CC-RT-BLE

eq3cli too:
pi@rpi:~ $ eq3cli --mac 00:1A:22:xx:xx:x1
[00:1A:22:xx:xx:x1] Target 4.5 (mode: manual off dst, away: no)
Locked: False
Batter low: False
Window open: False
Window open temp: 12.0
Window open time: 0:15:00
Boost: False
Current target temp: 4.5
Current comfort temp: 21.0
Current eco temp: 17.0
Current temp offset: 0.5
Current mode: manual off dst
Valve: 0

pi@rpi:~ $ eq3cli --mac 00:1A:22:xx:xx:x3
[00:1A:22:xx:xx:x3] Target 4.5 (mode: manual off dst low battery, away: no)
Locked: False
Batter low: True
Window open: False
Window open temp: 12.0
Window open time: 0:15:00
Boost: False
Current target temp: 4.5
Current comfort temp: 21.0
Current eco temp: 17.0
Current temp offset: 1.0
Current mode: manual off dst low battery
Valve: 0

from HASS virtualenv bluepy/btle.py also foud my eq3 devices:
(homeassistant) pi@rpi:~/homeassistant/lib/python3.7/site-packages/bluepy $ python3 btle.py 00:1A:22:xx:xx:x1
Connecting to: 00:1A:22:xx:xx:x1, address type: public
Service <uuid=Generic Access handleStart=256 handleEnd=337> :
    Characteristic <Device Name>, hnd=0x110, supports READ
    -> b'CC-RT-BLE'
    Characteristic <Appearance>, hnd=0x120, supports READ
    -> b'\x00\x00'
    Characteristic <Peripheral Privacy Flag>, hnd=0x130, supports READ
    -> b'\x00'
    Characteristic <Reconnection Address>, hnd=0x140, supports WRITE
    Characteristic <Peripheral Preferred Connection Parameters>, hnd=0x150, supports READ
    -> b'\x00\x00\x00\x00\x00\x00\x00\x00'
Service <uuid=Generic Attribute handleStart=512 handleEnd=544> :
    Characteristic <Service Changed>, hnd=0x210, supports READ INDICATE
    -> b'\x00\x00\x00\x00'
Service <uuid=Device Information handleStart=768 handleEnd=801> :
    Characteristic <Manufacturer Name String>, hnd=0x310, supports READ
    -> b'eq-3'
    Characteristic <Model Number String>, hnd=0x320, supports READ
    -> b'CC-RT-BLE'
Service <uuid=3e135142-654f-9090-134a-a6ff5bb77046 handleStart=1024 handleEnd=1072> :
    Characteristic <3fa4585a-ce4a-3bad-db4b-b8df8179ea09>, hnd=0x410, supports READ WRITE
    -> b'\x03\x13\x07\x13\x11\x14:\x00\x00\x00\x00\x00\x00\x00\x00\x00'
    Characteristic <d0e8434d-cd29-0996-af41-6c90f4e0eb2a>, hnd=0x420, supports READ WRITE NOTIFY
    -> b'\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00'
Service <uuid=9e5d1e47-5c13-43a0-8635-82ad38a1386f handleStart=65280 handleEnd=65287> :
    Characteristic <e3dd50bf-f7a7-4e99-838e-570a086c666b>, hnd=0xff01, supports WRITE NOTIFY INDICATE
    Characteristic <92e86c7a-d961-4091-b74f-2409e72efe36>, hnd=0xff04, supports WRITE
    Characteristic <347f7608-2e2d-47eb-913b-75d4edc4de3b>, hnd=0xff06, supports READ
    -> b'\x00\x10\x03\x02'

but HASS still show that devices are unavailable.

HASS log:
2019-07-19 16:58:25 INFO (MainThread) [homeassistant.components.climate] Setting up climate.eq3btsmart
2019-07-19 16:58:33 WARNING (SyncWorker_3) [homeassistant.components.eq3btsmart.climate] Updating the state failed: Device disconnected
2019-07-19 16:58:35 WARNING (MainThread) [homeassistant.components.climate] Setup of platform eq3btsmart is taking over 10 seconds.
2019-07-19 17:02:51 WARNING (SyncWorker_18) [homeassistant.components.eq3btsmart.climate] Updating the state failed: Failed to connect to peripheral 00:1A:22:xx:xx:x3, addr type: public
2019-07-19 17:03:49 WARNING (SyncWorker_6) [homeassistant.components.eq3btsmart.climate] Updating the state failed: Failed to connect to peripheral 00:1A:22:xx:xx:x3, addr type: public
2019-07-19 17:06:02 WARNING (SyncWorker_16) [homeassistant.components.eq3btsmart.climate] Updating the state failed: Failed to connect to peripheral 00:1A:22:xx:xx:x3, addr type: public
2019-07-19 17:07:58 WARNING (SyncWorker_14) [homeassistant.components.eq3btsmart.climate] Updating the state failed: Failed to connect to peripheral 00:1A:22:xx:xx:x3, addr type: public

Last 4 log entries inform about the device that is a long distance away, it has always been like that,
but there is no information about the error of reading the first device with mac 00:1A:22:xx:xx:x1,
so I conclude that it is read but because of some reason it is not shown

I have completly the same error - something is broken regarding eq3 bluetooth termostat...
I tried restart several times, also restart HassOS, but nothing helps...
Is there any solution to that? I'm using 0.96.3..


Code for Kinetics dataset

Hi, thanks for releasing your code.
Do you have plans to release your training code for Kinetics dataset anytime soon?

Sorry for the late reply and thanks for your attention.
We have plan to release kinetics code in the future. But it is not on the schedule yet.
You are welcome to ask any question regarding Kinetics training.


[ENG-1973] Allow admin app to associate registration forms with providers

Purpose
Allow admin app to add/remove registration schemas with registration providers.
Changes

created html page with view to control registration schemas
adds tests

Change schema page

Change schema button

QA Notes

Does this change require a data migration? If so, what data will we migrate?
No migration
What is the level of risk?
Low risk only admin

Any permissions code touched?
No
Is this an additive or subtractive change, other?
additive

How can QA verify? (Through UI, API, AdminApp or AdminAdminApp?)
QA can verify by using the admin app to change the associated schemas

If verifying through API, what's the new version? Please include the endpoints in PR notes or Dev docs.
⚠ remember to update docs

What features or workflows might this change impact?
None
How will this impact performance?
No
-->

Documentation
⚠ update api docs when complete
Side Effects
None that I know of.
Ticket
https://openscience.atlassian.net/browse/ENG-1973

Looks good! I tested it locally and everything works well. I think a nice addition could be including an allowed schemas section on the RegistrationProvider detail page.

This will be merged via #9436


song port + note freezing

Describe your problem here.
When i port a song it like freezes the notes-
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
N O Y O U M O T H E R F U C K E R * dies *

What did you edit
if you ported a song then you must have edited something

is the json file name the same as the name of the song

yes, i was trying to port Exanimation (from the fnf x red mod)

You're missing the Instrumental file.
Put it inside mods/songs/your-song-name/Inst.ogg

cough cough Im recharting Exanimation

e


Vulcanize doesn't understand new.target

Using test.html:
<script src="test.js"></script>
and test.js:
"use strict";

class Foo {
	constructor()
	{
		this._target = new.target;
	}
};

then running vulcanize test.html fails with "Error parsing script in test.js at 6:21: unexpected token .". It looks like it's inside the espree module. I checked it separately and it does understand new.target, but only if you set ecmaVersion to 6, so I think vulcanize is either on an old version or has mis-configured something, so it doesn't correctly understand all ECMAScript 6 features.

Meta-bug: why is vulcanize even attempting to parse .js files if all it does is emit a <script src="..."></script> tag referencing the file?

@AshleyScirra If you reinstall vulcanize, it will understand es6 classes.

This isn't about ES6 classes, it's about new.target, or am I missing something?


Add initial support for pfsfuse dependencies

I still cannot compile pfsfuse for win32, maybe I just didn't find proper library.
In file included from ../iomanx_adapter.c:10:
/usr/include/fuse/fuse.h:33:10: fatal error: sys/statvfs.h: No such file or directory
   33 | #include <sys/statvfs.h>
      |          ^~~~~~~~~~~~~~~
compilation terminated.


Support logprobs for chat completion api

See https://platform.openai.com/docs/api-reference/chat/create#chat-create-logprobs

@WoosukKwon @zhuohan123 Could you please have a look at this PR?

Close this PR as this feature has been supported in https://github.com/vllm-project/vllm/pull/2918. Thanks!


Bug fix: Wrong date printed when lease started and spend calculated are on the same day. #350

Proposed changes

Bug fix: Wrong date printed when lease started and spend calculated are on the same day.
#350
Types of changes

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Refactor (changes to code, which do not change application behavior)

Checklist

[x] I have filled out this PR template
[ ] I have read the CONTRIBUTING doc
[ ] I have added automated tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (README.md, inline comments, etc.)
[ ] I have updated the CHANGELOG.md under a ## next release, with a short summary of my changes

Relevant Links

Further comments

/azp run Optum.dce


Can two robotic arms be generated to operate？

I ran the program and found that the generated videos were all of a robotic arm in motion. But I researched the code and couldn't find a definition for the number of robotic arms. Can this project generate two robotic arms for collaborative motion? Looking forward to your reply!

@peppaaaaaaa, in theory, a bimanual setup should be doable. You need to edit environment.py to suit your needs. All expert demos should also be adapted to bimanual setups.

Thanks for your reply!  I have tried and successfully generated it.


[Feature][Workflow Configuration Page] Dynamic rendering of the related functions of the workflow configuration page.

Search before asking

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

Description
The configuration of the current workflow is not divided into different components according to different tools, which is not convenient for subsequent maintenance and update of function points. Therefore, it is planned to split different tools into different components by refactoring the workflow configuration. At the same time, part of the access points of upper-level components, data access methods, and method invocation methods will be part of the reconstruction process. optimization.
The refactored workflow configuration is a way to drive view rendering with data components. The tool configuration of the left toolbar remains unchanged, and the form configuration of different tools is completely written in accordance with the standard in form-create + element-ui. In addition, there must be complete Chinese and English support for the refactored components, so in the subsequent functional changes and expansion, the Chinese and English need to be improved or modified. Under normal circumstances, most of the subsequent custom development and expansion scenarios will be supported in this refactoring, and the code changes and labor costs involved in the subsequent changes are not very large.
Use case

Keep the tasksType constant in config.js, and use this constant to render the tools in the toolbar
The forms of all tools are rendered in the existing formModel.vue, so all form-related code in this file is discarded, including the structure and validation of the form
Manage the form configuration of different tool components through form-create to facilitate subsequent expansion, such as increasing the management function of tool components or changing low-code configurations, etc.
Create a new module folder under the formModel folder to create a new base.js and the js file of the corresponding tool. The base.js file stores the common configuration in the tool form, and the rest js stores the configuration of the corresponding tool. Conditional rendering is performed in the formModel.vue according to the tasksType in the config.js
The formModel.vue file does not use the Message component to prompt for form verification errors, and uniformly uses the form of form verification
The formModel.vue file no longer uses the position layout method after refactoring
Optimize how the parent component calls form-create
Deal with the problem of data confusion as much as possible. In this part of the function, the existing implementation uses a variety of data transfer and method invocation methods, such as mixins, provide, inject, props,  localStorage, vuex, emit and directly calling child component methods through ref in the parent component are not conducive to subsequent maintenance
Change the irregular naming, such as the method addTaskInfo that is exposed to the parent in formModel.vue is replaced with confirm, etc.

Related issues
No response
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

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

Looking forward!

Note that the formModel is not just called in the DAG page

Note that the formModel is not just called in the DAG page

Thanks for reminding


Metrics for DQN

[2019-11-26 16:47:57] step 1080000 | mem_cnt: 4419996 | epsilon: 0.010000 | beta: 1.000000 | avg_reward: 13.777500 | avg_len: 18.70 | avg_steps: 100.20 | model saved | loss: 0.001516 | target net replaced
Can you explain what these metrics stand for ?

Please review the source for these metrics first and see if you can understand them.


Switch off duplication

Output of restic version
future Version
What should restic do differently? Which functionality do you think we should add?
Is it possible to tell restic do not duplicate?
What are you trying to do?
Our compliance have the rule that no backup tool should de-duplicate data.
What's your opinion about this topic and are there any plans to add a switch not to de-duplicate?

I think it's plainly impossible with restic. Because it employs content addressing, that is, chunks of data are stored in files whose names are derived from content hashes.

@cfbao yep, that's an option. I just wanted to ask the community if it's an option.

As @cfbao already said, the code in restic is built on having the deduplication built in and is a core functionality.
There won't be an option to disable this functionality.

While we may add a little bit of redundancy (see #256), maybe by just storing all blobs twice, such a feature would mean restructuring how restic works. That's not realistic. So, I'm closing this feature request for now, I hope you find another backup system you can use! :)


World temperature system

Period of the day
Minecraft Time
Temperature
Temp. increase till next period

Midnight
00:00
-2°C
↑2.5°C every hour

Dawn
06:00
13°C
↑5°C every half hour

Noon
12:00
60°C
↓5°C every hour

Dusk
18:00
35°C
↓4°C every half hour

A temperature > 45°C will slightly blur the players vision and cause nausea; given there are no
blocks above the player providing protection from the sun.

A temperature > 55°C will reduce the player's current health by 20 every half hour (Minecraft Time - MT).

A temperature < 4°C will gradually reduce the player's movement speed; and will reduce their health by 10 every half hour (MT) .

Any source of fire (e.g. campfire, torch) will increase the temperature within the a certain radius by 15°C  .

See #4 to understand the day/night cycle implementation
This temperature system should (for now) only operate when the player enters a Desert biome specifically.

Am I right to assume that these temperature values are calibrated for the desert biome?

Pretty much, yes.

Does this mean that temperature feature will only work in deserts?
After reading this post on Reddit it would seem that Minecraft already does have a temperature system where each biome is assigned a constant temperature value used to color foliage.

btw I would like to catch up on the main feature developments on the 10th if possible over voice chat if that's ok with you.

Not much to catch up at the moment, I am currently working on this feature and am in general informing you of my progress regularly. As you've seen a day or so ago in a live-stream I did me and Rellit found out how the GUI is being drawn on the screen and were able to draw a purple xp bar in place of the vanilla one.
Now that the GUI system has been researched I will focus on implementing the temperature system in the next two days in which I will hopefully have an early system ready.

Ah, sorry I misinterpreted.
The temp values are based upon the temperature range that occurs in real life deserts.
Our temperature system should be kept separate.

I would like to discuss a shadow/ray detection system to help implement a system where shade/caves are able to provide relief by reducing the temperature.

That's a great idea! Let's move this discussion in #24


Make requiring order of spec/support/**/*.rb to be fixed

Dir.glob is not guaranteed order.
This change may help who faced the problem about test success and failure across some environments.

Thanks for your suggestion, this is best left to the projects to decide, if they require files in a specific order they should require them in that order, sort merely makes them alphabetical which is also not necessarily the correct order.

ok. thanks.


Update feature sections design

Changed UI

Button color (All)

Section cloud sync

Section engine's img

Add

Section manageable work

Changed UI

Button color (All)

Section cloud sync

Section engine's img

Add

Section manageable work


One-liner to fix Heron Tracker when results aren't found

When components are not found, a None exception is returned. With this fix, a valid response is returned instead.

lgtm. 👍


[Basic Roleplaying System]Forgot to remove the energy counter

Forgot to remove the energy counter on repeating armors
Changes / Comments
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[ X ] Does the pull request title have the sheet name(s)? Include each sheet name.
[ X ] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.


Question regarding docket entry attachments

Say I get a scrape result for a case, that has an entry like figure 1:
Figure 1
{
    "demand": "",
    "date_terminated": null,
    "docket_entries": [{
        "date_filed": "2020-08-20T00:00:00",
        "document_number": "561",
        "description": "TRANSCRIPT OF PROCEEDINGS before Judge James E. Boasberg held on 8-10-2020; Page Numbers: 1-27. Date of Issuance:8-20-2020. Court Reporter/Transcriber Lisa Griffith, Telephone number<PHONE_NUMBER>, Transcripts may be ordered by submitting the Transcript Order FormFor the first 90 days after this filing date, the transcript may be viewed at the courthouse at a public terminal or purchased from the court reporter referenced above . After 90 days, the transcript may be accessed via PACER. Other transcript formats, (multi-page, condensed, CD or ASCII) may be purchased from the court reporter.NOTICE RE REDACTION OF TRANSCRIPTS: The parties have twenty-one days to file with the court and the court reporter any request to redact personal identifiers from this transcript. If no such requests are filed, the transcript will be made available to the public via PACER without redaction after 90 days. The policy, which includes the five personal identifiers specifically covered, is located on our website at www.dcd.uscourts.gov. Redaction Request due 9/10/2020. Redacted Transcript Deadline set for 9/20/2020. Release of Transcript Restriction set for 11/18/2020.(Griffith, Lisa) (Entered: 08/20/2020)",
        "pacer_doc_id": "04508005331",
        "pacer_seq_no": "2434"
    }],
    "docket_number": "1:16-cv-01534",
    "date_discharged": null,
    "date_converted": null,
    "jurisdiction": "U.S. Government Defendant",
    "mdl_status": "",
    "case_name": "STANDING ROCK SIOUX TRIBE v. UNITED STATES ARMY CORPS OF ENGINEERS",
    "parties": [],
    "date_filed": "2016-07-27T00:00:00",
    "nature_of_suit": "893 Environmental Matters",
    "jury_demand": "None",
    "assigned_to_str": "James E. Boasberg",
    "court_id": "dcd",
    "cause": "05:0706 Judicial Review of Agency Actions",
    "referred_to_str": ""
}

I'm wondering what the best practice is for retrieving the actual document associated with pacer_doc_id:<PHONE_NUMBER>1. I've seen some "download_pdf" methods, but those seemingly are about getting a PDF of the entire docket entry, not the PDF attachment itself (and I could be WAY wrong on that).
I've done a cursory review of the documentation, and some of the seemingly related code. Any guidance on where I should look in the codebase for that functionality, if it indeed exists ?
/tag question

download_pdf will work for attachments or main documents. They'll have different pacer_doc_id values.

Thanks for the tip @mlissner !  I have district court PDF downloads working with the attachments.
If I might be so bold... related question on accessing related PDF documents from Appeals counts. ... the Appeals download_pdf seems to yield  None response.
Say we have a function called "retrieve_documents" that seeks to get PDF content for a particular pacer_doc_id listed in an appellate docket entry:
def retrieve_documents(court_id, docket, pacer_doc_id,
                       user=None, pwd=None):
    adr = AppellateDocketReport(court_id=court_id,
                                pacer_session=PacerSession(username=user, password=pwd))
    response = adr.download_pdf(pacer_doc_id)

court_id = "ca3"
docket = "19-1191" (un-used at the moment)
pacer_doc_id = "003013389677"

The response comes back from juriscraper as None... digging further, the response that comes back from the session.get request is actually a 200 with some strange content. It seems to suggest my PACER creds are invalid, but I have checked them 5 times over, and used them to login to PACER directly to acquire this same document.
ipdb> !r
<Response [200]>
ipdb> !r.text
u'<meta http-equiv="REFRESH" content="0; url=TransportRoom?servlet=InvalidUserLogin.jsp&validLogin=&strJSPName=ShowDoc%26pacer=i%26dls_id=003113389677">\n'

Is the way I'm calling for that PDF content the intended design, or is there a more idiomatic incantation ?

This seems right, but we don't do a lot with appellate content unfortunately. It could be that it broke since we last looked at it. It looks like it's probably the login code that's having some sort of issue. If that's the case, before you debug too much further, you should probably look at how PacerSession is set up. It has some funky heuristics that it uses to determine if the login succeeded that are something of a whitelist. Could be the whitelist needs another entry, or that the approach to getting appellate PDFs isn't quite working.

Understood. If anything interesting turns up, I'll turn it into a PR.


fix: changed dependency-check to use execSync with node env

This PR...
Changes

Fixes

#620

How to test it

Screenshots

Checklist

[X] tested locally on MacOS 11.6
[ ] added new dependencies
[ ] updated the docs
[ ] added a test

Working on MacOS 12


Mekanism Mechanical Pipe disconnect from Refined Storage Reader/Writer

Also reported on SevTech ages and Refined Storage trackers
Issue description:
Rebooting the game causes the majority of "Mekanism" item "xxx Mechanical Pipe" and "xxx Universal Cable" to Disconnect from "Refined Storage" item "Reader" & "Writer".
Liquid example
Energy example
Had a person in chat also say the item and gas pipes disconnect from other Mekanism machines as well(?)
Steps to reproduce:

Set up a Reader and Writer with the same channel
Connect a "Mekanism" item "xxx Mechanical Pipe" to a source into the Reader
Set up a tank to pump the liquid out of the Writer
Reboot game
Pipes(s) would have disconnected from the Reader and Writer

Version (make sure you are on the latest version before reporting):
Forge: <IP_ADDRESS>79
Mekanism: 1.12.2-<IP_ADDRESS>5
Other relevant version: SevTech: Ages 3.0.7

While I have no idea what the reader/writer do, the pipes check for the capability and will connect if it is found in a usable mode. The check will happen again when a neighbour block update is received.


Index Time - Last Time / Last Ingest / Last Tiime IDX not updating

On our test devices in a SHC indexing fields last time | last ingest | last time IDX is not updating.  Appears to show only time app first installed.  See screen shot attachment.  Confirmed data is still being indexed and searchable.  What may cause this issue?

Hi @itspaulp
There are no reason why this wouldn't update unless the tracker reports are not enabled or scheduled, or there is an issue in the KVstore synchronisation in your SHC.
TrackMe entirely relies on KVstore based lookups, a first thing you would verify is if running the tracker manually updates the table as expected, you can simply click on the run tracker buttons now, how does it look like ?
You can see the scheduled reports in the app too, and check if there is any complain of the Splunk scheduler, therefore the app will say it too when you will run the tracker within the app.

Hi Guilhem,
Thank you for the prompt response! I found my first issue is that we have some orphan searches that been set to admin by default and we had deleted the default admin account. I Created a service account and reassigned all searches from within the app as well so the dates are adjusting now.
So part two of this was kind of weird. Originally all the dates when I first installed had shown as 12/03/2020 and stuck there but then had moved to the 03/04/2020 date that you saw on my first report. now the dates and times are cycling, but they're two months ahead. I thought it might be related to the known date time XML issue, but I've checked my environment and they all seem to be up to date. I am going to go ahead and recheck this one more time.
Thanks,
Paul Perron
Sent from Yahoo Mail on Android
On Sat, Apr 4, 2020 at 9:44 AM, Guilhem<EMAIL_ADDRESS>wrote:
Hi @itspaulp
There are no reason why this wouldn't update unless the tracker reports are not enabled or scheduled, or there is an issue in the KVstore synchronisation in your SHC.
TrackMe entirely relies on KVstore based lookups, a first thing you would verify is if running the tracker manually updates the table as expected, you can simply click on the run tracker buttons now, how does it look like ?
You can see the scheduled reports in the app too, and check if there is any complain of the Splunk scheduler, therefore the app will say it too when you will run the tracker within the app.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Closed due to inactivity, please re-open this issue shall this be required.


modal displaying under backdrop on iPhone

Hi,
I have a problem with modal on Iphone. I use Bootstrap v3.2.0
On Iphone, when I open a modal, it display under back-drop. I researched in many topic but no working.
This problem only occur on Iphone, isn't occur on PC or Android devices.
Please help.
Sorry if I post a duplicate issue.

Please upgrade to Bootstrap v3.3.7.
Also, you will need to post a JS Bin or other live example that demonstrates the problem, per the issue tempate.

Hi @cvrebert
I'm sorry, maybe my mistake. Although I haven't found the cause yet, but it's seem only occur on my project. Please remove this issue.
I'm so sorry for this inconvenient.
Thanks.

Hi @cvrebert,
I just found the root cause of my issue.
I use: -webkit-overflow-scrolling: touch in my code.
Is this similar to #17695 ?

Quite possible. (Though I haven't heard of webkit-overflow-scrolling affecting backdrop placement before, but clearly it's a bug-prone property.)


Fix #25 test_import unit tests

What was wrong?
Issue #25
How was it fixed?

Some modules create variables at import time. Thus, the module imports needed to be cleaned between tests so that the variables are created cleanly for each test. Otherwise, the module would be imported only once and the variables would be used in the next tests.

For tests that depended that some packages didn't exist to get an ImportError, system modules were mocked so that the tests can't find the packages regardless of whether they actually were installed or not.

Cute Animal Picture

@jluttine there are some minor linting errors (sorting of imports) that are causing CI to fail.  Can you clean those up?

@pipermerriam Ok, done, if I understood correctly what and how to fix.


Auto-Update?

Hi @JamesWoolfenden
I noted you've been releasing a lot lately, so I wondered if it might be an idea to have an update option in pike where you can update to the latest release using the tool?
For Windows/Linux I wrote a couple of scripts that I've scheduled to check daily & pull the latest version, but might be an idea?
Scripts for info, or if you can't be bothered to do an auto-updater, maybe you could include these somewhere if someone else might want to use?  (I'm don't have a Mac so can't write something for that).
Windows/Powershell:  (uncommenting the last line if you want options to be different)
function GetLatestPike {
  param (
    [parameter(Mandatory = $false)]$Match = "windows_amd64",
    [parameter(Mandatory = $false)]$Root = "$env:userprofile\Downloads",
    [parameter(Mandatory = $false)]$Target = "C:\Windows"
  )
  $ProgressPreference = "SilentlyContinue"
  [Net.ServicePointManager]::SecurityProtocol = [Net.SecurityProtocolType]::Tls12

  $GithubUrl = "https://api.github.com/repos/JamesWoolfenden/pike/releases/latest"
  $Response = Invoke-WebRequest -Uri $GithubUrl -MaximumRedirection 0 -ErrorAction SilentlyContinue
  $LatestRelease = $Response.Content | ConvertFrom-Json
  $LatestUrl = $LatestRelease.assets | Where-Object { $_.browser_download_url -like "*$Match*" }
  $LatestName = $LatestUrl.name
  $zipDownloadUrl = $LatestUrl.browser_download_url

  $DLFol = Join-Path -Path $Root -ChildPath $LatestName.Replace(".zip", "")

  if (-not (Test-Path -Path $DLFol -PathType Container)) {
    New-Item -Path $DLFol -ItemType Directory | out-null
    Write-Host "Folder: $DLFol created.." -Fore Cyan
  } else {
    Write-Host "The latest version is already present...  exiting.." -Fore Yellow
    break
  }

  $zipFilePath = Join-Path -Path $DLFol -ChildPath $LatestName
  Invoke-WebRequest -Uri $zipDownloadUrl -OutFile $zipFilePath
  Expand-Archive -Path $zipFilePath -DestinationPath $DLFol -Force
  Remove-Item -Path $zipFilePath
  Copy-Item -Path "$DLFol\pike.exe" -Destination "$Target\pike.exe" -Force

  Write-Host "Latest pike installed from $DLFol.." -Fore Green
}

GetLatestPike #-Match windows_amd64 -Root C:\tmp -Target c:\Windows

Linux/Bash:  (uncommenting the last line if you want options to be different)
#!/bin/bash

get_latest_pike() {
    local Match="${1:-linux_amd64}"
    local Root="${2:-/home/alex/Downloads}"
    local Target="${3:-/usr/local/bin}"

    local GithubUrl="https://api.github.com/repos/JamesWoolfenden/pike/releases/latest"
    local LatestRelease
    LatestRelease=$(curl -s "$GithubUrl")

    local LatestUrl
    LatestUrl=$(echo "$LatestRelease" | jq -r '.assets[] | select(.browser_download_url | contains("'"$Match"'")) | .browser_download_url')

    if [ -z "$LatestUrl" ]; then
        echo "No matching asset found for $Match"
        exit 1
    fi

    local LatestName
    LatestName=$(basename "$LatestUrl")
    local ext=".tar.gz"
    local zipDownloadUrl="$LatestUrl"
    local DLFol="$Root/${LatestName%$ext}"

    if [ ! -d "$DLFol" ]; then
        mkdir -p "$DLFol"
        echo "Folder: $DLFol created.."
    else
        echo "The latest version is already present... exiting.."
        exit 0
    fi

    local zipFilePath="$DLFol/$LatestName"
    curl -L -o "$zipFilePath" "$zipDownloadUrl"
    tar -xzf "$zipFilePath" -C "$DLFol"
    rm -f "$zipFilePath"
    cp "$DLFol/pike" "$Target/pike"
    echo "Latest pike installed from $DLFol.."
}

get_latest_pike #"other_match" "/path/to/root" "/path/to/target"

Cheers

id prefer a go based solution?  pike is supported in both brew and scoop so updating should be pretty easy?

I'll leave that with you :)


Update dependencies

Closes #71

Ah my bad! Good catch and thanks for the PR! Will push the new version of the gem after merging :)


[WIP] Head styles include boostlook.css

@julioest @rbbeeston
I updated the PR with the --skip-boostlook option for gulp. So, we have three workflows:

When pushing to the repo (CI) or building the UI bundle for the first time, the file won't be there and gulp build will download it.
The bundle will reuse the file when working on the content of the components (Peter Turcan's workflow). It will attempt to update the stylesheets once an hour.
When working on boostlook.css itself, you can build it with gulp build --skip-boostlook so that it doesn't attempt to update the local file.

This should cover all cases. I hope that's OK.

@julioest @rbbeeston I need a review here.
To make this work, I had to study Gulp more in-depth, so I refactored the Gulp files to make them more straightforward.
As you'll see, the integration is working, but the problem is that it just looks too ugly. So, I need you guys to decide what the workflow should be now.

Maybe you guys want me to integrate it differently in head-styles.hbs
Maybe there's work to do in Boostlook before merging this here
Maybe there's work to do here in site.css before merging this PR

@rbbeeston @alandefreitas Thanks for the notes and hard work on this!
The issue was due to old code I was experimenting with. It will be fixed once my PR is merged (https://github.com/cppalliance/boostlook/pull/1)

The UI issue was due to old CSS grid code I was experimenting with. It will be fixed once my PR is merged (https://github.com/cppalliance/boostlook/pull/1).

Does that mean https://github.com/cppalliance/boostlook/pull/2 is obsolete? It'd be great if we could merge only https://github.com/cppalliance/boostlook/pull/1.


Remove depracated composer target-dir usage

Ref: https://getcomposer.org/doc/04-schema.md#target-dir
See also: https://github.com/maglnet/ComposerRequireChecker/issues/55

Thanks!


Using the action produces set-output deprecation warnings

Using the action with version v0.9.0 (and currently pinned to b7355a1) produces set-output deprecation warnings:
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-

Other deprecation warnings like theses were addressed for this issue https://github.com/yokawasa/action-setup-kube-tools/issues/33, but it seems there are ones remaining.

@msg-freiheit  Thank you for the inssue!! I'll take a look!

Thank you! I only had a very cursory look, but couldn't see any problems in the code of this action itself right away, so it might even be sufficient to just rebuild the dist/index.js file, maybe there were some unintended changes in there from this PR. I could create a PR for that if you like?

@msg-freiheit That would be great if you could try and create PR for the issue - Thank you!

@msg-freiheit  v0.9.1 has been released! Thanks!
https://github.com/marketplace/actions/setup-kubernetes-tools


Placeholder width too small

Some of my placeholders are too small. I saw that a CSS override was suggested for #3824, but I see that the min width is determined this way by cms.structureboard.js:
min = (item.width()) ? 0 : 150;
Do you see any problems with changing this to something like
min = item.width() < 150 ? 150 - item.width() : 0;
So that it pads say 15px placeholders with 135px instead? Happy to submit a pull request to this effect. Thanks.

The structure mode rendering is going to be adapted in the next release to overcome situation like yours. It's not an quick fix, but we'll make it better over the time

Fixed in 3.2


Failure to find org.pentaho:pentaho-aggdesigner-algorithm:jar:5.1.5-jhyde

Got latest code from master branch. Was getting error message given below:
Failure to find org.pentaho:pentaho-aggdesigner-algorithm:jar:5.1.5-jhyde in https://repository.cloudera.com/artifactory/repo/
Added following repository in the root pom.xml to fix it:
    <repository>
        <id>spring-libs</id>
        <url>http://repo.spring.io/libs-milestone/</url>
    </repository>

@vinothchandar Sorry to be a pain, I just hit this too on a clean clone/build too. I can't see that jar in either of the two configured repositories.

http://central.maven.org/maven2/org/pentaho/
https://repository.cloudera.com/artifactory/repo/org/pentaho/

@bvaradar as well. Did you hit this issue by any chance when you did the docker stuff?
Should we just add the repository.

Not seeing this issue anymore. Please reopen if you still see the problem

Hello everyone,
I am hitting the same error while trying to build Apache Hudi as described in the github REAMDE.md:
[ERROR] Failed to execute goal on project hudi-hadoop-mr: Could not resolve dependencies for project org.apache.hudi:hudi-hadoop-mr:jar:0.9.0-SNAPSHOT: Failure to find org.pentaho:pentaho-aggdesigner-algorithm:jar:5.1.5-jhyde in https://repo.maven.apache.org/maven2 was cached in the local repository, resolution will not be reattempted until the update interval of Maven Central has elapsed or updates are forced -> [Help 1]

I tried what @ajaychitre did but without any success.
Many thanks for your feedback

Hi, is this issue resolved.
Even after cloning the https://github.com/vinothchandar/hudi/tree/remove-spring-deps adn attemp to mvn build, getting the same error.
Please help.
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal on project hudi-hadoop-mr: Could not resolve dependencies for project org.apache.hudi
:hudi-hadoop-mr:jar:0.9.0-SNAPSHOT: Failed to collect dependencies at org.apache.hive:hive-exec:jar:core:2.3.1 -> or
g.apache.calcite:calcite-core:jar:1.10.0 -> org.pentaho:pentaho-aggdesigner-algorithm:jar:5.1.5-jhyde: Failed to rea
d artifact descriptor for org.pentaho:pentaho-aggdesigner-algorithm:jar:5.1.5-jhyde: Could not transfer artifact org
.pentaho:pentaho-aggdesigner-algorithm:pom:5.1.5-jhyde from/to maven-default-http-blocker (http://<IP_ADDRESS>/): Blocked
mirror for repositories: [datanucleus (http://www.datanucleus.org/downloads/maven2, default, releases), glassfish-r
epository (http://maven.glassfish.org/content/groups/glassfish, default, disabled), glassfish-repo-archive (http://m
aven.glassfish.org/content/groups/glassfish, default, disabled), apache.snapshots (http://repository.apache.org/snap
shots, default, snapshots), central (http://repo.maven.apache.org/maven2, default, releases), conjars (http://conjar
s.org/repo, default, releases+snapshots)] -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/DependencyResolutionException
[ERROR]
[ERROR] After correcting the problems, you can resume the build with the command
[ERROR]   mvn  -rf :hudi-hadoop-mr

Also facing with this issue...is there any solution?

reopening this. @n3nash could you please try and repro this? If the jar is already cached in local .m2, it won't repro. So we need to manually delete that or force update/fetch.

Yes, as @vinothchandar commented, it started to happen after I've removed .m2/ directory.
(I had back-up of this folder, and it worked well again after I reverted this)

@vinothchandar Any solution if I deleted the m2 directory and no backup for it?

@giridharmunagala  Oh, sorry for confusion. Does not work though I've revert the backup m2 storage...

Looks like this was being fetched out of the spring repo. and we removed the spring repository recently(#2481  to fix #2479), since it was retired.
So now the issue is maven central does not have this jar.
https://mvnrepository.com/artifact/org.pentaho/pentaho-aggdesigner-algorithm/5.1.5-jhyde

Many thanks for your prompt feedback @vinothchandar. I stay tuned. Cheers.

So this artifact is accessible from none of the repositories anymore. Good grief. :) Trying to exclude it from the build, none of the code actually uses it. its a third level transitive dependency

lets see if  #3034 passes integration tests. For now, it should resolve your build issue itself.

I ran into this just now. I got a new laptop and tried hudi for first time in clean slate and ran into the exact issue.

That patch seems to work. I ll land, once the next round of tests pass.
@nsivabalan yes. as I memtioned, that artifact is not served by maven central amymore (for whatever reason). so new installs will see this.

For Chinese developers,
Added following repository in the root pom.xml to fix it - works at 2024-04-30:
<repository>
    <id>huawei-maven</id>
    <url>https://repo.huaweicloud.com/repository/maven/huaweicloudsdk/</url>
</repository>