llama_cpp/libllama.so not found

Describe the bug
When trying to install InstructLab on my M2 Max, I can successfully install and initialize, but when I try to run ilab chat, I get this error:
FileNotFoundError: Shared library with base name '/home/developer/miniconda3/envs/labEnv/lib/python3.11/site-packages/llama_cpp/libllama.so' not found
Output:
    sys.exit(ilab())
  File "/Users/RobertGreenberg/miniconda3/envs/native/lib/python3.10/site-packages/click/core.py", line 1161, in __call__
    return self.main(*args, **kwargs)
  File "/Users/RobertGreenberg/miniconda3/envs/native/lib/python3.10/site-packages/click/core.py", line 1082, in main
    rv = self.invoke(ctx)
  File "/Users/RobertGreenberg/miniconda3/envs/native/lib/python3.10/site-packages/click/core.py", line 1697, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/Users/RobertGreenberg/miniconda3/envs/native/lib/python3.10/site-packages/click/core.py", line 1697, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/Users/RobertGreenberg/miniconda3/envs/native/lib/python3.10/site-packages/click/core.py", line 1443, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/Users/RobertGreenberg/miniconda3/envs/native/lib/python3.10/site-packages/click/core.py", line 788, in invoke
    return __callback(*args, **kwargs)
  File "/Users/RobertGreenberg/miniconda3/envs/native/lib/python3.10/site-packages/click/decorators.py", line 33, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/Users/RobertGreenberg/miniconda3/envs/native/lib/python3.10/site-packages/instructlab/clickext.py", line 323, in wrapper
    return f(*args, **kwargs)
  File "/Users/RobertGreenberg/miniconda3/envs/native/lib/python3.10/site-packages/instructlab/model/chat.py", line 232, in chat
    backend_instance = backends.select_backend(
  File "/Users/RobertGreenberg/miniconda3/envs/native/lib/python3.10/site-packages/instructlab/model/backends/backends.py", line 124, in select_backend
    from .llama_cpp import Server as llama_cpp_server
  File "/Users/RobertGreenberg/miniconda3/envs/native/lib/python3.10/site-packages/instructlab/model/backends/llama_cpp.py", line 14, in <module>
    from llama_cpp import llama_chat_format, llama_token_get_text
  File "/Users/RobertGreenberg/miniconda3/envs/native/lib/python3.10/site-packages/llama_cpp/__init__.py", line 1, in <module>
    from .llama_cpp import *
  File "/Users/RobertGreenberg/miniconda3/envs/native/lib/python3.10/site-packages/llama_cpp/llama_cpp.py", line 83, in <module>
    _lib = _load_shared_library(_lib_base_name)
  File "/Users/RobertGreenberg/miniconda3/envs/native/lib/python3.10/site-packages/llama_cpp/llama_cpp.py", line 74, in _load_shared_library
    raise FileNotFoundError(
FileNotFoundError: Shared library with base name '/home/developer/miniconda3/envs/labEnv/lib/python3.11/site-packages/llama_cpp/libllama.so' not found

Screenshots

Device Info (please complete the following information):

Hardware Specs: Apple M2 Max Chip
OS Version: Mac OS 15.1.1
Python Version: Python Version 3.10
InstructLab Version: instructlab-0.22.1

Any help would be appreciated!

 File "/Users/RobertGreenberg/miniconda3/envs/native/lib/python3.10/site-packages/llama_cpp/llama_cpp.py", line 83, in <module>
    _lib = _load_shared_library(_lib_base_name)
  File "/Users/RobertGreenberg/miniconda3/envs/native/lib/python3.10<<<<<<<<<</site-packages/llama_cpp/llama_cpp.py", line 74, in _load_shared_library
    raise FileNotFoundError(
FileNotFoundError: Shared library with base name '/home/developer/miniconda3/envs/labEnv/lib/python3.11 <<<<<<------/site-packages/llama_cpp/libllama.so' not found

Looks like the env is using the mix python version, it is possible to only use python3.11 to setup the venv ?? And then install.

Will try to create a new venv w/ Python 3.10 only. ilab only supports <= 3.10 afaik

Set up a new env w/ 3.10 and still getting the same error. How can I delete '/home/developer/miniconda3/envs/labEnv/lib/python3.11/site-packages/llama_cpp/libllama.so' to look in 3.10 lib folder?

Python 3.10 or Python 3.11

📋 Python 3.12 is currently not supported. Some InstructLab dependencies don't work on Python 3.12

Python3.11 should be better.
Seems related to miniconda3 , could you please try without it?

  File "/Users/RobertGreenberg/Documents/InstructLab/venv/bin/ilab", line 8, in <module>
    sys.exit(ilab())
             ^^^^^^
  File "/Users/RobertGreenberg/Documents/InstructLab/venv/lib/python3.11/site-packages/click/core.py", line 1161, in __call__
    return self.main(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/RobertGreenberg/Documents/InstructLab/venv/lib/python3.11/site-packages/click/core.py", line 1082, in main
    rv = self.invoke(ctx)
         ^^^^^^^^^^^^^^^^
  File "/Users/RobertGreenberg/Documents/InstructLab/venv/lib/python3.11/site-packages/click/core.py", line 1697, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
                           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/RobertGreenberg/Documents/InstructLab/venv/lib/python3.11/site-packages/click/core.py", line 1443, in invoke
    return ctx.invoke(self.callback, **ctx.params)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/RobertGreenberg/Documents/InstructLab/venv/lib/python3.11/site-packages/click/core.py", line 788, in invoke
    return __callback(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/RobertGreenberg/Documents/InstructLab/venv/lib/python3.11/site-packages/click/decorators.py", line 33, in new_func
    return f(get_current_context(), *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/RobertGreenberg/Documents/InstructLab/venv/lib/python3.11/site-packages/instructlab/clickext.py", line 323, in wrapper
    return f(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^
  File "/Users/RobertGreenberg/Documents/InstructLab/venv/lib/python3.11/site-packages/instructlab/model/chat.py", line 232, in chat
    backend_instance = backends.select_backend(
                       ^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/RobertGreenberg/Documents/InstructLab/venv/lib/python3.11/site-packages/instructlab/model/backends/backends.py", line 124, in select_backend
    from .llama_cpp import Server as llama_cpp_server
  File "/Users/RobertGreenberg/Documents/InstructLab/venv/lib/python3.11/site-packages/instructlab/model/backends/llama_cpp.py", line 14, in <module>
    from llama_cpp import llama_chat_format, llama_token_get_text
  File "/Users/RobertGreenberg/Documents/InstructLab/venv/lib/python3.11/site-packages/llama_cpp/__init__.py", line 1, in <module>
    from .llama_cpp import *
  File "/Users/RobertGreenberg/Documents/InstructLab/venv/lib/python3.11/site-packages/llama_cpp/llama_cpp.py", line 83, in <module>
    _lib = _load_shared_library(_lib_base_name)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/RobertGreenberg/Documents/InstructLab/venv/lib/python3.11/site-packages/llama_cpp/llama_cpp.py", line 74, in _load_shared_library
    raise FileNotFoundError(
FileNotFoundError: Shared library with base name '/home/developer/miniconda3/envs/labEnv/lib/python3.11/site-packages/llama_cpp/libllama.so' not found

Same issue with a new venv created and it's all on python 3.11 now.

miniconda messed it all up, I unset my LD_LIBRARY_PATH and LLAMA_CPP_LIB to the new venv and got it working!


双显卡，两张3090显卡，全精度推理， CUDA out of memory

是否已有关于该错误的issue或讨论？ | Is there an existing issue / discussion for this?

[X] 我已经搜索过已有的issues和讨论 | I have searched the existing issues / discussions

该问题是否在FAQ中有解答？ | Is there an existing answer for this in FAQ?

[X] 我已经搜索过FAQ | I have searched FAQ

当前行为 | Current Behavior
两张3090显卡，用官方脚本推理对话，对话轮次三四轮后，出现
torch.cuda.OutOfMemoryError: CUDA out of memory. Tried to allocate 20.00 MiB. GPU 0 has a total capacty of 23.69 GiB of which 16.19 MiB is free. Including non-PyTorch memory, this process has 23.66 GiB memory in use. Of the allocated memory 22.34 GiB is allocated by PyTorch, and 207.43 MiB is reserved by PyTorch but unallocated. If reserved but unallocated memory is large try setting max_split_size_mb to avoid fragmentation.  See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF
实际上其中一张目前只占用了大概6G，24G显存还有很多没用
期望行为 | Expected Behavior
两张显卡显存可以平分，不是一张满了，另外一张还比较空
复现方法 | Steps To Reproduce
readme中的推理脚本，再加两三轮
运行环境 | Environment
- OS:Ubuntu 20.04
- Python:3。9
- Transformers:4.27.1
- PyTorch:2.1.0
- CUDA (`python -c 'import torch; print(torch.version.cuda)'`):12.1

备注 | Anything else?
No response

由于您已在https://github.com/QwenLM/Qwen/issues/450中表示相关问题已解决，本issue先关闭了。


more error message for remote training service

What would you like to be added: could add more error information for remote training service. for example, if the directory /tmp/nni-experiments has incorrect permission, error message should be output in nnimanager.log. Second, when trialkeeper is not successfully started, error message should be output in nnimanager.log and nnimanager should do something for it.
Why is this needed: more friendly to users and developers
Without this feature, how does current nni work：it can work normally

both issues had been fixed in 2.0 bug bash - @QuanluZhang to verify.

both issues had been fixed in 2.0 bug bash - @QuanluZhang to verify.

@liuzhe-lz Please help check this issue, thanks!


Add cargo test step to CI

This step would be a stopgap until colcon test is implemented for the ament_cargo build type.

I'll do it.

Do we run this for every package?

Let's exclude rclrs_examples for now too.

Forgot to close this!


Update jekyll.yml

GitHub Copilot Summary
This pull request includes updates to the Jekyll site deployment workflow to ensure it runs on both push and pull request events, and adds conditional steps for pull requests. The most important changes include adding the pull_request trigger, and adding conditions to the build and deploy steps.
Improvements to workflow triggers and conditions:

.github/workflows/jekyll.yml: Added pull_request trigger to run the workflow on pull requests targeting the master branch.
.github/workflows/jekyll.yml: Added condition to the Upload artifact step to skip it during pull request events.
.github/workflows/jekyll.yml: Added condition to the deploy job to skip it during pull request events and specified that it needs the build job to complete.

Breaking change

Don't know, maybe.
Proposed change

Build on PR, but no upload of artifact nor deploy job.
Intent is to build on PR to catch build errors before merge to master. Did I get this right? Or will this change mess something up?
Type of change

[ ] Dependency upgrade
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature
[ ] Breaking change (fix/feature causing existing functionality to break)
[x] Code quality improvements to existing code or addition of tests

Additional information

This PR fixes or closes issue: fixes #
This PR is related to issue:
Link to documentation pull request:

Checklist

[ ] The code change is tested and works locally.

The way it built and excluded the deploy https://github.com/frdedynamics/hvl_robotics_website/actions/runs/10925925037 looks promising. I guess this should work

any other concenrs @MOJOliciousFTW ?

Nice 💯 now we just need more pull requests hehe


oauth2client now / still installs test files

after commit 93c56c1152dab1fb6f3b9c12f120a3ac77d4c442 by @dhermes, python setup.py install now includes the tests/ subdir when it should not.  it seems the change to the MANIFEST file is not sufficient.
$ python setup.py build
$ python setup.py install --root $PWD/foo
$ find foo/ -name tests
foo/usr/lib64/python3.4/site-packages/tests

Thank you for the report!

Interesting. The whole point of #382 was to keep them out, and prior to that they were included. @vapier Can you confirm the latest commit in your checkout?
git log -1 --pretty=%H

i noticed using the 2.0.0 release tarball, so i then grabbed git and bisected it down to that commit.  i doubt my local git repo being at the wrong git sha1 is the issue.

Ha sounds like you know what you're doing :grinning:. I'm AFK and was just hoping it'd be an easy resolution.

@vapier I'm not sure why this occurs, but at least MANIFEST.in keeps them out of the distribution that gets uploaded to PyPI:
$ tar -xvzf oauth2client-2.0.0.tar.gz
oauth2client-2.0.0/
oauth2client-2.0.0/oauth2client/
oauth2client-2.0.0/oauth2client/contrib/
oauth2client-2.0.0/oauth2client/contrib/django_util/
oauth2client-2.0.0/oauth2client/contrib/django_util/__init__.py
oauth2client-2.0.0/oauth2client/contrib/django_util/apps.py
oauth2client-2.0.0/oauth2client/contrib/django_util/decorators.py
oauth2client-2.0.0/oauth2client/contrib/django_util/signals.py
oauth2client-2.0.0/oauth2client/contrib/django_util/site.py
oauth2client-2.0.0/oauth2client/contrib/django_util/storage.py
oauth2client-2.0.0/oauth2client/contrib/django_util/views.py
oauth2client-2.0.0/oauth2client/contrib/__init__.py
oauth2client-2.0.0/oauth2client/contrib/appengine.py
oauth2client-2.0.0/oauth2client/contrib/devshell.py
oauth2client-2.0.0/oauth2client/contrib/dictionary_storage.py
oauth2client-2.0.0/oauth2client/contrib/django_orm.py
oauth2client-2.0.0/oauth2client/contrib/flask_util.py
oauth2client-2.0.0/oauth2client/contrib/gce.py
oauth2client-2.0.0/oauth2client/contrib/keyring_storage.py
oauth2client-2.0.0/oauth2client/contrib/locked_file.py
oauth2client-2.0.0/oauth2client/contrib/multistore_file.py
oauth2client-2.0.0/oauth2client/contrib/xsrfutil.py
oauth2client-2.0.0/oauth2client/__init__.py
oauth2client-2.0.0/oauth2client/_helpers.py
oauth2client-2.0.0/oauth2client/_openssl_crypt.py
oauth2client-2.0.0/oauth2client/_pure_python_crypt.py
oauth2client-2.0.0/oauth2client/_pycrypto_crypt.py
oauth2client-2.0.0/oauth2client/client.py
oauth2client-2.0.0/oauth2client/clientsecrets.py
oauth2client-2.0.0/oauth2client/crypt.py
oauth2client-2.0.0/oauth2client/file.py
oauth2client-2.0.0/oauth2client/service_account.py
oauth2client-2.0.0/oauth2client/tools.py
oauth2client-2.0.0/oauth2client/util.py
oauth2client-2.0.0/oauth2client.egg-info/
oauth2client-2.0.0/oauth2client.egg-info/PKG-INFO
oauth2client-2.0.0/oauth2client.egg-info/SOURCES.txt
oauth2client-2.0.0/oauth2client.egg-info/dependency_links.txt
oauth2client-2.0.0/oauth2client.egg-info/requires.txt
oauth2client-2.0.0/oauth2client.egg-info/top_level.txt
oauth2client-2.0.0/MANIFEST.in
oauth2client-2.0.0/README.md
oauth2client-2.0.0/setup.py
oauth2client-2.0.0/PKG-INFO
oauth2client-2.0.0/setup.cfg

@tseaver any idea why python setup.py install --root doesn't use MANIFEST.in
StackOverflow seems to indicate that include_package_data=True does the trick but I just tried and tests are still included.


Changelog not auto updating with content

When semantic-release/release-notes-generator is triggered, it had a blank output.
This is likely due to using custom triggers. The last update to 1.0.2 didn't have feat, fix, or breaking, but instead triggered on (api).
See https://github.com/semantic-release/release-notes-generator/issues/153

:tada: This issue has been resolved in version 1.0.4 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Re-using the C parser

Hi,
I was wondering if one might be able to reuse the C parser. Since I use the C to MIR translator in Ravi, I would like to also implement an LLVM backend for the MIR C parser, so that I can easily compare the performance of LLVM vs MIR.
Is there a reason why this would not be possible?
Thanks and Regards
Dibyendu

There are no obstacles to do this according to my license.  It is only difficult to me understand what are you going to achieve.

If you want to compare speed of code generated by c2mir to clang/llvm, you can do this now.  I use make c2mir-bench to evaluate a generated code performance for a new port.  You can do the same for bigger benchmarks.  Of course, if they are written on standard C11 (many most interesting program/benchmarks use a lot of C extensions).
If you want to compare C parsers speed (c2mir and clang), I can say that raw speed of clang should be faster because c2mir parser was not designed for the speed.  It uses backtracking and pass separation for simplicity.  C2MIR parser should have much faster startup than Clang or GCC parser.  So it should be faster on small C programs (saying < 50 lines).
If you want to use C2MIR parser as replacement for clang, it is very difficult work.  Clang and GCC supports so many extensions and builtins.  Implementing it would mean the same amount of work as writing the current c2mir itself.  Also nobody is interesting in C only parsers.  People are interesting in C/C++ parsers and that is in order magnitude bigger work.

Also implementing MIR C/LLVM could be done from MIR C AST or from MIR itself.  I think MIR->LLVM IR is more doable work.  But I can see right now where it could be used.  Therefore I don't put this on my possible future diagram https://github.com/vnmakarov/mir#the-possible-future-state-of-mir-project
For example, MIR->Java byte code has sense because it permits to run MIR code on JAVA VM.  Java byte code -> MIR could be used for alternative implementation of languages targeting JVM.  Goals MIR->LLVM IR could be achieved by MIR->C.  The speed of MIR to C is not issue as a C parser in optimized compilers consumes a tiny portion of CPU time.

Hi,
I was thinking of starting at the AST level and translating to LLVM IR. Just as C2MIR translates to MIR. I already have LLVM support in Ravi ... the alternative is to link Libclang and do it that way but Libclang is a huge library. But maybe using Libclang is better as then I would be able to see the real world difference between Clang and MIR.
The goal is to see the runtime performance of JITed code - comparing LLVM with MIR - both starting with C code

The goal is to see the runtime performance of JITed code - comparing LLVM with MIR - both starting with C code.

I see.  Yes, it is interesting.  Actually the whole project was started because to solve some problems with current Ruby JIT which uses GCC/LLVM as a JIT compiler and C as an interface language.
Definitely the code generated by MIR will be slower (may be even in 2 times) but because of faster MIR compilation I can use more aggressively inlining and speculation/deoptimization technique (inlining and speculation have a very good potential to generate a good JITted code for Ruby). The future will tell how my ideas will work when MIR is used for Ruby JIT.  I am looking forward to start MIR Ruby JIT work but MIR is not ready for this yet.  I still need to implement speculation support in MIR and C2MIR, stabilize MIR project, and make a MIR release for this.

I should add that I need to see this in context - i.e. with Ravi, rather than standalone C programs, because optimizing Ravi code is more challenging

The future will tell how my ideas will work when MIR is used for Ruby JIT. I am looking forward to start MIR Ruby JIT work but MIR is not ready for this yet. I still need to implement speculation support in MIR and C2MIR, stabilize MIR project, and make a MIR release for this.

My experience is that MIR does well when type information is available and is with 2x of LLVM performance. But when type info is lacking, I don't see much difference compared to interpreted code, even worse sometimes.


chore: update README with new NodeJS version

changes node version requirements to 19 to support crypto being an included module by default in global

@Ansonhkg @DashKash54
Curious how you think we should handle this. Thinking we could also just require node v20 as it is a named LTS release. Testing has found it to support global crypto out of the box.

Closing as for implementations directly in our SDK packages we do not require a version bump to the runtime. Will be handled with documentation updates.


Fix Links

closes #5

Should work now — the autolinker package didn't convert mails properly, fixed it in fork. Lmk what you think.

bump

bump

just merged it in! had to fix some packages that were blocking deployment so sorry it took so long, and thanks for the contribution!


Resource sharing amongst users at the template level

Summary
I would like to create workflows on behalf of 3rd party users, and I would like those users to be able to share resources equally at the template level via "user-level" parallelism.
Potential interface
One instantiation of such a feature could look like this (note that I continue to update this as I experiment more with Argo codebase):
The workflow spec has a ResourceSharingID that identifies a single participant who will receive the same number of resources as all other participants (basically, a "user").
A workflow could have this ID passed along like this:
apiVersion: argoproj.io/v1alpha1
kind: Workflow
metadata:
  generateName: pipeline-
spec:
  entrypoint: sg-wrapper
  resourceSharingID: abc-1233333
  templates:

but more realistically, when the workflow is created, it'd be passed in dynamically:
argo submit -l resourceSharingID=abc123333 workflow.yaml

Resource limits could be set in the following way. In workflow-controller-configmap:
  resourcesAvailable: |
    template-name-1: "4"
    template-name-1: "20"

Any templates that are not defined here (or are not used in steps / group) will bypass resource sharing checks, (but will still be subject to all other parallelism checks).
Details / edge cases:

If parallelism for a particular step is defined, AND a resource limit is set for it's template, the behavior is undefined but the Workflow will submit.

Workflows that do not contain resourceSharingID will have no limits set on pods with regards to available resources.

Notes:

I do not think such behavior exists (trivially) at the moment, but if I have missed it I'd love to be pointed in the right direction! (I have written a cron that adjusts template-level parallelism on an interval, but this approach seems bad for a variety of reasons)

I have already set up the codebase in a dev container and have been reading/experimenting with it. I'm beginning to get a feel for what would need to be done - I would be more than happy to work with a mentor to bring this feature back upstream (as I need this feature for a project, I'll probably be implementing either way).

Use Cases
I am currently building software for a render farm that processes long-running workflows (several hours to days). Access to our resources is bottlenecked by a finite number of provisioned software licenses for our farm.

Message from the maintainers:
Love this enhancement proposal? Give it a 👍. We prioritise the proposals with the most 👍.

Slides: https://docs.google.com/presentation/d/1izeZeHZxz54P3njVRAJVVczMpDmGtkU7hD_CUtbaqgY/edit#slide=id.g13af2caef18_0_159


Add tests to verify binding to default address when one loopback is already bound to port 5000

As a follow-up to https://github.com/aspnet/KestrelHttpServer/pull/1451#pullrequestreview-25352477

It's tricky to replicate the expected behaviour in the tests. We fail to start localhost if either of the localhost addresses (ipv4 and ipv6) is already bound. However, if we receive an UVException that is not EADDERINUSE then we try the other address and only fail if neither address works.

@JunTaoLuo Is this good to close now?

We did not come up with a good way to test this. As mentioned above, it's hard to reproduce the expected behaviour.

Oh yeah, this was about trying to replicate the behavior we see in Antares (where we fail to bind on IPv6 but not due to EADDRINUSE and just log a warning). EADDRINUSE is already covered in existing tests.


rgw: remove variable radosgw_num_instances

Change the existing rgw naming conventions to use
the inventory_hostname instead of rgw+{radosgw_num_instance}.
This allows multiple rgws to be declared in the
inventory on the same host but with a different
port.
Signed-off-by: Ali Maredia<EMAIL_ADDRESS>

@alimaredia I think we have to discuss this before going further

unstale

Closing in favor of https://github.com/ceph/ceph-ansible/pull/5138


DetourCreateProcessWithDllsW with LPCWSTR dll names

Hello Detours Team,
currently, it is not possible to call DetourCreateProcessWithDllsW with wide character dll names, because DetourCreateProcessWithDllsW uses LPCSTR and StringCchLengthA to count the dll name length. Please change the DetourCreateProcessWithDllsW, so that it can consume wide chars for dll names respectively.
Thank you!
https://github.com/microsoft/Detours/blob/edc8b07ae7e7325d9b9d551b46122a82665161b8/src/creatwth.cpp#L1505

I think it comes down to UPDATE_IMPORTS_XX in uimports.cpp. Unfortunately, I do not know enough of process structure to guess what you would need to do with this for it to work with wide LPCWSTRs.
In my case I currently just copy the DLL to the target directory if needed, though have also considered using subst.

use LPCSTR is because createwth.cpp use IAT to inject your dlls to destination process， so the imported dll name items must be ANSI version string，so it is not a problem.

Closing this as won't fix, as @sonyps5201314's explanation is correct. There's not much we can do here.

Closing this as won't fix, as @sonyps5201314's explanation is correct. There's not much we can do here.


Adding arduino.path to settings requires a restart

Editing arduino.path requires a restart of VSCode in order to start the Board Manager.

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

ID: 0cf9b53c-0438-f23c-fd2e-d57481d57e85
Version Independent ID: f01afd66-d093-ac44-2b20-df225530b39e
Content: IoT DevKit to cloud -- Connect IoT DevKit AZ3166 to Azure IoT Hub
Content Source: articles/iot-hub/iot-hub-arduino-iot-devkit-az3166-get-started.md
Service: iot-hub
GitHub Login: @wesmc7777
Microsoft Alias: wesmc

@whaakman Thanks for letting us know! We're investigating this and will get back to you shortly.

@whaakman Thanks for the feedback. It is mentioned in the VS code and also in the description page.

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


When using a TestRunner, scenarios & steps considered to be tests

Hey,
We have hooked up our Karate suite with TeamCity for CI reporting.
We were outputting to JSON and using https://github.com/damianszczepanik/maven-cucumber-reporting to convert the JSON into HTML.
When we run the test suite via a test runner (either via TeamCity or IntelliJ), we see odd reporting
6 Scenarios (6 passed)
117 Steps (117 passed)
0m4.857s
[INFO] Tests run: 123, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 5.346 s - in XTestRunner
[INFO]
I would expect a scenario to be a test, and a step to just be a step. Why are we adding steps + scenarios = no of tests?
If we run just the feature file invoked by the runner we get
16:35:41.985 [main] DEBUG com.intuit.karate.http.HttpClient - response time in milliseconds: 78
6 Scenarios (6 passed)
117 Steps (117 passed)
0m4.884s
I have a feature file with 6 scenarios but TeamCity shows 123 tests, I want it to show just 6 tests!
In addition - I am struggling how to work out how to add a custom formatter such as
https://github.com/ankurcha/cucumber_teamcity/
https://github.com/kevinrood/teamcity_formatter
I have seen the plugin_classes in the PluginFactory.java file within cucumber.core:1.2.5
Sorry if this isn't a true issue per sai - I couldn't find a link to the Karate DSL Slack channel to ask this on instead
Thanks

yes the "default" JUnit test reporting will count every step as a test. do the following:

use the parallel test runner: https://github.com/intuit/karate#parallel-execution
set it up in maven so that only this test-runner is executed, refer to the last 2 rows on this page: https://github.com/intuit/karate/tree/master/karate-demo
I am assuming team-city understands the JUnit XML which the above will output.
I would try to get the above working before looking at custom formatters. cucumber has some issues so if you use a custom formatter, you will lose the ability to run tests in parallel

Fab thanks!
Run 52 shows the changes after switching to the junit runner.
#52	 Tests passed: 10
#51	 Tests passed: 64

an update, v0.4.2 now supports integrating 3rd party reports which can do this: https://github.com/intuit/karate/tree/master/karate-demo#example-report


TextBox.cs OnKeyDown does not respect e.Handled = true

After code review i did not found any line that checks if e.Handled == true
on base OnKeyDown function entry.
I have an inhertited TextBox Control (SetStyle).
I overwrite OnKeyDown.
I press a key and set e.Handled = true;
The base TextBox does not check if e.Handled is true;
I circumvent this issue by not calling base.OnKeyDown if i do not
want allow the key being forwarded to base

Your OnKeyDown handler is most likely called after textbox has already handled the event. You need to subscribe to the event with Tunnel routing strategy (aka PreviewXXX in WPF).

Also code example:
https://github.com/AvaloniaUI/Avalonia/issues/5244#issuecomment-753239690

How can the TextBox call OnKeyDown when i overwrite it.
This breaks any rule.
Whe i overwrite OnKeyDown and do not call base.OnKeyDown
then the OnKeyDown of the base class is never allowed to execute.
Or do i miss something in OOP?
regards.

If it will help you to understand, KeyDownEvent property is the entry-point of this event.
When you subscribe to C#-event KeyDown += Control_KeyDown it adds handler to this KeyDownEvent with Bubble routing strategy (event handling order is from the source where event occurred, textbox in this case, to the root).
When you override OnKeyDown event, it has pretty much the same behavior, because OnKeyDown is invoked also with Bubble routing strategy (see here, bubble is default strategy, so it's not passed as a parameter explicitly).
So when you want to handle event with Tunnel routing strategy (when event raises handlers in order from the root to the source), you need to use KeyDownEvent property directly, without KeyDown C#-event or OnKeyDown virtual method:
var textBox = new TextBox();
textBox.AddHandler(InputElement.KeyDownEvent, TextBox_KeyDown, RoutingStrategies.Tunnel);

Also this documentation page describes pretty much the same, but in general case: https://avaloniaui.net/docs/input/events

Thank you.
Your reply helped a lot.


Warning: Support for defaultProps will be removed from function components in a future major release

I am hoping to use this with Next.js v13/v12. Just tried and got this error on terminal:
Warning: ReactSpreadsheetImport: Support for defaultProps will be removed from function components in a future major release. Use JavaScript default parameters instead.
    at ReactSpreadsheetImport (webpack-internal:///(sc_client)/./node_modules/.pnpm/react-spreadsheet-import@2.0.8_5dirdw6vksawy263lri76kextq/node_modules/react-spreadsheet-import/dist-commonjs/ReactSpreadsheetImport.js:30:38)
    at Home (webpack-internal:///(sc_client)/./app/(dashboard)/dashboard/page.tsx:13:17)

This problem should be fixed with version >4.

@masiulis I just faced the same warning using v4.6.1 in a fresh<EMAIL_ADDRESS>app. Any ideas?


API search endpoint

Hello all,
Is there more information than this about the search endpoint available?
E.g. is it possible to search for ranges or dates older/newer than etc.?
Thank you and best regards,
Stefan

Hello,
the API documentation is not complete yet. I have created a new ticket in our issue tracker https://issues.shopware.com/issues/NEXT-4643
You can do the following to filter dates:
POST http://shopware.next/api/v1/search/product
{ 	"filter": [ 		{ 			"type": "range", 			"field": "createdAt", 			"parameters": { 				"gt": "2018-10-05 0:0:0" 			} 		}, 		{ 			"type": "range", 			"field": "createdAt", 			"parameters": { 				"lt": "2020-10-05 0:0:0" 			} 		} 	] }

Hey,
Awesome! Thank you very much. I'll give it a try.
Best regards,
Stefan


Support RDS IAM authentication

Hi,
It seems that currently RDS IAM authentication for mysql/postgresql users is not possible with the provider.
Here are the official docs: https://docs.aws.amazon.com/AmazonRDS/latest/UserGuide/UsingWithRDS.IAMDBAuth.DBAccounts.html
For mysql it is necessary to set AWSAuthenticationPlugin when creating the user.
For postgresql the user must be granted the rds_iam role.
This possible with the Terraform providers:
https://registry.terraform.io/providers/bangau1/mysql/latest/docs/resources/user
https://registry.terraform.io/providers/cyrilgdn/postgresql/latest/docs/resources/postgresql_grant_role

Hi,
I want to add the AWSAuthenticationPlugin  to MySQL.
Would someone be able to point me to / guide me?
I'm good with GO but didn't contribute here yet :)

This works for RDS IAM auth:
apiVersion: postgresql.sql.crossplane.io/v1alpha1
kind: ProviderConfig
metadata:
  name: default
spec:
  defaultDatabase: example
  sslMode: require
  credentials:
    source: PostgreSQLConnectionSecret
    connectionSecretRef:
      namespace: default
      name: example-secret
---
apiVersion: postgresql.sql.crossplane.io/v1alpha1
kind: Role
metadata:
  name: examplerole
  annotations:
    crossplane.io/external-name: "you_can_also_have_underscores_in_role_name"
spec:
  forProvider:
    privileges:
      login: true
---
apiVersion: postgresql.sql.crossplane.io/v1alpha1
kind: Grant
metadata:
  name: mainrole
spec:
  forProvider:
    memberOf: exampleuser
    roleRef:
      name: examplerole
---
apiVersion: postgresql.sql.crossplane.io/v1alpha1
kind: Grant
metadata:
  name: iamrole
spec:
  forProvider:
    memberOf: rds_iam
    roleRef:
      name: examplerole

Considering the last answer, it seems that this is solved.


Unable to trigger "ZotLit: Update literature note" with hotkey

I know this action is available from the kebab menu on the note pane, but I typically navigate the interface with my keyboard for actions like this. I am able to assign the hotkey in the settings, but the plugin doesn't respond to it.
Steps to reproduce:

Open Zotero and Obsidian with ZotLit installed on both.
Create a new literature note from Zotero.
In Obsidian, open Settings > Hotkeys and search "Update" in the search filter.
Give ZotLit: Update literature note a hotkey that is not currently being used. (For me, Ctrl + U was free.)
In Zotero, make an edit that will appear in your literature note on an update, such as a new annotation.
In Obsidian, navigate to your matching literature note, ensure cursor focus is in that tab/pane, and press your selected hotkey.

Result:

No change in literature note.
Triggering an update from the kebab menu works as expected.
No warnings or errors provided by the Obsidian interface or in the inspector.

Other Notes:

This does not seem to be an issue with Obsidian or this particular hotkey assignment. I moved the assignment to a different plugin (Zotero Integration) and the action fired as expected from them.
This action also doesn't show up in the command pallet, so even though I can pin it there from the settings menu, it still doesn't show up and can't be activated by a few keystrokes.
Closing Obsidian and launching it again does not fix this issue.

I'm sorry to tell you that the authors have informed you that this feature is not being developed at the moment, just a preset button in the zotero plugin. Currently, only the open button is working normally.
When the page is literature for note, you can click on more options in the upper right corner and then select "Update Documentation Notes" to update annotations. Note: This is a complete override operation.

Thanks, I noted this behavior already in the "Other Notes" section above.
Would love to contribute to some of the missing features, but I can't seem to find a build guide.


Jcenter

The last available version of the Parceler on jcenter is 1.0.1 https://jcenter.bintray.com/org/parceler/parceler/

Parceler is distributed via maven central: http://search.maven.org/#artifactdetails|org.parceler|parceler|1.1.5|jar

hmm, I wonder if there is a way to sync with jcenter...

Ah, it seems that when you take ownership of your repository in jcenter it stops automatically syncing from maven central.  Therefore, I gave up ownership of the parceler repositories and it re-synced with maven central.  You'll find the latest in jcenter now.

Thank you very much for help! Everything works perfect now :)


fix: Zero translated string field

1. Why is this change necessary?
For example try to create a variant with the name "0":

2. What does this change do, exactly?
Change the hydration of translated properties, to check explicitly against null instead of an empty value.
3. Describe each step to reproduce the issue or behaviour.
Create a property option with name "0".
4. Please link to the relevant issues (if any).
-
5. Checklist

[x] I have rebased my changes to remove merge conflicts
[x] I have written tests and verified that they fail without my change
[x] I have created a changelog file with all necessary information about my changes
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

The tests seems to fail, since the job started before 0 UTC, but the document was generated after 0 UTC.

Thanks!


Documentation: Write Parley Motivation/Introduction Section

Problem
While the current README.md documents usage and development, it is lacking the "motivation" behind the project: what problem does this software aim to solve?
Solution
Need to add an introduction section where this is discussed.

Implemented with: 14c74fa0638738a0f27e57a722249fbfd67bc776


Towny doesn't restrict custom enchantments.

Towny is able to restrict PVP in users' towns but even if PVP is restricted in users' towns they can still use custom enchantments on swords to hit each other, a prime example being using the tornado enchantment even if PVP is disabled. The sword won't register the hit but it will still execute the tornado effect.

Looks like Towny handles and cancels events on HIGH/HIGHEST instead of LOW/LOWEST priority, which would cause issues with a lot of plugins (not just mine) listening for events on lower priorities such as NORMAL.
Fixed with this PR to Towny: https://github.com/TownyAdvanced/Towny/pull/3329

Re-opening because apparently Towny's priorities are a hack for WG's priorities, so now I get to add in a hack for this plugin.
Plugins are lovely.

Just read the thread, that's unfortunate they needed to use that priority just for worldguard. Thanks for the quick support and dive into the issue, your plugin's great!


Support for nx v18

npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/@nx/devkit
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer @nx/devkit@">= 16 < 18" from<EMAIL_ADDRESS>npm ERR! node_modules/@koliveira15/nx-sonarqube
npm ERR!   dev<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.

@jahusa02 thanks for the contribution ❤ Please address my comments in the PR & getting a passing CI run and I will accept and merge.


#992 Add documentation for mbeans configuration

LGTM

If i am reading the network right it needs a rebase against 3.0.

Did a minor addition to the doc.
As for the rebase, no as it needs to be merged to master as well in a separate PR.
I am long overdue for a mail to ehcache-dev indicating the bug fixing strategy we decided upon.


Remove the bogus setup_requires() section.

Its unnecessary, and likely the cause for pip installs of the library through other libraries failing.

Sorry @brantai, my previous commit included a bug that only exhibited itself when you installed python-rightscale as a dependency for another package. Strange corner case, but its fixed here thanks to the guys on the #pypa channel.

seems like this pr should be closed.  @diranged please reopen if i've misunderstood.

So I can't re-open this .. but its actually a valid pull request. Without this code change, you can't install python-rightscale as a dependency of another application (like we do with Kingpin). I highly suggest re-opening this and merging it. I checked in with the #PyPA channel guys and they recommended this fix.


[Bug]: Welcome Mailer received on every signin request

Is there an existing issue for this?

[X] I have searched the existing issues

Description
Everytime I login using google, I receive a welcome email.
Slack Thread: https://theappsmith.slack.com/archives/CGBPVEJ5C/p1694063599126149
Steps To Reproduce

Signin on app.appsmith.com using google auth
Check your inbox for the welcome emailer.

Edit
Signin using username password also triggers welcome emailer
Ref: https://theappsmith.slack.com/archives/CGBPVEJ5C/p1694433365069269?thread_ts=1694063599.126149&cid=CGBPVEJ5C
Public Sample App
No response
Environment
Production
Issue video log
No response
Version
v1.9.36-SNAPSHOT

Validated on release Appsmith v1.9.39-SNAPSHOT


UI for New Schedule Page

Problem Description
Based on the code that was written by Willie, I will use Material-UI and make that page look pretty.

Closing due to being stale and not including enough information to be actionable.


Configure logging with utility package

This will make the logs look more consistent, and allow for better run-
time configuration of both logs this controller creates, and the logs
that libraries might create.
Signed-off-by: Justin Kulikauskas<EMAIL_ADDRESS>

/test sonarcloud
(some kind of 500 error?)

Weird. The 500 definitely looks environmental:
Caused by: Something went wrong while trying to get the pullrequest with key '186'
Caused by: org.sonarqube.ws.client.HttpException: Error 500 on https://sonarcloud.io/api/alm_integration/show_pullrequest?project=open-cluster-management_governance-policy-propagator&pullrequestKey=186 : {"errors":[{"msg":"An unexpected error occurred. Please try again later."}]}

Strange there's no retry on that command...


Issue/3067 notification center (#3179)

Pull request opened by the merge tool on behalf of #3179

Merged in 91b4ba63d1288c6180feb4b28b6751fdfb3a0aa6


Spec typos

In section 4.5.6. of the spec:

The return type of the method in the second code example should be Callable<Comparison,Integer,Integer> (two Ls, not one)
The third code example is missing a + before lastName in the return statement.

This issue belongs in ceylon-spec.

Now ceylon/ceylon-spec#222


bitcoin: broken on aarch64-darwin

the build ends up with:
$ nix-shell -p bitcoind
...
...
buildPhase completed in 3 minutes 50 seconds
running tests
check flags: -j8 -l8 SHELL=/nix/store/xksljrjzgx6qc6xvv2gz81vg27xf3rki-bash-5.1-p16/bin/bash LC_ALL=en_US.UTF-8 check
Making check in src
make[1]: Entering directory '/private/tmp/nix-build-bitcoind-23.0.drv-0/bitcoin-23.0/src'
make[2]: Entering directory '/private/tmp/nix-build-bitcoind-23.0.drv-0/bitcoin-23.0/src'
make  check-TESTS check-local
make[3]: Entering directory '/private/tmp/nix-build-bitcoind-23.0.drv-0/bitcoin-23.0/src'
Running tests: addrman_tests from test/addrman_tests.cpp
make[4]: Entering directory '/private/tmp/nix-build-bitcoind-23.0.drv-0/bitcoin-23.0/src'
Running tests: base58_tests from test/base58_tests.cpp
Running tests: base32_tests from test/base32_tests.cpp
/nix/store/xksljrjzgx6qc6xvv2gz81vg27xf3rki-bash-5.1-p16/bin/bash: line 1: 92424 Killed: 9               test/test_bitcoin --catch_system_errors=no -l test_suite -t "`cat test/base58_tests.cpp | grep -E "(BOOST_FIXTURE_TEST_SUITE\\(|BOOST_AUTO_TEST_SUITE\\()" | cut -d '(' -f 2 | cut -d ',' -f 1 | cut -d ')' -f 1`" -- DEBUG_LOG_OUT > test/base58_tests.cpp.log 2>&1
/nix/store/xksljrjzgx6qc6xvv2gz81vg27xf3rki-bash-5.1-p16/bin/bash: line 1: 92425 Killed: 9               test/test_bitcoin --catch_system_errors=no -l test_suite -t "`cat test/base32_tests.cpp | grep -E "(BOOST_FIXTURE_TEST_SUITE\\(|BOOST_AUTO_TEST_SUITE\\()" | cut -d '(' -f 2 | cut -d ',' -f 1 | cut -d ')' -f 1`" -- DEBUG_LOG_OUT > test/base32_tests.cpp.log 2>&1
make[3]: *** [Makefile:19610: test/base58_tests.cpp.test] Error 1
make[3]: *** Waiting for unfinished jobs....
make[3]: *** [Makefile:19610: test/base32_tests.cpp.test] Error 1
/nix/store/xksljrjzgx6qc6xvv2gz81vg27xf3rki-bash-5.1-p16/bin/bash: line 1: 92377 Killed: 9               test/test_bitcoin --catch_system_errors=no -l test_suite -t "`cat test/addrman_tests.cpp | grep -E "(BOOST_FIXTURE_TEST_SUITE\\(|BOOST_AUTO_TEST_SUITE\\()" | cut -d '(' -f 2 | cut -d ',' -f 1 | cut -d ')' -f 1`" -- DEBUG_LOG_OUT > test/addrman_tests.cpp.log 2>&1
make[3]: *** [Makefile:19610: test/addrman_tests.cpp.test] Error 1
PASS: univalue/test/unitester
PASS: univalue/test/object
PASS: univalue/test/no_nul
PASS: minisketch/test
============================================================================
Testsuite summary for Bitcoin Core 23.0.0
============================================================================
# TOTAL: 4
# PASS:  4
# SKIP:  0
# XFAIL: 0
# FAIL:  0
# XPASS: 0
# ERROR: 0
============================================================================
make[4]: Leaving directory '/private/tmp/nix-build-bitcoind-23.0.drv-0/bitcoin-23.0/src'
make[3]: Leaving directory '/private/tmp/nix-build-bitcoind-23.0.drv-0/bitcoin-23.0/src'
make[2]: *** [Makefile:17911: check-am] Error 2
make[2]: Leaving directory '/private/tmp/nix-build-bitcoind-23.0.drv-0/bitcoin-23.0/src'
make[1]: *** [Makefile:17569: check-recursive] Error 1
make[1]: Leaving directory '/private/tmp/nix-build-bitcoind-23.0.drv-0/bitcoin-23.0/src'
make: *** [Makefile:817: check-recursive] Error 1
error: builder for '/nix/store/v3nxvlbki9hbmd6x29gv96mvq7mna5rs-bitcoind-23.0.drv' failed with exit code 2

We can trivially workaround this by disabling the tests for aarch64-darwin:
diff --git a/pkgs/applications/blockchains/bitcoin/default.nix b/pkgs/applications/blockchains/bitcoin/default.nix
index 5ecbf4fc979..6cca61cd6f4 100644
--- a/pkgs/applications/blockchains/bitcoin/default.nix
+++ b/pkgs/applications/blockchains/bitcoin/default.nix
@@ -76,7 +76,7 @@ stdenv.mkDerivation rec {

   checkInputs = [ python3 ];

-  doCheck = true;
+  doCheck = !(stdenv.isDarwin && stdenv.isAarch64);

   checkFlags =
     [ "LC_ALL=en_US.UTF-8" ]

but it would be much better to understand the underlying issue and fix that instead

Fixed by #179795

follow-up fix in PR: https://github.com/NixOS/nixpkgs/pull/183951

but it would be much better to understand the underlying issue and fix that instead

I can see two PRs disabling hardening features, #179795 and #183951, but neither have an explanation of the problem, or why turning off hardening features was the best / correct fix? Can you explain further?
I see the workarounds are still present: https://github.com/NixOS/nixpkgs/blob/866431777b6c766cea0db7aad347824d3816be68/pkgs/applications/blockchains/bitcoin/default.nix#L79 and It would be great to know why this is necessary for Nix. These hardening features work fine for native builds, and are used in our release binaries.

It would be great to know why this is necessary for Nix.

This used to fix "Killed: 9" error while running the tests, but it seems this is not helping anymore. My guess is that code signing is somehow messing with the test binaries.

I think the hardening feature was broken when building with gfortran. The issue has been fixed recently.

Still broken on Hydra with "Killed: 9". Looks like binaries are not signed before checkPhase.

Still broken on Hydra with "Killed: 9". Looks like binaries are not signed before checkPhase.

As can be seen here, the issue is intermittent and does not appear all the time: https://hydra.nixos.org/job/nixpkgs/nixpkgs-unstable-aarch64-darwin/bitcoin.aarch64-darwin
This makes it harder to debug.

Fix in https://github.com/NixOS/nixpkgs/pull/202815


[Usage] The difference between finetune_lora.sh and finetune_task_lora.sh

Describe the issue
Issue:
Hi, you provided two scripts for finetuning with lora: finetune_lora.sh and finetune_task_lora.sh. What's the difference between their usages?
The most difference in this two scripts is: in finetune_lora.sh, the model_name_or_path(line 6) is set to be "lmsys/vicuna-7b-v1.5", and this value is "liuhaotian/llava-v1.5-7b" in finetune_task_lora.sh. Do you type it by mistake or it is exactly right? I'm really confused.
From my best understanding, if I have enough data, I should use finetune_lora.sh; if I have relatively small amount of data, then I should use finetune_task_lora.sh. Am I right?

Hi there, I have been thinking the same question. From my point of view, finetune_lora.sh means finetune from the pretrained mlp layer, so there is an augment --pretrain_mm_mlp_adapter in  finetune_lora.sh, which in absent in  finetune_task_lora.sh. And finetune_task_lora.sh means finetune the llava model, such as llava-1.5, lllava-1.6... Hope that will be helpful.


template: motd: rename Twitter product name to X

It's quite controversial but things kind of change.
I recently pulled the 24.1 update, and noticed it was still mentioned as Twitter.
Here's a patch to correctly refer to the product as of now, which is X.

I'd rather drop this for Reddit (if it fits)

What about now? :)

thanks for your effort, but you need to keep the alignment otherwise you break console rendering

(I mean not shifting everything further right)

Well, in such case, what about if we name the Reddit handle instead of the full URL? I don't think someone would copy the URL from the console and then paste it in a browser. Something like "r/opnsense" or so on?

there should be enough space between the link and the label

This is another feasible option, but to my personal opinion it looks bad without spacing at the right side. If you are good to go with it I'll do another commit.

We can also get that space back if we downgrade the URL to http, but that will look weird considering the other URLs use HTTPS.

how about aa2fc872e3 ?!

Well, that seems to be an option.
The thing is I believed you wouldn't want to remove the TAB/CONTROL
characters, else I'd have replaced them with spaces :P.
But yeah, your commit makes the trick and hence then, this pull request can
be closed.
On Wed, Jul 24, 2024 at 4:01 PM Franco Fichtner @.***>
wrote:

how about aa2fc87
https://github.com/opnsense/core/commit/aa2fc872e3e66c5bba4dd20a919d3e0baa6efb3e
?!
—
Reply to this email directly, view it on GitHub
https://github.com/opnsense/core/pull/7643#issuecomment-2248047497, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AD4TACOBG4PASYXKFGSUZSTZN6XR7AVCNFSM6AAAAABLMKXFI6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENBYGA2DONBZG4
.
You are receiving this because you authored the thread.Message ID:
@.***>

yes, well, it served a lazy purpose when we didn't have to salvage these characters for URLs. I'll merge and close then. Appreciate the PR and discussion!


<optim.lbfgs> 	creating recyclable direction/step/history buffers

Hello. I am trying to  run a command and when It starts "optimization with L-BFGS", doesn't continue.
LOL~th neural_style.lua -gpu -1 -style_image /Users/Meir/neural-style/examples/inputs/starry_night.jpg -content_image /Users/Meir/neural-style/examples/inputs/brad_pitt.jpg
[libprotobuf WARNING google/protobuf/io/coded_stream.cc:537] Reading dangerously large protocol message.  If the message turns out to be larger than<PHONE_NUMBER> bytes, parsing will be halted for security reasons.  To increase the limit (or to disable these warnings), see CodedInputStream::SetTotalBytesLimit() in google/protobuf/io/coded_stream.h.
[libprotobuf WARNING google/protobuf/io/coded_stream.cc:78] The total number of bytes read was 574671192
Successfully loaded models/VGG_ILSVRC_19_layers.caffemodel
conv1_1: 64 3 3 3
conv1_2: 64 64 3 3
conv2_1: 128 64 3 3
conv2_2: 128 128 3 3
conv3_1: 256 128 3 3
conv3_2: 256 256 3 3
conv3_3: 256 256 3 3
conv3_4: 256 256 3 3
conv4_1: 512 256 3 3
conv4_2: 512 512 3 3
conv4_3: 512 512 3 3
conv4_4: 512 512 3 3
conv5_1: 512 512 3 3
conv5_2: 512 512 3 3
conv5_3: 512 512 3 3
conv5_4: 512 512 3 3
fc6: 1 1 25088 4096
fc7: 1 1 4096 4096
fc8: 1 1 4096 1000
Setting up style layer  	2	:	relu1_1
Setting up style layer  	7	:	relu2_1
Setting up style layer  	12	:	relu3_1
Setting up style layer  	21	:	relu4_1
Setting up content layer	23	:	relu4_2
Setting up style layer  	30	:	relu5_1
WARNING: Skipping content loss
Running optimization with L-BFGS
<optim.lbfgs> 	creating recyclable direction/step/history buffers
....and not response. What am I doing wrong?
P.S. OS X

By default it only prints every 50 iterations. Since you are running on CPU it might take a while to complete 50 iterations and it looks like it's hanging. Try adding the flag -print_iter 1 to print progress every iteration.

Thanks


[es] feature: add new messages

Gracias

Hola,
can you help us and translation in json file "Verify Email Address" ?
Gracias

Sure! Just a minute.


osx support

On osx-arm64, conda build complains about (I think) a path which is too long. Compressing the electron package appeases it.
Easy: programmatically compress and extract the UI files (this should be done from the python cli, not via post_install, I think)
Preferred: Separate build recipe for UI (this was not possible before, but now that voila is not initialized from javascript, it may be possible now).

Need to get the --platform tag up and running.
When running x86 conda on arm64 hardware, constructor builds for x86 but electron builds for arm64.
At the very least they should agree.

Update. Zipping the output on osx-64 breaks the app. This could be where electron-zip-packager comes in


Initalize the builder for mainnet

Currently the builder is initialized for test-net.
https://github.com/Zondax/ledger-zcash-rs/blob/de3c54f2f4a82991aad176ea278b937bcaabfad1/zcash-hsmbuilder/src/lib.rs#L250

Depends on integration


High level pointer events

As the use cases come up, we can implement the remaining pointer events.

Over
Enter
Leave
Out
Got capture
Lost capture
Cancel

Reference docs: https://developer.mozilla.org/en-US/docs/Web/API/Pointer_events
I think maybe over/out are most immediately valuable in pygfx to support material swapping on hover.

Do you maybe want to keep this open for the other high level events?

For now I don't really see good use cases for them. We can open new issues later.


[signal SIGSEGV: segmentation violation code=0x1 addr=0x48 pc=0x6b096b]

What's the problem (or question)?

Attempting to run it for yahoo.com generates some errors.

I don't believe I'm having any internet issues on my computer. Other scripts that pull data from the web work fine. DNS resolution is working. No firewalls.
Do you have an idea for a solution?

Errors should be handled gracefully. The user should not see the traceback etc. They should be logged, and the user should be shown a friendly error message saying why it failed.
How can we reproduce the issue?

I did the following:
./subfinder -d yahoo.com
Got this response:
[-] Searching For Subdomains in Ask
[-] Searching For Subdomains in Baidu
[-] Searching For Subdomains in Bing
[-] Searching For Subdomains in Censys
[-] Searching For Subdomains in Crt.sh
[-] Searching For Subdomains in CertDB
[-] Searching For Subdomains in Certspotter
[-] Searching For Subdomains in Dnsdb
[-] Searching For Subdomains in Threatcrowd
[-] Searching For Subdomains in Findsubdomains
[-] Searching For Subdomains in DNSDumpster
[-] Searching For Subdomains in PassiveTotal
[-] Searching For Subdomains in PTRArchive
[-] Searching For Subdomains in Hackertarget
[-] Searching For Subdomains in Virustotal
[-] Searching For Subdomains in Securitytrails
[-] Searching For Subdomains in WaybackArchive
[-] Searching For Subdomains in ThreatMiner
[-] Searching For Subdomains in Riddler
[-] Searching For Subdomains in Netcraft

[+] Finding subdomains for : yahoo.com
error: Get https://www.threatcrowd.org/searchApi/v2/domain/report/?domain=yahoo.com: dial tcp [2400:cb00:2048:1::681f:b9a]:443: connect: network is unreachable

error: Get https://www.bing.com/search?q=domain%3Ayahoo.com&go=Submit&first=0: dial tcp: lookup www.bing.com on <IP_ADDRESS>:53: read udp <IP_ADDRESS>:47844-><IP_ADDRESS>:53: i/o timeout

error: Get http://ptrarchive.com/tools/search2.htm?label=yahoo.com&date=ALL: dial tcp: lookup ptrarchive.com on <IP_ADDRESS>:53: read udp <IP_ADDRESS>:45089-><IP_ADDRESS>:53: i/o timeout

error: Get http://www.dnsdb.org/f/yahoo.com.dnsdb.org/: dial tcp [2400:cb00:2048:1::681f:5769]:80: connect: network is unreachable

error: Get https://www.threatminer.org/getData.php?e=subdomains_container&q=yahoo.com&t=0&rt=10&p=1: dial tcp [2400:cb00:2048:1::6819:b105]:443: connect: network is unreachable

error: Get https://api.hackertarget.com/hostsearch/?q=yahoo.com: dial tcp: lookup api.hackertarget.com on <IP_ADDRESS>:53: read udp <IP_ADDRESS>:56787-><IP_ADDRESS>:53: i/o timeout

error: Get https://certdb.com/domain/yahoo.com: dial tcp: lookup certdb.com on <IP_ADDRESS>:53: read udp <IP_ADDRESS>:36248-><IP_ADDRESS>:53: i/o timeout

error: Get https://certspotter.com/api/v0/certs?domain=yahoo.com: dial tcp: lookup certspotter.com on <IP_ADDRESS>:53: read udp <IP_ADDRESS>:38982-><IP_ADDRESS>:53: i/o timeout
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x48 pc=0x6b096b]

goroutine 21 [running]:
github.com/Ice3man543/subfinder/libsubfinder/sources/riddler.Query(0xc4200ba200, 0xc420096420)
	/home/user/go/src/github.com/Ice3man543/subfinder/libsubfinder/sources/riddler/riddler.go:56 +0x24b
created by github.com/Ice3man543/subfinder/libsubfinder/engines/passive.Discover
	/home/user/go/src/github.com/Ice3man543/subfinder/libsubfinder/engines/passive/passive.go:151 +0xb3b

What are the running context details?

Installation method (e.g. pip, apt-get, git clone or zip/tar.gz): go get
Client OS (e.g. Microsoft Windows 10): Ubuntu 18.04
Program version (see banner): v0.1.0
Relevant console output (if any): See above.
Exception traceback (if any): See above.

Most probably, your internet is not working correctly with SubFinder. Will look at it in a moment.

It seems to be working now. Maybe it was some connectivity issue. Please look into this.
On a side note, I believe it should handle the errors more gracefully. Suggestions:

Before attempting to start getting the data from different sources, ensure that the Internet is working fine, and proceed only if it is working. This could include checking for the response codes etc. from the websites. It would help if the output said which sources did not get any data and why.

If some errors are encountered, they must be logged. Users should only see the basic error messages - just the bare minimum. They don't really want the tracebacks etc. If they need more detailed output, they can use the verbose mode. Or check the logs.

What are your thoughts?


Request Entity Too Large

Describe the bug
While uploading "large" files the upload UI is not very clear on why an upload is failing.
Doing some debugging in the web console I was able to see "Request Entity Too Large".
Pressing the upload button a few times also causes the progress bar to go.. weird
is there a setting to allow larger files?
is there also a setting to allow to queue more analysis ?
To Reproduce
Steps to reproduce the behavior:

EMBArk installation (using EMBABox)
Start EMBArk: sudo ./run-server.sh
Use the firmware available here:430 MB file (ota update)

See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. Kali Linux 2022.01]

Additional context
Add any other context about the problem here.
Not sure but I also got this message (perhaps a race condition while uploading?)
2022-04-19 10:21:09,597 INFO     Listening on TCP address <IP_ADDRESS>:8001
Error notifying listener
Traceback (most recent call last):
  File "/home/vagrant/embark/.venv/lib/python3.9/site-packages/apscheduler/schedulers/base.py", line 836, in _dispatch_event
    cb(event)
  File "/home/vagrant/embark/.venv/lib/python3.9/site-packages/django_apscheduler/jobstores.py", line 100, in handle_execution_event
    job_execution = DjangoJobExecution.atomic_update_or_create(
  File "/home/vagrant/embark/.venv/lib/python3.9/site-packages/django_apscheduler/util.py", line 99, in func_wrapper
    result = func(*args, **kwargs)
  File "/home/vagrant/embark/.venv/lib/python3.9/site-packages/django_apscheduler/models.py", line 165, in atomic_update_or_create
    job_execution = DjangoJobExecution.objects.select_for_update().get(
  File "/home/vagrant/embark/.venv/lib/python3.9/site-packages/django/db/models/query.py", line 443, in get
    raise self.model.MultipleObjectsReturned(
django_apscheduler.models.DjangoJobExecution.MultipleObjectsReturned: get() returned more than one DjangoJobExecution -- it returned 2!

It looks like the size can be increased in the startup script .
https://github.com/e-m-b-a/embark/blob/main/run-server.sh#L26

is it now working? I will include a note in the wiki

Increasing the limit does appear to work (hence good to document how to increase the limit) but I will do more testing tomorrow. The UI issue remain. it appears it is possible to press the upload button multiple times causing multiple uploads (arguably this is a different issue) but still related to large files


Flink Oracle CDC data loss[Bug]

Search before asking

[X] I searched in the issues and found nothing similar.

Flink version
nothing
Flink CDC version
2.3.0
Database and its version
nothing
Minimal reproduce step
multiple log events(commit log event) have the same scn value

What did you expect to see?
io.debezium.connector.oracle.logminer.TransactionalBuffer
offsetContext.getCommitScn().compareTo(scn) >= 0
'>= '  should be  '>'
// On the restarting connector, we start from SCN in the offset. There is possibility to
    // commit a transaction(s) which were already committed.
    // Currently we cannot use ">=", because we may lose normal commit which may happen at the
    // same time. TODO use audit table to prevent duplications
    if ((offsetContext.getCommitScn() != null
                    && offsetContext.getCommitScn().compareTo(scn) >

        || lastCommittedScn.compareTo(scn) > 0) {
    LOGGER.debug(
            "Transaction {} already processed, ignored. Committed SCN in offset is {}, commit SCN of the transaction is {}, last committed SCN is {}",
            transactionId,
            offsetContext.getCommitScn(),
            scn,
            lastCommittedScn);
    streamingMetrics.setActiveTransactions(transactions.size());
    return false;
}

What did you see instead?
 if ((offsetContext.getCommitScn() != null                         && offsetContext.getCommitScn().compareTo(scn) **>=** 0)                 || lastCommittedScn.compareTo(scn) > 0) {             LOGGER.debug(                     "Transaction {} already processed, ignored. Committed SCN in offset is {}, commit SCN of the transaction is {}, last committed SCN is {}",                     transactionId,                     offsetContext.getCommitScn(),                     scn,                     lastCommittedScn);             streamingMetrics.setActiveTransactions(transactions.size());             return false;         }
Anything else?
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

Oracle cdc connector may fix this issue in 2.4.0. @qiufang1314 Would you mind helping to check whether this bug still exists in version 2.4.0?

Oracle cdc connector may fix this issue in 2.4.0. @qiufang1314 Would you mind helping to check whether this bug still exists in version 2.4.0?

hello，which version has fixed this bug, we meet the same problem ,wo can check  whether  this bug exists in your version

Oracle cdc connector may fix this issue in 2.4.0. @qiufang1314 Would you mind helping to check whether this bug still exists in version 2.4.0?

hello，which version has fixed this bug, we meet the same problem ,we can check whether this bug exists in your version

@ruanhang1993

@qiufang1314  Had fixed in Release 3.0.1, Please update and retry.
If the issue still exists, please report it on Apache Jira under the project Flink, using the component tag Flink CDC. Thank you!


The smclicshv extension naming (i.e. sm*) suggests it only has M-mode features but it affects S-mode and has one S-mode CSR bit

The smclicshv extension naming (i.e. sm*) suggests it only has M-mode features. Does it have S-mode features too? I ask because the smclicshv chapter uses the 'x' prefix notation instead of 'm'. What gives?

From my side this extension is affecting all mode supported by the CLIC.
If CLIC is supporting only M mode, then this extension will affect only M registers
If CLIC is supporting M and S mode, then this extension will affect M and S mode.
As soon as you get Selective Hardware vectoring supported on M mode, the cost to support it for S is for free.
So x prefix is the good one from my side.
Maybe I forget some aspects. We can discuss this tonight.

i think there is no precedent for an x prefix in an extension.
the normal thing to do is to have a smclicshv and a ssclicshv.
However, here we do not want them to be selectable individually.
So if smclicshv and sslic are supported, ssclicshv is mandatory. Otherwise, ssclicshv is forbidden.

Precedent in Priv ISA manual:
Chapter 5. "Smcsrind/Sscsrind" Indirect CSR Access, Version 1.0.0
5.1. Introduction
Smcsrind/Sscsrind is an ISA extension that extends the indirect CSR access mechanism originally
defined as part of the Smaia/Ssaia extensions, in order to make it available for use by other extensions
without creating an unnecessary dependence on Smaia/Ssaia.
This extension confers two benefits:

Another:
Chapter 4. "Smstateen/Ssstateen" Extensions, Version 1.0.0


Meters showing under minutes

Never seen this before... Is this what is supposed to show if the API returns distance bus is from stop but not time?

@bbodenmiller Are you using Puget Sound region (vs. a custom API URL)?

Yep on Puget Sound.

I found the responsible code here:
https://github.com/OneBusAway/onebusaway-iphone/blob/f41cac0d55d1a48f0e006c72326d89b4702645a8/controls/table_cells/OBAArrivalEntryTableViewCellFactory.m#L50

Yeah, looks like iOS shows distance if there is real-time info but no
predicted departure time.  All arrivals-and-departures-for-stop API
responses include distance to stop info in the status element, even when
real-time info isn't available (in that case it's interpolated).  I believe
it's typically a fairly rare occurrence where real-time is available but no
departure time exists, which is why this doesn't surface more often.  There
might have been a server side code change or source data change that caused
this to surface too.  For example, I think you would see this if the source
GTFS-rt TripUpdates feed stopped but the VehiclePositions feed continued.
On Nov 13, 2015 9:01 PM, "Tom Lai"<EMAIL_ADDRESS>wrote:

I found the responsible code here:
https://github.com/OneBusAway/onebusaway-iphone/blob/f41cac0d55d1a48f0e006c72326d89b4702645a8/controls/table_cells/OBAArrivalEntryTableViewCellFactory.m#L50
—
Reply to this email directly or view it on GitHub
https://github.com/OneBusAway/onebusaway-iphone/issues/458#issuecomment-156607968
.

I saw this yesterday too, but I haven't been able to repro it yet today. If we can come up with data that reproduces it, I'll happily get a fix together asap, but otherwise, let's keep this as a placeholder for a bug fix for a future release.

Looks like others are seeing this as well: https://m.facebook.com/groups/367248491525?view=permalink&id=10153223813876526


Update on top of the current import attribute proposal and the recent ECMA-262 changes

This PR also updates ecmarkup to the latest version.

The only behavioral changes introduced by this PR are caused by "inheriting" the new semantics of import attributes; everything else is purely editorial rebase.


LIBCLOUD-606 Can not connect to AWS with IAM role temporary credentials

Connect to AWS using temporary IAM role credentials
Description
Can not connect to AWS with IAM role temporary credentials. Doesn't seem like the supplied token is in the header
From: LIBCLOUD-606
Status
done, ready for review
Checklist (tick everything that applies)

[ ] Code linting (required, can be done after the PR checks)
[ ] Documentation
[ ] Tests
[ ] ICLA (required for bigger changes)

hi @alejandrogr thanks for the contribution, if you look at the test runs, every class that derived from this connection class (and didn't create a property self.token) is failing.
could you please add a supporting test for this scenario as well, the docs say that this is supported, so is this a bugfix or a new behaviour? http://libcloud.readthedocs.io/en/latest/compute/drivers/ec2.html#using-temporary-security-credentials

Good catch - thanks!
And yeah, what Anthony said wrt tests :)

@alejandrogr can you run flake8 --ignore=E402 --exclude=test libcloud/ locally and just check the linting errors please? Travis is giving a failure.

Hi,
I've just checked the linting problems and solved them. I'm pretty new to python and libcloud so please let me know if something can be improved.
Should other derived drivers from BaseEC2NodeDriver supports the temporary token too?
Cheers

Merged into trunk, thanks.
For now, I think it's the best to only support token in the EC2 driver since I believe it's EC2-specific functionality.


Delagger Broken on Windows?

I'd love if someone who is getting lagger issues on windows could run with --trace and submit their log.


Code lens test button does not pass the correct filter string to cargo test

The name of the trait or struct is omitted from the filter string.
pub mod quax {
    pub struct Foo;

    pub trait Bar {
        fn bar(&self);

        /// Bar
        /// ```
        /// # use my_crate::quax::{Foo, Bar};
        ///
        /// let foo = Foo;
        /// foo.baz();
        /// ```
        fn baz(&self) {
            self.bar();
        }
    }

    impl Bar for Foo {

        fn bar(&self) {
            println!("bar");
        }
    }
}

What happens when pressing the "Play" button for on the doctest:
Filter: quax::baz
Executing task: cargo test --doc --package my_crate -- quax::baz --nocapture
...
running 0 tests.
...

What I would expect to happen when pressing the "Play" button:
Filter: quax::Bar::baz
Executing task: cargo test --doc --package my_crate -- quax::Bar::baz --nocapture
...
running 1 test
test src\lib.rs - quax::Bar::baz (line 1246) ... ok
...

Should be ok in the next release. Fixed by #4491

Confirmed that it's fixed in today's nightly. 👍


cannot change log level

log level should be configurable via config or env

Also, instance creation errors should probably be logged as errors instead of debug? Unless the error is that an instance is already running, in which case nothing should be logged. I guess this requires differentiating the errors with more error types.


Documentation not update after 2.6 release

Environment

Python version:  3.7
NetBox version:  2.6

Steps to Reproduce

go to https://netbox.readthedocs.io/en/stable/

Expected Behavior
Should show documentation for 2.6 for latest

Observed Behavior
its documentation doesn't show any updated for 2.6 include required setting, Prometheus and new changes for power.
I can see those in git develop branch /docs directory.

Normally when this has happened in the past, the docs fail to build on the RTD side.
I don't have access to check it, so @jeremystretch will have to have a look.

I've manually rebuilt the docs, so they should be good for now. Looking at the integrations too. Looks like we were using GitHub services, which have been deprecated, to trigger the rebuild. Working on setting up a webhook for RTD now but I'm not sure whether it's working.

Docs are being rebuilt now on changes to both master and develop.


prov: prefix provider internal functions with ossl_

Also convert the names to lower case at the same time.

Merged to master, thanks for the review.


fix: disabled retention by default and added basic example

Description
https://github.com/terraform-ibm-modules/terraform-ibm-cos/issues/366: set default retention policy to false, leave the retention enable only in the fscloud profile
https://github.com/terraform-ibm-modules/terraform-ibm-cos/issues/363 :  Create a simple basic example for the module without replication, encryption, etc.
Types of changes in this PR
No release required

[ ] Examples or tests (addition or updates of examples or tests)
[ ] Documentation update
[ ] CI-related update (pipeline, etc.)
[ ] Other changes that don't affect Terraform code

Release required

[x] Bug fix (patch release (x.x.X): Change that fixes an issue and is compatible with earlier versions)
[ ] New feature (minor release (x.X.x): Change that adds functionality and is compatible with earlier versions)
[ ] Breaking change (major release (X.x.x): Change that is likely incompatible with previous versions)

Release notes content
disabled retention policy by default
added basic example

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

Merge actions for mergers

Merge by using "Squash and merge".

Use a relevant conventional commit message that is based on the PR contents and any release notes provided by the PR author.
The commit message determines whether a new version of the module is needed, and if so, which semver increment to use (major, minor, or patch).

I am currently occupied with auditree work and I won't be able to allocate time for this issue/pr hence un-assigning myself from the issue and closing this PR


Meta: Add API Logger

WIP
I added as a new option since having it on by default is going to be annoying. Including in the other logging option does not make any sense.

@fregante this logger is scaring me. I did not realize how many API requests we make!

Instead of a public option I was suggesting it to be enabled only for local installs (so only for contributors) and with console.debug, which can be hidden in the console

console.debug calls can be hidden in the console so since it's enabled only for local installs it shouldn't be much a problem.>

But I don't like that idea, for the same reason. That means that I have more things in my console that I do not need. (I need to turn my console setting to verbose)

I did not realize how many API requests we make!

Wait till we do the same for every fetch!

I think I added it all over, do you see anything missing?

I think I added it all over

"Wait until …!" is not an invitation 😅

@fregante any comments before we merge?

I disabled the automatic release because there isn't much to release. We'll do it manually when worth it


Vulnerability in hoek lib

Github has sent an automatic vulnerability alert for the repo:

A dependency defined in ClientApp/package-lock.json has known security vulnerabilities and should be updated.

Further details: https://nvd.nist.gov/vuln/detail/CVE-2018-3728
Suggested fix: hoek ~> 5.0.3
Need to find out how the lib can be updated and apply a fix if possible.

The output of npm ls hoek:
`--<EMAIL_ADDRESS>  `--<EMAIL_ADDRESS>    `--<EMAIL_ADDRESS>      `--<EMAIL_ADDRESS>        +--<EMAIL_ADDRESS>        | `--<EMAIL_ADDRESS>deduped
        +--<EMAIL_ADDRESS>        `--<EMAIL_ADDRESS>          `--<EMAIL_ADDRESS>deduped

There is an issue for that on angular cli backlog: https://github.com/angular/angular-cli/issues/10480
Need to update angular cli once it is resolved.

Removed blocked tag in the reason that referred issue in Angular CLI was fixed

It's blocked again. Angular CLI has been updated, but now there is s dependency from @angular-devkit:
`--<EMAIL_ADDRESS>  +--<EMAIL_ADDRESS>  | `--<EMAIL_ADDRESS>  |   `--<EMAIL_ADDRESS>  |     +--<EMAIL_ADDRESS>  |     | `--<EMAIL_ADDRESS>deduped
  |     +--<EMAIL_ADDRESS>  |     | `--<EMAIL_ADDRESS>  |     |   `--<EMAIL_ADDRESS>deduped
  |     +--<EMAIL_ADDRESS>  |     `--<EMAIL_ADDRESS>  |       `--<EMAIL_ADDRESS>deduped
  `--<EMAIL_ADDRESS>    `--<EMAIL_ADDRESS>      `--<EMAIL_ADDRESS>        +--<EMAIL_ADDRESS>        | `--<EMAIL_ADDRESS>deduped
        +--<EMAIL_ADDRESS>        `--<EMAIL_ADDRESS>          `--<EMAIL_ADDRESS>deduped

Github issue: https://github.com/angular/angular-cli/issues/10827


policy fails with jax 0.4.36 from IndexError: list index out of range in backward fn

Hello all!
There's a new error in mctx since JAX version 0.4.36. When running the policy improvement demo, it fails with error IndexError: list index out of range stemming from the backward fn. This is with python 3.11, and it runs fine with JAX version 0.4.35. I didn't see any indexing related changes in jax's change log, but maybe I'm missing something. Here is the error:
/opt/anaconda3/envs/jax/bin/python /Users/evanwalters/mctx/examples/policy_improvement_demo.py
INFO:2024-12-06 01:00:48,626:jax._src.xla_bridge:927: Unable to initialize backend 'rocm': module 'jaxlib.xla_extension' has no attribute 'GpuAllocatorConfig'
I1206 01:00:48.626108<PHONE_NUMBER> xla_bridge.py:927] Unable to initialize backend 'rocm': module 'jaxlib.xla_extension' has no attribute 'GpuAllocatorConfig'
INFO:2024-12-06 01:00:48,627:jax._src.xla_bridge:927: Unable to initialize backend 'tpu': INTERNAL: Failed to open libtpu.so: dlopen(libtpu.so, 0x0001): tried: 'libtpu.so' (no such file), '/System/Volumes/Preboot/Cryptexes/OSlibtpu.so' (no such file), '/opt/anaconda3/envs/jax/bin/../lib/libtpu.so' (no such file), '/usr/lib/libtpu.so' (no such file, not in dyld cache), 'libtpu.so' (no such file), '/usr/local/lib/libtpu.so' (no such file), '/usr/lib/libtpu.so' (no such file, not in dyld cache)
I1206 01:00:48.627507<PHONE_NUMBER> xla_bridge.py:927] Unable to initialize backend 'tpu': INTERNAL: Failed to open libtpu.so: dlopen(libtpu.so, 0x0001): tried: 'libtpu.so' (no such file), '/System/Volumes/Preboot/Cryptexes/OSlibtpu.so' (no such file), '/opt/anaconda3/envs/jax/bin/../lib/libtpu.so' (no such file), '/usr/lib/libtpu.so' (no such file, not in dyld cache), 'libtpu.so' (no such file), '/usr/local/lib/libtpu.so' (no such file), '/usr/lib/libtpu.so' (no such file, not in dyld cache)
Traceback (most recent call last):
  File "/Users/evanwalters/mctx/examples/policy_improvement_demo.py", line 149, in <module>
    app.run(main)
  File "/opt/anaconda3/envs/jax/lib/python3.11/site-packages/absl/app.py", line 308, in run
    _run_main(main, args)
  File "/opt/anaconda3/envs/jax/lib/python3.11/site-packages/absl/app.py", line 254, in _run_main
    sys.exit(main(argv))
             ^^^^^^^^^^
  File "/Users/evanwalters/mctx/examples/policy_improvement_demo.py", line 135, in main
    rng_key, output = jitted_run_demo(rng_key)
                      ^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/evanwalters/mctx/examples/policy_improvement_demo.py", line 73, in _run_demo
    policy_output = mctx.gumbel_muzero_policy(
                    ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/evanwalters/mctx/mctx/_src/policies.py", line 189, in gumbel_muzero_policy
    search_tree = search.search(
                  ^^^^^^^^^^^^^^
  File "/Users/evanwalters/mctx/mctx/_src/search.py", line 111, in search
    _, tree = loop_fn(
              ^^^^^^^^
  File "/Users/evanwalters/mctx/mctx/_src/search.py", line 103, in body_fun
    tree = backward(tree, next_node_index)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/evanwalters/mctx/mctx/_src/search.py", line 290, in backward
    tree, _, _ = jax.lax.while_loop(cond_fun, body_fun, loop_state)
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
IndexError: list index out of range
--------------------
For simplicity, JAX has removed its internal frames from the traceback of the following exception. Set JAX_TRACEBACK_FILTERING=off to include these.

I'll try to take a closer look tomorrow, but maybe someone with more experience with mctx could help me track this error down.

I was able to reproduce the issue with jax 0.4.36.
When using JAX_TRACEBACK_FILTERING=off, I can see the cause of the error.
The error is from this line:
https://github.com/jax-ml/jax/blob/2ff90382d2b046dd4474a778aced59461c30c6f7/jax/_src/lax/control_flow/loops.py#L386
Have you found more details?

The bug was fixed in jax 0.4.37:
https://github.com/jax-ml/jax/releases/tag/jax-v0.4.37

@fidlej thank you!!


there is no picture 725

https://picsum.photos/id/725/640/480
Any idea why? Are there any other missing images between 1...1000?

duplicates https://github.com/DMarby/picsum-photos/issues/83

Does the list of valid id ever change? Would it be acceptable to download the list of valid ids from the list endpoint and cache that?


Investigate failure-on-synch modalities

There seems to be confusion as to what happens when there is a synch failure. Per Bin Ma, who investigated the current behavior of synching on the application, the following takes place:
Start Bin Ma comments:
About our app's syncing strategy:
automatic sync is triggered hourly, manual sync will trigger after user clicks the sync button.
About our app's syncing work flow:
1)Start the sync: retrieve all the data set that marked as unsynched from the database and push to the server.
2)Sync succeeds: mark all unsynched data set as synced.
3)Sync fails: all unsynched data set stays unsynched.
Conclusions:
There are several conclusions based on the sync process. If we failed to sync data to the server, the unsynched data set will be send to the server again after one hour. So if there is no error about the server’s calculation, then the SOH statistics during a relatively long interval (monthly statistics e.g) will not appear any significant deviation, while in a relatively short interval(less than a day), it may appear some deviation.
What we can do from the client side:
We can add some extra retry strategy to every single sync(retry for 3 times, interval time are 10s, 30, 90s) to reduce the deviation in short interval, but as is analysed above, failure on a single update should not affect the statistics in long interval that much. So we may investigate more about previous SOH statistics problem.
End Bin Ma comments.
KB:
We should investigate whether there are other failure modes, such as

Client synchs to server, receives confirmation of synch, but server doesn't store values somehow
Client synchs to server, receives failure (or no response), but client does not retry

All of this comes down to ensuring that a distributed transaction (e.g. between server and client) is robust.
To investigate this further, we could perhaps set up a mock instance of DHIS2 that responds with different failure modes (e.g. no response, failure response, success response, success response (code 200) with failure or other error in the JSON response, etc, if such doesn't exist already, so that we can confirm behavior of client application. Additional testing would have to be done on the server side, as well as investigations of the logs, to see if there are cases where a success message is sent to the client but the server still fails to save the data.

@all please hold off on this for now until we settle #253. Whatever we decide there will impact this card.

#253 has been closed. And I don't think we may spend time on the issue, cause we have changed the whole process of SOH Syncing should concentrate on current implementation.


merge package template from arlac77/npm-package-template

package.json

chore(devDependencies): update<EMAIL_ADDRESS>from template

:tada: This PR is included in version 1.0.7 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Can't get Featured image

I have a custom post type called "dicono-di-noi" and if I try to get the entries with:
$dicono_di_noi = DiconoDiNoi::published()->with('attachment')->orderBy(DB::raw('RAND()'))->get();

I'm not able to get the image.
This is my view:
 @foreach($dicono_di_noi as $dicono)
        <div class="item">
            {!! $dicono->image !!}
        </div>
  @endforeach

I used $dicono->image according to this issue: https://github.com/jgrossi/corcel/issues/94 but this is the result:
SQLSTATE[42S02]: Base table or view not found: 1146 Table 'testsite.postmeta' doesn't exist (SQL: select * from `postmeta` where `post_id` is null and `post_id` is not null)

Hi @ChristianGiupponi are you working with a custom connection name called testsite, right? Is this set in your custom post type class?

Hi @jgrossi the connection is named wordpress:
'wordpress' => [
            'driver'    => 'mysql',
            'host'      => env('DB_HOST', 'localhost'),
            'port'      => env('DB_PORT', '3306'),
            'database'  => env('DB_DATABASE', 'forge'),
            'username'  => env('DB_USERNAME', 'forge'),
            'password'  => env('DB_PASSWORD', ''),
            'charset'   => 'utf8',
            'collation' => 'utf8_unicode_ci',
            'prefix'    => 'wp_',
            'strict'    => false,
            'engine'    => null,
        ],

and this is my model:
<?php

namespace App\Models\Wordpress;

use Caffeinated\Presenter\Traits\PresentableTrait;
use Corcel\Attachment;
use Corcel\Post as Corcel;

class Testimonial extends Corcel
{
    use PresentableTrait;

    /**
     * @var string
     */
    protected $presenter = 'App\Presenters\Testimonials\Testimonial';

    /**
     * @var string
     */
    protected $postType = 'testimonial';

    /**
     * @var string
     */
    protected $connection = 'wordpress';

}

right now to get the image this is the only thing that works for me:
foreach($testimonial->attachment as $attachment){
            $image = $attachment->guid;
            $name = $attachment->post_name;
        }

@ChristianGiupponi here is the code to get the image. Please check if the _thumbnail_id meta exists for this post:
/**
     * Gets the featured image if any
     * Looks in meta the _thumbnail_id field.
     *
     * @return string
     */
    public function getImageAttribute()
    {
        if (!is_null($this->meta->_thumbnail_id)) {
            $image = Attachment::find($this->meta->_thumbnail_id);

            if (!is_null($image)) {
                return $image->guid;
            }
        }
    }

Thanks.

Should I add this to my Model right?

No @ChristianGiupponi this method looks for a image attribute, not imageAttribute. We missed the tests for this feature I think.
The error you're getting is about the database testsite and the table postmeta. The weird here is you've set the connection to wordpress and it has the wp_ preffix, and the class is looking for postmeta, not wp_postmeta.

@jgrossi yeah, sorry, I forgot to remove the attribute.
I have tried again but nothing changed, for some reason it ignore the wp_ and set all to null
SQLSTATE[42S02]: Base table or view not found: 1146 Table 'testsite.postmeta' doesn't exist (SQL: select * from `postmeta` where `post_id` is null and `post_id` is not null)

Is the database name of your default connection the same testsite?
I think I know what's happening. When using ->attachment we are using the hasMany() method from Corcel\Model so it forwards the connection name to all the relationships, but when using ->image it looks this is not working. This method looks for $this->meta->_thumbnail_id and get the corresponding attachment.
Can you check if $this->meta->_thumbnail_id is an integer?

The default connection is mysql, the laravel default one and yes, the $this->meta->_thumbnail_id is an integer.
I've investigate in Attachment and I've noticed that Corcel\Post to not have the $connection properties.
I have added
protected $connection = 'wordpress';

and now the code print the image url, could be this the problem?

No, Corcel should not have this attribute, because this must be set by your Testimonial class, and it extends Corcel\Post. So this is really a problem of connection config of Corcel.
You got the image url using ->image or ->attachment?

If I remove the connection from Corcel\Post it stop working.
I get it with ->image.
BTW: I'm using dev-master, could this be the problem?

I don't think so. The master branch is updated with the last changes we've made.
Please, can you debug Corcel\Post::getImageAttribute()? You've said the $this->meta->_thumbnail_id is an integer, so the problem should be in Attachment::find() call. I think when calling this is where the error is being triggered.
Can you check that?

Yes, the error is here, I have tried to add the number directly and it still fail.
What should I check?

I think Attachment does not know the connection, but that's weird because you said you set the $connection to Corcel\Post and it works.

Yes, but in my code I do not extends Attachment anywhere...should I extent it? Maybe is for this reason that do not see the $connection.

You should not extend Attachment, you only should have to extend your custom post type to extends Corcel\Post and just it.
So when you set $connection in Corcel\Post were you extending Attachment? Is this correct?

When I set $connection in Corcel\Post I'm extending Model:
class Post extends Model.
and
Attachmend extends Post

Try to make Attachment extending Model instead of Post and check if it'll work please.

If I extend Model I get:
SQLSTATE[42S02]: Base table or view not found: 1146 Table 'testsite.attachments' doesn't exist (SQL: select * from `attachments` where `attachments`.`id` = 53 limit 1)

If you want to test I sued this plugin for testimonials: https://wordpress.org/plugins/easy-testimonials/

Yep, that's definitely a connection problem. Sorry, but we have to check that first, maybe using getImageAttribute() should not be the best solution. ASAP I'm going to check that, but if you want to, any help is welcome.

Looks like the connection isn't passed to the Attachment model.
Maybe something like this could fix that issue ?
$attachment = new Attachment();
$attachment->setConnection($this->getConnection()->getName());
$image = $attachment->find($this->meta->_thumbnail_id);

@ChristianGiupponi can you  make the test @rundef said? I cannot do that at the moment.
Replace $image = Attachment::find($this->meta->_thumbnail_id); line 403 in Corcel\Post to:
$attachment = new Attachment();
$attachment->setConnection($this->getConnection()->getName());
$image = $attachment->find($this->meta->_thumbnail_id);

Thanks!

And thanks @rundef

@rundef this fix the issue!

Awesome @ChristianGiupponi and thanks a lot @rundef. Can anyone send the PR? I can do that only in 1 or 2 hours, so if you can, just send. Thanks again

Sure, just 2 minutes and I send it

Thanks @ChristianGiupponi

Fixed in v1.0.5 ;-)

Now the problem is the huge amount of query for each image.
I have tried to implement this: https://github.com/jgrossi/corcel/issues/107 but seems that it do not use the right connection:
SQLSTATE[42S02]: Base table or view not found: 1146 Table 'testsite.postmeta' doesn't exist (SQL: select * from `postmeta` where `meta_key` = _thumbnail_id and `postmeta`.`post_id` in (56, 52))


[stable/prometheus-operator] defaultRules.rules.kubernetesSystem in k8s 1.14

Describe the bug
Setting defaultRules.rules.kubernetesSystem = false does not disable the same alerts in k8s version >= 1.14 as it does in k8s version < 1.14.  I think this is caused by the differences in these 2 files:

< 1.14: https://github.com/helm/charts/blob/master/stable/prometheus-operator/templates/prometheus/rules/kubernetes-system.yaml

{{- if and (semverCompare ">=1.10.0-0" $kubeTargetVersion) (semverCompare "<1.14.0-0" $kubeTargetVersion) .Values.defaultRules.create .Values.defaultRules.rules.kubernetesSystem }}

>= 1.14: https://github.com/helm/charts/blob/master/stable/prometheus-operator/templates/prometheus/rules-1.14/kubernetes-system-apiserver.yaml

{{- if and (semverCompare ">=1.14.0-0" $kubeTargetVersion) (semverCompare "<9.9.9-9" $kubeTargetVersion) .Values.defaultRules.create }}

Version of Helm and Kubernetes:
Helm: 2.15.1
Kubernetes: 1.14.7-gke.23
Which chart:
stable/prometheus-operator
What happened:
Alerts in templates/prometheus/rules-1.14/kubernetes-system-apiserver.yaml are not disabled when defaultRules.rules.kubernetesSystem: false
What you expected to happen:
Alerts in templates/prometheus/rules-1.14/kubernetes-system-apiserver.yaml are disabled when defaultRules.rules.kubernetesSystem: false
How to reproduce it (as minimally and precisely as possible):
Set defaultRules.rules.kubernetesSystem: false and view the generated CRDs defined by prometheus/rules-1.14/kubernetes-system-apiserver.yaml to verify that they are created.
Anything else we need to know:

Thanks for raising this, are you interested in contributing a fix to the scripts that sync the alerts?

I'm interested.  I could submit a pull request to resolve this issue.  I think the resolution should be straightforward if using defaultRules.rules.kubernetesSystem to control alerts in templates/prometheus/rules-1.14/kubernetes-system-apiserver.yaml is an acceptable solution.

Yes, this sounds right to me!

I opened #19285 to address this ticket.  I updated the chart version to 8.4.0 because the behavior of .Values.defaultRules.rules.kubernetesSystem changes.  Please let me know if another chart version would be more appropriate.


Initial Support for Azure Blob Storage Object

Description
What is the current behavior?
Currently, the Astro Python SDK doesn't offer an interface to communicate with the Azure Blob Storage system through the WASB protocol.
related: https://github.com/astronomer/astro-sdk/issues/905
What is the new behavior?
Add support for the WASB protocol

Interact with the file system

As our next goal we're going to implement the non-native load file for azure.
Does this introduce a breaking change?
No
Checklist

[x] Establish connectivity with WASB Hook
[x] Build list function with prefix intelligence
[x] Collect size metadata information
[x] Apply tests for coverage

Test case for Size and Openlineage is missing .

Follow up tickets:

https://github.com/astronomer/astro-sdk/issues/1446
https://github.com/astronomer/astro-sdk/issues/1447
https://github.com/astronomer/astro-sdk/issues/1448

The newly introduced Github secret AZURE_WASB_CONN_STRING is misbehaving in this PR (created from a fork). This behaviour could also be observed when creating a new PR from the original repo using the same commits:
https://github.com/astronomer/astro-sdk/pull/1455
However, when creating a new commit with the same changes, by a repo maintainer, from the original original, things seem to be working:

https://github.com/astronomer/astro-sdk/actions/runs/3719231698/jobs/6307958744
https://github.com/astronomer/astro-sdk/pull/1454


Teach The Legacy Driver to Unpack Incremental Dependency Information From swiftmodule Files

Nothing has been turned on yet.

An incremental build involving incremental external dependencies behaves as a hybrid between an external dependency and a normal swiftdeps-laden Swift file.
In the simplest case, we will fall back to the behavior of a plain external dependency today. That is, we will check its timestamp, then schedule all jobs that involve these external dependencies if it is out of date.
Where things get interesting is when cross-module incremental builds are enabled. In such a case, we know that a previous compiler has already emitted serialized swiftdeps information inside of a swiftmodule file. Moreover, we know that that swiftmodule file was loaded by the build of the current swift module. Finally, thanks to the previous stack of commits, we now know exactly how to extract this information from the swiftmodule file. To bring this all home, we unpack incremental dependency information from external dependencies, then integrate them into the current dependency graph - as though they were any other swiftdeps file. This neatly extends the single-module incremental logic to the multi-module case.

@swift-ci test

Build failed
Swift Test OS X Platform
Git Sha - e35f0601443636cfa9ee75f1ede41e4cd09b9782

@swift-ci test

Build failed
Swift Test OS X Platform
Git Sha - e35f0601443636cfa9ee75f1ede41e4cd09b9782

@swift-ci test macOS

Build failed
Swift Test OS X Platform
Git Sha - e35f0601443636cfa9ee75f1ede41e4cd09b9782

@swift-ci smoke test macOS

⛵


Minor fixes for the examples

The java based food ordering example still seems to be broken for me. Just before reaching the delivered state, we switch back to scheduling delivery. Not sure what is happening there.

I think the problem with the Java food ordering example seemingly to run multiple times is connected to either producing or consuming from Kafka.


⚠ Bofrost.lu has degraded performance

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

HTTP code: 200
Response time: 875 ms

Resolved: Bofrost.lu performance has improved in 7a9a557 after 14 minutes.