Cert Rotation - kube-apiserver certs did not rotate

What kind of request is this (question/bug/enhancement/feature request): bug
Steps to reproduce (least amount of steps as possible):

Have a DO rke cluster on rancher:v2.1.8.
run the following command
./system-tools cert rotate --url https://x.x.x.x/v3 --token "token-xxxxxxxx" --cluster c-xxxx (system tools from : https://github.com/rancher/system-tools/releases/tag/v0.1.1-rc7)

Expected Result:
certs should be rotated and 10 years should have been added to the expiry date, in all the nodes.
Actual Result:

after running the cert rotate command, on running a cert info command with the system tools, gave the below output. kube-apiserver certs' did not rotate and certs' expiry did not get the 10 years.

INFO[0000] Check certificates Info for cluster [c-g8tvb]
--
INFO[0000]   Setup rkeconfig for cluster [c-g8tvb]
INFO[0001]   Get kubeconfig for cluster [c-g8tvb]
INFO[0001]   Write temporary kubeconfig for cluster [c-g8tvb]
INFO[0001]   Successfully wrote temporary kubeconfig file for cluster [c-g8tvb] at   [./kube_config_c-g8tvb]
INFO[0001]   Fetching cluster [c-g8tvb] full state from kubernetes
INFO[0002]   possible legacy cluster, trying to fetch certs from kubernetes
INFO[0002]   [certificates] Getting Cluster certificates from Kubernetes
INFO[0003]   [certificates] Successfully fetched Cluster certificates from   Kubernetes
INFO[0003]   Certificate [kube-apiserver] has expiration date: [2020-04-10 21:24:01 +0000   UTC]
INFO[0003]   Certificate [kube-controller-manager] has expiration date: [2029-04-08   21:45:16 +0000 UTC]
INFO[0003]   Certificate [kube-scheduler] has expiration date: [2029-04-08 21:45:17 +0000   UTC]
INFO[0003]   Certificate [kube-proxy] has expiration date: [2029-04-08 21:45:17 +0000   UTC]
INFO[0003]   Certificate [kube-node] has expiration date: [2029-04-08 21:45:17 +0000   UTC]
INFO[0003]   Certificate [kube-admin] has expiration date: [2029-04-08 21:45:17 +0000   UTC]
INFO[0003]   Certificate [kube-apiserver-requestheader-ca] has expiration date:   [2029-04-08 21:24:03 +0000 UTC]
INFO[0003]   Certificate [kube-apiserver-proxy-client] has expiration date: [2029-04-08   21:45:17 +0000 UTC]
INFO[0003]   Certificate [kube-etcd-134-209-170-141] has expiration date: [2029-04-08   21:45:17 +0000 UTC]
INFO[0003]   clean temporary kubeconfig for cluster [c-g8tvb]

kube-apiserver.pem on the control node, did not get new certs, certs' expiry date did not get the 10 years.
kube-node.pem and  kube-proxy.pem on the worker node, did not get new certs, certs' expiry date did not get the 10 years. - issue tracked in https://github.com/rancher/rancher/issues/19620

Other details that may be helpful:
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): rancher:v2.1.8
Installation option (single install/HA): single install

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported): DO cluster - hosted

I also have the same error
2019/04/13 07:43:15 [ERROR] API error response 422 for PUT /v3/clusters/c-25d48?_replace=true. Cause: json: cannot unmarshal string into Go struct field RotateCertificates.services of type []string
when trying to save cluster
Validation failed in API: Cluster spec conversion error

Instead of system-tools, cert rotation was backported to rancher rke/rancher. The feature can be tested with rancher v2.1.9-rc2. Closing this issue.


Drag sensitivity and problems with inViewThreshold

Bug is related to

[X] embla-carousel (core package)
[ ] embla-carousel-react
[ ] embla-carousel-vue
[ ] embla-carousel-svelte
[ ] embla-carousel-autoplay
[ ] embla-carousel-auto-height
[ ] embla-carousel-class-names
[ ] embla-carousel-docs (documentation website)

Embla Carousel version

v7.0.5

Describe the bug

Thanks for the nice plugin, its really good! It seems like the inViewThreshold not working properly or maybe we have misunderstood the underlying concept behind it. All we need is: when user tries to drag next or previous slides, he has to drag the upcoming slide around 50% and then only the slide comes under the "view". Is it possible to relax this limitation using an option, such as, so a user drags only 10% of the upcoming slide and when the mouse is release the new slide comes under the "view"? Thanks

CodeSandbox

https://codesandbox.io/s/embla-carousel-loop-vanilla-forked-fjkosg?file=/src/js/index.js

Steps to reproduce

So in the codesandbox, when you drag the next slide for the first time, you need to drag around 50% and then only it comes under the view.

Expected behavior

If a user just drags 10% of previous or next slide, the new slide must come under the view immediately rather than vigorous sliding.

Hi @ravi-magecroco,
Thank you for your question. This is not a bug but rather a question about how to achieve something so please prefer creating a Q&A discussion instead of creating a bug report:

Click here to create the discussion

You may copy your question and link to CodeSandbox to the discussion.
Best,
David

Hi @davidjerleke - thanks for responding back quickly. I have created a discussions item over here: https://github.com/davidjerleke/embla-carousel/discussions/419 can you please help me out? Thanks a lot again.


[WIP] Readme 2.0

An attempt to address #30. The "Subproject" section still needs to be reworked, especially for iOS, but I wanted to get feedback on the direction I'm going, especially on the "Features" section. I started it as a quick list of functionality, but wanted to incorporate links so passers-by can quickly get to the relevant API  or examples. Now I'm debating how much value that provides over having broken links as things change.

I like it! :+1:

I wanted to get feedback on the direction I'm going, especially on the "Features" section. I started it as a quick list of functionality, but wanted to incorporate links so passers-by can quickly get to the relevant API or examples. Now I'm debating how much value that provides over having broken links as things change.

I worry about that too, and whether we'll remember to keep it up to date :disappointed:

This is ready for review now.

Fantastic :metal:


String interpolation API is repetitive, unintuitive, and has bugs

If we take a String like
"foo${bar}baz"

Then the API here is hard to use.
The overall string is a StringInterpolation but the first segment "foo and last segment baz" are both InterpolationStrings. For a user, the difference between a StringInterpolation vs InterpolationString is not obvious. Could maybe be StringInterpolationLiteral and InterpolationStringSegment?
These InterpolationStrings have both a value and contents. Values will be foo and baz, and contents will be a token "foo and baz". The difference between value and contents is not obvious. It should probably be value and token? Note that value includes resolving escapes.
The InterpolationStrings also have contentsOffset and contentsEnd. These do not match contents.offset and contents.end, because the contents token includes the opening and end quotes, but contentsOffset and contentsEnd do not. Perhaps these should be called valueOffset and valueEnd? Note that since value includes resolving escapes, this is also a little dubious because valueOffset + value.length will not be equal to valueEnd when an escape sequence is involved.
A CL I'm working on now fixes a bug where contentsOffset and contentsEnd were wrong because the code assumed every InterpolationString was first and last (ie, "foo", however, that is a StringLiteral and not a StringInterpolation). Therefore quotes would be stripped from the beginning and the end in every InterpolationString's contentsOffset/contentsEnd.
However, even after my CL there is still a bug: each InterpolationString has no knowledge of what type of string it is in. Therefore an unterminated string literal that ends in a quote will have the wrong contentsEnd, ie, "foo$bar''' will get an InterpolationString at the end with no length (in terms of contentsOffset/contentsEnd) even though it should be of length 3. Note that the value in this case is both correct and incorrect as well, it correctly includes ''' but it incorrectly includes the synthetic closing " even though quotes are supposed to not be a part of value.

While I agree that the API is poor in this area, I have to question whether the benefit of changing it outweighs the cost. I kind of hope it does, but I have to raise the question.

For a user, the difference between a StringInterpolation vs InterpolationString is not obvious.

If we're going to change the name StringInterpolation then I'd like to propose InterpolatedStringLiteral. It's more consistent with SimpleStringLiteral.
I'd suggest leaving InterpolationString unchanged because it's consistent with InterpolationExpression, and I think that changing one of the names should be enough to reduce the confusion. If we must change it, then we should change both kinds of elements to keep them consistent.

The difference between value and contents is not obvious. It should probably be value and token?

Yes, contents should probably either be token or literal (the latter for consistency with SimpleStringLiteral), but  think the docs also need to be updated to help explain the difference.

The InterpolationStrings also have contentsOffset and contentsEnd.

Note that these reflect the contentsOffset and contentsEnd defined by SingleStringLiteral. Please don't change these names.

These do not match contents.offset and contents.end, because the contents token includes the opening and end quotes, but contentsOffset and contentsEnd do not.

I suspect that the only reason you thought this odd is because contents should have been called something different. If it were renamed, then you probably wouldn't have associated them.

Perhaps these should be called valueOffset and valueEnd? Note that since value includes resolving escapes, this is also a little dubious because valueOffset + value.length will not be equal to valueEnd when an escape sequence is involved.

The AST nodes representing string literals consistently refer to the actual characters in the source code that are inside the quotes (however many quotes there might be) as the "contents" of the string, and the result of removing leading white-space, translating escape characters, etc. as the "value". Calling these valueOffset and valueEnd would be very confusing because they're related to the contents of the string, not the value of the string!

However, even after my CL there is still a bug: each InterpolationString has no knowledge of what type of string it is in.

I don't know what that means. Every InterpolationString is a child of a StringInterpolation. The parent knows whether it's a raw string, whether it uses single or double quotes, and whether there it's single- or multi-line. I think we should have all the information we need in order to fix this bug.

cc @matanlurey who had to navigate this API in deciding if a String is potentially constant for a security API.


implement Coveralls and appveyor

Automate testing

Coveralls and CI works now on my fork. I used Travis instead of appveyor. We can add appveyor later if we want to specifically test on a windows env but I think that is unnecessary for now.
For this to work on the LCfP repo the organization needs to give access to coveralls and travis-ci. Not something I can do. Maybe @ApiumNL can initiate this?

On this!

I think I gave access, but we'll see soon enough with new PRs.


idea task not compatible with Configuration Caching

idea task to create project files for IntelliJ IDEA fails with configuration caching on in Gradle
Expected Behavior
Task should be successful
Current Behavior
27 problems were found storing the configuration cache, 8 of which seem unique.
- Task `:ideaModule` of type `org.gradle.plugins.ide.idea.GenerateIdeaModule`: cannot serialize object of type 'org.gradle.api.internal.project.DefaultProject', a subtype of 'org.gradle.api.Project', as these are not supported with the configuration cache.

Context
We still use old style IntelliJ project files and hence need to run the idea task. A large number of projects are impacted by it.
Steps to Reproduce
Take any project using with idea plugin on Gradle 7, enable configuration cache and run idea task

Thank you for your interest in Gradle!
This issue will be closed as a duplicate of

#13480


Implement search

This should be done after #7. We can start with something simple using Django QuerySets:
https://docs.djangoproject.com/en/1.8/ref/models/querysets/#icontains

If we want to implement fuzzy search, it's supported by PostgreSQL, but integration with Django will be non-trivial:
https://michalcodes4life.wordpress.com/2014/06/03/full-text-search-and-fuzzy-search-with-postgresql-and-django/
See also this rationale behind postgresql being "good enough." In our case, we want to keep things as simple as possible so that it's not hard to administer this application:
http://rachbelaid.com/postgres-full-text-search-is-good-enough/


Query Logging in <IP_ADDRESS>

#3566 from what I have read, turn off log-enabled = false within the [http] section should disable the query logging.
What I don't understand is why I am still seeing that in our logs even though we have this disabled.
What I see is something a little new...maybe, but I have not been able to find out what can turn these off...
[query] 2015/09/16 20:55:06 SELECT last(value) .... {cut some due to private details}.
Did I miss some setting that should be disabled?

No @jgerschk -- you did not miss anything. You are only disabling logging at the HTTP level, but logging in the query engine was recently added. This is what is being added.
I can make that configurable too.

@otoolep Thanks for the reply.  So we should see this in a future release -- to be able to disable this as I understand it....Logging those can create a huge amount of unneeded data at times...

Yes, it will be coming very soon.

Fixed by https://github.com/influxdb/influxdb/pull/4141

Thank you kindly @otoolep .

Fixed in next nightly build, and in 0.9.5.

@otoolep, Thanks again, will give the nightly a spin.


libxslt: tests should handle changed XSLT template resolution

What problem is this PR intended to solve?
Upstream libxslt started failing after https://gitlab.gnome.org/GNOME/libxslt/-/commit/b0074eeca3c6b21b4da14fdf712b853900c51635 which addresses https://gitlab.gnome.org/GNOME/libxslt/-/issues/55.
Update the test to order the templates appropriately to reflect this change to how templates are resolved.
Have you included adequate test coverage?
This change affects only tests, not functional behavior.
Does this change affect the behavior of either the C or the Java implementations?
This change to the tests is forward-looking, as the next version of libxslt is likely to include the related change to the xslt template resolver.

See https://github.com/sparklemotion/nokogiri/runs/3006765122?check_suite_focus=true for the failing test suite that drove this change.


Upgrade master json-jwt dependency to fix CVE-2018-1000539

Fixes CVE-2018-1000539

Hmm, I don't think it's even used.

Oh, it's a dependency of the LE certificate client dependency Acme::Client, better upgrade that.
Comes from #830

Needs spec fixes

acme-client 0.5.0 has breaking changes for our implementation:

Drop Acme::Client#challenge_from_hash and Acme::Client::Resources::Challenges::Base#to_h in favor of the new API.

I wonder how that vulnerability affects the master

I wonder how that vulnerability affects the master

CVE-2018-1000539 looks like it affects JWT token validation: https://github.com/nov/json-jwt/pull/62
I would assume that acme-client only uses json-jwt to to build authentication tags, not validate them... but I haven't checked.

acme-client 0.3.4 only uses JWT to sign tokens: https://github.com/unixcharles/acme-client/blob/1b231de016a8df1ba2f4745f5bee612f73e7b2e4/lib/acme/client/crypto.rb
$ rgrep -i jwt vendor/bundle/ruby/2.3.0/gems/acme-client-0.3.4
vendor/bundle/ruby/2.3.0/gems/acme-client-0.3.4/acme-client.gemspec:  spec.add_runtime_dependency 'json-jwt', '~> 1.2', '>= 1.2.3'
vendor/bundle/ruby/2.3.0/gems/acme-client-0.3.4/lib/acme-client.rb:require 'json/jwt'
vendor/bundle/ruby/2.3.0/gems/acme-client-0.3.4/lib/acme/client/crypto.rb:    jwt = JSON::JWT.new(payload || {})
vendor/bundle/ruby/2.3.0/gems/acme-client-0.3.4/lib/acme/client/crypto.rb:    jwt.header.merge!(header || {})
vendor/bundle/ruby/2.3.0/gems/acme-client-0.3.4/lib/acme/client/crypto.rb:    jwt.header[:jwk] = jwk
vendor/bundle/ruby/2.3.0/gems/acme-client-0.3.4/lib/acme/client/crypto.rb:    jwt.signature = jwt.sign(private_key, :RS256).signature
vendor/bundle/ruby/2.3.0/gems/acme-client-0.3.4/lib/acme/client/crypto.rb:    jwt.to_json(syntax: :flattened)

The fix for CVE-2018-1000539 only touches JWT::JWE#decrypt!, and I don't think that's on the code path for JWT::JWT#sign or to_json.
So I don't think kontena-server is affected by CVE-2018-1000539.

That is the case. Can be ignored for now in my opinion.
$  bundle show acme-client
/usr/local/Cellar/rbenv/1.1.1/versions/2.5.0/lib/ruby/gems/2.5.0/gems/acme-client-0.3.4

$ ag -B 1 JWT /usr/local/Cellar/rbenv/1.1.1/versions/2.5.0/lib/ruby/gems/2.5.0/gems/acme-client-0.3.4
/usr/local/Cellar/rbenv/1.1.1/versions/2.5.0/lib/ruby/gems/2.5.0/gems/acme-client-0.3.4/lib/acme/client/crypto.rb

8-  def generate_signed_jws(header:, payload:)
9:    jwt = JSON::JWT.new(payload || {})

$ ag generate_signed /usr/local/Cellar/rbenv/1.1.1/versions/2.5.0/lib/ruby/gems/2.5.0/gems/acme-client-0.3.4

/usr/local/Cellar/rbenv/1.1.1/versions/2.5.0/lib/ruby/gems/2.5.0/gems/acme-client-0.3.4/lib/acme/client/faraday_middleware.rb

11:    @env.body = crypto.generate_signed_jws(header: { nonce: pop_nonce }, payload: env.body)

Ignoring though means that we need to disable the vulnerability check in specs or it can't do the  release.

Ignoring though means that we need to disable the vulnerability check in specs or it can't do the release.

That would be bundle audit check --ignore CVE-2018-1000539


[BUG] Soundwaves: the NPS graph can go out of bounds (on the top)

It seems to be more common/pronounced on charts with low NPS.

^ dance

^ pump. Simfile: Memories of You (ATLUS Meguro Remix).ssc.txt
Attached logs but I think they don't show anything.
log.txt

Fixed for the next version.


[Beta] Integration of @mui/packages v5.x

This PR upgrades from material/ui packages to mui (v5.x).
It contains NO breaking changes from the user perspective, apart from required update of peerDependencies in client project.
  "@emotion/react": ">= 11.8.2 < 12",
  "@emotion/styled": ">= 11.8.1 < 12",
  "@mui/icons-material": ">= 5.5.1 < 6",
  "@mui/lab": ">= 5.0.0-alpha.75 < 6",
  "@mui/material": ">= 5.5.3 < 6",
  "@mui/styles": ">= 5.5.3 < 6",
  "react": ">= 17 < 18",
  "react-dom": ">= 17 < 18"

With this release the flow of docs is also split: users will have docs for beta version as well.
The beta version will be distributed on the dedicated channel.
npm i @melfore/mosaic@beta

:tada: This PR is included in version 6.5.0-beta.1 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 7.1.0-beta.1 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 7.2.0-beta.1 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 7.5.0-beta.1 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 7.6.0-beta.1 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 7.7.0-beta.1 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Docathon: pulse channels

Summary
Blocked by #3813 and #3814
Details and comments
I verified changes locally. I think I need to move the Channel class level docstring so that it shows up somewhere in the API Reference

I guess python35 doesn't support attribute type hinting? Will probably switch to inline type hint name = value  # type: <type>


Button2 and Button3 are mixed up in MouseReleased event

package version is 5.3.7
I guess it's necessary, so here is my OS info:

Hi! Thanks for posting this issue! There is indeed a problem with button 2 and 3 for mouse button release on Linux. I will fix this and release a new version in the next couple days.

I've just released version 5.3.8 which fixes this bug, so the issue can be closed.


bugfix needed for handling non-text identifiers when searching for GRB changes

when seraching for GRB changes a geometry_util is used that does not handle non-string IDs corrrectly. this has to be fixed.

Already fixed: https://github.com/OnroerendErfgoed/brdr/blob/70ddcee335ab9b3585209ea7c25d74104c2c7ddc/brdr/geometry_utils.py#L449C1-L451C34

issue closed; already fixed on development


Not overriding fzf's Ctrl+R binding

If fzf's key-bindings.fish is enabled as described in the Arch Wiki, and mcfly-fzf is also enabled, mcfly-fzf is not overriding fzf's Ctrl+R binding. Here are the relevant config files:
~/.config/fish/config.fish:
if status is-interactive
    command -q mcfly && mcfly init fish | source
    command -q mcfly-fzf && mcfly-fzf init fish | source
end

~/.config/fish/functions/fish_user_key_bindings.fish:
function fish_user_key_bindings
    fzf_key_bindings
end

I should mention that it's not desirable to turn off fzf's key-bindings.fish because it also contains other key bindings apart from Ctrl+R, such as Ctrl+T and Alt+C.

I've pushed a test fix for this to the tmux-fix branch. I don't use tmux personally, so I don't have anything set up to test if this works.
Would you be willing to download and run cargo install from that branch to see if it works? Also if you run bash shell/dev.fish it should build a copy and attach it to a new fish shell without adjusting any of your existing settings.
One other followup question: what precisely is the misbehavior that you observe as a result of this? On my own machine, I think the existing keybindings work fine even from within a tmux session


Adding next_higher_level to PageLevelIndex

I don't know if this is really useful, but for me, it is (or my code just
sucks) and I think that having this function doesn't hurt but I can understand
if you think that it's useless.

Thanks for the PR! We already have next_lower_level, so adding the counterpart seems reasonable.


Type info used as item name

Feb 2, 2022 4:29am deleted sniper rifle but name is "Level 1 Sniper Rifle"
Might be a stock item deletion only issue

Probably fixed in version 0.6.8 along with reduced type spam due to storing level information in a lot of types.


Cleaned up Dimension casting and name lookup

The code is currently littered with isinstance(dim, Dimension) checks, which is not optimal because it's duplicative but more importantly because it does not check for the full range of valid dimension specs which include tuples and dictionaries. This PR adds two utilities one to cast Dimension-like objects to Dimensions if it isn't already one and the other that looks up the name on Dimensions or Dimension-like objects. It then uses these utilities throughout the code.

[x] Adds unit test

Ready to review/merge.

So this is a refactor as opposed to a feature or bug fix?

So this is a refactor as opposed to a feature or bug fix?

It also fixes bugs when constructing Datasets with tuple dimensions.

Looks good. I'll merge when the tests are green.


Beginner questions

Hello i am new to empire world and i have many questions but start for this one how to connect an agent if open empire after the victim machine start up? bat file input to start up folder in victim machine when open empire before victim machine start up i have active agent but what hapend open empire after? sorry my english is not very well

Are you better in french?

no comment

someone else that knows?

.... I meen "sorry my english is not very well" and I speek french....

Because I can't understand your question... Words are missing and all... So if you speek french, then write it in french so I can help you.

no i am not speak french ...you speak Greek?

Nah sorry. Can you try and explain you question better?

yes i try ....you have open now empire and the victim open his machine in 60 minutes you have an active agent ok so far? what happent if victim nachine open now and you open empire 60 min after you dont have active agent ....exists some methods to activate the agent?

someone tell me about a autorun script something like that but i dont know where can i found that and how to use? or some other methods

Ok so if I understand correctly, you are asking what happens if the Empire Server is not up when the agent ping back and only up 60 minutes later?
When you create the listener, you can specify "DefaultLostLimit" which is "Number of missed checkins before exiting". You can either put it really High or to 0 for infinite.
To get this information, you write "listeners" in empire's main menu and then "info".
To get more info on how to use Empire, read: http://www.powershellempire.com/

And for your autorun script question, you might want to read: http://www.powershellempire.com/?page_id=139

thank you ....."and only up 60 minutes later?" not only 60 what about for more time? the next day for example?

When you create the listener, you can specify "DefaultLostLimit" which is "Number of missed checkins before exiting". You can either put it really High or to 0 for infinite.

I understand for a new listener ..... for listener have already create can i do something in option now?

I am not sure but I don't think so. Try it and give me feed back?
Or you could create a second listener and move your agent to that listener.
I won't explain how to do it but it is something possible.

ok thank you so much for your help

You can either create a new listener or update the "DefaultLostLimit" value in the backend database under the "listeners" tab by opening "empire.db" with sqlitebrowser.


Update db2_data_source.py

This is needed in order to fix the connectivity issues for db2 from soda-core.
Please add this change as part of the soda-core release

 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.

once the approvals are done and the changes are done, how soon can this be added to the new release?

@m1n0 @vijaykiran Hi Milan as per our discussions earlier can you please review this PR and approve it and merge it for release?

Hi @4rahulae, thanks for your contribution 🙌   @m1n0 is currently traveling back from our company offsite, so he'll likely not reach out today.
I'm myself not able to test this at the moment, but could you assist by verifying that your changes do not impact configurations where security is not set?  According to the documentation I can see that it only accepts SSL as a valid value, so I would like to make sure this does not introduce backwards compatability issues for anyone who does not define security in their configuration.yml.
(For example a possible scenario could be that Python translates it into an empty string or None, triggering DB2 to fail)

Hi @4rahulae, thanks for your contribution 🙌 @m1n0 is currently traveling back from our company offsite, so he'll likely not reach out today.
I'm myself not able to test this at the moment, but could you assist by verifying that your changes do not impact configurations where security is not set? According to the documentation I can see that it only accepts SSL as a valid value, so I would like to make sure this does not introduce backwards compatability issues for anyone who does not define security in their configuration.yml.
(For example a possible scenario could be that Python translates it into an empty string or None, triggering DB2 to fail)

thanks for the contrib @4rahulae ! I would suggest a bit more conservative way to do this though - pythons get without a second param returns NULL, so the VERIFY_SSL= will contain nothing, I would suggest only appending it to the connection string if it does have a non-null value, would you agree?
Also, I think it would make sense to match the yaml key we read with the connection string argument, so read verify_ssl instead of security if that makes sense.

hi, I don't know what the default value for SECURITY is, so maybe I would suggest a more safe/generic way to do this:
self.security = data_source_properties.get("security")

conn_str =  f"DATABASE={self.database};HOSTNAME={self.host};PORT={self.port};UID={self.username};PWD={self.password}"

if self.security is not None:
  conn_str += f";SECURITY={self.security}"

what do you think?

Hi @m1n0 added changes as per your suggestion and tested them successfully on my local, please review the changes and let me know if it looks good to be merged?

Thanks for the contrib, merging!


Role Not Include access_token in new version in net5.0

Hi, I have several IDS running but I update one of them to the latest version in net5.0 and role is not include access_token any more
installing nuget packages:
   <PackageReference Include="Microsoft.AspNetCore.Identity.UI" Version="5.0.1" />
    <PackageReference Include="IdentityServer4.AspNetIdentity" Version="4.1.1" />
    <PackageReference Include="IdentityServer4.EntityFramework" Version="4.1.1" />
    <PackageReference Include="IdentityServer4.EntityFramework" Version="4.1.1" />

Client config:
response_type: 'id_token token',
scope: 'openid roles profile api_gateway',
redirect_uri: `${process.env.VUE_APP_ADMIN_ADDRESS}/callback.html`,
silent_redirect_uri: `${process.env.VUE_APP_ADMIN_ADRESS}/silent-renew.html`,

I have roles in Id_token and api does not authorize Id_token any more. previously I could use this when I had problem.
here are tokens:
Access Token
{
  "alg": "RS256",
  "kid": "C49BF6CF4B87872DB24E288C39F06D1E9F3440E5RS256",
  "typ": "at+jwt",
  "x5t": "xJv2z0uHhy2yTiiMOfBtHp80QOU"
}.{
  "nbf":<PHONE_NUMBER>,
  "exp":<PHONE_NUMBER>,
  "iss": "https://localhost:44310",
  "aud": "api_gateway",
  "client_id": "xxxx",
  "sub": "31914615-a193-45c0-bd60-c829ca11ff7a",
  "auth_time":<PHONE_NUMBER>,
  "idp": "local",
  "sid": "CCB90B0BE4326EC44D02FC772947FB39",
  "iat":<PHONE_NUMBER>,
  "scope": [
    "openid",
    "roles",
    "profile",
    "api_gateway"
  ],
  "amr": [
    "pwd"
  ]
}.[Signature]

and here is id_token
{
  "alg": "RS256",
  "kid": "C49BF6CF4B87872DB24E288C39F06D1E9F3440E5RS256",
  "typ": "JWT",
  "x5t": "xJv2z0uHhy2yTiiMOfBtHp80QOU"
}.{
  "nbf":<PHONE_NUMBER>,
  "exp":<PHONE_NUMBER>,
  "iss": "https://localhost:44310",
  "aud": "xxxxx",
  "nonce": "cb1ba8dd02af4caa92c113750616e2f2",
  "iat":<PHONE_NUMBER>,
  "at_hash": "_uKrgwya12q1jl0Bne9YQw",
  "s_hash": "UCrd9pwu43r5IO9BDcP8eg",
  "sid": "CCB90B0BE4326EC44D02FC772947FB39",
  "sub": "31914615-a193-45c0-bd60-c829ca11ff7a",
  "auth_time":<PHONE_NUMBER>,
  "idp": "local",
  "name": "my name",
  "role": [
    "admin",
    "provider"
  ],
  "preferred_username": "name",
  "amr": [
    "pwd"
  ]
}.[Signature]

Client allowed scopes:
email openid roles profile api_gateway

I have roles in identity resources and role as a clame of roles
this is the config that everytime worked for me but not this time.
I tried implicit and also authorization_code flows but same result.
Best Regards,
Farshid.

I tried email claimes as well. I didn't get it in access token too.

Not enough info. We'd need to see the scope definitions (and the associated user claims), and if you have a profile service configured.


Normative: Throw RangeError for ms/mics/ns if display is always

IF the previous style is 'numeric' and
the current style is 'numeric' and
the unit is 'milliseconds', 'microseconds' or 'nanoseconds' then that unit will not be output by the format/formatToParts but be fraction of the previous unit. Therefore we should throw RangeError if the display of that unit is 'always'
Close #142
@ryzokuken  @ben-allen @sffc

@FrankYFTang we needed to present this to TG1 first to get final consensus.


AP-2370 Include results by proceeding type on Means report

Include results by proceeding type on Means report
Link to story
This PR is branched off AP-2159, so best to wait until that is merged and this branch rebased before reviewing.
Describe what you did and why.
Checklist
Before you ask people to review this PR:

[ ] Tests and rubocop should be passing: bundle exec rake
[ ] Github should not be reporting conflicts; you should have recently run git rebase master.
[ ] There should be no unnecessary whitespace changes. These make diffs harder to read and conflicts more likely.
[ ] The PR description should say what you changed and why, with a link to the JIRA story.
[ ] You should have looked at the diff against master and ensured that nothing unexpected is included in your changes.
[ ] You should have checked that the commit messages say why the change was made.

Codecov Report

Merging #2647 (d09dab2) into master (1600db1) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master     #2647   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          530       530
  Lines        10888     10907   +19
=========================================
+ Hits         10888     10907   +19

Impacted Files
Coverage Δ

...providers/capital_assessment_results_controller.rb
100.00% <100.00%> (ø)

...rs/capital_income_assessment_results_controller.rb
100.00% <100.00%> (ø)

app/models/cfe/v4/result.rb
100.00% <100.00%> (ø)

app/services/results_panel_selector.rb
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 1600db1...d09dab2. Read the comment docs.


pipenv requirements does not include markers

I just tried to use pipenv reqirements instead of pipenv lock --requirements but it lacks markers so windows only dependencies from my lock file makes pip install -r ... fail.
example transitive dependency form lock file that at some point was locked on a windows machine:
        "twisted-iocpsupport": {
            "hashes": [
                "sha256:306becd6e22ab6e8e4f36b6bdafd9c92e867c98a5ce517b27fdd27760ee7ae41",
                "sha256:3c61742cb0bc6c1ac117a7e5f422c129832f0c295af49e01d8a6066df8cfc04d",
                "sha256:72068b206ee809c9c596b57b5287259ea41ddb4774d86725b19f35bf56aa32a9",
                "sha256:7d972cfa8439bdcb35a7be78b7ef86d73b34b808c74be56dfa785c8a93b851bf",
                "sha256:81b3abe3527b367da0220482820cb12a16c661672b7bcfcde328902890d63323",
                "sha256:851b3735ca7e8102e661872390e3bce88f8901bece95c25a0c8bb9ecb8a23d32",
                "sha256:985c06a33f5c0dae92c71a036d1ea63872ee86a21dd9b01e1f287486f15524b4",
                "sha256:9dbb8823b49f06d4de52721b47de4d3b3026064ef4788ce62b1a21c57c3fff6f",
                "sha256:b435857b9efcbfc12f8c326ef0383f26416272260455bbca2cd8d8eca470c546",
                "sha256:b76b4eed9b27fd63ddb0877efdd2d15835fdcb6baa745cb85b66e5d016ac2878",
                "sha256:b9fed67cf0f951573f06d560ac2f10f2a4bbdc6697770113a2fc396ea2cb2565",
                "sha256:bf4133139d77fc706d8f572e6b7d82871d82ec7ef25d685c2351bdacfb701415"
            ],
            "markers": "platform_system == 'Windows'",
            "version": "==1.0.2"
        },

Running pipenv lock --requirements --keep-outdated  (on linux) results in:
twisted-iocpsupport==1.0.2; platform_system == 'Windows'

Running pipenv requirements (on linux) has no marker so the requirements file fails to install on linux.
twisted-iocpsupport==1.0.2

pipenv, version 2022.4.30

We are also having this issue, and have reverted back to the lock mechanism.

I looks like micropipenv requirements provides expected output.

@matteius shall I create a PR for this over the weekend?

@ImreC That would be awesome 😎

Ok, forgot to do this. Will pick it up end of this week.

PR open https://github.com/pypa/pipenv/pull/5114


Extension running on eBay.de blocks scrolling and forces page to one viewport

Hey,
I'm running into an issue that whenever extension is enabled, the eBay.de gets forced into one viewport and it's impossible to scroll through the page anymore.
It happens even after the extension is not actively running on the website.
Disabling extension in Firefox solves the issue.
macOS Mojave 10.14.6 (18G84), Firefox Quantum 68.0.2 (64-bit) running on MacBook Pro (15-inch, 2018)

Are you using Microsoft or Google Translator?

Google Translate

I tried, but I cannot reproduce this issue on Firefox 68.0.2 (64-bit), macOS 10.14.6, MacBook Pro (13-inch, 2016, Thunderbolt).
VIdeo: https://vimeo.com/356497153

I've made a quick screencap of it not working. At first, you can see the behaviour with extension disabled - scrolling is normal. After I enable it back, it blocks the viewport further.
Quick link to the search I used: here


feat: upgrade to Angular 17

I got this error after I use the ng17 branch in my package.json:
An unhandled exception occurred: Cannot find module 'single-spa-angular/libs/single-spa-angular/webpack'
Require stack:

/Users/alexawu/Documents/upgrade-projects/hk-aws-agt-rpt-admin-sws/extra-webpack.config.js
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>See "/private/var/folders/hz/c6mk40394n57p3y5090264w83xhtk8/T/ng-KCoLJA/angular-errors.log" for further details.

And here is my package.json:
"dependencies": {
"@angular-builders/custom-webpack": "^17.0.0",
"@angular/animations": "^17.0.8",
"@angular/cdk": "^17.0.4",
"@angular/common": "^17.0.8",
"@angular/compiler": "^17.0.8",
"@angular/core": "^17.0.8",
"@angular/forms": "^17.0.8",
"@angular/platform-browser": "^17.0.8",
"@angular/platform-browser-dynamic": "^17.0.8",
"@angular/router": "^17.0.8",
"@manulife-design/md": "^1.0.5",
"file-saver": "^2.0.5",
"numeral": "^2.0.6",
"powerbi-client-angular": "^3.0.5",
"rxjs": "~7.8.1",
"single-spa": "^5.9.5",
"single-spa-angular": "github:single-spa/single-spa-angular#ng17",
"tslib": "^2.1.0",
"xlsx": "^0.18.5",
"zone.js": "^0.14.2"
},
"devDependencies": {
"@angular-devkit/build-angular": "^17.0.8",
"@angular/cli": "^17.0.8",
"@angular/compiler-cli": "^17.0.8",
"@angular/language-service": "^17.0.8",
"@types/file-saver": "^2.0.4",
"@types/jasmine": "^4.3.5",
"@types/jasminewd2": "~2.0.10",
"@types/node": "~16.11.7",
"codelyzer": "^6.0.2",
"jasmine-core": "~4.6.0",
"jasmine-spec-reporter": "~7.0.0",
"karma": "~6.4.2",
"karma-chrome-launcher": "~3.2.0",
"karma-coverage-istanbul-reporter": "~3.0.3",
"karma-jasmine": "^5.1.0",
"karma-jasmine-html-reporter": "^2.0.0",
"protractor": "~7.0.0",
"sonar-scanner": "^3.1.0",
"systemjs-webpack-interop": "^2.3.7",
"ts-node": "~10.9.1",
"tslint": "~6.1.3",
"typescript": "^5.2.2",
"webpack": "^5.88.2"
}

Why would you use the ng17 branch?

@arturovt Because our project uses Angular17, but single-spa-angular does not yet support the angular17 version, but I saw the commit you mentioned, so I want to try it with your ng17 branch.

The single-spa-angular is compatible with angular 17.

OK, thanks

@arturovt - Is there anything I can do to help bump this merge along? I'm sure you're super busy, but this has been open for a few months now and I figured it wouldn't hurt to merge it in before angular 18 rolls around 😄

I see that all changes about ng17 are inside the main branch. Is there a plan to relase npm version?


Gitter channel

If you are looking for additional collaborators in this project, may I suggest to open a gitter channel? that will improve communication a lot...

Sounds like a good idea -- I'll give it a try.


Improve stringify test coverage

This PR adds missing structs and slices to the stringify tests for better test coverage.

@Parker77 - do you have time for a code review?

Thank you, @Parker77 !
Merging.


feat: port NumberTheory.SumFourSquares

IMHO, the proof was not very readable, so I've rewritten large parts of it. Also, I moved some results that look useful to lemmas.

bors merge


Menu slug error when Using Woocommerce

Expected behavior
$context['menu'] = new TimberMenu('main');
{%  for el in menu.get_items %}
    {{el.slug}} // This doesn't work / breaks the site
    {{el.post_name}} // This returns IDs
    {{el.title}} // This works
 {% endfor %}

Actual behavior
Get a bunch of errors in different files. (Tmber and Woocommerce)
Timber // /plugins/timber-library/lib/Twig.php(43): call_user_func_array('do
What version of WordPress, PHP and Timber are you using?
WP: 4.8.3
Timber: 1.5.2
Woocommerce: 3.2.3

Closing since this is quite an old issue. Let us know if this issue still persists by opening in a new issue.


Opening and deleteing nodes broken

Nodes appear to be deleted, but they're not being removed from the nodes object. So on open they reappear. Opening is also not repositioning the nodes to their saved positions: the positions are saved correctly in the .dominodes file, but on load they become "NaNvw". They then render on top of on another in the top left corner:

Automated testing definitely would have caught this issue; as if we didn't have incentive enough to set it up ASAP...


[incubator-kie-drools-5957] executable model fails to handle property…

… reactive with null-safe-dereferencing
Issue:

https://github.com/apache/incubator-kie-drools/issues/5957

@mariofusco @lucamolteni Please review, thanks!


Rounding property in styles does not affect windows

When setting the FrameRounding and GrabRounding  style properties, only the docking icons that pop up seem to be affected.
local ImGui = require("lib.cimgui")
local style = ImGui.GetStyle();
style.FrameRounding = 20;
style.GrabRounding = 2;

Is the problem that you are also expecting the windows to be rounded?

The window rounding is controlled through a different property (WindowRounding). I would recommend activating the style editor (through ShowStyleEditor) for playing around with this kind of things.


Support multiple devices

Replace simple function "libusb_open_device_with_vid_pid" with full USB bus scan to allow for processing multiple hyb-usb units on the bus.
fix a build warning and unify indentation style with
indent -nbad -bap -nbc -bbo -hnl -br -cdw -brs -c33 -cd33 -ncdb -ce -ci4 -cli0 -d0 -di1 -nfc1 -i8 -ip0 -l80 -lp -pcs -prs -npsl -sai -saf -saw -ncs -nsc -sob -nfca -cp33 -ss -ts8 -il1 -brf main.c
and a manual sed for space before semi-columns

Thank you !
I knew I had this to do, but time is flying !


feat(cli): Force arguments for the rebuild command

fixes #3445
Motivation
Allowing to accept no arguments to rebuild all the integrations is a bit risky so a new flag for this purpose must be explicitly provided by the end user.
Modifications:

Add the flag --all to the rebuild command to rebuild all integrations
Sanitize the name of the integration to align the code with the delete command

Release Note
The rebuild command no more supports empty arguments, either the flag `--all` must be set to rebuild all integrations,
or the list of integration names to rebuild needs to be provided

@tadayosi @squakez Thx for your feedback, actually the remarks that you both raised are about the code that I reused from the delete command for the sake of uniformity since the need is about the same. It is up to you but if I change this, maybe I should also align the code of the delete command, don't you agree?

We can take the opportunity and do the necessary cleaning on the other commands as well.

Ok, No problem, I will do that next week then thx


Improved bevymark: no bouncing offscreen and spawn waves from CLI

Objective

In bevymark, bevies were very bouncy and could go off screen after a while, skewing the FPS

https://user-images.githubusercontent.com/8672791/146540848-282fa11b-d058-4da9-8e95-688ae67d9406.mp4
Solution

Make bevies bounce also on top of the screen
I also changed how the examples read args from CLI to be able to spawn waves: cargo run --example bevymark --release -- 5 100 (first is number of bevy per wave, second is number of wave, with one wave every 0.2 seconds). This makes it easier to reproduce some exact situations

https://user-images.githubusercontent.com/8672791/146542857-2f953fa0-7b8d-4772-93f8-b8d7a31259dc.mp4

bors r+


Update README.md

修正区间范围

Codecov Report

Merging #79 (595d4cd) into master (8401264) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #79   +/-   ##
=======================================
  Coverage   71.51%   71.51%
=======================================
  Files         528      528
  Lines        9487     9487
=======================================
  Hits         6785     6785
  Misses       2442     2442
  Partials      260      260

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 8401264...595d4cd. Read the comment docs.


CI/CD tries to deploy to a non-existing not setup environment

Hi,
I have a simple .github/AL-Go-Settings.json file with 3 environments defined.
{
  "type": "PTE",
  "templateUrl": "https://github.com/microsoft/AL-Go-PTE@preview",
  "templateSha": "3cc6fcecbe3916c40e0e8bee473b9a51feaa9dfb",
  "NextMinorSchedule": "0 22 * * 0",
  "NextMajorSchedule": "0 23 * * 0",
  "environments": [
    "TEST",
    "DEV",
    "UAT"
  ],
  "DeployToTEST": {
    "EnvironmentName": "TEST",
    "continuousDeployment": true
  },
  "DeployToDEV": {
    "EnvironmentName": "DEV",
    "continuousDeployment": true
  },
  "DeployToUAT": {
    "EnvironmentName": "UAT",
    "continuousDeployment": true
  }
}

When the CI/CD pipeline runs, the deploy matrix is trying to deploy to 4 environments:

In my repository setting are no environments, but only 2 secrets:

AUTHCONTEXT
GHTOKENWORKFLOW

Under GitHub Settings -> Environments - is there an environment called TEST*?
Then delete that.

Indeed, a TEST* environment was created, but I’m uncertain about who or how it was created.
Could it be that the Publish To Environment workflow generates an environment if it doesn’t already exist?


Various tweaks

Marked as draft until we have the SVG logo and redo the images.
Preview: https://trusting-pare-a516e2.netlify.com/

@XhmikosR Sorry for the delay, I was out of town/AFK for the Holiday weekend. I'll bring in the SVG and review the PRs tonight. 🙂

No worries! After we have the SVG I'll redo this PR.

SVG: https://github.com/taniarascia/new-moon/blob/master/images/new-moon.svg

@taniarascia: I think this is ready. I didn't go with 8-bit for the new images because this can cause issues with some image editing programs, so I'd rather do it in a separate commit later (so that still have access to the original files via git log). That being said, we'd save another ~1.5-2.0 MB.
A few notes:

You should use the SVG logo (and remove the old 2 PNGs) when possible like in README.md etc. In this PR I use it for the docs.
The subpackages' package.json files need some love, their info seems wrong in some cases like repo URL etc. Also, if they are not supposed to be published they should be marked as private to be safe
Finally, the whole PNG optimization step you should do it when committing the new files so that we don't bloat the repo :) I'd go with lossless optimization which is perfectly safe, but you can experiment with lossy too. For the docs images, we already use 8-bit PNGs BTW. I assume you are on macOS, so you can use https://kraken.io/web-interface or https://imageoptim.com/mac if you prefer a GUI.

Sorry, not sure why notifications are intermittent for me so I took a while to get back to this. Looks great. I still need to go through and fix the subpackages package.json/README files and bring them all in.


tracking errors

i'm trying to track AdsLoader errors, e.g. when the plugin warns AdsLoader Error 1009, i want to be able to listen to that event and capture/log the error and error data.  i'm not sure how to access the AdsLoader directly through the plugin, and therefore how to attach onerror events to it.  thoughts?

Hi @khatibda,
We just added some events to access things like the AdsLoader as soon as they are ready. Take a look at the Plugin Ad Events section of the readme for details on the implementation.
This is new in v1.7.0 of the videojs-ima plugin, so make sure you are building with that version for these events. Please follow up if that works for your implementation.
Hope that helps,
Jackson
IMA SDK DevRel

great thanks!!!!

This is helpful, but can you fix the broken links to the IMA documentation here: https://github.com/googleads/videojs-ima#ima-plugin-ad-events


investigate ViT and interpretability

set up and run a simple ViT model
ViT and interpretability? How was interpretability measured in the FunnyBirds article for ViT models?

ViT architectures:
vit_b_16
vit_b_32
vit_l_16
vit_l_32
vit_h_14

question: how resource intense is it to train a ViT model on ImageNet from zero?
Are there other pre-trained ViT models available?
https://huggingface.co/models?search=vit

In the FunnyBirds article, post-hoc methods were used to measure the interpretability of ViT models


Parallelize produce requests

Currently, we sequentially send requests to each broker involved when flushing messages from the producer. We could parallelize these requests in order to improve performance.

again - celluloid might be useful for that

I actually think there's a simpler way – just write the request to each connection before starting to read any responses. It's mainly an issue of making the API nice.


edit bug

when editing, script tried to recreate edited stuff. dunno the details. needs investigation.

Could repro the issue now: After renaming, the Script tried to recreate the PoI with the new name.


terraform-cli-refactor

Refactoring the yaml to not have to press enter

Issue fixed with branch 1


sentinel has limitation of max of 3000 incident creation per day- add it to public docs.

[Enter feedback here]
sentinel has limitation of max of 3000 incident creation per day.
its not documented anywhere can you please document it.

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

ID: fc50e485-e943-d5dd-180f-68ddd8326811
Version Independent ID: 856810e4-c13f-97f8-6d19-33c58041a871
Content: Microsoft Sentinel service limits
Content Source: articles/sentinel/sentinel-service-limits.md
Service: microsoft-sentinel
GitHub Login: @yelevin
Microsoft Alias: yelevin

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

@yelevin
Could you Please review this and update as appropriate.

@ankkp
I'm going to assign this to the document author so they can take a look at it accordingly

@ManoharLakkoju-MSFT @RamanathanChinnappan-MSFT @ankkp
I'm looking into this issue.
May I ask what's the source of your information that there is such a limit?

#label:"backlog-item-created"

@ankkp Thanks for your patience!
I have investigated and documented this limit, which isn't exactly a hard limit, as you will see at this link:
Service limits for Microsoft Sentinel | Incident limits
I will consider this issue closed, but please feel free to comment further if necessary.
#please-close
cc: @batamig @ManoharLakkoju-MSFT @RamanathanChinnappan-MSFT

#please-close


scrollBottom && scrollRight position issue

Hi @souhe ,
When i try to scroll to bottom and the content height is smaller than the container height, the content will be pushed down(because topPosition become positive), so may you want to fix the following:
    scrollBottom(){
      if(this.state.realHeight>this.state.containerHeight)
        this.setState({topPosition: -(this.state.realHeight - this.state.containerHeight)});
    }

the same for scrollRight.
Thanks.

Fixed in version v.0.4.0.
Thanks for notice that issue.


User cannot copy data point's colour to itemStyle even though built-in themes can

Version
4.9.0
Steps to reproduce
By default, itemStyle displays items using data point's colour as its border colour. In "dark" theme, this is reversed so that the item's fill colour is the data point colour. However, it seems there is no way for the user to express this, because series-line.itemStyle.color and series-line.itemStyle.borderColor can only be fixed colours, gradients or textures: they cannot reference the data point's colour.
I also checked the source for the dark theme to see how it was doing this and there is nothing in the code related to changing the itemStyle's colour to match the data point colour.

Default theme
Dark theme

What is expected?
User should be able to set the colour using the data point's colour assigned to it by visualMap.
What is actually happening?
User can only set a fixed colour that is not related to the data point's colour.
I cannot just change the entire theme to get this effect, because (besides being unintuitive) changing the entire theme has other consequences.

no-stale


test-branch/execute-job/wrong-commit-sha-on-pr/c57d2e02-1417-4ca8-8885-4fb6b3894dcd

PR for testing f47cd1823e6541f89f0678f69d79826886023ee9

/canonical/self-hosted-runners/run-workflows 931ad21dd9ace68aaa0a7d341e94de96ad93fa6b


[LED] Assert fault LED's when association is set and cleared

https://github.com/openbmc/openbmc/issues/941
I doubt whether we have it, We can evaluate accordingly.

As per team discussion and interlock, Assigning it to Swetha.

@rahulmah Could you pls work with Swetha on this.

@sivassrr : I have already helped her on this story. She has also dropped the code for this.

In Review
Waiting for the development code to be merged for the functionality to work
https://gerrit.openbmc-project.xyz/#/c/4389/
https://gerrit.openbmc-project.xyz/#/c/4668/

Code in Review
https://gerrit.openbmc-project.xyz/#/c/4039/


French branch

Not sure if I'm supposed to do this? Also, it doesn't include the changes from my English version pull request.

Ohhh got it, sorry for getting this wrong!


To fix the errors and improve the code, I would suggest these chunks : )

fix the errors

What errors?
I'm sorry, but I see no benefit in what you've changed. use statements are there to make the code more readable, instead of using long, full paths to every type and function. Some of what you changed isn't even valid Rust, for example returning an i32 from main. Tests & Clippy are failing because the program doesn't compile now.
Please update the code to at least make it compile and pass CI checks.

In Rust, the default integer type is i32 because it is generally the fastest, even on 64-bit systems.
Yes, I agree with your CI checks

In response to your message on Matrix:

I have submitted the pull request, and I understand your concerns. However, I would like to know why the tests and Clippy are failing. Additionally, could you provide guidance on how to successfully compile the program?

The code from the main branch of this repository compiles successfully. All you have to do is run cargo build for the debug build, or cargo build --release for the optimized build.
The tests and Clippy are failing, because your code does not compile. There are a couple reasons for that. Previously, you've changed the return type of main to i32, which does not implement the Termination trait. Please see the CI logs for more information - the compiler tells you exactly what is wrong. You can also check if the CI will fail before creating a PR - just try to compile and test the program, make sure cargo clippy doesn't output any warnings and format your code with rustfmt.
The current version from your fork does not compile successfully, because you're trying to call a non-const function in a static declaration:
error[E0015]: cannot call non-const fn `<AtomicBool as std::default::Default>::default` in statics
  --> src/main.rs:22:47
   |
22 | static QUIET: std::sync::atomic::AtomicBool = std::sync::atomic::AtomicBool::default();
   |                                               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   |
   = note: calls in statics are limited to constant functions, tuple structs and tuple variants
   = note: consider wrapping this expression in `Lazy::new(|| ...)` from the `once_cell` crate: https://crates.io/crates/once_cell

For more information about this error, try `rustc --explain E0015`.

The fact that it's not allowed aside, may I ask, why are you even changing this? Would
static QUIET: std::sync::atomic::AtomicBool = std::sync::atomic::AtomicBool::default();

really be more readable to you than
use std::sync::atomic::AtomicBool;

static QUIET: AtomicBool = AtomicBool::new(false);

? What's the advantage of using full paths everywhere?

In response to your message on Matrix:

I have submitted the pull request, and I understand your concerns. However, I would like to know why the tests and Clippy are failing. Additionally, could you provide guidance on how to successfully compile the program?

The code from the main branch of this repository compiles successfully. All you have to do is run cargo build for the debug build, or cargo build --release for the optimized build.
The tests and Clippy are failing, because your code does not compile. There are a couple reasons for that. Previously, you've changed the return type of main to i32, which does not implement the Termination trait. Please see the CI logs for more information - the compiler tells you exactly what is wrong. You can also check if the CI will fail before creating a PR - just try to compile and test the program, make sure cargo clippy doesn't output any warnings and format your code with rustfmt.
The current version from your fork does not compile successfully, because you're trying to call a non-const function in a static declaration:
error[E0015]: cannot call non-const fn `<AtomicBool as std::default::Default>::default` in statics
  --> src/main.rs:22:47
   |
22 | static QUIET: std::sync::atomic::AtomicBool = std::sync::atomic::AtomicBool::default();
   |                                               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   |
   = note: calls in statics are limited to constant functions, tuple structs and tuple variants
   = note: consider wrapping this expression in `Lazy::new(|| ...)` from the `once_cell` crate: https://crates.io/crates/once_cell

For more information about this error, try `rustc --explain E0015`.

The fact that it's not allowed aside, may I ask, why are you even changing this? Would
static QUIET: std::sync::atomic::AtomicBool = std::sync::atomic::AtomicBool::default();

really be more readable to you than
use std::sync::atomic::AtomicBool;

static QUIET: AtomicBool = AtomicBool::new(false);

? What's the advantage of using full paths everywhere?

Thanks for explaining, and I've understood your point.
This is more readable 👍🏻

use std::sync::atomic::AtomicBool;

static QUIET: AtomicBool = AtomicBool::new(false);

I am more familiar with this way of representing it as a bool.

static QUIET: std::sync::atomic::AtomicBool = std::sync::atomic::AtomicBool::default();

No harm done; you're right, as always. Once again, thanks for taking the time to explain things to me.


$scraper->user->videos('iratee');

Unable to retrieve NEXT_DATA from DOM )

Hey there, this is a recent change with verification pages. I will be updating this tonight with better error handling. I am still working on the best work around for captcha right now.


Remove token balances in deposit dropdown list

resolves #144

We have update deposit page token list to display balance from user wallet


Rendering gradients leaves white solid fill

In attempting to draw an SVG that contains a gradient a solid while fill of the same shape as the svg is just left

im not too sure why this happens sorry but the intended result is for it to draw something that looks like:

Heres the svg file im trying to render:
platformsvg.txt

Hm...i'll take a look at it. Thanks for reporting! 🙂

So...it doesn't work because bevy_svg currently only draws usvg::NodeKind::Path (here) and ignores every other kind of node.
Looking at resvg makes me think, that a not too small amount of work would be needed to make gradients work. I would really like to have this in bevy_svg, but i don't currently have the bandwidth to do it myself. So, if anyone wants to step up, feel free to do so and ask me if you need any help.


Cleanup use analysis

This PR is a first pass at simplifying some of the logic inside of the useAnalysis hook.
Notable changes are:

Setters are "pure" and only call state setters.
The createAnalysis function argument no longer returns the new analysis id. Because it performs the redirect, there isn't much reason to return the id, and this change simplifies expectations.
To that end, all redirect logic not occurs in createAnalysis (see changes in StartPage.tsx). This has a side benefit of addressing the pesky "Visualization not found" that appears when a new viz is created in a new analysis.
Some uses of useState were removed.

We might want to consider combining useAnalysis and useWorkspaceAnalysis into a single hook. I'm finding the indirection that passing in createAnalysis introduces to be confusion and maybe unnecessary.

We might want to consider combining useAnalysis and useWorkspaceAnalysis into a single hook. I'm finding the indirection that passing in createAnalysis introduces to be confusion and maybe unnecessary.

I see the merit - as I recall, useAnalysis was originally meant to be a generic "core interface" for interacting with EDA analyses, and potentially used in UIs other than the EDA workspace (e.g., the standalone MapVeu app). It's come to pass that we're not really using us[ing]Analysis outside of the EDA workspace, so I think it's fine to combine useAnalysis and useWorkspaceAnalysis.

We might want to consider combining useAnalysis and useWorkspaceAnalysis into a single hook. I'm finding the indirection that passing in createAnalysis introduces to be confusion and maybe unnecessary.

I see the merit - as I recall, useAnalysis was originally meant to be a generic "core interface" for interacting with EDA analyses, and potentially used in UIs other than the EDA workspace (e.g., the standalone MapVeu app). It's come to pass that we're not really using us[ing]Analysis outside of the EDA workspace, so I think it's fine to combine useAnalysis and useWorkspaceAnalysis.

We've also been homing in on the idea that MapVeu will use the EDA workspace

Aha, we should not "create" if only uiState is modified. I think this can be supported easily

Aha, we should not "create" if only uiState is modified. I think this can be supported easily

See https://github.com/VEuPathDB/web-eda/pull/1284/commits/50c8ba82867ac3f189eeed88856404bda5e8b621

@jtlong3rd I've consolidated the hook logic and removed useWorkspaceAnalysis. Take a look when you get a chance and let me know what you think. Thanks!

I think this is ready for review. I plan to merge this into main once #1283 is merged. I will then open a new PR to handle batching analysis update requests to the user service.

I removed the file I accidentally added, and I rebased on the latest from the target branch.

Shall I make a ticket for removing all the MapVeu [sic] code, prior to work starting on full screen viz's?

Ah, yes, that's probably a good idea.

Cool - here it is https://github.com/VEuPathDB/web-eda/issues/1300
On Tue, Aug 9, 2022 at 6:19 PM Dave Falke @.***> wrote:

Shall I make a ticket for removing all the MapVeu [sic] code, prior to
work starting on full screen viz's?
Ah, yes, that's probably a good idea.
—
Reply to this email directly, view it on GitHub
https://github.com/VEuPathDB/web-eda/pull/1284#issuecomment-1209654818,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACLLHZSBAASVMEK4TKQA63VYKHJLANCNFSM5453JJ7A
.
You are receiving this because you commented.Message ID:
@.***>


Added support of new operating systems

Because new kinds of servers are running on these platforms, it can be cool to show them too (instead of "Other")

I really really doubt you can run a Minecraft server on any Apple mobile OS. Correct me if I'm wrong though.

I'm sorry for you but I'm working on this and...

This isn't something that I imagine will get mass adoption - I can't see anyone at all using an Apple Watch or Apple TV to run a Minecraft server, and running a server off an Android phone or iOS device is something most people would only do for the novelty of it. These stats aren't really useful for the vast majority of developers.

This project is managed by @GroupeMINASTE and it runs flawlessly on iOS, watchOS, tvOS and of course macOS. Not only does it runs smoothly, but the power this project has allows it to run on any smart device with an incredibly low CPU and RAM consumption, as it is not developed in Java. I’m ready to bet you will soon regret using a Java Minecraft server haha

An Apple TV as a Server makes perfect sense though, it has Gigabit Ethernet connectivity and is on the network 24/7. And having it on your phone allows you to carry a desktop-class server with you and play with your friends anywhere thanks to the Mobile Hotspot feature.

@md678685 Even if it's not used by a lot of people, it's great to have its support integrated

IMHO this is a perfect example of something that should be in the "Other" category. Also it doesn't seem like your server is a Bukkit or Sponge server, which is primarily what bStats tracks. Either way, why not just create your own server implementation page, and create your own breakdown of other OSes there?

I’m ready to bet you will soon regret using a Java Minecraft server haha

@NathanFallet @PlugNCS It's time for a reality check - you need it.
Alternative Java Edition server implementations written in programming languages other than Java have been a thing for years (Cuberite is probably the best-known - it's written in C++). You're bringing yet another implementation, this time in Swift. None of them have really achieved mass adoption. Even alternative implementations that are written in Java (such as Sponge) are far outstripped by Bukkit-derived implementations.
You do not realize the magnitude of the task you have ahead. Cuberite has had a multi-year head start, and they're nowhere close to a full server implementation. It seems you started this particular project about three weeks ago, and the current state of your project is not particularly impressive. (I could probably get just as far as you have in about three weeks if I were to use a language I'm not familiar with, perhaps much further ahead if I had a good understanding of the language I chose to work with.)
I have never worked with Swift (too busy with other stuff), so I'm not going to comment on code quality or anything (I'd be utterly unqualified to discuss such matters). But as someone who's had an interest in Minecraft server implementations, I can say with high certainty that your task ahead is hard and will be filled with major obstacles you're going to need to solve.
As for this PR - I think it's niche to include these implementations in the general OS charts. Your alternative server implementation brings almost nothing new to the table.

Really great project you've started, but I agree with the others - it's not mature enough to warrant leaving the "Other" category. Maybe if/when it achieves a following or widespread usage.

Considering that there are no plugins for these kind of servers that support bStats, it would be pointless to add them to the server chart.


Cant start qbit websocket standalone example

Untouched download, last stable realese, eclipse, without gradle (i dont use gradle only maven).
i put breakpoint on
HttpServer httpServer = httpServerBuilder().setPort(8080).build();
in debugger last place before NullPointerExpection is
public static HttpServerFactory getHttpServerFactory() {
return httpServerFactoryRef.get();
}
in FactoryPsi.java
debbuger show that httpServerFactoryRef == null
and breakpoint placed on setHttpServerFactory, dont showedup.
so maybe it is not so standalone? or there is someway to set it?
full stacktrace :
Exception in thread "main" java.lang.NullPointerException
at io.advantageous.qbit.Factory.createHttpServer(Factory.java:394)
at io.advantageous.qbit.http.server.HttpServerBuilder.build(HttpServerBuilder.java:226)
at websocket.EchoWebSocket.main(EchoWebSocket.java:39)

QBit does not have an IO lib.
It uses Vertx as an IO lib. You will need to include the qbit-vertx jar.


HttpConnectionDispatcher.ExecuteAsync failing with error 400 and no logs

I have a routing handler that maps query strings to socket endpoints that was working fine  up to 1.0.0-preview3-25476. No it is failing with no logging out put with a 400. The code I have is pretty straight forward.
    public class SocketRouteBuilder
    {
        private readonly HttpConnectionDispatcher _dispatcher;
        private readonly RouteBuilder _routes;

        public SocketRouteBuilder(RouteBuilder routes, HttpConnectionDispatcher dispatcher)
        {
            _routes = routes;
            _dispatcher = dispatcher;
        }

        public void MapSocket(string path, Action<ISocketBuilder> socketConfig) =>
            MapSocket(path, new HttpSocketOptions(), socketConfig);

        public void MapSocket(string path, HttpSocketOptions options, Action<ISocketBuilder> socketConfig)
        {
            var socketBuilder = new SocketBuilder(_routes.ServiceProvider);
            socketConfig(socketBuilder);
            var socket = socketBuilder.Build();
            _routes.AddPrefixRoute(path, new RouteHandler(c => _dispatcher.ExecuteAsync(c, options, socket)));
        }

        public void MapEndPoint<TResolver, TEndPoint>(string path)
            where TResolver : EndPointResolver<TEndPoint>
            where TEndPoint : EndPoint
        {
            _routes.AddPrefixRoute(path, new RouteHandler(context =>
            {
                var endpointResolver = context.RequestServices.GetService<TResolver>();
                var endpoint = endpointResolver.Resolve(path, context);
                return _dispatcher.ExecuteAsync(context, new HttpSocketOptions(), endpoint.OnConnectedAsync);
            }));
        }
    }

The endpoints resolve and are valid, but something is failing and I have nothing to go by except for the 400 going back to the client.
rich

Lots of code changed recently so it's likely that you're making the wrong requests from the client. It's hard to know without seeing more data.

the culprit seems to be
`   public static class HttpDispatcherAppBuilderExtensions
{
public static IApplicationBuilder UseSocketsEx(this IApplicationBuilder app, Action callback)
{
var dispatcher = app.ApplicationServices.GetRequiredService();
        var routes = new RouteBuilder(app);

        callback(new SocketRouteBuilder(routes, dispatcher));

        app.UseWebSocketConnections();
        app.UseRouter(routes.Build());
        return app;
    }

`
UseWebSocketConnections lingered and should be UseWebSockets
rich


Fix frontend routes so logout is working without throwing errors

Describe the issue
As a dev, I'd like to have frontend routes working as expected so the logout functionality is working without throwing errors.
Additional context
If users go to the service while logged out and click on the main logo, it tries to get users to goto the dashboard which throws an error.
If users login to the registry and then logout, it throws another error as it again tries to route users to the dashboard which is obviously unauthorized now.
Definition of done
When the above is fixed.

**Bugfix updated on the platform-registry-prod silver cluster: Jan 28, 2021

**Bugfix updated on the platform-registry-prod silver cluster: Jan 28, 2021


MultiMedia not displayed in Table view

Hello,
In the table view, for MultiMedia fields, the images are not displayed anymore, only their links.
I would expect images to be displayed in the Table View.
Using Metadata-Menu 0.8.4 (and Obsidian 1.5.8) in the sandbox vault:

Using Metadata-Menu 0.7.7:

The same applies when adding several images.
Using Metadata-Menu 0.8.4:

Using Metadata-Menu 0.7.7:

Thank you!

Hi I can't reproduce this. Can you paste the yaml definition of your fileclass please?

good catch. will be fixed in 0.8.6

fixed in 0.8.6

Thank you, I tested it with 0.8.7 and it works 👍
The thumbnail is a bit small ( width: 40px ) but that's easily tweakable with a bit of CSS.


Implement "ladder notification"

Notify the user when another player in his range is searching for a 1v1 ranked game opponent.
 Think of some better way to notify the user. One idea:
Display a transient notification that, when the user clicks on it, opens the ranked1v1 screen.

Seems like this very important feature was never implemented?

I believe this actually resulted in the current toast we have now


feat: added sort styles

Hi @RodSarhan,
I've added sort keys rule, the only thing I did not implement is the astHelpers.isUseStylesHook VariableDeclaration, if you think we need this, can you please take a moment and implement it?

Hey @efstathiosntonas , Thank you!
Do you mind adding some tests before we merge this? We can edit the ones from the react native plugin repo as a start and add the cases specific to UniStyles (function styles) later.

@RodSarhan totally missed that there was a tests folder :face_palm:
I'm really bad at writing tests, will try to do my best

merged and released the version that includes the new rule, no need to worry about the tests for now as i'll add some while trying to make this rule work with the styles returned from functions.

@RodSarhan thanks, sorry for not pushing, I'm in the edge of a huge release and got 0 time :/


What other events should we add?

Currently, we have events for prompts, tasks, and subtasks. What other events should we add to to the framework?

Closing, the events we've got seem to be sufficient enough for now. We can track specific missing events in newer issues.


refactor: make number-input component headless

Closes #1887
Description
make number-input component headless
Current behavior (updates)
New behavior
Is this a breaking change (Yes/No):
No
Additional Information

Thanks!


NetSuite - Infinite sync when entering incorrect credentials

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

Version Number:
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail: Error tests - Connect Netsuite
Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause - Internal team
Action Performed:

Login with an account that is an admin of a control workspace (or create one in OD if needed)
Enable accounting in workspace settings
Navigate to Accounting
Click on the connect button next to NetSuite
Reach the credentials step
Enter incorrect the NetSuite tokens

Expected Result:
Entering the Netsuite credentials incorrectly should trigger an error state and RBRs to indicate the connection error.
Actual Result:
Infinite loading is displayed and there's no way to cancel it. No error about incorrect credentials is triggered.
Workaround:
No
Platforms:

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

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

Screenshots/Videos

View all open jobs on GitHub

Asked a design question about this here: https://expensify.slack.com/archives/C06ML6X0W9L/p1721862798362719

Does this need to be External at all @yuwenmemon? Or will you handle it?

Nah this will be all me.

Any reason why this issue is still open? The linked PR hit IS production.

Pretty sure this can be closed now.


Virtual Assistant: change "send" utterance when adding participant for new meeting and email

Project
Name: VA
Language: C#
Description

Testing shows users feel confused when adding participant(s) and utterance back from VA is "send", they did not include enough info yet for completion.
Utterance: Alright, I will send this to [contact name + email adress] and [contact name + email adress]. What is the subject of the event?
To Reproduce

Expected behavior

New utterance, change "send" into "add"
Alright, I will add [contact name + email adress] and [contact name + email adress]. What is the subject of the event?
Screenshots

Additional context

Logic change for showing participants:
!Read out the display names of the alias of all participant the email will be send to.
1-5 participants: read out all alias names.

Rule if >5 participant, add this prompt…

Add an extra prompt:
Bot: I'm ready to create the event/send the email. The event is on [date] at [time start] till [time end], with [contact person + # more]. The subject of the event is [subject]. Do you want to go through the whole list of participants?
User: Yes
Bot: [List of recipients].
Bot: [After list of recipients or user says “No"] Can you confirm this event/email?
User: Yes
Bot: Okay, I have added the event to your calendar, named [subject].
.


Support for EF.Functions

Do you plan PostgreSQL support for EF.Functions?
Loosely related to #173 Translate DateTime.Add* and #118
EF Core links
#9585 Implement useful mappings in EF.Functions
pull #10264 Implementation Translate DateDiff

@mudrz, it's important to distinguish between translating standard .NET functions (such as DateTime.Add*, and special database functions under EF.Functions. For the former, Npgsql already translates a lot of .NET functions, including some PostgreSQL-specific ones (e.g. regular expressions). For the latter Npgsql does support EF.Functions.Like() and also the PostgreSQL-specific EF.Functions.ILike().
So basically this should be treated on a function-per-function basis - if there's something you'd like to see translated, open an issue requesting it (or vote on the existing issue).

@roji, thanks for the explanation, makes sense


LocalEntryNotFoundError

from typing import Any

from pydantic import BaseModel
from unstructured.partition.pdf import partition_pdf

# Path to save images
path = "/home/zhanglv/code/small-project/data/"
filename =path + "LLaVA.pdf"
# Get elements
raw_pdf_elements = partition_pdf(
    filename=path + "LLaVA.pdf",
    # Using pdf format to find embedded image blocks
    extract_images_in_pdf=True,
    # Use layout model (YOLOX) to get bounding boxes (for tables) and find titles
    # Titles are any sub-section of the document
    infer_table_structure=True,
    # Post processing to aggregate text once we have the title
    chunking_strategy="by_title",
    url=None,
    # Chunking params to aggregate text blocks
    # Attempt to create a new chunk 3800 chars
    # Attempt to keep chunks > 2000 chars
    # Hard max on chunks
#     max_characters=4000,
#     new_after_n_chars=3800,
#     combine_text_under_n_chars=2000,
#     image_output_dir_path=path,
)

executed in 18.9s, finished 00:08:42 2023-11-25

---------------------------------------------------------------------------
LocalEntryNotFoundError                   Traceback (most recent call last)
Cell In[1], line 10
      8 filename =path + "LLaVA.pdf"
      9 # Get elements
---> 10 raw_pdf_elements = partition_pdf(
     11     filename=path + "LLaVA.pdf",
     12     # Using pdf format to find embedded image blocks
     13     extract_images_in_pdf=True,
     14     # Use layout model (YOLOX) to get bounding boxes (for tables) and find titles
     15     # Titles are any sub-section of the document
     16     infer_table_structure=True,
     17     # Post processing to aggregate text once we have the title
     18     chunking_strategy="by_title",
     19     url=None,
     20     # Chunking params to aggregate text blocks
     21     # Attempt to create a new chunk 3800 chars
     22     # Attempt to keep chunks > 2000 chars
     23     # Hard max on chunks
     24 #     max_characters=4000,
     25 #     new_after_n_chars=3800,
     26 #     combine_text_under_n_chars=2000,
     27 #     image_output_dir_path=path,
     28 )

File ~/venv/small-project/lib/python3.8/site-packages/unstructured/documents/elements.py:306, in process_metadata.<locals>.decorator.<locals>.wrapper(*args, **kwargs)
    304 @functools.wraps(func)
    305 def wrapper(*args: _P.args, **kwargs: _P.kwargs) -> List[Element]:
--> 306     elements = func(*args, **kwargs)
    307     sig = inspect.signature(func)
    308     params: Dict[str, Any] = dict(**dict(zip(sig.parameters, args)), **kwargs)

File ~/venv/small-project/lib/python3.8/site-packages/unstructured/file_utils/filetype.py:551, in add_metadata_with_filetype.<locals>.decorator.<locals>.wrapper(*args, **kwargs)
    549 @functools.wraps(func)
    550 def wrapper(*args: _P.args, **kwargs: _P.kwargs) -> List[Element]:
--> 551     elements = func(*args, **kwargs)
    552     sig = inspect.signature(func)
    553     params: Dict[str, Any] = dict(**dict(zip(sig.parameters, args)), **kwargs)

File ~/venv/small-project/lib/python3.8/site-packages/unstructured/chunking/title.py:277, in add_chunking_strategy.<locals>.decorator.<locals>.wrapper(*args, **kwargs)
    275 @functools.wraps(func)
    276 def wrapper(*args: _P.args, **kwargs: _P.kwargs) -> List[Element]:
--> 277     elements = func(*args, **kwargs)
    278     sig = inspect.signature(func)
    279     params: Dict[str, Any] = dict(**dict(zip(sig.parameters, args)), **kwargs)

File ~/venv/small-project/lib/python3.8/site-packages/unstructured/partition/pdf.py:157, in partition_pdf(filename, file, include_page_breaks, strategy, infer_table_structure, ocr_languages, languages, max_partition, min_partition, include_metadata, metadata_filename, metadata_last_modified, chunking_strategy, links, **kwargs)
    151         languages = convert_old_ocr_languages_to_languages(ocr_languages)
    152         logger.warning(
    153             "The ocr_languages kwarg will be deprecated in a future version of unstructured. "
    154             "Please use languages instead.",
    155         )
--> 157 return partition_pdf_or_image(
    158     filename=filename,
    159     file=file,
    160     include_page_breaks=include_page_breaks,
    161     strategy=strategy,
    162     infer_table_structure=infer_table_structure,
    163     languages=languages,
    164     max_partition=max_partition,
    165     min_partition=min_partition,
    166     metadata_last_modified=metadata_last_modified,
    167     **kwargs,
    168 )

File ~/venv/small-project/lib/python3.8/site-packages/unstructured/partition/pdf.py:287, in partition_pdf_or_image(filename, file, is_image, include_page_breaks, strategy, infer_table_structure, ocr_languages, languages, max_partition, min_partition, metadata_last_modified, **kwargs)
    285 with warnings.catch_warnings():
    286     warnings.simplefilter("ignore")
--> 287     _layout_elements = _partition_pdf_or_image_local(
    288         filename=filename,
    289         file=spooled_to_bytes_io_if_needed(file),
    290         is_image=is_image,
    291         infer_table_structure=infer_table_structure,
    292         include_page_breaks=include_page_breaks,
    293         languages=languages,
    294         metadata_last_modified=metadata_last_modified or last_modification_date,
    295         **kwargs,
    296     )
    297     layout_elements = []
    298     for el in _layout_elements:

File ~/venv/small-project/lib/python3.8/site-packages/unstructured/utils.py:178, in requires_dependencies.<locals>.decorator.<locals>.wrapper(*args, **kwargs)
    169 if len(missing_deps) > 0:
    170     raise ImportError(
    171         f"Following dependencies are missing: {', '.join(missing_deps)}. "
    172         + (
   (...)
    176         ),
    177     )
--> 178 return func(*args, **kwargs)

File ~/venv/small-project/lib/python3.8/site-packages/unstructured/partition/pdf.py:377, in _partition_pdf_or_image_local(filename, file, is_image, infer_table_structure, include_page_breaks, languages, ocr_mode, model_name, metadata_last_modified, **kwargs)
    373         process_with_model_kwargs[key] = value
    375 if file is None:
    376     # NOTE(christine): out_layout = extracted_layout + inferred_layout
--> 377     out_layout = process_file_with_model(
    378         filename,
    379         is_image=is_image,
    380         extract_tables=infer_table_structure,
    381         model_name=model_name,
    382         pdf_image_dpi=pdf_image_dpi,
    383         **process_with_model_kwargs,
    384     )
    385     if model_name.startswith("chipper"):
    386         # NOTE(alan): We shouldn't do OCR with chipper
    387         final_layout = out_layout

File ~/venv/small-project/lib/python3.8/site-packages/unstructured_inference/inference/layout.py:481, in process_file_with_model(filename, model_name, is_image, fixed_layouts, extract_tables, pdf_image_dpi, **kwargs)
    469 def process_file_with_model(
    470     filename: str,
    471     model_name: Optional[str],
   (...)
    476     **kwargs,
    477 ) -> DocumentLayout:
    478     """Processes pdf file with name filename into a DocumentLayout by using a model identified by
    479     model_name."""
--> 481     model = get_model(model_name, **kwargs)
    482     if isinstance(model, UnstructuredObjectDetectionModel):
    483         detection_model = model

File ~/venv/small-project/lib/python3.8/site-packages/unstructured_inference/models/base.py:55, in get_model(model_name, **kwargs)
     53 elif model_name in YOLOX_MODEL_TYPES:
     54     model = UnstructuredYoloXModel()
---> 55     initialize_params = {**YOLOX_MODEL_TYPES[model_name], **kwargs}
     56 elif model_name in CHIPPER_MODEL_TYPES:
     57     logger.warning(
     58         "The Chipper model is currently in Beta and is not yet ready for production use. "
     59         "You can reach out to the Unstructured engineering team in the Unstructured "
   (...)
     63         "zt-1x7cgo0pg-PTptXWylzPQF9xZolzCnwQ",
     64     )

File ~/venv/small-project/lib/python3.8/site-packages/unstructured_inference/utils.py:44, in LazyDict.__getitem__(self, key)
     42 evaluate = value.evaluate
     43 args, kwargs = value.info
---> 44 value = evaluate(*args, **kwargs)
     45 if self.cache:
     46     self._raw_dict[key] = value

File ~/venv/small-project/lib/python3.8/site-packages/huggingface_hub/utils/_validators.py:118, in validate_hf_hub_args.<locals>._inner_fn(*args, **kwargs)
    115 if check_use_auth_token:
    116     kwargs = smoothly_deprecate_use_auth_token(fn_name=fn.__name__, has_token=has_token, kwargs=kwargs)
--> 118 return fn(*args, **kwargs)

File ~/venv/small-project/lib/python3.8/site-packages/huggingface_hub/file_download.py:1291, in hf_hub_download(repo_id, filename, subfolder, repo_type, revision, library_name, library_version, cache_dir, local_dir, local_dir_use_symlinks, user_agent, force_download, force_filename, proxies, etag_timeout, resume_download, token, local_files_only, legacy_cache_layout)
   1285         raise LocalEntryNotFoundError(
   1286             "Cannot find the requested files in the disk cache and"
   1287             " outgoing traffic has been disabled. To enable hf.co look-ups"
   1288             " and downloads online, set 'local_files_only' to False."
   1289         )
   1290     else:
-> 1291         raise LocalEntryNotFoundError(
   1292             "Connection error, and we cannot find the requested files in"
   1293             " the disk cache. Please try again or make sure your Internet"
   1294             " connection is on."
   1295         )
   1297 # From now on, etag and commit_hash are not None.
   1298 assert etag is not None, "etag must have been retrieved from server"

LocalEntryNotFoundError: Connection error, and we cannot find the requested files in the disk cache. Please try again or make sure your Internet connection is on.

the error，how to process？

Also stuck here.

Hello, I got the same issues when infer_table_structure=True :
LocalEntryNotFoundError: An error happened while trying to locate the file on the Hub and we cannot find the requested files in the local cache. Please check your connection and try again or make sure your Internet connection is on.

Any solution to avoid this error?

This error from the huggingface_hub Python library indicates that library is unable to connect to the HuggingFace API to download a model used by unstructured-inference.
If you do not have outbound internet connectivity this can cause this error. Also, if the huggingface API is unresponsive for whatever reason, this can also produce this error.
If you have outbound connectivity, retrying several times, as odd as that seems, is reported to help. Perhaps under load the HF API will sometimes respond if you try repeatedly.

Hello, I got the same issues when infer_table_structure=True :
LocalEntryNotFoundError: An error happened while trying to locate the file on the Hub and we cannot find the requested files in the local cache. Please check your connection and try again or make sure your Internet connection is on.

Any solution to avoid this error?

@Auronus92 were you able to resolve this issue? I am having the same error.


Consider renaming repo

As Solana "smart contracts" are called programs, the name of the repo might be misleading.
Consider renaming it to:

protocol-programs: would be consistent with protocol-contracts

As "programs" might be seen as a generic word, maybe: protocol-solana-programs

Let's keep it this way for consistency


Use DEK from geth wallet instead of redux everywhere

Expected Behavior
When the DEK is used in the app, it should be from the native geth wallet instead of the redux store. Also consider securing the DEK with a passphrase
Current Behavior
Currently we only do this for uploading and downloading profile info, but not other DEK functions (ex. comment encryption). Currently the passphrase is an empty string

DEK - data encryption key, dataEncryptionKey.ts

delete dataEncryptionKey from State in web3/reducer.ts
fix all type errors
switch to use getWallet to get the encryption key
getWallet example:  profileInfo.ts


Fix bug: Return empty access_keys object when access keys do not exist

While running the AWS sync in Cartography, the following error occurs, causing the sync process to fail:
ERROR:cartography.sync:Unhandled exception during sync stage 'aws'
Traceback (most recent call last):
  File "/home/REDACTED/cartography/cartography/sync.py", line 113, in run
    stage_func(neo4j_session, config)
  File "/home/REDACTED/cartography/cartography/util.py", line 197, in timed
    return method(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/REDACTED/cartography/cartography/intel/aws/__init__.py", line 298, in start_aws_ingestion
    sync_successful = _sync_multiple_accounts(
                      ^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/REDACTED/cartography/cartography/intel/aws/__init__.py", line 169, in _sync_multiple_accounts
    _sync_one_account(
  File "/home/REDACTED/cartography/cartography/intel/aws/__init__.py", line 64, in _sync_one_account
    RESOURCE_FUNCTIONS[func_name](**sync_args)
  File "/home/REDACTED/cartography/cartography/util.py", line 197, in timed
    return method(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/REDACTED/cartography/cartography/intel/aws/iam.py", line 819, in sync
    sync_user_access_keys(neo4j_session, boto3_session, current_aws_account_id, update_tag, common_job_parameters)
  File "/home/REDACTED/cartography/cartography/util.py", line 197, in timed
    return method(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/REDACTED/cartography/cartography/intel/aws/iam.py", line 795, in sync_user_access_keys
    access_keys = get_account_access_key_data(boto3_session, user["name"])
                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/REDACTED/cartography/cartography/util.py", line 197, in timed
    return method(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/REDACTED/cartography/cartography/intel/aws/iam.py", line 230, in get_account_access_key_data
    for access_key in access_keys['AccessKeyMetadata']:
                      ~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^
KeyError: 'AccessKeyMetadata'
ERROR:__main__:Error in AWS account sync REDACTED: 'AccessKeyMetadata'

The fix involves returning the access_keys object directly from the get_account_access_key_data function. This ensures that the function returns the correct data structure, even if the AccessKeyMetadata key is missing.
The function was tested with various AWS accounts to ensure it correctly handles cases where the AccessKeyMetadata key is present and when it is missing.
Verified that the sync process was completed successfully without any errors.

LGTM

Hi @austincomstockzoom, thank you for the fix! Can you please sign the CLA?

Check again, thanks


Merge gd.py and gd.api

Since gd.api is essentially just a wrapper around gd.py's system, we could integrate it into gd.py itself.
I propose to either create a new subpackage, or add a new module; named simply server. This way, people could do something like:
import gd
gd.server.start()

Started working on this issue. Commit: 87af563.

I think the issue is generally resolved. gd.api was deleted and now most endpoints + some new ones (like /api/daily) are here. :)

Forgot to close this one for no reason... So yeah, closing it now.


Update Network test to avoid broken build

Next Network release has a bugfix that's going to cause breaking change in SDK (changing type of property from RouteFilter to Id-reference type). It has already been merged in Swagger and this causes our validation scripts to fail on step with SDK build. This PR fixes that.

cc: @MikhailTryakhov


Set accept and content-type headers to application/vnd.blackducksoftware.user-4+json

Hi,
According to documentation and my tests, accept and content-type headers need to be set to application/vnd.blackducksoftware.user-4+json. If not, some request may fail with 406 error code (for example http//BLACKDUCK_URL/api/users/USER_UID/last-login).
I would recommend to change Client.py code :
if 'accept' not in lc_keys and 'content-type' not in lc_keys:
                # set default media type only if neither 'accept' nor 'content-type'
                # exist as some endpoints may only accept one or the other but not both
                lc_keys['accept'] = "application/json"
                lc_keys['content-type'] = "application/json"

to
if 'accept' not in lc_keys and 'content-type' not in lc_keys:
                # set default media type only if neither 'accept' nor 'content-type'
                # exist as some endpoints may only accept one or the other but not both
                lc_keys['accept'] = "application/vnd.blackducksoftware.user-4+json"
                lc_keys['content-type'] = "application/vnd.blackducksoftware.user-4+json"

Most endpoints use their own content type versioning. By design, client provides sensible defaults that can easily be overridden by users.
Headers can be passed to the underlying request via the kwargs.


DaemonSetChecker should pass additional information on check failure

We should pass up more information when checks fail

Should pass up the node that failed to build the DS pod
Should pass the kube error for the failure

Recent DS Check refactor #515 addresses this in latest image: kuberhealthy/daemonset-check:v3.0.0


input-error on some themes doesn't look like a error

Hello, before the update the input error was showing the color red, but after appear like a low pink or something I attach an image, in another themes don't happen that
Dark theme

in Bumblebee theme

this happens in some themes like light, dark,Halloween and more..

Hey,
I have the same issue after update. The faulty inputs are almost invisible.

This will be fixed today

Fixed in 2.2.0

Thank you for your great work!


this.properties not initialized with default values (SPFX Client Webpart)

Category

[x] Question
[ ] Typo
[ ] Bug
[ ] Additional article idea

Hi all,
I'm developing a SPFX Modern Site Client webpart that has a large number of properties (currently around 40 and growing).
Due to that, Ive moved the non-critical properties to a form on the webpart itself when in Edit Mode.
The issue I'm having is that all properties in "this.properties" are null on a new Webpart, essentially not using the defaults.
I'm defining all defaults on Webpart.manifest.json
Any ideia why the object is not initialized with the defaults?
Thanks.

Hi Ralms, thanks for your feedback.
Could you double-check the contents of the manifest? Press Ctrl+F12 in the page and go to the Manifests tab. Find your web part manifest there. Does it contain all the properties in the right preconfigured entries?
Are the properties that you care about "null" or "undefined" (or not defined at all). It's a small but importance difference in JavaScript.
You mentioned that you moved non-critical properties. Before doing that was "this.properties" correctly set up? Now you don't get any property at all in "this.properties", or it's just missing the ones you moved?

Because I don't see what exactle resolved the issue for @Ralms I will leave my case for further readers. I had the same problem when properties initial values were specified in manifest file, but none of them were set on webpart.
In my case the problem was old manifest being served by gulp. Until I've restarted the gulp serve command old manifest were being used (that didn't contain new properties).
Suggestion by @mpasarin of using Ctrl+F12 to search for manifest was the key here.