facebook backfeed via email notifications (unlikely)

this is a kinda crazy, somewhat dangerous, generally inadvisable idea that we almost certainly shouldn't do...at least, not as a service. 😎
@chrisaldrich had the interesting idea yesterday that since facebook sends email notifications for comments, likes, etc. i tried it, and i was surprised to find that they send email for individual likes, shares, etc as well as comments, and the comment emails include the full contents!

relevant part of the HTML email contents:
Ryan Barrett
 commented on the
link
 you shared.

looks like i can get the commenter's profile URL, the comment URL, and the post URL out of that.
hmm...

Not necessarily a good idea, but I don't think it's too complicated for most people to set up email forwarding, so they could theoretically set up any email from facebook.com to get forwarded to a bridgy address.
I'm not sure if that would avoid the password reset issue as bridgy would probably still receive the emails, but the user would receive the email to their account too.
Also potentially interesting for other silos too, as bridgy could easily map the user email address to their url and figure out what the email means.

thanks @grantcodes!
i could come up with gmail, sieve, etc email filters for people to install that would forward all facebook emails except password resets. or maybe only notification emails. still seems iffy security wise, and awful UX, but...meh?
quick poll time. hey @aaronpk @chrisaldrich @jgmac1106 @grantcodes @dshanske @tantek and others: if you could get facebook backfeed again by setting up an email filter like this, but it'd be somewhat sketchy security wise (discussion above), and might be inconsistent and incomplete, and the bridgy UI might be weak or nonexistent...would you do it? feel free to emoji respond 👍 for yes, 👎 for no. also feel free to @-mention anyone else who might vote.

I should have outlined this originally... Likely safer (for Bridgy and the end user) would be to follow the model of posting to WordPress via email (or services like Reading.am which allow posting to it via custom email addresses). Bridgy could provide users with a private hashed email address like<EMAIL_ADDRESS>which could be linked to their particular account to which they could manually (or automatically) forward the relevant Facebook notification emails. Upon receipt, Bridgy would know which account sent it and could also match it to the user's post URL as a check before sending the appropriate webmentions.
This would leave Bridgy free from being the potential source for security leaks and put the onus on the end user. You'd naturally need to have the ability to reset/change the user's hash in the case that they accidentally allowed their custom email address to leak, although generally this isn't a huge issue as emails which don't match the user's account/endpoints would be dropped and not send webmentions in any case. (In some sense it's roughly equivalent to my being able to visit https://brid.gy/twitter/schnarfed and clicking on the Poll now or Crawl now buttons. It's doable, but doesn't give a bad actor much. You'd probably want to rate limit incoming emails to prevent against mass spam or DDoS sort of attacks against Bridgy.)
A side benefit of all of this is that those who have kept their old email notifications could relatively easily get much of their past missing back feed as well. Or if they're missing back feed for some reason, they could easily get it by re-sending the relevant emails instead of some of the current manual methods. Perhaps allowing preformatted emails with those same manual methods could be used to do back feed for Facebook or other providers as well?
We could also put together some forwarding filters for common platforms like gmail to help people set up autoforwarders with appropriate keywords/data to cut down on the amount of false positive or password containing emails being sent to Bridgy.
The one potential privacy issue to consider(?) is that this set up may mean that Bridgy could be sending webmentions for private messages since users get both private and public message notifications whereas the API distinguished these in the past. To remedy this, the comment URL could be tested to see if/how it renders as a test for public/private prior to sending. Separately, since Bridgy doesn't need to store or show these messages (for long?), private messages could be sent, but potentially with a payload that allows the receiving end to mark them as private (or to be moderated to use WordPress terminology). This would allow the user's website to receive the notifications and give them the decision to show or not show them, though this may be a potential moral gray area as they could choose to show responses that the originator meant to be private communication. The API would have prevented this in the past, but this email method could potentially route around that.
(Originally published at https://boffosocko.com/2019/04/28/55750290/)

quick poll response without catching up on the full thread:

I generally trust bridgy to not abuse its access to things I authorize it
when I don't trust an app to access my email, I will go into gmail and set up a forwarding rule to send specific emails to a forwarding address so that I know it can only see those things
with those two things in mind, I would gladly authorize bridgy to read my emails and look at only the facebook ones, but would also gladly set up a forwarding rule to send facebook notifications to a special address

thanks guys! tentatively reopening.

Bridgy could provide users with a private hashed email address like<EMAIL_ADDRESS>which could be linked to their particular account to which they could manually (or automatically) forward the relevant Facebook notification emails.

yup! i expected i'd need to use account-specific, secret email addresses like this to protect against spoofed notification emails.

Bridgy could be sending webmentions for private messages since users get both private and public message notifications whereas the API distinguished these in the past. To remedy this, the comment URL could be tested to see if/how it renders as a test for public/private prior to sending.

ugh, true. worse, "could be tested" means i'd have to scrape, which...bleh.

private messages could be sent...though this may be a potential moral gray area as they could choose to show responses that the originator meant to be private communication.

right. you'd need the responder's permission too, which is why bridgy has generally drawn a hard line on backfeeding only fully public responses.

ugh, true. worse, "could be tested" means i'd have to scrape, which...bleh.

looking at this example post, here's the bit of HTML for the audience indicator. it looks more or less usable. (sigh.)

rough, in progress task breakdown for if we try to do this:

[ ] docs for which FB notification emails to turn on
[ ] email filter that matches only FB notification emails and forwards them
[ ] new FacebookEmail source with per-user email addresses<EMAIL_ADDRESS>
[ ] what's short name? facebook-email? fb-email?
[ ] new email feature?

[ ] new model for storing notification emails themselves
[ ] notification email HTML parser
[ ] new mf2 handler that renders from stored notification emails
[ ] handler to receive emails (API docs)

[ ] fetch FB post on web, scrape, check that it's public
[ ] create Response, insert propagate task

create accounts and email addrs manually for now.
open questions:

do i put any of this in granary? or just make a fake GR_CLASS?
still probably need OPD. so do we still poll, and just do OPD?

Just remembered this other relevant detail... Google now requires a security review in order to get access to the email API.
https://cloud.google.com/blog/products/g-suite/elevating-user-trust-in-our-api-ecosystems
Part of the review is hiring an outside consultant to do an audit, which has an expected cost of "$15,000 to $75,000 (or more)". So I say scrap the API plan and just do the forwarding rule version.
Gmail does require verification of the forwarding address before you can add the rule, so Bridgy will also need to provide a minimal "inbox" at the forwarding address so that the user can see the confirmation email Google sends and click the link there to confirm it.

ok all, this is ready for beta testers! let me know if you're interested.

I'm interested

@xavierroy great! grab me on IRC or email and i can set you up.

turn on facebook email notifications: https://www.facebook.com/settings?tab=notifications&section=email&view
see emails facebook has sent to you in the last 2 days (click on Other Emails; i think this includes notifications): https://www.facebook.com/settings?tab=security&section=recent_emails&view
email notifications help: https://www.facebook.com/help/530847210446227/

@snarfed I've enabled them. What next?

@xavierroy great! like i mentioned, next step is to talk to me on IRC or email or somewhere else private. i need to send you your secret email address to forward notifications to.

first confirmed backfed comment! yay. https://xavierroy.com/2019/07/05/weekend-plans/#comment-3021

I'm certainly interested in trying this.

@chrismaverick sure! have you used bridgy before? I'm happy to have more beta testers, but this won't be the best entry point if you're new to the service.

@snarfed Thanks!
I actually have been playing with IndieWeb and Bridgy yesterday today as a replacement for NextScripts SNAP (SOOOOO many issues) which I used as a replacement for AddLinkToFacebook, which fell apart after Cambrige Analytica.
I have a couple different sites, my personal blog (chrismaverick.com where I'd be testing it for now) and my podcast (voxpopcast.com, where I'll be holding off until I'm satisfied). I've got about 20 years of software dev and design experience... I'm not worried about a little learning curve growing pains if you don't mind having me.

@chrismaverick sure! sounds good. I'm on vacation without my computer right now, but i can set you up when I'm back in a week.

Out of curiosity, did anyone else have a problem getting email notifications turned on from FB? That is, I'm not having a problem with the bridgy at all (yet) because for some reason even though I have enabled all FB email notifications, it doesn't seem to actually email me any of them. I get the push notifications but not the emails. Which sort of makes it impossible to forward anything because I have nothing to forward. Was that an issue for anyone else?

(note, they're not being spam collected or anything. According to https://www.facebook.com/settings?tab=security&section=recent_emails&view I'm not being sent any)

@chrismaverick sigh, yes, i had the exact same problem with my own facebook account. i never managed solve it; i ended up using a different test account to get any notification emails.
(also props for finding that recent emails page! it was definitely useful for debugging.)

adoption of this has been underwhelming. one big cause may be that facebook is flaky and unreliable at actually sending email notifications, especially if you've turned them off before. see above.
bridgy hasn't received a forwarded FB email notification for over two months now. given that, i may turn this off entirely soon.

I've never gotten it to work. I even went as far as turning on ALL facebook notifications... which is 1) annoying and 2) literally sends me everything EXCEPT the one thing I care about in order to use this.

Looks like i was the only successful backfeeder. And would you know it, today I shared something on FB after a long while. FB email notifications are a bit finicky and they seem to work only when certain celestial stars are in the same line.
But it was a good effort and I guess we can shut down this experiment.

frustrating because this shouldn't be that hard except that FB makes it hard. We shouldn't even need to backfeed through email. The API is capable of handing comment over... and it used to until the Cambridge Analytica nonsense... (and still does for pages... just not personal accounts). But they turned it off for "security" after that... which is dumb, because that's not what the problem ever was.... if a post is publicly available, and the comments are public, there's no security issue with just copying it...  I could do it manually with a crawler... it's just that that's gross.

agreed on all counts. i share your frustration!
i actually wrote that crawler you mentioned, a bit ago. www.facebook.com uses JS, so crawling that is hard, but m.facebook.com (with a logged in session cookie) is pure HTML and eminently scrapeable, and has all the comments, likes, and reactions we want. plus, i already had experience scraping in https://facebook-atom.appspot.com/ . so, i added it to granary, with the hope of eventually using it in bridgy:
https://github.com/snarfed/granary/blob/1cb18202bfd9e0ef4e8380ad6bbfc8530d94964f/granary/facebook.py#L1821-L2046
sadly, FB's bot detection caught me pretty quickly as soon as i started trying this in bulk. i tried rotating IPs, different User-Agents, and all sorts of other browser and HTTP fingerprints they might be triggering on, but never got it working consistently. they obviously have more resources for that arms race than me. 😎
so, having exhausted three different approaches - API, email, scraping - i think it's time to throw in the towel. i'm open to other ideas though!

oddly enough... I literally just got three FB comment email notifications in the last couple minutes. I have no idea why. They're not linked to a blog post and I'm not home right now to try testing.... but I may have to play with it again later. Clearly... THEY'RE WATCHING!!!

no usage for almost 3 mos, so i'm planning to turn this off. let me know if you're about to start using it again!


Add new relic agent for cassandra

Will need an additional env variable:
https://github.com/ZhenzhenYin/cassandra-on-kubernetes/blob/6bc27ce2f30323a9323b22924a786634e33f9cde/cassandra-replication-controller-test.yaml#L32

nice, sry for the delay.   I think I need to make a closed sbbits/cassandra repo since the yml has an account key


Yarn or NPM?

@addyosmani I have spotted your Travis configuration is using Yarn, and there's no yarn.lock in the repo (there's package-lock.json file instead). I don't know whether it's done intentionally or it's just a simple mistake :) In order to get consistent installs across machines, it would be good to pick one of them.
WDYT?

Thank you for catching this, @mobily. This was an oversight after dropping our yarn lock from the repo and I should have switched over to npm for the Travis configuration too.  Happy to accept a PR to update otherwise this should be quick to update later in the day.

@addyosmani switched to npm here https://github.com/GoogleChromeLabs/react-adaptive-hooks/pull/17


chore(go.mod): replaces owned by 🐉

Description
Assign replacements to @stackrox/draco
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
Here I tell how I validated my change
go mod tidy
Reminder for reviewers
In addition to reviewing code here, reviewers must also review testing and request further testing in case the
performed one does not seem sufficient. As a reviewer, you must not approve the change until you understand the
performed testing and you are satisfied with it.

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

#10409  👈
#10408
#10407
#10373 : 1 other dependent PR (#10390 )
#10375
master

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

Join @janisz and the rest of your teammates on  Graphite

/retest

Merge activity

Mar 19, 7:03 PM GMT+1: Graphite rebased this pull request after merging its parent, because this pull request is set to merge when ready.


HTTPS is better than HTTP

find src/ -type f -print0 | \
  xargs -0 sed -i 's$http://phpmd.org$https://phpmd.org$g'

https://github.com/phpmd/phpmd/pull/509

I removed the final newlines (that were useful for cat).

Thanks 🍸


UnhandledPromiseRejection with the reason: ufo/provider/lsp.lua:61: service not started

Neovim version (nvim -v | head -n1)
NVIM v0.7.2
Operating system/version
archlinux
How to reproduce the issue
Install Coc & setup ufo with:
local ufo = require('ufo')

ufo.setup({
  open_fold_hl_timeout = 0,
})

vim.keymap.set('n', 'zr', ufo.openAllFolds)
vim.keymap.set('n', 'zm', ufo.closeAllFolds)

Then do some work, and at some point the following message keeps appearing at every 5 seconds (triggered by many actions, such as write file, or cursor movement):
Error executing vim.schedule lua callback: UnhandledPromiseRejection with the reason:
...vim/site/pack/packer/start/nvim-ufo/lua/ufo/provider.lua:51: ...site/pack/packer/start/nvim-ufo/lua/ufo/provider/lsp.lua:61: service not started
...re/nvim/site/pack/packer/start/nvim-ufo/lua/ufo/fold.lua:89: in function <Anonymous:70>

No ufo.log found.
Expected behavior
No error.
Actual behavior
Error.

https://github.com/neoclide/coc.nvim/blob/43744ba82bd348ea971b87483a13f06f3e46c94b/plugin/coc.vim#L134
Look like your coc.nvim issue. Run UfoDisable to verify whether or not.

Yeah, sometimes it's stopped. It would be nice if ufo could handle that state more gracefully.

Fixed, ignore service not started error message directly.


Update devops-faq.md

adding link to MSFT privacy statement re: data handling

@ElazarK

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Add scale to zero support for grpc ingress

Add scale to zero support for grpc ingress

Codecov Report

Merging #95 into master will decrease coverage by 0.74%.
The diff coverage is 14.28%.

@@            Coverage Diff             @@
##           master      #95      +/-   ##
==========================================
- Coverage   81.12%   80.37%   -0.75%
==========================================
  Files          32       32
  Lines        1907     1921      +14
==========================================
- Hits         1547     1544       -3
- Misses        334      349      +15
- Partials       26       28       +2

Impacted Files
Coverage Δ

pkg/controller/cell/resources/gateway.go
72.72% <0%> (-4.7%)
:arrow_down:

...troller/gateway/resources/istio_virtual_service.go
79.09% <0%> (-8.79%)
:arrow_down:

pkg/controller/service/resources/serving.go
90.9% <42.85%> (-5.15%)
:arrow_down:

Continue to review full report at Codecov.

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


[Menu] How to tell which MenuItem is focused

Description
I need to change focus to another element if the user presses the up-arrow when a Menu element's focus is on the first MenuItem.  There is no a way to do this currently, right?
I would like to submit a PR to implement this, but I would like to get your thoughts on my high-level approach before I do any more coding.
My idea is to add a new property named onFocusIndexChange to Menu.  This would be a callback that gets called when the Menu element's focus index is changed (via Menu.setFocusIndex(), Menu.constructor(), or Menu.componentWillReceiveProps()).  The arguments to the callback would be:

event - the event that triggered the change or null if the change was triggered by a React lifecycle event.
newFocusIndex - the index of the focused MenuItem within the Menu or -1 if no MenuItem is focused.

What do you think?  Should I move forward with coding this or is there a better approach I could take?

cc @oliviertassinari, @nathanmarks.

I am also happy to implement this by adding a new onArrowUpFromTopMenuItem function property to Menu (hopefully with a better name than that).  My above proposal was an attempt to come up with a generic API that isn't limited to my use case.

Your earlier API proposal sounds good to me, we would normal focus on bug fixes for the master branch. If you write some tests for that new onFocusIndexChange property, that should be fine 👍.

I'll do that.  Thanks!


Reevaluate use of Goose for migrations

During the implementation of #647 we determined a migration was needed to change the original schema value. Ideally, we wanted to implement this as part of the Goose migrations under the policy service, with the intention to support varying situations.
This proved more complicated than expected. The solution we tried to take was creating a migration with a 001 id and using this script. If failed due to how we create schemas outside Goose and figuring out the right conditions to preserve state.
It's curious if using a tool like atlasgo.io had resolved it, since you describe the state of the tables rather than the migration and let the tool determine the migration path.
DO $$
BEGIN
  -- if opentdf schema does exist and opentdf_policy schema does exist but contains no tables, drop it
  IF EXISTS (SELECT 1 FROM information_schema.schemata WHERE schema_name = 'opentdf') THEN
    IF EXISTS (SELECT 1 FROM information_schema.schemata WHERE schema_name = 'opentdf_policy') THEN
      -- if opentdf_policy schema contains only goose_db_version table, drop it
      IF (SELECT count(*) FROM information_schema.tables WHERE table_schema = 'opentdf_policy') = 1 THEN
        DROP SCHEMA opentdf_policy CASCADE;
        ALTER SCHEMA opentdf RENAME TO opentdf_policy;
      END IF;
    ELSE
      ALTER SCHEMA opentdf RENAME TO opentdf_policy;
    END IF;
  END IF;
END $$;

We used Alembic in the past, in a python service.   The nice feature was offline mode: https://alembic.sqlalchemy.org/en/latest/offline.html
Please consider a tool with this feature
Reference:
https://github.com/JeffGradyAtVirtru/python-example


Platform questions, Not an Issue!

Hi there.
I heard that there is something in the pipeline to add web support to the output platforms, something like javascript, maybe polymer-PWA?
Flutter is really nice for ios and android, but if it could output for the web as well, it would be amazing.
Any news on this topic.
(ps. sorry for asking here, wasn't sure where to ask)

That is explicitly stated as a non-goal for now. Theoretically this could change in the future.
If you keep the view layer as thin as possible you could still share a lot of code with AngularDart and only would need to reimplement the view layer in AngularDart.
Closing because there is nothing to do for the Flutter team.


Make a Handleable module to replace HTTP::Handler

This commit creates a handleable module that is identical to
HTTP::Handler. It then makes Handler a dummy class that includes
Handleable. This shouldn't break compatibility with any existing
applications that may inherit from Handler, while allowing more
composability when applications would rather include a module rather
than inherit from a class (such as when there is an existing class
hierarchy in place).
This is a proof of concept of what I mentioned in #3690

Before we hit 1.0 we usually don't care about introducing breaking changes. I'd directly make HTTP::Handler a module and make the necessary changes instead of adding another type.

@asterite made the changes!

If someone wants to cancel all those now useless Travis CI jobs that are queued, that would be nice :p I didn't mean to clog the tubes 😅

Even though this is a breaking change, the fix is easy (include instead of inherit). Let's do this!
@andrewhamon Thank you!


Wrong method?

Describe the bug
#args() method returns single TranslatableComponent instead of List<Component>
To reproduce
import:
    org.bukkit.event.player.PlayerAdvancementDoneEvent

    net.kyori.adventure.text.format.NamedTextColor

    net.kyori.adventure.text.serializer.gson.GsonComponentSerializer

    net.kyori.adventure.text.TranslatableComponent

    java.lang.Class

on PlayerAdvancementDoneEvent:
    set {_message} to event.message()
    set {_args} to {_message}.args()
    send "args %{_args}%" to (ops and console)

Expected behavior
#args() method to return List
Screenshots

Server information

skript-reflect: 2.3
Skript: 2.6.4
Bukkit: purpur-1876 (1.19.3)
Minecraft: 1.19.3
Java: 17
OS: CentOS 7

Additional context
I've tried with [TranslatableComponent]args() too

Issue is caused by these methods:

Since the second is a varargs method, {_var}.args() is also a valid call for that method.
Fixed in https://github.com/TPGamesNL/skript-reflect/commit/8f6bb82950906489a5e382efef04f82690ea757e by prioritizing non-varargs methods.
If anyone has this situation (two methods with same signature apart from varargs) and wants to use the varargs one instead, use either argument specification (e.g. {-msg}.args[ComponentLike[]]()) or specify a zero-length array in the method (e.g. {-msg}.args(new ComponentLike[0]))


Latest ledger ethereum app upgrade to 1.2.13 blows away user's contract data setting

Contract data is turned off even if it was previously on, when updating the ledger nano s or ledger x .   Thus, many interactions that were working for the user will no longer work, until somehow they figure out the cryptic error message and then manually allow contract data on the ledger device.
Instead, existing settings should be preserved where they are still supported in the new version.

@pscott, was there a PR merged in that you can link to this issue?

@cappaberra Hi!
There was no PR merged :(
I closed this issue for two reasons:

Unfortunately, this is not an issue we plan on fixing anytime soon.
If this were to be fixed, it would have to come from the OS side of things (BOLOS).

In the meantime, users will have to remember to put back their settings :(


Use gophercloud in Openstack scalers instead of interacting with the REST Api

Proposal
Right now, we have our own code to interact with openstack instead of using any native SDK. https://github.com/gophercloud/gophercloud is a native Golang SDK for Openstack that we are already using for e2e tests and it seems that brings all the capabilities that we need.
Using a documented and maintained SDK is easier and better for KEDA maintainability
Use-Case
No response
Anything else?
No response

Hi @JorTurFer,
I interested to take this, Could you please also give more context on how to work on this?

Hey!
Sure, thanks for helping us ❤
The main idea is to replace our self-maintained code for openstack with their supported SDK. There is a lot of code inside OpenStack scalers using the rest client directly, but it means that if there are any change in the apis, we need to address them instead of just bumping a SDK


Getting Start Smart Contract with solidity

This will helpful for everyone to access and explore community events source code.

Thanks @BunHouth for your contribution ❤


Add appropriate headings

Need to disable text overflow in test result descriptions before merging

Changing the the headers to use the h1/2/3 will change the formatting of the module reports from markdown that use these and likely breaking the formatting there.  Unclear on the need to change the class vs updating the CSS for this header classes.  If this is necessary, the markdown translation formatting will need to be addressed before merging.

We need to change the class so that when it is converted into a PDF, the headers are suitable for screen readers. This is a blocking feature for our accessibility rating. We can deal with any issues in the module reports once release/v1.1.2 is merged into dev.

If this change breaks other elements of the report, we shouldn't approve and merge until the break is addressed within the same PR.  No need to knowingly introduce bugs especially into a release branch.

Looking back at this, there is no plan to release the module reports in this release. Module reports will be released end of Q1 whilst we expect to release this end of this month at the latest. When the time comes, and we open the PR to merge release/v1.1.2 into dev (which may reveal these bugs) I will fix any CSS issues.

PDF report currently cuts off overflow of test descriptions. This should be updated to text-wrap or shorten test descriptions to improve accessibility.


bug: vap and vip triggers which-key even when typed within the delay

Did you check docs and existing issues?

[X] I have read all the which-key.nvim docs
[X] I have updated the plugin to the latest version before submitting this issue
[X] I have searched the existing issues of which-key.nvim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
NVIM v0.10.0
Operating system/version
Arch Linux
Describe the bug
If there is no further input after entering char-wise visual mode, which-key window opens.
But if the user immediately expands the selection within the delay using motions/textobjects, it does not.
This is a great default behavior, and it seems to apply to all motions and textobjects such as vj, vw, vaw, vis, and even the mappings from other plugins.
However, I found 2 exceptions to this: vap and vip.
Steps To Reproduce
Typing vap or vip in quick succession within the delay still opens which-key.
Typing viw, vas etc. in quick succession does not trigger which-key.
All of them correctly opens which-key and closes it after typing the final key, if typed slowly.
Expected Behavior
I don't think ap and ip in particular were meant to be exceptions.
Health
which-key: require("which-key.health").check()

- OK Most of these checks are for informational purposes only.
  WARNINGS should be treated as a warning, and don't necessarily indicate a problem with your config.
  Please |DON't| report these warnings as an issue.

Checking your config ~
- WARNING |mini.icons| is not installed
- OK |nvim-web-devicons| is installed

Checking for issues with your mappings ~
- OK No issues reported

checking for overlapping keymaps ~
- WARNING In mode `n`, <gc> overlaps with <gcc>:
  - <gc>: Toggle comment
  - <gcc>: Toggle comment line
- OK Overlapping keymaps are only reported for informational purposes.
  This doesn't necessarily mean there is a problem with your config.

Checking for duplicate mappings ~
- OK No duplicate mappings found

Log
No response
Repro
vim.env.LAZY_STDPATH = ".repro"
load(vim.fn.system("curl -s https://raw.githubusercontent.com/folke/lazy.nvim/main/bootstrap.lua"))()

require("lazy.minit").repro({
  spec = {
    { "folke/which-key.nvim", opts = {} },
    -- add any other plugins here
  },
})

Blazingly Fast (Adjective)
Like Folke; Reminding of Folke.
A phrase created in attempt to describe the amount of work Folke gets done per day.


Fixed Css file applying margin to both pages, began working on 2nd HTML

Fixed CSS file applying margin to both pages, began working on 2nd HTML CSS framework

Fixed merge conflict, however we aren't exactly sure why there was a conflict to begin with..


fix for 843

What this PR does / why we need it:
this fixes issue 843
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
#843
Special notes for your reviewer:
Release note:

Remove the mysql container. Install the mysql client in s3 container. Add a script in the s3 container to connect to tidb and initialize it (with a retry mechanism, to retry 5 times, 5 seconds apart), then start the s3 process

@kumarashit @sfzeng @nguptaopensds please review, thanks


Remove optional edition_factory arg from Section#initialize

The default value of this argument was only ever overridden in specs. We can use partial mocking to avoid needing to inject the edition factory in the specs and thus simplify the code.

@chrislo: Thanks for reviewing.
Rebasing against master and force-pushing in preparation for merging.


Individual status codes in batch request available after exception

Is your feature request related to a problem? Please describe.
Following guide for .net FineGrainedBatchingAsync I am trying to get response codes for each sub request of batch request for both, blob deletions and tier change. Example below is for tier change requests, m_workingContainer is BlobContainerClient.
auto batch = std::make_shared<Azure::Storage::Blobs::BlobContainerBatch>(m_workingContainer.CreateBatch());
std::vector<std::pair<std::string, std::shared_ptr<Azure::Storage::DeferredResponse<Azure::Storage::Blobs::Models::SetBlobAccessTierResult>>>> tierChangeResults;
...
for(auto &item : items)
{
  auto tierChangeResult = batch->SetBlobAccessTier(item.blobName, Azure::Storage::Blobs::Models::AccessTier::Cool);
  auto tierChangeResultP = make_shared<Azure::Storage::DeferredResponse<Azure::Storage::Blobs::Models::SetBlobAccessTierResult>>(std::move(tierChangeResult));
  tierChangeResults.push_back(std::make_pair(std::move(item.blobName), std::move(tierChangeResultP)));
}
...
try {
  auto batchRes = m_workingContainer.SubmitBatch(*batch);
  for(auto &res : tierChangeResults)
  {
	  auto batchResponseCode = static_cast<int>(res.second->GetResponse().RawResponse.get()->GetStatusCode());
          std::cout << "Blob " << res.first << " status " << batchResponseCode << std::endl;
  }
}
catch (const Azure::Storage::StorageException& e)
{
...
}

If all blobs exist, the batch request returns without exception and I can iterate over all sub requests and get status 200 for each one. If I remove one of blobs before tier change request is sent, SubmitBatch throws exception:
if (httpStatusCode != Core::Http::HttpStatusCode::Accepted)
{
  throw StorageException::CreateFromResponse(std::move(pRawResponse));
}

When pRawResponse is moved to exception constructor, it is no longer available in DeferredResponse. If any sub request fails, even though all other blobs will be moved to cool tier I do not know about it because whole batch fails and I cannot get response codes. How can I get sub request status after the exception?
Describe the solution you'd like
pRawResponse copy to exception constructor
Describe alternatives you've considered
StorageException API that returns list of blob names and status codes
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[x] Description Added
[x] Expected solution specified

@Essyer  How about this:
try {
  auto batchRes = m_workingContainer.SubmitBatch(*batch);
} catch (StorageException& e) {
  // batch request fail, none of the subrequests were executed
}

for(auto &res : tierChangeResults) {
  try {
    auto batchResponseCode = static_cast<int>(res.second->GetResponse().RawResponse.get()->GetStatusCode());
    // subrequest successful
  }
  catch (StorageException& e) {
    // subrequest fail
  }
}

@Jinming-Hu Thank you very much, with your example I get all information I need.


Missing Linux from the comment in the IServiceCollectionExtensions class in the Shared project of the TodoTemplate project template

There is a comment on line 9 of the IServiceCollectionExtensions.cs file in the Shared project of the TodoTemplate project template that does not include Linux:

the correct state of the change should be something like this:

here is a link to this line on GitHub:
https://github.com/bitfoundation/bitplatform/blob/fcb83f2c6da2a1b42c792497b39f977426e96917/src/Templates/TodoTemplate/Bit.TodoTemplate/src/Shared/Extensions/IServiceCollectionExtensions.cs#L9

resolved!


GSD statiscs incorrectly displays actual species & other taxa numbers in a CoL release if sync has been cancelled

For @mdoering & @thomasstjerne attention:
Statistic pages in the PREVIEW & PORTAL shows full set of species & higher taxa for WoRMS Mollusca in CoL of 2024-03-25, but Ari reported that only 792 name usages from WoRMS Mollusca are present in CoL: https://github.com/CatalogueOfLife/data/issues/634

Here are links with "fake" statistics:
https://www.catalogueoflife.org/data/dataset/1130
https://www.catalogueoflife.org/data/source-datasets
Investigating the case, we learned that WoRMS Mollusca automatic sync was cancelled. As a result, data in CoL were not properly updated, but statistic reports failed to display actual numbers in the release.

That indeed is sth we havent properly catered for. When a sync starts it first deletes the previous sector data. When it then gets cancelled none or just a few records exist and no new metrics are created. We need to build new metrics also for canceled or failed syncs


Fix a Bug that options set manager always stuck at the first option …

Fix a Bug that options set manager always stuck at the first option set field that it found.
The case that if there are multiple options sets fields in the same entity, your code will always get the text of the options from the first options set field, and that's not correct.

Hi @alduzzen1985 , why did you close my pull request, it's fixing some issue with getting the options

Hi @alduzzen1985 , why did you close my pull request, it's fixing some issue with getting the options

HI, @MaherSaleem I did a merge in the main branch. I'll release in the next few hours a new version with all your fix.


RST - field with one or more space(s) in a string raises ValueError: max() arg is an empty sequence

Dataset.get_rst raises exception if there's a field with one or more space(s) in a string:
The background is actually that I wanted to create an empty row, but rst apparently ignored a line with only "" and so I came across this error.
Python 3.6.6
tablib v2.0.0
import tablib
recs = [
  {"name": "john", "desc":"some guy"},
  {"name": "mary", "desc": " "},  # note a space in desc
]
t = tablib.Dataset()
t.dict = recs
t.get_rst()

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "D:\WinPython-64bit-<IP_ADDRESS>Qt5\python-3.6.6.amd64\lib\site-packages\tablib\core.py", line 240, in _get_in_format
    return registry.get_format(fmt_key).export_set(self, **kwargs)
  File "D:\WinPython-64bit-<IP_ADDRESS>Qt5\python-3.6.6.amd64\lib\site-packages\tablib\formats\_rst.py", line 244, in export_set
    column_widths = cls._get_column_widths(dataset, max_table_width)
  File "D:\WinPython-64bit-<IP_ADDRESS>Qt5\python-3.6.6.amd64\lib\site-packages\tablib\formats\_rst.py", line 94, in _get_column_widths
    str_lens, word_lens = cls._get_column_string_lengths(dataset)
  File "D:\WinPython-64bit-<IP_ADDRESS>Qt5\python-3.6.6.amd64\lib\site-packages\tablib\formats\_rst.py", line 53, in _get_column_string_lengths
    word_lens[i] = max(word_lens[i], _max_word_len(text))
  File "D:\WinPython-64bit-<IP_ADDRESS>Qt5\python-3.6.6.amd64\lib\site-packages\tablib\formats\_rst.py", line 27, in _max_word_len
    return max(len(word) for word in text.split()) if text else 0
ValueError: max() arg is an empty sequence

Many thanks for the report. You might check if the pull request linked above fixes the issue for you.

Thanks, problem solved.
Unfortunately, RST still cuts off the spaced line.

You might open a new issue about the entire blank line, with example data.


Add some debug utils

These debug utils can be invoked directly from a debugger or in the code to trigger conditional breakpoints, dump state, or other similar things.

Current dependencies on/for this PR:

master

PR #41

PR #42

PR #43

PR #44   (this PR)

This comment was auto-generated by Graphite.


Feature Request: insert selected commands in shell history

is it possible to insert selected commands in shell history ?

We could append to .bash_history if it exists, right now we are pretty shell agnostic so not sure whether we want to adopt the burden of doing this for all the various shells. But zsh and bash probably divide the market so could be worth it. Wdyt?

I think it makes sense to support zsh and bash.
My thinking is the behavior would be to append the ran command as the most recent. Pressing up again would give you the query command itself.

We could append to .bash_history if it exists

This won't actually work, in this case it only shows up after shell restart. I think we're going to have to go in the direction of the code @rvangsgaard linked

Probably won't go with this for shell-ai, it will require being inside a terminal emulator like tmux or screen to have buffer access, I'll let other projects go down that route


is_on_floor triggers even if the side or top of a character body is touching something while is_on_ceiling and is_on_wall don't work when floor_max_angle is above 4

Godot version:
v4.0.dev.custom_build.7085c0d80
OS/device including version:
Ubuntu 20.04
Issue description:
when ever a character body is touching a surface it will say its on the floor even if its on a wall or touching the ceiling. this will only happen if floor_max_angle value is above 4 if its below 4 then this bug docent happen
Steps to reproduce:
make a CharacterBody3D with a collision shape and have set floor_max_angle to 4 then move the character body into a wall and ceiling and have it print if its on the floor or not
Example project
CharaterBody3D.zip

Please provide a test project otherwise the devs may be stuck in helping you.

ok i figured out that this only happens when floor_max_angle has a value of 4 or above and i provided an example project to show how it works

It looks like the expected behavior. floor_max_angle is currently the maximum floor angle in radians.
A value of 3 is 171.887 degrees so the walls are considered floor.
A value of 4 is 229.183 degrees so the walls and ceilings are considered floor.
I'm going to change it to work with degrees to make things clearer though, it doesn't make a lot of sense to set an angle in radians in the editor.


Add support for hamlib rig control

At a minimum, be able to read radio parameters like frequency and mode.

I have the https://github.com/k0swe/kel-agent/tree/gohamlib branch which integrates https://github.com/dh1tw/goHamlib/. The integration was pretty smooth and accomplishing the functionality I wanted. However, it creates a dynamically linked dependency on hamlib, requiring the user to already have it installed on their system whether they want to use hamlib or not.
I'd rather decouple the system by integrating with the rigctld TCP protocol. That means a bit of work either finding or creating a replacement, but I think it will be worth it.

The https://github.com/k0swe/kel-agent/tree/gohamlib branch has been languishing too long. I'm definitely considering the TCP route again.

https://github.com/k0swe/rigctld-go exposes a couple of useful methods now.


If game starts lobby should be set to not joinable or deleted

@weberlii @danielLutziger was macht da am meiste sinn? Lobby lösche oder eif "not joinable" wenns game afangt?

lobby is already not joinable as soon as gameId is no longer null (game starts), but deleting it might also make sense

see #168


Use the 'open' package instead of child_process

Node notifier works great on windows but the exec command opening the cart does not. I think the open package that I used is cross platform. Let me know what you think!
Cheers!
Arjen

oops, let me update testing...

Oke, so a package like 'open' or 'opn' works well, but I lack the knowledge of sinion to write a test case...


Merge pending for gtk 0.1.0 release

We're using a way too old rust version on appveyor. We should update it. I merge. Thanks @gkoz!


feat(aws setup): adding downloadable aws setup scripts

First pass at scripting the aws setup.

will add links and a wget command to download at the top of aws account page
@robzienert and I want to provide a better initial vpc setup, and potentially switch this over to teraform. But this is a start.

@lwander aws.tar.gz is a tar of this directory. The way our site seems to work, you can't use wget to download a folder. Instead, I created a build script to bundle everything up, so I can actually download all the files. This is in prep for the nice docs I'm gonna add to the site.
In regards to idempotency, I would say these script is a solid 'meh'. Things don't fail completely when you re-run them (amazon will fail to create identical subnets or roles, for example, but it won't fail the script). If things go really funky, you're probably gonna have to start commenting out functions, or at worst, just following all the steps one at a time from the script. That's why this is a first pass - @robzienert and I think that terraform is a better solution for this but we don't have time this quarter. I'd rather contribute something (at least running the AWS CLI commands is easier than clicking around in the silly AWS UI) than keep putting it off.
@dorbin as I mentioned in a comment above, I do want to move this readme.md content into that page, that is just a second step that will happen after these script files get hosted on the site. These scripts that whole page so that you don't have to click around in the console.

This PR just turned 1 today! How do people feel about it now that it's a year old?

@anotherchrisberry I'll close it. But first, I will toast to the long life it has had. It's seen so much.

:birthday:


feat: Facts about the coherent topology

We show several facts about effective epimorphic family and the coherent topology in precoherent categories.

bors r+


canShareVia('instagram'..) returns null despite instagram installed

Hello,
Sharing an image through instagram works on android, however on ios 9 it is not.
Instagram is whitelisted in the .plist within LSApplicationQueriesSchemes.
window.plugins.socialsharing.canShareVia('instagram', null, null, share_image_uri, null, function (e) {
console.log(e);
// returns null
});
Where else should I add configuration parameters to make it work?
Thanks!

With this in your .plist:
<key>LSApplicationQueriesSchemes</key>
<array>
  <string>instagram</string>
</array>

And this button in your html:
<button onclick="window.plugins.socialsharing.canShareVia('instagram', 'msg', 'subj', ['www/img/ionic.png'], null, function() {alert('yep')}, function(errormsg){alert(errormsg)})">can use instagram?</button>

You should see an alert: yep when running it on iOS with Instagram installed. Reopen if not but also share your app with me.

Thanks, so far this is what I did except msg and subj were null and image uri not being an array.
Will test for this also.

Hi! I'm try this solution, but this not working for me.
Can't run Facebook, Messanger, Watsapp and other apps via 'canShareVia' on iOS (but this work fine on Android). Have any ideas?


What happened with the ES5 Version of popperjs

I tried to update to the newest version of popper today. My old version was 1.0.6. In the newest version of popper, the compiled es5 Version of Popper seems to be gone (With Ticket https://github.com/FezVrasta/popper.js/pull/220) and replaced by an umd and an esm build.
When I use the esm version of popperjs in my build, my gulp-uglify breaks.
CodePen demo
Build Stack issues cannot be showcased on codepen.
Steps to reproduce the problem

Setup an FE Build Stack with Gulp and Popper.js
Add popper.js (vs.) to your build with the esm version
use gulp-uglify

What is the expected behavior?
There is a compiled es5 version of popperjs next to the esm / umd / es6 versions
What went wrong?
The current behaviour is, that popper.js does not get included because of the above problem.
Any other comments?

The old ES5 version is now called UMD (since now both UMD and ESM are transpiled to ES5).
If you want the ES6 version, it's in /dist/index.js

Ah yeah, now it works.. thanks for that blazing fast anwer..


Component reconcilers concept draft

 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.


Support Scala 2.11

Using the plugin with a Scala 2.11 Play project leads to
[error] Modules were resolved with conflicting cross-version suffixes in {file:/Users/mariussoutier/Workspace/Play.workspace/graylog2-demo/}root:
[error]    com.typesafe.play:play-functional _2.11, _2.10
[error]    com.typesafe.akka:akka-actor _2.11, _2.10
[error]    com.typesafe.play:play-json _2.11, _2.10
[error]    com.typesafe.play:play _2.11, _2.10
[error]    com.typesafe.play:play-iteratees _2.11, _2.10
[error]    com.typesafe.play:twirl-api _2.11, _2.10
[error]    com.typesafe.akka:akka-slf4j _2.11, _2.10
[error]    org.scala-stm:scala-stm _2.11, _2.10
[error]    com.typesafe.play:play-datacommons _2.11, _2.10

Do you have any updates related to this? Is there risk if I exlude 2.10-versioned dependencies and just user this plugin?

I implemented a scala version of this plugin (using scala 2.11.5). https://github.com/mcornejo/play2-graylog2 @dennisoelkers do you think if it's worth to create a pull request?


Add nonce to create message endpoint

https://github.com/discord/discord-api-docs/pull/5078

HTTP wrappers was removed in #201 so this is not needed


glxinfo: replace with mesa-demos

Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Diff LGTM, need to test everything.

Bump :P

By the way @SuperSandro2000 https://github.com/NixOS/nixpkgs/blob/ef953b49524346ba0a55a5025f9b7fe5c9307301/pkgs/games/grapejuice/default.nix you accidentally added back Grapejuice with this change (it was removed because its dead and its useless now that Roblox blocked wine)

my rebasing skills 💀

@K900 we want to merge this?


Select cheat feature and bug fixes regarding type conversions

I have implemented the feature to select the cheat dll from the current directory.
Bug fixes regarding type conversions.

Thank you! Changes approved!


[DO NOT MERGE] test AddMember implemented using AddMembers

Piggy back on the existing AddMember tests to validate AddMembers

same lint failure :)


Use pathogen-repo-build workflow for builds

Original commit is from @joverlee521, I just rebased it on master
Simplify build workflows by using the reusable Nextstrain pathogen-repo-build workflow.
This does not update the fetch-and-ingest-branch workflow since that will require additional changes to support the custom configs.
This does remove the notify_pipeline_failed step in the workflows. They can be added as a separate job in the workflow if we think they are still helpful.

IIUC, notify_pipeline_failed only notified when the Github action failed, not when the build itself failed. Am I right @joverlee521?

This looks good to me, I'll merge and bear in mind if it fails

IIUC, notify_pipeline_failed only notified when the Github action failed, not when the build itself failed. Am I right @joverlee521?

Yup, this was added to catch errors that were raised outside of the build itself. We should still get Slack notifications if there is an error during the build.


Test container ooms within conmon-rs

We are relying on OOM testing within crio. We need to add native OOM testing within this repo.
ref: https://github.com/containers/conmon-rs/pull/365

This is done :+1:


Bp++: Rangeproof PR

This second PR introduces Bulletproofs++ Rangeproof argument. Builds on top of #205 . The companion protocol is described here:
BP_PP_proofs.pdf
The companion rust code is available at https://github.com/sanket1729/rust-bulletproofs-pp
This includes the core protocol with some optimizations but does not include

Complex multiplication.
Early termination.
Stack space efficient prover/verifier.
An implementation using gej_add_gej
Another trick in BP++ to reduce the size of c_vec. This is a non-trivial one to prove, but IMO the benefits are not worth the complexity.

Benchmarks show 0.95 ms for verification and ~4 ms prover time for 64 bit rangeproofs

PR #216 contains two commits that fix compilation and tests.

I think you already did these changes, but, for reference, I made a checklist for rebasing https://github.com/BlockstreamResearch/secp256k1-zkp/discussions/266.

Is there anything which has to be done to get this merged?
I mean I am trying to estimate when we may have BPP rangeproofs in secp256k-zkp

Is there anything which has to be done to get this merged?

Yes, this needs a lot more review and polishing. As far as I know, no one is working on this right now.

Thank you for providing the update! So I assume it sounds more like ~year(s) and certainly not weeks/month


Move off rollup-plugin-ts

Resolves: #135, #112, and maybe #133
Because tsc/glint and rollup each have their own cleanup phase, we can't use reliably the same dist directory anymore.  So, for type-declarations, I've chosen the creatively named "dist-types" for where the declarations will go and configured in the tsconfig.json to output the declarations there.
We originally(?) had a setup similar to this, but had wanted to have type-declaration generation combined with the rollup build. @rollup/plugin-typescript and rollup-plugin-ts both offer this, but rollup-plugin-ts does way too much, and has been causing us problems. @rollup/plugin-typescript doesn't integrate with Glint/gjs/gts, so that's not an option either. (See #135, and related links for more context).
Additionally, the approach here, with manually specifying extensions looks annoying, but it's flexible, and I have an addon using gjs/gts with this approach already.

For v1 addons (since merging https://github.com/ember-cli/ember-cli/pull/10283) we have dist (used by Ember CLI, not published) and declarations for the types. Should we align to this here?

probably, yeah.
I'll make the change once I can figure out how to get CI to a green state.
Right now it looks like I need to first rip out the @types/* packages

The failing test seems unrelated to @types AFAICT, and quite close to passing though?

That is different from what I saw locally!
That is quite exciting.
Also, at some point, I should figure out how to get CI / local not able to be different

oh, and look at that, it's green! yay!

Before merging the pull request, I think it'd be a good idea to see the suggested change in action. That is, check that the code change works on existing v2 addons (at least two---one with Glint and another without). The pull requests for those addons could serve as a migration guide. What do you think?
On a related note, will there be an announcement from the Embroider team to v2 addon authors that they should migrate away from rollup-plugin-ts soon?

that they should migrate away

@ijlee2 Personally, I don't see that v2 addon authors, v1 addon authors, and app authors would have to adhere to whatever is in the blueprint. If rollup-plugin-ts is working for folks, then they shouldn't feel like they need to change anything.
Also, we have tests that generate addons and ensure that they are buildable :tada:

announcement from the Embroider team to v2 addon authors

This is typically the blueprint changelog, yeah?

I did one yesterday: https://github.com/universal-ember/ember-primitives/blob/main/ember-primitives/package.json#L17
Tho, still using dist-types instead of declarations

Looks like Glint doesn't have a way to not clear the screen in watch mode -- I'll open an issue over there

Nice work.

It's a team effort! I super appreciate the level of depth you all put in to reviewing this PR! ♥


Updates from QPT 1.1.27 release

Thank you! we'll merge internally and let you know.

Merged these changes in this internal PR:
https://git.corp.adobe.com/AdobeDocs/commerce-knowledge-base.en/pull/204
Verified articles changed.
Complete!

Closing PR.


Installer should skip packages when a newer version is already installed

Description of problem
When upgrading Dnn, the process will fail if you have a newer version of an extension already installed which is also included in the platform upgrade package. We have run into this with sites when they upgrade to v8.0.4 due to the SecurityAnalyzer and version mismatches.
Description of solution
We should modify the upgrader to skip any packages which a newer version is already installed.
Affected version

[x] All Versions

IMO we should detect just the latest version in that folder and delete all other packages prior to install (like the currently installed one is deleted after the install)

@sleupold that wouldn’t solve the issue. The problem isn’t we have multiple versions staged to install. The problem is we have a newer version already installed and dnn isn’t prepared for such a situation and errors out.

AFAIK the installer terminates with an error, when a newer version of the same module is already installed. This used to be in place since DNN 4

That’s the problem. It errors out when it could easily continue.

ah, I see, it should log the issue and continue with the next package.

Just to confirm the current behavior. If you have DNN 7.04.02 with Security Analyzer 8.1.3 already installed, when you upgrade to 8.00.04 it errors out because Security Analyzer (v1.0.2 I think?) is in the Install/Modules folder. Further... if you delete the Security Analyzer module package from that folder and then attempt to restart/resume the DNN upgrade install, it finishes successfully (or at least appears to do so). So I agree, log/report the error and continue sounds great to me! Especially since a user new to upgrading DNN would have almost no idea how to work through this error and stop leaving a seemingly incomplete upgrade.

Yes, I think it should just log as Info and continue with next package.

I am closing this issue since it had no progress. If someone plans on submitting a PR for that any soon, please comment or reopen

Reopening this one as it is a needed item to provide for a safer upgrade experience going forward.


Changes to the query page

Type of change
Update
Does it contain breaking changes?
No
Description
Due to a change in the searchseco-api, a GitHub personal access token is no longer required to query. The field has been removed.
We can now specify a branch for our query through a new field.
𝕨𝕖 𝕤𝕖𝕔𝕦𝕣𝕖 𝕤𝕖𝕔𝕠𝕤.

Please do not merge this pull request yet! The searchseco-api dev branch has not been merged with main currently.


Extend redfish API for Chassis

Add Chassis root and collections path
Add Thermal reference object
Add Power reference object
Add unit-tests

@RackHD/corecommitters @zyoung51 @keedya @uppalk1 @tannoa2

*** BUILD #195 ***
Test Name: Redfish Chassis Root should return a valid chassis root
Error Details: expected 200 "OK", got 500 "Internal Server Error" Response body: { name: 'TypeError',   message: 'ejs:9\n    7|     "Members@odata.count": <%= nodes.length %>,\n    8|     "Members": [\n >> 9|         <% nodes.forEach(function(node, i, arr) { %>\n    10|             {\n    11|                 "@odata.id": "<%= basepath %>/Chassis/<%=node.id%>"\n    12|             }\n\nObject #<Object> has no method \'forEach\'',   stack: 'TypeError: ejs:9\n    7|     "Members@odata.count": <%= nodes.length %>,\n    8|     "Members": [\n >> 9|         <% nodes.forEach(function(node, i, arr) { %>\n    10|             {\n    11|                 "@odata.id": "<%= basepath %>/Chassis/<%=node.id%>"\n    12|             }\n\nObject #<Object> has no method \'forEach\'\n    at eval (eval at <anonymous> (/var/lib/jenkins/jobs/on-http/workspace/build/node_modules/ejs/lib/ejs.js:485:12), <anonymous>:19:14)\n    at returnedFn (/var/lib/jenkins/jobs/on-http/workspace/build/node_modules/ejs/lib/ejs.js:514:17)\n    at Object.exports.render (/var/lib/jenkins/jobs/on-http/workspace/build/node_modules/ejs/lib/ejs.js:316:37)\n    at /var/lib/jenkins/jobs/on-http/workspace/build/lib/services/redfish-validator-service.js:9:3179\n    at b (domain.js:183:18)\n    at tryCatcher (/var/lib/jenkins/jobs/on-http/workspace/build/node_modules/on-core/node_modules/waterline/node_modules/bluebird/js/main/util.js:26:23)\n    at Promise._settlePromiseFromHandler (/var/lib/jenkins/jobs/on-http/workspace/build/node_modules/on-core/node_modules/waterline/node_modules/bluebird/js/main/promise.js:503:31)\n    at Promise._settlePromiseAt (/var/lib/jenkins/jobs/on-http/workspace/build/node_modules/on-core/node_modules/waterline/node_modules/bluebird/js/main/promise.js:577:18)\n    at Promise._settlePromises (/var/lib/jenkins/jobs/on-http/workspace/build/node_modules/on-core/node_modules/waterline/node_modules/bluebird/js/main/promise.js:693:14)\n    at Async._drainQueue (/var/lib/jenkins/jobs/on-http/workspace/build/node_modules/on-core/node_modules/waterline/node_modules/bluebird/js/main/async.js:123:16)\n    at Async._drainQueues (/var/lib/jenkins/jobs/on-http/workspace/build/node_modules/on-core/node_modules/waterline/node_modules/bluebird/js/main/async.js:133:10)\n    at Async.drainQueues (/var/lib/jenkins/jobs/on-http/workspace/build/node_modules/on-core/node_modules/waterline/node_modules/bluebird/js/main/async.js:15:14)\n    at process._tickDomainCallback (node.js:459:13)',   context: {} } Error: expected 200 "OK", got 500 "Internal Server Error"     at Test.assert (/var/lib/jenkins/jobs/on-http/workspace/build/node_modules/supertest/lib/test.js:205:15)     at assert (/var/lib/jenkins/jobs/on-http/workspace/build/node_modules/supertest/lib/test.js:132:12)     at /var/lib/jenkins/jobs/on-http/workspace/build/node_modules/supertest/lib/test.js:129:5     at Test.Request.callback (/var/lib/jenkins/jobs/on-http/workspace/build/node_modules/supertest/node_modules/superagent/lib/node/index.js:746:30)     at Test.<anonymous> (/var/lib/jenkins/jobs/on-http/workspace/build/node_modules/supertest/node_modules/superagent/lib/node/index.js:135:10)     at Test.EventEmitter.emit (events.js:95:17)     at IncomingMessage.<anonymous> (/var/lib/jenkins/jobs/on-http/workspace/build/node_modules/supertest/node_modules/superagent/lib/node/index.js:938:12)     at IncomingMessage.EventEmitter.emit (events.js:117:20)     at _stream_readable.js:920:16     at process._tickDomainCallback (node.js:459:13)
Stack Trace: Error: expected 200 "OK", got 500 "Internal Server Error"
    at Test.assert (node_modules/supertest/lib/test.js:205:15)
    at assert (node_modules/supertest/lib/test.js:132:12)
    at node_modules/supertest/lib/test.js:129:5
    at Test.Request.callback (node_modules/supertest/node_modules/superagent/lib/node/index.js:746:30)
    at Test.<anonymous> (node_modules/supertest/node_modules/superagent/lib/node/index.js:135:10)
    at IncomingMessage.<anonymous> (node_modules/supertest/node_modules/superagent/lib/node/index.js:938:12)
    at _stream_readable.js:920:16

^^ just pushed a fix for that jenkins failure, next build should pass.

*** BUILD #197 ***
Test Name: Http.Api.Skus "before each" hook: reset test DB for "should process a .tar.gz file with a sku"
Error Details: timeout of 2000ms exceeded. Ensure the done() callback is being called in this test.
Stack Trace: Error: timeout of 2000ms exceeded. Ensure the done() callback is being called in this test.

test this please

+1

-1 until the promise chains are fixed, then +1

@benbp updated to address comments.

Thanks! :+1:

@RackHD/corecommitters is this OK to merge now?


Added filesystem-watcher bevy feature for cfg(unix). Fixed outdated comment and broken link in readme.

Fixes #172 for tiled examples (Added filesystem-watcher dependency)
Updated readme to reflect correct location of tiled-rotated.rs and no tiled_map feature.

@theseatoad is file-system-watcher unix only?

Looking at https://github.com/bevyengine/bevy/blob/main/crates/bevy_asset/src/io/file_asset_io.rs ... it stated "Watch for changes requires the filesystem_watcher feature and cannot be used on wasm32 / android targets". I put this dependency under unix since that is a known working spot.


Conflict with the InventoryHUD+ mode

(I am a player from China forgive me for my poor English and can only use translation software for reporting)
Describe the bug
The mod link : https://www.curseforge.com/minecraft/mc-mods/inventory-hud-forge
-->
InventoryHUD+ is a secondary module. It can display equipment, handhelds, etc. on the screen.
As soon as the player wears any MWC hat gear (including night vision devices, gas masks), these features disappear
but as long as the player is armed with an MWC guns, these functions are restored!
Reproducibility

[x] I reproduced this issue with as few other mods as possible installed.
[ ] I am unable to reproduce this issue consistently.

Steps to reproduce the behavior:

Bring any WMC hats
Do not hold any MWC guns

Expected behavior
Wearing any MWC helmet will not affect the functionality of the InventoryHUD+ mod
Screenshots or/and videos
In the picture, I am wearing a GPNVG-18 night vision device and holding an M4A1

After that I switched to other items and the ability to show my equipment disappeared

When I took off the GPNVG-18 night vision device it was back to normal

Logs or/and crash reports
https://mclo.gs/oJMQHkE
Versions

Modern Warfare Cubed: MWC-0.1-Dev-11
Forge: 1.12.2 - <IP_ADDRESS>60
Java: jre 8 - 8.0.301
Java JVM: Hotspot

Specification:

OS: Windows 10
CPU: Inter i5 2500k
GPU: GTX 950
RAM Allocated: 16GB

All of these info below are not necessary but may become necessary depending on the issue

CPU Speed: 3.30GHz Boost around 4.4Ghz
CPU Core: 4
CPU Threads: 4
RAM Size: 16GB
RAM Speed: 1866MHz
Storage Type: SSD
GPU VRAM: 1GB

Most of these information can be found in the task manager

Additional context
Add any other context about the problem here.

this is pretty weird, I also use inventory hud+ but I can't replicate the issue. Possible mod incompatibility?

Can you tell me your inventory HUD+ mod version?

this is pretty weird, I also use inventory hud+ but I can't replicate the issue. Possible mod incompatibility?

Can you tell me your inventory HUD+ mod version ?

头盔有HUD, 只是被覆盖了, betterPVP也会这样

this is pretty weird, I also use inventory hud+ but I can't replicate the issue. Possible mod incompatibility?

Can you tell me your inventory HUD+ mod version ?

InventoryHUD-1.12.2.forge-3.4.4

Fixed in #106


PWX-33543

The PR uses a Describe() block for each individual test as opposed to and It() block, which is used for test steps.
This allows us to specify either individual test name(eg, StoragePoolExpandDiskAdd or test suite StoragePoolExpandSmoky in FOCUS_TEST field in a jenkins job.
It is also more aligned with how tests are arranged in history.
Test run:
https://jenkins.pwx.dev.purestorage.com/job/Users/job/alyu/job/pxdv-7-dev/4/console
Note that AddDisk test was a failure and torpedo correctly and clearl reported that  -- which is a legit product bug that I'm going to look at tomorrow.
The torpedo code runs as expected.

@alicelyu-px can you check this
https://aetos.pwx.purestorage.com/resultSet/testSetID/333815
every test name is coming as StoragePoolExpandDiskResize


Inline basic docs? Plugins docs?

Just saw a PR with lots of nice aliases and functions supporting git operations. It's nice to discover the new functionalities through the pull requests, but is it also documented in a more orderly fashion somewhere? The Wiki is quite empty, and the plugin directory has no readme(s).
Also, when I first ran the installation, I was asked about which plugins and aliases to turn on, but not a himt about what each option brings with it. I sent the repo to a friend and he just enabled everything, which resulted in a conflict of two plugins and an unusable shell.
Bottom line is, are you planning any sort of one-line descriptions for each component and a longer description somewhere as well, or is it intentionally geared towards people who want to dig into each file and figure it out for themselves? :) I prefer some basic starter documentation to nothing at all...

I think a good compromise would be to add descriptive comments for each function inside of the plugin.bash files.The git.plugin.bash for example has no comments at all and therefore is very difficult to read

Plugins that conflict should be reported as bugs with as much detail as can be provided.
I know this thread is old, but it is an interesting suggestion, so bump  - perhaps plugins should be required to have some sort of minimal documentation?

@nwinkler I think this can be closed.

Woah. I didn't notice I've been using bash-it for that long...
Well, I'm not sure this has been resolved but if it's not gaining any votes, may as well close it.


perf(gateway): poor performance when usingtower-http 0.4.0

Description

Upgradingtower-http https://github.com/fleek-network/ursa/pull/420 introduced a regression.
Running with tower-http = "0.4.0"
Summary:
  Success rate:    1.0000
  Total:    67.1743 secs
  Slowest:    7.6177 secs
  Fastest:    0.2376 secs
  Average:    3.2824 secs
  Requests/sec:    14.8866

  Total data:    25.61 MiB
  Size/request:    26.22 KiB
  Size/sec:    390.35 KiB

Response time histogram:
  0.238 [1]   |
  0.976 [5]   |
  1.714 [42]  |■■■
  2.452 [99]  |■■■■■■■■■
  3.190 [319] |■■■■■■■■■■■■■■■■■■■■■■■■■■■■■
  3.928 [346] |■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■
  4.666 [125] |■■■■■■■■■■■
  5.404 [38]  |■■■
  6.142 [14]  |■
  6.880 [5]   |
  7.618 [6]   |

Latency distribution:
  10% in 2.2874 secs
  25% in 2.7222 secs
  50% in 3.2680 secs
  75% in 3.7787 secs
  90% in 4.3584 secs
  95% in 4.8468 secs
  99% in 6.3910 secs

Details (average, fastest, slowest):
  DNS+dialup:    3.3122 secs, 0.9046 secs, 3.4577 secs
  DNS-lookup:    0.0000 secs, 0.0000 secs, 0.0000 secs

Status code distribution:
  [200] 1000 responses

Running with tower-http = "0.3.5"
Summary:
  Success rate:    1.0000
  Total:    5.3613 secs
  Slowest:    3.5667 secs
  Fastest:    0.0480 secs
  Average:    0.2648 secs
  Requests/sec:    186.5223

  Total data:    25.67 MiB
  Size/request:    26.28 KiB
  Size/sec:    4.79 MiB

Response time histogram:
  0.048 [1]   |
  0.400 [949] |■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■
  0.752 [0]   |
  1.104 [0]   |
  1.455 [0]   |
  1.807 [0]   |
  2.159 [0]   |
  2.511 [0]   |
  2.863 [0]   |
  3.215 [0]   |
  3.567 [50]  |■

Latency distribution:
  10% in 0.0727 secs
  25% in 0.0839 secs
  50% in 0.0937 secs
  75% in 0.1120 secs
  90% in 0.1326 secs
  95% in 3.3333 secs
  99% in 3.5269 secs

Details (average, fastest, slowest):
  DNS+dialup:    3.3217 secs, 1.0806 secs, 3.4888 secs
  DNS-lookup:    0.0000 secs, 0.0000 secs, 0.0000 secs

Status code distribution:
  [200] 1000 responses

Notes

Checklist

[x] I have ensured that my version is up-to-date
[x] I have ensured that my issue is reproducible
[x] I have ensured that my issue is not a duplicate

@ozwaldorf this one would be a good one to tackle to do more research. First I would talk to the team though to discuss before writing code.


Describe SemaphoreSlim timeout of 0 (zero) milliseconds

Fixes #1936
I also caught a few smart quotes in there as well and :punch: knocked those out.

@rpetrusha Fixed the exception condition, which is a reversion to the prior language.

@Guardrex, no, I should pay more careful attention. It doesn't apply to cref. This looks good to merge.


Use a single property to refer to tests in build tests and global tests.

It is confusing that builds running inside tests are using the "tests" properties while global tests are using "tasks".

Taking a more detailed look this is somehow intended. Tests from within the build context are a list of scripts to run at the end of the build but global tests are more complex where a test represent a unit of work running on a separate build which can also run multiple scripts(tests). The tasks property was selected to avoid the need to have a tests property within an element of the tests list.
Simplifying this will require some design.

@zanderso FYI

engine-flutter-autoroll:flutter-engine-flutter-autoroll-fadd0b52-b0ab-442b-bf42-d4bb567ad1f0-1692949426

#100074

Hi @bhalprathamesh6 this issue is unrelated to quarantining devices with incorrect rotation setting.


Able to launch kubectl shell, but commands are not working.
Getting the certs fixed will be tested with https://github.com/rancher/rancher/issues/12283


Mleap 0.10.1 exception with Spark 2.2.1

Due to adaption of Mleap to Spark 2.3.0 in pull request #364  this error occurs when executing the with Spark 2.2.1 Spark-Example shown in the readme.md of this repository.
Exception in thread "main" java.lang.NoSuchMethodError: org.apache.spark.ml.feature.StringIndexerModel.getStringOrderType()Ljava/lang/String; 	at org.apache.spark.ml.bundle.ops.feature.StringIndexerOp$$anon$1.store(StringIndexerOp.scala:23) 	at org.apache.spark.ml.bundle.ops.feature.StringIndexerOp$$anon$1.store(StringIndexerOp.scala:14) 	at ml.combust.bundle.serializer.ModelSerializer$$anonfun$write$1.apply(ModelSerializer.scala:87) 	at ml.combust.bundle.serializer.ModelSerializer$$anonfun$write$1.apply(ModelSerializer.scala:83)

I am also facing with this error recently when trying to dump a pipeline that contains StringIndexer.

@stefanm7 @Khiem-Tran I've picked up this issue, will keep you posted. Thanks for the issue report.

Once merged, https://github.com/combust/mleap/pull/383 should fix this issue, thank you!

Closing, as PRs has been merged to master.

how could I solved this problem?


Adding applicationinsights(1), costmanagement(1), labservice(1), medi…

…a(1), orbital(1) resources

Description of your changes

Fixes #366

applicationinsights

[x] azurerm_application_insights_standard_web_test

costmanagement

[x] azurerm_cost_anomaly_alert

labservice

[x] azurerm_lab_service_lab

media

[x] azurerm_media_services_account_filter

orbital

[x] azurerm_orbital_contact_profile

I have:

[x] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

Manually and Uptest:

/test-examples="examples/costmanagement/costanomalyalert.yaml"

/test-examples="examples/insights/applicationinsightsstandardwebtest.yaml"

/test-examples="examples/labservices/labserviceslab.yaml"

/test-examples="examples/media/serviceaccountfilter.yaml"

/test-examples="examples/orbital/contactprofile.yaml"


Alarm Sound Dropdown Never Saves Selected Option | Ubuntu 17.10

I haven't tried to reproduce this on other distributions but in Ubuntu 17.10 x64, I cannot seem to select and save a different sound for the Alarm than "beeps.ogg".
Reproduction:

-Right click on Extension in top bar and select: Open Settings
-Select the Alarms tab
-Select a different Alarm Sound
-Note no Save button or any other button than the "X" (Close button) on the top right
-Click Close Button
-Repeat navigation to Alarm Sound area and note, it's now set back to "beeps.ogg"

Fixed.

Let me know whether it works.

Apologies for the slow response. I have just checked and confirmed that it is fixed! Great job, thanks for the quick fix.


Points not loading on Firefox Web

Points not loading on Firefox web browser -
Version - 132.0.2 (64-bit)
OS - Windows 11
Bug with Points not loading -

Thanks for reporting. I'm unfortunately not able to reproduce this on Firefox on macOS, where everything seemingly works fine, and I don't have access to a windows machine. If you are able to track down the issue, or submit a pull request with the fixes, I'm happy to incorporate.


Slow import times

I've been seeing this for a long time but oh-my-posh is very slow to import.  Anyone else seeing the same behavior?  It's taking 13 seconds to import in a new PS session.
Here's a snip from my profile loading log.  The other two modules I explicitly import are much faster.
[00.2775] Importing PSReadline
[00.2846] Importing oh-my-posh
[13.7760] Importing z
[14.8844] Setting theme parameters

I don't have that issue (it would annoy the crap out of me). What's your setup/settings?

Nothing special.  This is just the initial import of OMP.  The delay is before I apply my theme settings, which is fast.  When I get some spare time I'll create a clean test user and see if I can track down the cause.

K. I'm currently trying to get everything to work on Linux as I switched environment 🐧

I've been having similar slowness - I ran powershell whilst importing with PSDebug
This is the result I had... I hope this helps in identifying the bottleneck.

Time
Command

104
>>>> Import-Module PSColor   -ErrorAction Stop >$null

9
>>>> @{

0
! CALL function ''    (defined in file   'C:\Users\ehiller\AppData\Local\omega\system\psmodules\PSColor\<IP_ADDRESS>\PSColor.psd1')

2
>>>> Add-Type   -assemblyname System.ServiceProcess

0
! CALL function ''    (defined in file   'C:\Users\ehiller\AppData\Local\omega\system\psmodules\PSColor\<IP_ADDRESS>\PSColor.psm1')

4
>>>> .   "$PSScriptRoot\PSColorHelper.ps1"

1
>>>>   ##############################################################################

0
! CALL function ''    (defined in file   'C:\Users\ehiller\AppData\Local\omega\system\psmodules\PSColor\<IP_ADDRESS>\PSColorHelper.ps1')

5
>>>> .   "$PSScriptRoot\FileInfo.ps1"

1
>>>>

0
! CALL function ''    (defined in file   'C:\Users\ehiller\AppData\Local\omega\system\psmodules\PSColor\<IP_ADDRESS>\FileInfo.ps1')

6
>>>> .   "$PSScriptRoot\ServiceController.ps1"

1
>>>> # Outputs a line   of a ServiceController

0
! CALL function ''    (defined in file   'C:\Users\ehiller\AppData\Local\omega\system\psmodules\PSColor\<IP_ADDRESS>\ServiceController.ps1')

7
>>>> .   "$PSScriptRoot\MatchInfo.ps1"

1
>>>> function   MatchInfo {

0
! CALL function ''    (defined in file   'C:\Users\ehiller\AppData\Local\omega\system\psmodules\PSColor\<IP_ADDRESS>\MatchInfo.ps1')

8
>>>> .   "$PSScriptRoot\ProcessInfo.ps1"

1
>>>> function   Write-Color-Process

0
! CALL function ''    (defined in file   'C:\Users\ehiller\AppData\Local\omega\system\psmodules\PSColor\<IP_ADDRESS>\ProcessInfo.ps1')

12
>>>> $global:PSColor   = @{

0
! SET $PSColor = 'System.Collections.Hashtable'.

35
>>>>   $script:showHeader=$true

0
! SET $showHeader = 'True'.

37
>>>>   New-CommandWrapper Out-Default -Process {

55
function New-CommandWrapper  >>>> {

0
! CALL function 'New-CommandWrapper'    (defined in file   'C:\Users\ehiller\AppData\Local\omega\system\psmodules\PSColor\<IP_ADDRESS>\PSColorHelper.ps1')

90
>>>> Set-StrictMode   -Version Latest

93
>>>> $target = $Name

0
! SET $target = 'Out-Default'.

94
>>>> $commandType =   "Cmdlet"

0
! SET $commandType = 'Cmdlet'.

98
if( >>>> Test-Path   function:$Name)

98
if( >>>> Test-Path   function:$Name)

106
>>>> $proxy = @'

0
! SET $proxy = '

0
CMDLET_BINDING_ATTRIBUTE

0
param(

0
...'.AMETERS__

244
>>>> $originalCommand   = Get-Command $target

0
! SET $originalCommand = 'Out-Default'.

245
>>>> $metaData =   New-Object System.Management.Automation.CommandMetaData `

0
! SET $metaData = 'System.Management.Automation.CommandMetadata'.

247
>>>>   $proxyCommandType = [System.Management.Automation.ProxyCommand]

0
! SET $proxyCommandType = 'System.Management.Automation.ProxyCommand'.

251
>>>> $proxy =   $proxy.Replace("CMDLET_BINDING_ATTRIBUTE",

0
! SET $proxy = '

0
[CmdletBinding(HelpUri='http://go.microsoft.com/fwlin...'.

253
>>>> $proxy =   $proxy.Replace("COMMAND_NAME", $target)

0
! SET $proxy = '

0
[CmdletBinding(HelpUri='http://go.microsoft.com/fwlin...'.

254
>>>> $proxy =   $proxy.Replace("COMMAND_TYPE", $commandType)

0
! SET $proxy = '

0
[CmdletBinding(HelpUri='http://go.microsoft.com/fwlin...'.

257
>>>>   $newParamBlockCode = ""

0
! SET $newParamBlockCode = ''.

261
>>>> $beginAdditions   = ""

0
! SET $beginAdditions = ''.

264
>>>>   $currentParameter = $originalCommand.Parameters.Count

0
! SET $currentParameter = '13'.

265
if( >>>>   $AddParameter)

326
>>>> $parameters =   $proxyCommandType::GetParamBlock($metaData)

0
! SET $parameters = '

0
[switch]

0
${Transcript},

0
0

0
[Paramet...'.

327
if( >>>>   $newParamBlockCode) { $parameters

328
>>>> $proxy =   $proxy.Replace('PARAMETERS', $parameters)

0
! SET $proxy = '

0
[CmdletBinding(HelpUri='http://go.microsoft.com/fwlin...'.

331
>>>> $proxy =   $proxy.Replace('CUSTOM_BEGIN', $Begin)

0
! SET $proxy = '

0
[CmdletBinding(HelpUri='http://go.microsoft.com/fwlin...'.

332
>>>> $proxy =   $proxy.Replace('CUSTOM_PARAMETER_PROCESSING', $beginAdditions)

0
! SET $proxy = '

0
[CmdletBinding(HelpUri='http://go.microsoft.com/fwlin...'.

333
>>>> $proxy =   $proxy.Replace('CUSTOM_PROCESS', $Process)

0
! SET $proxy = '

0
[CmdletBinding(HelpUri='http://go.microsoft.com/fwlin...'.

334
>>>> $proxy =   $proxy.Replace('CUSTOM_END', $End)

0
! SET $proxy = '

0
[CmdletBinding(HelpUri='http://go.microsoft.com/fwlin...'.

337
>>>> Write-Verbose   $proxy

338
>>>> Set-Content   function:\GLOBAL:$NAME $proxy

342
if( >>>> $commandType   -eq "Cmdlet")

344
>>>>   $originalCommand.Visibility = "Private"

346
>>>> }

61
>>>>   Export-ModuleMember

Seems to be PSColor which is no longer a dependency, I now get import times which are a lot faster on my latest branch. Let's see if that fixes everything.

Can you validate if 2.0.162 resolves the issue?

Closing due to no feedback

Sorry for the delay, its been hectic to be sure. Just to be clear, while PSColor is no longer a dependency, it is still required if we want colorization of ls output aka Get-ChildItem correct? I didn't see any replacement code.

Yes. That's correct. I have my own fork, not sure if I should keep the defaults in there. Although it doesn't break if you don't have it.


ValueError: could not broadcast input array from shape

import os

import numpy as np
from ffcv.fields import NDArrayField
from ffcv.writer import DatasetWriter

class LinearRegressionDataset:
    def __init__(self, N, d):
        self.X = np.random.randn(N, d)
        self.Y = np.random.randn(N)

    def __getitem__(self, idx):
        # FIXME: returning a tuple is forced here
        return self.X[idx].astype('float32')

    def __len__(self):
        return len(self.X)

def run():
    cwd = os.getcwd()
    dataset_path = os.path.join(cwd, "random")

    # https://docs.ffcv.io/writing_datasets.html#writing-a-dataset-to-ffcv-format
    N, d = 100, 6
    dataset = LinearRegressionDataset(N, d)
    fields = {"covariate": NDArrayField(shape=(d,), dtype=np.dtype("float32"))}
    writer = DatasetWriter(dataset_path, fields)
    writer.from_indexed_dataset(dataset)

if __name__ == "__main__":
    run()

  File "/home/carlos/miniconda3/envs/ffcv/lib/python3.8/site-packages/ffcv/fields/ndarray.py", line 94, in encode
    data_region[:] = field.reshape(-1).view('<u1')
ValueError: could not broadcast input array from shape (4,) into shape (24,)

The cause is that NDArrayField requires that the output of __getitem__ is a tuple:
-        return self.X[idx].astype('float32')
+        return (self.X[idx].astype('float32'),)

resolves the issue.
Is this a bug or a hard-requirement?

__getitem__ need to return an object (Tuple, ndarray, list...) such that it is indexable and has a length equal to the number of fields specified in the DatasetWriter so that each field can be associated with the corresponding data. Here you have 1 field so, the length should be 1 but
self.X[idx].astype('float32')

has a length of d so it won't work as the first dimension (d) will be interpreted as the field dimension.


Not composable: maybe([None])[0].is_none()

Kubuntu 18.04 x64
Anaconda 3.6, pip install pymaybe
maybe([None])[0].is_none() is True.
maybe([None])[0].get() raises Exception, despite [None][0] is None.
maybe({'a': None})['a'].get() raises Exception, despite {'a': None}['a'] is None.

https://www.lucidchart.com/techblog/2015/08/31/the-worst-mistake-of-computer-science/
Despite being a maybe "implementation", this library fails to distinguish a missing element from a deliberate None value.

Just change this to return Nothing()
https://github.com/ekampf/pymaybe/blob/f755600f5478b6b595e2100417b39a7bc3de2c6f/pymaybe/__init__.py#L23-L24
Seems like you're supposed to use the or_else() method instead of get(), but then why provide a get() method anyway?

There is a similar issue also with obj.member = None; maybe(obj).member.get().
I think the root cause is maybe(None).get() raises.
To me it's more logical to have maybe(None).get() == None.
This can be done if we update the code to have maybe(None) == Something(None).


Make a Node.js library

[ ] API Manager, include for uploading data
[ ] TOA Models
[ ] TOA Types

Use the GitHub Package Registry.

Done thanks to @CrispyBacon1999.
https://www.npmjs.com/package/@the-orange-alliance/api
https://github.com/orange-alliance/the-orange-alliance-nodejs
[Still in beta]


CORE-17383 Add new token selection worker

Adds a new worker to host the Token Selection Processor.

LGTM - will approve once Jenkinsfiles are reverted

@jacob-scott3141
We need to merge this as-is, then merge the pipeline changes, then revert the jenkins changes back to 5.1 otherwise we will break the CI builds.

Looks broadly fine.
We probably need corda-api PR as well to introduce this Worker. At least to specify which topics this worker will be consuming from and producing to.

We can do that after this goes in


chore: update Admin Gui and infra

 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.


Live UI

Implement UI for live events

Done by adding <video-live-sign> and <video-timeline disabled> state


Merge history from zoneinfo-parse before fork

Merge the history from zoneinfo-parse from before this crate forked in 2017.
The merge was done with git merge d533d0a64c51507aa5f7c3b0b4d19ae5b0568ca3 --allow-unrelated-histories, where I made sure all files remained unchanged.

Closing in favor of #14.


Enable google/cloud/asset

We cannot compile google/cloud/asset with protobuf <= 3.19.3 (current as-of 2022-01-28) because the protos define TRUE and FALSE as enums, while they are macros on macOS:
In file included from bazel-out/darwin-fastbuild/bin/external/com_google_googleapis/google/cloud/asset/v1/asset_service.pb.h:40:
bazel-out/darwin-fastbuild/bin/external/com_google_googleapis/google/cloud/asset/v1/assets.pb.h:4015:36: error: expected member name or ';' after declaration specifiers
  static constexpr EvaluationValue TRUE =
  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ^
/Applications/Xcode_12.0.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.15.sdk/usr/include/mach/boolean.h:81:17: note: expanded from macro 'TRUE'
#define TRUE    1
                ^

Windows bazel builds are still blocked. E.g. https://source.cloud.google.com/results/invocations/90512c0b-7383-4221-a311-b49200c0e635
They fail on the TRUE, FALSE definitions. port_def.inc inside google3 handles these (internal link), but the open source doesn't (link). I did not look into why these are out of sync, despite the internal changes being made in 2021-04

Need to wait until Protobuf v23 and the corresponding changes in gRPC.

Only cleaning up some CI builds remains.  We need to wait until a release of google-cloud-cpp, and a release of vcpkg that includes Protobuf 23 and our library.

This is now fixed.


improve error message for unsupported EDITOR

track something to improve error messaging for unsupported $EDITORs? nvim isn't supported right now, and it isn't clear at all that it doesn't work

Hmmmm
Try running git peek nextcloud/ios -e nvim
Does nvim open directly in the terminal for you or does it open in a separate app? Right now, the code assumes any vim will open in the terminal

There's a difficult to reproduce issue where the $EDITOR var is not propagating to git-peek's node.js instance and I'm not sure why.

nvim is now fixed (as of v1.3.17) and fixed a related bug with detecting the editor


'lib/moduleCache' added

Coverage increased (+0.05%) to 37.585% when pulling cb11bf2c8e3eb19a9938bce5526240d127c62a58 on module_cache into 45c76ad6e441aaa729f44fa17a3c40167ace7f5e on master.


New version: Sohu.SHPlayer version <IP_ADDRESS>

Created by 🥟 Dumplings in workflow run #6969.

Log
Updated: <IP_ADDRESS> → <IP_ADDRESS>
Submitting WinGet manifests
Creating manifests
Uploading manifests and making commits
Creating a pull request

Microsoft Reviewers: Open in CodeFlow

/azp run


remove duplicate changelog entries

As title
Reference an existing issue
#7172

I discovered these by carefully looking at the git history, it would probably be good if someone else were to verify them.

Looks like we missed this one for RC2 -- must go in for RC3.


Fred work2

Navigation bar created (still needs some touch ups), Intro page: the content I want on it is there I still have to fix the layout. Readme contain modified.

Let me know once you finish working and I can approve the PR.

Hi Chen, for week 1 I think that is all I have for the moment the other changes that I will push will be for week 2. Later if there is time I will come back to weak 1 and make more improvements.

Forgot to say I will also be changing the the links for so that it opens a new page when clicked on but I can commit those changes when I start commit the week2 changes that I have. But you can go ahead and approve this PR, thank you for all your help.


Help with custom type

Hey I have a postgres postgis db with column type geometry and thought the docs here would be the best place. I'm unsure what's the best way to use a postgres function to set the value
convertToDatabaseValue(value: any, platform: Platform) {
        console.log(value)
        if (value instanceof Geometry) {

            return `ST_SetSRID(ST_MakePoint(${value.lng}, ${value.lat}),4326)`;

        }
        throw ValidationError.invalidType(GeometryType, value, 'JS');
}

Best would be to parameterize the values, figured accessing knex would be best option? I'm not sure about the integration with orm to achieve this. Any help is appreciated

Hmm maybe you could use knex.raw?
return knex.raw(`ST_SetSRID(ST_MakePoint(?, ?),4326)`, [value.lng, value.lat]);

I have never tried knex-postgis but maybe you could use it.

@ryankauk any update on this?

Closing for inactivity, let me know if it helped, will be happy to reopen if not.

hey @B4nan sorry for the late reply, didn't see your reply and was busy refactor my server to your orm. Really love the project so far. My favourite long needed orm for node.js.
I just tried out your solution here's a little more context
const location = new Location({
    ...input,
    lat: point.lat,
    lng: point.lng,
    geom: new Geometry(point.lng, point.lat),
});
await db.em.persistAndFlush(location);

convertToDatabaseValue(value: any, platform: Platform) {

    if (value instanceof Geometry) {
        return knex.raw(`ST_SetSRID(ST_MakePoint(?, ?),4326)`, [value.lng, value.lat]);
    } else if (value === null) {
        return value;
    }

    throw ValidationError.invalidType(GeometryType, value, 'JS');
}

However this ends up throwing an error "TypeError: Converting circular structure to JSON" from here:
https://github.com/mikro-orm/mikro-orm/blob/e3e02ea566703d3bfa36429095a92d96f0c01a22/lib/query/QueryBuilderHelper.ts#L75-L77
as data[k] ends up evaluating as a knex Raw object in this situation to:
Raw {
  client: Client_PG {
    config: { connection: undefined, client: 'pg' },
    logger: Logger {
      _inspectionDepth: 5,
      _enableColors: true,
      _debug: undefined,
      _warn: undefined,
      _error: undefined,
      _deprecate: undefined
    },
    connectionSettings: {},
    connectionConfigExpirationChecker: null,
    valueForUndefined: Raw {
      client: [Circular],
      sql: 'DEFAULT',
      bindings: undefined,
      _wrappedBefore: undefined,
      _wrappedAfter: undefined,
      _debug: undefined
    },
    makeKnex: [Function: makeKnex],
    _events: [Object: null prototype] {
      start: [Function],
      query: [Function],
      'query-error': [Function],
      'query-response': [Function]
    },
    _eventsCount: 4
  },
  sql: 'ST_SetSRID(ST_MakePoint(?, ?),4326)',
  bindings: [ -50, 50 ],
  _wrappedBefore: undefined,
  _wrappedAfter: undefined,
  _debug: undefined
}

Maybe adding an extra check for knex.Raw should be sufficient?

I guess we could make it even better, we know that the property is custom type, therefore we do not need any additional processing of the value (if so, that is what should be part of the convertToDatabaseValue method).
Please try to adjust the condition in mikro-orm/lib/query/QueryBuilderHelper.ts to see if that woks:
if (Array.isArray(data[k]) || (Utils.isObject(data[k], [Date]) && !prop.customType)) {
  data[k] = JSON.stringify(data[k]);
}

If so, will be happy to accept PR :]

Just tried this out, there seems to be a similar issue in "processParams" in query/SmartQueryHelper.ts. However it seems here there's no access to the props

Keep me posted about any updates because I'm going to try PostGIS' Geography Type in the next couple of days.

@darkbasic mikro-orm still works with postgis, it exposes knex so you can use knex-postgis, there's just some of the convenience factors that there's an issue with. Right now I just create an object with knex itself along with querying.

I guess you are talking about this part? What if you comment it out?
https://github.com/mikro-orm/mikro-orm/blob/master/lib/query/SmartQueryHelper.ts#L27
I guess we could also check the constructor name of the object to see if its a POJO or not (if not, we can ignore it there)

@ryankauk it seems that knex-postgis doesn't support ST_GeographyFromText
@B4nan commenting it out leads to the following:
(node:58381) UnhandledPromiseRejectionWarning: TypeError: Converting circular structure to JSON
    --> starting at object with constructor 'Client_PG'
    |     property 'valueForUndefined' -> object with constructor 'Raw'
    --- property 'client' closes the circle
    at JSON.stringify (<anonymous>)
    at escapeObject (/home/niko/WebstormProjects/beach/mikro-orm/node_modules/knex/lib/dialects/postgres/index.js:86:19)
    at escapeFn (/home/niko/WebstormProjects/beach/mikro-orm/node_modules/knex/lib/query/string.js:47:18)
    at Client_PG.finalEscape [as _escapeBinding] (/home/niko/WebstormProjects/beach/mikro-orm/node_modules/knex/lib/query/string.js:18:12)
    at /home/niko/WebstormProjects/beach/mikro-orm/node_modules/knex/lib/client.js:145:19
    at String.replace (<anonymous>)
    at Client_PG._formatQuery (/home/niko/WebstormProjects/beach/mikro-orm/node_modules/knex/lib/client.js:137:16)
    at /home/niko/WebstormProjects/beach/mikro-orm/node_modules/knex/lib/interface.js:10:26
    at arrayMap (/home/niko/WebstormProjects/beach/mikro-orm/node_modules/lodash/lodash.js:639:23)
    at map (/home/niko/WebstormProjects/beach/mikro-orm/node_modules/lodash/lodash.js:9554:14)
    at Builder.Target.toQuery (/home/niko/WebstormProjects/beach/mikro-orm/node_modules/knex/lib/interface.js:9:12)
    at Builder.toString (/home/niko/WebstormProjects/beach/mikro-orm/node_modules/knex/lib/query/builder.js:72:17)
    at PostgreSqlConnection.getSql (/home/niko/WebstormProjects/beach/mikro-orm/dist/connections/AbstractSqlConnection.js:71:23)
    at PostgreSqlConnection.executeKnex (/home/niko/WebstormProjects/beach/mikro-orm/dist/connections/AbstractSqlConnection.js:63:26)
    at PostgreSqlConnection.execute (/home/niko/WebstormProjects/beach/mikro-orm/dist/connections/AbstractSqlConnection.js:31:31)
    at QueryBuilder.execute (/home/niko/WebstormProjects/beach/mikro-orm/dist/query/QueryBuilder.js:185:59)
    at PostgreSqlDriver.nativeInsert (/home/niko/WebstormProjects/beach/mikro-orm/dist/drivers/AbstractSqlDriver.js:61:43)
    at ChangeSetPersister.persistEntity (/home/niko/WebstormProjects/beach/mikro-orm/dist/unit-of-work/ChangeSetPersister.js:32:37)
    at ChangeSetPersister.persistToDatabase (/home/niko/WebstormProjects/beach/mikro-orm/dist/unit-of-work/ChangeSetPersister.js:20:20)
    at UnitOfWork.commitChangeSet (/home/niko/WebstormProjects/beach/mikro-orm/dist/unit-of-work/UnitOfWork.js:208:39)
    at processTicksAndRejections (internal/process/task_queues.js:97:5)
    at UnitOfWork.persistToDatabase (/home/niko/WebstormProjects/beach/mikro-orm/dist/unit-of-work/UnitOfWork.js:342:13)
(node:58381) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 2)
(node:58381) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Also, even if I was ok without the knex interpolation there is the following problem if I simply return a string:
insert into "location" ("city", "created_at", "geog", "id", "latitude", "longitude", "updated_at") values ('Osimo', '2020-03-25 09:29:51.452', 'ST_SetSRID(ST_MakePoint(13.478303, 43.489405),4326)', 8, 43.489405, 13.478303, '2020-03-25 09:29:51.481') returning "id"

Before the ST_SetSRID function it adds a ' which breaks the query.