Bi-directional output binding for PostgreSQL

There has been a few issues open now against Dapr for some form of a more granular state query capability (e.g. state filtering or SQL against state). Also, in the current implementation, Dapr doesn't allow for sharing of state between applications. There are some good reason for this (microservices, preventing key collision etc), and while there is an ongoing discussion WRT enabling state grouping to solve for that, I've put together a more targeted solution to help us validate these use-cases using binding.
This bi-directional output binding for PostgreSQL supports 3 operation types (kinds):

exec for DDL operations, like table creation, as well as INSERT, UPDATE, DELETE operations which return only metadata (e.g. number of affected rows)
query for SELECT operations which returns the metadata as well as data in a form of array of rows
close for explicitly closing the DB connection and returning it to the pool

Config
Binding expects only one required parameter in the init metadata url. The URL can include any number of connection and connection pool configurations (e.g. connect_timeout=5)
apiVersion: dapr.io/v1alpha1
kind: Component
metadata:
  name: mydb
spec:
  type: bindings.postgres
  metadata:
  - name: url
    value: postgres://daprtest@localhost:5432/daprtest?application_name=test&connect_timeout=5

Caveats

No parameterized queries, all SQL needs to be passed as string in metadata
Metadata vs Data on bindings.InvokeRequest to to simplify the use and not require users to serialize their intent on each operation
Resulting bindings.InvokeResponse includes metadata with things like duration of query or number of rows affected. Only in case of a query operation is the response Data populated with an array of rows returned from DB
Binding uses connection pool. To explicitly close the connection and return it to the pool users need to use the close operation type
Binding itself doesn't prevent SQL injection, like with any DB-based application, validate the input before executing query.

Checklist

[x] Code compiles correctly
[x] Created/updated tests
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

Addresses dapr/#2088


Failed to generate a build because of the jar is unavailable

Library information:

Version: 1.1.0 - 2.0.0
View or Compose module: [e.g. view]

Describe the bug**

What went wrong:
Execution failed for task ':app:dataBindingMergeDependencyArtifactsProductionDebug'.

Could not resolve all files for configuration ':app:productionDebugRuntimeClasspath'.
Failed to transform CalendarView-1.1.0.aar (com.github.kizitonwose:CalendarView:1.1.0) to match attributes {artifactType=android-databinding, org.gradle.category=library, org.gradle.dependency.bundling=external, org.gradle.libraryelements=aar, org.gradle.status=release, org.gradle.usage=java-runtime}.
Could not find CalendarView-1.1.0.aar (com.github.kizitonwose:CalendarView:1.1.0).
Searched in the following locations:
https://jitpack.io/com/github/kizitonwose/CalendarView/1.1.0/CalendarView-1.1.0.aar

To Reproduce (if applicable)
Steps to reproduce the behavior:
1- Trying to generate a build in Bitrise
Expected behavior (if applicable)
A clear and concise description of what you expected to happen.
Screenshots? (if applicable)
If applicable, add screenshots or screen recordings to help explain the problem.
Additional information
Add any other information about the bug here.

+1 to this problem. cc @kizitonwose
I've also opened an issue on the Jitpack side in case its a problem on their end https://github.com/jitpack/jitpack.io/issues/6400

For those who are struggling with the same problem, the artifact ID seems to have changed from
CalendarView -> Calendar
Try using com.github.kizitonwose:Calendar:1.1.0 instead.

Yep, it's a JitPack issue. Seems their repo has become somewhat unreliable recently. Someone reported a missing artifact a few weeks ago #555

Closing this since there's nothing to do on my side.


Libscotch7.0.5

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I was trying to look for recipes to lint for you, but it appears we have a merge conflict. Please try to merge or rebase with the base branch to resolve this conflict.
Please ping the 'conda-forge/core' team (using the @ notation in a comment) if you believe this is a bug.

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.

@Kabbone sry for pinging you here but I do not know how to debug the test errors. Maybe you have an idea.

you build scotch INT32 and PaStiX is default INT64, if you want INT32 you need to pass a build flag "PASTIX_INT64=OFF"

Thanks foryour response @Kabbone. I think the scotch builds have int64 support. Also this feedstock seems to build fine. What is failing now are the ctests.

You see in the build of pastix that scotch 8 Byte test of cmake is failing and 4 Byte successful. Or did I look at the wrong build?
On November 18, 2024 10:58:15 PM GMT+01:00, lorenz @.***> wrote:

Thanks foryour response @Kabbone. I think the scotch builds have int64 support. Also this feedstock seems to build fine. What is failing now are the ctests.
--
Reply to this email directly or view it on GitHub:
https://github.com/conda-forge/pastix-feedstock/pull/5#issuecomment-2484216486
You are receiving this because you were mentioned.
Message ID: @.***>

Ah thanks, yes true, so it's still the same issue.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe/meta.yaml:

ℹ The recipe is not parsable by parser conda-souschef (grayskull). Your recipe may not receive automatic updates and/or may not be compatible with conda-forge's infrastructure. Please check the logs for more information and ensure your recipe can be parsed.

This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/12097411640. Examine the logs at this URL for more detail.


Not search open MarkdownLivePreview:open preview

I install MarkdownLivePreview - Packages - Package Control.but when Enter cmd+shift+p .
I didn't find MarkdownLivePreview:open preview, but I found a Open setting.Why???

I'm experiencing the same issue.

https://github.com/math2001/MarkdownLivePreview/issues/98 does your file have the right extension (.md)?

i have same issue and yes - file is *.md

Like others, I'm facing the same issue.


tests: fix Array test

The Array/removeNonUnique test is not working with an old runtime.
rdar://problem/58471030

@swift-ci smoke test and merge


Added log formatter.

r? @ustulation
(maidsafe_highfive has picked a reviewer for you, use r? to override)


Add Materialized helper with implicit Serdes in Kafka Streams Scala

Provide a tiny helper for creating Materialized with implicit Serdes.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

I don't believe there's much purpose to abstract Materialized to help with implicit SerDes now that the internal changes to propagate the SerDes for each of these aggregation operators has been completed in #4919.  As @guozhangwang mentioned in that PR:

This is a meta explanation about the serde inheritance across multiple internal impl classes:
reduce: inherit the key and value serdes from the parent XXImpl class.
count: inherit the key serdes, enforce setting the Serdes.Long() for value serdes.
aggregate: inherit the key serdes, do not set for value serdes internally (line 92 here is for this case).

Indeed this doesn't bring too much value so I removed the add of the Materialized helper and left only some refactorings

Thanks @joan38 ! Merged to trunk.

Thanks @guozhangwang


Record cheatcodes special parameters

Cheatcodes like pranks, expects and boradcasts appear on special functions such as call and end_call, and have related structs structs such as Cheatcodes, Prank, ExpectRevert, etc.
We should include these structures/functionalities when encoding their related cheatcodes. See that #1349 is already going in that direction.

I'm going to close this issue for now. PRs like #1459 , #1448, #1436 add new subconfigurations for expected reverts, calls and events, to the <foundry/> K configuration, which is an extension of the <kevm/> configuration.


interface PaymentMethodData is not correct. paymentMethod does not exists on type PaymentMethodData

Describe the bug
Hi, I upgraded to a new version 1.2.0, and found out that the interface PaymentMethodData is not correct.
When I call onSubmit should return data: PaymentMethodData
export interface PaymentMethodData {   paymentMethod: {     type: string;     [key: string]: any;     checkoutAttemptId?: string;   };  .... }
but got instead
export interface PaymentMethodData {   paymentData: {     type: string;     [key: string]: any;     checkoutAttemptId?: string;   };  .... }
Can you fix the interface definition to be the right shape with paymentData or did I do something wrong?
Tested on Android
"react-native": "0.73.2",
"@adyen/react-native": "1.2.0",

Hey @JBlazej
The source code is correct.
Could you provide log and check what is inside this object ?
console.log(JSON.stringify(paymentData, null, ' '))

const onSubmit = async (
    payload: Omit<PaymentMethodData, 'paymentMethod'> & {paymentData: PaymentData},
    _?: AdyenActionComponent
  ) => {
    console.log(JSON.stringify(payload, null, ' '))
  }
// console.log result
{
 "paymentData": {
  "storePaymentMethod": false,
  "browserInfo": {
   "userAgent": "Mozilla/5.0 (iPhone; CPU iPhone OS 17_0 like Mac OS X) AppleWebKit/605.1.15 (KHTML, like Gecko) Mobile/15E148"
  },
  "paymentMethod": {
   "encryptedExpiryYear": "eyJhbGciOiJSU0EtT0FFUC0yNTYiLCJ2ZXJzaW9uIjoiMSIsImVuYyI6IkEyNTZDQkMtSFM1MTIifQ.FoqS_Mm8SPO9vQ5QYHzNSlHIh2i6onF0pFXJlzLG5NPkVpUPgVTC4qRrrvA2xfa4o8neehELctVqE-at7l4VhcyGUB_6TP748aObygPKC1wYk2a_-TCKrTdgC8_II9NNbqHwbBCoWmindHvkTkzOpljMcygozeck4wfoCRLGQgLP5C27Lk8efjnquMH_Rsy6h9SwYO4vNwyn_3KnjIVF94ua3VWZQraxeobKZxGOdTwGW240AWPhgKuX_hDJfSUdlm5_5IijU6nCIKMbFNiJaa55YC1eWgGyZ4l5XckLy2sVRtk2AnGHHqtTng69c0whbH1bBgslqM5IlAVN05cEFQ.YOzgPvXISMbdptk88NhaHw.xHaiWfErcNYtW-U7nStXHyEp1Sdnhwfgu8msEiAxh0SkzXkU6d7jmwaarWV0vB2sjYuL8bX8xizCeVM9LxGdvQ.Uf6qPUuhzR4yNE-RtbvQhfnw7h1NBk-aQjsktVwl6iM",
   "encryptedSecurityCode": "eyJhbGciOiJSU0EtT0FFUC0yNTYiLCJ2ZXJzaW9uIjoiMSIsImVuYyI6IkEyNTZDQkMtSFM1MTIifQ.Fb1e5GTXHANt0NnEEVm_y3ZVfwolfCj6sbCGxY10tmI449J2aPsOFwGuMybXM2HH3rch3u4gVqGaGSai7b0sFQbwi5Jeo1zWi5rajktVx5YSTpgTc8m8rykHit1LpUyIGYqJGb3Sh9fr_RKacl3Ep-7vYM6MQmGgDainkusJaohuq7zp2QcbjvwWS6vCZdU9GvLFb1R8w5AaM0xsDCE4FdJc7-Lef2ySNjcTPtNH2ELj98q-W6mWFwCZy3r_SmfwnaR_Uv2KEkf_XJs-x7QG3YqtrKdcT5pgH1RLcrbAzQSefamTgLH-3CrwmjsxnHkXaZnc5BTCVBqo0497qmK6Lw.Xk1ceBETvinDmOmREwpR_Q.XlZUNZ7Vh_KOs5BJ_PBj-vAVrRvdAL9YN1LdAp4lsQ5c8S4OthMIPc__CTV17QylEGP_DD6ybrZEXcgvEIvVwg.xCcIXo924BGKYToswu-iwFiz-dpzi0iGlwc2J1gA76c",
   "encryptedCardNumber": "eyJhbGciOiJSU0EtT0FFUC0yNTYiLCJ2ZXJzaW9uIjoiMSIsImVuYyI6IkEyNTZDQkMtSFM1MTIifQ.THVqvdYj86XUnZMKIQpGeBvxTksYBZgPDs8WhyEU-EGUCLYUxOG7CVtk_QKmAF7PF-ppiicenuHwvUgJmGA2VkVG87znF6YM4dkIJlgAEFGeO6-D9QijfTWDUJBD9hSwxAAUwvi2pM8SqngFkbzWCBC9LofjONhbAJ83JeStNeGxWPIiGnYLaFGl5DHv-XHcdIrmNV8DELHSnLCBQK2lxs7A_cDOpTBbjCkU69YXdQywzjiQPmn2RL7ubKK3AGPIU8nI__hjDiPlxQFDeHZYnhsgSl6v5omsCkf82HjPFPyRAMaqr3zdnALhOtpsgbaRqfYN_TUbvjXm9pWNeaCBeA.QqxmkJenIZkwEprNif9u5Q.ZwPKDpTns_lrQPakthrwTDmvT2Fm3qTDrMI_QaVYA1D8WdLYzLB1QicbBlJSoPg4IVHAWOM7dRjp_TpghE7lcDHP8Tkr63PmaTVWuKexxNs.IHTEdC8w3xgYiniRXXambbgjXMQVkQ6kmw2sj_IgxJw",
   "brand": "mc",
   "threeDS2SdkVersion": "2.3.3",
   "type": "scheme",
   "encryptedExpiryMonth": "eyJhbGciOiJSU0EtT0FFUC0yNTYiLCJ2ZXJzaW9uIjoiMSIsImVuYyI6IkEyNTZDQkMtSFM1MTIifQ.tGowKPGSYcSYmXJkHbm_7sMkzXQUdfrDYu0Ydn0hX_RAuKvO8JqkcPeQasoPpprGt1qJimm6thKLQhQ0vRrjOynNgKWC5iBFudSR3QPKB3JMsaU5pApaa0FRVw_ouxXTDLIOSDG45tABVnNfs2sQHmWSCOll55EZEtzd__q9240-kX7CgKGHiU60AH9f_lSCJx0lyQLVe-Jvz4-L3mNyWbTy3JQ9yJNL8WiAvQDT3qyFYDlRMzwFWVE1CXN_fOQzQODkcd2anBW1j4KvpIvEmRHI4azuHoaenhDpl6z0-FjKtUVS6P85hGoJUqF05VPASk1Vd8goS73mV75vqLr3Ww.7Hx8nMIk4r_Y20WjGnplHw.dLxd5uopnEZm7c5CMz_AGAixBr6R-sAUkWS-oHBAZihYjJmzBshn7irGD07umcU7iSnsBQqwmUVibD_XtcHi4Q.mz0aO21rWD_8cidpOV4v06z_glPxkUkU9MfsPac4x54"
  }
 },
 "extra": null
}

I am using AdyenDropIn.open(paymentMethods, newConfig) with combiniation NativeEventEmitter() and emitter.addListener(Event.onSubmit, event => onSubmit(event))
Payment method is nested in paymentData object

Hey @JBlazej

I am using AdyenDropIn

that is the reason 😅
Raw payload from native event is a bit more complicated - it consist of paymentData and extra properties.
Here paymentData is PaymentMethodData  and extra - any additional from "express-payments" like ApplePay and GooglePay.

Make sure to set redirectUrl from paymentData:  https://github.com/Adyen/adyen-react-native/blob/develop/src/AdyenCheckoutContext.tsx#L127-L141.

Hey @gunnartorfis
Thanks for feedback!
Indeed, now I see that reference to PaymentMethodData is missing on src/AdyenCheckoutContext.tsx.
Weird coincidence that VS Code can still resolve it from "Window APIs" 🤯

Hey @descorp
Any idea when BETA 2 will be released?
To give context, we are essentially just about to release our Adyen integration. We don't feel very comfortable releasing it since it depends on a beta version of this package, and getting the fixes from beta 1 -> beta 2 out before releasing would at least be much better.

Released 2.0.0 BETA 2

Hey @descorp this has indeed been fixed for onSubmit.
However, the onAdditionalDetails is still incorrectly typed - the data I receive in the callback is defined here: https://github.com/Adyen/adyen-react-native/issues/352#issuecomment-1921316469 - and it does not include a paymentMethods object as the onSubmit does.

The same goes for the standalone action handling. The object I receive has a "details" key, but the type suggests a "data" key.

Thanks @gunnartorfis 💚
Indeed! We will look into that..

#373 PR


Bindings Contribution Strategy - Python

The contribution workflow for Python bindings seem cumbersome in terms of local implementation and testing. It is possible through a hacky method of tweaking the func core tools path to a python deps location where new rich type binding needs to be copied.
Need  a cleaner approach.
Python Functions Binding Contribution.docx

Thanks for compiling this @priyaananthasankar!

I’m working on a way to make this process easier.

@elprans : I have raised PR's for the Kafka trigger/bindings in both azure-functions-python-worker and library repos. Can this be merged soon?

@elprans, can you provide details on how the entry point will be used once it is loaded? For example, is it sufficient to just include import kafka at the top of a function and then use the type annotation kafka.KafkaEvent or is there something else that needs to happen? Currently, the wiki has information on what we can do to register the binding but not how we use it.

Nothing special needs to be done. Just import and use.

I am not sure if there is any action item here. Closing this out.


WIP: Added Support to preview endpoints to list branches or pull requests …

…for a commit
Fixes #1151

Does the "WIP" in the PR title means that it is not ready to be reviewed, or that it doesn't fully fix #1151 ?
If the former, it would be better for the maintainers if you keep developing in your own fork until you are ready for a review, and then create the PR to the main repo.
If the latter, then the description should say "Helps #1151." instead of "Fixes #1151".
Either way, please don't put "WIP" in the PR name, as "work in progress" is obviously confusing.
Thanks.

@vaibhavsingh97 - can you please give a status update on this PR?

Sorry, @gmlewis for leaving this PR in middle like this.

please don't put "WIP" in the PR name, as "work in progress" is obviously confusing.

Sure, I will keep in mind :)
I am closing the PR, since @sks444 already opened the PR


Support AI Backend based ratelimit

Problem
We are introducing AI APIs to apk. In the AI usecases its a common usecase to limit the requests based on the token counts(prompt, completion, total).
Solution
Add a feature to support ratelimiting AI requests based on the token count.
Affected Component
Adapter
Version
1.2.0
Implementation
No response
Related Issues
No response
Suggested Labels
No response

Feature added using this PR: https://github.com/wso2/apk/issues/2479


Fix local command execution issue closes #494

This PR will fix a command execution issue in the extension.
execa version 1.0 used in this extension searches the local path first when trying to find the php executable. By this a crafted project can override the php binary and execute arbitrary code.

@felixfbecker is there anything more I could do to help getting this merged and the command execution issue resolved?


Decentralize message type generation

We want to support many different message type, some from the Cosmos SDK and some from custom blockchains that are not standardized. The user of CosmJS should have the most common message types ready to be used shipped with this library. But it should also be possible to add new message types from the caller code.
Some thoughts that might help getting there:

Protojs allow dynamically adding .proto files: https://github.com/protobufjs/protobuf.js/blob/master/README.md#using-proto-files
Maybe we can utilize JSON descriptors or reflection to dynamically add types
The library @node-a-team/ts-amino uses decorators to add encoding information to TypeScript types as shown here: https://github.com/chainapsis/cosmosjs/blob/master/src/common/stdTx.ts. Maybe we can use decorators as well to start with a TypeScript class and annotate it with all the necessary runtime information, like:

@CosmosMessage("cosmos.bank.MsgSend") // adds this type to a registry using the string as the `type_url`
export class MsgSend {
  @Field.Bytes(1 /* protobuf field number goes here*/)
  public fromAddress: Uint8Array;

  @Field.Bytes(2, { fieldName: "to_address" /* runtime field name for protobuf; probably irrelevant since not encoded into the proto bytes*/  })
  public toAddress: Uint8Array;

  @Field.Repeated(3, { type: Type.Defined }) // don't know what this does but if it worked for Amino, it can probably be adapted
  public amount: Coin[];
}

Decorators look pretty nice but are currently only a stage 2 proposal for JS and an experimental feature in TS.

https://github.com/tc39/proposal-decorators
https://www.typescriptlang.org/docs/handbook/decorators.html

Good table of tradeoffs here:
https://github.com/protobufjs/protobuf.js/blob/7bacfc8/README.md#reflection-vs-static-code

Decorators look pretty nice but are currently only a stage 2 proposal for JS and an experimental feature in TS.

Could you check the browser support for them? The Angular framework uses them for many years now, starting from Angular 2, the first version that uses TypeScript released in 2016. Maybe there is a way to transpile them into older JavaScript?

Doesn't look good:
https://kangax.github.io/compat-table/esnext/#test-Class_and_Property_Decorators_a_href=_https://github.com/wycats/javascript-decorators_class_decorators_/a

Ah, the support is there for "Type-Script + core-js 3", which is the combination socumented in the Angular browser support, where they use the term "ES2015 polyfill" for core-js 3.
I think it is feasable to require the combination Type-Script + core-js 3 when creating custom message types in the application. And for the types shipped with the library we don't use decorators.

Closed in #221


updates

Yes. Can you please add this in alpha release? This one was necessary to
integrate the updates in cell generator.
Thanks
Kunal
On Wed, Jul 17, 2019 at 8:05 AM Steve Burns<EMAIL_ADDRESS>wrote:

@stevenmburns approved this pull request.
Looks good. Do you want this in the alpha release? (I can redo it.)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ALIGN-analoglayout/ALIGN-public/pull/78?email_source=notifications&email_token=AJUB53KFC7B6RZUDNGIU6C3P74KJVA5CNFSM4IEQC5RKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB6WTYQI#pullrequestreview-263011393,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJUB53NOEJWTXG4TAA464H3P74KJVANCNFSM4IEQC5RA
.


Add AAD Integration repo

Do you need to add entries to artifacts.props as well to account for the new package?

This was merged already.

Yes. Thanks!


Integration for Selenium

Currently when integrating with selenium, sentry produces httplib breadcrumbs on every browser interaction, but not much detail is revealed about the action being executed.
I think it would be nice to show things link the URL when we perform a GET operation, or the element selector when we're reading or writing to the DOM.
Does anyone else have interest in this feature ? I could help implementing it.

Hello @eduardovra
Thanks for writing in. As it looks, there is not that much demand for this feature. Sorry.
I will close this issue now.
If someone is coming to this issue and is also interested in this feature, please open a new issue!


build: add azure pipelines

:tada: This PR is included in version 0.323.0 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Documentation improvements

There are some mistakes in the documentation with relations english words. If you prefer to fix them I can show to you or I can do it with pull request.
Congrats to project  :crab:

Thanks for fixing those typos :)


Problem: some client only support fetching individual index in store

Solution: remove trait Storage, add RequireAccountStorage and make
the original HashMapStorage emit that error

Coverage decreased (-1.3%) to 84.637% when pulling 2f56b399e2d4ed342a7f72e059f083b31ee187ea on sorpaas:master into 819d9e8adc6953ccaad50b5d90a376fc25320aad on ethereumproject:master.

Coverage decreased (-1.3%) to 84.637% when pulling 2f56b399e2d4ed342a7f72e059f083b31ee187ea on sorpaas:master into 819d9e8adc6953ccaad50b5d90a376fc25320aad on ethereumproject:master.

Coverage decreased (-1.3%) to 84.637% when pulling 2f56b399e2d4ed342a7f72e059f083b31ee187ea on sorpaas:master into 819d9e8adc6953ccaad50b5d90a376fc25320aad on ethereumproject:master.

Coverage decreased (-1.3%) to 84.637% when pulling 2f56b399e2d4ed342a7f72e059f083b31ee187ea on sorpaas:master into 819d9e8adc6953ccaad50b5d90a376fc25320aad on ethereumproject:master.

Coverage decreased (-1.3%) to 84.637% when pulling 2f56b399e2d4ed342a7f72e059f083b31ee187ea on sorpaas:master into 819d9e8adc6953ccaad50b5d90a376fc25320aad on ethereumproject:master.

Coverage decreased (-1.3%) to 84.637% when pulling 2f56b399e2d4ed342a7f72e059f083b31ee187ea on sorpaas:master into 819d9e8adc6953ccaad50b5d90a376fc25320aad on ethereumproject:master.

Coverage decreased (-1.3%) to 84.637% when pulling 2f56b399e2d4ed342a7f72e059f083b31ee187ea on sorpaas:master into 819d9e8adc6953ccaad50b5d90a376fc25320aad on ethereumproject:master.

Coverage decreased (-1.3%) to 84.637% when pulling 2f56b399e2d4ed342a7f72e059f083b31ee187ea on sorpaas:master into 819d9e8adc6953ccaad50b5d90a376fc25320aad on ethereumproject:master.


Assign music selection where there is none

Issue

Rarely a zone will have [no music selected] as shown in the Sonos S1 app. When trying to set the input and play anything, typically via a scene the accessory will come on for a few seconds then go off again without playing anything. Once a selection is made in the Sonos app everything works correctly.
Log Messages

NOTE: First attempt is to set selection to Magic Soul from the Home app, when the Sonos amp turns itself off, then the second attempt is a response to a change initiated in the Sonos app.
[6/3/2022, 11:06:26 PM] [Sonos] Lounge Sonos: set Track from "Magic Soul" to "Out Of Reach - Gabrielle"
[6/3/2022, 11:06:42 PM] [Sonos] Lounge TV: set Active Identifier to "Magic Radio"
[6/3/2022, 11:06:43 PM] [Sonos] Lounge TV: warning: set Active Identifier: timed out
[6/3/2022, 11:06:44 PM] [Sonos] Lounge Sonos: set On from true to false
[6/3/2022, 11:06:44 PM] [Sonos] Lounge TV: set Active from 1 to 0
[6/3/2022, 11:06:44 PM] [Sonos] Lounge Sonos: set Track from "Out Of Reach - Gabrielle" to "Magic Radio"
[6/3/2022, 11:06:44 PM] [Sonos] Lounge Sonos: set Active Identifier from 24 to 23
[6/3/2022, 11:06:46 PM] [Sonos] Lounge Sonos: set On from false to true
[6/3/2022, 11:06:46 PM] [Sonos] Lounge TV: set Active from 0 to 1
[6/3/2022, 11:06:51 PM] [Sonos] Lounge Sonos: set Track from "Magic Radio" to "STOP ADBREAK 3"

I don’t think I can do much about this.  A zone player won’t accept Play when no input is selected, and a HomeKit television accessory won’t accept input change when it’s off.  I’m using HomeKit scenes for my favourite radio stations, setting both On and Active Identifier from the Sonos service.  Most times they work, but sometimes I need to revert to the Sonos app.

Thanks and understood, way of the world sometimes.


[Feature] Edit or Delete comments

I would like to have the ability to edit or delete my comments, or in case I am the admin, to do so with other users comments.

:+1:


alternative to .env when running server locally?

At present, the .env file for configuring networking is listed in the .gitignore. I think this makes sense, as for security reasons it's good to not have .env files hosted in public repos. But we'll need a workaround for when someone wants to run the server.js locally. One option is to use a startup cli flag, like npm start --local to tell the script to run locally with predefined variables for that context. @DMGregory would you dm me the .env file details so I make sure I'm setting this up correctly?

@michaelpalumbo I don't seem to have a .env file locally.

@DMGregory oh weird. weren't you working with one last week when we were refactoring connect.js to merge.js? I think we went over that when deciding to what to do with the port number config for heroku or computer canada?

@michaelpalumbo I don't remember that. I think everything we looked at port-wise was in server.js.

The idea was to have three modes, each with their own http and port configs: .env for Heroku, .env for Alice (Compute Canada), and if there's no env it defaults to localhost.

ah i'm gettign my meetings crossed. This discussion was with @n1ckfg last week. Sorry about that. Ok, I'm closing this issue and will open a more relevant one

Ah that was it, thanks for clarifying Nick. Is there a branch where this is
implemented?
On Tue, Nov 23, 2021 at 4:04 PM Nick Fox-Gieg @.***>
wrote:

The idea was to have three modes, each with their own http and port
configs: .env for Heroku, .env for Alice (Compute Canada), and if there's
no env it defaults to localhost.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/worldmaking/nodelab/issues/8#issuecomment-977167391,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AASLT4JQWCS5TVTWMBONWS3UNP6VJANCNFSM5IULZGPQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Am opening this back up as per recent discussion

Yup, it's on main, just pulled and restarted the Alice server to check

Excellent. thanks @n1ckfg
Since we're using main for CC server, develop for heroku, some point today i'll merge that .env into develop and ensure it still runs on heroku

I think you can also set the environment variables on Heroku directly, in case we don't want a platform-specific file in the git repo (not sure if that messes with any of the other platforms?) https://catalins.tech/heroku-environment-variables

Shall we rewrite the startup port logic to simplify things? Heroku / localhost if no .env, Alice setup if .env. See example here: https://github.com/worldmaking/nodelab/tree/nick-dev

@jon-Silveira identified that heroku can't have more than one port open. this may lead to a solution: https://stackoverflow.com/questions/37636580/heroku-node-js-i-have-a-server-which-uses-multiple-ports-how-can-i-get-herok

Solved by @n1ckfg today:
// this will be true if there's no .env file
const IS_HTTP = (!process.env.PORT_HTTP);

const PORT_HTTP = IS_HTTP ? (process.env.PORT || 3000) : (process.env.PORT_HTTP || 80);
const PORT_HTTPS = process.env.PORT_HTTPS || 443;
const PORT = IS_HTTP ? PORT_HTTP : PORT_HTTPS;


Pin setuptools to <72.0.0 to unblock Neko tests

Summary
See https://github.com/pypa/setuptools/issues/4519
Details and comments

The pin doesn't seem to be enough though, I think I might need to pin setuptools-rust too

It's still installing setuptools 72
Collecting setuptools
  Downloading setuptools-72.0.0-py3-none-any.whl (2.3 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 2.3/2.3 MB 50.5 MB/s eta 0:00:00

This is probably neko's responsibility to fix, although I am still not sure how, so far I have https://github.com/Qiskit/qiskit-neko/pull/47

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Details

0 of 0   changed or added relevant lines in 0 files are covered.
5 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.01%) to 89.754%

Files with Coverage Reduction
New Missed Lines
%

crates/qasm2/src/lex.rs
5
92.62%

Totals

Change from base Build<PHONE_NUMBER>9:
0.01%

Covered Lines:
66788

Relevant Lines:
74412

💛  - Coveralls


How can i drag elements from sidebar (overflow-y: auto) to container ?

Demo: http://codepen.io/desandro/pen/cbhDG
I want to be able to drag a slide from sidebar to container and drop there. Is this possible ?
For example with jQuery UI Draggable you can achieve this result by using an option called { "helper": "clone" }. I didn't find something like that in documentation.
Waiting for an answer,
Thanks !

You can do this with Draggabilly, but only by adding your own remove+drop+clone behavior code, making use of Draggabilly's API. Sorry, but there's no easy way to this now.


Data Quality / Labeling Tools?

Any interest in starting a new section for data labeling and/or data quality tools? I have a tool I'd like to PR into the list but not sure it fits into the sections that already exist. Thanks in advance.

Namely, thinking about this tool in particular. Thx!

bump

Hi @taylorfturner, sorry for the delay, I am helping the maintainer now. thank you for the suggestion. If you think this should be added, please create a PR.

Thanks, @vordimous! Will add one

#158 added


Min max count

Adds the ability to supply minCount and maxCount on your collection schema.  #2138 [WIP]

I have a cleaner pr coming up.


Update 01-spell-check.md

i have found some spelling error

Thank you @amanpro30  for your PR!
Since we want to keep the mistakes for other participants to work on, we wouldn't be merging this PR. Hope the practice was good nevertheless :)


Problem with colors in some kde applications with theme applied

It's so difficult to see some elements with this text colors for example

I also have problems with menus on Kdenlive how you can see in this screenshot

Also with this


Fixed relative paths inside themed and localized files

Q
A

Is bugfix?
yes

New feature?
no

Breaks BC?
no

Tests pass?
yes

Fixed issues
#14636

AppVeyor fail isn't connected with this pull request.

Changelog is needed.

Any idea why HHVM fails?

:( No idea.

@samdark I think this is ready :)

Merged. Thank you!


wh:commits and wh:comments don't respect devbox port

The two webhook faker tasks use a hardcoded port number that doesn't match the port ex-remit listens on in dev mode

This is fixed, also added non Devbox web server port support.


istioctl manifest testing: security

istioctl will be the default installation method for 1.4 onwards. We need to verify that the Istio installed using istioctl manifest apply works with the following tasks:
https://preliminary.istio.io/docs/tasks/security/

Note: tests should be done with version that has #18342.

Assigned @pitlv2109 who has been working on running through the tasks with istioctl - can you provide any info on what works and any issues?

Closing this, since security team went though the tasks and all issues were resolved in release (except SDS profile).


Created new function to truncate long file names

This PR creates a new function to truncate long file names to be shown on PASS.
Any filename shown above 25 characters will get truncated by this function, otherwise it'll stay at the same length as before.
https://github.com/codeforpdx/PASS/assets/14917816/9321fbbc-f8d1-4e73-a0d0-a131ca4d71c0
https://github.com/codeforpdx/PASS/assets/14917816/7febd0c4-21eb-4c55-a41a-92b2f46a0593
Technically, this function could potentially be renamed/reused in the future for truncating any long strings in general, not just file names.
Included a unit test to ensure it works as intended.

I see the problem, but I feel like this would be better solved by an overflow property on an HTML element. That way the user would still be able to copy-paste the full file name, and we'd be able to reactively truncate the name based off the available screen space. We also wouldn't risk an inattentive programmer accidentally using the truncated value instead of the real value.
Kinda like this using CSS:
p {
  white-space: nowrap;
  overflow: hidden;
  text-overflow: ellipsis;
  max-width: 200px;
}

Probably an indication we need StatusNotification/StatusMessage refactored, but file name is currently being called together under a single string. So if we truncate that string, it'll cut off the message instead of just the file name.
We could parse file name out and place it at the end, but we'll probably have to rephrase and restructure the StatusMessages.
To avoid the kind of problem you mentioned, how about explicitly calling the function only in places where truncation is needed as oppose to being called in one central location. An alternative is to just rename the variable for fileName to processedFileName to indicate it have been "processed with a function".

Probably an indication we need StatusNotification/StatusMessage refactored.

That seems like a good inferrence to me. So why not fix it there? If the issue is "StatusNotification doesn't display very long file names well" then it seems to me the issue is with StatusNotification, not the file name.

To avoid the kind of problem you mentioned, how about explicitly calling the function only in places where truncation is needed as oppose to being called in one central location.

That sounds reasonable. How would you enforce that? In OOP terms that sounds a lot private member of a class or component. Maybe add it as an internal function to StatusNotification? Or allow StatusNotification to take in a data transform function?
In general, it seems to me that this fix is conflating a display issue with a data issue. The data (filename) is correct. No need to edit it. The issue is the display (html and css) does not show the data to the user well. So we should try editing the display first, before trying to edit the data.

Probably an indication we need StatusNotification/StatusMessage refactored.

That seems like a good inferrence to me. So why not fix it there? If the issue is "StatusNotification doesn't display very long file names well" then it seems to me the issue is with StatusNotification, not the file name.

To avoid the kind of problem you mentioned, how about explicitly calling the function only in places where truncation is needed as oppose to being called in one central location.

That sounds reasonable. How would you enforce that? In OOP terms that sounds a lot private member of a class or component. Maybe add it as an internal function to StatusNotification? Or allow StatusNotification to take in a data transform function?
In general, it seems to me that this fix is conflating a display issue with a data issue. The data (filename) is correct. No need to edit it. The issue is the display (html and css) does not show the data to the user well. So we should try editing the display first, before trying to edit the data.

Think I'll get a crack at that. Let's see what I can do.

Have altered approach to truncating file name using styling instead of dedicated function. As a result StatusMessage/StatusNotification has been updated.


Node.js Foundation Community Committee (CommComm) Meeting 2017-04-27

Meeting 2017-04-27Community Committee (CommComm)

Time
UTC Thu 27-Apr-2017 20:30 (08:30 PM):

Timezone
Date/Time

US / Pacific
Thu 27-Apr-2017 13:00 (01:30 PM)

US / Mountain
Thu 27-Apr-2017 14:00 (02:30 PM)

US / Central
Thu 27-Apr-2017 15:00 (03:30 PM)

US / Eastern
Thu 27-Apr-2017 16:00 (04:30 PM)

Amsterdam
Thu 27-Apr-2017 22:00 (10:30 PM)

Moscow
Thu 27-Apr-2017 23:00 (11:30 PM)

Chennai
Fri 28-Apr-2017 01:30 (02:00 AM)

Tokyo
Fri 28-Apr-2017 05:00 (05:30 AM)

Sydney
Fri 28-Apr-2017 06:00 (06:30 AM)

Or in your local time:

http://www.timeanddate.com/worldclock/fixedtime.html?msg=Node.js+Foundation++Meeting+2017-04-27&iso=20170427T2030
or http://www.wolframalpha.com/input/?i=8:30pm+UTC%2C+Apr+27%2C+2017+in+local+time

Links

Minutes Google Doc: https://docs.google.com/document/d/1YevHuA_csoTltDJ6mHr21fetZshNqWpsRbBRkSHhmYE/
Previous Minutes Google Doc: https://docs.google.com/document/d/18d2XeUP_dB4dyLzjO6wqs6NiaSaUxLFDyXYL_HFTEDc

Agenda
Extracted from cc-agenda labelled issues and pull requests from the nodejs org prior to the meeting.
nodejs/board

Clarify By-Laws WRT Community Committee #57

nodejs/community-committee

Moderation shortage #33
Elect a Chairperson #30
Collaborator's Summit agenda #28
governance: who has voting rights? #21
Add some scope (aka: stake your claim!) #13
Central communication medium for chatting in Node.js #11
Adding folks as members #38

Invited

Ashley Williams @ashleygwilliams (CommComm)
Bryan Hughes @nebrius (CommComm)
Emily Rose @emilyrose (CommComm)
Gregor Martynus @gr2m (CommComm)
James Snell @jasnell (CommComm)
Jenn Turner @renrutnnej (observer)
JP Wesselink @jpwesselink (CommComm)
Mikeal Rogers @mikeal (observer/Node.js Foundation)
Myles Borins @MylesBorins (CommComm)
Rachel White @rachelnicole (CommComm)
Rod Vagg @rvagg (CommComm)
Tracy Hinds @hackygolucky (CommComm)
William Kapke @williamkapke (CommComm)

Notes
The agenda comes from issues labelled with cc-agenda across all of the repositories in the nodejs org. Please label any additional issues that should be on the agenda before the meeting starts.
Joining the meeting
Google Hangouts link
A standing invitation exists for all @nodejs/TSC members who are not also CommComm members to attend in observer capacity.
Public participation
We stream our conference call straight to YouTube so anyone can listen to it live, it should start playing at https://www.youtube.com/c/nodejs+foundation/live when we turn it on. There's usually a short cat-herding time at the start of the meeting and then occasionally we have some quick private business to attend to before we can start recording & streaming. So be patient and it should show up.

Note: I don't think we've updating any labels since the last meeting. Everyone please review the agenda and update cc-agenda labels as appropriate.

@nebrius as stated in https://github.com/nodejs/community-committee/issues/25, can we move this 30  minutes later? We didn't get any objections to this and the Node.js Foundation has a weekly meeting that conflicts with this.

Whoops, sorry, I forgot to update it (meeting times are embedded in the creation tool). Done!

gracias!
On Mon, Apr 24, 2017 at 11:30 PM, Bryan Hughes<EMAIL_ADDRESS>wrote:

Whoops, sorry, I forgot to update it (meeting times are embedded in the
creation tool). Done!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nodejs/community-committee/issues/36#issuecomment-296886847,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB46oBfxxxH4Gmyr8JJ1_yxpyBifY_Mtks5rzWjCgaJpZM4NG9Dq
.

Won't be able to make it this week... Going to be airborne.
On Apr 25, 2017 12:07 AM, "Tracy"<EMAIL_ADDRESS>wrote:

gracias!
On Mon, Apr 24, 2017 at 11:30 PM, Bryan Hughes<EMAIL_ADDRESS>wrote:

Whoops, sorry, I forgot to update it (meeting times are embedded in the
creation tool). Done!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<https://github.com/nodejs/community-committee/issues/36#
issuecomment-296886847>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AB46oBfxxxH4Gmyr8JJ1_
yxpyBifY_Mtks5rzWjCgaJpZM4NG9Dq>
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nodejs/community-committee/issues/36#issuecomment-296896161,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAecVwmQ0sy6hcyqPQnd9S3ROrYHzesZks5rzXFvgaJpZM4NG9Dq
.

commenting to remind myself not to forget -_-

Hm. I remember objections to NOT matching the TSC time. @hackygolucky you said you see if they could move the Foundation meeting.

I unfortunately cannot make it today. I will listen to the recording later (please record it!)
PS: @rachelnicole Here's another reminder for you. 😆

Also- it looks like there this meeting now overlaps the @nodejs/diagnostics meeting. You may be blocking the YouTube live stream.

@williamkapke just fyi, looking at nodejs/diagnostics#94 it doesn’t seem like that is going to be a problem today

is there a participation link? :-)

here! sorry was just getting it @jasnell https://hangouts.google.com/hangouts/_/6sctyenduffjhkqgcgfd6pd4nee

My flight is super delayed and I don't think I'm going to be able to make the call :(

We did not broadcast the meeting due to a lack of quorum, but we did take discussion notes. A few issues will be filed and we'll revisit this week's agenda at the next meeting!

Sorry to miss it! I'll catch up with the notes.

@msmichellegar no worries! We're going to try and adjust the time to be a little friendlier to y'all in the EU.

Note: CTC and TSC meetings often do not have quorum, was there an actual meeting? It should still be broadcast-able then unless there was no public agenda items?

@Fishrock123 There were 4 people who showed, and we discussed whether we should record because no decisions could be made and we didn't think 4 people was representative enough to have proper feedback. We took the discussion notes but did not record, thinking it would not be helpful to do so. Thanks for this input! Good to know for the future.

It's ok. Sometimes it's hard to know if it is "meeting-worthy" when just a few people show up. This kinda thing has come up many times for the TSC/CTC in the past too.
My general measure would be: if we were taking or were going to be taking any significant notes I'd want to broadcast, at least for TSC/CTC meetings.

This meeting has been held, so closing


Add support for conversion between grades

Current:
Conversion function exists, but is totally untested and unused
Expected:
Throughly fix and test the conversion functionality between grading systems.
Non-goal:
Don't try to support converting between bouldering and routes, only within bouldering grades and within routes.

As mentioned in https://github.com/OpenBeta/open-tacos/issues/520, I can help look into this one if I could be assigned to it. Thanks.

Is this done in #47?

Yes. The library supports US and French.


Add gearman/gearmand to layout.yaml

Moar beta users (Pay no attention to that SpamapS behind the curtain)

Build failed.

http://logs.bonnyci.org/check_github/BonnyCI/hoist/253/1490105821.93/bonnyci-run-check-multinode : FAILURE in 8m 03s


Not running on iOS 11

Guys
Some of you are getting problems on iOS11? It's not running anymore in this OS. Basically, when enter on code "await CrossGeolocator.Current.StartListeningAsync(...", does not return.
Bug Information
Version Number of Plugin: 4.0.1
Device Tested On: iPhone 7 / iOS11 (iPhone 5 / iOS 10.3.3 runs fine)
Simulator Tested On: iPhone 8 / iOS11 (iPhone 7 / iOS10 runs fine)
Version of VS: VS for Mac, 7.1.5 (Build 2)
Version of Xamarin:
.NET Core
Runtime Versions:
1.1.1
1.0.4
SDK Version: 1.0.3
Xamarin.iOS
Version: <IP_ADDRESS> (Visual Studio Enterprise)
Hash: 152b654a
Branch: xcode9
Build date: 2017-09-15 02:25:56-0400
Versions of other things you are using:
I'm using the same exact code in the sample to start the service. It runs fine on previous versions of iOS, but on version 11 the method StartListeningAsync "does not return":
public async Task StartListening()
{
if (CrossGeolocator.Current.IsListening)
return;
try
{
await CrossGeolocator.Current.StartListeningAsync(TimeSpan.FromSeconds(30), 10, true, new Plugin.Geolocator.Abstractions.ListenerSettings
{
ActivityType = Plugin.Geolocator.Abstractions.ActivityType.Fitness,
AllowBackgroundUpdates = true,
DeferLocationUpdates = true,
DeferralDistanceMeters = 1,
DeferralTime = TimeSpan.FromSeconds(5),
ListenForSignificantChanges = true,
PauseLocationUpdatesAutomatically = false
});
CrossGeolocator.Current.PositionChanged += Current_PositionChanged;
}
catch (System.Exception ex)
{
...
}
    }

After call CrossGeolocator.Current.StartListeningAsync start to listen Location services.
Until version 11, no problem.

Thank you @alanspires .
In fact I do not observed this new plist: NSLocationAlwaysAndWhenInUseUsageDescription.
I suggest update documentation to show that.
All the best

For me setting the Location access to "Always" in Settings/Privacy/Location services/AppName did the trick.


ff info command

We should add an ff info command to the CLI that will show which version of each component is in a stack

This was resolved in https://github.com/hyperledger-labs/firefly-cli/pull/29


Merge gsoc-develop to develop

A few outstanding issues before we can merge into develop:

Update deprecated messenger
Update comments
Edge case
nits like these

Another issue: the icon turns green when using the seamless posting form, but it doesn't change away from green when the input context leaves the posting iframe.

Another issue: The posting button won't appear on issues of the privly-chrome repository. I think there is a magic string somewhere that checks for privly-chrome in the path?

Another issue: the icon turns green when using the seamless posting form, but it doesn't change away from green when the input context leaves the posting iframe.

I think I have fixed this in previous commits and it should works in most of the cases. Could you open individual issue for this if there are problems in certain conditions?

Another issue: The posting button won't appear on issues of the privly-chrome repository. I think there is a magic string somewhere that checks for privly-chrome in the path?

Fixed in this PR: https://github.com/privly/privly-applications/pull/285

Issues listed here are fixed in latest commits in this PR: https://github.com/privly/privly-chrome/pull/129

I'll open a new issue if the problem persists after I merge all your more recent changes.


Improve documentation

I'll wait for #35, rebase, and clean up the Ajax options docs afterwards (trying to reduce merge conflicts as much as possible).

Merged #35

This is ready.

Looking really nice, thanks @blink1073!

Is there a readthedocs-style service for hosting js docs? We could use Travis + gh-pages, but I really don't like filling a code repo with incremental builds of docs.

We could deploy from Travis directly to a host somewhere (potentially at Rackspace), and not store the incremental builds.


Enable Istio build on Arm64 platform

The Makefile.core.mk is modified for adapting multi-arch
platform.
In init.sh scripts, replace the "amd64" with "GOARCH"
Add arch parameter in dockerfile for choosing build
architecture.
Add build options for building multi-arch images in
istio-docker.mk file
Add arm64 option for helm template in values.yaml

Signed-off-by: Jingzhao<EMAIL_ADDRESS>

Hi @Jingzhao123. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

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.

/ok-to-test

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

Test name
Commit
Details
Rerun command

unit-tests_istio
6572ad63e070b38f7cf7b5077b803d0c7ecbc49e
link
/test unit-tests_istio

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

@Jingzhao123: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

unit-tests_istio
6572ad63e070b38f7cf7b5077b803d0c7ecbc49e
link
/test unit-tests_istio

pilot-multicluster-e2e_istio
6572ad63e070b38f7cf7b5077b803d0c7ecbc49e
link
/test pilot-multicluster-e2e_istio

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

/retest

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

Test name
Commit
Details
Rerun command

pilot-multicluster-e2e_istio
6572ad63e070b38f7cf7b5077b803d0c7ecbc49e
link
/test pilot-multicluster-e2e_istio

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

/retest

You can build them but do they run properly?
On Thu, Dec 5, 2019, 6:04 PM Jingzhao123<EMAIL_ADDRESS>wrote:

@Jingzhao123 commented on this pull request.
In pilot/docker/Dockerfile.proxyv2
https://github.com/istio/istio/pull/19404#discussion_r354636338:

TODO(https://github.com/istio/istio/issues/17656) clean up this hack

COPY --from=default /sbin/xtables-multi /sbin/iptables* /sbin/ip6tables* /sbin/ip /sbin/
-COPY --from=default /usr/lib/x86_64-linux-gnu/xtables/ /usr/lib/x86_64-linux-gnu/xtables
-COPY --from=default /usr/lib/x86_64-linux-gnu/ /usr/lib/x86_64-linux-gnu
+COPY --from=default /usr/lib/${ARCH}-linux-gnu/xtables/ /usr/lib/${ARCH}-linux-gnu/xtables
I have successfully built the corresponding images on x86 & arm64 with
following command. So, I think it is OK.
make build
make docker
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/pull/19404?email_source=notifications&email_token=AAEYGXO4JUCYMOIRODSF55DQXGXJPA5CNFSM4JVSVZR2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCOGE43A#discussion_r354636338,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEYGXPUVDM3NO3L3GQBSOTQXGXJPANCNFSM4JVSVZRQ
.

Hi @howardjohn
The purpose of this patch only focuses on building Istio on Arm64. If want to run the istio on Arm64, it still need to do more works, just like supporting multi-arch docker image, updating envoy component. So, in this patch, i only do some modifications on Makefile and Dockerfile. I will upstream other patches for further porting work.

Hi @howardjohn
I have been build the 1.13 K8s environments for testing Istio on x86. Could you please tell me how to verify the Istio images and configurations ? If those images could be run correctly on it, does it means that those images are OK?
On the other hand, for arm64 platform, i just want to enable to build istio, but not running it (actually, there may be lots of work to do). So, I will not verify it on it. I will do more contributions about it. Is this feasible?

On the other hand, for arm64 platform, i just want to enable to build istio, but not running it (actually, there may be lots of work to do). So, I will not verify it on it. I will do more contributions about it. Is this feasible?

I am perfectly happy adding code to get arm64 working, but I am not sure what is the point of building the images if they do not run? Given that adding support for another platform adds some complexity, it would be good to know that it actually works before merging.

Hi, @howardjohn
Thanks for your comments. I also agree with your opinion. And i will verify those modification on both platforms. On the other hand, i want to ask a few questions about it.

How to verify those functions ?
If those images could be run correctly on k8s, does it means that those images are OK?
Now, i want to build the arm64 docker images by qemu tools. Is it OK?
Now, as far as i known, since there is no arm64 platform, Istio arm64 images can not be built on Google cloud. Could i build those image by qemu tools ? Just like Kubernetes.

Hi, @howardjohn
Thanks for your comments. I also agree with your opinion. And i will verify those modification on both platforms. On the other hand, i want to ask a few questions about it.

How to verify those functions ?
If those images could be run correctly on k8s, does it means that those images are OK?
Now, i want to build the arm64 docker images by qemu tools. Is it OK?
Now, as far as i known, since there is no arm64 platform, Istio arm64 images can not be built on Google cloud. Could i build those image by qemu tools ? Just like Kubernetes.


Difficulty using WithLatestFrom

I'm trying to use WithLatestFrom to print words with their respective lengths. My thinking was that if a word got emitted at time X, its length would be calculated at time X+1. I hoped to use WithLatestFrom to pair each calculated length with the most recent word. It doesn't work. I think I know what is happening but am not sure. When a word is emitted, maybe synchronous callbacks notify WithLatestFrom about the length of a word BEFORE it hears about the word. And since the left sequence (length) has a value but the right sequence (word) does not, nothing gets emitted. If I replace the calculated lengths with explicit calls like word.OnNext("cat"); length.OnNext(3); it all works fine.
static void Main(string[] args) {
    Subject<string> word = new Subject<string>();
    IObservable<int> length = word.Select(i =>i.Length);
    var report = Observable
        .WithLatestFrom(
            length,
            word,
            (l,w) => new { Word = w, Length = l })
        .Subscribe(Console.WriteLine);
    word.OnNext("cat");
    word.OnNext("donkey");
    word.OnNext("rhinoceros");
    Console.ReadLine();
}

The output is:
  Cat 6
  Donkey 10

You have to zip them together. As you said, the animal gets onNexted first but at that time, there is no latest from on the other 'rail' and the value gets dropped.

This is maybe related to #152 and would support my claim that the intuitive behavior of WithLatestFrom is to subscribe to the context-observable first, then to the actual source, not vice versa as it is now. But this is just an assumption. Maybe you can merge and test the mentioned pull request locally against your code example ?

The example uses subjects and the subscription order doesn't matter here. Even if the length is reaches WithLatestFrom first, there is no main value so it won't pair up.

Seconding what @akarnokd suggested, using Observable.Zip here seems to show the desired behaviour.
@danielcweber interesting theory: I'll see if I can test that Observable.WithLatestFrom example against master when I get a chance...

Zip works in this case but I'm trying to find a general solution where each input generates 0 or 1 output. So if I receive an output, I know it is related to the most recent input.

I'm trying to get my head around how this Rx stuff works. The order items are emitted seems to be based on the subscription order. It's not intuitive to me. This code...
 word.Subscribe(Console.WriteLine);
 length.Subscribe(Console.WriteLine);
 word.OnNext("cat");
 word.OnNext("donkey");

Generates this result...
 cat
 3
 donkey
 6

But if I swap the subscription order, I get...
 3
 cat
 6
 donkey

Or for another example, the results change if you swap number and squared in the Merge method.
Subject<int> number = new Subject<int>();
IObservable<int> squared = number.Select(i => i * i);
Observable.Merge(number, squared).Subscribe(Console.WriteLine);
number.OnNext(4);
Console.ReadLine();

Subjects generally use an internal array for the current subscribed Observers and they forward signals to them in that specific order.

word.Subscribe(Console.WriteLine); // obs 1
length.Subscribe(Console.WriteLine); // obs 2

In this case, you subscribe to the word subject first, then subscribe to the length Observable which in turn subscribes to word too. word then has [obs1, obs2] in its internal list.

length.Subscribe(Console.WriteLine); // obs 2
word.Subscribe(Console.WriteLine); // obs 1

If you swap them around, thw length subscribes to word and then you subscribe to word directly; word then has [obs 2, obs 1].

It's not intuitive to me.

What would be more intuitive to you?

I'm confused by "you subscribe to word first." It looks to me like length is the first piece of code to observe word.
Subject<string> word = new Subject<string>();
IObservable<int> length = word.Select(i => i.Length);
IDisposable lengthConsole = length.Subscribe(Console.WriteLine);
IDisposable wordConsole = word.Subscribe(Console.WriteLine);

Reading from top to bottom and filling in those internal arrays...
 word
      has observer 1 length
      has observer 2 wordConsole
 length
      has observer 1 lengthConsole

So when you emit a word, this happens...
 word
      is sent to observer 1 length
           is sent to observer 1 lengthConsole which prints it
      is sent to observer 2 wordConsole which prints it

So the output would be as follows. Still kind of weird because it looks like the length is being calculated before the word is available.
 3
 cat
 6
 donkey

The output would be exactly the same if the lengthConsole and wordConsole lines were swapped. This consistency makes sense to me because from a functional perspective (see dependencies below) nothing has changed.
 word = f() // no dependencies
 lengthConsole = f(length)
 wordConsole = f(word)
 length = f(word)

Subjects are lazy when establishing a subscription so even if you assemble sequences in some order, the subscription via subscribe determines the actual order in which events in those subjects are dispatched.

The output would be exactly the same if the lengthConsole and wordConsole lines were swapped.

In pure mathematics perhaps but this is computer science, here there are ordering of events, especially on a single thread, and things attract side-effects pretty quick - such as printing to the console.

Ok I understand now why I'm not getting the results I want. At the bottom is another version of the original problem that works as expected, but only because I inserted an artificial delay before each result is published.
I'm using Rx for view models. The user manipulates various properties while filling in forms. I do calculations on those properties, like fullName = firstName + lastName. Sometimes I do simple synchronous validation, like checking if an email address matches a regular expression pattern. In those situations, Zip can be used to correlate inputs and outputs. But if someone is choosing a new username I might do a web query to validate whether the username is taken. Maybe I'll throttle the input so the check doesn't happen on every keystroke. Zip won't work in that case but WithLatestFrom or some kind of Join might. If I want a GENERAL solution for correlating inputs and outputs, WITHOUT having to know how those calculation occur, I'm kind of stuck. I think my best option is to make sure that all my function outputs INCLUDE their associated inputs, like new { Email =<EMAIL_ADDRESS>IsFormattedCorrectly = true} and new { Url = "www.google.com", IsWebsiteUp = true }. That way I won't have to correlate the input and output streams. Alternatively I could introduce artificial delays to make every calculation asynchronous use something like WithLatestFrom everywhere.
Subject<string> word = new Subject<string>();
IObservable<string> wordPub =
    word
    .Select(i => Observable.Return(i).Delay(TimeSpan.FromSeconds(1)))
    .Concat();
IObservable<int> lengthPub =
    wordPub
    .Select(i => Observable.Return(i.Length).Delay(TimeSpan.FromSeconds(0.5)))
    .Concat();
Observable
    .WithLatestFrom(lengthPub, wordPub, (l, w) => new { Word = w, Length = l })
    .Subscribe(Console.WriteLine);
word.OnNext("cat");
word.OnNext("donkey");
word.OnNext("rhinoceros");
Console.ReadLine();

I opened this bug but I don't have a stake in it anymore. I figured out how to work around the behavior. It seems like the current behavior is dependent on the order the method subscribes to the first and second sequences. Because it subscribes to the first sequence first, it learns about length before it hears about the word that produced it. Is this some kind of race condition? From experimentation, I think this is similar to the behavior of Join.
I'm just getting started with rx and will leave it up to the experts to decide if anything needs to change here.

Has this changed with the fix of #152?

Closing for inactivity.


Revised grammar with latest changes from Facebook IDE team

Update the syntax highlighting grammar file with the latest changes from Facebook.

Syntax highlighting for attributes
Some fixes for shapes
Support for "as"
Various other bugfixes

Third time is the charm 🎉

Build failure is a known issue, so I'm just going to merge this


Prompt on exit does not work without information header

The prompt on exit does not work when the argument to not display the information header is given.
For example, with the following constellation, there is no prompt on exit.
./salomon.sh -i /var/log/foobar.log -a monitor -p --no-info

This is mode independent (also applies to analyzing mode).

Fixed in version 1.18.1.

Fixed in version 1.18.1.


Make code generation for shader boolean uniform instances easier

Improve code generation for boolean uniform instances in shader compiler introduced in https://github.com/godotengine/godot/pull/67112

Looks like I rebased by mistake after merging this locally, so GitHub didn't notice it was merged.
Merged as 099f127cc33c5daa9a3e36b8cdb407e32249b003.


fix: check driver is Cake Driver before trying to access

I’ve discovered an error when using CakePHP 5 and using elastic search
This results in the below error:
Fatal Error: Uncaught Error: Call to undefined method Elastica\Client::getLogger() in /vendor/lordsimal/cakephp-sentry/src/Http/SentryClient.php:62
This pull request ensures that it checks the Driver is actually a Driver before it tries to access this which is the same way CakePHP itself is doing.

Thanks, I will fix stan after this is merged


Add regression test for EH Batch code

The current regression testing for EH only uses the surrogate model. Once the EH batch code is migrated to the public repository, we should:

[ ] Add the public repo as a new submodule
[ ] Write a regression test for it in test_RunFunctions.py.

this repo was never made public and this model has been superseded by the surrogate.


[Enhancement] Map partition value to partition range using MAX_VALUE instead of Infinity value

What type of PR is this：

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

#11116
Problem Summary(Required) ：

Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] I have added user document for my new feature or new function

run starrocks_admit_test

run starrocks_admit_test

run starrocks_admit_test

run starrocks_admit_test

run starrocks_admit_test

run starrocks_fe_unittest

run starrocks_fe_unittest

[FE PR Coverage Check]
:heart_eyes: pass : 1 / 1 (100.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/connector/PartitionUtil.java
1
1
100.00%
[]


unknown model connecting with openrouter claude-3.5-sonnet

Issue
aider --model openrouter/anthropic/claude-3.5-sonnet
Model openrouter/anthropic/claude-3.5-sonnet: Unknown model, context window size and token costs unavailable.
Did you mean one of these?

openrouter/anthropic/claude-3-sonnet
openrouter/anthropic/claude-3-opus
openrouter/anthropic/claude-2
https://aider.chat/docs/llms/warnings.html
Model openrouter/anthropic/claude-3-haiku-20240307: Unknown model, context window size and token costs
unavailable.
Did you mean one of these?
openrouter/anthropic/claude-3-haiku
https://aider.chat/docs/llms/warnings.html

Version and model info
Aider v0.40.0

Thanks for trying aider and filing this issue.
The model to use is openrouter/anthropic/claude-3-5-sonnet with a 3-5 not 3.5.

Sorry, you were correct. It should be openrouter/anthropic/claude-3.5-sonnet with a 3.5.
You can ignore the warning, which is what the URL it provides indicates. It's just a warning.

I'm going to close this issue for now, but feel free to add a comment here and I will re-open or file a new issue any time.

Error is here:
ModelSettings(
    "openrouter/anthropic/claude-3.5-sonnet",
    "diff",
    weak_model_name="openrouter/anthropic/claude-3-haiku-20240307",
    use_repo_map=True,
    examples_as_sys_msg=True,
    can_prefill=True,
),

should be
ModelSettings(
    "openrouter/anthropic/claude-3.5-sonnet",
    "diff",
    weak_model_name="openrouter/anthropic/claude-3-haiku",
    use_repo_map=True,
    examples_as_sys_msg=True,
    can_prefill=True,
),

It's strange that there is no error during operation, because there is no such model as claude-3-haiku-20240307 in Open Router, unless in this case summaries and commit texts are made via Sonnet?


Adding Prisoner Dilemma game and Text Analyzer Tool

Thanks @iamrahulmahato  for Giving me chance
here i have added prisoner dilemma game please accept request

here added text analyzer

Thankyou.

mention the issue no

mention the issue no

issue no #96 and issue no #104

mention the issue no

thankyou @iamrahulmahato  for reply


Text won't show correctly

Hey guys!!
Came up with a new idea, although, with my/our "basic" knowledge of the language, it's archaic as heck, lol.
anyway, my code is mostly finished...
I just need a spot of help with the text at the bottom (see copied code below)..
I cannot for the life of me, get it to display without adversely affecting my other code...
anyone wanna have a crack at it?
Code as follows:

I would try using push() and pop() to isolate part of your code and see if
it helps.
On Tue, Sep 25, 2018, 18:46 Nathaniel Hagen<EMAIL_ADDRESS>wrote:

Hey guys!!
Came up with a new idea, although, with my/our "basic" knowledge of the
language, it's archaic as heck, lol.
anyway, my code is mostly finished...
I just need a spot of help with the text at the bottom (see copied code
below)..
I cannot for the life of me, get it to display without adversely affecting
my other code...
anyone wanna have a crack at it?
Code as follows:
[image: ss1]
https://user-images.githubusercontent.com/42828220/46050796-47439f00-c0f3-11e8-99a5-fcea2b5e134d.PNG
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Montana-Media-Arts/120_CreativeCoding/issues/148, or mute
the thread
https://github.com/notifications/unsubscribe-auth/Ao1K1ZQ8IgL75TtaGxWsHgszOGNkr-k5ks5ues5dgaJpZM4W5th3
.

Try changing the quotation marks to apostrophes.  Or nesting the quote in apostrophes if you need them.

I think your text might be outside of your canvas. Try text("Time is Money My Friend", 640, 120)
I think that by default text is anchored to the lower left.
Also, I think that there should not be a space between the function() and (parameters). This will cause most languages to throw up, and might cause your javascript to not be interpreted properly by some browsers, and might be a source of your problem as well.


Error on Android build

Trying to build on Android and getting
e: /Users/marcogaletta/.pub-cache/hosted/pub.dartlang.org/biometric_storage-3.0.0/android/src/main/kotlin/design/codeux/biometric_storage/BiometricStoragePlugin.kt: (275, 89): Unresolved reference: R
e: /Users/marcogaletta/.pub-cache/hosted/pub.dartlang.org/biometric_storage-3.0.0/android/src/main/kotlin/design/codeux/biometric_storage/BiometricStoragePlugin.kt: (280, 51): Unresolved reference: R

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':biometric_storage:compileDebugKotlin'.
> Compilation error. See log for more details

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 59s
Exception: Gradle task assembleDevelopmentDebug failed with exit code 1

used ext.kotlin_version = '1.4.31' in build.gradle and added biometric_storage: ^3.0.0 to pubspec.
It builds on ios simulator but never prompt me the faceID popup.
Do you have any guess?

Try to use compileSdkVersion 30 in your app/build.gradle

HI @hpoul thanks for the quick response. I already have compileSdkVersion set to 30. Do you any other ideas? Seems doesn't recognize the android version when doing val biometricOnly options.androidBiometricOnly || Build.VERSION.SDK_INT < Build.VERSION_CODES.R;.

Same error here. We also have compileSdkVersion set to 30.

btw, have you tried 3.0.1-dev.1?

3.0.1-dev.1 works for me

ok, i've pushed 3.0.1 as stable. if there are still problems let me know.

Hey didn’t try it yet but great news! I want to ask (without opening other issue) if you now why on ios simulator (don’t know yet on real device) the fadeid system prompt doesn’t  shows up.
btw thank you for your job!


Priority UI Dropdown on task create should allow selection by typing

Expected Behavior
When the Priority drop-down has focus, typing the first letter of a priority should immediately select the appropriate priority
Current Behavior
When the Priority drop-down has focus, typing H,M, L does nothing.
Pressing down opens the drop-down, but then pressing any of H, M, L(or any character really) causes Medium to be selected.
Note, two things to correct,
a) allow the drop-down to be immediately changeable with a keypress when it has focus
b) allow the drop-down values to be selected by typing
Context (Environment)

Obsidian version: 0.13.23
Tasks version: 1.4.1
[x] I have tried it with all other plugins disabled and the error still occurs

Thanks, @madorb. The current workaround is using the arrow keys.

Selecting the priority using 4 radiobuttons would be also much faster than a dropdown.

@Cito disagree. may be slightly better than current as you could tab through them instead of moving to arrow keys. but a type-ahead drop-down where one just needs to type say H to have high selected, then hit tab to go to move onto question would be optimal

@madorb One does not exclude the other. For use with the keyboard, it would be best if you could tab to the priority radioboxes and then select the priority with either HML or cursor keys. I.e. same behavior as the dropdown - it should not tab through the individual radiobuttons, but only to the currently selected one, so tabbing through the fields would be just as quick. But when using the mouse, it would be much faster.

This feature was released in 1.16.0 and then improved in 1.17.0
Docs: https://obsidian-tasks-group.github.io/obsidian-tasks/getting-started/create-or-edit-task/#keyboard-shortcuts


pacu: init at 1.5.2

Description of changes
AWS exploitation framework
https://github.com/RhinoSecurityLabs/pacu
Related to https://github.com/NixOS/nixpkgs/issues/81418

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

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

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

2 packages built:

pacu
pacu.dist


Hello world example not working on Windows

[Details]
I just copied the hello world example located at: https://www.mandarinets.org/docs/master/mandarine/hello-world
and when I run it it says:
PS C:\Users\name\Desktop\deno-test> deno run -c tsconfig.json --allow-net --allow-read --allow-env .\app.ts
Check file:///C:/Users/name/Desktop/deno-test/app.ts
error: TS2345 [ERROR]: Argument of type 'T | undefined' is not assignable to parameter of type 'T | PromiseLike<T>'.
  Type 'undefined' is not assignable to type 'T | PromiseLike<T>'.
      resolve(value);
              ~~~~~
    at https://raw.githubusercontent.com/mandarineorg/mandarine-postgres/v1.1.0/utils.ts:94:15

[Environment]

Mandarine version: 2.3.0
Deno version: 1.6.3 (release, x86_64-pc-windows-msvc)
Deno v8 version: 8.8.294
Deno typescript version: 4.1.3

I was able to make it run by adding "strict": false, to tsconfig.json even though it is not mentioned in the official docs as a necessary config.
see: https://www.mandarinets.org/docs/master/mandarine/main-configuration

I was able to make it run by adding "strict": false, to tsconfig.json even though it is not mentioned in the official docs as a necessary config.
see: https://www.mandarinets.org/docs/master/mandarine/main-configuration

Hello there. This issue is solved in Master and will be resolved in the next release which is 2.4.0

Hello there. This issue is solved in Master and will be resolved in the next release which is 2.4.0


Introduce CallFailedRuntimeException in client

@ptuomola @vidakovic would this be OK for you? As I'm using the FineractClient to write tests, I'm already tired of adding throws IOException to every @Test.

FYI the build on this actually passed and was fully green, and continuous-integration/travis-ci/pr still is, but continuous-integration/travis-ci/push restarted because of an experiment I did (I've also pushed the same branch to the apache/fineract instead of only vorburger/fineract, to try out something).. please ignore, and consider this green.


xubuntu only booting into GRUB

When I boot into Xubuntu it only shws GNU GRUB terminal like thing, how do I boot into Xubuntu?

Try booting into Windows, shutting down properly, and then booting into Xubuntu

Tried doing that, still boots into GRUB, tried making another partition for the install but it says the EFI partition is missing, the installation goes on and reboots into GRUB

The boat manager (Grub) is broken you have to reinstall xubuntu and try again!

same issue

Completely reinstall xubuntu if you haven't already.
Closed due to no response in 16 days.

I did that and it still wouldn’t boot past the GRUB command line. I pretty much gave up. It’s not that big of a deal though because I have a Mac as well, I just wanted to try this out.

Alright.


not recognizing the GPU and prompting that it is an uncompatable GPU

Please ensure your GPU is supported.

Closing due to inactivity.


Sorry for writing here. I have little question)

What theme you using on  https://atom.io/packages/autocomplete-java ???

You probably mean the preview animation on the autocomplete-java README.md. I'm not sure, but at the time I was probably using One Dark theme with One Dark syntax.


fix(nginx): update to version 1.20.1

Signed-off-by: Felipe Zipitria<EMAIL_ADDRESS>Fixes #65.
Can you take a look @franbuehler ?

Maybe we need to switch to stable tag, so it will be rebuilt every week?

Maybe we need to switch to stable tag, so it will be rebuilt every week?

Yes. Sounds good. Stable seems to be the new "latest"?

Initially merging this one then. We can switch to stable in a new PR.


<s class="aside-left"> overflows into main text

Describe the bug
Text in left aside overflows into main text
To Reproduce
make a left-aside html element
Expected behavior
Do not flow into main text
Screenshots

Version [Desktop or Mobile]:
latest Obsidian, latest Sanctum

Does this happen with any the recommended fonts as well (iA Writer & IBM Plex)?

Just changed back to default fonts. Still there:

PS: I'm on Windows, so it's IBM Plex

Do you have any active snippets or external plugins? I'd recommend to disable them 1 by 1.

Deactivated all community plugins (safe mode on). Still there:

Do you have any snippets affecting padding or readable line length perhaps?

If you changed line width in the Style Settings, that might be affecting it.

I've managed to finally reproduce this. It will be fixed next update.

This should now be fixed with the new update. :)


Data download error

Hi! The data URLs return the 404 Not Found error. Is the server down?

It seems to be working now. Thanks.


Support for Host Port

If I want my app to handle URLs with a custom port, the <intent-filter> in the AndroidManifest.xml should look like this:
<data android:host="foo.bar" android:port="1234" />
Currently, there doesn't seem to be a way to specify a port in the config.xml file though. If I add the port to the host property: <host name="foo.bar:1234"> then the port is copied as is into the AndroidManifest.xml's android:host property, and the intent doesn't function. If I write <host name="foo.bar" port="1234"> then it's completely ignored.
Any plans to support this?

Sure, this can be added. Although, don't think that iOS supports this, so it's gonna be an Android-only feature... But why not.

Did this get added?
Just new to the whole cordova stuff and we were trying to add this in our app.  Trying to prove that it can work with just a local server (which has an port).  As far as I can tell this is the one thing that is holding it up from working.


No results from the scanner on > iOS 14

The scanning functionality of Quagga stopped working. I tried different aspectRatios and different configurations regarding the camera selection without any success.
The camera selection, on the other hand, and general implementation work fine – the stream starts without any issues. No barcodes (ean_code) are recognized though.
Android works without any issues.
 const QuaggaInit = () => {
        Quagga.init(
            {
                inputStream: {
                    type: "LiveStream",
                    constraints: {
                        width: {
                            min: 450
                        },
                        height: {
                            min: 300
                        },
                        deviceId: selectedDevice ? selectedDevice : "default",
                        facingMode: "environment",
                        /*aspectRatio: {
                            "min": 1,
                            "max": 2
                        }*/
                    }
                },
                locator: {
                    "patchSize": "medium",
                    "halfSample": true
                },
                numOfWorkers: 2,
                frequency: 10,
                decoder: {
                    readers: [
                        "ean_reader",
                    ]
                },
                locate: true
            }, err => {
                if (err) {
                    console.log(err, "error msg");
                }
                Quagga.start();
                return () => {
                    Quagga.stop()
                }
            });
    }

    React.useEffect(() => {
        getDevices();
        checkPermissions();
        Quagga.onProcessed(result => {
            var drawingCtx = Quagga.canvas.ctx.overlay,
                drawingCanvas = Quagga.canvas.dom.overlay;

            if (result) {
                if (result.boxes) {
                    drawingCtx.clearRect(
                        0,
                        0,
                        Number(drawingCanvas.getAttribute("width")),
                        Number(drawingCanvas.getAttribute("height"))
                    );
                    result.boxes
                        .filter(function (box) {
                            return box !== result.box;
                        })
                        .forEach(function (box) {
                            Quagga.ImageDebug.drawPath(box, { x: 0, y: 1 }, drawingCtx, {
                                color: "green",
                                lineWidth: 2
                            });
                        });
                }

                if (result.box) {
                    Quagga.ImageDebug.drawPath(result.box, { x: 0, y: 1 }, drawingCtx, {
                        color: "#00F",
                        lineWidth: 2
                    });
                }

                if (result.codeResult && result.codeResult.code) {
                    Quagga.ImageDebug.drawPath(
                        result.line,
                        { x: "x", y: "y" },
                        drawingCtx,
                        { color: "red", lineWidth: 3 }
                    );
                }
            }
        });

        Quagga.onDetected(detected);

    }, []);

    const detected = result => {
        onDetected(result.codeResult.code);
    };

Test device: iPhone 12 Pro

I'm told that selecting an aspectRatio: 1 in your constraint works there, although I can't confirm that since I lack any iOS devices.

just realized it's aspectRatio: { ideal: 1 } ..
might take a look through this thread for some ideas https://github.com/ericblade/quagga2/issues/419

@ericblade
Thank you. I tried this configuration. Still not working properly.
The iPhone camera appears to be incredibly picky. I scanned one EAN code succesfully, which had nearly the exact camera format. As soon as some text is around the EAN, or the EAN is quite small and can not be recorded in the camera's format due to exceeding the minimum focus distance of the iPhone's camera, the scanner always fails.
On the other hand, Android scans EANs without any problems, even if they are not in the camera format, surrounded by text or very small.
So would you suppose that this is an issue related to the library (The iPhone camera itself works perfectly, focusses correctly) or due to Safari/iOS?

I wish I had any more insight to that.  The control that a web app has, and more so on iOS, compared to a native app, over the camera functionality is pretty minimal.   It could be hardware, it could be OS/Browser, or it c ould be the library.  Could be a combination of all of that just not working together in expected ways that work elsewhere. :|

does you fixed it? i have same issues, this beacause onProcessed() function run only one time when init. So it never get onDetected()

@JonSnow-vn
Nope, but Android works fine. onDetected() is called correctly each time on detection :)

@JonSnow-vn Nope, but Android works fine. onDetected() is called correctly each time on detection :)

yes, me too, this problem only on iphone. Like what i say, on iphone, onProcessed() function run only one time when init. I test on macbook pro and ipad pro its work fine. I don't konw ipad mini is work? i dont have this.

Interesting observation, I have not debugged it that in-depth.

I wish I had any more insight to that. The control that a web app has, and more so on iOS, compared to a native app, over the camera functionality is pretty minimal. It could be hardware, it could be OS/Browser, or it c ould be the library. Could be a combination of all of that just not working together in expected ways that work elsewhere. :|

you have any idea?

Unfortunately, I don't.  The one person using my app on iOS isn't reporting any problems.

Unfortunately, I don't. The one person using my app on iOS isn't reporting any problems.

I can confirm, is working fine here.

Anyone have any other ideas to resolve this? I'm still unable to scan on my iPhone 13 mini after messing around with setting aspectRatio.

Is this issue resolved?

@PriyaN-d Yes, we were able to get this working again last year - though we did have to add a way for users to try out each available camera themselves (different models expose different settings/lenses, front/back/wide angle etc...)

though we did have to add a way for users to try out each available camera themselves (different models expose different settings/lenses, front/back/wide angle etc...)

this is probably the way.  On a lot of Android devices, the default camera selection provides fisheye view, which mangles the image


Fix bug in README

Resolves #1100
Checklist before requesting a review

[x] I have performed a self-review of my code
[x] I assure there is no similar/duplicate pull request regarding same issue

Pls do some proper changes. 😞


Fix panic if no annotation is provided for rabbitmq LoadBalancer override

Jira: https://issues.redhat.com/browse/OSPRH-8066

/cherry-pick 18.0.0-proposed


Improve README.md

Right now the readme has very little information. We should add enough text so people don't need to go read the docs to get an initial impression of what Apollo Client is and how it would make their life easier.

I find it extremely frustrating that apollo-* README.md does not contain configuration and API documentation. I spend most of the time looking at the source code to understand the API.

I would say, Apollo Client is a bit too involved to be explained in a single MD file. That is why they have their docs, which is linked front and center in the readme.
http://dev.apollodata.com/react/
Scott

@smolinari As a developer, all I need is API and configuration explanation. For anything else, I will figure it out from the source code.
That documentation you referred to is a recipe book. It is not a documentation one could use as a reference. Not to mention that it is not versioned.

I didn't say the docs were good. :smile:
Scott

BTW, @gajus - did you see this part of the docs?
http://dev.apollodata.com/core/apollo-client-api.html
@helfer - Shouldn't the link in the docs go to the "Core JS" section?
Here? http://dev.apollodata.com/core/index.html
That link through me off (too).
Scott

@smolinari you're right, the link was wrong. I think @stubailo fixed it in #655, right?

@gajus What parts of the API would you want to see explained in the readme? Because I don't think it makes sense to document the full API, that's what we have the API docs for.

@helfer - yes. The link goes to the right place now.
Scott

Given the limited interest I'm going to close this. If someone wants to give it a shot, I'm still open to a PR that improves the readme.


bump scalapb to 0.9.7

arrow flight needs a newer version of grpc https://github.com/ConsultingMD/potluck/pull/2572

meant to add more color here.  the issue iirc (ran into it initially during the hackathon) is that something in the dependencies of a scala grpc service requires fastparse v2.  scalapb 0.8.4 uses fastparse v1 which results in some class not found errors at run time (when parsing the catalog prototxt files).  scalapb 0.9.7 upgraded to fastparse2 (scalapb > 1.x.x said f*$% it and wrote their own parser).
this is all pretty annoying and basically because the maven coordinates used by rules_scala aren't controlled by bazel-deps.  the newest versions of rules_scala let you configure "deps_providers" that set the scala toolchain / scalapb toolchain classpath w/ your own dependencies so this shouldn't occur.  we should be able to upgrade to them now that we have upgraded bazel.