Visual Studio Code Windows command needs quotes around the '-outDir "Pages\Movies"' tag

Following along with the RazorPages tutorial and running the codegenerator, I got stuck for a bit after running this command:
dotnet aspnet-codegenerator razorpage -m Movie -dc RazorPagesMovieContext -udl -outDir Pages\Movies --referenceScriptLibraries
The next step was to run the app and check the localhost:port/movies page, and I got not found error. The problem was solved when I put quotes around the outDir tag like so:
dotnet aspnet-codegenerator razorpage -m Movie -dc RazorPagesMovieContext -udl -outDir "Pages\Movies" --referenceScriptLibraries

@DutchBoyd ... For a faster response, close this issue and re-open it using the This Page feedback button+form at the bottom of the English topic. It will add metadata to cross-link the topic and automatically ping the author.

@DutchBoyd  not sure what you did wrong but that doesn't need to be quoted.


Added two new examples for a garage door and a door sensor.

I used a NodeMCU board to develop a garage opener - started out with a door sensor (also provided) - as a new example.
I hope, this is useful to someone.

Currently in the Makefile. I found it to be appropriate as it is a kind of Projekt configuration.
But it can certainly also be done in code. This should be documented, though.

I found that both the the REED_PIN and RELAY_PIN are high by default. I connected an LED to GPIO 2 and it only goes low when the request to open or close comes from the homekit app. Was this the intended functionality?
I connected a reed switch from the relay pin to ground and found that the system shows the door is open when the magnet is in contact with the switch and indicates the door is closed when the magnet is not in contact. The switch I’m using is normally open when the magnet is disengaged. Is there an easy way to reverse these states?
Thanks.

@g3ntleman I just tried to run your example and I didn't work for me at first (contact_sensor_intr_callback wasn't called). After investigation I found that in my case I had to also enable GPIO gpio_enable(sensor->gpio_num, GPIO_INPUT); before pullup set https://github.com/maximkulkin/esp-homekit-demo/blob/master/examples/door-sensor/contact_sensor.c#L53 . I'm not sure why in your case it worked. I use a reed switch like this https://belchip.by/sitedocs/31153.pdf with Sonoff SV (ESP8266)


How to Make VS Code look like the featured image

I was switching between all my theme options but I still can't get my IDE to look like this (the first, biggest theme displayed).

Are there any specific settings I need to set in order to accomplish that look? Thanks in advance.

Hi @SethuSenthil you just need to activate the Ocean theme variant.


Copy Link Option [NOT WORKING]

Description
The Copy Link option which appears after clicking the share option is not working.
Screenshots

Additional information
This is My First Open Source Issue.

Thanks for noticing, this is a duplicate so I’ll close it for now.

Thanks for noticing, this is a duplicate so I’ll close it for now.

I'll see if I can fix this, can you please mention the original issue?

Thanks for noticing, this is a duplicate so I’ll close it for now.

I'll see if I can fix this, can you please mention the original issue?

It's because of the http not secure issue, the copy/paste API only works on https so it's blocked until that is fixed.

Ok, cool
On Mon, 7 Feb 2022 at 17:01, Emma Dawson @.***> wrote:

Thanks for noticing, this is a duplicate so I’ll close it for now.
I'll see if I can fix this, can you please mention the original issue?
It's because of the http not secure issue, the copy/paste API only works
on https so it's blocked until that is fixed.
—
Reply to this email directly, view it on GitHub
https://github.com/EddieHubCommunity/LinkFree/issues/1079#issuecomment-1031365353,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUGJUTPI2HGRWGN2ISS4GGDUZ6UQXANCNFSM5NWVGJMQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you commented.Message ID:
@.***>

Ok , Thanks for letting me know.
On Mon, Feb 7, 2022 at 5:06 PM Kendall Pinto @.***>
wrote:

Ok, cool
On Mon, 7 Feb 2022 at 17:01, Emma Dawson @.***> wrote:

Thanks for noticing, this is a duplicate so I’ll close it for now.
I'll see if I can fix this, can you please mention the original issue?
It's because of the http not secure issue, the copy/paste API only works
on https so it's blocked until that is fixed.
—
Reply to this email directly, view it on GitHub
<
https://github.com/EddieHubCommunity/LinkFree/issues/1079#issuecomment-1031365353
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AUGJUTPI2HGRWGN2ISS4GGDUZ6UQXANCNFSM5NWVGJMQ
.
Triage notifications on the go with GitHub Mobile for iOS
<
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
<
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub
.
You are receiving this because you commented.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
https://github.com/EddieHubCommunity/LinkFree/issues/1079#issuecomment-1031369687,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVSMLDOFPEFSAXKGWCTHSBLUZ6VDLANCNFSM5NWVGJMQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


Initrd setup root

LGTM

The setup-root script assumes it is being sourced into the dracut shell environment, simply running it stand-alone isn't going to work without modifications to the script.

LGTM


Editor crash when closed

After upgrading from 2.0.1 to 2.0.2 on godot 4.2, the editor crashes when it is closed.
Godot(48212,0x7ff860710640) malloc: *** error for object 0x7feaf700d588: pointer being freed was not allocated
Godot(48212,0x7ff860710640) malloc: *** set a breakpoint in malloc_error_break to debug

I'm using macOS 13.4 on an intel macbook pro

It doesn't matter if it is enabled or if I have an export template using OpenXR. And I've tried it removing the folder and replacing it with the one still in my downloads folder (the original).
It doesn't crash for the first time. First time, it says the editor needs a restart for some extensions to work properly. In this session, exiting Godot is without a problem. After opening it again, now it crashes every time I exit.

I have the same issue on Linux mint 21.3 Cinnamon. When executing Godot from the command line I can see an error, even the first time. Here it is:
ERROR: Condition "!EditorSettings::get_singleton() || !EditorSettings::get_singleton()->has_setting(p_setting)" is true. Returning: Variant()
   at: _EDITOR_GET (editor/editor_settings.cpp:1140)

I hate the crash logs created by this.

The issue still happens on version 2.0.3. The error I mentioned above is no longer here. It was therefore, unrelated. Since #64 is just an update on the automatic way the add-on is built and it would only be possible to cause an issue only if the builds are made with this automated script which would be very surprising.
To finish this comment, another info about the issue: by deleting every vendor directory, it disappears, which sounds logic said like that but if we only leave one vendor the issue persists. So it is not specific to a vendor.

The root cause for this issue has been addressed in the Godot core codebase. The fix will roll-out with the Godot 4.3 and Godot 4.2.2 release.

I tested if the "fix" works on both Godot 4.2.2rc2 and 4.3dev5 by installing 2.0.3 in brand new projects and closing the editor. The editor crashed and created a crash log on both versions.
Am I missing something?
If not, then I want to bring back the fact that 2.0.1 works but not 2.0.2 (as I remember from testing on Godot 4.2.1), I'm assuming this means that the faulty code isn't in the engine.

I tested if the "fix" works on both Godot 4.2.2rc2 and 4.3dev5 by installing 2.0.3 in brand new projects and closing the editor. The editor crashed and created a crash log on both versions.
Am I missing something?
If not, then I want to bring back the fact that 2.0.1 works but not 2.0.2 (as I remember from testing on Godot 4.2.1), I'm assuming this means that the faulty code isn't in the engine.

@nontoxicguy I've not been able to reproduce the issue on the master version of this plugin. Before the fix, trying to reload the project would cause the crash preventing the project from re-opening. With the latest dev version of Godot, I'm able to reload a project with this plugin and it re-opens as expected.
Can you test with the latest dev versions of the plugin and see if you can repro the crash. You can download a build from https://github.com/GodotVR/godot_openxr_vendors/actions/runs/8320916845

I've not been able to reproduce the issue on the master version of this plugin.

Same for me, so this issue is fixed.
I did more tests and my results are that whether the bug is here doesn't depend on Godot's version, it only depends on this plugin's version.
So that pull request you mentioned didn't fix the issue? I'm very unqualified to answer the question, but according to my tests the issue was in this plugin (or godot-cpp which is a dependency).
Anyways, I have no reason to talk about this anymore.

@nontoxicguy Turns out your fix in https://github.com/GodotVR/godot_openxr_vendors/pull/80 was also needed to fix the crash.
For the issue to be resolved, we needed the fix in Godot 4.3 and your fix in https://github.com/GodotVR/godot_openxr_vendors/pull/80. We're cherry-picking that fix for version 2.0.4 of the plugin.


Adds VisSense.js to add_my_project_to_inch_ci.yml

Adds VisSense.js to add_my_project_to_inch_ci.yml. It'd be awesome to test your service.

I sent you a PR!

.. and I happily accepted it! Thank you :)


Fix MusicBrainz.Picard version 2.11.0

Resolves #136673
Related to #136352

Microsoft Reviewers: Open in CodeFlow

Thanks for this fix. I need to take closer attention to the results of "wingetcreate update" it seems.
I had noticed it had moved the "AppsAndFeaturesEntries" section up one level, but thought this might be intentional if there is only one installer.
Is this something that should be reported for winget-create?

Thanks for this fix. I need to take closer attention to the results of "wingetcreate update" it seems.
I had noticed it had moved the "AppsAndFeaturesEntries" section up one level, but thought this might be intentional if there is only one installer.
Is this something that should be reported for winget-create?

Moving it up one level shouldn’t be an issue; In fact, I missed that it was there.  The primary issue is that DisplayVersion was missing


After upgrading from 2.0.5 to 2.1.3 the deprecation system of “whitelist” -> “allow” does not work

After upgrading from 2.0.5 to 2.1.3 I’m seeing this behaviour when testing the deprecation system of “whitelist” -> “allow”:
In 2.0.5 this works fine (-f means application/x-www-form-urlencoded):
[kong-2.0.5:/kong]# http :8001/services/ServiceA/plugins name=ip-restriction config.whitelist=<IP_ADDRESS> -f
HTTP/1.1 201 Created
...
...

After I upgraded to 2.1.3 I tried using the same HTTPie command:
[kong-2.1.3:/kong]# http :8001/services/ServiceB/plugins name=ip-restriction config.whitelist=<IP_ADDRESS> -f
HTTP/1.1 400 Bad Request
Access-Control-Allow-Origin: *
Connection: keep-alive
Content-Length: 149
Content-Type: application/json; charset=utf-8
Date: Fri, 21 Aug 2020 22:00:30 GMT
Server: kong/2.1.3
X-Kong-Admin-Latency: 3

{
    "code": 2,
    "fields": {
        "config": {
            "allow": "expected an array"
        }
    },
    "message": "schema violation (config.allow: expected an array)",
    "name": "schema violation"
}

With config.allow instead of config.whitelist it works, though:
[kong-2.1.3:/kong]# http :8001/services/ServiceB/plugins name=ip-restriction config.allow=<IP_ADDRESS> -f
HTTP/1.1 201 Created
...
...

The problem is that shorthands are not inferred thus all the type etc. conversion needs to happen in  shorthand function itself. This same applies for bot-detection and acl.

@murillopaula With JSON requests it works and the issue appears only for form-encoded, right? (i.e. using http with -f)
If so, can we fix the bug title?

Plugins
⚠ Change authentication plugins to standardize on allow and deny as terms for access control. Previous nomenclature is deprecated and support will be removed in Kong 3.0.
ACL: use allow and deny instead of whitelist and blacklist
bot-detection: use allow and deny instead of whitelist and blacklist
ip-restriction: use allow and deny instead of whitelist and blacklist #6014
plugins\ip-restriction\schema.lua
	return {
	  name = "ip-restriction",
	  fields = {
		{ protocols = typedefs.protocols_http },
		{ config = {
			type = "record",
			fields = {
			  { allow = { type = "array", elements = typedefs.ip_or_cidr, }, },
			  { deny = { type = "array", elements = typedefs.ip_or_cidr, }, },
			},
			shorthands = {
			  **-- deprecated forms, to be removed in Kong 3.0**
			  { blacklist = function(value)
				  return { deny = value }
				end },
			  { whitelist = function(value)
				  return { allow = value }
				end },
			},
		  },
		},
	  },
	  entity_checks = {
		{ at_least_one_of = { "config.allow", "config.deny" }, },
	  },
	}

@hishamhm it fails with JSON too:
[kong-master:/root]# http :8001/plugins < ip-restriction.json
HTTP/1.1 400 Bad Request
Access-Control-Allow-Origin: *
Connection: keep-alive
Content-Length: 149
Content-Type: application/json; charset=utf-8
Date: Thu, 17 Sep 2020 14:52:04 GMT
Server: kong/2.1.3
X-Kong-Admin-Latency: 1

{
    "code": 2,
    "fields": {
        "config": {
            "allow": "expected an array"
        }
    },
    "message": "schema violation (config.allow: expected an array)",
    "name": "schema violation"
}

Closed by https://github.com/Kong/kong/pull/6364


This library vs unplugin-macros

Hi I am trying to understand how two libraries compare? Just to better understand usecases for both.
Here is the link to unplugin-macros
Thank you

I found this. Is it accurate for these two libraries as well?
https://parceljs.org/features/macros/#:~:text=Differences from Bun&text=Parcel supports returning functions from,Response objects%2C or Blob objects.

Yes those are all true. Returning functions and generating assets are the main things. However, unplugin-macros has very limited evaluation support as well. I believe the arguments and return value must be literals (eg strings and numbers), not objects/arrays, and there is no support for evaluating constants either.


ci: add prettify code ci workflow

Pull Request Template
Description
Please include a summary of the change. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (keploy/keploy#407)
Type of change
Please delete options that are not relevant.

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

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Additional Context (Please include any Screenshots/gifs if relevant)
...
Checklist:

[ ] My code follows the style guidelines of this project.
[ ] I have performed a self-review of my own code.
[ ] I have made corresponding changes to the documentation.
[ ] I have added corresponding tests.
[ ] Any dependent changes have been merged and published in downstream modules.

Hey @nehagup, can you please review my PR ?


[Suggestion] Toggle all Extractor's redstone control

Ability to toggle fluid and energy extractors to not require a redstone signal, in line with the item one. This would be especially useful when the block being extracted from is sensitive to redstone itself.

Yep this is a great addition.   Adding a screen with the standard redstone button and I will default them to Always On

LOL so this is out.  I was actually already working on this before you posted so your next suggestion probably wont be as quick 👍
https://minecraft.curseforge.com/projects/cyclic/files/2533820


Suggested fix for issue 1067: add 'nonzero_occurrences' to solve igno…

…red Resolves for status history like: 1,1,1,2,0. https://github.com/sensu/sensu/issues/1067
Requires a PR for sensu as well (same title)

Wrong answer, closing.  Please ignore this.


chore: release main

:robot: I have created a release beep boop
stdlib: 1.1.8
1.1.8 (2024-01-05)
Bug Fixes

update versions (6035bcc)

fiber: 1.2.3
1.2.3 (2024-01-05)
Bug Fixes

update versions (6035bcc)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/pavophilip/react-shader-fiber/releases/tag/stdlib-v1.1.8 :sunflower:

:robot: Release is at https://github.com/pavophilip/react-shader-fiber/releases/tag/fiber-v1.2.3 :sunflower:


Add destroy image callback

[x] Allow to define "ion-item-destroy-callback" to destroy images.
[x] Add button over each photo if  photo.destroy_enabled == true
[x] Launch destroy callback on click button
[x] Update readme

@pedroabreu maybe this feature is interesting to merge in your project.
I'm newbie on ionic framework and i am not sure if i'm missing something important. I didn't see any tests to complete.
Feel free to suggest anything.
Thank you for this repo anyway, it saved my day.


ToCode Solution: תרגול רשימות

01: Instead of just range(ord('a'),123) it would be probably clearer if you wrote range(ord('a'), ord('z')+1)
02: words, not numbers 😄  but you can also write much simpler code using on one variable for c1.
03: you don't need 3 lists (chr_lst1, chr_lst2, chr_lst3) after all they have the same content.

Thank you!
Fixed

👍


Plot a Secondary Target Range

Plot both the control range (as determined from pump settings) and a larger target range (user-configurable) with different opacities. For me, that would be 90 - 120 and 80 - 180. Would give a better visual indication that I'm still doing well even if I'm not exactly in the 90 - 120.

This was started with hard-coded values in #38. We'll leave the ticket open to make these configurable.


Project structure

Structure the project based on endpoint and add endpoint for FCE.

Resolved by 9569fcb and the commits following.


Book: fix bad usage of MD

When I was looking through the book related, I noticed this unusual way of including. I looked up the documentation for Markdown and found out that it doesn't support directly including complex content management features. This kind of syntax might be used in tools like GitBook, so there might be a mistake in how it's being used here.

Yeah, this is used in mdBook, and having the summary markdown in the CLI reference page is kind of nice. It's included in this way because the content is generated by a script. This is because we need to make sure the book is up to date when we add new commands. I'm going to close this, since I would prefer to use mdBook-specific features if they're available. Thank you for the effort though!
The include statement: https://rust-lang.github.io/mdBook/format/mdbook.html#including-portions-of-a-file


⚠ Bofrost.lu has degraded performance

In 2d93daf, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1190 ms

Resolved: Bofrost.lu performance has improved in 186d8cc.


Create Article “puritys-territory”

Automatically generated by Netlify CMS

👷 Deploy Preview for 4shame processing.

Name
Link

🔨 Latest commit
3d06d31e59ff3ba70f5a92459c401bfec2b57d50

🔍 Latest deploy log
https://app.netlify.com/sites/4shame/deploys/633b0f0b2e9269000a2a782b


Updated to version 20161009

If there’s a checkbox you can’t complete for any reason, that's okay, just explain in detail why you weren’t able to do so.
After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[ ] brew cask style --fix {{cask_file}} reports no offenses.
[x] The commit message includes the cask’s name and version.

Additionally, if adding a new cask:

[ ] Named the cask according to the token reference.
[ ] brew cask install {{cask_file}} worked successfully.
[ ] brew cask uninstall {{cask_file}} worked successfully.
[ ] Checked there are no open pull requests for the same cask.
[ ] Checked that the cask was not already refused in closed issues.

This is the latest version of Mactex
For brew cask style fix I get an error "undefined method `join' for nil:NilClass".
Might be my setup.

Pretty sure the test failure is just the download timing out. The file is 2.7GB.


BUGFIX shadowoftheweirdwizard.html

Fixed loss of data in Coin field added value=""

Submission Checklist
Required

[x ] The pull request title clearly contains the name of the sheet I am editing.
[x ] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[x ] The pull request makes changes to files in only one sub-folder.
[ x] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

New Sheet Details

The name of this game is: <   >  (i.e. Dungeons & Dragons 5th Edition, The Dresden Files RPG)

The publisher of this game is: <   > (i.e. Wizards of the Coast, Evil Hat)

The name of this game system/family is: <   > (i.e. Dungeons & Dragons, FATE)

[ ] I have followed the Character Sheets Standards when building this sheet.

[ ] I have authorization from the game's publisher to make this an official sheet on Roll20 with their name attached.
[ ] This game is not a traditionally published game, but a copy of the game rules can be purchased/downloaded/found at: <   >
[ ] This sheet is for an unofficial fan game, modification to an existing game, or a homebrew system.

Changes / Description

Fixes bug where players data in Coins is lost between sessions

Character Sheet Info Roll20 Internal Use only.


feat: re-implemented actor and reactor packages, and updated denoifier

The @rimbu/actor and @rimbu/reactor have been completely rewritten to be similar to Redux ToolKit. Also, denoify needed to be updated to support npm package imports
BREAKING CHANGE: The @rimbu/actor and @rimbu/reactor packages have a completely new API (but they were and are still experimental)
Fixes #126

Codecov Report
Base: 93.50% // Head: 93.52% // Increases project coverage by +0.01% :tada:

Coverage data is based on head (ad43faf) compared to base (3622270).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #127      +/-   ##
==========================================
+ Coverage   93.50%   93.52%   +0.01%
==========================================
  Files         216      219       +3
  Lines       14215    14191      -24
  Branches     2883     2894      +11
==========================================
- Hits        13292    13272      -20
+ Misses        837      833       -4
  Partials       86       86

Impacted Files
Coverage Δ

packages/actor/src/main/index.ts
100.00% <ø> (ø)

packages/actor/src/immer/index.ts
100.00% <100.00%> (ø)

packages/actor/src/immer/slice-immer.ts
100.00% <100.00%> (ø)

packages/actor/src/main/action.ts
100.00% <100.00%> (ø)

packages/actor/src/main/actor.ts
100.00% <100.00%> (ø)

packages/actor/src/main/generate-uuid.ts
100.00% <100.00%> (ø)

packages/actor/src/main/internal.ts
100.00% <100.00%> (ø)

packages/actor/src/main/lookup.ts
100.00% <100.00%> (ø)

packages/actor/src/main/slice-config.ts
100.00% <100.00%> (ø)

packages/actor/src/main/slice.ts
100.00% <100.00%> (ø)

... and 10 more

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

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


ZipMap: ([x], x → y) → [(x, y)]

This was initially proposed by @Orace as part of the discussion in #624 but I am adding it here with the proposed name of ZipMap:
public static IEnumerable<(TSource Item, TResult Result)>
    ZipMap<TSource, TResult>(this IEnumerable<TSource> source, Func<TSource, TResult> func)

Element names are up for discussion.

Can't this be a composition of functions like IEnumerable<(A Left, A Right)> SelectPairs(IEnumerable<A> source) and IEnumerable<(A Left, C RIght)> MapRight<A, B, C>(IEnumerable<(A Left, B Right)> source, Func<B, C> f) which may become a good generic addition to MoreLinq?

@declard Interesting idea! I like the generalization.

Interesting idea, but I'm not sure it adds value. Is there a use-case for SelectPairs() outside of ZipMap; that is, is there a reason someone would want a single value as both parts of a tuple other than for use with a MapRight method?
Similarly, while I can see potential use-cases, I'm not sure that MapRight() (or an expected matching MapLeft()) is something that would be used often. Even when used in queries, ValueTuples are generally used wholly, not partially, in my experience. As such, it looks like a solution in search of a problem.
ZipMap is a fairly simple function in the first place, basically .Select(x => (x, func(x))), but eliminating the closure. I'm not sure that further generalization would be helpful.

Agree, ValueTuple<,> is not the best candidate. We have SelectPairs, MapKeys, and MapValues working with KVP<,> on a project, which allow things like
collection
    .SelectPairs().MapValues(SomeFuncReturningOptionals).CollectSomeOptValues().MapValues(AnotherFunc).ToDictionary()`, very convenient.

If closure is in concern, then why not introducing something generic like
```csharp
IEnumerable<TResult> Select<TElement, TPayload, TResult>(
    this IEnumerable<TElement> collection,
    TPayload payload,
    Func<TElement, TPayload, TResult> func)
{
    using var iterator = collection.GetEnumerator();

    while (iterator.MoveNext())
    {
        yield return func(iterator.Current, payload);
    }
}

?
collection.Select(func, (x, func) => (x, func(x))) doesn't look as nice as collection.ZipMap(func), unless the (x, func) => (x, func(x)) part is abstracted into a function on its own, but it's still strictly more powerful and has exactly same memory footprint

As such, it looks like a solution in search of a problem.

What would that problem be?

I don't know; that's why I made that statement. The solution given by MapRight() is to execute a projection on part of a tuple while keeping the other element of the tuple as-is. My statement is that I don't know what problem this solves, so it is a solution in search of a problem.
However, @declard has offered some code where such a method may be useful. If one is keeping a source value as well as the net transformation of that source, then such a method would allow a simpler projection.
@declard My concern was never with the closure, ValueTuple, or with ZipMap. In fact, I have ZipMap implemented in elsewhere - it is definitely cleaner to have source.ZipMap(func) than source.Select(x => (x, func(x))). My concern was with the additional level of abstraction suggested via SelectPair() and MapRight(). I would still argue, even in your example, that SelectPair specifically is not an inherently useful method. It would be better to use ZipMap for the first projection, and then MapRight for any future projections.
I would still want to argue the cost-benefit of a method like MapRight, since it is a proxy for source.Select(x => (x.Left, Right: func(x.Right))); though if ZipMap passes the bar for usefulness (which it does in my book), then I would be hard-pressed to counter-argue that MapRight does not pass the bar.

By the way, collection.ZipMap(func) on KVPs is expressed as collection.SelectPairs().MapValues(func)

Or is it only a useful method because of MapValues()?

well, because of MapKeys too, maybe some other things which take KVPs, like ToDictionary

I have ZipMap implemented in elsewhere - it is definitely cleaner to have source.ZipMap(func) than source.Select(x => (x, func(x))). My concern was with the additional level of abstraction suggested via SelectPair() and MapRight(). I would still argue, even in your example, that SelectPair specifically is not an inherently useful method. It would be better to use ZipMap for the first projection, and then MapRight for any future projections.
I would still want to argue the cost-benefit of a method like MapRight, since it is a proxy for source.Select(x => (x.Left, Right: func(x.Right))); though if ZipMap passes the bar for usefulness (which it does in my book), then I would be hard-pressed to counter-argue that MapRight does not pass the bar.

Thanks for expanding. While I said I like the generalizations, I don't think SelectPairs and MapRight have a place in MoreLINQ. In fact, I could extend that to Match (#624) and ZipMap here too, which is why I haven't jumped at adding those. These are all very trivial implementations that seem to have to do more with lifting of functions into wrapped or inner structures while maintaining the wrapper or outer structure (the sequence) than something algorithmically interesting for sequences. I'm not debating their utility for some people, but just that they probably belong in a separately maintained library, may be one centered around tuples (like I did for KeyValuePair<,> and sequences thereof with KeyValuePairs). MapRight is especially dangerous because it could lead to scope creep (apply a function to an element of N-tuple or apply the N-tuple as arguments to a function, etc.). I had imagined that Match and ZipMap could be interesting for quick and interactive explorations in tools like LINQPad), but I use the query syntax whenever possible and almost always create tuples with meaningful element names for readability (at which point you're back to relying on select/Select), so I've not needed or used them personally, but then again, this is not about just me. I'm happy to close these (considering they've been open for comment for a while) unless someone can demonstrate a strong case, especially for ZipMap that's the topic of this issue.


isort removed my # noqa comment

As you can see below noqa comment has disappeared for the second import entry. How can I avoid this?
(My noqa comment is for flake8)
# cat fields.py

from wtforms_components import (  # noqa
    ColorField,
    DateField,
    DateIntervalField,
    DateTimeField,
    DateTimeIntervalField,
    DecimalField,
    DecimalIntervalField,
    DecimalSliderField,
    EmailField,
    FloatIntervalField,
    GroupedQuerySelectField,
    IntegerField,
    IntegerSliderField,
    IntIntervalField,
    PassiveHiddenField,
    PhoneNumberField,
    SearchField,
    SelectField,
    SelectMultipleField as SelectMultipleField_,
    SplitDateTimeField,
    StringField,
    TimeField,
    WeekDaysField,
)

# isort fields.py && cat fields.py

from wtforms_components import SelectMultipleField as SelectMultipleField_  # noqa
from wtforms_components import (ColorField, DateField, DateIntervalField,
                                DateTimeField, DateTimeIntervalField,
                                DecimalField, DecimalIntervalField,
                                DecimalSliderField, EmailField,
                                FloatIntervalField, GroupedQuerySelectField,
                                IntegerField, IntegerSliderField,
                                IntIntervalField, PassiveHiddenField,
                                PhoneNumberField, SearchField, SelectField,
                                SplitDateTimeField, StringField, TimeField,
                                WeekDaysField)

I'll look into this, thanks for reporting! Sorry for the long wait, been heads down working on hug: https://github.com/timothycrosley/hug
~Timothy

No worries at all. Looks like an interesting project!

Hi @jmagnusson I think the best way to avoid this is to add isort:skip to that single import to have handled special for isort (just as it is for flake8) it's very difficult for isort to know if a comment was meant for a group or single import, and even harder for it to keep track of where to place it. It took quite a while for isort to attempt to handle comments at all (originally it would strip them all!) It's better now, but I'm not sure how to handle this particular case other then to tell isort not to mess with the import. I hope this is helpful.
Thanks!
~Timothy


Use the tagged version of Documenter

@MichaelHatherly You put in the clone originally, just double checking that you are ok with this change here.

Thanks, that was probably there from before the package was registered I guess.


[BUG] gerring error while running project

getting error while trying to login in the project using user name and password.
Symfony\Component\ErrorHandler\Error\FatalError
Declaration of App\Menus\MenuFilter::transform($item, JeroenNoten\LaravelAdminLte\Menu\Builder $builder) must be compatible with JeroenNoten\LaravelAdminLte\Menu\Filters\FilterInterface::transform($item)
Expected behavior
home page should be open

EnvironmentComplete the next environment information.

Item
Version

Laravel
8

OS
Windows 10

Symfony\Component\ErrorHandler\Error\FatalError
Declaration of App\Menus\MenuFilter::transform($item, JeroenNoten\LaravelAdminLte\Menu\Builder $builder) must be compatible with JeroenNoten\LaravelAdminLte\Menu\Filters\FilterInterface::transform($item)

It appears that your custom MenuFilter filter is not compatible with the new FilterInterface. Probably, because you have updated the package from a very old version. Read Custom Menu Filters Examples to check how you must defined your filters and adequate your code.


Using $apply as mentioned in #42

This makes the hub service use the apply function, making it so that you
do not have to call $apply() in every callback.
At the same time, invoke has been updated to return a angluar promise,
for the same reason

@dimsedane I see that you also changed the invoke method. Care to explain your reasoning there?

Same reason really, I turn the signalR promise into a $q promise, which is scope aware.

Ok, I didn't realize that $q was scope aware so that's why I asked. I learn new things about angular all the time :)
Will this be a braking change for people who currently have the $scope.$apply() inside their method? I would assume this would cause Angular to throw the "already in a digest error".

Yes, it will break for them, and the $q callback will also be a breaking change, since the function is named "then" rather than "done", which signalR uses.

Are there not scenarios where you don't really need the $apply() call? Anything you do in the background that doesn't require the user to see it right away...
I'm just thinking "out loud" here. At the moment I myself have always used $apply() because I needed it. Trying to think bigger than only my situation.

There may be scenarios where that is the case, but the $scope.$apply should happen as close to the async event as possible.
If you are writing an app where there are performance problems with using $scope.$apply in the hub class, there could be an option to keep doing it "the old way"?
On another note, working with this in my own project, it is clear that more work and testing is needed before this should be merged, and I am therefore closing the pull request until that is done. I do still believe that some form of this is the right way to go.

Thanks for the heads up @dimsedane. After hearing @DeBiese's thoughts I think that if/when we do add this it should be toggleable via an option or something.


withQuery handles inline JSON wrongly.

When using e.g. {json: '{"test":["content"]}'} as query object, the result differs a lot from URLSearchParams and some systems do not accept the input.
Reproduction (StackBlitz):
import { withQuery } from 'ufo'

const content = {json: '{"test":["content"]}'}

//  json=%7B%22test%22%3A%5B%22content%22%5D%7D
const urlSearchParams = new URLSearchParams(content).toString()

// json={%22test%22:[%22content%22]}
const ufoResult = withQuery('', content)

While the colon : does not need to be encoded necessarily, the square and curly brackets should be. Some APIs won't accept input then.
Example calls:

https://api.x.immutable.com/v1/orders?buy_token_type=ETH&direction=asc&include_fees=true&order_by=buy_quantity&page_size=10&sell_token_address=0xacb3c6a43d15b907e8433077b6d38ae40936fe2c&status=active&sell_metadata={%22proto%22:[%2294%22],%22quality%22:[%22Meteorite%22]} - does not work

https://api.x.immutable.com/v1/orders?buy_token_type=ETH&direction=asc&include_fees=true&order_by=buy_quantity&page_size=10&sell_token_address=0xacb3c6a43d15b907e8433077b6d38ae40936fe2c&status=active&sell_metadata=%7b%22proto%22:%5b%2294%22%5d,%22quality%22:%5b%22Meteorite%22%5d%7d - does work

Hi, this is a major issue for us and prevents us from migrating to Nuxt 3. I hope this fix can be merged as soon as possible.


Please help with installation and running necrobrowser with Muraena

Hello,
Please help with installation and runnnit necrobrowser with Muraena as I am not able to find any such guide in github or elsewhere.
Please help
Thanks

@ohpe @antisnatchor Please help.
Thanks

@ohpe @antisnatchor
Please advice.
Thanks

@Avinashjaiswal420 Hi Avinash, can you please drop me a line?


Missing error on duplicated key in map literal

Iff all keys in a map literal are compile-time constants, it is a compile-time error if two keys are equal.
Caught by language_2/compile_time_constant_o_test.

The Dart VM does not report a duplicate key error, the output from running this test
Unhandled exception:
Expect.identical(expected: <{foo: 499}>, actual: <{foo: 42, foo: 499}>) fails.
#0      Expect._fail (package:expect/expect.dart:651:5)
#1      Expect.identical (package:expect/expect.dart:177:5)
#2      main (file:///usr/local/google/home/asiva/workspace/dart-tot/sdk/tests/language_2/compile_time_constant_o_test.dart:32:10)
#3      _startIsolate.<anonymous closure> (dart:isolate/runtime/libisolate_patch.dart:289:19)
#4      _RawReceivePortImpl._handleMessage (dart:isolate/runtime/libisolate_patch.dart:171:12)

This issue is still actual.
The code below works and doesn't produce any error
class C {
  final a;
  final b;

  const C(this.a, this.b);

  bool operator== (other) => this.a == other.a;
}

main() {
  const m1 = <int, String>{1: "val1", 1: "val2"};
  var m2 = const <String, int>{"key1": 1, "key2": 2};
  const m3 = <C, int>{const C(1, 2): 1, const C(1, 3): 2};
  var m4 = const <C, int>{const C(1, 2): 1, const C(1, 3): 2};
}

Dart VM version: 2.2.1-dev.1.1 (Fri Mar 15 10:54:11 2019 +0100) on "windows_x64"

The example here should definitely produce errors, but a couple of them may not be relevant: We should get an error because m1 has two keys with the value 1, and ditto for m1 and "key1"; but with m3 and m4 we should get an error because C does not have a primitive operator == (spec, section 'Maps'):
It is a compile-time error if the operator
‘==’ of the key of an entry in a constant map literal is not primitive (10.1.3).

So we would have to rely on canonicalization and Object.== (that is, not overriding it in C) in order to get the errors associated with duplicate keys in constant maps.
Also, it might remove a source of confusion to use hashCode => a.hashCode (even though the dynamic error associated with returning an instance of C won't occur as long as we are considering an example with compile-time errors).

This will be fixed when the new front end constant evaluator is enabled.


Add cgi module

Added the cgi module(https://docs.python.org/3.8/library/cgi.html).
There are still three failing tests in test_cgi.py, one of which requires #935, and the other two needs 'w+' file mode support.
This also has updated the urllib stdlib package to match the CPython 3.8.3 version.

@coolreader18 @palaviv @windelbouwman Do we have any reason to use BufferedIO instead of std::io::BufRead? https://doc.rust-lang.org/std/io/trait.BufRead.html


Make the second constructor public on SeleniumWebDriver to allow the …

…driver to be provided.
In my particular case, I needed to tame the logging to the console for the PhantomJs browser by providing the command line argument.  The existing constructor instantiates the PhantomJSDriver before I can get the parameter to it.
Open to other solutions - I appreciate this opens up some implementation you may not want to expose at this level.
// check for Phantom JS and if we are using that browser, tone down the logging
// into the console

var driverService = PhantomJSDriverService.CreateDefaultService();
driverService.AddArgument("--webdriver-loglevel=ERROR");

var driver = new PhantomJSDriver(driverService);

var browser = Browser.PhantomJS;
**var seleniumDriver = new SeleniumWebDriver(driver, browser);**

return new BrowserSession(sessionConfiguration, seleniumDriver);

Thanks. How people normally do this is like this:
https://github.com/featurist/coypu#using-a-custom-selenium-webdriver-profile
That work for you?
Having said that, it wasn't obvious to you (and you're not the first) so can't really see why not to make that constructor public, be more discoverable for sure.

I simply did not put 2 and 2 together when reading that link.  Maybe we can use my example as well (I think everyone would want to turn of the huge dump PhantomJs to putting to my console!).  That might help with the discoverability.  I'll give it a try and report back here.

I ended up with this and it works without a code change:
using Coypu.Drivers;
using Coypu.Drivers.Selenium;
using OpenQA.Selenium.PhantomJS;
using OpenQA.Selenium.Remote;

namespace Pharmerica.Efile.Pcc.ScreenScraper
{
    /// <summary>
    /// Class CustomPhantomJsProfileSeleniumWebDriver implements a custom Web Driver that allows for
    /// command line parameters to be configured on the PhantomJs driver.
    /// </summary>
    public class CustomPhantomJsProfileSeleniumWebDriver : SeleniumWebDriver
    {
        #region constructors

        // ReSharper disable once UnusedParameter.Local (browser) - required parameter for reflection activation in
        // Coypu framework.

        /// <summary>
        /// Initializes a new instance of the <see cref="CustomPhantomJsProfileSeleniumWebDriver"/> class.
        /// </summary>
        /// <param name="browser">The browser.</param>
        public CustomPhantomJsProfileSeleniumWebDriver(Browser browser) : base(CustomProfileDriver(), Browser.PhantomJS)
        {
        }

        #endregion

        #region methods

        private static RemoteWebDriver CustomProfileDriver()
        {

            var driverService = PhantomJSDriverService.CreateDefaultService();

            // tone down the logging into the console
            driverService.AddArgument("--webdriver-loglevel=ERROR");

            // we don't need the images
            driverService.AddArgument("--load-images=false");

            return new PhantomJSDriver(driverService);
        }

        #endregion
    }
}

Ok great. But I still like your change. May as well make things easier. If you can just add an acceptance test that works the original way (calling the new public constructor in this way), and also update the Readme section I pointed you to, to show this method, then I'll merge this.

Suggested changes not provided. Also PhantomJS is not supported anymore.


fix(plugins/plugin-client-common): editing a commentary link can fail with uncaught decodeURIComponent error

Fixes #7688

Description of what you did:

My PR is a:

[ ] 💥 Breaking change
[x] 🐛 Bug fix
[ ] 💅 Enhancement
[ ] 🚀 New feature

Please confirm that your PR fulfills these requirements

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

/lgtm


Document minimal dependency versions

ThorsLostKitten: @Tsoding figured it out, I was defaulting to g++ 5, using 7.4.0 fixes some issues. the rest was fixed by specifying libpng16-dev explicitely (had libpng12-dev by default) - so my system was just outdated

Probably already resolved by #239

But libpng is not used by current version of "Something".


Extend bad-dunder-method-name to permit __html__

The __html__ dunder method is used by Jinja2, Django and other frameworks (?) to return the "safe" html representation of an object.
There doesn't seem to be an equivalent of pylint's good-dunder-names in ruff. Would it be acceptable to add __html__ to the allow list?

I'm willing to make a PR if this is accepted

Yes that seems reasonable. PR appreciated!


[SAGE-780] SageGrid column class fix

Description

SAGE-780
Addresses a bug in the 12-column grid when using Rails SageGridCol component by removing the duplicate sage-col class when a responsive breakpoint is provided. In certain cases, this caused irregular sizing issues due to conflicting media queries.
Most easily noticeable when the lg breakpoint is in use. For example, open the "Help & Feedback" modal in the KP sidebar, and note that the sage-col--lg-6 columns follow the sage-col-6 media query when resized.
Screenshots

Before
After

Testing in sage-lib

Navigate to the Sage docs homepage
Inspect any of the content columns (within .docs-section)
Confirm the grid columns are only rendering responsive grid column classes (ex. sage-col--md-4)

Testing in kajabi-products

(HIGH) Modifies output of SageGridCol component

[ ] Website -> Blog posts -> Edit post
[ ] Sidebar -> "Help & Feedback" modal

Related

Love it! Nice work. Not a blocker, but I wonder if this could/should be a method on the Grid component itself rather than in a helper?

That's a great point, separating out into a helper is probably unnecessary. Since this was intended as a somewhat temporary patch until the 12-col grid situation is fleshed out, I'm inclined to wait to fold it in until we have a chance to refactor the component.


unable to locate find_cell_id_.py

Hi,
I cannot seems to locate find_cell_id_.py?

find_cell_id.py was an old script who is integrate in simple_IMSI-catcher.py


Captive Portal Allowed Addresses not saving

Hi,
After configuring CP, i wanted to allow some hosts in the Guest zone to bypass CP and thus added <IP_ADDRESS>/32 in Allowed Addresses, saved and Apply. However the host is still being redirected to the CP login page. Checking the configuration, the Allowed Address is black. I tried various permutations including excluding the entire subnet. I am on 17.7.5

You mean empty with "black"? Enter "," after the IP address and then save.

Got it! Thanks!


TimeSeries delete missing method

Fix from https://github.com/basho/riak-php-client/issues/150

Or cherry-pick https://github.com/ramon-ga/riak-php-client/commit/1f1318a3702a7f86f41eb0841db54a0ea6e87b8f

Thanks I'll check it tomorrow when coming home :p
Cheers,
Georged
On Wed, Jun 6, 2018, 12:57 AM Anthon Pang<EMAIL_ADDRESS>wrote:

Or cherry-pick ramon-ga/riak-php-client@1f1318a
https://github.com/ramon-ga/riak-php-client/commit/1f1318a3702a7f86f41eb0841db54a0ea6e87b8f
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/PHPSocialNetwork/riak-php-client/pull/2#issuecomment-394886552,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABRTZ33u6GNkzMOjQBUg-KRHJslOL_uvks5t5wzRgaJpZM4UbuGW
.


 I expected fetch to throw an exception but it didn't. Is it a bug?

Fetch does not throw exceptions when the network disconnects. If configured correctly, it will gracefully retry the download. If not configure to retry, the downloads will fail and you will get the reason on a listener callback.


[instancer] Fix bug in _instantiateFeatureVariations()

Fixes https://github.com/fonttools/fonttools/issues/2733

we can add a test later, I want to include this in the upcoming release


chore: improves loggers by adding closer.

Thank you for contributing to Coraza WAF, your effort is greatly appreciated
Before submitting check if what you want to add to coraza list meets quality standards before sending pull request. Thanks!

Make sure that you've checked the boxes below before you submit PR:

[ ] My code includes positive and negative tests.
[ ] I have an appropriate description with correct grammar.
[ ] I have read Contribution guidelines, maintainers note and Quality standard.
[ ] My code is properly linted and passes pre-commit tests.

Thanks for your contribution :heart:

Codecov Report
Base: 76.75% // Head: 76.99% // Increases project coverage by +0.24% :tada:

Coverage data is based on head (e6988e3) compared to base (b987f0f).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           v3/dev     #415      +/-   ##
==========================================
+ Coverage   76.75%   76.99%   +0.24%
==========================================
  Files         136      136
  Lines        5966     5960       -6
==========================================
+ Hits         4579     4589      +10
+ Misses       1118     1104      -14
+ Partials      269      267       -2

Flag
Coverage Δ

no-tinygo
77.12% <100.00%> (+0.24%)
:arrow_up:

overall
76.99% <100.00%> (+0.24%)
:arrow_up:

tinygo
75.41% <ø> (+0.04%)
:arrow_up:

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

Impacted Files
Coverage Δ

logger.go
64.86% <ø> (+4.86%)
:arrow_up:

loggers/concurrent_writer.go
83.01% <100.00%> (+10.46%)
:arrow_up:

loggers/serial_writer.go
89.28% <100.00%> (+22.61%)
:arrow_up:

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

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


[Feature Request] Auto edit html tag

When edit the front tag, the end tag will also sync with the front tag: <textar></textar>
Example:
https://github.com/formulahendry/vscode-auto-rename-tag

That would be cool, indeed.

any update?

The feature will be available in the next release (this week).

Released.


Access  elements?

When you use the method paper.text(0, 0, ["Some", "example", "text"]); I can't seem to find a way to get an Element handle on any of the <tspan> elements...
What if I want to use that method and then manually adjust the dy and dx values for each? I'd almost like to be able to do something like:
textElement = paper.text(0, 0, ["Some", "example", "text"]);
textElement.tspans[0].attr({ dy: '1em' });

@DmitryBaranovskiy Could I recommend this little snippet is added to the documentation for the paper.text method?

I couldn't agree more.

Hello from 2022! I agree with the suggestion to add that snippet to the documentation.
I've been looking around the web, and this Github issue (which is useful) is buried under other pages, which suggest other approaches. Before stumbling upon this discussion, I was leaning towards a variation of this (i.e., manually build the string and parse it):
//method 3, simply 'parse' the relevant text you want to add
var frag = Snap.parse('<text x="40" y="90" style="font-size:40px id="text3340">1<tspan style="font-size:20px;baseline-shift:super;" id="tspan3344">3</tspan><tspan id="tspan3352" style="font-size:22.5px">4</tspan></tspan></text>')
s.append(frag)

There's a few different ways to do it! Both of those should be fine.


Change browser

Hello. I would like to know if I can change Chromium to another browser, such as Firefox or Midori, and if so, how can I do it?
Thank you for your work

You should be able to install e.g. Firefox just as you normally would onto a Raspbian.
That said, not all of the scripts will then work as before.
Good luck!

"not all of the scripts will then work as before"  That's my problem, what changes should I make to my script for the chilipie-kiosk to work properly with firefox?
As far as I understand with Chromium I can no longer use chrome://flags/#enable-experimental-fullscreen-exit-ui, and the consequence is that it becomes too easy to exit fullscreen, making the kiosk vulnerable to interaction of the users.
Maybe you know how to avoid that behavior in Chromium?
Thank you very much for your work and dedicate your time to answering the doubts of all users.

I'm sorry but I don't enough about Firefox CLI flags to tell you that off the top of my head.
I'm sure they exist, though, check out the Firefox docs.


Minor modifications on readme

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

Comparison is base (1e4105d) 80.05% compared to head (a2624c4) 79.98%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #131      +/-   ##
==========================================
- Coverage   80.05%   79.98%   -0.07%
==========================================
  Files          66       66
  Lines        2943     2943
==========================================
- Hits         2356     2354       -2
- Misses        587      589       +2

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


Fixes Typescript: At Issue #1027

Fixes the issue #1027 that while compiling, Timer is Not  assignable to parameter of type 'string', So Changes the NodeJs.Timer to NodeJs.Timeout
After that it's gives another compile error in library so in tscondig.json added flag to skipLibCheck
Now it's compiling successfully

npm install --save @walletconnect/client @walletconnect/qrcode-modal

npm install --save @walletconnect/client @walletconnect/qrcode-modal

Sorry didn't get it what it means to be? add some additional information! Thanks

I believe this was fixed when I updated all the samples to TS 5.5. If there are still issues, please let me know and update this PR


[1.18.2] Black hole inventory on multiplayer not showing

Information
Industrial Foregoing Version: industrial-foregoing-1.18.2-<IP_ADDRESS>-9
Titanium Version: titanium-1.18.2-3.5.7-39
There is no crashlog nor any logs on the minecraft's console
Description
Black hole units do not open their respective inventory on multiplayer, they still work fine but no interface whatsoever.
The host (if any) seems to be capable of opening said interfaces

Try updating titanium


deleting tags in between other tags leaves a space

It looks like the intention here is to leave room while someone types in a new tag, but if you're just deleting tags, that's not the desired behavior and the extra space looks a bit weird.

Deleting a tag basically acts as if you clicked on it and deleted its contents. I guess placing the input to the right of every tag would make sense in these cases.

I liked it better. Resolved in 1.1.0

:+1: thanks!

Note that walking around the tags with the keyboard would still result in an "empty space" in the middle if you remove a tag (until you walk somewhere else), but in those cases it does make a lot more sense than when clicking on the remove buttons, so that behavior isn't changing.


Cacheable broker epoch

Adding the field maxBrokerEpoch to newer versions of the 3 controller requests:
LeaderAndIsrRequest, UpdateMetadataRequest, and StopReplicaRequest.
The purpose is to make the UpdateMetadataRequest cacheable on the controller.
In the current code, when the controller sends out a UpdateMetadataRequest, it needs to instantiate
many copies of the request, with each one targeting a different broker, and thus having a different brokerEpoch value.
This PR will change the 3 controller requests, including the UpdateMedataRequest,
to be using the maximum broker epoch across the cluster,
such that all brokers can share the same object. As a result, the memory footprint
on the controller should be reduced by a factor of O(number of brokers in the cluster).
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

is this making it upstream, or will this be a linkedin-specific optimization ?

assuming maxBrokerEpoch = ZK epoch when newest broker was created, why not use highest ZK epoch controller has "consumed"? (would be > maxBrokerEpoch and would give a more accurate estimate of the data upon which any controller commands are based?)

is this making it upstream, or will this be a linkedin-specific optimization ?

I'll push for this change in upstream. But that may take a long time, and I don't want that to delay our optimization.


Add extensive .gitignore to prevent accidental committed files

Added a generated .gitignore from gitignore.io (see here)

Thank you for your contribution!


chore(master): release 1.15.2

:robot: I have created a release beep boop
1.15.2 (2024-10-18)
Bug Fixes

improve @mention parsing for GitHub usernames (#33) (925765f)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/SethCohen/github-releases-to-discord/releases/tag/v1.15.2 :sunflower:


feat: Implement spanner::Value support for TIMESTAMP and DATE

TIMESTAMP is represented by std::chrono::system_clock::time_point,
and DATE by a new, simple "YMD" type, google::cloud::spanner::Date.
Also add conversion functions for google::protobuf::Duration and
google::protobuf::Timestamp to/from std::chrono::nanoseconds
and std::chrono::system_clock::time_point respectively.
Fixes #50 and #51.

I think you need to rebase to fix the build failures.

General question: Is there any reasonable way for us to use cctz directly (https://github.com/google/cctz) for these time functions? Either as a real dependency, or maybe even as a copy in our source in a unique namespace so it never conflicts with anything else? Or is this too much work for the minimal time processing that we need?

I don't think this is worth doing anything about until we can just move to Abseil.

nit (maybe): blank line?

So added.

Actually it is stdlibc++ that does not provide those functions right? We just don't compile with Clang on a distro that has such an old version of the library.

Acknowledged.  By "GCC" I was trying to refer to the entire (default) toolchain/libraries.  If there is a better term for that, I'm happy to add it.
That said, the code actually reads __GNUC__ < 5, so even if you think of "GCC" as just the compiler, the comment reflects that.
And all that said, I'd be happy if you came up with some cmake magic that could say whether the standard library it thinks I'm eventually going to link against has std::get_time() or std::put_time(), but that's no panacea either.

So we already have a FormatRfc3339 in google::cloud::internal, why not use that one? Or if this one is better, why not put this implementation there?
Similarly, we have a ParseRfc3339 function in google::cloud::internal, why not use that one and/or improve it?

Why not use them?  Because they do things like use mktime() and then try to determine the local UTC offset to compensate, and assume that the system_clock epoch is the UNIX epoch, etc.  In general I would say we're best off with std::get_time() and std::put_time() [modulo having to deal with libstdc++ history, which hopefully we can stop doing at some point].  Those APIs also don't fully support dates.
Why not improve them? That's what this is trying to do.
Why not put this implementation there?  Only because that is a bigger step.  In the end I imagine they wouldn't be there or here, but rather in the "common" repository.  But you have to start somewhere.

The include order is backwards here, we should include <time.h> after all the other headers. And do we want HAVE_GET_TIME defined before we included system headers?

OK, revamped the ordering.  Not defining something locally, like HAVE_GET_TIME, means we have to repeat the test, but that's OK.

Note that this is now ~1200 lines, should we split this?

OK.  Closing.

I know this is now closed, just a couple of things to follow up on:

Why not improve them? That's what this is trying to do.
Why not put this implementation there?  Only because that is a bigger step.  In the end I imagine they wouldn't be there or here, but rather in the "common" repository.  But you have to start somewhere.

Ack. That makes sense, maybe a TODO+GitHub issue?

OK, revamped the ordering. Not defining something locally, like HAVE_GET_TIME, means we have to repeat the test, but that's OK.

I did not express myself well.  I was thinking that the #include <time.h> should go last, then you can have the local defines without accidentally changing how the system includes interpret HAVE_GET_TIME.

That makes sense, maybe a TODO+GitHub issue?

Shall do (re: reconciling string <=> time/date functions).

I was thinking that the #include <time.h> should go last

Ack, although I'm thinking that repeating the include test in the code (as opposed to introducing new, local defines) isn't so bad.  It is straightforward, and it is self testing (if it links, it is right).


Document Id

Hi,
The document ids for the corresonding wiki pages seems to be wrong.
Ex : I gave a query doctitle:(cowboys  AND aliens)
gave results with docids which are either wrong page ids or pointing to some other pages.
Could anyone please confirm if the document ids to get the corresponding wiki pages are right

Can you share your code so that we can reproduce the problem you are encountering?


Choose Camera possible?

Is it possible to choose what camera this uses?

Yes.

If you want to this more programatically and for example by default use rear camera on mobile phone instead of front camera you can:

This looks amazing!

I've been attempting to get a second camera working for awhile now. I could use a bit of guidance.
So far I have the demo of this repo working, I use xirsys as my STUN/TURN service and PubNub for Signalling. All is well.
Within the startcamera() function on line 442 in the webrtc-v2.js I've modified that function and created a Option / Select for Camera 1,  Camera 2, etc using the, "navigator.mediaDevices.enumerateDevices()" Found here - https://raw.githubusercontent.com/webrtc/samples/gh-pages/src/content/devices/input-output/js/main.js
This actually works and is allowing me to select from a drop down a USB camera, and then back to the built in camera.  This functionality also changes the source "blob".  All is well!
However now, what can I do to get the other video stream to update?  When I change camera source, the other video freezes and can never recover, short of a full page refresh.
Help?!
@stephenlb @Pacific112

Hello,
I'll need assistance in this question too. I've found out, that the client muss send a new sdp offer or answer in the onnegotiationneeded event but all of my attempts failed. When changing the local camera, that the screen on the other side is always freezing, because there is no more stream data.
I've tried this:
     if(options.direction === ghWebRtcConversationDirection.incoming) {         console.log('create answer on local stream change');         if(_descriptionPaket) {           _peerConnection.setRemoteDescription(             new RTCSessionDescription(_descriptionPaket),             function() {               _peerConnection.createAnswer(                 function(answer) {                   _peerConnection.setLocalDescription(answer, $.noop, handleError);                   transmit(options.sessionIdentifier, answer, 2);                 },                 handleError,                 {}               );             }           );         }       }       else if(options.direction === ghWebRtcConversationDirection.outgoing) {         console.log('create offer on local stream change');         _peerConnection.createOffer(           function(offer) {             transmit(options.sessionIdentifier, offer, 2);             console.log('sent offer to other client');           },           handleError,           {}         );       }
I think there is something missing in the function add_sdp_offer(message). There is a line of code:
if (type in talk)   return; talk[type] = true;
So a "reoffer" is always dropped. Commenting out this, results in many "invalid rtc state errors" of the browser. So I really don't know how to change this.
Does anyone have some suggestions?

You can best achieve this by ending the existing session and starting a new one.  If you want a Seamless Transition, then start the new stream and connect to your caller, upon success, allow the receiving client to close the old connection.  You can overlay the original video feed with the new video feed to make this appear seamless.
This simple approach should solve your needs for all WebRTC clients, even older outdated WebRTC Clients will accept this approach.

Removing and adding a whole connection seems quite complicated. Think about a multi call scenario with many connections to different clients. Is there really no way to use the desired onnegotationneeded event? It's fired on the side where the stream is changed. I'm trying to send an offer/answer to the other side in this case. But the new offer is ignored on the other side. Removing the if-constraint there will result in multiple errors "invalid rtc state errors".
I like to have a "modern" solution. I've control over the webbrowser the user uses. So there is no need to support older browsers. Did you ever tried the modern approach or are there other problems involved, which makes this impossible?


Wrong steps (count/length) in treadmill

Don't know, if it is only with my runs, but I get a wrong Step-Count and -Length (in treadmill):
Values: 160 spm, 0,64m (step-count: 9616)
calculated: 8 spm, 2,55m (step-count: 481)
The failure is within the "export_tcx.c".
The Code from line 265 (with my correct working version):
// BEGIN ORIGINAL
                if ((int)record->treadmill.steps <= 4) // max 4 * 60 = 240 spm
                    cadence_avg = (0.05 * 30 * (int)record->treadmill.steps) + (1.0 - 0.05) * cadence_avg;
                fprintf(file, "                                <RunCadence>%d</RunCadence>\r\n", (int)cadence_avg);
// END ORIGINAL
// BEGIN MW (WORKING)
//                if ((int)steps <= 4) // max 4 * 60 = 240 spm
//                    cadence_avg = (0.05 * 30 * (int)steps) + (1.0 - 0.05) * cadence_avg;
//                fprintf(file, "                                <RunCadence>%d</RunCadence>\r\n", (int)cadence_avg);
// END MW (WORKING)

It seems like the values within record->treadmill are not correct (it contains total-steps!), but the steps are...

Yes, you're correct. I have pushed an update with the fix. Thanks for tracking this down. It's been there for a long time; I guess not many people use the treadmill function.

Thanks a lot!


chore: Better server logging infra

Description
The primary intent of this PR was to include request metadata into Sentry for the error logging that we have. This was not possible to do so far because the GlobalExceptionHandler loses track of the context of the user by the time flow of execution reaches there.

We now explicitly pass the context map via Appsmith exceptions so that the exception handler can use this information while logging errors.
Introduced a hook on to Webflux signals so that the MDC context from the original subscription flows into all the threads that are spawned during the course of an API request execution. This makes sure that the error signals also retain the original MDC context data so that it may flow through to Sentry
A side effect of this is that every thread within our server now logs with context (yay!)
Also worth mentioning is that we used to record the original thread of the request in the MDC context. Since these threads are meant to be changing across the course of execution, the context has been made to update with the new thread names where applicable. However, I wonder about the usefulness of this information at the present moment. Leaving the information in till we qualify it as unnecessary, if at all.
The plugin modules were not printing slf4j logs. The issue was minor here, we hadn't enabled debug level logs for that directory.
The print statements have also been converted into the appropriate slf4j log statements. This has been done in an almost 1-to-1 manner to avoid complicating the PR. We can revisit cleaning up logs in a subsequent change.
And of course, the addition of this information into Sentry.
Have also now included an ISO8601 formatted timestamp to the start of every log statement.

Fixes #9457
Type of change

Chore

How Has This Been Tested?

Manually tested by comparing logs and object built for Sentry

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

/ok-to-test sha=2670579

/ok-to-test sha=e734ac4

Added test plan approved since the change only requires regression testing.


docs: Document Release Process

As part of https://github.com/influxdata/pbjson/issues/127 I had to figure some of this out, and it would be nice to have it written down for next time

Thanks @tustvold


networking.nameservers use only the first nameserver on the list

Issue description
Given in configuration.nix the following entry:
networking.nameservers=[
    "2a00:6d40:0060:6a7f::0001" "2001:470:8388:2:20e:2eff:fe63:d4a9"
    "<IP_ADDRESS>" "<IP_ADDRESS>"
  ];

When <IP_ADDRESS> was down the dns query did not use <IP_ADDRESS> as backup server,even through this server was working
Steps to reproduce

set networking.nameservers to a pair [non_working_dns working_dns]
nixos-rebuild switch
try and fail to do any kind of dns query

Technical details

System:18.03pre119292.cfafd6f5a8 (Impala)
Nix version: nix-env (Nix) 1.12pre5663_c7af84ce
Nixpkgs version:18.03pre119292.cfafd6f5a8
Sandboxing enabled: true

Also I needed to wait for my dns to work again before I could correct the problem with my dns.
Why nixos-rebuild need to fetch only /nix/store/blph23whgr3a9svzp0vi5a2hpzw7kgw6-stage-2-init.sh,when every other component was built locally?

Apparently I needed to use networking.networkmanager.appendNameservers instead


[Scenario] Perfidia

Hello!
I would like to publish my 5th scenario.
Thank you.
What kind of mission do you want to add (mark with an X, e.g. [X] Descent)?

[X] Mansions of Madness
[ ] Descent

What expansions does the scenario require?

[ ] Conversion Kit (CK)
[ ] First Edition (MoM1E) - Also Recurring Nightmares
[ ] Fatal Alchemy (FA) - Also Suppressed Memories
[ ] Call of the Wild (CotW) - Also Suppressed Memories
[ ] Beyond the Threshold (BtT)
[X] Streets of Arkham (SoA)
[ ] Sunctum of Twilight (SoT)
[ ] Horrific Journeys (HJ)
[ ] Path of the Serpent (PotS)

What expansions can the scenario optionally use?

[ ] Conversion Kit (CK)
[ ] First Edition (MoM1E) - Also Recurring Nightmares
[ ] Fatal Alchemy (FA) - Also Suppressed Memories
[ ] Call of the Wild (CotW) - Also Suppressed Memories
[ ] Beyond the Threshold (BtT)
[ ] Streets of Arkham (SoA)
[ ] Sunctum of Twilight (SoT)
[ ] Horrific Journeys (HJ)
[ ] Path of the Serpent (PotS)

Please enter the GitHub Url to your scenario here
https://github.com/DaniMassaro/Perfidia

Scenario added to the manifest.
It should be available whithin an hour.

added to Statistics sheets


ValueToJSON may emit invalid JSON for block strings

As mentioned in #295, raw string content in the AST is unquoted in block strings, but the ValueToJSON method assumes string content is quoted. This means invalid JSON may be produced when block strings are serialized to JSON.
For example, the following GraphQL block string:
"""Hello
there"""

will be serialized to JSON as:
"Hello
there"

It should instead be:
"hello\nthere"

is this fixed?

No. #295 made it so triple quotes are preserved in GraphQL documents, but triple-quoted strings may not be valid when serialized to JSON.


Add cam_orientation to deployment table

In GitLab by @Tim-Hofmeester on Sep 22, 2020, 12:36
Many (but not all) studies standardize the compass orientation in which cameras are placed (often facing north in the northern hemisphere). As this might influence how easy it is to identify images (the number of unknown animal records) this might be important information to collect.

In GitLab by @peterdesmet on Sep 23, 2020, 10:00
Thanks for reporting this! This issue has already been suggested in #42. Closing here as duplicate.


add missing trailing parentheses in code snippet

semaphore.signal → semaphore.signal()

Thank you very much @tayloraswift 🙂


Use unchecked (unstaged) for conflicted and disabled

fix checkstate for conflicted and disabled stagestate. Use unchecked (unstaged) for conflicted and disabled
@exactly-one-kas what do you think about?

Filetype
Old location
New location

dirty submodules
Staged and Unstaged Tree
Unstaged Tree

conflicted files
Staged and Unstaged Tree
Unstaged Tree

@exactly-one-kas what do you think?


Change validator for wxCheckList and wxRearrangeList

Description:

wxCheckList and wxRearrangeList both use a "List Validator" -- but the help for these is a generic description for wxArrayInt. We need to indicate that this validator is used to indicate which items are checked. Additionally, if the user is has filled out the choices property, then we need to initialize this variable. That of course needs to wait until we have a custom editor for checked choices (see #154).

Removing from beta -- I don't think having a static list of choices for these is going to be important enough to try to get in before the beta.

We've got the custom editor now, so this should be part of the 1.1 release.

The contents property for both wxRearrangeCtrl and wxCheckListBox already uses the new custom editor to allow setting checkboxes.
The "List Validator" is used for the above two control and also wxListBox which does not support a checked array list.
The wxWidgets documentation for the array is quite lengthy -- too lengthy for our help description:
The data structure used to allow this is the order array which contains the items indices
indexed by their position with an added twist that the unchecked items are represented
by the bitwise complement of the corresponding index (for any architecture using two's
complement for negative numbers representation (i.e. just about any at all) this
means that a checked item N is represented by -N-1 in unchecked state). In practice this
means that you must apply the C bitwise complement operator when constructing the
order array.

I'm going to close this since the contents editor has been done, and I don't think the lengthy description for the
validator is practical.


UI code reusability: pagination

Resolves: https://github.com/trustification/trustification/issues/152
The following code tries to separate "UI Component definition (HTML)" from the "Data management (data injected in the UI component)"

UI Component definition (HTML): SimplePagination is a UI component that should render the Patternfly Pagination component
Data management (data injected in the UI component): use_pagination_state is a hook that should manage the pages state logic and avoid repeating this code multiple times.

This PR is only replacing the current logic of https://github.com/trustification/trustificatispog/ui/src/components/vulnerability.rs but the same logic could be applied to any other component that uses pagination.

The use_pagination_state is promoting the usage of Page and Page Size rather than limit and offset because for UI components it makes sense to define pages and pageSizes because that is the natural language of Tables, although I'm aware that for REST APIs limit and offset might make more sense. What do you think?

This is just a draft PR and I'd would love to get your feedback guys.

Let me know when you think this is ready for a full review and we can merge it.

@lulf I think this PR does what I wanted to implement. When you have time, I would appreciate a full review of this PR please :)

@carlosthe19916 Can you rebase this branch with the latest changes from origin/main? The merging seems to encounter a conflict.

@carlosthe19916 Can you rebase this branch with the latest changes from origin/main? The merging seems to encounter a conflict.

Sorry, it must've been something I did with my IDE. Now I made sure I've rebased latest changes from main branch into my working branch

@carlosthe19916 It is still conflicting from what I can tell. I tried to rebase your branch manually to main:
[lulf@pteppic trustification]$ git checkout paginationState
branch 'paginationState' set up to track 'carlos/paginationState'.
Switched to a new branch 'paginationState'
[lulf@pteppic trustification]$ git rebase origin/main
Auto-merging spog/ui/src/components/advisory.rs
CONFLICT (content): Merge conflict in spog/ui/src/components/advisory.rs
Auto-merging spog/ui/src/components/vulnerability.rs
CONFLICT (content): Merge conflict in spog/ui/src/components/vulnerability.rs
Auto-merging spog/ui/src/utils/mod.rs
CONFLICT (content): Merge conflict in spog/ui/src/utils/mod.rs
error: could not apply ddcc2c6... Enhance page validation and apply pagination to vexination.rs and vulnerability.rs
hint: Resolve all conflicts manually, mark them as resolved with
hint: "git add/rm <conflicted_files>", then run "git rebase --continue".
hint: You can instead skip this commit: run "git rebase --skip".
hint: To abort and get back to the state before "git rebase", run "git rebase --abort".
Could not apply ddcc2c6... Enhance page validation and apply pagination to vexination.rs and vulnerability.rs

@carlosthe19916 It is still conflicting from what I can tell. I tried to rebase your branch manually to main:
[lulf@pteppic trustification]$ git checkout paginationState
branch 'paginationState' set up to track 'carlos/paginationState'.
Switched to a new branch 'paginationState'
[lulf@pteppic trustification]$ git rebase origin/main
Auto-merging spog/ui/src/components/advisory.rs
CONFLICT (content): Merge conflict in spog/ui/src/components/advisory.rs
Auto-merging spog/ui/src/components/vulnerability.rs
CONFLICT (content): Merge conflict in spog/ui/src/components/vulnerability.rs
Auto-merging spog/ui/src/utils/mod.rs
CONFLICT (content): Merge conflict in spog/ui/src/utils/mod.rs
error: could not apply ddcc2c6... Enhance page validation and apply pagination to vexination.rs and vulnerability.rs
hint: Resolve all conflicts manually, mark them as resolved with
hint: "git add/rm <conflicted_files>", then run "git rebase --continue".
hint: You can instead skip this commit: run "git rebase --skip".
hint: To abort and get back to the state before "git rebase", run "git rebase --abort".
Could not apply ddcc2c6... Enhance page validation and apply pagination to vexination.rs and vulnerability.rs

My apologies for causing too much trouble with with simple task of rebase.
I've just rebased again and fixed conflicts. Now when I execute git rebase origin/main this is what I get
[cferiavi@dhcp-10-40-4-237 trustification]$ git checkout paginationState
Switched to branch 'paginationState'
Your branch is up to date with 'origin/paginationState'.
[cferiavi@dhcp-10-40-4-237 trustification]$ git rebase origin/main
Current branch paginationState is up to date.

@carlosthe19916 No worries, the GitHub behaviour around merge queues are still a bit confusing, no error shown :shrug:


Marshmallow/Doze preventing wallpaper rotation

I've noticed Doze stops wallpaper rotation, completely. Android developer documentation says "Standard AlarmManager alarms are deferred to the next maintenance window" but this doesn't seem to be the case - the alarms are ignored and never run. I've noticed this with my plugin, which I've set to instruct Muzei to change wallpaper every 3 hours - it will be days before it changes, and only then because my phone happened to be awake when the alarm was scheduled to fire. It would make sense to update Muzei so API 21+ devices use JobScheduler, and let older devices continue to use AlarmManager.set().

Yeah I've noticed behavior similar to this randomly. /cc @ianhanniballake for ideas?

Alarms are not ever ignored by doze / App Standby. Can you check the results of adb shell dumpsys alarm (with the help of this guide) alongside the guide for testing with doze?
I do have some designs for reworking the API to move away from alarms, but as part of a larger effort to provide more control over rotation times in Muzei

Yeah I suspect if anything it's a very delayed alarm, not a completely ignored one. I personally haven't repro'd in a while. /cc @mdtardif

It is happening for my extension too. Users even sent information about it sayin that it can be related to Doze mode.
My personal observation is that wallpaper rotates successfully when the devices is connected to charger (which disables Doze)

While that works, it does use marginally more battery as Muzei will wake up your device and access the network to download new artwork when your device would otherwise be asleep although unless you have an extension that loads artwork at a very fast rate (<5 minutes), you're right that this will have no noticeable impact on battery life for daily use.
Still working on this though!

I think I got the same problem for my extension too. Rotating seems to work if the phone is awake and charging.

Any progress on this? Nexus 5X, LineageOS 14.1 (Nougat), battery optimalization disabled for Muzei, latest beta of Muzei from Play Store, using default "daily artwork" source and I'm looking at the same wallpaper for 3 days.
If I enable battery optimization, I'll eventualy get some default Material Design wallpaper after couple of days (#207).

@Pitel - this bug does not affect the included Sources - only third party ones. Please send a bug report to<EMAIL_ADDRESS>if you're having issues with the included Sources or if Muzei gets replaced with the default wallpaper.

Muzei 3.0's new API does not require disabling battery optimizations.


Privly posting doesn't work with Youtube comment

I think it is because Youtube comment box will be destroyed once it lose focus without any content.

Can you provide a link to an example?

Great! Thanks for the very clear example.  Do you know if there are any messages in the console when insertion of the link fails?

Sorry there are no error messages in the console.
I found that when clicking the privly button, the commentbox is closed. When opening commentbox again, the original DOM has been destroyed (it creates a new DOM for user to input comments).

Thanks for the bug report. While I don't think YouTube comments are necessarily our target use case, we certainly don't want it to break like this.
Without testing this myself, I would say @SummerWish has characterized this properly. When you click on the form element on YouTube, Google's JS creates the real form element. If the form element loses focus when it does not have any contents, Google's JS destroys the form. This is likely going to become a common web development approach over time.
Options for solving:

Record click events and an xpath to the real form element when posting initiates. When posting completes, fire the click event again and select the proper form element with the xpath.
Ensure there is placeholder text before opening the posting window. Replace the placeholder text when posting completes or the posting window closes.
Others. I will think about it.

Hello. I implemented a pretty hacking solution for this issue. It works for Youtube but I have not yet tested edge cases. Please help review at this solution.

I just briefly read through your commit. Preventing the element from losing focus should be good in most cases. The only edge case I see is when the user doesn't complete posting and they navigate around the host page before returning to the posting application.

@smcgregor Yeah, thank you for the idea. Currently no workarounds for this.

I think the posting window closes at the moment regardless of whether it can successfully paste the link to the host page. If we fix #113, then the big UX problem with YouTube will be fixed.

seamless-posting works in Youtube comment, issue closed.


Fix published_at when editing a post

When editing a Post, the published_at time is set to 00:00:00.
This fix the problem by keeping the good date time.

@ftomasi Are you trying to create a new post? Or publish an existing one?

I have a published post. If I edit it, the date TIME goes wrong and is set to 00:00:00


Rclone website

Page http://rclone.org/docs/ , "Exit Code" section at the bottom
I think the sentence:  " [...] and only exit with an NON-zero exit code if (after retries) there were NO transfers with errors remaining. " should have that 'NON' xor that 'NO' ,  but not both because, if I got it right:
==0 no errors
!=0 still errors after retries
Bye & thanks for your work

Fixed - thank you!


Fix /part command

Fixes the /part command closing the wrong window. The current implementation simply passes all arguments to slate, which ended up parting every arguments (and never sending the part message)
This changes the command to /part message, and always parts the current window. This will be fixed further once irc-framework is merged.

@maxpoulin64, so it means that if the user is currently on channel #one and types /part #two, they will leave both #one and #two? If so, that seems odd, don't you think?
Correct me if I'm wrong but the current implementation of /part seems similar (or at least, aims to be similar) to the /invite command:

/invite #channel nick invites nick in #channel; /part #channel parts the current user from #channel
/invite nick is a shorthand to invite nick to the current channel; /part is a shorthand to part the current user from the current channel

Am I being mistaken in understanding your change?

@astorije

so it means that if the user is currently on channel #one and types /part #two, they will leave both #one and #two? If so, that seems odd, don't you think?

Actually that's precisely what this fixes. I will change it to /part [channel] [message] once irc-fw is merged, because I can't reliably know if [channel] is a valid channel.

I will change it to /part [channel] [message] once irc-fw is merged, because I can't reliably know if [channel] is a valid channel.

Makes sense.
I also notice that the doc says "Leave the current channel.", ouch. So doc and code were not completely consistent :-/
So now, if I type /part #channel hello world, it means I will part from the current channel and the message will be #channel hello world?

So now, if I type /part #channel hello world, it means I will part from the current channel and the message will be #channel hello world?

Yes, unfortunately. That will have to wait for irc-fw however, because I need access to CHANTYPES to fix is properly.
Note that the old behavior was to try to quit everything, so /part #channel hello world would have actually attempted to quit #channel, hello and world with a part message of "undefined".

Yes, unfortunately. That will have to wait for irc-fw however, because I need access to CHANTYPES to fix is properly.

Fine, even if not perfect, definitely much better than the current state then :-)
I therefore :+1: this PR a posteriori :D


Remove creds; re-gen services

PR summary
This PR removes the encrypted credentials files from the project and includes a re-gen of most services.
PR Checklist
Please make sure that your PR fulfills the following requirements:

[x] The commit message follows the Angular Commit Message Guidelines.
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] New tests
[ ] Build/CI related changes
[ ] Documentation content changes
[ ] Other (please describe)

What is the current behavior?

What is the new behavior?

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

:tada: This PR is included in version 0.12.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Grid: Nested grid sorting issue

E.g:
Grid 1
----- grid 1.1
----- grid 1.2
---------- subgrid 1.2.1
---------- subgrid 1.2.2
----- grid 1.3
----- grid 1.4
this is how the list of grids and subgrids was formed during initialization, and this is how it looks after clicking on grid 1.2
When I sort the grid x.x in descending order the display looks like this:
----- grid 1.4
----- grid 1.3
---------- subgrid 1.2.1
---------- subgrid 1.2.2
----- grid 1.2
----- grid 1.1
how to refresh after sorting?
Originally posted by @prizmasoftware in https://github.com/vikramlearning/blazorbootstrap/discussions/784#discussioncomment-10247528

@prizmasoftware Thank you for using BlazorBootstrap. I will check this one.

Thank you.

@prizmasoftware Please share a sample GitHub repository to replicate your scenario with minimal steps. This will help us better understand how you are using the components. If possible, please attach a screen recording.
Note: Please do not attach ZIP files.

@gvreddy04  I sent you an invite for the repo

@gvreddy04 I sent you, one more, invite for the repo.
https://github.com/prizmasoftware/Grid-Nested-grid-update-issue.git


new(modern_bpf): add support for futex syscall

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area API-version

/area build

/area CI

/area driver-kmod

/area driver-bpf

/area driver-modern-bpf

/area libscap-engine-bpf

/area libscap-engine-gvisor

/area libscap-engine-kmod

/area libscap-engine-modern-bpf

/area libscap-engine-nodriver

/area libscap-engine-noop

/area libscap-engine-source-plugin

/area libscap-engine-savefile

/area libscap-engine-udig

/area libscap

/area libpman

/area libsinsp

/area tests

/area proposals

Does this PR require a change in the driver versions?

/version driver-API-version-major

/version driver-API-version-minor

/version driver-API-version-patch

/version driver-SCHEMA-version-major

/version driver-SCHEMA-version-minor

/version driver-SCHEMA-version-patch

What this PR does / why we need it:
Add support for futex syscall in modern_bpf
Which issue(s) this PR fixes:

Part of: #723
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: loresuso
Once this PR has been reviewed and has the lgtm label, please assign lucaguerra for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/milestone next-driver

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, loresuso
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 9608b88f5f6e72590f8b43937dacd51a4a5a6025

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP, loresuso
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


MobileDatePicker

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Summary 💡
https://mui.com/x/api/date-pickers/mobile-date-picker/
Any chance of providing support for MobileDatePicker?
Thanks.
Examples 🌈
No response

Isnt MUI rendering the mobile-date-picker on the specified breakpoint?

It works as you said, but if I want to completely cancel the keyboard input, the only solution is to use MobileDatePicker.
In the document it states:

The MobileDatePicker component which works best for touch devices and small screens. It renders the view inside a modal and does not allow editing values directly inside the field.

If you want, I can create a pull request for this component.

sure go ahead

released with 6.3.0


Add Enum Type

Closes #15. Wasn't too sure about the String representation so I followed the example of Nullable.

Wasn't too sure about the String representation so I followed the example of Nullable.

Perhaps ("ms" | "seconds") is better than (Enum ("ms", "seconds")). This wouldn't provide a great way to represent an Enum with fewer than two possible values, but we shouldn't be bothered by such an absurd scenario. :)

Yep, I prefer that.

We should update equalTypes to recognize enumerated types. I believe duplicating the 'RECORD' case and changing the value to 'ENUM' should do the trick.

Good catch, I would have completely missed that one.

Hmm, how do the other parts of that case statement get tested?

Hmm, how do the other parts of that case statement get tested?

Let me investigate.

This'll restore coverage to 100%:
//  SillyType :: Type
var SillyType = $.EnumType(['foo', true, 42]);

var def = $.create($.env.concat([SillyType]));

//  id :: a -> a
var id = def('id', {}, [a, a], R.identity);

eq(id(['foo', true]), ['foo', true]);

throws(function() { id(['foo', false]); },
       errorEq(TypeError,
               '‘id’ received ["foo", false] as its first argument, ' +
               'but this value is not a member of any of the types ' +
               'in the environment:\n' +
               '\n' +
               '  - (Array ???)\n' +
               '  - Boolean\n' +
               '  - Date\n' +
               '  - Error\n' +
               '  - Function\n' +
               '  - Null\n' +
               '  - Number\n' +
               '  - Object\n' +
               '  - RegExp\n' +
               '  - String\n' +
               '  - Undefined\n' +
               '  - ("foo" | true | 42)'));

Nice one! I wasn't sure where to put it so I just added it in with the other tests. I'm wondering should I separate the two tests by adding it("can define a custom enumerated type)`` and it("can be tested for equality)``` functions.

:deciduous_tree:


fix(backup): fix emoji encoding, remove from broken backups

Description

fix incorrectly encoded emojis in backup json
add backwards fix for already broken backups
add text-encoding dependency, remove unused helpers

Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Refactoring (improving code without creating new functionality)

Tests

[ ] Detox test
[ ] Unit test
[x] No test

QA Notes
Make sure you can backup/restore data with emojis in tags & invoice notes

News, Blocks & Facts widgets are down right now, that's why e2e tests are failing.


Solve the ARIA-ROLE error thrown @Part 2/Actions/The use directive

Added a role attribute to remove the warning in both unsolved and solved one

thank you! @geoffrich can I ask you to take a look at this — is presentation the right role, or would button be more correct in this case?

Oh and there's also #567 which additionally adds an aria-label — is that appropriate in this context?

Hm, I think this is actually a case where it's perfectly valid to ignore the warning. Maybe we want to add the svelte-ignore comment to this example?
In general, you don't want to put click handlers on divs. However, in this case a screen reader can close the modal with the Escape key, so we aren't locking any functionality behind clicking with a mouse.
I'd have to test with a screen reader to be sure, but I don't think role="presentation" will actually do anything tangible other than silence Svelte's warning. A div is already meaningless, semantically.