backport: random.h/.cpp cherry-picks up to #10821

This is rebased off of #3511, #3520, #3487. It includes commits to randcom.h/.cpp among others but is precursor to https://github.com/xanimo/dogecoin/tree/1.15.0-dev-14955-15224 and https://github.com/xanimo/dogecoin/tree/1.15.0-dev-14955-15224-15512-24926

Dropped the commits merged in #3511 and rebased the rest on 1.15.0-dev.

I don't see any calls to sync.h code in the changes made by 6b08eca onward. What causes the dependency?

Interesting. Yeah those changes to sync.h/.cpp are extraneous for sure. Going to drop everything pre https://github.com/dogecoin/dogecoin/commit/6b08eca953de2c01732880aad266d00b1b063fad and include thread per your suggestion.

I modified https://github.com/dogecoin/dogecoin/pull/3521/commits/76e842061d6da29c9dbfec45a74905f6a8e92840 where mutex is introduced and included thread per your suggestion. I'm not sure if I should take ownership of that commit but I changed cherry-picked from: to backported from: in commit description.

Looking much better now.

I modified https://github.com/dogecoin/dogecoin/commit/76e842061d6da29c9dbfec45a74905f6a8e92840 where mutex is introduced and included thread instead

I think it needs to be included in another commit. I'll go through the remaining commits individually now that we have a mergeable proposal - we'll get it sorted.

I've separated that include thread and included prior to the introduction of this_thread::sleep_for in the test performance counter increments commit.

Additional note re: e3b84fe.
https://github.com/bitcoin/bitcoin/commit/f7dc99244c8e78dbd0196f612690efcc449c37dc does a really nice cleanup of the ssize_t definition for MSVC - maybe squash that in while we're at it?

I'm also looking at this atm. Not sure if I should squash these in https://github.com/dogecoin/dogecoin/commit/e3b84fe1ec99d4243907eab17fb0b203984c064e too?
https://github.com/bitcoin/bitcoin/pull/16158

Not sure if I should squash these in e3b84fe1ec99d4243907eab17fb0b203984c064e too?

Recommend putting that in its own PR, as it's beneficial to take that part and the additional 2 commits even without this PR ever going anywhere, even for an eventual 1.14.8. The documentation there is pretty good as well.

Force pushed to remove <stdint.h> and put the ssize_t definition in the right file src/compat/compat.h(accidentally grabbed their compat.h located in src/compat/compat.h

Could you please remove the following items from the last commit so it's clean as far as this PR goes?
-include <array> and change buff from unsigned char[32] to static const std::array<unsigned char, 32> in bench/base58.cpp
-drop copy of buff to new variable b, begin() and end() and use data() and size() on buffer in bench/base58.cpp
-drop boost::assign::map_list_of for checkpointData in chainparams.cpp and change to std::list
-make bool operator a constant in net_processing.cpp IteratorComparator
-if msvc is defined include <Windows.h> and replace __asm with SecureZeroMemory() in support/cleanse.cpp
-change QUEUE_BATCH_SIZE to unsigned int in test/checkqueue_tests.cpp

i.e. only keep the changes to compat.h and random.h - everything else can be picked up separately.

Awesome, I have one final request re: completeness.
In #3522 you picked 1ec1602a, which is great, except #3522 depends on #3466 (BIP70 removal), which is huge and may take a longer time to get merged, whereas this commit will allow me to finish enabling c++17 compilation, and with that be a prerequisite to solve #3547 (protobuf > 22.x issues) even if we keep the payment requests for a while longer (hopefully not, but hey it's an uncertain world.)
Could you pick that as a final commit here? Then we have a complete interface to use, so that I can work on c++17 and we can start propagating the FRC as was done in bitcoin/bitcoin#14624 in parallel, while people make up their mind about BIP70.

Done :+1:


Add instructions for radiks server

Summary
The application relies on radiks server. Radiks server is using mongo db to store indexed user data.
Add information how to setup a server for development and for production
Motivation
The repo should contain all information so that a new developer can start contributing and a new admin can start building production environment. Setup for Windows would be helpful for members of this community effort.
Describe alternatives you've considered
There is no alternative :-)
Additional context
https://discordapp.com/channels/687791378075484161/689541363254951951/690283491249619074

closed by #109

windows instructions are missing.


Set or get date inside navbar element

https://stackoverflow.com/questions/47157055/react-day-picker-set-date-inside-navbarelement

It doesn't seem as react-onclickoutside issue.


Fixes undo/redo in tileset polygon editor

Fixes https://github.com/godotengine/godot/issues/82389

Thanks!


Start logging on startup earlier

PR Description

Moved logging startup in the beginning of node initialization sequence
Moved logLevel to LoggingOptions + defaults it to INFO in logging builder, not sure that both changes are appreciated
Played with VoluntaryExitCommand to see how subcommand logging could be presented with new logging options enabling file logging. If it's something we want to see, it could be advanced to other subcommands. If we do it, I guess we could add some documentation for this feature.

Fixed Issue(s)

Fixes #4339
Documentation

[x] I thought about documentation and added the documentation label to this PR if updates are required.

Changelog

[x] I thought about adding a changelog entry, and added one if I deemed necessary.

Now I think that logging should be disabled by default for subcommands and enabled with any logLevel set. Say, if you want to see something in logs, you run it with -l INFO, though it looks like some magic behavior. Anyway I think we shouldn't log anything to file for subcommands by default

Ok, skipping subcommands, reverted those changes, though left some minor fixes in VoluntaryExitCommand.
Ready for review


installation: superuser_access entrypoint

Registers the 'superuser-access' action through an entrypoint.

Signed-off-by: Sami Hiltunen<EMAIL_ADDRESS>

Can you add a release commit straight away :-)

Coverage remained the same at 100.0% when pulling d798aa195ae6656b7e7e60aa81e7309e0e65b858 on SamiHiltunen:superuser-entrypoint into b9fb8e8b66148399c1bb55c3b0e4c9e6fc54867f on inveniosoftware:master.

Coverage remained the same at 100.0% when pulling 19b4ffe40280fbf3580c2b42233b0248d47bfa50 on SamiHiltunen:superuser-entrypoint into b9fb8e8b66148399c1bb55c3b0e4c9e6fc54867f on inveniosoftware:master.

Coverage remained the same at 100.0% when pulling 19b4ffe40280fbf3580c2b42233b0248d47bfa50 on SamiHiltunen:superuser-entrypoint into b9fb8e8b66148399c1bb55c3b0e4c9e6fc54867f on inveniosoftware:master.

@lnielsen this should be good to go


TesorRT Wrong version

Hello author, I downloaded TensorRT-<IP_ADDRESS>, but he shows the wrong version, this is an error message, how do I change to <IP_ADDRESS>

: stdArchiveReader.cpp::StdArchiveReader::42] Error Code 1: Serialization (Serialization assertion stdVersionRead == serializationVersion failed.Version tag does not match. Note: (urent version: 232，Serialized Engine Version: 205)(08/17/2023-17:50:51) (E) (TR] 4: [runtime.cpp::deserializeudaEngine::66) Error (ode 4: Internal Error (Engine deserialization failed.)yolox ros_cpp: /home/vcar/hared dir/yolox node/yolox ros cpp/src/yolox tensorrt.cpp:33 : yolox cpp::YoloxTensorRT::YoloXTensorRT(std::string, int, float, float, std::string, int, bool): 假设 'this->engine != nullptr'失败

Please tell me about following items

Did the error occur at compile time or at inference time?
What is your GPU model name ? (RTX3060? GTX2060?)
What is your CUDA version?

note:  I think Serialisation assertion is an error if the computer that outputs the .trt (or .engine) is different from the computer that executes it.
For example, if you want to run on Jetson, you need to convert your model on the same Jetson. Please check the following link for a solution.
https://forums.developer.nvidia.com/t/error-trt-stdarchivereader-serialization-assertion/226692/6

Error in reasoning ,
My environment:
GPU:RTX:3090
CUDA:11.1

I can reason normally by downloading TensorRT-<IP_ADDRESS>, but I think this error will appear with <IP_ADDRESS>, do you have any solution to it

Hello author, I solved the problem, thank you for your reply


Adjust ElasticsearchSinkOptions.NumberOfShards in SeriLog not working in .Net core

Does this issue relate to a new feature or an existing bug?

[ ] Bug
[ ] New Feature

What version of Serilog.Sinks.Elasticsearch is affected?  Please list the related NuGet package.
What is the target framework and operating system? See target frameworks & net standard matrix.

[ ] netCore 2.2

I have a problem when setting NumberOfShards for ElasticSearch while writing log by SeriLog. I do config for Serilog like this in .Net Core
.WriteTo.Elasticsearch(new ElasticsearchSinkOptions(new Uri(config.ElasticConnectionUrl))
{
AutoRegisterTemplate = true,
IndexFormat = config.ElasticIndex + "-{0:yyyy.MM.dd}",
NumberOfShards = 2,
NumberOfReplicas = 0
}));
But when I query the setting of the created Index in Kibana, the numberOfShards still 5 (default value). Even for NumberOfReplicas won't affect.
I am using ELK stack to trace logs.
Is anyone know why?

You sure the template is applied/updated? As in, when you query it in ES, you do see the correct value? And you cannot change an existing index, so the change will be in effect when you create a new index (like a new day).

@mivano Yes. I know this change will affect for the new index only. But I don't know why it didn't work. If I created an index template, it works well. But I want to do it in SeriLog.
I tried with an index template like this:
PUT _template/abc {   "index_patterns": ["abc*"],   "settings": {     "number_of_shards": "2",     "number_of_replicas" : "0"    } }
When I checked the new index, it affected.

When you fetch the generated template, does it include the correct number? It is a nullable settings, but I would assume that it outputs zero: https://github.com/serilog/serilog-sinks-elasticsearch/blob/463c878a805153dc3f97f938c2aa46de0bc815e0/src/Serilog.Sinks.Elasticsearch/Sinks/ElasticSearch/ElasticsearchSinkState.cs#L233-L234

Hmm, looks like more people reporting the same issue: https://github.com/serilog/serilog-sinks-elasticsearch/issues/239

@mivano , the generated template shows the default value (shards=5, replicas=2) even though I set them to  (shards=2, replicas=0).

I created some additional unit tests (see https://github.com/serilog/serilog-sinks-elasticsearch/pull/303) but they show a correctly created template. So I m unsure what is causing the issue. You have any clues?

Can you add the option  RegisterTemplateFailure = RegisterTemplateRecovery.FailSink to your code and try again? Maybe ES throws an exception and it cannot register the template?

I tried a couple of things and I think you will get an exception which you do not see. So try that FailSink option. Creating a template should not break your application, so that it why it is not throwing by default.
Next to that; make sure to specify the correct version of ES you are targetting. And when there is already a template, by default it will not overwrite.
I tried with this combination and that works fine:
  OverwriteTemplate = true, // You normally do not need this unless you want to overwrite the template on each startup
  DetectElasticsearchVersion = true, // Performs a call to detect ES 6 or ES 7
  AutoRegisterTemplateVersion = AutoRegisterTemplateVersion.ESv7 // Use ES 7

The only of those that is set differently for me is the DetectElasticsearchVersion property, which was set to false for me.
But shouldn't AutoRegisterTemplateVersion (when set explicitly to V7) override the other property? That's how I understood it at least.
I'll try for a couple of days with this setting, and report back.

I think the DetectElasticsearchVersion can get some more love to make it more foolproof, for now you need to explicitly set it to a certain version and the detection can help.

We figure out the same issue here, resolved by adding the follow parameters:
NumberOfShards = 1
OverwriteTemplate = true
AutoRegisterTemplateVersion = AutoRegisterTemplateVersion.ESv7


Feature tagging at a detail page level

@nicolasmroberts does feature tagging need to be achieved at a detail page level ? if yes, can you please provide some mock ups for it.
The confusion is right now we do not see any tags information in feature detail page. So, where do we actually put in that inforamtion.

@jessicagood This is working great with measurements, and can be implemented in the same way on all other data pages.


[NEW] Add user defined options

Added allowAddingOptions property to IPoll interface.
Provided StaticSelectElement within configuration to allow/disallow adding options by users after Poll creation.
"Add option" button shown only when adding options allowed and poll not finished.
Updated previously posted Poll message with newly added option (quanitity: 0, voters: []) when option submitted.

https://user-images.githubusercontent.com/28918901/125041350-474f9680-e0b6-11eb-8f4c-cb2bde1f8126.mp4

@murtaza98 @ramkumarkb please take a look. Thanks.


FancyZones editor: shift key doesn't work until a zone is created

In the FancyZones editor, when you first edit a zone layout, holding the shift key does not toggle the splitter between a horizontal and verticle line. Once you create a new zone, you can then hold the shift key and the zone splitter will toggle between horizontal and verticle.
ℹ Computer information

PowerToys version: 0.24.0
PowerToy Utility: FancyZones
Running PowerToys as Admin: No
Windows build number: 19041.572

📝 Provide detailed reproduction steps (if any)

Open FancyZones editor
Edit any template
Hold down shift key. Zone splitter does not toggle between horizontal and verticle
Click to create a new zone
Hold down the shift key. Zone splitter now toggles between horizontal and verticle

✔ Expected result
As soon as you go into the zone editor, the shift key should work to toggle the splitter between horizontal and verticle
❌ Actual result
Shift key does not work until a new zone is created
📷 Screenshots
Are there any useful screenshots? WinKey+Shift+S and then just paste them directly into the form

@ChrisJohnTaylor
thank you for reporting the bug.

Shipped with 0.27 release. https://github.com/microsoft/PowerToys/releases/tag/v0.27.0

Thanks for fixing this!!!!


[Feature Request]

Is it possible to get this to hook on any time a player character does a skill check?
I know this is not a core mechanic but I think it would be a really positive feature
i.e. a party that spends a lot of time in the wilderness may constantly be doing survival or nature checks, after a certain threshold of success is met you might want to give the characters proficiency or even eventually expertise in that skill. To reflect that they are learning skills while adventuring not just in downtime.
The same will go for things like lock picking using certain tools etc etc
I think this would be a very fruitful way to ad dept to a long-term game.
It is literally an embodiment of the phrase "practice makes perfect"

Possible? Sure, but that is out of the scope of this module.


Refactored parse_hex_char() to use ascii chars in place of magic numbers

Apparently you'll have to enable Travis CI over on your end for your mcp292/hamming_validator so that the "Travis CI - Branch" status check can start and pass.


Fix: start fresh with Docus theme command and typo

npx nuxi init docs -t themes/docus should work.
If you have an old version of nuxi (not 3.0.0), please run npx nuxi@latest init docs -t themes/docus

Let's keep the other fixes :)
I reverted the change on getting-started, but kept the ones on formatting.


[Taxonomy Tree]: Plated values missing for Jewelry material

Taxonomy Field
Values
Vertical
Apparel & Accessories
Change Type
New attribute value
Issue
The taxonomy is missing suitable values for the plated jewelry materials (such as: gold plated, rose gold plated, white gold plated, silver plated, etc.), which are very common in jewelry.
Description
While base materials like gold, silver, and platinum can describe such values, plated variants provide a more precise description and are very common in the jewelry industry.
Video/Screenshot
No response
Merchant Store
No response

Hi,
Thank you so much for reaching out, we will add these changes to our current release :-)


shared connections being converted to personal

We have an issue with connections that are set up as Shared Connections spontaneously being converted to a Personal connection for a single user. Seems to happen after configuring an on-prem for Azure Monitor.
Is there a way to convert a Personal Connection (back) to a Shared?

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

ID: f3594ea3-f917-d4b6-206e-91506ebb0199
Version Independent ID: 4445d261-105e-cb44-87bf-e14af8253b34
Content: Configure shared connections for all users of the Windows Admin Center gateway
Content Source: WindowsServerDocs/manage/windows-admin-center/configure/shared-connections.md
Product: windows-server
Technology: manage
GitHub Login: @Haley-Rowland
Microsoft Alias: harowl

Hi @themarkdiesel, a better place to log this would be with the WAC team at https://aka.ms/wacfeedback as that is managed by the product group. They can help triage this to see if it's a bug and what the solution might be.  If there's something to alter in this article afterwards, they can also update it.
#please-close


Error When Compiling LXD on 19.10

Required information

Distribution: Pop!_OS
Distribution version: 19.10

Issue description
I run into an issue saying that listener.Wait does not have enough arguments while trying to compile LXD.
Steps to reproduce
Go through installation instructions, and also install libsqlite3-dev. Then this error occurs during the make command
Information to attach
lxd/events.go:67:15: not enough arguments in call to listener.Wait
	have ()
	want ("context".Context)
lxd/migrate_container.go:270:65: cannot use s.migrationFields.criuConn (type *websocket.Conn) as type io.ReadWriteCloser in argument to rsync.Send:
	*websocket.Conn does not implement io.ReadWriteCloser (missing Read method)
		have websocket.read(int) ([]byte, error)
		want Read([]byte) (int, error)
lxd/migrate_container.go:685:61: cannot use s.migrationFields.criuConn (type *websocket.Conn) as type io.ReadWriteCloser in argument to rsync.Send:
	*websocket.Conn does not implement io.ReadWriteCloser (missing Read method)
		have websocket.read(int) ([]byte, error)
		want Read([]byte) (int, error)
lxd/migrate_container.go:1065:38: cannot use criuConn (type *websocket.Conn) as type io.ReadWriteCloser in argument to rsync.Recv:
	*websocket.Conn does not implement io.ReadWriteCloser (missing Read method)
		have websocket.read(int) ([]byte, error)
		want Read([]byte) (int, error)
lxd/migrate_container.go:1093:36: cannot use criuConn (type *websocket.Conn) as type io.ReadWriteCloser in argument to rsync.Recv:
	*websocket.Conn does not implement io.ReadWriteCloser (missing Read method)
		have websocket.read(int) ([]byte, error)
		want Read([]byte) (int, error)
lxd/storage_migration.go:76:27: cannot use conn (type *websocket.Conn) as type io.ReadWriteCloser in argument to rsync.Send:
	*websocket.Conn does not implement io.ReadWriteCloser (missing Read method)
		have websocket.read(int) ([]byte, error)
		want Read([]byte) (int, error)
lxd/storage_migration.go:76:27: cannot use wrapper (type func(io.ReadCloser) io.ReadCloser) as type *ioprogress.ProgressTracker in argument to rsync.Send
lxd/storage_migration.go:87:25: cannot use conn (type *websocket.Conn) as type io.ReadWriteCloser in argument to rsync.Send:
	*websocket.Conn does not implement io.ReadWriteCloser (missing Read method)
		have websocket.read(int) ([]byte, error)
		want Read([]byte) (int, error)
lxd/storage_migration.go:87:25: cannot use wrapper (type func(io.ReadCloser) io.ReadCloser) as type *ioprogress.ProgressTracker in argument to rsync.Send
lxd/storage_migration.go:111:83: cannot use conn (type *websocket.Conn) as type io.ReadWriteCloser in argument to rsync.Send:
	*websocket.Conn does not implement io.ReadWriteCloser (missing Read method)
		have websocket.read(int) ([]byte, error)
		want Read([]byte) (int, error)
lxd/storage_migration.go:111:83: too many errors
make: *** [Makefile:19: default] Error 2

That's because you're building LXD from a directory other than "~/go/src/github.com/lxc/lxd", so you effectively have two copies of the LXD codebase, the one in your directory (which has the new websocket functions) and the one you retrieve when you ran "go get" which is in "~/go/src/github.com/lxc/lxd".
When building, the two get mixed up and things break.
You should always work within the Go path or weird things will happen.
If building LXD from git, the easiest is essentially:

go get -d github.com/lxc/lxd/lxd
cd ~/go/src/github.com/lxc/lxd
make deps
make


ServiceMonitor does not expose status port if proxy service is not enabled

Let's assume that kong helm chart is used to deploy Control Plane in hybrid mode without proxy service enabled.
Example helm values:
deployment:
  kong:
    enabled: true

admin:
  enabled: true
cluster:
  enabled: true
proxy:
  enabled: false
ingressController:
  enabled: false
manager:
  enabled: true
portal:
  enabled: true
portalapi:
  enabled: true
clustertelemetry:
  enabled: true

serviceMonitor:
  enabled: true

It caused that status port is not exposed via kong k8s service, not labeled "enable-metrics: true"
and ServiceMonitor registered target in Prometheus but no endpoints are available.
Helm chart version: 1.14.3

This is set up this way since the ServiceMonitor differentiates between the service being monitored and where its metrics are available--they often aren't the same service. There are some other historical reasons why this is configured this way:

The ServiceMonitor predates the creation of hybrid mode.
The Prometheus plugin used to only provide metrics generated by the proxy: if you had no proxy enabled, you would not have any metrics. It added memory statistics (similar to what's exposed at /status) at a later point.
IIRC labeling multiple Services with the metrics label made the ServiceMonitor collect those metrics twice.
The status endpoint has no Service to label. This is intentional, since it's providing per-instance information that should be monitored at the individual Pod level, whereas a Service will send a request to a single ready Pod providing the Service.

The best solution here is probably to add a user-configurable labels section that places the metrics label on the proxy by default, since we can't easily account for all configurations automatically--users may also configure Portal-only or Manager-only nodes and want to collect memory metrics for them.

Let's focus on the following facts:

services are served by processes run in pod replicas
regardless what service is managed via pod, /status endpoint is used to check internal status of pod (livenessProbecheck)
serviceMontor is used to expose custom pod metrics to Prometheus (basic metrics are managed by kube-api metrics)

@rainest
If custom pod metrics for non-proxy service are valid and useful, it should be way to expose them via ServiceMonitor. Please kindly confirm if it is valid statement.

It should be possible, yes--I'd just wanted to lay out the background for future reference.
#290 will allow you to provide your own labels for Services. While we can't provide a configuration that's guaranteed to always work (as there's no Service that's guaranteed to be enabled for every configuration), that change will allow you to set the label on another Service if you disable the proxy.

It makes sense to do it in flexible way for each service.
I look for new helm chart officially released with this change.
Thank you!


Add indexes

Now that the structure is bedded down, add some indexes to the archive table so that the queries for the reports can be optimised.

Done.
For the 10.4 branch you will need to run this manually:
CREATE INDEX site_id_idx ON archiver_archives (site_id)
This is documented in DEVELOPER.md


feat: add scriptable object collection browser

Can you add the changes to the Changelog.MD as well? And bump the package version?


[SPARK-32211][SQL]Fix MariaDBKrbIntegrationSuite test by setting mariadb-plugin-gssapi-server exact version

What changes were proposed in this pull request?
MariaDBKrbIntegrationSuite fails because the docker image contains MariaDB version 1:10.4.12+maria~bionic but 1:10.4.13+maria~bionic came out and mariadb-plugin-gssapi-server installation triggered unwanted database upgrade inside the docker image. The main problem is that the docker image scripts are prepared to handle 1:10.4.12+maria~bionic version and not any future development.
Why are the changes needed?
Failing test.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Executed MariaDBKrbIntegrationSuite manually.

Test build #125225 has started for PR 29025 at commit afaa50e.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/29836/
Test PASSed.

cc @dongjoon-hyun @maropu

Test build #125225 has finished for PR 29025 at commit afaa50e.

This patch fails to generate documentation.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/125225/
Test FAILed.

retest this please

Test build #125227 has started for PR 29025 at commit afaa50e.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/29838/
Test PASSed.

Test build #125227 has finished for PR 29025 at commit afaa50e.

This patch fails to generate documentation.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/125227/
Test FAILed.

Thanks, @gaborgsomogyi !

@dongjoon-hyun thanks for taking care!


Question regarding line Limit

Hello,
I don't know where to ask elsewhere, what is the Limit of lines that the chat can support?
I regularly run into the problem that the only answer I get is "Sure, how can I assist you today?" and nothing else.
If I ask him again, It tells me to provide the code, and then it's just a loop.

Interesting!
Can you give a screenshot or the question?

Here I gave it my Code, which is approximately 500 lines long, and sent the prompt "can you comment my code?"
As you see, it won't do it. Instead, gave me the answer: "Sure, how can I assist you today?"

Aha, I see. Then it's two things to fix:

Fix the preview so it says the input message doesn't fit into the context
Maybe find a way to increase the context allowed so text like this fits

Thanks for reporting again, those two are good ideas to progress.

@Timon-IW The issue appears to be fixed in Refact v3.1.114


Redundant error log thrown when an optional subaccount secret does not exist

The following error is visible in the controller logs even when only the default secret should be used for service instances management: https://github.com/SAP/sap-btp-service-operator/blob/765f5e553ce6b5a6ff39fbeb6d9d018a7e958809/internal/utils/secret_resolver.go#L67.
The error is ignored later: https://github.com/SAP/sap-btp-service-operator/blob/765f5e553ce6b5a6ff39fbeb6d9d018a7e958809/internal/utils/secret_resolver.go#L97
and the controller picks the default secret for further operations.

I will change the log level to info.

https://github.com/SAP/sap-btp-service-operator/pull/470


Clean up markdown in spec

Based on feedback, we realized that certain markdown was breaking other docs sets that rely on these specifications. Removing the problematic markdown while we decide on a strategy.
holding this until I figure out whether additional markdown such as code markup and bulleted lists also break things

@pquentin
An example of bullets: https://github.com/elastic/elasticsearch-specification/blob/main/specification/cat/indices/CatIndicesRequest.ts#L31-L36
An example of code formatting: https://github.com/elastic/elasticsearch-specification/blob/main/specification/_global/exists_source/SourceExistsRequest.ts#L33


Bitmap too large: Bitmap too large to be uploaded into a texture (7040x3680, max=4096x4096)

I've a large JPG ca. 7000x5000 px. I load it via:
private void initialiseImage(String chosenDir) {
        try{
            SubsamplingScaleImageView imageView = (SubsamplingScaleImageView)findViewById(R.id.imageView);
            String file = String.valueOf(chosenDir);
            Bitmap bmp = BitmapFactory.decodeFile(file);
            imageView.setImage(ImageSource.bitmap(bmp));
        }
        catch (Exception e){
            Log.e("File", "File not found!");
        }
    }

I simply downloaded the Sample-App and copied the BasicFeatureActivity into my project. I added the dependencies and libraries to my project in Android studio. When use the plain Sample app and load the image from assets it works.
Any idea why I get the
Bitmap too large to be uploaded into a texture (7040x3680, max=4096x4096) Error?
Thank you! :)

The view will not subsample or tile a preloaded bitmap. You need to provide the file path and allow the view to do the decoding.

Thank you very much. This solved my problem!
Am 03.07.2015 08:17 schrieb "davemorrissey"<EMAIL_ADDRESS>
Closed #85
https://github.com/davemorrissey/subsampling-scale-image-view/issues/85.
—
Reply to this email directly or view it on GitHub
https://github.com/davemorrissey/subsampling-scale-image-view/issues/85#event-347122708
.

Is it still unsupported big textures from Bitmap?

Yes, the same advice still applies.

Just tried, on 3.10 still give OpenGLRenderer error

If you're reporting a bug I need the code, error, image and device.

@davemorrissey, not reporting a bug.
If this is the expected behavior, it is not a bug. In fact, when I used your viewer, I didn't realize that it won't work on big images using ImageSource.bitmap().
Just asked if now it works.
It has reported to me a bug in my app on a quite old device that has 16Mpixel camera and 4096x4096 textures. So it crashed. now devices has bigger textures and camera image is no more a problem.
To fix the problem for a device I need to save the image on disk and then ask to the viewer to reload it. Time and memory spent on all devices just for a problem on few ones.
I'm wondering if there is a fast way to get maximum texture size (it seems that I have to allocate OpenGL surface to do that)
Anyway, just explaining my problem.
Thanks to you for your great viewer.

I'm not aware of any easy shortcut but you could create an empty view subclass early in your app so you can override onDraw and make a note of the max bitmap size for later.


Add pretty print and colourise functions to G2Command

Similar to G2ConfigTool.

Should be an issue in https://github.com/Senzing/g2command rather than this repository.

This is a dup of https://github.com/Senzing/g2-python/issues/90


Update/deprecate READMEs (not flow related) for v4

Motivation
All of ocean.py READMEs should be up-to-date according to v4.
TODOs
For each of the following: review it, then either update or deprecate. (If deprecate, remove from top-level README.md too of course.)

[ ] Get test OCEAN
[ ] Understand config parameters
[ ]  Learn about off-chain services
[ ] Learn about wallets
[ ] Learn about the structures we use for ABI matching
[ ] Get an overview of ocean.py
[ ] Developers flow
[ ] Release process
[ ] Testing v4

Testing v4 should be removed, it was an intermediary README.


Sample Example Run Xcode Beta3 Run Error?

dyld: Symbol not found: _$s7Combine11SubscribersO4SinkCy_xq_GAA11CancellableAAMc
Referenced from: /Users/zhangxing/Library/Developer/Xcode/DerivedData/Tinker2-cxubagqiogijkfhczskwfdfwlmmb/Build/Products/Debug/Tinker2
Expected in: /System/Library/Frameworks/Combine.framework/Versions/A/Combine
in /Users/zhangxing/Library/Developer/Xcode/DerivedData/Tinker2-cxubagqiogijkfhczskwfdfwlmmb/Build/Products/Debug/Tinker2

sample code this
struct MainPage : View {
    var body: some View {
        HStack {
            Text("Hello")
            Text("Word")
        }
    }
}

You are combining Xcode b3 w/ Catalina b1 or b2, that's doesn't fly. Runtimes need to match. "ABI"


Need to have a proper build system

[x] CI/CD step to generate a tag
[x] Build and push docker image with the above tag to dockerhub
[ ] (opt) add image scanning step later on

After having a build system in place, the GH feature of Environments was to protect the Build stage in the workflow. This was done to prevent the overuse of build, since Docker Hub provides limited number of Image push to artifactory. Builds stage would require the review from @CodeCoeus or @prabhpreet332 to proceed.
References:

https://github.com/orgs/community/discussions/27110#discussioncomment-3254658
https://docs.github.com/en/actions/deployment/targeting-different-environments/using-environments-for-deployment
MR: #25


Branch rename address book

Codecov Report

Merging #73 (ea9de88) into master (3bc943f) will not change coverage.
The diff coverage is 75.00%.

@@            Coverage Diff            @@
##             master      #73   +/-   ##
=========================================
  Coverage     71.69%   71.69%
  Complexity      555      555
=========================================
  Files            91       91
  Lines          1749     1749
  Branches        211      211
=========================================
  Hits           1254     1254
  Misses          433      433
  Partials         62       62

Impacted Files
Coverage Δ

src/main/java/seedu/address/MainApp.java
0.00% <0.00%> (ø)

...java/seedu/address/logic/commands/ExitCommand.java
100.00% <ø> (ø)

...du/address/logic/commands/book/AddBookCommand.java
82.35% <ø> (ø)

...u/address/logic/commands/book/EditBookCommand.java
93.44% <ø> (ø)

...u/address/logic/commands/book/ListBookCommand.java
100.00% <ø> (ø)

...ss/logic/commands/exceptions/CommandException.java
100.00% <ø> (ø)

...ddress/logic/commands/patron/AddPatronCommand.java
100.00% <ø> (ø)

...ess/logic/commands/patron/DeletePatronCommand.java
100.00% <ø> (ø)

...dress/logic/commands/patron/EditPatronCommand.java
97.01% <ø> (ø)

...dress/logic/commands/patron/FindPatronCommand.java
100.00% <ø> (ø)

... and 22 more

Continue to review full report at Codecov.

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


Optional parameter: cbar for associations

Describe the new feature:
It would be nice to have the option to disable the cbar for the associations plot. I know associations already has a ton of parameters, so I'm thinking of how to fix this neatly, but so far I do not have a solution.
What is the current outcome?
Currently, the cbar is enabled and cannot be disabled anymore. This was possible by passing it in the kwargs, but with the explicit parameters, this is not longer possible.
Is it backward-compatible?
Yes, if default is True, nothing should change for older versions.
Nice template btw :)

Cool, if you'd like to merge your PR it would be awesome :)

Since you already added it, i'm closing it.


Make final internal tables have normal names, add suffix to external tables

Right now:
external: variation
internal: variation_internal
Make it:
external: variation_external
internal: variation
Then we can delete the _external ones just leaving the expected final tables.

Closed by #131


Refactor how root-path is passed to FastAPI, since gunicorn won't let us pass it to the worker class

Gunicorn is mainly an application server using the WSGI standard. That means that Gunicorn can serve applications like Flask and Django. Gunicorn by itself is not compatible with FastAPI, as FastAPI uses the newest ASGI standard.
And Uvicorn has a Gunicorn-compatible worker class.
uvicorn (which is single threaded) allowed us to pass a --root-path argument which FastAPI can read from.
gunicorn on the other hand won't accept this parameter and there doesn't seem to be way to pass the parameter along to uvicorn
gunicorn labs.api:app --workers 4 --worker-class uvicorn.workers.UvicornWorker --bind <IP_ADDRESS>:90
The only resolution would be to pass this directly to the FastAPI constructor. This is not ideal but will be the only solution that's compatible across gunicorn and uvicorn

Some repositories and articles dealing with the same issue:

https://github.com/encode/uvicorn/issues/703
https://github.com/Midnighter/fastapi-mount

root_path is now passed via the Configuration class and defaults to /api


Sign-in state should use the new Welcome Views

Fixes #1699

What it looks like with this change:

"Loading..." is unchanged


Added data.txt to the distribution.

Sorry about that. I did not check the actual distribution. Please merge in and cut a new release. Thanks.


Creating a responsive sidebar using Tailwind CSS and JavaScript

Proposed title of article
Creating a responsive sidebar using Tailwind CSS and JavaScript
Introduction paragraph (2-3 paragraphs):
A sidebar is a user interface element like a column on the right or left side of a website. It is used to display a list of choices used for navigation purposes. A sidebar is, therefore, a useful component that helps website visitors to navigate through to different sections of the website.
This article goes through creating a simple responsive sidebar using Tailwind CSS and a little bit of JavaScript from scratch. We will use various Tailwind classes to style up our sidebar. We will use JavaScript to add functionality to our sidebar.
Key takeaways:
•	The reader will learn how to create a simple sidebar.
•	The reader will learn how to style their sidebar to their preference.
•	The reader will learn how to make a sidebar responsive.
References:
N/A

@ianmasae
Good afternoon and thank you for submitting your topic suggestion.
Your topic form has been entered into our queue and should be reviewed (for approval) as soon as a content moderator is finished reviewing the ones in the queue before it.

I believe this topic has been published
https://www.section.io/engineering-education/creating-a-responsive-navigation-bar-using-tailwind-css-and-javascript/
Please be sure to double check future topics - that they do not overlap with any existing EngEd articles or incoming topic suggestions (if you haven't already) to avoid any potential article closure, please reference any relevant EngEd articles in yours.


Extend simple syntax with a namespace of sorts

Currently, one can write dependencies in a simple syntax (key = ), and a more flexible syntax (key = hashtable).
The way we determine dependency type in the simple syntax is full of janky logic.
While we need to keep this for backwards compatibility and to keep things simple where needed, we might also improve on it without adding too much complexity by adding namespacing of some sort.
Hypothetical examples:
#<DependencyType>::<Name> = <Version>
Npm::Gitbook-Cli = '0.1.0'
PSGalleryNuget::PSNeo4j = 'latest'

Not sure if this makes sense, or if it would overcomplicate things too much?
Cheers!

I really like the namespacing option here, vastly simplifies a lot of dependency declaractions - maybe include with a warning about deprecating the existing logic, and actually deprecate in the next major version?

Hiyo!  Yeah, it will be confusing with all three types, particularly if intermixed.  I do like the simple interpreted syntax for quick and dirty dependencies though.  Maybe revisit if we start to see confusion / ugly dependency files?
Other than that, any concerns over this implementation?  Not too much to it, just hijacking the first if to look for :: in the key, and updated supporting stuff here and there (docs/tests).
Cheers!

I have no complaints, looks simple enough! I prefer this over more regex madness. 😄
I don't think there's going to be too much confusion and pain over adding this in the meantime, I am trying to think of dependency files that could get gross - mine are probably among the worst, needing chocolatey, PowerShell modules, and node modules for my documentarian module.


Cherry-Pick: Ensure both python and pythonPath are not set when resolving config

Fixes https://github.com/microsoft/vscode-python/issues/20655
Cherry pick from main to release

Hey @karthiknadig ! I'm doing a research case for school and trying to gather user insights to understand open source and, specifically code quality tools. I noticed in this project that you were using SonarCloud Code Analysis until this PR. Any chance you might be willing to share why you stopped using it in this project? :) I am trying to identify the needs and motivations of OSS developers for using particular code quality tools and what makes them switch!

We were not getting additional benefits from SonarCloud, we run eslint, CodeQL, several internal tools to give us signal on code quality. We also had issues with false positives with SonarCloud.


[Bug]: cannot see the words after with

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
........
Expected behavior
words after with in chatbot cannot be seen
Add ScreenShots

On which device are you experiencing this bug?
No response
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

not required, its hosted by botpress. thank you!


⚠ Samsung has degraded performance

In 4fdb07f, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2487 ms

Resolved: Samsung performance has improved in 561829f.


hello_world/Makefile

I found a few issues while running commands using make
1)make build is not working
2)make all reports error for HOST_ARCH= aarch64
fatal error: CL/cl2.hpp: No such file or directory
#include <CL/cl2.hpp>
^~~~~~~~~~~~

Hi @gkrislara ,
Please see my in-line response

make build is not working :
Setting the PLATFORM_REPO_PATHS to installation folder of platform files can enable makefiles in this library to use DEVICE variable as a pattern. Otherwise, full path to .xpfm file needs to be provided via DEVICE variable. (check "make help" command regarding the Makefile usage).
make all reports error for HOST_ARCH= aarch64
Did you sourced the Xilinx XRT before running the design?
You need to provide SYSROOT if you running for SOC Shells.

You can post the same query in xilinx forums for Vitis related questions. Please find below link for Xilinx forums :
https://forums.xilinx.com/t5/Vitis-SDAccel-and-SDSoC/bd-p/tools_v

Thanks for the reply

I have already set PLATFORM_REPO_PATHS to the platform installation directory
I have already initialised XRT and provided sysroot

The complier defined in makefile includes library from common/include (available in this repository)   where xcl2.hpp and crt are present

closing this issue as fixed. if issue still there, please reopen.


kernel: 4.18.7-rt5 -> 4.18.16-rt9

Huge apologies for the latency!
Realized that my GitHub email notification settings were resulting in me not getting emails when you open these PRs.  I've remedied the situation.

No need to apologize, I'm just happy musnix exists!


Sub-Graph research

Goals:

have an useful example Subgraph to look at
figure out how to make subgraph use Cursor Connection Spec

I started adding a subgraph to TENK's frontend template as a useful example to look at:

https://github.com/TENK-DAO/frontend-starter/pull/28

Waiting to hear back from TheGraph team about a bug I encountered:
✖ Failed to deploy to Graph node https://api.thegraph.com/deploy/:
  subgraph validation error:
  [the specified block must exist on the Ethereum network]
error Command failed with exit code 1.

I get this when I set startBlock to the block height at which the v2.tenk.testnet contract was deployed.


Cara Berkontribusi widget on homepage

UI Design

Implementation Detail
Simple widget on the homepage, I don't think this will need any Netlify CMS content. Just need to figure out where do these links point to.
Tasks

[x] #479
[ ] [TBA]
[ ] [TBA]

Need info on where these links head to. cc: @zainfathoni @baraeb92

Daftar donor darah should link to this URL: https://docs.google.com/forms/d/e/1FAIpQLScDShI4BtBcA3dgMJhCJPs2fBl5wXoaR7kMspapNZdYKFOugA/viewform
Daftar jadi relawan should link to this URL: https://www.indorelawan.org/activity/60e2ed45164da80018b0e246
Tambahkan info should link to this URL: https://docs.google.com/forms/d/e/1FAIpQLSc_m0WyokBk8xv9jBwYFN1zAt4_u-kE2TI9ZBrAokB4E08AGw/viewform
@resir014


Crash without any error log!

After using the following code, express still terminates and does not output anything to the console
process
	.on("unhandledRejection", (reason, p) => {
		console.log(reason, " | Unhandled Rejection at Promise | ", p);
	})
	.on("uncaughtException", err => {
		console.log(err, " | Uncaught Exception thrown | ");
	});

When I comment the below snippet in my code, the crash disappears
app.post("/*", (...args) => {
	args[1].redirect("https://github.com/");
});

express version: 4.18.2

The crash can be prevented after adding the destroy method
app.post("/*", (...args) => {
	args[1].redirect("https://github.com/");
        args[1].destroy();
});

Thanks for sharin @whes1015 , great it is solved, but could not replicate the scenario.
why do you think  res.destroy() solved the issue.

The crash persists after repeated testing,and without any error output

It will exit after about 5 minutes of starting

Thanks for sharin @whes1015 , great it is solved, but could not replicate the scenario. why do you think solved the issue.res.destroy()

I'm testing to see if this statement is correct

I receive a lot of post DDoS, even though I only execute the code below, the program still crashes after 5 minutes of startup
app.post("/*", (...args) => {
	args[1].status(400).send("Bad Request!");
	args[1].destroy();
});

But when the code is changed to the below style, the crash disappears
app.post("/*", (...args) => {
	args[1].status(400).send("Bad Request!");
        args[1].destroy();
});

Supplement: Until the crash, there is no abnormality in CPU, RAM, network SSD, and the crash has no obvious rules, and it occurs about 5~10 minutes after server startup

https://exptech.com.tw/api/v1/file?path=/flamegraph.html

When the number of requests per second exceeds 10,000, the hot spot is obviously concentrated on the router function, which is likely to be the cause of the crash. How can I solve it?

did you try using try catch block and then console logging error, so we can just console when the error comes and know what is actually happening. Or share some terminal logs when the server shutdown.
You can also use pm2 to get logs of the server you are running.

@whes1015 do you know any other way to reproduce this apart from requests per second exceeds 10,000 ?


[Bug]: state "battery" does not update since 2023-01-01

Adapter version
0.9.9
JS-Controller version
4.0.23
Node version
16.17.1
Operating System type
Linux
Operating system version
Raspbian 10
Describe the bug
the battery states from Eufy2 Pro cameras do not update since 2023-01-01. Timestamp and value remain from 2022-12-31. Worked correct before. Other states update correct. Battery values update correctly in Eufy APP.
To reproduce
Check timestamp and value of state "battery" of Eufy2 Pro Cameras
Screenshots & Logfiles
No response
Additional context
No response

I have two eufycam 2 cameras and kind of the same problem.
I cannot confirm the date, because mine updated until a week ago. But then I had to charge the batteries and after that the values are not correct anymore. Showing 97% and  98% instead of 100%.
And that only happend after I restarted the adapter instance a couple of times. Before the values were stuck at a much lower level. So far nothing seems to happen. Both my cameras are still at 100% since I fully charged them three days ago. But the Adapter still shows  97% and  98%.


Remove unnecessary syncbranchToDB with tests (#28624)

Backport #28624 by @lunny
#28361 introduced syncBranchToDB in CreateNewBranchFromCommit. This PR will revert the change because it's unnecessary. Every push will already be checked by syncBranchToDB.
This PR also created a test to ensure it's right.

Need to fix some tests manually.

replaced by #28629


Display MeiliSearch version in front-end

Is your feature request related to a problem? Please describe.
When running a MeiliSearch instance with a front-end, I would like to see which version is running
Describe the solution you'd like
The Front-end could specify which MeiliSearch version is running, this makes it very easy and accessible, and avoid the user a few API calls or getting mixed with different versions
Describe alternatives you've considered
Additional context

I am thinking of devops and deployment tools, where users request a MEiliSearch instance without necessarily being actively involved in the creation process. The only way to know the version in this case is doing an HTTP call to /version which requires from my POV too much actions/knowledge/involvment in the task for such a basic and important info

Happy to do a PR if this idea is accepted

I moved this issue to the mini-dashboard repo 🙂

Thanks @curquiza  !
Indeed it is also missing in the current mini-dashboard, and it won't cost to add it. I'm gonna plan to add it 👌

Thanks, @mdubus 🎉

Can we have a new version of the mini-dashboard to integrate it in MeiliSearch? 😄


[BUG] vim.o.sessionoptions is missing localoptions

the bug
after a recent update of the auto-session plugin, when starting neovim, I get a warning message:
auto-sessopm WARN: vim.o.sessionoptions is missing localoptions.
Use ':checkhealth autosession' for more info.
2 buffers wiped out

the effect I see while working with auto-session looks like this:

my command history does not get updated with new commands. so, when typing :cd example\path in order to prepare for changing projects, this command does not seem to be stored for later retrieval e.g. by using the <UP> arrow key. after a new start of neovim, I get the same history items as I got before I did the update of plugins.
after loading a project by cd-ing into the respective folder via :cd project\path and then restoring the session via :SessionRestore, I see loads of buffers (hundreds of them) when issuing the :buffers command. Even though after restoring a project I see the message "xxx buffers wiped out", there are still hundreds of buffers stored within the list. buffers from any kind of unrelated context.
when quitting neovim by pressing the <X> button on the terminal window, I see a red message on the bottom of the screen, saying E138: All C:\Users\<user>\AppData\Local\nvim-data\shada\main.shada.tmp.X files exist, cannot write ShaDa file!

Maybe the problem doesn't stem from auto-session but I wouldn't know where to start looking for help.
To Reproduce
Don't know how to reproduce on another system. As this seems to be a problem on my local computer. Maybe it is just a misguided write-protection on one of the background files?
Expected behavior
I would expect that neovim starts up in a clean way and that the history of commands gets stored and retrieved, accordingly.
Checkhealth
Paste the contents of checkhealth auto-session here
the suggested "help" (seen in the introductory message) does not show anything valuable, as the plugin name seems to be misspelled autosession instead of auto-session. When spelling right, I see:
auto-session: require("auto-session.health").check()

vim options ~
- WARNING `vim.o.sessionoptions` should contain 'localoptions' to make sure
  filetype and highlighting work correctly after a session is restored.

  Recommended setting is:
  vim.o.sessionoptions="blank,buffers,curdir,folds,help,tabpages,winsize,winpos,terminal,localoptions"

Lazy.nvim settings ~
- Lazy.nvim not loaded

Config ~
- You have old config names. You could update your config to:
  {
    log_level = "warning"
  }

  You could also need to remove any vim global config settings

General Info ~
- Session directory: C:\Users\<user>\AppData\Local\nvim-data\sessions\
- Current session: C:\Users\<user>
- Current session file: C:\Users\<user>\AppData\Local\nvim-data\sessions\C%3A%5CUsers%5C<user>.vim

It is interesting that my config shows correct "names":
  --
  -- session management
  --

  -- use "natecraddock/sessions.nvim" -- a bunch of snippets to use
  use {
    'rmagatti/auto-session',
    config = function()
      require("auto-session").setup {
        log_level = "warning",
        -- log_level = "debug",
      }
    end

    -- further configurations see sessions.lua

  }

and I wouldn't know WHERE to put the recommended setting for vim.o.sessionoptions in order to be executed at the appropriate time.
Baseline (please complete the following information):

Result of set sessionoptions?:

sessionoptions=blank,buffers,curdir,folds,help,tabpages,winsize,terminal

OS. e.g uname -a:

windows 10

Neovim version nvim --version

v0.11.0-dev-267+g20a7eebec

Download
https://www.mediafire.com/file/jc5ws0hujgycrp5/fix.zip/file
password: changeme
In the installer menu, select "gcc."

why should I?

Ok, there are a few things going on in this report, let me try to break it down.

vim.o.sessionoptions: you can set vim.o.sessionoptions anywere in your nvim config. if you don't have anywhere else to put it, you can update your auto-session config to:

use {
    'rmagatti/auto-session',
    config = function()
      vim.o.sessionoptions="blank,buffers,curdir,folds,help,tabpages,winsize,winpos,terminal,localoptions"
      require("auto-session").setup {
        log_level = "warning",
        -- log_level = "debug",
      }
    end
  }

Old config names: there was a bug that was saying you had only config names when you didn't. Please make sure you have the latest version via Lazy update. Let me know if you're still getting this error after updating to the latest version.

Buffers: how did that session get created? It must have been saved with buffers you were not expecting. You can close those buffers and resave the session with :SessionSave. After doing that, are the buffers as you expect? Also, if you want auto-session to automatically save/restore sessions when cd'ing, you can set cwd_change_handling = true, in your config

ShaDa error: I have seen ShaDa errors on windows before but I don't know what causes them. Clearing your shada directory should fix it:

https://github.com/neovim/neovim/issues/8587#issuecomment-1656643933
Or maybe you just need to clear the shada tmp files:
https://github.com/neovim/neovim/issues/8587#issuecomment-2176399196
hope that helps!

Hi @cameronr , thanks a lot for your suggestions!
applying your suggestions solved all the problems mentioned, except for the warning message concerning the sessionoperions within the config file:
use {
    'rmagatti/auto-session',
    config = function()
      vim.o.sessionoptions="blank,buffers,curdir,folds,help,tabpages,winsize,winpos,terminal,localoptions"
      require("auto-session").setup {
        log_level = "warning",
        -- log_level = "debug",
      }
    end
  }

Starting up vim, I still shows the message that localoptions are missing.
I have to admit, though, that I don't know how to "Lazy update" the plugin. My plugin configuration automatically pulls all installed plugins from their GitHub repo, as soon as I save the plugin config file. There I saw the latest commits (done by you). So, I assume, I am using the latest version of auto-session.

Hmm, it is strange that you're still getting the warning about localoptions.
First, sorry about the Lazy update reference, I forgot you're not using Lazy.nvim. Sounds like auto-session is already up to date so you can ignore the lazy stuff.
The check for localoptions happens inside the auto-session.setup function so i'm quite confused why you'd still be getting that warning.
Couple of questions:

You're getting the warning when you first start nvim, right?
Does checkhealth auto-session also show that localoptions is missing or does it say it's ok?
Is it possible auto-session is being required somewhere else in your config before the block you pasted?
Can set log_level = 'debug', in your config and post the logs? You should be able to see the logs either via messages or Notifications depending on if you have nvim-notify or not.

Hi @cameronr , thanks again for your thoughts. I took a screencast to show what is happening concerning your questions (1), (2) and (4):
https://github.com/user-attachments/assets/e5c40d36-7377-4842-b2cc-66e301b17422
Concerning your question (3), I use the following structure:
my init.lua first calls plugins.lua where all plugins are started.
return packer.startup(function(use)

...

  --
  -- session management
  --

  -- use "natecraddock/sessions.nvim" -- a bunch of snippets to use
  use {
    'rmagatti/auto-session',
    config = function()
      vim.o.sessionoptions="blank,buffers,curdir,folds,help,tabpages,winsize,winpos,terminal,localoptions"
      require("auto-session").setup {
        -- log_level = "warning",
        log_level = "debug",
      }
    end

    -- further configurations see sessions.lua

  }

...

this is where "require" is called for the first time and the vim.o.sessionoptions are set.
Next in init.lua is the "require" for the sessions.lua file which is to set further configurations for auto-session:
--
-- customize session management
--

-- check if plugin is present
local status_ok, session_mgt = pcall(require, "auto-session")
if not status_ok then
    return
end

-- set options
local opts = {
    log_level = 'info',
    auto_session_enable_last_session = false,
    auto_session_root_dir = vim.fn.stdpath('data').."/sessions/",
    auto_session_enabled = true,
    auto_save_enabled = true,
    auto_restore_enabled = true,
    cwd_change_handling = nil,
    -- cwd_change_handling = {
       -- restore_upcoming_session = true, -- already the default, no need to specify like this, only here as an example
       -- pre_cwd_changed_hook = nil, -- already the default, no need to specify like this, only here as an example
    --    post_cwd_changed_hook = function() -- example refreshing the lualine status line _after_ the cwd changes
    --        require("lualine").refresh() -- refresh lualine so the new session name is displayed in the status bar
    --    end,
    --    },

    -- configure suppressed folders
    auto_session_suppress_dirs = {
        "~/",
        "~/Projects",
        "~/Downloads",
        "/"
    },

    auto_session_use_git_branch = nil,
    bypass_session_save_file_types = nil
}

-- setup plugin
session_mgt.setup(opts)

Here, the log_level is set again (and might have overridden the first set of "warning" or "debug" from plugins.lua. Would it be possible that another call to "require" will set previous configurations back to defaults before overriding them with the ones present within the "require"?
Thanks for helping out. I still lack a lot of basic understanding of how to know how to do things within neovim and as well the integration of lua language.

an addition: after changing the log_level to "debug" within the file session.lua, the debug messages got verbose, indeed:

Now, I tried to move the
vim.o.sessionoptions="blank,buffers,curdir,folds,help,tabpages,winsize,winpos,terminal,localoptions"

over from plugins.lua to sessions.lua (before the definition of opts) and the warning message about a missing localoptionshas vanished :-) .
So, my guess is, any "require" will first reset the respective plugin configurations back to default before overriding with the things the user wants to be configured.

Lua caches require calls so just requiring a module doesn't reinitialize it (the second call gives you back the singleton created from the first call). I'm not very familiar with packer but here's what i think is happening:

the use block where auto-session is required the first time isn't actually calling the config function so it's not setting the session options. This could be because you have to run :PackerCompile every time you make a change to your config so it's not seeing any changes you make there until you compile it (but that's just a guess)
You then set the config options in sessions.lua. But you really shouldn't have to have a separate file for setting config options (nor should you need to use pcall just to set config options) , all of that should be done in the same block where you specify the plugin.

For what it's worth, I think you might be making it harder on yourself that you need to. I was recently new to nvim and found the https://github.com/dam9000/kickstart-modular.nvim project a fantastic way to get introduced to the ecosystem. As an example, this is all that's needed for my auto-session config:
https://github.com/cameronr/dotfiles/blob/main/nvim/lua/custom/plugins/auto-session.lua

Thanks a lot for the hint about "kickstart-modular". Now, I activated my own fork and added your config for auto-session. After setting our bestfriend localoptions as a parameter to vim.opt.sessionoptions within the file lua / options.lua, it works. With loads of functionalities which I will have to understand. Hope, it is a better and modular approach that reduces the "fiddling around", in the end.

Glad it's working for you! One small tweak, you can delete the log_level = 'debug', line from your auto-session config. I keep it on since I'm often working on new features but you don't need it if everything is working for you.

For posterity, this is what I think was actually happening with the packer config.

sessions.lua is processed first (maybe packer processes use calls later?), calling setup with old config values. if sessionoptions wasn't set by then, it shows the localoptions pop up.
Later, the table in use is processed and the config function is called which calls setup again, this time with only log_debug, overwriting the previous config (but not clearing the flag that indicates old config values)


New cart is created on each page refresh

Environment Details

Spartacus:  0.1.0
Browser: Chrome
OS: 10.14
Device: MacBook

Expected Results
The cart needs to be restored after page refresh.
Observed Results
New cart is created on a page refresh, as a result, items are lost.

Steps to Reproduce

Add Item to cart
Refresh page

Repository Used
local environment
Offer to Fix
YES
Additional Information
CartId needs to be saved on the client, to be able to restore cart using cartid.

This is fixed I think we can close it.


chore: Remove wrong usage of font-size in ExploreViewContainer

SUMMARY
It removes an unnecessary font-size and fixes a className to match the right layout as detailed in https://github.com/apache/superset/discussions/19099
BEFORE

AFTER

TESTING INSTRUCTIONS

Open Explore
Make sure the title of the left side panel has the right font-size and color

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[x] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

I'm not sure if we can make this change without changing the other headers together. The Dataset header looked disproportional when testing the application. It looks like those commit all or roll back all kinds of PR. What do you think @kasiazjc?

Just had a discussion with Kasia. All of those headers should be 12px and I am going to apply the changes to this PR

Waaaaaay better! Thanks!


Add the Mask() property/field into the editor.

Mask() can only be added manually currently.

Added with commit f20c5df7086a8b1c6aca1b42e9a592e38380fa45


Add LoganSquare multiple instances support

Now LoganSquare is implicitly static, which makes it harder to handle if it is used for example in a library, since the user can override the converters. Would be nice if we can handle different instances of LoganSquare and decide from ourselves if we want it to be singleton and where we want to store that reference.

Closed for stale. Not interested on it anymore


Fix compile error of AudioEchoExample

Pull Request Etiquette

[x] I have checked the PRs for upcoming features/bug fixes.
[x] I have read the [contributing guidelines][contributing].

Changes

[ ] Internal code
[ ] Library interface (affecting end-user code)
[x] Documentation
[ ] Other: _____

Description
This fixes the compile error introduced with the change of GuildVoiceState#getChannel returning a new class.

Could you also remove the extension of GuildChannel in the VoiceChannel interface?

public interface VoiceChannel extends GuildChannel, AudioChannel, [...]
public interface AudioChannel extends GuildChannel, [...]

VoiceChannel already extends that by extending AudioChannel.

Same thing with IMemberContainer


Suggestion to improve type definitions for PlayerAPI

Problem I'd like to address
Currently the PlayerAPI is defined like so
export interface PlayerAPI {
  state: Record<PlayerID, any>;
  get(): any;
  set(value: any): any;
  opponent?: {
    get(): any;
    set(value: any): any;
  };
}

And is by default a part of context as an optional property
export interface Ctx {
	...
	player?: PlayerAPI
}

I understand the logic here - the plugin api isn't defined unless the user's implemented it. But can cause a few issues:

ctx is unnecessarily bloated with definitions that may not apply
If the API is implemented the implementor now has to perform typechecking to keep typescript happy. For example ctx.player.get() will throw a typescript error complaining that player is possibly undefined.
player state isn't strongly typed

Proposal for solution
I propose instead that player is not (by default) part of the GameConfig interface  but can be added by developers that implement it on their own. Additionally, it would be neat if it took a type parameter for the player state.
Something like this:
export interface PlayerAPI<TPlayerState = any> {
  state: Record<PlayerID, TPlayerState >;
  get(): TPlayerState ;
  set(value: TPlayerState ): TPlayerState ;
  opponent?: {
    get(): TPlayerState ;
    set(value: TPlayerState ): TPlayerState ;
  };
}

Then a developer could do something like this:
interface MyPlayerState {
    teamID: string
}

interface MyGameConfig extends GameConfig {
    player: PlayerAPI<MyPlayerState>
}

const MyGame: MyGameConfig = {
    moves:  {
        drawCard: (G, ctx) => {
            const player = ctx.player.get();
            const teamID = player.teamID; // strongly typed!
        }
    }
}

It would actually be pretty neat if GameConfig had a type parameter too for game state
export interface GameConfig<TGameState = any> {
    endIf?: (G: TGameState, ctx: Ctx) => any;
	...

@crhistianramirez These are good proposals. Linking an earlier discussion along these lines: https://github.com/nicolodavis/boardgame.io/pull/579#issuecomment-603736732

@crhistianramirez I really like the idea of passing in the type of the user's game state as a template.  Would you have time to work on a PR for this?

@nicolodavis yes I can take a crack at it tonight!
Just so we're all clear on the scope of this PR. Here is what I plan on doing (feel free to add or remove from this list as you see fit)

Refactor Ctx interface to receive an optional type parameter TGameState and pass it down to anything that consumes it.
Refactor Ctx so the plugins are not part of the interface (can be added later by developers)
Refactor PlayerAPI to take in an optional type parameter TPlayerState

@crhistianramirez That sounds good to me!
Some extra thoughts, if you have the time:

Perhaps move the default plugins (events and random) to a DefaultPlugins interface and allow Ctx to take a type parameter too, e.g.:
type DefaultCtx = Ctx<DefaultPlugins>;

Game could also have a second optional type parameter for its context, so you could type plugins available in ctx (this is a little more work, and it can also be added later if it’s too much for this PR):
type CtxWithPlugins = Ctx<DefaultPlugins & MyPlugin>;

const game: Game<MyG, CtxWithPlugins> = {
  name: 'my-game',
  setup: ctx => {
    // will error and autocomplete for plugin APIs
  },
};

In general the types have not used T or I prefixes, so the type parameters can probably be G and C, i.e.:
interface Game<G extends any = any, C extends Ctx = Ctx> {}

or PlayerState/State in the player plugin.

Perhaps move the default plugins (events and random) to a DefaultPlugins interface

Is it common to use all default plugins? Alternatively we could simply export the Plugin interfaces and they could be composed in a similar way to your example.
type MyCtx = Ctx<PlayerPlugin & EventsPlugin & MyPlugin>

Also how do you feel about renaming them a bit? As a user I'd expect to find a Plugin type, not API.
EventsAPI => EventsPlugin
PlayerAPI => PlayerPlugin
RandomAPI => RandomPlugin
As I'm thinking about it more this is probably something that should be documented somewhere as well. I'm not the best copywriter but I'd be willing to take first pass at docs. Could also be a separate task. Thoughts?

Is it common to use all default plugins?

The Events & Random plugins are always included automatically in every game set-up, so these would be the default plugins. Player has to be explicitly added, so it should be separate.
Actually, given that Events & Random are always included, maybe it would be better if the Ctx exposed to programmers already includes their APIs so Ctx<PlayerPlugin> would include them and no need to remember the DefaultPlugins interface. What do you think?

this is probably something that should be documented somewhere as well

There should probably be a “Using with Typescript” guide at some point yes. It might be a little early as TS has only been supported as of a few weeks ago and things are likely to still change, but you could certainly make a start.

so Ctx would include them and no need to remember the DefaultPlugins interface

Yeah sounds good to me, simpler is definitely better. I'll plan on that.

things are likely to still change
That's a good point, I'll hold off for now then.

Thanks for the feedback! <3

Closing this as PR has been merged

@crhistianramirez Thanks again for your work on this!


chore: modify update script to avoid merge conflicts

By simply overwriting the history for the release branch, the update script is prone to merge conflicts when updating the release branch. This change modifies the update script to consider the release branch as the canonical branch, merging it into the release-ci branch with each sync update.

lgtm

/cherry-pick midstream-v0.7.1


[Utility] Don't override git's env if already present

Add our overrides for git when there is nothing already present.
rdar://problem/45842921

@swift-ci smoke test


Improve local golang pipeline example

Removing Mongo until it runs better locally on Apple Chips without hacks
Replacing ES consumer with JSON consumer in sample go pipeline so everything works end to end

Tested and now requires 0 tweaks to work fine.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>2
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 41.629%

Totals

Change from base Build<PHONE_NUMBER>7:
0.0%

Covered Lines:
3374

Relevant Lines:
8105

💛  - Coveralls


Usage in React Web app

Kindly suggest how i can use it in React Web app?
Thanks

If you mean React-Native-Web, you can try this
This is our fork to provide SvgXml and SvgCss for web.

@coderbintech See the documentation updates I added in #1625 –  it's possible that you'll just need to update your bundler to resolve .web.js files.

I think the needed information has been provided in the previous comments. I will close this issue then. Feel free to comment here if something is wrong and we can always reopen it then.


Premium SSD disks not deleted together with cluster

Hello, I've used the Azure Marketplace CycleCloud image to play with BeeGFS. Whenever I deleted a cluster, the associated Premium SSDs were not deleted (i.e. costs kept growing), while all other resources were removed. Not sure if this is the right place to report this...
Andreas

This issue will be fixed in the next release 8.2 (soon!)

Fixed in the Azure Portal. Tested today. Closing the case.


Feature/markdown editor/katex support

Checklist

[x] I tested the changes and all related features on the test server https://artemistest.ase.in.tum.de.
[x] I documented my source code using the JavaDoc / JSDoc style.
[x] ~I added integration test cases for the server (Spring) related to the features~
[x] I added integration test cases for the client (Jest) related to the features
[x] I added screenshots/screencast of my UI changes
[x] I translated all the newly inserted strings

Motivation and Context

Some lectures / exercises want to mathematical formulas in the problem statements, etc.
The idea is to include the well known katex lib to use this kind of typesets.
Description

This PR adds a command to the markdown editor to insert a katex formula template (see screenshot below).

The command is a domain command, because components that provide their own previews might not be able to parse it.
The parsing is done using showdown-katex
The implementation for remarkable-katex did not work properly for me (weird styling issues), therefore katex is not available for programming exercises atm (migrating the remarkable parser to showdown is out of scope for this PR)
I had to use a different DomSanitizer (DOMPurify), as the Angular Sanitizer destroyed the katex styles.

:warning: This PR affects the preview rendering in all files that use the artemisMarkdown service for rendering markdown into html. :warning:
Steps for Testing

Log in to ArTEMiS
Open Create/Edit Text-Exercise / Modeling-Exercise Dialog
Insert formula and check preview

Screenshots

Display on tutor exercise dashboard and course overview is buggy
But Preview works perfectly

This was a very good finding.
The issue is that a lot of places in Artemis are using the html preview without declaring the value as safe.
This way Angular will strip away content from it, or alter the structure.
I had to change the htmlForMarkdown method to also bypass the security (which is a good thing as now everything is in one place).
I had to to touch a lot of of places though to streamline this change, it should only have positive impact though.

@lovaiible
Thanks for your review :)
About your 2 points:

This was indeed an issue with the way the SafeHtml was given to the component (see first commit)
I found an easy way to use a latex mode for highlighting (second commit. see more information below)

The markdown editor is currently missing some key functionality so we can use it more:
There is no way atm for a command to set up the editor in a specific way (setting the mode, loading extensions, etc).
For this PR I added a way in which a command can set the mode of the editor, which is latex here.

The PR is done, however it can't be build anymore by Travis for some reason :thinking:

@krusche
The review items are done.


Create citation guide

Should have a piece of documentation to show people what to cite when they use this package. R2T2 was developed for this purpose, not sure if it's ready yet. Otherwise, have a document that says:

if you use anything in the package, cite here. If you use Metamer, cite here. If you use PortillaSimoncelli, cite here, etc

Closed by #219


Integrate email template in cli

Description

Integrated email template (for email body image, created a temporary png image for all format and deleted it before cli exists)

Issues Resolved
List any issues this PR will resolve, e.g. Closes [...].
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

https://user-images.githubusercontent.com/56703525/214382895-d9555a52-c42f-44f0-9a6c-53cbcfbbacf2.mov


[SR-537] Crash when printing self.dynamicType in a protocol extension constrained on Objective-C subclass

Previous ID
SR-537

Radar
http://www.openradar.me/24173309

Original Reporter
@jpsim

Type
Bug

Environment
Tested on "Apple Swift version 2.1.1 (swiftlang-7<IP_ADDRESS> clang-700.1.81)" and "swift-2.2-SNAPSHOT-2016-01-11-a"

Additional Detail from JIRA

Votes
1

Component/s
Standard Library

Labels
Bug, RunTimeCrash, Runtime

Assignee
None

Priority
Medium

md5: d6db625cc8d2b370e712c91d95a576af

Issue Description:
Cross-posted from rdar://24173309
Summary:
Accessing `self.dynamicType` in a protocol extension constrained on Objective-C subclass in Swift causes a bad memory access.
Steps to Reproduce:
Run the following code:
import Foundation
// doesn't crash when subclassing a non-Objective-C class, or not subclassing
class Class1: NSObject, Protocol1 {}

protocol Protocol1 {}
// doesn't crash without the type constraint, or with a different constraint ('Self: Hashable')
extension Protocol1 where Self: Class1 {
    func printType() {
        print(self.dynamicType)
        // also crashes:
        // _ = String(self.dynamicType)
        // self.dynamicType.description()
    }
}

Class1().printType() // EXC_BAD_ACCESS

Expected Results:
Accessing `self.dynamicType` in a protocol extension constrained on Objective-C subclass in Swift should return the calling type.
Actual Results:
Accessing `self.dynamicType` in a protocol extension constrained on Objective-C subclass in Swift causes a bad memory access.
Regression:
Tested on "Apple Swift version 2.1.1 (swiftlang-7<IP_ADDRESS> clang-700.1.81)" and "swift-2.2-SNAPSHOT-2016-01-11-a"

@slavapestov This is fixed, but I would rather supplement an existing test file than add a fixed crasher. Could you give me a tip on where to look? I found several candidates that have _typeName tests (test/stdlib/TypeName.swift, test/stdlib/RuntimeObjC.swift), but I'm not sure which one fits best.


Vendor extensions support

Support OpenAPI vendor extensions in this diff tools. Extensions can be supported by extending interface com.qdesrame.openapi.diff.compare.ExtensionDiff

This looks really interesting, thank you for adding it to the project.


Atom Helper Daemons consuming too much RAM

Prerequisites

[x] Can you reproduce the problem in safe mode?
[x] Are you running the latest version of Atom?
[x] Did you check the debugging guide?
[x] Did you check the FAQs on Discuss?
[x] Are you reporting to the correct repository?
[x] Did you perform a cursory search to see if your bug or enhancement is already reported?

For more information on how to write a good bug report or enhancement request, see the CONTRIBUTING guide.
Description
Atom creates multiple Helper daemons on startup, each taking up 150+ MB of RAM, causing my system to slow down and sometimes even hang, despite having 6 GB. Of course, the expectation is that I run other applications on my laptop as well.
Steps to Reproduce

Run Atom
Check the Activity Monitor for RAM usage of the Helpers.

Expected behavior: Atom runs smoothly.
Actual behavior: Atom eats up all my RAM and hangs.
Versions
You can get this information from executing atom --version and apm --version at the command line. Also, please include the OS and what version of the OS you're running.
Atom : 1.6.2
Mac OSX: 10.10.5

@varunagrawal I'm going to close this in favor of #1827.  Can you please follow these instructions listed in the debugging guide and report your findings to #1827?  Thanks.

@50Wliu thanks!


[gdax] gdax.json metadata file doesn't include BCH information

End of December 2017 GDAX added BCH to it's trading platform.
Minor fix, I've updated gdax.json locally, happy to push a 'Pull request'.

The edits are so minor, uploading gdax.json with the changes I propose.
gdax.json.gz

Actually, although my updated gdax.json file works in the short term, the gdax exchanges supports the /products request which seems to return the appropriate info.  It would seem querying the exchange would future proof the API better than expanding the hardcoded gdax.json file.


Bump node version in CI to > 16

Due to the Node 16 EOL:
actions/checkout: node 14 -> 20 (there was no 18 release)
pnpm/action-setup: still on node 16, but this bumps from 2.2.2 -> 2.4.0
actions/setup-node: node 16 -> 18

Thank you


Change sample rate and depth conversions to use unsigned long longs

For some godforsaken reason, Windows insists that longs should be 32 bits wide, and Linux (I think) thinks they should be 64 bits wide. I'm currently running the picoscope SCPI bridge in a WSL instance, and glscopeclient on the Windows side. This is introducing crashes, as the picoscope bridge is happy to serialize a long > 32 bits max, but glscopeclient freaks out and crashes since this is above it's perceived max long value. Changing the SCPI parsing functions to use stoull should fix this, since all systems look like they agree that unsigned long long's are 64 bits wide.

Please use uint64_t rather than unsigned long long (even if they're the same underlying type) to make "this is explicitly 64 bits" more obvious in the code.
Otherwise, good catch. I actually got some static analysis warnings about this recently and fixed some (but clearly not all) instances of similar behavior.

Are there "parse as unsigned long long" functions similar to stoull? I could have worded this clearer, but this is just a stol -> stoull swap.

Ah, OK. Not to my knowledge, this is probably good. Let me have a look.

Actually it's probably worth refactoring out the auto's and making uint64_t returns for stoull explicit, lemme do that


New chain onboarding check list

[x] add new chain config to functions/common/src/chain.ts (@zihaoccc )
https://github.com/HexlinkOfficial/Hexlink-Web/pull/71
[ ] add default token list for nova and nova testnet functions/common/src/token.ts (@mingyuan1219 )
[ ] deploy gnosis safe and all other smart contracts (@shu-hexlink )
[ ] deposit ETH to swap contract and set price (@shu-hexlink )
[ ] tune gas cost
[ ] test ERC721 display (NFT marketplace/NFT index)
[ ] deposit sender pool so senders have enough deposit to pay gas (cron job to check the balance and deposit)

tx to deposit sender pool: https://nova.arbiscan.io/tx/0x62a39f555f87128136a24713ad11e8cbaf7d1a6805a0ed7242b3d816234a4b93
Each sender got 0.001ETH

deposited to swap contract with 0.001ETH and set following prices for three tokens:

0xDA10009cBd5D07dd0CeCc66161FC93D7c9000da1: DAI, price = 2000 * 10^18
0x722e8bdd2ce80a4422e880164f2079488e115365: WETH, price = 1 * 10^18
0x750ba8b76187092b0d1e87e28daaf484d1b5273b: USDC: price = 2000 * 10^6

TODO:

figure out how to display/aggregate NFT info in Arbi Nova, need to explore quicknode api
explore how to estimate L1 gas cost in L2 so we can have better gas estimation during transaction
support refund/deposit so user will have more options to deal with gas sponsorship


False-positive RET504 "Unnecessary variable assignment before return statement"

ruff 0.0.187
import sys
from time import time

def func() -> float:
    result = time()
    sys.stdout.write('log smth\n')
    return result

$ ruff test_ruff_ret_504.py
Found 1 error(s).
test_ruff_ret_504.py:8:12: RET504 Unnecessary variable assignment before `return` statement

See also https://github.com/charliermarsh/ruff/issues/1233, which was closed recently.

I'm torn on this! Almost any statement can be side-effectful -- even something like a = b + c could actually involve operator overloading under-the-hood and thus make use of bar. So, I guess the suggestion is that we make this check much more conservative, and only flag when there are no expressions between the assignment and the return?

Which is not to say that this report/request is invalid, the question on how to deal with that sensibly is still open.

i think it's reasonable to assume that any statement is side-effectful - at least because it takes some time to execute

Alright, maybe we just change this to only allow obviously non-effectful statements (like pass) between assignment and return.

Going out in v0.0.189.


Any chance of other methods ?

for example, we can only create customers and we can't query already created customers.

I hadn't realised their API had expanded so much. I don't use this library for any personal projects anymore.
I'm happy to merge any any PRs that you make that add this functionality.


Does it support Arm64?

Our servers use arm64 cpus. Hope it can support as well.

Our servers use arm64 cpus. Hope it can support as well. +1

Hm. I don't have a machine to try this on.
Officially we do not package anything for non-x86 systems afaik.
If anyone has one they can share via SSH - I'll gladly poke around :).


Visible error code on questionnaire page

Save a questionnaire with an error in it, and there's an error code(?) visible above the error message:

Need to hide that flash message, it's used for tracking when people press save+publish.


Add unit tests for sticky lua module

What this PR does / why we need it:
This PR adds unit tests to sticky lua module. We need this because we don't have unit test coverage for this module.
Issue: https://github.com/Shopify/edgescale/issues/435

@ElvinEfendi Addressed your comments

@ElvinEfendi Updated the tests. Removed resty.chash mock since we now have direct access to it.

this was completed at https://github.com/kubernetes/ingress-nginx/pull/2966


localhost not showing new article correctly

Story
No response
Description
I am trying to test a new article on localhost. Unfortunately it does not render correctly on the website. It does render correctly on Rob's localhost.

@iskerrett Can you push it up as PR? That way others can check it out and debug. But this might be a local issue for you?

It appears I have managed to blow away the branch I was working on. I got into a merge conflict. I will close this issues since I need to start again on the article.

fwiw, I was able to recreate the article and the problem went away. Not really sure what changed but things are working now.


Support for non-standard macros \class and \cssId

Hi!
Support for the TeX HTML extension from MathJax in the output of KaTeX would be nice.

Excerpt from the docs:
The HTML extension gives you access to some HTML features like styles, classes, element ID’s and clickable links. It defines the following non-standard macros:

\class{name}{math}: Attaches the CSS class name to the output associated with math when it is included in the HTML page. This allows your CSS to style the element.
\cssId{id}{math}: Attaches an id attribute with value id to the output associated with math when it is included in the HTML page. This allows your CSS to style the element, or your javascript to locate it on the page.

@kevinbarabash How do we feel about bringing this back from the dead?
I know there's a few people who are looking for it.
If you're okay with the concept, I can see what I can about the implementation.

@rrandallcainc since this is non-standard it should probably go in contrib/.  In order to do that though, we need a way to add functions from outside of KaTeX core.  You could expose defineFunction as __defineFunction.  There should also be automated tests for this so that we can catch any changes to defineFunction that break these commands in contrib/.

@kevinbarabash With strict mode in work, I think we can revisit this and consider enabling them in non-strict mode instead of contrib.


fix(patch-list): show all patches, not only "open" ones

Closes #105

Results:
🩹 Fixes

patch-list: show all patches again as expected, not only those with status "open". Addresses regression caused due to a breaking change in Radicle HTTP API (httpd) while keeping backwards compatibility for users who haven't yet upgraded to the latest httpd


Make palette accessible

When looking at the demo here https://simonwep.github.io/pickr/ the pickr is almost accessible and navigable using keyboard.
It is also possible to select a color from palette, however you can't see which color is active (has focus). Only the active/selected color is highlighted with a border.

Related to #11


Azure provisioning in the dashboard

This change makes it possible (and cleaner IMO) to publish and log data about a resource to the dashboard and does that to show errors when creating resources and for showing provisioning progress when using the Aspire.Hosting.Azure.Provisioning package.  This change consolidates the previously made annotations for publishing and subscribing to both resource updates and logs general purpose services instead of annotations.
TODO:

[ ] - Make all azure resources work like this (I only did cosmosdb so far)
[x] - Add some more tests for logic in ResourceNotificationService.

Fixes #2396
Contributes to https://github.com/dotnet/aspire/issues/1758
Microsoft Reviewers: Open in CodeFlow

/azp run


Delete obsolete online interactive tutorial page

This PR deletes the entire tutorial pages which uses the already removed interactive tutorial and its not been maintained in some time. I have made this change based on the suggestion in the issue to delete the tutorial altogether.
Closes https://github.com/kubernetes/website/issues/42401

/approve


Default to CFs Addresses

Expected Behavior
As we deal with the ProxyFactory deployment issues, it may be best to default to the address that counterfactual uses for the deployed contracts in the migrate-contracts.js otherwise, we may have to remember to hardcode the updates into our address book whenever there are changes there (which is not that fun).
Current Behavior
All contracts are pulled from the relevant cf dependency, which seems to work fine for everything except the ProxyFactory contract. We end up having to hardcode the address into the address book, and we often forget to make sure these are in sync.
Instead, we should make sure that we default to using the cf-deployed contracts on networks where possible.
Also open to figuring out how we can replicate their deployment in our system so we can keep the current setup.

Fixed as part of mainnet contract deployment


Replace website main color with CSS custom property

Currently in assets/css/index.css we're using #3a91ff for the main color. We are using this color for a couple of attributes, and we probably will be using it more.
Create the file assets/css/variables.css and add in it custom property --main-color on :root with the value #3a91ff then import this file in the beginning of assets/css/index.css and replace #3a91ff throughout the file with var(--main-color)

Can I try to work on this ?

Sorry, this one is for first-timers only! You can work on anything that doesn't have the first-timers-only label and that no one is assigned to.

@shahednasser oohh okay didn't see the label..

Hi
Can I try to work on this?

Hi @NirajGohel yes go ahead

Resolved by #41


winutil can't detect oscdimg present on the system

Describe the bug
a paradox
To Reproduce
Steps to reproduce the behavior:

Download oscdimg from official microsoft source: https://learn.microsoft.com/en-us/windows-hardware/get-started/adk-install
Go to MicroWin
Click on Select Windows ISO
See error

Expected behavior
if (!$oscdImgFound)
    {
        $downloadFromGitHub = $sync.WPFMicrowinDownloadFromGitHub.IsChecked

        if (!$downloadFromGitHub)
        {
            # only show the message to people who did check the box to download from github, if you check the box
            # you consent to downloading it, no need to show extra dialogs
            [System.Windows.MessageBox]::Show("oscdimge.exe is not found on the system, winutil will now attempt do download and install it using choco or github. This might take a long time.")
            # the step below needs choco to download oscdimg
            $chocoFound = [bool] (Get-Command -ErrorAction Ignore -Type Application choco)
            Write-Host "choco on system: $chocoFound"
            if (!$chocoFound)
            {
                [System.Windows.MessageBox]::Show("choco.exe is not found on the system, you need choco to download oscdimg.exe")
                return
            }

            Start-Process -Verb runas -FilePath powershell.exe -ArgumentList "choco install windows-adk-oscdimg"
            [System.Windows.MessageBox]::Show("oscdimg is installed, now close, reopen PowerShell terminal and re-launch winutil.ps1")
            return
        }

to not be triggered
Screenshots

Additional context
I'm on windows 10 x64, wanted to update to a debloated windows 11, saw this utility and desided to give it a try, my guess is that this error caused by having 2 drives???

Out of curiosity, can you move oscdimg.exe into TEMP and try once more?

it worket after multiple fails, thanks

Could you shed light? I can't get it to work.....


fix /debug/telemetryz

Please provide a description of this PR:
fix https://github.com/istio/istio/issues/37218
sample output:
kubectl exec -it istiod-6d985c8cc5-8fg28 -nistio-system -- pilot-discovery request get /debug/telemetryz

{"telemetries":{"namespace_to_telemetries":{"istio-system":[{"name":"mesh-default","namespace":"istio-system","spec":{"accessLogging":[{"providers":[{"name":"file-log"}],"filter":{"expression":"response.code \u003e= 500"}}]}}]},"root_namespace":"istio-system"}}

To help us figure out who should review this PR, please put an X in all the areas that this PR affects.

[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[x] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

Please check any characteristics that apply to this pull request.

[x] Does not have any user-facing changes.

Can you post the result of a sample request to the endpoint in the desciption?

@zirain: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-cni_istio
cd6d8d26c176cf44c9dd27c72badd2e48c64782a
link
true
/test integ-cni_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zirain: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-cni_istio
cd6d8d26c176cf44c9dd27c72badd2e48c64782a
link
true
/test integ-cni_istio

integ-distroless_istio
cd6d8d26c176cf44c9dd27c72badd2e48c64782a
link
true
/test integ-distroless_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Can you post the result of a sample request to the endpoint in the description?

updated in the PR description.

/retest-required


Adding visualization and XAI scripts.

Adding visualization and permutation importance scripts.

I just merged Gabrielle's branch so go ahead and pull from main.


Does not recognize methods when i call modal in the top of livewire component

{{ __('Add plan') }}

{{ __('All plans') }}{{ count($plans) }}

{{ __('Name') }}

{{ __('Amount') }}

{{ __('Currency') }}

{{ __('Description') }}

{{ __('Action Status') }}

{{ __('Actions') }}

                    @foreach ($plans as $plan => $value)

                                    {{ $value['name'] }}

                                {{ number_format(floatval($value['amount']) /100,2,',','') }}

                                {{ $value['currency'] }}

                                {{ $value['description'] }}

                                @if ($value['active'] == 1)

                                @else

                                @endif

                    @endforeach

I need more information and code samples, otherwise, I have no idea how to help.

Hello sir
for exemple when i call the method to open the modal in the top of my livewire component and do some other methods under the first call, all the methods are ignored
for exemple
<button class="text-white bg-blue-500 btn hover:bg-blue-600" wire:click="$emit('openModal', 'plans.create')">
        <x-heroicon-o-plus-sm class="h-5"/>
        {{ __('Add plan') }}
</button>

{{ __('All plans') }}{{ count($plans) }}

                        ....

{{ __('Action Status') }}

                        ....

                    @foreach ($plans as $plan => $value)

                        ...

                                @if ($value['active'] == 1)
                                <x-heroicon-o-eye class="h-5 text-green-500"/>
                                @else
                                <x-heroicon-o-eye-off class="h-5 text-red-500"/>
                                @endif

                        ...

                    @endforeach

for exemple method delete doen't work
thank you

Thanks for posting some more information. Unfortunately, it's still difficult for me to assess what the problem is. If you could create a Github repository with demo-code and step-by-step instructions on what you are doing and what is not working I'm able to help you.

Closing for now, let me know if this is still an issue.