bump 1.12.0

Codecov Report

Merging #37 (625df78) into master (0ff17b2) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #37   +/-   ##
=========================================
  Coverage     77.16%   77.16%
  Complexity      266      266
=========================================
  Files            37       37
  Lines           959      959
  Branches         42       42
=========================================
  Hits            740      740
  Misses          196      196
  Partials         23       23

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0ff17b2...625df78. Read the comment docs.


getProducts returns empty array in newest version RN

Version of react-native-iap
^2.4.7
Version of react-native
0.59.4
Platforms you faced the error (IOS or Android or both?)
iOS
Expected behavior
I expect getProducts to return an array of available purchases.
Actual behavior
The returned device is empty.
Tested environment (Emulator? Real Device?)
Both.
Steps to reproduce the behavior
yield RNIap.prepare()
const products = yield RNIap.getProducts(iapBundleIds(content))

console.log(iapBundleIds(content))
// ["org.truthordrink.TruthOrDrink.pack1", "org.truthordrink.TruthOrDrink.pack2", "org.truthordrink.TruthOrDrink.pack3"]
console.log(products)
// []

The above code in the same xcode project works correctly using:
"react-native": "0.56.0",
"react-native-iap": "^2.4.0-beta6"

There are several issues related to this which are already closed. Please follow that and this should be working.

Only one i could find that remotely matched mine was #340, but you closed that one instantly too without any answer.
For me, everything is working great in react-native 0.56.0, but not react-native 0.59.4, can't find anything related to that. Could you point me to one of the several ones related to my ticket?

Same problem here

Same here

I'm seeing the same. Are there any plans to address what seems like a confirmed issue with 0.59.4?

Why is this issue even closed? getProducts returns empty array in react-native 0.59.8 as well.

@hyochan: per your above comment that presumably resulted in this issue being closed, please kindly let us know what issues this one duplicates.
This is the only library of its kind that I know of, AND it is not working in current versions of RN. It would be great to get a better sense of what the issue is so the community can work towards getting it resolved. We need this library!

Related #407

I solved it! I am using these versions,
react-native-iap: ^4.3.0
react-native: 0.60.5
just do it like this
const itemSkus = Platform.select({
    ios: [
        '100coins' // just remove bundle id from product id
    ],
    android: [
        '100coins' // just remove bundle id from product id
    ]
});


Add SententialDecisionDiagrams.jl as an optional dependency

Add https://github.com/pedrozudo/SententialDecisionDiagrams.jl as an optional dependency (https://github.com/JuliaLang/Pkg.jl/issues/1285).
At minimum, make it easy to convert the native SDD data structure to Julia structs in the LogicCircuit hierarchy, so we can do things like compute flows efficiently.

I guess Requires.jl also should work perfectly for this. And the main useful integrations are:

Import Vtree from SDD.jl to Juice
Import SDDs from SDD.jl to Juice

Do we need anything else? Maybe later on case also add more interactive scenarios.


Seeing latency in collector while reading from cross-region kinesis

Collector works great while reading from kinesis stream in same region. But shows latency issues with significant traffic when reading cross-region. It kinda throttles and hence shows lag.

Fixed in the release: https://github.com/ExpediaDotCom/haystack-collector/releases/tag/1.1.8


Fix update_option

Fix update DB on every page load for numeric options https://github.com/soberwp/intervention/issues/64

Thanks @pacotole, this should be a non-issue since changing to pre_option as no DB updates are being run now, under #64.


bug in Confirm mail

Always when i send to http://localhost:3000/api/v1/auth/email/register
i get: {"statusCode":404,"message":"Cannot GET /confirm-email/0ace23e1ad6fde095f933e65b8ff2b461c122197e9f6321b04690ae99fc0315d","error":"Not Found"}
why? I must add some specific settings?

@adixoncode I have the same problem.

@jkoutavas it should be a path to your frontend app, not the backend server.
example: https://my-frontend-app.com/confirm-email/0ace23e1ad6fde095f933e65b8ff2b461c122197e9f6321b04690ae99fc0315d


Error while deploying todo sample

Description:
Tried to execute deploy todo sample from the image in cellery hub. Could observe below errors and logs.
Hasinthas-MacBook-Pro:6 hasinthaindrajee$ cellery run wso2cellery/todo-cell:latest-dev -n todos
⠋ Extracting cell image
Pulling image wso2cellery/todo-cell:latest-dev
⠇ Extracting cell image
Image Digest : sha256:d4e22f11648fe328417d8e26149e91d0b8b5004f4f629b0363e96190a4e877e4
✔ Pulling cell image
Saving new Image to the Local Repository
✔ Extracting cell image
⠋ Starting main instance todos---------------------true
✔ Creating temporary executable main bal file
---------------------/Users/hasinthaindrajee/.cellery/tmp/cellery-cell-image787409567/src/todo
⠧ Starting main instance todos----------------------------/home/cellery/.cellery/tmp/cellery-cell-image787409567/src/todo
----------------------------[docker exec -e CELLERY_HOME=/Users/hasinthaindrajee/work/celleryhome -e CELLERY_IMAGE_DIR=/home/cellery/.cellery/tmp/cellery-cell-image787409567 -w /home/cellery/.cellery/tmp/cellery-cell-image787409567/src/todo -u 1000 45513cfeb55448e07cb1a3e66fd17257f6c4ba6980e9e3f4f8e7fada3ea65a42 /usr/lib/ballerina/ballerina-1.0.3/bin/ballerina run todo run {"org":"wso2cellery","name":"todo-cell","ver":"latest-dev","instanceName":"todos","isRoot":true} {} false false]
Compiling source
celleryio/todo:0.6.0
Creating balos
target/balo/todo-2019r3-any-0.6.0.balo
Generating executables
target/bin/todo.jar
Running executables
Info: Main Instance: todos
Info: Validating dependencies
Info: Instances to be Used
INSTANCE NAME                  CELL IMAGE                          USED INSTANCE             KIND            SHARED
todos                          wso2cellery/todo-cell:latest-dev    To be Created             Cell            -
Info: Dependency Tree to be Used
No Dependencies
Warning: Error from server (BadRequest): error when creating "/home/cellery/.cellery/tmp/cellery-cell-image787409567/artifacts/cellery/todo-cell.yaml": admission webhook "validating.mesh.cellery.io" denied the request: validation failed: Cell.mesh.cellery.io "todos" is invalid: [spec.components[0].spec.template.containers[0].volumeMounts[0].name: Not found: "init-sql", spec.components[0].spec.template.containers[0].volumeMounts[1].name: Not found: "data-vol", spec.components[1].spec.template.containers[0].volumeMounts[0].name: Not found: "db-credentials"]
error: io.cellery.exception.BallerinaCelleryException message=Unable to apply composite yaml /home/cellery/.cellery/tmp/cellery-cell-image787409567/artifacts/cellery. Command kubectl apply -f  /home/cellery/.cellery/tmp/cellery-cell-image787409567/artifacts/cellery/todo-cell.yaml exited with exit code 1 message: Error from server (BadRequest): error when creating "/home/cellery/.cellery/tmp/cellery-cell-image787409567/artifacts/cellery/todo-cell.yaml": admission webhook "validating.mesh.cellery.io" denied the request: validation failed: Cell.mesh.cellery.io "todos" is invalid: [spec.components[0].spec.template.containers[0].volumeMounts[0].name: Not found: "init-sql", spec.components[0].spec.template.containers[0].volumeMounts[1].name: Not found: "data-vol", spec.components[1].spec.template.containers[0].volumeMounts[0].name: Not found: "db-credentials"]
❌ Starting main instance todos
Failed to start main instance todos
Cellery run command failed: failed to run bal file, failed waiting to execute run method Compiling source	celleryio/todo:0.6.0Creating balos	target/balo/todo-2019r3-any-0.6.0.baloGenerating executables	target/bin/todo.jarRunning executableserror: io.cellery.exception.BallerinaCelleryException message=Unable to apply composite yaml /home/cellery/.cellery/tmp/cellery-cell-image787409567/artifacts/cellery. Command kubectl apply -f  /home/cellery/.cellery/tmp/cellery-cell-image787409567/artifacts/cellery/todo-cell.yaml exited with exit code 1 message: Error from server (BadRequest): error when creating "/home/cellery/.cellery/tmp/cellery-cell-image787409567/artifacts/cellery/todo-cell.yaml": admission webhook "validating.mesh.cellery.io" denied the request: validation failed: Cell.mesh.cellery.io "todos" is invalid: [spec.components[0].spec.template.containers[0].volumeMounts[0].name: Not found: "init-sql", spec.components[0].spec.template.containers[0].volumeMounts[1].name: Not found: "data-vol", spec.components[1].spec.template.containers[0].volumeMounts[0].name: Not found: "db-credentials"]

Fixed with https://github.com/wso2/cellery-controller/pull/153


Error reading the Service Account json

Hi all,
a bit of a weird issue. Have some code I've repeatedly used along with a service account json file I've also used a few times. Porting the code to a new micorservice and getting this error:
Failed to initialize FCM client: Oauth(ServiceAccountKeyReadingFailed(Custom { kind: InvalidData, error: "Bad service account key: expected value at line 1 column 1" }))
Error seems to suggest its an issue reading the json rather than a handshake issue or anything else like that. This is running from inside a docker container. Have tried changing the json file but always getting the same error.
code looks like this:
use firestore::{FirestoreDb, FirestoreDbOptions};

    let project_id = env::var("PROJECT_ID").unwrap();
    let db = FirestoreDb::with_options_service_account_key_file(
        FirestoreDbOptions::new(project_id),
        "/etc/project/project.json".into(),
    ).await?;

Flummoxed at this point and reaching out to see if anyone had come across this before.
debug info:
[2024-08-11T00:33:16Z INFO  firestore::db] Creating a new database client. database_path="projects/test1-j1vzp4/databases/(default)" api_url="https://firestore.googleapis.com" token_scopes="https://www.googleapis.com/auth/cloud-platform"
[2024-08-11T00:33:16Z DEBUG gcloud_sdk::api_client] Creating a new Google API client for https://firestore.googleapis.com. Scopes: ["https://www.googleapis.com/auth/cloud-platform"]
[2024-08-11T00:33:16Z DEBUG rustls::webpki::anchors] add_parsable_certificates processed 140 valid and 0 invalid certs
[2024-08-11T00:33:16Z DEBUG hyper::client::connect::dns] resolving host="firestore.googleapis.com"
[2024-08-11T00:33:16Z DEBUG hyper::client::connect::http] connecting to <IP_ADDRESS>:443
[2024-08-11T00:33:16Z DEBUG hyper::client::connect::http] connected to <IP_ADDRESS>:443
[2024-08-11T00:33:16Z DEBUG rustls::client::hs] No cached session for DnsName("firestore.googleapis.com")
[2024-08-11T00:33:16Z DEBUG rustls::client::hs] Not resuming any session
[2024-08-11T00:33:16Z DEBUG rustls::client::hs] Using ciphersuite TLS13_AES_256_GCM_SHA384
[2024-08-11T00:33:16Z DEBUG rustls::client::tls13] Not resuming
[2024-08-11T00:33:16Z DEBUG rustls::client::tls13] TLS1.3 encrypted extensions: [Protocols([ProtocolName(6832)])]
[2024-08-11T00:33:16Z DEBUG rustls::client::hs] ALPN protocol is Some(b"h2")
[2024-08-11T00:33:16Z DEBUG h2::client] binding client connection
[2024-08-11T00:33:16Z DEBUG h2::client] client connection bound
[2024-08-11T00:33:16Z DEBUG h2::codec::framed_write] send frame=Settings { flags: (0x0), enable_push: 0, initial_window_size: 2097152, max_frame_size: 16384 }
[2024-08-11T00:33:16Z DEBUG h2::proto::connection] Connection; peer=Client
[2024-08-11T00:33:16Z DEBUG h2::codec::framed_write] send frame=WindowUpdate { stream_id: StreamId(0), size_increment: 5177345 }
thread 'main' panicked at src/main.rs:252:10:
Failed to initialize FCM client: Oauth(ServiceAccountKeyReadingFailed(Custom { kind: InvalidData, error: "Bad service account key: expected value at line 1 column 1" }))
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Wondering if anything changed, like maybe a downstream app or library used to read and validate the json file. This behaviour literally started overnight.

Hey,
AFAIK, nothing related to key JSON files was changed recently. It works in my environment as well.
Looking at error it complains about JSON structure just at the first line.
Please verify if you have appropriate JSON file there that looks similar to this:

The error looks like it may be not even JSON file.

Failed to initialize FCM client: Oauth(ServiceAccountKeyReadingFailed(Custom { kind: InvalidData, error: "Bad service account key: expected value at line 1 column 1" }))

By the way, I don't recognize this error. FCM/Oauth/ServiceAccountKeyReadingFailed aren't coming neither from firestore or gcloud sdk.
Is it Firebase Cloud Messaging? This crate is not related to anything else except Firestore (database).

Thanks! combing through the code found the issue. It was another library with a very similar connection scheme. The connection method was named the same but works differently. Apologies p firestore-rs was working fine the whole time.


python310Packages.aioesphomeapi: 16.0.0 -> 16.0.1

Diff: https://github.com/esphome/aioesphomeapi/compare/refs/tags/v16.0.0...v16.0.1
Changelog: https://github.com/esphome/aioesphomeapi/releases/tag/v16.0.1
Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

40 packages built:

esphome
esphome.dist
home-assistant-component-tests.airthings_ble
home-assistant-component-tests.aranet
home-assistant-component-tests.bluemaestro
home-assistant-component-tests.bluetooth_adapters
home-assistant-component-tests.bluetooth_le_tracker
home-assistant-component-tests.bthome
home-assistant-component-tests.dormakaba_dkey
home-assistant-component-tests.esphome
home-assistant-component-tests.eufylife_ble
home-assistant-component-tests.fjaraskupan
home-assistant-component-tests.govee_ble
home-assistant-component-tests.homekit_controller
home-assistant-component-tests.ibeacon
home-assistant-component-tests.inkbird
home-assistant-component-tests.kegtron
home-assistant-component-tests.keymitt_ble
home-assistant-component-tests.ld2410_ble
home-assistant-component-tests.led_ble
home-assistant-component-tests.moat
home-assistant-component-tests.mopeka
home-assistant-component-tests.oralb
home-assistant-component-tests.qingping
home-assistant-component-tests.rapt_ble
home-assistant-component-tests.ruuvitag_ble
home-assistant-component-tests.sensirion_ble
home-assistant-component-tests.sensorpro
home-assistant-component-tests.sensorpush
home-assistant-component-tests.snooz
home-assistant-component-tests.switchbot
home-assistant-component-tests.thermobeacon
home-assistant-component-tests.thermopro
home-assistant-component-tests.tilt_ble
home-assistant-component-tests.xiaomi_ble
home-assistant-component-tests.yalexs_ble
python310Packages.aioesphomeapi
python310Packages.aioesphomeapi.dist
python311Packages.aioesphomeapi
python311Packages.aioesphomeapi.dist


Shark: LLVM 2.8 %R4Unsupported operand type for movw/movt

xranby@ac100-armel:/media/d3f564a8-fdb8-4516-88bf-6a87c212e471/qemuarm/usr/lib/jvm/java-6-openjdk/jre/lib/arm$ appletviewer -J-shark http://oneslime.net
%R4Unsupported operand type for movw/movt
UNREACHABLE executed at /usr/src/openembedded/oe-core/build/tmp-eglibc/work/armv5te-oe-linux-gnueabi/llvm2.8-2.8-r0/llvm-2.8/lib/Target/ARM/ARMCodeEmitter.cpp:261!
Stack dump:
0.	Running pass 'ARM Machine Code Emitter' on function '@"java.lang.String::hashCode"'
Aborted (core dumped)
Revert back to the patched LLVM 2.7 that are known to be working from oe-classic?

Hmm.. how about adding the following to the beginning of the failing if else block. R4 are a register.
if (MO.isReg())
return ARMRegisterInfo::getRegisterNumbering(MO.getReg());
else if ,,,

Same problem for armv7a:
root@beaglebone:~/java# java -shark CaffeineMarkEmbeddedApp
%R4Unsupported operand type for movw/movt
UNREACHABLE executed at /OE/tentacle/build/tmp-angstrom_2010_x-eglibc/work/armv7a-angstrom-linux-gnueabi/llvm2.8-2.8-r0/llvm-2.8/lib/Target/ARM/ARMCodeEmitter.cpp:261!
Stack dump:
0.	Running pass 'ARM Machine Code Emitter' on function '@"SieveAtom::execute"'
Aborted

This bug are got introduced with LLVM 2.8 and are still unfixed in the last LLVM 3.0 release. I have created a patch and sent it upstream for review: http://llvm.org/bugs/show_bug.cgi?id=11736


Update tests and docs for AzureCloudInstance support

I just see docs, are there tests coming as well?

Codecov Report

Merging #4502 (16b59d7) into dev (0a8e67b) will not change coverage.
The diff coverage is n/a.

Flag
Coverage Δ

*Carryforward flag

msal-angular
96.39% <ø> (ø)

Carriedforward from 0a8e67b

msal-browser
87.50% <ø> (ø)

Carriedforward from 0a8e67b

msal-common
85.08% <ø> (ø)

Carriedforward from 0a8e67b

msal-core
82.67% <ø> (ø)

Carriedforward from 0a8e67b

msal-node
82.25% <ø> (ø)

msal-node-extensions
76.03% <ø> (ø)

Carriedforward from 0a8e67b

msal-react
92.70% <ø> (ø)

Carriedforward from 0a8e67b

*This pull request uses carry forward flags. Click here to find out more.

I just see docs, are there tests coming as well?

@sameerag I'm assuming this is still a draft or is incomplete?

adding a reminder for linking & updating the authority doc here

Closed. Will update for 3.x as applicable.

Abandoning to be updated for 3.x


Cannot Locate Guidance For Vulnerability Scanning Reports.

Hi, my name is klawrawkz. I am fine. How are you?
I have been reading and re-reading the documentation pertaining to Azure Defender  (or Microsoft Azure Defender For SQL or SQL Azure Microsoft SQL Azure Defender for Bunch of Stuff Wadded Up Out In The Universe Somewhere or Whatever Amorphous Moniker You Want To Use Today, Because It Will Change Tomorrow (Probably)) and I have not been able to figure out how to run a SQL Vulnerability Scan and produce a report detailing the results of all our fine security work.
Here's what we "learn" from the documentation, which as you see below claims to have content pertinent to "Scan[ning] your Azure SQL databases for vulnerabilities." To me this implies that we end users are able to both scan and to produce reports containing information about the state of our compliance/security regarding Azure SQL server and/or database(s). Nowhere in the documentation are we provided any actionable information as to what is the procedure, methodology, or steps that allow us to run a vulnerability scan and produce a report detailing the scan results.
From the document we are told:

SQL vulnerability assessment helps you identify database vulnerabilities
SQL vulnerability assessment is an easy-to-configure service that can discover, track, and help you remediate potential database vulnerabilities.
Vulnerability assessment is part of Microsoft Defender for Azure SQL, which is a unified package for advanced SQL security capabilities.
Vulnerability assessment can be accessed and managed from each SQL database resource in the Azure portal.
What is SQL vulnerability assessment?
SQL vulnerability assessment is a service that provides visibility into your security state.
Vulnerability assessment includes actionable steps to resolve security issues and enhance your database security.
Vulnerability assessment is a scanning service built into Azure SQL Database.
The rules are based on Microsoft's best practices and focus on the security issues that present the biggest risks to your database and its valuable data.
They cover database-level issues and server-level security issues, like server firewall settings and server-level permissions.
Results of the scan include actionable steps to resolve each issue and provide customized remediation scripts where applicable.
You can customize an assessment report for your environment by setting an acceptable baseline.
So on and so forth.

That's great. How do I create a report that highlights all this wonderful work we have just accomplished?? You'd think that this type of activity would be prominent in the solution because we need to provide evidence that compliance and security are being maintained. Am I looking in the wrong place for reporting documentation? I have spent a couple of hours reading the documentation about Defender and how wonderful all the automation is.  Each time I think I may be on to something about how to create a report, I am redirected back to THIS DOCUMENT. The documentation is circular, and I go round and round without finding anything that educates me as to HOW TO PRODUCE A REPORT.
So I'm a bit frustrated, I admit. Where am I going wrong? Surely you must have some guidance that is useful in assisting us create compliance and security reports. Right??
Apologies for the rant, I Thank you for reading it, and I thank you for guidance you can provide as to where the reporting guidance is hidden.
Yours in frustration,
klaw
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 71dc027c-f37c-050f-c656-b37b8710bdeb
Version Independent ID: aa373cbb-f201-c71b-1ab2-2d22adc83290
Content: Scan your Azure SQL databases for vulnerabilities using Microsoft Defender for Cloud
Content Source: articles/defender-for-cloud/sql-azure-vulnerability-assessment-overview.md
Service: defender-for-cloud
GitHub Login: @bmansheim
Microsoft Alias: benmansheim

@klawrawkz
Thanks for your feedback! We will investigate and update as appropriate.

@bmansheim
Kindly take a look at this

@klawrawkz You can use the recommendations for vulnerabilities to see the vulnerabilities found by the vulnerability scan.
If you need help with that, please contact support.
#please-close


Add SLE HPC support for LSF

Adapted off https://github.com/Azure/cyclecloud-lsf/pull/39
@staer @mvrequa @ryanhamel @anhoward

Looks fine to me, but I'm not an authority here. Closed #39 since this PR contains and enhances the other.


Can you provide more details for the advanced settings Filter Subsites

The Filters section of the migration settings includes a property for Filter Subsites. Can you provide more details such as:

should this be the absolute or relative
does this automatically exclude all child sites?
how to separate multiple urls? Commas, semicolons?
should spaces and other special characters be url encoded?
is there a maximum length for this property

Similar feedback for the Filter lists and libraries and Filter content types.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: dad86450-d9ac-4cc4-fc75-63f8b285c298
Version Independent ID: d1e8c85a-b365-78c0-be1a-5a03ee082491
Content: Available settings in SharePoint Migration Tool - Migrate to SharePoint and OneDrive
Content Source: migration/SPMT-settings.md
Service: sharepoint-online
GitHub Login: @JoanneHendrickson
Microsoft Alias: jhendr

@spg-iwilson Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

@spg-iwilson thank you for reaching out. Please have another look at the article, I see it was updated a few days ago after your initial comment. Please let me know if you need further assistance.

The only information for Filter Subsites is the text "Enter any subsite names you want to exclude from migration". This doesn't cover any of points I raised in my comments

@spg-iwilson  Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.

@scanum Can this be re-opened? The documentation I'm seeing (https://learn.microsoft.com/en-us/sharepointmigration/spmt-settings) still doesn't provide any of the details that @spg-iwilson asked for. Thanks.


feat: Define redirect uris with env variables

We are planning to use Zitadel as an OpenID authentication provider for some of the Camptocamp services. Such a simple customization makes example server more convenient to create E2E tests. Using environment variables instead of command line arguments or config files for easier integration with Docker.

Can you please merge main into the PR branch? It seems maintainer push is disabled. Thanks.

Can you please merge main into the PR branch? It seems maintainer push is disabled. Thanks.

Done. Waiting for workflows


Can't install @mui-treasury/components

npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/@material-ui/core
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/@mui-treasury/components
npm ERR!   @mui-treasury/components@"*" from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR!
npm ERR! See C:\Users\tanshul\AppData\Local\npm-cache\eresolve-report.txt for a full report.
npm ERR! A complete log of this run can be found in:

We are moving away from npm packages, please check the new instruction https://mui-treasury.com/


Added ALT Tags for Product.rst

Added ALT Tag for the following:

Image Link
Image Link
Image Link

/check-cla

Hello @jaskiratsingh2000,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.


dispatch fails with jquery

I am testing redux action creators dispatch,which contains a jquery that hides the modal which calls this function. But my problem is the dispatch after the jquery is not detected by redux mock store dispatch. I am using jest to test my react application

I closed this because I found the solution.To solve the problem we have to add two lines of code in our test, which are:
global.JQuery = require('jquery');
require('bootstrap');


Remove spawn capability from everyone beside admin

Is there a way to make it so only admin can spawn it in

this script is meant for Vmenu servers that allow people to spawn cars.... so people can use /car and not have to go through vmenu


Update deps

IMPORTANT: Please do not create a Pull Request without creating an issue first.
Fixed issues (PRs without this will not be accepted)

Updates all dependencies

Changes

Update dependencies in package.json
Implement Apollo Server Express 3
Keep graphql-upload at v15 (v16 now uses ES Modules) 😒

PR Checklist

[ ] Does your PR contain relevant tests that pass?


WIP: Automated cherry pick of #11254 upstream release 1.0

Assuming #11254 will not need further changes

The doc is going to move, so the doc link in the message will need to be updated.
cc @davidopp

LGTM
doc/services-firewalls.md is the one remaining doc that I felt should move out of docs/ but couldn't figure out which subdir to put it in. So anyway, it's still at top-level, so this PR doesn't need to be rebased or anything.

@zmerlynn

LGTM. Checked the patches hadn't drifted before taking off the WIP.


Create a Help section

In my opinion, this should be as simple as an explanation and some links to the wiki and the web site.

It would be nice to provide the new html pages containing the documentation as part of our RPM


Mitaka

@szakeri
What issues does this address?
What's this change do?
Bringing mitaka branch up-to-date with liberty changes.
Where should the reviewer start?
Any background context?

retest this please

retest this please

@szakeri: this PR is ready to go. The smoke build succeeded, but it is duplicated in the GUI here because I added a new line 'Running smoke tests on PR', which did success. You can click on the 'Details' for that build and it will show the results.


Support VoiceOver

I think we have to support voice over. #128
There are two approach.

Use UIAccessibility.Notification.announcement
It does not moves focus on ToastView, but voice over read the text directly.

Use UIAccessibility.Notification.screenChanged
It moves focus on ToastView, and read text on `ToastView.
However, a user may lose the context in which she or he is using.

Therefore, I choose first approach.

@Coeur Thank you for your feedback.
I updated my code a17620a


Refactor observation storage: master list and read-only copies

Summary
As we move towards a more final-state layout for out data, as brought up by @pavel-ilin and expanded on by @njordhov here, we should aim to have a "master file" of all observations in order to easily manipulate, iterate through, and work with our data for all observations, as well as our pre-existing individual copies of each observation.
That way, when we want to access all, say, temperatures, we can read in our master observations.json list, and iterate through the JSON-ized version of that. Additionally, if we know we want a specific list, we can still access that particular observation just as easily.
Motivation
This added redundancy will allow for the easiest interaction with our observations that we can get.
Describe alternatives you've considered
Combining Blockstack API calls like getFile() and putFile() with React Blockstack hooks like useFile() and useFilesList()

I'm on this, as it will affect work on #338 which I have already been assigned

Consider keeping the aggregated "master file" in the Local Storage of the browser (rather than in Gaia), generated initially from the persistent content in Gaia then later kept updated with changes in the state.

This was brought up here in #457 - closing this issue as this new one has more up-to-date information


Enhancement Use IdGeneratorBuffer to get better performance for getNextId operation when create table, truncate table, add partition and so on

Proposed changes
Issue Number: close #11478
Problem summary
Describe your changes.
Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[x] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[ ] No
[x] No Need

Has document been added or modified:

[ ] Yes
[x] No
[ ] No Need

Does it need to update dependencies:

[ ] Yes
[x] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[x] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

Is there any performance tests?

Is there any performance tests?

the main cost is EditLog write, I use thread.sleep(2) to Simulate editlog write and here is the example for single thread
 MetaIdGenerator idGenerator = new MetaIdGenerator(0);
        int size = 50000;
        long[] idList = new long[size];
        long startTime = System.currentTimeMillis();
//        for (int i = 0; i < size; i++) {
//            idList[i] = idGenerator.getNextId();
//        }
        IdGeneratorBuffer idGeneratorBuffer = idGenerator.getIdGeneratorBuffer(size);
        for (int i = 0; i < size; ++i) {
            idList[i] = idGeneratorBuffer.getNextId();
        }
        System.out.println("cost " + (System.currentTimeMillis() - startTime));

cost is 123ms vs 4ms this example is for creating table on single thread


Discussion of API Naming Scheme

While looking at the public API as a whole, I have been wondering once again "Is there really not a pretty interface?". I personally like snake_case and have a slight aversion to camelCase. Furthermore, both the std and tpie library use this style, which makes Adiar's codebase consistent.
What I personally would hope for is something that would look like this.
if (adiar::bdd::is_true(f)) {
  adiar::bdd h = adiar::bdd::and(f,g);
}

Here the function and(f,g) is a static function of the bdd class. Yet, since and, or, and xor are keywords in C++ (due to historical reasons), this is a mere utopian dream of mine.
I hope this issue can foster a discussion for what would be the preferred naming scheme. I would then switch to the result of this conversation when releasing Adiar v2.0 or more likely v3.0.
Option 1: {prefix}_{nocase} (current)
if (adiar::bdd_istrue(f)) {
  adiar::bdd h = adiar::bdd_and(f,g);
}

Arguments:
This style matches the C and C++ interface of BuDDy and the C interface of Sylvan. Furthermore, it does not clash heavily with the use of snake_case in the rest of the codebase.
Informal Survey Results:
The people preferring this style usually refer to (1) the consistency to other implementations and (2) a better consistency with snake_case rather than the pleasantness of the style itself.
Option 2: {prefix}_{camelCase}
if (adiar::bdd_isTrue(f)) {
  adiar::bdd h = adiar::bdd_and(f,g);
}

Arguments:
This style is close to the current API while being slightly easier on the eyes. Yet, question is whether the rest of the codebase then needs to switch to a camelCase style to be consistent? If so, what should change and what should be kept - and why?
Informal Survey Results:
This seems to be the most popular one (not taking history into account).
Option 3: {Prefix}_{CamelCase}
if (Adiar::BDD::IsTrue(f)) {
  Adiar::BDD h = Adiar::BDD::And(f,g);
}

Arguments:
This style circumvents the keywords by capitalizing the first symbol. Yet, by switching to CamelCase on the function, the style propagates to the class name bdd (which either should be Bdd or BDD). Yet having the namespace still uncapitalized looks really weird, so the namespace also changes.
Furthermore, this truly calls for the entire codebase to change its naming scheme.
Informal Survey Results:
While not really something people have disliked, they do not seem to favour it either.
Option 4: Synonyms for and, or, and xor
if (adiar::bdd::is_true(f)) {
  adiar::bdd h = adiar::bdd::conj(f,g);
}

Maybe we can get around this entire issue by finding synonyms for the unfortunate keywords (without running into new ones)

and: conj, ...
or: disj, ...
xor: eor, exor, ...

Arguments:
This achieves the desired style at the cost of having odd choice for names of the three most common operators.

On email, Andrew Miner has sent me the following preference due to consistency:

(1) {prefix}_{nocase}
(3) {Prefix}::{CamelCase}

On email, Jaco van de Pol believes all three naming schemes would work. But, he has a slight preference for (2) {prefix}_{camelCase}.

On email, Anna Blume Jakobsen sent me the following preference list:

(2) {prefix}_{camelCase}
(1) {prefix}_{nocase}
(3) {Prefix}::{CamelCase}

Talking with Erik Funder Carstensen, he advocates for (1) to be consistent with BuDDy and Sylvan (i.e. not only should Adiar be consistent, but it should maintain an inter-library consistency)

On email Henrik Bærbak Christensen also emphasizes to be consistent with what is expected in a language. That is, he argues for (1) or (3).


Typo at https://docs.microsoft.com/en-us/azure/governance/management-groups/overview#moving-management-groups-and-subscriptions

Moving management groups and subscriptions
To move a management group or subscription to be a child of another management group, three rules need to be evaluated as true.
If you're doing the move action, you need:
Management group write and Role Assignment write permissions on the child subscription or management group.
Built-on role example Owner
Management group write access on the target parent management group.
Built-in role example: Owner, Contributor, Management Group Contributor
Management group write access on the existing parent management group.
Built-in role example: Owner, Contributor, Management Group Contributor

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

ID: 1527b80e-5270-b5f0-9034-747c6f92ae4e
Version Independent ID: 50b9882c-dcb0-ecc1-241b-1361ad2fff1e
Content: Organize your resources with management groups - Azure Governance - Azure governance
Content Source: articles/governance/management-groups/overview.md
Service: governance
GitHub Login: @rthorn17
Microsoft Alias: rithorn

@mbendana
Thanks for your feedback! We will investigate and update as appropriate.

@mbendana
I've created a PR to update the documentation and will now close out this issue. Please allow some time for the changes to reflect and if you don't see the document update, please let me know.
I will be more than happy to re-open this issue and look into what happened.
Thank you for your time and patience!


Fix typos on Demos.

Actually. On second thought, should the change be written in tests/dummy/app/templates/index.hbs ?

Yup, that would be the more appropriate place since the js-file gets overwritten based on that.
Thanks for your contribution though!

Alright thanks; I'll amend this PR shortly.

Can't change which branches are in this PR, so I'll be sending a new one instead!


Containers that produce no output produce an error instead of no output on ReadLogs

Kubernetes expects that an empty logfile exists if a container is running (maybe created?), even if there are no logs.
journal2cri only creates a logfile when an application first outputs data.
This will probably be fixed with the attach refactor in rkt.

This is not true anymore with https://github.com/kubernetes-incubator/rktlet/pull/147. Closing.


Update ru_RU.lang

Used AE2's official Russian localization to make closest translation.

Fixed a typo.

Thanks!


server crash pollitation step or smth idk

server crash pollitation step
crash-2024-01-28_02.19.49-server.txt

I fixed this like 3 days ago. Update to 1.4.3 of Brainier Bees

very cool


fix: dashboard: usage stats

Codecov Report
Patch and project coverage have no change.

Comparison is base (607d89e) 87.07% compared to head (7290260) 87.07%.

:exclamation: Current head 7290260 differs from pull request most recent head 8903e6a. Consider uploading reports for the commit 8903e6a to get more accurate results

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the GitHub App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2218   +/-   ##
=======================================
  Coverage   87.07%   87.07%
=======================================
  Files          85       85
  Lines        9314     9314
  Branches      494      494
=======================================
  Hits         8110     8110
  Misses       1204     1204

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Incorrect types sugestion with overload

TS Template added by @mjbvz
TypeScript Version: 3.8.0-dev.20200201
Search Terms

overload

Incorrect types sugestion with overload
https://gitlab.com/kamidere-laboratory/duat/tree/testy
overload is declarated in src/database/Controller.ts method create

VSCode Version: 1.41.1
OS Version: Windows_NT x64 10.0.18362

Steps to Reproduce:

Write Database.create(DatabaseType.Graphite, {}) // now we get Influx overload xD
Change DatabaseType.Graphite to DatabaseType.Sqlite // now we get Sqlite overload :D
Change back to DatabaseType.Graphite // now we still get Sqlite overload :(
https://youtu.be/a7FHxIItceA

Does this issue occur when all extensions are disabled?: Yes

interface a {
  a: string;
  b: string;
  c: {
    a:number;
    b:string;
  }
}

interface b {
  a: string;
  b: string;
  c: {
    a: number;
    b: string;
    c: string[];
  }
}

interface c {
  a: string;
  b: string;
}

enum d {
  "A" = 'a',
  "B" = 'b',
  "C" = 'c',
}

class testClass {
  constructor(arg1: d, arg2: a | b | c) {
    console.log(arg1, arg2)
  }
  static doSomething(arg1: d.A, arg2: a): testClass;
  static doSomething(arg1: d.B, arg2: b): testClass;
  static doSomething(arg1: d.C, arg2: c): testClass;
  static doSomething(arg1: d, arg2: a | b | c): testClass {
    return new this(arg1, arg2);
  }
}

testClass.doSomething(d.C, {});

Looks like the selectedItemIndex we get back from TS changes on the retrigger
First Trigger: (after typing (d.C,)
[Trace  - 19:04:35.548] <semantic> Sending request: signatureHelp (383). Response expected: yes. Current queue length: 0
Arguments: {
    "file": "/Users/matb/projects/san/index.ts",
    "line": 43,
    "offset": 29,
    "triggerReason": {
        "kind": "characterTyped",
        "triggerCharacter": ","
    }
}
[Trace  - 19:04:35.548] <syntax> Response received: updateOpen (235). Request took 1 ms. Success: true
Result: true
[Trace  - 19:04:35.552] <semantic> Response received: signatureHelp (383). Request took 5 ms. Success: true
Result: {
    "items": [
        ...
    ],
    "applicableSpan": {
        "start": {
            "line": 43,
            "offset": 25
        },
        "end": {
            "line": 43,
            "offset": 29
        }
    },
    "selectedItemIndex": 2,
    "argumentIndex": 1,
    "argumentCount": 2
}

Retriever (after typing {)
[Trace  - 19:02:44.608] <semantic> Sending request: signatureHelp (344). Response expected: yes. Current queue length: 0
Arguments: {
    "file": "/Users/matb/projects/san/index.ts",
    "line": 43,
    "offset": 31,
    "triggerReason": {
        "kind": "retrigger"
    }
}
[Trace  - 19:02:44.609] <syntax> Response received: updateOpen (207). Request took 1 ms. Success: true
Result: true
[Trace  - 19:02:44.614] <semantic> Response received: signatureHelp (344). Request took 6 ms. Success: true
Result: {
    "items": [
       ...
    ],
    "applicableSpan": {
        "start": {
            "line": 43,
            "offset": 25
        },
        "end": {
            "line": 43,
            "offset": 32
        }
    },
    "selectedItemIndex": 0,
    "argumentIndex": 1,
    "argumentCount": 2
}


Documentation

 Issue by parthas1
Thursday Oct 03, 2019 at 19:25 GMT
Originally opened as https://github.com/GazzolaLab/elastica-python/issues/1

Code needs better documentation, using Sphinx or other popular libraries.

 Comment by nmnaughton
Friday Mar 20, 2020 at 16:48 GMT

Bumping this. Along with Sphinx, we can use read the docs (https://docs.readthedocs.io/en/stable/intro/getting-started-with-sphinx.html)
We need to decide what branches we want to document. Are we documenting the old, slow python code where it is easier to understand? Or the current implementation with numba?

 Comment by tp5uiuc
Friday Dec 24, 2021 at 18:00 GMT

Bumping this again. The current version online here has no API documentation.
@skim0119 @armantekinalp do we add this to the milestones for the next quarter?

 Comment by skim0119
Monday Dec 27, 2021 at 00:36 GMT

@tp5uiuc Okay. We can revisit overall documentation next quarter, especially when some of us wants to organize the front-face of the project for portfolio/resume. We should take care of this along with #94.
@nmnaughton @tp5uiuc Please comment below if you have any suggestions to proceed this issue. We can revisit Sphinx as well.


More fixes for wrapping in :args

Codecov Report

Merging #4737 into master will decrease coverage by 1.27%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #4737      +/-   ##
==========================================
- Coverage   81.39%   80.12%   -1.28%
==========================================
  Files         115      111       -4
  Lines      145353   138277    -7076
==========================================
- Hits       118309   110790    -7519
- Misses      27044    27487     +443

Impacted Files
Coverage Δ

src/version.c
91.54% <100%> (-0.53%)
:arrow_down:

src/gui_gtk_x11.c
47.96% <0%> (-10.01%)
:arrow_down:

src/gui.c
56.69% <0%> (-7.02%)
:arrow_down:

src/os_unix.c
59.19% <0%> (-4.14%)
:arrow_down:

src/pty.c
56.25% <0%> (-3.75%)
:arrow_down:

src/libvterm/src/unicode.c
91.17% <0%> (-3.7%)
:arrow_down:

src/gui_gtk.c
27.9% <0%> (-3.56%)
:arrow_down:

src/if_xcmdsrv.c
82.64% <0%> (-3.36%)
:arrow_down:

src/buffer.c
81.75% <0%> (-3.02%)
:arrow_down:

src/beval.c
80.95% <0%> (-3.02%)
:arrow_down:

... and 106 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0aca293...a3c8586. Read the comment docs.


API(added):fantacy-cricket-api added

🎯 Title and Issue Number

Closes #<issue_no>

🗒 Checklist:

[] I have mentioned the issue number in my Pull Request.
[]  I have gone through rules of when PR will get merged.
[] I have commented my code, particularly in hard-to-understand areas
[] I have created a helpful and easy to understand README.md
[] I have followed proper file structure.
[] I have gone through CONTRIBUTING GUIDELINES.md.

Thank you for contributing!❤
Don't forget to show some love by ⭐ the repository!

Hey @Diyouf, Thank you for creating PR! We will review it super soon. 😇❣
As we continue to evolve and innovate, we encourage you to share your ideas, feedback, and code to help us push the boundaries of what's possible in the world of APIs🚀.
We hope you have followed CONTRIBUTING GUIDELINES.
Thank you for being an essential part of our community. Together, we'll keep building amazing things!
Don’t forget to ⭐ our repository!
Happy Coding!✨
Will See you soon❣

Inavlid PR, issue number not mentioned in PR.

@Diyouf ,I wanted to reach out and let you know that we have decided to close your pull request.🙇
In case of any issues, you can contact us on Discord.
Thank you! ❣
Happy Coding! ✨
Will See you soon❣


Copy 3.6 patches to 3.7

Closing. Superseded by:

https://github.com/python-cmake-buildsystem/python-cmake-buildsystem/pull/317


nixos/filesender: init

Description of changes
Packaging of Filesender, a file sharing software written in PHP.
This PR adds NixOS manual entry and NixOS test.
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

@drupol hello, can you check this one too?
filesender requires simplesalphp so i cherry-picked commits from master, but looks like git doesn't like that.
for some reason CI can't find branch.

Hey!
Actually you need to rebase your branch, not cherry pick stuff.

Hey!
Actually you need to rebase your branch, not cherry pick stuff.

How can I rebase branch properly? I'm a bit newbie to git :)

git checkout master
git reset --hard
git pull
git checkout filesender-packaging-filesender-module
git rebase master
fix the conflict(s)
git push --force-with-lease

If you don't know enough Git, I suggest that you read some tutorials before contributing to a open-source project, it's the first requirement.

Setting the PR in draft mode so it won't ping maintainers in case of issue. Feel free to remove it from draft only when everything is OK.

git checkout master
git reset --hard
git pull
git checkout filesender-packaging-filesender-module
git rebase master
fix the conflict(s)
git push --force-with-lease

If you don't know enough Git, I suggest that you read some tutorials before contributing to a open-source project, it's the first requirement.

I thought I need something more advanced than just rebasing my changes on top of master. Thanks for help anyway!

Hm, intresting.

selenium.common.exceptions.WebDriverException: Message: Unsupported platform/architecture combination: linux/aarch64

Hm, interesting.

selenium.common.exceptions.WebDriverException: Message: Unsupported platform/architecture combination: linux/aarch64

I will mark test as broken on AArch64.

Have you already marked it as broken?

Yep, on this line.

Thank you!


Add field to explicitly define Istio kind for config data

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: diemtvu
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS [diemtvu]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: diemtvu
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS [diemtvu]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@diemtvu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
7356ac02fa66537d2b57f710627d9dabcebd3f74
link
/test istio-integ-k8s-tests

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.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: diemtvu, yangminzhu
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS [diemtvu]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Performance

I've found the performance long way out from the quoted 100ms.
Even when using a local install a simple query takes 1.2 s
Wondering if this is an exception to the rule, and if it is do you have any pointers where to look.

Actually lite mode was faster, but still pretty slow

Thanks!

Could you please also try to access it directly?
$ time curl cht.sh/python/for+loop
#  Python for loop decrementing index
#
#  I would like to go over the range() function yet again through the
#  documentation as provided here: Python 3.4.1 Documentation for
#  range(start, stop[, step])
#  (https://docs.python.org/3/library/functions.htmlfunc-range)
#
#  As shown in the documentation above, you may enter three parameters
#  for the range function 'start', 'stop', and 'step', and in the end it
#  will give you an immutable sequence.
#
#  The 'start' parameter defines when the counter variable for your case
#  'i' should begin at. The 'end' parameter is essentially what the size
#  parameter does in your case. And as for what you want, being that you
#  wish to decrease the variable 'i' by 1 every loop, you can have the
#  parameter 'step' be -1 which signifies that on each iteration of the
#  for loop, the variable 'i' will go down by 1.
#
#  You can set the 'step' to be -2 or -4 as well, which would make the
#  for loop count 'i' down on every increment 2 down or 4 down
#  respectively.
#
#  Example:

for x in range(9, 3, -3):
    print(x)

#  Prints out: 9, 6. It starts at 9, ends at 3, and steps down by a
#  counter of 3. By the time it reaches 3, it will stop and hence why '3'
#  itself is not printed.
#
#  EDIT: Just noticed the fact that it appears you may want to decrease
#  the 'i' value in the for loop...? What I would do for that then is
#  simply have a while loop instead where you would have a variable
#  exposed that you can modify at your whim.

test = 0
size = 50
while (test < size)
    test += 1
    if (some condition):
        test -= 1

real    0m0.146s
user    0m0.030s
sys     0m0.012s

In my case it takes 140ms at the moment. Not 100 ms of course, but not far from it

I guess it was a network issue. What is the purpose of a local install if it still needs to connect to an external server?

Even local server needs to query external server for StackOverflow answers. For local script that connects to local server, it probably just a matter of time to port server calls to local library calls.

There are several situations where local installation could be beneficial.

In case, you don't need SO-based cheat sheets, all cheat sheets collections are available offline,
Additionally to that, you can use your own local cheat sheets collections.


Update bower?

Could you update the bower version to allow us to use the new fixes?

It's available, re-install it. The version is 1.8.0 but the version number is 1.7.1 (didn't update the number in bower.json but it's the right version).


Do like saberhq anchor-contrib then it works for vec and array

I don't know what i am doing but it works, inspired from @saberhq/anchor-contrib
https://github.com/Arrowana/saber-common/blob/b8f3825f8338807841cb6c3ab7a87efa5b4a9036/packages/anchor-contrib/src/index.ts#L115-L129
To test it, i hacked in EscrowAccount some more fields, until i got delicious typing going. Yes!
pub struct EscrowAccount {
    pub initializer_key: Pubkey,
    pub initializer_deposit_token_account: Pubkey,
    pub initializer_receive_token_account: Pubkey,
    pub initializer_amount: u64,
    pub taker_amount: u64,
    pub bla: [u8; 32],
    pub v: Vec<Pubkey>,
}

Before they would show unknown[] and unknown, not what one would expect,

LGTM, though the lint is failing.


Use of SymbolTables is inconsistent in GTC backends

The GTC backend passes have a pattern that simply takes the node.symtable_ from the top-level node and passes that as a keyword argument to all children visitor methods. This works when the IRs have a single symbol table, but in reality none of them do, because common.BlockStmt derives from SymbolTableTrailt.
When adding a For node that defines a local ScalarDecl for the counter, we are running into the problem that the gtir_dtype_resolver is failing because the symbol table used there (from the top-level) does not contain the for loop counter's ScalarDecl.

The best solution would be to make the visitor automatically add the symbol table to the keywords if it exists on the node. If the node has one but already gets one from higher up, it should pass on a chain map with it's own table at the end.


[$250] Subject of the room briefly displayed {"html": "subject here"} before it finished rendering

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number:
Reproducible in staging?: Needs Reproduction
Reproducible in production?: Needs Reproduction
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @coleaeason @jliexpensify
Slack conversation: https://expensify.slack.com/archives/C05LX9D6E07/p1726583603519899
Action Performed:

Go the social room using very slow internet connection

Expected Result:
Room details loads without any error
Actual Result:
Subject of the room briefly displayed {"html": "subject here"} before it finished rendering
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

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

Screenshots/Videos

Add any screenshot/video evidence

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~021838604064730634601
Upwork Job ID:<PHONE_NUMBER>730634601
Last Price Increase: 2024-09-24

Issue OwnerCurrent Issue Owner: @abdulrahuman5196

This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989

Jli is BZ and reported this issue so I'll unassign myself

Hmm this isn't an issue for me anymore - I had a period of slow internet while I was at a cafe yesterday and didn't see any problems. @coleaeason what about you - still seeing it?

Had a report of this issue over the weekend from DB

I just got this again, as I started this morning:

Precondition: Offline from ND for an extended period (e.g. 8 hours/overnight)

Open laptop and click on #social
Observe HTML and also notice no chats loaded since 2:38am AEST
Type message and refresh
Wait ~60 seconds
See the channel title reflect properly

Is this something you can look into @techievivek?

Since this keeps happening, let's add the EXTERNAL label and see if we can get anyone in the community to identify what is going on here

Proposal
Please re-state the problem that we are trying to solve in this issue.

Subject of the room briefly displayed {"html": "subject here"} before it finished rendering

What is the root cause of that problem?

Something, BE return the report's description is:

{"html":"Everything about nothing"}

but our function to get the report's description text is:
https://github.com/Expensify/App/blob/48e9c176b3111870ebe5e6193312334e32c249ea/src/libs/ReportUtils.ts#L4100-L4106
What changes do you think we should make in order to solve the problem?

We should handle the case report's description data is something like {html: ...} as well:

function getReportDescriptionText(report: OnyxEntry<Report>): string {
    if (!report?.description) {
        return '';
    }
    if (report?.description?.html) {
        return Parser.htmlToText(report?.description?.html);
    }
    return Parser.htmlToText(report?.description);

The same logic should be applied to other positions which display report's description as well.

What alternative solutions did you explore? (Optional)

Thanks for the proposal.

- Something, BE return the report's description is:
{"html":"Everything about nothing"}

Though we can handle this in the frontend, I think we can update the backend to be consistent here.

@techievivek Sure. I just wanted to share my investigation and suggested solution. If a backend fix is more appropriate, please proceed with that.

Hmm, I checked the backend, and it seems like there isn't much consistency in how this is handled. For example, here we extract the HTML part of the description: https://github.com/Expensify/Auth/blob/86e9858b23fdd6a3cf15932cf315a80616832083/auth/lib/Report.cpp#L9202, but in other places, we simply return the rnvp as it is, like here: https://github.com/Expensify/Auth/blob/86e9858b23fdd6a3cf15932cf315a80616832083/auth/command/PreviewReport.cpp#L33 and https://github.com/Expensify/Auth/blob/86e9858b23fdd6a3cf15932cf315a80616832083/auth/lib/ReportAction.cpp#L1805. I think the best solution would be to support both versions, as @truph01 suggested above.
CC @abdulrahuman5196, could you please review the above proposal?

Waiting on a review from @abdulrahuman5196

Checking now

@techievivek / @jliexpensify Does there need to be any condition for us to repro this issue?
I tried creating a new room and provided some bold title and some other formatted subject. But I am unable to repro this issue. Even tried with slow network. No luck.
Does it need to be in that same room? Or some other condition?

@abdulrahuman5196 After creating a new room, you can try sending a new message to the room and see if the bug appears.

Honestly, I am not sure - I have only seen this HTML issue for the #social channel. If I had to guess, it would be a channel with thousands of chats/threads? @techievivek what about you? Have you seen this pop up anywhere else or do you have any theories?

As @truph01 mentioned, this issue can be reproduced by sending a new message to a room. The backend includes the description with an html key in every report update for new actions, so that seems to be how it's triggered. I could have updated the backend to send only plain text in the description, but there are other similar cases where the description is sent as an object with the html key so it's better to just handle both the variant in frontend.

It feels like we should fix the bug in the back end, not create a weird front-end hack.

@quinthar Aaah, I totally agree with your point and suggested the same here https://github.com/Expensify/App/issues/49355#issuecomment-2373515129. However, after digging deeper into the codebase, I noticed that the issue is actually rooted in the DB structure itself. For example, reports like tasks store the description as a plain string, while for rooms with HTML content, it’s stored under a separate html key. Because of this inconsistency in the DB, our backend is already trying to handle it in multiple places as mentioned here https://github.com/Expensify/App/issues/49355#issuecomment-2376801009
Since fixing it in the backend alone won’t fully resolve the issue without making changes to the way the data is stored, I think a better short-term solution is to make the frontend flexible enough to support both formats. This way, we won’t have to worry about breaking other parts of the app, and it’ll be a lot safer and quicker to implement. Let me know what you think, thanks.

Well let's make changes to how the data is stored -- if we are storing it inconsistently in the database, let's fix that.  Why are we storing HTML content in a JSON object with a single field?  That feels like a bad design we should unwind.

Yeah, let's clean this from the root, I have started a discussion on the #eng-chat here https://expensify.slack.com/archives/C03TQ48KC/p1727928821339289 to understand the historical reason and also discuss the potential solution.

Not overdue. Seems an internal discussion going on here. https://github.com/Expensify/App/issues/49355#issuecomment-2390481233

I am working on an internal migration to update the description for all reports to be stored as JSON with an HTML key. In the meantime, we can support both variants so let's work on the frontend change as part of this GH. I have created a GH internally to track the migration, in that we have a checklist to ensure we remove this temporary fix once all clients have migrated to the new syntax, which we can do after a month or so since the description fields get overridden for every new action update and on OpenReport call.

@abdulrahuman5196 Can you review the proposal by @truph01 above?

Hi, Thank you for the update. The proposal seems reasonable to me, will do a little more testing and update in my morning.

Ok, great.

Also, when editing or setting a description, can we make sure that we’re sending a JSON string instead of a plain string to the backend? I’m adding checks on the backend, but I want to ensure we’re sending the correct data from the frontend as well.

@truph01 's proposal here https://github.com/Expensify/App/issues/49355#issuecomment-2373492257 looks good and works well.
🎀 👀 🎀
C+ Reviewed

Also, when editing or setting a description, can we make sure that we’re sending a JSON string instead of a plain string to the backend? I’m adding checks on the backend, but I want to ensure we’re sending the correct data from the frontend as well.

AFAIK, Currently we are sending api request with html code in case of formatted text and plain text in the other case. If we need to change that, kindly let us know on what would be the expected request structure? I think we wouldn't need separate proposal for this since @truph01 proposal already addresses the core issue, and since this could be a small change we can cover the same in PR.

FYI, I see some back and forth on reproducing above, just wanted to chime in that this is happening for me on web staging now. It went away after I navigated to a different chat then back.

AFAIK, Currently we are sending api request with html code in case of formatted text and plain text in the other case. If we need to change that, kindly let us know on what would be the expected request structure? I think we wouldn't need separate proposal for this since @truph01 proposal already addresses the core issue, and since this could be a small change we can cover the same in PR. Correct me if wrong.

In cases where we are currently sending a plain text description, we want to switch to sending a JSON object with an HTML key, where the value is the plain text: {"HTML": "plain text content"}. There's no need for a separate proposal; we can include this change as part of the upcoming PR.

@AndrewGable

FYI, I see some back and forth on reproducing above, just wanted to chime in that this is happening for me on web staging now. It went away after I navigated to a different chat then back.

Yeah, a new call to OpenReport fixes the issue since we send the room's description in the way frontend expects but once you receive a new action, it goes back to displaying in json.

@techievivek I created draft PR here.
I encounter an issue with the change log message: When I tried to update the report's description, I sent description: {"html":"test"} to BE via API UpdateRoomDescription based on comment. The response contains the change log:
<muted-text>set the room description to: {"html":"test"}</muted-text>

So in FE, it is displaying that message as:

@truph01 I am working on the backend changes for this and realized we can just send the description in plain text and then in backend I can add checks to confirm we insert it as json.

PR is ready

Bumping @abdulrahuman5196 for the checklist please

BugZero Checklist:

[x] [Contributor] Classify the bug:

Bug classification

Source of bug:

[ ] 1a. Result of the original design (eg. a case wasn't considered)
[ ] 1b. Mistake during implementation
[x] 1c. Backend bug
[ ] 1z. Other:

Where bug was reported:

[x] 2a. Reported on production
[ ] 2b. Reported on staging (deploy blocker)
[ ] 2c. Reported on a PR
[ ] 2z. Other:

Who reported the bug:

[ ] 3a. Expensify user
[ ] 3b. Expensify employee
[ ] 3c. Contributor
[x] 3d. QA
[ ] 3z. Other:

[x] [Contributor] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake.
Link to comment: This is bug caused by backend-Frontend incompatability. We don't have access to backend code to report this.

[x] [Contributor] If the regression was CRITICAL (e.g. interrupts a core flow) A discussion in #expensify-open-source has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner.
Link to discussion: N/A

[x] [Contributor] If it was decided to create a regression test for the bug, please propose the regression test steps using the template below to ensure the same bug will not reach production again.

Regression Test Proposal Template

[ ] [BugZero Assignee] Create a GH issue for creating/updating the regression test once above steps have been agreed upon.
Link to issue:

Regression Test Proposal
Precondition:

Test:

Go to any room
Update the room's description
Send any message
Verify the room's description displayed properly

Do we agree 👍 or 👎
@jliexpensify

Payment Summary

C: @truph01 $250 (owed via Upworks)
C+: @abdulrahuman5196 $250 (owed via ND payments)

Upwork job

$250 approved for @abdulrahuman5196


Fix documentation regarding uid and gid in legacy docker container

Description
Fix documentation regarding uid and gid in legacy docker container.
closes #7674
Self Check:

[x] Attached issue to pull request
[x] Changelog entry
[ ] Type annotations are present
[ ] Code is clear and sufficiently documented
[ ] No (preventable) type errors (check using make mypy or make mypy-diff)
[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[ ] Correct, in line with design
[x] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )
[ ] If this PR fixes a race condition in the test suite, also push the fix to the relevant stable branche(s) (see test-fixes for more info)

Processing this pull request

Merged into branches master in 74956e52f8979ff54bf7477d1124966f5c870a22

Processing #7812.

Processing #7813.


Fix tests

Rack 3 does not guarantee that the body is rewindable unless you add Rack::RewindableInput::Middleware
https://github.com/rack/rack/blob/main/UPGRADE-GUIDE.md#rackinput-is-no-longer-required-to-be-rewindable

Thanks @pbstriker38!


Fix Drone CI and Codacy

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


hot update issue.

Hot updating will work fine If i create a class extends from React.Component directly.
But i got two issues about hot updating when debugging at local without extends from React.Component directly.

hot updating will not execute if i extends from React.PureComponent. And even it will throw an exception of  'Maximum call stack size exceeded'. Screen also would not made any change.
hot updating will not execute if i extends from self-defined Class that extends from React.Component. It will give same error like above.
I am not sure whether it is related the mechanism of hot updating? or anything else ?

mark

Hey, thanks for reporting this issue!
It looks like your description is missing some necessary information. Can you please add all the details specified in the template? This is necessary for people to be able to understand and reproduce the issue being reported.
I am going to close this, but feel free to open a new issue with the additional information provided. Thanks!


Hide page control on last page

Added new option that hide page control on last page

Hey, thanks for proposal!
But it adds feature bloat without any advanced customization possible. Now this can be accomplished by page lifecycle blocks on any page, not only last, like:
    page4.onPageDidAppear = ^{
        intro.pageControl.hidden = YES;
    };
    page4.onPageDidDisappear = ^{
        intro.pageControl.hidden = NO;
    };

And implementing #165 will add ability to tweak it further and add any kind of animation.


Noisy log "http: proxy error: context canceled"

Set up:
Runtime: Node.js 20.x
Layer 1: Datadog-Node20-x - version 115 - arn:aws:lambda:us-east-1:464622532012:layer:Datadog-Node20-x:115
Layer 2: Datadog-Extension - 63 - arn:aws:lambda:us-east-1:464622532012:layer:Datadog-Extension:63

Cdk Version: 2.138
datadog-cdk-constructs-v2: ^1.12.0

Problem is I'm getting a sea of "http: proxy error: context canceled"
I traced this down to this log shows whenever a cloudWatch log stread/log group ends.
Each group starts with
{"DD_EXTENSION_FAILOVER_REASON":"appsec_enabled"}

Setting enableDatadogASM to false seemed to clear up that noisy log, but I would like to keep that enabled since it is for Threat Detection. I know it's just noise logs and can be ignored but it would be really nice to have them not show.
cdk:
this.datadog = new Datadog(this, 'Datadog', {
        nodeLayerVersion: 100,
        extensionLayerVersion: 55,
        site: 'datadoghq.com',
        apiKey: process.env.DD_API_KEY,
        enableDatadogTracing: true,
        enableDatadogASM: true,
        env: process.env.DD_ENV,
        service: process.env.DD_SERVICE,
        version: process.env.DD_VERSION
      });
      this.datadog.addLambdaFunctions([this]);

Hi @howdy10, this was fixed in v64. Can you upgrade and let us know if it goes away.


onnx-mxnet pip package prevents gpu use

Using pip to install mxnet-cu90 and onnx-mxnet causes pip to reinstall mxnet, removing GPU support. Changing the order the packages are installed does not address this issue.

@KellenSunderland

@jwfromm The above issue should not exist in the current mxnet v1.2 release. Please reopen the issue if it persists.
@marcoabreu please close this issue


Render fonts with Godot

For better quality and consistency, we could draw the font atlases ourselves using Godot's Font APIs.
Low priority.
https://github.com/ocornut/imgui/tree/master/misc/freetype
https://github.com/ocornut/imgui/blob/master/docs/FONTS.md#using-custom-colorful-icons
This will also let us support .woff2 files, and FontVariation, and SystemFont. We can (optionally) render the whole font atlas once and cache it as an image in the user data directory.

Something like this is probably fine:
        RID vprid = RenderingServer.ViewportCreate();
        RID canvasrid = RenderingServer.CanvasCreate();
        RID cirid = RenderingServer.CanvasItemCreate();

        RenderingServer.ViewportSetActive(vprid, true);
        RenderingServer.ViewportSetUpdateMode(vprid, RenderingServer.ViewportUpdateMode.Once);
        RenderingServer.ViewportSetSize(vprid, 256, 256);
        RenderingServer.ViewportAttachCanvas(vprid, canvasrid);
        RenderingServer.CanvasItemSetParent(cirid, canvasrid);

        RenderingServer.CanvasItemAddRect(cirid, new(0, 0, 256, 256), Color.Color8(0, 0, 0));

        var font = GD.Load<FontFile>("res://Hack-Regular.ttf");
        font.DrawChar(cirid, new(8, 16), 'X', 16, Color.Color8(255,255,255));
        // ...

        var mainvp = GetTree().CurrentScene.GetViewport().GetViewportRid();
        RenderingServer.ViewportSetActive(mainvp, false);
        RenderingServer.ForceDraw();
        RenderingServer.ViewportSetActive(mainvp, true);

        var img = RenderingServer.Texture2dGet(RenderingServer.ViewportGetTexture(vprid));
        // ...

        RenderingServer.FreeRid(cirid);
        RenderingServer.FreeRid(canvasrid);
        RenderingServer.FreeRid(vprid);

TextServer.FontGetGlyph* allows direct access to the font texture(s) and data we need, use that instead.

Probably not going to do this, since it would be complicated and hacky. Would rather focus on providing a good C++ interface, since most large Godot projects will probably have some C++ component, and then it's easy to just use the FreeType font builder.
So for now, "low priority" is downgraded to "not planned".


Feature/shorthand for border

I use this shorthand for border in CSS a lot.
border: 1px solid #000;

It would be really convenient to have a function like this.
...border('1px', 'solid', '#000')

That outputs something like this.
'borderWidth': '1px';
'borderStyle': 'solid';
'borderColor': '#000';

And/Or functions for each of the four directions like this.
...borderTop('1px', 'solid', '#000')
...borderRight('1px', 'solid', '#000')
...borderBottom('1px', 'solid', '#000')
...borderLeft('1px', 'solid', '#000')

This PR could help with this issue ( #264 )
If this change can be helpful, I could add it to docs as well.

@klshriharsha Since most CSS-In-JS libraries use standard CSS syntax is there a reason why you can't just do?
const div = styled.div`
  border: 2px solid #000;
`

I use a small tool called Fela (https://github.com/rofrischmann/fela) to write styles. It's configured to extract all the CSS in JS and create a CSS file. It does this by taking sets of common styles across the app and creating a CSS class for each set of common styles. But it doesn't try to do this with shorthand properties in CSS like border: 1px solid #000. So instead, if I use borderStyle, borderWidth and borderColor separately, it allows the tool to check if the same are used in other places as well and if so, create a class for it.

@klshriharsha We would need to see if there was more demand for this use case before including it in the library. It is a pretty narrow use case for us to add a new module to address. I'm going to add your description to the original issue and mark it as a request to gauge demand.


NTR: [mannose:sodium symporter activity]& [fructose:sodium symporter activity]

Hello,
I would like to annotate SLC5A10_ A0PJK1 transporter activity in GO and wonder if it’s reasonable to create specific GO terms for mannose and fructose substrates:
*
new GO: mannose:sodium symporter activity
child of
GO:0015578    mannose transmembrane transporter activity
GO:0005402    carbohydrate:cation symporter activity
GO:0015370    solute:sodium symporter activity
Definition: Enables the transfer of a solute or solutes from one side of a membrane to the other according to the reaction: D-mannose(out) + Na+(out) => D-mannose(in) + Na+(in).
RHEA:72908 D-mannose(out) + Na+(out) => D-mannose(in) + Na+(in)
Evidence:
PubMed=24573086, see Fig. 2 and 4.
PubMed=22212718, see Fig. 2.
PubMed=23451068, see Fig. 1 and 2c,d (D-mannose explicitly mentioned)
https://www.rhea-db.org/rhea/72908

new GO: fructose:sodium symporter activity
child of
GO:0005353    fructose transmembrane transporter activity
GO:0005402    carbohydrate:cation symporter activity
GO:0015370    solute:sodium symporter activity
RHEA:72915 D-fructose(out) + Na+(out) = D-fructose(in) + Na+(in)
Evidence:
PubMed=22212718, see Fig. 2.
PubMed=23451068, see Fig. 1 and 2c,d (D-Fuctose explicitly mentioned)
http://internal.rhea-db.org/rhea/72915
Definition: Enables the transfer of a solute or solutes from one side of a membrane to the other according to the reaction: D-fructose(out) + Na+(out) = D-fructose(in) + Na+(in).

Looking forward to your feedback.
Many thanks
Nevila Hyka-Nouspikel

+[Term]
+id: GO:0140929
+name: mannose:sodium symporter activity
+namespace: molecular_function
+def: "Enables the transfer of a solute or solutes from one side of a membrane to the other according to the reaction: D-mannose(out) + Na+(out) = D-mannose(in) + Na+(in)." [PMID:22212718, PMID:23451068, PMID:24573086]
+intersection_of: GO:0015293 ! symporter activity
+intersection_of: has_primary_input CHEBI:16024 ! D-mannose
+intersection_of: has_primary_input CHEBI:29101 ! sodium(1+)
+property_value: term_tracker_item https://github.com/geneontology/go-ontology/issues/24185 xsd:anyURI
+created_by: pg
+creation_date: 2022-10-20T14:15:09Z
+
+[Term]
+id: GO:0140930
+name: fructose:sodium symporter activity
+namespace: molecular_function
+def: "Enables the transfer of a solute or solutes from one side of a membrane to the other according to the reaction: D-fructose(out) + Na+(out) = D-fructose(in) + Na+(in)." [PMID:22212718, PMID:23451068]
+intersection_of: GO:0015293 ! symporter activity
+intersection_of: has_primary_input CHEBI:15824 ! D-fructose
+intersection_of: has_primary_input CHEBI:29101 ! sodium(1+)
+property_value: term_tracker_item https://github.com/geneontology/go-ontology/issues/24185 xsd:anyURI
+created_by: pg
+creation_date: 2022-10-20T14:15:18Z

Thanks a lot !


Node modules not getting installed in the custom path

Node modules not getting installed in the custom path i.e the directory specified in outputs.dir.
My build.gradle file:
plugins {
  id "com.moowork.node" version "0.12"
}

task installpackage(type: NpmTask) {
    args = ['install', "package-name"]
}

npm_install {
        outputs.dir '/Users/user/Documents/dummy/node_modules'

}

node {
  version = '5.4.0'
  npmVersion = '3.7.5'
  distBaseUrl = 'https://nodejs.org/dist'
  download = true
  workDir = file("${project.buildDir}/nodejs")
  nodeModulesDir = file("/Users/user/Documents/dummy/node_modules")
}

when i run gradle installpackage, the packages are installed under /users/user/node_modules and not in /Users/user/Documents/dummy/node_modules...

Which working dir are you running gradle from?

running gradle from project directory ie /Users/user/Documents/project

Thanks for the report. One more question. You are running OSX, right?

yes

@srs can you help me with this issue?

Yes, I will check it out and try to replicate it in my environment.

@srs i have a question is there any way to use this plugin in java servlet instead of a gradle task..

First off, the nodeModulesDir property probably is badly named. It is where the package.json and node_modules directory should. So in your code it should be written the following:
node {
  ...
  nodeModulesDir = file("/Users/user/Documents/dummy")
}

But even if you do that, then NpmTask will not use it. Only NpmInstallTask will. This is now fixed in issue #107 and will be available in version 0.13.


HOTR for Custom Data

Hello I am currently writing my master thesis in the field of HOI detection. I would like to use custom data for this. However, I still lack any clues on how to annotate this data.
Can you recommend a tool for this? I would also like to investigate HOTR in more detail in my thesis and write a paper about it. I would be very happy to receive a response.
Best regards

Me too. Attention.


fixed database service test

The trick is to assign the mock app object to the SQLAlchemy object (provided by flask-sqlalchemy). E.g.
from accounts.services import database

mock_app = mock.MagicMock()
mock_app.config = {
    'SQLALCHEMY_DATABASE_URI': DATABASE_URL,
    'SQLALCHEMY_TRACK_MODIFICATIONS': False,
    'TESTING': True
}
mock_app.extensions = {}
mock_app.root_path = ''

database.db.app = mock_app     # Here.
database.db.init_app(mock_app)

The reason is that SQLALchemy.app is set in its constructor, but not by the init_app method. When create_app is called, the get_app method is called; in a testing situation, the flask.current_app proxy will not be set, so if SQLALchemy.app is also not set you will encounter the RuntimeError that you were seeing.
I took out the flask_testing stuff. One of the drivers for the current architectural pattern was to make it possible to test components without extra packages. Let's see how far we can get without it?

I reverted my branch to the previous commit and it works now, except on my desktop, I still get the same error. I'll have to come back to this later.

Pull Request Test Coverage Report for Build 70

2 of 2 (100.0%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 84.055%

Totals

Change from base Build 68:
0.03%

Covered Lines:
427

Relevant Lines:
508

💛  - Coveralls


[NSE-931] Reuse partition vectors for arrow scan

What changes were proposed in this pull request?
Every time we read a arrow record batch from Parquet，we will use partitionValues to create partitionVectors and add them to record batch. Actually, the partition value won't change in the reading of one PartitionFile. We can create one partitionVectors for each PartitionFile and reuse it until task finished.
How was this patch tested?
unit tests.

@jackylee-ch can you please do a rebase?

@jackylee-ch can you please do a rebase?

I have fixed the conflict.


Update dependencies and README

Codecov Report

Merging #8 (da14c12) into master (bbc387a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master       #8   +/-   ##
=======================================
  Coverage   73.45%   73.45%
=======================================
  Files           2        2
  Lines         162      162
=======================================
  Hits          119      119
  Misses         23       23
  Partials       20       20

Flag
Coverage Δ

unittests
73.45% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bbc387a...da14c12. Read the comment docs.


Unable to get mock mode controller working for endpoints with paramters in path

Hello friends
I am trying to set up a mock API to work on my client side app independent of efforts from my server side team. And I want these mocks to be more meaningful instead of simple mock data as provided by swagger out of the box.
I am using swagger project start -m to start serving the apis from the default http://localhost:10010/. And I am serving some mock jsons from /api/mocks/users.js which exports multiple methods.
I am using correct values inside my swagger-spec for x-swagger-router-controller and operationId.
And I have got one path /users return the mock values for both GET and POST
However whenever I am trying to connect /users/{user-id} with my mock controller, its returning 404.
I am starting to think dynamic paths like /users/{user-id} cannot be handled by mock mode. Am I right to think so or is there a way to mock such paths?

I ran into same issue,
Not sure but this seems to be the issue,
if you replace dash(-) with underscore( _ ) in the path, that seems to work. Trying to find a way around this.


Update to PIXI v6 with individual package imports

Conversion work for #151
Here's a list of what I did to make the package work with PIXI v6 using the individual modules:

Had to update dts-bundle-generator dependency because PIXI's type only import type imports did not work with the old one.
Had to leave pixi.js as a dev dependency since it's needed for the examples to work.
Had to enable esModuleInterop tsc flag for @pixi/utils EventEmitter to work.
Updated the buildAmbientTypes.js script to work with multiple imports and verified that the new output is similar to the old one.
Tested that all examples under docs work and correctly report PIXI v6 in the debug console.
Also specifically tested that all the different containers in the performance test work too.
Got rid of a bunch of as any cast workarounds in the LinkedListContainer thanks to improved PIXI types.
Didn't touch the tests at all since in package.json the test script is a simple echo done? Maybe they're not implemented yet?

As for some breaking changes I dropped support for v4 for the shared Ticker and Texture.fromImage usage. Since v4 doesn't have the individual packages I'm not sure if it's possible to keep supporting v4 and v6 at the same time?
I didn't yet do anything with the LinkedListContainer renderWebGL, renderAdvancedWebGL and renderCanvas functions. I think they're only for v4 so they could probably be removed if the v4 support is dropped?
I also didn't touch README.md since I'm not sure what is going to happen with v4 support yet.
Feedback would be welcome.

Dropping support for v4 was expected - honestly I was expecting to drop support for v5, but if it still works for that then great!
You are correct that in LinkedListContainer, renderWebGL and renderAdvancedWebGL can be dropped as v4-only methods. renderCanvas is worth keeping as it is used with the canvas rendering plugin in v5/6.
What is here looks good, I think the only further thing to do is to update the manual tests in the /tests folder - replacing the v4 tests with v6 and changing the modules tests to use the individual modules instead of the bundle.

So I played around with the tests a bit which was quite interesting. Here's a summary of what I did with them in addition to removing the old v4 tests:
pixi-v5-iife
The v5 iife version worked pretty much out of the box just by enabling the esModuleInterop tsc setting.

pixi-v5-module
For the v5 module test I had to add one as any cast workaround for the emitterContainer. This is caused by pixi-particles using v6 type for Container so assigning a v5 Container with a different signature does not work. The underlying types work though since I think v6 has only made some previously hidden internals visible due to a different process for generating the type definitions.
This is a bit "hacky" but I suppose it's still worthwhile to keep v5 as some sort of backwards compatibility.

pixi-v6-module
I got the v6 module test working with importing the full pixi.js v6 package. I tried changing it to use the individual packages but that turned out to be a bigger challenge with the tsconfig path overrides, yarn workspace no-hoist configuration and the override-require implementation. I'm not sure whether it's worth converting the test to use the individual packages since the functionality of the code should remain the same - just with different imports.

pixi-v6-iife
The v6 iife version is where things get interesting: PIXI v6 no longer has ambient declarations so it's not really possible to make the types work simply by adding /// <reference ... to them. So to make the tests work I basically had to remove all PIXI and pixi-particles types from them. I think this test is still good to have since this is probably the way people without complex TypeScript build pipelines use it.
Since v6 no longer really supports the ambient declarations they could probably be completely removed from the package if v5 support is dropped.

@andrewstart what do you think about dropping the v5 support? The types don't really work so it's a bit of a hack to use it with TypeScript. Or should there be a mention in the readme that modules and types only work for v6, and v5 and v6 bundle don't have type support?
It might be cleaner if the package only supported v6 and the ambient types were dropped completely.

Dropping v5 support and ambient types would be fine. It would be nice to get the pixi-v6-module test working with the individual modules, just to make sure that we always work with the bare minimum of packages. The package management should be a bit easier with only one version to keep track of.

The v6 module test now works with the individual packages. I tested it so that you can delete node_modules/@pixi from the root dir and the test still works. For that to work I had to add some "dependencies of dependencies" also to the package.json. But at least it's very nicely isolated now so you could for example make copies of the test and test it with different minor versions of PIXI.
Also the v6 tests and ambient types are gone.

Seems I was a bit enthusiastic about removing pixi.js and accidentally broke the examples. They should work again now...

Any idea by when this will be done?

Soon-ish. I'm working on another breaking change and want to fold them into one update.


docker de-crufting (pt1)

I spent an hour or two this weekend trying to un-cruft the Docker infra to take a step towards having CI. Unfortunately in the time I had I wasn't able to get to a working state so we can consider this "part 1 of ???" (See https://github.com/icing/mod_tls/issues/2 for details).
Each commit in this series tries to describe the error it resolved. I recommend reviewing commit-by-commit.

Thanks, I merged this in. Can only improve things. Let's see if I find the time this week to add a github workflow for the tests.


AMBARI-24135. Stop of Knox Gateway fails after deleting Hive from clu…

…ster post upgrade. (dlysnichenko)
What changes were proposed in this pull request?

Deployed cluster with Ambari version: <IP_ADDRESS>-155 and HDP version: <IP_ADDRESS>-205
Ambari upgrade - <IP_ADDRESS>-709
Stack upgrade to <IP_ADDRESS>-1478
Delete Hive from the cluster
Stop All Services
Stop of Knox Gateway fails with below error:

Traceback (most recent call last):
File "/var/lib/ambari-agent/cache/stacks/HDP/3.0/services/KNOX/package/scripts/knox_gateway.py", line 215, in <module>
KnoxGateway().execute()
File "/usr/lib/ambari-agent/lib/resource_management/libraries/script/script.py", line 353, in execute
method(env)
File "/var/lib/ambari-agent/cache/stacks/HDP/3.0/services/KNOX/package/scripts/knox_gateway.py", line 152, in stop
import params
File "/var/lib/ambari-agent/cache/stacks/HDP/3.0/services/KNOX/package/scripts/params.py", line 27, in <module>
from params_linux import *
File "/var/lib/ambari-agent/cache/stacks/HDP/3.0/services/KNOX/package/scripts/params_linux.py", line 229, in <module>
hive_server_host = hive_server_hosts[0]
IndexError: list index out of range

How was this patch tested?
checked on live cluster, also ran unit tests

Test failure is irrelevant
Test Result (1 failure / +1)
org.apache.ambari.server.agent.TestHeartbeatHandler.testComponents


Fix tooltip non activated trading accounts

https://github.com/AugurProject/augur/issues/8399
No tooltip if user hasn't activated their trading account.

tooltip the same when account is activated.

@Mergifyio backport dev


build: set the minimum version of "urllib3" back to v1.15

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

7 of 7 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 83.044%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
5265

Relevant Lines:
6340

💛  - Coveralls


What is the purpose of the Dynamic store access?

Hi there,
I will just repeat my question. :) What is the purpose of the Dynamic store access?
Thank you for your answer!
Nobo

It allows you to use javascript to change the data that you retrieve from the store. I added it because I needed it when I was working a data table component.

Thank you @cklmercer  but I still don't understand how you would change the data when you're using a string? In the example the string 'user.name':
store: {
        name() {
            return 'user.name';
        }
    },

That's a very naive example, which I should update. In reality you might do something more like this..
store: {
  userId() {
    return someConditional ? 'user.username' : 'user.email';
  }
}

Oh my, I can see now!


docs: modify the format and add Golang rendering

What type of PR is this?
docs

What this PR does / why we need it (English/Chinese):
en: modify the format and add Golang rendering
zh: 修改格式并增加Go语言渲染

Which issue(s) this PR fixes:
none

LGTM


Add more dApps to Predefined List

Hi teams, please add the following dApps to the browser Predefined List. Thank you!

Multichain
https://app.multichain.org/#/router
Celer cBridge
https://cbridge.celer.network/
Wormhole Bridge
https://www.portalbridge.com/
Verse DEX
https://verse.bitcoin.com/#

Version test: v1.0.4(170)
Devices test: IP 11, MI 8 Lite
Validation passed.


NodeMCU-32s does not work using your arduino library

Serial output info:
rst:0xc (SW_CPU_RESET),boot:0x13 (SPI_FAST_FLASH_BOOT)
configsip: 0, SPIWP:0xee
clk_drv:0x00,q_drv:0x00,d_drv:0x00,cs0_drv:0x00,hd_drv:0x00,wp_drv:0x00
mode:DIO, clock div:1
load:0x3fff0010,len:4
load:0x3fff0014,len:588
load:0x40078000,len:0
load:0x40078000,len:10472
entry 0x40078a28
Starting BLE work!
Brownout detector was triggered
ets Jun  8 2016 00:22:57
rst:0xc (SW_CPU_RESET),boot:0x13 (SPI_FAST_FLASH_BOOT)
configsip: 0, SPIWP:0xee
clk_drv:0x00,q_drv:0x00,d_drv:0x00,cs0_drv:0x00,hd_drv:0x00,wp_drv:0x00
mode:DIO, clock div:1
load:0x3fff0010,len:4
load:0x3fff0014,len:588
load:0x40078000,len:0
load:0x40078000,len:10472
entry 0x40078a28
Starting BLE work!
Brownout detector was triggered
ets Jun  8 2016 00:22:57
rst:0xc (SW_CPU_RESET),boot:0x13 (SPI_FAST_FLASH_BOOT)
configsip: 0, SPIWP:0xee
clk_drv:0x00,q_drv:0x00,d_drv:0x00,cs0_drv:0x00,hd_drv:0x00,wp_drv:0x00
mode:DIO, clock div:1
load:0x3fff0010,len:4
load:0x3fff0014,len:588
load:0x40078000,len:0
load:0x40078000,len:10472
entry 0x40078a28
Starting BLE work!
Brownout detector was triggered
ets Jun  8 2016 00:22:57
rst:0xc (SW_CPU_RESET),boot:0x13 (SPI_FAST_FLASH_BOOT)
configsip: 0, SPIWP:0xee
clk_drv:0x00,q_drv:0x00,d_drv:0x00,cs0_drv:0x00,hd_drv:0x00,wp_drv:0x00
mode:DIO, clock div:1
load:0x3fff0010,len:4
load:0x3fff0014,len:588
load:0x40078000,len:0
load:0x40078000,len:10472
entry 0x40078a28
could you help me find what's wrong, thank you.

The notion of brownout detection seems to come from the ability for the ESP32 itself to detect a low voltage situation in its own operation.  How are you powering the device?  If USB, consider trying alternate USB outlets or different cabling.  There also appears to be some ESP32 specific settings that one can change to affect the level of brownout detection.
I have listed some forum posts at the end here that appear to provide some additional thoughts.  They seem show show brownout detection when the WiFi radio is initialized.  This would also be a clue to our puzzle here.  When we enable the BLE subsystem, although obviously not WiFi, they have a common denominator which is radio work.
Let's see if we can first resolve the issue through alternate power source/location/cable and see where we go from there.
See also:
https://esp32.com/viewtopic.php?f=2&t=2784
https://www.esp32.com/viewtopic.php?t=2462

Howdy my friend, I saw you closed the issue ... I hope that means something is now working.  Could you comment on what changed to help me help others better?


[feature request] join lines in selection with ctrl+j

For VSCode, you can select multiple lines and join them into one line with a shortcut. Is it possible for this plugin?

Yes that can be done. Tracked here: https://github.com/timhor/obsidian-editor-shortcuts/projects/1#card-80593623.
To clarify, you're talking about selecting multiple lines in one go before joining, right? As opposed to adding a new cursor to each line.

yes


chore: Update termtree

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.02%) to 29.38%

Files with Coverage Reduction
New Missed Lines
%

src/git/repo.rs
1
44.57%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
1650

Relevant Lines:
5616

💛  - Coveralls


Extra closing curly bracket (}) in the code (Module 05)

Module: 05
Lab/Demo: 05
Task: 1.3 (Data Operations)
Step: 03
Description of issue:
There’s an extra closing curly bracket (}) in the code snippet after retrieving the inspections, which is causing a syntax error.
Screenshot attached:

OK will fix

Thanks for the quick fix!


Updating selectors after delegate addition failed

Describe the bug
This plugin doesn't work in iOS 13.6, and show me some errors:

Updating selectors after delegate addition failed with: Error Domain=NSCocoaErrorDomain Code=4097 "connection to service on pid 368 named com.apple.commcenter.coretelephony.xpc"

Is this still an issue?
What plugin are you referring to?
Are are you trying to do?