[#208] Example code on spec website doesn't compile

This fixed #208

Make sure that:

[x] You have read the contribution guidelines.
[x] You have created a feature request first to discuss your contribution intent. Please reference the feature request ticket number in the pull request.
[x] You use the code formatters provided here and have them applied to your changes. Don't submit any formatting related changes.
[x] You submit test cases (unit or integration tests) that back your changes.

Issue description

New Public APIs

Additional context

Welp. I can't seem to sign off my commit if I do them directly from the github website :(

Alright. Sign-off has worked now

Thank you for your contribution. That's merged and backported now.


[FFmpegPostProcessor] Default of prefer ffmpeg

When no downloader is passed to FFmpegPostProcessor
an exception was raised trying to get the prefer ffmpeg param.
AttributeError: 'NoneType' object has no attribute 'params'

This fixes and defaults to False.

Thanks.


Body style attribut is not set properly

When trying to set <body> style attribute like this:
<Helmet>
  <body style={{background: 'black'}}/>
</Helmet>

It is actually set to <body style="[object Object]"/>
I figured out that I need to pass style attribute as a string to make it work:
<Helmet>
  <body style="background: black"/>
</Helmet>

This is working in the browser but for SSR I get this error:

The style prop expects a mapping from style properties to values, not a string.

Here is how I use helmet for SSR:
const Html = ({children}) => {
  const helmet = Helmet.rewind();
  return (
    <html lang="en" {...helmet.htmlAttributes.toComponent()}>
        <body {...helmet.bodyAttributes.toComponent()}>
          {children}
        </body>
    </html>
  )
}

Am I missing something?

I am having the very same problem with react-helmet 5.2.0.

@janjarfalk if it helps, here is the simple workaround I used:
const getBodyStyleAttribute = (
  backgroundUrl?: string,
  backgroundColor?: string,
) => isClient
  ? [
    backgroundUrl && `background-image: url(${backgroundUrl});`,
    backgroundColor && `background-color: ${backgroundColor};`,
  ].filter(s => s).join('')
  : {
    backgroundImage: backgroundUrl ? `url(${backgroundUrl})` : undefined,
    backgroundColor: backgroundColor || undefined,
  };

@janjarfalk @testerez I am using a similar solution; though more generic.
import kebabCase from 'lodash/kebabCase';

const getHelmetBodyStyleAttributeValue = (style) => {
  if (typeof window === 'undefined') {
    return style;
  }
  return Object.keys(style).map(key => `${kebabCase(key)}: ${style[key]}`).join(';');
};

It seems to work for everything  I have needed so far (though might fail on non standard things as it transforms keys to kebabCase)

Thanks for sharing @AubreyHewes what about having something similar built into react-helmet?

@testerez Good point, I will get a PR going ;-)

@AubreyHewes @testerez Any update on the PR? Looks stale

@matt-whitaker I never had any feedback on it.

@AubreyHewes thats unfortunate. Feels like a bug.

Still feels like a bug, era 2021.

I'm using react-helmet with Gatsbyjs. I couldn't understand @testerez and @AubreyHewes examples. I used useEffect to add style attribute to body:
useEffect(() => {
    document.body.setAttribute('style', 'background-color: red;');
}, []);


Automatic version update doesn't work as expected

Dependency resolution logic should attempt to complete the dependency graph as many times as needed. Seems like the current implementation stops after the first iteration.

This is fixed now.


feat: kadena provider changes

FYI raised two PRs against this https://github.com/enkryptcom/enKrypt/pull/382 and https://github.com/enkryptcom/enKrypt/pull/380


special characters (french...)

Hi,
Do you know how can I integrate special characters (for example, for french, éèàà ) ?
thank you

The unicode fonts only support certain symbols so you won't find variations that include those.
You can however try using Combining Characters to add accents to any symbol.
Eg:
◌̀ ◌́

These links will allow you to copy the accent symbol to your clipboard:
https://unicode-explorer.com/c/0300
https://unicode-explorer.com/c/0301
Then they can be pasted after any character to make it appear above.


docs(readme): add a note about the currently targeted ros version

humble branch has been setup to mirror main. As dev-image-ros2 and base-image-ros2 get bumped to future LTS versions, new branches will be added with a new mirror target


Hostname argument

Closes #440
The user can now pass a -hostname to server.py as an argument. Prior to this PR you could only set a HOSTNAME env variable. Adding this argument helps the user to understand they can set a custom hostname without going through the source code and spot the env variable lines.
How Has This Been Tested?
Tested in a conda env with python3.6 with the following commands:

python -m visdom.server --> original behvoiur (serves on http://localhost:8097)
python -m visdom.server -hostname <IP_ADDRESS>
python -m visdom.server -hostname localhost
python -m visdom.server -hostname localhost2 --> serves on localhost2 (?!)

Unfortunately I couldn't run demo.py. See the error trackback below. As suggested I tried pip install -e . or easy_install . but still didn't work.
Exception in user code:
------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/erotundo/miniconda3/envs/visdom_dev/lib/python3.6/site-packages/urllib3/connection.py", line 171, in _new_conn
    (self._dns_host, self.port), self.timeout, **extra_kw)
  File "/Users/erotundo/miniconda3/envs/visdom_dev/lib/python3.6/site-packages/urllib3/util/connection.py", line 79, in create_connection
    raise err
  File "/Users/erotundo/miniconda3/envs/visdom_dev/lib/python3.6/site-packages/urllib3/util/connection.py", line 69, in create_connection
    sock.connect(sa)
ConnectionRefusedError: [Errno 61] Connection refused

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/Users/erotundo/miniconda3/envs/visdom_dev/lib/python3.6/site-packages/urllib3/connectionpool.py", line 600, in urlopen
    chunked=chunked)
  File "/Users/erotundo/miniconda3/envs/visdom_dev/lib/python3.6/site-packages/urllib3/connectionpool.py", line 354, in _make_request
    conn.request(method, url, **httplib_request_kw)
  File "/Users/erotundo/miniconda3/envs/visdom_dev/lib/python3.6/http/client.py", line 1239, in request
    self._send_request(method, url, body, headers, encode_chunked)
  File "/Users/erotundo/miniconda3/envs/visdom_dev/lib/python3.6/http/client.py", line 1285, in _send_request
    self.endheaders(body, encode_chunked=encode_chunked)
  File "/Users/erotundo/miniconda3/envs/visdom_dev/lib/python3.6/http/client.py", line 1234, in endheaders
    self._send_output(message_body, encode_chunked=encode_chunked)
  File "/Users/erotundo/miniconda3/envs/visdom_dev/lib/python3.6/http/client.py", line 1026, in _send_output
    self.send(msg)
  File "/Users/erotundo/miniconda3/envs/visdom_dev/lib/python3.6/http/client.py", line 964, in send
    self.connect()
  File "/Users/erotundo/miniconda3/envs/visdom_dev/lib/python3.6/site-packages/urllib3/connection.py", line 196, in connect
    conn = self._new_conn()
  File "/Users/erotundo/miniconda3/envs/visdom_dev/lib/python3.6/site-packages/urllib3/connection.py", line 180, in _new_conn
    self, "Failed to establish a new connection: %s" % e)
urllib3.exceptions.NewConnectionError: <urllib3.connection.HTTPConnection object at 0x10f9a67b8>: Failed to establish a new connection: [Errno 61] Connection refused

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/Users/erotundo/miniconda3/envs/visdom_dev/lib/python3.6/site-packages/requests/adapters.py", line 445, in send
    timeout=timeout
  File "/Users/erotundo/miniconda3/envs/visdom_dev/lib/python3.6/site-packages/urllib3/connectionpool.py", line 638, in urlopen
    _stacktrace=sys.exc_info()[2])
  File "/Users/erotundo/miniconda3/envs/visdom_dev/lib/python3.6/site-packages/urllib3/util/retry.py", line 398, in increment
    raise MaxRetryError(_pool, url, error or ResponseError(cause))
urllib3.exceptions.MaxRetryError: HTTPConnectionPool(host='localhost', port=8097): Max retries exceeded with url: /env/main (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x10f9a67b8>: Failed to establish a new connection: [Errno 61] Connection refused',))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/Users/erotundo/git/visdom/py/visdom/__init__.py", line 446, in _send
    data=json.dumps(msg),
  File "/Users/erotundo/miniconda3/envs/visdom_dev/lib/python3.6/site-packages/requests/api.py", line 112, in post
    return request('post', url, data=data, json=json, **kwargs)
  File "/Users/erotundo/miniconda3/envs/visdom_dev/lib/python3.6/site-packages/requests/api.py", line 58, in request
    return session.request(method=method, url=url, **kwargs)
  File "/Users/erotundo/miniconda3/envs/visdom_dev/lib/python3.6/site-packages/requests/sessions.py", line 512, in request
    resp = self.send(prep, **send_kwargs)
  File "/Users/erotundo/miniconda3/envs/visdom_dev/lib/python3.6/site-packages/requests/sessions.py", line 622, in send
    r = adapter.send(request, **kwargs)
  File "/Users/erotundo/miniconda3/envs/visdom_dev/lib/python3.6/site-packages/requests/adapters.py", line 513, in send
    raise ConnectionError(e, request=request)
requests.exceptions.ConnectionError: HTTPConnectionPool(host='localhost', port=8097): Max retries exceeded with url: /env/main (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x10f9a67b8>: Failed to establish a new connection: [Errno 61] Connection refused',))
The visdom experienced an exception while running: AssertionError('No connection could be formed quickly',)
The demo displays up-to-date functionality with the GitHub version, which may not yet be pushed to pip. Please upgrade using `pip install -e .` or `easy_install .`
If this does not resolve the problem, please open an issue on our GitHub.

Types of changes

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

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[ ] For JavaScript changes, I have re-generated the minified JavaScript code.

Hi @enricorotundo, thanks for the contribution! Can you add the same argument to the demo.py script and then pass it into the visdom instance in there? That should get the demo to work.

I'm running my local server with python -m visdom.server in a terminal and installed the local visdom with pip install -e . (in the same conda env) but python example/demo.py fails with the following message:
'>' not supported between instances of 'float' and 'NoneType'
'>' not supported between instances of 'float' and 'NoneType'
'>' not supported between instances of 'float' and 'NoneType'
Visdom python client failed to establish socket to get messages from the server. This feature is optional and can be disabled by initializing Visdom with `use_incoming_socket=False`, which will prevent waiting for this request to timeout.
The visdom experienced an exception while running: AssertionError('No connection could be formed quickly',)
The demo displays up-to-date functionality with the GitHub version, which may not yet be pushed to pip. Please upgrade using `pip install -e .` or `easy_install .`
If this does not resolve the problem, please open an issue on our GitHub.

If I a visdom client from a python repl it works; any idea? I guess it's due to the connection check in the beginning of the demo script.

Ah no worries in that case - we're still working through the issues brought up by #450. As I have the fix for that surely working I implemented the last things I wanted to see. Thanks again for laying the groundwork here and for contributing in the first place!


Expose useErrorMessageId hook necessary for importing the BasicSendbox component for customization

Is your feature request related to a problem?
Regarding IcM #414844341.
The customer is attempting to make some Web Chat customizations that pertain to integrating a validation box/toast notification into the BasicSendBox component. However, they are unable to move forward as the hook, useErrorMessageId, is inaccessible.
From the customer in an email thread:

We are looking into creating some minor customizations in-house in the interim. However, there appears to be one small issue standing in our way. Would you consider proceeding with a smaller fix in order to enable us to do some customizations on our own?
There is one hook that is not exposed in this sendbox, related to accessibility error messages: useErrorMessageId. Every other hook in the library is exposed, but this one is not exposed in the same path. It seems to be the last hurdle in allowing us to import the BasicSendbox component for customization, and we are hopeful it could be a quick fix from Microsoft’s end...

Describe the suggestion or request in detail
Expose the useErrorMessageId hook so it is accessible when imporing the BasicSendBox component.
From the customer:

Could you help us understand if the useErrorMessageId hook could be exposed allowing the customers to make the customizations for validation box? They are looking into creating some minor customizations in-house, however there is an issue there since useErrorMessageId hook is not exposed. Every other hook in the library is exposed, but this one is not exposed in the same path. It seems to be the last hurdle in allowing them to import the BasicSendbox component for customization.

Describe alternatives you have considered
The customer is unsure as to what other options, if any, are available to try. However, they are "open to other ideas or suggestions about a better approach."
Additional context
Customer's full email response regarding the inability to customize built-in Web Chat components:

I definitely understand your perspective in terms of not offering infinite customization, but from my perspective what is currently offered is really a “half-customizable” user interface. The webchat ui can be thought of in terms of input & output: output from the bot to the user, and input from the user to the bot. Since the “SendBox” component comprises the entire input section of the application, only half of the webchat application is actually customizable.
For example, the SendBox really includes: quick responses, text input area, send button, file upload button, microphone button, and dictation interims components. However, we are currently unable to modify, remove, or add to these inner components because the SendBox is not customizable. The naming of SendBox would lead you to believe this is a small area of the ui, but it’s not.
I have to say we have been impressed with the output half of the application, as it has a great deal of customizability included, and we have taken advantage of that attention to detail in several important cases. But it seems that the input half has been rather neglected.
While we still desire the ability of full customization of the input section, we are looking into creating some minor customizations in-house in the interim. However, there appears to be one small issue standing in our way. Would you consider proceeding with a smaller fix in order to enable us to do some customizations on our own?
There is one hook that is not exposed in this sendbox, related to accessibility error messages: useErrorMessageId. Every other hook in the library is exposed, but this one is not exposed in the same path. It seems to be the last hurdle in allowing us to import the BasicSendbox component for customization, and we are hopeful it could be a quick fix from Microsoft’s end, that would enable us to proceed in-house. We will present the path forward at today’s meeting.

@compulim doing a check in if this is something PG will be looking at doing?


Offset is not updating in tgcf.config.yml

I checked the recent release of tgcf
And tried both tgcf past and tgcf live feature
Both are working fine. But
When using tgcf in past mode , It is not updating it's offset in tgcf.config.yml

Hi @ankit-sinha-18, this will be resolved today.

Thank you


RHCLOUD-30826: Remove use of POSTGRESQL_MASTER env vars

Currently when the ephemeral DB's start up, the container in the pod always restarts once.
If you look at the container logs, you can see it is running into this error:
waiting for server to start....2024-02-05 16:09:50.580 UTC [35] LOG:  starting PostgreSQL 12.14 on x86_64-redhat-linux-gnu, compiled by gcc (GCC) 13.0.1 20230208 (Red Hat 13.0.1-0), 64-bit
2024-02-05 16:09:50.581 UTC [35] LOG:  listening on Unix socket "/var/run/postgresql/.s.PGSQL.5432"
2024-02-05 16:09:50.582 UTC [35] LOG:  listening on Unix socket "/tmp/.s.PGSQL.5432"
2024-02-05 16:09:50.599 UTC [35] LOG:  redirecting log output to logging collector process
2024-02-05 16:09:50.599 UTC [35] HINT:  Future log output will appear in directory "log".
 done
server started
/var/run/postgresql:5432 - accepting connections
createuser: error: creation of new role failed: ERROR:  role "postgres" already exists

And I think this is due to us setting the POSTGRESQL_MASTER_USER to be postgres -- it causes the scl container scripts to invoke this:
https://github.com/sclorg/postgresql-container/blob/16cd132d4986523b531a353401e6ed4ee0703c8b/10/root/usr/share/container-scripts/postgresql/common.sh#L241
Reading through this: https://github.com/sclorg/postgresql-container/tree/16cd132d4986523b531a353401e6ed4ee0703c8b/examples/replica#postgresql_master_password it seems that the MASTER_ env vars only relate to DB replication. We don't use this feature so we don't even need to set these env vars. We should use POSTGRESQL_ADMIN_PASSWORD though to configure the admin password: https://github.com/sclorg/postgresql-container/tree/16cd132d4986523b531a353401e6ed4ee0703c8b/examples/replica#postgresql_password-and-postgresql_admin_password
We also should not need to set PGPASSWORD on the pods anymore, so I removed that. It doesn't look like any of the images at https://github.com/sclorg/postgresql-container use this environment variable.

/retest


Side Panel React AI enhancements

Add the slug and gradient to the IBM Products side panel: https://ibm-products.carbondesignsystem.com/?path=/docs/ibm-products-components-side-panel-sidepanel--docs

@lee-chase @jeffchew @ljcarot @jeanservaas per our conversation today, for phase 1, this is an enhancement to the existing React Side Panel in @carbon/ibm-products.
Graduating this component to @carbon/react is a separate effort, that will be considered independent of the Carbon for AI effort.

@aagonzales @mattrosno this is now published here
https://ibm-products.carbondesignsystem.com/?path=/story/ibm-products-components-side-panel-sidepanel--slide-over&args=slug:1
In the "Controls" section you can adjust various side panel properties including size, title contents and the number of buttons.

@lee-chase thank you. Can this issue be closed?

In the original spec, the icons were flush to the edge but looks like in C4IP the close icon is actually inset. So I guess if that's the case and the close can't be moved the the AI slug should also be inset to align with the close icon.
Original spec

Align with C4IP placement

I don't see any meaningful reason why the C4IP side panel differentiates from the modal and tearsheet, I feel this is an oversight. I think it would be worthwhile making the change to the original side panel to be consistent (as I think the slug placement looks better in the original spec :)

The close position is a good point @aagonzales I am uncertain as to why the sidepanel, unlike the other modal forms (including Tearsheet) have a larger close button usually always 48px.
I've asked Laura to comment and see she's quicker off the mark than me.
Looking at the designs here, it might be a mistake.
https://www.figma.com/file/Yz74b1hPqqYWwEtmcPqojq/(v10)-Gray-10---IBM-Products?node-id=8858%3A390246&mode=dev
https://pages.github.ibm.com/cdai-design/pal/components/side-panel/usage
Will make changes as suggested.


How should I convert items

For example,convert cavesandcliffs:deepslate to quark:deepslate, their dictionaries all include forge: stone and cavesandcliffs:deepslate. I've been trying for a long time but haven't figured it out. Can you help me answer this question

You'll probably want to use the tagSplits or customtags fields to make your own tags to be unified. tagSplits was designed for this exact use case!


microk8s setup failed in M1

Summary
Darwin Alvaros-MacBook-Pro.local 22.5.0 Darwin Kernel Version 22.5.0: Thu Jun  8 22:22:20 PDT 2023; root:xnu-8796.121.3~7/RELEASE_ARM64_T6000 arm64
After installing microk8s on my system, I got this output message:
warning: "--mem" long option will be deprecated in favour of "--memory" in a future release. Please update any scripts, etc.
launch failed: The following errors occurred:
microk8s-vm: timed out waiting for response
warning: "--mem" long option will be deprecated in favour of "--memory" in a future release. Please update any scripts, etc.
launch failed: instance "microk8s-vm" already exists
warning: "--mem" long option will be deprecated in favour of "--memory" in a future release. Please update any scripts, etc.
launch failed: instance "microk8s-vm" already exists
warning: "--mem" long option will be deprecated in favour of "--memory" in a future release. Please update any scripts, etc.
launch failed: instance "microk8s-vm" already exists
warning: "--mem" long option will be deprecated in favour of "--memory" in a future release. Please update any scripts, etc.
launch failed: instance "microk8s-vm" already exists
warning: "--mem" long option will be deprecated in favour of "--memory" in a future release. Please update any scripts, etc.
launch failed: instance "microk8s-vm" already exists
warning: "--mem" long option will be deprecated in favour of "--memory" in a future release. Please update any scripts, etc.
launch failed: instance "microk8s-vm" already exists
warning: "--mem" long option will be deprecated in favour of "--memory" in a future release. Please update any scripts, etc.
launch failed: instance "microk8s-vm" already exists
warning: "--mem" long option will be deprecated in favour of "--memory" in a future release. Please update any scripts, etc.
launch failed: instance "microk8s-vm" already exists
warning: "--mem" long option will be deprecated in favour of "--memory" in a future release. Please update any scripts, etc.
launch failed: instance "microk8s-vm" already exists
warning: "--mem" long option will be deprecated in favour of "--memory" in a future release. Please update any scripts, etc.
launch failed: instance "microk8s-vm" already exists
An error occurred with the instance when trying to launch with 'multipass': returned exit code 2.
Ensure that 'multipass' is set up correctly and try again.

What Should Happen Instead?
Mmm... works?
Reproduction Steps
Setup microk8s from brew as documentation shows.
Introspection Report
sudo microk8s inspect
Password:
MicroK8s is not installed. Please run `microk8s install`.

Are you interested in contributing with a fix?
Yes

I've turned up the vm from multipass and I got this when I choose open shell
launch failed: The following errors occurred:
primary: timed out waiting for response

Saving session...completed.
Deleting expired sessions...       7 completed.

[Process completed]

Hi @sincorchetes
It looks like there is already a VM with the name microk8s-vm (perhaps from a previous installation?) and that's causing the installer to have a bad time. Can you try to see if force deleting the VM can unblock the install?
multipass delete microk8s-vm
multipass purge

microk8s install

Hi @sincorchetes
It looks like there is already a VM with the name microk8s-vm (perhaps from a previous installation?) and that's causing the installer to have a bad time. Can you try to see if force deleting the VM can unblock the install?
multipass delete microk8s-vm
multipass purge

microk8s install

Hi @neoaggelos ,
$ microk8s uninstall
Thank you for using MicroK8s!

multipass delete microk8s-vm
delete failed: The following errors occurred:
instance "microk8s-vm" does not exist

$ multipass purge
$ microk8s install --cpu 2 --mem 10 -y
warning: "--mem" long option will be deprecated in favour of "--memory" in a future release. Please update any scripts, etc.
launch failed: The following errors occurred:
microk8s-vm: timed out waiting for response
warning: "--mem" long option will be deprecated in favour of "--memory" in a future release. Please update any scripts, etc.
launch failed: instance "microk8s-vm" already exists
warning: "--mem" long option will be deprecated in favour of "--memory" in a future release. Please update any scripts, etc.
launch failed: instance "microk8s-vm" already exists
warning: "--mem" long option will be deprecated in favour of "--memory" in a future release. Please update any scripts, etc.
launch failed: instance "microk8s-vm" already exists
warning: "--mem" long option will be deprecated in favour of "--memory" in a future release. Please update any scripts, etc.
launch failed: instance "microk8s-vm" already exists
warning: "--mem" long option will be deprecated in favour of "--memory" in a future release. Please update any scripts, etc.
launch failed: instance "microk8s-vm" already exists
warning: "--mem" long option will be deprecated in favour of "--memory" in a future release. Please update any scripts, etc.
launch failed: instance "microk8s-vm" already exists
warning: "--mem" long option will be deprecated in favour of "--memory" in a future release. Please update any scripts, etc.
launch failed: instance "microk8s-vm" already exists
warning: "--mem" long option will be deprecated in favour of "--memory" in a future release. Please update any scripts, etc.
launch failed: instance "microk8s-vm" already exists
warning: "--mem" long option will be deprecated in favour of "--memory" in a future release. Please update any scripts, etc.
launch failed: instance "microk8s-vm" already exists
warning: "--mem" long option will be deprecated in favour of "--memory" in a future release. Please update any scripts, etc.
launch failed: instance "microk8s-vm" already exists
An error occurred with the instance when trying to launch with 'multipass': returned exit code 2.
Ensure that 'multipass' is setup correctly and try again.

OK, I think I see what is happening.
The microk8s install command will eventually issue a command like multipass launch 22.04 --cpus 2 --mem 10G --name microk8s-vm. It appears that the launch command times out and the installer simply retries it a few times. However, the VM already exists and that's the error that you see.
See the relevant code in https://github.com/canonical/microk8s/blob/20ec6d5586dca71098ef2dd6a26ce1aecb1be499/installer/vm_providers/_multipass/_multipass.py#L67-L81
Can you please test whether you can launch a simple ubuntu VM with the following command?
multipass launch 22.04 --name test-vm --cpus 2 --mem 10G

Thanks!

OK, looks like multipass is having issues with starting VMs. Can you share more details like multipass ls and multipass info test-vm to see what status the VM is in?
This looks like an issue with multipass rather than microk8s, could you create an issue in https://github.com/canonical/multipass with more information? Thank you

Hi @neoaggelos ,
I got this:
multipass ls
Name                    State             IPv4             Image
test-vm             Unknown           --               Ubuntu 22.04 LTS

$ multipass info test-vm
Name:           test-vm
State:          Unknown
IPv4:           --
Release:        --
Image hash:     9c59c6097711 (Ubuntu 22.04 LTS)
CPU(s):         --
Load:           --
Disk usage:     --
Memory usage:   --
Mounts:         --

Hi @sincorchetes
To me, this feels like multipass is having issues starting VMs on your machine. The reason for that is hard for me to understand. Can you please create an issue in https://github.com/canonical/multipass with this info and see if the multipass team can go into more details?
Thanks, I want to leave this issue open until we have figured out the cause.

@neoaggelos , It's works following this guide: https://github.com/canonical/multipass/issues/3200#issuecomment-1687203485

I am getting the same error while running microk8s install
Trying to run microk8s on MacOS (macOS 13.5.1 / M1)
Steps followed here - https://microk8s.io/docs/install-macos

Sorry, I've de-installed it and used a Linux environment to launch that
because It's broken...
On Mon, Apr 15, 2024 at 3:19 PM asksmruti @.***> wrote:

@neoaggelos https://github.com/neoaggelos , It's works following this
guide: canonical/multipass#3200 (comment)
https://github.com/canonical/multipass/issues/3200#issuecomment-1687203485
Did you provision the VM first with multipass and then setup microk8s on
it ?
Like the steps mentioned here - https://microk8s.io/docs/install-multipass
—
Reply to this email directly, view it on GitHub
https://github.com/canonical/microk8s/issues/4154#issuecomment-2056837889,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHP5X25MNYICEW6IJRAHOH3Y5PHW5AVCNFSM6AAAAAA3SB34AOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJWHAZTOOBYHE
.
You are receiving this because you were mentioned.Message ID:
@.***>

I can echo everything said here. This was a completely fresh install, never run microk8s on this mac before. I get those errors. It seems that something is broken on your end. I'm using an M3 MBP.

I am seeing this exact error as well. did anyone come up with a solution?

Sorry, I’ve sold the M1
On Tue, Dec 3, 2024 at 18:22 graham-atom @.***> wrote:

I am seeing this exact error as well. did anyone come up with a solution?
—
Reply to this email directly, view it on GitHub
https://github.com/canonical/microk8s/issues/4154#issuecomment-2515285947,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHP5X2Y6SPAWRPR67G45BF32DXZIFAVCNFSM6AAAAAA3SB34AOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMJVGI4DKOJUG4
.
You are receiving this because you were mentioned.Message ID:
@.***>


Cannot reuse statements since 2.4.7

Since go-ora version 2.4.7 once prepared statements cannot be reused for multiple queries. It errors out with ORA-01001: invalid cursor.
Example:
func must(err error) {
	if err != nil {
		panic(err)
	}
}

func main() {
	db, err := sql.Open("oracle", "oracle://user:pw@server:/service")
	must(err)
	defer db.Close()
	stmt, err := db.Prepare("select :s from dual")
	must(err)
	defer stmt.Close()

	rows, err := stmt.Query("hi")
	must(err)

	var s string
	for rows.Next() {
		err = rows.Scan(&s)
		must(err)
		log.Println(s)
	}
	must(rows.Err())
	rows.Close()

	rows2, err := stmt.Query("hello")
	must(err) // panic: ORA-01001: invalid cursor
	for rows2.Next() {
		err = rows2.Scan(&s)
		must(err)
		log.Println(s)
	}
	must(rows2.Err())
	rows2.Close()
}

In version 2.4.6 and below this was working.

Prepare creates a prepared statement for later queries or executions. Multiple queries or executions may be run concurrently from the returned statement. The caller must call the statement's Close method when the statement is no longer needed.

from https://pkg.go.dev/database/sql#DB.Prepare

That is not the solution.
Prepared statements should be cachable and reusable regardless of any result sets. Statements are prepared against the connection pool, so even if the connection of a prepared statements is closed, it should automatically re-prepared with a new connection. This what the database/sql package provides.

now working you can test it

It seems to be OK in 2.4.11. Thanks for the quick fix :)

this problem  still exist


[WIP] Add 0.10.3 of secretgen-controller

DO NOT MERGE UNTIL TKG AND TCE HAVE VIEWED / MADE CHANGES TO ACCOMMODATE

This simplfies the process of ingesting secretgen as a package
TCE just needs the published artifacts from SGC github now

Signed-off-by: Neil Hickey<EMAIL_ADDRESS>What this PR does / why we need it
This PR simplifies the process for secretgen controller PackageCR management from 0.10.3 onwards.
It is also a signal that the PackageCR management will be done by the carvel team in the OSS repo
https://github.com/vmware-tanzu/carvel-secretgen-controller/tree/develop/config
Describe testing done for PR
The release process for SGC runs our e2e tests against the PackageCR installed via a PackageInstallCR
Special notes for your reviewer
The values schema is different now - this could have implications on any downstream consumers of TCE
For the values schema... checkout https://github.com/vmware-tanzu/carvel-secretgen-controller/blob/develop/config/package-bundle/config/values-schema.yml

👋 Hey @seemiller , what exactly is the README used for? We were chatting about it, and it doesn't feel right to have the README as an artifact that we produce.
All the information in a README should be derivable from the PackageCR itself, perhaps some tool could exist / be created that generates the README from the PackageCR.
Some the questions boil down to:

Do we really need a README.md?
Can the readme just live at the top level (not associated with a particular version)?

The readme is basically a quick reference for the package within TCE. If you're browsing the codebase in GitHub, it's useful for description, usage, parameters, etc; the readme's are also copied to the documentation site for reference. If we don't have a readme, I think that we will end up with a dead link in the left nav menu for packages on that page.

Although, I just noticed that we are not exposing secretgen-controller in the user-managed package repo, so those docs will not show up on the documentation site anyway.

@seemiller Makes sense! Thanks for the background, so is it fine to have a README at the top level that is not version specific?


recipe in smp

hello
minecraft 1.7.10
1.2.0a-Mustard
IC2 2.2.658
i can't craft afsu on my server , i see afsu block on output slot but i can't take it
i got no prob in solo
thanks

Bleah... that's a weird issue. Have you had any issues with other blocks/items? Testing the AFB would be good. Also, do you have Cauldron/MCPC?
Good to have an excuse to make an update to add the Russian translation in, I must say :P

hello
no more issue for me
i have no cauldron/mcpc only forge
thanks

What was the problem?

I think it wasn't really an issue.

hello
with the normal version i can craft afb but i can't craft afsu (recipe was correctly displayed but i can't take it, look like recipe only registered client side)
and if i use applied energetic i can't take afb out of my network (that was strange).
i see it on my network (maybe don't like to be stacked)
i will try with a fresh server with only ic2 and afsu to see if i get this issue
i use inventory tweak maybe issue come with it (sync prob)

I can confirm this problem still exists using IC2 2.2.722, AFSU 1.2.1a Mustard and Forge <IP_ADDRESS>1.
Unable to craft the AFSU in both AE crafting terminal and vanilla crafting table when playing on a dedicated server. Output shows up but couldn't pick it up (like the recipe isn't registered on the server).
After disconnecting and connecting again to server, I could complete the crafting (only tested in vanilla table).

Dang it, what a weird bug...

Update IC2-Experimental to at least version "2-2.2.726".
AFSU is not the problem, IC2-Experimental is.

Greetings!, not trying to revive a 6+ years old github, nor i am trying to ask for an update or anything, but since the issue is still happening i figured i would make suggestions to people who want to use the mod:
Latest 1.7.10 ic2 (2-2.2.828-experimental)
Latest 1.7.10 Forge (<IP_ADDRESS>)
What i found out:

DO NOT PLACE THE AFB IN AN AE SYSTEM, YOU WONT BE ABLE TO TAKE IT OUT EASILY (dont really know why, but i had to export it using an export bus with a fuzzy card, otherwise can't take it out of the storage from the terminal)
In case you have the issue of the recipes, grab the stuff in your inventory, disconect from the server, connect again, use a vanilla workbench and you will be able to craft the AFSU
The probable cause is some other mod using an old ic2 api or something, wasnt able to figure it out

+++ for last comment, i used ME Chest to get it out. Will post MineTweaker 3 script here to replace AFP core on AFPU craft shortly

`#Fix AFSU Craft by removing AFB core from recipe and adding 2 more iridium plates and u-matter cell
recipes.addShaped(AFSU:AFSU,
[[IC2:blockElectric:2, IC2:itemPartIridium, IC2:blockElectric:2],
[IC2:itemPartIridium, IC2:itemCellEmpty:3, IC2:itemPartIridium],
[IC2:blockElectric:2, IC2:itemPartIridium, IC2:blockElectric:2]
]);
`


Hide PLAY button in vertical orientation

fixes #518
First things first, we can't detect when FGO is closed.
The WINDOW_STATE_CHANGED event is received only when apps are opened/resumed. Even then, it includes events from the System Launcher, Status UI and FGA's overlays too, so checking the latest event isn't an option. So, we have to stick to: 'Hide if screen orientation changed to vertical'. Because FGO surely doesn't run in a vertical orientation.
So, the behaviour is: Once you press the START button (previously called TOGGLE SERVICE), the PLAY button doesn't show up, but, the Foreground Service notification, as well as the MediaProjection/Root screenshot code/permissions, are set up.
A toast is shown suggesting to switch to FGO to start scripts.
The PLAY button shows up once you switch to FGO.
When you switch out of FGO into a vertical orientation, the PLAY button disappears.
But, if you're on an emulator or a tablet (where landscape orientation is the default), or have turned ON Auto-Rotate on your phone and switch to another screen in landscape mode, the PLAY button won't disappear.
Things to consider:

If you switch out of FGO (into some other app in vertical orientation) while a script is running, it should ideally PAUSE. Right now it keeps running. I think it's time to make PAUSE a non-optional feature.

When you lock your screen while in FGO and then unlock it, the play button is gone until you switch to another app and back.

When you lock your screen while in FGO and then unlock it, the play button is gone until you switch to another app and back.

On my phone PLAY button still displays, regardless of whether I unlock directly with fingerprint or by drawing the pattern in portrait mode.

Seems like it was a one-time glitch. Worked every time when I tried again.

I spammed screen lock/unlock and the PLAY button sometimes does not show like you said.
Correct case:
V/FGA: Detected FGO En
D/WindowManager: Add to mViews: com.mathewsachin.fategrandautomata.ui.HighlightView{2707390 V.ED..... ......I. 0,0-1080,2292}, this = android.view.WindowManagerGlobal@e7a7232,pkg= null
V/ViewRootImpl: The specified message queue synchronization  barrier token has not been posted or has already been removed
D/WindowManager: Add to mViews: android.widget.FrameLayout{2b39caf V.E...... ......ID 0,0-162,162}, this = android.view.WindowManagerGlobal@e7a7232,pkg= null
V/ViewRootImpl: The specified message queue synchronization  barrier token has not been posted or has already been removed
V/FGA: Horizontal: com.coloros.floatassistant
V/FGA: Horizontal: com.android.systemui
V/FGA: Horizontal: com.mathewsachin.fategrandautomata.debug
V/FGA: Horizontal: com.mathewsachin.fategrandautomata.debug

Faulty case:
V/FGA: Detected FGO En
V/FGA: Vertical: com.coloros.floatassistant
V/FGA: Vertical: com.android.systemui
V/FGA: Horizontal: com.coloros.floatassistant

System UI glitched a bit when that happened. I guess this can be solved if instead of checking vertical, we check for horizontal (after an FGO opened event has been received).

I changed the behaviour to: 'PLAY button hides in portrait orientation and shows up in Landscape orientation'.
And this seems to work fine because it's independent of FGO now.

@reconman What should we do about the case of switching to portrait orientation while the script is running?

I forgot about the pausing suggestion, I agree with that.


Fix AffineTransform initializer

This simply cherry-picks #4327 to master.

@swift-ci please test and merge

@swift-ci please test and merge


Line Chart Time Series markers not correct after zoom

Describe the bug
When you use the zoom bar at the bottom and then use the mouse to hoover over the chart, the markers are not lined up correctly with the line.
To Reproduce
Steps to reproduce the behavior:

Go to Top 10 California Names Timeseries
Zoom in with the zoom bar at the bottom
Click on the chart to show the vertical line
The markers are too much to the left (it is not taking into consideration it is zoomed)

Expected behavior
Markers are on the line as is the case without zoom
Screenshots
No zoom:

With zoom:

Environment (please complete the following information):

superset-ui version: v0.999.0dev
Node version: v12.20.0
npm version: 6.14.8

The codebase on this repo has been moved to the main Apache Superset repo, and consequently the repo is in the process of being archived. See the Superset Improvement Proposal for details: https://github.com/apache/superset/issues/13013 . While all currently open issues and PRs will be closed, we encourage you to reopen this issue on the main repo if it is still relevant.


Feature Request: Enhance Code Outline Panel Width

The Code Outline panel's limited width truncates long method names, hindering code navigation. Please consider expanding the panel's adjustable width options.

Looks like breadcrumbs drop down.
There's separate outline view/paned in sidebar

@IllusionMH
While I understand the desire to expand the sidebar for better navigation, making it wider would unfortunately reduce the space available for the code area. Given the limited screen size, this could hinder our ability to view and edit code effectively.

I mean issue title didn't match element on screenshot, that's all.

@IllusionMH
Thank you for your feedback! I’ve revised the issue title to better match the element shown in the screenshot.


Portuguese (Brazil) Documentation Translation

Is your feature request related to a problem? Please describe.
Currently, the documentation is only available in English, which can be challenging for Portuguese-speaking users in Brazil. This language barrier may lead to misunderstandings, frustration, and difficulty in fully utilizing the product or service.
Describe the solution you'd like
A complete translation of the documentation into Brazilian Portuguese, allowing users to easily understand and navigate through the instructions, features, and guidelines. This would include translating all relevant sections, terms, and instructions while maintaining the context and technical accuracy.
Describe alternatives you've considered
Alternatives include using automated translation tools like Google Translate or adding a summary of key sections in Portuguese. However, these options might not be as accurate or user-friendly as a dedicated, professionally translated version of the documentation.
Additional context
Providing a high-quality translation can significantly enhance user experience and accessibility for the Brazilian market. Screenshots, diagrams, and examples should also be adapted to Portuguese to ensure clarity and effectiveness.

Hi @payam-zahedi,
For this translation, I’m considering creating a new directory within the /doc folder, named either /langs or /translations. This will serve as a dedicated space for all current and future translations, keeping things organized and scalable.
What do you think?

Hi @AntonioAEMartins,
Thank you for suggesting support for multiple languages. While it's a great idea, it's important to consider that maintaining and updating several languages can be quite resource-intensive and complex over time.
An alternative could be to leverage the built-in translation features available in browsers like Google Chrome and Firefox, which offer instant translation capabilities. These tools provide users with a convenient way to view content in their preferred language without adding the overhead of maintaining translations within the platform.


[Improve] workspace local permission check

[Improve] workspace local permission check

cc @tisonkun PTAL, thx

commit ac85eb42fa8334f294a59a511585b0fc4a9f97d7 (HEAD -> workspace)
Author: tison<EMAIL_ADDRESS>Date:   Thu Apr 13 09:00:46 2023 +0800

    local dir = workspace

    Signed-off-by: tison<EMAIL_ADDRESS>
diff --git a/deploy/helm/streampark/conf/streampark-console-config/application.yml b/deploy/helm/streampark/conf/streampark-console-config/application.yml
index 422fd0ae3..714636563 100755
--- a/deploy/helm/streampark/conf/streampark-console-config/application.yml
+++ b/deploy/helm/streampark/conf/streampark-console-config/application.yml
@@ -83,7 +83,7 @@ streampark:
   hadoop-user-name: hdfs
   # local workspace, used to store source code and build dir etc.
   workspace:
-    local: /opt/streampark_workspace
+    local: workspace
     remote: hdfs://hdfscluster/streampark   # support hdfs:///streampark/ 、 /streampark 、hdfs://host:ip/streampark/

   # remote docker register namespace for streampark

I guess it's as simple as the patch above. Perhaps you should ensure streampark.sh cd to $APP_HOME. Always the path is resolved against current directory.

Is it possible we just change the default value to a folder under current working dir? For example, workspace.local = ./streampark_workspace or just ./workspace.

Thanks for your review, it is not recommended to defined workspace directory under the project($APP_HOME/workspace), because users will upload jars to this workspace directory. If the streampark upgraded or redeployed, this path will change, which will lead to The dependency jars in many jobs in the user's historical jobs cannot find the path. Therefore, this path needs to be configured separately. Configured to a safe directory


Perform a Pull on Each Repository

It would be nice to pull down any upstream changes on master for origin and/or upstream.

I'm not sure this fits with the goal of CIG ("Can I go?"). Executing a pull makes me think more of "Let's Start Work".
If it were to be added as a feature, some questions need to be answered:

How would you handle conflicts for each individual repo?
How do we know which Git remote to pull from?


Accessibility

According to the GOV.UK guidance on the accessibility regulations this project is covered by the public sector bodies accessibility regulations 2018, so it needs to be a) made accessible and b) publish an accessibility statement.

Possible accessibility issues:

text contrast
text size
some indication of elements that are links


Use environment variable for Splunk Token

I'm trying to remove the Splunk Token from source and instead pull the value from an Environment Variable.  We're deploying to Azure and the Env Variables are coming from App Settings which are using a KeyVault reference to pull the secret from KeyVault.  When I log in via Kudu and run the following command to retrieve the Env Var:
PS C:\home> ${env:nLogSplunkHecToken}
${env:nLogSplunkHecToken}
<Correct Splunk Token returned>

So I'm pretty sure that I have the wire up correct there.
In my NLog.config file I have this:
<target name="SplunkNlogConfig"
            xsi:type="SplunkHttpEventCollector"
            serverUrl="https://splunkhec.tpower.net.nz"
            token="${environment:nLogSplunkHecToken}"
            channel="channel-guid"
            retriesOnError="0"
            batchSizeBytes="0"
            batchSizeCount="0"
            includeEventProperties="true"
            includeMdlc="false"
            includePositionalParameters="false"
            MaxConnectionsPerServer="10"
            IgnoreSslErrors="false">
      <contextproperty name="host" layout="${machinename}" />
      <contextproperty name="threadid" layout="${threadid}" />
      <contextproperty name="logger" layout="${logger}" />
      <contextproperty name="clientIp" layout="${aspnet-request-ip:CheckForwardedForHeader=true}" />
      <contextproperty name="requestHost" layout="${aspnet-request-host}" />
      <contextproperty name="iisSiteName" layout="${iis-site-name}" />
      <contextproperty name="requestUrl" layout="${aspnet-request-url:IncludeQueryString=true}" />
      <contextproperty name="responseStatusCode" layout="${aspnet-response-statuscode}" />
      <contextproperty name="requestIp" layout="${aspnet-request-ip}" />
      <contextproperty name="requestMethod" layout="${aspnet-request-method}" />
    </target>

Note the attempt to use the Env Variable for the token: token="${environment:nLogSplunkHecToken}".  In the NLog internal logs I see the following entry:
Error SplunkHttpEventCollector(Name=SplunkNlogConfig): Failed to send LogEvents Exception: Splunk.Logging.HttpEventCollectorException: Failed to send request: Forbidden - {"text":"Invalid token","code":4}

Which I presume means I'm trying to pass the literal text as the token rather than the resolved Env Variable value.  When I hard code the token, logging works as expected.
So I'm not sure if this is something that cannot be done (and I'm wasting my time) or if I've missed something obvious.  These are the Nuget packages we're using:
NLog: 4.7.6
NLog.Config: 4.7.6
NLog.Extensions.Logging: 1.7.0
NLog.Schema: 4.7.6
NLog.Targets.Splunk: 2.2.1
Nlog.Web: 4.10.0

Hmmm, it seems that if I bump all those Nuget packages to the latest stable versions, then it works.  Not sure which of the packages would have been the culprit...
NLog: 4.7.14
NLog.Config: 4.7.14
NLog.Extensions.Logging: 1.7.4
NLog.Schema: 4.7.14
NLog.Targets.Splunk: 3.0.0
NLog.Web: 4.14.0


Can we access the handler for theme change?

Can we access the handler for theme change? for dark and light
I think it would be useful.

Funding

You can sponsor this specific effort via a Polar.sh pledge below
We receive the pledge once the issue is completed & verified

It's not even clear to me if it's possible to change the theme. It seems to be just based on TailwindCSS and is set based to be based on users system preference.
https://github.com/albingroen/react-cmdk/issues/14

But style is hardcoded in css file you are providing and classnames looks like hardcoded too. To change color of an icon I need to copy whole css file and then do the trick with changing [pic rel]

to different values for the same className. That sould be more dynamic or I don't get something right


Respect the message handler setting on the service isolate as it is shutting down.

The service isolate is currently always shut down on a VM managed thread. This is in spite of the message handler for this isolate being updated via Dart_SetMessageNotifyCallback in the isolate create callback.

cc @rmacnak-google @a-siva

That call to MessageHandler::Run you linked to, and the corresponding ShutdownIsolate callback, are almost identical to the way Isolate::Run works.
Can you give me some more details on the specific issue you're seeing?


query result read stream does not end

The query result stream seems to get stuck and never emits  end event. I can see all the documents being processed correctly in data callback but stream does not end. I have to retry couple of times to return results, sometimes it works in one try and sometime it takes couple of times.
The equivalent .result() version works just fine
tried on latest official and on develop branch, same issues on both
db.documents.read({
                    uris: uris, // array of 11 uris,
                    transform: ['all-transforms'] // no issues with transforms script as i can see all documents in *data* callback
                })
              .stream()
              .on('data', function(document) {
                        var content = (!_.isObject(document.content)) ? JSON.parse(document.content) : document.content;
                        if (!_.isNull(content) || content.length > 0) {
                            diffsOnEachPlatform[content[0].platform.name] = _.pluck(content, 'name');
                        }
                    })
              .on('error', function(error) {
                        console.error(error);
                    })
              .on('end', function() {
                      // control never gets here, it often takes multiple tries to work
                        console.log(diffsOnEachPlatform);
                    });

Hi, Sudhakar:
I'm puzzled, because there are examples, unit tests, and regression tests for .on('end').
How does this case differ?  Can you provide a way to reproduce the issue outside of the specific environment?
The release is impending, so the sooner we can isolate the issue, the better.
Thanks for looking into it.

Here's the test that demo's the bug. It sometimes runs successfully but sometimes fails because the mocha timeout caused due to the stream not ending
note: Don't change the db connection details
it('should end the stream after processing all documents', function(done) {

		var db = marklogic.createDatabaseClient({
        host:     'staging-bugtrack.marklogic.com',
        port:     8006,
        user:     'admin',
        password: 'admin',
		authType: 'DIGEST'
		});
		var processedUris = [];
		db.documents.read({
			uris: ['/b4_0/2010-01-12/linux-32-rh4/platform-diffs.xml',
				'/b4_0/2010-01-12/linux-64-rh4-cluster/platform-diffs.xml',
				'/b4_0/2010-01-12/linux-64-rh5v-cluster/platform-diffs.xml',
				'/b4_0/2010-01-12/linux-64-rh5v/platform-diffs.xml',
				'/b4_0/2010-01-12/solaris-32-sol9-cluster/platform-diffs.xml',
				'/b4_0/2010-01-12/solaris-32-sol9/platform-diffs.xml',
				'/b4_0/2010-01-12/solaris-64-sol10-cluster/platform-diffs.xml',
				'/b4_0/2010-01-12/winnt-32-w2k3/platform-diffs.xml',
				'/b4_0/2010-01-12/winnt-64-w2k3-cluster/platform-diffs.xml',
				'/b4_0/2010-01-12/winnt-64-w2k3/platform-diffs.xml',
				'/b4_0/2010-01-12/winnt-64-w2k8sp/platform-diffs.xml'
			],
		}).stream()
			.on('data', function(document) {
				console.log(document.uri)
				processedUris.push(document.uri)
			})
			.on('error', function(error) {
				console.error(error);
				done(error);
			})
			.on('end', function() {
				console.log(processedUris)
				done();
			});
	})

Hi, Sudhakar:
Regarding:
"but sometimes fails because of the mocha timeout caused due to the stream not ending"
How have you determined that the end event is not firing instead of Mocha not waiting long enough time for the end event to fire?
We've seen case of Mocha giving up too soon for a long-running or large tasks or due to network issues.
Have you tried running mocha with  --no-timeouts or increasing the timeout, as in:
describe('...', function(){
this.timeout(5000);
Do you ever see the same issue outside of Mocha?
Thanks for looking into it,

I found this issue in my app. The http call which is hooked to this controller code gets stuck and does not return the response, even console.log() in end callback is not printed but all documents in data is getting streamed properly, hence my assessment that stream does not end.
I tried mocha with 5000ms and 10,000ms timeout but it still timesout

Can you provide a more complete reproduction case? We have tests with the same shape as the code above, so that code alone is not sufficient to reproduce the problem.

The test provides the isolated env for you, I'm not sure what else I can do. I'm slightly suspecting that the size of the documents returned may have some impact on it, the existing tests might not be exactly same in that sense. I'll be happy to show in person if that'll make things clearer.

Did you try the mocha test with the --no-timeouts argument?
If the size of the documents simply means that the response takes longer, no change to the Node API will reduce that time.

Yes i tried and mocha still timesout

I modified the example to remove mocha from the equation:
 var db = marklogic.createDatabaseClient({
        host:     'staging-bugtrack.marklogic.com',
        port:     8006,
        user:     'admin',
        password: 'admin',
        authType: 'DIGEST'
        });
        var processedUris = [];
        db.documents.read({
            uris: ['/b4_0/2010-01-12/linux-32-rh4/platform-diffs.xml',
                '/b4_0/2010-01-12/linux-64-rh4-cluster/platform-diffs.xml',
                '/b4_0/2010-01-12/linux-64-rh5v-cluster/platform-diffs.xml',
                '/b4_0/2010-01-12/linux-64-rh5v/platform-diffs.xml',
                '/b4_0/2010-01-12/solaris-32-sol9-cluster/platform-diffs.xml',
                '/b4_0/2010-01-12/solaris-32-sol9/platform-diffs.xml',
                '/b4_0/2010-01-12/solaris-64-sol10-cluster/platform-diffs.xml',
                '/b4_0/2010-01-12/winnt-32-w2k3/platform-diffs.xml',
                '/b4_0/2010-01-12/winnt-64-w2k3-cluster/platform-diffs.xml',
                '/b4_0/2010-01-12/winnt-64-w2k3/platform-diffs.xml',
                '/b4_0/2010-01-12/winnt-64-w2k8sp/platform-diffs.xml'
            ],
        }).stream()
            .on('data', function(document) {
                console.log(document.uri);
                processedUris.push(document.uri);
            })
            .on('error', function(error) {
                console.error(error);
                done(error);
            })
            .on('end', function() {
                console.log('done '+processedUris.length);
                console.log(processedUris);
            });

I get the following output:
/b4_0/2010-01-12/linux-32-rh4/platform-diffs.xml
/b4_0/2010-01-12/winnt-64-w2k8sp/platform-diffs.xml
/b4_0/2010-01-12/linux-64-rh4-cluster/platform-diffs.xml
/b4_0/2010-01-12/linux-64-rh5v-cluster/platform-diffs.xml
/b4_0/2010-01-12/solaris-32-sol9/platform-diffs.xml
/b4_0/2010-01-12/solaris-32-sol9-cluster/platform-diffs.xml
/b4_0/2010-01-12/winnt-32-w2k3/platform-diffs.xml
/b4_0/2010-01-12/linux-64-rh5v/platform-diffs.xml
/b4_0/2010-01-12/winnt-64-w2k3-cluster/platform-diffs.xml
/b4_0/2010-01-12/winnt-64-w2k3/platform-diffs.xml
/b4_0/2010-01-12/solaris-64-sol10-cluster/platform-diffs.xml
done 11
[ '/b4_0/2010-01-12/linux-32-rh4/platform-diffs.xml',
'/b4_0/2010-01-12/winnt-64-w2k8sp/platform-diffs.xml',
'/b4_0/2010-01-12/linux-64-rh4-cluster/platform-diffs.xml',
'/b4_0/2010-01-12/linux-64-rh5v-cluster/platform-diffs.xml',
'/b4_0/2010-01-12/solaris-32-sol9/platform-diffs.xml',
'/b4_0/2010-01-12/solaris-32-sol9-cluster/platform-diffs.xml',
'/b4_0/2010-01-12/winnt-32-w2k3/platform-diffs.xml',
'/b4_0/2010-01-12/linux-64-rh5v/platform-diffs.xml',
'/b4_0/2010-01-12/winnt-64-w2k3-cluster/platform-diffs.xml',
'/b4_0/2010-01-12/winnt-64-w2k3/platform-diffs.xml',
'/b4_0/2010-01-12/solaris-64-sol10-cluster/platform-diffs.xml' ]
If you run my code, do you see the same output.

hmm..Ya i see the same output, does not hang, works fine

Thanks for checking it out.  It looks like there's a Mocha issue.  If you agree, please close this bug.

well, the request was timing out in my app too, not just in mocha. not sure what's wrong. Thanks for looking into it


Network Watcher Deprecation? Internal Server Error

Module: 08
Lab/Demo: 01
Task: 03 and 04
Description of issue
Keeps getting Internal Server Error when trying the Connection Trouble shoot

Do you have any more specific information about the issue? Thanks.


WAFv2 to host multiple internal port 80 sites

I have a WAFv2, trying to host 2 sites.

www.testprod.com
www.teststag.com

i configured my WAFv2 with a public frontend IP. Since i am in testing stage, i can only use port 80. So i created 2 multi-site listeners and 2 rules to point them to the appropriate backend pools defined. All works as per expected, local PC host file set the same public ip to both urls to simulate DNS records.
Then i was told that, using the same url, from internal networks the url will use internal DNS to resolve into the WAF private ip. So the same url will be using external dns to resolve into the WAF public ip. Public side is tested above, working well. WAF is able to know which url to send to which backend pool.
My problem: after the above 2 listener at port 80 for public frontend is working, i tried to create a 3rd listener for private frontend. But i have an error:
Error: Two Http Listeners of Application Gateway XXXXXXXX-waf are using the same Frontend Port.
It seems like i could create multiple multi-site listener for port 80 if the frontend ip is public. why is there a problem when creating the multi-site listener for port 80 on the frontend private side?

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

ID: 32e7a4ca-bf44-675a-189d-addc8284e1ab
Version Independent ID: 37e99fb1-3fbd-727a-468e-fe73b6ca13cb
Content: Hosting multiple sites on Azure Application Gateway
Content Source: articles/application-gateway/multiple-site-overview.md
Service: application-gateway
GitHub Login: @vhorne
Microsoft Alias: amsriva

@cowking06 , There is a known issue where you will not be able to add the same frontend port for both Public and Private Frontend IPs.
As of now, as a workaround use different frontend port for the Private Frontend IP.

@cowking06 , There is a known issue where you will not be able to add the same frontend port for both Public and Private Frontend IPs.
As of now, as a workaround use different frontend port for the Private Frontend IP.

When do you think will this be resolved? Changing of ports may require me to change the custom application. Is there a workaround within WAF which is transparent to the users?

@cowking06 , Not really sure about the ETA on the fix. But you don't need to change the backend port. Only the frontend port needs to be changed.
Your application will continue  to listen on port 80.

@cowking06 , I will go ahead and close this issue. Please tag me if you have any further questions.


Fix: Fix chart error on switch back to recent

Description
Chart stops updating after switching back to "Recent".
Apparently it's a string, not a number.
Type of change

Bug fix (non-breaking change which fixes an issue)

Checklist

[x] My code follows the style guidelines of this project
[x] I ran ESLint and other linters for modified files
[x] I have performed a self-review of my own code and tested it
[x] I have commented my code, particularly in hard-to-understand areas
(including JSDoc for methods)
[x] My changes generate no new warnings
[ ] My code needed automated testing. I have added them (this is optional task)

Screenshots (if any)

2022-05-01T09:45:41.348Z [PING_CHART] DEBUG: this.chartPeriodHrs type number, value: 0
...
2022-05-01T09:49:16.644Z [PING_CHART] DEBUG: this.chartPeriodHrs type string, value: 0

I found that sometimes it could be number type.

https://github.com/louislam/uptime-kuma/commit/a12dffd1bc6bb9955f1d66ab1f9c1d269c051bb5
Maybe it is not worth to spend time on fixing this, I fallback to == with eslint-disable-next-line.


[feature] I don't want an item and a ghost of the item

I don't know if this is possible with the existing options but I couldn't figure it out. I saw that there's a ghost class option, but I think that's just for modifying the ghost, not for doing away with it?
I am sorting a row of images. When I start dragging an image, what I drag is a ghost of the image. The "real" image itself stays in place until I've dragged far enough that I'm over a new position, at which time the "real" image moves (with an animation if I've chosen).
The thing is, I don't want to drag a ghost and have the real image "follow along". I want to actually drag the real image. When I've dragged it far enough, I want the other items to move, showing the space where my image would go. But I don't want the dragged image to snap to that space until I release the image. (At which point it would animate nicely to the snapped position.)
It seems to me that what I'm describing is the standard behavior that I'm used to across apps, operating system functions, etc.
Does sortablejs always do the "ghost follow-along" thing, or is there an option to show only one copy (i.e., the one "real" item, from the user's point of view) of the item I'm dragging, with a float to the snapped position when I release?

I realize I was being dumb about one thing. The "ghost" I'm talking about is the browser thinking I'm dragging the image. But when I disable the browser drag of the image
e.on( "dragstart", function( e ) { e.preventDefault(); } );
it kills the drag of sortablejs also. Anyway, I still don't know how to get the behavior described above, but I'll submit another issue once I try to work around the browser not dragging the image (but still allowing sortable to drag the item).


Add a vision document

This PR adds the 99-vision.md document, which outlines our larger vision and asks for community contributions to the compression requirements.

I am just wondering about the fields in the google docs table.
How were they selected?
Aren't some missing? Like e.g. 2 metre temperature

I tried to combine the levels from everything in ERA5. Since there were near duplicates, I combined fields (e.g. 2m temperature surface field with multi-level temperature) as the requirements should be at least very similar.
If you can think of extra rows to add, please add them :)

How are the levels set and what does single level mean? E.g. Specific rain water content is a model level field in the IFS.

Single level usually means surface level (from the ERA5 single level dataset). Should we rename it?
If the field makes sense on more than a single level, you can change it.

Is there anything we should change about this Markdown document or that we should add to the table?

I am just wondering about the fields in the google docs table.
How were they selected?
Aren't some missing? Like e.g. 2 metre temperature

I tried to combine the levels from everything in ERA5. Since there were near duplicates, I combined fields (e.g. 2m temperature surface field with multi-level temperature) as the requirements should be at least very similar.
If you can think of extra rows to add, please add them :)

How are the levels set and what does single level mean? E.g. Specific rain water content is a model level field in the IFS.

Single level usually means surface level (from the ERA5 single level dataset). Should we rename it?
If the field makes sense on more than a single level, you can change it.

Ok, I will check what is missing and add it.

Is there anything we should change about this Markdown document or that we should add to the table?

No, I think it looks good. I wouldn't add the table.

I am just wondering about the fields in the google docs table.
How were they selected?
Aren't some missing? Like e.g. 2 metre temperature

I tried to combine the levels from everything in ERA5. Since there were near duplicates, I combined fields (e.g. 2m temperature surface field with multi-level temperature) as the requirements should be at least very similar.
If you can think of extra rows to add, please add them :)

How are the levels set and what does single level mean? E.g. Specific rain water content is a model level field in the IFS.

Single level usually means surface level (from the ERA5 single level dataset). Should we rename it?
If the field makes sense on more than a single level, you can change it.

I added some more Model level fields and renamed Single to Single/Surface. Furthermore, I added the label Pressure+Model and marked fields that occur on both level types with that.
@juntyr In the vision document, we maybe should explicitly encourage scientists to add new fields to the table.


How to enumerate audio devices in Windows XP

Is there any way to use this library to enumerate audio input devices in Windows XP?  I'm trying to use the MMDeviceEnumerator?
Thanks!

I'm afraid you can't use MMDeviceEnumerator with Windows XP, as that uses an API new to Windows Vista. For XP you should use the WaveIn class.


[DOC] Update changelog

Reference Issues/PRs
Fixes #1723
What does this implement/fix? Explain your changes.
Update changelog

pushing changelog through without review in agreement with @mloning at release meeting, so we have something there for release. We can still change this tomorrow.


⚠ Bofrost.lu has degraded performance

In 41b7ad3, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1248 ms

Resolved: Bofrost.lu performance has improved in 72c9e85 after 17 minutes.


Wrong conversion from RT90 to WSG84

Not sure if this a pyproj or proj4 bug. There seems to be wrong conversion results when converting from RT90 coordinate system (used in Sweden) to WGS84. Code to reproduce:
rt90Proj = Proj(init='epsg:3021')
wgs84 = Proj(init="epsg:4326")
print(transform(rt90Proj, wgs84, 6266075, 1536570))

Output:
(55.89475274389442, 10.703814100405289)

Desired output:
(56.520780, 16.399381)

I verified it with numerous online conversions. And I also now the correct location of the object. It is indeed (56.52, 16.40).

It looks like PROJ 4 version 4.9.3 is giving the same results.
C:\Users\mcochran\Desktop>cs2cs
Rel. 4.9.3, 15 August 2016

C:\Users\mcochran\Desktop>cs2cs +init=epsg:3021 +to +init=epsg:4326 -f %.12f -v
# ---- From Coordinate System ----
#Transverse Mercator
#       Cyl, Sph&Ell
# +init=epsg:3021 +proj=tmerc +lat_0=0 +lon_0=15.80827777777778 +k=1
# +x_0=1500000 +y_0=0 +ellps=bessel
# +towgs84=414.1,41.3,603.1,-0.855,2.141,-7.023,0 +units=m +no_defs
# ---- To Coordinate System ----
#Lat/long (Geodetic alias)
#
# +init=epsg:4326 +proj=longlat +datum=WGS84 +no_defs +ellps=WGS84
# +towgs84=0,0,0
6266075 1536570
55.894752743894 10.703814100405 -363.890933852643

The inverse conversion has the same issues.

Thanks. I've reported it to Proj4 project.


BoxedMontyForm::pow() gives only one ?

I'm struggling trying to use BoxedMontyForm pow() but it keeps giving one ?
e.g. trying 2^16 == 65536
let two = BoxedUint::from(2 as u8);
let three = BoxedUint::from(3 as u8);
let boxed_16 = BoxedUint::from(16 as u8);
let two_monty = BoxedMontyForm::new(two, BoxedMontyParams::new(Odd::new(three).unwrap()));

dbg!(two_monty.pow(&boxed_16).retrieve());

I wonder why is it giving me BoxedUint::one() ?
src/main.rs:18:5] two_monty.pow(&boxed_16).retrieve() = BoxedUint(0x0000000000000001)

It should give
BoxedUint(0x0000000000010000)

Oh this was my epic misunderstanding, I should have just used U256 pow - not sure why I did modpow on this lol


cilium: endpoint routes + veth fast redirect support

Both settings should work with fast redirect for 5.10+.

@borkmann Is the PR #14382 related to this issue?

@borkmann Is the PR #14382 related to this issue?

@lyveng It is, and it handles the endpoint routes case.

@lyveng It is, and it handles the endpoint routes case.

@borkmann Is the PR #14382 related to this issue?

Yeah, still needs more work though since policy case does not fully function there yet. Will resume in Jan.

@borkmann Is the PR #14382 related to this issue?

Yeah, still needs more work though since policy case does not fully function there yet. Will resume in Jan.

any updates on this?

any updates on this?

PR for testing is here: https://github.com/cilium/cilium/issues/14240

When is this going to be upstreamed? I anticipated 1.13.2 but it looks like it's still not here.

Hi @buroa, it has been merged into Cilium OSS master branch ~2 weeks ago, so it can be tested via images built from master (https://github.com/cilium/cilium#development) and next v1.14.0-snapshot.2 once the latter is out.


[Bug] Cannot read api schema in pipeline menu

pipeline.js:303 Uncaught TypeError: Cannot read properties of undefined (reading 'content')
    at showpipelineSettingsModal (pipeline.js:303:54)
    at HTMLButtonElement.<anonymous> (pipeline.js:293:5)

Are you running any nginx layer or non default port? Any unique Auth etc?

I'm running the latest exeversion in release in win10. I think it is just a bug? There is no apiDocs[operation].post.requestBody, you need to use apiDocs[operation].post.response instead.

the post looks like
{
    "tags": [
        "Misc"
    ],
    "summary": "Extract images from a PDF file",
    "description": "This endpoint extracts images from a given PDF file and returns them in a zip file. Users can specify the output image format. Input:PDF Output:IMAGE/ZIP Type:SIMO",
    "operationId": "extractImages",
    "parameters": [
        {
            "name": "request",
            "in": "query",
            "required": true,
            "schema": {
                "$ref": "#/components/schemas/PDFWithImageFormatRequest"
            }
        }
    ],
    "responses": {
        "200": {
            "description": "OK",
            "content": {
                "*/*": {
                    "schema": {
                        "type": "array",
                        "items": {
                            "type": "string",
                            "format": "byte"
                        }
                    }
                }
            }
        }
    }
}```


End of deployment telemetry doesn't fail silently

Which version of kubefirst are you using?
2.0.1
Which installation type?
None specific
Which distributed Git provider?
None specific
What is the issue?
End of deployment telemetry doesn't fail silently if segment.io cannot be reached for any reasons. We had that issue for beginning of the installation a while ago, and it was fixed, but I assumed we added new telemetry calls.
segment 2023/04/17 11:41:51 ERROR: sending request - Post "https://api.segment.io/v1/batch": dial tcp <IP_ADDRESS>:443: connect: bad file descriptor                                                                                ╰──────╯
segment 2023/04/17 11:41:51 ERROR: sending request - Post "https://api.segment.io/v1/batch": dial tcp <IP_ADDRESS>:443: connect: bad file descriptor
segment 2023/04/17 11:41:51 ERROR: sending request - Post "https://api.segment.io/v1/batch": dial tcp <IP_ADDRESS>:443: connect: bad file descriptor
segment 2023/04/17 11:41:51 ERROR: sending request - Post "https://api.segment.io/v1/batch": dial tcp <IP_ADDRESS>:443: connect: bad file descriptor
segment 2023/04/17 11:41:52 ERROR: sending request - Post "https://api.segment.io/v1/batch": dial tcp <IP_ADDRESS>:443: connect: bad file descriptor
segment 2023/04/17 11:41:54 ERROR: sending request - Post "https://api.segment.io/v1/batch": dial tcp <IP_ADDRESS>:443: connect: bad file descriptor
segment 2023/04/17 11:41:57 ERROR: sending request - Post "https://api.segment.io/v1/batch": dial tcp <IP_ADDRESS>:443: connect: bad file descriptor
segment 2023/04/17 11:42:03 ERROR: sending request - Post "https://api.segment.io/v1/batch": dial tcp <IP_ADDRESS>:443: connect: bad file descriptor
segment 2023/04/17 11:42:13 ERROR: sending request - Post "https://api.segment.io/v1/batch": dial tcp <IP_ADDRESS>:443: connect: bad file descriptor
segment 2023/04/17 11:42:23 ERROR: sending request - Post "https://api.segment.io/v1/batch": dial tcp <IP_ADDRESS>:443: connect: bad file descriptor
segment 2023/04/17 11:42:33 ERROR: 1 messages dropped because they failed to be sent after 10 attempts

Code of Conduct

[X] I agree to follow this project's Code of Conduct

It was fixed.


executor/common_bsd.h: Add network packet injection for NetBSD

Before sending a pull request, please review Contribution Guidelines:
https://github.com/google/syzkaller/blob/master/docs/contributing.md

@krytarowski Can you take a look?

Codecov Report

Merging #1286 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1286   +/-   ##
=======================================
  Coverage   67.49%   67.49%
=======================================
  Files         120      120
  Lines       21005    21005
=======================================
  Hits        14177    14177
  Misses       6154     6154
  Partials      674      674

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 e6fb0f1...cf719fb. Read the comment docs.

looks good


Fixed bug where inheriting an abstract class with a DataObject breaks /dev/build

Fixed bug where inheriting an abstract class with a DataObject breaks /dev/build

I remember trying to solve this issue before in the same way. ;) Unfortunately it's not quite as simple as this.

I think we need to fix this in master, since we can't properly support abstract data objects without a more significant rewrite. For the time being, mandating non-abstract dataobjects in 3.x will probably stick around. :)
If you want to take this on, feel free, and I'll do my best to review. :D


Doc: SqlFirehose(Postgres) to Input source.

Since Firehoses are deprecated, needed guidance in finding doc for converting PostgreSQL SqlFirehose to Input source.

Hi @pktippa, the SqlFirehose is not deprecated yet. I admit the doc (https://druid.apache.org/docs/0.17.0/ingestion/native-batch.html#sqlfirehose) is confusing. It should be fixed.

thanks @jihoonson


Cursor Issue

Challenge Taste the Bootstrap Button Color Rainbow has an issue.
User Agent is: Mozilla/5.0 (Windows NT 10.0; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/52.0.2743.116 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

<link href="https://fonts.googleapis.com/css?family=Lobster" rel="stylesheet" type="text/css">

CatPhotoApp

Click here for cat photos.

  <button class="btn btn-block">Like</button>

Things cats love:

cat nip

laser pointers

lasagna

Top 3 things cats hate:

flea treatment

thunder

other cats

  <form action="/submit-cat-photo">
    <input type="radio" name="indoor-outdoor"> Indoor
    <input type="radio" name="indoor-outdoor"> Outdoor
    <input type="checkbox" name="personality"> Loving
    <input type="checkbox" name="personality"> Lazy
    <input type="checkbox" name="personality"> Crazy
    <input type="text" placeholder="cat photo URL" required>
    <button type="submit">Submit</button>
  </form>

@Jenny1985 unfortunately, this is a known error https://github.com/FreeCodeCamp/FreeCodeCamp/issues/7847. Check out the discussion over in that issue for some ways around it. It is caused by the editor we are using and it has issues with wide code in the editor. I hope that helps and sorry for the inconvenience. Happy coding!


Swift: adopt new hashing thing instead of hashValue

hashValue no longer appears anywhere in the code, including submodules.


Error: can't find python executable

Node Version: 9.8.0
Platform: Windows

Verbose output (from npm or node-gyp):

You need Python 2.7, 3.x.x is not supported.

I had same issue and same node version, it got resolved after downgrading node to version 8.11.1.
https://github.com/nodejs/node-gyp/issues/809

Same issue.


Ability to ignore/exclude a specific plugin in redundantPlugins check

Is your feature request related to a problem? Please describe.

I want to enable redundantPlugins check for all modules:
dependencyAnalysis {
    issues {
        all {
            onRedundantPlugins {
                severity("fail")
            }
        }
    }
}

For some modules, I got false-positive errors: #392, #411
I'm ok with it. False-positive is inevitable for any linter tool.
Describe the solution you'd like

I'd like to ignore a specific plugin in a module.
Probable API:
project(":problematic-module") {
   issues {
            onRedundantPlugins {
                exclude("java-library") // `java-library` is added implicitly by `java-gradle-plugin`
            }
    }
}

Describe alternatives you've considered

I have to ignore the check:
project(":problematic-module") {
   issues {
            onRedundantPlugins {
                severity("ignore")
            }
    }
}

The task is not failing but still prints the error:
Plugin advice:
- java-library: this project has both java-library and org.jetbrains.kotlin.jvm applied, which is redundant. You can remove java-library

Additional context

Plugin version: 0.74.0
You can reproduce it in this project: https://github.com/avito-tech/avito-android/tree/dependency-analysis-reproducer-427
./gradlew -p subprojects :gradle:build-properties:projectHealth

Thanks for the report. The printed error looks like a bug, but the rest is a FR. I think it makes sense.

Should I create a separate issue for printed error? It looks like a good first issue.

Sure, that makes sense.

#431

I'm experiencing the same issue with kotlin-kapt.

side note on this, for some of my modules kotlin-kapt is reported as redundant but is required when building, I'll raise an issue if I can isolate in a reproducible project.

@KatieBarnett you might be facing https://github.com/autonomousapps/dependency-analysis-android-gradle-plugin/issues/411
I would certainly appreciate a reproducer as that would be key in helping to resolve this issue.


Allow only running a specific set of tests

Describe the problem you are trying to solve
It's currently not possible (to my understanding) to filter out multiple tests with Cargo. It would be nice to be able to rerun all the tests in a CI build that failed, without running all the passing tests too.
Describe the solution you'd like
A --tests option that lets you pass a list of tests to run, excluding all other tests. Although I would like it if it were not limited to just integration tests (as --test is), and have it work on unit tests too. If I understand correctly, this would be the equivalent of passing --test to the underlying test runner, except that the test runner does not allow multiple --test options:
error: Option 'test' given more than once

Also it would be nice if at the end of a test run where there were failed tests, Cargo printed out the command that you can run to only re-run the failed tests.
Notes
I find the --test option confusing as it seems to only run integration tests. Could this be changed to allow you to use it with unit tests as well?
See https://rust-lang.zulipchat.com/#narrow/stream/122651-general/topic/.60cargo.20test.20--test.20foo.60.20for.20multiple.20tests for more context.
Cc @jyn514

Minor correction: cargo test test_name passes test_name to the test runner, not --test test_name.

Maybe this should be a feature request for libtest? The bit about --test only working for integration tests is a good one though.

There is an important distinction about --test vs a test name filter, and they can't really be unified (cargo has no insight into where tests are defined).  One way to think of it is like an onion, such as:
workspace > package > target crate > modules > #[test] function

-p flag determines the workspace and package (defaults to the package in the current directory, or the entire workspace in the root of a virtual workspace), the target selection flags select the target (--lib, --bin, --example, --test, etc.), and then the actual test name can be passed as an argument like cargo test my_fancy_test.  If you don't pass a target flag, Cargo has no choice but to build all targets and pass the filter to all of them.
As for passing multiple filters, that is part of the libtest harness which lives in the rust-lang/rust repository. The issue tracking that is https://github.com/rust-lang/rust/issues/30422.  I personally would like to see support for multiple filters added, and I think it should be relatively easy.  Unfortunately there isn't any active work or maintenance happening with libtest, but if you're interested you could maybe see if the libs team would accept/review such a change.  dtolnay (a libs team member) mentioned here that they would be ok with it, although that was 3 years ago, so things might have changed.

#7864 was our issue for --test and <TEST_NAME> being confused which was fixed in #12915.
rust-lang/rust#30422 was closed.
So it seems like there is nothing left for this issue

It may be better if it supports cargo test --test TESTNAME1, TESTNAME2, FOO::TESTNAME3.

Having --test accept test names, in place of test targets, is potentially breaking could be quite confusing.

From this, I feel like should there be a parameter to support re-running test cases that failed in the previous round of testing?
like：cargo test --rerun-failed-tests

Huh, not seeing an issue for this.  There'd be a lot of design issues to work out but its useful.

Huh, not seeing an issue for this. There'd be a lot of design issues to work out but its useful.

#10673?


Unused interfaces are not shown in graphviz UI

Originally reported on Google Code with ID 664
Owned by vivianxinxinkou

With the following mode, you will see the Person interface
interface Person {}
class Mentor { isA Person; }
But, if the interface is unused you don't see it

Reported by @aforward on 2015-01-16 15:02:37

I've checked both graphical representation and it looks like the bug has been fixed.


Contact us UI

Kindly assign me the task to improve Contact us for UI

Assign me this under hactoberfest and ggsoc-ext

@tushargupta1504 Assign this to me under hacktoberfest and gssoc-extd. I'll make it look better.

pls assign me this issue I would like to improve its UI @tushargupta1504


Improve 4.0 upgrading guide with Robbie's feedback

Robbie's notes at  https://docs.google.com/document/d/1DI3Cf111SWyEVdIUENSRaYGt3pwBR9FwF5ZyYZlF02w/edit#heading=h.vf0es2ofzk5e
Reflect those in https://github.com/silverstripe/silverstripe-framework/blob/master/docs/en/04_Changelogs/4.0.0.md

Ensure you mention DataObject ownership more clearly (particularly since you have to change any objects relating to File in your code now): https://docs.silverstripe.org/en/4/developer_guides/model/versioning/#dataobject-ownership


Error while consuming the lib from another Angular4 project

Hi,
Thanks for building such a great quick-start library repo. It includes almost every aspects I can think of. I ran into a problem and struggled for a couple of days. Hope someone can help.
I follow the guide and
npm run build
then I want consume it in my another project by
npm install --save my/local/path/to/angular-quickstart-lib/dist
then in app.module.ts, I do:
import { BrowserModule } from '@angular/platform-browser';
import { NgModule } from '@angular/core';
import { LibModule } from 'quickstart-lib';

import { AppComponent } from './app.component';

@NgModule({
  declarations: [
    AppComponent
  ],
  imports: [
    BrowserModule,
    LibModule
  ],
  providers: [],
  bootstrap: [AppComponent]
})
export class AppModule { }

then I use
<my-lib></my-lib>
in app.component.html
Run my project, in the browser console it shows:
Uncaught Error: Unexpected value 'LibModule' imported by the module 'AppModule'. Please add a @NgModule annotation.
    at syntaxError (compiler.es5.js:1689)
    at compiler.es5.js:15373
    at Array.forEach (<anonymous>)
    at<EMAIL_ADDRESS>(compiler.es5.js:15356)
    at<EMAIL_ADDRESS>(compiler.es5.js:26679)
    at<EMAIL_ADDRESS>(compiler.es5.js:26652)
    at<EMAIL_ADDRESS>(compiler.es5.js:26581)
    at<EMAIL_ADDRESS>(core.es5.js:4595)
    at<EMAIL_ADDRESS>(core.es5.js:4581)
    at Object.../../../../../src/main.ts (main.ts:11)

I am using the same version of Angular (4.2.5) to avoid potential failure (but I do not think it is the root cause). Do you have any ideas?
Actually I tried to build a library without using any seeds previously, following the steps of ngc, rollup, etc. and it ends up with this same problem.
Node version: 8.0.0
Npm version: 5.0.1
Typescript version: 2.3.4

Please check if  the lib is present in your node_modules folder

Hi @zigzag95, it's there. The directory is like:
node_modules/
├── ...
├── quickstart-lib/
|  └── bundles/
|  | ├── quickstart-lib.umd.js
|  | ├── quickstart-lib.umd.js
|  | ├── quickstart-lib.umd.min.js
|  | └── quickstart-lib.umd.min.js.map
|  └── src/
|  |  ├── component/
|  |  |  └── lib.component.d.ts
|  |  ├── service/
|  |  |  └── lib.service.d.ts
|  |  └── module.d.ts
|  └── index.d.ts
|  └── quickstart-lib.d.ts
|  └── quickstart-lib.es5.js
|  └── quickstart-lib.es5.js.map
|  └── quickstart-lib.metadata.json
|  └── LICENSE
|  └── package.json
|  └── ...
├── ...

The doc from jvandemo/generate-angular2-library saved me.
Turns out that I forgot to set up the path mapping in /src/tsconfig.app.json in my "Angular CLI-generated" consuming application (not library).
{
  "compilerOptions": {
    // ...
    "paths": {
      "@angular/*": [
        "../node_modules/@angular/*"
      ]
    }
  }
}

I have path mapping in /src/tsconfig.app.json but still getting this error, Also tried ng serve --preserve-symlinks


bump alpine image

/hold

#58 does this along fixing thr build with 3.15. We should probably update that PR.

#58 does this along fixing the build with 3.15. We should probably update that PR.

@unleashed just noticed your previous PR too, would it make sense to bump alpine to 3.16 instead of 3.15?

#58 does this along fixing the build with 3.15. We should probably update that PR.

@unleashed just noticed your previous PR too, would it make sense to bump alpine to 3.16 instead of 3.15?

Definitely.

merged #58, please rebase and update both builder and runtime image


[Autocomplete] onUpdateInput does not support Japanese

Problem description
Autocomplete onUpdateInput does not support Japanese.
For example, when entering "か", "k a" is entered, but events occur at k and a respectively.
As a result, "kあ" is displayed.I want you to be displayed "か".
Link to minimal working code that reproduces the issue
<AutoComplete
  anchorOrigin={{ vertical: 'top', horizontal: 'left'}}
  targetOrigin={{ vertical: 'bottom', horizontal: 'left' }}
  menuProps={{maxHeight: 100}}
  filter={AutoComplete.noFilter}
  fullWidth={true}
  searchText={this.state.text}
  dataSource={this.dataSouce}
  multiLine={true}
  value={this.state.text}
  style={styles.textarea}
  underlineShow={false}
  onUpdateInput={this.handleForm.bind(this)}
/>
handleForm(value){
    this.setState({
      text: value,
    });
}

Versions

Material-UI:0.17.3
React:15.5.4
Browser:Google Chrome 58.0.3029.81

Images & references

I would like to enter "こんにちは"....

Hey @1234224576, this is a really odd issue, <AutoComplete /> simply passes the event's target.value to the function you supplied: https://github.com/callemall/material-ui/blob/master/src/AutoComplete/AutoComplete.js#L346-L348.
Would you be able to provide a running case on https://www.webpackbin.com/bins/-Kh7G86UTg0ckGC2hL94?

Closed by #4783


Streamlit App for Depression Support  Issue no #99

This pull request aims to integrate an NLP-based chatbot, designed to provide support for users dealing with depression, into a Streamlit application. The chatbot leverages the power of SpaCy for natural language processing and a pre-defined dataset for generating responses. A detailed video presentation demonstrating the features and functionality of the integrated chatbot is attached. The video showcases the user interface, the chatbot's responses, and the overall user experience:
https://github.com/TAHIR0110/ThereForYou/assets/109272714/5567db44-1823-4cb5-987b-c58c2d3cb2a5
I am Participating in GSSOC'24. Please Label this PR as Level 3

@revanth1718 merged!

Thank you

@TAHIR0110 @revanth1718 This code needed to be cleaned before merging it,
this is a dirty commit, as it contains cache files and a virtual python environment, consider having an .gitignore file.

I have made a pr cleaning this thing and added black formatting changes.
#136


[Fix #9433] Fix an error for Style/EvalWithLocation

Fixes #9433
This PR fixes an error for Style/EvalWithLocation when using eval with block argument.

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

Thanks @koic!

Thanks @koic!


[TFLite] Add feature to access tflite::LoggerOptions from C API

Overview
Issue: https://github.com/tensorflow/tensorflow/issues/74882
This PR will provide C API with a more detailed or suppressed way of logging by adding feature to access tflite::LoggerOptions from C API.
Here is the code to verify the working of this PR.
Changes

Added TfLiteLogSeverity enum corresponding to tflite::LogSeverity enum
Added binding functions for tflite::LoggerOptions::GetMinimumLogSeverity() and tflite::LoggerOptions::SetMinimumLogSeverity()
Reflect in build config to explicitly use newly included files
Added unit test

Review points

New APIs are implemented in the C experimental API since tflite::LoggerOptions is described as an experimental API in logger.h.
I could not use the tflite::LogSeverity enum definition directly because logger.h is a C++ header, so I had to define a new corresponding TfLiteLogSeverity enum in c_api_experimental.h.

To prevent the two definitions from mismatching, I added LINT.IfChange like the TfLiteType.
Like other C APIs, I named constants in the enum like constant names (e.g., kFooBar). However, constants in tflite::LogSeverity are named like macro (e.g., FOO_BAR).

Is testing required? I added a test as well as minimal_logging_test.cc, but often there are no tests for other functions that just call the C++ API.
There is a part in c_api_experimental_test.cc that is changed by clang-format, so I have applied it only to the part I implemented in this PR.

Hi @yishuangP, Can you please review this PR? Thank you !

Hi @yishuangP, Can you please review this PR? Thank you !

Hi @kokeshing, Can you please resolve the conflicts? Thank you!

Hi @haozha111, Could you please review this PR ? Thank you .


How dream fusion that takes input as an image

Hello,
Is it possible, instead of text to generate 3D image. How about If i upload any other image instead and use dream fusion to convert that image to 3D.
Is this possible or certain tweak can be applied, please let me know

@shanalikhan Hi, as far as I know, there are already several papers related to this:

https://github.com/VITA-Group/NeuralLift-360
https://github.com/lukemelas/realfusion

Hi @shanalikhan We've released our implementation that supports image-to-3D (based on the current repo) https://github.com/VITA-Group/NeuralLift-360

I found another implementation that generates 3D models from images; it uses Stable Diffusion, but not Stable-DreamFusion.