Enable the ability to strip leading zeros on the NumeralFieldFormatter

Hello! Im using the NumeralFieldFormatter to display the current value in Brazilian Reais, however im currently trying to format the value in order to display the leading zeros. For example if the user types in '10' it should format to 0,10 representing 10 cents. Cleave.js has a boolean value in the NumeralFieldFormatter called stripLeadingZeroes which by default it is set to true, it would be great to have this option on the TextFieldFormatter addon.

Fixed in 4.1


font-size-adjust is supported by firefox, but there is no information about other browsers.

font-size-adjust
is supported by firefox, but there is no information about other browsers.

This issue was imported from Google Moderator
Moderator votes: +3
Issue added by David Scheiber-Jakob on 2013-04-25
To vote this issue up or down, simply include +1 or -1 in your comment.

Already added at http://caniuse.com/#feat=font-size-adjust


Should make clear custom claim needs to be name with extension_ prefix

I tried to do the example using field myOwnId. The trustframework files loaded OK, but on loading profileedit.xml page it came up with error.  Fixed by adding extension_ prefix to myOwnId.

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

ID: 29fb9816-df6a-6f60-35d0-c2a04732f295
Version Independent ID: ffc8b13a-b2c9-a502-1580-abea596194ab
Content: Add your own attributes to custom policies in Azure Active Directory B2C
Content Source: articles/active-directory-b2c/active-directory-b2c-create-custom-attributes-profile-edit-custom.md
Service: active-directory
Sub-service: b2c
GitHub Login: @mmacy
Microsoft Alias: marsma

Hi @tomkan08 Thank you for your feedback! We will review and provide an update as appropriate.

We tested this internally and it worked as described in the article for us. If you got any error then it may require extended troubleshooting. I would suggest you to send us an email on azcommunity[at]microsoft[dot]com along with your azure subscription id and we can help you further. We will close this issue for now and would wait for your email on the same in case you still have an error as we were unable to replicate the same on our end.
Thank you .


Support  in pyvespa

In order to run https://github.com/vespa-engine/sample-apps/tree/master/multi-vector-indexing as a pyvespa app, one must be able to add this to services.xml:
        <component id="e5-small-q" type="hugging-face-embedder">
            <transformer-model path="model/e5-small-v2-int8.onnx"/>
            <tokenizer-model path="model/tokenizer.json"/>
        </component>

Discussed a little with @lesters , decided to add support for adding a generic Component to an app
CC @jobergum

Added in https://github.com/vespa-engine/pyvespa/pull/548


this.$refs.scaleTree.zoomOut is not a function

when i want to handle zoomout or zoomin, it casts 'this.$refs.scaleTree.zoomOut is not a function', why?

Did you ever figure out how to zoom on the tree chart?

@wmo123 It turns out that the current build in library/vue-tree-chart.js is a build of older code not the current code with the zoom functionality. @ssthouse just needs to run webpack on the current code to update the build. Then you should be able to use the zoom functionality.

hi, I have published a new version 0.4.3, and write a simple demo to verify it:
https://codesandbox.io/s/wispy-water-x65ct?file=/src/App.vue
The zoom API should be working now.


[zh-CN]Update translation(By December 7)

Hello! I've finished updating the translation! #14
I hope you will continue to maintain the launcher and add more content!
Thank you for your development!

Hi @misaka10843,
Thank you for the update.
I will look into it once I'm back home 👍

Hi @misaka10843,
Sorry for my very late reply on this since I'm being busy with my RL job recently.
It's looking good for me, so I think we can merge this Update right away.


Do not lose focus while typing in product description field

I want to merge this change because it fixes very annoying bug causing input to lose focus.
Fixes #4470
Pull Request Checklist

[ ] Privileged views and APIs are guarded by proper permission checks.
[ ] All visible strings are translated with proper context.
[ ] All data-formatting is locale-aware (dates, numbers, and so on).
[ ] Database queries are optimized and the number of queries is constant.
[ ] Database migration files are up to date.
[ ] The changes are tested.
[ ] The code is documented (docstrings, project documentation).
[ ] GraphQL schema and type definitions are up to date.
[x] Changes are mentioned in the changelog.

Here is the report for 5435dd5040eb4779afca5c3ef34a7b6885c507f3 (mirumee/saleor @ fix/product-description)
Base comparison is cf43e0e604c88a84606e8cc0fd6bee84bb660079.
No differences were found. (click me)

# api.benchmark checkout
  test name                           	left count 	right count	duplicate count
  ------------------------------------	-----------	-----------	---------------
  add billing address to checkout     	         34	         34	             20
  add shipping to checkout            	          7	          7	              0
  checkout payment charge             	         14	         14	              0
  complete checkout                   	          6	          6	              0
  create checkout                     	         41	         41	             20

# api.benchmark homepage
  test name                           	left count 	right count	duplicate count
  ------------------------------------	-----------	-----------	---------------
  retrieve main menu                  	          5	          5	              0
  retrieve product list               	          4	          4	              0
  retrieve secondary menu             	          5	          5	              0
  retrieve shop                       	          2	          2	              0

# api.benchmark product
  test name                           	left count 	right count	duplicate count
  ------------------------------------	-----------	-----------	---------------
  product details                     	         13	         13	              3

# api.benchmark variant
  test name                           	left count 	right count	duplicate count
  ------------------------------------	-----------	-----------	---------------
  retrieve variant list               	          9	          9	              2

Codecov Report

Merging #4549 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4549      +/-   ##
==========================================
- Coverage   90.64%   90.62%   -0.02%
==========================================
  Files         297      297
  Lines       17437    17437
  Branches     1748     1748
==========================================
- Hits        15805    15803       -2
- Misses       1121     1122       +1
- Partials      511      512       +1

Impacted Files
Coverage Δ

saleor/payment/utils.py
90.47% <0%> (-0.87%)
:arrow_down:

Continue to review full report at Codecov.

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


Pop not working correctly in multiple stack in tabs

Version
"react-native-router-flux": "^4.0.1",
"react-native": "0.54.4",
I have 2 tabs with stack and scene
Tab1
Stack
Scene 1-1
Scene 1-2
Tab2
Stack
Scene 2-1
Scene 2-2
I go to scene scene 1-1 => scene 2-1 => scene 1-2
then i use pop function
Expected behaviour
I expect go to scene 2-1
Actual behaviour
router go to scene 1-1
because scene 1-1 & scene 1-2 is in same stack and scene 2-1 in another stack
I cant reopen #3238 @daviscabral
I upgrade my router flux to 4.0.1 version but my problem yet exist in pop navigation

Sorry, but your desired behavior is not supported - you are manipulating with scenes from different stacks and tabs are automatically switched.


[Request] Draw checkbox before label instead of after

Is usually annoying how the checkbox is drawn and how often it hides the field label:

There's an expensive serializer that can draw the checkbox before the label with the attribute [ToggleLeft] (image taken from their site):

Is possible to add this to Naughty Attributes?
Thanks in advance, your open source project helps a lot of people like me who live in countries where an unity asset can cost half of the average people monthly income.

Unity's editor UI is working that way - the checkbox is on the right. I can probably make something about it, but I don't think it's worth it. The way to do it is to create a custom property drawer for boolean fields


docs Upload Doris-Streamloader docs of CN/EN Version

Proposed changes
Issue Number: close #xxx

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

run buildall


send_action's pipe_io.writeline error

Hi! I'm interested on replicating your paper results but I'm hitting a problem when sending action values to the pipeline.
Below are some information about my efforts:

OS: Ubuntu 18.04.02 (on VirtualBox, host is Windows 10)
Version: EnergyPlus v8.8.0
Patch: Followed the patch instructions using 8.8.0 patch version and it seems to be working well (except for this problem)
Command: poetry run time python3 -m baselines_energyplus.trpo_mpi.run_energyplus --num-timesteps 10000000

Please see the traceback message below.

From the traceback, it seems that line 213 from energyplus_env.py formats the action into a string. However, the action value contains a list, hence the error. Is this really the expected behavior or did I miss a step?
I hope for your help as I'm really eager to test your testbed out!
Thank you.

Hi @JerelynCo. Can you print/debug the content of action as received at https://github.com/IBM/rl-testbed-for-energyplus/blob/bba0be7413680bd3f80b11f76cb6d21dec1d5145/gym_energyplus/envs/energyplus_env.py#L181
Also, what version of gym and baselines are you using?

Hi @JerelynCo. Can you print/debug the content of action as received at https://github.com/IBM/rl-testbed-for-energyplus/blob/bba0be7413680bd3f80b11f76cb6d21dec1d5145/gym_energyplus/envs/energyplus_env.py#L181
Also, what version of gym and baselines are you using?

Hi @antoine-galataud , thanks for your quick response!
Here's the screenshot of action inside step function:

As for my dependencies version:

Just a quick note: I'm using Poetry as my package manager. I also had a problem installing baselines (mujoco-py problem) using the pypi version and so I installed from the baselines repository: https://github.com/openai/baselines
Kindly let me know if you need more information.
Thanks!

Hi @antoine-galataud , thanks for your quick response!
Here's the screenshot of action inside step function:

As for my dependencies version:

Just a quick note: I'm using Poetry as my package manager. I also had a problem installing baselines (mujoco-py problem) using the pypi version and so I installed from the baselines repository: https://github.com/openai/baselines
Kindly let me know if you need more information.
Thanks!

Hi @antoine-galataud , I tried versions 9.1 and 9.2 for the mean time. However, I encounter different problems. I understand that the details below might be better posted in a separate issue, but the problems I'm having might be connected to each other (perhaps library incompatibility issue?).
For 9.1
It seems that rl-testbed-for-energyplus repo doesn't contain model objects for 9.1.
https://github.com/IBM/rl-testbed-for-energyplus/tree/master/EnergyPlus
As a workaround, I copied the Model-8-8-0 to Model-9-1-0 and updated the idf file version to 9.1.

However, I encountered problem as described in the eplusout.err file:

For 9.2
In this case, I encountered a patching warning.

I continued to run a simulation anyway and encountered the problem with the same message as this issue https://github.com/IBM/rl-testbed-for-energyplus/issues/29.
It might be because of the patching warning I got.
If you have any idea, please let me know. Thank you!

Hi @antoine-galataud , I tried versions 9.1 and 9.2 for the mean time. However, I encounter different problems. I understand that the details below might be better posted in a separate issue, but the problems I'm having might be connected to each other (perhaps library incompatibility issue?).
For 9.1
It seems that rl-testbed-for-energyplus repo doesn't contain model objects for 9.1.
https://github.com/IBM/rl-testbed-for-energyplus/tree/master/EnergyPlus
As a workaround, I copied the Model-8-8-0 to Model-9-1-0 and updated the idf file version to 9.1.

However, I encountered problem as described in the eplusout.err file:

For 9.2
In this case, I encountered a patching warning.

I continued to run a simulation anyway and encountered the problem with the same message as this issue https://github.com/IBM/rl-testbed-for-energyplus/issues/29.
It might be because of the patching warning I got.
If you have any idea, please let me know. Thank you!

For the action type, I don't get why the array is wrapped in another one. Can you try with gym 0.17 instead? Otherwise you'll have to slightly modify the code, something like action = action[0] at the beginning of step
For EnergyPlus models: you can't just update model version, you need to use IDFVersionUpdater (in PreProcess/IDFVersionUpdater/ of EnergyPlus home installation).
For EnergyPlus 9.2 path: this can be safely ignored, it works fine even with this warning.

For the action type, I don't get why the array is wrapped in another one. Can you try with gym 0.17 instead? Otherwise you'll have to slightly modify the code, something like action = action[0] at the beginning of step
For EnergyPlus models: you can't just update model version, you need to use IDFVersionUpdater (in PreProcess/IDFVersionUpdater/ of EnergyPlus home installation).
For EnergyPlus 9.2 path: this can be safely ignored, it works fine even with this warning.

@antoine-galataud , thanks for your response.

For the action type, I don't get why the array is wrapped in another one. Can you try with gym 0.17 instead? Otherwise you'll have to slightly modify the code, something like action = action[0] at the beginning of step

Understood. I tried changing the gym version to 0.17 but it still resulted to the same problem. As a result, I tweaked the code to action = action[0] as you recommended and it did run. I'm now waiting for it to finish to run.

For EnergyPlus models: you can't just update model version, you need to use IDFVersionUpdater (in PreProcess/IDFVersionUpdater/ of EnergyPlus home installation).
Understood. I haven't really tried using EnergyPlus software. Will explore more on this later.

For EnergyPlus 9.2 path: this can be safely ignored, it works fine even with this warning.
I see. Will try this again and post another issue if the [https://github.com/IBM/rl-testbed-for-energyplus/issues/29#issuecomment-514471328](same problem) persist.

Thanks for your kind support.

@antoine-galataud , thanks for your response.

For the action type, I don't get why the array is wrapped in another one. Can you try with gym 0.17 instead? Otherwise you'll have to slightly modify the code, something like action = action[0] at the beginning of step

Understood. I tried changing the gym version to 0.17 but it still resulted to the same problem. As a result, I tweaked the code to action = action[0] as you recommended and it did run. I'm now waiting for it to finish to run.

For EnergyPlus models: you can't just update model version, you need to use IDFVersionUpdater (in PreProcess/IDFVersionUpdater/ of EnergyPlus home installation).
Understood. I haven't really tried using EnergyPlus software. Will explore more on this later.

For EnergyPlus 9.2 path: this can be safely ignored, it works fine even with this warning.
I see. Will try this again and post another issue if the [https://github.com/IBM/rl-testbed-for-energyplus/issues/29#issuecomment-514471328](same problem) persist.

Thanks for your kind support.

I have had the same problem in version 9.3.0, adding action = action [0] as @antoine-galataud comments solves the problem (in if action is not None block)

@AlejandroCN7 this will be solved with #54. @takaomoriyama let me know if there’s anything else to update in it.


Add phone to serverpod_user_info table

Minor feature request, but one that I think would be nice to have:
Many auth providers like firebase provide a phone number on authentication. If provided, it would be nice to store this in UserInfo.
It can be an optional, nullable field, and would not be a breaking change.  Happy to provide a PR for firebase if you think it is a good idea.

Friend it exist. you can modify the class user
class: UserServer
table: users
fields:
  uid: String
  name: String
  email: String
  photo: String
  phone: String?

@hortigado  If I understand your comment, you are suggesting creating my own custom class? I can't see that table in the server_pod docs or the source code.
If you are suggesting adding my own custom class - that is what I've done.
The ask is really around including more of the standard OIDC / user info parameters provided by the auth provider (e.g. firebase) as out of the box features in the UserInfo class. I can add my own custom class for this - but then I have to link it with the UserInfo class. All quite possible - but not as convenient.  Unless I am missing what you are suggesting.
Perhaps a better solution is to allow the UserInfo class to be generated/extended by the developer? But then you'd have to integrate that with the auth modules...
This isn't a show stopper - more of a paper cut - which is why it's a feature request (not a bug..)

I think this could be a useful addition, especially for implementing 2fa.

Is serverpod_user_info meant to be developer customizable or reference-able?
That would be great, as every app will want to customize the user table fields. Would be nice to have standard way/docs to create our own User class that can extend serverpod's UserInfo. And of course the ability to use that User class in all auth methods/endpoints/backend functions.
I'm trying to do the following, and can't, because serverpod generate gives the following error (yes, I did add tables_serverpod_auth.pgsql -- as renamed -- to my project, and the tables do exist in my Postgres DB):
class: Playlist
table: playlist
fields:
  userId: int, parent=serverpod_user_info

serverpod generate
...
FormatException: The table "playlist" (class "Playlist" is referencing a table that doesn't exist (serverpod_user_info).)

And I'm thinking adding this playlist_serverpod_user_info_fk.pgsql might work, but perhaps a bit cumbersone:
ALTER TABLE ONLY "playlist"
  ADD CONSTRAINT playlist_serverpod_user_info_fk
    FOREIGN KEY("userId")
      REFERENCES serverpod_user_info(id)
        ON DELETE CASCADE;

Knew I had seen this earlier looking through serverpod code, but had to dig for it again. Deserves a doc addition, maybe I'll do a PR.
This is related to this topic because of frequent need to reference the user info in apps.
To get this to work (reference the user_info table as well as the UserInfo serializable model), it seems I do not need to add an extraClasses entry in generator.yaml. But since serverpod_auth has nickname: auth, specifying the "fully qualified" class as module:auth:UserInfo? as follows in the protocol generated a protocol file that included the serverpod_auth_server module and compiled properly:
class: Favorite
table: favorite
fields:
  userId: int # FK defined in user_info_favorite.pgsql
  user: module:auth:UserInfo?, api
...

Oh, found it: https://docs.serverpod.dev/concepts/modules#referencing-a-module

Oh, found it: https://docs.serverpod.dev/concepts/modules#referencing-a-module

I was looking for this, I think that the documentation can be more explicit about doing this. This is very common.


Disable validation of state change for action and button

Fixes #938

Codecov Report

Merging #939 (ce71c49) into devel (f5c1286) will increase coverage by 0.00%.
The diff coverage is 100.00%.

: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           @@
##            devel     #939   +/-   ##
=======================================
  Coverage   91.97%   91.98%
=======================================
  Files          28       28
  Lines        4821     4826    +5
=======================================
+ Hits         4434     4439    +5
  Misses        387      387

Impacted Files
Coverage Δ

hahomematic/entity.py
96.55% <100.00%> (+0.01%)
:arrow_up:

hahomematic/generic_platforms/action.py
100.00% <100.00%> (ø)

hahomematic/generic_platforms/button.py
100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


(node/lsstcam-quad.cp.lsst.org) deploy host as el9

Please squash 3b86858

Commit was squashed on to: cf05f59

Please squash 3b86858

why squashing? perhaps a more descriptive commit message is better?


Playbook run throws error for docker

When I run
ansible-playbook -i inventories/my-ansible-nas/inventory nas.yml -b -K
Everything seemingly works out until I get a fail at the end.
TASK geerlingguy.docker : Add docker repository
Says it failed due to apt cache update failure
I grepped around to find out more but couldn't

@Luckst4r - you need to "grep" the open issues here. :)
REF: https://github.com/davestephens/ansible-nas/issues/200

Why not state that 20.04 isn't supported in the docks?
On Tue, Jul 14, 2020, 6:23 PM bcurran3<EMAIL_ADDRESS>wrote:

@Luckst4r https://github.com/Luckst4r - you need to "grep
https://github.com/davestephens/ansible-nas/issues?q=is%3Aissue+is%3Aopen+20.04"
the open issues here. :)
REF: #200 https://github.com/davestephens/ansible-nas/issues/200
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/davestephens/ansible-nas/issues/349#issuecomment-658460328,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQDQZSQI7ZS45INMCCED73DR3TSGJANCNFSM4OZZUFWA
.

@Luckst4r , I didn't write them but the docs state:

Requirements
Ansible NAS targets the latest Ubuntu LTS release, which is currently Ubuntu Server 18.04.3 LTS.
You can run Ansible-NAS on whatever you like, read the docs for more info. I use an HP Microserver.

Out of date since they say "currently."
But if you read the issue I pointed you to, and then the issue that it points to, you'll understand the current situation and when/how it will be resolved so that we can move on to using 20.04.

Much appreciated!
I'll downgrade to 18.04
On Tue, Jul 14, 2020, 6:52 PM bcurran3<EMAIL_ADDRESS>wrote:

@Luckst4r https://github.com/Luckst4r , I didn't write them but the
docs state:
Requirements
Ansible NAS targets the latest Ubuntu LTS release, which is currently
Ubuntu Server 18.04.3 LTS.
You can run Ansible-NAS on whatever you like, read the docs for more info.
I use an HP Microserver.
Out of date since they say "currently."
But if you read the issue I pointed you to, and then the issue that it
points to, you'll understand the current situation and when/how it will be
resolved so that we can move on to using 20.04.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/davestephens/ansible-nas/issues/349#issuecomment-658468114,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQDQZSSJP5MJCKHTPY7YU6TR3TVTTANCNFSM4OZZUFWA
.

Sucks, I know!
I'll put in a PR to update the docs to help avoid the obvious confusion.
Let's hope @davestephens is OK and comes back from recess soon to play with Ansible-NAS some more soon.

Much appreciated
On Tue, Jul 14, 2020, 6:55 PM bcurran3<EMAIL_ADDRESS>wrote:

Sucks, I know!
I'll put in a PR to update the docs to help avoid the obvious confusion.
Let's hope @davestephens https://github.com/davestephens is OK and
comes back from recess soon to play with Ansible-NAS some more soon.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/davestephens/ansible-nas/issues/349#issuecomment-658468915,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQDQZSVT4XT25FBET6PPI2DR3TV6XANCNFSM4OZZUFWA
.

Done!
https://github.com/davestephens/ansible-nas/pull/350

Closing, this was prior to the Ubuntu 20.04 docker repos being available.

Closing, this was prior to the Ubuntu 20.04 docker repos being available.


AES-GCM bandwidth

hi, i adding a support to AES-GCM to the engine and i getting a low bandwidth when im using iperf with tls.. when i using iperf with software i getting 10-12 Gbits and when im using the engine i get 3 Gbits..
this is logical?? or i didnt use the engine properly??
thanks

Hi @benishay, that would depend on a number of things:

Is you application using the AES-GCM in the engine asynchronously or synchronously, i.e. is the application/libssl creating an ASYNC_WAIT_CTX and a job for each connection/request?
What size of requests are you making to AES-GCM? The larger the request (up to 16KB) the more efficient it will be to offload.
Is your application running single or multithreaded? Is it doing much in parallel, i.e. are there multiple connections or just a single one for the benchmarks you are running?

Running asynchronously with 16KB requests to encrypt I would expect the performance to be fairly comparable (if there are plenty of multiple requests going on in parallel, much slower if not). Using smaller packets I would expect SW to see better performance however you run. Running synchronously I would expect SW to give you several times better performance if you are not running lots of connections in parallel threads due to the latency of offloading an individual request.
AES-GCM is best utilized directly at the QuickAssist API rather than through an OpenSSL engine if at all possible so you can eliminate as much as possible the copying between memory buffers that adds overhead.
Kind Regards,
Steve.

Hi Steve,

My  testing setup is the Iperf application using OPENSSL api to create a SLL_CTX which by using it will encrypt and send data to the server. I am testing the additions I made to the engine code by sending packets of ~16KB using OPENSSL API with one thread over 30 seconds.
My application is using synchronously in the ENGINE. I do know asynchronously will increase the performance of the application by some levels. but I have hard time doing it correctly and would be happy to some material on the matter.
I am aware of the engine overhead, but I must use it is supposed to be used by the OPENSSL api.
We tested the memory bandwidth of the card using Intel-PCM and the result was 100 times more then not using it - does it make sense?

Would appreciate your help,
Ben


Yatin: Watch (?-i:TBfmqQuq2BybRj4hrQoUAqwM7t61Jmd9y3)(?# tronscan smart contract)

Yatin requests the watch of the watch_keyword (?-i:TBfmqQuq2BybRj4hrQoUAqwM7t61Jmd9y3)(?# tronscan smart contract). See the MS search here and the Stack Exchange search in text, in URLs, and in code.

(?-i:TBfmqQuq2BybRj4hrQoUAqwM7t61Jmd9y3)(?# tronscan smart contract) has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by tripleee in Charcoal HQ


operands could not be broadcast together with shapes (1560224,) (1489274,)

Hello
when I run "python ./-Results/Generate_detection.py --model <your test output directory, under ./-Results by default>"
I get the following error:
ValueError: operands could not be broadcast together with shapes (1560224,) (1489274,)
This error is related to line 62 that is:
mask = inter_det_mask * nis_mask
please help

It is because the inter_det_mask and nis_mask are using different detections. If you are using customized object detection result, you might need to also generate a customized NIS file (to provide the nis_mask), or you could also drop the nis_mask term though considerable performance drop might be observed.

Hello
When I removed the  nis_mask, the performance decreased greatly. I used the object detection that you used, I don't know why the sizes are different. How can I generate a customized NIS file?


Add tests for OCR parsing code

Another round of tests, this time for the hOCR/ALTO/IIIF Anno parsing logic. Line coverage is pretty much complete, although we're still missing a few branches that cover edge cases, tests for those will follow when we encounter our first bugs :-)

Codecov Report

Merging #8 into master will increase coverage by 36.05%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           master       #8       +/-   ##
===========================================
+ Coverage    7.71%   43.76%   +36.05%
===========================================
  Files          10       10
  Lines         389      393        +4
  Branches      120      121        +1
===========================================
+ Hits           30      172      +142
+ Misses        287      176      -111
+ Partials       72       45       -27

Impacted Files
Coverage Δ

src/lib/ocrFormats.js
93.42% <100.00%> (+93.42%)
:arrow_up:

Continue to review full report at Codecov.

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


docs: Add MySQL preset to documentation

Summary
Inspired by MongoDB && DynamoDB presets, here I provide the jest-mysql preset which should help out for an easier setup of testing environment where MySQL is used as a storage.
Test plan
Test are passing and there is 91% coverage of the library: https://app.codecov.io/gh/Daniel-Yonkov/jest-mysql

Codecov Report

Merging #11223 (56c6fee) into master (2d96526) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #11223   +/-   ##
=======================================
  Coverage   64.24%   64.24%
=======================================
  Files         308      308
  Lines       13499    13499
  Branches     3288     3288
=======================================
  Hits         8673     8673
  Misses       4117     4117
  Partials      709      709

Continue to review full report at Codecov.

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

Hey guys, is there any reason for not merging the PR?

It's been more than a year since this PR has been proposed and no reaction nor a comment from facebook... such a dissapointment. I will be closing this PR


[Issue] When use the MMS host model. e.g. HuggingFace Model, no any info in cloudwatch log and can not use ssh

model_data = 's3://kraft-source-bucket/huggingface_model/model.tar.gz'
from sagemaker.huggingface import HuggingFaceModel
from sagemaker_ssh_helper.wrapper import SSHModelWrapper
import sagemaker
create Hugging Face Model Class
huggingface_model = HuggingFaceModel(
transformers_version='4.17.0',
pytorch_version='1.10.2',
py_version='py38',
dependencies=[SSHModelWrapper.dependency_dir()],
model_data=model_data,
role=role
)
ssh_wrapper = SSHModelWrapper.create(huggingface_model, connection_wait_time_seconds=0)
huggingface_model.deploy(initial_instance_count=1,instance_type="ml.g4dn.xlarge",wait=False)
model.tar.gz

pytorch.xx.bin
code/

inference.py

cat inference.py
import argparse
import io
import json
import logging
import os
import sys
import subprocess
import torch
import torch.distributed as dist
import torch.nn as nn
import torch.nn.functional as F
import torch.optim as optim
import torch.utils.data
import torch.utils.data.distributed
from PIL import Image
from torchvision import datasets, transforms
from torchvision.transforms import ToTensor
from model import Net
logger = logging.getLogger(name)
logger.addHandler(logging.StreamHandler(sys.stdout))
logger.info(os.system("nvidia-smi"))
sys.path.append(os.path.join(os.path.dirname(file), "lib"))
import sagemaker_ssh_helper
sagemaker_ssh_helper.setup_and_start_ssh()
def model_fn(model_dir):
print(model_dir)
logger.info(model_dir)
device = torch.device("cuda" if torch.cuda.is_available() else "cpu")
model = torch.nn.DataParallel(Net())
with open(os.path.join(model_dir, "model.pth"), "rb") as f:
model.load_state_dict(torch.load(f))
return model.to(device)
def load_from_bytearray(request_body):
image_as_bytes = io.BytesIO(request_body)
image = Image.open(image_as_bytes)
image_tensor = ToTensor()(image).unsqueeze(0)
return image_tensor
def input_fn(request_body, request_content_type):
# if set content_type as 'image/jpg' or 'applicaiton/x-npy',
# the input is also a python bytearray
if request_content_type == "application/x-image":
image_tensor = load_from_bytearray(request_body)
else:
print("not support this type yet")
raise ValueError("not support this type yet")
return image_tensor
Perform prediction on the deserialized object, with the loaded model
def predict_fn(input_object, model):
output = model.forward(input_object)
pred = output.max(1, keepdim=True)[1]
return {"predictions": pred.item()}

Serialize the prediction result into the desired response content type
def output_fn(predictions, response_content_type):
return json.dumps(predictions)
I run this code:
instance_ids = ssh_wrapper.get_instance_ids()  # <--NEW--
print(f'To connect over SSM run: aws ssm start-session --target {instance_ids[0]}')
no any output and in cloudwatch log has no any related info about sagemaker-ssh-helper

Hi, @KraftZzz. Thanks you for raising this issue.
I have few questions to clarify:
1/ You cannot use ssh / ssm, but do you get any prediction results?
2/ If you don't see any logs in CloudWatch, you probably have misconfigured permissions (no access to CloudWatch). Can you try any of the SageMaker examples, e.g. Deploy a pretrained PyTorch BERT model from Hugging Face and confirm that this example is working in your environment and that you can see the logs for it?
If you have the same issue even if you don't use SageMaker SSH Helper, you might need to reach out to the AWS Support.

I can got the prediction results and endpoint is inService status.
I am sure that the permission is not bad and I can see the MMS(multi-model-server) launch log, but no any ssm info or some infos about ssh-helper. And I checked the examples(Deploy a pretrained PyTorch BERT model from Hugging Face) you provided, in this example, using Pytorch Model and is a single model, no multi model.
I refer this example:https://github.com/aws/amazon-sagemaker-examples/tree/main/advanced_functionality/multi_model_bring_your_own, could you please help me to confirm whether is available in this example

OK, got it. Could you try instead of dependencies parameter add requirements.txt to the code/?
In the requirements add SageMaker SSH Helper :
sagemaker-ssh-helper

I've tried your example and with it works for me with this approach.
As a side comment, all examples including your code are single-model endpoints. "Multi-model-server" name is somewhat confusing. If you really want to deploy a multi-model endpoint, you will need to use MultiDataModel and SSHMultiModelWrapper. See the FAQ for more details.

You add this code:
import os
import sys
sys.path.append(os.path.join(os.path.dirname(file), "lib"))
import sagemaker_ssh_helper
sagemaker_ssh_helper.setup_and_start_ssh()
in inference.py, right?
I mention MMS because I see the following information in the endpoint log:
Warning: MMS is using non-default JVM parameters: -XX:-UseContainerSupport
2023-04-26T04:35:25,060 [INFO ] main com.amazonaws.ml.mms.ModelServer -
MMS Home: /opt/conda/lib/python3.8/site-packages
Current directory: /
Temp directory: /home/model-server/tmp
Number of GPUs: 1
Number of CPUs: 4
Max heap size: 3500 M
Python executable: /opt/conda/bin/python3.8
Config file: /etc/sagemaker-mms.properties
Inference address: http://<IP_ADDRESS>:8080
Management address: http://<IP_ADDRESS>:8080
Model Store: /.sagemaker/mms/models
Initial Models: ALL
Log dir: null
Metrics dir: null
Netty threads: 0
Netty client threads: 0
Default workers per model: 1
Blacklist Regex: N/A
Maximum Response Size: 6553500
Maximum Request Size: 6553500
Preload model: false
Prefer direct buffer: false
2023-04-26T04:35:25,118 [WARN ] W-9000-model com.amazonaws.ml.mms.wlm.WorkerLifeCycle - attachIOStreams() threadName=W-9000-model
2023-04-26T04:35:25,179 [INFO ] W-9000-model-stdout com.amazonaws.ml.mms.wlm.WorkerLifeCycle - model_service_worker started with args: --sock-type unix --sock-name /home/model-server/tmp/.mms.sock.9000 --handler sagemaker_huggingface_inference_toolkit.handler_service --model-path /.sagemaker/mms/models/model --model-name model --preload-model false --tmp-dir /home/model-server/tmp
2023-04-26T04:35:25,180 [INFO ] W-9000-model-stdout com.amazonaws.ml.mms.wlm.WorkerLifeCycle - Listening on port: /home/model-server/tmp/.mms.sock.9000
2023-04-26T04:35:25,180 [INFO ] W-9000-model-stdout com.amazonaws.ml.mms.wlm.WorkerLifeCycle - [PID] 72
2023-04-26T04:35:25,180 [INFO ] W-9000-model-stdout com.amazonaws.ml.mms.wlm.WorkerLifeCycle - MMS worker started.
2023-04-26T04:35:25,180 [INFO ] W-9000-model-stdout com.amazonaws.ml.mms.wlm.WorkerLifeCycle - Python runtime: 3.8.10
2023-04-26T04:35:25,181 [INFO ] main com.amazonaws.ml.mms.wlm.ModelManager - Model model loaded.
2023-04-26T04:35:25,187 [INFO ] main com.amazonaws.ml.mms.ModelServer - Initialize Inference server with: EpollServerSocketChannel.
2023-04-26T04:35:25,199 [INFO ] W-9000-model com.amazonaws.ml.mms.wlm.WorkerThread - Connecting to: /home/model-server/tmp/.mms.sock.9000
2023-04-26T04:35:25,256 [INFO ] main com.amazonaws.ml.mms.ModelServer - Inference API bind to: http://<IP_ADDRESS>:8080
Model server started.

no any sagemaker-ssh-helper worklog in endpoint cloudwatch log, so I run:
instance_ids = ssh_wrapper.get_instance_ids()
print(f'To connect over SSM run: aws ssm start-session --target {instance_ids[0]} --region {sess.boto_region_name}')
no any output

Could you please share your step ?

My steps are the following:
1/  Added to inference.py the following lines:
+import os
+import sys
+sys.path.append(os.path.join(os.path.dirname(__file__), "lib"))
+
+import sagemaker_ssh_helper
+sagemaker_ssh_helper.setup_and_start_ssh()
+
+
 from transformers import AutoTokenizer, AutoModel
 import torch
 import torch.nn.functional as F

2/ Modified in sagemaker/17_custom_inference_script/sagemaker-notebook.ipynb and executed the following cell:
from sagemaker.huggingface.model import HuggingFaceModel
from sagemaker_ssh_helper.wrapper import SSHModelWrapper  # <--NEW--

# create Hugging Face Model Class
huggingface_model = HuggingFaceModel(
   model_data=s3_location,       # path to your model and script
   role=role,                    # iam role with permissions to create an Endpoint
   transformers_version="4.26",  # transformers version used
   pytorch_version="1.13",        # pytorch version used
   py_version='py39',            # python version used
)

ssh_wrapper = SSHModelWrapper.create(huggingface_model, connection_wait_time_seconds=0)  # <--NEW--

# deploy the endpoint endpoint
predictor = huggingface_model.deploy(
    initial_instance_count=1,
    instance_type="ml.g4dn.xlarge"
    )

After endpoint has been deployed, I was able to fetch the instance_ids():
ssh_wrapper.get_instance_ids()

INFO:sagemaker-ssh-helper:Querying SSM instance IDs for endpoint huggingface-pytorch-inference-2023-04-24-17-00-23-155
INFO:sagemaker-ssh-helper:Got preliminary SSM instance IDs: ['mi-01234567890abcd00']
INFO:sagemaker-ssh-helper:Got final SSM instance IDs: ['mi-01234567890abcd00']

['mi-01234567890abcd00']

Oh, To my confusion, I managed to get the Mi-xxxx in one of my experiments yesterday. But I don't modify any code...

Thanks for your share

You're welcome! Let me know if you managed to make your code work, so we can close this issue.


Ranks

Hello,
I have one problem, so I'd like to know if is possible to configure it.
Let's take an example: Player A has VIP rank. I'll add VIP+ to player A, then player A will have VIP and VIP+ ranks, but prefix shows as VIP. VIP+ is higher than VIP, so it should show VIP+ not VIP.
Like.. it should show player rank (prefix) which has higher integer.
default 1
VIP 2
VIP+ 3
Elite 4
Helper 5
Moderator 6
Administrator 7

Use /pex group  weight [weight] commnad. Lower weight value has more priority than a group with a higher value.

lele, rip prica

BlackRavenMC, thank you, I didn't notice this weight command in Commands page.


Warn or throw when returning a value not in a response schema

Currently it appears that APIFlask silently filters out values from the response that are not in the schema.
Trivial example:
class PetOut(Schema):
    id = Integer()
    name = String()

@app.get('/pet')
@app.output(PetOut)
def get_pet():
    return {
        'id': 1234,
        'name': 'Fido',
        'category': 'Dog'  # will not appear in response
    }

Would it be possible to add an opt-in assertion to fail for this case, or at least create a warning? This behaviour can result in subtle, hard to diagnose bugs.

This behaviour is intentional, please see https://github.com/marshmallow-code/marshmallow/issues/1198 for the details.


fh: 0.1.1 -> 0.1.2

Diff: https://github.com/DeterminateSystems/fh/compare/v0.1.1...v0.1.2
Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

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

1 package built:

fh


cncli: Fix build for FreeBSD port using SODIUM_LIB_DIR

Description
Modified build to work with FreeBSD port. Added ability to set SODIUM_LIB_DIR env variable to skip getting sodium from github.
Where should the reviewer start?
Test build with libsodium not in system.
Motivation and context
FreeBSD ports are unable to access the internet during the build process and use git commands. Due to the fact that the git command is always executed, build was failing.
I could have just made a patch... but decided that maybe it's best to add this feature upstream? Others might use this?
How has this been tested?
Tested as part of the FreeBSD port build process using:
https://github.com/cardano-bsd-alliance/freebsd-ports-cncli/blob/main/Makefile

Maybe should be documented that lib will be auto-added after SODIUM_LIB_DIR: SODIUM_LIB_DIR/lib

Let me know if you need me to do anything else, but otherwise it builds/runs fine with the modifications and I performed all tests suggested.


chore: add release schedule

Which problem is this PR solving?
Quick PR to add target release dates to the README following @tedsuo's comment on gitter.

My request: We’d like to start communicating more clearly about the state of each implementation. Please update the README to have section at the top with the following:
Current state of the project, with a focus on the items listed in the alpha description
Installation and super basic quick start guide.

Codecov Report

Merging #268 into master will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master    #268      +/-   ##
=========================================
- Coverage   98.92%   98.9%   -0.03%
=========================================
  Files          66      66
  Lines        2521    2562      +41
  Branches      156     170      +14
=========================================
+ Hits         2494    2534      +40
- Misses         27      28       +1

Impacted Files
Coverage Δ

...ckages/opentelemetry-core/src/common/NoopLogger.ts
14.28% <0%> (-5.72%)
:arrow_down:

...ges/opentelemetry-core/src/common/ConsoleLogger.ts
91.66% <0%> (-3.08%)
:arrow_down:

...s/opentelemetry-core/test/context/B3Format.test.ts
100% <0%> (ø)
:arrow_up:

...s/opentelemetry-core/test/trace/tracestate.test.ts
100% <0%> (ø)
:arrow_up:

...entelemetry-core/test/common/ConsoleLogger.test.ts
100% <0%> (ø)
:arrow_up:

...lemetry-core/test/trace/ProbabilitySampler.test.ts
100% <0%> (ø)
:arrow_up:

...pe-async-hooks/test/AsyncHooksScopeManager.test.ts
100% <0%> (ø)
:arrow_up:

.../opentelemetry-core/src/trace/spancontext-utils.ts
100% <0%> (ø)
:arrow_up:

...s/opentelemetry-core/test/trace/NoopTracer.test.ts
100% <0%> (ø)
:arrow_up:

...ckages/opentelemetry-core/test/platform/id.test.ts
100% <0%> (ø)
:arrow_up:

... and 12 more

How confident are you about meeting the Sept 30 date?

We are on right track to complete the API/SDK/Exporters needed for ALPHA release. Starting this week focus is about ironing out missing pieces, README, documentations, examples etc. Please feel free to open an issue if you think anything is missing.


[Production & Staging] unlogged in view of registrations of a project has link to register parent project

 Steps

Go to osf.io
Go to a project without being logged into the osf
Go to a public project that is a child project/component of another project and view registrations page (example: https://osf.io/sd7kg/registrations/)

 Expected behavior
I expect to see any registrations that are there. If there are no registrations I expect to see a link to the most viewed and most recent public registrations on the OSF.
 Observed behavior
While the registration link, or the link to the most viewed and pubic registrations on the OSF appears, there is also text that reads that to register the entire parent project to click here (with a link to the registration page of the parent project). This is expected if I am logged into the account (along with the option to register that given project/component), but not if I'm viewing another project I'm not a contributor on.

@TimErrington ... I'm not seeing what you saw.

I'm not seeing it now either. Maybe it is gone now that the other registration problem was addressed (the template error one - https://github.com/CenterForOpenScience/osf.io/issues/3210#issuecomment-113280336)

I'll close this and we can reopen/resubmit if the behavior is experienced again.


[Bug]: Minifier generates invalid code when compiling d3 geoAlbersUsa code

System Info
System:
OS: macOS 14.3.1
CPU: (8) x64 Intel(R) Core(TM) i7-1068NG7 CPU @ 2.30GHz
Memory: 29.32 MB / 32.00 GB
Shell: 5.1.8 - /usr/local/bin/bash
Binaries:
Node: 18.8.0 - ~/.asdf/installs/nodejs/18.8.0/bin/node
Yarn: 1.22.21 - ~/.asdf/installs/nodejs/18.8.0/bin/yarn
npm: 8.18.0 - ~/.asdf/plugins/nodejs/shims/npm
pnpm: 8.11.0 - ~/.asdf/installs/nodejs/18.8.0/bin/pnpm
Browsers:
Chrome: 121.0.6167.184
Edge: 122.0.2365.52
Safari: 17.3.1
npmPackages:
@rspack/cli: latest => 0.5.4
@rspack/core: latest => 0.5.4
Details
The following construct in the d3 source code
return point = null,
    (lower48Point.point(x, y), point)
    || (alaskaPoint.point(x, y), point)
    || (hawaiiPoint.point(x, y), point);

https://github.com/d3/d3-geo/blob/719819bc604eb99438006f922bb586c01283b224/src/projection/albersUsa.js#L33-L39
Is being transpiled/minified to the following in production mode
return (
    e.point(i, u),
    (t = null),
    (n.point(i, u), t) || (o.point(i, u), t)
);

Which is invalid, because the first call to .point() modifies the point var.
I tried to reproduce this in the swc playground, but couldn't - I'm unsure exactly what swc version and options are being used in the minifier, but they do appear to be producing invalid code in this case.
Admittedly the input code is pretty weird, but that's d3 for you.
I've included a demonstration against the d3 package, but also a standalone repro that only pulls in the relevant code
Reproduce link
https://github.com/glenjamin/rspack-minify-repro
Reproduce Steps

npm run build:rspack
open rspack-dist/d3.html and rspack-dist/standalone.html
Note that in both cases some coordinates are printed to the page
npm run build:rspack -- --mode=production
Refresh the pages
Note that in both cases null is printed to the page

Confirmed that the nightly version works 👍


Accessibility - card section does not accept aria-label property

we are unable to provide a label to the card section simple element, there is a constant "placeholder" now
reporting url https://ibmdocs-dev.mybluemix.net/docs/en/cloud-paks
the tool to check errors is "IBM Equal Access Accessibility Checker", plugin for chrome
i would suggest that ALL dotcom components accept the basic aria properties so anybody can set them
<div role="region" aria-label="placeholder" class="bx--card-group__card">

This is an accessibility issue.
Read more here, too:
https://www.w3.org/TR/wai-aria/#aria-label

The card group simple component does not accept an aria-label, it has a constant of "placeholder" set as aria-label.

Hi @kyjak! Looks like you are using Carbon for IBM.com v1.18.1. The appropriate aria-labels have been fixed and included in v1.19.0. Let us know if you still are facing issues once you upgrade.

Fixed with the upgrade.


Partial Kibana screenshot in email report

Sentinl and Kibana version: 6.2.3
Search Guard 6.2.3
Chromium-browser 66.0.3359.181-0ubuntu<IP_ADDRESS>

Issue description: Emailed report screenshot only contains the Kibana sidebar, not the dashboard content.

Reproducing the issue step-by-step: Create fault state in ES, run report, wait for email.
3a. Raw report

Expected results vs. actual results: Expect to see a screenshot of a full Kibana page but just get the left sidebar.

Sample of data: n/a

Additional details: Kibana config

Brand new Sentinl user here. Sorry if this is basic or I've done something stupid here - it's my first day using Sentinl and I feel like I'm going in circles with this report issue.
I've made a very simple email watcher, which works fine. I copied the bulk of the watcher code into a new report, and attempted to get a screenshot of a Kibana dash which shows the cause of the error, but can not get it to capture the full screen. SG auth does not appear to be the problem I can see it fetching Kibana assets after the login screen.
I'm not sure if it's my Sentinl config, or possibly something to do with chromium browser.
tl;dr: Report runs, email arrives, but the screenshot is incomplete.
Any help please?

@lmangani - perfect.  I increased the delay and I have the full page.
Thanks very much!


Docs: Wrong Gradle Dep Scope

The docs Gradle syntax uses the wrong scope and could be simplified.  For example here it has:
compile group: 'org.springframework.cloud', name: 'spring-cloud-gcp-starter-storage'

This should be (for build.gradle.kts):
implementation("org.springframework.cloud:spring-cloud-gcp-starter-storage")

There are ~25 places in the docs that need this fix.

@jamesward - So just to clarify, we should replace all of our gradle snippets
from
dependencies {
    compile group: 'org.springframework.cloud', name: 'spring-cloud-gcp-starter-storage'
}

to this:
dependencies {
    implementation("org.springframework.cloud:spring-cloud-gcp-starter-storage")
}

Yup! That is the build.gradle.kts syntax.  What I'm not sure about is if there is a different syntax for build.gradle - let me know if you want me to investigate that.

@jamesward What do you mean by "wrong scope"? Sorry, I'm not very familiar with Gradle.

compile is a scope and has been deprecated:
'compile(Any): Dependency?' is deprecated. The compile configuration has been deprecated for dependency declaration. Please use the 'implementation' configuration instead.


Istio-proxy pretty much always initially fails readinessProbes

Not urgent, but the default initalDelaySeconds on the istio-proxy readinessProbe of 1 second always results in at least one failure, which when you've got 500+ proxies, it's a lot :D
42m         Warning   Unhealthy           Pod          Readiness probe failed: HTTP probe failed with statuscode: 503
https://github.com/istio/istio/blob/0e9cecab053aab744a7c3a731aacb07fd794d5f9/install/kubernetes/helm/istio/templates/sidecar-injector-configmap.yaml#L202
This is on a cluster with basically next to no services too so probably thinking the initial delay of 1 second is a bit too low.  We've bumped global.proxy.readinessInitialDelaySeconds to 5.

@mandarjog @jnativio - thoughts?  okay to make this change as default?

I don't think this is a good idea. It will delay directly the startup of a pod with sidecar. What is the impact of the warning message?

Yeah I think I active with @jnativio - from a serverless perspective perhaps it needs to be on 1s and then the goal is to get that sidecar start time right down.
@jnativio in terms of impact we alert when apps fail their readiness probes successively so at the moment we get alerts every time a proxy starts.  We can tweak these values internally though.

I'm going to close this, it's configurable at the end of the day :)

hi how to change to global config parameter.. pls provide me the command to change to global value.. below cmd not working for me kubectl get cm istio-sidecar-injector -n istio-system -o yaml | sed -e 's/initialDelaySeconds  1/initialDelaySeconds  10/' | kubectl apply -f -

@prabaharanit You can use: helm template --set global.proxy.readinessInitialDelaySeconds=10  helm/istio --name istio --namespace istio-system | kubectl apply -f -

how can i do it without helm?
cause, this global.proxy.xxx not found ...

We aren't support customized config without helm today, you'd have to dig through the code to check what yaml are changed for DIY.


Better instructions maybe????

I did install especificlly the two programs in my pc in order to run the program but not only doesnt work... now im getting warigs about VCRUNTIME not found when opening hte installer and ASLO numerous warnings every time i turn on my pc.


Update argocd login flow in docs

What type of PR is this?
/kind documentation
What does this PR do / why we need it:
Updating flow to login to the Argo CD UI, replacing former instructions to change to developer perspective

/lgtm
/approve


Fixed carousel navigation on mobile devices

Issue Reference
Fixed carousel navigation on mobile devices

@shivendra-webkul
Checked and Found that the crousal issue in the mobile view has been fixed, now we can change the images with the help of touch response. Previously we used to see the navigation buttons in it.
Video
https://github.com/bagisto/bagisto/assets/92721837/c03d488e-fe5a-4840-b760-38657fa91591

Checked and Found that it is working fine after code refactor. Please check the video for reference.
Video
https://github.com/bagisto/bagisto/assets/92721837/14d2fa9c-e406-4212-9ddd-751b4520fc7d


License Question

Is there an open source license noted somewhere I am not finding for this repository? It would be great to link to as an example of implementing Sockets in Foundry, but if this code is all rights reserved (as implied by having no license) I don't think it's open game to use code from it in different projects. It also makes third-party contributions more questionable.
Wanted to ask about this to double check.

I love this module. It does what I should have done in my modules ages ago. However, I also have the same concern about the license, is it meant to be "open" and the lack of license is an oversight or is it mean to be all  rights reservered?

You could call the fact that this module doesn't have a license attached an oversight. Since obviously many people care about (and depend on) this module I'm licensing this under MIT, so people can actually go ahead and improve stuff in this module if they'd like to.


Fix [IOS] Payment Intent status

@triniwiz IOS return success when failed a 3DS payment.
I think this solution should be the same with android .. I have fixed this issue on android based on the paymentIntent status but I think it should be more complete such this way!
I am a little confused about those status, I think they will not be triggered on the payment process, for that I put them with the success
STPPaymentIntentStatusRequiresPaymentMethod
STPPaymentIntentStatusRequiresSource
STPPaymentIntentStatusRequiresSourceAction

What do you think ?

You can always introduce another state

I think thees status are for internal use by stripe or to show a progress payment.
we should consider  STPPaymentIntentStatusSucceeded and STPPaymentIntentStatusRequiresCapture as success but I think we can keep it as is for the moment


rtmp: rewrite implementation of rtmp connection

Codecov Report

Merging #1047 (1660e06) into main (ef3e18a) will increase coverage by 0.67%.
The diff coverage is 67.23%.

@@            Coverage Diff             @@
##             main    #1047      +/-   ##
==========================================
+ Coverage   59.68%   60.36%   +0.67%
==========================================
  Files          66       84      +18
  Lines        7199     7970     +771
==========================================
+ Hits         4297     4811     +514
- Misses       2577     2744     +167
- Partials      325      415      +90

Impacted Files
Coverage Δ

internal/rtmp/message/msg_audio.go
85.18% <ø> (ø)

internal/rtmp/message/msg_setpeerbandwidth.go
0.00% <0.00%> (ø)

internal/rtmp/message/reader.go
70.00% <0.00%> (ø)

internal/core/rtmp_conn.go
67.67% <50.00%> (-0.18%)
:arrow_down:

internal/rtmp/handshake/c1s1.go
90.00% <60.00%> (-1.18%)
:arrow_down:

internal/rtmp/message/msg_command_amf0.go
71.42% <60.00%> (ø)

internal/core/rtmp_source.go
73.75% <61.36%> (+0.33%)
:arrow_up:

internal/rtmp/handshake/c2s2.go
85.71% <66.66%> (+0.52%)
:arrow_up:

internal/rtmp/conn.go
68.37% <70.10%> (+1.85%)
:arrow_up:

internal/rtmp/chunk/chunk0.go
93.75% <100.00%> (ø)

... and 27 more

Continue to review full report at Codecov.

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


Update thrift to 0.13

Requirement - what kind of business use case are you trying to solve?
Use the most recent thrift 0.13 dependency to overcome CVEs.
Problem - what in Jaeger blocks you from solving the requirement?
uber/tchannel-go dependency is currently using thrift 0.9.3.
Proposal - what do you suggest to solve the problem or improve the existing situation?
Either:

Update tchannel-go to use thrift 0.13
Remove the dependency on tchannel-go.

Any open questions to address

The thrift 0.13.0 image hasn't been published yet. It depends on this PR https://github.com/ahawkins/docker-thrift/pull/22.
https://hub.docker.com/_/thrift/?tab=description

@pavolloffay any status update on this issue?
Related issue in a different project: the OpenTelemetry Collector includes Jaeger and Prometheus receivers, so it has Go Module dependencies on both.
When I update the Prometheus dependency to the latest v2.19.0 release, it brings the Thrift v0.13.0 dependency along, which breaks the Jaeger receiver compilation and complains about tchannel-go method mismatch, since it's using older Thrift version as described above.
Seems like there were breaking changes in Thrift API between v0.9.3 and v0.13.0, and the only way to fix this dependency hell is to update uber/tchannel-go to use apache/thrift v0.13.0.

I was planning to have a look at this soon. If you have free cycles you can take it.
Some time ago I have forked the thrift image and published 0.13.0 tag https://github.com/jaegertracing/docker-thrift

We have removed the tchannel receiver from the collector in the previous release. Maybe there is still a reference to in the tests though.

We have removed the tchannel receiver from the collector in the previous release.

Looks like this was done in v1.18.0, but OpenTelemetry Collector depends on v1.17.0: https://github.com/open-telemetry/opentelemetry-collector/blob/b28cbd9c8e2e70bf385c3e4788ada839aa457b7d/go.mod#L28
There's actually still a dependency on tchannel-go in https://github.com/jaegertracing/jaeger/blob/b46738c68a906f7330e5e1a5ba6937b58e52b9dc/go.mod#L75, but it may not be needed for OpenTelemetry Collector with the latest release.
Raised the issue https://github.com/open-telemetry/opentelemetry-collector/issues/1126

Note that this is still blocking Prometheus updates in OpenTelemetry Collector, see comment https://github.com/open-telemetry/opentelemetry-collector/issues/1124#issuecomment-645079077

These are minimal direct references to tchannel-go that would need to be tackled to remove the dependency completely and upgrade Thrift:
./cmd/collector/app/sampling/handler.go:18:	"github.com/uber/tchannel-go/thrift"
./cmd/collector/app/span_processor.go:22:	tchannel "github.com/uber/tchannel-go"
./examples/hotrod/services/driver/server.go:21:	"github.com/uber/tchannel-go"
./examples/hotrod/services/driver/client.go:23:	"github.com/uber/tchannel-go"

The examples should be re-implemented using gRPC.


1 inch is not 25.4000508mm since 1959

Hello James,
we all like to be as precise as possible, however, since 1959, the US inch is the same as the UK inch and the Canadian: exactly 25.4mm
25.4000508 looks more precise, but is not.
It is not a big issue, though (50.8nm), I guess nobody will notice an effect.

Ah interesting, that's a good bug report. Thanks!
On Mon, 28 Nov 2016 at 09:44, okroth<EMAIL_ADDRESS>wrote:

Hello James,
we all like to be as precise as possible, however, since 1959, the US inch
is the same as the UK inch and the Canadian: exactly 25.4mm
25.4000508 looks more precise, but is not.
It is not a big issue, though (50.8nm), I guess nobody will notice an
effect.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/MrRio/jsPDF/issues/960, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAC5s-t24wiJIBY6k_Lk2JiwkxCPMPlCks5rCqKRgaJpZM4K9k_x
.

--
James Hall
Director
Parallax Agency Ltd
M.<PHONE_NUMBER>0

#1671

fixed with #1671


1.0.21 release missing

Hello,
dm_control 1.0.20 is not compatible with the latest mujoco (3.2.0).
From https://github.com/google-deepmind/dm_control/commit/a495469c67265b082ff27e64010f844798ba1c2b, it seems that you are ready to release 1.0.21.
Is there any blocker for this or is it an omission ?
Thanks in advance :)
Context: bumping mujoco's version in nixpkgs: https://github.com/NixOS/nixpkgs/pull/327508

I believe this is fixed now.


Backmerge release 0.14.0

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 75.528%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
505

Relevant Lines:
680

💛  - Coveralls


[MNT] dask CI fix and expanded full pytest actions tag

Stops dask from crashing the Python 3.11 CI by lowering the version bound again. If someone wants dask for these versions, they will have to update to the latest version and resolve the resulting issues.
Updates the full pytest tag to use all Python versions in PRs.

Im going to run this without the tag quick just to check its still ok.

nope, still a bit broken unfortunately 🙁. No idea why it is running all Python versions without the tag

Ok looks good now. Also fixed a Sphinx issue thats going to impact every PR. Suddenly popped up while testing this.


Feature: Cached factory delegate container

This feature has started from the idea of combining the ReflectionContainer functionality with the self-discoverable factories (see issue #202 ). This is how it should work
This "CacheReflectionContainer" delegate container would use autoreflection to find the dependencies of an auto-wired class and would generate a file on disk containing a "Factory" class which can be auto-loaded. The end result would be a file like so:
namespace League\Container\AutowiredFactories;

class Factory_{sha1_of_the_class_name} implements Factory
{
   public function __invoke(ContainerInterface $container) {
       return new class_name($container->get('param1'), $container->get('param2');
   }
}

I’m currently working on a tool to compile a container, but that’s with explicit definitions and not with reflection.
I do see the benefit of this for people who use reflection but I really try to discourage the use of it so I’d struggle to put something like this in to the main repository.
I wouldn’t be against linking to something from the docs though that could do this.
Sent from ProtonMail Mobile
On Mon, Jun 22, 2020 at 17:24, Adrian Miu<EMAIL_ADDRESS>wrote:

This feature has started from the idea of combining the ReflectionContainer functionality with the self-discoverable factories (see issue #202 ). This is how it should work
This "CacheReflectionContainer" delegate container would use autoreflection to find the dependencies of an auto-wired class and would generate a file on disk containing a "Factory" class which can be auto-loaded. The end result would be a file like so:
namespace
League
\
Container
\
AutowiredFactories
;
class
Factory_
{sha1_of_the_class_name}
implements
Factory
{
public
function
__invoke
(
ContainerInterface
$
container
) {
return
new
class_name(
$
container
->
get
(
'param1'
),
$
container
->
get
(
'param2'
);
}
}
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

I’m currently working on a tool to compile a container, but that’s with explicit definitions and not with reflection.
It would be hard to compile the container with reflection anyway (you would most likely find false positives)
I wouldn’t be against linking to something from the docs though that could do this.
You mean linking to a third-party/community package? If so, that would be an endorsement and it would be hard to track its development

The tool I’m building has an interactive element, would be quite easy to do but would introduce a bunch of complexity that I don’t really want.
It very much wouldn’t be an endorsement, just a “I don’t recommend reflection but if you insist on using it, this exists” but in more words with the reasoning.
Sent from ProtonMail Mobile
On Mon, Jun 22, 2020 at 18:15, Adrian Miu<EMAIL_ADDRESS>wrote:

I’m currently working on a tool to compile a container, but that’s with explicit definitions and not with reflection.

It would be hard to compile the container with reflection anyway (you would most likely find false positives)

I wouldn’t be against linking to something from the docs though that could do this.

You mean linking to a third-party/community package? If so, that would be an endorsement and it would be hard to track its development
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

Closing this as work is ongoing on a compiler


nv-codec-headers-12: <IP_ADDRESS> -> <IP_ADDRESS>; ffmpeg: 6.0 -> 6.1; jellyfin-ffmpeg: 6.0-7 -> 6.0.1-1

Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

@ofborg build jellyfin-ffmpeg

Why pipewire failed to build?

@ofborg build ffmpeg_6

@ofborg build jellyfin-ffmpeg

@ofborg build jellyfin-ffmpeg

@ofborg build jellyfin-ffmpeg

So, there's something specific about x86_64-linux that requires ffmpeg and it's building with the new version.

10000+ rebuilds estimated, you might need to target the staging branch.

@justinas @Atemu I'm deeply sorry. I was having problems with the jellyfin-ffmpeg and thought this was the issue, but I might have fixed the real issue in the process. Removed the unnecessary override.

@ofborg build jellyfin-ffmpeg

Using 6ea3a51 I can build jellyfin-ffmpeg, Jellyfin transcodes.
I can not speak to any potential impact of changing nv-codec-headers-11 to nv-codec-headers-12 in general.

This is needed, otherwise it won't build.

The commit message contains a bunch of unnecessary historic detail, we don't need that.

The problem is at line 349 of ffmpeg/generic.nix.
The substitution should be removed.
    '' + lib.optionalString withVulkan ''
    # FIXME: horrible hack, remove for next release
    substituteInPlace libavutil/hwcontext_vulkan.c \
      --replace VK_EXT_VIDEO_DECODE VK_KHR_VIDEO_DECODE
  '';

Furthermore at line 355 in the patches attribute there's an unneeded patch that should be removed too.
    ++ (lib.optional (lib.versionAtLeast version "6" && lib.versionOlder version "6.1")
      { # this can be removed post 6.1
        name = "fix_aacps_tablegen";
        url = "https://git.ffmpeg.org/gitweb/ffmpeg.git/patch/814178f92647be2411516bbb82f48532373d2554";
        hash = "sha256-FQV9/PiarPXCm45ldtCsxGHjlrriL8DKpn1LaKJ8owI=";
      }

I had assumed I wrote that terrible patch but a git blame revealed @K900. Shame, shame, shame :p

Hey, it said "horrible hack".

The problem is at line 349 of ffmpeg/generic.nix. The substitution should be removed.
    '' + lib.optionalString withVulkan ''
    # FIXME: horrible hack, remove for next release
    substituteInPlace libavutil/hwcontext_vulkan.c \
      --replace VK_EXT_VIDEO_DECODE VK_KHR_VIDEO_DECODE
  '';

Furthermore, at line 355, in the patches section, there's an unneeded patch that should be removed as well.
    ++ (lib.optional (lib.versionAtLeast version "6" && lib.versionOlder version "6.1")
      { # this can be removed post 6.1
        name = "fix_aacps_tablegen";
        url = "https://git.ffmpeg.org/gitweb/ffmpeg.git/patch/814178f92647be2411516bbb82f48532373d2554";
        hash = "sha256-FQV9/PiarPXCm45ldtCsxGHjlrriL8DKpn1LaKJ8owI=";
      }

As jellyfin-ffmpeg is using version 6.0.1, the patch is needed. They are going to wait to 7.0 to update, so it will only be needed until next ffmpeg's release.

Btw, if you'd generally like to help maintain this package, feel free to add yourself to the maintainers :)

Btw, if you'd generally like to help maintain this package, feel free to add yourself to the maintainers :)

I learned so much about ffmeg that I think I'll help with it lol.

Regarding OBS Studio they have an open PR about this.

@Atemu I built xdg-desktop-portal successfully.

Fixed mpd, it will probably build just fine in the next release.

The update already happened in master two days ago: https://github.com/NixOS/nixpkgs/pull/268890
Just drop your bump, it'll all work in integration.
With mpd fixed, I think this should be GTG.

The update already happened in master two days ago: #268890
Just drop your bump, it'll all work in integration.
With mpd fixed, I think this should be GTG.

Done.

Also, I found some rust projects depending on this library which have an open PR about ffmpeg 6.1.

That one only claims to support ffmpeg_4 anyways. It was probably already broken by yours truly. Could you make it use ffmpeg_4?

That one only claims to support ffmpeg_4 anyways. It was probably already broken by yours truly. Could you make it use ffmpeg_4?

This might sound strange but I fixed wl-screenrec build using ffmpeg_5 instead of ffmpeg or ffmpeg_4, going to see if other packages behave the same.

Re-open in a new PR.


LineEdit command history

I am currently working on expression evaluator for Godot debugger and I implemented a command history feature that is present pretty much in any console type of application. For those who aren't sure what I have in mind, when you enter the command in the console, I store it in history that I can browse later, in my case pressing up and down arrow keys, user can get back to previously entered commands.
I implemented this as a custom Control for the tool only. However, I wanted to ask whether this would be feature that other users would find useful too. If yes, I could move this to LineEdit.

Illustration of current usage

If you coded an already working feature, it would be probably better to create a PR, not an issue. Whether we need it also depends on how complex code it introduces. (feature-alone, looks useful to me, e.g. for chats)

I don't think this will be merged, the use case is too specific and is not needed in the editor.

@KoBeWi I implemented the feature as a custom Control node, not in the LineEdit. Moreover, I implemented it specifivally for my usecase, so I did not implement full generic API for it, only what I needed. Talking to Juan, he suggested opening an issue for this.

@groud Well, it is used in the editor now - it will be, better said. 😀 But I agree that the use case is quite specific. That's why I opened this issue, to see opinion of other users and contributors.

this is why gdscript exists! the power of gdscript interweaves with ui nodes at full force in this thread! great work!


Velocity slower than $.animate for animating scrollTop

After running many tests, could determine that $.velocity is worse than $.animate in terms of performance to animate window scrollTop.
Ran the following benchmark test to verify:
var time = Date.now();

for( var i=0; i<100000; i++ ){
    $( 'body' ).animate( {
        scrollTop: '100px'
    }, 0 );
}

console.log(  Date.now() - time );

vs
var time = Date.now();

for( var i=0; i<100000; i++ ){
    $( 'body' ).velocity( {
        scrollTop: '100px'
    }, 0 );
}

console.log(  Date.now() - time );

Several issues with your premise there:

You're adding to a queue, the final time is related to setting up the queue, and not the animation.
Velocity handles scroll differently, see http://velocityjs.org/#scroll - so the Velocity test isn't actually doing anything at all.
Your Velocity code is also setting up a jQuery object first (and while Velocity might give a jQuery method, that's convenience and not requirement).

While you might be right, it's unlikely - and your code is incapable of proving anything I'm afraid.


Message stopping

Hi,
I was able to successfully install launch mitmproxy how  I wanted. Now a have a question about packages.
As I have found out mitmproxy doesn't do any assembling of messages. But I need to wait the very end of the message (suppose I know how it should end and maybe it can be large enough to be broken into peaces), check smth and only then send the whole stuff. Are the operations similar to these possible?

Or maybe there are places where I can implement this logic of waiting of another message by myself?


Issue in replacing force graph nodes with image

One-line summary [问题简述]
Issue in replacing force graph nodes with image both 'symbol: image://url' and 'color: {image: imageDom}' not working.
Version & Environment [版本及环境]

ECharts version [ECharts 版本]: 4.0.2
Browser version [浏览器类型和版本]: Chrome/IE/Edge
OS Version [操作系统类型和版本]: Windows 10

Expected behaviour [期望结果]
I'm trying to replace the symbol with an image. The node gets replaced with a rectangle (as my image is rectangular) but it shows node filled with default color instead of the image that i want to display.
ECharts option [ECharts配置项]

     {
          'name': 'Name',
          'symbol': 'image:<image url from local folder>',
          'symbolSize': 80,
          'draggable': 'false',
          'value': 5,
       }............

Other comments [其他信息]

Found some issue with the image i was using. Closing it.


update readme with badges

Description
Requires https://github.com/terraform-ibm-modules/documentation/pull/22
add build status and latest release
Also

update contributing section for local issues tab
move content to list at top of readme

Check the relevant boxes:

[ ] Bug fix (nonbreaking change that fixes an issue)
[ ] New feature (nonbreaking change that adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Examples or tests (addition or updates of examples or tests)
[x] Documentation update
[ ] CI related update (pipeline, etc.)

Checklist

[ ] If relevant, a test for the change has been added or updated as part of this PR.
[ ] If relevant, documentation for the change has been added or updated as part of this PR.

Merge

Merge using "Squash and merge".
Make sure to use a relevant conventional commit message that is based on the PR contents. The commit message determines whether a new version of the modules needs to be released, and if so, which semver number to use).

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

GitHub release
v2.3.2

Your semantic-release bot :package::rocket:


Cache files don't work if the directory does not exist

It would be nice if a command like:
aws-export-credentials \
  --debug \
  --profile $profile \
  --cache-file ~/.aws/aec/$profile.cache

would work even if the aec directory does not exist

Fixed in v0.11


fix: grammar typo for MultiActon doc comment

Noticed this small typo when digging into some of the examples :)
Thank you for all that you are doing!

Thanks!


[SUPPORT] Found in-flight commits after time :20210129225133, please rollback greater commits first

I am no longer able to write to a hudi table because of the error mentioned in the subject.This happened probably because 2 simultaneous EMRs were running at the sametime.I did terminate one EMR but I keep getting the above error.I also tried to running hudi-cli clean and rollback operations based on https://github.com/apache/hudi/issues/1785 but those are failing as well,I have added the stacktrace and commit details.Let me know if anything more is needed
Environment Description

Hudi version : 0.6.0

Spark version : 2.4.4

Hive version : 2.3.7

Hadoop version : 2.10.0

Storage (HDFS/S3/GCS..) : S3

Running on Docker? (yes/no) : no

Stacktrace:
2021-02-01T19:13:38.849+0000: 84.629: Application time: 0.0001674 seconds
2021-02-01T19:13:38.849+0000: 84.630: Total time for which application threads were stopped: 0.0001441 seconds, Stopping threads took: 0.0000495 seconds
2021-02-01 19:13:38 ERROR AbstractAsyncService:149 - Monitor noticed one or more threads failed. Requesting graceful shutdown of other threads
java.util.concurrent.ExecutionException: org.apache.hudi.exception.HoodieException: Failed to rollback s3://akctct-transient-dev-us-east-1-eigi-datalake/hudi_deltastreamer_large_100/engmt_open_cow_1 commits<PHONE_NUMBER>5129
	at java.util.concurrent.CompletableFuture.reportGet(CompletableFuture.java:357)
	at java.util.concurrent.CompletableFuture.get(CompletableFuture.java:1908)
	at org.apache.hudi.async.AbstractAsyncService.lambda$monitorThreads$1(AbstractAsyncService.java:147)
	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
	at java.lang.Thread.run(Thread.java:748)
Caused by: org.apache.hudi.exception.HoodieException: Failed to rollback s3://akctct-transient-dev-us-east-1-eigi-datalake/hudi_deltastreamer_large_100/engmt_open_cow_1 commits<PHONE_NUMBER>5129
	at org.apache.hudi.utilities.deltastreamer.HoodieDeltaStreamer$DeltaSyncService.lambda$startService$0(HoodieDeltaStreamer.java:595)
	at java.util.concurrent.CompletableFuture$AsyncSupply.run(CompletableFuture.java:1604)
	... 3 more
Caused by: org.apache.hudi.exception.HoodieRollbackException: Failed to rollback s3://akctct-transient-dev-us-east-1-eigi-datalake/hudi_deltastreamer_large_100/engmt_open_cow_1 commits<PHONE_NUMBER>5129
	at org.apache.hudi.client.HoodieWriteClient.rollback(HoodieWriteClient.java:505)
	at org.apache.hudi.client.HoodieWriteClient.rollbackPendingCommits(HoodieWriteClient.java:708)
	at org.apache.hudi.client.HoodieWriteClient.startCommit(HoodieWriteClient.java:576)
	at org.apache.hudi.utilities.deltastreamer.DeltaSync.startCommit(DeltaSync.java:465)
	at org.apache.hudi.utilities.deltastreamer.DeltaSync.writeToSink(DeltaSync.java:381)
	at org.apache.hudi.utilities.deltastreamer.DeltaSync.syncOnce(DeltaSync.java:245)
	at org.apache.hudi.utilities.deltastreamer.HoodieDeltaStreamer$DeltaSyncService.lambda$startService$0(HoodieDeltaStreamer.java:579)
	... 4 more
Caused by: org.apache.hudi.exception.HoodieRollbackException: Found commits after time :20210129225129, please rollback greater commits first
	at org.apache.hudi.table.action.rollback.BaseRollbackActionExecutor.validateRollbackCommitSequence(BaseRollbackActionExecutor.java:145)
	at org.apache.hudi.table.action.rollback.BaseRollbackActionExecutor.doRollbackAndGetStats(BaseRollbackActionExecutor.java:171)
	at org.apache.hudi.table.action.rollback.BaseRollbackActionExecutor.execute(BaseRollbackActionExecutor.java:108)
	at org.apache.hudi.table.HoodieCopyOnWriteTable.rollback(HoodieCopyOnWriteTable.java:193)
	at org.apache.hudi.client.HoodieWriteClient.rollback(HoodieWriteClient.java:494)
	... 10 more

Commit
Loaded instants [
[20210129211107__commit__COMPLETED],
[20210129211434__clean__COMPLETED],
[20210129211434__commit__COMPLETED],
[20210129211810__clean__COMPLETED],
[20210129211810__commit__COMPLETED],
[20210129212234__clean__COMPLETED], [20210129212234__commit__COMPLETED], [20210129212622__clean__COMPLETED], [20210129212622__commit__COMPLETED], [20210129212913__clean__COMPLETED], [20210129212913__commit__COMPLETED], [20210129213326__clean__COMPLETED], [20210129213326__commit__COMPLETED], [20210129213638__clean__COMPLETED], [20210129213638__commit__COMPLETED], [20210129214016__clean__COMPLETED], [20210129214016__commit__COMPLETED], [20210129214432__clean__COMPLETED], [20210129214432__commit__COMPLETED], [20210129214813__clean__COMPLETED], [20210129214813__commit__COMPLETED], [20210129215121__clean__COMPLETED], [20210129215121__commit__COMPLETED], [20210129215538__clean__COMPLETED], [20210129215538__commit__COMPLETED], [20210129215816__clean__COMPLETED], [20210129215816__commit__COMPLETED], [20210129220159__clean__COMPLETED], [20210129220159__commit__COMPLETED], [20210129220532__clean__COMPLETED], [20210129220532__commit__COMPLETED], [20210129220839__clean__COMPLETED], [20210129220839__commit__COMPLETED], [20210129221301__clean__COMPLETED], [20210129221301__commit__COMPLETED], [20210129221623__clean__COMPLETED], [20210129221623__commit__COMPLETED], [20210129221930__clean__COMPLETED], [20210129221930__commit__COMPLETED], [20210129222350__clean__COMPLETED], [20210129222350__commit__COMPLETED], [20210129222736__clean__COMPLETED], [20210129222736__commit__COMPLETED], [20210129223034__clean__COMPLETED], [20210129223034__commit__COMPLETED], [20210129223422__clean__COMPLETED], [20210129223422__commit__COMPLETED], [20210129223818__clean__COMPLETED], [20210129223818__commit__COMPLETED], [20210129224110__clean__COMPLETED], [20210129224110__commit__COMPLETED], [20210129224446__clean__COMPLETED], [20210129224446__commit__COMPLETED], [20210129224849__clean__COMPLETED],
[20210129224849__commit__COMPLETED],
[20210129225129__clean__COMPLETED],
[==>20210129225129__commit__INFLIGHT],
[20210129225133__commit__COMPLETED],
[20210201183500__clean__COMPLETED]]

@n3nash @bvaradar : any suggestions on cleaning /repairing Hudi if 2 concurrent jobs have messed up things.

Caused by: org.apache.hudi.exception.HoodieRollbackException: Found commits after time :20210129225129, please rollback greater commits first

if you try rolling back<PHONE_NUMBER>5133 and then retrying or even that fails? If you don't want to lose [20210129225133__commit__COMPLETED]. you can do some sort of easy manual repair, given it s copy on write table.
a) Stop the pipeline
b) delete all files that have<PHONE_NUMBER>5129 in its name
c) delete<PHONE_NUMBER>5129__commit__INFLIGHT
next run should start flowing again

@vinothchandar yes I tried rolling back<PHONE_NUMBER>5133 ..I get the same exception.I had to stop the pipeline as I kept getting the above exception.I will try manually deleting and see if I am able to proceed Thanks

@vinothchandar I was able to proceed after deleting the<PHONE_NUMBER>5129 files from .hoodie folder in s3.But I wanted to know whether anything else needs to be cleaned up ? For example there might have been some versions created for commit time<PHONE_NUMBER>5129 which i can get from<PHONE_NUMBER>5129.commit file.Does that need to be cleaned up and if not why ?

b) delete all files that have<PHONE_NUMBER>5129 in its name
actually they need to be cleaned as well.

the hudi cli restore should have been able to do this for you actually, as long as you were okay to lose the later commit. I was planning to investigate why that did not work for e,g

So I did not find hudi-cli restore option in the available options
bootstrap index showmapping - Show bootstrap index mapping
* bootstrap index showpartitions - Show bootstrap indexed partitions
* bootstrap run - Run a bootstrap action for current Hudi table
* clean showpartitions - Show partition level details of a clean
* cleans refresh - Refresh table metadata
* cleans run - run clean
* cleans show - Show the cleans
* clear - Clears the console
* cls - Clears the console
* commit rollback - Rollback a commit
* commits compare - Compare commits with another Hoodie table
* commit showfiles - Show file level details of a commit
* commit showpartitions - Show partition level details of a commit
* commits refresh - Refresh table metadata
* commits show - Show the commits
* commits showarchived - Show the archived commits
* commits sync - Compare commits with another Hoodie table
* compaction repair - Renames the files to make them consistent with the timeline as dictated by Hoodie metadata. Use when compaction unschedule fails partially.
* compaction run - Run Compaction for given instant time
* compaction schedule - Schedule Compaction
* compaction show - Shows compaction details for a specific compaction instant
* compaction showarchived - Shows compaction details for a specific compaction instant
* compactions show all - Shows all compactions that are in active timeline
* compactions showarchived - Shows compaction details for specified time window
* compaction unschedule - Unschedule Compaction
* compaction unscheduleFileId - UnSchedule Compaction for a fileId
* compaction validate - Validate Compaction
* connect - Connect to a hoodie table
* create - Create a hoodie table if not present
* date - Displays the local date and time
* desc - Describe Hoodie Table properties
* exit - Exits the shell
* export instants - Export Instants and their metadata from the Timeline
* hdfsparquetimport - Imports Parquet table to a hoodie table
* help - List all commands usage
* metadata refresh - Refresh table metadata
* quit - Exits the shell
* refresh - Refresh table metadata
* repair addpartitionmeta - Add partition metadata to a table, if not present
* repair corrupted clean files - repair corrupted clean files
* repair deduplicate - De-duplicate a partition path contains duplicates & produce repaired files to replace with
* repair overwrite-hoodie-props - Overwrite hoodie.properties with provided file. Risky operation. Proceed with caution!
* savepoint create - Savepoint a commit
* savepoint delete - Delete the savepoint
* savepoint rollback - Savepoint a commit
* savepoints refresh - Refresh table metadata
* savepoints show - Show the savepoints
* script - Parses the specified resource file and executes its commands
* set - Set spark launcher env to cli
* show archived commits - Read commits from archived files and show details
* show archived commit stats - Read commits from archived files and show details
* show env - Show spark launcher env by key
* show envs all - Show spark launcher envs
* show fsview all - Show entire file-system view
* show fsview latest - Show latest file-system view
* show logfile metadata - Read commit metadata from log files
* show logfile records - Read records from log files
* show rollback - Show details of a rollback instant
* show rollbacks - List all rollback instants
* stats filesizes - File Sizes. Display summary stats on sizes of files
* stats wa - Write Amplification. Ratio of how many records were upserted to how many records were actually written
* sync validate - Validate the sync by counting the number of records
* system properties - Shows the shell's properties
* temp_delete - Delete view name
* temp_query - query against created temp view
* temp delete - Delete view name
* temp query - query against created temp view
* temps_show - Show all views name
* temps show - Show all views name
* utils loadClass - Load a class
* version - Displays shell version

yeah, unfortunately we don't expose this as of today. We have a tracking ticket https://issues.apache.org/jira/browse/HUDI-1593. We will try to get it in next release.
@abhijeetkushe : can we close this support ticket if you are good w/ clarifications.

Yeah this ticket can be closed.We are using EMR built in version of hudi so will be hard for us to upgrade.So for now we have build checks to ensure this does not happen

got it, thanks.

repair corrupted clean files

Can this thing be automated by using some shell script that takes the input of the last successful commit timestamp and deletes the files after that timestamp in the .hoodie folder and other files that should be deleted?

I was able to proceed after deleting the<PHONE_NUMBER>5129 files from .hoodie folder in s3.But I wanted to know whether anything else needs to be cleaned up ? For example there might have been some versions created for commit time<PHONE_NUMBER>5129 which i can get from<PHONE_NUMBER>5129.commit file.Does that need to be cleaned up and if not why ?

Can this thing be automated by using some shell script that takes the input of the last successful commit timestamp and deletes the files after that timestamp in the .hoodie folder and other files that should be deleted?

@souradeep852 yes I automated deletion of this file.But I think they have made some fixes in the higher version so upgrade might solve this


setData / commit completion handler not called

[READ] Step 1: Are you in the right place?

For issues or feature requests related to the code in this repository
file a Github issue.

If this is a feature request make sure the issue title starts with "FR:".

For general technical questions, post a question on StackOverflow
with the firebase tag.
For general Firebase discussion, use the firebase-talk
google group.
For help troubleshooting your application that does not fall under one
of the above categories, reach out to the personalized
Firebase support channel.

[REQUIRED] Step 2: Describe your environment

Xcode version: 10.1
Firebase SDK version: 5.18.0
Firebase Component: Firestore
Component version: 1.0.2

[REQUIRED] Step 3: Describe the problem
Steps to reproduce:
My batch completion block is not always called.
My device is online (I can access Google through Safari without any problem).
I don't use any authentication system: all users are anonyme and able to write into the database.
It is possible that the client tries to perform another batch about the same documents before the previous batch completion block has been called.
I am currently having the problem from Brazil. Connection looked fine all day long but it might be an issue...
Relevant Code:
        print("Saving selectable images remotely...")

        let batch = NetworkManager.database.batch()

        for selectableImage in selectableImages {
            let ref = NetworkManager.database.document("selectableImages/\(selectableImage.id)")

            let selectableImageDictionary = formatForFirebase(dictionary: selectableImage.dictionary)

            batch.setData(selectableImageDictionary, forDocument: ref)
        }

        batch.commit { (error) in
            // NOT ALWAYS CALLED
        }

Also, my app now crashes with Message from debugger: Terminated due to memory issue when I try to restart it. Could it be because Firestore SDK still try to send all documents waiting to get saved?
I am really starting to think that the issue is due to my Internet connection.

I just removed my app from my phone and reinstalled it again: no more issues at the moment.
I guess the app had a lot of documents waiting to be sent, maybe causing an overflow or something?
I will re-open this if the problem shows again.


[TODO] activation fn registry

While the most widely-used activation functions are predefined, it'd be better to have some sort of registry so that the serde deserialization works with custom activations.
It would also be good to note that there should be a separate flag or something for those activation functions which do not want to be randomly mutated to/fro.

hmm this seems impractical to implement. delaying.


Encode Document Reference So that It can be written in userdefaults as well

Hi Support!
First of all thanks for a wonderful framework!
Actually, I was trying to encode DocumentReference as well but it shows me an error, Unsupported Type for UserDefaults DocumentReference.
Can you guys help me in this issue.

Hi! Have you used the steps from here which encoder are you using?

Yes I also used extensions which are defined in that doc.
It encodes but it doesn't encode in the way it can save into UserDefaults.

Well, there is no way to decode a Document Reference into JSON. You have use it's documentData and decode it turn.

The document reference is something specific to Firebase. You should only use it with FirestoreEncoder that is why it gives you an error.

Yes agree, I have to use FirestoreEncoder, and it encode it, but I can't save the object which contains DocumentReference and encoded with FirestoreEncoder

If you need to use it somewhere else, you need to avoid having DocumentReference in your model whatsoever and use something else. I hope this is helpful, but it is a limitation that you need to work with


Update sbt-scala-native to 0.4.14

About this PR
📦 Updates org.scala-native:sbt-scala-native from 0.4.4 to 0.4.14
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-native", artifactId = "sbt-scala-native" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scala-native", artifactId = "sbt-scala-native" }
}]

labels: sbt-plugin-update, early-semver-minor, semver-spec-patch, commit-count:1

Superseded by #195.


bug: Missing Nonce - Sign In with Apple on iOS with Javascript SDK

It seems that the nonce returned by FirebaseAuthentication.signInWithApple() is invalid, or provider.credential is not building the credential correctly.
Code:
export async function appleLogin() {
  // 1. Create credentials on the native layer
  const result = await FirebaseAuthentication.signInWithApple()
  // 2. Sign in on the web layer using the id token and nonce
  const provider = new OAuthProvider('apple.com')
  const credential = provider.credential({
    idToken: result.credential?.idToken,
    rawNonce: result.credential?.nonce
  })
  await signInWithCredential(auth, credential)
}

Error:
Unhandled Promise Rejection: FirebaseError: Firebase: Nonce is missing in the request. (auth/missing-or-invalid-nonce).

Thank you for your request. Are the two properties result.credential?.idToken and result.credential?.nonce defined or undefined?

having the exact same error, but idToken and nonce are defined (at least in my case)

@robingenz both are defined :/ I think it's an issue with the provider.credential method

code:
const credential = new OAuthProvider('apple.com').credential({
    idToken: result.credential?.idToken,
    rawNonce: result.credential?.nonce
  })
  console.log('credenital:' + JSON.stringify(credential))

result:
credenital:{"idToken":"XXXX","pendingToken":null,"providerId":"apple.com","signInMethod":"apple.com"}
as you can see, it build a credential without the nonce

and credential.rawNonce warns: Property 'rawNonce' does not exist on type 'OAuthCredential'.

@mdepasquale21 @robingenz Found a workaround for now:
  const credential = OAuthProvider.credentialFromJSON({
    providerId: 'apple.com',
    signInMethod: 'apple.com',
    idToken: result.credential?.idToken,
    nonce: result.credential?.nonce,
    pendingToken: null
  })

Great! Thanks for your investigation @roldyclark.
In this case, nonce appears to be neither missing nor invalid.
For this reason, I am closing this issue.

@roldyclark
this release note seems sus....

I'll check it and report my findings : D


Update nf-d3d11-id3d11videocontext-decoderbeginframe.md

Remove the redundant function name to avoid confusion.

Yes, I removing this is ok (the two titles are still unique). The title for ID3D11VideoContext3::DecoderBeginFrame1 should say something like "Starts a decoding operation to decode a video frame, with histogram options". That's how we should be disambiguating titles, not just sticking the API's name at the end of the description. Which is, as you say, confusing.


Good Work 👍

Good job this is much better than the Arduino ide in my preference keep up the work.

Closing issue as it is more of a comment than a issue.