[SPARK-31658] Fix SQL UI not showing write commands of AQE plan

What changes were proposed in this pull request?
Show write commands on SQL UI of an AQE plan
Why are the changes needed?
Currently the leaf node of an AQE plan is always a AdaptiveSparkPlan which is not true when it's a child of a write command. Hence, the node of the write command as well as its metrics are not shown on the SQL UI.
Before

After

Does this PR introduce any user-facing change?
No
How was this patch tested?
Add UT.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

ok to test

cc @maryannxue @JkSelf

Test build #122405 has started for PR 28474 at commit 9d539e0.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/27069/
Test PASSed.

Test build #122405 has finished for PR 28474 at commit 9d539e0.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/122405/
Test FAILed.

Test build #122422 has started for PR 28474 at commit 86fc020.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/27085/
Test PASSed.

@cloud-fan @maryannxue updated. Please review again.

Test build #122422 has finished for PR 28474 at commit 86fc020.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/122422/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/27090/
Test PASSed.

Test build #122427 has started for PR 28474 at commit 78045ee.

Test build #122427 has finished for PR 28474 at commit 78045ee.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/122427/
Test FAILed.

retest this please

Test build #122432 has started for PR 28474 at commit 78045ee.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/27094/
Test PASSed.

Test build #122432 has finished for PR 28474 at commit 78045ee.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/122432/
Test FAILed.

Test build #122439 has started for PR 28474 at commit 2726f3d.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/27101/
Test PASSed.

Test build #122439 has finished for PR 28474 at commit 2726f3d.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/122439/
Test PASSed.

Test build #122442 has started for PR 28474 at commit 4409abd.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/27103/
Test PASSed.

Test build #122442 has finished for PR 28474 at commit 4409abd.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/122442/
Test FAILed.

Test build #122443 has started for PR 28474 at commit 9111ceb.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/27104/
Test PASSed.

Thanks! Merged to master/3.0

Test build #122443 has finished for PR 28474 at commit 9111ceb.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/122443/
Test PASSed.


Upgraded redcarpet gem due to vuln

As per http://seclists.org/oss-sec/2015/q2/818
redcarpet has a vuln. This upgrade fixes it.

I'm going to merge this as the failing test is addressed in #148 PR.


bad request when using maxDuration or minDuration search condition

the zipkin version is 2.22.1. other query conditions were fine. just duration return code 400

should be better than 400.. no error is shown? there is a constraint between the two https://github.com/openzipkin/zipkin-api/blob/master/zipkin2-api.yaml#L115-L126

i try it again . only maxDuration response 400 . minDuration works for me ...


Building package libusb:x64-linux failed with: BUILD_FAILED

I was trying to install the libfreenect2 via vcpkg on Ubuntu 16.04 and got the following error:
The following packages will be built and installed:
libfreenect2[core]:x64-linux

libusb[core]:x64-linux
Additional packages (*) will be modified to complete this operation.
Starting package 1/2: libusb:x64-linux
Building package libusb[core]:x64-linux...
-- Using cached ~/vcpkg/downloads/libusb-libusb-v1.0.22.tar.gz
-- Using source at ~/vcpkg/buildtrees/libusb/src/v1.0.22-991efb2d0f
-- Building msvc/libusb_static_2015.vcxproj for Release
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:56 (message):
Command failed: msbuild;~/vcpkg/buildtrees/libusb/x64-linux-rel/v1.0.22-991efb2d0f/msvc/libusb_static_2015.vcxproj;/p:Configuration=Release;/t:Rebuild;/p:Platform=x64;/p:PlatformToolset=external;/p:VCPkgLocalAppDataDisabled=true;/p:UseIntelMKL=No;/p:WindowsTargetPlatformVersion=;/m;/p:WholeProgramOptimization=false
Working Directory: ~/vcpkg/buildtrees/libusb/x64-linux-rel/v1.0.22-991efb2d0f
Error code: 1
See logs for more information:
~/vcpkg/buildtrees/libusb/build-x64-linux-rel-out.log

Call Stack (most recent call first):
scripts/cmake/vcpkg_install_msbuild.cmake:157 (vcpkg_execute_required_process)
ports/libusb/portfile.cmake:29 (vcpkg_install_msbuild)
scripts/ports.cmake:71 (include)
Error: Building package libusb:x64-linux failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with .\vcpkg update, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
Package: libusb:x64-linux
Vcpkg version: 2018.11.23-unknownhash
I installed vcpkg at location: ~/vcpkg. Besides, I can install libfreenect2 without vcpkg successfully on my platform.

Hi @hygxy, thanks for posting this issue. Can you provide this log?
~/vcpkg/buildtrees/libusb/build-x64-linux-rel-out.log

Hi @wangli28 , here is the log file :
Microsoft (R) Build Engine version <IP_ADDRESS> for Mono
Copyright (C) Microsoft Corporation. All rights reserved.
Parallel builds (/m: or /maxcpucount:) are not yet supported on Mono/Unix. Defaulting to /m:1
Build started 6/15/2019 11:00:33 PM.

Project "~/vcpkg/buildtrees/libusb/x64-linux-rel/v1.0.22-991efb2d0f/msvc/libusb_static_2015.vcxproj" (Rebuild target(s)):
~/vcpkg/buildtrees/libusb/x64-linux-rel/v1.0.22-991efb2d0f/msvc/libusb_static_2015.vcxproj(26,3): error MSB4019: The imported project "/Microsoft.Cpp.Default.props" was not found. Confirm that the path in the  declaration is correct, and that the file exists on disk.
Done building project "libusb_static_2015.vcxproj" -- FAILED.
Build FAILED.
~/vcpkg/buildtrees/libusb/x64-linux-rel/v1.0.22-991efb2d0f/msvc/libusb_static_2015.vcxproj(26,3): error MSB4019: The imported project "/Microsoft.Cpp.Default.props" was not found. Confirm that the path in the  declaration is correct, and that the file exists on disk.
0 Warning(s)
1 Error(s)
Time Elapsed 00:00:00.19

The imported project "/Microsoft.Cpp.Default.props" was not found

This port is up


Originally, we set paged_kv_indptr to be 0s for both profile run and prefill run. We don't use flashinfer in the profile run, so it's good. But we use flashinfer in the prefill run, and the prefill run reads the kv cache. Setting paged_kv_indptr to be all 0s disable flashinfer from reading the kv cache, which corrupts the output.

test failures are not related and are fixed in the main branch.


Fix is variable set test

Issue
Currently, using the nvm.sh script from another script where the check for unbound variables is on (set -u) will give the following error:
NVM_USE_OUTPUT: unbound variable

Note
The issue described above occurs on an Ubuntu 14-04 environment Bash version 4.3.11.
However, it didn't occur on my MacBook Pro (v10.11.4) with Bash version 3.2.57.

Not a huge deal, but it'd be awesome if when rebasing in the above comment, you changed the commit message so it starts with "[Fix] `set -e`: " :-)

Not a huge deal, but it'd be awesome if when rebasing in the above comment, you changed the commit message so it starts with "[Fix] set -e: " :-)

Updated. However, I wrote "set -u" and not "set -e" as I guessed that is what you actually meant...?

Thanks!


BUG - getting ConvergenceWarning despite convergence of the solver

script to reproduce

import numpy as np
from numpy.linalg import norm

from celer import celer_path
from celer.utils.testing import build_dataset

n_samples, n_features = 30, 50
X, y = build_dataset(n_samples, n_features)

alphas_max = norm(X.T@y, ord=np.inf) / n_samples
alphas, coefs, gaps = celer_path(X, y, 'lasso', max_iter=1, alphas=[
                                 alphas_max / 100.], verbose=2)

Logs

#########################
##### Computing alpha 1/1
#########################
Iter 0: primal 39.3909227664, gap 3.86e+01, 10 feats in subpb (50 left)
Epoch 10, primal 11.2687274551, gap: 3.82e-04
Epoch 20, primal 11.2687274532, gap: 2.60e-08
Exit epoch 20, gap: 2.60e-08 < 7.88e-05
c:\users\hp\desktop\celer-repo\celer\celer\homotopy.py:312: ConvergenceWarning: Objective did not converge: duality gap: 38.607043403374846,
tolerance: 7.87818426033482e-05. Increasing `tol` may make the solver faster without affecting the results much.
Fitting data with very small alpha causes precision issues.
  sol = celer(

Got it!
Convergence of the inner problem doesn't imply convergence of the global problem.


Docs: Coding Bug within docs

Path --> /layout/grid/#how-it-works
Here is the bug

As per i see it , its doesn;t have to do anything with the framework but instead docs.min.css file with below styling
.bd-example-row-intro .container {
    padding: 1px;
    border: 1px solid rgba(217,83,79,.5);
}

If padding: 1px; is commented out, it will be back to the normal 15px padding to left and right and will resolve the issue respectively

@mdo Best Option would be to remove  the bd-example-row-intro class from the parent div of the container div . as removing it is just simply resolving the issue completely including those unneccesary border outlines.

I think this is intentional. If you actually take a look at the docs it mentions those outlines:
- Containers (outlined in blue above) provide a means to center your site's contents. Use `.container` for fixed width or `.container-fluid` for full width.
- Rows (outlined in red above) are horizontal groups of columns that ensure your columns are lined up properly. We use the negative margin method on `.row` to ensure all your content is aligned properly down the left side.

I assume the padding: 1px is just to make it stand out a little bit more

Ah, yeah, I meant to come back to this. The borders aren't doing what I wanted them to to show how things overlap and make use of the negative margin. Going to remove it outright instead of fiddling with it for now. Thanks for reporting!


README: mention that sub_module is a required parameter and target ne…

…eds to include the port (fixes: #28)
Also add a helpful hint to modbus.yml how to turn "two registers wide" into a datatype.
This is info that would have helped me a lot when starting to use modbus_exporter. I hope this PR can make it more visible.

looks good - thx @fschlich !

Related, should we default sub_target to 1 if it's not set?
Sent by mobile; please excuse my brevity.
On Sun, Oct 9, 2022, 16:38 bastischubert @.***> wrote:

@.**** commented on this pull request.
looks good
—
Reply to this email directly, view it on GitHub
https://github.com/RichiH/modbus_exporter/pull/36#pullrequestreview-1135353760,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFYII5VOM5JKK66J6SMU63WCLKH7ANCNFSM6AAAAAARAJCW2Y
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Size is wrong in constraintlayout

Hello Anas, thanks for the great library!
I ran into a slight problem with the sizing of the component in my layout.  See in the attached image how the speedometer on the left is quite a bit taller than it should be?

Here is the XML for this layout:
<?xml version="1.0" encoding="utf-8"?>
<android.support.constraint.ConstraintLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:app="http://schemas.android.com/apk/res-auto"
    android:id="@+id/constraintLayout"
    android:layout_width="match_parent"
    android:layout_height="match_parent">

    <com.github.anastr.speedviewlib.SpeedView
        android:id="@+id/leftGauge"
        android:layout_width="0dp"
        android:layout_height="wrap_content"
        app:layout_constraintEnd_toStartOf="@+id/rightGauge"
        app:layout_constraintStart_toStartOf="parent"
        app:layout_constraintTop_toTopOf="parent"
        app:sv_speedometerMode="NORMAL" />

    <com.github.anastr.speedviewlib.SpeedView
        android:id="@+id/rightGauge"
        android:layout_width="wrap_content"
        android:layout_height="0dp"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintHeight_percent=".7"
        app:layout_constraintTop_toTopOf="parent"
        app:sv_speedometerMode="RIGHT" />

</android.support.constraint.ConstraintLayout>

well @jonbhanson
after long adaptation and hard work i found that SpeedView doesn't have any problem in measuring dimensions.
the issue with ConstraintLayout is that the last is force SpeedView to change its aspect ratio.
in other word, you can tell ConstraintLayout to keep SpeedView's aspect ratio by simple line app:layout_constraintDimensionRatio="1:1" add it to leftGauge.
and you may need to set layout_height="0dp" as well.

@anastr it seems that I sent you on a wild goose chase.  Sorry for the trouble.  Thanks again for a great library!


AccessToken Lifetime

When defining the client you have the option to define AccessTokenLifetime . I have set this value to 1.
I am using the MVC client. I authenticate fine and can access the methods marked with [Authorize]. I know I need to check the AccessToken with each request to see if is still valid. Where and how would I do this?  Any additonal reading would be great.

The access token is used to call APIs, not authenticate to the client.
For learning and questions about the basics of the protocols, we'd encourage you to use stack overflow.
https://stackoverflow.com/questions/tagged/?tagnames=identityserver3&sort=newest
We actively monitor SO as well but hope to create a community there so questions can get answered faster when we are not available.


how to use uninstall playbook

There is an uninstall playbook under playbook/adhoc - probably it is meant to uninstall packages installed by ansible. Question is how do I use it? (of course, I am an ansible newbie)

I went ahead and tried this and it seems to be working:
ansible-playbook -i $INVENTORY playbooks/adhoc/uninstall.yml


Calling mju_error in a grpc server crashes the process without outputting the error.

We should avoid using mju_error (which crashes the process), and instead use an error reporting mechanism that plays nice with the grpc machinery.

@nimrod-gileadi ?


TypeError: 'staticmethod' object is not callable

Hey, I just tried to install this on my Debian 11 server. When running any command (I tried it with speedport and speedport --help) I get this error:
Traceback (most recent call last):
  File "/usr/local/bin/speedport", line 5, in <module>
    from speedport.__main__ import start
  File "/usr/local/lib/python3.9/dist-packages/speedport/__init__.py", line 1, in <module>
    from .speedport import Speedport
  File "/usr/local/lib/python3.9/dist-packages/speedport/speedport.py", line 19, in <module>
    class Speedport:
  File "/usr/local/lib/python3.9/dist-packages/speedport/speedport.py", line 129, in Speedport
    async def ip_data(self):
TypeError: 'staticmethod' object is not callable

$ python --version
Python 3.9.2
$ lsb_release -a
No LSB modules are available.
Distributor ID: Debian
Description:    Debian GNU/Linux 11 (bullseye)
Release:        11
Codename:       bullseye

Hey, the used syntax seems to be incompatible with python <3.10, should be working now.
Thanks for reporting! :slightly_smiling_face:


We shouldn't extract domains from title of answers

Example: https://metasmoke.erwaysoftware.com/post/34824
That probably makes little to no sense, right?

This one looks easy and not urgent, maybe make it a hacktoberfest?

@Undo1 The title belongs to the question, not the answer. If an answer occasionally shows up under a bad question, it would only confuse the domain record.

Oh, duh, thanks.

@ArtOfCode- @Undo1 Can you make this a hacktoberfest? I've deleted my fork so no random commits here.

Summary
Difficulty:

Scope:

Issue Owner: @ArtOfCode-
This issue needs you to modify an existing process with a new condition. metasmoke receives two types of posts: questions and answers. Questions have titles and bodies. Answers also have titles and bodies, but the title comes from the associated question and so is actually unrelated to the answer. We'd like to stop domain names being parsed from titles if we've received an answer.
Guidelines
The relevant code is currently on line 140 of app/models/post.rb - the parse_domains method. You'll need to modify this method to check whether the current post is an answer (there's a handy answer? method for that) and if so, exclude the title from URI extraction.
Once you're happy with your work, send us a pull request and we'll review it together.
How to get help
For additional support in completing this issue, either:

comment on this thread
if you have a Stack Overflow account, drop into our chatroom (you'll need 20 reputation - if you don't have that, let the issue owner know your username or ID here so they can arrange for access)
email the issue owner at<EMAIL_ADDRESS>

Hi @ArtOfCode- 👋 would love to help out with this one!

Go for it @jpotts244. Ask questions here or (better) in https://chat.stackexchange.com/rooms/11540 and someone will help :)

Closed with #514


WARNING: This is a development server. Do not use it in a production deployment

hi,
i am having an issue saying WARNING: This is a development server. Do not use it in a production deployment
how can i fix this

That's not a problem.. flask server always shows that
Put your proxies in a text file and paste that text file location instead of the proxy you have typed when asked for proxy file name.
Also get new credentials for your proxies. You have shared your proxy password here anyone can use that now. Always be careful before posting any ss.


Update the Behaviors section to reflect the change in #1444

cc @arthurevans

We recently made a major change to the master branch of this repo with the 0.9 release - would you mind re-submitting a PR here rebasing from the latest master? Thanks!

I'm unclear on what the exact behavior is, and it sounds like some of the Polymer team are as well, given that there are some unanswered questions in this thread. Maybe someone on the team can make the change, since they will know how best to phrase things.


fortio-daily: error in json file should be a build failure instead of saving invalid json

https://fortio-daily.istio.io/data/qps_max-s2_to_s1-master-20180618-09-15-2018-06-18-15-25.json
contains
upstream connect error or disconnect/reset before headers

instead of being a failed build
Where are the logs of those dailies ? we need to add a check for errors
and for readyness/sanity check
some of it did run but with max latency of 16s and 100% errors
https://fortio-daily.istio.io/browse?url=qps_max-ingress_to_s1-master-20180618-09-15-2018-06-18-15-25.json

probably related to regpatrol detected failure #6363

@hklai kindly gave me the link to the logs/builds: https://prow.istio.io/?author=istio-release-robot&job=daily-perf-tests


Problem accessing response data.

At the moment I am using the following JS:
$('#attachmentFile').fileupload({
    url: '/uploadAttachment',
    done: function (e, data) {
        $.each(data.result, function (index, file) {

        }
});

This initialises the upload fine and my server code is invoked and the file uploaded. I then return the JSON response:
[{"fileSize":"86 B","id":"3f25af0b-00b5-4754-9b61-56dd79fe73f7","displayText":"activation.log","status":"COMPLETE"}]
However I am unable to access this within the callback function under:
file.id, file.fileSize etc as I would expect. These are just coming back as undefined.
Is there something I am missing?
Many Thanks
Chris

I was having a similiar problem.
Here's how I resolved it:
@RequestMapping(value = "/upload", method = RequestMethod.POST)
public void  handleUpload( @RequestParam("fileToUpload") CommonsMultipartFile uploadFile, ServletResponse response){
List status = new ArrayList();
UploadStatus uploadStatus = new UploadStatus();
status.add(uploadStatus);
if(uploadFile == null || StringUtils.isBlank(uploadFile.getOriginalFilename())){
    uploadStatus.setMessage(new Message(MessageType.important, "File name must be specified."));
}else{
    uploadStatus.setName(uploadFile.getOriginalFilename());
    uploadStatus.setSize(uploadFile.getSize());
}
ObjectMapper mapper = new ObjectMapper();
try {
    JsonGenerator generator = mapper.getJsonFactory().createJsonGenerator(response.getOutputStream(), JsonEncoding.UTF8);
    mapper.writeValue(generator, status);
    generator.flush();
} catch (Exception e) {
    e.printStackTrace();
}

}


Make Add Name Case Insensitive

Probably also need to reflect in UG that 2 names eg: AnDy and aNdY are considered the same student.


Limit exposure time

Hi,
Thanks for this repository @Ar-Ray-code, you did an awesome work here.
Is there a way to set the minimum and maximum exposure time?
I'm using the ov9281 sensor, Bookworm OS. My image is saturated, so I though to limit the maximum exposure time and reduce the minimum exposure time.
I tried to set:
this->camera_->options->setExposureMode(Exposure_Modes::EXPOSURE_SHORT);

but its not enough for me.
didn't manage to work with the capturePhoto function.
Thanks in advance.

Hello.
Thank you for your report. The current version of libcamera may be different from when this repository was created, so we will start working on it as soon as the current project is finished. (but that will likely be after April)
There is probably a C++ wrapper for the RaspberryPi here (raspberrypi/rpicam-apps), so you can try that first.
Any additional hardware requirements or benchmarking methods would help me get started on the fix sooner, and any PR would be greatly appreciated.
Thanks.

Thanks for the quick answer.
I did manage to use the shutter option to limit the exposure time and the image is not longer saturated.
I'll add this as a new parameter in a new PR.
Thanks again.


New package: Equalizer.APO version 1.3.2

Pull request has been created with komac v2.1.0 :rocket:
Resolves  https://github.com/microsoft/winget-pkgs/issues/145074
Microsoft Reviewers: Open in CodeFlow

@microsoft-github-policy-service agree

Automatic Validation ended with:
(Automated response - build 870.)

This package can only be installed interactively. Applications must be able to install unattended


vagrant: Re-download corrupted cached files

During VM provisioning, we cache some downloaded files outside the VM, to speed up subsequent provisionings. If a cached file is present we consider that it's safe to reuse. That is however not the case when the download was interrupted (e.g., because of a network error or because VM provisioning was interrupted). Ideally we would check the signature of downloaded files, but the associated projects have different ways of exposing the signatures.
This commit abstract the common download+decompress case behind download_and_decompress_to() and extends that function to re-download the file on decompress errors.

Coverage increased (+0.05%) to 45.632% when pulling 8967485f425ab60aa42425d155cf4db777281eeb on pr/pchaigno/vagrant-redownload-corrupted-cached-files into 1fb5ffe41da2f925b0c4381d7d03557c0514851c on master.

Hm, there's actually a much simpler way to fix this. We can download those files into /tmp/ and move them to the cache directory only once the download ends. That way we can trust that if a file exists in the cache directory, it isn't corrupted. I'll rework the PR.

Hm, there's actually a much simpler way to fix this. We can download those files into /tmp/ and move them to the cache directory only once the download ends. That way we can trust that if a file exists in the cache directory, it isn't corrupted. I'll rework the PR.
EDIT: Plus, that fix will also work for downloaded files that are not archives.

nice idea!

Closing in favor of #10660 which implements the above-mentioned change in addition to a few other adjustments.


libsForQt5.libopenshot-audio: 0.1.4 -> 0.1.5

Semi-automatic update generated by https://github.com/ryantm/nix-update tools. These checks were done:

built on NixOS
Warning: no binary found that responded to help or version flags. (This warning appears even if the package isn't expected to have binaries.)
found 0.1.5 with grep in /nix/store/0a0pk23wn2jqnkc9h7jb0i0wp8b5zkzm-libopenshot-audio-0.1.5
found 0.1.5 in filename of file in /nix/store/0a0pk23wn2jqnkc9h7jb0i0wp8b5zkzm-libopenshot-audio-0.1.5
directory tree listing: https://gist.github.com/6096ee421bf911b453a5b862e61452ec

cc @AndersonTorres for review

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: libsForQt5.libopenshot-audio
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowBroken = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowBroken = true; }
to ~/.config/nixpkgs/config.nix.

Success on aarch64-linux (full log)
Attempted: libsForQt5.libopenshot-audio
Partial log (click to expand)
-- Installing: /nix/store/7zi02yar3zsk8a5vmg3ijnzmylijdzi7-libopenshot-audio-0.1.5/bin/openshot-audio-test-sound
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/7zi02yar3zsk8a5vmg3ijnzmylijdzi7-libopenshot-audio-0.1.5
shrinking /nix/store/7zi02yar3zsk8a5vmg3ijnzmylijdzi7-libopenshot-audio-0.1.5/bin/openshot-audio-test-sound
shrinking /nix/store/7zi02yar3zsk8a5vmg3ijnzmylijdzi7-libopenshot-audio-0.1.5/lib/libopenshot-audio.so.0.1.5
gzipping man pages under /nix/store/7zi02yar3zsk8a5vmg3ijnzmylijdzi7-libopenshot-audio-0.1.5/share/man/
strip is /nix/store/3zq400fri5dv7d30lpxlqm2v9y1iis6j-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/7zi02yar3zsk8a5vmg3ijnzmylijdzi7-libopenshot-audio-0.1.5/lib  /nix/store/7zi02yar3zsk8a5vmg3ijnzmylijdzi7-libopenshot-audio-0.1.5/bin
patching script interpreter paths in /nix/store/7zi02yar3zsk8a5vmg3ijnzmylijdzi7-libopenshot-audio-0.1.5
checking for references to /build in /nix/store/7zi02yar3zsk8a5vmg3ijnzmylijdzi7-libopenshot-audio-0.1.5...

Success on x86_64-linux (full log)
Attempted: libsForQt5.libopenshot-audio
Partial log (click to expand)
-- Installing: /nix/store/0a0pk23wn2jqnkc9h7jb0i0wp8b5zkzm-libopenshot-audio-0.1.5/bin/openshot-audio-test-sound
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/0a0pk23wn2jqnkc9h7jb0i0wp8b5zkzm-libopenshot-audio-0.1.5
shrinking /nix/store/0a0pk23wn2jqnkc9h7jb0i0wp8b5zkzm-libopenshot-audio-0.1.5/bin/openshot-audio-test-sound
shrinking /nix/store/0a0pk23wn2jqnkc9h7jb0i0wp8b5zkzm-libopenshot-audio-0.1.5/lib/libopenshot-audio.so.0.1.5
gzipping man pages under /nix/store/0a0pk23wn2jqnkc9h7jb0i0wp8b5zkzm-libopenshot-audio-0.1.5/share/man/
strip is /nix/store/fzcs0fn6bb04m82frhlb78nc03ny3w55-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/0a0pk23wn2jqnkc9h7jb0i0wp8b5zkzm-libopenshot-audio-0.1.5/lib  /nix/store/0a0pk23wn2jqnkc9h7jb0i0wp8b5zkzm-libopenshot-audio-0.1.5/bin
patching script interpreter paths in /nix/store/0a0pk23wn2jqnkc9h7jb0i0wp8b5zkzm-libopenshot-audio-0.1.5
checking for references to /build in /nix/store/0a0pk23wn2jqnkc9h7jb0i0wp8b5zkzm-libopenshot-audio-0.1.5...


Small refinements when information is not present in dataset View

[x] When the Sample, Data or Abstract is not provided (http://localhost:8000/app/dataset.html#/PXD000003*PeptideAtlas) The headline should be removed because do not contain that information. For example in pubmed when the abstract is not present them the system do not show the ABSTRACT word see example http://www.ncbi.nlm.nih.gov/pubmed/24304322

[x] It would be good for the publication also show the information of the authors, and the citation like pubmed shows, because the users are use to that. For example http://www.ncbi.nlm.nih.gov/pubmed/24870542

done


Migrate Tutorials from Qiskit/qiskit-tutorials

Summary
Important: This PR should not be "squashed" otherwise the whole migration commit history will be lost.
Details and comments

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.4 out of 5 committers have signed the CLA.:white_check_mark: manoelmarques:white_check_mark: nonhermitian:white_check_mark: woodsp-ibm:white_check_mark: Cryoris:x: ZoufalcYou have signed the CLA already but the status is still pending? Let us recheck it.


Login not possible because JavaScript is disabled

When I try to log in I get the following error

I have actually the same problem ...

Same here; Windows 10 x64.
Solved by deleting %APPDATA%\yakyak, launching the app and signing in afresh.

Just tried partlyhuman's fix, and it worked like a charm. On Mac, the command is rm -rf ~/Library/Application\ Support/yakyak/

Instead of deleting your app data, you can also go back to the first screen where it shows you the account you're trying to log in to, then click "Add account" below that, and log in normally using the same account.

Solution also works for Linux, remove ~/.config/yakyak

Instead of deleting your app data, you can also go back to the first screen where it shows you the account you're trying to log in to, then click "Add account" below that, and log in normally using the same account.

This did not work for me back when I first started banging my head against this, and today it did. Thanks for motivating me to try it again.

Instead of deleting your app data, you can also go back to the first screen where it shows you the account you're trying to log in to, then click "Add account" below that, and log in normally using the same account.

This did not work for me back when I first started banging my head against this, and today it did. Thanks for motivating me to try it again.


feat: @type 'neuroglancer_legacy_mesh' triggers legacy mesh format

Currently, neuroglancer treats missing mesh info files as legacy
format, but we found that it is necessary to add some attributes
to mesh info files in order to deduplicate chunk boundaries. However,
this leaves us with no way to read legacy formats. Thus, we add
neuroglancer_legacy_mesh as an explicit type to enable continued use.

Google storage returns 403 when a file is missing, so I added it as an or condition with 404.

Thanks!


integration tests: fix integration tests for preview 10

PR Checklist
PR Structure

[ ] This PR has reasonably narrow scope (if not, break it down into smaller PRs).
[ ] This PR avoids mixing refactoring changes with feature changes (split into two PRs
otherwise).
[ ] This PR's title starts with name of package that is most changed in the PR, ex.
services/friendbot, or all or doc if the changes are broad or impact many
packages.

Thoroughness

[ ] This PR adds tests for the most critical parts of the new functionality or fixes.
[ ] I've updated any docs (developer docs, .md
files, etc... affected by this change). Take a look in the docs folder for a given service,
like this one.

Release planning

[ ] I've updated the relevant CHANGELOG (here for Horizon) if
needed with deprecations, added features, breaking changes, and DB schema changes.
[ ] I've decided if this PR requires a new major/minor version according to
semver, or if it's mainly a patch change. The PR is targeted at the next
release branch if it's not a patch change.

What
[TODO: Short statement about what is changing.]
Why
[TODO: Why this change is being made. Include any context required to understand the why.]
Known limitations
[TODO or N/A]

a follow-on pr to fix remaining SAC, contract tests is in process. asset stats, effects(contract events parsing), related to different vec/map layout in ContractDataEntry and ScString usage in sac event topics instead of ScBytes on asset symbol.


Add safe navigation ?. and null coalescing ?? operators

I propose that Beanshell implement a safe navigation operator, like many other languages. This would be a deviation from Java syntax, but Beanshell already has enough of those that this shouldn't be a serious argument against it.
Specifically, the operator ?. would behave as . if the LHS is not null. Otherwise, the entire expression would return null. If we're really going all-out, copying C#'s safe array dereference a?[0] syntax and CoffeeScript's a?() would also be neat.
We'd need to discuss how to handle chaining. Groovy attempts to invoke each item in the chain a?.b()?.c()?.d(), even if a is null. That means that if a ?. is accidentally missed (easy to do), a NPE is thrown. I feel this is not intuitive and that the lookup and invocation of c should never even be attempted after the first ?. check has a null LHS.

This would be a deviation from Java syntax, but Beanshell already has enough of those that this shouldn't be a serious argument against it.

Oi! We are are strictJava all the way!! =)
I'm sure you don't know the least of it. Have you looked through the discussions specifically at Arrays 3.0 #89
I like the idea a lot but lets keep it on the back burner for now there are other priorities... do I hear someone say: lambda support yes please.
We need to be a little careful with bells and whistles though. Let me put it this way, first and foremost beanshell is a Java interpreter. This is not our only goal though, making beanshell a kickass scripting language is a close second. Something which has already caused some waves in the community and lead to me writing the BeanShell vs Java #501 post.
I also know there exists an anti groovy sentiment who think it's too radical. But that doesn't matter we don't need another groovy anyway and the way I see it if you don't use a feature then it shouldn't have to bother you. Finding a middle ground is what put me on the natural evolution of BeanShell mantra, the closer we stay to Java to smoother everything goes. Just something to keep in the back of your mind as I suspect we're going to get more of these requests from you. =)

I feel this is not intuitive and that the lookup and invocation of c should never even be attempted after the first ?. check has a null LHS.

I agree with you, it doesn't compute otherwise... what's the point then. I just got a brainwave, what if you implement it without the need of an operator. Default behaviour is no NPE. Hmmmmmm pondering..... what are the caveats?

Java is also still missing null coalescing ?? which is fairly easy to implement.
No more:
if (obj != null) {
   // do something
}

Added 5 new operators ??, ??=, ?:, ?., and <=> see:
https://github.com/beanshell/beanshell/blob/67a7c655c16e222d8d0a1a16eddb7d075c5f8fa6/src/test/resources/test-scripts/operators.bsh#L54-L189


remove genericons

ive never used genericons on a project. i dont think it needs to be included by default in the boiler. maybe put it in the sync instead??

There are the 2 other icon fonts in the boiler now as well. I kinda agree they are a bit bloaty. I wonder if we could keep everything tho and have some kind of script than cleans out that we can use as part of the go live checklist? Maybe would have to be bash? @mixcas ?

but ive never used genericons on a project

as i understand it the boilerplate contains things we expect to use at least most of the time

i think we should remove them. most of the time we end up using svgs

Yes the question is more about the nature of what goes in a boiler really. My feeling is that the boiler is things we use all the time (grid, basic styles, CMB2), but also could include a tool box of stuff that can be useful like icon fonts or (I want to add this but haven't got round to it) moment-php things like that

yea but if i had a really weird screwdriver, like 15 sided bit or something, and i never used it and i needed room in my toolbox, or like my toolbox was already kinda heavy, id probably leave that shit at home in my drawer

also i said i didnt like this like as soon as we started the using this boil, but i let it go cause i thought ok maybe hes got a point lets see, one year later - never touched

damn now that i know yal are making branches to improve the boil im going innnnnnn

what you mean your going in? imo thats a bad analogy as it costs nothing, but yh iconfonts are very infrequently used. imo if genericons are going to go we should get rid of the other 2 iconfonts as well. i dont see the point in removing really but if thats consensus then fine by me

its not consensus unless you agree mine guy

idk whatever lets keep it for now. we can revisit this in a few months


[Bug] Fix issues in OC Data sheet

Is there an existing issue for this?

[X] I have searched the existing issues

What product(s) are you seeing the problem on?
OpenChallenges
Current behavior
There are several issues in the OC Data sheet:

There seems to be a "bug" that replaced the separator - by :. This affects several fields.

E.g. challenge name Structure:free protein:ligand affinity prediction : Task 2 Fitting

Replace em dash by regular dash. E.g. – => -
Do no include space before colon. E.g. Hidden Treasures : Warm Up => Hidden Treasures: Warm Up
Use the default sheet font for all value. Some recent values uses the font Calibri or a Google font
Do not use _ as separator in slugs: E.g. cmu_dnanexus_2023 => cmu-dnanexus-2023
Do not use capital letters in slugs: E.g. QBI-hackathon-2023 => qbi-hackathon-2023
Do not surround description by double quotes or newline character: E.g. "This fall, ..." => This fall, ...
Incorrect capitalization of constant values in fields like incentives, submission_types, etc.
Invalid constant values in fields like incentives, submission_types, etc.

@vpchung @gaiaandreoletti
Expected behavior
No response
Anything else?
Related to #2064
Commit ID
No response
Are you developing inside the dev container?

[X] I am using the dev container

Code of Conduct

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

@tschaffter  the following fields were corrected:

[X] - replaced the separator : by -
[X] - Replaced em dash by regular dash. E.g. – => -
[X] -  changed the inclusion of space before colon.
[X] - Used the default sheet font for all value. Arial 10
[X] -  used - as separator in slugs - fix bug specifically for Kaggle challenges
[X] - Did not use capital letters in slugs
[X] - Removed double quotes in description
[X] - fixed Incorrect capitalization of constant values in fields like incentives, submission_types, etc.
[X] - fixed constant values in fields like incentives, submission_types, etc.
in addition:
[X] - added EDAM nomenclature under input_data_types
[X] - as the SOP did not specify, added input_data which reflects  input_data_types EDAM onthology

I think this is safe to close now?

@vpchung Go ahead and close the issue if the spreadsheet looks good to you.

Some formatting changes needed to be reverted, but other than that, everything looks good on my end!
Closing now.


[MM-40890] Fix DND Calendar Date Hover

Summary

Fixes hover effect not being centered over dates in calendar select for custom status expiry and do not disturb. This also retains current styling for the date selector in search.
Ticket Link

Fixes MM-40890

Screenshots

Before
After

Release Note

NONE

/update-branch


update.py problems

Please tick all applicable boxes.

[ x] I am using Python 3.5 or higher (run python --version on the command line)
[x ] I have followed the official guides to install the bot for my system
[ ] I have updated my dependencies to the latest version using the appropriate update script

Which version are you using?

[ x] The latest master version (1.9.8)
[ ] The latest review version

What type of issue are you creating?

[ x] Bug
[ ] Feature request
[ ] Question

Description of issue
update.py isn't working for me. when i try to use it, i get the following messages:
./update.py
./update.py: line 1: import: command not found
./update.py: line 2: import: command not found
./update.py: line 3: import: command not found
./update.py: line 5: syntax error near unexpected token (' ./update.py: line 5: def y_n(q):'
Steps to reproduce

run update.py

Log file
Please attach your MusicBot log file (located at logs/musicbot.log) to this issue. You can do so by dragging and dropping the file here. If you do not include your log file, you WILL be asked to provide one.

First, check your version of Python by using python --version
If you're using Python 3.5.*, try using python3 -m pip install -U -r requirements.txt
Make sure that you run that within the base directory of your bot.
Since I'm using Python 3.6.*, the command that I use is python3.6 -m pip install -U -r requirements.txt
Best of luck, Hayden M.

These errors are caused by trying to run update.py as if it were a bash script. To run update.py, use ./update.sh or python3 update.py

This is the problem I am running into now.
./update.sh
Starting...
Passed Git checks...
You have modified files that are tracked by Git (e.g the bot's source files).
Should we try resetting the repo? You will lose local modifications. (y/n): y
HEAD is now at 1adb604 Merge branch 'review'
Checking if we need to update the bot...
Already up-to-date.
Attempting to update dependencies...
Python 3.5.4 (default, Mar  6 2018, 07:16:04)
[GCC 6.3.0 20170516] on linux
Type "help", "copyright", "credits" or "license" for more information.

There was a problem fetching your current bot version. The installation may not have completed correctly.
Done!

This is what happens when i try using python3 -m pip install -U -r requirements.txt
Collecting pynacl==1.2.1 (from -r requirements.txt (line 1))
  Downloading https://www.piwheels.org/simple/pynacl/PyNaCl-1.2.1-cp35-cp35m-linux_armv7l.whl (422kB)
    100% |████████████████████████████████| 430kB 323kB/s
Collecting discord.py[voice] from<EMAIL_ADDRESS>(from -r requirements.txt (line 2))
  Cloning https://github.com/Rapptz/discord.py (to rewrite) to /tmp/pip-build-0h0zwgpd/discord.py
Collecting pip (from -r requirements.txt (line 3))
  Using cached https://files.pythonhosted.org/packages/5f/25/e52d3f31441505a5f3af41213346e5b6c221c9e086a166f3703d2ddaf940/pip-18.0-py2.py3-none-any.whl
Collecting youtube_dl (from -r requirements.txt (line 4))
  Using cached https://files.pythonhosted.org/packages/0c/47/da1b2ce86420312d01dbe2e2362e6491fe2bbac21efb2418729cf72afa42/youtube_dl-2018.8.22-py2.py3-none-any.whl
Collecting colorlog (from -r requirements.txt (line 5))
  Using cached https://files.pythonhosted.org/packages/69/eb/58ae10d3c46a0195ffdd0e3943d255d0d5029d71e5457785ecd665bcf0f3/colorlog-3.1.4-py2.py3-none-any.whl
Collecting cffi (from -r requirements.txt (line 6))
  Using cached https://www.piwheels.org/simple/cffi/cffi-1.11.5-cp35-cp35m-linux_armv7l.whl
Requirement already up-to-date: six in /home/pi/.local/lib/python3.5/site-packages (from pynacl==1.2.1->-r requirements.txt (line 1))
Collecting aiohttp<3.4.0,>=3.3.0 (from discord.py[voice]->-r requirements.txt (line 2))
  Using cached https://files.pythonhosted.org/packages/72/6a/5bbf3544fe8de525f4521506b372dc9c3b13070fe34e911c976aa95631d7/aiohttp-3.3.2.tar.gz
Collecting websockets<7.0,>=6.0 (from discord.py[voice]->-r requirements.txt (line 2))
  Using cached https://www.piwheels.org/simple/websockets/websockets-6.0-cp35-cp35m-linux_armv7l.whl
Requirement already up-to-date: pycparser in /home/pi/.local/lib/python3.5/site-packages (from cffi->-r requirements.txt (line 6))
Requirement already up-to-date: attrs>=17.3.0 in /usr/local/lib/python3.5/site-packages (from aiohttp<3.4.0,>=3.3.0->discord.py[voice]->-r requirements.txt (line 2))
Requirement already up-to-date: chardet<4.0,>=2.0 in /home/pi/.local/lib/python3.5/site-packages (from aiohttp<3.4.0,>=3.3.0->discord.py[voice]->-r requirements.txt (line 2))
Collecting multidict<5.0,>=4.0 (from aiohttp<3.4.0,>=3.3.0->discord.py[voice]->-r requirements.txt (line 2))
  Using cached https://www.piwheels.org/simple/multidict/multidict-4.3.1-cp35-cp35m-linux_armv7l.whl
Collecting async_timeout<4.0,>=3.0 (from aiohttp<3.4.0,>=3.3.0->discord.py[voice]->-r requirements.txt (line 2))
  Using cached https://files.pythonhosted.org/packages/96/0f/e6357458c87fb4ed8f3df215773f3caad40968f10e05552cbd8bd28415e4/async_timeout-3.0.0-py3-none-any.whl
Requirement already up-to-date: yarl<2.0,>=1.0 in /usr/local/lib/python3.5/site-packages (from aiohttp<3.4.0,>=3.3.0->discord.py[voice]->-r requirements.txt (line 2))
Requirement already up-to-date: idna-ssl>=1.0 in /usr/local/lib/python3.5/site-packages (from aiohttp<3.4.0,>=3.3.0->discord.py[voice]->-r requirements.txt (line 2))
Requirement already up-to-date: idna>=2.0 in /usr/local/lib/python3.5/site-packages (from yarl<2.0,>=1.0->aiohttp<3.4.0,>=3.3.0->discord.py[voice]->-r requirements.txt (line 2))
Installing collected packages: cffi, pynacl, multidict, async-timeout, aiohttp, websockets, discord.py, pip, youtube-dl, colorlog
  Found existing installation: cffi 1.11.4
    Uninstalling cffi-1.11.4:
      Successfully uninstalled cffi-1.11.4
  Rolling back uninstall of cffi
Exception:
Traceback (most recent call last):
  File "/home/pi/.local/lib/python3.5/site-packages/pip/basecommand.py", line 215, in main
    status = self.run(options, args)
  File "/home/pi/.local/lib/python3.5/site-packages/pip/commands/install.py", line 342, in run
    prefix=options.prefix_path,
  File "/home/pi/.local/lib/python3.5/site-packages/pip/req/req_set.py", line 784, in install
    **kwargs
  File "/home/pi/.local/lib/python3.5/site-packages/pip/req/req_install.py", line 851, in install
    self.move_wheel_files(self.source_dir, root=root, prefix=prefix)
  File "/home/pi/.local/lib/python3.5/site-packages/pip/req/req_install.py", line 1064, in move_wheel_files
    isolated=self.isolated,
  File "/home/pi/.local/lib/python3.5/site-packages/pip/wheel.py", line 345, in move_wheel_files
    clobber(source, lib_dir, True)
  File "/home/pi/.local/lib/python3.5/site-packages/pip/wheel.py", line 323, in clobber
    shutil.copyfile(srcfile, destfile)
  File "/usr/local/lib/python3.5/shutil.py", line 121, in copyfile
    with open(dst, 'wb') as fdst:
PermissionError: [Errno 13] Permission denied: '/usr/local/lib/python3.5/site-packages/_cffi_backend.cpython-35m-arm-linux-gnueabihf.so'
You are using pip version 9.0.1, however version 18.0 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.
pi@raspberrypi:~/MusicBot $ ./run.sh
[INFO] launcher: Starting sanity checks
[INFO] launcher: Checking for Python 3.5+
[INFO] launcher: Checking console encoding
[INFO] launcher: Ensuring we're in the right environment
[CRITICAL] launcher: This version of MusicBot requires a newer version of discord.py (1.0+). Your version is 0.16.12. Try running update.py.
Press enter to continue . . .

I have the same issue. All packages are installed in ~/.local/lib/python3.5/site-packages.
also: you really, really shouldn't need to install anything with sudo for this bot, beyond maybe a systemctl script to start it

Should be fixed in review?


Nfts: minor fixes

Renames let MintWitness { owner_of_item } to let MintWitness { owned_item }
Since we need to specify here item's id, and not the owner
Moves AttributeNamespace into the pallet-nfts types file. At some point, we needed to have the AttributeNamespace defined in a general file in order to use them in nonfungible traits, but later we removed that dependency and could safely put that enum back

bot merge


root: 6.24.06 -> 6.26.04

Description of changes

Things done

Built on platform(s)

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

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

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

cc @ShamrockLee @yipengsun

Result of nixpkgs-review pr 182037 run on x86_64-darwin 1

6 packages built:

hepmc3
python310Packages.hepmc3
python39Packages.hepmc3
root
xfitter
yoda-with-root

So we need to patch llvm9 (which means re-compile it I guess) because this PIE flag is backported by ROOT developer?
Is this flag information only (mostly)? I'm just wondering if we want to jump to ROOT 6.26 in the future, we can just use the official llvm9 w/o this patch to speed up compilation :-)

I'm looking at the original pull request for llvm: https://reviews.llvm.org/D80871
Not sure if the llvm/tools/llvm-readobj/ELFDumper.cpp should also be patched? Maybe root itself never uses this tool so the change is not carried over?

So we need to patch llvm9 (which means re-compile it I guess) because this PIE flag is backported by ROOT developer?
Is this flag information only (mostly)? I'm just wondering if we want to jump to ROOT 6.26 (in the research group I belong to) in the future, we can just use the official llvm9 w/o this patch to speed up compilation :-)

I'm not sure. It was originally suggested in #169579 and it does fix the compilation. You should be able to get the overridden version of llvm_9 from cache.nixos.org same way as before once this is merged.

I'm looking at the original pull request for llvm: https://reviews.llvm.org/D80871
Not sure if the llvm/tools/llvm-readobj/ELFDumper.cpp should also be patched? Maybe root itself never uses this tool so the change is not carried over?

Seems to be not needed. I assume that llvm-tools are not used by ROOT.

6 packages built:
hepmc3 python310Packages.hepmc3 python39Packages.hepmc3 root xfitter yoda-with-root
Build fine on Linux too.

Sorry for the delay in #169579 . I was busy working on something else.
The work involves fetching the patch from root-project/root and patch it to the official llvm source code, which requires manual specification of the destination file.
I'll try fetchPatch again this weekend. It would also be great if anyone is willing to help.

@ShamrockLee If this works, you can rebase your change and focus it on just the path hardcoding.


rpi-base: Fix wic image kernel dependency

wic images depend on the kernel device trees, and therefore should depend on virtual/kernel:do_deploy to make sure these are present in the deploy directory.
Most of the time, this dependency is satisfied indirectly since a rootfs image will depend on the kernel, but add it explicitly for the cases where it is not.

Thanks. Merged to master.


Typos in parameters names of consul watch

While the readme specifies the parameters of consul watch in camelCase, the code that uses them use only lowercase letters.
backoffFactor
backoffMax
maxAttempts
vs
https://github.com/silas/node-consul/blob/master/lib/watch.js#L42

All parameters are normalized internally to lowercase.
See the normalizeKeys call in the linked function:
https://github.com/silas/node-consul/blob/40d9d814d94500fe60cfb9e85809e92d2846897a/lib/watch.js#L26
And it's implementation:
https://github.com/silas/node-consul/blob/40d9d814d94500fe60cfb9e85809e92d2846897a/lib/utils.js#L51-L63
Feel free to re-open if something else isn't working.


Issue when installing Pandas 0.16 on Windows using Conda

Hi,
I tried installing pandas .16 using conda in windows but am getting the following issue. I tried reinstalling using an admin console but it doesn't work. I've downgraded to 0.15.2 which works again but could use help.
This is my first time posting so if this is not the appropriate channel please let me know.
Thanks in advance
C:\Users\kumarra>ipython
Python 3.4.1 |Continuum Analytics, Inc.| (default, Jun 11 2014, 17:27:11) [MSC v
.1600 64 bit (AMD64)]
Type "copyright", "credits" or "license" for more information.
IPython 3.0.0 -- An enhanced Interactive Python.
Anaconda is brought to you by Continuum Analytics.
Please check out: http://continuum.io/thanks and https://binstar.org
?         -> Introduction and overview of IPython's features.
%quickref -> Quick reference.
help      -> Python's own help system.
object?   -> Details about 'object', use 'object??' for extra details.

In [1]: import pandas
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
pandas\tslib.pyx in pandas.tslib.Timedelta.__new__ (pandas\tslib.c:29661)()

TypeError: 'nanoseconds' is an invalid keyword argument for this function

During handling of the above exception, another exception occurred:

ValueError                                Traceback (most recent call last)
<ipython-input-1-d6ac987968b6> in <module>()
----> 1 import pandas

C:\Miniconda3\lib\site-packages\pandas\__init__.py in <module>()
	 45
	 46 # let init-time option registration happen
---> 47 import pandas.core.config_init
	 48
	 49 from pandas.core.api import *

C:\Miniconda3\lib\site-packages\pandas\core\config_init.py in <module>()
	 15                                 is_instance_factory, is_one_of_factory,
	 16                                 get_default_val)
---> 17 from pandas.core.format import detect_console_encoding
	 18
	 19

C:\Miniconda3\lib\site-packages\pandas\core\format.py in <module>()
	 22 import csv
	 23
---> 24 from pandas.tseries.period import PeriodIndex, DatetimeIndex
	 25
	 26 docstring_to_string = """

C:\Miniconda3\lib\site-packages\pandas\tseries\period.py in <module>()
	  6 from pandas.core.base import PandasObject
	  7
----> 8 import pandas.tseries.frequencies as frequencies
	  9 from pandas.tseries.frequencies import get_freq_code as _gfc
	 10 from pandas.tseries.index import DatetimeIndex, Int64Index, Index

C:\Miniconda3\lib\site-packages\pandas\tseries\frequencies.py in <module>()
	  7
	  8 from pandas.core.algorithms import unique
----> 9 from pandas.tseries.offsets import DateOffset
	 10 from pandas.util.decorators import cache_readonly
	 11 import pandas.tseries.offsets as offsets

C:\Miniconda3\lib\site-packages\pandas\tseries\offsets.py in <module>()
   2150
   2151
-> 2152 class Nano(Tick):
   2153     _inc = Timedelta(nanoseconds=1)
   2154     _prefix = 'N'

C:\Miniconda3\lib\site-packages\pandas\tseries\offsets.py in Nano()
   2151
   2152 class Nano(Tick):
-> 2153     _inc = Timedelta(nanoseconds=1)
   2154     _prefix = 'N'
   2155

pandas\tslib.pyx in pandas.tslib.Timedelta.__new__ (pandas\tslib.c:29707)()

ValueError: cannot construct a TimeDelta from the passed arguments, allowed keyw
ords are [days, seconds, microseconds, milliseconds, minutes, hours, weeks]

Probably a good idea to re-file this over at anaconda-issues

I did have a similar sounding issue updating last time where another python process was open. Fixed by closing all the processes and conda install -f pandas.

yes you need to force install
you had Python open when u installed

I did a force install after ensuring that Python wasn't open but I'll try it again. I'll move this. Thank you guys


Limit number of packets per transaction

If there are many (like 100) packets to be relayed (result of getPendingPackets), they are all sent in one transaction (in receivePackets). The auto-gas simulation then fails once the simulation hits the block gas limit.

Ah, yes, we definitely need to break them into groups.
Also handle the case when one of them fails or uses too much gas (this will happen more with ICA), such that we don't keep trying it and blocking others.
A lot of these edges to make it robust enough for the wild.


Switched to async block commits

The consensus contract needs to be updated to reflect the regular commit process from the Fuel chain when we upgrade to future models. This removes the previous method of providing the single sequencer signature at the time of message relaying

Codecov Report

Merging #19 (bc94ff1) into master (5f49c4c) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master       #19   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            8         7    -1
  Lines          142       129   -13
  Branches        44        40    -4
=========================================
- Hits           142       129   -13

Flag
Coverage Δ

unittests
100.00% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

contracts/fuelchain/FuelMessagePortal.sol
100.00% <ø> (ø)

contracts/messaging/FuelMessagesEnabled.sol
100.00% <ø> (ø)

contracts/fuelchain/FuelChainConsensus.sol
100.00% <100.00%> (ø)

contracts/lib/Cryptography.sol

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


Use slash commands whereever possible

Spectre uses $ as a prefix as an indicator for commands.
In general slash commands are better for this as they can also display useful descriptions and from a UX perspective are nicer when it comes to parsing args as well.
EDIT: Ok Reaper apparently also already uses Slash commands but they don't seem to work rn? Maybe perms issue?

From what I'm looking at all of these commands are @commands.hybrid_command which makes them normal and slash commands. To actually create them in discord you need a command or an on_ready listener to run await self.client.tree.sync(). I'll check if this already exists and make a PR if necessary when I get home.

Upon further inspection there is a $sync command. As long as the bot was invited with permission to create commands (if not you will have to re-invite it), using the sync command should create every slash command.

Can confirm seems to work

Although newly added commands don't seem to show up after sync. Not sure if that's a cache thing or another issue.

Syncing has a cooldown of something like 30s IIRC, and from my usage of it on my own bot slash commands are a total pain to get working ever. I think it takes a few minutes to register after adding one.

Yeah so basically we already do this after all so essentially closing as completed lol


Update kind actions with correct kubectl version and remove 1.14 and 1.18 support

Description
Update kind actions with correct kubectl version and remove 1.14 and 1.18 support
Issues Resolved
#355
Check List

[x] Commits are signed per the DCO using --signoff

For any changes to files within Helm chart directories:

[ ] Helm chart version bumped
[ ] Helm chart CHANGELOG.md updated to reflect change

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Hi @DandyDeveloper @TheAlgo @prudhvigodithi there are issues starting 1.14 and 1.18 clusters through kind action now.
If anyone have insight please let me know.
This is blocking PR checks in this repo, thanks.

@peterzhuamazon Let's merge this PR for the time being. Atm I am not aware of any such issue with kind


Take latest patch of k8s 1.26

Description
Per #8974 currently have cve
https://avd.aquasec.com/nvd/2023/cve-2023-5528/
This would fix it by jumping to fixed k8s client patch (still in 1.26)

Related issues/PRs

Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

TBD

Reminder for the reviewer
Make sure that this PR has the correct labels and milestone set.
Every PR needs one docs-* label.

docs-pr-required: This change requires a change to the documentation that has not been completed yet.
docs-completed: This change has all necessary documentation completed.
docs-not-required: This change has no user-facing impact and requires no docs.

Every PR needs one release-note-* label.

release-note-required: This PR has user-facing changes. Most PRs should have this label.
release-note-not-required: This PR has no user-facing changes.

Other optional labels:

cherry-pick-candidate: This PR should be cherry-picked to an earlier release. For bug fixes only.
needs-operator-pr: This PR is related to install and requires a corresponding change to the operator.

/sem-approve

@caseydavenport fixed merged conflict if you're still considering this.

/sem-approve


Is Traditional Chinese continue supported ?

Environment
Package name: form_builder_validators
Describe your question

It was documented on README, but I can't see messages_zh_hant.dart on lib/localization/intl
messages_zh_hant.dart was deleted on commit 46562bd792f59ef1a3a12538dd5d49063055fab6

To someone who facing the same problem, downgrade form_builder_validators to ^8.3.0 will fix.

I think that is replace by zh, I don't know why.
If you can @dessonchan, please create a PR to restore this translate file
Thanks!


React step 2

Closes #
{{short description}}
Changelog
New

{{new thing}}

Changed

{{change thing}}

Removed

{{removed thing}}

Hi,
I need to get this up.
Em sáb., 25 de abr. de 2020 às 01:10, stale[bot]<EMAIL_ADDRESS>escreveu:

We've marked this as stale because there hasn't been any activity for 30
days. If there's no further activity on this issue in the next three days
then we'll close it. You can keep the conversation going with just a short
comment.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/carbon-design-system/carbon-tutorial/pull/3789#issuecomment-619316738,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABGXFCGZGPBP46CFJ7WTHZTROJPDBANCNFSM4K4LKCLA
.

--
Att.
André V P Cardoso
Desenvolvedor (C++,C#,VB,plataforma .NET, Python)
Fone : 51-8022 7183

Thank you for applying for the badge. I recommend that you address the feedback that Alison provided in her review and once that's fixed, please apply again for the badge.


No job functions found. Try making your job classes and methods public.

I'm running the instructions from Create your first containerized Azure Functions.  Nothing fancy just following the steps.  Everything is fine until testing the container running locally.
Running locally using func start works as expected.
Repro steps
Provide the steps required to reproduce the problem:

func init --worker-runtime node --language javascript --docker
func new --name HttpExample --template "HTTP trigger" --authlevel anonymous
docker build --tag azurefunctionsimage:v1.0.0 .
docker run -p 8080:80 -it azurefunctionsimage:v1.0.0

Expected behavior
Provide a description of the expected behavior.
After running step 4, I expect to see that
info: Host.Startup[327]
      1 functions found (Custom)
info: Host.Startup[315]
      1 functions loaded

And when using the endpoint:  http://localhost:8080/api/HttpExample?name=Functions it returns the correct output:
Hello, Functions!

Actual behavior
After running step 4, this is what I see
info: Host.Triggers.Warmup[0]
      Initializing Warmup Extension.
info: Host.Startup[503]
      Initializing Host. OperationId: 'd1637a21-6030-4123-a1e3-d629205936ce'.
info: Host.Startup[504]
      Host initialization: ConsecutiveErrors=0, StartupCount=1, OperationId=d1637a21-6030-4123-a1e3-d629205936ce
info: Microsoft.Azure.WebJobs.Hosting.OptionsLoggingService[0]
      LoggerFilterOptions
      {
        "MinLevel": "None",
        "Rules": [
          {
            "ProviderName": null,
            "CategoryName": null,
            "LogLevel": null,
            "Filter": "<AddFilter>b__0"
          },
          {
            "ProviderName": null,
            "CategoryName": "Host.Function.ToolingConsoleLog",
            "LogLevel": "Information",
            "Filter": null
          },
          {
            "ProviderName": "Microsoft.Azure.WebJobs.Script.WebHost.Diagnostics.SystemLoggerProvider",
            "CategoryName": null,
            "LogLevel": "None",
            "Filter": null
          },
          {
            "ProviderName": "Microsoft.Azure.WebJobs.Script.WebHost.Diagnostics.SystemLoggerProvider",
            "CategoryName": null,
            "LogLevel": null,
            "Filter": "<AddFilter>b__0"
          }
        ]
      }
info: Microsoft.Azure.WebJobs.Hosting.OptionsLoggingService[0]
      ConcurrencyOptions
      {
        "DynamicConcurrencyEnabled": false,
        "MaximumFunctionConcurrency": 500,
        "CPUThreshold": 0.8,
        "SnapshotPersistenceEnabled": true
      }
info: Microsoft.Azure.WebJobs.Hosting.OptionsLoggingService[0]
      FunctionResultAggregatorOptions
      {
        "BatchSize": 1000,
        "FlushTimeout": "00:00:30",
        "IsEnabled": true
      }
info: Microsoft.Azure.WebJobs.Hosting.OptionsLoggingService[0]
      SingletonOptions
      {
        "LockPeriod": "00:00:15",
        "ListenerLockPeriod": "00:01:00",
        "LockAcquisitionTimeout": "10675199.02:48:05.4775807",
        "LockAcquisitionPollingInterval": "00:00:05",
        "ListenerLockRecoveryPollingInterval": "00:01:00"
      }
info: Microsoft.Azure.WebJobs.Hosting.OptionsLoggingService[0]
      ScaleOptions
      {
        "ScaleMetricsMaxAge": "00:02:00",
        "ScaleMetricsSampleInterval": "00:00:10",
        "MetricsPurgeEnabled": true,
        "IsTargetScalingEnabled": true,
        "IsRuntimeScalingEnabled": false
      }
info: Microsoft.Azure.WebJobs.Hosting.JobHostService[0]
      Starting JobHost
info: Host.Startup[401]
      Starting Host (HostId=a882533c5da5-2137340777, InstanceId=a7bb0cd1-fe7b-4d7a-b631-927e7af59d75, Version=<IP_ADDRESS>, ProcessId=1, AppDomainId=1, InDebugMode=False, InDiagnosticMode=False, FunctionsExtensionVersion=(null))
info: Host.Startup[314]
      Loading functions metadata
info: Host.Startup[326]
      Reading functions metadata (Custom)
info: Host.Startup[327]
      1 functions found (Custom)
info: Host.Startup[315]
      0 functions loaded
info: Host.Startup[0]
      Generating 0 job function(s)
warn: Host.Startup[0]
      No job functions found. Try making your job classes and methods public. If you're using binding extensions (e.g. Azure Storage, ServiceBus, Timers, etc.) make sure you've called the registration method for the extension(s) in your startup code (e.g. builder.AddAzureStorage(), builder.AddServiceBus(), builder.AddTimers(), etc.).
info: Microsoft.Azure.WebJobs.Script.WebHost.WebScriptHostHttpRoutesManager[0]
      Initializing function HTTP routes
      No HTTP routes mapped

info: Microsoft.Azure.WebJobs.Hosting.OptionsLoggingService[0]
      HttpOptions
      {
        "DynamicThrottlesEnabled": false,
        "EnableChunkedRequestBinding": false,
        "MaxConcurrentRequests": -1,
        "MaxOutstandingRequests": -1,
        "RoutePrefix": "api"
      }
info: Host.Startup[412]
      Host initialized (51ms)
info: Host.Startup[413]
      Host started (63ms)
info: Host.Startup[0]
      Job host started
Hosting environment: Production
Content root path: /azure-functions-host
Now listening on: http://[::]:80
Application started. Press Ctrl+C to shut down.
info: Host.General[337]
      Host lock lease acquired by instance ID '000000000000000000000000B79AB779'.

And when using the endpoint:  http://localhost:8080/api/HttpExample?name=Functions it returns a 404
Related information
Provide any related information
nodejs 18.18
Azure Functions Core Tools
Core Tools Version:       4.0.5441 Commit hash: N/A  (64-bit)
Function Runtime Version: <IP_ADDRESS>64

Source
const { app } = require('@azure/functions');

app.http('HttpExample', {
    methods: ['GET', 'POST'],
    authLevel: 'anonymous',
    handler: async (request, context) => {
        context.log(`Http function processed request for url "${request.url}"`);

        const name = request.query.get('name') || await request.text() || 'world';

        return { body: `Hello, ${name}!` };
    }
});

{
  "name": "",
  "version": "1.0.0",
  "description": "",
  "main": "src/functions/*.js",
  "scripts": {
    "start": "func start",
    "test": "echo \"No tests yet...\""
  },
  "dependencies": {
    "@azure/functions": "^4.0.0"
  },
  "devDependencies": {
    "azure-functions-core-tools": "^4.x"
  }
}

# To enable ssh & remote debugging on app service change the base image to the one below
# FROM mcr.microsoft.com/azure-functions/node:4-node18-appservice
FROM mcr.microsoft.com/azure-functions/node:4-node18

ENV AzureWebJobsScriptRoot=/home/site/wwwroot \
    AzureFunctionsJobHost__Logging__Console__IsEnabled=true

COPY . /home/site/wwwroot

RUN cd /home/site/wwwroot && \
    npm install

{
  "IsEncrypted": false,
  "Values": {
    "FUNCTIONS_WORKER_RUNTIME": "node",
    "AzureWebJobsFeatureFlags": "EnableWorkerIndexing",
    "AzureWebJobsStorage": ""
  }
}

@pplavetzki Try adding the worker indexing feature flag in your Dockerfile. Something like this:
ENV AzureWebJobsScriptRoot=/home/site/wwwroot \
    AzureFunctionsJobHost__Logging__Console__IsEnabled=true \
    AzureWebJobsFeatureFlags=EnableWorkerIndexing

We made it so people don't need that flag in the majority of cases, but I guess this Docker scenario was missed. Should be fixed by https://github.com/Azure/azure-functions-host/issues/9536 though, which I hope happens soon.

Thank you @ejizba that indeed fixed my issue.  Thank you.

Reopening for tracking purposes


implemented free choice of angle for rotating images

Hi,
I've implemented a second rotation function that is able to rotate by any given angle.
I left the original one in, just because I think it's optimized for special values of angle (multiples of 90°).
Additionally I've added missing semicolons and put IntelliJ IDEA project files on ignore list.
Any feedback is welcome.

Sounds fab. That's a great improvement. Thanks! I'll pull it in an probably to a publish over the weekend.

Thank you for this feedback.
I found two things I want to mention, which may prevent this change to be released in a public release (but can be merged in master for further reviews or work).

When doing 90 degree rotation, the width/height changes - but not when one does 89.9 degree. the public rotate() function isn't clear about this behaviour.
We should make the API at this point clear and explizit. To achieve that we can provide two rotation functions, or add a parameter where the user can enable/disable rotation, or we can always rotate, as soon as we round to the next 90 degree.
--> What do you think?

Just 5 seconds before reading your comment, I've found a small bug, which causes an exception. currently i'm investigating this (message: cannot read property height of undefined)
I'm taking care of this.

Problem "cannot read property height of undefined" is fixed now.

About the unexpected rotation behavior,
what do you think about making both method public:
rotate(angle, cb) {...} ==> a rotation with angles 0..360 degree, doesn't change the width nor height of the image
rotate90DegreeSteps(angle, cb) {...} ==> a rotation in 90 degree steps, also reflects in changed width and height of the image.
What do you think?

What's wrong with updating the width and height regardless of angle? Obviously this will result in areas of the new image with blank pixels (0x00000000) but that probably desirable in circumstances where someone is rotating to an angle other than multiples of 90 degrees.
I'm thinking for, example, of users where two images might be combined using the composite method. In that case, compositing one image, rotated at strange angle with lots of blank pixels, over another will produce a nicely formed composition.
On the other hand, if you don't update the width and height on rotation, won't you end up with a clipped image?

I just want to point out, that Jimp.rotate() now has inconsitent behavior (incl. the patch above).
If one does rotate(89.99) it will not change width/height, but when rotate(90) it will.
As a user of Jimp I wouldn't expect that. My wish is to make the behavior consistent.
That's why I proposed two methods instead of one.
Of course, this means that rotating will result in a clipped image. I don't see a problem in that, because it's the same most graphic tools (e.g. Gimp) work. If one needs another behavior,
one can use other transformation or copy functions beforehand.

@oliver-moran
Our discussion is already paused for a week now.
May I ask you to continue and see if we can work out a solution we both feel good with.
So, what do you think about my previous proposal?

Sorry, @nitram509, not ignoring you. Let me pull in your changes and get back to you in a day or two.

OK. I've (finally!) merged this in. I've have it in as "rotate2" for now. Let me mull over whether the discussion above for a bit before making it public. But will do that very shortly!

OK. I've made the new method public. There is now just one image.rotate( deg ); method. If the value is a multiple of 90 then the existing ("simple") function is used. If the value is not a multiple of 90 then the new ("advanced") function is used.
I've altered the advanced function so that its behavior is consistent with the existing function. The clincher was to maintain consistency and backwards compatibility.
However, the method can now be overloaded. So if you call it as image.rotate( deg, bool ); then you can specify in bool if you want the image resized (true) or not (false).

There is a seemingly unrelated bug in the test script when trying to manipulate two clones images at the same time. I'll dig into that before releasing.
Thanks for your fantastic contribution!

thanks you.

Fix the bug. Should be all good now.

I'm curious - do you plan to release soon?

Released now. There were an awful lot of changes in this release. (Too many for one release, to be honest.) I wanted to test it before pushing the button.


File preview handler fails if file extension is in uppercase

Microsoft PowerToys version
0.59.0
Running as admin

[ ] Yes

Area(s) with issue?
Developer file preview
Steps to reproduce

Create a file containing some valid Pascal code (or any other valid programming language).
Save the file as "test1.pas" - with the file name all in lower case.
Copy that file and tename it "test2.PAS" with the file extension in upper case.
Now click on the first file - the preview window should show it with syntax highlighting.
Now click on the second file - the preview window will show the file, but without syntax highlighting.

File 1 (correct):

File 2 (incorrect):

✔ Expected Behavior
Both files should display with syntax highlighting if the code file is a recognized type even if the file extension is uppercase.
❌ Actual Behavior
Files with uppercase extensions do not display with syntax highlighting even if the contents are identical to files with lowercase extensions which do work properly.
Other Software
No response

Test code used, for reference:
unit PascalPreviewTest
interface

type TTest = array of integer;

procedure TestProcedure;

implementation

procedure TestProcedure;
begin
  WriteLn('This should be syntax highlighted');
end;

@checkdigits This should have been fixed some versions ago. Would you mind trying uninstalling and installing PowerToys again?
/needinfo

Oddly enough I just tested it on another machine which is running 0.58, not 0.59, and the preview works.
I will uninstall/reinstall on the machine which shows the weird behavior and let you know if it fixes it.

OK, before I went to uninstall I tested to make sure I could still repeat the behavior. It now works.
My guess is that closing the Explorer window last night and then opening it fresh this morning resolved some left-over process?
It is working as expected now without having to reinstall.


Fix probes for internal addresses when using one contour

Changes

:bug: Use ports from Endpoints when probing, rather than fixed constant

/kind bug
/kind cleanup

Release Note

Gateway-api now probes the first port in the gateway-service's endpoints, rather than fixed ports.

Docs

Not needed at this time

/retest
/lgtm
/hold
Thank you! @carlisia Please feel free to unhold.

Looking now...

/unhold
/lgtm
/approve
Thanks for this and especially for the test instructions! 🙏


Lookup validation - Ignore local=true

We've modified a couple of our Azure AD rules to use local=true as the ESCU searches fail on our cloud ES search head.
After converting our savedsearches.conf search back to YAML and running validation over the rules, contentctl complains that the local=true lookup doesn't exist. This simple change adds an an additional non-capture group to ignore this option.

I think we will have to make this (and the larger lookup matching regex) a bit more robust against reordering parameters: https://github.com/splunk/contentctl/pull/82/files#diff-4ade9637ffaa744323786af0102ad18f3f46ea560f36f867aba8cc66c05d171aR51
It will be a fairly complicated regex and will take a bit of experimentation. I'll reach out to someone on my team who is far better at regexes than I am to refine it :)

This has turned into a much larger PR than I originally thought. We will work towards supporting things like reordering, inputlookup, and outputlook as well.
As such, I am closing this one out in favor of the new one: https://github.com/splunk/contentctl/pull/274


Add Travis integration and build badges

Let's get that build badge and Travis CI going for this repo. I don't think this will take a lot of work, especially since the .travis.yml file is already written.

See PR #89.
I just configured Travis for my own project, and since I used this repo as my seed I figured I'd pay it forward.
Thanks!

Fixed with #89 / #91.


Update the Identity key in Redis when client IP address has changed

Fix of issue 1350

I spend too much time on github action issue... I should be able to look at this next week :crossed_fingers:

Integrated in master (commit : 1b74596d5be45d364ee196ae62df40c1918bd1cb)
Thx @JaroslawLegierski  for the patch :+1:


[MODCON-12] - Implement endpoint to remove user association from a tenant

Purpose

https://issues.folio.org/browse/MODCON-12
Approach

TODOS and Open Questions

Learning

Pre-Merge Checklist:
Before merging this PR, please go through the following list and take appropriate actions.

Does this PR meet or exceed the expected quality standards?

[ ] Code coverage on new code is 80% or greater
[ ] Duplications on new code is 3% or less
[ ] There are no major code smells or security issues

Does this introduce breaking changes?

[ ] Were any API paths or methods changed, added or removed?
[ ] Were there any schema changes?
[ ] Did any of the interface versions change?
[ ] Were permissions changed, added or removed?
[ ] Are there new interface dependencies?
[ ] There are no breaking changes in this PR.

If there are breaking changes, please STOP and consider the following:

What other modules will these changes impact?
Do JIRAs exist to update the impacted modules?

[ ] If not, please create them
[ ] Do they contain the appropriate level of detail?  Which endpoints/schemas changed, etc.
[ ] Do they have all the appropriate links to blocked/related issues?

Are the JIRAs under active development?

[ ] If not, contact the project's PO and make sure they're aware of the urgency.

Do PRs exist for these changes?

[ ] If so, have they been approved?

Did you modify code to call some additional endpoints?

[ ] If so, do you check that necessary module permission added in ModuleDescriptor-template.yaml file?

Ideally, all the PRs involved in breaking changes would be merged on the same day
to avoid breaking the folio-testing environment.
Communication is paramount if that is to be achieved,
especially as the number of inter-module and inter-team dependencies increase.
While it's helpful for reviewers to help identify potential problems,
ensuring that it's safe to merge is ultimately the responsibility of the PR assignee.

The flow

Process of deleting association from a tenant
Before Data

We have a testtenant1 and a active shadow user in testtenant1_mod_user

testtenant1 exist in our diku_mod_consortia.tenant and user_tenants table

Process
I requesting with like this ({{protocol}}://{{url}}:{{okapiport}}/consortia/5e5edc70-6b7d-4b3f-9c6d-2d5c5a3c3e3f/user-tenants?userId=f0eebc99-9c0b-4ef8-bb6d-6bb9bd380a11&tenantId=testtenant1)

I removed tenantId=testtenant1 affiliation with userId=f0eebc99-9c0b-4ef8-bb6d-6bb9bd380a11

I set shadow user reactive (active = false)

Return 204 No Content


Concatenation is not working with '&'

Hi.
I have a problem with column concatenation.
It's not working with '&'; Example: =A2&","&B2
But it working with standart function CONCAT; Example: =CONCAT(A2&","&B2)
Result of these formulas is equal in MS Excel / LibreOffice Calc, but after parsing by xlsx library result of first example is "0" and result of second example is concatinated columns values

@VSelemanov can you share the file that is not being read properly by sheetjs?  likely the file did not store the correct results when it was saved

@SheetJSDev test file: https://drive.google.com/open?id=15OPP4LE9Xi9QtzWPZ8U2rkvSlk-ubNlN
I want to get value of FacilitryIDPick column.
In first row of data library returns calculated value from column after parsing.
In second row of data library returns "0" after parsing.

@SheetJSDev I think that this is problem with cell types when I try to parse file.
When I copy the CONCAT formula to bellow cells, the problem of this issue returns.
If I input CONCAT formula manualy - cell type is t: "s", if I copy CONCAT formula - cell type is t: "n"

@SheetJSDev I think that I found the problem: when I try to parse .xls file - problem is staying, but if I try to parse .xlsx file - problem solved

This is an error in the tool that wrote the file.  To be clear, the record for that file is of type Formula and has the following payload:
01 00 0a 00 34 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 17 00 23 46 00 00 00 24 01 00 00 c0 17 01 00 2c 24 01 00 03 c0 42 04 ff 00

Breaking it down: the address is "K2" and the formula value is
00 00 00 00 00 00 00 00

To interpret that, see the docs for type FormulaValue
The last two bytes are 0x0000.  Since they are not 0xFFFF, the whole thing is interpreted as a 64 bit floating point number whose value is new Float64Array(new Uint8Array(Array.from({length:8}, ()=>0)).buffer)[0] == 0
--
In any case, this has been reported to LO https://bugs.documentfoundation.org/show_bug.cgi?id=125466 and that's about as far as we can go.


dynamic generation of SMILES for PTMs

You can find the description and examples in docs/nbs/tutorial_smiles.ipynb

I would suggest to integrate the data with the modification.tsv and the amino_acids.yaml:
https://github.com/MannLabs/alphabase/blob/737f25c79c34de5182140543bf887ab61d7e53d5/alphabase/constants/const_files/amino_acid.yaml
So we don't have to add a new constants folder and need to have double book keeping of modification names. I think we can add two keys for each entry: sum/composition and smiles.
In the modifications.tsv you can just add a smiles column.

@boopthesnoot Have a look at #200.
I created an extra and a decorator for rdkit.

I would suggest to integrate the data with the modification.tsv and the amino_acids.yaml: https://github.com/MannLabs/alphabase/blob/737f25c79c34de5182140543bf887ab61d7e53d5/alphabase/constants/const_files/amino_acid.yaml
So we don't have to add a new constants folder and need to have double book keeping of modification names. I think we can add two keys for each entry: sum/composition and smiles.
In the modifications.tsv you can just add a smiles column.

@GeorgWa But the smiles in the modifications.tsv will be a mess, some of them will be AA's with PTMs, some of them will be terminal modifications only, without the AA, and we would still have to store which is which somewhere.
By adding a key for each of the AAs in amino_acids.yaml we'll still have double bookkeeping of the atomic composition because we can infer it from SMILES. Ofc it would mean having a rdkit dependency for the whole package x)

@GeorgWa But the smiles in the modifications.tsv will be a mess, some of them will be AA's with PTMs, some of them will be terminal modifications only, without the AA, and we would still have to store which is which somewhere. By adding a key for each of the AAs in amino_acids.yaml we'll still have double bookkeeping of the atomic composition because we can infer it from SMILES. Ofc it would mean having a rdkit dependency for the whole package x)

We could resolve this by looking up the localizer @Any N-Term. Alternatively we can also introduce a second column location = {'N','C','AA'} which would use dynamic or fixed smiles depending of the value.
In alphabase the modification names likeDimethyl@K are the primary keys across all applications. I think this primary key should only be defined once. Furthermore, the master record in modifications.tsv is updated automatically from unimod if more modifications are added. This way everything will stay in sync.

@GeorgWa But the smiles in the modifications.tsv will be a mess, some of them will be AA's with PTMs, some of them will be terminal modifications only, without the AA, and we would still have to store which is which somewhere. By adding a key for each of the AAs in amino_acids.yaml we'll still have double bookkeeping of the atomic composition because we can infer it from SMILES. Ofc it would mean having a rdkit dependency for the whole package x)

We could resolve this by looking up the localizer @Any N-Term. Alternatively we can also introduce a second column location = {'N','C','AA'} which would use dynamic or fixed smiles depending of the value.
In alphabase the modification names likeDimethyl@K are the primary keys across all applications. I think this primary key should only be defined once. Furthermore, the master record in modifications.tsv is updated automatically from unimod if more modifications are added. This way everything will stay in sync.

Yes, I think we should use only one PTM and AA defination file to remove ambiguity in the future.

I just catched that the dtype of unimod column in the modification.tsv changed to float. Can we move back?


Can not build with sqlcipher_flutter_libs

Flutter 3.25.0-0.1.pre
drift: ^2.20.2
drift_flutter: ^0.1.0
sqlite3: ^2.4.6
sqlcipher_flutter_libs: ^0.6.4
ERROR: /build/sqlcipher_flutter_libs/.transforms/13943aee4b8def63a7de1c2cd27fda0d/transformed/classes/classes.dex: D8: Type eu.simonbinder.sqlite3_flutter_libs.Sqlite3FlutterLibsPlugin$1 is defined multiple times: //build/sqlcipher_flutter_libs/.transforms/13943aee4b8def63a7de1c2cd27fda0d/transformed/classes/classes.dex, /*/build/sqlite3_flutter_libs/.transforms/980d91e5ec18f34eae5e772c644e6301/transformed/classes/classes.dex
com.android.builder.dexing.DexArchiveMergerException: Error while merging dex archives:
Please answer and update docs(because docs non-obvious)

If delete all dependencies of sqlite3_flutter_libs it starts


add referals list of sites

👷 Deploy Preview for laughing-lumiere-a8c520 processing.
🔨 Explore the source changes: 2ad2329b5c98aed7b931eee15224b970d257c4d9
🔍 Inspect the deploy log: https://app.netlify.com/sites/laughing-lumiere-a8c520/deploys/619bbfe738157e00076ffcdc


Fontawesome

I find it to be a very common use-case to use FontAwesome, maybe we could make a config for it?
Something like, fontawesome = true which would bring the font into spec automatically, allowing for
Pretty good use-case:

(I did this in the most illegible way possible, inlining the SVGs in HTML)

I am fine with adding FontAwesome so long as its disabled by default. I would rather pull the SVG icons that I need out of fontawesome and pull them into the _icons.scss (which I have done in the past for some icons.)
On my own blog I try to keep things minimal and dont want to load the entire FontAwesome Library for a few icons.
If you have a list of icons in Fontawesome I can grab them for you. the current icons in _icons.scss can be used like this:

Yep, that is the idea. I plan on releasing a blog soon "Blogging like a Rustacean", and my goal these weeks has been to shepherd a clear winner blogging solution for Zola which is fully featured and accessible for most.
This would just be one option to widen the accessibility - I think a majority might not care about the bundle size, even if it's great that you give it special attention.


⚠ Bofrost.lu has degraded performance

In 0fd3d41, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1864 ms

Resolved: Bofrost.lu performance has improved in 1701c93 after 29 minutes.


feat: add support of null to remove the CORS configuration from bucket

Fixes #437

@frankyn PTAL


Dictionary syntax improvements

I think the current dictionary syntax could use a few additions.
(samples are not syntax proposals)
Allow adding already defined words as prefixes/affixes
> human/s/ist/esque
> super[^] // -> superhuman/superhumans/superhumanist/superhumanesque
> berry/-ies
> rasp[^] // -> raspberry/raspberries
> straw[^^] // -> strawberry/strawberries
> ball/s
> meat[^]
  | class edible
> basket[^]
  | class sports_equipment

// and / or

> human/s/ist/esque
> berry/-ies
> super[human] -> superhuman/superhumans/superhumanist/superhumanesque
> rasp[berry]
> straw[berry]
> ball/s
> meat[ball]
  | class edible
> basket[ball]
  | class sports_equipment

This feature would be extremely useful for languages that use a lot of compound words (like Finnish, Swedish and German).
The second idea is somewhat similar to the first one:
List & prefix-with directives
@list define berries [straw, rasp]

> berry/-ies
@prefix-with berries

Once again, definitely more useful for non-English languages. Syntax is just an idea.
Both of these can be accomplised with patterns (especially after #46 is implemented), but I feel like these are good additions to the dictionary syntax to reduce unneeded repetition and the spelling mistakes that come with it.
I can help with the (possible) implementation if needed.

A motivating example in Finnish:
@list define talot [parlamentti, eduskunta, maja, ilo]

// The word 'talo' (house) in most of it's grammatical cases
// nominative, plural nominative, genetive, plural genetive, inessive, plural inessive,
// allative, plural allative, illative, plural illative, partitive, plural partitive
// translative, plural translative, essive, plural essive, adessive, plural adessive
// ablative, plural ablative, abessive, plural abessive, comitative, plural comitative
> talo/t/n/jen/ssa/issa/on/ihin/sta/ista/a/ja/ksi/iksi/na/ina/lla/illa/lta/ilta/tta/itta/ineen/ineen

> parlamenttitalo/t/n/jen/ssa/issa/on/ihin/sta/ista/a/ja/ksi/iksi/na/ina/lla/illa/lta/ilta/tta/itta/ineen/ineen
> eduskuntatalo/t/n/jen/ssa/issa/on/ihin/sta/ista/a/ja/ksi/iksi/na/ina/lla/illa/lta/ilta/tta/itta/ineen/ineen
> majatalo/t/n/jen/ssa/issa/on/ihin/sta/ista/a/ja/ksi/iksi/na/ina/lla/illa/lta/ilta/tta/itta/ineen/ineen
> ilotalo/t/n/jen/ssa/issa/on/ihin/sta/ista/a/ja/ksi/iksi/na/ina/lla/illa/lta/ilta/tta/itta/ineen/ineen

// OR

> talo/t/n/jen/ssa/issa/on/ihin/sta/ista/a/ja/ksi/iksi/na/ina/lla/illa/lta/ilta/tta/itta/ineen/ineen
@prefix-with talot

And yes, Finnish is weird and unreasonable (although you won't need most of those cases in everyday real-world use).

Great ideas! I'll go over these with @cpancake and get his input. I'm especially interested in the cross-entry referencing suggestion you made; it's a useful and clever concept that I would personally get a lot of use out of.

This looks like it would be really useful. The syntax might need to be tweeked a bit but I think the idea is good.

Just a quick update: I went ahead and implemented an initial version of dictionary references on my own. You can take a look at it here. Feedback is appreciated.

I went through your code and checked everything out. Nice work! I like your use of anonymous types in the loader, it's a C# feature I should really take advantage of more often.
I have a few suggestions to make:
Add the ability to mark table entries used solely as references as "hidden". This will allow the creation of entry "building blocks" that are not counted by the loader as entries themselves.
Improve bracket balancing for references. You currently use a String.Split call to look inside bracketed values; the problem with this is that it does not check if the starting/ending brackets are the correct ones. So, for example, I could change >^ rasp[berry] to >^ rasp]berry[ and it still worked:

This should not be allowed. If you don't want to mess with bracket balancing, you could use a single delimiter token like % or $ instead.
Increment the minimum required #version for new features. This prevents programs using older versions of Rant from incorrectly loading tables using a newer version of the format.
Hope this helps.

Add the ability to mark table entries used solely as references as "hidden".
Good idea. I will implement this soon.
Improve bracket balancing for references.
I've now fixed that by using regex instead of string.Split.
Increment the minimum required #version for new features.
Okay.


Add experimental flag to compile a static binary.

I'm experimenting with compiling static binaries for easier distribution of compiled Pony programs.
I'm not sure yet everything that will be required, but this PR is a first step. Using this flag, I was able to compile a static binary within an alpine docker container, and run it on my host.
At some point after merging this, I want to formalize this process and write up some instructions for it. Then we can remove the "experimental" warning from this flag.

I really like this.
Just one question from a linker noob:
Is it possible to statically link shared objects, in the case of using the C FFI  (libssl when using ssl or libpcre when using regex) and in case we build on a system having libllvm as a shared object? So can we reliably get fully static binaries without external dependencies?

Just incase you didn't know, I added a "use" option to statically link LLVM
when compiling ponyc, see PR #2425
https://github.com/ponylang/ponyc/pull/2425:
USE OPTIONS:
valgrind
pooltrack
dtrace
actor_continuations
coverage
llvm_link_static
scheduler_scaling_pthreads
On Fri, Apr 6, 2018 at 1:37 AM Matthias Wahl<EMAIL_ADDRESS>wrote:

I really like this.
Just one question from a linker noob:
Is it possible to statically link shared objects, in the case of using the
C FFI (libssl when using ssl or libpcre when using regex) and in case we
build on a system having libllvm as a shared object? So can we reliably get
fully static binaries without external dependencies?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ponylang/ponyc/pull/2635#issuecomment-379186941, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA-hHFLxOJDkCE07t0x6XwoS_gU9j9IBks5tlyklgaJpZM4TJpiO
.

This is now marked as Linux-only, which @mfelsche requested in the sync call.


Add class to remove the 40px for the first row on table align header with a blankslate in error

Proposed Changes

Potential Breaking Changes

Acceptance Criteria

[ ] The proposed changes are covered by unit tests
[ ] The potential breaking changes are clearly identified
[ ] README.md is adjusted to reflect the proposed changes (if relevant)

Codecov Report

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

@@          Coverage Diff           @@
##           master   #1349   +/-   ##
======================================
  Coverage    99.4%   99.4%
======================================
  Files         370     370
  Lines        9233    9233
  Branches     2235    2235
======================================
  Hits         9178    9178
  Misses         55      55

Impacted Files
Coverage Δ

.../src/components/blankSlate/BlankSlateWithTable.tsx
100% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Homework LineSearch

Is there supposed to be a way for us to test our solution to the line search question? There is no 'tests pass' print after the assertion, and I am unable to edit the assertion box to add one.

It's correct if there are no errors when you run the assertion cells.


PAINT_002_001 draw a point assigned to wrong button

draw a point feature was assigned to the brush button but should be assigned to circle button

Fixed in #26