Update fuzzing corpus.

Codecov Report
Base: 25.40% // Head: 25.40% // No change to project coverage :thumbsup:

Coverage data is based on head (3dbe6d3) compared to base (60b8a15).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##           stable     #802   +/-   ##
=======================================
  Coverage   25.40%   25.40%
=======================================
  Files          70       70
  Lines        5833     5833
  Branches      313      313
=======================================
  Hits         1482     1482
  Misses       4290     4290
  Partials       61       61

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

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


MFU calculator

As a customer I would like to be able to calculate the MFU (Model FLOPS Utilization) of my model. This only requires the throughput as input (or derived from the training metrics).
An example of how to calculate the MFU can be found here.

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread. Thank you!

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread. Thank you!

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread. Thank you!

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread. Thank you!

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread. Thank you!

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread. Thank you!

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread. Thank you!

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread. Thank you!

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread. Thank you!


docs(trace): add TractionBatteryCode aspect model

This pull request adds information, links, and example payloads of the new TractionBatteryCode aspect model to the traceability KIT.
The link to the standardization document will be added after the standard is released.

Approved from my side, can be merged.


Update package dependencies

Reference Issues or PRs
Closes #1985
Pins setuptools as a dev dependency to get around #1984

What does this implement/fix?
Put a x in the boxes that apply

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds a feature)
[ ] Breaking change (fix or feature that would cause existing features not to work as expected)
[ ] Documentation Update
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[x] Build related changes
[ ] Other (please describe):

Testing

[x] Did you test the pull request locally?
[ ] Did you add new tests?

Any other comments?

Given that we are also looking at upgrading Pydantic, this is blocked by #1928.

Given that we are also looking at upgrading Pydantic, this is blocked by #1928.

We don't have to wait for #1928. For Pydantic, we could bump it to the last v1 version 1.10.12 for now and migrate to v2 after the next release.

Thanks for reviewing @fangchenli! Unfortunately merging is still blocked. @pavithraes do you mind reviewing when you get a chance? Thank you both :)

Can merge after CI passes :)


Serial Port needs to be configured

When running api/index.py the serial port needs to be configured to the serial device controlling the quadcopter line 21 otherwise you get the error below
serial.serialutil.SerialException: [Errno 2] could not open port /dev/cu.usbserial-02X2K2GE: [Errno 2] No such file or directory: '/dev/cu.usbserial-02X2K2GE'

Yeah, that should definitely be configured some other way. Perhaps a command line argument or even a dropdown on the front end.

Hello, is it necessary to configure the serial port to serial device controlling the quadcopter in order to run the command api/index.py?? Also I want to ask if this step is necessary for the cameras to stream and display at the frontend because I have run the frontend, but there is no displaying for the cameras.


Split Framework Module Env variable.

Since we use a single container image for training/serving. This splits
the framework_module property of Env into 2 different environment
variables, one for training and one for serving.
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #49 into r2.0 will decrease coverage by 0.2%.
The diff coverage is 88.88%.

@@            Coverage Diff             @@
##             r2.0      #49      +/-   ##
==========================================
- Coverage   98.65%   98.44%   -0.21%
==========================================
  Files          14       14
  Lines         447      451       +4
  Branches       26       26
==========================================
+ Hits          441      444       +3
- Misses          3        4       +1
  Partials        3        3

Impacted Files
Coverage Δ

src/sagemaker_containers/env.py
99.44% <88.88%> (-0.56%)
:arrow_down:

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 81a052a...f433503. Read the comment docs.


Use PSA_ERROR_DATA_INVALID where warranted

Description
Resolves #3275
Version 1.0.0 of the PSA Crypto API 1.0.0 added a new error code PSA_ERROR_DATA_INVALID which is distinct from PSA_ERROR_DATA_CORRUPT and PSA_ERROR_STORAGE_FAILURE.
The goal of this task is to change the relevant PSA crypto storage-related code and documentation to return PSA_ERROR_DATA_INVALID where warranted, and update or add tests accordingly.
Requires Backporting
NO
Todos

[x] Tests
[x] Documentation

It seems we have two reviewers on this one now. Thus I am removing the "needs: reviewer" label.

It seems we have two reviewers on this one now. Thus I am removing the "needs: reviewer" label.


Report error message when launched

It will report:
Error detected while processing ColorScheme Autocommands for "*":
E121: Undefined variable: g:colors_name
E116: Invalid arguments for function CocHelperCallFn_m1_v0_13_0

on top of the colorscheme vim file, g:colors_name is defined.
no idea how it was cleared in your case. Could you share a minium vimrc/init.lua I can reproduce.
" Colorscheme: " License: MIT""

if exists('g:colors_name')
    highlight clear
    if exists('syntax_on')
        syntax reset
    endif
endif
let g:colors_name="aurora"

I just use Plug to install aurora, and then I manually set the color scheme with :colorscheme aurora. Is it related to coc?

A complete configure file is here, I use a random choice plugin to choose color scheme. But if I manually change the color scheme with :colorscheme aurora, the error is the same.

I noticed there is a commit I did not pushed. Could you update and try it again.

Seems OK now. The issue should be closed. Thanks very much.


Allow sidenav to scroll with page on desktop

I'd really like to see this feature be implemented as having two seperate scrollbars is really counterintuitive when browsing on desktops.

What does this mean? Do you want it so that both the sidenav and the page scrolls at the same time? Because that sounds a lot more counterintuitive.

Something like this.. http://leafo.net/sticky-kit/
And why would it be more counterintuitive? Desktop browsing with two seperate scroll bars is a huge hassle for many users..

Based on the design specification, I believe the current behavior is the correct behavior.
http://www.google.com/design/spec/patterns/navigation-drawer.html
See "Scrolling" on this page.

@kabaka Yes, it is thought out great, and works great, on mobile devices. In my opinion less so when utilizing it on platforms where it gets in the way of a great user experience as it is pretty counterintuitive on desktops.
The question arises, should we blindly follow the specification at the cost of user experience? In the specification all I see is screenshots from the mobile platform, not desktop.
The specification website seems to incorporate material design as well though here the navigation is filled up to about the height of 1080 pixels, which is probably the majority of desktop PCs (1980 x 1080), so it isn't a real issue if your navigation doesn't exceed that height, but my navigation exceeds it..

I think its counterintuitive because the user may not always want to scroll both with one scroll. In the case of a side nav, these elements are usually separate enough that you should scroll individually. When you want to read more of the main content it scrolls, and if you want to see more options, the side nav scrolls.
If the two things are more interconnected I could see your point but in this case I don't.

I think that if people want this particular type of scrolling, it will be up to them. We will be leaving the current behavior as is.


Upgrade fs-extra to 1.0.0

fs-extra's first stable release is out since 1 of November.
I think its time to upgrade to 1.0.0. We've been running 1.0.0 locally and is having no problems with it.
Test plan (required)
Walked through the commits from 0.30 to 1.0.0 and no breaking changes has been made to the repository. https://github.com/jprichardson/node-fs-extra/compare/1.0.0...1.x

LGTM, someone on FB's side will have to merge it.

This will need some manual import work on our side due to changes to the package.json file. We're on it.

I apologize for the delay in getting this in. I've now patched this in internally and the commit should be syncing to open source shortly.


Add a brief user guide and a quick example for Kubeflow Pipelines benchmark

Add a brief user guide and a quick example for Kubeflow Pipelines benchmark.

/assign @joeliedtke

/lgtm
for me
Please wait for @joeliedtke 's review

/assign @joeliedtke
Preview at https://deploy-preview-1972--competent-brattain-de2d6d.netlify.app/docs/pipelines/tutorials/benchmark-examples/

/assign @joeliedtke

/assign @joeliedtke sorry for the delay. I just moved the two files used in this example (taxi_updated_pool.yaml and run_service_api.ipynb) to the ml-pipeline project's public GCS bucket, since ml-pipeline project's buckets seem to be a common place for hosting many example scripts and data for KFP.

/lgtm
/approve
Thanks for adding this doc, @jingzhang36!


Retitle “Configuring kubelet Garbage Collection"

This PR fixes #25300

Hi @kbhawkey  while i addressing the comments i observe that it took the domain name for github rather than k8s.io (https://github.com/kubernetes/website/blob/master/content/en/docs/concepts/cluster-administration/docs/concepts/containers/#container-images) , so should i keep the link as  (https://kubernetes.io/docs/concepts/containers/#container-images)

Page preview:
https://deploy-preview-25457--kubernetes-io-master-staging.netlify.app/docs/concepts/cluster-administration/kubelet-garbage-collection/

Hi @Kartik494 . Looks great! ❇
The links from the GC cluster-administration page to the container page seem fine.
@sftim, the issue suggested:
Hyperlink from within https://kubernetes.io/docs/concepts/containers/ to the page https://kubernetes.io/docs/concepts/cluster-administration/kubelet-garbage-collection/

Instead, do you want to add a link to the GC page from this page:
https://deploy-preview-25457--kubernetes-io-master-staging.netlify.app/docs/concepts/containers/#what-s-next

/lgtm

/approve


Down Stream Key Fade Not Working

I was attempting to define a button to do a fade on BlackMagic Design (ATEM 1) down stream keyer number 2. I was able to make the button work with DSK 1, fade and cut. I then changed the button to do a fade on DSK 2. Nothing happened. Changed the button to do a cut on DSK 2. Worked just fine. Changed back to fade on DSK 2 and nothing happened.
The problem seems to be related, and confined, to using a fade on DSK 2. Cuts work. Fade and cuts on DSK 1 work. Just not fades from DSK 2.
Any way to resolve this issue?

What version of companion and atem firmware are you using?
This sounds like an issue that was fixed in companion 2.0, which was introduced in atem firmware v8.0.1
Related: https://github.com/bitfocus/companion-module-bmd-atem/issues/38 #32

Julian:
I am using version 8.1 software on the ATEM. I am using 1.4.0 of the companion software. The only download for version 2.0 that I saw on the website was for the beta version with a cautionary message about not relying on that version to be stable. Beta software is in the process of being developed. Thus I did not consider version 2.0 to be something that should be used in a production environment. To quote “Don't blame us if these builds ruin your show. These are experimental developer builds, and should never be considered show safe. Use at your own risk!”. Sounds alarming and something to be avoided until stable.
Ray Thompson
Media Director
Central Baptist Church
From: Julian Waller<EMAIL_ADDRESS>Sent: Tuesday, April 7, 2020 4:39 AM
To: bitfocus/companion-module-bmd-atem<EMAIL_ADDRESS>Cc: raythompsontn<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [bitfocus/companion-module-bmd-atem] Down Stream Key Fade Not Working (#54)
What version of companion and atem firmware are you using?
This sounds like an issue that was fixed in companion 2.0, which was introduced in atem firmware v8.0.1
Related: #38 https://github.com/bitfocus/companion-module-bmd-atem/issues/38  #32 https://github.com/bitfocus/companion-module-bmd-atem/issues/32
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/bitfocus/companion-module-bmd-atem/issues/54#issuecomment-610256917 , or unsubscribe https://github.com/notifications/unsubscribe-auth/APCWKYQLARUDTQKDRSLHNE3RLLRDRANCNFSM4MCXMNOA .  https://github.com/notifications/beacon/APCWKYR2XVPIFECU35RDMHDRLLRDRA5CNFSM4MCXMNOKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOERP4QFI.gif

Companion 2.0 is actually more stable than 1.4 at this point. I believe the release was delayed due to the coronavirus debacle. It was announced somewhere in the Facebook group.

Yeah, 2.0 is just a little bit of paperwork off being declared stable so it is as safe to use as 1.4 is.
But this is already fixed in 2.0, and there is nothing more I can do on this for you.
If you dont want to update companion, then you could either downgrade your atem firmware, or work around the problem with macros


2099 Namespaces should be http not https

'Hypertext Transfer Protocol' (without secure-part) http vs https
This is a pull request for Issue 2099
as showcased in the uwp-sample Package.appxmanifest

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

Docs Build status updates of commit 9cd985c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/implementing-3d-app-launchers.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Update semantic-layer-sync.md

🚀 Deployed on https://666a3d294ac3e65e58b72de4--cozy-fox-0defec.netlify.app


Attempt to decode throws a malloc error

./lzfse -decode -i inputfile -o outputFile
lzfse(34763,0x10ee8b5c0) malloc: can't allocate region
*** mach_vm_map(size=264883518046208) failed (error code=3)
lzfse(34763,0x10ee8b5c0) malloc: *** set a breakpoint in malloc_error_break to debug
malloc: Cannot allocate memory

I met the same problem.

I met the same problem.


Port some fixes

Ports PRs #25524 and #25531 to release-3.0

no need to port. i will merge master into release-3.0

🎈


scripts: compliance: add a check for missing west area maintainer enties

Add a compliance check to ensure all modules have a matching maintainer entry, add the two missing ones (added in https://github.com/zephyrproject-rtos/zephyr/pull/42580 and https://github.com/zephyrproject-rtos/zephyr/pull/56347)
@kristofer-jonsson-arm @najumon1980

So then when you load the manifest using Manifest.from_file(), it could be opening a completely different file -- as determined by the manifest.path and manifest.file variables in the local workspace. There is no enforced connection with any west.yml file.

My best interpretation of this is something like: "get the file name from manifest.path after loading it and use that for the check instead of the hardcoded one".
Cool, done!

@MaureenHelm please revisit

Rebased - had the first patch merged separately and the rebase wasn't skipping it correctly for some reason. Main patch is unchanged.


When will ES6 be released?

When will ES6 be released?

Note that this issue tracker is designed for reporting bugs in MathJax or for requesting new features.  Questions like this one are better asked in the MathJax user's forum.
mathJax v4.0.0, now out in beta release, has both MJS (ES6) and CJS (ES5) versions.  The 4.0.0-beta.7 version is the current release candidate for 4.0.0, and is feature complete.  We are in the process of a code clean-up which should lead to the official 4.0 release later this year.

Sorry, I'm not very good at English.
Now, my project installed 'mathjax@beta', but I don't know how to import the MathJax.
I don't know which file to import so that I can use MathJax in es6 mode.

Note that this issue tracker is designed for reporting bugs in MathJax or for requesting new features. Questions like this one are better asked in the MathJax user's forum.
mathJax v4.0.0, now out in beta release, has both MJS (ES6) and CJS (ES5) versions. The 4.0.0-beta.7 version is the current release candidate for 4.0.0, and is feature complete. We are in the process of a code clean-up which should lead to the official 4.0 release later this year.

Sorry, I'm not very good at English.
Now, my project installed 'mathjax@beta', but I don't know how to import the MathJax.
I don't know which file to import so that I can use MathJax in es6 mode.

Closing this in favor of #3263.


Automatic ultrasound sector cropping

I am trying to crop GUI elements from single ultrasound images but it does not work (tried different thresholds), what should I do for this case?
importer = fast.ImageFileImporter.create(str(image_file))
cropper = fast.UltrasoundImageCropper.New().create(
    staticCropping=False, # Estimate cropping on first frame only or not
    thresholdVertical=50,
    thresholdHorizontal=10,
).connect(importer)
renderer = fast.ImageRenderer.create()\
    .connect(importer)

renderer2 = fast.ImageRenderer.create()\
    .connect(cropper)

fast.DualViewWindow2D.create()\
    .connectLeft(renderer)\
    .connectRight(renderer2)\
    .run()

The UltrasoundImageCropper is far from perfect. Especially for sector scans.
It can be hard to find a threshold that works perfectly for all cases.
Still, looking at your input image, in this particular case, it should work with the right threshold values.

For each column and each row it counts how many pixels have a value above 0.
To find the min x and max x positions, it starts from the middle and stops at the first x position which has a number of pixels equal to 0 above the thresholdHorizontal. It does this both in the left and right direction.
Then this is repeated for the vertical direction, y using thresholdVertical.

I have added more documentation on this method. Thanks. Closing.


Update etcd library to v3.6.0-alpha.0

Close https://github.com/wind-c/comqtt/issues/120

If you are hesitant to use the alpha library, feel free to close this pull request.

I have not identified which commit resolved the issue.

Edit:
The commit is in https://github.com/etcd-io/etcd/pull/13645.

It seems unnecessary.

I merged first and upgraded the version after etcd stable release.


How to actively terminate this session during streaming?

For blockingForEach ？ just throw exception

terminate externally, I use the thread approach.


Setting params.name in the env file overrides deployment name

Closes #213

@dennisjbell It is le done

@dennisjbell bump


[FLINK-16097][docs-zh] Translate "SQL Client" page of "Table API & SQL" into Chinese

What is the purpose of the change
(For example: This pull request makes task deployment go through the blob server, rather than through RPC. That way we avoid re-transferring them on each deployment (during recovery).)
Brief change log
(for example:)

The TaskInfo is stored in the blob store on job creation time as a persistent artifact
Deployments RPC transmits only the blob storage reference
TaskManagers retrieve the TaskInfo from the blob cache

Verifying this change
(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (100MB)
Extended integration test for recovery after master (JobManager) failure
Added test that validates that TaskInfo is transferred only once across recoveries
Manually verified the change by running a 4 node cluser with 2 JobManagers and 4 TaskManagers, a stateful streaming program, and killing one JobManager and two TaskManagers during the execution, verifying that recovery happens correctly.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@flinkbot run azure

<EMAIL_ADDRESS>review this if you are free.Thank you

@XBaith thanks for your contribution, could you please clean the git history(especially git rid of the git merge commit), and please update the pr description according to the changes


Getting experiential on a Quarkus project

Article checklist
ATTENTION: In your PR - add the ISSUE (topic form) # that this PR corresponds to. You can do this by also adding a comment that states "This fixes #(enter your Idea Content Form #)" to link and close your Idea Content (issue) when the PR is merged.
This fixes #2975.
NOTE: (Please ensure that you have only one open issue + linked pull request at a time. This will ensure that we complete the article in a timely manner from inception to publishing.)
If you have not already please go over our Resources Page for more tips and tricks on how to improve our overall technical writing.
Preliminary Checks - Formatting and Structure

[x] Is the article you are submitting an in-depth and unique article? Does it go beyond what is in the official docs and what is covered in other blog sites. See these articles as examples.
[x] Does your article follow any of the suggested structure formats? - see example formats
[x] Is your article properly formatted in Markdown? - see Github Markdown guide
[x] Have you used the correct folder and file structure? - see Contribution Guidelines
[x] Is your article over 750 words? - Tool: Wordcounter.net
[x] Have you used our preferred heading structure? - H3 (###) and up
[x] Does your article provide enough value and detail about your topic? - Articles should be clear, accurate, and fully explained.
[x] Can your article be understood by beginners? - Assume the audience is smart but has no prior exposure to the common terminology in your article.
[x] Have you included a hero image that is 600x400 pixels and have the copyright to use it? - Tools: Pixlr Image Editor for resizing and Unsplash for Creative Commons images
[x] In your PR - add a comment that states "This fixes #(enter your Idea Content Form #)" to link and close your Idea Content (issue) when the PR is merged.

Grammar & plagiarism checks

[x] Have you spellchecked and grammar checked your article? - Tools: Write&Improve - ProWritingAid -  Grammarly
[x] Please place your article through a 3rd party plagiarism checker? We suggest using Quetext, this tool is free and has a daily limit. We typically accept articles with 10% or less.
[x] Have you checked your article for readability? - Tool: Hemmingway
[x] Have you added sources for quotes and images that aren't yours?

Technical checks

[x] Are your code snippets properly formatted for syntax highlighting - see Syntax guide

Contribution guidelines
For first-time contributors and for more details, see Contributing Guidelines

This fixes #2975

plagiarism check done

@RisoriTofa Hello, a nice piece here
When describing a process, please put it in point form to avoid lengthy paragraphs. Do that for all the steps you describe to make the article easier to read.
Secondly, eliminate lengthy paragraphs by communicating only a single idea in one paragraph.
Make the changes above then we proceed to tech review. Otherwise, your article is clean and easy to follow.

Ok, and thanks!

@RisoriTofa, Quite a detailed article :rocket: check on the corrections made to improve your articles.

Great article @RisoriTofa.
Thank you for the review @jerimkaura.

Great article @RisoriTofa.
Thank you for the review @jerimkaura.


Added request headers to http tests

Build is failing because Travis is pinned to go1.11 up to @aelsabbahy to weigh in on either updating the go version or maintain compatibility:
diff --git a/system/http.go b/system/http.go
index fc18225..7de0646 100644
--- a/system/http.go
+++ b/system/http.go
@@ -74,7 +74,7 @@ func (u *DefHTTP) setup() error {
        if err != nil {
                return u.err
        }
-       req.Header = u.RequestHeader.Clone()
+       req.Header = u.RequestHeader
        if u.Username != "" || u.Password != "" {
                req.SetBasicAuth(u.Username, u.Password)
        }

I'm fine with updating go if it doesn't break anything else.

https://github.com/aelsabbahy/goss/blob/master/.travis.yml#L4
I think this is what you meant to change.

Ahh that's what I was looking for! Want me to backout the changes on go.mod?

Can you point me to the docs on the go.mod change. Go has changed dep management systems so many times, I actually haven't read up on go.mod yet.
Guess what I'm saying is, I don't understand the implications of the go.mod change you made. :)

If you get this PR green today I'll probably cut a Goss release with it tomorrow.

This is the best doc I've found to describe the go directive in go.mod. Looks like it defines the expected language version. Because header.Clone was released in go 1.13 I think it makes sense to set the go version.

Looks like the build if failing in Travis. I've noticed these fail locally prior to making any changes. If you think they're just flaky tests would you mind retrying the build in Travis so I don't have to push a no-op change?

Count: 94, Failed: 0, Skipped: 5
+[[ centos7 == \a\r\c\h ]]
+egrep -q 'Count: 93, Failed: 0, Skipped: 5'
The counts are mismatching since you added a test.

Also, docs need to be updated. I'm doing quick eyeball review on my phone currently.

Thanks for the document on go.mod, explained it very well.


Add<EMAIL_ADDRESS>

Add early testing release candidate for the Pico SDK.

Hello @bazelbuild/bcr-maintainers, modules without existing maintainers (pico-sdk) have been updated in this PR. Please review the changes.

@kilograham This is adding the Pico SDK to Bazel's main package repository. I've added you as a maintainer, but I suspect you won't want to ever have to touch this so I've also added the Pigweed team as maintainers so we can manage these. Please let me know if you'd like me to adjust the maintainer list in any way.

Re: BcrValidationResult.FAILED<EMAIL_ADDRESS>is using an unstable source url
Is it possible to bypass for this PR? I'm ok with yanking this release after we get a full release of 1.6.0 cut if that's required.


PHP 8.1 Passing null to parameter #1 ($haystack) of type string is deprecated

Hi
we just upgraded to Laravel 9 and PHP 8.1 and noticed the following message in the logs:
strpos(): Passing null to parameter #1 ($haystack) of type string is deprecated in vendor/fruitcake/php-cors/src/CorsService.php on line 108

Did you make sure to have the config/cors.php file?
Did you make sure that the 'allowed_origins' variable in this file has some URLs?

Closing for inactivity


Imput X contains NaN

Hi, I'm using the MarkerCount function to predict the cell type labels in my sc RNA-Seq data, but when I call the function like that:
df_res = MarkerCount( X=X, mkr_mat=marker_matrix, log_transformed = True, verbose = True )
It produces this error message:
ValueError: Input X contains NaN.
GaussianMixture does not accept missing values encoded as NaN natively. (...)
I have checked both input data and didnt find NaN values so I dont know what else to do.
My X dataframe looks like this:

And my marker_matrix like this:

Hi,
I guess the problem is the input matrix contains negative values. I guess you z-score scaled after the log-transformation. Why don't you try without z-score scaling so that the matrix contains only non-negative values.
By the way, there is an upgrade version of MarkerCount; called HiCAT, https://github.com/combio-dku/hicat/. I suggest to try it as it performs better and is more stable.

If you want to try HiCAT to annotate pancreas tissue, you may use this one. (You can edit the list of markers with yours)
cell_markers_rndsystems_with_pancreas_hs.tsv.txt

If you want to try HiCAT to annotate pancreas tissue, you may use this one. (You can edit the list of markers with yours)
cell_markers_rndsystems_with_pancreas_hs.tsv.txt

Hi, I tried to use the non scaled data but i'm still getting the same problem. I also tried the HiCAT package and I got a "LinAlgError: SVD did not converge in Linear Least Squares".
The X data I'm using is (it is a subsample) :
Cells_expression_matrix_subsample.tsv.txt
The cell_markers data I'm using is the one you gave me.
Thank you for your help!

Hi, I looked at your data and find that it is quite different from normal single-cell count matrix (even if it is normalized and log-transformed.) Most of all, all the entries in the data below are all non-zero. Since both hicat and markerCount uses binary information, either express or not, if all the expression value is non-zero (greater than 0), the number of expressed marker genes will be the same for all the cells. I think this was the problem HiCAT and MarkerCount were issued error.
Therefore, I tried as follows

First find the median values from your matrix, i.e., med = X.median().median()
Then, set entries to zero if it is less than the median value, X = X*(X >= med)
Then, half of the entries in X will be zero
With this matrix, I ran HiCAT and got the result
hicat_result.csv

As I said, HiCAT and MarkerCount uses binary information (either expressed or not). And some of expression values must be zeros for the tools to work properly. I hope this resolve your problem.

Yes it resolved my problem, thank you so much for your help!


🐛 BUG: workers.api.error.unknown [code: 10013]

Which Cloudflare product(s) does this pertain to?
Workers for Platforms, Wrangler
What version(s) of the tool(s) are you using?
"wrangler": "^3.72.1"
What version of Node are you using?
v20.3.0
What operating system and version are you using?
MacOs 14.5 (23F79)
Describe the Bug
Observed behavior
wrangler deploy failed with unknown error
Expected behavior
wrangler deploy successfully
Steps to reproduce
see below the minimal reproduction git repo in the readme.md
Please provide a link to a minimal reproduction
https://github.com/yquan/hammal
Please provide any relevant error logs
wrangler-2024-08-22_04-07-04_207.log

resolved with https://www.cloudflarestatus.com/incidents/x54t30777c2q


Xfail some tests on k8s

To make the k8s lane the test cases that fail can be marked as xfail and the is_ovs_running has to be removed (don't know if it's really needed).

/retest

@qinqon If you are changing test python code, you need to use python-black to format the code and pylink/flake8 to do lint checking also.

@qinqon If you are changing test python code, you need to use python-black to format the code and pylink/flake8 to do lint checking also.

Done

@ffmancera did you have some comments on the PR ?

/retest

/retest

/retest

@cathay4t @ffmancera k8s lane is green now, we can merge this, and maybe mark the lane as required.


--list-file-types

The feature, file-types to restrict search to certain types is great.
But do we have any option to re-define the file types
for example, in my project, I hope to search not only C++ file type, but also some config file like *.abi
I hope add the "abi" file to C++ types list.

Is this a duplicate of #688?


Image Access Denied

I get Access Denied message for image block. How to access the URL? My notion is public.
https://s3-us-west-2.amazonaws.com/secure.notion-static.com/6b42051e-e390-4f74-a790-a9347a5adc65/1.jpg

I've been getting image errors lately too. Haven't made much progress in figuring out why. Anyone else?

Oh, here is a PR into react-notion to get it fixed. Has some info on what the cause is. https://github.com/splitbee/react-notion/pull/32

Yes this was fixed within react-notion
https://github.com/splitbee/react-notion/pull/32


Added time based filtering to ingest and optimize_rasters functions

Description
This PR introduces the --ignore-older-than flag to both the optimize_rasters() and ingest() functions, allowing users to skip processing files that are older than a specified cutoff based on their last modified time.
Key Changes

New --ignore-older-than flag:
Enables users to set a time threshold, ignoring files older than the provided value.

Supported formats for the --ignore-older-than flag:

Time durations: 5d, 2h, 30m, 10s (representing days, hours, minutes and seconds, respectively)
Specific timestamps: YYYY-MM-DD HH:MM:SS

Behavior

Files with a last modified time older than the given threshold will be skipped and not optimized or ingested.

Thanks @mmarfat but I'm not sure I get it. Why does Terracotta need this feature?

I suppose Terracotta itself doesn't really "need" this feature, rather it's something that was useful for my specific application. As far as I can tell there is no explicit way to define which files should be ingested (other than trough the naming pattern).
In my specific case this was limiting because I'm working with a folder over which I have no control and it's gotten quite big over time. This causes the ingest operation to be quite slow. To avoid ingesting all the files all the time I implemented this so it would only ingest the "newest" files.
I know it's not the most attractive feature but since it's not taking away any functionality I thought I'd suggest it for a PR.

Hmm I see. Any reason why you didn't want to reach for the Python API for this?
I'm not strongly opposed to this feature and the implementation looks high quality, but I'm trying to understand if there's a genuine blind spot that's addressed here or whether it adds complexity for little net-gain.

I was thinking the same - even considering the "batteries included" philosophy of Terracotta, this looks like a case for writing your own little CLI script that implements your specific file filtering logic.
Also agree that this looks very nicely made!
So I'd recommend not to implement this, since

The logic is quite specific to your use case
It would add complexity to users looking at the CLI to understand and decide whether they want to use this - and in most cases probably not
Adds a lot of code (+tests)
There is another simple way to achieve this (custom script)

But keep it as a very nice reference how to solve this that hopefully others will find useful - and if we get more reactions, we can reconsider.
my 2ct

Fair enough, I agree that this could be considered too specific to my use case and therefore not really in style of the current lightweight implementation. If someone finds this useful, they can always refer to my fork.
Regardless, thanks for the discussion @j08lue & @dionhaefner


Bug: Interface implementations allow optional properties to be required

🔎 Search Terms
Interface optional properties required
🕗 Version & Regression Information
Bug exists for a long time, so no big new change for that here but I couldn't find a previous issue that discussed it.
⏯ Playground Link
https://www.typescriptlang.org/play?#code/JYOwLgpgTgZghgYwgAgJIHEoQpKyDeAUMsgOZY4AUIcAthAPwBcyAzmFKKQJQsBuAe2AATANyEAvoUIIANnFatkmbLmTBaAB1kR64JRgpqiJAPSnkAFQAWwJaAQCoWBGFkBPdVp16wyMNYooLjwSAA0bA5BfgqsAK709n4A7sCysshwsslw7kqkOJnINPTEZEbUdBAs7JwgPPxCwgRlJI4grAI6AHSyAqSUAAYAEhDpAsjCEHB4ACT4JRASg9ziJFJShObIAHIT0FBOyABGEAhwcawoAUHg0KEowgIQrCAA5H5YAI5xwFhFfCycWuE1OyE0sQgwhkAg6fnIqmgqDusEQ1TQKhw0GQAF5ihBksojNBKKtCAisVBkSE0d0KWBSeIthYAKLOI6nc6Xa6BZByWKZVxxLIeZDfX5YJRwZCA2TAmFw8qIqAAYXkilx+MJmNwjPJxNV6tYdIqqyAA
💻 Code
interface IGreeter {
  greet(name?: string): void;
}

class Greeter implements IGreeter {
  // This incorrectly implement the interface, since it assumes it will always get a name
  greet(name: string): void {
    console.log(`Hello dear ${name}`);
  }
}

// No error because the interface doesn't require a value to be passed
const greeterInterface: IGreeter = new Greeter();
greeterInterface.greet();

// Error because the class actually requires a value
const greeterClass = new Greeter();
greeterClass.greet();

🙁 Actual behavior
It allows you to implement the IGreeter interface incorrectly by requiring a parameter in a method which is optional according to the interface.
🙂 Expected behavior
An error would be expected in the Greeter class that indicates that the IGreeter interface is not correctly implemented.
Additional information about the issue
For strict mode, I can't think of a reason why this isn't throwing an error. It does throw an error if you define the property in the interface as  | undefined. There is a similar issue for interface properties, where public str?: string; doesn't require it to be implemented in the class but public str: string | undefined does.

Per https://github.com/microsoft/TypeScript/wiki/FAQ#common-bugs-that-arent-bugs :

Methods are always bivariant in their argument, while function properties are contravariant in their argument under strictFunctionTypes. More discussion here.

To make your expected behavior, you should declare greet as a function property.
interface IGreeter {
  greet: (name?: string) => void;
}

That doesn't explain why it works for string | undefined and not for ?: string.
I see that PR is merged in 2017 and the whole ecosystem got way more compatible with Typescript and strict typing, so it might be a good moment to re-evaluate and see if this change would still break a lot of code.

That doesn't explain why it works for string | undefined and not for ?: string.

No, it doesn't work for string | undefined for the same reason. There will still be an potential runtime error as below shows.
interface IGreeter {
  greet(name: string | undefined): void;
}

class Greeter implements IGreeter {
  // This incorrectly implement the interface, since it assumes it will always get a name
  greet(name: string): void {
    console.log(`Hello dear ${name.concat('')}`);
  }
}

// No compiler error because the method will accept undefined
const greeterInterface: IGreeter = new Greeter();
greeterInterface.greet(undefined);

//But you will get runtime error: Cannot read properties of undefined (reading 'concat')

Ah yes you're right, it only throws a compiler error if you call it like this (playground):
const greeterInterface: IGreeter = new Greeter();
greeterInterface.greet();

Which is also odd. Either way I still think this is an important bug, curious what the maintainers think since I also can find ways to work around it.

Well, this is another issue which is being discussed at https://github.com/Microsoft/TypeScript/issues/12400. At the moment, undefined parameter and optional parameters are not equivalent. You must explicitly provide an undefined value for the former when call the function.

way I still think this is an important bug

But it's not a bug. It's very much intentional, although unfortunate. The FAQ provides slightly more information: https://github.com/microsoft/TypeScript/wiki/FAQ#why-are-function-parameters-bivariant

Also from https://github.com/microsoft/TypeScript/pull/18654:

Methods are excluded specifically to ensure generic classes and interfaces (such as Array<T>) continue to mostly relate covariantly. The impact of strictly checking methods would be a much bigger breaking change as a large number of generic types would become invariant

Yes, code in the wild has gotten more strict-friendly since then, but as breaking changes go, making all collections invariant still seems like a bridge too far, even today.

so it might be a good moment to re-evaluate and see if this change would still break a lot of code.

Nothing has changed regarding the underlying problem here (arrays need to be covariant so that you can ever make one), so I don't see what a re-evaluation would do.


Ability to deploy UXP via OLM in view-only mode

What problem are you facing?
Currently, the permission level is set in build time due to constraints of not having an operator and we choose edit in build time but that doesn't allow users to install in view mode.
How could Universal Crossplane help solve your problem?
There are a few options:

Bootstrapper to manage RBAC objects, i.e. ClusterRoleBinding and ClusterRoles that we conditionally deploy.

The permission level would be set either on a new configuration CRD or in an existing ConfigMap we have, like the version ConfigMap.

Have different listings in OperatorHub; Universal Crossplane - Edit, Universal Crossplane - View, Universal Crossplane.

Not sure if it'd be allowed and it could be confusing.

Agent could have a ConfigMap that will allow users to toggle which groups to use.

upbound:view and upbound:edit would still be created but users would have the ability to configure which one(s) are allowed to be used through agent configuration.

Closing the issue as its not a focus area for now.


[4allportal]: WebDAV fails on paths with spaces and special characters

Probably needs quoting

Closed by #11


Unable to run built web version (App.js / dry-di)

Hello!
Changes in the source of 'App.js' after #1246c8996b108c286c1fae5d9e7cdf08cfd9f81c commit causes following error in browser after web version is built:

Uncaught Error: No viable providers for App. #126

After reverting back changes to previous version app runs without any problem.

Sorry about that... the current master is going through a major refactor. Since I'm generally the only one messing with the code and fixes are quick, nobody generally notices when I break master. :stuck_out_tongue:
If you don't mind me asking, is there any reason why you're building from source? Just trying to better understand how people use ABE.


[Feature]: Replace google short urls, since it will not work start of 25th August 2025

🚀 Feature Proposal
Replace google short links to something other (or use not shorten links)
Motivation
Google announced "Google URL Shortener links will no longer be available".
Jest uses Google short URLs at least in snapshot files. Example: Jest Snapshot v1, https://goo.gl/fbAQLP
Example
No response
Pitch
Start of 25th August 2025 all shorten links will not be work, so users will be confused by navigating to error page.

Not stale

Not stale

Not stale

Up

OK computer


service/header/p2p_exchange: Revert exchange protocol upgrade

Resolves problems reported in #511 (and consequently #510) for now as we pushed a protocol upgrade without handling for backwards compatibility with the latest release / devnet.
We will re-introduce this upgrade in the next release.

👌🏻
Hope we won't forget to bring it back

I'll make a PR immediately after we merge this.


make third_party problem finding jni.h despite $JAVA_HOME being set.

This one could just be user error.  On OS X I assumed that if $JAVA_HOME is set, (in my case using the following line in .bash_profile)
export JAVA_HOME=`/usr/libexec/java_home -v 1.7\`

then the headers for jni.h would be found in $JAVA_HOME/include.  It seams they are not and compiling the third_party fails with:
clang++ -fPIC -std=c++11 -mmacosx-version-min=10.8 -stdlib=libc++ -O1 -DNDEBUG -Isrc -Iexamples -Isrc/gen -I/Users/user/Documents/repositories/or-tools/dependencies/install/include -DARCH_K8 -Wno-deprecated -I/Users/user/Documents/repositories/or-tools/dependencies/install/include -I/Users/user/Documents/repositories/or-tools/dependencies/install/include -DUSE_CBC -I/Users/user/Documents/repositories/or-tools/dependencies/install/include -DUSE_CLP      -DUSE_GLOP -DUSE_BOP -I -I/darwin -c src/gen/constraint_solver/constraint_solver_java_wrap.cc -o objs/swig/constraint_solver_java_wrap.o
src/gen/constraint_solver/constraint_solver_java_wrap.cc:173:10: fatal error: 'jni.h' file not found
#include <jni.h>
         ^
1 error generated.
make: *** [objs/swig/constraint_solver_java_wrap.o] Error 1
MacBook-Pro:or-tools user$ echo $JAVA_HOME
/Library/Java/JavaVirtualMachines/jdk1.7.0_79.jdk/Contents/Home
MacBook-Pro:or-tools user$

You need to set the right variable in Makefile.local.
I guess you are not using the default java install.
Thanks
Le 15 déc. 2015 00:39, "Martin West"<EMAIL_ADDRESS>a écrit :

This one could just be user error. On OS X I assumed that if $JAVA_HOME is
set, (in my case using the following line in .bash_profile)
export JAVA_HOME=/usr/libexec/java_home -v 1.7\
then the headers for jni.h would be found in $JAVA_HOME/include. It seams
they are not and compiling the third_party fails with:
clang++ -fPIC -std=c++11 -mmacosx-version-min=10.8 -stdlib=libc++ -O1 -DNDEBUG -Isrc -Iexamples -Isrc/gen -I/Users/user/Documents/repositories/or-tools/dependencies/install/include -DARCH_K8 -Wno-deprecated -I/Users/user/Documents/repositories/or-tools/dependencies/install/include -I/Users/user/Documents/repositories/or-tools/dependencies/install/include -DUSE_CBC -I/Users/user/Documents/repositories/or-tools/dependencies/install/include -DUSE_CLP      -DUSE_GLOP -DUSE_BOP -I -I/darwin -c src/gen/constraint_solver/constraint_solver_java_wrap.cc -o objs/swig/constraint_solver_java_wrap.o
src/gen/constraint_solver/constraint_solver_java_wrap.cc:173:10: fatal error: 'jni.h' file not found
#include <jni.h>
^
1 error generated.
make: *** [objs/swig/constraint_solver_java_wrap.o] Error 1
MacBook-Pro:or-tools user$ echo $JAVA_HOME
/Library/Java/JavaVirtualMachines/jdk1.7.0_79.jdk/Contents/Home
MacBook-Pro:or-tools user$
—
Reply to this email directly or view it on GitHub
https://github.com/google/or-tools/issues/130.

I'm closing this as I have not encountered the problem since.


New package: Enlsip v1.0.0-DEV

Registering package: Enlsip
Repository: https://github.com/UncertainLab/Enlsip.jl
Created by: @pierre-borie
Version: v1.0.0-DEV
Commit: a1951d6d42e711ff0a748912de815a936de69c70
Git reference: HEAD
Description: Enlsip.jl is the Julia version of a Fortran77 optimization library designed to solve nonlinear least squares problems under general nonlinear constraints.

@JuliaRegistrator register()


failed precondition: error while reading resource variable block8_sepconv3_bn/gamma

Hello,
I have found very few documentation concerning the error : failed precondition: error while reading resource variable block8_sepconv3_bn/gamma from Container: localhost. This could mean that the variable was uninitialized. Not found : Resource localhost/block8_speconv3_bn/gamma/class tensorflow::Var does not exist
I guess the problem must be in the train function before the model.predict and has something to do with the sessions and threads but I would be happy to get any help I can get.
I use :
Python 3.7.8
Cuda 10.1
CUDNN v7.6.5
Tensorflow 2.3.0
keras 2.4.3
You can find my code in attachment or here : https://github.com/8rax/Carla/blob/master/JDO_Tutorial_5.py
Carla_Env_DQN.zip

Can you check https://github.com/tensorflow/tensorflow/issues/28287#issuecomment-495168033

Hello, I indeed had a look at that thread as well and tried the solution. But I might have done it wrong.
I created on line 70 my gpu_options and my sess at the begining of my script after the regular imports:
gpu_options=tf.compat.v1.GPUOptions(per_process_gpu_memory_fraction=MEMORY_FRACTION) #TENSORFLOW DON'T USE ALL MY GPU PLZ
sess=set_session(tf.compat.v1.Session(config=tf.compat.v1.ConfigProto(gpu_options=gpu_options)))

I understand I should also declare here the graph globally instead of declaring it in the init function of my DQNAgent and should therefore also put :
graph = tf.compat.v1.get_default_graph()

then, if my understanding of the problem is correct, I should then set the session every time I am calling the .predict method in the train function of my DQNAgent class.
I suppose that I should then use the following syntax:
global sess
global graph
with self.graph.as_default():
  set_session(sess)
  current_qs_list = self.model.predict(current_states, PREDICTION_BATCH_SIZE)
  new_current_states = np.array([transition[3] for transition in minibatch])/255

global sess
global graph
with self.graph.as_default():
  set_session(sess)
  future_qs_list = self.target_model.predict(new_current_states, PREDICTION_BATCH_SIZE)

Not sure I should add it here though as there is no graph involved :
def get_qs(self, state):
        return self.model.predict(np.array(state).reshape(-1, * state.shape)/255)[0]

would that make sense?

I think that it is not a bug so you can close this.
But if you need a specific support on your use case I suggest you to use our channel at:
https://stackoverflow.com/questions/tagged/tensorflow

Hi,
Alright thank you for the suggestion, posted it there : https://stackoverflow.com/q/64004994/14319427


Fix pynecone.json version issue when web init(#884)

All Submissions:

[x] Have you followed the guidelines stated in CONTRIBUTING.md file?
[x] Have you checked to ensure there aren't any other open Pull Requests for the desired change?

Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)

New Feature Submission:

[x] Does your submission pass the tests?
[x] Have you linted your code locally prior to submission?

Changes To Core Features:
After these steps, you're ready to open a pull request.
a. Give a descriptive title to your PR.

This pull request is related to the issue https://github.com/pynecone-io/pynecone/issues/884
Sometimes when the core developer upgrades the version, they must remember to change the pynecone.json in the template by hand. And sometimes, we may forget it.
We must prevent the application developer from getting the wrong information from pynecone.json.
Having an intuitive way to have the correct package's version in the .web/pynecone.json is better.
And let's put this action in the web's init.
b. Describe your changes.

c. Put `closes #XXXX` in your comment to auto-close the issue that your PR fixes (if such).

Thanks! ~ You are so nice! I very much enjoy contributing pynecone.


Resources cannot be loaded when running examples from the IDE context

https://youtrack.jetbrains.com/issue/KT-24463
Workaround: Use the provided gradle tasks.

While still a bug, it's not an issue for three.kt anymore after we moved glsl code into the source


phpixie.com broken

Sorry for my off topic issue.
certificate which used in phpixe.com seem to be broken.
So we can not access it.
My chrome said : ERR_CONNECTION_RESET
Would you like to fix it?
Thanks.

Fixed! Thanks!


no-tick Post release branch version bump, per versioning standards.

Bumping to 1.5.0 on master after the release of 1.4.0.

@Fraser999 has made a tool which bumps all versions but also crates versions. Not sure if we should use it for this PR. Here is how I've run it
cd ci/casper_updater/
cargo run --package=casper-updater -- --root-dir=. -b minor
Exsample PR after running it:
https://github.com/CasperLabs/test-casper-node-cicd/pull/6

Tried this and it bumps ALL libraries.  Not sure if we wanted to do that.  I know we were trying to only version those when they changed before, but that might have changed.  I did run his tool, but omitted the -b minor to not bump everything.
We might want to discuss this wider on a sprint entry sidebar on Monday, re: version bump of libraries..

bors r+


Some checksum files point into subdirectory target/**/*.tar.gz

This means that a sha265sum -c *.sha256.txt fails if the tar.gz was downloaded into the same folder as the sha256.txt.

Dup of https://github.com/adoptium/temurin-build/issues/3058


chart.series.addPoint is not a function

Cannot get many functions in Highcharts API to work. Getting errors like 'chart.series.addPoint is not a function' and 'chart.addPoint is not a function' for addPoint, setData, etc.
var chart = this.refs.chart.getChart();
chart.series.addPoint({
    color: "#000",
    name: "Adding A Point",
    value: "$ 1234.00",
    y: 1234.00
});

Hi Miapoulos,  what about chart.series[0].addPoint ?

That did it, thanks!! Was following the example... might be worth it to update it.

Which example are you talking about?

In the README.md:
componentDidMount() {
let chart = this.refs.chart.getChart();
chart.series.addPoint({x: 10, y: 12});
}

Thanks a lot, I have updated the readme!


android: update compiler to use the "new" toolchain

Update the Android support in the compiler to use of the new official Android toolchain based on gradle and CMake, it replaces the old toolchain based on ndk-build and Android.mk files. The use of the new toolchain makes it easier to connect with libraries, it follow the current examples, it is a bit faster to compile, it automatically downloads the SDK files for the target platforms and it has a decent quiet compilation mode!
The new toolchain brings the following changes to the generate Android projects:

It adds a new project layer above the app layer, this results in moving most of the generated code down in the app/src/main folder.
It moves some settings (API levels, etc.) out of AndroidManifest.xml to a build.gradle.
It replaces the makefiles Android.mk by CMakeLists.txt.

I took advantage of these changes to bring back the GC support for Nit on Android. The new toolchain executes the standard libgc/bdwgc CMakeLists.txt with only small modifications (based on the Unity fork) and the GC works as expected on Android (with thread support). Having a working GC required tweaking some existing Nit code that did not take into account a functioning GC in order to correctly pin references to Nit objects from Java.
I've also updated the full Docker image to support the new toolchain.
Note that the new Android toolchain supports only 64 bits host platforms with Java 8 JDK.
Future work:

Support more ABIs, incl. arm64 and x86_64, they should be activated only when targetting their minimum supported platform.
Compile libunwind with its CMakeLists.txt.
Support adding extra lines to the gradle files, or accept full gradle files from clients.

test windows please
test osx please

test this please

The tests should be fixed, compileSdkVersion is now set to the value of targetSdkVersion.

Changed the config so the lint trigger on compiling a release version does not abort the build.
test android please

test android please

test android please

test this please
test android please

Not happy about gradle and all the new dependencies :(

Sorry about that, this change was required to support recent Android libraries. @rgh36167 did you manage to get it working on your system?

Linux seems working after ignoring the errors, Android needs downloading all the stuff.

is there some way to cache the gradle tarball locally so it won't be redownloaded too often? Thinking that pointing gradle-wrapper.properties:distributionUrl to a local file might work but not yet tested.
Likewise, the boehmgc repos could be cloned localy and only checked out into the buidltree. May seem like peanuts but as things tend to grow it will make a difference.

I'm not sure I see your point. Gradle should already be downloaded only once and cached in ~/.gradle/wrapper/dists/gradle-4.1-all. Similarly, boehmgc should be cloned once and reused by all Nit projects targetting Android.
However, Gradle auto-downloads different Android SDK depending on the API target of each project. It's a bit long at first, but they are cached too.
If you get more redownloads, there may be a problem with our scripts...


Property reported as missing when using enum

Flow version:
I use 0.93, the issue seems to be introduced in 0.75
Expected behavior
No error, because Action type should be refined to the first branch
Actual behavior
Cannot get `action.foo` because property `foo` is missing in object type

Link to Try-Flow or Github repo:

https://flow.org/try/#0MYewdgzgLgBAKgTwA4FMIwLwwN4CgCQAYgPLEBcMAFAOQnHUW2nUCUANAcXABICiASgH06FGlz5C6DGNXEDhzdgQBCAQX6jqa-tK3rWHAL65cUZChiEQIRKky4YMAD4wzqEADN45iADo6Ds6u5p7eqH5ykqQmbhaqwFAAluD2ji7YsRRWNuZsMADUHtYA-BTQAE6JYADmxmk4mcHuXrZovtrGuB4ArmAJyWAwSOUgwGgQlACG-eAU8UngeY3ZrSw4gRAA7olQwAAWVNMLYL6xa3iOjsCTEBaZgZcw5ShQ3eWDRwO+RSAPMAAmKA8k26ABsoGQ-o5nq93jAwGDQYFjIYgA
The problem magically disappears if I add  || '', but it's a violation of sketchy-null lint rule :
https://flow.org/try/#0MYewdgzgLgBAKgTwA4FMIwLwwN4CgCQAYgPLEBcMAFAOQnHUW2nUCUANAcXABICiASgH06FGlz5C6DGNXEDhzdgQBCAQX6jqa-tK3rWHAL65cUZChiEQIRKky4YMAD4wzqEADN45iADo6Ds6u5p7eqH5ykqQmbhaqwFAAluD2ji7YsRRWNuZsMADUHtYA-BTQAE6JYADmxmk4mcHuXrZovtrGuB4ArmAJyWAwSOUgwGgQlACG-eAU8UngeY3ZrSw4gRAA7olQwAAWVNMLYL6xa3iOjsCTEBaZgZcw5ShQ3eWDRwO+RSDOLtTUB4wAAmKA8k26ABsoGQgY5nq93jAwFDIYFjIYgA

Wow, I actually reported the same issue a while ago =D
https://github.com/facebook/flow/issues/6609


Repost feature enhancement

I find this feature in instadub.
for the one who grow repost and paid promote account, this is very useful to steal followers.
the idea is to download tagged images from another account

and upload it in ours. include caption and tag or mention to the owner of image.

Done in current version.


Migrate existing content, start expanding documentation

Summary
Migrates existing wiki content (now disabled), and starts expanding content. First of several docs PRs.

This is awesome!

@autata I went ahead and pulled in your content from https://github.com/slackhq/astra/discussions/799 as part of this, which you can see here - https://slackhq.github.io/astra/getting-started.html.
These are (mostly) Markdown files located in the docs folder in this repo, and use Writerside if you want to propose any changes.


UI improvements

Hi Yall! I have some starting point feedback/questions. I'd love to provide more granular feedback once we align on some of these functional details and feasibility.
Blog MVP UI Feedback
Figma Design here
Blog Landing Page

Blog Posts become Cards, matching the design of the News cards from the home page.
How many posts appear on page load?

Can we start with 9 and lazy load using “Load More” button?
Can we have the latest post appear as a larger module at the top of the page?
Stretch goal: This module could be manually controlled and admin can select a specific post to be featured here.

Blog Post

Are we locked into the layout inherited from the Posts or can we override this?

Referencing: The author/date appearing above the image, the line at the left of the post

This is the design I’d like us to get closer to
I'm not able to see what the typography waterfall is currently (what H1, H2, H3, P is set to currently). We have these variables defined in the Design System (DIG). Is it okay if I provide feedback based on the Design System?

TLDR: I'll provide a typography waterfall based on the DIG?
What are the Markdown variables that posts support/has stylistically defined?
I’d like to start with

H1 [Title]
H2
H3
P [Body]
Link style
Caption style

Hey Vanessa, Thanks for the feedback!

We can have the posts appear as cards, which elements in the figma are you referring to and I should be able to get what we have in line with the design

I think we would implement pagination like the existing blog, but infinite scroll could work here too as we already do that with existing posts.
a. load more button is also an option, whichever makes the most sense. I'd say leveraging the infinite scroll is the easiest followed by the loads button and then pagination.

Providing feedback in that way is fine, but I am currently unfamiliar with it. (This can help bring me up to speed!)

We are not locked into the Post style layout. We're leveraging existing post widgets, but can create more blog centric ones to align with the link you provided.

As for the markdown here is a link to what markdown supports. we do have some ability to style the supported elements as we see fit in our widget style which will be implemented with standard css.

Blog Post Design
I've updated the Figma to show designs for each breakpoint and our typography

Layout:

On desktop the page text spans the 8 center columns of the page container.
On smaller breakpoints the text spans the full page container width

2. Cover image should be constrained to/cropped to a 2:1 aspect ratio. This will give us some visual consistency for the template.
3. Title text inherits the H1 style. All type styles are defined for desktop and mobile in Figma.
4. Post details (author and date) should sit below the cover image. And remove the left-border-line inherited from the social post.

@vanessadespain @jackson-harris-iii  from those suggestion, here are the UI improvements I recommend we delay to post-MVP.

Load more scrolls (for MVP we can limit the number of posts)
Latest post as the top featured (for MVP we can hard code a specific blog post, ie. chain abstraction)

Any objection or additional suggestions of things to save for later?
This way we can work toward deprecating wordpress faster, and finishing re-post.

Sounds good to me!

Hey @jackson-harris-iii ,
Reviewing the latest on bosblog. Here's some feedback to help guide improvements to match the designs.
Blog Landing Page:
Figma Link

Page layout grid:
We should use the same page grid from marketing pages. On desktop the max-width of the page content is 1224px. Currently we're extending beyond this, a good measure is to look at the left and right edge of the navigation — page content should not extend beyond this.
In production reference:

Latest Post:
Should be 42px Medium FK Grotesk (in DIG "text-3XL-medium")
Individual Post Cards:
Instead of image and text side-by-side horizontally, these should stack vertically. See original comment here
Post title should be FK Grotesk, 24px Medium (in DIG "text-XL-medium")
Post date should be below post title, Mona Sans 14px Regular (in DIG "text-s-regular") using the date format Mon DD, YYYY (Aug 12, 2024) instead of 3/4/24.
Remove gray background

Blog Post:
Figma Link
https://www.figma.com/file/nqgzMvVASTSHVqUQPFQWsV/WORKING-%2F-Q1-'24-%2F-Website---v3.2024?type=design&node-id=585%3A29426&mode=design&t=wMRhE1UovKLYukwq-1
Page layout grid:
We should use the same page grid from marketing pages. On desktop the max-width of the page content is 1224px. Currently we're extending beyond this, a good measure is to look at the left and right edge of the navigation — page content should not extend beyond this.
In production reference:

• The image should fit the max-width of the page (1224px on desktop).
Post Typography
Link to the type waterfall
Title: Should be 42px FK Grotesk Medium (in DIG "text-3XL-medium")
Date: Should be 14px Mona Sans Regular (450) (in DIG "text-s-regular")
Post Author: Should be 14px Mona Sans Regular (450)  (in DIG "text-s-regular")
Post Author Handle: Should be 14px Mona Sans Bold (in DIG "text-s-bold")
Text link color: Should be hex6E56CF (in DIG "Violet-light-9")
Body text: Should be font-weight 450 Mona Sans 16px (in DIG "text-base")
List items: Add more spacing between line items when using the bulleted list  .... Can we add 8px margin-bottom to list items? Should help these be more legible. Can these use 14px Mona Sans Regular (450) (in DIG "text-s-regular")
More Markdown Examples of Typography
I couldn't find post examples that use different headline styles (H1, H2, H3, etc) Is this something we can do so that we can demonstrate the possibilities of type sizes to the NF Marketing team? For example the Etherland post could use H3s where we currently use bold text for headlines in the post body.

@vanessadespain The UI updates are live on prod now. Here's an example post can you give this another once over and let me know if there is any additional feedback you have. Thanks!

Priority UI Update:
@charleslavon  and I chatted about this one. He was able to show me the "Promote" flyout menu. We chatted about trying to simplify this a bit by combining the "Promote to blog" and "Repost to Activity Feed" into one menu. So the option would be "Repost" (or Promote if you decide to keep as is) and the secondary menu items would be "Activity Feed" and "My Blog".
Current implementation:

From Design:

In my design explorations I had looked at placing repost in the 3-dot menu and noticed the similarity between "Repost" and "Report" as well as decreased discoverability of placing Repost in this menu. I went for placing the Repost in the tray of icons alongside Comment, Share, etc. Food for thought if there's interest to push this a bit further.
Nice to have UI Improvements to match Design:

The width of the Title + Author section should match the blog post width. Instead the min-width seems to be determined by the length of the title. Longer titles DO match the width of the blog post.
Short titles:

Date and Author text should be 14px (text-small)
Author name should be bold while the Author @ handle should be regular weight.
Body text should be font-weight 450
Smaller breakpoints should have an external margin of at least 24px so the Image and Back to Posts button don't touch the edge of the view port. In Design the smaller breakpoints the post body and the image have the same width.

I couldn't verify any other headline type styles in the page body.

Thanks for the guidance @vanessadespain.  @jackson-harris-iii let's go with the removal of the "Promote" menu item, replacing this with the "Repost" icon which has two options -> Activity Feed, My Blog


StyleBoxTexture does not propagate its Texture2D's changed signal

Tested versions

Reproducible in v4.2.1.stable [b09f793f5]

System information
Godot v4.2.1.stable - Windows 10.0.19045 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 4060 Ti (NVIDIA; <IP_ADDRESS>29) - 13th Gen Intel(R) Core(TM) i7-13700KF (24 Threads)
Issue description
StyleBoxTexture does not connect its Texture2D's changed signal to its own, so modification to the texture aren't automatically propagated.
Steps to reproduce
Using the attached project:

Run the project and click "Update Texture".
The button on the right won't have changed, mouse over it to force it to redraw and show the change (the button's icon toggling between showing the left and right side).
Click "Toggle Connection" to connect the texture's changed signal to the stylebox's emit_changed.
Now clicking "Update Texture" will immediately change the button on the right.
Click "Toggle Connection" again to disconnect the changed signal.
Now once again clicking "Update Texture" won't show any change until you mouse over the button on the right.

Minimal reproduction project (MRP)
StyleBoxTexture-Changed-Signal-Propagation.zip

This is true for most resources though, the ones that propagate change are the exception

From a surface level, all the other inbuilt StyleBox propertues are non-reference Variants, StyleBoxTexture's texture property is the only StyleBox property that could emit a changed signal, IDK how the the Godot team thinks of this, kinda seems like a unique case and it feels like it be more intuitive, least to me, for this case to be part of the exception.

More so than Material? Or MultiMesh? Or GradientTexture?

Sure, but aren't there already exceptions in some cases for this type of behavior?

Very few, why should this one be an exception?

I think it would depend on the justification for the other exceptions before it could be reasoned if this would be a valid concern then.


Open a new website on another new tab

please I have a problem, I use the driver to open a website and I want to open another new website that I want on another tab, how do I do that?

I tried using this sentence but it gives the following error  ->  driver.script("window.open('https://www.w3schools.com')")

@anhteo21499 please ask questions like this on stack overflow: https://stackoverflow.com/a/66813293/143475

Thank you I did it


orderBy doesn't work

Laravel-mongodb Version: 3.8.4
Laravel Version: v8.50.0
PHP Version: 8.0.8
Database Driver & Version: mongodb/mongodb 1.9.0

Description:
orderby doesn't work
Steps to reproduce

$query = Model::where('blah', 'blah')->orderBy('age', 'desc');

Expected behaviour
should return a query
Actual behaviour
an exception is raised
ErrorException
Undefined array key "column"
Logs:
Illuminate\Foundation\Bootstrap\HandleExceptions::handleError
vendor/laravel/framework/src/Illuminate/Database/Query/Grammars/Grammar.php:740

nevermind, looks like there was some issue on my code. I swear I've been testing it for a lot but it seems it was my fault

It's working, i think you need to check your name of columns. Then you should to add "->get()" end of query. If you don't add "->get()" it returns "Jenssegers\Mongodb\Eloquent\Builder".

But this does not work for yajra datatables with mongo db ....on adding order by it returns access value of array offset in int when doing order by


fix: Ensure executor is shutdown

Description

Calls shutdown on the executor in batchCheck
References

Fixes #132
Review Checklist

[ ] I have clicked on "allow edits by maintainers".
[ ] I have added documentation for new/changed functionality in this PR or in a PR to openfga.dev [Provide a link to any relevant PRs in the references section above]
[ ] The correct base branch is being used, if not main
[ ] I have added tests to validate that the change in functionality is working as expected

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 31.72%. Comparing base (5c39f01) to head (972cc80).

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #133      +/-   ##
============================================
+ Coverage     31.71%   31.72%   +0.01%
  Complexity      885      885
============================================
  Files           169      169
  Lines          6310     6311       +1
  Branches        699      699
============================================
+ Hits           2001     2002       +1
  Misses         4207     4207
  Partials        102      102

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

@jimmyjames can we make sure we're doing the same in non-transactional writes?

Yep; there's no other usage of executor in the SDK.


Courses show up twice on student view

It seems like each course is showing up twice.

Investigating now.

@albert12132 Is it possible that you were enrolled twice in a course? I'm unable to reproduce the error, and my account with the deployed version doesn't have problems. It'd still be a bug, but at least I'd know what to fix.

Maybe because he's both staff and a student (at least he appears on both
the staff and student lists of the currently deployed version)
On Sun, Jun 21, 2015 at 8:37 PM, Alvin Wan<EMAIL_ADDRESS>wrote:

@albert12132 https://github.com/albert12132 Is it possible that you
were enrolled twice in a course? I'm unable to reproduce the error, and my
account with the deployed version doesn't have problems. It'd still be a
bug, but at least I'd know what to fix.
—
Reply to this email directly or view it on GitHub
https://github.com/Cal-CS-61A-Staff/ok/issues/474#issuecomment-113994384
.

Ohh, that's probably it. Thanks.

John's correct. Removing Albert as a student fixed it.
On Sun, Jun 21, 2015 at 8:45 PM, Alvin Wan<EMAIL_ADDRESS>wrote:

Ohh, that's probably it. Thanks.
—
Reply to this email directly or view it on GitHub
https://github.com/Cal-CS-61A-Staff/ok/issues/474#issuecomment-113994871
.


fix StateDot

PR includes

[x] Bug Fix

Related issues

Resolve #1004
QA Instructions, Screenshots, Recordings

/canary


(OraklNode) Update redis wrapper for sorted set

Description
RDB sorted set wrapper to be utilized from trading volume fetcher
Type of change
Please delete options that are not relevant.

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

Checklist before requesting a review

[x] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.

Deployment

[ ] Should publish npm package
[x] Should publish Docker image

Won't be using this feature


About Data processing

Hello, I would like to ask you some questions. In the part of video data processing, I use UCF-101 data set, how to use the script? I didn't understand the official information.The part I don't know how to manipulate is here:
................................................................................................................................................
A custom format is used for the train/val label files of each datasets:
This can be done through the scripts provided in labels
( If I use the ucf101 dataset, how do I generate labels using scripts? What do time_start and time_end mean here?)
We have tested our code over the following datasets:
Conversion of videos to SQLite3
(How to convert a video to SQLite3?
Instead of extracting video frames stored as image files (//etc.) that dramatically increase the number of inodes use, we use files for each video and store frames as BLOBS..png.jpeg.db
You can use the pypi package or repo to convert video files to SQL:dataset2databse
..............................................................................................................................................................................................................................
I don't know much about the code, and I really hope to get your help. I really appreciate it，Looking forward to your reply! If possible, I hope to have your contact information, wechat or ins, etc

Hi @ya-92626 , the latest commit loads video files and folders containing frames directly. So you can instead use this if you weren't able to store videos as SQL blobs.


ssync: collision retry

depends on PR #27

I've been running 'round the clock random 10GB volume sync tests for 10 days: no issues found


docs: removed reference to language tabs

Description

Removed references to language tabs from PlaceholderAdminMixin documentation.
Related resources

Outdated PlaceholderAdminMixin docs

Checklist

[x] I have opened this pull request against develop
[x] I have added or modified the tests when changing logic
[ ] I have followed the conventional commits guidelines to add meaningful information into the changelog
[ ] I have read the contribution guidelines  and I have joined #workgroup-pr-review on Slack to find a “pr review buddy” who is going to review my pull request.

Hi @fsbraun, thank you for your guidance! Excuse me as I am still learning about the codebase. I took a look at how the language_tabs context could make its way to admin/placeholders/placeholders/change_form.html and my understanding is that this would be set in FrontendEditableAdminMixin:edit_field. However, you mentioned this might need to be set in PlaceholderAdminMixin so am I missing something? Also, is there any test case I can use to verify any of these changes?


Linking a descriptive task (for design to review and us as a team to use) and link it to a specific work item in the VA repo

https://github.com/microsoft/botframework-solutions/issues/3510


Move build script to NUKE

[x] Removes FAKE build script
[x] Adds NUKE build script
[x] Implements coverage report generation
[ ] Implement original versioning schema

I apologize if this piece of feedback comes in a bit late, but, as a member of the @cake-build project, I think it would be more advantageous to implement our build process in Cake. Putting aside any discussions about which one is "better" or "worse", if nothing else, I feel I can provide better support for Cake, being familiar with how it works.
If everyone in @AutoFixture/core agrees, I'll be happy to convert the existing build process to Cake.

@ecampidoglio this has been discussed in #1206 to some extent. Honestly it's hard to say anything without touching on the subject which is better.
The original intent here was to make the build script more accessible and avoid the bus factor. Moving to Cake I think creates more of the same issues we currently have.

@aivascu I reviewed the conversation in #1206, but I couldn't find any direct comparison between Cake and Nuke.

Moving to Cake I think creates more of the same issues we currently have.

Could you be more specific? Also, let me clarify that I'm not trying to be confrontational here. Just wanting to understand the perceived benefits of using Nuke.

@ecampidoglio don't get me wrong, Cake is good enough tool. I have used it in the past and I'm grateful for it, however my experience with it has not been so smooth. I haven't used Cake for almost a year so my opinions might be outdated.
One of the most impactful things for me is that is uses a special flavor of C#, which is not supported out of the box on any editors and the plugins on VS and VS Code were not working most of the time. NUKE on the other hand has native support on any editor since it uses vanilla C# projects.
Then there's the adapter libraries for things like GitVersion, that were most of the time outdated or had environment requirements. NUKE though has build in support for most tools and there's little need to seek third-party libraries.
Another thing is the documentation. It's true that both tools mostly rely on generating the documentation from source but NUKEs documentation seems more complete.
Finally there are some features that I haven't seen implemented implemented in other tools, like script generation for different CI/CD environments.

I also give my voice to usage of Nuke. I've been using all of them in different projects and Nuke is the best because you have native IDE support for it no matter where you open your solution. There is no need to learn any DSL or install any plugin - you just run it as a regular console app. The same if you need to debug - just put a breakpoint and that's it, as it's just a console app.
After giving it a try I cannot vote for anything else anymore 😊

@aivascu @zvirja Thank you for sharing your opinions. I see your points.
As the appointed lead maintainers of AutoFixture, I think you have the right to choose the build automation tool you're most comfortable with. If that happens to be Nuke, then so be it. 🙂
Obviously, I would have rather seen that AutoFixture used Cake, but that's just my preference.

@zvirja I'm not that familiar with AppVeyor. Could you give me a hand with the test results upload?

@aivascu you can check my repos (like this one) where I combine Cake with AppVeyor :)

will reopen after more testing


Better selection macro

[ ] selection! macro add arbitrary values Eg. selection!(user, { __type: "user", name, email }); - This would be cool but may not be possible
[ ] selection! macro subselection Eg. selection!(userWithPets, { name, email, pets: { name, email });

Maybe move this out into Specta?


Flaky test: TestStreamSecretsForGatewaySds

This blocked a daily build of master: https://prow.istio.io/view/gcs/istio-prow/pr-logs/pull/istio-releases_pipeline/1219/unit-tests_pipeline/8

@pitlv2109 since you add this test recently, could you please take a look?

https://prow.istio.io/view/gcs/istio-prow/pr-logs/pull/istio_istio/16800/unit-tests_istio/859 is another example

https://prow.istio.io/view/gcs/istio-prow/pr-logs/pull/istio_istio/17550/unit-tests_istio/1503

https://prow.istio.io/view/gcs/istio-prow/pr-logs/pull/istio_istio/18989/unit-tests_istio/4111

https://prow.istio.io/view/gs/istio-prow/logs/unit-tests_istio_postsubmit/3413

https://prow.istio.io/view/gs/istio-prow/pr-logs/pull/istio_istio/26779/unit-tests_istio/19406

it seems the same the connection are not cleaned:
expect "total_stale_connections" to be 0, got 1.000000
deeping dive into the real root cause

Just wondering why the unit test testing both GatewaySDS and WorkloadSDS not flaky
https://github.com/istio/istio/blob/07782fd8d2ff0225af494b172dc459e5c9ace04a/security/pkg/nodeagent/sds/sdsservice_test.go#L113
but the unit test that only testing the GatewaySDS works https://github.com/istio/istio/blob/07782fd8d2ff0225af494b172dc459e5c9ace04a/security/pkg/nodeagent/sds/sdsservice_test.go#L102

I run this flaky test TestStreamSecretsForGatewaySds for about 100 times in my own linux. but it doesn't fail any time

The root cause is that before each test the staledClientKeys and totalStaleConnCounts should be reset


Can't use Workspace if I changed account

Describe the bug
Can't use Workspace if I changed account
To Reproduce
Steps to reproduce the behavior:
Need 2 accounts

Login via first account
Go to Workspace
Click select element
Click "Personal profile"
Logout
Login via second account
Go to Workspace
See message:

You need to be logged in as an admin

And DashboardNavigationState variable from localStorage contains wrong slug value.
Expected behavior
My second profile instead of the message.
Screenshots

Desktop (please complete the following information):

OS: [e.g. iOS]
Browser: [e.g. chrome, safari]
Version: [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser: [e.g. stock browser, safari]
Version: [e.g. 22]

Additional context
Add any other context about the problem here.

Triage Template (core team only)
This template is for members of the team to triage for prioritisation. For more guidance see https://www.loom.com/share/369ab467fbc64dec848085d38ff57ca0:
P1 high frequency, high impact
P2 low frequency, high impact
P3 high frequency, low impact
P4 low frequency, low impact
Examples of high impact - a problem affects users during an essential process (expenses and payments > onboarding and registration > contributing) with no workaround.
High frequency - >10% of users affected (measured as a proportion of total potential users for this case).

cc @gustavlrsn

Thanks for your detailed reports @NikitaKorevo!

@NikitaKorevo a followup question:
Given that you use the same account to administer multiple entities on the platform (collectives, organizations, hosts, funds, etc.) and that now, with Workspace, it is easy to switch between them, why do you need multiple platform logins?

@iamronen  I only used the second account to access the fiscal host settings as admin. I just forgot that I can join the fiscal host as admin from the first account.


Add support for multiple Jira instances

Multi instance support
This PR adds support for multiple Jira instances. The configuration format can be used as today, or by adding an instances array of Jira instances, as described in the README changes.
Context
For companies using multiple Jira instances, this PR enables them to use this. One instance can be called default, in which case entities don't need to specify the instance, ~and for other entities, the annotation jira.com/instance is used~ otherwise the project-key can be prefixed with instance-name/ similar to the SonarQube configuration.
Issue ticket number and link

Fixes #198

Checklist before requesting a review

[x] I have performed a self-review of my own code
[x] I have verified that the code builds perfectly fine on my local system
[x] I have verified that my code follows the style already available in the repository
[x] A changeset describing the change and affected packages. (more info)
[x] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)

Fixed the prettier and build issues, and also changed from having a separate annotation for other-than-default Jira instances, to be able to prefix the project-key annotation with the instance name instead.
The reasoning was to be closer to how other plugins do it, but also that when looking closer at your code, it seems that you might want to allow multiple project keys in the future (being comma separated). Coupling the instance and key in the same annotation will allow an entity to (again, in the future) be able to specify multiple projects in different Jira instances.

Nice, thank you! Looks good so far, I'll try to take a closer look soon!

This is a very valuable enhancement
My organization is looking for this feature
Thanks  @grantila for taking this task

Thank you @grantila, that's awesome 🎉🎉🎉
@anicke et al., what are the current plans for the next release ?

@chicobento The github releases are broken but all packages have been published to npm! So please try it out!


mod_set_forwarding crashes [JIRA: RIAK-1459]

handle modstate={deleted,_} in riak_kv_vnode:mod_set_forwarding/2

Adding to Jira to track bug found in testing 2.0.4 and with issue in GH.
[posted via JIRA by Deb Rakow]


[hotfix][runtime] Fix log info

What is the purpose of the change
Fix log info
Brief change log
Fix log info
Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers:  no
The runtime per-record code paths (performance sensitive):  no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper:  no
The S3 file system connector:  no

Documentation

Does this pull request introduce a new feature?  no
If yes, how is the feature documented? no

@wuchong @JingsongLi  Hi, can you review it?


dotty port

Co-authored-by: jodersky<EMAIL_ADDRESS>Co-authored-by: manojo<EMAIL_ADDRESS>

Still work in progress, the PR was created for convenient diffing

Hello! Glad I found this :)
What is the status of this work? If it needs help, I would like to volunteer my time :)

I have a fork at https://github.com/edwardcwang/scalatags with a proof of concept that works for dotty. It would be nice to be able to get official upstream Scala 3 support though :)

@edwardcwang could you open a PR here? It looks like your branch is furthest in getting Scala 3 working.

Hi all, sorry for responding so late! Indeed, apologies to @keynmol, @edwardcwang  and @nightscape for not being active. Unfortunately I've been quite busy recently, and I'm not sure to guarantee time at this point. Very happy that you are taking over, I'd be happy to help in any reviews if you're taking over!

Looks like someone opened #226

Closing this as superseded by #226


OpenWeather UV values above 10

OpenWeather UV indexes over 10 generate an error message.   I'm not sure about OpenWeather, but I know NOAA now uses values above 10.  11 is the maximum now.
_[7/27/2022, 4:44:17 PM] [homebridge-weather-plus] This plugin generated a warning from the characteristic 'UV Index': characteristic was supplied illegal value: number 11 exceeded maximum of 10. See https://homebridge.io/w/JtMGR for more info.
[7/27/2022, 4:44:17 PM] [homebridge-weather-plus] This plugin generated a warning from the characteristic 'UV Index': characteristic was supplied illegal value: number 11 exceeded maximum of 10. See https://homebridge.io/w/JtMGR for more info.
[7/27/2022, 4:44:17 PM] [homebridge-weather-plus] This plugin generated a warning from the characteristic 'UV Index': characteristic was supplied illegal value: number 11 exceeded maximum of 10. See https://homebridge.io/w/JtMGR for more info.
_

Fixed in v3.2.7


Fix [Nexus] tests

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @PyvesB!

  Generated by :no_entry_sign: dangerJS against aee4101d6323be334add3727018803e47d0dd423


Keyphrases retrieved do not match regex

Since stopword prefixes and suffixes are removed from the keyphrases, some keyphrases do not match the provided regex.
Let's say my regex only selects group of exactly two nouns. The output could contain one-word keyphrases if some keyphrases contain stop words.
This is by design but it is a problem for our implementation.

You could just define stop_words=None which doesn't remove any stop words. Would this be an option for you?

The v0.0.8 release allows to set stop_words=None and should solve this issue.


Remove redundant getTable call during getPartition/getPartitionByNames

I am concerned about potential inconsistency implications of this change. (I am not saying there are any.)

I don't think such design was intentional. In some cases caller does have Table at hand, but doesn't use it

I don't think such design was intentional. In some cases caller does have Table at hand, but doesn't use it

I understand. But now the Table becomes bound in the cache key, extending object lifecycle.
And this leads to another concern -- memory footprint of the Table objects referenced by the cache.

And this leads to another concern -- memory footprint of the Table objects referenced by the cache.

These table objects would be mostly there already in io.prestosql.plugin.hive.metastore.cache.CachingHiveMetastore#tableCache

Is it possible to add a test for the Avro schema change? Do we have tests for the other variant?

The second commit title is a bit long. Let's change to

Remove redundant getTable call when fetching partitions

Do we have tests for the getPartitionsByNames variant?

There is TestAvroSchemaUrl#testPartitionedTableWithLongColumnType.

Is it possible to add a test for the Avro schema change?

I think the only sages of getPartition were during adding/updating of partition. I'm not sure I can repro that in Presto

AC @electrum moved code to HiveSplitManager and HiveWriteManager


fixed the issue #212 which will most likely fix the issue #217 as well

@maximusunc or @kennethmorton Could one of you do a quick review of this PR? Once it is approved, I can deploy it to dev instances for @karafecho to test.

All tests have passed. This PR is ready for review now.

@maximusunc Have addressed your code review comments. See if they look good now.

@maximusunc I have added the check for equal operator to make sure the value exists in the table column for computing feature associations. I believe this will fix the issue #217. Please take another round of code review when you get a chance.

@hyi : Your explanation makes sense to me, and I agree that it will address multiple issues. Thanks for giving this some thought.


Feat:-Allowed tests to skip asynchronously in Jest circus

I tried to make a small proof of concept, about asynchronously skipping tests as discussed in #8604. this seems a reasonable change to me
As we saw in #8604 and #7245 when using jest for e2e tests with e.g. puppeteer or detox, there are many valid use cases for wanting to abort tests programmatically based on previous test results. To enable doing that asynchronously, the API needs to add a skip function that is available during test execution and skips the test when called.
This PR adds a skip: () => Promise function to the testContext in jest-circus, so it can be called as return this.skip() in the test fn to skip the currently running test. Alternatively the skip function could be passed to the test fn as a second argument, similar to how done is passed (this might be the more modern approach, as it would support arrow functions).
The main point is to demonstrate that the implementation of this in jest-circus is pretty trivial and adds minimal complexity as all that skip needs to do is to dispatch the test_skip event and prevent the dispatch of test_done.
Alternatively a new canceled state could be added to tests and a test_cancel event could be fired, but I would advise against that, as it will increase the API service even more.

Codecov Report

Merging #12279 (c602e08) into main (4f4be57) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main   #12279      +/-   ##
==========================================
- Coverage   67.52%   67.50%   -0.02%
==========================================
  Files         328      328
  Lines       17244    17249       +5
  Branches     5069     5070       +1
==========================================
  Hits        11644    11644
- Misses       5567     5572       +5
  Partials       33       33

Impacted Files
Coverage Δ

packages/jest-circus/src/run.ts
5.68% <0.00%> (-0.35%)
:arrow_down:

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 4f4be57...c602e08. Read the comment docs.

@SimenB


Add stack traces on fatal signals

When a fatal signal is fired to a task that links against caffe2 this PR adds stacktraces from every thread that's currently running. Only linux is supported currently. The signals that are currently supported are SIGABRT, SIGINT, SIGILL, SIGFPE, SIGBUS and SIGSEGV (more signals can easily be added, but for now this seemed like the major signals that might be fired - see signal_handler.cc:138 for the table of signals).
I've added tests that verify that each of those signals indeed output the expected number of stacktraces.
We need to add linking against libdl since on linux apparently it's not implicitly always linked in (I'm coming from macOS where I believe it is).
Example output can be found here - note that the signal name changes depending on the signal that was sent (as well as the number in parenthesis that corresponds to the specified signal).

Restarted Travis build. This should now come back green.

Thanks @danzimm!