fatal error: 'cstddef' file not found

zdsoft@looxoojoo:~$ brew install --HEAD usbmuxd
==> Cloning https://git.sukimashita.com/libusbmuxd.git
Updating /Users/zdsoft/Library/Caches/Homebrew/usbmuxd--git
==> Checking out branch master
Already on 'master'
Your branch is up to date with 'origin/master'.
HEAD is now at c75605d Return meaningful error codes from usbmuxd_connect()
==> ./autogen.sh
==> ./configure --disable-silent-rules --prefix=/usr/local/Cellar/usbmuxd/HEAD-c75605d_1
==> make install
Last 15 lines from /Users/zdsoft/Library/Logs/Homebrew/usbmuxd/03.make:
In file included from /usr/local/include/string.h:25:
/usr/local/include/plist/Node.h:26:10: fatal error: 'cstddef' file not found
#include
^~~~~~~~~
1 error generated.
make[1]: *** [collection.lo] Error 1
make[1]: *** Waiting for unfinished jobs....
In file included from socket.c:28:
In file included from /usr/local/include/string.h:25:
/usr/local/include/plist/Node.h:26:10: fatal error: 'cstddef' file not found
#include
^~~~~~~~~
1 error generated.
make[1]: *** [socket.lo] Error 1
make: *** [install-recursive] Error 1

这问题是是brew 环境的问题
用命令: brew doctor 诊断一下，
然后根据提示把没有作用的头文件全部删除掉，在重新安装应该就没问题了。
原因： 很可能是其他无用的库和头文件导致了cstddef 这个文件的冲突。

currently, i am facing the same issue.
can anybody suggest me how to fix it.

currently, i am facing the same issue.
can anybody suggest me how to fix it.

Wrong header file was placed at /usr/local/include .
run brew doctor  and delete Unexpected header files
for this case just remove
/usr/local/include/string.h

Hi, could you please confirm if this problem still exists with the latest versions on stable or master?
Also does this happen on Android and iOS?
Thanks a lot

Without additional information, we are unfortunately not sure how to resolve this issue. We are therefore reluctantly going to close this bug for now. Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.


Setting up database on AWS

Setting up database on AWS
Steps implemented in Order (30-07-24)

Setup a postgresql Database RDS Database AWS
Connected it to local instance using postgresql library
Uploaded data in the format with primary keys and secondary keys
Updated Django Database Connections in Settings.py
Created Models.py, Serializers.py and updated views.py

Current State

Data is accessible through the cmd interface after connecting it to the server using the AWS provided host link
Data is not accessibile through the django setup backend

@anmolsingh0219 let's try the following 2 experiments to try and scope the bug.

From the ec2 instance running your django app, execute

$ docker run -it --rm postgres psql -h jaltoldatabase-1.cnyqeok8kne6.ap-south-1.rds.amazonaws.com -U admindatabase -d postgres -p 5432

If that works, try deploying a very basic django app.

Say something that writes a timestamp to the db everytime you visit /, and reads it back to print on the screen.
# myproject/models.py
from django.db import models

class Visit(models.Model):
    timestamp = models.DateTimeField(auto_now_add=True, verbose_name="Timestamp")

    def __str__(self):
        return f"Visit at {self.timestamp}"

# myproject/urls.py
from django.contrib import admin
from django.urls import path
from django.http import HttpResponse
from .models import Visit

def hello_world(request):
    Visit.objects.create()
    recent_visit = Visit.objects.latest('timestamp')
    return HttpResponse(f"Hello, world! {recent_visit.timestamp}")

urlpatterns = [
    path('', hello_world),
]

Run that django project in docker the same way, maybe something like
$ docker build -t my_django_app .
$ docker run --name my_django -p 8000:8000 -v $(pwd):/app -d my_django_app
$ docker exec -it my_django python manage.py migrate
...

That is 90% real code, 10% pseudocode.
If it's confusing lmk and ill upload my test project.
Make sure your settings.py has the same info in (1)
Depending on which one of those fails, we'll know how to proceed.

RDS works.

standalone script using django models to populate file based json data into db

Can you close this issue?


Missing layout confirmation

Microsoft PowerToys version
0.25.0
Running as admin
Area(s) with issue?
FancyZones
Steps to reproduce
After pressing the hotkey to select zone layout I can easily select a new layout, but then what?...
✔ Expected Behavior
Some kind of way to confirm my selection and close the layout selector.

Double click on a layout to select it and close the dialog
A confirm or close button

❌ Actual Behavior
Only the X in the top right corner closes the windows, but there is no clear indication that my selection is preserved then.
Other Software
No response

Just a thought: should the editor open at all?

@niels9001 I think he is talking purely on win+ `` , not the hot swap of ctrl+win+alt+Number
@hultqvist As a lot of our software is designed around instant responses, just like the Windows settings and PowerToys settings system. Your background has an overlay that does reflect the change of layout on selection.
I could see a double click to instantly confirm but i don't agree with the confirmation as it doesn't fit any of the interaction models we hold in the entire suite.
I'd like to see some community feedback on this interaction shift as i do see people do double clicks when a single click is appropriate

@crutkas Ah yes, I misunderstood.
@hultqvist for the layout editor, we follow modern application standards, meaning direct, single-click interaction. This should reduce the amount of clicks needed to switch between layouts. I'd be interested to learn what feedback you'd like to see (e.g. is the preview not clear enough? Do layouts get applied incorrectly?).
I wouldn't be in favor of a double click, since this is only used in explorer.exe. Click + apply button would be the better solution then - or a dedicated apply button on each layout card itself.

For double click to apply and exit, the tracking issue is https://github.com/microsoft/PowerToys/issues/6098

@hultqvist If this is right,#6098 is the tracking item.

@hultqvist If this is right,#6098 is the tracking item.

Yes it's discussion covers this one.


feat: add notification if prisoner on csip already

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

Project coverage is 90.92%. Comparing base (9036047) to head (8f9efc0).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #348      +/-   ##
==========================================
+ Coverage   90.87%   90.92%   +0.04%
==========================================
  Files         349      349
  Lines        5324     5331       +7
  Branches      770      770
==========================================
+ Hits         4838     4847       +9
+ Misses        471      469       -2
  Partials       15       15

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


"-dce full" flag crashes the Android Application

Hi there,
When I use -dce full haxeflag in my project (even it's empty) and exported to android, the app immediately crashes when opened.
When I delete the flag, it runs normally.
Thanks.

Just tested with current builds on "Pirate Pig", and it worked fine. Marking as closed :smile:


Simple enhancements to the networking spec discovered after implementing it

The Nimbus team has just completed the implementation of the currently published LibP2P networking spec. This gave us a chance to think through some of the details in the spec and several practical issues were discovered. In this PR, I'm trying to address them with minimal modifications to the spec:
Problem 1: The messaging described in the spec follows the client-server paradigm too much.
In the current spec, one of the peers sends requests that are identified by a numeric request id and the other peer replies with a response referencing the same id. The request and response structures are defined in a way that makes it impossible to discriminate requests from responses on the wire and this forces the peers into a more traditional client-server relationship where the one side sends only requests and the other side sends only responses. This is not quite appropriate for a peer-to-peer network such as Ethereum where both peers are expected to make requests.
Few possible ways to solve this problems were considered:

Assign even IDs to the requests and odd IDs to the responses
This seems to be the least invasive change and it's the one being implemented in this PR.
From our experience, It introduces only a very slight implementation complexity cost.

Open two streams per connection and define the client-server roles within each stream.
This seems to require a much higher implementation complexity cost.

Assign separate method id to the responses.
This would have been easy to implement as well, but it would increase the size of the response structure that will need to store the additional method id.

Include a request/response bit in the enclosing message frames.
Another easy option that was discarded purely on subjective grounds.

Problem 2: The high-level spec is not taking full advantage of LibP2P.
One of the benefits we hoped to reap with LibP2P is the ability to stream very large responses asynchronously as the data required to serve them is gradually fetched from some slower I/O device. In LibP2P, this is possible due to the central concept of stream-multiplexing. The requesting peer can initiate a request by opening a stream and the serving peer can gradually write data to that stream as it becomes available (i.e. without the need to buffer large amounts of data in memory in order to send it in a single response message).
Before Multistream-2.0 is implemented, the use of multiple streams in LibP2P introduces some overhead, which informed the current design of the spec relying on a single stream. Nevertheless, we believe it's easy to emulate the presence of a stream-like device by introducing support for "Chunked responses". This gives us a valuable high-level protocol design tool and prepares us for a future where all requests and responses can be mapped cleanly to the Multistream-2.0 streams.
Other minor changes:

A timeout has been specified in the handshake procedure.
This should prevent a simple DoS attack trying to exhaust the number of open sockets by a client.

A number of additional disconnection reasons were added.
While we were implementing the spec, we noticed that many of the typical checks and safe-guards present in our ETH1 devp2p stack are still relevant for ETH2. Adding explicit disconnection reasons for them serves as a reminder what a typical client should implement.

cc: @AgeManning

Yep there has been discussion about this offline.
I believe the consensus is to use substreams per request.
In this scenario, a new substream is opened on each request and it awaits a response before closing the substream.
Lighthouse will be implementing this approach and we should have a greater visibility on this in a week or two. I imagine the RPC Id's just need to be unique in this case.

We also agree that dedicating a separate LibP2P stream for each request is the approach that follows the architecture of the LibP2P most closely. Since our own implementation is quite flexible, we've included support for both schemes. Here, I'll try to provide a precise specification for how requests can be mapped to streams:
libp2p protocol names
A "Protocol ID" in libp2p parlance refers to a human-readable identifier libp2p uses in order to identify sub-protocols and stream messages of different types over the same connection. Peers exchange supported protocol IDs via the Identify protocol upon connection. When opening a new stream, peers pin a particular protocol ID to it, and the stream remains contextualized thereafter. Since messages are sent inside a stream, they do not need to bear the protocol ID.
RPC-over-libp2p
Terminology:

Message name -- each RPC request is identified by a name consiting of English letters, digits and underscores (_).

Message schema -- the message schema defines the data types that MUST be transmitted with each request and response.

Schema version -- a semantic version consiting of one or more numbers separated by dots (.). Each schema is versioned to facilitate backward and forward-compatibility when possible.

Encoding -- while the schema defines the data types in more abstact terms, the encoding describes a specific representation as bytes that will be transmitted over the wire.

Protocol Prefix -- the RPC messages are grouped into families identified by a shared LibP2P protocol name prefix. A conforming implementation is expected to support either all messages appearing within a family or none of them.

Request -- Requests are initiated by opening a LibP2P stream with the following protocol ID:
<ProtocolPrefix>/<MessageName>/<SchemaVersion>/<Encoding>
After opening the stream, the initiator MUST transmit within 3 seconds the serialized contents of the message, according to the specified schema version and the encoding being used.

Response code -- a single-byte prefix determinig the contents of the response. It can have one of the following values:

0: Success -- a normal response with contents matching the expected message schema and encoding specified in the request.

1: EncodingError -- the receiver was not able to decode and deserialize the data tranmitted in the request. The response content is empty.

2: InvalidRequest -- The contents of the request are semantically invalid. The response content is a message with the InvalidRequestMsg schema.

3: ServerError -- The receiver encountered an error while processing the request. The response content is a message with the ServerErrorMsg schema.

Some requests MAY use response codes above 128 to indicate alternative request-specific responses.

Response -- After receiving a request, the responding party MUST reply on the same stream within 10 seconds with a response code, followed by the contents of the response.
Certain RPCs MAY allow for multiple responses to be generated for each request (e.g. publish/subscribe schemes). The responder MUST transmit the Success response code before each serialized message written to the stream. The responder MUST close the stream when no more messages are expected. If an error is encountered after some messages have already been written, the responder MUST transmit the ServerError response code, followed by a serialized mesage using the ServerErrorMsg schema.
The request initiator MAY close the streat at any time.

Internal schemas:
InvalidRequestMsg
(errorMessage: string)

ServerErrorMsg
(errorMessage: string)

The protocol prefix for all currently defined RPC messages used for syncing the beacon chain is /ETH/BeaconChain.
The supported encodings are SSZ and SSZ+Snappy.
The names of the messages are the ones specified in the rpc-interface spec.
All schema versions start as 1.

I'm finally getting around to implementing this.
I'm not sure about go, but in rust, implementing a protocol-id per request is quite difficult. All other protocols that I've seen in libp2p that use RPC-like requests are grouped under a single protocol-id. With this being said, I can see the benefits of using multi-stream select to do the RPC message-type segregation.
An alternative, would be encoding the type in the data sent (although, apart from ease of implementation, and not polluting the protocol-id namespace, I don't have strong arguments for doing this).
Hoping @raulk has some insights on this.
Finally, in your implementation, how does the sending node communicate it has completed the request and is closing the stream?

Generally one protocol per message type (x2 if you model request and response separately, like I’ve seen in some places) introduces overhead for little gain in exchange. I’d recommend coarser protocols, one per functional area of the system. With multistream 1.0.0 you’d be incurring in the negotiation cost per stream; this will be death by a thousand cuts now. Multiselect 2.0 will make this much cheaper (negligible). I recommend you stick to a single long-lived stream per protocol, and move to a single stream per request once we have multiselect 2.0.

Thanks for the input @raulk!
It seems in rust we are doing a multi-stream select on each substream regardless so for me there will be minimal extra overhead in building it this way. However, another point was raised, that typically a protocol-id will have a version associated with it. With a protocol-id per request, we may potentially run into more complex versioning, where individual requests now have their own version.
Although the multistream-select overhead is a non-issue for my particular implementation, it seems more logical to localize an entire protocol to a specific protocol id. This way other implementations don't have extra over-heads, we can version all the requests in a single id and the protocols that a node support will be simpler.
Furthermore, I think we should use size-prefixed messages which will indicate the end of a stream, prevent DOS vectors and allows for slower implementations to buffer and send responses gradually over the stream.
What are you thoughts on this @zah?

The original PR here tries to provide the minimal modifications needed for the spec in order to make the approach suggested by @raulk workable (that is, using a single long-lived stream over which both peers can send requests freely).
The rest of the discussion is a bit more forward-looking. It tries to identify what would be the best approach in a future where Multiselect-2.0 is available. If this future is sufficiently close, I think it would be worth simplifying the client implementations now at the cost of some transient overhead brought by the current sub-stream negotiation.
Our own client currently implements both the minimally modified spec proposed here and this forward-looking alternative spec as an experiment. We've done some field testing with both approaches in our public testnet and it's clear that the implementation code is slightly simpler when more of the complexity is pushed over to libp2p.
@raulk, can you further clarify the trade-offs between finer and coarser sub-protocols, assuming Multiselect-2.0 is available? It's clear that the advertisement messages will balloon in size, but what do you mean when you say "correlation and state keeping"? Perhaps Multiselect-2.0 can be further extended to allow for a family of sub-stream types to be negotiated just by transmitting an identifier describing the whole family.

@AgeManning, I've missed to mention that all SSZ messages have a size prefix indeed. This is an inherent requirement due to the way SSZ deserialisation works and our current implementation of the alternative spec uses a 4-byte length. The clients are definitely allowed to close the stream after reading the expected response.

I agree with the protocol segregation.
I've tried to consolidate everything discussed here in #1281. I've specified the protocol-ids for each message, added some clarity in some sections.
I have used /eth/serenity as the protocol-prefix to be consistent with other libp2p protocols, and have used an unsigned varint to length-prefix the contents of the RPC requests/responses.
Please let me know if there are still discrepancies between your current implementation.

closing in favor of #1281


the derived table store constantly updates, causing costly re-renders

I am noticing that the derived table / model store gets updated a lot, most likely due to the syncing. I believe it's the lastUpdated field that's changing. This is causing a lot of rerenders in the assets pane that I'm noticing in this PR. How do you think we should control for that? It seems like if an entity isn't really updating, we shouldn't update the lastUpdated field.
I'm wondering if this pattern of automatically updating the lastUpdated field is worth it. This is the second time that doing it automatically has caused issues downstream – the first being when we attempt to import a table but the duckdb import fails, we need to unwind the state change, but we've automated the lastUpdated field so the timestamp is not actually reflective of the last update. I think we should consider having an explicit function for updating that field and perhaps consider turning it off by default.

Chatted with Aditya about this – we've got a fix for this and I'm going to take this on. There are two issues:

an active entity (like a model) – this is entirely a frontend fix. Essentially, $store.activeEntity.lastUpdated updates a ton, and we reactively pull out the active model by listening to $store.activeEntity.id, but the whole object updates a bunch. The solution is practically trivial – just pull out the id and let Svelte's basic primitive reactive values do the rest (e.g. $: activeEntityID = $store?.activeEntity?.id will run, but all downstream references to activeEntityID do not unless the variable value changes).
tables – we just need to move the entity status change to after the check.


SEARCH-665 Added support for http status code 301

Coverage increased (+0.5%) to 100.0% when pulling 54f2579d828c026610f9a7e1e76b540fe4f8a862 on smolowik:bugfix/SEARCH-665 into 591936e16886930bf7a207c54037a53a49af334f on simgroep:master.


chore: remove debug

Hi 👋, I'm sorry to just delete the line, but since there is not logging system builtin yet, I prefer to make things simple and let you choose how you want to provide verbose logging later
Motivation: each time I run gci against my project, I got all the following lines, and since my plan is to run gci often (githooks, Makefile pre-rule, etc); I clearly need a default mode that only prints real problems.
I know that I can hack using pipefail or 3 step commands in Makefile, but it makes things a lot more complex.
foo@bar:~ $ go run github.com/daixiang0/gci -w -local berty.tech .
skip file cmd/berty/doc.go since no import
skip file cmd/berty/mini/doc.go since no import
skip file cmd/rdvp/doc.go since no import
skip file framework/bertybridge/doc.go since no import
skip file internal/cryptoutil/doc.go since no import
skip file internal/discordlog/doc.go since no import
skip file internal/grpcutil/doc.go since no import
skip file internal/handshake/doc.go since no import
skip file internal/ipfsutil/doc.go since no import
skip file internal/multipeer-connectivity-transport/addr.go since no import
skip file internal/multipeer-connectivity-transport/driver/bridge_unsupported.go since no import
skip file internal/multipeer-connectivity-transport/driver/mc-driver/cgo_bridge_darwin.go since no import
skip file internal/multipeer-connectivity-transport/driver/mc-driver/cgo_bridge_unsupported.go since no import
skip file internal/notification/manager.go since no import
skip file internal/notification/manager_noop.go since no import
skip file internal/testutil/doc.go since no importskip file internal/tinder/doc.go since no import
skip file pkg/banner/doc.go since no importskip file pkg/bertymessenger/doc.go since no import
skip file pkg/bertymessenger/version.go since no importskip file pkg/bertyprotocol/doc.go since no import
skip file pkg/bertytypes/doc.go since no importskip file pkg/bertytypes/events_account.go since no import
skip file pkg/errcode/doc.go since no importskip file pkg/errcode/stdproto.go since no import

Thank you for this project 👍

Thanks for your contribution, now it is fixed.


Editing Feature for Meals

User Story

As a user
I want to edit my recipes
so that I can add or change something on a meal

AC

edit icon on MealCard
when clicking on icon a edit form is opened

Design/Wireframe
Edit Form:

Edit Icon on MealCard:

Tasks

[ ] create new branch
[ ] create new component EditForm
[ ] add pen icon to MealCard
[ ] create editing state
[ ] add onClick function to edit icon that sets editing state and renders EditForm
[ ] create function for submit in Edit Form that sets new input in edited meal
[ ] write test for form

Complexity
medium

Hi Laura,
good job! 👍 I only have three remarks:

I think the "Edit" button should be a "save" button as you want to save the changes after editing via this form or?
I think it would be more realistic to show that the input fields are filled out as this is the edit feature 🤔
It could be nice to know where the edit icon is. Maybe you could add this to your description via a second wireframe that shows where the edit icon is. :)

If hope that my remarks are helpful for you! If you have further questions, feel free to ask me! :)

Hi Laura,
yes! Now it is better to understand! :) I approve this user story. ✅


Prepare Teams App Dependencies returns error

Module: LAB_01
Lab/Demo: LAB_01
Task: Exercise 1 - Create a message extension, Task 3 - Prepare resources
Step: VS step 2
Run "Prepare Teams App Dependencies", click Continue, VS prompted error, output windows show:
12:48:55:141	{
12:48:55:141	  "errorType": "SystemError",
12:48:55:141	  "source": "Fx-VS",
12:48:55:141	  "name": "UnknownError",
12:48:55:141	  "message": "Unable to cast object of type 'Microsoft.VisualStudio.CommonIDE.Solutions.ProjectSystems.StubHierarchy' to type 'Microsoft.VisualStudio.Shell.Interop.IVsProject'.",
12:48:55:141	  "stack": "   at TeamsFx.VisualStudio.Helper.HierarchyHelper.d__4.MoveNext()\r\n--- End of stack trace from previous location where exception was thrown ---\r\n   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()\r\n   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\r\n   at TeamsFx.VisualStudio.Commonlib.PrepareLocalDebug.d__0.MoveNext()\r\n--- End of stack trace from previous location where ...
12:48:55:141	  "innerError": null,
12:48:55:141	  "userData": null,
12:48:55:141	  "timestamp": "2024-09-05T12:48:55.1315891+08:00",
12:48:55:141	  "displayMessage": "Unable to cast object of type 'Microsoft.VisualStudio.CommonIDE.Solutions.ProjectSystems.StubHierarchy' to type 'Microsoft.VisualStudio.Shell.Interop.IVsProject'.",
12:48:55:141	  "helpLink": null,
12:48:55:141	  "issueLink": null
12:48:55:141	}

Hi @ningweinw,
Please make sure that when you are entering the code snippets into Visual Studio, that they are copied exactly as they appear.  You may need to copy them into Notepad first and then into Visual Studio.  It's an issue with the VM environment and Visual Studio.
These errors are telling me there is a missing or extra '}' character somewhere.


Lovely folk

+1 for stickers

you need to comment on the issue here: https://github.com/GH-Event-Demos/random-name-picker/issues/13


vendor: update seccomp/containers-golang to v0.4.1

Signed-off-by: Giuseppe Scrivano<EMAIL_ADDRESS>

Vendor test is failing, but LGTM once that's resolved

/retest

/retest

/lgtm


/hopper.lua:407: bad argument (table expected, got nil) (BETA BRANCH)

cmd: hopper *hopper* *barrel*

does not instantly appear, I am using a string duper thats putting string in the hoppers, if that helps.

The error message means it's failing to do a .list() on either a hopper or a barrel, which shouldn't be happening in the first place so I have no clue what could be the cause.
Does the hopper continue to hopper even after the error appears?

Update: I made errors disappear from the screen if they don't occur again on the next iteration. This doesn't fix your issue since it is still erroring, and it's erroring frequently enough to affect operation speed.
I'll close the issue for now and will reopen it if more information appears.


Move OME-Zarr opening to MoBIE plugin tree

@d-v-d
@joshmoore
@constantinpape
@bogovicj
I feel I would like to move
Plugins › BigDataViewer › OME-Zarr › Open OME-Zarr From S3...
Plugins › BigDataViewer › OME-Zarr › Open OME-Zarr From File System...
Plugins › BigDataViewer › OME-Zarr › Open OME-Zarr From S3 with Credentials...
to
Plugins › MoBIE › OME-Zarr › Open OME-Zarr From S3...
a.s.o.
because OME-Zarr Java support is still under discussion and development and I am not sure whether what we currently do in mobie-io will be the default mechanism in the future.
Would you be fine with this?

Would you be fine with this?

Yes, but I don't really have a horse in the race. Will macros, etc. break?

Will macros, etc. break?

I don't think so as macro calls are not recording the full path but only the last bit, which would not change.

I don't have a strong opinion here.

I moved it to within MoBIE.


How to interprete MUC status codes?

When I am using JoinMUCNoHistory to join a MUC the returned status is different from the ones specified here. E.g. after a successful join I got 140 which is not listed there.
Do I have to do some transformation with the status returned from the function?

I'll close this issue as I misinterpreted the return values meaning.


Export types

Can you export types like CreateEmailResponse?

I agree, here is what I did in the meantime.

const resend = new Resend();
type Resend_Send = Parameters<(typeof resend.emails.send)>
export type CreateEmailOptions = Resend_Send[0];
export type CreateEmailRequestOptions = Resend_Send[1];

+1
I'm currently needing CreateBatchOptions since to effectively use the functionality a list should be created first, but we have no way to type the list other than the hackery shown above.

#396 appears to resolve this

Thanks @robert-bo-davis - not sure why I was unable to find these exports before, maybe I finally bumped to a new enough resend js library version.


Add support for Windows

Windows support to Gloo
Windows supports for DistributedDataParallel
Pls check and make some test

Pytorch supports DistributedDataParallel on windows after 1.7. I have tested single GPU training on windows. But I don't have multi-gpu PC with windows. I may test it later.


"Has a Dispersion Point" should exclude spaces of size <= 2?

"Has a Dispersion Point" (https://topology.pi-base.org/properties/P000045) says that a dispersion point is one whose removal makes a connected space totally disconnected. But this appears to mean that any connected space of size 1 or 2 has a dispersion point, since any space of size 0 or 1 is totally disconnected (https://topology.pi-base.org/properties/P000047). This contradicts, for example, "Indiscrete Topology on a Two-Point Set" (https://topology.pi-base.org/spaces/S000004/properties).
Maybe we should follow Wikipedia (https://en.wikipedia.org/wiki/Dispersion_point) and say that "Has a Dispersion Point" also requires the space to have at least 3 points.

Good call. #471 will say

Totally Disconnected and Has distinct points

mirroring https://topology.pi-base.org/theorems/T000052

I am not sure I necessarily agree with this change of the definition of P45.  A few observations:
(1) We all agree that the interesting cases are spaces with a good number of points.  The spaces with very small (<= 3) cardinality are fringe cases.  Whether we want them covered or not by some definitions does not really matter.  We either specify explicitly that some small spaces (even though they match the definition) will not have the property.  Or we don't specify that.  It's a matter of either keeping the definition simple, or of making some theorems simpler.
(2) Wikipedia is not an authoritative source of information, unless they are backed up by reliable sources.  In this case, https://en.wikipedia.org/wiki/Dispersion_point has no inline citation to back up their choice of requiring a space with a dispersion point to have at least 3 points.  And furthermore, the article by Abry, Dijkstra, van Mill in the References section does not have that requirement.
So we should investigate this more and see what other sources are saying.
(3) for the case of S4 (indiscrete space with two points), the reason pi-base marks it as having no dispersion point is due to T92 (dispersion point ==> biconnected) in combination with the assertion Biconnected = false.
But that last assertion is in fact incorrect.  Given the definition of Biconnected that we use, for a connected space to be biconnected, it must not be the union of two disjoint connected subsets, each with size at least two.  So to witness that a connected space is not biconnected, you need at least four points.  Any connected space with at most three points will be vacuously biconnected.
So fixing the assertion about biconnectedness will solve that immediate problem.
(4) It may be true though that the definition for P45 is not optimal.  I looked at some of the early sources that introduced the notion of biconnectedness (Knaster & Kuratowski 1921) and dispersion point, and they are sometimes sloppy in the statements of their theorems.  They seem to totally ignore the cases of very small cardinality, where their results are not strictly true.
As an instructive example worth taking a look at, the short paper: Kline (1922) "A theorem concerning connected point sets" (http://matwbn.icm.edu.pl/ksiazki/fm/fm3/fm3124.pdf) proves that a connected space can have at most one dispersion point (equivalent to our definition).  But that's patently false for both the indiscrete space with two points and for the Sierpinski space.
For now I am inclined not to change the P45 definition, but we need to give this more thought.  Any comments?

At a glance, I think the suggested change would allow http://matwbn.icm.edu.pl/ksiazki/fm/fm3/fm3124.pdf to be correct: the author assumes that the removal of a dispersion point creates a non-connected space, which is true for a totally disconnected space with multiple points.
I think this assumption is more natural for P45, but not enough to spend a lot of time deliberating it for the handful of topologies that have less than four points. (Hmm, does S162 have a dispersion point?)

I guess we could go that way.  To address my point (3) above, we should also fix the biconnected trait for S4.
@hew-wolff What's your take on this?

I'm not sure what the best fix is--I'm just learning about these terms and I don't have any relevant texts. But as a learner, I would prefer that we have definitions that work in all cases, even very small spaces. Otherwise it gets confusing.
Can we find some fairly recent sources and work out a fix that's compatible with them?

I am not sure I understand what "definitions that work in all cases" or a fix "compatible with very small spaces" would even mean.
It's similar to the fact that the empty set is compact (because it vacuously satisfies the definition of compact), or a singleton is connected (because it vacuously satisfies the definition of connected).  The corresponding (modern) definitions are whatever they are, with no special provision for degenerate cases, and very small spaces happen to satisfy the definition or not depending on the property.
The alternative would have been to specify for example that connected spaces also require at least two points.  In fact, that's exactly what pioneers like Knaster & Kuratowski (1921) and others did in their original works.  Later on, I guess it was realized that to get simply phrased theorems like the continuous image of a connected space is connected, it was easier to allow singletons to also be connected instead of adding some extra condition in the theorem itself.  For "dispersion point", one can similarly either put an extra size condition in the definition or not. Both notions will coincide if the space has enough points.  And the cases of very small spaces are degenerate cases that are of little interest.
(but one could look at a few recent sources like you suggest)

The continuous image of connected is connected, and every constant map is continuous, so we probably want singletons to be connected.

For what it's worth, I looked at old and more recent papers and could not find an instance where the definition is stated as requiring at least three points.  But it does not matter really.  Enough time spent on this, I'll approve whatever you guys want to do.
One thing though: https://topology.pi-base.org/spaces/S000010/properties/P000045 should be changed at the same time as the definition change.

I would just like to avoid statements that are false. I feel they might bite me unexpectedly.
If this issue is too small to raise, or if it should come with a more detailed proposal, let me know and I'll try to do better next time.

@hew-wolff thanks for your contribution. I what this and the conversation at https://math.stackexchange.com/questions/4812062/ has revealed is the need for properites to designate spaces with 3 or 4 points, after which we can introduce theorems to enforce whatever decision we want for dispersion point.
I'm going to edit #471 to introduce the property Cardinality \geq 3 and Cardinality \geq 4, and appropriate theorems to wire those with dispersion points and biconnectedness. I'm actually unsure now whether to keep the has multiple points requirement for dispersion point, but maybe with having these cardinalities expressed as properties and enforced by theorems the right solution will pop out...

As expected I eventually hit a contradiction with the assertion at https://topology.pi-base.org/spaces/S000010/properties/P000045 (thanks @prabau for pointing this out so I wasn't caught off-guard), see https://github.com/pi-base/data/actions/runs/6990837011?pr=471
@jamesdabbs originally made pi-Base to digitize Steen/Seebach's Counterexamples. I think we correctly go against Counterexamples for some conventions, e.g. $T_3$ should imply $T_2$. But in this case I don't think one convention is better than the other, and Counterexamples clearly defines a dispersion point to create a totally disconnected space, and clearly believes discrete implies totally disconnected. So I'm updating #471 to revert to our current definition of a dispersion point to keep in line with Counterexamples.

Here's a fun puzzle:
build: spaces/S000001/README.md#L1
properties=P000175,P000129,P000052,P000078 contradict
theorems=T000092,T000095,T000429,T000088,T000316,T000280,T000249,T000218,T000187,T000042,T000078

One of these theorems must assume spaces have at least three points... (-:

Posted at https://math.stackexchange.com/questions/4814244/where-is-pi-base-assuming-the-existence-of-at-least-three-points though I'm still working on it.

Got it, we do require connected for a dispersion point, so my new T429 requires connected as an assumption.

Okay #471 gets us a few steps in the right direction, but I got carried away with adding a bunch of three-point spaces and need to let someone approve the work so far before doing more.


WIP: Replumbing: Add constructor of libcrypto internal method  structures

This queries the provider for its available functionality (unless a
matching method structured is already cached, in which case that's
used instead), and creates method structure with the help of a passed
constructor.  The result is cached if the provider allows it (or if
caching is forced).
Checklist

[x] documentation is added or updated
[ ] tests are added or updated

Note that this PR is made on top of #8286 and #8287, so there are a number of commits that don't really belong here.  To avoid confusion, I'd like for the other two to be merged before commentary here, but I thought I'd provide this to show how the other two are going to be used.

#8287 is now merged.  I've rebased this and squashed all fixups

I know that @paulidale approved this a while ago, but I think we do need a final review now that it doesn't depend on any other PR any more.

Final review, please?

Merged.
9e11fe0d85c7d8bd2b77076c8b2e93433091e765 Replumbing: Add constructor of libcrypto internal method structures
099bd33920e775eb75f4daee5f09b24f17bc136d Replumbing: Add support for the provider query_operation function
85e2417c0d81cfe97586b74c410ef37595aea72d Replumbing: Add an OSSL_PROVIDER iterator with callback


Implement suggested features for Sorting Algorithms interactive

Resolves #972

[x] Add URL parameters for data types to sort: sorted, almost, reverse and the default random
[x] Use 'Points to heavier box' instead of heaviest
[x] Add a URL parameter that lets users peek at the values being sorted
[ ] Improve consistency in JS code
[x] Mention the parameters/link somewhere in a teacher panel in the content

[ ] (and get approval of said panel)

Draft teacher panel:

The eye option in action:

I would be keen to see a simple URL creator in the teacher mode, allowing teachers to pick from options using HTML form elements and a URL is created below for them to possibly copy and send to students. Creating a query string is not simple (hence the long instructions) and would probably be harder once translated to other languages. Thoughts?

I would be keen to see a simple URL creator in the teacher mode, allowing teachers to pick from options using HTML form elements and a URL is created below for them to possibly copy and send to students. Creating a query string is not simple (hence the long instructions) and would probably be harder once translated to other languages. Thoughts?

Reminds me of the Packet Attack level creator. I think that's a better idea, though with only 2 options and the general 'for teachers' vibe I'm wondering if it should be an uninteractive so it doesn't show up in the interactives page

I think it would be good if the eye icon was larger, it took me a moment to find it and I was confused about why I wasn't seeing the numbers. Or maybe have a picture of it on the teacher panel beside the checkbox, so you know what you are looking for when the interactive opens.
I'd like it if I could also click on the eye to make it stay open and let me see the numbers all the time, rather than having to hover over it every time.

I think it would be good if the eye icon was larger, it took me a moment to find it and I was confused about why I wasn't seeing the numbers. Or maybe have a picture of it on the teacher panel beside the checkbox, so you know what you are looking for when the interactive opens.
I'd like it if I could also click on the eye to make it stay open and let me see the numbers all the time, rather than having to hover over it every time.

I've added an icon symbol to the configurator, do you still think I should make it larger/more noticable?
I can't easily make the peek functionality constant all the time because when the user goes to drag the box they pick up either the box and not the number or vice versa (can't remember which). I could make it remain on until the user picks up a box? Or if needed I'll figure out how to join the box and number together

I've added an icon symbol to the configurator, do you still think I should make it larger/more noticable?

That looks really good, I think it's fine at that size.

I can't easily make the peek functionality constant all the time because when the user goes to drag the box they pick up either the box and not the number or vice versa (can't remember which). I could make it remain on until the user picks up a box? Or if needed I'll figure out how to join the box and number together

I think it's ok without the checkbox if it would be difficult. I think it would be a bit weird if they stayed visible and then disappeared when you picked one up, it might look like it's actually a bug or something. But it would be good to see it if that wouldn't be too annoying to implement.

Need to fix a bug when hiding the second row of boxes while images are in them


Docker image?

Are there plans to have a docker image for this lovely exporter?
Here is an example: https://github.com/docker-infra/container_exporter/blob/master/Dockerfile
Let me know if you want me to help with anything :wink:

hi @MrMMorris
you are welcome to help! I just added you to the repo as collaborator with push permissions

Done! Do you have a docker hub account? or should I put it on mine? I don't want to steal any of your thunder :wink:

go ahead Marcus, you can use yours

Did this get pushed to Docker hub?

@dcu can you please create a docker hub account and setup an automated build.

Hi there. Any updates with Docker image on Docker Hub?

Example one: https://hub.docker.com/r/loefflefarn/dcu_mongo_exporter


Xcode 10.1

The code won't compile or convert with Xcode 10.1. :(

I'll take a look later this week.


Could not find argument id for type

When I use this query
mutation {
  updateStory(where: {
    id: "cjrodoikb001z0860g7wzuk21"
  }, data: {
    content: "asd"
  }) {
    id
  }
}

I got this error message
{
  "data": {
    "updateStory": null
  },
  "errors": [
    {
      "message": "Could not find argument id for type Story",
      "locations": [
        {
          "line": 2,
          "column": 3
        }
      ],
      "path": [
        "updateStory"
      ]
    }
  ]
}

@lynkxyz  Im having the same issue, did you find a fix yet?

@danramosd No, I didn't find any solution.

Hey there,
This is now fixed by 35d96740bdfd4c6e58a29236294b59c8560a93d8.
I'll close the issue once it's published 🙌

Published under<EMAIL_ADDRESS>closing now


VScode installed via snap is no longer detected

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

There was #54358 that was addressed at the time but the directory structure of the vscode snap package has changed a while ago and this was failing since then.
Steps to reproduce

install vscode via snap: sudo snap install code
run flutter doctor
observe vscode version check failing

Expected results
flutter doctor finds the vscode snap installation
Actual results
flutter doctor isn't able to find the vscode snap installation
Flutter Doctor output
Doctor output
Building flutter tool...
Resolving dependencies... (2.3s)
Got dependencies.
    • Framework revision faf0c8e640 (15 minutes ago), 2023-10-21 03:29:07 +0200
    • Engine revision e4fc0c4e2b
    • Dart version 3.3.0 (build 3.3.0-48.0.dev)
    • DevTools version 2.28.1
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git"
      directly to perform update checks and upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /home/aresminos/Android/Sdk
    • Platform android-34, build-tools 34.0.0
    • Java binary at: /snap/android-studio/128/android-studio/jbr/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at google-chrome

[✓] Linux toolchain - develop for Linux desktop
    • Ubuntu clang version 14.0.0-1ubuntu1.1
    • cmake version 3.22.1
    • ninja version 1.10.1
    • pkg-config version 0.29.2

[✓] Android Studio (version 2022.3)
    • Android Studio at /snap/android-studio/128/android-studio
    • Flutter plugin version 75.1.2
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)

[✓] VS Code (version unknown)
    • VS Code at /snap/code/current
    • Flutter extension version 3.74.0
    ✗ Unable to determine VS Code version.

[✓] Connected device (2 available)
    • Linux (desktop) • linux  • linux-x64      • Ubuntu 22.04.3 LTS 6.2.0-35-generic
    • Chrome (web)    • chrome • web-javascript • Google Chrome 118.0.5993.88

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

Thanks for your report! I can reproduce this issue with VS Code installed via Snap store. The issue doesn't appear with APT installation.

flutter doctor -v (stable & master - via Snap)
[✓] Flutter (Channel stable, 3.13.8, on Ubuntu 22.04.3 LTS 6.2.0-34-generic, locale en_US.UTF-8)
    • Flutter version 3.13.8 on channel stable at /home/huynq/Documents/Working/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 6c4930c4ac (5 days ago), 2023-10-18 10:57:55 -0500
    • Engine revision 767d8c75e8
    • Dart version 3.1.4
    • DevTools version 2.25.0

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0-rc3)
    • Android SDK at /home/huynq/Android/Sdk/
    • Platform android-33, build-tools 34.0.0-rc3
    • Java binary at: /home/huynq/Documents/android-studio/jre/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7590822)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at google-chrome

[✓] Linux toolchain - develop for Linux desktop
    • clang version 10.0.0-4ubuntu1
    • cmake version 3.22.1
    • ninja version 1.10.1
    • pkg-config version 0.29.2

[✓] Android Studio (version 2021.1)
    • Android Studio at /home/huynq/Documents/android-studio/
    • Flutter plugin version 67.0.1
    • Dart plugin version 211.7817
    • android-studio-dir = /home/huynq/Documents/android-studio/
    • Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7590822)

[✓] VS Code (version unknown)
    • VS Code at /snap/code/current
    • Flutter extension version 3.74.0
    ✗ Unable to determine VS Code version.

[✓] Connected device (2 available)
    • Linux (desktop) • linux  • linux-x64      • Ubuntu 22.04.3 LTS 6.2.0-34-generic
    • Chrome (web)    • chrome • web-javascript • Google Chrome 118.0.5993.70

[✓] Network resources
    • All expected network resources are available.

• No issues found!

[!] Flutter (Channel master, 3.16.0-15.0.pre.42, on Ubuntu 22.04.3 LTS 6.2.0-34-generic, locale
    en_US.UTF-8)
    • Flutter version 3.16.0-15.0.pre.42 on channel master at
      /home/huynq/Documents/Working/flutter_master
    ! Warning: `flutter` on your path resolves to /home/huynq/Documents/Working/flutter/bin/flutter,
      which is not inside your current Flutter SDK checkout at
      /home/huynq/Documents/Working/flutter_master. Consider adding
      /home/huynq/Documents/Working/flutter_master/bin to the front of your path.
    ! Warning: `dart` on your path resolves to /home/huynq/Documents/Working/flutter/bin/dart, which is
      not inside your current Flutter SDK checkout at /home/huynq/Documents/Working/flutter_master.
      Consider adding /home/huynq/Documents/Working/flutter_master/bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 0f5bf521b2 (2 hours ago), 2023-10-23 04:46:09 -0400
    • Engine revision 1dc66e8f2f
    • Dart version 3.3.0 (build 3.3.0-48.0.dev)
    • DevTools version 2.28.1
    • If those were intentional, you can disregard the above warnings; however it is recommended to use
      "git" directly to perform update checks and upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0-rc3)
    • Android SDK at /home/huynq/Android/Sdk/
    • Platform android-33, build-tools 34.0.0-rc3
    • Java binary at: /home/huynq/Documents/android-studio/jre/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7590822)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at google-chrome

[✓] Linux toolchain - develop for Linux desktop
    • clang version 10.0.0-4ubuntu1
    • cmake version 3.22.1
    • ninja version 1.10.1
    • pkg-config version 0.29.2

[✓] Android Studio (version 2021.1)
    • Android Studio at /home/huynq/Documents/android-studio/
    • Flutter plugin version 67.0.1
    • Dart plugin version 211.7817
    • android-studio-dir = /home/huynq/Documents/android-studio/
    • Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7590822)

[✓] VS Code (version unknown)
    • VS Code at /snap/code/current
    • Flutter extension version 3.74.0
    ✗ Unable to determine VS Code version.

[✓] Connected device (2 available)
    • Linux (desktop) • linux  • linux-x64      • Ubuntu 22.04.3 LTS 6.2.0-34-generic
    • Chrome (web)    • chrome • web-javascript • Google Chrome 118.0.5993.70

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

flutter doctor -v (stable & master - via APT)
[✓] Flutter (Channel stable, 3.13.8, on Ubuntu 22.04.3 LTS 6.2.0-34-generic, locale en_US.UTF-8)
    • Flutter version 3.13.8 on channel stable at /home/huynq/Documents/Working/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 6c4930c4ac (5 days ago), 2023-10-18 10:57:55 -0500
    • Engine revision 767d8c75e8
    • Dart version 3.1.4
    • DevTools version 2.25.0

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0-rc3)
    • Android SDK at /home/huynq/Android/Sdk/
    • Platform android-33, build-tools 34.0.0-rc3
    • Java binary at: /home/huynq/Documents/android-studio/jre/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7590822)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at google-chrome

[✓] Linux toolchain - develop for Linux desktop
    • clang version 10.0.0-4ubuntu1
    • cmake version 3.22.1
    • ninja version 1.10.1
    • pkg-config version 0.29.2

[✓] Android Studio (version 2021.1)
    • Android Studio at /home/huynq/Documents/android-studio/
    • Flutter plugin version 67.0.1
    • Dart plugin version 211.7817
    • android-studio-dir = /home/huynq/Documents/android-studio/
    • Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7590822)

[✓] VS Code (version 1.83.0)
    • VS Code at /usr/share/code
    • Flutter extension version 3.74.0

[✓] Connected device (2 available)
    • Linux (desktop) • linux  • linux-x64      • Ubuntu 22.04.3 LTS 6.2.0-34-generic
    • Chrome (web)    • chrome • web-javascript • Google Chrome 118.0.5993.70

[✓] Network resources
    • All expected network resources are available.

• No issues found!

[!] Flutter (Channel master, 3.16.0-15.0.pre.42, on Ubuntu 22.04.3 LTS 6.2.0-34-generic, locale en_US.UTF-8)
    • Flutter version 3.16.0-15.0.pre.42 on channel master at /home/huynq/Documents/Working/flutter_master
    ! Warning: `flutter` on your path resolves to /home/huynq/Documents/Working/flutter/bin/flutter, which is not inside your current Flutter SDK checkout at /home/huynq/Documents/Working/flutter_master. Consider adding /home/huynq/Documents/Working/flutter_master/bin to the front of your path.
    ! Warning: `dart` on your path resolves to /home/huynq/Documents/Working/flutter/bin/dart, which is not inside your current Flutter SDK checkout at /home/huynq/Documents/Working/flutter_master. Consider adding /home/huynq/Documents/Working/flutter_master/bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 0f5bf521b2 (2 hours ago), 2023-10-23 04:46:09 -0400
    • Engine revision 1dc66e8f2f
    • Dart version 3.3.0 (build 3.3.0-48.0.dev)
    • DevTools version 2.28.1
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0-rc3)
    • Android SDK at /home/huynq/Android/Sdk/
    • Platform android-33, build-tools 34.0.0-rc3
    • Java binary at: /home/huynq/Documents/android-studio/jre/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7590822)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at google-chrome

[✓] Linux toolchain - develop for Linux desktop
    • clang version 10.0.0-4ubuntu1
    • cmake version 3.22.1
    • ninja version 1.10.1
    • pkg-config version 0.29.2

[✓] Android Studio (version 2021.1)
    • Android Studio at /home/huynq/Documents/android-studio/
    • Flutter plugin version 67.0.1
    • Dart plugin version 211.7817
    • android-studio-dir = /home/huynq/Documents/android-studio/
    • Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7590822)

[✓] VS Code (version 1.83.0)
    • VS Code at /usr/share/code
    • Flutter extension version 3.74.0

[✓] Connected device (2 available)
    • Linux (desktop) • linux  • linux-x64      • Ubuntu 22.04.3 LTS 6.2.0-34-generic
    • Chrome (web)    • chrome • web-javascript • Google Chrome 118.0.5993.70

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

@huycozy you can assign me to this issue.

Sure, thanks for the contribution!

Closing as the https://github.com/flutter/flutter/pull/136997 fix has been merged.


fix: error logging in firefox

Firefox has an error type with a .stack property that is an empty string, so guard on the various properties being empty before logging them.
Change checklist

[x] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation if necessary (this includes comments as well)
[ ] I have added tests that prove my fix is effective or that my feature works

Which error has the empty stack property?

I've seen it with errors emitted from WebTransport streams.


[MM-35188] Make short description required for "Create Ticket" form

Summary
This PR makes the Short Description field required for the Create Ticket form. It is required for the "Create Ticket" modal and the command.
Ticket Link
Fixes https://mattermost.atlassian.net/browse/MM-35188

@DHaussermann If the UX for the command is not ideal (you now have to provide the summary before submitting the command that opens the modal) then we will need to adjust the code to have the command and modal have two different forms so the command does not have the field as required.
@larkox I tried to do this when I implemented this PR but it either resulted in messy code or I was concerned about breaking things. Do you know how that may be accomplished gracefully with the project's structure?

If I am not mistaken, where the form is created for the command is on routers/mattermost/fCreateTicket.go/fCreateTicketForm, and the only way to reach that part of the code is by fetching the form from the command. There we can add a for loop setting all the required fields to not required, and let the command just open the modal.

/update-branch

@mickmister is this still needed? If so, can you pull master up into this branch?
Currently - I'm not able to get this to serve me a manifest. I think the make file is old and I can't remember where it served the .json I've adjusted parameters in my local version but no luck.

@DHaussermann The PR is now up to date.

@DHaussermann Would you mind merging the PR as it is as it contains a simple one line change?

note I'm not able to adjust labels in this repo.


Improve performance of widget motorcycle_accidents_vs_all_accidents

Possibly with agg tables / cache, @ziv17 is leading this.
Ziv - I suggest we'll perform this after deciding on architecture.

FE issue - waiting for API - https://github.com/hasadna/anyway-newsflash-infographics/issues/541

Hi @atalyaalon I do not understand what is required here.
Cache is already implemented.

Hi @atalyaalon
What is the use-case that this issue intends to resolve? I am asking, because the way I see it, the solution is either small tables, or cache, which we already have.

@ziv17 I agree.
If you find options for performance improvements while working on this widget - please improve.


Prevent fly-by fixture from powering containment field generator

About the PR
Fixes #29222
TL;DR: the fly-by fixture on the emitter bolt can power the containment field generator.
Why / Balance
I believe the original behaviour to be unintended, and I personally witnessed it being relied on in a round on Salamander to power a Tesla containment field with only a single emitter.
Technical details
Adds a field to ContainmentFieldGeneratorComponent, SourceFixtureId, so we can generate power only when the emitter bolt's "projectile" fixture collides with the generator. Otherwise we also catch the "fly-by" fixture.
Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes
It might break your tesla meta, but should not have any knock-on effects.
Changelog
:cl:

fix: Containment field generators can no longer be powered by an emitter bolt flying past them. The bolt must hit them directly.

what makes you think this behavior was unintended?

what makes you think this behavior was unintended?

well, when something hits something that is a whole tile away and not in the trajectory, one could consider that to be pretty weird and unintended

considering that emitter lasers have a secondary collision thingy that powered the pylons i don't think it was unintended, whoever made emitter bolts wouldn't add a secondary collision thingy for no reason.

considering that emitter lasers have a secondary collision thingy that powered the pylons i don't think it was unintended

the reason it powers them is because it does a very dumb blanket collision subscription without checking a fixture ID, an extremely common projectile bug that has happened countless times.

whoever made emitter bolts wouldn't add a secondary collision thingy for no reason

you're right! which is why, just like every single other fly-by fixture in the codebase, it exists to play the sfx of a bullet whizzing by. This is well known because basically every single projectile has this fixture and they all make checks to exclude it because it's not meant to actually count as a projectile collision.
even ignoring the intent, it makes 0 sense for a laser to collide with things when there is clearly no visible collision occurring with the sprite, so this entire tangent doesn't even matter to begin with.

Fixes #25528


EPIC-KITCHENS Action Features

Thank you so much for releasing this amazing work. I am really interested in this project and would like to try EPIC-KITCHENS action detection on custom video data with custom noun and verb sets. I'd like to try with EPIC-KITCHENS data first and I see feature zoo links aren't updated.
I'm wondering:

Would it be possible to briefly describe the format details of feature generation for custom data?
Do you know when the EPIC-KITCHENS feature links will be uploaded/fixed?

我已经收到您的邮件，将尽快回复。

Hi, thanks for the attention and sorry for the late reply.
The features for epic-kitchens by TAda2D has been released in the feature zoo. You are welcome to try it out. The features by ViViT will be soon released as well.


Update dependecies

Update Consul to 0.8.0 and logrus to 0.11.5.
Update tests to reflect changes in Consul.

Coverage increased (+0.03%) to 87.48% when pulling 4d9f3e4c24afcb08517054cab00960838767bc5f on update_consul into 185e134f8f6793b33e16c2d6d43b7fbb477aaf00 on master.


DS-3390 Item submission throws NullPointerException

https://jira.duraspace.org/browse/DS-3390
I've modify the "if condition" so it first checks if the RP is assigned to any group before accessing the groups name

Closing as this PR must be recreated on dspace-6_x branch. XMLUI has been removed from master. See this mailing list notice: https://groups.google.com/d/msg/dspace-devel/aEFqSn46ryM/iPRr_IMLCwAJ


Update for Org store GA

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

powerbi-docs/power-bi-custom-visuals.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@tsikiksr : Thanks for your contribution! The author, @markingmyname, has been notified to review your proposed change.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

powerbi-docs/power-bi-custom-visuals.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

#sign-off

@tsikiksr : Thanks for your contribution! The author, @markingmyname, has been notified to review your proposed change.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

powerbi-docs/power-bi-custom-visuals.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Add "ordered" Parameter to Devices executeCommand()

Is your feature request related to a problem? Please describe.
While writing a custom script to trigger functionality in my Samsung Fridge, I noticed that this SDK did not support the REST API's ordered  parameter in the SDK
Describe the solution you'd like
Add an optional ordered parameter that adds an ordered parameter to the executeCommands payload when ordered === true
Describe alternatives you've considered
None at this time - you could technically add another method to the SDK, but the suggested implementation is simpler and wouldn't break the existing API
Additional context

I've already implemented this in my own project and can push a branch containing the simple changes specified above (with accompanying tests)

It turns out this parameter is no longer useful. The documentation for it has been updated.


feat(Probability/Independence): define independence wrt a kernel and a measure

We introduce a new notion of independence with respect to a kernel and a measure. The plan is to eventually express both independence and conditional independence as particular cases of this new notion (see #6098).
Two sigma-algebras m and m' are said to be independent with respect to a kernel κ and a measure μ if for all m-measurable sets t₁ and m'-measurable sets t₂, ∀ᵐ a ∂μ, κ a (t₁ ∩ t₂) = κ a t₁ * κ a t₂.
Independence is the special case where κ is a constant kernel. Conditional independence can be defined by using the conditional expectation kernel condexpKernel.

The file is a copy of Independence/Basic with minor modifications to use the generalized independence definition.

bors r+

bors r+


Support Tesla Wallbox Gen2 via RS485 adapter

I'd like seeing support added for the CH341 serial RS-485 adapter that is commonly used to control the Tesla Wallbox (Tesla Wall Connector) set to 'slave' mode.
There is a quite successful project called TWCManager that allows to control a Tesla Wall Connector programmatically through a cheap (< $5) RS-485 USB serial adapter (using the CH341 driver).
A fork of this project has added some new features but the whole thing is a bit dated...
While I have mangled through the code of the TWCManager to add support for Kostal Plenticore Inverters successfully, I'm unfortunately not familiar with programming Go, otherwise I would have contributed a charger module. Such module should enable to run evcc from a raspberry pi zero connected (or even put inside a Tesla Wall Connector like I did) and "convert" the original Tesla Wall Connector to an intelligent charger.
If nobody is willing to add such driver, I might need to find time to teach myself with Go and write it on my own. Please consider that this would take me endless hours while an experienced Go programmer might hack it together in a fraction of time...
Thanks!

@hopfi2k seems this will oly work for Gen2 TWC, no longer for Gen3 so it seems usefulness is limited in future. As for CH341 I don‘t think there is any support needed. EVCC will see a linux device regardless of which adapter you use, so this comes down to supporting TWC Gen2.

Correct it comes down to accessing a serial device. TWC Gen 2 have been sold for the past years and it's safe to assume that hundred thousands are installed world wide. Even newer Gen 3's have an RS-485 port, but since it supports WLAN connectivity it might not be necessary to use a RS-485 adapter to control charge limits (depending on future Tesla software upgrades). Nevertheless even Gen 3 support slave mode.
From my point of view, TWC's most probably have the broadest installation base of all chargers currently supported by EVCC. And adding the RS_485 adapter (internally or externally) and putting the rotary switch to position 'F' (slave mode) is extremely simple. Giving the right software support I'm pretty much sure that more people would be willing to 'modify' their TWC to benefit from using it with EVCC. Worth considering?

I‘ve had a look at the protocol nd it seems pretty straightforward. Would be willing to tackle this but would need access to a twc2 for testing. If you want to grant access please pm me at<EMAIL_ADDRESS>

For all concerned: a Protocol 2 version is necessary. Basic communication with the charger is already possible (pending debugging). We need to understand if it is possible to start/stop the charge process using the charger instead of the car API (as TWC does).
Any additional insights welcome.


Unable to push the application

Hi Team,
We are unable to push the application to cloud foundry due to below error
Runner error: staging task with guid 832d0b8c-2f19-446d-8558-24cba11534ab failed to start",
Attaching the screenshot for the same. Kindly provide us a solution to this.

Regards,
Kulasekhar

Hi ,
Does any looking into the issue?
Regards.
Kulasekhar

Hi @kulabtech, can you please provide more context around your issue. Specifically, it would be nice to have some steps to reproduce the error that you're seeing.

Closing due to inactivity.


typo: teh -> the

@nschonni : Thanks for your contribution! The author, @PanosPeriorellis, has been notified to review your proposed change.

@nschonni : Thanks for your contribution! The author, @PanosPeriorellis, has been notified to review your proposed change.

@nschonni : Thanks for your contribution! The author, @PanosPeriorellis, has been notified to review your proposed change.

sign-off


[WIP] Adds next-auth for Discord Oauth

This PR looks into using next-auth for Discord Oauth. I used this PR for reference and to get started I followed the setup guide provide by next-auth at this link
Achieved so Far
I created a session page so you can see what type of data gets returned from the discord signin. Here is before and after signing with Discord.
Before:

After:

I was also able to connect the Prisma Schema with the Discord provider following the instructions here and for adding the Discord provider I followed the steps here. For the schema I had to modify the Users and Session table. A new Accounts table was added in that keeps track of the next-auth providers (in this case Discord). This however breaks the current logging with username and password.
SignIn Flow
When you click on the Sign In button on the session page, it takes you to next-auth's already built in sign in page:

Then Discord will ask if it is okay for the authorization:

Then it will take you back to the session page with your data displayed shown above. During the process next-auth looks up the user's id in the database and if it can't find the specific id, it will look for the email. If it doesn't find an email, then prisma will create the user. It will then proceed to create the Account in the Accounts table and link it to the User. After the account has been linked up, the session is created for the User.
Issues/Concerns
I am not sure what to do next because next-auth provides a session and we currently are using express-session to create a session. With the changes in the PR the current Login Page with username and password doesn't work. Any feedback will be appreciated on how to better integrate next-auth!

I think if next-auth would make it easier to integrate with other login services such as GitHub in the future and Discord currently, we should let next-auth handle it all include signing with username and password.
Amay pointed there would be database issues though. I don't know much about database, so I can't tell if it's easy to migrate.


Link to page

How can I access my pages (other page, not the initial page?
I already configure my site.

You can use Markdown and https://quartz.jzhao.xyz/features/wikilinks syntax

You can use Markdown and https://quartz.jzhao.xyz/features/wikilinks syntax

I'm sorry, but I couldn't. Maybe it's the configuration I made.
Here "quartz.config.ts" I did
filters: [Plugin.ExplicitPublish()],
Here is my file
content/03 - Resources/Wiki/Metodo PARA.md
Inside my file I write: publish: true
date: 2024-09-19T19:59
tags:

wiki
organizacao
publish: true

My text here


review 4.0 Japanese translation draft

Sent comments to team. Scheduled to discuss translation with @peterspdx on Friday (May 8) during weekly translation call.

sarahpearson, you stated in the email to the Japanese team that you had a question you wanted to discuss with me.  can you give me a head's up on what the question is?


Can't install on Apple Silicon

Hi! I'm trying to install grpcio on an Macbook pro (14-inch, Monterey, M1 Max), but I seem to consistently get the following error:
Could not find <Python.h>. This could mean the following:
      * You're on Ubuntu and haven't run `apt-get install python3-dev`.

I have installed Google's JAX, which is a moderately complex package, with no problem, so I don't think the python installation is corrupted.
I have also tried the solutions on stackoverflow that add GRPC_PYTHON_BUILD_SYSTEM_OPENSSL=1 GRPC_PYTHON_BUILD_SYSTEM_ZLIB=1 before the pip incantation to no avail
Any idea what the issue could be and how to fix it? Is this a known bug?

I install grpcio by pip get error, but successfully installed using conda install grpcio.

if doesn't resolved above, try install python 3.9.8 or superior..

This seems to be an issue with an individual environment.


Add cross-platform build support

Resolve #63

Wow! This contribution is amazing @vincentducorps !! First of all, congratulations 🎉
The code LGTM! 👌
BTW I don't know why the GitHub actions jobs are not triggered in this PR 😕 Maybe is it a GitHub issue? 🤨
However, it would be very useful to add a job at https://github.com/vinissimus/next-translate/blob/master/.github/workflows/test.yml with runs-on: windows-latest to prevent to broke the CLI behavior in Windows!

This will be released on 0.14.0, however, I'm going to create a canary version 0.14.0-canary.1  just today with this PR included.
I'm going to merge the PR. For the yml of GitHub actions, I created another issue https://github.com/vinissimus/next-translate/issues/101 to solve it in another PR before 0.14.0 release.

This will be released on 0.14.0, however, I'm going to create a canary version 0.14.0-canary.1 today with this PR included.
I'm going to merge the PR. For the yml of GitHub actions, I created another issue #101 to solve it in another PR before 0.14.0 release.

Thanks.
I made a new PR to fix that #103


u128 doesn't work for Value

Looks like there's a bug in serde_json. This program:
#[macro_use]
extern crate serde_json;
use std::str::FromStr;

fn main() {
    let string = format!("{}", u128::max_value());
    println!("{}", string);
    let val = serde_json::value::Value::from_str(&string);
    let val2 = serde_json::to_value(u128::max_value());
    println!("{:#?}, {:#?}", val, val2);
}

outputs:
340282366920938463463374607431768211455
Ok(
    Number(
<PHONE_NUMBER>20938500000000000000000000000.0
    )
), Err(
    Error("u128 is not supported", line: 0, column: 0)
)

playground
if you put arbitrary_precision feature, then Value works 100% without cutting precision, if you do it via u128 => String => Value. However direct conversion u128 => Value with to_value results in error.

Thanks! I would accept a PR to support u128 => Value in the case that arbitrary_precision is enabled.

Hi, I'm getting the same error with this code:
use serde::Serialize;

extern crate serde_json;

#[derive(Serialize)]
struct Data {
    num: u128,
}

fn main() {
    println!("{}", serde_json::json!(Data { num: 1 }))
}

$ cargo run
    Finished dev [unoptimized + debuginfo] target(s) in 0.00s
     Running `target/debug/serde-json-u128-bug`
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: Error("u128 is not supported", line: 0, column: 0)', src/main.rs:11:20
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Code: https://github.com/josecelano/serde-json-u128-bug/blob/main/src/main.rs
Using serde 1.0.152 and serde_json 1.0.91:
[[package]]
name = "serde"
version = "1.0.152"
source = "registry+https://github.com/rust-lang/crates.io-index"
checksum = "bb7d1f0d3021d347a83e556fc4683dea2ea09d87bccdf88ff5c12545d89d5efb"
dependencies = [
 "serde_derive",
]

[[package]]
name = "serde_json"
version = "1.0.91"
source = "registry+https://github.com/rust-lang/crates.io-index"
checksum = "877c235533714907a8c2464236f5c4b2a17262ef1bd71f38f35ea592c8da6883"
dependencies = [
 "itoa",
 "ryu",
 "serde",
]

I too am getting the u128 is not supported error when using serde_json::from_str(...) method. Similar struct layout to @josecelano above. Is it not supported?

Fixed in serde_json v1.0.93.

Hi, with serde_json v1.0.93 it's now working fine for serialisation, but when I use let payload = serde_json::from_str::<MyType>(...);, I'm still getting the u128 not supported error. Am I missing something? When I change to u64 it all works, so am I missing a setting?

Here is what I tried and it seems to work.
use serde::Deserialize;
use serde_json::json;

#[derive(Deserialize, Debug)]
pub struct Data {
    pub num: u128,
}

fn main() {
    let j = r#" {"num":1} "#;
    println!("{:?}", serde_json::from_str::<Data>(j).unwrap());

    let j = json!({"num": 1});
    println!("{:?}", serde_json::from_value::<Data>(j).unwrap());
}

Okay, so I did some digging and came up with the following. Is there a different handling when it comes to deserializing enum?
Works fine:
#[derive(Debug, serde::Deserialize)]
#[serde(rename = "a", tag = "type")]
struct A {
  test: u128
}

...
match serde_json::from_str::<A>(&text) {
  Ok(a) => {
    info!("{:#?}", a);
    info!("{:#?}", a.test);
  }
  Err(err) => {
    error!("{}", err.to_string());
  }
};

Fails with u128 is not supported:
#[derive(Debug, serde::Deserialize)]
#[serde(tag = "type")]
enum Opts {
    #[serde(rename = "a")]
    A { test: u128 },
}

...

match serde_json::from_str::<Opts>(&text) {
    Ok(Opts::A { test }) => {
        info!("{:#?}", test);
    }
    Err(err) => {
        error!("{}", err.to_string());
    }
};

Have I made a mistake somewhere? Any guidance would be much appreciated.

Have I made a mistake somewhere? Any guidance would be much appreciated.

Internal tagging is implemented with buffering because the content could come before the tag. But the Content type does not support 128-bit values, and thus you see the error. See these other issues https://github.com/serde-rs/serde/issues/2230 and https://github.com/serde-rs/serde/issues/1183.


Bug: elements of _items_vector  are not updated when multi_index is moved to new instance

https://github.com/EOSIO/eos/blob/develop/contracts/eosiolib/multi_index.hpp#L270
The elements of _items_vector are not not updated to the address of new instance when multi_index is moved causing certain operations to fail. Affected member functions are iterator_to, erase and modify. All three functions fill throw an assert exception with message object passed to <member_function> is not in multi_index on cached items when index was moved to new instance.
The core of this problem is that all successful query operations carried against multi_index, cache loaded items. Cached Items store the address of multi_index instance in __idx member variable for identification purposes. The __idx variable is never updated when index is moved to the new instance.
Items are cached here:
https://github.com/EOSIO/eos/blob/develop/contracts/eosiolib/multi_index.hpp#L590
and here:
https://github.com/EOSIO/eos/blob/develop/contracts/eosiolib/multi_index.hpp#L1696
Example of a bug:

multi_index_t get_multi_index_of(uint64_t key)
{
    multi_index_1 mi1;
    if(mi1.find(key) != mi1.end()) {
        return mi1; // returns mi1 via RVO, no deep copying involved
    }

    multi_index_2 mi2;
     ......
}

///////////////////////////////////////////////////////

auto mi = get_multi_index_of(some_key);
auto it = mi.find(some_key);
mi.modify(it, payer, [](auto& obj){...}) // Error

Possible solutions:

Replace member variable __idx with unique identifier which is not tied to the address of index instance. Since __idx is not used for other than identifying parent index, it's safe to replace it with short UID.
Explicitly delete move constructor / move assignment operator. I would also not recommend this solution because there are cases where you want to move multi_index from one object to another.
Make custom move constructor / move assignment operator and do not move _items_vector to new instance. The problem with this solution is, that new instance looses all cached items and has to fetch them again from db.

Other possible solution:

Make custom move constructor / move assignment operator and update items of _items_vector to the new instance before move. This solution is very expensive in terms of CPU cycles so I wouldn't consider it.

I don't think copying the entire multi_index table is a good practice because of several reason:

you can't copy _items_vector, because there's a std::unique_ptr inside each of its element owning the corresponding cache
changing from std::unique_ptr to std::shared_ptr to allow copying is inefficient, which will only result in more memory and time overhead.

workaround: put your multi_index instance as member of your class, so that your function get_multi_index_of  should only return the pointer of your multi_index instance.

Thank you for the suggestion but I think this issue should not be closed yet since move and RVO interface of multi_index is broken. No deep copying ever occurs because copy constructor is implicitly deleted due to member of item_ptr being type of std::unique_ptr.
https://github.com/EOSIO/eos/blob/develop/contracts/eosiolib/multi_index.hpp#L265
Please note I've update the code example with a comment so it's more clear what it does.
@taokayan @tbfleming

It's a confirmed bug (#4577). For efficiency reasons we're more likely to delete copy and move.

Would you maybe consider making explicitly defined move constructor where _items_vector is not moved to the new instance? It's a quick fix and it works. The only drawback is that cached items are not preserved.

I agree, but in this case are only cached items which would not be available to the new instance. The state of _items_vector in both instances (old and new) remains valid so nothing breaks. The only drawback is that the instance index moved to has to (lazily) load again already cached items. Its temporarily solution but it would fix current broken interface.


Fix code scanning alert - Does not check for buffer overflows (CWE-120).

Tracking issue for:

[ ] https://github.com/morkhet/subrip_shifter/security/code-scanning/20

fixed with commit


Ability to migrate from svn

I think it would be great to be able to migrate from svn (like gogs-url/repo/migrate for git repos. Now I use git-svn to do this locally.

I did this recently using a custom bash script and the gogs api to create and populate the git repositories.  You'll want to use a custom script to do the migration since your SVN repository can be laid out in any number of different ways and it'll enable you to finetune the translation of the svn authors -> git authors, svn comments, and how to handle svn branches and tags when migrating to git.

the solution its to use the svn2git warpper and then once the git repo was created from svn conversion locally, upload to the git repository finally


[Feature] Admin able to Mark Spam

Proposal
Needs #4527. Enables the confirming of Spam or Ham by the Admin. This interacts with the OSF db.
Repercussions
Admins with power.

Needs #4527.

It has it! That PR is merged. :-) Do we need a rebase here?

Alright, that's all I got in me tonight (spent most of it on #5040 ;). I will plan to pick up with this on Monday! :)

@Ghalko First CR pass done. Swing by if there are questions or problems. Sorry for the massive delay.

@cwisecarver Finished response pass. This was quite old, funny to see how far my understanding of CBV has matured over the last month. It will probably break several of my subsequent PRs, but all for the betterment. The faster we turn this around, the better the other PRs are going to look.

Other than the last comment everything looks good. I did a quick functional test too and it appears to be functional. :shipit: Sorry again for the delays @Ghalko.

@Ghalko Can you please update this PR with develop? That should fix the build.


Image Slider has a bad rating at pagespeed tests

The page loads, then it arranges all elements of the slider at full width beneath each other (height more than 2000px), then calculates the width of each so 4 get shown and groups them next to each other (height switches to 600px).
This causes crazy amounts of CLS, 0.655 in our case, which is a bad rating and stops us from getting a green value at page speed tests.

[public] Automated response: This issue is linked to the internal issue https://shopware.atlassian.net/browse/NEXT-34506.
[created from NEXT-34506, comment 485844]

Any news on this? Our page speed is still problematic due to this.

[public] Hello,
Within our company roadmap and work capacity, we try to address each bug or improvement request but admit that not each one will be resolved. To continue our culture of honesty and openness, we are closing this issue to focus on our roadmap on behalf of all Shopware users.
If you feel like this issue is still impacting you, please create a new issue and let us know. Thank you again for your valuable feedback.
[created from NEXT-34506, comment 501194]


Improve VS Code Dark Plus theme accuracy

Hi, a quick fix for a number of color inaccuracies in the VS Code Dark Plus theme.
Try out the theme switcher dropdown on the website below for a detailed overview of the changes:
Website: https://objective-williams-02e373.netlify.app/
Repo: https://github.com/karlhorky/prismjs-vs-code-dark-theme-fix
Preview (may be outdated):
Old

New

That looks really good!
Could you please run the npx gulp screenshot command to update the screenshot? After that, we can merge this.

Also, you might be interested to know that Prism also has a regex language that will add itself to JS, JSX, TS, and TSX. I gave it some highlighting a while ago but it's far from ideal.
Also also, should I add an alias for control-flow keywords (e.g. if, else, try, return, aka the purple ones) to JS Extras?

Also also, should I add an alias for control-flow keywords (e.g. if, else, try, return, aka the purple ones) to JS Extras?

That would be great, cheers! This along with https://github.com/PrismJS/prism/issues/2528 would probably cover a lot of the remaining missing things.

Prism also has a regex language that will add itself to JS, JSX, TS, and TSX

Added to the demo, thanks! I also added a new escape token color.

Could you please run the npx gulp screenshot command to update the screenshot?

Sure, done!

Also styled the .token.keyword.control-flow classes: 6449e45151d8aad735d1035a3d214764c9d17f61

@RunDevelopment Added the color changes from .token.imports .token.maybe-class-name and .token.exports .token.maybe-class-name (https://github.com/PrismJS/prism/issues/2528 and https://github.com/PrismJS/prism/pull/2533).
I guess this PR is now ready to go?

Thank you for contributing @karlhorky!

Glad to help! Thanks for the merge and your tireless work on this!! 💯

@RunDevelopment would it be possible to get a new version published to npm with these changes? Version 1.4.0 does not have them...

@mAAdhaTTah Could you please handle the release? The next version will be 1.4.1, I think.

@RunDevelopment @karlhorky done. v1.4.1 is live in npm.

Thanks @RunDevelopment @mAAdhaTTah !


ViewManipulate dosn't change the state of IsUsing or IsOver

Hi,
ViewManipulate should affect the state of IsUsing and IsOver.
I was also thinking that if you set the background color to completely transparent (as is my case), it should only change when interacting with the actual cube faces rather than the outside rect.
Should be an easy change, let me know what you think.
Cheers

Hi !
I'm preparing a PR with some changes, I'll put that in.
Thank you for the suggestions :)

partially done in #336 which add IsUsingViewManipulate().


Prevent inclusion of es6 source in distribution package

Added build.files array to prevent the es6 source from being included in the final app distribution as per issue #525.

LGTM, but maybe just add allow files will be better?
"files": [
  "dist/",
  "app.html",
  "main.js",
  "main.js.map",
  "package.json"
]


dts: Fix warning related to arm,v{6,7,8}m-nvic yaml files

The nvic yaml files were missing an id property.  So we'd get a warning
like:
extract_dts_includes.py: 'id' property missing in 'ARMv7-M NVIC Interrupt Controller' binding. Using '
Add the id to fix this warning.
Signed-off-by: Kumar Gala<EMAIL_ADDRESS>

Codecov Report

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

@@           Coverage Diff           @@
##           master    #8768   +/-   ##
=======================================
  Coverage   52.36%   52.36%
=======================================
  Files         195      195
  Lines       24713    24713
  Branches     5136     5136
=======================================
  Hits        12940    12940
  Misses       9699     9699
  Partials     2074     2074

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 41dd662...8e5b149. Read the comment docs.


store: add a subscriber api, and update the HUD async

Hello @landism,
Please review the following commits I made in branch nicks/sub:
43a83c75d84883cc0f346ea94ccf89f8edf6abb1 (2018-10-12 15:51:48 -0400)
store: add a subscriber api, and update the HUD async

(the goal of this is to move towards an API where we can do lots of things in parallel, like builds and port-forwarding and HUD updates and not have to worry so much about blocking the main loop)

gah i need to fix up a bunch of tests


[RFC] Parametrized Copr builds

Hey!
While using Packit in various upstream projects a one potentially useful feature came to mind that's (AFAIK) currently not possible with Packit: do multiple Copr builds with different parameters and run the same tests on all such builds.
The main use case (from my side) is to do both a regular build and a sanitized build (using ASan/UBSan/etc.) in a single PR and then execute the same set of tests on both builds.
The easiest way I could think of right now would be probably allowing passing custom arguments to rpmbuild (if that's even possible from Packit's side), i.e. something like:
jobs:
- job: copr_build
  trigger: pull_request
  rpmbuild_params: --define "sanitized_build 1"
  targets:
    - fedora-stable
    - centos-stream-8-x86_64

Where the respective spec file would need to be adjusted to account for the just defined option (or maybe even something like --define "build_cflags %{__build_flags_lang_c} %{?_distro_extra_cflags} -fsanitize=address,undefined" would do, which wouldn't require spec file modifications, but it's way less obvious). But that's just an initial idea, there's probably a better way to achieve this.
Would something like this make sense in Packit or be even possible?
(/cc @evverx)

a regular build and a sanitized build (using ASan/UBSan/etc.) in a single PR and then execute the same set of tests on both builds.

I'd add Valgrind to this list :-)
I think at some point I tried to figure out how to move stuff run on GHActions to Packit and I'm not sure it's possible. I guess it should be possible to cover, say, unit tests where it's relatively easy to switch compilers/compiler options/environment variables but with integration tests where unit files have to be edited and coredumps/ASan/UBSan/Valgrind backtraces have to be collected I'm not sure how it can be achieved. I think partly it has something to do with the fact that Packit should test packages but at the same time specfiles get in the way when test builds and test environments differ from the normal Fedora environment significantly. (For the same reason I don't use the mkosi systemd testing infrastructure much because it's kind of annoying to override the distro defaults by flipping various flags to enable various features to cover more code. Just to clarify I'm not saying it's bad. It's just that its goal is different)
That being said I think it would be really great if it was possible to somehow make it possible to do https://github.com/dbus-fuzzer/dfuzzer/issues/21 with, say, polkit https://github.com/polkit-org/polkit/issues/515 with ASan/UBSan and all those things.

hi! Sorry for a bit late response, I somehow missed it in my notifications.
The approach you mention sounds doable to me, maybe @praiskup could provide more info if this is already somehow supported with Copr or could be added so that we could pass these arguments from Packit?

rpmbuild_params: --define "sanitized_build 1"

This is currently not possible. One has to package an /usr/lib/rpm/macros.d/ file and make it available by default in corresponding copr buildroots, if this is what you are going to do - I'd suggest building the package into two different copr projects (once for normal builds, one for sanitized builds).  Recently reported against copr, btw: https://github.com/fedora-copr/copr/issues/3508


Face Recognition Using JavaScript Under LGM-SOC'21

I want to make a script of Face recognition which every one can use and test in there browser without any additional install.
@akshitagupta15june , Can you assign me this issue under LGM-SOC'21

ok add your implementation inside face recognition folder

I think I had done the same previously https://github.com/akshitagupta15june/Face-X/pull/901#issue-668909903.
So plz check, if not same sorry for bothering


remove searchAll field from grid toolbar issue

(with last w2ui 5.x)
how to remove searchAll field from grid toolbar
I use show:{searchAll: false} in configuration options but no luck
https://jsfiddle.net/samoilis/ouxje8b3/1/

http://w2ui.com/web/docs/w2grid.multiSearch

grid.show.toolbarInput = false
https://jsfiddle.net/ouxje8b3/3/

@mpf82 works!
Thank you for your answers


Convert defines to constants when possible

It's typical to have constants defined with #define.
cxgo should recognize those patterns and declare the corresponding constant in Go.
Unfortunately last time I checked, cc had no info for macro expansion. Need to double-check this and possibly contribute support, if it's not there yet.

From chat with cznic:
ccgo uses https://godoc.org/modernc.org/cc/v3#AST.Eval for that.

From chat with cznic:
ccgo uses https://godoc.org/modernc.org/cc/v3#AST.Eval for that.

cxgo will now define simple constants declared with #define.
It will still inline all the value for all usages of the #define though. We need some support from CC here.
There are also issues with string constants. Investigating it.

cxgo will now define simple constants declared with #define.
It will still inline all the value for all usages of the #define though. We need some support from CC here.
There are also issues with string constants. Investigating it.

https://gitlab.com/cznic/cc/-/issues/115

https://gitlab.com/cznic/cc/-/issues/115

The missing piece: https://gitlab.com/cznic/cc/-/issues/115#note_512283629

Implemented for a single-token constant expressions.


Default environment filtering to on for all users.

This will ship as part of Homebrew 1.4.0.

Maybe this should wait a bit until the inevitable Xcode 9.2 nonsense appears and dies down.  Also, maybe not.

We could also wait for the next full moon.

Maybe this should wait a bit until the inevitable Xcode 9.2 nonsense appears and dies down. Also, maybe not.

@chdiza Just out of interest: why would that be a good idea?

@MikeMcQuaid I was just thinking that, if the ENV filtering gets merged now, when the inevitable nonsense (from Xcode 9.2) appears it might be harder to tell whether the cause is Xcode 9.2 or some bug in the ENV filtering PR.  Best not to have two major changes at once, because it makes diagnosis and recovery more difficult.  That sort of thing.
On the other hand, perhaps the nature of the present PR is such that maintainers will be very easily able to pin blame on the appropriate suspect.  I'm not knowledgeable enough to tell.

E.g., CLT 9.1 broke MacVim.  If a big change (like this PR) had landed right about when CLT 9.1 did, I know I would be wondering what the likely culprit was.

@chdiza Ah, I see. In this case I think as we've been running this for CI, HOMEBREW_DEVELOPER and dev-cmd-running folks for a few weeks it won't change behaviour for any of them so I think we're fine.


SNS endpoint creation

When you attempt to create two SNS platform_endpoints with the same token in boto, it normally throws a client error the second time, telling you that the endpoint has already been created. (See http://stackoverflow.com/questions/22227262/aws-boto-sns-get-endpoint-arn-by-device-token). However moto doesn't check if there are any pre-existing platform_endpoints with your token, it just happily creates new ones, so I'm unable to test my error handling for this case using moto. Can you ensure that attempting to create a second platform_endpoint with an already used token throws a ClientError similar to boto's behavior? Thanks!

Similarly, no exception is thrown on PlatformEndpoint.publish calls against a disabled PlatformEndpoint - instead the message is published. Can this also throw the standard boto exception?

Thanks for opening this! It should be fixed now.


Added error dictionary

Added error dictionary as well as logic to distinguish between:

Server/network connection error
Selector not found
Selector not visible

The dictionary source can either be a url string or url object (https://nodejs.org/api/http.html#http_http_request_options_callback). It can be set as env var NIGHTWATCH_ERROR_DICTIONARY or in nightwatch.json testSetting.default.errorDictionary
Example dictionary: https://gecgithub01.walmart.com/cahrens/test/blob/master/dictionary.json

we need a readme for the dictionary feature, including how to enable dictionary,  its format and how it works.