Fixed Box-QP + removed unnecessary virtual functions in SolverDDP and SolverBoxDDP - [closed]

In GitLab by @cmastalli on Oct 13, 2019, 21:51
Merges topic/fix-box-ddp -> devel
The BoxDDP was not running in computer because we need to defined the dimension of H_inv.
This is a quick fix, however we need to work properly on the Box-QP and to add an abstract for QP solvers.
@wxmerkt could you review this PR?

In GitLab by @wxmerkt on Oct 13, 2019, 22:24
This fix is already part of !244 - please use !244 in place of this one.
I don't think we can remove the virtual keywords. This was the source of some bugs where the incorrect function in the vlookup table was called before (cf. !234) due to hiding.

In GitLab by @cmastalli on Oct 13, 2019, 22:52
I didn't know it, I forgot about this PR. But I can see that the dimension of H matrix is wrong, isn't it?
I removed the virtual functions that are not necessary in the SolverDDP. I only keep functions that are re-implemented in BoxDDP, i.e. allocateData, computeGains and forwardPass. This code is OK because the SolverDDP.backwardPass at runtime takes SolverBoxDDP.computeGains and the SolverDDP.tryStep takes at runtime the SolverBoxDDP.forwardPass. If you define unecessary virtual functions then we create a vtable that is resolved at runtime, so it is not very efficient.
On the other hand, it is not needed to define virtual functions in the SolverBoxDDP; we are not deriving anything from this class.

In GitLab by @cmastalli on Oct 13, 2019, 22:54
I merged !244. If the dimension is wrong, then I will fix it here. Otherwise I will revert the change of the dimension.

In GitLab by @wxmerkt on Oct 13, 2019, 23:07
Ok, but I left this on purpose to allow switching the underlying solver with a few lines

In GitLab by @wxmerkt on Oct 13, 2019, 23:08
Please leave the virtual keywords. If you are "overriding" a virtual declared in the base class, you need to have the keyword in the derived class. This was the bug fixed earlier in !234

In GitLab by @wxmerkt on Oct 13, 2019, 23:09
I think we should revert the changes to the virtual keywords. Can you show a benchmark that shows a performance/efficiency penalty? (vs the use of boost.python for instance)

In GitLab by @Ivan on Oct 13, 2019, 23:19
I agree with Wolf here. If the method is virtual in the parent class it has to be virtual in the child class. If you don't want it to be virtual, rename the method. If you want to prevent the child classes further down the inheritance tree (not the case here), c++11 has a final keyword for that. c++98 doesn't support that.
This FAQ talks about different reasons why this is a bad idea. Even though it's legal and it may produce some minor speed improvement (doubtful), it will definitely open up the code for a lot of bugs. Compilers are clever and will probably optimize most of the cases where vtable doesn't have to be used.

vtable that is resolved at runtime, so it is not very efficient

This is just not true in general and completely negligible on any method that takes longer than 10ns (possibly less) to run. This post stated 7ns overhead on a CPU from 2009. These optimizations break the code and are mostly based on forum comments from 10 years ago or more. I strongly recommend running tests to see if these improvement make a difference (on several CPUs) and using tools like valgrind to find bottlenecks rather than misusing the language.

In GitLab by @cmastalli on Oct 14, 2019, 17:54
@wxmerkt I see your point but I believe we should have two solvers: BoxDDP and BoxFDDP.
I am planning to start to work on this topic, and I am particular interested in the BoxFDDP. I think it would be a nice extension of the submitted paper.

In GitLab by @wxmerkt on Oct 14, 2019, 18:05
We have discussed working in this direction as well. Perhaps let's sync up when you're here.

In GitLab by @cmastalli on Nov 7, 2019, 09:45
No need of this PR, it was handle in !234

In GitLab by @cmastalli on Nov 7, 2019, 09:45
closed


Dose floating-pointing compute instruction check NaN when FLEN > SEW ?

If No,
1. it is possible to load double-precision by fld, compute it as single point in  vector and store back to floating-point register.  The result is meaningless
2.  implementation such as spike needs two different API to access the floating-point register
Based on v-spec 11.1
"For floating-point operation, the scalar can be taken from a scalar f register. If FLEN>SEW, the least-significant bits of the fregister are used. If FLEN<SEW, the value is NaN-boxed (one-extended) to SEW."
Based on user-spec  2.2 p56
"Floating-point compute and sign-injection operations calculate results based on the FLEN-bit values held in the f registers. A narrow n-bit operation, where n < FLEN, checks that input operands are correctly NaN-boxed, "

It looks like the spec currently says the answer is “no”, but I’m not sure why. Checking for a NaN box in an f-register when FLEN > SEW seems like a reasonable thing to do. @kasanovic any comment?

This was an oversight - the scalar args to vector-scalar operations should behave the same as regular scalar args.


select 'IDE Packages' does implicit set lua version for the plugin to '5.1', while selcted version is different

status ZeroBrandStudio:

Project-LuaInterpreter: Lua 5.4
Project Modules shows_ "x modules found (Lua 5.4)"
UserModules shows:  "x modules found (Lua 5.4)"

select "IDE Packages"...

Project Modules shows_ "x modules found (Lua 5.1)"
UserModules shows:  "x modules found (Lua 5.1)"

This is an expected behavior. ZeroBrane Studio's internal interpreter is LuaJIT (compatible with Lua 5.1).
I'm considering removing the interpreter version label from the "IDE Packages" to avoid confusion.

Good catch! Fixed. Thanks!
Could you update this module and test to verify if it is behaving correctly now? :)

Thanks for the fast response.
ZBS starts in the LuRocks-Plugin with "Project Modules (Lua 5.4)". I then switch to "IDE Packages" gives info "(Lua 5.1)" After some in-between also Project-Modules get (Lua5.1) again - from my point of view not what was expected.

Thanks for the report. I confirmed this issue as well. I rewrote the code for the 'IDE Packages' tab so that this doesn't happen anymore. Can you confirm for me if it's okay after updating the package again?

Looks very good! Fine! .. as expected! - THX
The questions is how to deal with "IDE Packages". From the log in the terminal, you could see that 5.1 is retrieved. For me it seems reasonable to show than also "(Lua 5.1)" - alternatively all are handled in the same way and IDE will be empty for >5.1.
What also could be a good idea: To provide a possibility (toggle button)  to include the repos to the package.path and package.cpath for execution/debug. That would be really nice!!

Looks very good! Fine! .. as expected! - THX

Great. Thank you for testing!

The questions is how to deal with "IDE Packages". From the log in the terminal, you could see that 5.1 is retrieved. For me it seems reasonable to show than also "(Lua 5.1)"

The 'IDE Packages' tab was quite challenging to implement. ZeroBrane's packages must always be installed in the same version as the IDE interpreter (Lua[JIT] 5.1). Therefore, when changing the interpreter, one should always revert to Lua 5.1 when accessing this tab.

alternatively all are handled in the same way and IDE will be empty for >5.1.

I understand. For example, if I install the 'document map' package, but my project uses Lua 5.4, it doesn't make sense for me to have to change the interpreter to Lua 5.1, remove the package, and then revert again.
It is convenient for this plugin to make this temporary change automatically.
I can add a label in the 'IDE Packages' tab with (ZBS Lua version: 5.1) to imply that it refers to the IDE's internal Lua interpreter.

What also could be a good idea: To provide a possibility (toggle button) to include the repos to the package.path and package.cpath for execution/debug. That would be really nice!!

Good idea. Currently, I don't have time to make such significant changes :)
An alternative is to manually edit the 'packages.path' with the modules you want to debug. Or, temporarily uninstall the modules and leave only the desired ones.
-- Thanks!

would like it for the IDE packages: "(ZBS lua 5.1)"
a great step forward  - Thanks a lot!

would like it for the IDE packages: "(ZBS lua 5.1)"

Added. Now displays (ZBS Lua 5.1) in the IDE Packages tab. It relies on Lua's _VERSION.
-- Thank you for the suggestions!


346 tls support

Fixes #346

Tests only fail when run with the Earthly target.
I changed qemu to send the serial output to a file. I use find inside the earthly container to find the state dir (which I print before the tests start). Then I tail the serial.log file in that directory. The result is that the installation finishes but when it reboots, there are failures among which:
[  OK  ] Finished cOS system early rootfs setup.
[FAILED] Failed to start kcrypt online mount.
See 'systemctl status kcrypt.service' for details.

so something is wrong when trying to decrypt the filesystem.

I think I know what the problem might be. Locally I built the .iso using code that includes this fix: https://github.com/kairos-io/kairos/pull/836 . Earthly doesn't have an iso around so it uses BASE_IMAGE=quay.io/kairos/core-opensuse:latest to build one. This doesn't include the DNS fix, thus booting fails. I should have thought of this earlier :facepalm:
All we need is an updated BASE_IMAGE with the fix.

I think I know what the problem might be. Locally I built the .iso using code that includes this fix: kairos-io/kairos#836 . Earthly doesn't have an iso around so it uses BASE_IMAGE=quay.io/kairos/core-opensuse:latest to build one. This doesn't include the DNS fix, thus booting fails. I should have thought of this earlier facepalm
All we need is an updated BASE_IMAGE with the fix.

Additionally, now opensuse images are split in opensuse-leap and opensuse-tumbleweed, so the BASE_IMAGE used here is not getting any updates (see https://kairos.io/docs/reference/image_matrix/)

I split each test to its own job (with matrix). This way, they can run in parallel (provided we have enough workers) and they can be re-tried separately if one fails.
I'm also thinking about running then outside Earthly. The Earthly target is rather simple, just running a script. Hopefully we can run them with KVM enabled when if we manage to enable that on our self-hosted workers. This will reduce the total time by a lot.

After splitting the tests in separate jobs the tests went green: https://github.com/kairos-io/kcrypt-challenger/pull/12/commits/1e3efb57cc64e3a05456d171f10d454b25ce1119
I did one more change to run the tests without earthly: https://github.com/kairos-io/kcrypt-challenger/pull/12/commits/27114b8db8472edba5b7c4813899cf482b670662
and tests are still green.
I want to try to run some self hosted runners with KVM enabled and see what change that makes. This can happen on another PR but it will define whether we run inside Earthly or outside. I can split that last commit to a separate PR and experiment there.

I have no idea how this works, codewise looks good to me

Well, the implementation is pretty much these 2 lines: https://github.com/kairos-io/kcrypt-challenger/pull/12/files#diff-94d75f189ea8b05ff1476dce6652e33bf43bdb1fddade2736156f8366b4eb22bR72-R73
The rest is dancing with tests. This feature needs documentation. I created a new issue for this here: https://github.com/kairos-io/kairos/issues/890


Missing code block on step 4 of building a balance sheet

Describe the Issue
in step 4 of building a balance sheet, the hint in line 16 is missing code blocks

Your existing span element should have the class attribute set to flex.

This should be changed to

Your existing span element should have the class attribute set to flex.

https://github.com/freeCodeCamp/freeCodeCamp/blob/85a110e2642eee73f94b450ddf5ddff9c13854db/curriculum/challenges/english/14-responsive-web-design-22/learn-more-about-css-pseudo-selectors-by-building-a-balance-sheet/61fd67a656743144844941cb.md#L16
Additional context
If you would like to fix this issue, please make sure you read our guidelines for contributing, we prioritize contributors following the instructions in our guides. Join us in our chat room or the forum if you need help contributing, our moderators will guide you through this.
Sometimes we may get more than one pull requests. We typically accept the most quality contribution, followed by the one that is made first.
Happy contributing.

I can take care of this!


Implements identity match

fixes #9
👋
I did my best to follow design patterns herein, but feel free to nitpick on style. I'll take no offense if you decide to merge it and rewrite it in your style, too! 😄
I added a few typo-fix commits as well, if you're welcome to them.
In addition to the mock-http style testing, I've tested this by linking our app to the local copy of elixir-plaid and running the request against the sandbox API.

Thank you very much for contributing! ❤
I've merged this and will publish an updated version of the package here shortly.

Alright I've pushed up v1.2.0 with your changes! Check out the docs https://hexdocs.pm/elixir_plaid/Plaid.Identity.html#match/3
I did end up making some minor changes for consistency within the project. You can see that commit here.
Thanks again for your help!


Update checkout action from v2 to v3

Close: #33

Codecov Report
Base: 100.00% // Head: 100.00% // No change to project coverage :thumbsup:

Coverage data is based on head (48c78a6) compared to base (0b26130).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #34   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            2         2
  Lines           23        23
=========================================
  Hits            23        23

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


LCS Asset Library Data Residency?

Is there information anywhere about the geographic location of the LCS asset library anywhere?  In particular, where do database exports (bacpacs) reside?  I haven't been able to find a clear answer

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

ID: 53431d28-b101-57eb-84bf-738bdef09ba9
Version Independent ID: 68aabf63-900f-beb3-21cf-a936c08f6e7a
Content: Guide to the GDPR for Finance and Operations - Finance & Operations | Dynamics 365 | #MSDyn365FO
Content Source: articles/dev-itpro/gdpr/gdpr-guide.md
Service: dynamics-ax-platform
GitHub Login: @toddlefor
Microsoft Alias: ToddLefor

I think this has been addressed. Please reopen if not.


Opening Document to Read Properties

When opening a document to read properties, I am trying to use the SolidEdgeCommunity.Reader.SolidEdgeDocument.Open method but it is requiring that the file is not already open and it opens the file as write. I tried to use a STREAM of the file instead but when trying to get the PropertySet, I get
"Unable to cast object of type 'SolidEdgeCommunity.Reader.Native.ReadOnlyIStorage' to type 'SolidEdgeCommunity.Reader.Native.IPropertySetStorage'."
Is there something wrong here or can we not read an already opened file with Community.Reader?
Code with FileStream:
using (var fs = new FileStream(fileLocationPath, FileMode.Open, FileAccess.Read))
{
var document = SolidEdgeCommunity.Reader.SolidEdgeDocument.Open(fs);
var propertySets = document.PropertySets;
}

I agree. You should be able to call SolidEdgeCommunity.Reader.SolidEdgeDocument.Open() on a file that is currently open in Solid Edge. I'll dig into it and see if I can get it fixed. Thanks for letting me know.

I have a fix for both issues that you've raised and am in the process of testing.

Awesome, thanks Jason.
On Monday, August 31, 2015, Jason Newell<EMAIL_ADDRESS>wrote:

I have a fix for both issues that you've raised and am in the process of
testing.
—
Reply to this email directly or view it on GitHub
https://github.com/SolidEdgeCommunity/SolidEdge.Community.Reader/issues/3#issuecomment-136546100
.

Published 2.0.0 to NuGet. Update your package reference and see how it goes. Thanks.

Closing old issue.


force disable code signing in iOS (probably needed better coding)

due I'm thinking about what if there's people using code signing??
if u guys want me change smth u can add me in discord (cuz I can't sometimes see gh notifications and username mc.9)

What if someone wants to sign their code using <certificate team-id="..." /> in project.xml? This has been in place for at least 6 years.
I'd say try project.xml first, and if you're certain it doesn't work for you, IOSPlatform.hx is the correct place to make the change.

What if someone wants to sign their code using <certificate team-id="..." /> in project.xml? This has been in place for at least 6 years.
I'd say try project.xml first, and if you're certain it doesn't work for you, IOSPlatform.hx is the correct place to make the change.

that's what I meant to say before but making them empty doesn't do anything and gives "code signing requires by (app name)" at actions

making them empty (in Project.xml) doesn't do anything

I doubt an empty string is a valid team ID, so I suppose we could skip IOSHelper.sign() in that case.

many users can't or don't know how to get DUNS number

...But you don't need a DUNS number, you need an Apple-assigned certificate ID. And last I checked, Apple requires all developers to get one, no matter what kind of app they're making.
Unless you're saying there's a GitHub Action that accesses your Apple account and does that for you?

...But you don't need a DUNS number, you need an Apple-assigned certificate ID. And last I checked, Apple requires all developers to get one, no matter what kind of app they're making.

I know and tried in actions with my account didn't work (doesn't have any DUNS number)

Unless you're saying there's a GitHub Action that accesses your Apple account and does that for you?

There's something here for make code signing but many users I talk says "I don't trust" idk really

There's something here for make code signing but many users I talk says "I don't trust" idk really

Most of that looks correct, though I'm a bit wary of how they're using environment variables. Any program you ran afterwards would have access to the secrets, though presumably you'd run only trusted ones. Maybe it's fine?
But yeah, you're going to have to do something along those lines no matter what. And once you have the keystore set up, you ought to be able to use Lime's built-in code signing.


Refact deploy script to support gpg and nvsec signature [skip ci]

Signed-off-by: Tim Liu<EMAIL_ADDRESS>

build

CI got PASS for both gpg & nvsec signature

build


Password

I would like to know how to protect the admin panel with a password.
I can see there is a login.html file but I can't find where the password is processed.
Must I build all the validation for the password?

Yes, you must implement this functionality. Here is an example based on PHP & Symfony: Gentelella
If you have no more questions, please close this ticket.

Since this issue has been openes a while ago and you already have a solution we will close it for now but if you have more questions about it , just reopen it , good luck !


Correcting usage link to point to using.md

ooo nice, thanks @cagiti !! looks like you're going to need to sign the cla before we can merge this
/ok-to-test
(also note that #460 is completely removing using.md - but i should make sure to update usage.md to point at the right docs now 😅 )

Hi @bobcatfish, I signed the CLASS after I pushed the PR... not sure whether it is able to check again? 😀

@cagiti looks like #460 resulted in a conflict for your PR, sorry about that. Would you mind resolving the conflict?

I think this is actually fixed in the latest version of the tutorial docs - sorry about this @cagiti , I thought it got merged before I made my change :(
(now that you've got the CLA signed tho we'd love any more fixes you notice :pray: :smile: )


Add FontFace component

:sparkles: Add FontFace resource
:books: Document <FontFace \> and useFontFace
:point_right: Add <FontFace /> example
Resolves #76

When reviewing especially keep an eye on:

Typescript definitions (I'm by no means an expert, would love to get feedback)
Error handling if the browser doesn't support window.FontFace (https://caniuse.com/#feat=font-loading). It throws a TypeError that can be caught by adding an error boundary (not sure if this is the right approach?).

interesting use of suspense. why not offer url and format as props?

No, it's absolutely a valid comment. In fact, I thought about url and format as props myself. The reason I didn't do it is because with source you can add a comma separated list of url and format pairs (like the CSS @font-face-rule); e.g.:
<FontFace
  family="Name"
  source="url(path/to/name.woff2) format('woff2'), url(path/to/name.woff) format('woff')" />

This is great to provide fallbacks if the preferred format is not supported or to refer to a local font as the primary option.
An alternative API could be using source as an array:
<FontFace
  family="Name"
  source={[
    {
      url: 'path/to/name.woff2',
      format: 'woff2',
    },
    {
      url: 'path/to/name.woff',
      format: 'woff',
    },
  ]}
/>

hmm, yea i see. idk, you can also leave it as a string and ask for OSS to write validation regex :)

You do get limited runtime feedback. If you call window.FontFace with an invalid source it will throw a DOMException. You're not completely left in the blind. Might just stick with a string. The reason I opted for a string in the first place was because it aligns with the spec. It's also the same as the @font-face-rule so it's well known and well documented.


Do not strip .0 version for compatible releases

... as the complete version number is important.
See PEP 440 for more details
( https://www.python.org/dev/peps/pep-0440/#compatible-release ).
Fixes #24
modified:   src/tox_ini_fmt/formatter/requires.py
modified:   tests/formatter/test_requires.py

Codecov Report

Merging #30 into main will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main       #30   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            9         9
  Lines          247       247
  Branches        55        55
=========================================
  Hits           247       247

Flag
Coverage Δ

#tests
100.00% <100.00%> (ø)

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

Impacted Files
Coverage Δ

src/tox_ini_fmt/formatter/requires.py
100.00% <100.00%> (ø)

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 b630270...082c071. Read the comment docs.

As mentioned in #24 I have only taken care of the == and >= cases, e.g. not for the <= case.
It is too late to think this problem through :-)


Web- LHN- Black screen appears at the right side of LHN when log in with account

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

Action Performed:

Go to staging.new.expensify.com
Log in with account -<PHONE_NUMBER>/Feya86Katya

Expected Result:
History conversation appears at the right side of LHN when log in with account
Actual Result:
Black screen appears at the right side of LHN when log in with account
Workaround:
Unknown
Platforms:

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

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/93399543/223204783-c13ff6cc-dbda-465b-8309-933febc8c410.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01a7358c7bdf598634
Upwork Job ID:<PHONE_NUMBER>974841344
Last Price Increase: 2023-03-07

Triggered auto assignment to @bfitzexpensify (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

Bug0 Triage Checklist (Main S/O)

[ ] This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
[ ] This bug is not a duplicate report (check E/App issues and #expensify-bugs)

If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead

[ ] This bug is reproducible using the reproduction steps in the OP. S/O

If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
If the reproduction steps aren't clear and you determine the correct steps, please update the OP.

[ ] This issue is filled out as thoroughly and clearly as possible

Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.

[ ] I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

Job added to Upwork: https://www.upwork.com/jobs/~01a7358c7bdf598634

Pretty sure the problem is that we are trying to open a "policy room" and those are likely not accessible to tester. There is a beta called policyRooms that expensifail.com can be used to create policy rooms.

Triggered auto assignment to Contributor Plus for review of internal employee PR - @parasharrajat (Internal)

@marcaaron so the issue is one of limited access via testing, not something we need to do something about?

I think we should look into it and fix it. I'd be surprised if it affects any users besides testers.

Triggered auto assignment to @luacmartins (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

Cool, assigning internally

I'll try to get to it this week.

No updates because of ECX

Working on it this week!

@luacmartins @parasharrajat @bfitzexpensify this issue was created 2 weeks ago. Are we close to a solution? Let's make sure we're treating this as a top priority. Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

I'll try to work on this tomorrow

Reviewing label has been removed, please complete the "BugZero Checklist".

The solution for this issue has been :rocket: deployed to production :rocket: in version 1.2.89-0 and is now subject to a 7-day regression period :calendar:. Here is the list of pull requests that resolve this issue:

https://github.com/Expensify/App/pull/16406

If no regressions arise, payment will be issued on 2023-04-03. :confetti_ball:
After the hold period is over and BZ checklist items are completed, please complete any of the applicable payments for this issue, and check them off once done.

[ ] External issue reporter
[ ] Contributor that fixed the issue
[ ] Contributor+ that helped on the issue and/or PR

As a reminder, here are the bonuses/penalties that should be applied for any External issue:

Merged PR within 3 business days of assignment - 50% bonus
Merged PR more than 9 business days after assignment - 50% penalty

BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:

[ ] [@parasharrajat / @luacmartins] The PR that introduced the bug has been identified. Link to the PR:
[ ] [@parasharrajat / @luacmartins] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:
[ ] [@parasharrajat / @luacmartins] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:
[ ] [@bfitzexpensify] Determine if we should create a regression test for this bug.
[ ] [@parasharrajat] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.
[ ] [@bfitzexpensify] Link the GH issue for creating/updating the regression test once above steps have been agreed upon:

@parasharrajat could you please complete the checklist above?

@parasharrajat thanks for letting me know! Np, I completed the checklist already. Thanks!

Upwork is being a pain - making a note for myself to come back to this later on today to get the job set up for the C+ review here


Use literal syntax to create dict and list

Using the literal syntax can give minor performance bumps compared to using function calls to create dict, list and tuple.
Fixes these DeepSource.io alerts:
https://deepsource.io/gh/DimitriPapadopoulos/numcodecs/issue/PTC-W0019/occurrences
TODO:

[ ] Unit tests and/or doctests in docstrings
[ ] tox -e py39 passes locally
[ ] Docstrings and API docs for any new/modified user-facing classes and functions
[ ] Changes documented in docs/release.rst
[ ] tox -e docs passes locally
[ ] GitHub Actions CI passes
[ ] Test coverage to 100% (Coveralls passes)

I'll leave this open a bit to see if it kicks off any style controversies.

Yes, this one is really debatable.
I do find dict() more readable than {}, at least for empty dictionaries. But then DeepSource.io tags this as a "performance" issue, not a readability issue.

Actually, there's a good reason to change to literal syntax ({} instead of dict()), and that's static typing with mypy. Have a look at the "Word frequencies with a dictionary" example here:
Mypy Examples


Online Editor View issue - doesnt show the nested data objects that it used to show previously

Looks like the 'online swagger editor' got some updates recently which are causing issue with the view section. The nested objects are now hidden for some cases (in object definitions and example section of i/o parameters to the methods). It used to show all objects properly in previous versions.
Attached are before and after screenshots of same yaml file.
Methods - I/O Example section - before

Example section - after

Models - Before

Models - After

Please advise and fix.

Please file the ticket on swagger-editor and follow the issue template.


Update google_drive gem version

fastlane 2.148.0  and babelish 0.6.4  are not compatible because of some dependencies.
gem 'babelish', '0.6.4'
gem 'fastlane', '2.148.0'

Here is the error.
Bundler could not find compatible versions for gem "google-api-client":
  In snapshot (Gemfile.lock):
    google-api-client (= 0.36.4)

  In Gemfile:
    fastlane (= 2.148.1) was resolved to 2.148.1, which depends on
      google-api-client (>= 0.37.0, < 0.39.0)

    babelish (= 0.6.4) was resolved to 0.6.4, which depends on
      google_drive (~> 3.0.4) was resolved to 3.0.4, which depends on
        google-api-client (>= 0.11.0, < 0.37.0)

Running `bundle update` will rebuild your snapshot from scratch, using only
the gems in your Gemfile, which may resolve the conflict.

This PR fixes this issue.
Thanks

@4brunu so I checked and I think there is no need for a new release yet as the dependency is ~> 3.0.4. A bundle update  will resolve to 3.0.5


Icons from icon-park slow page loading.

@alili Let's see what we can do about it.

We packed the icon file in the repo.


[QUESTION] How to achieve grayscale processing and binary processing of images?

How to achieve grayscale processing and binary processing of images?

Something like this should work for grayscale.
using (var bitmap = SKBitmap.Decode("path/to/image"))
using (SKCanvas canvas = new SKCanvas(bitmap))
{
  using var paint = new SKPaint();

  // Define a grayscale color filter to apply to the image
  paint.ColorFilter = SKColorFilter.CreateColorMatrix(new float[]
  {
          0.2126f, 0.7152f, 0.0722f, 0, 0,  // red channel weights
  0.2126f, 0.7152f, 0.0722f, 0, 0,  // green channel weights
  0.2126f, 0.7152f, 0.0722f, 0, 0,  // blue channel weights
  0,       0,       0,       1, 0   // alpha channel weights
  });

  // redraw the image using the color filter
  canvas.DrawBitmap(bitmap , 0, 0, paint);
  }

@cake-escape Thinks

This issue can be marked as solved and be closed.


fix: refactor the context handling of buckets code

so we always pass in a context so its easier to handle timeouts and defer better

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign jstrachan
You can assign the PR to them by writing /assign @jstrachan in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

/lgtm

@jstrachan: you cannot LGTM your own PR.

In response to this:

/lgtm

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

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

Needs approval from an approver in each of these files:

OWNERS [jstrachan]

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


Debug console truncates strings longer than 100 characters - PHP

VSCode Version: 1.15.1
OS Version: Windows

Steps to Reproduce:

Debugging Mode,
When I inspect a string longer than 1.026 characters is truncated.

Reproduces without extensions: Yes/No

Please move this issue to the repository of the PHP debugger you are using.


Fix for gh-pages

@j-delaney : changed the url to the raw readme file.

Thanks but I already made this change!


obevo cli and nested tables oracle

Expected Behavior
When I run the deploy.bat command. I should be able create a export file with DML, DDL.
Actual Behavior
When run the deploy.bat command. The command fails when the DBMS_METADATA runs for nested tables.
Caused by: java.sql.SQLException: ORA-31603: object "TEST_NT" of type TABLE not found in schema "TEST"
ORA-06512: at "SYS.DBMS_METADATA", line 6069
ORA-06512: at "SYS.DBMS_METADATA", line 8666
ORA-06512: at line 1
    at oracle.jdbc.driver.T4CTTIoer11.processError(T4CTTIoer11.java:494)
    at oracle.jdbc.driver.T4CTTIoer11.processError(T4CTTIoer11.java:446)
    at oracle.jdbc.driver.T4C8Oall.processError(T4C8Oall.java:1054)
    at oracle.jdbc.driver.T4CTTIfun.receive(T4CTTIfun.java:623)
    at oracle.jdbc.driver.T4CTTIfun.doRPC(T4CTTIfun.java:252)
    at oracle.jdbc.driver.T4C8Oall.doOALL(T4C8Oall.java:612)
    at oracle.jdbc.driver.T4CStatement.doOall8(T4CStatement.java:213)
    at oracle.jdbc.driver.T4CStatement.doOall8(T4CStatement.java:37)
    at oracle.jdbc.driver.T4CStatement.executeForRows(T4CStatement.java:896)
    at oracle.jdbc.driver.OracleStatement.executeMaybeDescribe(OracleStatement.java:933)
    at oracle.jdbc.driver.OracleStatement.doExecuteWithTimeout(OracleStatement.java:1082)
    at oracle.jdbc.driver.OracleStatement.executeQuery(OracleStatement.java:1276)
    at oracle.jdbc.driver.OracleStatementWrapper.executeQuery(OracleStatementWrapper.java:366)
    at org.apache.commons.dbcp.DelegatingStatement.executeQuery(DelegatingStatement.java:208)
    at org.apache.commons.dbcp.DelegatingStatement.executeQuery(DelegatingStatement.java:208)
    at com.gs.obevo.db.impl.core.jdbc.JdbcHelper.queryAndLeaveStatementOpenInternal(JdbcHelper.java:203)
    ... 10 common frames omitted

Obevo Version where this issue was observed
7.0.2
Steps to reproduce the behavior
Run the deploy.bat command for a oracle db with nested tables.

FYI - I added fixes to properly reverse-engineer user types and nested tables. You can try it here -
https://oss.sonatype.org/content/repositories/snapshots/com/goldmansachs/obevo/obevo-cli/master-SNAPSHOT/obevo-cli-master-20190217.190948-12-dist.zip
See the example here: https://github.com/goldmansachs/obevo/tree/master/obevo-db-impls/obevo-db-oracle/src/test/resources/platforms/oracle/example1/step1/schema1. Note the type created under "usertype" and the "NESTED_TABLE" example under /table

@shantstepanian  This is still erroring out on Nested tables.
Caused by: java.sql.SQLException: ORA-31603: object "TEST_NT" of type TABLE not found in schema "TEST"
ORA-06512: at "SYS.DBMS_METADATA", line 6069
ORA-06512: at "SYS.DBMS_METADATA", line 8666
ORA-06512: at line 1
    at oracle.jdbc.driver.T4CTTIoer11.processError(T4CTTIoer11.java:494)
    at oracle.jdbc.driver.T4CTTIoer11.processError(T4CTTIoer11.java:446)
    at oracle.jdbc.driver.T4C8Oall.processError(T4C8Oall.java:1054)
    at oracle.jdbc.driver.T4CTTIfun.receive(T4CTTIfun.java:623)
    at oracle.jdbc.driver.T4CTTIfun.doRPC(T4CTTIfun.java:252)
    at oracle.jdbc.driver.T4C8Oall.doOALL(T4C8Oall.java:612)
    at oracle.jdbc.driver.T4CStatement.doOall8(T4CStatement.java:213)
    at oracle.jdbc.driver.T4CStatement.fetch(T4CStatement.java:1009)
    at oracle.jdbc.driver.OracleStatement.fetchMoreRows(OracleStatement.java:3353)
    at oracle.jdbc.driver.InsensitiveScrollableResultSet.fetchMoreRows(InsensitiveScrollableResultSet.java:736)
    at oracle.jdbc.driver.InsensitiveScrollableResultSet.absoluteInternal(InsensitiveScrollableResultSet.java:692)
    at oracle.jdbc.driver.InsensitiveScrollableResultSet.next(InsensitiveScrollableResultSet.java:406)
    at org.apache.commons.dbcp.DelegatingResultSet.next(DelegatingResultSet.java:207)
    at org.apache.commons.dbcp.DelegatingResultSet.next(DelegatingResultSet.java:207)
    at org.apache.commons.dbutils.handlers.AbstractListHandler.handle(AbstractListHandler.java:47)
    at org.apache.commons.dbutils.handlers.AbstractListHandler.handle(AbstractListHandler.java:33)
    at com.gs.obevo.db.impl.core.jdbc.JdbcHelper.query(JdbcHelper.java:173)
    ... 8 common frames omitted

Is there a way to exclude these problem tables in obevo ? Some how Nested tables and DBMS_metadata are not working together. I see below message in sql developer.
-- Unable to render TABLE DDL for object TEST.TEST_NT with DBMS_METADATA attempting internal generator.

Will look into allowing exclusions to be specified. Will aim for end of next week
Separately, are you able to get the table DDL for TEST_NT via SQL Developer? If so, can you send here?

@shantstepanian I can get you the DDL. The problem seems to be with dbms_metadata.getddl and nested tables. The dbms_metadata.getddl doesnt support nested tables. This is confirmed by oracle support. I have found few articles on stackoverflow as well. If you see below thread they are excluding nested tables. Looks like nested tables don't have DDL. The DDLis part of main table.
https://stackoverflow.com/questions/10886450/how-to-generate-entire-ddl-of-an-oracle-schema-scriptable
http://www.orafaq.com/forum/t/181601/0/
comment from above thread:
Of course, there is no DDL for a nested table (this is why I said you have to exclude some kinds of table).
A nested table is not a standalone object, it is part of another table. It is the same than you cannot have the DDL for a column.

I should have a snapshot ready for you to test in a couple days. A couple notes:

I implemented your requirement in a different way. Instead of allowing you to exclude objects to reverse-engineer, I improved the Oracle reverse-engineering to be on a best-effort basis. So instead of failing the whole process, any "bad" objects will be written to a "unableToClassify-pleaseAnalyze" folder, which you can take a look at and either resolve or ignore/delete once you've taken a look

I did previously try to handle nested tables - see the example code (notably the examples named NESTED_TABLE) and the reverse engineering code (notably the clause "tab.NESTED is null OR tab.NESTED = 'NO'"). So ideally, this code would work and handle your use case; otherwise, I'm missing a way to detect nested tables. However, given the fix I did above, you should be able to proceed with reverse engineering

Here is the updated SNAPSHOT version with the fixes described in the previous comment - https://oss.sonatype.org/content/repositories/snapshots/com/goldmansachs/obevo/obevo-cli/master-SNAPSHOT/obevo-cli-master-20190404.025731-16-dist.zip
Let me know how that works for you

@shantstepanian  I am getting below error. It doesnt like dbtype oracle for some reason.
java.lang.IllegalArgumentException: Failed to validate argument ORACLE for java.beans.PropertyDescriptor[name=dbTypeStr; propertyType=class java.lang.String; writeMethod=public void com.gs.obevo.db.apps.reveng.AquaRevengArgs.setDbTypeStr(java.lang.String)]
at com.sampullara.cli.Args.setProperty(Args.java:267)
at com.sampullara.cli.Args.processProperty(Args.java:221)
at com.sampullara.cli.Args.parse(Args.java:39)
at com.gs.obevo.util.ArgsParser.parse(ArgsParser.java:45)
at com.gs.obevo.util.ArgsParser.parse(ArgsParser.java:40)
at com.gs.obevo.dist.Main$7.value(Main.java:206)
at com.gs.obevo.dist.Main$7.value(Main.java:203)
at com.gs.obevo.dist.Main.execute(Main.java:119)
at com.gs.obevo.dist.Main.execute(Main.java:87)
at com.gs.obevo.dist.Main.main(Main.java:69)

@shantstepanian
I am getting below error. It used to work. Am I missing something ?
java.lang.IllegalArgumentException: Failed to validate argument ORACLE for java.beans.PropertyDescriptor[name=dbTypeStr; propertyType=class java.lang.String; writeMethod=public void com.gs.obevo.db.apps.reveng.AquaRevengArgs.setDbTypeStr(java.lang.String)]
at com.sampullara.cli.Args.setProperty(Args.java:267)
at com.sampullara.cli.Args.processProperty(Args.java:221)
at com.sampullara.cli.Args.parse(Args.java:39)
at com.gs.obevo.util.ArgsParser.parse(ArgsParser.java:45)
at com.gs.obevo.util.ArgsParser.parse(ArgsParser.java:40)
at com.gs.obevo.dist.Main$7.value(Main.java:206)
at com.gs.obevo.dist.Main$7.value(Main.java:203)
at com.gs.obevo.dist.Main.execute(Main.java:119)
at com.gs.obevo.dist.Main.execute(Main.java:87)
at com.gs.obevo.dist.Main.main(Main.java:69)

@shantstepanian  Looks good. It is able to classify the nested tables in to unableToClassify-pleaseAnalyze folder. But I do see a editionable trigger. Below is the DDL.
CREATE OR REPLACE EDITIONABLE TRIGGER "TEST"."AABED_LOGON_TRG"
AFTER LOGON ON AABED.schema
DECLARE
BEGIN
EXECUTE IMMEDIATE 'ALTER SESSION SET CURRENT_SCHEMA=TEST_APPS';
END;
/
ALTER TRIGGER "TEST"."AABED_LOGON_TRG" ENABLE
GO

It can, though possibly we haven't enabled it for Oracle. It would be a quick change, I'll check it within the next month

Continuing this issue for the trigger question on ticket #232


[MVP]integrate LDAP as external auth service

see #1 . This MVP version of LDAP integration as an external authentication service does not currently support TLS (Transport Layer Security). Other detailed explanations can be found in the README.md file.

seems the github action will only run after this PR merged? Let's finished the smalls comments, and merged it as the first version. Thanks!

It will run in my repository, and you can find it there.


color output

Hi,
I see colors been given for error, normal,.. output here ConsoleReporter.js . I don't see any color in my terminal (Mac). Doest it supposed to work?

recently using jest exclusively. so don't need this anymore.


added Greater Houston, TX, USA

The low ratio of points to city blocks in downtown Houston is really odd-looking:

My guess is big office towers.
On Mon, Apr 10, 2017 at 11:23 AM, Michal Migurski<EMAIL_ADDRESS>wrote:

The low ratio of points to city blocks in downtown Houston is really
odd-looking:
[image: screen shot 2017-04-10 at 11 22 36 am]
https://cloud.githubusercontent.com/assets/58730/24876316/192a5a4e-1de0-11e7-8553-71d8f24e7d60.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/openaddresses/openaddresses/pull/2840#issuecomment-293036067,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA0EO8_NDhoWMYLd4k7L1YStJZwDCKFIks5runOqgaJpZM4M5GOy
.

That area of Houston appears to host a large number of parking lots:  https://www.google.com/maps/place/Polk+St,+Houston,+TX/@29.7517589,-95.3684643,1194m/data=!3m1!1e3!4m5!3m4!1s0x8640bee8b4f9acb9:0xb7dbdd853b26cc1d!8m2!3d29.7411006!4d-95.3427029


How to save a fixed-size svg or png?

I have adjust the width(1000) and height(400) of the pane in chrome, but still get 700*450 svg or png when saving the pane.

Interacting with the pane doesn't actually resize the svg, just the view of it. Thus saving it with the download link isn't modifying your uploaded svg at all.

Interacting with the pane doesn't actually resize the svg, just the view of it. Thus saving it with the download link isn't modifying your uploaded svg at all.

Thx. So how to get the fixed-size svg since there are some labels that need to be displayed but are omitted

You will have to use an svg editor of some sort? I'm not really sure what you're trying to accomplish.

You will have to use an svg editor of some sort? I'm not really sure what you're trying to accomplish.

I mean how to download a  svg as the view of its pane?
For example, I want to save a svg with 1000400 instead of 700400 because I want its x-axis to display more columnnames.


Fix link checker

Description
Added missing files to fix link checker check
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

would you mind update the snapshots as well?


Refactor REST layer into multiple classes

FHIRResource implements the entire HTTP API surface area of the FHIR server and weighs in at over 4000 lines of code.
Although it hasn't shown up on the Profiler yet, my understanding is that the JAX-RS framework instantiates this class for each request, which may limit our scalability.
It should be possible to give each Rest endpoint its own JAX-RS class, while keeping the doX methods (doCreate, doUpdate, etc), which perform the heavy lifting, in some static/stateless layer.
Alternatively, John mentioned we could use a Registry approach where the rest layer is able to look up interactions dynamically. This might simplify the design of the Operations framework as well which currently accepts a FHIRResourceHelper, which is actually backed by the full FHIRResource instance, on each invocation.
Can we make better use of IBM CDI (context and dependency injection)?

@lmsurpre let's talk about this.  It's not actually accurate to say it's instantiated on every request.

I think I actually did this already as part of issue #17

Reviewed as part of the merged code which splits into REST Helper and corresponding classes.  This reduced the cognitive load for the FHIRResource Class.


Add a test for the '--emit binary' option

Progress!
Only thing left is to figure out how to setup clang for Windows CI.


Prevent cluster getting stuck in "upgrading" state after upgrading the rke2 version for an imported rke2 cluster

Rancher Server Setup

Rancher version: 2.6.6
Installation option (Docker install/Helm Chart): Helm

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc): RKE2

Rancher 2.6.6 Local Cluster ( v1.21.7+rke2r2 )
I ran into this issue when upgrading my Cluster from v1.21.7+rke2r2  to v1.22.10+rke2r2
There is a similar older issue here #34067 for reference.
Most nodes did work it seems, but now its stuck with two nodes left

With this failed job in the cluster

This is the failed jobs info ( kubectl describe)

Click to expand!
❯ kubectl describe job apply-rke2-worker-plan-on-rke2-downstream1-agent-2-with-7-94b29 -n cattle-system
Name:                     apply-rke2-worker-plan-on-rke2-downstream1-agent-2-with-7-94b29
Namespace:                cattle-system
Selector:                 controller-uid=7f1c561d-09c0-4bd5-ae29-959bd2faff63
Labels:                   objectset.rio.cattle.io/hash=6641dc529612dc10f1658bf00333df03480e1367
                        plan.upgrade.cattle.io/rke2-worker-plan=75375c2bb1dd80940ce0f2f48102d8f4aef7c8b3fb6febf3cd9f82a1
                        upgrade.cattle.io/controller=system-upgrade-controller
                        upgrade.cattle.io/node=rke2-downstream1-agent-2
                        upgrade.cattle.io/plan=rke2-worker-plan
                        upgrade.cattle.io/version=v1.22.10-rke2r2
Annotations:              batch.kubernetes.io/job-tracking:
                        objectset.rio.cattle.io/applied:
                          H4sIAAAAAAAA/+xY227bOBN+lf/nteRKspPYBvbCG7tbo41j1GkXRREEFDmyuaFILTlyYhh+9wUp2ZEbO4cuFuhFUKARKc5wDt83mvGa5ICUU6SkvyZUKY0UhVbWLXX6FzC0gC0jdI...
                        objectset.rio.cattle.io/id: system-upgrade-controller
                        objectset.rio.cattle.io/owner-gvk: upgrade.cattle.io/v1, Kind=Plan
                        objectset.rio.cattle.io/owner-name: rke2-worker-plan
                        objectset.rio.cattle.io/owner-namespace: cattle-system
                        upgrade.cattle.io/ttl-seconds-after-finished: 900
Parallelism:              1
Completions:              1
Completion Mode:          NonIndexed
Start Time:               Tue, 05 Jul 2022 09:45:47 +0200
Active Deadline Seconds:  900s
Pods Statuses:            0 Active / 0 Succeeded / 1 Failed
Pod Template:
Labels:           controller-uid=7f1c561d-09c0-4bd5-ae29-959bd2faff63
                  job-name=apply-rke2-worker-plan-on-rke2-downstream1-agent-2-with-7-94b29
                  plan.upgrade.cattle.io/rke2-worker-plan=75375c2bb1dd80940ce0f2f48102d8f4aef7c8b3fb6febf3cd9f82a1
                  upgrade.cattle.io/controller=system-upgrade-controller
                  upgrade.cattle.io/node=rke2-downstream1-agent-2
                  upgrade.cattle.io/plan=rke2-worker-plan
                  upgrade.cattle.io/version=v1.22.10-rke2r2
Service Account:  system-upgrade-controller
Init Containers:
 prepare:
  Image:      rancher/rke2-upgrade:v1.22.10-rke2r2
  Port:       <none>
  Host Port:  <none>
  Args:
    prepare
    rke2-master-plan
  Environment:
    SYSTEM_UPGRADE_NODE_NAME:             (v1:spec.nodeName)
    SYSTEM_UPGRADE_POD_NAME:              (v1:metadata.name)
    SYSTEM_UPGRADE_POD_UID:               (v1:metadata.uid)
    SYSTEM_UPGRADE_PLAN_NAME:            rke2-worker-plan
    SYSTEM_UPGRADE_PLAN_LATEST_HASH:     75375c2bb1dd80940ce0f2f48102d8f4aef7c8b3fb6febf3cd9f82a1
    SYSTEM_UPGRADE_PLAN_LATEST_VERSION:  v1.22.10-rke2r2
  Mounts:
    /host from host-root (rw)
    /run/system-upgrade/pod from pod-info (ro)
 drain:
  Image:      rancher/kubectl:v1.20.2
  Port:       <none>
  Host Port:  <none>
  Args:
    drain
    rke2-downstream1-agent-2
    --pod-selector
    !upgrade.cattle.io/controller
    --ignore-daemonsets
    --delete-local-data
    --force
  Environment:
    SYSTEM_UPGRADE_NODE_NAME:             (v1:spec.nodeName)
    SYSTEM_UPGRADE_POD_NAME:              (v1:metadata.name)
    SYSTEM_UPGRADE_POD_UID:               (v1:metadata.uid)
    SYSTEM_UPGRADE_PLAN_NAME:            rke2-worker-plan
    SYSTEM_UPGRADE_PLAN_LATEST_HASH:     75375c2bb1dd80940ce0f2f48102d8f4aef7c8b3fb6febf3cd9f82a1
    SYSTEM_UPGRADE_PLAN_LATEST_VERSION:  v1.22.10-rke2r2
  Mounts:
    /host from host-root (rw)
    /run/system-upgrade/pod from pod-info (ro)
Containers:
 upgrade:
  Image:      rancher/rke2-upgrade:v1.22.10-rke2r2
  Port:       <none>
  Host Port:  <none>
  Environment:
    SYSTEM_UPGRADE_NODE_NAME:             (v1:spec.nodeName)
    SYSTEM_UPGRADE_POD_NAME:              (v1:metadata.name)
    SYSTEM_UPGRADE_POD_UID:               (v1:metadata.uid)
    SYSTEM_UPGRADE_PLAN_NAME:            rke2-worker-plan
    SYSTEM_UPGRADE_PLAN_LATEST_HASH:     75375c2bb1dd80940ce0f2f48102d8f4aef7c8b3fb6febf3cd9f82a1
    SYSTEM_UPGRADE_PLAN_LATEST_VERSION:  v1.22.10-rke2r2
  Mounts:
    /host from host-root (rw)
    /run/system-upgrade/pod from pod-info (ro)
Volumes:
 host-root:
  Type:          HostPath (bare host directory volume)
  Path:          /
  HostPathType:  Directory
 pod-info:
  Type:  DownwardAPI (a volume populated by information about the pod)
  Items:
    metadata.labels -> labels
    metadata.annotations -> annotations
Events:
Type     Reason            Age                  From            Message
----     ------            ----                 ----            -------
Normal   SuccessfulCreate  22m                  job-controller  Created pod: apply-rke2-worker-plan-on-rke2-downstream1-agent-2-with-7-6xlpz
Normal   SuccessfulDelete  7m6s                 job-controller  Deleted pod: apply-rke2-worker-plan-on-rke2-downstream1-agent-2-with-7-6xlpz
Warning  DeadlineExceeded  7m6s (x2 over 7m6s)  job-controller  Job was active longer than specified deadline

And in the cluster view

@erSitzt When you look at the logs for that job, do you see anything that sticks out?
That job is what we use with the system-upgrade-controller with https://github.com/rancher/rke2-upgrade to perform the actual upgrade of RKE2.

Do you mean the logs of the pod that was started by the job ?
The pod was already deleted:
Events:
  Type     Reason            Age    From            Message
  ----     ------            ----   ----            -------
  Normal   SuccessfulCreate  18m    job-controller  Created pod: apply-rke2-worker-plan-on-rke2-downstream1-agent-2-with-7-kxt6f
  Normal   SuccessfulDelete  3m23s  job-controller  Deleted pod: apply-rke2-worker-plan-on-rke2-downstream1-agent-2-with-7-kxt6f
  Warning  DeadlineExceeded  3m23s  job-controller  Job was active longer than specified deadline

If there is anything useful i can provide, i can leave the cluster in this state, otherwise i could try to update to the next version and get the logs while the update is running...

not stale

@erSitzt - are you still stuck in this issue? I believe the root cause seems to be clear - the PDB has blocked the drain and the drain timed out.. or do I miss something? So what type of help do you need, now?

Let me rephrase the issue

Ok - so changing the default job timeouts should be one thing that should be done..  that gives more time to debug and troubleshoot in case of PDBs blocking or other issues are there..
The other thing to check "are all applications implemented in a fault tolerant way with nothing having replica=1" is something we could add to documentation I assume, even though I assume this might be considered as kubernetes best practices..

not stale...


Refactor tab 'focus' event to be emitted when the tab is already rendered

Send proper events for

tab added
tab removed
tab show
tab focused

so other components can react to that.

Sending this back for now. We need to find a more complete solution for the tab life-cycle.

WOOOOOON'T fix.


16-bit depth PNG too bright when imported as texture

Godot version:
3.2.2
and also on master 6e55a4fe8c799b5f19e03e9cdc959772efeedc55
OS/device including version:
Xubuntu 20.04, GTX1080
Issue description:
16-bit depth PNG images become brighter when imported into Godot. Flat surfaces
Steps to reproduce:
Place a 16-bit depth PNG file in a Godot res:// directory so it is imported
Minimal reproduction project:
png16.zip

Providing a screen shot in case it's caused by some third party library and not reliably reproducible. The images look identical outside Godot.

Duplicate of https://github.com/godotengine/godot/issues/37220? Godot doesn't support 16-bit PNG images. To import HDR images, you need to use the Radiance HDR (.hdr) or OpenEXR (.exr) formats instead.

Yep, didn't find it when I was searching for old tickets


Get Hadamard Basis as output in the draw function?

What should we add?
Is it possible to visualize the latex state in the Hadamard Basis instead of the normal Computational Basis. In the state_vector.draw(output='latex'), can a basis param be added to visualize it in Hadamard basis instead of computational.
Please excuse if this is already implemented. Is there already a workaround present for this? I want to deal with Dirac notation and the draw function really helps. I don't want to get into the Bloch sphere part. I just need my bra-kets to contain +/-

We don't currently support this, and there isn't a particular workaround as things stand because the drawers all just use the internal memory of the statevector, which is stored in the computational basis, and generate the ket labels for the indices of each data element.
I'm somewhat hesitant about this feature, because it would be quite specific to this one drawer and might have a messy interface if we try to be too general, but I think if the feature is kept very small in scope, it could be ok to include.
The approximate plan would be to make qiskit.visualization.state_visualization.state_to_latex take an extra keyword argument ket_basis that defaults to z, but can take hadamard (and x or y if they're useful).  It'd pass those down to whatever functions also needed them, including ket_name.  To actually do the mapping, probably we'd just apply x, y or h on all the qubits as appropriate, and then pass the new array down to the drawers.
This could be good for an external contributor looking for a little more than a bugfix.

I am looking at this particular function.
The basic idea is to modify the **drawer_args  to include a ket_basis argument for this particular drawer.
It's always an np.log2 size for List[complex]
I can therefore make sure that I modifying a copy of the data variable to change the indices and the data to get the right ket basis
For example -
[alpha beta delta gamma] to [1/2 (a + b + g + t) 1/2 (a - b - g + t) 1/2 (a + b - g - t) 1/2 (a - b + g - t)] would mean applying an H gate. Since the data stored is in the state_vector's data variable is always in the computational basis, does the above logic make sense?
I have a few questions -

Can I add more comments on my feat branch to understand it a bit better?
How is latex managed on Github issues? It seems like an open issue

I plan on changing this particular function -
https://github.com/Qiskit/qiskit-terra/blob/0041b6ff262c7132ee752b8e64826de1167f690f/qiskit/visualization/state_visualization.py#L1238-L1262

@jakelishman - Is there someone who can help me out with this?

Sorry about that, I missed the comment somehow.

Yeah, applying the relevant operators to a copy of the Statevector (Operator(HGate()) @ sv or something) is a fine way to go.  You can use the regular machinery in Operator to handle all the gates/tensor-product ordering for you.
It's fine to add a comment or two if it's confusing how/why something works - it wasn't a conscious choice not to comment the drawers.
You're in luck - LaTeX support was (finally) added to GitHub issues about two days after your comment (use $ <inline equation> $ or $$ <displayed equation> $$).

$$
\begin{pmatrix}
\alpha\
\beta
\end{pmatrix}
\to \frac1{\sqrt2}
\begin{pmatrix}
\alpha + \beta
\
\alpha - \beta
\end{pmatrix}
$$
(though just trying it then, I will say its whitespace handling leaves something to be desired...)

@jakelishman  - Any idea on how to get this fix in place?

Do I need to downgrade setuptools for this to work?

Command to reproduce  - pip install .
   File "/usr/lib/python3/dist-packages/pkg_resources/__init__.py", line 792, in resolve
          raise VersionConflict(dist, req).with_context(dependent_req)
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
      pkg_resources.VersionConflict: (setuptools 45.2.0 (/usr/lib/python3/dist-packages), Requirement.parse('setuptools>=58.0'))

Ah glad to hear it!  I'm guessing it was to do with your Python install being from your package manager, and some of the core machinery (setuptools, pkg_resources and things like that) being a bit out of date.
Just in case you didn't know: we usually suggest working in virtual environments, so you can install everything you need without affecting your system's Python installation.  We have a little bit of documentation on setting up virtual environments if you need it.


Wildcard capture oddities.

When capturing wildcards at the beginning or end of an input, I've noticed some inconsistent behaviour.
+ *(1-5) test
> input: this too test => expected: this too => captured: this too (AS EXPECTED)
> input: what you want test => expected: what you want => captured: what be you want (UNEXPECTED)
>>> the word you gets replace with 'be you' (this only happens when it's captured at the BEGINNING of an input)

+ test *(1-5)
> input: test this too => expected: this too => captured: this (UNEXPECTED)
> input: test what you want => expected: what you want => captured: what you want (AS EXPECTED)
>>> the word 'too' gets dropped (this only happens when it's captured at the END of an input)

Okay, so the good news is the capture system is working as expected. In both cases you are getting hit by text normalization rules.
"what you" is becoming "what are you" and then being changed to the root form "be"
And we strip "too" off the end of the sentences because it adds no extra value to the input (just noise)
https://github.com/silentrob/normalizer/blob/master/data/substitutes.txt#L1459

Right - makes sense.  Thx @silentrob.
Also leaving this here for future reference: weird behaviour can easily be investigated by running the DEBUG util: DEBUG=* node index.js


feat: 🔥 support bodyFormat

added new "bodyFormat" configuration option that accepts string or function, and added function option to "headerFormat"
PR Checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] UI changes have been reviewed
[ ] No UI review needed

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: N/A
What is the new behavior?
Does this PR introduce a breaking change?

[ ] Yes
[ ] No

Other information

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


UNIT TESTS - components/DropdownInput

What
Create unit tests using Jest and React Testing Library.
Here are a couple examples of how unit tests should be created:

Unit tests for our A component
Unit tests for Icon component
Unit tests for Button component

General Acceptance Criteria for writing Unit tests

Should create a [nameOfComponent].test.tsx within the same directory of the component.
Should have tests that tests every prop of the component.
If there are any conditional logic within the component, there should be unit tests that accounts for each conditional branches.
npm run test should be passing.

Specific Acceptance Criteria

make sure you tests for each prop

I'll take this.

@denistsoi please do self assign the issue / get it assigned by commenting and check if the issue is already assigned to someone else before you start working on it. Much easier to keep track that way, and would save everyone's. Say for instance, this issue itself. I was working on this issue, I would've moved on to something else if I knew you were working on it too.


Not working in Mac OS Big Sur Version 11.0.1 (20B29)

Description:
UI elements do not display correctly, getting a null pointer exception when trying to open an imported project.

Suggested Labels:

Suggested Assignees:

Affected Product Version:
Integration Studio 7.1.0
OS, DB, other environment details and versions:
Mac OS 11.0.1 (20B29)
Steps to reproduce:
Open integration studio and try to open a project in workspace.
Related Issues:

Have same issue. Fix it please.

1 for a fix

Hi All,
This happens due to the underlying eclipse framework we are using in Integration Studio.
Following are some of the related issues.
https://stackoverflow.com/questions/64838171/eclipse-oxygen-3-a-not-working-macos-big-sur-java-1-8-0-271
https://bugs.eclipse.org/bugs/buglist.cgi?quicksearch=[big sur]
We are actively working on fixing this issue. Hopefully, we'll able to provide a fix soon.
~ WSO2 Integration Team

Hi,
Any update on the above?

Hi All,
We have released WSO2 Integration Studio 7.2.0-Beta on top of the Eclipse 4.16.0.v20200604 version that works on macOS Big Sur. The tool can be downloaded via the following link:
https://github.com/wso2/devstudio-tooling-ei/releases/tag/v7.2.0-beta
~ WSO2 Integration Team

Edit>Undo
Edit>Redo
Edit>Copy
Edit>Paste
Those options are not working at all.

Hi All,
We have fixed the undo/redo issue and released a beta 2 version. Please try beta2[1] and provide feedback.
Thanks!
[1] https://github.com/wso2/devstudio-tooling-ei/releases/tag/7.2.0-beta2

@rosensilva connectors are not getting imported.


Appstore paging fix

Problem

[x] quickhide widget button should have confirm dialogue
[x] can't page apps
[x] remove featured apps from all apps
[x] feature dartfrog
[x] app entries make too many requests to /apps

Testing

[x] Try to hide a widget. Confirm popup asks permission.
[x] Page apps.
[x] Confirm no featured apps show up in appstore "all apps" section.
[x] Find dartfrog in featured apps list.
[x] On the main appstore page, verify the Network tab shows that only a couple requests to /apps have been made, as opposed to each app button making its own request.

Notes
Sorry about the messy rebase.

np, rebase looks good


Feature Request: numbered entities with custom counters

Having a way to provide a counter for an enum to use would be great for one use case I have right now, which involves defining a custom numbered list, but it would also enable resuming the counting of one enum from another, like when you "suspend" the list with some text in between items.
Replicating this idea on other numbered entities would also easen customization, like defining separate numbering on front matter / main matter / back matter, or continuous numbering for Theorems, Lemmas, etc.

I have a different use case, but I believe this is relevant here.
I feel that we should be able to provide a counter for an enum, or be able to retrieve the value of a counter from an enum.
*My use case: * I'm trying to create a link to a particular step of an enum.
I can use a link successfully with a label, but I wondered if I can auto-generate the text from the enum's counter, e.g. See #link()[...].  I tried using a ref but this of course currently requires a figure.
I tried to use locate() and query() to display a counter value, but when trying to use counter(enum),  I get error: enum is not locatable
I tried using a custom figure kind, and my own counter, but had problems there too.
From looking at the typst source, it appears enums indeed do not use counters. Perhaps they should, like headings? The logic seems to be sufficiently similar (e.g. use of numbering vs level, etc)

Better integration between enums and counters seems very reasonable. One big difference though is that counters are always global to the document whereas enums are generally counted locally.

I ended up here by searching how to skip a number when using a numbered list. The idea is that something like this:
#set enum(start: 2)

+ Bla

+ Bla bla

#counter(enum).step()

+ Bla bla bla

Should be printed as:
2. Bla
3. Bla bla
5. Bla bla bla

I had a very similar problem to the one of @lingo, but my workaround didn't work perfectly:
source

#locate(loc => counter("source").final(loc).first())
#show enum.item: it => {
  counter("source").step()
  it
}
+ item
+ item
+ item

output

Using set enum(numbering) instead of show enum.item works flawlessly:
source

#locate(loc => counter("source").final(loc).first())
#set enum(full: false, numbering: (n) => {
  counter("source").update(n)
  numbering("1.", n)
})
+ item
+ item
+ item

output

I can't seem to be able to adapt @Andrew15-5 's workaround to both manage multi-leveled numbered lists (such as "1)a)") and reuse the saved counter in a new list. Does anyone have an idea ?

Yeah, multi-level numbering is not as simple. I haven't tried it with that, but it might work with some improved version. For that, the argument must be ..n instead of n, that is for sure.

 query(<step-link>, before: loc).first()

This part of @lingo's example currently gives the error "missing argument: location". How should one write this with the current syntax?
I'm trying to figure out if the above examples from lingo and @Andrew15-5 will work with the solution given here to allow enums to be referenced later in the document (that solution works, but only if you number your numbered list manually as in the example
1. amongus <label>
2. amongus
3. trimongus <labelled>

because the number property in typst only seems to be set if you number your list manually, not automatically.
...which makes that solution a lot less useful, and also seems to make it impossible to reference those items in a way not using numbers (roman numerals, letters, etc.)

The 0.10 (currently released) syntax is query(selector(<step-link>).before(loc), loc). With the upcoming 0.11, it will become query(selector(<step-link>).before(here())) and should be wrapped in a context block instead of a locate callback.

Will the current usage/syntax be deprecated or removed completely?

@laurmaedje, thank you, that works. Is there a way to put @lingo's solution into a function, though, so that it's not so wordy for every reference to a label?
I tried
let l (label, numbering: "1.") = {
  link(label)[#locate(loc => {
      let item = query(selector(label).before(loc), loc).first();
      [#{numbering(numbering,..s.at(item.location()))}]
  })]
}

But it says "only element functions can be used as selectors".
If I can get syntax of the function correct, I'd test it like so:
#[
  // create a counter
  #let s = counter("number")
  #{ s.update(c => 1) } // start at 1

  // ensure all enums (in this block?) update the counter before rendering
  #set enum(numbering: it  => {
    s.update(number => it)
    numbering("1.", it) // can't seem to find out what numbering the enum is using
  })

  Text

  + point 1
  + point 2 <step-link2>

Reference above point: #l(<step-link2>, "A)")
]

Ultimately it would be nice to use @step-link2, but I'm just looking for something that works and is relatively concise for now.


feat(@angular/google-maps): Add cluster support

Feature Description
Currently @angular/google-maps does not support marker clusters (https://github.com/googlemaps/js-markerclustererplus). It'd be great if it did.
Use Case
We're trying to transition off AGM (https://github.com/SebastianM/angular-google-maps) and onto the official package but cannot do so fully until marker clusters are supported.

This was added in https://github.com/angular/components/pull/20944 which should be released with v11.1.0.

@crisbeto is a timeline for when v11.1.0 will be released - Looking forward to use this app


Fixes typo in contributing.md file

Small PR to fix a typo in the contributing.md file.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Ignore load err that stops Python Ext from loading

For #5145

Codecov Report

Merging #5146 (9293fb0) into main (3d558a3) will increase coverage by 1%.
The diff coverage is 66%.

@@          Coverage Diff           @@
##            main   #5146    +/-   ##
======================================
+ Coverage     71%     72%    +1%
======================================
  Files        402     402
  Lines      26392   26393     +1
  Branches    3798    3798
======================================
+ Hits       18741   19098   +357
+ Misses      6091    5707   -384
- Partials    1560    1588    +28

Impacted Files
Coverage Δ

src/client/extension.ts
63% <66%> (-2%)
:arrow_down:

src/client/datascience/kernelSocketWrapper.ts
45% <0%> (-30%)
:arrow_down:

src/client/common/application/encryptedStorage.ts
36% <0%> (-22%)
:arrow_down:

src/client/common/configuration/service.ts
67% <0%> (-15%)
:arrow_down:

...ascience/raw-kernel/rawNotebookSupportedService.ts
75% <0%> (-12%)
:arrow_down:

src/client/datascience/liveshare/liveshare.ts
66% <0%> (-10%)
:arrow_down:

src/client/datascience/notebook/introStartPage.ts
91% <0%> (-9%)
:arrow_down:

.../client/datascience/notebook/kernelWithMetadata.ts
78% <0%> (-8%)
:arrow_down:

src/client/datascience/jupyter/serverUriStorage.ts
82% <0%> (-6%)
:arrow_down:

src/client/datascience/datascience.ts
88% <0%> (-3%)
:arrow_down:

... and 73 more


Caret position not updating when setting selectedRange

What happened?
When manually setting selectedRange after inserting new text the visual position of the caret is wrong. Once I start typing, the text goes to the position specified by selectedRange and the caret updates. It seems like there may be caret position updating logic in the replace() function that is conflicting with the selectedRange setter.
What are the steps to reproduce?
replace(NSMakeRange(0, 0), withText: "Hello, world!")
selectedRange = NSMakeRange(5, 0)

What is the expected behavior?
Caret position is display based on the selectedRange property.

Thanks for opening the issue. I believe it's addressed in #364.


Support for custom icons in AnyCut

It'd be great to see support for custom icons in Anycut.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 3 Apr 2009 at 4:35

i agree - especially custom icons for browser links or set it to use the .ico
for url's

my samsung moment has custom icons for browser shortcuts to facebook and cnn, i
just
want to make more good looking links like those.

Original comment by<EMAIL_ADDRESS>on 3 Dec 2009 at 12:51

Yes Please... i don't want to get some whole custom interface, just the ability
to
customize some choice shortcuts.  Even the ability to color-code would be nice.

Original comment by<EMAIL_ADDRESS>on 17 Feb 2010 at 5:46

Original comment by<EMAIL_ADDRESS>on 17 May 2009 at 4:54

Changed state: Accepted
Added labels: Type-Enhancement, Application-AnyCut
Removed labels: Type-Defect


ModuleNotFoundError: No module named 'colorama'

After downloading with pip install colorama == 0.4.1 command I had tough time dealing and importing the colorama library into PyCharm. Although, importing into the python interpreter is still working. Why is it so? I am using interpreter v3.7.2 on windows 10.
By trying to import colorama in PyCharm, an exception rises:
ModuleNotFoundError: No module named 'colorama'
Any suggestions?

i installed requirements with command : pip install -r requirements.txt and PyCharm detects it well. In File (at the top left corner) -> Settings, please check your project interpreter, see the picture below.

@sean1245780 why are you struggling use the udemy-dl inside PyCharm when its a command line utility, are you planing to  modify the code or you want to use this inside some sort of project.
you don't need to use PyCharm if you want to run udemy-dl.
PS:
don't mind just asking.

@r0oth3x49 fact: pycharm downloads 10 times faster, it's like a horse vs a turtle

@r0oth3x49 fact: pycharm downloads 10 times faster, it's like a horse vs a turtle

strange although on both the code is same.

The easiest solution to your problem is to run
$ pip install --upgrade --user awscli
as this will make sure you have all needed dependencies.


Serde json fails to deserialize vector of enum values with serde(untagged) if one of enum values is a scalar

Serde fails to deserialize vector of untagged enum values when possible enum options are structs and scalars.
Serialization works. Deserialization for single values works, but not for vector.
Err value: Error("data did not match any variant of untagged enum Foo", line: 0, column: 0)'
Full repro - Playground
use serde::{Deserialize, Serialize};

#[derive(Serialize, Deserialize, Debug)]
#[serde(untagged)]
enum Foo {
    A(usize),
    B{x: u8, y: u8 },
}

fn main() {
    let foo_a = Foo::A(10); // Works
    let foo_b = Foo::B {x: 1, y: 2}; // Works
    let vector = vec![foo_a, foo_b]; // Vector doesn't work

    let serialized = serde_json::to_string(&vector).unwrap();
    println!("serialized = {}", serialized); // serialized = [10,{"x":1,"y":2}]

    let deserialized: Foo = serde_json::from_str(&serialized).unwrap();
    println!("deserialized = {:?}", deserialized); // Fails with "did not match any variant"
}

You are serializing a Vec<Foo> but deserializing only a Foo. The code works if you change it to let deserialized: Vec<Foo>.

Jonas, thanks for pointing to a mistake. I created a bad repro example and that lead me to a wrong conclusion. The original deserialization error I have in my code is related
to an already opened bug Deserializing a RawValue inside an untagged enum always fails.
Closing, as it's not a bug/no repro.


Include code to generate figure 3 and 4

Add relevant anomaly scores to results/models
Update analysis.pyandnotebooks/analyze_results.ipynb` with figure 3 and 4

LGTM.


Can't click and drag 360 image on chrome on mac os

The image at vrview-web can't be clicked and dragged on chrome on mac OS, works fine using Firefox. It used to drag fine so just letting you guys know

Just tested it on my mac in chrome. It works fine for me.

That's great, must be just on mine then. Thanks for checking


Slow Query Performance with 100 Threads

Describe the bug
I have using clickhouse-http-client 0.4.5 and I am executing single query where retrieve all records from the 'table' where the primary key is equal to a specific value something like this: "Select * from table where primary_key = ''".
When I run with single threads, the perfomance is around than 30 ms.
When I run with 50 threads, the perfomance is around 200-500 ms.
When I run with 100 threads, the perfomance is around 1 seconds
When I run this command multiple times, I can not see that httpConnection is more than 16 connection

Expected behaviour
I am expecting that even with 100 threads the query should be less than 100 ms
Code example

Error log
Configuration
Environment

Client version:clickhouse-http-client 0.4.5
Language version: Java 8
OS: linux

ClickHouse server

ClickHouse Server version:<IP_ADDRESS>
ClickHouse Server non-default settings, if any:
CREATE TABLE statements for tables involved:
Sample data for all these tables, use clickhouse-obfuscator if necessary

@BorisPengili, Hi can you please change to our new version v0.5.0 we are using a different underlying layer (Apache HTTP Client)
Another question

Is the bottleneck on the ClickHouse server or on the client side
Can you provide more details about CPU consumption on both client & server
Client and server machine types


Undefined imagecreatefromjpeg function

I installed PHP 5.6 with the +gd variant, when I prompt phpinfo() the GD variant is there and enabled, however trying to call imagecreatefromjpeg says the function is undefined. I'm on OSX Yosemite with the Homebrew lookups and gd installed in Homebrew, is there any particular command I need to run?

@Anahkiasen
Hi. Could you please paste the result by the following command:
$ php -r 'var_dump(gd_info());'

According to http://php.net/manual/en/function.imagecreatefromjpeg.php#70832, if jpeg isn't supported on your PHP, you can't call imagecreatefromjpeg.

array(11) {
  'GD Version' =>  string(26) "bundled (2.1.0 compatible)"
  'FreeType Support' =>  bool(false)
  'T1Lib Support' =>  bool(false)
  'GIF Read Support' =>  bool(true)
  'GIF Create Support' =>  bool(true)
  'JPEG Support' =>  bool(false)
  'PNG Support' =>  bool(true)
  'WBMP Support' =>  bool(true)
  'XPM Support' =>  bool(false)
  'XBM Support' =>  bool(true)
  'JIS-mapped Japanese Font Support' =>  bool(false)
}

@Anahkiasen
Thank you! Since JPEG Support seems to be off, you should enable it. Mac OS X has a known issue on building PHP with GD extension, so try to install PHP with the following options (specify jpeg dir):
$ phpbrew install php-5.6.4 +default \ -- --with-libdir=lib/x86_64-linux-gnu \ --with-gd=shared \ --enable-gd-natf \ --with-jpeg-dir=/usr \ --with-png-dir=/usr

Running the following command I get this error during Configuring extensions:
configure: error: Cannot find OpenSSL's libraries

Configuring extensions
checking size of long... (cached) 8
checking size of int... (cached) 4
checking for int32_t... yes
checking for uint32_t... yes
checking for sys/types.h... (cached) yes
checking for inttypes.h... (cached) yes
checking for stdint.h... (cached) yes
checking for string.h... (cached) yes
checking for stdlib.h... (cached) yes
checking for strtoll... yes
checking for atoll... yes
checking for strftime... (cached) yes
checking which regex library to use... php
checking whether to enable LIBXML support... yes
checking libxml2 install dir... /usr
checking for xml2-config path... /usr/bin/xml2-config
checking whether libxml build works... yes
checking for OpenSSL support... yes
checking for Kerberos support... no
checking whether to use system default cipher list instead of hardcoded value... no
checking for DSA_get_default_method in -lssl... no
checking for X509_free in -lcrypto... yes
checking for pkg-config... /usr/local/bin/pkg-config
configure: error: Cannot find OpenSSL's libraries

Hi @Anahkiasen. I think you should specify openssl option explicitly. Add +openssl=/usr to the previous options and run phpbrew install again.

Hm same result it seems

Uhm... I don't use phpbrew on Mac OS X and I have no idea. Since openssl problem isn't a matter of concern, let's ignore it for now.
I suggested the above options to show how to give --with-jpeg-dir option, so you can use your current options which succeeded in running phpbrew install. The combination of your current options and --with-jpeg-dir option will fix this issue.

I think you can install gd extension separately by running the command below?
$ phpbrew install gd2 -- --with-libdir=lib/x86_64-linux-gnu --with-gd=shared  --enable-gd-natf --with-jpeg-dir=/usr \ --with-png-dir=/usr

Hi @Anahkiasen you don't have to recompile php, please take a look at this entry: https://github.com/phpbrew/phpbrew/wiki/Cookbook#setting-up-prefered-lookup-prefix

Also this one: https://github.com/phpbrew/phpbrew/wiki/Troubleshooting#i-cant-install-extension-why

Homebrew prefix lookup should already be improved, please try self-update and install that again.


corrected examples/package.json so that it is valid

The examples/package.json had one extra comma and an invalid package name.

Thanks!


document the breaking changes with the upcoming garm operator

What is the feature you would like to have?
the upcoming garm operator version requires some manual changes (github credetials name --> github credentials CR).
We have to document the implementation behavior and the needed steps in the release notes.
Anything else you would like to add?
We should also update the entire documentation to make use of v1beta

dedicated issue for garm-provider-k8s: https://github.com/mercedes-benz/garm-provider-k8s/issues/71


Unity 2017 beta, error on compile

In case this comes up for anyone else, I just installed the latest Unity beta, 2017.1.0b56 to give it a try, and using latest HoloLens code. A single error blocks build:
CustomMaterialEditor.cs, line 103
matEditor.BeginAnimatedCheck(scaleOffsetProp);

Looks like the signature changed and it now requires (Rect totalPosition,...)  as a first param. Easy fix to get it to compile (though not sure of any other impact).
matEditor.BeginAnimatedCheck(new Rect(), scaleOffsetProp);

Thanks for the report. We don't normally support the beta versions of the editor. Looks like those methods got depreciated. Let's wait and see if the release will automatically update these or mark them at least as obsolete.

@StephenHodgson Understood.
Just wanted to try out using .NET 4.6 with Unity. That is the only issue I have hit so far.


Brilliance period length loop

Calculation of loop period x length for brilliance

@Gabrielrezende-asc @SergioLordano
Upgrades and fixes finalized


Button with shadow is slow to render on Android

Which platform(s) does your issue occur on?

Android

Please, provide the following version numbers that your issue occurs with:

CLI: 5.2
Cross-platform modules: 5.2.1
Runtime(s): 5.2
Plugin(s): latest

Please, tell us how to recreate the issue in as much detail as possible.

Enable timeline view: https://www.nativescript.org/blog/deep-dive-into-nativescript-3.1-performance-improvements
<Button shadow="4"></Button>

Result:

Yes, that's 600ms to create a button.
Additional context
We it to a label and that got rid of the issue. I believe this is an issue with the state animator (maybe it's too heavy?). Using forcePressAnimation=true with a Label also takes 600ms to create. What's weird about this is that no matter how many Labels you have, only a single instance of a 600ms load will take place, even when they are all the same. Destroying and recreating the component (via *ngIf) also only shows slowdown on the first load.

Further testing needed with 5.3. This might have been an issue with code generation, which was refactored: https://github.com/NativeScript/android-runtime/issues/689

Ok, what the actual hell.
https://play.nativescript.org/?template=play-ng&id=28zxPb
Just calling android.R.integer.config_shortAnimTime and other android.R stuff adds 500ms to the time. andoird.R.integer is a static Java class... I'll have to open an issue with NS to check this

I confirmed that reading android.R variables make the application lag.
Demo application: https://play.nativescript.org/?template=play-ng&id=28zxPb&v=5
I'll check if the same happens on NS 5.3 and open an issue if it does


RuntimeError: Ninja is required to load C++ extensions

Hi,
I'm currently getting the error listed below indicating that I need to have Ninja installed when running deepspeed despite having Ninja 1.10.2 installed. I also tried installing Ninja 1.8 and Ninja 1.9 and followed the steps outlined here yet I still face the same error.
Here's the full traceback:
Using /workspace/.cache/torch_extensions as PyTorch extensions root...
Traceback (most recent call last):
  File "transformers/examples/pytorch/summarization/run_summarization.py", line 651, in <module>
    main()
  File "transformers/examples/pytorch/summarization/run_summarization.py", line 573, in main
    train_result = trainer.train(resume_from_checkpoint=checkpoint)
  File "/workspace/.local/lib/python3.6/site-packages/transformers/trainer.py", line 1166, in train
    self, num_training_steps=max_steps, resume_from_checkpoint=resume_from_checkpoint
  File "/workspace/.local/lib/python3.6/site-packages/transformers/deepspeed.py", line 426, in deepspeed_init
    deepspeed_engine, optimizer, _, lr_scheduler = deepspeed.initialize(**kwargs)
  File "/workspace/.local/lib/python3.6/site-packages/deepspeed/__init__.py", line 129, in initialize
    config_params=config_params)
  File "/workspace/.local/lib/python3.6/site-packages/deepspeed/runtime/engine.py", line 294, in __init__
    self._configure_optimizer(optimizer, model_parameters)
  File "/workspace/.local/lib/python3.6/site-packages/deepspeed/runtime/engine.py", line 1132, in _configure_optimizer
    self.optimizer = self._configure_zero_optimizer(basic_optimizer)
  File "/workspace/.local/lib/python3.6/site-packages/deepspeed/runtime/engine.py", line 1397, in _configure_zero_optimizer
    communication_data_type=self.communication_data_type)
  File "/workspace/.local/lib/python3.6/site-packages/deepspeed/runtime/zero/stage2.py", line 131, in __init__
    util_ops = UtilsBuilder().load()
  File "/workspace/.local/lib/python3.6/site-packages/deepspeed/ops/op_builder/builder.py", line 370, in load
    return self.jit_load(verbose)
  File "/workspace/.local/lib/python3.6/site-packages/deepspeed/ops/op_builder/builder.py", line 409, in jit_load
    verbose=verbose)
  File "/usr/local/lib/python3.6/dist-packages/torch/utils/cpp_extension.py", line 1091, in load
    keep_intermediates=keep_intermediates)
  File "/usr/local/lib/python3.6/dist-packages/torch/utils/cpp_extension.py", line 1302, in _jit_compile
    is_standalone=is_standalone)
  File "/usr/local/lib/python3.6/dist-packages/torch/utils/cpp_extension.py", line 1373, in _write_ninja_file_and_build_library
    verify_ninja_availability()
  File "/usr/local/lib/python3.6/dist-packages/torch/utils/cpp_extension.py", line 1429, in verify_ninja_availability
    raise RuntimeError("Ninja is required to load C++ extensions")
RuntimeError: Ninja is required to load C++ extensions

I'm using the following libraries:
torch: 1.8.0
transformers: 4.15.0.dev0
deepspeed: 0.5.8
ninja: <IP_ADDRESS>
The original code I was trying to run was:
deepspeed --num_gpus=1 transformers/examples/pytorch/summarization/run_summarization.py \
--deepspeed transformers/tests/deepspeed/ds_config_zero2.json \
--model_name_or_path t5-small --per_device_train_batch_size 2   \
--output_dir output_dir --overwrite_output_dir --fp16 \
--do_train --max_train_samples 500 --num_train_epochs 3 \
--dataset_name samsum

Apologies, actually managed to fix this by adding "/workspace/.local/bin:" to my PATH:
my_env = os.environ.copy()
my_env["PATH"] = "/workspace/.local/bin:" + my_env["PATH"]
os.environ.update(my_env)


Update cast handling to deal with new libcudf casting limitations

libcudf recently changed handling of timestamp casting, see https://github.com/rapidsai/cudf/pull/5745.  cudf will no longer support casting from an arbitrary type into a timestamp -- it must be either a duration or an integer of INT32 for TIMESTAMP_DAYS or INT64 for the other timestamp types.

build

build


Scheduled upward merges for numbered branches in wazuh-security-dashboards-plugin repository - Fortnight #11

Description
The goal is to complete the following tasks, each task must have a PR associated with this issue.
The branches must follow the naming: merge-<ORIGIN_BRANCH>-into-<DESTINATION_BRANCH>. For example, merge-4.8.1-into-4.9.0.
Tasks
4.8.1

[ ] Update changelog.
[ ] Merge branch 4.8.1 into branch 4.9.0.

Repository created in 4.9.0, no action needed


testing/prediction code

Hello, thank you for your open source code!
Could you provide the code related to testing or prediction?

From the code you published, I could not find any relevant code about model inference after training.


Forward flags to clang-format

I have been using the clang-format package, which is obsolete, and I've been looking for replacements. This one caught my eye recently for being used by node itself. However, it doesn't work for my use case.
I have a script that expects the clang-format command to be either the exact same as or a thin wrapper around the actual clang-format binary, and passes it flags such as --dry-run and --Werror. I was quite surprised when doing that with this package causes a TypeError of all things!
Will this package support forwarding flags like these, or is that out of scope for this project?

This is one of the objectives of this project.
This project has two objectives:

To provide a simple string-to-string function devoid of IO operations for direct use in browsers.
To offer a drop-in alternative to the original clang-format, leveraging wasm to achieve better cross-platform capabilities than the original clang-format.

Currently, the first objective has been achieved, and the second objective is to wrap the format function and re-implement it in node.
I am considering directly compiling and exporting the main function from the original clangFormat.cpp to reduce the cost of maintenance.

I just tried again with the new release, and got this absolutely massive syntax error:
log.txt
npx clang-format gives me a syntax error right away, but node node_modules/.bin/clang-format doesn't. I think it might be trying to execute the JS as bash? The file doesn't appear to have a shebang, unlike the neighboring clang-format-diff.

You are right, it lacks the shebang.

The upcoming LLVM repository version is 20; I'm unsure if releasing a 19.1.1 is advisable.
I hope there's no version number conflict with the LLVM repository.


fix: update to SCC profile IBM Cloud Framework for Financial Services 1.7.0

/run pipeline

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

GitHub release
v1.18.10

Your semantic-release bot :package::rocket:


[Feature Request]: Add support for Cohere

Problem Description
Could you add support for Cohere?
Solution Description
API Reference: https://docs.cohere.com/reference/about
Alternatives Considered
No response
Additional Context
No response

https://github.com/ChatGPTNextWeb/ChatGPT-Next-Web/issues/4030   We will gradually support existing large models and providers in the near future

You can use OpenRouter, it supports Cohere models.
OpenRouter : https://openrouter.ai/
OpenRouter Integration : https://github.com/ChatGPTNextWeb/ChatGPT-Next-Web/discussions/3361#discussioncomment-8429206

Thank you very much.
于 2024年4月11日 GMT+08:00 15:17:22, Andrew Wu @.***> 写到:

You can use OpenRouter, it supports Cohere models.
OpenRouter : https://openrouter.ai/
OpenRouter Integration : https://github.com/ChatGPTNextWeb/ChatGPT-Next-Web/discussions/3361#discussioncomment-8429206
--
Reply to this email directly or view it on GitHub:
https://github.com/ChatGPTNextWeb/ChatGPT-Next-Web/issues/4481#issuecomment-2049067312
You are receiving this because you authored the thread.
Message ID: @.***>


Temporary workaround for Crash in Date.description[SR-2462]

I think the fundamental questions I had about this patch are still there.
Where else in Foundation do we need to do this to avoid a crash? Or is it specific to time zone for some reason? Maybe it's a CFTimeZone bug if so, and we should fix it there.

@saiHemak Have you been able to reproduce this crash lately? I filed the original bug SR-2462 but I have just tried to recreate the test cases in the original jira and I cant reproduce the crash anymore use the latest master or 3.1 snapshots.
@parkera If I recall correctly the original issue was not specific to CFTimeZone but it was due to a swift compiler bug in the generation of an extra release under some circumstances. I will have another look at the testcase when I get some time but it may well be that the issue has been fixed unless there is still a known issue.

@spevans  yes its resolved with latest master .Thank you .


Sa/bug fix oct23

Updated atan2 to new firedrake syntax. Also commenting out ray hooks until that gets implemented.
Most importantly, updated the IPCS step function in solver.py in accordance with this breaking update to Firedrake. It seems that performing an "fd.assemble" on our integral expression over the control surface no longer outputs a Function to represent this operation, but uses a CoFunction to represent the dual space. We require the l2 Riesz Representation (my limited understanding is that this is used for discrete operations, whereas L2 is for continuous and H1 is the Sobolev space for solving the weak form of an integral) to get the integral map in the form of a Function so that we can manipulate this function to apply control via Dirichlet boundary conditions as we had been before this update.
resolves #102, resolves #104

Will review later today!

Will review later today!

great, thanks!


Add the option to set as default browser on XP and below

On Windows XP and below, you can't set Supermium as your default browser because of the message, "Supermium cannot determine or set the default browser." Could you please remedy this issue??

I was able to set Supermium as the default browser from the Control Panel in XP. Under Add/Remove Programs, on the left side is Set Program Access and Defaults. I set Supermium as the default browser and htm/html files open in Supermium by default now.

Supermium doesn't show up in the control panel. I used the installer.


Update sbt-idea-plugin to 3.21.1

About this PR
📦 Updates org.jetbrains:sbt-idea-plugin from 3.18.3 to 3.21.1
📜 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.jetbrains", artifactId = "sbt-idea-plugin" } ]

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

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

Superseded by #685.


Update actions

Ref:https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/
This updates the actions to use Node16.
[skip actions]

:shipit:


Fix sequence diagram of update workload

Introduced with #1 the Agent Manager does only receive the ExecutionCommands of the server, but forwards the actions to RuntimeManager. Thus the requirement and the sequence diagram needs an update.
I have made the message "update workload" to the Runtime Manager from the Agent Manager async to decouple at this time point, even if it is sequentially because of the await. The real decoupling happens later in the Runtime Facade but this is not part of this diagram. But at least with the async arrow it behaves correctly that the Agent Manger is not blocked.

Definition of Done
The PR shall be merged only if all items mentioned in CONTRIBUTING.md have been followed. In case an item is not applicable as described, please provide a short explanation in the description.

[x] All steps in CONTRIBUTING.md have been handled

Looks good, just a small suggestion.

I have updated with your suggestion. Thank you.