[BUG] Continuously rebuild when auto-balance==least-effort and existing node becomes unschedulable

Describe the bug
I was just noticing some weird interactions beet auto-balance "least-effort" and unschedulable replicas when I had to cordon a few of my nodes.  Here's a quick description:
I have a volume with 3 replicas, with auto-balance configure to "ignored", so that it would fall back to my system default of least effort. Each of the replicas is assigned to a different zone. When I had to cordon a few of the nodes, all of the nodes from one of the zone became unschedulable and there were literally only two zones left. However, the replica on the unschedulable node was still running. So far, so good.
Longhorn then started to build a fourth replica in one of the two regions already being used. When it was done, it deleted one of the previously existing ones. And the it did that again. And again. And again. It actually never stopped building new replicas and deleting the old ones.
I've seen that behavior already a few times. What helped in that situation was setting auto balance to disabled. In that case, it will finish the cycle that it's currently in, then stop
To Reproduce
See above.
Expected behavior
Even though one of the replicas is on an unschedulable node, I'd expect longhorn to realize that it already has achieved the best balance regarding fault-tolerance. I would definitely not expect it to continue recreating and deleting replicas forever.
Log or Support bundle
n/a
Environment

Longhorn version: 1.3.1
Installation method (e.g. Rancher Catalog App/Helm/Kubectl): Helm
Kubernetes distro (e.g. RKE/K3s/EKS/OpenShift) and version: k3s v1.24.4

Number of management node in the cluster:  3
Number of worker node in the cluster: 11

Node config

OS type and version: openSUSE MicroOS
CPU per node: 4 cores
Memory per node: 4-12 MB
Disk type(e.g. SSD/NVMe): SSD
Network bandwidth between the nodes: 8x10Gbe, 3x1Gbe

Underlying Infrastructure (e.g. on AWS/GCE, EKS/GKE, VMWare/KVM, Baremetal): KVM & Baremetal
Number of Longhorn volumes in the cluster: 37

Additional context
n/a

Thanks for reporting. We will look into this.

Looks like the same thing (or maybe the opposite?) happens if set to best-effort and it is auto-upgrading the image and one of the nodes goes away during the upgrade.

Auto-balance best-effort will first go through the logic to achieve the balance for least-effect. So the infinite loop goes for both cases. We need to fix this bug so the setting recognizes replicas already on an un-schedulable node.
However, I am not expecting it to auto-upgrading the image, can you give some more info about the behavior?

I had the "auto-upgrade engine" turned on, and during the upgrade to 1.3.1 from 1.3.0, one of the nodes turned off while upgrading some engines. The volume then went into an infinite loop of creating replicas and deleting them until the node returned online sometime later.
I came here to report the issue and saw this here.
There were only three nodes, and each volume had three replicas (then two nodes and three replicas). It is hard to tell, but it may not be the same issue but the same behavior.

I am seeing something similar just when using nodeSelector. When the nodeSelector only matches nodes in two different regions, but the strategy is set of least-effort (and maybe also best-effort) then longhorn will continuously rebuild.

Verified passed on master-head (longhorn-manager aa79220) by executing the test_replica_auto_balance_when_replica_on_unschedulable_node automated test case and manually running the test steps following https://github.com/longhorn/longhorn/issues/4502#issue-1355305466.


Add npm package

There seems to already be support for npm install using a path: pr 191, but can you add an actual package in npm

Please check this link

@rajeshwarpatlolla is there any chance you can update the npm package with the latest version anytime soon?


Split Video on Page Scroll

Splitting of the running background video on scrolling using CSS effect.
Fixes # 127

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

[ ] 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

Text-editors used: VS Code

[ ] My code follows the style guidelines of this project

[ ] I have performed a self-review of my own code

[ ] My changes generate no new warnings

[ ] I have added tests that prove my fix is effective or that my feature works

[ ] New and existing unit tests pass locally with my changes

[ ] Any dependent changes have been merged and published in downstream modules

https://user-images.githubusercontent.com/62896706/126450238-247600f7-3784-4b04-80bf-c0270dd540cf.mp4

@ankitapuri please review
do we have any bug in PR template ??

@ankitapuri please review
do we have any bug in PR template ??

I'll check it

@arpit456jain u can merge it

@ankitapuri please review
do we have any bug in PR template ??

Its working fine @arpit456jain


Issue33

Fixed issue where Test-TargetResource would incorrectly return false inside of Set-TargetResource.  Two scenarios have been identified to cause this:

An asterisk is prepended to a SID (ex. "*S-1-5-32-544")
A local account is specified with a computer name (ex. localhost\TestUser1)

Codecov Report

Merging #44 into dev will increase coverage by <1%.
The diff coverage is 71%.

@@        Coverage Diff         @@
##           dev   #44    +/-   ##
==================================
+ Coverage   81%   81%   +<1%
==================================
  Files        4     4
  Lines      308   315     +7
==================================
+ Hits       250   256     +6
- Misses      58    59     +1

@jcwalker,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Popup to allow selection of Team or Folder

New streamlined pages for selecting a team or folder for use in importing COAs to other apps in bulk (augmenting single COA selection we have now).  So I'm thinking:

New popup-friendly page to search through your teams
click on a team and then either "USE" it or look through folders
if you pick a folder, the only option is to use it.
Once selected, we'd go to the db and get all COAs attached to that team/folder that the user can see.
Send that back to the calling app as an array of COAs (name & chart string)

Also we'd need a new window.Finjector.findChartSegmentString popup method which clients can call.  It'd do almost the exact same thing but call a different url by default to show the new UI.

https://github.com/ucdavis/finjector/pull/266


segment/unavailable/count metric is misleading during handoff

Affected Version
Probably all versions.
Description
The segment/unavailable/count metric is computed as the number of published segments that are not being served by historicals. But this is not the correct definition of "available": it should include realtime tasks as well. The effect is that the metric is misleading during handoff: it appears that unavailability spikes up before the new segments are loaded by historicals, even if all segments actually are continuously available on some combination of realtime tasks and historicals.
See https://druid.apache.org/docs/latest/design/architecture.html#indexing-and-handoff for definitions of terms that we should be using here.

I see this mentioned in the 0.18 release notes, but I should take your "affected version" above to mean that you don't believe this to be a recent regression?

@glasser no, I don't think it's a regression. I believe this issue has been there ever since that metric was supported.


LLAMA-7026: [SAP] Avoid crash when secure connection is trying to be established

Reason for change: SAP crashes when stop is called during secure connection handshake.
Test Procedure: Start SAP and stop it within 5s time interval to interupt secure
connection if client does not support it.
Risks: Small
Signed-off-by<EMAIL_ADDRESS>

 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.sebastian.kowalewski seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


kick them out does not work and xerxes doesent work either

Checklist for submitting an issue to KickThemOut:

[ ] I have carefully read the README file and haven't managed to resolve my issue.
[ ] I have searched the issues of this repo and believe that this is not a duplicate.
[ ] I am running the latest version of KickThemOut.

OS name & version:
Python version:
Scapy version:
Nmap version:
Link of Gist:

Description:

kick them out does not work and xerxes does'ent work


Wiki shortname link goes to non-localized root, but only on homepage

Seems like the link is generated differently on the homepage for some reason.
(23 May 2022)

Fixed in 942052df2375e66a82acd8adbec0131971126e82!


Reverse the left/right terms in the diff

The reference configuration is the "baseline" and the live cluster is what is being validated. Reversing the order in the diff ensures that the + means "live cluster has this additional line" and - means "live cluster is missing this line"

/lgtm

/test all

/lgtm

/lgtm

/approve


Labels on issues are hard to read

Some labels have a dark blue that makes it difficult to read.

If you can show an example, I could take a swing at this one.

Definitely, I'll add examples soon. Thank you!

I have a similar/related issue, my labels are showing up with a very light background and light blue text, also making them hard to read.  Here's an example:

@brianbeergod example as requested :)

I have a similar/related issue, my labels are showing up with a very light background and light blue text, also making them hard to read. Here's an example:

Thank you. I had to create a new Jira account and try my theme there but there are different options on what to display on your project board.

some other areas that also need work:

Here's the example for this particular issue:

I think I have some of these solved. Let me test and I'll submit a PR.

should be solved in #38

awesome thank you!


Resize filter not returning new path

Expected behavior
Resize filter should return new path
I'm experiencing a similar issue to #237. Resize filter was working, but now it is not. The image is still being created (it is in the content/uploads directory), but the original upload URL is being included instead. Oddly, it is working just fine on my staging site and local server with the same codebase and Timber / WP versions. I've had this happen recently with another site too, but the error occurred on the staging site rather than production.
I noticed the issue after upgrading to Timber 1.2.1, but I am using the same version of Timber across all of my installs and not experiencing the problem everywhere.
Questions from Issue #237:

Are you running WP from a subdirectory?

No

Do you have "normal" setup with /wp-content/uploads/ etc.?

Yes

Do all images fail?

Yes
Actual behavior
Resize filter creating cropped image but not returning the path
Steps to reproduce behavior
{% for accessory in post.get_field('featured_accessory') %}
  <img alt=""
    src="{{ TimberImage(accessory.image).src | resize(400, 266) }}"
    sizes="(max-width: 47.9375em) 100vw, (max-width: 59.9375em) 50vw, 33vw"
    srcset="{{ TimberImage(accessory.image).src | resize(400,266) }} 400w, {{ TimberImage(accessory.image).src | resize(750,500) }} 750w">
{% endfor %}

What version of WordPress, PHP and Timber are you using?
WordPress 4.7.1,  Timber 1.2.1
How did you install Timber?
Upgraded to newest version via plugin updater in WordPress dashboard.

Um... so this problem seems to have fixed itself after WPEngine cache purge. Closing!

I can confirm this issue too. Also on WPEngine the issue however seems to be strictly over SSL for when it's generating a path that includes https.

Digging in further I'm seeing this isn't just a WPEngine issue since both @saralohr and I seem to be WPEngine users.
With debug mode enabled I get a hard error:

Notice: Undefined index: dirname in [...]/wp-content/plugins/timber-library/lib/ImageHelper.php on line 383
Notice:  Undefined index: extension in [...]/wp-content/plugins/timber-library/lib/ImageHelper.php on line 385

With the debug mode disabled I just get the default path over HTTPS.
With HTTPS: https://timber.dev/wp-content/uploads/2017/01/z55cr_d0ayg-andrew-neel.jpg
With HTTP: http://timber.dev/wp-content/uploads/2017/01/z55cr_d0ayg-andrew-neel-600x0-c-default.jpg

I also was wondering if this was an SSL issue, as I am using https on production but not on staging. However, the other site where I experienced the same issue a couple of months ago does not use SSL, and I am experiencing the issue on staging but not production. I just double checked the Timber version on that site and it is still 1.1.12.

So you're experiencing the issue on 1.1.12 or that's the production system where it's not happening?

Sorry, so many to keep track of. It is happening on two sites:
Site number 1

Issue on PRODUCTION:

https
Timber version 1.2.1
reverting to Timber 1.1.3 fixes the issue

Not experiencing the issue on STAGING

http
Timber version 1.2.1

Site number 2

Issue on STAGING:

http
Timber version 1.1.12

Not experiencing the issue on PRODUCTION

http
Timber version 1.1.12

Okay interesting. My situation is like your site number 1. Except reverting to 1.1.12 fixes the issue just fine too.

BTW, both of these use WPEngine, no CDN.
To make this more interesting, yet another site I work on uses WPEngine with a CDN, recently upgraded to Timber 1.2.1, as is NOT experiencing the issue.

No CDN on mine either.
If you enable WP_DEBUG do you get the same or a different error message as the one I posted? You have to view source since it's within the img tag so it doesn't actually render.

I haven't been able to replicate the issue locally, unfortunately, so I'm
not getting any error messages.
On Wed, Jan 25, 2017 at 1:07 PM, Tim Knight<EMAIL_ADDRESS>wrote:

No CDN on mine either.
If you enable WP_DEBUG do you get the same or a different error message as
the one I posted? You have to view source since it's within the img tag so
it doesn't actually render.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/timber/timber/issues/1303#issuecomment-275233137, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AJbi07cbbCpfJ7NrKkH3PhcMBjX7FFu_ks5rV7mYgaJpZM4Lqehn
.

What about one of your staging sites? Can you put those in debug mode? Also are you using free ACF or ACF Pro?

@timknight @saralohr — thanks for the reports on these. I found some stuff in #1293 that deals with the errors @timknight is seeing with...

Notice: Undefined index: dirname in [...]/wp-content/plugins/timber-library/lib/ImageHelper.php on line 383
Notice: Undefined index: extension in [...]/wp-content/plugins/timber-library/lib/ImageHelper.php on line 385

... I'm hoping solving those might relate here

Thanks @jarednova 👍

Thanks @jarednova!
@timknight I turned on DEBUG on the staging site where I am seeing the issue and didn't get an error. I noticed something else funny happening, though. The images where it is happening (it's not all of them), redirect to the production URL when I open them in a new tab. They have the staging path in the page though.

Also are you using free ACF or ACF Pro?

I am using ACF Pro.

+1 getting @saralohr issue from her example 'Site number 1'

I've got the exact same issue on WPEngine.
Timber version 1.2.3.
It doesn't work on https Production or Staging. It works on http Staging though.

Can confirm same issue on WPEngine:
Staging http works.
Production http works.
Production https doesn't.
Timber version 1.2.4.

Still seems this is an issue for me as well. I'm still seeing this over https in general (production or staging) anytime the resize filter is used.

 Those of you having the issue, in your General Settings do you have your site address and your WordPress with the https: in the address?
tl;dr; - make sure your site address and wordpress address have the https in it.
Here's what it looks like is happening to me @jarednova.
The analyze_url function within ImageHelper.php uses the wp_upload_dir() to determine if the image is within the uploads folder. If you're going to "https" in the browser while the settings still have "http", the check strpos($tmp, $upload_dir['baseurl']) will fail since it can't find the "https" string within $upload['baseurl'] that will then take it down to the other condition used for content-based items where it fails as well.
This then sends it back up to _operate where it checks $op->run($source_path, $destination_path) and fails (because the final condition within analyze_url for wp-content doesn't construct a real subdir from theme_url_to_dir when it's actually an upload file). So after all that _operate just returns the source hitting the // in case of error, we return source file itself part of the condition.

Thanks @timknight, I was experiencing this issue (https, WP Engine, with CDN, Timber 1.2.4) and (after reading your comment) noticed the WordPress path was set to "http", while the site address was "https". Updating the WordPress path to "https" fixed the issue.

My entire multisite had http as site URLs instead of https. Problem now solved for me.
Thanks guys.

I have the same issue. I'm on HTTPS, and those two settings are greyed out for me, but do start https:. I'm using Trellis and Bedrock.
I'm on Timber 1.3.0. If I roll back to 1.2.4, the problem disappears.

Thanks so much everyone (especially @timknight who left all the breadcrumbs here). I just submitted a PR to hopefully tackle this one. Tests are running right now — but try applying it to your install and see if it resolves!

This does not fix the issue for me. I will now bisect and try to find the regression which is affecting me.

Ugh, sorry to hear that @tremby — but bisecting would be incredibly helpful so we can pinpoint what caused this. Thanks for looking further into it!

73bd23ef8b2005b5e9db7122e61415cf55bf9798 is the first bad commit
commit 73bd23ef8b2005b5e9db7122e61415cf55bf9798
Author: gillesgoetsch<EMAIL_ADDRESS>Date:   Fri Mar 3 19:44:45 2017 +0100

    [Fix] Wrong image path breaking timber resize when using external wp-content-dir (WP_CONTENT_DIR)

:040000 040000 7e084ac9830f221b49aeee722a6b7db3e298d108 8508891e48f45096783a2d6de7085ee0e5484e14 M	lib

Investigating further...

OK, in this chunk of code
			if ( false === strpos($upload_dir['baseurl'], site_url()) ) {
				// use HOME_URL and relative image path
				$tmp = get_home_url().$tmp;
			} else if ( !$result['absolute'] ) {
				$tmp = site_url().$tmp;
			}
			if ( URLHelper::starts_with($tmp, $upload_dir['baseurl']) ) {
				$result['base'] = self::BASE_UPLOADS; // upload based
				$tmp = str_replace($upload_dir['baseurl'], '', $tmp);
			} else if ( URLHelper::starts_with($tmp, content_url()) ) {
				$result['base'] = self::BASE_CONTENT; // content-based
				$tmp = self::theme_url_to_dir($tmp);
				$tmp = str_replace(WP_CONTENT_DIR, '', $tmp);
			}

I have

$upload_dir['baseurl']: https://ceso.test/app/uploads
site_url(): https://ceso.test/wp
$tmp: https://ceso.test/app/uploads/2016/03/1.png

and so we enter the first branch of that conditional. My get_home_url() is https://ceso.test so this sets $tmp to https://ceso.testhttps://ceso.test/app/uploads/2016/03/1.png which is obviously no good.
Neither branch of the second conditional is entered: the $tmp string does not start with my upload dir baseurl, nor does it start with my content_url(), which is https://ceso.test/app.
My directory layout, as you can see, is non-standard. It's what is set out by Roots Bedrock, which is a pretty popular thing.

Hello, another Bedrock user here experiencing the same problem as outlined above, $tmp becomes:
http://foo.devhttp://foo.dev/app/uploads/2017/05/bar.jpg
As a hacky fix as I don't have the time to look through the code and comprehend it, I added the following:
if ( false === strpos($upload_dir['baseurl'], site_url()) ) {
    // use HOME_URL and relative image path
    if (false === strpos($tmp, get_home_url())) {
        $tmp = get_home_url().$tmp;
    }
} else if ( !$result['absolute'] ) {
    $tmp = site_url().$tmp;
}

I'd make a pull request but it would likely do more harm than good!

Hi all,
I'm experiencing the same issue on multiple sites with version 1.3.0. Reverting to 1.2.4 eliminates the issue.
We're using non-standard site and content urls, configured with constants. So the cause seems to be the same as the one identified by @tremby.

Also having the same issue with similar circumstances. HTTPS site hosted on WPEngine, full sized images served to non logged in users and under HTTPS. Logged in users see resized images, without https everyone gets the resized images, so the problem seems to result from the combination of https and logged off.

Thanks everyone for your reports. There are a few different circumstances listed here, but the most frequent info seems to be regarding Timber 1.3.0 and Roots/Bedrock (@krisscox, @tremby). I was able to get a Bedrock/Timber instance up-and-running and confirmed that v 1.3.0 broke resizing, but luckily also confirmed that v1.3.1 fixed it.
The issue that @lmartins is referring to seems distinct, so I'm going to open a new thread to focus on that.


fix: variable referenced before assignment

This one was missing:
except *Exception as e*:  print(e.args())```

Sorry for taking too long. I have been busy


[BUG] Harvester not freeing resourses after migrating VM to another host.

Describe the bug
Since harvester created all my VMs on the same host, I thought of just migrating them manually to a new host, until v1.0.1 has been released. It looks like harvester is not freeing up the resourses after the migration or something isn't getting updated as it should.
To Reproduce
Steps to reproduce the behavior:

Go to Virtual Machines.
Migrate one VM to another host.
Check capacity on dashboard.

Expected behavior
Correct reporting of capacity on dashboard.
Support bundle
https://drive.google.com/file/d/130fkGekU_ZvabFYiwZ_dz37eTuc2R_Yc/view?usp=sharing
Environment:

Harvester ISO version: v1.0.0
Underlying Infrastructure (e.g. Baremetal with Dell PowerEdge R630): 3x Dell R730 (12c, 64GB Ram)

https://ibb.co/x2x8cR1
https://ibb.co/J5VvM2Q

Can not reproduce this issue in v1.0.3-rc1

before create  vm

create vm in the node harv103-rc1-3

after migrate vm to  harv103rc1


Time Machine

I used disk arbitrator on two mac osx mountain lion, and at the end (while stopping the app) both of my macs advertised me that the time machine disks were on read only. I still did not figured out how to get them back to RW.

It's been more than 3 years, closing.


Sentry error fixes

Closes #1623

[x] Ship errors to Sentry
[x] Display friendlier error screen to the user
[x] Give user a Sentry User Feedback screen when error occurs

This is not skinnable, boo

[x] Generate and ship source map files to Sentry during build
[x] Include a persistent "User Bug Report / Feedback" button on app. See customization capabilities here: https://docs.sentry.io/platforms/javascript/guides/react/user-feedback/configuration/

@xraystyle1980 @nicolaus-sherrill @xanaramoss awaiting your advice on what copy to use in either/both Crash Report Modal / User Feedback form button

@adamgall @xraystyle1980 @nicolaus-sherrill
Copy Suggestions:
Button Text:
“Report an Issue” or “Submit Report”
Title:
Crash Report & User Feedback
Description:
Option 1
We're dedicated to making our platform better every day. Help us improve your experience by reporting any issues or providing feedback below.
Option 2
Encountered a problem or have feedback? Let us know! Your input helps us improve.


Open Folder on hover

Description
it is not clear to me when the desired folder is selected when hovering under the mouse pointer. it is obscured for me by the display of the information bar. furthermore, due to my poor eyesight, the time is too short before the folder opens and i can actually see which one is active.
unfortunately, i can't find a way to switch off the feature or to adjust the time for automatic opening.
as nice as files is, that's one reason why i don't want to work with it anymore. it's too uncertain where files might be moved or too tedious to switch back to the directory again and again.
Steps To Reproduce
select a file and move hover an folder
Requirements
there should be an option to deactivate the function, or it should also be possible to adjust the time when the folder opens itselfhere should be an option to deactivate the function, or it should also be possible to adjust the time when the folder opens itself
Files Version
<IP_ADDRESS>
Windows Version
10.0.19045.4046
Log File
debug.log

@mikex2023 can you clarify if you're talking about when drag and droping items?

@yaira2 yes exactly

Thank you for the feedback, an option to disable this feature isn’t available but it should be fairly easy to add if there is enough interest. Can you open a new issue using the feature request template so we can this to our project board?


can you add password guarded access?

so i can share the blog with people who can enter the right password instead of showing to public?

Hi, unfortunately it's not impossible to do that at the moment, In fact, we can get the PAGE ID in the browser.This means everyone can access to the original database easily, so it doesn't make much sense to use a password to guarded it.
Since the official api has launched, maybe there will be a better solution later


Tweak up for Workiva

Summary
Adds the required Workiva files for CI

Dockerfile for Workiva Build
skynet.yaml for Skynet and release pipeline
aviary.yaml for Infosec

Updates to allow analyzer 1

Updated the http_parser version range to allow v4
removed the dependency on test_coverage which is old and a flutter specific package

Updates to allow Workiva Build to pass (You can test the commands locally)

skipped a test that was failing, the rest pass

Public API Changes
:scream: Error
Semver Audit Service experienced an error during its report.
For details, view the service logs for the correlation ID: 86078e19-f86a-4684-a036-17f7ad1c8aed
Showing results for 89e6286398f7e574b0c86d89a88e98c7f9e32e73

Powered by semver-audit-service. Please report any problems by filing an issue.
Reported by the dart semver audit client 2.2.2
Browse public API.

QA+1 Workiva CI is now running and passing!

@Workiva/release-management-p


CLI11 warnings on BlueOS+CUDA.

These warnings were reported by the Ares team and they would like them to be corrected. It looks like they are building on BlueOS with CUDA enabled.
/usr/gapps/bdiv/blueos_3_ppc64le_ib_p9/clang-ibm-16.0.6-cuda-11.8.0-gcc-11.2.1/axom/0.10.0-cuda/include/axom/CLI11.hpp(1001): warning #20236-D: pragma "diag_suppress" is deprecated, use "nv_diag_suppress" instead
/usr/gapps/bdiv/blueos_3_ppc64le_ib_p9/clang-ibm-16.0.6-cuda-11.8.0-gcc-11.2.1/axom/0.10.0-cuda/include/axom/CLI11.hpp(1005): warning #20236-D: pragma "diag_default" is deprecated, use "nv_diag_default" instead

Should we update our version of CLI11? Maybe this is fixed in a newer version.

It does seem to be fixed in the current CLI11:
https://github.com/CLIUtils/CLI11/blob/5a03ee58384ce18b6b3de5a2256e18f5ef103633/include/CLI/TypeTools.hpp#L190-L194


[Settings] Reset Activation Key to Default Value

Summary of the Pull Request

Add Default value to Properties
Add Reset button in shortcut dialog

PR Checklist

[x] Closes: #22568
[x] Communication: I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected
[ ] Tests: Added/updated and all pass
[x] Localization: All end user facing strings can be localized
[ ] Dev docs: Added/updated
[ ] New binaries: Added on the required places

[ ] JSON for signing for new binaries
[ ] WXS for installer for new binaries and localization folder
[ ] YML for CI pipeline for new test projects
[ ] YML for signed pipeline

[ ] Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

@microsoft-github-policy-service agree

I don't like how my implementation of the button looks.
Do you have any suggestions on the location or appearance of the button? Should it be placed inside the shortcut menu? Is an "Undo" icon the right label?

I don't like how my implementation of the button looks. Do you have any suggestions on the location or appearance of the button? Should it be placed inside the shortcut menu? Is an "Undo" icon the right label?

I would prefer if the reset button is inside the pop-up which appears when you click the Edit button on the Shortcut. Also the icon should be more like , the current one you have used is more Undo than Reset and people might expect Undo behavior which can be different from Reset.

@svenlaa To keep the UI as clean as possible, would it make sense to have the reset button in the actual shortcut dialog?

Personal idea: I would - at least - place the reset button inside the same area, in line with the edit button. Moving it to the fly-out is also possible, but that does mean more clicks, which I find not user friendly.
Next idea is that the Click action could be narrowed down to the actual button/icon. (in stead of the larger area)

Awesome feature! Thanks a lot for this contribution! Is it ready for review?

Awesome feature! Thanks a lot for this contribution! Is it ready for review?

Not quite. A few other folks have commented on it and I'd like to implement their feedback. I got a few personal things going on, so I won't be available for coding 'till Thursday

Thanks for the contribution and the update :)
Just press "Ready for review" when you want the core team to review.
Thanks a lot, once again.

Sure, it not that bad. What about Save-Reset-Cancel in one line?

Sure, it not that bad. What about Save-Reset-Cancel in one line?

I really like this one, and it was easier to implement than I thought. I used a "Use Default" label instead of "Reset" which is clearer in my opinion. What should it reset to? The default shortcut

I think "use default" and "reset" are both possible. Personally I would go for "reset" because it's nice and short. Also thinking about translation + available space.
@niels9001 is the comment (in shortcutcontrol.cs) about C# versus XAML still valid?

When running/debugging: It looks and works good.
When building, I get a few issues. (don't know how much it's related to this branch)

@niels9001. I just updated the screenshot in the main post. Glad you like the approach!

Yep, simple and clean solution. 👌🏻
Just hoping it will work when a different language uses long words. 🤞🏻

Merged it. Thank you for the contribution!


Atom not saving theme settings.

Prerequisites

[ ] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: https://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: https://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
Although I change theme, everytime I start atom, default theme is loaded. No external package is installed yet.
Steps to Reproduce
1.- Change default theme
2. Close Atom
3. Open Atom
Expected behavior: The theme I choose is loaded. Welcome tabs are not opened.
Actual behavior: Default theme is loaded. Welcome tabs are opened.
Reproduces how often: Everytime I run Atom
Versions
Atom    : 1.31.0
Electron: 2.0.7
Chrome  : 61.0.3163.100
Node    : 8.9.3
Distributor ID:	Ubuntu
Description:	Ubuntu 18.04.1 LTS
Release:	18.04
Codename:	bionic
Additional Information
Any additional information, configuration or data that might be necessary to reproduce the issue.

@servicioswebavr The current version of Atom is 1.31.1
Please update, as this issue should be patched in that release.

@Aerijo is correct, this issue will be fixed by updating to Atom 1.31.1.  I'll close this issue for now, but if the issue persists after updating let us know and we will investigate.  Thanks!


Add setup.py

This will allow obsgit to be installed from Pip.

@aplanas feel free to suggest changes in the setup.py.
I tried to run test, but many are failing due some refactoring in previous commits, yet some are working fine.
I sugested Python 3.6 as default because it's the one in Leap and SLE, but tests require at least 3.8 to run.

Good one indeed.

@aplanas any updates regarding my latest comments?

No sorry. All OK from my side. Lets merge this and later update the package. Thanks!

@aplanas great, thanks.


docs/contributing: update documentation for running single tests

Description
Enhance the documentation to clarify how to run individual tests when developing new plugins. Update the instructions for executing the full test suite to improve clarity.
Issue
#2503

@mergify queue

May need some help merging hah! @MattSturgeon

May need some help merging hah!

Yeah, even once approved someone with write permissions needs to do the merge

May need some help merging hah!

Yeah, even once approved someone with write permissions needs to do the merge

Thank you sir!

Thanks for contributing this excellent improvement!


Refactor shim

[ ] Don't utilize stdio (prevents writing from stdout)
[ ] Refactor the JS (js streams are a hot mess)

I would be happy to hack on this, but a bit more detail surrounding "refactor the JS" would be helpful. could you elaborate a bit on what you were thinking here?

I'd be awesome to remove the byline.js thing I copy/pasted in there, simple stuff like that seems really difficult in Node haha. My index.js isn't too clean either but at least it's acceptable. Testes wouldn't hurt, pretty important part to not break hahaha 😄


Add react tooltips

add react tooltips (there's one in dev_builds.rb) https://www.npmjs.com/package/react-tooltip

With blazor there seems to be at least a few premade implementations, so this should be much easier. For example: https://blazor.radzen.com/tooltip


Bump FAB to 1.12.3

Related CHANGELOG:
https://github.com/dpgaspar/Flask-AppBuilder/blob/master/docs/versions.rst#improvements-and-bug-fixes-on-1123

Codecov Report

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

@@           Coverage Diff           @@
##           master    #6854   +/-   ##
=======================================
  Coverage   56.31%   56.31%
=======================================
  Files         527      527
  Lines       23529    23529
  Branches     2782     2782
=======================================
  Hits        13251    13251
  Misses       9866     9866
  Partials      412      412

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 4638618...80f0814. Read the comment docs.


Change direction?

Is it possible to change the direction of movement without changing the speed?

Hey. Thanks for using the plugin.
I believe changing the speed is the only way to control the direction of movement (if we're not talking about changing axis, just direction).
Wouldn't inverting the speed number achieve what you want?
For example, if you're element has a speed of -3, changing it to 3 would change the direction while keeping the same speed.
Let me know if that works for you.

right, but can it be implemented?

I'm not sure I understand your question correctly and I'll be honest here, I'm not entirely up-to-date with some of the features, but, I believe wherever there's a speed, you can invert it to make it go the other way.
This goes for vertical and horizontal parallax.
I understand you do not wanna change the speed, just the direction, but, if you invert the speed's value, the element will keep the same exact speed but will travel in the opposite direction.
Now, in case you wanna make it travel horizontally instead of (the default) vertically, for example, you'd have to follow the readme (sorry, I can't help much with that part, I never used it myself).
But then, the same principle would apply.
To make it go, for example, left to right instead of right to left, you'd just need to invert the speed's value.
(Someone correct me if I'm spreading fake news here, please, thanks lol)
Play around with those settings a bit and let me know how it goes.

Changing the data-rellax-speed attribute to a negative value doesn't change the direction, it still applies positive transform values. I'm looking for a solution as well. The documentation says "A negative value will make it move slower than regular scrolling, and a positive value will make it move faster.", so the negative value used in the attribute is for something else. Would love a solution because I can't use this now.

If you want to globally reverse the Y axis parallax to move elements up as you scroll instead of down, you'll need to edit the source on line 285 and change "result.y = self.options.round ? Math.round(valueY) : Math.round(valueY * 100) / 100;" to "result.y = self.options.round ? -Math.round(valueY) : Math.round(valueY * 100) / 100;"
The difference being an added "-" to the first instance of "Math.round". Not of fan of doing this, would love a built in means to do it. I have zero clue if this will create issues with functionality but it works.


Update performance-test-results-mi.md

Purpose
This PR fixes issue: #7088
Documentation
This PR updates : https://apim.docs.wso2.com/en/latest/install-and-setup/setup/deployment-best-practices/performance-test-results-mi/

Was this tested locally?


Added the same logic of statink to check if Splatfest or not.

Additionally checks context['game']['is_fes'].

Thanks!


Trying to copy and paste an image into a DM

gw/</<@https://cosmic.church/apps/talk/assets/index.4502d6d8.js:398:309054
gw/<@https://cosmic.church/apps/talk/assets/index.4502d6d8.js:398:308991
q0e@https://cosmic.church/apps/talk/assets/index.4502d6d8.js:97:6696
e.unstable_runWithPriority@https://cosmic.church/apps/talk/assets/index.4502d6d8.js:82:3886
P0@https://cosmic.church/apps/talk/assets/index.4502d6d8.js:93:38710
Yh@https://cosmic.church/apps/talk/assets/index.4502d6d8.js:97:6157
U0e/<@https://cosmic.church/apps/talk/assets/index.4502d6d8.js:97:6069
T@https://cosmic.church/apps/talk/assets/index.4502d6d8.js:82:2925
P.port1.onmessage@https://cosmic.church/apps/talk/assets/index.4502d6d8.js:82:1551

Can you try again? Appears to work without issues.


README portuguese Brazil version

In Brazil we are quickly adopting Go for new services and I believe that kitex can be used, I am now creating the readme in Brazilian Portuguese but I also want to translate the documentation in the future.

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

Hello, thanks for your PR and attention.
As for the PR, I prefer not to merge it. Adding a new version of Readme will increase the cost of maintaining.
If you want to share this project to more Brazilian, I would suggest to use AI for translation. If they want to read the code, English is a must.


[WIP] Arrow conversion at partitions

What changes were proposed in this pull request?
Changed conversion to be iterator based, instead of Array[InternalRows]
Moved Dataset conversion to partitions, then collect as byte array
Added some pydocs
How was this patch tested?
Currently failing unit tests for Scala and Python

Benchmarks seem promising so far (unless something is terribly wrong)
collect as internal rows
count    50.000000
mean      0.213734
std       0.038923
min       0.159922
25%       0.188007
50%       0.205784
75%       0.224724
max       0.392772
dtype: float64
toPandas with arrow
count    50.000000
mean      0.064689
std       0.035405
min       0.050057
25%       0.055823
50%       0.059600
75%       0.062314
max       0.305428
dtype: float64
toPandas without arrow
count    50.000000
mean      2.174661
std       0.116211
min       2.011181
25%       2.098154
50%       2.138159
75%       2.226889
max       2.515083
dtype: float64

@wesm @icexelloss this is a first stab at running the Dataset conversion at partition level so it can be done by executors.  It seems promising so far, but I ran into some issues with the Java tests (see above) and Python tests, so I was hoping to get your input.
Python tests fail when using a String column and there is an empty partition.  In arrow-cpp adapter.cc calls LoadBinary here and tries GetBuffer even though the length is 0.  Is is possible to put the same checks for this as in LoadPrimitive?

hi @BryanCutler I created https://issues.apache.org/jira/browse/ARROW-544, I will try to reproduce and fix

Nice work!

collecting as arrow batch is 3x faster than collecting as internal rows!
Honestly I don't understand why it is the case, @wesm

@icexelloss it's because the conversion is being parallelized on task executors -- serializing Array[InternalRow] is not free. Presumably moving the serialized byte arrays adds comparatively overhead

I might be missing sth here.
Collect as internal rows (SparkPlan.executeCollect) doesn't do any arrow conversion, it does the normal unsafe row serialization which is just pretty much just sending the underlying byte array of an unsafe row. I wonder why it's 3x slower than doing the arrow conversion and then sending the arrow byte array.
Overhead of compression maybe?

@icexelloss the conversion from Array[InternalRow] to Arrow record batches is being parallelized. Before, it was serial and single-threaded

I'm getting a 38% performance improvement on the test case I have in my talk in about an hour -- I'll include the updated benchmarks (I already had slides about executor-local serialization as a "TODO"), so thank you

Thanks for the review @icexelloss and @wesm!  I was a little surprised too that the benchmarks are showing a speedup over collecting internal rows, but I think it's mostly due to the data being transferred as large batches instead of individual rows.

@BryanCutler , I have issues running code from your branch stc_toPandas_with_arrow.
Do you need to build pyarrow locally to make it work?

Yeah, I built it with the latest from a few hours ago.  I updated the
timestamp writer, was that the issue?
On Feb 9, 2017 7:07 PM, "Li Jin"<EMAIL_ADDRESS>wrote:

@BryanCutler https://github.com/BryanCutler , I have issues running
code from your branch stc_toPandas_with_arrow.
Do you need to build pyarrow locally to make it work?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/BryanCutler/spark/pull/23#issuecomment-278845963, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEUwdV2WIVDi1Rtd3-fcshItPulvu_jJks5ra9R7gaJpZM4L7eqL
.

No, I think I don't have the updated pyarrow ( I was using pyarrow-0.1 from condaforge before) so it's complaining about "stc_toPandas_with_arrow"
Then I tried to install pyarrow locally, but hit:
(python setup.py build_ext --inplace)
[  2%] Building CXX object CMakeFiles/pyarrow.dir/src/pyarrow/adapters/pandas.cc.o
/home/icexelloss/workspace/arrow/python/src/pyarrow/adapters/pandas.cc:52:14: error: ‘arrow::DictionaryType’ has not been declared
using arrow::DictionaryType;
I have pyarrow installed in /usr/local:
(spark-dev) icexelloss@icexelloss-VirtualBox:~/workspace/arrow/python$ echo $LD_LIBRARY_PATH
/usr/local/lib
(spark-dev) icexelloss@icexelloss-VirtualBox:~/workspace/arrow/python$ ls $LD_LIBRARY_PATH
libarrow.a  libarrow_io.a  libarrow_io.so  libarrow_ipc.a  libarrow_ipc.so  libarrow_jemalloc.a  libarrow_jemalloc.so  libarrow.so  pkgconfig  python2.7  python3.5
I wonder if you have seen this when building pyarrow?

Not that exact error, but I've always just built it locally and start with
building arrow-cpp first and installing that to ARROW_HOME.
On Feb 9, 2017 7:40 PM, "Li Jin"<EMAIL_ADDRESS>wrote:
No, I think I don't have the updated pyarrow ( I was using pyarrow-0.1 from
condaforge before) so it's complaining about "stc_toPandas_with_arrow"
Then I tried to install pyarrow locally, but hit:
(python setup.py build_ext --inplace)
[ 2%] Building CXX object CMakeFiles/pyarrow.dir/src/
pyarrow/adapters/pandas.cc.o
/home/icexelloss/workspace/arrow/python/src/pyarrow/adapters/pandas.cc:52:14:
error: ‘arrow::DictionaryType’ has not been declared
using arrow::DictionaryType;
I have pyarrow installed in /usr/local:
(spark-dev) icexelloss@icexelloss-VirtualBox:/workspace/arrow/python$ echo
$LD_LIBRARY_PATH
/usr/local/lib
(spark-dev) icexelloss@icexelloss-VirtualBox:/workspace/arrow/python$ ls
$LD_LIBRARY_PATH
libarrow.a libarrow_io.a libarrow_io.so libarrow_ipc.a libarrow_ipc.so
libarrow_jemalloc.a libarrow_jemalloc.so libarrow.so pkgconfig python2.7
python3.5
I wonder if you have seen this when building pyarrow?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/BryanCutler/spark/pull/23#issuecomment-278849701, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEUwdZUdHrmBrl6eXEMjUm9qi0PDUuSIks5ra9wpgaJpZM4L7eqL
.

@icexelloss the issue is that the libarrow you have installed is outdated. I have bash functions like these to help automate things
function parquet_cpp_update {
    mkdir -p ~/code/parquet-cpp/library-build
    pushd ~/code/parquet-cpp/library-build
    rm -rf *
    cmake -DCMAKE_INSTALL_PREFIX=$TP_DIR \
          -DCMAKE_CLANG_OPTIONS=$CMAKE_CLANG_OPTIONS \
          -DCMAKE_BUILD_TYPE=$TOOLCHAIN_BUILD_TYPE \
          -DPARQUET_ARROW=on \
          -DPARQUET_ZLIB_VENDORED=on \
          -DPARQUET_BUILD_TESTS=OFF \
          -DPARQUET_BUILD_EXECUTABLES=OFF \
          ..
    make -j8
    make install
    popd
}

function arrow_cpp_update {
    mkdir -p ~/code/arrow/cpp/library-build
    pushd ~/code/arrow/cpp/library-build
    rm -rf *
    cmake -DCMAKE_INSTALL_PREFIX=$TP_DIR \
          -DCMAKE_BUILD_TYPE=$TOOLCHAIN_BUILD_TYPE \
          -DARROW_BOOST_USE_SHARED=off \
          -DARROW_BUILD_BENCHMARKS=off \
          -DARROW_JEMALLOC=off \
          -DARROW_BUILD_TESTS=off \
          -DCMAKE_BUILD_TYPE=$TOOLCHAIN_BUILD_TYPE ..
    make -j8 VERBOSE=1
    make install
    popd
}

function update_tp_toolchain {
    arrow_cpp_update
    parquet_cpp_update
}

you may wish to remove the rm -rf * if you don't have any thirdparty $*_HOME variables set

If you are building from source, you must be careful about release vs. debug builds. I maintain both debug and release toolchains and have helper functions to switch between them:
function debug() {
    export TP_DIR=$DEBUG_TP_DIR
    export TOOLCHAIN_BUILD_TYPE=debug
    set_build_env
}

function release() {
    export TP_DIR=$RELEASE_TP_DIR
    export TOOLCHAIN_BUILD_TYPE=release
    set_build_env
}

Thanks @wesm. I will give it a try.

Hi @icexelloss , just wondering if you were able to run this branch yet and confirm it works for you too?


Creating tags on findings require superuser rights

Bug description
Using a non-superuser user and wanting to tag findings through, it does not work. It fails with a 403.
staff is not needed, but superuser is.
A bit bothersome for service accounts :-/

Can you provide some info on how to reproduce this? I assume this is referring to the API as in the UI it seems to work fine.
Are you using a PATCH/POST on the finding itself? Or are you using the /findings/tags endpoint?

Is it something that will change/be modified with auth v2 system?

Yes, I was also thinking we should maybe just make sure that it works in the new system and not spent too much time on the old system. The "official" python API wrapper needs a superuser anyway to work, so easier to accept for now and fix everything when the new permissions are there.

With auth v2 you can add tags when you have the role Writer, Maintainer or Owner, just checked that.


CSRF token

Deployment Method

[ ] Installer
[x] Docker Development
[ ] Docker Production
[ ] Bare metal Development
[ ] Bare metal Production
[ ] Digital Ocean image
[ ] Other (please provide a link Steps to Reproduce

Describe the problem*
Hi!
My problem is that I can't log in and try as I might, I can't solve it.
I can't log in with the old UI, but I can with the new one. With the old one I get an "Authentication failure" in the log and "Forbidden (CSRF cookie not set.): /accounts/login/", but with the new platform I can log in.!
`# Site URL - update this to match your host
I run this behind traefik and I also use cloudflare proxy.
InvenTree version: 0.16.7
#INVENTREE_SITE_URL="http://localhost"
INVENTREE_SITE_URL="http://stock.domain.tld"
COMPOSE_PROJECT_NAME=inventree
INVENTREE_WEB_PORT=1020
INVENTREE_TAG=0.16.7
#INVENTREE_ALLOWED_HOSTS=localhost
#INVENTREE_ALLOWED_HOSTS=stock.domain.tld
#INVENTREE_ALLOWED_HOSTS=*
#INVENTREE_TRUSTED_ORIGINS=['http://localhost:1020', 'http://localhost']
INVENTREE_TRUSTED_ORIGINS='https://stock.domain.tld,https://domain.tld,http://localhost,http://localhost:1020,http://<IP_ADDRESS>,http://<IP_ADDRESS>:8000,http://inventree-server:8000'
#INVENTREE_TRUSTED_ORIGINS="https://stock.domain.tld"
#INVENTREE_CORS_ORIGIN_ALLOW_ALL=True
#INVENTREE_USE_X_FORWARDED_HOST=True
#INVENTREE_USE_X_FORWARDED_PORT=True
INVENTREE_CORS_ORIGIN_WHITELIST=https://stock.domain.tld`
Image
PS: In TRUSTED_ORIGINS => I added everything I tried
Steps to Reproduce
Setup Inventree in docker and after setup, unable to login
Relevant log output
Forbidden (CSRF cookie not set.): /accounts/login/

In the 0.16.7 release notes there is a breaking change which is likely related to this.
You should change the value of INVENTREE_COOKIE_SAMESITE - as per the changelog in the linked release. This should address the issue for you, I think.

Okay, you've solved in ~two seconds a problem that I've been struggling with for two days. Thank you very much @SchrodingersGat

@Sube22 happy to help :) And make sure to read the release notes when you update ;)


Sort tables / columns

Currently tables / columns are ordered in random.  It should be sorted in alphabetical order (for tables), or in the same order as DB table (for columns).

++


feat(servicerelease): get inreview and active service

added filter for inreview and active status for endpoint api/services/servicerelease/inReview
ref:-CPLP-2417

Changes were already merged for other PR https://github.com/catenax-ng/tx-portal-backend/pull/280 . Hence closing this PR


empty predicate

🚀 Feature
true example
if:
  - empty: []
  - empty: ""

false example
if:
  - empty: [hello]
  - empty: hello

Motivation
to fulfill more scenarios; commonly I want to know if something is empty

It might be more generalized to use `eq: [0, $(length)]. Then length could be anything. Closing for now.


(docs): improved syntax in example

Hi, thanks for the PR but I'm not really sure this change adds any value to the code sample (it is legit JavaScript as the error is optional)


fix: OpenAPI table in dark mode

Hey, I just made a Pull Request!

Fix https://github.com/backstage/backstage/issues/24014#issuecomment-2042310796
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[x] Added or updated documentation
[x] Tests for new functionality and regression tests for bug fixes
[x] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Sorry for the delay in this! Looks like auto-merge got disabled!


Uninstalling Push Jobs leaves opscode-pushy-server in status output

A customer and I noticed that if the Push Jobs Server is uninstalled - https://docs.chef.io/uninstall.html#push-jobs-title
When I do a chef-server-ctl status the listing for opscode-pushy-server remains, just in the down state.
root@chef-server:~# chef-server-ctl status
run: bookshelf: (pid 7599) 3581s; run: log: (pid 4725) 4747s
run: nginx: (pid 10314) 336s; run: log: (pid 4887) 4742s
run: oc_bifrost: (pid 7619) 3581s; run: log: (pid 4439) 4754s
run: oc_id: (pid 7344) 3590s; run: log: (pid 4458) 4753s
run: opscode-erchef: (pid 7492) 3584s; run: log: (pid 4768) 4746s
run: opscode-expander: (pid 7416) 3585s; run: log: (pid 4641) 4749s
run: opscode-expander-reindexer: (pid 7459) 3584s; run: log: (pid 4656) 4748s
down: opscode-pushy-server: 135s, normally up; run: log: (pid 10278) 337s
run: opscode-solr4: (pid 7701) 3580s; run: log: (pid 4541) 4751s
run: postgresql: (pid 7265) 3597s; run: log: (pid 4365) 4755s
run: rabbitmq: (pid 6691) 3612s; run: log: (pid 4331) 4756s
run: redis_lb: (pid 12236) 9s; run: log: (pid 4871) 4743s

It'd be a nice to have for opscode-pushy-server to be removed from the output of chef-server-ctl status if the Push Jobs package has been uninstalled.

I recently fixed a similar issue in reporting, we can probably borrow the
answer from that.
On Tue, Nov 3, 2015, 6:42 PM Tyler Fitch<EMAIL_ADDRESS>wrote:

A customer and I noticed that if the Push Jobs Server is uninstalled -
https://docs.chef.io/uninstall.html#push-jobs-title
When I do a chef-server-ctl status the listing for opscode-pushy-server
remains, just in the down state.
root@chef-server:~# chef-server-ctl status
run: bookshelf: (pid 7599) 3581s; run: log: (pid 4725) 4747s
run: nginx: (pid 10314) 336s; run: log: (pid 4887) 4742s
run: oc_bifrost: (pid 7619) 3581s; run: log: (pid 4439) 4754s
run: oc_id: (pid 7344) 3590s; run: log: (pid 4458) 4753s
run: opscode-erchef: (pid 7492) 3584s; run: log: (pid 4768) 4746s
run: opscode-expander: (pid 7416) 3585s; run: log: (pid 4641) 4749s
run: opscode-expander-reindexer: (pid 7459) 3584s; run: log: (pid 4656) 4748s
down: opscode-pushy-server: 135s, normally up; run: log: (pid 10278) 337s
run: opscode-solr4: (pid 7701) 3580s; run: log: (pid 4541) 4751s
run: postgresql: (pid 7265) 3597s; run: log: (pid 4365) 4755s
run: rabbitmq: (pid 6691) 3612s; run: log: (pid 4331) 4756s
run: redis_lb: (pid 12236) 9s; run: log: (pid 4871) 4743s
It'd be a nice to have for opscode-pushy-server to be removed from the
output of chef-server-ctl status if the Push Jobs package has been
uninstalled.
—
Reply to this email directly or view it on GitHub
https://github.com/chef/chef-server/issues/604.

Well that's very interesting @marcparadise - do you have the commit(s) from where you did that work?  If it's similar enough I'm really good at cargo culting code changes.  Lemme know!

So this is sorta an old thread but I ran into it with a recent version and thought I would post what I did.  I ended having to remove the opscode-pushy-server link from here:
/opt/opscode/service/
Once I did that I restarted chef-server and all was well.

Closing due to inactivity.


Add Codes (codesystem and valueset) for DosageOverrideReason

@costateixeira to chase @recipe-ruben

After an internal review, we concluded that nothing similar exists in Kmehr. At this point, we don't see any significant benefit, as this will primarily be used in MedicationLine and MedicationDispense. It was likely added to ensure compliance with most fields in MedicationLine.
We also noticed in one of the latest business documents from Nathan that "DosageOverrideReason" is a free text field rather than a code system.


procodile --procfile Procfile.dev start not working,  Error: Invalid command '--procfile'

Following is some describe.
when i first use procodile, i try to get help message like this
 ╰─ $ procodile -h
Usage: procodile -h [options]
    -r, --root PATH                  The path to the root of your application
        --procfile PATH              The path to the Procfile (defaults to: Procfile)

Okay,  because i use procodile only on development laptop, so, i use one separated
file, Procfile.dev, so, i try like this.
 ╰─ $ 1  procodile --procfile Procfile.dev start
Error: Invalid command '--procfile'

I know it working when reverse order.
 ╰─ $ procodile start --procfile Procfile.dev
No processes to start.

But, the first help message,  make me belive --procodile Procfile.dev should be a global
option, so, i can wrap procodile into a bash script like this:
#!/bin/bash

procodile --procfile Procfile.dev "$@"

But, it never working.
So, i think it should be a issue, thank you.
Another issue, when we have a Procfile with no content, when run procodile start,
get a not so friendly message.
 ╰─ $ procodile start
Traceback (most recent call last):
        2: from /home/zw963/Dropbox/common/ruby/procodile/bin/procodile:85:in `<main>'
        1: from /home/zw963/Dropbox/common/ruby/procodile/bin/procodile:85:in `new'
/home/public/Dropbox/common/ruby/procodile/lib/procodile/config.rb:29:in `initialize': undefined method `each_with_index' for false:FalseClass (NoMethodError)

Any update on this?

I fixed this issue on my Crystal fork, now, you can use like this:
 ╰──➤ $ bin/procodile --procfile spec/apps/basic/Procfile start
Started Procodile supervisor with PID 2080648


[ci skip] adding user @brendonwaters

Hi! This is the friendly automated conda-forge-webservice.
I've added user @brendonwaters as instructed in #12.
Merge this PR to add the user. Please do not rerender this PR or change it in any way. It has [ci skip] in the commit message to avoid pushing a new build and so the build configuration in the feedstock should not be changed.
Please contact conda-forge/core to have this PR merged, if the maintainer is unresponsive.
Fixes #12

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Paid support of the software

Hello, Alexsey!
We are going to install powergslb in our infrastructure. We need to make some fixes in it and also we are interested in paid support of it. Can we discuss it with you? If so please send me email to iprok at kinetiksoft dot com .
With best regards, Ivan.

Hi Ivan,
PowerGSLB is an educational project. It is not optimized for performance, security or multiple tenants. Of course, you can use it in the production infrastructure, but without any guarantees. If you find any errors or want to implement some additional features, please write them here and I will answer whether it is possible to implement them without a huge refactoring. As for paid support, I do not have enough free time to provide a decent level of service.
Aleksey

Thank you. It's very good for educational project.


Migrate codebase to Typescript

I really liked the idea of fastboot in the browser after seeing TopJohnWu tweet about it. I think every Javascript project would be better off using Typescript, so here we are.
This project is using a pretty unknown library for handling zip files, and it doesn't have Typescript support, so I ignored type errors on the factory.ts file.
In addition, I added and configured ESLint and Prettier for better code uniformity.
I also updated the project to Yarn 2 with PnP to replace node_modules. This system is faster and arguably cleaner than traditional node package resolution.
Some new scripts have also been added for ease.
yarn build: transpiles Typescript files to Javascript with separate definitions, good for distribution on NPM.
yarn bundle: bundles into a single file for browser usage.
yarn lint: runs ESLint to autofix issues and warn about others.
yarn format: runs Prettier to clean up formatting.

Thanks for working on this. I put off the code review until the core library became more stable because all of the source code is 100% changed by the reformatting, but I've added Prettier formatting and ESLint in upstream should you want to try again. The diff should hopefully be much smaller and manageable to review this time.
I'm also not sure if all the extra complexity added by PnP is worth it at this scale, but I'm open to switching to TypeScript.
zip.js has outdated bindings that could be updated to enable type checking for factory.js: https://github.com/DefinitelyTyped/DefinitelyTyped/tree/master/types/zip.js

Thanks for working on this. I put off the code review until the core library became more stable because all of the source code is 100% changed by the reformatting, but I've added Prettier formatting and ESLint in upstream should you want to try again. The diff should hopefully be much smaller and manageable to review this time.
I'm also not sure if all the extra complexity added by PnP is worth it at this scale, but I'm open to switching to TypeScript.
zip.js has outdated bindings that could be updated to enable type checking for factory.js: https://github.com/DefinitelyTyped/DefinitelyTyped/tree/master/types/zip.js

Migrated to TypeScript: https://github.com/kdrag0n/fastboot.js/commit/e9d34b97a08b31850057815beead5ff047e8e6b3
Sorry for not using your work; this was mostly an exercise for me to learn TypeScript, and I wanted to minimize the amount of unrelated changes. I've credited you for the suggestion in the merge commit.


unset storage_method (bsc#1001193)

due to the number of nodes in the ha_compute job ceph gets used
as a storage_method.
this is problematic on cloud7 as it will set the compute nodes to
be sles12-sp1. but sles12-sp2 is needed for HA by Crowbar, as there
are no more HA repos in the cloud-repos.yml file under the sles12-sp1
platform
This should fix https://ci.suse.de/view/Cloud/view/Cloud7/job/cloud-mkcloud7-job-ha-compute-x86_64/ and https://bugzilla.suse.com/show_bug.cgi?id=1001193
it may still be needed to push that change to jenkins after merging, not sure

I just realized that the gating for ha_compute is completely useless here, as it is an internal jenkins change

the latest gating for that job https://ci.suse.de/job/openstack-mkcloud/37227 went a bit further to fail at applying the nova proposal.


Transaction functions should be able to return values

Often times when you perform a transaction on the database, you want to end by retrieving a value.
The current implementation of transact/2 does not allow you to return a value from the nested function. Since all values are scoped, it is awkward to retrieve a simple value from within a transaction. This is a very common use case, and it should be easy.
transact/2 should return the value returned from within the transaction.
This change is backwards incompatible for any programs that relied on the returned :ok value from the transaction. The value returned after this change will be whatever is returned by the last expression executed within the transaction function.

fixed by [99939133052a266da934f56e89b0d9fe5792fed4]


extension host terminated unexpectedly

Issue Type: Bug
I am getting this error
At first I was not able to use emmet which I think is a default extension in 1.44.0 then uninstalled and re-installed vs code . After which I started getting this error. The settings and extensions information was still saved when i uninstalled and re-installed.
VS Code version: Code 1.44.0 (2aae1f26c72891c399f860409176fe435a154b13, 2020-04-07T23:31:18.860Z)
OS version: Windows_NT x64 10.0.17763

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5 CPU       M 450  @ 2.40GHz (4 x 2394)

GPU Status
2d_canvas: unavailable_softwareflash_3d: disabled_softwareflash_stage3d: disabled_softwareflash_stage3d_baseline: disabled_softwaregpu_compositing: disabled_softwaremultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: disabled_offrasterization: disabled_softwareskia_renderer: disabled_off_okvideo_decode: disabled_softwareviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: unavailable_softwarewebgl2: unavailable_software

Load (avg)
undefined

Memory (System)
3.80GB (0.38GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions: none

If you are using Quick Heal antivirus see #94858


Annotate over image

Hi Robert @haesleinhuepf,
As discussed, I created this notebook to teach users how to overlay texts on top of labeled elements with matplotlib annotate.
I placed it where I thought it would make sense.
Let me know what you think and I can fix/update it if necessary!
Best,
Marcelo

Awesome, thanks Marcelo @zoccoler !


layers: Validate correct vkSetEvent() use

Make sure vkSetEvent() is not called on an event that's in use
by and in-flight cmd buffer. Such use violates queue forward progress.

+1

cherry-picked and pushed to master


[Enhancement]: UI of "Guess The Correct Logo Game". and adding new features.

Do you want to have the enhancement of existing game ? 😀 Describe yourself..
I played this game and found that it's a nice game for children and adults who keep good interest in popular brands as there is always a  discussion between friends what's this brand's logo etc. But when I clicked on the game the thing I do not liked was that there is no catchy and attractive background and also the instructions are given in very dull manner with no attractive colors and nice fonts as well as there should be a background color of the start button. Also during playing , I found that the background there is also black which should be changed to a attractive and fancy as well as brighter one. I am also thinking of adding more logos and pause, resume  buttons if possible , I will try this one also...
Describe the solution you'd like
Firstly, I will improve the initial page UI then I will make the main page attractive and also change the fonts and background of the button ( start button ).
Also , if possible I will try my best to add more logos and a pause, resume button.
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

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

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

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

I have a query that this issue is closed at present , then what should I do?

Can you please tell me that should I self assign me this issue instead of caring about the issue is closed or open , will it be counted as my contribution and also will I be able to make a pull request for this?

/assign


setRowHeight() throws "can't call method [setHeightInPoints] on object, object is null"

https://dev.lucee.org/t/spreadsheetsetcellvalue-not-setting-as-a-number-in-excel/12671/9

Thrown if the specified row doesn't exist. Need to improve the exception message.


Enhance description editor to be rich markdown editor

Enhance description editor to be rich markdown editor

something similar to what GitHub uses
support various text controls and styling
support expanding text input area
continue to support preview
support pasting and uploading images/screenshots (#1641)

PR: #1715

:+1:


Also ensure that the system is exiting with an error status for errors and not for help.
/cc @Fryguy

Hope these checks can help us nail down #57 and determine if there are more concerns to handle.
@aredbeck can you comment whether test_invalid_option_with_simple_interface and test_error_with_standard_exception_handling meet your expectations?

Coverage remained the same at 100.0% when pulling e32aed48b2e48fad2c9728d177e508fd6bf80b20 on kbrock:test_stderr into 973c3c5888f7ec6d101047143a0f776c5d778f35 on ManageIQ:master.

Coverage remained the same at 100.0% when pulling e32aed48b2e48fad2c9728d177e508fd6bf80b20 on kbrock:test_stderr into 973c3c5888f7ec6d101047143a0f776c5d778f35 on ManageIQ:master.

Coverage remained the same at 100.0% when pulling 5822493aa6c54ddbbc45b0fb79f8aa6d4ef5ffee on kbrock:test_stderr into 973c3c5888f7ec6d101047143a0f776c5d778f35 on ManageIQ:master.

Coverage remained the same at 100.0% when pulling 5822493aa6c54ddbbc45b0fb79f8aa6d4ef5ffee on kbrock:test_stderr into 973c3c5888f7ec6d101047143a0f776c5d778f35 on ManageIQ:master.

Coverage remained the same at 100.0% when pulling 5822493aa6c54ddbbc45b0fb79f8aa6d4ef5ffee on kbrock:test_stderr into 973c3c5888f7ec6d101047143a0f776c5d778f35 on ManageIQ:master.

.. can you comment whether test_invalid_option_with_simple_interface and test_error_with_standard_exception_handling meet your expectations?

Assuming I'm reading this correctly, they do not.  I don't see a way to specify the exit code.  It seems to be a hardcoded -1.

@jaredbeck cool. I thought you were saying that it returns the wrong exit code.
So your request is to specify an exit code when the parameters are not correct?
And you want this on error (after the educate) rather than when the user specifies --help.
That sounds easy enough.
Let me know if I heard that incorrectly.


[BUG FIXED] use bash shell to run the create_venv.sh script

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


About sciplex3 dataset in Figure 2

Hi, thanks for your nice work!
I am interested in the results from Figure 2, but I couldn't find the sciplex dataset: sci-Plex samples (n = 290,889) of A549, K562, and MCF7 cell lines.
Could you please let me know how to obtain the preprocessed dataset as described in your code?
adata = sc.read('/home/icb/carlo.dedonno/projects/cpa-reproducibility/datasets/sciplex3_new.h5ad')
adata_old = sc.read('/home/icb/carlo.dedonno/projects/cpa-reproducibility/datasets/sciplex3_old_reproduced.h5ad'
Thank you very much!

@Naghipourfar and @ArianAmani Dear Mohsen and Arian, sharing those data would be very helpful for me as well. Thanks a lot!


feature: filetype specific plugins.

Did you check the docs?

[X] I have read all the lazy.nvim docs

Is your feature request related to a problem? Please describe.
I'm just wondering whats the correct way of settings up filetype specific plugins like rust-tools with lazy.nvim. Currently there isn't a clear example for this or is using the normal method with ftplugin folder suggested?
Describe the solution you'd like
More like a question. Asking for solutions.
Describe alternatives you've considered
ftplugin folder.
Additional context
No response

It depends on the plugin. If you want to lazy-load it on filetype, you can use ft=.
However, rust-tools needs to be integrated with lsp, so you should not lazy-load it on ft.
You can check this config for integrating rust-tools: https://github.com/appelgriebsch/Nv/blob/615e5cd0f480570988b24661c4e6acdfb2738b04/lua/plugins/extras/lang/rust.lua

@latipun7 your question had nothing to do with op's question. See my response.

@folke thanks again for a fast and helpful reply 👍


Target and input mismatch?

Hi i am getting a target and input mismatch. I'm unsure how the input has gotten to this shape, as it starts with a shape (b,t,c,h,w) of [1,12,15,256,256]. Do you have any suggestions on where to look in the code?
UserWarning: Using a target size (torch.Size([1, 1, 256, 256])) that is different to the input size (torch.Size([1, 20, 256, 256])). This will likely lead to incorrect results due to broadcasting.

Hi,
I guess you need to squeeze the second dimension of your target.
See the doc of the cross entropy loss implementation of PyTorch.
Cheers


Abort due to constraint violation (PRIMARY KEY must be unique)

http://pastebin.com/WyEiUXxj

give link to download fix version please


Generated files have multiple trailing newlines

$ Pods/SwiftGen/bin/swiftgen --version
SwiftGen v3.0.0

Build phase:
SWIFTGEN_BINARY="$PODS_ROOT/SwiftGen/bin/swiftgen"
SWIFTGEN_L10N_NAME="UIStrings.swift"
SWIFTGEN_IMAGES_NAME="UIImage+GeneratedAssets.swift"

SOURCE_FILES_PATH="REDACTED_APP_NAME"
SWIFTGEN_GENERATED_LOCATION="${SOURCE_FILES_PATH}/SwiftGen/Generated"
STRINGS_FILE_PATH="${SOURCE_FILES_PATH}/en.lproj/Localizable.strings"
ASSETS_PATH="${SOURCE_FILES_PATH}/Assets.xcassets"

if which ${SWIFTGEN_BINARY} >/dev/null; then
${SWIFTGEN_BINARY} strings --enumName UIStrings ${STRINGS_FILE_PATH} > ${SWIFTGEN_GENERATED_LOCATION}/${SWIFTGEN_L10N_NAME}
${SWIFTGEN_BINARY} images ${ASSETS_PATH} > ${SWIFTGEN_GENERATED_LOCATION}/${SWIFTGEN_IMAGES_NAME}
else
echo "warning: SwiftGen not installed, download from https://github.com/AliSoftware/SwiftGen"
if [[ ${CONFIGURATION} = "Debug" ]]; then
exit 0
fi
fi

Generated file:
// Generated using SwiftGen, by O.Halligon — https://github.com/AliSoftware/SwiftGen

import Foundation

// swiftlint:disable file_length
// swiftlint:disable type_body_length
enum UIStrings {
  /// An error occurred while attempting to access this data.
  case EmptyStateGenericError
  // snip
}
// swiftlint:enable type_body_length

extension UIStrings: CustomStringConvertible {
  var description: String { return self.string }

  var string: String {
    switch self {
      case .EmptyStateGenericError:
        return UIStrings.tr("empty.state.generic.error")
        // snip
  }

  private static func tr(key: String, _ args: CVarArgType...) -> String {
    let format = NSLocalizedString(key, comment: "")
    return String(format: format, locale: NSLocale.currentLocale(), arguments: args)
  }
}

func tr(key: UIStrings) -> String {
  return key.string
}

Notice the multiple trailing newlines, which are triggering SwiftLint. I had hoped that the manual SwiftLint rule disabling would mean I could stop ignoring these files, but currently they still warn on the trailing newlines.

Could you elaborate exactly where SwiftLint warns you? Is it only the trailing newline at the very end of the file or are there other warnings in these middle of the file too?

Also does it do the same when using --output PATH instead of using shell indirections > PATH?
(Btw using --output is recommended is it won't touch the output file if there is no modifications, avoiding Xcode to consider the file as changed and triggering a recompilation the next time for nothing, as Xcode is based on the modification date of the files)

It warns here in both the strings and images files:

And it does not have the same problem if you use --output PATH, so, I now have a great (and supported!) workaround. Thanks for the tip. Feel free to close this issue if that is satisfactory for you.


[Question] Is there a way to use your object generator to generate objects for an arbitrary OpenAPI Spec?

I'm writing an integration with Argo Workflows and I was thinking it'd be handy to reuse your object generator to process their OpenAPI spec but I wasn't able to find a way to integrate with it (I'm still fairly new to the scala ecosystem so I may just have missed it)

There are some general object generation tools in the ecosystem, but they are very general and on the other hand, parsing the spec is very easy as it is just normal json and we can just model what we want to create using a few transformations.
Object generation in this library is specifically made for kubernetes and is not suited for general object generation from OpenAPI, as there are some specific details about kubernetes spec that are not handled in general object generators, like types, helpers and custom patch commands.
However you can use the OpenAPI parsing and sbt plugin section as an idea or even a starting point to build your own.
This is the data model for parsing OpenAPI, which is gen translated into kubernetes object models and then several different representation of each is generated (object, pointers, encoders, decoders, ...).
All of these are located in here

Thanks for the quick response! Yeah I skimmed through the generation code, it generally made sense but I was struggling to find an entrypoint - I picked up https://github.com/eikek/sbt-openapi-schema but for a giggle I'll see if I can extend your stuff a bit as well

Spent a whole bunch of time levelling up on sbt and figured it out - not in a nice way or anything, I just ran the object generation plugin, then replaced the downloaded file with the argo spec, reran it, and scooped the generated objects.
sbt 'project objectsJVM' '++ 3'
<wait>
<replace api spec under /specifications>
sbt 'project objectsJVM' '++ 3'
cp modules/objects/.jvm/target/scala-2.12/src_managed/main <target>

It actually worked really well! Not perfect but pretty darn good, considering - definitely better than any of the other openapi-scala generators I found.


Updated Config Server section for Spring Boot 2.4+

Currently, the documentation does not state anything about the new config server property configuration (ref.: https://docs.spring.io/spring-cloud-config/docs/current/reference/html/#_client_side_usage -> "If you use Spring Cloud Config Client, you need to set the spring.config.import property in order to bind to Config Server. You can read more about it in the Spring Cloud Config Reference Guide.")
Hence, all Spring Boot 2.4 apps break when just adding the config-client dependency. In order to resolve the issue, the above-mentioned configuration has to be added.
A better approach would probably be to let Azure Spring Cloud handle this requirement by injecting those properties automatically, but I guess we arent there yet for this change right?

@bmitchell287 - can you review?

any updates on this @Jak-MS ?

tagging: @barbkess just in case @bmitchell287  is oof.

#assign: @karlerickson

We have already fixed this on service side so users no longer need to specify spring.config.import. We can close this PR.
Thanks a lot @dennis-menge for raising this!


Update codec version to the 4.1 version

#2855 was accidentally merged (and reverted) before it was ready, so here is an updated version.

@andresilva still has the commit in https://github.com/paritytech/substrate/pull/2801 .

@DarkEld3r I merged master into #2801 when master still had this change in. It's fine to merge it in this PR instead so you can keep attribution (I'll rebase and remove it from my PR once it is in). The issue is around synchronization of merges: we can't release finality-grandpa as is, we need to merge https://github.com/paritytech/finality-grandpa/pull/53 first since it has breaking changes, and the changes necessary to support that exist in #2801. And we can't merge this PR because it depends on an unreleased finality-grandpa.

also parity-codec 4.1 needs the fix of for crafted input that trigger huge allocation https://github.com/paritytech/parity-scale-codec/issues/109

@DarkEld3r is this good to be merged now?

@gavofyork I suppose it makes sense to wait a little bit more for 4.1.1 to be released.

@gavofyork I suppose it makes sense to wait a little bit more for 4.1.1 to be released (paritytech/parity-scale-codec#114).

Updated.

Can we not change the destination branch to @andresilva's grandpa branch for #2801? Merge it into this branch and merge into master with #2801 together.

@gavofyork I suppose it is OK to merge now.


Bots in UsGov environments stop working if you use the BotFrameworkHttpAdapter adapter

Github issues should be used for bugs and feature requests. Use Stack Overflow for general "how-to" questions.
Version
C# 4.9.2 and 4.9.3
Describe the bug
In the echo bot if you changes this line:
services.AddSingleton<IBotFrameworkHttpAdapter, AdapterWithErrorHandler>();
to
services.AddSingleton<IBotFrameworkHttpAdapter, BotFrameworkHttpAdapter>();
Bot auth will fail and the bot stops working.
To Reproduce
Steps to reproduce the behavior:

Create a new echo bot in UsGovPrd
Chat with it in web chat... it works
Change the line in the Describe the bug section and redeploy
Chat with it in web chat... it doesn't work

Expected behavior
It continues to chat. Should not have to use the correct (protected) ctor to get things working.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

Closing this issue as we now have a new adapter called CloudAdapter that support basic scenarios in any configured cloud environment.
The missing features are Streaming, OAuthPrompt support and Skills support.
https://github.com/microsoft/botbuilder-dotnet/issues/4850
https://github.com/microsoft/botbuilder-dotnet/issues/4851
https://github.com/microsoft/botbuilder-dotnet/issues/4853


Index total applied / failed user commands

Blocks: https://github.com/Granola-Team/mina-block-explorer/issues/898

Closing as dupe of #1204


Improve error messages

There is no good error handling and reporting functionality in this crate. This was reported several times, including by @pacak in #37:
% cargo doc2readme
     Reading /path/src/lib.rs
thread 'main' panicked at 'Unable to read file: expected literal', src/main.rs:194:71
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

I'm not sure how much that error can be improved (i.e. if syn returns any relevant details like line numbers). Ideas and PRs welcome.

Things syn returns include Span somewhere in their structure, Span can be used to identify lines: https://docs.rs/proc-macro2/1.0.36/proc_macro2/struct.Span.html
For this specific call you should be able to attach anyhow's context around this line: https://github.com/msrd0/cargo-doc2readme/blob/main/src/input.rs#L221


Function options set in nuxt.config.ts do not work

Version
@nuxtjs/algolia: latest
nuxt: 3.6.5
Reproduction Link
This can be easily replicated in the module's playground. See steps below.
Steps to reproduce
From Nuxt's documentation:

Your runtime config will be serialized before being passed to Nitro. This means that anything that cannot be serialized and then deserialized (such as functions, Sets, Maps, and so on), should not be set in your nuxt.config.

Therefore non of the options that take functions are working right now if set from nuxt.config.ts. This includes docsearch.transformItems,  docsearch.hitComponent, docsearch.transformSearchClient,  docsearch.navigator,  docsearch.getMissingResultsUrl. There might be others I missed.
You can easily see that these are missing, if you bind the public runtime config to the DOM (or just console.log() it):
// nuxt.config.ts
export default defineNuxtConfig({
  algolia: {
    apiKey: process.env.ALGOLIA_API_KEY,
    applicationId: process.env.ALGOLIA_APPLICATION_ID,
    docSearch: {
      transformItems(items) {
          return items
      },
      indexName: ""
    },
})

<template>

{{ algolia }}

</template>

What is Expected?
That the functions are working from the config, but since that is impossible, it would be better to remove these options and only allow them as component properties. Something like this:
<template>
  <AlgoliaDocSearch :transform-items="transformItems" :navigator="navigator" />
</template>

This looks cleaner than using one object as a prop:
<template>
  <AlgoliaDocSearch
       :options="{
          transformItems,
          navigator,
        }"
      />
</template>

Setting the apiKey and applicationId should also be inherited from the config file, because it is more convenient to set those values there. Right now if we use the component's option property, we cannot use those values set in the config.
Edit: I'm happy to work on this.

Heyo,
This is indeed an issue that cannot be resolved as it is how Nuxt runtime config works.
But your solution with providing functions as props to AlgoliaDocSearch seems like a very good idea!
Could you please create a PR with your proposed solution? I will review it in the upcoming days :)

Sure! I will try to find some time to do it this week


🐛 [firebase_messaging] Get token error Ios

The problem of getting a token seems to occur with conflicts to certain VPN's (In my case using the openVPN protocol)when the vpn is removed token is fetched normally.
`[connection] nw_socket_handle_socket_event [C8.1:3] Socket SO_ERROR [61: Connection refused]
[connection] nw_resolver_start_query_timer_block_invoke [C8] Query fired: did not receive all answers in time for device-provisioning.googleapis.com:443
Connection 8: received failure notification
Connection 8: failed to connect 1:61, reason -1
Connection 8: encountered error(1:61)
Task <03C242C6-D3F6-4AFB-BA14-7B205AC8F91C>.<4> HTTP load failed, 0/0 bytes (error code: -1004 [1:61])
Task <03C242C6-D3F6-4AFB-BA14-7B205AC8F91C>.<4> finished with error [-1004] Error Domain=NSURLErrorDomain Code=-1004 "Could not connect to the server." UserInfo={_kCFStreamErrorCodeKey=61, NSUnderlyingError=0x280756100 {Error Domain=kCFErrorDomainCFNetwork Code=-1004 "(null)" UserInfo={_NSURLErrorNWPathKey=satisfied (Path is satisfied), interface: utun3, ipv4, dns, _kCFStreamErrorCodeKey=61, _kCFStreamErrorDomainKey=1}}, _NSURLErrorFailingURLSessionTaskErrorKey=LocalDataTask <03C242C6-D3F6-4AFB-BA14-7B205AC8F91C>.<4>, _NSURLErrorRelatedURLSessionTaskErrorKey=(
    "LocalDataTask <03C242C6-D3F6-4AFB-BA14-7B205AC8F91C>.<4>"
), NSLocalizedDescription=Could not connect to the server., NSErrorFailingURLStringKey=https://device-provisioning.googleapis.com/checkin, NSErrorFailingURLKey=https://device-provisioning.googleapis.com/checkin, _kCFStreamErrorDomainKey=1}

FLTFirebaseMessaging: An error occurred while calling method Messaging#getToken, errorOrNil => {
    NSErrorFailingURLKey = "https://device-provisioning.googleapis.com/checkin";
    NSErrorFailingURLStringKey = "https://device-provisioning.googleapis.com/checkin";
    NSLocalizedDescription = "Could not connect to the server.";
    NSUnderlyingError = "Error Domain=kCFErrorDomainCFNetwork Code=-1004 \"(null)\" UserInfo={_NSURLErrorNWPathKey=satisfied (Path is satisfied), interface: utun3, ipv4, dns, _kCFStreamErrorCodeKey=61, _kCFStreamErrorDomainKey=1}";
    "_NSURLErrorFailingURLSessionTaskErrorKey" = "LocalDataTask <03C242C6-D3F6-4AFB-BA14-7B205AC8F91C>.<4>";
    "_NSURLErrorRelatedURLSessionTaskErrorKey" =     (
        "LocalDataTask <03C242C6-D3F6-4AFB-BA14-7B205AC8F91C>.<4>"
    );
    "_kCFStreamErrorCodeKey" = 61;
    "_kCFStreamErrorDomainKey" = 1;
}

flutter: Fetching token error: [firebase_messaging/unknown] Could not connect to the server.`
Pub:
Firebase Messaging: 11.2.11
Flutter Doctor:
[✓] Flutter (Channel stable, 2.10.3, on macOS 12.3.1 21E258 darwin-arm, locale en-DE)
[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 13.3)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2021.1)
[✓] VS Code (version 1.61.0)
[✓] Connected device (3 available)
[✓] HTTP Host Availability

Same issue here

In my case using the openVPN protocol)when the vpn is removed token is fetched normally.

I doubt if messaging plugin would have anything do with it or would be actionable from flutterfire side. I suggest you ask this question in firebase support for further resolution.
Closing from here as this doesn't seem specific to messaging. If you disagree, write in comments and I'll reopen it.


[CP] DevTools 2.28.3

Commit(s) to merge
https://dart-review.googlesource.com/c/sdk/+/336827
Target
stable
Prepared changelist for beta/stable
https://dart-review.googlesource.com/c/sdk/+/336827
Issue Description

https://github.com/flutter/devtools/commit/265103e0b66657c739b5cc3799b594c7c7ba7be4 - This will allow us to run a better quality survey in early Q1 and reach users on stable. There was a bug that prevented links from working in VS code (including the link to open the survey) that is now fixed on the latest flutter stable.
https://github.com/flutter/devtools/commit/e10051a92cdf104762e56218466be1b6a88c0369 - Copy does not work in VS code without this commit. This issue is severely affecting VS code user workflows and their DevTools sentiments
https://github.com/flutter/devtools/commit/8efcaed21640b5570d5fdbce51913c3e2586ecb4 - Prevents DevTools from inadvertently causing a crash on impeller.
https://github.com/flutter/devtools/commit/5701174c1c685f3852ed78451ada4c905cc7d8a5 - The new VS code sidebar that we shipped with Flutter stable was not respecting the user's IDE theme. This commit fixes that.
https://github.com/flutter/devtools/commit/f9a77cea3b5e869056344e71b4d7827bc6580907 - This adds a link in the DevTools footer to the DevTools video that our Flutter DevRel team published in October. This will help with publicizing that work and will help address user feedback from survey that they need more tutorials.
https://github.com/flutter/devtools/commit/276d7d9939ac9c790d8b5bf0243d28001776079a - prevents an upstream bug causing links to be broken when DevTools is embedded in VS code

What is the fix
Release a cherry-pick release of DevTools into the Dart / Flutter SDKs that includes the above 6 commits.
Why cherry-pick
For issues and reasons why we selected each commit, see above.
Risk
low
Issue link(s)
https://github.com/flutter/devtools/issues/6752
Extra Info
No response

https://dart-review.googlesource.com/c/sdk/+/336827 CQ dry run has passed and @athomas has approved. Running Submit to CQ currently. When will this CP roll into the Flutter stable branch?

Marking this as approved, it has already been merged.

Released in 3.2.1.


allow local coercions in contexts

From @rlewis1988
Hi Leo/all,
I ran into the following problem yesterday. Jeremy, Floris and I aren't sure if there's a nice way around this.
I want to define a coercion locally, to a type determined by a parameter. For example:
context
parameter A : Type
parameters (a : A) (s : A → A)
definition foo (n : ℕ) : A := nat.rec_on n a (λn' IH, s IH)
local attribute foo  [coercion]
end
Lean complains that coercions cannot be defined in contexts. But there shouldn't be anything wrong with this, right? Is it possible to do?
(This came up while proving facts about rational sequences, on the route to constructing R. Many of these theorems are true for arbitrary ordered fields F, not just Q, so we can reuse proofs by making things parametric in F. But it's convenient to have a coercion num -> F for writing things like eps / 2.)
Thanks!
-Rob

Actually the previous commit did not fully fix the problem. It allows local coercions.
However, we also need to allow coercions to parameters

I'm working on #668, #666 and #679. Since I'm revising the coercion module, I tried to accommodate this issue. Well, this one is a mess. Recall that the section mechanism in Lean is quite different from Coq.
I described it at #554. The summary is, the type of foo inside and outside of the section is Pi (A : Type), A -> (s : A -> A) -> num -> A. However, inside the section we maintain the alias foo := @foo A a s.
To close this issue, we have to handle aliases in the coercion management module, manage these parameters, and propagate them as we compute the transitive closure.
The benefit of implementing this issue is quite small. BTW, @rlewis1988 used a simple workaround for the eps / 2 problem. He defined
notation 2 := pnat.pos 2 dec_trivial

For these reasons, I'm closing this issue. The only reasonable way to support it would be to move to Coq's approach for handling sections, but this would create other problems (see issue #554).

In fact, the definition
notation 2 := pnat.pos 2 dec_trivial```
was not a workaround for this problem. This originally came up before we had rat as a concrete structure: the thought was that some of the work in constructing the reals could be reused, by making the construction parametric in the field being completed. This seemed to be a fair amount of trouble for little benefit, though, so once Jeremy finished rat I just used that explicitly. Since there's a (global) coercion num -> rat, this issue became irrelevant.

The `2` notation was simply for the positive natural number 2. I could have used a global coercion num -> pnat for this as well, but it seemed easier to avoid having so many coercions from num open when I only needed the pnats 2 and 3.

In any case, the original need for this feature is gone, and I doubt it will come up very often at all- I won't miss it.


[TESTING] does proxy test work magically?

Please confirm that your PR fulfills these requirements

[ ] Multiple commits are squashed into one commit.
[ ] The commit message follows Conventional Commits, which allows us to autogenerate release notes; e.g. fix(plugins/plugin-k8s): fixed annoying bugs
[ ] All npm dependencies are pinned.

wow my test magically catches the tab bug in proxy! 😮

Closed because the PR was just for testing, and we aren't going to move ahead with the approach in this PR.


java: Relax the visibility of Unpickler.NO_RETURN_VALUE class constant

Related to issue #49. Currently, I'm working around the private access control like this:
public static Object noReturnValue(){
	try {
		Field field = Unpickler.class.getDeclaredField("NO_RETURN_VALUE");
		if(!field.isAccessible()){
			field.setAccessible(true);
		}
		return field.get(null);
	} catch(Exception e){
		throw new RuntimeException(e);
	}
}

uggh, nifty hack :)  Will make it protected, I guess you're needing it in a subclass of Unpickler?

made remaining private fields protected in commit 7b20264bfecc7cff7862b3a7dc8c8af6b4b24f9f

This class constant should have the same visibility as the Unpickler#stack instance field. And protected is better than <package private>, because there are legitimate scenarios where one might want to subclass Unpickler and manipulate its state.

While we're at it, then I also have a scenario where I'm using reflection to access net.razorvine.pickle.objects.ClassDictConstructor#module and net.razorvine.pickle.objects.ClassDictConstructor#name fields in my application code.
I'm using it to perform programmatic "bulk mapping" from Python classes to Java classes. I know that you've introduced a special API for that in newest Pyrolite versions, but I haven't had a chance to research this area properly. Reflection did it for me at the moment.

It is not clear to me why you have to access those two fields again. They're passed as-is to the ClassDictConstructor's constructor when it's created and I assume you do the same if you subclass it yourself. So you already know the values of both fields beforehand, no?
The API to convert custom classes is really simple, it's also based on the same two values actually!  You register your own class creator instance for the given module+class name using the Unpickler.registerConstructor method.


WIP: Reimplement the bundle with jax-rs-pac4j

Hi,
As promised some time ago, I made a jax-rs integration of pac4j that works well with Jersey and dropwizard (see https://github.com/pac4j/jersey-pac4j/pull/1, the name of the project will change to jax-rs-pac4j).
So here is a reimplementation of the dropwizard-pac4j bundle.
I think this should simply replace the previous one because it adheres much better to the way dropwizard is meant to work: jax-rs-pac4j provides annotations to inject the filters on specific resource methods on top of the possibility to have multiple global filters for the whole application.
So with this new implementation of dropwizard-pac4j, we can do what was doable before (i.e., having one global SecurityFilter) and more (i.e. having multiple global SecurityFilter + injection of SecurityFilter, LogoutFilter and CallbackFilter).
The global filters are configured in the yml file.
The pac4j config must be configured programmatically using ConfigSingleton (provided by pac4j) in the application run method (because sharing an object between the application run method and the bundle run method is really complex in dropwizard). I validated that it was correct to do so (i.e. the content of the config is never used before the filters are actually processing requests).
I updated the tests, except for the Pac4jBundleTest because its content was almost the same as reading the java file… I can readd it, but I don't see the point.
Finally, it relies on SNAPSHOT version of pac4j (need some bugfixes) and jax-rs-pac4j (not yet released).
I'm of course open to any review and anyway we will need to wait for jax-rs-pac4j to be merged (and deployed) before this can be merged too :)

This is great! The J2E integration was a bit of a hack to begin with, so the JAX-RS/Jersey support is a much better fit for Dropwizard applications. This definitely moves us in the direction of providing the least amount of glue to connect a Dropwizard configuration file to pac4j.
Removing Pac4jBundleTest is fine. No sense keeping it around unnecessarily.
Let's reconvene on this PR once jax-rs-pac4j merges and a new version of pac4j is released.

jax-rs-pac4j is now available.
@evnm this PR looks good to me as well. I think we should merge it now (and not wait for next pac4j release which will be in any case a minor version)

Yes, it would be great if we can release a version of dropwizard-pac4j after 1.9.3 is released.
By the way @evnm, is this maven artifact deployed to maven central like the rest of the dropwizard artifacts?

If we move it under the pac4j organization, we'll need to change the groupId to org.pac4j so that we can release it.

@victornoel, no artifacts have been released for this library. I held off on releasing anything until APIs stabilized and we wanted to move forward with a 0.0.1 stable release.
@leleuj, I will take care of updating the package names after migrating the library over to the pac4j org.

Thanks guys :)