debugpy halts any build at the check phase

Describe the bug
I use debugpy indirectly using vscode-extensions.ms-python.python. Whenever I build my configuration with this extension, it halts at pytestCheckPhase at 94%. Even after waiting for hours, it remains like this.
Steps To Reproduce
Steps to reproduce the behavior:

Build debugpy using nix-shell -p python310Packages.debugpy
See build halt at 94%

Expected behavior
The package builds without hanging.
Screenshots

Additional context
I have tested this with debugpy using python311 instead of python310 and it works just fine.
Notify maintainers
debugpy - @kira-bruneau, if possible, fix this issue with Python 3.10
ms-python - @jraygauthier @jfchevrette, if possible, use Python 3.11 so that this bug is fixed until it's fixed in debugpy itself
Metadata
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.1.55, NixOS, 23.11 (Tapir), 23.11.20230925.6500b45`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.17.0`
 - nixpkgs: `/nix/store/i6hq63805y784dml4j8pgkrz3lryxzld-source`

Are you still seeing this hang? I'm guessing that it was either affected indirectly by a dependency or it non-deterministically hangs.
Hydra is building it fine, but there was a point where it also hanged: https://hydra.nixos.org/build/236258670.

Yup, I reproduced it again just now; attaching a screenshot as well.

Building debugpy as a dependency of another package (obs-studio-plugins.obs-backgroundremoval), at fd309df083ec8a904f2ccb3fceb39bfb523c32be, appears to have hung even earlier, at 17%.

I think this is the same problem. Mine is hanging with the following output:
$ nix build target_that_includes_debugpy
...
python3.10-debugpy> rootdir: /build/source
python3.10-debugpy> configfile: pytest.ini
python3.10-debugpy> testpaths: tests
python3.10-debugpy> plugins: timeout-2.2.0, xdist-3.5.0
python3.10-debugpy> 2 workers [1216 items] m
[1/0/22 built] building python3.10-debugpy-1.8.0 (pytestCheckPhase)

Still running into this, no matter how often I update my flake inputs...
Not sure what it even is a dependency of, but seems like the pytestCheckPhase of python3.10-debugpy-1.8.0 just does not finish 🫤

I can build it by disabling the pytest check:
pkgs.python310.debugpy.overrideAttrs (self: super: {
    pytestCheckPhase = ''true'';
});

but I don't know how to overlay this in other packages (I don't really understand that aspect of Nix).

@charmoniumQ Hey! I've got the same issue. Building python3.11-debugpy-1.8.0 stuck at 17%. I've worked around by overriding the package debugpy by setting doCheck = false.
In my case, for an overlay usage, here's my config
final: prev: with prev; {
  ...
  python3 = lib.recursiveUpdate python3 {
    pkgs = lib.recursiveUpdate python3.pkgs (callPackage ./python3 { });
  };
}

After digging a little bit, I've found it was used by ms-python.python (at least in my case). I don't know what is the python3 package for now, but it is different from python311Packages, python310Packages and I guess python3Packages. I've tried override it using them without success.
After applying the overlay and the doCheck = false;, everything works fine on my end :+1: !

As a temporary workaround, I added the following overlay:
nixpkgs.overlays = [
  (final: prev: {
    python3 = lib.recursiveUpdate prev.python3 {
      pkgs = lib.recursiveUpdate prev.python3.pkgs {
        debugpy = (prev.python3.pkgs.debugpy.overrideAttrs (oldAttrs: {
          doCheck = false;
          pytestCheckPhase = ''true'';
        }));
      };
    };
  })
];

Can you try using an overlay with pytest-timeout (https://pypi.org/project/pytest-timeout/#:~:text=pytest --timeout%3D300) and see which test times out, so that we can inform upstream about this and disable the test?

I can reliably reproduce hangs across variety of Intel and AMD CPUs. With a timeout of 15 minutes I see for example tests/debugpy/test_args.py::test_args[attach_pid-program]  hang.
https://gist.github.com/mweinelt/1d89eca9f13f6c8d2b0d5f42606592fb

Running with a timeout of 5 minutes, these are all the failing tests I have: https://gist.github.com/999eagle/99344e055b307e549d8e60426bbcad9f
All of these do some kind of attach_pid, so maybe that is the cause here?

Seems to have been fixed by #296478. Feel free to re-open if this recurs.


Add section for supported backend versions in README

Signed-off-by: Alexandre Beslic<EMAIL_ADDRESS>

@abronan should we add boltdb ?

@vieux Yes
@sanimej Do you have any specifics to add on this doc about the boltDB version supported by libkv?

@abronan AFAIK we don't have any specific version dependencies for boltdb.


RTC + SRAM

What happen here?

Moved to 0.12. Need more Gekko speed to test it.

IPL Menu is behaving strangely. It constantly asks to change the date. But TBR is set correctly and works synchronously (which is already good). It is necessary to study what is going on there and fix it (bootrtc.cpp).

This is an si.cpp / pad.cpp problem :lmao:
For some reason, the stick to the right is stuck. The same thing happens in WARIO WARE. Also SSBM requires all 4 controllers, which is strange.
Everything is fine with bootrom, RTC and SRAM.


OSS-Fuzz Integration

My name is McKenna Dallmeyer and I would like to submit python-docx to OSS-Fuzz.
If you are not familiar with the project, OSS-Fuzz is Google's platform for continuous fuzzing of Open Source Software.
In order to get the most out of this program, it would be greatly beneficial to be able to merge-in my fuzz harness and build scripts into the upstream repository and contribute bug fixes if they come up. Is this something that you would support me putting the effort into?
Thank you!

Hi,
Following up on the feature request I submitted in June. Looking forward to hearing from you soon!


Fixing update code

There is an error in the update code: if you try to update the sample template with a dataframe (i.e. a new sample template) that has less samples that the one in the database, the current system will fail with the lovely error: Can only compare identically-labeled DataFrame objects
However, this should be allowed: if I submit a new sample template with only one row and one column, because I want to update only such value, I should be able to do it. This can minimize the errors introduced during sample template updates.
Another advantage of this, is that then I can use the interface to extend the current sample template by adding only the new samples (and leave the old ones as they're). Since the interface performs the extend and update at the same time, this was failing, but now this is possible.
@antgonza @ElDeveloper possible a quick review here? This will be needed for the AG work

:+1:


collection plugin for backend admin screen

collection plugin pulls in metadata for collections in archivesspace
plan is to have a screen where person enters in archivesspace uri to create a new collection, also option to add a thumbnail. thumbnail will be stored locally? other potential option is generating thumbnails on the fly, selecting an image from within the collection
option depends o how many collections we have, who will do the manual work

to discuss complete workflow with jeff + fernando
where to source images for private content if it's not available in public client interface.
right now in the repository the workflow is

create a collection
import objects into collection (import folder matches collection pid)
go into collection and select an object to use for thumbnail

this is done in three steps, we haven’t built out the ability to do it all at once. collections always have to exist first before you import objects and select a thumbnail

basically, we just need to figure out where backend will get thumbnails from and when

depends on https://github.com/dulibrarytech/digitaldu-frontend/issues/204

for archival objects, use "object" property
for collection objects, use "thumbnail" property
workflow for thumbnails for collections:

a collection gets created uses collection edit form
collection gets default DU shield thumbnail
user wants to change the thumbnail, goes into collection edit form
user enters url for an image
thumbnail gets reindexed in collection elasticsearch index, "thumbnail" property gets reindexed and changed. archival objects are different because thumbnails sourced from "object" property

done thanks fernando!


Why did you redo so much code?

(not an issue, but couldn't quickly find another way to contact you)
Why did you redo so much code? Particularily JS parsing... uglifyjs already does a pretty good job at it (you get a nicer AST with my lib https://github.com/thejh/node-astjourney ).

Also, your code looks like it would have a huge benefit from streamlinejs. Just saying.

The specific reason for this project was uglifiy's lackluster support for
comments in js. We have very specific needs around this in our current
project. The initial idea was to augment ulgify to expose the functionality
we need. This was not feasible, however.
On Thu, Jan 5, 2012 at 11:03 AM, Jann Horn <
<EMAIL_ADDRESS>
wrote:

(not an issue, but couldn't quickly find another way to contact you)
Why did you redo so much code? Particularily JS parsing... uglifyjs already does a pretty good job at it
(you get a nicer AST with my lib https://github.com/thejh/node-astjourney).

Reply to this email directly or view it on GitHub:
https://github.com/jdeal/doctor/issues/1

--
For science

I think Bryan said it well enough, but I'll just elaborate.
Uglify is not friendly to comments. (I want the code to generate the documentation, but you still at least have to use comments for descriptions.) Even its own readme explains this. I tried with ridiculous effort to make it work, but after a couple weeks, I gave up. I was able to get PEG.js to give me a comment-friendly AST in a few hours. Enough said.
Uglify also doesn't make a very nice AST in general, and yes, your module helps, but I started working on doctor before you announced your module.
As far as streamline, that's subjective. (Although I'm sure Bruno would disagree.) Most of the ugliness in my code is due to node.js's damned synchronous require. I wrote a lot of nice asynchronous code with the async library, and then I had to rip it all out because I wanted doctor to be able to run behind a require. See my discussion with Bruno on the node.js mailing list about this. Streamline cannot help. Nor can fibers.


default value doesn't work

Hello guys,
I have a little problem, I believe that is at least. Well, one of my tables has a default value, but the field with the default value is Boolean and is not null. When I pass a false value (which my business rule allows) Entity Framework changes to true always.
When I took this line of code, everything worked correctly again.
.HasDefaultValueSql("true");
Can anybody help me?
I use the EF CORE 1.1.2 with NPGSQL Driver

Duplicate of #7089 and see also  #8400 and #7163.


Provide authentication for Facebook, Twitter and GitHub too

Implemented, seems to work for others, hopefully only my phone screws up the redirection...

Maybe #44 is the way to go then we don't have to deal with this at all.

Opening a new ticket for the error


clarification on adding / (if it not possible - letting user know) specific tags of repos in marathon add

➜  Desktop marathon help add
📦  add
Add a package from a given URL to be able to use it from your scripts
👉  Usage: 'marathon add '
ℹ  You can also use a 'Marathonfile' to automatically add packages. See https://github.com/johnsundell/marathon for more information

eg. want to add tag 0.0.1
marathon add https://github.com/johndpope/swift-grpc-tensorflow.git
https://github.com/johndpope/swift-grpc-tensorflow/tree/0.0.1

Version pinning is not yet supported, but it's something that we do want to add support for. I added this issue: https://github.com/JohnSundell/Marathon/issues/82 to keep track of this new feature, so feel free to put any comments/suggestions you might have in there 🙂 Closing this in favor of the linked issue.


JAMES-3395 make examples not inherit from parent, move package of examples out from org.apache.james

Thanks for the initiative :) I don't think I have issues with that, on the contrary
Just a question, is org.apache.examples.james as a groupId a good one? (Opened question, I don't pretend having the answer)

one alternative to that would be org.example.james

I don't think you can omit apache in there, as those modules belong to the apache foundation right?

Merged. Can you please help close the PR @aasaru ? Thank you very much for your contribution :)


[BUG] Log warning in sublime text

Describe the bug

I have two files: rich_log.py and config.py.
In rich_log, I set up a logger and run a build in Sublime Text, everything is good.
In config, I import the log from rich_log and then run it, but there's a warning message.

rich_log.py
#!/usr/bin/env python3
# -*- coding: utf-8 -*-

import logging
import os

from rich.logging import RichHandler

def set_log(logger_name='rich') -> logging.Logger:
    logger = logging.getLogger(logger_name)
    logger.setLevel(logging.DEBUG)

    # rich handler
    rh = RichHandler()
    logger.addHandler(rh)

    # file handler
    # ...

    return logger

log = set_log(__name__)

if __name__ == '__main__':

    from rich import inspect
    inspect(log)
    log.debug("Log message")
    log.info("Log message")
    log.error("Log message")
    log.warning("Log message")
    log.critical("Log message")

The file builds without any warnings in Sublime Text.
config.py
from rich_log import set_log, test_log

from rich import inspect

log = set_log()
inspect(log)
log.debug("Log message")
log.info("Log message")
log.error("Log message")
log.warning("Log message")
log.critical("Log message")

Platform

MacOS
SublimeText 4152
Python 3.11
rich 12.6.0

I noticed that when I added rh = RichHandler(show_time=False) in rich_log.py, it caused a change in the warning message in config.py. This leads me to believe that there may be something related to the rich library.

And if I set a logger from logging, there's no warning.
# rich_log.py
log1 = logging.getLogger()

# config.py
from utils.logging_lite import set_log, log1

Does "rich" do something special or return something?

Nothing to do with Rich. It is something your editor is doing.


[BUG] SwaggerUI options displayRequestDuration missing

Information

Version: 5.59.1
Packages: @tsed/swagger

ISwaggerUIOptions no longer contains displayRequestDuration in the new TSED versions. Was this removed?
https://github.com/TypedProject/tsed/blob/abbdb6f9831cd5e605f29da3c84ed5c1b4ba89c3/packages/swagger/src/interfaces/ISwaggerSettings.ts
Example
@Configuration({
    rootDir,
    swagger: [
        {
            path: path.resolve(mountPath, 'docs'),
            options: {
                 displayRequestDuration: true  // this was available before
            }
            spec: {
                info: {
                    title: 'API',
                    version: '1'
                },
            }
        }
    ]
})

I never seen this options before (an the commit history doesn't mention that). I think it's just because the typings on this object is strict.
Meanwhile, you can do that:
@Configuration({
    rootDir,
    swagger: [
        {
            path: path.resolve(mountPath, 'docs'),
            options: {
                 displayRequestDuration: true  // this was available before
            } as any,  // add this to force your options
            spec: {
                info: {
                    title: 'API',
                    version: '1'
                },
            }
        }
    ]
})

See you

:tada: This issue has been resolved in version 5.59.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Moving lines in text editor results in strange scroll change

Moving a line with ALT+UP/DOWN in the text editor makes the line snap to the very bottom, which makes moving lines very annoying.

I can't reproduce this (or I don't know what you mean 😓 )


Make library more generic

Fixes #11
This change was actually relatively easy, but it will require a few changes:

Rename the repo to eslint-plugin-playwright
Publish the new package to npm
Deprecate the old package with a note to use the new package name.

Shall we migrate to the https://www.npmjs.com/package/eslint-plugin-playwright instead in your opinion?

Yeah, I think so.  Having the jest-playwright prefix in the name doesn't make sense if this also works for playwright test.  As long as we deprecate with a message about the new lib name, we should be good.


Add 1 click trash, heal and speed changed to 1 click

1 click trash:

Toggle Trash hotkey: adds/removes current hand from 1 click trash
Quick Trash: trashes items based on list

1 click speed and heal:

modified speed and heal to run in 1 click
sorry I forgot to fetch before writing this so there might be merge conflicts but should work out fine

yo im confused
how tf do I install this shi and what version is this even for

yo im confused how tf do I install this shi and what version is this even for

If you specifically want my fork you can find it here:
https://github.com/pixelizedgaming/DungeonMew/releases/tag/v1.0.3
It's for version 1.20.1


handleBeforeInput not working with Composition mode

Do you want to request a feature or report a bug?
BUG
What is the current behavior?
handleBeforeInput does not capture emoji input from emoji picker
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem.
https://jsfiddle.net/m6z0xn4r/179/
Try press Ctrl + Cmd + Space to invoke the emoji picker. Select an emoji as input, the handleBeforeInput did not trigger.

What is the expected behavior?
handleBeforeInput should capture all inputs either from emoji picker or typing.
Which versions of Draft.js, and which browser / OS are affected by this issue? Did this work in previous versions of Draft.js?
MacOS and Firefox is affected.

After reading some code, I found that when input emoji using emoji picker on Firefox, the editor mode switch to composition mode, so the onBeforeInput handler changed from editorOnBeforeInput to DraftEditorCompositionHandler.onBeforeInput which doesn't invoke handleBeforeInput.

Closing this issue because it's a duplication of https://github.com/facebook/draft-js/pull/844


Don't use bash double-bracket conditional in the Makefile because it doesn't work in CI

CI logged this error:
Dec 04 18:12:22 /bin/sh: 1: [[: not found

This appears to be because the shell used by make doesn't support double-bracket conditional syntax, so change it to use a different command.  The error itself is more noise than anything else, as the conditional is only there to prevent accidentally deploying pachyderm to the wrong kubernetes cluster which has no reason to even be configured on a CI machine, and it doesn't appear to be blocking anything.

Lgtm, thanks!

Please merge when tests pass


Consider adding a round of Newton's method to sqrt

Right now, sqrt is incredibly fast, but the ~5%  max error really makes it a poor fit for many uses. Adding a single round of Newton's method makes it much more precise, at the expense of a division. This could be behind a feature flag, or the current sqrt could be kept available as fast_sqrt or similar.

We could potentially add a feature like precision to improve precisions. Of course, it would have global impact.


Cursor grabbing

Replaces grab_cursor and ungrab_cursor by set_cursor_state.
Partially implement this on win32.
A few minor fixes.

cc @aepsil0n

Nice. I guess it makes sense to have a single method and also to separate the hide/grab states.

I added that last commit, but I need to wait for an update in winapi before it compiles.

The functions have been added to winapi, but it can't be published: https://github.com/rust-lang/cargo/issues/1465


Discuss removing DisposableAction to discourage bad use.

https://github.com/aspnet/Razor/pull/86#discussion_r16626731

It looks like in Razor Evolution this type moved into a Legacy parser folder: https://github.com/aspnet/Razor/blob/dev/src/Microsoft.AspNetCore.Razor.Evolution/Legacy/DisposableAction.cs
Does that mean this issue will be resolved when we switch to Evolution?

Nope, still an "issue". If/when we re-do the parser this issue will go away. I'd close it since the only time we'd move away from it is on a parser re-write.


MetaVoice?

MetaVoice seems amazing as a TTS allowing any voice model 1 shot training instantly (sounds too good to be true).
https://github.com/metavoiceio/metavoice-src/issues/1
It has some issues with MPS of course and would be nice to put into candle, is this something that technically is possible?
I haven't looked close, but could try myself but also suspect it's a big job? Putting it on the radar if not already since I really need this :D and many others certainly do too since a missing piece in "good" TTS that is fully open/free and especially being run in Rust like this!

We're certainly lacking a good TTS example at the moment, as pointed out in #1428 (we already cover speech to text with whisper, and both image to text and text to image). I started putting up a musicgen example but didn't finish it, it's based on encodec which metavoice also use so I might well resume work on this. I think it's actually a bit of work as it's a new type of modality but probably not something impossible neither.

Nice, yes I love musicgen too so that sounds amazing!

An initial version of metavoice is now available, #1717 , you can give this a shot with this example.
Please let us know how it goes, note that speaker embeddings are not available at the moment so no voice cloning, and that quality can probably be improved.

Just tried it out on my M2(CPU), took about a minute but it works!

Exciting amazing progress!
I seem to get a failure with using cpu or metal. With CPU you can see here it outputs information but doesn't use GPU/CPU and sits there forever not doing anything. With Metal it outputs about a missing function...
MacBook-Pro:candle christi$ cargo run --example metavoice --release -- --prompt "This is a demo of text to speech by MetaVoice-1B, an open-source foundational audio model." --out-file out.wav --tracing

    Finished release [optimized] target(s) in 0.32s
     Running `target/release/examples/metavoice --prompt 'This is a demo of text to speech by MetaVoice-1B, an open-source foundational audio model.' --out-file out.wav --tracing`
avx: false, neon: false, simd128: false, f16c: false
Running on CPU, to run on GPU, build this example with `--features cuda`
prompt: 'This is a demo of text to speech by MetaVoice-1B, an open-source foundational audio model.'
[2133, 2153, 2320, 2388, 2307, 2434, 2158, 2160, 2328, 2305, 2150, 2169, 2165, 2327, 2311, 2456, 2150, 2419, 2452, 2428, 2377, 2146, 2135, 2160, 2355, 2150, 2094, 2098, 2115, 2093, 2399, 2313, 2161, 2325, 2094, 2164, 2483, 2374, 2323, 2514, 2487, 2380, 2307, 2166, 2149, 2154, 2160, 2321, 2160, 2149, 2150, 2157, 2095, 2561]
^C

MacBook-Pro:candle christi$ cargo run --example metavoice --release --features metal -- --prompt "This is a demo of text to speech by MetaVoice-1B, an open-source foundational audio model."

    Finished release [optimized] target(s) in 0.92s
     Running `target/release/examples/metavoice --prompt 'This is a demo of text to speech by MetaVoice-1B, an open-source foundational audio model.'`
avx: false, neon: false, simd128: false, f16c: false
prompt: 'This is a demo of text to speech by MetaVoice-1B, an open-source foundational audio model.'
[2133, 2153, 2320, 2388, 2307, 2434, 2158, 2160, 2328, 2305, 2150, 2169, 2165, 2327, 2311, 2456, 2150, 2419, 2452, 2428, 2377, 2146, 2135, 2160, 2355, 2150, 2094, 2098, 2115, 2093, 2399, 2313, 2161, 2325, 2094, 2164, 2483, 2374, 2323, 2514, 2487, 2380, 2307, 2166, 2149, 2154, 2160, 2321, 2160, 2149, 2150, 2157, 2095, 2561]
Error: Metal error Error while loading function: "Function 'cast_bf16_f32' does not exist"

Caused by:
    Error while loading function: "Function 'cast_bf16_f32' does not exist"
    ```

    Thank you!

Exciting amazing progress!
I seem to get a failure with using cpu or metal. With CPU you can see here it outputs information but doesn't use GPU/CPU and sits there forever not doing anything. With Metal it outputs about a missing function...
MacBook-Pro:candle christi$ cargo run --example metavoice --release -- --prompt "This is a demo of text to speech by MetaVoice-1B, an open-source foundational audio model." --out-file out.wav --tracing

    Finished release [optimized] target(s) in 0.32s
     Running `target/release/examples/metavoice --prompt 'This is a demo of text to speech by MetaVoice-1B, an open-source foundational audio model.' --out-file out.wav --tracing`
avx: false, neon: false, simd128: false, f16c: false
Running on CPU, to run on GPU, build this example with `--features cuda`
prompt: 'This is a demo of text to speech by MetaVoice-1B, an open-source foundational audio model.'
[2133, 2153, 2320, 2388, 2307, 2434, 2158, 2160, 2328, 2305, 2150, 2169, 2165, 2327, 2311, 2456, 2150, 2419, 2452, 2428, 2377, 2146, 2135, 2160, 2355, 2150, 2094, 2098, 2115, 2093, 2399, 2313, 2161, 2325, 2094, 2164, 2483, 2374, 2323, 2514, 2487, 2380, 2307, 2166, 2149, 2154, 2160, 2321, 2160, 2149, 2150, 2157, 2095, 2561]
^C

MacBook-Pro:candle christi$ cargo run --example metavoice --release --features metal -- --prompt "This is a demo of text to speech by MetaVoice-1B, an open-source foundational audio model."

   Finished release [optimized] target(s) in 0.92s
    Running `target/release/examples/metavoice --prompt 'This is a demo of text to speech by MetaVoice-1B, an open-source foundational audio model.'`
avx: false, neon: false, simd128: false, f16c: false
prompt: 'This is a demo of text to speech by MetaVoice-1B, an open-source foundational audio model.'
[2133, 2153, 2320, 2388, 2307, 2434, 2158, 2160, 2328, 2305, 2150, 2169, 2165, 2327, 2311, 2456, 2150, 2419, 2452, 2428, 2377, 2146, 2135, 2160, 2355, 2150, 2094, 2098, 2115, 2093, 2399, 2313, 2161, 2325, 2094, 2164, 2483, 2374, 2323, 2514, 2487, 2380, 2307, 2166, 2149, 2154, 2160, 2321, 2160, 2149, 2150, 2157, 2095, 2561]
Error: Metal error Error while loading function: "Function 'cast_bf16_f32' does not exist"

Caused by:
   Error while loading function: "Function 'cast_bf16_f32' does not exist"
   ```

   Thank you!

Metal isn't supported yet due to missing kernels, but for CPU it also took a bit of time for me. You just gotta let it run, it will finish eventually.

Ah yes I see that now after quite awhile. Yes works here too, thank you!
Running on CPU, to run on GPU, build this example with `--features cuda`
prompt: 'This is a demo of text to speech by MetaVoice-1B, an open-source foundational audio model.'
[2133, 2153, 2320, 2388, 2307, 2434, 2158, 2160, 2328, 2305, 2150, 2169, 2165, 2327, 2311, 2456, 2150, 2419, 2452, 2428, 2377, 2146, 2135, 2160, 2355, 2150, 2094, 2098, 2115, 2093, 2399, 2313, 2161, 2325, 2094, 2164, 2483, 2374, 2323, 2514, 2487, 2380, 2307, 2166, 2149, 2154, 2160, 2321, 2160, 2149, 2150, 2157, 2095, 2561]
text ids len: 55
sampling from logits...
codes: [[[1109, 1129, 1296, ...,  738,  408, 1024],
  [1024, 1024, 1024, ...,  913,  424, 1024],
  [1024, 1024, 1024, ...,  786,   36, 1024],
  ...
  [1024, 1024, 1024, ...,  881, 1011, 1024],
  [1024, 1024, 1024, ..., 1015,  853, 1024],
  [1024, 1024, 1024, ..., 1019,  948, 1024]]]
Tensor[[1, 8, 538], u32]
text_ids len: 54
audio_ids shape: [1, 8, 483]
output pcm shape: [1, 1, 154930]

Yeah it takes a bit of time to get the generation back, maybe we should have some progress bar or some other way to know that the process is not stuck. I'm also looking at getting this to run on metal though currently it doesn't seem to bring much speedup on a M2.

Very fast now on metal!

Hey,
Can you please help me. I'm trying without installing xformers and q=q.half(), it's getting stuck while running as shown in screenshot without any error. I'm using mac m2 with CPU.


Document which NetBox object types are discovered by the NAPALM agent

It's currently necessary to look in https://github.com/netboxlabs/diode-agent/blob/develop/diode-napalm-agent/diode_napalm/translate.py to see which NetBox object types would be discovered by the agent.
It would be more convenient to have this documented in the README over here: https://github.com/netboxlabs/diode-agent/blob/develop/diode-napalm-agent/README.md
Possibly here on the main docs too: https://docs.netboxlabs.com/netbox-extensions/diode-agent/

Added in https://github.com/netboxlabs/diode-agent/pull/11


[pom] Add project build output timestamp for reproducible builds

Coverage remained the same at 93.333% when pulling 4287706d63923a60a9dcb5fa95b842332b238328 on hazendaz:master into 37dd8a74b31aaf2f105782799b1e42c3d95f6557 on mybatis:master.


Add trigger for setting vlucas/phpdotenv to mutable

For allowing .env files to override environment variables defined in Homestead.yaml.
As Homestead has been designed with the idea of hosting multiple projects from a single Vagrant box I had assumed that any environment variables defined by its YAML configuration file were simply defaults, and that specifying a different value for a given variable in an individual project's .env file would override it. After about an hour of debugging and discussion in the #laravel IRC channel it became apparent that this is caused by Dotenv being set to immutable by default, something which Laravel does not provide a means to alter.
I propose adding a trigger which developers can use by adding an environment value:key pair to their Homestead.yaml file. If this is found, Dotenv will be set to mutable and it will then be possible to override YAML-defined environment variables on a per-project basis while still providing a catch-all default value.
variables:
    - key: VAR_MUT
      value: mutable

This will not change how Laravel and Homestead operate for the vast majority of users, and environment variables will remain immutable by default. However, it will provide a quick and easy way to make them mutable for those who feel they benefit from it.

Fix the cs please.

Just to be sure, is it just that you want me to add braces to the if? Or are there other changes I should make as well?

@GrahamCampbell is the cs fine on this now?

Ill have an alternative to this soon probably.

Have to say I was excited about becoming a Laravel contributor, even just by adding something this small, so I'm a little sad it wasn't approved. I am very much looking forward to seeing your solution though, @taylorotwell, given your intimacy with the codebase.


Params do not encode %20 to + or space

What version of Elysia is running?
1.1.25
What platform is your computer?
wsl
What steps can reproduce the bug?

import Elysia, { t } from "elysia";

const api = new Elysia().get("/:id", ({ params }) => params, {
  params: t.Object({
    id: t.String(),
  }),
});

const url = new URL("http://localhost:3000/hello world");
console.log(url.href);

const result = await api
  .handle(new Request(url.href))
  .then((response) => response.json() );
console.log("🚀 ~ result:", result)
//🚀 ~ result: { id: 'hello%20world' }

What is the expected behavior?
The id in params should be hello world
What do you see instead?
hello%20world
Additional information

Have you try removing the node_modules and bun.lockb and try again yet?
yes

i’d say behaving as expected — url parameters shouldn’t have special characters, otherwise it could behave erratically with routing (e.g.: /hello%2Fworld, would id be hello/world? Then if i go to /hello/world how will it know to escape the slash or not?)
to put it simply, the encoding of URI components in the pathname happens on the requester’s end (postman, insomnia, the browser) and elysia just takes it as it comes without decoding.

but it already parsed and go through root endpoint ** shouldn't it just automatically encode it, if the id hello/world then it should go to hello first and then tried to encode world
this is the default behavior for queryString too btw

The PR https://github.com/elysiajs/elysia/pull/958 also fixes this issue


Run with json

runs can now be configured with a json file.

solves: https://github.com/dyigitpolat/mimarsinan/issues/34


network filters: add onNewConnection() callback

This is more fallout from c172a4. For network filters we want the
ability to do things when the connection is created, but also to
allow stopping iterations of further filters. In the previous change,
we made it so that the proxy filter will always make a new connection
to upstream regardless of what other filters do. We change that now
such that the tcp proxy filter will only make the upstream connection
in the onNewConnection() callback. Other filters (currently client_ssl
and ratelimit) now do their work on their own onNewConnection() callback
and stop iteration if necessary to prevent the tcp proxy filter from
making the upstream connection until it is supposed to.

@lyft/network-team @jsnoop36

:+1:

+1


ENH: LBPROC IMDI

Currently iris does not accept LBPROC values of IMDI.  The UM F03 spec specifies:
"It should be 0 if no processing has been done, otherwise the relevant numbers from the list below are added together"
...
  File "/home/h05/cpelley/git/iris/lib/iris/fileformats/pp.py", line 694, in __init__
    raise ValueError('Negative numbers not supported with '
ValueError: Negative numbers not supported with splittable integers object

In reality, real datasets can be found with its value set to IMDI.  I see no problem in interpreting LPBROC of IMDI as a 0.  This PR does exactly this.

I see no problem in interpreting LPBROC of IMDI as a 0.

If IMDI should be interpreted as 0 then that needs to be made clear in F3. IMDI is effectively "I don't know what's been done", which isn't the same as "nothing has been done".

OK thanks @rhattersley I'll run it passed them and post an update.

I'll run it passed them and post an update

That'd be very helpful - thanks @cpelley. :+1:

It seems there is no appetite to cater for this case.  I'll close this PR.
Does beg the question though, people should have a means to apply a callback of sorts on the field so that they can load it into iris (they know what it means), why should they be forced to change their source data?

It seems there is no appetite to cater for this case

Hi @cpelley - just to be clear, who has no appetite? The guardians of F3?

Apologies for my vagueness, I meant both colleagues and 'guardians of F03'.
Thanks for your interest on this in any case, appreciated.


remove double word

I noticed the word complex appears twice in this sentence.
Using person-shaped glyphs (as opposed to abstract symbols like circles or squares)
has been shown not to produce additional empathic responses [55], but including actual
photographs of people helps readers gain interest in, remember[56, 57],
and communicate complex complex phenomena [58] using visualizations.

which I thought was a typo.

Great catch, thanks @eitanlees!


flimmaker fails to compile with “fatal error: 'libavformat/avformat.h' file not found”

flimmaker fails to compile on my M1 Mac running macOS Monterey:
$ make
c++ -std=c++2a -c -O3 -I liblzg/src/include flimmaker.cpp -o flimmaker.o
flimmaker.cpp:253:35: warning: 'tmpnam' is deprecated: This function is provided for compatibility reasons only.  Due to security concerns inherent in the design of tmpnam(3), it is highly recommended that you use mkstemp(3) instead. [-Wdeprecated-declarations]
    std::string cache_file = std::tmpnam( nullptr );
                                  ^
/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/usr/include/stdio.h:186:1: note: 'tmpnam' has been explicitly marked deprecated here
__deprecated_msg("This function is provided for compatibility reasons only.  Due to security concerns inherent in the design of tmpnam(3), it is highly recommended that you use mkstemp(3) instead.")
^
/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/usr/include/sys/cdefs.h:208:48: note: expanded from macro '__deprecated_msg'
        #define __deprecated_msg(_msg) __attribute__((__deprecated__(_msg)))
                                                      ^
1 warning generated.
c++ -std=c++2a -c -O3 imgcompress.cpp -o imgcompress.o
c++ -std=c++2a -c -O3 image.cpp -o image.o
c++ -std=c++2a -c -O3 -I liblzg/src/include watermark.cpp -o watermark.o
c++ -std=c++2a -c -O3 ruler.cpp -o ruler.o
c++ -std=c++2a -c -O3 reader.cpp -o reader.o
reader.cpp:4:10: fatal error: 'libavformat/avformat.h' file not found
#include <libavformat/avformat.h>
         ^~~~~~~~~~~~~~~~~~~~~~~~
1 error generated.
make: *** [reader.o] Error 1

It doesn’t seem to find libavformat/avformat.h, even though ffmpeg was installed with brew install ffmpeg and the header files do exist:
find /opt -name avformat.h
/opt/homebrew/Cellar/ffmpeg/5.1/include/libavformat/avformat.h
<EMAIL_ADDRESS>
It is working fine on Ubuntu 22.04 (Intel).

Ouch, gotta love  C++. My Mac is only a Big Sur Intel, so I can't reproduce.
However, there is nothing special in the Makefile, everything is searched for at standard locations.
On my machine:
src % ls -l /usr/local/lib/libavformat.*
lrwxr-xr-x  1 fred  admin  56 Nov 13  2021 /usr/local/lib/libavformat.58.76.100.dylib -> ../Cellar/ffmpeg/4.4.1_3/lib/libavformat.58.76.100.dylib
lrwxr-xr-x  1 fred  admin  49 Nov 13  2021 /usr/local/lib/libavformat.58.dylib -> ../Cellar/ffmpeg/4.4.1_3/lib/libavformat.58.dylib
lrwxr-xr-x  1 fred  admin  42 Nov 13  2021 /usr/local/lib/libavformat.a -> ../Cellar/ffmpeg/4.4.1_3/lib/libavformat.a
lrwxr-xr-x  1 fred  admin  46 Nov 13  2021 /usr/local/lib/libavformat.dylib -> ../Cellar/ffmpeg/4.4.1_3/lib/libavformat.dylib
src % ls -l /usr/local/include/libavformat
lrwxr-xr-x  1 fred  admin  44 Nov 13  2021 /usr/local/include/libavformat -> ../Cellar/ffmpeg/4.4.1_3/include/libavformat
src %

I guess brew install did not add those links in your case, not sure why.
Note that the Makefile format is completely vanilla, you can probably add the right CFLAGS and LDFLAGS or the -I and -L in the commands if you want to compile flimmaker with this install.

OK, good to know there’s nothing that would generally prevent it from working on M1. I’ll see if I can get those links fixed and let you know.

I’m really not so familiar with C++ and Makefiles, but I found the following issue which might also apply here?
https://github.com/pixop/video-compare/issues/9#issuecomment-979957156
Let me know what you think.

Homebrew installs to /opt/homebrew on Apple Silicon (incl. M1) Macs, as opposed to /usr/local on Intel Macs. I don't think the Makefile is at fault, since it doesn't hardcode any include or lib paths.
Rather, I think Apple's GCC (built-in) doesn't know to look in /opt/homebrew for headers / libraries; therefore, one would need to add the following to their environment variables:
CFLAGS += -I/opt/homebrew/include/
LDLIBS += -L/opt/homebrew/lib/

...though, I'm not sure what the issue would be if you're using Homebrew's GCC. The above vars would probably still be the (temporary) solution though.

Thx to you both. This indeed seems to be the problem.
@aaronk6, can you git pull and verify you can compile it on M1 by either:

uncommenting the CFLAGS and LDLIBS flags in the Makefile (line 2 and 3)

OR

invoking make using CFLAGS=-I/opt/homebrew/include/ LDLIBS=-L/opt/homebrew/lib/ make (longer to type but you won't have merge conflict later).

While I did some test, I could not verify that it works on a real M1. If that doesn't work, post the error and reopen this defect.

It’s working now! 🎉 I also had to downgrade FFmpeg to 4.x though. Not a big deal, but I anyway captured this in a separate issue: https://github.com/fstark/macflim/issues/35
Regarding the include paths, your suggestions worked perfectly, but I eventually ended up adding the following to my .zshrc after doing some more research:
export CPATH=/opt/homebrew/include
export LIBRARY_PATH=/opt/homebrew/lib

It read that Homebrew on M1 deliberately doesn’t the include paths to the native install in /opt/homebrew to avoid conflicts with the Rosetta install in /usr/local. However, since I always want the native installation to take precedence, I think it’s a good approach to set the paths globally.
If you don't mind, I’m happy to create a PR for the README that explains those caveats (potentially also mentioning the FFmpeg 5 issue). Let me know!

No problem, feel free to update the README and make a PR.
Keep things as simple as possible, so I guess the general build section should mention that for now we use ffmpegv4 and update the brew instructions to match that. Then let's add an M1 section to the Mac instructions with the Makefile edit. If that grows too long (README is already too long, so at some time I will split some of the advanced parameter in different file), then just push a README.OSX and we'll insert a link to it for detailled instructions. Not sure if I'm clear.
Let's keep #35 for eventual support of ffmpeg5.

Here’s the PR: https://github.com/fstark/macflim/pull/36


@ashfurrow => some changes to expose more in our api

I am trying to migrate eigen's zoomable artwork image view to use ARTiledImageScrollView, which works well in Folio. I found myself wanting some more things available to me, so these are the changes I came up with. We can decide what does or doesn't make sense, this is just an idea. Here is the corresponding eigen PR, which is also a WIP https://github.com/artsy/eigen/pull/1971.

@1aurabrown Does this still need to be merged?
@dblock I would need write access to the repo to merge, anyway :)

I gave @ashfurrow and @1aurabrown access to this, looking forward to seeing this project evolve!

@1aurabrown any update on this?

@ashfurrow yeah, this can be ditched. this was to deal with the artwork zoom transition, but more recent changes mean i'd write it differnetly anyway. I'd rather start from scratch.


Issue when importing the ng2-tag-input module.

I'm submitting a ...  (check one with "x")
[ x] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request

Current behavior
On version 1.0.1, whenever the ng2-tag-input module is loaded, I'm getting the following error:
ERROR Error: Uncaught (in promise): TypeError: Cannot read property 'setGlobalVar' of null
TypeError: Cannot read property 'setGlobalVar' of null
    at _createNgProbe (ng2-tag-input.bundle.js:8521)
    at BotIdeModuleInjector.createInternal (module.ngfactory.js:176)
    at BotIdeModuleInjector.NgModuleInjector.create (core.es5.js:3557)
    at NgModuleFactory.create (core.es5.js:3530)
    at MapSubscriber.project (router.es5.js:523)
    at MapSubscriber._next (map.js:77)
    at MapSubscriber.Subscriber.next (Subscriber.js:89)
    at PromiseObservable.js:66
    at ZoneDelegate.invoke (zone.js:365)
    at Object.onInvoke (core.es5.js:4145)
    at ZoneDelegate.invoke (zone.js:364)
    at Zone.run (zone.js:125)
    at zone.js:760
    at ZoneDelegate.invokeTask (zone.js:398)
    at Object.onInvokeTask (core.es5.js:4136)
    at _createNgProbe (ng2-tag-input.bundle.js:8521)
    at BotIdeModuleInjector.createInternal (module.ngfactory.js:176)
    at BotIdeModuleInjector.NgModuleInjector.create (core.es5.js:3557)
    at NgModuleFactory.create (core.es5.js:3530)
    at MapSubscriber.project (router.es5.js:523)
    at MapSubscriber._next (map.js:77)
    at MapSubscriber.Subscriber.next (Subscriber.js:89)
    at PromiseObservable.js:66
    at ZoneDelegate.invoke (zone.js:365)
    at Object.onInvoke (core.es5.js:4145)
    at ZoneDelegate.invoke (zone.js:364)
    at Zone.run (zone.js:125)
    at zone.js:760
    at ZoneDelegate.invokeTask (zone.js:398)
    at Object.onInvokeTask (core.es5.js:4136)
    at resolvePromise (zone.js:712) [angular]
    at resolvePromise (zone.js:683) [angular]
    at polyfills.bundle.js:7225:17 [angular]
    at Object.onInvokeTask (core.es5.js:4136) [angular]
    at ZoneDelegate.invokeTask (zone.js:397) [angular]
    at Zone.runTask (zone.js:165) [<root> => angular]
    at drainMicroTaskQueue (zone.js:593) [<root>]

This error is occurring in the core codebase, as well as when trying to inject ng2-tag-input in any tests.
This was not occurring as of the previous version I was using (0.9.10)
Expected behavior
The module loads correctly.
Minimal reproduction of the problem with instructions
Create a module that imports ng2-tag-input, either in tests or in code.
Here is an example module that is failing:
import { TagInputModule } from 'ng2-tag-input';
import { RendererSettingsComponent } from './renderer-settings/renderer-settings.component';
import { RendererPermissionsComponent } from './renderer-permissions/renderer-permissions.component';
import { RendererOverviewComponent } from './renderer-overview/renderer-overview.component';
import { rendererIdeRouting } from './renderer-ide.routes';
import { RendererResolve } from './shared/renderer-resolve.service';
import { RenderersService } from '../../shared/renderers.service';
import { CoreService } from './../../shared/core-service.model';
import { CoreIdeModule } from './../../core-ide/core-ide.module';
import { NgModule }       from '@angular/core';
import { CommonModule }       from '@angular/common';
import { FormsModule } from '@angular/forms';
import { ModalModule } from 'ng2-bootstrap';

@NgModule({
    declarations: [
      RendererOverviewComponent,
      RendererPermissionsComponent,
      RendererSettingsComponent
    ],
    imports: [
      CommonModule,
      CoreIdeModule,
      FormsModule,
      ModalModule,
      TagInputModule,
      rendererIdeRouting
      ],
    providers: [
      RendererResolve,
      {provide: CoreService, useExisting: RenderersService},
      RenderersService
    ],
})
export class RendererIdeModule { }

Here is an example of an import in a test that is failing:
    TestBed.configureTestingModule({
      declarations: [
        RendererSettingsComponent
      ],
      imports: [
          RouterTestingModule,
          TagInputModule,
          FormsModule,
          ModalModule.forRoot()
      ],
      providers: [
        {provide: ActivatedRoute, useValue: routerStub },
        {provide: CoreService, useValue: renderersServiceStub},
        {provide: ImagesService, useValue: imagesServiceStub}
      ]
    });

What do you use to build your app? (SystemJS, Webpack). Please specify the version
Using 1.0.0 of the angular-cli tool (webpack)

Angular version: 4.0.0

Browser:
Chrome and PhantomJS

Hi @rrfenton,
are you using disableDebugTools anywhere?

I am not.

By reading here https://github.com/qdouble/angular-webpack2-starter/issues/263 and here<EMAIL_ADDRESS>looks like that would be the issue. I'll let you know if I find anything.

@Gbuomprisco I have looked into both these issues and have determined these to not be the core cause.
As I said in the main post, this problem occurs after upgrading from 0.9.10 of ng2-tag-input to version 1.0.1.  Both my core application, and my tests work fine when on 0.9.10 (minus some warning regarding ng-template in the ng2-tag-input codebase).
Because of this peculiarity, I can only assume it's not an issue with my codebase, as the only common issue are modules where ng2-tag-input are imported.

Right now I have no idea what that error could be about. I'd suggest you to revert to the previous version if it works

I am also having the same issue:
27 03 2017 14:27:52.535:INFO [karma]: Karma v1.5.0 server started at http://<IP_ADDRESS>:9876/
27 03 2017 14:27:52.537:INFO [launcher]: Launching browser Chrome with unlimited concurrency
27 03 2017 14:27:52.544:INFO [launcher]: Starting browser Chrome
27 03 2017 14:27:53.482:INFO [Chrome 57.0.2987 (Mac OS X 10.12.3)]: Connected on socket jZm2nju6Do4C399LAAAA with id 18851526
Chrome 57.0.2987 (Mac OS X 10.12.3) TagInputComponent Basic behaviours should have 2 tags set by ngModel FAILED
TypeError: Cannot read property 'setGlobalVar' of null
at _createNgProbe (spec-bundle.js:9:3406677)
at DynamicTestModuleInjector.createInternal (ng:///DynamicTestModule/module.ngfactory.js:207:5)
at DynamicTestModuleInjector.NgModuleInjector.create (spec-bundle.js:9:164127)
at NgModuleFactory.create (spec-bundle.js:9:162854)
at TestBed._initIfNeeded (spec-bundle.js:9:1298533)
at TestBed.createComponent (spec-bundle.js:9:1304808)
at Function.TestBed.createComponent (spec-bundle.js:9:1291594)
at Object. (spec-bundle.js:9:4066832)
at Object. (spec-bundle.js:9:1280034)
at ZoneDelegate.invoke (spec-bundle.js:9:3756618)
at ProxyZoneSpec.onInvoke (spec-bundle.js:9:3721661)
at ZoneDelegate.invoke (spec-bundle.js:9:3756468)
at Zone.run (spec-bundle.js:9:3734988)
at Object. (spec-bundle.js:9:3703965)
at spec-bundle.js:9:1267879
at ZoneDelegate.invoke (spec-bundle.js:9:3756618)
at AsyncTestZoneSpec.onInvoke (spec-bundle.js:9:3679479)
at ProxyZoneSpec.onInvoke (spec-bundle.js:9:3721455)
at ZoneDelegate.invoke (spec-bundle.js:9:3756468)
at Zone.run (spec-bundle.js:9:3734988)
at AsyncTestZoneSpec._finishCallback (spec-bundle.js:9:1267512)
at spec-bundle.js:9:3679027
at ZoneDelegate.invokeTask (spec-bundle.js:9:3759102)
at Zone.runTask (spec-bundle.js:9:3738266)
at ZoneTask.invoke (spec-bundle.js:9:3762828)
at timer (spec-bundle.js:9:3831194)

It started happening when I upgraded to Angular 4.

It works with the 1.0.0

Same here. Ng-4.0.0 tag-input 1.0.1.
Also getting some warnings around template syntax, but that's another issue.

Can also confirm 1.0.0 works with ng-4.0.0.

I have the same issue with Angular 4.0.0 and "ng2-tag-input": "1.0.1"
1.0.0 works good.

Hi everyone - can anyone try<EMAIL_ADDRESS>I removed the animations module as I assume it should only be imported in the root - but am usure whether it was the actual issue?

It is working. Thanks.
I had to import the BrowserAnimationsModule in the root module, so I think it should be added to the readme.

true - i'll update the doc

@Gbuomprisco 1.0.2 works good (with imported BrowserAnimationsModule in app.module), thanks!
But I still have this warning: https://github.com/Gbuomprisco/ng2-tag-input/issues/297

Closing this :) please reopen if still an issue for anyone


Plug gazelle_cabal.ImportData in Resolve to allow both gazelle extensions to run in the same gazelle_binary

An attempt to solve #40
This applies the methodology suggested as the third item in this answer https://github.com/tweag/gazelle_haskell_modules/issues/40#issuecomment-1266854710
Now, gazelle_haskell_modules depends of gazelle_cabal, allowing us to pattern match on the type of imports in the Resolve function.

This is missing a test, right?

This is missing a test, right?
It is, I would like to be able to run the current examples before creating new ones.

For now, I'm closing the PR for considerations made here.
But I think this problem would be an opportunity to follow up with gazelle devs to make the integration of extensions easier.


Add processing option for manually rotating image

We have images w/o metadata that are rotated by 90°. We need to rotate them in a transformation. Since rotating is already there (by exif metadata) – would it be hard to add another processing option to achieve this?

I need this too – my use case involves nondestructive rotation. I know libvips supports this easily.

I need this too – my use case involves nondestructive rotation. I know libvips supports this easily.

rotate processing option was added to the latest build.

rotate processing option was added to the latest build.


Disable X11 support for TeX Live

Fixes https://github.com/conda-forge/texlive-core-feedstock/issues/13
Seems to be causing failures on OS X. Once we sort out how we want to handle X11 in conda-forge, we can probably make use of our own X11 libraries to provide support for this.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Artifact creation: Break retry on any 4xx code except 429

There's a debate around which 4xx code we're going to use. #2697 mentions 422 in the title but adds handling for 403. Some people think 429 is better.
Rather than playing whack-a-mole with all the 4xx codes across multiple releases, I think we should do the same thing as the log uploader and break on any of them.

Thanks, that makes sense. Changed it.


api-linter: init at 1.54.1

Description of changes
Add api-linter.
https://github.com/googleapis/api-linter
https://github.com/googleapis/api-linter/releases/tag/v1.54.1

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

@sikmir Thank you for the review.


Usage of flash attention

Consider wrapping the call to self.attention in InterpretableMultiHeadAttention with
with torch.backends.cuda.sdp_kernel(enable_flash=True, enable_math=False, enable_mem_efficient=True):
In order to improve speed and memory efficiency.

no https://github.com/pytorch/pytorch/issues/125674


[Test] Restoration of config upgrades

Write test for Restoration of config upgrades.
For example, this test to restore upgrades did not work because assumeStateWork in LedgerManagerImpl ends up being null. We can investigate this.

Closing, config upgrades have been switched to TEMPORARY so they are not restorable. #3990 adds tests enforcing config upgrade lifetimes.


TC-1 | TC-98 | Add message for Vote Casting

Explanation
New feature for vote!
More information
Screenshots/Screencaps

View

Manual testing steps
For view message - needs vote Against

Max votedEvent - 118, it's too early for pagination and batch batchRequest )

Statistic for Proposal vote
Proposal 13
fetchVotedEvents 34
fetchVotedEvents uniq 32
Proposal 12
fetchVotedEvents 44
fetchVotedEvents uniq 41
Proposal 11
fetchVotedEvents 29
fetchVotedEvents uniq 26
Proposal 10
fetchVotedEvents 52
fetchVotedEvents uniq 44
Proposal 9
fetchVotedEvents 16
fetchVotedEvents uniq 14
Proposal 8
fetchVotedEvents 11
fetchVotedEvents uniq 11
Proposal 7
fetchVotedEvents 46
fetchVotedEvents uniq 40
Proposal 6
fetchVotedEvents 29
fetchVotedEvents uniq 28
Proposal 5
fetchVotedEvents 52
fetchVotedEvents uniq 50
Proposal 4
fetchVotedEvents 69
fetchVotedEvents uniq 61
Proposal 3
fetchVotedEvents 17
fetchVotedEvents uniq 16
Proposal 2
fetchVotedEvents 31
fetchVotedEvents uniq 30
Proposal 1
fetchVotedEvents 118
fetchVotedEvents uniq 111


Get Teams and Channel Information from Call

This is not an issue but just a question that once bot receives an incoming call (online meeting/group call) is it possible to get Teams and Channel information  (e.g. Teams id and Channel id) from where the call/meeting was initiated? I have tried to examin by debugging the call handler  but not able to find any information. Or is there any Graph API endpoint that may take call id and return such information? Your suggestions are greatly appreciated.

As far as i know it's not possible to receive these information directly but with the chat endpoint in the Graph beta you can access chat messages by using chats/{thread Id from joinurl}/messages it's also possible to subscribe this with the subscription api but it's right now quite buggy


ENH: set default third derivative operator

The third derivative is implemented but the "scale dependent statistical property" fails because the default third derivative stencil is not set.

Sorry, this is wrong. The stencils are actually defined here but not tested. We should add tests at some point, I am approving this for now.


doesn't dump armv7

Hello, i installed an application from itunes on my iphone, so it should contain arm64 and armv7 arch, only it only pulls out the arm64, could you make an option for that it will dump both? or choose.

No, Can only dump the currently running architecture.


Screenshot

HOME

POST-LIST

POST

Can I change the lion image to my custom image?

@nhsprite You can do whatever you can.
p.s. If anyone have good idea about the logo of this theme, please comment here 😄。

hmm...the logo is nice, but I want to change to my own logo =。=....Anyway, how can I add comment to this theme?

@nhsprite
Just attach logo image file by dragging & dropping or pasting to the comment section.


adding dependencies broken

Gitea version (or commit ref):1.14.0+dev-604-gbc05ddc0e
Encountered on try.gitea.io
Can you reproduce the bug at https://try.gitea.io:

[x] Yes (provide example URL)  https://try.gitea.io/MathJo/Merge_dependency_test/pulls/3
[ ] No

Log gist:

Description
Steps to reproduce:
Go to try.gitea.io and try to add a dependency to a PR

This is not reproducible anymore


yaml file on downloadable

Below the yaml file is not downloadable to be able to use to create the stack.
Click on the link below to deploy the stack. This will take you to the CloudFormation console in your account. Use walab-ops-base-resources as the stack name, and take the default values for all options.
us-west-2 : here
ap-southeast-2 : here
ap-southeast-1 : here

@dekcaj23 you do not need to download the YAML templates.  Instead make sure you are logged into the AWS account you want to use, and then click the link.  It will automatically bring you to the CloudFormation console, and load the template

@sssalim-aws please take a look at this.  Looks like you may not have your templates in S3 setup for public read

@setheliot yes, that was the error I was getting.

@dekcaj23 Thank you for bringing this to our attention.
The issue now fixed, you should be able to deploy the using the link.
Since the fixed has been applied, I will now close this issue.


Add supportedQueryInterfaces support to PanelPlugin interface

See 'Decoupling data querying from panel plugin' discussion #956, this feature is follow up now that we have DataQueriesProvider.
Currently, we only have TimeSeriesQuery plugins that are used in our core panels (TimeSeriesChart, GaugeChart, StatChart). We also want the ability to add more query plugins such as TraceQuery and LogQuery. To support these we need a way for a panel plugin to specify which interfaces it supports (see PR #922 for more info in comment here)
export interface PanelPlugin<Spec = UnknownSpec> extends Plugin<Spec> {
  PanelComponent: React.ComponentType<PanelProps<Spec>>;
  supportedQueryInterfaces?: Array<'TimeSeriesQuery' | 'LogQuery' | 'TraceQuery'>; // NEW
  panelOptionsEditorComponents?: Array<PanelOptionsEditorComponent<Spec>>;
  hideQueryEditor?: boolean; // can be removed
}

In the example above, a future Table panel plugin would support all query interfaces, while TimeSeriesChart would not. hideQueryEditor can be removed and inferred when supportedQueryInterfaces is empty Array
Also, relates to #1151 for adding instant query support since a new query plugin is the ideal way to implement that functionality.

After #1318 is merged, we can introduce “supportedQueryInterface” in panel plugins. The idea here is to give the framework the ability to render the appropriate query editor depending on which panel plugin is selected, enhancing dashboard user experience. For example, if user selects a time series chart, our panel editor will know to render a time series query editor.
Additionally, the panel plugin can determine the active query interface to dynamically adjust its options. To do this, we can have PanelEditor becomes the children of DataQueriesProvider. This way, the options editor components can access useDataQueries or other relevant hooks to identify the data type returned by the query.

Hello! if you don´t mind I'm gonna take this one this week ;) I will ask you directly if I face any issue. Assigning to myself.

Regarding the PR upper 👆, I will close it and split it into three parts in order to be more readable and more feasible for me to implement

:one: the PluginEditor should support several query types`` (this is almost done in that PR)

:two: build a QueryEditor like TimeSeriesQueryEditor or TraceQueryEditor but that takes the QueryType as parameter (let's create a separate one and deprecate the two others)

:three:  Introduce the supportedQueryInterfaces (or supportedQueryTypes , or queryTypes) field and use it in the PanelSpecEditor to trigger usage of the new QueryEditor

Then it would be up to the plugin developer to use the generic QueryEditor or not through this new field.


Desktop Wallet Docs update

Transactions

[x] Send & Receive Page => replaced by Asset Page
[x] Asset selection & transaction flow update

Staking Operations

[x] We now have a more advanced Staking Page, with Staking Summary at the top
[x] Undelegate & Redelegate moved from Home Page to Staking Page
[x] Undelegation status review

Cronos Bridge Transfer

[x] Transaction Flow
[x] Bridge Config Settings

Ledger Support for Cronos

[x] Installation of Ethereum App for Cronos transactions
[x] Confirm transactions with Crypto.org / Ethereum App based on their sending assets

/staging

/staging

/staging

/staging

/staging

/staging


test, upcoming: [M3-7134, M3-7314] - Add integration test for AGLB Edit Certificate flow and improve form validation

Description 📝
This PR does two things, because fixing the validation was necessary for writing a complete integration test that confirmed all input errors were handled gracefully.

Add Cypress integration tests to confirm the UI flow for AGLB certification editing works as expected.
Provide form validation to prevent a user from being able to edit a certificate and provide a blank label.

Changes  🔄
Integration tests in load-balancer-certificates.spec.ts test the following:

User can edit certificates from the AGLB certificate page (route: /loadbalancers/:id/certificates)
Certificates page updates to reflect edited certificates
Cloud Manager handles user input errors (e.g. empty or invalid fields/selections) gracefully
Cloud Manager handles server errors (e.g. HTTP 500 responses from the API) gracefully

In the Cloud Manager UI:

A validation error ("Label must not be empty.") is provided when a user attempts to submit the Edit Certificate drawer with a blank label field.

Preview 📷

Before
After

How to test 🧪
Prerequisites
(How to setup test environment)

Checkout this PR.
Make sure the aglb feature flag is on and turn mocks on.

Reproduction steps
(How to reproduce the issue, if applicable)

Checkout develop.
Go to http://localhost:3000/loadbalancers/0/certificates.
Use the action menu (...) to select the Edit action and open the Edit Certificate drawer.
Clear the pre-populated label field.
Submit the form.
Observe there is no validation error and in the request payload, the blank label is submitted. (Due to the mocks, this isn't visible in the UI.)

Verification steps
(How to verify changes)

On this branch, go to http://localhost:3000/loadbalancers/0/certificates.
Use the action menu (...) to select the Edit action and open the Edit Certificate drawer.
Clear the pre-populated label field.
Submit the form.
Observe there is a validation error Label must not be empty and the form will not submit until a label is provided.

Also confirm the following validation still works as expected by checking the request payload:

When editing a TLS certificate:

A user can edit just the form label, and the form will submit. The certificate will be excluded from the payload since it has not changed.
A user can edit just the key, and the form will submit. The existing certificate will be included in the payload, since updating a key requires a cert.
A user cannot edit just the cert and will see a validation error on Private Key. This is invalid because updating a cert requires a key, and API does not return the key for security.
A user can edit both the cert and the key, and the form will submit. Both the edited cert and edited key will be in the payload.

Confirm the integration test passes:
yarn cy:run -s "cypress/e2e/core/loadBalancers/load-balancer-certificates.spec.ts"

As an Author I have considered 🤔
Check all that apply

[x] 👀 Doing a self review
[x] ❔ Our contribution guidelines
[x] 🤏 Splitting feature into small PRs
[x] ➕ Adding a changeset
[x] 🧪 Providing/Improving test coverage
[x] 🔐 Removing all sensitive information from the code and PR description
[x] 🚩 Using a feature flag to protect the release
[ ] 👣 Providing comprehensive reproduction steps
[ ] 📑 Providing or updating our documentation
[ ] 🕛 Scheduling a pair reviewing session
[ ] 📱 Providing mobile support
[ ] ♿  Providing accessibility support

Could we add a scroll to error, especially for the forms that have the Private Key field?

@coliu-akamai Good call out for smaller screens!
I implemented scroll-into-view for the Edit drawer in 423e64c.
https://github.com/linode/manager/assets/114685994/966fe97c-3c8e-4088-8e8c-ade7a5363e29
Since we'd want the same behavior in the Create drawer, so I made the change there too. (Also noticed that the Create drawer is not an expanded width, which is a bad UX when the user pastes long certs, so I adjusted in 7624cec.)
https://github.com/linode/manager/assets/114685994/47370313-d08f-4da8-aacf-37adb5ca001c


Correct the names of languages "Traditional chinese" and "Chinese" to the original spelling

In this section:

Can someone check the spelling?

I think the Quickdraw content in Traditional Chinese can be changed like this to make it better.
在開啟後 5 分鐘內關閉問題或拉取請

⬇ change to ⬇
在 PR 開啟 **5 分鐘內**關閉 PR

@ridemountainpig Hello! Can you make a PR?

@ridemountainpig Hello! Can you make a PR?

No problem 😊


Enemy Commander FSM and patrols

Q
A

Bug fix?
no

New feature?
yes

Needs wipe?
yes

Fixed issues
#574

Description:
Provides the first iteration on the enemy commander FSM, which will be used to add (re)actions from enemy side against the players.
First thing implemented in the basic structure of the enemy commander states depending on the awareness and strength. Also spawning of infantry or light vehicle patrols near the "frontline" which get waypoints between 2-4 sectors.
Content:

[x] Enemy Commander FSM
[x] States for enemy commander
[x] Patrol spawning
[x] "FOB not known directly" mechanic Will be moved to next sprint
[x] Recon team to search for FOBs in player territory, to discover them for the enemy commander Will be moved to next sprint
[x] Small code/format tweaks/optimizations

Successfully tested on:

[ ] Local MP
[ ] Dedicated MP

State of the commander FSM after this first iteration.


[Bug]: Air Gapped instance - Generate page with Crud - Success message image missing

Is there an existing issue for this?

[X] I have searched the existing issues

Description
In an airgapped intstance, when I try to generate data with crud, the success message image is missing
Steps To Reproduce

Pull the image with the command docker pull ankitakinger/appsmith:airgap
Switch off internet and execute the following:
docker run -e APPSMITH_AIRGAP_ENABLED=true -e KEYGEN_LICENSE_VERIFICATION_KEY=b2b98c313548c14cb22ddb4afb9124553c4c829c290d8a3cbf01ff8ee57624dc -d --name appsmith-airgap -p 80:80 -v "$PWD/stacks:/appsmith-stacks" ankitakinger/appsmith:airgap
Now do the set up
Add air gapped license key
Create a new DS and try and generate page with data
Note that the success message does not have the image.

The image on release is as below:

Public Sample App
No response
Environment
Release
Issue video log
No response
Version
Self Hosted

@berzerkeer FYI these are the images we are sending in a response from server:

https://assets.appsmith.com/crud/workflow_sql.svg
https://assets.appsmith.com/crud/workflow_s3.svg

Right, I will check. Thanks :D

Verified


Updating Install commands for Fedora

Fedora uses an updated version of yum, would like to mention that in the file.

Go ahead !


fixed issue #573 (only numbers in repo names), added repo name to zip/ta...

...r ball for better identification

@bjoernffm thanks for this. Would you be able to add some tests for repositories with only numbers?

@garygreen Could you explain how? Special framework needed, etc.? I'm not really familiar with automatic testing.


added unit test for setting the user and password in the uri for tcp …

…transport, used in JNDI.
Could not find an example in the existing unit tests.

Thanks for the test!
For the sake of thoroughness it would be best if security was enabled on the test broker and the test ensured the credentials set in the URL actually resulted in an authenticated session.  If you don't think you can add that I'll go ahead and merge what you have and try to get to it later.

@jbertram sure, I will give that a go

@jbertram,
as you suggested, enabled security on the test broker and add a role to ensure credentials passed with connection. Also enabled the default user to ensure existing tests passed without modification - I was trying to keep a small footprint.
Thanks
Pat

@pgfox  you know you can also do new ActiveMQConnectionFactory(uri);, right?
Just asking.. maybe that's something we need to blog about it.. or maybe make it clearer on the docs.


Rename baseserver to LCD

Thank you a lot for contributing to Cosmos Voyager!

Issue

closes: #585
Screenshots

❤ Thank you!

i love this. 🚀 📈 🍾


no_std example build

@vitiral @sbourdeauducq both no_std.rs example and no_std crate compiled successfully based upon current master. are they actually valid examples using no_std?
@vitiral when trying cargo test --features no_std on nightly like the ones in #13, it complains error: Package 'crc v1.5.0 (file:///.../rui/crc-rs)' does not have these features: 'no_std', wondering what was your setup?

The "standard" (there is no standard...) way is to run it with:
cargo test --no-default-features

Basically it was done in reverse: std is a "default feature" and you disable it for no_std crates.

@vitiral thanks, tried that and it built & passed the tests. Do you see master it not working for no_std?

the current way of having default = ["std"] in Cargo.toml is conventional and correct I believe.


[mwc-checklist-item] selected state won't apply if not rendered to dom

Describe the bug
https://user-images.githubusercontent.com/15140652/128464410-680c9665-f272-4a51-afe5-c4fbaf408b7b.mov
toggling the selected state of non-rendered items will not apply the change until rendered back, results in a quick jumpy repainting of the checkbox
To Reproduce
Steps to reproduce the behavior:

apply display none to mwc-list containing mwc-checklist-items
toggle all items' selected state
remove display none from mwc-list
the list will reveal and show the selected state change

Expected behavior
change should occur while not rendered and render back as the actual selected state
Screenshots
https://user-images.githubusercontent.com/15140652/128464410-680c9665-f272-4a51-afe5-c4fbaf408b7b.mov

This is a consequence of using animations to reflect state changes. Browsers queue up animations on an element that is hidden until it becomes visible, similar to what can be seen in this watered down example: https://codepen.io/allanchenmdc/pen/xxdBePY
One way to deal with this is to disable the animations when the element is hidden.

As in https://developer.chrome.com/blog/timer-throttling-in-chrome-88/#animation

Obsolete with M3, checkbox list items are built manually (we'll have prebuilt ones again in the future)


Hugo: remove deprecated social.twitter from config

Removed the social key in the site configuration. Wanted to add it to the params.toml, but noticed the DO NOT EDIT note.
It looks like social.twitter is not used anywhere. If it is, it needs to be added to the params.toml and can be called with Site.Params.social.twitter
For https://linear.app/usmedia/issue/CUE-345

Removed the social key in the site configuration. Wanted to add it to the params.toml, but noticed the DO NOT EDIT note.

This is not needed because the one in params.toml is about the twitter card and I think that one is still used for the twitter meta tags (when you share a page on twitter)

Removed the social key in the site configuration. Wanted to add it to the params.toml, but noticed the DO NOT EDIT note.

This is not needed because the one in params.toml is about the twitter card and I think that one is still used for the twitter meta tags (when you share a page on twitter)

Can you elaborate, since I do not understand what you want exactly.

I thought you weree talking about removing the twitter items in the params.toml : that is not needed. But I might have misread your comment because I think in the end it was ok to just delete the unused param

@JorindeUsMedia @anne-usmedia To confirm my understanding of your conversation, before I import: this PR's removal of a Twitter-related key from hugo/config/_default/config.toml is correct and sufficient; we don't (currently) need to remove a similar key from any other files.
Have I understood correctly?

@jpluscplusm Tthere was only a question if the twitter config had to be relocated from the config.toml to the params.toml. But since it is not being used, removing it from the config.toml is sufficient

Imported as https://review.gerrithub.io/c/cue-lang/cuelang.org/+/1172308

Closed via 4c42f40b91322868759dcc4c052183e3e88c3b0a


Upgrading the kubectl version in cr.sas.com/viya-4-x64_oci_linux_2-docker/sas-orchestration:1.116.2-20240312.1710256388641

Viya4 Deployment Version Details
While I am trying to deploy the sas viya, I get this error:
{"level":"info","version":1,"source":"sas-orchestration","messageKey":"sas-orchestration.lifecycle.operation.log","messageParameters":{"log":"version difference between client (1.27.9) and server (1.29.6-gke.1326000) exceeds the supported minor version skew of +/-1\n"},"properties":{"logger":"internal/lifecyclehelper","caller":"lifecyclehelper/lifecyclehelper.go:111"},"timeStamp":"2024-08-22T08:44:10.585277+00:00","message":"version difference between client (1.27.9) and server (1.29.6-gke.1326000) exceeds the supported minor version skew of +/-1\n"} Is there any option to control the version of this kubectl? Or I just need to downgrade my kubernetes cluster?
The pods that I am getting error:
sas-deployment-operator-reconcile-2664fc69-7cvhf   0/1     Error     0          10m
sas-deployment-operator-reconcile-2664fc69-cw6xd   0/1     Error     0          13m
sas-deployment-operator-reconcile-2664fc69-dkwvv   0/1     Error     0          15m
The image that is being used: cr.sas.com/viya-4-x64_oci_linux_2-docker/sas-orchestration:1.116.2-20240312.1710256388641
Ansible Variable File Details
No response
Steps to Reproduce
Setup the gke (with viya4-iac-gcp setting kubernetes_channel "STABLE") and run the ansible-playbook command to deploy the viya.
Expected Behavior
Expected behavior is deployment operator starts deploying the Viya in the cluster.
Actual Behavior
Deployment itself is successful.
Additional Context
No response
References
No response
Code of Conduct

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

@lthebe, based on the sas-orchestration version number, it looks like you're trying to deploy LTS 2024.03. The support range for 2024.03 is k8s 1.26 - 1.28.

@ephesused Thank you very much.


Implement selector based on PF name + VF index

As was discussed in issue #157.

Pasting errors from CI tests, lint check failed:
/home/travis/gopath/src/github.com/intel/sriov-network-device-plugin/.gopath/src/github.com/intel/sriov-network-device-plugin/pkg/resources/deviceSelectors.go:106:5: var vfId should be vfID /home/travis/gopath/src/github.com/intel/sriov-network-device-plugin/.gopath/src/github.com/intel/sriov-network-device-plugin/pkg/resources/pciNetDevice.go:51:2: var vfId should be vfID /home/travis/gopath/src/github.com/intel/sriov-network-device-plugin/.gopath/src/github.com/intel/sriov-network-device-plugin/pkg/utils/utils.go:338:1: exported function GetVFId should have comment or be unexported /home/travis/gopath/src/github.com/intel/sriov-network-device-plugin/.gopath/src/github.com/intel/sriov-network-device-plugin/pkg/utils/utils.go:338:31: func result vfId should be vfID Makefile:131: recipe for target 'lint' failed

@ark-g Can you please add relevant unit tests for the new function you've implemented. Specially GetVFID()  in pkg/utils/utils.go?

@ark-g Can you please add relevant unit tests for the new function you've implemented. Specially GetVFID() in pkg/utils/utils.go?

Done

/lgtm

@ark-g thanks for adding the doc in Readme!
/lgtm

@zshi-redhat
You are welcome. :)
I hope that now the feature will be more clear for users and reviewers.
Frankly, this should be done at earlier stages of development.
@ahalim-intel
Please let me know if some additional things needed prior to merge the feature.

@ahalim-intel
Are there any issues that prevents merging ?


set secrets and URLs in a separate file

I modified config.example.js and index.html to set secrets in a separate file, so the config.js of a running setup can be easy shared on github

In all fairness this creates even more problems for those getting started and I hardly find JSON.parse() justifiable for such cause let alone debugging becomes impossible. I have decent idea about splitting config into multiple files which I’m planning to share once I find a moment and my health condition will allow it.


Migrate primary branch from master to main

See https://github.com/Senzing/stream-loader/issues/283

Pull requests:

https://github.com/Senzing/g2loader/pull/59


Make ore bags printable on autolathe

Why it's not a feature by now, it's just a bag with funny magnet cmon.
Make it require cloth and steel (due to magnet part) idfk.

https://github.com/Goob-Station/Goob-Station/pull/1255


Feature/define and call functions

Create 2 new functionalities on pt_translator:

Create functions
Call functions

close #3


[Doc] Translate the design document (tracing.md)

Search before asking

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

Documentation Related
Please translate the docs/en/design-document/07-tracing.md (English version) into docs/zh/design-document/07-tracing.md (Chinese version).
By doing so, you can get a deeper understanding about the workflow module in EventMesh.
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

please assign this to me!

please assign me this task I'll complete it

Hi, I'm new to open source and want to try to solve some simple problems. Can you assign this issue to me? I would appreciate it!


feat/donation tests

initial
add github token
feat/clean-up
change project id
feat/adjust action for secrets
feat: clean up
more clean up

Checklist:

[ ] I have read freeCodeCamp's contribution guidelines.
[ ] My pull request has a descriptive title (not a vague title like Update index.md)
[ ] My pull request targets the main branch of freeCodeCamp.
[ ] I have tested these changes either locally on my machine, or GitPod.

Closes #XXXXX

Note to self:
Although the element with the class donation-elements is visible in the Cypress snapshots, the third party tests fail since .donation-elements does not seem to be passed to the Cypress-plugin-stripe-elements on time in the Github action.
As a result, Cypress throw the following assertion error:
Expected to find element: undefined, but never found it.

So github secrets are not being passed to pr github actions: however, they are passed once the pr is merged.

There are two ways to do this:

You can address any pending review commits and land this on a branch other than main on the main repo. That is, change the PR target (base) branch to a branch that exists on the main repo.

OR

You can test the CI on your fork by adding your keys in there & changing the setting on your fork to run workflows if it is disabled.

I have previously pushed it as an freeCodeCamp branch and the secrets are passed as expected; however, when this pr lands, the action will fail on all upcoming prs.

@raisedadead, I have previously pushed it as an freeCodeCamp branch and the secrets are passed as expected; however, when this pr lands, the action will fail on all upcoming prs.

Yes - that is correct, and this workflow will then be limited to not being available to PRs. That is we can disable the on: pull-request even for a while and then reactivate in a month or so when the PRs have been normalized and new PRs are up to date?

So since github secrets (except github token, etc) are not passed to github action workflows that are triggered by a pull request, the actions will fail regardless of when it triggered.
An example of a secret that is not being passed to pr triggered wokflows is secrets.CYPRESS_RECORD_KEY

I am not sure I understand this correctly. We already have Cypress workflows, and they work on PRs. Are mistakenly confusing on: pull_request & on: pull_request_target?

We already have Cypress workflows, and they work on PRs

I believe the CYPRESS_RECORD_KEY only handles the video/screenshot uploads, and doesn't fail the action.
__
  Summary: :hammer_and_wrench: This code needs a fix

firefox will be removed from stripe donation tests since:
Your project has set the configuration option: chromeWebSecurity to false

This option will not have an effect in Firefox. Tests that rely on web security being disabled will not run as expected.

I have pushed this as a branch and thing are working as expected.
https://github.com/freeCodeCamp/freeCodeCamp/actions/runs/2790419385
Let me know what you think.

firefox will be removed from stripe donation tests...

I feel your pain.

Just to confirm, have we resolved the secrets issue?

@naomi-lgbt yes. 🙏

Here is the last changes pushed on another branch. https://github.com/freeCodeCamp/freeCodeCamp/actions/runs/2816411977


Add support for CommandBehavior.SequentialAccess to ExecuteReader

Fix #114

Updates: i've stripped this down to just ignoring SequentialAccess.

:shipit:


File constructor does not allow str

Summary
The docs for File() do not say you can use a string path yet you can
Reproduction Steps
View docs, view editor, view source
Minimal Reproducible Code
n/a

Expected Results
Str in typings and docs pls
Actual Results
No typings and docs ;(
Intents
N/a
System Information
potato
Checklist

[X] I have searched the open issues for duplicates.
[X] I have shown the entire traceback, if possible.
[X] I have removed my token from display, if visible.

Additional Context
sorry for the memey completion of form

bytes is also accepted.
I can make a PR for this


Show open dialog effect

Open a dialog window for the open dialog effect.
Depends on #25.

Depends on #28.


scheduler kubernetes secrets

Signed-off-by: lidor ettinger<EMAIL_ADDRESS>
What does your PR do?
In the same way as we can add secrets for web and workers we would like to do also for scheduler.
This PR provides a way to add a credential for a local repository, such as nexus, in order to download its dependencies.
Checklist

[ ] Commits are signed
[ ] Commits are squashed (if appropriate)
[ ] Chart version bumped
[ ] Chart documentation updated
[ ] Passes linting

ok for me /lgtm


Removed Broken Link

Removed Broken Link
Screenshot of the broken link

Removed it from all the language-specific files
Full Report : https://gtmetrix.com/reports/expressjs.com/pXc1bK8U

I think this is a leftover from StrongLoop/IBM days when they were using Google Analytics to track visits to the site.  AFAIK, we are no longer doing this, so removing this stuff is fine.  This just removes the "noscript" version (ie. if browser JS is not enabled).
My only question is whether we should remove the "normal" version (if client JS is enabled, which I assume is the norm), e.g. https://github.com/expressjs/expressjs.com/blob/gh-pages/_includes/footer/footer-en.html#L4-19
I can't see any reason to keep it, unless there's a GA account that someone is using and I don't know about it.
Thoughts? @dougwilson @wesleytodd

unless there's a GA account that someone is using and I don't know about it.

Yeah, this would be good to know.  Maybe @dougwilson has it?  If so I would love to see it.  If not I agree we should remove it all.

I don't have a GA account; my understanding is that it was from StrongLoop/IBM. We can probably just get rid of it unless we want to actually set up an account and point to that.

I think it would be good to have analytics.  I am not a fan of GA, but that is really the most common and powerful free solution.  I wonder if there is something the OpenJS Foundation would think of doing for member projects, of if one of the paid providers  with better data privacy policy would be willing to offer it for free in exchange for logo placement or something?


Add new node versions for CI

Add node versions 8, 9, and 10 for Travis CI.

https://github.com/alexa-js/alexa-app-server/pull/117/files


Only highlights are imported but not the whole article

New user here :)
With a problem though: only highlights are imported but not the whole article.
The drop-down option is set accordingly. {{{content}}} or {{content}} do not appear in the template, I if insert either manually in the template nothing it still only imports the title, link the article and the highlights.
Thank you! I look forward to using the plugin!

{{{ content }}}

https://docs.omnivore.app/integrations/obsidian.html#importing-the-full-article-content

{{{ content }}}

https://docs.omnivore.app/integrations/obsidian.html#importing-the-full-article-content

Inserting {{{ content }}} anywhere in the template did not work for some reason but copying the code example of the link worked.
Thanks!


[?] Add support for new /local preflight query parameters

https://github.com/kadena-io/chainweb-node/pull/1585
In Chainweb 2.17, we removed error reporting from /send. However, we didn't have "best simulations" for users to try their code out in as close to real world context as possible. /local used to fill this gap, but it was not robust enough. This PR enhances the /local endpoint to allow the user toggle on or off "preflight" (as close as we can get to mainnet) simulations, as well as the ability for the user to specify rewinds and simulation at a particular block height, and the ability to bypass user signature verification.

The usage is thus:

The behavior of the bare /local endpoint has not changed, allowing people to keep their work flows.
The /local?preflight=<bool> query flag allows people to run their commands against the full mainnet workflow. If false or left unspecified, reverts to the legacy behavior (running a tx through the interpreter to get outputs)
The /local?rewindDepth=<word (non-negative integer)> parameter allows people to rewind to a particular block height rewindDepth-many block heights in the past and run their commands (think: regressing against forks). If left unspecified, defaults to running at the most current block height on that chain.
The /local?signatureVerification=<bool> query flag allows people to run their commands with or without user signature verification (useful in the case of detached signature signing). If unspecified, defaults to verifying the tx.
An example full query would be /local?preflight=true&rewindDepth=100&signatureVerification=true to run a full command workflow at a blockheight 100 heights in the past, with full signature verification.

KadenaJs needs to support these new query parameters for the local call.

Duplicate of #103 ?


xwayland: transparency in firefox windows

Firefox seems to show junk around the borders with drop shadows (depending on the GL clear color).
I think it is choosing the wrong visual depth. On rootston, it chooses a 24-bit depth but on weston it chooses a 32-bit depth for windows with drop shadows.
I know we support transparency on X11 windows for sure when they choose the correct depth. For instance, this program demonstrates that wlr-xwayland is capable of displaying windows with transparency.
Maybe it is copying this attribute from the parent (root window) which does not have an alpha channel. In that case, we might have to reparent the window to get Firefox to behave correctly.

I investigated this issue a little.
Firefox renders correctly in wlc. wlc uses the color depth to know if a surface has an alpha channel (weston and wlroots do the same). It uses this info to change the wl_surface format (https://github.com/Cloudef/wlc/blob/master/src/xwayland/xwm.c#L386, https://github.com/Cloudef/wlc/blob/f4ff049cc30242eec9f33af7129d2e2a2865fb5a/src/platform/render/gles2.c#L544). But in wlroots has_alpha is always false, so I'm not sure it's related to this issue (it seems that Firefox doesn't choose a 32-bit color depth, like @acrisci said). wlc does nothing special to set a particular visual ID/colormap to its root window, or to reparent client windows.
Weston uses a "frame" window. It's a window created when an xwayland window is mapped that has the same size as the client window. The frame window is set as the parent of the client window. The frame window has the colormap chosen in xwm_get_visual_and_colormap. I'm not sure if/why it's necessary to do this.

My theory (might be incorrect, just guessing).
I think there might be a COPY_FROM_PARENT somewhere that chooses the visual depth of the parent window. If the parent is root, it will choose the root visual depth which I think is 24. If it is reparented, it will choose the visual depth of the parent which we can set to 32.

I still don't understand why it works on wlc and doesn't on wlroots. wlc uses xcb_composite_redirect_subwindows_checked but that doesn't change anything.
Choosing a 32-bit color depth for the root window hides all xwayland windows.

Choosing a 32-bit color depth for the root window hides all xwayland windows.

I think this is barking up the right tree. Disclaimer: I know next to nothing about X11.

xwininfo on the main window of firefox looks about the same between rootston on and sway.
The only difference is on the popup. So the parent of the main window shouldn't be the cause of this.
xwininfo of the root looks mostly equivalent as well.
The biggest difference I can make out atm. Is the different NET_SUPPORTED atoms.
Sway on wlc has a lot of them, while the ones on rootston are rather limited
_NET_SUPPORTED(ATOM) = _NET_WM_CM_S0, _NET_WM_PID, _NET_WM_NAME, _NET_WM_STATE, _NET_WM_STATE_FULLSCREEN, _NET_SUPPORTING_WM_CHECK, _NET_WM_WINDOW_TYPE, _NET_WM_WINDOW_TYPE_DESKTOP, _NET_WM_WINDOW_TYPE_DOCK, _NET_WM_WINDOW_TYPE_TOOLBAR, _NET_WM_WINDOW_TYPE_MENU, _NET_WM_WINDOW_TYPE_UTILITY, _NET_WM_WINDOW_TYPE_SPLASH, _NET_WM_WINDOW_TYPE_DIALOG, _NET_WM_WINDOW_TYPE_DROPDOWN_MENU, _NET_WM_WINDOW_TYPE_POPUP_MENU, _NET_WM_WINDOW_TYPE_TOOLTIP, _NET_WM_WINDOW_TYPE_NOTIFICATION, _NET_WM_WINDOW_TYPE_COMBO, _NET_WM_WINDOW_TYPE_DND, _NET_WM_WINDOW_TYPE_NORMAL

vs
_NET_SUPPORTED(ATOM) = _NET_WM_STATE, _NET_ACTIVE_WINDOW, _NET_WM_MOVERESIZE, _NET_WM_STATE_FULLSCREEN, _NET_WM_STATE_MAXIMIZED_VERT, _NET_WM_STATE_MAXIMIZED_HORZ

Tried running xdpyinfo which prints info about visuals, they're almost the same in wlroots and in weston, the differences aren't very interesting.
diff --git a/home/simon/weston-xdpyinfo.log b/home/simon/wlroots-xdpyinfo.log
index b697a3af..ca843aa8 100644
--- a/home/simon/weston-xdpyinfo.log
+++ b/home/simon/wlroots-xdpyinfo.log
@@ -1,8 +1,8 @@
-name of display:    :2
+name of display:    :0
 version number:    11.0
 vendor string:    The X.Org Foundation
-vendor release number:    12000000
-X.Org version: 1.20.0
+vendor release number:    12099001
+X.Org version: <IP_ADDRESS>
 maximum request size:  16777212 bytes
 motion buffer size:  256
 bitmap unit, bit order, padding:    32, LSBFirst, 32
@@ -17,8 +17,8 @@ supported pixmap formats:
     depth 24, bits_per_pixel 32, scanline_pad 32
     depth 32, bits_per_pixel 32, scanline_pad 32
 keycode range:    minimum 8, maximum 255
-focus:  PointerRoot
-number of extensions:    25
+focus:  None
+number of extensions:    24
     BIG-REQUESTS
     Composite
     DAMAGE
@@ -32,7 +32,6 @@ number of extensions:    25
     RANDR
     RECORD
     RENDER
-    SECURITY
     SHAPE
     SYNC
     X-Resource
@@ -48,7 +47,7 @@ default screen number:    0
 number of screens:    1

 screen #0:
-  dimensions:    1024x640 pixels (271x169 millimeters)
+  dimensions:    1600x900 pixels (423x238 millimeters)
   resolution:    96x96 dots per inch
   depths (7):    24, 1, 4, 8, 15, 16, 32
   root window id:    0x39c
@@ -58,9 +57,10 @@ screen #0:
   default number of colormap cells:    256
   preallocated pixels:    black 0, white 16777215
   options:    backing-store WHEN MAPPED, save-unders NO
-  largest cursor:    1024x640
-  current input event mask:    0x580000
-    SubstructureNotifyMask   SubstructureRedirectMask PropertyChangeMask
+  largest cursor:    1600x900
+  current input event mask:    0x5a0000
+    StructureNotifyMask      SubstructureNotifyMask   SubstructureRedirectMask
+    PropertyChangeMask
   number of visuals:    270
   default visual id:  0x24
   visual:

Just lost a few more hours trying to fix this.
Both Weston and wlc have a 24-bit color depth for the root window and the WM window. Setting a 32-bit color depth for the WM window doesn't change anything.

Just a passing idea: Does it work if you set "mozilla.widget.use-argb-visuals" to true in firefox about:config?
https://github.com/mozilla/gecko-dev/blob/f51c4fa5d92d59fcb46f314e94edbf045cb3067c/widget/gtk/nsWindow.cpp#L3723
Firefox does some strange things...
https://github.com/mozilla/gecko-dev/blob/fd5c37f1dd9a0d1e327a6c6b4d81ea92f52c4330/layout/xul/nsMenuPopupFrame.cpp#L309

Doesn't appear to work for me.

Doesn't work for me.
But hey, wtf https://github.com/mozilla/gecko-dev/blob/fd5c37f1dd9a0d1e327a6c6b4d81ea92f52c4330/layout/xul/nsMenuPopupFrame.cpp#L311

#2902 is fixed but I have noticed another minor bug with text dragging. There's a 'white flash' for a brief moment. I wouldn't say it's worth chasing after unless you already know the cause.


Fixed mingw CI builds

These commits fix issues with the mingw builds when CI was upgraded to Ubuntu 20.04.

LGTM


"memory access out of bounds" after updating to 1.15.0

Overview
Hi, I was updating some modules in a repo for duckdb-wasm-examples and I found that updating @duckdb/duckdb-wasm from 1.14.4 to 1.15.0 broke the example's capability to read parts of the parquet file being used. The UI of the example allows one to display/filter entries by year, and when attempting to select past a certain year a "memory access out of bounds" error would pop up and no access to any of the data would work including those years that functioned previously.

Steps to reproduce the functioning example

Clone duckdb-wasm-examples/sveltekit-typescript
Run a clean install using npm ci. It must be clean so it installs duckdb-wasm 1.14.4 instead of skipping over to 1.15.0
Start the example using npm run dev and confirm that all years work (click around and see that the speeches load)

Steps to reproduce the broken example

Update duckdb-wasm to 1.15.0 using npm i @duckdb/duckdb-wasm
Start the page again using npm run dev
Click through some years: earlier than 1959 seem to still work fine but past 1959 tend to crash the example.

Please feel free to let me know if I should clarify anything or narrow down the steps. Thank you for any help you can offer!

Hi @jzavala-gonzalez! There have been many improvements to duckdb-wasm, currently version 1.24.0, and while the report was useful, it's now not worthwhile going back and investigate what might have happened moving between 1.14.0 and 1.15.0, so I would think it's OK to close this issue.
Thanks for raising this issue in the first place though, and hopefully give duckdb-wasm another go and keep being involved.


求适配Android studio 3.4.1

问题详细描述 Detailed description of the problem
求适配Android studio 3.4.1
复现问题步骤 Steps to reproduce the problem

其它重要信息 Other important information
replugin-host-lib/gradle Version:
rePlugin-plugin-lib/gradle Version:
Android API Version：
Android 手机型号&ROM(Phone model & ROM):
Logcat上下文 Logcat context

have you use replugin in androidx? i use in andriodx, i got "java.lang.ClassNotFoundException: Didn't find class "android.support.v4.content.LocalBroadcastManager" on path: DexPathList", i add dependencie "implementation 'androidx.legacy:legacy-support-v4:1.0.0'" also don't working. thks.


Can't cmd+z to undo in Find box after pressing cmd+f with some text in the editor selected

Steps:

Press cmd+f to open Find box
Type something in the find box. (If you press cmd+z now you will see that it undoes whatever you typed in the box)
Double click some text in the open editor and press cmd+f again. The text in the Find box should be replaced with the text you selected in the editor.
Now if you press cmd+z in the Find box it won't undo to the previous text before you pressed cmd+f.

Expected result: On step 4 I would expect that pressing cmd+z should undo the text to whatever was in the box before you pressed cmd+f. That is, this text history in the Find box shouldn't get reset each time you press cmd+f when you have some text selected in the editor.
Version: 1.85.1
Commit: 0ee08df0cf4527e40edc9aa28f4b5bd38bbff2b2
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
Embedder: vscode.dev

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!