Plans to add a "plugin remove_jar" command?

I downloaded the latest presto-admin tool and realized there was plugin add_jar. How about if I want to remove the jar? :)
Any plans to add it to the tool any time soon? Thanks.

Thanks for opening the request! No one is currently working on this feature right now. We can let you know when someone picks it up.
If you're interested in adding it, we'd love to start getting more contributions from the presto community! Let us know and we can help.

Cool..I will like to contribute to the community. Is there a process in place currently or once I pull the repo and make changes I can push it back in for review before it becomes public?
Please advise

So glad you'd like to contribute!   We work on pull requests, so to contribute, you should create a fork of this repo, and put your changes there.  Then create a pull request with your changes and we'll be able to review it and merge them in.

@nanaessuman How are things going? Let us know if you need any help.


feat(node-18): make node 18 the minimum requirement

Current dependencies on/for this PR:

main

PR #39

PR #40

PR #42

PR #43   👈

PR #41

This comment was auto-generated by Graphite.

Current dependencies on/for this PR:

main

PR #39

PR #40

PR #42

PR #43   👈

PR #41

This comment was auto-generated by Graphite.


Missing examples in documentation for text prompt

Link
There are no code examples on this page.

@OronDF343 Has the lack of examples stopped you from using the text prompt? If not, how did you figure out the usage? (I'm trying to triage the priority of this issue)

@FrankRay78 speaking of my personal experience on the issue, the lack of examples on the main website lead me to the github to try and figure out how to use it, my eyes glanced at the examples folder and so i opend it and saw the prompts example, one thing i figured out was that there are some options, one i can use AnsiConsole.Ask to ask a question in the terminal, but also the same thing could be implemented by just puting a new TextPrompt in a Prompt, which beged the question wether its even needed to use the Ask because you might need to customize the prompt for example to allow empty strings and defaults which are not included in Ask, but also imo there arent a lot of variations of the uses of each class, which is not a problem but i would've loved to see anything but text being used in panels and layouts, another issue i found was that there is no spot on the main website for Altscreen, i found out about it in the examples folder too, so thats my experience. i am a begginer to C# and im trying to implement an application i've made in C# which was originally written in go, and the reason is basically this project, so i would love to send my thanks to the team behind Spectre.Console, stay safe.

As a first time user, I figured it out by going to the next documentation page down and guessing based on the API shape. So it's not a problem once you know how to use the library.
On August 24, 2024 6:05:48 PM GMT+03:00, Frank Ray @.***> wrote:

@OronDF343 Has the lack of examples stopped you from using the text prompt? If not, how did you figure out the usage? (I'm trying to triage the priority of this issue)
--
Reply to this email directly or view it on GitHub:
https://github.com/spectreconsole/spectre.console/issues/1615#issuecomment-2308424368
You are receiving this because you were mentioned.
Message ID: @.***>
--
Sent from my Android device with K-9 Mail. Please excuse my brevity.

Thanks @OronDF343, @Mehybinator, that's really helpful. I've long suspected the examples solution would benefit from more prominence. I know it's mentioned on the main GitHub readme, but I think we only directly link to examples from two pages in the main spectre website, and these are DI and the help writer, not any widget pages. I'm going to have a little think how this might be improved for new devs using the library.

Thanks @FrankRay78, here you can find the PR.
I've also leaved a comment for you in the example I've mentioned above, about the Optional TextPrompt.

Hi, @FrankRay78 I think this could be closed.
@OronDF343 Please check the doc and let us know if it meets what you requested 🙇

Resolved in #1636


dag: add status dag for all components

as a user of TDP, I would like to get all component status after cluster has been deployed.
proposition : use ansible service_facts

Add _check playbook in the DAG.


samples: net: google_iot_device: bsp: iotc_bsp_io_net_zephyr: Use bar…

…e constants POLLIN/POLLOUT/POLLERR.
The rest of this file uses bare POSIX names, not "zsock_" prefixed
variants, so it only makes sense to do the same for these constants.
And next step of porting the sample to native Zephyr targets is to
enable POSIX subsystem (CONFIG_POSIX_API), which only exports standard
POSIX name, so it's required for it too.
This changes should be squashed into the original commit introducing iotc_bsp_io_net_zephyr.c.
Signed-off-by: Paul Sokolovsky<EMAIL_ADDRESS>

@atigyi: As the commit message suggest, feel free to squash this into the related commit right away. Otehrwise, we'll need to do that later. Thanks!

Did the git squash into the commit where the file in question was introduced, please check the commit history.

@atigyi : Thanks! Looks good from first look, will look in more detail tomorrow. Btw, I by habit say "squash", but when you do git rebase --interactive, it oftentimes makes sense to select "f" aka "fixup" action instead "squash". "squash" combines 2 commit messages (and allows to edit), while "fixup" keeps commit message of first commit. It's not a big deal anyway, we'll be squashing more stuff around, and will clean up commit messages at the end.

:) 👍 Yes, I saw the combined commit messages, still learning the preferred git history structure.

@atigyi: Sounds good ;-). And just to confirm, everything indeed looks well, and I tested the latest google_iot_device_sdk_integration branch with native_posix (I'm sure you did either, just trying to be explicit about that, so we could trace regressions if they appear).

@pfalcon Thanks for testing. Yes, I did the tests as well.
What does Zephyr do to provide QA for samples? Continuous build at least I guess. Is there anything else?

@atigyi: Yes, there's CI, which builds stuff (a subset on each related PR, whole (and that's a lot) by cron). Where possible, stuff is also run in qemu_x86 (and other qemu's). But that concerns only small part of networking-related tests (unittests and stuff which can be run locally via loopback).
It's ongoing theme of our (Linaro's) work to be able to test things on real hardware, but in regard to networking testing, we're at the about same stage.
We however have some CI for bigger Zephyr projects (scripting language ports - MicroPython, JerryScript, Zephyr.js), and perhaps could set up a similar adhoc test for Google Cloud, based on (suitably modified) sample app. But again, networking set up (all the firewalls, NATs, DNSes, etc.) is a challenge as well as reliability. That's definitely something to discuss on Linaro/Google level after we complete the initial porting work.


Remove custom PADD install as it is included in the base image

See: https://github.com/pi-hole/docker-pi-hole/pull/1105

Reminder to disable the Renovate tracking of PADD.

Running in production

Same!

Seeing some weirdness in my versions file.
root@pihole:/# cat /etc/pihole/versions
CORE_BRANCH=master
WEB_BRANCH=master
FTL_BRANCH=master
CORE_VERSION=v5.14.2
WEB_VERSION=v5.18
FTL_VERSION=v5.20
GITHUB_CORE_VERSION=v5.14.2
GITHUB_WEB_VERSION=v5.18
GITHUB_FTL_VERSION=v5.20
GITHUB_DOCKER_VERSION=2022.12
DOCKER_VERSION=2022.12
CORE_HASH=00345387
GITHUB_CORE_HASH=8d2e023e
WEB_HASH=ad2d4386
GITHUB_WEB_HASH=ad2d4386
FTL_HASH=cba74934
GITHUB_FTL_HASH=cba74934

CORE_HASH and GITHUB_CORE_HASH should be the same and as a result PADD is flagging Pi-hole as out-of-date.
Either that or CORE_BRANCH should not be master. It doesn't look like an issue on our end so I may end up opening an issue upstream when I get a chance.

I'm seeing the same stuff in my non-balena Pihole containers.

Seems related to https://github.com/pi-hole/PADD/pull/274

Looks like it will be fixed with https://github.com/pi-hole/PADD/pull/302

Looks like it will be fixed with pi-hole/PADD#302

Good find! I'm tempted to wait until the next release of the Pi-hole image that should include that fix. I don't think there is anything urgent we need from this release?

Yeah, I agree!

Waiting on v2022.12.1 with multiple upstream fixes.


:bug: fix shared hover bug

TODO:

[ ] Add new test that validates whether bug is fixed

Codecov Report

Merging #29 (e2a1c8f) into main (2ef7f3b) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #29      +/-   ##
==========================================
+ Coverage   97.53%   97.55%   +0.01%
==========================================
  Files           5        5
  Lines         325      327       +2
==========================================
+ Hits          317      319       +2
  Misses          8        8

Impacted Files
Coverage Δ

plotly_resampler/figure_resampler.py
97.44% <100.00%> (+0.02%)
:arrow_up:

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 2ef7f3b...e2a1c8f. Read the comment docs.

I will add the test!

@jvdd!


WIP: using Firefox , Safari compatible namespace

According to Mozilla cross-browser compitability guide:

There are two API namespaces in use among the main browsers:
browser.*, the proposed standard for the extensions API used by Firefox and Safari.
chrome.* used by Chrome, Opera, and Edge.

Firefox also supports the chrome.* namespace for APIs that are compatible with Chrome, primarily to assist with porting. However, using the browser.* namespace is preferred. In addition to being the proposed standard, browser.* uses promises—a modern and convenient mechanism for handling asynchronous events.

manifest 3 migration guide also recommends to move to browser. namespace:

Chromium introduces promise support to many methods with the goal of eventually supporting promises on all appropriate methods. This will provide for greater compatibility between Firefox and Chrome extensions, given that Firefox already supports promises when using the browser.* namespace.

This is a proposal to rename API namespace to browser. to benefit from promises and to have more code compitability with other browsers.

After some local testing with Chrome, it doesn't seem to be working, though.

I have to take into this later...

No need for this PR, this was resolve in #9


Fix crash issue when app control event was invoked

LaunchFromAppID could returning the NULL pointer.
but it does not checked before used

Thanks for the patch, @myroot. Can you explain under which circumstances you're seeing this behavior (app being NULL)? Before adding this check, I'd like to understand the situation and make sure we're not just treating a symptom of the problem instead of the root cause.

Hello,
Thanks for your attention.
First, It was tested in Tizen 3.0 Platform.
The application_system_tizen.cc was using tizen appfw API.
AE_RESET event was called when the another application was sending a event into this application.
So, AE_RESET event could be called more than once.
When AE_RESET event was called, application_system_tizen.cc was use ApplicationServiceTizen::LaunchFromAppID and ApplicationService::Launch method.
ApplicationService::Launch was returning the application handle.
But, If Application was already launched, It just returning NULL value.
and application_system_tizen was use the returning value without NULL checking.

OK, thanks for the explanation. Merging.


Writing testcase for reading strings to objects

Completing the tests and refectoring for reading the whole mission details

LGTM


Feat (tooltip) refactor (popover) to base it off tooltip

Adding tooltip to angular library
Description
refactored popover to base it off tooltip
Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots:
Before (If applicable):

After:

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation update
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Other... Please describe:

Checklist:

[x ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x ] I have read the CONTRIBUTING document.
[ x] I have added tests to cover my changes.
[x ] All new and existing tests passed.

lets squash and rebase @mathsusan


ConsoleTable

Created the ConsoleTable class and tested it on windows and arch linux

All good for a v1, since this need to be available soon
All tests are ok!
The code will need some refactoring in the future, there is some repetitive code.
Aside from that, the table is looking clean! Good work on this PR 👍


Update options module

Does this look correct @natemcmaster

Yes, thank you.

Can we put this in the readme for Universe?  I had to search my email inbox to find the instructions on how to update universe...
2.	aspnet/Universe uses git submodules to pin to exact commit shas of sub-repos. You should checkout the release/2.0.0 branch of Universe and also update the git submodule to point to the new commit you’ve added. You can do this with these steps

cd Universe
git checkout release/2.0.0
git pull
git submodule update --init –-remote modules/MyRepo
git add modules/MyRepo
git commit -m “Updated MyRepo to include changes from aspnet/MyRepo#123”

So, something like this? https://github.com/aspnet/Universe/blob/master/docs/Submodules.md#saving-an-update-to-a-submodule

Yeah maybe just a link then :)  I searched my email for submodule instead of the repo


update runc binary to v1.1.4

https://github.com/opencontainers/runc/pull/3564

Fixes

https://github.com/containerd/containerd/issues/7219

@thaJeztah The latest download from here still points to runc version 1.1.13 do they need a re-upload after the backport here or do you recommend moving to containerd.io-1.6.8-3.1.el7.x86_64.rpm

@thaJeztah Please ignore my earlier comment I got your response from another thread.
https://github.com/moby/moby/issues/43969#issuecomment-1228358971


[depthai_bridge] Fix incorrect include path in TFPublisher

Overview
This commit corrects an erroneous include path in the TFPublisher.cpp file. The original code incorrectly repeated the 'tf2_geometry_msgs' segment in the include directive, leading to a compilation error. The updated include statement correctly references 'tf2_geometry_msgs/tf2_geometry_msgs.hpp' without repetition.
This change ensures that the depthai_bridge package compiles successfully in ROS2 Humble by resolving the missing header file issue and aligns with standard ROS practices for including external dependencies.
Author: Nuno Marques (@TSC21).
Issue

File affected: depthai_bridge/src/TFPublisher.cpp
Issue: Compilation error due to incorrect header file path
Resolution: Correct the include path to the standard format used in ROS packages

Changes
ROS distro: ROS 2 Humble
List of changes: depthai_bridge/src/TFPublisher.cpp - change can be found on the diff.
Testing
Hardware used: OAK-D W
Depthai library version: the one which the Humble version of this package imports.
Visuals from testing
N.A. This can be validated by buiding the package from source against an environment built from source (In this case, with the srcs generated using  rosinstall_generator and using vcs import). Fixing this include allowed to build depthai_bridge from source.

@saching13 @Serafadam would appreciate your review here. Thanks!

Thanks for the PR @TSC21. The CI had some problems in the past with this header so this was a temporary solution, but it seems that it has been fixed upstream :slightly_smiling_face:


shopware-cli project proxy - matching sales channel

Question
I recently came across the undocumented command shopware-cli project proxy.
The following config is set in the env.local.
APP_URL=http://<IP_ADDRESS>:8020
TRUSTED_PROXIES=<IP_ADDRESS>,::1

Database: sales_channel_domain.url = http://<IP_ADDRESS>:8020
Proxy URL: https://poster-crops-que-pencil.trycloudflare.com/

Result:
Unable to find a matching sales channel for the request: "https://poster-crops-que-pencil.trycloudflare.com/". Please make sure the domain mapping is correct.

What am I doing wrong?

Are you on 6.5 or 6.6?

Tested on both 6.5 and 6.6. Both versions gave the same error.

Ahhh you need to create a sales channel or add a new domain with the given URL 😅 . It does not auto replace it 😓

Aha oké, make sense 😃. I thought it would work the same as ngrok or expose. With those services you can keep using the local domain


Update swearwords.go

Added a few swear words

I personally think that adding some swearwords from other languages (for example German) will be a good idea but I do not think this will get accepted because you can simply just add those in your own list on your server and adding swearwords from every language would be quite a challange.

I personally think that adding some swearwords from other languages (for example German) will be a good idea but I do not think this will get accepted because you can simply just add those in your own list on your server and adding swearwords from every language would be quite a challange.

A okay yea I see your point. Just had the idea suddenly 😂 Sry then


Patch 5

Teacher module: tweaked translations / added last missing translations.
"Computer Science" translated to "Informatik"
More capitalization issues solved (Dich ->dich etc.)
Fixed identation error that seems to have caused failure. (Old pull request closed)

Coverage increased (+0.01%) to 56.352% when pulling 2a9afe84757f25d393b8d064fca870e710232cc4 on joca16:patch-5 into 74c51de6a198a7ba2dfa6ec08a4db21293b726b5 on codecombat:master.

Thanks. Merged


chore: add hydro analytics to storybook

Closes https://github.com/github/primer/issues/3269

Add support for hydro analytics to storybook in production deployments. This is only used to get page views of different URLs in storybook.
Changelog

New

Changed

Update storybook config to include hydro tags when deployed to production

Removed

Rollout strategy

[x] None; if selected, include a brief description as to why

This is a change to our storybook
Testing & Reviewing

Verify that hydro events are being emitted in preview (temporarily enabled for this PR)

I'll make sure to remove support for page views in previews after we verify that it is wired up correctly 👍

Verified analytics are being sent for the preview over in: https://github.slack.com/archives/C02NUUQ9C30/p1714755458846759?thread_ts=1714728330.477579&cid=C02NUUQ9C30
Updated it so that we now will only report in production and not previews 👍


"npm install oltb" issue

"I am adding it to my React project with npm install oltb and following the examples in the npm directory within the examples directory. However, it still throws numerous errors like the ones below. When used as an npm package, there seems to be an issue with the references."
ERROR in ./node_modules/oltb/src/oltb/js/oltb.js 2:0-37
Module not found: Error: Can't resolve 'ol/proj' in 'D:\Users\Gokhan\reactoltb\my-app\node_modules\oltb\src\oltb\js'
Did you mean 'proj.js'?
BREAKING CHANGE: The request 'ol/proj' failed to resolve only because it was resolved as fully specified
(probably because the origin is strict EcmaScript Module, e. g. a module with javascript mimetype, a '.mjs' file, or a '.js' file where the package.json contains '"type": "module"').
The extension in the request is mandatory for it to be fully specified.
Add the extension to the request.
ERROR in ./node_modules/oltb/src/oltb/js/oltb.js 3:0-96
Module not found: Error: Can't resolve 'ol/interaction' in 'D:\Users\Gokhan\reactoltb\my-app\node_modules\oltb\src\oltb\js'
Did you mean 'interaction.js'?
BREAKING CHANGE: The request 'ol/interaction' failed to resolve only because it was resolved as fully specified
(probably because the origin is strict EcmaScript Module, e. g. a module with javascript mimetype, a '.mjs' file, or a '.js' file where the package.json contains '"type": "module"').
The extension in the request is mandatory for it to be fully specified.
Add the extension to the request.
ERROR in ./node_modules/oltb/src/oltb/js/oltb.js 4:0-113
Module not found: Error: Can't resolve 'ol/events/condition' in 'D:\Users\Gokhan\reactoltb\my-app\node_modules\oltb\src\oltb\js'
Did you mean 'condition.js'?
BREAKING CHANGE: The request 'ol/events/condition' failed to resolve only because it was resolved as fully specified
(probably because the origin is strict EcmaScript Module, e. g. a module with javascript mimetype, a '.mjs' file, or a '.js' file where the package.json contains '"type": "module"').
The extension in the request is mandatory for it to be fully specified.
Add the extension to the request.
ERROR in ./node_modules/oltb/src/oltb/js/oltb.js 7:0-41
Module not found: Error: Can't resolve './browser-prototypes/json-cycle' in 'D:\Users\Gokhan\reactoltb\my-app\node_modules\oltb\src\oltb\js'
Did you mean 'json-cycle.js'?
BREAKING CHANGE: The request './browser-prototypes/json-cycle' failed to resolve only because it was resolved as fully specified
(probably because the origin is strict EcmaScript Module, e. g. a module with javascript mimetype, a '.mjs' file, or a '.js' file where the package.json contains '"type": "module"').
The extension in the request is mandatory for it to be fully specified.
Add the extension to the request.
ERROR in ./node_modules/oltb/src/oltb/js/oltb.js 8:0-37
Module not found: Error: Can't resolve './browser-prototypes/string' in 'D:\Users\Gokhan\reactoltb\my-app\node_modules\oltb\src\oltb\js'
Did you mean 'string.js'?
BREAKING CHANGE: The request './browser-prototypes/string' failed to resolve only because it was resolved as fully specified
(probably because the origin is strict EcmaScript Module, e. g. a module with javascript mimetype, a '.mjs' file, or a '.js' file where the package.json contains '"type": "module"').
The extension in the request is mandatory for it to be fully specified.
Add the extension to the request.
ERROR in ./node_modules/oltb/src/oltb/js/oltb.js 9:0-43
Module not found: Error: Can't resolve './browser-prototypes/slide-toggle' in 'D:\Users\Gokhan\reactoltb\my-app\node_modules\oltb\src\oltb\js'
Did you mean 'slide-toggle.js'?
BREAKING CHANGE: The request './browser-prototypes/slide-toggle' failed to resolve only because it was resolved as fully specified
(probably because the origin is strict EcmaScript Module, e. g. a module with javascript mimetype, a '.mjs' file, or a '.js' file where the package.json contains '"type": "module"').
The extension in the request is mandatory for it to be fully specified.
Add the extension to the request.
ERROR in ./node_modules/oltb/src/oltb/js/oltb.js 13:0-52
Module not found: Error: Can't resolve './ui-common/ui-toasts/toast' in 'D:\Users\Gokhan\reactoltb\my-app\node_modules\oltb\src\oltb\js'
Did you mean 'toast.js'?
BREAKING CHANGE: The request './ui-common/ui-toasts/toast' failed to resolve only because it was resolved as fully specified
(probably because the origin is strict EcmaScript Module, e. g. a module with javascript mimetype, a '.mjs' file, or a '.js' file where the package.json contains '"type": "module"').
The extension in the request is mandatory for it to be fully specified.
Add the extension to the request.
ERROR in ./node_modules/oltb/src/oltb/js/oltb.js 14:0-52
Module not found: Error: Can't resolve './ui-common/ui-modals/modal' in 'D:\Users\Gokhan\reactoltb\my-app\node_modules\oltb\src\oltb\js'
Did you mean 'modal.js'?
BREAKING CHANGE: The request './ui-common/ui-modals/modal' failed to resolve only because it was resolved as fully specified
(probably because the origin is strict EcmaScript Module, e. g. a module with javascript mimetype, a '.mjs' file, or a '.js' file where the package.json contains '"type": "module"').
The extension in the request is mandatory for it to be fully specified.
Add the extension to the request.
ERROR in ./node_modules/oltb/src/oltb/js/oltb.js 15:0-55
Module not found: Error: Can't resolve './ui-common/ui-dialogs/dialog' in 'D:\Users\Gokhan\reactoltb\my-app\node_modules\oltb\src\oltb\js'
Did you mean 'dialog.js'?
BREAKING CHANGE: The request './ui-common/ui-dialogs/dialog' failed to resolve only because it was resolved as fully specified
(probably because the origin is strict EcmaScript Module, e. g. a module with javascript mimetype, a '.mjs' file, or a '.js' file where the package.json contains '"type": "module"').
The extension in the request is mandatory for it to be fully specified.
Add the extension to the request.
ERROR in ./node_modules/oltb/src/oltb/js/oltb.js 16:0-56
Module not found: Error: Can't resolve './browser-constants/settings' in 'D:\Users\Gokhan\reactoltb\my-app\node_modules\oltb\src\oltb\js'
Did you mean 'settings.js'?
BREAKING CHANGE: The request './browser-constants/settings' failed to resolve only because it was resolved as fully specified
(probably because the origin is strict EcmaScript Module, e. g. a module with javascript mimetype, a '.mjs' file, or a '.js' file where the package.json contains '"type": "module"').
The extension in the request is mandatory for it to be fully specified.
Add the extension to the request.
...

Hi @gokhanax
These problems can be a bit tricky. The history of TS/JS and the different types of build systems has brought us here. But they usually can be fixed in some way.
When installing OLTB through "npm install oltb" other packages will also be installed (those specified under dependencies in package.json in OLTB).
Step 1:
Check that your node_modules contains these packages.
"dependencies": {
    "a-color-picker": "^1.2.1",
    "axios": "^1.7.2",
    "browser-dtector": "^4.1.0",
    "html2canvas": "^1.4.1",
    "jsts": "^2.11.3",
    "lodash": "^4.17.21",
    "many-keys-map": "^2.0.1",
    "moment": "^2.30.1",
    "ol": "^10.0.0",
    "proj4": "^2.11.0",
    "screenfull": "^6.0.2",
    "sortablejs": "^1.15.2",
    "tippy.js": "^6.3.7",
    "uuid": "^10.0.0"
},

Step 2:
Check if your package.json has "main" or "module" specified.
Step 3:
React uses Webpack to run, build and bundle.
Try adding this configuration to tour configuration.
resolve: {
    fullySpecified: false
}

More info here: https://webpack.js.org/configuration/module/#resolvefullyspecified
Can you share more info about your project:

package.json
tsconfig.(ts | json)
webpack.config.(js | ts | json) (or if you are using Next.js or Vite).

Right now OLTB is written in vanilla js. Sometimes a little more configuration is required when you want to use this in a typescript project. Eventually I will lift everything to TypeScript, but not in the next few months.

Hi @qulle
Thank you very much for your response. I will try again, paying attention to the points you mentioned. Additionally, would it be possible for you to share a sample React project with npm added?
Best regards.

Yes i can add one example for React and one for Angular.
Any progress on your issue?


ENSEMBL-4232 - 'BLAST this sequence' strips out the FASTA ID

This pull request is in combination with the pull request https://github.com/Ensembl/public-plugins/pull/47

Hi Sanjay, could you try using '_seq' class instead of 'text_sequence_identifier'? You probably won't need https://github.com/Ensembl/public-plugins/pull/47 then.

Yup.. done. Thanks.


avoid data race on grace.hooks

What problem are we solving?
https://github.com/seaweedfs/seaweedfs/issues/3564
How are we solving the problem?
https://github.com/seaweedfs/seaweedfs/issues/3564
How is the PR tested?
Checks

[ ] I have added unit tests if possible.
[ ] I will add related wiki document changes and link to this PR after merging.

Hey, I just saw that the new e2e tests started failing with this commit and I checked why.
I found that the filer service in the workflow didn't come up (port did not open).
It seems that the new RLock in the goroutine is at fault, because all OnInterrupt() calls would now block forever, for example in NewFilerServer(). This leads to the filer server never opening up it's port.

fixed https://github.com/seaweedfs/seaweedfs/pull/3575

@chrislusf Let's temporarily turn on CGO for darwin, since the tests are not informative, they always fall because of this ?
    pre_command: export CGO_ENABLED=0 && export GODEBUG=http2client=0

You also need to "fix" the CodeQL test


ValueError: low >= high

See output:
(.venv) l1000323954@senor0lun1xsc01:~/github/model_optimization$ git log -1
commit a87c7d0a99856ecf180c077b6f0d25c41cb317e6 (HEAD -> main, origin/main, origin/HEAD)
Merge: dffac56 8fe2bf7
Author: HVH<EMAIL_ADDRESS>Date:   Sun Oct 31 17:06:02 2021 +0200

    Merge pull request #3 from sony/bug_2_image_loader_one_image

    Fix bug in FolderImageLoader

(.venv) l1000323954@senor0lun1xsc01:~/github/model_optimization$ python tutorials/example_keras_mobilenet.py
2021-11-01 14:28:12.754791: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libcudart.so.11.0'; dlerror: libcudart.so.11.0: cannot open shared object file: No such file or directory
2021-11-01 14:28:12.754814: I tensorflow/stream_executor/cuda/cudart_stub.cc:29] Ignore above cudart dlerror if you do not have a GPU set up on your machine.
Starting Scanning Disk: /path/to/images/folder
Finished Disk Scanning: Found 0 files
2021-11-01 14:28:14.310772: I tensorflow/stream_executor/platform/default/dso_loader.cc:53] Successfully opened dynamic library libcuda.so.1
2021-11-01 14:28:14.352714: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1733] Found device 0 with properties:
pciBusID: 0000:21:00.0 name: Quadro P400 computeCapability: 6.1
coreClock: 1.2525GHz coreCount: 2 deviceMemorySize: 1.93GiB deviceMemoryBandwidth: 29.88GiB/s
2021-11-01 14:28:14.352935: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libcudart.so.11.0'; dlerror: libcudart.so.11.0: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: /home/l1000323954/github/model_optimization/.venv/lib/python3.8/site-packages/cv2/../../lib64:
2021-11-01 14:28:14.353069: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libcublas.so.11'; dlerror: libcublas.so.11: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: /home/l1000323954/github/model_optimization/.venv/lib/python3.8/site-packages/cv2/../../lib64:
2021-11-01 14:28:14.353189: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libcublasLt.so.11'; dlerror: libcublasLt.so.11: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: /home/l1000323954/github/model_optimization/.venv/lib/python3.8/site-packages/cv2/../../lib64:
2021-11-01 14:28:14.353310: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libcufft.so.10'; dlerror: libcufft.so.10: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: /home/l1000323954/github/model_optimization/.venv/lib/python3.8/site-packages/cv2/../../lib64:
2021-11-01 14:28:14.353428: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libcurand.so.10'; dlerror: libcurand.so.10: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: /home/l1000323954/github/model_optimization/.venv/lib/python3.8/site-packages/cv2/../../lib64:
2021-11-01 14:28:14.353578: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libcusolver.so.11'; dlerror: libcusolver.so.11: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: /home/l1000323954/github/model_optimization/.venv/lib/python3.8/site-packages/cv2/../../lib64:
2021-11-01 14:28:14.353772: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libcusparse.so.11'; dlerror: libcusparse.so.11: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: /home/l1000323954/github/model_optimization/.venv/lib/python3.8/site-packages/cv2/../../lib64:
2021-11-01 14:28:14.353957: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libcudnn.so.8'; dlerror: libcudnn.so.8: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: /home/l1000323954/github/model_optimization/.venv/lib/python3.8/site-packages/cv2/../../lib64:
2021-11-01 14:28:14.353998: W tensorflow/core/common_runtime/gpu/gpu_device.cc:1766] Cannot dlopen some GPU libraries. Please make sure the missing libraries mentioned above are installed properly if you would like to use GPU. Follow the guide at https://www.tensorflow.org/install/gpu for how to download and setup the required libraries for your platform.
Skipping registering GPU devices...
2021-11-01 14:28:14.354638: I tensorflow/core/platform/cpu_feature_guard.cc:142] This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN) to use the following CPU instructions in performance-critical operations:  AVX2 AVX512F FMA
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
2021-11-01 14:28:14.355829: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1258] Device interconnect StreamExecutor with strength 1 edge matrix:
2021-11-01 14:28:14.355866: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1264]
  0%|                                                                                | 0/10 [00:00<?, ?it/s]
Traceback (most recent call last):
  File "tutorials/example_keras_mobilenet.py", line 73, in <module>
    quantized_model, quantization_info = mct.keras_post_training_quantization(model,
  File "/home/l1000323954/github/model_optimization/.venv/lib/python3.8/site-packages/model_compression_toolkit/keras/quantization_facade.py", line 219, in keras_post_training_quantization
    tg = _post_training_model_builder(in_model,
  File "/home/l1000323954/github/model_optimization/.venv/lib/python3.8/site-packages/model_compression_toolkit/keras/quantization_facade.py", line 124, in _post_training_model_builder
    mi.infer(representative_data_gen())
  File "tutorials/example_keras_mobilenet.py", line 68, in representative_data_gen
    return [image_data_loader.sample()]
  File "/home/l1000323954/github/model_optimization/.venv/lib/python3.8/site-packages/model_compression_toolkit/common/data_loader.py", line 102, in sample
    self._sample()
  File "/home/l1000323954/github/model_optimization/.venv/lib/python3.8/site-packages/model_compression_toolkit/common/data_loader.py", line 85, in _sample
    index = np.random.randint(0, self.n_files, self.batch_size)
  File "mtrand.pyx", line 746, in numpy.random.mtrand.RandomState.randint
  File "_bounded_integers.pyx", line 1254, in numpy.random._bounded_integers._rand_int64
ValueError: low >= high

(.venv) l1000323954@senor0lun1xsc01:~/github/model_optimization$ pip list
Package                        Version
------------------------------ -------------------
absl-py                        0.15.0
astunparse                     1.6.3
cachetools                     4.2.4
certifi                        2021.10.8
charset-normalizer             2.0.7
constrained-model-optimization 3.14
cycler                         0.11.0
decorator                      5.1.0
dm-tree                        0.1.6
flatbuffers                    1.12
gast                           0.4.0
google-auth                    2.3.2
google-auth-oauthlib           0.4.6
google-pasta                   0.2.0
grpcio                         1.34.1
h5py                           3.1.0
idna                           3.3
imageio                        2.10.1
joblib                         1.1.0
keras-nightly                  2.5.0.dev2021032900
Keras-Preprocessing            1.1.2
kiwisolver                     1.3.2
Markdown                       3.3.4
matplotlib                     3.4.3
networkx                       2.5
numpy                          1.19.5
oauthlib                       3.1.1
opencv-python                  <IP_ADDRESS>
opt-einsum                     3.3.0
Pillow                         8.4.0
pip                            21.3.1
protobuf                       3.19.1
pyasn1                         0.4.8
pyasn1-modules                 0.2.8
pyparsing                      3.0.4
python-dateutil                2.8.2
PyWavelets                     1.1.1
requests                       2.26.0
requests-oauthlib              1.3.0
rsa                            4.7.2
scikit-image                   0.18.3
scikit-learn                   1.0.1
scipy                          1.7.1
setuptools                     56.0.0
six                            1.15.0
tensorboard                    2.7.0
tensorboard-data-server        0.6.1
tensorboard-plugin-wit         1.8.0
tensorflow                     2.5.0
tensorflow-estimator           2.5.0
tensorflow-model-optimization  0.7.0
termcolor                      1.1.0
threadpoolctl                  3.0.0
tifffile                       2021.10.12
tqdm                           4.62.3
typing-extensions              <IP_ADDRESS>
urllib3                        1.26.7
Werkzeug                       2.0.2
wheel                          0.37.0
wrapt                          1.12.1

(.venv) l1000323954@senor0lun1xsc01:~/github/model_optimization$ python --version
Python 3.8.12

Hi,
When using the FolderImageLoader you can not use an empty folder. The folder has to contain at least one image.
The error message now is updated.


Feature Request: button controls for slide navigation

For some instances (touch screen displays, users being able to view the slides on their computers), it would be great to have button controls on the edges that let a viewer navigate to the next or previous slides. The transition times and all would remain the same, but if the user wants, they can go back to the previous slide, or to the next one.

This could indeed be a useful feature. I'll add it to my TODO list.

This is now implemented in next and will be shipped in v1.1.0.


add arm64 syscall numbers

add arm64 syscall numbers only

I lifted the platform build constraint, as like previously stated the only exception is MIPS and ALPHA, we could have this later. Fixed by https://github.com/ii64/gouring/commit/bd50af5d2125ee858ea1f7804a5795db00a48776


[BUG] when i open bloxstrap it says you must install .NET Desktop Runtime to run this application so i downloaded it and it still says the same thing how to fix?

Acknowledgement of preliminary instructions

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

What problem did you encounter?

Bloxstrap Log
N/A

Just to make sure, you clicked yes on the first time when it popped up?

i did

Idk then honestly.

hold up

maybe it's interfering with bloxstrap?

let me delete it to see if it is

nope

i cant delete it

i fixed it


Use repository settings from piped config

What this PR does / why we need it:

planner/scheduler should follow repository settings in piped config.
plan preview builder should not check Remote and Branch because repository settings in app config metadata are not reliable (they can be outdated by modifying piped config)

Which issue(s) this PR fixes:
Fixes #4618
Does this PR introduce a user-facing change?:

How are users affected by this change: None
Is this breaking change: No
How to migrate (if breaking change): None

Thank you for the PR :)
Actually, this change may affect the behavior of executing the pipeline.
We need to check and discuss the effect of it.
So it may take more time to examine the effect. 🙏

📝 We should investigate the effect because some features including Deployment are based on the information of Application.GtiPath.
https://github.com/pipe-cd/pipecd/blob/master/pkg/model/application.pb.go#L110C6-L110C6

refactored: override repository with piped config (decent one) when creating deployment.


Allow to load documents from io.Reader

It will be convenient to allow users to load documents from custom io.Reader, not just files and urls. Use case includes compressed files, for example.

Hi @dennwc,
Many thanks for your work!
It looks good overall but please take a note that you are now deferring res.Body.Close() twice, first on line 89:
https://github.com/dennwc/json-gold/blob/cf1f0d071754949e2070d5097a1e4cf8631fcee5/ld/document_loader.go#L89
and then again on line 115.
I suspect it may cause issues in some scenarios.

Oh, sorry! Will fix that shortly.

Looks good now, thank you!


Result null for valid request

I perform a POST request to an endpoint, which spits out a valid result every time (I have tested extensively with an HTTP client).  Yet I get the following when trying to debug a nil model:
2015-10-24 18:48:11.880 AppName[60332:1645100] <OVCResponse: 0x7f8de8f08930> {
    HTTPResponse = "<NSHTTPURLResponse: 0x7f8de8d55be0> { URL: http://localhost:8080/api/v1/users/authenticate } { status code: 200, headers {\n    \"Access-Control-Allow-Origin\" = \"*\";\n    Connection = \"keep-alive\";\n    \"Content-Length\" = 1109;\n    \"Content-Type\" = \"application/json\";\n    Date = \"Sat, 24 Oct 2015 22:48:11 GMT\";\n    \"X-Powered-By\" = Express;\n    \"access-control-allow-headers\" = \"X-Requested-With\";\n} }";
    result = "<null>";
    resultClass = XYModelClass;
}

AKA, result = "<null>";.  I have found that if I change the status code server side to 400, 403, or really an other status code than 200, I can log a non "<null>"  result...  However nothing for 200.  Any ideas/suggestions?

Hi, could I see the JSON output of the URL at http://localhost:8080/api/v1/users/authenticate and also the implementation of + (NSDictionary *)JSONKeyPathsByPropertyKey in the XYModelClass class?
Thanks.

Hey @sodastsai thanks for the quick reply...
Here's what my JSON looks like:
{
    "token": "USER TOKEN",
    "user": {
        "id": "562bcbe3cc27e1f5df000002",
        "email"<EMAIL_ADDRESS>        "updated_at": "2015-10-24T18:20:19.684Z",
        "created_at": "2015-10-24T18:20:19.681Z",
        "name": {
            "first": "First",
            "last": "Last"
        }
    }
}

And the model class is:
+ (NSDictionary *)JSONKeyPathsByPropertyKey {
    return @{
             @"token" : @"token",
             @"user" : @"user"
             };
}

+ (NSValueTransformer *)userJSONTransformer {
    return [MTLJSONAdapter dictionaryTransformerWithModelClass:XYUser.class];
}

Excuse me for another question, is the user dictionary valid for Mantle?
i.e. I mean is
{
    "id": "562bcbe3cc27e1f5df000002",
    "email"<EMAIL_ADDRESS>    "updated_at": "2015-10-24T18:20:19.684Z",
    "created_at": "2015-10-24T18:20:19.681Z",
    "name": {
        "first": "First",
        "last": "Last"
    }
}

valid for MTLJSONSerializer to convert to a model instance?

if possible, could you set a breakpoint at +[OVCResponse responseWithHTTPResponse:JSONObject:resultClass:error:] to check why the result property is nil?
Thanks.


Access through hostname is not working after enabling Private endpoint

We enabled Private endpoint on a PostgreSQL Server. After enabling that we can no longer access using the hostname of the server while we can access using Private IP of the endpoint. Is it by design? If not, what is the solution?

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

ID: 77e23acd-1ca5-ee48-5706-273f1d4327b9
Version Independent ID: 31d5e234-c8eb-3bf2-1492-b7c48ccba0dc
Content: Private Link - Azure Database for PostgreSQL - Single server
Content Source: articles/postgresql/concepts-data-access-and-security-private-link.md
Service: postgresql
GitHub Login: @kummanish
Microsoft Alias: manishku

@vikaskumawat80 We are checking this and we will get back to you.

@vikaskumawat80 Please check the below link as it mentions hostname can be used for the connnection:
https://techcommunity.microsoft.com/t5/azure-database-for-postgresql/introducing-private-link-for-azure-database-for-postgresql/ba-p/1088068
It mentions that :
"You can pass the server name in the host name field demoprivatelinkserver.postgres.privatelink.database.azure.com"
Please verify your set up according to this page and let us know if this helps.

@vikaskumawat80 Please let us know if you need any further information regarding the same.

@vikaskumawat80 We are going ahead and closing this thread for now. Please get back to us if you have any further questions.

No, it's not working even in the above link it is using Private IP.


Remove items from grid

As far as I see there is no public API to remove items.
https://github.com/BTMorton/angular2-grid/blob/master/src/directives/NgGrid.ts#L354 presents a public removeItem function, however I see no way to access it from the template or component using the directive (especially with removeItem requiring a reference to the GridItem).
Removing items from the bound item collection using items.splice(x, 1) causes really weird behaviour (items are being duplicated, wrong item is deleted, ...).
On side note: if the direcives used @Directive({ selector='[ngGrid]', exportAs='ngGrid' }) and @Directive({ selector='[ngGridItem]', exportAs='ngGridItem' }) we could probably at least do something like
<div #grid [ngGrid]="gridconfig">
    <div #gridItem *ngFor="let item of itemconfig; let i = index" [(ngGridItem)]="itemconfig[i]">
        <button (click)="grid.removeItem(gridItem)">Remove</button>
    </div>
</div>

To be more concrete on the "weird behaviour":
With
0|1|2
3|4|5
6|7

Delete 1

0|4|2
3|4|5
6|7
-|7

all items cascading up are duplicated.

So, you shouldn't need to call the removeItem method at all. When you remove the item from the array that is bound to the ngFor, it should remove the item from the grid automatically.

Yes, I did realize that after looking at the example and fixing the bug described above (gonna file a seperate issue for that).


Homebrew installation returns "No available Cask for openrefine-dev"

I have installed many things through homebrew on my computer, which is a MacBook Pro running macOS Sierra Version 10.12.3. However, when I enter the code brew cask install openrefine-dev, it returns the following errors:
Error: No available Cask for openrefine-dev
Error: nothing to install
I have not been able to replicate this with other programs. Thanks.

@joewiz Can you assist @Archaeo-Programmer with this issue ?

@Archaeo-Programmer Could you try this command?
brew tap caskroom/versions

Then do the install again:
brew cask install openrefine-dev

Please let me know if you have any success.

Yes, that did the trick. Everything installed just fine. Thanks!

@Archaeo-Programmer Thanks for the report! I've updated the homebrew-based installation instructions with this additional step. I think I had already performed that step on my own system to access other version-specific casks, so I neglected to include it in the steps that first-time users would need to run. Thanks again!

Great, thanks so much for the reply, and updating the instructions with additional step. I was unaware that this was a necessary step for some casks. Thanks!

Yes, Caskroom only puts an app's current stable release in the primary caskroom tap. Pre-releases, betas, and version-specific builds go into the versions tap. OpenRefine this fits into the latter, and as soon as a final release is issued we will submit a PR for a new openrefine cask in the core caskroom tap.


Sync changes of the consolidation of model type from TCGC

Sync changes from https://github.com/Azure/autorest.csharp/pull/4781

API change check
API changes are not detected in this pull request.


Fix missing param count in compilation error message

A private macro definition with one parameter which is called with no parameter resulted in a compiler error message where the "expected" count was missing. From the original issue:
private macro foo(x)
end

foo # Error: wrong number of arguments for macro 'foo' (given 0, expected )

The error was caused by a statement that rejected private macros, which resulted in all_arguments_sizes being empty.
The removed code was added in cabb0219a6ecda8777d16bf2ffe511381f74c8d7. The affected specs are green, so I assume there's no regression without that statement.
Fixes https://github.com/crystal-lang/crystal/issues/13872

Congrats on the first PR 🚀

Thanks :party:

LGTM, just a minor detail could be improved.

Updated.


Unable to build with MinGW-w64 (GCC) on windows

Hello, for some reason I am unable to build the rendering engine, its my first time building a project from source but I am pretty sure I am doing it properly...
I am getting this error in the command prompt:
D:\Rendering Engine\bgfx>make mingw-gcc-release64
../bx/tools/bin/windows/genie --with-tools --with-combined-examples --with-share
d-lib --os=windows --gcc=mingw-gcc gmake
Set MINGW environment variable.
Building configurations...
Running action 'gmake'...
Generating ../.build/projects/gmake-mingw-gcc/Makefile...
Generating ../.build/projects/gmake-mingw-gcc/bgfx.make...
Generating ../.build/projects/gmake-mingw-gcc/bx.make...
Generating ../.build/projects/gmake-mingw-gcc/bimg.make...
Generating ../.build/projects/gmake-mingw-gcc/bimg_decode.make...
Generating ../.build/projects/gmake-mingw-gcc/bimg_encode.make...
Generating ../.build/projects/gmake-mingw-gcc/example-glue.make...
Generating ../.build/projects/gmake-mingw-gcc/example-common.make...
Generating ../.build/projects/gmake-mingw-gcc/examples.make...
Generating ../.build/projects/gmake-mingw-gcc/example-25-c99.make...
Generating ../.build/projects/gmake-mingw-gcc/bgfx-shared-lib.make...
Generating ../.build/projects/gmake-mingw-gcc/spirv-opt.make...
Generating ../.build/projects/gmake-mingw-gcc/glslang.make...
Generating ../.build/projects/gmake-mingw-gcc/glsl-optimizer.make...
Generating ../.build/projects/gmake-mingw-gcc/fcpp.make...
Generating ../.build/projects/gmake-mingw-gcc/shaderc.make...
Generating ../.build/projects/gmake-mingw-gcc/texturec.make...
Generating ../.build/projects/gmake-mingw-gcc/texturev.make...
Generating ../.build/projects/gmake-mingw-gcc/geometryc.make...
Done. Generated 19/19 projects.
make -R -C .build/projects/gmake-mingw-gcc config=release64
make[1]: Entering directory 'D:/Rendering Engine/bgfx/.build/projects/gmake-ming
w-gcc'
==== Building bx (release64) ====
Creating ../../win64_mingw-gcc/obj/x64/Release/bx
Creating ../../win64_mingw-gcc/obj/x64/Release/bx/bx/src
Creating ../../win64_mingw-gcc/bin
allocator.cpp
/usr/bin/sh: /bin/x86_64-w64-mingw32-g++: No such file or directory
make[2]: *** [bx.make:402: ../../win64_mingw-gcc/obj/x64/Release/bx/bx/src/alloc
ator.o] Error 127
make[1]: *** [Makefile:21: bx] Error 2
make[1]: Leaving directory 'D:/Rendering Engine/bgfx/.build/projects/gmake-mingw
-gcc'
make: *** [makefile:122: mingw-gcc-release64] Error 2

What am I doing wrong? here are is what I do so this can be reproduced:

Download bgfx, bx, bimg master branches as ZIP by the web browser (Google Chrome)
I create a new folder for everything to store in, in this case I called it "Rendering Engine"
In the new ""Rendering Engine" folder I create these new folders: bgfx, bx, bimg
Then I extract the contents from the bgfx-master folder in the ZIP to the new bgfx folder
I extract the contents from the bx-master folder in the ZIP to the new bx folder
I extract the contents from the bimg-master folder in the ZIP to the new bimg folder
Now I go to the bgfx folder in the "Rendering Engine" folder and open a command prompt there (Shift and right-click then open command prompt here)
Then I type this into the command prompt: make mingw-gcc-release64
That is it! It starts doing something but fails!

So what is happening here? I am running windows 7 64-bit, the operating system is not the issue here either! Do I need to install MSYS2 or Cygwin to compile it for ONLY windows 64-bit?
Note:
I have MinGW-w64 installed properly, I have the MinGW environment variable set to the mingw64\bin folder.
The environment variable looks like this: C:\MinGW\mingw64\bin and If I run "gcc" or "g++" or "make" it does execute properly, as expected.

I would like to mention that if I try to compile bx by it self, I get this error:
D:\Rendering Engine\bx>make mingw-gcc-release64
../bx/tools/bin/windows/genie --gcc=mingw-gcc gmake
Set MINGW environment variable.
Building configurations...
Running action 'gmake'...
Generating ../.build/projects/gmake-mingw-gcc/Makefile...
Generating ../.build/projects/gmake-mingw-gcc/bx.make...
Generating ../.build/projects/gmake-mingw-gcc/bin2c.make...
Generating ../.build/projects/gmake-mingw-gcc/lemon.make...
Generating ../.build/projects/gmake-mingw-gcc/bx.test.make...
Generating ../.build/projects/gmake-mingw-gcc/bx.bench.make...
Done. Generated 6/6 projects.
make -R -C .build/projects/gmake-mingw-gcc config=release64
make[1]: Entering directory 'D:/Rendering Engine/bx/.build/projects/gmake-mingw-
gcc'
==== Building bx (release64) ====
Creating ../../win64_mingw-gcc/obj/x64/Release/bx
Creating ../../win64_mingw-gcc/obj/x64/Release/bx/src
Creating ../../win64_mingw-gcc/bin
allocator.cpp
/usr/bin/sh: /bin/x86_64-w64-mingw32-g++: No such file or directory
make[2]: *** [bx.make:402: ../../win64_mingw-gcc/obj/x64/Release/bx/src/allocato
r.o] Error 127
make[1]: *** [Makefile:17: bx] Error 2
make[1]: Leaving directory 'D:/Rendering Engine/bx/.build/projects/gmake-mingw-g
cc'
make: *** [makefile:63: mingw-gcc-release64] Error 2

It appears that its unable to source files, in this case its expecting for the allocator.cpp to be compiled into a allocator.o file but its not! No compiling happens what so ever, I wonder why... the sources are in the original location, the bx/src folder, the make file is in bx folder, I just don't understand why...
From trying to understand the error, maybe its unable to find "/bin/x86_64-w64-mingw32-g++" but its there! The file does exist in the MinGW bin folder and if I execute it normally in a command prompt, it does work!

The issue has been fixed in the issue #1476
Thank you for your help and sorry for any inconvenience I've caused!


[MINOR][PYTHON] Enable parity test test_different_group_key_cardinality

What changes were proposed in this pull request?
Enable parity test test_different_group_key_cardinality, by trigger the analysis
Why are the changes needed?
for test coverage
Does this PR introduce any user-facing change?
no, test only
How was this patch tested?
ci
Was this patch authored or co-authored using generative AI tooling?
no

Merged to master.


Smaller default width

I have a few chart on my page, I didn't set the charts width so they could scale down with the screen width. But something was breaking on really small screens.
I struggled for 3 hours trying to find the reason until I find out that my chart was 300px wide instead of 295px and then another half an hour to find out it was being set by this lib.
I'm not sure what value it should be, but I think it shouldn't prevent the charts from becoming smaller.

Do you have a wrapping component around the chart?

Just a simple div without any styles

If I recall correctly canvas does needs a fix width and height to start so there are default props for height and width of 150 and 300 respectively. Does turning the responsive flag on make any sort of difference?

It doesn't since it's already on by default

Closing for now. Please supply a sandbox or code example with the latest versions of both libraries. If the problem persists, feel free to open a new issue.


Only show modules with published versions in the Recent Modules

This was brought up by someone on the Discord channel, the "recently added" section currently shows any newly registered module, even if they have no published versions. It would be nice to only show modules that have at least one published version.

Just took a quick look at the code, we would have to join the builds and modules collections like in #143

Closing as we dont have recent modules anymore


Fix Mikhail's bug

This fixes Mikhail's bug along with a number of other related corner-case issues in the Set Operations.

An overview and explanation of the key changes contained in the latest updates can be read on the website:
https://datasketches.apache.org/docs/Theta/ThetaSetOpsCornerCases.html

The related documentation on the website is very instructive and helped with my previous confusion about the different cases that were being tested.   The use of enums to convey the corner case in question back to the AnotB set operation was also a nice approach.

If there are no further issues, I will merge this PR as it more than satisfies the objective of fixing Mikhail's bug.  Nonetheless, there are some other checks that I would like to do, but those can be in separate PRs.  I'll leave this open for another 24 hours.

I expect to take a closer look at the logic while porting the tests (and, perhaps, adjusting the implementation) to C++


Unexpected timings when using TimeUpdateStrategy::ManualDuration

Bevy version
v0.12.1
What you did
I was writing tests to validate my systems are working correctly after varying amounts of time have passed, but I encountered discrepancies between Time<Real> and Time (virtual).
Minimal example which demonstrates this:
use bevy::prelude::*;
use bevy::time::TimeUpdateStrategy;
use std::time::Duration;

fn main() {
    let mut app = App::new();
    app.add_plugins(MinimalPlugins)
        .insert_resource(TimeUpdateStrategy::ManualDuration(Duration::from_millis(
            600,
        )))
        .add_systems(Update, print_elapsed);

    app.update();
    app.update();
    app.update();
}

fn print_elapsed(time: Res<Time>) {
    println!("Time Elapsed: {:?}", time.elapsed());
}

What went wrong
The code above produces the following output with incorrect timings:
Time Elapsed: 0ns
Time Elapsed: 250ms
Time Elapsed: 500ms

(using a different manual duration has no impact - the step is always 250ms as far as I can tell)
Changing from Time to Time<Real> produces this output which does have correct timings:
Time Elapsed: 0ns
Time Elapsed: 600ms
Time Elapsed: 1.2s

So there's a discrepancy between the Real and Virtual timings even though I haven't changed the time scale, paused etc.
As the systems in my App need to be affected by time scaling, I can't replace their usage of Res<Time> with Res<Time<Real>> as a workaround.

What you're seeing is Time<Virtual> clamped by its default max_delta of 250ms. If you'd like to disable that behavior, you can access the Time<Virtual> resource and call time.set_max_delta(Duration::MAX).


Re2 Room 202 Marvin as Licker

Seed R420-0000-XQHHHMZLLB2
Claire B, Marvin is replaced by a Licker who attacks during the first encounter with Marvin cut-scene.

There is no cut scene for B scenario, not sure what you mean.

Sorry I was playing the wrong scenario. But I did find in room 109 of Claire B, instead of crows, spiders spawned and they are invincible.

Sorry I was playing the wrong scenario. But I did find in room 109 of Claire B, instead of crows, spiders spawned and they are invincible.

Probably related to #7


chore: sync Node.js install script with upstream install

Fixes #939 by adding the new release key introduced in that release.
Also syncs the rest of the script with the upstream Dockerfile from docker-node

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


Create loadbalancer.newLBinstance doesn't honour security group annotations

Create loadbalancer.newLBinstance doesn't honour security group annotations, leading to #865

I signed it

This alone is not sufficient but I can implement rest of logic after a discussion.

Hi @rverma-nikiai
Thanks for contributing this.
I personally haven't tried vpc sharing and the mechanism behind that.
In my understanding, vpc(is owned) by account1, where (subnet1) is owned by account2, and our ALB ingress controller is runnings on worker nodes in subnet1, am I correct?
Have you tested whether you will running into problem with this patch when you delete the LB? This part of code will try to set the securityGroup on ALB to be the default SG of VPC(in another account), i'm not sure whether there will be error(I suppose there will be, since normally when create ALB without sg specified, it will use VPC's default SG)

Update:
I didn't noticed you don't have securityGroup available in lb package. You will need to move the logic of  resolveSecurityGroupIDs out of sg/association.go to lb/loadbalancer.go, and pass externalSG as parameter to sg.AssoicationController

@M00nF1sh I unintentionally confused you. It's actually not a shared vpc but a shared subnet. AWS doesn't allow you to share a vpc. If you share a subnet than its vpc get automatically shared(can't edit apart from adding tags).
I haven't debugged much but found that the createLoadbalancer request is having below body
{
  IpAddressType: "ipv4",
  Name: "ff987766-echoserver-echose-2ad7",
  Scheme: "internet-facing",
  Subnets: [
    "subnet-03f458f395230b30d",
    "subnet-0ca65fb76018ab187"
  ],
  Tags: [
    {
      Key: "kubernetes.io/cluster/audit_niki_ai",
      Value: "owned"
    },
    {
      Key: "kubernetes.io/namespace",
      Value: "echoserver"
    },
    {
      Key: "kubernetes.io/ingress-name",
      Value: "echoserver"
    }
  ],
  Type: "application"
}```
The security group is missing from post request. Since the default value can't be used (we can't access the default security group from another account), this failed with corresponding reason as you assumed also.
What I was wondering if rather than resolving the default sg at each ingress level, I believe it will make more sense to allow the use of another security group as default. Something like a parameter as `--default-sg=sg-xxx` makes more sense to me. If you think its fine then what's left is adding logic to controller.getDefaultSecurityGroupID() to use env variable first, if absent try for default.

@rverma-nikiai
From this blog, "https://aws.amazon.com/blogs/networking-and-content-delivery/vpc-sharing-a-new-approach-to-multiple-accounts-and-vpc-management/", it said "VPC sharing participants can reference security group IDs of each other.", would you help test whether explicitly set the sgID to the defaultSG of VPC when creating LoadBalancer works? I'll test it as well once i got my second account.
BTW,
The default SG is actually unnecessarily needed, it used it as an trick to group the logic of create/attach securityGroup into a single method(association.Reconcile). so it works like

createLB with defaultSG ---> create LBSG -> attach LBSG to LB --> detach LBSG from LB(reset to defaultSG) ---> delete LBSG ---> delete LB.

However, ideally we don't need the defaultSG If we reorder to process to be below(though the sg related logic is more coupled around LB)

create LBSG ---> createLB with LBSG --> attach LBSG/externalLBSG to LB --> delete LB --> delete LBSG.
I'm working on a CRD based new version, if you can wait a bit, i'll include this logic. otherwise, i think an undocumented environment variable like VPC_DEFAULT_SG_ID can surve as an temp fix for your case 😄

This is good, I will wait for your fix. When you release an alpha version
let me know the branch I will deploy and test this :).
On Sat, 23 Feb, 2019, 2:35 AM M00nF1sh<EMAIL_ADDRESS>wrote:

@rverma-nikiai https://github.com/rverma-nikiai
From this blog, "
https://aws.amazon.com/blogs/networking-and-content-delivery/vpc-sharing-a-new-approach-to-multiple-accounts-and-vpc-management/",
it said "VPC sharing participants can reference security group IDs of each
other.", would you help test whether explicitly set the sgID to the
defaultSG of VPC when creating LoadBalancer works? I'll test it as well
once i got my second account.
BTW,
The default SG is actually unnecessarily needed, it used it as an trick to
group the logic of create/attach securityGroup into a single
method(association.Reconcile). so it works like

createLB with defaultSG ---> create LBSG -> attach LBSG to LB -->
detach LBSG from LB(reset to defaultSG) ---> delete LBSG ---> delete
LB.

However, ideally we don't need the defaultSG If we reorder to process to
be below(though the sg related logic is more coupled around LB)

create LBSG ---> createLB with LBSG --> attach LBSG/externalLBSG to
LB --> delete LB --> delete LBSG.
I'm working on a CRD based new version, if you can wait a bit, i'll
include this logic. otherwise, i think an undocumented environment variable
like VPC_DEFAULT_SG_ID can surve as an temp fix for your case 😄

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kubernetes-sigs/aws-alb-ingress-controller/pull/867#issuecomment-466547383,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AkjFoB4twZjerYXsYbqlgPkL1M42cqjeks5vQFt7gaJpZM4bJcfL
.

@M00nF1sh found that waf support having some issues while running in a region where waf regional services are not available. E.g. ap-south-1. Getting exception as
E0223 13:02:59.540879       1 :0] kubebuilder/controller "msg"="Reconciler error" "error"="error getting web acl for load balancer arn:aws:elasticloadbalancing:ap-south-1:614927918057:loadbalancer/app/ff987766-analytics-superse-acf8/b39236ab7a2497eb: RequestError: send request failed\ncaused by: Post https://waf-regional.ap-south-1.amazonaws.com/: dial tcp: lookup waf-regional.ap-south-1.amazonaws.com on <IP_ADDRESS>:53: no such host"  "controller"="alb-ingress-controller" "request"={"Namespace":"analytics","Name":"superset"}

Waf-regional is not available in ap-south-1 and we are using waf global.

@M00nF1sh with the above fix the load balancer get created but still there is no target group.

@M00nF1sh were you able to address this in your last release? As mentioned I was able to create the load balancer by adding the unofficial variable but there is no target group launched. Is this something expected?

Is there an ETA on this?


[3.x] feature: allow non-LW auth routes to be used

Overriding login/registration/password related components for a particular context is useful but it's limited to only Livewire components. This isn't very useful if your app has it's own auth flow already. The only way around it right now is to extend Filament's Authenticate middleware and overwrite the redirectTo() method.
These changes will allow somebody do this:
$context
	->login(route('my-custom-login'))

Must simpler, and something that I tried to do initially.

How about accepting any "route action". So a callable (invokable class, closure, [array, style])
Then we just pass it to the route. And you can redirect to your own login from within a simple closure. More flexible, right? We already do that for the billing portal feature.

Ah, if that's what we're doing for billing, then yeah, let's do that. I'll update this in a minute!

You can rename getLoginPage() to getLoginRouteAction(). For the other auth pages too?

Yeah, can update the other ones too.


fix: add dive validation to Subgraphs field of HeaderRules config struct

Motivation and Context

[ ] Tests or benchmark included
[ ] Documentation is changed or added on https://app.gitbook.com/
[ ] PR title must follow conventional-commit-standard

@cs-clarence thanks for all the PRs, I'd love to chat with you! Are you available for a call? You can book here: https://wundergraph.com/meet/jens

@jensneuse sure, I booked a call

@cs-clarence thank you


Revert k256 bump

This k256 version breaks revm, as documented in
https://github.com/RustCrypto/elliptic-curves/issues/988, which breaks foundry.
While I would instead wait to bump revm, foundry nightly (the only supported version per the provided GH action) is broken and I'd rather correct it ASAP.
Motivation
Foundry nightly is broken.
Solution
Revert the k256 bump until revm supports k256 0.13.2.

You'll want to lock it in a manifest otherwise anyone can bump it without even meaning to.

Done