Systemd::JournalError: No such file or directory retrying in 1s

Hello, I'm trying out your plugin, but I'm stuck at this error.
#0 Systemd::JournalError: No such file or directory retrying in 1s

Configuration:
<source>
    @type systemd
    @log_level debug
    path /var/log/journal
    tag systemd
    read_from_head true
    path /var/fluentd/journal.pos
</source>

<match systemd>
  @type stdout
</match>

Directories (Fluentd is run as root):
 ▲ com.anrisoftware/fluentd-elk/fluentd-elk-image docker exec -it fluentd-elk-debian bash
root@20ffbc8bdcd3:/home/fluent# cd /var/log/journal/
root@20ffbc8bdcd3:/var/log/journal# ls -l
total 4
drwxr-xr-x 2 root root 4096 Apr 23 14:43 c956513c70744b3483ee4ea2b0d1b5c9
root@20ffbc8bdcd3:/var/log/journal# cd c956513c70744b3483ee4ea2b0d1b5c9/
root@20ffbc8bdcd3:/var/log/journal/c956513c70744b3483ee4ea2b0d1b5c9# ls -l
total 49156
-rw-r-----  1 root root 41943040 Apr 23 16:26 system.journal
-rw-r-----+ 1 root root  8388608 Apr 23 15:47 user-1000.journal

Changing path to /var/log/journal/c956513c70744b3483ee4ea2b0d1b5c9 or /var/log/journal/c956513c70744b3483ee4ea2b0d1b5c9/system.journal doesn't help.
Versions.
2017-04-23 16:25:12 +0000 [info]: reading config file path="/fluentd/etc/fluent.conf"
2017-04-23 16:25:12 +0000 [info]: starting fluentd-0.14.14 pid=1
2017-04-23 16:25:12 +0000 [info]: spawn command to main:  cmdline=["/usr/local/bin/ruby", "-Eascii-8bit:ascii-8bit", "/usr/local/bundle/bin/fluentd", "-c", "/fluentd/etc/fluent.conf", "-p", "/fluentd/plugins", "--under-supervisor"]
2017-04-23 16:25:12 +0000 [info]: gem 'fluent-plugin-elasticsearch' version '1.9.3'
2017-04-23 16:25:12 +0000 [info]: gem 'fluent-plugin-kubernetes_metadata_filter' version '0.26.3'
2017-04-23 16:25:12 +0000 [info]: gem 'fluent-plugin-record-reformer' version '0.9.0'
2017-04-23 16:25:12 +0000 [info]: gem 'fluent-plugin-secure-forward' version '0.4.3'
2017-04-23 16:25:12 +0000 [info]: gem 'fluent-plugin-systemd' version '0.2.0'
2017-04-23 16:25:12 +0000 [info]: gem 'fluentd' version '0.14.14'

Humm, this is weird, it looks like everything is set up correctly...
I take it you are running fluentd in a container?  How have you mounted /var/log/journal? What capabilities does the container have? What version of systemd are you using in the container... and on the host?

Hello, thank you, but it was my fault. I had two times path.
path /var/log/journal
path /var/fluentd/journal.pos

🎉


Change the header color throughout the app

Let's set a header color that falls into the background a bit more. Let's still make sure the color comes from constants/Colors.js, of course.

Oh, and if you don't already have Atom configured to highlight hex colors in the code with the color itself, I highly recommend it!

@marcelomorgado I think we can mark this as closed

Closing.


ical generation

Description

When a user joins an event session, they should be given icalendar entries generated for them.

Maybe something like this? https://github.com/jekuer/add-to-calendar-button


Offpunk 2.1

Description of changes
https://git.sr.ht/~lioploum/offpunk/refs/v2.1
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 273814 run on x86_64-linux 1

2 packages built:

offpunk
offpunk.dist


Switch from text command to slash command

As Discord is evolving months after months, a new feature has been added to use slash commands instead of text commands [1]. This improves accessibility for Discord users. That's why I thinking of moving the module in that way and deprecate text commands.
Also to support users privacy, a change wiil be made next year to Discord API to limit certified bots to use message contents [2]. I think it's a good decision and I would like to do the same for the module.
Feel free to give your feedbacks about that idea :smile:
[1] Discord.js article about the subject: https://blog.discord.com/slash-commands-are-here-8db0a385d9e6
[2] Modifications on Discord.js API : https://support-dev.discord.com/hc/en-us/articles/4404772028055

Yes , i was gonna ask for it , you should add ways to use slash module through slash commands :)

What I propose in details to implement this feature and also to keep text commands if needed by some users:

[ ] By default slash commands will be used starting from v2.2.x (at some point we can say that its a breaking change, but as this now the normal way of interacting with users, this is reasonable)
[ ] command option to set the slash/text command name will become optional and its default will be 'tictactoe'
[ ] Add a new option called useTextCommand (boolean value) to keep using legacy text command
[ ] Text commands will be deprecated from v2.2.x (no enhancement about them) and definitly removed from v3.x.x

An example on how to register a slash command using Discord v12.5:
https://github.com/redteadeveloper/Slasher/blob/f41169b49911a14fb04ebe512510fbf52cee7fc8/index.js#L29-L33

I believe the option to use text commands should always be there in addition to slash commands, since discord is only removing access to message content for verified applications all unverified bots will have access to message content and every verified application who has been granted the intent will also.
In this scenario in which the developer has access to message content it may be within your best interest to continue to provide an option for text based commands because it could be what a small server prefers, the standard for other commands the bot has or even just what they are used to.
Text commands being removed from version 3 seems it may limit or confuse users who do not want to use slash commands (especially when considering message content will not be out of reach for some developers)
Therefore surely it would be upmost inclusive to provide the option for one, the other or both even after the 9 month lee-way period from Discord themselves...
Just an idea

Also perhaps a quicker way to keep the node requirement down to only v14 could be to install a master branch version that was created prior to the addition of the version 16 requirement (I'm using a version of v13 that functions on node 14 like this)

In this scenario in which the developer has access to message content it may be within your best interest to continue to provide an option for text based commands because it could be what a small server prefers, the standard for other commands the bot has or even just what they are used to.

Thank you for the feedback. I understand your point of view and I know that a part of developers will continue to use text commands. As the module seems to be relatively stable, the third version of it should not be out before 2022-2023, so I will support both options for now and I will see how it goes in next months.
I do not want to confuse users and I would like to be as inclusive as possible. The module is designed with latest Discord guidelines and bot design practices in mind. If users are still using text commands I will leave the option as is and do not remove it completly. I have edited my last comment to take this idea into consideration.

Therefore surely it would be upmost inclusive to provide the option for one, the other or both even after the 9 month lee-way period from Discord themselves...

You are talking about the possibility to enable both at the same time? Is that really useful for users?

Also perhaps a quicker way to keep the node requirement down to only v14 could be to install a master branch version that was created prior to the addition of the version 16 requirement (I'm using a version of v13 that functions on node 14 like this)

Thank you for the suggestion, I'll check it out! I'm a bit confused because Discord.js v13 is not backwards compatible at all, so I'm afraid it will not be easy to have the same source code for both versions. I'm still working on this.

Ah alright that sounds good
And yes I wondered whether the option to enable both slash commands and text based at the same time allows for users who understand the previous system to interact while still providing the new command meta to which Discord recommend
Certainly not a need more of a convenience oriented idea
The discordjs version I linked should be able to run v13 on node 14 the only inconvenience is as you would expect from a dev branch some bugs might be present, as long as no changes affect the functionality it could work. See v13.1.0 changelog for issues present within v13.0.0 and previous commits
If it works you wouldn't even need an option for 16 you could just have the package functioning on node 14+

Slash commands are coming soon! :eyes:
In both releases 3.0.0 (discord.js v13) and 2.2.0 (discord.js v12)..

To avoid API rate limit errors, I have also decided to add two admin commands ?tttdeploy and ?tttdelete to register or unregister the command which can be defined in configuration and locale files. A new wiki page will explain how to setup it. The README will also be updated soon.


dotnet test -f net472 fails with net472/win-x86 error

I have a multitargeting test project: net472 and net8.0.
Both run tests fine with Visual Studio Test Explorer. But from the CLI, dotnet test only works on the .NET 8 target. The .NET Framework target fails with:

Nerdbank.MessagePack.Analyzers.Tests net472 failed with 1 error(s) (0.1s)
C:\Program Files\dotnet\sdk\9.0.101\Sdks\Microsoft.NET.Sdk\targets\Microsoft.PackageDependencyResolution.targets(266,5): error NETSDK1047: Assets file 'D:\source\Nerdbank.MessagePack\obj\test\Nerdbank.MessagePack.Analyzers.Tests\project.assets.json' doesn't have a target for 'net472/win-x86'. Ensure that restore has run and that you have included 'net472' in the TargetFrameworks for your project. You may also need to include 'win-x86' in your project's RuntimeIdentifiers.

Does TUnit really bring in native binaries such that I need to commit my .NET Framework test projects to architecture specific builds?

I'd need to investigate. I don't see what's causing this. Can you create a minimal repro

It turns out it's about as minimal as you can possibly imagine. This reproduces it.
repro1465.zip

Cheers. I'm out today but will take a look when I get a chance

Hmm yeah weird. dotnet test -r win-x86 works. Only thing I can think is this must be some testing platform issue? I'll ask.


Shapefile to Annotations does not work for long-lat coordinates

Reported in #674, user has found that utm coordinates work fine, but longlat coordinates show a shift. I sense, have not spent enough time to be sure, that this is a decimal precision problem with subtracting degrees.
Current workaround
Reproject images and shapefiles into UTM. This is a reasonable workaround and not terrible. Until this issue is solved, we should warn users about this known error.

Here are data to confirm.
shp = shapefile_to_annotations(rgb="/Users/benweinstein/Downloads/example.tif",
                                shapefile="/Users/benweinstein/Downloads/Plot13Ortho_example.shp")

image = rasterio.open("/Users/benweinstein/Downloads/example.tif").read()
plotted_image = plot_predictions(image=image, df=shp, thickness=3, color=(255, 0, 0))
plt.imshow(plotted_image)
plt.show()

example_data_small.zip

but transformed looks like


Announcement delegates -> responders

Small one combining both Announcement responder conversions.

That's probably a bug, then, because on the release branch this breakpoint gets called when deselecting a category:

Oops, I didn't mean to merge this one. I'll revisit the deselection code in another PR.

Oops, actually I posted this comment to the wrong PR!  It was supposed to go in #323, which just got merged. 😢🎺


CORS errorr on /traits/bulk

I get method disallowed by preflight response on /traits/bulk call. Is this something with the new edge API, or something we need to fix?

Hi @momrak - yes - this has come out of the Edge API - this should not be present in the Flutter SDK but it must have slipped through. Apologies for this. We'll work to take it out of the Flutter SDK but for now you can consider that feature deprecated.

You can achieve a bulk trait update with something like this:
## Get Flags With Identifier and Traits
curl -X "POST" "https://edge.api.flagsmith.com/api/v1/identities/?identifier=winning2" \
     -H 'X-Environment-Key: <key>' \
     -H 'Content-Type: application/json; charset=utf-8' \
     -d $'{
  "traits": [
    {
      "trait_value": 1000,
      "trait_key": "gagan_1"
    },
    {
      "trait_value": true,
      "trait_key": "my_other_key"
    },
    {
      "trait_value": "ben2",
      "trait_key": "my_other_key_string"
    }
  ],
  "identifier": "winning2"
}'

I see. I am not manually calling this, but I guess it is done when the updateTraits is called here?
      await _client.initialize();
      await _client.updateTraits(value: traits);
      await _client.getFeatureFlags(user: Identity(identifier: id), reload: true);

Also, all the flags calls you see above, I noticed on your webpage it says

How does it work?
Every time a flag is evaluated within the SDK (generally a call to a method like flagsmith.hasFeature("myCoolFeature")), the SDK keeps a track of the flag name along with an evaluation count.
Every n seconds (currently set to 10 seconds in the JS SDK) the SDK sends a message to the Flagsmith API with the list of flags that have been evaluated and their count. If no flags have been evaluated in that time window, no message is sent.

However all these calls are empty and no flags have been evaluated, so I would think now message should be sent. Is this a bug?

0 is also logged to the console for each of these requests, except when there actually was an evaluation, then it logged 1, so I guess it logs the number of evaluated flags or so, which is a bit annoying to have show up in the console

Hi @momrak  - we're looking into this and will have an update early next week - apologies for this miss.

Cool, thanks for your help :)

Fixed in current version


openvas deploy into openshift v3.5

Hi, Mike.
I'm getting this error during deploy on openshift.
I will describe the tasks I did:

Push docker image to my local repository.
Create a project in openshift with that image.
Create a persistent volume attached to /var/lib/openvas/mgr/.
Create a route to get the service. (Port 443 and 9390)

The pod is running but when I'm trying to connect to the console I can't. I'm getting: "502 Bad Gateway
The server returned an invalid or incomplete response".
I will atach the errors.
Thank you.

==> /var/log/openvas/openvassd.messages <==
[Fri Dec 15 17:48:30 2017][858] openvassd 5.1.1 started
[Fri Dec 15 17:49:43 2017][1451] Client closed the communication
[Fri Dec 15 17:50:00 2017][858] Received the Terminated signal
[Tue Jan  9 00:52:23 2018][25] openvassd 5.1.1 started
[Tue Jan  9 00:52:55 2018][118] Client not present
==> /var/log/openvas/openvasmd.log <==
md   main:MESSAGE:2018-01-09 16h18.31 utc:367:    OpenVAS Manager version 7.0.1 (DB revision 184)
md   main:   INFO:2018-01-09 16h18.31 utc:367: rebuild_nvt_cache_retry: Reloading NVT cache
md   main:   INFO:2018-01-09 16h18.31 utc:368: update_or_rebuild_nvt_cache: Rebuilding NVT cache
base gpgme:MESSAGE:2018-01-09 16h18.34 utc:368: Setting GnuPG dir to '/var/lib/openvas/openvasmd/gnupg'
base gpgme:MESSAGE:2018-01-09 16h18.34 utc:368: Using OpenPGP engine version '2.1.11'
md   main:   INFO:2018-01-09 16h18.38 utc:368:    Updating NVT cache.
==> /var/log/openvas/openvassd.messages <==
[Tue Jan  9 16:19:23 2018][385] Client closed the communication
==> /var/log/openvas/gsad.log <==
gsad main:WARNING:2018-01-09 19h01.10 utc:48: MHD: Error: received handshake message out of context
gsad main:WARNING:2018-01-09 19h01.10 utc:48: MHD: Error: received handshake message out of context
gsad main:WARNING:2018-01-09 21h10.07 utc:48: MHD: Error: received handshake message out of context

This may be related to the hostname issue discussed here: https://github.com/mikesplain/openvas-docker/pull/169

I'm currently trying to deploy this onto OpenShift Origin 3.9 and, so far, haven't gotten it to get past the Rebuilding NVT cache stage as it keeps restarting with no error log.
Here's the template manifest I'm using:
#------------------------------------------------------------------------------#
kind: Template
apiVersion: v1
metadata:
  name: openvas
  labels:
    application: openvas
objects:
#------------------------------------------------------------------------------#
- kind: ServiceAccount
  apiVersion: v1
  metadata:
    name: openvas
#------------------------------------------------------------------------------#
- kind: DeploymentConfig
  apiVersion: v1
  metadata:
    name: openvas
    labels:
      application: openvas
  spec:
    strategy:
      type: Rolling
    triggers:
    - type: ConfigChange
    replicas: 1
    selector:
      name: openvas
    template:
      metadata:
        labels:
          name: openvas
      spec:
        containers:
        - name: openvas
          image: mikesplain/openvas:${VERSION}
          ports:
          - containerPort: 443
            protocol: TCP
            name: web
          - containerPort: 9390
            protocol: TCP
            name: management
          imagePullPolicy: Always
          livenessProbe:
            httpGet:
              path: /
              port: 443
              scheme: HTTP
            successThreshold: 1
            failureThreshold: 10
            timeoutSeconds: 5
            periodSeconds: 5
            initialDelaySeconds: 30
          readinessProbe:
            httpGet:
              path: /
              port: 443
              scheme: HTTP
            successThreshold: 1
            failureThreshold: 10
            timeoutSeconds: 5
            periodSeconds: 5
            initialDelaySeconds: 30
          resources:
            limits:
              cpu: ${CPU_LIMIT}
              memory: ${MEMORY_LIMIT}
          terminationMessagePath: /dev/termination-log
          env:
          - name: PUBLIC_HOSTNAME
            valueFrom:
              fieldRef:
                fieldPath: metadata.name
          - name: OV_PASSWORD
            value: ${PASS}
          volumeMounts:
          - name: data
            mountPath: /var/lib/openvas/mgr
        serviceAccountName: openvas
        restartPolicy: Always
        dnsPolicy: ClusterFirst
        volumes:
          - name: data
            persistentVolumeClaim:
              claimName: openvas-data-pvc
#------------------------------------------------------------------------------#
- kind: PersistentVolumeClaim
  apiVersion: v1
  metadata:
    name: openvas-data-pvc
    labels:
      application: openvas
  spec:
    accessModes:
    - ReadWriteOnce
    resources:
      requests:
        storage: 10Gi
#------------------------------------------------------------------------------#
- kind: Service
  apiVersion: v1
  metadata:
    name: openvas-svc
    labels:
      application: openvas
  spec:
    ports:
    - port: 443
      name: web
      targetPort: 443
      protocol: TCP
    - port: 9390
      name: management
      targetPort: 9390
      protocol: TCP
    selector:
      name: openvas
    type: ClusterIP
    sessionAffinity: None
  status:
    loadBalancer: {}
#------------------------------------------------------------------------------#
- kind: Route
  apiVersion: v1
  metadata:
    name: openvas-route
    labels:
      application: openvas
  spec:
    port:
      targetPort: web
    to:
      kind: Service
      name: openvas-svc
    tls:
      termination: Passthrough
      insecureEdgeTerminationPolicy: Redirect
#------------------------------------------------------------------------------#
parameters:
- name: PASS
  displayName: Admin Password
  from: "[a-zA-Z]{10}"
  generate: expression
- name: VERSION
  displayName: OpenVAS Version
  value: "9"
- name: CPU_LIMIT
  displayName: OpenVAS CPU Limit
  value: "60"
- name: MEMORY_LIMIT
  displayName: OpenVAS Memory Limit
  value: 5Gi

I run the deployment with an openvas Service Account which has the privileged Security Context Constraint, which allows the deployment to to host mount and use anyuid.

Seems like it was the liveness/readiness probes that were breaking the deployment.
After I took them out I got an SSL error, it was the hostname environment variable.
Fixed by setting the hostname variable to the value of the ROUTE_HOST variable
#------------------------------------------------------------------------------#
kind: Template
apiVersion: v1
metadata:
  name: openvas
  labels:
    application: openvas
objects:
#------------------------------------------------------------------------------#
- kind: ServiceAccount
  apiVersion: v1
  metadata:
    name: openvas
#------------------------------------------------------------------------------#
- kind: DeploymentConfig
  apiVersion: v1
  metadata:
    name: openvas
    labels:
      application: openvas
  spec:
    strategy:
      type: Rolling
    triggers:
    - type: ConfigChange
    replicas: 1
    selector:
      name: openvas
    template:
      metadata:
        labels:
          name: openvas
      spec:
        containers:
        - name: openvas
          image: mikesplain/openvas:${VERSION}
          ports:
          - containerPort: 443
            protocol: TCP
            name: web
          - containerPort: 6379
            protocol: TCP
            name: redis
          - containerPort: 9390
            protocol: TCP
            name: management
          imagePullPolicy: Always
          resources:
            limits:
              cpu: ${CPU_LIMIT}
              memory: ${MEMORY_LIMIT}
          terminationMessagePath: /dev/termination-log
          env:
          - name: PUBLIC_HOSTNAME
            value: ${ROUTE_HOST}
          - name: OV_PASSWORD
            value: ${PASS}
          volumeMounts:
          - name: data
            mountPath: /var/lib/openvas/mgr
        serviceAccountName: openvas
        restartPolicy: Always
        dnsPolicy: ClusterFirst
        securityContext:
          fsGroup: 0
        nodeSelector:
          kubernetes.io/hostname: alfred.zenaptix.net
        volumes:
          - name: data
            persistentVolumeClaim:
              claimName: openvas-data-pvc
#------------------------------------------------------------------------------#
- kind: PersistentVolumeClaim
  apiVersion: v1
  metadata:
    name: openvas-data-pvc
    labels:
      application: openvas
  spec:
    accessModes:
    - ReadWriteOnce
    resources:
      requests:
        storage: 10Gi
#------------------------------------------------------------------------------#
- kind: Service
  apiVersion: v1
  metadata:
    name: openvas-svc
    labels:
      application: openvas
  spec:
    ports:
    - port: 443
      name: web
      targetPort: 443
      protocol: TCP
    - port: 9390
      name: management
      targetPort: 9390
      protocol: TCP
    - port: 6379
      name: redist
      targetPort: 6379
      protocol: TCP
    selector:
      name: openvas
    type: ClusterIP
    sessionAffinity: None
  status:
    loadBalancer: {}
#------------------------------------------------------------------------------#
- kind: Route
  apiVersion: v1
  metadata:
    name: openvas-route
    labels:
      application: openvas
  spec:
    host: ${ROUTE_HOST}
    port:
      targetPort: web
    to:
      kind: Service
      name: openvas-svc
    tls:
      termination: Passthrough
      insecureEdgeTerminationPolicy: Redirect
#------------------------------------------------------------------------------#
parameters:
- name: PASS
  displayName: Admin Password
  from: "[a-zA-Z]{10}"
  generate: expression
- name: VERSION
  displayName: OpenVAS Version
  value: "9"
- name: CPU_LIMIT
  displayName: OpenVAS CPU Limit
  value: "60"
- name: MEMORY_LIMIT
  displayName: OpenVAS Memory Limit
  value: 5Gi
- name: ROUTE_HOST
  displayName: OpenVAS Hostname route
  value: openvas.apps.zenaptix.net

Is it really necessary for OpenVAS to rebuild the NVT Cache every time it gets run?
Is there any way to persist the NVT Cache so that it only gets run once?
How about multi-threading the process?


Fix the missing lines in the graph track

With this change, we fix the problem of missing horizontal lines in the
graph track by drawing lines at the center of pixel.
Bugs: http://b/192195534
Test: Profile OrbitTest and check whether there are missing lines in the variable tracks.
Before the change:
https://screenshot.googleplex.com/3PfhJDR3UFPe7MH
Some lines are missing in the variable track DynamicName_4 and double_var.
After the change:
https://screenshot.googleplex.com/ApX5U6USdLZCQCA

Hi @reichlfl , @pierricgimmig ! Could you help to provide some suggestions on this problem?

Hi @reichlfl , @pierricgimmig ! Could you help to provide some suggestions on this problem?

I assume the reason this fixes it is that OpenGl defines the pixel center to be offset by (0.5, 0.5) from the pixel coordinates (from the OpenGl 3.0 spec https://www.khronos.org/registry/OpenGL/specs/gl/glspec30.pdf, page 112), so with your change, the y-coordinate lies exactly at the pixel center.
It's hard to say what exactly caused the issue - in the end, OpenGl isn't well suited for pixel-perfect drawing, and especially with single-pixel elements you're always a bit at the mercy of driver implementation and the hardware rasterizer. Were you able to reproduce this with multiple setups? It could be something that is only happening with certain drivers, remote desktop, GPUs...
Which also means that this isn't guaranteed to fix it in all setups, but it looks like an edge case or even bug to me if the line just disappears completely, and my guess would be that this should guarantee the line to show up given that it's exactly at pixel center. To be sure, we could think about extending the line width (https://www.khronos.org/registry/OpenGL-Refpages/gl4/html/glLineWidth.xhtml) to be more than 1 pixel, but I don't know if that will still look good?

Increasing the line width also solves the problem. This is how the line graph looks like when line widths = 2
https://screenshot.googleplex.com/6SEJTQFV5nvv7dg.
I'm not sure whether adding a (0.5, 0.5) offset will fix the problem for all the settings (only tested on my workstation and cloudtop). Maybe extending the line width is a more reliable solution.

Increasing the line width also solves the problem. This is how the line graph looks like when line widths = 2
https://screenshot.googleplex.com/6SEJTQFV5nvv7dg.
I'm not sure whether adding a (0.5, 0.5) offset will fix the problem for all the settings (only tested on my workstation and cloudtop). Maybe extending the line width is a more reliable solution.

I actually think this looks better than before. Maybe increase the size of the bullets as well so they are centered around the lines?

Increasing the line width also solves the problem. This is how the line graph looks like when line widths = 2
https://screenshot.googleplex.com/6SEJTQFV5nvv7dg.
I'm not sure whether adding a (0.5, 0.5) offset will fix the problem for all the settings (only tested on my workstation and cloudtop). Maybe extending the line width is a more reliable solution.

I actually think this looks better than before. Maybe increase the size of the bullets as well so they are centered around the lines?

Increasing the line width also solves the problem. This is how the line graph looks like when line widths = 2
https://screenshot.googleplex.com/6SEJTQFV5nvv7dg.
I'm not sure whether adding a (0.5, 0.5) offset will fix the problem for all the settings (only tested on my workstation and cloudtop). Maybe extending the line width is a more reliable solution.

Could it be that lines are not all equal in the screenshot? (at least speaking about color)

Could it be that lines are not all equal in the screenshot? (at least speaking about color)

Looks a bit like it. Not sure if this is due to the screenshot, it could also result from Antialiasing.


Expose Git SHA of webservice build

Feature Request
Now that we are looking at deploying daily builds internally, it would be useful to know the Git SHA of the last commit of the particular build. This hasn't been an issue for our prod and staging environments because those are based on GitHub releases.
Looks like we can use the commit-id plugin to gather the information. The question is how to expose it:

We always add it to the version, writing it out to the manifest, which then gets surfaced in the API. Could make the footer in the UI look a little messier, although that could be handled in the UI (display commit id only if version does not contain SNAPSHOT).
We conditionally add it to the version, i.e., only add if web service is not being built off a release. This looks tricky in Maven though.
We expose a new endpoint, something like /metadata/buildinfo where this additional information is exposed.

┆Issue is synchronized with this Jira Story
┆Issue Type: Story
┆Fix Versions: Dockstore 1.9
┆Sprint: Seabright Sprint 28 Cuttlefish
┆Issue Number: DOCK-1159
┆Epic: Continuous Deploy

Visible with metadata endpoint in WS 1.9.0-alpha.9-SNAPSHOT


add package exports

(for Typescript node16 package resolution)

@gunters63 thanks for this, I'm not familiar at all with module resolution to be honest, I'm relying on preconstruct to bundle that for me. There's an export option that maybe we can use to automatize all this? I'll dig deeper into this next week.
PS: in the repro you've posted in #544 @use-gesture/react is missing from deps, but I was able to reproduce the issue after installing it.

@gunters63 do you have a sample repro case that I could investigate to check out how exactly it's failing today with TS moduleResolution: node16?

I played a bit with the preconstruct export option and I think it cannot be used for now as it does not generate "types" entries needed by Typescript >= 4.7

@gunters63 note that types entries are not strictly required. You can rely on "sibling lookup" and it works just fine - this is what preconstruct does at the moment. For each entry in exports you should have a sibling .d.ts file that TS should pick up.

@Andarist: Does the repo given by @a-type in #544 work?

@gunters63 note that types entries are not strictly required. You can rely on "sibling lookup" and it works just fine - this is what preconstruct does at the moment. For each entry in exports you should have a sibling .d.ts file that TS should pick up.

Ah ok, so maybe adding the export option to package.json and run preconstruct fix will work.

@gunters63 yes, I've just missed it at first since it wasn't linked to from the description of this PR. Thank you for pointing me in the right direction.

Ah ok, so maybe adding the export option to package.json and run preconstruct fix will actually work.

I've checked that and ti fixes the problem of looking up the correct type definition file but that file looks, at the moment, like this:
export * from "./declarations/src/index";

and that in turn fails on the lack of the extension here

@Andarist: Interesting. I thought the extension is only needed for Node. The generated esm and cjs files by preconstruct all have extensions.

@Andarist: Interesting. I thought the extension is only needed for Node. The generated esm and cjs files by preconstruct all have extensions.

Unfortunately, it seems that they are now required in .d.ts too. I made it work locally by adjusting the files in node_modules - the .js extension has to be added to all imports within .d.ts

I generated the exports again with preconstruct fix (with exports option enabled) and build with preconstruct build.
When I manually copied the package over to my repos node_modules folder it seemed to work.
Typescript did not choke about the missing extensions in export * from "./declarations/src/index";
I will do more testing tomorrow.

I updated the PR now to use preconstruct export .

@Andarist: Interesting. I thought the extension is only needed for Node. The generated esm and cjs files by preconstruct all have extensions.

Unfortunately, it seems that they are now required in .d.ts too. I made it work locally by adjusting the files in node_modules - the .js extension has to be added to all imports within .d.ts

Where did the missing .js extension in the .d.ts files make problems?
I used the repo from #544 as base, fixed the missing dependency and just add edthe missing exports to the package,json of core use-gesture manually. Everything was fine after that, typings working, pnpm build also. I then extended the repo with some sample code from use-gestures sandboxes to make sure it was also working at runtime.

I experienced the missing extension causing issues in .d.ts with another package (https://github.com/stitchesjs/stitches/pull/1086). Not sure exactly what conditions cause it, but it was definitely showing up for me there.

Hmm, if that would be that case (.d.ts files missing file extensions in imports/exports generally cause problems with type resolution under nodenext) then many packages on npm would be broken.
I just checked in our Monorepo (which is all nodenext resolution), most direct dependencies have .d.ts files without extensions.
It would be really interesting to know when exactly this causes a problem.
In any case, the added exports fixed the problem in the reproduction repo (and in our project). I propose to merge this and file a bug report in the preconstruct issue tracker about the extensionless exports/imports.

In any case, the added exports fixed the problem in the reproduction repo (and in our project). I propose to merge this and file a bug report in the preconstruct issue tracker about the extensionless exports/imports.

Thought a little bit about it more. Adding exports to a npm package can be a breaking change for some consumers if they use sub path imports of  files which are restricted now through the new exports. Not sure it can be the case here.

Where did the missing .js extension in the .d.ts files make problems?

I didn't recheck this right now to be 100% confident but I think it starts to be required when you are using moduleResolution: node16 and when the imported package has package.json#exports. This is specifically to match the "modern" node.js requirements (the presence of exports in node.js requires the extension for all relative imports/requires in that package).
I definitely had this problem here, when playing with this locally. I can't be sure how our testing methodologies were different and that we didn't experience the same thing.

Hmm, if that would be the case (.d.ts files missing file extensions in imports/exports generally cause problems with type resolution under nodenext) then many packages on npm would be broken.

That's true - and that's why I believe that the second precondition is required for this. This is only required when a package has package.json#exports (and most packages on npm don't have that, that's why extensionless sources are OK, even with moduleResolution: node16).

In any case, the added exports fixed the problem in the reproduction repo (and in our project). I propose to merge this and file a bug report in the preconstruct issue tracker about the extensionless exports/imports.

I'm against that. I can't be sure without further analysis but I think there is a potential for this PR (in its current form) to break some consumers. I would only proceed with this after we ensure that extensions are there.

Thought a little bit about it more. Adding exports to a npm package can be a breaking change for some consumers if they use sub path imports of files and those are restricted now. Not sure it can be the case here.

That's true. It's up to the maintainer to decide if adding exports to their package is a breaking change or not. I think that it should be mostly fine in this case here - if only done right.

Where did the missing .js extension in the .d.ts files make problems?

I didn't recheck this right now to be 100% confident but I think it starts to be required when you are using moduleResolution: node16 and when the imported package has package.json#exports. This is specifically to match the "modern" node.js requirements (the presence of exports in node.js requires the extension for all relative imports/requires in that package).
I definitely had this problem here, when playing with this locally. I can't be sure how our testing methodologies were different and that we didn't experience the same thing.

I tried exactly this with the reproduction repo, patched use-gesture's package.json to have those exports and changed the repo's tsconfig.json to moduleResolution node16 and didn't have problems.  We should really try to nail this down.
I am not sure the node16 resolution requirement (always have extensions) fully applies to .d.ts files too (which are invisible to Node of course). I didn't find an exact specification in the Typescript documentation.

Hmm, I've done some new tests after creating a fresh copy of that repro and it seems to work correctly now (even without extensions). I wonder what did I do differently during the weekend - since I definitely experienced a problem like that 🤔

Let's go then! Thanks everyone.

@dbismut feel free to ping me if anything goes south. I'm happy to help you investigate any potential issues.

I just tried 10.2.23 in our monorepo and the typing problems went away :)
I am happy to help too if problems arise of course.

Thanks a lot for this!!

@Andarist: Btw, I found this interesting issue in the typescript tracker: https://github.com/microsoft/TypeScript/issues/50482
use-gesture is not marked as "module" so we can get away with missing extensions in the .d.ts files here I guess :)

Hey @Andarist @gunters63 hope you're well and having a nice Christmas break.
I was made aware that this sandbox doesn't work properly on Codesandbox since last upgrade.
https://codesandbox.io/s/github/pmndrs/use-gesture/tree/main/demo/src/sandboxes/card-zoom
It displays a warning related to importing dragAction and pinchAction, which are exported variables from '@use-gesture/core/actions':
import { createUseGesture, dragAction, pinchAction } from '@use-gesture/react'

Not sure what happens, as when I download the sandbox locally (from csb) it seems to be working normally. When downgrading to 10.2.22 the sandbox seems to work again.

I believe that this is the very same problem that I was reporting to the CodeSandbox team a couple of months ago (thread in their Discord). It seems that the issue has not been resolved.
The problem is that the bundler~ that they are using for their online codesandboxes has bugs when it comes to package resolving. That's why it works locally - since then you are using webpack directly and it handles this situation correctly.
I would recommend reaching out to the CodeSandbox team again, letting them know about this issue and hoping that they find the time to fix this. When I reported this, I was looking for a place to fix this but IIRC I've concluded that it's in their private repos so I couldn't do much about this myself.

@Andarist thanks for the detailed explanation. When I look for the @use-gesture/core module in the Sources tab from Chrome inspector:

this is what I see locally:

this is what I see on csb:

I don't know if this is related but indeed something is different.

Yeah, exactly - as you can see on CodeSandbox a mix of CJS and ESM files was loaded. This creates a "dual package problem". You can read more about dual package hazard here. I call it a problem here since it's not a hazard anymore, it created a problem :P.
However, this just shouldn't happen here. There is no hazard in the package setup and thus the problem should not occur. The broken resolution algorithm in CodeSandbox creates that hazard and thus the problem. This looks exactly like what I reported.

I find it odd that Codesandbox transpiles App.tsx always to CJS, even when you set   "type": "module", in package.json.
This is not the first time I had problems like this in Codesandbox too. The Javascript world should really move forward to ESM now...

Yes, and with 10.2.22 (without the exports) Codesandbox only loads the ESM modules from use-gesture:

I think that it's best to wait till January and then try to ping Ives/CSB team about this issue through the appropriate support channels.

Guys I'm not sure what to do. Half of my interactions on use-gesture are about the lib not working on CSB. It's quite a big deal, I can't really ignore the fact that usage with CSB is probably greater than custom module resolution settings for typescript (please prove me wrong!)

Its really unfortunate that CSB is hindering adopting new Javascript standards.
The user base relying on exports is surely much smaller (Maybe I am the only one :D). I could live with a rollback, I use pnpm and could use its patch feature in my repository.
Another solution could probably be to release a second module use-gesture-nodenext on npm which contains the package exports, the normal use-gesture package would have the old-style package.json. This should probably be quite easy to do.

There is also a chance that adding exports to the "nested" package.json files (such as this one) would solve the problem.
This shouldn't be needed - and in fact, that could be very misleading. exports are only read from the root package.json file. The only thing that should be read from the "closest" one is the type property. I've seen bundlers misimplementing this though - and perhaps that's the case here. So there is a chance that we could, temporarily, make their broken implementation work.

One last huge thanks to @Andarist that helped Codesandbox fix this! Case closed.


[Proposal] Add the ability to right click property/class/any midl entity and click "Copy $entity to clipboard"

Is your feature request related to a problem? Please describe.
While the update header file approach is great, the git diff editor is a bit overkill. A common workflow of idl editing is just adding a property or two, and it'd be a visual and tooling overhead for user to have to stare at the diff window to look for the properties he wanted.
Describe the solution you'd like
Adding a contextual "Copy $midl-enity to clipboard" command eliminates this overhead completely. Typical user story is that the user just edit the idl -> right click the entity -> click the command -> wait for idl to compile -> see success status -> go back to header file -> paste.
In the long run, it'd be ideal if the extension can understand text selection and copy more than 1 properties/events/class/enum/any midl entities. For now, even just the ability to copy a single entity's definition would suffice.
Describe alternatives you've considered
N/A
Additional context
Since this extension is midl tool, not specifically for C++, ideally the context menu should be able to detect current language, and provide C++ or, say. rust suggestion. As a fallback when the extension could not detect the language, a sub menu in the context menu should always be present and provide a list of supported language for users to choose from.
Implementing this feature with languages other than C++ in mind could potentially help even more people in the future.

The naming isn't right yet but the feature is there, albeit with some issues (sometimes too much stuff is copied). Will open follow up issues.


test(scalable): ⚡ use const collection literals whenever possible

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1061

Relevant Lines:
1061

💛  - Coveralls


Overhaul transitive module handling in dependency resolution

This PR the way transitive dependencies, but also transitive BOM dependencies and dependency management, are passed to coursier.
Before this commit, Mill was walking its own module graph, and dependencies / BOM dependencies / dep management were manually aggregated beforehand. During dependency resolution, coursier also does such aggregations for external dependencies.
Having both Mill and coursier handle that concern could lead to discrepancies or duplicate work (in https://github.com/com-lihaoyi/mill/pull/4068 in particular). So this commit passes non-processed dependencies / BOM dependencies / dep management as-is to coursier, and lets coursier handle / compose those.
With this PR, dependencies / BOM dependencies / dep management are passed around solely via JavaModule#coursierProject / JavaModule#transitiveCoursierProjects. This PR gets rid of JavaModule#{transitiveIvyDeps,transitiveCompileIvyDeps,transitiveRunIvyDeps,processDependency,processedIvyDeps} (some only recently introduced in https://github.com/com-lihaoyi/mill/pull/3924 - stubs should need to be re-instated before merging for some, to preserve bin-compat).

This voluntarily removes JavaModule#{transitiveIvyDeps,transitiveCompileIvyDeps,transitiveRunIvyDeps}, in order to see what break

I'm going to try to rebase https://github.com/com-lihaoyi/mill/pull/4068 on top of that, to have a better sense of whether: the changes here are a nice-to-have, or are required or workarounds would be too hard to implement.

Some tests will fail until courier 2.1.22 can be used (the changes here need https://github.com/coursier/coursier/pull/3189)

Going to describe more in length what these changes are about - how the Mill module graph and dependency details are now converted to coursier data structures. Maybe in a comment in the code.


Unit | Component | sl progress bar: There are no references to Ember.$, $ or jQuery

not ok 567 PhantomJS 1.9 - Unit | Component | sl progress bar: There are no references to Ember.$, $ or jQuery
    ---
        actual: >
            null
        message: >
            Died on test #1     at test (http://localhost:7357/assets/test-support.js:3025)
                at testWrapper (http://localhost:7357/assets/test-support.js:6192)
                at test (http://localhost:7357/assets/test-support.js:6205)
                at http://localhost:7357/assets/tests.js:25920
                at http://localhost:7357/assets/vendor.js:152
                at tryFinally (http://localhost:7357/assets/vendor.js:33)
                at http://localhost:7357/assets/vendor.js:158
                at http://localhost:7357/assets/test-loader.js:60
                at http://localhost:7357/assets/test-loader.js:51
                at http://localhost:7357/assets/test-loader.js:82
                at http://localhost:7357/assets/test-support.js:6024: Attempted to wrap $ which is already wrapped
        Log: |
    ...

Will likely be resolved by https://github.com/softlayer/sl-ember-components/issues/1503

Resolved by https://github.com/softlayer/sl-ember-components/issues/1503


Load Material Icons Font From Assets Pipeline

Closing this in favor of https://github.com/navinpeiris/pnotify-rails/pull/12


Swap events between two scheduler

For the reasons of absence of nested resources in scheduler, i've got an idea to render two separate schedulers. Can i drag and drop from one scheduler to another ones events i needed?

@KirillTerentiev
Yes, it's possible, see this issue


[Activation] Fix TxMempoolBlockDoublespend

looks good.
however I'm against spawning more and more different types of consensus errors.


Made documentation locations more clear.

Puts them all under the the Documentation header, and adds the wiki as an explicit source of documentation.

Coverage increased (+3.05%) to 84.283% when pulling c64da8e89f048e10904d78b5fd41c5d6303637a9 on Phlosioneer:patch-2 into ec2c5fe5f400810133d352ec7bbeddb8ce1aca87 on Geal:master.

Hello, thanks for the suggestion, but I put the contact information at the top on purpose, so that people find it easily. So I'd prefer you do it the other way, putting the documentation link at the top with a clear message.

the documentation links changed


Update DEVELOPERS.md

fix links to menu 'Writing Documentation' and Lerna
Obs: There's no issue to this correction
Changes

Fix some links in DEVELOPERS.md

Writing Documentation line 181
Lerna line 31

Coverage remained the same at 99.584% when pulling 3bfe3777627c12c169fd0f38ec3c2eed9647c61b on alvesgabriel:update-developers into 4e4fcea8c327f61ea47315792a3593842d4a58dc on accordproject:master.


Instance: SlicerMorph 101; Meir Barak

Cloud Computing Instance Flavor
g3.xl - GPU instance (32 CPUs, 125 GB RAM and A100 GPU)
Description
I'm an Associate Professor at Long Island University, College of Veterinary Medicine. I'm working on bone structure function relationship (bone modeling and remodeling). My current project looks at the structure of sesamoid bones.  I have micro-CT scans of about 150 specimens, and would like to use SlicerMorph to segment and extract models.
ORCID
0000-0003-4953-4380
Email
U2FsdGVkX1+FmvsNXIL+7kH4FUu0bN2jB935P5lI/o0nIG5ukX5GwgYfaGoAQrNT
Confirm Email
U2FsdGVkX1+FmvsNXIL+7kH4FUu0bN2jB935P5lI/o0nIG5ukX5GwgYfaGoAQrNT

/create

/unshelve

/unshelve

/unshelve

/unshelve


[Swift APIView] Remove "textDescription" crutch from TypeModel

Several types are not parsed out completely and instead use textDescription as a hack. These should be removed.

[x] TypeIdentifier
[x] FunctionType
[x] DictionaryType

These were fixed in #2535.


Tooltip does not show up again after hide on ios when virtual keyboard closes

When the virtual keyboard disappears, the tooltip appears on top of the page for a second and disappears . After that, clicking on the span to show the tooltip does not show it anymore. I'd have to click somewhere else first before clicking on the span again for the tooltip to show.
Is there a way around this?
EDIT: I tried adding position: absolute to the tooltip and it doesn't show on the top of the page anymore. However, it still disappears when the virtual keyboard closes and won't open again until I tap somewhere else first then tap on the icon for the tooltip again.

@itmart have you found a solution?


Altering waypoints

Is it possible to alter the number of waypoints or set a maximum distance between two waypoints?

The number of waypoints comes from the algorithm. Of course we can describe a straight line with more points, but why would that be beneficial?

So when dealing with obstacles, it skips the waypoints as intended but at the cost of large areas not being covered - with more waypoints I am hoping I can achieve better coverage
Any tips where I should look into for adding more points?

That sounds like it's a local-planner issue. So the planner should try to stick to the global plan as good as possible. I don't think it's the responsibility of this global planner to cover that?


Broken Links on PyPI Page

The URLs provided for the implementation examples in the "Examples" section on the PyPI page for the solana-actions package appear to be broken. The links are intended to direct users to examples using Flask and FastAPI, but they currently lead to non-existent pages.
Broken Links:
Using Flask - https://pypi.org/project/solana-actions/examples/flask/
Using FastAPI - https://pypi.org/project/solana-actions/examples/fastapi/
Steps to Reproduce:

Go to the PyPI page for solana-actions.
Scroll down to the "Examples" section.
Click on the links labeled "Using Flask" and "Using FastAPI".

The issue seems to be with the URLs provided on the PyPI page. Kindly update the links to direct users to the correct examples.
Thank you for your attention to this matter.

Nice find. Thanks for pointing this out.
Fix is in the pipelines. Pypi should be updated soon


SqlClient in "Migrate from ASP.NET Core 2.2 to 3.0"

The document should indicate the needed of include a package reference to System.Data.SqlClient since some third-party library (like Dapper) may stop working properly.

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

ID: 76f39eec-9dd1-6167-56ec-47b26e0b5063
Version Independent ID: f9f2b0f3-01eb-cf58-8767-194fe0fc0e44
Content: Migrate from ASP.NET Core 2.2 to 3.0
Content Source: aspnetcore/migration/22-to-30.md
Product: aspnet-core
Technology: aspnetcore-migration
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Because this is an implementation detail for Dapper, we won't be adding it to our docs. It's already mentioned in the release notes for Dapper v2.0.4 (see https://stackexchange.github.io/Dapper/). If possible, you may want to consider using Microsoft.Data.SqlClient as a drop-in replacement for System.Data.SqlClient. Reasons are outlined here: https://devblogs.microsoft.com/dotnet/introducing-the-new-microsoftdatasqlclient/.


Move context cancellation handling from context callbacks to executor…

…s. The possibility for an executor to reject a callback is well defined and libraries that use executors should work properly with these semantics.
Currently, the cancellation happens within the callback, instead of a user callback. This can work in some cases, but not in many (e.g., usually an error in a user callback will be propagated to a different, error-handling user callback. If this later one is also cancelled by the executor, there is no way to handle the error). Replacing with rejection of the context during submission to an executor should prevent this issue as task rejection is a native feature of executors.

@trustin @kojilin PTAL

I'm not sure if rejecting at submission time solves the case where a transaction is composed of multiple Runnables (e.g. a state machine) and one of them is a clean-up task. A submitter will not be able to submit the clean-up task to the executor because the context has been timed out already.

@trustin Updated to remove automatic exceptions

hmm, so don't reject anymore and let user using isTimedOut ?

Yeah I guess it's more general, not sure of a good way to safely and smartly reject in the context.

Thanks, @anuraaga!


Fix 'FieldInfo' is not declared in all

Change Summary
Sometimes we need FieldInfo for type annotation, this PR add FieldInfo to __all__

Related issue number

Checklist

[ ] The pull request title is a good summary of the changes - it will be used in the changelog
[ ] Unit tests for the changes exist
[ ] Tests pass on CI
[ ] Documentation reflects the changes where applicable
[x] My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

Selected Reviewer: @sydney-runkle

please review

Hi! Thanks for the PR!
We generally try to make sure that if something is included in __all__ for pydantic.some_module, we also include it in __all__ for pydantic. We don't want this to be the case for FieldInfo. FieldInfo isn't private per-say, but I do think we want to maintain a less public presence for this member so that we have a bit more autonomy to change things if necessary.
Going to close for now. cc @Viicos, feel free to follow up if you disagree with this decision.

@sydney-runkle
Thank you for your answer.
But I still don't understand how to solve this problem?

We generally try to make sure that if something is included in __all__ for pydantic.some_module, we also include it in __all__ for pydantic.

Maybe we could revisit this rule at some point, but indeed would be great to stabilize the FieldInfo API first.

But I still don't understand how to solve this problem?

You can probably configure PyCharm to ignore this kind of errors or add some kind of ignore comment here.


No podcast browser window

Hello, I have installed the plugin, and added few podcasts in the plugin settings. Unfortunatelly, nothing happens when I try to "Play podcast" cannot find the window where I can actually browse my selected podcast.
Using Macbook air with M2. No errors in Obsidian console.

Hi @Matoo125,
Do you have this command available to you?

If so, then activating it would make the pane appear. It appears in the right sidebar with an icon denoting playback:

If you do not have the command accessible, that means Obsidian believes the pane to already exist somewhere. In which case we'll need to do some debugging.


Use current working directory if --input parameter is not specified.

Solves #1301

@marschwar the original issue didn't close when this PR was merged - see here for keywords Github recognises in PR descriptions and commit comments to automatically close issues when PRs are merged: https://help.github.com/articles/closing-issues-using-keywords/

Thanks @3flex for pointing that out. Turns out, solves seems to be about the only word that does not work :+1:


fix: fetch the larger image urls, remove default param

This PR resolves []
Description

The image quality in certain images in articles was still suffering because we weren't fetching high quality image urls. Instead just fetching url and getting back the medium url which is very small and resizing that.
This fetching was just copied from force so 👀 appreciated.
Before

After

PR Checklist

[ ] I have tested my changes on iOS and Android.
[x] I hid my changes behind a feature flag, or they don't need one.
[x] I have included screenshots or videos, or I have not changed the UI.
[x] I have added tests, or my changes don't require any.
[x] I added an app state migration, or my changes do not require one.
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

Higher quality images in articles - Brian

iOS user-facing changes

Android user-facing changes

Dev changes

Need help with something? Have a look at our docs, or get in touch with us.

This PR contains the following changes:

Cross-platform user-facing changes (Higher quality images in articles - Brian)

  Generated by :no_entry_sign: dangerJS against 70194025adbbe1cb643ac629f2b361e38e8e4113


Add performance tests

This PR adds a new CI workflow which runs performance tests for 2 kind clusters.
The tests are:

vl3
kernel2wireguard2kernel

Issue: https://github.com/networkservicemesh/integration-k8s-kind/issues/796
Example of a run:
https://github.com/d-uzlov/integration-k8s-kind/actions/runs/4625605544/jobs/8181518622
The results can be downloaded as an artifact and inspected locally.

@d-uzlov Do you have any updates?


Window resizing causes numbers to be positioned incorrectly.

After resizing the window and the font sizes changes, the numbers seem to be positioned incorrectly.
I think it's because the translations are all based on pixels (height of numbers divs).
Is there a way to use percentages for the translation? That way it will always stay relative to the container/font size.

Hi, did you manage to solve this issue? I got the same problem.

I am also experiencing this issue.

Here's a super hacky temporary fix... assuming you're using react, add this to your functional component:
    // this is used to trigger a re-render because of the bug in AnimatedNumbers
    const [, updateState] = useState<{}>()
    const rerender = useCallback(() => updateState({}), [])

    useEffect(() => {
       addEventListener("resize", rerender)
        return () => {
            removeEventListener("resize", rerender)
        }
    }, [])

Essentially, whenever the window size changes, we force the current component (containing the AnimatedNumbers component) to re-render so that the numbers get positioned correctly. Just be mindful that this is bascially bypassing the optimisations in the react rendering lifecycle so yeah.


Fix: Shows error when report has invalid logical table

Would crash UI otherwise
Likely to happen when users share reports to a table that
one user doesn't have access
Also hides components that would crash without metadata
that user wouldn't be able to use anyway
Allows users to select a valid table.
Changes error message to reflect the actual problem

Will choosing a table with the table-selector cause the dimensions and metrics to load? This screenshot still looks like it would crash the UI.

Yes the user can recover from the error. We show the table selector here for that purpose. Choosing a valid table will cause the metrics and dimensions to load.


[FLINK-32664][table] Fix TableSourceJsonPlanTest.testReuseSourceWithoutProjectionPushDown

What is the purpose of the change
The PR is aiming to fix  failing ci tests especially TableSourceJsonPlanTest.testReuseSourceWithoutProjectionPushDown which became broken after https://issues.apache.org/jira/browse/FLINK-32657
https://github.com/apache/flink/commit/6a3808213d334de614e162362d1583f3e5322358
Verifying this change
The change is change of tests
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): ( no)
The public API, i.e., is any changed class annotated with @Public(Evolving): ( no)
The serializers: ( no )
The runtime per-record code paths (performance sensitive): ( no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: ( no)
The S3 file system connector: ( no)

Documentation

Does this pull request introduce a new feature? ( no)
If yes, how is the feature documented? (not applicable)

//cc @LadyForest , @twalthr

@flinkbot run azure

flink-table-module built. Merging this now to make master stable again.


Organize folders

Put the files in the correct folders structure to be more easy to update

First, thanks for the contribution. Cleaning code is always a good idea.
Regarding the second commit, there's validatemodule.sh to find out whether the change is good. One runs it like this:
cd modules/blocksearch
../../tools/validatemodules.sh

This script also supports a number of parameters, run it with -h to get a description.
Regarding the first commit, there should be no need for such code changes. There are "magic" places where templates, JS and CSS get found by just their name. However, I'm not aware of documentation other than the code. The secret lies somewhere between lines 3075 and 3200 in classes/module/Module.php for templates. And somewhere in Media.php and/or Controller.php for JS and CSS.

I run the validation, have much things to change, in next days i will update it.


Transactions page not showing all transactions

For some collectives(it's certainly happening to most hosts), the list of transactions is missing a lot of transactions.
Examples:

https://opencollective.com/opencollective-host/transactions

https://opencollective.com/opensourcecollective/transactions

technical info: It's something wrong with the method Collective.getTransactions.

In the end this wasn't actually a problem, everything looks good so I'm closing this issue...
Though, I can propose as just a small improvement:
Original query
"Original" query before the UsingVirtualCardFromCollectiveId change:
{ where: { CollectiveId: 11004 },
  order: [ [ 'createdAt', 'DESC' ] ],
  limit: 20 }

UsingVirtualCardFromCollectiveId query
After considering the field UsingVirtualCardFromCollectiveId on this query we have 2 options:

Collective.prototype.getTransactions with includeUsedVirtualCardsEmittedByOthers true:

{
   where:{
        [Symbol(or)]:[
            {
                type:'DEBIT',
                [Symbol(or)]:{
                    CollectiveId:11004,
                    UsingVirtualCardFromCollectiveId:11004
                }
            },
            {
                type:'CREDIT',
                CollectiveId:11004
            }
        ]
   },
   limit:20,
   order:[[ 'createdAt', 'DESC']]
}

Collective.prototype.getTransactions with includeUsedVirtualCardsEmittedByOthers false:

{
   where:{
      [Symbol(or)]:[
         {
            type:'DEBIT',
            [Symbol(or)]:[
               {
                  CollectiveId:11004,
                  UsingVirtualCardFromCollectiveId:null
               },
               {
                  UsingVirtualCardFromCollectiveId:11004
               }
            ]
         },
         {
            type:'CREDIT',
            CollectiveId:11004
         }
      ]
   },
   limit:20,
   order:[['createdAt', 'DESC']]
}

Small improvement
This query works fine but I believe we could make it simpler:

Collective.prototype.getTransactions with includeUsedVirtualCardsEmittedByOthers true:

{
   where:{
        [Symbol(or)]:[
            {
                CollectiveId:11004
            },
            {
                type:'DEBIT',
                UsingVirtualCardFromCollectiveId:11004
            }
        ]
   },
   limit:20,
   order:[['createdAt', 'DESC']]
}

Collective.prototype.getTransactions with includeUsedVirtualCardsEmittedByOthers false:

{
   where: {
      CollectiveId:11004
   },
   limit:20,
   order:[['createdAt', 'DESC']]
}

@Betree does this make sense? I'll create a PR and reference this issue so you can review.


⚠ Bofrost.nl has degraded performance

In 67e0747, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 1078 ms

Resolved: Bofrost.nl performance has improved in 67f77a0.


RHINENG-12334: filters for incidents

Added functionality for filters
I may need to fix things in the future when real incident data is present in the Prometheus database.
I added a JSdoc for the filter function, I found it much more useful than any typescript interfaces and other stuff, because it provides highlights and allows you to explain how the function works + provide an example

/lgtm

/ok-to-test


Create hamburger menu for nav options on mobile

The nav bar takes up too much space on mobile - I would like to design it so the nav bar options are put into a hamburger menu when on mobile and display a header inside the nav bar instead.

Hamburger menu is made and JS added to make it interactive. Still having issues with the flex properties in trying to make it drop down in a column vs across the bar as a row.


0.6.2 Tag branch

Is this tag branch up to date?
https://github.com/kermitt2/grobid/tree/0.6.2
I see that some of the files are old like
https://github.com/kermitt2/grobid/blob/0.6.2/CHANGELOG.md

Hi @HulioVRD !
I am updating the changelog, readme and the doc files after tagging and release, assuming that users look only at the current master version for those files - an error it seems :D

I see. If everything else is up to date, I think we could close it. Changelog and docs were the first thing that I looked at, but the code is most important for me.
Thank you for the fast response and have a nice day!


Streaming swaps may be borked

Overview
Note the may: this has been tested with the monkey patch below, which was previously proven to work to get a fake streaming rate, and was able to be executed e2e:
diff --git a/packages/swapper/src/swappers/ThorchainSwapper/utils/getL1Rate.ts b/packages/swapper/src/swappers/ThorchainSwapper/utils/getL1Rate.ts
index 54eddd355b..3a8dfd498d 100644
--- a/packages/swapper/src/swappers/ThorchainSwapper/utils/getL1Rate.ts
+++ b/packages/swapper/src/swappers/ThorchainSwapper/utils/getL1Rate.ts
@@ -87 +87 @@ export const getL1Rate = async (
-      sellAmountCryptoBaseUnit,
+      sellAmountCryptoBaseUnit: bnOrZero(sellAmountCryptoBaseUnit).times(1000000).toFixed(0),
@@ -103 +103 @@ export const getL1Rate = async (
-          sellAmountCryptoBaseUnit,
+          sellAmountCryptoBaseUnit: bnOrZero(sellAmountCryptoBaseUnit).times(1000000).toFixed(0),

References and additional details
See monkey patch above
Acceptance Criteria

Note, dev endpoints for THOR are currently borked so you'll need to use app env to test this
Confirm whether or not there is an actual issue or caused by the monkey-patch above, and fix if applicable

Need By Date
No response
Screenshots/Mockups
No response
Estimated effort
No response

False alert - confirmed streaming swaps are not borked: https://jam.dev/c/48bd3234-0f54-4ec3-8118-8808e905a98e
The correct monkey patch is below:
diff --git a/packages/swapper/src/swappers/ThorchainSwapper/utils/getL1Rate.ts b/packages/swapper/src/swappers/ThorchainSwapper/utils/getL1Rate.ts
index 54eddd355b..3a8dfd498d 100644
--- a/packages/swapper/src/swappers/ThorchainSwapper/utils/getL1Rate.ts
+++ b/packages/swapper/src/swappers/ThorchainSwapper/utils/getL1Rate.ts
@@ -84,7 +84,7 @@ export const getL1Rate = async (
     {
       sellAsset,
       buyAssetId: buyAsset.assetId,
-      sellAmountCryptoBaseUnit,
+      sellAmountCryptoBaseUnit: bnOrZero(sellAmountCryptoBaseUnit).times(1000000).toFixed(0),
       receiveAddress,
       streaming: false,
       affiliateBps: requestedAffiliateBps,
@@ -100,7 +100,7 @@ export const getL1Rate = async (
         {
           sellAsset,
           buyAssetId: buyAsset.assetId,
-          sellAmountCryptoBaseUnit,
+          sellAmountCryptoBaseUnit: bnOrZero(sellAmountCryptoBaseUnit).times(1000000).toFixed(0),
           receiveAddress,
           streaming: true,
           affiliateBps: requestedAffiliateBps,
diff --git a/packages/swapper/src/swappers/ThorchainSwapper/utils/getL1quote.ts b/packages/swapper/src/swappers/ThorchainSwapper/utils/getL1quote.ts
index 3b16832b3d..39f0226822 100644
--- a/packages/swapper/src/swappers/ThorchainSwapper/utils/getL1quote.ts
+++ b/packages/swapper/src/swappers/ThorchainSwapper/utils/getL1quote.ts
@@ -84,7 +84,7 @@ export const getL1Quote = async (
     {
       sellAsset,
       buyAssetId: buyAsset.assetId,
-      sellAmountCryptoBaseUnit,
+      sellAmountCryptoBaseUnit: bnOrZero(sellAmountCryptoBaseUnit).times(1000000).toFixed(0),
       receiveAddress,
       streaming: false,
       affiliateBps: requestedAffiliateBps,
@@ -100,7 +100,7 @@ export const getL1Quote = async (
         {
           sellAsset,
           buyAssetId: buyAsset.assetId,
-          sellAmountCryptoBaseUnit,
+          sellAmountCryptoBaseUnit: bnOrZero(sellAmountCryptoBaseUnit).times(1000000).toFixed(0),
           receiveAddress,
           streaming: true,
           affiliateBps: requestedAffiliateBps,


Make image registries configurable

Hey :wave: first of all, thanks for maintaining this project! It's a really nice addition
We'd like to be able to template in alternative registries for running in environments where pulling from Docker Hub isn't an option. So, this PR adds that

Hey, thank you for your feedback and this contribution. I will try it out and make a new release if everyhting. So ignore the failing chart linting check because of the missing version update.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1505

Relevant Lines:
1629

💛  - Coveralls

@stone-z your change is released with 1.9.4


Future and Deprecation warnings

There's at least two warnings being raised from datashader right now:
datashader/transfer_functions.py:21: FutureWarning: xarray subclass Image should explicitly define __slots__
  class Image(xr.DataArray):

datashader/bundling.py:325: FutureWarning: The 'get_values' method is deprecated and will be removed in a future version. Use '.values' or 'np.asarray(..)' instead.

Looks like those are new deprecations from xarray and pandas, respectively:
http://xarray.pydata.org/en/stable/whats-new.html#v0-13-0-17-sep-2019
https://pandas.pydata.org/pandas-docs/stable/reference/api/pandas.DataFrame.get_values.html
I'll see if I can make a PR to update our usage of those features.


Write signed registry txs to disk

I modeled this as closely as possible to the stake subcommand but I hacked it together quickly cause I found this was a pain point for joining the testnet. See the section here: https://github.com/oasislabs/docs/blob/9428add3df027ba1ab3d7a96013eed895a20fa8d/src/operators/joining-the-testnet.md#registering-you-entity

I guess? I'm not sure how useful this will be in practice, registry transactions have a definite shelf life due to the timestamp.

Ya, the timestamp was something I was wondering about. That being said, the issue is mostly around keeping the entity's private key offline. For simplicity's sake, I wanted to keep the instructions for joining the testnet restricted to using the node that the operator is deploying. So the options for registering an entity after the genesis file has been set:

Have the node operator do the registration on the deployed node (didn't want to suggest that)
Allow entity registration over the internet via something we provide or (like we do currently) ssh tunneling to the deployed node.
Allow for the signing of a registry request (this PR)

The better long term solution is probably a combination of:

Allow entity registration over the internet via something we provide
Delaying the node registration till the node is synced (should be done already), AND the registry contains the node's owning entity (with the node listed in the entity descriptor), while allowing the node to submit txes.

But I guess this is ok for now.

Before I approve this, should using a saved transaction file (the 2 step process) be mandatory like how it is in the staking client?

Before I approve this, should using a saved transaction file (the 2 step process) be mandatory like how it is in the staking client?

Hrm. I personally don't have an opinion on the matter. I would only say it might make sense to make it mandatory for consistency's sake.

I would be for mandatory.

I would be for mandatory.

Sounds good

Don't forget to update registerCmd.Short and deregisterCmd.Short to reflect the new behavior (See how the staking app describes the operations).

@kostko I tested the changes on the initial commit but I haven't tested it recently. I'll do that and report the results here.

@kostko tested. Works!


feat(Backend) : Added Email Service

Issue that this pull request solves
closes #152
Brief description of what is fixed or changed
Added an email helper function that takes email id and a template type to send emails
Types of changes
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update (Documentation content changed)
[ ] Other (please describe):

Checklist
Put an x in the boxes that apply

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] My changes does not break the current system and it passes all the current test cases.

@priyanshu20 Are you taking this us this? We need to close this by weekend. Let me know if you cannot take it up, I will complete it and merge

@priyanshu20 Are you taking this us this? We need to close this by weekend. Let me know if you cannot take it up, I will complete it and merge

I cannot do this right now, sorry for inconvenience

@priyanshu20 Are you taking this us this? We need to close this by weekend. Let me know if you cannot take it up, I will complete it and merge

I cannot do this right now, sorry for inconvenience

No issues, you have done most of the part. I can take this from here

Hey @jackfrost13 ,
I'm free and can finish it if you have not yet found the chance to look into it.


feat(quick-start): improved guide

Based on issue #2669

@TkDodo @tannerlinsley   Is everything all right... are there any other issues/contributions I can take?

Thanks for the contribution. I think we have to work on the wording a little bit, but I haven't gotten around to it yet, sorry

@TkDodo cool... could you let me know where it feels off and I'll rewrite that

@TkDodo I've made the necessary changes, let me know if it's alright. If everything's good, could you mark this PR as hacktoberfest-accepted?

sorry, I don't think this is making the guide any better than it currently is. I'll try to make the docs code blocks better, maybe with line highlighting and then we can revisit this.


fix(first): force unsubscription logic on complete and error

Description
Force unsubscription logic when operator completes or errors, so that source Observable is only subscribed as long as it has to, even when combined with operators that do not immediately unsubscribe
from 'first' when it completes or errors.
Introduces test-only doNotUnsubscribe operator, which simulates operators that are unreliable with managing subscriptions (like switchMap for example - see #2459).
Related issue (if exists):
Closes #2455
This issue is a concrete example of more general one: #2459

Coverage increased (+0.001%) to 97.689% when pulling a55f8dac1a876093c459265f11514f1c5b52a87d on Podlas29:first-with-switch-map-fix into 69d051bc56cb337148db30515029363c1e78b90d on ReactiveX:master.

While this is a fix, it also might be a breaking change for some. @staltz, @kwonoj what do you think? Next major version?

Yeah it's major. If it could possibly break existing behavior for anyone out there, it's a breaking change. There's no shame in bumping the major num. In the changelog we can say "this will probably not affect you, but" type of message.


Floating button follows active tab space

Pretty nice feature but the button doesn't stay in the place you put it on each tab space. It just goes back to the default which may be inadapted for certain sites
(archlinux)

This is a know issue, normally Faymoon is working on it. I will ask him when it will be ready.


Interactive plot with selection does not render

I attempted the tutorial on creating a plot with selection, but was met with an infinitely spinning spinner.
Here is a Colab notebook that demonstrates the issue. I am using datamapplot==0.4.2:
https://colab.research.google.com/drive/14Taj65FefFboB-7Ipi-RAZ1ADVeZWvpc?usp=sharing
I encountered the same issue in a local marimo notebook. I did not try in a local Jupyter notebook.

There is some special code that hacks the html so that things work within Jupyter. I believe colab has slightly different paths/properties. I'm sure Marimo doesn't have Jupyter's convolutions. We may need to have similar contortions specific to colab, and then other handling for Marimo. Let me see if I can work out colab. For your reference the handling is here, and amounts to working around the fact that you can't actually fetch files via jupyters web address because jupyter intercepts the requests, so it rewrites what works for ordinarily served web-pages to work within jupyter, as well as trying to handle the fact that we are buried in a srcdoc, so some other wrangling needs to be done to get the server address.

So inline_data=False is simply not going to work with colab unfortunately, since it doesn't save files in a way that you can fetch them without integrati8ng with google drive (which I can understand people not wanting to do). What I can hopefully do is detect if it is being run in google colab and providing a warning that things won't work as intended.

Thanks for the source pointer. I can see if I can get it working in marimo


chore: added favicon

🛠 Closes #166

👨💻 Changes proposed

Added favicon png in the assets folder
Modified index.html

✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers

📷 Screenshots

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


Polymorphic top-level Hugrs

This seems a good thing to have, and could potentially simplify many tests (which pull in some extension just so we have a concrete leaf type - they could just use a type variable instead).
We could store the binders in the Hugr, but then we should be careful that Hugrs rooted at FuncDefn/FuncDecl don't have any binders themselves.
The simplest/least option would just be to pass the decls ([TypeParam]) into the top-level Hugr::validate() (and friends update_and_validate etc.....).
And, follow-up (if we have not by then decided to go back to having only FunctionType in the type hierarchy and only monomorphic lambdas) - allow storing the decls alongside the Hugr in a Const too.

So, probably this is some kind of a wrapper around (a Hugr and some TypeParams), that one can validate (-> calls Hugr::validate passing in those TypeParams) and other things.

Should include an instantiate method that (given TypeArgs correct for all its TypeParams) produces a non-polymorphic Hugr.  (Partial instantiation probably not required)
Given #904, unclear whether you can have a "polymorphic top level hugr" (with some TypeParams) whose root node is a FuncDefn (binding more TypeParams). If so, then we retain DeBruijn even after #904; if not, then DeBruijn disappears with #904 (as nested FuncDefns can't refer to their enclosing binders, so we never have more than one set of TypeParams in scope)

This'll also give us a way to do inference separately for subgraphs and thus a better solution to inserting a subgraph than that of test_validate_with_closure from #884

It is just possible that maybe we could treat FuncDefn as the only binder with TypeParams. We would still need a method to apply a substitution across the Hugr (i.e. instantiate a FuncDefn-rooted Hugr to a monomorphic, e.g. DFG-rooted, Hugr)


Add stable_lm script

I'm closing this PR since it contained a merge commit and was created from a branch in the main repo, not a forked repo. I have opened a fresh PR. https://github.com/nod-ai/SHARK/pull/1331


Timeless and classics zero Incompatibility

When a TAC zero weapon is held, the players arms hold the gun pointing down

When a TAC zero weapon is held, the players arms should hold the gun pointing forward (like in the unmodded TAC animation).

Serious player animations: 1.1.1 - TAC zero: 1.0.0-hotfix4-beta
-1.20.1
-Forge

Maybe it would be possible to add custom animations for TACZ too since default ones don't look great

On Discord I have uploaded a version that solves this problem

On Discord I have uploaded a version that solves this problem

Which server? Version of what?
You can do PR to this repo if you know how to fix it 🤷 (or TAC:Z repo if issue is on their side)

On the discord server for this mod. https://discord.com/invite/gQuypb5n

PR created, also added support for Vic's Point Blank mod
https://github.com/RazorPlay01/Serious-Player-Animations


JSON API Spec: 1.1 - Profiles

Are Profiles going to be supported by this package? They are being released in JSON API Spec 1.1.

Hi! Yes, the aim of this package is to support the spec as a whole. So when profiles get added we will aim to support them at some point. The only thing I'm not sure about is when that will be, as we get sudden bursts of activity on this package and there's already a to-do list! We are making progress though so hopefully profiles will be supported sometime soon.
We'd need to upgrade the underlying neomerx/json-api package, which would be a breaking change. So profiles will probably land in 2.0 of this package.

@lindyhopchris thanks for your work on this package!
I would be happy to help develop on the package as I get time. We have been using this package for every API we write since I found it. Super handy to not have to do all the repetitive endpoint logic.
Scott

@plunkettscott glad to hear you're using the package on multiple APIs and it's useful. Sorry for the code churn but it's definitely settling down and we should hit 1.0 once we merge #254
Any help you could give in the future would be great - just let me know if you get any time.

Going to close this as JSON:API still haven't officially released profiles. Definitely something to support when they do but for the moment I don't see the point in leaving the issue open after 2 years!


TPCPortal Initial Merge for Certification

Please check the following conditions for your PR.

[ ] apiDefinition.swagger.json is validated using paconn validate command.
[ ] apiProperties.json has a valid brand color. Invalid brand colors are #007ee5 and #ffffff.

@jdusome Could you please sign the CLA that the microsoft-cla bot linked to above? Thanks!


Add API change to CHANGELOG using cargo public-api diff

This will be possible after https://github.com/wcampbell0x2a/backhand/pull/294 is merged (syn 1.0 -> 2.0)


Shadow database 'resolve error' list update

Context
https://prisma-company.slack.com/archives/C5Z9TH6N9/p1621929601052600

validate the info here:

expand list of DB providers with Supabase and Digital Ocean

(Supabase does not require manual handling of shadow databases any more, it should just work)

The information on the page is up to date. I've created a PR to make a few minor adjust wording and expand the example hosted DB providers


Git date experiment 8, new git_commit_hash reader and use /tmp/ as cwd for retrieveSQLTables

…

👍


HorizontalAlignment missing a stretch value

Right now there's only left, center, and right. How would an image that stretches to the width be displayed?

Is this still valid? The image seems to support stretch and auto now:
http://adaptivecards.io/explorer/#Image

Stretch is specified as the size, not the horizontal alignment.


give different access to a user who can access different websites

we develop a CMS based on yii2, on this CMS superuser can add unlimited website, therefor we have a websiteId attribute in all tables.
each user can access to many websites, it means that we have a many to many relation between websites and users.
we use yii2 authorization to give access for each module and it's actions to users, now we have a problem. we want to give different access to actions on each website.
for example, Bob has access to Module1, Module2 on website1 and Module2, Module3 on website2.
he can only create and update data on website1.Module2 and can delete, read on website2.Module2
and he can delete and create on website1.Module1 and have access to delete, read, create, update on website2.Module3
These access can be different for another user names Alice.
I tried to add a column to auth_assignment table, but it seems that I cant add attribute to this table, it just returns userId, roleName , createdAt
is there any other way to do this?
What steps will reproduce the problem?
DbManager->getAssignment()
What is the expected result?
[ 'userId' ,
'roleName',
'createdAt' ,
'websiteId' ,
]
What do you get instead?
[ 'userId' ,
'roleName',
'createdAt' ,
]
Additional info

Q
A

Yii version
2.0.15

PHP version
5.6.33

Operating system
ubuntu 16.04

I think it need to be a feature, because if I wanted to add attribute to auth_assignment table, I have to add a property to rbac/Assignment class, then overwrite almost all methods in rbac/DbManager
I think it should become more dynamic.


add a note about private classes

I have mixed feelings about making this a blanket rule. It should generally be assumed that init.pp is the entry point and all other classes are private.

@juniorsysadmin I updated it


BlockParametersNode flag for ambiguous_param0

Closes #1538

Thanks for your work on this @haldun but we're going to not go in this direction anymore.


Cycle bookmarks of the CURRENT file only

Labeled Bookmarks is an excellent VSC extension. However, I am used from the also very solid Bookmarks extension to being able to cycle through only the current file's bookmarks, because they have the two settings:
Bookmarks: List List all bookmarks in the current file
Bookmarks: List from All Files List all bookmarks from all files
This is nice because it doesn't suddenly carry you away from, let's say a C++ project to some AHK script you might be doing on the side (and bookmarking there, too).
To maybe even top the Bookmarks competition, may I suggest a third option...
Bookmarks:  List files from current project  List all bookmarks of  included /  user-selected files (only)
?
Can you please add those two + one extra feature to your extension? Then you'd be the best without any discussion! :-)

I'm second to this!
This is a GREAT extension, but I was forced to disable it and still use "Bookmarks" - exactly because of it is jumping between the files when I'm trying to concentrate on just one file bookmarks...
Please please fix this.


Add state from event tuple handler

Resolves L2B-3547
This PR is here so it's easy to pick up when we kick off the LZv2 CR.

Works just fine:


[New game]:  🎮 Chess Game with compute Request

🎮 Game Request
Description:
We propose the development of an AI-powered chess game that offers an engaging and challenging experience to players of all levels. The game will leverage advanced algorithms to provide intelligent gameplay while also offering a user-friendly interface for an immersive gaming experience.
Game Logic:
The AI chess game will follow standard chess rules, allowing players to enjoy the classic game in a digital format. The game will include features such as move validation, checkmate detection, and various difficulty levels to accommodate players from beginners to advanced.
Point down the features
Intelligent AI Opponent: The game will feature an AI opponent with varying levels of difficulty, from beginner to grandmaster, to provide a challenging experience for players of all skill levels.
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @thevijayshankersharma !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

/assign

/assign

I have already raised an PR

Hey @thevijayshankersharma ! , This issue is already assigned to @dnyandeepchute. 💗
You can work on other issues 🚀
If there is anything wrong you can contact us on Discord🕹👀

@ishita-43 someone else got assigned.

The issues are assigned on first come first serve basis. if the first person will not be able to complete pr another will get the issue. And also always make sure if you are intended to work on created issue then assign it for yourself before other try to do that!!

The issues are assigned on first come first serve basis. if the first person will not be able to complete pr another will get the issue. And also always make sure if you are intended to work on created issue then assign it for yourself before other try to do that!!

Thank you. Could you check my PR related to this.

Hey @thevijayshankersharma ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


[ENH] Allow fractional seconds in StartTime

closes #1447

Thanks for clarifying this, that will be helpful in the future!


Javascript/Typescript not compiling / Stuck at pending

I was excited to start using this but I can't get anything javascript/typescript to compile.
No errors pops up, it's just stuck at Pending.  I have tried installing the Node REPL extension in vscode but that didn't help.
I have typescript installed globally

Typescript version: 4.4.4
Node version: 12.20.0
npm version: 6.14.8

If I switch the compiler to HTML everything is rendered.

I've managed to repro this, will be releasing a new version ASAP.

I"ve uploaded a new version, please let me know if that fixes it.
If not, please do feel free to re-open this issue or create a new one.

Hi, I have the same issue after installing for the first time.

node -v: v16.6.1
npm -v: 8.1.3


[GHSA-jhv5-cxf2-r67v] UVDesk Community Skeleton v1.1.1 allows unauthenticated...

Updates

Affected products
Severity
Source code location
Summary

Comments
I submitted this vulnerability with my organisation eSecForte Technologies, so I know what package and version were affected. If a detailed Video Poc is required I'll share that as well.

Hi @sahildari, similar to my colleague @JonathanLEvans's comment in https://github.com/github/advisory-database/pull/4208, we would like to see documentation (such as a link to the fix commit) that demonstrates that CVE-2023-37635 is fixed in version 1.1.3.

👋 This pull request has been marked as stale because it has been open with no activity.  If you do nothing this pull request will be closed eventually by the stale bot. Please see CONTRIBUTING.md for more policy details.


Inquiry for hlca_map_classify.ipynb

Hi,
Thank you for the great work.
I have encountered an issue in the hlca_map_classify.ipynb.
I cannot proceed due to an error in the "Load reference model and set relevant query variables" section as follows.
surgery_model = sca.models.SCANVI.load_query_data(
adata_query,
ref_model_dir,
freeze_dropout=True,
)

KeyError                                  Traceback (most recent call last)
Cell In [39], line 1
----> 1 surgery_model = sca.models.SCANVI.load_query_data(
2     adata_query,
3     ref_model_dir,
4     freeze_dropout=True,
5 )
File /opt/conda/lib/python3.10/site-packages/scvi/model/base/_archesmixin.py:122, in ArchesMixin.load_query_data(cls, adata, reference_model, inplace_subset_query_vars, accelerator, device, unfrozen, freeze_dropout, freeze_expression, freeze_decoder_first_layer, freeze_batchnorm_encoder, freeze_batchnorm_decoder, freeze_classifier)
116 if _SETUP_ARGS_KEY not in registry:
117     raise ValueError(
118         "Saved model does not contain original setup inputs. "
119         "Cannot load the original setup."
120     )
--> 122 setup_method = getattr(cls, registry[_SETUP_METHOD_NAME])
123 setup_method(
124     adata,
125     source_registry=registry,
(...)
128     **registry[_SETUP_ARGS_KEY],
129 )
131 model = _initialize_model(cls, adata, attr_dict)
KeyError: 'setup_method_name'
I would appreciate it if you could advice me on how to trouble shoot it. Thank you for your help.
Best regards,
TN

Hi, there was a similar problem recently https://github.com/theislab/scarches/issues/252

Thanks!


goto component

Basically goto("nodeid") short syntax for link(function () return "nodeid" end)

Note: if you type checked the first argument for link you could do link("nodeid") which would be even better than goto("nodeid")


Add an option to configure bitcannon binding IP

This PR makes it possible to configure bitcannon's web server binding address.
I think it can be useful to restrict the binding address of the server for several reasons, starting with security.
To avoid BC breaks, I set it to <IP_ADDRESS> in the default config and fallback in the code. But I think its default value should be <IP_ADDRESS>.

I like it! Thanks for the contribution.


Feature/offline mode

Move Skopeo for copying to repo-host by Ansible and add message for long running repo playbook.

Move Skopeo for copying to repo host and add message for long running repo task


Support for cargo

Fixes #1367
Changes proposed:

[x] Add
[ ] Prepare
[ ] Delete
[ ] Fix

Things I've done:

[x] My pull request fixes an issue, I referenced the issue.

I don't know where you got it but it's good. Saved me the trouble of creating one.

I've created it from scratch using Illustrator. I'm not so Rusty as I thought! I'm glad you like it!

I'm impressed. 😮

The pity is that when viewed in vscode it loses a lot of detail and it's not really as good as it may seem at first sight... but at least is recognizable. 😉

I knew this would be the case from the start and thought of proposing to use the crate icon but it doesn't represent Cargo, so I hashed.

Yep, that's the problem with complex icons. I think that the result is not optimum in this case also but I guess that is fairly acceptable given the circumstances.

You know that those icons shine on a 4K 32" monitor, right?

😂 I don't happen to have a 4K 32" monitor... for the moment! 😂 I guess you are one of the lucky ones?

You kidding me?!? It's not Black Friday yet.


chore: add Chuks

Add Chuks
##What does this PR do?
Add my profile

Change the PR title to - chore: add Chuks

@syedsohan Please can you leave this to the maintainers to change, it's fine for new contributors to choose the title themselves :)

Change the PR title to - chore: add Chuks

@syedsohan Please can you leave this to the maintainers to change, it's fine for new contributors to choose the title themselves :)

Ok. 🙂

I dont think @ChuksMbanaso is too active on github?

@ChuksMbanaso ill close this pull request for now. Feel free to open it again if you wish to continue working on it.


Better A/B Slots Support

This PR introduses a "proper" support for Android A/B slots system:

New Driver BootSlotDxe is a replacement for unstable Op6tSlotDxe and properly updates active slot bits (recovers all unbootable slots just in case if any and marks current active slot as successful to fool abl);
New EDK Boot Manager's "Reboot to other slot" entry that switches the current active slot. Useful to boot Android without ever using fastboot and PC (and Android is able to switch the slot out of the box too so we can achieve a "dualboot" now, kinda).

Unlike Op6tSlotDxe, edk2 with this driver can be flashed to either A or B slot without any problem.
The core implementation is based on qcom's abl but adapted to work on edk2 environment and it should be generic for all qcom devices with A/B system.
Enabled for fajita and enchilada.

hmm, will fix build asap