[eslint] Rewired eslint is using wrong version of eslint and plugins

Thanks for this super-helpful project!  I really like how it's designed and how extensible it is.
I ran into an interesting issue.  I've found a work-around, but I'm opening this issue to see if this is something you'd like to have fixed here instead.
In my project, I want to use my own eslint configuration, which includes newer versions of some of the plugins that create-react-app uses.
When I run eslint directly, everything works fine.  But when CRA runs eslint as part of its webpack pipeline, it runs the version of eslint (and therefore its plugins) that are down in node_modules/react-scripts/node_modules.
My work-around is to use the undocumented overrides parameter from #122 to re-resolve the location of eslint being passed to eslint-loader relative to my project:
const rewireEslint = require("react-app-rewire-eslint");

const overrideOptions = options => {
  options.eslintPath = require.resolve("eslint");
};

module.exports = function override(config, env) {
  config = rewireEslint(config, env, overrideOptions);
  return config;
}

Is this something that should be done in this package instead?  Maybe with some protection to only do the replacement if there is a version of eslint installed locally?
If so, I can try to find some time to work on a PR for it.  At the very least, the override option needs to be documented.  I'll try to work on a PR for that as well.

Update: documentation PR is up: https://github.com/timarney/react-app-rewired/pull/216

Merged the PR (for the docs) - will wait it see if anyone else comments.  I don't use this package myself so would rather get thoughts from people who do.

Is this something that should be done in this package instead? Maybe with some protection to only do the replacement if there is a version of eslint installed locally?

@chrbala @dawnmist any thoughts?

I'm not sure if this is my issue or not.  I added this to overrides
 config = injectBabelPlugin('@babel/plugin-proposal-optional-chaining', config);
and changed a line to
return formatName(borrower?.name);
and I'm getting the compile error
`Failed to compile.
./src/application-summary/components/application-summary-sidebar.js
/Users/jaustill/git/mortgage-operator-ui/src/application-summary/components/application-summary-sidebar.js
55:33  error  Unexpected use of 'name'  no-restricted-globals
✖ 1 problem (1 error, 0 warnings)`
Is this related to the eslint version being wrong?  I'm using react-app-rewired@next and<EMAIL_ADDRESS>if that helps.

@jlaustill the next version of this package is stale at the moment while react-scripts does it's thing.  That is to say don't try using it yet.
@randycoulman seems this issue is also stale with no feedback from some folks who have been helping maintain that package.  Going to close and leave it at the documentation update that was merged.

@timarney Bummer, but thanks for taking the time to respond!


WIP: [DEMO] Added detection logger and event-based metric script for Smart Classroom

@snosov1 @LeonidBeynenson @sovrasov @grib0ed0v


Add stop-gap solution for e2es

Fixes

Part of #554 by @sarayourfriend
Description

Adds a step to run the e2e suite in the CI workflow. This isn't the ideal situation as it doesn't actually replicate the production environment because it doesn't have the wp.org wrapper. We can continue to explore a solution that accomodates that but for now this is better than nothing.
Testing Instructions

Let the CI run and verify that it works.
Checklist

[x] My pull request has a descriptive title (not a vague title like Update index.md).
[x] My pull request targets the default branch of the repository (main) or a parent feature branch.
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[N/A] I added or updated tests for the changes I made (if applicable).
[N/A] I added or updated documentation (if applicable).
[N/A] I tried running the project locally and verified that there are no visible errors.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

The tests are failing because I added a really hacky test for 'Meta search' sources. It checks for the number of buttons, and it wasn't updated when a new image source was added recently. So, it returns 6 instead of the previously-correct value of 5.
Also, I wonder if it's a good idea to merge this before adding the SSR mocking. Currently, we only mock the network requests on Client-side rendering, and anything rendered on the Server-side goes through the actual server, which takes a long time, and is flaky.

The tests currently fail also fail for me locally.

Also, I wonder if it's a good idea to merge this before adding the SSR mocking. Currently, we only mock the network requests on Client-side rendering, and anything rendered on the Server-side goes through the actual server, which takes a long time, and is flaky.

Ah, good point. We should do the mocking for the server side as well 😕

@obulat what do you think about us merging this as is for now and then opening another issue to investigate how to get Audio passing consistently (while also relying only on text on the screen that users will be able to interact with?)

I agree, @sarayourfriend


CB3S BK7231N Garage Door Opener ATHOM Homekit Tuya Smart WiFi Garage Door Opener Controller Siri Voice Control Works with Alexa Goggle - EU Plug

Hello, next dump for you:
garage_door_opener_CB3S.zip
This device:

Have fun, I personally don't need the profile, but I am dumping each device I document. Thanks!

An update has been pushed that should include a profile for this device.  Please update your repo and give it a try.  Please see the instructions page ( https://github.com/tuya-cloudcutter/tuya-cloudcutter/blob/main/INSTRUCTIONS.md ) for any command or input changes.
If the new profile does not work, please re-open this ticket with details (including logs) of the failure.
Any feedback is appreciated.


use forward slash to work in linux/osx

Sparks Jar ClassPath doesn't work in in linux because of back slash.  This fixes that.

Hi @shakdoesgithub, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

Current coverage is 80.91% (diff: 100%)

Merging #573 into master will decrease coverage by 0.08%

@@             master       #573   diff @@
==========================================
  Files            80         80
  Lines          7990       7990
  Methods        1330       1330
  Messages          0          0
  Branches        987        987
==========================================
- Hits           6472       6465     -7
- Misses         1150       1157     +7
  Partials        368        368

Powered by Codecov. Last update af71a32...c63db18

Thanks for your contribution @shakdoesgithub


[BUG] Incorrect TensorFlow Version in SMD 1.8 GPU

Category
Compatibility Issue
🐛 Describe the bug
Summary
: SMD 1.8.0+ has TensorFlow CPU installed instead of GPU
Replication steps

SageMaker Studio: Create JupyterLab private or shared space, use GPU instance type and above mentioned image and version.
Running the follow command for TensorFlow will show that there is no visible GPU devices

import tensorflow as tf
tf.config.list_physical_devices('GPU')

[]

pytorch will correctly show that CUDA is enabled.
running $conda list | grep tensorflow will show cpu build is installed

🐛 Describe the expected behavior

TensorFlow should list any GPU device passed to the container.
TensorFlow should be able to execute code using the GPU device.

Image Tags
1.8.0-gpu

Fixed in v1.9


Move info about IAP and TFServing to the docs website

Fixes #240
Kubeflow repo fix is in PR  #4760

This change is

/assign @sarahmaddox

Preview: https://deploy-preview-1666--competent-brattain-de2d6d.netlify.com/docs/components/serving/tfserving_new/#sending-prediction-request-through-ingress-and-iap
/lgtm
/approve
/hold
Thanks @janeman98 This looks good to me. I've applied a hold, in case we need to make more changes based on the review of PR https://github.com/kubeflow/kubeflow/pull/4760

/lgtm
/approve

/hold cancel

Thanks @sarahmaddox  and @jlewi !


feat: Contenido del curso

Signed-off-by: William Moreno<EMAIL_ADDRESS>

Codecov Report

Merging #6 (07c3018) into main (102c00f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #6   +/-   ##
=======================================
  Coverage   65.18%   65.18%
=======================================
  Files           3        3
  Lines         451      451
=======================================
  Hits          294      294
  Misses        157      157

Continue to review full report at Codecov.

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


Error 0xffffffff starting WSL from WT

Windows Terminal version
1.16.230126001
Windows build number
Microsoft Windows [Versione 10.0.19045.2788]
Other Software
No response
Steps to reproduce
I reported this also to WSL people #9875
I used WSL since it was added to W10 and followed its evolution migrating from WSL1 to WSL2 when it was added. I enabled systemd a few months ago and all worked just fine until today.
Now when I start WSL fro Windows Terminal (WT) it prints
[processo terminato con codice<PHONE_NUMBER> (0xffffffff)]

and in WT prints the message It is possible to configure termination... bla bla
I upgraded WT 1.16.230126001 on January from the msixbundle package and worked just fine
Now I get the above error: it occurs only when I start WSL from WT!!! If I start it using MinTTY, with a link (lnk) whose target is
C:\msys64\usr\bin\mintty.exe -c C:\msys64\home\user\.minttyrc -t "WSL Terminal" --position 20,450 /usr/bin/wslbridge2.exe -d Ubuntu -W "~"

WSL starts without error. As workaround, to start it from WT, I have to reboot but I can only start it once: if I close the Ubuntu tab and restart it the error reappears...
I did not change anything in the configuration: both in Windows and in WSL or WT. May be a Windows upgrade? there was one few days ago.. but until yesterday all worked.. I did also a windows cleaning (System - Archive - temp files etc..) but I worked with WSL up to March 29 21:47 (a timestamp of a file produced by TeXLive) so it is a mistery why now it does not work from WT but only with MinTTY...
Expected Behavior
WSL should start
Actual Behavior
WSL gives the above error

(I've edited your comment to fix the link, which was previously pointing at just "url".)

Yea, I'm pretty sure there was a change in the way WSL is delivered recently. Can you confirm if wsl --update works/?

It seems that something was corrupted in my system. Running
sfc /scannow
DISM /Online /Cleanup-Image /RestoreHealth

seems to have fixed the issue.
wsl --update works..
I am going to close this...


increase image sizes for better quality

Simple size changes, should be fine, but please see if it breaks some design patterns which I didn’t know about


Discussion: use psygnal events eventually?

🧰 Task
Over at https://github.com/napari/magicgui/pull/253, I have a PR that moves from using the events.py module (the same we have in napari) to psygnal (for reasons described below).  This is a task to discuss whether we should eventually do a similar conversion in napari, for what events, and what it would look like to transition.
difference between the event systems:
The EventEmitter that we currently used (borrowed from vispy) calls all callbacks with a single event object, where the attributes of that object are set dynamically based on the emitter
events.py pattern
def my_callback(event: 'napari.utils.events.Event'):
    # do something with event...
    xyz = event.value  # our convention

layer.events.data.connect(my_callback)
layer.events.data(value=new_val)

psygnal pattern
def my_callback(new_val: int):
    xyz = new_val

layer.events.data.connect(my_callback)
layer.events.data.emit(new_val)

why would we change?
In my opinion, the primary difficulties with events.py is that

the event object is a bit opaque for the "consumer" (it's an object that napari users deal with a lot):  It's not an object they encounter elsewhere and it's a hard object to inspect.  So they just "have to learn"...
it is so dynamic (lots of setattr, and dict keys): it's not clear what attributes the event will have, (we have to just enforce conventions like "event.value" that everyone has to learn).  The specific emitters in an EmitterGroup (like layer.events.data) are also set dynamically, so you can't infer much about what events will be found in there until runtime.  That would be fine if events themselves were more "dynamic"... but our list of emitters is rather static, with constant signatures.  This also makes it hard to provide good IDE typing and autocompletion support.
It's a "second" API that anyone new to the codebase has to learn (in addition to Qt signals/slots which many are already familiar with).

psygnal
psygnal implements the Qt signals & slots API (its's slight superset) in pure python ... so a number of people will already know how to use it (or at least not have to learn two different ways).  It provides nice type safety and autocompletion, like a regular qt signal would.  And it directly gives callbacks the value(s) emitted by the emitter (instead of an Event object).  Like Qt, it can also inspect callbacks and not provide arguments if the callback doesn't want them...  which removes the need for a lot of lambdas to swallow events (connect(lambda e: do_it())).  Generally, I think it's easier for end-users to learn, devs to work with, and docs to generate.
where would we change?
Generally, I think the psygnal pattern (providing callbacks with values not events) is more appropriate in places where Qt would have used signals & slots instead of QEvents.  For instance: widgets that have a valueChanged, or generally simple changes of state that other components need to know about, with ~0-2 parameters.  (This is the majority of our usage, including the EventedModel stuff).
On the other hand, UI events like mouse events and key events (things that have a lot of event parameters), or any happening that has a "QEvent" equivalent is a good candidate to stay with the current system.  (These are the vast majority of the events that vispy has, so it makes sense that this module came from there).  These events also tend to be something that end users don't deal with as much (though not never).  I don't think it would be too confusing to have both systems, with "mouse/key events" being a special object that someone could learn, but all the rest of the state-change emitters being psygnal style.
how would we change?
If you're curious what a deprecation pattern might look like, you can read the proposal at https://github.com/napari/magicgui/pull/253.  While the two systems can't literally overlap, i think it's possible to "warn" rather than "break" the vast majority of usage: my deprecation over there uses a named tuple to "mimic" the current Event object, and uses callback type annotations or parameter number to determine whether it's a "new-style" callback or not (and if it's unclear, the callback gets the old event object).

definitely in favour of a general move towards psygnal/Qt style events!
I agree that keeping a distinction between mouse/key events and state-change emitters currently will get us a long way very quickly - slightly related, longer term I think @nclack suggested that we simplify some aspects of the mouse events, especially with the 3d transforms (originally added in #3037) which may make the distinction slightly smaller 🙂


Rename MachineGraphSupport?

The new mixin architecture is great (as is the associated table in the README), but is there a reason MachineGraphSupport doesn't follow the rest of the nomenclature? It seems like it should be called GraphMachine for consistency. Any objections to renaming this class?

If I am not mistaken, @wtgee has chosen this name because the first version did not inherit from Machine (but I am not 100% sure). But since it does now, naming it GraphMachine sounds good to me.

:+1:


Add port for RTEMS

I am currently porting the rust compiler and stdlib to the RTEMS realtime operating system (https://www.rtems.org/).
This PR is the prerequisite for my changes to the rust repo.
style.sh succeeds, the libc-test fails to compile on my machine, but since I only add things which are behind guards for target_os = "rtems". I hope CI will give a 👍 .
Once this passed. I will also add a PR for main branch.

Thanks! Could you add the target to rustc first? This crate follows rustc's platform support for adding/removing targets.

Ah, ok. Will do.

This looks pretty reasonable for me after https://github.com/rust-lang/rust/pull/127021 merges. Could you target main? I've been doing backports in batches from there.
Also, are headers / docs for the added API available? Trying to get those crosslinked on all PRs so there is always a reference.
@rustbot label +stable-nominated

This looks pretty reasonable for me after rust-lang/rust#127021 merges. Could you target main? I've been doing backports in batches from there.

@tgross35  I opened a new PR https://github.com/rust-lang/libc/pull/3866 for this as I couldn't find a way to change the target branch of this PR. I also added some more reference material in the descriptions over there.

@tgross35 I opened a new PR #3866 for this as I couldn't find a way to change the target branch of this PR. I also added some more reference material in the descriptions over there.

You would need to rebase (locally), force push, then click the "edit" button to the right of the title and change the dropdown. But no worries, a new PR works too.
I'll close this one so there isn't a review in two places.


Serial port disconnect + connect don't reset Arduino

Issue Type: Bug
When doing 'disconnect' then 'connect' on the Serial port, the attached Arduino Nano (FTDI) does not reset.
When doing the same with the serial port from a C# program it is possible to reset the Arduino by controlling the DTR line.
Seems like the VS-Code serial port controls does not control DTR line correctly.
Maybe it should be an option to be able to select if DTR and/or RTS should be controlled when disconnecting/connecting the Serial port.
In my mind, this is not caused the Arduino plugin, as the serial port is a VS-Code feature.
Preferrebly a setting to enable DTR/RTS control should be a project/workspace settting.
 Please also check if it is already covered by an existing one, like:

Select Serial Port not allowing to select Serial Port (#123603)

Not the same issue...

/causedByExtension
Arduino support comes from an extension you installed. Your issue needs to be investigated by the extension author.

Issue created in Arduino extension issues:
See vscode-arduino/issues/1285


PermissionError: [WinError 10013]

While trying to run the .exe version on a local machine, there's following error and app stops running.
PermissionError: [WinError 10013] An attempt was made to access a socket in a way forbidden by its access permissions

I believe this could be due to your firewall, antivirus or possibly admin rights (privileges/permissions).
As this .exe runs a python web server on your machine, it will need administrator rights and port 8501 should also be open.


Bug: Deletion in the wrong cluster through delay

Reproduce:

click on delete secret
switch to different cluster which has a secret with the same namespace and name
wait for delay to end
secret in switched cluster is gone

Hey @farodin91 thanks for bringing this to notice. May I know how long is the delay?

Oh damn. The delay doesn't really matter. I guess the problem is that we fetch the current cluster at the end of the delay, and if that has changed, then it'll use the different cluster 😵💫
We can add a way to prevent this right now (make it fail the deletion if the cluster was changed) until we have #1591 merged in as the latter is needed for a proper fix.


Config file error : PyYAML yaml.load(input) Deprecation

Error at line 18 in config.py
params = yaml.load(default_params_yaml)
Error:
local/lib/python2.7/site-packages/ILCC-0.2.dev0-py2.7.egg/ILCC/config.py:18: YAMLLoadWarning:
*** Calling yaml.load() without Loader=... is deprecated.
*** The default Loader is unsafe.
*** Please read https://msg.pyyaml.org/load for full details.
params = yaml.load(default_params_yaml)
https://github.com/yaml/pyyaml/wiki/PyYAML-yaml.load(input)-Deprecation

Changing to yaml.safe_load(default_params_yaml) works!
Should also be able to change to any of-
yaml.safe_load
yaml.full_load
yaml.unsafe_load


Can't download latest Ventura

Ventura was updated to Version 13.0 Beta 10 (22A5365d), and it's not available for download.

Mist relies on the Apple Software Update Catalogs (SUCatalog) for listing / downloading macOS release.
See https://github.com/ninxsoft/Mist/issues/12 for more info 👍

macOS Ventura 13.0 (22A5365d) is now available via Mist:

That is not beta 10.
On Sat, Oct 8, 2022 at 2:04 AM Nindi Gill @.***> wrote:

Closed #13 https://github.com/ninxsoft/Mist/issues/13 as completed.
—
Reply to this email directly, view it on GitHub
https://github.com/ninxsoft/Mist/issues/13#event-7547590017, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AASVPR2HYBLNRYW4HZPJ4CLWCEMI5ANCNFSM6AAAAAAQ44PLWY
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Gilbert J. Palau
@.***
http://gilbertpalau.com


fix issue with logrus capitalization

logrus changed their GitHub repo to all lowercase and broke all go packages that use their library.
See here and here

merging now, feel free to comment

LGTM, thx! Don't forget to clean up your branches after merging :)


Add -Wall to compiler flags

Also add -Wextra, where available. The warning flags are added to CMAKE_<LANG>_FLAGS so that one gets warnings also in release builds. 'Cause life is too short to deactivate or ignore compiler warnings!

This is good initiative. Let us make 'rule' that by default we switch on all compilers warnings.

Thank you! Very good path forward.


[QUESTION] Access To Matterport3D dataset

Hello!
Thank you so much for your wonderful work, I really learned a lot from your open-sourced project, now I want to run Hydra in Vision-Language-Navigation dataset (Matterport3D), and I noticed that in your IJRR paper, you mentioned that you have used the MP3D for generating Scene Graph for different scenes, which is shown as follows,

And I also find the MP3D-related launch and .yaml configuration file in your open-sourced Hydra code, so the question I want to ask you, is that can you provide the relevant toolbox to make Habitat Simulator compatible with ROS system, or some batch-processing program file to make habitat-sim work with ROS, if possible, could you please share the MP3D rosbag public? Thank you so much!
I'm waiting for your reply, Thank you so much!

Hi, thanks for your interest in our work! With our recent release, we've included the python bindings and habitat interface for running with MP3D, which can be found here. Please note that this interface is not the most stable, so there may be issues.


os: update vmware guide

Users should be using the standard VMware images. This suggests that
they use the OVF image so it can be directly imported into ESXi.

I haven't yet tested this guide.

Text looks good to me other than the tab question.


Online Users Not Accurate

When users disconnect/join the online users often gets glitched and does not display the correct users.

I believe this is caused by the double socket connect. When a socket disconnects, it removes two entries

Resolved. Socket will only remove from the users array if it's exact name is in the array.


Is Flynn in active development / maintenance only?

I'm curious to know if the Flynn project is in maintenance-only mode.
It seems that the roadmap has stalled:
https://github.com/flynn/flynn/projects/1
The two "ongoing" issues #1995 and #267 have been in progress for years.
Nearly every datastore (mariadb, redis, mongo) Flynn ships with is outdated with no documentation on how to self-upgrade. This is in addition to not supporting Ubuntu 18.04.
There additionally has been no "stable" release for over a year. I know that the  plan [#4319] is to switch to nightly-only channel releases, but that was never done. A short blurb to use the nightly channel could have been added to the installation docs/README.
A lot of PRs from the nightly builds don't have an associated issue — so I'm running under the assumption that Flynn is still used internally, but no longer a focus?
Flynn is a great project and while it would be unfortunate to hear that it's in maintenance mode I think the roadmap + its future should be clearly stated.

Good point describing our situation here as well.

Every now and then there seems to be a push in development for a few days which then stops. Like switching to gRPC for the internal APIs (https://github.com/flynn/flynn/pull/4307), the (nice) Prometheus metrics with Grafana (https://github.com/flynn/flynn/pull/4373) or some Docker refactoring (https://github.com/flynn/flynn/pull/4367) and a branch where LetsEncrypt support was  visible (https://github.com/flynn/flynn/tree/lets-encrypt).
Some CI improvements (https://github.com/flynn/flynn/pull/4425) give a bit of hope that the infrastructure gets some maintenance as preparation for further development though. :)
All in all we stick with Flynn as it still fullfils our needs and there is nothing better for us yet.

We use Flynn extensively in production, but haven't had a lot of development bandwidth to push features forward. That said, we do have a variety of projects in progress (including the PRs that @philiplb) that will be released in the coming months. Thanks for using Flynn!

@titanous Thanks for the reply! Have you thought about allowing financial contributions to the project(i.e opencollective)?
I appreciate that opensource dev offers 0 monetary incentive for Flynn since you have a paid support. But I imagine if the community at least provided something to offset the costs Prime Directive may focus on it a bit more.


stdlib: implement POSIXErrorCode for FreeBSD

It shares many of the same values as OpenBSD (which both inherited from 4.4BSD), and we wouldn't expect those to diverge due to the respective platforms' bincompat goals.

@swift-ci please smoke test


connectivity: remove Warn(f) from Test and Action

This proved a bit difficult to use in practice. With output of successful test runs now being hidden in CI, things need to fail to pop up. We cannot have things failing and throwing warnings without anyone noticing.
Failed finalizers will now cause an error. Finalizers are not supposed to fail, at least not silently.
Errors polling the Hubble buffer now also fail the test, but this behaviour is already being removed in the open Hubble streaming PR. (#319)
Note: ConnectivityTest still has Warn(f), but it's purely cosmetical and not integrated into the rest of the test suite.
This is an alternative proposal to #327.

rebasing to pick up https://github.com/cilium/cilium-cli/pull/356

hey @jrajahalme could you take a look

resolved conflicts

ok i'm merging this


NavigationView: Close pane in non Left displaymode when selecting an item from AutoSuggestBox

Description

Listening to QuerySubmitted event of AutoSuggestBox and closes pane when not in DisplayMode left
Motivation and Context

Closes #3241
How Has This Been Tested?

Add interaction test.
Screenshots (if appropriate):

/azp run

@chingucoding this looks like a real test failure to me

@StephenLPeters Test failure should be fixed now. @ranjeshj I added a check to ensure we only close the pane if the selected suggestion is not null, if there are any precautions we should take there please let me know.

@chingucoding can you address the merge conflict as well?

Oh right, totally missed. Merged with master now.

Looks like an unlucky unreliable tests.

/azp run


Should it works with Qlima SC5248 ?

May be , it s me  or it s not compatible.
I'm not an expert with ESP module but I think I ve done the correct setup.
I bought : https://www.amazon.fr/gp/product/B07Y8JZNCQ/ref=ppx_yo_dt_b_asin_title_o01_s00?ie=UTF8&psc=1
i've flash with esp-link V2.2.3  ( and 3.0.14 to test)
I can connect to the esp-link web page and configure wifi and other params (perhaps it s something there)
but , after that , I think i 'skip' something.
if I run the serialbridge sample (and yes I put my ip and port ;) ) and go step by step in visual studio code
the connect step is ok , but the following operation catch error "No response receive".

If I look at the "Microcontroller Console" on the esp web interface, nothing appears.
and in the debug console , I get "286732> Accept port 23, conn=0x3fff7580, pool slot 0"

Are you sure the TX and RX pins are not switched?

I ve try the option « swap » uart pins in the pin assignment screen.
May be the usb connector is not soldered how it should to work with ac ?
But , I think if it was not correct , the esp should not boot and works when I plug in ac usb slot

I do not understand what you are trying to say.

This will bot work swapping means using complete different pins but the levelshifter adapter is connect to specific pins. What I meant is did you try to switch the rx and tx pins, so where you connected rx now connect tx and vice versa.

I use a usb to esp bought on Amazon

And , if I monitor usb on my computer, i can send and receive in esp-link console.
I bough this module because I'm not very good at doing electronics

That does not work. Although your AC unit has an usb connector it is not a usb interface, it is just a serial ttl interface.

That's what I deduced from your previous comments, I ordered the model you show on GitHub.
Thx

All works perfectly now. with the correct shifter and usb pin assignment.
Thx again ;)
Now I should look at the code.


Feat: Add option to search by singular labels

Description

I use Payload in my native language. My clients use the CMS in their native language, therefore I have different slug and singular label. It's currently not possible to search blocks by their singular label. That's why I created this commit. I don't think it's needed to search by plural form, as a block name is typically displayed in singular form.
One possible improvement could be to search only based on a chosen language. Let me know what you think about it.
I used filter and includes for better readability as the performance difference is neglectable (mainly with low number of records ie. blocks).

[x] I have read and understand the CONTRIBUTING.md document in this repository

Type of change

[x] New feature (non-breaking change which adds functionality)

Checklist:

[ ] I have added tests that prove my fix is effective or that my feature works
[x] Existing test suite passes locally with my changes
[ ] I have made corresponding changes to the documentation

Hey @choutkamartin thanks for this!
Made a couple changes to reduce complexity. Now we are just searching on the singular labels of the blocks instead of the slugs/singular-labels/plural-labels.
💥

@JarrodMFlesch Thanks! Tested it and it works just fine! 👍


Finch cannot resolve special IP host-gateway

Describe the bug
Finch cannot resolve special IP host-gateway in finch run --add-host name:host-gateway IMAGE
Steps to reproduce
finch run --add-host name:host-gateway -it public.ecr.aws/amazonlinux/amazonlinux:2 bash
# cat /etc/hosts
...
host-gateway    name
...

Expected behavior
finch run --add-host name:host-gateway -it public.ecr.aws/amazonlinux/amazonlinux:2 bash
# cat /etc/hosts
...
<resolved-ip>    name
...

Screenshots or logs
Additional context
In Docker,
docker run --add-host name:host-gateway -it public.ecr.aws/amazonlinux/amazonlinux:2 bash
# cat /etc/hosts
...
<IP_ADDRESS>	name
...

Can reproduce in nerdctl in Finch VM:
nerdctl run --add-host name:host-gateway -it public.ecr.aws/amazonlinux/amazonlinux:2 bash
# cat/etc/hosts
...
host-gateway    name
...

In Finch, to make the feature parity with Docker, the ideal user stories will be:
As a Finch user, if I use host-gateway in my finch run with configuring host-gateway-ip in Finch config, when I try to access host with the alias in container, it will use the ip I configured in Finch config.
cat ~/.finch/finch.yaml
...
host-gateway-ip: "<IP_ADDRESS>"
...

finch run --add-host name:host-gateway -it public.ecr.aws/amazonlinux/amazonlinux:2 bash
# cat /etc/hosts
...
<IP_ADDRESS>    name
...

## curl name:<port> with a valid port will succeed if <IP_ADDRESS> is a valid host gateway ip

As a Finch user, if I use host-gateway in my finch run without configuring host-gateway-ip in Finch config, my container can still use the IP alias to access host.
finch run --add-host name:host-gateway -it public.ecr.aws/amazonlinux/amazonlinux:2 bash
# cat /etc/hosts
...
<default-ip>    name
...
## curl name:<port> with a valid port will succeed

The solution could be split to multiple stages:
At the first stage, we can resolve host-gateway to a static IP without being configurable.
Lima has a static host IP <IP_ADDRESS> that can be accessed in containers. The value can be retrieved from code. We can directly resolve host-gateway to it before passing to nerdctl. The value retrieved by Finch will change if Lima changes the value. E2e tests will also be added to validate the ip equals to the value and it can be used to access host.
Making host-gateway-ip configurable will come later.

The first stage is already completed. The next thing to do it to make it configurable in Finch config for feature parity. It requires the nerdctl change to be promoted to Finch.

Hi @ningziwen, I want to work on this issue. Can you guide me around it?

@TrungBui59
Sure, thanks for the interest! Assigned the issue to you. Feel free to raise any questions directly in the issue. I will be actively monitoring it and helping you.

Thank you very much, I am kind of clueless, so can you give me a rough idea on how I should tackle the issue?

@TrungBui59 Yes. You may follow these steps to start working on this.

Understand what is Finch overall, and read the code to understand how Finch uses Cobra as CLI tool and how Finch interacts with Lima and Nerdctl.
Understand what is this issue for from the description. (basically for docker feature parity)
Read the existing converation and PRs to understand current workaround in Finch, and the existing fix in Nerdctl.
Think how to use the existing Nerdctl fix to resolve the issue in Finch and remove the workaround.

@ningziwen I see. Thank you

@ningziwen

I thought this is the expected behavior?

@ningziwen Just to make sure my solution is acceptable. I will add a new field called host-gateway-ip on the finch.yaml. I will then parse by changing the pkg/config.go file and then get the data and pass to the SlirpGateway. Then Im gonna use it when user specify the ip

My understanding is SlirpGateway IP is a constant value in Lima, which is not designed to be configure.
This nerdctl change introduced a host-gateway-ip field in nerdctl config, which is designed to be configurable. https://github.com/containerd/nerdctl/pull/1978/files
So my suggestion is if host-gateway-ip exists in finch config, pass it to host-gateway-ip in nerdctl config. Otherwise, pass SlirpGatewayIP to host-gateway-ip in nerdctl config.
Let me know if you have different thoughts.

@ningziwen  I have some issue with the tests. Unit tests keep giving me race condition on file I dont even touch. and e2e test is output something that I dont quite understand
e2e:

unittest:

@ningziwen so for the e2e test, what does that error mean?

@ningziwen  This is all it shown me. It even shows this before I add some code change

@TrungBui59 For e2e tests, most of the Finch e2e tests are in common-tests repo. And here is the failing test scenario from your log. https://github.com/runfinch/common-tests/blob/6c7275007bf34fb6ddecc4013c16f1d79ff6d1d0/tests/run.go#L335
Finch e2e tests use a framework called Ginkgo.
The tests are failing because your change broke them. You also need to scroll up to find the failed scenario and get more useful logs from there.

@ningziwen I have finished the task, but there is one question I want to make sure. If the user update the host_gateway_ip in the finch config file, do we only allow to update it when they init the VM or they can update when they start the VM?

@TrungBui59
We think we should try to allow them update when they start (restart) the VM.
Why close the PR?

@ningziwen Got it. For the PR, it was an accident, I type a command and somehow it closed the PR

@ningziwen sorry for not really finishing earlier, I was quite busy with my work. Is this issue still open? If it is, I will be able to finish this by the couple next days

@ningziwen sorry for not really finishing earlier, I was quite busy with my work. Is this issue still open? If it is, I will be able to finish this by the couple next days

@TrungBui59 No worries. It is still open. Feel free to complete it.

Retested this, seems this has been implemented:
<IP_ADDRESS>     name


U/sschmidt23/mwdust

Creating a new PR since the one in #137 did not include these two latest commits.  As stated, this PR switches the catalog from 2.2i_dr3 to 2.2i_dr6a and switches from a pre-fetched parquet file to a query of the star truth GCRCatalogs data.
I also added a separate very short notebook on calculating A/E(B-V) that might be helpful, or may just be too much detail.  Not sure whether this needs to be included in the contributed notebooks or not.

Ok, I made the recommended changes (and removed a line where I force overwrote the filter set  with ugrizy regardless of function input).  I also copied the appropriate DC2 throughputs to the data subdirectory.  Things should work well now.


Add Tests for Engine API Wrapper

Test:

Import of jwt.hex / verification of secret
Testing different calls to different endpoints with different params
Verifying endpoint responses

This could be done in a manner similar to how ethers-rs does.


better json serializer appending

By using the jsoner package we can append to JSON objects in a file
without having to load the whole file. This removes the need for those
various options to handle maximum number of points and maximum filesize.

@davidvgalbraith @demmer wanted to add the -format to file adapter but before I can do that I think we should clean the file adapter to do "better" writing of JSON format that doesn't include having to read the whole file into memory in order to append. I used a package I developed on my own free time and then tweaked it a bit just now to make it more friendly for our use jsoner. Let me know what you think but this removes limitations of writing out large JSON files.
With this change I can then follow up with adding the -format option so we can write jsonl and csv and also expose an -append option to dictate if you want to append to the existing file or simply truncate the file which. Since JSON would have this existing append mechanism and if you were not appending we could rely on the existing serializer/json.js to take care of business.
@dmehra with this you wouldn't need to output the data in jsonl and could just continue to use JSON.

@rlgomes I was also thinking that it would make more sense to have an -append option that would default to false since I suspect that many cases of using the file adapter would actually be more inclined to overwrite as opposed to append.
As far as jsoner goes, I took a quick look and the biggest hesitation I would have is that you seem to only expose a blocking API and we definitely want to keep all our adapter I/O asynchronous.

@demmer I was thinking about exposing a non blocking ;) just took the "easy" route when writing this to see if would even work.

+1 to append being not the default behavior, it has surprised me a few times when i didn't realize i already had an output file with the same name, and ended up with huge concatenations. It's great to have the option to append, but the default be replace.

@davidvgalbraith @demmer updated and ready for your review

bump!

jsoner.append still has calls to statSync and writeFileSync and openSync and closeSync, can we async the whole lot of it?

You are correct I will fix that all up..
On Tue, Jan 26, 2016, 22:05 Dave Galbraith<EMAIL_ADDRESS>wrote:

jsoner.append still has calls to statSync and writeFileSync and openSync
and closeSync, can we async the whole lot of it?
—
Reply to this email directly or view it on GitHub
https://github.com/juttle/juttle/pull/257#issuecomment-175421190.

@davidvgalbraith fixed up the stray synchronous fs calls.

@davidvgalbraith @demmer so how about I make a PR that basically removes the appending behavior when doing write file and simply exposes -format with the default of json which truncates and uses the available serializers to write out data (just like write stdio does). Then we can add the -append option for formats that support this easily such as jsonl and possibly 'csv'. ? Then I can simply not do any of this magic with the jsoner library and we can add that back if someone REALLY wants to do appending to a JSON file.

moved to simply rip out the whole append behaviour: https://github.com/juttle/juttle/pull/280


The blocks were retrieved using omni_getseedblocks and formatted accordingly for seedblocks.cpp.
This PR was tested & can be verified by running a full reparse (--startclean) with this updated seed block list and confirming that it passes the checkpoints here https://github.com/OmniLayer/omnicore/pull/371

Sorry for the delay, and thanks! :)


Asset Automation 2024-02-20T12:13:23.159Z

feat(assets): Asset Automation 2024-02-20T12:13:23.159Z

test pr


Should as_ptr_or_null really convert empty slices to null?

Context
I'm currently trying to create an application that uses windows graphics capture.
This apparently requires that I use the D3D11CreateDevice function: alexchandel/screenshot-rs
According to the C++ API, this function accepts pFeatureLevels as a pointer (to an array) : D3D11CreateDevice C++ API
HRESULT D3D11CreateDevice(
  [in, optional]  IDXGIAdapter            *pAdapter,
                  D3D_DRIVER_TYPE         DriverType,
                  HMODULE                 Software,
                  UINT                    Flags,
  [in, optional]  const D3D_FEATURE_LEVEL *pFeatureLevels,
                  UINT                    FeatureLevels,
                  UINT                    SDKVersion,
  [out, optional] ID3D11Device            **ppDevice,
  [out, optional] D3D_FEATURE_LEVEL       *pFeatureLevel,
  [out, optional] ID3D11DeviceContext     **ppImmediateContext
);

If the pointer is null then it will use some system defined default set of features (this is what I want).
Problem
However in rust this function currently only takes pFeatureLevels as a slice: D3D11CreateDevice Rust binding (View raw Line 7)
It's not intuitively clear how to set it to null.
I even tried std::ptr::null() and *std::ptr::null() as arguments. But the former doesn't compile, the latter gives unexpected behaviour.
Digging into the code, I realized the slice was being converted to a ptr for the C API using as_ptr_or_null:  D3D11CreateDevice Rust binding (View raw Line 14)
And as_ptr_or_null is defined as:
https://github.com/microsoft/windows-rs/blob/3376a148f56c1de3aff2d7a000ec9941dec3e68f/crates/libs/windows/src/core/mod.rs#L103-L109
So according to this if I want to pass a nullptr to the underlying C function, I just have to pass an empty array to the rust function. Yay problem solved?? Yes but....
What if the intention was to pass in an empty array of features? There would be no way to do that from rust because that would be interpreted as null and a null value triggers the default feature list? Since as_ptr_or_null is a core function... I imagine there are other functions that use it this way.
Solution?
Wouldnt it be better to use something like Option<&[T]> for array arguments than can also be null?

Does D3D11CreateDevice actually behave differently in C++ depending on whether you pass an empty list or a null pointer? Just trying to understand whether this is an actual limitation or just a hypothetical question.

I made a small test program.
// main.cpp
#include <d3d11.h>
#include <iostream>
#include <stdlib.h>

using namespace std;

bool InitializeD3D11(const D3D_FEATURE_LEVEL *levels, int lvl_count) {
	D3D_FEATURE_LEVEL valid_feature_level;

	IDXGIAdapter *adapter = nullptr;
	ID3D11Device *dev = nullptr;
	ID3D11DeviceContext *ctx = nullptr;
	HRESULT hr = D3D11CreateDevice(adapter, D3D_DRIVER_TYPE_HARDWARE, nullptr, 0,
			levels, lvl_count, D3D11_SDK_VERSION, &dev,
			&valid_feature_level, &ctx);

	if (dev) return true;
	return false;
};

int main() {

	cout << "D3D11 with feature_levels = nullptr."
		<< "\n\tSuccess: "
		<< InitializeD3D11(nullptr, 0) << "\n";

	D3D_FEATURE_LEVEL empty[] = {};
	cout << "D3D11 with feature_levels = {}. "
		<< "\n\tSuccess: "
		<< InitializeD3D11(empty, 0) << "\n";

	D3D_FEATURE_LEVEL feature_levels[] = {
		D3D_FEATURE_LEVEL_11_1,
		D3D_FEATURE_LEVEL_11_0,
	};
	cout << "D3D11 with feature_levels = "
		<< "{D3D_FEATURE_LEVEL_11_1, D3D_FEATURE_LEVEL_11_0}."
		<< "\n\tSuccess: "
		<< InitializeD3D11(feature_levels, std::size(feature_levels)) << "\n";
}

~/dev/temp/d3d11test$ g++ main.cpp -o d3d11test -ld3d11
~/dev/temp/d3d11test$ ./d3d11test.exe
D3D11 with feature_levels = nullptr.
        Success: 1
D3D11 with feature_levels = {}.
        Success: 0
D3D11 with feature_levels = {D3D_FEATURE_LEVEL_11_1, D3D_FEATURE_LEVEL_11_0}.
        Success: 1

Turns out no device is created if an empty array is passed it. So in this case I suppose as_ptr_or_null is harmless.
But there is still the issue that the rust binding behaves slightly different from official documentation and there is no way to know how to pass a "null" unless you find the rust definition of the function.

In practice, it has been quite challenging to find a happy medium between the APIs and the metadata when it comes to array parameters. I've already scaled back the win32 array support substantially. If you find an API where the bindings don't allow the API to be used correctly, do let us know.

Sounds good. I'll be sure to make an issue if I run into any blockers. Thanks for all the awesome work and your speedy reply.
I'll go ahead and close this since my question has been answered.


Ft values exceeding 30 days 145028385

What does this PR do?
As a user I want to be able configure long term checks for checks longer than 30 days
As a user I want to be able to configure daily, weekly and monthly checks
Description of Task to be completed?
GIVEN that as a user I want to set up a new job
WHEN selecting a grace period for that job
THEN I should have the option of setting a grace period value of more than 30 days
How should this be manually tested?
Clone the repository
cd into the repository
run the command ./manage.py runserver
-start server and run the app
Screenshots (if appropriate)

Coverage remained the same at 81.067% when pulling 6af634c18ffd94c23fef31bb824e490b8349c3a1 on ft-values-exceeding-30-days-145028385 into d66f387b0db94f24bdbd680160a6038560e81e38 on develop.

Coverage remained the same at 81.067% when pulling ed42beb038495067fc212005b06679a50b061207 on ft-values-exceeding-30-days-145028385 into d66f387b0db94f24bdbd680160a6038560e81e38 on develop.

Coverage remained the same at 81.067% when pulling ed42beb038495067fc212005b06679a50b061207 on ft-values-exceeding-30-days-145028385 into d66f387b0db94f24bdbd680160a6038560e81e38 on develop.


Add recipe re duplicity for backup

Done, at https://geek-cookbook.funkypenguin.co.nz/recipies/duplicity/


Actually make auth tokens longer (40 chars)

Fixes #2724
Both reset password token and auth token are 40 chars long.
Database is changed to take 255 long token (for the future)
Auth token are generated on demand. Deactivating the remote login deletes the token. Renabling it creates a new token.

Yes @michael-e there are no hurry to merge it anyways. I thought it would be small.. but you know how it goes ;)

Oh yes, I know. Features are like icebergs. You only see a small part…

Thanks @michael-e !


Escape characters when using knex.raw

I know this is probably a question that has been answered before but I was looking into old issues and still haven't found the right solution for my case.
I'm using knex.raw to call a function I created on my postgres db. The thing is that I pass some parameters with special characters which I need to escape.
Is there an easy way doing that using knex?
Thanks

Which are those special characters? Probably you should check out how underlying node-postgres driver handles them.
Some escaping stuff done in knex side is found here: https://github.com/tgriesser/knex/blob/2e1a459a9e740f24b9a4647bd4da427854e551dd/src/query/string.js#L4

Special characters such as $.
I know there are inner functions that deal with it but I was looking for a function I can use straight from knex API.
I was using it to pass hashed password but now I just transform the hashed password to base64 string so I don't have this problem no more.

I am also having the same problem, how do I store  special characters in the db with  knex.raw ?

you should be able to use $ without any tricks:
> knex('$table$').select('$column$').where('$something$', '$').toSQL()
{ method: 'select',
  options: {},
  timeout: false,
  cancelOnTimeout: false,
  bindings: [ '$' ],
  __knexQueryUid: '7d88f4b0-cff2-461b-a799-bf5b2fc43c29',
  sql: 'select "$column$" from "$table$" where "$something$" = ?' }
>

When passing string the contains $ to a function in postgres using knex.raw there's a problem

@roytz could you add an example code which shows the problem to check it out?

@elhigu
Still having this issue. I'm trying to pass parameter with $.
Works
knex().raw( "select * from users where id = :_user_id", { _user_id: user?.id || null });

Doesn't work
knex().raw( "select * from users where id = :$user_id", { $user_id: user?.id || null });

Yeah, probably regular expression there requires that named replacement is alphanumerical or something like that.


Removed duplicate content from ReadMe.

Description of the change: Removed duplicate content from ReadMe.

@ksigler7

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

hi @vigzmv - thanks so much for these updates! We are not currently accepting major changes to our readme because we are in the process of rewriting it based on feedback we have gotten from our contributors during Hacktoberfest. We really appreciate the work you put in, though, so please check back to our open issues often, as we are constantly adding new ones!


Union Lens Feature

Added support for the UnionLens lens type.
3 additional fields:
unionRelations: A set of relations to compute the union of.
makeDistinct: Boolean value determining if result should be made distinct.
provenanceColumn: If provided, adds an additional column in which the source location of each data row is included.
Also added high-level tests:

An example usage scenario, testing if provenance columns are included correctly and if all items are included in the union.

And low-level tests:

Unions of two or more tables or lenses are performed correctly
"Negative" test cases of union lenses in a format that is not supported (i.e. attributes have different types/names)
Test cases on the inference of unique constraints (if all union tables have a unique constraint on an attribute and there exists a provenance column, then the full union has one too)
Other added integrity constraints

Thanks Damian!


Allow to hide Vite action icon

I've noticed that the setting volar.icon.vite was removed since https://github.com/johnsoncodehk/volar/commit/4e138d44a9758efe6dc256919a7e78a9de33bc09 . Is it intentional?
It would be better that allows us to hide those icons to keep the editor simple and neat.

I don't know someone needs to disable it. 😅
I think the go to code feature on preview is very useful for development, is it not suitable for you?

Yes, I always use browsers to develop and never open preview in VS Code, so I disabled this.

Preview is not the purpose of volar, do you know the go to code feature? https://twitter.com/johnsoncodehk/status/1496978502385422336

While "Go to Code" seems good, but I won't use a embedded WebView to debug and develop. Of course, they're disadvantages of WebView, not Volar itself, but at least it prevents me from using this feature.

Preview window is created for go to code and component preview features, not for development and debugging. In my experience, go to code is almost a necessary feature to find the code that needs to be modifiy. Then you can close the preview window and continue in browser development.
I want to minimize volar settings so I just wanted to make sure you don't need go to code feature at all in development so still want to hide the icon?

@fardolieri Of course with authentication I will add this setting, but authentication is no need any more after 0.33.0...

you don't need go to code feature at all in development

For me, the answer is yes.

I agree @fardolieri 's opinion:

please don't force it on us by removing the option to hide the icon.

An extension for public should be able to be customized to face many users, not just for some parts of people.

I agree @fardolieri 's opinion:

please don't force it on us by removing the option to hide the icon.

An extension for public should be able to be customized to face many users, not just for some parts of people.

@fardolieri's opinion is outdated, I think he misunderstood me removing the setting to impose a feature that requires authentication, which I certainly wouldn't do.
I think it's quite the opposite, I think this setting is just for some parts of people. But I will add this setting if someone does need it.

I think this setting is just for a few users.

I don't disagree this.
But, at least for me, I need that setting, please.

I would like to hide the vite icon in my editor. I used the setting "volar.icon.vite": false before, and I thinks it's a pity, that it is now gone. I would like to have it back.
When I was talking about authentication issues, I meant I had issues with my specific application, because it attempts to redirect to a different page when not being signed in (my own authentication logic). But that's not really the point here.

@fardolieri Sorry I misunderstood. 😅
Can you change the url on the status bar to enter the login process? If it can't works, as a last resort, you can run Developer: Toggle Developer Tools command open dev console to change the login session.

I really appreciate your willingness to help me right of the bat, and after signing in, your feature does indeed work even with my codebase (nice work!). But I think that is not the point of this thread.
I still think many people would appreciate the opt-out option to hide/show the icon according to their preference.

@fardolieri Don't worry I'll add it (I just said it :) )


Dataset Unavailable

Excellent work!
But I can't download the dataset from the url
https://amsuni-my.sharepoint.com/:u:/g/personal/v_pal_uva_nl/ETD5Y1R9jQhPvuXws_hFaEoBSiuHhJ55p6JmFbiRnHwn8Q?e=luLLDx
The error is "De koppeling is verlopen."

Hi, I have updated the url. The dataset can be downloaded now!

The updated url does not work


updated cyclonedx dependency

I've had a quick look at #575 and have seen that it is quite straight  forward. Therefore, I've had a go at it.
To check that the same result is generated I've updated a test to check that the JSON is the same. The only change in the generated JSON was that the vulnerabilities got sorted and in the dependencies the value 'dependsOn': [], is missing.
Since it is not the aim of the test to test the exact output I did not commit it. If you are interested in the test I changed I can submit it as well. https://github.com/pypa/pip-audit/blob/f7a22cf193569aaa6ad2c00874ebf07fda65ddd5/test/format/test_cyclonedx.py#L21
Additionally, I don't think that the _PipAuditResultParser is necessary and it could be a function, because the BaseParser does not really add functionality. If  you feel the same way I can refactor it. https://github.com/pypa/pip-audit/blob/f7a22cf193569aaa6ad2c00874ebf07fda65ddd5/pip_audit/_format/cyclonedx.py#L24

Thanks a ton @ThunderKey!
I agree that testing the exact output is not important here: our CycloneDX compatibility guarantee is only that we comply with the schema, and nothing beyond that. Given that you haven't bumped the schema in this PR, any breaking changes here would be a bug on CycloneDX's side (if they are bugs).

Additionally, I don't think that the _PipAuditResultParser is necessary and it could be a function, because the BaseParser does not really add functionality. If you feel the same way I can refactor it.

I'm not opposed to that -- IIRC we did it that way because it was recommended by the library's documentation, but I agree that it isn't necessary.
That being said, I'd prefer to keep this PR hermetic. If you feel like doing that, let's do it in a separate PR 🙂

LGTM, thank you!


notebook image cleaning automation

Notebook will automatically remove un-used image attachments from a cell's metadata, storing them in a cache (to allow for user to move text around and not lose image data).
Cache is structured with entries for each notebook uri, and then each cell fragment under those. top level uri entries will be deleted when notebook editors are closed. The entire cache is disposed of upon the extension closing with the workspace.

One issue still pending on this is the case of a user editing and then closing the notebook document within the 500ms delay on the edit trigger. I wanted to just run the cleaner before deleting the cache entry, but onDidCloseNotebook gives a NotebookDocument whereas the cleaning function was built to accept a NotebookDocumentChangeEvent from onDidChangeNotebookDocument.
Still thinking on the best way to resolve this -- I think there is probably a way to check if there is a delay pending inside delayTrigger, but not sure how to do that atm.

Future thoughts/known issues:

The notebookImagePaste.ts file should probably have access to the cache of images, to avoid conflict between pasting in a filename that has already existed in the cache.

@mjbvz I picked up this PR since @Yoyokrazy is out. By reading through the suggestions above, I have made following changes:

Diagnostic provider for missing attachments in markdown cells
Proper debouncer which captures markdown cells that are modified with delay. At the end of the delay, analyze all cells being captured for unused attachments
On extension activation, scan open notebook cell text documents, validate missing attachments
On notebook cell change, capture the cell and request validation for missing and unused attachments
On file willRename, transfer the cache to new resource
On notebook document close, clear the cache and diagnostics

Along with changes in the clean up tool, also made required changes to the notebook diff editor to ensure the attachment metadata is presented in the diff editor.
I didn't try to tackle the race condition between notebook document change and document close. It might require introducing code action for notebook document save (which only exists in text document) as current approach of attachment cleanup is responding to cell document changed event, but not as a pre-change hook. We can discuss what's the best approach to tackle this. @mjbvz it would be great if you can take another look at this PR.

@mjbvz thanks for the review, resolved your comments ;)


[BUG] Set-PnPPage not working with Page-Object but with name

Reporting an Issue or Missing Feature
The Set-PnPPage doesn't seem to work with a page object but with the name.
Expected behavior
I'd expect to run Set-PnPPage with a variable to contains a page object as the archieved version of PnP.
Actual behavior
I ran the following code:
Set-PnPPage -Identity $page -HeaderType Custom -PromoteAs Template -ThumbnailUrl "$($placeholderImg.ServerRelativeUrl)" -ServerRelativeImageUrl "$($placeholderImg.ServerRelativeUrl)" -TranslateX 51.6301443078568 -TranslateY 36.0714285714286 -Publish | Out-Null

I got the following error:
Set-PnPPage : Insufficient arguments to update a client side page
At D:\TFS\Create-ContentTemplate.ps1:280 char:5
+     Set-PnPPage -Identity $page -HeaderType Custom -PromoteAs Templat ...
+     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : WriteError: (:) [Set-PnPPage], Exception
    + FullyQualifiedErrorId : EXCEPTION,PnP.PowerShell.Commands.Pages.SetPage

The following code does not throw an error (Differs in the Identity parameter).
Set-PnPPage -Identity $page.Name -HeaderType Custom -PromoteAs Template -ThumbnailUrl "$($placeholderImg.ServerRelativeUrl)" -ServerRelativeImageUrl "$($placeholderImg.ServerRelativeUrl)" -TranslateX 51.6301443078568 -TranslateY 36.0714285714286 -Publish | Out-Null

Steps to reproduce behavior
See above
What is the version of the Cmdlet module you are running?
1.4.0
Which operating system/environment are you running PnP PowerShell on?

[x] Windows
[ ] Linux
[ ] MacOS
[ ] Azure Cloud Shell
[ ] Azure Functions
[ ] Other : please specify

How did you acquire the page object?

$Template = 'Content Template full screen'
$page = Add-PnPPage -Name $Template -LayoutType Article -PromoteAs Template

Any progress here?

We found a bug causing this. It will be fixed in the nightly release that is available tomorrow (1.4.15-nightly)


No host resolved when loglevel is set in headless mode

Prerequisites

[X] I am using the latest version of Locust
[X] I am reporting a bug, not asking a question

Description
This works fine
locustfile = locustfile.py
headless = false
host = "http://localhost:8080"
process = 4
spawn-rate = 10
run-time = 5m
users = 100

when adding these debug settings
locustfile = locustfile.py
headless = false
host = "http://localhost:8080"
loglevel = DEBUG
logfile = locust.logfile
process = 4
spawn-rate = 10
run-time = 5m
users = 100

locust shows this and freezes
Error resolving address: [Errno 8] nodename nor servname provided, or not known

this does not happen in headless mode
Command line
locust --config local.conf
Locustfile contents
as shown

Python version
3.11.10
Locust version
2.32.1
Operating system
Mac OS 15.1 Build 24B83

I guess adding quote marks doesnt help?
logfile = "locust.logfile"
What is the full output?

yes that did help and solve the issue.
The full output was just that line
Error resolving address: [Errno 8] nodename nor servname provided, or not known
nothing happend than as far I could see
and couldn't reproduce the issue now...sad but could be a pain for new users like me

I'll reopen this in case anyone wants to have a go at it. What I'm thinking could be done is send output to stderr if logging setup fails like this.
I dont have time to look at it myself rn, but maybe later...


The list can not be rendered correctly

like the demo code:
- [Introduction](#introduction)
- [Paragraphs](#paragraphs)
- [Headings](#headings)
  * [Atx Style](#atx-style)
  * [Setext style](#setext-style)
  * [Header IDs](#header-ids)
- [Blockquotes](#blockquotes)

As stated in the documentation, you need to use the option disableForced4SpacesIndentedSublists.

thx


Wrong "Verify this certification" link in Data Visualization Certification

I noticed the "Verify this certification" link at the bottom of the Data Visualization Certification is not correct. It ends with "/data-vis-certification" while it should end with "/data-visualization-certification".
It looks like the problem lays at line 5 here.

Your Journeyman project could be to file the PR to fix this.  Good catch!

If the issue is still unfixed, can I send the Pull Request?

You may absolutely @anuragaryan - That's what it's here for!
Make sure you follow the guidelines in Contributing.md and if you get stuck, ask for help in HelpContributors.


[production] File upload stops part-way through and does not resume

Steps

Go to a newly created project on production (or if you don't have one, create one)
Go to the files tab
Drag and drop 10+ files of varying types to upload them into OSF Storage
Wait for uploads
If one upload pauses, press the x button to cancel that upload
Wait for other files to upload

Expected
I expect that all files would upload within a reasonable amount of time. For step 6, I expect after canceling one upload, all remaining files would continue to upload
Actual
On three different occasions, upon trying to upload a group of files, upload stopped after uploading only one or a couple of files. I have been waiting ten minutes for tulips.jpg to upload:

After pressing the x to cancel that one upload, none of the other files continue with upload. The page just sits with no apparent processes running, although trying to leave the page gives a pop-up modal saying that files are being uploaded, even though they are not.

@chrisseto ..... WaterButler? Thanks.

My first guess is Treebeard/Fangorn, @caneruguz?

This is very hard to tell what exactly is going on, I need to try to replicate this to understand but I don't know if this is strictly fangorn. I'll find out though.
@lbanner can you assign me?

Done!

Just as an update, this occurred for me again today, so I know that it is still an issue.

@caneruguz -- Had time to look into this one? TY

I'm working on upload canceling but I can't replicate this case again. @caileyfitz any further details would help but it looks tought to pin down.

@caneruguz Now that we have parallel file uploads, I am no longer seeing this issue, so I am going to close this.

Okay I also sent a fix to improve cancel upload consistency so hopefully this will take care of things.

@caneruguz Great, thanks!


sink(ticdc): add batch buffer for flush worker

What problem does this PR solve?

Issue Number: ref https://github.com/pingcap/tiflow/issues/4423
What is changed and how it works?
add batch buffer for flush worker. Avoid frequent memory allocation here.
Check List
Tests

Unit test

Code changes
None
Side effects
None
Related changes
None
Release note
None

/run-all-tests

/run-all-tests

/run-all-tests

/merge

/run-verify

Codecov Report

Merging #4884 (c210381) into master (687e248) will increase coverage by 0.1761%.
The diff coverage is 58.5421%.

:exclamation: Current head c210381 differs from pull request most recent head 5c069f9. Consider uploading reports for the commit 5c069f9 to get more accurate results

Flag
Coverage Δ

cdc
60.9722% <59.1240%> (+0.3736%)
:arrow_up:

dm
52.4596% <50.0000%> (-0.0100%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
@@               Coverage Diff                @@
##             master      #4884        +/-   ##
================================================
+ Coverage   56.1240%   56.3001%   +0.1761%
================================================
  Files           522        524         +2
  Lines         65325      65522       +197
================================================
+ Hits          36663      36889       +226
+ Misses        25094      25031        -63
- Partials       3568       3602        +34

/run-kafka-integration-test

/run-kafka-integration-test
/run-verify

/run-verify


orloj 0.10: rewrite from "andark"

Change the look from (copying from andark), while still providing sunset time and sun position.

Looks good, happy to merge with the little fix

Fixed, thanks for review.

You're welcome - thanks for the PR


Shapes layer does not display text if string does not include feature

🐛 Bug
When adding shapes to napari with viewer.add_shapes(), text is not displayed along with the shape unless the text string includes a feature string. So for example if features={"index": np.ones(10)} then text={'string': "{ind}"} will be displayed but text={'string': "sample text"} will not be.
In this context, I was trying to manually position a scale bar, so I wanted the text to be identical for all of the shapes. Probably not the most common use case
To Reproduce
Steps to reproduce the behavior:
This will not display any text
import numpy as np
import napari

imgs = np.random.rand(3, 512, 512)

viewer = napari.Viewer()

viewer.add_image(imgs)

bar_len = 200

lines = np.array([[[ii, 400, 100],
                   [ii, 400, 100 + bar_len],]
                    for ii in range(imgs.shape[0])])

viewer.add_shapes(lines,
                  shape_type='line',
                  name="scale bar",
                  features={"bar_len": np.ones(imgs.shape[0]) * bar_len},
                  text={'string': "200",
                        'size': 30,
                        'color': 'red',
                        'translation': np.array([5, 0]),
                        },
                  edge_width=2,
                  edge_color=[255, 0, 0, 255],
                  face_color=[0, 0, 0, 0])

viewer.show(block=True)

But this will
import numpy as np
import napari

imgs = np.random.rand(3, 512, 512)

viewer = napari.Viewer()

viewer.add_image(imgs)

bar_len = 200

lines = np.array([[[ii, 400, 100],
                   [ii, 400, 100 + bar_len],]
                    for ii in range(imgs.shape[0])])

viewer.add_shapes(lines,
                  shape_type='line',
                  name="scale bar",
                  features={"bar_len": np.ones(imgs.shape[0]) * bar_len},
                  text={'string': "{bar_len:.0f}",
                        'size': 30,
                        'color': 'red',
                        'translation': np.array([5, 0]),
                        },
                  edge_width=2,
                  edge_color=[255, 0, 0, 255],
                  face_color=[0, 0, 0, 0])

viewer.show(block=True)

Expected behavior
I expect both code snippets to display line segments with the text "200" nearby
Environment
napari: 0.4.16
Platform: Windows-10-10.0.19042-SP0
Python: 3.9.7 (default, Sep 16 2021, 16:59:28) [MSC v.1916 64 bit (AMD64)]
Qt: 5.15.4
PyQt5: 5.15.7
NumPy: 1.23.1
SciPy: 1.9.0
Dask: 2022.05.2
VisPy: 0.10.0
OpenGL:

GL version: 4.6.0 NVIDIA 511.23
MAX_TEXTURE_SIZE: 32768

Screens:

screen 1: resolution 3440x1440, scale 1.0

Plugins:

animation: 0.0.3.dev79+g54aa0e5
console: 0.0.4
micromanager: 0.0.1rc6.dev39+g98741c5
napari-svg: 0.1.6
psfmodels: 0.3.2
scikit-image: 0.4.16

Additional context
I also tried this with a fresh environment with the same results. Fresh environment info:
napari: 0.4.17
Platform: Windows-10-10.0.19042-SP0
Python: 3.9.15 | packaged by conda-forge | (main, Nov 22 2022, 08:41:22) [MSC v.1929 64 bit (AMD64)]
Qt: 5.15.2
PyQt5: 5.15.7
NumPy: 1.23.5
SciPy: 1.9.3
Dask: 2022.11.1
VisPy: 0.11.0
magicgui: 0.6.1
superqt: 0.4.0
in-n-out: 0.1.6
app-model: 0.1.1
npe2: 0.6.1
OpenGL:

GL version: 4.6.0 NVIDIA 511.23
MAX_TEXTURE_SIZE: 32768

Screens:

screen 1: resolution 3440x1440, scale 1.0

Sorry for the issue and thanks for the detailed write-up and example!

In this context, I was trying to manually position a scale bar,

This doesn't negate the bug you found, but I just want to make sure you are aware of viewer.scale_bar, which allows programmatic control of the built-in scale bar in napari. That only allows to position the scale bar in one of the four corners of the canvas, so doesn't quite match your desired usage. But I think it would be reasonable to add an offset/translation attribute to ScaleBar to allow for it - I can add a feature request if that's desired.

I wanted the text to be identical for all of the shapes. Probably not the most common use case

I believe this was a deliberate decision for strings because we thought it wouldn't be common like you suggested. And we thought it would be more common to have the convenience of specifying a feature name or format string as a plain string.
You can explicitly declare a constant string using text={'string': {'constant': '200'}, ..., but unfortunately that triggers other issues  right now v0.4.16, v0.4.17, and the main branch... I'll write up those issues here or in new bugs, and will look for workarounds and fixes!

Thanks for the reply! Yes, this time I wanted to control exactly where the scale bar showed up, so maybe for that usage more programmatic control over position/length of viewer.scale_bar would be helpful. I don't care if I do this with the shapes layer or the scale bar, but maybe there are performance reasons to prefer doing this with scale bar, particularly when looking at 3D time-lapse data
If the behavior stays the same as it is now, it would be helpful if this quirk was noted in the docstring for napari.layers.Shapes. Right now under text it says "Text to be displayed with the shapes. If text is set to a key in properties, the value of that property will be displayed. ...". It might be clearer if it also said "If the string does not include any keys, no text will be displayed"

Will definitely consider the documentation update, though may need to be more extensive as we do intend to support constant strings for text annotations (i.e. here with 'string': {'constant': '200'}).
I opened https://github.com/napari/napari/pull/5387 to fix one issue we have with that on main right now.
The other change you'd need to make to your example to make it work with that PR's branch is make the translation 3D since it should be the same dimensionality as the shapes layer (this was recently fixed in https://github.com/napari/napari/pull/5321).

If support for constant string annotations is added (which I think is very important) another in my opinion helpful addition would be adding a text field or popup to the GUI which one can use to set the string/text. Right now when adding or editing a shape there is already the option "display text",  next to it and edit field or button (which opens a small popup) could be placed to actually edit the text, see mockup below:

This would be extremely helpful when using napari for manually segmenting or labeling imaging data. For example, when I load an MRI scan and do some segmentation of bones and other structures I would like to be able to directly set the text/labels of the shapes.


Fix config reload when TLS is enabled

Problem
With TLS enabled, the POST method on /-/reload is failing.
Related: #58
Solution

Use internal url instead of external url. Otherwise we'd need to have the external ca cert.
Use proper tls context.

Fixes #207.
Upgrade notes
To upgrade from an older revision of charmed alertmanager, you may need to re-relate the tls-certificates relation to the local CA, or manually copy /usr/local/share/ca-certificates/cos-ca.crt from the alertmanager to the charm container.

@PietroPasotti I opened #212 for adding a test.


initial commit of assignment kernel error mitigation decoration

Signed-off-by: Tyler Kharazi<EMAIL_ADDRESS>

This looks good and @kharazity is covered under the UT-Battelle MLA.


--version doesn't work

Bug Description
--version doesn't show the correct version.
fix it
Steps to Reproduce
No response
Your operating system
None
Your Python version (python --version)
None
Your Robyn version
None
Additional Info
No response

@sansyrox is this related to the lastest release version??

Hi, can you be more specific I think Robyn CLI commands do not have "--version". How do I reproduce it?

@Oluwaseun241 , yes.

Hey @TroyFPV48 👋
Thanks for pointing it out. We need to add a --version flag then 😅


Twitch Dashboard claims Access Denied

Not really sure what's up with it.  It worked in the past.  Maybe something to do with the changes to the Twitch Dashboard recently?

It still works for me. That would seem to indicate you're trying to highlight a video on some other channel.
Can you confirm your username is correct in your highlighter link?

Username is correct.  Forgot Twitch is case sensitive about usernames though, and it forces all lowercase.  That fixed it.  Might be worth making of a note of that somewhere on the site so people as dumb as I am don't run into the same issue?  ;)

I didn't know about the case sensitivity! I'll make it convert to all lowercase.
Thank you.


capsolver typo

please dont PR to my master branch, i will only accept it to the dev branch

Thanks :)


Missing aria props

When running apps using this library with the latest https://github.com/sveltejs/vite-plugin-svelte, it complains about the following issues:
.../node_modules/carbon-components-svelte/src/Toggle/Toggle.svelte:57:4 A11y: Elements with the ARIA role "switch" must have the following attributes defined: "aria-checked"
.../node_modules/carbon-components-svelte/src/UIShell/HeaderSearch.svelte:64:7 A11y: Elements with the ARIA role "combobox" must have the following attributes defined: "aria-controls", "aria-expanded"

Fixed in v0.70.6


Assigning Single Char with Sqlpp11 Fails Compilation

Should a char element be assignable in the standard way? Like so: element = 'A';
I am observing that for a table defined thus:
CREATE TABLE orders (
    timestamped  timestamp NOT NULL,
    symbol       varchar(8) NOT NULL,
    side         char NOT NULL,
    price        integer NOT NULL,
    quantity     integer NOT NULL
);

And performing the below as a test, the second char assignment case does not compile.
        if (readyOrder.m_OrderSide == ProtocolDefinitions::OrderSide_t::BUY)
        {
            m_TheDatabaseConnection(insert_into(ordersTable).set(
                 ordersTable.timestamped = dt,
                 ordersTable.symbol = readyOrder.m_StockTickerSymbol.data(),
                 ordersTable.side = "B", // This standard string way with double-quotes compiles and works.
                 ordersTable.price = readyOrder.m_Price,
                 ordersTable.quantity = readyOrder.m_Quantity));
        }
        else if (readyOrder.m_OrderSide == ProtocolDefinitions::OrderSide_t::SELL)
        {
            m_TheDatabaseConnection(insert_into(ordersTable).set(
                 ordersTable.timestamped = dt,
                 ordersTable.symbol = readyOrder.m_StockTickerSymbol.data(),
                 ordersTable.side = 'S', // This standard char way with single-quotes does not compile.
                 ordersTable.price = readyOrder.m_Price,
                 ordersTable.quantity = readyOrder.m_Quantity));

        }

In case it helps for debug, the compilation error in the second case is:
/usr/local/include/sqlpp11/column.h: In instantiation of ‘sqlpp::assignment_t<sqlpp::column_t<Table, ColumnSpec>, typename sqlpp::wrap_operand<T, void>::type> sqlpp::column_t<Table, ColumnSpec>::operator=(T) const [with T = char; Table = Model::public_::orders; ColumnSpec = Model::public_::orders_::Side; typename sqlpp::wrap_operand<T, void>::type = sqlpp::integral_operand]’:
SessionManager.cpp:392:37:   required from here
/usr/local/include/sqlpp11/column.h:90:7: error: static assertion failed: invalid rhs assignment operand
       static_assert(_is_valid_assignment_operand<rhs>::value, "invalid rhs assignment operand");
       ^~~~~~~~~~~~~

It would be nice if the standard char way of assignment could also work. That way I can reduce a lot of my boilerplate by not performing if conditionals on my char underlying enum types and then doing the appropriate double-quote assignments. I could then simply do:
        Model::public_::orders ordersTable;

        m_TheDatabaseConnection(insert_into(ordersTable).set(
             ordersTable.timestamped = dt,
             ordersTable.symbol = readyOrder.m_StockTickerSymbol.data(),
             ordersTable.side = Utility::ToUnderlyingType(readyOrder.m_OrderSide),
             ordersTable.price = readyOrder.m_Price,
             ordersTable.quantity = readyOrder.m_Quantity));

Thanks

You might be better off defining your size column as tinyint or sticking with "B". Without knowing for sure I'd say tinyint would likely allow you to use 'B' though it would be binary comparison and not lexicographical. Keep in mind that in SQL even if you say 'B' it's still a string and not a character value. You can't, for instance, say SELECT 'Z' - 'A' to get 26.
This is what I get when I try in mariadb:
select CAST('Z' AS CHAR) - CAST('A' AS CHAR);
+---------------------------------------+
| CAST('Z' AS CHAR) - CAST('A' AS CHAR) |
+---------------------------------------+
|                                     0 |
+---------------------------------------+

1 row in set, 2 warnings (0.001 sec)
show warnings;
+---------+------+---------------------------------------+
| Level   | Code | Message                               |
+---------+------+---------------------------------------+
| Warning | 1292 | Truncated incorrect DOUBLE value: 'Z' |
| Warning | 1292 | Truncated incorrect DOUBLE value: 'A' |
+---------+------+---------------------------------------+

That makes sense @Erroneous1 . It seems sometimes we forget that the types within the DB are rather moreso for 'our viewing pleasure' than for the DB itself being strongly-typed. I will stick to using "S". Now that I know the assignment format the DB expects, it should be fairly straightforward for me to adapt my received characters with a utility lambda into that format. That allows me to fulfill my goal of avoiding all that boilerplate conditionals.
Done! Tried such an approach and it works.
You can close this issue. Thanks.

Thanks @Erroneous1 !
On the C++ side of things: The way the library interprets C++ types, it has to decide whether something is a text, an integral value, a date, etc. The dual nature of char cannot be modeled with the current code.
sqlpp11 is following std::is_integral, which is true_type for char.
Stay safe and Happy Easter!


Add translator support for Assignment XOR token

PR Description
This PR implements support for the Assignment  XOR token in the CrossGL project, addressing issue https://github.com/CrossGL/crosstl/issues/114. The token is now properly translated across the DirectX, Metal, and OpenGL backends.
Related Issue

shader Sample

Checklist

[ ] Have you added the necessary tests?
[ ] Only modified the files mentioned in the related issue(s)?
[ ] Are all tests passing?

@samthakur587 PTAL


Manifests|Powershell: Add 'BlockScriptOnPolicyFailure'

The default behavior with WDAC and Powershell, is that when a script fails to pass policy, the language mode that the script is executed under changes to ConstrainedLanguageMode.
This new policy setting changes that behavior when set, and instead blocks the script from executing entirely.
This setting is supported on both Windows PS (on select OS') and PS Core.

@microsoft-github-policy-service agree


[BUG] Roblox is not opening

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.
[X] I have thoroughly looked through the available Wiki articles and could not find a solution to my problem.
[X] I am using the latest version of Bloxstrap.
[ ] I did not answer truthfully to all the above checkboxes.

Bloxstrap Version
v2.8.1
What problem did you encounter?
When my Bloxtrap installs Roblox and tries to open it an error appears saying "Roblox failed to launch."

Bloxstrap Log
N/A

help me and him

Does it show any other dialog boxes like I've been experiencing? If so, then it may have something to do with Bloxstrap. Also, try running Roblox without Bloxstrap installed, as that made my Roblox installation work. Bloxstrap seems to break the Roblox installation in my case.

Client issues are no longer accepted.


go: implement smoke test

Some shared code is modified so that request-object methods
work properly.

Codecov Report

Merging #1218 into master will increase coverage by 0.05%.
The diff coverage is 85.41%.

@@             Coverage Diff              @@
##             master    #1218      +/-   ##
============================================
+ Coverage     86.15%   86.21%   +0.05%
- Complexity     3766     3775       +9
============================================
  Files           380      380
  Lines         14932    14972      +40
  Branches       1664     1669       +5
============================================
+ Hits          12865    12908      +43
+ Misses         1619     1616       -3
  Partials        448      448

Impacted Files
Coverage Δ
Complexity Δ

...m/google/api/codegen/transformer/SurfaceNamer.java
78.69% <0%> (ø)
170 <0> (ø)
:arrow_down:

...gle/api/codegen/transformer/go/GoSurfaceNamer.java
92.23% <100%> (+0.15%)
54 <2> (+2)
:arrow_up:

...e/api/codegen/transformer/TestCaseTransformer.java
89.89% <61.53%> (+2.67%)
32 <2> (+5)
:arrow_up:

.../transformer/go/GoGapicSurfaceTestTransformer.java
98.97% <96.87%> (-1.03%)
12 <2> (+2)

Continue to review full report at Codecov.

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

@shinfan PTAL

Sorry for the churn: as generated the smoke test will work on my desktop because it has application default cred. However, it won't work on CI, because creds are passed in another way. https://github.com/googleapis/toolkit/pull/1238 will help with this.

@shinfan @jba PTAL

To avoid a lot of churn, you can remove the t.Log in the future.

@jba sounds good, I'll merge this now for expedience. Reported your concern at https://github.com/googleapis/toolkit/issues/1249 so I don't loose track.


Split test for a new age step in the mailing list wizard

Trello card
Trello-1879
Context
We want to experiment with asking users to provide their age when signing up to the mailing list. Before we actually start gathering this information we want to try a few different methods for asking users their age in order to observe the impact this has in drop-off/form completion rates. The data collected will not be retained to begin with and we want to have the following variations presented to users randomly:

No age step (current state of form)
Ask for date of birth (input for day, month, year)
Ask for year of birth (dropdown)
Ask for age range (radio button of different age brackets)

We want to track when users are presented with one of the different age input variations so that we can evaluate the data after running the experiment for a period of time.
If a user is presented with one variation they should continue to see that variation if they navigate forward/backward in the wizard flow or restart the sign up.
Changes proposed in this pull request

Add timeliness gem
Add new age step to mailing list wizard
Serve random (but consistent) age question
Track age step as virtual page view
Rename mailing list feature specs (single page)
Clean up the mailing list feature specs

Guidance to review
I did some additional refactoring in the test suite so this PR contains a fair number of changes; best reviewed by-commit.
I haven't updated the single-page mailing list sign up form as we are not currently using this and it may not be used in the future (to be checked).
We still need content/copy for the age step variations; it should be fine to pick this up in a follow-up PR though as this is all behind a feature switch/not enabled in production.

Date of Birth
Year of Birth
Age Range

Going to close this for now, resurrect if needed.


[Fix] Fix Cassandra Document loader default page content mapper

We can't use json.dumps by default as many types returned by the cassandra driver are not serializable. It's safer to use str and let users define their own custom page_content_mapper if needed.

@baskaryan @eyurtsev Hi, could this get reviewed ?


Yatin: Watch conceptdigitalmarketing.com

Yatin requests the watch of the watch_keyword conceptdigitalmarketing\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by tripleee in Charcoal HQ


Return RGB Object Instead of Array

Any particular reason you don't like working with the current data structure? The order of the colors in the array seems intuitive (RGB).


error in running ipvlan in CentOS

It is not possible to run the ipvlan tests (https://github.com/kata-containers/tests/blob/master/integration/network/ipvlan/ipvlan_driver.bats)  on CentOS as we are getting the following error
Error response from daemon: kernel version failed to meet the minimum ipvlan kernel requirement of 4.2, found 3.10.0

Yeah looks like you need kernel 4.2 for ipvlan. Lets skip ipvlan test for Centos.

@amshinde , yes that is the problem, I am skipping these tests kata-containers/tests#900


lncli+docs: fix and document remote signing setup upgrade to lnd v0.15.2-beta

Change Description
Fixes the lncli wallet accounts import command to accept the address type p2tr which is required in order to upgrade a remote signing setup to lnd v0.15.x-beta.
We also add an instruction on how to upgrade such a setup to the latest lnd release.
Migrating a remote signing setup from 0.14.x to 0.15.x
If you were running a remote signing setup with lnd v0.14.x-beta and want to
upgrade to lnd v0.15.x-beta, you need to manually import the newly added
Taproot account to the watch-only node, otherwise you will encounter errors such
as account 0 not found when doing on-chain operations that require creating
(change) P2TR addresses.
NOTE: For this to work, you need to upgrade to at least lnd v0.15.2-beta
or later!
The upgrade process should look like this:

Upgrade the "signer" node to lnd v0.15.x-beta and unlock it.
Run lncli wallet accounts list | grep -A5 TAPROOT on the "signer" node
and copy the xpub... value from extended_public_key.
Upgrade the "watch-only" node to lnd v0.15.x-beta and unlock it.
Run lncli wallet accounts import --address_type p2tr <xpub...> default on
the "watch-only" node (notice the default account name at the end,
that's important).
Run lncli newaddress p2tr on the "watch-only" node to test that everything
works as expected.

Steps to Test

Create a watch-only and signer node pair on lnd v0.14.3-beta.
Upgrade both watch-only and signer node to lnd v0.15.1-beta and attempt to create a new P2TR address on the watch-only node. Expect an error to be returned (account 0 not found).
Upgrade both nodes to run this PR, then run the above migration step.
Check that creating a P2TR address now works.

Pull Request Checklist
:heavy_check_mark:

I think we may also need to bump the btcwallet version so this PR is included: https://github.com/btcsuite/btcwallet/pull/806

I think we may also need to bump the btcwallet version so this PR is included: btcsuite/btcwallet#806

Yes, you are correct. Though the bump to that version was already done in https://github.com/lightningnetwork/lnd/pull/6775 (commit 394ee84120f522960bfd4bb27d10c9f5382f1fcb), so we need to cherry-pick that commit when creating the 0.15.2-beta branch.
I tested this PR on top of v0.15.1-beta + 394ee84120f522960bfd4bb27d10c9f5382f1fcb and that works.

CI seems to be paused on this for some reason?

CI seems to be paused on this for some reason?

Ah, I added the [skip ci] suffix to the last commit since this PR only contains changes to the docs and lncli (which don't have tests at the moment). And I ran the linter locally.


Implement No-CORS on all RPC Fetch Requests on Palomabot and Curvebot.ai

Implement no-cors on all rpc fetch requests on Paloma and Curvebot

This must be enabled on the upstream side, so based off the following error message:
Access to fetch at 'https://eth-protect.rpc.blxrdn.com/'
from origin 'https://www.palomabot.ai' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present curve-leverage-lending/:1
on the requested resource. If an opaque response serves your needs, set the request's mode to 'no-cors' to fetch the resource with CORS disabled.

the Access-Control-Allow-Origin header needs to be present on https://eth-protect.rpc.blxrdn.com/ – or, as the message says - you could also include no-cors to your fetch request.

Implement no-cors on all rpc fetch requests on Paloma and Curvebot

This must be enabled on the upstream side, so based off the following error message:
Access to fetch at 'https://eth-protect.rpc.blxrdn.com/'
from origin 'https://www.palomabot.ai' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present curve-leverage-lending/:1
on the requested resource. If an opaque response serves your needs, set the request's mode to 'no-cors' to fetch the resource with CORS disabled.

the Access-Control-Allow-Origin header needs to be present on https://eth-protect.rpc.blxrdn.com/ – or, as the message says - you could also include no-cors to your fetch request.

from @rootedbox no-cors will only work if we don't care about the response


Update Vulkan and related libraries to <IP_ADDRESS>

Updates glslang, spirv-reflect, volk and vulkan to <IP_ADDRESS>.
Some patches might need some changes (one was implemented, so I deleted).

All fixed I guess

@akien-mga Running any project still says the Vulkan version is 1.3.236. Is this right or something I forgot?

That's probably the Vulkan version installed on your OS which is reported.
Updating our version gives us access to more APIs but at the end it's the installed driver version which matters.

Weird because my OS's Vulkan is actually older than that

It should report the api_version from the icd file it's using. For me on Linux with Mesa 23.1.2:
$ grep api_version /usr/share/vulkan/icd.d/radeon_icd.x86_64.json
        "api_version": "1.3.246",

So that seems to be the version Mesa 23.1.2 compiles for, despite my distro Vulkan-Loader being a bit older too (<IP_ADDRESS>).


Please consider upgrading dependencies to Dart3/Flutter 3.1, e.g. http->1.0.0

When trying to use the latest http v1.0.0 package, I get this dep. error:
Because spotify 0.11.0 depends on http ^0.13.3 and no versions of spotify match >0.11.0 <0.12.0, spotify ^0.11.0 requires http ^0.13.3.
So, because my_app depends on both http ^1.0.0 and spotify ^0.11.0, version solving failed.

Please consider using the latest http package.
Thanks.

@MagnusJohansson As a workaround, you could try to override the http dependency:
name: my_app
dependencies:
  spotify: ^0.11.0
dependency_overrides:
  http: '1.0.0`


Monoid complete

So, I figured that committing to master all the time was probably a bad idea. Just rounding out the Monoid module. It's pretty small, but was good to get the document generator working the way I like it, and whatnot.

@jenncoop, this totally works. It's more to 1) get you used to reading Elixir / running the tests, and 2) lower the bus factor. If you could read the docs for the Monoid protocol (and the Functions and Properties modules) and let me know if they're comprehensible, that would be awesome :smile:

Awesome proofreading, @jenncoop! :smile:

Hey @jenncoop, I've had to branch from this PR. Am I good to merge this into master?
Don't worry about CI failing; CircleCI doesn't support Elixir out of the box (I need to configure it, there's a story). They pass locally (run mix test from the project root).


Refactor of multiple components to functional components

Refactored several components into functional components. One questionable item was the renderRow method of TimeGridHeader, where I found no references in code using the method. Also went through these and traded let for const in areas where the particular variable wasn't being later mutated.

I can absolutely see where conversion of some of these other components to functional components can get problematic. Biggest issue being around measuring content for layout. I've switched most findDOMNode references in code to use a ref, either created from createRef or useRef, and hammer tested everything in Storybook while making my changes. The only one I couldn't do this with was the dragAndDrop EventContainerWrapper, as it has no true container to apply a ref towards (cloning children)


Update hostdb.py: create confdir not world-readable

grep passwd ~/.config/amtctrl/hosts.cfg would reveal amt-password, better be safe than sorry.

Thanks!


Try to fix the build of CI and Docker image

Checklist
Not all of these might apply to your change but the more you are able to check
the easier it will be to get your contribution merged.

[ ] CI passes
[ ] Description of proposed change
[ ] Documentation (README, docs/, man pages) is updated
[ ] Existing issue is referenced if there is one
[ ] Unit tests for the proposed change

cc #40

@mrtazz please take at a look at this one first, looks like the build need to be fixed, before I can verify #40, thanks.


what would be the estimated effort for Python 3 compatibility?

Since twistedpy now is working with python 3.4, I wonder how much effort it would be to make this project also py3 compatible.
I noticed mainly two things:
(1) except Exception, e
(2) str vs byte
I'm not asking anyone to do the work, just curious if someone has an idea if this is hard.
I can educate myself about writing py2/3 compatible code and help as well.
Thanks!

Question would be what's the status of Python 3 and Twisted. I haven't touched any of this in a while and have no desire to do so. In fact I gave up on Python because of this messed up transition from 2 to 3.

With a good unit test suite like txredisapi has, I dont think it would be terribly difficult.  Rely on six for handling the details and it usually goes quickly.

Thanks fiorix, dpkg for the information.
I will study six (and how other libs use it) and try my best to contribute.
xihan