How to register action group and set anchor

When write a plugin project, i can register action group and set anchor and look like this.

So is the way to do the same by liva-plugin？

You can try liveplugin.PluginUtil#registerAction() function which has actionGroupId but no way to specify anchor.
Also the main idea behind liveplugin is to just evaluate code inside IDE JVM. You don't have to use liveplugin.PluginUtil and can use IntelliJ API directly. In particular, you could lookup the action group object using ActionManager.instance.getAction(actionGroupId) and then use com.intellij.openapi.actionSystem.DefaultActionGroup class to add action to the group with the anchor (I think this is what xml config is doing under the hood anyway).

work well by use DefaultActionGroup#addAction(AnAction,Constraints)
thank you very much !


Per module-script tree information

Looking through whatwg module spec and I think that the module-script assumes per-module-script-tree information. For example, nonce is propagated through the module script tree fetching.
However, in the current loader spec, it seems that BrowserLoader's @@fetch hook just takes the entry and the key. So the above information seems discarded through the pipeline. Is there any discussion to solve this?
Maybe, I misunderstand something. Please point it.
Personally, the easiest way to solve this issue is, instantiate BrowserLoader per script tag.
OR, passing metadata info throught the pipeline is one choise.

metadata was discussed here: https://github.com/whatwg/loader/issues/37, is that enough?

Looks nice. We need to discuss it in #37.


Schema Nullability Extensions

Add Schema Nullability Extensions. This is similar to a @nonnull schema directive but works with lists.
Given the following SDL:
# schema.graphqls
type Query {
  random: Int
  list: [String]
  required: Int!
}

You can extend it like so:
# extra.graphqls
extend type Query {
  # make random non-nullable
  random: Int!
  # make list and list items non-nullable
  list: [String!]!
  # make required nullable
  required: Int
  # add a new field
  new: Float
}

this isn't allowed in the spec, right?

Indeed, this is not. allowed. Only new fields are allowed. This is an Apollo addition.


Updates the version of Microsoft.Extensions.Logging.AzureAppServices

Current version of "ASP.NET Core Logging Integration" extension on Azure App Service is 6.0.3 (which is the latest version of released nuget package). Since the current project was refering v 6.0.0, it created conflict during startup of sample appliaction when deployed to AzureAppService. The sample application would work for the first time when deployed, but will fail after subsequent restart (after the extension gets added). Ref: Tutorial: Deploy an ASP.NET Core and Azure SQL Database app to Azure App Service

project has just recently been updated to 8.0.


only_track_filter: whitelist of pokemon ids to track and catch

defaults to normal tracking/catching behavior. when list of ids is supplied, only those corresponding pokemon will be tracked.
uses:

drop bot in abra nest with "only_track_filter": "63,64,65" and everything other than abra, kadabra, and alakazam will be ignored
when completing pokedex, insert ids of missing pokemon

Please modify README.me as well for the usage. and the contributor.
An example or suggested config is also welcomed.

Please help to update the README.me, to have this feature documented well. This is a killer feature.

This is fully implemented in the new config.json setup that is on dev.


Update secrets provider to retrieve batch secrets base64-encoded

Is your feature request related to a problem? Please describe.
A new feature was added to Conjur enabling a user to request secrets from the batch endpoint encoded in base64. This allows a user to retrieve binary secrets from the endpoint without Conjur throwing errors. Secrets provider for k8s should be updated to use this new feature to ensure there are no errors with secret retrieval.
Describe the solution you would like
Secrets provider now performs all batch secret requests with the Accept: base64 header specified.
Additional context
Here is the original issue referencing the use of secrets provider for k8s.

thanks for creating this issue @telday !
The secrets-provider uses the conjur-api-go for retrieving secrets so the fix will go there.
Just to clarify - to use this feature we need to add the new header and then base-64 decode each secret in the JSON object?

thanks for creating this issue @telday !
The secrets-provider uses the conjur-api-go for retrieving secrets so the fix will go there.
Just to clarify - to use this feature we need to add the new header and then base-64 decode each secret in the JSON object?

@orenbm yes, that is the planned implementation, @izgeri recommended editing the go client requests to the batch secrets endpoint to use the new header and automatically decode the returned JSON

@orenbm yes, that is the planned implementation, @izgeri recommended editing the go client requests to the batch secrets endpoint to use the new header and automatically decode the returned JSON

@orenbm yup!
I think we would update the go client so that this request
https://github.com/cyberark/conjur-api-go/blob/b3f7d8f046f8d48d072fa028217d1add8a5a294e/conjurapi/router_v5.go#L141
sends the header, and this method base64 decodes it: https://github.com/cyberark/conjur-api-go/blob/b3f7d8f046f8d48d072fa028217d1add8a5a294e/conjurapi/variable.go#L27
then the secrets provider would just have to bump its go client version.

@orenbm yup!
I think we would update the go client so that this request
https://github.com/cyberark/conjur-api-go/blob/b3f7d8f046f8d48d072fa028217d1add8a5a294e/conjurapi/router_v5.go#L141
sends the header, and this method base64 decodes it: https://github.com/cyberark/conjur-api-go/blob/b3f7d8f046f8d48d072fa028217d1add8a5a294e/conjurapi/variable.go#L27
then the secrets provider would just have to bump its go client version.

beauty!

beauty!


ci: skip smoke test on EOLed version 1.16

Skip the smoke test on branch release-1.16 for
v1.16 has been EOLed since December 21, 2022
Signed-off-by: Hai He<EMAIL_ADDRESS>

/ok-to-merge


Verify RSA-PSS Signatures in JS

Make sure that verifying RSA signatures still works with the updated cose-js lib.

Two PRs have been submitted to upstream:
https://github.com/erdtman/cose-js/pull/47
https://github.com/erdtman/cose-js/pull/48
Until then, the library depends on JesusMcCloud/cose-js:feature/rsa-pss


Add correlationId, requestId, and statusCode to default meta

This trio is a common refrain with the addition of an application operation (or logical thread) I'd.

correlationId added in v6.1.0


[BUG] OAuth2 auth code flow error redirects are not handled properly

Describe the bug:
This redirect URL results in a mostly blank page: https://localhost:8011/remote?tab=user&paneId=servicenow&paneTab=detail&status=error&messageTitle=Authorization%20Failed&message=Qorus%20error&messageDuration=7000
whereas this one works - the only difference is the status value: https://localhost:8011/remote?tab=user&paneId=servicenow&paneTab=detail&status=success&messageTitle=Authorization%20Failed&message=Qorus%20error&messageDuration=7000
To Reproduce:
check the URLs above (adjust for your Qorus / UI instance)
Expected behavior:
error message works
Screenshots or logs:

Desktop (please complete the following information):
Additional context

status must be danger, not error


DNSMOS API response

Hello,
Most of the requests to the DNSMOS API service is met with <Response [502]> error.
For very short clips (~3s), it is possible to get scores for few samples. However, the service seems to be a bit slow and gets no response in between.
I understand the issue has been raised before, but it still seems not resolved. I am not sure if anyone else still face this issue.
If I am missing something, please let me know.
Thanks!

@rohithmars same issue here. Quite painful to use this API to get a smooth experience for scoring ...

Is it possible to release a .exe file or the pre-trained model so that everyone can run the model on their local?

@JasonSWFu that's a great question! I'll assign the issue to @vishakg who's the main driving force behind the service. I am not sure if we'll be able to release the model, but I am pretty sure we can speed up the service and enable batch operation (as mentioned in #71). Stay tuned!

@vishakg Hello, At present,when I use the DNSMOS scoring tool,the response 502 is still returned to me,which is very unstable.Has it been improved now?thanks.

This issue is even worse for me. The <response 502> is returned without exception... @daxiong0

Hello,
Can anyone confirm whether this issue has been resolved?
@motus  @vishakg Is it possible to speed up the service? I still seem to get the 502 response.
Thanks a lot!

@daxiong0 , @wangshuo182 @rohithmars I have scaled up the service to handle the huge surge in requests we are seeing at the moment. Like I mentioned earlier, this service being a free service suffers from a noisy neighbor problem where too many requests from a single user result in denial of service to the others.


Douane Application Firewall

Project description
The Douane firewall filters and limits the outgoing network traffic per application.
You can allow network traffic for some applications and deny network traffic for others.
Metadata

homepage URL: https://douaneapp.com
source URL:https://gitlab.com/douaneapp/Douane/-/wikis/home
license: gpl2
platforms:  linux

Douane doesn't seem to be actively developed. Have you gave OpenSnitch a try? It's a little buggy so I haven't used it in a while but it worked for the most part.

I am using Opensnitch now, icmp is not working, but it is adapted in beta now.
I am closing this as you say it is not active. Thanks.


Update dependencies

Pull request
This PR updates the following dependencies to the currently latest versions:

scmdata
pandas (to version 2)
pyam-iamc
pytest

the biggest change is pandas 2 which broke a number of tests.
I have done my best to implement fixes related to deprecated pandas functionality.
There's still currently a couple of tests failing on my machine.
They are:

test_load_rewritten_scen7 failing because there's two warnings instead of one
test_writing_identical_rcpdat all the tests with add_extra_data = True are failing (fixed per 2af4eb1)
test_default_config and test_readme seem to be failing because I don't have wine installed. I tried to install it but somehow my antivirus marks it as a virus and does not let me install it.

[ ] Tests added
[ ] Documentation added (where applicable)
[ ] Example added (either to an existing notebook or as a new notebook, where applicable)
[ ] Description in CHANGELOG.rst added

@znicholls or @lewisjared I checked Allow edits by maintainers in case you have time to take a look. Could be easier for you to fix the remaining failing tests.

Awesome. I've got to do some other work but I can then take a look at getting the CI running again and see where we end up...

Looks like we'll have to way goodbye to python3.7 and 3.8 with this PR as scmdata 0.16.1 requires >=3.9.

Alright @phackstock here goes nothing

Thanks for your help getting this started, still lots to do but that's a job for another day


FormDataParam is treating as FormParam

Hi,
We are using latest swagger-maven-plugin. Our api's have both @FormParam and @FormDataParam. But swagger is treating both parameters as Form parameters. How to resolve this issue?

@ShrikanthAn Do you have an example which demonstrates the problem?

@POST
Response createProject(@NotNull @PathParam("spaceId") String spaceId,
@NotNull @FormDataParam("projectName")
@ApiParam(value = "The project name", required = true) String projectName)
In above example "projectName" swagger json is same @FormParam. I want this parameter as FormDataParam.


Handle cases where the database is in an inconsistent state

This can happen during replication, e.g. if a parent gets written before a child or a child gets written before a parent. I guess the proper thing to do would be to catch 404s and return nothing for that dependency.
I admit, though, that we are working against the grain of how CouchDB is designed.

I believe not saving the ids on the parent/hasMany side would remove some of these problems. #61 would save the relationships more like CouchDB is designed in my opinion.


Fix subscription tab please

Using TamperMonkey and userscript for Google Chrome.
The subscription tab does not load correctly and the thumbnails overlap.

Do you have grid subscriptions turned off in YTC settings?

Yes, the feature was turned on. Turning it off fixed the problem, thank you!


Ingawei/add contract firebase rules

I think I've moved most contract fetches to supabase so that firebase rules shouldn't impact anything, but not sure if there's some more things I've missed.
@jahooma , I've added marshall's realtime hooks if you'd like to take another look. @mqp if you get the chance I'd appreciate if you could check that I'm using these correctly

Also, note that the realtime channel subscription tools at a lower level than useSubscription are unreliable, in the sense that they will not deliver all changes in cases where the channel gets disconnected (they have no way of getting the ones that came in while they were disconnected.) So don't use them in a context where it would be very bad for there to be outdated data. For example, it would be inappropriate to use them in a naive way to keep the contract page contract up-to-date, because if you left the tab and came back, it would just show an outdated contract forever until a new update came in.

@jahooma I've addressed Marshalls concerns, pls look again


Added extra parameters for certain configurations

Extra parameters added as certain configurations by PAY.ON requires them.

Thanks!


SSL Pinning support

Summary
As part of some of the requirements by XXXXX we needed to add support for SSL Pinning when building your own app.
The way is being built is a combination of the app and the network-client library, the app will hold the certificate assets while the network-library will use these certs to verify the server trust.
The cert files should have either a .crt or .cer extension while the name of the file must be the domain name.
The reason we support two files extensions per domain is to be able to build the app taking into account server cert rotation and minimizing the amount of app distribution through updates in order to validate the certs.
This SSL Pinning certificates should be included in the app at build time.
Ticket Link
https://mattermost.atlassian.net/browse/MM-59055

@enahum Same deal with this PR, I am removing myself as a reviewer. Thanks for including me to observe and VERY slowly understanding all the changes that are happening here.

Sure @rahimrahman I can explain at some point in the near future about all this


The command ng serve is not responding to changes to ng module

🐞 Bug report
Command

[ ] new
[ ] build
[x] serve
[ ] test
[ ] e2e
[ ] generate
[ ] add
[ ] update
[ ] lint
[ ] xi18n
[ ] run
[ ] config
[ ] help
[ ] version
[ ] doc

Description
ng serve either is not responding to changes made to the declarations property of an NgModule.
🔬 Minimal Reproduction

ng new pokemon --routing=false --style=css
ng generate component pokedex
set up pokedex to be used by the root component
ng serve
remove pokedex from the declarations field of the NgModule in app.module.ts

Expected
The app crashes
Actual
The app continues working
Additional Information
Restarting ng serve fixes the issue.
This also works vice-versa, so if the application is crashing due to pokedex not being in the declarations of app.module.ts, adding it without restarting ng serve will still be crashing.
🌍 Your Environment

Demo
I've created a demo project here.
To reproduce this issue

git clone https://github.com/wagnermaciel/Pokedex.git
npm install
ng serve
Go to http://localhost:4200/pikachu
remove PokedexComponent from the declarations of the NgModule in src/app.app.module.ts

I think that's actually related to @angular/core and the rest of the framework being at rc.2. I know that there were some incorrect rebuild states in that version. Can you try with the latest RC?

@filipesilva Sorry, I'm still a little new here. Could you clarify what you mean by "the latest RC"? Based on the output from running ng version, I thought that all of my angular env is using rc.5

Yes rc.5 is the latest RC, but https://github.com/wagnermaciel/Pokedex/blob/master/package.json was using rc.2 so I just wanted to make sure. Will look into it, thanks!

Ah, got it. I'll take another look when I get the chance. Thank you!

I tried this out again and it seems to be fixed - somewhere between rc.5 and rc.8. There is still some weirdness where deleting a necessary declaration of a component will rebuild successfully (which is the unexpected/incorrect outcome) and then after a couple seconds (correctly) fail, but this is much more minor than the issue I initially observed.
@filipesilva I spoke with @dgp1130 and we concluded that although package.json says rc.2 it seems like my global version is what was getting used - in the above case rc.5 and currently rc.8. I believe this is because of the semver's in the angular version declarations in package.json.


ValidationCost increases after each pass

Hi I tried to train a model but seems the validation cost increase after each pass as below, is it normal?
[2018-03-21 00:00:10] ------- Time: 22227 sec,  Pass: 0, ValidationCost: 18.345830903
[2018-03-21 06:10:07] ------- Time: 22185 sec,  Pass: 1, ValidationCost: 18.3184596212
[2018-03-21 12:20:30] ------- Time: 22209 sec,  Pass: 2, ValidationCost: 17.5856525281
[2018-03-21 18:27:57] ------- Time: 22034 sec,  Pass: 3, ValidationCost: 18.2304718057
[2018-03-22 00:34:54] ------- Time: 22004 sec,  Pass: 4, ValidationCost: 17.9108289895
[2018-03-22 06:42:42] ------- Time: 22056 sec,  Pass: 5, ValidationCost: 18.7010796109
[2018-03-22 12:49:11] ------- Time: 21976 sec,  Pass: 6, ValidationCost: 19.2663258581
[2018-03-22 18:57:12] ------- Time: 22070 sec,  Pass: 7, ValidationCost: 19.5299177824
[2018-03-23 01:10:56] ------- Time: 22409 sec,  Pass: 8, ValidationCost: 20.5845607126
[2018-03-23 07:20:32] ------- Time: 22163 sec,  Pass: 9, ValidationCost: 20.757628163
[2018-03-23 13:28:11] ------- Time: 22045 sec,  Pass: 10, ValidationCost: 20.8393640953
[2018-03-23 19:41:08] ------- Time: 22363 sec,  Pass: 11, ValidationCost: 21.0724439657
[2018-03-24 01:53:30] ------- Time: 22327 sec,  Pass: 12, ValidationCost: 21.9896265413
[2018-03-24 08:01:36] ------- Time: 22073 sec,  Pass: 13, ValidationCost: 22.531096637
[2018-03-24 14:09:42] ------- Time: 22073 sec,  Pass: 14, ValidationCost: 22.3551265387
[2018-03-24 20:21:39] ------- Time: 22303 sec,  Pass: 15, ValidationCost: 23.8237918068
[2018-03-25 02:33:52] ------- Time: 22318 sec,  Pass: 16, ValidationCost: 24.2891556082
[2018-03-25 08:41:56] ------- Time: 22072 sec,  Pass: 17, ValidationCost: 24.6508512178
[2018-03-25 14:49:34] ------- Time: 22044 sec,  Pass: 18, ValidationCost: 25.2140230672
[2018-03-25 21:01:05] ------- Time: 22276 sec,  Pass: 19, ValidationCost: 23.3633104355
[2018-03-26 03:12:44] ------- Time: 22285 sec,  Pass: 20, ValidationCost: 27.1979689747
[2018-03-26 09:23:12] ------- Time: 22214 sec,  Pass: 21, ValidationCost: 23.4449897665
[2018-03-26 15:32:47] ------- Time: 22160 sec,  Pass: 22, ValidationCost: 24.0375935756
[2018-03-26 21:42:57] ------- Time: 22197 sec,  Pass: 23, ValidationCost: 24.7978525064
[2018-03-27 03:51:56] ------- Time: 22125 sec,  Pass: 24, ValidationCost: 24.6670613937
[2018-03-27 10:01:58] ------- Time: 22187 sec,  Pass: 25, ValidationCost: 25.072318309
[2018-03-27 16:17:09] ------- Time: 22496 sec,  Pass: 26, ValidationCost: 26.3841101534
[2018-03-27 22:32:06] ------- Time: 22482 sec,  Pass: 27, ValidationCost: 26.4974455705
[2018-03-28 04:45:17] ------- Time: 22377 sec,  Pass: 28, ValidationCost: 25.0781797638

May encounter overfitting. You can modify the learning rate to 1/10 of original learning rate and resume the training from the latest checkpoint.

thanks, the original learning rate is learning_rate=5e-4, so i shall change it to 5e-5?

Yes, you can do some tuning, like try 1e-4, 5e-5


Auth submit

I believe PR #40 should be merged before PR #41

Since this PR is based off of #40 I added a WIP label for clarity.

Not based off of, just should be done first. Why create a link to
registeration when it doesn't exist?
On Aug 17, 2017 9:13 PM, "Ricky Barillas"<EMAIL_ADDRESS>wrote:
Since this PR is based off of #40
https://github.com/pyatl/podium-django/pull/40 I added a WIP label for
clarity.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/pyatl/podium-django/pull/41#issuecomment-323234009,
or mute
the thread
https://github.com/notifications/unsubscribe-auth/ATx8wg6R2t7x5r3DY9Oeqk4IvbGxTE0fks5sZOUlgaJpZM4O6ooB
.

Well I said based off of since the first commit in this PR is the commit for the other PR. Also I'm a little confused by your other message. Was that a reply to my comment regarding the registration link?

Registration update is in PR #40, it includes all necessary code.
On Aug 17, 2017 10:24 PM, "Ricky Barillas"<EMAIL_ADDRESS>wrote:

@jbonzo commented on this pull request.
In podium/talks/templates/talks/submit.html
https://github.com/pyatl/podium-django/pull/41#discussion_r133863314:

  	</form>

	{%else%}

A verified email is required to submit a talk propasal, and you must be logged in.
 Please Register or Login.

In regards to the link tag for Register, where should the link go? Or is
the registration page not yet completed?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/pyatl/podium-django/pull/41#pullrequestreview-57093535,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATx8wsqZj_iK-W2SSYTI-iKfIyv8bPAkks5sZPXIgaJpZM4O6ooB
.

Right. I think we have a slight misunderstanding but I will wait to review this PR anymore until #40 is merged.

Was that a reply to my comment regarding the registration link?
No, It's my reason for making the initial comment, just in question form.
Nothing personal.
In regards to the link tag for Register, where should the link go? Or is the registration page not yet completed?
Registration update is in PR #40, it includes all necessary code, links and all.
Sorry for the confusion.

Closed to correct conflicts.


Fixes to reporting

Minor change to reduce the error to info for a method that checks if a page exists.
Split source and target urls

Related to #162

thx @pkbullock, good improvements. Especially the one about file exists being marked as an error in the logs :-)


Digicert CIS plugin calling missing endpoints?

While playing with the Digicert plugin, initially for certificate sync, found that Digicert endpoint is always refusing our REST api calls with "400 Unauthorized". We found that the plugin is calling endpoints such as:  "{0}/platform/cis/certificate/{1}" but couldn't find documentation for those in https://www.digicert.com/services/v2/documentation.
We might be missing something (either in the doc or in Lemur's code) but at this point I'm not sure if that API exists anymore in Digicert's side.
We're happy to try to fix things for that plugin if necessary but would like to know first if it makes sense to migrate those calls to the documented API.

Are you sure you want to use the CIS plugin? I believe the CIS api is a legacy API. I believe you want to use the the other digicert plugin that uses the cert central api (the one you linked to).

Well, I wasn't sure about what Digicert CIS was and couldn't find any reference to that API. My only motivation to use the CIS plugin was that it already implemented the SourcePlugin, whereas the current DigiCertSourcePlugin doesn't. I guess I can try to implement the missing method if that's fine. Thanks!


feat(RingTheory/Flat): flat is a local property of ring homomorphisms

We show that RingHom.Flat satisfies RingHom.PropertyIsLocal.

[x] depends on: #19786
[x] depends on: #19788

maintainer delegate

Thanks!
bors d+

Thanks for the reviews!
bors r+


[Improve][Doc] Add flow control parameters and instructions

Description
At present, the Common parameter has the description of the parallelism setting and dirty data setting, lacks the flow control parameter and description

we need to add  the description of the flow control in the common setting
reader_transport_channel_speed_byte         控制单并发读的流量,  X字节/S
reader_transport_channel_speed_record     控制单并发读的速度，X条/S
writer_transport_channel_speed_byte          控制单并发写的流量,   Y字节/S
writer_transport_channel_speed_record       控制单并发写的速度,   Y条/S
BitSail Component or Code Module
BitSail Doc
Are you willing to submit PR?

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

Code of Conduct

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

Hi there, I would like to finish this task.

great， look forward to your contribution @Fnck

Hi Fnck, thanks for your contribution to the BitSail community. we build a group for all contributor.Here is the QR code. If it expires, you can add WeChat ID<PHONE_NUMBER>7. Look forward to your arrival! @Fnck


CodeFactor PR check flags up problems on ontouched and unrelated lines/methods

As per title, the CodeFactor check makes most PRs red because it flags up issues that are unrelated to the PR.
This is due to the files containing a lot of other code. Neither do I want to change unrelated code as part of the PR nor seeing the PR checks as red, we should not have a '50 shades of red' build.
The team should clean up the warnings first or suppress some of the warnings since so far most of them are purely stylistic.
.Net Core has the Microsoft.CodeAnalysis.FxCopAnalyzers NuGet package, which is a subset of the CA warnings that most people are familiar with from the full .Net framework, I found those warnings more useful rather than having a linter check, since linters are more useful when being used in VS Code extensions

ref: #6930  #4708
I personally think CodeFactor should be disabled until it can be useful. That could be either after CodeFactor makes it more surgically applied to the PR changes or until the project has a cohesive style that rule and CI checks can actually be applied against.
I definitely agree with you on not wanted PS merged with failed checks. Right now CodeFactor is noise.

Currently CodeFactor is optional and don't block current PRs.
We have already opened #6930 to get feedback about CodeFactor. Please leave comments there what rules should be turned off or configured.

We continue to adjust CodeFactor setting step by step. Please discuss next steps in 4708.


David's branch

Still trying to get VS and GitHub desktop to behave, but for now I'm going to commit this from here.


Mechanics occ fixes

face edge contact issue fixed and joint with clearance added in occ_slider_crank.py
normalFromFace1 removed (normal is always on face2). offset2 added on second contact.

Thanks


Is there an tvm working example that use symbol and params generated by mxnet?

checkout https://github.com/dmlc/nnvm


Build for IOS not working

When I do npm install and pod update && pod install.. it works fine but when i try to run the ios app using npm run ios command... it errors out.
Logs attached here:
logs.txt

Same issue


using get.docker.com sh script when using Debian Jessie and apt-cache fails with 403

when Debian Jessie and using apt-cache the install curl -sSL https://get.docker.com/ | sh fails with the following output...
(output is coming out of Fabric)
<EMAIL_ADDRESS>out: Get:4 http://http.debian.net jessie-backports/main Translation-en/DiffIndex [1,609 B]
<EMAIL_ADDRESS>out:
<EMAIL_ADDRESS>out:
<EMAIL_ADDRESS>out: 100% [Sources 32.3 MB] [4 Translation-en 0 B/1,609 B 0%]
<EMAIL_ADDRESS>out:
<EMAIL_ADDRESS>out: 100% [Sources 32.3 MB]
<EMAIL_ADDRESS>out:
<EMAIL_ADDRESS>out: 100% [Working]
<EMAIL_ADDRESS>out:
<EMAIL_ADDRESS>out: 100% [Packages 0 B]
<EMAIL_ADDRESS>out:
<EMAIL_ADDRESS>out: 100% [Packages 33.8 MB]
<EMAIL_ADDRESS>out: 100% [Packages 33.8 MB]
<EMAIL_ADDRESS>out:
<EMAIL_ADDRESS>out: 100% [Working]
<EMAIL_ADDRESS>out:
<EMAIL_ADDRESS>out: 100% [Sources 0 B]
<EMAIL_ADDRESS>out:
<EMAIL_ADDRESS>out: 100% [Working]
<EMAIL_ADDRESS>out: 100% [Working]
<EMAIL_ADDRESS>out:
<EMAIL_ADDRESS>out: 100% [Packages 0 B]
<EMAIL_ADDRESS>out:
<EMAIL_ADDRESS>out: 100% [Working]
<EMAIL_ADDRESS>out:
<EMAIL_ADDRESS>out: 100% [Translation-en 0 B]
<EMAIL_ADDRESS>out:
<EMAIL_ADDRESS>out: 100% [Working]
<EMAIL_ADDRESS>out:
<EMAIL_ADDRESS>out: Fetched 4,090 B in 5s (764 B/s)
<EMAIL_ADDRESS>out: W: Failed to fetch https://get.docker.com/ubuntu/dists/docker/main/binary-amd64/Packages  Received HTTP code 403 from proxy after CONNECT
<EMAIL_ADDRESS>out:
<EMAIL_ADDRESS>out: E: Some index files failed to download. They have been ignored, or old ones used instead.
<EMAIL_ADDRESS>out: Reading package lists...
<EMAIL_ADDRESS>out: Building dependency tree...
<EMAIL_ADDRESS>out: Reading state information...
<EMAIL_ADDRESS>out: E: Unable to locate package lxc-docker
<EMAIL_ADDRESS>out:

Hi!
If you are reporting a new issue, PLEASE make sure that does not have any duplicates already open.
We would like to take this time to remind you of the information we need to debug the problem you are seeing. This is an automated response so if this ticket is not about a bug, do not fret.
If you fail to provide this information within 7 days, we will close this because we cannot debug your issue. We can reopen whenever the information is provided.
Thank you.
Please see:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues
Description of problem:

`docker version`:

`docker info`:

`uname -a`:

Environment details (AWS, VirtualBox, physical, etc.):

How reproducible:

Steps to Reproduce:
1.
2.
3.

Actual Results:

Expected Results:

Additional info:

#ENEEDMOREINFO

As a work around, removing apt-cache works.

Description of problem: using get.docker.com sh script when using Debian Jessie and apt-cache fails with 403
docker version: n/a
docker info: n/a
uname -a: Linux modev 3.16.0-4-amd64 #1 SMP Debian 3.16.7-ckt9-3~deb8u1 (2015-04-24) x86_64 GNU/Linux
Environment details (AWS, VirtualBox, physical, etc.): VirtualBox
How reproducible: Easy
Steps to Reproduce:

install apt-cache and configure /etc/apt/apt.conf.d/01aptcache

Acquire::http::Proxy "http://localhost:3142";

Run curl -sSL https://get.docker.com/ | sh

Actual Results: See log from initial bug report
Expected Results: Docker to be installed
Aditional info: none

i think this is because of your proxy


Rotate display using the lv_disp_drv_t rotated field.

Currently we rotate the display by swapping LV_HOR_RES_MAX and LV_VER_RES_MAX in lv_conf.h, we should use the rotated field in lv_disp_drv_t instead.
Is this correct @kisvegabor, @embeddedt ?

rotated is a bit of a strange flag at the moment - I don't completely understand how it works myself. I think the actual rotation still ends up being done by the display driver, not by LVGL itself, so I don't know if using the rotated flag really changes that much. I guess it might matter for rotating the screen on the fly.

I used it with the ILI9341 display I have, setting it to 1 rotates the display without any change on the driver. Do you mean that?

rotate only swaps the horizontal and vertical resolution of the display now and the real rotation really should be done in the display driver. In the future, we can have a set_orientation_cb in the display driver which can be called according to the current rotation settings.

rotate only swaps the horizontal and vertical resolution of the display now and the real rotation really should be done in the display driver.

This is the silly workaround I did on the lv_config.h file, swapping the resolutions (I didn't knew the rotate option back then). Do you think is sane to remove the workaround and use rotate instead?

Do you think is sane to remove the workaround and use rotate instead?

Yes, that's why we have rotate. :slightly_smiling_face:  (although it's very limited now).

Not stalled

on the backburner


how to print sql query log

I am using<EMAIL_ADDRESS>to serve vector tile , and I want to know how to print mapnik  sql query log when mapnik query data from postgis.
thanks

afaik the only way currently is to enable query logging in postgresql server
https://www.postgresql.org/docs/12/runtime-config-logging.html#GUC-LOG-STATEMENT

@lightmare  thank you so much


Prefer always adding the attachDevTools parameter in generated InitializeComponent of a Window

Is your feature request related to a problem? Please describe.
In Release mode, attachDevTools is ignored, regardless of the value of the parameter, as shown in the generated code of InitializeComponent:
#if DEBUG
            if (attachDevTools)
            {
                this.AttachDevTools();
            }
#endif

However, in Release mode, there is no such parameter generated.
If I want to change the keybind of the attached dev tools, I have to resort to manually disabling the automatic attachment of the dev tools and manually attach it with my options. Therefore, I have to set attachDevTools to false, which results in the following workaround:
#if DEBUG
        InitializeComponent(attachDevTools: false);
#else
        InitializeComponent();
#endif

Describe the solution you'd like
I'd like the attachDevTools to always be available, even when unused in Release mode. Optionally, a warning about the parameter being unused could be suppressed with #pragma warning disable XYZ in the generated code. This avoids the need for a conditional check as shown above.
Describe alternatives you've considered
No response
Additional context
No response

AttachDevTools is part of the DevTools package (ie. Avalonia.Diagnostics). Most people don't want to add extra debugging dependencies at runtime, so the conditional compilation is there to ensure it doesn't make it into release mode. DevTools is also incompatible with NativeAOT. So this is a reasonable default, IMO.
attachDevTools is maybe a bit poorly described for a parameter that's only intended to work in debug builds, but renaming this would be a breaking change.
The easiest workaround for both debug+release configs is probably to handle attaching yourself:
public MainWindow()
{
	InitializeComponent(attachDevTools: false);
	this.AttachDevTools();
}

I think you've completely missed the point. The parameter name can stay as is, all I'm asking for is to also add it in Release builds. No behavior change, just allowing me to refer to that parameter in Release mode because currently the parameter is unavailable.

attachDevTools is only available if Avalonia.Diagnostics is available. The source generator checks for this and generates different code based on its availability:
https://github.com/AvaloniaUI/Avalonia/blob/master/src/tools/Avalonia.Generators/NameGenerator/InitializeComponentCodeGenerator.cs#L27
The default project templates exclude DevTools in Release builds, so your project probably has something like:
<PackageReference Condition="'$(Configuration)' == 'Debug'" Include="Avalonia.Diagnostics" />

When it should be like this if you always want it:
<PackageReference Include="Avalonia.Diagnostics" />

Huh, I didn't know that. Then, wouldn't it be convenient to still add the parameter, regardless of whether the package is included?

Maybe it was done to restrict devs from using the parameter when DevTools isn't available and silently doing nothing (or throwing?). I originally thought it was always available, too, until I checked the source.


Fixed overlay observation wallet bug

What does this implement/fix? Explain your changes.
修复覆盖观察钱包时取消之后再次导入报文件已存在
Does this close any currently open issues?
If it fixes a bug or resolves a feature request, be sure to link to that issue.
OneKeyHQ/TaskHub#599
Pull request type

Put an x in the boxes that apply

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

Where has this been tested?
…
Any other comments?
…

@shuaiLiWang  帅姐，也帮忙review下

@shuaiLiWang  帅姐，也帮忙review下

@shuaiLiWang 帅姐，也帮忙review下
request reivewer 在右上角可以选

@shuaiLiWang 帅姐，也帮忙review下
request reivewer 在右上角可以选


feat: support managed envoy service type Loadbalancer & ClusterIP.

support #1202
support ClusterIP for cluster-internal proxy: just for cluster-internal
support Loadbalancer for edge proxy: can work for internal with eg svc and for internet directly
but doesn't support NodePort

Codecov Report

Merging #1205 (ff71a06) into main (3845773) will increase coverage by 0.07%.
The diff coverage is 95.12%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main    #1205      +/-   ##
==========================================
+ Coverage   63.25%   63.33%   +0.07%
==========================================
  Files          79       79
  Lines        9809     9830      +21
==========================================
+ Hits         6205     6226      +21
  Misses       3187     3187
  Partials      417      417

Impacted Files
Coverage Δ

internal/envoygateway/config/config.go
80.35% <60.00%> (ø)

api/config/v1alpha1/validation/envoyproxy.go
74.39% <100.00%> (+6.20%)
:arrow_up:

...nternal/infrastructure/kubernetes/proxy_service.go
83.92% <100.00%> (-1.08%)
:arrow_down:

internal/infrastructure/kubernetes/service.go
65.00% <100.00%> (+10.16%)
:arrow_up:

internal/xds/translator/cluster.go
100.00% <100.00%> (ø)

... and 3 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@arkodg pls review again, thanks

@qicz we will also need to fetch the right IP address and set it in the status here, for this implementation type to be conformant
https://github.com/envoyproxy/gateway/blob/384577334dea2c0ba28800643a2b94fea4eed244/internal/status/gateway.go#L32
and ensure conformance tests pass
https://github.com/envoyproxy/gateway/blob/main/docs/latest/dev/README.md#run-gateway-api-conformance-tests
can be done in this PR or a follow up (if so lets track it with an issue)

@qicz we will also need to fetch the right IP address and set it in the status here, for this implementation type to be conformant
https://github.com/envoyproxy/gateway/blob/384577334dea2c0ba28800643a2b94fea4eed244/internal/status/gateway.go#L32
and ensure conformance tests pass
https://github.com/envoyproxy/gateway/blob/main/docs/latest/dev/README.md#run-gateway-api-conformance-tests
can be done in this PR or a follow up (if so lets track it with an issue)

I will follow up with a new issue

@arkodg all case resolved pls review again

@qicz we will also need to fetch the right IP address and set it in the status here, for this implementation type to be conformant
https://github.com/envoyproxy/gateway/blob/384577334dea2c0ba28800643a2b94fea4eed244/internal/status/gateway.go#L32
and ensure conformance tests pass
https://github.com/envoyproxy/gateway/blob/main/docs/latest/dev/README.md#run-gateway-api-conformance-tests
can be done in this PR or a follow up (if so lets track it with an issue)

I will follow up with a new issue but there is no API about ClusterIP @arkodg

you can probably set the ClusterIP value


Fix for issue 2334 : VAD fails on short audio

What does this PR do?

Fixes #2334

Before submitting

[x] Did you read the contributor guideline?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[ ] Did you make sure to update the documentation with your changes? (if necessary)
[ ] Did you write any new necessary tests? (not for typos and docs)
[ ] Did you verify new and existing tests pass locally with your changes?
[x] Did you list all the breaking changes introduced by this pull request?
[x] Does your code adhere to project-specific code style and conventions?

PR review

Reviewer checklist

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[ ] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[x] Add labels and milestones (and optionally projects) to the PR so it can be classified
[x] Confirm that the changes adhere to compatibility requirements (e.g., Python version, platform)
[x] Review the self-review checklist to ensure the code is ready for review

Thank you @gaspardpetit


Add support for OkHttp library

As of Android 4.4, OkHttp became the engine that powers HttpUrlConnection. I want to keep Fetch dependency free as much as possible, but If you would like to use OkHttp, you can fork the project and easily replace HttpUrlConnection with OkHttp.


Update pool address

xdagmine.com:13654  is one of official pool addresses

I know that. But @iGod ask to use this one.

@swordlet Can you commit this ?


Spark on YARN requires HDFS and YARN

Running Spark on YARN requires the HDFS directory for the HistoryServer.
[2015-06-24T20:08:14+00:00] ERROR: ruby_block[initaction-create-hdfs-spark-userdir] (hadoop_wrapper::spark_historyserver_init line 24) had an error: Mixlib::ShellOut::ShellCommandFailed: execute[hdfs-spark-userdir] (hadoop::spark_historyserver line 25) had an error: Mixlib::ShellOut::ShellCommandFailed: Expected process to exit with [0], but received '1'
---- Begin output of hdfs dfs -mkdir -p hdfs://localhost/user/spark && hdfs dfs -chown -R spark:spark hdfs://localhost/user/spark ----
STDOUT:
STDERR: mkdir: Call From localhost/<IP_ADDRESS> to localhost:8020 failed on connection exception: java.net.ConnectException: Connection refused; For more details see:  http://wiki.apache.org/hadoop/ConnectionRefused
---- End output of hdfs dfs -mkdir -p hdfs://localhost/user/spark && hdfs dfs -chown -R spark:spark hdfs://localhost/user/spark ----

Putting hadoop-hdfs-namenode and hadoop-yarn-resourcemanager in runtime uses will force the spark-historyserver initialize to run after HDFS is started.

:+1:


ports other than 3000 are failing

Main problem 🙍
I'm using port 8080 everything works fine, the problem occurs when you access another route apparently an error occurs in the main port. This error is only presented using the constructor @now/node@canary.
NOTE: this problem does not occur if @tootallnate/node is used

How to reproduce it ? 🐛

clone the following repository

delete the folder .git to avoid compilation failures

run the command DEBUG=*fun* node /Users/path/now-cli/dist/index.js dev --debug --port 8080

This turned out to be a local npm caching issue. npm cache clean --force fixed the issue.
In reality, @tootallnate/node and @now/node@canary are the exact same code so this shouldn't have been happening, but the local npm cache got in the way.

@TooTallNate Thanks for help ! :)


Want to contribute. Looking for ideas.

Hi,
I was going through the paper:
https://arxiv.org/pdf/2403.11805v1
and was researching around it. I reached the profile of Mengwei Xu (https://xumengwei.github.io/index.html) and landed here. I am highly interested in this project and would like to contribute.
Is there anything that needs support or work that I can research and pick up?
I will be going through the code to see if anything can be updated or discussed.
Thanks a lot in advance!

Thanks for searching my profile :) In general mllm itself is an engineering-oriented system, atop which my team is validating and experimenting new research ideas. If you are passionate about on-device LLM, you can find things to do with mllm in our Roadmap. In general, I would suggest beginning with adding more model support (especially multimodal LLMs).
Regarding research, I probably cannot give any concrete ideas, but in general I believe LLM-as-a-system-service would open rich research opportunities in scheduling, reliability, persistence, and performance optimization. Have fun with it!

thanks a lot! I will go through the roadmap, and hopefully would be able to contribute to the project soon!


[firebase_messaging] Type mismatch: inferred type is PluginRegistry? but FlutterEngine was expected

Following the instructions on pub.dev for handling background messages doesn't compile for Android since the GeneratedPluginRegistrant.registerWith method takes a FlutterEngine object as a parameter. Using the PluginRegistry provided by the method throws the error
Type mismatch: inferred type is PluginRegistry? but FlutterEngine was expected

Please see the below code for reference
package ...

import io.flutter.app.FlutterApplication
import io.flutter.plugin.common.PluginRegistry
import io.flutter.plugin.common.PluginRegistry.PluginRegistrantCallback
import io.flutter.plugins.GeneratedPluginRegistrant
import io.flutter.plugins.firebasemessaging.FlutterFirebaseMessagingService

class Application: FlutterApplication(), PluginRegistrantCallback {
    override fun onCreate() {
        super.onCreate()
        FlutterFirebaseMessagingService.setPluginRegistrant(this)
    }

    override fun registerWith(registry: PluginRegistry?) {
        GeneratedPluginRegistrant.registerWith(registry) //Error thrown here
    }
}

To reproduce
Steps to reproduce the behavior:

Follow the steps in the given link up to step 1 of Optionally handle background messages
Build the project

Expected behavior
Somehow a FlutterEngine object should be passed to the method in order to successfully build for Android

Fixed by replace with:
class MainApplication : FlutterApplication(), PluginRegistrantCallback {

    override fun onCreate() {
        super.onCreate()
        FlutterFirebaseMessagingService.setPluginRegistrant(this);
    }

    override fun registerWith(registry: PluginRegistry?) {
        registry?.registrarFor("io.flutter.plugins.firebasemessaging.FirebaseMessagingPlugin");
    }

}

Hi @Jacobsjj2
does the solution proposed by @hsangtini  work for you?
thank you

This does allow me to compile, but I'm not sure if the background service is being registered correctly since I get the error
E/flutter (12123): [ERROR:flutter/lib/ui/ui_dart_state.cc(157)] Unhandled Exception: MissingPluginException(No implementation found for method FcmDartService#initialized on channel plugins.flutter.io/firebase_messaging_background)
E/flutter (12123): #0      MethodChannel.invokeMethod (package:flutter/src/services/platform_channel.dart:319:7)
E/flutter (12123): <asynchronous suspension>
E/flutter (12123): #1      _fcmSetupBackgroundChannel (package:firebase_messaging/firebase_messaging.dart:51:21)
E/flutter (12123): #2      _AsyncAwaitCompleter.start (dart:async-patch/async_patch.dart:45:6)
E/flutter (12123): #3      _fcmSetupBackgroundChannel (package:firebase_messaging/firebase_messaging.dart:24:32)
E/flutter (12123): #4      _runMainZoned.<anonymous closure>.<anonymous closure> (dart:ui/hooks.dart:239:25)
E/flutter (12123): #5      _rootRun (dart:async/zone.dart:1126:13)
E/flutter (12123): #6      _CustomZone.run (dart:async/zone.dart:1023:19)
E/flutter (12123): #7      _runZoned (dart:async/zone.dart:1518:10)
E/flutter (12123): #8      runZoned (dart:async/zone.dart:1502:12)
E/flutter (12123): #9      _runMainZoned.<anonymous closure> (dart:ui/hooks.dart:231:5)
E/flutter (12123): #10     _startIsolate.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:307:19)
E/flutter (12123): #11     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:174:12)

Doesn't the replacement simply return a Registrar object without registering anything?

This does allow me to compile, but I'm not sure if the background service is being registered correctly since I get the error
E/flutter (12123): [ERROR:flutter/lib/ui/ui_dart_state.cc(157)] Unhandled Exception: MissingPluginException(No implementation found for method FcmDartService#initialized on channel plugins.flutter.io/firebase_messaging_background)
E/flutter (12123): #0      MethodChannel.invokeMethod (package:flutter/src/services/platform_channel.dart:319:7)
E/flutter (12123): <asynchronous suspension>
E/flutter (12123): #1      _fcmSetupBackgroundChannel (package:firebase_messaging/firebase_messaging.dart:51:21)
E/flutter (12123): #2      _AsyncAwaitCompleter.start (dart:async-patch/async_patch.dart:45:6)
E/flutter (12123): #3      _fcmSetupBackgroundChannel (package:firebase_messaging/firebase_messaging.dart:24:32)
E/flutter (12123): #4      _runMainZoned.<anonymous closure>.<anonymous closure> (dart:ui/hooks.dart:239:25)
E/flutter (12123): #5      _rootRun (dart:async/zone.dart:1126:13)
E/flutter (12123): #6      _CustomZone.run (dart:async/zone.dart:1023:19)
E/flutter (12123): #7      _runZoned (dart:async/zone.dart:1518:10)
E/flutter (12123): #8      runZoned (dart:async/zone.dart:1502:12)
E/flutter (12123): #9      _runMainZoned.<anonymous closure> (dart:ui/hooks.dart:231:5)
E/flutter (12123): #10     _startIsolate.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:307:19)
E/flutter (12123): #11     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:174:12)

Doesn't the replacement simply return a Registrar object without registering anything?

Try this replace @Jacobsjj2
    override fun registerWith(registry: PluginRegistry?) {
  io.flutter.plugins.firebasemessaging.FirebaseMessagingPlugin.registerWith(registry?.registrarFor("io.flutter.plugins.firebasemessaging.FirebaseMessagingPlugin"));
    }

Sorry for the delay! @hsangtini's latest suggestion works. I guess it would also be necessary to update the documentation to that then?

I also had this problem, I changed the "Application.kit" to the suggested one. and now it's compiling, I just haven't tested the notifications yet.

How is your android/app/src/main/AndroidManifest.xml?
Do you import? both of them?
        <activity
            android:name=".MainActivity"
// something

        <activity
            android:name=".Application"
        />

Hey all 👋
As part of our roadmap (#2582) we've just shipped a complete rework of the firebase_messaging plugin that aims to solve this and many other issues. Additionally, messaging now supports v2 embedding.
If you can, please try out the dev release (see the migration guide for upgrading and for changes) and if you have any feedback then join in the discussion here.
Given the scope of the rework I'm going to go ahead and close this issue in favor of trying out the latest plugin.
Thanks everyone.


Consider utilizing a SASS map for variables

Since we have a very very hefty variables partial, it may be beneficial organization-wise to utilize SASS maps in some areas, with an accompanying function to pluck values from that map (See http://codepen.io/jakealbaugh/post/using-sass-functions-to-access-complex-variable-maps for a great explanation on how to accomplish this.)
In areas like https://github.com/JDRF/design-system/blob/master/src/scss/_variables.scss#L71 and pretty much any other variable that is prefixed with $font- we can employ a SASS map such as the following:
$font: (
	weight: (
		thin: "Gotham Thin", sans-serif,
		light: "Gotham Light", sans-serif,
		light-italic: "Gotham Light Italic", sans-serif,
		regular: "Gotham Book", sans-serif,
		italic: "Gotham Book Italic", sans-serif,
		medium: "Gotham Medium", sans-serif,
		medium-italic: "Gotham Medium Italic", sans-serif
	),
	size: (
		size-lg: 0.875rem, /* 14px */
		size-sm: 0.8125rem, /* 13px */
		size-h1: 2rem, /* 32px */
		size-h1-sm: 2.25rem, /* 36px */
		size-h1-md: 3rem, /* 48px */
		size-h2: 1.5rem, /* 24px */
		size-h2-sm: 1.75rem, /* 28px */
		size-h2-md: 2.25rem, /* 36px */
		size-h3: 1.25rem, /* 20px */
		size-h3-sm: 1.5rem, /* 24px */
		size-h4: 1.25rem,
		size-h5: 1rem,
		size-h6: 0.875rem
	)
);

This approach will hopefully help to keep our variables partial nice and tidy!

@magnificode
I gave this a try locally and realized some of the obstacles we'll have if we went down this route.
The sass map variables work only for files where we're both overwriting the variables as well as the styles - as in they're disconnected from their Bootstrap imports. For components we haven't disconnected or variables we aren't overwriting, in order to pursue this path, we'd have to not only change all variables we're overwriting from Bootstrap, but disconnect all components from their imports. This way we can control where those variables are called and replace it with get_map.
Therefore due to the complexity and possibility that we may have to disconnect a lot of styles even if we're not overwriting, for the sake of maintainability, it makes sense not to use sass map vars for the DS at this point in time.

@RachelRVasquez totally understandable, thanks a bunch for the explanation!


Remove bespoke SendMessage implementation

Problem
Copying the kncloudevents.Dispatcher smells a bit, and I have a proposal to remove it.
We can implement our own WithRetryConfig and WithDeadLetterSink options which control the dispatcher based on current state. The one thing I have a solution for, but 50/50 if I like it, is signalling when to nak/term messages...
I have this bit of code in WIP:
func ReportWithoutRetry(result *error, attempt int, cfg *kncloudevents.RetryConfig) *kncloudevents.RetryConfig {
	if cfg == nil {
		return nil
	}

	next := *cfg

	next.CheckRetry = func(ctx context.Context, resp *http.Response, err error) (bool, error) {
		retry, err := cfg.CheckRetry(ctx, resp, err)

		if retry {
			if cfg.Backoff != nil {
				multierr.AppendInto(result, NewDelayNak(cfg.Backoff(attempt, resp)))
			} else {
				multierr.AppendInto(result, protocol.ResultNACK)
			}
		}

		return false, err
	}

	return &next
}

We take in the RetryConfig defined on the subscription, and decorate it with our own CheckRetry function which always returns false but sets a protocol.ResultNACK error. You can then use it like so:
var retryError error

defer func() {
	err = msg.Finish(multierr.Append(err, retryError))
}()

...

dispatchExecutionInfo, err := c.dispatcher.SendMessage(
	ctx,
	internal.WithoutFinish(msg),
	c.sub.Subscriber,
	internal.WithRetryReporter(&retryError, attempt, c.sub.RetryConfig),
	internal.WithDeadLetterOnLastAttempt(attempt, c.sub.RetryConfig, c.sub.DeadLetter),
	kncloudevents.WithReply(c.sub.Reply),
	kncloudevents.WithTransformers(&te),
)

The finish method of the message (see #542) then understands how to check the errors to determine whether to nack, and if so what delay.
At the moment, my dead-letter handling will just ack the message. I'm not sure if this should result in a Term or not?
Persona:

System Integrator

Exit Criteria
No copy/paste of kncloudevents.
Time Estimate (optional):
1 day

/remove-lifecycle stale


zsh gives resty:34: parse error near `)'

Same

Same here, i think it has something to do width zsh shell, as when sourcing the file under bash it doesnt give the same error (OSX El Captain here)

It has something to do with "$()" command subtitution and embedded newlines.  I was able to fix it by putting it all in one line.  If resty works with stock zsh, then it should be fixed.  I've got a few zsh modules loaded and am not sure if they're causing the problem.  My "git diff" of resty:
diff --git a/resty b/resty
index 87fbed4..8fce400 100755
--- a/resty
+++ b/resty
@@ -29,8 +29,7 @@ function resty() {
   host="$datadir/host"
   cookies="$datadir/c"
   method="$1"; [[ $# > 0 ]] && shift
-  h2t=$((exec 2>&-; (which lynx >/dev/null && echo lynx -stdin -dump) \
-                || which html2text || which cat) |tail -n 1)
+  h2t=$((exec 2>&-; (which lynx >/dev/null && echo lynx -stdin -dump) || which html2text || which cat) |tail -n 1)
   editor=$((exec 2>&-; which "$EDITOR" || which vim || echo "vi") |tail -n 1)

   [ "${method#P}" != "$method" ] || [ "$method" = "TRACE" ] && wantdata="yes"
@@ -82,11 +81,7 @@ function resty() {
       [ -n "$dat" ] && opt="--data-binary"
       [ "$method" = "HEAD" ] && opt="-I" && raw="yes"
       [ -f "$confdir/$domain" ] && eval "args2=( $(cat "$confdir/$domain" 2>/dev/null |sed 's/^ *//' |grep ^$method |cut -b $((${#method}+2))-) )"
-      res=$((((curl -sLv $opt "$dat" -X $method \
-              -b "$cookies/$domain" -c "$cookies/$domain" \
-              "${args2[@]}" "${curlopt2[@]}" "${curlopt[@]}" "$_path$query" \
-        |sed 's/^/OUT /' && echo) 3>&2 2>&1 1>&3) \
-        |sed 's/^/ERR /' && echo) 2>&1)
+      res=$((((curl -sLv $opt "$dat" -X $method -b "$cookies/$domain" -c "$cookies/$domain" "${args2[@]}" "${curlopt2[@]}" "${curlopt[@]}" "$_path$query" |sed 's/^/OUT /' && echo) 3>&2 2>&1 1>&3) |sed 's/^/ERR /' && echo) 2>&1)
       out=$(echo "$res" |sed '/^OUT /s/^....//p; d')
       err=$(echo "$res" |sed '/^ERR /s/^....//p; d')
       ret=$(echo "$err" |sed \


ReferenceError: ResizeObserver is not defined

We have raised the OL version in our project from 6.14.1 to 7.2.2 and changed the imports from "ol/PluggableMap" to "ol/Map".
When we start our project in the browser everything runs as it should, but in our tests we get the following error in various places:
ReferenceError: ResizeObserver is not defined

     97 | }
     98 |
   > 99 | const map = new Map({})
        | ^

Does anyone have an idea what is missing here?

Your tests are probably running on JSDom, where ResizeObserver, which we use now to be able to resize the map in all possible scenarios, is not supported. You'll need to use a polyfill for that. See e.g. https://stackoverflow.com/questions/64558062/how-to-mock-resizeobserver-to-work-in-unit-tests-using-react-testing-library.

Thanks for the tip. Following code before creating the map has solved our problem.
window.ResizeObserver =
  window.ResizeObserver ||
  jest.fn().mockImplementation(() => ({
    disconnect: jest.fn(),
    observe: jest.fn(),
    unobserve: jest.fn()
  }))

const map = new Map({})


Crash: Fatal Exception: NSRangeException

[REQUIRED] Step 1: Describe your environment

Xcode version: 11.3.1
Firebase SDK version: 6.14.0
Firebase Component: Core, Database
Component version: 6.14.0
Installation method: Carthage

[REQUIRED] Step 2: Describe the problem
Steps to reproduce:
We're observing a crash that's slowly increasing when executing this code block:
database.purgeOutstandingWrites()
app.delete()

Stack Trace
Fatal Exception: NSRangeException
*** -[__NSArrayM objectAtIndexedSubscript:]: index<PHONE_NUMBER>854775807 beyond bounds for empty array

Fatal Exception: NSRangeException
0   CoreFoundation                      0x00007fff23c7127e __exceptionPreprocess + 350
1   libobjc.A.dylib                     0x00007fff513fbb20 objc_exception_throw + 48
2   CoreFoundation                      0x00007fff23d03ab1 _CFThrowFormattedException + 194
3   CoreFoundation                      0x00007fff23b8572e -[__NSArrayM objectAtIndexedSubscript:] + 142
4   Foo                                 0x0000000103c22a0c -[FWriteTree removeWriteId:] + 171
5   Foo                                 0x0000000103c0b2bd -[FSyncTree ackUserWriteWithWriteId:revert:persist:clock:] + 166
6   Foo                                 0x0000000103bf9a1d -[FRepo ackWrite:rerunTransactionsAtPath:status:] + 212
7   Foo                                 0x0000000103bf876b __39-[FRepo update:withNodes:withCallback:]_block_invoke + 114
8   Foo                                 0x0000000103bf1361 -[FPersistentConnection ackPuts] + 343
9   Foo                                 0x0000000103bf0de5 -[FPersistentConnection purgeOutstandingWrites] + 54
10  Foo                                 0x0000000103bf9144 -[FRepo purgeOutstandingWrites] + 287
11  Foo                                 0x0000000103bd37be __37-[FIRDatabase purgeOutstandingWrites]_block_invoke + 52
12  libdispatch.dylib                   0x000000010d90fdd4 _dispatch_call_block_and_release + 12
13  libdispatch.dylib                   0x000000010d910d48 _dispatch_client_callout + 8
14  libdispatch.dylib                   0x000000010d9175ef _dispatch_lane_serial_drain + 788
15  libdispatch.dylib                   0x000000010d91817f _dispatch_lane_invoke + 422
16  libdispatch.dylib                   0x000000010d923a4e _dispatch_workloop_worker_thread + 719
17  libsystem_pthread.dylib             0x00007fff524636fc _pthread_wqthread + 290
18  libsystem_pthread.dylib             0x00007fff52462827 start_wqthread + 15

Looks like the purgeOutstandingWrites call does some async work on [FIRDatabaseQuery sharedQueue] and we should wait that out when an app is deleted.
This should likely be done in the following: https://github.com/firebase/firebase-ios-sdk/blob/9bfbb9ece0d9a7094a890696d663a20cf8627e85/Firebase/Database/Api/FIRDatabaseComponent.m#L85-L100
@min: are you doing anything after app.delete()? It does have a completion handler you can provide that's called when all cleanup is done for each SDK.
Also it's always helpful to hear what the use case for app.delete() is so we can better understand what developers are doing with it.
Thanks!

@ryanwilson we’re basically calling reset to clear out the session when a user logs out. wondering if the purge is even necessary.

It's likely not for the moment to unblock you, but something we should still try to fix :)

@ryanwilson thanks!

@ryanwilson Should the label be changed to database?

Great call - not sure why Googelbot thought it was auth...


Failed to create CvCapture

When calling these line of code
FrameCapture = Cv.CreateFileCapture(ConfigurationManager.AppSettings["VideoFile"].ToString());
in App.config
" add key="VideoFile" value="C:_OpenCV\pos.avi""
showing following error
************** Exception Text **************
OpenCvSharp.OpenCvSharpException: Failed to create CvCapture
at OpenCvSharp.CvCapture..ctor(String filename)
at OpenCvSharp.Cv.CreateFileCapture(String filename)
at PositiveBuilder.Form1.SetupVideo() in e:\Shared\Riyas\Projects\Image Recognition\opencv_positive_builder_64bit\opencv_positive_builder\PositiveBuilder\Form1.cs:line 134
at PositiveBuilder.Form1.Form1_Load(Object sender, EventArgs e) in e:\Shared\Riyas\Projects\Image Recognition\opencv_positive_builder_64bit\opencv_positive_builder\PositiveBuilder\Form1.cs:line 60
at System.Windows.Forms.Form.OnLoad(EventArgs e)
at System.Windows.Forms.Form.OnCreateControl()
at System.Windows.Forms.Control.CreateControl(Boolean fIgnoreVisible)
at System.Windows.Forms.Control.CreateControl()
at System.Windows.Forms.Control.WmShowWindow(Message& m)
at System.Windows.Forms.Control.WndProc(Message& m)
at System.Windows.Forms.Form.WmShowWindow(Message& m)
at System.Windows.Forms.NativeWindow.Callback(IntPtr hWnd, Int32 msg, IntPtr wparam, IntPtr lparam)

C:_OpenCV\pos.avi -> C:\OpenCV\pos.avi ??
The problem could be due to wrong file path or lack of video codec.


Setting firehose output causes errors in tests

Background
In global.json under "infrastructure": { at
https://github.com/airbnb/streamalert/blob/73d2c9aed65e3d3d24a3389a70f73f9c59fbad23/conf/global.json#L9
add
"firehose": {
      "enabled": true,
      "s3_bucket_suffix": "streamalert.data",
      "buffer_size": 64,
      "buffer_interval": 300,
      "compression_format": "GZIP"
    }

as described on https://streamalert.readthedocs.io/firehose.html
Then run ./tests/scripts/rule_test.sh this results in the following error repeated:
backoff [ERROR]: Backing off firehose_request_wrapper(...) for 0.9s (ResourceNotFoundException: An error occurred (ResourceNotFoundException) when calling the PutRecordBatch operation: Stream streamalert_data_systemlogs under account<PHONE_NUMBER>12 not found.)
StreamAlertShared [INFO]: [Backoff]: Trying again in 0.907155 seconds after 1 tries calling firehose_request_wrapper

Desired Change
Allow the tests to run.  I'm still trying to figure out how to resolve this.

@0xdabbad00 are you on latest?

Looks like this is not an issue with the latest code.  I'll sync our local repo with upstream and hopefully that resolves it.  We had just sync'd last week.

I updated to latest and still ran into the issue.  Eventually, I figured out the issue seems to be due to us not running us-east-1.  Change the region in conf/global.json to us-west-2, and you'll run into the same issue on the latest code.  This indicates there seems to be a dependency in the test code on us-east-1.

I set region_name to us-west-2 here and the tests run fine now: https://github.com/airbnb/streamalert/blob/f6b5f6e4296090ded404c55fa9a64d2e5eaacce9/stream_alert/rule_processor/handler.py#L122
so I'm trying to figure out where that gets set.

I changed the code at https://github.com/airbnb/streamalert/blob/82b6119c6a8b2fa3a9e62f576fd833a182f0e814/stream_alert_cli/helpers.py#L478 to:
if not cluster:
        region = config['global']['account']['region']
        context.invoked_function_arn = (
            'arn:aws:lambda:{}:123456789012:'
            'function:test_streamalert_processor:development').format(region)

Now the tests work.  Want me to send a PR for that short change? I don't know what all else this impacts.

@0xdabbad00 sounds good


redirect tmp dir from java

java storing tmp dir for USeq to iCLIP dir - need to redirect to output to ensure dir size is sufficient

resolved with commit 8e6d88177192e4bbf9cde636da73c8326bf06936

resolved with commit 8e6d88177192e4bbf9cde636da73c8326bf06936


new console mode $_WD_DEBUG_Full

Feature request
Is your feature request related to a problem? Please describe
A clear and concise description of what the problem is. For example: I'm always frustrated when [...] .
Describe the solution you'd like
I would like to add new enum:
$_WD_DEBUG_Testing
in this way:
Global Enum _
		$_WD_DEBUG_None = 0, _ ; No logging to console
		$_WD_DEBUG_Error, _    ; logging to console in case of Error
		$_WD_DEBUG_Info, _     ; logging to console with additional information
		$_WD_DEBUG_Testing     ; logging to console with information for the developer of WebDriver UDF

and such function:
Func __WD_ConsoleTesting($sMsg, $iError = @error, $iExtended = @extended)
	If $_WD_DEBUG = $_WD_DEBUG_Testing Then __WD_ConsoleWrite('TESTING: ' & $sMsg)
	Return SetError($iError, $iExtended)
EndFunc   ;==>__WD_ConsoleWrite

Which will be possible to use instead:
If Not @Compiled Then ConsoleWrite("! IFNC: TESTING NEW FEATURES #" & @ScriptLineNumber & $s_Parameters_Info & @CRLF)

Describe alternatives you've considered
none
Additional context
I know that there will be many other place to check/change
For example it will be needed to check all If $_WD_DEBUG = $_WD_DEBUG_Info Then
Also in Func __WD_Error
		Case $_WD_DEBUG_Error, $_WD_DEBUG_Testing
			If $iErr <> $_WD_ERROR_Success Then ContinueCase

If you accept this FR then I create PR.

From your statement, I conclude that you accept the change.
I will prepare an appropriate amendment (PR) taking into account your sugestions.

Maybe $_WD_DEBUG_Full or $_WD_DEBUG_Debug

$_WD_DEBUG_Debug sounds strange, I choose $_WD_DEBUG_Full

Perhaps you meant to add the new debugging mode to the existing Case $_WD_DEBUG_Info line?

I think you are right, thanks for pointing it out.
I just didn't have a chance to test it because it was just a concept.

I would like to do also something like:
Func __WD_ConsoleInfo($sMsg, $iError = @error, $iExtended = @extended)
	If $_WD_DEBUG = $_WD_DEBUG_Info Then __WD_ConsoleWrite($sMsg)
	Return SetError($iError, $iExtended)
EndFunc   ;==>__WD_ConsoleInfo

to avoid multile usage :
	If $_WD_DEBUG = $_WD_DEBUG_Info Then
		__WD_ConsoleWrite($sFuncName & .............)
	EndIf

I would like to do also something like: Func __WD_ConsoleInfo(

Instead of adding another function, have you considered modifying __WD_ConsoleWrite to support an optional debugging level parameter? Something like this --
Func __WD_ConsoleWrite($sMsg, $iDebugLevel = Default, $iError = @error, $iExtended = @extended)
	If $iDebugLevel = Default Or $_WD_DEBUG >= $iDebugLevel Then
		If IsFunc($_WD_CONSOLE) Then
			Call($_WD_CONSOLE, $sMsg & $_WD_CONSOLE_Suffix)
		ElseIf $_WD_CONSOLE = Null Then
			; do nothing
		Else
			FileWrite($_WD_CONSOLE, $sMsg & $_WD_CONSOLE_Suffix)
		EndIf
	EndIf
	Return SetError($iError, $iExtended)
EndFunc   ;==>__WD_ConsoleWrite

Now this --
	If $_WD_DEBUG = $_WD_DEBUG_Info Then
		__WD_ConsoleWrite($sFuncName & ': ' & $sResponse)
	EndIf

would become this --
	__WD_ConsoleWrite($sFuncName & ': ' & $sResponse, $_WD_DEBUG_Info)

I would also propose to modify Func __WD_Error
in this part:
		Case $_WD_DEBUG_Info
			Local $sExtended = (($iExt <> Default) ? (" / " & $iExt) : (""))
			$sMsg = $sWhere & " ==> " & $aWD_ERROR_DESC[$iErr] & " (" & $iErr & $sExtended & ")"
			If $iExt = Default Then $sMsg &= (($_WD_HTTPRESULT) ? (" HTTP status = " & $_WD_HTTPRESULT) : ("")) ; _WD_LastHTTPResult()
			$sMsg &= (($sMessage) ? (" : " & $sMessage) : (""))
			__WD_ConsoleWrite($sMsg)

by moving message processing to inner of __WD_ConsoleWrite()

by moving message processing to inner part of __WD_ConsoleWrite()

I'm not sure that I understand. Can you show an example of your proposed change? Also, explain why this change is needed or the benefit it provides.

......
......
......
Func __WD_Error($sWhere, $iErr, $sMessage = Default, $iExt = Default)
	Local Const $sFuncName = "__WD_Error"
......
......
......
		Case $_WD_DEBUG_Info
			__WD_ConsoleWrite($sMsg, $iErr, $iExt)
......
......
......
EndFunc __WD_Error
......
......
......
......
Func __WD_ConsoleWrite($sWhere, $sMsg, $iErr = @error, $iExt = @extended)
	Local $sExtended = (($iExt <> Default) ? (" / " & $iExt) : (""))
	$sMsg = $sWhere & " ==> " & $aWD_ERROR_DESC[$iErr] & " (" & $iErr & $sExtended & ")"
	If $iExt = Default Then $sMsg &= (($_WD_HTTPRESULT) ? (" HTTP status = " & $_WD_HTTPRESULT) : ("")) ; _WD_LastHTTPResult()
	$sMsg &= (($sMessage) ? (" : " & $sMessage) : (""))

	If IsFunc($_WD_CONSOLE) Then
		Call($_WD_CONSOLE, $sMsg & $_WD_CONSOLE_Suffix)
	ElseIf $_WD_CONSOLE = Null Then
		; do nothing
	Else
		FileWrite($_WD_CONSOLE, $sMsg & $_WD_CONSOLE_Suffix)
	EndIf
	Return SetError($iErr, $iExt)
EndFunc   ;==>__WD_ConsoleWrite
......
......
......

Also, explain why this change is needed or the benefit it provides.

unification, of messages even when you use:
	If $_WD_DEBUG = $_WD_DEBUG_Info Then
		__WD_ConsoleWrite($sFuncName & ': ' & $sResponse)
	EndIf

I'm not in agreement on this change.

ok.
so is it correct to use:
__WD_Error($sFuncName, @error, "TESTING NEW FEATURES #" & @ScriptLineNumber & $s_Parameters_Info , @extended)

instead:
If Not @Compiled Then ConsoleWrite("! IFNC: TESTING NEW FEATURES #" & @ScriptLineNumber & $s_Parameters_Info & @CRLF)

which together with:
Func __WD_Error($sWhere, $iErr, $sMessage = Default, $iExt = Default)
......
......
......
		Case $_WD_DEBUG_Info, $_WD_DEBUG_Full
......
......
			__WD_ConsoleWrite($sMsg)
......
......
......
EndFunc __WD_Error

do the same job as $iDebugLevel which was proposed by you in this modyfication:
Func __WD_ConsoleWrite($sMsg, $iDebugLevel = Default, $iError = @error, $iExtended = @extended)

Yes, that should work. However, you would still need to modify __WD_ConsoleWrite as proposed by me if you want to achieve the following --

to avoid multiple usage :
If $_WD_DEBUG = $_WD_DEBUG_Info Then
__WD_ConsoleWrite($sFuncName & .............)
EndIf

Now that I think about this further, it seems like you are wanting to use __WD_Error for general logging, which isn't its intended purpose. Why not use the proper function, which is __WD_ConsoleWrite?

ok so we stick with:
Func __WD_ConsoleWrite($sMsg, $iDebugLevel = Default, $iError = @error, $iExtended = @extended)

and then I do all my test and I will see if it fits all my needs.
Ok ?

oh...
I forgot that in such case I will not have this feature:
Local $sExtended = (($iExt <> Default) ? (" / " & $iExt) : (""))
$sMsg = $sWhere & " ==> " & $aWD_ERROR_DESC[$iErr] & " (" & $iErr & $sExtended & ")"
If $iExt = Default Then $sMsg &= (($_WD_HTTPRESULT) ? (" HTTP status = " & $_WD_HTTPRESULT) : ("")) ; _WD_LastHTTPResult()
$sMsg &= (($sMessage) ? (" : " & $sMessage) : (""))

for this reason I was proposing to move them to inner procesing of __WD_ConsoleWrite()

You are trying to make __WD_ConsoleWrite into an extension of __WD_Error, which is not its intended use.

No. I want to have __WD_Error as it is and use them only in Return SetError(__WD_Error($sFuncName, .....)

And as you've pointed out in your recent edit, it would require a change to every existing instance of __WD_ConsoleWrite, which doesn't make sense to me.

this is not a problem for me to do all the changes.

No. I want to have __WD_Error as it is and use them only in Return SetError(__WD_Error($sFuncName, .....)

That's not the impression I got when you asked the following --
Is it correct to use:
__WD_Error($sFuncName, @error, "TESTING NEW FEATURES #" & @ScriptLineNumber & $s_Parameters_Info , @extended)
instead:
If Not @Compiled Then ConsoleWrite("! IFNC: TESTING NEW FEATURES #" & @ScriptLineNumber & $s_Parameters_Info & @CRLF)

To me, that is you asking if it's ok to use __WD_Error instead of a more general logging option.

I just need unification for all messages generated by __WD_ConsoleWrite()

I'm not sure that everyone would want or need this. IMO, every message written to a log doesn't need to contain error codes, HTTP status, etc.
Could you accomplish this task using a custom logging function assigned to $_WD_CONSOLE?

I think yes. I accompilish this task in this way I mean still using __WD_ConsoleWrite()

$_WD_DEBUG_Full support is only paritailay PR for this PR.
Later I will add new PR related to this:

I'm not sure that everyone would want or need this. IMO, every message written to a log doesn't need to contain error codes, HTTP status, etc.
Could you accomplish this task using a custom logging function assigned to $_WD_CONSOLE?

I think yes. I can accompilish this task in this way I mean still using __WD_ConsoleWrite()

#279 should be marked as Draft because:

it makes changes to __WD_ConsoleWrite that are beyond the scope of what we discussed above
it doesn't correctly adjust the existing usage of __WD_ConsoleWrite to the new implementation

it makes changes to __WD_ConsoleWrite that are beyond the scope of what we discussed above

I revert them back

it doesn't correctly adjust the existing usage of __WD_ConsoleWrite to the new implementation

Do not understand, please elaborate.

I revert them back

Do not understand what you mean, please elaborate.

I haven't checked your "revert", but what I was trying to indicate above was that there were instances of __WD_ConsoleWrite in other sources files (ie: wd_core, wd_helper, etc) that no longer functioned after your prior revision.

I haven't checked your "revert", but what I was trying to indicate above was that there were instances of __WD_ConsoleWrite in other sources files (ie: wd_core, wd_helper, etc) that no longer functioned after your prior revision.

Ok.
No problem with reverting.
I will take a look again on this maybe in one of next PR ;)


Fix ARM build

Hi, currently this doesn't build on M1 Mac. This has been fixed in libpg_query in this commit.
I see that the current commit does not have this fix yet. Can you publish a new release with the latest patches from libpq_query?
If it is as simple as just updating the commit in that file, happy to send a PR too.
Thanks!

Thanks for the help! @inian please check if the latest works for you @13.1.1

Works now. Thanks a lot Dan!

@pyramation this is fine for people running node in Rosetta, but if we want to run this natively, it seems currently you are only generating a single binary for Darwin? I was able to compile it myself using the instructions on this repo, but figure it may be easier to just have it done here?

pardon my experience with Rosetta, is this the new mac?
yea we should probably do a build and push the binary to npm. However, I don't have a rosetta machine so we'll have to collaborate here somehow.

@pyramation Just messaged you on Keybase

@inian Just confirming, are you running this in Rosetta or not?

this is fine for people running node in Rosetta, but if we want to run this natively, it seems currently you are only generating a single binary for Darwin

Is this still an open issue? When I install + use libpg-query via SafeQL on an M1 (process.arch === 'arm64'), it is failing with the following error:
dyld[...]: missing symbol called

Also reported downstream in SafeQL here: https://github.com/ts-safeql/safeql/issues/65
Or maybe this is unrelated?

Hmm... actually getting this again now. It seems to be an edge case related to running the installation in a VS Code "Intel Chip" terminal - even though the Node.js version installed on the machine is an arm64 version.
Reproduction steps:

Open an "Intel Chip" version of VS Code on an M1 and open the terminal (the installed version of Node.js should be arm64 architecture)
Clone https://github.com/upleveled/next-js-example-fall-2022/tree/994747b1d24298ff8b27926e34792d7faecdd39f (commit hash 994747b1d24298ff8b27926e34792d7faecdd39f)
Run yarn to install libpg-query and other dependencies - the installation will succeed
Run yarn eslint . --max-warnings 0 - this will fail with the dyld[...]: missing symbol called error as in the output below

$ yarn eslint . --max-warnings 0
yarn run v1.22.19
$ /Users/k/projects/next-js-example-fall-2022/node_modules/.bin/eslint . --max-warnings 0
dyld[30868]: missing symbol called
error Command failed with signal "SIGABRT".
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

So the solutions / workarounds for this are:
A) Install an "Apple Silicon" or "Universal" version of VS Code
or
B) Run the installation in a terminal running in arm64 mode


binding rework

Opening this PR to make it a bit easier to talk about the changes

splitting this one up into smaller prs ;)


Arm64 Support

Describe the Enhancement

It would be awesome if this pack supports ARM64 builds. I know there is a ongoing discussion from the buildpack community around this topic, and currently we don't have a general solution, but I did have success in build for ARM64 a spring boot application using the following builder:
tasks.bootBuildImage {
    builder.set("dashaun/builder:tiny")
    buildpacks.set(listOf( "urn:cnb:builder:paketo-buildpacks/java"))
}

But if a set the newrelic pack, I get this error: fork/exec /cnb/buildpacks/paketo-buildpacks_new-relic/7.6.1/bin/detect: exec format error
Which means this pack is not compiled with a ARM64 architecture. This is the config used:
tasks.bootBuildImage {
    builder.set("dashaun/builder:tiny")
    buildpacks.set(listOf( "urn:cnb:builder:paketo-buildpacks/java", "docker.io/paketobuildpacks/new-relic"))
    bindings.set(listOf(project.rootDir.absolutePath + "/buildpack/bindings/new-relic:/platform/bindings/new-relic"))
}

Check https://github.com/dashaun/paketo-arm64 for this work around.
Steps to reproduce: Create a project in https://start.spring.io/ just with spring web dependency for test.
Add the configuration above in build.gradle
Run the command:
./gradlew bootBuildImage

This will generate a optimized container using the the spring boot buildings, verified its working on Oracle Cloud ARM64 Machine!
Possible Solution

I'm not sure about the specifics, but maybe just release a image builded in arm64 emulator or native machine and it can work. Please check docker buildx for cross compilation in docker.
Motivation

ARM64 are cheaper and available in many mainstrain cloud providers. Developer machines on apple silicon will also apreciate.
I have moved all my kubernetes cluster to ARM64 and I'm experimenting with buildpacks to build a more optimized image, but lack of oficial support for ARM64 is a blocker. I'm considering not using new relic, or setup in a different workaround (maybe editing the final image?) or maybe do not use buildpacks for the moment, but It is close to work for my specific case!

We are not providing ARM64 builds right now. It is on the roadmap and we working towards it as quickly as possible. Because this is a larger workstream for us, I'm going to close out this issue. You can follow the Paketo blog, we'll absolutely be posting more as we make progress with ARM64 support.
In the meantime, you can build and package your own buildpacks as ARM64 buildpacks using the instructions in this repo: https://github.com/dmikusa/paketo-arm64#package-your-buildpacks. It's the find general steps at the top, you can also see the scripts for the commands being run (although the script is not meant to build individual buildpacks). That is roughly the same process that @dashaun follows with his builder, he just doesn't package up this particular buildpack.
The other option would be to manually install/include the New Relic agent for the time being. I don't know exactly what that process would look like though. If you can get Maven/Gradle to bundle in the required JAR files with your executable JAR, you might be able to pass through the additional arguments in the JAVA_TOOL_OPTIONS env variable at runtime to enable the agent.


RequestIdTooLong due to concatenating ids

the partner id and the requestid are concatenated cf code)
I am getting
200 RequestIdTooLong - Request Id Too Long, Max allowed length is 19
could we just use the provided request id? what is the rationale behind concatenating both ids?

Yeah, same issue as described in #2, will make a change when I got some time, thanks!

Fixed through https://github.com/listia/aws_agcod/commit/c468e1a66779e2af333e1208e7622b42b5a3c862

The string is still long in my case. why should we concatenate the partner_id to the provided request id at all? why don't we "ask don't tell" here by letting the client manage the request id unicity as he sees fit?

In https://s3.amazonaws.com/AGCOD/tech_spec/AGCODTechSpec.pdf
VI. AGCOD Service: Real-Time Request-Responses > 1. Definitions > 2. creationRequestId

A unique identifier for every CreateGiftCard call that results in the creation of a
gc. The partner must generate a new value for every Create request (except for retries). To keep this
a globally unique transaction identifier, the partner needs to:

 Generate an alphanumeric value that is unique within the partner’s systems
 Use the partnerId as a prefix to the …RequestId <<

Note: that the total size limit on the field is 19 chars
If a request comes in with this value repeated, the AGCOD Gateway will return the original status
that was created the first time the creationRequestId was seen.

This is actually the reason why I choose to prefix partner id as part of the creation request id.

Thank you for the quick answer and for the great libraray :+1:
on another topic: I don't see how we can access the expiration_date here?

on another topic: I don't see how we can access the expiration_date here?

Yeah... I don't actually want to expose everything, but that sounds like a good attribute to add, can you make a PR or feature request as an issue? Thanks!


Installer: Unexpected token <

I installed the development version of pagekit, but when I load up the site, it gives me the error: Uncaught SyntaxError: Unexpected token <
Chrome dev tools points to the <!DOCTYPE html> when giving the error.

Did you run gulp succesfully ?
Where URL did you visit ?
More info about your error, myself see that error too, but disappear after reload.

We released the Pagekit Beta today. I close this issue because the code base completely changed. Please open a new issue if it still exists.


Why is profiler dir deleted for non-chief nodes in MWMS in keras?

Hi,
Based on the keras tensorboard callback: https://github.com/tensorflow/tensorflow/blob/master/tensorflow/python/keras/callbacks.py#L2472
The profiler dir of non-chief node is set to a temp dir when MWMS is being used, which will be deleted later on:
https://github.com/tensorflow/tensorflow/blob/master/tensorflow/python/keras/callbacks.py#L2227
https://github.com/tensorflow/tensorflow/blob/da15b34d11432b789527e4a3c3ad4eac5193a5f0/tensorflow/python/keras/callbacks.py#L2256
I understand checkpoints for non-chief nodes should be cleaned up, but why profiling result?
Happy to contribute a PR if this is a bug.
CC @jbaiocchi  @omalleyt12

@burgerkingeater,
Neither _log_write_dir nor log_dir is a temporary directory but they are the actual directories in which Tensorboard Events are stored.
Please let me know if I have understood your question correctly.
Thanks!

@rmothukuru thanks! But _log_write_dir is a temp dir for MWMS's non-chief node: https://github.com/tensorflow/tensorflow/blob/cda12732ca4610434937259b881965165564193d/tensorflow/python/keras/distribute/distributed_file_utils.py#L96

@burgerkingeater thanks for the question. With MWMS the workers are expected to have the same metrics with all-reduces. Is there a reason why the logging on chief alone isn't sufficient?

@rchao thanks for the reply. Trace view in the profiler is not necessarily to be the same across the nodes, right? E.g, two nodes are running with different hardware configuration

@burgerkingeater I see. Would you mind giving a try to not remove the profiling on non-chiefs (and re-build TensorFlow), and see if it matches what you're expecting?

CC myself @yisitu

@rchao yes, and I submitted a PR to fix this issue: https://github.com/tensorflow/tensorflow/pull/50068/, would you mind taking a look?

@rchao also I wonder why not just not writing summary if it's non-chief? Why is the overhead of creating the temp dir, writing summary to temp dir, and deleting the temp dir necessary?

Thanks @burgerkingeater! With MWMS, any API calls can trigger all-reduces across the workers and if we only perform that on chief, there is a chance that things will just hang (because it keeps waiting to all-reduce with other workers). Therefore in general all workers (including chief) should perform the same actions, but if it involves file writing to a centralized place, we'd need temporary locations for non-chief for later deletion.

Related PR has been merged , closing this issue. Thank you


Add ohai_time to minimal_ohai filter

We need ohai_time to get the 'Last Check-In' time on the Chef Server.
There is really no risk to including it.

breaking CI is unrelated :(


Problem with plugin in my cordova application

Hi, in a first place sorry for my english...
I have two questions:

If I want to install the plugin in my app, I must execute cordova plugin add [dir_where_git_repo_was_cloned]\TestNFCReaderACR35\acr35\plugin . I'm right?

Then, I try to execute my app in an android device (Samsung Pocket with Android v 4.0.4) and call the function acr35.read(), nothing happens. But when I execute the app in browser platform (Chrome) and press F12 to see the developer tools, in the javascript console an error appears indicating "Uncaught ReferenceError: acr35 is not defined" when I want to call the function acr53.read(). I followed the steps you write in repo description to install the plugin, and when I compare my app to yours, I do not see anything weird.

What could be happening @stuart-xyz ?

The error was generated because there were two  referencing two javascript CDNs.


1.0.8

add Context with storage for getting psr request and response after making request
add tests

Codecov Report
Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (2b84b78) 79.60% compared to head (2d6e4d7) 85.21%.
Report is 3 commits behind head on main.

Files
Patch %
Lines

src/HTTP/Context/ContextStorage.php
82.35%
3 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff              @@
##               main       #9      +/-   ##
============================================
+ Coverage     79.60%   85.21%   +5.61%
- Complexity       65       75      +10
============================================
  Files            12       14       +2
  Lines           201      230      +29
============================================
+ Hits            160      196      +36
+ Misses           41       34       -7

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


Fix for private chef group add actor for 'user'

Private chef bombs on the ID for users and the fact it wants to check on clients/groups.  Not sure if this is the right way to do this, but it works for our private chef installation.  Current knife-acl bomps like so for 'knife group add actor admins username':
INFO: HTTP Request Returned 500 Internal Server Error: Client pivotal does not exist
ERROR: Server returned error for https://chefserver/organizations/infrastructure-56m/groups/admins, retrying 3/5 in 13s
If I just remove the clients section, I get:
INFO: HTTP Request Returned 500 Internal Server Error: Users 3029dc16617811e298d590b11c089a5b do not exist
ERROR: Server returned error for https://chefserver/organizations/infrastructure-56m/groups/admins, retrying 1/5 in 4s
It seems private chef wants the username, not the ID?  This works well against a test installation, but wanted to get an official review before using it.

@jtolsma hi, we'd love to get this Pull Request merged, but as Seth said a couple of years ago we need you to sign our Contributor License Agreement (CLA). There are a lot more details on our documentation site.

Hi thommay, is this still applicable to newer versions of private chef servers?  I have not used it since 2013 and cannot test it.

@thommay @jtolsma We likely want to make sure this is fixed in the improvements @jeremiahsnapp has worked on here: https://github.com/chef/knife-acl/pull/19

Hello @jtolsma thanks a lot for your PR. Sorry it's been overlooked for such a long time. knife-acl hasn't received a lot of attention until recently. I recently made some significant changes/improvements to knife-acl and released them as version 1.0.0.beta. I'm not having any problems adding users to groups using the following command syntax.
knife group add user test-user admins

If you or anyone else if still having problems adding users to groups using the 1.0.0.beta version please let me know. In the meantime I'm going to close this PR


Since the update 3.2 the agenda doesn't work

Search first

[X] I searched and no similar issues were found

What Happened?
I have started Logseq and after the update the quick access to agenda on my Logseq (the icon star) is not longer in the bar and it seems it isn't running at all as I cannot access the calendar, nor anything.
Thank you
Reproduce the Bug

I updated agenda from 3.1 to 3.2
The icon for access the agenda is gone after.
The agenda is intalled
I restarted Logseq. Nothing happens.
When I click on previous dates and times nothing happens.

Screenshots

Environment Information
Windows 10, Logseq 0.9.20, Agenda 3.2
Additional Context
No response
Are you willing to submit a PR? If you know how to fix the bug.

[ ] I'm willing to submit a PR (Thank you!)

Only the pinned icon will be displayed on toolbar. You can take a look.

Hi Hayde,
Yes, that didn’t appeared until later in the afternoon. Maybe it was a conflict that fixed itself.
Thank you so much for your answer and your amazing contribution.
Have a great weekend!
Guadalupe
On Nov 10, 2023 at 21:49 -0500, Hayden Chen @.***>, wrote:

Only the pinned icon will be displayed on toolbar. You can take a look.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


Edited image is not saving when I click check mark

After I make edits to the image, when I click the check, the edited image is not saving. After some print debugging, it seems the issue is that the ImageEditorOption object is not actually storing any changes and just return the same exact picture, but I have no clue how to resolve it. Can someone help? Thanks.

@PhoenixPhighter Okay, I am taking a look at this, and will update here if it's solved.


Enhance submit votes button for when it is submittable

Currently, it is green and faded out when it is disabled. While this is neat enough for me, kids are kids, and they may not understand that they need to submit their votes using the dedicated button.
Therefore, I should make it great when disabled, green when available and green faded away when submitting.
Also, to further improve discoverability, I should add a subtle animation of the button shrinking and growing when available to submit, with sometimes few "bell jiggles" animations just to take their eyes' attention.

I've done a quick demo for an animation, here's my results :
button {
  animation: available 6s;
  animation-iteration-count: infinite;
}

@keyframes available {
    0% { transform: scale(1); }
    5% { transform: scale(0.97); }
    10% { transform: scale(1.07); }
    15% { transform: scale(0.97); }
    20% { transform: scale(1.07); }
    25% { transform: scale(0.97); }
    30% { transform: scale(1.07); }
    35% { transform: scale(0.97); }
    40% { transform: scale(1.07); }
    45% { transform: scale(0.97); }
    50% { transform: scale(1.07); }
    55% { transform: scale(0.97); }
    60% { transform: scale(1.07); }
    65% { transform: scale(0.97); }
    70% { transform: scale(1.07); }
    75% { transform: scale(0.97); }
    80% { transform: scale(1.07); }
    85% { transform: scale(0.97); }
    90% { transform: scale(1) rotate(0deg); }
    92% { transform: scale(1) rotate(-2deg); }
    94% { transform: scale(1) rotate(2deg); }
    96% { transform: scale(1) rotate(-2deg); }
    98% { transform: scale(1) rotate(2deg); }
    100% { transform: scale(1) rotate(0deg); }
}


Doesn't work correctly with react-native-gesture-handler's NativeViewGestureHandler

Current behavior
FlashList is intercepting gesture events
Expected behavior
FlatList doesn't intercept these when using NativeViewGestureHandler and simultaneous handlers. Expect FlashList to behave the same. I believe this is in the scope of FlashList,  as I think that the underlying ScrollView is not being passed all the props. Also looking for any workaround if anyone has any ideas.
To Reproduce
Minimal repro comparing FlashList and FlatList behaviour:
https://github.com/JackShannon/flashlistgesture
Try scrolling up and down the list, the PanGestureHandler, the simultaneous handler won't activate with vertical gestures.
Change line 6 in App.js to:
const USE_FLASHLIST = false
Using FlatList the simultaneous handler will activate.
Platform:

[X] iOS
[ ] Android

Environment

1.2.1

You can pass your own ScrollView as renderScrollComponent and try.

Same problem on v 1.2.1. I have a GestureDetector wrapped around FlashList (see https://github.com/4TWIGGERS/react-native-fresh-refresh). GestureDetector is not firing any events. It does when I use FlatList.
Any workarounds? I tried using renderScrollComponent, without success (although I am not sure how it could help in my case).

I have a workaround of sorts that worked for our needs.
Used useAnimatedScrollHandler and used the contentOffset for the scrollY value, with bounce enabled. This replaced the translateY value that we wanted to get from the gesture handler. We had a requirement that our scrollview shouldn't have a bounce, so had to hide that this was happening by translating the top list item by negative scroll value when it is below 0.
Hope this helps.

Try using this as a workaround
https://github.com/software-mansion/react-native-gesture-handler/issues/2175

software-mansion/react-native-gesture-handler#2175

using ScrollView from react-native-gesture-handler in renderScrollComponent works for me


Update ROS repository key instructions

Closes #224
This updates the instructions for adding the ROS repository key to the most recent instructions here: http://wiki.ros.org/noetic/Installation/Ubuntu#Installation.2FUbuntu.2FSources.Set_up_your_keys.
Signed-off-by: Christophe Bedard<EMAIL_ADDRESS>

Thanks for the patch. I rebased it after merging #227 which fixed the unrelated CI failures.


Exact amount weirdness with multiple Insert cards

LaserIO version
1.5.2
Minecraft Version
1.19.2
Forge Version
43.2.14
Modpack & Version
All The Mods 8
Do you have optifine installed?
No
Describe the issue
Extract Card extracts half of the set exact amount of fluid when connected to two Inserter Cards
Steps to reproduce

Three fluid containers with a shared Laser node
One has Extract Card set to 2000 amount (with Overclocker), set to Exact
The other two have Inserter Cards with default settings
Placing one bucket of water (1000mB) will result in the fluid being extracted and inserted into one of the other tanks despite being half of the exact amount specified

https://github.com/Direwolf20-MC/LaserIO/assets/103251857/bdb2b6de-4a57-44d4-9b8b-bae4cf82edbf
Expected behaviour
One bucket of water shouldn't be extracted if the exact amount is set to 2000
Screenshots
No response
Log files
No response
Additional information
No response

This should be resolved in the next version, thank you for the detailed report! :)


Can you provide the import function of SolidWorks model?

I'm not involved with the repo at all so I can't say for certain, but I think that's probably pretty unlikely, though maybe STEP import is more likely which you should be able to export from SolidWorks.

Hi @WREQI, thanks for opening this! OCCT will allow me to implement the import for STEP and IGES, but not for SolidWorks directly as pointed out by @Irev-Dev. I suppose SolidWorks can export to these formats, but I understand that this, depending on your use case, might be either acceptable or not.
I did not yet implement Import just yet because of a generic approach I'm taking towards asset management and this project in general. I am currently building a cloud backend for bitbybit which will allow users to create, share and remix projects of others. This will also allow them to persist assets in cloud storage associated to their projects so that you would not need to re-import those files every time you restart the browser. In principle this will work for step, iges, obj, stl, txt and other formats that might be needed in the future...  This will provide a generic approach to file management on the app.
This does not mean that the whole app will be closed behind the signup, but it does mean that for those more advanced scenarios requiring persistant storage and a database you would need an account.
I can't say exactly when this cloud integration will be finished, but I'm hoping to have first release somewhere this month. It already ate more time than I imagined, so I'm sorry that I can't make a big promise just now.
I hope this answers the question for now. When I'll have more info about Importing assets I will update this thread.

Hi @WREQI and @Irev-Dev, it took a while and I'm not sure if this is still relevant to you, but with the latest release we support import of assets in STEP and IGES file formats. I know it's not direct Solidworks support, but I've prepared a short video showing how STEP exports/imports work between Rhino and Bit by bit.
https://www.youtube.com/watch?v=TTAsTwruDNU
Also be aware that large files having a lot of faces can take a long time to load or even crash the browser ;) I'll be looking at performance improvements a bit later.