Add support for flannel and calico.

Default remains weave. User can now set --cni-model [weave|flannel|calico]
on create. For flannel and calico one must also set pod-network-cidr
(as per https://kubernetes.io/docs/setup/independent/create-cluster-kubeadm/).
Add ability to set cluster-cidr.

OK. I made the above changes, I did some fairly simple testing w/ calico + flannel, they seem to work ok.

Thanks.
Now coding style issues are gone. Looks good.
However, first of all, can you please squash commits to a single one? Previously it was one commit, but now it has 4 commits, each of which has the same commit message.
As for rktlet, my co-worker said that it's not supposed to be a blocker anyway, because rktlet has other big issues with k8s >= 1.10. So I think it's ok to merge it without testing it with rktlet.
With container runtime being only Docker, I've tested with the following combinations:

Kubernetes: v1.9, v1.10, v1.11
CNI model: weave, calico, flannel.

As for weave, everything works fine.
As for calico, k8s v1.9 and v1.10 work fine, though k8s v1.11 does not work. It has something to do with coredns.
$ kubectl --namespace=kube-system describe pod coredns-78fcdf6894-cmrdl
...
1m            1m              4       kubelet, kube-spawn-default-master-q4s7vo                       Warning               FailedCreatePodSandBox  (combined from similar events): Failed create pod sandbox: rpc error: code = Unknown desc = failed to set up sandbox container "ff6f22a43fedad0c7f6b43578f7d3c2ee309f2dbd48db093364c43577691719e" network for pod "coredns-78fcdf6894-cmrdl": NetworkPlugin cni failed to set up pod "coredns-78fcdf6894-cmrdl_kube-system" network: no podCidr for node kube-spawn-default-master-q4s7vo
...

As for flannel, when testing with k8s v1.10 and v1.11, it fails with the following error:
$ kubectl --namespace=kube-system describe pod kube-dns-6f4fd4bdf-8fjlv
...
  2m            2m              12      kubelet, kube-spawn-default-master-uj5epv                       Warning               FailedCreatePodSandBox  Failed create pod sandbox: rpc error: code = Unknown desc = NetworkPlugin cni failed to set up pod "kube-dns-86f4d74b45-8r4k8_kube-system" network: open /run/flannel/subnet.env: no such file or directory
...

Right now I have no time to have a deeper look.
Do you have any idea how these errors should be resolved?

OK, have made above changes.
have rebased (note: in other projects they usually object to rebasing and do a squash when they accept the code).
as for the flannel 'open /run/flannel/subnet.env' error, this implies that you did not pass the --pod-network-cidr usually (https://github.com/kubernetes/kubernetes/issues/36575).
Iet me try w/ 1.11 to see
as for the 'the cni plugin dir is /go/bin, can you filter the right files', I' not sure how to approach this, I could hard-code a list of the current cni plugins i guess.

Thanks @donbowman

as for the 'the cni plugin dir is /go/bin, can you filter the right files', I' not sure how to approach this, I could hard-code a list of the current cni plugins i guess.

Yes, a map[string][]string that holds the names of the required files per plugin is OK I think.
var cniFiles = map[string][]string{
	"flannel": {"..."},
	"weave":   {"..."},
        ....
}

ok map added.


Incrementing minor version with each release

Currently, the build number is incremented with each release in line with the recommended PCF deployment practices.
The solution is version stamped with the same version as the PCFs.
To make it easier for forks to build/test new versions, what do you think about moving the release increment to the minor version instead? This will make it easier for branches/forks to deploy/test new versions without preventing future import version collisions due to the local version being higher than the new release version.
E.g.

Released version -> 1.8.0
Forked/branch version deployed to test environment ->1.8.1 -> 1.8.2 etc.
New version released ->1.9.0
Forked/branch version deployed to test environment ->1.9.1 -> 1.9.2 etc.
And so on....

This would need a change to the release workflow, moving the incrementing version to the minor variable, rather than the build version.
Thoughts?

Completely agree on this. Moving to minor version instead of major for incremental release will be ideal.


fix: wrong the detection method of nilable

fix #339

Codecov Report

Merging #375 (2ea7ab6) into master (3fdc55a) will decrease coverage by 0.05%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #375      +/-   ##
==========================================
- Coverage   79.95%   79.90%   -0.06%
==========================================
  Files          52       52
  Lines       17555    17561       +6
==========================================
- Hits        14036    14032       -4
- Misses       2951     2960       +9
- Partials      568      569       +1


anvil: no withdrawalsRoot returned in block data

adding to this--how do I know if anvil is running --shanghai by default? when i do cast block against either anvil or anvil --hardfork shanghai, I am not seeing withdrawalsRoot returned in the block data.
Originally posted by @d1onys1us in https://github.com/foundry-rs/foundry/issues/4916#issuecomment-1587153571

perhaps we're missing formatting for this?

Yeah probably—I'm pretty sure the data is there, just might not be showing up


[Bug]: Openhands container shuts down immediately (UID not unique)

Is there an existing issue for the same bug?

[X] I have checked the existing issues.

Describe the bug and reproduction steps
I ran make start-backend; openhands was unable to connect to the docker container and exceed max retries. Upon further inspection it seems the container immediately shut down shortly after it is created. I ran docker logs d9af0976fdb54a78b216f16ec8602a5541eac1ca2f5c6345b54b2251a2ce3a65 for more info and found the following stack trace
INFO:     Started server process [8]
INFO:     Waiting for application startup.
ERROR:    Traceback (most recent call last):
  File "/openhands/poetry/openhands-ai-5O4_aCHf-py3.12/lib/python3.12/site-packages/starlette/routing.py", line 693, in lifespan
    async with self.lifespan_context(app) as maybe_state:
               ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/openhands/micromamba/envs/openhands/lib/python3.12/contextlib.py", line 210, in __aenter__
    return await anext(self.gen)
           ^^^^^^^^^^^^^^^^^^^^^
  File "/openhands/code/openhands/runtime/action_execution_server.py", line 347, in lifespan
    client = ActionExecutor(
             ^^^^^^^^^^^^^^^
  File "/openhands/code/openhands/runtime/action_execution_server.py", line 97, in __init__
    _updated_user_id = init_user_and_working_directory(
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/openhands/code/openhands/runtime/utils/runtime_init.py", line 100, in init_user_and_working_directory
    raise RuntimeError(
RuntimeError: Failed to create user `openhands` with UID 0. Output: [useradd: UID 0 is not unique
]

ERROR:    Application startup failed. Exiting.

08:52:31 - openhands:ERROR: agent_session.py:197 - Runtime initialization failed: HTTPConnectionPool(host='localhost', port=31482): Max retries exceeded with url: /alive (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x12a0e4bc0>: Failed to establish a new connection: [Errno 61] Connection refused'))
Traceback (most recent call last):
  File "/Users/rohitmalhotra/Library/Caches/pypoetry/virtualenvs/openhands-ai-nue4wPdC-py3.12/lib/python3.12/site-packages/urllib3/connection.py", line 199, in _new_conn
    sock = connection.create_connection(
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/rohitmalhotra/Library/Caches/pypoetry/virtualenvs/openhands-ai-nue4wPdC-py3.12/lib/python3.12/site-packages/urllib3/util/connection.py", line 85, in create_connection
    raise err
  File "/Users/rohitmalhotra/Library/Caches/pypoetry/virtualenvs/openhands-ai-nue4wPdC-py3.12/lib/python3.12/site-packages/urllib3/util/connection.py", line 73, in create_connection
    sock.connect(sa)
ConnectionRefusedError: [Errno 61] Connection refused

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

Traceback (most recent call last):
  File "/Users/rohitmalhotra/Library/Caches/pypoetry/virtualenvs/openhands-ai-nue4wPdC-py3.12/lib/python3.12/site-packages/urllib3/connectionpool.py", line 789, in urlopen
    response = self._make_request(
               ^^^^^^^^^^^^^^^^^^^
  File "/Users/rohitmalhotra/Library/Caches/pypoetry/virtualenvs/openhands-ai-nue4wPdC-py3.12/lib/python3.12/site-packages/urllib3/connectionpool.py", line 495, in _make_request
    conn.request(
  File "/Users/rohitmalhotra/Library/Caches/pypoetry/virtualenvs/openhands-ai-nue4wPdC-py3.12/lib/python3.12/site-packages/urllib3/connection.py", line 441, in request
    self.endheaders()
  File<EMAIL_ADDRESS>line 1331, in endheaders
    self._send_output(message_body, encode_chunked=encode_chunked)
  File<EMAIL_ADDRESS>line 1091, in _send_output
    self.send(msg)
  File<EMAIL_ADDRESS>line 1035, in send
    self.connect()
  File "/Users/rohitmalhotra/Library/Caches/pypoetry/virtualenvs/openhands-ai-nue4wPdC-py3.12/lib/python3.12/site-packages/urllib3/connection.py", line 279, in connect
    self.sock = self._new_conn()
                ^^^^^^^^^^^^^^^^
  File "/Users/rohitmalhotra/Library/Caches/pypoetry/virtualenvs/openhands-ai-nue4wPdC-py3.12/lib/python3.12/site-packages/urllib3/connection.py", line 214, in _new_conn
    raise NewConnectionError(
urllib3.exceptions.NewConnectionError: <urllib3.connection.HTTPConnection object at 0x12a0e4bc0>: Failed to establish a new connection: [Errno 61] Connection refused

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

Traceback (most recent call last):
  File "/Users/rohitmalhotra/Library/Caches/pypoetry/virtualenvs/openhands-ai-nue4wPdC-py3.12/lib/python3.12/site-packages/requests/adapters.py", line 667, in send
    resp = conn.urlopen(
           ^^^^^^^^^^^^^
  File "/Users/rohitmalhotra/Library/Caches/pypoetry/virtualenvs/openhands-ai-nue4wPdC-py3.12/lib/python3.12/site-packages/urllib3/connectionpool.py", line 843, in urlopen
    retries = retries.increment(
              ^^^^^^^^^^^^^^^^^^
  File "/Users/rohitmalhotra/Library/Caches/pypoetry/virtualenvs/openhands-ai-nue4wPdC-py3.12/lib/python3.12/site-packages/urllib3/util/retry.py", line 519, in increment
    raise MaxRetryError(_pool, url, reason) from reason  # type: ignore[arg-type]
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
urllib3.exceptions.MaxRetryError: HTTPConnectionPool(host='localhost', port=31482): Max retries exceeded with url: /alive (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x12a0e4bc0>: 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/rohitmalhotra/Documents/Projects/OpenHands/openhands/server/session/agent_session.py", line 195, in _create_runtime
    await self.runtime.connect()
  File "/Users/rohitmalhotra/Documents/Projects/OpenHands/openhands/runtime/impl/eventstream/eventstream_runtime.py", line 219, in connect
    await call_sync_from_async(self._wait_until_alive)
  File "/Users/rohitmalhotra/Documents/Projects/OpenHands/openhands/utils/async_utils.py", line 18, in call_sync_from_async
    result = await coro
             ^^^^^^^^^^
  File<EMAIL_ADDRESS>line 58, in run
    result = self.fn(*self.args, **self.kwargs)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/rohitmalhotra/Documents/Projects/OpenHands/openhands/utils/async_utils.py", line 17, in <lambda>
    coro = loop.run_in_executor(None, lambda: fn(*args, **kwargs))
                                              ^^^^^^^^^^^^^^^^^^^
  File "/Users/rohitmalhotra/Library/Caches/pypoetry/virtualenvs/openhands-ai-nue4wPdC-py3.12/lib/python3.12/site-packages/tenacity/__init__.py", line 336, in wrapped_f
    return copy(f, *args, **kw)
           ^^^^^^^^^^^^^^^^^^^^
  File "/Users/rohitmalhotra/Library/Caches/pypoetry/virtualenvs/openhands-ai-nue4wPdC-py3.12/lib/python3.12/site-packages/tenacity/__init__.py", line 475, in __call__
    do = self.iter(retry_state=retry_state)
         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/rohitmalhotra/Library/Caches/pypoetry/virtualenvs/openhands-ai-nue4wPdC-py3.12/lib/python3.12/site-packages/tenacity/__init__.py", line 376, in iter
    result = action(retry_state)
             ^^^^^^^^^^^^^^^^^^^
  File "/Users/rohitmalhotra/Library/Caches/pypoetry/virtualenvs/openhands-ai-nue4wPdC-py3.12/lib/python3.12/site-packages/tenacity/__init__.py", line 418, in exc_check
    raise retry_exc.reraise()
          ^^^^^^^^^^^^^^^^^^^
  File "/Users/rohitmalhotra/Library/Caches/pypoetry/virtualenvs/openhands-ai-nue4wPdC-py3.12/lib/python3.12/site-packages/tenacity/__init__.py", line 185, in reraise
    raise self.last_attempt.result()
          ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File<EMAIL_ADDRESS>line 449, in result
    return self.__get_result()
           ^^^^^^^^^^^^^^^^^^^
  File<EMAIL_ADDRESS>line 401, in __get_result
    raise self._exception
  File "/Users/rohitmalhotra/Library/Caches/pypoetry/virtualenvs/openhands-ai-nue4wPdC-py3.12/lib/python3.12/site-packages/tenacity/__init__.py", line 478, in __call__
    result = fn(*args, **kwargs)
             ^^^^^^^^^^^^^^^^^^^
  File "/Users/rohitmalhotra/Documents/Projects/OpenHands/openhands/runtime/impl/eventstream/eventstream_runtime.py", line 404, in _wait_until_alive
    send_request(
  File "/Users/rohitmalhotra/Documents/Projects/OpenHands/openhands/runtime/utils/request.py", line 60, in send_request
    response = session.request(method, url, **kwargs)
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/rohitmalhotra/Library/Caches/pypoetry/virtualenvs/openhands-ai-nue4wPdC-py3.12/lib/python3.12/site-packages/requests/sessions.py", line 589, in request
    resp = self.send(prep, **send_kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/rohitmalhotra/Library/Caches/pypoetry/virtualenvs/openhands-ai-nue4wPdC-py3.12/lib/python3.12/site-packages/requests/sessions.py", line 703, in send
    r = adapter.send(request, **kwargs)
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/rohitmalhotra/Library/Caches/pypoetry/virtualenvs/openhands-ai-nue4wPdC-py3.12/lib/python3.12/site-packages/requests/adapters.py", line 700, in send
    raise ConnectionError(e, request=request)
requests.exceptions.ConnectionError: HTTPConnectionPool(host='localhost', port=31482): Max retries exceeded with url: /alive (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x12a0e4bc0>: Failed to establish a new connection: [Errno 61] Connection refused'))
ERROR:asyncio:Future exception was never retrieved
future: <Future finished exception=ConnectionError(MaxRetryError("HTTPConnectionPool(host='localhost', port=31482): Max retries exceeded with url: /alive (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x12a0e4bc0>: Failed to establish a new connection: [Errno 61] Connection refused'))"))>
Traceback (most recent call last):
  File "/Users/rohitmalhotra/Library/Caches/pypoetry/virtualenvs/openhands-ai-nue4wPdC-py3.12/lib/python3.12/site-packages/urllib3/connection.py", line 199, in _new_conn
    sock = connection.create_connection(
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/rohitmalhotra/Library/Caches/pypoetry/virtualenvs/openhands-ai-nue4wPdC-py3.12/lib/python3.12/site-packages/urllib3/util/connection.py", line 85, in create_connection
    raise err
  File "/Users/rohitmalhotra/Library/Caches/pypoetry/virtualenvs/openhands-ai-nue4wPdC-py3.12/lib/python3.12/site-packages/urllib3/util/connection.py", line 73, in create_connection
    sock.connect(sa)
ConnectionRefusedError: [Errno 61] Connection refused

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

Traceback (most recent call last):
  File "/Users/rohitmalhotra/Library/Caches/pypoetry/virtualenvs/openhands-ai-nue4wPdC-py3.12/lib/python3.12/site-packages/urllib3/connectionpool.py", line 789, in urlopen
    response = self._make_request(
               ^^^^^^^^^^^^^^^^^^^
  File "/Users/rohitmalhotra/Library/Caches/pypoetry/virtualenvs/openhands-ai-nue4wPdC-py3.12/lib/python3.12/site-packages/urllib3/connectionpool.py", line 495, in _make_request
    conn.request(
  File "/Users/rohitmalhotra/Library/Caches/pypoetry/virtualenvs/openhands-ai-nue4wPdC-py3.12/lib/python3.12/site-packages/urllib3/connection.py", line 441, in request
    self.endheaders()
  File<EMAIL_ADDRESS>line 1331, in endheaders
    self._send_output(message_body, encode_chunked=encode_chunked)
  File<EMAIL_ADDRESS>line 1091, in _send_output
    self.send(msg)
  File<EMAIL_ADDRESS>line 1035, in send
    self.connect()
  File "/Users/rohitmalhotra/Library/Caches/pypoetry/virtualenvs/openhands-ai-nue4wPdC-py3.12/lib/python3.12/site-packages/urllib3/connection.py", line 279, in connect
    self.sock = self._new_conn()
                ^^^^^^^^^^^^^^^^
  File "/Users/rohitmalhotra/Library/Caches/pypoetry/virtualenvs/openhands-ai-nue4wPdC-py3.12/lib/python3.12/site-packages/urllib3/connection.py", line 214, in _new_conn
    raise NewConnectionError(
urllib3.exceptions.NewConnectionError: <urllib3.connection.HTTPConnection object at 0x12a0e4bc0>: Failed to establish a new connection: [Errno 61] Connection refused

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

Traceback (most recent call last):
  File "/Users/rohitmalhotra/Library/Caches/pypoetry/virtualenvs/openhands-ai-nue4wPdC-py3.12/lib/python3.12/site-packages/requests/adapters.py", line 667, in send
    resp = conn.urlopen(
           ^^^^^^^^^^^^^
  File "/Users/rohitmalhotra/Library/Caches/pypoetry/virtualenvs/openhands-ai-nue4wPdC-py3.12/lib/python3.12/site-packages/urllib3/connectionpool.py", line 843, in urlopen
    retries = retries.increment(
              ^^^^^^^^^^^^^^^^^^
  File "/Users/rohitmalhotra/Library/Caches/pypoetry/virtualenvs/openhands-ai-nue4wPdC-py3.12/lib/python3.12/site-packages/urllib3/util/retry.py", line 519, in increment
    raise MaxRetryError(_pool, url, reason) from reason  # type: ignore[arg-type]
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
urllib3.exceptions.MaxRetryError: HTTPConnectionPool(host='localhost', port=31482): Max retries exceeded with url: /alive (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x12a0e4bc0>: 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<EMAIL_ADDRESS>line 58, in run
    result = self.fn(*self.args, **self.kwargs)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/rohitmalhotra/Documents/Projects/OpenHands/openhands/server/session/agent_session.py", line 92, in _start_thread
    asyncio.run(self._start(*args), debug=True)
  File<EMAIL_ADDRESS>line 194, in run
    return runner.run(main)
           ^^^^^^^^^^^^^^^^
  File<EMAIL_ADDRESS>line 118, in run
    return self._loop.run_until_complete(task)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File<EMAIL_ADDRESS>line 687, in run_until_complete
    return future.result()
           ^^^^^^^^^^^^^^^
  File "/Users/rohitmalhotra/Documents/Projects/OpenHands/openhands/server/session/agent_session.py", line 108, in _start
    await self._create_runtime(
  File "/Users/rohitmalhotra/Documents/Projects/OpenHands/openhands/server/session/agent_session.py", line 195, in _create_runtime
    await self.runtime.connect()
  File "/Users/rohitmalhotra/Documents/Projects/OpenHands/openhands/runtime/impl/eventstream/eventstream_runtime.py", line 219, in connect
    await call_sync_from_async(self._wait_until_alive)
  File "/Users/rohitmalhotra/Documents/Projects/OpenHands/openhands/utils/async_utils.py", line 18, in call_sync_from_async
    result = await coro
             ^^^^^^^^^^
  File<EMAIL_ADDRESS>line 58, in run
    result = self.fn(*self.args, **self.kwargs)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/rohitmalhotra/Documents/Projects/OpenHands/openhands/utils/async_utils.py", line 17, in <lambda>
    coro = loop.run_in_executor(None, lambda: fn(*args, **kwargs))
                                              ^^^^^^^^^^^^^^^^^^^
  File "/Users/rohitmalhotra/Library/Caches/pypoetry/virtualenvs/openhands-ai-nue4wPdC-py3.12/lib/python3.12/site-packages/tenacity/__init__.py", line 336, in wrapped_f
    return copy(f, *args, **kw)
           ^^^^^^^^^^^^^^^^^^^^
  File "/Users/rohitmalhotra/Library/Caches/pypoetry/virtualenvs/openhands-ai-nue4wPdC-py3.12/lib/python3.12/site-packages/tenacity/__init__.py", line 475, in __call__
    do = self.iter(retry_state=retry_state)
         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/rohitmalhotra/Library/Caches/pypoetry/virtualenvs/openhands-ai-nue4wPdC-py3.12/lib/python3.12/site-packages/tenacity/__init__.py", line 376, in iter
    result = action(retry_state)
             ^^^^^^^^^^^^^^^^^^^
  File "/Users/rohitmalhotra/Library/Caches/pypoetry/virtualenvs/openhands-ai-nue4wPdC-py3.12/lib/python3.12/site-packages/tenacity/__init__.py", line 418, in exc_check
    raise retry_exc.reraise()
          ^^^^^^^^^^^^^^^^^^^
  File "/Users/rohitmalhotra/Library/Caches/pypoetry/virtualenvs/openhands-ai-nue4wPdC-py3.12/lib/python3.12/site-packages/tenacity/__init__.py", line 185, in reraise
    raise self.last_attempt.result()
          ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File<EMAIL_ADDRESS>line 449, in result
    return self.__get_result()
           ^^^^^^^^^^^^^^^^^^^
  File<EMAIL_ADDRESS>line 401, in __get_result
    raise self._exception
  File "/Users/rohitmalhotra/Library/Caches/pypoetry/virtualenvs/openhands-ai-nue4wPdC-py3.12/lib/python3.12/site-packages/tenacity/__init__.py", line 478, in __call__
    result = fn(*args, **kwargs)
             ^^^^^^^^^^^^^^^^^^^
  File "/Users/rohitmalhotra/Documents/Projects/OpenHands/openhands/runtime/impl/eventstream/eventstream_runtime.py", line 404, in _wait_until_alive
    send_request(
  File "/Users/rohitmalhotra/Documents/Projects/OpenHands/openhands/runtime/utils/request.py", line 60, in send_request
    response = session.request(method, url, **kwargs)
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/rohitmalhotra/Library/Caches/pypoetry/virtualenvs/openhands-ai-nue4wPdC-py3.12/lib/python3.12/site-packages/requests/sessions.py", line 589, in request
    resp = self.send(prep, **send_kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/rohitmalhotra/Library/Caches/pypoetry/virtualenvs/openhands-ai-nue4wPdC-py3.12/lib/python3.12/site-packages/requests/sessions.py", line 703, in send
    r = adapter.send(request, **kwargs)
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/rohitmalhotra/Library/Caches/pypoetry/virtualenvs/openhands-ai-nue4wPdC-py3.12/lib/python3.12/site-packages/requests/adapters.py", line 700, in send
    raise ConnectionError(e, request=request)
requests.exceptions.ConnectionError: HTTPConnectionPool(host='localhost', port=31482): Max retries exceeded with url: /alive (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x12a0e4bc0>: Failed to establish a new connection: [Errno 61] Connection refused'))

OpenHands Installation
Development workflow
OpenHands Version
main
Operating System
MacOS
Logs, Errors, Screenshots, and Additional Context
No response

Setting in my environment didn't work for me. I added it to my toml and that worked! Thanks for the help, closing this issue.


Confirm programs with null pointer exception that are labelled with "No issues found"

Hi,
I am generating benchmarks with null pointer exception and on these benchmarks I am currently running Infer to see whether the NPE issue is found by Infer. It seems that my benchmarks are labelled with "No issues found", i.e. Infer does not find the NPE.
I was wondering if you could confirm the two attached examples of benchmarks.
Thank you,
Madalina
Relevant information:

Infer version v1.1.0
MacOS Big Sur, version 11.6.2. No Docker.
Command I ran: infer run -- javac  Program.java

RootProgram1Mnpeiancm.java.txt
RootProgram4DtmcIfwono.java.txt

Hi @mdlnhurmuz, thank you for your report! For your first example, infer does report an issue, is that not the one you wanted?
$ ./infer-linux64-v1.1.0/bin/infer -- javac RootProgram1Mnpeiancm.java
[...]
RootProgram1Mnpeiancm.java:50: error: Null Dereference
  object `box` last assigned on line 48 could be null and is dereferenced by call to `test(...)` at line 50.
  48.           Box<Integer> box = null;
  49.           Hello h = new Hello();
  50. >         h.test(box);
  51.       }
  52.   }

The second one is indeed a false negative. It seems that we don't take the implicit zero-initialisation in Java constructors into account, we should fix that! Here's a more minimal example that shows the problem:
class Box {
  public Object g;
  public Box() { } // the summary of the constructor does not say that g = null
}

class NullInitDeref {
  void FN_null_init_deref_bad()
    Box b = new Box();
    b.g.toString(); // missing NPE report here
  }
}

Actually, infer does find the NPE in the second example as well, but filters it out by default so you need to pass -F to see it:
$ ./infer-linux64-v1.1.0/bin/infer -F -- javac RootProgram4DtmcIfwono.java
RootProgram4DtmcIfwono.java:60: error: Null Dereference
  object `o` last assigned on line 59 could be null and is dereferenced by call to `print(...)` at line 60.
  58.           Box<Integer> b = new Box<Integer>().bvits();
  59.           Object o = b.g;
  60. >         print(o);
  61.       }
  62.   }

Hey @jvillard,
Thank you so much for your quick reply.
I can indeed replicate the command and behaviour on the second example and it shows me the NPE, but on the first one I still do not get the NPE issue.
I followed the instructions here  to install Infer and running only the command I mentioned.
Is there something else I could check to fix running Infer on my computer?
Many thanks,
Madalina

What version of Java do you have on your machine? javac -version and java -version. You may need to force the Java version with --java-version.

I was using Java 15.0.2 and downgraded now to 1.8.0 and it seems to be working. Thanks a lot for the prompt answers. Closing this issue now.

Hi,
Reopening this bug with some minor questions.

Could you confirm this program as false negative?

public class RootProgram5 {
    public static void main(String[] args) {
        String[] array = new String[3];
        String s = array[0];
        System.out.println(s.toString());
    }
}

I was wondering what are the current java versions that Infer supports on MacOS.  Is it documented up to which java version Infer works as expected?

Many thanks,
Madalina

@jvillard  so this only supports jdk 1.8? > 1.8 it doesn't?


Cache not updating in production

It took a while, but I figured it out.  Here is my service.js file the way it is now.  The key was the part that deletes the old caches.  You can see it in the answer here: http://stackoverflow.com/questions/37117933/service-workers-not-updating
I also threw in some extra debugging code while I was  figuring it out.  I also added the 'self.skipWaiting()'  This solved an issue that I was seeing where the update was being held up by the service worker getting stuck in a waiting state.  I found this on the MDN site describing the clients.claim() function.
I hope this might be helpful for others.
const DEBUG = false;

const CACHE_NAME = 'escape-1';

self.addEventListener('install', e => {
    function onCacheOpen(cache) {
        DEBUG && console.debug('Service cache');
        return cache.addAll([
            '/',
            '/build/bundle.js',
            '/build/bundle.css'
        ]);
    }

    e.waitUntil([caches.open(CACHE_NAME).then(onCacheOpen), self.skipWaiting()]);
});

self.addEventListener('activate', e => {
    DEBUG && console.debug('Service listener - activate');
    e.waitUntil([
        self.clients.claim().then(() => DEBUG && console.debug('Service listener - clients.claim')),
        caches.keys().then(function (cacheNames) {
            return Promise.all(
                cacheNames.map(function (cacheName) {
                    if (cacheName !== CACHE_NAME) {
                        DEBUG && console.debug('deleting', cacheName);
                        return caches.delete(cacheName);
                    }
                })
            );
        })
    ]);

});

self.addEventListener('fetch', e => {
    e.respondWith(
        caches.match(e.request).then(response => {
            // Cache hit - return response
            if (response) {
                DEBUG && console.debug(`cache hit (${CACHE_NAME})`, e.request.url);
                return response;
            }

            // IMPORTANT: Clone the request. A request is a stream and
            // can only be consumed once. Since we are consuming this
            // once by cache and once by the browser for fetch, we need
            // to clone the response.
            let fetchRequest = e.request.clone();
            DEBUG && console.debug('fetch: ', fetchRequest.url);
            return fetch(fetchRequest)
                .then(cacheResponse)
                .catch(cacheResponse);

            function cacheResponse(response) {
                // Check if we received a valid response
                // Don't cache non-CDN requests
                const isInvalid = !response || response.status !== 200 || response.type !== 'basic';
                if (isInvalid) {
                    DEBUG && console.debug('Cache excluded:', e.request.url)
                    return response;
                }

                // IMPORTANT: Clone the response. A response is a stream
                // and because we want the browser to consume the response
                // as well as the cache consuming the response, we need
                // to clone it so we have two streams.
                let responseToCache = response.clone();

                caches.open(CACHE_NAME)
                    .then(function (cache) {
                        DEBUG && console.debug('caching: ', e.request.url);
                        cache.put(e.request, responseToCache);
                    });

                return response;
            }

        })
    );
});

@scottburch This is pretty cool, thanks!

@nightwolfz No issue.  Thanks for the base, it is very helpful to understanding service workers.


CrProductDirName doesn't have effect

introduced by https://github.com/brave/muon/commit/be85e8eda01f3df84f75bb128fe2c26e56517395#diff-5d1dc1824328b9f5b351cfc5b0ff5fd7
CrProductDirName is required by
https://github.com/brave/electron-packager/commit/0dcbc2d5b56b058e8ee9a2ec1f43f96d94f4925e for mac package build user folder override

merged into https://github.com/brave/muon/pull/376


Implement output transform

This commit adds two command line switches. -r rotates the output 90
degrees clockwise and can be specified multiple times. -f flips the
output.
Fixes #55

Thanks and sorry for not getting back to you earlier!

This is ready for a second review.

Can we get this to also rotate inputs? I use this on a touchscreen, and the touch input doesn't rotate currently.

Well that's weird, at the time I implemented this touch was rotated correctly. But now that I've updated my buildroot system I'm having this problem as well. The touch input handling code in cage and wlroots hasn't changed since then AFAICT, which is where I would have expected this to be done. Mouse input is still correct though.

See #126


stream-engine: add Engine service interface definitions

Closes #412, closes #415

@huachaohuang Why define a proto instead of adding a rust trait like:
trait Engine {
     fn list_tenants();
     fn create_tenants():
     ...
}

After some chat with @w41ter-l , we agree to define the client interfaces instead of the service APIs first. The service APIs are actually for internal implementation, not public APIs.

Hmm, I realize that the protobuf definitions are still necessary to implement the client. So I think I can simply add the client interfaces along with this PR.

I added some basic client interfaces. If we need to add other implementations, we can mock the MasterClient. For example, we can simply send requests to a local master.
@w41ter-l do you think this is something we can start with?

@w41ter-l do you think this is something we can start with?

@huachaohuang Yes, I think it is enough.


Add support for Azure SQL Managed Instance

Added support for Azure SQL Managed Instance naming according to MS abbreviation recommendation
Pre-commit hook ran (other outdated information out of the scope of this change were updated along in README file)

@microsoft-github-policy-service agree


Adding Sleeperbot back to Showcase

Purpose: Adding Sleeperbot to Showcase
The top rated Fantasy Football app on Android and iOS - built mostly in React Native.
Funding Info:
https://www.crunchbase.com/organization/sleeperbot
https://angel.co/sleeperbot
Post:
https://medium.com/sleeperbot-hq/switching-to-react-native-in-production-on-ios-and-android-e6b675402712

We're not accepting PRs on this file at this time. Thanks for your understanding.


conflict with corrent version

Hello
I wanted to install the extention for the first time,
Juoyterlab is upgraded, but I got the following error:
>=3.0.11 <3.1.0         >=1.0.1 <2.0.0  @jupyterlab/application
>=3.0.9 <3.1.0          >=1.0.1 <2.0.0  @jupyterlab/codemirror
>=3.0.11 <3.1.0         >=1.0.1 <2.0.0  @jupyterlab/notebook

I'm having the same issues, but with slightly different version numbers:
❯ jupyter labextension install jupyterlab_vim
An error occured.
ValueError: The extension "jupyterlab_vim" does not yet support the current version of JupyterLab.

Conflicting Dependencies:
JupyterLab              Extension       Package
>=3.0.10 <3.1.0         >=1.0.1 <2.0.0  @jupyterlab/application
>=3.0.8 <3.1.0          >=1.0.1 <2.0.0  @jupyterlab/codemirror
>=3.0.10 <3.1.0         >=1.0.1 <2.0.0  @jupyterlab/notebook

Looks like development has moved over to https://github.com/axelfahy/jupyterlab-vim.
@jwkvam , if you could update the repository/README to reflect that, it would be appreciated.

@jrwrigh thanks for your comment.
I could manage to install the right Extension and it works smoothly.


Wasm text handling panic

Describe the bug
This bug is VERY random, but still occurs. Egui in the browser can randomly panic in relation to passStringToWasm0 or passStringFromWasm0 and failing to decode/encode text, and freeze the entire application, requiring a refresh of the page to work again.
To Reproduce
Steps to reproduce the behavior:

Sit on a page for an extended period of time
Randomly a panic in relation to passStringToWasm0 or passStringFromWasm0 will occur and application will freeze

Screenshots
Firefox:

Firefox:

Chromium:

Desktop (please complete the following information):

OS: Arch Linux
Browser: Firefox and Chromium
Version: 100 and 99

Is this on egui.rs (0.17) or latest master?

Is this on egui.rs (0.17) or latest master?

Both

Discovered that this issue is not related to text_edit_singleline and can occur when simply having the page open for an extended period of time. I edited this issue to reflect that

So just visiting egui.rs and leaving it open for a few minutes(?) will get you a passStringToWasm0/passStringFromWasm0 error?

It's EXTREMELY random. I cannot find a reliable way of reproducing it. Sometimes it happens within a minute or so of being on the page, sometimes it never occurs. I've been trying for ages to find a reliable way to reproduce it to see if I could fix it myself, but I couldn't, so I created this issue here.

I did some more digging. Seems it's actually a failure to parse an error in relation to the inability to borrow something mutably. I'm doing further digging.

Found the issue:

@juancampa said this on the discord server:

FWIW, I've seen panics like these because the .js file doesn't match the .wasm. When it happened (months ago) I discovered that the reason was the service worker that comes with eframe_template was caching one (or both?) files
I would disable that if you have it, just in case. Again, might be totally unrelated

That actually seems to be the issue, I implemented a system in which md5sums are appended to the end of file names, and it seems to work! This is my (probably poorly implemented) method of doing it here: https://github.com/Titaniumtown/YTBN-Graphing-Software/commit/aad08d06ab286c87d45827f790d0bb80ebf92630
I'll close this issue for now, if the issue persists I'll reopen it!

Oof 😞
I wonder if we can do anything in the default setup to help prevent this. This is not a great developer experience, to say the least!

Hashing like @Titaniumtown did is the way to go IMO

Changing the file name on each build sounds like a pretty robust way to always defeat the cache. Perhaps a bit hacky, but if it works it works.
Anyone feel like making a PR out of https://github.com/Titaniumtown/YTBN-Graphing-Software/commit/aad08d06ab286c87d45827f790d0bb80ebf92630 to egui and eframe_template?


Add PaNdata Software Catalog service + ESRF VISA service

Links to https://software.pan-data.eu/ and https://visa.esrf.fr respectively. I've added the software catalog to all providers as per Andy's request.

Sorry, force-pushed to fix typo: Catalogue instead of Catalog.


Fix wrong error message when index out of bounds is input for `assign…

…andunassign` command

Codecov Report

Merging #238 (74c487d) into master (c6b80c7) will not change coverage.
The diff coverage is 0.00%.

@@            Coverage Diff            @@
##             master     #238   +/-   ##
=========================================
  Coverage     48.17%   48.17%
  Complexity      594      594
=========================================
  Files           125      125
  Lines          2788     2788
  Branches        305      305
=========================================
  Hits           1343     1343
  Misses         1352     1352
  Partials         93       93

Impacted Files
Coverage Δ

...va/seedu/address/logic/commands/AssignCommand.java
0.00% <0.00%> (ø)

.../seedu/address/logic/commands/UnassignCommand.java
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


Added attributes to the statsd data

Description:
Took a hint from the graphite component and added entity attributes to the statsd payload.
Documentation
https://github.com/home-assistant/home-assistant.io/pull/625
Checklist:
If code communicates with devices, web services, or a:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass

If the code does not interact with devices:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] Tests have been added to verify that the new code works.

Looks good! :dolphin:


2770 admin light profile tab updates

Added program id, start date, end date, funding status, outcome themes, and gait / fund code / donors fields
Kept some code in that will may be used in ticket 2820 that will make this form editable and able to update fields in the demo, devs, and localhost environments.

It seems that the outcome themes in Admin Lite are displayed incorrectly. The serializer is returning a list of [ID, name], so both is displayed in the field instead of just the name. Would you like me to change the serializer to just return the name?

Good catch. I just made a change that fixes it to only display the names. I'll still want the IDs to come in from the serializer so that when we do the update ticket 2820, I can send back the update with ids and names.


Added an automatic detector for a WAD's associated README / Text File

What
Added the ability for the Wadinator to detect a WAD's matching text file. Currently limited to text files that match their WAD's name in all but extension. For example: gd.wad and gd.txt match, but some.wad and something.txt does not.

Can be configured to either print a command line to open an editor in read-only mode with the text file open, or to print out the whole contents of the text file in console.
Editors can be configured fully in Wadinator.toml.

Why
It seemed like a nice convenience feature that matches the suggested game command line feature to remove some manual work from the user when looking for readmes / text files associated with a WAD picked by the Wadinator, especially in very populated WAD directories.

Might attempt to deal with text files living in a separate directory before this is completely ready though.

Holy crap, this is awesome. This is actually the last feature I wanted to implement before the next tagged release (either 1.2.0 or 2.0.0…probably the latter). Thanks! :D
If you could have it recursively search directories, that would be immensely helpful for my use case. I primarily use this to randomly play D!ZONE WADs, and in the earlier editions of the disc they have this sort of structure:
MYWAD.WAD
TXT/MYWAD/MYWAD.TXT

Sometimes they'll even use the following (slightly more maddening) pattern:
MYWAD.WAD
TXT/MYWAD/MINE.TXT

I dumped the LEVELS directory from the DOS version of the discs and put them here if you want to see some practical examples: https://files.spectere.net/doom/
No worries if this is a bit outside of your intended scope. I can always extend upon your work if your needs have already been met. You've already been a great help. :)

Hm, would something like that be better as a flag (like -dzone) or a fully-fledged configuration option?
It shouldn't be too far out the scope to add something like this.

I envisioned it as a configuration option due to my use case (creating YouTube videos featuring the "best" WADs ever made). I made enough silly typos using the current build that ideally I'd like to just call the binary and have it do everything for me, haha.
Given the slapdash way these compilations were put together, this seems like the best approach to me, if directory search is enabled:

Do a recursive search for a text file sharing the WAD's name (since sometimes the TXT/* name is wrong, but the text file inside shares the WAD's name).
Do a recursive search for a directory sharing the WAD's base name and return a TXT file inside that directory. FILE_ID.DIZ should be ignored, as those files are largely useless in D!ZONE.

Just some of that fun DOS-era jankiness.

Awesome! I'll take a closer look when I get off work. Thanks again for all your help!

Alright, that should be that pattern tweaked to match any text file in the matching directory.

Looking good! Thanks again for your help. :)
As far as edge cases are concerned, it wouldn't surprise me if a few popped up. Pretty much the whole D!Zone series is one giant edge case, haha. At least modern stuff tends to be a bit more sane.
Before I close this PR: how would you like me to credit you?

Before I close this PR: how would you like me to credit you?

Honestly, I haven't a clue. This is my first PR to a repository that isn't something I'm doing for university or something like that.

No worries! I just used your GitHub username for now. If you would prefer something else down the road, feel free to modify CREDITS.md and open a pull request. :)


Cant use Form Request with name "Filter"

I apologize for the disturbance. The bug was in the form request itself, but log pointed other file. Can close it.

Glad you figured it out. You can actually close the issue yourself instead of waiting on a maintainer.


Add CredentialProvider option for Auth.js

Hi,
I love this project, it saves time through the setup and giving a proper folder structure.
In this setup when i choose Auth.js i see Discord, Google, GitHub, but no Credentials Provider option.
Auth.js official website states that they are discouraging "Credentials Provider" as part of there providers setup but in real world most people still use UserName/Email and Password login.
At the end of the day developer have to do what CLIENT wants, but not the other way around. Can you please provide Credentials option in Auth.js,
since setting up Credentials option is not as easy as setting up google.

Hey! Given this is something the authjs team has recommended against and it's not the easiest to implement, I've decided to go with adding Lucia as an authentication provider as an alternative. Hope it helps!


⚠ Bofrost.at has degraded performance

In fbf6fa0, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1006 ms

Resolved: Bofrost.at performance has improved in 01272a0 after 13 minutes.


Fix resource leaks and warnings in Junit tests #1332

FIxed resource leak warnings. Please review

@smadasu how / when do you see this warnings?. In your editor?.
As these are unit tests, it's not super important to do resource cleanup. I'm not sure about this change
@HeartSaVioR @xetorthio ?

Yeah. They are all on testa, unless I missed something. If that is the case
then I wouldn't worry too much.
On Mon, Jul 11, 2016, 07:50 Marcos Nils<EMAIL_ADDRESS>wrote:

@smadasu https://github.com/smadasu how / when do you see this
warnings?. In your editor?.
As these are unit tests, it's not super important to do resource cleanup.
I'm not sure about this change
@HeartSaVioR https://github.com/HeartSaVioR @xetorthio
https://github.com/xetorthio ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/xetorthio/jedis/pull/1348#issuecomment-231702444, or mute
the thread
https://github.com/notifications/unsubscribe/AAIhp3NqLGtyRknlf6n_b9WpGIaKxyPxks5qUh_sgaJpZM4JI93E
.

Yes. They are all unit tests. It may solve some of the random unit test failures we have seen in the past. But it's up to you guys to merge. This is not a high priority PR.

@marcosnils @xetorthio
I like the change though they're just tests. Someone starts from test codes to see the usage patterns, so though they're not critical, but also not bad to change them.
@smadasu
There're too many inconsistent style issues, whether try after space or not, if after space or not, ( after space or not, and so on. Yes we don't use checkstyle and also don't specify style guideline so you may wonder why I'm saying about style, but even narrowing code to only your changeset, the code style is inconsistent.
Please have a look at other code blocks and try to keep the code style. We might need to discuss the code style or even set up checkstyle in maven process.


Pre-GPU Hack Cleanup: Eliminate specification of fluid_index in input file

Because the fluid species struct and the kinetic species struct are separate objects which have to be looped over and we do not know a priori how the user will order them, right now the input file has an extra layer of complication for introducing coupling between a fluid species and kinetic species. For example, inside the kinetic species struct the collisions struct looks like
   .collisions = {       .collision_id = GKYL_LBO_COLLISIONS,       .ctx = &ctx,       .self_nu = evalNuElc,       .collide_with_fluid = "Pperp_elc",       .fluid_index = 0,     },
where fluid_index has to be consistent with the order you initialize the fluid species in.
Goal: eliminate this extra complication by getting both the index and the name of the species being collided with in the find_species methods

@JunoRavin Can this issue be closed? I think you fixed this?

Issue solved as of https://github.com/ammarhakim/gkylzero/commit/5ef794665a78c51fe6b380e31316f0ab55aad9e2


Fix: Implement Dynamic Width for LoadingBar.vue to Accurately Reflect Loading Progress

Hi there,
I noticed that the .loader-progress-indicator in the LoadingBar. vue component lacks a dynamic width property based on the progress prop.  Adding width: "${progress}%" to the indicator’s style is necessary to ensure it accurately reflects the loading progress.
This addition will improve the user’s perception of the loading process and provide a more engaging visual experience.  Could you please implement this change? Thanks!

Hello there,
Thank you for your suggestion!
Upon checking, the LoadingBar.vue component already dynamically adjusts the width of the loader based on the progress prop by leveraging the --progress-updated CSS variable. Here’s the relevant portion of the code for reference:
<div
  class="loader-progress-indicator"
  :style="{
    '--progress-updated': background ? `-${100 - progress}%` : '0%',
    '--color-primary': colors.primary,
  }"
/>

This ensures that the loading progress is accurately represented visually, as the --progress-updated variable is dynamically updated with the progress value.
To clarify further, I’ve included screenshots comparing the patch-2 branch and the develop branch for your review:

patch-2 branch:
develop branch:

As seen, the current implementation already handles the progress dynamically. Please feel free to review, and let me know if there are any other concerns!
Best regards,
Jonathan


Add a Project

I have added a Python Project: Scan and Generate QR Code.

Fetch Upstream and add your project info in Project_doc.md

you haven't fetch upstream which has caused a merge conflict now

resolve conflicts and raise another PR


Build order checkout page

Displays the products from the users cart
And calculates the order total
When submit order is clicked a loading spinner is displayed
Meanwhile it makes a request to the backend with the order info
If there is an error it shows it
If order succeeds send the user to the order confirmation screen with the order id
This screenshot shows the idea/inspiration but you don't have to copy exactly

Assigned this to myself, because I would like to have a hand in this, but definitely open to pair programming or sharing responsibilities if anyone else is interesting in working on this as well.


The wrong ⎋esc button

The ⎋esc button doesn't exactly same as that in the native Control Strip, it seems to that the buttons are 'locked' when pressing some keys:
I cannot exit from Switch bars (⌘ ⇥tab) without triggering anything (when holding ⌘ ) by pressing esc.
Force Quit... ( ⌘⌥ ⎋esc ) cannot be triggered.
Furthermore:
(very low possibility and can be denied) Finder Browsers in some specific apps cannot be closed by pressing ⎋esc.

I am also not able to use my esc button to execute force quit.  Holding command option esc does nothing

I think I have the same issue. Can't seem to make the ESC button to work.


Remove unnecessary whitespace

Rejecting due to: https://github.com/MewPurPur/GodSVG/blob/main/CONTRIBUTING.md#code-style
The reason for the code style guideline is that empty lines are harder to work with than ones that retain their scope - for example, you have to add the tabs back if you want to put a debug print statement.


Possible solution to maintaining backwards compat but simplifying mgmt group customization.

Unstable managmentGroups module which simplifies customizing management group hierarchy while maintaining backwards compatibility with existing module. Resolves #158
This PR fixes/adds/changes/removes

Adds parManagementGroupHierarchy parameter which allows for customizing the management group hierarchy.

Breaking Changes
None
Testing Evidence

As part of this Pull Request I have

[x] Checked for duplicate Pull Requests
[ ] Associated it with relevant ADO items
[x] Ensured my code/branch is up-to-date with the latest changes in the main branch
[x] Performed testing and provided evidence.
[ ] Updated relevant and associated documentation.

Making a call on this one that we will not implement until custom types and in Bicep are possible and supported

https://github.com/Azure/bicep/issues/4158
https://github.com/Azure/bicep/issues/3723


Documentation: fix incorrect hyphen underlining for R88

What does this refer to?

https://github.com/snowplow/snowplow/blob/master/CHANGELOG#L2


[ci][docs] downdrage sphinx and fix LaTeX error in R-package tests

Fixed #2974.
While we are waiting new breathe release which will be compatible with sphinx 3, let's downgrade sphinx version to fix our CI.
Refer to https://github.com/michaeljones/breathe/pull/491.

😱 oh no

This PR is facing the same issue https://github.com/microsoft/LightGBM/pull/2976#issuecomment-609807046. I think we should extract workaround from @henry0312's PR to make it clean.

oh nice ok!  @henry0312 you can ignore https://github.com/microsoft/LightGBM/pull/2976/files#r404111998, we'll just bundle it into this PR

I have no idea about /Library/TeX/texbin/tlmgr: unexpected return value from verify_checksum: -5
https://dev.azure.com/lightgbm-ci/lightgbm-ci/_build/results?buildId=5753&view=logs&j=2e42d92c-6989-5c5a-9adb-4d8d43874989&t=7c5d831f-c13d-5c19-c317-c20b1d55f3d3&l=75

@StrikerRUS since this is blocking other PRs, I think we should do this:
if [[ $OS_NAME == "macos" ]]; then
    R CMD check ${PKG_TARBALL} \
        --no-manual \
    || exit -1
else
    R CMD check ${PKG_TARBALL} \
        --as-cran \
    || exit -1
fi

Then I can work on fixing it today, but other PRs will be unblocked

@jameslamb

since this is blocking other PRs, I think we should do this:

As other PRs are not urgent (maintaince or new feature related), I believe they can wait while we are fixing this error.

@jameslamb

since this is blocking other PRs, I think we should do this:

As other PRs are not urgent (maintaince or new feature related), I believe they can wait while we are fixing this error.

I'm ok with that. If they become more urgent, just know I'm ok with that temporary solution.

Just reverted tlmgr-related changes as it is "every year" problem and we have nothing to do with it. Refer to https://github.com/microsoft/LightGBM/pull/2976#issuecomment-609897646.

OK, as it was suggested in maillist, I added --verify-repo=none option to install all needed stuff.

Merging this to unblock all waiting PRs.


on Bootstrap LESS error #136

I just closed the issue #136 by accident!
I  also did everything fresh but the grunt copy task don't work.
Here is the error when I run grunt:
Warning: Error compiling public/less/bootstrap-build.less Use --force to continue.
       Aborted due to warnings.

and when I run grunt cory:
minitouesspyros:drywall Spyros$ grunt copy
Running "copy:vendor" (copy) task
      Done, without errors.

I'm having this issue on Ubuntu 15.04. Grunt v0.4.5, npm 1.4.21, and Bower 1.4.1. After following all troubleshooting steps listed above, npm start is still failing after it hits the Bootstrap less files.

@jonsparks Drywall is no longer using bower. What is the less error? It's most commonly due to variables changing in Bootstrap.

`Running "less:layouts" (less) task

FileError: '../vendor/bootstrap/less/mixins.less' wasn't found. Tried - public/vendor/bootstrap/less/mixins.less,public/vendor/bootstrap/less/mixins.less,../vendor/bootstrap/less/mixins.less in public/less/bootstrap-build.less on line 3, column 1:
2 @import "bootstrap-vars.less";
3 @import "../vendor/bootstrap/less/mixins.less";
4
Warning: Error compiling public/less/bootstrap-build.less Use --force to continue.

Aborted due to warnings.`

@jonsparks let's open a new issue with what you just posted.


Watching the $route changes to fetch data

Somehow i am not able to watch $route. I have two components: One for the view and one to add new items.
Routes:
{
    path: '/addTodo/addTodo',
    component: require('./assets/todos/pages/todos/addTodo.vue')
  },
  {
    path: '/todos/',
    component: require('./assets/todos/pages/viewTodos.vue')
  }

From viewTodos.vue i switch to addTodo.vue but then after $route.back() my view does not update.
I use:
watch: {
  '$route': 'doSomething'
},

Here are the $route values for each component:
viewTodo.vue
Url: /todos/
Path: /todos/
Route:  {
"path": "/todos/",
"component": {
"watch": {
"$route": "loadTodos"
},
"methods": {},
"computed": {},
"staticRenderFns": [],
"__file": "C:\Users\SgtPepper\phonegap\mytime\src\assets\todos\pages\viewTodos.vue",
"beforeCreate": [
null
],
"beforeDestroy": [
null
],
"_Ctor": {}
},
"pagePath": "/todos/"
}
addTodo.vue
Url: /todos/addTodo/
Path: /todos/addTodo
Route:  {
"path": "/todos/addTodo/",
"component": {
"methods": {},
"staticRenderFns": [],
"_scopeId": "data-v-6bd83e6e",
"__file": "C:\Users\SgtPepper\phonegap\mytime\src\assets\todos\pages\todos\addTodo\addTodo.vue",
"beforeCreate": [
null
],
"beforeDestroy": [
null
],
"_Ctor": {}
},
"pagePath": "/todos/addTodo/"
}

I tried a lot the last days, but was not able to watch the route.
I solved it this way - in the "addTodo" component i used:
this.$f7.views.main.refreshPreviousPage();


[Button] Not passing 'type' prop down to its

Problem description
<Button>s do not pass 'type' prop down to their inner <button> and hence cause all <Button>s to submit the form if they are found inside one (as per html specification, all <button> tags with unset type attribute are considered as <button type="submit"> buttons and will cause the form to be posted.
Catching onSubmiton <form> or preventing default on every onClick event done by any of the <Button>s inside a from is possible but introduces additional code that would otherwise be unneeded.
Note: in version .14 <Button>s had set type="button" but in .15 this changed to role="button"
Link to minimal working code that reproduces the issue

https://www.webpackbin.com/bins/-KlIN6ZbgYwGwJtGhOt5
Versions

Material-UI<EMAIL_ADDRESS>React: react@15-4-2
Browser: Chrome  58.0.3029.110 (64-bit)

I'm looking into it.

That was fixed with #6958

Thanks for raising it.


Date Picker Date and Time input simultaneously

Hey is it possible to get this date-picker to show date and time simultaneously as follows:

I'm currently getting date and time sequentially presented. Tried a few formats but mostly getting the app crashing when I thinker with those.
& on a related note: is it possible to hide the 'date preview' at the top?
Thanks!

@joplaete When I started the development of the plugin, I tried my best to keep the consistency between the displays for android and ios, do you know any way to apply this display for android?
Another thing that we can pay attention to would be where would we put this exhibition, if we treated it as a theme or mode exhibition? I, in particular, would prefer to treat this as a theme, but I would like to know what you think of it.

Thanks for getting back to this @danielprrazevedo.
Not that familiar with android so not sure what the android flavour of this would look like sorry.
In terms of what would drive it.. you could go for the theme route although to me this feels more functionality over styling so perhaps mode is a better way to define it? e.g. 'dateThenTime' or something along those lines?  Up to you though, don't have a strong preference.

I was thinking of some ways to perform this feat, I think that while I can add one more option to the PickerOptions, something like mergedDateAndTime: boolean where it will have something specific for ios, as well as the color customizations of the picker, which you think?

Sounds good to me!


All Custom field report page requirement

after click on Next value save nhi ho rahi, so please next pe bhi value save krna hai. taki user ko problem na ho
after click Save , popup box close ho jana chahiye, abhi nhi ho raha.

Save and next karne par save nai ho raha means when we click on save&next and close the tab without pressing save button it should save but in this case it does not happened


Fix typo from adding test code for asset repo queries

Coverage decreased (-0.3%) to 27.365% when pulling 7084df32693c24bda688f35e9cd29b54aaaffc4e on asset-typo-fix into b8e2dbfbbae98eafd77178347bfa1eb3f7022059 on next.


kitty-items marketplace is not starting on testnet

Hi.
We want to review kitty-items marketplace. We installed env. tools and then we want to start on dev testnet. We took an error like this:
`⠖ Deploying contracts to:  0xf8d6e0586b0a20c7 (emulator)node:internal/errors:841
const err = new Error(message);
^
Error: Command failed: flow project deploy -o json --network=emulator -f flow.json --update
Command Error: failed to deploy all contracts
at ChildProcess.exithandler (node:child_process:398:12)
at ChildProcess.emit (node:events:527:28)
at maybeClose (node:internal/child_process:1092:16)
at Process.ChildProcess._handle.onexit (node:internal/child_process:302:5) {
code: 1,
killed: false,
signal: null,
cmd: 'flow project deploy -o json --network=emulator -f flow.json --update',
stdout: '\n',
stderr: ' Command Error: failed to deploy all contracts'
}`
We want to review kitty-items.

PS D:\sources\flow\kitty-items> flow version Version: v0.34.0 Commit: 6c240a76ec2bb5d5685afeb0898eed0ea1bd0059 PS D:\sources\flow\kitty-items> npm --version

Hi @sonerkoylu , try updating your flow-cli & kitty items repo to the latest version. Your error may have been addressed in a recent update.


[BUG] List database error: The requested operation requires an element of type 'Number', but the target element has type 'String'

Library name and version
Azure.ResourceManager.Sql 1.0.0-beta.4
Describe the bug
When list database, I got following error
System.InvalidOperationException: The requested operation requires an element of type 'Number', but the target element has type 'String'.
   at System.Text.Json.JsonDocument.TryGetValue(Int32 index, Int64& value)
   at System.Text.Json.JsonElement.GetInt64()
   at Azure.ResourceManager.Sql.SqlDatabaseData.DeserializeSqlDatabaseData(JsonElement element)
   at Azure.ResourceManager.Sql.Models.DatabaseListResult.DeserializeDatabaseListResult(JsonElement element)
   at Azure.ResourceManager.Sql.DatabasesRestOperations.ListByElasticPool(String subscriptionId, String resourceGroupName, String serverName, String elasticPoolName, CancellationToken cancellationToken)

From fiddler, the list request uses api version 2014-04-01, the maxSizeBytes in response is string instead of number.
Expected behavior
If I use following code to force the latest api version, the list request works fine.
var armOpt = new ArmClientOptions();
armOpt.SetApiVersion(SqlDatabaseResource.ResourceType, "2021-11-01-preview");
var armClient = new ArmClient(new DefaultAzureCredential(), null, armOpt);

This library should use correct api version by default to make everything work.
Reproduction Steps
var armClient = new ArmClient();
var resId = "/subscriptions/xxx/resourceGroups/xxx/providers/Microsoft.Sql/servers/xxx/elasticPools/xxx";
var pools = armClient.GetElasticPoolResource(new ResourceIdentifier(resId)).GetDatabases().Count();

Environment
.net 6.0 console app

Seems the api and model are generated by latest api version, but the request uses the oldest(2014-04-01), I found other apis return invalid data too.

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

Duplicate of #28387. The fix will be released in the next beta version this month.


Training Error

Getting the following when running on OS X, Python 2.7.10. Any ideas?
Traceback (most recent call last):
File "train.py", line 11, in
from model import Model
File "/word-rnn-tensorflow-master/model.py", line 2, in
from tensorflow.models.rnn import rnn_cell
File "/tensorflow/lib/python2.7/site-packages/tensorflow/models/rnn/rnn_cell.py", line 21, in
raise ImportError("This module is deprecated.  Use tf.nn.rnn_cell instead.")
ImportError: This module is deprecated.  Use tf.nn.rnn_cell instead.

Yes. We were aware of this issue. If you can fix it, please send us patch.
Sung
On Wed, Jul 13, 2016 at 10:20 AM, Joe Rivera<EMAIL_ADDRESS>wrote:

Getting the following when running on OS X, Python 2.7.10. Any ideas?
Traceback (most recent call last):
File "train.py", line 11, in
from model import Model
File "/word-rnn-tensorflow-master/model.py", line 2, in
from tensorflow.models.rnn import rnn_cell
File
"/tensorflow/lib/python2.7/site-packages/tensorflow/models/rnn/rnn_cell.py",
line 21, in
raise ImportError("This module is deprecated. Use tf.nn.rnn_cell instead.")
ImportError: This module is deprecated. Use tf.nn.rnn_cell instead.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/hunkim/word-rnn-tensorflow/issues/4, or mute the
thread
https://github.com/notifications/unsubscribe/AA3DV_cNdCuLxhdFItT9--iqNdvyrBo2ks5qVD1CgaJpZM4JK-kg
.

I don't know how to send a patch, but I was able to find a fix from https://github.com/tensorflow/tensorflow/issues/2685
import tensorflow as tf
#from tensorflow.models.rnn import tf.nn.rnn_cell
#from tensorflow.models.rnn import seq2seq

import numpy as np

class Model():
    def __init__(self, args, infer=False):
        self.args = args
        if infer:
            args.batch_size = 1
            args.seq_length = 1

        if args.model == 'rnn':
            cell_fn = tf.nn.rnn_cell.BasicRNNCell #changed
        elif args.model == 'gru':
            cell_fn = tf.nn.rnn_cell..GRUCell #changed
        elif args.model == 'lstm':
            cell_fn = tf.nn.rnn_cell.BasicLSTMCell #changed
        else:
            raise Exception("model type not supported: {}".format(args.model))

        cell = cell_fn(args.rnn_size)

        self.cell = cell = tf.nn.rnn_cell.MultiRNNCell([cell] * args.num_layers) #changed

        self.input_data = tf.placeholder(tf.int32, [args.batch_size, args.seq_length])
        self.targets = tf.placeholder(tf.int32, [args.batch_size, args.seq_length])
        self.initial_state = cell.zero_state(args.batch_size, tf.float32)

        with tf.variable_scope('rnnlm'):
            softmax_w = tf.get_variable("softmax_w", [args.rnn_size, args.vocab_size])
            softmax_b = tf.get_variable("softmax_b", [args.vocab_size])
            with tf.device("/cpu:0"):
                embedding = tf.get_variable("embedding", [args.vocab_size, args.rnn_size])
                inputs = tf.split(1, args.seq_length, tf.nn.embedding_lookup(embedding, self.input_data))
                inputs = [tf.squeeze(input_, [1]) for input_ in inputs]

        def loop(prev, _):
            prev = tf.matmul(prev, softmax_w) + softmax_b
            prev_symbol = tf.stop_gradient(tf.argmax(prev, 1))
            return tf.nn.embedding_lookup(embedding, prev_symbol)

        outputs, last_state = tf.nn.seq2seq.rnn_decoder(inputs, self.initial_state, cell, loop_function=loop if infer else None, scope='rnnlm') #changed
        output = tf.reshape(tf.concat(1, outputs), [-1, args.rnn_size])
        self.logits = tf.matmul(output, softmax_w) + softmax_b
        self.probs = tf.nn.softmax(self.logits)
        loss = tf.nn.seq2seq.sequence_loss_by_example([self.logits], #changed
                [tf.reshape(self.targets, [-1])],
                [tf.ones([args.batch_size * args.seq_length])],
                args.vocab_size)
...

https://github.com/hunkim/word-rnn-tensorflow/pull/5 fixes it

#5 merged!


CHECK IF SUBARRAY WITH SUM ZERO EXISTS OR NOT

Hello mentors :)
I want to add one problem that goes like:
Statement- Given an array, check if subarray with sum  zero exists or not.
Example-
INPUT- 2 5 1 -8 2
OUTPUT- Yes
Explanation- One such subarray { 2 , 5 , 1 , -8 } as 2+5+1+(-8)=0
I would like to work on this issue. Kindly assign it to me.
Thankyou!
@anubhavitis @dheerajkotwani @tarun26091999 @todi-2000 @geekquad @plazzy99 @rudrakshi99

Hi,
I am GSSOC'21 participant
My code will be in JAVA with proper comments and test cases.
@rudrakshi99 @nakul-19 @plazzy99 @tarun26091999  please assign me this issue

hey @tarun26091999 @rudrakshi99 under gssoc'21 can i work on this issue in python
Thank you!


Draft: integrate oklt transpiler

This PR is related to the release version of OCCA. But originaly it must be done to the latest main.
All changes are merged in different branch: integrated_oklt_transpiler
That branch must be used farther


Fixing typo: paramter -> parameter

Updating README.md
Update SwaggerConfig.cs.pp
Update SwaggerConfig.cs

AppVeyor fails with unrelated error:

Specify which project or solution file to use because the folder contains more than one project or solution file.

Can you please review and maybe merge as-is with broken build since this pr contains only text change?

Ping?

Ping?

Ping?

@domaindrivendev ping, please.

@domaindrivendev ping, please.

@domaindrivendev ping, please.

@domaindrivendev ping, please


inconsistent LaTeX \texttt highlighting - highlighted as an ordinary command when within parentheses

Steps to reproduce

create a .tex file with the following content:

\documentclass{article}
\begin{document}
\texttt{test}
(\texttt{test})
(  \texttt{test}   )
\texttt{test})
((blahblah)\\
\texttt{test}
\texttt{test}
)
(lorem ipsum)
\texttt{test}
( \textit{test} \textbf{test})
\end{document}

\texttt will be highlighted as an ordinary command when in parentheses

Expected behaviour
All instances of \texttt{} get highlighted the same way (green)
Version of Vim
9.1.8166
Environment
OS: Arch Linux
Kernel: x86_64 Linux 6.12.1-arch1-1
Terminal: konsole 24.08.
Shell: bash 5.2.37
Logs and stack traces
No response

To be clear this is not specific to \texttt, it seems to apply to \textbf, etc. that fall under texTypeStyle.

Please try the following patch: https://groups.google.com/g/vim_dev/c/R3-V6KrQsHc/m/lqCPuFSEAgAJ

Didn't work, sorry.
Additionally, \textbf and \textit are now also broken:


Documentation project on readthedocs is taken

See if we can work something out with @bitkits.
See https://github.com/bitkits/bootstrap4-django/issues/1

Seems to be fixed: http://django-bootstrap4.readthedocs.io/en/latest/

https://github.com/rtfd/readthedocs.org/issues/3195

Fixed by readthedocs


revise doc for making keyGenerateAlgorithm as a spring bean.

Fixes #5041.
revise doc for making keyGenerateAlgorithm as a spring bean.

Codecov Report

Merging #5061 into docs5.x will not change coverage by %.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             docs5.x    #5061   +/-   ##
==========================================
  Coverage      53.79%   53.79%
  Complexity       395      395
==========================================
  Files           1152     1152
  Lines          20482    20482
  Branches        3713     3713
==========================================
  Hits           11018    11018
  Misses          8786     8786
  Partials         678      678

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 2aa4053...e950c05. Read the comment docs.

Pull Request Test Coverage Report for Build 10819

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 57.167%

Totals

Change from base Build 10811:
0.0%

Covered Lines:
11709

Relevant Lines:
20482

💛  - Coveralls


Embedding input in TILE layout, despite ttnn.embedding doesn't support it

When running Llama prefil I ran into following assert on metal:

It says that input to the embedding op is in tile layout. When I check ttnn IR that is generated, I see that input %3 is derived from %1 (applied ttnn.to_device and type_cast) which is in tile layout.
This is how ttir looks like:

And here is ttnn IR part relevant to this embedding op. (Note that this is not the whole ttnn IR but just part that refers to embedding).
Llama_embedding_input_mem_layout.json

Here is the ttir and ttn of embedding test that fails with same error. Use this for repro...Embedding_input_tile_layout_ttnn.txt
Embedding_input_tile_layout_ttir.txt

Problem is following. If we have below IR:
ttir.to_layout()
ttir.typecast()
ttir.embedding()
To layout op conversion to ttnn checks if users of to_layout are operations which require row_major layout. If there is such user we force to_layout op conversion into row_major.

Since typecast is not among those ops (nor it should be) we force to_layout into tile layout.
This will result in error when running program since after to_layout op, input tensor for typecast will be in tile layout and after typecast (which just cast tensor from one data type to other) tensor layout will remain unchanged and we will run into assert into embedding requesting that first tensor input should be in row major when embedding on device.
When doing typecast we don't have any verifier which confirms that both input tensor and output tensor layout attributes differ only in DataType. In the case above we end up with input which is tile layout and output which is scalar. So we can end up like this:
ttnn.typecast(%2) { dtype = tt.data_type<Uint32> } (tensor<64x64xi32, tt.layout<...memref<2x2x!tt.tile<32x32xi32> ... >) -> tensor<64x64xi32, tt.layout<...memref<64x64xi32 ...> ... >
If we for example typecast from i32 to uin32 we only change typecast op attribute to reflect this change, we don't change RankedTensorType. So typecast will look like
ttnn.typecast(%2) { dtype = tt.data_type<UInt32> } (tensor<2x2xi32>) -> tensor<2x2xi32>
For the example shown in this issue typecast is actually redundant operation since we  typecast to same type.
%1 = "ttir.typecast"(%arg0, %0) <{operandSegmentSizes = array<i32: 1, 1>, operand_constraints = [#any_device, #any_device]}> {dtype = "Int32"} : (tensor<1x12xi32>, tensor<1x12xi32>) -> tensor<1x12xi32>

If I understand correctly second point: typecast op doesn't change memory layout from tile to row major, but from ttnn.typecast we see that layout is changed on output?
Also looking into ttnn IR it seems like to_device changes data type from int32 to uint32. And then we have type cast after that...

@mtopalovicTT As we agreed offline I found where we add this cast in forge, and removed it (turns out we were casting no matter the input df).
Now running llama embedding test with blofat16 produces tensor mismatch on output. Using float32 wights asserts in metal, therefore the need for 'blofat16'. I am attaching simple ttir repro. Llama_emb_bfloat16_tensor_mismatch_ttir.txt


Scan custom batch script option

Kick off a custom job on scan complete. Useful for weird custom stitching flows not integrated into GUI

Added, but you must activate developer mode to see it


GCP: Refactor data disk(s) creation and setup to instance template v2

What does this PR do?
Make additional disks part of the instance templates. This is a pre-requirement for scaling up. Unfortunately, GCP doesn't provide native way how to attach empty disk to instance within instance template, so we are using workaround including empty 1gb disk image. Contributed by @bdurrow Original PR: #505
How should this be manually tested?
Verify that node instances have attached 2 additional disks, and that those disks are properly configured:
$ lsblk
NAME                                                                                MAJ:MIN RM  SIZE RO TYPE MOUNTPOINT
sda                                                                                8:0    0   25G  0 disk
└─sda1                                                                                8:1    0   25G  0 part /
sdb                                                                                8:16   0   25G  0 disk
└─sdb1                                                                                8:17   0   25G  0 part
  ├─docker--vol-docker--pool_tmeta                                                           253:0    0   28M  0 lvm
  │ └─docker--vol-docker--pool                                                               253:2    0 23.8G  0 lvm
  │   ├─docker-8:1-12595346-cb4a5b21dc08aff11945180d602327f0080e53835b1c4acd0ead0a983fb1c239 253:3    0    3G  0 dm
  │   ├─docker-8:1-12595346-c1aad8d38a50bca1df990833b6e7ca01b40f7c4a07a2d11e23c6f0807fde1f15 253:5    0    3G  0 dm
  │   ├─docker-8:1-12595346-94e5ba10e0fe75e99f68d7c332de56765472c2bda1efb778e28b3cb5eb577ecf 253:6    0    3G  0 dm
  │   └─docker-8:1-12595346-7fe666e2d3e213391d72ad2638516dd1328d1df9c1479937dcf860e66fba4962 253:7    0    3G  0 dm
  └─docker--vol-docker--pool_tdata                                                           253:1    0 23.8G  0 lvm
    └─docker--vol-docker--pool                                                               253:2    0 23.8G  0 lvm
      ├─docker-8:1-12595346-cb4a5b21dc08aff11945180d602327f0080e53835b1c4acd0ead0a983fb1c239 253:3    0    3G  0 dm
      ├─docker-8:1-12595346-c1aad8d38a50bca1df990833b6e7ca01b40f7c4a07a2d11e23c6f0807fde1f15 253:5    0    3G  0 dm
      ├─docker-8:1-12595346-94e5ba10e0fe75e99f68d7c332de56765472c2bda1efb778e28b3cb5eb577ecf 253:6    0    3G  0 dm
      └─docker-8:1-12595346-7fe666e2d3e213391d72ad2638516dd1328d1df9c1479937dcf860e66fba4962 253:7    0    3G  0 dm
sdc                                                                                8:32   0   50G  0 disk /var/lib/origin/openshift.local.volumes

Master instances should have attached one additional disk for docker in case of containerized deployment.
Is there a relevant Issue open for this?
Only the PR #505
Who would you like to review this?
cc: @bdurrow @e-minguez @cooktheryan  PTAL
Resolves: #454 #447

 (8) OpenShift-Ansible-Contrib issues and PRs

Here is the link to the issue that kept using image family for data disks in the instance template.  Solution should be rolled out in GCP within 2 weeks: https://issuetracker.google.com/issues/63526399

OK, so I'll rework it with wait_for module run from localhost, otherwise it won't cover this use case.

@bdurrow wait for the instances should be now bulletproof

@e-minguez or @cooktheryan please merge


Refactoring: _maybe_dereference_node

On master, the Node._dereference_node method returns Optional[Node]. This leads to some assert node is not None statements in the code (for the benefit of mypy).
This PR does two things:

introduce a new method, Node._maybe_dereference_node, which returns Optional[Node]
change the type signature of _dereference_node to retrurn Node (not optional).

This change lets us delete some of the node is not None assertions.
This PR is a followup to https://github.com/omry/omegaconf/pull/502#discussion_r608711688.

Looking for _dereference_node under the omegaconf module, I think there are more cases where you can eliminate the assert.

I can't find any more...


equivalent of ip netns attach my-ns 1

Just attempting to understand the capabilities of this package. I didn't see the ability to create a named network namespace and attach it to a specific process ID. Is that accurate? Or am I missing something.

The manipulation of network namespaces is provided in https://github.com/vishvananda/netns


Giveaway seems to select only users from certain roles

I ran 2 giveaways (basic v2) in the last couple days on my server. The first one seems to work no problem, but when the second ran, it chose only users that were a part of a certain role, which included the user that won the first giveaway.
Now, I understand it is a random draw, but I specified the winners of the second giveaway should total 4 winners, the giveaway ended with only 2 winners selected, and 2 rerolls after that grabbed the same users.
Again, I understand randomness and this COULD happen, but knowing the 2 users that have won are a part of a role that has permission to run the command and are the only 2 users from the role (of a total of 5 users in the role) that entered the giveaway, this seems like SOMETHING is off.

The most likely explanation for this is that the reaction handler command is only picking up reactions from that role. This could be due to a number of reasons (an example I can think of off the top of my head is hitting the limit of 3 custom commands executed per action).
In any case, due to the fact that you are the only one to report this issue, it's probable that this is a configuration issue of some sort on your end. I'm happy to help debug the issue over Discord through the YAGPDB support server if you'd like.

This seems to have been resolved over Discord, so I'll close this. If the fix discussed doesn't work, feel free to reopen.


Cross compile splicing should be preserved in .override, .overrideAttrs, .withPackages, etc

Describe the bug

Isn't inherit (buildPackages) boost-build; effectively implied when you're only using it in nativeBuildInputs?

Yes, but since we override the boost-build derivation later, the splicing magic won't be able to process it and pass it through as is.

There's no good reason for this not to work automatically.
Steps To Reproduce
Steps to reproduce the behavior:

Apply my suggestion here https://github.com/NixOS/nixpkgs/pull/132597#discussion_r682456483
Cross compile fails

Expected behavior
It should just work. I don't know if this is possible fundamentally, because packages in the lexical closure of f in .override f may need different treatment depending on which spliced variation it is applied to.
.override is the domain of callPackage, which is also responsible for splicing. This one may be relatively easy to write.
.overrideAttrs needs a bit more work. Ideally we can support any [public] attribute(https://github.com/NixOS/nixpkgs/pull/119942#issuecomment-889049698) that produces new derivations, like withPackages. This may require some cooperation from derivation authors, to give the opportunity to override the results.
Idea:

The splicing logic uses .overrideAttrs instead of //
The splicing logic sets/overrides more attributes that can be used by the original package to support splicing in attributes like withPackages. I haven't figured out the details of this, but it will probably involve calling the same attribute as the argument to a higher-order function provided by the splicing logic.
The original derivation can use https://github.com/NixOS/nixpkgs/pull/119942 to access self, which includes attributes from future overrides, such as the one(s) suggested earlier in this list.

Screenshots
Additional context
Add any other context about the problem here.
Notify maintainers

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
output here

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute:
# a list of nixos modules affected by the problem
module:

If we can't make this work, we'll have to recommend putting every overridden package back into the nixpkgs fix-point so splicing can pick up on it. That leads to many ad-hoc attributes, which might be considered not to be a good thing. Perhaps these could be local to a callPackage invocation somehow? callPackageWithLocals localPackages packageFn args or something.

overrideAttrs will be fixed in #201734


Action freezes for 3 minutes at installing extensions and then fails

Describe the bug
Action freezes for 3 minutes at installing extensions and then fails
Version

[ ] I have checked releases, and the bug exists in the latest patch version of v1 or v2.
[x] v2
[ ] v1

Runners

[x] GitHub Hosted
[ ] Self Hosted

Operating systems
ubuntu-latest
PHP versions
8.0
To Reproduce
- name: Setup PHP, with composer and extensions
        uses: shivammathur/setup-php@v2
        with:
          php-version: "8.0"
          tools: composer:v2.1.3, cs2pr:v1.4.0
          extensions: curl, mbstring, xml, ctype, iconv, intl, pdo_sqlite, mysql, bcmath, iconv, gd, intl, mysqli, pdo_mysql, zip, redis, apcu, pdo_sqlsrv, sqlsrv
          coverage: none
        env:
          fail-fast: true

Expected behavior
It should install PHP and extensions
Screenshots/Logs

Additional context
This workflow has been pretty much working since November 2020. It stopped today all of sudden.
Are you willing to submit a PR?

@faizanakram99
There is no php8.0-sqlsrv package, so it is installing sqlsrv and pdo_sqlsrv from PECL.
You can use shivammathur/setup-php@verbose to see the logs.
I could not reproduce the issue though you faced, so might be a one-off. Please try again using the verbose branch and provide the logs if it happens again.
Test runs:
v2: https://github.com/shivammathur/test-setup-php/runs/3322292269?check_suite_focus=true
verbose: https://github.com/shivammathur/test-setup-php/runs/3322234906?check_suite_focus=true

@shivammathur
Yep, looks like a one off error.
Thanks


Option to disable the "Show/Quit Franz" menu

Add an option to disable the "Show/Quit Franz" menu in the system menu bar
Motivation and Context
Already too many programs in the system menu bar.

Duplicate of #2


Model inputs and outputs

I want to make the model output t1 before the change, t2 after the change, and the label image label. However, according to the code provided, it seems that only the final change map is output. How can I achieve the output of the three images I want?

The released model was only trained for building change detection, though it is a multi-task architecture. So, it only yielded binary change prediction. If you want to obtain the temporalwise semantic prediction, you can consider training this model with multi-task supervision.


In IPFIX DeltaMicroseconds case, basetime must be converted to MS

*1000  missing to convert basetime from seconds to milliseconds in two cases

follow-up on https://github.com/netsampler/goflow2/pull/81
@lspgn

Was go fmt ran?
Thanks for the fix!

Was go fmt ran?

yes that's precisely why it changed the format on a few other lines

Thank you :)


how can I use .pug with templateUrl?

Support Question
Please do not submit support requests or "How to" questions here. Instead, please use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/

Hi there! It appears this isn't really an issue, and more of a question, so Im going to close this. Please check out the forum as this is a better place for questions. Thanks!


BYPASS_MONGO_VALIDATION=true is not bypassing the MongoDB version check

Description:
Environment variable BYPASS_MONGO_VALIDATION=true is not bypassing the MongoDB version check
Steps to reproduce:

Install via snap sudo snap install rocketchat-server or switch branches to "4.x/stable"
Get system message: "MongoDB version 3.6.23 is deprecated, please upgrade your installation."
Add export BYPASS_MONGO_VALIDATION=true to /etc/profile

Expected behavior:
MongoDB version check should be bypassed and Admins DO NOT receive a Rocket.Cat alert
Actual behavior:
MongoDB version check runs and Admins DO receive a Rocket.Cat alert
Server Setup Information:

Version of Rocket.Chat Server: 4.1.2
Operating System: Ubuntu 18.04.5 LTS on Raspberry Pi 4
Deployment Method: snap
Number of Running Instances: 1
NodeJS Version: v12.22.7
MongoDB Version: 3.6.23 / wiredTiger (oplog Enabled)

Additional context:
I have exactly this issue where i've upgraded Rocket Chat to 4.1.2 via a Snap and i'm getting the message "MongoDB version 3.6.23 is deprecated, please upgrade your installation.".
The thread goes on to state "The mongo update will happen shortly. You can ignore the deprecation message in the meantime.", but that was 9 months ago.
I've no issue waiting, however, every Admin is getting frequently spammed with Rocket.Cat alerts in the meantime for which there is no corrective action available to us.
This post states:
It's possible to bypass the MongoDB version check by using the environment variable BYPASS_MONGO_VALIDATION=true. Bare in mind that this work around should be used only when extrictly necessary and may prevent some functionalities to work properly or even prevent the server to start.
This would seem the best course of action but it doesn't work. I've added the variable to /etc/profile and confirmed it is available before Rocket Chat server starts.
Help

You can't set environment variables for snaps like that. Read here. Closing since not a bug. Please feel free to continue asking any other question you might have :)

Thanks @debdutdeb but i still can't get this to work.
Here's the output from my terminal:
user@server: /var/snap/rocketchat-server/common$ cat bypass_mongo_validation.env
BYPASS_MONGO_VALIDATION=true

Where i then restart the services and/or restart the server, i still get the "MongoDB version 3.6.23 is deprecated, please upgrade your installation." Rocket.Cat notification

Which version are you in? In the latest it works that I can gurantee. You should update, considering it's complaining about mongo 3.6, you must be on some old track.

Secondly, the environment variable is in case mongodb version is unsupported. It does not handle rocket.cat notifications.

Here's the package info:
channels:
  latest/stable:    3.18.2 2021-12-06 (1495) 217MB -
  latest/candidate: 3.18.2 2021-12-06 (1495) 217MB -
  latest/beta:      3.18.2 2021-12-06 (1495) 217MB -
  latest/edge:      3.18.2 2021-12-06 (1495) 217MB -
  4.x/stable:       4.1.2  2021-12-15 (1496) 212MB -
  4.x/candidate:    4.1.2  2021-12-15 (1496) 212MB -
  4.x/beta:         4.6.4  2022-06-07 (1519) 282MB -
  4.x/edge:         4.1.2  2021-11-19 (1496) 212MB -
  3.x/stable:       3.18.2 2021-11-28 (1495) 217MB -
  3.x/candidate:    3.18.2 2021-11-28 (1495) 217MB -
  3.x/beta:         3.18.2 2021-11-28 (1495) 217MB -
  3.x/edge:         3.18.2 2021-11-28 (1495) 217MB -
  2.x/stable:       2.4.9  2020-02-19 (1423) 272MB -
  2.x/candidate:    ↑
  2.x/beta:         ↑
  2.x/edge:         ↑
installed:          4.1.2             (1496) 212MB -

The challenge i have is that i'm running the latest stable version, which is the version triggering the deprecated message.
There's no issue with the installation so i'm happy to wait for the next release, however, we get spammed daily by the Rocket.Cat alert and hence want to prevent the check.

What is your snapd version? And host information? Please share.

Version of Rocket.Chat Server: 4.1.2
Operating System: Ubuntu 18.04.5 LTS on Raspberry Pi 4
Deployment Method: snap
Number of Running Instances: 1
NodeJS Version: v12.22.7
MongoDB Version: 3.6.23 / wiredTiger (oplog Enabled)

snap    2.57.6
snapd   2.57.6
series  16
ubuntu  18.04
kernel  5.3.0-1042-raspi2


use autogenerated gitignore (heavier but more reliable)

I don't see which problem it solves. And I don't think there's any problem with .gitignore. See, this project is small and has only one maintainer. I'd prefer to understand what's written inside .gitignore than to dig into confusing problems when some file is ignored and I don't understand why.

@amaembo Fine for me, I understand your point
My problem was that I'm working with eclipse and it's generating a lot of useless file that I had to ignore locally.
(especially in the .settings/ directory)

Ok, I added /.settings and /.metadata entries. Hopefully this helps


Added support for spellfix1 module

This PR adds general support for loading statically linked extensions via sqlite3_auto_extension and is also adding the spellfix1 extension (https://www.sqlite.org/spellfix1.html).

@simolus3  can you please check, if you pushed the last changes for sqlite-native-libraries?
The patch version should be 2 as referenced in https://github.com/simolus3/sqlite3.dart/blob/1621cce660ee4f64992910a20b5492112f675334/sqlite3_flutter_libs/android/build.gradle#L29 but it is still 0 (https://github.com/simolus3/sqlite-native-libraries/blob/a1e3bafbb2cd27d9a8a4526b09681ecd65da0d44/sqlite3-native-library/build.gradle.kts#L12). I remember when first compiling the android native libraries I had to increase this number. If this here is ok I will issue an PR there as well.
Is the extension loading itself is approved by you I will add some tests for this.

My bad on not publishing that earlier, done now: https://github.com/simolus3/sqlite-native-libraries/commit/b2933517ceee7fb49d5a9900e6695fe5647da1f3.
I'm in favor of this feature, but I do wonder if we might need to add SQLITE_CORE as a define while compiling these extensions? So far, I haven't really used those features of sqlite3, but from this section it sounds like that might be helpful for SQLITE_EXTENSION_INIT1 to become a no-op since we won't ever load it dynamically?

I think we don't have to care about that. The extension itself is not different from the already included ones like json1, rtree or fts5. It's just that the source file is not included by default. The structure of the extension is the same. I think this is a historical part of the documentation, when there were no extensions really added yet.
Also see: https://sqlite-users.sqlite.narkive.com/rep6Tusc/sqlite-core-use-for

I did some more investigation and indeed, it seems to only affect internal extensions and the way the are build (like stated by the creator https://sqlite-users.sqlite.narkive.com/rep6Tusc/sqlite-core-use-for#post4). When SQLITE_CORE is NOT set, only pure SQLite is available and internal modules have to get loaded manually. If not, the internal extensions are getting compiled to expose a special init function e.g. sqlite3Fts5Initfor fts5. SQLite is aware of its internal modules then and uses this specific init to load the extension directly.
Also I found the following code:
/*
** The following functions are used to register the module with SQLite. If
** this module is being built as part of the SQLite core (SQLITE_CORE is
** defined), then sqlite3_open() will call sqlite3Fts5Init() directly.
**
** Or, if this module is being built as a loadable extension,
** sqlite3Fts5Init() is omitted and the two standard entry points
** sqlite3_fts_init() and sqlite3_fts5_init() defined instead.
*/
#ifndef SQLITE_CORE
#ifdef _WIN32
__declspec(dllexport)
#endif
int sqlite3_fts_init(
  sqlite3 *db,
  char **pzErrMsg,
  const sqlite3_api_routines *pApi
){
  SQLITE_EXTENSION_INIT2(pApi);
  (void)pzErrMsg;  /* Unused parameter */
  return fts5Init(db);
}

#ifdef _WIN32
__declspec(dllexport)
#endif
int sqlite3_fts5_init(
  sqlite3 *db,
  char **pzErrMsg,
  const sqlite3_api_routines *pApi
){
  SQLITE_EXTENSION_INIT2(pApi);
  (void)pzErrMsg;  /* Unused parameter */
  return fts5Init(db);
}
#else
int sqlite3Fts5Init(sqlite3 *db){
  return fts5Init(db);
}
#endif

So it seems like SQLITE_CORE is already set, since sqlite3_fts5_init is not getting exported when looking at the symbols of the compiled library. Also, nowhere in the source sqlite3_fts5_init is referenced, so it cannot get automatically loaded anywhere.
However, for spellfix SQLITE_CORE is not relevant, because it cannot get internally called anyways (since its not built in extension) and we have to call the sqlite3_spellfix1_init ourselfs.
/*
** Register a statically linked extension that is automatically
** loaded by every new database connection.
*/
int sqlite3_auto_extension(
  void (*xInit)(void)
)

So, sqlite3_auto_extension is especially for statically linked extensions. Right now all dynamic loading is already disabled, because SQLITE_OMIT_LOAD_EXTENSION is already set. We don't have to change anything here.

Do you think it will be possible to also load extensions dynamically on Android and iOS?
I found out I need some internal things of the FTS5 extension, for which I wrote an own extension to expose them. Nothing you want to statically link for sure. So maybe it makes sense to go into the direction of dynamic loading here, so spellfix could also packed into an own package.
What do you think?

I tried to implement dynamic loading in https://github.com/simolus3/sqlite3.dart/pull/117#issuecomment-1226523834.
However, we currently omit the extension loading mechanism from our sqlite3 builds, it is also disabled on iOS/macOS by default. Given the security considerations of dynamic extension loading, I don't think that we should expose the automatic interface loading.
I'm totally fine with an API that you give a function pointer to and it loads that as an extension. This is essentially what is sqlite3_auto_extension is, and I'm happy to expose it directly. It doesn't even require static linking, you can put your extension into another shared object, do DynamicLibrary.open in Dart, lookup the pointer with dart:ffi and then call sqlite3_auto_extension on that. Again, from a security perspective, I really don't want this library dynamically loading code. Being given a function pointer that you loaded is fine with me.

So maybe it makes sense to go into the direction of dynamic loading here, so spellfix could also packed into an own package.

That would be an interesting approach. On iOS and macOS, I managed to configure it so that spellfix1 can be included as an optional dependency (https://github.com/clemensg/sqlite3pod/pull/19). So we could have another Flutter package (say sqlite3_spellfix1_flutter_libs :D) only providing that extension on iOS/macOS. On Android, we can pull the extension back out of sqlite3_native_libraries and also include it with a native build in that package (or perhaps as a Gradle feature variant in sqlite3_native_libraries, but I'm not too familiar with those).

Ohh yeah, I totally forgot that we are also able to call the entry point of other modules :D
I changed the extension loading to make it more dynamical and will do some tests on the weekend.

It looks like support for (statically linking) spellfix1 has landed in the sqlite3 pod.
I have just released 3.39.3 of the native libraries for Android. You can try upgrading that and iOS to 3.39.3. On iOS, you can also add a dependency to the subpod sqlite3/spellfix1 now.

I checked your latest commit, do you think its okay to pass the dynamic library to an isolate? I Was not sure about thr behaviour here, thats why I used the resolve.

Thinking about this some more, we should have support for automatically loading extensions, but I don't believe we should ship any of them by default.
There are integration tests for this feature so we know it works, but since it is a pretty advanced use case we might as well expect the user to set up the libraries manually.

do you think its okay to pass the dynamic library to an isolate?

Hm, dart:ffi types can't be used in isolate messages but I'm not sure if there's a use case for sending this class across isolates in the first place? If you're using this package in a a background isolate, couldn't you just call the method on that isolate.
(well technically you could call that method on any isolate, extensions are global and visible across isolates either way)


add: request sharding, diskless playback; fix: #10, #14

Solves #10 by splitting text into different requests and then combining the output to produce an playable audio stream.
Also tried to help with #14 and #15 by editing the Readme a bit.
Tried to do #5 but I've still not found any docs for the api, or anybody using it with a request that uses fields that this module doesn't. If you don't mind me asking, how did you find out about it? Seems so obscure, lol

sorry for the delay man. the truth is, I'm pretty sure there's a bug that will cause the algorithm to break for at least chinese and arabic and I wanted to find the time to sit down and properly figure it out but I just cant recover the train of thought I had to notice it. Never having been much of a linguist, I wasn't sure where to start either, lol.
That said, I notice the changes aren't in in the main branch. If you want, to re-open, I can at least implement the requests you made. Otherwise, I could resubmit with them in mind.