Setup: Correct the number of SECRETS being stored or describe the number of SECRETS dynamically

For now, we can just edit to correct this to 4 secrets. In future, can we use a variable to count the number of secrets? We'll need to store them in a list or something.
Feedback
on the page next steps, you've got the code for:
  % if secrets_need == 'org' or secrets_need == 'repo':
  3 [GitHub secrets](https://docs.github.com/en/actions/reference/encrypted-secrets) were set to store {encrypted docassemble login information} safely:

  ${ secrets_list }
  % endif

for me this rendered as:
3 GitHub secrets were set to store encrypted docassemble login information safely:
* PLAYGROUND_EMAIL
* PLAYGROUND_PASSWORD
* PLAYGROUND_ID
* SERVER_URL

that's clearly 4 secrets, not 3

Closed by PR #178


pkg/trace/info: document some TraceWriterInfo fields

This change adds clarification to the Bytes{Uncompressed,Estimated}
fields and removes the SingleMaxSize field which was unused.

@ajgajg1134 that was an unused field.

@ajgajg1134 closing for now, please feel free to re-open and merge if you think it adds value.


Last Tab adds a Scroll

When clicked the last tab in mid to small screen, a Horizontal Scroll is created within the ul. Please look into it

See same issue on larger screens too.
Tested on Chrome 49.0.2623.87 m (64-bit) with latest Materialize as of yesterday.
Dirty quick fix was to add this to my CSS:
ul.tabs {overflow: hidden; }

That seems to be a calculating problem as mentioned in #2635 and #2584.

Fixed in 4f18709


Not able to deploy using Azure git

Describe the bug
Azure git doesn't support SHA2 algorithm for Open SSH 8.8 as mentioned by several users here: https://developercommunity.visualstudio.com/t/Missing-Support-of-SHA-2-algorithm-for-O/1562711
Azure reported there are no plans to support SHA2 despite in this blog it seems to be supported https://devblogs.microsoft.com/devops/supporting-sha-2-algorithm-in-ssh-on-azure-devops/
One of the suggested things is to add this config in the ~/.ssh/config file
Host ssh.dev.azure.com HostkeyAlgorithms +ssh-rsa PubkeyAcceptedAlgorithms +ssh-rsa
The same result I was able to get by using, GIT_SSH_COMMAND='ssh -o PubkeyAcceptedKeyTypes=+ssh-rsa -o HostKeyAlgorithms=+ssh-rsa' before pull from git
To Reproduce
Steps to reproduce the behavior:

Set a project with azure git
Deploy
See error
Unable to negotiate with <IP_ADDRESS> port 22: no matching host key type found. Their offer: ssh-rsa fatal: Could not read from remote repository.

This issue is also reported more recently (October 2021) here: https://developercommunity.visualstudio.com/t/Git-SSH-access-offers-weak-algorithms-r/1547526
And the state of the Azure SSH service is still the same today:
$ ssh -V
OpenSSH_8.8p1 Debian-1, OpenSSL 1.1.1m  14 Dec 2021
$ ssh -vvv ssh.dev.azure.com 2>&1 | grep -A16 'KEXINIT proposal'
debug2: local client KEXINIT proposal
debug2: KEX algorithms: curve25519-sha256,curve25519-sha256@libssh.org,ecdh-sha2-nistp256,ecdh-sha2-nistp384,ecdh-sha2-nistp521,diffie-hellman-group-exchange-sha256,diffie-hellman-group16-sha512,diffie-hellman-group18-sha512,diffie-hellman-group14-sha256,ext-info-c
debug2: host key algorithms: ssh-ed25519-cert-v01@openssh.com,ecdsa-sha2-nistp256-cert-v01@openssh.com,ecdsa-sha2-nistp384-cert-v01@openssh.com,ecdsa-sha2-nistp521-cert-v01@openssh.com,sk-ssh-ed25519-cert-v01@openssh.com,sk-ecdsa-sha2-nistp256-cert-v01@openssh.com,rsa-sha2-512-cert-v01@openssh.com,rsa-sha2-256-cert-v01@openssh.com,ssh-ed25519,ecdsa-sha2-nistp256,ecdsa-sha2-nistp384,ecdsa-sha2-nistp521,sk-ssh-ed25519@openssh.com,sk-ecdsa-sha2-nistp256@openssh.com,rsa-sha2-512,rsa-sha2-256
debug2: ciphers ctos: chacha20-poly1305@openssh.com,aes128-ctr,aes192-ctr,aes256-ctr,aes128-gcm<EMAIL_ADDRESS>debug2: ciphers stoc: chacha20-poly1305@openssh.com,aes128-ctr,aes192-ctr,aes256-ctr,aes128-gcm<EMAIL_ADDRESS>debug2: MACs ctos: umac-64-etm@openssh.com,umac-128-etm@openssh.com,hmac-sha2-256-etm@openssh.com,hmac-sha2-512-etm@openssh.com,hmac-sha1-etm@openssh.com,umac-64@openssh.com,umac-128@openssh.com,hmac-sha2-256,hmac-sha2-512,hmac-sha1
debug2: MACs stoc: umac-64-etm@openssh.com,umac-128-etm@openssh.com,hmac-sha2-256-etm@openssh.com,hmac-sha2-512-etm@openssh.com,hmac-sha1-etm@openssh.com,umac-64@openssh.com,umac-128@openssh.com,hmac-sha2-256,hmac-sha2-512,hmac-sha1
debug2: compression ctos: none,zlib@openssh.com,zlib
debug2: compression stoc: none,zlib@openssh.com,zlib
debug2: languages ctos:
debug2: languages stoc:
debug2: first_kex_follows 0
debug2: reserved 0
debug2: peer server KEXINIT proposal
debug2: KEX algorithms: diffie-hellman-group1-sha1,diffie-hellman-group14-sha1,diffie-hellman-group-exchange-sha256
debug2: host key algorithms: ssh-rsa
debug2: ciphers ctos: aes256-cbc,aes192-cbc,aes128-cbc,aes128-ctr,aes256-ctr
debug2: ciphers stoc: aes256-cbc,aes192-cbc,aes128-cbc,aes128-ctr,aes256-ctr
debug2: MACs ctos: hmac-sha2-256,hmac-sha2-512
debug2: MACs stoc: hmac-sha2-256,hmac-sha2-512
debug2: compression ctos: none
debug2: compression stoc: none
debug2: languages ctos:
debug2: languages stoc:
debug2: first_kex_follows 0
debug2: reserved 0
debug1: kex: algorithm: diffie-hellman-group-exchange-sha256
debug1: kex: host key algorithm: (no match)
Unable to negotiate with <IP_ADDRESS> port 22: no matching host key type found. Their offer: ssh-rsa

Technically it might be possible to re-enable SHA1 for the host-key algorithm but it is known to be insecure and we also have requirements from customers to ensure SHA1 is disabled everywhere in Lagoon for security reasons.
So therefore I don't know if there's anything we can do in Lagoon to fix this. On the Azure side they need to either add a non-RSA host key, or enable an RSA SHA2 host key algorithm.

The solution some customers have come up with is to run an ssh-proxy, there is a write up at https://www.cando-image.com/de/setting-azure-git-ssh-proxy


rmf_traffic_editor: 1.3.0-1 in 'rolling/distribution.yaml' [bloom]

Increasing version of package(s) in repository rmf_traffic_editor to 1.3.0-1:

upstream repository: https://github.com/open-rmf/rmf_traffic_editor.git
release repository: https://github.com/ros2-gbp/rmf_traffic_editor-release.git
distro file: rolling/distribution.yaml
bloom version: 0.10.7
previous version for package: null

rmf_building_map_tools
* Added support for fuel textures (#342 <https://github.com/open-rmf/rmf_traffic_editor/issues/342>)
* Convert wall textures from 1d to 2d (#338 <https://github.com/open-rmf/rmf_traffic_editor/issues/338>)
* [Optimization] Remove duplicated textures (#337 <https://github.com/open-rmf/rmf_traffic_editor/issues/337>)
* clean dep and update readme (#336 <https://github.com/open-rmf/rmf_traffic_editor/issues/336>)
* building_map_server: don't crash when missing image file (#334 <https://github.com/open-rmf/rmf_traffic_editor/issues/334>)
* Fix material values for sdf compliance (#330 <https://github.com/open-rmf/rmf_traffic_editor/issues/330>)
* avoid crashing when generating undefined floor polygons. cleanup. (#322 <https://github.com/open-rmf/rmf_traffic_editor/issues/322>)
* improve usage of Shapely on very complex floor polygons (#321 <https://github.com/open-rmf/rmf_traffic_editor/issues/321>)
* auto download crowdsim models (#316 <https://github.com/open-rmf/rmf_traffic_editor/issues/316>)
* rename building_map_tools (#310 <https://github.com/open-rmf/rmf_traffic_editor/issues/310>)
* Account for package rename
* Rename packages and delete moved packages (#308 <https://github.com/open-rmf/rmf_traffic_editor/issues/308>)
* migration to open-rmf org, rename to rmf_building_map_tools
* Contributors: Geoffrey Biggs, Luca Della Vedova, Morgan Quigley, youliang

rmf_traffic_editor
* Feature/display layer transforms in freefleet format (#347 <https://github.com/open-rmf/rmf_traffic_editor/issues/347>)
* Feature/layer rendering palette mapping (#344 <https://github.com/open-rmf/rmf_traffic_editor/issues/344>)
* Fix asset path after package renaming (#341 <https://github.com/open-rmf/rmf_traffic_editor/issues/341>)
* Automatic alignment of robot-map layers to floorplans (#340 <https://github.com/open-rmf/rmf_traffic_editor/issues/340>)
* Fix/ci package name (#339 <https://github.com/open-rmf/rmf_traffic_editor/issues/339>)
* clarify labels on property add/delete buttons (#326 <https://github.com/open-rmf/rmf_traffic_editor/issues/326>)
* handle editing multiple layers with same name. (#328 <https://github.com/open-rmf/rmf_traffic_editor/issues/328>)
* fix #324 <https://github.com/open-rmf/rmf_traffic_editor/issues/324>, update layer image immediately after OK button (#327 <https://github.com/open-rmf/rmf_traffic_editor/issues/327>)
* Bug/add layer button not visible in new building (#313 <https://github.com/open-rmf/rmf_traffic_editor/issues/313>)
* provide zoom-reset and clamp on scale factor (#318 <https://github.com/open-rmf/rmf_traffic_editor/issues/318>)
* avoid crash in empty crowdsim save routine (#312 <https://github.com/open-rmf/rmf_traffic_editor/issues/312>)
* rename building_map_tools (#310 <https://github.com/open-rmf/rmf_traffic_editor/issues/310>)
* Rename packages and delete moved packages (#308 <https://github.com/open-rmf/rmf_traffic_editor/issues/308>)
* Refactoring and Migration #308 https://github.com/open-rmf/rmf_traffic_editor/pull/308
* Contributors: Geoffrey Biggs, Luca Della Vedova, Morgan Quigley, youliang

rmf_traffic_editor_assets
* Rename package
* Merge remote-tracking branch 'sourcedir/master'
* Move files in preparation for repository shift
* Contributors: Geoffrey Biggs

test_maps
* rename building_map_tools (#310 <https://github.com/open-rmf/rmf_traffic_editor/issues/310>)
* Rename packages and delete moved packages (#308 <https://github.com/open-rmf/rmf_traffic_editor/issues/308>)
* Contributors: Geoffrey Biggs, youliang

Thanks for point it out @sloretz. Fixed here and sent now there's a new PR for it: https://github.com/ros/rosdistro/pull/29749


Help with Camunda's Run distro (Docker)

Camunda has a self-contained distro called Run. There is also a docker variant which I'm using.
As I'm a novice to Java, Tomcat, etc, I'm struggling to overlay your instructions onto Camunda Run. In particular, I'm reluctant to tinker with the internal folder as it implies "off limits" :).
Could some instructions to integrate this plugin into Camunda Run (docker variant) be created? Some ideas:

craft a child Dockerfile to build a custom image with this plugin already "deployed"
use -v [local file system]:[container file system] volume mapping(s) to inject this plugin at container start

I use the latter technique to inject my custom .bpmn files into the /configuration/resources folder.  However, I've found that only resources in the same deployment (startupDeployment) can be referenced in form keys (embedded:deployment://). Whereas I'd prefer a once-off deployment of this plugin be accessible from all processes, even those done after startup (e.g. via Modeler).

@julrichkieffer yes I have had a similar request on the forums.  The camunda run Distro uses springboot and there is auto-scanning for components: so we can package the plugin as a jar just for the Run Distro and it will be picked up by Run (all variants) when you drop it into the library folder.
I will look to do it this week.

That's awesome, and thanks for your effort in advance. Might this jar become a release artefact so that any plugin updates are automatically included in the jar?  (I'm particularly keen on including the FormIO json within the .bpmn file, so that it's versioned with the process itself. Though nested forms, validations, boundary events, pre-setting submision values may thwart this approach 🍰).
Also, will this jar simply drop into the /configuration/userlib folder (not Tomcat's lib folder) to align with Run's design? I would then be able to volume map it when I start Camunda Run as a docker image.

yes it would be a jar in the release
see the conversations here: https://forum.camunda.org/t/formio-plugin-for-camunda-tasklist/22215/26  It covers discussion about using forms with the BPMN file, pros, cons, etc.
and yes we can build a example of build a docker container with the Run container and align with it's intended usage.

Any news on a release .jar that would be compatibale with Camunda Run's /configuration/userlib extensibility?

yes it would be a jar in the release
see the conversations here: https://forum.camunda.org/t/formio-plugin-for-camunda-tasklist/22215/26 It covers discussion about using forms with the BPMN file, pros, cons, etc.
and yes we can build a example of build a docker container with the Run container and align with it's intended usage.

Any news on a release .jar that would be compatibale with Camunda Run's /configuration/userlib extensibility?
#NotAJavaDeveloper #CitizenDeveloper


ModuleNotFoundError: No module named 'pillow_avif'

Everything was working fine yesterday on Google Collab
but today at the "Start Stable-Diffusion" stage I have a sudden error message ending by ModuleNotFoundError: No module named 'pillow_avif', anyone can help me on this?

i had the same "no module named 'pillow_avif' error, so i deleted the whole SD folder and reinstalled a new copy of the automatic1111. now i have a different error, which is this. hope someone can look into this as well.

ModuleNotFoundError                       Traceback (most recent call last)
 in <cell line: 6>()
4 import sys
5 import fileinput
----> 6 from pyngrok import ngrok, conf
7 import re
8
ModuleNotFoundError: No module named 'pyngrok'

NOTE: If your import is failing due to a missing package, you can
manually install dependencies using either !pip or !apt.
To view examples of installing some common dependencies, click the
"Open Examples" button below.

Ditto, I get the same pillow_avif error in my typical notebook as well as a new one with fresh install

If you're missing pillow-avif add a line of code and run -  !pip install pillow-avif-plugin

i had the same "no module named 'pillow_avif' error, so i deleted the whole SD folder and reinstalled a new copy of the automatic1111. now i have a different error, which is this. hope someone can look into this as well.
ModuleNotFoundError Traceback (most recent call last) in <cell line: 6>() 4 import sys 5 import fileinput ----> 6 from pyngrok import ngrok, conf 7 import re 8
ModuleNotFoundError: No module named 'pyngrok'
NOTE: If your import is failing due to a missing package, you can manually install dependencies using either !pip or !apt.
To view examples of installing some common dependencies, click the
"Open Examples" button below.

Im getting this error today

same error today appeared

You can solve this problem by installing the library.
For your reference
https://colab.research.google.com/drive/14vEexu7JNPFfI4pSoeghXZA4GCgh2B-v?usp=sharing

You can solve this problem by installing the library.
For your reference
https://colab.research.google.com/drive/14vEexu7JNPFfI4pSoeghXZA4GCgh2B-v?usp=sharing

If you're missing pillow-avif add a line of code and run - !pip install pillow-avif-plugin

Thanks for the solution

If you're missing pillow-avif add a line of code and run - !pip install pillow-avif-plugin

thanks a lot, works perfect

fixed

Yes that fixed it for me too, I had added both of those lines before Requirements step
!pip install diskcache !pip install pillow-avif-plugin

It worked for me from updating to this newer version. Thanks!


TDI mode cannot be set to True

Steps to reproduce:

Start a new project
Add NewTab for VB6 in components dialog
Add NewTab to default form
Accept default name of "NewTab", or change to anything else
Try to set property TDIMode to True
Message: This property needs to be set inmediately (sic) after adding the NewTab control (without changing other properties first)

Cannot reproduce. Are you using a previous version or the current one?

November 5 edition .ocx

I have created custom themes with previous versions on other projects. Is there a chance of registry residue?

Check if the key HKEY_CURRENT_USER\Software\VB and VBA Program Settings\NewTab exists, if it exists, please export its content and paste it here.

No registry key exists.

I'm unable to reproduce the issue because it does not happen here with the same ocx version (that is the last/current one).
If there are no registry keys, then something must be changing some property.
Maybe an add-in?
One thing you could do is to use the NewTab in source code, use it in a test project, put a break point at the line mControlJustAdded = False and at mControlJustAdded = nValue too.
Also at the two lines: MsgBox "This property needs to be set i...
And when the program stops in one of then, press Control+L and check from where it is being called (and post the screenshot here).

Only reason I wanted to try TDI is because of poor performance changing tabs while using an anchor.ocx.
Will TDI improve performance?

I don't understand the question. TDI is for Tabbed Documents Interface, as opposed to MDI (Multiple Documents Interface) and SDI (Single Document Interface).
It does not have to do with "performance". What performance are you talking about?
Anyway, a tab control is unlikely to have anything to do with "performance", it is only a visual control.

Thank you for the source code suggestion.
In regards to my question about improving performance, there is a delay when switching tabs and it seems to be a moving/repainting controls issue.
Without stepping through the source code, this appears to be what is slowing performance:

Click a new tab, the ocx moves the contained controls from offscreen left or right
The anchor control resizes the controls, and paints them
The tab control then repaints the controls again
I've included the newtab.refresh event and it seems to marginally help. I will test the .ocx as a .ctl to find out more information. In the meantime, it would be helpful to see screenshots of the difference between newtab TDI, MDI, and SDI so I can visualize if we are on the same page.

I have no idea about that "anchor control" and how it works together with the NewTab control, but yes, there is a small delay when changing tabs due to a visual effect.
To change that, set the TabTransition property to ntTransitionImmediate (and please tell me if that solved your issue)

Yes, I have already tried that setting. I will just live with the performance as it is. Thanks for an excellent tab control.

If you could make a sample project where you can show the "performance" issue I could see it.
There is no much performance involved, just to show and hide controls.
My guess is that the issue must be some other control, that anchor control or something else.

Please recommend an anchor / resize / move controls solution that is compatible with NewTab01... obviously, a user will resize the form, so what is the best way to resize a form's control with NewTab01? Thanks

I can't recommend because I don't use such controls, I do that by code.
I don't think that it is a good idea to automatically resize controls in most of cases.

I start the form at a common laptop resolution of 1366x768. If the user maximize the form on 1920x1080, it looks strange for the listviews to not size with the form.
Also, I am not sure if it is just my system, but when the Windows graphics Scale is set to anything but 100%, for example 200%, the version 4.0 of newtab01 changes scale in a weird way in the IDE...?

Add exceptions to the resizer for not to handle these controls, and position/size the tab control and its contained controls by code.
There is another option: put the content of each tab into a PictureBox containing all the controls of that tab and see if that fixes it.

when the Windows graphics Scale is set to anything but 100%, for example 200%, the version 4.0 of newtab01 changes scale in a weird way in the IDE...?

The tab control sets the non-visible controls (the controls in tabs that are not the active one) to a left position shifted, for example -75000 twips. This is the same as the SSTab control does.
But... the problem is that at 200% DPI or 300% DPI the screen does not have 75000 twips to the left, but less.
Then instead of shifting -75000 twips, I shift less, based on the DPI setting.
And if the DPI setting changes, all is converted automatically.
But... if a resizer control stores some values at some DPI setting, and the DPI setting changes... yes, those stored values won't work correctly.
For sizing/placing controls I made an event: TabBodyResize
for example:
Private Sub NewTab1_TabBodyResize()
    NewTab1.ControlMove "Picture1", NewTab1.TabBodyLeft, NewTab1.TabBodyTop, NewTab1.TabBodyWidth, NewTab1.TabBodyHeight
End Sub

For getting or setting the left of the controls, there is the method .ControlLeft and for moving the controls .ControlMove
Why? Because if the controls are not in the active tab, their Left property that you can read is changed, and if you set it, they will go to a wrong position. So use NewTab1.ControlLeft and NewTab1.ControlMove instead.
With the SSTab control there is the same issue but it does not have these helper functions.
So, in summary, try with pictureboxes on each tab containing all the controls. If that does not work, then place/size the controls by code in the TabBodyResize event using NewTab1.ControlLeft and NewTab1.ControlMove if you are going to work with controls that are in tabs that are not the active one (sizing positioning non currently visible controls).
I think I'll have to write a help for this, but it is basically what I tried to explain here.
A third option is to use another tab control, a tabstrip. I guess the resizer should work with the tabstrip control.

PS: I just issued an update with several minor bug fixes.

Thank you Eduardo! You're the best. I will install the update, and I will try your suggestions and report back ASAP.

Eduardo, your suggestion of using 'picturebox as a container for controls' works with Scale <> 100%, and with the resizing anchor control as well. It also seems to switch tabs more efficiently with this new update. Thanks again for everything!

Hello, I would need a sample project and/or the steps to reproduce the issue.

the following is repeatable under Windows 10 & 11, without anchor controls

right-click desktop, select Display Settings
under Scale & Layout, select 100% scale (all monitors)
start new .exe project in vb6, add VBCCR17 and newTab components
add form and set scalemode to pixels
size form to 1366 x 768
Add tab control, make 4 tabs, segoe mdl2 assets icons for each, and a caption
add a listview and listbox to each tab, run and save.
go back to Display settings and set windows scale to 200% (all monitors)
reboot, open project
refer to youtube video https://youtu.be/BOhLIzXgw5U for behavior

Hello, I could not replicate the issue, it does not happen here.
Please make a sample project, zip it and upload it here, so we can be sure we are testing the same thing (and verify that the issue happens in that sample project).

Excellent. You have definitely improved the issue in the IDE. However, some strange artifacts remain. Please check your email for more descriptions.

It must be something with your tab: my guess: many tabs and little screen space.
But anyway I don't know how to recreate it, so I need a sample project where the issue occurs.

Hello, I made another change that might improve the issue at run-time. The project is updated.

Hi Eduardo, I had to revert to the previous version.
The newest version at runtime and compiled .exe makes the tabs disappear -
and the Segoe MDL icons get stacked on top of each other on the far left.

I need a sample project (simple, basic) to be able to reproduce the issue, and then study what is happening.

I sent an email to Balkesoft and never received a reply.

It was in the spam folder.

I answered the email, I don't know if you received it.
Anyway I issued an update. I'm quite sure that this version should not have the issue you mentioned regarding the previous version.


Bad Gateway Response

I have setup Localtunnel, and have successfully received the sub domain, however any attempt to test my api with the url results in a "502 Bad Gateway" response.
Is there something i didn't set up correct? I can confirm my localhost:5000 is working, and that is the port I provided.
Thanks

sorry I meant for this to be in the client - I will move there.

I am having the same issue, any updates on this?


Error importing LinearCorePotential

https://github.com/ContactEngineering/Adhesion/blob/master/examples/adhesive_plastic_simulation_cone.ipynb
I think it should be from Adhesion.Interactions.cutoffs import LinearCorePotential instead of from Adhesion.Interactions import LinearCorePotential
I was getting this error:

but changing it fixed the error:

However, I cannot make changes to the function definitions, and I guess that it is the same issue that is causing this error:

@sannant Please let me know if this is something that needs to be fixed on your end, or is it something that I am doing wrong?

@sannant It looks like this is related to #63 . Is this something that I can fix/submit a request, or does this need to be done from your end?

You are very welcome to submit a pull request. We will review it anyway.

@sannant could you please give me the access rights to submit pull requests?
I'm trying to push the modified code and it is giving me the following error:

Thanks, I have submitted the pull request.

@jotelha @sannant
The change (line 44 - linearize_core) shows up in contact.engineering github after being merged:

However, when I am running a code that calls that function using singularity run docker://imteksim/jupyterlab-surfacetopography:latest , the change is not being reflected; the error shows the old code (line 44 -  LinearCorePotential):

Is there something that needs to be done for the changes to be reflected when executing using Docker images?

Hi @sitangshugk95 the containers won't update by themselves, we will build a new one with some future release soon.

Thanks @jotelha.
Could you give me a rough estimate of when you expect to do it? It would help me plan my work accordingly, as I can't proceed unless it is done.
Also, since it looks like Docker images is the approach I would have to stick to to use contact.engineering for the near future, what would be a realistic frequency (biweekly/monthly/other) to update the containers based on your workload? I don't think it would be needed very frequently anyway once the basic debugging is done, but I would still like to have that info and inform my advisor.

Can't give any estimate here, since ContactEngineering is an academic project, and many of the repositories like Adhesion grow organically as people collaborate loosely while working on their PhD and master projects, without the resources to provide support. My own project is rather unrelated and I am only helping a little on the side with the container-related stuff.
If you need your own modifications in the container timely, you can bind your local copy of whatever repository you changed into the container instance and override whatever is already present within the container.
Have a look at https://github.com/ContactEngineering/ce-container-stack/tree/master/docker/jupyterlab-SurfaceTopography#changes-to-the-container
Let's say you want a modified Adhesion package and you have it locally at /path/to/git/repo/Adhesion. In short, what you could do to have the modified repository in the container, is:

Launch container instance with bind mount, i.e.docker run -v /path/to/git/repo/Adhesion:/home/jovyan/Adhesion -p 8888:8888 imteksim/jupyterlab-surfacetopography:latest

Enter a terminal within the container as usual via the JupyterLab web interface
As described at https://github.com/ContactEngineering/ce-container-stack/tree/master/docker/jupyterlab-SurfaceTopography#changes-to-the-container, use

conda deactivate and
export PATH=$(echo "$PATH" | sed -e 's|/opt/conda[^:]*:||g')  to remove any conda-related residue from the path to break out of the container's default conda environment.

change to /home/jovyan/Adhesion or wherever you mounted your repo within in the container
install with pip install . or pip install -e . , latter for editable in-place. That should override whatever Adhesion version is already available within the SurfaceTopography kernel in the container.
If you need a public online branch, you may install directly from the web, i.e. with pip install git+https://github.com/ContactEngineering/Adhesion.git@master, the suffix pointing to the actual branch

Hope that helps.

Thank you for the very detailed answer, so sorry again for my unfamiliarity with Linux.

Launch container instance with bind mount, i.e.docker run -v /path/to/git/repo/Adhesion:/home/jovyan/Adhesion -p 8888:8888 imteksim/jupyterlab-surfacetopography:latest

@jotelha  Quick question - could you please tell me how to modify the above command to use it with singularity without root privileges? eg; I generally use singularity run docker://imteksim/jupyterlab-surfacetopography:latest to launch the images, and I have it locally in /home/grads/s/sitangshugk95/Adhesion .
I tried the following, but it was probably wrong:

This question boils down to looking into the singularity documentation, https://docs.sylabs.io/guides/3.0/user-guide/bind_paths_and_mounts.html will tell you to run
singularity run --bind /home/grads/s/sitangshugk95/Adhesion:/home/jovyan/Adhesion docker://imteksim/jupyterlab-surfacetopography:latest

but isn't your home directory mounted anyway with singularity?

All I to do run the docker images is type singularity run docker://imteksim/jupyterlab-surfacetopography:latest in console - I am not entirely sure if my home directory is mounted with singularity: does it matter and how do I check it?
Anyway, I used singularity run --bind /home/grads/s/sitangshugk95/Adhesion:/home/jovyan/Adhesion docker://imteksim/jupyterlab-surfacetopography:latest and followed the steps you mentioned here and it works perfect - the changes have been reflected. Thanks a lot! Are these changes stored permanently? Or is it only local to that instance of the container, and will go away once I close that and open a new instance?


Postgres support for UUID?

I was looking through the source code a bit, but wasn't certain if UUID is a natively supported field.
I have some basic tables that look like this:
CREATE TYPE operation AS ENUM ('create', 'update', 'delete');

CREATE TABLE IF NOT EXISTS events (
    event_id    uuid PRIMARY KEY NOT NULL,  -- id of event
    namespace   uuid NOT NULL,              -- user's "namespace" where their events are stored
    event_time  bigint NOT NULL,            -- when event occurred
    op          operation NOT NULL,         -- operation that occurred
    acted_on    uuid NOT NULL               -- key of item that was acted on
);

I'm trying to do a test query on it to see if the whole setup works. I have a local instance of the db set up, and the DATABASE_URL properly set, but when I compile I get an error from some pretty simple code:
async fn get_all_in_ns(pool: PgPool) -> anyhow::Result<()> {
    let rows = sqlx::query!(
        r#"SELECT
             event_id, namespace, event_time, op, acted_on
             FROM events
             WHERE event_time > 5"#,
    )
        .fetch(&mut pool)
        .await
        .unwrap();
    Ok(())
}

-> "unknown field type ID: 2950"
I haven't seen any other errors like this, and my assumption is that sqlx doesn't support postgres UUIDs, but I'm not certain.
Could someone advise? If I can use statically typed sql queries, that would be a big win. We use UUIDs extensively from uuid-ossp

Thank you for reporting the issue.
We should support UUID (with the uuid feature) for Postgres but it doesn't look like we have an integration test for that type to prove it. Can you try enabling the feature and see if that allows it to work (we'll note a task to add an integration test for UUID support in any case).
# something like this
sqlx = { version = "0.1.1", features = [ "uuid" ] }

We do not support custom types at this time however. You'll need to do op::text in your SELECT ... in order to fetch the operation.

I had no idea this feature existed. I'll enable it when I get home and try it out. Thank you :D

That did it (the op::text too) thank you!

I might recommend a FAQ in the docs somewhere about enums and features for SQL and stuff and all that.
The library itself is great though. Thank you for all of the hard work!

I know this probably isn't the perfect place to ask, but what's the correct way to do an INSERT of a Uuid value?
I have a Uuid from the uuid crate, and I'm trying to .bind() it into a raw SQL insert, like this:
      sqlx::query_as::<_, db_types::Record>(
        r#"INSERT INTO evaluations (part_id, evaluation_id, guid, started_at)
                            VALUES ($1, $2, $3, $4, $5);
        "#,
    )
    .bind(&part_id)
    .bind(&part.evaluation.evaluation_id)
    .bind(&part.evaluation.guid)  // <-  type is uuid::Uuid
    .bind(&part.evaluation.started_at)

and I'm seeing this errror:
thread 'test_batch_queries' panicked at 'called `Result::unwrap()` on an `Err` value: ColumnDecode { index: "\"guid\"", source: "mismatched types; Rust type `alloc::string::String` (as SQL type `TEXT`) is not compatible with SQL type `UUID`" }', apps/api/tests/db_tests.rs:39:10

the .bind() is coercing the uuid to string maybe?


[Event Request] Codeunit 5817 "Undo Posting Management" - OnUpdatePurchLineAfterModify

Hi, would it be possible to get an event OnUpdatePurchLineAfterModify in codeunit 5817(Undo Posting Management), like this:
procedure UpdatePurchLine(PurchLine: Record "Purchase Line"; UndoQty: Decimal; UndoQtyBase: Decimal; var TempUndoneItemLedgEntry: Record "Item Ledger Entry" temporary)
    var
        xPurchLine: Record "Purchase Line";
        PurchSetup: Record "Purchases & Payables Setup";
        ReservePurchLine: Codeunit "Purch. Line-Reserve";
        IsHandled: Boolean;
    begin
        PurchSetup.Get();
        with PurchLine do begin
            xPurchLine := PurchLine;
            case "Document Type" of
                "Document Type"::"Return Order":
                    begin
                        "Return Qty. Shipped" := "Return Qty. Shipped" - UndoQty;
                        "Return Qty. Shipped (Base)" := "Return Qty. Shipped (Base)" - UndoQtyBase;
                        InitOutstanding;
                        if PurchSetup."Default Qty. to Receive" = PurchSetup."Default Qty. to Receive"::Blank then
                            "Qty. to Receive" := 0
                        else
                            InitQtyToShip;
                        OnUpdatePurchLineOnAfterSetQtyToShip(PurchLine);
                        UpdateWithWarehouseReceive;
                    end;
                "Document Type"::Order:
                    begin
                        "Quantity Received" := "Quantity Received" - UndoQty;
                        "Qty. Received (Base)" := "Qty. Received (Base)" - UndoQtyBase;
                        InitOutstanding;
                        if PurchSetup."Default Qty. to Receive" = PurchSetup."Default Qty. to Receive"::Blank then
                            "Qty. to Receive" := 0
                        else
                            InitQtyToReceive;
                        OnUpdatePurchLineOnAfterSetQtyToReceive(PurchLine);
                        UpdateWithWarehouseReceive;
                    end;
                else
                    FieldError("Document Type");
            end;
            Modify;
            IsHandled:=false;
            OnUpdatePurchLineAfterModify(PurchLine,IsHandled);
            if not IsHandled then
                RevertPostedItemTracking(TempUndoneItemLedgEntry, "Expected Receipt Date");
            xPurchLine."Quantity (Base)" := 0;
            ReservePurchLine.VerifyQuantity(PurchLine, xPurchLine);

            UpdateWarehouseRequest(DATABASE::"Purchase Line", "Document Type", "Document No.", "Location Code");

            OnAfterUpdatePurchline(PurchLine);
        end;
    end;

[IntegrationEvent(false, false)]
local procedure OnUpdatePurchLineAfterModify(PurchLine: Record "Purchase Line"; var IsHandled: Boolean)
begin
end;

Thanks

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


CI for #4179

The Lightstep tracer supports multiple propagation modes.
See https://www.envoyproxy.io/docs/envoy/latest/api-v3/config/trace/v3/lightstep.proto#enum-config-trace-v3-lightstepconfig-propagationmode
With this PR it is now possible to set a list of propagation modes in
the config field of the TracingService.
Signed-off-by: Paul Salaberria<EMAIL_ADDRESS>
[x] I made sure to update CHANGELOG.md.
[x] This is unlikely to impact how Ambassador performs at scale.
[ ] My change is adequately tested.
[x] No need to update DEVELOPING.md.

Excellent! I'll go land #4179 now that we have a good CI run here. Thanks, @psalaberria002!


Add extra debug logging for DB connection in minder-server

This is helpful for debugging DB configuration issues in k8s.
Summary
Provide a brief overview of the changes and the issue being addressed.
Explain the rationale and any background necessary for understanding the changes.
List dependencies required by this change, if any.
Fixes #(related issue)
Change Type
Mark the type of change your PR introduces:

[ ] Bug fix (resolves an issue without affecting existing features)
[x] Feature (adds new functionality without breaking changes)
[ ] Breaking change (may impact existing functionalities or require documentation updates)
[ ] Documentation (updates or additions to documentation)
[ ] Refactoring or test improvements (no bug fixes or new functionality)

Testing
Outline how the changes were tested, including steps to reproduce and any relevant configurations.
Attach screenshots if helpful.
Review Checklist:

[x] Reviewed my own code for quality and clarity.
[ ] Added comments to complex or tricky code sections.
[ ] Updated any affected documentation.
[ ] Included tests that validate the fix or feature.
[ ] Checked that related changes are merged.

coverage: 52.331% (-0.005%) from 52.336%
when pulling 3f91c0df223cf25f88da9d1467c5e3d66e5f5a51 on debug-info-db-server-cli
into bebb68983b73a1b7b43ef9fcd6ca75f86d306e43 on main.


fix(protocol): fix the message.to == etherVault bug

Reported by Victor from QuillAudits: if message.to == etherVault bug, then message can call releaseEther to drain all Ether from the audit.
I also added a modifier onlyWhenNamed so make sure if we change the "ether_vault" mapping to a different address mistakenly (we will probably register more address with different names, and it is possible that we accidentally change "ether_vault"), the the original EtherVault contract will no longer trust (or give access to) the Bridge contract. Otherwise, if the bridge still have access to the old EtherVault contract, message.to can be set to the old EtherVault contract to withdraw Ether.

[ ] @adaki2004 can you verify the bug is real or not?
[ ] @adaki2004 could you remove authorize() from EtherVault and use onlyFromNamed("bridge") instead?

@adaki2004 could you remove authorize() from Bridge and use onlyFromNamed("bridge") instead?

Reported by Victor from QuillAudits: if message.to == etherVault bug, then message can call releaseEther to drain all Ether from the audit.
* [ ]  @adaki2004 can you verify the bug is real or not?

I analyzed the code and i see no bug here - maybe a "mistake" like: why would anyone want to send ether/erc20 to our EtherVault ? Is there a way to get a more detailed description / workflow of the problem or a contact to Victor ?
How i see:
EtherVault has a receive() fallback, so the _invokeMessageCall(message, msgHash, gasLimit) could succeed.
If it succeeds, how anyone would get over this check below in the recallMessage ?
bool received = _proveSignalReceived(
            _signalForFailedMessage(msgHash), message.destChainId, proof
        );

I do not see here a bug (maybe we can restrict the sending to EtherVault, the before mentioned reasons, to be stupid-proof) but i so far i could not figure out how it could drain all ether. 🤔

@adaki2004 a design decision to be made here: if EtherVault is not shared by more than one Bridge, maybe we simply take custody of Ether in the Bridge, I think that's even simpler.
If we do need to share the same Ether Vault among multiple Bridges, then we still need the authorize() function. Now the question is: do we really want to share the EtherVault among multiple Bridges/Rollups, what can such a shared EtherVault offer

@adaki2004 a design decision to be made here: if EtherVault is not shared by more than one Bridge, maybe we simply take custody of Ether in the Bridge, I think that's even simpler.
If we do need to share the same Ether Vault among multiple Bridges, then we still need the authorize() function. Now the question is: do we really want to share the EtherVault among multiple Bridges/Rollups, what can such a shared EtherVault offer

I let it answer @Brechtpd
But just to highlight this design convo is a bit unrelated to the initial finding: a 'draining issue if EtherVault is the message.to reported by Quill'. I still could not figure it out how is it possible, could you please ask Victor to comment in this PR ?


[Bugfix] Fix CLI arguments in OpenAI server docs

vllm-openai-server command is not defined so the documentation failed to generate the sphinx-argparse docs. In the future, this patch should be replaced by the changes made by #4167.
FIX #4522
FIX #4707

Clicked too soon...
https://vllm--4709.org.readthedocs.build/en/4709/serving/openai_compatible_server.html#command-line-arguments-for-the-server

Build log here https://readthedocs.org/projects/vllm/builds/24329938/ but i'm not sure what happened...

https://github.com/vllm-project/vllm/pull/4729 @simon-mo


Verification in the form and return of "success" when submitting the form

Is your feature request related to a problem? Please describe.
It's not a problem. And yes an improvement in the form
Describe the solution you'd like
Simple verification in the name and message field (more than one character) and in the email field.
The message "successfully sent", when submitting the form, could be a "toasts"
Additional context
This update could be for Hacktoberfest

@rayanepimentel r u working??

@Amit366 Hi, sorry for the delay.
I will start now.

Can you please update on the current status for that issue, especially about the integration of email verification.

@MrSingh2000 basically when user clicks on send in the contact form then it should be received in the form of mail


Tests

@DivySrivastava If you have a small amount of time then let's add some tests to make sure things are working such as the database is online etc etc. If you have more time tho work on git-chat

Sure! @dan-online
So, how do plan to add the tests?

Shall we use mocha?

Sure!

Add some tests in this commit 8c00f1ec03c114c7db261ac44b3384b9c92c2af3.
Use npm test to run!


docs.google.com listed at API

Hi,
https://etherscamdb.info/api/blacklist/ lists docs.google.com - would adding docs.google.com to the _data/legit_urls.yaml allow the API to skip? If so I can submit a PR?
Thanks,
Paul

Thanks for the report.
We should get a fix in place to check the whitelist beforehand (so a PR would work in the future) soon - currently at ETHBerlin so in the coming days


Add get comments endpoint

@FawnPug maybe I'll add your tips on testing as an Issue and next week we'll discuss it with @kciesielski and @adamw?
@FawnPug @adamw  @kciesielski because in this PR everything is probably already corrected and this is quite a big PR, so there are often conflicts with the master, so please approve ;)


fix escaped HTML entities in definitions

There are 19 HTML entities in definitions (rdfs:comment) i.e. occurrences of & followed by HTML coding:
grep -c "&" *.ttl|grep -v ':0'
AssessedElement-AP-Voc-RDFS2020.ttl:1
AvailabilitySchedule-AP-Voc-RDFS2020.ttl:1
EquipmentReliability-AP-Voc-RDFS2020.ttl:2
GridDisturbance-AP-Voc-RDFS2020.ttl:1
Header-AP-Voc-RDFS2020.ttl:3
PowerSchedule-AP-Voc-RDFS2020.ttl:1
RemedialAction-AP-Voc-RDFS2020.ttl:1
RemedialActionSchedule-AP-Voc-RDFS2020.ttl:1
SecurityAnalysisResult-AP-Voc-RDFS2020.ttl:1
StateInstructionSchedule-AP-Voc-RDFS2020.ttl:1
SteadyStateHypothesisSchedule-AP-Voc-RDFS2020.ttl:5
SteadyStateInstruction-AP-Voc-RDFS2020.ttl:1

A few examples:
AssessedElement-AP-Voc-RDFS2020.ttl

electrical power (I²R or VIcos(phi))

Header-AP-Voc-RDFS2020.ttl:

eumd:Model.applicationSoftware e.g. <tool_name>-<major_version>.<minor_version>.<patch>,
md:Model.created: e.g. <md:Model.created>2014-05-15T17:48:31.474Z</md:Model.created>.
md:Model.scenarioTime: e.g. <md:Model.scenarioTime>2030-01-15T17:00:00.000Z</md:Model.scenarioTime>.

The problem is that HTML is not allowed (not interpreted) in RDF strings.
You could use the ^^rdf:HTML datatype, but that's more complex and unnecessary (and no guarantee that tools will interpret it in rdfs:comment)
Tasks:

[ ] Replace Unicode escapes with the real Unicode char (RDF/XML and Turtle allow UTF8 chars)
[ ] Remove <...> or replace with <...>

[ ] Replace the first example with <tool_name>-<major_version>.<minor_version>.<patch>
[ ] <md:Model.created>...</md:Model.created> is syntax specific to RDF/XML, and you don't need to repeat the prop name in the comment: so remove it

I would like to avoid the need for using   ^^rdf:HTML . The NC profile might have been fixed in the latest update, but it can be a problem for CGMES.

This query finds 2776 instances of the problem (I guess some are duplicated between 2.3 and 3.0 CIM namespaces):
select * {
    ?x ?p ?label
    filter(regex(?label,"[&<][^ =]|\\\\"))
}

It includes:

false hits like e.g. <tool_name>-<major_version>.<minor_version>.<patch> (these are not HTML tags, but "meta-variables")
benign entities like ² (GraphDB workbench displays it as the unicode char ² but maybe that's a misfeature)
HTML block markup like \n
\n\t
 .... This is nok: markdown is ok (\n- ...)
HTML inline markup like field voltage (Efd). This is nok: markdown is ok (*Efd*)
useless HTML markup like ...

The reason I say Markdown is ok is because people can read it more or less ok, when newlines are rendered as newlines.
owl-cli does that: it uses """ and uses real newlines not \n.
This is a large data cleaning task because all occurrences need to be analyzed, then fixing patterns should be defined.


merging in experimental version

Codecov Report

Merging #468 into master will decrease coverage by 39.82%.
The diff coverage is 43.12%.

@@             Coverage Diff             @@
##           master     #468       +/-   ##
===========================================
- Coverage   85.76%   45.93%   -39.83%
===========================================
  Files          30       54       +24
  Lines        3315     3677      +362
===========================================
- Hits         2843     1689     -1154
- Misses        472     1988     +1516

Impacted Files
Coverage Δ

setup.py
0% <ø> (ø)
:arrow_up:

vent/menus/main.py
10.1% <10.1%> (ø)

vent/core/rmq-es-connector/rmq_es_connector.py
100% <100%> (+10%)
:arrow_up:

vent/core/file-drop/test_file_drop.py
100% <100%> (ø)
:arrow_up:

tests/test_api_plugins.py
100% <100%> (ø)

vent/core/file-drop/file_drop.py
100% <100%> (ø)
:arrow_up:

tests/test_menu.py
100% <100%> (ø)

vent/core/rq-worker/test_rq_worker.py
100% <100%> (ø)
:arrow_up:

tests/test_api_templates.py
100% <100%> (ø)

...ent/core/rmq-es-connector/test_rmq_es_connector.py
100% <100%> (ø)
:arrow_up:

... and 84 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 fb36f7e...4501d6d. Read the comment docs.


I cannot work how to complete pull request!

@metazool has added a post but I can't work out how to publish it or even where it is. I obviously set this repo to have protections over PRs but am not sure how to change these so that I can actually complete PR. Previously, I have overcome this by pulling locally and merging, but when I pull locally, @metazool's post is not in the repo on any branch. I could just copy and paste it, but am using this repo to try to learn about more than pull, push, merge!

It seems I did it... :-o


Exception handling and GC Info

Our EH support is currently non-functional

Unwind info for methods is known to be not generated properly
When a method has multiple funclets (ie, a finally block), crossgen emits each as a separate runtime function  whwereas we're putting such a collection in a single method object.
Any explicit reference to System.Exception in R2R code crashes the compiler due to the inability to encode an ExternalTypeNode for it.
GC Info is incomplete or outright wrong

Fixed by https://github.com/dotnet/corert/pull/6321


Object Dectection Notebook: The operation, 'image_tensor', does not exist in the graph.

I followed your whole tutorial and it worked until I tried to reference my frozenmodel in the notebook. In the last step the output is: ""The name 'image_tensor:0' refers to a Tensor which does not exist. The operation, 'image_tensor', does not exist in the graph.""
Can you help?

Hello @hemaAI, did you found a solution? I have also the same issue

I think it was another tensor name. You have to print out your tensor names.
In my case it was something like input_tensor instead of image tensor.

When i displayed all tensors names, i found one with "input" as name but i used it i got this error:
Cannot feed value of shape (1, 576, 768, 3) for Tensor 'input:0', which has shape '(?, 299, 299, 3)'

thats exactly where I ended up and did not find a solution...

Sorry for not replying. It seems like there are multiple things that can trigger this issue.
For some people replacing:
image_tensor = tf.get_default_graph().get_tensor_by_name('image_tensor:0')
with:
image_tensor = 'image_tensor:0'
solved the problem.
I will close this issue for now. If anyone experiences any further issues feel free to reopen it.


how to run the stream Chinese demo?

./bin/sherpa-ncnn
Usage:
./bin/sherpa-ncnn
/path/to/tokens.txt
/path/to/encoder.ncnn.param
/path/to/encoder.ncnn.bin
/path/to/decoder.ncnn.param
/path/to/decoder.ncnn.bin
/path/to/joiner.ncnn.param
/path/to/joiner.ncnn.bin
/path/to/foo.wav [num_threads] [decode_method, can be greedy_search/modified_beam_search]
I didn't know where to download param and bin?
also the token.txt? Does this can be packaged in a zip and included in github release page?

ok, I downloaded the model, but don't how to run in stream? I want it listen to my microphones of my mac

Have you found the solution from the doc?

you can either use the binary
sherpa-ncnn-microphone
or the Python API.


menu.init()

Greetings,
I would like to initialise the backbutton and mainMenuButton text outside the module where I call menu.init().
Is there method/function to set these attributes instead of passing an object to the init() method ?
Thank you.

Im not sure I understand what you are trying to accomplish.
Currently you can do everything inside menus and also export the menus to be included somewhere else. As submenu or via bot.use(menu.init(…)).
You can either pass the backButtonText in via function and define the function in another module. Or you can export the menu and do the bot.use(menu.init(…)) step in the other module.
Does that already help? If not do you have a small example of what you want to do?

Hi,
Thank you for replying.
Well I eventually created a function (getBckBtnCfg) to return the object that contains the data object required by the init(...) function in order to isolate this data in the menu module.
I guess it would have been cleaner to set this data using a function like  simpleButton in order to call init() without parameters.
But this is no big deal, the library is great and that's what matters. :-)

The downside of having a function like simpleButton… it is callable multiple times and only one is relevant. Also its only relevant for the init.
Maybe helpful in your case: you can supply functions to the backButtonText and mainMenuButton like this:
function somewhereElseDefined(ctx) { return ctx.from.first_name }

bot.use(menu.init({
  backButtonText: ctx => ctx.from.first_name,
  mainMenuButtonText: somewhereElseDefined
}))


memory allocation failed

Hello,
Thank you for your good documented package. I try to run it on a ESP32 with only two other packages, but I get a memory issue for importing HttpResponse
The complete message seems like this:
Traceback (most recent call last):
  File "main.py", line 11, in <module>
  File "/lib/MicroWebSrv2/__init__.py", line 18, in <module>
  File "/lib/MicroWebSrv2/microWebSrv2.py", line 8, in <module>
  File "/lib/MicroWebSrv2/httpRequest.py", line 8, in <module>
MemoryError: memory allocation failed, allocating 136 bytes

Have you an idea? It's every run 136 bytes
In another post I have red, dividing the file in to is possible, but I don't know, how I can divide a class...
Thank you very much!

Hello @GitHubForTester,
Ok, I think that you have an ESP32 without extended SPIRAM.
MicroWebSrv2 code is too large for import (and eval) it on the fly.
You can use an ESP32 with 4MB of SPIRAM or you can also embed it as a micropython library before compiling the firmware yourself (the module will be already compiled in your ESP32 and the memory usage will be less).

Hello @jczic,
thank you for your fast response.
This can be possible. Have you a link with a instruction to embed it as a micropython library? I have no idea, how it's possible.
Actually I have no ESP32 with 4MB SPIRAM, at that term I must pay attention while order? Simple 4MB SPIRAM?
Thank you very much!

Ok thank you, I think it's a little bit overkill for my application, but thank you very much!

If anyone wants to compile custom Micropython firmware without messing their already installed and working ESP-IDF build environment, I made a docker container with build tools for the ESP32.
https://hub.docker.com/r/tionebrr/micropython-tools-esp32


Update site manifest to make start_url and icons relative paths

This allows the PWA install icon to be seen in the long-lived offline-mode branch

Example url to see the install badge show:
https://activity-player-offline.concord.org/branch/update-pwa-manifest/index.html


CIM - skip empty YAML files while creating MD files

LINK: https://github.com/hunters-forge/OSSEM/blob/master/common_information_model/entities/meta_dst_host_name.md
We can keep a list of entities that we think are useful and have community feedback

We need to remove empty entities from the MD tables:
https://github.com/hunters-forge/OSSEM/tree/master/common_information_model

fixed by https://github.com/hunters-forge/OSSEM/pull/86


Assign FHIR and DICOM roles automatically

Resolves #3103
What is being addressed
Assign everyone with Workspace Researcher role to Fhir Contributor/Dicom Data Owner role upon health services creation.
How is this addressed

Get all entities assigned with Workspace Researcher role on the workspace client id
Assign them with Fhir Contributor/Dicom Data Owner role  accordingly

/test-force-approve

/test

/test


idf.py menuconfig always ask confirm saving even no changing (IDFGH-2148)

I just type idf.py menuconfig and then exit without any changes, but is still ask confirm saving.
This behavior is not same as in 3.x or most other project using menuconfig

@keelung-yang Thanks for reporting.

This issue existed in new menuconfig mconf-v<IP_ADDRESS>-idf-20190628-win32


3060M  12G ok

https://github.com/user-attachments/assets/9945a044-349b-4306-9990-d9fe7b0397db

Good job! And I can verify that 4070 Laptop with only 8GB VRAM is also OK.
At least I set the same resolution as you did...


exclude function is not working after past midnight?

I tried to exclude hours after midnight
example
$hours = HoursHelper::create('18:00', '02:00', 60, 'H:i', [
            ['21:00', '23:00'],
            ['01:00', '02:00'],
        ]);

// output
  0 => "18:00"
  1 => "19:00"
  2 => "20:00"
  3 => "00:00"
  4 => "01:00"
  5 => "02:00"

the interval between 21:00 to23:00 is excluded but not for 01:00 to 02:00 is still exists

Thanks for reporting the issue


Added timeout parameter to wait_for_event in stick.py

wait_for_event method can set a timeout after which it will return, by taking an additional default parameter called "timeout".
Why is this useful:

To prevent a thread from being blocked on user input, even when there is nothing more to do. This way the thread can check if it must terminate.

@waveform80 wasn't there a reason we left this out?

I opened the pull request because I noticed that since wait() method has been designed to return false if the timeout expires, giving it the possibility to actually return false seems legit to me. Moreover, adding a default parameter does not invalidate everyone's code. But I'm just a student willing to learn from my mistakes, so..
Hope this can help.

Ah, I'm afraid the reason this isn't included is because the implementation is not that simple. You've not considered the case where _read returns None because the event in the queue was a non-key event. Walking through your implementation:

We enter the top of the while loop and wait timeout seconds for an event to arrive.
A non-key event arrives which _read() returns.
We return to the top of the loop and again wait timeout seconds for an event to arrive. Note at this point we've potentially waited longer than timeout seconds, and that this can go on for as long as non-key events arrive.

It is possible to add a timeout to this method, but it's non-trivial. Also, as you've pointed out the major use-case for this is to permit threads to avoid blocking indefinitely for user input. However, my thinking was: if you want to use threads, why not use the callback mechanism provided in which case you get the background thread for free? Given the availability of the callback mechanism, and that this was the only major use-case I could come up with for wait-with-timeout, I decided to keep the code simple and exclude it in my original PR.

You are right, I did not put enough attention to that details.
Thanks for your suggestion!


Added Sniff for the suppress_filters in get_posts() args.

In WordPress-VIP cs, I was getting the warning when using the get_posts() function which saying to include suppress_filters=>false in get_posts() as args.
Problem: This warning is not disappearing even when I am passing the suppress_filter=>false to it.
As mentioned in to do comment inside the https://github.com/WordPress-Coding-Standards/WordPress-Coding-Standards/blob/develop/WordPress/Sniffs/VIP/RestrictedFunctionsSniff.php#L160 for the get_posts() function to make the sniff to check if suppress_filter=>false is passed or not.
So here, I have tried to solve this issue, It is still incomplete, tweaks mentioned below can complete this PR.

If there is no suppress_filters argument is passed or suppress_filter is passed but the value is not false then show the default error which is showing currently.

get_posts() is discouraged in favor of creating a new WP_Query() so that Advanced Post Cache will cache the query unless you explicitly supply suppress_filters => false.';

Please guide or correct if I am wrong anywhere.
Thanks
Fixed #660

FYI: While someone might review your sniff, it will not be merged for a little while yet while we are working on making WPCS compatible with PHPCS 3.x.
Once that project has finished, your PR will need to be updated to be in line with the changes needed for PHPCS 3.x.
Ref: https://github.com/WordPress-Coding-Standards/WordPress-Coding-Standards/issues/1040

@jrfnl Please do let me know whenever any changes are required on this PR.

@jrfnl I have made changes according to your above comments and pushed the commits along with the test unit files. Please check and do let me know if any changes are required. :+1:

@jrfnl I have started working on these changes.

@jrfnl I have updated this PR with the current changes made in develop, is it fine or should I have to revert it back ? :(

@emgk Govind, please have a look at the unit tests which are failing: https://travis-ci.org/WordPress-Coding-Standards/WordPress-Coding-Standards/builds/258087412

@jrfnl I am working on this.

@jrfnl could you please check ?

Now I've finished work on #1208, I'm finally getting back to this sniff. Sorry for the delay @emgk
To start with: could you please rebase the PR on the current develop ? A number of new sniffs have been added to WPCS since you originally pulled yours and your sniff as it currently is does not comply with some of the new rules so that needs to be fixed.
  65 | WARNING | [ ] This comment is 49% valid code; is this commented out code?
     |         |     (Squiz.PHP.CommentedOutCode.Found)
  94 | ERROR   | [x] Expected 1 space after comma in function call; 2 found
     |         |     (Generic.Functions.FunctionCallArgumentSpacing.TooMuchSpaceAfterComma)
  96 | ERROR   | [x] No space found after comma in function call
     |         |     (Generic.Functions.FunctionCallArgumentSpacing.NoSpaceAfterComma)
 104 | ERROR   | [x] No space found after comma in function call
     |         |     (Generic.Functions.FunctionCallArgumentSpacing.NoSpaceAfterComma)
 104 | ERROR   | [x] No space found after comma in function call
     |         |     (Generic.Functions.FunctionCallArgumentSpacing.NoSpaceAfterComma)
 107 | WARNING | [x] Equals sign not aligned with surrounding assignments; expected 9 spaces but found 12
     |         |     spaces (Generic.Formatting.MultipleStatementAlignment.NotSameWarning)
 108 | WARNING | [x] Equals sign not aligned with surrounding assignments; expected 4 spaces but found 7
     |         |     spaces (Generic.Formatting.MultipleStatementAlignment.NotSameWarning)
 109 | WARNING | [x] Equals sign not aligned with surrounding assignments; expected 6 spaces but found 9
     |         |     spaces (Generic.Formatting.MultipleStatementAlignment.NotSameWarning)
 110 | WARNING | [x] Equals sign not aligned with surrounding assignments; expected 9 spaces but found 12
     |         |     spaces (Generic.Formatting.MultipleStatementAlignment.NotSameWarning)
 111 | WARNING | [x] Equals sign not aligned with surrounding assignments; expected 9 spaces but found 12
     |         |     spaces (Generic.Formatting.MultipleStatementAlignment.NotSameWarning)
 163 | ERROR   | [x] No space found after comma in function call
     |         |     (Generic.Functions.FunctionCallArgumentSpacing.NoSpaceAfterComma)
 165 | WARNING | [x] Equals sign not aligned with surrounding assignments; expected 5 spaces but found 1
     |         |     space (Generic.Formatting.MultipleStatementAlignment.NotSameWarning)

I was also wondering if you could replace the @link in the class documentation with one specifically pointing to a section in the VIP handbook which supports the need for this sniff.
On the page the link points to now, I couldn't find any information about suppress_filter=>false is not allowed.

Hey @jrfnl, Sorry for the delayed response. yes I am going to work on this PR on this weekend.

@emgk No worries. I'll look forward to the new version.

@emgk Hi Govind, just checking in again to see how work is progressing. If you haven't got time to finish this, please let us know and we'll see if we can find someone to take over.

Hi @jrfnl Sorry for delay, I have just worked on the changes you requested for. The PR is almost completed  just need your help regarding below things:
In below syntax, what is the way to check if the array in variable contains the value I want to check ?
$args4 = array();
$args4['suppress_filters'] = false;

get_posts( $args4 );

@emgk The WordPress-VIP ruleset in WPCS is going to be deprecated in favour of using the Automattic VIP Coding Standards standard.
See #1309, #1409
So, I'm now wondering if this sniff should still go into WPCS or if it should go into the Automattic VIP Coding Standards ?
@tomjn I think @emgk has done great work here, though it does need some final touches. Is this already covered in the Automattic VIP Coding Standards sniffs ? In your opinion, should this sniff maybe be pulled there instead ?

IMO the suppress filters sniff is of value to all not just VIP, it'd be great if this could be included in WPCS under one of the other rulesets rather than the VIP ruleset

IMO the suppress filters sniff is of value to all not just VIP, it'd be great if this could be included in WPCS under one of the other rulesets rather than the VIP ruleset

@tomjn Could you please explain this ? Maybe in a new issue in which this can be further discussed ?

Thank you @emgk for working on this and I'm sorry that it took us so long to get back to it.
I believe the discussion points in the underlying ticket #1416 (still) need to be resolved first before this PR can be considered for merging.
Additionally the PR would need an update as the code base has undergone quite some changes in the mean time.
So for now, I'd like to suggest we close this PR with the option to re-open & rebase in the future.
Thank you for your willingness to contribute to WordPressCS. I hope future PRs go more smoothly and I look forward to your continued contribution.


Age Difference Enforcement: Temporary Fix

User Story:
As a child entering foster care, I need to be cared for by an Office Mom or Dad who is at least [TBD] years older than I am so that I feel comfortable and so that OMD avoids risk.
Dependencies:

Full fix outlined in #178. This fix should be undone once #178 is complete.

Acceptance Criteria:
Temporary Fix: Please add this text to this profile field:
Current Text: Select one or more age ranges that you would like to work with

[ ] New Text: Select one or more age ranges that you would like to work with. Do not select "13-17" unless you are at least [TBD] years old."

@benjaminwood, added dev effort per your comment above, adjust as necessary

@sarahwicks Have you decided how much older the person caring for a child between the age of 13-17 needs to be yet?


Schematize the PyPI vulnerability API

Related to #72: the PyPI vulnerability API, which is part of Warehouse's codebase, is not currently schematized.
Schematizing it would increase our confidence in pip-audit's reliability, and would us to write more type/construction friendly interactions with the API, e.g. with Pydantic.

CC @cooperlees

Just keeping the links up to date here:
PEP: https://peps.python.org/pep-0691/
Discussion: https://discuss.python.org/t/pep-691-json-based-simple-api-for-python-package-indexes/15553


Process.h does not compile under windows

It's linux specific, the ScatterVisual seem to depend on it. -Will there be a windows port of Process.h ?

N/A


bug fix for model service

Updating how we define metricSources list. List<? extends MetricSource> metricSources results in an injection error. We will have to cast it wherever it's required.

Not required anymore. https://github.com/ExpediaDotCom/adaptive-alerting/pull/481 took care of this.


task escape token with a slash adding a back slash

Question, Bug, or Feature?
Type: bug
Enter Task Name: qetza.replacetokens.replacetokens-task.replacetokens@3
Environment

Server - Azure Pipelines

Agent - Private:

microsoft/dotnet:2.2-sdk
agent version 2.155.1

Issue Description
token with a slash: *x;/?)]%$U=;
when replaces the variable in the file, azure escape the slash adding a back slash
on the file the result is: *x;\/?)]%$U=;

didnt know about the escapeType option, i added:
escapeType: none


Incorrect test in longest-substr-2-uniques

12: Test command: /home/travis/build/dubzzz/various-algorithms/build/test/longest-substr-2-uniques
12: Test timeout computed to be: 9.99988e+06
12: [==========] Running 5 tests from 1 test case.
12: [----------] Global test environment set-up.
12: [----------] 5 tests from longestsubstr2uniques
12: [ RUN      ] longestsubstr2uniques.EmptyString
12: [       OK ] longestsubstr2uniques.EmptyString (1 ms)
12: [ RUN      ] longestsubstr2uniques.UniqueCharacter
12: [       OK ] longestsubstr2uniques.UniqueCharacter (0 ms)
12: [ RUN      ] longestsubstr2uniques.TwoCharacters
12: [       OK ] longestsubstr2uniques.TwoCharacters (0 ms)
12: [ RUN      ] longestsubstr2uniques.SubstringOfMainString
12: [       OK ] longestsubstr2uniques.SubstringOfMainString (0 ms)
12: [ RUN      ] longestsubstr2uniques.RandomData
12: Using configuration: seed=3920587640228839387
12: /home/travis/build/dubzzz/various-algorithms/rapidcheck/extras/gtest/include/rapidcheck/gtest.h:29: Failure
12: Failed
12: Falsifiable after 31 tests and 2 shrinks
12:
12: std::tuple<char, char>:
12: (1, 0)
12:
12: longest part with 2 different chars:
12: "\0\x1\x1\x1\x1\x1"
12:
12: length for the part before longest:
12: 0
12:
12: before longest chars:
12: ""
12:
12: length of the part after longest:
12: 2
12:
12: after longest chars:
12: "\xFF\x1"
12:
12: /home/travis/build/dubzzz/various-algorithms/algorithms/contiguous-data/longest-substr-2-uniques/tests.hpp:78:
12: RC_ASSERT(answer_str.size() == longest_size(data))
12:
12: Expands to:
12: 6 == 7
12:
12: [  FAILED  ] longestsubstr2uniques.RandomData (17 ms)

Should be fixed with fffc2dd89db517b3f30b4b48bae5c2791853c76e


Egress through proxy - example in documentation

Is your feature request related to a problem? Please describe.
Istio allows to define external services e.g. http://www.google.pl/
In some environments access to public internet is not available directly, only through HTTP proxy.
Istio has no entry in documentation that shows how to configure it for such use case.
According to attached issue it should be possible, but I can't find e2e test that covers this use case.
Describe the solution you'd like
There should be example that shows how to configure egress traffic through HTTP proxy to external services.
Describe alternatives you've considered
Additional context
https://github.com/istio/istio/issues/4969

We have the same problem.
I can't find any documentation on this.
@rshriram You referenced to a test case and an implementation. Can you point out where I can find these?

One of the solutions is here https://github.com/istio/istio/issues/7302 for those who are interested.

@xaka Perfect, this works.
Maybe the doc should be updated to cover this case.


Set port environment variable by default

Many Dockerfiles will follow this pattern:
ARG PORT=8080
ENV PORT=$PORT
EXPOSE $PORT

Setting this in the generated manifest will increase the likelihood that the deployed container will be healthy without any changes to the manifest:
variables:                    # Pass environment variables as key value pairs.
  PORT: 80

To make sure that I understand the ask correctly, this would mean that we:

Parse the PORT from the manifest file.
Pass it as a value to the --build-arg flag while building the container image.

So that PORT argument in the Dockerfile is replaced with whatever value we set in the manifest.

No just setting the env variable in the task definition is sufficient

Oh hmmm I need to test that, I dont know if task def port mappings override the build time exposed port


last yii2-bootstrap (bootstrap 4.1.1)

I've upgraded yii2-bootstrap and all layout be corrupted... maybe last bootstrap version (4.1.1) ?
Thanks

Hi paskuale75, unfortunately I wont update my package to support new boostratp version. Tested version is 3.3.7.
Feel free to fork an update.

Ok, unfortunately, they have updated the yii2 framework and together with the bootstrap version, thanks too


fix(pdoc): install dependencies before generating pydoc

:tada: This PR is included in version 0.0.14 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Errors on iOS Safari

Hey, the latest iOS Safari does not work with the latest web-components (0.12.3) version. The -lite version works fine, but due to this issue I need to use the full version.
The line in question is:

webcomponents.js - 235
webcomponents.min.js - 170

The code in question is inside installProperty, specifically this line:
if (typeof descriptor.value === "function") {
  if (allowMethod) {
    target[name] = getMethod(name);
  }
  continue;
}

(And apparently descriptor is undefined)
webcomponents.js Error: TypeError: undefined is not an object (evaluating 'descriptor.value')
webcomponents.min.js Error: TypeError: undefined is not an object (evaluating 'd.value')

Additional information: the descriptor comes from var descriptor = getDescriptor(source, name); where name is showModalDialog and source is Window. The installProperty has been invoked with EventTarget as a target.

Ok, so the problem seems to be this:
Safari on iOS includes 'showModalDialog' in the list the you get when you invoke Object.getOwnPropertyNames(window), but returns undefined when you then invoke Object.getOwnPorpertyDescriptor(window, 'showModalDialog') (which seems to be a bug IMO.
Adding this line above works, but I am afraid that this is not the intended behaviour...
if (descriptor == null) continue;

I have alos found this thread: https://www.bountysource.com/issues/37861877-throws-on-ios-10
In that case, they return a dummyDescriptor. Not sure which one of them is better suited.

I just saw that this was fixed by https://github.com/webcomponents/webcomponentsjs/commit/e6a0d18cd338424467eab9622ab064f0d8cb1f54
And has been released 12 days ago as v0.7.23.
Any chance we could upgrade soon?


[HOLD] Support new API formats in Network.jsx

If API uses a new format /api/command? then this will properly strip out the command and add it to the URL.
Fixed Issues
For https://github.com/Expensify/Web-Expensify/pull/41231
Tests

What unit/integration tests cover your change? What autoQA tests cover your change?
What tests did you perform that validates your changed worked?

QA

What does QA need to do to validate your changes?
What areas to they need to test for regressions?

Removing puller since this is still WIP

I'll reassign you @MariaHCD once this is done so you can get the review credit 😄

This works with https://github.com/Expensify/App/pull/38083!

Works with https://github.com/Expensify/Web-Expensify/pull/41231 with localJS!


Acceptance tests cannot be run with scriptcs 0.14.1

What is the current state of the acceptance tests? I am unable to get them to run successfully.

The path to the output folder as defined in Bau.Test.Acceptance/Support/Baufile.cs line 33-39 does not exist. To workaround this I replaced the entire #if DEBUG with Path.GetDirectoryName(Assembly.GetExecutingAssembly().Location)`.
In the Run method in the same file, the FileName to run is set to "scriptcs". Windows complains that it can't find this file, since it is actually called "scriptcs.bat". Changing the FileName to "scriptcs.bat" allowed me to continue.
Now, for all tests, scriptcs always returns an exit code of -1, causing the tests to fail, even though running scriptcs from the command line seems to succeed.

Any ideas? Is anyone running the acceptance tests on a regular basis?

I ran them last night. I just run "build accept" from the repository root. They all worked me using the command line, and take forever. Running the from ncrunch most of them fail, even after setting up certain folders.

Also, I had to upgrade scriptcs, I am at the latest version: "cup scriptcs"

I am still unable to run them, nor do I see how they could run in their current state given the problems I noted in my issue description. Here's what I have:
Windows 7 64-bit
Dev branch (5c23bd63209df43b013615c61e7e24112ee1d92b)
ScriptCS 0.14.1 (latest version)
cmd (also tried Visual Studio 2013 Developer Command Prompt)
build accept
This results in the problems I described. Any idea why this doesn't work for me?

For me these folders map to src\Bau\bin\Release for release builds and a similar folder for Bau.Exec. Also the build based on baufile.csx appears to be hard coded to Release.
I'm not sure how but it appears that the tests are running for me through NCrunch using a leftover ScriptCs.0.13.2 I have instead of the ScriptCs.0.14.1 it should be using. I'm going to look into that.
I'm seeing some of the non-zero codes in my VS runners, possibly related to the version of scriptcs it is using.

Try this, I'm curious if it changes anything for you.

Nuke all bin directories under all projects
Nuke the artifacts folder
Run build accept

I fixed my scriptcs install so it is now using 0.14.0 and as a result I get 2 failures when running build accept and 5 when running them through NCrunch for that commit. Very strange! I guess one thing to do is make sure you actually have 0.14.1 used when you run scriptcs as there was a change in how it was installed a few months back. It used to be installed to the user folder for only one user per system which was odd, they changed it to be a system wide install. I don't think the scriptcs upgrade worked for me correctly so I had to take care of some leftovers manually.

A second run of build accept passed all 288 tests. I think they need some love!

Deleting all "bin" directories and the "artifacts" directory did not help. Nor does #199.
(1) does not seem to be a problem any more (not sure what happened, will investigate this further), but I am still stuck at (2) (unless I change the FileName from "scriptcs" to "scriptcs.bat"). Are you building on Windows 8? Or are there any other differences from my setup above that could be causing different behavior here?

@eatdrinksleepcode I guess it's scriptcs.bat for you because you installed scriptcs via svm?
Even so, things should work in the general case. I'll take a look at @aarondandy's PR.
It's highly possible that something was broken here by scriptcs 0.14, which contained breaking changes.

bin was changed to scriptcs_bin causing it to only copy the bau.dll and bau.exec.dll files once, ignoring changes to them. The PR fixes that for sure. I am not using svm and may keep it that way so I can be a test case for chocolatey. The only differences with my setup are that I use cinst and Windows 8.1 .

@eatdrinksleepcode this should all be fixed now.
As long as you have scriptcs 0.14.1 installed either via svm or chocolatey, the acceptance tests should now work for you. I'm not really sure where you got scriptcs.bat from. Are you perhaps using a very old version of chocolatey?

BTW - feel free to re-open this issue if you are still having problems.

I have scriptcs installed via Chocolatey. My Chocolatey version is probably pretty old, I will try a newer version later today.

On Apr 25, 2015, at 12:46 PM, Adam Ralph<EMAIL_ADDRESS>wrote:
BTW - feel free to re-open this issue if you are still having problems.
—
Reply to this email directly or view it on GitHub.

It was the old version of Chocolatey. Updated and then re-installed scriptcs and the acceptance tests now all pass (although at one point I got a Watson error saying Bau.Test.Acceptance.CreateFile stopped working; but that didn't seem to cause a test failure).


New mode request 'Out'

I could not find a corresponding mode setting, so I propose adding it.
Mode setting:
setting_name = Out
setting_synonym = out, gone out, out mode, out setting
Represents user being not present for a short while, ergo not having gone on vacation but merely left the house to f.ex go to the store.
Thanks in advance!

Wouldn't that be the same as an Away mode?

The way I understand it, Away represents going long periods away like a vacation/holiday (usually for many days or weeks).
Out would represent going away for less than a day like going the store or whatever.
The distinction would be especially useful for thermostats who would have no reason to be heating the building if the user is Away for many days, but if the user is merely at the store for a few hours they would likely want their building heated when they return.

This has been submitted, and should roll out in a few days.


[HOTFIX][ENG-2924] Don't load keen in didInsertElement

Ticket: []
Feature flag: n/a

Purpose

Summary of Changes

Side Effects

QA Notes

It doesn't make sense why didUpdateAttrs is called on initial render (see https://guides.emberjs.com/v3.4.0/components/the-component-lifecycle), how did you come to this solution?

When tested locally, didUpdateAttrs is indeed called on initial render. There are no other ways for the loadKeen task to be called twice.


Ah, sorry.
/* Current */
"http://example.com/foo/%27bar.jpg"

/* Expected */
"http://example.com/foo%27bar.jpg"

I think the backslash before the ' isn't necessary (but encoding the character should be). However I dug a little further and it looks like this is coming from URL.parse:
Url {
  protocol: 'http:',
  slashes: true,
  auth: null,
  host: 'example.com',
  port: null,
  hostname: 'example.com',
  hash: null,
  search: null,
  query: null,
  pathname: '/foo/%27bar.jpg',
  path: '/foo/%27bar.jpg',
  href: 'http://example.com/foo/%27bar.jpg' }

Note that this test passes:
test('should handle single quotes', t => {
	t.is(m('http://example.com/foo\'bar.jpg'), 'http://example.com/foo%27bar.jpg');
});

But this does not:
test('should handle escaped single quotes', t => {
	t.is(m('http://example.com/foo\\\'bar.jpg'), 'http://example.com/foo%27bar.jpg');
});

I'm not sure if accounting for what wrapping quotes a URL string uses is within the scope of this module but it would be important for my use case! 😄

Sorry to bother you with this Sindre, I'm going to fix this in cssnano instead by changing the wrapping quotes when possible. I don't think it should be this module's job to do.


Uncaught SyntaxError: Unexpected token ;

In docs.js line 165.
Stops execution of js and seems to break hover tooltips.

@LHSnow, thanks for the bug report. Fix on the way.


fix: Pasting MsoListParagraph from MS Word throws an error

Please check if the PR fulfills these requirements

[x] It's the right issue type on the title
[ ] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[ ] The commit message follows our guidelines
[ ] Tests for the changes have been added (for bug fixes/features)
[ ] Docs have been added/updated (for bug fixes/features)
[x] It does not introduce a breaking change or has a description of the breaking change

Description
My users tried to copy text from a rather convoluted Word document. The relevant HTML markup looks like this:

Some
text ABC<o:p></o:p>

More text<o:p></o:p>

The problem here is that p.MsoListParagraph does not have the mso-list CSS property that is expected by createListItemDataFromParagraph:
https://github.com/nhn/tui.editor/blob/e5fd1affef9aa6b5bcd9eff751d8b218a9dfb763/apps/editor/src/js/utils/wwPasteMsoList.js#L51-L53
The solution I came up with is to just not treat such elements as list elements. That avoids the error, but I'm not sure it is a proper solution.
This issue also seems to affect v3.x but I can't reproduce it because text copied from that Word document is inserted as a Base64-encoded image instead.

@paddya
I'm so sorry I haven't answered for a long time 🙏. I was busy with 3.x bug fixes and other projects. I will review this PR within this week. Thank you!!


reading and writing from/to js strings (or Uint8Array)

At the moment we only read and write files. It would also be interesting to read and write from js values so we can do things like:

Write a grafana dashboard into a string, use that string to forge a ConfigMap object
Read the output of something else than a file, say something like std.exec (#147) and parse it like read does.

Both read and write have a parsing/generating capability that could not only work with file but with any buffer in memory. This issue is about extracting that capability as a separate function. I think read/write should still be kept the current way, for convenience.

I'm thinking of introducing a Value object that could work like this:
import * as std from '@jkcfg/std';

const o = {
  b: 2,
  a: 1,
};

std.log(std.Value(o).asString({format: std.Format.YAML}));

We now have std.parse and std.stringify that provide this functionality.


Cursor control on Samsung One UI keyboard should NOT work when selection length > 0

gif: https://github.com/flutter/flutter/issues/61954#issuecomment-775742834
This problem is only happen on Samsung Keyboard
Reproducible step

'Setting' of phone -> touch 'serach icon' on right top -> search 'Keyboard swipe controls' -> select 'Cursor control'
Make TextField. minimal code is below
Type some text and select text entirely with double tap -> touch any key on keyboard
expected: all text is deleted and insert a new character
actual: append a new character to original text

import 'package:flutter/material.dart';

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      home: Scaffold(
        body: SafeArea(
          child: TextField()
        )
      ),
    );
  }
}

Cursor control is comfortable function that can move cursor of TextField by dragging on broad keyboard layout (not on small TextField layout).
However it should NOT work when selection length > 0 like Android native

closing as a duplicate of https://github.com/flutter/flutter/issues/75776

@TahaTesser why closed? https://github.com/flutter/flutter/issues/61954 has not enough information than this


Sentinel Detection Analytics

Describe the bug
There are many different detection methods that are generating a lot of alerts, even though they are being blocked by our firewall. Are these alerts supposed to be changed or just a "example" so we can build out own queries off them?
Example query is: Senntinel/Detections/CommonSecurityLog/HighConfTIDetection.txt
We can write new queries and change them, but just wanted to confirm the "best use case" of these queries included on GitHub?
Thank you
@shainw

The Detections are meant to be used as is, but in some cases may need some tuning for a given environment.  The HighConfTIDetection is looking at connections that contain a MaliciousIP which is identified by our internal TI feed and automatically joined to the Firewall data as context.  While our TI team tries to keep this from getting stale or incorrect data, it is possible the TI is incorrect. In the case of the HighConfTIDetection, it looks like we missed making sure it was a connection versus a blocked connection.  If you have other detections that are firing alerts that are incorrect or inconclusive, can you submit additional Issues for those or potentially use our request feedback form to send directly to my team.  The form is at the bottom of this Blog - https://techcommunity.microsoft.com/t5/Azure-Sentinel/Azure-Sentinel-Blog-Table-of-Contents/ba-p/731727
I am out of the office until Wednesday, but will have someone look at the HighConfTIDetection while I am out.
Thanks,
Shain

@DSharpPro thanks for the feedback, as @shainw said we want to filter out blocked connections. I have made some changes to a couple of rules including HighConfTIDetection.txt, you can see what we have done in the PR (https://github.com/Azure/Azure-Sentinel/pull/214).
Please give this updated query a test in your Sentinel workspace and let us know if this improves the fidelity of this detection. We create these queries with the aim to be as widely applicable as possible but vendor log messages differ so we might not have quite the right syntax for every vendor. Can you let us know what firewall vendor you are using and I can try and make sure that is covered in the scope of this query.
Thanks,
Pete

@DSharpPro - Thanks for the participation in general and if you happen to be at Blackhat our team will be at Blackhat and several of us will be at the Azure Sentinel booth, including Pete who did the fix you mentioned above.  Hope to see you and any of our other Github contributors.  Thanks Shain

Hi @petebryan, @shainw,
We have now updated the queries to reflect the changes you've made. We are running Cisco ASA's.
Also, is it possible to get a list of "malicious IP's" from Microsoft so we can upload to our firewalls so the block list contains both Cisco and Microsoft's data base?
Thanks,

Hey @DSharpPro,
That is not exposed directly that I know of, but I will see if there is a possibility of doing so.
Shain

@DSharpPro - I chatted with our internal TI folks, I was explained that at this time Microsoft does not expose any TI feeds to the public.


This app does not provide any candidate for installation

The title says it.
I'm trying to install glimpse image editor.

use lowercase

lmao it was that simple thanks
I just assumed it needed to be uppercase because it said so in search lol

It uses the case as specified in this URL: https://g.srev.in/get-appimage/glimpse-image-editor/

I see

Where did you get that command from? Can you put the link?

What do you mean?
I searched for glimpse (zap search glimpse) and it said to run this to install

ah, I see. I completely forgot about the search feature I implemented :facepalm: , thanks for providing me the pointer, it is clearly a bug in zap


Using oc-mirror with headsonly fails digest errors referring to article #6138332

Version
# oc-mirror version
Client Version: version.Info{Major:"", Minor:"", GitVersion:"4.12.0-202302280915.p0.g3d51740.assembly.stream-3d51740", GitCommit:"3d517407dcbc46ededd7323c7e8f6d6a45efc649", GitTreeState:"clean", BuildDate:"2023-03-01T00:20:53Z", GoVersion:"go1.19.4", Compiler:"gc", Platform:"linux/amd64"}

What happened?
When mirroring content from registry.redhat.io/redhat/redhat-operator-index:v4.10 with the following imageset-config.yaml (which is based on https://github.com/openshift/oc-mirror/blob/main/docs/examples/imageset-config-catalog-headsonly.yaml):
kind: ImageSetConfiguration
apiVersion: mirror.openshift.io/v1alpha2
storageConfig:
  registry:
    imageURL: myregistry.mydomain:5000/redhat-operators
    skipTLS: false
mirror:
  operators:
  - catalog: registry.redhat.io/redhat/redhat-operator-index:v4.10

then run:
# oc mirror --config=./imageset-config.yaml docker://myregistry.mydomain:5000/redhat-operators
error: unable to retrieve source image registry.redhat.io/openshift-service-mesh/grafana-rhel8 manifest sha256:070d8ffd0a11158a8a5769ef9854109e70d911bb9adcce35449db33037df99d0: unsupported: Not Found, or unsupported. V2 schema 1 manifest digest are no longer supported for image pulls. Use the equivalent schema 2 manifest digest instead. For more information see https://access.redhat.com/articles/6138332

Attaching my  .oc-mirror.log.
What did you expect to happen?
For it to mirror the operator containers to my local registry. It works when a few containers are chosen (such as the serverless-operator)
How to reproduce it (as minimally and precisely as possible)?

oc mirror --config=./imageset-config.yaml docker://myregistry.mydomain:5000/redhat-operators

error: unable to retrieve source image registry.redhat.io/openshift-service-mesh/grafana-rhel8 manifest sha256:070d8ffd0a11158a8a5769ef9854109e70d911bb9adcce35449db33037df99d0: unsupported: Not Found, or unsupported. V2 schema 1 manifest digest are no longer supported for image pulls. Use the equivalent schema 2 manifest digest instead. For more information see https://access.redhat.com/articles/6138332

Full log in log-file.
And then the command exits.
Anything else we need to know?
I haven't tested this for 4.11 and 4.12
References

https://docs.openshift.com/container-platform/4.12/installing/disconnected_install/installing-mirroring-disconnected.html#installing-mirroring-disconnected

oc-mirri-log-20230303.txt
Attached log file.

I ran this on a different host, and got these results (which I've experienced before):
$ cat imageset-config.yml
kind: ImageSetConfiguration
apiVersion: mirror.openshift.io/v1alpha2
storageConfig:
  registry:
    imageURL: otherregistry.mydomain:5000/redhat-operators
    skipTLS: false
mirror:
  operators:
  - catalog: registry.redhat.io/redhat/redhat-operator-index:v4.10

$ oc mirror --config=./imageset-config.yml docker://otherregistry.mydomain:5000/redhat-operators
Checking push permissions for local-registry.pc.therealworld.info:5000
Found: oc-mirror-workspace/src/publish
Found: oc-mirror-workspace/src/v2
Found: oc-mirror-workspace/src/charts
Found: oc-mirror-workspace/src/release-signatures
WARN[0026] DEPRECATION NOTICE:
Sqlite-based catalogs and their related subcommands are deprecated. Support for
them will be removed in a future release. Please migrate your catalog workflows
to the new file-based catalog format.

The rendered catalog is invalid.

Run "oc-mirror list operators --catalog CATALOG-NAME --package PACKAGE-NAME" for more information.

error: error generating diff: channel fast: head "devworkspace-operator.v0.18.1-0.1675929565.p" not reachable from bundle "devworkspace-operator.v0.18.1"


Handle offline firebase actions properly

As a workaround, offline mode warning added until new architecture is realized.


Relicense to Apache 2.0

@Manuel-Materazzo would you be OK if we relicensed to Apache 2.0 (instead of MIT) to make it easier to incorporate code from the Home Assistant frontend project?

Also requested BrianEH approval in: https://community.home-assistant.io/t/map-card-map-change/98618/24

Sure, it's perfectly fine by me

Closing this one, most of the features of the native ha-map-card have been re-implemented from scratch.


pull request for e2e test at 11/10/2016 09:45:09

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

Details

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


status of this project

Hi,
Firstly, congrats for this great work. We have been using it for quite a long time and we haven't had issues. Now, we would like to use it in another project but there are some concerns about the support of this. It looks like there has been no activity in the last months.
Could anyone, please, give us some update on this regard?
Many thanks,
David

Agreed, please confirm that this project is still actively maintained via return comment.


Upgrade for MapProxy 2.0.2 Image on Debian Bookworm

Now the MapProxy Docker image has been upgraded, this image needs upgrading as well. A default install on Debian Bookworm will upgrade from:
debian:buster-slim (v10) - MapProxy 1.13.2 - Mapnik 3.0.22 - Mapnik-Python 0.1
to
debian:bookworm-slim (v12) - Mapproxy 2.0.2 - Mapnik 3.1.0 - Mapnik-Python 3.0.23.

Done!


[SEO] Index Now to run upon AEM/Crosswalk publish

Would it be possible to implement this.  We can discuss.
Index now for Google:
https://developers.google.com/search/apis/indexing-api/v3/quickstart
Index now for Bing:
https://www.bing.com/indexnow
For Bing, an API is called which tells it the page to index and the key, I set this up, here is an example:
https://www.bing.com/indexnow?url=https://lifesciences.danaher.com/us/en/products.html&key=f8c27713c75e4e9186b8a5dd54d6eb582

@rgravitvl , @sharanagoudapatil  and @mhaack  - Lars kindly pointed me to this, which might be able to do what we want:  https://www.aem.live/developer/github-actions

@sharanagoudapatil  - here is a link to somebody that has done that:  https://github.com/niemal/seo-auto-index?tab=readme-ov-file


Fix memory issue when setting owner references to CSVs

Description

For the cluster-scoped operators, every namespace has a copy of the CSVs.
So when running load test with thousands of namespaces, setting owner references to the CSVs of mongodb atlas operator, crunchy bridge operator and service binding operator will cause the dbaas operator to load the thousands copies of CSVs to the cache, as a result the memory limit is exceeded.
The solution is to bypass the cache and read the CSVs directly from the API server.
Verification Steps

Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist (Definition of Done)

[ ] All acceptance criteria specified in JIRA or in issue number have been completed
[ ] Unit tests added that prove the fix is effective or the feature works
[ ] Documentation added for the feature
[ ] all relevant tests are passing
[ ] Code Review completed
[ ] Verified independently by reviewer

Fix for https://github.com/RHEcosystemAppEng/dbaas-operator/pull/84

do we know what the impact to the stability of the api server would be with a change like this? how often are these calls made?

do we know what the impact to the stability of the api server would be with a change like this? how often are these calls made?

I could reproduce the memory issue by simply installing the dbaas operator. There is no stability impact by only installing the dbaas operator based on my observation.
If we want to create thousands of namespaces at the same time, the impact of creating the namespaces and other objects is definitely much more significant than this. We will have to run load test to figure out the difference.
At most three separate calls are made to read a CSV by its name and namespace for reconciling the dbaasplatform CR once.
There is only one instance of the dbaasplatform CR, and the reconciling takes place most often during the startup of the operator to install the platform resources, around three times or less in every one second in my test. So there could be around ten calls to the api server made in every one second.

DBaaS operator pod resource metrics for load test  with 2000 users, projects and tenants.
It does take very long time to install the platform resources (i.e. operators), though.