Add note about MA by default

https://github.com/MicrosoftDocs/OfficeDocs-O365ITPro/issues/758

@nenonix @JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks.

Quick fyi: Added note is based on another note found on this article

@Sirkkuw - PR has been copy edited and is ready for final review. This is based on issue #758
Thank you.
cc: @kenwith

@mypil approved and merged.


Refactor autogen ext agents namespace

Why are these changes needed?

Removes autogen_ext.agents  init module and migrates magentic_one and openai assistant agents into its own namespaces.
Related issue number

#4408
Checks

[ ] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[ ] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[ ] I've made sure all auto checks have passed.

@afourney  @husseinmozannar @gagb  for awareness of the move.


Askar error encountered on first startup when upgrading an agent from 0.7.4-rc4 to 0.7.4

The following error was encountered when upgrading an agent already configured to use Askar from 0.7.4-rc4 to 0.7.4.  This error only occurred on the first startup.  Subsequent restarts occurred successfully without error.  The agent is configured to use Askar storage, and is hosted in OCP.
2022-07-08 17:43:35,904 aries_cloudagent.commands.start ERROR Exception during startup:
Traceback (most recent call last):
  File "/home/indy/.pyenv/versions/3.6.13/lib/python3.6/site-packages/aries_cloudagent/askar/store.py", line 160, in open_store
    self.key,
  File "/home/indy/.pyenv/versions/3.6.13/lib/python3.6/site-packages/aries_askar/store.py", line 328, in open
    return Store(await bindings.store_open(uri, key_method, pass_key, profile), uri)
  File "/home/indy/.pyenv/versions/3.6.13/lib/python3.6/site-packages/aries_askar/bindings/__init__.py", line 91, in store_open
    return_type=StoreHandle,
  File "/home/indy/.pyenv/versions/3.6.13/lib/python3.6/site-packages/aries_askar/bindings/lib.py", line 355, in invoke_async
    name, argtypes, *args, return_type=return_type
  File "/home/indy/.pyenv/versions/3.6.13/lib/python3.6/asyncio/futures.py", line 327, in __iter__
    yield self  # This tells Task to wait for completion.
  File "/home/indy/.pyenv/versions/3.6.13/lib/python3.6/asyncio/tasks.py", line 250, in _wakeup
    future.result()
  File "/home/indy/.pyenv/versions/3.6.13/lib/python3.6/asyncio/futures.py", line 243, in result
    raise self._exception
aries_askar.error.AskarError: Backend error
Caused by: error communicating with database: No route to host (os error 113)

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/home/indy/.pyenv/versions/3.6.13/lib/python3.6/site-packages/aries_cloudagent/commands/start.py", line 72, in init
    await startup
  File "/home/indy/.pyenv/versions/3.6.13/lib/python3.6/site-packages/aries_cloudagent/commands/start.py", line 28, in start_app
    await conductor.setup()
  File "/home/indy/.pyenv/versions/3.6.13/lib/python3.6/site-packages/aries_cloudagent/core/conductor.py", line 121, in setup
    self.root_profile, self.setup_public_did = await wallet_config(context)
  File "/home/indy/.pyenv/versions/3.6.13/lib/python3.6/site-packages/aries_cloudagent/config/wallet.py", line 55, in wallet_config
    profile = await mgr.open(context, profile_cfg)
  File "/home/indy/.pyenv/versions/3.6.13/lib/python3.6/site-packages/aries_cloudagent/askar/profile.py", line 261, in open
    opened = await store_config.open_store(provision=False)
  File "/home/indy/.pyenv/versions/3.6.13/lib/python3.6/site-packages/aries_cloudagent/askar/store.py", line 171, in open_store
    raise ProfileError("Error opening store") from err
aries_cloudagent.core.error.ProfileError: Error opening store

Shutting down

This is resolved. It looks like it was an environmental issue.

How could You solve this problem?


Voroni class can miss points

For "bad" geometry, the geometry.Voroni class, when generated polygons at the boundary, can fail to generate a polygon which actually includes point of interest.
That is,

def polygon_for(self, point_index, inf_dist=1):

May produce a polygon not including point_index.
Can we think of a better way, rather can scaling inf_dist, to stop this happening?

Added the new method polygon_for_by_distance which allows dynamically changing the size of the polygon so that the "edge at infinity" is a certain distance form the containing point.


 I can't get the examples to run.

I assume i can go to apps/example and do npm install - but this fails
If i go to packages/react-navigation and do npm install it also fails

npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/react
npm ERR!   dev<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!   packages/apollo-client/webui
npm ERR<EMAIL_ADDRESS>npm ERR!     node_modules/@dev-plugins/apollo-client-webui
npm ERR!       workspace packages\apollo-client\webui from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>|| ^16.3.0 || ^15.5.4" from<EMAIL_ADDRESS>npm ERR! node_modules/react-json-view
npm ERR!   dev<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!   packages/apollo-client/webui
npm ERR<EMAIL_ADDRESS>npm ERR!     node_modules/@dev-plugins/apollo-client-webui
npm ERR!       workspace packages\apollo-client\webui from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR!
npm ERR!
npm ERR! For a full report see:

hi there. this is the steps to run example
$ git clone https://github.com/expo/dev-plugins.git
$ cd dev-plugins
$ yarn
$ cd apps/example
$ yarn ios

i'll also update the readme for running example. thanks!

Only posting so other can find. npm install won't work, you must use yarn
npm install
npm error code ERESOLVE
npm error ERESOLVE unable to resolve dependency tree
npm error
npm error While resolving<EMAIL_ADDRESS>npm error Found<EMAIL_ADDRESS>npm error node_modules/react
npm error   dev<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm error   packages/apollo-client/webui
npm error<EMAIL_ADDRESS>npm error     node_modules/@dev-plugins/apollo-client-webui
npm error       workspace packages/apollo-client/webui from the root project
npm error
npm error Could not resolve dependency:
npm error peer<EMAIL_ADDRESS>|| ^16.3.0 || ^15.5.4" from<EMAIL_ADDRESS>npm error node_modules/react-json-view
npm error   dev<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm error   packages/apollo-client/webui
npm error<EMAIL_ADDRESS>npm error     node_modules/@dev-plugins/apollo-client-webui
npm error       workspace packages/apollo-client/webui from the root project
npm error
npm error Fix the upstream dependency conflict, or retry
npm error this command with --force or --legacy-peer-deps
npm error to accept an incorrect (and potentially broken) dependency resolution.
npm error
npm error
npm error For a full report see:
npm error ~/.npm/_logs/2024-11-18T19_49_37_641Z-eresolve-report.txt
npm error A complete log of this run can be found in:~/.npm/_logs/2024-11-18T19_49_37_641Z-debug-0.log

thanks. we now migrate from yarn to bun in this repo. also updated README that i forgot to update previously


JSON Schema support

Any plans on making angular-form-for compatible with JSON-Schema spec?

Unfortunately no, I have no plans of supporting this. JSON schema is very complicated and supporting it in a significant way would require a lot of work.
If you're looking for an Angular form abstraction that supports JSON schema I suggest you check out something like Textalk/angular-schema-form.


Error The partial [payregister/stripe] is not found

Somehow the partial for stripe was never added with payregister? I checked the parent theme as well and it's not there either. Reinstall?

Resolved. Needed to be activated.


Problem with some language code

Hi guys,
my integrate Gettext with .po and .mo file and work fine.
Yesterday i added new language Czech, after generete .po and .mo file ad uploaded to dedicated folder this language does't work.
I try to reboot my server, change language code from cs_CS to cz_CZ but still don't work.
This problem affect other 2 language that is Danish and Swedish.
How i do resolve this issue?
Thanks in advance for support.
Best.

Can you open the po file and see if "Language: \n" has the "Language: cs_CS\n" value? In my case the changes made to the po file with this lib deletes the language code from it.

I have same problem too.


Doesn't support SIIT

As per wikipedia:

Stateless IP/ICMP Translation (SIIT) translates between the packet header formats in IPv6 and IPv4. The SIIT method defines a class of IPv6 addresses called IPv4-translated addresses. They have the prefix ::ffff:0:0:0/96 and may be written as ::ffff:0:a.b.c.d, in which the IPv4 formatted address a.b.c.d refers to an IPv6-enabled node. The prefix was chosen to yield a zero-valued checksum to avoid changes to the transport protocol header checksum

SIIT addresses are not parsed correctly

::ffff:<IP_ADDRESS>
IPv6 { parts: [ 0, 0, 0, 0, 0, 65535, 32512, 1 ] }

should evaluate to <IP_ADDRESS>

It does in fact, it just does not perform conversion automatically; if you're parsing an IPv6 address, you get an IPv6 address structure back. See ipaddr.IPv6.isIPv4MappedAddress and ipaddr.IPv6.toIPv4Address.


Add QNSPSA optimizer demo.

Before submitting
Please complete the following checklist when submitting a PR:

[ X] Ensure that your tutorial executes correctly, and conforms to the
guidelines specified in the README.

[ ] Add a thumbnail link to your tutorial in beginner.rst, or if a
QML implementation, in implementations.rst.

[ X] All QML tutorials conform to
PEP8 standards.
To auto format files, simply pip install black, and then
run black -l 100 path/to/file.py.

When all the above are checked, delete everything above the dashed
line and fill in the pull request template.

Title:  Quantum natural SPSA optimizer
Summary: Tutorial of the QNSPSA optimizer
Relevant references: https://arxiv.org/abs/2103.09232
Possible Drawbacks: None
Related GitHub Issues: None
Notes:
1, A few latex math issue exists during the conversion. Need some help addressing them.
2, The final cell in the code sends a Braket Job to the Braket service. Might want to hard-code to stop the execution.

Hey @duanyh12! Thanks for this 💪
The reason the build is currently failing is that Sphinx is attempting to execute the tutorial: https://github.com/PennyLaneAI/qml/runs/6929254751?check_suite_focus=true#step:12:1446
This is because we have it currently set up that all files with name matching tutorial_*.py are executed. To deactivate this, simply rename your file to remove the initial tutorial_ from the filename.

@josh146 Thanks! Two remaining tasks from me:

the way I am using to reference to equations \eqref seems to be not converted well. Any suggestions on resolving this? Should I just change to hard-code the equation index?
for the braket hybrid jobs example at the end, the link to the job script is a temporary one. Need to change it to a permanent one after it is ready.

Let me know your feedback!

the way I am using to reference to equations \eqref seems to be not converted well. Any suggestions on resolving this? Should I just change to hard-code the equation index?

I'm not actually sure, I've never tried using this feature here 🤔 A quick google search brings up this:
https://stackoverflow.com/questions/67856170/how-to-refer-to-an-equation-in-sphinx
.. math::
    :name: eq:6

    F=\begin{bmatrix} \lambda_1 & 0 & 0\\0 & \lambda_2 & 0\\0 & 0 & \lambda_3\end{bmatrix}

I need to refer to this :ref:`Link title <eq:6>`

Alternatively, if that doesn't work, I found another solution here https://stackoverflow.com/a/72324328/10958457:
.. math::
     :label: eq_a

     e^i + 1 = 0

Which can be referenced in the text using :eq:`eq_a` and so on.
Note the blank lines and the back-tics in the reference.
Also, there may not be a space between :eq: and `eq_a` in the reference!
Further, no back-tics around the eq_a at the :label: line!

I had a quick skim through the demo, it looks great! I especially like the thumbnail.
One thing to keep in mind: since this demo is not executed, there is no output rendered on the website for cells. If you want to display output, you will need to hardcode this in. For example, see how the other Braket demo does this:
https://github.com/PennyLaneAI/qml/blob/master/demonstrations/braket-parallel-gradients.py#L189-L200

@alvaro-at-xanadu Again, really appreciate all the effort.  See if this version makes sense.


lightkube doesnot work with httpx 0.28.x

Running lightkube get/list commands failed with Unauthorized error.
Ran a sample from documentation:
from lightkube import Client
from lightkube.resources.core_v1 import Node

client = Client()
for node in client.list(Node):
    print(node.metadata.name)

Error message:
Traceback (most recent call last):
  File "/home/ubuntu/venv/lib/python3.10/site-packages/lightkube/core/generic_client.py", line 235, in raise_for_status
    resp.raise_for_status()
  File "/home/ubuntu/venv/lib/python3.10/site-packages/httpx/_models.py", line 829, in raise_for_status
    raise HTTPStatusError(message, request=request, response=self)
httpx.HTTPStatusError: Client error '401 Unauthorized' for url 'https://<IP_ADDRESS>:6443/api/v1/nodes'
For more information check: https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/401

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/ubuntu/venv/lib/python3.10/site-packages/lightkube/core/generic_client.py", line 318, in list
    cont, chunk = self.handle_response("list", resp, br)
  File "/home/ubuntu/venv/lib/python3.10/site-packages/lightkube/core/generic_client.py", line 251, in handle_response
    self.raise_for_status(resp)
  File "/home/ubuntu/venv/lib/python3.10/site-packages/lightkube/core/generic_client.py", line 237, in raise_for_status
    raise transform_exception(e)
lightkube.core.exceptions.ApiError: Unauthorized

Tried the same by downgrading httpx package to 0.27.x and it worked.
There are ssl related changes to httpx 0.28.0 which might cause this issue, see release notes
https://github.com/encode/httpx/releases/tag/0.28.0

Hi, what authentication method are you using? Can you post the corresponding configuration section (properly anonymized)?
I cannot reproduce in my setup and it doesn't seems lightkube is using any of the removed features of 0.28 (app and proxies). There are few deprecations that will need to be addressed, but they are just warning at the moment and so it should still work.

There seems to be actually a bug that will be fixed in 0.28.1: encode/httpx#3442

I released v0.15.6 to temporary avpod httpx version 0.28.0. Fixing the deprecation will require first compatiblity of RESPX lundberg/respx#277 with the changes to 0.28 and also the release of httpx 0.28.1 encode/httpx#3445

0.28.1 was supposed to fix that issue, that's why I kept it as a valid dependency version. Is the issue you encounter still reproducible with 0.28.1? If so, please provide your auth configuration

I have a fresh install of lightkube (version 0.15.6) which brings in httpx 0.28.1 as dependency and I'm also having this issue.
$ kubectl config current-context
k3d-test
$ kubectl get nodes
NAME                STATUS   ROLES                  AGE     VERSION
k3d-test-agent-0    Ready    <none>                 7d22h   v1.30.6+k3s1
k3d-test-server-0   Ready    control-plane,master   7d22h   v1.30.6+k3s1

from lightkube import Client
from lightkube.resources.core_v1 import Node

client = Client()
for node in client.list(Node):
    print(node.metadata.name)

Traceback (most recent call last):
  File "/Users/logileifsson/tmp/registrator/__pypackages__/3.11/lib/lightkube/core/generic_client.py", line 235, in raise_for_status
    resp.raise_for_status()
  File "/Users/logileifsson/tmp/registrator/__pypackages__/3.11/lib/httpx/_models.py", line 829, in raise_for_status
    raise HTTPStatusError(message, request=request, response=self)
httpx.HTTPStatusError: Client error '401 Unauthorized' for url 'https://<IP_ADDRESS>:57271/api/v1/nodes'
For more information check: https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/401

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/logileifsson/tmp/registrator/__pypackages__/3.11/lib/lightkube/core/generic_client.py", line 318, in list
    cont, chunk = self.handle_response("list", resp, br)
                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/logileifsson/tmp/registrator/__pypackages__/3.11/lib/lightkube/core/generic_client.py", line 251, in handle_response
    self.raise_for_status(resp)
  File "/Users/logileifsson/tmp/registrator/__pypackages__/3.11/lib/lightkube/core/generic_client.py", line 237, in raise_for_status
    raise transform_exception(e)
lightkube.core.exceptions.ApiError: Unauthorized

client.config
SingleConfig(context_name='k3d-test', context=Context(cluster='k3d-test', user='admin@k3d-test', namespace=None), cluster=Cluster(server='https://<IP_ADDRESS>:57271', certificate_auth=None, certificate_auth_data='LS0tLS1CRUdJTiBDRVJUSUZJQ0FURS0tLS0tCk1JSUJlRENDQVIyZ0F3SUJBZ0lCQURBS0JnZ3Foa2pPUFFRREFqQWpNU0V3SHdZRFZRUUREQmhyTTNNdGMyVnkKZG1WeUxXTmhRREUzTXpNeU1qZ3hOVFV3SGhjTk1qUXhNakF6TVRJeE5UVTFXaGNOTXpReE1qQXhNVEl4TlRVMQpXakFqTVNFd0h3WURWUVFEREJock0zTXRjMlZ5ZG1WeUxXTmhRREUzTXpNeU1qZ3hOVFV3V1RBVEJnY3Foa2pPClBRSUJCZ2dxaGtqT1BRTUJCd05DQUFTQW9odmVFQUVCRUpuQjBoRFJXak9KN1gvVDZTWHhFSk4vV3hPS0Vac2YKd2wzRlI0bGFqM0RtL2JPZFM4ano0SkxmcnNyVDRBbll5bEF2bFBmRVNPSkdvMEl3UURBT0JnTlZIUThCQWY4RQpCQU1DQXFRd0R3WURWUjBUQVFIL0JBVXdBd0VCL3pBZEJnTlZIUTRFRmdRVWxqNk9pbTZUbUFFak95b28rWk1LClBCQWVIWUF3Q2dZSUtvWkl6ajBFQXdJRFNRQXdSZ0loQUxoVnZwVnNvdXBaZjdOb2N1RXF4S0JEenZwdCtwNVIKMU5QTjdwQmJnUWFhQWlFQXliQmVRMFNnem9iZ3V0VUwxaklHTjNiMVpNcDBySWtkK3ljTW1qWlFLeEE9Ci0tLS0tRU5EIENFUlRJRklDQVRFLS0tLS0K', insecure=False), user=User(exec=None, username=None, password=None, token=None, auth_provider=None, client_cert=None, client_cert_data='LS0tLS1CRUdJTiBDRVJUSUZJQ0FURS0tLS0tCk1JSUJrRENDQVRlZ0F3SUJBZ0lJYzN3UEt6SVErS2d3Q2dZSUtvWkl6ajBFQXdJd0l6RWhNQjhHQTFVRUF3d1kKYXpOekxXTnNhV1Z1ZEMxallVQXhOek16TWpJNE1UVTFNQjRYRFRJME1USXdNekV5TVRVMU5Wb1hEVEkxTVRJdwpNekV5TVRVMU5Wb3dNREVYTUJVR0ExVUVDaE1PYzNsemRHVnRPbTFoYzNSbGNuTXhGVEFUQmdOVkJBTVRESE41CmMzUmxiVHBoWkcxcGJqQlpNQk1HQnlxR1NNNDlBZ0VHQ0NxR1NNNDlBd0VIQTBJQUJMYjFac2lwbTV3ZENTeisKaUFKd1J0K2FqUGVNZ2lwaHJYWDE3Q1ZqYmhPeTllZkVHMW5LNzlDYS9ybTRNenFYQitLcElsa2Y4cWtxSWJOZwpBOXFTakYyalNEQkdNQTRHQTFVZER3RUIvd1FFQXdJRm9EQVRCZ05WSFNVRUREQUtCZ2dyQmdFRkJRY0RBakFmCkJnTlZIU01FR0RBV2dCU2R1QmxscjNTN2JZNXA5MlphdWZqZUtZTkY3ekFLQmdncWhrak9QUVFEQWdOSEFEQkUKQWlCMFkzNFAvbTF4U1A1Z0pIUWpwNmJ6Z09jUGVkQlFCR01EdmxxWjNwZjUrUUlnTjZYekMvY1FvUXcwSC9OTQpJY0xCUUNNOUtiR0dqTDJSQWdmajg3eDJRSWM9Ci0tLS0tRU5EIENFUlRJRklDQVRFLS0tLS0KLS0tLS1CRUdJTiBDRVJUSUZJQ0FURS0tLS0tCk1JSUJkekNDQVIyZ0F3SUJBZ0lCQURBS0JnZ3Foa2pPUFFRREFqQWpNU0V3SHdZRFZRUUREQmhyTTNNdFkyeHAKWlc1MExXTmhRREUzTXpNeU1qZ3hOVFV3SGhjTk1qUXhNakF6TVRJeE5UVTFXaGNOTXpReE1qQXhNVEl4TlRVMQpXakFqTVNFd0h3WURWUVFEREJock0zTXRZMnhwWlc1MExXTmhRREUzTXpNeU1qZ3hOVFV3V1RBVEJnY3Foa2pPClBRSUJCZ2dxaGtqT1BRTUJCd05DQUFUOXFSR3lBNTZ5cE1ySytKU1BJSllld2Z4L0hRNldKdnNrZDVYUzhtbWgKVkszVFdyS0lVV2Qvb3pjYWZhNWdLRFhkaXVEWEM0OFBFQVRCVUtneFBXMFVvMEl3UURBT0JnTlZIUThCQWY4RQpCQU1DQXFRd0R3WURWUjBUQVFIL0JBVXdBd0VCL3pBZEJnTlZIUTRFRmdRVW5iZ1paYTkwdTIyT2FmZG1Xcm40CjNpbURSZTh3Q2dZSUtvWkl6ajBFQXdJRFNBQXdSUUlnQXdDUDJEZmYrallLZjltVmFBOXdLQ2hNZUR6NlpHbU8KUUlnbnQ2ZUo0MUVDSVFDSDhmbUkybjZNVlFSd3MzUk9SZTVJMWdKU0FEbTlQVVVhMWxmVVc5cUt6QT09Ci0tLS0tRU5EIENFUlRJRklDQVRFLS0tLS0K', client_key=None, client_key_data='LS0tLS1CRUdJTiBFQyBQUklWQVRFIEtFWS0tLS0tCk1IY0NBUUVFSUM4dzR5SjlCU0hMdTIyeTlTbHhoOEhYZENFeUxGbHZiVkJMU2xacnl0VkFvQW9HQ0NxR1NNNDkKQXdFSG9VUURRZ0FFdHZWbXlLbWJuQjBKTFA2SUFuQkczNXFNOTR5Q0ttR3RkZlhzSldOdUU3TDE1OFFiV2NydgowSnIrdWJnek9wY0g0cWtpV1IveXFTb2hzMkFEMnBLTVhRPT0KLS0tLS1FTkQgRUMgUFJJVkFURSBLRVktLS0tLQo='), fname=PosixPath('/Users/logileifsson/.kube/config'))

And I can confirm that downgrading httpx to 0.27.2 fixes the issue

Thanks for the details, this is going to be useful for the investigation. For now I just released v0.15.7 with httpx<0.28.0.


drivers: usb_dc_sam: get the maximum speed from DT as enum

For historical reasons, the maximum speed of the Atmel SAM USB
controller is done by comparing the maximum-speed property string from
the DT using strncmp at runtime.
Now that the DT_ENUM_IDX macro exists, we can use it to get the
maximum-speed property at build time and without string comparison.
Unsupported speed can also be reported at build time.
Note that the default speed in case the optional maximum-speed property
isn't present in the DT is changed from full-speed to high-speed to
match the property description. This is a no-op in practice as this
properties is defined at the soc level.
Signed-off-by: Aurelien Jarno<EMAIL_ADDRESS>

Note that there is the same issue on the STM32 driver. I'll do the same change if/when I get positive feedback about that PR.

In the generated device tree header, we have:
#define DT_N_S_soc_S_usbd_40038000_P_maximum_speed "full-speed"
#define DT_N_S_soc_S_usbd_40038000_P_maximum_speed_ENUM_IDX 1
#define DT_N_S_soc_S_usbd_40038000_P_maximum_speed_EXISTS 1

While some might consider using enum index a "cleaner" approach, we risk introducing serious bugs that can easily go unnoticed if we ever make changes to the enum orders in the future.

I doubt they will be changed without double checking their usage, but I get your point. Note that there are other locations where DT_ENUM_IDX is used that way, for example in the Atmel SAM GMAC driver to get the PHY connection type.

IMHO, the compile-time string comparison approach using DT_N_S_soc_S_usbd_40038000_P_maximum_speed is preferable to the index comparison approach using DT_N_S_soc_S_usbd_40038000_P_maximum_speed_ENUM_IDX (cc @galak).

DT_N_S_soc_S_usbd_40038000_P_maximum_speed is what is used in the existing code through DT_INST_PROP(0, maximum_speed). However you suggest to use compile-time string comparison approach, do you have an idea how to do that? There are many tricks existing using preprocessor macros, but they look hackish and I am not sure they are portable.

Hi @aurel32 re: compile time string comparison, @galak might be talking about how GCC will optimize out a strcmp of two constants.  For example, strcmp("foo", "foo") will be converted to constant 0.
I gave it a quick test and it doesn't work for strcmp() on Zephyr but does for __builtin_strcmp.  I wonder why?

Removing the stale tag that has just been added, I still believe this is an improvement over comparing strings.

Might be nice to have some #define s somewhere for the magic numbers.

While some might consider using enum index a "cleaner" approach, we risk introducing serious bugs that can easily go unnoticed if we ever make changes to the enum orders in the future.

Seems like this could be avoided in the short term by using macros for the enum indexes that get updated if necessary. There is an enhancement open to make this better (https://github.com/zephyrproject-rtos/zephyr/issues/21273) which we could look at if that's no good.


Add policy serialisation primitives to mismi-iam

I added types/serialisation stuff for policy documents in moog, but it's generally applicable and should probably go in mismi-iam instead, unless there's a better place I'm not aware of.

mismi-iam is probably good @nhibberd wrote some in bucket with some quasi-quoters as well that should probably come in.

Yeah ideally will lift bucket code up

I just did plain Haskell types with aeson - don't have a strong preference, but it seems like the QQ implementation in bucket doesn't do a lot to ensure that the provided policy is well-formed (e.g., version string is actually a valid version, et cetera)?

How did you do the permissions?

@nhibberd currently "Action" is just a newtype around Text (like this); wasn't sure if it'd be worth enumerating everything at the type level - open to suggestions there.


Examples

First example, TwoTickers runs

Note that merge does not work well - Sink only sees the ticker with the higher frequency. I guess this is because race kills the loser, so we never see the slower ticks.


Add support for running Ultralytics-based models

Our new team member, @YellowFish96, is building models that segment both voids and dislocations. He uses https://github.com/ultralytics/ultralytics

Ultralytics also has a tracking code and @YellowFish96 has done some systematic analysis. Found DeepSORT and BoTSORT to work very well


NPM dependencies need to be updated

A couple dozen dependencies are out of date and causing security alerts

False alarm, we have a bot doing this, most vulns are dev


Fix bug overwrite when using prefix paths

Was accidently reusing the path parts when creating a new path entry This meant that prefixed paths would work the first time but would fail on any attempts after that.

/cc @AlinaSecret @natifridman

/lgtm
/approve


[bug] default theme color problem

Hi there!
Using DBeaver 6.0.* default theme in Ubuntu, there are some strings that are undearable;
In General -> Editors (to name one), you can read
"See                              for associating editors with file type"
Screenshot here: https://www.dropbox.com/s/4v5p0lcjr2sxmja/DBeaver604.jpg?dl=0

cannot reproduce issue on build 6.0.4 on Ubuntu 19.04.

Might have forgotten: Ubuntu Mate 18.04, default theme
On 23:23, Fri, 10 May 2019 uslss<EMAIL_ADDRESS>wrote:

cannot reproduce issue on build 6.0.4 on Ubuntu 19.04.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dbeaver/dbeaver/issues/5869#issuecomment-491434304,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA57UL4CSTARCTSPIVMDLNTPUXRUTANCNFSM4HLHVE5Q
.

Not on the same place but probably related: default tab color text are hard to read on my Manjaro Linux:

Not on the same place but probably related: default tab color text (light theme) are hard to read on my Manjaro Linux:

I had the same issue with Manjaro due to its Gtk default dark theme.
Solved running dbeaver with a different theme.
env GTK_THEME=Adwaita:light dbeaver

env GTK_THEME=Adwaita:light dbeaver

Where should i paste it?

I have the same issue, I am running on Elementary OS with the dark variant enabled. When I run GTK_THEME=elementary:light dbeaver I get:

But GTK_THEME=elementary:dark dbeaver gives me this difficult to use version:

Going into Window>Preferences>General>Appearance and changing the theme to Dark gave me the following, now perfectly usable without any special environment variables:

Would it be possible to have the program automatically select the light or dark theme based on the user's current settings? I see theme settings for dark variants of windows and mac, so I think it would be a nice quality of life improvement for first-time users.

@serge-rider I have a similar question on flatpak side of things, can we change the default behavior. Currently, for flatpak, I'm just enforcing light theme. More info here: https://github.com/flathub/io.dbeaver.DBeaverCommunity/issues/34

Some issue about colors https://github.com/dbeaver/dbeaver/issues/8299

I am still facing the theme-related issue on the latest version.
System: Ubuntu 22.04.2 LTS
DBeaver: Version <IP_ADDRESS>304091457
Steps to reproduce :
Change the system theme to Dark
Change the DBeaver theme to light. (Windows -> Preferences -> User Interface -> Appearance)

If I switch back to the light theme it gets fixed. Seems like the DBeaver UI setting is not honoring the appearance settings.
Hack for anyone who wants to use a light theme on Ubuntu but can't

Install Ubuntu Tweaks and change the application theme to light

I am still facing the theme-related issue on the latest version.
System: Ubuntu 22.04.2 LTS DBeaver: Version <IP_ADDRESS>304091457
Steps to reproduce : Change the system theme to Dark Change the DBeaver theme to light. (Windows -> Preferences -> User Interface -> Appearance)

If I switch back to the light theme it gets fixed. Seems like the DBeaver UI setting is not honoring the appearance settings.

you can fix it with this theme https://marketplace.eclipse.org/content/darkest-dark-theme-devstyle

Tengo el mismo problema pensé que eral algo nuevo pero veo que no va a tener solución por ahora 😥

@Numaherrera How do you install that into DBeaver? Linux Mint dark theme also unreadable.


Error:  Cannot find module './lib/WebSocket

Is this file missing?  I have run NPM install on the clone and received no errors.  Thanks!

Please disregard, I missed a line on the stacktrace!

@Christopheraburns I got this same error, Could you tell me how to fix it ??


fix: use scroll margin instead of padding to align scroll headers

Updates our scroll alignment code to use scroll-margin to line up the scroll rather than using a negative margin trick. I hadn't used it before when I first did this alignment fix but browser support seems good 🚀
https://github.com/scalar/scalar/assets/6374090/b418a519-b4d8-4b41-979a-76a818a04f83

Unreal work @hwkr !!! Let's go :)

sick! this was annoying me haha


Font scaling doesn't work if the window or DOM element is resized

Fons aren't resized by the resize function (in labels, legends, etc).


adds file_info endpoint method

The people have spoken: https://test-i-o.slack.com/archives/C078C712WP4/p1720521196417309


fix 'rmi -f imgID' problem

If an image has been tagged to multiple repos and tags, 'docker rmi -f imgID' will just untag one repo other than untagging all repos and deleting the image. This patch fix it.
before:

after:

I think 'rmi -f imgID' just untag a random repo:tag is wired. Deleting all repos and tags may be proper

Can you please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "upstream-master" git@github.com:x1022as/docker.git somewhere
$ cd somewhere
$ git rebase -i HEAD~2
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

This will update the existing PR, so you do not need to open a new one.

@GordonTheTurtle，thanks :)

this was fixed with https://github.com/docker/docker/pull/11011 but i think we would still like your test can you make sure that rebasing on master still makes your test pass

ping @coolljt0725 to make sure

@jfrazelle that PR (#11011) is different in that it only solves the (incorrect) warning a user receives on  an image/tag that isn't going to be removed anyway.. in this PR, the patches are solving the issue that rm -f doesn't actually remove all the repos/tags existing for an image.  It would be good to decide if that is the intended behavior, because the current behavior is somewhat random (one tag of the various tags of an image will get removed, but the rest are left around).

ah ok thnks for clearing it up, the PRs are merging together at this point
in my head
On Mon, Mar 2, 2015 at 11:08 AM, Phil Estes<EMAIL_ADDRESS>wrote:

@jfrazelle https://github.com/jfrazelle that PR (#11011
https://github.com/docker/docker/pull/11011) is different in that it
only solves the (incorrect) warning a user receives on an image/tag that
isn't going to be removed anyway.. in this PR, the patches are solving the
issue that rm -f doesn't actually remove all the repos/tags existing for
an image. It would be good to decide if that is the intended behavior,
because the current behavior is somewhat random (one tag of the various
tags of an image will get removed, but the rest are left around).
—
Reply to this email directly or view it on GitHub
https://github.com/docker/docker/pull/11058#issuecomment-76784601.

@x1022as thanks for the test changes; I've reviewed the code and it looks correct, however, I also looked at the docs and there is no clear specification of what -f will do with an image tagged in multiple repos or removal of tag by ID with -f.
I think you will need to add to this PR an update to the docs (commandline/cli.md) with specific details on what -f will do, and in some sense this is then a "proposal" of behavior and I want other input from maintainers on whether this is desired result of -f.

there is no clear specification of what -f will do with an image tagged in multiple repos or removal of tag by ID with -f

Ah, yes. Just remembered I bookmarked this issue for that reason. I hadn't "decided" yet what would be the most wanted outcome of running this command. iirc, some commands will produce an error if multiple results (images/containers) are returned; should this do the same (for consistency)?. The fun thing here is of course, should this be regarded multiple images or a single image, with multiple names/tags?
It could surprise people if they want to remove a specific tag of an image, but use the "id" of the image for convenience and end up removing all aliases of that image.
Perhaps I'm overthinking it, in which case, ignore my rambling

I think they are mostly good thoughts and why I think this needs broader review as it is more like a proposal and not a simple bug fix.  To your comment:

It could surprise people if they want to remove a specific tag of an image, but use the "id" of the image for convenience and end up removing all aliases of that image.

I would say that in this case the user has been very silly.. if they are using an ID but wish to only remove a specified tag, they have gone the wrong direction :)  And with the fix from #11011 it should now work properly to remove that single tag without getting a warning about other repo/tags on the same ID.

I would say that in this case the user has been very silly

True, happened to do it once, though, to clean up an untagged image, ending up with another one being untagged image :smile:

This PR is  different from PR( #11011 ) which is fix the misleading/confusing warning when remove a image whose parents has tagged to multiple repository, this PR is to remove a image which has tagged to multiple repository through ID. I have meet this situation when I work PR#11011, at that time I just think it was a desired behavior and didn't realize it was a bug. Since an image could tagged to multiple repository and if I want to remove all the repository of this image, use the image ID to remove all the repository is good way,  or else we have to remove the repository one by one. So I think this PR is good

@estesp Thanks for you comment,  I've change this PR according to your advice. please let me know if this PR need any modifications.

@jfrazelle when I perform '$ docker rmi IMAGE_ID', docker shows "Conflict, cannot delete image 4986bf8c1536 because it is tagged in multiple repositories, use -f to force"
So I thought ‘$docker rmi -f IMAGE_ID’ will complete this to untag all the repos and tags  and delete the image.
while -f just untag a random repo, which is wired. I thought this should be fixed

Ping @dmcgowan @jlhawn @stevvooe: I'm not sure if this is desirable, or has potential interactions with #11109? Thanks!

@icecrime The proposed behavior is correct. If one requests the forced removal of an image id, all references to that image id should be removed, as well. Basically, a "-f" accompanied command should do everything in its power to succeed. Anything else would be incorrect if we adhere to a reasonable data model.
Looking at #11109, it's clear that one PR must consider the other.

I am also +1 for the specified rmi -f behavior. For #11109, these should definitely be tested with each other.

This pr has little conflict with #11109 on code. and yes, the functionality need to be tested. Comparing the code, it seems that rmi could work fine.

Rebase needed, now that #11109 is merged I will do another round of testing after rebase.

Following code in integration-cli:
d := NewDaemon(t)
if err := d.StartWithBusybox(); err != nil {
        t.Fatal(err)
}

always failed in windows build, how to deal with this problem?

Tested with changes from pull by digest, works as expected.  Just waiting for tests to pass

Test failed, and I don't know why. Can anyone show me how to fix this?

It is a registry issue they are looking into it
On Saturday, March 21, 2015, x1022as<EMAIL_ADDRESS>wrote:

Test failed, and I don't know why. Can anyone show me how to fix this?
—
Reply to this email directly or view it on GitHub
https://github.com/docker/docker/pull/11058#issuecomment-84316187.

LGTM

LGTM for code; needs quick docs review--I made a suggestion on clarifying the sentence: @SvenDowideit @moxiegirl @fredlf can make other requests/comments

Docs need some minor edits.

@jamtur01 done, thanks.

LGTM

Needs rebase

I did the rebase and push to my upstream-master branch, but why this PR not updated?

@x1022as I was having trouble earlier with getting PRs updated, I think github is having difficulties as I was getting a 500 attempting a recreate.  Let's check back tomorrow to see if it is resolved.

@dmcgowan thanks, I will check this tomorrow :)

Still not updated, how to update this forcibly?

Try to re-commit and force push again and see if it works. I tried closing and re-opening on another commit and it wouldn't let me re-open.

@x1022as was there a problem with the rebase? I see a merge commit found its way in

@x1022as Some files are not go fmt'ed: can you please fix?
These files are not properly gofmt'd:
 - daemon/image_delete.go

Please reformat the above files using "gofmt -s -w" and commit the result

@dmcgowan I did the rebase on another machine, I am not sure if this is the cause.
I did the push again, and now it works . Thanks

@icecrime sorry for that, I'm used to depend on go-fmt tools

looks like stripTrailingCharacters got removed in commit b9d2ede4d4dc67b97abd0254a9e431aa9c89b932
Sorry about that--you need to change to strings.TrimSpace as that commit shows. Hopefully this is the last hiccup to get your PR done! :)

@estesp Done :)

I would merge, but I would like @fredlf's comment to be addressed as it seems like he has reasonable concerns about clarity of the doc language: https://github.com/docker/docker/pull/11058#discussion_r27808543

I've changed the doc, ping @fredlf to check.

Didn't know this issue before,so I opened #12135,they are related,but focused on different situation. I think this issue is about the -f flag,we don't have a specified behaviour for it.  In my opinion, it should be restricted to the relationship with containers. Here is all the situations:

docker rmi [-f] <image id>
(1)  mapping to one repo and one tag,If no container use it, untag and delete. with containers use this image(not running),-f can untag and delete it ,or print error message
(2) mapping to one repo and multiple tags,If  no container use,untag all and delete. with containers use  it (not running) ,-f can untag all and delete it,or print error message
(3) mapping to multiple repo and multiple tags, If no container use ,untag all and delete. with containers use it(not running) ,-f can untag all and delete it, or print error message

#12135 is about 1.(2)

docker rmi [-f] <name:[tag]>  (1) this is the  only repo:tag of the image id, same as 1.(1) (2) one repo and many tags mapping to this image, if no container use it, untag all and delete .with containers use it(not running),If no-f, remove this tag ,no deletion. with -f, untag all and delete  (3) multiple repo and tags mapping to this image, if no container use it, untag all and delete.with containers use it(not running), if no -f, remove this tag,no deletion. with -f`,untag all and delete

@jamtur01 ping back

Docs LGTM

LGTM

Thanks everyone for the reviews!
LGTM

..but @x1022as it needs a rebase again :( sorry

so sorry I think we just need a rebase :(

that's ok, rebase done


Update infrastructure docs

Remove references to RE-D because organisationally that team no longer exists.
Replace references to RE-D with RE-Gov.Wifi because that's the new team.
Fix typo.

There are four repeated commits because I retroactively signed my commits and now they're marked as verified.
These are the four repeats:

Update reliability engineering support team (d7ea8cf)
Update more reliability engineering support team references (ea485b2)
Remove erroneous . in GovWifi team name (6454adb)
Fix missing closing parentheses to enable hyperlink (2ebaa68)


Cannot connect multiple DB configured with SSH tunnel

Description
Since version 24.0.1, I can't connect at the same time to multiple DB configured with an SSH tunnel.
I have this error message, after one DB is connected and I try to connect another one :
FATAL: password authentication failed for user "XXX"

DBeaver Version
Community Edition 24.0.1
Operating System
Debian GNU/Linux 11 (bullseye)
Database and driver

PostgreSQL 14.10
PostgreSQL JDBC Driver 42.2.5

Steps to reproduce

I connect a database A configured with an SSH tunnel configured
Connection is established correctly
I try to connect another database B configured with an SSH tunnel configured (same host/same key)
Connection is not established : eeror message :point_right: FATAL: password authentication failed for user "XXX"
If I disconnect database A and I retry to connect database B, no errror, and connection is established correctly

Additional context
It was working smoothly with previous DBeaver versions. A collegue, tried with version Community Edition 24.0.0 and the same configurations. There is no problem.

Hello @frederic-pierre,
Can you please provide log files? You can find instructions here: https://github.com/dbeaver/dbeaver/wiki/Log-files#log-files

Here is an extract of the debug log file, when I reproduce the error. Is it enough ? I cannot share with you the entire log file without removing all sensitive information.
2024-03-26 14:54:47.414 - Connect with 'foobar' (postgres-jdbc-XXXX-XXXX)
2024-03-26 14:54:47.417 - SSHSessionController: Set up port forwarding :52988 <- data-base-host.com:37666
2024-03-26 14:54:47.418 - Listening on localhost/<IP_ADDRESS>:52988
2024-03-26 14:54:47.419 - Execution context opened (foobar; Main <foobar>; 24)
2024-03-26 14:54:47.420 - Initiate connection to PostgreSQL database<EMAIL_ADDRESS>for Main <foobar>
2024-03-26 14:54:47.592 - SSHSessionController: Remove port forwarding :52988 <- data-base-host.com:37666
2024-03-26 14:54:47.592 - Closing listener on localhost/<IP_ADDRESS>:52988
2024-03-26 14:54:47.592 - SSHSessionController: Releasing session for foobar [PostgreSQL]
2024-03-26 14:54:47.592 - FATAL: password authentication failed for user "foobar"
org.postgresql.util.PSQLException: FATAL: password authentication failed for user "foobar"
	at org.postgresql.Driver$ConnectThread.getResult(Driver.java:405)
	at org.postgresql.Driver.connect(Driver.java:263)
	at org.jkiss.dbeaver.model.impl.jdbc.JDBCConnectionOpener.run(JDBCConnectionOpener.java:109)
	at org.jkiss.dbeaver.model.impl.jdbc.JDBCConnectionOpener.run(JDBCConnectionOpener.java:83)
	at org.jkiss.dbeaver.model.impl.jdbc.JDBCDataSource.openConnection(JDBCDataSource.java:221)
	at org.jkiss.dbeaver.ext.postgresql.model.PostgreDataSource.openConnection(PostgreDataSource.java:565)
	at org.jkiss.dbeaver.model.impl.jdbc.JDBCExecutionContext.connect(JDBCExecutionContext.java:106)
	at org.jkiss.dbeaver.model.impl.jdbc.JDBCRemoteInstance.initializeMainContext(JDBCRemoteInstance.java:102)
	at org.jkiss.dbeaver.ext.postgresql.model.PostgreDatabase.checkInstanceConnection(PostgreDatabase.java:233)
	at org.jkiss.dbeaver.ext.postgresql.model.PostgreDatabase.<init>(PostgreDatabase.java:120)
	at org.jkiss.dbeaver.ext.postgresql.model.PostgreDataSource.createDatabaseImpl(PostgreDataSource.java:278)
	at org.jkiss.dbeaver.ext.postgresql.model.PostgreDataSource.initializeRemoteInstance(PostgreDataSource.java:173)
	at org.jkiss.dbeaver.model.impl.jdbc.JDBCDataSource.<init>(JDBCDataSource.java:103)
	at org.jkiss.dbeaver.model.impl.jdbc.JDBCDataSource.<init>(JDBCDataSource.java:94)
	at org.jkiss.dbeaver.ext.postgresql.model.PostgreDataSource.<init>(PostgreDataSource.java:108)
	at org.jkiss.dbeaver.ext.postgresql.PostgreDataSourceProvider.openDataSource(PostgreDataSourceProvider.java:115)
	at org.jkiss.dbeaver.registry.DataSourceDescriptor.openDataSource(DataSourceDescriptor.java:1402)
	at org.jkiss.dbeaver.registry.DataSourceDescriptor.connect0(DataSourceDescriptor.java:1239)
	at org.jkiss.dbeaver.registry.DataSourceDescriptor.connect(DataSourceDescriptor.java:1025)
	at org.jkiss.dbeaver.runtime.jobs.ConnectJob.run(ConnectJob.java:77)
	at org.jkiss.dbeaver.runtime.jobs.ConnectJob.runSync(ConnectJob.java:104)
	at org.jkiss.dbeaver.ui.actions.datasource.DataSourceHandler.connectToDataSource(DataSourceHandler.java:117)
	at org.jkiss.dbeaver.ui.actions.datasource.UIServiceConnectionsImpl.initConnection(UIServiceConnectionsImpl.java:87)
	at org.jkiss.dbeaver.model.DBUtils.initDataSource(DBUtils.java:2488)
	at org.jkiss.dbeaver.model.navigator.DBNDataSource.initializeNode(DBNDataSource.java:158)
	at org.jkiss.dbeaver.model.navigator.DBNDatabaseNode.getChildren(DBNDatabaseNode.java:227)
	at org.jkiss.dbeaver.model.navigator.DBNDatabaseNode.getChildren(DBNDatabaseNode.java:1)
	at org.jkiss.dbeaver.model.navigator.DBNUtils.getNodeChildrenFiltered(DBNUtils.java:91)
	at org.jkiss.dbeaver.ui.navigator.database.load.TreeLoadService.evaluate(TreeLoadService.java:49)
	at org.jkiss.dbeaver.ui.navigator.database.load.TreeLoadService.evaluate(TreeLoadService.java:1)
	at org.jkiss.dbeaver.ui.LoadingJob.run(LoadingJob.java:89)
	at org.jkiss.dbeaver.ui.LoadingJob.run(LoadingJob.java:73)
	at org.jkiss.dbeaver.model.runtime.AbstractJob.run(AbstractJob.java:114)
	at org.eclipse.core.internal.jobs.Worker.run(Worker.java:63)
2024-03-26 14:54:47.596 - <<copyDone>> woke to: java.net.SocketException: Socket closed

I need the complete logs that include information about the first initiated connection to understand why the second one fails.
You can delete existing log files and reproduce the issue from the beginning. Logs should contain a minimum amount of sensitive information, so removing it all should be fairly easy.
Speaking of privacy, we have a ticket regarding hiding sensitive information in log files: #17088

It's been a while since no update here. If the issue is still actual please let me know and provide additional information.

Hello, sorry for the delay, here is the log file for this issue that is still actual :
dbeaver-debug.log

Please try upgrading to the latest version and let me know if your issue still reproduces there: https://dbeaver.io/download/

I am already using DBeaver Community 24.0.3. The last logfile correspond to that version.

I didn't notice that from the logs; my bad. I don't see anything suspicious in the logs.
I'm connecting to two PostgreSQL databases on the same host through the same SSH tunnel but bound to different ports and using different credentials. Both connections were established fine without any issues.
Can you please try turning off this option on both connections (or in the profile) to see if it has any effect?

It's working now that I disabled tunnel sharing ! Thanks a lot @ShadelessFox

It's not a solution, unfortunately. It's supposed to work with this option enabled.

In our case, we try to connect to two DB configured with the same tunnel SSH (same host / same port) but the DB host are different.

So the hostnames are not localhost, and both point to different IP addresses even though you have an SSH tunnel configured?

Exactly :

for DB A : host is app-A.foobar.com and port XXXX
for DB B : host is app-B.foobar.com and port YYYY

Thanks for the info.

It's working now that I disabled tunnel sharing ! Thanks a lot @ShadelessFox

Definitely works after disabling tunnel sharing. Thanks.

Can you please confirm that you have the exact case as described here https://github.com/dbeaver/dbeaver/issues/23238#issuecomment-2079019450? @borkor

Can you please confirm that you have the exact case as described here #23238 (comment)? @borkor

Exactly the same issue. Multiple hosts, which are not localhost and the same connection data for tunne.

Thank you! This info will help us reproduce the issue.

Unfortunately, we were unable to reproduce the issue.
I have two databases accessible via different IPs, ports, and credentials over the same SSH tunnel. Connections are established without any issues in any order; revalidation also works.
Can you please confirm that you don't have explicit port forwarding configured in the advanced settings? https://github.com/dbeaver/dbeaver/issues/23238#issuecomment-2077596466 Maybe there are some other deviations from the default configuration?
Are you using an SSH tunnel via profiles or configure SSH tunnels separately for each connection?
@frederic-pierre @borkor

I have the issue with the SSH tunnel configured via a profile as well as separately, and I do not have port forwarding configured in the advanced settings.

Thanks for the information.

Unfortunately, we were unable to reproduce the issue.
I have two databases accessible via different IPs, ports, and credentials over the same SSH tunnel. Connections are established without any issues in any order; revalidation also works.
Can you please confirm that you don't have explicit port forwarding configured in the advanced settings? #23238 (comment) Maybe there are some other deviations from the default configuration?
Are you using an SSH tunnel via profiles or configure SSH tunnels separately for each connection?
@frederic-pierre @borkor

I am configuring ssh tunnel for each connection. I do not use profiles (did not know something like this existed)

Also, I cannot reproduce the issue


Cannot include

React Boilerplate
Before opening a new issue, you may find an answer in already closed issues:
Checked but didnt find related fix / issue
Issue Type

[X ] Bug (https://github.com/react-boilerplate/react-boilerplate/blob/master/.github/CONTRIBUTING.md#bug-reports)

Description
Cannot install aws-iot-device-sdk - cannot work out how to change webpack to cater for suggested fixes from elsewhere on the web.  I'm thinking I cannot be the first person to try and include an AWS JS SDK into a React-boilerplate project.
npm run build:dll

<EMAIL_ADDRESS>build:dll /home/jamesl/dev/client
node ./internals/scripts/dependencies.js

Building the Webpack DLL...
Hash: d54eb1a94554297255b7
Version: webpack 3.5.5
Time: 1419ms
Asset     Size  Chunks                    Chunk Names
reactBoilerplateDeps.dll.js  3.63 MB       0  [emitted]  [big]  reactBoilerplateDeps
chunk    {0} reactBoilerplateDeps.dll.js (reactBoilerplateDeps) 3.05 MB [entry] [rendered]
ERROR in ./node_modules/aws-iot-device-sdk/common/lib/tls-reader.js
Module not found: Error: Can't resolve 'fs' in '/home/jamesl/dev/client/node_modules/aws-iot-device-sdk/common/lib'
@ ./node_modules/aws-iot-device-sdk/common/lib/tls-reader.js 17:14-27
@ ./node_modules/aws-iot-device-sdk/device/index.js
@ ./node_modules/aws-iot-device-sdk/index.js
@ dll reactBoilerplateDeps
ERROR in ./node_modules/aws-iot-device-sdk/device/index.js
Module not found: Error: Can't resolve 'fs' in '/home/jamesl/dev/client/node_modules/aws-iot-device-sdk/device'
@ ./node_modules/aws-iot-device-sdk/device/index.js 29:9-22
@ ./node_modules/aws-iot-device-sdk/index.js
@ dll reactBoilerplateDeps
ERROR in ./node_modules/aws-iot-device-sdk/device/lib/tls.js
Module not found: Error: Can't resolve 'tls' in '/home/jamesl/dev/client/node_modules/aws-iot-device-sdk/device/lib'
@ ./node_modules/aws-iot-device-sdk/device/lib/tls.js 17:10-24
@ ./node_modules/aws-iot-device-sdk/device/index.js
@ ./node_modules/aws-iot-device-sdk/index.js
@ dll reactBoilerplateDeps
(Add images if possible)
Steps to reproduce
Simply clone the repo as suggested on main README.MD and then 'npm install --save aws-iot-device-sdk' then run 'npm run build:dll'
Versions

React-Boilerplate (see package.json):  3.5.0
Node/NPM:  8.4.0 / 5.5.1
Browser: Chrome

I found the following article:  https://www.andrewhfarmer.com/aws-sdk-with-webpack/
I include the aws-sdk as it suggests and now there are no errors. Yay.
I didn't have to do the
module: {
noParse: [
/aws-sdk/,
]
}
part.


add demos

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@7ea7600). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #2   +/-   ##
=======================================
  Coverage        ?   99.08%
=======================================
  Files           ?        1
  Lines           ?      109
  Branches        ?       38
=======================================
  Hits            ?      108
  Misses          ?        1
  Partials        ?        0

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 7ea7600...1954ecb. Read the comment docs.


CI: run e2e tests against all supported Kubernetes versions

Closes #323.
Signed-off-by: Steve Kriss<EMAIL_ADDRESS>

Having trouble getting all the jobs to pass at the same time, we may need to do some more flake-fighting here.

@skriss do you know why the test-e2e-linux and test-example-linux CI jobs are reporting "Waiting for status to be reported"?

@skriss do you know why the test-e2e-linux and test-example-linux CI jobs are reporting "Waiting for status to be reported"?

Yeah, each of those have now been replaced by the three matrix jobs (e.g. test-e2e-linux (kubernetes:latest), test-e2e-linux (kubernetes:n-1), ...), but the original non-matrix jobs are still marked as required. Once we're happy with this PR, I'll update the branch protection rules to require the matrix jobs and not require the old jobs.

@skriss this PR has 2 approvals but I'll let you merge as I don't fully understand the steps for changing the branch protection rules.

OK, I'm going to get this in now and other PRs will have to update.


Improve the presentation of Checkboxes

This PR improves the presentation of check boxes, particularly in light mode (fixes the color of the tick). It adds a disabled state presentation and improves the alignment of the info icon when the checkbox has a tooltip. Finally, it adds checkboxes to the design system form control page to easily see the changes that have been made.
Also, updated indeterminate presentation to better match the ticked presentation and to display better when used in table headers (particularly light mode). Also fixed the width of the indeterminate indicator so its correctly centred in the box.
Validated both light and dark mode.
Before:

After:

This isn't new because it's in master now and looks the same-ish, but the indeterminate bg + border makes it invisible against the table header bg (which is the only place indeterminate is used):

This isn't new because it's in master now and looks the same-ish, but the indeterminate bg + border makes it invisible against the table header bg (which is the only place indeterminate is used):

This isn't new because it's in master now and looks the same-ish, but the indeterminate bg + border makes it invisible against the table header bg (which is the only place indeterminate is used):

For that state it could be more similar to the checked state with a blue bg and white - for those cases

This isn't new because it's in master now and looks the same-ish, but the indeterminate bg + border makes it invisible against the table header bg (which is the only place indeterminate is used):

For that state it could be more similar to the checked state with a blue bg and white - for those cases

@vincent99 @lvuch I've updated this PR with a fix for the indeterminate checkbox style.


New version: Catsxp.Catsxp version <IP_ADDRESS>

Created by 🥟 Dumplings in workflow run #8341.

Log
Updated: <IP_ADDRESS> → <IP_ADDRESS>
Submitting WinGet manifests
Creating manifests
Uploading manifests and making commits
The manifests for the last version <IP_ADDRESS> will be removed. Reason: This task is configured to remove the last version
Creating a pull request

Microsoft Reviewers: Open in CodeFlow

@wingetbot run


Fix graphs docstring

N_EDGE definition in the docstring is now fixed.
Explicitly mention features for nodes and edges.

Small typo, I came across while going through the code :)

@googlebot I signed it!

Thanks for the PR!


SoundCloud: Unable to download JSON metadata: HTTP Error 401

$ youtube-dl -j "https://soundcloud.com/universalrepublic/jessie-j-ariana-grande-nicki-minaj-bang-bang"
ERROR: Unable to download JSON metadata: HTTP Error 401: Unauthorized (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Duplicate of #6930.


Relation to logsearch/logsearch-boshrelease

Hi, how is this related to https://github.com/logsearch/logsearch-boshrelease?
It seems to have the same content, so why do we also have this here?

Hi @voelzmo!
https://github.com/logsearch/logsearch-boshrelease is not developing anymore. As per discussion with @mrdavidlaing, we'd created a copy of that and going to develop and support it from now on.

Thanks @axelaris for the info. Would you mind updating both repositories so that this is obvious to anyone dropping by?
Also: bosh.io still refers to the other repository http://bosh.io/releases/github.com/logsearch/logsearch-boshrelease?all=1 that should probably also be updated.

Sure, @voelzmo.
Thanks for heads up!

FYI - People who are as un-aware as me are opening issues on the old repo: https://github.com/logsearch/logsearch-boshrelease/issues/257

I know, we're going to review all issues in the old repo in the next couple of weeks.

@axelaris https://github.com/logsearch/logsearch-boshrelease/pull/258

Hi,
I found also this repo pivotal-cf/logsearch-boshrelease which seems to have activities, with new recent BOSH release. I know it's a fork from the non maintained - but quid of the new releases, sharing the same blobstore ?
Sylvain

@muconsulting pivotal-cf/logsearch-boshrelease is used in the PCF Log Search tile. It will be made private soon.
@voelzmo https://github.com/logsearch/logsearch-boshrelease/pull/259

logsearch/logsearch-boshrelease has been replaced with cloudfoundry-community/logsearch-boshrelease on bosh.io


fix ignored tests in branch #48-otp

fix this tests for brach  #48-otp:

org.itevents.service.transactional.MyBatisUserServiceTest#shouldThrowEntityAlreadyExistsServiceExceptionWhenAddExistingSubscriber

2)public void shouldSendEmailWithActivationLink() throws Exception

org.itevents.service.transactional.MyBatisUserServiceTest#shouldAddSubscriber

@AndriyBaibak time tracking: 1 hour spent

blocked with #156 @AndriyBaibak @Garazd pls fix it ASAP

@AndriyBaibak time tracking: 30 minutes spent

@AndriyBaibak please, make 2-nd task inactual here, because it is actual in issue #155

@IgorMaksymov @romach update task


Add header

Add header to the sample-showcase.
These changes will only be apparent on deploy. As such, NugetPackage.nuspec will now bundle files into /sample-showcase/frontend-sample-showcase, and /sample-showcase/index.html now links to an iframe.
There is an added change in window.history.replaceState to accommodate for URL parameters propagating up to the parent page.

Do not approve yet - build pipeline in Azure needs to be changed.


Bump CoreOS stable to latest version

1298.6.0 fixes some sporadic network issues. It also includes docker
1.12.6 which includes several stability fixes for kubernetes.
Signed-off-by: Sergii Golovatiuk<EMAIL_ADDRESS>

ci check this


Fix #508 (TypeError: Cannot read property 'getRecord' of null)

This resolves #508.
Since the issue is related to using firebase adapter alongside other adapters within one Ember application, this change adds a new model to the test dummy app (a widget model), that uses JSONAPIAdapter.
The stub data is hard-coded in the adapter. I chose to do this over using something like ember-cli-mirage, as it doesn't seem like this project needs much more of this kind of thing and so ember-cli-mirage feels like overkill for the app. However, I'm happy to try a different approach to testing this if desired.

I signed it.

Hi, the test suite is passing locally for me. I'm not sure why it's failing on CI. It looks like an issue with npm...

UPDATE: It looks like the CI issue is resolved by using yarn instead of npm. ember-try uses yarn if yarn it is installed, and travis automatically installs yarn if a yarn.lock file is present. In my EXPERIMENT commit, I just added an empty yarn file. I'll drum up a new PR that adds a full yarn.lock file...

Thanks so much @jamesdaniels! Will there be a release so we can use the official version instead of our own branch?

We'll cut a release in the next couple days

Excellent. Thanks!


HeaderPropagation: propagate incoming request headers to outgoing HTTP requests

This PR introduces a MessageHandler for the HttpClientFactory to forward headers received in the current MVC request.
It is a port of https://github.com/aspnet/Extensions/pull/1099 which
Addresses https://github.com/aspnet/Extensions/issues/526
Main feedback on the original PR
@natemcmaster : it can't be added to https://github.com/aspnet/Extensions as it introduce a circulr dependency
@rynowak has suggested to move this to this repo as middleware.
@Tratcher it is not handling concurrency in accessing the Context if multiple HttpClient requests are sent in parallel
@Tratcher @rynowak it should not need to be based on the HttpClientFactory but it could be a DelegatingHandler and a Middleware to initialize the state.
Current Plan

Collect feedback on this PR from relevant stake holders (should I open a issue as well?)
In the meantime, I'll work on the DelegatingHandler with a initializer middleware, locking the access to the Context if needed, similar to https://github.com/Microsoft/ApplicationInsights-aspnetcore/issues/373

Original description
I believe it is a common use case which deserves to be included in the Extensions namespace, so I'd like to help to land this feature. Its main use case is probably to track distributed transaction which requires the ability to pass through a transaction identifier as well as generating a new one when not present.
The code is loosely based on this gist from @davidfowl with some extra features and a different behaviour as, by default, this PR doesn't add the header if it's already present in the HttpRequestMessage.
Features:

Forward header from current request MVC context to outgoing HttpClient requests, allowing to use a different header name. The header is forwarded only if present and all values are used.
Always forward the header to the HttpRequestMessage (AlwaysAdd=true) or only when the header is not already present (default).
Define a default value(s) to be added to the HttpRequestMessage if the incoming request does not contain the header
Define a default value(s) generator to be used to add the header value to the HttpRequestMessage if the incoming request does not contain the header. The generator can rely on the HttpRequestMessage or HttpContext to decide the value. (note: in my mind the use case is to generate a new GUID/identifier for a transaction chain if it's not in the incoming request, I'm not sure if having access to the request and context add any value)

The tests should clarify the behaviour if needed.
It definitely needs some polishing on the definition of the configuration as well as probably removing some of the different behaviours, however I believe opening a PR is the best way to move forward the discussion and shape the behaviour.
Probably the discussion on the requirements should stay on the issue to keep the focus of the discussion here on the implementation.
Please Note:

I didn't have explicit feedback to open the PR following my comment on the issue, however the issue if marked as up-for-grabs
Part of this PR is based on this gist from @davidfowl , I'm not sure what is the correct way to give attribution.

I really appreciate you taking the time to review this PR.
Thanks,
Alessio

I've introduced the Middleware to solve the concurrency issue.
It's also not tightly couple with the HttpClientFactory, although it still include an extension method to easily add the DelegeatingHandler to the client setup in the factory, as an opt-in for each client.
The configuration, however, is shared for all the clients. This is to simplify the behavior of the middleware, as it needs to know all the possible headers to be forwarded.
The Middleware take the input headers and put those in the state with the name defined as output, eventually generating/using the default value when not present.
The DelegatingHandler simply add the headers from the state to the outgoing HttpRequestMessage headers, eventually skipping the one already present based on the value of AlwaysAdd.
I'm not convinced about the configuration... the behavior it's probably not really straightforward and can be simplified.

@stevejgordon HYPE

Wasn't there some earlier discussion about collecting many headers up front but choosing in the outgoing handler which headers applied to that scenario? Either wanting to customize it per outgoing request or at least being able to have multiple handlers with different outgoing headers?
The current design frontloads everything in the collecting middleware, there doesn't seem to be a way to customize it later.

Really glad this is getting worked, thanks @alefranz ! Is this going to be available in 2.1/2.2, or just 3.0? Is there a way to access the value of a generated header when handling the incoming request (like if you want to log it/stash it somewhere)?

Is there going to be a way to wire up one of the incoming (or generated if missing) headers to be the TraceIdentifier? At a higher level, is the type of functionality currently available in something like @stevejgordon's https://github.com/stevejgordon/CorrelationId package going to be in scope here (or elsewhere)? It seems like a natural fit. Thanks everyone!

Apologies everyone for the lack of progress. I'll get back to this tonight.
@rynowak @Tratcher I've really appreciated your valuable feedback. I will address it.
@ericsampson yes, a scenario like the one from the library is already supported in the current form. You will be able to setup the middleware to capture the TraceIdentifier and have it automatically added to outgoing request. A GUID for new requests can then be generated using the DefaultValuesGenerator. Am i missing something else?
@Tratcher yes it has been mentioned, and yes I believe it should be the case. However I wasn't sure on the best approach to handle the configuration for it and I would like to know your point of view. As the configuration object will be one, it means it can be customized for each client that use AddHeaderPropagation, and the overload with an action to "Configure" is not the correct approach as it will modify the shared configuration. Should I provide an overload to accept a configuration object to use for the particular client/handler (eventually with a configurationbuilder to simplify it)? However that would mean it would be only possible to configure it in code, not through the standard configuration infrastructure. Is there a different pattern to follow for this scenario?
Also as it would be nice to keep the configuration at minimum and avoid possible errors. Setting up the headers in the middleware and then in the handler specify which ones to use is probably the easier solution but offer a bit of duplication, which maybe can be alleviated making the default behaviour to pass all the headers, so the "allow list" is opt-in. The alternative approach I wanted in the first place was was to define the list of headers when setting up the clients and then having the middleware pulling all this configurations to decide which headers to collect. This allow to specify the config in the client/handler, where it feels more natural, as well as defining different outgoing header names per client however it makes the implementation less clean and it creates ambiguity on the defaultvalues generation (per outgoing request or shared). Sorry I'm just thinking out loud, not sure if it makes sense.
@rynowak about AlwaysAdd, in the current implementation it's a awkward and not really fulfilling it's purpose. I need to rethink that. The scenario I had in mind was to track a chain of requests (so adding a new id to the list of incoming Ids) or the tracestate in the W3C trace-context draft spec where you have to chain with identifier other vendors identifiers. I think a better place to handle this would be in the middleware to have something to control the behavior to generate a new value even when one is in the incoming requests, as I believe in this scenarios it is a better approach to generate a new id that will be used by all the outgoing HTTP calls as part of that requests. Anyway this will not necessary be the desired behavior for everyone, as well as there are other aspects, like if the new value should be appended or put in front, as well if it should be multiple heaver values or a single value comma separated (as it is required for tracestate). Probably a flexible solution to handle all the possible scenario will increase complexity without bringing much values, so I'm tempted to leave this out for now (as it can eventually be added later as a non breaking change) eventually adding some extensibility points to be able to inject a behavior for the specific scenario. What do you think?

@rynowak do you have a pattern for using named options with handlers added to the HttpClientFactory?

@alefranz , the CorrelationId library has the ability to overwrite the default TraceIdentifier with the value of a given header from the incoming message, that's what I was wondering about. Also, I'm still curious if it's going to be possible to access the value from the DefaultValuesGenerator in the controller that handled the incoming message?

Wasn't there some earlier discussion about collecting many headers up front but choosing in the outgoing handler which headers applied to that scenario? Either wanting to customize it per outgoing request or at least being able to have multiple handlers with different outgoing headers?

Does it make sense to try and lock down what's currently in this PR and then send another to consider scenarios like this?

do you have a pattern for using named options with handlers added to the HttpClientFactory?

We don't have a specific pattern, but you get access to services when creating a handler, so things like this can be built easily.

from my side, I'm happy to try to get this PR locked down and then raise it a new one to add the ability to specify the headers to include on a per client basis.

@alefranz - sorry for the delay on this. It's MVP summit this week so the team is pretty busy. I'll take a look at this again in detail as soon as I can.

@ryonwak thank you for the update. Enjoy the summit :)

FYI - you're running into this build error:

One or more errors occurred. (One or more errors occurred. (The process cannot access the file 'E:\A_work\1\s\src\Components\test\E2ETest\node_modules\selenium-standalone.selenium\selenium-server\3.141.5-server.jar' because it is being used by another process.)) (The process cannot access the file 'E:\A_work\1\s\src\Components\test\E2ETest\node_modules\selenium-standalone.selenium\selenium-server\3.141.5-server.jar' because it is being used by another process.)

That's unrelated to your changes, and can be fixed by rebasing on origin/master

I'm going through and looking at minor details like doc comments and such now, hopefully this is the last round 👍

Hi @rynowak , thank you for the review and apologies for the delay on my side :)
I should have addressed the feedback.
Let me know what you think about the documentation, specifically the parts I skipped (as it doesn't seems common practice looking at other projects in the solution) and the attempt to explain the logic at https://github.com/aspnet/AspNetCore/pull/7921/files#diff-ceea95213282add8708d3dc7be38d089.
I'll have a look tomorrow at why it's not passing on the CI

Looks good from my POV - I want to get a :shipit: from @dougbu and @anurse before we merge this bad boy!

@alefranz - could you rebase? after that I think we're ready to go.

Thanks everyone!

ci tests on windows are failing, but it looks unrelated
 [xUnit.net 00:04:51.46]     Templates.Test.RazorPagesTemplateTest.RazorPagesTemplate_IndividualAuthImplAsync(useLocalDB: True) [FAIL]
  Failed   Templates.Test.RazorPagesTemplateTest.RazorPagesTemplate_IndividualAuthImplAsync(useLocalDB: True)
  Error Message:
   System.Net.Http.HttpRequestException : The SSL connection could not be established, see inner exception.
  ---- System.IO.IOException : Authentication failed because the remote party has closed the transport stream.

@aspnet-hello triage https://dev.azure.com/dnceng/public/_build/results?buildId=137434

This comment was made automatically. If there is a problem contact ryanbrandenburg.
I've triaged the above build.

Merged ! Thanks @alefranz  and everyone else who helped review.

@rynowak What is the policy about back-porting this kind of packages to 2.2?

@alefranz - we don't backport features to a,ready shipped releases.
If you want a version of this you can use with 2.2, I would suggest grabbing the code and building it from source - it should be straightforward since this package doesn't have tendrils anywhere else.. @Eilon will correct me if I'm wrong but our license allows you to copy as much code as you want (with attribution).

Yup, what @rynowak said.

@rynowak I didn't get a chance to review the entire changeset closely, but is it implemented to reduce allocations as much as practical?

Nice work @alefranz :)

I didn't get a chance to review the entire changeset closely, but is it implemented to reduce allocations as much as practical?

Our work is never over. If you think it can be improved, collect some data and send a PR 👍

I didn't get a chance to review the entire changeset closely, but is it implemented to reduce allocations as much as practical?

Our work is never over. If you think it can be improved, collect some data and send a PR 👍

👍 Maybe I'll see if I can nerd-snipe @benaadams into taking a peek first :) Cheers all


"Remove WikiLinks" doesn't work

The Remove WikiLinks format in selection command has no effect.
I tested this issue in a fresh Sandbox vault with only obsidian-text-format installed. Preview mode as well as Source mode.
Original text:
**Ginger beer** is a traditional fermented drink that can be made as either a soft drink or an alcoholic drink. Make it in a carboy with more sugar or [malt extract](https://en.wikibooks.org/w/index.php?title=Cookbook:Malt%5FExtract&action=edit&redlink=1 "Cookbook:Malt Extract (does not exist)") and with longer fermentation time to get alcoholic ginger beer.

Debug info:
SYSTEM INFO:
	Obsidian version: v1.4.16
	Installer version: v1.4.16
	Operating system: Windows 10 Home 10.0.22621
	Login status: not logged in
	Insider build toggle: off
	Live preview: on
	Legacy editor: off
	Base theme: dark
	Community theme: none
	Snippets enabled: 0
	Restricted mode: off
	Plugins installed: 1
	Plugins enabled: 1
		1: Text Format v2.2.8

Close as the issue is inactive for a long time. Feel free to re-open this issue if the problem is not solved.


ionic 3

Hi
Thank you for your great project!
Is there some version of your project available to ionic 3?
Thank you very much.

Hi @marciborba , with pleasure.
Currently there is no version aside to Ionic 1, sorry.


Public Access Live Test Failures

Description

/azp run java - storage - tests

API change check
API changes are not detected in this pull request.

/azp run java - storage - tests

API change check
API changes are not detected in this pull request.

/azp run java - storage - tests

API change check
API changes are not detected in this pull request.

/azp run java - storage - tests

API change check
API changes are not detected in this pull request.

/azp run java - storage - tests

API change check
API changes are not detected in this pull request.

API change check
API changes are not detected in this pull request.

API change check
API changes are not detected in this pull request.

/azp run java - storage - tests

/azp run java - storage - tests

/azp run java - storage - tests

/check-enforcer override

Overriding since the tests related to public access are passing now, other test failures are not related.


conent bleading issue when using bootstrap card-columns CSS class

Hi Bootstrap team,
We have facing some strange scenario while using bootstrap card-columns CSS class. Here, we have rendered our scrollable data table in one bootstrap card-column class then its working fine. But we have faced the bleeding content issue (in all Grid except first one), when we render the same table in more than one inside bootstrap card-columns.
Here, we have attached sample and screenshot for your reference for your reference.
Sample: https://codepen.io/Rahul23/pen/NZZbpV
ScreenShot;

Note: The reported problem is occurred in Chrome browser
Could you please check this and update further details.
Regards,
Rahul

CSS columns are super buggy across browsers. Please change to using the Bootstrap grid and you should have fewer issues.


ISPN-5484 RpcManager.invokeRemotelyInFuture should not use a transport thread

https://issues.jboss.org/browse/ISPN-5484
Includes https://github.com/infinispan/infinispan/pull/3493

Haven't had a chance to look it all over, but looks promising so far :+1:
There are some test failures currently http://ci.infinispan.org/viewLog.html?buildId=27110&buildTypeId=bt9&guest=1
Also I noticed quite a few classes say since 7.0, we should update that to 8.0

I should have used the Preview label, I didn't have the patience to run all the tests on my machine :)

I was wrong, the core testsuite was also failing because of the way I had integrated CompletableFutures into the distributed executor (which must return NotifyingFutures). Everything should be ok now, the 3 tests that are failing in CI have had random failures for a while.

Needs rebasing

The more I am thinking about this I wonder if we should be using different abstractions based on the usage.  It seems that CompletableFuture handles both the sync and async cases.  I wonder if instead we should separate these usages to the sync (Future) and async (CompletableStage) in our RpcManager/Transport code.  This way the code is clearly defining its expectations by invoking a specific method.  Right now the underlying code has to possibly work with any usage since the RpcOptions defines whether or not it is async really or not.
I am just trying to think this out, maybe I am over complicating things, but wanted to put this out there.

@wburns Do you know of anyone using CompletionStage independently? I'm not at all convinced that having to worry about how your clients will use your return value is a good idea.

I am not saying we have to do something, I am just throwing this out here so we give it at least half of a thought :)  As I mentioned I am probably just over thinking it, but wanted to make sure we thought about it as well.  I am fine with just sticking with CompletableFuture.  It just seems odd that they would add an interface that no one uses :P

Seems okay to me, if you just want to update the 1 comment still outstanding.

Yeah, you may be over-thinking it :) Considering that you can always call completionStage.toCompletableFuture.get(), I really have no idea why CompletionStage doesn't extend Future.

Pulling...

Integrated into master, thanks @danberindei !

Well I think normally a CompletionStage would be used with something like whenComplete etc.  Otherwise as you said it is basically a Future.


Keep the same img name

It would be nice if the downloaded image kept the same name as the original image. I tried to do it myself but I'm too noob

@OneEyeMekhet
I can do this but this but this is not that much needed. so my suggestion is to use your own image name.


java 1.7 supports

hi  why this project not supports java 1.7?

This project was created to support the ea-agent-loader project which requires Java 8 and as such J as Java 7 support isn't required for our use cases.
I'd be happy to review a pull request that introduced Java 7 support but we currently have no plans to do this work ourselves.
Hope that answers your question.


Fixes #844. The SelectionChanged action it's now working correctly.

Did a checkout of this PR and things seem to behave like expected now. In folder chooser mode, picking .. still picks the same viewed directory. If nothing is selected and you press open, it also chooses the current directory. If you select a folder and hit open, that folder is returned. So now I can select current directory or subdirectory in a UI consistent manner 👍

In folder chooser mode, picking .. still picks the same viewed directory when returned on FilePath

FilePath always return the current path plus the selected items. To check what it's returned from OpenDialog always use the FilePaths for both, folder(s) and file(s). If nothing is selected FilePath return the current folder and FilePaths return a empty list.
FYI since I implemented the wheel mouse feature into GUI it's possible to select many itens at once by clicking the first item and using the Shift key with the mouse select the last item and the items status will be toggled, thus marrking or unmarking dependent the previous item status.


If we add an array element through add button, after deleting any of the element in the array, empty object is added instead of adding the object with its properties

Schema used :
{
"title": "Person",
"type": "object",
"properties": {
"Samples": {
"type": "array",
"title": "Pets",
"items": {
"type": "object",
"title": "Pet",
"properties": {
"type": {
"type": "string"
},
"name": {
"type": "string"
}
}
}
}
}
}
In the above schema I have added the array pets. in the json there is no format(table , tabs) specified
When i tap the "Add Pet" in the UI it adds the array element perfectly. if i tap "Add Pet", after deleting one element from the pets array then the empty object is added without having any properties. In the following resulted json we can see the empty object in 3rd and 5th element of the array
Repro:

add three elements by tapping add pet button
delete the second one
tap again the add pet button
we can see the array element added with empty object that doesnt have any properties
Resulted JSON:
{
"Samples": [
{
"type": "",
"name": ""
},
{
"type": "",
"name": ""
},
{},
{
"type": "",
"name": ""
},
{
"type": "",
"name": ""
},
{}
]
}

The following are the links to the schema for reproducing the issue:
http://jeremydorn.com/json-editor/?schema=N4IgLglmA2CmIC4QAVYCcDOB7AdiANOAJ4AO8SWARgFawDGYBIJaWZaksGioAygIYBbEnG4JQYUuRD80afkSaQY01GG6EosQWIlTEIKrQZKocA2qYs26TruJkeD6RjBoIOAOYgAvoRxC5HqOSK7uXr4+UVFAAA==&value=N4IgyghgtgDgNgUwM4gFwG1QBcCeMFoggA0IAdtAakQL7HZ5VGkVRMh3Cci76EnlK/br3Ysh1DvRoBdGkAA=&theme=bootstrap2&iconlib=fontawesome4&object_layout=normal&show_errors=interaction
http://jeremydorn.com/json-editor/?schema=N4IgLglmA2CmIC4QAVYCcDOB7AdiANOAJ4AO8SWARgFawDGYBIJaWZaksGioAygIYBbEnG4JQYUuRD80afkSaQY01GG6EosQWIlTEIKrQZKocA2qYs26TruJkeD6RjBoIOAOYgAvoRxC5HqOSK7uXr4+UVFAAA==&value=N4IgyghgtgDgNgUwM4gFwG1QBcCeMFoggA0IAdtAakQL7HZ5VGkVRMh3Cci76EnlK/br3Ysh1DvRoBdGkAA=&theme=bootstrap2&iconlib=fontawesome4&object_layout=normal&show_errors=interaction&disable_properties

I'm seeing exactly the same problem, checking getValue() the array item still has got some data in it (from the template I think), and is identical to a 'correct' new array item, but will not display correctly, and there is no way to fix it apart from create additional items and then delete the broken one(s)

Just to confirm #543 completely fixes this for me

Actually this is creating messy tony. sometimes even the added second element is also empty. We can't ask the user to add as many items until it gets the eleents displayed

#543 is fixed for me as well. #547 is working fine. Thank you for providing the fix..


[test] Remove extra -serialize-diagnostics

The -serialize-diagnostics-path option implies -serialize-diagnostics; there's no point in specifying both. Remove the extra flags from the tests.

@swift-ci Please smoke test and merge

Thanks, @jrose-apple! By the way, do you have any thoughts on SR-2683?


Sign hashids created with hashids to prevent decoding ambigious ids

There are some cases where a standard integer can be decoded as a hashid. A solution to this would be to encode hashids as an array. This is already built in functionality for Hashids (https://github.com/peterhellberg/hashids.rb#encoding-several-numbers). A constant integer (72 = H in ascii) could be set in the first array position and used as a check, then the second would be the actual integer. If the array is not two positions, or if the first position in the array is not 72, then we know the hashid should not be decoded.
HASHID_KEY = 72
#=> 72
id = 518
#=> 518
hashids.encode(HASHID_KEY, id)
#=> "gB0NV05e"
decoded = hashids.decode("gB0NV05e")
#=> [72, 518]
decoded.first == HASHID_KEY && decoded.last == id
#=> true

This has been resolved in the latest release.


Added a Neural Network as an option for image comparison

This change probably decrease the margin error in the agregator

wow..
ok thank you, i will check this out in more detail soon.

First off, I would like to thank you for this contribution. A lot of this stuff is currently out of my scope, as I'm still expanding my knowledge in the area. It's been my goal to expand this project once I had a better understanding.
I'm very interested in the way these neural networks are trained, I'm still regularly reading that book you recommended. Would it be possible to provide the structure for the testing/training dataset? This would really help in my understanding, and also enable me to develop this project further, as the game changes over time.

No problem, I will provide the initial dataset later with a small script that adjusts it to fit the model.


cygwin drops x86 support, script changes required

script no longer runs properly on x86 systems due to cygwin starting to drop x86 support:

https://cygwin.com/pipermail/cygwin/2022-November/252542.html
http://www.crouchingtigerhiddenfruitbat.org/Cygwin/timemachine.html

the current version of cygwin errors out on x86 systems by default. to get it going on 32-bit windows 7, cygwin mirror in line 47 needs to be set to http://ctm.crouchingtigerhiddenfruitbat.org/pub/cygwin/circa/2022/11/23/063457 and the --allow-unsupported-windows option parameter needs to be added to line 123.
the script also needs to be altered to fetch cygwin 3.3.6 (setup 2.924) on x86 systems, as that version is the last one that's going to support x86 with those extra parameters.

To be honest I don't think I want to do this in the main script. I have no particular interest in supporting x86 as this project is targeted to x86_64 anyway. The first line of the read is

This is a deployment tool that allows you to install a portable and configured Cygwin x64 installation to use with the LFTP4WIN-CORE solution.

It would be better if you forked the script it and made the changes and/or i'll just pin the issue so others can made the required modifications themselves.

ok done: https://github.com/glodomorrapl/LFTP4WIN-LEGACY
i am clueless about github or software licensing in general, if there is anything i need to change, please let me know and i'll comply

You just add your credits above mine here to be inline with the Apache license.
https://github.com/glodomorrapl/LFTP4WIN-LEGACY/blob/master/LFTP4WIN-installer.cmd#L8
Like i did above the original script credits.
https://github.com/glodomorrapl/LFTP4WIN-LEGACY/blob/master/LFTP4WIN-installer.cmd#L12
That's it really.


Initialize with apollo-server, MongoDB, Firebase

Upgrade node to LTS version
Migrate from npm to yarn
Initialize apollo-server-express
Initialize MongoDB, Mongoose, and Firebase Admin SDKs

Check ESLint check errors
In imports you have written .js at end of file names, that is against our ESLint rules.
console is also throwing error, can you make it winston log? We will use winston to log to console now. We will make it console to file later
Also, Cloudinary?

Check ESLint check errors
In imports you have written .js at end of file names, that is against our ESLint rules.
console is also throwing error, can you make it winston log? We will use winston to log to console now. We will make it console to file later
Also, Cloudinary?

Also, send me service account

Also, send me service account

Add a .env.example file

Add a .env.example file

Check ESLint check errors
In imports you have written .js at end of file names, that is against our ESLint rules.
console is also throwing error, can you make it winston log? We will use winston to log to console now. We will make it console to file later
Also, Cloudinary?

Okay so this is related to imports in node.js. As of now node.js requires the extension of the file also to import it. Without the extension it throws an error, module not found.

Check ESLint check errors
In imports you have written .js at end of file names, that is against our ESLint rules.
console is also throwing error, can you make it winston log? We will use winston to log to console now. We will make it console to file later
Also, Cloudinary?

Okay so this is related to imports in node.js. As of now node.js requires the extension of the file also to import it. Without the extension it throws an error, module not found.

Check ESLint check errors
In imports you have written .js at end of file names, that is against our ESLint rules.
console is also throwing error, can you make it winston log? We will use winston to log to console now. We will make it console to file later
Also, Cloudinary?

Okay so this is related to imports in node.js. As of now node.js requires the extension of the file also to import it. Without the extension it throws an error, module not found.

Then modify the ESLInt config

Check ESLint check errors
In imports you have written .js at end of file names, that is against our ESLint rules.
console is also throwing error, can you make it winston log? We will use winston to log to console now. We will make it console to file later
Also, Cloudinary?

Okay so this is related to imports in node.js. As of now node.js requires the extension of the file also to import it. Without the extension it throws an error, module not found.

Then modify the ESLInt config

@riteshsp2000 have given you merge rights, either squash merge when ready or ping me here

@riteshsp2000 have given you merge rights, either squash merge when ready or ping me here

@DesignrKnight  all SDKs are initialized, could you check once before merging

@DesignrKnight  all SDKs are initialized, could you check once before merging


404 at /registry/spec/api/

I found a broken link : https://docs.docker.com/registry/spec/api/
I arrived on this page through: https://www.google.com/

Thanks for reporting! Looks like this was a temporary glitch; we published the docs again on docs.docker.com, and the URL now works


Added python script and gitignore

This is so I can have the python_analysis file in the git without it causing conflicts in other branches

DO NOT DELETE LA BRANCHE
-David


Wrong button internal padding in cards

Bug, feature request, or proposal:
Bug
What is the expected behavior?
The md-button text should be aligned with the md-card-title, md-card-content, md-card-subtitle, etc. According to the specs, buttons in dialogs and cards have 8dp internal padding.
https://material.io/guidelines/components/cards.html#cards-content-blocks
https://material.io/guidelines/components/buttons.html#buttons-style

What is the current behavior?
padding: 0 16px

What are the steps to reproduce?
<md-card>
  <md-card-title-group>
    <md-card-title>Shiba Inu</md-card-title>
    <md-card-subtitle>Dog Breed</md-card-subtitle>
  </md-card-title-group>
  <md-card-actions>
    <button md-button>LIKE</button>
    <button md-button>SHARE</button>
  </md-card-actions>
</md-card>

What is the use-case or motivation for changing an existing behavior?
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
https://github.com/angular/material2/commit/de68202971aa5a1ce7e36d3e2c2fc26c6cdc4314
Is there anything else we should know?
Need testing when the button is aligned to the right.
Some Material Specs examples are using the same button texts with different margin and padding (first two examples). I can't say witch one is right!

Related to https://github.com/angular/material2/issues/4245, https://github.com/angular/material2/issues/3528

@crisbeto is there anything here not captured by those other two issues?

It looks like it would be covered by https://github.com/angular/material2/issues/4245. I'll close it as a dupe.


Bump paas-admin from v0.97 to v0.104

What
See https://github.com/alphagov/paas-admin/compare/v0.97.0...v0.104.0
This includes these PRs by humans :meat_on_bone:

alphagov/paas-admin#188 [#164493738] Remove currency rate and fix code to GBP

And these PRs by robots :space_invader:

alphagov/paas-admin#183 Bump csurf from 1.9.0 to 1.10.0
alphagov/paas-admin#184 Bump source-map-support from 0.5.8 to 0.5.12
alphagov/paas-admin#187 Bump helmet from 3.13.0 to 3.16.0
alphagov/paas-admin#190 Bump @types/nock from 9.3.0 to 10.0.1
alphagov/paas-admin#191 Bump @types/route-parser from 0.1.1 to 0.1.2
alphagov/paas-admin#192 Bump tslint-consistent-codestyle from 1.13.3 to 1.15.1
alphagov/paas-admin#185 Bump showdown from 1.8.6 to 1.9.0
alphagov/paas-admin#193 Bump passport-oauth2 from 1.4.0 to 1.5.0

How to review

Check the diff
Confirm this has run in a dev environment (will be running down towers)

Who can review
Not @richardtowers

This has (mostly) made its way down https://deployer.towers.dev.cloudpipeline.digital/teams/main/pipelines/create-cloudfoundry now - just waiting for the acceptance tests. Once that's done I'll get someone to review and we can :shipit: this.


Add support for self-signed SSL certificates

Self-signed certificates are very common in developer environments. Network calls are done using static function on HTTP struct and with URLSession.shared which silently ignores auth challenges and simply leads to failure branch.

Moving discussion to #21


docs: Setup directory structure

Structure can be found at https://github.com/CosmWasm/docs/issues/212

Docs structure is done, can close the issue


Test fork PR Workflow

Testing Fork GitHub Actions workflow

Closing, need a lower privilege account for this test.


fix: Fixed issue where no named elements would cause a compiled class that does not compile

Description
Fixed issue where no named elements would cause a compiled class that does not compile
Ticket(s)
https://tactileentertainment.atlassian.net/browse/USM-1471

This pull request is included in version 1.7.1


ANC Progress Report - Medical/Obstetric Diagnosis Formatting [2h]

Issues

Patient = Dukuze Soline - Rukura Health Center (Nutrition environment)

Formatting should be consistent with the smaller font and the dash list separator (also review spacing so it looks like one list)
Diagnoses should have a date they were recorded
Diagnosis should not appear twice, if they were diagnosed twice there should be two dates in order of earliest to latest.
(not indicated) Assuming that "underweight" were diagnosed at different times, if not the more severe case should be shown. Otherwise there should be a diagnosis date for clarity.

Tasks

Fix the format changes indicated above

@adamhstewart
For the context, what we have on the bottom of Medical and Obstetrical panes are not diagnoses.
These are alerts, that we developed for initial ANC, which was only for CHW.
Never got an answer on what to do with them when nurse views the report, so they remained there.
This was mentioned at https://github.com/TIP-Global-Health/eheza-app/issues/353#issuecomment-1250215829 already, and now I think maybe we should be hiding them for nurses?
Some of these alerts are duplicated of proper diagnoses. For example, Pregnancy induced hypertension, as @balagan73 found at point 1 of https://github.com/TIP-Global-Health/eheza-app/issues/340#issuecomment-1253308729 and 'High risk of Preeclampsia' @balagan73 found at https://github.com/TIP-Global-Health/eheza-app/issues/353#issuecomment-1207722371.

Formatting should be consistent with the smaller font and the dash list separator

Actually, at CSS, there's not dash separator on list items. We have that dash, because it's a <p> whith text prefixed with dash.

So, seems to me that fix should be to implement this as real list ul with li items.
The font will be larger, similar to font at other types of progress reports.

(also review spacing so it looks like one list)

@adamhstewart Depoyed to Nutrition env for QA


Add Swift packaging

Can the Protocols and Utilities folders be bundled as something that can be added as a dependency to other projects?

Solved with the commits created today.


"Edit about me" button is in the wrong spot

It's right beside "shared". Shouldn't it be to the right of "about"? Also, you can see that the bottom text of "edit about me" is just above the top of "shared". I would think that the (vertical) middle of both texts would be on the same line.

Screenshot pls? Perhaps what you think is the right spot?

(tfw enter submits issue)

Haven't finished writing it lol.

Okay, finished writing! Thanks GitHub.

I'm not even sure we should have "Edit xxx" buttons in the first place.
We have the style scratch uses already - a textarea.

Yeah, the button just adds clutter.

And they're a pain to style for. +1 for removing it

+1 for removing it

Remove it is

Done


In the dockerfile, the port is mapped to 34651. However, the message in terminal still shows 8888. This may confuse people who do not carefully read instructions. Is there any way to resolve it?

After speaking with Daniel, we learned this was absolutely necessary for us to avoid conflicts with a local computer's 8888 port. Without mapping it to 34651 (a unique port that will likely not conflict with a common jupyter lab port like 8888), we will get conflicts when we have 8888 already busy.