Program killed accidentally

Hi, @adepierre ~
I found a weird problem when training.
It is weird that the program is killed unexpectedly when iteration is 62250.
But nobody kills the program.
There are 753 image pairs for training.
Loss on real A data: 0.0568343 on generated A data: 0.0570172
Loss on real B data: 0.0519266 on generated B data: 0.0524207
Cyclic loss on A data: 0.229246
Cyclic loss on B data: 0.164716
I0712 23:00:19.389642 45804 sgd_solver.cpp:112] Iteration 62250, lr = 0.0002
I0712 23:00:19.398865 45804 sgd_solver.cpp:112] Iteration 62250, lr = 0.0002
I0712 23:00:19.401080 45804 sgd_solver.cpp:112] Iteration 62250, lr = 0.0002
I0712 23:00:19.401469 45804 sgd_solver.cpp:112] Iteration 62250, lr = 0.0002
Killed
Could you please answer this question?
Many thanks!!

Hi
I'm not sure of what you mean by "killed". Isn't it just the end of the training?
You should see some "A_to_B_epoch_N.png" and "A_to_B.png" images from the training in the directory. Are they here?
If you want to restart the training from a checkpoint you can set the --snapshot_gen_X_to_Y flags with Caffe's snapshots files corresponding to the checkpoint you want to restart from (something like CycleGAN_Discriminator_X_to_Y.solverstate)

I am pretty sure that it is not the end of my training.
'Caurse i set the start_epoch =0, end_epoch = 200.
There are "A_to_B_epoch_42.png" and "B_to_A_epoch_42.png" images in the directory.
I think that it should run 753 X 200 = 150600 iterations. (there are 753 images in training set A.)
But it terminated when iteration is 62250 = 753 X 82 + 500.
Is there any problem in the code(main.cpp)?

I found that it may be a problem about runing out of swap.
https://stackoverflow.com/questions/726690/what-killed-my-process-and-why?answertab=active#tab-top
Maybe I need to extend the space of swap or enable the application to consume less memory...

You're right about the number of iterations, it's min(num_images_A, num_images_B) * num_epoch / batch_size.
I am not sure I can reproduce this bug now, but I'm pretty confident the program was not killed because of bug in the code. Maybe it's a memory issue or something like that. If you use the solverstate files you should be able to start the training from approximately where it stopped


[WARN]  New subscription discovered on topic '/scan', requesting incompatible QoS

Setup: Ubuntu 22.04, ROS humble
I am writing a ROS driver for a laser scanner and want to view the scan data in RViz. I have used SensorDataQoS while creating the publisher
scan_publisher_ = node_->create_publisher<sensor_msgs::msg::LaserScan>(scan_topic, rclcpp::SensorDataQoS());

and have also set the QoS values in .rviz as well
      Topic:
        Depth: 5
        Durability Policy: Volatile
        Filter size: 10
        History Policy: Keep Last
        Reliability Policy: Best Effort
        Value: /scan

when I run the driver, I get the following warning
[ros_main-1] [WARN] [1700130825.414718492] [ros_main]: New subscription discovered on topic '/scan', requesting incompatible QoS. No messages will be sent to it. Last incompatible policy: RELIABILITY_QOS_POLICY

But I can still view the data in RViz. Also, following is verbose output for the scan topic info:
ros2 topic info /scan --verbose
Type: sensor_msgs/msg/LaserScan

Publisher count: 1

Node name: ros_main
Node namespace: /
Topic type: sensor_msgs/msg/LaserScan
Endpoint type: PUBLISHER
GID: 01.0f.25.3c.d6.25.3e.<IP_ADDRESS>.<IP_ADDRESS>.<IP_ADDRESS>.<IP_ADDRESS>.00
QoS profile:
  Reliability: BEST_EFFORT
  History (Depth): UNKNOWN
  Durability: VOLATILE
  Lifespan: Infinite
  Deadline: Infinite
  Liveliness: AUTOMATIC
  Liveliness lease duration: Infinite

Subscription count: 1

Node name: rviz
Node namespace: /
Topic type: sensor_msgs/msg/LaserScan
Endpoint type: SUBSCRIPTION
GID: 01.0f.25.3c.da.25.19.b<IP_ADDRESS>.<IP_ADDRESS>.<IP_ADDRESS>.<IP_ADDRESS>.00
QoS profile:
  Reliability: BEST_EFFORT
  History (Depth): UNKNOWN
  Durability: VOLATILE
  Lifespan: Infinite
  Deadline: Infinite
  Liveliness: AUTOMATIC
  Liveliness lease duration: Infinite

So the settings do actually match. Why am I getting the warning in that case? And this happens only on humble (galactic and foxy works). I can do a test with the latest rolling as well, just in case, to narrow down that the problem is in the humble version.

Hi @ipa-hsd, could you provide the full code for a minimal reproducible example? Also, which DDS are you using?
Thanks for making the issue.

I am experiencing this as well using pointcloud_to_laserscan, anytime I start Rviz2, it seems to be messing up the topic and pointcloud_to_laserscan stops publishing to the /scan topic. I am using what should be the latest available, ROS 2 Jazzy, Ubuntu Noble, packaged versions.
[pointcloud_to_laserscan_node-6] [WARN] [1732215357.520418540] [pointcloud_to_laserscan]: New subscription discovered on topic '/scan', requesting incompatible QoS. No messages will be sent to it. Last incompatible policy: RELIABILITY_QOS_POLICY
[pointcloud_to_laserscan_node-6] [INFO] [1732215357.523424432] [pointcloud_to_laserscan]: Got a subscriber to laserscan, starting pointcloud subscriber
[pointcloud_to_laserscan_node-6] [INFO] [1732215482.382114669] [pointcloud_to_laserscan]: No subscribers to laserscan, shutting down pointcloud subscriber

I definitely changed it in Rviz2 to be best effort, to match the rest, I have no idea where the reliability policy is coming from. Like the original report, when I look at the topic information under ros2, rviz2 is using best effort. It seems to be happening on initial connect or something else in Rviz2.  I can see about providing additional information but not sure I can provide a reproducible example. Thank you!


Add codecov

@J08nY Can this be incorporated also into local development? I.e. can we create a pre-commit task for this? It's uncomfortable to discover failing pipelines only when they get to remote.

Well you can track code coverage when running tests. I do not think it makes sense to include anything based on that to a pre-commit. Otherwise every commit would run the test suite, I think that should not happen and be separate. If you want to look at the coverage diff you can check codecov manually, or just push and see the pipeline results.


Setting up MCycle issue

I have the following issue:

import mcycle as mc
import CoolProp as CP

mc.DEFAULTS.PLOT_DIR = "" # will not create a new folder for plots
mc.DEFAULTS.PLOT_DPI = 200
mc.DEFAULTS.updateDefaults()
Traceback (most recent call last):
File "", line 1, in
AttributeError: module 'mcycle.DEFAULTS' has no attribute 'updateDefaults'

Cython, numpy, scipy, matplotlib and CoolProp are all up to date

Thanks for pointing this out, the name of that function has changed to checkDefaults(). I've updated that example now so it should run fine for you.

Thank you very much!
I appreciate your attention.


remove blank files

2 kotlin files are blank in the view model package

some files are blank

some files are blank

Hi there, I think #682 covers the same changes in this PR.  If that's the case, can you close this PR? Thank you!

Hi there, I think #682 covers the same changes in this PR.  If that's the case, can you close this PR? Thank you!


Upgrade OpenAI SDK to v4

Upgrades OpenAI SDK from v3 to v4 according to the migration guide at https://github.com/openai/openai-node/discussions/217, with assistance from the Grit automated migration. SDK v4 comes with many new features including streaming support, improved types, and improved latency (release notes at https://github.com/openai/openai-node/discussions/182).

This is awesome!
I do want to be careful to not break people where we can (e.g. add a shim mapping basePath -> baseURL) and support some of the old Axios configs if possible.
Also, definitely need to test on Azure OpenAI deployments - there were some issues with the latest preview versions I believe - but you can leave that one to me once the rest is ready!

This is awesome!
I do want to be careful to not break people where we can (e.g. add a shim mapping basePath -> baseURL) and support some of the old Axios configs if possible.
Also, definitely need to test on Azure OpenAI deployments - there were some issues with the latest preview versions I believe - but you can leave that one to me once the rest is ready!

thanks @jacoblee93! Any other specific shims/args you want to keep support for? (I have just rebased on main, so should now be ready for review)

Will give it a deeper dive later but broadly looks awesome - thanks again! I'll aim to get it merged by midweek.

@jacoblee93 addressed first round of review comments


Moderator/Admin help system

We need an ahelp system added to the existing chat system.

We have ahelps now. Might need some more features but that should be detailed in another issue


Maximum size for OTA Firmware file

Hi,
I want to know limitation about size of file that can be import for OTA process which is updated Firmware in swu file.
My Requirement is used a large file about 5 GB for OTA update.
Does current implementation if DU service support the 5GB size of file for OTA update.
Regards
Gaurav Choubey

Hi @GauravChoube, sorry, the largest file size we currently support is 2 GB. Please refer to this for the limits.

If you don't mind sharing, what kind of device requires such a large firmware?

Thanks for reply @devlie
our client want to update hub kind of system which has file size about 5GB.


Add a tutorial on proper benchmarking

Given we've had so many issues about this topic, this adds a concept guide on the subject. Let me know if I seem to have missed any, but these are the main three I've observed
https://github.com/huggingface/accelerate/issues/526 https://github.com/huggingface/accelerate/issues/59#issuecomment-1167814595

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Connection doesn't work after update

Last week I updated to the latest loxone config. Since the. The connection between loxone and ha isn't working anymore.I can't figure out why it isn't working any more

I also get an error message after the last update. My error in the logfile is:
2023-06-12 07:46:42.362 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry PyLoxone for loxone Traceback (most recent call last):   File "/usr/src/homeassistant/homeassistant/config_entries.py", line 387, in async_setup     result = await component.async_setup_entry(hass, self)              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "/config/custom_components/loxone/__init__.py", line 195, in async_setup_entry     if not await miniserver.async_setup():            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "/config/custom_components/loxone/miniserver.py", line 140, in async_setup     request_code = await self.lox_config.getJson()                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "/config/custom_components/loxone/api.py", line 122, in getJson     api_resp = await client.get("/jdev/cfg/apiKey")                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "/usr/local/lib/python3.11/site-packages/httpx/_client.py", line 1757, in get     return await self.request(            ^^^^^^^^^^^^^^^^^^^   File "/usr/local/lib/python3.11/site-packages/httpx/_client.py", line 1530, in request     return await self.send(request, auth=auth, follow_redirects=follow_redirects)            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "/usr/local/lib/python3.11/site-packages/httpx/_client.py", line 1617, in send     response = await self._send_handling_auth(                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "/usr/local/lib/python3.11/site-packages/httpx/_client.py", line 1645, in _send_handling_auth     response = await self._send_handling_redirects(                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "/usr/local/lib/python3.11/site-packages/httpx/_client.py", line 1703, in _send_handling_redirects     raise exc   File "/usr/local/lib/python3.11/site-packages/httpx/_client.py", line 1685, in _send_handling_redirects     await hook(response)   File "/config/custom_components/loxone/api.py", line 84, in raise_if_not_200     raise LoxoneHTTPStatusError( custom_components.loxone.api.LoxoneHTTPStatusError: Miniserver response code 500
I am unsure if and update for pyloxone is needed or the miniserver has a bug.

Are you both using a gen 2 minieserver? I can not guarantee a stable connection to a gen2 as they changed again the communication. I do not have a gen2 so I can not implement it for the newest changes.

Yes, I am using a gen2. I created a small test python script to see what happens in the getJson function. To simplify my test I used the ip address, port 80 (normaly using 443 as configured in the miniserver) and I get no error. So I went back to home assistant used the ip address and port 80. voila it's working again. I don't know why. Switched back to 443 and it's also working. Before I used the loxone dns name. Maybe this isn't working anymore.


adds macro for making bounds checks a no-op

Summary
From issue #2420 the default in Stan now is to always perform bounds checks for single index access. This has a noticeable slowdown for users. This PR uses a STAN_NDEBUG macro which if defined then makes the range and bounds checks into no-ops. It does this by defining another macro STAN_NO_RANGE_AND_SIZE_CHECK which is equal to return; if STAN_NDEBUG is defined.
At first I thought we could just use NDEBUG but it seems that is specifically to turn on and off asserts such as eigen_assert() or the standard assert(). Our bounds checks are sort of an assert, though I think it's safer just to have a new macro.
It's possible for the user to specify this in the makefile, or we could have an option for the compiler like -fno-debug which at the top of the stan model would define STAN_NDEBUG
Tests
tbh I'm not really sure how to test this since it changes behavior in the preprocessor. We could have a separate jenkins node that does the checks just for these errors? Since they are all no-ops it would be pretty fast
Side Effects
Yes this whole PR is a side effect to turn off bounds checks.
Release notes
Adds STAN_NDEBUG macro which if defined turns off bounds and range checks
Checklist

[x] Math issue #2420

[x] Copyright holder: Steve Bronder
The copyright holder is typically you or your assignee, such as a university or company. By submitting this pull request, the copyright holder is agreeing to the license the submitted work under the following licenses:
- Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
- Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

[x] the basic tests are passing

unit tests pass (to run, use: ./runTests.py test/unit)
header checks pass, (make test-headers)
dependencies checks pass, (make test-math-dependencies)
docs build, (make doxygen)
code passes the built in C++ standards checks (make cpplint)

[ ] the code is written in idiomatic C++ and changes are documented in the doxygen

[x] the new changes are tested

If we will be defining a separate macro, which I think is the right call here, might as well give it a more descriptive name? Most of Stan users are not C++ developers and NDEBUG does not mean much to them. Maybe STAN_NO_RANGE_CHECKS or just NO_RANGE_CHECKS.

As for testing, I think easiest is a Github Action job that runs a list of tests. These standalone short tests are perfect for GHA as its free.

If we will be defining a separate macro, which I think is the right call here, might as well give it a more descriptive name? Most of Stan users are not C++ developers and NDEBUG does not mean much to them. Maybe STAN_NO_RANGE_CHECKS or just NO_RANGE_CHECKS.

I have to think about that. Messing with macro's is certainly an advanced user function. I use STAN_NO_RANGE_CHECKS for the return;, though I could check if it exists and then undef and def it

As for testing, I think easiest is a Github Action job that runs a list of tests. These standalone short tests are perfect for GHA as its free.

Nice! Yeah that makes total sense I'll add the tests and make a gh action

I use STAN_NO_RANGE_CHECKS for the return;, though I could check if it exists and then undef and def it

You should probably rename that to STAN_NO_RANGE_CHECKS_RETURN and define that if a user defines STAN_NO_RANGE_CHECKS.


In prod environment, files should not be changed

By default, on prod environment, some features should be disabled (gitignore updates, ...)

gitignore feature should definitively be disabled by default on any non-dev environment.


Abstract calculator

Here is an attack that centralizes all the looping over tasks while still being fully functional via decorator use. Users/devs only need to write a function that takes a single structure and the desired tasks and returns a dictionary of the results (with the correct task-related names), and the decorated function will be able to handle full task-dictionaries. This is dead-simple for the ASE example, and the Lammps example shows how you can layer on complexity by wrapping a raw lammps library driver with the decorator, and simply wrapping that in another function to give a function with a bit more hand-holding (e.g. no more remembering to lmp.close()).
Because data extraction is now done inside the inner-most loop, this closes #27.
I don't have the various QM codes running on my machine, so here I only updated the Lammps and EMT tests. Right now these changes are (almost) entirely under the hood, so you can see the use-case in the tests is (almost) unchanged, just a different import path. (Almost because I changed the ASE signature to take a calculator instance directly instead of the class+args+kwargs.)
This can be further streamlined in the future. E.g. right now I'm a little bit unhappy that the user/dev needs "secret knowledge" of what the valid "task" names are and how the other workflow component s expect these task names to relate to output field names (e.g. "calc_energy" -> "energy"). These are sensible, but still kind of arbitrary. As we nail more stuff down,  I hope we can get these encoded more rigorously than as strings here and there; once this is done maybe more of the data extraction part can be abstracted away to make the calculator definitions here even tighter.

BTW @jan-janssen, I seem to have nothing beyond read privileges on this repo. That means I can't assign labels, nor can I assign you and @pmrv to review. Indeed, the autocomplete for the @ window doesn't even show Marvin, just some core team (you, Joerg, Tilmann, and the runner). This is quite inconvenient for a pyiron project.

BTW @jan-janssen, I seem to have nothing beyond read privileges on this repo. That means I can't assign labels, nor can I assign you and @pmrv to review.

I fixed the permissions.

@liamhuber Can you remove the previous implementation? Just to make sure the notebooks and everything is updated correctly.

@liamhuber Can you remove the previous implementation? Just to make sure the notebooks and everything is updated correctly.

The notebooks are certainly not yet working properly as the import paths have changed. If you're content with this direction id be happy to propagate the changes further in notebooks and tests. I'll probably have time for that on Wednesday, which also allows a bit of time in case you and Marvin have any more feedback on the base class.

As a general comment, I'd simply drop the distinction between "calc_energy" and "energy" in the dictionaries being passed around.  This is just asking for small bugs, imo.

I've made some modifications in PR over in your repository pertaining to the type defs and such.

As a general comment, I'd simply drop the distinction between "calc_energy" and "energy" in the dictionaries being passed around.  This is just asking for small bugs, imo.

I 100% agree. It's outside the scope of this pull though, as these "magic strings" get used over in the Workflows too.

@jan-janssen @pmrv, I'm content. If there's no further feedback by Monday I'll go ahead and merge this.

Ok, I'm still having authentification issues (only to this repo...) so I can't push and stack here -- since the tests are all passing and I've addressed all the major comments, I'm just going to merge now so the next PR can target the correct head.

Ok, I'm still having authentification issues (only to this repo...) so I can't push and stack here -- since the tests are all passing and I've addressed all the major comments, I'm just going to merge now so the next PR can target the correct head.

Maybe the issue with not being able to push and stack is related to working on your own fork.

@jan-janssen @pmrv, I'm content. If there's no further feedback by Monday I'll go ahead and merge this.

Please give the other developers at least 24 hours to comment on a pull request before merging.

Ok, I'm still having authentification issues (only to this repo...) so I can't push and stack here -- since the tests are all passing and I've addressed all the major comments, I'm just going to merge now so the next PR can target the correct head.

Maybe the issue with not being able to push and stack is related to working on your own fork.

Hopefully! I'll try some independent (probably trivial docs update) PR soon to check.


[VTA] Fix VTA function Vivado Compile Error.

Issue:
when using vivado compile vta.cc with top function 'vta', vivado
report deadlock error like '...with default size is used in a non -dataflow
region, which may result in deadlock Please consider to resize the
stream using the directive ‘set_directive_stream’ or the ‘HL S stream’
pragma.'
Solution:
give the queue a default size as 8. after fix, compile error gone, we can run test bench with this 'vta' function and get correct result.

@tmoreau89 could help to review this patch?

Which version of Vivado were you using @huajsj

And could you provide more details on reproducing the bug?

@tmoreau89 following are the detailed information about the reproduce and vivado_hls information.

vivado_hls version is 2108.2

Reproduce as following
2.1 run vivado_hls 2018.2
2.2 add vta.cc and vta.h  into source
2.3 copy vta_config.json macro define into hw_spec.h to fix macro not existing issue when do
synthesis.
2.4. run sysnthesis
2.5. saw the vivado_hls compile error.
   ERROR: [XFORM 203-733] An internal stream 'tmp_load_queue.V.V' (vm/vta/hardware/xilinx/src/vta.cc:588) with default size is used in a non-dataflow region, which may result in deadlock. Please consider to resize the stream using the directive 'set_directive_stream' or the 'HLS stream' pragma.

ERROR: [HLS 200-70] Pre-synthesis failed.

Thanks for the details; I believe we should migrate to 2019.1 (I'll issue a PR for this). Could you check that this fix also works in 2019.1?
Thanks

@tmoreau89 , about 2019.1, just verified,  the fix work good under 2019.1, steps as following.
#1. using vivado 2019.1 create solution add vta.cc and vta.h, fix header file path/macro issue.
#2. compile , saw the deadlock error and synthesis failed.
#3 apply this patch, issue go away.


How to run seeds in a test?

Is your feature request related to a problem? Please describe.
I'm using wp-cypress to test a plugin. Some tests requires a valid license key, some not.
My idea is to create a seed which I can use for the tests with valid license, where a valid license is stored in the database.
Sadly, I'm not able to get it running. I created a new seed, like the following which is located in the seeds folder.
<?php

use WP_Cypress\Seeder\Seeder;

class ValidLicenseSeeder extends Seeder
{
    public function run()
    {
        update_option('my_plugin_license_key', 'I_AM_A_VALID_LICENSE');
    }
}

But how do I execute this seed in the cypress test? I have not found an example or documentation which shows, how i can run a seed in a test?
Describe the solution you'd like
Please update documentation with an example.
Describe alternatives you've considered
Additional context

Sorry, my mistake. It's in the documentary, I just missed it.
It's super easy. Just call
cy.seed('ValidLicenseSeeder');


Conditions etcd kas

What this PR does / why we need it:
Bubble up KubeAPIServerAvailable and EtcdAvailable from HCP into HC
Goes after https://github.com/openshift/hypershift/pull/1851 and https://github.com/openshift/hypershift/pull/1852
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
ref https://issues.redhat.com/browse/HOSTEDCP-643
Checklist

[ ] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

@csrwng addressed comments on the original PR and updated this one to only contain its relevant commit

/test e2e-aws


Add a basic modal implementation

What It Does
This adds a basic modal implementation, below are images that show what the example looks like. Will update examples with a gif after getting initial feedback.

What is not included:

callbacks for when modal opens/closes
examples with forms (common usage?)
should maybe consider making a more fundamental component that modal can be built off of, for example, the same logic might help with other types of pop-ups?

Related Issues

Thanks for the PR! I like the idea, but as-is it doesn't seem to add much utility. This achieves the same effect:
diff --git a/examples/modal.rs b/examples/modal.rs
index c8c23f5..d5e924c 100644
--- a/examples/modal.rs
+++ b/examples/modal.rs
@@ -56,7 +56,7 @@ fn Popup(mut hooks: Hooks) -> impl Into<AnyElement<'static>> {
                         content: "Lorem Ipsum is simply dummy text of the printing and typesetting industry. Lorem Ipsum has been the industry's standard dummy text ever since the 1500s, when an unknown printer took a galley of type and scrambled it to make a type specimen book. It has survived not only five centuries, but also the leap into electronic typesetting, remaining essentially unchanged. It was popularised in the 1960s with the release of Letraset sheets containing Lorem Ipsum passages, and more recently with desktop publishing software like Aldus PageMaker including versions of Lorem Ipsum."
                     )
                 }
-                Modal(is_open: should_popup.get()) {
+                #(should_popup.get().then(|| element! {
                     Box(
                         align_items: AlignItems::Center,
                         border_style: BorderStyle::Round,
@@ -65,7 +65,7 @@ fn Popup(mut hooks: Hooks) -> impl Into<AnyElement<'static>> {
                     ) {
                         Text(content: "안영하세요!")
                     }
-                }
+                }))
             }
         }
     }

Is there additional functionality that Modal could have that would better justify its existence?

Thanks for the PR! I like the idea, but as-is it doesn't seem to add much utility. This achieves the same effect:
diff --git a/examples/modal.rs b/examples/modal.rs
index c8c23f5..d5e924c 100644
--- a/examples/modal.rs
+++ b/examples/modal.rs
@@ -56,7 +56,7 @@ fn Popup(mut hooks: Hooks) -> impl Into<AnyElement<'static>> {
                         content: "Lorem Ipsum is simply dummy text of the printing and typesetting industry. Lorem Ipsum has been the industry's standard dummy text ever since the 1500s, when an unknown printer took a galley of type and scrambled it to make a type specimen book. It has survived not only five centuries, but also the leap into electronic typesetting, remaining essentially unchanged. It was popularised in the 1960s with the release of Letraset sheets containing Lorem Ipsum passages, and more recently with desktop publishing software like Aldus PageMaker including versions of Lorem Ipsum."
                     )
                 }
-                Modal(is_open: should_popup.get()) {
+                #(should_popup.get().then(|| element! {
                     Box(
                         align_items: AlignItems::Center,
                         border_style: BorderStyle::Round,
@@ -65,7 +65,7 @@ fn Popup(mut hooks: Hooks) -> impl Into<AnyElement<'static>> {
                     ) {
                         Text(content: "안영하세요!")
                     }
-                }
+                }))
             }
         }
     }

Is there additional functionality that Modal could have that would better justify its existence?

At this point in the PR, no, it doesn't have anything special. The value of a modal component, from what I can tell, is to be able to have fns called on open and close of the modal. I am also thinking it could be used for composition with future components that require popup functionality. But as we spoke about whenever ago, if it seems like a useful future component we can add/discuss or can just close :) Am good with either!


Support for s3-compatible backends that aren't on amazon

I can get to it with the awscli, and it's using the same underlying api, so it shouldn't be that difficult.
Here's what I do in awscli:
aws --endpoint-url https://some.url.that.hosted.internally/ s3 ls

+1

+1

+1 we need this

+1

I create a image that use rclone to sync to different backends including custom S3: https://github.com/atende/infra-volume-backup
Rclone: http://rclone.org/
It basically mount the volume in the container and sync its contents.
Would be nice to have such integration in convoy.

Supported added by #228


Support for Spring Boot 2.0

I've just tried it out, and whilst this project works great with Spring 5.0.0 and Spring Boot 1.5.x, it fails when using Spring Boot 2.0.x. The "graphiql-spring-boot" project works fine, but the actual GraphQL handlers don't ever start.

There's a PR open for updating to support Spring Boot 2.0.0, (#36), but I'm waiting for it to have a non-milestone release first.

Shouldn't Boot 2.0 support also include support for Spring WebFlux ?

What parts of webflux would the servlet need to support?

It seems a nice idea if the entire graphql server could be non blocking,
but I'm not sure how feasible that is since the graph structure
necessitates some parts to be computed only after other parts have finished.
On Wed, 13 Dec 2017, 15:39 Andrew Potter<EMAIL_ADDRESS>wrote:

What parts of webflux would the servlet need to support?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/graphql-java/graphql-spring-boot/issues/40#issuecomment-351428541,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAECGJUcgtye0qgDskrGbn-1xxvM6AL0ks5s_--jgaJpZM4PtqGh
.

@apottere I guess the integration would not be as a Servlet, see https://docs.spring.io/spring/docs/current/spring-framework-reference/web-reactive.html#spring-webflux. For example functional endpoints allow programmatic registration of routes.
@sazzer I am not familiar with GraphQL, but even if a full non-blocking solution for the graph could be appealing, maybe you could just support a similar integration than with Servlet as a first step, just leveraging WebFlux infrastructure instead of Servlet one. Does GraphQL processing requires using blocking IO ?

I've no idea if it requires blocking IO per-se. However, what you often
have is almost reentrant processing - child nodes can only be evaluated
after the parent nodes are completely loaded, and I'm not sure if that
would be a problem at all. All the webflux examples I've seen have been:
read from database, convert to output, stream to client, and this would end
up being that structure repeated several times for one request.
On Wed, 13 Dec 2017, 16:16 Sébastien Deleuze<EMAIL_ADDRESS>wrote:

@apottere https://github.com/apottere I guess the integration would not
be as a Servlet, see
https://docs.spring.io/spring/docs/current/spring-framework-reference/web-reactive.html#spring-webflux.
For example functional endpoints allow programmatic registration of routes.
@sazzer https://github.com/sazzer I am not familiar with GraphQL, but
even if a full non-blocking solution for the graph could be appealing,
maybe you could just support a similar integration than with Servlet as a
first step, just leveraging WebFlux infrastructure instead of Servlet one.
Does GraphQL processing requires using blocking IO ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/graphql-java/graphql-spring-boot/issues/40#issuecomment-351440374,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAECGKJj9QNisP2hS4XJ5zQS5SoaBqv3ks5s__g_gaJpZM4PtqGh
.

Spring Boot 2.0.0 GA was released today. Probably good to upgrade this project as well and maybe make a new release?

Anything we can do to help out with the release?

I guess we need @apottere ..

Hello!
For those interested, I wrote an application in kotlin with graphq-java which uses webflux directly (no servlet).
I hope it will be useful if you guys decide to support webflux in the future.

Sorry it took me so long to get to this, it's released in 4.0.0


Add CORD-19 Instructions to Elastirini Doc

@stephaniewhoo @shaneding can you try these instructions out?

@stephaniewhoo @shaneding can you try these instructions out?

not sure if I missed anything on my local machine.
after elastirini/bin/kibana, my terminal keeps telling me
  log   [01:31:30.122] [warning][data][elasticsearch] Unable to revive connection: http://localhost:9200/   log   [01:31:30.123] [warning][data][elasticsearch] No living connections   log   [01:31:30.123] [warning][licensing][plugins] License information could not be obtained from Elasticsearch due to Error: No Living connections error   log   [01:31:30.315] [warning][admin][elasticsearch] Unable to revive connection: http://localhost:9200/   log   [01:31:30.316] [warning][admin][elasticsearch] No living connections   log   [01:31:32.815] [warning][admin][elasticsearch] Unable to revive connection: http://localhost:9200/   log   [01:31:32.816] [warning][admin][elasticsearch] No living connections   log   [01:31:35.317] [warning][admin][elasticsearch] Unable to revive connection: http://localhost:9200/   log   [01:31:35.319] [warning][admin][elasticsearch] No living connections   log   [01:31:37.819] [warning][admin][elasticsearch] Unable to revive connection: http://localhost:9200/   log   [01:31:37.819] [warning][admin][elasticsearch] No living connections

@stephaniewhoo I think you need to have elasticsearch running.

@stephaniewhoo I think you need to have elasticsearch running.

Thanks, and I have tried those instructions. It worked for me. @lintool

Thanks @HangCui0510 for the contribution!

I received the error:
ERROR [pool-2-thread-1] index.IndexCollection$ESIndexerThread (IndexCollection.java:575) - Error sending bulk requests to Elasticsearch
org.elasticsearch.ElasticsearchStatusException: Unable to parse response body
when trying to index the abstract, body and paragraph, although it didn't seem to stop the indexing.
However, on kibana I can't seem to view any of the docs, and the count aggregate was 0 for me, so I was thinking if there was a loading error or something? Although this could be wrong as I am not very familiar with how kibana works.

we need to first load elasticsearch index

For visualization, we need to adjust parameters in "Metrics" and "Buckets" to get desired graphs.
Example:

I am still getting the same error, and the index is available to me, it just seems like that there is no data associated with it. This might just be an issue for my laptop though, since others were able to replicate it.


EPUB 2 meta tags should be superseded by EPUB 3 ones

When an OPF contains both an EPUB 2 (<meta name=>) and EPUB 3 (<meta property=>) meta tags for the same key, the EPUB 3 one should take precedence and the other one ignored. Currently both are added to the otherMetadata dictionary.
Here's an example:
<meta property="media:active-class">-epub-media-overlay-active</meta>
<meta name="media:active-class" content="text-highlight" />

See the decision in the last meeting minutes.

By definition, what ends up in otherMetadata is not well-known and agnostic to the property: you can't know if the property is single or multiple value. As sometimes Epub 2 style properties appear even in EPUB 3, we can't ignore them. I don't know how we can deal with that (it looks like I attended this meeting, my bad).

Yes that's the idea. I updated the OP.


example is wrong?

Hello.
I am a beginner programmer.
I think example may be wrong.
In sendmessage
method, "text.length > 0" would be "msg.length > 0", wouldn't be?
I hope you can check.
Thank you.

Hello 👋
Can you specify file and line where you have doubts?

Hello, mattmezza.
Thank you for your rapid response, and I'm sorry for not responding sooner.
I made a code just like a example. I thought example's 'text' means argument of sendmessage method, wasn't it? If I misunderstood, would you please tell me the correct way?

Thank you.

Please change msg into text (or text into msg). The important thing is that the formal and actual parameter name is the same.
👍

On 3 Jul 2018, at 2:38 PM, tomofu74<EMAIL_ADDRESS>wrote:
Hello, mattmezza.
Thank you for your rapid response, and I'm sorry for not responding sooner.
I made a code just like a example. I thought example's 'text' means argument of sendmessage method, wasn't it? If I misunderstood, would you please tell me the correct way?
Thank you.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Thank you!  Finally, I see!
And  it is really nice component!


Make websockets a bit nicer.

I'd be interested to hear what you think of this. I was trying to get a pure python implementation of websockets working with Flask (no gevent, no uwsgi, no twisted, no autobahn) and I failed. But on the way I did get a version working with werkzeug and wsgiref (some monkeypatching of wsgiref required).  This pull request fixes that. In case anyone is interested my websocket implementation is at https://github.com/mbirtwell/simple-websockets and at the time very incomplete.
To create a websocket it is necessary to set the connection and upgrade
headers and not set the content-length header. At the moment werkzeug
will set a second connection header for close in this situation, this as it
happens doesn't seem to bother the couple of browser I've tried this with
(Chrome and IE) but it's not pretty.

its likely problematic to put websockets into the builtin webserver like this,
its mainly a very minimal server implementation for debugging and local dev
and this change is just a minimal untested ad-hoc change to get "something working somehow"
while i'm pretty sure this works instinct tells me there is a problem hiding
with my current knowledge i strongly suggest to use either external web-socket integration's
or a fully blown integration that properly handles concurrency and process state
i will take a look at your lib later today to make suggestions wrt steps to make your integration viable
and make an attempt to see whats missing to make it work with flask

Closing due to age, and I'm not clear that this change is correct. I'd rather focus on ASGI for async applications.


Coverage with working directory and include file (pxi) in pyx files

Hello,
I want to use coverage with Cython for a large project, but i have a problem when using Coverage and working-directory command line option with pxi files.
In my project, i want to create cython module in a build repository and after i made a copy into install directory so i use working directory
The coverage report fail on pxi files if use working directory in src directory.
I reproduce the pb on a small case with this source_tree :
├── [2.0K]  src
│   ├── [880]  hello.pyx
│   ├── [0]  __init__.py
│   ├── [102]  other.pxi
│   └── [2.0K]  wrap
│       ├── [0]  __init__.py
│       ├── [48]  wrap_other.pxi
│       └── [209]  wrap.pyx

coverage_2.zip
I write two script :

compile.sh : this one works and i have the good coverage on pxi and pyx files
compile_fail.sh : this ones return error on Coverage plugin and the report not contains the other.pxi

Best regards,
Bruno

Thanks for the report. It could just be that the path lookup doesn't work for your setup.
There are several tests in tests/run/coverage*.srctree (which use a simple single file directory format), and the implementation is in Cython/Coverage.py. Maybe you can investigate a bit what goes wrong in your case?

This sounds similar to https://github.com/cython/cython/issues/3636 in a way


seperate configuration file for master and slave are required.

Additionaly configuration need to be reworked to allo running master and slave node in same MCU


PrometheusHighQueryLoad alert: use configured selector

Currently we're hardcoding job="prometheus-k8s" as selector. This
doesn't work if your prometheus is elsewhere. Fortunately we have
prometheusSelector in $._config  which all the other alerts use.
Use that here too.

/cc @beorn7
(I don't appear to be able to request a review directly!)


fix typo which generates segv

because of typo ctx->time_key was not initialized , hence call of flb_sds_len(ctx->time_key) leads to segv
see gdb output here https://pastebin.com/EqwK1Q2i (specifically Line 103)

.


Fix issues around embedded entities

Entities in another entity's properties will not have a key.
Ensure such entities can be decoded properly.
Also ensure such entities can be encoded properly.
[fixes #857]

Great work getting this done so quickly @blowmage!


can freeReplyObject many times?

void freeReplyObject(void *reply) {
redisReply *r = reply;
'''
'''
free(r)
}
==>
void freeReplyObject(void *reply) {
redisReply *r = reply;
'''
'''
free(r)
//add to NULL
r=NULL;
}

That will do exactly nothing.


PHOENIX-4180 - Modify tests to generate unique table names and not us…

The following tests was changed.
./phoenix-core/src/it/java/org/apache/phoenix/end2end/ArrayIT.java
./phoenix-core/src/it/java/org/apache/phoenix/end2end/ClientTimeArithmeticQueryIT.java
./phoenix-core/src/it/java/org/apache/phoenix/end2end/ColumnProjectionOptimizationIT.java
./phoenix-core/src/it/java/org/apache/phoenix/end2end/ConcurrentMutationsIT.java
./phoenix-core/src/it/java/org/apache/phoenix/end2end/CursorWithRowValueConstructorIT.java

Already merged.


cargo-mutants: 24.5.0 -> 24.7.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/cargo-mutants/versions.
meta.description for cargo-mutants is: Mutation testing tool for Rust
meta.homepage for cargo-mutants is: https://github.com/sourcefrog/cargo-mutants
meta.changelog for cargo-mutants is: https://github.com/sourcefrog/cargo-mutants/releases/tag/v24.7.0
Updates performed

Rust version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 24.7.0 with grep in /nix/store/v77p596h72n67d98f59vh9bg8jxv3fxh-cargo-mutants-24.7.0
found 24.7.0 in filename of file in /nix/store/v77p596h72n67d98f59vh9bg8jxv3fxh-cargo-mutants-24.7.0

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

cargo-mutants

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/v77p596h72n67d98f59vh9bg8jxv3fxh-cargo-mutants-24.7.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A cargo-mutants https://github.com/r-ryantm/nixpkgs/archive/a71021619cf0a79b788c93d0081f4f75de0311cb.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/a71021619cf0a79b788c93d0081f4f75de0311cb#cargo-mutants

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/v77p596h72n67d98f59vh9bg8jxv3fxh-cargo-mutants-24.7.0
ls -la /nix/store/v77p596h72n67d98f59vh9bg8jxv3fxh-cargo-mutants-24.7.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

cargo-mutants

Maintainer pings
cc @figsoda for testing.

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

1 package built:

cargo-mutants


chainmon: Create block watcher

Block watcher should allow replay of blocks (disconnects and connects) from synchronization and as new blocks arrive.

Codecov Report

Merging #191 (72f6985) into main (d605c88) will decrease coverage by 0.21%.
The diff coverage is 76.66%.

@@            Coverage Diff             @@
##             main     #191      +/-   ##
==========================================
- Coverage   93.59%   93.37%   -0.22%
==========================================
  Files         179      182       +3
  Lines        4697     4757      +60
  Branches      551      561      +10
==========================================
+ Hits         4396     4442      +46
- Misses        165      174       +9
- Partials      136      141       +5

Impacted Files
Coverage Δ

packages/chainmon/lib/BlockWatcher.ts
69.04% <69.04%> (ø)

packages/chainmon/lib/BlockDiffer.ts
92.85% <92.85%> (ø)

packages/chainmon/lib/BlockDiffResult.ts
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Connect to every containerPort in pod

fixes #535
starts tunnel to every containerPort in pod spec

Essentially, when doing draft connect with the deployment below, you get:
<tunnels to all ports of all containers>

Connect to multi-port:80 on localhost:53010
Connect to multi-port:81 on localhost:53012
Connect to go:8080 on localhost:53014

<logs from all containers, including ones without ports>

The --container/-c flag works as expected, creating tunnels only for all the ports of the selected container.
Best tried with the following deployment template, which has containers with 0, 1 or 2 defined containerPort
apiVersion: extensions/v1beta1
kind: Deployment
metadata:
  name: {{ template "fullname" . }}
  labels:
    draft: {{ default "draft-app" .Values.draft }}
    chart: "{{ .Chart.Name }}-{{ .Chart.Version | replace "+" "_" }}"
spec:
  replicas: {{ .Values.replicaCount }}
  template:
    metadata:
      labels:
        draft: {{ default "draft-app" .Values.draft }}
        app: {{ template "fullname" . }}
    spec:
      containers:

      - name: multi-port
        image: radumatei/multi-port
        ports:
        - containerPort: 80
        - containerPort: 81

      - name: no-port
        image: ubuntu
        command: ["/bin/sh"]
        args: ["-c", "while true; do echo hello; sleep 1;done"]

      - name: {{ .Chart.Name }}
        image: "{{ .Values.image.repository }}:{{ .Values.image.tag }}"
        imagePullPolicy: {{ .Values.image.pullPolicy }}
        ports:
        - containerPort: {{ .Values.service.internalPort }}
        resources:
{{ toYaml .Values.resources | indent 12 }}

For #429 and #536 to make sense, we should probably have draft connect -p <local_port>:<containerPort> -p <another_local_port>: <another_containerPort> - and maybe the logs should still come from all ports?

Thanks @radu-matei !


Compilation on Windows fails when md5sum is unavailable

Running a compile program on Windows e.g.
$ ck compile program:tool-print-cuda-devices

fails with "md5sum is not recognized as an internal or external command..."
I've nearly solved it by downloading "File Checksum Integrity Verifier" (fciv.exe) from Microsoft into ~CK/ck/bin and placing a file md5sum.bat there:
fciv.exe -md5 %1

But the Microsoft tool prints a comment before the actual checksum:
//
// File Checksum Integrity Verifier version 2.05.
//
2a084556e80926ce59f81cd51520ae08 a.exe

If this is an acceptable solution, CK should ignore the lines starting with //.

Moved this ticket to ck-autotuning repo: https://github.com/ctuning/ck-autotuning/issues/8


Implement .env chain source

Did you search for existing issues already?
Yes
Is your feature request related to a problem? Please describe.
If I have multiple secrets, I have to put them each one file. Also if I want secrets used for different environments, more files should be created.
Describe the solution you'd like
There could be a chain which reads dynamic variables from .env file. And also as possibility from environment variables (for cli execution).
Describe alternatives you've considered
I can read values, one by one from file.

I'm not sure I exactly understand the request, but you can already ready environment variables in a template via the {{env.VARIABLE}} key (see docs). This is not very visible in the docs so that could probably be improved. Does that cover your request or are you asking for something else?

Thank you for info. That covers my request partly. Is it possible that environment variables got read from .env file too? Or does it reads it already?

Also there is little issue with password read without chain. It has no ability to set it as sensitive.

It does not currently read .env file, and I'd prefer to keep it that way. My goal is to keep Slumber as focused as possible, and there are already other tools (e.g. source) that you can use to read .env before launching Slumber. That does have the downside of not reloading Slumber when the .env file changes. Here's a possible solution for you:
chains:
  username:
    source: !command
      command: ["bash", "-c", "source .env; echo $USERNAME"]
    trim: both
  password:
    source: !command
      command: ["bash", "-c", "source .env; echo $PASSWORD"]
    sensitive: true
    trim: both

It is repetitive and clunky so I definitely agree there should be a better way to do this, but I'd like it to be more flexible than something specific to .env files. I'm going to think about possible solutions, and in the mean time I'll add a !env chain type for loading a single env variable, so you can mark env variables as sensitive. The {{env.VARIABLE}} syntax predates the sensitive field (and all other chain configuration), so it wasn't a consideration then and I haven't thought much about it since.

!env chain source added in #266, it'll be in the next release

Thank you.


about improve

Perf boost on last version is great !!
( the build in Examples/js/  is not the good one )
I start deep test on PHY, i tell you soon what need
this engine have raycast ?

Hello,

the build in Examples/js/ is not the good one

There is no build in this repository anymore. The build was moved to an npm package.

this engine have raycast ?

Yes, but I didn't expose it in the js interface yet.

keep the build maybe in build folder ( npm package is great but have it directly with github is usefull )
this jolt-physics.wasm-compat.js is a very good idea js and wasm in same module file is cool.

one question on shape creation what is 0.05
new Jolt.BoxShape( this.v.set( 1, 1, 1), 0.05, null );
margin / density or mass ?

yes perf is very good 1000 object at 60 fps


DO NOT MERGE arquillian extra logging

This should hopefully give some more info on the intermittent failures in container stop

retest this please


small view change spacing on enemy

https://github.com/yawhide/Lol-personal-counters/commit/5510d28d2140a872fd06bf49b18d1dd0bc7bd350


Extra margin around videos in Tab and Do-dont component

https://next.carbondesignsystem.com/guidelines/motion/basics
Motion page videos appear affected by a recent change. There's extra margins on left and right:

These videos are wrapped inside <tab> component and <do-dont-example>


do not cache category page

Hi! v2pagecache should cache the category page? at my site module caches only home page and products

Yes, it should cache category pages.
But, if you have installed modules, or a 3rd party theme, there are things that will keep pages from being cached.
For example, it won't cache urls with patterns that match the $skip_urls settings.  It also won't return cached pages if you're logged in, or have items in the cart, etc.

yes, i have installed modules, and a 3rd party theme
but another module super page cache works ok
how to make yours module to the cache category pages?

I assume "Super Page Cache" initially had issues with mega filter as well.   Because it's a paid module, the author had the time and money to go out and buy "mega filter" and figure out what the conflict was.
This is a free module, and we don't have the time and money to troubleshoot compatibility with all of the popular 3rd party modules.
You could try, however, adding   '#module/mega_filter#' to the $skip_urls setting.  That may fix at least one problem.

great! it works! thnx


ci: upgrade ubuntu runners to ubuntu-22.04

#665  👈 (View in Graphite)
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


Include local_settings in celery containers if present

If you want to add a new CELERY_BEAT_SCHEDULE or override deduplication algorythms by way of HASHCODE_FIELDS_PER_SCANNER and friends, the configuration needs to also be propagated to the celery containers and not only uWSGI.

Until now we have not configured these bind mounts in the docker-compose release setup. As these will override any local_settings.py file that someone might have created in dev mode inside dojo/settings/ folder.
So updating the entrypoint scripts make sense, but the bind mounts may conflict with what the intention was for dev mode.
I am not sure what the best option is, but I am thinking we should completely seperate "dev" mode from "release" mode.
Have 1 monster docker-compose file with everything needed for dev, local source mounted into the container, etc.
And a seperate docker-compose setup for production hosting.

Until now we have not configured these bind mounts in the docker-compose release setup. As these will override any local_settings.py file that someone might have created in dev mode inside dojo/settings/ folder.

Wouldn't https://github.com/DefectDojo/django-DefectDojo/blob/master/docker-compose.yml#L40-L42 make it also available in release mode?
Seems it does for me.

wow, forgot about that I added that. I guess it all works as expected until you add a local_settings.py in the extra_settings folder. that would overwrite the one in dojo/settings. maybe just merge it and see what happens :-D

My use-case is that I guess test in dev or ptvsd, and then I know that putting that same file over to extra_settings would give me the same thing in release. I guess it adds up in my head :man_shrugging: but I may be missing something -- would not surprise me, especially these days.

But we should at least be consistent! So I guess we should merge this anyways.


Transaction confirmation/Token name issue

Issue with updated token names for tokens launched on Stackswap's Token Launchpad.
Issue is occurring on v3.7.1
Tokens launched from the Launchpad before the last update are showing correctly, but tokens launched since then are showing as "TOK" ticker instead of the updated ticker.
e.g. SP1Z92MPDQEWZXW36VX71Q25HKF5K2EPCJ304F275.tokensoft-token-v4ktqebauw9
This token should be displaying a ZERO ticker.

Thanks for the report. We have a release ready for tomorrow that solves it.

This might be related to an issue in stacks.js (https://github.com/hirosystems/stacks.js/pull/1230) username validation, which fails to completely authenticate users. The latest fixes are released in<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

The latest fix<EMAIL_ADDRESS>fixes the issue of an account with bns name to sign in successfully to https://app.stackswap.org/swap, this however failed in v 3.7.1 testing.
cc @andresgalante @janniks

Would it just be an issue of updating the token metadata on the DB about once a day?

zero : SP1Z92MPDQEWZXW36VX71Q25HKF5K2EPCJ304F275.tokensoft-token-v4ktqebauw9
wagmi : SP1Z92MPDQEWZXW36VX71Q25HKF5K2EPCJ304F275.tokensoft-token-v4ka1lgcc8y
In the above examples, Zero token is not updating to new values, but WAGMI token shows up as expected.

@andresgalante Let's assign to a dev?

yes @diwakergupta, the metadata is correct.

@timstackblock can you provide the Explorer URL of your TTT token here so we can check to see if the API is returning its data fine?

I'm getting blank symbol (and other) data from the API for wagmi, zero and ttt when I try today:
https://stacks-node-api.mainnet.stacks.co/extended/v1/tokens/SP1Z92MPDQEWZXW36VX71Q25HKF5K2EPCJ304F275.liquidity-token-v5ke9jtb4ij/ft/metadata

@kjchiang3 I presume that's not actually @timstackblock's TTT since it was deployed a month ago whereas Tim tested 10 days ago?

@kjchiang3 I presume that's not actually @timstackblock's TTT since it was deployed a month ago whereas Tim tested 10 days ago?

it could be, as we deploy a template token ahead of time, then change metadata using our Launchpad.

I understand this isn't what Hiro had intended at first, but considering that our project for 1-step token generation grant had been approved last year(https://github.com/stacksgov/Stacks-Grants/issues/173), I do believe it is imperative to have this simple functionality included. Can't you just update your server on token metadata once a day or so? This cannot be a difficult job.

After talking to @rafaelcr and @zone117x, we can't think of a workaround that doesn't deviate from what the API already represents and its design principles. While we can attempt to flag specific contract metadata for reprocessing in the database, that manual chore is not scalable. It still won't cover this scenario for Devs relying on either self or other hosted API instances.
I was exploring to see if we can add an on-demand API to flag contract metadata for reprocessing, but that would be Hiro-specific. It doesn't promote a standard way of accomplishing this from the Stacks ecosystem perspective.
While digging further, I was kindly pointed to hirosystems/stacks-blockchain-api/pull/576, and this comment, in particular, the details captured in that discussion are worth reading if you aren't familiar with the initial Metadata design. The ideal solution is to standardize the event payload representing the contract metadata change. And the API can react to the change and do the right thing behind the scenes to reprocess the metadata.
All that said, there are a couple of concrete next steps that @zone117x and @rafaelcr have proposed.
1- We are going to be drafting a SIP on dynamic metadata change events.
2- To build on (1), we will implement the API scaffolding to react to, and process metadata change events.

I'm closing this out as a wallet-specific issue since it appears that any next steps will take place on the SIP or API side. 🙏

FYI @stackswap @markmhx @kjchiang3 a draft SIP to solve metadata updates is now submitted https://github.com/stacksgov/sips/pull/72


Does not build [Win7]

Hi all
Unfortunately I cannot build as described. I setup a clean node.js environment on Windows 7, setup the build environment successfully as described (thanks for the hints!). When running:
gulp clean build package --release

I receive the following error messages:
[22:28:52] Using gulpfile ~\Downloads\ledger-wallet-chrome\gulpfile.js
[22:28:52] Starting 'clean'...
[22:28:52] Starting 'build'...
[22:28:52] Starting 'package'...
[22:28:52] Finished 'clean' after 109 ms
[...]\Downloads\ledger-wallet-chrome\compilation\gulp-flavors.coffee:36
    _ref1 = file.path.match(/.+\/(.+)$/), __ = _ref1[0], filename = _ref1[1];
                                                    ^
TypeError: Cannot read property '0' of null

Could very well be my mistake. Any help would be appreciated!

Hi,
I found the problem, it was due to Windows path separators. We never tested compilation on Windows until now ;). You can pull the master branch and run npm install to update node modules. The application compiles now though a bug remains related to less files compilation. The application is working but looks ugly. I'm still looking for a fix apparently this is caused by the 'gulp-less' plugin.

After updating gulp-less to its latest version, everything seems to be alright. Just run npm update after pulling the master branch and all should be ok.

Hi
Thanks for looking into this, now it builds without errors. However, I do not see any output file (crx) upon running
gulp clean build package --release

I also tried a simple
gulp package --release

but it also did not create anything.
You write

This will create a crx file into the dist directory.

There are two issues with that:

I do not see any dist directory in the main folder. I assume you mean "compilation\dist"?
The above mentioned folder is empty after the build.

I would be happy if you could resolve this one as well! Thanks!

Hi,
This should work now ;). The dist/signature directories created in the compilation folder was a bug. You should see the dist directory in the root dir of the repo.

Thanks for your responsiveness and fixing this. Seems to be all working now!


Time picker

Thank you for the minimal date picker.
Coming to the issue, creating a placeholder for time picker (As also requested in HN comments)
Thanks in advance.

Added basic support for this in #28, which makes it possible to select the time on the inputs, and makes it easy to integrate with a custom time picking UI (which is out of the scope of this library).

Would love a default Time Picker like Material UI's https://material-ui.com/components/pickers/
I think this is the only thing missing :)


[MemoryPackUnionFormatter] (net7.0) yields compiling error: CS8987 The ‘scoped’ modifier of parameter ‘value’ doesn’t match overridden or implemented member

I have various Types defined in 3 libs, namely Model, Man, Config,  and these types are finally used in the lib: Server
i.e. Server depends on Model, Man, Config, otherwise Man and Config depends on Model.
And, All these Types inherits a single interface  IMyMemoryPackable defined in Model.
I wrote a MyMemoryPackableFormtter.cs in lib Server like this to register all these types:
[MemoryPackUnionFormatter(typeof(IMyMemoryPackable))]
    [MemoryPackUnion(0, typeof(Type1))]
    [MemoryPackUnion(1, typeof(Type2))]
    [MemoryPackUnion(2, typeof(Type3))]
    public partial class MyMemoryPackableFormatter
    {
    }

but compiler yields:
Server (net7.0) :
compiling error: CS8987 The ‘scoped’ modifier of parameter ‘value’ doesn’t match overridden or implemented member
However,
if i split this MyMemoryPackableFormatter  into three separate Formatters(note all these formatters register for the same interface, namely IMyMemoryPackable) into three libs (no matter their names r the same or not), the compiler compiles OK and the program can launch, but the generated code seems only to use one Formatter to do serialize Job (other Types cannot be found ).

Is the target framework the same for all projects?
MemoryPack is not compatible with netstandard2.1 and net7.

All these libs are dual-targeted, and have passed compiling before MemoryPackUnionFormatter was introduced, that proves not possibly the case of incompatibility.
And please note the case in my descrption after "However,...",  in that case the program is successfully compiled, only throws expception on Type missing, as also proves no problem of incompatibility.

Sorry, it's a bit tough to read, can you give me a reproduction repo?

https://github.com/xiaoyuvax/MemoryPackUnionFormatterRepro.git
compile the Server project u'll see the error.

i suddenly found it's possibly because of the LangVersion not compatible

Confirmed: the server used LangVersion=9.0 , it is not qualified to compile , set it to 11.0, compile passed!


feat(angle): add FromDegrees() func

This PR introduces a FromDegrees() function to simplify creating a new angle given a value in degrees.

@einride/team-ads-platform could you please set the ci/make check as required in the branch protections of master?
Currently, the check is not required:

This is what we'd like to have:

Redundant to

#148


Distill TOC not working with special characters (e.g. ', ?)

Acknowledge the following

[x] I carefully read and followed the Getting Started guide.
[x] I read through FAQ and searched through the past issues, none of which addressed my issue.
[x] The issue I am raising is a potential bug in al-folio and not just a usage question.  [For usage questions, please post in the Discussions instead of raising an issue.]

Describe the bug
I have added the TOC to my distill blog posts, following #470. That works fine. However, if the section or subsection contain special characters (as for instance apostrophe ' or question mark ?), the links in the table of contents do not work.
To Reproduce
Steps to reproduce the behavior:
Simply create a distill blog post, add a section with a question mark and add it to the TOC.
An example is present in my blog post here. See also the source code on Github here.
Expected behavior
TOC link should work.
System (please complete the following information):

OS: Linux, Ubuntu 20.04
Browser (and its version) Firefox 94.0
Jekyll version: 3.9.0
Ruby version: ruby 2.7.0p0 (2019-12-25 revision 647ee6f091) [x86_64-linux-gnu]

@LoryPack, thanks for raising this issue.
do you mind testing something quick for me in your blog? if you edit _layouts/distill.html and change #{{ section.name | downcase | replace: ' ','-' }} to #{{ section.name | handleize }} on lines 75 and 79, does that help? thanks.

Hello, thanks for that. I have tried what you suggest, but if you do that no link in TOC works!
I realized the issue in the first place is not due to the links in TOC. In fact, if I remove the TOC from the blog post, still the links with # go to the right section, except for the ones in which a "?" or " ' " appear. For instance, in my example, going to
http://www.lorenzopacchiardi.me/blog/2021/generalizedBayes/#power-likelihood
goes to the right section, while
http://www.lorenzopacchiardi.me/blog/2021/generalizedBayes/#so-what-is-wrong?
does not, even if no TOC is present.

@LoryPack, makes sense. looks like handleize is not a valid jekyll filter; the correct one is slugify. #519 should fix the issue (see code changes here). please check and let me know if it works on your end. thanks!

Seems to be working now, thanks a lot!


NoSuchMehodError: The getter 'pasteButtonLabel' was called on null happened when textField long press

Hi,
Our app needs to support multi-languages, here is code as below,
@override
Widget build(BuildContext context) {
return _isLoadConfig?new MaterialApp(
title: 'Oil Loan App',
theme: new ThemeData(
primaryColor: themeColor
),
home: new Scaffold(body: showWelcomePage(context)),
locale: _locale,
localizationsDelegates: [
GlobalMaterialLocalizations.delegate,
GlobalWidgetsLocalizations.delegate,
CustomLocalizations.delegate
],
supportedLocales: CustomLocalizations.supportedLocales,
onGenerateRoute: Application.router.generator,
):new Center(child:CircularProgress());
}
but we faced the error as below when any textfield long press
NoSuchMehodError: The getter 'pasteButtonLabel' was called on null
receiver: null
tried calling: pasteButonLabel
This error happened on iOS, but we never faced it for android which flutter version is 1.0.0. Recently we are going to upgrade to 1.2.1 and expect to solve this issue once flutter is upgraded to 1.2.1. but the test result is still not as expectation. this error not happened on iOS simulator(iOS 12.1 and iPhoneX), but this error happened on iPhone device. the iPhone device is iPhoneX, the iOS version is 12.2. any ideas?
thanks, Yawei

This is the exact error reported in:
https://github.com/flutter/flutter/issues/13452

@HansMuller, @xster ins't this resolved by https://github.com/flutter/flutter/pull/32513 ?

solved with a new class to implement CupertinoLocalizations for specified language.


Key going from string to number to string before sending request

I might very well be doing this wrong but according to the docs

A part of a Path that can be any JSON value type. All types are coerced to string, except null. This makes the number 1 and the string "1" equivalent.

The route is setup like so
route: "item[{keys:ids}]['name']"
And the get I am doing
model.get(['item',"16140997201362921294", 'name'])
    .then(data => {
        console.log('fetching item');
        console.log(data);
      })
    .catch(error =>{
      console.log(error);
    })

However when I log the requests server side they are listed as
/model.json?paths=[[%22item%22,16140997201362921000,%22name%22]]&method=get
Showing the ids field having been changed from string to a number and then back again.

@enderv we coerce to integers during path collapsing so we can collapse contiguous ranges ([0, 1, 2, 3], etc.) into {from, to} keys. Can you verify that's what's happening here?

Yeah thats the spot. Just followed it down and its passing the isNumber check so the rounded number value is being added to the keys array.
Is that as intended or could another condition be added to isNumber to check the string length as well?

FYI same issue as https://github.com/Netflix/falcor/issues/704#issuecomment-173440152

Oops. Thanks @chrisvariety I'll close this one out as a dupe then.


[BEAM-11691] Skip JavaUdfLoaderTest instead of failing when jar path …

…system properties aren't set.
Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Run SQL Postcommit

Run SQL Postcommit

R: @amaliujia
cc: @suztomo

R: @amaliujia
cc: @suztomo


Show IP adresses based on network ports

For Cloud VMs show IPs and MACs based on network_ports
For Cloud VMs show IPs and MACs based on network_ports. Which is
also fixing showing ip's when VM have multiple ports. And for
ipv6, showing all IPs, even when multiple IPs are associated to
one port.
For now, only Google Provider is not supporting this, until it will
be rewritten to use a NetworkManager.
Fixes BZ:
https://bugzilla.redhat.com/show_bug.cgi?id=1303997
https://bugzilla.redhat.com/show_bug.cgi?id=1316026
Also adding mac_addresses and ip_addresses to several summaries and lists

@dclarizio @blomquisg please review

@Ladas Can you add some before/after screenshots?  Thx, Dan

@dclarizio here it goes :-)
network port list before:

network port list after:

network port summary before:

network port summary after:

cloud vm summary before:

cloud vm summary after:


Various fixes open qasm reader

This is the same as the closed #11, #13 moved to a stand-alone branch for convenience.
@crazy4pi314 @RolfHuisman

Build is failing, because the sources are still using the old syntax. Currently have a merge request in the LCM branch which fixes this.

Hi @RolfHuisman :)
Thanks for your review!
The .qs files' style here may not be the nicest, but I think the syntax is correct (the Q# code compiles).
Anyway, the style in #16 is better (and this PR's branch is merged into #16's branch), so I guess we can close this PR.


Issue when setting primary display with xrandrHeads

Issue description
With this configuration
{
# ...
services.xserver.xrandrHeads = [ "HDMI1" "HDMI2" ];
}

or the equivalent:
{
# ...
services.xserver.xrandrHeads = [ { output = "HDMI1"; primary = true; } "HDMI2" ];
}

xrandr --current reports:
Screen 0: minimum 8 x 8, current 3840 x 1200, maximum 32767 x 32767
DP1 disconnected primary (normal left inverted right x axis y axis)
HDMI1 connected 1920x1200+0+0 (normal left inverted right x axis y axis) 520mm x 320mm
   1920x1200     59.95*+
   ...
HDMI2 connected 1920x1080+1920+0 (normal left inverted right x axis y axis) 510mm x 290mm
   1920x1080     60.00*+  60.00    50.00    50.00    59.94
   ...
HDMI3 disconnected (normal left inverted right x axis y axis)
VIRTUAL1 disconnected (normal left inverted right x axis y axis)

while I would expect it to report:
Screen 0: minimum 8 x 8, current 3840 x 1200, maximum 32767 x 32767
DP1 disconnected (normal left inverted right x axis y axis)
HDMI1 connected primary 1920x1200+0+0 (normal left inverted right x axis y axis) 520mm x 320mm
   1920x1200     59.95*+
   ...
HDMI2 connected 1920x1080+1920+0 (normal left inverted right x axis y axis) 510mm x 290mm
   1920x1080     60.00*+  60.00    50.00    50.00    59.94
   ...
HDMI3 disconnected (normal left inverted right x axis y axis)
VIRTUAL1 disconnected (normal left inverted right x axis y axis)

like when I first run:
xrandr --output HDMI1 --auto --primary --output HDMI2 --auto --right-of HDMI1

Why is DP1 set as the primary display in the first case?
Steps to reproduce
I can provide more information on my setup if this bug is not easy to reproduce.
Technical details

System: 17.09.1716.68ef4b14bc
Nix version: nix-env (Nix) 1.11.15
Sandboxing enabled: build-use-sandbox = true

I had a similar issue; in my case two screens were being loaded in the wrong order. After taking a look at the system journal, it seemed that X was ignoring the configured screens, as I got the log message
   Using a default monitor configuration.```

I managed to get a workaround by adding the following to my `configuration.nix`:

{
...
services.xserver.screenSection = "Monitor "multihead3"";
...
}

I had the same issue and this worked for me.
{
  services.xserver.displayManager.setupCommands = ''
  ${pkgs.xlibs.xrandr}/bin/xrandr --output HDMI-1 --left-of DP-1
  '';
}

Same problem here. Using @energizah solution works for me.


Fix #1226, Add Contributing Guide

Describe the contribution
Fix #1226
Added a contributing guide that links to the main cFS contributing guide.
Expected behavior changes
Users should be able to view the contributing guide contents easily from the cFE repo.
Contributor Info - All information REQUIRED for consideration of pull request
Ariel Adams, ASRC Federal

Also fixes #405


Missing braces for some codes

First of all, thank you—this was a fantastic addition to make moving from v1 to v2 tolerable. I used it convert 41 shortcut keys.
I found 2 that just didn't convert properly. Pretty much all of them inserted the braces where needed—e.g.
; v1
<^>!C::
	send, {U+00A9}
	Return

became
<^>!C::
{ ; V1toV2: Added bracket
	Send("{U+00A9}")
	Return
} ; V1toV2: Added bracket in the end

But these 2 did not include braces:
; v1
!=::
	send, {U+2248}
	Return

became
!=::
	Send("{U+2248}")
	Return

and
; v1
!+=::
	send, {U+00B1}
	Return

became
!+=::
	Send("{U+00B1}")
	Return

which both result in error Hotkey or hotstring is missing its opening brace.

Thanks for the report! Hopefully someone will pick this up to fix

There's a part of the regex related to this that I'm not clear on what was trying to guard.
This part [^;\n\r\{}\[\]\=:] at lines 3334 and 3337
https://github.com/mmikeww/AHK-v2-script-converter/blob/882e221ff70ce0a5808644e87ce8530bd63db25d/ConvertFuncs.ahk#L3334-L3337
Currently replacing it with a dot doesn't break any test
but it would be nice to know a bit more about it.

git blame shows that @dmtr99 added this section of code, maybe he can elaborate
But these types of things are exactly the reason why extensive tests for every single corner case is useful... then when we make a change we can see what breaks and what doesnt

I looked at it and guess this was not necessary. Changed in the master. Tests seems to be fine.


Fix anchor link navbar offset

Summary
This quickfix fixes selected content being not visible behind the navbar e.g. for glossary items. This happened when linking glossary items directly e.g. as the Console does.
Changes
Add CSS to add an offset to selected anchor links.
Checklist

[x] Scope: The referenced issue is addressed, there are no unrelated changes.
[x] Run Locally: Verified that the docs build using make server, posted screenshots, verified external links. Test with HUGO_PARAMS_SEARCH_ENABLED=true if style changes will affect the search bar.
[x] New Features Marked: Documentation for new features is marked using the new-in-version shortcode, according to the guidelines in CONTRIBUTING.
[x] Style Guidelines: Documentation obeys style guidelines in CONTRIBUTING.
[x] Commits: Commit messages follow guidelines in CONTRIBUTING, there are no fixup commits left.

@kschiffer you can merge


Step 2 of ruff integration: fix issues

Needs to be reviewed after merging #938
This is step 2 of the ruff integration which fixes all warnings with the following exceptions:

simtools/camera_efficiency.py used variables called N1, N2, etc, which are used in this way in the DB and in scripts of sim-telarray. Disabled here ruff (we had it disabled in pyling)
temporarily disabled these two issues regarding tests:

PT011 pytest.raises(ValueError) is too broad (I think we should fix those)
PT012 pytest.raises() block should contain a single simple statement (Not sure if this is absolutely necessary; but easy to fix)

This is with "D" (documentation checks) switched off. Ruff suggest a lot (thousands) of changes to the docstrings and we need to discuss how we do this (also in light of #886). I think we first want to understand how we want our docstrings to look like (I don't like all ruff suggestions.)

I mark this ready for review - codecov complaints about missing tests, but this PR is not about test coverage and I suggest to ignore these.

Having a broad test is still better than having no test. I think it is strange to remove tests only to have ruff not complaining about them.

As discussed in person, PT011  and PT012 is something we need to discuss if we want to integrate them. We are not obliged to use all options proposed by ruff. I am opening an issue on it.
Note also that these two are very independent of the rest of the code.

As @GernotMaier prefer to ignore the failing coverage test and deal with it later, I am merging this.


low performance: Check AUgemntation and Normalization

Issue

Model performance rather low on validation dataset:

Solutions

check for better data normalization

train and val

double check Augmentation functionality

torchvision vs albumentations

Normalization is now implemented

went back to albumentations: https://github.com/initze/thaw-slump-segmentation/commit/56b11f8fed2b0d0afbcbdf9632e89a8611826791

required some reshaping of target/mask/label data
Should work fine now

https://pytorch.org/vision/stable/transforms.html#transform-classes-functionals-and-kernels


VCS syncing issues with multiple AS project windows

Let's say we have 2 Android Studio (AS) projects with AS has multiple windows enabled.
Project A has VCS dependency of A_ApiModule
Project B has VCS dependency of B_ApiModule
When open project A in AS and let it finished syncing Gradle, everything works just fine here (repo of A_ApiModule is cloned into dir of A/libraries/A_ApiModule).
Now if we'll open project B in another AS window and let it sync Gradle, the scenarios are as follows:

B_ApiModule is correctly cloned into dir of B/libraries/B_ApiModule
After that, content in A_ApiModule dir of A/libraries/A_ApiModule are cleared (except IDE generated file, e.g. *.iml, and the folder itself is not removed)
B_ApiModule is now also cloned into dir of A/libraries/B_ApiModule

Now, when we going back to work on project A again, we'll need to remove VCS dir and resync the Gradle again. Otherwise, the project will failed with message of Destination path "A_ApiModule" already exists and is not an empty directory (since its content is removed as mentioned in 2.)
Updated: Sometime it in this sequence:

B_ApiModule is cloned into dir of A/libraries/B_ApiModule (no VCS dir created in project B directory)
Project B finished syncing with failed.

Gradle 4.6 : Android Studio 3.2.1

This should be fixed in v2.0.0. Can you please check it?

Looks like it works as expected now. No issues so far. Thanks!


Action on Docker mentions in Concepts > Extending Kubernetes

Umbrella issue: #30771
Fixes #30922
https://kubernetes.io/docs/concepts/extend-kubernetes/compute-storage-net/network-plugins/
FIle: kubernetes/website/content/en/docs/concepts/extend-kubernetes/compute-storage-net/network-plugins.md

Description: Docker is mentioned three times in the doc and dockershim once

Line: 24, 31, 33, 149

Actions

For Installation, not sure. The description of how the kubelet looks for plugins probably needs to change.
For Network plugin requirements, update the URL in the link to the CRI equivalent (perhaps k8s.io/cri-api/pkg/apis/runtime/v1, not sure)
For Network plugin requirements, remove like Docker with a bridge.
For Customizing the MTU, remove Docker.

/sig docs
/language en
/help-wanted

/triage accepted
/assign

What to do with these? @kubernetes/sig-docs-en-owners

For Installation, not sure. The description of how the kubelet looks for plugins probably needs to change.
For Network plugin requirements, update the URL in the link to the CRI equivalent (perhaps k8s.io/cri-api/pkg/apis/runtime/v1, not sure)

I am asking as @shannonxtreme is not sure.

@reylejano Any thoughts here?

There is an open PR to update this page, PR https://github.com/kubernetes/website/pull/31667 but still need a few fixes for this issue:

For Network plugin requirements, SIG Node would have to confirm what the updated link should be to

I agree with @shannonxtreme on the Network plugin requirements section to remove like Docker with a bridge

Okay, so someone is working on this, I am unassigning myself 🙂
Thank you for the updates!

/assign @chrisnegus
To follow up on the status of this issue.

An org member can close this. What's the reason / justification for closing?

@sftim Looking over the content, it seems like the issue has been resolved. I reviewed this with the team in the "SIG Docs: 1.24 Dockershim removal" meeting, and we agreed that this looks like it should be closed.

/close


Drag and drop is not behaving properly

Explore this solution:
https://nextjs-ai-chat-bice.vercel.app/chat/7xhPRza

In order to achieve the described behavior of styling an item that is being dragged to be rotated, using HTML, JavaScript, and TailwindCSS for styling, you will need to implement a drag and drop interface and manipulate the styles of the draggable element during the drag events. Since TailwindCSS is a utility-first CSS framework, you can apply a rotation class to the item being dragged. However, TailwindCSS, by itself, does not directly handle JavaScript events or the logic for showing/hiding elements during drag operations, so you'll need to write some custom JavaScript for that.
Here is a step-by-step example of how you might implement this:

HTML Setup: Define your draggable item in HTML. Make sure to include TailwindCSS in your project.

<!DOCTYPE html>
<html lang="en">
<head>
    <meta charset="UTF-8">
    <meta name="viewport" content="width=device-width, initial-scale=1.0">
    <title>Draggable Element</title>
    <link<EMAIL_ADDRESS>rel="stylesheet">
</head>
<body>

<div id="draggable" class="p-4 bg-blue-500 text-white cursor-move" draggable="true">
    Drag me
</div>

<script src="app.js"></script>
</body>
</html>

JavaScript (app.js): Add the drag event listeners to the draggable item. When dragging starts, you clone the item and apply rotation using a TailwindCSS utility class. TailwindCSS offers rotation classes such as rotate-45 for a 45-degree rotation. You also hide the original item and move the clone with the cursor.

document.addEventListener('DOMContentLoaded', function () {
    const original = document.getElementById('draggable');
    let clone = null;

    original.addEventListener('dragstart', function (e) {
        clone = original.cloneNode(true);
        clone.classList.add('rotate-45', 'opacity-50'); // Rotate and make it semi-transparent
        document.body.appendChild(clone);

        // Adjust position
        const rect = original.getBoundingClientRect();
        clone.style.position = 'absolute';
        clone.style.left = `${rect.left}px`;
        clone.style.top = `${rect.top}px`;

        // Hide the original element
        original.style.visibility = 'hidden';

        // Necessary to make the drag work
        e.dataTransfer.setDragImage(clone, 0, 0);
    });

    original.addEventListener('dragend', function () {
        // Remove the clone and show the original element
        if (clone) {
            document.body.removeChild(clone);
        }
        original.style.visibility = 'visible';
    });

    document.addEventListener('dragover', function (e) {
        e.preventDefault(); // Necessary for the drag to work
        if (clone) {
            // Update clone position
            clone.style.left = `${e.clientX - clone.offsetWidth / 2}px`;
            clone.style.top = `${e.clientY - clone.offsetHeight / 2}px`;
        }
    });
});

Explanation:

When the drag starts (dragstart event), you clone the original item, apply the rotation and semi-transparency using TailwindCSS classes (rotate-45, opacity-50), and adjust the clone's position to follow the cursor.
During the drag (dragover event), you update the clone's position to be under the cursor.
When the drag ends (dragend event), you remove the clone and make the original item visible again.

This is a basic implementation to give you the desired effect of having an item rotate and follow the cursor while being dragged. Depending on your project's requirements, you may need to enhance this script, especially if you're implementing a complex drag-and-drop interface.


How would I add a class specifically for the questions of panel type

Are you requesting a feature, reporting a bug or ask a question?
Question
What is the current behavior?
With the custom css classes, I am unable to add a class to questions of panel type(like class="panel-container"). The panel type questions I am referring to:

Does survey.js currently have a method of solving this issue?
Thanks

@john1625b I have added to classes: panel.title and panel.container
Thank you,
Andrew

@andrewtelnov Thank you so much for adding this feature. How would I go about getting these changes? I have upgraded my node modules so now my survey-react version is ^0.12.23. I added the panel object to my custom styles like so:
panel: {title: "panel-title", container: "panel-container"},
But when I inspect the panel element on my project, I do not see the relevant classes showing up. Has the commit not been released to npm yet?
Thanks

@john1625b We will release, I believe, in the middle next week.
I may update the dev version here.
Thank you,
Andrew


Allow use of alternate branding by the DEV server

Added instructions for how to run the dev server with alternate branding.

For example, here is my dev server with RHV branding:


Fix issue with an overflowing settings dropdown in the module list

Current implementation:

After fix:

Assigning to @Coenego for review.


Backslash breaks fuzzy search

In below gif, note how typing n fails to trigger #nabla snippet as expected, but typing n triggers the \\nabla snippet.

please provide those snippets in source

	"nabla": {
		"prefix": "\\nabla",
		"body": [
			"∇"
		],
		"description": "nabla"
	},

please also add what exactly you are typing and when intellisense shows and whether you trigger IntelliSense manually or not

The gif shows exactly what I was typing. I use a keyboard shortcut to jump between panes but that's it. I literally type the letter "n" and "\nabla" shows up as suggestion, but to get "#nabla" as suggestion, typing "n" is not sufficient, which is as expected, just as typing the letter "o" will not bring up suggestion for "for".

So, this happens when typing only, right? What happens when you explicitly request suggestion via ctrl+space?

So, this happens when typing only, right? What happens when you explicitly request suggestion via ctrl+space?


send frequent data to clients

HI Arunoda :
I need to send values of a counter on server side which gets changed evey 80 miliseconds to all clients. The reason for this is to move an object on a canvas for all clients base on this counter.  I used MongoDb and Publish /Subscribe first  . But I am not getting some of the updates to the server side counter in clients . So I need another solution and I ran into Your stream package.  Before implementing it, wanted to see if this is the right solution or you know of any better supported mechanism .
Thanks for your suggestions.

I ended up using Streamy package and it does the job very well.


How to solve "10104 Error ()" at API gs.get_notes(uid)?

It's not related to whether your battle chronicle is public or not. I tried it with a friend's battle chronicle (we have the same settings related to Battle Chronicle in "Privacy Settings") and I couldn't fetch their data but I was able to fetch mine.

Considering this error is still a thing I'm assuming you can only ever get your own notes. I can't view my friend's notes on the website either so mihoyo most likely just doesn't want us to have access to it at all.


Update temptifly and remove pf-c-content style which is no longer req…

…uired
Signed-off-by: Rastislav Wagner<EMAIL_ADDRESS>

/hold
waiting for https://github.com/stolostron/temptifly/pull/173

/hold cancel


refactor: add symlink support

#11  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @zakuciael and the rest of your teammates on  Graphite


⚠ GitLab Server has degraded performance

In 150cf1d, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 7038 ms

Resolved: GitLab Server performance has improved in abe0092.


Match statement

first of all, I'm no expert in Python.
but, from what I've read, the match/case statement should be supported as of version 3.10 of python.
In HA, we are at 3.11 now. When retrieving the python version in pyscript it says 3.11
So, when I try to use match/case like
testvar='value'
match testvar:
     case 'value':
          pass
      case _:
          pass

this gives me error:
NotImplementedError: ..... not implemented ast ast_match

Pyscript implements a python interpreter using async methods.  I wasn't aware this language feature was added, so I'll need to add support for it.

I reviewed PEP 636 and it looks quite difficult to implement in the pyscript interpreter.  So I'm not planning to support this.
I'd recommend putting your match code in a function with the @pyscript_compile decorator.  That way you'll have access to full native Python functionality inside that function.

Thx!

@stefanuytterhoeven could you please update or close the issue? Thank you

Solved


Updating release v0.1

Updating release v0.1

Codecov Report

:exclamation: No coverage uploaded for pull request base (release-v0.1@9e857b6). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@               Coverage Diff                @@
##             release-v0.1       #61   +/-   ##
================================================
  Coverage                ?   100.00%
================================================
  Files                   ?         1
  Lines                   ?         8
  Branches                ?         0
================================================
  Hits                    ?         8
  Misses                  ?         0
  Partials                ?         0

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.


Additional validation when changing a relationship

If you now attempt to change a relation to a value that isn't a resource, jsonapi-client will throw an error:
somePhoto.relationships("photographer").add("foobar");

     Error: Expected Resource, got String
      at Resource._checkIsResource (lib/Resource.js:36:9)
      at Resource._addRelation (lib/Resource.js:140:8)
      at Object.add (lib/Resource.js:129:12)
      at Context.<anonymous> (test/fetch.js:53:49)  <-- this will be the line number matching the snippet above.

Addresses #6

Code changes look solid and the new regression unit test is passing so: :+1: :shipit: from me.

:+1:


Cutted text in NAME_LEFT.html

There's lack of delay line in NAME_LEFT.html template that is causing cutting text during playback. It's present in NAME_RIGHT.html and it works properply there.
                        delay: 0.2, <------
                        width:function(i, target)
                            {
                            target.style.width = "auto";
                            var width = target.offsetWidth;
                            target.style.width = "0px"; //now reset it to 0
                            return width; //return the natural height
                            },

Yup. A "known issue", that the animation sometimes never "completed". The animations in the default templates (1.3.x) have been redone, and is not an issue anymore.


Module not found

Hi,
i want install this fantastic library, i have read the readme and when i have write in my App.js:
import 'react-flexview/lib/flexView.css' // FlexView is useless without its style

I have recive this error:
./src/App.js
Module not found: Can't resolve 'react-flexview/lib/flexView.css' in ...

Do you have any idea for this problem?
P.S. into node_modules/react-flexview/lib/ i don't find flexView.css
Thank you so much

Hi @MateYoutility
it appears that after thew refactor in TS we're no longer transpiling the SASS file in CSS in the lib directory... definitely a bug!
Thanks for the report, I'll fix it asap


Added swirl animation background

Fixes Issue🛠

Closes #63
Description👨💻

Under the background component, added a swirl animation background.
Type of change📄

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

This has been tested✅

Checklist✅

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have added demonstration in the form of GIF/video file
[x] I am an Open Source Contributor

Screenshots/GIF📷
https://github.com/Rakesh9100/Beautiify/assets/125905089/1b05493d-f979-4dc4-ac10-41cef6009d5c

Good, Merged!! 🥳 @saurav2208

Good, Merged!! 🥳 @saurav2208

Thank you 😊


Using an SSL certificate to connect to Mariadb

Thanks for your library.
To connect to my Mariadb I need to use an SSL certificate.
I could not find in the code how I could transfer the SSL certificate. Perhaps I somehow have to provide it via the connection string?
Could you suggest how I can do this? Thanks.
class Connection:
    """
    Representation of a socket with a mysql server.

    The proper way to get an instance of this class is to call
    connect().

    Establish a connection to the MySQL database. Accepts several
    arguments:

    :param host: Host where the database server is located.
    :param user: Username to log in as.
    :param password: Password to use.
    :param database: Database to use, None to not use a particular one.
    :param port: MySQL port to use, default is usually OK. (default: 3306)
    :param unix_socket: Use a unix socket rather than TCP/IP.
    :param read_timeout: The timeout for reading from the connection in seconds (default: None - no timeout)
    :param write_timeout: The timeout for writing to the connection in seconds (default: None - no timeout)
    :param charset: Charset to use.
    :param sql_mode: Default SQL_MODE to use.
    :param read_default_file:
        Specifies  my.cnf file to read these parameters from under the [client] section.
    :param conv:
        Conversion dictionary to use instead of the default one.
        This is used to provide custom marshalling and unmarshalling of types.
        See converters.
    :param use_unicode:
        Whether or not to default to unicode strings.
        This option defaults to true.
    :param client_flag: Custom flags to send to MySQL. Find potential values in constants.
    :param cursor_cls: Custom cursor class to use.
    :param init_command: Initial SQL statement to run when connection is established.
    :param connect_timeout: The timeout for connecting to the database in seconds.
        (default: 10, min: 1, max: 31536000)
    :param ssl: Optional SSL Context to force SSL
    :param read_default_group: Group to read from in the configuration file.
    :param autocommit: Autocommit mode. None means use server default. (default: False)
    :param local_infile: Boolean to enable the use of LOAD DATA LOCAL  (default: False)
    :param max_allowed_packet: Max size of packet sent to server in bytes. (default: 16MB)
        Only used to limit size of "LOAD LOCAL INFILE" data packet smaller than default (16KB).
    :param auth_plugin_map: A dict of plugin names to a class that processes that plugin.
        The class will take the Connection object as the argument to the constructor.
        The class needs an authenticate method taking an authentication packet as
        an argument.  For the dialog plugin, a prompt(echo, prompt) method can be used
        (if no authenticate method) for returning a string from the user. (experimental)
    :param server_public_key: SHA256 authentication plugin public key value. (default: None)
    :param binary_prefix: Add _binary prefix on bytes and bytearray. (default: False)
    :param db: **DEPRECATED** Alias for database.

    See Connection <https://www.python.org/dev/peps/pep-0249/#connection-objects>_ in the
    specification.
    """

Use the param ssl


lldb-wasm.patch apply doesn't work on the main branch

https://github.com/bytecodealliance/wasm-micro-runtime/blob/main/doc/source_debugging.md?plain=1#L42

Hi @g0djan
Thanks for reporting this issue, the document is outdated, I've submitted a PR https://github.com/bytecodealliance/wasm-micro-runtime/pull/2051 to fix it
BTW, currently WAMR will create pre-built lldb binary for every release, you can download a binary for your platform directly.


ci: add plugin and docs labels

labels config file like this ui/raidboss/raidboss_config.js with config, too?

label config file like this ui/raidboss/raidboss_config.js with config, too? #2984

Also done!