Support Amazon Redshift

Is your feature request related to a problem? Please describe.
Amazon Redshift is famous DW in Cloud environment.
We need to develop the ability to access Redshift from workbench first, and then create a datasource and dataset from Redshift.
Describe the solution you'd like
Describe alternatives you've considered
Additional context

Amazon Redshift icon 2가지 스타일 (color/gray)퍼블에 전달했습니다-


.zip and .tar.gz statistics

It would be possible to implement the funcinality of showing up the .zip and .tar.gz statistics?.
Thanks in advance.

No. Github's public API doesn't provide that.


Gateways upgrade for dual control plane should not happen immediately

Current default behavior is to immediately upgrade gateways in place which defeats the purpose of canary upgrades. A possible solution is to split GW management from the main installation manifest. This requires documentation and testing.

Duplicate of https://github.com/istio/istio/issues/27511 since this is a documentation task.


Opened tabs at startup behavior

Firstly Auto-open a terminal at app start is enabled in my settings.
When I'm launching terminus normally it is came with a terminal as expected. But when I right click my desktop and select Open Terminus here it is came with two tabs. One is my desktop, and one is default. I found it a little annoying. I think it only launch with one tab at my desktop.
Also when you close Terminus while Settings tab is open, after relaunch Settings tab is back. I think this behavior is completely unnecessary.

Also when you click Open Terminus here at desktop then close terminus while 2 tabs are open and launch terminus directly, terminus comes with 2 tabs; a blank tab with "Terminal" title, and my my default custom shell.

@Eugeny This issue is still not resolved.

@Ndelt this is the expected behaviour - tabs are initialized when first clicked.
In the original issue, other tabs stayed blank permanently

@Eugeny Then, how to I make terminal to only one tab is opened after launching it?

@NDelt you can't do that - Terminus remembers the tabs that were previously open and restores them on start.

@Eugeny May I open a feature request to add an option make Terminus to not remember the tabs previously opened?

Certainly


Using image with Transformation as LoadingPlaceholder

I want to use an image with different Transformations as LoadingPlaceholder.
In this issue:
https://github.com/luberda-molinet/FFImageLoading/issues/555
I create a colored  circle image, this I want to use as LoadingPlaceholder
How can this be done?

It's possible with the newest prerelease :)
See the example here: https://github.com/luberda-molinet/FFImageLoading/blob/master/samples/ImageLoading.Forms.Sample/Shared/Pages/Transformations/TransformationsSelectorPageModel.cs#L21-L43
public class CustomTransformationSelector : ITransformation
{
    readonly ITransformation PlaceholderTransformation = new CircleTransformation(5d, "#EEEEEE");
    readonly ITransformation ImageTransformation = new GrayscaleTransformation();

    public string Key
    {
        get
        {
            return "CustomTransformationSelector";
        }
    }

    public IBitmap Transform(IBitmap sourceBitmap, string path, ImageSource source, bool isPlaceholder, string key)
    {
        if (isPlaceholder)
        {
            return PlaceholderTransformation.Transform(sourceBitmap, path, source, isPlaceholder, key);
        }

        return ImageTransformation.Transform(sourceBitmap, path, source, isPlaceholder, key);
    }
}


Updated plugin but not documentation

This documentation is from 2013, but the plugin code has changed...

Please update documentation for proper working of IE8 support and no-query methods.

+1


remove static members from AnalogyMessageProducer

Avoiding concurrent targets locking eachother for no reason
Making disconnect with subsequent reconnect possible

Hi @BinaryCraX
thanks for the improvements. Not sure why I set it as static originally


fixFix AnalysisTaskExecutor unit test timeout. (#36456)

Deencapsulation.getField sometime doesn't work as expected, which may cause FE unit test timeout. Add get function in source code to make it easier.
backport: https://github.com/apache/doris/pull/36456

run buildall

run buildall


Daemon mode

In daemon mode, minion and master should not start more than once: "Daemon already running".

Invalid.


Add make

The latest release of pynacl 1.3.0 (a transitive dependency of Airflow) needs make to install.
While building this locally I ran into this issue  unpacking the HADOOP tar, which the --absolute-names fixed for me locally.

I have pushed a build of this to dockerhub jhofman/incubator-airflow-ci:latest and am running the integration tests with the new image on a fork I am working on: https://travis-ci.org/johnhofman/incubator-airflow/builds/433982418.
The master branch still fails due to other dependency problems relating to the latest click release being incompatible with flask-appbuilder.  That needs to be pinned to click==6.7 in the airflow setup.py.

@johnhofman I have raised a PR to pin click to 6.7 .However, I am not entirely sure if that would solve the issue.

@kaxil Cool! That will fix the click issue, but I think your build will still fail until this PR is also merged. Combined we should get a green build again.

I have added WIP because the tests on my fork failed. Because this fork includes other changes I won't know if it is the new ci image or my airflow changes. I will re-run the tests with the branch that @kaxil just patched to fix the click dependency issue.

@johnhofman Looks like the tests are going through. And should be fine. Can you just address the comments that @ashb has in this PR.

The tests.operators.test_virtualenv_operator.TestPythonVirtualenvOperator tests are now failing for the py27_xxx_env_docker variants.
Anybody have any insight into what dependencies could have broken those tests?

@johnhofman I am looking into it now.

Still not entirely sure on why the tests related to PythonVirtualenvOperator is failing. It is somehow using protocol=3 for pickle.dump causing it to fail. Can you just add the comment that @ashb asked, we should merge this PR than. Let's figure out the PythonVirtualenvOperator in a separate PR.

(Not familiar with this image, so this suggestion might not make sense) Could we fix this by explicitly mentioning pip2 or pip3?

I tried using pip2 or pip3 explicitly but it doesn't change the test results. I think the system python of the container has changed from Python2.7 to to Python3.5, probably from the updates to the xenial base image and/or the apt repositories.
This means all the python based tools like tox and virtualenv are now running in Python3.5, and the tests are assuming they run Python2.7.
I don't know which of these tools causes the problem, but I would guess virtualenv since it is the PythonVirtualenvOperator that is failing.

I have merged this one.
Good catch @johnhofman regarding python3 being used.
Let's figure out how we can resolve it now.

@seelmann @Fokko @dimberman Can one of you guys take a dig at resolving the above-mentioned issues. tests.operators.test_virtualenv_operator.TestPythonVirtualenvOperator are failing.

@kaxil I have a PR that fixes those tests. But now the KubernetesPodOperatorTests are timing out. I don't know if it is related to the changes I made or a different external dependency.

I think it has fixed it. Thanks @johnhofman


strange error

when i test a demo.py( actually the official support: assistant_library_demo.py )
the python program take me to a website like below:

when i enter in

when i click the allow button，i meet an error

please help me ( i am really anxious )

Did you also try assistant_grpc_demo.py ?

Did you also try assistant_grpc_demo.py ?

no， i haven't

how could i do,i am really anxious
Manoj<EMAIL_ADDRESS>于 2018年12月20日周四 下午3:52写道：

Did you also try assistant_grpc_demo.py ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/google/aiyprojects-raspbian/issues/548#issuecomment-448905698,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AcXyGEB8SQtK7KTAqukHYoWKQGvJOqkuks5u60HSgaJpZM4ZbrFt
.


Switch StartOffsetNewest/Latest to enum

See https://github.com/dropbox/kafka/pull/52

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


error: ‘i2c_smbus_write_word_data’ was not declared in this scope

I tried to raspberry this, and i went to ArduCAM/examples/RaspberryPi and typed make
g++ -std=c++0x -I./ -I../../../ArduCAM -c arducam_arch_raspberrypi.c
arducam_arch_raspberrypi.c: In function ‘uint8_t arducam_i2c_word_write(uint16_t, uint8_t)’:
arducam_arch_raspberrypi.c:125:3: error: ‘i2c_smbus_write_word_data’ was not declared in this scope
i2c_smbus_write_word_data(FD, reg_H, value);
^~~~~~~~~~~~~~~~~~~~~~~~~
arducam_arch_raspberrypi.c:125:3: note: suggested alternative: ‘i2c_smbus_ioctl_data’
i2c_smbus_write_word_data(FD, reg_H, value);
^~~~~~~~~~~~~~~~~~~~~~~~~
i2c_smbus_ioctl_data
arducam_arch_raspberrypi.c: In function ‘uint8_t arducam_i2c_word_read(uint16_t, uint8_t*)’:
arducam_arch_raspberrypi.c:140:7: error: ‘i2c_smbus_write_byte_data’ was not declared in this scope
r = i2c_smbus_write_byte_data(FD,reg_H,reg_L);
^~~~~~~~~~~~~~~~~~~~~~~~~
arducam_arch_raspberrypi.c:140:7: note: suggested alternative: ‘i2c_smbus_ioctl_data’
r = i2c_smbus_write_byte_data(FD,reg_H,reg_L);
^~~~~~~~~~~~~~~~~~~~~~~~~
i2c_smbus_ioctl_data
arducam_arch_raspberrypi.c:143:13: error: ‘i2c_smbus_read_byte’ was not declared in this scope
*regDat = i2c_smbus_read_byte(FD);
^~~~~~~~~~~~~~~~~~~
arducam_arch_raspberrypi.c:143:13: note: suggested alternative: ‘i2c_smbus_data’
*regDat = i2c_smbus_read_byte(FD);
^~~~~~~~~~~~~~~~~~~
i2c_smbus_data

I added #include <smbus.h>
to arducam_arch_raspberrypi.c
and copypasted the content of the .c file from
https://stackoverflow.com/questions/14268530/linking-smbus-c-to-smbus-h
(I know, hackish)
I got it compiling!
now I need to figure out how to plug the pins, I know how to wire 4 wires now, 3.3v, gnd and SCL and SCK
running: sudo ./ov2640_capture -c test.jpg 320x240
I get: Unable to open I2C device: No such file or directory

Hi,
You may not turned on the I2C device, so you can try this link.

I have i2c working since i can run my gyro no poblem. I'm using only 4 pins right now, 3.3v, gnd and sck sda its not even detecting the camera with i2cdetect -y 1

Perhaps you have a problem with the I2C method, I suggest you refer to our example on raspberry pi.

Perhaps you have a problem with the I2C method, I suggest you refer to our example on raspberry pi.
Link

Okay, i'll try that, what about the pin configurations, is having only sda and sck okay for spi?

The I2C pins of the camera are SCL and SDA.
The SPI pins are CS、MOSI、MISO and SCK, is your I2C clock using SCK?

no, its using SCL

So, I wire up both I2C and SPI before running the program?

https://pinout.xyz/pinout/spi#

so,
pins are
BCM9   = 21 (MISO)
BCM10 = 19 (MOSI)
BCM11 = 23 (SCLK)
BCM0  = 27 (SPI0)
I'm going to wire up SPI only.

so,
pins are
BCM9 = 21 (MISO)
BCM10 = 19 (MOSI)
BCM11 = 23 (SCLK)
BCM0 = 27 (SPI0)
I'm going to wire up SPI only.

BCM0 is not used, please wire the CS to BCM17.
You could see the README.

This is from the readme,

//SPI pins
//set 9 (MISO), 10 (MOSI), 11 (SCLK) alt fxn 0 (SPI0)

I dont know what alt fxn means thoough..

Sorry for the readme mislead you. Just ignore the alt fxn (SPI0), which just want to tell you to use SPI0.

Lets draw a picture here, where is MISO MOSI SCLK and SPI0 on the camera?

The ArduCAM/RaspberryPi/SPI_Camera/ is used for the specified camera, as shown in the following figure, for reference: https://www.arducam.com/product/arducam-2mp-spi-camera-b0067-arduino/
Not the parallel camera in your picture.

I have the same question at first。finally ，I can compile it.
but when I run the command line sudo ./ov5642_capture -c test.jpg 320x240
I get  the result as below
OV5642 detected.
Changed resolution1 to 320x240
Start capture
CAM Capture Done
Reading FIFO and saving IMG
The length is 2792
but ,the size of picture is 0
WHY?

Hi,
We noticed that there is no clearing FIFO in your output. Please clear FIFO before you start capture. In addition, you can print the data read from FIFO to the terminal. If the data is jpeg data, then there may be a problem with the way you save the image.

thanks .but how to clear FIFO?
in fact ,I bought a ARDUCAM OV5642 and raspberry pi 4B ,I want to use the camera to take some photo or video.but I didn`t find any tutorial to do that ,so I use this lib to try

Are you using SPI camera on Raspberry Pi? If yes, please refer to https://github.com/ArduCAM/RaspberryPi/tree/master/SPI_Camera

of cause I try it  before .
but when run the command line sudo ./run_Arducam_Demo
But I can't get any results, and the program won't report an error.

Hi,
For PI4, please execute the following command to update the wiringpi library.

Then you need to make the following changes in bcm283_board_driver.h:

Finally, recompile and execute.

according to your advise，I install the latest version of wiringpi，and change the bcm283_board_driver.h file
But there's a problem.

I connect the ARDUCAM ov5642 and pi4 according tothe picture。

Hi,
Please refer to this connection.

finally，it works！
but，my mobile can not connect to the camera，
I have check the IP and port 。
my mobile status is always connecting...

here is the screenshot

Hi,
Due to our negligence, there is a problem with the latest version code. Please use the code of this version. The part that needs to be modified has been explained in the reply above.
https://github.com/ArduCAM/RaspberryPi/tree/9ebf547e2795f81835d0490a957aaad6e70e6595/SPI_Camera

I am sorry but I really know how to download this version of demo.
can you show me the command?

thanks for your help.
I can use it now .
but I still have a question ,can I use it only by mobile?
can I save photoes on raspberry derecctly?

Hi，
You can save images directly to the Raspberry Pi, but there is no example of directly saving images on our Raspberry Pi. We suggest you refer to the Raspberry Pi code and the Arduino version code for testing.
We are very busy lately, sorry for not being able to write a detailed example for you.

in fact, I got the code by  git clone https://github.com/ArduCAM/Arduino.git
I told you that I capture a photo but the size is zero
now ,I change the connection according to the picture you show ,now I got  a error show as below
SPI interface error!

Hello,
The repository you cloned above is an example of using the SPI camera on the Arduino platform. You can refer to the process of taking pictures and saving images, which cannot be used directly on the Raspberry Pi.
Now that you have run through our reference example on the Raspberry Pi, I suggest you modify it on the run through example.
In addition, the SPI interface error is either a connection error or a problem with the spi driver.


Improve session handling

Sessions can now be retrieved from a backend in a more general manner.
Multi-backend gets back its session field that contains the session
if one was created, removing the interfacing from multi backend with the
drm backend directly. This adds the possibility to use sessions even
without the drm backend.
It additionally fixes the bug that 2 session objects got created when
WLR_BACKENDS were set to "libinput,drm".
To allow vt switching without drm backend (and drm fd) on logind, start
listening to PropertiesChanged signals from dbus and parse the session
"Active" property when no master fd was created (this does not change
current drm backend behaviour in any way).

The main reason for this pr is to allow future session-based backends other than the drm one (like a VkDisplayKHR backend, i did a rather basic implementation here but it depends on the vulkan pr so it may take a while until it hits master (if it will at all...) and i figured it's a good idea submitting these rather independent changes/fixes already). The interface changes make session interaction in general simpler though, see the rootston/keyboard.c change for vt switching. Rootston no longer has to directly interface with the multi backend.
This is my first time really touching dbus/the sdbus library so a review of that part by someone with more experience is appreciated. Note that weston does handle it the same way; although it leaves the decision how/when to (de)activate sessions explicitly to the backend (it seems) and does not decide it implicitly by whether or not a drm device is taken.

Currently on vacation, will come back to this next week.

Should be fixed now.

Thanks!


Spot Size and Clean Twice not showing up with D7

I saw you added features for spot size and clean twice, however I'm not seeing either of these features with my D7. I do have a switch for Spot Clean, but that is it. I only noticed because it cleaned a really small spot compared to when I do it with the Neato app.
Is there something I'm missing in my config? Currently it looks exactly like the example config.
I've completely uninstalled and reinstalled the plugin and it still looks like this screenshot.

Just realized I need a third party home app to adjust this. Could you specify this in the readme?


docs(flux): fix a typo

There was a typo attachStore.

Thanks.

@krasimir so efficient. 👏
BTW, I' working on the Chinese translation of this cookbook which I love.


[WFLY-12865] Avoid circular recursion searching for capability status

We could have capabilities with circular requirements, e.g. an attribute that adds a capability reference from a parent capability to a child capability when the child capability already requires its parent.
In those cases, when the server is in restart-required and the capability to check is not the one that was recorded as restart-required, a stack overflow exception is thrown.
Jira issue: https://issues.redhat.com/browse/WFLY-12865

Core - Full Integration Build 9245 outcome was UNKNOWN using a merge of 38aed9ffd4b85e2cd9ba80360b0132d1755de6fa
Summary: Canceled (Tests passed: 999, ignored: 14; exit code 143 (Step: Build & test full (Maven)) (new)) Build time: 00:21:13

Core - Full Integration Build 9243 outcome was UNKNOWN using a merge of 38aed9ffd4b85e2cd9ba80360b0132d1755de6fa
Summary: Canceled (Tests passed: 1857, ignored: 35; exit code 143 (Step: Build & test full (Maven)) (new)) Build time: 00:21:13

@yersan thanks


ETOS Docker API build fails with 3.9.0 base image

Description
When building ETOS API image based on Python 3.9.0 image, there is a dependency conflict with grpcio/enum (most likely caused by opentelemetry modules).
Additional Context
No response
Logs
No response
Expected Behavior
The ETOS API image shall be able to use the latest available base image of the 3.9 branch
Steps To Reproduce
Build the ETOS API docker with docker build . -t myapi
The version of this project/repo, if applicable
56d94a790204f25db90f3bc72fbbd78ac92c73c8
The version/edition of the Eiffel Protocol used, if applicable
latest

Fix merged.


Cosmos DB links are too vague

Since Cosmos Db is a multi-model database service, it would be much better if Cosmos Db links are linked to the exact DB type/API/offering. For example key/value storage should be linked to Table API documentation or service. So it would be better if the link is https://docs.microsoft.com/en-us/azure/cosmos-db/table-introduction.
Since the page is about data stores and what Azure offers; Table API should be mentioned when talking about key/value store and Cosmos DB main link is way to vague. An alternative may be to share both. Same thing is also valid for some other types...

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

ID: a6a2dd11-b2f0-7821-a877-fc09af4ac967
Version Independent ID: f7537dfd-5472-f0d2-d344-db85fd65bf71
Content: Choose the right data store - Azure Application Architecture Guide
Content Source: docs/guide/technology-choices/data-store-overview.md
Service: architecture-center
Sub-service: reference-architecture
GitHub Login: @MikeWasson
Microsoft Alias: pnp

Thanks for reporting - this issue is under review.  This is a Microsoft Internal DevOps Bug ID AB#

Hi @canuzun Thanks for your feedback. We will review and update as appropriate.

@canuzun PR has been created for this which will address this. Thanks so much for providing the feedback that help us in improving the documentation. For now I will proceed with closure of this and If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Cannot read property 'length' of null

Hi, thanks for the plugin.
Issue description:  I encountered an error when destroing a plugin on a specific breakpoint. An error appears if the window is resized immediately after launch destroy method.
Apparently, the variable slideItems at this point has already been reset, and the forEach is trying to use it.
setTimeout(function() {
  forEach(slideItems, function(el) {
    removeClass(el, 'tns-moving');
  });
}, 300);

Demo link/slider setting:   Example
Tiny-slider version:  2.9.1
Browser name && version:  Google Chrome 71.0.3578.98
OS name && version:   Windows 10

the same for me, after resizing window:
forEach.js:3 Uncaught TypeError: Cannot read property 'length' of null
    at forEach (forEach.js:3)
    at tiny-slider.js:1956
forEach @ forEach.js:3
(anonymous) @ tiny-slider.js:1956
setTimeout (async)
updateGallerySlidePositions @ tiny-slider.js:1955
resizeTasks @ tiny-slider.js:1444
(anonymous) @ tiny-slider.js:1212
requestAnimationFrame (async)
onResize @ tiny-slider.js:1212

This error occurs because the script is trying to create a slider out of an element that likely doesn't exist on your page. I am using this script for the first time and came across this too.
You need to do a check to see if the slider element exists first if you're running the script across your entire site
eg.
if ( $('.feature.slider').length > 0 ) {
    var slider = tns({
    //options here
    });
}

I feel like this is something that should be handled by the script itself though, hopefully it will be updated in the future.

This error occurs because the script is trying to create a slider out of an element that likely doesn't exist on your page. I am using this script for the first time and came across this too.
You need to do a check to see if the slider element exists first if you're running the script across your entire site
eg.
if ( $('.feature.slider').length > 0 ) {
    var slider = tns({
    //options here
    });
}

I feel like this is something that should be handled by the script itself though, hopefully it will be updated in the future.

Seconded. It'd be great if it could handle the slider not being present yet, or being empty.

if u in use react put tns in useEffect like that
useEffect(() => {
tns({
container: '.slider-list',
items: 1,
slideBy: 'page',
autoplay: true,
})
}, [])


932200 Linux RCE evasion with (more) uninitialized bash parameters

This PR enhances the anti-evasion protection for uninitialized variables provided by rule 932200. Following example payloads will additionally be blocked:
c$@at /et$@c/pas$@swd
c$!at /et$!c/pas$!swd
c$*at /et$*c/pas$*swd
c$?at /et$?c/pas$?swd
c$-at /et$-c/pas$-swd
c$_at /et$_c/pas$_swd
c$$at /et$$c/pas$$swd
A full list of special bash parameters can be found in the bash reference
These parameters might be uninitialized - and as a consequence are translated into empty strings by bash - depending on the system and how RCE was achieved. I was able to find all listed parameters uninitialized, with the exception of $# (which is already implemented), $_ and $$, but I recommend to add them as precaution. I expect them to produce much less FPs than some of the already implemented variables such as $1.

Very cool. Thank you @flo405. Would you mind adding a test or two for these enhancements?

thank you @flo405! I've updated the regex101 link with all your new payloads here https://regex101.com/r/V6wrCO/1 would you mind updating the comment section too? thanks!

Meeting decision May https://github.com/coreruleset/coreruleset/issues/2053#issuecomment-831462202:
@theMiddleBlue will add some tests. PR can be merged then.

Hey @theMiddleBlue, did you have time to come up with some tests? If you're quick, we can merge before the meeting.

Hey @theMiddleBlue, did you have time to come up with some tests? If you're quick, we can merge before the meeting.

Meeting decision July: https://github.com/coreruleset/coreruleset/issues/2141#issuecomment-874288876:
@azurit volunteers to write some tests.

sorry I was a little bit unresponsive on this task, I'm going to open a new PR replacing this with tests and comments

replaced by #2151


bourne shell alpha

This branch now contains a ~75% functioning alpha build. Still some work is needed to improve checks over multiple commits and to make more transaction templates. Organization guilds also do not work properly.
Still, I believe it is approaching final state. At this point it is ready for alpha testing by the adventurous.
@cindy-zimmerman @d3brouille

Merging this since it is more complete than what is in master... Lets move forward with one path!


[-] Driver Unload Failed!!

[<] Loading vulnerable driver, Name: GYhcycvwBxNPnkCrYQMwAHuzynRcy
[+] NtLoadDriver Status 0x0
[-] Can't read module headers
[-] No module address to find pattern
[-] Can't read module headers
[-] No module address to find pattern
[-] Can't read module headers
[-] No module address to find pattern
[-] Warning PiDDBLock not found
[-] Failed to ClearPiDDBCacheTable
[<] Unloading vulnerable driver
[+] NtUnloadDriver Status 0xc0000010
[-] Driver Unload Failed!!

Some antivirus or anticheat is blocking the vulnerable driver

gf


Upgrade to qiskit_sphinx_theme 1.12 and Sphinx 7

The 1.12 theme release is focused on stability, including fixing 6 long-standing styling issues. See https://github.com/Qiskit/qiskit_sphinx_theme/releases/tag/1.12.0rc1 for the changelog.
This PR also:

Pins Sphinx to 7. It brings some bug fixes and improvements from Sphinx 5. We pin to 7 to avoid breakage from Sphinx 8's release.
Adds tox -e docs-clean to fix when autodoc has bad state.
Allows an admin to trigger a docs deploy manually in GitHub's UI.

Allows an admin to trigger a docs deploy manually in GitHub's UI.

When you get a chance, a docs deploy would be great. https://qiskit.org/ecosystem/mthree/ has an out-of-date theme.


Bump typescript

Breaking out part of #4516 to isolate test flakiness

Build is failing due to some @web/dev-server plugin types. Perhaps TS update is not possible without updating @web/ stuff too.

I do not understand why this built locally for me. Only failing on CI makes fixing it quite annoying :/


Make graphs in emails link to a filled out graphing page

Make graphs in notifications link to the graph page in bosun with the information for that graph (so it can be tuned)

Have function to construct rule link, so this should be pretty easy.

Implemented this at some point


Nl jest setup

Description
Final version of main FE codebase with Jest
Fixes # (issue)
AWS deployment from Heroku
Type of change
Major, redid Front End
Please delete options that are not relevant.

[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Change Status

[x] Complete, tested, ready to review and merge

How Has This Been Tested?

[x] Tested Locally

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] My changes generate no new warnings
[x] New and existing unit tests pass locally with my changes
[x] There are no merge conflicts

great job


Code doesn't work with modern cuda sdk

Dear Rio,
Can you let me know, if you tried to run gemsfmm code with more or less modern version of cuda sdk, because when I tried to compile the existing code after it "trivial" modifications, I have
Kernel execution failed in file 'gpukernel_p3.cu' in line xxx : invalid device function.
This line corresponds to lines 605 and 606 of your original code:
p2p_kernel<<< grid, block >>>(deviceOffset,devicePosTarget,devicePosSource,deviceAccel);
CUT_CHECK_ERROR("Kernel execution failed");
Thank you in advance!

I have updated the code so that it works with CUDA 11.2.


Relation extension after truncation doesn't work

It is possible that we dropped a relish and then created it again within the same InMemoryLayer.
What is the meaning of drop_lsn then? Now we just don't see the version of the segment after drop.
How should we treat the same scenario in a delta layer?
What if we will drop this relish once more?
I wrote two tests to illustrate the problem:
#568
RUST_BACKTRACE=1 cargo test test_drop_extend is a bit synthetic.
I cannot come up with a realistic workload where we create relish shortly after it was dropped. In theory, it may happen in the case of SLRU wraparound.  And I think it is a reason why #403 fails.
RUST_BACKTRACE=1 cargo test test_truncate_extend simulates more realistic example of workload.

@alubennikova Why is it okay that we don't do anything with relmapper.c in zenithdb? Have you seen any failures involving VACUUM FULL or CLUSTER specifically?

@alubennikova Why is it okay that we don't do anything with relmapper.c in zenithdb? Have you seen any failures involving VACUUM FULL or CLUSTER specifically?

We put 'pg_filenode.map' into postgres data directory before compute node start. See RelishTag::FileNodeMap in pageserver -- we intercept WAL that changes FileNodeMap and maintain it up to date.


Add racket support

I think it works now.
I also found a problem. If here_string not in atom_nodes,
It will show a weird result:

What happened here? Does it considered a parser error?

I just fixed another bug in here_string. It's done now.
The problem above still exists.

This is a limitation of difftastic today: it doesn't handle string interpolation well. Difftastic assumes that all parts of the syntax tree have their own position, whereas things like "foo $bar baz" tend to just have (string_lit outer_position (interpolated inner_position '$baz')). Treating string literals like heredocs as atoms is the right thing to do.
Could you add syntax highlighting too? I see https://github.com/6cdh/tree-sitter-racket/blob/main/queries/highlights.scm exists.

It's completed now.

Merged in 5e659e2d9862d10499850a762212fcd94ba84181, thanks!


Corrected the isogram string project .

Closes: #667

Title : Correction in Isogram String project .
Name: : Meet Arora
Idenitfy yourself: : GSSOC

Describe the add-ons or changes you've made 📃
Corrected the description of the isogram string project so that now only those strings are isogram in which no letter is repeated more than once .
How Has This Been Tested? ⚙
I have tested this on my own .
Screenshots 📷

Type of change ☑

What sort of change have you made:

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist: ☑

[x] My code follows the Contributing Guidelines & Code of Conduct of this project.
[x] This PR does not contain plagiarized content.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly wherever it was hard to understand.
[x] My changes generate no new warnings.

Note to reviewers 📄

@Avdhesh-Varshney done


Update CentOS and RHEL image aliases to v20170829.

Latest versions identified via:
gcloud compute images list | egrep '^(centos|rhel)-6'

This is a short-term manual fix for issue #136, which should get a proper implementation.

Sorry, I forgot that the tests verify image version numbers, so I have to update both the config and the tests the same way.

This is quite strange; the JDK 8 tests passed; the JDK 7 failed with:
$ mvn install -DskipTests=true -Dmaven.javadoc.skip=true -B -V
The JAVA_HOME environment variable is not defined correctly
This environment variable is needed to run this program
NB: JAVA_HOME should point to a JDK not a JRE

The command "eval mvn install -DskipTests=true -Dmaven.javadoc.skip=true -B -V " failed. Retrying, 2 of 3.

The JAVA_HOME environment variable is not defined correctly
This environment variable is needed to run this program
NB: JAVA_HOME should point to a JDK not a JRE

The command "eval mvn install -DskipTests=true -Dmaven.javadoc.skip=true -B -V " failed. Retrying, 3 of 3.

The JAVA_HOME environment variable is not defined correctly
This environment variable is needed to run this program
NB: JAVA_HOME should point to a JDK not a JRE

The command "eval mvn install -DskipTests=true -Dmaven.javadoc.skip=true -B -V " failed 3 times.

It's unclear why this is happening, given that the Travis CI config is identical for the two JDKs:
language: java

jdk:
  - oraclejdk8
  - oraclejdk7

script:
  - mvn clean install

Looks like JDK 8 installs well:
$ jdk_switcher use oraclejdk8
Switching to Oracle JDK8 (java-8-oracle), JAVA_HOME will be set to /usr/lib/jvm/java-8-oracle

while JDK 7 does not:
$ jdk_switcher use oraclejdk7
Switching to Oracle JDK7 (java-7-oracle), JAVA_HOME will be set to /usr/lib/jvm/java-7-oracle
update-java-alternatives: directory does not exist: /usr/lib/jvm/java-7-oracle

It's actually unclear whether Travis CI still supports Oracle JDK 7, because the top-level docs say:

The Travis CI environment provides Oracle JDK 8 (default), Oracle JDK 9, OpenJDK 6, OpenJDK 7, Gradle 2.0, Maven 3.2 and Ant 1.8, and has sensible defaults for projects that use Gradle, Maven or Ant.

Note the curious mention of "Oracle JDK 7", only "OpenJDK 7", but below, it does mention it:

Travis CI provides OpenJDK 6, OpenJDK 7, Oracle JDK 7, and Oracle JDK 8. Sun JDK 6 is not provided, because it is EOL as of November 2012. OpenJDK 8 is available on our Trusty images, specify dist: trusty to make use of it.

I'll try updating the Travis config to add dist: trusty and see if that fixes it, though it really should have no effect on the JDK availability.

And it's still broken, for the same reason.
Turns out, Travis CI can no longer support Oracle JDK 7 as per this comment:

oraclejdk7 is unfortunately no longer available in Trusty after our latest updates, since we can no longer support oraclejdk7 on our recent build images due to Oracle's withdrawal. See: http://www.webupd8.org/2017/06/why-oracle-java-7-and-6-installers-no.html
If this is an option for you, I'd recommend switching to jdk: openjdk7, which is available in the new Trusty images.

Per the cited webpage which quotes the Oracle downloads page:

Updates for Java SE 7 released after April 2015, and updates for Java SE 6 released after April 2013 are only available to Oracle Customers through My Oracle Support (requires support login).
Java SE Advanced offers users commercial features, access to critical bug fixes, security fixes, and general maintenance.

It seems like no one will be using Oracle JDK 7 unless they're an Oracle customer, so the choices are:

replace oraclejdk7 with openjdk7
remove oraclejdk7 and leave just oraclejdk8 as the only option

The choice as to which to take here is up to Cloudera, since Cloudera is ultimately shipping this software and dictating customer requirements, so it's up to @jadair-cloudera, @bhavanki, et al. to choose the path forward. Let me know which you would prefer me to do, and I'll do that.

Filed this issue with Travis CI to ask them to update the docs accordingly.

We will discuss.

@jadair-cloudera — any updates?

Sorry, we discussed right away but I forgot to comment here. We think openjdk7 should be fine.

@jadair-cloudera — updated to use OpenJDK 7 instead of Oracle JDK 7, please review this PR once the Travis CI build and tests are green.


Check if env is actually an Env object before attempting to add pointers.

pointers.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#1031  👈
#1029
main

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

Join @rohinb2 and the rest of your teammates on  Graphite


Update about minikube: multi-node clusters are now supported

This is a Feature Request

The following information is outdated:
https://github.com/kubernetes/website/blob/ed460eb88886e826abf61792377e596b96cf5ccf/content/ja/docs/setup/learning-environment/minikube.md?plain=1#L504-L506
Therefore, I suggest removing the relevant section.
Why is this needed
The information is outdated
Comments

URL: https://kubernetes.io/ja/docs/setup/learning-environment/minikube/#既知の問題

FYI:

https://github.com/kubernetes/minikube/issues/94
https://github.com/kubernetes/minikube/pull/6787
https://minikube.sigs.k8s.io/docs/tutorials/multi_node/

/language ja
/assign

Also, I think it would be better to change the following description:
https://github.com/kubernetes/website/blob/618bc1b15c7705c910231adee98653b830dcdc90/content/en/docs/reference/glossary/minikube.md?plain=1#L18
like this:
https://github.com/kubernetes/website/blob/618bc1b15c7705c910231adee98653b830dcdc90/content/en/docs/tasks/tools/_index.md?plain=1#L44
/language en

Thanks!!
Please separate issues into English and Japanese.
Localization in Japanese and other languages should follow the English version.

@Okabe-Junya
I'm so sorry for the delay.
I created the [en] issue #49116 and changed this issue to [ja].


Added rod cutting problem in dynamic programming

Added rod cutting problem in dynamic programming

@abranhe  added rod cutting problem in C++


Configurable JsonFactory and HttpTransport

Admin Java SDK uses a default JsonFactory and a default HttpTransport everywhere. But users should be allowed to configure these where appropriate. For instance, when the SDK is deployed behind a proxy, it becomes unable to make any HTTP requests, since the default HttpTransport is setup without proxy support. If this was configurable, users can init a transport with proxy support and pass it in.
I believe this needs to happen in 2 places:

Expose the credential factory methods that accept JsonFactory and HttpTransport arguments. These methods already exist, but are package-protected.
Allow setting a JsonFactory and HttpTransport via FirebaseOptions. These can be plumbed into everywhere else in the SDK.

It is possible to have the credentials also take the parameters from FirebaseOptions, but that's going to be a breaking change, and we probably can live without it.

Released in v5.2.0


fix: dropdown now stays within the viewport

SUMMARY:
CSS change to ensure dropdown menu stays within the viewport
GITHUB ISSUE (Open Source Contributors)
JIRA TASK (Eightfold Employees Only):
ENG-39856
CHANGE TYPE:

[x] Bugfix Pull Request
[ ] Feature Pull Request

TEST COVERAGE:

[ ] Tests for this change already exist
[ ] I have added unittests for this change

TEST PLAN:
Ensure https://eightfoldai.github.io/octuple.github.io/?path=/story/dropdown--dropdown-button continues to work as expected.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@f97a7df). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #512   +/-   ##
=======================================
  Coverage        ?   84.76%
=======================================
  Files           ?      785
  Lines           ?    16451
  Branches        ?     5696
=======================================
  Hits            ?    13944
  Misses          ?     2441
  Partials        ?       66

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.


[iOS] Account Settings Cleanup - Support / FAQ / Email (2U)

Goal
This project involves adding dedicated support page, FAQ, Email and viewing Terms of Service / Privacy Policies inside application
Slack: #wg-product-mobile
### Tasks
- [ ] https://github.com/openedx/openedx-app-ios/issues/118
- [ ] https://github.com/openedx/openedx-app-ios/issues/119
- [ ] https://github.com/openedx/openedx-app-ios/issues/120

This effort would benefit from some UX definition and clarity - we definitely need to have the account settings page be more flexible and include out of the box additional configurable links / references for student support and help.

@moiz994 could we specify which exactly fields/areas should we have in this View?
Should we follow current edX app? All these fields are required fr openEdX too or we need hide something with Feature Flags?
Like some list or something like this.
Thank you
cc @k1rill

Hey Anton, good question. We should be following the prod edX app for our case but for open edX the following are not required:

Cookie Policy
Do Not Sell my Personal Information
Delete my Account

Please see the below screenshot of the current prod app profile screen for context:

Also, quick question, are we currently just implementing the design for the screen or also looking into the functionality of the buttons on this screen?
@marcotuts I think we need to align here as well on the design and scope of the profile tab.

Delete account is now inside Edit Account screen in the open edX app

oh, yes! Thanks for sharing this. I think there was some compliance issue that needed us to bring it out in the front for some reason but I can not recall exactly. I think product teams need to sync up on this. Let's keep it as it is for now. We can create separate tickets for compliance considerations later.

@k1rill can you share a video or screenshot of what the current implementation is that awaiting PR approval?


[Windows] [9.1.1] MS Defender detects Trojan:Win32/Wacatac.B!ml

Version: 9.1.1
Describe the issue
It is most likely a false positive but Microsoft Defender detects a Trojan:Win32/Wacatac.B!ml in latest version of the CLI v9.1.1

@nikosatwork Thanks for reporting. We took a look and indeed some machine language generated false positives. Both VirusTotal and FileScan confirm. We did find a few improvements in our build process we'll implement, as well as reporting the FP to the vendors that flagged it as such.


Effects imported by multiple lazy loaded modules loaded multiple times

Now that the https://github.com/angular/angular/issues/12869 is fixed, providers no longer always instantiate twice with lazy loading.
However, there is another related issue that still exists, I believe. If you have an effect that is used by more than one lazy loaded endpoint, it will be loaded multiple times if you go from route to route between them.
For example, we have a tagging system, which has its own nxrx/effects. The tags are used by the images, the files, and the videos systems (each lazy loaded).
Ideally, the tagging effects (with dependencies) would only be loaded for these particular pages. However, if they are loaded in the imports section of each lazy-loaded module, they are instantiated each time the user uses a routerLink to jump from one type to another. That means that the effects are dispatched more than once if the user has visited different sorts of pages.
Of course, we could solve this by importing any effects that are needed for multiple lazy loaded modules in the root app module, or in some module imported by the app module. But that is obviously not ideal from a lazy loading perspective, since we don't need tags on all pages, just a small subset of them.
I'm not sure of the best solution for this--here are a couple of naive suggestions, though. One option, though, would be to change EffectsModule.run so that it skips Effects classes that have been previously imported. Another option would be to somehow limit the effects that run on a lazy-loaded route to those that were imported and run by the lazy loaded module or the AppModule. That would mean that if a user starts on a file page, and imports the Tag effects, then switches to the image page, the image page module would also import its own instance of the Tag effects, but the Tag effects previously imported in "files" would be inactive.
I don't know enough about Angular internals or how ngrx/effects works to know if either of these are workable solutions.

Just came across repo where effects are implemented in lazy modules FabienDehopre/ngrx-lazy-load. IMO approach is little sketchy because it uses global effectsAdded for caching effect subscriptions. But that's the best solution I've found so far.

I am also experiencing this issue, I don't think it's limited to effects imported by multiple lazy loaded modules?
 What you describe happened with Angular 2, but has been fixed in the newest version.

@karptonite Yes, still on Angular 2. Will try upgrading and report back 👍

Our attempt at getting around this problem:
Every module that has effects uses a new helper to provide the effects:
@NgModule({
    imports: [
        SomeImport
    ]
})
export class FeatureModule {
    public static withEffects(): ModuleWithProviders {
        return {
            ngModule: FeatureModule,
            providers: [
                addEffectOnce(MyCoolEffects)
            ]
        };
    }
}

Then when importing this module, say into AppModule, or into a lazy-loaded parent module, do:
imports: [
    FeatureModule.withEffects()
]

The magic happens inside this little helper, add-effect-once.ts
import { EffectsModule } from '@ngrx/effects';

const effects = new Map<any, any>();

export function addEffectOnce (effect) {
    if (effects.has(effect)) {
        return [];
    } else {
        effects.set(effect, true);
        return EffectsModule.run(effect).providers;
    }
}

Pros:
Pretty simple to understand what's going on, seems to work pretty well
Cons:
Requires you to remember to call withEffects() on your modules, and structure your code a bit differently
Doesn't currently work with AoT, getting the classic Error encountered resolving symbol values statically., function calls are not supported.

@elwynelwyn It seems that your solution won't work with lazy loaded module since you have to call the withEffects() method when importing the module.

Yeah, in our app we are lazy loading a "parent" module which is a module for an area / page of the app, and that "parent" module pulls in the individual feature modules which call withEffects().
But being able to have effects run on the lazy loaded module itself would certainly be much more flexible.

OK previous workaround has not been working for us ^^ so I've made a PR that builds on @FabienDehopre's repo.
@MikeRyanDev / @brandonroberts interested to hear your thoughts, and let me know if you'd rather I make the PR against the new monorepo

For those following along on this issue, the PR is now over at the new monorepo.
https://github.com/ngrx/platform/pull/23

Please check this against NgRx v4, and if it’s still an issue, please reopen on https://github.com/ngrx/platform. Thanks!


ENH:optimize: Enforce minimum bounds in armijo

Reference issue
Closes #17511
What does this implement/fix?

Additional information

Thanks @gabrielfougeron for the PR and please excuse the very long response time. Could you add the example of your original issue as a unit test? Your fix is small but the actual issue is not reproducible for reviewers at the moment.

Hey @dschmitz89, thanks for the follow up.
I won't be adding a mwe for the issue for a few reasons:

Firstly, it's litterally been a year and a half since I wrote the PR.
The function being minimized in the example is not a one liner, but is rather complicated and I don't feel like investing time on replicating the bug on a simpler function.
The issue does exist and the output alone proves it.
The logic of the code is simple enough to be worked out in your head.
Even if there actually were no issue, the fix I provided is so small that it has virtually no effect on performance, readability or maintainability, and is litterally just a translation of the existing doc. This peace of mind is certainly worth two lines of code.

Cheers,

While I understand the frustration about the long waiting period, I beg to differ @gabrielfougeron .
There is no way for a reviewer to print the same output as your original issue without the code. How could I confirm that this update does what it is supposed to do? I trust that you are a domain expert (likely more than me) but even if I dig through the code long enough so that I am convinced that this PR is correct, we all know how easy it is to introduce small bugs. That is why we put such a strong emphasis on tests. Under testing is a constant source of churn for old projects such as scipy. This strict review process might appear one sided but it has proven to be necessary.
Are you sure that there is no simple objective function to reproduce the issue?

Hum good question. My particular example was an action extremizer in celestial mechanics using JFNK search.
This being said, I guess virtually any function (even linear) would work as long as the Jacobian is poorly approximated / the linear solves have high error. if my sole goal were to find an example quicky, I'd brute force random linear solves with approximate Krylov solvers with different stating positions and checks. Not very elegant but it could do the trick.

Hey @dschmitz89,
I followed my own advice and came up with the following mwe that triggers the bug.

import numpy as np
import scipy
import scipy.optimize
import functools

def main():

    seed = 66
    print()
    print(f'{seed = }')
    np.random.seed(seed)

    n = 10
    A = scipy.linalg.hilbert(n)
    b = np.random.random(n)

    def res(x,A,b):
        return np.matmul(A,x) - b

    f = functools.partial(res, A=A, b=b)

    verbose = True
    # verbose = False

    eps = 1e-5

    try:
        res = scipy.optimize.newton_krylov(f, b, f_tol=eps, verbose=verbose, maxiter=5)

    except scipy.optimize._nonlin.NoConvergence:
        pass

if __name__ == "__main__":
    main()

On my machine, the output without vs with the PR looks like this (see how the output differs from iteration number 4 onwards)

(base) scipy_test$ python armijo_bug.py

seed = 66
0:  |F(x)| = 0.754572; step 1
1:  |F(x)| = 0.26888; step 1
2:  |F(x)| = 0.238358; step 0.147441
3:  |F(x)| = 0.240504; step 0.00913213
4:  |F(x)| = 0.237482; step 0.054765
(base) scipy_test$ conda activate scipy-dev
(scipy-dev) scipy_test$ python armijo_bug.py

seed = 66
0:  |F(x)| = 0.754572; step 1
1:  |F(x)| = 0.26888; step 1
2:  |F(x)| = 0.238358; step 0.147441
3:  |F(x)| = 0.243392; step 0.0214219
4:  |F(x)| = 0.240874; step 0.00511968
(scipy-dev) scipy_test$

It does reproduce the example and highlights the fix, but I would not consider it a test though and I'm reluctant to add it to scipy's test suite.

@andyfaff @dschmitz89 any thoughts?

I'm not a domain expert in this either. However, the code looks and smells correct. I'm inclined to merge.

Although the patch is small, the discussion above is a mixed bag of "let's just try it" and confusion, so I'm not quite convinced I should merge just hours before branching for the 1.15.0 release.
If the "reproducer" of sorts above presents a clear improvement to domain experts, we could perhaps try something like that in the testsuite by using pytest capsys to probe the verbose output that gets printed, but at my level of understanding the printed output isn't decisive, perhaps because I don't understand it well enough.

I agree with https://github.com/scipy/scipy/pull/17565#discussion_r1866018323 . I'd like to check where overrides the default amin=0 to understand where this will have an impact. I don't have access to a computer but will circle back to it when I do


:memo: :lipstick: cursors look better if they indicate the direction of movement

Visual :

Great project :rose: :+1:

Thanks! This is why the CSS is completely customizable, there is no project default. I'm going to close this, but I appreciate the pull request!

Surely these are more sensible defaults, though.

You're absolutely right.

You're absolutely right.

Cheers :rose:

Released in 1.0.6. Thanks!


feat: use security-group module instead of resource

what

use security-group module instead of resource
update tests

why

more flexible  than current implementation
bring configuration of security group/rules to one standard

references

CPCO-409

/test all

/test all

/test all

/test all

/test all

Please also add release notes similar to https://github.com/cloudposse/terraform-aws-efs/releases/tag/0.31.0 prior to merging.

I was about to open a similar PR, when tfsec complained about the open security groups. Any ETA on when this may be merged in?

@tuxtek at the moment the security group module is under some refactoring and before it's rolled out it needs to be tested. No eta at the moment but this module is on our list to update.
To get unblocked for now, please feel free to fork the module, update it, and use the fork until we complete the refactoring.

@tuxtek on second thought, please feel free to submit a pr for the tfsec warnings. We can review that while we wait on a refactored pr for the security group module.


Go 1.15.2

This builds on top of PR #751 to avoid code conflicts and updates to Go 1.15.2 without changing how certificates are handled, i.e. the changed default in the Go runtime is reset to what it was before the update.

Updated PR:

operator itself also needs GODEBUG
lz4 installed in build container - that was broken in "devel" by the switch to Debian

lz4 installed in build container - that was broken in "devel" by the switch to Debian

That's... hogwash! lz4 is needed on the Jenkins workers. We have an apt-get install liblz4-tool in the worker's install script in Jenkins for that. What must have happened in the failed job is that the command failed and the worker came up anyway - a known issue of our CI.

@avalluri : this seems to be ready.


Fix pyenv slow issue

Based on rbenv performance improvements commit.
@gabetax 's improvement - https://github.com/robbyrussell/oh-my-zsh/commit/daedfcab510b01d566a2dff6bc94a9e91a563a18#diff-3538c2cbfa22922156729920217acdee
@sheerun 's improvement - https://github.com/robbyrussell/oh-my-zsh/commit/0ed1adeaadfc03a5abb767df5ad76a1ddd2d7c69
This fix improve zsh startup time when pyenv plugin is enabled.

The new version doesn't use homebrew so this PR is obsolete. Thanks for your effort though!

@mcornella thank you! I'm happy with @cyphus 's work merged.


erdiko-slim prototype

formerly ER-162 on Trello

initial prototype is done and in a private repo.  Code has been moved in to erdiko/erdiko branch base:erdiko2


SyntaxError: Unexpected token 'export'

After updating to 2.0.0:
export function trimTrailingLines(value) {
^^^^^^

SyntaxError: Unexpected token 'export'

Yup, as mentioned in the v2 changelog, this package is now ESM only.
I’ve just added a link with more information to the readme too!

Oops sorry, I missed it. Thanks!


Data-entry app repeats table name

Going to https://synapse-dev.isrd.isi.edu/chaise/data-entry/#1/Zebrafish:Subject shows two header fields, both generated from the table name:

New Subject
SUBJECT
...

With the completion of #330, the second header has been replaced with a notice about asterisks indicating required fields.


nomination-pools fix

Fixed the error that occurred during pallet-nomination-pools compilation.
The error message was: not a member of trait `sp_staking::StakingInterface, this referred to the nominations function.

@bkchr @kianenigma Can we merge this if the PR is fine?

bot merge


HTTP: lift header max size security

@diceytroop sbt publishLocal on gatling and then gatling-highcharts.

thanks so much! i'm running into this error mid-compilation -- i realize this really isn't what these issue threads are for, though, so if there's another place where you'd suggest i post newb questions like this please let me know.
[info]  published ivy to C:\Users\tfitzgerald\.ivy2\local\io.gatling\gatling-core\3.1.0-SNAPSHOT\ivys\ivy.xml
[error] java.lang.IllegalStateException: Ivy file not found in cache for io.gatling#gatling-bundle;3.1.0-SNAPSHOT!

Please ask on our community mailing list.

Hi, please can I understand when this version will be released? Thanks

Beginning of next week. Next time, please use the community mailing list for questions.


recyclarr: refactor - build from source

Description of changes
Build from source, using changes suggested from @aldoborrero (https://github.com/NixOS/nixpkgs/commit/fb9b1b98e379c592e9b25b5485937bdbfa0d5666).
Requires patching out a Git version check in source (as the fetchFromGithub helper doesn't pull entire git history and .git folder).
update.sh changed to update source, rather than binary. Package bumped to 6.0.2.

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/3723

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/4381


WFLY-6405 WeldDeployment.getBeanDeploymentArchive method is synchronized

Linux with security manager Build 4453 outcome was FAILURE using a merge of 02b7492f911820141d51847a88019f18b3649d30
Summary: Tests passed: 743, ignored: 93; exit code 1 (new) Build time: 00:03:57

Linux Build 9651 outcome was FAILURE using a merge of 02b7492f911820141d51847a88019f18b3649d30
Summary: Tests passed: 743, ignored: 93; exit code 1 (new) Build time: 00:04:18

Windows Build 4675 outcome was FAILURE using a merge of 02b7492f911820141d51847a88019f18b3649d30
Summary: Tests passed: 743, ignored: 93; exit code 1 (new) Build time: 00:05:55

Linux Build 9652 outcome was FAILURE using a merge of b7baaf4fb4438f51ed15d9c596f9bf2a70edf090
Summary: Tests passed: 743, ignored: 93; exit code 1 Build time: 00:03:53

Linux with security manager Build 4454 outcome was FAILURE using a merge of b7baaf4fb4438f51ed15d9c596f9bf2a70edf090
Summary: Tests passed: 743, ignored: 93; exit code 1 Build time: 00:03:56

Windows Build 4676 outcome was FAILURE using a merge of b7baaf4fb4438f51ed15d9c596f9bf2a70edf090
Summary: Tests passed: 743, ignored: 93; exit code 1 Build time: 00:05:16


Update cs.json

Completed and improved Czech translation.

The translation is still a bit clumsy because of the following logic:
https://github.com/MrBartusek/MeteoalarmCard/blob/31f419d37b7e3a8af198b6bf333331b44f57d9b7/src/events-praser.ts#L196-L203
It uses the same event name as a standalone text and as a part of a sentence. Having them separated could result in a much nicer translation.

Before this PR it is: Povodeň / Žluté povodeň varování
This PR workarounds lack of declination as: Povodeň / Žluté varování: povodeň
Even better would be: Povodeň / Žluté varování před povodní

@martin131 Can you take a look at that?

The translation is still a bit clumsy because of the following logic:
https://github.com/MrBartusek/MeteoalarmCard/blob/31f419d37b7e3a8af198b6bf333331b44f57d9b7/src/events-praser.ts#L196-L203
It uses the same event name as a standalone text and as a part of a sentence. Having them separated could result in a much nicer translation.

Before this PR it is: Povodeň / Žluté povodeň varování
This PR workarounds lack of declination as: Povodeň / Žluté varování: povodeň
Even better would be: Povodeň / Žluté varování před povodní

Quite honestly, I do not speak Czech and I've no idea how to proceed here. I think we can leave this as is for the time being unless anybody would want to provide PR for that.

I think what this PR is good enough, it's just a note that the code could be improved to better support localization.

Thank you!


Several issues related to List

Create a List, then create a Map and add it to the List, any modification to the Map's property will then throw an exception. If I change to put a Map as a property of another Map, then no problem.

The indexOf() method of List always returns -1.

This not necessarily an issue but a suggestion - currently no insert() method available to List, I'll have to clear all and re-add all items, so it would be good to have this method.

Thank you for reporting this issue. I am working in a fix to be delivered
asap.
2017-11-18 5:14 GMT+01:00 ngwaitat<EMAIL_ADDRESS>

Create a List, then create a Map and add it to the List, any
modification to the Map's property will then throw an exception. If I
change to put a Map as a property of another Map, then no problem.
2.
The indexOf() method of List always returns -1.
3.
This not necessarily an issue but a suggestion - currently no insert()
method available to List, I'll have to clear all and re-add all items, so
it would be good to have this method.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/P2Pvalue/swellrt/issues/224, or mute the thread
https://github.com/notifications/unsubscribe-auth/AEP6DyjJbRd3dssBykhR4qwR6j8rJOczks5s3lnCgaJpZM4Qi3qU
.

Great, thanks. :+1:

Hi @ngwaitat, after looking into your issues, I couldn't reproduce any of them. I am not sure if there was some misunderstood with the provided documentation.  In any case, your issues have gave me a chance to review the api again and fix some minor issues. Thank you very much, and please contact me for any other question or suggestion.
Please, check how you have created maps and list. There is a big difference whether you create a Swell map or list with create() methods:
obj.node('list').add(swell.Map.create());
obj.node('list').add(swell.List.create());

or you creates Javascript objects and arrays:
obj.node('list').add({ a: "x", b: "y" });
obj.node('list').add([ a, b, c]);

in the second form, SwellRT handles the JS map or array as a whole object, storing it as JSON internally. You can't make changes to inner properties nor listen for changes. You have to replace the whole object in order to do an update.
However, in order to initialize Swell maps and lists with Javascript syntax, create() method has an extended syntax:
obj.node('list').add(swell.Map.create({ a: "x", b: "y" }));
obj.node('list').add(swell.List.create([ a, b, c]));

Create a List, then create a Map and add it to the List, any modification to the Map's property will then throw an exception. If I change to put a Map as a property of another Map, then no problem.

See some examples:
// run this with ./gradlew serverlessRun

var service = swell.runtime.get();
var obj = service.open("o1");

obj.set("list", swell.List.create());
obj.node("list").add(swell.Map.create());
obj.node("list.0").set("p1", "value 1");
obj.node("list.0").set("p2", 2);
obj.node("list.0").set("p3", true);

obj.node("list").add("x");
obj.node("list").add("y");

// modify properties of the map inside the list
// update map property
obj.set("list.0.p2", 20); // same as -> obj.node("list").pick(0).set("p2", 20);
// remove map property
obj.delete("list.0.p2");

// remove list item
obj.node("list").remove(2); // remove 'y', same as -> obj.delete("list.2");

The indexOf() method of List always returns -1.

indexOf only works for nodes:
var n2 = obj.node("list.2");
obj.node("list").indexOf(n2); // returns 2

This not necessarily an issue but a suggestion - currently no insert() method available to List, I'll have to clear all and re-add all items, so it would be good to have this method.

Three methods are provided to add elements to a swell list push(path, value) / add(value) / addAt(value, index)

Thanks Pablo. When trying to repeat the list/map problem, I found a new bug, this is the complete code to repeat the problem:
		var createLoginInfo = function() {
			return {
				id: swellrt.Constants.ANONYMOUS_USER_ID,
				password: ''
			};
		};

		var createOpenObjInfo = function() {
			return {
				// Increate id to test a new object.
				id: '001'
			};
		};

		var createMap = function() {
			var result = swellrt.Map.create();

			result.set('name', 'm1');

			return result;
		};

		var createList = function() {
			var result = swellrt.List.create();

			result.add(createMap());

			return result;
		};

		var objOpened = function(obj) {
			var l1;
			var m1;

			if (obj.get('inited') !== true) {
				obj.set('inited', true);

				obj.setPublic(true);

				l1 = createList();
				obj.set('l1', 11);

				console.log(l1.pick(0).get('name'));

			} else {
				l1 = obj.node('l1');
				console.log(l1.pick(0).get('name'));
			}
		};

		var serviceLoggedIn = function(service) {
			var openobjInfo = createOpenobjInfo();

			service.open(openobjInfo)
				.then(obj => {
					objOpened(obj);
				})
				.catch(error => {
					alert(error);
				});
		};

		swellrt.onReady(service => {
			console.log
			var loginInfo = createLoginInfo();

			service.login(loginInfo)
				.then(profile => {
					serviceLoggedIn(service);
				})
				.catch(error => {
					alert(error);
				});
		});

The problem is, when I load the saved object, the call to pick() will fail since there is no pick() method for the loaded list l1. Any idea why the loaded l1 is not the same list structure as it was created? Thanks.

Obviously that is a bug, but I couldn't reproduce it in my latest version.
Which version are you using?
2017-11-28 11:12 GMT+01:00 ngwaitat<EMAIL_ADDRESS>
Thanks Pablo. When trying to repeat the list/map problem, I found a new
bug, this is the complete code to repeat the problem:
  var createLoginInfo = function() {
  	return {
  		id: swellrt.Constants.ANONYMOUS_USER_ID,
  		password: ''
  	};
  };

  var createOpenObjInfo = function() {
  	return {
  		// Increate id to test a new object.
  		id: '001'
  	};
  };

  var createMap = function() {
  	var result = swellrt.Map.create();

  	result.set('name', 'm1');

  	return result;
  };

  var createList = function() {
  	var result = swellrt.List.create();

  	result.add(createMap());

  	return result;
  };

  var objOpened = function(obj) {
  	var l1;
  	var m1;

  	if (obj.get('inited') !== true) {
  		obj.set('inited', true);

  		obj.setPublic(true);

  		l1 = createList();
  		obj.set('l1', 11);

  		console.log(l1.pick(0).get('name'));

  	} else {
  		l1 = obj.node('l1');
  		console.log(l1.pick(0).get('name'));
  	}
  };

  var serviceLoggedIn = function(service) {
  	var openobjInfo = createOpenobjInfo();

  	service.open(openobjInfo)
  		.then(obj => {
  			objOpened(obj);
  		})
  		.catch(error => {
  			alert(error);
  		});
  };

  swellrt.onReady(service => {
  	console.log
  	var loginInfo = createLoginInfo();

  	service.login(loginInfo)
  		.then(profile => {
  			serviceLoggedIn(service);
  		})
  		.catch(error => {
  			alert(error);
  		});
  });

The problem is, when I load the saved object, the call to pick() will fail
since there is no pick() method for the loaded list l1. Any idea why the
loaded l1 is not the same list structure as it was created? Thanks.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/P2Pvalue/swellrt/issues/224#issuecomment-347475728,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEP6D1wq9sGQztpBCrSpJomiW0eqbQbZks5s69yXgaJpZM4Qi3qU
.

It is 1.0.3-beta.

Let me check again but I used that version this morning.
El 4 dic. 2017 16:27, ngwaitat<EMAIL_ADDRESS>escribió:
It is 1.0.3-beta.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.


Implement Spek JVM runtime

Implementation is pure kotlin and it does not depend on JUnit 5. This is still a work in progress, just want you guys to do the initial review.  Up next is implementing our own gradle/maven plugin.
All previous tests have been ported to the new runtime and all are passing.

Don't merge this please, this will be the base for next major version of Spek.

@artem-zinnatullin @hhariri @ming13 can you checkout 51ad7c8? Most of spek's engine logic is extracted to SpekJvmRuntime and SpekRuntime, junit 5 here is acting like a wrapper.

I still have a few changes locally that fixes several issues with JUnit Platform.

Latest commits fixes some issue w/ report generation, cleanup of changes that should be in another PR and the requested changes. Currently I've added a module for the new implementation of the JUnit TestEngine, shall I remove the old one and replace it with the new one?

I've changed base branch to 2.x, now we really can do breaking changes :)
@ming13, @hhariri PTAL

shall I remove the old one and replace it with the new one?

Let's do it separately so we could review it faster :)

Bumping this up. @artem-zinnatullin


Not compatible with tf.float16

System information

OS Platform - Windows10 (64-bit OS and x64-based processor):
TensorFlow installed from Anaconda:
TensorFlow version - 1.11.0:
Python version - 3.6.5:

float16 not in the list of allowed values.
Command:
a = tf.placeholder(tf.float16, shape=[2, 2])
gen_linalg_ops.qr(a)

TypeError                                 Traceback (most recent call last)
 in
1 a = tf.placeholder(tf.float16, shape=[2, 2])
----> 2 gen_linalg_ops.qr(a)
G:\Anaconda3\envs\tensorflow\lib\site-packages\tensorflow\python\ops\gen_linalg_ops.py in qr(input, full_matrices, name)
1486     full_matrices = _execute.make_bool(full_matrices, "full_matrices")
1487     _, _, _op = _op_def_lib._apply_op_helper(
-> 1488         "Qr", input=input, full_matrices=full_matrices, name=name)
1489     _result = _op.outputs[:]
1490     _inputs_flat = _op.inputs
G:\Anaconda3\envs\tensorflow\lib\site-packages\tensorflow\python\framework\op_def_library.py in _apply_op_helper(self, op_type_name, name, **keywords)
607               _SatisfiesTypeConstraint(base_type,
608                                        _Attr(op_def, input_arg.type_attr),
--> 609                                        param_name=input_name)
610             attrs[input_arg.type_attr] = attr_value
611             inferred_from[input_arg.type_attr] = input_name
G:\Anaconda3\envs\tensorflow\lib\site-packages\tensorflow\python\framework\op_def_library.py in _SatisfiesTypeConstraint(dtype, attr_def, param_name)
58           "allowed values: %s" %
59           (param_name, dtypes.as_dtype(dtype).name,
---> 60            ", ".join(dtypes.as_dtype(x).name for x in allowed_list)))
61
62
TypeError: Value passed to parameter 'input' has DataType float16 not in list of allowed values: float64, float32, complex64, complex128

Added the PR #23841 for the fix.

Nagging Assignee @harshini-gadige: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

@tensorflowbutler I don't have permission to update the label. It would be great if you could do that!

@Shwetago  Awaiting for the PR to get merged.

Okay. Could you please explain what are the changes you have made? Is it working? @harshini-gadige


Invocations of C++ compiler by Lake tests don't pass -isysroot on macOS

Prerequisites

[X] Check that your issue is not already filed:
https://github.com/leanprover/lean4/issues

Description
When C++ files are built as part of Lake tests, they do not pass -isysroot to Clang, which causes Clang to be unable to find system libraries.
Steps to Reproduce

Configure Lean's build with cmake --preset release.
Build Lean with make -C build/release -j<N>.
Run tests with make -C build/release test -j<N>.

Expected behavior:
Invocations of the C++ compiler by Lake succeed.
Actual behavior:
The C++ compiler emits errors indicative of an unspecified sysroot, e.g.
2307: trace: .> MACOSX_DEPLOYMENT_TARGET=99.0 /Library/Developer/CommandLineTools/usr/bin/cc -o ././.lake/build/bin/hello ././.lake/build/ir/Main.c.o.export ././.lake/build/ir/Hello.c.o.export -L /Users/zwarich/lean4/build/release/stage1/lib/lean -lleancpp -lInit -lStd -lLean -lleanrt -lc++ -lLake  /opt/homebrew/lib/libgmp.dylib /opt/homebrew/lib/libuv.dylib
2307: info: stderr:
2307: ld: library 'c++' not found

Versions
lean4 d9d54c1f9
macOS Sonoma 14.7.1

This appears to be caused by the test Makefiles setting LEAN_CC but not passing the additional flags that leanc would add. Commenting out the line setting LEAN_CC fixes the problem, but I don't know if that's acceptable for all targets. It does look like this change to set LEAN_CC was a few weeks prior to the creation of leanc, so maybe it's no longer necessary?


Add organisers section as well as founders section to about page

Adds an organisers section to the main page - I didn't realise Emmy wasn't on there! 😱  and adds Paz too.
Marking as WIP - @pazbc, I think we need your bio for this, same as what we've been asking OLS-6 participants for.

Hi Yo, Malvika and I added my bio :)

looks great - thanks both! 🙌


[New game]: Catch the emoji game

🎮 Game Request
Game logic and basic description

Click on the randomly appearing emojis at the right time to score
Score will be calculated based on the number of emojis caught within the given time

Point down the features
Game points
More you catch the emoji , more will be the points
Catch maximum emojis in given time
https://github.com/kunjgit/GameZone/assets/162733812/896436b2-9022-4a9f-9792-9502bc849420
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @Maana-Ajmera , You can't have another issue before completing the previous one 😀
you already have the following 1 open issues 👀 ! :
#4794

Hey @Maana-Ajmera ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇

Hey @Maana-Ajmera !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹


Coverage validity not verified in MapieRegressor when the number of calibration data is very small

Describe the bug
In MapieReggresor, the quantile calculation does not allow us to check the coverage validity when the number of calibration data is very small.
To Reproduce
A rigorous approach to testing the validity of the coverage was carried out by Mahdi Torabi Rad (see his notebook: https://github.com/mtorabirad/MLBoost/blob/main/Episode15/Episode15Main.ipynb) in the special case where the number of calibration data is very small (6 in the following illustration).
Expected behavior
Use the corrected quantile ($|(n+1)(1−α)|/n$) to check the validity of the coverage for any number of calibration data.
Screenshots

Issue #451 provides a link to the video presenting the notebook.


Is this still needed for api-platform?

I am confused - is this bundle still needed for api-platform?
Is it true that api-platform does not come with csrf protection out of the box???

According to the API Platform Docs, only raw documents are supported as request input by default. It also says: "Adding support for application/x-www-form-urlencoded makes your API vulnerable to CSRF attacks. Be sure to enable proper countermeasures such as DunglasAngularCsrfBundle." [1]
For a CSRF attack, one of the content types

"application/x-www-form-urlencoded",
"text/plain" or
"multipart/form-data"

is needed to be supported since these skip the CORS preflight which would prevent access from the bad origin [2]. Allowed origins in API Platform can be set via environment variable (CORS_ALLOW_ORIGIN) or in "nelmio_cors.yaml". This of course needs nelmio/cors-bundle installed, which I think is done by default.
The first two of the listed content types are disabled by default [1][3], the third one is, as far as I found, only used by the GraphQL entrypoint [4].
Thus, you should be safe in most cases but when enabling one of the listed content types, you should use this bundle.


storage: implement StateRead::nonverifiable_range

This PR adds support for range queries over the nonverifiable storage. It does so by:

extending the StateRead trait with a new method:

    /// Retrieve all values for keys in a range from the non-verifiable key-value store, as raw bytes.
    /// This method does not support inclusive ranges, and will return an error if passed one.
    ///
    /// Users should generally prefer to use wrapper methods in an extension trait.
    fn nonverifiable_range_raw(
        &self,
        prefix: Option<&[u8]>,
        range: impl RangeBounds<Vec<u8>>,
    ) -> Result<Self::NonconsensusRangeRawStream>;

use a rocksdb range iterator to create an underlying stream of keys and values
implements a StateDelta-level stream that interleaves caching layers and underlying storage.

Example API:
The first intended use of range queries is to fetch a range of compact_blocks from an arbitrary height:
let start_height = compact_block::height(20); // "000000000000000020"
// Return a stream of compact blocks starting at height 20 to the latest one.
let mut compact_block_stream = state.nonverifiable_range_query(Some(compact_block::prefix()), start_height..);

Alternative APIs considered:
        fn nonverifiable_range_raw<T, U>(
        &self,
        prefix: Option<&[u8]>,
        range: T,
    ) -> anyhow::Result<Self::NonconsensusRangeRawStream>
    where
        T: std::ops::RangeBounds<U>,
        U: Into<Vec<u8>>;

and
    fn nonverifiable_range_raw<T, U>(
        &self,
        prefix: Option<&[u8]>,
        range: T,
    ) -> anyhow::Result<Self::NonconsensusRangeRawStream>
    where
        T: std::ops::RangeBounds<U>,
        U: AsRef<[u8]>;

either are fine choices, but my preference goes to the simpler signature.

Is there any reason not to merge this for the current testnet?

Not really, I left it open to invite reviews but merging now


[Certification Projects] Update the instruction to mention that the users need to manually save their code

Description
On practice challenge pages, the code is automatically saved to the browser storage when the user clicks the test button. This isn't the case with the certification project pages, the code is not saved unless the user clicks the "Save your Code" button.
We have received multiple reports of users losing their code changes after they closed the tab or navigated away from the page, without pressing the save button.
I think one way to prevent this is to update the instruction text and mention that the user needs to manually save their progress.
Though I'm not sure where the text should be added. Perhaps at the end of the instruction and as a note?
Fulfill the user stories and pass all the tests below to complete this project. Give it your own personal style. Happy Coding!

Notes:
- [Other notes related to the project requirements]
- Your code is not automatically saved. Be sure to click the "Save your Code" button to save your progress.

Affected pages
Certification projects.
Example: https://www.freecodecamp.org/learn/2022/responsive-web-design/build-a-survey-form-project/build-a-survey-form
Additional context
Other UI/UX improvements are discussed in #53000.

What if we add a beforeUnload event? https://developer.mozilla.org/en-US/docs/Web/API/Window/beforeunload_event

Even if we're not blocking navigation, which we probably shouldn't, can we save on navigation? That combined with saving on beforeunload should prevent learners from losing their work most of the time.

Why don't we use the visibilitychange instead of beforeunload? https://developer.mozilla.org/en-US/docs/Web/API/Document/visibilitychange_event
I mean it's just as likely to trigger and seems more reliable according to what I read.

Even if we're not blocking navigation, which we probably shouldn't, can we save on navigation?

I think I like this approach best.

that the save action could fail

True, but then they'd be no worse off than they are now.

I'd like to work on this issue.

Yup, that works, but we need a little more than that.
The beforeunload event can only handle the case where the user is closing to the tab, but not if the user is navigating away from the page by clicking a link (say, the block or superblock link in the breadcrumb). Link navigation is handled by @reach/router, but the library explicitly states that it doesn't support navigation blocking. I could find a workaround, and it seems to work as intended.
I have the implementation somewhat ready. Though we talked a bit about this change in the team huddle, and Quincy suggested finding other alternatives to prompts. So I think adding a note to the instruction could help a bit (while I'm figuring out what else we can do).

Are we relying on adding a note as a temporary solution?

I think the note works as a temporary solution. I moved this back to discussing because I don't think I like the proposed solution.

save on navigation ... saving on beforeunload

I don't particularly like these, we are automatically saving the users work without them doing anything - maybe that's fine I guess, but in my experience... if I am working on codepen, and make a bunch of changes, sometimes I don't like those changes and just want to close the tab and have it revert to my last save. These solutions would not allow me to do that. Perhaps it's better than what we currently have.

Quincy suggested finding other alternatives to prompts

I think the prompts are the way to go. Feels like that's kind of standard.
Other than that, I guess I don't really have any better solutions. If we want to go the auto-save on change or whatever, that's fine. Maybe add the auto-save with an option to disable it?

As a friendly reminder,  unload and beforeunload are getting removed by Chrome.
I've tried using visibitychanged a handful of times and I'm having trouble making it take. I don't actually remember off the top of my head if our alerts actually display.
But I'm also confused about the autosave approach. How often should we automatically save the code or should it be triggered by anything?


add stellaswap L1 token pools

"stellaswap-usdc-ftm"
vaultAddr = "0x622d8A33d67464596b6612BB2d396a8d7870B536"
stratAddr = "0xBA061Ff2009905Cb45bABeb3eB7CFb8f97873555"
wantAddr =  "0x077Fc7B4455050249D7F5511ED588a665E03e6C5"
"stellaswap-wglmr-ftm"
vaultAddr = "0xbCF6b67A5Cd2642503b92c6367F9F0C592ACA744"
stratAddr = "0xe9A624096E3dc62d897EE9E9A20c2aF1c364452D"
wantAddr =  "0x2Afab635EA2DC4B498EF1C00E63b7a7dBa9c93C6"
"stellaswap-avax-wglmr"
vaultAddr = "0x016A2b0AA20F595f3d692529d74bF107E394B69f"
stratAddr = "0xAe6b83c138f84fa9E90CDbCDD105021173c8E579"
wantAddr =  "0x2F412C140080039eaF9dE01c95e84cFb1c8D98aD"
"stellaswap-matic-wglmr"
vaultAddr = "0xcA010C39E70Dc625Ec4f10a9f5d7d28d3EAD2703"
stratAddr = "0x7783d23C0880B553f97f084008b8ab7eE445701F"
wantAddr =  "0x6775ff3828a19EB5926C0C4D572183cA15Aa4C08"
"stellaswap-luna-wglmr"
vaultAddr = "0x253f0C0b847d516B49F8481df318d3DEcB048FC5"
stratAddr = "0x8F8d1dE7f7E2B8C32BABFa1CE21613e8EcB49593"
wantAddr =  "0xE2BFE2F1437f249B15B7111011219995ecc43155"

Routing is wrong on FTM-USDC. There is no liquidity GLMR-FTM. Also, you need to setPendingRewardName or call reward shows 0.

Routing is wrong on FTM-USDC. There is no liquidity GLMR-FTM. Also, you need to setPendingRewardName or call reward shows 0.

Nevermind, I see the glmr ftm farm. Its always safer to route through the pair.

https://github.com/beefyfinance/beefy-contracts/pull/165/commits/0512823fafe92f67581a4e53f8c7912e699b2656

beefyfinance/beefy-contracts@0512823

Please add correct creationdate timestamps and push again


Prefer type hints over docstring annotations

Since we only support 3.6+ we can use typehints only and do not need to put the information in the docstring.

Agreed.
Bye-bye, Python 2.


[Feature][Sort] Add MySQL CDC append

Description
[Feature][sort] add mysql cdc append
Use case
No response
Are you willing to submit PR?

[X] Yes, I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

link issue [INLONG-3956][Sort] Add Hive connector to support CDC (https://github.com/apache/incubator-inlong/pull/3963)


API endpoint POST transaction is inconsistent with CREATE

Posting a CREATE transaction, will use transform_create and messes up a lot

It shouldnt use that, needs to be consistent
it alters the output conditions, so you cannot CREATE a hashlock or others
the owners_before and owners_after get inverted

Is already handled in https://github.com/bigchaindb/bigchaindb/pull/590

@diminator I think this was resolved.
I'll close. Feel free to reopen if there's still smth missing.


[BUG] QueueUriBuilder ignores AccountName in ConnectionString if local Docker host name is used

Describe the bug
When a custom docker hostname is used in the connection string for a Queue, the account name is not correctly resolved.
This was previously mentioned in #9404 .. which was subsequently fixed (for BLOBS only) in PR #13124
Expected behavior
A connection string with a host name should correctly extract the relevant sections.
Actual behavior (include Exception or Stack Trace)
The account name is not extracted. This results in an HTTP 400 result when using the SDK to do stuff against a queue.
Environment:

Azure.Storage.Queues 12.4.2
dotnet core 3.1
Visual Studio 16.7.7

Basically, the previous issue reported occurred for Blobs and Queues. I'm assuming Tables also, but I don't use Tables so I'm not checking that. The PR fix to previous issue only updated/fixed Blobs, so Queues still have this problem.
I found this issue because I'm doing some .net core web api's targetting Azurite as the storage emulator, all in a docker-compose file and the api can't connect to the azurite (docker) instance. For arguments sake, lets assume that I got all my docker ports correct, etc.
So, when I create the test in the current code base (copied from the PR), I get the following:

Okay, great - we have a bug.
I'll see if I can create a PR to fix this. 🤞🏻

Thank you for your feedback.  Tagging and routing to the team best able to assist.

Please see related PR:  #16543

Hi,
This seems like a duplicate of this existing issue.
https://github.com/Azure/azure-sdk-for-net/issues/15403
Could you confirm if you're having the same problem so we can label this issue accordingly?
I also replied to your PR and noted that I actually resolved that today in the PR https://github.com/Azure/azure-sdk-for-net/pull/15972 that I merged.

Yep - confirmed duplicate issue #15403 and yep #15972 includes the same fix. Doh! oh well, there goes a few hours.
Great to see that this is now fixed, though. Well done team!

Yeah sorry about that! Thank you for your time and contribution though. We appreciate it!


Sync *.po files with the original *.pot

Is there a way to sync existing *.po files with the *.pot?
For example, there're new stings in theme files, we generate the new *.pot file with wp-pot, but the existing translation files de_DE.po will not contain the new strings.
Thanks!

No, this tool only generates the template file(pot). I have no plans making this a complete translation tool, I recommend poedit or the web based poeditor.


Not paginating when showing all objects

When showing all objects, we don't use paging (we're showing all objects after all). But since Kaminari only adds extension methods when paging is used, we end up in a situation where the paging methods are missing when we're showing all objects.
Related issue:
https://github.com/sferik/rails_admin/issues/2503

Note, there are no unit tests (yet). I haven't had the time to figure out how to add/run the tests yet

Now with tests (and removing unnecessary 'fix' to app/views/rails_admin/main/index.html.haml)

Merged in, thanks!


Add JsonFullValidator which validates the entire record

This PR adds a JsonFullValidator class which validates an entire record.
I also added some tests although their style is a little different because it was harder to write the tests with everything mocked out than it was to write them with just some real objects.

Hi @nightscape,
This is an interesting idea but I don’t think it fits with the goal of the gem. The use case is to validate JSON data with a JSON schema. A whole ActiveRecord record is not JSON data. Unfortunately, I don’t think we want to support this.
I’m going to close this PR but feel free to comment on it. Thank you!


Custom ApplicationModuleDetectionStrategy breaks cycle detection

Hello,
I've been experimenting with Spring Modulith in an existing Spring Boot project, and I have a need to override the standard module detection strategy as described in the reference documentation.
After doing so, the modules seem to be correctly identified:

The generated components.puml diagram shows the desired modules
Visibility constraints (no access to subpackages by default) are checked and violations are reported

However, the cycle detection seems to be broken.
I created two Spring beans, one in each application module. I proceeded to inject them into each other, and calling each others methods. The Spring Boot application won't start and correctly report the cycle, but Spring Modulith won't, even tho the cycle is visible in the generated components.puml diagram.
When working with the default module detection strategy, this works correct and Spring Modulith will report an error.
At first glance, it seems that the module detection strategy plays no role in the cycle detection, instead it seems that an ArchUnit rule is being executed, that always only considers direct children of the root package (and optionally additional packages provided through the Modulithic annotation).
This leads to quite confusing behaviour in my project, which I tried to replicate in the following examples:

When not using the Modulithic annotation to provide additional packages, the verification in this example does not report a cycle, when it should
When using the Modulithic annotation to specify the parent package of one and two as an additional package, the Arch Unit rule will detect the cycle, but will also consider the package three, which was never detected as a module by the custom ApplicationModuleDetectionStrategy (see example)
When components with cycles are defined at various package depths, the cycle detection will (sometimes) report a cycle between "slices", that are not themselves modules (see example)

I believe that all of this would be addressed, if cycle detection would solely check cycles between modules detected by the (custom) ApplicationModuleDetectionStrategy, instead of relying on a completely separate mechanism, that only works for the default ApplicationModuleDetectionStrategy.

Good catch! Would you mind trying the latest snapshots? I've applied a fix and your sample project now shows the expected behavior (all examples properly reporting the "one <-> two" cycle).

Thanks for the quick fix!
Can confirm that all examples work as expected with the 0.6.0-SNAPSHOT


Object's Speed can not be zero

hello,i am very confused that when i use the Body.speed , i see the Body has been stoped, however in fact, its speed is not zero, hero is the screen shot:

and i do this just because i want to judge if a Body has been stoped or not?
could you please help me with it?Or could you please give me some other way to deal with my demand？
thanks!

Unfortunately bodies often don't truly come exactly to rest for various reasons, so what you need to do is use a threshold and check the motion over a few updates. This is how the built in sleeping feature works which is worth looking at here Matter.Sleeping.

Unfortunately bodies often don't truly come exactly to rest for various reasons, so what you need to do is use a threshold and check the motion over a few updates. This is how the built in sleeping feature works which is worth looking at here Matter.Sleeping.

thank you for your reply, your project help me great, and wish you become better!


Fix SendGrid API key secret

What this PR does / why we need it:
The SendGrid API key feature is buggy. When configured, the deployment cannot be created because .Values.mailer.sendgridApiKey is checked but the faulty non-existant name .Values.mailgun.sendgridApiKey is included.
Checklist

[x] DCO signed
[ ] Chart Version bumped

Thanks!

The fix will be included in chart v0.3.0.


IdentityContracts > GnosisSafe - Gas usage assertion

Include gas consumption expectations on test cases comparing estimated gas calculated from GnosisSafe contract with gas spent on the transaction.
Note: Calculate gas used from sender account balances before and after the transaction.
Refs: https://github.com/tasitlabs/TasitSDK/pull/217#discussion_r260876379

Another improvement is doing the same assertion as GnosisSafe DailyModule test suite:

Withdrawal should fail because of ERC20 transfer revert
Withdrawal should fail because of ERC20 transfer out of gas
Withdrawal should fail because of ERC20 transfer returns false