bug: Debug: Attach to Node Process sends SIGUSR1 and attaches to wrong process

When using "Debug: Attach to Node Process", it puts the wrong process into debug mode and attaches to it

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.88.1
OS Version: Darwin arm64 23.4.0

Issue started on 1.88.0 and was not present on 1.87.2
Steps to Reproduce:
What I set up to repro this is two scripts that just report their pid and then sleep in a while loop.

Run script A
Run script B
Copy the pid of script A process
Use command palette to "Debug: Attach to node process"
Paste in pid of script A process
VSCode sends a SIGUSR1 to script B process and attaches debugger (separately confirmed with an explicit process.on('SIGUSR1' ...) listener)

This seems to only reproduce when I am trying to attach to the lower numbered pid process, with the higher numbered one getting attached to incorrectly (I think I had one case of a lower number pid getting attached to, but can't repro reliably). If I separately send a SIGUSR1 to the process running script A, and then attempt to debug it, the debugger correctly attaches to it, but also incorrectly sends a SIGUSR1 to script B process (where it naturally complains that port 9229 is already in-use).

It's not clear to me if the issue is specific to debug-related features, or maybe something like the process picker after first selecting Debug: Attach to node process in the command palette
Most of this was copied from comments in https://github.com/microsoft/vscode/issues/209785 which was deemed a separate issue.

I get this error when i use "Debug: Attach to Node Process"，when i upgrade to version 1.88。downgrade to 1.86.2 didn't help.
Could not read source map for file:///~/.vscode-server/cli/servers/Stable-e170252f762678dec6ca2cc69aba1570769a5d39/server/out/vs/out-vscode/vs/loader.js: ENOENT: no such file or directory, open '~/.vscode-server/cli/servers/Stable-e170252f762678dec6ca2cc69aba1570769a5d39/server/out/vs/out-vscode/vs/nls.js.map'
Version: 1.86.2
Commit: 903b1e9d8990623e3d7da1df3d33db3e42d80eda
Date: 2024-02-13T19:42:12.210Z
Electron: 27.2.3
ElectronBuildId: 26908389
Chromium: 118.0.5993.159
Node.js: 18.17.1
V8: <IP_ADDRESS>-electron.0
OS: Darwin arm64 23.4.0

Some notes from exploring this a bit today:
It also repros on:
Version: 1.88.1
Commit: e170252f762678dec6ca2cc69aba1570769a5d39
Date: 2024-04-10T17:34:12.840Z
Electron: 28.2.8
ElectronBuildId: 27744544
Chromium: 120.0.6099.291
Node.js: 18.18.2
V8: 12.0.267.19-electron.0
OS: Linux x64 6.8.0-76060800daily20240311-generic

I cannot repro developing on vscode running in a devcontainer at current main or 1.88.1. It's not clear to me if there's some nondeterminism or race condition that is less likely under this setup. I'm guessing the root cause is somewhere around  the processPicker over in https://github.com/microsoft/vscode-js-debug/blob/main/src/ui/processPicker.ts#L135 but I had some trouble getting set up while debugging to artificially add some waits and latency when trying to force it to repro. I'm also wondering if there's some correlation with the number of processes being scanned and added to the picker and at what rate, based on the fact that in the more limited container env with few processes I can't repro, but outside of that context on either my personal machine running a bunch of stuff or my corporate machine with some massive quantity of background processes it does.
Sorry that's not much to go on but just thought I'd follow up with as far as I was able to get on my own.

Since the display is right, and also because it likes to select the adjacent item, I'm suspicious of the picker itself. In the quickpick we add items as we discover them and re-assign the array, which I probably uncommon: https://github.com/microsoft/vscode-js-debug/blob/0b40365364f2867691701bff5aedf68d7fa0fe9d/src/ui/processPicker.ts#L169-L181
@TylerLeonhardt do you already have an issue tracking this?

Can you try VS Code Insiders? I believe this is already fixed there.

@TylerLeonhardt  Thank you! This appears fixed on insiders. Have confirmed on:
Version: 1.89.0-insider (Universal)
Commit: b58957e67ee1e712cebf466b995adf4c5307b2bd
Date: 2024-05-01T02:05:29.995Z
Electron: 28.2.8
ElectronBuildId: 27744544
Chromium: 120.0.6099.291
Node.js: 18.18.2
V8: 12.0.267.19-electron.0
OS: Darwin arm64 23.4.0

Just for my curiosity, any chance you could point me to the PR or commit that fixed it?
Thank you both!

This is what originally fixed your issue (this fix is in 1.89 stable): https://github.com/microsoft/vscode/pull/210803
But in the latest insiders (1.90) has an improvement on that with this fix: https://github.com/microsoft/vscode/pull/211399


fix #602

Sorry, previous PR needs a fix for #602.

This PR is superseded by #604, in case the CI idea is accepted. Otherwise this is still good.

Not needed anymore


Stop landing pages from rendering at the dynamic route

What does this change?
#11341
Currently in prod, we render all pages tagged with the correct section at /[section]/[uid], which means that the landing page (e.g. 'visit-us'), which is tagged with the 'visit-us' section, also renders there: https://wellcomecollection.org/visit-us/visit-us
We never link to that URL, but I'm thinking we also don't want to allow it, so this covers that.
How to test
Run locally and try all landing, like http://localhost:3000/visit-us/visit-us (or get-involved, or about-us)
Make sure it doesn't break anything that should work though.
How can we measure success?
Just less confusion and clearer rules for Page paths.
Have we considered potential risks?
N/A if we test right and tests pass.

Address this comment within this PR https://github.com/wellcomecollection/wellcomecollection.org/pull/11353/files#r1822641388


Update the Gemfile.lock for security purposes.

This PR updates the dependencies in the Gemfile.lock to ensure we have the latest security updates.

Thanks @caleb-livefront


Frame: add mapping for CornerRadius, BorderColor

i hate the autoformater!!!
closing because missleading changes from bot.
Description of Change
Frame: add mapping for CornerRadius, BorderColor
refactored from: https://github.com/jsuarezruiz/maui-linux/pull/51
Issues
needs testing in eg. Control.Samples.Gtk
currently showing a frame without content doesn't work in LayoutView

i hate the autoformater!!!
closing because missleading changes from bot.

@knocte looks better now


Quests not loading or able to be created

(Required) What is your Quests version / build number (do not say "latest")?
Quests-3.4.2   on minecraft 1.13.2 with paper spigot
(Required) Please describe the problem and how to replicate it. Any console errors?
Basically, I load in a fresh install of quests and open up the editor. Everything works completely fine and I'm able to do everything I've tested. I can create the quest, edit stages- etc.
The problem starts with finishing the quest. I save it and am then prompted with reloading the quests plugin with the command. That command ( /questadmin reload) doesn't work and it leaves an "Internal error etc." message. That leaves an error in the server and nothing happens. Restarting the server just removes the quest and then it's completely gone and my progress disappears and there are no quests.
Command Error : https://pastebin.com/iLPy1XmX
Full Log : https://pastebin.com/t8bpGLcf

Looks like a combination of issues. First, you have two copies of Vault in your /plugins folder - one named “Vault.jar” and “Vault-1.6.7.jar”. Secondly, the MythicMobsQuests module is failing to load. Not sure as to the cause, but that’d be a question for the developer of that module.
Feel free to open a new issue if you still experience hiccups after resolving those problems.
Sent with GitHawk

Yea sorry, I didn't notice those problems. Unfortunately, neither fix the issue of quests not loading. To make sure I loaded a new server with only quests on it and I'm still receiving the same errors. My bad, I'm sure it's something dumb like having two vault jars but I just can't seem to figure it out : (
I also tried a few other things, like running in spigot over paper but it seems to have the same issue. The same thing seems to happen on 1.13.1. Just to clarify, there aren't any requirements for quests to run correctly, are there? I had quests on 1.12 and it worked perfectly, and I might have forgotten something as we ported the server over. I looked over the spigot page several times and didn't really see anything.
On Clean Server with only quests.jar (3.4.2) :
Command Error: https://pastebin.com/p6Qvs8Pm
Full Log: https://pastebin.com/kCWuCG5X

Looks like this actually a duplicate of #521
It's already been fixed, so using the latest dev build should resolve the issue. I'll go ahead and publish it to Spigot as well.

Ahh, you're right. Sorry for being redundant, I should have checked the latest GitHub builds asjdfkla;sjdflka. I just stupidly assumed since they were named the same it wasn't different! Thanks for the quick responses and help, quests is now working perfectly again : )


Artifacts on iPad Air 2 Reported

I am receiving customer reports that most of the images they use have square artifacts evident in the my filtered output. I am just using the levels filter for this app. I am not able to reproduce the issue here.
An example of the issue is here (created running on an iPad Air 2 iOS 8.1):
http://3DTOPO.com/IMG_4638.jpg
And the original source image provide is here:
http://3DTOPO.com/orig-IMG_4638.jpg
I thought at first I might be doing something dumb like using a float for pixel dimensions, or maybe I was using an odd row or column of pixels, but it all looks fine.
Any ideas would be super appreciated!

I should also mention the same customer reports this original image works fine for them (all else the same in the app):
http://3DTOPO.com/IMG_4626.JPG

This sounds like the exact same issue as #1823, and looks to be somehow related to the GPUs in the iPad Air and iPhone 6 models. Nothing else exhibits this, which makes me think it's either a precision-related bug or a driver issue.

Thank you very much for the link and info Brad. I wish I could reproduce the issue so I could look into it further. Is there anything you want me to log remotely from the customers install? They are willing to install a special version for testing.

Honestly, I don't know what could be causing it, and I think we've pretty well identified that it's just the iPad Air 2 and iPhone 6 / 6+ models under iOS 8 experiencing this. Don't know what other information could help to diagnose it. If you have one of those devices handy, you should be able to replicate this yourself pretty easily.
Maybe performing a frame capture analysis in Xcode could let you step through and see where this might originate, but if it's a driver bug there might be little we can do until Apple fixes this short of filing a bug report or using an ADC incident.

OK - I am able to repo the issue here - sorry I realized I hadn't checked my iPhone 6+ until after I had sent the message.
I am able to resolve the issue by simply passing NO to initWithImage:smoothlyScaleOutput: instead of YES. Perhaps that might shed some light? I will do more prodding.

Interesting. Mipmapping is enabled with smoothlyScaleOutput: is set to YES. I wonder if there's a breakdown in the mipmapping process on those devices. It's also resizing the image to power-of-two dimensions to support the mipmapping, so there might be something in that process going wrong as well.

Interesting!
Hey Brad here are a couple promo codes for my ComicBook! app that uses your lib for some of the filters (was just Starbuck's Pick of the Week): XLWJF69939JF W69JRMMY6HXE
https://itunes.apple.com/us/app/comicbook!/id436114747?mt=8


Flaky Test : TestLinksUpdateOnRestart

03:41:17 ----------------------------------------------------------------------
03:41:17 FAIL: docker_cli_links_test.go:130: DockerSuite.TestLinksUpdateOnRestart
03:41:17
03:41:17 docker_cli_links_test.go:158:
03:41:17     ip = getIP(content, "one")
03:41:17 docker_cli_links_test.go:144:
03:41:17     c.Assert(matches, checker.NotNil, check.Commentf("Hostname %s have no matches in hosts", hostname))
03:41:17 ... value [][]uint8 = [][]uint8(nil)
03:41:17 ... Hostname one have no matches in hosts
03:41:17
03:41:18
03:41:18 ----------------------------------------------------------------------

https://jenkins.dockerproject.org/job/Docker-PRs-userns/6545/console
https://jenkins.dockerproject.org/job/Docker-PRs-userns/6548/console

If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
For more information about reporting issues, see CONTRIBUTING.md.
You don't have to include this information if this is a feature request
(This is an automated, informational response)

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO


(Draft) WIP: Dependency analyzer

Closing this PR for since we have a working implementation. We can revisit this once regorus is feature complete.


Where can I report vulnerability details?

Hi, I found some vulns, Is there any security related mail I can send my report?
P.S. I checked the website and GitHub, but I couldn't find where I can send the report :(

any update you can share about this?

i've received two reports. one is not about koa itself but one of its middleware (which is this issue). another is about setting headers, which was vague because they didn't describe the actual attack. yeah, a client can set whatever headers they want...

thanks for the update @jonathanong

yeah, a client can set whatever headers they want...

@jonathanong I can't seem to find the comment, but vaguely recall semi-consensus around adding some form of app.trustHeaderfield() API?

I would also add a possibility of spoofing an IP address when behind an Nginx server: https://github.com/koajs/koa/issues/599#issuecomment-239493311


[MM-17285] Subscribe modal causes blank screen on desktop

Summary

We were using Array.flat() which isn't polyfilled by Babel until core-js 3, which we won't have until 5.16. So, this PR manually polyfills the function.
Unrelated cleanup: some props were marked as required but were being fetched async (so were often nil when the component was loaded) causing red in the console. I want to clean that up.

Link

Fixes MM-17285

Unrelated cleanup: some props were marked as required but were being fetched async (so were often nil when the component was loaded) causing red in the console. I want to clean that up.

Excellent :+1:

I think this is good to merge. Getting the flat() fix in will avoid rework.


feat(esp32c3): Support esp32c3 ble function

Summary
Adapter esp32c3 ble function.
Impact
Testing
NOTE: The CI will fail because are some pre-existing Mixed Case in the esp32c3_ble_adapter.c file

I've highlighted mostly styling issues and some questions about implementation.
I also think that creating a virtual serial port is incorrect, you should be exposing this as a bt_driver so that it ties as a network adapter.

@btashton can you verify this? Maybe I'm missing something, but I don't think this is the proper way to expose the BLE controller.

I've highlighted mostly styling issues and some questions about implementation.
I also think that creating a virtual serial port is incorrect, you should be exposing this as a bt_driver so that it ties as a network adapter.

@btashton can you verify this? Maybe I'm missing something, but I don't think this is the proper way to expose the BLE controller.

The BLE driver mainly for application level BT/BLE stack which R/W based on character device, and we will add btuart_driver for NuttX BT driver like sim's btuart_driver: https://github.com/apache/incubator-nuttx/blob/1bded73f9fb55a8183065c395eb6f1b82ad7b3b2/boards/sim/sim/sim/src/sim_bringup.c#L360 .

@v01d  is correct this should be exposed directly via the VHCI interface you already have not by creating a uart and then binding that uart to the BT stack.

I've highlighted mostly styling issues and some questions about implementation.
I also think that creating a virtual serial port is incorrect, you should be exposing this as a bt_driver so that it ties as a network adapter.

@btashton can you verify this? Maybe I'm missing something, but I don't think this is the proper way to expose the BLE controller.

The BLE driver mainly for application level BT/BLE stack which R/W based on character device, and we will add btuart_driver for NuttX BT driver like sim's btuart_driver:
https://github.com/apache/incubator-nuttx/blob/1bded73f9fb55a8183065c395eb6f1b82ad7b3b2/boards/sim/sim/sim/src/sim_bringup.c#L360
.

The right way to expose a BLE controller is via this interface: https://github.com/apache/incubator-nuttx/blob/master/include/nuttx/wireless/bluetooth/bt_driver.h as a wireless net device. NuttX host layer is based around sockets (nimBLE also uses this interface). I believe you can then expose this as a virtual tty device if you're using another stack which expects that interface.

Exposing bt_driver_s directly has some limitation at this stage:

NuttX just support BLE stack, but many SoC support BT and BLE, we need provide a solution for combo device.
NuttX doesn't have a full feature BLE userspace stack(something like bluez), it waste flash/memory to integrate other monolithic bluetooth stack(e.g. nimBLE) through the raw socket because:
a. All L2 layer code duplicate in kernel(wireless/bluetooth) and uerspace(e.g. nimBLE)
b. BLE only SoC has to bring in the unnecessary network infrastructure code(socket/bind/send/recv...)
Expose BT/BLE through tty driver interface, chip vendor can support all cases with one driver implementation:
a. nimBLE/bluedriod/btstack can work with /dev/tty directly, we don't need provide any wrapper at all.
b. With NuttX's bt_uart_shim driver, we can support NuttX BLE stack without any additional code.

From the architecture perspective, the best way is to implement/port bluez(profile only) liked stack to NuttX. All other monolithic stack support(nimBLE/bluedriod/btstack) should be dropped to promote the best architecture.

I appreciate we need to discuss a better option to NuttX's stack. I mentioned this in the past several times.
However, until the current situation changes, new code needs to conform to current state. That is, exposing BLE controllers via the corresponding bt_driver_s interface. Note that this doesn't mean the alternative tty-like interface supported in this PR needs to be eliminated, they can coexist (selected by an option, if necessary). This way, both use cases are possible until anything else changes in NuttX. Otherwise, we would be just choosing what interface to support as a personal preference and leaving options unsupported (such as nimBLE) in the meantime.
I don't think it is such a big effort to support, it is a very simple interface. I would prefer that this PR implements this interface for it to be merged.
@xiaoxiang781216 do you want to open an issue to discuss what to do w.r.t. NuttX host layer? We should discuss this separately and track it explicitly.

I appreciate we need to discuss a better option to NuttX's stack. I mentioned this in the past several times.
However, until the current situation changes, new code needs to conform to current state. That is, exposing BLE controllers via the corresponding bt_driver_s interface. Note that this doesn't mean the alternative tty-like interface supported in this PR needs to be eliminated, they can coexist (selected by an option, if necessary). This way, both use cases are possible until anything else changes in NuttX. Otherwise, we would be just choosing what interface to support as a personal preference and leaving options unsupported (such as nimBLE) in the meantime.

Yes, I agree in this stage, we need support all possible selection: NuttX BLE stack(L2 only), nimBLE and others.

I don't think it is such a big effort to support, it is a very simple interface. I would prefer that this PR implements this interface for it to be merged.

Vendor doesn't need provide a special bt_driver_s for it's SoC explicitly, NuttX already have a general implementation can be shared for all chipset. We just need call btuart_shim_getdevice in the board init like this:
#ifdef CONFIG_BLUETOOTH_UART_SHIM
  ret = btuart_register(btuart_shim_getdevice("/dev/ttyXXX"));
  if (ret < 0)
    {
      syslog(LOG_ERR, "ERROR: btuart_register() failed: %d\n", ret);
    }
#endif

bt_uart shim driver is designed for this:
https://github.com/apache/incubator-nuttx/blob/master/drivers/wireless/bluetooth/bt_uart_shim.c

@xiaoxiang781216 do you want to open an issue to discuss what to do w.r.t. NuttX host layer? We should discuss this separately and track it explicitly.

Yes, I think so.

Vendor doesn't need provide a special bt_driver_s for it's SoC explicitly, NuttX already have a general implementation can be shared for all chipset. We just need call btuart_shim_getdevice in the board init like this:
#ifdef CONFIG_BLUETOOTH_UART_SHIM
  ret = btuart_register(btuart_shim_getdevice("/dev/ttyXXX"));
  if (ret < 0)
    {
      syslog(LOG_ERR, "ERROR: btuart_register() failed: %d\n", ret);
    }
#endif

bt_uart shim driver is designed for this:
https://github.com/apache/incubator-nuttx/blob/master/drivers/wireless/bluetooth/bt_uart_shim.c

But that shim makes sense if we have an actual tty (such as a off-chip BLE chip with UART communication). In this case the code is emulating a tty to satisfy the interface you expect and on top of that you place the shim to expose it as a netdev. This is extremely convoluted and unnecessary. For an on-chip BLE controller the interface should be exposed via VHCI as Brennan mentioned. The right interface to do that is bt_driver_s. On top of that interface you could create a generic tty->bt_driver_s or simply expose it to the networking layer. Otherwise we're piling abstraction layers which do not represent a hardware requirement.
In other words, this shim should not be used for this purpose. But a different adapter which optionally exposes a bt_driver_s as a tty should be added.

Vendor doesn't need provide a special bt_driver_s for it's SoC explicitly, NuttX already have a general implementation can be shared for all chipset. We just need call btuart_shim_getdevice in the board init like this:
#ifdef CONFIG_BLUETOOTH_UART_SHIM
  ret = btuart_register(btuart_shim_getdevice("/dev/ttyXXX"));
  if (ret < 0)
    {
      syslog(LOG_ERR, "ERROR: btuart_register() failed: %d\n", ret);
    }
#endif

bt_uart shim driver is designed for this:
https://github.com/apache/incubator-nuttx/blob/master/drivers/wireless/bluetooth/bt_uart_shim.c

But that shim makes sense if we have an actual tty (such as a off-chip BLE chip with UART communication). In this case the code is emulating a tty to satisfy the interface you expect and on top of that you place the shim to expose it as a netdev. This is extremely convoluted and unnecessary.

Yes, it's little bit convoluted, we just want to reuse the stock infrastructure as much as possible.:(

For an on-chip BLE controller the interface should be exposed via VHCI as Brennan mentioned. The right interface to do that is bt_driver_s. On top of that interface you could create a generic tty->bt_driver_s or simply expose it to the networking layer. Otherwise we're piling abstraction layers which do not represent a hardware requirement.
In other words, this shim should not be used for this purpose. But a different adapter which optionally exposes a bt_driver_s as a tty should be added.

Yes, it's a reasonable solution, we will provide a generic tty->bt_driver_s to avoid the duplication.

Great, thanks. I think it will also reduce complexity of the ESP32C3 BLE driver.

@v01d bt_driver_s is coupled with bt_buf_s which has some problem:

bt_buf_s is part of bluetooth core statck
bt_buf_s invoke bt_hci_cmd_xxx directly
bt_buf_s need work with iob buffer

It doesn't make sense to bring above to tty->bt_driver_s wrapper, so I propose that bt_driver_s send/recv the raw buffer instead of bt_buf_s and change bt_hci_receive to callback like this:
https://github.com/xiaoxiang781216/incubator-nuttx/commit/a2ad5d922868d5ef4f9dbc7b6fbee3a641a06fb9
This change mimic the relationship between net_dev_s::d_buf and iob_s.

Ok, I understand the situation better now, thanks. You're right that there's too much coupling right now, so moving the receive into a callback is a good start in that direction.

Hi @all,
https://github.com/apache/incubator-nuttx/pull/3754 : Here is decoupling implementation, please help to review further, thanks!

Hi @protobits
I have refactored the tty driver, could you please help re-reivew this MR.

@xiewenxiang please fix all nxstyle warning except "Mixed case identifier found":
https://github.com/apache/incubator-nuttx/pull/3543/checks?check_run_id=2884573243

@xiewenxiang please fix all nxstyle warning except "Mixed case identifier found":
https://github.com/apache/incubator-nuttx/pull/3543/checks?check_run_id=2884573243

I have fix the warning except "Mixed case identifier found":

Please run:
./tools/refresh.sh --silent esp32c3-devkit/sta_softap
and fix the compiler error.

@xiewenxiang please fix the build break.


Speed up MessagePackPrimitives writers by using ref byte

It turns out that indexing into span, and using MemoryMarshal.Write, results in significantly more compiled assembly code and slower perf.

Thank you to @pCYSl5EDgo for showing me how much better ref byte can be.


Dev

PR into Azure/mlops-v2
Checklist
I have:

[x] read and followed the contributing guidelines

Changes

using mlflow run instead of azureml sdk run to get information about the current run id and existing registered models.
simplifying evaluate step (one output dir instead of multiple).
code quality check and review with pylint
adding first version of unit tests

fixes #

new unit tests structure

@setuc Responsible AI is now enabled for aml-cli-v2 and integrated with the Monitoring feature additions as well


Latency fps bar wrong position

The latency or fps bar is on the wrong position of the screen, this should be on the minimap, but is on the left top corner of the screen

This is intended to free up space on the minimap.


Is it possible to use images to redefine the  symbols?

Hi, is it possible to redefine the rating symbols in the following way?
I have tried, but it looks quite ugly...
css
.symbol-empty {
background-image: url("/ecosystem/images/market/st.h.png");
}
.symbol-filled {
background-image: url("/ecosystem/images/market/st.o.png");
}
.symbol {
width: 20px;
height: 20px;
}

Technically yes. You can use a background-image as symbol.
Not sure what you mean by looks quite ugly. Could you create a fiddle so I can try to give you more feedback? You can base it on this one.

You can try the following example, it seems that when rated the background image disappeared...
1.The html code is not modified.
2.The css file is modified as follows
css
.symbol {
width: 32px;
height: 32px;
}
.symbol-empty {
background-image: url('http://download.easyicon.net/png/1129027/32/')
}
.symbol-filled {
background-image: url('http://download.easyicon.net/png/1129028/32/');
}

You need to make symbols inline-block:
.symbol {
  display: inline-block;
  width: 32px;
  height: 32px;
 }
.symbol-empty {
  background-image: url('http://download.easyicon.net/png/1129027/32/')
}
.symbol-filled {
  background-image: url('http://download.easyicon.net/png/1129028/32/');
}

Check fiddle.
NOTE: You will see a little glitch the first time you hover a symbol. It takes a little bit to show the filled symbol. I guess it is because it needs to retrieve the filled image. Once this image is loaded/cached then everything works as expected :smile:. If this is the case you can try some sort of image preload.

@dreyescat Greate, it works perfectly. Thanks a lot for your kindly response.


Quality after exporting pointcloud or mesh

Great work! It seems that nerfStudio's pointcloud or mesh export does not retain the best quality when viewed in the web viewer. Can you show me the exported mesh file of the demo video? I need them to display on mesh file readers like meshlab or threejs.
Thanks.

I've never did an export of the NeRF in the demo video, it was a screen capture of the viewer! If you're looking for advice with creating better meshes I would recommend posting in the Nerfstudio Discord channel. The folks there will probably know more about tweaking your setup for mesh generation.
Also, I would recommend testing on a static dataset extracted from a rosbag, and then once you have your settings locked in you can transfer to Nerfbridge.

Closing due to inactivity.


use Tables to organize Events

Tables make it easy to see patterns and trends in Events.
Tables make it easy to compare different Events and date.
Tables are a concise way to present data.
Tables can help to reduce errors.
Tables can help to save time in reading data

Name
Description
Timeline
Date
Venue

[Hacktober ]
Description of Hacktober
Jan 1 - Jan 5
Jan 3, 2023
Venue 1

here is an example..
if you don't mind I done and I will like open a PR


Make visibility range (HLOD) dithering work when prepasses are enabled.

Currently, the prepass has no support for visibility ranges, so artifacts appear when using dithering visibility ranges in conjunction with a prepass. This patch fixes that problem.
Note that this patch changes the prepass to use sparse bind group indices instead of sequential ones. I figured this is cleaner, because it allows for greater sharing of WGSL code between the forward pipeline and the prepass pipeline.
The visibility_range example has been updated to allow the prepass to be toggled on and off.

https://github.com/user-attachments/assets/d6dcbcf6-2e03-4318-8524-e59fd2b4f532
There is a very noticeable flicker when the model changes with the prepass enabled.

Turns out that the bug that @BenjaminBrienen noticed is actually a regression from a much earlier PR. I submitted a fix at https://github.com/bevyengine/bevy/pull/16468.


java_stub_template to use argument file instead of classpath jar

Description
java_stub_template to use argument file instead of classpath jar
This shaves 1s-1.5s when classpath is long.
Requires java > 8 so to properly add it in we'll probably have a toggle or use the toolchain or something
https://github.com/bazelbuild/bazel/issues/6354
Motivation
performance

@liucijus we're good to go. Build is green.
One thing I really don't like about the current test is that it doesn't really test anything.
At maximum it tests there is such an attribute in the toolchain.
It doesn't check how it's used.
The main reason is because I'm not sure how to write non-flaky tests that compare perf.
Also the perf is felt when the classpath is huge so that is another problem.
I thought about maybe grepping through the bash script and see if it has True or False (after the substitutions that happen in starlark) but that feels wrong.
WDYT?

Btw, I think there's something flaky
https://buildkite.com/bazel/rules-scala-scala/builds/3951#0184ab62-2028-4d07-8a2f-d2048ed3a80f:

2022/11/24 20:48:36 Using unreleased version at commit 6d07ee5927283ecdab18822cc5a1a8a9a64d4074

Loading:

Loading: 0 packages loaded

INFO: Build option --extra_toolchains has changed, discarding analysis cache.

Analyzing: target //test_expect_failure/missing_direct_deps/internal_deps:unstamped_jar_dependent_on_some_java_provider (0 packages loaded, 0 targets configured)

INFO: Analyzed target //test_expect_failure/missing_direct_deps/internal_deps:unstamped_jar_dependent_on_some_java_provider (0 packages loaded, 894 targets configured).

INFO: Found 1 target...

[0 / 2] [Prepa] BazelWorkspaceStatusAction stable-status.txt

ERROR: /workdir/test_expect_failure/missing_direct_deps/internal_deps/BUILD:117:14: scala @//test_expect_failure/missing_direct_deps/internal_deps:unstamped_jar_dependent_on_some_java_provider failed: (Exit 1): scalac failed: error executing command (from target //test_expect_failure/missing_direct_deps/internal_deps:unstamped_jar_dependent_on_some_java_provider) bazel-out/k8-opt-exec-2B5CBBC6/bin/src/java/io/bazel/rulesscala/scalac/scalac ... (remaining 1 argument skipped)

error: Target 'Unknown label of file bazel-out/k8-fastbuild/bin/test_expect_failure/missing_direct_deps/internal_deps/transitive_dependency_without_manifest.jar which came from @//test_expect_failure/missing_direct_deps/internal_deps:unstamped_direct_java_provider_dependency' is used but isn't explicitly declared, please add it to the deps.

You can use the following buildozer command:

buildozer 'add deps Unknown label of file bazel-out/k8-fastbuild/bin/test_expect_failure/missing_direct_deps/internal_deps/transitive_dependency_without_manifest.jar which came from @//test_expect_failure/missing_direct_deps/internal_deps:unstamped_direct_java_provider_dependency' //test_expect_failure/missing_direct_deps/internal_deps:unstamped_jar_dependent_on_some_java_provider

one error found

Build failed

java.lang.RuntimeException: Build failed

	at io.bazel.rulesscala.scalac.ScalacWorker.compileScalaSources(ScalacWorker.java:290)

	at io.bazel.rulesscala.scalac.ScalacWorker.work(ScalacWorker.java:61)

	at io.bazel.rulesscala.worker.Worker.persistentWorkerMain(Worker.java:86)

	at io.bazel.rulesscala.worker.Worker.workerMain(Worker.java:39)

	at io.bazel.rulesscala.scalac.ScalacWorker.main(ScalacWorker.java:25)

Target //test_expect_failure/missing_direct_deps/internal_deps:unstamped_jar_dependent_on_some_java_provider failed to build

Use --verbose_failures to see the command lines of failed build steps.

INFO: Elapsed time: 1.029s, Critical Path: 0.93s

INFO: 4 processes: 2 internal, 2 worker.

FAILED: Build did NOT complete successfully

'bazel build //test_expect_failure/missing_direct_deps/internal_deps:unstamped_jar_dependent_on_some_java_provider' should have logged "Unknown label of file .*transitive_dependency_without_manifest.jar which came from //test_expect_failure/missing_direct_deps/internal_deps:unstamped_direct_java_provider_dependency".

 Test "test_scala_library_expect_better_failure_message_on_missing_transitive_dependency_labels_from_other_jvm_rules" failed  (1 sec)

@liucijus we're good to go. Build is green. One thing I really don't like about the current test is that it doesn't really test anything. At maximum it tests there is such an attribute in the toolchain. It doesn't check how it's used. The main reason is because I'm not sure how to write non-flaky tests that compare perf. Also the perf is felt when the classpath is huge so that is another problem. I thought about maybe grepping through the bash script and see if it has True or False (after the substitutions that happen in starlark) but that feels wrong.
WDYT?

@ittaiz I don't think you should test perf with such tests. The goal of such test should be to understand if everything wires up correctly and does correct build. I don't think we have any easy way to do perf regression here. So we have to trust that you've done your own measurements.

Btw, I think there's something flaky https://buildkite.com/bazel/rules-scala-scala/builds/3951#0184ab62-2028-4d07-8a2f-d2048ed3a80f:

2022/11/24 20:48:36 Using unreleased version at commit 6d07ee5927283ecdab18822cc5a1a8a9a64d4074

Loading:

Loading: 0 packages loaded

INFO: Build option --extra_toolchains has changed, discarding analysis cache.

Analyzing: target //test_expect_failure/missing_direct_deps/internal_deps:unstamped_jar_dependent_on_some_java_provider (0 packages loaded, 0 targets configured)

INFO: Analyzed target //test_expect_failure/missing_direct_deps/internal_deps:unstamped_jar_dependent_on_some_java_provider (0 packages loaded, 894 targets configured).

INFO: Found 1 target...

[0 / 2] [Prepa] BazelWorkspaceStatusAction stable-status.txt

ERROR: /workdir/test_expect_failure/missing_direct_deps/internal_deps/BUILD:117:14: scala @//test_expect_failure/missing_direct_deps/internal_deps:unstamped_jar_dependent_on_some_java_provider failed: (Exit 1): scalac failed: error executing command (from target //test_expect_failure/missing_direct_deps/internal_deps:unstamped_jar_dependent_on_some_java_provider) bazel-out/k8-opt-exec-2B5CBBC6/bin/src/java/io/bazel/rulesscala/scalac/scalac ... (remaining 1 argument skipped)

error: Target 'Unknown label of file bazel-out/k8-fastbuild/bin/test_expect_failure/missing_direct_deps/internal_deps/transitive_dependency_without_manifest.jar which came from @//test_expect_failure/missing_direct_deps/internal_deps:unstamped_direct_java_provider_dependency' is used but isn't explicitly declared, please add it to the deps.

You can use the following buildozer command:

buildozer 'add deps Unknown label of file bazel-out/k8-fastbuild/bin/test_expect_failure/missing_direct_deps/internal_deps/transitive_dependency_without_manifest.jar which came from @//test_expect_failure/missing_direct_deps/internal_deps:unstamped_direct_java_provider_dependency' //test_expect_failure/missing_direct_deps/internal_deps:unstamped_jar_dependent_on_some_java_provider

one error found

Build failed

java.lang.RuntimeException: Build failed

	at io.bazel.rulesscala.scalac.ScalacWorker.compileScalaSources(ScalacWorker.java:290)

	at io.bazel.rulesscala.scalac.ScalacWorker.work(ScalacWorker.java:61)

	at io.bazel.rulesscala.worker.Worker.persistentWorkerMain(Worker.java:86)

	at io.bazel.rulesscala.worker.Worker.workerMain(Worker.java:39)

	at io.bazel.rulesscala.scalac.ScalacWorker.main(ScalacWorker.java:25)

Target //test_expect_failure/missing_direct_deps/internal_deps:unstamped_jar_dependent_on_some_java_provider failed to build

Use --verbose_failures to see the command lines of failed build steps.

INFO: Elapsed time: 1.029s, Critical Path: 0.93s

INFO: 4 processes: 2 internal, 2 worker.

FAILED: Build did NOT complete successfully

'bazel build //test_expect_failure/missing_direct_deps/internal_deps:unstamped_jar_dependent_on_some_java_provider' should have logged "Unknown label of file .*transitive_dependency_without_manifest.jar which came from //test_expect_failure/missing_direct_deps/internal_deps:unstamped_direct_java_provider_dependency".

�[0;31m Test "test_scala_library_expect_better_failure_message_on_missing_transitive_dependency_labels_from_other_jvm_rules" failed �[0m (1 sec) �[0m

I think tests got broken with  Bazel 6 changes in backwards incompatible way.

Re perf and test- my point is that if you would keep the attribute on the toolchain but stop using it and also of course not pass it to the binary runner script the test still passes.
I can grep through the script to see if the value (True/False) is passed to the script but that test is more brittle and white box.
Wdyt?

I think grepping for the value is useful check

I'll change the test and assert on that

@liucijus done

I think the macos build is stuck for some reason. Maybe buildkite related. Running for 20m.
I tried canceling it from the UI but that didn't work.
Might need to timeout and  then rerun.
All other builds pased

@liucijus if you have a way to maybe stop and start the build that would be great.

pushed another commit since that build is stuck

./test_reproducibility.sh  is running for 2h. Probably flaky.

Thanks, @ittaiz!

😊 sure thing. Thank you both for your review!

Oh, I just realized I didn't update the docs :(


[Miner] feat: add TxClient

Summary
Human Summary
---
title: TxClient Dependency Graph
---

flowchart

    subgraph tclient[TxClient]
        tctx[TxContext]
        builder[TxBuilder]
        keyring[Keyring]

        bclient[BlockClient]
        eclient[EventsQueryClient]

        tclient_internal((_))
    end

    chain[Blockchain]

    tctx --"#GetKeyring()"--> keyring
tctx --"#SignTx()"--> builder
tctx --"#EncodeTx()"--> builder
tctx -."#BroadcastTx()".-> chain
tctx -."#QueryTx()".-> chain

eclient -."websocket connection".-> chain
bclient --"committed  block subscription"--> eclient
tclient_internal --"own tx subscription"--> eclient

AI Summary

Summary generated by Reviewpad on 03 Nov 23 20:39 UTC
This pull request introduces various changes to multiple files.
In the file godoc.go, a new package-level file is included that provides an abstraction layer for interacting with blockchain functionalities using external libraries like cosmos-sdk and cometbft.
In the file context.go, changes are made to improve function names and comments related to transactions in the cosmos-sdk client package. The BroadcastTx function is modified to use BroadcastTxAsync instead of BroadcastTxSync, enhancing code clarity.
A new file errorsmod.go is added in the pkg/client/tx directory, which defines error variables and their respective error messages. These error variables are registered using the Register function from the errorsmod package.
A new test file client_integration_test.go is added to the pkg/client/tx directory, which includes a test function for the SignAndBroadcast method of the TxClient struct. The test function sets up required dependencies and asserts that no error occurs during the execution.
Changes are made to improve the readability of the code in a file without impacting its functionality.
A new file options.go is added in the pkg/client/tx directory, which introduces functions WithCommitTimeoutBlocks and WithSigningKeyName to set timeout duration and signing key name options for the TxClient struct.
A new file events_query_test.go is added that includes test helper functions and mocks for testing events query functionality. It also updates the NewLocalnetClient function to allow for configuration options.
The README.md file undergoes changes, including removal of lines describing key functions and types, modification of a link to the complete API details, and removal of sections on contributing, the changelog, and license information.
A new file services.go is added that contains a function to generate a slice of application service configs for testing purposes.
A new file encoding.go is added that includes functions to normalize transaction hash hexadecimal strings.
A new README.md file is added to provide documentation for the client package, which abstracts the complexities of interacting with blockchain data and ensures a consistent experience across different blockchains.
Changes are made to the NewReplayObservable function, adding a new parameter and updating the NewObservable function call.
Modifications are made to the interface.go file, including the addition of //go:generate statement for generating a mock implementation, modification of existing interface methods, and addition of new function types.
New code is added to the internal/testclient/testblock/client.go file, including package comments, import statements, and functions for creating and utilizing mock implementations and utilities for blockchain-related functionality.

Issue
Relates to:

#13

TxClient & TxContext are used to build, encode, sign, and  broadcast transactions from an off-chain context.
Type of change
Select one or more:

[x] New feature, functionality or library
[ ] Bug fix
[ ] Code health or cleanup
[ ] Documentation
[ ] Other (specify)

Testing

[x] Run all unit tests: make go_test
[x] Verify Localnet manually: See the instructions [here](TODO: add link to instructions)

Sanity Checklist

[x] I have tested my changes using the available tooling
[x] I have performed a self-review of my own code
[x] I have commented my code, updated documentation and left TODOs throughout the codebase

@bryanchriswhite Checking in whether this is ready for another review yet or not?


Configure CORS for proxy routes

What should we add or change to make your life better?
Support adding CORS metadata for individual routes and group routes (https://github.com/microsoft/reverse-proxy/issues/164).
CorsMiddleware supports three types: IDisableCorsAttribute, ICorsPolicyMetadata and IEnableCorsAttribute.
I suggest YARP should only supports IDisableCorsAttribute and  IEnableCorsAttribute. It would be the same as authorization that we can only set Policy name and not authentication schemes or roles.
Why is this important to you?
I want to use YARP as a backend for a SPA and I want to check CORS before sending the request to the API Server.

Forked from https://github.com/microsoft/reverse-proxy/issues/87#issuecomment-652933954


fixed merge error

Sorry for this kasemir, I've noticed and issue I've generated during the merge.

Should already be fixed on 'master'


i try to publish on vm

Problem encountered on https://dotnet.microsoft.com/learn/web/aspnet-hello-world-tutorial/run
Operating System: Linux Ubuntu 18.04
Provide details about the problem you are experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.
This site can’t be reached <IP_ADDRESS> refused to connect.
Search Google for<PHONE_NUMBER>
ERR_CONNECTION_REFUSED

@rakesh4951 what is the command you run?
Did you check your OS? Did you try another (brand new) VM to avoid firewall and other VM-setup-specific problems?

No response, closing. Feel free to reopen when there's more info. Thanks!


moved empty descriptions from subelement connection-selection to post…

fixes #826 empty views are no longer visible when there are connections present

Nope. the matches and incoming request views in the need detail view are still broken

sent requests is also empty (when I request contact for a match)

when connected, messages are empty too


Fix ignore match expression logging warning message (#41546)

pick: #41546
when match do not contains slot reference it would throw an exception when translate to original planner expr.
this kind of message is not need to be recorded
Proposed changes
Issue Number: close #xxx

run buildall


Updates to Deploy Documentation

Would it be possible to provide updated install docs?  The step by step doc seems to focus on Tiny Blazor Admin and not the shortner app.  Its mildly confusing.
This project is great and love the notes about the re-write. Looking forwarding to being able to successfully run this

Yes, good idea. Thank you for the suggestion @fatpacket

I will happily test and validate documentation. Run through it and make edits as needed, but as of this moment I could not get it to work.

@fatpacket the documentation for the AzUrlShortener has been updated here: https://github.com/FBoucher/AzUrlShortener/wiki/Deployment-with-the-Deploy-to-Azure-button

Thank you! I got the URL Shortner installed and the test api redirect worked.  When I inserted a record to the table I am unable to access the short link via the functions url + / {{RowKey}}  I get a 404 not found.  So I tried to install BlazorTinyAdminv3 and followed those instruction but get an error that the SWA needs content deployed to it.  See Issue Here
This function/api is exactly what I am looking along with the nice smooth front end. I am really hope you might be able to help me get this working.  Definitely want to discuss how to secure the api as well so that only my website can submit requests for shortening.
Thank you for all the work to you and the team

@fatpacket until you have configured a custom domain and applied some rules you will need to use the .../api/redirect/{vanity} URL.
I created a new URL with Azure Storage following these instructions, and it works. Have a look maybe you forgot something.
We will review the documentation about how to set up a custom domain, and deploy the TinyBlazorAdmin.
Stay tuned for more.

Presuming the custom domain needs to be hosted with Azure.  Whats the linkage from the function to the custom domain? Where is all that setup? Or is it just the config variable?

@fatpacket all the doc is now up to date in the wiki.

Domain can come from elsewhere if you want my c5m.ca was bought on goDaddy...

@fatpacket is anything still missing?

I have run through the documentation and the custom domain video and documentation is working well.
The deployment works great from the button and creates all the required resources.
The Storage explorer is a decent front end but the documentation should include a how to for the TinyAdminv3 front end as well. When you review the TinyAdminv3 documentation you see my issue over there.   TinyAdminv3 #94


ftplugin not loaded

Explain the problem here ...

Type:

[x] Bug
[x] Enhancement
[ ] Feature Request
[ ] Question

OS:

[ ] All/Other
[ ] Linux
[ ] OS X
[ ] Windows

Vim:

[ ] Terminal Vim
[ ] GVim
[ ] Neovim

Not enough information, and I haven't experienced any problem with ftplugin.


inference  problem

FCN8sShuffleNet  last layer  is upscore  with  deconv stride  8  .this layer is very  time comsuming  .on jetson tx2 .this layer  occupy 180 ms .  any optimization ？

last layer deconv  stride is big,that make slow .  how to address  this problem?

If you use the tensorflow optimize_inference tool it will improve the overall performance of the network to run faster. We are working on a version of the code that has optimized inference for all the architecture and will release it. It is initially available in optimize_inference branch you can take a look there.


How to install substreams CLI for ubuntu

I try to install by docs
But it's not run on ubuntu.

Please provide details of exactly the steps you are taking and what is the error messages.
Also you might want to start a discussion in the StreamingFast discord channel instead of using github issues to do troubleshooting.


Find all refs/rename for mapped type symbols via Record

TypeScript Version: 2.9.2
Code
Renaming property a in object o should rename both the property definition in type T and the property usage in value o, but it doesn't:
{
    type T = { a: number };
    type O = Record<keyof T, {}>;
    var o: O = { a: 3 };
}

Also observe that selecting the property a does not highlight the corresponding type:

However, if we replace Record with its underlying implementation, renaming does work:
{
    type T = { a: number };
    type O = { [key in keyof T]: {} };
    var o: O = { a: 3 };
}

Observe that selecting the property a correctly highlights the corresponding type:

I originally opened this at https://github.com/Microsoft/TypeScript/issues/20272, however it was marked as a duplicate which I believe was incorrect: https://github.com/Microsoft/TypeScript/issues/20272#issuecomment-348968423.
Why does this work when using mapped types and keyof directly, but not when using the Record type alias which does exactly the same thing under the hood? Given they are essentially the same thing, surely this difference in behaviour is a bug?

Thanks @andy-ms. Good to know.


ci: added support for additional cockroachdb tests

This PR adds cockroachdb tests for:

[x] db execute
[x] migrate dev
[x] migrate diff

Closes #https://github.com/prisma/prisma/issues/13268

Note: merging https://github.com/prisma/prisma/pull/13547 is necessary to make this PR's MacOS tests succeed.


Fix S3Producer

Changed S3Producer to use S3Constants.S3_OPERATION instead of EC2Constants.OPERATION

Thanks for spotting. I'm gonna merge.

Thanks for the PR :-)


polygon.py example at http://toblerity.org/shapely/manual.html#polygons doesn't work

kbriggs:~> python polygon.py
Traceback (most recent call last):
File "polygon.py", line 3, in
from descartes.patch import PolygonPatch
ImportError: No module named descartes.patch

@pelson wrote:

Looks like figures is in the same directory. Are you running from within the folder containing polygon?

Most likely I'm missing something, but I'm running into the same issue (ImportError: No module named figures). To which directory are you referring to exactly?

I believe the example code isn't installed by pip. You can get the full source of the examples on github, including the necessary figures.py https://github.com/Toblerity/Shapely/tree/master/docs/code


Update AZ400_M10_Creating_a_Release_Dashboard.md

Module: 10
Lab/Demo: 02
Fixes #80.
Changes proposed in this pull request:

Adds instruction to change DevOps starter setting from GitHub to Azure DevOps.

This has been fixed with PR #69.


Changing the CNI_CONF_NAME to 10-calico.conflist in calico yaml causes CNI to fail

Changing the CNI_CONF_NAME to 10-calico.conflist in calico yaml causes CNI to fail.
Expected Behavior
The CNI should continue to function when the name is changed to .conflist to support multiple plugins
Current Behavior
In hosted calico for kubernetes, when the CNI_CONF_NAME is changed to 10-calico.conflist (I did this to add the portmap plugin along with the calico cni plugin I already had there) and then calico-node is restarted on the worker nodes, the old 10-calico.conf file is not removed.  Having both the old and new configuration files in /etc/cni/net.d/ causes the CNI to fail, and new pods can not get an IP address.
Possible Solution
When the .conflist (or file with a different new name) is generated by the install-cni container, it could look for other calico config files and either delete them, or rename them in such a way that CNI ignores them (if this is possible)
Steps to Reproduce (for bugs)

Install kubernetes with hosted calico, using the default cni conf file name and just the calico plugin
Update the calico-config config map to add the portmap plugin, and update the calico-node daemonset to set CNI_CONF_NAME is changed to 10-calico.conflist
Wait for a few minutes for the calico-node pods to restart and for the CNI config change to be written to the new conflist file
Try to create a pod, and see that it will not get an IP address (CNI errors are reported in the logs, I believe due to multiple config files
Delete the old config file on each worker node manually, and the cluster will start working again

Context
Trying to update calico to use the portmap plugin as well causes the whole cluster to be hosed.  Manually deleting the conf file from each worker nodes first requires that you know the inner workings of calico to know how the CNI conf file is handled and where it is, and also can require a lot of manual steps (or the writing of a script) for clusters with many worker nodes.
Your Environment

Calico version: 2.6.1
Orchestrator version (e.g. kubernetes, mesos, rkt): kubernetes 1.7.4
Operating System and version:  Ubuntu 16.04

@bradbehle thanks for raising.
We've actually merged this PR, which does what you're suggesting: https://github.com/projectcalico/cni-plugin/pull/392
It's in v3.0 but hasn't made it to v2.6 yet. I've added this to the v2.6.4 milestone to get this cherry-picked.

(CNI errors are reported in the logs, I believe due to multiple config files

One thing though, I'm surprised there would be errors related to multiple config files. I think the kubelet just picks the first one and ignores the rest.
Could you share some of these logs?

Here is an excerpt of the syslog on the worker node where I found the CNI unable to handle the two conf files (one with .conf extension and one with .conflist extension).  Let me know if you want other logs, but I think this should show it clearly.
cni_extra_conf_files_logs.txt

NetworkPlugin cni failed to set up pod "nginx-2589664690-tx4ks_default" network: no plugin name provided

Looks like this is the primary error I'm seeing in those logs, which I think just means the top-level config is missing the name field.

Is there a way to work around this so we can update to use portmap (and the needed .conflist suffix) and not hit this CNI error?  Or do we need the fix that is coming in 2.6.4?

@bradbehle you shouldn't need the fix coming in 2.6.4.
I would think that a CNI config called 10-calico.conflist with the following contents would work:
    {
      "name": "k8s-pod-network",
      "cniVersion": "0.3.0",
      "plugins": [
        {
          "type": "calico",
          "etcd_endpoints": "__ETCD_ENDPOINTS__",
          "log_level": "info",
          "mtu": 1500,
          "ipam": {
              "type": "calico-ipam"
          },
          "policy": {
              "type": "k8s",
               "k8s_api_root": "https://__KUBERNETES_SERVICE_HOST__:__KUBERNETES_SERVICE_PORT__",
               "k8s_auth_token": "__SERVICEACCOUNT_TOKEN__"
          },
          "kubernetes": {
              "kubeconfig": "/etc/cni/net.d/__KUBECONFIG_FILENAME__"
          }
        },
        {
          "type": "portmap",
          "snat": true,
          "capabilities": {"portMappings": true}
        }
      ]
    }

@bradbehle did you have any luck with adding a name to the config?

The soon-to-be-released v2.6.4 will delete the old 10-calico.conf file which so there won't be multiple manifests.

every time i change things in /etc/cni/net.d/10-calico.conflist it gets overwritten on restart. Any idea how could i change it permanently?

@vzuevsky that file is written from a configmap. Are you using the operator install or manifest install?

Hi Casey, figured that out. Manifest was updated. I am curious though, where all those settings end up living after the manifest is applied - couldn't find them by grepping. Re-applying would be the right way anyway, so feel free to close this chain.


Is the RMSprop func right?

The function create a new shared variable named 'acc' when it is called. Will the values be accumulated in 'acc'?

Yes, it's a bit weird to think about but the RMSprop function is only "called" once with the goal of creating sets of operations for theano to compile/execute many times.
What's important is that the updates function is tuples/list of lists/ordered dict of the parameters at the current time step and then a sequence of expressions resulting in the "updated" parameters, which is what this function does.

I see. Thanks a lot.


[CI] Add documentation in main README for how to start task runner/execute project code

Migration to UV requires update of documentation


Search has unintended behavior when searching for "new"

Searching for "new" results in just a normal search page. This is because we have the search form being displayed on search_queries/new, which has a higher priority than showing the search_query for new. This should probably be restructured so that the new search form is visible on the index page.

Good catch! I can live with that edge case for now.


Correction that org managers can rename orgs

Implements Tracker story: #157755345
Story details say to wait to merge until CAPI PM @zrob can verifies the change.

I've made my membership to cloudfoundry public. Can someone please close & reopen this PR so @cfdreddbot is happy?

hey @zrob & @infews - i'd like to port this change to pcf docs too-- what version(s) would this apply to?

@seviet it would apply to all versions. this has always been inaccurate


Refactor the SQLOperator, creating a BaseOperator for the SDK operations

Describe the context
This task is part of the Astro Python SDK refactor.
Acceptance criteria

[ ] Identify current responsibilities of the SQLDecorator (e.g. execute SQL, templating, resolve queries, create tables, etc) and propose a minimum interface for the new version
[ ] Implement the minimal interface necessary for the Base SQLDecorator class (e.g. templating?)
[ ] All the quality checks passed
[ ] All the functions have typehints
[ ] All the new functions and modules have docstrings
[ ] The introduced code has 100% of test coverage

@utkarsharma2 started made a draft PR related to this work:
https://github.com/astro-projects/astro/pull/322


Enable integration testing as if cfnlint is a module

Issue #, if available:
Description of changes:

Add testing for cfn-lint used inside other packages

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Pre-work as I work to rewrite some of the core functionality.  This should help with testing and validating that I'm maintaining backwards compatibility.


What do we do with the streamed video from Python ?

Trying to be successful to get the streaming part correct for now.
I am using 'Motion' to stream from pi camera onto an ip address... now I changed the IP address(<IP_ADDRESS>:8081) in the 2 codes.. (stream_server_test.py which runs on laptop and stream_client.py which runs on RPi) I am getting errors on both the programs..
I want to know what software you used in Pi to stream on the IP address? I used Motion and it clearly doesn't work :( Please help
EDIT: I've spent atleast 5 days just trying to get stream working via python/OpenCV from my laptop..
tried using Gstreamer/ffmpeg/Motion.

what errors are u getting ? post them here @M-Salman1337

I first make sure I am streaming on the ip then I run the computer and then the pi application (but computer and pi programs are giving errors)
Error on computer:
Traceback (most recent call last):
File "C:/Users/tiger/Desktop/take_the_stream_from_pi.py", line 43, in
VideoStreamingTest()
File "C:/Users/tiger/Desktop/take_the_stream_from_pi.py", line 11, in init
self.server_socket.bind(('<IP_ADDRESS>', 8081))
OSError: [WinError 10049] The requested address is not valid in its context
Error on Pi:
mmal: mmal_vc_port_enable: failed to enable port vc.null_sink:in:0(OPQV): ENOSPC
mmal: mmal_port_enable: failed to enable connected port (vc.null_sink:in:0(OPQV))0x11c2330 (ENOSPC)
mmal: mmal_connection_enable: output port couldn't be enabled
Traceback (most recent call last):
File "stream_from_pi.py", line 21, in
with picamera.PiCamera() as camera:
File "/usr/lib/python3/dist-packages/picamera/camera.py", line 433, in init
self._init_preview()
File "/usr/lib/python3/dist-packages/picamera/camera.py", line 513, in _init_preview
self, self._camera.outputs[self.CAMERA_PREVIEW_PORT])
File "/usr/lib/python3/dist-packages/picamera/renderers.py", line 558, in init
self.renderer.inputs[0].connect(source).enable()
File "/usr/lib/python3/dist-packages/picamera/mmalobj.py", line 2212, in enable
prefix="Failed to enable connection")
File "/usr/lib/python3/dist-packages/picamera/exc.py", line 184, in mmal_check
raise PiCameraMMALError(status, prefix)
picamera.exc.PiCameraMMALError: Failed to enable connection: Out of resources
What did u use to stream and use his code ?

@hamuchiwa  Please reply how did u manage to use that code? like what streamer did u use in the Raspberry Pi

I used this code after some corrections...
import numpy as np
import cv2
import socket

class VideoStreamingTest(object):
    def __init__(self):

        #self.server_socket = socket.socket()
        #self.server_socket.bind(('<IP_ADDRESS>', 8081))
        #self.server_socket.listen(0)

        #self.connection, self.client_address = self.server_socket.accept()
        #self.connection = self.connection.makefile('rb')
        #self.streaming()
        self.socket = socket.socket()
        self.connection = self.socket.connect(('<IP_ADDRESS>', 8081))
        #self.socket.connect(('<IP_ADDRESS>', 8081))
        self.streaming()

    def streaming(self):

        try:
            #print ("Connection from: ", self.client_address)
            print ("Streaming...")
            print ("Press 'q' to exit")

            stream_bytes = b' '
            while True:

                stream_bytes += self.socket.recv(1024)
                first = stream_bytes.find(b'\xff\xd8')
                last = stream_bytes.find(b'\xff\xd9')
                if first != -1 and last != -1:
                    jpg = stream_bytes[first:last + 2]
                    stream_bytes = stream_bytes[last + 2:]
                    #image = cv2.imdecode(np.fromstring(jpg, dtype=np.uint8), cv2.IMREAD_GRAYSCALE)
                    image = cv2.imdecode(np.fromstring(jpg, dtype=np.uint8), cv2.IMREAD_COLOR)
                    cv2.imshow('image', image)

                    if cv2.waitKey(1) & 0xFF == ord('q'):
                        break
        finally:
            #self.connection.close()
            self.socket.close()

if __name__ == '__main__':
    VideoStreamingTest()

Will ask further doubts as they arise!

# Answer these 3 questions please! :-)
1.But why do you have 2 programs running 1 from pi and 1 on computer?
2.and also why self.connection = self.connection.makefile('rb') is this required?
3. (are you maybe saving stream on the computer? I'm not sure..)
I am currently always set stream option when pi turns on.. at <IP_ADDRESS>:8081
and I run the above code to get it with python/OpenCV. (So turn pi on and run 1 program on computer)
Please give your insights!

@M-Salman1337

One program is for server which in our case is our pc and other is client which is pi. We are getting streams of images from client and get them through server
2.makefile means to use the program make which updates targets from newer dependencies, rb is for reading it in binary mode where r is for reading and b is for binary mode because we are expecting data in bytes.
Yes we can save stream, we are already saving them as images. See code of training in computer folder.

In your case 192.168.06 is your local ip. if you are on windows type ipconfig in cmd to confirm your local ip. 8081 is port which u have to open from firewall if you are using windows. i am not sure about others OSes.


[WIP] midburn session

Encode/Decode midburn session

read data from drupal including roles
encrypt/decrypt jwt token from http only cookie

todo: need to replace hardcoded keys with proper RSA keys.
waiting to be integrated

@noam-almog It is a bit hard to review because many files have changed, and a lot of code got commented.
In general:

I like the code organization and abstraction.
Did you removed the drupal_login logic? I didn't find it anymore and we should keep it.

@roy-zahor  re-added the drupal flow after getting some feedback from @LeonFedotov


[GSSoC'23] Changing the Get In touch Card

What happened?
I found that the Get In Touch Card is not looking that much beautiful.
How can we reproduce this bug?
I want to work on this issue and make the card more beautiful for the visitors to see.
Desktop Information (Optional)
No response
Urgency (Optional)
Medium priority

I would like to work on this project. Please assign it to me.

Sure @parag477

Assign me I am ready to change the Get in touch card issue under the GSSOC23


add DS418play to compatability list

Can confirm traffic is flowing after following the latest fixes in #10
$ uname -a
Linux Diskstation 4.4.59+ #24922 SMP PREEMPT Fri May 10 02:59:42 CST 2019 x86_64 GNU/Linux synology_apollolake_418play

$ sudo wg
interface: wg0
  public key: <redacted>
  private key: (hidden)
  listening port: 993

peer: <redacted>
  endpoint: <redacted>
  allowed ips: <redacted>
  latest handshake: 53 seconds ago
  transfer: 23.11 MiB received, 383.24 MiB sent

peer: <redacted>
  endpoint: <redacted>
  allowed ips: <redacted>
  latest handshake: 56 seconds ago
  transfer: 113.68 MiB received, 243.09 MiB sent

Thank you!


How to use cleave.js with react-final-form?

I followed this React Final Form example to use third-party components: Third Party Components Example and I also checked this issue: How to use with material-ui TextField and React-Final-Form? .
I'm not using Material-UI components. So I did the following based on previous example:

Create adapter:

import React from 'react'
import Cleave from 'cleave.js'

export default ({ input, ...rest }) => <Cleave {...input} {...rest} />

Use it:

import CleaveAdapter from './CleaveAdapter.jsx'

<Field name="sample" component={CleaveAdapter} options={{numeral: true, delimiter: '-'}}/>

But it shows the following console error:
TypeError: owner is undefined
And it points to this line: https://github.com/nosir/cleave.js/blob/996f977ff8f8397b026c3a0f99ecc225546300da/dist/cleave.js#L77
Can you please help me?

@mrjdavidfg try importing Cleave like this:
import Cleave from 'cleave.js/react'

Hi @mrjdavidfg, I don't know if you managed to solve your problem, but here is a simple example of using cleave.js with react-final-form. Hopefully someone finds it useful 😀


80956_E2E_TCs

ADO-81768
Description
List of proposed changes:

Test cases added to test Profile page heading in EN and FR
Test case to validate French button click goes to /fr/profile
Test case added to validate the breadcrumb are present on profile page

What to test for/How to test
Validate Profile page heading and breadcrumbs using cypress tests
Additional Notes

Closing this in favour of a new PR.


build: fix circleci branch filter not working

Due to some incorrect branch filter configuration, currently we publish build snapshots for all upstream branches (such as branches for quick reverts).

@jelbourn We should be good running the other jobs on upstream branches. I don't want to repeat this filters for each job... Unfortunately that's the only allowed way to filter for branches within CircleCI

I would prefer to filter out all non-release branches even if it's a lot of boilerplate; the _presubmit branch in particular will otherwise cause a ton of jobs to be run.

Caretaker note: When merging, please unmark the publish_snapshots job as required job in the repo settings.


How do I setting for custom analyzers?

Hi. I'm newbie.
I found anything in readme, so I cant find anyway to custom analyzers for my index.
Can u help me?
Example, I want to have a custom analyzers bellow:
"analyzer": {               "did_you_mean": {                   "filter": ["lowercase"],                   "char_filter": ["html_strip"],                   "type": "custom",                   "tokenizer": "vi_tokenizer"               },               "autocomplete": {                   "filter": ["lowercase", "autocomplete_filter"],                   "char_filter": ["html_strip"],                   "type": "custom",                   "tokenizer": "standard"               },               "default": {                   "filter": ["lowercase", "stopwords_filter", "stemmer_filter"],                   "char_filter": ["html_strip"],                   "type": "custom",                   "tokenizer": "standard"               }           }
Thanks for help.

Did you mean?
curl -XPUT "http://localhost:9200/plastic" -d'
{
  "settings": {
    "analysis": {
      "analyzer": {
        "did_you_mean": {
          "type":      "custom",
          "tokenizer": "standard",
          "filter": [
            "standard",
            "lowercase",
            "asciifolding",
            "edge_ngram"
          ]
        }
      }
    }
  }
}'

So I use a library custom analyzer "vi_tokenizer" in here: https://github.com/duydo/elasticsearch-analysis-vietnamese.
But It show a message : "Failed to find vi_tokenizer".
I have installed this plugin by a command "bin/elasticsearch-plugin install".
Can u help me the solution?
Thanks

Just follow the Installation section and use analyzer in application.
$map->string('title', [
    'analyzed' => true,
    'analyzer' => 'vi_analyzer'
]);


vertical-front-matter is broken

https://github.com/sindresorhus/refined-github/blob/main/.github/ISSUE_TEMPLATE/bug_report.md
No errors in the console. Tested in Safari

There's no more [data-table-type="yaml-metadata"]. I wonder how should we handle this. Perhaps always format first .markdown-body > table?


Performance

Compared to raw text manipulations that were previously used in geojson-modelica, this is ~100x slower.
After some initial benchmarking it seemed like ~1/2 of the time was spent just parsing the files. It might be worth a bit of effort profiling/documenting performance.
Also might be worth checking out 'speedy-antlr-tool': https://github.com/amykyta3/speedy-antlr-tool

Just ran some tests locally to get an idea of possible performance improvements.
Edited the example repo to use our modelica grammar, and ran the benchmark on the test_1.mo file from geojson-modelica-translator (here).
Results:
py_elapsed:  330.613 ms
cpp_elapsed: 55.355 ms
Speedup: 5.97

The cpp parser was about 6x faster than using the python parser (what we use currently).
Note that I didn't do any tests to verify the AST accuracy as that'd be a bit more involved work to get it fully integrated with our package.

Ran some more comparisons after integrating speedy-antlr with the package (see #16 )
I used pytest-benchmark fixture, and had it parse a small file (the DCMotor.mo from test data) and a larger file (Office.mo) using the C++ and python parsers.
Results

name (time in ms)
min
max
mean
stddev
median

python-small-file
1.2806
96.4835
2.4554
7.9437
1.3993

cpp-small-file
1.3121
111.0004
2.1051
6.5126
1.4461

python-large-file
64.2107
198.7874
136.4350
47.8739
150.0920

cpp-large-file
65.2889
151.0811
111.1692
27.7759
114.6904

Using C++ parser was about 1.2x faster than the python parser.

After further investigation, sparked by the strange reduction in performance from 5x to 1.2x, I finally managed to get the results I would expect. I think there were some issues with my virtual environment's installed package.
See the results here: https://gist.github.com/macintoshpie/3ee17ba3257be021252f6060f8b6e6bb
TLDR; C++ parser is 3-4x faster

We've decided that using the speedy-antlr-tool is not necessary at this time. See the archive/speedy-antlr-tool tag for future reference on how to do this integration.


[AVM Module Issue]:

Check for previous/existing GitHub issues

[X] I have checked for previous/existing GitHub issues

Issue Type?
Feature Request
(Optional) Module Version
No response
(Optional) Correlation Id
No response
Description
Hello,
I want to ask is there any possibility to access the VM instances to configure a backup for them or an input to define a specific recovery vault?
In the AVM VM module you have the configuration for an azure backup. My plan is it to provide the scale set also a specific recovery vault. In my project, is it necessary to have only one recovery service vault for all the machines.
I tried to loop over all the VM instances and apply a backup setting, but in the ressource output of the scale set there is no reference to them.
Thank you very much.

Thanks for the suggestion @adamevoila.  The underlying provider does not currently support adding backups.  I'm wondering if a better approach would be to create the VM with backup and add it to the VMSS?  I will check with our engineering teams.


Example for controlling tabs with code

Maybe a sample would be useful, as it was not initially obvious to me how to do this.
Rest of the changes are just black formatting, sorry.

 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.


Replace "builder.alloca" by "allocate_varlen_buffer"

As title. Patch extracted from #538

Once this one gets merged I can create another release of RBC


Generalize the setup-macos.sh to get BREW_PATH dynamically

The current code has a hard coded arch check for where the brew is located, which may be changed at anytime in the future. This change use the which to locate the brew binary location.

cc @miaoever

Thank you for this contribution @syscl . Theres already a similar PR : https://github.com/facebookincubator/velox/pull/5152 .


Contact

Hi,
IS there a way to contact you for support? I can't find a contact us page on your website.
If I wanted to remove a series I created. Want to try something else, how can I delete it. It gives me an error when trying to do so.

That's by design to prevent abuse.

I created one video and in few minutes another one was created by the system itself without any touch from me.
I was going to try another style to find what suits me before subscribing.

Also, nest post is said to be on 21/03/2025 not next week as advertised.


added formFieldsDirection prop for CreditCardInput, added style for formFieldsDirection and updated readme.md

Derived form orientation from props for <CreditCardInput/> to advance the feature of orientation, PFA.

this looks good please check #193 and send PR, thank you


Lesson Checkpoint Feature: Display Last Read Point on Homepage

I've implemented a new feature in a file called pickup.js that saves the last point in the lessons that a user has read. This feature is designed to enhance the user experience by allowing them to easily pick up where they left off in their learning journey.
It saves the pathname and the scrolled value of the last visited lesson and continues at that point
Changes Made

Added a function in pickup.js to store the last read lesson point in local storage.
Modified the homepage to display a message that guides users to continue their lessons from the last read point.

Motivation
I found it difficult to remember where I left off in a series of lessons. So I thought I should contribute

Here is a gif demonstrating how it works

anyone?

I really need this feature 😭

two weekends gone :( still waiting

There is a problem in phone that was unnoticed by me


Disable strict SSL in ./node_modules/vscode/bin/install script?

I'm on a corporate network (not a proxy) that imposes its own self-signed SSL certificate on outbound https traffic.
I already have this in my settings.json, which is insecure but at least allows me to install vscode extensions:
    "http.proxyStrictSSL": false

Likewise, I can turn off strict ssl to install npm modules, with npm set strict-ssl false.  This is git bash on Windows:
$ npm config ls -l | grep strict-ssl
strict-ssl = false

$ cat package.json
{
  "name": "test-vscode-install",
  "version": "1.0.0",
  "description": "",
  "main": "index.js",
  "engines": {
    "vscode": "^1.0.0"
  },
  "scripts": {
        "vscode:prepublish": "node ./node_modules/vscode/bin/compile",
        "compile": "node ./node_modules/vscode/bin/compile -watch -p ./",
        "postinstall": "node ./node_modules/vscode/bin/install"
  },
  "author": "",
  "license": "MIT",
  "devDependencies": {
    "typescript": "^2.2.1",
    "vscode": "^1.0.5"
  }
}

$ npm install

><EMAIL_ADDRESS>postinstall C:\path\test-vscode-install
> node ./node_modules/vscode/bin/install

Detected VS Code engine version: ^1.0.0
Error installing vscode.d.ts: Error: self signed certificate in certificate chain

So, to summarize:

:+1:  vscode settings.json is configured for insecure SSL, and can install extensions from the marketplace.
:+1: npm is configured for insecure SSL, and can install ordinary npm packages without trouble.
:x: The vscode "postinstall" node script is still choking on the self-signed cert.

Is there a way to fix this?  Thanks!

@bpasero The code to fetch vscode.d.ts should maybe pay attention to npm's value of strict-ssl?

This issue was moved to Microsoft/vscode-extension-vscode#57


With output.environment.module, packaged dependency gets incorrect reference to external dependency

Bug report

What is the current behavior?
I'm building a react component library, so I declare react and react-dom as externals. I'm using webpack to build our library as a module, with experiments.outputModule = true, output.environment.module = true, and output.library.type = 'module'.
I'm using a library that also declares react as an external dependency.
When I do my build:

In the compiled code, we end up getting a reference to react as external_react_
my code's calls to React.createContext get compiled correctly as external_react_.createContext()
my dependency's calls to React.createContext get compiled incorrectly as external_react_["default"].createContext()

This results in a hard crash in consumers of my library, because external_react_["default"] is undefined and then external_react_["default"].createContext() throws.
If the current behavior is a bug, please provide the steps to reproduce.
reproduction case: https://github.com/MustafaHaddara/webpack-external-repro-case

What is the expected behavior?
My dependency's calls to React.createContext should get compiled as external_react_.createContext()

Other relevant information:
webpack version:  5.65.0
Node.js version: 14.16.0, also reproduced with 16.7.0
Operating System:
Additional tools:

But I don't see any problems, put console.log(external_react_["default"]) and look at output, all works fine...

How did you import it? I realized I got the main and module keys wrong in my package.json in my repro case so you might have been importing the root index.js, I've updated my reproduction case now.
I say this because when I import my component into a create-react-app project, I get this crash:
warning.js:62 Uncaught TypeError: __webpack_require__.r is not a function
    at Object../node_modules/webpack-external-repro-case/es/index.js (warning.js:62)

When I switch optimization.minimize to true, I get the initial error I reported:
Uncaught TypeError: Cannot read properties of undefined (reading 'createContext')
    at index.js:2
    at Module../node_modules/webpack-external-repro-case/es/index.js (index.js:2)
    at __webpack_require__ (bootstrap:856)
    at fn (bootstrap:150)
    at Module.<anonymous> (index.module.css?438e:82)

(this suggests to me that I should be reporting a second bug related to disabling minification?)

Your first example seems to work for me, though. I can't tell from the docs, what are the drawbacks of mixing CJS with ESM?
Does it boil down to these chunks being different in the output?
module:
import * as __WEBPACK_EXTERNAL_MODULE_react__ from "react";
// truncated

/***/ 810:
/***/ ((module) => {

var x = y => { var x = {}; __webpack_require__.d(x, y); return x; }
var y = x => () => x
module.exports = __WEBPACK_EXTERNAL_MODULE_react__;

/***/ })

vs. commonjs:
/***/ 556:
/***/ ((module) => {

module.exports = require("react");

/***/ })

How did you import it? I realized I got the main and module keys wrong in my package.json in my repro case so you might have been importing the root index.js, I've updated my reproduction case now.

I tested it using https://github.com/MustafaHaddara/webpack-external-repro-case/blob/main/es/index.js, just run this in browser or using Node.js (document will be undefined)
What is webpack version in CRA? Please run npm ls webpack

CRA is using<EMAIL_ADDRESS>via<EMAIL_ADDRESS>

There is the problem, webpack v4 doesn't fully support ESM, you need to update CRA and use webpack v5

If I can't currently update CRA, is there a config option to teach webpack v4 to not use ESM and import CJS instead? My actual package ships both ESM and CJS builds. Or is that a bad idea?
Also, do you want another bug report about the minification or is that another known Webpack v4 bug?

If I can't currently update CRA, is there a config option to teach webpack v4 to not use ESM and import CJS instead? My actual package ships both ESM and CJS builds. Or is that a bad idea?

CRA doesn't provide ability to change options, so I think there is only one solution - only commonjs, not it is not bad idea, but in future make sense to use only ESM

Also, do you want another bug report about the minification or is that another known Webpack v4 bug?

Probably yes, webpack v4 is real old and contains old version of terser

ok, that makes sense.
thanks for your help!


prepare_data.py is appending multiple (nine) copies of source and target lines to train and val files

The issue is lines 38 and 39  in prepare_data.py are being called once for every naughty_string type, so source.append and target.append are being called nine times for every valid source/target line (once for each type of naughty_string).
lines 38 and 39 should not be in the loop on line 35, and should be move left one level.
38         source.append(source_line)
39         target.append(target_line)
Best Regards

Great catch thanks!
Potentially the partial cause of this bug by overloading SentencePiece but I won't be able to fully test for another week.


💄 Add a model image to the model selector

Pull Request Checklist
Note to first-time contributors: Please open a discussion post in Discussions and describe your changes before submitting a pull request.
Before submitting, make sure you've checked the following:

[x] Target branch: Please verify that the pull request targets the dev branch.
[x] Description: Provide a concise description of the changes made in this pull request.
[x] Changelog: Ensure a changelog entry following the format of Keep a Changelog is added at the bottom of the PR description.
[x] Testing: Have you written and run sufficient tests for validating the changes?
[x ] Code review: Have you performed a self-review of your code, addressing any coding standard issues and ensuring

Changelog Entry
Description
💄 Add a model image to the model selector
Added
💄 Add a model image to the model selector
Changed
💄 Add a model image to the model selector
Screenshots or Videos

This would be a welcome addition in my honest opinion. Seeing the model's image that I've set for each model, but inside of the model dropdown selector would indeed help me to find models that I'd like to use, quicker.

Thanks!


Bump gradle version to 7.4

For three reasons:

for consistency with https://github.com/runelite/example-plugin/commit/dc2102c
to allow the plugin to build with JDK 17
to make it so I don't need to keep making this change locally whenever I make/pull a new branch

Reasons 2 and 3 go together. I hadn't noticed Reason 1 until now, which is why I didn't make this PR earlier. 😅

Are you sure that's the right gradle-wrapper jar? The wrapper jar from the 7.4 distribution zip has a different sha256

Checks out with the hash for v7.4.2 displayed here on my end

Indeed it does :+1: I guess the one in the distribution zip is just slightly different for whatever reason

Yeah, who knows why :shrug:. I'm happy to see that you're checking this sort of thing :slightly_smiling_face:

Indeed it does 👍 I guess the one in the distribution zip is just slightly different for whatever reason

Sometimes Intellij Caches it and u have to revalidate your caches and reload, I had the issue a few times on other Projects


notification when a slayer task has ended

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always
frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've
considered.
Additional context
Add any other context or screenshots about the feature request here.

You can already get a notification based on the slayer task completion message: You've completed 123 tasks and received 15 points, giving you a total of 1,234; return to a Slayer master. Using the string "return to a Slayer master." in the Chat Notifications plugin with the Notify on highlight option enabled would achieve this.

To add, there is also a plugin on the plugin hub called Task Jingle if looking for an audible cue on task completion.


Created a Model for KiotaSearchResult and implemented SearchUI.

I've implemented the Search UI feature, allowing users to click on the search icon beside the description field. This action will trigger a popup window where users can search for specific terms. Once they enter the search term and initiate the search, a new popup window will display the search results. Users will then have the option to select a relevant OpenAPI description from the results presented in the popup.

@MIchaelMainer


New format for codeformatter

Hi,
I am using Laravel (.blade.php) files and I'd like to format my HTML with this extension.
Now, I need to set my format to HTML and then format my code. I think it should be an easy thing to do but couldn't find it out.
Best regards

+1

I added this line:
'laravel blade highlighter' : HtmlFormatter,
to self.classmap array in formatter.py and it is working fine now.
But it would be fine to add it in next release for everyone.

will be fixed in #140

Are you sure? It does not seem to work. Yet.

@phaberest you can add blade in html syntaxes like this https://github.com/akalongman/sublimetext-codeformatter/commit/cb45b30ae307718e0868204d4a5b72194ba82570
in next release version this will be in default settings


feat: svg captcha

SVG captcha as per https://github.com/mayeaux/nodetube/issues/398.

Added GET /captcha and POST /captcha endpoints for getting and validating the captcha respectively. A bit confused as where to put controllers as the rest in the file are locked behind /api route, which requires auth.
Reworked the signup page with some styling and additional client-side validation. Though I am not sure how google captcha will look in the new layout.
Added more validation to POST /signup, but more rigid validation requires complete controller rewrite.

Looks really good! Want to go over this and get it merged? Sorry for the delayed response on this.

That's quite a delay. But writing client code without feeding it into a bundler first is a lost cause. So this PR can be shitcanned.
Maybe when I'll have time I'll add a webpack into the repo and then rewrite this PR (as a new PR).

That's quite a delay. But writing client code without feeding it into a bundler first is a lost cause. So this PR can be shitcanned.
Maybe when I'll have time I'll add a webpack into the repo and then rewrite this PR (as a new PR).

Can you explain that point about a bundler?  The code seems fine to me personally

Does it even work?
Bundling allows to write modular client code of arbitrary structure and output it as a handful of files. This in turn allows to bust the browser cache when needed and use npm packages in the client code without a hassle.
In case of this script when it goes live it will either needlessly redownload itself again or won't update on the client side in case of further changes, depending on how caching is handled on the prod server.

Fair enough. Honestly I'm not the biggest fan of build tools in the world. Part of my approach with this code is just avoiding complexity as much as possible. You might be horrified by this but I would simply drop it in the pug as a script, or import it as a Pug file. That's what I've been doing to avoid running into caching issues. And I haven't tested this code but just examining it it's definitely a step in the right direction with how it adds JSDocs, fixes Nodemon etc. I want NodeTube to be as accessible as possible to newbies so any kind of build tool or things that could add hurdles or complexity for people I want to avoid.


Add GTK key mapping

The Linux shell is currently mapping key events to GLFW key codes. The GTK key mapping should be implemented in Flutter so this can be removed.

Duplicate of https://github.com/flutter/flutter/issues/57603


Bluetooth LE / Mi Flora

Home Assistant release (hass --version):
0.51.2 - hass.io
Python release (python3 --version):
3.6
Component/platform:
Bluetooth / BT LE tracker / Mi Plant/Flora
Description of problem:
/bin/sh: gatttool: not found
Result: Bluetooth LE and Mi Flora are not working at all
gattlib python package is not installing because of its dependencies to glib
Expected:
gatttool to exists in system
Problem & Investigation
The problem is bluez package on Alpine distribution do not have "utils" like hcitool or gatttool
I was trying to fix it by building from source..
I ended up with this (or something similiar):
wget -q -O /etc/apk/keys/sgerrand.rsa.pub https://raw.githubusercontent.com/sgerrand/alpine-pkg-glibc/master/sgerrand.rsa.pub
wget https://github.com/sgerrand/alpine-pkg-glibc/releases/download/2.25-r0/glibc-2.25-r0.apk
apk add glibc-2.25-r0.apk
apk add boost-dev bluez-dev g++ glib-dev
apk add python3-dev

pip3 download gattlib
tar xvzf ./gattlib-0.20150805.tar.gz
cd gattlib-0.20150805/
sed -ie 's/boost_python-py34/boost_python-py36/' setup.py
pip3 install .

But this is also not working due to requirements of header for boost-python and boost-thread. But those packages are installed by boost-dev - in different versions than required (that is my assumption). Alpine packages website reports that those packages should be included in boost-dev.

TL:DR: Problem installinggattlib for python, because no glib on Alpine (because of missing deps), and no gatttool because of missing utils in bluez package on Alpine distro..

I have no idea what to do next..

Could this help:
https://github.com/sgerrand/alpine-pkg-glibc

Add this into that: https://github.com/home-assistant/hassio-build/blob/master/homeassistant/base/Dockerfile
And active the library in: https://github.com/home-assistant/hassio-build/blob/master/homeassistant/generic/Dockerfile

Sorry @pvizeli !
I did not explain myself clearly:
The snippet I have here does not work at the end.. Still needs some deps to work.
I'm out of ideas what to do next with it.

It look like sed -ie 's/boost_python-py34/boost_python-py36/' setup.py it not Support python 3.6. I think that need fix on gattlib

apk update

wget -q -O /etc/apk/keys/sgerrand.rsa.pub https://raw.githubusercontent.com/sgerrand/alpine-pkg-glibc/master/sgerrand.rsa.pub
wget https://github.com/sgerrand/alpine-pkg-glibc/releases/download/2.25-r0/glibc-2.25-r0.apk
apk add glibc-2.25-r0.apk
apk add bluez-dev boost-dev glib-dev g++
apk add python3-dev

pip3 download gattlib
tar xvzf ./gattlib-0.20150805.tar.gz
cd gattlib-0.20150805/
sed -ie 's/boost_python-py34/boost_python3/' setup.py
sed -ie 's/boost_thread/boost_thread-mt/' setup.py
pip3 install .

This fixes problem with installation of gattlib.
The only thing left is missing gatttool

Hey @adrian-ludwig,
does this still exist in the current hassio version?
If you're using Alpine: gatttool seems to be available as part of the bluez-deprecated package:
https://pkgs.alpinelinux.org/contents?file=gatttool&path=&name=&branch=&repo=&arch=

I migrated from the AIO to Hass.io today and experience the same problem with the MiFlora BLE sensor. So I can confirm that the problem persits.

looks like a duplicate of https://github.com/home-assistant/home-assistant/issues/3458 which will be fixed in the next release of hassio

@rockinglama I got the same error in versions 0.62.0 and 0.62.1

Same error here. Since 0.62.1 I no longer get any updates from the MiFlora sensors.


chore(*): update ng2-tree to latest angular

Hi @rychkog, thanks for doing the upgrade : ) Are you planning on publishing a new package (0.0.2-8?) with this change?

@stevedenman Thanks! I was silent so long cause was working on tree stabilization and bringing there some tests etc. Will try to publish new versions on a regular basis


Implement [build] system tests

The following is necessary action items and tests that will allow for a better implementation of PRESM's [build] process:

[x]  Decide on how PRESM will be executed initially (not necessarily final, sugar CLI form)
[x]  Potentially refactor loader structure based on this
[ ]  Tests for building with TS transpilation
[ ]  Tests for building with bare specifiers
[ ]  Tests for building with internal module specifiers
[ ]  Tests for building with YAML transformer
[ ]  Tests for building with imported CJS (?)

Tests now exist at #14 and #16


Request to Add code of conduct.

This repo does not contain any code of conduct, i would like to work on this @eddiejaoude @nhcarrigan.

Sure 👍


Please update YARN cluster example on wiki which can not work now.

I've updated the wiki instructions for building the latest python/pip, which required some tweaks.  If that doesn't work for you, please point to the specific parts that fail for you.


Feature recurring tasks

Adding, editing and deleting recurring tasks work very similar to normal tasks, except to specify a recurring task you add  | FREQUENCY  at the end of the task after specifing a DATE AND TIME. The supported frequency right now are daily, weekly and monthly. If the current date is past the task date of a recurring task, it would auto generate the next task based on the frequency. If the user inputs a recurring task but with the task date in the past, it would automatically generate all the recurring tasks that would have occured up until the most recent upcoming recurring task.

Codecov Report
Base: 67.25% // Head: 65.57% // Decreases project coverage by -1.67% :warning:

Coverage data is based on head (57bb118) compared to base (dd5d611).
Patch coverage: 17.50% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #252      +/-   ##
============================================
- Coverage     67.25%   65.57%   -1.68%
- Complexity      663      665       +2
============================================
  Files           127      128       +1
  Lines          2336     2414      +78
  Branches        251      269      +18
============================================
+ Hits           1571     1583      +12
- Misses          691      754      +63
- Partials         74       77       +3

Impacted Files
Coverage Δ

...n/java/seedu/uninurse/logic/parser/ParserUtil.java
81.15% <0.00%> (-8.45%)
:arrow_down:

src/main/java/seedu/uninurse/model/Model.java
100.00% <ø> (ø)

.../main/java/seedu/uninurse/model/task/DateTime.java
57.14% <0.00%> (-27.07%)
:arrow_down:

.../java/seedu/uninurse/model/task/RecurringTask.java
12.50% <12.50%> (ø)

.../main/java/seedu/uninurse/model/task/TaskList.java
62.74% <23.07%> (-13.58%)
:arrow_down:

...n/java/seedu/uninurse/storage/JsonAdaptedTask.java
57.89% <40.00%> (-23.93%)
:arrow_down:

...c/main/java/seedu/uninurse/logic/LogicManager.java
66.66% <100.00%> (+1.44%)
:arrow_up:

...c/main/java/seedu/uninurse/model/ModelManager.java
85.24% <100.00%> (+0.50%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


feat(BUX-99): validation and customization of mAPI config

Instead of just deleting some miners from go-minercraft, as we discussed in https://github.com/tonicpow/go-minercraft/issues/95, I decided that it will be better to validate mApi config on start of bux, as at any time some miner can also may become unavailable. (But maybe I still should delete all mentions of these 2 miners from default miners in go-minercraft too? Need your advice @mrz1836)
Also adds usage of mocked Minecraft to all tests, as from now validation of miners is required when creating a new client.
 Successful response - valid miner, error - delete
this miner from config.
Also change adds opportunity to override mApi config(custom token, custom endpoint and etc.) by using WithOverridenMAPIConfig().
There will be a subsequent PR for bux-server with the addition of the ability to override the configuration.
Example of overriding:

@vitalibalashka just fyi - the two miners are already removed from the default miner config