[bug]: 1-Click installer on Windows 10 stops at python deps

Is there an existing issue for this?

[X] I have searched the existing issues

OS
Windows
GPU
cuda
VRAM
8GB
What happened?
Extracted the installer to C:\tools\invokeAI. I use C:\tools a little like /opt on linux; a generally writeable space where I have full permissions. So the permission situation shouldn't be a problem here.
The following error occurs way after conda has downloaded a new local version of python; 3.10.7 as shown.
Now, I am not very good with python - so I really don't know where to look for a proper and precise error output...
Collecting basicsr==1.4.2
  Downloading basicsr-1.4.2.tar.gz (172 kB)
     ---------------------------------------- 172.5/172.5 kB ? eta 0:00:00
  Preparing metadata (setup.py) ... error
  error: subprocess-exited-with-error

  × python setup.py egg_info did not run successfully.
  │ exit code: 1
  ╰─> [33 lines of output]
      C:\tools\InvokeAI\.venv\lib\site-packages\setuptools\installer.py:27: SetuptoolsDeprecationWarning: setuptools.installer is deprecated. Requirements should be satisfied by a PEP 517 installer.
        warnings.warn(
      WARNING: The wheel package is not available.
      Traceback (most recent call last):
        File "<string>", line 2, in <module>
        File "<pip-setuptools-caller>", line 34, in <module>
        File "C:\Users\Ingwie Phoenix\AppData\Local\Temp\pip-install-gt2k2s5c\basicsr_1963b0a70b694a4d8bf9f0c029cd3ba0\setup.py", line 147, in <module>
          setup(
        File "C:\tools\InvokeAI\.venv\lib\site-packages\setuptools\__init__.py", line 86, in setup
          _install_setup_requires(attrs)
        File "C:\tools\InvokeAI\.venv\lib\site-packages\setuptools\__init__.py", line 80, in _install_setup_requires
          dist.fetch_build_eggs(dist.setup_requires)
        File "C:\tools\InvokeAI\.venv\lib\site-packages\setuptools\dist.py", line 875, in fetch_build_eggs
          resolved_dists = pkg_resources.working_set.resolve(
        File "C:\tools\InvokeAI\.venv\lib\site-packages\pkg_resources\__init__.py", line 789, in resolve
          dist = best[req.key] = env.best_match(
        File "C:\tools\InvokeAI\.venv\lib\site-packages\pkg_resources\__init__.py", line 1075, in best_match
          return self.obtain(req, installer)
        File "C:\tools\InvokeAI\.venv\lib\site-packages\pkg_resources\__init__.py", line 1087, in obtain
          return installer(requirement)
        File "C:\tools\InvokeAI\.venv\lib\site-packages\setuptools\dist.py", line 945, in fetch_build_egg
          return fetch_build_egg(self, req)
        File "C:\tools\InvokeAI\.venv\lib\site-packages\setuptools\installer.py", line 87, in fetch_build_egg
          wheel.install_as_egg(dist_location)
        File "C:\tools\InvokeAI\.venv\lib\site-packages\setuptools\wheel.py", line 96, in install_as_egg
          self._install_as_egg(destination_eggdir, zf)
        File "C:\tools\InvokeAI\.venv\lib\site-packages\setuptools\wheel.py", line 104, in _install_as_egg
          self._convert_metadata(zf, destination_eggdir, dist_info, egg_info)
        File "C:\tools\InvokeAI\.venv\lib\site-packages\setuptools\wheel.py", line 125, in _convert_metadata
          _unpack_zipfile_obj(zf, destination_eggdir)
        File "C:\tools\InvokeAI\.venv\lib\site-packages\setuptools\archive_util.py", line 129, in _unpack_zipfile_obj
          with open(target, 'wb') as f:
      FileNotFoundError: [Errno 2] No such file or directory: 'C:\\Users\\Ingwie Phoenix\\AppData\\Local\\Temp\\pip-install-gt2k2s5c\\basicsr_1963b0a70b694a4d8bf9f0c029cd3ba0\\.eggs\\torch-1.13.0-py3.10-win-amd64.egg\\torch\\include\\ATen\\ops\\_test_autograd_multiple_dispatch_view_copy_compositeexplicitautogradnonfunctional_dispatch.h'
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed

× Encountered error while generating package metadata.
╰─> See above for output.

note: This is an issue with the package mentioned above, not pip.
hint: See above for details.

Screenshots
Hope my logs count :)
Additional context
I ran the installer twice. After it errored the first time, I re-ran install.bat again to see if something would change - which did not happen, sadly.
Contact Details
Discord IngwiePhoenix#6637 - Twitter IngwiePhoenix

Have you tried the manual install method? I just updated and had to require pip via the environment.yml to install basicsr==1.4.2 as your install.bat but also had to require realesrgan==0.3.0.

Ingwie, your logs do count! Have a look at how loooooong that directory is on that FileNotFoundError line. It's over 260 characters: not a problem for Linux but a big no-no for Windows. My identical error returned a path 6 characters shorter than yours and it was still breaking the rules!
Not your fault — but something for devs to consider documenting for Windows people or working around it in their batch file because this is a default environment variable location.
Head to Control Panel > System > Advanced > Environment Variables and change your Windows TEMP path(s) to a folder closer to the root, try again and report back.

There is a .bat file in installer called WinLongPathsEnabled.reg. If you double-click it, it will enable long file names in your Windows Registry. If the problem is indeed the long name, then this will fix it.

All thats left is to learn what does what exactly, in terms of the settings on the left, and how I can possibly feed some of my own images in? If that is even possible since those seem to be checkpoints.

Join the Discord! Lots of help to be had in there.


Move tests from Array to Collection

There are many tests for arrays that are actually for the collection class. These tests should be moved to be clearer.

@danielgtaylor My original thoughts with collection was a simple type of element from which other things could be extended. I currently don't think it's useful, so I'm proposing we move all functions and tests to the array element and have object extend it.

Handled in #54


Make exp_with_args.sh agnostic to hard coded vars

Automatic determination of the CodeT5 directory based on the parent of the directory the script resides in

I signed the CLA, please refresh the check if you can


Add script for running app consoles.

Supports Rails apps and apps with a console
binary in the root (such as rummager).

😍 merge when green 👍

❤

💋

I'm not sure this will actually end up on the box, it might need to be included in the govuk_scripts class.

@alexmuller spoil sport: https://github.com/alphagov/govuk-puppet/pull/4352


Covalent docking of Inhibitors to crystal

hello, I am curious whether this approach of covalent docking can  be extended to non-protein structures like heme and other small molecules.

It can, yes. Do you have more details about your system?

So I am trying to dock one of the molecules of this dimer(https://www.ccdc.cam.ac.uk/structures/Search?Ccdcid=AYILIE&DatabaseToSearch=Published) onto the other guided via covalent docking. In short, I am splitting the dimer into monomers and see how they interact. When I did the conventional docking, the rings' pi-pi stack on each other which may not be the actual crystal structure so I am trying to see if the covalent or reactive docking works in this regard.

I think conventional docking is more appropriate for this system. You may want to try the AD4 scoring function with +2 charge on the irons.


Support for continuously valued validation metric "losses"

Is it possible for validation metrics to return non-boolean answers?
This could bring a lot more control over the optimization strategy, e.g. loss weighting for different metrics.
Another idea is natural language "losses", kind of like RLAIF. AKA self-critique.

This is absolutely on the short-term map. We just need to identify exactly how to use these scores for building demonstrations though.
right now, you can implement a threshold function (if score > threshold, return true).
Did you have a particular pattern in mind

Oh btw. Continuous metrics are already fully supported for optimization but not for candidate selection.
This may already be all you need.
If trace is None, you can return a continuous score and that will be used for random search or Optuna optimization.

I did not have a particular pattern in mind, I've only started to dig deeper into this library a couple of hours ago. Very happy that I can play with continuous losses however! Going to give that a try.
I'm going to have a retrieval-augmented agent evaluate results according to a criteria and give continuous feedback.

Oh btw. Continuous metrics are already fully supported for optimization but not for candidate selection.
This may already be all you need.
If trace is None, you can return a continuous score and that will be used for random search or Optuna optimization.

Hello,
Would you please elaborate more on this? What is trace here?
thanks!

Doesn't seem like these scores are currently used for Optuna optimization: https://github.com/stanfordnlp/dspy/blob/a102f3aba3aa02ed7db5d468ef1c3d463c97710b/dspy/teleprompt/teleprompt_optuna.py#L43
I would also love to see entire devset metrics be defineable for optimization, so that for example I could optimize for precision!

Answered in depth now at: #298
One note: trace is the variable of the full trace of the DSPy program, which is given to your metric during bootstrapping new examples so you can "look back" at all inputs and outputs of all steps of the program.


Coonection Refused

Hi, here are logs, I am simple writing data to realtime, Streaming data works perfectly,, I faced this issue suddenly, can you please help.
Thanks
Connected with IP: <IP_ADDRESS>
[V][ssl_client.cpp:56] start_ssl_client(): Free internal heap before TLS 216348
[V][ssl_client.cpp:58] start_ssl_client(): Starting socket
[V][ssl_client.cpp:93] start_ssl_client(): Seeding the random number generator
[V][ssl_client.cpp:102] start_ssl_client(): Setting up the SSL/TLS structure...
[I][ssl_client.cpp:156] start_ssl_client(): WARNING: Use certificates for a more secure communication!
[V][ssl_client.cpp:180] start_ssl_client(): Setting hostname for TLS session...
[V][ssl_client.cpp:195] start_ssl_client(): Performing the SSL/TLS handshake...
[V][ssl_client.cpp:216] start_ssl_client(): Verifying peer X.509 certificate...
[V][ssl_client.cpp:225] start_ssl_client(): Certificate verified.
[V][ssl_client.cpp:240] start_ssl_client(): Free internal heap after TLS 175132
[V][ssl_client.cpp:279] send_ssl_data(): Writing HTTP request...
[V][ssl_client.cpp:56] start_ssl_client(): Free internal heap before TLS 171060
[V][ssl_client.cpp:58] start_ssl_client(): Starting socket
[V][ssl_client.cpp:93] start_ssl_client(): Seeding the random number generator
[V][ssl_client.cpp:102] start_ssl_client(): Setting up the SSL/TLS structure...
[I][ssl_client.cpp:156] start_ssl_client(): WARNING: Use certificates for a more secure communication!
[V][ssl_client.cpp:180] start_ssl_client(): Setting hostname for TLS session...
[V][ssl_client.cpp:195] start_ssl_client(): Performing the SSL/TLS handshake...
[V][ssl_client.cpp:216] start_ssl_client(): Verifying peer X.509 certificate...
[V][ssl_client.cpp:225] start_ssl_client(): Certificate verified.
[V][ssl_client.cpp:240] start_ssl_client(): Free internal heap after TLS 131592
[V][ssl_client.cpp:279] send_ssl_data(): Writing HTTP request...
[V][ssl_client.cpp:56] start_ssl_client(): Free internal heap before TLS 127516
[V][ssl_client.cpp:58] start_ssl_client(): Starting socket
[V][ssl_client.cpp:93] start_ssl_client(): Seeding the random number generator
[V][ssl_client.cpp:102] start_ssl_client(): Setting up the SSL/TLS structure...
[I][ssl_client.cpp:156] start_ssl_client(): WARNING: Use certificates for a more secure communication!
[V][ssl_client.cpp:180] start_ssl_client(): Setting hostname for TLS session...
[V][ssl_client.cpp:195] start_ssl_client(): Performing the SSL/TLS handshake...
[V][ssl_client.cpp:216] start_ssl_client(): Verifying peer X.509 certificate...
[V][ssl_client.cpp:225] start_ssl_client(): Certificate verified.
[V][ssl_client.cpp:240] start_ssl_client(): Free internal heap after TLS 88044
[V][ssl_client.cpp:279] send_ssl_data(): Writing HTTP request...
[V][ssl_client.cpp:56] start_ssl_client(): Free internal heap before TLS 83912
[V][ssl_client.cpp:58] start_ssl_client(): Starting socket
[V][ssl_client.cpp:93] start_ssl_client(): Seeding the random number generator
[V][ssl_client.cpp:102] start_ssl_client(): Setting up the SSL/TLS structure...
[I][ssl_client.cpp:156] start_ssl_client(): WARNING: Use certificates for a more secure communication!
[V][ssl_client.cpp:180] start_ssl_client(): Setting hostname for TLS session...
[E][ssl_client.cpp:33] _handle_error(): [start_ssl_client():190]: (-32512) SSL - Memory allocation failed
[E][WiFiClientSecure.cpp:132] connect(): start_ssl_client: -32512
[V][ssl_client.cpp:248] stop_ssl_socket(): Cleaning SSL connection.
[V][ssl_client.cpp:248] stop_ssl_socket(): Cleaning SSL connection.
[V][ssl_client.cpp:56] start_ssl_client(): Free internal heap before TLS 85444
[V][ssl_client.cpp:58] start_ssl_client(): Starting socket
[V][ssl_client.cpp:93] start_ssl_client(): Seeding the random number generator
[V][ssl_client.cpp:102] start_ssl_client(): Setting up the SSL/TLS structure...
[I][ssl_client.cpp:156] start_ssl_client(): WARNING: Use certificates for a more secure communication!
[V][ssl_client.cpp:180] start_ssl_client(): Setting hostname for TLS session...
[E][ssl_client.cpp:33] _handle_error(): [start_ssl_client():190]: (-32512) SSL - Memory allocation failed
[E][WiFiClientSecure.cpp:132] connect(): start_ssl_client: -32512
[V][ssl_client.cpp:248] stop_ssl_socket(): Cleaning SSL connection.
[V][ssl_client.cpp:248] stop_ssl_socket(): Cleaning SSL connection.
[V][ssl_client.cpp:56] start_ssl_client(): Free internal heap before TLS 84940
[V][ssl_client.cpp:58] start_ssl_client(): Starting socket
[V][ssl_client.cpp:93] start_ssl_client(): Seeding the random number generator
[V][ssl_client.cpp:102] start_ssl_client(): Setting up the SSL/TLS structure...
[I][ssl_client.cpp:156] start_ssl_client(): WARNING: Use certificates for a more secure communication!
[V][ssl_client.cpp:180] start_ssl_client(): Setting hostname for TLS session...
[E][ssl_client.cpp:33] _handle_error(): [start_ssl_client():190]: (-32512) SSL - Memory allocation failed
[E][WiFiClientSecure.cpp:132] connect(): start_ssl_client: -32512
[V][ssl_client.cpp:248] stop_ssl_socket(): Cleaning SSL connection.
[V][ssl_client.cpp:248] stop_ssl_socket(): Cleaning SSL connection.
[V][ssl_client.cpp:56] start_ssl_client(): Free internal heap before TLS 83740
[V][ssl_client.cpp:58] start_ssl_client(): Starting socket
[V][ssl_client.cpp:93] start_ssl_client(): Seeding the random number generator
[V][ssl_client.cpp:102] start_ssl_client(): Setting up the SSL/TLS structure...
[I][ssl_client.cpp:156] start_ssl_client(): WARNING: Use certificates for a more secure communication!
[V][ssl_client.cpp:180] start_ssl_client(): Setting hostname for TLS session...
[E][ssl_client.cpp:33] _handle_error(): [start_ssl_client():190]: (-32512) SSL - Memory allocation failed
[E][WiFiClientSecure.cpp:132] connect(): start_ssl_client: -32512
[V][ssl_client.cpp:248] stop_ssl_socket(): Cleaning SSL connection.
[V][ssl_client.cpp:248] stop_ssl_socket(): Cleaning SSL connection.
[V][ssl_client.cpp:56] start_ssl_client(): Free internal heap before TLS 83428
[V][ssl_client.cpp:58] start_ssl_client(): Starting socket
[V][ssl_client.cpp:93] start_ssl_client(): Seeding the random number generator
[V][ssl_client.cpp:102] start_ssl_client(): Setting up the SSL/TLS structure...
[I][ssl_client.cpp:156] start_ssl_client(): WARNING: Use certificates for a more secure communication!
[V][ssl_client.cpp:180] start_ssl_client(): Setting hostname for TLS session...
[E][ssl_client.cpp:33] _handle_error(): [start_ssl_client():190]: (-32512) SSL - Memory allocation failed
[E][WiFiClientSecure.cpp:132] connect(): start_ssl_client: -32512
[V][ssl_client.cpp:248] stop_ssl_socket(): Cleaning SSL connection.
[V][ssl_client.cpp:248] stop_ssl_socket(): Cleaning SSL connection.
[V][ssl_client.cpp:56] start_ssl_client(): Free internal heap before TLS 83112
[V][ssl_client.cpp:58] start_ssl_client(): Starting socket
[V][ssl_client.cpp:93] start_ssl_client(): Seeding the random number generator
[V][ssl_client.cpp:102] start_ssl_client(): Setting up the SSL/TLS structure...
[I][ssl_client.cpp:156] start_ssl_client(): WARNING: Use certificates for a more secure communication!
[V][ssl_client.cpp:180] start_ssl_client(): Setting hostname for TLS session...
[E][ssl_client.cpp:33] _handle_error(): [start_ssl_client():190]: (-32512) SSL - Memory allocation failed
[E][WiFiClientSecure.cpp:132] connect(): start_ssl_client: -32512
[V][ssl_client.cpp:248] stop_ssl_socket(): Cleaning SSL connection.
FAILED
REASON: connection refused

I need the source code, please


Implement pathfinding, write docstrings, and refactor

Writes some more docstrings
Simplify sleep/wake logic
Make organisms metabolize once per frame and once per cell moved
Refactor resolve_feeding to meet, which returns information about the relationship between two organisms
Fix bugs from trying to remove an organism from a cell and the list of organisms
Implement pathfinding

Search all visible cells
Move towards friendly and prey organisms
Move away from predator organisms
Wander if no organisms seen

Nice! We may want to eventually pathfind to run away from nearby predators during an organism's normal sleep time


Not a complete answer (paragraph) in the FAQ

Hi,
The paragraph of the answer is not complete:
https://docs.microsoft.com/en-us/azure/vpn-gateway/nat-overview#can-i-use-different-egresssnat-rules-to-translate-my-vnet-address-space-to-different-prefixes-to-different-on-premises-networks

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

ID: b4ea585e-6ab4-38b6-0b75-6b53ae438d7d
Version Independent ID: 21dbf063-9b96-9936-8daa-79a3cb831198
Content: About NAT (Network Address Translation) on VPN Gateway - Azure VPN Gateway
Content Source: articles/vpn-gateway/nat-overview.md
Service: vpn-gateway
GitHub Login: @yushwang
Microsoft Alias: yushwang

Thanks for the feedback and bringing this to our notice . At this time we are reviewing the feedback and will update the document as appropriate .

@yushwang Can you please look into this?

Hi @SaiKishor-MSFT,
Just wanted to follow up with you about this issue, it is still missing the rest of the text:

Created https://github.com/MicrosoftDocs/azure-docs-pr/pull/225695
#please-close


Support filesystem and file view correctly on mobile

Describe the feature
See title
Relevant code
No response

In addition, is there any way to apply for the following permissions: scan QR code, use fingerprint authentication, authenticate with system password, apply for VPN service, background process service, read phone number, receive SMS, store, etc


Presentation: Correctly determine select classes based on available fields

Fixed select classes lookup based on fields. Some condition were backwards and was falling in some cases.

Lookup based on properties field was working because properties stored in field had class used to access that property (select class) instead of actual class where that property is defined. Because this condition is associative, I turned around it to match other cases.
Lookup based on related content field was not working correctly when same field is created for multiple select classes that share base source class. Turned around condition to check if select class is equal or derived from related field source class.

@mergifyio backport release/4.x


(WORK)Freelancers (Not bug/feature)

This is not a bug or feature request.
I'm looking for freelancers who know this boilerplate and loves working with it.
I hope its okay I have created this issue, otherwise, let me know :)
If there is anybody who is interested in some freelancer assignments with this boilerplate, I would love to hear from you.
Please contact me privately on<EMAIL_ADDRESS>with info on your react/redux experience, experience with this repo and your expected hourly rate.
Really looking forward to hearing from you guys :)

@pziemkowski Why a down vote?

I think the truth is we need a leader of this boilerplate, i can work for him without hourly rate. >_<

Hey @AndersGerner
That's normal reaction from Polish guys :) I think they want to advertise yourself using "down vote" method :)
I am freelancer but I still learning React. I am gonna ask my friends if they are interested.
Best Regards,
Mateusz Paluszkiewicz

@AndersGerner This is not a place for your personal advertisements. Simple as that.
It's highly unprofessional unless you want to pay people for contributions to this repository.
Maybe it was your initial intention, but your post doesn't imply it.

@TheAifam5 Thanks for your reply :) Hope the learning is going well - Thanks for mentioning my request :)
@pziemkowski Sorry for the late reply. I must have made my self unclear.
My intention was that I would like to give the developers using this boilerplate a chance to work with me for my clients and get paid for it.
If we create something that could be added to this repo and shared with everyone else, I'm of course open for that. I love open source, and I do benefit a lot from all the work that has been done!
I'm very glad you took the time to reply, and I sincerely didn't mean offend anybody or to do any personal advertising. I didn't try to offer my own services in exchange for money. It was my honest intention to find people to work with on production projects with the boilerplate and pay them for their time.
I don't see any issue in that - Do you?
Thanks again :)

Closing because this was totally inappropriate to ask here.

Sorry @gretzky - just trying to give people a chance to work with something they love and get paid for it.
I actually got responses on this and have met a very cool dude who is working with me now.

Very weird...
People put in hard work for the benefit of all and some (who have contributed very little) attack them for it.
As a past contributor, I find this attitude sad. It means RBP is not suitable for professional use.
In my opinion @AndersGerner's post was not only appropriate, but adds considerable value to the RBP project.
@AndersGerner I'm glad you and others found a successful match.

@AndersGerner, didn't mean to come off offensive in any way. I've seen your type of question asked before, and it definitely sparks a debate. Personally, I find it inappropriate to post here- because this should be used to track issues-- there are several other places around the net to ask.
But, I definitely could've been more empathetic in my response and directed you to those other places. I'm glad you did find someone to work with you, and I wouldn't want something I say or do to prevent that-- apologies.


ebpf: CPU and memory improvements to HostProc

What does this PR do?
In some of our (CWS) code, HostProc can be called in the hot path , as such this PR tries to improve a bit the performance characteristics of this function. To do that:

it extracts the core algorithm in a testable function (one that does not call ProcFsRoot since it's a cached function with no way to override, clear it)
adds some benchmarks
adds some tests

Benchmarks:
Before:
BenchmarkHostProc-4   	 9296641	       128.1 ns/op	      24 B/op	       2 allocs/op
After:
BenchmarkHostProc-4   	15349244	        77.94 ns/op	      16 B/op	       1 allocs/op

Note: the panic for int overflow are there to match the strings.Join behavior
Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

This kind of seems like you are trying to optimize filepath.Join? I wonder if you would get equivalent improvements, by replacing kernel.HostProc(a, b) with filepath.Join(kernel.ProcFSRoot(), a, b) so there is only one filepath.Join call instead of two.

The main issue is that you cannot really do that because of the combineWith.... AFAIK as I know it's not possible to do
filepath.Join(ProcFSRoot(), combineWith...)

but yeah this is exactly what I would like to do if possible

The main issue is that you cannot really do that because of the combineWith...

Yeah, I was suggesting changing the call sites of kernel.HostProc to avoid that issue.

I could do this change in specifically the hot code path I'm looking at yes, that would work. Especially as a first step


GPG keys need to imported just once

Motivation
During release process, there are 2 deploy phases, once for Sonatype and one for GitHub releases upload. The before_deploy script is current being executed twice and fails the 2nd time.
Modifications
Import the GPG keys just once in the before_script phase.
Result
Releases tar.gz files will be automatically uploaded to github.

CLA is valid!


1.3.0 fail to load linters

I'm running Atom 1.0.3 on 64bit Linux.
$ uname -a
Linux ceres 3.13.0-37-generic #64-Ubuntu SMP Mon Sep 22 21:28:38 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux
$ atom --version
1.0.3

I upgraded linter to the latest version (1.3.0) and it stopped working.
I can reproduce by
apm uninstall linter
apm install<EMAIL_ADDRESS>
Linter works, loads linter-rubocop, and happily lints my files.
Then
apm uninstall linter
apm install linter

This installs 1.3.0 and nothing gets linted...
Here's the packages I have at that point:
/home/egeland/.atom/packages (5)
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>

I should mention that I also tried another linter - linter-ruby and that didn't work with 1.3.0 either..

Yep, we've removed the legacy API completely, if you still want it, do
apm install<EMAIL_ADDRESS>

So I should raise this against the failing linters, I guess..
Thanks for creating linter, by the way, makes life so much easier!

Yes, I believe that's the right place for this issue.
and Pleasure :bow:


registerComponent in MRML?

MJML has a registerComponent function. Does MRML have an analogous function? I did trying looking. Apologies if I missed it. If not, do you have a recommendation for using custom components with MRML? Thanks!

Hey!
Thanks for your interest in MRML!
Unfortunately, right now, it's not possible to implement your own component in MRML.
In the meantime, if your component can be split into known components/elements, you can use mj-include instead.
I'll start thinking about it though.

To get more into details about this. Right now, all the components are built on top of enums. Which means there is no dyn components, which avoids the use of a dynamic table, which improves the performances by a lot.
Having a way to register components would imply a degradation of the performance.
I will create a poc for that and see if I can make a feature for that so that it's an opt in... I'll keep thinking about it...

Thanks for the update and for your consideration!


Memory leak with raised error in cascaded overloads

Writing test-cases for some overloads I stumbled across the following memory-leak bug in master.
Minimal reproducer:
import numpy as np

import numba.unittest_support as unittest
from numba import njit
from numba.extending import overload
from numba.tests.support import MemoryLeakMixin, TestCase

def func1():
    # Stub for overload
    pass

def func2(bar):
    # Stub for overload
    pass

@overload(func1)
def _func1_ovl():
    def func1_impl():
        raise ValueError('error message')
        return

    return func1_impl

@overload(func2)
def func2_ovl(bar):
    def func2_impl(bar):
        func1()
        return bar

    return func2_impl

class TestArrayManipulation(MemoryLeakMixin, TestCase):
    def test_func2(self):
        @njit
        def test_func2_use(a):
            return func2(a)

        arr = np.arange(1)
        with self.assertRaises(ValueError) as raises:
            test_func2_use(arr)

if __name__ == '__main__':
    unittest.main()

Output:
F
======================================================================
FAIL: test_func2 (__main__.TestArrayManipulation)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "C:\Users\My User\Documents\My Project\libraries\numba\numba\tests\support.py", line 694, in tearDown
    self.memory_leak_teardown()
  File "C:\Users\My User\Documents\My Project\libraries\numba\numba\tests\support.py", line 668, in memory_leak_teardown
    self.assert_no_memory_leak()
  File "C:\Users\My User\Documents\My Project\libraries\numba\numba\tests\support.py", line 677, in assert_no_memory_leak
    self.assertEqual(total_alloc, total_free)
AssertionError: 1 != 0

----------------------------------------------------------------------
Ran 1 test in 0.227s

FAILED (failures=1)

I just realised that this is probably a known issue since some tests use self.disable_leak_check() with explicit comments to errors leaking references.
Please feel free to close this, if it is a known issue.

@ARF1 this is correct, exceptions leak references and any tests that test exceptions will require you to explicitly disable the leak checks.


Lazy import click to save 10ms of import time

I went on a little bit of a rabbit hole of trying to improve the import time of AiiDA.  :sweat_smile: More info here:
https://aiida.discourse.group/t/why-is-aiidalab-base-widget-import-so-slow/32/15
This pgsu package is currently always imported (e.g. by simply running import aiida). Currently this package brings 'click` by default, which add 10ms of import time, i.e. half of the import time of this package.
In the grand scheme of things, this obviously doesn't amount much by itself, but combining all the other work I was able to get import aiida down to 75ms from the original 300ms. :-)

thanks @danielhollas , this is much appreciated - both the general effort to bring down import time, and this fix in particular!

Codecov Report
Patch coverage: 100.00% and project coverage change: -2.26 :warning:

Comparison is base (45e35f2) 96.04% compared to head (9aa0028) 93.78%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #34      +/-   ##
==========================================
- Coverage   96.04%   93.78%   -2.26%
==========================================
  Files           2        2
  Lines         177      177
==========================================
- Hits          170      166       -4
- Misses          7       11       +4

Impacted Files
Coverage Δ

pgsu/__init__.py
93.93% <100.00%> (-1.82%)
:arrow_down:

... and 1 file with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

started new pypi publish https://github.com/aiidateam/pgsu/actions/runs/5515739615
let me know if it doesn't work

@ltalirz thank you for a super quick release! :clap: I've tested the install and verified the improved import timings.


Ensure correct order of pseudoelement/classes.

Addresses #36

Nice work. I'll merge this and fix those media query issues shortly.


Refactor into TextFieldDropdown

Codecov Report

Merging #437 (03f3e09) into main (69cf875) will decrease coverage by 0.01%.
The diff coverage is 85.86%.

@@            Coverage Diff             @@
##             main     #437      +/-   ##
==========================================
- Coverage   75.88%   75.86%   -0.02%
==========================================
  Files          79       80       +1
  Lines        4084     4024      -60
  Branches      692      681      -11
==========================================
- Hits         3099     3053      -46
+ Misses        585      580       -5
+ Partials      400      391       -9

Impacted Files
Coverage Δ

...omposelife/ui/action/settings/DarkThemeConfigUi.kt
49.09% <82.35%> (-11.55%)
:arrow_down:

...anyo/composelife/ui/component/TextFieldDropdown.kt
82.97% <82.97%> (ø)

...fe/ui/action/settings/AlgorithmImplementationUi.kt
50.00% <87.50%> (-12.93%)
:arrow_down:

...vanyo/composelife/ui/action/InlinePaletteScreen.kt
63.95% <100.00%> (-4.12%)
:arrow_down:

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


How to generate contour images and keypoints?

Dear author,
I notice that the contour image and keypoints are both in the dataset. However, I wanna train my own dataset. Could you please upload the code about how to generate contour images and keypoints?

Hello, the code for the generated contour can be found in 'graspScript/visualize_pose.py'. The code for generating keypoints we will provide later, you can manually select some keypoints to train your own dataset for now.

Hello, the code for the generated contour can be found in 'graspScript/visualize_pose.py'. The code for generating keypoints we will provide later, you can manually select some keypoints to train your own dataset for now.

Hello, the code for the generated contour can be found in 'graspScript/visualize_pose.py'. The code for generating keypoints we will provide later, you can manually select some keypoints to train your own dataset for now.
请问生成关键点的代码现在有了吗？


WASM - f64.mul/div interpreter precision on x86

Wasm: Manually implement div/mul for double to have the same precision in interpreter and jitted code.
Fixes #2145
Revert "Disable tests in float_misc.wast until fix has been found"
This reverts commit 1e77788b7d40cf77f9a80baffc535e498ba2c081.

This change is

@MikeHolman @LouisLaf Can you guys reviews this please ?

IMO better is to use SSE2 library like we have for JavascriptMath.

@MikeHolman I'm not sure I understand what exactly makes JavascriptSSE2MathOperators different from JavascriptMathOperators other than the #define SSE2MATH
Are you saying there is a way to tell the compiler to use SSE2 math on a file by file basis ?

Yes, the JavascriptSSE2MathOperators file is compiled with -arch:SSE2 which allows the compiler to use SSE2 instructions (note however that it is not guaranteed).  You can also use the new __vectorcall calling convention to pass arguments in SSE2 registers, which will make it way more likely for the compiler to use SSE2 instrs for the computations.  Note that __vectorcall is theoratically not legal at the call-site which is probably not compiled with -arch:SSE2....  We don't support however non-SSE2 machines anymore, so it's ok to break that path.

In reply to: 265549927 [](ancestors = 265549927)

I think I found the real source of the problem
AsmJsMath.inl resides in Chakra.Runtime.Math which sets the /arch:sse2 flag.
However, since the implementations are in the .inl file that is included in JavascriptMathOperator which resides in Chakra.Runtime.Language, it is compiled without the /arch:sse2 flag.
I'll try to move the includes around see if I can make asmjsmath to be compiled with /arch:sse2

So the only way to make sure that code is compiled with /arch:sse2 it had to be moved to the cpp file.
I moved the implementation of div & mul there and did my best to keep it clean I also took the opportunity to clean up some includes and an outdated div implementation in WasmMath

ping

:shipit:


Remove MobileAnalytics (Pinpoint preferred)

Amazon Pinpoint has superseded mobile analytics, and requires no platform specific customizations. Therefore, MobileAnalytics code will be removed and customers will be encouraged to migrate to pinpoint APIs to use the .NET Standard 2.0 implementation of the AWS SDK for .NET.

Discussed on Friday, this has been addressed


Add another Data other than Imdb

Sir we need to change the database of all programming scrape in Python. SO Pls Assign me in this project.

Can you be spesfic on this part? What changes, and how it will be efficient than the current one. with ss. Thanks

Yes Sir I will solve this code error with minor changes of program solution PLS assign me


Question about  roiaware_pool3d_cuda

I want to ask the question about the roiaware_pool3d_cuda in roiaware_pool3d_utils.points_in_boxes_gpu(https://github.com/zhuoxiao-chen/ReDB-DA-3Ddet/blob/7d24bd0f1d0ea14efbaf539c4717ffd28cd30ef8/pcdet/datasets/nuscenes/nuscenes_dataset.py#L387), when come to this step, it will break out an error:CUDA kernel failed : no kernel image is available for execution on the device.

It's the code's problem?

I want ask where is the roiaware_pool3d_cuda? The code doesn't have it.


[interop][SwiftToCxx] update status doc for class init support

Class initializers are supported.
Also mention that methods aren't virtual ATM.

@swift-ci please smoke test


use contextmanager for marathon app tests

Tests should clean up state even if they fail.

This is already done in #181


[Golang] unmarshal to struct

Is your feature request related to a problem? Please describe.
when i marshal a map in java,and send it by internet to a golang program,i only can unmarshal it to a interface{},but i want and need unmarshal it to a struct in golang so that i can get or set attribute easily,but no i unmarshal it to a struct ,i will get a err like:"reflect.Value.IsNil"
Describe the solution you'd like
support unmarshal to struct in golang
Additional context

Hi @Nyx2022, thanks for reporting this issue. Could you provide some test code here?

val := event.Query{
Nat:    1,
Ip:     "<IP_ADDRESS>",
Url:    "http://baidu.com",
Status: 0,
}
fury_ := fury.NewFury(true)
if err := fury_.RegisterTagType("example.SomeClass", event.Query{}); err != nil {
panic(err)
}
bt, err := fury_.Marshal(&val)
if err != nil {
    b.Log(err)
    return
}

   _ = fury_.Unmarshal(bt, &newval)
   }
   报错
   panic: reflect.Set: value of type *event.Query is not assignable to type event.Query

而且Readme里的文档 是用interface{}, 不是结构体。
测试无论是序列化反序列化 比Go标准库的json 的时间都不如


Update protobuf.rs

For the next time: you don't have to close a PR when it is not complete. You can edit the description in GitHub interface. The code can be changed by amending the git commit and force pushing that to your branch.


Allow for proxies at the route level path '/'

To allow for RESTful API's Testem should support proxies that start at the route level.
For example the following paths:
POST /users
PUT /users/1
GET /posts

Currently by setting a the '/' in the proxies testem.json config the test response in browsers return the server response at '/' rather than the testing interface.
{
  "framework": "qunit",
  "test_page": "tests/index.html",
  "launch_in_ci": [
    "PhantomJS"
  ],
  "launch_in_dev": [
    "PhantomJS",
    "Chrome"
  ],
  "proxies": {
    "/": {
      "target": "http://localhost:8000"
    }
  }
}

This means that for the browser interface to return that is being tested that testem uses content negotiation to look at headers like content type. For example behaviour Ember CLI applications support this behaviour.
Original ticket: https://github.com/ember-cli/ember-cli/issues/3227#issuecomment-73463386

I would definitely accept a PR for this, but won't have time to implement this myself in the near future. If anybody wants to tackle this, all express / proxy configuration is done here: https://github.com/airportyh/testem/blob/master/lib/server/index.js#L124

@johanneswuerbach I started on this in: https://github.com/jonathanKingston/testem/commit/9a296c5ddf7f6a98e557a769b79bc4d82d0559aa
Which works for the cases I wanted however tests are failing because I assume they are not sending content-type headers.
Would you prefer the content negotiation to be part of the config or globally on like this is?
Will tackle the tests tomorrow and any updates of docs.

Thanks @jonathanKingston, this looks great. I think we should add an option to enable / disable this as I think there are definitely cases, where you want to serve local json files.
What do you think about onlyContentTypes: ['xml', 'json'] as a setting inside a proxy object?

@johanneswuerbach should be ready to merge.
As for serving JSON assets locally, I was thinking long term the test code, its assets  and the HTML should be on a separate port to the application anyway; this would vastly simplify this proxy code and peoples use.


[Workspaces] Workspaces renamed with name_map incorrectly being shown as active

Describe the bug
I am using this configuration:
      "all_monitors": false,
      "name_map": {
        "11": "1",
        "12": "2",
        "13": "3",
        "14": "4",
        "15": "5",
        "16": "6",
        "17": "7",
        "18": "8",
        "19": "9"
      },
      "type": "workspaces"

I have workspaces 11-19 assigned to my second monitor. There is a bug occurring where whichever workspace is currently focused, is shown as the active workspace for both monitors.  Eg. if I am on workspace 1 on my main monitor, the bar on my other monitor will also show workspace 1 as active, even though this workspace is actually 11 being renamed to 1. It seems that when a workspace is renamed using name_map, it is being treated the same as the original workspace.
To Reproduce
Steps to reproduce the behavior:

Rename workspaces with name_map in the way shown above
Focus workspace 1 on your primary monitor
See that workspace 1 on the other monitor is also shown as active.

Expected behavior
The 1st workspace on my second monitor should not be shown as active.
System information:

Distro: nixos-unstable
Compositor: Hyprland
Ironbar version: ironbar 0.16.0-pre [rev b0132b]

Configuration

Share your bar configuration and stylesheet as applicable:
https://github.com/michaelBelsanti/nixconfig/blob/main/modules/nixos/desktops/hyprland/ironbar.nix
https://github.com/michaelBelsanti/nixconfig/blob/main/modules/home/theming/catppuccin/ironbar.nix

Screenshots
Both workspaces being active. Right monitor is primary, left is secondary. Left monitor is actually on workspace 2, but workspace 1 is being focused on the right monitor, so the bar is showing it as active for both.

Thanks for reporting & apologies for the delay. I thought this was going to be a nasty one but a nice easy fix. I'll merge in a few mins and the fix will be in -git

Thanks for reporting & apologies for the delay. I thought this was going to be a nasty one but a nice easy fix. I'll merge in a few mins and the fix will be in -git

glad it was so simple. I assumed the same thing when I made this bug haha. turns out it was easier to fix than it was to explain the bug lmao


(docs) Field(min/max_length) should also refer to lists

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
The current docstring for the parameter is misleading (since max_items was removed in v1):
min_length: Minimum length for strings.
max_length: Maximum length for strings.

Maybe we should reword this to:
min_length: Minimum length for strings and lists.
max_length: Maximum length for strings and lists.

This was probably missed during the v1 -> v2 migration
Example Code
No response
Python, Pydantic & OS Version
-

@straygar,
You're correct, thanks for catching this! Could you please open a PR with a fix? I'd be happy to review your changes.

Sure! I guess this also applies to sets and bytes? What'd be a good generic text? Do we even want to specify types here?

@straygar you can use "iterable" as a catch all term here. Thanks!

Yes! @straygar iterable is more inclusive as then it is implied it works for ANY iterable (tuples, dicts, etc.)


Markdown doesn't render correctly on Android when a line ends with two spaces followed by a newline.

About this issue
Markdown doesn't render correctly on Android when a line ends with two spaces followed by a newline.
Based on the markdown syntax, two space and a newline should be represented as a line break.
But it has been rendered three line breaks in this library.
I've went though  the code and it seems the parsing isn’t accurate.
For instance, take the string "abc  \n" where there are two spaces after "abc".
After parsing, the output incorrectly includes the elements TEXT, BR, and EOL.
And according the logic here, the BR will be translated to two '\n'. As a result, the render will not be as expected.
btw, if we have a string like "abc \n".(one spaces after abc).
After parsing, the output includes the children like TEXT/WHITESPACE/EOL.
Could some one help check on this issue?
Details

[x] Used library version
0.26.0
[x] Used platform
android
[x] Used support library version
[x] Used gradle build tools version
[x] Used tooling / Android Studio version
[x] Other used libraries, potential conflicting libraries

Checklist

[x] Searched for similar issues
[x] Checked out the sample application
[x] Read the README
[x] Checked out the CHANGELOG
[x] Read the FAQ
[x] Checked out the MIGRATION GUIDE

Thank you for the report.
Can you provide a few sample markdowns with various combinations? This would help a great deal to look into this more.

@mikepenz
Thanks for the response.
Here is one example,
"**Aug 24, 2023**  \n6:00 AM - 7:00 AM MST  \n8:00 AM - 8:30 AM MST\n\n**Aug 28, 2023**  \n8:00 AM - 8:30 AM MST\n\n**Aug 30, 2023**  \n8:00 AM - 8:30 AM MST \n11:00 AM - 12:00 PM MST  \n"
There are two spaces after **Aug 24, 2023**, followed by a line break.


Update linux-build-install.sh

?

You're only updating one of the scripts. I will update both later with some other changes.

ok


Fix cursor alignment on empty field

Ok, this should be my last amendment to this pr. As you can see, I left END and START open so that we could add these at a later time if necessary. I don't think they actually do anything right now anyway. In the future, it will be necessary to also make a minor adjustment for bullets. Are bullets even working now?


Stop using reserved JavaScript keyword package

In the process of upgrading babel in Atom core (https://github.com/atom/atom/pull/13823) we have noticed this package uses JavaScript features that are either deprecated or that are incompatible with the language specification.
This pull request changes the code to stop using such features so that the package keeps working when the babel upgrade will be merged and released.
I have enabled edits from maintainers, so feel free to make any changes to this branch! Also, please let me know if you have any questions or concerns and if I can help somehow. Thanks!

@as-cii
Right! And thank you for giving PR!
Actually, I also noticed this problem and opened the issue https://github.com/s-shin/atom-copy-path/issues/9 previously.
Though I made one comment, can you fix it?
If you can and did, I will merge and release this.
Thanks,

I have enabled edits from maintainers

I had overlooked this, so now that was fixed :)


Move to<EMAIL_ADDRESS>and use ember-cli dependencies

Thanks again @chadhietala!
I should have just included this in v0.1.0, but instead I had to update to v0.2.0 with the new dependencies.


'MissingDefinitionParts' error when attempting to import PBIB report to workspace

Using the sample script
# Authenticate
Set-FabricAuthToken -reset

# Ensure workspace exists
$workspaceId = New-FabricWorkspace  -name $workspaceName -skipErrorIfExists

# Import the semantic model and save the item id
$semanticModelImport = Import-FabricItem -workspaceId $workspaceId -path $pbipSemanticModelPath

# Import the report and ensure its binded to the previous imported report
$reportImport = Import-FabricItem -workspaceId $workspaceId -path $pbipReportPath -itemProperties @{"semanticModelId" = $semanticModelImport.Id}

Getting the following error with no indication of what its looking for or how to correct it
Existing items in the workspace: 2
Payload parts:
part: /definition.pbism
part: /diagramLayout.json
part: /model.bim
Creating a new item
Exception: Response status code does not indicate success: 400 (Bad Request).; API error
code: 'MissingDefinitionParts'

I have discovered that this is an issue with $partPath = $filePath.Replace($itemPathAbs, "").TrimStart("\").Replace("\", "/") in FabricPS-PBIP.psm1. I've made a change to detect the OS and trim ("/") instead on unix systems.

Thanks! Will look into your PR.

I have the same error but then running the deploy from a windows machine.
To be more precise.
Azure DevOps:
Image: windows-latest
Agent: Hosted Agent


Add language field to all API call

Language parameter is used in many API methods (e.g getLeagueListing) but the retrieved API documents does not have them in the optional parameteres. That means with current state of getParams, even if we pass language into steamObj, it will be discarded most of the time.
Without the language field, the data return won't be in readable state. Thus I proposed add the language parameter to all method calls (it doesn't cause unnecessary side-effect anyway).

Thanks


multiple optimal solutions

I have a problem of retrieving all feasible (my case optimal) solutions of MIPS.
As I followed the instruction given on SCIP webpage that retrieving all optimal solutions of a problem is a two way stage: that is first solve the problem through simple scip model call then take optimal value as right hand side of the constraint and constraint is the objective function being added to the original problem.
Predefined counter settings for the save count and before calling counter method call the function and set the values like this :"primePattern.setBoolParam("constraints/countsols/collect", True)" .
Now it keep on solving the problem(untill infeasibility reach) and giving all feasible solutions:
My problem is: I need to access all the solutions and from each solution the optimal solution of each variable which can be accessed through :
void SCIPgetCountedSparseSolutions (SCIP *scip, SCIP_VAR *** vars, int
*nvars, SPARSESOLUTION ***sols, int *nsols)
(located in "cons_countsols.h")
as given on guide.
But I couldn't access this function. Could you please explain how to call it and please let me know if something I have explained above went wrong.
It would be highly appreciated.
Thank you,

This functionality is currently not available through PySCIPOpt, i.e., SCIPgetCountedSparseSolutions() is not interfaced. We would still miss the transformation from the sparse solution to the original problem space. Within SCIP this is only possible by writing out all collected solutions.
@fserra do you have an idea on how this could be implemented best?

Any other suggestion regarding retrieving multiple optimal using PySCIPopt.

duplicate of #248


podman logs should print container names instead of ids

/kind feature
Description
As soon as you print the logs of more than one container you can hardly read the output. (Especially in combination with  #4961)
Steps to reproduce the issue:

Run podman logs <some-container> <another-container> ...

Describe the results you received:
aac136ff9cc9 11:39:24,169 INFO  [org.jboss.modules] (main) JBoss Modules version 1.9.0.Final
ac0e1a1880d3 11:39:45,141 INFO  [org.jboss.modules] (main) JBoss Modules version 1.9.0.Final

Describe the results you expected:
some-container    11:39:24,169 INFO  [org.jboss.modules] (main) JBoss Modules version 1.9.0.Final
another-container 11:39:45,141 INFO  [org.jboss.modules] (main) JBoss Modules version 1.9.0.Final

An option to print logs of different containers in different colours would help even more [see docker-compose].

@Akasurde Any progress?

@rhatdan I have an initial version of the fix in #5244. Could you please review? Thanks.


Add support for Python 3.11

Description
Contributes to https://github.com/rapidsai/build-planning/issues/3
This PR adds support for Python 3.11.
Notes for Reviewers
This is part of ongoing work to add Python 3.11 support across RAPIDS.
The Python 3.11 CI workflows introduced in https://github.com/rapidsai/shared-workflows/pull/176 are optional... they are not yet required to run successfully for PRs to be merged.
This PR can be merged once all jobs are running successfully (including the non-required jobs for Python 3.11). The CI logs should be verified that the jobs are building and testing with Python 3.11.
See https://github.com/rapidsai/shared-workflows/pull/176 for more details.

I merged the upstream to fix conflicts with #345.

Since Python 3.11 requires NumPy 1.23 at a minimum ( https://github.com/rapidsai/cucim/pull/704#issuecomment-1967552641 ), should we update lines like this one?
https://github.com/rapidsai/kvikio/blob/27abc155320e083c7a3b1610f516af4ef8e23775/dependencies.yaml#L241
Also noticing that NumPy 1.20 is being used here, which needs an update
https://github.com/rapidsai/kvikio/blob/27abc155320e083c7a3b1610f516af4ef8e23775/conda/recipes/kvikio/meta.yaml#L72

Just pushed a commit updated to numpy 1.23+, per https://github.com/rapidsai/build-planning/issues/3#issuecomment-1967952280.

/merge


Upgrade PUDL to Pandas 2.0

PR Overview
Addresses https://github.com/catalyst-cooperative/pudl/issues/2394 and I will use that issue as a task tracker

PR Checklist

[x] Merge the most recent version of the branch you are merging into (probably dev).
[x] Merge in pandas-2 branch
[ ] All CI checks are passing. Run tests locally to debug failures
[ ] Make sure you've included good docstrings.
[ ] For major data coverage & analysis changes, run data validation tests
[ ] Include unit tests for new functions and classes.
[ ] Defensive data quality/sanity checks in analyses & data processing functions.
[ ] Update the release notes and reference reference the PR and related issues.
[ ] Do your own explanatory review of the PR to help the reviewer understand what's going on and identify issues preemptively.

I'm pretty sure it doesn't have a green build -- for some reason it seems to have been waiting for approval to run the CI, even on the original pandas-2.0 branch. I think there are still many remaining issues. If you run tox locally you can see what goes wrong -- mostly I suspect it'll be in the integration tests (which run the full ETL).

It's getting a divide-by-zero error in pudl.transform.ferc1.reconcile_table_calculations() when it's being used in the transformation of the income_statement_ferc1 table.
To replicate this error outside of the test environment for debugging and without running the full ETL I think you should be able to just extract the FERC databases and then attempt to materialize the income_statement_ferc1 asset (and any of its upstream dependencies) in the Dagster UI. I suspect the error will come up in many tables, but that's the one it's tripping up on first in the integration tests.  You can also just try and materialize everything in the norm_ferc1 asset group and see which tables encounter this problem.
Maybe somehow a value that was previously NA is now showing up as a zero, and that's breaking a division?

Thank you for all your work on this @nelsonauner! I dragged it over the finish line with many commits from your branch in #2320

Yay!


Add test case for Interchain Accounts

Description
This PR adds an example test case to examples/ibc that simulates a basic flow with regards to Interchain Accounts.
The test case walks through the following:

Creating a new ICA path between a Host and Controller chain
Registering a new ICA on the host chain, on behalf of a user account on the controller chain
Querying for the ICA on the host chain
Sending a bank transfer message from the controller chain to the ICA on the host chain
Letting a packet timeout and ensuring the channel is closed on packet timeout
Reinitializing the channel following a channel closing

Checklist

[x] I have made sure the upstream branch for this PR is correct
[x] I have made sure this PR is ready to merge
[x] I have made sure that I have assigned reviewers related to this project

Notes
This requires bumping the ibc-go version to v5 stable
Closes #13

Woohoo ICA tests :fire: :robot:


ExtentReports 4.1.0 Not Displaying Passed %

Good Morning,
I run my selenium tests in a .NET Core project using Extent Reports 4.1.0 and my reports are generating successfully. However, the "Passed %" does not display if I have failed tests. It only displays if all the tests pass.
Any suggestions?
Thanks!

@anshooarora  can you please update us on this bug ?

same in ExtentReports v5.0.0-alpha6

And fast forward a year later, is there ANYONE looking into the above issue?  If not, can you suggest another report that will give the percentage correctly.

Using Extent Reports 5.0.0 in the year 2023. The issue seems to be still unresolved. The pass percentage is not showing correct numbers in any of the tables in the dashboard.

Thanks a lot @anshooarora :)


check integerish values

This may need rebase once #2 is merged
The failure is fixed in #4

This is weird - the result should lead to int|string. Can you rebase and investigate why it isn't the case? Thanks.

@ondrejmirtes I'll check ASAP, thx for feedback

@ondrejmirtes was a typo, integrish instead of integerish (missing E after G)
Also added an extra fix since CI is broken

Thanks!


Add support for loading plugins from packages directory

#503 add support for loading unpacked Thunderstore mods directly from packages. However it doesn't support loading plugins yet.
As plugins can be uploaded to Thunderstore, this functionality should be added to NorthstarLauncher.

Is there any example for a thunderstore plugin?

https://northstar.thunderstore.io/package/cat_or_not/kyurid/


Tab.Item onClick is full outlined.

[ x] I have searched the Issues to see if this bug has already been reported
[ x] I have tested the latest version

Steps to reproduce

Go to '...' https://flowbite-react.com/docs/components/tabs
Click on '....' React state options example
click between the tabs

Current behavior
Describe what is currently happening and why it's a problem.

The tab becomes fully outlined. Only when you click outside of the tab does the outline correct itself.
Expected behavior
Describe what you expected to happen.
That the tab is not fully outlined - I believe this is a bug as it's reproducible on many other variants. Unless I'm wrong and this is by design?

Add this to your theme config
  tabs: {
    tablist: {
      tabitem: {
        base: 'flex items-center justify-center rounded-t-lg p-4 text-sm font-medium first:ml-0 disabled:cursor-not-allowed disabled:text-gray-400 disabled:dark:text-gray-500'
      }
    }
  },


help: Does autok3s support distributing custom registries.yaml to nodes

hello @orangedeng  autok3s Does support  Customize content distribution to/etc/rancher/k3s/registries.yaml
mirrors:
  docker.io:
    endpoint:
      - "https://mycustomreg.com:5000"
configs:
  "mycustomreg:5000":
    auth:
      username: xxxxxx # 这是私有镜像仓库的用户名
      password: xxxxxx # 这是私有镜像仓库的密码
    tls:
      cert_file: # 镜像仓库中使用的cert文件的路径。
      key_file:  # 镜像仓库中使用的key文件的路径。
      ca_file:   # 镜像仓库中使用的ca文件的路径。

I saw the --registry /etc/autok3s/registries.yaml parameter. Will this be distributed to other hosts

Yes, you can config your customized registries.yaml file and pass it to AutoK3s when creating a cluster. AutoK3s will help you to config the registries.yaml file to each K3s nodes.

Closing this issue. Feel free to reopen this issue if you have following questions.


Export Get, Post, Put, Patch and Delete methods.

This is needed for custom API using Apex REST.
https://developer.salesforce.com/page/Creating_REST_APIs_using_Apex_REST

Looks good to me :+1: Thanks for the pr!


Add Codec type class (#133)

This implementation follows what is done in https://github.com/jto/validation with Format. I think is fulfills the first 4 requirements listed in #133, 5. does not quite work unfortunately: 96a57230a903bc7778aed49e3bbaf32fc7ec78d0.
If you think that's still acceptable I would be happy to continue the PR with tests & co :)

[Current coverage][cc-pull] is 81.49%

Merging [#301][cc-pull] into [master][cc-base-branch] will decrease coverage by 0.07%

[][cc-pull]

Powered by Codecov. Last updated by [daba818...577dbd0][cc-compare]
[cc-base-branch]: https://codecov.io/gh/travisbrown/circe/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/travisbrown/circe/compare/daba818c0b0a8b2cbaf3885e1af3174f009231b4...577dbd0ca7f3bf822b0d9439ff6d9c840ece22ab
[cc-pull]: https://codecov.io/gh/travisbrown/circe/pull/301?src=pr

@travisbrown Any thought on this PR?

Sorry for the long delay in responding, @OlivierBlanvillain and @julienrf. I'm still not 100% sure I like this, but there does seem to be demand for it, and would like to make a decision before 0.7.0.
I probably won't have time to play around with it until later this week, but will do my best to follow up in more detail before next week.

Wouldn't it be much simpler to have just the following?
  type Codec[A] = Encoder[A] with Decoder[A]

  def deriveCodec[A](implicit encode: Lazy[DerivedObjectEncoder[A]],
                     decode: Lazy[DerivedDecoder[A]]): Codec[A] =
    new Encoder[A] with Decoder[A] {
      val enc = deriveEncoder[A]
      val dec = deriveDecoder[A]
      def apply(a: A): Json = enc(a)
      def apply(c: HCursor): Result[A] = dec(c)
    }

// and then just derive both Encoder and Decoder altogether
implicit val foo: Codec[Foo]  = deriveCodec

Codec is an Encoder and a Decoder and it's generate in one shot. Nothing more than that.
And this technique could be applied to any other variable, like:
// for enumerations
def deriveEnumCodec[A](implicit encode: Lazy[EnumerationEncoder[A]],
                       decode: Lazy[EnumerationDecoder[A]]): Codec[A] =
    new Encoder[A] with Decoder[A] {
      val enc = deriveEnumerationEncoder[A]
      val dec = deriveEnumerationDecoder[A]
      def apply(a: A): Json = enc(a)
      def apply(c: HCursor): Result[A] = dec(c)
    }

// for configured encoders and decoders
def deriveCodec[A](implicit encode: Lazy[ConfiguredObjectEncoder[A]],
                   decode: Lazy[ConfiguredDecoder[A]]): Codec[A] =
    new Encoder[A] with Decoder[A] {
      val enc = deriveEncoder[A]
      val dec = deriveDecoder[A]
      def apply(a: A): Json = enc(a)
      def apply(c: HCursor): Result[A] = dec(c)
    }

Included in #1151.


Version bump to fix SECURITY-3430 / CVE-2024-43044

Fixes #1827

/ok-to-test

This can be included in this PR as well.
https://github.com/openshift/jenkins/pull/1758#issuecomment-2275205514

Moving this change to https://github.com/openshift/jenkins/pull/1758

/ok-to-test


Typo in examples README

Happened to find a typo in the examples README.
Lines 60 and 148 use the option fb16, which I believe should be fp16.
Thank you for this new library and launch tool!

That's right, thanks! Do you want make a PR to fix it?

Sure -- https://github.com/huggingface/accelerate/pull/28

Fixed by #28 thanks :-)


Bump Monaco Language Client to 8.8.3

Bump Monaco Language Client to 8.8.3.

The websocket of the language client is failed to establish connection. we need to fix it.

Fixed.

@aglinxinyuan can we close this now?


mysql: always use column alias when available

If a query has a column alias available we should use that name instead
of the column name.
Fixes #385

Thanks for the fix @blackwolf12333

Do you mind adding a test to tests/mysql/describe ? Just a simple query that uses an alias to ensure we don't regress on this.

Added a small test case :)


WIP: Add list helper

To make handling lists easier #28 . Should work for 99% of use cases.
Usage:
function main(sources) {
    const childrenSinks = List(s => s.myArray, ChildComponent)(sources)
    ...
}

I think this would be nice to have

@jvanbruegge I'm worried about the Cycle DOM dependency here, because then this library isn't anymore doing just one concern. Couldn't this List helper be published separately, perhaps in a helpers library like Ramda or Lodash style?

I could include it in cyclejs-utils or publish it seperately. The other option would be to remove the dom dependency by either making the wrapper not optional or by manually creating a vnode with selector and children.

@jvanbruegge With a manual vnode creation there is still mixing of concerns, where this library has some influence over DOM purposes, besides just influence over state management.

Plus it's not a very clean solution. I would make it required, is the easiest solution

It seems reasonable for me for the consuming application to provide whatever VDOM wrapping function makes the most sense in context. Sometimes that might be a div, other times a ul, ol, etc. It doesn't feel that important to me, to have it default to div.

So, cleaned it up

@milankinen's solution is much cleaner


scratch extension integration

i have a idea as to how we can allow projects using custom extensions to be added to scratch ,
add a export to scratch button , it will prompt you   if the extensions are optional or mandatory , if they are mandatory it will add a sprite that pops-up prompting to use turbowarp if no turbowarp is found  , (and stopping all scripts) if not it is assumed the user has used the is_turbowarp block to use custom blocks to gain more features in turbowarp while still allowing code to work on scratch  ,   information about where costum blocks are is located in comment (as well as about extensions used), this would allow people to share advanced scratch projects using turbowarp blocks

my understanding is that the Scratch Team would not be a big fan of this

my understanding is that the Scratch Team would not be a big fan of this

why would they not be ?

my understanding is that the Scratch Team would not be a big fan of this

why would they not be ? , it would generate more interest in scratch and it is not like these advanced features would be easily available to the 3 year olds

It's not exactly that Scratch doesn't want younger users to access the features, however the Scratch Team has certain visions for their platform they want to meet, and one of those visions is the site being self-contained. Doing this would provide a better alternative to the traditional Scratch editor, however would also require the use of a whole other website to access them. And like most things off-site, the Scratch team has the (valid) concern that it could be misused or mislead its users. Think browser extensions; although they provide objectively better features for Scratch, the Scratch Team has banned comment on them due to browser extensions being off-site.

not possible for technical reasons
if a large amount of projects start being uploaded to Scratch that are not compatible with Scratch (no matter how much we ask people to not do that, it is inevitable), the Scratch Team would probably be not happy (providing storage space for projects is not cheap!)
https://share.turbowarp.org/ will improve over time


Is the gap analysis accurate?

When referencing a page updated more recently (https://docs.microsoft.com/en-us/azure/azure-monitor/agents/azure-monitor-agent-migration) things like IIS and custom logs look to be supported in the newer agent.

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

ID: d3c8f766-6dbf-7cc0-321f-c3a14bcca27b
Version Independent ID: 0c3aca67-18e9-f31c-0334-58ffe845283e
Content: Migrate to the Azure Monitor agent (AMA) from the Log Analytics agent (MMA/OMS) for Microsoft Sentinel
Content Source: articles/sentinel/ama-migrate.md
Service: microsoft-sentinel
Sub-service: microsoft-sentinel
GitHub Login: @batamig
Microsoft Alias: bagol

#assign:yelevin

#assign:@limwainstein

#assign:cwatson-cat

#assign:batamig

@batamig I'm already working on this.

#label:"backlog item created"

#label:"backlog-item-created"

This was corrected long ago.
#please-close


slerp functions for directions (Vec2 and Vec3)

What problem does this solve or what need does it fill?
Linear interpolation is unsuited for transitioning of directions.
What solution would you like?
I would like a spherical interpolation function (slerp) for Vec2 and Vec3 directions.
Additional context
This is a common feature in existing game engines:
https://docs.godotengine.org/en/stable/classes/class_vector3.html#class-vector3-method-slerp
https://docs.unity3d.com/ScriptReference/Vector3.Slerp.html

Linear interpolation is unsuited for transitioning of directions.

Vec2 and Vec3 are unsuited for representing directions, so current implementation looks just fine to me.
PS: use angle-as-f32 and Quat instead

Vec2 and Vec3 is part of glam crate, this problem should be solved there, futhermore there is already long living issue

@bugsweeper You're right that the vector types are owned by glam, but on the other hand, we could easily implement slerp functions for the Dir2/Dir3 types that we do own, and I think those would be quite useful.


HOW CAN  I SET IT UP?DOSE IT NEED SOMETHING EXTRA?

I START A NEW ubuntu-lts on vmw and get update，then i start the commd
bash docker/scripts/install_docker.sh
and it runs like there is no problem，then i continue。
bash docker/scripts/dev_start.sh
and the err inf like  Permission _denied
so i try to add sudo ，and the download speed is so slow。
and another question is do i need to set up apollo-kernel and apollo-platform？？
AND what about DOCKER??DO i need to set up and use a docker image？？
and my english is not good，i feel sorry about Syntax error and i will appreciate you can help me！

Duplicate with #201


Terraform GitHub Action

This PR lays the groundwork for GitHub Actions to manage Terraform. I have started with a relatively low-risk Terraform workflow that focuses on validating our Terraform configuration.
Changes

Adds a new Terraform workflow with two different jobs:

Validate Terraform YAML

Performs a terraform validate on every module
This is environment-agnostic and is validating syntax and resource usage

Generate Terraform Plan for Staging

Performs a terraform plan on every module against staging
This logs into Azure and connects to the VPN to perform the action
After generating the plan, the .tfplan and a plain text version of the plan are uploaded to an Azure Storage Account

An Azure Storage account was chosen instead of using a GitHub artifact since the plan may contain sensitive information
While GitHub Action artifacts require user authentication to download, you do not have to be a contributor to the repository to download build artifacts (this was tested and confirmed)
Using an Azure Storage Account is private and a safe location to store the plan, since it’s marked as private, requires a connection over the VPN, and an authorized Active Directory user

Once the plan is uploaded, a command is outputted that will download the plan

This is to make it easy for the reviewer to download the plan on their location machine

Adds two new targets to the operations Makefile:

tf-validate

Runs terraform validate within the container

tf-plan

Run terraform plan-file within the container
See below for more information on plan-file command

Enhances the tf script to support new Terraform actions

Will not initialize the backend if the command does not require it
Adds a custom plan-file command that outputs the plan as a file and redirects output into a plain text file

Grants the Terraform GitHub service principal access to read secrets

This was the only permission it was missing for the ability to generate plans

Renames GitHub Actions for consistency
Adds a new GitHub Action for building the tfcli container
Adds a new GitHub Action for publishing the tfcli container on merge

This has been tested with this run: https://github.com/CDCgov/prime-reportstream/pull/1668/checks?check_run_id=3146748587
Package available here: https://github.com/CDCgov/prime-reportstream/pkgs/container/prime-reportstream_tfcli

Changes the docker-compose.yml to use the published container, if available
Changes the tfcli version references to be arguments instead of hard-coded in the Dockerfile

Checklist
Testing

[x] Tested locally?
[ ] Ran quickTest all?
[ ] Ran ./prime test against local Docker ReportStream container?
[ ] Downloaded a file from http://localhost:7071/api/download?
[ ] Added tests?

Specific Security-related subjects a reviewer should pay specific attention to

Does this PR introduce new endpoints?

No

Does this PR include changes in authentication and/or authorization of existing endpoints?

No

Does this change introduce new dependencies that need vetting?

No

Does this change require changes to our infrastructure?

Yes, additional permissions are granted to the GitHub Action Azure service principal

Does logging contain sensitive data?

Yes
Terraform plans may contain secret values or other sensitive information
This information has been excluded from the logs and is routed to a file, which is then stored in a private Azure Storage Account (the same one our Terraform state exists in)
A potential way information could leak is if there is a flaw in the way the output is routed to a file

I’m wondering if all Terraform output should be suppressed since log files are public
This would account for a scenario where a bug in Terraform exists or an unknown error path is trigger and sensitive information is logged

A reviewer should focus in on any ways my approach may fail

Does this PR include or remove any sensitive information itself?

Yes
Outside of the logs, the .tfplan itself is sensitive
Like the logs, this is stored in a Storage Account and never exposed via GitHub artifacts

If you answered 'yes' to any of the questions above, conduct a detailed Review that addresses at least:

What are the potential security threats and mitigations? Please list the STRIDE threats and how they are mitigated

Spoofing (faking authenticity)

Threat T, which could be achieved by A, is mitigated by M

Tampering (influence or sabotage the integrity of information, data, or system)
Repudiation (the ability to dispute the origin or originator of an action)
Information disclosure (data made available to entities who should not have it)
Denial of service (make a resource unavailable)
Elevation of Privilege (reduce restrictions that apply or gain privileges one should not have)

Have you ensured logging does not contain sensitive data?

See above

Have you received any additional approvals needed for this change?

Process

[x] Are there licensing issues with any new dependencies introduced?
[x] Includes a summary of what a code reviewer should verify?
[x] Updated the release notes?
[x] Database changes are submitted as a separate PR?
[x] DevOps team has been notified if PR requires ops support?

To Be Done

Once merged, I would like to mark the Terraform workflow as required for merging a PR

I know this is outside of what this PR touches but I'll ask for it anyway:
can you fix the example entry that does @echo " make TF_ENV=\"test\" all" since we don't have an all taget

I like this thoughtfulness:
"After generating the plan, the .tfplan and a plain text version of the plan are uploaded to an Azure Storage Account
An Azure Storage account was chosen instead of using a GitHub artifact since the plan may contain sensitive information
While GitHub Action artifacts require user authentication to download, you do not have to be a contributor to the repository to download build artifacts (this was tested and confirmed)
Using an Azure Storage Account is private and a safe location to store the plan, since it’s marked as private, requires a connection over the VPN, and an authorized Active Directory user

"
We also already use the azure backend for our tfstate anyway, so this is fine

Since the make command involves running a container and this container is unlikely to change (cf. prime-router_builder); should we also just publish this as a pre-build image into the packages?

All changes made! Great suggestions @td-usds.

@jdorothy


Can I conduct error analysis for each individual sample?

Are there any documents about error analysis?

Try to use tools/analysis_tools/analyze_results.py. And the tutorial is in https://mmdetection.readthedocs.io/en/latest/useful_tools.html#result-analysis

Try to use tools/analysis_tools/analyze_results.py. And the tutorial is in https://mmdetection.readthedocs.io/en/latest/useful_tools.html#result-analysis

Thank you. I tried to use it, but I didn't find a file containing the mAP for a single image. I can only find the folder containing the top-k good and bad image.
Are there anything I should assign?

I think the parameters here means the mAP of each sample.
https://github.com/open-mmlab/mmdetection/blob/70f6d9cfade4a2f0b198e4f64776521d181b28be/tools/analysis_tools/analyze_results.py#L134
I wonder what's the sequence of this parameter? It is a dictionary, but its keys are the index number(0,1,2,3,...) instead of image index, and I cannot find their corresponding relationships


[SPARK-31166][SQL] UNION map<null, null> and other maps should not fail

What changes were proposed in this pull request?

After https://github.com/apache/spark/pull/27542, map() returns map<null, null> instead of map<string, string>. However, this breaks queries which union map() and other maps.
The reason is, TypeCoercion rules and Cast think it's illegal to cast null type map key to other types, as it makes the key nullable, but it's actually legal. This PR fixes it.
Why are the changes needed?

To avoid breaking queries.
Does this PR introduce any user-facing change?

Yes, now some queries that work in 2.x can work in 3.0 as well.
How was this patch tested?

new test

cc @maropu @HyukjinKwon

Test build #119875 has started for PR 27926 at commit 125d094.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/24603/
Test PASSed.

Test build #119875 has finished for PR 27926 at commit 125d094.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/119875/
Test PASSed.

thanks for review, merging to master/3.0!


Save stream to image

How I can save stream to image in the browser?
I want render image of pdf in IE or EDGE browser, because they do not support blob url's
It is impossible?

Could dump the blob out in a file reader and return it as file. https://developer.mozilla.org/en-US/docs/Web/API/FileReader/readAsDataURL

@darkliahos it's doesn't work(
        target.classList.remove('loading')
        const image = document.createElement('img')
        document.body.appendChild(image)
        //window.navigator.msSaveOrOpenBlob(blob, pdfName)
        const reader = new FileReader()

        reader.addEventListener(
          'load',
          function() {
            image.src = reader.result
          },
          false,
        )
        reader.readAsDataURL(blobImage)

I am sorry I misread the question and a bit puzzled, why are you trying to convert a PDF to an image file? Surely if you need as an image maybe you are using the wrong library here.
Surely it would be worth doing the layout as an SVG and then exporting it as PNG or another format

In IE11 and Edge browsers preview of blob doesn't work(
On this browser I can only call window.navigator.msSaveOrOpenBlob :(
I think I understand what you're implying. My svg element is hidden for @media screen and it's showing on @media print. I can copy dom node with svg on IE and Edge and mount to preview window
Thx)))


Update notification

I got an update notification to update to android 13. Is this safe to do? The updater actually allows me to install the update.

No don't do it.
Also it's sad to tell that the support has been dropped for our aosp variants.
We'll comeback soon with a CodeLinaro based variant.
You can check the updates on it on our telegram group:-
https://t.me/ZephyrusOP8


Update README.md

Dropped infos to unsupported windows version
Moved reference to puppeteer sharp 3 release
Fixed some markdown linter warnings

Thank you!


Core: crash when launching Simulation with a feeder from IntelliJ 2019.2.1

java.lang.UnsupportedClassVersionError: org/simpleflatmapper/util/ParallelReader has been
compiled by a more recent version of the Java Runtime (class file version 53.0),
this version of the Java Runtime only recognizes class file versions up to 52.0

We pull org.simpleflatmapper:lightning-csv:7.0.3 with no classifier as we want to be compatible with JDK8.
Sadly, IntelliJ messes up with transitive dependency management and pulls org.simpleflatmapper:sfm-util:7.0.3:jdk9 instead of org.simpleflatmapper:sfm-util:7.0.3.
This happens probably because IntelliJ resolves dependency based on the JDK it uses to run instead of the one configured for the module.
The issue doesn't happen when launching from command line, eg maven. It only happens when launch from IntelliJ with the Engine main class.
The workaround is for us to exclude sfm-util from lightning-csv dependencies and explicitly pull it without classifier.

This is indeed an IntelliJ issue that has already been reported, see
https://github.com/arnaudroger/SimpleFlatMapper/issues/673
https://youtrack.jetbrains.com/issue/IDEA-203432


Mongo ssl

Got things set up for secure socket layer.
Should be good to go for when I switch over to atlas in production.

Gewd!


[armel] "go test" fails at TestTplGoFuzzReports

According to the Debian armel (soft-float ARM platform) build daemon build log, go test fails at TestTplGoFuzzReports:
=== RUN   TestTplGoFuzzReports
--- FAIL: TestTplGoFuzzReports (0.05s)
    template_test.go:141: #2 Test 1 should have errored
FAIL
FAIL    github.com/spf13/hugo/tpl   0.310s

I was able to reproduce this on Debian armel developer machine abel.debian.org, but I need more time to investigate the cause.
Note that this problem does not affect armhf (ARM hard-float).

What Go version is this?
I wrote these tests, and these are REALLY corner cases in the Go stdlib ... so please just remove the entire TestTplGoFuzzReports if you want. This was interesting when I tested these (and found a couple of stdlib bugs), but now the Go team can manage this.

It is with go version go1.5.1 linux/arm, or, more specifically, 1.5.1-4, that was compiled from source by Debian.
I am glad that you wrote these corner tests, and I am happy to keep the TestTplGoFuzzReports around even for Debian's armel build until the said problem is solved by the Go team.
If any armel people complains, I could always disable the go test line altogether, or just add a || : at the end to let the go test passes regardless of the actual test result.  ;-)

I'm pretty impressed by the Debian test setup.

I'm pretty impressed by the Debian test setup.

Me too!  :wink:  Though at times I took it for granted, and mistakenly thought some Debian infrastructures as antiquated.  But indeed, Debian does cover a lot of architectures, and its build farms not only bring as many software as possible to lesser-used platforms, but also provide a great testing infrastructure for free!
Here is the current cross-platform support status for Hugo:

https://buildd.debian.org/status/package.php?p=hugo

I wonder if I could use gccgo (instead of golang-go) to get Hugo on more platforms (such as alpha, mips, s390x, sparc64, etc.), see https://bugs.debian.org/785649 for go-md2man as an example.  The go-md2man package itself has been patched to allow gccgo, but not its dependencies (see https://buildd.debian.org/status/package.php?p=go-md2man), so that is what I will try next.  :-)

Update for v0.16:
=== RUN   TestTplGoFuzzReports
--- FAIL: TestTplGoFuzzReports (0.05s)
	template_test.go:274: #2 Test 1 should have errored
FAIL
FAIL	github.com/spf13/hugo/tpl	0.658s

Full build log at https://buildd.debian.org/status/fetch.php?pkg=hugo&arch=armel&ver=0.16-1&stamp=1465729600

This issue has been automatically marked as stale because it has not been commented on for at least six months.
The resources of the Hugo team are limited, and so we are asking for your help.
If this is a bug and you can still reproduce this error on the master branch, please reply with all of the information you have about it in order to keep the issue open.
If this is a feature request, and you feel that it is still relevant and valuable, please tell us why.
This issue will automatically be closed in the near future if no further activity occurs. Thank you for all your contributions.


Not supporting ES2.X

Installation is failing on ES2.X

The current release does not support es 2.x.
Any contributions are welcome!


Issue #2: Manipulate the Pricing Data

⚙ Feature
What capability will users have when this feature is merged?

📐 Functionality
A description of how the application will work from the perspective of the user.
User will see a map with marked locations of Olive Gardens close to their current area.
🖼 Wireframes
Wireframe screenshot(s) of what the app should look like when it is complete.
✅ Task Lists
Commit checklists for tasks needed to complete the feature.
🐘 Database Tasks
[ ]
☁ Server Tasks
[ ]
⚡ Client Tasks
[ ]
🧑⚕ Clean Up Tasks
[ ] Make sure that no React state objects are being mutated.
[ ] Make sure that no event listeners are manipulating the DOM directly.
[ ] Make sure that click targets are either  or  elements.
[ ] Make sure that all click targets show a cursor: pointer when hovered with the mouse.
[ ] Remove all buttons and placeholders that do not serve the functionality of the feature.
[ ] Remove all console logs.
[ ] Remove all commented out code.
[ ] Remove all CSS properties that have no effect.
[ ] Check all code for proper formatting and indentation.
[ ] Confirm that there are no errors in the console while using the application.
[ ] Confirm that all previous functionality still works without errors.
[ ] Confirm that the user interface looks natural on both mobile and desktop screens.
[ ] Create GIFs of the feature being used on both mobile and desktop screen sizes for inclusion in Pull Request.

Hey Alex, do you need any help with this task?
If so I'd love to help.


Warnings even if the right json version is available

Hi guys,
I'm hitting this warning:
WARNING: jmespath gem requires json gem >= 1.8.1; json 1.8.0 already loaded

Even if the right version of json/json_pure is available:
# gem list

*** LOCAL GEMS ***

aws-sdk (2.2.37)
aws-sdk-core (2.2.37)
aws-sdk-resources (2.2.37)
jmespath (1.2.4)
json (1.8.3)
json_pure (1.8.3)
root@maxscale01:~#

Current ruby version:
# ruby --version
ruby 1.9.3p484 (2013-11-22 revision 43786) [x86_64-linux]

Which comes from https://github.com/jmespath/jmespath.rb/blob/master/lib/jmespath.rb
Doing some testing I see that if I don't specify any gem, it shows a json version 1.8.0:
# irb
irb(main):001:0> require 'json'
=> true
irb(main):002:0> Object.const_defined?(:JSON)
=> true
irb(main):003:0> JSON::VERSION < '1.8.1'
=> true
irb(main):004:0> JSON::VERSION
=> "1.8.0"

The thing is that if I load json_pure, I get the right JSON::VERSION:
# irb
irb(main):001:0> gem('json_pure', '>= 1.8.1')
=> true
irb(main):002:0> require 'json'
=> true
irb(main):003:0> JSON::VERSION
=> "1.8.3"
irb(main):004:0>

And if I call for the json gem directly I also have the right version:
# irb
irb(main):001:0> gem('json', '>= 1.8.1')
=> true
irb(main):002:0> require 'json'
=> true
irb(main):003:0> JSON::VERSION
=> "1.8.3"
irb(main):004:0>

Shouldn't we try to load the json gem, fallback to json_pure and warn if both gem loads fail?
I can push a PR for that if you want, should be a quite easy fix, but I'd like your opinion 1st. :)
Thanks
Joseph

Proposed a "fix" in #28

The JSON::VERSION check up front is intentional. In Ruby 1.9.3, it does not matter if you have json version 1.8.1+ installed. Calling a blind require 'json' will actually load an older version of json. This means if a user has the following code in Ruby 1.9.3:
require 'json'
require 'jmespath'

Then they will likely get a warning. Can you try adding the following to your Gemfile to see if this fixes the issue?
gem 'json', '>= 1.8.1'

No I can't I'm using this in a script to set a puppet fact.
I don't understand what's wrong with the PR. I was working on the different environments I tested it with.
As I explained in the description I already have the json gen installed in the right version.

With Ruby 1.9.3-p551 and the following gems installed:
json (1.8.3, 1.8.0)
json_pure (1.8.3, 1.8.0)

I ran the following code:
require 'json'
JSON::VERSION
#=> 1.5.5

Now, in the same process, I ran the code from the pull request:
def fallback_gem
  begin
    # Fallback on the json_pure gem dependency.
    gem('json_pure', '>= 1.8.1')
    require 'json'
  rescue Gem::LoadError
    if Object.const_defined?(:JSON) && JSON::VERSION < '1.8.1'
      warn("WARNING: jmespath gem requires json gem >= 1.8.1; json #{JSON::VERSION} already loaded")
    else
      warn("ERROR: unable to load any json library")
    end
  end
end

begin
  # Attempt to load the native version if available, not availble
  # by default for Ruby 1.9.3.
  gem('json', '>= 1.8.1')
  require 'json'
rescue Gem::LoadError
  fallback_gem
end

JSON::VERSION
#=> 1.5.5

The code fails to produce the desired warning because the code in the first (bottom) begin rescue block will not raise. It simply returns false because require 'json' has already happened. The result is version 1.5.5 JSON is loaded and no warning is emitted. This is not desired.
Also, care is required to ensure that you do not call require 'json/ext' or require 'json/pure' afterwards or things start to break.

Hi @trevorrowe,
No offense, but are you sure your environment was clean when you did the test?
Using a freshly installed 1.9.3 rvm environment:
$ ruby --version
ruby 1.9.3p551 (2014-11-13) [x86_64-linux]

I did 3 tests:
Content of the script used it's exactly the same version as you, just printing the last line:
def fallback_gem
  begin
    # Fallback on the json_pure gem dependency.
    gem('json_pure', '>= 1.8.1')
    require 'json'
  rescue Gem::LoadError
    if Object.const_defined?(:JSON) && JSON::VERSION < '1.8.1'
      warn("WARNING: jmespath gem requires json gem >= 1.8.1; json #{JSON::VERSION} already loaded")
    else
      warn("ERROR: unable to load any json library")
    end
  end
end

begin
  # Attempt to load the native version if available, not availble
  # by default for Ruby 1.9.3.
  gem('json', '>= 1.8.1')
  require 'json'
rescue Gem::LoadError
  fallback_gem
end

puts "Final version of the gem: #{JSON::VERSION}"

Test 1: No Json gem with the right version (only the 1.5.5 that comes with ruby) and no json_pure either
The script cannot find the JSON constant, so the gem 1.5.5 is not loaded.
$ gem list

*** LOCAL GEMS ***

bigdecimal (1.1.0)
bundler-unload (1.0.2)
executable-hooks (1.3.2)
gem-wrappers (1.2.7)
io-console (0.3)
json (1.5.5)
minitest (2.5.1)
rake (<IP_ADDRESS>)
rdoc (3.9.5)
rubygems-bundler (1.4.4)
rvm (<IP_ADDRESS>)
$ ruby test.rb
ERROR: unable to load any json library
test.rb:32:in `<main>': uninitialized constant JSON (NameError)
$

Test 2: with json_pure 1.8.1 installed
The script loads the json_pure gem:
$ gem install json_pure -v 1.8.1
Fetching: json_pure-1.8.1.gem (100%)
Successfully installed json_pure-1.8.1
Installing ri documentation for json_pure-1.8.1
1 gem installed
$ gem list

*** LOCAL GEMS ***

bigdecimal (1.1.0)
bundler-unload (1.0.2)
executable-hooks (1.3.2)
gem-wrappers (1.2.7)
io-console (0.3)
json (1.5.5)
json_pure (1.8.1)
minitest (2.5.1)
rake (<IP_ADDRESS>)
rdoc (3.9.5)
rubygems-bundler (1.4.4)
rvm (<IP_ADDRESS>)
$ ruby test.rb
Final version of the gem: 1.8.1
$

Test 3: with json = 1.8.2
$ gem install json -v 1.8.2
Fetching: json-1.8.2.gem (100%)
Building native extensions.  This could take a while...
Successfully installed json-1.8.2
Installing ri documentation for json-1.8.2
1 gem installed
$ gem list

*** LOCAL GEMS ***

bigdecimal (1.1.0)
bundler-unload (1.0.2)
executable-hooks (1.3.2)
gem-wrappers (1.2.7)
io-console (0.3)
json (1.8.2, 1.5.5)
json_pure (1.8.1)
minitest (2.5.1)
rake (<IP_ADDRESS>)
rdoc (3.9.5)
rubygems-bundler (1.4.4)
rvm (<IP_ADDRESS>)
$ ruby test.rb
Final version of the gem: 1.8.2
$

Test 4: with a require json done 1st
Now, if in my script I load the gem 1.5.5 first, yes, I get a bad behavior. Is that the test case you are talking about?
Adding those 2 lines at the top of the script:
require 'json'
puts "Initially loaded version of gem: #{JSON::VERSION}"

Gives me indeed the result you were talking about:
$ ruby test.rb
Initially loaded version of gem: 1.5.5
Final version of the gem: 1.5.5
$

But that's easily fixable. I just need to add the following code to unload the gem and corresponding constant if the module has been loaded in an older version:
  if Object.const_defined?(:JSON) && JSON::VERSION < '1.8.1'
    if Object.const_defined?("JSON")
      Object.send(:remove_const, "JSON")
    end
    $".delete_if {|s| s.include?('json') }
  end

Which would give us something like:
def fallback_gem
  begin
    # Fallback on the json_pure gem dependency.
    gem('json_pure', '>= 1.8.1')
    require 'json'
  rescue Gem::LoadError
    if Object.const_defined?(:JSON) && JSON::VERSION < '1.8.1'
      warn("WARNING: jmespath gem requires json gem >= 1.8.1; json #{JSON::VERSION} already loaded")
    else
      warn("ERROR: unable to load any json library")
    end
  end
end

begin
  if Object.const_defined?(:JSON) && JSON::VERSION < '1.8.1'
    if Object.const_defined?("JSON")
      Object.send(:remove_const, "JSON")
    end
    $".delete_if {|s| s.include?('json') }
  end
  # Attempt to load the native version if available, not availble
  # by default for Ruby 1.9.3.
  gem('json', '>= 1.8.1')
  require 'json'
rescue Gem::LoadError
  fallback_gem
end

Would you be ok with this behavior and code?
If so, can the PR be reopened so that I add this chunk of code to it?
Thanks
Joseph

Yes, the scenario I'm concerned about is when JSON is loaded prior to jmespath being loaded. I'm not sure how I feel about removing a loaded version of JSON. I would prefer to avoid such tactics. These things tend to alway go bad in the end.
What issue do you have with the current behavior? Is it the warning that you concerned with? The warning can be improved with guidance on how to resolve this.

My issue is that now, when I'm running puppet, it loads custom facts that use the aws-sdk gem (which calls jmespath). Somewhere in the facts or in a 3rd party library, the require 'json' is called without caring about the version.
Because of that we now have warnings showing up every time we run puppet. Warnings we cannot do anything about it as it comes from one of the latest version of this lib. All the requirements are in place. I'm just trying to make sure the lib uses what is available properly as everything it requires is already there.
I don't see any other way than unloading the lib and reloading it with the right version. As I was saying earlier, the json_pure >= 1.8.1 is a prerequisite of this gem anyway, so it should be at least available.
There is really little chances that the require 'json' done previously will not work in the new version (If it's the case the lib in question should probably update).

Closed by #35. We've removed the dependency on json_pure, and have added some additional code to deal with the old Ruby 1.9.3 & JSON 1.5.5 differences.


Out-of-bounds read vulnerability found in python:3.9-alpine Docker image due to e2fsprogs package

The python:3.9-alpine Docker image and its corresponding SHA256 image digest (python@sha256:e1141f10176d74d1a0e87a7c0a0a5a98dd98ec5ac12ce867768f40c6feae2fd9) are found to contain an Out-of-bounds read vulnerability. This vulnerability is associated with the e2fsprogs package versions earlier than 1.46.6-r0, which is included in the python:3.9-alpine image.
This vulnerability could potentially be exploited by a remote attacker to cause a denial-of-service condition or execute arbitrary code on the affected system. The issue has been assigned the identifier SNYK-ALPINE317-E2FSPROGS-3339842 and is rated as high severity.
Docker users who have based their images on the python:3.9-alpine image or are using the image in their deployments are advised to update the e2fsprogs package to the latest version (1.46.6-r0 or higher) to mitigate the vulnerability. It is also recommended to monitor for any suspicious activity on the system and apply security best practices. More information about this vulnerability and its impact can be found at https://security.snyk.io/vuln/SNYK-ALPINE317-E2FSPROGS-3339842.

As far as I can tell, the vulnerable implementation is not included in the image (it's specifically e2fsck that is vulnerable, and our image only contains libcom_err-1.46.6-r0 which does not include e2fsck).
See also https://github.com/docker-library/faq#why-does-my-security-scanner-show-that-an-image-has-cves
Given that this comes in the layer which installs Python, it should be updated the next time Python bumps or the next time Alpine does a rebuild, whichever is sooner.


chore(main): release 5.6.6

:robot: I have created a release beep boop
5.6.6 (2023-11-02)
Minor changes and bug fixes

add user in footer on validation warnings (eb26f5a)
playerSkip: only log on error (f5a39f4)
update extractor to stable (d7ef706)

Miscellaneous

deps-dev: bump @types/node from 20.8.8 to 20.8.10 (8d35f0b)
deps: bump discord-player from 6.6.5-dev.4 to 6.6.6 (34ad44e)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/mariusbegby/cadence-discord-bot/releases/tag/v5.6.6 :sunflower: