padrino rake auto-loads test tasks in production environment

it seems that when loading padrino to perform a rake task with the -e production flag on, the test rakefile is loaded as well. because the gem for the test framework is typically not installed in production evironments (true of the initial Gemfile padrino itself generates) this will immediately cause an error.
the following script will reproduce this in a new project.
padrino g project project -t rspec
cd project
mkdir tasks bundle
echo 'task :pro' > tasks/pro.rake
sudo bundle install --gemfile Gemfile --path /bundle --deployment --quiet --without development test
bundle exec padrino rake pro -e production

thanks!

Interesting, thanks for the report. Thoughts @DAddYE ?

Yep this i normal, because rake loads al rakes defined in project.
So in spec/spec.rake there is this:
require 'rspec/core/rake_task'

which cause the problem since rspec is not part of our bundle.
It's quite complex for padrino dynamically loads correct rakes for the current environment. So my suggestion is to edit spec/spec.rake in this way:
begin
require 'rspec/core/rake_task'

spec_tasks = Dir['spec/*/'].map { |d| File.basename(d) }

spec_tasks.each do |folder|
  RSpec::Core::RakeTask.new("spec:#{folder}") do |t|
    t.pattern = "./spec/#{folder}/**/*_spec.rb"
    t.rspec_opts = %w(-fs --color)
  end
end

desc "Run complete application spec suite"
task 'spec' => spec_tasks.map { |f| "spec:#{f}" }
rescue LoadError
  puts "RSpec is not part of this bundle, skip specs."
end

Right, but perhaps this should still be fixed in the generators? It's just odd that a blank project would fail in production without any modifications.


Question on how to set dataset size for training

Dear Mr. Zbontar,
after having gone through the main.lua I can't seem to find the argument which to pass in order so set the size for the dataset on which to train the network. I do know that with "./main.lua kitti fast -a train_tr" I train the network on a subset of the kitti dataset but I would like to do a test just as stated in table 9, pg. 26 of your paper with say just 20% of the entire picture set.
Any help is much appreciated.
Regards,
Raphael.

You can use the -subset option.
https://github.com/jzbontar/mc-cnn/blob/master/main.lua#L621-L647

so the command would be something like:
'./main.lua kitti2015 slow -a train_tr -subset -200'? for 50% of the kitti2015 dataset since the set itself contains 400 images, correct?
Thank you very much.

I think it's more like this: ./main.lua kitti2015 slow -a train_tr -subset 0.5

thank you very much!

Hallo once again Mr Zbontar,
After successfully running through 14 epochs of training using the -subset option, the shell returns the following error:
THCudaCheck FAIL file=/tmp/luarocks_cutorch-scm-1-6134/cutorch/lib/THC/generic/THCStorage.cu line=66 error=2 : out of memory
luajit: ...torch/install/share/lua/5.1/cudnn/SpatialConvolution.lua:144: cuda runtime error (2) : out of memory at /tmp/luarocks_cutorch-scm-1-6134/cutorch/lib/THC/generic/THCStorage.cu:66
stack traceback:
[C]: in function 'resize'
...torch/install/share/lua/5.1/cudnn/SpatialConvolution.lua:144: in function 'createIODescriptors'
...torch/install/share/lua/5.1/cudnn/SpatialConvolution.lua:188: in function 'updateOutput'
./main.lua:911: in function 'forward_free'
./main.lua:945: in function 'stereo_predict'
./main.lua:1198: in main chunk
[C]: at 0x004064f0
Do you have any suggestions on how to solve this issue? I am training the net on the following GPU:
+-----------------------------------------------------------------------------+
| NVIDIA-SMI 375.66                 Driver Version: 375.66                    |
|-------------------------------+----------------------+----------------------+
| GPU  Name        Persistence-M| Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp  Perf  Pwr:Usage/Cap|         Memory-Usage | GPU-Util  Compute M. |
|===============================+======================+======================|
|   0  Tesla K40c          Off  | 0000:04:00.0     Off |                    0 |
| 29%   53C    P0    69W / 235W |  10939MiB / 11439MiB |      0%      Default |
+-------------------------------+----------------------+----------------------+
Best regards,
Raphael.
Von: Jure Zbontar<EMAIL_ADDRESS>Gesendet: Dienstag, 11. Juli 2017 15:37
An: jzbontar/mc-cnn<EMAIL_ADDRESS>Cc: Behrendt, Raphael Eric Alfred<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Betreff: Re: [jzbontar/mc-cnn] Question on how to set dataset size for training (#33)
I think it's more like this: ./main.lua kitti2015 slow -a train_tr -subset 0.5
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/jzbontar/mc-cnn/issues/33#issuecomment-314445623, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Acc2ysXtx5XA3uHsqlDJqdgbpQKeOODjks5sM3pjgaJpZM4OLh2k.


Mmonit 4.2.0 very slow in use ....

As described above, I have updated on the Synology from 4.0.2 to 4.2.0. The container starts after a long "PRAGMA check" ... but the whole system responds very slow !! ... it takes minutes to load the different pages :-(

This is not the place to report issues with M/Monit, only the container.
I also saw 4.2.0 being very slow and moved back to 4.0.2.


Kong Upstream health Check is not working for https protocol

Summary
We are trying to configure healthcheck for Kong Upstream, it works with http protocol, but fails for https protocol.
As an example, we have used postman-eho.com as the upstream
Steps To Reproduce
Non-SSL Health Check - (Working fine)
Configure upstream in Kong
curl -i -X POST  --url http://localhost:8001/upstreams/   --data slots=10  --data name=postmanEchoNonSSL
Configure Target for same Upstream:
curl -i -X POST  --url http://localhost:8001/upstreams/postmanEchoNonSSL/targets --data target:80=postman-echo.com
Configuring health check (LB_STATUS) in upstream:
curl -i -X PATCH --url http://localhost:8001/upstreams/postmanEchoNonSSL --data healthchecks.active.http_path=/get --data healthchecks.active.healthy.interval=1 --data healthchecks.active.unhealthy.interval=1 --data healthchecks.active.unhealthy.http_failures=1 --data healthchecks.active.healthy.successes=1 --data healthchecks.active.unhealthy.tcp_failures=1 --data created_at=1556520868
Create a Service in kong with the upstream
curl -i -X POST --url http://localhost:8001/services/ --data name=postmanNonSSLService --data url=http://postmanEchoNonSSL/get
Create a Route to  Service
curl -i -X POST --url http://localhost:8001/services/postmanNonSSLService/routes --data paths=/getPostmanNonSSL
Working request
curl -i -X GET http://localhost:8000/getPostmanNonSSL?foo1=bar1
Response:
HTTP/1.1 200 OK
Content-Type: application/json; charset=utf-8
Content-Length: 234
Connection: keep-alive
Date: Sun, 09 Jun 2019 07:54:39 GMT
ETag: W/"ea-GvAjNpHCdApDI5prvM98Kj0uwN8"
Server: nginx
set-cookie: sails.sid=s%3Axr7lMUT6SzKcZf6vGQGiMAl8WPOk89oL.L%2Btw4aj2h%2BXmUtM4R0rzDQ3LyXbxhKPYD%2Fe2tCzKG40; Path=/; HttpOnly
Vary: Accept-Encoding
X-Kong-Upstream-Latency: 523
X-Kong-Proxy-Latency: 38
Via: kong/1.0.0
{"args":{"foo1":"bar1"},"headers":{"x-forwarded-proto":"https","host":"postman-echo.com","accept":"/","user-agent":"curl/7.57.0","x-forwarded-host":"localhost","x-forwarded-port":"80"},"url":"https://postman-echo.com/get?foo1=bar1"}
SSL Health Check (Not working)
Configure SSL upstream in Kong
curl -i -X POST  --url http://localhost:8001/upstreams/   --data slots=10  --data name=postmanEchoSSL
Configure Target for same Upstream:
curl -i -X POST  --url http://localhost:8001/upstreams/postmanEchoSSL/targets --data target:443=postman-echo.com
Configuring health check (LB_STATUS) in upstream:
curl -i -X PATCH --url http://localhost:8001/upstreams/postmanEchoSSL --data healthchecks.active.http_path=/get --data healthchecks.active.healthy.interval=1 --data healthchecks.active.unhealthy.interval=1 --data healthchecks.active.unhealthy.http_failures=1 --data healthchecks.active.healthy.successes=1 --data healthchecks.active.unhealthy.tcp_failures=1 --data created_at=1556520868
Create a Service in kong with the upstream
curl -i -X POST --url http://localhost:8001/services/ --data name=postmanSSLService --data url=http://postmanEchoSSL/get
Create a Route to  Service
curl -i -X POST --url http://localhost:8001/services/postmanSSLService/routes --data paths=/getPostmanSSL
Request
curl -i -X GET http://localhost:8000/getPostmanSSL?foo1=bar1
Response
HTTP/1.1 503 Service Temporarily Unavailable
Date: Sun, 09 Jun 2019 07:27:00 GMT
Content-Type: application/json; charset=utf-8
Connection: keep-alive
Content-Length: 58
Server: kong/1.0.0
{"message":"failure to get a peer from the ring-balancer"}
Tried making verify certificate false ,(no luck)
curl -i -X PATCH --url http://localhost:8001/upstreams/postmanEchoSSL --data healthchecks.active.https_verify_certificate=false
Tried making verify certificate true and provided SNI name of server ,(no luck)
curl -i -X PATCH --url http://localhost:8001/upstreams/postmanEchoSSL --data healthchecks.active.https_verify_certificate=true --data healthchecks.active.https_sni=postman-echo.com
Additional Details & Logs

Kong version 1.0.0
Error Logs
2019/06/09 07:40:27 [error] 35#0: *19085 [lua] healthcheck.lua:989: log(): [healthcheck] (postmanEchoSSL) failed to receive status line from '<IP_ADDRESS>:443': closed, context: ngx.timer, client: <IP_ADDRESS>, server: <IP_ADDRESS>:8001
2019/06/09 07:40:27 [error] 35#0: *19085 [lua] healthcheck.lua:989: log(): [healthcheck] (postmanEchoSSL) failed to receive status line from '<IP_ADDRESS>:443': closed, context: ngx.timer, client: <IP_ADDRESS>, server: <IP_ADDRESS>:8001
2019/06/09 07:40:28 [error] 35#0: *19099 [lua] healthcheck.lua:989: log(): [healthcheck] (postmanEchoSSL) failed to receive status line from '<IP_ADDRESS>:443': closed, context: ngx.timer, client: <IP_ADDRESS>, server: <IP_ADDRESS>:8001
2019/06/09 07:40:28 [error] 35#0: *19099 [lua] healthcheck.lua:989: log(): [healthcheck] (postmanEchoSSL) failed to receive status line from '<IP_ADDRESS>:443': closed, context: ngx.timer, client: <IP_ADDRESS>, server: <IP_ADDRESS>:8001
2019/06/09 07:40:30 [error] 36#0: *19117 [lua] healthcheck.lua:989: log(): [healthcheck] (postmanEchoSSL) failed to receive status line from '<IP_ADDRESS>:443': closed, context: ngx.timer
2019/06/09 07:40:30 [error] 36#0: *19117 [lua] healthcheck.lua:989: log(): [healthcheck] (postmanEchoSSL) failed to receive status line from '<IP_ADDRESS>:443': closed, context: ngx.timer

Please let us know, what we are missing here.

@premaind did you have any luck getting this going? i have a similar issue, loading the ca_cert to /ca_certificates has no impact on getting health checks working with verify on. In your example i cant see how you've loaded any cert for kong to verify against?

Dear contributor,
We're closing this issue as there hasn't been any update to it for a long time. If the issue is still relevant in the latest version, please feel free to reopen it. We're more than happy to revisit it again.
Please have a look at our pledge to the community for more information.
Sincerely,
Kong Gateway Team


Bump the dependency on 'snap-core'.

Allows using snap-blaze with the Git version of Snap.

@jaspervdj Ping.

I don't really mind merging this in, but are we sure it is going to work with the upcoming version? It is still possible for the API to change in the git version, no?

Snap 1.0 is basically done, the only thing they want to change API-wise is some command-line processing code. It's highly unlikely that those changes will break snap-blaze.

This change has already been applied in #7. Can be closed.

@wereHamster Thanks for the heads-up!


CI/CD: flaky tests

While running GitHub Action tests for any Pull Request changes, the receiver tests somehow fail irrespective of the codechange made(or documentation changes). Seems like an example of a flaky test -
For Example - Flaky test observed in this #3630
FAIL
FAIL	github.com/thanos-io/thanos/test/e2e	111.932s
?   	github.com/thanos-io/thanos/test/e2e/e2ethanos	[no test files]
FAIL
make: *** [test-e2e] Error 1
Error: Makefile:234: recipe for target 'test-e2e' failed
Error: Process completed with exit code 2.

This is to broad. Let's focus on particular test which is flaky.

I agree with Bartek. The specific case you are pointing to already has issue #2622 that is already closed (by #3798), so I am closing this issue for now as well as I think we don't have anything actionable here now. Let's reopen it if that's not the case.


nixos-install does not check for signature when downloading substitutes?

Issue description
Apparently, roight?!, apparently, nixos-install does not check for signature when downloading substitutes
but nixos-rebuild does check!
Steps to reproduce

boot from an iso (eg. nixos-minimal-16.09pre83100.25e3c09-x86_64-linux.iso)
follow the manual to get to the point to nixos-install
run nixos-install by avoiding the error Curl error 77 like so:

# SSL_CERT_FILE="/etc/ssl/certs/ca-bundle.crt" nixos-install
What you see(no mention of signed):

This is how nixos-rebuild does it(mention of signed):

Technical details

System: (NixOS: nixos-version, Ubuntu/Fedora: lsb_release -a, ...)
Nix version: (run nix-env --version)
Nixpkgs version: (run nix-instantiate --eval '<nixpkgs>' -A lib.nixpkgsVersion)

If this was ever a problem, it should definitely be fixed in newer nix versions.


Text spam in worlds not in condition list

So when I accepted a quest in a valid world, all is good. But if I go to an invalid world thru /mv tp creative. Anytime I do anything (break blocks, right-click blocks, place blocks) it says something along the lines of "You have not met the conditions for [quest]"

You are missing your Quests version. Please follow the issue template next time.
This is not a bug; the player should simply quit the quest. Alternatively, the "Fail the quest" setting can be enabled in the Condition Editor.

can there be an option in config to turn that off then, please? I'd like my players to be able to switch from survival to creative worlds and not be spammed with quests chat, and not have to lose all progress.


Rename outdated to upgrade

Attempt to rename outdated command to upgrade. I've made two commits to ease the review:

4dda1bf4f52f8138b449a48ed8538c99d719818e Actual command renaming
8b9138268eecb6671ce109fc0ac3494674444f95 Renaming files containing outdated

Since mnemonic u is already used by the use command, I've proposed g as the mnemonic for upgrade.
I've kept backwards compatibility for outdated command and its mnemonic o.

Wow, yet another great PR! Thanks so much for your contributions. Please keep them coming!!!

I've applied the code review changes, rebased to master and squashed into a single commit.

Great, thank you. We have a single failing test left. If that is resolved I'll merge and release it for you.

Right, my bad.

Thanks again for the contribution!


Could I use tabular with "Ms SQL"?

I plan to use Ms SQL by node-mssql of npm.
Could I use tabular with "Ms SQL"?
Pl help me.

This plugin only works with Mongo, because is reactive...

Thanks for your reply, have any package on node for this?

If not using Mongo/DDP subscriptions, you really can just use the DataTables package directly with an ajax callback: https://datatables.net/reference/option/ajax


Allow empty Stats.init

2912077
Proposed change
For a convenient way of using Stats - as it uses attribs and returns just as well - it should initialize without any args for __init__ passed.

Or perhaps the fruitful functions from prowler.stats should be moved to the top level and the prowler.stats.Stats class should be renamed to Selector or something like that.


How do I add some basic HTTP authentication?

Is there a built in way, if you don't use a VPN connection?
Like there is with transmission daemon.

This is my current state so far:
version: '3'

services:
  web:
    container_name: jd_container1
    restart: unless-stopped
    image: jlesage/jdownloader-2
    volumes:
      - "/docker/appdata/jdownloader-2:/config:rw"
      - "/home/Downloads:/output:rw"

  auth:
    container_name: basic-auth
    restart: unless-stopped
    image: beevelop/nginx-basic-auth
    ports:
      - "5800:80"
    environment:
      - HTPASSWD=foo:$$apr1$$odHl5EJN$$KbxMfo86Qdve2FH4owePn.
      - FORWARD_PORT=5800
    links:
      - web

But when I try to access JDownloader with name foo and password bar, the page is staying like this (white):

I haven't found anything useful in the logs so far, maybe you have any idea?

The built-in way to perform basic HTTP authentication is to set the VNC_PASSWORD environment variable.


Stream cleanup

Fix #213

These affect performance

Yeah, I see that. Do you know the reason? The changes should have been purely cosmetical?

The changes should have been purely cosmetical?

The local stream allows the compiler to assume there is no aliasing

@sdkrystian knows more

@grisumbras @vinniefalco is correct -- creating the local streams allows for the compiler to assume no aliasing. Without them, it must operate under the assumption that the referenced stream may be modified by writes through aliasable pointer types within the function (or ones deeper within the call stack).

In retrospect maybe we should have left a comment on the declaration for local_stream explaining intent.


ECRRepository is storing ECRRepositoryImage data, rather than ECR-related ones

Upon running Cartography 0.29.1, and ingesting ECR data for the first time I can see that ECRRepository is not actually carrying ECR-related info, but it is actually representing specific images.
The query below returns the full list of images within the repo, rather than info of the repo itself:
MATCH (a:AWSAccount)-[:RESOURCE]->(r:ECRRepository) RETURN a.name, r.name, r.region, r.uri

The following query (which should return a list of images), returns empty:
MATCH (a:AWSAccount)-[:RESOURCE]->(r:ECRRepository)-[:REPO_IMAGE]->(i:ECRRepositoryImage) RETURN *

Paging @ngowraj

That seems really really odd; it confuses me that this test still passes. You might want to manually delete these bad nodes and try again using the latest master since I removed the slower running image scan findings code. If needed I'll release a 0.29.2.

Ok, apparently a case of stale data. Resolved


Trying MessageBox.Avalonia on Raspberry Pi

I'm trying create message box like this:
var messageBoxStandardWindow = MessageBoxManager.GetMessageBoxCustomWindow
(new MessageBoxCustomParams
{
ContentMessage = "Done.",
ContentTitle = "Ок",
ButtonDefinitions = new[] { new ButtonDefinition { Name = "Ok" } },
Topmost = true,
}
).Show();
In Windows new window creates is topmost, in Raspbian it's creates under main window.

try to update all to 11.0 Avalonia and msbox 3.0

I tryed project MessageBox.Avalonia/Exmaple2.0 and yes it's work, problem is solwed. Thanks!


Ghost death flash is in wrong spot when invisible.

Killing ghosts when they're invisible (with or without a weapon) will cause their death sprite flash to occur in their location from the previous turn.

Good catch! I actually had a TODO in the code about this, but forgot to fix it. This should be fixed in v1.0.9.
BTW, since you seem to have been playing this for a while, I'm curious if you have any thoughts about features as well as bugs?

Great! Had a feeling that might be a known issue.
Hard to say what features I'd like to see, it's a such a perfect little game. Thinking on it though, I've played it a bunch on both gameboy and the original pico-8 version and I have never actually finished it. I've made it to floor 9 but never past that. So having a more casual option that starts you with more hearts might be nice.
If adding save functionality is possible a stats screen would be a nice addition. Something that lets me know how many of each monster I've defeated (which might give a hint that the worstlord that chases you can be killed, though that was fun to discover for myself). Furthest floor reached, what difficulties I've beaten, number of wins etc. Could even be something more advanced similar to the book in Spelunky.
More weapons, levels, enemies would be great but I'd definitely be worried about throwing off the balance the game has achieved. I wonder if Krystian has ideas that ended up on the cutting room floor due to time? If there is additional game content added it would be nice to have it as a separate option. Eg. pick between Porklike Classic and Porklike Extended at the title screen.


Add automatic steps / scale calculation

If scale or steps is omitted but not the other, use steps ~= scale * 4.2 to derive the other

Now that we have a collection of samplers to choose from again, this is unfortunately impractical, as the ratio varies too much between samplers and there are other subtleties. Better to just let users choose their scale explicitly, no fiddling.


Update dashboard Kubernetes / Views / Global ( 36 → 37 )

This PR contains the following updates:

Package
Update
Change

Kubernetes / Views / Global
major
36 -> 37

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Edited/Blocked Notification
Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.
You can manually request rebase by checking the rebase/retry box above.
⚠ Warning: custom changes will be lost.


Latest HA 2023.9.0 breaks SolArk Integration

Everything worked fine until I updated to 2023.9.0, reverting to 2023.8.4 allows it to work again.
This error originated from a custom integration.

Logger: custom_components.solark_modbus.hub
Source: custom_components/solark_modbus/hub.py:114
Integration: SolArk Inverter Modbus (documentation, issues)
First occurred: 8:28:47 PM (51 occurrences)
Last logged: 8:32:57 PM

Reading realtime data failed! Inverter is unreachable.

I've scraped the changelog for anything modbus related, two stand out:

Modbus switch, allow restore “unknown” (@janiversen - #99533) (modbus docs) (beta fix)
use write_registers also for target temp (@brunoenten - #97475) (modbus docs)
Bump pymodbus v3.4.1. (@janiversen - #97612) (modbus docs)
modbus: use pb not pymodbus consistently as name. (@janiversen - #97780) (modbus docs)
Fix Flexit mypy error in pymodbus (@janiversen - #97799) (flexit docs)
Handle explicit Modbus NaN values (@joanwa - #90800) (modbus docs)
Replace Float ‘nan’ with None for modbus floats (@String-656 - #93832) (modbus docs)
modbus: remove unused constants and get 100% coverage. (@janiversen - #97779) (modbus docs)
Move all used modbus constants to Stiebel (@joostlek - #98044) (stiebel_eltron docs)
Make changes in modbus trigger a full CI run (@emontnemery - #98055)
Modbus: set state_class etc in slaves. (@janiversen - #98332) (modbus docs)
Address late modbus review (@janiversen - #99123) (modbus docs)
Add possibility to have multiple values for every modbus hvac mode (@escoand - #98829) (modbus docs)
Add modbus test for configuration errors (@janiversen - #98697) (modbus docs)
Modbus switch, allow restore “unknown” (@janiversen - #99533) (modbus docs) (beta fix)

can confirm this problem exists in 2023.9.0. Redownloading from HACS does NOT fix the issue.

You need to ask the custom component to update.

What update is required?  I do not see what exactly has broken.
I saw all these modbus related changes in the changelog for HA and decided to wait a bit before upgrading my system even to 2023.8.4 (many modbus related regressions had to be addressed in the changelog I saw due to changes in 8.0 it appeared to me).
Anyway I have not yet had a chance to investigate and would welcome any input.

Possible regression in 2023.9.0.  Please avoid updating for now and let's see what becomes of this upstream.
https://github.com/home-assistant/core/issues/99784

Looks like regression #1758 in pymodbus 3.5.0 is responsible for this.
Home Assistant 2023.9.1 is slated to move to v3.5.1 to address this.
https://github.com/home-assistant/core/pull/99940
I think there will be a new release soon so please hang on a little longer.
A confirmation by anyone upgrading to 2029.9.1 or later in this thread would be appreciated so I can close this issue.

I upgraded to 2023.9.1 and it is still broken

Same, running 2023.9.1 and I'm still getting a Inverter is unreachable exception at solark_modbus/hub.py:114

Wait for 2023.9.2. It has the actual fix in it.

Hi @pbix,
sorry to chime in, but I thought it might help: for my component the problem was byteorder=Endian.Big. Had to change all those to byteorder=Endian.BIG.
The change in pymodbus 3.5.0 was documented here: https://github.com/pymodbus-dev/pymodbus/blob/dev/API_changes.rst

Hope it helps.

Hi @alexdelprete,
Thanks for the tip.
Did you have to version detect to use lower case for versions prior to 3.5.0 and upper case for later versions?
What a pain.

Hi Paul,

Did you have to version detect to use lower case for versions prior to 3.5.0 and upper case for later versions?

No, I decided to force the use of HA 2023.9.1 and pymodbus 3.5.1 so I could immediately release a working version. I hope to not cause too many problems to users. I'll wait for feedbacks and then, if needed I might implement what you suggested. I saw your code, and I noticed you already check pymodbus version for other things, it's a good idea.

What a pain.

Absolutely, I spent many hours debugging this (mainly because I'm not really a developer) and I found the solution only because I clicked on the right link and when debugging, examining the exception I was receiving, it contained the string "Big". So I associated Big to that Endian case change, and that was definitely it. A little bit of luck and years of experience in troubleshooting IT issues. But you're right: "what a pain..." :)

Update to Main branch allows this integration to operate despite the breaking changes introduced without warning into the pymodbus library.  The SolArk integration should now work with a releases of HA.
Thanks to @alexdelprete for pointing out one of the breaking changes saving me many hours if debug no doubt.  Your integration may benefit from my resolution method as it works with all versions of HA.  Just avoid using the pymodbus Endian module altogether.
Would love to hear from other users that they met success after updating to this release.

I've just updated (redownloaded from HACS to be sure) and the integration still seems broken after restarting HA

Thanks to @alexdelprete for pointing out one of the breaking changes saving me many hours if debug no doubt.

glad to help. I hope that's the only issue we're facing: I have 3 users reporting it worked, but now 1 user tells me it's still not working. :(

I've just updated (redownloaded from HACS to be sure) and the integration still seems broken after restarting HA

Do you have anything in your logs to report?
It is working for me using a TCP connection and HA 2023.9.1.
What type of connections are you using?

I added a log of the ConnectionException that is raised on hub.py:114 and it only says [Connection] Failed to connect[ModbusTcpClient(solark.local:502)] which isn't super helpful, I can confirm that connecting to solark.localdomain port 502 works on my local network though...

Just to be sure I tried adding the device in HA by IP address to rule out any DNS resolution issue and I confirmed mbusb is running on port 502 on the host that is connected to SolArk (this was working and hasn't changed)
I'll wait for other users to report before digging more

I made another update.  Try again.

9d17f3e057b9d5ae29eac01df708e2fa5141e38e worked, thanks!

can confirm it works on 2023.9.1

I made another update. Try again.

Paul, so for the pymodbus issues you dumped the endian constants and then did an explicit connect(), correct?
I already had the explicit connect() in my code, so after fixing the Endian, I guess there's no other things to fix. Correct me if I'm wrong.
Thanks.

Alex,
You are correct.  I was not doing an explicit connect before as it was always automatically done.
A correction for this is in v3.5.2 but I did not want to wait.
https://github.com/pymodbus-dev/pymodbus/pull/1772
For the Endian problem, which was an undocumented breaking change BTW.  I dumped the Endian module and just used the native '>' & '<' characters so as to avoid any backwards compatibility issues.
Hopefully we are set until the next breaking change comes our way.

update: all sensors changed to unknown after about 30 minutes. Rolling back home assistant to 2012.8.4 for now.

Paul,
thanks a lot. For my next major version, I'll be moving from modbus to REST/json. The ABB/FIMER inverters use a card that does Modbus, but we discovered it also supports REST calls and provide json data with all necessary information. Unfortunately the vendor told me the REST API is not documented / official, so they wouldn't provide assistance/information, but luckily we managed to find out how it works.
So long to modbus for me...it was fun learning new things and all the register map parsing, but I don't like to depend on pymodbus etc. This last incident was the classic straw that broke the camel's back. :)
Thanks for your advices.

update: all sensors changed to unknown after about 30 minutes. Rolling back home assistant to 2012.8.4 for now.

@legomind
I have tested both with v2023.9.1 and v2023.8.4 with the latest on github and am unable to reproduce your results.  To simulate a communications error I disconnected and reconnected my serial link cable.  The both version recovered and not errors were shown in the logs.  I am using a TCP connection.
Are you using a serial or TCP connection?
I will need help from you to document a reproducible scenario for me.

I am using a tcp connection. I will update once more tonight and let you know the results & include logs & configs.

Ok exactly 15 minutes after upgrade, the following message is logged, and all sensors changed to unavailable.
I am using a modbus mtu gateway connected via Ethernet cable. What else do you need from me to troubleshoot?
Logger: custom_components.solark_modbus.hub
Source: custom_components/solark_modbus/hub.py:116
Integration: SolArk Inverter Modbus ([documentation](https://github.com/pbix/HA-solark-PV), [issues](https://github.com/pbix/HA-solark-PV/issues))
First occurred: 8:29:01 PM (92 occurrences)
Last logged: 8:36:36 PM

Reading realtime data failed! Inverter is unreachable.

Working overnight on 2023.9.2

Resolved at v 1.3.0


Problem with pabot execution

Bug
I have two problem using pabot with the RobotEyes library.

When I execute the tests using pabot with testlevelsplit then in the pabot_results folder the library generates an unnecessary folder with the test case name -1 character instead of using the existing one named same as the test case.
For example in case of test case name This is a dummy test. The pabot_results will contain two folder: This is a dummy test and This is a dummy tes. The former will contain the output.xml, robot_stderr.out, robot_stdout.out and every screenshot I do using the capture page screenshot keyword, and the latter will contain the visual_images folder.

After the execution finishes I should be able to generate the reports using the reportgen cmd but I would have to call this command for every test case since the visual_images folder is separated for every test case.

To Reproduce
Start the execution using pabot and testlevelsplit
Expected behavior
I want to be able to generate the report using the command only once (all the visual_images in the same folder separated by test cases, just like in case of robot cmd)
Desktop:

OS: Win 10
Browser Chrome
Version 74

@bioneyez The library isn't designed to work with pabot at a testlevel split. Any particular reason as to why you want use a test level split?

@jz-jess Yes. My test cases does not depend on each other so I want to execute them parallelly, but I don't want to make a test suite for each of them.

@bioneyez Implementing support for test level split might take time since I need to understand pabot's implementation of it.
Currently the library gets the test case name and output directory using robot framework's APIs.
Pabot executes each suite individually with a pybot/robot command and stores the results in a temp folder and merges them at the end. Library takes the path till pabot_results folder and then creates the visual_images folder at the parent of pabot_results. The images are then stored in the same fashion as in a sequential execution. I am not sure why the images are getting stored within pabot_results. Fixing this should ideally fix the duplicate test case folder creation issue as well.
I'd suggest that you use the default suite level split until its permanently fixed.

@jz-jess Alright. I'll try to find some time and help out if I can with this issue.

@bioneyez I finally took some time to investigate this. It looks like the issue was with pabot instead of the robot eyes.
I upgraded pabot to 1.8.1 which is the latest. Ran the following command:
pabot --processes 2 --testlevelsplit -d results -v images_dir:base Tests
Everything seems to have run fine for me. The visual_images folder got generated at the root of results folder and not within pabot_results. Inside pabot_results I could see folders 0, 1, 2 ... so on.
Unless there is another way to reproduce this issue, I think testlevelsplit is compatible with RobotEyes.

Closing this since issue is not reproducible and the thread is inactive. Feel free to reopen if needed


Slow rendering performance on scroll

I've been using the last few versions of react-virtual and i love the hook based api. However, I am getting some poor performance on row rendering. It's the classic empty and then flash to content on scroll. I thought perhaps this could be an issue with emotion (which i use for styling), however after stripping away all the emotion styles on the rows, it still performs quite slowly. I have broken everything up quite a bit and made sure that rerenders are localized only to components which have state changes. I've also played around with the overscan quite a bit, and can't seem to find a setting which renders smoothly. For the record, I am also using this with react-table. I am wondering if I am using the keys incorrectly somehow, or if this is just expected behavior. Below I've included some code snippets and a video:
virtualized-table-row.js
import { Flex } from "@chakra-ui/react";
import { Fragment } from "react";

const VirtualizedTableRow = ({ virtualRow, row }) => (
  <Flex
    ref={virtualRow.measureRef}
    h={`${virtualRow.size}px`}
    bg={row.isSelected ? "indigo.50" : "white"}
    _hover={{
      bg: row.isSelected ? "indigo.100" : "gray.100",
    }}
    position="absolute"
    w="100%"
    top="0"
    left="0"
    alignItems="center"
    style={{ transform: `translateY(${virtualRow.start}px)` }}
    minWidth="min-content"
    zIndex="1"
    role="group"
  >
    {row.cells.map((cell) => (
      <Fragment key={cell.getCellProps().key}>{cell.render("Cell")}</Fragment>
    ))}
  </Flex>
);

export default VirtualizedTableRow;

virtualized-table-body.js
/* eslint-disable react/jsx-props-no-spreading */
import { Box, Center, Spinner } from "@chakra-ui/react";
import { useCallback, useEffect } from "react";
import { useVirtual } from "react-virtual";
import VirtualizedTableRow from "./virtualized-table-row";
import VirtualizedTableRows from "./virtualized-table-rows";

const VirtualizedTableBody = ({
  getTableBodyProps,
  prepareRow,
  rows,
  memoizedData,
  container,
  getNextPage = null,
  page,
  scrollYProgress,
  isLoading,
}) => {
  const rowVirtualizer = useVirtual({
    size: rows.length || 5,
    parentRef: container,
    estimateSize: useCallback(() => 42, []),
    overscan: 100,
    keyExtractor: (index) => rows[index]?.id,
    paddingEnd: getNextPage && page > 1 ? 50 : 0,
  });

  console.log("rerender");

  useEffect(
    () =>
      scrollYProgress.onChange((latest) => {
        if (latest === 1 && !isLoading) {
          getNextPage();
        }
      }),
    [getNextPage, isLoading, scrollYProgress]
  );

  return (
    <Box
      {...getTableBodyProps()}
      minWidth="min-content"
      h={`${rowVirtualizer.totalSize}px`}
      position="relative"
    >
      <VirtualizedTableRows
        rowVirtualizer={rowVirtualizer}
        rows={rows}
        memoizedData={memoizedData}
        prepareRow={prepareRow}
      />

      <Box position="absolute" h="50px" w="100%" bg="white" bottom="0">
        {isLoading && page > 1 && <Spinner />}
      </Box>
    </Box>
  );
};

export default VirtualizedTableBody;

https://user-images.githubusercontent.com/8886318/119013404-f13b5e00-b964-11eb-887e-0c479e764ef0.mp4
Thanks again!

Probably not relevant anymore, but I had the same issue with Chakra Table.
When I switched out every Chakra component to a 'normal' html element it rendered the rows much much faster.

@jsbeckr so we mananged to speed up the rendering quite a bit by changing some chakra setting on the popovers we used. Essentially they were being rerendered on each scroll, and we were able to prevent that. I still do have some concerns generally about rows with a lot of information because as of now each cell in a row still gets rerendered by default, even if remaining in view. I did memoize quite a bit of the table to enhance the performance out of the box.

Yeah, this has a lot to do with the components you are rendering and how many you are rendering. We'll keep improving the perf in react-virtual as much as we can, but can't control everything :)
Let us know if you run into more perf problems!

@jsbeckr This was spot on for me... Chakra made everything crazy slow. I guess I won't the using a components library after all, neither MUI, Chakra nor the one from Microsoft... at least not for tables

Does this slow row rendering issue apply to mui Components? For me, List items consist of the Mui timeline component with a bunch of nested Components (TimelineContent, Typography, Box, etc...)

Does this slow row rendering issue apply to mui Components? For me, List items consist of the Mui timeline component with a bunch of nested Components (TimelineContent, Typography, Box, etc...)

It appears so. I'm using a bunch of MUI components in my table (Checkbox, InputBase, Button, Typography, etc.) and can't figure out a way to fix the flickering without removing the MUI components.

I'm experiencing the same. Trying to migrate our custom table component from react virtuoso to tanstack virtual, because tanstack virtual is much more flexible. Performance was perfect in react virtuoso, but with tanstack virtual it is something like in your video.

I'm experiencing the same. Trying to migrate our custom table component from react virtuoso to tanstack virtual, because tanstack virtual is much more flexible. Performance was perfect in react virtuoso, but with tanstack virtual it is something like in your video.

@justasdev tanstack virtual is not doing any performance optimization while scrolling, so on every scroll everything is re-rendered, hard to say what is happening without reading the code, but try to pin point what is slow.

I have the exact same problem. Any guidance on how to handle rendering of MUI components?
Using pure html solves the problem, but that is unfortunately not a viable option as my app is deeply committed to MUI.

Anyone having issues with fixing their scroll flickering or lagging this has done wonders for me https://github.com/TanStack/virtual/issues/619#issuecomment-1969516091

I solved the issue by wrapping every item in React memo. This seems to mitigate unnecessary rerendering on scroll.
Here is a full example for dynamically sized items:
function VirtualList({ data }: { data: ListItemType[] }) { r

    const virtualizer = useWindowVirtualizer({
        count: data.length,
        estimateSize: (i) => 1000,
        overscan: 5,
    })

    const virtualItems = virtualizer.getVirtualItems()

    return (
        <div style={{
            position: 'relative',
            height: `${virtualizer.getTotalSize()}px`,
        }}>
            <div style={{
                position: 'absolute',
                top: 0,
                left: 0,
                width: '100%',
                transform: `translateY(${virtualItems[0]?.start ?? 0}px)`,
            }}>
                {virtualItems.map( (virtualItem) => (

                        <MemoizedListItem data={data[virtualItem.index]} />

                ))}

    )
}

const MemoizedListItem = memo(({ data }: { data: ListItemType[] }) => {
    return (

            <ListItem data={data} />

    )
})

@Torsteinws Good to hear that wrapping items in React.memo helped with the performance! Optimizing virtualized lists is definitely not a trivial task. Performance can be influenced by many factors beyond just code, like hardware capabilities and how quickly the browser can render elements. It's all about finding the right balance for your specific use case, but every bit of optimization helps overall.
Let me know if you come across any other findings!

It seems to me that the virtualizer is triggering a bunch of "unnecessary" rerenderings when scrolling. In my case, it caused me to feel a fair amount of friction when getting started with the library. Perhaps this is something the TanStack Virtual team can consider to address somehow?
If it is difficult to optimize the library in a way that fits all use cases, maybe it is possible to expose an option that somehow let you limit the amount of rerenders? If not, it would go a long way to just document best practices for having a performant virtual list.
Otherwise, kudos for making a great api. The virtualizer feels ergonomic to use.

It seems to me that the virtualizer is triggering a bunch of "unnecessary" rerenderings when scrolling.

@Torsteinws This should be fixed alredy via https://github.com/TanStack/virtual/pull/806 if you can pin point where it happens in latest version, let's fix it.

I have been using version 3.10.8, so I should have this update already.
I wonder if all items in a list are rerendered whenever virtualizer.getVirtualItems() changes. If that is true, I can see why it can be expensive to scroll – especially if the items are a bit heavy to render or if the user scrolls a bit fast.


Logic for filepaths is not foolproof

All helper functions should take the arg filename, which is either a string, or use str(filename.name) if it's a Path. This is then concatenated to the appropriate folder for that function. This helps avoid type errors down the line.

Yeah, we need some uniformity here.


traceur 404 not found

Quick start angular 4 project run with npm start but when I moved files into asp.net mvc 5 project this error appeared :
GET http://localhost:47917/traceur 404 (Not Found)
scheduleTask @ zone.js:1990
ZoneDelegate.scheduleTask @ zone.js:384
Zone.scheduleTask @ zone.js:209
Zone.scheduleMacroTask @ zone.js:232
(anonymous) @ zone.js:2014
.....

Oh! changing "es2015" to "commonjs" inside tsconfig.json solved my problem.
I hope your problem same as my problem because I don't know why traceur error related to many different problems!
See here

This repository is now deprecated. The Angular Quickstart project was a nice starting point for creating Angular applications. Now we recommend using the Angular CLI to create new Angular projects.
Starting from 1 November 2017, all the Angular documentation, at angular.io, is based on the Angular CLI.
Let's get started


Improve the dumping of literals

Guard lit_dump_literals with JERRY_ENABLE_LOG (both in source
and in header).
Change printfs to JERRY_DLOG.
Make lit_dump_literals be called in lit_finalize (it was dead
code).
Remove its duplicate declaration from lit-literal.h

JerryScript-DCO-1.0-Signed-off-by: Akos Kiss<EMAIL_ADDRESS>

LGTM

LGTM


Custom sound...

I've tryed everythig (i think)
Nothing had worked so far...
I've added my sound to /res/raw and tryed with .mp3 and wav
nothing....
Anyone could help?

Hi
Im using the ringtone picker to allow the users to select their own sound. It used to work just fine right up until Android introduced these silly channels and now its a mess. The file path I am using is like this "content://media/internal/audio/media/27"
Will the channel be able to use this or am I being dumb and if so how can I make it work

As far as I know you cannot create channels with this plugin so I installed the push plugin to create channels: https://ionicframework.com/docs/v3/native/push/
this.push.createChannel({
        id: "your-custom-channel-id",
        description: "Your custom channel name",
        sound: "trumpet"
      }).then(() => console.log('Channel created'));

Sound is your audio file name without extension, I use trumpet.mp3. Once the channel as been created (in your app) you can send notification from your server specifying "your-custom-channel-id" and it will play the sound.
The problem is once the channel as been created you cannot change the sound. Even if you delete the channel and create it with a different sound it will keep the first one.
I have only 5 sounds so my solution was to create 5 channels. Based on the user choice I will update the user settings (on the server) so I know which channel to send the notification on.

@DavidSousa  where exactly I should add the code you pasted above??

Is there a solution to this issue??


[Program]: Write a Ruby program to find sum of ap series

Description
Write a Ruby program to find sum of ap series
An arithmetic progression (AP) is a sequence of numbers such that the difference of any two consecutive terms is a constant. The nth term of an AP is given by the formula an = a1 + (n-1)d where a1 is the first term and d is the common difference.
a = 2
n = 5
d = 3

Output : 40

How to contribute

Save the solution in program/FindSumOfApSeries.rb file
Add FindSumOfApSeries.rb file in find-sum-of-ap-series folder

!assign


implement most important API methods

Implements the following API methods

RequestDocs
RequestSubmittedDocs
SendInvoices
CancelInvoice

Codecov Report

Merging #2 (347c1d6) into main (f117410) will decrease coverage by 0.46%.
The diff coverage is 92.06%.

@@            Coverage Diff             @@
##             main       #2      +/-   ##
==========================================
- Coverage   93.44%   92.98%   -0.47%
==========================================
  Files           2        2
  Lines          61      114      +53
==========================================
+ Hits           57      106      +49
- Misses          2        4       +2
- Partials        2        4       +2

Impacted Files
Coverage Δ

pkg/mydata/client.go
92.30% <92.06%> (+0.15%)
:arrow_up:

Continue to review full report at Codecov.

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

Codecov Report

Merging #2 (90c7bf8) into main (f117410) will increase coverage by 0.25%.
The diff coverage is 93.42%.

@@            Coverage Diff             @@
##             main       #2      +/-   ##
==========================================
+ Coverage   93.44%   93.70%   +0.25%
==========================================
  Files           2        3       +1
  Lines          61      127      +66
==========================================
+ Hits           57      119      +62
- Misses          2        4       +2
- Partials        2        4       +2

Impacted Files
Coverage Δ

pkg/mydata/client.go
92.30% <92.06%> (+0.15%)
:arrow_up:

pkg/models/response_doc.go
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


toJsonSchema: Only disallow additionalProperties for strictObject

additionalProperties: true was preventing validation of valid intersected objects.

Fixes #990.

Thank you! I plan to review and merge this PR in the next days.

According to my research, additionalProperties is true by default. So I will remove additionalProperties for the looseObject schema to reduce the bundle size.

Sure, sounds good.


Fix View tests for Windows environments

PhpUnit fails of Windows environments with "core.autocrlf" = "true" because checked out fixtures contain CRLF line endings.

You haven't configured git correctly then.

Making this change makes the tests fail if git is correctly configured...

No, Git isn't configured correctly or incorrectly when the tests fail. The tests are brittle if they depend on such things.
It should be possible to configure this correctly with a .gitattributes file. I don't really know how, though, or I would have fixed this long ago.


Update: Spanish

Strings.es.resx needs to be updated.
@bcien or @pablomh maybe you can assist :)

@pablomh thank you very much! merged! 👍


admin config fix

The admin config page is broken because TaskList has been replaced by QueueLength as explained by @tboerger in the following issue https://github.com/gogits/gogs/issues/1392
This pull request should solve the problem.

The go-bindata needs an update :)

Thanks your PR!


Tutorials: Rust basic dependency tutorial

Hello, I tried to reproduce examples from your guide for rust: https://sdk.dfinity.org/docs/rust-guide/multiply-dependency.html.
First of all this link isn't working https://sdk.dfinity.org/quickstart/quickstart.html#download-and-install
Then I made all steps that were in the instruction, changing paths to my local and after I successfully started the Internet Computer network and called in second terminal "dfx deploy" I received such an error :
'cargo build --target wasm32-unknown-unknown --package  rust_deps --release'
thread 'main' panicked at 'Could not run custom tool.: Os { code: 2, kind: NotFound, message: "No such file or directory" }', src/dfx/src/lib/builders/custom.rs:181:18
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace
Could you please help me to understand what could I do wrong?
Examples with Motoko worked perfect for me, but in case with rust it can't find some directories.
upd. Same error for other examples

Thanks for trying this out and letting us know about the problems you ran into. There have been a lot of changes under the hood and now the tutorials for Rust need to be refreshed. We'll try to get updated docs out soon!

@ililic , any chance you can take a look at this to see if we can track down how to fix the Rust tutorials?

I get similar error:
The post-build step failed for canister 'rrkah-fqaaa-aaaaa-aaaaq-cai' with an embedded error: No such file or directory (os error 2)
and I can't figure out where the issue is from.

$ ls .dfx/local/canisters/idl/: directory exists, but is empty.
=> so canister.rs:282 must  be failing

hello
I'm following the tutorial
https://sdk.dfinity.org/docs/rust-guide/rust-quickstart.html
and have problems to I fixed with this steps
$ rustup install nightly-x86_64-unknown-linux-gnu
$ rustup target add wasm32-unknown-unknown --toolchain nightly
$ rustup run nightly cargo build --target wasm32-unknown-unknown
$ rustup target add wasm32-unknown-unknown
$ dfx deploy

but now I landed in same error that was mentioned by @ggreif
$ dfx deploy
The post-build step failed for canister 'rrkah-fqaaa-aaaaa-aaaaq-cai' with an embedded error: No such file or directory (os error 2)
mario@koakh-laptop:~/Development/DFinity/rust_hello> ls .dfx/local/canisters/idl/
ls: cannot access '.dfx/local/canisters/idl/': No such file or directory
mario@koakh-laptop:~/Development/DFinity/rust_hello> Jun 06 21:44:51.887 WARN s:/n:/ic_crypto_internal_csp/proto_store WARNING: deleting key with ID KeyId(0x8bdba52ef24fe4216783e17e8d5f6542e22e5d3a734086edce80c4b5701fee59) with scope Const:NiDkgThresholdKeys
Jun 06 21:49:11.891 WARN s:/n:/ic_crypto_internal_csp/proto_store WARNING: deleting key with ID KeyId(0xc987f77814fc8881bf80592f074929172191dc512d30028295d656e8a8e4fae2) with scope Const:NiDkgThresholdKeys

any help to solve this problem of '.dfx/local/canisters/idl/': No such file or directory
thanks

Possibly the same issue I reported in #100 with a similar fix to the one I made in #101.

thanks @paulyoung It is working the problem is a wrong file location in file src/rust_hello/rust_hello.did
tutorial says
OK Create a new file in the src/rust_hello/ directory named rust_hello.did.
KO Open the src/rust_hello/src/rust_hello.did file in a text editor.
but it is
OK Create a new file in the src/rust_hello/ directory named
OK Open the src/rust_hello/rust_hello.did file in a text editor.

I had a similar issue when trying https://smartcontracts.org/docs/rust-guide/multiply-dependency.html, and was solved by putting the .did file in the location as specified in my dfx.json.

The post-build step failed for canister 'rrkah-fqaaa-aaaaa-aaaaq-cai' with an embedded error: No such file or directory (os error 2)

This error also occurs if the package name contains a hyphen, e.g. foo-bar, because the generated Wasm file ends up at target/wasm32-unknown-unknown/release/foo_bar.wasm


fixing pg serarch

(and a bit of defensive programming)

All q's are now officially bjsq's

need to check something before merging

ok, should be fine

#1061 duplicate fix


Update for Godot 4

Converted and updated tweening to work in Godot 4

Hey @RobMacKay, thanks for the update, everything seems fine

Hey @RobMacKay, thanks for the update, everything seems fine

No probs - thought it would be nice to update a great little template :-)


Web page goes away when refresh page after login

I am using implicit flow and can login to the identity server and get redirected back to my SPA. However, I discovered that if you refresh my SPA the page just goes blank with no errors and/or there is not token anymore. I also have auth guards.
export const authConfig: AuthConfig = {
// Url of the Identity Provider
issuer: environment.identityServerUri,
clientId: 'cds',
scope: 'unifi openid profile',
requireHttps: false,
redirectUri: window.location.origin,
responseType: 'token',
showDebugInformation: true,
sessionChecksEnabled: false
};
this.oauthService.configure(authConfig);
this.oauthService.tokenValidationHandler = new JwksValidationHandler();
this.oauthService.loadDiscoveryDocumentAndLogin()
.then(() =>
{
if (!this.oauthService.hasValidAccessToken())
{
// No valid token login to identity server
this.oauthService.initImplicitFlow()
this.oauthService.setupAutomaticSilentRefresh();
}
})
.catch(err =>
{
console.error(err);
});
canActivate(route: ActivatedRouteSnapshot, state: RouterStateSnapshot): boolean
{
if (this.oauthService.hasValidAccessToken())
{
return true;
}
// Navigate back to root so identity server logs in
this.router.navigate(['']);
return false;

}
I am using Angular 6.12 and angular-oauth2-oidc 4.0.2
Thanks
Ken

It would help if you edit your post and use the markdown help and preview to format the code block (use three backticks above/below a block to format it as code).
Skimming your code as is, I'd guess that your guard is called before the ...AndLogin(...) starts running (which will run asynch, after a delay caused by loading the disco document).

He @jeroenheijmans, I am not exactly sure what you mean by 'a delay caused by loading the disco document'. Everything works great. It is only when I refresh the page that things go haywire. I also forgot to mention I set the initialNavigation to false when defining my routes.

I could really use some additional help here.
Thanks in advance.
Ken

Same issue here ... if I F5 the page I get :
Failed to load https://xxx.xxx-web.xxx-csc.xxx/acv-be/.well-known/openid-configuration: Response for preflight is invalid (redirect)

I'm seeing a similar problem when using "ng serve" in development mode. We normally run ng serve to host the angular application. Now that we've incorporated angular-oauth2-oidc, whenever a change is made to a .ts file, ng serve automatically rebuilds the app and the browser refreshes. But this leads to a blank page. I can't tell if it's this package or the routing table or the auth guard....

So I have a solution (or at least a work-around?). My app.component configures the auth service from its constructor and started with a very basic approach, something similar to this pseudo-ish code:
this.oauthService.configure(authConfig);
this.oauthService.tokenValidationHandler = new JwksValidationHandler();
this.oauthService.loadDiscoveryDocumentAndLogin();
this.oauthService.initImplicitFlow();

We then tried testing whether we had a token and would redirect accordingly:
if (!this.oauthService.hasValidAccessToken()) {
    this.oauthService.initImplicitFlow();
} else {
    this.router.navigate(['']);
}

hoping that the routing table would take care of it but that wasn't enough and there didn't seem to be a way of knowing where we were prior to a source code file change (from ng serve) or a browser refresh with F5.
Finally, I added a hook into the NavigateEnd event to capture the location so I could restore it on the next instantiation so the constructor became:
router.events.subscribe(event => {
    if (event instanceof NavigationEnd) {
        sessionStorage.setItem('activeUri', window.location.pathname);
    }});

this.oauthService.configure(authConfig);
this.oauthService.tokenValidationHandler = new JwksValidationHandler();

this.oauthService.loadDiscoveryDocumentAndLogin().then(() => {
    if (!this.oauthService.hasValidAccessToken()) {
        this.oauthService.initImplicitFlow();
    } else {
        // here's the important bit ---
        let uri = sessionStorage.getItem('activeUri');
        if (uri !== null) {
            this.router.navigate([uri], { skipLocationChange: true });
	}
    }})	.catch(err => {
        console.error(err);
    });

It would be great if angular-oauth2-oidc could provide something like this since it's such a common use case.

Did the follow-up comments give enough solutions and workarounds? Or should the issue remain open?

Why was this closed?  Is there a best practice for angular-oauth2-oidc surviving a F5 or shift-F5?

I closed it for a few reasons:

activity died out and I got no response to my latest comment
the original post was really hard to follow
the original poster stopped interacting
another user posted their solution
I posted my solution and linked to an extensive sample repository which you can clone and just run, where the logged in state survives a refresh

For extensive details, please check my example.
To reiterate the short version here: both providing custom storage (localStorage or sessionStorage) and trying silent-refresh on startup will work to keep the user logged in after a page reload.
If there's still open issues, don't hesitate to open a fresh issue or Stack Overflow question, adding the details of your situation.


on disk caching weird behavior when I want to support ASNetworkImageNode

I want to add support for ASNetworkImageNode to use AlamofireImage to downloading image, but

folder on top is my project, the folder Hierarchy is not right ,
below is AlamofireImage demo's folder Hierarchy
It makes AlamofireImage can not load image from disk
just like https://github.com/Alamofire/AlamofireImage/issues/22

It just work fine when I start a new project, using the extension.
But in my project, It just appears like the screenshot

Now configure it out ,It was problem when using with SDWebImage

using Alamofire before AlamofireImage cause the abnormal folder Hierarchy

Is there  something conflict with AlaofireImage's cache, when I turn off Internet, can not load cache form disk?
what may cause this?
requestCachePolicy is UseProtocolCachePolicy
FAILURE: Error Domain=NSURLErrorDomain Code=-1009 "The Internet connection appears to be offline." UserInfo={NSUnderlyingError=0x7ffafa29edf0 {Error Domain=kCFErrorDomainCFNetwork Code=-1009 "(null)" UserInfo={_kCFStreamErrorCodeKey=8, _kCFStreamErrorDomainKey=12}}, NSErrorFailingURLStringKey=http://7xkdk4.com2.z0.glb.qiniucdn.com/pics/vol/556d584be2aaa.jpg?imageView2/1/w/640/h/452/q/50, NSErrorFailingURLKey=http://7xkdk4.com2.z0.glb.qiniucdn.com/pics/vol/556d584be2aaa.jpg?imageView2/1/w/640/h/452/q/50, _kCFStreamErrorDomainKey=12, _kCFStreamErrorCodeKey=8, NSLocalizedDescription=The Internet connection appears to be offline.}


Firefly is missing identity details, causing all calls to the firefly contract to be rejected

Hi, I am trying to run Firefly with a Besu network, however, when I first call http://localhost:5000/api/v1/network/organizations/self?confirm=true, the request times out. The block browser indicates that the call to the contract was successful, however the FF logs indicate that the result was rejected because the caller key was not recognized.
[2023-06-28T09:47:39.907Z] ERROR Ignoring network action batch_pin from unknown identity 0x66f683f75f726e1d47321d8d0ff3775b5546ee33 ns=default pid=1 role=event-manager

I have reproduced this error over and over again while trying different configurations. However I had an almost exact replica of this setup running on the Polygon Mumbai testnet about a month ago, so I believe this may be caused by a fairly recent regression.
Steps to reproduce

Setup a dev Besu network (npx quorum-dev-quickstart, default options, no private transactions)
Ensure Besu is started with free gas by checking the QBFT genesis json and adding  "zeroBaseFee": true if it is missing
Start the Besu network and deploy the Firefly contract using one of the default Besu wallets from the genesis json.
Run ff init with the following options

ff init ethereum sdx 1 \
 --multiparty=true \
 -n remote-rpc \
 --remote-node-url http://host.docker.internal:8545 \
 --chain-id 1337 \
 --contract-address ${Contract address from step 2}

Start ff, and POST to /api/v1/network/organizations/self?confirm=true

Expected
The org identity should be claimed, and I should be able to claim the network identity next.
Actual result
The transactions executes successfully on the chain, however when the result is returned, it is rejected as being from an unrecognized identity. Nothing is returned from Firefly and the POST times out after 2 minutes.
Logging into the ff-core container and running select * from identities; shows that there are no records at all in the table.
Extra thanks to @matthew1001 who has been helping me debug this error and I believe he has also reproduced it.

Thanks for raising this @sapient Yeah I can recreate the same behaviour locally and one of the differences in the logs between the working case (where identities are registered by the ff-cli) and the failing case (where identities are registered manually) is that on processing the batch pin event, in the successful case we see:
[2023-06-29T13:51:51.191Z] DEBUG Verifier '0x44d2885bec70019d7af1f0cb83190bb070da14a2' not found dbtx=CXBrMDu5 ns=default pid=1 role=aggregator
[2023-06-29T13:51:51.191Z]  INFO Processing system definition 'ff_identity_claim' [6f611455-3b81-4ee7-98c3-64e1320ca0d5] dbtx=CXBrMDu5 ns=default pid=1 role=aggregator

So the verifier isn't found (because it's currently being registered) but there is a call showing that it is being treated as a system definition, which ultimately results in the identity and verifier being added to the DB:
[2023-06-29T13:51:51.195Z] DEBUG SQL-> insert verifiers dbtx=CXBrMDu5 ns=default pid=1 role=aggregator
[2023-06-29T13:51:51.196Z] DEBUG SQL<- insert verifiers sequences=[1] dbtx=CXBrMDu5 ns=default pid=1 role=aggregator
[2023-06-29T13:51:51.196Z] DEBUG SQL-> insert identities dbtx=CXBrMDu5 ns=default pid=1 role=aggregator
[2023-06-29T13:51:51.197Z] DEBUG SQL<- insert identities sequences=[1] dbtx=CXBrMDu5 ns=default pid=1 role=aggregator

In the failing case we simply see:
[2023-06-29T14:16:06.961Z] DEBUG Verifier '0x3c584f63cc838935a4626e07a1ddf90c8df31a1d' not found ns=default pid=1 role=event-manager
[2023-06-29T14:16:06.961Z] ERROR Ignoring network action batch_pin from unknown identity 0x3c584f63cc838935a4626e07a1ddf90c8df31a1d ns=default pid=1 role=event-manager

@sapient Thanks for opening this. Definitely doesn't sound like the expected behavior to me. Is this running on the latest release, or on a newer build from main?
Does the log message: [2023-06-28T09:47:39.907Z] ERROR Ignoring network action batch_pin from unknown identity 0x66f683f75f726e1d47321d8d0ff3775b5546ee33 ns=default pid=1 role=event-manager appear on the FireFly node that is attempting to register, or on another node?
@matthew1001 The only reason the FireFly CLI doesn't automatically register identities on a remote chain, is that it doesn't want to assume that the price of gas is free on that chain. If it is not, you may not have had a chance to add any tokens to pay for gas yet at that point during the startup.

@nguyer The message is appearing on the node which is attempting to register the identity. As for version, I believe its just the newest main version.
❯ ff version
{
  "Version": "v1.2.1",
  "License": "Apache-2.0"
}

This seems to be fixed by the above PR linked, closing for now


Kingfisher Image Cache

I've created a wrapper around Kingfisher to forward ASNetworkImageNode's delegate calls to it. It will improve UI as user won't see "gray squares" so frequently anymore.

How is this in terms of performance?

@RamonGilabert can't feel the difference. Without network connection it shows normal images, not grey squares.
Probably, a bit faster as now there is no need to waste time on networking. But no performance "kick".
Should help users which are on mobile, not Wi-Fi


Support for PettingZoo?

Hi,
I would like to ask, are there any plans to make this library compatible with PettingZoo? I've read the overview and docs but I can't seem to find any reference to that at all, and it's a shame since PZ would work extremely well (in principle) with another dedicated MARL library.

AFAIK, cooperative MARL has the same interface as OpenAI's Gym (which is the bigger project here). You can easily define a tuple of observations (one tuple element for each agent), actions, etc - and a single reward (because it's cooperative). That's identical to Gym. I don't see the reason for supporting a redundant API but we will be happy to merge a wrapper if someone wants to contribute it.

I do not believe that supporting easy interdependency with PettingZoo, one of the most well-supported libraries for MARL environments "redundant". Requiring potential users to dabble in low-level environment manipulation to conform to a Gym specification and not having comprehensive documentation is problematic for wider adoption, in my view.

Hi! Sorry for the late follow-up.
There have been several difficulties of an improved integration of PettingZoo due to backwards incompatible changes introduced in later gym and more recently gymnasium versions which PettingZoo relies and that were not compatible with EPyMARL and several of our environments that relies on the outdated gym==0.21 version.
That being said, I am very glad that we just published a big update to EPyMARL including lots of new features and a move to gymnasium from gym (as well as support for training in non-common-reward games, weights and biases logging, and more)! As part of this move to gymnasium, we could now easily integrate PettingZoo environments. I have tested it with MPE environments of PettingZoo as well as some simpler environments and it seems to work well. Note, that continuous control tasks will not be supported.
I will close this issue since PettingZoo integration now works, but please let us know if there are any outstanding issues!


jupyterlab not working, though the install proceeded without complaint

[x] I am on the latest Poetry version.
[x] I have searched the issues of this repo and believe that this is not a duplicate.
[x] If an exception occurs when executing a command, I executed it again in debug mode (-vvv option).

OS version and name: macOS Mojave 10.14.6
Poetry version: 0.12.17
Link of a Gist with the contents of your pyproject.toml file: https://gist.github.com/philipobrien/3153875e64cee2a87d00918ac8e734cb

Issue
I have just tried out poetry for the first time, having been using pipenv previously. Unfortunately I have run straight into an issue that wasn't occuring when performing the same steps with pipenv.
Basically, I installed jupyter lab, the install proceeded without issue, but when attempting to kickstart it using jupyter lab from within the poetry shell I'm getting
[C 16:19:57.384 LabApp] Bad config encountered during initialization:
[C 16:19:57.384 LabApp] The 'contents_manager_class' trait of <jupyterlab.labapp.LabApp object at 0x1091af2d0> instance must be a type, but 'jupytext.TextFileContentsManager' could not be imported

I'm not seeing this at all with pipenv so I'm presuming it's poetry related, though I donb't even know how to go about confirming that.

Apologies, you can close this. I have discovered that it is nothing to do with poetry. There was a missing install of the jupytext package that my global jupyter config was expecting. All it took was a poetry add jupytext and it works perfectly.


Added tri-state checkboxes to conform to Emacs' org mode checkboxes

Some internal API changes as a result, but now a checkbox list performs as Emacs' orgmode checkboxes do.
This fixes an orgmode file import issue from my emacs install to sublime text where indeterminate checkboxes (a checkbox is marked as indeterminate "[-]" to indicate not all children are checked) are present.  As well as fixing the import, I've also ported the functionality of indeterminate checkboxes into checkbox lists.
Behaviour:

Added indeterminate state checkboxes for situations where some child checkboxes are checked, but not all.
Checking a child checkbox where all children are already unchecked will cause the parent checkbox to be marked as indeterminate.
Checking a child checkbox where there are already children checked (but not all) will cause the parent to remain indeterminate.
An indeterminate state checkbox will not count as checked, and so doesn't count as such in summaries.
Toggling an indeterminate checkbox will check it (and all children).

I'm not a python programmer (C/C++ is more my thing), so this may be a little un-python-like, sorry.

Looks nice, will try find time to test it =)

Better late then never, huh =)

Good things come to those that wait 😃


AMBARI-25935: Upgrade Ambari Infra Solr to version 8 and ensure it is…

… consistent with the Solr version in Ambari 2.8 stack
What changes were proposed in this pull request?
Remove the dependency on "fileupload" as compared to Solr 7, since Solr 8 no longer depends on "fileupload".
Remove the copying of "lucene6-core-jar" in the Infra client, since Solr 8's "lib" directory includes "lucene8-core".
Solr 8 uses lucene8.
How was this patch tested?
manual test

Please review Ambari Contributing Guide before opening a pull request.

@JiaLiangC
Thank you for your work, but I need you to resolve a conflict.

@JiaLiangC Thank you for your work, but I need you to resolve a conflict.

@timyuer ok , i will work on it


Recommended changes resulting from automated audit

We performed an automated audit of your Cake addin and found that it does not follow all the best practices.
We encourage you to make the following modifications:

[x] You are currently referencing Cake.Core 0.27.1. Please upgrade to 0.28.0
[x] Your addin should target netstandard2.0. Please note that there is no need to multi-target, netstandard2.0 is sufficient.
[x] The nuget package for your addin should use the cake-contrib icon. Specifically, your addin's .csproj should have a line like this: <PackageIconUrl>https://cdn.rawgit.com/cake-contrib/graphics/a5cf0f881c390650144b2243ae551d5b9f836196/png/cake-contrib-medium.png</PackageIconUrl>.

Apologies if this is already being worked on, or if there are existing open issues, this issue was created based on what is currently published for this package on NuGet.org and in the project on github.

Implemented just now. Will be releases soon


Issues with Show-PodeWebColumn and Hide-PodeWebColumn

Hello everyone,
I just tried the examples for the Show-PodeWebColumn and Hide-PodeWebColumn commands with the latest Pode.Web module.
It doesn't work.
Best regards,
Thomas

The examples have the wrong function name in the button's scriptBlock. They are missing the word table, so should be changed to Hide-PodeWebTableColumn and Show-PodeWebTableColumn

It works, thanks for your help.


SPIKE: Design production ready ROSA compatible SCPs and deployment workflow

Describe the issue
The current SCPs that were modified for ROSA are not production ready. Design a set of SCPs and a deployment workflow that is production ready (PBMM compliant)
Additional context

The current ROSA SCPs were modified to get the installer to work without much thought towards compliance
We are assuming that this is the solution, Ryan J might have another idea that expands the scope

Definition of done

Meet with Ryan J and document any PBMM violations due to the current ROSA SCPs
Work with Ryan J to design new SCPs that allow ROSA to be deployed and are still PBMM compliant
Document any resources such as AWS roles that would need to be created by the ASEA or by the BC Gov automation as part of the account provisioning
Document any violations

We might not need to do this based on new info from the RedHat Black Belt team (Warren investigating, needs to be validated though)

Here is a link to the blog the outlines the ROSA deployment without modified SCPs
https://mobb.ninja/docs/rosa/rosa-asea-landing-zone/


Aws support

Some utilities to handle CloudTrailEvents easier.

after clarify the question, the PR can be merged.


FR: Display string as "UTF-8"

What is the feature you'd like to have?
When a library contains strings of non-English words (in stack, in global variable .bss, etc), when convert them using key a, it transformed into python bytes representation (like \xAB\xCD), I want to add an option to "Display as" menu to transform them into UTF-8 string representation. (And possibly add an option that "If I press a, force to convert to UTF-8 string rather than python bytes" in settings)

Sorry, I misunderstood the issue. This should already be supported - do you have the relevant blocks in analysis.unicode.blocks?

Thanks, I just searched "utf-8" and see the checkbox, but missed the Unicode blocks option.

Good point -- I just added "UTF" to the description of that setting to make it easier to find.


Library needs to be updated to latest ThreeJS version

Getting the error: "Octree.js:2094 Uncaught TypeError: Cannot read property 'faces' of undefined".
I'm pretty sure this is because the library is out of date.
My temporary fix (to help anyone else affected by this) is to change line 2085 to be:
        if ( object.object instanceof THREE.Object3D ) {             return this.intersectOctreeObject(object.object, recursive);
The "return this.intersectOctreeObject" forces it to keep recursing until it finds the type of non-octree object it's looking for.
Anyways, thought I'd let you know. I'd also like an updated version of the library in case I run into some weird edge cases in the future.

Line 2085 is https://github.com/collinhover/threeoctree/blob/master/threeoctree.js#L2085. Are you sure you're using the latest version?
/ping @Wilt you're the most familiar with the latest THREE + Octree, any thoughts?

Mine is also "threeoctree.js (r60)" so I assume I am.
I did copy the raw file text and paste it into a new js file though (as opposed to downloading it), so maybe some things in the ifle we're moved around when I did that.
In the file you linked, the line I changed was 2150.

The fix needs to check for intersections on the THREE object. I believe this line is the problem: https://github.com/collinhover/threeoctree/blob/master/threeoctree.js#L2158. Internally THREE must have changed how the geometry/faces are accessed on an object.

Probably you have a buffer geometry and buffer geometries don't have faces.

I was working on adding support for buffer geometries but I have lots of other work waiting for me...

I am using buffered geometry, you're right. In the meantime hopefully people will see my hacky fix and use that.


Suggestion : Plymouth boot splash

I Suggest adding in Plymouth for a fancy boot splash screen and maybe throwing in your own custom watermark logo/image that says like ArchTitus or something cool

Would be very cool, I'll do this in the future.


Ruby 3.2.0preview2

test against Ruby 3.2.0preview2 via CI
see
changes

This PR was used simply to test changes against the nightly cron tests. The actual work went into #1436, which has now been merged.


[ci] release

This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated.
Releases
<EMAIL_ADDRESS>Patch Changes

#101 6a2c0df Thanks @TheOtterlord! - Add better error messages for starlight config

size-limit report 📦

Path
Size

/index.html
9.53 KB (0%)

/_astro/*.js
16.51 KB (0%)

/_astro/*.css
9.11 KB (0%)


TLS certificate error in containers

Description
@marcverhagen brought this up when we met yesterday, so I did some investigation.
The problem is that when you build some CLAMS app image, you get some kind of TLS/SSL certificate errors that prevent the image from being built.
After some digging into this rabbit hole, I found that when you run clamsproject/clams-python-based images on Apple silicon, the CA certificates comes with the debian image (basis for clams-python) does not verify your connection to pypi.org or other websites.
(I don't know if the the certificate issue Marc has encountered in the past is the same kind of issue, though.)
Reproduction steps
Run this script;
if type -fP docker 2>/dev/null; then
    export dk=docker
else
    export dk=podman
fi
printf "\n\n=== %s (%s %s @ %s): %s\n" "$(hostname)" "$(uname -s)" "$(uname -m)" "$(uname -r)" "$($dk --version)"
for i in python:3.8-slim-buster ghcr.io/clamsproject/clams-python:0.5.3 docker.io/clamsproject/clams-python:0.5.2 ; do
    $dk rmi $i ; $dk run --rm $i pip install dateutils
done

It will try to install a simple pypi package using three different images, and only one of them (python:3.8-slim-buster) is built for x64 AND arm64. Hence, if you run the script on x64/x86 machines, it'll be okay. But on arm-based machines (such as Apple silicon), the second and third images are either unavailable to pull or failing to run pip install because of SSLCertVerificationError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] ...) errors.
Expected behavior
No response
Screenshots
![DESCRIPTION](LINK.png)

Browsers
No response
OS
No response
Additional context
No response

To fix this, I suggest we additionally build and push multi-platform images.


plot subsetted rotated array

Codecov Report

Merging #377 (f5ed4cf) into main (ae34e83) will decrease coverage by 0.33%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #377      +/-   ##
==========================================
- Coverage   79.33%   79.00%   -0.33%
==========================================
  Files          38       38
  Lines        3135     3163      +28
==========================================
+ Hits         2487     2499      +12
- Misses        648      664      +16

Impacted Files
Coverage Δ

src/sources/rasterdatasources.jl
32.87% <0.00%> (+6.21%)
:arrow_up:

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


Make logging cleaner, more compatible

This PR cleans up the logging to a file, and has it using the normal, Craft compatible error logging via:

Craft::trace()
Craft::debug()
Craft::info()
Craft::warning()
Craft::error()

...rather than the cumbersome:
use yii\log\Logger;

Craft::getLogger()->log('Backtrack limit was exhausted!', Logger::LEVEL_ERROR, 'sprig');

You just do:
Craft::error('Backtrack limit was exhausted!', __METHOD__);

...and it all "just works" in exactly the same log format, to exactly the same file as before. It's just a single line change where the logger is allocated:
                'categories' => ['putyourlightson\sprig\*'],

Thanks!


Support Running Linter Command Multiple Times

The linter command will try to checkout the branch each time it is run, which causes the following crash:
Fetching remote branch "***"
From ***
 * branch            *** -> FETCH_HEAD
Switching to the "**" branch
fatal: Cannot force update the current branch.
Error: Error: Command failed: git branch --force *** --track origin/***
fatal: Cannot force update the current branch.

/home/runner/work/_actions/wearerequired/lint-action/v1/dist/index.js:1108
	throw new Error(`Exiting because of unhandled promise rejection`);
	^

Error: Exiting because of unhandled promise rejection
    at process.<anonymous> (/home/runner/work/_actions/wearerequired/lint-action/v1/dist/index.js:1108:8)
    at process.emit (events.js:210:5)
    at processPromiseRejections (internal/process/promises.js:201:33)
    at processTicksAndRejections (internal/process/task_queues.js:94:32)```

However, this means that I cannot run the command multiple times per action.
I would like to do this because I have a mono-repo setup where each sub-folder has its own linting config that must be run. This means that I have to maintain 3-4 different files for linting instead of relying on one file to run the various linters.

@vpillinger, were you able to find a solution for this issue? I am also wanting to run wearerequired/lint-action twice:
1st - Prettier with auto_fix targeting specific files.
2nd - ESlint, Prettier, Stylint without auto_fix targeting the whole codebase.


Fix/docstring warnings 2

This PR fixes mostly indentation errors in several doc-strings. The content of the documentation is not affected.
Before:
when building the documentation with:
cd doc && make html

miscellaneous warnings appeared, most of them indentation errors.
With this PR:

fixed indentations

Only 4 Warning  remain:
elephant/doc/reference/spade.rst:13: WARNING: duplicate label "Stella, 2019" for keys "citations-Stella2019_104022" and "spade-Stella2019_104022"
elephant/doc/reference/spike_train_generation.rst:43: WARNING: duplicate label "Staude, 2010" for keys "cubic-Staude2010_327" and "generation-Staude2010_327"
elephant/doc/reference/spike_train_surrogates.rst:12: WARNING: duplicate label "Louis, 2010" for keys "surrogates-Louis2010_127" and "surrogates-Louis2010_359"
elephant/doc/reference/unitary_event_analysis.rst:20: WARNING: duplicate label "Gr{\"u}n, 2002" for keys "unitary_event_analysis-Gruen02_43" and "unitary_event_analysis-Gruen02_81"

See log: Warnings_log_8_make html_after_fix.txt

Coverage: 87.662% (-0.3%) from 87.996% when pulling 54f88f3cd5651f2987e710be62a638bb4f8ffbd0 on INM-6:fix/docstring_warnings into a8aab43c8e5bbef20613f713f147713ac3557a0e on NeuralEnsemble:master.


[login page] design descrepancies

Environment
iPhone 8 / browser: Safari
Steps to repro

Access https://develop--frontend.preview.coucher.org/
Click "Log in" button

Expected results

Actual results

missing "Password" field
missing "-- or --" separator
missing "Login with facebook" and "Login with google" buttons
"No account yet? Sign up" link is hidden by unnecessary spacing (need to scroll down to render it visible)

missing "Password" field

This is intentional, since not all users have a password set. Pressing "continue" checks that and shows the password field if they have one.

missing "-- or --" separator
missing "Login with facebook" and "Login with google" buttons

These are irrelevant for now as we aren't supporting Facebook/Google auth for the time being.
So the only bug here as far as I'm concerned is the last point.

Fair enough, thanks for looking into this @darrenvong !

Duplicate of #966

@thdk good spot! I'll close the other one since I commented here so whoever works on this knows to only the fix the scroll problem

This was fixed


Kangaroos, boxer improvements, boxing bell and hamlet updates.

video will be uploaded soon

Resprites kangaroo and space-kangaroo
Added space kangaroo salvage and ordinary spawners.
Removed the boxing kangaroo sprites, now kangaroos can be equipped with boxing gloves.
Breath masks and oxygen tanks for kangaroos and hamlet
Fixed a pixel in hamlets sprite.
Added new bounce sound effect for kangaroo movement and kangaroo grunt for kangaroo scream.
Added a kangaroo livestock crate that can be ordered through cargo.
Added a kangaroo pet spawner (willow).
Added a new wallmounted boxing bell, clicking on the bell makes the boxing ding ding sound.
Boxer now spawns with the championship belt and shoes.
Boxer now has maintenance access.

Better make a spawner of skippies that come with the gloves equipped

Better make a spawner of skippies that come with the gloves equipped

the pet comes with them equipped, rest dont but u can equip them

space hampter instamerge

Better make a spawner of skippies that come with the gloves equipped

the pet comes with them equipped, rest dont but u can equip them. I searched to see if the boxing kangaroo was in use anywhere and it didnt look like it

If I want boxing kangaroo swarm, I don't want to equip all of them manually.

Better make a spawner of skippies that come with the gloves equipped

the pet comes with them equipped, rest dont but u can equip them. I searched to see if the boxing kangaroo was in use anywhere and it didnt look like it

If I want boxing kangaroo swarm, I don't want to equip all of them manually.

good point, i will make a spawner just for a kangaroo with boxing gloves

do you want them to be agressive? i assume so?


Fix elapsed log option

oops, sorry for the delay here, nice catch! I see an error given the elapsed time can be higher than zero, will patch after merge


Storage heavy packages (around 12 packages) are failing because of missing space

Potential solutions:

Install without DEV code?

Deleting large npm files when we don't need them?

Attach EFS to Lambda ?

Move to ECS?

@RomainMuller can you please update the PR which is in review to the ticket?

PR is #261  (now shipped)


Translation / Localization Guide

Hey, its me again.
It would be great if there is a guide about adding localization.
Thanks.

Hey, its me again.
It would be great if there is a guide about adding localization.
Thanks.

hi @FloorD this is a good idea, can you perhaps add a file for guidance?

I am about to start localizing in Dutch for the 1-getting started lessons section.
@hexatester / @Fortenberryles41 there are translations folders and these are suitable for localized content. The files in there are named README..md where country code is a two letter abbreviation (ISO_3166-1_alpha-2 standard).

@marcduiker would you mind popping that into a markdown file and linking it from the main README? thanks!

super, thank you!!


Need function learning_rate_plot() which can be used to tune model by user

We need a function learning_rate_plot() which provides a user with a plot for diagnosing the learning rate. We also need directions in the vignette that shows the user how to use this to tune their model.

Implemented with autoPLIER.plot


Fixed: Re-newed import/export request failed immediately if the previous failed

Motivation and context
Depends on #8648
To reproduce import annotations to a job for example:
First import must be failed (e.g. wrong archive structure)
Second import must have correct data, but also will be failed
How has this been tested?

Checklist

[x] I submit my changes into the develop branch
[x] I have created a changelog fragment
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] I have linked related issues (see GitHub docs)
[ ] I have increased versions of npm packages if it is necessary
(cvat-canvas,
cvat-core,
cvat-data and
cvat-ui)

License

[x] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.

Summary by CodeRabbit

New Features

Introduced enhanced import/export functionality, allowing for better handling of requests and progress updates.
Added new functions for importing datasets and backups with improved parameter handling.
Added a new export, StorageLocation, expanding the API surface.

Bug Fixes

Resolved issues preventing successful retries of import/export requests after failures.
Fixed an error related to accessing undefined properties, improving application stability.

Documentation

Added comments to clarify the purpose and usage of new functions in import/export actions.

Style

Minor formatting improvements for better readability in the codebase.

Codecov Report
Attention: Patch coverage is 60.60606% with 26 lines in your changes missing coverage. Please review.

Project coverage is 74.22%. Comparing base (6a60642) to head (3efcb71).

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #8649      +/-   ##
===========================================
- Coverage    74.30%   74.22%   -0.09%
===========================================
  Files          401      401
  Lines        43421    43425       +4
  Branches      3951     3948       -3
===========================================
- Hits         32265    32231      -34
- Misses       11156    11194      +38

Components
Coverage Δ

cvat-ui
78.46% <60.60%> (-0.19%)
:arrow_down:

cvat-server
70.58% <ø> (ø)


Add L7 Antrea native NetworkPolicy datapath support

This PR depends on #4380 #4406
Signed-off-by: Hongliang Liu<EMAIL_ADDRESS>

Could you help review this PR? @tnqn @qiyueyao  I have polished the code and I'll add e2e tests in this PR later.

/test-all

and subsequent rules will not be considered.

/test-all

/skip-all which have succeeded


Is http://www.ngdc.noaa.gov/geoportal/csw down?

http://www.ngdc.noaa.gov/geoportal/csw seems to be down.
I am getting:
Service Temporarily Unavailable

The server is temporarily unable to service your request due to maintenance
downtime or capacity problems. Please try again later.

Its back.


Servo/Event Ejector

Implement a servo and event ejector for ball devices

My vote would be event ejector only, since it would cover all servo use cases plus others too i think? There could be separate events for eject and retry eject, and anything more complex could be handled with an event player or show?

Event ejector done and tested. Servo ejector should be possible with events.


Make the Stratis Core accept command-line arguments in Dev mode

Something went wrong with the builds. Rebuilding.


Allow System.AccessToken (OAuth 2.0 Token) to be used to register an agent

The Agent registration mechanism currently supports quite a few registration possibilities such as PAT.
VSTS/ADOS leverages an OAuth 2.0 authentication token to be used in scripts (the System.AccessToken environment variable); a result of a successful authentication by the platform.
The token has the lifetime of the running build or release.
As part of the process we implement we create agents "on the fly" and destroy them after publishing the artifacts. The agents currently use a PAT to authenticate to VSTS/ADOS, however PAT's have a limited lifetime and is therefore not desired to use long-term.
Ideally the System.AccessToken is used - this lives just during the build/release. During registration of the agent a JWT token for internal communication is created/generated (so when the OAuth token would expire, the service would still be able to start/stop, hence the reason with PAT you have to re-enter the PAT during remove).
What considerations would have with the introduction of this token authentication for registering agents?

Please do note I will prepare a PR with the functionality requested.

@m-vanengelen-triple i would not suggest doing this, since the scope of System.AccessToken is either per team project or the entire vsts account. By default, System.AccessToken doesn't have permission to configure/unconfigure agents (pool management permission), by doing this, any person who has permission to queue a build might able to break your agent pool (unconfigure some agent).
have you try choose PAT during agent configure, and paste System.AccessToken as the PAT?

@TingluoHuang, the lifespan of our agents is very minimal.
 In the scenario that you describe the person who queue's the build also needs to have permission to create a definition with a task to kill the pool.
It's very unfortunate the pool security system is flawed. There is no scenario where an administrator creates a pool, where a user or bit more privileged user can register an agent in the pool.
Certainly hope this will change in the past (and I'm sure the setup I'm proposing will change then as well).

The PR that I made doesn't consider the System.AccessToken to be of any concern.
That we would be using it to feed that specific token to the --auth oat --token <TOKEN> is a concern of ours. I don't know for what kind of other scenario's this might be useful for others.

@m-vanengelen-triple
i just did a test, you don't have to modify the agent code in order to use System.AccessToken to configure the agent.
you can use that token as PAT during agent configure.

the configure failed since i didn't grant it permission, but you can use the token to authenticate with service during config.

i am closing this issue for now, please reopen if my suggestion doesn't work.

I feel like there was not an actual suggestion here and there was never a solution.
Is it possible to change the security/scope of the access tokens that are created during a release?

I feel like there was not an actual suggestion here and there was never a solution.
Is it possible to change the security/scope of the access tokens that are created during a release?

The suggestion/workaround as proposed by @TingluoHuang was to use the System.AccessToken (which is an OAuth token), and use this in the PAT field (as it accepts these).
The downside of using this approach, is you must grant the Build Service user (which is collection based, not project based) with explicit permissions to manage a agent pool and/or deployment group.
It works as a charm though...!

@m-vanengelen-triple @TingluoHuang thank you, I misunderstood the suggestion. We were using the system access token similar to the original ticket, and running into identifier issues which we thought had to do with permissions. It did, and to do with the global vs project pool identifiers. We wiped everything and used new groups with the same permissions and everything works as expected, as a charm. IDK! Thanks for following up with me

If you are not used to admin privileges in Azure DevOps like me, save yourself some trouble. You need to grant administrator to the project scoped identity (https://learn.microsoft.com/en-us/azure/devops/pipelines/process/access-tokens?view=azure-devops&tabs=yaml#scoped-build-identities) which is formatted like so: {Project Name} Build Service ({Org Name})
You need to grant that to your pool at the organization level. You can either go to Pools in organization settings or select "Organization" on the dropdown from the top-right corner in the list of permissions editor.

If you are not used to admin privileges in Azure DevOps like me, save yourself some trouble. You need to grant administrator to the project scoped identity (https://learn.microsoft.com/en-us/azure/devops/pipelines/process/access-tokens?view=azure-devops&tabs=yaml#scoped-build-identities) which is formatted like so: {Project Name} Build Service ({Org Name})
You need to grant that to your pool at the organization level. You can either go to Pools in organization settings or select "Organization" on the dropdown from the top-right corner in the list of permissions editor.

I'm using System.AccessToken to clone another repo in the pipeline both of the repos are in the same organization and private, and I don't want to disable the "Limit job authorization scope to current project for non-release pipelines" for the whole organization,
I added the builder of the first project and gave permissions on the other but still no luck


Test against redis-rb 5.x

redis-rb 5.x use redis-client under the hood. We're benchmarking against 4.x on purpose.