Interface to associate GatewayDispatchEvents and relevant DataPayload (proposed code below)

Feature
A good addition would be an interface to associate a GatewayDispatchEvents event with the relevant DataPayload dispatched along this event.
Ideal solution or implementation
The following interface only works for V10 :
interface GatewayDispatchData {
	[GatewayDispatchEvents.ChannelCreate]: GatewayChannelModifyDispatchData
	[GatewayDispatchEvents.ChannelDelete]: GatewayChannelModifyDispatchData
	[GatewayDispatchEvents.ChannelPinsUpdate]: GatewayChannelPinsUpdateDispatchData
	[GatewayDispatchEvents.ChannelUpdate]: GatewayChannelModifyDispatchData
	[GatewayDispatchEvents.GuildBanAdd]: GatewayGuildBanModifyDispatchData
	[GatewayDispatchEvents.GuildBanRemove]: GatewayGuildBanModifyDispatchData
	[GatewayDispatchEvents.GuildCreate]: GatewayGuildCreateDispatchData
	[GatewayDispatchEvents.GuildDelete]: GatewayGuildDeleteDispatchData
	[GatewayDispatchEvents.GuildEmojisUpdate]: GatewayGuildEmojisUpdateDispatchData
	[GatewayDispatchEvents.GuildIntegrationsUpdate]: GatewayGuildIntegrationsUpdateDispatchData
	[GatewayDispatchEvents.GuildMemberAdd]: GatewayGuildMemberAddDispatchData
	[GatewayDispatchEvents.GuildMemberRemove]: GatewayGuildMemberRemoveDispatchData
	[GatewayDispatchEvents.GuildMembersChunk]: GatewayGuildMembersChunkDispatchData
	[GatewayDispatchEvents.GuildMemberUpdate]: GatewayGuildMemberUpdateDispatchData
	[GatewayDispatchEvents.GuildRoleCreate]: GatewayGuildRoleModifyDispatchData
	[GatewayDispatchEvents.GuildRoleDelete]: GatewayGuildRoleDeleteDispatchData
	[GatewayDispatchEvents.GuildRoleUpdate]: GatewayChannelModifyDispatchData
	[GatewayDispatchEvents.GuildScheduledEventCreate]: GatewayGuildScheduledEventCreateDispatchData
	[GatewayDispatchEvents.GuildScheduledEventDelete]: GatewayGuildScheduledEventDeleteDispatchData
	[GatewayDispatchEvents.GuildScheduledEventUpdate]: GatewayGuildScheduledEventUpdateDispatchData
	[GatewayDispatchEvents.GuildScheduledEventUserAdd]: GatewayGuildScheduledEventUserAddDispatchData
	[GatewayDispatchEvents.GuildScheduledEventUserRemove]: GatewayGuildScheduledEventUserAddDispatchData
	[GatewayDispatchEvents.GuildStickersUpdate]: GatewayGuildStickersUpdateDispatchData
	[GatewayDispatchEvents.GuildUpdate]: GatewayGuildModifyDispatchData
	[GatewayDispatchEvents.IntegrationCreate]: GatewayIntegrationCreateDispatchData
	[GatewayDispatchEvents.IntegrationDelete]: GatewayIntegrationDeleteDispatchData
	[GatewayDispatchEvents.IntegrationUpdate]: GatewayIntegrationUpdateDispatchData
	[GatewayDispatchEvents.InteractionCreate]: GatewayInteractionCreateDispatchData
	[GatewayDispatchEvents.InviteCreate]: GatewayInviteCreateDispatchData
	[GatewayDispatchEvents.InviteDelete]: GatewayInviteDeleteDispatchData
	[GatewayDispatchEvents.MessageCreate]: GatewayMessageCreateDispatchData
	[GatewayDispatchEvents.MessageDelete]: GatewayMessageDeleteDispatchData
	[GatewayDispatchEvents.MessageDeleteBulk]: GatewayMessageDeleteBulkDispatchData
	[GatewayDispatchEvents.MessageReactionAdd]: GatewayMessageReactionAddDispatchData
	[GatewayDispatchEvents.MessageReactionRemove]: GatewayMessageReactionRemoveDispatchData
	[GatewayDispatchEvents.MessageReactionRemoveAll]: GatewayMessageReactionRemoveAllDispatchData
	[GatewayDispatchEvents.MessageReactionRemoveEmoji]: GatewayMessageReactionRemoveEmojiDispatchData
	[GatewayDispatchEvents.MessageUpdate]: GatewayMessageUpdateDispatchData
	[GatewayDispatchEvents.PresenceUpdate]: GatewayPresenceUpdateDispatchData
	[GatewayDispatchEvents.Ready]: GatewayReadyDispatchData
	[GatewayDispatchEvents.Resumed]: never
	[GatewayDispatchEvents.StageInstanceCreate]: GatewayStageInstanceCreateDispatchData
	[GatewayDispatchEvents.StageInstanceDelete]: GatewayStageInstanceDeleteDispatchData
	[GatewayDispatchEvents.StageInstanceUpdate]: GatewayStageInstanceUpdateDispatchData
	[GatewayDispatchEvents.ThreadCreate]: GatewayChannelModifyDispatchData
	[GatewayDispatchEvents.ThreadDelete]: GatewayChannelModifyDispatchData
	[GatewayDispatchEvents.ThreadListSync]: GatewayThreadListSyncDispatchData
	[GatewayDispatchEvents.ThreadMembersUpdate]: GatewayThreadMembersUpdateDispatchData
	[GatewayDispatchEvents.ThreadMemberUpdate]: GatewayThreadMemberUpdateDispatchData
	[GatewayDispatchEvents.ThreadUpdate]: GatewayChannelModifyDispatchData
	[GatewayDispatchEvents.TypingStart]: GatewayTypingStartDispatchData
	[GatewayDispatchEvents.UserUpdate]: GatewayUserUpdateDispatchData
	[GatewayDispatchEvents.VoiceServerUpdate]: GatewayVoiceServerUpdateDispatchData
	[GatewayDispatchEvents.VoiceStateUpdate]: GatewayVoiceStateUpdateDispatchData
	[GatewayDispatchEvents.WebhooksUpdate]: GatewayWebhooksUpdateDispatchData
}

Alternative solutions or implementations
No response
Other context
No response

I've actually had the need for this myself and simply hacked it up:
import type { GatewayDispatchEvents, GatewayDispatchPayload, GatewaySendPayload } from 'discord-api-types/v10';

type _DiscordEventsMap = {
	[K in GatewayDispatchEvents]: GatewayDispatchPayload & {
		t: K;
	};
};

export type DiscordEventsMap = {
	// @ts-expect-error
	[K in keyof _DiscordEventsMap]: _DiscordEventsMap[K]['d'];
} & {
	send: GatewaySendPayload;
};

It doesn't play nicely with TSC, given the needed @ts-expect-error and the fact that it needs two types (you'd think you could just inline the first one, but it does not behave the same way), but it gets the job done.

The reason your hack requires @ts-expect-error is because some events don't have an associated payload (like ApplicationCommandPermissionsUpdate). You can fix it like this :

type _Payload = {
	[E in GatewayDispatchEvents]: GatewayDispatchPayload & { t: E }
}

type GatewayDispatchData = {
	[E in keyof _Payload]: "d" extends keyof _Payload[E] ? _Payload[E]["d"] : never
}

I edited my original post to include it as an alternative implementation.

Good catch! That explains why it only recently started occurring, thanks.

Thats quite a big issue if there's events without associated payloads... PRs are more than welcome!!

Thats quite a big issue if there's events without associated payloads... PRs are more than welcome!!

Done.

I will, but I noticed that GatewayDispatchPayload doesn't include every dispatch payloads, the following are missing :
GatewayAutoModerationRuleModifyDispatch
GatewayAutoModerationRuleCreateDispatch
GatewayAutoModerationRuleDeleteDispatch
GatewayAutoModerationActionExecutionDispatch

By missing, I mean that they're already implemented but not included in the union. Do you want me to add them as part of this PR or to create a new PR ?


Distracting progressbar

double tap for seek in video is a quick way to move in video.
but when i double tab always show progress it is good if have option to hide progress bar in double tap gesture.

Added a setting to do that in 61d80b3021a017ea9ed4d83af07558d1db8037ba


python3Packages.minikerberos: init at 0.2.7

Motivation for this change
Kerberos manipulation library in pure Python.
https://github.com/skelsec/minikerberos
Related to #81418
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@fabaff please fix the merge conflict.

@fabaff please fix the merge conflict.

@fabaff please fix the merge conflict.

Done

@fabaff please fix the merge conflict.

Done

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 108800 run on x86_64-linux 1

6 packages built:

python37Packages.asysocks
python37Packages.minikerberos
python38Packages.asysocks
python38Packages.minikerberos
python39Packages.asysocks
python39Packages.minikerberos

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.


Android Compose Generics Objects and Extensions

Generics, Objects, and extensions
URL: https://developer.android.com/codelabs/basic-android-kotlin-compose-generics?continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-compose-unit-3-pathway-1%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fbasic-android-kotlin-compose-generics#5
Describe the problem
It doesn't work as described. It said: Unresolved reference: progressText
In which lesson and step of the codelab can this issue be found?
"Add an extension property" in step 6.
Versions
Kotlin Playground v 1.9.10
Additional information
Add any other context about the problem here.
codelab: basic-android-compose-training-add-scrollable-list
Here is the code:
fun main() {
    println(Quiz.progressText)
}

class Quiz{
    // Declare a companion object in a class. This is also singleton object
    companion object StudentProgress {
    	var total: Int = 10
    	var answered: Int = 3
	}

    val Quiz.StudentProgress.progressText: String
    get() = "${answered} of ${total} answered"
}

solved


⚠ 366.ru has degraded performance

In b7d6ce7, 366.ru (https://366.ru) experienced degraded performance:

HTTP code: 200
Response time: 1642 ms

Resolved: 366.ru performance has improved in a7409d3.


Create data source for tracking breadth of catalogue accessed

Currently we're using the search relevance "Search result selected" metric to infer how much of our catalogue is being accessed (being the main entry of people getting to works).
We should start collecting work pages being accessed to be a more accurate indication of this.

Are you suggesting creation of a new index for work pages accessed by more than "Search result selected"? What would this look like?

@taceybadgerbrook would the first thing to collect be "What works have been accessed"?
Currently we will track the ID of the work. What other data about the work do you need?
e.g.
{
  "event": "Work viewed",
  "data": {
    "id": "njfu764k"
  }
}

I'm linking this to #4005 assuming this new index is the same as the index for conversion.
In addition to workId, I'd also like anonymousId and sessionId to be collected (just in case we finally stop collecting anonymousId.)
And if access is being compared to taking a book down from the shelf to look at it, should we also include clicking on archive tree? @jennpb what do you think?

@jamesgorrie Jenn's recommended that "access" stays defined as search result selected and not to include navigating archive trees.

So if you track workId and search result selected, that's all I need.


Add missing bookmark block types

Bookmark Blocks were documented on this API Reference page, but there are no type definitions of it.

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


Controllerno

Though when I double click, it open in VS Code

Assets\MoralisWeb3ApiSdk\Nethereum\Nethereum.KeyStore\Crypto\SCrypt.cs(44,37): error CS0227: Unsafe code may only appear if compiling with /unsafe. Enable "Allow 'unsafe' code" in Player Settings to fix this error.

This flag has solved the issue


Sourcery generating mocks from a protocol that does not exist anymore

Hello, I have done the following:

Created a protocol Foo and marked it with AutoMockable annotation (using default template)
Generated the mocks
Renamed Foo to Bar
Generated the mocks

The result is that in the generated file I have now both FooMock: Foo and BarMock: Bar, which prevents my project from building. Temporarily I have added an empty Foo protocol to tackle this but there must be a way to somehow fix this.
I have already tried reinstalling Sourcery, using --no-cache flag too, to no avail.
Environment:
Sourcery 1.9.0, using .yml config file, installed via Homebrew, M1 Pro Mac, Xcode 14.1.

can you provide sample project that reproduce the issue? it has to be something with cache

can you provide sample project that reproduce the issue? it has to be something with cache

Faced the same problem. What could be the problem?
Version 2.0.2

I'd nuke the cache folders, this could only happen if old AST Data was loading

👋🏻 Hello, thank you for creating the issue 👍🏻
Indeed, the only source of such (mis)behaviour might be the cache. Try using --disableCache flag and see if the mock for the removed (un-annotated) protocol is generated still.
If not, you can either set --cacheBasePath path for cache, or clean up the default cache location at ~/Library/Caches/Sourcery.
Feel free to re-open in case this is not solved.


Make a new config setting

Cuz why not :P

Ignore top 3, they wont do anything.

There are complications please  fix them

@Fan7asy Examples?

ahh, i see why

Let me try and fix this.


English documentation

Hi
Do you have any plans to write documentation in English?
I am in love with VueJs and this project seems may have a large future for mobile apps, but without documentation in English it is impossible to use it.

Don't worry<EMAIL_ADDRESS>(will release in few days) will have documentation in English.


Make a base class or protocol for ExternalBotHandler that zulip/zulip’s EmbeddedBotHandler can subclass

zulip/zulip currently needs an unchecked cast to convert a custom EmbeddedBotHandler class into an ExternalBotHandler. Since EmbeddedBotHandler is not a subclass of ExternalBotHandler (because it works by invoking internal_send_message rather than making API calls), this is unsafe, and it breaks in practice when we try to upgrade python-zulip-api to 0.7.1 (zulip/zulip#16511).
To fix this, we need a common base class or Mypy protocol, say BotHandler, that can be subclassed by both ExternalBotHandler and EmbeddedBotHandler. The zulip_bots functions must be relaxed to accept BotHandler instead of requiring ExternalBotHandler.
If the StubBotHandler class used in tests could subclass BotHandler too, I think this would also let us correct various bot_handler: Any types to bot_handler: BotHandler.

@andersk can I work on this?

@andersk can I work on this?

@zulipbot claim

@zulipbot claim

@zulipbot claim

@andersk Can I work on this issue as the in progress one is in review?

@zulipbot claim

@zulipbot claim


Slack Screen sharing not working on macOS Catalina

Hi, I want to share my screen on macOS Catalina while a slack call, but I always get the notification that I should use the Desktop App of Slack.
Franz Version 5.5.0 (<IP_ADDRESS>2)
macOS: 10.15.4

Same for Debian 10 (using Xorg)


Q: Anything similar to SQLCipher for libsql?

Is there anything to handle encryption / decryption at rest for libsql?

Hey @dougg0k, no SQLCipher for libSQL yet.

Anything related being worked on?

Hi! Is encryption / decryption handling near in your roadmap ?

Hey @dougg0k and @FrancoRATOVOSON, there is now encryption at rest support as of cd82068edf797d812d1fcc2a4b9ac482fe8cb7ce. We integrated the sqlite3mc extension which also supports SQLCipher that I intend to make the default cipher (see #893).


How to provide home position locally.

Hi,
I am flying my single copter indoor without GPS. I am using px4flow with i2c sonar. Its doing auto takeoff upto my defined altitude(1.3m). I have provided mission(if any) in "COM_TAKEOFF_ACT" parameter and uploaded a small mission already.
But it does not do mission and just keep holding position. Does it require gps necessarily for mission? Any suggestion?

GPS is a requirement for missions currently.


BinarySearch In Java

Hey @pranav-nb, hope you are doing well.
can I ask you ? is there any problem in my code or something else because you did not gave the label( hacktoberfest-accepted )in this pull request  .


Add CLI to install dependencies

What does this PR do?
Working with TPU requires installing some dependencies that are as simple to install as adding a pip dependency in the main package. This is even more evident for Jetstream Pytorch, that needs to be installed from a git repository.
Previously I had run the actions manually, added a target on Makefile and a script for it.
@mfuntowicz suggested that might not be practical for someone having the optimum-tpu package but not the sources:
https://github.com/huggingface/optimum-tpu/pull/99#discussion_r1786205247
As a result, I added a script and integrated it as the optimum-tpu CLI to provide a simple command to install Jetstream Pytorch:
optimum-tpu  install-jetstream-pytorch

Note that I also added an option to install pytorch xla. This is because in the future I might remove this dependency from optimum-tpu, since it is not required for inference when using Jetstream.
Before submitting

[x] Did you make sure to update the documentation with your changes?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Github is still showing the old date

Locally:

Github:

That's not a lot of details, so I'll be equally brief:

cannot reproduce
might be caching?
if you had given more details, I would try debugging things (ie do both commits show the same commit ID etc), but alas
if you want to debug this yourself, go to eg this url and check that both the date and the commit ID match your local data
???
profit
for plausible deniability, consider anything you've pushed public, so pushing first one date and then the other to github should always imply that the first date is public and known


Include the default exclusions for the detection rules

[Enter feedback here]
Some rules have default exclusions configured by default like the DNS: https://techcommunity.microsoft.com/t5/microsoft-defender-for-identity/default-exclusions-in-suspicious-communication-over-dns-sa/m-p/285348
Include in the documentation which rules have default exclusions shipped with the product

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: ab71dff0-18cc-3267-52ab-66f01986edb0
Version Independent ID: 2b6fe284-8481-ebb2-8d2e-4af0a91d968b
Content: Microsoft Defender for Identity configure detection exclusions
Content Source: ATPDocs/configure-detection-exclusions.md
Service: microsoft-defender-for-identity
GitHub Login: @dcurwin
Microsoft Alias: dacurwin

Thanks again for your suggestion. We've updated the documentation to mention those default exclusions. It should be visible shortly.


yum_package version property needs to be stricter about requiring the epoch

Description

yum_package don't keep idempotency if set version on Chef 15
Chef Version

Chef Client 15.3.14
Platform Version

CentOS 7
Replication Case

yum_package 'java-11-openjdk' do
  version '<IP_ADDRESS>-0.el7_6'
end

Client Output

Starting Chef Infra Client, version 15.3.14
       Creating a new client identity for test-centos-7-chef15 using the validator key.
       resolving cookbooks for run list: ["test::_test"]
       Synchronizing Cookbooks:
       Installing Cookbook Gems:
       Compiling Cookbooks...
       Converging 1 resources
       Recipe: test::_test
         * yum_package[java-11-openjdk] action install
           - install version <IP_ADDRESS>-0.el7_6 of package java-11-openjdk

       Running handlers:
       Running handlers complete
       Chef Infra Client finished, 1/1 resources updated in 50 seconds
       Starting Chef Infra Client, version 15.3.14
       resolving cookbooks for run list: ["test::_test"]
       Synchronizing Cookbooks:
       Installing Cookbook Gems:
       Compiling Cookbooks...
       Converging 1 resources
       Recipe: test::_test
         * yum_package[java-11-openjdk] action install
           - install version <IP_ADDRESS>-0.el7_6 of package java-11-openjdk

       Running handlers:
       First chef run should have reached a converged state.
       Resources updated in a second chef-client run:
       - yum_package[java-11-openjdk]
       bash: line 6: 14963 Killed                  sh -c '
       TEST_KITCHEN="1"; export TEST_KITCHEN
       sudo -E /opt/chef/bin/chef-client --local-mode --config /var/tmp/kitchen/client_no_updated_resources.rb --log_level auto --force-formatter --no-color --json-attributes /var/tmp/kitchen/dna.json --chef-zero-port 8889
       '

Stacktrace

You need to include the epoch:
#!/usr/bin/env chef-apply

yum_package 'java-11-openjdk' do
  version '1:<IP_ADDRESS>-0.el7_6'
end

results in:
# ./apply.rb
Recipe: (chef-apply cookbook)::(chef-apply recipe)
  * yum_package[java-11-openjdk] action install (up to date)

alternatively it works when including the version in the name:
#!/usr/bin/env chef-apply

yum_package 'java-11-openjdk-<IP_ADDRESS>-0.el7_6'

results in:
# ./apply.rb
Recipe: (chef-apply cookbook)::(chef-apply recipe)
  * yum_package[java-11-openjdk-<IP_ADDRESS>-0.el7_6] action install (up to date)

using the version in the name with yum_package is encouraged over using the version as the property, particularly around handling epochs.

[2019-09-13T17:39:26+00:00] TRACE: Resources for generic yum_package resource enabled on node include: [Chef::Resource::YumPackage]
[2019-09-13T17:39:26+00:00] TRACE: Resource for yum_package is Chef::Resource::YumPackage
Recipe: (chef-apply cookbook)::(chef-apply recipe)
  * yum_package[java-11-openjdk] action install[2019-09-13T17:39:26+00:00] INFO: Processing yum_package[java-11-openjdk] action install ((chef-apply cookbook)::(chef-apply recipe) line 3)
[2019-09-13T17:39:26+00:00] TRACE: Providers for generic yum_package resource enabled on node include: [Chef::Provider::Package::Yum]
[2019-09-13T17:39:26+00:00] TRACE: Provider for action install on resource yum_package[java-11-openjdk] is Chef::Provider::Package::Yum
[2019-09-13T17:39:27+00:00] TRACE: sending '{"action":"whatinstalled","provides":"java-11-openjdk"}' to python helper
[2019-09-13T17:39:27+00:00] TRACE: got 'java-11-openjdk 1:<IP_ADDRESS>-0.el7_6 x86_64' from python helper
[2019-09-13T17:39:27+00:00] TRACE: parsed java-11-openjdk-1:<IP_ADDRESS>-0.el7_6.x86_64 from python helper
[2019-09-13T17:39:27+00:00] TRACE: sending '{"action":"versioncompare","versions":["1:<IP_ADDRESS>-0.el7_6.x86_64","<IP_ADDRESS>-0.el7_6"]}' to python helper
[2019-09-13T17:39:27+00:00] TRACE: got '1' from python helper
[2019-09-13T17:39:27+00:00] TRACE: sending '{"action":"whatavailable","provides":"java-11-openjdk","version":"<IP_ADDRESS>","release":"0.el7_6"}' to python helper
[2019-09-13T17:39:27+00:00] TRACE: got 'java-11-openjdk 1:<IP_ADDRESS>-0.el7_6 x86_64' from python helper
[2019-09-13T17:39:27+00:00] TRACE: parsed java-11-openjdk-1:<IP_ADDRESS>-0.el7_6.x86_64 from python helper
[2019-09-13T17:39:27+00:00] TRACE: sending '{"action":"versioncompare","versions":["1:<IP_ADDRESS>-0.el7_6.x86_64","<IP_ADDRESS>-0.el7_6"]}' to python helper
[2019-09-13T17:39:27+00:00] TRACE: got '1' from python helper
[2019-09-13T17:39:27+00:00] TRACE: yum_package[java-11-openjdk] java-11-openjdk 1:<IP_ADDRESS>-0.el7_6.x86_64 needs updating to <IP_ADDRESS>-0.el7_6
[2019-09-13T17:39:27+00:00] TRACE: sending '{"action":"whatinstalled","provides":"java-11-openjdk.x86_64"}' to python helper
[2019-09-13T17:39:27+00:00] TRACE: got 'java-11-openjdk 1:<IP_ADDRESS>-0.el7_6 x86_64' from python helper
[2019-09-13T17:39:27+00:00] TRACE: parsed java-11-openjdk-1:<IP_ADDRESS>-0.el7_6.x86_64 from python helper
[2019-09-13T17:39:27+00:00] TRACE: sending '{"action":"versioncompare","versions":["1:<IP_ADDRESS>-0.el7_6.x86_64","1:<IP_ADDRESS>-0.el7_6.x86_64"]}' to python helper
[2019-09-13T17:39:27+00:00] TRACE: got '0' from python helper

Although it looks like there's some kind of wonky bug here since with that last line it looks like we've correctly done a versioncompare against the installed and available versions of the package and determined equality and its not at all clear why it then goes and decides to install the package (which is an "install" not and "upgrade" so why if we have anything that satsifies the "whatinstalled" check are we doing anything?
I'm guessing something went wrong with patching the superclass.

yeah when it works we get a match:
  * yum_package[java-11-openjdk] action install[2019-09-13T17:53:25+00:00] INFO: Processing yum_package[java-11-openjdk] action install ((chef-apply cookbook)::(chef-apply recipe) line 3)
[2019-09-13T17:53:25+00:00] TRACE: Providers for generic yum_package resource enabled on node include: [Chef::Provider::Package::Yum]
[2019-09-13T17:53:25+00:00] TRACE: Provider for action install on resource yum_package[java-11-openjdk] is Chef::Provider::Package::Yum
[2019-09-13T17:53:25+00:00] TRACE: sending '{"action":"whatinstalled","provides":"java-11-openjdk"}' to python helper
[2019-09-13T17:53:25+00:00] TRACE: got 'java-11-openjdk 1:<IP_ADDRESS>-0.el7_6 x86_64' from python helper
[2019-09-13T17:53:25+00:00] TRACE: parsed java-11-openjdk-1:<IP_ADDRESS>-0.el7_6.x86_64 from python helper
[2019-09-13T17:53:25+00:00] TRACE: sending '{"action":"versioncompare","versions":["1:<IP_ADDRESS>-0.el7_6.x86_64","1:<IP_ADDRESS>-0.el7_6"]}' to python helper
[2019-09-13T17:53:25+00:00] TRACE: got '0' from python helper
[2019-09-13T17:53:25+00:00] TRACE: sending '{"action":"versioncompare","versions":["1:<IP_ADDRESS>-0.el7_6.x86_64","1:<IP_ADDRESS>-0.el7_6"]}' to python helper
[2019-09-13T17:53:25+00:00] TRACE: got '0' from python helper
[2019-09-13T17:53:25+00:00] TRACE: yum_package[java-11-openjdk] java-11-openjdk 1:<IP_ADDRESS>-0.el7_6.x86_64 satisifies 1:<IP_ADDRESS>-0.el7_6 requirement
[2019-09-13T17:53:25+00:00] TRACE: yum_package[java-11-openjdk] is already installed - nothing to do
 (up to date)
[2019-09-13T17:53:25+00:00] DEBUG: Exiting

something is up with that versioncompare, not even sure what that is necessary when we're doing :install.

Ah I see.
When you specify it in the name with no version check then chef does not do any management of the version since it doesn't parse the version out of the name, that's just a label shipped to the underlying python, so there's no need to check the version.
When you specify the version in the version property you are telling chef what version to install.  The version comparison against the installed package is correct, it does not match since the epoch does not match.  We can't actually hack that up to do fuzzy matching.
We may need to make the version property stricter to always match epochs, and then provide better error messaging around the version property to force people to match epochs if the available package requires an epoch (although literally trying to do that will probably produce awful code, so we need a solution a bit more elegant than the design in my head right now).

TL;DR: This is more improper use of Chef than bugginess in Chef.  Users need to include epochs in version if the package has an epoch.  Chef should probably be stricter about enforcing the use of epochs and should probably error out rather than installing packages that have non-zero epochs.  When it errors out it should give a more useful error message that the user may have omitted an epoch and steer them better towards including an epoch in the version.  The need to include the epoch in the version is not itself a bug, that is the correct fix for users.

closing as a dup of #9965

I think this is closed by  #10823 I may have actually managed to fix this.


Update BYON-CallHandling-Service.yaml

Responding to the  #issue24 proposed by @deepakjaiswal1
We have added a  201 type response wich is returning a JSON object regarding the session parameters.
Feel free to add comments
@TEF-RicardoSerr
@pradeepachar-mavenir
@sushanthmavenir

@TEF-RicardoSerr @JavierVillaLabarra I believe it's unnecessary to include {status: 201, code: CREATED} in the responseBody for a 201 created response.

@JavierVillaLabarra I think we can omit the ObjectResponse schema. I would update Gereric201
FROM:
Generic201:
description: Created
content:
application/json:
schema:
$ref: '#/components/schemas/ObjectResponse'
examples:
SessionCreatedMT183:
$ref: '#/components/examples/exMT201_183'
SessionCreatedMT200:
$ref: '#/components/examples/exMT201_200'
TO:
Generic201:
description: Created
content:
application/json:
schema:
$ref: '#/components/schemas/ReceiverSessionStatus'
examples:
SessionCreatedMT183:
$ref: '#/components/examples/exMT201_183'
SessionCreatedMT200:
$ref: '#/components/examples/exMT201_200'

Hello @deepakjaiswal1 @pradeepachar-mavenir
Just made the new commit, please check if it is aligned with your thoughts


Always align canvas with Cytoscape canvases

I found this was needed to get the canvas aligned under Cytoscape's canvases when using a flex HTML layout. So far I've been unable to produce a small example demonstrating the problem, but the following screen screenshots show what happens. The first one is the desired result; the second when things go wrong.

I would really like to see the problem for myself before approving. Could you fork the demo and get an example of the problem. Thanks!

Hmm everything now works fine for me, so am closing the PR. I'm still not sure why things didn't work out of the box when I first installed the extension, and my attempt to get a MWE wasn't helped by me having changed two things (removing the absolute position certainly breaks positioning unless the background canvas is inside the inner <div>).
Thanks for your time.


Error: Cannot load from mysql.proc. The table is probably corrupted

Describe the bug
Hi, when I try to connect to my database I get the following error
Error: Cannot load from mysql.proc. The table is probably corrupted
In my opinion, an option should be included in the settings to skip control commands during the connection such as
SHOW FUNCTION STATUS
in fact if I try to connect to the same database with other clients like dbeaver or HeidiSQL the connection is successful
Application:

App client MySQL
App version 0.7.29
Installation source: Linux Store

Environment:

OS name: Ubuntu
OS version: 24.04.1 LTS

Duplicate of #159

Hi, thanks for the reply, I read that discussion, from what I understand it is enough to repair that table, but perhaps it is worth considering adding an option to skip those requests during connection, also because with other clients the same database works correctly.

Hi Salvatore,
from the next release these errors will no longer block the connection.

Ciao Fabio, grazie mille per l'implementazione 🖖🏻 sei un grande.
Colgo l'occasione per ringraziarti per l'ottimo lavoro che stai facendo, nell'open source mancava un client esteticamente bello e completo.


Accept scikit-bio OrdinationResults as input

code review idea (c/o @cameronmartino)

Yeah, this is done now


Registry: Reject any non-enrolled public key from registering

Part of #2289
Will follow up with a Flash related PR.

So I assume you don't have a use-case for a standalone registry ?

Yeah, I see no reason why we should have that. But, we either need to fix Flash to not rely on registry or have a seperate registry  hat validates open channels for Flash.

Rebased and squashed related commit (otherwise the CI wouldn't pass on the first and second commit).

@omerfirmak : Would it be complex to relax the restriction to allow not-yet-enrolled validators ?

@omerfirmak : Would it be complex to relax the restriction to allow not-yet-enrolled validators to register ?
This would allow a validator to ensure it's reachable from the second it becomes a validator.

We could also check the enrollment pool. I'll take a look.


document.querySelector

document.querySelector

This issue was imported from Google Moderator
Moderator votes: +3
Issue added by Tonait Weburn on 2012-04-18
To vote this issue up or down, simply include +1 or -1 in your comment.

Already added at http://caniuse.com/#feat=queryselector


Issue #39: Landing page content (membership splash, events list blocks)

Added membership splash for logged (only text) and non-logged (text and visual) users. (The action button is temporarily linked to the main page.)
Adaptive layout: trim too long title string with an ellipsis.
Removed unused 'onevent' templates.
Added can_register method to the Event data model, the 2nd events list on the main page: to show user's own event (where they already made a registration), data preparation for this second list. Utilized original events-list-block.django asset to display any kind of events list.
Updated version: 0.0.14.

Splash view for the logged user (text only):

Splash view for non-logged user (text and visual):

Adaptive layout (trim title on small screens):

Rebased conflicting main branch. Updated definition and usage of event.can_register.


Filter out virtual field names when performing mongo db.find with fields

includes pinning to mongodb 2.2.x driver series

Fixed in https://github.com/simpleviewinc/mongolayer/commit/08c22ceda0f5041798de2679e426139904255cf0 . Closing.


SOLR-15985 update mock-oauth-server version

https://issues.apache.org/jira/browse/SOLR-15985
This brings in a lot of other dependency changes, I'm actually a little scared to do this.

See #606 and SOLR-15907, where JWTAuthPlugin is moved to a module.
I ported over the other changes from this PR to that one, and think we can aim for moving the whole thing out of core at the same time.


Add puts to debug rate limiting

The header data doesn't appear to indicate the throttling about to occur.


Conditional on VirtualScroll causes no content to be displayed in any case

Ionic version:  (check one with "x")
[ ] 1.x
[X] 2.x
I'm submitting a ...  (check one with "x")
[X] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
Conditional on VirtualScroll causes no content to be displayed, even if there should be content. This is on rc4.
Expected behavior:
If there's a conditional, it should still work if the condition returns true. It used to work like this on rc2.
Related code:
html:
    <ion-list [virtualScroll]="dataStorage" [headerFn]="headerFn" *ngIf="dataStorage">

        <ion-item-divider *virtualHeader="let header" class="dividers">
            {{ header }}
        </ion-item-divider>

        <button ion-item *virtualItem="let data" (click)="openData(data.id)">
            {{ data }}
        </button>
    </ion-list>

ts:
    headerFn(record,recordIndex,records){
        if( recordIndex == 0){
            return;
        }
        return null;
    }

Other information:
If I take the same code and take out the *ngIf="dataStorage", it displays correctly.
Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
Cordova CLI: 6.3.1
Ionic Framework Version: 2.0.0-rc.4
Ionic CLI Version: 2.1.17
Ionic App Lib Version: 2.1.7
Ionic App Scripts Version: 0.0.47
ios-deploy version: Not installed
ios-sim version: Not installed
OS: Windows 10
Node Version: v6.7.0
Xcode version: Not installed

Having a similar issue although it was working for me back as recent as rc.3.

Well, I skipped rc3, hence why I only now saw it.

VirtualScroll with InfiniteScroll is more unreliable in RC 4 than it was in RC 3. It sometimes display images and sometimes it doesn't.  When InfiniteScroll loads the contents, VirtualScroll clears all contents from the screen, and just displays white area.  If we try to touch and scroll white area (up or down) it triggers InfiniteScroll and send request to server to fetch next page's record..

This issue appears for me with [ngSwith], as well.

I'm having the same issue.
Event surrounding the virtualScroll list in another div where I set the condition on doesn't fix the problem (yes, the condition is correct ;-))

virtualScroll does not work in segments in RC5.  As stated above [ngSwitch] is used as a conditional to change the content for the different segments.  The first shown segment when entering a page will display correctly, as soon as you navigate to another segment it's just a white page with no content shown.

#7734 - is related to this issue.  I have done a little troubleshooting and found something that might help the team to fix this:
attached are 2 screenshots, the first working.png is when I first enter a page that has a virtualscroll list inside a segment.  On that first load the segment shows the content correctly.
the second second screenshot shows when I navigate to a different segment and then come back to the original one.  It is now showing a blank white page instead of the list.  I can see that all of the list items are present in the DOM.
There is a clear issue with the CSS though:
working list has .ion-list.list.list-ios.virtual-scroll with a height value as it's first property
blank list has .ion-list.list.list-ios.virtual-loading with no height
working list has .virtual-scroll class with position: relative
blank list has .virtual-loading class with opacity: 0 and no position value.
seems like when there is a conditional virtual-scroll gets stuck in it's loading state and never transitions to being visible.

seems that when a conditional is in use this._init is not set so ngDoCheck() doesn't run and hence the virtualScroll list gets stuck in loading state.
If I edit the virtualScroll component and just remove this._init from the if statement in ngDoCheck() then the virtualScroll list is properly rendered when inside a conditional, although it does "pop" in as a result of this change.
ngDoCheck() {
if (this._init && this._hasChanges()) {
// only continue if we've already initialized
// and if there actually are changes
this.readUpdate();
this.writeUpdate();
}
}

I'm upgrading from rc.3 with a similar template as yours and it seems like this is still an issue in 2.0.0 final after playing around with it.

This is indeed still an issue for me in 2.0.0 final virtualscroll does not work inside ngSwitchCase

@jgw96 - for a final release there are still some really big bugs with virtualscroll, could they please get some attention?
Working with conditionals / segments is a key one, otherwise use cases are really limited.

This is still an issue more than 6 months after the bug was first raised.

fixed! https://github.com/ionic-team/ionic/commit/88b2e8316dad5c504695405fa2199aacbea5b5d6

@manucorporat:
I still have this issue in Ionic 3.9.2.
The problem seems to be that VirtualScroll isn't initialized because it listens to readReady/writeReady (ViewController), which only fires once(?) on page transition.
https://plnkr.co/edit/almnrjeRzA2zWq0S3KYj?p=preview

I still have this issue with 3.9.2. I can toggle between two virtualScroll lists that are built from the same array that are inside conditionals:
[<ion-list *ngIf="showList" [virtualScroll]="users" class="user-list" approxItemHeight="100px">
<ion-item tappable (click)="goToProfile(user)"  *virtualItem="let user"  class="user-list-item">

{{user.firstname}}, {{user.dateOfBirth | convertAge}}

{{user.city}}, {{user.state}}
{{user.looking}}

  <ion-list *ngIf="!showList" [virtualScroll]="users" class="user-grid" approxItemHeight="100px" >
    <ion-item tappable (click)="goToProfile(user)" *virtualItem="let user"  class="user-grid-item">
      <ion-avatar item-start class="user-grid-item-pic">

      </ion-avatar>

{{user.firstname}}, {{user.dateOfBirth | convertAge}}

{{user.city}}, {{user.state}}

    </ion-item>
  </ion-list>](url)

I opened a PR https://github.com/ionic-team/ionic/pull/13406 which should fix the problem here. Please try it and tell me if it's working :)

Is there a status update on this please?

@ryanhalley I made a PR #13406 but it's not merged yet. You can build ionic-angular by your own if you want, or download ionic-angular-3.9.2.tar.gz and replace it in your node_modules folder.

I made an update of my build as there was an unhandled edge case when VS is recreated after being destroyed... You can find it here: #13406

fixed:
@ViewChild('virtualScroll', { read: VirtualScroll }) virtualScroll: VirtualScroll;
setTimeout(() => {
this.virtualScroll['_ctrl'].readReady.emit();
setTimeout(() => this.virtualScroll['_ctrl'].writeReady.emit(), 300);
}, 300)

to get around this I change around the virtualScroll array

@442623641 Nice one! I just put this into a "refreshVirtualScroll" function and call it when the conditional changes and all behaves more like it should.  I dropped the timeout values down to 100 as well and still all looks good with a much less noticeable delay.
Hopefully this is no longer needed when ionic v4 arrives or the PR above gets merged for V3 so this isn't even needed any more.
For now though I can finally use virtualscroll inside a conditional.

A quick fix for now -
 @ViewChild(VirtualScroll) virtualScroll: VirtualScroll;
virtualScrollHack() {
this.cdRef.detectChanges();
this.virtualScroll['_ctrl'].readReady.emit();
this.virtualScroll['_ctrl'].writeReady.emit();
}

Can confirm that this bug still exists on Ionic Framework Version 3.9.2. Sadly, PR #13406 wasn't merged into master, because of significant master changes introduced by the porting of all of the Ionic components to web components...
@HugoHeneault
Could you refactor your solution and submit a new PR? pretty please O:-)

A quick fix for now -
 @ViewChild(VirtualScroll) virtualScroll: VirtualScroll;
virtualScrollHack() {
this.cdRef.detectChanges();
this.virtualScroll['_ctrl'].readReady.emit();
this.virtualScroll['_ctrl'].writeReady.emit();
}

What is this "this.cdRef" ??

@jayordway
its ChangeDetectorRef

@ShannonHall it doesn't work for me


scrape: Avoid creating a label map during conflict resolution

It was a little brain teaser, but suddenly a beautiful solution appeared on my screen. (o:
This also avoids the recursive function call.
@shirleyleu what do you think?

Really nice. That's much clearer!

Thank you everyone.


Add save button

Add save button to the card and the function that should execute when clicked.

save button added to card; currently non-functional. Development is still in progress.


Enhancement: avoid BE OOM in push task

Proposed changes
close #15572
Problem summary
Release memory pool held by the parquet reader when the data has been flushed by rowset writter.
Some test results:
1.unoptimized version
peak memory usage percent:  25%

optimized version

peak memory usage percent:  5%

Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[x] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[ ] No
[x] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[x] No Need

Does it need to update dependencies:

[ ] Yes
[x] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[x] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 36.5 seconds
load time: 647 seconds
storage size:<PHONE_NUMBER>5 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230104152925_clickbench_pr_73601.html


Chef-13: shell_out PATH fixes and path_sanity changes

Turns out polluting PATH with a global path_sanity actually tends
to cause problems via picking up stuff from e.g. embedded/bin
when users don't want it.  This change unsets the default setting
to add path sanity.

We already have the difference between internal and external uses
of shell_out, so what this does is take the internal use of
shell_out and always apply the PATH -- so anything calling
shell_out gets path_sanity.

It also modifies path_sanity to prepend the ruby bin and the
embedded bin into the PATH.  When we need those for internal use
we really want those first.


Unloading a world will sometimes put people into a restricted world.

For example if we have 4 worlds:
Lobby
Build
Survival
Secret
If we unload survival sometimes people who join will relog into Secret world, even though they shouldn't have access to it. Is there a way to fix this/prevent it?

Hmm. I thought there was a "default" world, that people got placed in if something else went wrong. Is your Secret world your first world on the server?

It honestly seems random what world they're place in sometimes it's normal worlds. We have a lot more than the example.


Intellisense really slow and taking lots of memory

Issue Description
While working on https://github.com/OrchardCMS/OrchardCore
The intellisense starts working only like 5 minutes after the working folder has been opened.
Also, Omnisharp memory usage indicates that I'm using 3 gb approximately while I'm just opening VS Code and not debugging anything. The memory usage seems to be related with loading document symbols for the intellisense to work. Omnisharp memory usage goes from 1 gb to 3 gb before I can see intellisence working.
I'm running this on a intel Core i7 with a NVME drive.
Will efforts done on this PR https://github.com/OmniSharp/omnisharp-vscode/pull/3817 will fix the issue eventually?
Environment information
VSCode version: 1.49.1
C# Extension: 1.23.2
Dotnet Information
SDK .NET Core (refl�tant tous les global.json)�:
 Version:   5.0.100-rc.1.20452.10
 Commit:    473d1b592e
Environnement d'ex�cution�:
OS Name:     Windows
OS Version:  10.0.19042
OS Platform: Windows
RID:         win10-x64
Base Path:   C:\Program Files\dotnet\sdk\5.0.100-rc.1.20452.10\
Host (useful for support):
Version: 5.0.0-rc.1.20451.14
Commit:  38017c3935
.NET SDKs installed:
3.1.100-preview1-014459 [C:\Program Files\dotnet\sdk]
3.1.402 [C:\Program Files\dotnet\sdk]
5.0.100-preview.8.20417.9 [C:\Program Files\dotnet\sdk]
5.0.100-rc.1.20452.10 [C:\Program Files\dotnet\sdk]
.NET runtimes installed:
Microsoft.AspNetCore.All 2.1.12 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.All 2.1.21 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.All 2.2.6 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.All 2.2.7 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.App 2.1.12 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 2.1.21 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 2.2.6 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 2.2.7 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 3.1.2 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 3.1.7 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 3.1.8 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 5.0.0-rc.1.20451.17 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 2.1.12 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 2.1.21 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 2.2.6 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 2.2.7 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 3.1.2 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 3.1.7 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 3.1.8 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 5.0.0-rc.1.20451.14 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.WindowsDesktop.App 3.1.2 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 3.1.7 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 3.1.8 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 5.0.0-rc.1.20452.2 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
To install additional .NET runtimes or SDKs:
https://aka.ms/dotnet-download

Visual Studio Code Extensions

Extension
Author
Version

csharp
ms-dotnettools
1.23.2

remote-wsl
ms-vscode-remote
0.44.5

vscode-docker
ms-azuretools
1.6.0

dupe of https://github.com/OmniSharp/omnisharp-vscode/issues/3421

Thanks I agree it's a duplicate but any answer on my question would have been appreciated.

sorry @Skrypt, I missed your question. As with anything performance related it is unwise to speculate, but here I go.
In general, we should have memory usage scale linearly with the number of projects/documents. We will always load and evaluate each file to build up a cache so things like find-all-references work quickly. Because of this design I would always expect some degree of CPU and memory useage spikes when a project or workspace is first opened regardless of how we improve performance. In your case the number sound rather excessive and we should be able to provide completions even with a partially completed model. However I would not expect the general design of open workspace -> compute and cache results -> update results incrementally to change.
This issue https://github.com/OmniSharp/omnisharp-vscode/pull/3817 is unrelated in my opionion. Once a everything is loaded, on each keystroke we send the entire contents of the text buffer to the omnisharp process. This has scalability issues but I would only expect them to show up after everything is loaded. If the file you are typing in is very large, then that may be the root cause but you are talking about it taking a long time to update after open. I assume things are not slow after that?

Once the Omnisharp process has completed processing everything works like expected. Though if I open up a different .cs file then it starts building some cache again and I'm waiting another 5 minute. So, it's basically non-usable for us in this state. On the opposite it seems to work fine on Visual Studio Community edition so I'm moving from one editor to the other lately because both have their caveats.

In your case the number sound rather excessive and we should be able to provide completions even with a partially completed model
I expect memory usage to scale linearly too but I'm surprised that I get delayed results like this. Should this not try to build a cache of the things we ask manually for first?

Could you point me in the direction of the code that we are talking about?


add DepthPrepass when DeferredPrepass is used

Objective
fix crash / misbehaviour when DeferredPrepass is used without DepthPrepass.

Deferred lighting requires the depth prepass texture to be present, so that the depth texture is available for binding. without it the prepass node doesn't run and nothing will be rendered.
When DeferredPrepass is used without other prepass markers, and with any materials that use OpaqueRenderMode::Forward, those entities will try to queue to the Opaque3dPrepass render phase, which doesn't exist, causing a crash.

Solution
add DepthPrepass marker to views with DeferredPrepass.
also changed some With<T>.is_some()s to Has<T>s

I think this definitely could make sense. Worth nothing though that this will create a discontinuity between the which prepass components the camera has in the main world vs the render world if the user only adds the deferred prepass.

i've been on a bit of a journey with this...
forcing the depth prepass whenever deferred is present is not ideal: if a scene has forward materials and deferred materials it forces the depth prepass to run for the forward materials even if not requested/required.
i thought maybe it would be better to make a new marker DepthPrepassTextureRequired and add it when DepthPrepass or DeferredPrepass are present. but, maybe somebody will implement a deferred lighting plugin that doesn't require the depth texture, so adding/copying the depth texture unconditionally still doesn't make sense.
also, dealing with shader defs gets a bit tricky: the prepass depth texture won't be populated for forward materials, so the def should not be present in the main pass for forward materials. but the view BindGroupLayout still needs to contain the corresponding entry to match the BindGroup, which will have the texture. currently both the layout and the def are based on MeshPipelineKey::DEPTH_PREPASS, and adding an extra key bit for "DEPTH_TEXTURE_IS_IN_THE_GROUP_BUT_SHOULDNT_BE_AVAILABLE_TO_THE_SHADER_BECAUSE_IT_WONT_BE_POPULATED" seemed like overkill.
so instead i just fixed the crash (we now check if the xxx3dPrepass phase exists before trying to queue to it), and added a note to the DeferredPrepass comment that it requires DepthPrepass as well. I also changed the condition on prepare_prepass_textures so the deferred textures are still created if only DeferredPrepass is present. this means the original issue of running depth prepass for forward materials is still present, but at least it doesn't crash, and if somebody wants to make a custom deferred pass that doesn't use depth then the DepthPrepass doesn't need to be added.

@robtfm I think this needs rebasing or something, the diff seems weird.


Move Logging docs

This MR includes simple move of old logging documentation. All code links were changed and now point to new repo which will be replaced after docs is finished.

@ilammy Thanks for your comments. Appreciate it too much:)


send_direct_message does not work anymore

Since twitter has changed the way the Direct Message API works on September 18th, all the functions related to direct messages in this module have turned obsolete.
I managed to get a hold of the new get_direct_message function from this fork : https://github.com/jvmaia/twython
But the send_direct_message behaviour has changed completely. It now requires:
type: 'message_create'
message_create.target.recipient_id: 'xxx'
message_create.message_data: 'xxx'
Would be very grateful if someone could help me change the POST request according to the sample post request which is provided on the Twitter Documentation here

Please see #500. @michaelhelmick haven't answered so far, unfortunately.

I got it to work by hard-coding the POST request and creating a new def post_message in the api.py
Here is the code for your reference:
def post_message(self, endpoint, params=None, version='1.1', json_encoded=False):
        """Shortcut for POST requests via :class:`request`"""
        return self.request('direct_messages/events/new', 'POST', params='{"event": {"type": "message_create",
	"message_create": {"target": {"recipient_id": "111111111"}, "message_data": {"text": "Test"}}}}',
	version=version, json_encoded=json_encoded)

Now the params is in the form of a string, so if you want to dynamically assign the recipient  ID and text  you can just  use '....'+myVariable+'...'


Create user rake task fails

When trying to create a user in single tenant mode, the rake task create_user.rake for performing this operation fails at the password option, as a result of an attempt to call a method that does not reveal user input on the command line.
This PR fixes the issue: https://github.com/DefactoSoftware/Hours/pull/465

I assume this has been fixed and can be closed.


Docker volume for /var/lib/rabbitmq

A new volume with a generic name gets created everytime you start the dev or other environment with docker. This volume contains files for RabbitMQ. https://hub.docker.com/_/rabbitmq states:

This image makes all of /var/lib/rabbitmq a volume by default

var/lib/rabbitmq contains RabbitMQ's internal database among other things.
This PR inserts a volume statement for docker-compose so that the same volume is used every time the container is started. Otherwise messages, that have not yet been consumed, could get lost with a restart.

Should we do the same for the k8s setup?

The Bitnami RabbitMQ chart has persistence enabled per default. In my installation RabbitMQ is a stateful set with a persistent volume. So I assume for k8s it is already done.


make a failing docker build

Summary by CodeRabbit

Chores

Updated Docker workflow to allow manual triggering and simplified job structure.
Modernized action versions for Docker setup and checkout processes.
Changed base image in Dockerfile from Alpine to Debian and explicitly defined package installations.
Removed redundant commands related to application execution and configuration in the Dockerfile.

@alexcrtestapp full review


artifact-server: add the ability to create and replace artifact contents

Currently artifact-server makes it easy to get JSON versions of protobuf-encoded artifacts. Can we extend this to also set artifacts from JSON? This might eventually be directly bundled into the core server for known types.

I think that we now prefer to replace this with YAML artifacts: https://github.com/apigee/registry/issues/946


resolving centos errors on travis

Description
resolves failing tests for centos due to missing attributes
Issues Resolved
Check List

[ ] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[ ] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable
[ ] The CLA has been signed. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

Looks like CentOS 6/7 and Ubuntu 16.04 are still broken with this change.

Also is this a dupe of #38

Going to fix ubuntu separately as that's going to require some service changes. As a bunch of OS's  currently broken in 'master' branch going to release this.


Update thread-pools.adoc

Did some small language tweaks

Can one of the admins verify this patch?


[MM-15755]  fix old links to use the new urls in system console

Summary

fix some old links in the en.json to use the new admin console links
update tests
fix hardcoded strings in the code

Ticket Link
jira: https://mattermost.atlassian.net/browse/MM-15759

Perhaps it could be a good idea to set up redirects for the old URLs in case there's anything else that we've missed? 0/5 on that though if we're sure we've gotten all the URLs

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: https://i-05e215ef62c980d06.test.spinmint.com
Test Admin Account: Username: sysadmin | Password: sysadmin
Test User Account: Username: user-1 | Password: user-1
Internal IP: <IP_ADDRESS>
Instance ID: i-05e215ef62c980d06

I verified on the spinmint that the cases listed in the ticket are fixed. 👍

Do we still need to do a sweep through all the repos to look for other cases?
Do we want to consider redirects like Harrison said, just in case?

It is not my call, however we can merge this and we can have a followup

Spinmint test server destroyed

cherry picked 21c0667e43b695d583b7d21f5aa5f49bdd60feee into release 5.12 branch


Match benchmark iOS project bundle identifiers

Description
Match benchmark Info.plist bundle identifiers to their project settings.  Don't reuse complex layout ID.

Not confident this will fix https://github.com/flutter/flutter/issues/65734, but landing on red to see!


Make manifest.json accessible

https://commafeed.com/manifest.json is not accessible currently
Not tested locally honestly because I don't own an Android phone with Lollipop. Also this probably is interesting: https://youtu.be/vl4-WMImj6I?t=88

thanks!


perf: use bumpalo for BVH

Looks to be small% faster or so compared to https://youtu.be/2dGtW7E0B3U?si=BziIclI4EfJi81SC

closing this for now. might re-impl in future


Custom hotspot always change the current scene with tour

Hi, @mpetroff
Thank you for your great open source project :)
I want some hotspots in tour without changing scenes.
But whenever I click one of custom hotspots in tour, the current scene will always change event if the custom hotspot does not have a 'scene' type.
How can I create a hotspot without changing scenes?

I've figured out the problem.
The problem doesn't have nothing to do with type but sceneId.
When every hotspot that has sceneId gets clicked, then the scene will move to the scene with the sceneId.


redshift source fails on copy command

INFO:  Table "customers" does not exist and will be skipped
INFO:  Load into table 'customers' completed, 100 record(s) loaded successfully.
Error: db error: ERROR: syntax error at or near "("

i believe this is redshift not supporting COPY FROM (x)

Yes. I'm remerging this with #26, and I'm going to tackle all the RedShift stuff together early next week, as discussed there, probably using a quick and dirty Avro converter for loading and CSV for dumping.


Basic gameplay features

Functionality needed:

Shooting basic projectiles and health systems
Killboards / players online UI
Implement bot behaviours

Need to make some dynamic behaviour switching

Stage 1: Basic movement and target locking

Default for bots is to be on patrol
If a player or cargo ship is in range, pursue
If a structure is in range and currently pursuing, break off pursuit and return to patrol location and pattern
Players and bots have their own target lock mechanics

Stage 2: Combat

If a player or structure attacks the bot, it flees for a few seconds and then doubles back to attack

If the player is in range, it will resume the fight
If the player has left the area, it will return to it's patrol

Behaviour probably needs to be part of the game/ folder so it can be on the server

Out of scope:

Implement Pirate aircraft and aircraft selection screens
Missions and objectives

Acceptance Criteria:

[x] Bots move around the map using automatic pathfinding
[x] Player can target lock bots
[ ] Mini map
[ ] Bots change behaviour based on others in proximity
[ ] Players and bots can shoot at, damage and destroy each other

#32 - Basic Gameplay

Just noticed there's no http server built into dev... never mind it was on https
Exporting a pirate model from Blender, could be good to add a thruster to it..

Never mind it looks wrong aesthetically and the file size is rough

Checking Yuka documentation for what I need to do for basic bot behaviours, looks like the options are quite API-ey so I need to think about what I want first and then figure out what parts of the spec I need to use
Brainstorming bot requirements. There are two sets of bots I want to build at this stage

Freight ships that travel between the structures

Could be fun to make them run away from attack so it's a game of cat and mouse to get ships to their destination

Pirate ships that are trying to raid the freight ships and attack players who approach them

A timer that spawns in the pirates initially and replacements
Randomise the spawn location between a couple of pre-set spots
They should target the nearest freight ship when spawned, fly towards it and begin attack
If a player approaches while they are engaging cargo ships, the player is ignored
If a player attacks them while they are attacking the ships, they switch targets to the player
If they are not near a freighter yet and the player intercepts them they will immediately aggro and target the player

Renaming "Ship" class to Player so that it makes more sense alongside the incoming Bot class

Testing, looks good

Reviewing Yuka documentation, looks like there's quite a lot to do in order to implement the behaviours
Adding the Raven model to the scene within the Bot class

Implementing the Wander steering behaviour so the 3D model just flies around at first

It's moving but very slowly and I can only seem to get it going backwards..
Fixed it up, it needs to be restricted to boundaries somehow
The camera isn't able to track the ship, need to look into that

#32 - Basic Gameplay

Checking project board to make sure I don't give myself tasks out of scope, looks good
Finishing off very basic movement AI for enemy bots

Restricting wandering to boundaries... tried to get ChatGPT's help but it has no idea what valid functions are for Yuka, it's hallucinated calls to non existent methods and I'm not sure if it's worth implementing them in case there's a better way to do this within Yuka.

Refactoring my implementation so Entity Manager sits further up in the app

Comparing time module to my delta, looks like I'm sending in the application uptime not a delta... investigating. The animation loop is just passing along currentTime which is passed in by requestAnimationFrame itself.

Trying to use ObstacleAvoidanceBehavior but I'm not sure it's compatible with wandering behavior

Implemented path following and now it seemingly won't need the obstance avoidance, might leave it with this for now

Fixed camera tracking issue, gave it an exemption for the vehicle entity from YUKA
Implementing cargo ship AI so they travel between structures

Reviewing possible improvements to the Bot/Cargo Ship/Player classes topology, but it's a bit unclear what it should be. Maybe "Entity" and "Scenery" at some point?
Adding code from Bot to the cargo ships, it all looks in order but it's not moving
Tried Wanderer Behavior also no go... looks like it was the MatrixAutoUpdate = false requirement, going to have to rememeber that one!

#32 - Basic Gameplay

Checking how the View/Edit buttons work, their commands are embedded in the UI template
Object Editor is currently not doing anything
Adding paths, the cargo ships are definitely not following the paths sigh

Gameplay #32

Implementing Overlays class to draw flight related overlays

Adding sensors class with initial hardcoded target of the bot

Having issues accurately scaling the overlay in 3D space, I believe I already have it's coordinates in 2D the issue is that I'm currently drawing a diamond in 3D space.

Trying browser side overlay to see if current 2D coordinates are accurate.. they are!

There's a weird ghost that appears but that's OK.

TODO:

Fix accuracy issues with scanner overlay
Players flight instruments overlay

Comment out current flight instrument and replace with a Tweakpane debug view for now

Langenium #32

Updated readme
Set ticket acceptance criteria
Merged WIP branch

#32 - Targetting UI

Checking past references for targetting UI, best bet is to just salvage the old icons
Creating icons for all scenery objects so they can be selected with the targetting scanner
Adding a targetting scanner area to the UI along with weapon launch icons

Scaffolding the new class under UI, overlays can stay in scenograph for now but it might make sense to move them out given they're HTML based
Currently the overlays display automatically even before the game starts, but the scanners should only appear on selected and targetted objects.

#32 - Targetting UI

Adding tray that targeted objects will be added to
Cleaning up overlays scanner class so I can get it to be dynamic and link it up with the UI targetting scanner class

Langenium #32 - Targetting UI

Thinking about how to do the target list VS locked targets
A table with a list of detected objects plus a list of locked targets like EVE Online would make sense but I don't think the mechanics being the same would make sense

Given players can free fly in Langenium, the target lock should be involve some kind of tracking or flight manouvering.

Langenium #32 - Targeting UI

Refactoring targeting UI to use PUG based templates
Updating distance of targets dynamically
Reducing UI update interval
Tried a refactor of the objects in current scene but the impetus to do so made less and less sense as I went down the rabbit hole so it's been rolled back. Turns out attaching objects to current_scene does make more sense than holding them in memory
Adding cargo ships to targeting scanner UI

Adding new metadata to meshes to track icon lookup keyname and standing with player to determine colour
Tweaking the UI and adding hover effect using CSS

Adding full list of targetable scene objects

Checking how scene overview works, might need to add a flag as I don't want the skybox and other scene objects in the list
Adding a 'targetable' flag to all scene objects
Refactoring the existing Targeting Scanner UI class and templates to be a sub component called Locked alongside List within Targeting.

Langenium #32 - Targeting UI

Adding pug-stylus to the repo to tidy up template styles

Getting some hectic console errors but the client is compiling and working OK
Deleting docs folder and fully rebuilding, came back great.
Ah, I had debug on for pug. Turned off, restarted and it's all good again.

Langenium #32

Prototyping the look and feel of the sensor list

Langenium #32

Prototyping the look and feel of the sensor list

Moving game client styles into a stylus file and refactoring for extension/maintainability
The sensor list is looking great, it's ready to be made dynamic

Adding full list of targetable scene objects

Referencing existing target lock code

Langenium #32

Adding remove and info buttons to the active targets
Adding HTML for a active target window
Fixing issues with client size and touch controls using wrong canvas element for their positioning
Adding icons for other classes and replacing ship icons with their targetting overlay symbols
Discussing with Garrett, this needs a rethink. There's too many clunky UI elements

Langenium #32 Targeting

Reviewing where we're at to remove the new UI's and replace with cleaner inline approach
Commenting out the new overlays
Shrinking the targeting reticule to be 25%of screen instead of 65%
Making targetables change to yellow when in the targetable zone
Tweaking colours and fixing div structure

Langenium #32 Targeting

Implementing targeting effect
Updating ticket
Minor tidy up and refactor
Merging PR #47

Langenium #32 Mini Map

Adding static HTML container with background colour
Aligning all the overlay colours and refactoring styles
Adding minimap class and player's FOV shape to the minimap

Adding icons
Replaced FOV shape with chevron

Adding portrait mode to HUD
Fixing game control positions after resizing, actually this needs more work because they need to debounce
Disabling overlays before game mode is active


Updated README.md

sudo su would do two nested processes, one for sudo running su and one for su running the root shell.
Instead, you can use sudo -i (where -i stands for interactive) and it would pop a root shell directly.

HI @antranigv. Thanks for the hint.


Review I/O of tasks

I'm reviewing the methods for task execution (in view of #261), and I (re)-discovered a somewhat cumbersome way we use to define I/O parameters of tasks. This was probably implemented with a clear target in mind (the typical image->zarr conversion), but it doesn't sound fully right any more.
--
The execute_tasks function (streamlined in #780, so that it's also a bit more readable now) iterates over a list of wftasks. For each wftask, either call_single_task or call_parallel_task is called (and waited for), and then the loop continues. The main I/O structure of these functions is defined in the task_pars argument and in the returned object, see
def call_single_task(
    *,
    wftask: WorkflowTask,
    task_pars: TaskParameters,
    workflow_dir: Path,
    workflow_dir_user: Optional[Path] = None,
  ) -> TaskParameters:
    # ...

Those are two TaskParameters objects, where
class TaskParameters(BaseModel):
    input_paths: list[Path]
    output_path: Path
    metadata: dict[str, Any]
    # ...

We should then review how the I/O TaskParameters objects are constructed for each wftask, and especially for

The very first task (which should get some info from the workflow input dataset);
Any intermediate task;
The final task (which should get some info from the workflow output dataset).

Tl;DR The current way these objects are constructed does not seem robust, and was probably fine-tuned to work only on the typical Fractal workflows.
First wftask
Input parameters
The definition of what will be passed as an input TaskParameter object to the first task is (e.g.) in app/runner/_local/__init__.py:
            task_pars=TaskParameters(
                input_paths=input_paths,
                output_path=output_path,
                metadata=input_metadata,
            )

where the input_paths, output_path and input_metadata values come directly from the DB and they are obtained (in app/runner/__init__.py) as
input_metadata=input_dataset.meta
input_paths = input_dataset.paths
output_path = output_dataset.paths[0]

There is something clearly suspicious here: the first task already needs some output_dataset properties, even if output_dataset is meant to be the output of the whole workflow.
Output parameters
At the end of call_single_task (and the same holds for call_parallel_task), there is this definition of the return value
    # ...
    out_task_parameters = TaskParameters(
        input_paths=[task_pars.output_path],
        output_path=task_pars.output_path,
        metadata=updated_metadata,
      )
    return out_task_parameters

This is strictly enforcing a task structure where every subsequent task will only act on a given path, both for input and output.
Also: this shows why we only support multiple paths as the input of the very first task (which essentially means only the create_ome_zarr_multiplex task).
Intermediate wftask
An intermediate wftask (starting from the second one) will always have an input TaskParameter object which was the output of a previous one, and then by construction it can only have input_paths and output_path identical (apart from the former being cast to a list).
The same holds for the output TaskParameter object, which will look like the output of the first wftask described above.
Last wftask
As should be clear by what is described above, the last wftask is not different from any intermediate wftask, because the output_dataset was already used in the very first task.
Notes and questions

What kind of workflows are actually supported?
If I'm getting it right, we only support a workflow such that:

The first task is somewhat flexible:

It can take N input paths (but it always has a single output path)
The input paths can obviously be different from the output path, meaning that the task has an input dataset A and an output dataset B.

Any other subsequent task must be a B->B task

Why does it currently work, for typical image->zarr workflows?
This use case perfectly matches what is described: the first task is the one that goes from the input (image) dataset to the output (zarr) dataset, and all the others keep using the output (zarr) dataset.

Why does it currently work, for a zarr->zarr workflow?
This use case always has the same (zarr) dataset both as an input and as an output, so that it's not affected by the current I/O definitions.

Can we make a clear example of a workflow that would fail because of how we define I/O parameters?
The following workflow is not supported:

Consider a single-path dataset (although adding the N-paths input dataset would make things even more complex)
Run some trivial pre-processing of images (e.g. with the compress_tif task - if it were available)
Run the typical Fractal workflow (create_ome_zarr and then yokogawa_to_ome_zarr).

Where to move from here?

Here's a rough scheme of how things are working now

Some other thoughts coming up:

The current version does allow some kind of flexible workflow execution, as was implemented in https://github.com/fractal-analytics-platform/fractal-server/issues/783. That is, if we have a "supported" workflow, then it's easy to split it into two sub-workflows which are still supported - and executing one sub-workflow at a time is not a problem.
I'm not saying that the current limited scope of supported workflows is the actual blocking factor for supporting use case 3 (namely restarting execution from a failed task - see #783 or https://github.com/fractal-analytics-platform/fractal-server/issues/261#issuecomment-1559130894). I'm rather saying that it doesn't sound right to build additional features on top of this restrictive structure. To elaborate further, what I have in mind is that if a task fails then we have to "cut" some connections in the diagram above, and store (where? that's another question) the information of what was the last task that went through. And for instance it would be very weird to handle differently the cases where the first task fails or an intermediate task fails.

Slightly more precise version:

This issue in the current form is made obsolete by ongoing V2 work.


[extractor/patreon] Fix and improve extractors

Template

Before submitting a pull request make sure you have:

[X] At least skimmed through contributing guidelines including yt-dlp coding conventions
[X] Searched the bugtracker for similar pull requests
[X] Checked the code with flake8 and ran relevant tests

In order to be accepted and merged into yt-dlp each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[X] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[X] Fix or improvement to an extractor (Make sure to add/update tests)
[ ] New extractor (Piracy websites will not be accepted)
[ ] Core bug fix/improvement
[ ] New feature (It is strongly recommended to open an issue first)

Description of your pull request and other information

DESCRIPTION

Use Patreon mobile user agent, which in my testing it seems to bypass Cloudflare captcha - Fixes https://github.com/yt-dlp/yt-dlp/issues/3631
Raise useful error messages when there are no formats  - Fixes https://github.com/yt-dlp/yt-dlp/issues/2914
Support m3u8 post file videos - Fixes https://github.com/yt-dlp/yt-dlp/issues/2277
Added post comments support for a laugh
Renamed PatreonUserIE to PatreonCampaignIE since user is something different to a campaign on patreon
Extract more useful metadata for PatreonCampaignIE playlist
Support alternative m/<campaign_id> campaign urls (also bypasses webpage download if that ever becomes a problem)

TODO

[ ]  I'm not really a fan of hard-coding the header when we can't override it with --add-header or --user-agent. Should we change the core behaviour of this?
[ ] Improve API error handling to cover more

Could also look into adding:

[ ] Login support
[ ] Cloudflare captcha detection (for useful error message)
[ ] Parse post filters in url (from the interface)

I'm not really a fan of hard-coding the header when we can't override it with --add-header or --user-agent. Should we change the core behaviour of this?

I don't think so. If extractors are setting UA, it must be for a good reason. Users passing other UAs will just unnecessarily break such extractors. If we ever implement a way for user to set the UA per-request/extractor, then this can be considered

You can merge when ready


Official ballot version

Intentionally a draft PR until we're ready to publish the official ballot version.

Still need to merge all open pull request in before this one.
This change will then freeze the official ballot version.


Confusing line in the this section of 2.0

Under the this section in 2.0 there is a line that says the following:
That means that this is of type Deck now, not any, so --noImplicitThis will not cause any errors.

Where is Deck even mentioned? It seems out of place.

Sorry that is wrong. should be fixed in https://github.com/Microsoft/TypeScript/wiki/What's-new-in-TypeScript now.


touch error

Hi, I have to face an error after I install the pod then copy-paste code into button click.i cannot click on any text field not working.clicking not working.

How were you able to resolve this issue

I'm facing the same issue. Who is able to resolve this?


Do not expand ${string} to string type

Suggestion

🔍 Search Terms

string literal expand
⭐ Suggestion
Currently if we will write
type A = `${string}`

type A will be expanded to string. It removes a constraint, that this type must be a string literal. I suggest to keep `${string}` `as is.
💻 Use Cases
It may be used for adding a string literal constraint for generics:
declare function a<T extends `${string}`>(literal: T): void

declare const b: string

// Currently: ok
// Expected: type error 'string' is not assignable to '`${string}`'
a(b)

Also we can tell type checker this way that we expect an object with determined keys.
declare function c<O extends Record<`${string}`, any>>(obj: O): void

declare const d: Record<string, any>

// Currently: ok
// Expected: type error
c(d)

Great suggestion!

I disagree with this, because `${string}` describes a string that is constrained to all strings, and therefore `${string}` = string.

This isn't a good fit for the reasons @ExE-Boss mentioned.

Wait, I'm confused.  Right now in TS4.1.5, you get the (declined) suggested behavior that string is not assignable to ${string}:
const a: `${string}` = "".toUpperCase() // error!
// Type 'string' is not assignable to type '`${string}`'.

Playground link
What am I missing?

@jcalz and all - this was the behavior in 4.1, but will change in 4.2+. This was discussed a few weeks back over here https://github.com/microsoft/TypeScript/issues/42644#issuecomment-773567965


AWS Load Balancer creates port 80 and 443 both with protocol SSL

Need to create AWS load balancer with SSL(443) and TCP(80) but creates SSL(443) and SSL(80)
A clear and concise description of what the bug is.
YAML file used to reproduce
apiVersion: v1
kind: Service
metadata:
  labels:
    service: ambassador
  name: ambassador
  annotations:
    service.beta.kubernetes.io/aws-load-balancer-ssl-cert: "arn:aws:acm:ap-southeast-1:123123123:certificate/443abcde-cfgh-klmn-6789-b9876ab01c67"
    service.beta.kubernetes.io/aws-load-balancer-ssl-ports: "*"
    service.beta.kubernetes.io/aws-load-balancer-backend-protocol: "tcp"
    service.beta.kubernetes.io/aws-load-balancer-cross-zone-load-balancing-enabled: "true"
    service.beta.kubernetes.io/aws-load-balancer-proxy-protocol: "*"
    getambassador.io/config: |
      ---
      apiVersion: ambassador/v0
      kind:  Module
      name:  tls
      config:
        server:
          enabled: True
          redirect_cleartext_from: 80
      ---
      apiVersion: ambassador/v0
      kind:  Mapping
      name:  frontend-web_mapping
      prefix: /
      service: frontend-web:3000
spec:
  type: LoadBalancer
  ports:
  - name: ambassador-http
    port: 80
    targetPort: 80
  - name: ambassador-https
    port: 443
    targetPort: 443
  selector:
    service: ambassador

Expected behavior
Need to either have a way to specify protocols for AWS Load Balancer listeners protocols or should specify protocols correctly
How do I make sure that the global redirection works from 80 to 443 and also listeners are created on load balancers with correct protocols and ports?

@krish512 Did you try service.beta.kubernetes.io/aws-load-balancer-ssl-ports: "443"?
Did you intentionally specify all the ports as SSL?

@dpankros I can confirm that service.beta.kubernetes.io/aws-load-balancer-ssl-ports: "443" solves the problem exposed by @krish512
Thanks!

Thanks this worked, but how do I create an AWS Application load balancer with this which supports HTTP2 rather than just a Classic load balancer supporting HTTP and HTTPS

@krish512 To my knowledge, you can only create ELBs (classic) or NLBs through kubernetes annotations.  Specifically, the value of service.beta.kubernetes.io/aws-load-balancer-type controls the type of load balancer.

If you don't specify a value, an ELB (classic LB) is created.
If you specify a value of nlb an NLB is created.
any other value will not be accepted.
Thus, for now, albs are not supported.  Maybe sometime in the future.

Of course, you could create one manually, but if ports or hosts ever change, you'd be left to manually update them which, IMHO, is just asking for trouble.

Using the same issue for a more queries on similar lines,
I was able to create HTTP and HTTPS listeners, now I need a third listener which is TCP on port 3306 for Database. How do I achieve this?

Did you try adding another port to your service?  I believe the ELB listeners get generated from that.  If you need it to be an ssl/tls listener ssl, you'll also need to add that to the annotation (e.g. "443,3306").
This isn't really an Ambassador question, however.  I recommend you consult the Kuberenetes Documenation on this topic for more information.

Well, that's when I'm creating a different load balancer, I want to use existing ambassador load balancer for this
Closing this issue


Add displayName

Add displayName

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Hasanthi Dissanayake seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


fix: Customize Theme was showing wrong example for color

This PR is fixing a documentation error specifically for the Button component. As we don't have any color option called as  Primary

Customize Theme - Option 3
the below is the fix of the documentation with the correct color option in the custom theme option.

Summary by CodeRabbit

Documentation

Updated the custom theme documentation to reflect the renaming of the color key from "primary" to "failure" for button styling, enhancing semantic clarity.

I also have this issue... need to manage all colors manually... 🥲 @SutuSebastian

There is work planned to introduce the primary color in tailwind.config.(js/ts/cjs/mjs) file, so I'm just gonna leave this here for now.


[#4110] Correct javadocs of MpscLinkedQueue

Motivation:
The javadocs are incorrect and so give false impressions of use-pattern.
Modifications:

Fix javadocs of which operations are allowed from multiple threads concurrently.
Let isEmpty() work concurrently.

Result:
Correctly document usage-patterns.

LGTM

:+1:

Cherry-picked into 4.0 (b960939ea04a31c9b273ebc46b34384fcdaf58b9) , 4.1 (eb1c97b3b99bca59971809260ad2977d5e5c886b) and master (15de021a867b862918e68955310876a899b4003a)


New Landscape Entry - Salus: Risc-V hypervisor for TEE development

Project Name
Salus: Risc-V hypervisor for TEE development
Description
Salus is a RISC-V 64 bit hypervisor used for TEE development. It is used to implement the TEE Security Manager (TSM) and the ABI proposed as part of the CoVE (Confidential VM Extension) proposed in the RVI AP-TEE (non-isa) Task Group. It will also be used to used to interface with any privileged ISA extensions developed as part of that TG (currently being discussed).
Homepage URL
https://github.com/rivosinc/salus
Repository URL
https://github.com/rivosinc/salus
Organization hosting or owning the project
Rivos Inc.
Evidence of RISC-V ecosystem support
AP-TEE CoVE specification: https://github.com/riscv-non-isa/riscv-ap-tee/tree/main/specification
Logo
SALUS
Category
Hypervisor

@bprestonlf +1.

+1


Identify duplicated foreign key constraints by prefix

When duplicating a column for backfilling, ensure that any FK constraints on the column are duplicated with a special prefix to identify them as duplicated constraints. Naming the duplicated FK constraints in this way allows the rename process to ensure that it only attempts to rename duplicated FK constraints.

I guess it's in the case you are actually adding it, which requires a duplicate?

Yes, that's it. We need to distinguish the case where the FK on a temporary column is a duplicate because an FK was already on the column when we set the column NOT NULL for example, vs the case where the FK is on the temporary column because it's new on the column (ie put there by a 'set FK' operation).


ajaxSniffer - Added support for responses which don't contain content type

Encountered the following error:
Uncaught TypeError: Cannot read property 'search' of null
It turned out the site wasn't returning a content type header at all.
I couldn't find any appropriate test but I've manually tested it on the offending site and on google.

Thanks @aidantreasure. Merging now.


Replace uglifier-js with terser to ensure correct minify step

s. https://github.com/collective-soundworks/soundworks-template/issues/5#issuecomment-668707188

Thanks


Changing CrossSiteScriptingProtectionParam to "no" does not delete XSS rule

If the existing stack has CrossSiteScriptingProtectionParam set to "yes", changing it to "no" and updating the stack results in an error:
DELETE_FAILED | AWS::CloudFormation::ManagedCustomResource | WAFXssRule | This entity is still referenced by other entities. (Service: AWSWAFRegional; Status Code: 400; Error Code: WAFReferencedItemException; Request ID: eb82f55e-e65b-11e7-852c-f5cacd8acf32)

However the stack ends in an UPDATE_COMPLETE state with reason "Update successful. One or more resources could not be deleted." and the XSS rule is still enabled in the WebACL and must be manually removed.

Hi @rectalogic
We've just pushed a new version. The custom resource was reviewed to avoid update issues like the one you've mentioned.
PS: Please note that the only update that is not support is when you try to change ALB stack to/from CloudFront. During such type of update, the Lambdas' permissions are changed ('waf-regional' to/from 'waf') and this lead us to permission error when deleting old resources.
The recommended approach, if you want to switch an ALB stack to/from CloudFront, is to delete and recreate a new stack.


Investigate if the creation of the environment can be further optimized

See if it can be done in less steps

Have you tried using micromamba? That's what we use with ODINN and it's generally very fast, and it uses the same steps as with conda (i.e. simpler than with poetry).


Fallback when celery is not installed.

Celery is an optional install for pinax-badges. This change means that importing pinax.badges.tasks does not fail if celery is not installed.
Fixes #31

@longhotsummer Thank you!


[Screen Reader-Help-Hardware] Link purpose is not descriptive enough for the ”Here” link on hardware pane.

User Experience:
If a link doesn't have valid text to identify itself, then screen readers will not be able to read the link clearly, it will just be read out to the visually impaired user as “link.” Which then makes it difficult for the blind users to understand the purpose or meaning on the link.
Test Environment
OS: RS2 Version 1703(OS Build 15063.483)
Platform: Edge.
Screen reader: Narrator
Repro Steps
1-Navigate to https://makecode.microbit.org/acc
2-Navigate to the Micro bit section element and select Code control given on it.
3-Navigate to the Help control lying in the header section on the page and select it.
4-Navigate to 'Hardware' control and select it.
5-Navigate to the various controls lying in the pane opened.
6-Verify, Here link on hardware pane is descriptive enough or not.
Actual Result
Link purpose is not descriptive enough for the ”Here” link on hardware pane. Due to this narrator is reading only here link.
Expected Result
Link Purpose should be descriptive for the 'Here' link. It should clearly provide the purpose of the link for example 'Uploading programs to your micro: bit’ here link.
Note:-
Same issue exists for LEDs and pins and USB links on hardware pane.
MAS Reference
https://microsoft.sharepoint.com/teams/msenable/_layouts/15/WopiFrame.aspx?sourcedoc={06a891af-210c-4c6d-bb3e-92babab209e7}
Suggested fix:-
1-Providing link text that describes the purpose of a link or a link for anchor elements
e.g.

Current routes at Boulders Climbing Gym

a href="routes.html
img src="topo.gif" alt="Current routes at Boulders Climbing Gym" />
/a
2-Providing text alternatives for the area elements of image maps
3-Using scripts to change the link text (choose short or long link text )
4- Use 'aria-labelledby' or 'aria-label' for link purpose
Key References for Fix:
http://www.w3.org/TR/UNDERSTANDING-WCAG20/navigation-mechanisms-refs.html
Please refer attachment for more details:
MAS2.4.4_Hardware_Links purpose is not descriptive enough for all links on hardware.zip

Fix available on https://makecode.microbit.org/

Verified on URL: https://makecode.microbit.org/acc and issue is fixed now. Hence closing the bug


HTTPS allows user tracking by OCSP Servers

Here the Dailystar site is linked. As site opens, the browser queries an SSL Server (Amazon CloudFront).

The key here is that many SSL certificates have a one-to-one relationship with a specific web site (i.e. not wildcard certificates), so a query to an OCSP responder for a web site certificate essentially tells the certificate issuer that you are accessing that site.

See: zscaler article
As the page is loaded, CSS Files are also fetched through HTTPS via CDN: Cloudfront is queried a second time and also 1e100.net (Alphabet Inc.) is queried. So Amazon and Google know that the user opened this site at a specific time.
DNT respecting sites, if delivered through HTTPS, make DNT useless:
https://cabforum.org/pipermail/public/2017-June/011346.html
IMHO HTTPS must be used only when the request include passwords and other personal information, otherwise big actors will track users browsing with ease.
Related to: https://github.com/ghacksuserjs/ghacks-user.js/issues/102

OCSP stapling prevents this.

I assume stapling goes by browser's session, so it's a mitigation IMHO

server support is required

And admins must enable it, e.g. digicert SSL Certificate Checker says:

Github OCSP Staple: Not Enabled

Related: https://github.com/ghacksuserjs/ghacks-user.js/issues/267
https://www.grc.com/revocation/ocsp-must-staple.htm

Yes, HTTPS has privacy implications currently, because of OCSP. But indeed OCSP stapling is solving the issue. It will takes time before this is generalised, but not using HTTPS is way worse: it means any machine on the network between you and the website or service you’re using knows everything you do. I’d rather much prefer some CAs to known that I went to this website at this time than anything I’ve just described.
One other thing you could do is disable OCSP while waiting for OCSP stapling to be widespread. Or disable it only when there is no stapling. But not sure if that is currently possible.
Also regarding ressources delivered by CDNs, this is a different topic. Either disable fonts/js/css (generally or through uMatrix for instance) and/or use Decentraleyes.

[Atavic said]

IMHO HTTPS must be used only when the request include passwords and other personal information, otherwise big actors will track users browsing with ease

Did you really just say that? Almost everything you do on the internet is personal - do you really want your ISP knowing exactly every single page you visited rather than just the domain. And besides, one of the points of HTTPS is that you know the data sent and received hasn't been tampered with. Case in point: blocklists as Francois pointed out elsewhere
[ArchangeGabriel said]

Also regarding resources delivered by CDNs, this is a different topic.

OCSP stuff: as per the user.js (see 1211), there is a trade-off here with privacy and security. If you want OCSP, then you have to connect to/ trust a 3rd party.
 * It's a trade-off between security (checking) and privacy (leaking info to the CA)

Not sure what the point of this issue is. You have to connect to DNS to resolve IP addresses. You have to rely on a VPN or your ISP or carrier for internet. You have to rely on GPS for using real time maps on your phone etc. There's generally always a 3rd party. The internet was never designed with privacy (or security for that matter) in mind from the ground up. Ever since then, it's been bandaid after bandaid.
If your threat model requires it, use TBB.

@Atavic, https://habr.com/post/247465/

I see valdikss created a solution here. I'm following his GoodbyeDPI Repository, but I have barely scratched its surface.


Merge release 7.3.2

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

OK, I've already triggered a build for you.


Added --spark-executor-instances to specify how many instances by worker

This PR makes the SPARK_EXECUTOR_INSTANCES be configurable.
I tested this PR by manually launching a cluster and running a job.

What's the motivation for adding this?
I am wary about adding more options for Spark since there is a potentially a lot of stuff we could end up duplicating into Flintrock. Typically, I'd just say "Use run-command to deploy whatever config changes you want to the cluster." It's clumsy but generic.
Regarding executor instances in particular, why not vary your instance size or count if you want to control how many executors you have?

@nchammas I'm sorry, but I'm so used to this feature that I don't even know how people do without it. I mean, how can you run the same job in a x1.32xlarge and a r3.xlarge without using this option or, alternatively, tweaking other JVM/Spark parameters?
Our solution is to break any big machine (big machine is any machine with more than 60GB of ram) in small machines (JVMs) with ~30GB. So in this example, we will set the x1.32xlarge to 64 executor instances. In this way we always have a homogenous cluster from the perspective of executor JVM, letting us use the same GC defaults for any configuration. Also, even considering the case where no GC tuning is needed, like when the machine is an r3.2xlarge, multiple smaller JVMs seems to run faster than a single big JVM (e.g networking/data transfer is improved).
So this simple single option let us avoid the need to use many other configurations and handling different machines in different ways.

@nchammas rewritten to conform with current master

Thanks for sharing the additional information. Makes sense to me. I'm curious if @BenFradet, @serialx, or any other users using Flintrock with large instance types have felt the need to use this setting before.
I'll try to code review this PR next, hopefully by next weekend (though, as with most open source, I can't make any promises).

Code seems to be okay to me. export SPARK_EXECUTOR_CORES="$(($(nproc) / {spark_executor_instances}))" could be a reasonable default.
It is recommended to set SPARK_EXECUTOR_INSTANCES when the instance has many cores.
I've also used to set SPARK_EXECUTOR_INSTANCES greater than 2 when we used spark-ec2 script. But resorted to using SPARK_EXECUTOR_INSTANCES=1 after we migrated to flintrock. It would be good to support this feature in flintrock. This settings also makes sense when doing the benchmarks too.

Sorry for dragging my feet on this. Thank you for contributing this feature and persisting through the review process @douglaz!
Thanks also to @serialx for providing his thoughts, which helped me decide what to do here.