Some tests to run a hello world ipynb inside new virtenv

thanks for the detailed steps in readme page...
can you give a few steps/commands, to see if the virtual environment with py-notebook is good to go...
i did,
source env/bin/activate
/usr/local/bin/jupyter console
It threw exception as,
  File "/usr/local/bin/jupyter-console", line 7, in <module>
    from jupyter_console import main

linux_requirements.txt must have jupyter_console, i think....

Good catch. ipython notebook became jupyter with the recent version. This worked on Mac. Will test on Linux and update the file.

Since we also require png and freetype, there can be simple shell script like ./test.sh which can open ipython with set of commands and check all necessary requirements.

Hi @madhavan020985 : We spent sometime figuring out the issue. We couldn't replicate this. We found this relevant issue: https://github.com/jupyter/jupyter_console/issues/18
Looks like you are using an older version of pip. Please do the following:
pip uninstall ipykernel
pip install ipykernel

Then do the following:

Delete the environment folder
Create a new virtual environment (as told in the setup instructions)
Activate the environment
Run the command again pip install -r requirements_linux.txt

To check: ipython notebook or jupyter notebook
Note: You might have to upgrade pip once again after activating the environment

oops... it looks, like jupyter console is a wrong subcommand... jupyter notebook launches my ipython server properly......
when i do jupyter notebook after activating the virtual environment, it is still using the globally installed ipython and hence, the virtual environment libraries are not available inside the notebook...
Doesn't the jupyter binary be lying inside/launched from env/bin/ folder?

Did you activate the virtual environment? If you are inside a virtual environment, it picks the local dependencies first before looking for global options. If ipython notebook works, you are good to go.

it works after i ran (env)weed$ python -m pip install ipython[notebook] inside the virtual environment.....


fix: kind cluster config member2

What type of PR is this?

What this PR does / why we need it:
For local develepment
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

@shauvet can you resolve the DCO problem ?

@shauvet can you resolve the DCO problem ?

OK, it's done now.

/lgtm
/approve


Fix dependencies vs devDependencies

Fixes

Fixes #400 by @zackkrida
Description

Moves all dependencies required to run npm run build into the dependencies list, so we can use npm ci --only=prod to install dependencies in our Dockerfile (see #388).
Testing Instructions

npm ci --only=prod && npm run build
Verify that the app built successfully
(optional) Click around the application after npm run start
(optional) npm run test to verify tests pass

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.
[ ] I added or updated tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[x] 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.

@krysal

running npm ci --only=prod gave me an error: sh: husky: command not found

hm, that's surprising to me because I moved husky to dependencies! It does need to be in prod because it's used to do a postinstall script that patches a package used in production.

I'm going to close this, since we will be following a multi-stage build approach I think it will be simpler to reduce size there rather than fuss too much over devDependencies vs. dependencies.


ABAPlint client: Testing server by RFC destination.

In the original repository, the server connection can only be tested by URL. Here we provide one more option so the server can be tested by RFC destination.

Hello @emrey99
unfortunately, no one picked up your PR. I didn't see this either. There have been many changes since you created this PR and resolving the conflicts will be too complicated.
Please feel free to submit a new PR based on the current repo.
Best,
Marc


[ci][Do Not Merge] Check Private CI with Clang

The intention of this PR is to cause a build in the Private CI environment that runs the chip-level tests, but builds them with clang rather than GCC (as private CI now has an up-to-date toolchain).
I think I have correctly changed sim.mk to pick up the right toolchain configuration, but I'm not sure any of the tests run on private CI actually build any software. I don't understand dvsim.py but also I cannot change the private CI build task in this PR anyway, so this might be pointless.
Let's see if this passes or fails, at any rate.

Ok, I worked out how to add a software test to the things being built. It could still fail, because apparently that test is flaky, but i want to see it at least build and potentially run.

From the build log:
Unable to find meson toolchain file at /tools/riscv/meson-riscv32-unknown-elf-clang.txt. Aborting
I don't think I can perform much more diagnosis on this today.

Seems like the updated image wasn't picked up by k8s. Poked the cluster, please give it another try.

Seems like the updated image wasn't picked up by k8s. Poked the cluster, please give it another try.

Thanks, done so!

meson.build:5:0: ERROR: Unknown compiler(s): [['/tools/riscv/bin/riscv32-unknown-elf-clang']]

Ok this seems like a fun one. Maybe ansible isn't dealing with relative symlinks properly? I'm not sure. in the tar file, riscv32-unknown-elf-clang is a symlink to clang which is a symlink to clang-11.

At least two problems as I can see.

 You should be able to build that container with instructions by early next week, as I work on cleaning some of it up.

I'm not sure where the first thing is coming from, we build clang on azure's ubuntu-16.04 image.
The second thing doesn't arise when i untar, but that's not using GNU tar. :(

Ok, this is currently failing dif_plic_sanitytest in DV, with a timeout, which tells me that the software is building and almost running. I'm deeming that a success, because this test is not expected to pass due to #2405.


Adding google authentication for login

@ankitapuri Please assign this task to me! I will add google authentication for login page which will be easier for person to login
Thanks

Being a participant you can't create a issue

@baibhav3211 I already have a working google authentication


Validate match argument when passed into rel/rels/etc...

Coverage increased (+0.02%) to 95.73% when pulling 093f69882bbccd068888bfc53827c6102bdd3083 on validate_match into f102a43d6a853272b462c5a5e7a2cc49340eaf4b on master.

Coverage increased (+0.02%) to 95.73% when pulling 093f69882bbccd068888bfc53827c6102bdd3083 on validate_match into f102a43d6a853272b462c5a5e7a2cc49340eaf4b on master.

Coverage increased (+0.02%) to 95.73% when pulling 093f69882bbccd068888bfc53827c6102bdd3083 on validate_match into f102a43d6a853272b462c5a5e7a2cc49340eaf4b on master.

Coverage increased (+0.02%) to 95.73% when pulling b8447c9b01e78198416f2eee653dcea3ed3e3407 on validate_match into f102a43d6a853272b462c5a5e7a2cc49340eaf4b on master.

Coverage increased (+0.02%) to 95.73% when pulling b8447c9b01e78198416f2eee653dcea3ed3e3407 on validate_match into f102a43d6a853272b462c5a5e7a2cc49340eaf4b on master.

Coverage increased (+0.02%) to 95.73% when pulling b8447c9b01e78198416f2eee653dcea3ed3e3407 on validate_match into f102a43d6a853272b462c5a5e7a2cc49340eaf4b on master.

Coverage increased (+0.02%) to 95.73% when pulling 1f2220de5f29bee51092c8b2a4b1b25a2321a219 on validate_match into f102a43d6a853272b462c5a5e7a2cc49340eaf4b on master.

Coverage increased (+0.02%) to 95.73% when pulling 1f2220de5f29bee51092c8b2a4b1b25a2321a219 on validate_match into f102a43d6a853272b462c5a5e7a2cc49340eaf4b on master.

Thanks coveralls....


Style changes

Make all methods which don't use self static
Removed redundant parenthesis on the ifs
Removed unused variable in line 264
Removed redundant tokens in regex at line 154

Thanks for making the PR! 😃
I have actually been making a lot of similar changes (e.g. the static methods) in the latest branch (https://github.com/xenova/chat-replay-downloader/tree/package), but I like your other changes 👍
I will review your other changes and add merge the pull request, however, I will merge the package branch with master soon.


Added 22. Devops Notes and Interview Questions

DevOps Notes and Interview Questions

Comment: Contains all DevOps concepts in PDF format.Contains all Concepts of Devops pdfs

Thanks @kranthi9558 😊


First examples and README

Description of distributed-ranges library in README.md + basic examples

Since Mateusz will be off next two days I've just merged the code and I will work further on it on main branch.


Messages-Admin-App - Sprint 2

Description:

Build the Screen to match the Figma design.
Integrate Screen with the Backend

@tamsay I am going to work on this issue.

@tamsay can you work on this when you finish your assigned issues

@Honeyloveet , alright I will.


Allow timeouts to be defined when getting instances from server

Hello,
I just had the problem, that the five seconds to get the instances list from SQLServer in no enough. I'm running into the following error:
Unable to get instances from Sql Server Browser on host %s: dial udp: i/o timeout

It seems the value for the timeouts is hardcoded in tds.go in the function getInstances(). Would be good to have another parameter to be passed by the connection string, instead of these hardcoded values.
Don't ask me why it's taking more then five seconds from to time to time. I'm still looking into this, as this should be an network or server related issue.
My current workaround is it, to define the instance port and skip passing the instance name in the connection string.

Looks like there may have been a reason to set it to 5 seconds (see commit ffa91a0c914fb9807b11c781b85ec095bdc46e24). @denisenkom is there a reason why the timeout was hard coded to 5 seconds? Do you agree with @fabiang that it should be a settable parameter in the connection string? If so, I can help contribute.

@fabiang seems like that's actually what the dial timeout property is for, but it is currently overwritten by the 5 seconds value. I'll created PR #523 addressing this issue. Please give it a try.

@fabiang the fix is merged now. Can you give it a try? You can set the timeout by using the dial timeout property in the connection string.

I've a job running every few minutes which is testing this right now. I'll give it time till tomorrow to verify it's fixed. Of cause it's possible that the network latency or the SQLServer load is better now.

250 runs later I can say it's working for me. Thank you very much!


Scripts for building and uploading the morpheus-core conda package

With this change two conda packages are generated -

morpheus - with all the components (core + llm + examples + tests). This is existing code.
morpheus-core - with only the core (sources, stages and sinks)
The morpheus-core package is uploaded to anaconda.org/nvidia for use by applications.

A new PR label "conda-core-build" is available to trigger building the morpheus-core conda package as a part of the CI pipeline.
Testing done:
Generated the morpheus-core conda package via  -
a. local ci runner -
ci/scripts/run_ci_local.sh conda_core

b. docker build -
DOCKER_IMAGE_TAG=latest DOCKER_IMAGE_NAME=conda_bld_morpheus_core DOCKER_TARGET=conda_bld_morpheus_core DOCKER_EXTRA_ARGS="--build-arg CONDA_UPLOAD=upload --build-arg CONDA_TOKEN=<my-token> --progress plain" ./docker/build_container.sh

And verified an application that uses the core stages.

/ok to test

/ok to test

/ok to test

/ok to test

/ok to test

/merge


Remove SmartDeviceLink.h import from SDLRPCMessage.m

Fixes #38
Anticipated Pull date Jan 23, 2015

We can review this in the meeting tomorrow, so replacing offline review with scheduled review.


Baseline problem

MetacelloNotification: Loaded -> BaselineOfCoypu --- unresolved
Error: The project spec 'Phausto' in project BaselineOfCoypu has incompatible specs. MetacelloProjectReferenceSpec and MetacelloPackageSpec are not compatible.

how can we solve this? should I change something in Phausto?


try to fit entire teamname in seitan text

cc @malgorithms: I went with a simpler version of what we talked about earlier; I trimmed extra words from the input and tried to fit as much teamname as possible into the text. The max length should be 153 characters.
@keybase/react-hackers

Drive by fixes:

Don't disable the button in a no-op while selecting a new role
Better check for if you're an owner to fix bugs while selecting roles for members of a team you're an implicit admin of

@chrisnojima addressed your comments & added a couple drive-bys that Miles pointed out. PTAL


Handling empty lists with AsyncValue/.when

When calling RESTful API's it is quite common to see empty arrays returned instead of null.
What is the best solution to handle this in Riverpod with AsyncValue?
Currently there is .whenOrNulland .when with data, error and loading. What I am missing is a empty function. As often developers forget to implement this state.
Describe the solution you'd like:
.when(
      data: (data) => print(data),
      empty: () => print('No results found.'),
      loading: () => print('loading'),
      error: (e, st) => print('Error: $e'),
    );

This is what i am doing currently:
.when(
      data: (data) {
        if (data.isEmpty) {
          print('No results found.');
        }

        print(data);
      },
      loading: () => print('loading'),
      error: (e, st) => print('Error: $e'),
    );

What you're currently doing is the expected solution.

Thanks for your fast response. Do you find the suggestion useful for Riverpod 2.0? It would make apps more fail save, when handling data.

:wave:
I don't agree with this change.
AsyncValue doesn't know what the data is. There's no way for it to know what "empty" could be. And the data is not restricted to being a List
I don't think there's anything wrong with the current syntax.

Yes you are right. There are too many variations of data.
Thanks 👍


python39Packages.ipywidgets: use pytestCheckHook and general cleanup

Description of changes
Modernizes the derivation a bit, including the use of pytestCheckHook.
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 171624 run on x86_64-linux 1

20 packages failed to build:

python310Packages.atomman
python310Packages.augmax
python310Packages.einops
python310Packages.pydeck
python310Packages.qiskit-aer
python310Packages.qiskit-finance
python310Packages.qiskit-ibmq-provider
python310Packages.qiskit-ignis
python310Packages.qiskit-nature
python310Packages.qiskit-optimization
python39Packages.atomman
python39Packages.pydeck
python39Packages.qiskit
python39Packages.qiskit-aer
python39Packages.qiskit-finance
python39Packages.qiskit-ibmq-provider
python39Packages.qiskit-ignis
python39Packages.qiskit-nature
python39Packages.qiskit-optimization
streamlit

38 packages built:

ihaskell
python310Packages.ansible-kernel
python310Packages.cufflinks
python310Packages.ipydatawidgets
python310Packages.ipympl
python310Packages.ipyvue
python310Packages.ipyvuetify
python310Packages.ipywidgets
python310Packages.jupyter
python310Packages.jupyter-sphinx
python310Packages.kmapper
python310Packages.lightparam
python310Packages.potentials
python310Packages.qiskit-terra
python310Packages.vega
python39Packages.ansible-kernel
python39Packages.augmax
python39Packages.cufflinks
python39Packages.einops
python39Packages.elegy
python39Packages.ipydatawidgets
python39Packages.ipympl
python39Packages.ipyvue
python39Packages.ipyvuetify
python39Packages.ipywidgets
python39Packages.jupyter
python39Packages.jupyter-sphinx
python39Packages.kmapper
python39Packages.lightparam
python39Packages.potentials
python39Packages.qiskit-machine-learning
python39Packages.qiskit-terra
python39Packages.scikit-tda
python39Packages.stytra
python39Packages.treex
python39Packages.vega
sage
sageWithDoc

The output from the derivation hasn't changed, so any failures are pre-existing.
If no one objects, I'll go ahead and merge tomorrow.


Plugin should auto insert closing for /*

We auto-close most of the block symbols. E.g. { adds a }, ( adds a ).
Consequently I keep expecting the editor to auto close /* with */ on the next line.

mifxed


Programming exercises: Fix an issue with ordered tasks in problem statements on Safari

Checklist
General

[x] This is a small issue that I tested locally and was confirmed by another developer on a test server.
[x] I chose a title conforming to the naming conventions for pull requests.

Client

[x] I followed the coding and design guidelines and ensured that the layout is responsive.
[x] I documented the TypeScript code using JSDoc style.
[x] I added multiple screenshots/screencasts of my UI changes.

Motivation and Context

There seems to be a bug in Safari that displays the marker of an ordered list in the wrong position if an element is injected into the list item. I created a minimal reproduction example: https://jsfiddle.net/L7ug9hkt/
This does not happen in Chrome or Firefox.
This affects programming exercise task lists as the status indicators are injected later:

Description

Added a zero-width space as initial content to the div anchor where the status gets injected so it actually consumes a line of space. Then, we only replace that space with the actual status indicator, preventing the incorrect placement of the marker.
Steps for Testing

Prerequisites:

1 Programming Exercise with a Task List (Default Bubble Sort)
1 User on Safari (macOS)

Log in to Artemis
Navigate to the exercise as a student
Ensure that Safari displays the ordered list numbers in the correct location (unlike the image above)

Review Progress

Code Review

[x] Review 1
[x] Review 2

Manual Tests

[x] Test 1
[x] Test 2

Thanks for the fast fix 👍 Great work


Correct hue light level units and value calculation

As pointed out by https://github.com/home-assistant/home-assistant/pull/22598#issuecomment-485518469

It seems that after updating from b2 to b3 the sensor values are extremely different, so I think there might be a minor bug here.
It used to report easy to read values like 3847 and now it's very long numbers that do not appear rounded 46.838178785024404 lx

Hmm. With such a high level of variance, there can be a very large
difference at low light levels between say 0.001 and 0.1. The UI should be
responsible for rounding IMO with the "real" ungrounded values available
for other uses still.
What we've run into really, is the very reason Philips choose not to
represent this value in lumens or lux.
On Tue, 23 Apr 2019, 20:14 SeanPM5<EMAIL_ADDRESS>wrote:

It seems that after updating from b2 to b3 the sensor values are extremely
different, so I think there might be a minor bug here.
It used to report easy to read values like 3847 and now it's very long
numbers that do not appear rounded 46.838178785024404 lx
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/home-assistant/home-assistant/pull/23309#issuecomment-485937303,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABD47RC2M5QFF7USNVJP33PR5NZ3ANCNFSM4HHSF3AQ
.

not sure if this is still allowed here, but the new light_level sensors are displaying lx, and not the native light_level off of the Hue hub. Might be better calling them likewise, sensor.X_sensor_lx.
Feel with @SeanPM5 on the rounding, which might be a bit less precise (put differently, please round to max 2 decimals). There's really no use case  for more precision, even at low lightlevels, and it either forces users to have horrible UI, or create extra template sensors to mitigate that... compare this:

to:

Please open an issue if you suspect a bug.
If you want to suggest an enhancement please open a feature request in the Feature Requests section of our community forum.
Merged PRs should not be used for enhancement discussion or bug reports. If you've found a bug it's ok to make a review with inline comments and link to an issue that reports the bug.
Thanks!


Error trying to build a container from within a container (DIND).

My build container...builds containers! 😆
Unfortunately, that means when I go to build, I get this message: "Got permission denied while trying to connect to the Docker daemon socket"
I have this in my yaml:
resources:
  containers:
  - container: "build-image"
    endpoint: "build-image"
    image: "[REDACTED].azurecr.io/[REDACTED]/build-image:latest"

pool:
  vmImage: 'Ubuntu 16.04'

container: "build-image"

steps:

  # ...other steps

  - task: Docker@1
    displayName: "Build Docker Container"
    inputs:
      imageName: "$(REGISTRY_URI):$(VERSION_SLUG)"

I'm able to accomplish this during my Travis-CI tasks, hoping I can here as well so that I can finish porting over our builds to your service! 😄

@dakale I don't know how to build a container from within a container - is there some syntax I can recommend to people?

Heres a sample I came up with, that works but is a little different from what you have:
resources:
  containers:
  - container: CONTAINER_NAME
    image: IMAGE

phases:
- phase: Build_container_in_container'
  queue:
    name: 'Hosted Ubuntu 1604'
    container: CONTAINER_NAME

  steps:
  - script: |
      cd SAMPLE_DIR && npm install && npm run build
    displayName: Build sample

  - script: |
      echo "Copying built dist into container (pseudo-multistage)"
     # Note: This Dockerfile copies the dist folder built above into the container
     # ie it uses the build containers context to build a container on the host
      sudo docker build SAMPLE_DIR
    displayName: Build container artifact

The reason I call it pseudo multistage is being this has the same basic outcome of Docker multistage builds without actually using the feature.
The benefit of the way you are doing it is, of course, you can execute tasks instead of just RUNning commands.
I think the workaround is going to be related to the sudo docker build
When the build runs in a container it creates a user inside the container to run as, which wont match the user on the host (a user called vsts, which is in the docker group, allowing you to run docker without sudo)
For now I recommend: just using a script to execute docker build directly, youll just need to template in your registry, image, version etc into the script but I think it will be easy to convert
If you dont want to change the Docker task step, you might be able to make something like this work:
https://vitorbaptista.com/how-to-access-hosts-docker-socket-without-root
where basically you add the user inside the container to the group with the GID that owns the docker socket on the host
Let me know if you have any questions


pornhub.com uploader/channel name and tagged pornstar support in filename output template

Checklist

[x] I'm reporting a site feature request
[x] I've verified that I'm running youtube-dl version 2020.05.08
[x] I've searched the bugtracker for similar site feature requests including closed ones

Description

I would like to see youtube-dl support adding things to the filename such as uploader or channel name and/or name of tagged pornstars via the output template for videos downloaded from pornhub. This is good for better file sorting and grouping files from a channel together, or when searching for downloaded videos with specific people in them
thank you! :)

uploader is supported. cast list is continued in #32936.


[zh-cn] sync tasks/administer-cluster/configure-upgrade-etcd.md

[zh-cn] sync tasks/administer-cluster/configure-upgrade-etcd.md

/approve

/lgtm


Segfault with set_use_custom_draw

That said, set_use_custom_draw does seem like the right approach for this. If you can reproduce the segfaults we should figure out why they occur.

Originally posted by @rtsuk in https://github.com/pd-rs/crankstart/issues/35#issuecomment-1483709249

Here's a minimal reproduction:
#![no_std]
extern crate alloc;

use {
    alloc::boxed::Box,
    anyhow::Error,
    crankstart::{
        crankstart_game,
        graphics::Graphics,
        sprite::{Sprite, SpriteManager},
        Game, Playdate,
    },
    crankstart_sys::{LCDBitmapFlip, PDRect},
};

struct State {
    // If the sprite is not stored, the crash doesn't happen
    _sprite: Sprite,
}

impl State {
    pub fn new(_playdate: &Playdate) -> Result<Box<Self>, Error> {
        // Load an image, assign it to a custom-draw sprite, add the sprite.
        // Crash doesn't happen unless a bitmap is assigned to the sprite.
        let bitmap = Graphics::get().load_bitmap("graphics/wedge")?;
        let sprite_manager = SpriteManager::get_mut();
        let mut sprite = sprite_manager.new_sprite()?;
        // No crash unless it's custom-draw.
        sprite.set_use_custom_draw()?;
        sprite.set_image(bitmap, LCDBitmapFlip::kBitmapUnflipped)?;
        sprite_manager.add_sprite(&sprite)?;
        Ok(Box::new(Self { _sprite: sprite }))
    }
}

impl Game for State {
    fn update(&mut self, _playdate: &mut Playdate) -> Result<(), Error> {
        Ok(())
    }

    fn update_sprite(&mut self, _: &mut Sprite, _: &mut Playdate) -> Result<(), Error> {
        Ok(())
    }

    // Same crash happens with or without this definition:
    fn draw_sprite(&self, _: &Sprite, _: &PDRect, _: &PDRect, _: &Playdate) -> Result<(), Error> {
        Ok(())
    }
}

crankstart_game!(State);

The result:
Loading C API game: target/Golf Date.pdx/pdex.so
16:23:47: Loading: OK

Thread 1 "PlaydateSimulat" received signal SIGSEGV, Segmentation fault.
0x0000555555968c88 in _compositeRow ()
(gdb) bt
#0  0x0000555555968c88 in _compositeRow ()
#1  0x00005555559735da in LCDBitmap_drawBitmap ()
#2  0x0000555555980d70 in LCD_drawBitmap ()
#3  0x00005555559828b9 in LCDSprite_draw ()
#4  0x000055555597d71c in LCDDisplayList_drawScreenRect ()
Dwarf Error: Cannot find DIE at 0xccf referenced from DIE at 0x1349f [in module target/Golf Date.pdx/pdex.so]

The backtrace is the same whether I define draw_sprite or not.  Is it not being hooked up properly?  Or is draw_sprite required to take some action that the SDK depends upon having happened?

I don't recall seeing any C examples that use custom draw and then also set an image on the sprite. I'm not sure what the Playdate will do in that case.

If it's just something that shouldn't be done, I could make a doc comment :)  A more mechanical fix might involve a bool on Sprite that tracks whether set_use_custom_draw has been called and whether an image has been set, failing if one or the other has already happened...

The backtrace suggests the crash is in the Playdate code, so maybe that's not a good thing to do? One could write a C version and see what happens. That, or ask on the forums.


feat(hummingbird): expose CampaignMain

Allow access to the main Campaign component through HB's API rather than shadowing. Makes customising the CampaignCard easier and more stable

Maybe we should support shadowing, so we can define custom behavior once? I already ran into this as well. We still need this export then. Let's think of a good name for this component before we expose it.


feat: can now specify timeout parameter for lambda tests

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Correction for raw affiliation  Laboratory of Innovation and Systems Engineering ; Mechanical and Structural Engineering Department ; ENSAM, Moulay Ismail University, Meknes, 50000, Morocco; ENSAM-Meknes: Universite Moulay Ismail Ecole Nationale Superieure d'Arts et Metiers

Correction needed for raw affiliation  Laboratory of Innovation and Systems Engineering ; Mechanical and Structural Engineering Department ; ENSAM, Moulay Ismail University, Meknes, 50000, Morocco; ENSAM-Meknes: Universite Moulay Ismail Ecole Nationale Superieure d'Arts et Metiers
raw_affiliation_name:  Laboratory of Innovation and Systems Engineering ; Mechanical and Structural Engineering Department ; ENSAM, Moulay Ismail University, Meknes, 50000, Morocco; ENSAM-Meknes: Universite Moulay Ismail Ecole Nationale Superieure d'Arts et Metiers
new_rors: 04cnscd67
previous_rors: 018pp1107;04cnscd67
works_examples: W4382059278
contact: c408ef855d03df2804ddd3519c3d9848:0da55ed6d2cd1057479f12df99 @ ensam.eu

This issue was accepted and ingested by the OpenAlex team on 2024-10-09. The new affiliations should be visible within the next 7 days.


release: Update release notes with GitHub issues

List all issues closed after the date of the 1.10 release (8th December
2017).
Signed-off-by: Carles Cufi<EMAIL_ADDRESS>

Note, I listed all the issues, including those that were closed as duplicates.

List obtained as per; https://github.com/zephyrproject-rtos/zephyr/wiki/Release-procedure#listing-all-closed-github-issues

Codecov Report

Merging #6539 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #6539   +/-   ##
=======================================
  Coverage   53.36%   53.36%
=======================================
  Files         435      435
  Lines       41308    41308
  Branches     7922     7922
=======================================
  Hits        22046    22046
  Misses      16044    16044
  Partials     3218     3218

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 1eacdc7...54145b7. Read the comment docs.

@dbkinder fixed

missed one (line 302)

@dbkinder fixed

@dbkinder CI passes now


can't change list to JS

"immutable": "^3.8.1"
create-react-app 1.4.0
This is my reducer
const initialState = Immutable.Map({
    resources: []
});
// payload is like this [{id: "1", createdAt:<PHONE_NUMBER>, title: "title 1"}, {id: "2", createdAt:<PHONE_NUMBER>, title: "title 2"}]
export default (state = initialState, { type, payload }) => {
    switch (type) {
    case GET_RESOURCES_SUCCESS: {
        return state.merge({ resources: payload });
    }
};

mapStateToProps function
const mapStateToProps = (state) => ({
    resources: state.getIn(['crud', 'resources']).toJS()
});

error: toJS() is not a function
if I remove toJS()
const mapStateToProps = (state) => ({
    resources: state.getIn(['crud', 'resources'])
});

and console.log(resources) in render function
it pulls out
List {size: 50, _origin: 0, _capacity: 50, _level: 5, _root: VNode, …}
Is there any way to change this List to Array?

I should use this toJS(); in reducer instead of in component


First draft discussion

NOTE: This PR will remain as a draft and be closed once a second draft is finalized. The actual merge from this branch into master will be done in the CF Cips repo, after third draft is finalized.

Right now, this pretty much just represents my understanding of how Nebula royalties work with CIP68.
There's a couple ideas I may want to add / propose, such as introducing asset-specific royalties. I haven't gotten to fleshing those out in my head yet though. If you guys have other ideas you'd like to propose, please include them here for discussion. I will drop my ideas here as well once I get my head around them.

Fixed the token name to match nebula implementation since I hadn't meant to depart from it.
I do have something of an issue with the way the nebula implementation tethers royalties to collections, but I think that's likely better addressed with a more robust datum structure than by allowing multiple royalty data. Still mulling this idea over.

Regarding the above ^ we decided against departing from the most basic implementation. We intend to first push a simple, forward compatible base for royalty information to ensure it passes the CIP process, and then follow with improvements such as multiple royalties which may be more controversial.

A couple realizations from my review today:

In the current implementation & specification designed by Ales for Nebula, neither to policyId nor the tokenName of the royalty NFT needs to be related to the collection. This completely breaks the lookup paradigm used for the 3 other token types.

Solutions I've come up with are:

Enforce that the policyId must match that of the user token
Leave out the lookup section, as looking up the royalty based on the user token is impossible

I lean toward the former, but that would break some of the Nebula royalty NFTs that have already been minted. I expect Ales  had some solution in mind for lookups, but want to get your feedback before I ask him. @colll78 @solidsnakedev

Although it isn't specified anywhere yet, I wonder if we may have more luck passing this solution if we specify some limitation on validation rules for the script the royalty NFT is locked at. (Probably just an always-fails script for now, complicating validation too much sounds like a good way to alienate people.)

I'm leaning away from this for our first/second drafts; I figure we can introduce that if people request it in the public review, but again interested in your input.

My understanding when reading our Onchain solutions is that , we mint 3 tokens with the same Minting Policy ID

NFT TokenName + label 100 -> lock at MetadataControl Script
NFT TokenName + label 222 -> lock at user wallet
Royalty token name + label 500 -> it's not clear yet but it may be lock at AlwaysFails script or a RoyaltyControl script with some metadata constrains

I think It's possible to find the royalty info if we do the follwing:

Fetch the NFT PolicyID from wallet's user
Build lucid Unit
royaltyUnit = toUnit ( PolicyID, fromText("Royalty"), 500)
Query Royalty token at AlwaysFails script or RoyaltyControl script
royaltyUTXO = lucid.utxosAtWithUnit( RoyaltyControlAddress, royaltyUnit)

IMO Nebula solution is far away from what we're proposing, since Ales made it so that it only works with his marketPlace contract

I went ahead and made the policyId match required. Since there are a lot of possible valid designs for royalty control, I left out specifying it at all.
I believe this should be ready to pass on to Ales / Nick etc. for their thoughts.

Closing. Will create a second draft PR so we have a clean slate as we gather collaborator input.


Do not try to install non-existing npm packages and warn instead?

I noticed that if you try to import a npm package that doesn't even exist (say you do import Notes from 'Notes.jsx';), npm-install-webpack-plugin will still kick in. It might be preferable to handle this some other way (skip install, warn somehow?).

Good point!  This would fail currently and break the process, right?
This is something I haven't tested very much, but I agree that it should be a warning & caught.

Good point! This would fail currently and break the process, right?

It's sketchy, yeah.

Coverage decreased (-0.4%) to 94.083% when pulling c727a04d5a78d24cfcae37470a2132d0421358ba on 48-ignore-erroneous-modules into 80d06ed3d124c294e9e2f7da501d2e1909fe59b2 on master.

Coverage decreased (-1.5%) to 92.982% when pulling 05c838542d3789a369ff5dc750c15c3230d78f1f on 48-ignore-erroneous-modules into 80d06ed3d124c294e9e2f7da501d2e1909fe59b2 on master.

@bebraw When testing this out (e.g. import something from "does.not.exist.jsx") within the examples, I got this annoying, cyclical 404 from npm install.
What do you think of the fix here?

It tracks erroneous modules for running process.

Meaning, if installing a module causes an error (for any reason), it's ignored.

There's a console.warn when there's a problem.

Still, Webpack will fail the build due to the module not installing correctly:

Cannot resolve module does.not.exist.jsx

So, the user has to take the following steps to resolve the issue:

Remove the offending line of code (or correct the dependency)
If the file didn't exist in lib/ (or whatever resolve.root may have) but then it shows up later, they'll likely have to kill & restart the process.  (However, I may need to test this since anything in lib/ would skip NPM entirely!)

I know the last is less than ideal, but tracking the erroneous dependencies within the plugin.js (which is stateful) vs. installer.js didn't work (somehow Webpack just kept trying!).
In a perfect world, and I think you'll agree, we would ignore the erroneous module once per save.

@ericclemmons Quick idea. Do a pre-emptive ping against https://www.npmjs.com/package/<package name>. If that 404's, throw a warning and do not even start installing. Maybe this would do it. Sure, there's an extra query, but it's a small price to pay for simplicity.

The problem with doing that is that it'd change this portion of the code to be async, which would cause a ripple effect through the rest of the code.
Another alternative would be:
$ npm info does.not.exist.jsx version
// 404s
$ npm info express version
// 4.13.4

No matter which method, the logic above would be similar.

Yeah, npm info would work too.
Sometimes it's like this with JavaScript. I've encountered async ripple in the past as well. 😄

So the crossroads I'm at mentally is whether it's better to pre-check or
fix cyclical issue currently affecting failed installs.
On Mon, May 16, 2016 at 8:25 AM Juho Vepsäläinen<EMAIL_ADDRESS>wrote:

Yeah, npm info would work too.
Sometimes it's like this with JavaScript. I've encountered async ripple in
the past as well. 😄
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/ericclemmons/npm-install-webpack-plugin/pull/48#issuecomment-219423875

--

Eric Clemmons

I'm going to get this resolved soon.

Coverage decreased (-1.5%) to 92.982% when pulling b374b5cbb533b69f632a2f2dfab24d0689d7756a on 48-ignore-erroneous-modules into 80d06ed3d124c294e9e2f7da501d2e1909fe59b2 on master.

Coverage increased (+0.2%) to 94.737% when pulling a386c9768d2e9ab4d9333846000875b063f1c7db on 48-ignore-erroneous-modules into 80d06ed3d124c294e9e2f7da501d2e1909fe59b2 on master.

> webpack --config webpack.config.server.js --watch

Hash: 1f94dda0f9d9f5ed90e1
Version: webpack 1.12.15
Time: 623ms
    Asset     Size  Chunks             Chunk Names
server.js  5.04 kB       0  [emitted]  server
    + 9 hidden modules
Listening on http://localhost:3000/
Installing does.not.exist.jsx...
npm ERR! Darwin 15.4.0
npm ERR! argv "/Users/Eric/.nvm/versions/node/v4.2.0/bin/node" "/Users/Eric/.nvm/versions/node/v4.2.0/bin/npm" "install" "does.not.exist.jsx"
npm ERR! node v4.2.0
npm ERR! npm  v2.14.7
npm ERR! code E404

npm ERR! 404 Registry returned 404 for GET on https://registry.npmjs.org/does.not.exist.jsx
npm ERR! 404
npm ERR! 404 'does.not.exist.jsx' is not in the npm registry.
npm ERR! 404 You should bug the author to publish it (or use the name yourself!)
npm ERR! 404
npm ERR! 404 Note that you can also install from a
npm ERR! 404 tarball, folder, http url, or git url.

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/Eric/Projects/ericclemmons/npm-install-webpack-plugin/example/webpack1/npm-debug.log
Ignoring does.not.exist.jsx due to errors installing...
webpack building...
webpack built 94340e44d77497bd75d2 in 4708ms
Hash: 94340e44d77497bd75d2
Version: webpack 1.12.15
Time: 4708ms
                               Asset       Size  Chunks             Chunk Names
f4769f9bdb7466be65088239c12046d1.eot    20.1 kB          [emitted]
                           client.js     799 kB       0  [emitted]  client
                  1.94340-d71599c.js  711 bytes       1  [emitted]
chunk    {0} client.js (client) 753 kB [rendered]
    [0] multi client 40 bytes {0} [built]
    ...

ERROR in ./lib/App/index.js
Module not found: Error: Cannot resolve module 'does.not.exist.jsx' in /Users/Eric/Projects/ericclemmons/npm-install-webpack-plugin/example/webpack1/lib/App
 @ ./lib/App/index.js 29:20-49
webpack building...
webpack built 8618e8e7c04dcfc9b964 in 6548ms
webpack building...
webpack built 8618e8e7c04dcfc9b964 in 149ms

I'm OK with this implementation.  It's fairly simple:

If a module fails to install, it's ignored for the remainder of the running process.
The user gets to see the NPM output directly  and a custom warning.
The only downside is that the main NPM process will continue to build.

Ok, this is cleaner output. I was able to prevent compilation if the pre-compiler fails:
> webpack --config webpack.config.server.js --watch

Hash: 1f94dda0f9d9f5ed90e1
Version: webpack 1.12.15
Time: 769ms
    Asset     Size  Chunks             Chunk Names
server.js  5.04 kB       0  [emitted]  server
    + 9 hidden modules
Listening on http://localhost:3000/
Installing does.not.exist.jsx...
npm ERR! Darwin 15.4.0
npm ERR! argv "/Users/Eric/.nvm/versions/node/v4.2.0/bin/node" "/Users/Eric/.nvm/versions/node/v4.2.0/bin/npm" "install" "does.not.exist.jsx"
npm ERR! node v4.2.0
npm ERR! npm  v2.14.7
npm ERR! code E404

npm ERR! 404 Registry returned 404 for GET on https://registry.npmjs.org/does.not.exist.jsx
npm ERR! 404
npm ERR! 404 'does.not.exist.jsx' is not in the npm registry.
npm ERR! 404 You should bug the author to publish it (or use the name yourself!)
npm ERR! 404
npm ERR! 404 Note that you can also install from a
npm ERR! 404 tarball, folder, http url, or git url.

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/Eric/Projects/ericclemmons/npm-install-webpack-plugin/example/webpack1/npm-debug.log

/Users/Eric/Projects/ericclemmons/npm-install-webpack-plugin/example/webpack1/node_modules/webpack-dev-middleware/middleware.js:106
			if(err) throw err;
			        ^
ModuleNotFoundError: Module not found: Error: Cannot resolve module 'does.not.exist.jsx' in /Users/Eric/Projects/ericclemmons/npm-install-webpack-plugin/example/webpack1/lib/App
    at /Users/Eric/Projects/ericclemmons/npm-install-webpack-plugin/node_modules/webpack/lib/Compilation.js:229:38
    at onDoneResolving (/Users/Eric/Projects/ericclemmons/npm-install-webpack-plugin/node_modules/webpack/lib/NormalModuleFactory.js:29:20)
    at /Users/Eric/Projects/ericclemmons/npm-install-webpack-plugin/node_modules/webpack/lib/NormalModuleFactory.js:85:20
    at /Users/Eric/Projects/ericclemmons/npm-install-webpack-plugin/node_modules/async/lib/async.js:726:13
    at /Users/Eric/Projects/ericclemmons/npm-install-webpack-plugin/node_modules/async/lib/async.js:52:16
    at done (/Users/Eric/Projects/ericclemmons/npm-install-webpack-plugin/node_modules/async/lib/async.js:241:17)
    at /Users/Eric/Projects/ericclemmons/npm-install-webpack-plugin/node_modules/async/lib/async.js:44:16
    at /Users/Eric/Projects/ericclemmons/npm-install-webpack-plugin/node_modules/async/lib/async.js:723:17
    at /Users/Eric/Projects/ericclemmons/npm-install-webpack-plugin/node_modules/async/lib/async.js:167:37
    at /Users/Eric/Projects/ericclemmons/npm-install-webpack-plugin/node_modules/enhanced-resolve/lib/UnsafeCachePlugin.js:24:19

Crap, nevermind.  If I interrupted the build, then the watch never happens again.

Coverage increased (+0.2%) to 94.706% when pulling ecee4efc792481fa7e066493029d13980d0e96e6 on 48-ignore-erroneous-modules into 80d06ed3d124c294e9e2f7da501d2e1909fe59b2 on master.


Change to card image on All components page

Tugboat has finished building the preview for this pull request!
Link:

https://pr80-6rxai7fhkafdai9eo0p0djabortnp0ok.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/5f7f4e390ec0b43ff7086c48

Tugboat has finished building the preview for this pull request!
Link:

https://pr80-6rxai7fhkafdai9eo0p0djabortnp0ok.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/5f7f4e390ec0b43ff7086c48

Tugboat has finished building the preview for this pull request!
Link:

https://pr80-6rxai7fhkafdai9eo0p0djabortnp0ok.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/5f7f4e390ec0b43ff7086c48

Tugboat has finished building the preview for this pull request!
Link:

https://pr80-6rxai7fhkafdai9eo0p0djabortnp0ok.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/5f7f4e390ec0b43ff7086c48

Tugboat has finished building the preview for this pull request!
Link:

https://pr80-6rxai7fhkafdai9eo0p0djabortnp0ok.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/5f7f4e390ec0b43ff7086c48


any immediate future to add reusing discord attachment

using discord attachment instead of rehosting which is a huge burden on some host like glitch.com
https://i.imgur.com/wLmKwzL.png

if theres money incentive i would gladly pay.

even with a paid host its still rather inefficent to rehost when you can use reuse discord cdn attachment files. Eventually the files will get larger so you eventually have to get largher file space and the solution to upgrade is just not a good idea


Strip empty FastHealing details on extract

This doesn't auto-populate, so just clear it from existing REs.

This doesn't auto-populate, so just clear it from existing REs.

Done.


Test failed: httpdaddrGetList fails to execute when hostname contains capital letters

Describe the bug
httpdaddrGetList fails to execute when hostname contains capital letters
To Reproduce
Steps to reproduce the behavior:
1.git clone https://github.com/OpenPrinting/cups.git
2.Compile and make check
Expected behavior
httpdaddrGetList() executed successfully
Screenshots

Hi,
I can't reproduce with the latest version.
Can you try with the latest version? What is your OS?

Also, CUPS is just passing the hostname through getnameinfo, and DNS is case-insensitive.

No response here for 3 months, closing the issue.


'getOwnedTokenIds' function in sdk v5 is a repitition of 'getOwnedNFTs' function

As described in the docs, 'getOwnedNFTs' does this:
"Retrieves the owned ERC1155 NFTs for a given wallet address."
While 'getOwnedTokenIds' does this:
"Retrieves the owned ERC1155 tokenIds & the owned balance of each tokenId for a given wallet address."
But there is no 'getOwnedTokenIds'  function exported from the ERC1155 extensions module.
Is this a mistake in the docs or there is no such function?

Hi @Kennybabs-py, the return types of these two functions are slightly different. getOwnedTokenIds is exported from the ERC1155 extension. Please make sure you're on the latest version.


Partitions and partial indexing

Hello,
Exploring the differences between pgvector and pgvecto.rs. Pgvector supports partial indexing (https://github.com/pgvector/pgvector?tab=readme-ov-file#frequently-asked-questions), but I don't see any examples of that working with pgvecto.rs. Tried it myself, but getting:
ERROR:  pgvecto.rs: The given vector is invalid for input.
ADVICE: Check if dimensions and scalar type of the vector is matched with the index.

Here's the script I'm using to test.
-- Verify PgVecto.rs
drop extension if exists vectors;
create extension vectors;

-- Create table
drop table if exists embeddings;
create table if not exists embeddings (
    model_id varchar(27) not null,
    inference_id varchar(100) not null,
    embedding vector not null,
    unique(model_id, inference_id)
) partition by list (model_id);

-- Create partitioned tables of varying embedding size
create table if not exists model_a_embeddings partition of embeddings for values in ('a');
create table if not exists model_b_embeddings partition of embeddings for values in ('b');
\d+ embeddings;

-- Create function to produce random array of size N
create or replace function random_array(dim integer) returns float[] AS $body$
begin
	return (select array_agg(random()) from generate_series (0, dim));
end
$body$ language plpgsql;

-- Insert embeddings
truncate table embeddings;
with arrays_a as (
    select random_array(2048)::real[] as arr
    from generate_series(1, 5)
)
insert into embeddings (model_id, inference_id, embedding)
select 'a' as model_id, 'a' || row_number() over () as inference_id, arrays_a.arr as embedding
from arrays_a;

with arrays_b as (
    select random_array(4096)::real[] as arr
    from generate_series(1, 5)
)
insert into embeddings (model_id, inference_id, embedding)
select 'b' as model_id, 'b' || row_number() over () as inference_id, arrays_b.arr as embedding
from arrays_b;

-- Create indexes
create index on embeddings using vectors ((embedding::vector(2048)) vector_l2_ops) WHERE (model_id = 'a');
-- or
create index on model_a_embeddings using vectors ((embedding::vector(2048)) vector_l2_ops);

create index on embeddings using vectors ((embedding::vector(4096)) vector_l2_ops) WHERE (model_id = 'b');
-- or
create index on model_b_embeddings using vectors ((embedding::vector(4096)) vector_l2_ops);

Any guidance would be appreciated. I'm using: tensorchord/pgvecto-rs:pg16-v0.2.0.

We do have support of partition. The test is at https://github.com/tensorchord/pgvecto.rs/blob/8f99933672a38c039d74c73cf797fb9d799aec76/tests/sqllogictest/partition.slt#L4

Can you try specify vector dimension inside create table command?

random_array is not correctly implemented.
usamoi=# select random_array(5);
                                                       random_array
---------------------------------------------------------------------------------------------------------------------------
 {0.6110589837409373,0.6974253227728624,0.7136032046268621,0.0018840354192570796,0.05279517919396892,0.008055504751462772}
(1 row)

It emits an array of (n+1) dimensions. So you need to create index on column casted to vector(2049) and vector(4097).

Thank you @usamoi! Can't believe I missed that.


Use with Java 7?

Although the docs state that Java 8 is required, this invalidates its use in my corporate code base which is back on Java 7.   I do see that the class files are Java 7 compliant however.  Is it possible to run under Java 7?  Are there versions of the API that work under 7?

Java 8 is only really required when running under App Engine; otherwise Java 7 is the minimum version ( see https://github.com/GoogleCloudPlatform/google-cloud-java#java-versions ).

Sigh.  Sorry about that.  Not sure where I got Java 8 from.  Thanks for the quick response.

Although it runs, I can't get the following test program to run under 0.36, 37, or 38.  It runs under Java 8 however so I suspect that maybe a dependency has a requirement?
With Java 7 I'm getting the java.lang.ClassNotFoundException: org/eclipse/jetty/alpn/ALPN class not found error which implies that it is in a different classloader.
Test program:
public class TestMain {
	public static void main(String[] args) throws Exception {
		Class.forName("org.eclipse.jetty.alpn.ALPN", true, TestMain.class.getClassLoader());
		String projectId = "my-project";
		String topicId = "test-topic";
		String subscriberId = "test-main";
		ProjectSubscriptionName subsciptionName = ProjectSubscriptionName.of(projectId, subscriberId);
		// SubscriptionName subsciptionName = SubscriptionName.of(projectId, subscriberId);
		Builder builder = Subscriber.newBuilder(subsciptionName, new OurReceiver());
		GoogleCredentials credentials = GoogleCredentials.fromStream(new FileInputStream("/tmp/creds.json"));
		builder.setCredentialsProvider(FixedCredentialsProvider.create(credentials));
		Subscriber subscriber = builder.build();
		subscriber.startAsync().awaitRunning();
		subscriber.awaitTerminated(10000000, TimeUnit.MILLISECONDS);
	}
	private static class OurReceiver implements MessageReceiver {
		@Override
		public void receiveMessage(PubsubMessage message, AckReplyConsumer consumer) {
			System.out.println("received message: " + message.getMessageId());
			consumer.ack();
		}
	}
}

My pom is just:
	<dependencies>
		<dependency>
			<groupId>com.google.cloud</groupId>
			<artifactId>google-cloud-pubsub</artifactId>
			<version>0.38.0-beta</version>
		</dependency>
	</dependencies>
	<build>
		<plugins>
			<plugin>
				<groupId>org.apache.maven.plugins</groupId>
				<artifactId>maven-compiler-plugin</artifactId>
				<configuration>
					<source>1.7</source>
					<target>1.7</target>
				</configuration>
			</plugin>
		</plugins>
	</build>

This looks like a problem with the netty gprc 1.9.0.  What a mess.

I'm going to close this because it is an overlap with https://github.com/GoogleCloudPlatform/google-cloud-java/issues/3044


When listing file.etags in a table there is no bullet when there is only 1 tag

I have a page where I show all the meetings that I participated in (and made a note about that meeting) so I can quickly go back to a certain meeting if I need to know what was discussed.
The query for that is fairly easy.
table file.ctime as aangemaakt, file.etags as tags from #overleg
sort file.ctime desc

Where #overleg is dutch for #meeting.
It is working correctly but when I have only 1 tag added to a meeting it is shown without a bullet and when I have multiple tags is shows them with bullets which results in a strange presentation.
See the image below with only the file.ctime and file.etags part visible.

Could you change the behaviour to always use bullets?
On a side note: Is it possible to exclude the #overleg from the file.etags field as that tag is already defining the selection in the from part of the query. So it is redundant.

Fixed for next release. As for the exclusion, will probably add list functions in the future so you can do something like file.etags - '#overleg'.

Fixed; lists always render as lists now.

@blacksmithgu ... Thanks for looking into this, the new presentation style is very ... space consuming. I was looking for a consistent way to present the tags ect. The adding of values and the number of values for the field (in this case tag) seems to be for me no valuable information. We know it is the tags fields and that it contains values, so values add's no information and we can see the number of tags. Only with a large number of tags on a page this 'might', in some cases, become usefull.
The example above now becomes this ...

Besides that other fields (created inline in the file) are still presented in the old style. With no bullet when there is only 1 and bullets when there are more values.

Hmm, that's because of a separate bug. Fixed in 0.3.7.


fix renovate config

Should fix #586
I think this should take care of skipping the packages.
https://docs.renovatebot.com/configuration-options/#matchpackagenames
Running this on the root of the project gives information for the configuration.
❯ npx renovate --platform=local

Saw these warnings but didn't take any actions
 WARN: Found renovate config warnings (repository=local)
       "warnings": [
         {
           "topic": "Configuration Error",
           "message": "The \"onboarding\" option is a global option reserved only for Renovate's global configuration and cannot be configured within a repository's config file."
         },
         {
           "topic": "Configuration Error",
           "message": "The \"platform\" option is a global option reserved only for Renovate's global configuration and cannot be configured within a repository's config file."
         },
         {
           "topic": "Configuration Error",
           "message": "The \"repositories\" option is a global option reserved only for Renovate's global configuration and cannot be configured within a repository's config file."
         },
         {
           "topic": "Configuration Error",
           "message": "The \"requireConfig\" option is a global option reserved only for Renovate's global configuration and cannot be configured within a repository's config file."
         }
       ]

Hey, thanks @stavros-k !


Is it possible to temporarily disable OAUTH2?

Problem statement
Is it possible to temporarily disable OAUTH2? Currently I'm getting a
{
"code": 401,
"message": "unauthenticated for invalid credentials"
}

But I would like to implement OAUTH later.

at this point there is no easy way to disable auth short of removing it from your spec


When will the code be released?

Hello! @hxwork
Thank you very much for your excellent work. When are you going to release the HandBooster training code?

Hi,
Thanks for your interest! We plan to release the code in early June.

Thank you for your prompt response!  We eagerly anticipate the release of your open source code.

When are you going to release the HandBooster training code?

Hi,
Thanks for your interest! We plan to release the code in early June.

So when？

Hi,
We released the key components for your reference. The current repo still has some issues due to the transfer from server to local, especially for the data.


Accessibility support

Want to know if chart.js is 100% accessibility compliance. Supports level A and AA as per WCAG 2 standards.

You create the canvas element yourself and Chart.js populates it with content. When you create the canvas object you will need to do it in a way that is accessible. Two examples are below which are from http://pauljadam.com/demos/canvas.html:
Example 1:
<canvas id="chart" height="400" width="650" role="img" aria-label="Bar Chart Values in Millions from 0 to 12000. Human: 11000, Chimp: 6200, Dolphin: 5800, Cat: 300"> </canvas>

Example 2:
    <canvas id="can2" height="400" width="650">Text alternative for this canvas graphic is in the data table below.</canvas>

      Neurons in Cerebral Cortex

          Value
          Human
          Chimp
          Dolphin
          Cat

          Millions
          11000
          6200
          5800
          300

We should add information about this to the documenation


Handle GLOBAL_DEF overload

GLOBAL_DEF has two overloads. One takes a string, and one takes a PropertyInfo. The latter is currently not handled so some entries in the project settings are not on Weblate.
This adds about 100 msgids:
#   Additions: 162 msgids.
#   Deletions: 33 msgids.

Thanks!


YouTube plugin has issues with spaces???

E.g.
.youtube litecoin

works great but
.youtube litecoin association

note: some things with spaces work fine. above term breaks it, even though "litecoin association" returns results on youtube just fine.
causes an exception:
2015-05-10 22:23:59,113 - cardinal.plugins - INFO - Calling callbacks for event: irc.privmsg
Unhandled Error
Traceback (most recent call last):
  File "/home/azureuser/.local/lib/python2.7/site-packages/twisted/protocols/policies.py", line 120, in dataReceived
    self.wrappedProtocol.dataReceived(data)
  File "/home/azureuser/.local/lib/python2.7/site-packages/twisted/words/protocols/irc.py", line 2430, in dataReceived
    basic.LineReceiver.dataReceived(self, data.replace('\r', ''))
  File "/home/azureuser/.local/lib/python2.7/site-packages/twisted/protocols/basic.py", line 571, in dataReceived
    why = self.lineReceived(line)
  File "/home/azureuser/.local/lib/python2.7/site-packages/twisted/words/protocols/irc.py", line 2438, in lineReceived
    self.handleCommand(command, prefix, params)
--- <exception caught here> ---
  File "/home/azureuser/.local/lib/python2.7/site-packages/twisted/words/protocols/irc.py", line 2482, in handleCommand
    method(prefix, params)
  File "/home/azureuser/Cardinal/cardinal/bot.py", line 150, in irc_PRIVMSG
    self.plugin_manager.call_command(user, channel, message)
  File "/home/azureuser/Cardinal/cardinal/plugins.py", line 695, in call_command
    command(self.cardinal, user, channel, message)
  File "/home/azureuser/Cardinal/plugins/youtube/plugin.py", line 62, in search
    message = self._parse_item(result['items'][0])
  File "/home/azureuser/Cardinal/plugins/youtube/plugin.py", line 110, in _parse_item
    video_id = video_id['videoId']
exceptions.KeyError: 'videoId'

Fixed. It was bringing up channels in search results, instead of just videos. :)


Feature hmget

Closes #650

Hi @apoorvyadav1111, I've added the comments. Let me know if those are fine.
Yes #705 is a duplicate, I've gone ahead and closed it.

@vishnuchandrashekar The PR is almost ready, let's try to address the comments and get it submitted soon.

Hey @JyotinderSingh , I've updated the tests and the addressed the comments. Please have a look.


Table 295 and 302 - New Functions and publisher for NoSeries handling

We need new functions TestNoSeries / GetNoSeriesCode / GetPostingNoSeriesCode with publishers OnBeforeTestNoSeries (see issue #1151) / OnAfterTestNoSeries / OnAfterGetNoSeriesCode / OnAfterGetPostingNoSeriesCode in these tables according to the functionality in Table 36 and 38.
In fact these functions should be added consistent in every table, where No. Series are used.

From my Point of view there are some Errors in the implementation of the Publishers in BC14 CU1 both in Table 295 and Table 302:

OnAfterGetNoSeriesCode: Parameter NoSeriesCode should be VAR
OnAfterGetIssuingNoSeriesCode: Parameter IssuingNos should be VAR
OnBeforeGetNoSeriesCode: Parameter NoSeriesCode and IsHandled should be VAR


HBASE-26308 Sum of multiplier of cost functions is not populated prop…

…erly when we have a shortcut for trigger
Moved the loading to the time configs are (re)loaded.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 25s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+1 :green_heart:
mvninstall
4m  3s
master passed

+1 :green_heart:
compile
0m 22s
master passed

+1 :green_heart:
shadedjars
8m 18s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 19s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 46s
the patch passed

+1 :green_heart:
compile
0m 20s
the patch passed

+1 :green_heart:
javac
0m 20s
the patch passed

+1 :green_heart:
shadedjars
8m 15s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 16s
the patch passed

_ Other Tests _

-1 :x:
unit
4m 49s
hbase-balancer in the patch failed.

32m  7s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3709/1/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3709

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 33e8df9a303b 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 121bdea230

Default Java
AdoptOpenJDK-1.8.0_282-b08

unit
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3709/1/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-hbase-balancer.txt

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3709/1/testReport/

Max. process+thread count
342 (vs. ulimit of 30000)

modules
C: hbase-balancer U: hbase-balancer

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3709/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 13s
Docker mode activated.

Open a new pull request.https://github.com/apache/hbase/pull/3710


Does this support a jsonAPI Pagination?

http://jsonapi.org/format/#fetching-pagination
Is there a way to retrieve the next and prev in link field.

#74 FYI.
The library do not provide out-of-box support to pagination, but you can implement one with JsonBuffer object.


Cannot swap shop pages.

I want to switch a shop page but that's not a feature. Also the way the config is managed it's hardly possible to do it manually.
Thanks in advance

What do you mean exactly? Do you mean it like being able to switch from shop page to another shop page?

The option to change the order of the shop pages.
Example:
I have three pages. page 1, 2 and 3
I want to be able to change the order to 2, 1, 3

I believe in the config you can set startpage to be the default to pop up when doing /shop. On there you can create custom buttons leading to your shops, but also to run any other command. You can move these with your middle mouse button

I am talking about when you're inside a shop.

Premium version includes this feature


How to delete the exported files?

Hi,
I would like to know if is there any way to delete these exported files? I have set up BBB to delete files if older than 45 days, so when the file is deleted, what will happen with the exported file?
And thank you so much for this great work, it saves me a lot of time.

Hi @dqcuong93,
I'm not sure if I understand your question, but if you delete the recording, the exported video file/PDF will be deleted too, just  as is the case with the other formats (e.g. presentation) provided by BBB.

Sorry for the late reply, I have been busy with Canvas because the whole database is lost, and my client is not happy about that. Thank you for your answer, I understand now.


build error

I am trying to build armnn on ubuntu18.04
Scanning dependencies of target armnnAclCommon
[ 15%] Building CXX object src/backends/aclCommon/CMakeFiles/armnnAclCommon.dir/ArmComputeTensorUtils.cpp.o
In file included from /home/zmou/armnn/src/backends/aclCommon/ArmComputeTensorUtils.cpp:6:0:
/home/zmou/armnn/src/backends/aclCommon/ArmComputeUtils.hpp: In function ‘arm_compute::ActivationLayerInfo::ActivationFunction armnn::ConvertActivationFunctionToAclActivationFunction(armnn::ActivationFunction)’:
/home/zmou/armnn/src/backends/aclCommon/ArmComputeUtils.hpp:68:79: error: ‘HARD_SWISH’ is not a member of ‘AclActivationFunction {aka arm_compute::ActivationLayerInfo::ActivationFunction}’
se ActivationFunction::HardSwish:     return AclActivationFunction::HARD_SWISH;
^~~~~~~~~~
/home/zmou/armnn/src/backends/aclCommon/ArmComputeTensorUtils.cpp: In function ‘arm_compute::DataType armnn::armcomputetensorutils::GetArmComputeDataType(armnn::DataType, bool)’:
/home/zmou/armnn/src/backends/aclCommon/ArmComputeTensorUtils.cpp:21:43: error: ‘BFLOAT16’ is not a member of ‘arm_compute::DataType’
return arm_compute::DataType::BFLOAT16;
^~~~~~~~
src/backends/aclCommon/CMakeFiles/armnnAclCommon.dir/build.make:62: recipe for target 'src/backends/aclCommon/CMakeFiles/armnnAclCommon.dir/ArmComputeTensorUtils.cpp.o' failed
make[2]: *** [src/backends/aclCommon/CMakeFiles/armnnAclCommon.dir/ArmComputeTensorUtils.cpp.o] Error 1
CMakeFiles/Makefile2:850: recipe for target 'src/backends/aclCommon/CMakeFiles/armnnAclCommon.dir/all' failed
make[1]: *** [src/backends/aclCommon/CMakeFiles/armnnAclCommon.dir/all] Error 2
Makefile:129: recipe for target 'all' failed
make: *** [all] Error 2

Hi @zx854217561 it looks like you're building with the master branch of Arm NN, but an older version of Compute Library.
If you need the master branch of Arm NN then you will also need a recent unreleased version of Compute Library. (From the development gerrit repo at review.mplatform.org)
Alternatively, you could use the latest released version of Arm NN (20.02) which will work fine with the corresponding Compute Library release (20.02)
Hope that helps,
Matthew

hi@MatthewARM
I follow with the https://github.com/ARM-software/armnn/blob/branches/armnn_20_02/BuildGuideCrossCompilation.md
may be i git the wrong version?
which version should i use?

Hi @zx854217561 it looks like you're building with the master branch of Arm NN, but an older version of Compute Library.
If you need the master branch of Arm NN then you will also need a recent unreleased version of Compute Library. (From the development gerrit repo at review.mplatform.org)
Alternatively, you could use the latest released version of Arm NN (20.02) which will work fine with the corresponding Compute Library release (20.02)
Hope that helps,
Matthew

hi@MatthewARM
I follow with the https://github.com/ARM-software/armnn/blob/branches/armnn_20_02/BuildGuideCrossCompilation.md
may be i git the wrong version?
which version should i use?

Strange, those instructions should get the 20.02 release, I'll check over them.
In any case, you can fix it by checking out the 20.02 release branch:
cd armnn
git checkout branches/armnn_20_02
git pull

Strange, those instructions should get the 20.02 release, I'll check over them.
In any case, you can fix it by checking out the 20.02 release branch:
cd armnn
git checkout branches/armnn_20_02
git pull

thank u so much

Both BuildGuideAndroidNDK.md  and BuildGuideCrossCompilation.md have been updated for the 20.05 release.
https://review.mlplatform.org/c/ml/armnn/+/3224
Colm.


Adding File Integrity Operator

This role has dependency on the cluster-health-check role.
Signed-off-by: Varad Ahirwadkar<EMAIL_ADDRESS>

/approve


Syntax highlighting does not cover the entire file

I have a fairly large handler library I used for resizing windows and I noticed that the syntax highlighting does not cover the entire file.  In addition, those handlers that are not highlighted do not appear in the list when I trigger the goto Symbol.
A large library is probably a bad idea and should be refactored, but in the meantime, it might be simpler not to have this limitation?
My library is 1004 lines with comments, inline unit tests.

I don't think that this an issue related to this package, but an issue with Sublime Text itself. I'm aware of limitations when it comes to very long lines, but not with long files. Maybe the parser takes a bit longer when it comes to long files?

I'm not really sure because I work on a legacy project with large files (10k lines) but I never encountered this issue. I've never worked on Sublime Text plugin but could try to dig when I get some free time and see if I can find anything. BTW thank you for your plugin, I find it very useful.


Feature: Round Corners

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Bug fix
Why was this change needed? (You can also link to an open issue here)
fixes #514
Other information:

@holopin-bot @SoumyadiptoPal  Just as rounded as we need it, thank you!


Re-introducing the graphic feature

I found an issue from almost a year ago #151 where the graphic extension is discussed.
However, it has been removed since then from the commander.js.
I couldn't find the version when this was changed, but I would be very interested in seeing it reintroduced.
Is anything planned in this regard?

Not really, it was complexifying the project too much so I took it off. It's best to use a translucent window overlay instead. I have no plans to bring it back, if you really need this feature, I recommend asking on the lines forum how different people achieve this in their videos.


[Improvement] Document component-level attributes

Description
It would be helpful to have general documentation for all base-level attributes available on any sage_component. This could be similar to the spacing page on the docs site.
test_id
html_attributes
spacer
css_classes
These attributes have been added for some components individually, but it's not immediately noticeable that all components can take advantage.

MOved to Jira https://kajabi.atlassian.net/browse/UXD-79


url_launcher cannot run with flutter SDK >= 2.7 < 3.0.0

url_launcher >=3.0.0 requires Flutter SDK version >=0.1.4 <2.0.0 and url_launcher <3.0.0 requires SDK version <2.0.0, url_launcher is forbidden

@duyjack
Please provide flutter doctor -v and pubspec.yaml.
Thanks.

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


[wasm] Add metadata registration SwiftRT-WASM.cpp

This patch adds the metadata registration for the wasm targets, and also adds build support for it.

@swift-ci Please smoke test

Can anything here be deduped from the ELF implementation? They two implementations are very similar.

Good point! I think the list of section names can be shared by X-macro.

@grynspan Thank you for feedback and sorry for my slow response (I was sick last week.. 😵)
I updated my patch to share the most of the content among ELF and Wasm versions :)

@swift-ci Please smoke test

This looks really good to me! Please be sure to get a review from the core Swift team before merging. :)

Thank you all for your kind reviews!

LGTM.
I wasn't 100% sure about the ELF and WASM versions sharing the .cpp file, but it looks like the implementation is very similar so I can see why you did that.

We can actually probably merge the COFF implementation in too, and then this file gets renamed "SwiftRT.cpp" and world peace is achieved.


Missing Stripe documentation

Is your improvement request related to a specific problem in the code or project structure? Please describe. 👨💻
Currently, it may be hard to run the application locally, as the application requires you to run a test stripe environment, with the added tunnel from your local machine to the stripe servers.
Additionally, there are 3 .env variables that are not well documented, and might be difficult to understand at first glance.
Describe the improvement or refactor you would like 🎯
This should be in the README file.
Containing the following:

How to setup a Test Stripe account (without having to provide banking information)
How to populate the .env variables (all of which are done from your stripe account):
3. NEXT_PUBLIC_STRIPE_PUBLISHABLE_KEY
4. STRIPE_SECRET_KEY
5. WEBHOOK_SECRET
The prerequisites required to run the command stripe:listen command in the package.json
a. By that I mean, installing the stripe cli for example
Finally, update the run locally section to include the stripe:listen command

Additional context or comments
Most of this can most likely be done by referencing Stripe's own documentation 🤔

I would like to work on this issue,

For setting up a Test Stripe account, would you want me to add a step-by-step guide or an article link that demonstrates how to set up Test Stripe account

If we include the article link, populating the .env file will be easy. Otherwise, we need to explain it in a readme file, But I think the readme file will be lengthy.

There is an extension available in the VS Code marketplace for Stripe CLI, which makes it easier for anyone to setup Stripe commands

Let me know what do you think?

@themujahidkhan by all means! ❤

Yes, I agree, that it would be better to link to their documentation about Setting up a Test Stripe Account;

Again, I agree, that linking to their documentation on populating the two .env variables (public key and secret) is a good idea; perhaps we can link to a related section within a guide relating "Connecting Stripe to NextJS" to simplify the process?

Currently there is a script in the package.json, called stripe:listen, which will automatically start listening for callbacks from Stripe. When you run the command, it also automatically displays the final .env key (the webhook related one) - that the user can plop right in to the .env file. However, it does require the user to have the stripe cli installed. One danger about relying on vs code extensions, is that not everyone uses vs code, given the rise of new IDEs 😳

One note on why the stripe:listen command is favoured, is because it also auto routes the callback to an endpoint within the app 😄
Granted, this also means we need to specify that the stripe:listen command as the 8th step in the run locally numeric guide:

Hey @suuunly
Screenshot for the changes

Looks good 😄
The only changes I would suggest, are:

Instead of going through the lengthy process of explaining how to get the webhook secret, you can say "run the yarn stripe:listen command in the project - this will reveal the webhook secret"
The final step in the Run locally section, should use yarn instead of npm run, and could be shortened to:
"Run yarn stripe:listen in a separate terminal to listen for stripe changes"

Looks good 😄 The only changes I would suggest, are:

Instead of going through the lengthy process of explaining how to get the webhook secret, you can say "run the yarn stripe:listen command in the project - this will reveal the webhook secret"
The final step in the Run locally section, should use yarn instead of npm run, and could be shortened to:
"Run yarn stripe:listen in a separate terminal to listen for stripe changes"

Done the changes,
Screenshot for Stripe

Screenshot for Run locally

Oh, sorry for the confusion. I meant that when you run the yarn stripe:listen command, it shows you your webhook key. Like so:

So in step 8 of your first screenshot, you can flip the two tasks. 😛 Run the command, and theeen then fill out the WEBHOOK_SECRET 👍
Also, just a minor thing, regarding your second screenshot, could you simplify the final line to just say yarn stripe:listen, and maybe make it inline. It is mostly just to keep the format across the 8 steps consistent 😁
But other than that, it looks good 🥳

oh one detail that is missing, is the prerequisite that you need to have the CLI installed on your machine 🤔
You can make it a quick one liner saying install the Stripe CLI on your machine, and refer to this link:
https://docs.stripe.com/stripe-cli
😁

I've made changes as suggested,
Prerequisite

Setting up stripe account

Run Locally

That looks great. The absolute fiiiinal note would be to mention the WEBHOOK_SECRET in the step 8, in the stripe account screenshot 😁
If that is done, then I'd say it is done 🥳

That is perfect! 👍🎉

Done, I've opened a PR

It has been approved and merged in 😄


Unclean a closed connection by close() websocket's method (1006)

Hi!
Sorry for my English :)
I have PHP (php 7.0) websocket server and JavaScript client.
When server has received a connection and anyone message from JS-client, WS-server is sending the close command to client.
$ws->onMessage = function($connection, $data)
{
 $connection->send('I am closing the connection\n');
 $connection->close('Closing');
};
But JS-client has received a code "1006" and variable wasClean == false, also JS-client is not received a reason of closed (because connection has uncleanly closed).
CloseEvent {isTrusted: true, wasClean: false, code: 1006, reason: "", type: "close", …
How can I close the connection cleanly?

Sorry, I can't reproduce this problem.
Is there any special operation?

No. I haven't called ws.close() at JS-client.
But I have tried it.
I have removed $connection->close('Closing'); in the Server-side and rebooted it.
Next I've tried to call ws.close() in the client-side (JavaScript), but I've received a code 1006 from ws.onclose.
Code:
$(document).ready(function(){
  if ("WebSocket" in window) {
  var ws_url = 'ws://*MyIPisDEL*/websocket/';
  var ws = new WebSocket(ws_url);
  ws.onopen = function() {
  ws.send('Good!');
  };
  ws.onerror = function(e) {
  console.log(e);
  };
  ws.onmessage = function(e) {
  console.log(e.data);
  ws.close(1000, 'Test reason');
  };
  ws.onclose = function(e) {
//always return code is 1006
  console.log(e);
   };
  } else {
   alert("No support a websocket");
  }
 });

I think you can send a message to js client tell the client to close the ws connection by it self.

have same problem on php 8.1


gmc: add oauth2-proxy support for authentication and authorization

Description
Support GUI-based authentication and authorization with the help of Istio, oauth2-proxy and OIDC provider.
Issues
#169
Type of change
List the type of change like below. Please delete options that are not relevant.

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

Dependencies
oauth2-proxy
Tests
n/a

Please do NOT merge before #260 got merged

@Ruoyu-y plz remove the draft status from this PR so that we can get it merged


Add raven_emu sensor platform

Description:
Implements support for the Rainforest RAVEn and EMU for monitoring electricity
usage and billing on smart meters.
Only tested with the RAVEn and PG&E in northern California, running on Linux.
Haven't done the documentation yet, but I figured I'd kick off the code review portion to make sure things are sane.
Pull request in home-assistant.github.io with documentation: home-assistant/home-assistant.github.io#[pending]
Example entry for configuration.yaml:
sensor:
    - platform: raven_emu
      device: /dev/ttyUSB0

Checklist:
If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] New dependencies have been added to the REQUIREMENTS variable (example).
[x] New dependencies are only imported inside functions that use them (example).
[x] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[x] New files were added to .coveragerc.

Looks like I have some delinting to take care of...

I'm obviously not that familiar with the hass architecture, but isn't the point of a platform to handle the device-specific stuff?
I'm assuming you're suggesting breaking out just the Device class (65 lines) -- that code implements a tiny subset of the raven/emu interface and wouldn't be very useful as a separate library.

Looks like CI failed because the linter picked up unrelated stuff in hass core...

@dnschneid Home Assistant does not contain protocol specific information. Instead, platforms will communicate with standalone Python libs that contain the protocol info. That way we don't tie people to use Home Assistant perse but they can also just use the 3rd party lib if they just want to communicate with the device.

I see, that makes sense, thanks.  There's an official python implementation here, but it's poorly written and doesn't work in python3, which is why I did a partial reimplementation for this module.
I don't have the test cases or the interest in turning the partial implementation of the API into a proper, full implementation that can be released standalone, so I'll just leave this here as-is.  If someone else is interested in taking the code and running with it, be my guest.


Inconsistent tests

Multiple times I've see individual tests fail on Travis, been unable to reproduce them, and when I re-ran on Travis they were gone. I want keep a list of them here, and if particular ones are more frequent than others, I want to note that.
0.0.57 in progress, Django2 environment, Travis fails:
tests.test_live.PathRoutingMechanicalSoupTests.testDebuggerLaunch
tests.test_live.HostRoutingClientTests.test_please_wait

======================================================================
ERROR: test_please_wait (tests.test_live.HostRoutingClientTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/travis/build/refinery-platform/django_docker_engine/django_docker_engine/container_managers/docker_engine.py", line 156, in get_url
    http_port_info = port_infos['{}/tcp'.format(container_port)]
KeyError: '80/tcp'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/travis/build/refinery-platform/django_docker_engine/tests/test_live.py", line 137, in test_please_wait
    labels={'subprocess-test-label': 'True'}
  File "/home/travis/build/refinery-platform/django_docker_engine/django_docker_engine/docker_utils.py", line 241, in run
    return self.lookup_container_url(container_spec.container_name)
  File "/home/travis/build/refinery-platform/django_docker_engine/django_docker_engine/docker_utils.py", line 83, in lookup_container_url
    return self._containers_manager.get_url(container_name)
  File "/home/travis/build/refinery-platform/django_docker_engine/django_docker_engine/container_managers/docker_engine.py", line 160, in get_url
    container_name, container_port, port_infos
django_docker_engine.container_managers.docker_engine.ExpectedPortMissing: On container container-name, port 80 is not available, but these are: {}

======================================================================
ERROR: testDebuggerLaunch (tests.test_live.PathRoutingMechanicalSoupTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/travis/virtualenv/python3.6.3/lib/python3.6/site-packages/urllib3/connectionpool.py", line 595, in urlopen
    chunked=chunked)
  File "/home/travis/virtualenv/python3.6.3/lib/python3.6/site-packages/urllib3/connectionpool.py", line 393, in _make_request
    six.raise_from(e, None)
  File "<string>", line 2, in raise_from
  File "/home/travis/virtualenv/python3.6.3/lib/python3.6/site-packages/urllib3/connectionpool.py", line 389, in _make_request
    httplib_response = conn.getresponse()
  File "/opt/python/3.6.3/lib/python3.6/http/client.py", line 1331, in getresponse
    response.begin()
  File "/opt/python/3.6.3/lib/python3.6/http/client.py", line 297, in begin
    version, status, reason = self._read_status()
  File "/opt/python/3.6.3/lib/python3.6/http/client.py", line 266, in _read_status
    raise RemoteDisconnected("Remote end closed connection without"
http.client.RemoteDisconnected: Remote end closed connection without response


Enhance the programmatic access to the KV secret engine

The purpose of this pull request is to enhance the programmatic access to the KV secret engine, to give it the ability to access a multi mount KV secret engine, it comes that in some production environnement secrets are stored on different mounts for different domains, and this is not allowed in the current version of Quarkus vault extension, even when using the programatic way it comes always to retrieve the secrets only the configured mount set up in the properties file
In the current pull request we overloaded some methods to give the extension this ability, means if we have three mounts for example we can read from them programmatically even if we configured a default mount to read from

The PKI engine uses a different method. It simply allows creating an "engine" for a given "mount"; instead of overloading every conceivable method.
I think that's the model to follow.

@kdubb  Yes it's a good idea I need a name for this engine, if you can help on finding a name for it it will be great for me, yes I agree it's better to define a new engine rather then overload the existing engine and method, notice that the other engine will always be of kind KV

You don't need a new engine. You just need to allowing creating VaultKVManager with a specific mount point (like PKI does).

@kdubb  I can get it now, I'll give it a try

@kdubb I made some changes for the pull request now the Oliver loaded methods are still here, it may be a kind of shortcut, in the next tastemaker I'll remove it

@kdubb I updated the pull request and I also rebased it please can you give it a check, Thanks in avance

@kdubb please provide me with a feedback after the latest changes

@bmscomp Not ignoring you or your contribution. I am swamped with current work and it's going to be a while before I can weigh in on this.

@kdubb  Thank you su much for the feedback

@kdubb  I'll rebase the brach and solve conflicts and then take care of the comment's reviews by the end of day

@bmscomp Please look at #214 & #215. It does this and a whole lot more.
If you can please build it locally and comment on #214 with ideas and/or #215 with specific suggestions for changes.

@kdubb Thanks for your answer, I checked the great work done to bring up a new client to vault extension for Quarkus, and I think the content of this pull request  still remains important as an evolution to the programmatic way to support multi without any breaking changes to the existing way to use a programatic way to get vault secrets, we can merge this pull request then after try to enhance the api with your implementation of vault client, since we still need in some cases something strongly coupled with Quarkus, this will also enrich the existing API, Please would you consider the updated review
Thanks in advance

@kdubb @vsevel  please any feedback about the current pull request

@bmscomp The new client is still deeply integrated into Quarkus. It has the added ability to be run standalone without using Quarkus static configuration. Additionally, all of the apis that support a dynamic mount path (e.g., any "secret" api like kv) can be easily accessed.
For example, to access a kv engine mounted at a specific path using the new client you would do this:
@Inject VaultClient client; // Inject Quarkus configured client

Map<String, Object> getSomeSecret(String mount) {
   return client.secrets().kv(mount).read("something").getData();
}

If you want a "preconfigured" mount you can create a CDI @Producer for it.
@Produces
VaultSecretsKV2 getSecretAPI(VaultClient client, @ConfigProperty("app.kv.mount") String mount) {
   return client.secrets.kv(mount);
}

And use it like so:
@Inject VaultSecretsKV2 kv2;

Map<String, Object> getSomeSecret() {
  return kv2.read("something").getData();
}

Given the advantages of the new client. The current thinking is to deprecate and remove the current "engines" because having and maintaining two separate implementations is a burden.
Currently PR #215 re-implements the "engines" using the new client simply to allow using it in its current form and validate that the new client is solid and performing correctly.
You are advocating for enhancing the current KV engine at the same time. This would keep users using the "old" system when we want them to be replacing it by using the new client approach.

@bmscomp I understand that this new client PR undermines the work you've done in this PR, but it was all the work that you were required to do that inspired me to get to work on the new client.
Transforming each engine into one that can be both statically configured and dynamically configured, as you now know, is quite a bit of work. Additionally, the CDI model is a bit complicated to ensure both a reactive and non-reactive api exists for everything.
I wanted to get away from this model... quickly. It slows down development and introduces a maintenance headed in the long run. The new client solves all these problems and more, all the while making development, whether adding new APIs or maintaining old ones, easier.

@kdubb  Then would you merge your pull request

Waiting for review. I've been in contact with @vsevelv and he's going to look at it early this week.

@kdubb  Can you please make public my answers to your reviews, at least other developers or reviews or developers can suggest more

@bmscomp I have no idea what you're requesting. I believe everything in Quarkiverse projects is public (and we team members do not control the project settings, that's done by the Quarkus admins).

@kdubb  it is totally my mistake apologies, I just could not submit them I think

Please @gsmet @kdubb  Can you take a look at this pull request, it's updated regarding the last comments and we really need this feature

Any news about this pull request ?

4.0.0-alpha.2 has been released. Injecting the VaultClient (via @Inject VaultClient client) and using client.secrets().kv1(mountPath) or client.secrets().kv2(mountPath) allows you to supply any mount path.
I'm closing this now that dynamic mounts should use the client directly instead.


Add build status badges for travis and appveyor

the travis job failed with:
Paket failed with:
	Could not retrieve data from https://api.github.com/repos/fsharp/FAKE/commits/master
 Message: The remote server returned an error: (403) Forbidden.

but isn't this a mistake on travise's part? can we do something about it?

?)

oh, now everything seems to be ok


Show cumulative file size for recursive inscriptions

As we see more recursive inscriptions consuming files of various sizes, I thought maybe it'd be informative to display the total size of all recursive elements consumed by an inscription.
Have some that are like 8mb combined by a small html file, and it is a little opaque and difficulty to see total file size.

It isn't possible, in general, to determine the size of recusive inscriptions. For HTML inscriptions, we could parse the HTML, and see if it links to /content. However, this would be expensive. And there are JS inscriptions, which refer to /content, but those are in code, and it would be impossible to find those references.


crypto/internal/nistec: incorrect number of loops in p256SelectAffine of s390x and ppe64le asm

Go version
Go1.19+
Output of go env in your module/workspace:
NA

What did you do?
according the precomputed p256AffineTable and the definition of function p256SelectAffine:
// p256AffineTable is a table of the first 32 multiples of a point. Points are
// stored at an index offset of -1 like in p256Table, and [0]P is not stored.
type p256AffineTable [32]p256AffinePoint

// p256Precomputed is a series of precomputed multiples of G, the canonical
// generator. The first p256AffineTable contains multiples of G. The second one
// multiples of [2⁶]G, the third one of [2¹²]G, and so on, where each successive
// table is the previous table doubled six times. Six is the width of the
// sliding window used in p256ScalarMult, and having each table already
// pre-doubled lets us avoid the doublings between windows entirely. This table
// MUST NOT be modified, as it aliases into p256PrecomputedEmbed below.
var p256Precomputed *[43]p256AffineTable

// p256SelectAffine sets res to the point at index idx in the table.
// idx must be in [0, 31]. It executes in constant time.
//
//go:noescape
func p256SelectAffine(res *p256AffinePoint, table *p256AffineTable, idx int)

The correct number of loops in p256SelectAffine should be 32 if takes one point at a time.
What did you see happen?

https://github.com/golang/go/blob/36b45bca66d86020f0b4daf1f15b02921a8dad43/src/crypto/internal/nistec/p256_asm_s390x.s#L511
https://github.com/golang/go/blob/36b45bca66d86020f0b4daf1f15b02921a8dad43/src/crypto/internal/nistec/p256_asm_ppc64le.s#L444
https://github.com/golang/go/blob/36b45bca66d86020f0b4daf1f15b02921a8dad43/src/crypto/internal/nistec/p256_asm_ppc64le.s#L294

This will impact ScalarBaseMult performance.
What did you expect to see?

https://github.com/golang/go/blob/36b45bca66d86020f0b4daf1f15b02921a8dad43/src/crypto/internal/nistec/p256_asm_s390x.s#L511
should be 33.
https://github.com/golang/go/blob/36b45bca66d86020f0b4daf1f15b02921a8dad43/src/crypto/internal/nistec/p256_asm_ppc64le.s#L444
should be 32.
https://github.com/golang/go/blob/36b45bca66d86020f0b4daf1f15b02921a8dad43/src/crypto/internal/nistec/p256_asm_ppc64le.s#L294
should be 16.

BTW, for s390x asm implementations:

Try to use VLM/VSTM;
https://github.com/golang/go/blob/36b45bca66d86020f0b4daf1f15b02921a8dad43/src/crypto/internal/nistec/p256_asm_s390x.s#L579
Can use VLEF directly since Go 1.19+.

cc @laboger . BTW, it seems p256NegCond of ppc64le will operate in variable-time.

ccing @pmur who is the current maintainer of Go on ppc64le as Lynn recently retired.

The ppc64 code does look quietly wrong. It was derived from the s390 implementation. I would have expected sporadic segfaults as this would imply its reading 2KB beyond the limits of table, but never using such values.

cc @golang/s390x @golang/ppc64 @golang/security