Thumbnails not working for png files

Description
Thumbnails not showing up for png files

.jfif .jpeg and .gif are working
Steps To Reproduce

Open a folder with png files
Change layout to small/medium/large icons

Expected behavior
Thumbnails should show up the same as .jpeg and other image types
Files Version
Version: <IP_ADDRESS>
Windows Version
Edition	Windows 11 Pro
Version	21H2
OS build	22000.258
OS Version: 10.0.22000.258
Relevant Assets
No response

Thanks for the report. Merging with #6130.


[Bug] No portrait for Yanfei

Describe the bug
Yanfei's "front" portrait is not showing up (see screenshots).
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

Additional context
In the HTML, the src attribute of the image has no value associated to it. In the character selection screen:

On Yanfei's page:

fixed in 5e91930.


Allow use of trix directly with trix_editor_tag by injecting into ActionView::Base

This feels a bit icky to me and like I'm not doing it the true "rails way", but it does work on my project. Any suggestions welcome!

Hi @zachfeldman! Two quick things about this PR--

Would you be able to take a look into Action View's source code to see how Rails does this? Like you pointed out, the solution is a little hacky right now 😬

Can you please add some additional test coverage of this new feature?

@maclover7 sure I'd be glad to help.

I've looked into Action View's source code and have not found an example similar enough to this project to understand how to autoload and inject this code without injecting into activerecord::base. Some of the examples I've looked at:

https://github.com/marcelorocks/filterable-models/blob/0e5d91410c65e979856f67e97cd7a6d62afc1023/lib/filterable-models.rb

https://github.com/oshchyhol/chargify_direct/blob/e13af53e99de06922795461d504b0db33c25447c/lib/chargify_direct.rb

https://github.com/rails/rails/blob/111736ddf6fe81f3e42af37dfa1b0520a8c91473/actionview/lib/action_view/helpers/form_tag_helper.rb (methods are defined directly in modules, not in a module then included in another module, then injected again?)
https://github.com/rails/rails/blob/111736ddf6fe81f3e42af37dfa1b0520a8c91473/actionview/lib/action_view/helpers/text_helper.rb

My main thesis is that I have to include trix_editor_tag in FormTagHelper so I can call it directly in my code. I've been able to resolve it as defined by defining it directly in the module like so:
module ActionView
  module Helpers

    module FormTagHelper
      def trix_editor_tag(object_name, value, options)
      end
    end

  end
end

However, from here I get stuck in a loop of, "wait, this is already defined above" and trying to include it there instead of defining it (include TrixEditorHelper instead of the method definition above). Or I try to use the Tags::TrixEditor.new(object_name, method, self, options).render method similar to the FormHelper implementation, but it seems silly because that's calling trix_editor_tag underneath it all.
Frankly I'm pretty stumped here, would appreciate any guidance @maclover7 as to what you consider a proper way to inject this into activerecord.
I'm glad to do number 2 once I can take care of number 1! Appreciate any insights you might have.


nd4j-native: leaky ReLU broken

Leaky relu seems to be returning all 0s.
    @Test
    public void testLeakyRelu(){
        INDArray arr = Nd4j.linspace(-1,1,10);
        double[] expected = new double[10];
        for( int i=0; i<10; i++ ){
            double in = arr.getDouble(i);
            expected[i] = (in <= 0.0 ? 0.01 * in : in);
        }

        INDArray out = Nd4j.getExecutioner().execAndReturn(new LeakyReLU(arr));

        INDArray exp = Nd4j.create(expected);
        assertEquals(exp,out);
    }

https://github.com/deeplearning4j/nd4j/commit/c7c9233f64cda7774c06fb4d7ccf6101bd813604

This doesn't appear to be correct for the negative side. I'm getting zeros there, not 0.01*input...
    @Test
    public void testLeakyRelu(){
        INDArray arr = Nd4j.linspace(-1,1,10);
        double[] expected = new double[10];
        for( int i=0; i<10; i++ ){
            double in = arr.getDouble(i);
            expected[i] = (in <= 0.0 ? 0.01 * in : in);
        }

        INDArray out = Nd4j.getExecutioner().execAndReturn(new LeakyReLU(arr));

        System.out.println("Expected: " + Arrays.toString(expected));
        System.out.println("Actual:   " + Arrays.toString(out.data().asDouble()));

        INDArray exp = Nd4j.create(expected);
        assertEquals(exp,out);
    }

Expected: [-0.01, -0.007777777910232544, -0.005555555820465088, -0.003333333432674408, -0.001111111119389534, 0.1111111119389534, 0.3333333432674408, 0.5555555820465088, 0.7777777910232544, 1.0]
Actual:   [-0.0, -0.0, -0.0, -0.0, -0.0, 0.1111111119389534, 0.3333333432674408, 0.5555555820465088, 0.7777777910232544, 1.0]

@AlexDBlack you need to ad 0.01 to the constructor.


SOQL / Collection Datatable - Allow custom relative height in Screen Flows

For the collection datatable it is currently not possible to set a custom relative hight leading to issues with UX as seen in video attached.
https://user-images.githubusercontent.com/30249667/103720285-e23cac00-4f7f-11eb-8830-2094f2f0709b.mov

Yep, as we discussed offline (of github) - the root cause is my decision to attach interactivity from within the cell itself - allowing me to control overflow vertically but within the bounds of the table.
There is a more complicated approach, one similar to SFDC list views which is to attach a hidden overlay to dynamically attach "popovers" which track the x-y coordinates of where the cell interactivity is launched from. This is too high LOE for my liking so we are stuck with....
The workaround is to specify a minimum height (which is a feature for soqlDatatable).
I'll port that over and add a disclaimer on the README for this behavior.

Yep, as we discussed offline (of github) - the root cause is my decision to attach interactivity from within the cell itself - allowing me to control overflow vertically but within the bounds of the table.
There is a more complicated approach, one similar to SFDC list views which is to attach a hidden overlay to dynamically attach "popovers" which track the x-y coordinates of where the cell interactivity is launched from. This is too high LOE for my liking so we are stuck with....
The workaround is to specify a minimum height (which is a feature for soqlDatatable).
I'll port that over and add a disclaimer on the README for this behavior.

Note to self - since screen flows can be limited in space (depending on if they're launched from flexipage, quick action, etc) it might be better to just add a "Table Height (px)" property with a default of null which solution designers can configure.
This is different than relative max height since that uses vh to calculate height and this one would use px

Note to self - since screen flows can be limited in space (depending on if they're launched from flexipage, quick action, etc) it might be better to just add a "Table Height (px)" property with a default of null which solution designers can configure.
This is different than relative max height since that uses vh to calculate height and this one would use px

I am wrong about the height thing being able to fix this.
That does indeed extend the container of the table however it doesn't affect the dropdown from the picklist. I am going to evaluate including a experimental checkbox that does a CSS hack that allows for overflow - but it has some caveats I'd like to document before introducing it.

I am wrong about the height thing being able to fix this.
That does indeed extend the container of the table however it doesn't affect the dropdown from the picklist. I am going to evaluate including a experimental checkbox that does a CSS hack that allows for overflow - but it has some caveats I'd like to document before introducing it.


bug fix for BundleLiterals

Related issue: #1510

Type of change: bug fix

Impact: API modification

Development Phase: implementation
Release Notes
wip, test seems failed, work on it tomorrow.

Not sure if I did correct thing, request review from @jackkoenig

This is a InferWidth during chisel elboration, I'm not sure if this is OK, another idea is fix this at firrtl.


Remove Bluebird

Change-type: patch
Signed-off-by: Josh Bowling<EMAIL_ADDRESS>

@balena-ci I self certify!


proxy: Limit Router cache size

The proxy can currently store an unbounded number of routes, which can potentially lead
memory exhaustion panics for applications that expose or connect to an unbounded number of
endpoints. Note that applications like monitoring systems (like Prometheus) can exhibit
these sorts of behaviors in dynamically scheduled environments.
This change adds a hard upper limit to the number of HTTP endpoints a proxy (i.e. either
inbound or outbound directions) may route to.
In followup changes:

An eviction policy should be put in place so that unused routes can be reaped to make
space for new routes.
Telemetry events should be added for router table modifications so diagnostics can be
instrumented.
Single-use routes should consume router capacity.

Oh, btw: I appreciate the unit tests you added in conduit-proxy-router, but I wonder if it might be worthwhile to add some e2e tests in tests/discovery as well?

I wonder if it might be worthwhile to add some e2e tests in tests/discovery as well?

Out of scope


Second newline

Description

Types of changes

[ ] Bug fix or improving code quality
[ ] New language
[ ] New feature

Checklist:

[ ] My code follows the code style of this project.
[ ] I have updated the documentation accordingly.

Got this


Replace M44 speedloader with MK80 magazines in dropship crew equipment rack

About the PR

Replaced M44 speed loader ammo with MK80 magazines

Why / Balance
The Dropship weapons vendor already provides standard M44 speedloaders for free. This change replaces the for-points speedloaders with MK80 magazines for consistency with command vendors.
Requirements

[X] I have read and am following the Pull Request and Changelog Guidelines.
[X] I have added media to this PR or it does not require an ingame showcase.

Changelog
:cl:

tweak: Replaced M44 speedloaders with MK80 magazines in the dropship crew equipment rack

THIS. IS. NOT. PARITY.

Feel free to close it as not-parity, but you're perfectly capable of communicating that like an adult.

Its an inside joke with the community, not intended to be taken offensively,
but anyway, we'll check this out later and see what is parity again, and if this is not, consider if its worth changing.
I havent checked yet but the changes are a bit strange, could you elaborate which vendors youre trying to match them with?

For the MK80, I’m checking the medic, engineer, and SL vendors, which offer it only in a case, not standalone. The Command vendor is the only one with the MK80 sold individually, and unlike the dropship crew vendor, it also stocks MK80 magazines.
I added the SU6 under Restricted Firearms to align with the setup in other vendors, though I wasn’t fully certain about it.
For M44 ammo, the equipment vendor sells it for 10 points, right next to a weapon vendor offering it for free. The dropship crew vendor is the only vendor that charges for standard M44 ammo.

I've changed this PR to only replace the standard M44 speedloader with MK80 magazines, maintaining near-parity.


Add Ruby 3.2 to CI

I am fine with this is theory but obviously CI should be passing before it is merged.

:wave: @bastelfreak This branch is in your fork of the repository. Is it okay to push attempts to fix CI to it?  If so I guess we can't amend the branch to fix broken fixes, in this case how should we proceed?  Otherwise, what should we do (open new PR based on this one)?

you have full permissions in my branch and can also do force pushes/modify history. Feel free to hack on it!

The rest of the failures are caused by:

inifile-3.0.0 -- See https://github.com/TwP/inifile/pull/56
beaker-4.40.0 -- We should probably update to 5.x


Lambda: Failure opening selector

Dear, there is an error in the execution of the lambda, when you see it, it may be an overload of the function for reading a large number of registers.
Error:
org.apache.http.nio.reactor.IOReactorException: Failure opening selector
java.lang.IllegalStateException: org.apache.http.nio.reactor.IOReactorException: Failure opening selector

Metric lambda:
https://ibb.co/tZpZmYt
Thank you for your reply.
Regards

We were just hitting the same issue:
Oct 07, 2020 12:41:25 PM org.apache.http.impl.nio.client.InternalHttpAsyncClient runorg.apache.http.impl.nio.client.InternalHttpAsyncClient run
SEVERE: I/O reactor terminated abnormally
org.apache.http.nio.reactor.IOReactorException: Failure opening selector
at org.apache.http.impl.nio.reactor.AbstractIOReactor.<init>(AbstractIOReactor.java:103)
at org.apache.http.impl.nio.reactor.BaseIOReactor.<init>(BaseIOReactor.java:85)
at org.apache.http.impl.nio.reactor.AbstractMultiworkerIOReactor.execute(AbstractMultiworkerIOReactor.java:321)
at org.apache.http.impl.nio.conn.PoolingNHttpClientConnectionManager.execute(PoolingNHttpClientConnectionManager.java:221)
at org.apache.http.impl.nio.client.CloseableHttpAsyncClientBase$1.run(CloseableHttpAsyncClientBase.java:64)
at java.base/java.lang.Thread.run(Unknown Source)
Caused by: java.io.IOException: Too many open files
at java.base/sun.nio.ch.EPoll.create(Native Method)
at java.base/sun.nio.ch.EPollSelectorImpl.<init>(Unknown Source)
at java.base/sun.nio.ch.EPollSelectorProvider.openSelector(Unknown Source)
at java.base/java.nio.channels.Selector.open(Unknown Source)
at org.apache.http.impl.nio.reactor.AbstractIOReactor.<init>(AbstractIOReactor.java:101)
... 5 more
Finished processing; flushing any remaining logs...
Elasticsearch processor shut down

It seems, that it "crashed" while the cluster was under heavy pressure, but was not able to recover from this. It only started working again, after I've changed the lambda and it implicitly restarted.

Hi,
I applied your solution in PR #22, and I still see these errors.
I tried to add more workers, but it doesn't help.
Do you know what else it can be?
Thanks.


Operator docker image

Support for building the docker image for Operator
Also contains a minor change to remove unneeded Namespace field from DeploymentSpec

Before discussing specific implementation details, can we discuss a documentation change that explains how the operator is meant to be used?

Before discussing specific implementation details, can we discuss a documentation change that explains how the operator is meant to be used?

I initiated a PR for documentation update.

We decided to not use a custom image for the operator because it's simpler to release that way.


Unkonwn errors if tried to run main.yml

Hey,
I'm totally new to Linux. I've installed Debian 11 on an old PC and tried this project.
In "inventory.ini" commented the upper line, and uncommented the line with "connect_local" and in "config.yml" I just changed the pihole pw (it's correct that the pw stands in "", right?).:
:~/internet-pi$ ansible-playbook main.yml
PLAY [internet_pi] ***************************************************
TASK [Gathering Facts] **********************************************
fatal: [<IP_ADDRESS>]: FAILED! => {"ansible_facts": {}, "changed": false, "failed_modules": {"ansible.legacy.setup": {"ansible_facts": {"discovered_interpreter_python": "/usr/bin/python3"}, "failed": true, "module_stderr": "sudo: Ein Passwort ist notwendig\n", "module_stdout": "", "msg": "MODULE FAILURE\nSee stdout/stderr for the exact error", "rc": 1}}, "msg": "The following modules failed to execute: ansible.legacy.setup\n"}
PLAY RECAP *********************************************************
<IP_ADDRESS>                  : ok=0    changed=0    unreachable=0    failed=1    skipped=0    rescued=0    ignored=0
(Ein Passwort ist notwendig = apassword is necessary)
and if I tried to run this via putty/ssh from another PC (in inventory.ini I filled in the (static) IP address of the PC and switched the commented lines:
:~/internet-pi$ ansible-playbook main.yml
PLAY [internet_pi] **************************************************************
TASK [Gathering Facts] *********************************************************
fatal: [<IP_ADDRESS>]: UNREACHABLE! => {"changed": false, "msg": "Failed to connect to the host via ssh<EMAIL_ADDRESS>Permission denied (publickey,password).", "unreachable": true}
PLAY RECAP ********************************************************************
<IP_ADDRESS>            : ok=0    changed=0    unreachable=1    failed=0    skipped=0    rescued=0    ignored=0
Sorry but I don't understand the errors and hope someone could help.
My goal is to integrate the airsensor, parts already ordered.
Greetings from Germany.

It looks like it's asking for a sudo password maybe? Try with the -K option and enter your sudo password when asked.

Thanks,
I solved this problem by typing in a sudo command (with entering the pw) first when opening the terminal... I thought my user had already sudo permissions, but never mind.
But now the next problem is showing up

("keine Berechtigung"=permission denied)
I thought it was maybe the same issue as #324 but I already restarted the PC, (physically) disconnected from ethernet, and tested the internet connection as I plugged in again.

thanks to yigit in 294, I had to change the ansible_user in inventory.ini form pi to my username because I'm running this on a debian server


add AKS-AzureDefender link

This preview never had a link in the readme, adding one

self merging


transfer_dust() for multiple currencies

How is it possible to do transfer_dust() for multiple currencies at once ?
With one it is working fine:
result = client.transfer_dust(asset='ONE')

from: https://python-binance.readthedocs.io/en/latest/binance.html#binance.client.Client.transfer_dust
But multiple currencies at once are also officially supported, like:
"The asset being converted. For example: asset=BTC&asset=USDT"

from: https://github.com/binance-exchange/binance-official-api-docs/blob/9dbe0e961b80557bb19708a707c7fad08842b28e/wapi-api.md#dust-transfer-user_data
I tried few ideas but with no success:
transfer = client.transfer_dust(asset='asset=TRX&asset=BTT')     # APIError(code=-5003): You don't have this asset.
transfer = client.transfer_dust(asset='TRX&asset=BTT')                # APIError(code=-5003): You don't have this asset.
transfer = client.transfer_dust(asset='asset=BTT')                         # APIError(code=-5003): You don't have this asset.
transfer = client.transfer_dust(asset='=BTT')                                  # APIError(code=-5003): You don't have this asset.
transfer = client.transfer_dust(asset='PAX TRX')                            # APIError(code=-5003): You don't have this asset.
transfer = client.transfer_dust(asset=['PAX', 'TRX'])                       # APIError(code=-1022): Signature for this request is not valid.

Any Ideas ?

Below done job for me.
{'asset': 'BTT,XLM'}
client._request_margin_api('post', 'asset/dust', True, data=assets)

Below done job for me.
{'asset': 'BTT,XLM'}
client._request_margin_api('post', 'asset/dust', True, data=assets)

this didn't work for me

Anyone have the solution for this issue?

Below done job for me.
{'asset': 'BTT,XLM'}
client._request_margin_api('post', 'asset/dust', True, data=assets)

this didn't work,

This works:
dust= 'BTT,XLM'
client.transfer_dust(asset=dust)

This works:
dust = 'BTT,XLM'
client.transfer_dust(asset=dust)

Worked!
Thank you very much.

Hello everyone excuse me, Im new in python-binance,  does the "client.transfer_dust(asset='ONE')" function allow me to change coins to other coins? i mean convert from BTC to ETH?

@newjoin2021 No, it converts small amount of coins to BNB.
You should use client.create_order
https://github.com/sammchardy/python-binance/blob/d2054d10ac5e787475a3a8dd551cca932c3e8457/binance/client.py#L1272

hello, the conversion of several assets is broken, the view client.transfer_dust(asset = 'BTC,ETH'), but does not work
APIError(code=-1102): illegal parameter


Cluster deployment fails with AI version stable.17.03.2021-13.18

We were using AI version stable.17.03.2021-13.18 from beginning for deploying cluster using sonofspike playbooks (since ZTP stable/4.7 branch which is used to add worker node uses above version, we are using same version for cluster deployment also )
It was working fine a week back, but now it fails.
[root@vcuhost ai-deploy-cluster-remoteworker]# aicli list hosts
+----------+------------------------+--------------------------------------+--------+--------+-----+
|   Host   |        Cluster         |                  Id                  | Status |  Role  |  Ip |
+----------+------------------------+--------------------------------------+--------+--------+-----+
| master-1 | controlplane-cluster-1 | cc83b048-6787-c9cb-1cc6-9123e682d483 | error  | master | N/A |
| master-2 | controlplane-cluster-1 | af193bfd-f64c-9180-b8e7-6163e263486f | error  | master | N/A |
| master-3 | controlplane-cluster-1 | c257da3c-cf96-65be-d465-402f9f8e0c46 | error  | master | N/A |
+----------+------------------------+--------------------------------------+--------+--------+-----+

Host failed to install due to timeout while connecting to host
[root@vcuhost ai-deploy-cluster-remoteworker]# aicli info host master-3
checked_in_at: 2021-08-17T01:59:36.092Z
cluster_id: 2ea2f2ae-5d52-4223-8dcb-dcf1320d10bd
created_at: 2021-08-16T14:51:05.738Z
id: c257da3c-cf96-65be-d465-402f9f8e0c46
installation_disk_path: /dev/sda
logs_info: completed
logs_started_at: 2021-08-16T14:58:57.913Z
ntp_sources: [{"source_name":"static.<IP_ADDRESS>.clients.your-server.de","source_state":"synced"},{"source_name":"<IP_ADDRESS>","source_state":"not_combined"},{"source_name":"ec2-13-126-27-131.ap-south-1.compute.amazonaws.com","source_state":"not_combined"},{"source_name":"<IP_ADDRESS>","source_state":"not_combined"}]
progress: {'current_stage': '', 'stage_started_at': '0001-01-01T00:00:00.000Z', 'stage_updated_at': '0001-01-01T00:00:00.000Z'}
requested_hostname: master-3
role: master
stage_started_at: 0001-01-01T00:00:00.000Z
stage_updated_at: 0001-01-01T00:00:00.000Z
status: error
status_info: Host failed to install due to timeout while connecting to host
status_updated_at: 2021-08-16T15:02:12.996Z
updated_at: 2021-08-17T02:00:20.969Z
user_name: admin

Playbooks fails with following message:
TASK [monitor_host : Wait for 60 min until the node master-1 reboots] ************************************************************************************
fatal: [master-1]: FAILED! => {"msg": "The conditional check '(host.json.progress.current_stage == 'Rebooting' and host.json.status == 'installing-pending-user-action') or host.json.progress.current_stage in ['Configuring', 'Done']' failed. The error was: error while evaluating conditional ((host.json.progress.current_stage == 'Rebooting' and host.json.status == 'installing-pending-user-action') or host.json.progress.current_stage in ['Configuring', 'Done']): 'dict object' has no attribute 'json'"}

TASK [monitor_host : Wait for 60 min until the node master-2 reboots] ************************************************************************************
fatal: [master-2]: FAILED! => {"msg": "The conditional check '(host.json.progress.current_stage == 'Rebooting' and host.json.status == 'installing-pending-user-action') or host.json.progress.current_stage in ['Configuring', 'Done']' failed. The error was: error while evaluating conditional ((host.json.progress.current_stage == 'Rebooting' and host.json.status == 'installing-pending-user-action') or host.json.progress.current_stage in ['Configuring', 'Done']): 'dict object' has no attribute 'json'"}

TASK [monitor_host : Wait for 60 min until the node master-3 reboots] ************************************************************************************
fatal: [master-3]: FAILED! => {"msg": "The conditional check '(host.json.progress.current_stage == 'Rebooting' and host.json.status == 'installing-pending-user-action') or host.json.progress.current_stage in ['Configuring', 'Done']' failed. The error was: error while evaluating conditional ((host.json.progress.current_stage == 'Rebooting' and host.json.status == 'installing-pending-user-action') or host.json.progress.current_stage in ['Configuring', 'Done']): 'dict object' has no attribute 'json'"}

PLAY RECAP ***********************************************************************************************************************************************
bastion                    : ok=47   changed=4    unreachable=0    failed=0    skipped=46   rescued=0    ignored=0
master-1                   : ok=7    changed=2    unreachable=0    failed=1    skipped=6    rescued=0    ignored=0
master-2                   : ok=7    changed=2    unreachable=0    failed=1    skipped=6    rescued=0    ignored=0
master-3                   : ok=7    changed=2    unreachable=0    failed=1    skipped=6    rescued=0    ignored=0

We observed following message in the installer pod logs:
Error: error creating container storage: the container name "assisted-installer" is already in use by "edb18819f2ec41bdd9bafc60b0c4940d5ee77d07bd91efe9bf71c2f7ad2da386". You have to remove that container to be able to reuse that name.: that name is already in use  cluster_id=a55a51de-cd4d-4109-ba2b-0987e4f5ae0c go-id=2706 host_id=af193bfd-f64c-9180-b8e7-6163e263486f pkg=Inventory request_id=3d6093d4-10d2-43c4-af0f-d33546ba88ab

We followed below resolution, but it did not help:
https://access.redhat.com/solutions/5067971
AI version:
[root@vcuhost ~]# podman ps
CONTAINER ID  IMAGE                                                      COMMAND               CREATED       STATUS           PORTS                                                                                NAMES
c32e32c3f6e4  registry.access.redhat.com/ubi8/pause:latest                                     11 hours ago  Up 11 hours ago  <IP_ADDRESS>:5432->5432/tcp, <IP_ADDRESS>:8000->8000/tcp, <IP_ADDRESS>:8080->8080/tcp, <IP_ADDRESS>:8090->8090/tcp  f2c0ab3855ea-infra
497f86808cfd  quay.io/ocpmetal/postgresql-12-centos7:latest              run-postgresql        11 hours ago  Up 11 hours ago  <IP_ADDRESS>:5432->5432/tcp, <IP_ADDRESS>:8000->8000/tcp, <IP_ADDRESS>:8080->8080/tcp, <IP_ADDRESS>:8090->8090/tcp  db
f5ff257ee7cc  quay.io/ocpmetal/ocp-metal-ui:latest                       /opt/bitnami/scri...  11 hours ago  Up 11 hours ago  <IP_ADDRESS>:5432->5432/tcp, <IP_ADDRESS>:8000->8000/tcp, <IP_ADDRESS>:8080->8080/tcp, <IP_ADDRESS>:8090->8090/tcp  ui
3478564033a9  quay.io/ocpmetal/assisted-service:stable.17.03.2021-13.18  /assisted-service     11 hours ago  Up 11 hours ago  <IP_ADDRESS>:5432->5432/tcp, <IP_ADDRESS>:8000->8000/tcp, <IP_ADDRESS>:8080->8080/tcp, <IP_ADDRESS>:8090->8090/tcp  installer

We are also seeing similar issue with worker node addition using ZTP playbooks:
https://github.com/redhat-ztp/ztp-cluster-deploy/issues/152

Also tried with AI version v<IP_ADDRESS> and saw same error.


Support CUB-backed cupy.ReductionKernel and @cupy.fuse

Close #3983. Follow-up of #3244. Currently this only supports 1 in + 1 out for efficiently accessing the global memory. N in + 1 out could be revisited in the future; IIUC ReductionKernel does not support multiple outputs.
New changes:

cupy.ReductionKernel can be backed by CUB (from cupy.core._cub_reduction)
Fusion (@cupy.fuse) with reduction can also use CUB now; this again assumes 1 in + 1 out, which holds IIUC
No user code change is required other than the need of setting CUPY_ACCELERATORS=cub.

Taking the example code from the tutorial:
import cupy as cp
import numpy as np
from cupyx.time import repeat

l2norm_kernel = cp.ReductionKernel(
    'T x',  # input params
    'T y',  # output params
    'x * x',  # map
    'a + b',  # reduce
    'y = sqrt(a)',  # post-reduction map
    '0',  # identity value
    'l2norm'  # kernel name
)

x_cp = cp.random.random(1000000, dtype=cp.float64)

cp.core.set_reduction_accelerators([])
print(repeat(l2norm_kernel, (x_cp,), n_repeat=100))
cp.core.set_reduction_accelerators(['cub'])
print(repeat(l2norm_kernel, (x_cp,), n_repeat=100))

y_cp = l2norm_kernel(x_cp)
x_np = cp.asnumpy(x_cp)
y_np = np.sqrt(np.sum(x_np**2))
print(cp.allclose(y_cp, y_np))

Output:
l2norm              :    CPU:   12.386 us   +/- 0.652 (min:   11.617 / max:   15.534) us     GPU-0:  701.201 us   +/- 1.012 (min:  699.232 / max:  705.760) us
l2norm              :    CPU:   24.327 us   +/- 0.496 (min:   23.527 / max:   27.277) us     GPU-0:   54.900 us   +/- 0.478 (min:   54.208 / max:   57.376) us
True

UPDATE: fusion example
import cupy as cp
import numpy as np
from cupyx.time import repeat

cp.core._accelerator.set_routine_accelerators([])

shape = (512, 512, 512)

a = cp.random.random(shape, dtype=cp.float32)
a_np = cp.asnumpy(a)

def my_op(x, sum_axes):
    # make the fuse function internal so that sum_axes can be varied
    @cp.fuse()
    def _my_op(x):
        y = cp.sin(x)
        y = cp.sum(y**2, axis=sum_axes)
        return y
    return _my_op(x)

for sum_axes in [(2,), (1,2), (0,1,2)]:
    print(f"axes={sum_axes}")
    cp.core._accelerator.set_reduction_accelerators([])
    print(repeat(my_op, (a, sum_axes), n_repeat=100))
    cp.core._accelerator.set_reduction_accelerators(['cub'])
    print(repeat(my_op, (a, sum_axes), n_repeat=100))
    out_cp = my_op(a, sum_axes)
    out_np = np.sum(np.sin(a_np)**2, axis=sum_axes)
    if not np.allclose(cp.asnumpy(out_cp), out_np, rtol=1E-4):
        print("cupy:", out_cp)
        print("numpy:", out_np)
    print()

Output:
axes=(2,)
my_op               :    CPU:  219.712 us   +/-10.059 (min:  213.884 / max:  303.909) us     GPU-0: 4580.685 us   +/-459.435 (min: 4371.840 / max: 5942.080) us
my_op               :    CPU:  220.783 us   +/- 3.813 (min:  216.017 / max:  237.544) us     GPU-0: 1743.457 us   +/-69.232 (min: 1566.688 / max: 1823.744) us

axes=(1, 2)
my_op               :    CPU:  217.272 us   +/- 4.960 (min:  211.869 / max:  243.187) us     GPU-0: 2183.468 us   +/-23.620 (min: 2097.472 / max: 2217.088) us
my_op               :    CPU:  220.258 us   +/- 3.579 (min:  216.138 / max:  239.502) us     GPU-0: 1948.207 us   +/- 7.690 (min: 1930.272 / max: 1974.240) us

axes=(0, 1, 2)
my_op               :    CPU:  224.835 us   +/- 5.922 (min:  214.166 / max:  253.125) us     GPU-0:138661.313 us   +/-26.385 (min:138596.954 / max:138738.693) us
my_op               :    CPU:  233.950 us   +/- 5.803 (min:  228.622 / max:  261.253) us     GPU-0: 1842.682 us   +/-46.423 (min: 1708.256 / max: 1912.768) us

Jenkins, test this please

Jenkins CI test (for commit ba53c20c358f3cd5d9779502be77f26a6f678fa7, target branch master) failed with status FAILURE.

Jenkins, test this please

Jenkins CI test (for commit 78f39587fa686f31fc45dd7dd8866102defccf4a, target branch master) failed with status FAILURE.

4 more errors to tackle...Looks like I need to look into how fusion works 😩

Jenkins, test this please

Jenkins CI test (for commit 4b98ef990bca4b4737c6f3ae0040ce31301de028, target branch master) failed with status FAILURE.

Jenkins, test this please

Jenkins CI test (for commit 706f1cd2534eaaacf251e2d336f4aa45e992792c, target branch master) succeeded!

OK I hope I fixed fusion. The PR description is updated. PTAL. I think we already have enough test coverage so we don't need new ones?

Jenkins, test this please

Jenkins CI test (for commit 706f1cd2534eaaacf251e2d336f4aa45e992792c, target branch master) succeeded!

Jenkins, test this please

btw I added a quick benchmark for CUB-backed fusion to the PR description.

Jenkins CI test (for commit 514042fb807f807e71caa1bb4f60115e2178fc3c, target branch master) succeeded!

Thanks @emcastillo @asi1024!


Appstudio update mvp-test-component

Pipelines as Code configuration proposal

Pipelines as Code CI/mvp-test-component-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
6 seconds
init

✅ Succeeded
20 seconds
clone-repository

✅ Succeeded
1 minute
build-container

✅ Succeeded
14 seconds
deprecated-base-image-check

✅ Succeeded
15 seconds
sanity-inspect-image

✅ Succeeded
17 seconds
clair-scan

✅ Succeeded
2 minutes
clamav-scan

✅ Succeeded
11 seconds
sbom-json-check

✅ Succeeded
24 seconds
sanity-optional-label-check

✅ Succeeded
24 seconds
sanity-label-check

✅ Succeeded
6 seconds
show-summary


Tests failing on PyTorch master

The following tests (list may increase) fail on PyTorch master, which seem unrelated to tensor casting issue in #944. I encountered these while fixing the tensor casting issues in #952. Easy way to replicate this would be to run bash scripts/install_pytorch.sh with commit 2df578a --> 1807bac in README.md*. I might skip these for the time being.

[ ] bayesian_regression.py - RuntimeError: you can only change requires_grad flags of leaf variables. cc. @jpchen
[x] Tests using ADVIMultivariateNormal in test_advi.py - advi covariance off. cc. @fritzo
[x] Tests in gp/test_models.py - test_inference, test_inference_svgp, test_inference_deepGP. Lapack Error in potrf : the leading minor of order 2 is not positive definite, and assertion failure on variance. cc. @fehiepsi
[x] Tests in gp/test_likelihoods - same as above.
[x] gp.ipynb same as above.

EDIT - After removing the mvn wrapper, all but the bayesian_regression.ipynb are fixed.
* Separately I will modify the script to take in an optional arg for build commit.

I think these lines trigger the "requires_grad flags" issue:
https://github.com/uber/pyro/blob/de50a8ee5854c4f00f37cbd851b536b8fbfe2151/examples/bayesian_regression.py#L76
https://github.com/uber/pyro/blob/de50a8ee5854c4f00f37cbd851b536b8fbfe2151/examples/bayesian_regression.py#L79
The issue is that w_mu is that the type_as comes after the requires_grad assignment, so w_mu is no longer a leaf variable. The existing code is equivalent to:
w_mu_hidden = torch.randn(1, p, requires_grad=True)  # leaf variable (optimizable)
w_mu = w_mu_hidden.type_as(data)  # backprops to of w_mu_hidden via backward type conversion

A fix is:
w_mu = torch.randn(1, p).type_as(data)
w_mu.requires_grad = True

(We need a better way to avoid or detect these sorts of issues in PyTorch)

I've been seeing spurrious failures in gp.ipynb. Are the random seeds being fixed? @fehiepsi any ideas? https://travis-ci.org/uber/pyro/jobs/360155682

I've been seeing spurrious failures in gp.ipynb. Are the random seeds being fixed? @fehiepsi any ideas? https://travis-ci.org/uber/pyro/jobs/360155682

I restarted the run, but I noticed it too. The issue is that the constraint checking complains that the precision matrix has invalid values, but the constructor was called without the precision matrix in the first place. I think it may have to do with the precision_matrix being lazy loading. The other thing is that validate_args is reversing all the gains by actually eagerly initializing all of these properties.

The issue is that w_mu is that the type_as comes after the requires_grad assignment, so w_mu is no longer a leaf variable. The existing code is equivalent to:

Thanks for explaining, @colesbury. That makes it super clear what's happening! cc. @jpchen.

@fritzo - #956 temporarily disables smoke test on the gp tutorial, until we investigate and fix that upstream.

@fritzo @neerajprad I run the notebook locally and was able to reproduce the issue. That covariance matrix is 100x100 dimensional and positive definite (after checking with torch.symeig) but its inverse (precision matrix) is not positive definite. Its smallest eigenvalue is about -0.2, all other eigenvalues are positive. I made a notebook to observe the behavior here: https://gist.github.com/fehiepsi/e33d8a0ffbb52ca3a8f7d589c9dcb9ab
I don't know if it is better to use pre version instead of PRE. @SsnL what do you think about calculating the lazy precision_matrix based on scale_tril instead of covariance_matrix?


release: v0.1.3

Description: 概要

Changes: 変更内容

Changes

fix: consider attribute setting is not set (closes #7) (46fb07f7a710dfaf47ac95821ada3b53fb738f00)

Expected Impact: 影響範囲

Operating Requirements: 動作要件

Additional context: 補足

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@738b645). Click here to learn what that means.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             master        #8   +/-   ##
==========================================
  Coverage          ?   100.00%
==========================================
  Files             ?         2
  Lines             ?        53
  Branches          ?         9
==========================================
  Hits              ?        53
  Misses            ?         0
  Partials          ?         0

Impacted Files
Coverage Δ

src/logger.ts
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 738b645...0d7ee18. Read the comment docs.


Refactor manage_iq_client

Running db:migrate will generate a few Unable to annote manage_iq_client/... lines in the log.
These can be ignored, according to @stackus they are generated because the manage_iq_client is not backed by a database.
We may want to revisit the client and complete its implementation.

Currently only rake tasks are using it in its incomplete state, but that could change going forward.

For reference: https://github.com/ctran/annotate_models

@mafernando is this still a thing?  If not, can you please close?

So as it turns out, there is a skip tag we can put on models that we want annotate to ignore. I discovered an issue with the way annotate checks for that tag, so I have proposed https://github.com/ctran/annotate_models/pull/243 to resolve the issue. Am awaiting on annotate to merge the PR.

This "client" isn't sacred and if it's not going to be used it can be deleted. If it ever needs to be resurrected, there's always the git history.

Totally agree with you, just wanted to update the annotate gem while the issue was fresh on my mind.
Maybe we add additional clients going forward and can tell annotate to skip them.
I also recognize that the Annotate.load_tasks unless ENV.key? 'DISABLE_ANNOTATIONS' check we put on auto_annotate_models.rake should more or less prevent rake db:migrate from producing too much noise.

Looks like my PR was merged into annotate.
I'll throw a skip tag on the manage_iq_client and then we can close out this.

[ ] Add skip tag to manage_iq_client

Taking care of this in https://github.com/projectjellyfish/api/issues/496
Closing.


difference between awk and goawk in csv mode

I have a csv file that, by necessity, includes some blank lines. If I print the file using awk, e.g.,
awk '{print}' filename

the blank lines are printed. If instead I print the file using goawk in csv mode, e.g.,
goawk -i csv -o csv '{print}' filename

the blank lines are skipped. Is there some way to include the blank lines while still using goawk in csv mode?

Hi Mike, for better or worse, this is by design, as you normally don't want blank lines included in the data: GoAWK's CSV reader skips blank lines (Go's encoding/csv.Reader does the same) because they're not part of the CSV data itself. So when you output the rows the blank lines are missing.
Can you give an example of your input, even just a few lines showing context? Or perhaps explain why the blank lines or needed here, and how we might be able to work around this?

Yes, I can give you the example, but it is somewhat frustrating. It is an
input file for the Weka machine learning system. The input file has a
header which includes blank lines, followed by csv data for the famous iris
dataset. It would be far better for Weka to use some kind of known standard
for its header, but this is what I have to work with. I accept that there
may be no solution beyond manually editing such files, but thought I'd give
it a try. The task here, as part of a chain of commands, is to alter some
of the csv data using Awk. I thought it would be a good place to try GoAwk
because the csv data to be altered is only in the third and fourth columns
of the versicolor section of the data.

Mick

The file follows:
@RELATION iris_discretized
@ATTRIBUTE sepallength {low,medium,high}
@ATTRIBUTE sepalwidth {low,medium,high}
@ATTRIBUTE petallength {low,medium,high}
@ATTRIBUTE petalwidth {low,medium,high}
@ATTRIBUTE class {Iris-setosa,Iris-versicolor,Iris-virginica}
@DATA
low,medium,low,low,Iris-setosa
low,medium,low,low,Iris-setosa
low,medium,low,low,Iris-setosa
low,medium,low,low,Iris-setosa
low,high,low,low,Iris-setosa
low,high,low,low,Iris-setosa
low,medium,low,low,Iris-setosa
low,medium,low,low,Iris-setosa
low,medium,low,low,Iris-setosa
low,medium,low,low,Iris-setosa
low,high,low,low,Iris-setosa
low,medium,low,low,Iris-setosa
low,medium,low,low,Iris-setosa
low,medium,low,low,Iris-setosa
medium,high,low,low,Iris-setosa
medium,high,low,low,Iris-setosa
low,high,low,low,Iris-setosa
low,medium,low,low,Iris-setosa
medium,high,low,low,Iris-setosa
low,high,low,low,Iris-setosa
low,medium,low,low,Iris-setosa
low,high,low,low,Iris-setosa
low,high,low,low,Iris-setosa
low,medium,low,low,Iris-setosa
low,medium,low,low,Iris-setosa
low,medium,low,low,Iris-setosa
low,medium,low,low,Iris-setosa
low,medium,low,low,Iris-setosa
low,medium,low,low,Iris-setosa
low,medium,low,low,Iris-setosa
low,medium,low,low,Iris-setosa
low,medium,low,low,Iris-setosa
low,high,low,low,Iris-setosa
medium,high,low,low,Iris-setosa
low,medium,low,low,Iris-setosa
low,medium,low,low,Iris-setosa
medium,medium,low,low,Iris-setosa
low,medium,low,low,Iris-setosa
low,medium,low,low,Iris-setosa
low,medium,low,low,Iris-setosa
low,medium,low,low,Iris-setosa
low,low,low,low,Iris-setosa
low,medium,low,low,Iris-setosa
low,medium,low,low,Iris-setosa
low,high,low,low,Iris-setosa
low,medium,low,low,Iris-setosa
low,high,low,low,Iris-setosa
low,medium,low,low,Iris-setosa
low,high,low,low,Iris-setosa
low,medium,low,low,Iris-setosa
high,medium,medium,medium,Iris-versicolor
medium,medium,medium,medium,Iris-versicolor
high,medium,medium,medium,Iris-versicolor
medium,low,medium,medium,Iris-versicolor
medium,medium,medium,medium,Iris-versicolor
medium,medium,medium,medium,Iris-versicolor
medium,medium,medium,medium,Iris-versicolor
low,low,medium,medium,Iris-versicolor
medium,medium,medium,medium,Iris-versicolor
low,low,medium,medium,Iris-versicolor
low,low,medium,medium,Iris-versicolor
medium,medium,medium,medium,Iris-versicolor
medium,low,medium,medium,Iris-versicolor
medium,medium,medium,medium,Iris-versicolor
medium,medium,medium,medium,Iris-versicolor
high,medium,medium,medium,Iris-versicolor
medium,medium,medium,medium,Iris-versicolor
medium,low,medium,medium,Iris-versicolor
medium,low,medium,medium,Iris-versicolor
medium,low,medium,medium,Iris-versicolor
medium,medium,medium,high,Iris-versicolor
medium,medium,medium,medium,Iris-versicolor
medium,low,medium,medium,Iris-versicolor
medium,medium,medium,medium,Iris-versicolor
medium,medium,medium,medium,Iris-versicolor
medium,medium,medium,medium,Iris-versicolor
high,medium,medium,medium,Iris-versicolor
high,medium,high,high,Iris-versicolor
medium,medium,medium,medium,Iris-versicolor
medium,low,medium,medium,Iris-versicolor
medium,low,medium,medium,Iris-versicolor
medium,low,medium,medium,Iris-versicolor
medium,low,medium,medium,Iris-versicolor
medium,low,high,medium,Iris-versicolor
low,medium,medium,medium,Iris-versicolor
medium,medium,medium,medium,Iris-versicolor
high,medium,medium,medium,Iris-versicolor
medium,low,medium,medium,Iris-versicolor
medium,medium,medium,medium,Iris-versicolor
medium,low,medium,medium,Iris-versicolor
medium,low,medium,medium,Iris-versicolor
medium,medium,medium,medium,Iris-versicolor
medium,low,medium,medium,Iris-versicolor
low,low,medium,medium,Iris-versicolor
medium,low,medium,medium,Iris-versicolor
medium,medium,medium,medium,Iris-versicolor
medium,medium,medium,medium,Iris-versicolor
medium,medium,medium,medium,Iris-versicolor
low,low,medium,medium,Iris-versicolor
medium,medium,medium,medium,Iris-versicolor
medium,medium,high,high,Iris-virginica
medium,low,high,high,Iris-virginica
high,medium,high,high,Iris-virginica
medium,medium,high,high,Iris-virginica
medium,medium,high,high,Iris-virginica
high,medium,high,high,Iris-virginica
low,low,medium,high,Iris-virginica
high,medium,high,high,Iris-virginica
high,low,high,high,Iris-virginica
high,high,high,high,Iris-virginica
medium,medium,high,high,Iris-virginica
medium,low,high,high,Iris-virginica
high,medium,high,high,Iris-virginica
medium,low,high,high,Iris-virginica
medium,medium,high,high,Iris-virginica
medium,medium,high,high,Iris-virginica
medium,medium,high,high,Iris-virginica
high,high,high,high,Iris-virginica
high,low,high,high,Iris-virginica
medium,low,high,medium,Iris-virginica
high,medium,high,high,Iris-virginica
medium,medium,medium,high,Iris-virginica
high,medium,high,high,Iris-virginica
medium,low,medium,high,Iris-virginica
high,medium,high,high,Iris-virginica
high,medium,high,high,Iris-virginica
medium,medium,medium,high,Iris-virginica
medium,medium,medium,high,Iris-virginica
medium,medium,high,high,Iris-virginica
high,medium,high,medium,Iris-virginica
high,medium,high,high,Iris-virginica
high,high,high,high,Iris-virginica
medium,medium,high,high,Iris-virginica
medium,medium,high,medium,Iris-virginica
medium,low,high,medium,Iris-virginica
high,medium,high,high,Iris-virginica
medium,medium,high,high,Iris-virginica
medium,medium,high,high,Iris-virginica
medium,medium,medium,high,Iris-virginica
high,medium,high,high,Iris-virginica
high,medium,high,high,Iris-virginica
high,medium,high,high,Iris-virginica
medium,low,high,high,Iris-virginica
high,medium,high,high,Iris-virginica
high,medium,high,high,Iris-virginica
high,medium,high,high,Iris-virginica
medium,low,high,high,Iris-virginica
medium,medium,high,high,Iris-virginica
medium,medium,high,high,Iris-virginica
medium,medium,high,high,Iris-virginica
On Tue, May 17, 2022 at 6:54 PM Ben Hoyt @.***> wrote:

Hi Mike, for better or worse, this is by design, as you normally don't
want blank lines included in the data: GoAWK's CSV reader skips blank lines
(Go's encoding/csv.Reader does the same) because they're not part of the
CSV data itself. So when you output the rows the blank lines are missing.
Can you give an example of your input, even just a few lines showing
context? Or perhaps explain why the blank lines or needed here, and how we
might be able to work around this?
—
Reply to this email directly, view it on GitHub
https://github.com/benhoyt/goawk/issues/118#issuecomment-1129397086, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAOTM3QM65TORNTL7DNO4JLVKQPSJANCNFSM5WGNFTVQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

Actually, let me not waste more of your time on this issue. Knowing that it is by design and is in the underlying Go code is enough. I can just work around it by stripping the header and printing it in a separate script without disturbing the workflow.

Yeah, that makes sense to me. In this case I think the answer is "that data is not exactly CSV", particularly the @... lines. For this kind of custom format you might be better using goawk (or plain awk/gawk) in non-CSV mode, just looking for the @Data and then splitting on FS="," (there doesn't seem to be quoting in this data in any case).


Update adium to <IP_ADDRESS>

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

Thank you for the contribution. It was merged directly as aabad3a349e5bee56920d483eba6284883ba9110. Your contribution is still credited to you.


Update defaults for distributed multigrid

This PR updates the defaults for distributed multigrid for the smoother and the coarse solver. It also adds a test for the distributed pgm preconditioned solver

@pratikvn I will fix the CI issues to push this for the release.

One thing to note is that the added tests are EXTREMELY slow. On nla-gpu the preconditioned test are 100x slower than the unpreconditioned tests. But I don't think this PR should fix performance issues, so I will just leave it as it is.

slow as in long runtime or as in slow convergence (or both)? In any case, I think we should reconsider whether to merge this PR in this state then

slow as in long runtime or as in slow convergence (or both)? In any case, I think we should reconsider whether to merge this PR in this state then

definitely runtime, but I think the number of iterations is also quite bad.
IMO this PR should still continue, since otherwise the user will get segfaults and the like. Even sequentially our mg is not very good, so it wouldn't be much of a difference to that.

The slow convergence is expected in the 3pt stencil because the PGM does not handle the equal-weight matrix well currently.
The corasening matrix might almost like the original matrix (n -> n - 10 for example)

Alright, then I withdraw my objection ;) that's an issue we should definitely tackle soon though. I think with some deterministic or randomized tie-breaking we should be able to handle that, create at least a 25%-33% reduction per level (haven't thought the worst case through entirely)

Given that we are testing that the solver + preconditioner  work, I think we should just set the factory parameter with_deterministic(false) for the PGM factory. That should speedup the convergence.

Given that we are testing that the solver + preconditioner work, I think we should just set the factory parameter with_deterministic(false) for the PGM factory. That should speedup the convergence.

Can't confirm that this has any effect (on nla-gpu).


Add reliable way of making changes for VT client

Followup on #362, we have now added guaranteed delivery of changes to the bus. However there is a chance that the VT rejects the changes on first or second try. Or somehow discards the message without notice. To make the application not have to worry about this, we can add a interface on top of the VT client class:
General idea:

[ ] structs to hold data for pending changes (to check responses against)
[ ] structs to hold data for responses (to be used in callbacks)
[ ] function to register tracking X with default value for Z)
[ ] function to update X
[ ] function reset X
[ ] separate interface to process responses, and allows application to indicate whether to retry sending or not
[ ] optionally, on reset of the pool, send default values to VT server to bring the pool up with a known state
[ ] listen to changes made by working set members to X

This way we can provide a nice out-of-the-box solution to applications wanting to sent changes to a object pool in a reliable way

This issue is too complicated and not efficient, it is half replaced by #403


.github: remove deprecated EdenGCP workflow.

The EdenGCP workflow was consuming runner time without executing real tests. Support for this workflow has been abandoned, making it ineffective for current needs. It only triggered after the Publish workflow, providing no value for PR testing.

I just want to add that previously we used GCP because Github runners do not provide nested virtualization, therefore we could not test deploying VMs or containers inside VMs. Now with buildjet we have support for nested virtualization and use it in the "Virtualization" test suite.
@OhmSpectator Do you want to add this to the commit message or should we just merge?

@milan-zededa, done. Feel free to merge =)


This is a follow-up to #1211. The apiserver changes are openshift/openshift-apiserver#309. The image-registry (consumer) changes are openshift/image-registry#347.

/assign @flavianmissi @deads2k

left a question, but /lgtm

/lgtm

@deads2k please take a look again. Validation is added at https://github.com/openshift/openshift-apiserver/pull/309.

reminder for future me:

The doc now makes sense, but the validation is not yet present in https://github.com/openshift/openshift-apiserver/pull/309/files#diff-4e280f22beaa1bd7ae3747b41a8f5797ad011d86d61adbf82500c73151daeddfR806

/approve
/assign @flavianmissi
@flavianmissi has lgtm

/lgtm

/label px-approved
/assign @stevsmit
for docs approval
/assign @xiuwang
for qe approval

/label qe-approved

/docs-approved

/label docs-approved


switch request content-type to html/plain

This is necessary due to some recent changes (https://github.com/RJMetrics/RJMetrics/pull/6548) to the dashboard framework that disallow text/plain requests for security reasons.

Approved.


Enable static build option for advanced augmentations

@r-abishek has merge conflicts

@r-abishek has merge conflicts

@kiritigowda Resolved now

@r-abishek can you resolve conflicts

@r-abishek can you resolve conflicts

@kiritigowda Resolved


Can some JS genius help?


🧱 Metadata extraction helper

Suggestion of @Thorsten-Behrens . @ThomasKnecht Once we are done with the other priorities, we could discuss about this. Where and how to implement.

there is already other open issue, considering this as duplicate, I'll close


Update UI to show table of items

Codecov Report

Merging #92 (d59691a) into main (aa35606) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #92   +/-   ##
=======================================
  Coverage   97.08%   97.08%
=======================================
  Files           7        7
  Lines         378      378
=======================================
  Hits          367      367
  Misses         11       11

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


Unable to connect to server

Hi,
I succesfuly uploaded a qGIS project to a Tablet with QField Could. I was also able to push the first batch of data back to the qGIS file without any problems. In the last days I've collected more data but now I am unable to connect to the qField Could on my tablet (I can log in on laptop). On tthe tablet I am asked to fill in my log in data but then i get an error message "Server not found, please check the server URL."  How can I fix this error? Or if it is not possible to fix easily - can I move the data to qGIS manually by copying files from the tablet? I am quite in a hurry to analyse these data, so I kindly ask for help.

Hi, @janakus .
Quite strange issue, looks more like your local network is acting weird. Have you tried internet in another networks (e.g. 4g, or other than your usual WiFi)?
Another thing that you can try is going to QFieldCloud login screen and double tap the logo. From the logo there will be a new "Server URL" text field appearing. Please try writing app.qfield.cloud there. It might disappear right after that, which is ok, because it is the default URL.
Finally, you can export your project easily. Open your project, go to the menu and click "Open Project Folder".
Would be great to try them in this sequence and understand the issue.


add @main entrypoint support for Scala 3

mainargs 0.2.3 added support for scala3
To test (manually):
test.sc:
@main def main(name: String) = {
  println(s"Hello, $name!")
}

mill -i "amm[3.2.0].assembly" && out/amm/3.2.0/assembly.dest/out.jar test.sc --name Michael
mill -i "amm[2.13.9].assembly" && out/amm/2.13.9/assembly.dest/out.jar test.sc --name Michael

Can you add/reuse a test for this functionality to run on Scala 3?

I reenabled the test - at least that looked like the right one...
Unfortunately it looks like my changes messed up the classpath for test. E.g. if I run mill -i "amm[3.2.0].test" it doesn't find mainargs any longer. I'm not really a mill expert - can you help me with that please?

I reenabled the test - at least that looked like the right one...
Unfortunately it looks like my changes messed up the classpath for test. E.g. if I run mill -i "amm[3.2.0].test" it doesn't find mainargs any longer. I'm not really a mill expert - can you help me with that please?

That's because compileIvyDeps no longer propagate to the test module (as it is no longer in ivyDeps). You need to add mainargs dependency to the dependencies of the test explicitly.

@lefou thanks, yes that fixed the test classpath issue.
I had to disable one specific case from the MainTests again - something related to a type in mainargs.ParserForMethods. I didn't dig in too deep though, maybe someone else knows what's going on. I pasted the scalac output into 8e519af.
Either way - @main entrypoints work now for Scala 3, (almost all) tests are reenabled, so I'd say we're much better off than before.

I tried to refactor the build a bit (remove the duplication in runIvyDeps), but didn't see a great way forward... in the end, we do want to be explicit that we're fiddling around here...

This thing is hard :(
For example, now the mainargs_2.13 is set as provided in the pom and there is no mainargs_3 in it.
This means that people using the Ammonite jar will not get the mainargs dependency and it will fail with a ClassNotFound.
I tried to fix this by manually editing the dependencies entering the pom by overriding def publishXmlDeps: Task[Agg[Dependency]] but it gets messier and messier. Probably a better long term strategy would be to help finish the port of fastparse to Scala 3 to we can switch to Scala 3 completely..

@mpollmeier I'm trying another approach. Instead of adding the _3 deps to runIvyDeps and the _2.13 deps to compileIvyDeps which create a weird pom, I'm adding the _3 deps to the pom but then I tweak the method that resolves the dependencies for the compileClasspath only to use the _2.13 deps.
Also I did some cleanups in the build and now it seems we don't have any duplicated deps like fansi_3 + fansi_2.13
https://github.com/com-lihaoyi/Ammonite/pull/1301
If it end up working fine, then we can rebase this PR on top and enjoy main methods on Scala 3.
About upickle it still doesn't work. I think there is some binary incompatibility between Scala 3 and Scala 2.13 which I'm not sure it comes from the upickle codebase or some difference between the two compilers.

About upickle it still doesn't work. I think there is some binary incompatibility between Scala 3 and Scala 2.13 which I'm not sure it comes from the upickle codebase or some difference between the two compilers.

There is a slight change, that the Scala version specific code in upickle is not interchangeable in a binary compatible sense. At least, there is no tooling ensures it.

About upickle it still doesn't work. I think there is some binary incompatibility between Scala 3 and Scala 2.13 which I'm not sure it comes from the upickle codebase or some difference between the two compilers.

There is a slight chance, that the Scala version specific code in upickle is not interchangeable in a binary compatible sense. At least, there is no tooling ensuring it.

It might be but it's not only that. Swapping fansi_2.13 with fansi_3 (the library doesn't have any version specific code) fails with:
X ammonite.terminal.HistoryTests.noHistory 23ms
  java.lang.NoClassDefFoundError: scala/runtime/function/JProcedure1
    fansi.Attrs$.apply(Fansi.scala:580)
    fansi.Attrs$.<clinit>(Fansi.scala:514)
    ammonite.terminal.HistoryTests$.$anonfun$tests$1(HistoryTests.scala:18)

I tried to bump the fansi version and now the test pass just fine :) .
Probably some Scala 3 compatibility issue was fixed in the Scala 2 compiler going from 2.13.4 -> 2.13.8.

I think I sorted it out 🎉
@mpollmeier @lefou would you mind reviewing #1301 which should solve the issue?
I think the solution can be extended to uPickle after we release a new version based on Scala 2.13.8+ (current version was compiled with 2.13.4 which created binary incompatibilities in fansi, so I expect it can be the same issue we have in upickle.

🎉 nice one, yes will do


Updated to new frameworks and CLI + extra sugaring in build script

Updated to the latest CLI and new framework names/packages.
Updating to the most recent CLI requires updating the version of "Microsoft.Extensions.DependencyModel", which brings with itself dependencies on a newer NETStandard library. Since changes were required to the project.json, I decided to also update the framework monikers.
Changes:

dnx451->net451

dnxcore50->netcoreapp1.0 for OmniSharp and tests and appropriate netstandard for the others

netstandard projects depend on NETStandard.Library (and other libraries if needed)

netcoreapp1.0 projects depend on NETStandard.Library and Microsoft.DotNet.ProjectModel (and other libraries if needed)

NETCore.App brings extra unneeded dependencies including conflicting CodeAnalysis

Changed NETStandard.Library, Microsoft.DotNet.*, Microsoft.Extensions.DependencyModel , System.* dependency versions to use wild-cards for now (see below)

Fixed issues rising from new libraries

Tried to eliminate some of the unnecessary dependencies in the project.json files

Clean-up of build.cake

I also removed Cake addins as they are third-party components targeting .NET 4.5, thus might not be available on a .NET Core version of Cake

Updated .tools path to match the recent change in moving cake-bootstrap.* into scripts

New utility target in build.cake named SetPackageVersions

Things to fix long-term:

Eliminate imports for netstandard, especially dnxcore50

Newtonsoft.Json only targets dnxcore50, this should change as soon as rc2 is released

Microsoft.Extensions.DependencyModel pulls in NuGet.Packaging, which pulls in System.Security.Cryptography.Hashing.Algorithms, which only targets dnxcore50 @troydai do you know any plans about this one?

SetPackageVersions
It uses a new file called depversion.json as a central dictionary to replace all the dependency versions from project.json files across the different folders with the one specified within the file. This can be used to quickly migrate to new package versions. Dependencies without entries in depversion.json are left alone.
This target is not executed as part of the build, but can be called separately, by using -target=SetPackageVersions with build.{ps1|sh}.
For now depversion.json contains the packages that I specified with wild-card in this PR. This allows the wildcards to be quickly replaced by any version you guys prefer. Long-term the other dependencies can also be added here.

@troydai @david-driscoll This is my own personal hack to try and get it to work. My understanding of the project.json system is still limited so I might have made some major mistakes, but I think there are some useful bits here.

I'm working on troy/update branch. I'll pin the version once I got the test work. I began working on it last night and didn't notice you were working on it. https://github.com/OmniSharp/omnisharp-roslyn/tree/troy/update

Most of the "redundant" dependencies look useless but they have their purpose. They were added at some point of the development to mitigate wrongfully introduced dependencies by other packages. They may not be necessarily now, but you have to verify them one by one. Remove one and restore to check the warning. There should be no unexpected package upgrade or downgrade.

Please ignore the portable name in the branch, I removed those parts from this PR and just wanted to focus on the dependency and CLI update.

The test won't work no matter what due to an internal inconsistency between ASP.NET and dotnet CLI. The dotnet packages used by two sides are not aligned cause issues in dependency resolve. OmniSharp relies on both side's packages the problem is unsolvable.
At this point we're heading to rc2 finish line, all the packages will be aligned eventually. I suggest we disable the tests temporary and leave E2E tests only.

I'll suggest not to combine multiple changes in one PR. We either upgrade all the versions and open another PR for __ SetPackageVersions__ work. Combine them together lead to confusions.

:shipit: I'll update the DependencyModel piece afterwards.

@troydai Could you please create a local branch that I can redo the PR against? I can do fixed versions there (based on what the wild-cards gave me) and skip the SetPackageVersions bits. That would allow you to fine-tune it before it goes to dev.

https://github.com/OmniSharp/omnisharp-roslyn/pull/486
@hal-ler are you able to make changes to update branch?


Add metrics-server helm chart.

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


Replace ChunkedDecoder with IterativeDecoder

See comparison here:
https://github.com/otosense/recode/wiki/Compare-performance-of-IterativeDecoder-and-ChunkedDecoder
┆Issue is synchronized with this Asana task by Unito

Already done in commit fb83289


Bump dependencies

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/2197033177
Status: success


There is no logic to let test pass if requireSameDimensions=false for backstop

As in backstopJS there is one option which can be set to false to let test pass is content is same (RGBS) but just height/width if difference. I mean if mismatch is 0.
Please see this issue for deep details and solution. It would be nice if we could take my suggested solution.
I checked all "js" files but no where logic on it. (find -name "*.js" | xargs grep "requireSameDimensions" )
thanks.
https://github.com/garris/BackstopJS/issues/456

I think the result when using diff.scaleToSameSize(); should reflect the original proportions. At the moment "dimensionDifference" will return no difference after scaling.  With the suggested change you intelligently report on comparisons where there are differences in dimension, but no RGBA differences after scaling.
I'll mark as a bug, happy to accept and review a Pull Request.


exchange bottom with left

Compared with the order of the CSS, I think it is a mistake.

Indeed I think you are right.

Thx

どうもありがとう @katsuyoshi さん！

You are welcom @jjaffeux @lrz


Be able to run updates

When update availability is detected or user triggers 'Update Angular' command in VS Code, be able to run ng update
VS Code command Update Angular

dep of #24


Geo-SAM fails to load QGIS 3.36.2

I am getting the following error when trying to initial Geo-SAM in QGIS v3.36.2:
ModuleNotFoundError: No module named 'rasterio'
Traceback (most recent call last):
File "C:\PROGRA~1/QGIS33~2.2/apps/qgis/./python\qgis\utils.py", line 404, in loadPlugin
import(packageName)
File "C:\PROGRA~1/QGIS33~2.2/apps/qgis/./python\qgis\utils.py", line 923, in _import
mod = builtin_import(name, globals, locals, fromlist, level)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users/marti/AppData/Roaming/QGIS/QGIS3\profiles\default/python/plugins\Geo-SAM_init.py", line 7, in
from .geo_sam_tool import Geo_SAM
File "C:\PROGRA~1/QGIS33~2.2/apps/qgis/./python\qgis\utils.py", line 923, in _import
mod = _builtin_import(name, globals, locals, fromlist, level)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users/marti/AppData/Roaming/QGIS/QGIS3\profiles\default/python/plugins\Geo-SAM\geo_sam_tool.py", line 10, in
from .tools.widgetTool import Selector, EncoderCopilot
File "C:\PROGRA~1/QGIS33~2.2/apps/qgis/./python\qgis\utils.py", line 923, in _import
mod = _builtin_import(name, globals, locals, fromlist, level)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users/marti/AppData/Roaming/QGIS/QGIS3\profiles\default/python/plugins\Geo-SAM\tools\widgetTool.py", line 7, in
import rasterio
File "C:\PROGRA~1/QGIS33~2.2/apps/qgis/./python\qgis\utils.py", line 923, in _import
mod = _builtin_import(name, globals, locals, fromlist, level)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
ModuleNotFoundError: No module named 'rasterio'
Python version: 3.12.3 (main, Apr 14 2024, 17:21:43) [MSC v.1938 64 bit (AMD64)]
QGIS version: 3.36.2-Maidenhead Maidenhead, 6d250527
Python Path:
C:\Users/marti/AppData/Roaming/QGIS/QGIS3\profiles\default/python/plugins\geometric_attributes
C:/PROGRA~1/QGIS33~2.2/apps/qgis/./python
C:/Users/marti/AppData/Roaming/QGIS/QGIS3\profiles\default/python
C:/Users/marti/AppData/Roaming/QGIS/QGIS3\profiles\default/python/plugins
C:/PROGRA~1/QGIS33~2.2/apps/qgis/./python/plugins
C:\PROGRA~1\QGIS33~2.2\apps\grass\grass83\etc\python
C:\Users\marti\Documents
C:\Program Files\QGIS 3.36.2\bin\python312.zip
C:\PROGRA~1\QGIS33~2.2\apps\Python312\DLLs
C:\PROGRA~1\QGIS33~2.2\apps\Python312\Lib
C:\Program Files\QGIS 3.36.2\bin
C:\PROGRA~1\QGIS33~2.2\apps\Python312
C:\PROGRA~1\QGIS33~2.2\apps\Python312\Lib\site-packages
C:\PROGRA~1\QGIS33~2.2\apps\Python312\Lib\site-packages\win32
C:\PROGRA~1\QGIS33~2.2\apps\Python312\Lib\site-packages\win32\lib
C:\PROGRA~1\QGIS33~2.2\apps\Python312\Lib\site-packages\Pythonwin
C:/Users/marti/AppData/Roaming/QGIS/QGIS3\profiles\default/python
C:\Users\marti\AppData\Roaming\QGIS\QGIS3\profiles\default\python\plugins\deepness\python3.12
C:\Users\marti\AppData\Roaming\QGIS\QGIS3\profiles\default\python\plugins\DigitizingTools\tools
C:\Users\marti\AppData\Roaming\QGIS\QGIS3\profiles\default\python\plugins\DigitizingTools
C:\Users\marti\AppData\Roaming\QGIS\QGIS3\profiles\default\python\plugins\HCMGIS/forms
C:\Users/marti/AppData/Roaming/QGIS/QGIS3\profiles\default/python/plugins\rvt-qgis
The problem does not occur in QGIS v3.36.1
There seem to have been a substantial number of Python version updates in 3.36.2: https://lists.osgeo.org/pipermail/qgis-user/2024-April/054334.html

Hi @Martin-Dawson
This is likely the path of the Python dependencies is incorrect. Please follow the tutorial and reinstall the python dependencies: https://geo-sam.readthedocs.io/en/latest/installation.html#for-windows-users. Since you have installed two versions of QGIS, there should be two OSGeo4W Shells. Make sure to select the one corresponding to the QGIS v3.36.2.

I've:

Uninstalled and reinstalled QGIS 3.36.2
Followed the installation instructions doing a force re-install.
Now getting the following error (again seems to be path error:
2024-05-05T12:06:17     CRITICAL    Traceback (most recent call last):
File "C:\PROGRA~1/QGIS33~2.2/apps/qgis/./python\qgis\utils.py", line 404, in loadPlugin
import(packageName)
File "C:\PROGRA~1/QGIS33~2.2/apps/qgis/./python\qgis\utils.py", line 923, in _import
mod = builtin_import(name, globals, locals, fromlist, level)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users/marti/AppData/Roaming/QGIS/QGIS3\profiles\default/python/plugins\Geo-SAM_init.py", line 7, in
from .geo_sam_tool import Geo_SAM
File "C:\PROGRA~1/QGIS33~2.2/apps/qgis/./python\qgis\utils.py", line 923, in _import
mod = _builtin_import(name, globals, locals, fromlist, level)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users/marti/AppData/Roaming/QGIS/QGIS3\profiles\default/python/plugins\Geo-SAM\geo_sam_tool.py", line 10, in
from .tools.widgetTool import Selector, EncoderCopilot
File "C:\PROGRA~1/QGIS33~2.2/apps/qgis/./python\qgis\utils.py", line 923, in _import
mod = _builtin_import(name, globals, locals, fromlist, level)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users/marti/AppData/Roaming/QGIS/QGIS3\profiles\default/python/plugins\Geo-SAM\tools\widgetTool.py", line 21, in
from torchgeo.datasets import BoundingBox
File "C:\PROGRA~1/QGIS33~2.2/apps/qgis/./python\qgis\utils.py", line 923, in _import
mod = builtin_import(name, globals, locals, fromlist, level)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\marti\AppData\Roaming\Python\Python312\site-packages\torchgeo\datasets_init.py", line 6, in
from .advance import ADVANCE
File "C:\PROGRA~1/QGIS33~2.2/apps/qgis/./python\qgis\utils.py", line 923, in _import
mod = _builtin_import(name, globals, locals, fromlist, level)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\marti\AppData\Roaming\Python\Python312\site-packages\torchgeo\datasets\advance.py", line 12, in
import torch
File "C:\PROGRA~1/QGIS33~2.2/apps/qgis/./python\qgis\utils.py", line 923, in _import
mod = builtin_import(name, globals, locals, fromlist, level)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\marti\AppData\Roaming\Python\Python312\site-packages\torch_init.py", line 141, in
raise err
OSError: [WinError 126] The specified module could not be found. Error loading "C:\Users\marti\AppData\Roaming\Python\Python312\site-packages\torch\lib\shm.dll" or one of its dependencies.
Python version: 3.12.3 (main, Apr 14 2024, 17:21:43) [MSC v.1938 64 bit (AMD64)]
QGIS version: 3.36.2-Maidenhead Maidenhead, 6d250527

Python Path:
C:\Users/marti/AppData/Roaming/QGIS/QGIS3\profiles\default/python/plugins\geometric_attributes
C:/PROGRA~1/QGIS33~2.2/apps/qgis/./python
C:/Users/marti/AppData/Roaming/QGIS/QGIS3\profiles\default/python
C:/Users/marti/AppData/Roaming/QGIS/QGIS3\profiles\default/python/plugins
C:/PROGRA~1/QGIS33~2.2/apps/qgis/./python/plugins
C:\PROGRA~1\QGIS33~2.2\apps\grass\grass83\etc\python
C:\Users\marti\Documents
C:\Program Files\QGIS 3.36.2\bin\python312.zip
C:\PROGRA~1\QGIS33~2.2\apps\Python312\DLLs
C:\PROGRA~1\QGIS33~2.2\apps\Python312\Lib
C:\Program Files\QGIS 3.36.2\bin
C:\Users\marti\AppData\Roaming\Python\Python312\site-packages
C:\PROGRA~1\QGIS33~2.2\apps\Python312
C:\PROGRA~1\QGIS33~2.2\apps\Python312\Lib\site-packages
C:\PROGRA~1\QGIS33~2.2\apps\Python312\Lib\site-packages\win32
C:\PROGRA~1\QGIS33~2.2\apps\Python312\Lib\site-packages\win32\lib
C:\PROGRA~1\QGIS33~2.2\apps\Python312\Lib\site-packages\Pythonwin
C:/Users/marti/AppData/Roaming/QGIS/QGIS3\profiles\default/python
C:\Users\marti\AppData\Roaming\QGIS\QGIS3\profiles\default\python\plugins\deepness\python3.12
C:\Users\marti\AppData\Roaming\QGIS\QGIS3\profiles\default\python\plugins\DigitizingTools\tools
C:\Users\marti\AppData\Roaming\QGIS\QGIS3\profiles\default\python\plugins\DigitizingTools
C:\Users\marti\AppData\Roaming\QGIS\QGIS3\profiles\default\python\plugins\HCMGIS/forms
C:\Users/marti/AppData/Roaming/QGIS/QGIS3\profiles\default/python/plugins\rvt-qgis
Suggestions please!

Hi @Martin-Dawson ，
This seems to be a problem with the PyTorch installation. Make sure you have installed the PyTorch version that corresponds to your CUDA version, or directly install the CPU version of PyTorch. The following links are for reference :
https://stackoverflow.com/questions/61488902/cannot-import-pytorch-winerror-126-the-specified-module-could-not-be-found
https://pytorch.org/get-started/locally/

QGIS v3.36.2 has a load of updates: https://lists.osgeo.org/pipermail/qgis-user/2024-April/054334.html.  I suspect this is the source of the issue I've raised rather than Pytorch-CUDA incompatibility. Note this update should solve the RTree issue.

Hi @Martin-Dawson ,
I tested it in Windows, and the code prompted a permission issue.
Try to launch the OSGeo4W Shell as administrator and run the following command.
pip3 install torch --force-reinstall

This worked for me.

Many thanks. Forcing the reinstall running the Shell as an administrator resolved the problem.  The earlier installs I did (not as Administrator) seem to have produced several invalid torch distributions (now removed to avoid warnings!)

From: Fanchengyan @.>
Sent: 07 May 2024 09:52
To: coolzhao/Geo-SAM @.>
Cc: Martin-Dawson @.>; Mention @.>
Subject: Re: [coolzhao/Geo-SAM] Geo-SAM fails to load QGIS 3.36.2 (Issue #44)
Hi @Martin-Dawsonhttps://github.com/Martin-Dawson ,
I tested it in Windows, and the code prompted a permission issue.
Try to launch the OSGeo4W Shell as administrator and run the following command.
pip3 install torch --force-reinstall
This worked for me.
—
Reply to this email directly, view it on GitHubhttps://github.com/coolzhao/Geo-SAM/issues/44#issuecomment-2097783094, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BC3PZPEDD23YFRR4SMXIA23ZBCI3JAVCNFSM6AAAAABHHCYW4CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOJXG44DGMBZGQ.
You are receiving this because you were mentioned.Message ID: @.***>


Docs now explain dangling images with repo:tag as :

Fixes #13397 .
The docs now explain that the untagged images(dangling) are the ones which have their repo:tag as <none>:<none>

Nice, thank you! Congratulations on your first contribution here :tada:
LGTM
ping @moxiegirl @SvenDowideit for review

LGTM :metal:


Can't find Punjabi language on Language and voice support for the Speech service

Hi there,
I am trying to develop a textToSpeech app for Punjabi language . I checked on below link and can't find a option to select language option as Punjabi language. Can you please advise where can I find the option to choose Punjabi language in Text to Speech.
There is only  one option to select Punjabi language into Text to Translate but there is no other option to speech to Text, Text to speech.
Would you be implementing Punjabi language text to Speech as well. pls advise
Here is the link that I am checking:
https://docs.microsoft.com/en-us/azure/cognitive-services/speech-service/language-support

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

ID: 4de08d1f-8114-7bef-7ef4-022f79c69742
Version Independent ID: 4c0ceab9-24ed-de01-db30-31acf1de0f48
Content: Language support - Speech service - Azure Cognitive Services
Content Source: articles/cognitive-services/Speech-Service/language-support.md
Service: cognitive-services
Sub-service: speech-service
GitHub Login: @trevorbye
Microsoft Alias: trbye

@dotnetexpo Thanks for the question. We are investigating the issue and will update you shortly.

@dotnetexpo Thanks, It's on road map to support for Punjabi in STT( Speech to Text), TTS( Text to Speech) in near future. We have forwarded to the product team. You can also raise user voice request here  for the same.
Please follow azure updates for the announcements.

We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

@ram-msft Thanks for helping out. Do you how long it take to implement road map to support for Punjabi language in STT and TTS?
Let me know if you need any help regarding Punjabi/Coding , I will be more happy to assist with gathering  requirements ,structure, grammar, vocabulary of Punjabi language in order to implement the feature.


CI for Node

App
✕ Contains the compiled JavaScript (8 ms)
Game
Game
✕ Initializes with two players (3 ms)
✕ Starts the game with a random player (1 ms)
turn
nextPlayer
✕ Sets the current player to be whoever it is not
hasWinner


Vram compressed textures not working on iOS export with GLES2

Godot version:
3.2.3 RC6

OS/device including version:
iPad Pro 9.5" 2016
GLES2
Issue description:
on GLES3, Vram compressed textures work as expected.
on GLES2, with prtvc enabled, the textures don't show and I get an debug error on Xcode:
ERROR: Failed loading resource: res://BrushTextures/smudge_01.jpg.
At: core/io/resource_loader.cpp:278:_load() - Condition "found" is true. Returned: RES()
etc

Steps to reproduce:
Minimal reproduction project:

Duplicate of https://github.com/godotengine/godot/issues/38040, https://github.com/godotengine/godot/issues/28683 and https://github.com/godotengine/godot/issues/28669.


Page gets immediately saved after generating URL if button has been clicked previously

This is a strange behavior in the content form.
Steps to reproduce:

Create a new page
Enter something in the article field (assuming the default template delivered with sulu-standard)
Enter a title into the title field
The resource locator is filled with a generated URL and the page gets immediately saved without another user interaction.

Line mentioned in https://github.com/sulu-io/sulu/pull/2006#discussion_r52904018. We should probably find a better solution for the URL here, the current approach seems quite error-prone.


Register imagePullThroughput and count with MiB

since we are on this, should we update the comment here to something like "image size in MB / image pull duration in seconds" ?https://github.com/containerd/containerd/blob/d92da14b169b98d7e48dd76c991bd2b717e46262/pkg/cri/server/metrics.go#L45-L46

It is added in https://github.com/containerd/containerd/pull/7313 and probably, and we should cherry-pick it to 1.7.

@Iceber @pacoxu I have added the same change to sbserver and modified the comments. PTAL， thanks for your review.

ping @Iceber @pacoxu PTAL.

/lgtm
/cc @samuelkarp  @ruiwen-zhao

ping @pacoxu @samuelkarp  PTAL, thanks.

Thanks @keloyang for fixing my mistake in last pr #7313 that is imagePullThroughput metrics is not registered.
/cc @fuweid
And we should cherry-pick it to 1.7.

And we should cherry-pick it to 1.7.

I don't see this PR cherry-picked to the release-1.7 branch..

/cherry-pick release/1.7

Hi folks,
I now created a manual cherry-pick: https://github.com/containerd/containerd/pull/9855.
Could you please help getting it in?


Add note for iOS9 url scheme changes

Thanks!
@facebook-github-bot shipit

@facebook-github-bot shipit


Correct SOLUTION_VERSION and remove unused test file

Issue #, if available:
#40
#39
#35
Description of changes:

Fixed issue with solution version in cdk.json. Changed to v1.1.0
Removed unused stepfunctions/workflows/test_salesforce_workflow.py file
Verified successful execution of unit tests in README.md

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Execution result of run-unit-test.sh
 ======= 33 passed in 376.60s (0:06:16) ============


How we get dynamic titles on all axis with getTitle?

when we displaying a graph of variations with a dynamic y-axis, it is not possible to show.
For example, when we displaying monthly earnings of someone. Will take as he earned On January $ 500, on February $30,000, march $5000. On  January I want to scale y-axis with $100 gap. (0- $100, $100 - $200 ... ). But it comes to February I want to change the y-axis to $1000 gaps.
In here, Graph change according to the highest-earning in what month of the whole year. It is also easy to see the difference between earnings. So how we manage this kind of thing?

I have a FAB that adjusts a value used in the title determination.  onTap function makes adjustments to the var maxX inside a setState.
The getTitles looks like this:
               getTitles: (value) {                   if (value % 5 != 0) return '';                   return (maxX - value - 1).truncate().toString();                 },
And titles are returned as calculated in the return statement.

Thanks @jonny7737! You are correct.


eslint checker does not engage automatically

I just did a clean install of syntastic (on vim 7.4.712) and would like to automatically check a javascript file.  When I have only eslint on the system, it is not enabled by default:
Syntastic version: 3.6.0-144 (Vim 704, Darwin)
Info for filetype: javascript
Global mode: active
Filetype javascript is active
The current file will be checked automatically
Available checker: eslint
Currently enabled checkers: -

However, when I issue :SyntasticCheck eslint it does correctly run eslint and shows up several errors.  The errors are not shown when the file is first loaded, nor when I issue :w.
When I have jshint (only) installed, as soon as the .js file is opened I see the errors window pop-up.
I have got no g:syntastic_* variables set, except for the recommended configuration from your documentation.
If I have jshint (only) installed, I get this:
Syntastic version: 3.6.0-144 (Vim 704, Darwin)
Info for filetype: javascript
Global mode: active
Filetype javascript is active
The current file will be checked automatically
Available checker: jshint
Currently enabled checker: jshint

and, as mentioned above, the file is checked when it's first loaded, and on subsequent saves.
Happy to send some debugging information if necessary (though not exactly sure what you'd need).

Please open Vim, run the following command, and read the very first paragraph: :help syntastic-quickstart.  Then read :help syntastic-filetype-checkers.

Thanks for the quick reply.  However I'm not sure that answers my question... I understand that there are default lists of checkers for given filetypes, and that these are kept minimal for performance reasons.  What I don't understand is, why does syntactic not use the only checker that it is aware of, given that checker is installed?
In my case, syntastic knows about both eslint and jshint -- it knows how to call them, and it knows when they are on the system.  But it will only use jshint automatically.  If I only have eslint, it knows it's there, but it won't use it.
This seems to me to be counter-productive 'out-of-the-box' behaviour.  Why not have syntastic activate eslint if it's the only checker on my system?

When I have tried editing other filetypes (such as HTML), it will both recognise and activate a checker on file load.
When I edit a JS file and have only jshint installed, it will activate the checker on file load.
When I edit a JS file and have only eslint installed, it will not activate the checker on file load (but it knows eslint is there).

Hope this helps clarify my question.  Please let me know if there's any debugging info I can provide to help track the cause of this apparently different behaviour for when eslint is installed.

This seems to me to be counter-productive 'out-of-the-box' behaviour.

Yup, you aren't supposed to run syntastic with the default configuration.  There is no one size fits all, it simply isn't possible to set defaults that are not harmful in some situations, let alone pick one that makes sense for most people.

Why not have syntastic activate eslint if it's the only checker on my system?

Because syntastic doesn't check for the existence of all checkers it knows of, it only checks for the existence of checkers in a predefined list.  Checking for rubocop on Mac OS-X used to take 5 seconds (!).  You don't want to wait for that if you don't care about rubocop.  It's much easier for you to tell syntastic which checkers you want to run.  You only have to do it once.

Please let me know if there's any debugging info I can provide to help track the cause of this apparently different behaviour for when eslint is installed.

This is the expected behaviour: jshint is on the default list, eslint isn't.  You need to set the g:syntastic_javascript_checkers list, that's all there is to it.

Thanks; this makes a lot more sense.  The thing that caused my confusion is that, to the best of my knowledge, eslint did appear within syntastic without me doing anything (I hadn’t run it from the command-line or set up an .eslintrc when it was first automatically detected by syntastic, IIRC).  That left me in the position of thinking: syntactic knows it’s there by default, knows it’s the only one I have, but still doesn’t enable it by default.  (Sorry if I’m wrong about that; seems I could well be.)
Understood that people may have other checkers they want to use — and 5 seconds: wow.
I still think that a reasonable behaviour to have would be for syntastic to enable the only checker found, in the absence of a user-specified g:syntastic_filetype_checkers variable, though I certainly appreciate that, as users are discouraged from relying on the defaults, this is likely to be a low-priority feature request :-).  Understood if you want to close this.
Thanks again for your quick and comprehensive replies (and for your work on syntactic).

The thing that caused my confusion is that, to the best of my knowledge, eslint did appear within syntastic without me doing anything

Actually, I was wrong, I remembered incorrectly what was going on.  :SyntasticInfo does probe for all checkers for the current filetype, that's where the "available" list comes from.  But this is way too slow to be done by default, especially for filetypes with many known checkers.  If you don't run :SyntasticInfo, only the checkers on the configured list (either the default, or the user-defined one) are probed.

I still think that a reasonable behaviour to have would be for syntastic to enable the only checker found, in the absence of a user-specified g:syntastic_filetype_checkers variable

As I said: this is slow.  Actually, looking for existence of the known checkers is reasonably fast, but validating them (checking versions and the like) can be dead slow.  It would be possible in theory to improve the selection of the default checkers.  However, this would involve adding a "maybe available" state to some already hairy code, and running that way isn't even supposed to be supported.
The entire list of default checkers exists for only one reason: to show people which install syntastic for the first time that it actually does something.  It was an idea that proved to be way more trouble that its worth, not only because it complicates the startup for no reason, but also because it gives people hope that they can get away with the defaults. :)


Fixing a bug in the replication metadata response logging

Codecov Report

Merging #1067 into master will decrease coverage by 0.06%.
The diff coverage is 83.33%.

@@             Coverage Diff              @@
##             master    #1067      +/-   ##
============================================
- Coverage     68.51%   68.45%   -0.07%
+ Complexity     4806     4802       -4
============================================
  Files           382      382
  Lines         30459    30461       +2
  Branches       3870     3872       +2
============================================
- Hits          20869    20852      -17
- Misses         8540     8552      +12
- Partials       1050     1057       +7

Impacted Files
Coverage Δ
Complexity Δ

...ub.ambry.protocol/ReplicaMetadataResponseInfo.java
79.16% <83.33%> (-0.84%)
15 <0> (+1)

...in/java/com.github.ambry.clustermap/Partition.java
84.81% <0%> (-3.8%)
25% <0%> (-1%)

...rc/main/java/com.github.ambry.store/BlobStore.java
81.64% <0%> (-3.2%)
68% <0%> (-4%)

...in/java/com.github.ambry.store/BlobStoreStats.java
70.41% <0%> (-1.08%)
101% <0%> (-1%)

...b.ambry.network/BlockingChannelConnectionPool.java
62.6% <0%> (-0.87%)
7% <0%> (-1%)

.../main/java/com.github.ambry.router/PutManager.java
87.17% <0%> (-0.65%)
26% <0%> (ø)

...in/java/com.github.ambry.network/SocketServer.java
82.78% <0%> (-0.41%)
13% <0%> (ø)

...va/com.github.ambry.replication/ReplicaThread.java
72.11% <0%> (-0.4%)
59% <0%> (-1%)

...ain/java/com.github.ambry.router/PutOperation.java
90% <0%> (-0.2%)
82% <0%> (ø)

...java/com.github.ambry.router/GetBlobOperation.java
88.75% <0%> (+0.4%)
32% <0%> (+1%)
:arrow_up:

... and 3 more

Continue to review full report at Codecov.

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


fix roles for planning a CHAOSScast episode

Hi @ElizabethN,
This PR fixes the roles for the episode planning steps.

Thanks @ElizabethN


generate-thumbnail Add the URLs to the Firestore not RTDB

From this readme (https://github.com/firebase/functions-samples/tree/Node-8/generate-thumbnail),
I can't pass on these two steps:

Go to your project's Cloud Console > IAM & admin > IAM, Find the App Engine default service account and add the Service Account Token Creator role to that member. This will allow your app to create signed public URLs to the images.

I still can't understand on those step and I skipped.. Please give more clear instruction for this step..
I also got an error in firebase functions log (Is this error relate to those step that i skipped?):
SigningError: A Forbidden error was returned while attempting to retrieve an access token for the Compute Engine built-in service account. This may be because the Compute Engine instance does not have the correct permission scopes specified. Permission iam.serviceAccounts.signBlob is required to perform this operation on service account<EMAIL_ADDRESS>at authClient.sign.then.catch.err (/srv/node_modules/@google-cloud/storage/build/src/file.js:1855:22)
at
at process._tickDomainCallback (internal/process/next_tick.js:229:7)

Go to the Firebase Console Database tab and see that a child of images has been created containing the Signed URLs for both images.

I can not find those data in the Database. What's wrong? And how if I need store the thumbnail URL to Firestore not in RTDB, please advice.
Thanks

I already got the answer..

i have same issue help!!!

I already got the answer..

You are the bane of my existence


Why there is a restriction for the container name?

In file docker-client/src/main/java/com/spotify/docker/client/DefaultDockerClient.java there is a restriction for the container name.
private static final Pattern CONTAINER_NAME_PATTERN = Pattern.compile("/?[a-zA-Z0-9_-]+");

For example, in docker envrionment, the . is allowed for the container name. e.g a.b.c-123 is a valid container name in docker.

Do you have a link to documentation? (Not that I'm doubting you are correct, I just want to make sure we get it right if we change something.)
I suspect that maybe a . cannot be the first character. So maybe the pattern would be something like /?[a-zA-Z0-9_-][a-zA-Z_-.]*.

My guess would be that docker used to accept a much smaller set of names for the container.
As best as I can tell from looking at the docker source code (at least in 1.13), this is the current validation for container name

here is the source for the master branch of docker (seems like everything is always moving around): https://github.com/docker/docker/blob/dba271a42ab4841dbcf2e953491e9ee728cd8e16/api/names.go#L5-L9
and we would be happy to accept patches to update the regex within docker-client 😄


feat: add assetsHash for cache naming

What is accomplished by your PR?
This PR adds another parameter to the options passed to the transformOptions function. It's an hash of the assets-filename array. I use this for the naming of the cache in the SW, because the jsonStats param does not contain anything useful for naming. Whenever the array of file(name)s to cache changes, so does the assetsHash therefore a new cache is opened and loaded with the new files.
Example implementation (a transformOptions function which passes on the assetsHash is needed):
var serviceWorkerOption = ...
const staticCacheName = 'static-' + serviceWorkerOption.assetsHash;
self.addEventListener('install', (event) => {
    event.waitUntil(caches.open(staticCacheName).then((cache) =>
        cache.addAll(serviceWorkerOption.assets)
    ));
});
self.addEventListener('activate', (event) => {
    const deleteOldCaches = caches.keys().then((cacheNames) =>
        Promise.all(cacheNames.filter((cacheName) => staticCacheName !== cacheName)
            .map((cacheName) => caches.delete(cacheName))
        )
    );
    event.waitUntil(deleteOldCaches);
});

Is there something controversial in your PR?
Maybe the default transformOptions function should pass on the assetsHash?
Checklist

[x] Run unit tests to ensure all existing tests are still passing
[x] Add new passing unit tests to cover the code introduced by your PR
[ ] Change the documentation
[ ] add an example???

pls merge this uwu


A way to reuse generic definition with custom parts

There are parts of ASN.1 definition which content depends on the object identifier.
For example, to encode/decode PKCS7 ContentInfo we need 2 steps:

decode generic ContentInfo
decode specific part based on content type using specific model.

http://tools.ietf.org/html/rfc2315#section-7

ContentInfo ::= SEQUENCE {
   contentType ContentType,
   content
     [0] EXPLICIT ANY DEFINED BY contentType OPTIONAL }

Is there a way to define ASN.1 js model to do that in one step? I.e. some kind of conditional typing/objid mapping, maybe...
The same problem goes with AlgorithmIdentifier.

Is this considered fixed?


There is a bug in the production alart  instance

I found a problem, I think is should be a bug, I will mention issues, send alarm time is in the workflow, if choose the alarm set, should be in the maintenance of the alarm instance, check on choosing the type of alarm set corresponding alarm is given priority to, the insert will be subject to the type of alarm group, if have no choice, alarm set the default to system default admin warning is the type of group, in a word, I can't write a default email type, logic is when new instances of the alarm,Be sure to check the warning type of the corresponding alarm group.Because the recipient and cc are still supported at present, there is a situation that I choose a message Alert group, and at the same time, I maintain the mailer, I personally suggest that as long as you write an email, send, and text message, in fact, is to change the Alert module.
class AlertManager

我发现个问题，我认为是个应该算是个bug，我会提issues的，就是在工作流发送告警时候，如果选择告警组，应当在维护告警实例时，查下选择的告警组对应的告警类型为主，insert的时候 以告警组的类型为准，如果没选择，默认以系统告警组default admin warning group的类型为准，总之一句话，不能一上来就写默认的邮件类型，  逻辑就是在新增告警实例的时候，一定要去查对应告警组的告警类型。因为目前还支持收件人和抄送人，存在一种情况是我选个短信告警组，同时，我又维护邮件人了，我个人建议只要写了邮件，就发，同时短信也要发，其实就是改Alert模块。
我只是实现了sms的那块的逻辑，  如果不改，永远会走email的，   我不改生成告警的实体类，只能在这上边判断告警组。。 默认系统告警组 就是邮件，只要选了不是系统告警组的，  而且是alert类型不是邮件的才能往下走我开发的sms逻辑

I would like to close this issue, if you have further information, you could reopen it @feiyalun


Support programmatically available version numbers

Currently, our client APIs do not provide the option of programmatically querying the respective version number during runtime. In connection with the preparation of a new version command for CLI (Next Gen) in carbynestack/cli-ng#20, it would therefore make sense to fill this void.

Depends on introduction of semantic versioning in CS.


 I'm open to suggestions so if someone figures out how to simplify the detection lmk

After running changed rules in this pull request (this bot only tests rules.ini changes):
Changed technologies: Engine.EGO
These apps now match: (45) 11440 12750 12830 12840 12880 12890 44300 44310 44320 44340 44350 44360<PHONE_NUMBER>70<PHONE_NUMBER>00<PHONE_NUMBER>40<PHONE_NUMBER>80<PHONE_NUMBER>70<PHONE_NUMBER>60<PHONE_NUMBER>70<PHONE_NUMBER>20<PHONE_NUMBER>20<PHONE_NUMBER>90<PHONE_NUMBER>00<PHONE_NUMBER>110 1134570 1244570 1307710 1585980 1692250 2108330 2297550 2488620 2705720
Please confirm that all these changed apps are correct.

yep it's all good, the regexes are bad though, and i still couldn't figure out how to simplify them