chore(deps): update styled-jsx to ^3.3.1

Fix for #519
Checklist

[ ] Fix linting errors
[ ] Tests have been added / updated (or snapshots)

Change information
Update styled-jsx to ^3.3.1 to allow compatibility with react@^17

styled-jsx#695 This issue has not been fixed and will not be packaged even if we upgrade.
It is not possible to upgrade this dependency yet, so perhaps we should look for another solution to replace styled-jsx.

Maybe the compatibility code in 695 is the way to go. (it's a way to add a hack to the runtime)

Hi @unix,
Well, at least could it be frozen to<EMAIL_ADDRESS>instead of<EMAIL_ADDRESS>?

Well, at least could it be frozen to<EMAIL_ADDRESS>instead of<EMAIL_ADDRESS>?

The current version we have released is frozen in version 3.3.0. I have tried to upgrade to 3.3.1, but the esm package is buggy and I still need to test and debug the esm bundle. 😢
It's really troublesome here, I will maintain this PR. (most of the work is needed to test the availability of the esm bundle)

I have tried to upgrade to 3.3.1, but the esm package is buggy and I still need to test and debug the esm bundle. cry
It's really troublesome here, I will maintain this PR. (most of the work is needed to test the availability of the esm bundle)

Okay =/.
I will stick to<EMAIL_ADDRESS>/ react@16 until https://github.com/vercel/styled-jsx/issues/695 is fixed.
Can't help you much around the esm bundle issues, my knowledge in js / ts / es modules are really limited

I tried to fix this issue with dirty code and so far it seems to be compatible with React 17, I did some local testing of the module but can't guarantee the coverage of the module.
I have moved the target to the rc branch and plan to release a few canary versions as trial versions, and if there are no problems, I will point stable to the latest version later.

@ofekashery
Recently, Vercel often fails to build because the default cache cannot be removed and each build reverts to an earlier cache configuration, so we have to manually redeploy to clear the cache.
Right now Vercel cannot manually configure the cache... 😿


Conditional logging of "Response-Body-Transformed: Dechunked"

Each time a rule triggers and is logged, it generates a line in the audit log with the "Response-Body-Transformed: Dechunked" in case the response was dechunked.
This wastes a lot of disk space and is the normal behaviour, not a real warning.
We should have a directive (or a compile flag) to disable that logging.
Ex. of compile flag: #marcstern@9e68805

Fixed at: 41ae8db5715a14807328082c31d126acd11d9f79


dont use Ledger's getDongle, to use properly specified device path

Currently specifying the device path doesn't really do anything aside from error if that device path specified doesn't exist at all.
As another side-effect is enumerate shows an error for the keyboard HID instead of falsely reporting a fingerprint for it:
[{"path": "0001:XXX:00", "type": "ledger", "serial_number": "0001", "fingerprint": "deadbeef"}, {"error": "Could not open client or get fingerprint information: Exception : Invalid sequence", "path": "0001:XXX:01", "type": "ledger", "serial_number": "0001"}]

When commands are put towards the keyboard HID I get:
btchip.btchipException.BTChipException: Exception : Invalid sequence raised. Should we capture this error and print something prettier?

Pushed an additional check to ignore the keyboard HID during enumeration. btchip-python does this exact check.

tACK f45b8e7119ec276c6bafc90df4e2c2e9f18ffd5e


How to install?

I figured out that you are using an innosetup script, but when I double-click that script I get:
Compiler error: Line 4: Required parameter FileName missing
Then kInd of figured out that you are using Rust, which I know nothing about.  Installed Rust, and tried:
rustc build.rs
Got this:
error[E0433]: failed to resolve: use of
--> build.rs:3:23
|
3 |         let mut res = winres::Windo
|                       ^^^^^^ use of
error: aborting due to previous error
Tried: cargo build
It downloaded a bunch of stuff but got some errors.  It created a target directory but I don't know what to do with it.  Tried to run the innosetup file again and got the same error as previous.
I"m lost as to how to get this installed.

The simplest way is to just use the release binaries: https://github.com/MCOfficer/bitwarden-autotype/releases
Otherwise:

Get Rust from https://rustup.rs/
You also need windres.exe (from MinGW) or rc.exe (from Microsoft) in your PATH
then just cargo build --release and check your target folder for the binary

As for Innosetup, I can't recall how I used it on my dev machine. The CI just invokes iscc:
https://github.com/MCOfficer/bitwarden-autotype/blob/8778fae401a0de7969195dfc2c72cd7f8cd8ae39/.github/workflows/build.yml#L30


error: TypeError: f.findLast is not a function

APKMirror Downloader v1.0.6
TypeError: f.findLast is not a function
    at C:\Users\Yakov\Downloads\apkmirror-downloader\bin\apkmd.cjs:31:395
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
Error: Could not find stable version
    at pb (C:\Users\Yakov\Downloads\apkmirror-downloader\bin\apkmd.cjs:29:1304)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async C:\Users\Yakov\Downloads\apkmirror-downloader\bin\apkmd.cjs:31:225

Press any key to exit ...

I used the default config provided in the README.md and running nodeJS version 16.13.0.
I appreciate you for maintaining.

I suspect that upgrading node version would fix this. Because .findLast and other array method were added in node 18.
Also you could replace arr.findLast with equivalent code like arr[arr.length - 1] for now.

That seems to be the issue, thank you!


question

can you list the differences between this and echox?

You can check them out by comparing the source! Join the discord for more support. discord.gg/dptDHp9p9k


Twitch badges failing

Are you experiencing an issue with...

[x] shields.io

:beetle: Description
Looks like our twitch badges are failing in production and in the service tests. I grabbed the TWITCH_CLIENT_ID and TWITCH_CLIENT_SECRET from heroku and ran the tests locally. The error is:
{"error":"Unauthorized","status":401,"message":"Client ID and OAuth token do not match"}

It looks like CircleCI is using different tokens than production, but I guess the failure is the same.
:link: Link to the badge
examples:
 - http://img.shields.io/twitch/status/andyonthewings
 - http://img.shields.io/twitch/status/noopkat
:bulb: Possible Solution
Clearly we need new tokens. @PyvesB I think the tokens are linked to your account, but also if these need rotating occasionally, maybe we need a better solution to this?

Thanks for reporting this.
Looking back at the Circle CI history, these tests have been failing for about a month, since May 12th.
I've just logged into the Twitch developer console, there's nothing that suggests that the credentials have expired. In particular, the ones on the CI are recent and were generated in March 2020 as part of #4061.
Generating new credentials does not help and I can't see any code changes on our side that could have obviously broken these badges.
Digging through some of the twitch announcements, I noticed this article. They made changes to their authentication mechanism, with a permanent switch on May 11th, which corresponds timing-wise. I haven't fully read the details, but I'm under the impression that we have to switch from query string auth to an Authorization: Bearer. I'll do some more digging and experimentation later in the week. @andyli you originally contributed these badges, feel free to help out as well. 😉
Note that I've double checked my emails, including the spam folder, I was not warned about such a change. No notification or anything similar on my Twitch account either. An obscure post on their own forums and scheduling 3 small windows where they failed non complying requests seems like an extremely poor way of warning developers about breaking changes to their API.

I haven't fully read the details, but I'm under the impression that we have to switch from query string auth to an Authorization: Bearer header.

Ignore this part of the analysis, it's wrong: I've read through our Twitch code again and reminded myself how it works, we are doing the right thing at first glance. However, I think there's a bug in the details of the code, we're not actually setting the Client-ID as required. Will investigate further and try to come up with a fix.


Ruby 3

Can't replicate failures on localhost 😕

Coverage increased (+69.7%) to 69.697% when pulling b3af3ca40f871c9ca0eb548ea4c8c90f422571f7 on ruby-3 into 94c5d7b645d6121aabf1c559420d38e9bd0e2796 on master.

Coverage increased (+69.7%) to 69.697% when pulling b3af3ca40f871c9ca0eb548ea4c8c90f422571f7 on ruby-3 into 94c5d7b645d6121aabf1c559420d38e9bd0e2796 on master.

Replicated, just rm -rf coverage

Replicated, just rm -rf coverage

And the coverage is back

And the coverage is back


Cannot load such file -- jekyll-art-gallery-generator

I have configured it as in the README, but I get
Dependency Error: Yikes! It looks like you don't have jekyll-art-gallery-generator or one of its dependencies installed. In order to use Jekyll as currently configured, you'll need to install this gem. The full error message from Ruby is: 'cannot load such file -- jekyll-art-gallery-generator' If you run into trouble, you can find helpful resources at https://jekyllrb.com/help/!

Hmm...double check that it's installed correctly and in the right folder for jekyll to find it


[Off-canvas] - Opening 1 off-canvas causes the other off-canvas opening too

When you have multiple Off-canvas components at 1 page, and tries to open/close 1 of them, all the others are opening/closing.
https://jsfiddle.net/6Lwz76hv/

@Silvester20 your syntax is messed up. You can only have one set of body tags. You want your page's main content to all be inside of the .off-canvas-content element. You also can only have one off-canvas menu per side. Your example uses two on the left.
Here is what your syntax should look like:
https://jsfiddle.net/qf4o4hdm/

@colin-marshall Thanks for the reply.
I just did a quick copy/paste from the docs so that's why the 2 body tags. Even when this is not valid W3 it doesn't matter so much for this example.
What you did is 1 off-canvas with 2 menu's inside it.
What I want is 2 off-canvas serperated; 1 off-canvas for a public chat and 1 off-canvas for a private chat in my webapplication.
So when I open the off-canvas related to the public chat, right now the off-canvas of the private chat is opening too. Here lies the bug(?).

I believe @colin-marshall has the correct solution though I'm not sure what you mean when you say you need 2 off-canvas separated.
What I would imagine you're asking for is an off canvas menu showing private chats and an off canvas menu showing public chats. Whichever is clicked, would reflect that data in the body content visible to the user. If this is the case, you'll need @colin-marshall's solution with a little more javascript manipulation on your part.
If this is still an outstanding issue, can you please provide some more details on what exactly you are trying to accomplish? This would help the community get a better understanding on how to correctly answer your question.

@mrtpain Thanks for the reply. This is still an issue of the latest version.

What I would imagine you're asking for is an off canvas menu showing private chats and an off canvas menu showing public chats. Whichever is clicked, would reflect that data in the body content visible to the user.

This is correct. However, you are talking here about 2 menu's

off canvas menu for public chat

off canvas menu for private chat

Want I mean is 2 off-canvas components (not menu's).
So 1 off-canvas component for public chat and 1 off canvas component for private chat at 1 page. The fiddle in my first post demonstrates exactly what I mean.

@Silvester20 I am also confused by what you mean here. As @mrtpain suggested I think what you need to do is use one off canvas and use some additional javascript to change the content depending on what button (public/private chat) is clicked. If you provide us with a visual of what you are trying to accomplish that would help us get on the same page.
I'm going to close this for now. Feel free to reopen and respond with the additional information I have requested and we will do our best to help you if you're still trying to tackle this. Thanks!

@Silvester20 could you please provide us with a visual of what you are trying to accomplish? Just want to make sure I fully understand what you're trying to do. Thanks!

One more thing, did you notice in my first response I said that there can only be one off-canvas area per side? If you are trying to make two different off-canvas areas slide out from the same side (the left in your example) it's not going to work. You would be better off having two buttons that trigger the same off-canvas area but have each button trigger a change in the content that is shown in the off-canvas area when it is clicked. I think that is basically what @mrtpain was suggesting.

what is the difference between an off-canvas component and a menu?

With "component" I mean a instance of the off-canvas, like
var component_1 =  new Foundation.OffCanvas($('#offcanvas1'));
var component_2 =  new Foundation.OffCanvas($('#offcanvas2'));
I appreciate the help but you guys still don't understand me, maybe it's because my English is not good enough haha :)
Imagine the following scenario: A admin can manage some account settings, like editing a password or editing the account details of registered users. These settings are divided into tabs using Foundation Tabs. And to make the settings responsive I decided to put the tabs inside a Foundation Off-canvas.
To manage the users I am using Foundation Reveal. But inside this reveal I am using a new Off-canvas to edit some user information. But when I am toggle this Offcanvas (in reveal), the first Off-canvas is toggled too!
Well, maybe this sounds a little bit confusing so a fiddle should help.
Also maybe @kball, @Owlbertz or @andycochran can take a look?

I'm pretty sure an off-canvas menu inside an active reveal is not possible/not the best UX decision. I get what you're thing to accomplish and still say, this is not possible without writing some additional JavaScript yourself.

Off-canvas is not intended to be nested inside other components.
The docs explain how to have Multiple Menus, but that is poorly named. We should not use the word "menu" when talking about off-canvas. This should maybe say "Multiple Off-canvases" or "Multiple Off-canvas Panels." /cc @brettsmason (for his refactor of Off-canvas)
I'm going to close this, as it's not a bug.

as it's not a bug.

Sure it is. This is what Foundation code is saying right now using data-toggle="myCanvasId"

Hey, somebody passend an element id myCanvasId. Actually I have to use this myCanvasId but screw it, I will search the whole page for any off-Canvas component and affect them all instead the given one only.

I have fixed this bug by meself anyway

@ghost it is not a bug because what you are trying to accomplish is not currently how off-canvas was intended to be used.
As @andycochran mentioned, @brettsmason is refactoring off-canvas for v6.3 and this might be something you could accomplish with it, but he would have to speak on that.

@ghost I'd like to see if I can help with this. As has been said I have reworked off canvas and I think it's now a lot more flexible. I don't fully understand your issue yet, but I'll try and take a look at your demo and see it can be done in the new version.

@ghost @colin-marshall @andycochran turns out this is pretty easy in the new version (assuming I understood your demo).
Take a look here: http://brettmason.co.uk/offcanvas/9093/
It's not exactly right, but should give you an idea that the off canvas panels should work how you want them to.

@brettsmason that is sweet! Nice work!


Add "percent processor time" to CPU collector

Currently, Percent Processor Time is in the Win32_PerfRawData_PerfOS_Processor struct but is not actually collected.  Is that something that can be added?  Judging by how the collector is laid out (I'm not super familiar with Go so I might be wrong) it looks to be a pretty simple addition.  Was it just omitted by accident in #26 ?

Meant to add-- According to the MSDN docs, Percent Processor Time is:

Percentage of time that the processor is executing a non-idle thread. This property was designed as a primary indicator of processor activity.

wmi_cpu_time_total should meet that need.

How so?  You either include the metric or have a continuous query or set of
queries to get a percentage instead of a number in the millions or billions
or more depending on uptime.  I understand the formula to do it but it's
hardly more efficient to not include it
On Jan 20, 2017 9:19 AM, "Brian Brazil"<EMAIL_ADDRESS>wrote:

wmi_cpu_time_total should meet that need.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/martinlindhe/wmi_exporter/issues/53#issuecomment-274082465,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB5zjs3myBtbhVMsMI6hVHPZAvoKOBSiks5rUMJlgaJpZM4LpWO_
.

You need queries either way to get to this data.

I'm not sure I understand your point.  I know they would both need a queries, my question is about the relative cost on the collector vs on the prometheus instance.  Could you give an example of the queries I'd need either way, in case I'm missing something about the complexity of getting an instantaneous usage percentage from at least 4 different metrics?

I don't have a Windows system to hand, but it'd be similar to Linux: https://www.robustperception.io/understanding-machine-cpu-usage/

Alright, I followed that and am now getting negative CPU usage:

Also, 100 - (avg (irate(wmi_cpu_time_total{mode="idle"}[5m])) * 100) vs avg (wmi_cpu_percentage) is what I was talking about w.r.t. relative complexity.

One potential idea: Do you have more than one server with wmi_exporter installed? You are averaging over all instances, rather than a specific one. For comparison, I have 1 - avg(irate(wmi_cpu_time_total{instance="$Server", mode="idle"}[2m])) in our dashboards (with Grafana templating filling in $Server).
Could you check what data is in prometheus for just wmi_cpu_time_total{mode="idle"} and share?

Nope, only one instance.

Ok, thanks. What are the specs of the instance - number of cores and version of Windows?

W7 enterprise, 40 cores.  (I know.. I don't ask questions when I get something like this where I work)

Hm, ok, I'll admit I haven't got anything like that to test on ;) But could you try to run the query on prometheus so we can look at the raw data?

I would, if my "graph" panel worked.  I updated recently, v1.4.1, and the "add graph" hasn't worked for me since.

Basically yes, just to see if the data seems unreasonable in itself (ie we are not interpreting the wmi output correctly). Doesn't need to be in the graph mode either if that is acting up, just raw query output might be enough.

yeah both the graphs and table isn't working properly at all (don't even have a query input) so here's the grafana table equivalent, columns are cores:

complete with ascii-sorted columns because no zero padding :D

Ah, makes sense now:
https://github.com/prometheus/prometheus/issues/2265

Hm, nothing obvious there. Could you try modifying your Grafana query like this 100 - (avg (irate(wmi_cpu_time_total{mode="idle"}[5m])) by (core) * 100) and using bar mode instead of lines? I'm expecting to see that a single core is responsible for the weird jumps.

avg (wmi_cpu_percentage)

This also requires a rate, as that's how the raw data is exposed.

@carlpett it's actually a bunch of them, weirdly enough:

@brian-brazil how is it exposed as a rate?  From the docs I was under the impression that it was a 0-100% depending on wmi's own sampling rate, using a similar process to the one you linked to above.

All WMI stats come with equations to use to convert them into what their actual name is. So in the case of a CPU percentage that's going to mean taking a rate.

So it's returning a cumulative percentage...?  I'm sorry, I understand how
the rate applies on the query side for cpu total time to convert to an
instant value, but not how it applies for an instantaneous value to stay an
instantaneous value.  The total time returns a cumulative counter, which
matches the name, why would the present utilization be different?
On Jan 20, 2017 12:50 PM, "Brian Brazil"<EMAIL_ADDRESS>wrote:

All WMI stats come with equations to use to convert them into what their
actual name is. So in the case of a CPU percentage that's going to mean
taking a rate.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/martinlindhe/wmi_exporter/issues/53#issuecomment-274134925,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB5zjsd7AhmJfMvY3pLiaFb4gxNmTTGEks5rUPPYgaJpZM4LpWO_
.

The total time returns a cumulative counter, which
matches the name, why would the present utilization be different?

It's no different. The percent utilisation is exposed as a cumulative counter.

Oh, so it doesn't match its name where the former does.  I understand
now... Sorry for the confusion.  In that case, yeah, it's a pretty useless
stat and only saves a single math operation and not even an aggregator.
On Jan 20, 2017 1:01 PM, "Brian Brazil"<EMAIL_ADDRESS>wrote:
The total time returns a cumulative counter, which
matches the name, why would the present utilization be different?
It's no different. The percent utilisation is exposed as a cumulative
counter.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/martinlindhe/wmi_exporter/issues/53#issuecomment-274137635,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB5zjkECxRt6kNidTZ424XnO_Fvr9yguks5rUPZhgaJpZM4LpWO_
.

Regarding the graph, I'm unsure how to interpret this.
@brian-brazil - could this be an artifact from the irate of some sort? I've seen similar things when using deriv and predict_linear when there are very large changes in a value between two scrapes, which would be expected, but I'm not sure how irate could cause this? Especially the negative spike preceeding the positive one.

could this be an artifact from the irate of some sort?

It's not unusual to see oddness of this sort due to various races.

If the entirety of the CPU raw data collector is basically cumulative counters, then this issue doesn't propose anything new except what amounts to a macro for some simple math.

Hi @pokemane I am new to windows and due to mssql dependency I have to setup wmi exporter. How did you set up the rules ? I am not able to understand what to do from this discussion. can you please help ? or can you point to example rules for wmi or mssql ?


Can not replication images to hub.docker.com

version:1.2.2
When i replication to hub.docker.com,it's failed with these logs.
I was confuse with "source URL: http://registry:5000",is this cause the error?
2017-11-01T10:59:29Z [INFO] initializing: repository: library/hello, tags: [], source URL: http://registry:5000, destination URL: https://hub.docker.com/dockerhubusername, insecure: false, destination user: dockerhubusername
2017-11-01T10:59:30Z [INFO] initialization completed: project: library, repository: library/hello, tags: [latest v1], source URL: http://registry:5000, destination URL: https://hub.docker.com/dockerhubusername, insecure: false, destination user: dockerhubusername
2017-11-01T10:59:33Z [INFO] project library is created on https://hub.docker.com/dockerhubusername with user dockerhubusername
2017-11-01T10:59:33Z [INFO] manifest of library/hello:latest pulled successfully from http://registry:5000: sha256:467af83b77759844196762fc8d9e981a0c3d4fa3c59d76445b7e35a6ee5ca916
2017-11-01T10:59:33Z [INFO] all blobs of library/hello:latest from http://registry:5000: [sha256:05a3bd381fc2470695a35f230afefd7bf978b566253199c4ae5cc96fafa29b37 sha256:0e8205f2763136ae97a4e1d00e2eb939a97e01c0cee8b3f2fa4bd593d16e779f]
2017-11-01T10:59:35Z [INFO] blob sha256:05a3bd381fc2470695a35f230afefd7bf978b566253199c4ae5cc96fafa29b37 of library/hello:latest already exists in https://hub.docker.com/dockerhubusername
2017-11-01T10:59:36Z [INFO] blob sha256:0e8205f2763136ae97a4e1d00e2eb939a97e01c0cee8b3f2fa4bd593d16e779f of library/hello:latest already exists in https://hub.docker.com/dockerhubusername
2017-11-01T10:59:36Z [INFO] blobs of library/hello:latest need to be transferred to https://hub.docker.com/dockerhubusername: []
2017-11-01T10:59:36Z [INFO] manifest of library/hello:latest exists on source registry http://registry:5000, continue manifest pushing
2017-11-01T10:59:37Z [ERROR] [transfer.go:491]: an error occurred while pushing manifest of library/hello:latest to https://hub.docker.com/dockerhubusername : 301 Moved Permanently. Redirecting to /dockerhubusername/v2/library/hello/manifests/latest/

In $harboe_home/common/config/nginx/nginx.conf
I can not see Https config,excpt
server {
listen 80;
#server_name harbordomain.com;
return 301 https://$host$request_uri;
}

Configuration file of Harbor
#The IP address or hostname to access admin UI and registry service.
#DO NOT use localhost or <IP_ADDRESS>, because Harbor needs to be accessed by external clients.
hostname = dockerhub.mycompany.com
#The protocol for accessing the UI and token/notification service, by default it is http.
#It can be set to https if ssl is enabled on nginx.
ui_url_protocol = https
#The password for the root user of mysql db, change this before any production use.
db_password = root123
#Maximum number of job workers in job service
max_job_workers = 3
#Determine whether or not to generate certificate for the registry's token.
#If the value is on, the prepare script creates new root cert and private key
#for generating token to access the registry. If the value is off the default key/cert will be used.
#This flag also controls the creation of the notary signer's cert.
customize_crt = on
#显示的认证及组织信息
crt_country = CN
crt_state = Jilin
crt_location = Changchun
crt_organization = jc
crt_organizationalunit = pr
crt_commonname = dockerhub.mycompany.com
crt_email =<EMAIL_ADDRESS>#The path of cert and key files for nginx, they are applied only the protocol is set to https
#ssl_cert = /data/cert/server.crt
#ssl_cert_key = /data/cert/server.key
ssl_cert = /data/cert/dockerhub.mycompany.com.crt
ssl_cert_key = /data/cert/dockerhub.mycompany.com.key
#The path of secretkey storage
secretkey_path = /data
#Admiral's url, comment this attribute, or set its value to NA when Harbor is standalone
admiral_url = NA
#The password of the Clair's postgres database, only effective when Harbor is deployed with Clair.
#Please update it before deployment, subsequent update will cause Clair's API server and Harbor unable to access Clair's database.
clair_db_password = password
#NOTES: The properties between BEGIN INITIAL PROPERTIES and END INITIAL PROPERTIES
#only take effect in the first boot, the subsequent changes of these properties
#should be performed on web ui
#BEGIN INITIAL PROPERTIES
#Email account settings for sending out password resetting emails.
#Email server uses the given username and password to authenticate on TLS connections to host and act as identity.
#Identity left blank to act as username.
email_identity =
email_server = smtp.mydomain.com
email_server_port = 25
email_username =<EMAIL_ADDRESS>email_password = abc
email_from = admin<EMAIL_ADDRESS>email_ssl = false
##The initial password of Harbor admin, only works for the first time when Harbor starts.
#It has no effect after the first launch of Harbor.
#Change the admin password from UI after launching Harbor.
harbor_admin_password = Harbor12345
##By default the auth mode is db_auth, i.e. the credentials are stored in a local database.
#Set it to ldap_auth if you want to verify a user's credentials against an LDAP server.
auth_mode = db_auth
#The url for an ldap endpoint.
ldap_url = ldaps://ldap.mydomain.com
#A user's DN who has the permission to search the LDAP/AD server.
#If your LDAP/AD server does not support anonymous search, you should configure this DN and ldap_search_pwd.
#ldap_searchdn = uid=searchuser,ou=people,dc=mydomain,dc=com
#the password of the ldap_searchdn
#ldap_search_pwd = password
#The base DN from which to look up a user in LDAP/AD
ldap_basedn = ou=people,dc=mydomain,dc=com
#Search filter for LDAP/AD, make sure the syntax of the filter is correct.
#ldap_filter = (objectClass=person)
The attribute used in a search to match a user, it could be uid, cn, email, sAMAccountName or other attributes depending on your LDAP/AD
ldap_uid = uid
#the scope to search for users, 1-LDAP_SCOPE_BASE, 2-LDAP_SCOPE_ONELEVEL, 3-LDAP_SCOPE_SUBTREE
ldap_scope = 3
#Timeout (in seconds)  when connecting to an LDAP Server. The default value (and most reasonable) is 5 seconds.
ldap_timeout = 5
#Turn on or off the self-registration feature
self_registration = on
#The expiration time (in minute) of token created by token service, default is 30 minutes
token_expiration = 30
#The flag to control what users have permission to create projects
#The default value "everyone" allows everyone to creates a project.
#Set to "adminonly" so that only admin user can create project.
project_creation_restriction = everyone
#Determine whether the job service should verify the ssl cert when it connects to a remote registry.
#Set this flag to off when the remote registry uses a self-signed or untrusted certificate.
verify_remote_cert = on
#END INITIAL PROPERTIES
#############

@cooljsz
Replicating images from Harbor to dockerhub is not supported yet, but it's already in planning. Thanks.

thx,我这么蹩脚的英文都看懂了,真难为了

The above message indicated in the planning is support for replicating Harbor to dockerhub.   Will this include replicating to AWS registry?

Are there any updates with supporting this in Harbor? If there is some progress, will it include just DockerHub, or does it include Amazon ECR as well?


76 make query editor vertically resizable

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

Visual Studio Marketplace
GitHub release

Your semantic-release bot :package::rocket:


Closes #1178

What changes are proposed in this pull request?
Hopefully closes #1178 with the help of Brian's tip.
How is this patch tested? If it is not, please explain why.
Unfortunately, I wasn't able to test it.
Release Notes
Is this a user-facing change that should be mentioned in the release notes?

[ ] No. You can skip the rest of this section.
[ ] Yes. Give a description of this change to be included in the release
notes for FiftyOne users.

(Details in 1-2 sentences. You can just refer to another PR with a description
if this PR is part of a larger change.)
What areas of FiftyOne does this PR affect?

[ ] App: FiftyOne application changes
[x] Build: Build and test infrastructure changes
[x] Core: Core fiftyone Python library changes
[ ] Documentation: FiftyOne documentation changes
[ ] Other

Thanks @ErfanTagh :smile:
Can you fix the line deletsions, etc. from https://github.com/voxel51/fiftyone/pull/1210/commits/cf8aec4be88247f035b21d445807f32488ff7d34?

Thanks @ErfanTagh
Can you fix the line deletions, etc. from cf8aec4?

Thanks @ErfanTagh
Can you fix the line deletions, etc. from cf8aec4?

I'm not sure what exactly caused this but I tried to fix it now  @benjaminpkane


Partial rerender is not stable

Re-rendering a dynamic partial always results in the DOM being invalidated, even when the partial is the same. Working on fixing this.

Looks like this can be closed?


Bump up protobuf library version

For memorialization: we discussed bumping to latest Google Protobuf minor version (3.21.x) but decided to delay that to a normal release cycle and just move to 3.19.5 in this case since 3.19 is still supported and .5 resolves the issue


docs(proposal): new artifacts distribution proposal

What type of PR is this?
/kind documentation

Any specific area of the project related to this PR?
/area proposals
What this PR does / why we need it:
This proposal aims to formalize the revamped way adopted by Falcosecurity to distribute artifacts. In fact, it reflects some topics widely discussed in the last few months, plus suggestions for some obvious action items to move forward.
Furthermore, most of the points reflect some already ongoing efforts, such as the revamp of falcoctl.
Any feedback is appreciated
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
/milestone 0.34.0
Does this PR introduce a user-facing change?:

NONE

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

Needs approval from an approver in each of these files:

OWNERS [leogr]

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

Dear @falcosecurity/core-maintainers
As discussed during the last community call, here's my proposal :)
So, PTAL :pray:
:hugs:

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

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,leogr]

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

/hold

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

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP,leogr]

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

As discussed in the current community call, it seems to me that nobody disagrees with this proposal, and we would like to deliver some action items within Falco 0.34, so
/hold cancel
:partying_face:


Using columnOrder API issue

Expected behavior and actual behavior.
Hi @TonyGermaneri,
I'm still not able to use grid.columnOrder to change the columnOrder either on initialization or on event listening using v0.21.2.
The getter seems to work, however the setter does not. Using the setter grid.columnOrder = [2, 1, 0];, I get the following console error Uncaught ReferenceError: s is not defined.
Below is my test.html code to reproduce the problem.
Steps to reproduce the problem.
<!DOCTYPE html>

Specifications like the version of the project, operating system, or hardware.
  I'll get it fixed right now. v0.21.3 will have the fix, I'll publish it in a moment.

Looks like the API is working with the fix. Thanks @TonyGermaneri!


Remove the ExceptT stack from the Neovim type

This should adress the following issues:

Needless exception stack as IO already supports exceptions
Since runNeovim now catches all exceptions, the plugin provider should not
crash anymore (and possibly cause neovim to hang)

Looks good :+1:

Do your test plugins still work with this?

Note that we do not handle stateful functions here. If an exception is thrown within such a function, the event loop is not going to read a request anymore and neovim will likely start to hang again. We have to restart the function's "loop" or unregister all affected functions from the event handler.

Do your test plugins still work with this?

It works, but since we lost MonadError String instance with this patch I need to throw an IOException.

Note that we do not handle stateful functions here. If an exception is thrown within such a function, the event loop is not going to read a request anymore and neovim will likely start to hang again. We have to restart the function's "loop" or unregister all affected functions from the event handler.

Yeah, we should send an error response when that happens to prevent vim from hanging.


EMAIL-SENDER

EMAIL-SENDER JS PROJECT

Not the project related to the tech stack neither does this PR follow the guidelines of this repo!


Cannot get the RF rx5808 module to work

I recently ordered 4 rx5808 modules from Banggood and when they finally arrived and I removed the metal casing, the resistor that you explain how to remove is not installed from the beginning. I solder the wires following the diagram and cannot get the app to let me calibrate/change the frequency or pilot. What does the "Lipo Voltage" go into (Red lead on the diagram), and how might I go about fixing this issue?

Please follow the guide in the Troubleshooting section to resolve the app issue.
The LiPo Voltage wire should go to + of the lipo you're using to power the Chorus

Thank You @voroshkov, My Bluetooth baud rate wasn't set.


fails to connect to device

Hey again,
I've moved over to this npm package but I'm getting the following error:
no code my end has been changed and this works in the 'cap-esp-idf' package.
{code: "connect-error-1", errorMessage: "Failed to connect to device: PROV_123456 Optional(ESPProvision.ESPSessionError.noUsername)", message: "Failed to connect to device: PROV_123456 Optional(ESPProvision.ESPSessionError.noUsername)"}
'cap-esp-idf' has some other bugs, so would prefer to use this if it's supported

sorry, let me give you a little more on this, I'm using the connect method and sending a valid device name and proofOfPossession. I am using the ESP-S3 and the firmware is flashed with the wifi provisioning example supplied by Expressif. Please let me know if you'd like any more info

So, I'd need to see a lot of code to be able to help. Be sure the devicePrefix your're using matches the prefix configured in your firmware. In the example above PROV.

Without more info there's not much I can do


Search for a different line when parsing plan.

Fixes plan display for 0.15
Also works for 0.14

@alex-goncharov the pipeline is failing on the unit tests for the publish plan results tests. This looks to be caused by the sample response still having the old message. I think the files that contain the mocked response just need to be updated with what we expect from 0.14 and 0.15.
1) Scenario: publish plan results enabled # src/tests/features/publish-plan-results/publish-plan-results.feature:7
   ✔ Before # node_modules/cucumber-tsflow/src/binding-decorator.ts:79
   ✔ Given terraform exists # node_modules/cucumber-tsflow/src/binding-decorator.ts:187
   ✔ And terraform command is "plan" with options "-detailed-exitcode" # node_modules/cucumber-tsflow/src/binding-decorator.ts:187
   ✔ And running command "terraform plan -detailed-exitcode" returns successful result with exit code 2 and stdout from file "./src/tests/features/publish-plan-results/plan-output-with-adds-destroys-and-updates.txt" # node_modules/cucumber-tsflow/src/binding-decorator.ts:187
   ✔ And publish plan result is "test_deploy.tfplan" # node_modules/cucumber-tsflow/src/binding-decorator.ts:187
   ✔ When the terraform cli task is run # node_modules/cucumber-tsflow/src/binding-decorator.ts:197
   ✔ Then the terraform cli task is successful # node_modules/cucumber-tsflow/src/binding-decorator.ts:207
   ✖ And a plan named "test_deploy.tfplan" is published with the following content from file "./src/tests/features/publish-plan-results/plan-summary-with-adds-destroys-and-updates.txt" # node_modules/cucumber-tsflow/src/binding-decorator.ts:207
       AssertionError
           + expected - actual

           +An execution plan has been generated and is shown below.
           +Resource actions are indicated with the following symbols:
           +  + create
           +  ~ update in-place
           +  - destroy
           +
            Terraform will perform the following actions:

              # azurerm_resource_group.test_01 will be updated in-place
              ~ resource "azurerm_resource_group" "test_01" {

           at TerraformSteps.planDetailsAreAttachedWithTheFollowingContentFromFile (/home/vsts/work/1/s/tasks/terraform-cli/src/tests/steps/task.steps.ts:2:9824)
           at World.bindingFunc (/home/vsts/work/1/s/tasks/terraform-cli/node_modules/cucumber-tsflow/src/binding-decorator.ts:172:22)
   ✔ After # node_modules/cucumber-tsflow/src/binding-decorator.ts:251
   ✔ After # node_modules/cucumber-tsflow/src/binding-decorator.ts:90

@charleszipp unit-test wise this should be gtg, but it seems smoke tests weren't able to install published beta task for whatever reason. Can you please check and if you're ok with test changes release this one? Thanks!
// cc: @alex-goncharov

@charleszipp unit-test wise this should be gtg, but it seems smoke tests weren't able to install published beta task for whatever reason. Can you please check and if you're ok with test changes release this one? Thanks!
// cc: @alex-goncharov

Rerunning the pipeline. I thought I had this fixed with the wait/polling step but, seems recently to be happening frequently :(. If i can get the pipeline through I will push this through. Thanks for updating the tests!


Add drag selected text in richtextbox.

Please add drag   selected text in richtext field.
And add mode drag  with down button Ctrl for copy selected text.

Your request it's not possible because the RichEditBox control that Notepads uses doesn't support such feature. Why? No idea, but you're better off asking for that in the Microsoft Xaml UI repo.

https://docs.microsoft.com/en-us/windows/apps/winui/winui3/
The site says WinUI 3 0.5 Preview. That this is the first stable version from the pre-release versions.
@XamDR
Do you think it is possible to already use this version in Notepads?
especially since only a small number of elements are used for Notepad.
The second case. At the moment, Notepads is already a stable and moderately working program. All new Notepads issues are related to the WinUI library.
Which suggests that you can already start using the new WinUI 3 0.5 because our task is to solve the problems of WinUI 2. And the version that is built on WinUI is already stable and functional to the best of its limitations.

https://docs.microsoft.com/en-us/windows/apps/winui/winui3/
The site says WinUI 3 0.5 Preview. That this is the first stable version from the pre-release versions.
@XamDR
Do you think it is possible to already use this version in Notepads?
especially since only a small number of elements are used for Notepad.

No it isn't possible to use WinUI 3 in Notepads, WinUI 3 at its current state lacks major functionality like acrylic and reveal which Notepads uses. Besides that Notepads uses WCT which isn't available in WinUI 3 for Notepads.

The second case. At the moment, Notepads is already a stable and moderately working program. All new Notepads issues are related to the WinUI library.
Which suggests that you can already start using the new WinUI 3 0.5 because our task is to solve the problems of WinUI 2. And the version that is built on WinUI is already stable and functional to the best of its limitations.
That is, within the framework of WinUI2, we already have nothing to solve, we will not be able to do anything new that needs to be done.

Notepads doesn't use WinUI 2 yet as it introduces some new bugs. As for WinUI 3 removing limitations of WinUI 2, that simply isn't true. WinUI 3 at its current state lacks many major functionality and also rigged with same limitations and bugs as WinUI 2. My guess is it would likely take MS 3 or 4 years to provide everything WinUI 2 offers and get rid of all its bugs and limitations.


New package: CompileBot v1.0.0

Registering package: CompileBot
Repository: https://github.com/aminya/CompileBot.jl
Created by: @aminya
Version: v1.0.0
Commit: 28f05eafd7c0fd329fd40895b214f0ceacbcf757
Reviewed by: @aminya
Reference: https://github.com/aminya/CompileBot.jl/commit/28f05eafd7c0fd329fd40895b214f0ceacbcf757#commitcomment-42882955

Why are you renaming the package? Doesn't that only cause trouble for users?

I explained here:
https://github.com/timholy/SnoopCompile.jl/pull/140#issuecomment-702272669
In short:

SnoopCompileBot was not a package that other packages would depend on. It is a package to be used on its own. Thus, changing the name does not have any disadvantages.
I had an idea for creating a bot for PackageCompiler. Now, that the repository is no longer a sub-directory of SnoopCompile, I can extend its purpose.

I did not just change the name without providing deprecation messages, so those who use the old version will be informed about this change:
https://github.com/timholy/SnoopCompile.jl/pull/140

@fredrikekre Is there anything else? The registration of this package has become a burden for me. I want to continue developing, but I can't.

@fredrikekre @DilumAluthge I am still waiting for this

You need to edit all of your comments to include the text [noblock]. Otherwise AutoMerge will not merge it.


Add yarn.lock file

Push the yarn.lock file to the repo leading to faster install times and better version control of the installed packages with yarn.

The developer instructions and workflows seem to use npm install, so that seems to be the preferred package manager.

https://github.com/corona-warn-app/cwa-website/blob/14ef9e3a6eef405785b2a29715b5a86a8a0a05b4/README.md#L35
https://github.com/corona-warn-app/cwa-website/blob/b1b3828cb90f0ba7fba587692398a9e8463c0c6d/.github/workflows/deploy-master.yml#L24
https://github.com/corona-warn-app/cwa-website/blob/1e2af038bb42e390632fff62f272992e7fc9acb0/.github/workflows/build.yml#L24

➡ Add package-lock.json instead of yarn.lock?

Sounds reasonable,
but it is listed in .gitignore right now...
https://github.com/corona-warn-app/cwa-website/blob/3849875c7db2407bdf0f76ad8d08c14750800877/.gitignore#L43
should i create a PR to remove it?

Yeah, you should always push those lock files

Yeah, you should always push those lock files

Agreed. It could also be interesting to use npm ci for the GitHub workflows. One idea of this is to ensure that ci/cd workflows use the same package versions of all (also transitive) dependencies as a developer does on their local machine.


TypeError: The value of a feed cannot be a tf.Tensor object

Hi, thanks before you see it.
I got an error when i run tensorflow model of cnn.
TypeError: The value of a feed cannot be a tf.Tensor object. Acceptable feed values include Python scalars, strings, lists, or numpy ndarrays.
This error tell me that I cant feed tensor,but i dont know how to modify my code .Any one can tell me how can i modify my code ?
Here is my code.
https://github.com/xuhuapeng/cifar/blob/master/cnn_1.py
https://github.com/xuhuapeng/cifar
Thanks very much.

Once you have launched a sess, you can use your_tensor.eval(session=sess) or sess.run(your_tensor) to get you feed tensor into the format of numpy.array and then feed it to your placeholder.

@zakizhou
I have change the code to :
 sess.run(train_step,feed_dict={x: images, y_: labels, keep_prob: 0.5})
But it give the same error as I pointed out before.
Also I had tried to  change the code to
sess.run(train_step,feed_dict={x: images.eval(), y_: labels.eval(),keep_prob:0.5}).
But this will lead the program to execute forever. In my point the images.eval() step can't done forever.
Can you see my code to give me some another advice ?? Thank you so much.

@xuhuapeng
The cnn_1.py in your git repo is modified from the original file in the official tutorial of  TF right?  If you have a close look at its original code, you will find no placeholders in it which means the input of the model is the second way(readings from files not feeding) according to  this, as a result, you can't feed any data to this model.
Since cifar10 is not a small dataset like mnist, placeholder is not suitable for this kind of large datasets, personally I think  readding inputs from files is the best way to train it. If you still want to use placeholder  you'd better write a next_batch method in mnist for cifar so as to feed the placeholder

yeah.I will try you suggestion.
One more question,please be patient with me. Why  I modify my code to  sess.run(train_step,feed_dict={x: images.eval(), y_: labels.eval(),keep_prob:0.5}). the program won't be stop? Doing the .eval() operation forever?

see this Creating threads to prefetch using QueueRunner objects section.
You delete the  tf.train.start_queue_runners ,coord and theads  from the training code which will start the input queue that the reader  need to read from, so you reader will hang there until the queue starts.

Thanks very much.
I had saw the link you  gave me, but there are lots thing I can't understand due to my poor English,. Maybe I should see it again.  👍 @zakizhou

You are welcome,byt,咱们中国人还是用中文交流比较好> < @xuhuapeng

Thanks @zakizhou for helping to resolve this problem for @xuhuapeng. If you have questions in the future, we recommend Stack Overflow for community support. We try to keep this issue tracker focused on bugs and feature requests.

@zakizhou 你好，我也遇到了相似的问题，系统刚开始和我说，输入必须是一个tensor，另一方面系统又和我说，feed不能是tensor，能不能麻烦大神帮我看一下，代码并不复杂。感谢
https://github.com/RayAnteku/tensorflow

@zakizhou 你好，根据你上边的回答我对我的程序做了修改，但程序还是一直阻塞，能办忙看一下代码吗，不胜感激，你github上的邮箱发不过去邮件，我的邮箱1107404430@qq.com。https://github.com/Tony-Hou/Img_classification.git

@RayAnteku , Do you solve this problem? I have the same error with you . If you did,please telll me how.


Update SnapshotTesting to v1.10.0

This change closes #105.
I have not updated the CocoaPods specification as SnapshotTesting has dropped support for it. Rather than removing support for CocoaPods from this library, I have opted to simply not include the new update there. This means CocoaPod users will not be able to update to v1.10.0, but can at least continue to use the last functioning version. A separate conversation may be worth having around keeping CocoaPods support going on as the general community is heading in a direction without it.
This is a breaking change for SPM users, as it does drop support for iOS 12 inline with the changes upstream. Again, this does not impact CocoaPod users.

Since this is now basically just bumping the deployment target to iOS 13, I think we should rescope this PR to essentially be the "drop iOS 12 support" change. That way we keep consistency across the project with what we support. This consists of:

Bump SPM deployment target to 13
Bump CP deployment target to 13
Bump sample app deployment target to 13
Remove the iOS 12 build job (and associated scripts/config)
Update requirements in README
Remove iOS 12-specific logic for matching VO behavior (I think there's only one case in UIAccessibility+SnapshotAdditions.swift)

Does that sound reasonable?

I think yes and no.
Right now this pull request is a blocker for numerous projects as folks are currently unable to use the latest version of SnapshotTesting if they also use AccessibilitySnapshot. There are important patches upstream so there is really a time critical aspect of this change.
Additionally, SnapshotTesting upstream has dropped support for CocoaPods. Bumping the CP deployment target to 13 is essentially pointless - rather the next natural step is to completely remove support for CocoaPods. This isn't a small piece of work though as it requires significant changes to the sample app and all of the CI scripts (it was where I first started).
I still think (in my opinion) this pull request is valuable on it's own in order to unblock teams who want/need to utilise the latest SnapshotTesting library. I then think there needs to be clarity on the next broader steps, in essence are you ready to drop CP, which can be handled as a separate PR.
Unrelated but the README states "We do not currently support AccessibilitySnapshot and iOSSnapshotTestCase through Swift Package Manager." which has been wrong for a while 😛

Apologies for the delay on this. I think we're good to merge this PR as-is (although I am going to rename it to better reflect what it's changing now) and follow up with the other steps I mentioned above.

rather the next natural step is to completely remove support for CocoaPods

I disagree here. Many folks are still using CocoaPods and we should continue to support it while we can. I suspect SnapshotTesting dropping CP means many of those developers will migrate to iOSSnapshotTestCase, but I don't think there's any reason not to support using AccessibilitySnapshot with an older version of SnapshotTesting through CP for now.

the README states "We do not currently support AccessibilitySnapshot and iOSSnapshotTestCase through Swift Package Manager." which has been wrong for a while 😛

🤦♂ Oops. I really need CI for my READMEs. #110


Add support for ros1_bridge

Description
As a ROS2 user, I should be able to run CI on packages depending on ros1_bridge.
ros1_bridge is a corner case because it depends on ROS1 and ROS2.
We need also to iron out what are the combinations of distros we want to support (kinetic, melodic, src) x (dashing, eloquent, source)...
Test Plan

End to end integration test

Documentation Plan

README.md update

Release Plan

Re-release the impact packages

Acceptance Criteria

[ ] Code has been implemented, reviewed and merged.
[ ] Test plan has been completed
[ ] Release plan has been completed

Once all above items are checked, this story can be moved to done.
Resources

ros1_bridge

This is blocked on getting ROS1 support for this.

https://github.com/ros2/ros1_bridge/pull/232

@zmichaels11  is this finished?


[Breaking Change][jvm-package] Rewrite XGBoost JVM Package

Reason
The XGBoost JVM package is the Java language binding for the XGBoost library. It is supposed to be a lightweight, thin wrapper around XGBoost. However, the current XGBoost JVM implementation is quite heavy-weight. For example, it groups the dataset using RDD for ranking, implements ranking within the XGBoostRegressor, samples the dataset for training and testing, and zips the training and evaluation datasets, duplicates some code usage. All of these additional features make the XGBoost JVM codebase difficult to read and maintain. Additionally, it is missing support for the latest XGBoost parameters and does not properly handle dense/sparse data usage.
Goal

[ ] Create a new XGBoostRanker for ranking problem.
[x] Code reusing.
[x] Support DART booster
[x]  Remove "grouping" for ranking problem.
[x]  Remove the trainTestRatio and its implementation
[x]  Remove "zip" train and eval dataset (add a new Boolean validation column to indicate if the instance is for training or for evaluating)
[x]  Catch up the latest parameters.
[x]  Add setter/getter for all parameters.
[x]  Support XGBoost style parameters when defining the parameters, (Like final val baseScore = new DoubleParam(this, "base_score", "The initial )
[ ]  Support dense/sparse when the input is vector type
[ ]   Support array input for both CPU and GPU
[ ]   Support columnar input for CPU ???
[x] Remove the way linking the xgboost4j/xgboost4j-spark to GPU
[ ] Use the existing fasterxml.jackson to handle the json.
[ ]  Avoid repartition if the number of input partittions is equal to num_workers.
[x] Fix surefire issue https://github.com/dmlc/xgboost/issues/10316
[x]  More scalastyle checking.
[x] Remove xgboost4j-gpu, move the implemenation into xgboost4j-spark-gpu directly.
[x] Remove the cudf dependency
[x]  Shade the xgboost4j/xgboost4j-spark into a single jar
[ ]  Shade the xgboost4j/xgboost4j-spark/xgboost4j-spark-gpu into a single jar
[ ] https://github.com/dmlc/xgboost/issues/10387
[x] Remove the RDD cache.
[ ] Fix https://github.com/dmlc/xgboost/issues/10200
[x] Fix https://github.com/dmlc/xgboost/issues/8294

These might be helpful:
https://github.com/dmlc/xgboost/blob/master/doc/contrib/featuremap.rst
https://github.com/dmlc/xgboost/blob/master/doc/contrib/community.rst


Implement collection of artes

Similarly to spells, the artes should be refactored into a respective JavaScript Object containing the properties:

Range: How far does this attack go?
TP Consumption: How much of the character's technical points does this move consume?
Damage: How much damage will this attack do?
Element: What is the element of this attack?

Implemented by commit 713995dda7756dcea119f482e795d7075a74e65d.


Any suggestions on how to prevent parent page from scrolling up?

I met a difficulty now, that is when I scroll down to the bottom of the table, the whole page is scrolling up if you continue to scroll down, can anyone help to tell me how you are preventing the parent page from scrolling?
I used this, and it does not work:
           jQuery(this.refs.myTable).on('mouseover', function() {                 document.body.style.overflow = 'hidden';             });             jQuery(this.refs.myTable).on('mouseout', function() {                 document.body.style.overflow = 'auto';             });
Any good suggestions?
Thanks

You can try this solution: http://jsfiddle.net/troyalford/4wrxq/4/

No, not working in fixed dataTable.
But I used a more simple solution:
_onScrollStartHanlder: function() {
document.body.style.overflow = 'hidden';
}
_onScrollEndHanlder: function() {
document.body.style.overflow = 'auto';
}
I believe biding event on onScrollStart/End is better solution for fixed
DataTable
On Mon, Aug 8, 2016 at 1:34 AM, nhducit<EMAIL_ADDRESS>wrote:

You can try this solution: http://jsfiddle.net/troyalford/4wrxq/4/
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/facebook/fixed-data-table/issues/438#issuecomment-238174116,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAN4c9f6cimSD4t7mWaED0t6WL4WLf-kks5qduoegaJpZM4JdXc0
.


Add test for 400ZR dB Q value streaming

Pull Request Functional Test Report for #2369 / a27ef0ee71fb1858d089f5d4800db458dc8ee516
No tests identified for validation.
Help


feat: Add NumberControl and NumericControl tests

Corresponding functional changes are in base branch
Add tests for NumberControl and related components/functions.

I'm wondering if we want to put all the test code in a test dir so it's easier not to bundle them within the build? Or perhaps that doesn't matter?

@TrangPham I don't think the bundling thing is much of a concern after the tsconfig change in this PR. Personally, I prefer to have a separate tests directory just for readability.


fix decimalToBinary function

The current implementation of decimalToBinary only works with base 2 numbers.
Calling decimalToBinary(3) returns 1.51, it should return 111.
The fix works with any decimal number.
Nice job with the repo, some good questions to look over 👀

Thanks! This was a helpful fix :D Just started this repo, so appreciate it!


[HOW-TO]

Question
**I'm new to EasyRepro and need pointers on how to select a single element from a pane (see attached image) I want to select the Dashboard element but the following bits of code do not work:
xrmBrowser.Navigation.OpenSubArea("Dashboard", "some-text-has-to-go-here");
xrmBrowser.Grid.SwitchView("Dashboard");
The first demands 2 values, and the second throws up an exception.
**

Hello
You use the wrong part of the api. xrmBrowser is for legacy web interface.
You should use the XrmApp client describe here:  XrmApp
Regards,
Nicolas

Thanks Paca, much appreciated


Ruff fixes

What's changing
This PR is an attempt to "clean" the repo from the errors/suggestion that ruff creates prior to configure it in the action (that PR will come later!)
How to test it
Additional notes for reviewers
Ruff fixes!
I already...

[ ] added some tests for any new functionality;
[ ] updated the documentation.

LGTM pending the numpy check, I also checked the rules that we disabled/re-enabled for the lm-buddy import https://github.com/mozilla-ai/lumigator/issues/225
and those look ok as well, cc @aittalam


[PUI] Date Input Rendering Issue

Noticed an issue with date input field rendering in PUI:

This seems to have been fixed now

Yes, I have fixed that with one of my last PRs. The Stylesheet for the date package was missing when migrating to mantine v7

Awesome, thanks for that @wolflu05


Can't find .bash_profile

Category

[ ] fzf binary
[ ] fzf-tmux script
[ ] Key bindings
[x] Completion
[ ] Vim
[ ] Neovim
[ ] Etc.

OS

[ ] Linux
[x] Mac OS X
[ ] Windows
[ ] Windows Subsystem for Linux
[ ] Etc.

Shell

[x] bash
[ ] zsh
[ ] fish

When typing vim bash**<TAB> it finds .bashrc but not .bash_profile even though I use it way more often.

In my case it fild .bash_logout and .bash_profile, but not .bashrc nor .bash_history.

The ** script uses the following code :
    command find -L "$1" \
      -name .git -prune -o -name .svn -prune -o \( -type d -o -type f -o -type l \) \
      -a -not -path "$1" -print 2> /dev/null | sed 's@^\./@@'

So you should try to run this command and see what it outputs.
You can redeclare _fzf_compgen_path to fix the issue on your side.
 _fzf_compgen_path() {
    echo "$1"
    command find -L "$1" \
      -name .git -prune -o -name .svn -prune -o \( -type d -o -type f -o -type l \) \
      -a -not -path "$1" -print 2> /dev/null | sed 's@^\./@@'
  }

I've had similar problems, but never posted a issue about it.
In my case it finds .bash_logout and .bash_profile, but not .bashrc nor .bash_history.
Plus, I have some folders (not empty, if it is of importance) in the home directory that I cannot cd to through Alt-C and I cannot search in through Ctrl-T. @redreceipt , do you really have problems only with those bash-related files you mentioned?
Now I've tried to rename a directory which is visible to fzf (in which I can both cd and search). It is no more visible now. I've tried to rename it back to its original name, but it is still invisible to fzf.
Maybe some kind of refresh is required?

@Aster89 I'm not sure what directories it's not working for, that's just the first one I've come across and it's frustrating because I edit it so much.

The ** script uses the following code :
    command find -L "$1" \
      -name .git -prune -o -name .svn -prune -o \( -type d -o -type f -o -type l \) \
      -a -not -path "$1" -print 2> /dev/null | sed 's@^\./@@'

So you should try to run this command and see what it outputs.
You can redeclare _fzf_compgen_path to fix the issue on your side.
 _fzf_compgen_path() {
    echo "$1"
    command find -L "$1" \
      -name .git -prune -o -name .svn -prune -o \( -type d -o -type f -o -type l \) \
      -a -not -path "$1" -print 2> /dev/null | sed 's@^\./@@'
  }

@edi9999 That script does nothing when I paste it in, but I'm not really sure what to expect or what it's supposed to do. but I can confirm fzf works in other instances.

I've had similar problems, but never posted a issue about it.
In my case it finds .bash_logout and .bash_profile, but not .bashrc nor .bash_history.
Plus, I have some folders (not empty, if it is of importance) in the home directory that I cannot cd to through Alt-C and I cannot search in through Ctrl-T. @redreceipt , do you really have problems only with those bash-related files you mentioned?
Now I've tried to rename a directory which is visible to fzf (in which I can both cd and search). It is no more visible now. I've tried to rename it back to its original name, but it is still invisible to fzf.
Maybe some kind of refresh is required?

@Aster89 I'm not sure what directories it's not working for, that's just the first one I've come across and it's frustrating because I edit it so much.

I can't find my .vimrc with fzf either. It's located in .vim/.vimrc.

I can see that the _fzf_compgen_path is the right thing.
If I overwrite _fzf_compgen_path() with the following :
  _fzf_compgen_path() {
	  echo "foobar"
	echo "$1"
    command find -L "$1" \
      -name .git -prune -o -name .svn -prune -o \( -type d -o -type f -o -type l \) \
      -a -not -path "$1" -print 2> /dev/null | sed 's@^\./@@'
  }

I can see foobar in the list.
The find command is executed with the . argument, so the command executed is actually :
command find -L "."        -print 2> /dev/null | sed 's@^\./@@'

The problem you get is probably that find is finding so many files that you don't see .bashrc because it is not loaded yet.

This is a bit linked to https://github.com/junegunn/fzf/pull/1380
I would advise you to add some ignores to the find command for directories that have lots of files but you don't need them.

@edi9999, I've hit Alt-C and written sufficient letters to identify the directoriey I'm looking for, then left the terminal alone. Eventually the directory is found. (I've done it for two directories.)
Concerning the dotted files, I couldn't wait enough time to have them found, not sooner than hitting system performances while working.

@redreceipt , what if you rename a directory that fzf successfully lists in reasonable time?
In my case, I renamed Dropbox to somthing else and then renamed it back to Dropbox, as a result, fzf finds it only after a long search.

I might have had a similar issue: I couldn't find directories that are beyond directories starting with a dot. Alt-C looking for  ytools which resides beyond .cache wasn't found. FZF Version 0.17.5 (b46227d).
I solved it with leaving out the -path parameter for find in the (zsh-)widget (~/.fzf/shell/key-bindings.zsh):

local cmd="${FZF_ALT_C_COMMAND:-"command find -L . -mindepth 1 \( -fstype 'sysfs' -o -fstype 'devfs' -o -fstype 'devtmpfs' -o -fstype 'proc' \) -prune
-o -type d -print 2> /dev/null | cut -b3-"}"

Original was:

local cmd="${FZF_ALT_C_COMMAND:-"command find -L . -mindepth 1 \( -path '/\.' -o -fstype 'sysfs' -o -fstype 'devfs' -o -fstype 'devtmpfs' -o -fstype 'proc' \) -prune
-o -type d -print 2> /dev/null | cut -b3-"}"

Maybe that helps!
I didn't elaborate much on the implications to leave out the -path-parameter.


Added the ability to use a custom property name mapper and property value parser

Fixes #19 and #5

This looks great - thanks @icnocop!
I have a little bit of a problem setting up my local dev environment and will be awhile before I get to this, to be able to recompile this.
Hold on - I will get here.
In the long run I will redo this package as a .NET core solution.

Very sorry that this got lost.
I've ported the whole project over so this pull request is very much conflicting now.
I'm closing it and should you want to see this implemented again - please open it again but update it for the code as it is now.


Add description to Join Waitlist

What is the waitlist? What are people signing up for?
Let's add a description so it's a bit clearer what the waitlist is for.

Was resolved in #299


DialogManager throws InvalidOperationException when DialogMessage is shown in second time (Threads and SimpleChildWindow involved)

Hello,
I have been being a problem with message dialogs when I'm using threads and child windows. Here is exception's information:
System.InvalidOperationException

Message:
   Sequence contains more than one element

StackTrace:
   at System.Linq.Enumerable.SingleOrDefault[TSource](IEnumerable`1 source)
   at MahApps.Metro.Controls.Dialogs.DialogManager.AddDialog(MetroWindow window, BaseMetroDialog dialog) in C:\Users\punker76\Documents\GitHub\MahApps.Metro\MahApps.Metro\Controls\Dialogs\DialogManager.cs:line 423
   at MahApps.Metro.Controls.Dialogs.DialogManager.SetupAndOpenDialog(MetroWindow window, BaseMetroDialog dialog) in C:\Users\punker76\Documents\GitHub\MahApps.Metro\MahApps.Metro\Controls\Dialogs\DialogManager.cs:line 413
   at MahApps.Metro.Controls.Dialogs.DialogManager.<>c__DisplayClass2_0.<ShowMessageAsync>b__1() in C:\Users\punker76\Documents\GitHub\MahApps.Metro\MahApps.Metro\Controls\Dialogs\DialogManager.cs:line 172
   at System.Windows.Threading.DispatcherOperation`1.InvokeDelegateCore()
   at System.Windows.Threading.DispatcherOperation.InvokeImpl()

This is my environment:

Windows 8.1 Pro (64 bits)
Visual Studio Ultimate 2013 Update 4

I'm using Mahapps stable libraies:

Mahapps.Metro <IP_ADDRESS>
Mahapps.Metro.SimpleChildWindow <IP_ADDRESS>

I developed an application to show that:
http://1drv.ms/1PZTAu1
There are 4 buttons or steps. You can reproduce the issue in 4 ways, following one of these steps sequences (only clicking step buttons and closing message dialogs):

1, 2, 2.
1, 2, 3, 4.
4, 4.
4, 1, 2.

This issue is opened on Mahapps.Metro too:
https://github.com/MahApps/MahApps.Metro/issues/2321

should be fixed with MahApps.Metro.SimpleChildWindow v1.3.0 and MahApps.Metro v1.2.3


Session stolen when connecting on the same user

Hi, so the issue I'm having which makes it weird to use RDP while I am using the desktop is that it steals the current session and transfers it to the 1st rdp connection. RDP wrap still works though as I can then connect on the desktop and have multiple sessions. This can be a problem because I have scripts to run when connecting on desktop vs RDP. I also find it to meddle with RDPWrap's goals. I'm sure there's a way to figure this out, I am not the only one this fix could be useful to. Thanks.

This is typical RDP (Terminal Services) behavior

Yes, still feels like an issue tho. I don't think I am qualified for the job but.. I think we could trick terminal services into not transfering the session over like it does with a 3rd device? Maybe start an empty dummy session that would get transfered/stop normal behavior.
Or make terminal services think we're already in rdp ?
Anyways you are probably more knowledgeful. Thanks

From: 1 @.>
Sent: Thursday, November 4, 2021 9:04:24 PM
To: stascorp/rdpwrap @.>
Cc: Cam @.>; Author @.>
Subject: Re: [stascorp/rdpwrap] Session stolen when connecting on the same user (Issue #1581)
This is typical RDP (Terminal Services) behavior
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/stascorp/rdpwrap/issues/1581#issuecomment-961548818, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABSTWRO6HY2TOXY7SNBTVN3UKM3SRANCNFSM5HHWIATA.

Not sure how deep that plumbing goes. Sounds like something you want is TS Shadowing ( https://docs.microsoft.com/en-us/troubleshoot/windows-server/remote/shadow-terminal-server-session )... something I haven't tried much since leveraging Jump Desktop, VNC, or TeamViewer for such cases.


Warn people about mistaking "M" suffix for "m" when setting resource limits

Although it's not recommended, we also support 'm' as an unit for memory which means milli-bytes.
https://github.com/kubernetes/apimachinery/blob/master/pkg/api/resource/quantity.go#L46
We need to document it.
https://kubernetes.io/docs/concepts/configuration/manage-resources-containers/#meaning-of-memory

@dialogbox can you explain why? I imagine it's in case someone writes a quantity as, eg, 256m meaning 256Mi, and they don't realize their error?
We do document quantities: https://kubernetes.io/docs/reference/glossary/?all=true#term-quantity
/priority awaiting-more-evidence
/language en

@sftim Yes. That is exactly the case. Although we have a document about the general quantities, this page would be the page someone who is searching about memory configuration.
And since this page also mentions but only mentions E, P, T, G, M, k, and since 'm' just works, I saw some people blindly assumed that the 'm' means 'Mega'. This might not be a big problem for resource limits because the differences are so huge like 1/1,000,000,000, most likely this mistake will cause cgroup OOM. However, when it comes to the resource requests, it's not easy to know there is some mistake. But pod scheduling might work incorrectly because the the requests is incorrect.
So I'm requesting to add this as a kind of warning.

/triage accepted
/kind feature
/priority backlog
/help
/language en
Thanks for clarifying. Let's try to use a glossary tooltip for what we mean by “quantity”, too, if we can.

/retitle Warn people about mistaking "M" suffix for "m" when setting resource limits

Hi, I would like to work on this.

/assign

I was thinking to solve the PR in the following manner:

On the page https://kubernetes.io/docs/concepts/configuration/manage-resources-containers/#meaning-of-memory I will add 'm (milli)' in the list of suffixes.
Before the example I would include a Caution/ Note telling people while setting resource limits that they should not mistake the m (mili) suffix as M  (mega) suffix of memory.
In the same Note/Caution I will try to add the glossary tooltip for memory.

@sftim @dialogbo Could be you please provide your inputs for my method above to fix this PR. Thanks.

@Babapool I love the idea of having the Caution/ Note. Can you please add the comment to the https://github.com/kubernetes/website/pull/30501?

@dialogbox Let me ask the reviewers on #30501 on how to proceed on the same and as per their advice/suggestion I will do the necessary changes.

@dialogbox Since you have squashed and merged the commits in #30501 can ask whether I could add the Caution/Note part ?

@Babapool I think making it as caution/note is a reasonable idea but I'm not sure if others would agree on that. How about keep having discussion here and filing a new pr for adding Caution/Note once we got concluded?

@dialogbox Thanks I will ask them again the PR conversation.

/remove-lifecycle stale

This issue is fixed by PR #30501. I believe that this issue can be closed. Thanks.

/close


Playing Hls content(Media Player)

Here The Media Player is not supporting to Play hls.m3u8 video urls. when we are playing that content error we are getting .It is still in implementing stage or is there any way to play this content,,, very late support from sdk team thats way i am raising issue Here
Url=https://bitmovin-a.akamaihd.net/content/playhouse-vr/m3u8s/105560.m3u8 please help me

Lightning framework is only responsible for apps rendering is scope of canvas HTML tag. Lightning is not responsible for Audio/Video playback (HTML5 audio / video tag).
Above HLS manifest file can be played via MSE interface (eg. with usage of hls.js or shaka library) or natively by browser:document.getElementsByTagName('video')[0].src = "https://bitmovin-a.akamaihd.net/content/playhouse-vr/m3u8s/105560.m3u8". It all depends on your browser capabilities.
URL to playback from hls.js based app (playback works for me): https://hls-js.netlify.app/demo?src=https%3A%2F%2Fbitmovin-a.akamaihd.net%2Fcontent%2Fplayhouse-vr%2Fm3u8s%2F105560.m3u8 &demoConfig=eyJlbmFibGVTdHJlYW1pbmciOnRydWUsImF1dG9SZWNvdmVyRXJyb3IiOnRydWUsInN0b3BPblN0YWxsIjpmYWxzZSwiZHVtcGZNUDQiOmZhbHNlLCJsZXZlbENhcHBpbmciOi0xLCJsaW1pdE1ldHJpY3MiOi0xfQ==
Streaming also can be verified with usage of ffplay (with switching in URL master manifest part to one with specific resolution):
ffplay https://bitmovin-a.akamaihd.net/content/playhouse-vr/m3u8s/105560_video_540_1500000.m3u8

Ticket should be rejected.

Hi @krishdarling143, @jacek-skiba-red,
It is possible to use the Lightning SDK VideoPlayer (which was recently moved to the Metrological SDK, but is still coming as part of the Lightning SDK - documentation here) to play HLS content. You just need a custom handler.
Please see the videoplayer-hls-example GitHub project. A similar approach can be used to play other formats, like Dash - there is also a dash-drm-example here.


Fix siginfo_t for Illumos

The siginfo_t struct appears to be incorrect for Illumos. It currently has si_code before si_errno and appears to be too large. I'm not familiar with this platform, so please let me know if this fix is inaccurate.
siginfo_t:
https://github.com/illumos/illumos-gate/blob/74e98f497ae878ae481ccba1f91fd8dc26fa6627/usr/src/boot/sys/sys/signal.h#L196
siginfo.h:
https://github.com/illumos/illumos-gate/blob/74e98f497ae878ae481ccba1f91fd8dc26fa6627/usr/src/uts/common/sys/siginfo.h#L164
I am not sure if the packed() attribute should remain, but I copied it from the Solaris definition.

Some changes occurred in solarish module
cc @jclulow,@pfmooney

r? @Amanieu
(rust-highfive has picked a reviewer for you, use r? to override)

siginfo_t: https://github.com/illumos/illumos-gate/blob/74e98f497ae878ae481ccba1f91fd8dc26fa6627/usr/src/boot/sys/sys/signal.h#L196

That's not the right header.  Not that it's under usr/src/boot which was imported with our copy of the FreeBSD boot loader.

Can I ask what lead you to make this change?  Did something fail to compile or work correctly on an illumos system?

@jclulow Yes, the motivation is this issue:
https://github.com/dylni/process_control/issues/12
libc currently does not define siginfo_t::si_status on Illumos, which is required by process_control. The signal codes are also required so were included with this change.
If you let me know the correct definition, I can certainly update this PR.

@jclulow Yes, the motivation is this issue: dylni/process_control#12
libc currently does not define siginfo_t::si_status on Illumos, which is required by process_control. The signal codes are also required so were included with this change.
If you let me know the correct definition, I can certainly update this PR.

I'm working on an update to expose the siginfo_t fields you need using function accessors like the other unix-y platforms have done.  I should have it posted in the next day or so.

Thank you @pfmooney! Please also include CLD_EXITED and the related constants from this PR if possible.


Adding: Anna Schaar

Onboarding checklist

[x] add to website
[x] add to<EMAIL_ADDRESS>[x] add to zulip

[x] add to core stream
[x] add to core group

[ ] moderation privileges on discourse
[ ] add to password manager
[x] add to github org and scverse/core github team
[x] invite to governance and community meetings
[x] add to google drive
[x] write permission on google calendar
[x] announce the new member on twitter

@ivirshup I see 2 unticked boxes. Would it be possible for you to please take care of them so that the onboarding process is finished for Anna, please?


Read channel capacity from env vars.

Instead of hard coding the channel capacity, use the env var to get it.

implemented in girish946@405b4369e387ed5e09faf2f036523c3e7d20ab5b


Upgrade to jasperreports-libraries 6.19.1

Here are the release notes for the new release:
JasperReports Library 6.19.0 Change Log

improved bulleted and numbered list support in styled text and HTML markup text fields including better indentation and support for nested lists;
support for accessible HTML output using semantic tags for marking up headings (h1 to h6), bulleted and numbered lists, alternate text for images and nested table tags for semantic tables;
support for Groovy closures in report expressions;
honour the EXIF orientation information found in photos taken with mobile digital cameras and thus match the way browsers make use of this information to auto-rotate photos in HTML;
deprecated Castor XML persistence layer used for data adapter files and XML chart theme files by moving it to a separate optional extension artifact (jasperreports-castor) while replacing it in the core library with an equivalent and compatible Jackson XML persistence layer;
introduced the *.jrdax file extension convention for data adapter XML files;
various dependencies upgrades including: Apache POI 5.1.0, Jackson 2.13.1, Apache Log4J 2.17.1, Mozilla Rhino 1.7.14, Spring 5.3.14, HyperSQL 2.6.1 and Jetty 9.4.44;
minor bug fixes and improvements;

JasperReports Library 6.19.1 Change Log

replaced Joda-Time with Java 8 Date/Time API in the functions extension;
minor bug fixes and improvements;


Added first version of NPC statistics page.

TODO added by Phil:

[x] Get plots working
[ ] Add description text
[ ] Add Swedish version of the page

Merging to branch npc-stats-page

Hmm, not really sure why I merged this - could have left it open. Ah well.


fix: handling of multipart/form-data requests and how we interact with FormData

Did you know that multipart/form-data requests is a HARD problem? And one that we're completely fumbling to solve right now to the point where some requests are being corrupted?
📖 Story time!
The most popular NPM package for handling the FormData API in Node, form-data, has a wholly non-spec compliant implementation of FormData. Not only does it ship an .append() API that has a non-compliant third argument, but it also supports file buffers. It also doesn't have an .entries() API so if you want to retrieve data out of it you need to pipe it through a Node stream.
Normally this wouldn't be a problem because if you're using form-data it's because you're in an environment where you don't have the native FormData API available to you. This obviously isn't the case for this library as we target both Node and browsers.
Well, that's fine because node-fetch suggests formdata-node and formdata-polyfill instead. Yeah...

formdata-polyfill is now an ESM-only package and the non-ESM version is no longer receiving updates, nor does it expose itself in a way that we can use it.
formdata-node works great... but requires you to use an encoder in order to use it with node-fetch@2. node-fetch@3 supports it on its own but is... you guessed it... an ESM-only package now.

Everything's fine I swear.

🏚 What we're doing wrong now
The way we're handling things currently is that because form-data doesn't expose .entries() API we can easily detect if it's form-data and not the real thing. When we encounter this we've got some branching logic to handle this non-compliant .append() API, and that works all just fine and dandy.
Not! haha got em

Because fetch-har is HAR based for files we're relying on Data URLs that we generate in our reference guide.
Because Data URLs are base64 encoded strings, if you base64 encode say the OpenAPI petstore example you're going to have a massive HAR.
Want to upload a binary? Yeah good luck with that, we can't handle that at all right now.
Want to upload an image? This sort of works, but we have bad logic currently where when we decode the data URL to supply it to the Blob API for FormData we're supplying a buffer -> string copy out of the data URL. This corrupts images!

🧰 What's being changed?
Still here? A lot's been changed to resolve [gestures at everything].
Starting off we're no longer allowing folks to use the non-compliant form-data package with this library. There are compliant options out there that don't require us to have gross hacks in place to support. If we detect that your FormData API is actually form-data, or otherwise non-compliant, we'll throw a hard error. Beyond that it gets a bit in the weeds:

[x] We're now supporting multipart/alternative, multipart/mixed, and multipart/related requests. These function identicaly to multipart/form-data.
[x] The existing userAgent second argument to fetch-har has been reworked into being an options object.
[x] A new files option has been added into this reworked argument. This option is a file -> Buffer|File mapping that folks can use if they don't want to, or don't have data URLs in their HAR for files.

In our reference guide this will start allowing us to accept large file uploads without having to deal with data URL conversions.

[x] A new multipartEncoder option has been added to supplement the formdata-node and node-fetch@2 interdependency that requires you to use form-data-encoder.

This is only a requirement if you're on Node and using node-fetch@2.
I should note that you definitely can't load form-data-encoder in the browser because it uses lookbehind regex that crashes the page in Safari; something I found out the hard way.

On a less exciting front I've also completely rewritten the test suite, because why not tack on another refactor to a refactor.

[x] Moved the test suite from using Jest to Mocha + Chai.

As much as I enjoy working with Jest, moving these tests over to Mocha allows us to use the same test code in all environments. More on that in a sec.

[x] All tests are now full integration tests that call https://httpbin.org

The existing mocks were fine but I have more confidence in seeing a real web server interact with and respond to requests this library is making. This has also allowed me to bulk up the tests for file uploads to make sure that they're actually working right.

[x] The existing browser suite that was using Puppeteer has also been moved over to Karma so we now have full coverage across Chrome, Firefox, and Safari.

And tests run on Windows in CI now too. 💅
🧬 Testing
There is a lot going on here, but everything is fully tested in Node, Chrome, Firefox, and Safari.  To run the Node test suite run npm test as you normally would.
For the run the browser tests you'll need to do the following:

Run npm test:browser
When Safari opens you'll need to click the open button to load the Karma runner as Safari requires user feedback (https://github.com/karma-runner/karma-safari-launcher/issues/29)
Wait

https://user-images.githubusercontent.com/33762/150459331-2a726af6-b306-443b-bed8-884de0b1b16d.mov
If you don't believe me that file uploads actually work now, give it a run yourself:
/* eslint-disable import/no-extraneous-dependencies, no-console */
require('isomorphic-fetch');
const fetchHar = require('.');
const parseDataUrl = require('parse-data-url');
const fs = require('fs').promises;
const { FormDataEncoder } = require('form-data-encoder');
const harExamples = require('har-examples');

globalThis.Blob = require('formdata-node').Blob;
globalThis.File = require('formdata-node').File;
globalThis.FormData = require('formdata-node').FormData;

fetchHar(harExamples['multipart-data-dataurl'], { multipartEncoder: FormDataEncoder })
  .then(res => res.json())
  .then(async res => {
    console.log(res);

    await fs.writeFile('image.png', parseDataUrl(res.files.foo).toBuffer());
  });

This code will upload an encoded image from a HAR (here) and then download and save what httpbin sent back. If you open up image.png you'll be greeted with a familiar face.
If you however you run this same HAR through the current release of fetch-har the call to parse-data-url on res.files.foo will fail because the contents of res.files.foo are corrupted:
require('isomorphic-fetch');
const fetchHar = require('fetch-har');
const parseDataUrl = require('parse-data-url');
const fs = require('fs').promises;
const harExamples = require('har-examples');

global.fetch = require('node-fetch');
global.Headers = require('node-fetch').Headers;
global.Request = require('node-fetch').Request;
global.FormData = require('form-data');

fetchHar(harExamples['multipart-data-dataurl'])
  .then(res => res.json())
  .then(async res => {
    console.log(res);

    await fs.writeFile('image.png', parseDataUrl(res.files.foo).toBuffer());
  });

{
  args: {},
  data: '',
  files: {
    foo: '�PNG\r\n' +
      '\x1A\n' +
      '\x00\x00\x00\rIHDR\x00\x00\x00\n' +
      '\x00\x00\x00\f\b\x06\x00\x00\x00[k,�\x00\x00\x01WIDATx\x01cp/�a�Q��U�\x10�\x06��1???��\x00�9�\r\x16p�W�I��j\x02\t�$Ab@��c��f�"�\n' +
      "W\bRT\x1Eh�#�J�IY���`��Q\x0E\x06�s��_z\x1B�Ԣ(�M��ߘ\x1D��&��?�\x1D�d��QK.\r��8+e��M\x1B�����ύ���v��MM����C�ì\x06;�$9��ľ��n.V��7��_6��\x7Fp�?\x18�k+l\x05�\x01��<��\x05�:�im��\x13�\f~\x01�`�A�\x06�a�U\x10�\x06rפ4��=�N(����\x7F���\x7F�\x1A\x06ce�� ��E>p<��\x0BL\x03\r\x00+�S\x10�� +��\x032\x1E�\x00���y�\x15�\x1A*�\x03�\x01��T]8� ��'�z���\x04{�[��W�b\r�0ŗA;ɇ����܉8\x17�[ �k)�\x10$f�� \rR\x03\x00���I��چ\x00\x00\x00\x00IEND�B`�"
  },
  form: {},
  headers: {
    Accept: '*/*',
    'Accept-Encoding': 'gzip,deflate',
    'Content-Length': '915',
    'Content-Type': 'multipart/form-data;boundary=--------------------------410362018522492426091525',
    Host: 'httpbin.org',
    'User-Agent': 'node-fetch/1.0 (+https://github.com/bitinn/node-fetch)',
    'X-Amzn-Trace-Id': 'Root=1-61ea47c0-5c15ca78234107880a7aa011'
  },
  json: null,
  origin: '<IP_ADDRESS>',
  url: 'https://httpbin.org/post'
}
(node:10510) UnhandledPromiseRejectionWarning: TypeError: parseDataUrl(...).toBuffer is not a function
    at /Users/erunion/Desktop/test/index.js:17:65
    at processTicksAndRejections (internal/process/task_queues.js:95:5)
(Use `node --trace-warnings ...` to show where the warning was created)
(node:10510) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 2)
(node:10510) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Yikes!

I'm guessing this is a major version bump right? 😅 (Not looked at the code yet)

@ilias-t we should consider integrating this in the Metrics Service!


Regression in 3.7.1 in calculating label floats compared to 2.9.3

Expected behavior
Before in 2.9.3:

Current behavior
Same code and same data in 3.7.1:

Reproducible sample
https://serpapi.com/status
Optional extra steps/info to reproduce
https://serpapi.com/status is still on 2.9.3 but you can upgrade the Chart.js then re-run the graph.
Possible solution
No response
Context
No response
chart.js version
v3.7.1
Browser name and version
Chrome latest
Link to your project
https://serpapi.com/

Can you add a reproducable sample with the template provided in the issue template. Since I dont see any way in the site you linked to switch between V3 and V2

I'm guessing this is coming from the formatter that's been added to add the % symbol. See https://jsfiddle.net/5n312Laf/1/ as an example. if the callback is removed, the labels show with the proper precision.
It probably makes sense to have the callback call the default and append the % sign. The default formatter is https://github.com/chartjs/Chart.js/blob/master/src/core/core.ticks.js#L28-L54.
Example calling the internal formatter first: https://jsfiddle.net/uvwf3zkg/


Ensure offload datastore and RDBMS are tied together

Is your feature request related to a problem? Please describe.
When using blob offload (e.g. Azure Blob) to store payloads, administrators must be extremely careful not to change the container or database used for a given tenant. If one service is changed without the other, data corruption will occur impacting the integrity of the tenant.
Describe the solution you'd like
Provide a metadata mechanism to establish a logical connection between the RDBMS and offload stores. This can be checked as part of a healthcheck operation or the first time a load/store operation is performed.
Describe alternatives you've considered
Leave as-is and accept the risk of a configuration issue causing data problems.
Acceptance Criteria

GIVEN [a precondition]
AND [another precondition]
WHEN [test step]
AND [test step]
THEN [verification step]
AND [verification step]

Additional context
Add any other context or screenshots about the feature request here.

One thought is to simply write a UUID value to both the RDBMS and blob container. As long as this value matches, the two services can be considered connected. Another simple solution would be to use a random value which acts as a salt to a SHA-256 hash of the tenant name and store both values in both services. This would provide further protection that each service is meant for the configured tenant.

maybe do at the same time as #3914 ?


Override memory limits of rook operator to 512Mi

Add tests for rook-ceph

Fixes: https://github.com/kinvolk/lokomotive/issues/932

make test is failing for me:
--- FAIL: TestConversion (0.16s)
    --- FAIL: TestConversion/EnvVars (0.00s)
        --- FAIL: TestConversion/EnvVars/CSI_PROVISIONER_NODE_AFFINITY (0.00s)
            component_test.go:198: expected value of env "CSI_PROVISIONER_NODE_AFFINITY":"storage_node_key1=storage_node_value1; storage_node_key2=storage_node_value2;", got "CSI_PROVISIONER_NODE_AFFINITY":"storage_node_key2=storage_node_value2; storage_node_key1=storage_node_value1;"
        --- FAIL: TestConversion/EnvVars/AGENT_NODE_AFFINITY (0.00s)
            component_test.go:198: expected value of env "AGENT_NODE_AFFINITY":"storage_node_key1=storage_node_value1; storage_node_key2=storage_node_value2;", got "AGENT_NODE_AFFINITY":"storage_node_key2=storage_node_value2; storage_node_key1=storage_node_value1;"
        --- FAIL: TestConversion/EnvVars/DISCOVER_AGENT_NODE_AFFINITY (0.00s)
            component_test.go:198: expected value of env "DISCOVER_AGENT_NODE_AFFINITY":"storage_node_key1=storage_node_value1; storage_node_key2=storage_node_value2;", got "DISCOVER_AGENT_NODE_AFFINITY":"storage_node_key2=storage_node_value2; storage_node_key1=storage_node_value1;"
FAIL
coverage: 77.4% of statements
FAIL    github.com/kinvolk/lokomotive/pkg/components/rook       0.267s

@johananl Thanks for catching that.

make test is failing for me:

I have filed an issue here: https://github.com/kinvolk/lokomotive/issues/942.
Also I have disabled the tests that have inconsistent output and are prone to fail randomly.

It seems this PR hit https://github.com/kinvolk/lokomotive/issues/940 many times :| And #952 simplifies modifying the timeouts.

It seems this PR hit #940 many times :| And #952 simplifies modifying the timeouts.

Yeah, I thought it was one off, but then my cluster that I created locally also timed out.

I think I have an alternative proposal to the current test structure. I'm still typing it but if you want to wait a bit I can push it somewhere.

Pushed an example here: https://github.com/kinvolk/lokomotive/blob/39e3a898fa955e8493d8ffca187ca5736d6d1050/pkg/components/rook/component_conversion_test.go


nodeos won't run properly, gives time error

This is what I get in my console when I run nodeos, this is on an Ubuntu 16.04 VM:
sunny@sunny-VirtualBox:~$ nodeos
3488600ms thread-0   chain_plugin.cpp:125          plugin_initialize    ] initializing chain plugin
3488600ms thread-0   main.cpp:99                   main                 ] 13 N5boost16exception_detail10clone_implINS0_19error_info_injectorINS_9gregorian8bad_yearEEEEE: Year is out of valid range: 1400..10000
Year is out of valid range: 1400..10000: unable to convert ISO-formatted string to fc::time_point_sec
{"what":"Year is out of valid range: 1400..10000"}
thread-0  time.cpp:44 from_iso_string
unable to convert ISO-formatted string to fc::time_point
{}
thread-0  time.cpp:64 from_iso_string
I built EOS properly and did sudo make install in the /build folder. What else should I try?

Did you run a newer version of nodeos with older version of block log file /shared_mem?

We haven't received any update for this. If you still have the problem and wish to discuss further, please post it in https://eosio.stackexchange.com


Created CHANGELOG.md

This is a sample changelog.md which can be fixed in the future as per the code changes.

Aight, first changes are up, want to update the changelog accordingly and I'll publish? 🙂


Use generic registration instead of reflection

Components will be of type Component<object> instead. This would supersede the change in https://github.com/castleproject/Windsor/pull/553.

@jonorossi I agree with this change :-)


sudo remnux install --mode=addon & remnux install/upgrade failing

Hello,
I am trying to install from a new SANS sift ova as an addon and the install keeps failing. Please see attached SaltStack.log. I have tried multiple things from peoples previous issues with no luck. Any help would be appreciated. Thank you.
saltstack.log

This worked for me thank you!


sync src with navalgazer & bump deps

works with iojs
passes all tests

it's pretty fast too.. so I ported watchify https://github.com/heavyk/gazeify

shit, well it seems that the tests hang on linux for node-0.12 and iojs
however, locally here on OSX and iojs it's all good

Rad thanks for attempting this!
Have you seen the navelgazer branch? In there I am consuming navelgazer as a dependency and it gets a bit further on the tests with linux. I believe only a couple are failing.

oh, no, I didn't see the navalgazer branch. I'll check it out.
this has been working for me flawlessly on OSX, so for now, I'm just going to leave it.
later today I will be putting this up on the server. it'll get some linux testing then.
when I do that, I'll try and see if I can't figure out why it's hanging on the rename test and get back to you
cheers

real quick: I read through some of your changes now. I had a similar issue when writing another thing.
essentially, when pattern is '/*.js' or anything with two '' then all subdirs must be explicitly watched.
lol, and I noticed that gaze04 was being used too for old node compat...
ok, I think I can probably get this :) cheers mate


Help!!! I transfer LUNA from BSC to Terra,and I do not receive my LUNA  on Terra .

this is the burn transaction hash:
https://bscscan.com/tx/0x6ee8b794039f09543db924d6a9cbe5dbfbfb9ba3d5dc24c1df9008ef9050cf28
and my Terra wallet address is:
terra16gzek2lzzn8hsmtpxk65ds0g9wd38vr2kwrwmx
PLEASE HELP ME! THANK YOU!

I use the SHUTTLE.

this is the burn transaction hash: https://bscscan.com/tx/0x6ee8b794039f09543db924d6a9cbe5dbfbfb9ba3d5dc24c1df9008ef9050cf28
and my Terra wallet address is: terra16gzek2lzzn8hsmtpxk65ds0g9wd38vr2kwrwmx
PLEASE HELP ME! THANK YOU!

Hello, A support ticket has been opened for you with Support ticket number TB7836HH. You should talk to a Terra money Dev on the LIVE SUPPORT PAGE So that your issue can be looked into. Thank you. CLICK HERE

I use the SHUTTLE.

Hello, A support ticket has been opened for you with Support ticket number FH7994HS. You should talk to a Terra money Dev on the LIVE SUPPORT PAGE So that your issue can be looked into. Thank you. CLICK HERE

this is the burn transaction hash: https://bscscan.com/tx/0x6ee8b794039f09543db924d6a9cbe5dbfbfb9ba3d5dc24c1df9008ef9050cf28
and my Terra wallet address is: terra16gzek2lzzn8hsmtpxk65ds0g9wd38vr2kwrwmx
PLEASE HELP ME! THANK YOU!

Hello, A support ticket has been opened for you with Support ticket number BN399UI. You should talk to a Terra money Dev on the LIVE SUPPORT PAGE So that your issue can be looked into. Thank you. CLICK HERE


Update Platforms

Add megaduck & g7400
Add Duckstation (Legacy) to psx platform for those using Duckstation Legacy app

For Duckstation Legacy apk I wasn't sure what the uid should be.

For Duckstation Legacy apk I wasn't sure what the uid should be.

Thanks for the pull request. Lemme do some research. :3

Thanks!
Noticed the Videopac+ G7400 was changed to Magnavox Odyssey 2. They're actually two different systems. G7400 was released only Europe (it was the successor to the Odyssey 2, which was also called Videopac in Europe).
I wanted to close #445 . Or did you want to combine it with Odyssey 2? (Which is already available)

Hello brother. I've followed the naming scheme from Retropie. And I thought they all share the same emulator. Perhaps a more generic naming will be better? So that it includes the consoles which the emulator supports, especially, consider that the ROM sets aren't very big, so it is better to combine them anyway. For instance, like "Philips VideoPac".

Hold on a second. There is odyssey 2 already.

Your naming scheme is 100% better. I am gonna revert it back. Thanks for the help brother!