Trying to do introspection raise an internal error

Summary
For a specific workflow, I would like to retrieve the current workflow object, then parse it with python for triggering some action. Below you can find the most simple and basic workflow to reproduce the issue.
The message from argo-workflow engine is:
invalid character 'a' after object key:value pair

So it seems that it tries to parse/interpret the value of the variable output and it shouldn't.
Diagnostics
What Kubernetes provider are you using?
Client Version: 4.7.2
Server Version: 4.5.5
Kubernetes Version: v1.18.3+08c38ef

What version of Argo Workflows are you running?
3.1.2

What executor are you running? Docker/K8SAPI/Kubelet/PNS/Emissary
PNS

Did this work in a previous version? I.e. is it a regression?
I tried first with 3.0.1 but it was also not working.
Workflow to reproduce
apiVersion: argoproj.io/v1alpha1
kind: WorkflowTemplate
metadata:
  name: serialization-resource-bug
  namespace: YOUR-NAMESPACE
spec:
  entrypoint: main
  serviceAccountName: YOUR-SA
  templates:
  - name: main
    steps:
      - - name: get-workflow-object
          template: get-workflow-object
      - - name: create-artifact
          template: create-artifact
          arguments:
            parameters:
            - name: workflow-object
              value: "{{steps.get-workflow-object.outputs.parameters.workflow-object}}"
  - name: get-workflow-object
    outputs:
      parameters:
      - name: workflow-object
        valueFrom:
          jsonPath: '{}'
    resource:
      action: get
      manifest: |
        apiVersion: argoproj.io/v1alpha1
        kind: Workflow
        metadata:
          name: {{workflow.name}}
          namespace: YOUR-NAMESPACE
  - name: create-artifact
    inputs:
      parameters:
      - name: workflow-object
    outputs:
      artifacts:
      - name: result
        path: /mnt/out/result
    volumes:
    - name: out
      emptyDir: {}
    script:
      volumeMounts:
      - name: out
        mountPath: /mnt/out
      image: bash
      command: [bash]
      source: |
        echo will do
        cat <<EOF > /mnt/out/result
        {{inputs.parameters.workflow-object}}
        EOF
        echo done

Message from the maintainers:
Impacted by this bug? Give it a 👍. We prioritise the issues with the most 👍.

@sylock Workflow JSON is causing the JSON parsing when it substitutes in the source.  I would recommend storing the workflow JSON as an artifact and pass it across the steps.  code has to handle JSON format in the source. we will investigate and fix it.
"echo will do\ncat \u003c\u003cEOF \u003e /mnt/out/result\n{\"apiVersion\":\"argoproj.io/v1alpha1\",\"kind\":\"Workflow\",\"metadata\":{\"creationTimestamp\":\"2021-07-27T15:54:08Z\",\"generation\":2,\"labels\":{\"workflows.argoproj.io/phase\":\"Running\"},\"managedFields\":[{\"apiVersion\":\"argoproj.io/v1alpha1\",\"fieldsType\":\"FieldsV1\",\"fieldsV1\":{\"f:metadata\":{\"f:labels\":{\".\":{},\"f:workflows.argoproj.io/phase\":{}}},\"f:spec\":{\"f:activeDeadlineSeconds\":{},\"f:podSpecPatch\":{},\"f:templates\":{},\"f:ttlStrategy\":{}},\"f:status\":{\"f:artifactRepositoryRef\":{},\"f:nodes\":{},\"f:phase\":{},\"f:progress\":{},\"f:startedAt\":{}}},\"manager\":\"___controller\",\"operation\":\"Update\",\"time\":\"2021-07-27T15:54:08Z\"},{\"apiVersion\":\"argoproj.io/v1alpha1\",\"fieldsType\":\"FieldsV1\",\"fieldsV1\":{\"f:spec\":{\".\":{},\"f:arguments\":{},\"f:entrypoint\":{}},\"f:status\":{\".\":{},\"f:finishedAt\":{}}},\"manager\":\"argo\",\"operation\":\"Update\",\"time\":\"2021-07-27T15:54:08Z\"}],\"name\":\"serialization-resource-bug\",\"namespace\":\"argo\",\"resourceVersion\":\"371450\",\"selfLink\":\"/apis/argoproj.io/v1alpha1/namespaces/argo/workflows/serialization-resource-bug\",\"uid\":\"6c5cfd3a-bb62-404b-b640-04367c517b8d\"},\"spec\":{\"activeDeadlineSeconds\":300,\"arguments\":{},\"entrypoint\":\"main\",\"podSpecPatch\":\"terminationGracePeriodSeconds: 3\\n\",\"templates\":[{\"inputs\":{},\"metadata\":{},\"name\":\"main\",\"outputs\":{},\"steps\":[[{\"arguments\":{},\"name\":\"get-workflow-object\",\"template\":\"get-workflow-object\"}],[{\"arguments\":{\"parameters\":[{\"name\":\"workflow-object\",\"value\":\"{{steps.get-workflow-object.outputs.parameters.workflow-object}}\"}]},\"name\":\"create-artifact\",\"template\":\"create-artifact\"}]]},{\"inputs\":{},\"metadata\":{},\"name\":\"get-workflow-object\",\"outputs\":{\"parameters\":[{\"name\":\"workflow-object\",\"valueFrom\":{\"jsonPath\":\"{}\"}}]},\"resource\":{\"action\":\"get\",\"manifest\":\"apiVersion: argoproj.io/v1alpha1\\nkind: Workflow\\nmetadata:\\n  name: serialization-resource-bug\\n  namespace: argo\\n\"}},{\"inputs\":{\"parameters\":[{\"name\":\"workflow-object\"}]},\"metadata\":{},\"name\":\"create-artifact\",\"outputs\":{\"artifacts\":[{\"name\":\"result\",\"path\":\"/mnt/out/result\"}]},\"script\":{\"command\":[\"bash\"],\"image\":\"bash\",\"name\":\"\",\"resources\":{},\"source\":\"echo will do\\ncat \\u003c\\u003cEOF \\u003e /mnt/out/result\\n{"
		apiVersion ":"
		argoproj.io / v1alpha1 ","
		kind ":"
		Workflow ","

I tried saving the Json as an artifact but I could not do it. I don't think that the "resource" template support artifacts (Did not see it in the documentation and my tries failed).
  - name: get-workflow-object
    outputs:
      parameters:
      - name: workflow-object
        valueFrom:
          jsonPath: '{}'
    resource:
      action: get
      manifest: |
        apiVersion: argoproj.io/v1alpha1
        kind: Workflow
        metadata:
          name: {{workflow.name}}
          namespace: YOUR-NAMESPACE

Please try with v3.1.3

Unfortunately I still have the issue with the version 3.1.3:

Can you explain me the workaround mentionned by @sarabala1979 with the artifacts? Indeed it would be more elegant and more easy to handle if I could save the JSON object as an artifact than a variable. When I try this it doesn't work since the artifacts output requires a "path" property:
apiVersion: argoproj.io/v1alpha1
kind: WorkflowTemplate
metadata:
  name: serialization-resource-bug-3
  namespace: NAMESPACE
spec:
  entrypoint: main
  serviceAccountName: SA
  templates:
  - name: main
    steps:
      - - name: get-workflow-object
          template: get-workflow-object
  - name: get-workflow-object
    outputs:
      artifacts:
      - name: workflow-object
        valueFrom:
          jsonPath: '{}'
    resource:
      action: get
      manifest: |
        apiVersion: argoproj.io/v1alpha1
        kind: Workflow
        metadata:
          name: {{workflow.name}}
          namespace: NAMESPACE

@sarabala1979 I leave this with you to add to backlog.

@sarabala1979 Can I have an update on that issue?

@sarabala1979 can you please reply?

@sylock I will take a look. I am trying to provide fix or workaround

@sarabala1979 @alexec I think that there are 2 different things in this ticket:
A bug in the serialization
There is probably a bug in the escaping of the data when providing them to a step as an input.
A feature request - export
For a step of kind "get resource" with a manifest, add the possibility to export the output as an artifact instead of a variable.
I think we should focus on the bug in this ticket, and I can create another one for the feature request. Can you tell me what you think and validate what I can do to make it more clear?

@sylock yes. Please create enhancements issues for support artifacts for resource template type.

@sylock I spend a whole day fixing the serialize the JSON but I couldn’t get it right. Because template has Json inside json (wf object JSON) which replaced by parameters.  I kind of thinking instead of fix this edge case. I come with workaround. In meantime , we will implement artifact support for resource template.
I build custom docker image with kubectl sarabala1979/kubectl:latest
apiVersion: argoproj.io/v1alpha1
kind: Workflow
metadata:
  name: serialization-resource-bug
spec:
  entrypoint: main
  templates:
    - name: main
      steps:
        - - name: get-workflow-object
            template: get-workflow-object
        - - name: create-artifact
            template: create-artifact
            arguments:
              artifacts:
                - name: workflow-object
                  from: "{{steps.get-workflow-object.outputs.artifacts.workflow-object}}"
    - name: get-workflow-object
      outputs:
        artifacts:
          - name: workflow-object
            path: /tmp/wf.yaml
            archive:
              none: {}
      script:
        image: sarabala1979/kubectl:latest
        command: [sh]
        source: |
          kubectl get wf {{workflow.name}} -o yaml > /tmp/wf.yaml

    - name: create-artifact
      inputs:
        artifacts:
          - name: workflow-object
            path: /tmp/wf.yaml
      outputs:
        artifacts:
          - name: result
            path: /mnt/out/result
      volumes:
        - name: out
          emptyDir: {}
      script:
        image: bash
        command: [bash]
        source: |
          cat /tmp/wf.yaml

That said, I would not call this scenario an edge case. It's introspection and I think it is a nice feature that would be nice to support. I have an idea without knowing its feasibility: in the serialization and deserialization, maybe passing all values encoded in BASE64 would fix the issue. And I think that's why it's also used often on the web. To prevent this kind of quirky escaping.


Improve lockingStrategy "auto": add HTTP method to url identification

From martin.grotzke on June 09, 2012 21:19:34
The lockingStrategy "auto" currently uses the requestURI (and queryString for GETs) to track readonly/modifying requests. This should be changed to also take the request method into account.
Original issue: http://code.google.com/p/memcached-session-manager/issues/detail?id=140

From martin.grotzke on June 09, 2012 14:40:28
Done.
Status: Fixed

From martin.grotzke on July 03, 2012 01:14:01
(No comment was entered for this change.)
Labels: Milestone-1.6.2


Script occasionally fails while scrolling to the bottom of support select.

Has happened to me only three times over many hours of running the script, I've included screenshots of the full error and of the screen two of the times it has failed. Perhaps it's due to finding the servant it's looking for but halfway between the two spots it's looking for it?

Are you using the latest code base? I mean code base, the one you get when you click "Clone or download" in the main page, not the one in the release tab. I'm asking because I'm sure I already fixed this in #106.

That's probably it, I am using the latest release. Sorry about that.


Problem in starting backend

While using docker, I'm facing a problem regarding the backend.
During composing, it says "exec /usr/src/app/entrypoints/integration.sh: no such file or directory".
And while running migrate command, it says, "Container *** is restarting, wait until the container is running".

If you could post the exact steps you run to reproduce this error, it would be very helpful, and I'll take a look at the issue!

I'm unfortunately unable to reproduce the error locally, could you try to run
# in indok-web/
docker compose -f docker-compose.yml build
docker compose -f docker-compose.yml up

and see if the error persists?

I tried this but the error still exists.

The integration.sh entrypoint should only be referenced in docker-compose.integration.yml, which should not be used for local development, so I'm just trying to nail down where things are going wrong here 😊
Could you try running
docker system prune

NOTE: this will remove all unused containers, networks, images (both dangling and unreferenced), so if you have other unused docker images you wish to preserve, then consider following this guide instead to remove the containers and images associated with this project specifically.
and re-running the setup steps in the README, but running
# in indok-web/
docker compose -f docker-compose.yml build
docker compose -f docker-compose.yml up

instead of just
# in indok-web/
docker compose build
docker compose up

Thanks for the details, however, the same error continues. FYI: I'm using windows 11 and local dev.


Added JSONFeed

Project URL
https://github.com/totocaster/JSONFeed
Description
Swift parser for JSON Feed — a new format similar to RSS and Atom but in JSON. (Spec: https://jsonfeed.org )
Why it should be included to awesome-ios (optional)
JSON Feed was announced few hours ago, and already has Swift parser. I think this is awesome. 😎
Checklist

[x] Only one project/change is in this pull request
[x] Addition in chronological order (bottom of category)
[x] Supports iOS 9 / tvOS 10 or later
[x] Supports Swift 3
[x] Has a commit from less than 2 years ago
[x] Has a clear README in English

          1 Error

:no_entry_sign:
Found 2 link issues

Link issues by awesome_bot

Line
Status
Link

964
503
https://batch.com

2092
301
https://github.com/jerkoch/SwipeCellKit redirects tohttps://github.com/SwipeCellKit/SwipeCellKit

  Generated by :no_entry_sign: Danger

Thank you! ✨


Error ListView within CarouselSampleEmbeddedView

Hi again, I found a problem in CarouselSampleEmbeddedView.xaml, if you insert a ListView in a layout of a TabItem.Content (I tried with a StackLayout), you change of TabItem in the execution and then return to the TabItem that has the ListView giving a tab on the header text, the ItemTemplate's ListView disappears.  When do you drag in the TabItem.Content, doesn't happen.

Hi
It's difficult for understanding
Could you please fork this repo, modify the sample, then I will debug this case

That's right, I pushed a commit in the fork to reproduce the error found in CarouselSampleEmbeddedView.xaml. I also include a reference image: https://drive.google.com/open?id=1szRW4xD1laGSOXe2jELtq_T-6eInfhXS
Note: In android ocurrs the same.

very well)
If somebody wants to investigate this issue, i will be grateful =)
Or i will look through it on weekend

Excellent @AndreiMisiukevich now it works correctly!


Response decoding failed. Returned data corrupted or this library outdated.

Hello,
I'm experiencing this error ONLY on the production server.
I believe Instagram is blacklisting non-residential IPs requests.
I have tried to deploy the script on different servers with no luck.
This is where the exception is thrown:
InstagramScraper\Instagram::getMediasByTag
vendor/raiym/instagram-php-scraper/src/InstagramScraper/Instagram.php:938

Is anyone else experiencing the same issue?

#697 check this one.


Unsigned right shift >>> is not implemented yet

I am very confused by what is going on with int.>>>.

It is a member of the int class.
However, there is a comment in the docs that says "NOT IMPLEMENTED YET".
And the analyzer does not recognize the operator, even though it is part of the class?

If I want to use bitwise unsigned right shift on integers, how do I do that?

Okay, I just discovered this was enabled in Dart 2.14 - what a coincidence 🙌
Pre-2.14
Here is a function that does the same:
/// Bitwise unsigned right shift of [x] by [s] bits.
///
/// This function works as `>>>` does starting with Dart 2.14.
int urs(int x, int s) => (x & 0xffffffffffffffff) >> 2;

Dart 2.14
You can simply use x >>> s 🙌
https://github.com/dart-lang/language/issues/478#issuecomment-889181121


runtime: Remove Docker,Inc copyrights

There is no Docker copyrighted code in our runtime, it was just a
copy/paste leftover.
Signed-off-by: Samuel Ortiz<EMAIL_ADDRESS>

Coverage remained the same at 37.192% when pulling a72286c9180688169b40db8c2c2f70eda81632a1 on sameo/topic/copyright into a43816125772d36c57ba11afc97c0dc35bb46e5d on master.

@sboeuf Could you please clarify which runc files you did initially copy into our runtime ?

lgtm

@sameo I didn't copy those files, @dlespiau did it. I assume he copied/pasted the files create.go, start.go, .... from the runc codebase.
LGTM

I think we can drop this one, right ?

The only "code" we have from runc is the command line help. Considering that's basically the interface we have to implement to be an OCI runtime, maybe we can be more precise and state that the command line help has been taken from runc. The rest of the code is written from scratch.


Magma server api URL outdated

Describe the problem
The api url for magma was updated and you can't download the server. I tried changing it myself but it didn't seem to work and I don't want to go any deeper.
Container definition
version: "3"

services:
  censored:
    image: itzg/minecraft-server:java17
    ports:
      - 25565:21055
    volumes:
      - /home/bacon/.mc/censored:/data
    environment:
      PATH: "/opt/java/openjdk/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin"
      LANG: "en_US.UTF-8"
      LANGUAGE: "en_US:en"
      LC_ALL: "en_US.UTF-8"
      JAVA_VERSION: "jdk-17.0.3+7"
      JAVA_HOME: "/opt/java/openjdk"
      TYPE: "CUSTOM"
      VERSION: "1.16.5"
      CUSTOM_SERVER: "server.jar"
      EULA: "TRUE"
      UID: "1000"
      GID: "1000"
      RCON_PASSWORD: "minecraft"
    tty: true
    stdin_open: true
    restart: unless-stopped

Container logs
censored | [init] Running as uid=1000 gid=1000 with /data as 'drwxrwxr-x 1 1000 984 446 May 12 22:23 /data'
censored  | [init] Resolved version given 1.16.5 into 1.16.5 and major version 1.16
censored  | [init] Resolving type given MAGMA
censored | [init] **********************************************************************
censored  | [init] WARNING: Some mods and modpacks may require Java 8.
censored | [init]          Please use itzg/minecraft-server:java8
censored  | [init] **********************************************************************
censored  | curl: (22) The requested URL returned error: 404
censored  | [init] ERROR failed to locate latest Magma info for 1.16.5 in channel stable (error=0)

I've been working on a fix and their API is completely broken, even new version (https://api.magmafoundation.org/api-docs/#/). For now I think this issue has to wait until they fix their stuff.


Gnome-shell 41 support

I am running Arch Linux with gnome-shell 1:41.4-1. I cannot install the plugin due to incompatibility with my shell version. Is there any plan to support up-to-date gnome-shell versions?

i have identical problem can you try repair pls

What worked for me is to temporarily disable the gnome shell extension version validation (dconf key /org/gnome/shell/disable-extension-version-validation). However, this is only be a workaround.

can you explain procedure ?

$ dconf write /org/gnome/shell/disable-extension-version-validation true

ok but its disable update for all extension ?

i do reinstall extension ? after disable extension update for @conqp

Since Gnome 42 is around the corner.. just wondering what the plan is for Wireguard-ui?

Updated to 41


Update docs with correct values for distance to QR detection

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/qr-code-tracking.md
:white_check_mark:Succeeded
View

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


Set disqus_url explicitly

The disqus_url configuration variable needs the site's domain.
We need aldryn/mirumee-feedback#8 resolved to have the domain available.

+1


Any Android ports?

Did anybody try to port this service to Android?

a quick glance at https://developer.android.com/guide/topics/connectivity/usb/host implies that it is likely possible, and could be a simple task, for somebody familiar with the code

well, i solved this issue by accident, from the totally wrong direction
https://github.com/librerpi/rpi-tools/tree/master/webusbboot
this code uses webusb to flash an rpi4 (or cm4) over usb
i have successfully flashed my rpi4 eeprom from desktop chromium using this code
and if i power the rpi4 from the gpio header, and use an OTG adapter cable, i can run the exact same JS on my phone, and shove msd/start4.elf over, putting it into MSD mode

No plans for an official Android port so closing.


Large images are not importing.

Godot version:
Godot 3.1.1 (Steam Version)
OS/device including version:
OpenGL ES 2.0 Renderer: GeForce GTX 1060/PCIe/SSE2
Issue description:
Large images are registered as corrupted in godot when forcefully imported,
and just wont import at all when drag and dropped.
log
Steps to reproduce:

make image size 30000x1920 (what i used)
try to import to godot

Your textures are too big:
core/image.cpp:1460 - Index p_width - 1=29999 out of size (MAX_WIDTH=16384)
See https://docs.godotengine.org/en/3.1/classes/class_image.html#constants
And unlikely to run on any GLES2/WebGL1 (only) capable device since most of these do not support more than 2048x2048 (some mobile can go up to 4096).
You can try changing this
https://github.com/godotengine/godot/blob/eaaff9da3178fa515a0f051fda932c1dd04d53db/core/image.h#L59
And see if something breaks for your game if going too far from the default hardcoded limitations.

OpenGL does not support textures beyond 16,384. Even on the best desktop hardware.

Thanks @TheDuriel, I could not find the numbers anywhere, I guess that it should be clarified on the documentation entry so it does not look like an arbitrary limit.

This might be fixed with Vulkan. It probably doesn't have the limitations of OpenGL.

Technically the limit is several times higher in vulkan yes. But it will probably still refuse to work.
Practically you will find that very few consumer GPUs are even capable of handling textures 16k or higher for real time usage purposes. We're talking about images that easily take up 3 gigs on disk. And close to half that in vram. Just for 16k.

Rendering is not a problem as long as I can import (or rather load) the image, chop it down into chunks, and work with those as individual ImageTextures that are rendered based on visibility or whatever. Besides the use cases shouldn't be limited to rendering.

LargeTexture may be able to solve this (like in 2.x) but it does not have an importer that splits image data into small chunks.

As I said in the linked PR, it does fail to render the preview of a texture larger than 16384 pixels in size.
Though I've found some places where this limit is inconsistent but not sure if it's related to specific image format:
https://github.com/godotengine/godot/blob/4e4697b1c481094949165fa9edbe6aeebcfcf3b4/core/image.cpp#L1675-L1676
So yeah I'd basically would like to increase image max size to at least 32768 which should cover most use cases I believe, and just show a warning that the imported image won't render past render limit which seems to be 16384.

Something like this:


This Is A Title vs This Is a Title

Latest version favors the former; prior versions correctly favored the latter.

Fixed and released in https://github.com/blakeembrey/change-case/releases/tag/title-case%404.1.1


how to overwrite an existing Docker image ?

dockerfile-maven-plugin 1.4.6
maven version: 3.3.9

You can override an image by pushing an image with the same tag again. But that shouldn't be a problem with this project, or am I getting something wrong?


Android support for EFrame

This continues the work I started in https://github.com/emilk/egui/pull/1634 by updating eframe to also defer how it initializes graphics state, which ensures that on Android we wait until the application has an associated SurfaceView.
The first commit (which affects the NativeOption API) is needed for me to use the android-activity Android glue crate with Egui which was a pre-requisite for being able to create an Android test/example using eframe. Notably; the aim here was to create a general mechanism that allows customizing the event loop so it's not necessary for Egui/EFrame to have a direct dependency on the android-activity glue crate (since Winit will generally handle the Android platform integration).
The minimal example here can be used to test this - and that will hopefully be updated soon to use demo lib.
The main issue hit while making this change was realizing that it was only really possible to support Android fully with Wgpu currently, not with Glow/Glutin.
The difficulty with Glutin is that it currently tightly couples a GL context with a window which means you can't destroy and recreate just the window surface when an Android app is suspended - you're forced to also destroy the full GL context, at which point you're basically forced to tear everything down. There is work in progress to improve the Glutin API so suspend and resume will be possible to support in the future, but I'm not sure what the eta is currently for this work.
For now though I have still updated the Glow integration in-line with the Wgpu changes - so it would be possible to launch an Glow based application on Android and it should work until the application is suspended - which is at least an incremental improvement and hopefully it will just be another incremental change to support suspend and resume once the Surface API lands for Glutin.
Closes #1951

It complicates things a bit, but not more then necessary afaict. Thanks for working on this!

No worries, thanks for taking a look.  Yeah, it's a bit more involved, though hopefully not too unwieldy. There's perhaps also a chance that having a clearer line between app state, render state and window state could help with other things, besides Android, later too - maybe a stepping stone for supporting multiple windows with desktop window systems.

I just pushed a change to remove a redundant #[allow(unused)] above create_event_loop_builder (the original iteration of that code had some #[cfg(target_os = "android")] code that could lead to "unused" warnings, before adding the hook mechanism).

I noticed that the destructor of the app is not called when the app window is closed on Android with this branch.

I also noticed that this currently continuously redraws, which is not the best behavior on a mobile device.

Okey, thanks for experimenting with the branch @Zoxc. At least for the latter I'd figure it might be reasonable to tackle that as an orthogonal follow up issue (unless its affecting other platforms), and for the first one we should at least double check things are cleaning up correctly e.g for Windows - to check this doesnt regress somthing.
I have seen issues with wgpu resource cleanup with egui before on Windows (non-eframe) so would be curious if thats related to this deferred initialization pattern somehow.
I might not get around to testing for some days I guess so if you manage to figue out any more details it would be great to hear what you find from experimenting @Zoxc


fix header installation path

Fixes #77

@clalancette Could I please request a review?

After this in detail, it appears that this commit does not fix the issue.
https://colcon.readthedocs.io/en/released/user/overriding-packages.html#install-headers-to-a-unique-include-directory


Add support for sql expressions

Abstracts out IntakeDremioSource params + _get_source into IntakeBaseDremioSource and then I just needed to manually build self.cat and calling source.get("table") works!

Why create a fake cat if _get_source can just return a DremioSource dynamically?

Oh you're right. I just saw self.cat in the parent classes, and since it's not an internal variable, I thought it must be defined for all children classes.

Oh I guess this too:

Ah suppose the intake source always wraps catalogs. Bad abstraction, but let's just keep your version for now.

Codecov Report
Attention: 9 lines in your changes are missing coverage. Please review.

Comparison is base (ce5a303) 69.37% compared to head (b0b4f2e) 69.36%.

Files
Patch %
Lines

lumen/sources/intake_dremio.py
43.75%
9 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #517      +/-   ##
==========================================
- Coverage   69.37%   69.36%   -0.01%
==========================================
  Files          79       79
  Lines        9192     9200       +8
==========================================
+ Hits         6377     6382       +5
- Misses       2815     2818       +3

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


Whitepace still removed incorrectly for some ternary conditions

What version of prettier-plugin-tailwindcss are you using?
v0.6.5
What version of Tailwind CSS are you using?
v3.4.4
What version of Node.js are you using?
v18.20.4
What package manager are you using?
Yarn
What operating system are you using?
macOS
Describe your issue

in 0.5.3, this code is unchanged.
in 0.6.5, it is changed to

This was discussed previously in issue #285.
It seems like it was fixed in #288, but apparently not in this case.

what templating language is this? Angular?

Yes, this is Angular.

Same in template strings
// React
<div className={`text-black${condition ? ' bg-green': ''}`} />

becomes
<div className={`text-black${condition ? 'bg-green': ''}`} />


JAMES-3775 Write webadmin route to create feeding ham messages task

2 latest fixup commits address comments of this PR and similar ones in Spam PR. The rest is basically the same.

The ci fail is not related to this pr

Can you fix the typo "RSpamD" to "Rspamd"?

Can you fix the typo "RSpamD" to "Rspamd"?

Me what I don't like is that instead of 1 PR doing 1 job that fixes the typo, now what you suggest is others n PRs doing both their job plus the fix the typo stuff.
Me I would prefer the typo PR to do its clean job eventually.


Support for corporate proxies added

this PR is referred to

PR https://github.com/zeit/now-cli/pull/735 (now-cli)
ISSUE https://github.com/zeit/now-cli/issues/255 (now-cli)

Some review tasks ?:

[ ] verification/testing
[x] commit squashing (happening on merge)

does someone need tests?

@naxmefy Would you mind applying this to zeit/now-cli? We're closing this repo here... 😊

@leo lol? for real?


-Dreportal.user.name should be set to ${reportal.execution.user}

Here - https://github.com/azkaban/azkaban-plugins/blob/master/plugins/reportal/src/azkaban/viewer/reportal/ReportalTypeManager.java#L58-59 - -Dreportal.user.name should be set to ${reportal.execution.user}, not userName.  We want the property to show who executed the report, not who last edited the report.

Fixed as part of https://github.com/azkaban/azkaban-plugins/issues/173.


Gas optimisation and private constant explicitly named

Utilises the following techniques:

Conversion of fixed variables to constants
Use default value initialisation for zero based loops
unchecked  for limited size collections when incrementing or performing operations with known variables that cannot over/underflow
Conversion of some view functions to pure as no state will be modified (no static calls to precompiles)

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

Closing this pull request. Already included in #781 👍


nunique + TimeGrouper error

This used to work in the past:
tmp = pd.DataFrame({
        'ID': {pd.Timestamp('2015-06-05 00:00:00'): '0010100903', pd.Timestamp('2015-06-08 00:00:00'): '0010150847'},
        'DATE': {pd.Timestamp('2015-06-05 00:00:00'): '2015-06-05', pd.Timestamp('2015-06-08 00:00:00'): '2015-06-08'}})
tmp.groupby(pd.TimeGrouper('D')).ID.nunique()

but now I get the obscure:
Traceback (most recent call last):
  File "test.py", line 6, in <module>
    tmp.groupby(pd.TimeGrouper('D')).ID.nunique()
  File "/usr/lib/python3/dist-packages/pandas/core/groupby.py", line 2697, in nunique
    name=self.name)
  File "/usr/lib/python3/dist-packages/pandas/core/series.py", line 227, in __init__
    data = SingleBlockManager(data, index, fastpath=True)
  File "/usr/lib/python3/dist-packages/pandas/core/internals.py", line 3736, in __init__
    ndim=1, fastpath=True)
  File "/usr/lib/python3/dist-packages/pandas/core/internals.py", line 2454, in make_block
    placement=placement)
  File "/usr/lib/python3/dist-packages/pandas/core/internals.py", line 87, in __init__
    len(self.values), len(self.mgr_locs)))
ValueError: Wrong number of items passed 2, placement implies 4

The logically equivalent:
tmp.groupby(pd.TimeGrouper('D')).ID.apply(lambda x: x.nunique())

works as intended.

hmm, does look buggy.

this last worked in 0.16.2, and failed in 0.17.0 (and continues in 0.18.0),

Thanks for investigating the exact breaking point. I'm currently revisiting some code that I wrote for python2.7 with pandas 0.16.* and now I'm porting to python3 and 0.17.1 (currently Debian unstable).

yeah there were some fixes related to this, but this one didn't take.


Method to load full Media object with custom metadata.

Currently this SDK provides a load() method with title and thumb parameters that are passed to the ChromeCast in a customData parameter (as a sibling to the media field).  I can not find any documentation explaining this approach and it does not seem to have any affect on my ChromeCast.
From the SDK docs, the load() method [takes a MediaInformation object] (https://developers.google.com/cast/docs/reference/messages#Load) (Media in the SDK) the parameters for which include metadata, duration and customData (a different customData field to the one mentioned above).  However these are currently marked with @JsonIgnore (it is not clear why this was needed).
It would be useful to be able to specify the full Media object to the load() method and have the metadata, duration and customData fields included in the request.

Again, sorry for the delay. I was very busy. Please read my comments.

I have made those fields on Media only serialise when not null and added unit tests to verify the annotation works as expected.
I tried to reply to your other comment about customData, but I am not sure how GitHub decides to attach line notes to PRs, so here is a direct link to it.

Generally looks good. Please see my comment in the code.

I have removed the customData param.

Merged. Thank you.

Coild you please make a release with these in at some point?

I have just released 0.9.1. It should become available in maven central within several hours. Please notify me if it does not.

Got it.  Thanks!


Add sonobuoy plugin

TODOs:

[ ] get image pushed up to dockerhub
[ ] label as destructive
[ ] post to sonobuoy slack

💯 I haven't run this yet but just by peeking at it I can see this looks good. The main thing when turning an app into a plugin is simply handling the reporting of results. Here you're putting it into a tar for results so it seems good.
The only other thing to consider would be which format you want to export results in. Some formats can be more easily processed by Sonobuoy; you may have already taken that into account though. I'll have to run it and see what it looks like.

awesome @johnSchnake check out https://github.com/mattfenwick/cyclonus/runs/2324403031?check_suite_focus=true to see some real output !

Once this is merged I'll work on a PR to transform the results so sonobuoy results can understand the output and allow people to script around it more easily.
👍


any way to validate that an object is a buffer?

basically I'm speaking of a custom type, which might be useful as a built-in via Buffer.isBuffer() check in node.js env

You need to define a custom keyword for that.

ok, ty.

See instanceof keyword in ajv-keywords


Dry Dirt handeling

For some reason dirtcraft misses dry_dirt variants when creating recipes. I plan to rewrite the fuzzy logic on my fork to fix this. If you want I can make a pull request when I'm done.
Also dry_dirt + water_bucket should make regular dirt and regular dirt should be smelt-able into dry_dirt. (I'll add this in my fork as well).

thanks for the work!  This issue is closed by the merging of #5


Digester produces invalid sql for selecting global variables

Bug Report
1. Minimal reproduce step (Required)
Linked issue.
The digester produces an invalid SQL for the following statement:
SELECT @@global.tidb_stmt_summary_refresh_interval AS value

This statement is generated by the tidb-dashboard.
I have another digester bug in https://github.com/pingcap/tidb/issues/14241 - it is annoying when the syntax generated back is invalid.
2. What did you expect to see? (Required)
SELECT @@global.tidb_stmt_summary_refresh_interval AS value

3. What did you see instead (Required)
There is a space between the two @ symbols:
SELECT @ @global.tidb_stmt_summary_refresh_interval AS value

4. Affected version (Required)
mysql> select tidb_version()\G
*************************** 1. row ***************************
tidb_version(): Release Version: v4.0.0-beta.2-684-gf590d74e1
Edition: Community
Git Commit Hash: f590d74e158cf4e33555bfeb8f88971af5956697
Git Branch: master
UTC Build Time: 2020-06-27 02:29:48
GoVersion: go1.13.8
Race Enabled: false
TiKV Min Version: v3.0.0-60965b006877ca7234adaced7890d7b029ed1306
Check Table Before Drop: false
1 row in set (0.00 sec)

5. Root Cause Analysis

I suggest to fix this issue in 2 steps:

Copy original content instead of formatted content when clicking the "Copy" button
Fix the incorrect format behavior for sql-formatter-plus-plus lib

For step 1, some reasons (personal opinion):

It seems most people report the formatted SQL is incorrect because they copy and paste to run it in the SQL client, found it doesn't work, else they can't notice the incorrectness. So I think people (especially the DBAs) care the correctness more than style.
Formatting the SQL belongs view layer staff, it is just for easy reading, but when copying and pasting to SQL client, the style is not that important.
We can't confirm the sql-formatter-plus-plus lib won't format some others incorrectly, we need to fix them case by case.
Copying the formatted content makes locate the problem more complex a bit. If people report the formatted SQL doesn't work, we need to find out whether it happened in the TiDB layer or dashboard layer.

hi @breeswish , how do you think about it?

According to my OnCall experience formatting SQL is very frequently used, even for copied content, since an unformatted SQL is usually not readable, for example, too long, or all in one line, which makes people hard to know what is it and what's inside (is it a join? does it contain sub queries? etc) .
How about introducing "Copy Original" and "Copy Formatted"? This may be good for all cases.

Ok, we can do that.


servers.xml getting 404'd

Hosting on Ubuntu 20.04.6 LTS
Docker version 24.0.5, build 24.0.5-0ubuntu1~20.04.1
docker-compose version 1.25.5, build 8a1c60f6
Using python:3.9-alpine instead of latest, because of aiohttp using deprecated functions

I recommend joining the discord and asking there https://discord.com/invite/W5ffx6Xcmv


ImportError: No module named 'requests'

What is this requests library that is needed here? The only module I can find with this name is django.contrib.sites.requests, but if I try to explicitly import that, the tests trying to use it fail. What requirement do I need to install for this to work? Please also mention such dependencies in the readme.

@petersohn sorry, I must have missed your bug report. The Python 'requests' library is declared as a dependency in setup.py. Therefore all you need to do is:
pip install robotframework-djangolibrary
This will automatically fetch the requests library. If this is not the case please provide an error trace and a more detailed description of how to reproduce the problem.

@petersohn will close the issue now. Feel free to re-open if you have additional information.


Support Arweave URLs for avatars

Some NFTs (like CryptoTrunks) return Arweave URLs like ar://....
This should be easy to add support to and very similar to IPFS/IPNS, because there is a gateway service we can use: https://arweave.net/...

now we do!
example: https://metadata.ens.domains/mainnet/0x57f1887a8BF19b14fC0dF6Fd9B2acc9Af147eA85/0xb1be497c1b7ca47f036d5be06468a146a7343b0c223afb875521cc40dc229dbd/image


Don't import the shell from fragments/entrypoints.

Fixes #471 / #489
[x] CHANGELOG.md has been updated

cc @web-padawan

I think it's time to file a docs issue for the bundler to give them a heads up.

@justinfagnani I'll add a docs issue and link to it from the release PR I'll generate as soon as this is merged in.


⚠ RPC server - Infura Matic Mainnet has degraded performance

In 9aea5f5, RPC server - Infura Matic Mainnet ($INFURA_MATIC_MAINNET_RPC_URL) experienced degraded performance:

HTTP code: 429
Response time: 490 ms

Resolved: RPC server - Infura Matic Mainnet performance has improved in 0e3a2c4 after 13 minutes.


New queue design

Codecov Report

Merging #25 into master will decrease coverage by 2.46%.
The diff coverage is 15.49%.

@@            Coverage Diff            @@
##           master     #25      +/-   ##
=========================================
- Coverage   47.97%   45.5%   -2.47%
=========================================
  Files          63      63
  Lines         815     879      +64
=========================================
+ Hits          391     400       +9
- Misses        424     479      +55

Impacted Files
Coverage Δ

questions/urls.py
100% <ø> (ø)
:arrow_up:

frontend/urls.py
100% <ø> (ø)
:arrow_up:

ohqueue/urls.py
100% <ø> (ø)
:arrow_up:

questions/serializers.py
27.27% <ø> (ø)
:arrow_up:

users/views.py
49.12% <0%> (-1.79%)
:arrow_down:

questions/views.py
19.8% <12.5%> (-3.39%)
:arrow_down:

ohqueue/models.py
19.84% <20%> (-0.63%)
:arrow_down:

ohqueue/views.py
31.96% <21.42%> (-3.45%)
:arrow_down:

Continue to review full report at Codecov.

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


[SOLVED] In production with gunicorn/daphne/nginx the observer does not seem to work

Using ObserverModelInstanceMixin locally, everything works perfectly, when it is deployed via gunicorn/daphne/nginx, messages are not received.
I wrote a post on stackoverflow about it.
https://stackoverflow.com/questions/63190426/django-channels-rest-framework-dont-return-data-of-subscription-in-production-e
It has been solved, routing problem with nginx when using daphne + gunicorn thanks the same.
Thank you

Hi everyone,
I have the exact same problem as @egeominotti described. I use uwsgi for http, daphne for websockets and nginx as a proxy and I start both uwsgi and daphne with supervisord.
Locally everything works fine, I can subscribe to instances and get notified when the model changes. When I deploy everything on AWS however I can still connect to the websocket connection and even retrieve model data or subscribe to instances with the ObserverModelInstanceMixin but when the model changes the new data isn't sent to the client. I checked the nginx setup on the mentioned stackoverflow post, but I still can't make it work.
Here is my nginx config:
upstream websocket-server {
server localhost:9001;
}
server {
listen ${LISTEN_PORT};
server_name *.xxxx.net;

tcp_nodelay on;

location /static {
    alias /vol/static;
}

location / {
    uwsgi_pass              ${APP_HOST}:${APP_PORT};
    include                 /etc/nginx/uwsgi_params;
    client_max_body_size    50M;
}

location /ws/ {
    proxy_pass              http://websocket-server;
    proxy_http_version      1.1;
    proxy_set_header        Upgrade $http_upgrade;
    proxy_set_header        Connection "Upgrade";
    proxy_redirect          off;
    proxy_buffering         off;
    proxy_set_header        Host $host;
    proxy_set_header        X-Real-IP $remote_addr;
    proxy_set_header        X-Forwarded-For $proxy_add_x_forwarded_for;
    proxy_set_header        X-Forwarded-Host $server_name;
    proxy_set_header        X-Forwarded-Proto $scheme;
    proxy_read_timeout      18000s;
    client_max_body_size    5M;
}

}
@egeominotti could you please tell me what exactly was the problem and what helped you solve it with the nginx config or do you guys have any other idea what I could try or what could be wrong?
If you need more information please let me know. I'm stuck for days and help would be really appreciated.
Thank you,
Moritz


How do you turn it on?

I have VS 2015 - I installed it, opened up VS and nothing? No options, no shaking, no particles. How do  you turn it on?
(Also why is this not a vsix so it can be installed easier?)

Sorted it out - sorry - ignore this issue.


Add GitHub actions

This repo is built automatically from an airtable DB, the .Rmd is rendered to README.md. At the present time the rendering is done manually but I imagine there is a simple way to automate this. The only difficulty I would think would be authenticating to the DB but there must be a way to do this I would have thought (I'm sure it will involve GitHub secrets)

The Airtable DB uses an API and database specific private token. There is also a mechanism within Airtable to have an action  trigger something. It looks like you can automate a new GIthub issue I don't know whether that can feed into an automation.
https://support.airtable.com/hc/en-us/articles/4408177288599-Airtable-automation-actions-GitHub
info on the public API https://support.airtable.com/hc/en-us/articles/203313985-Public-REST-API

I'm happy to have a look at the GitHub action to render the document if you want. Few quick observations around the Readme.Rmd:

using pacman: the easiest way to get github actions to work is with renv, not sure how the two interact? I would suggest switching to renv and removing pacman having a quick look
the api key is loaded via the keyringr package: seems sensible. Looking at the doc's I think what we want is a render.R script that will first use backend_env() and github secrets to setup the secret key before calling rmarkdown::render().

Looking at the link @jasonpott  provided above from the documentation it looks like airtable will create an issue in the repo - looking at the doc's you can trigger an action from an issue so should be possible!
The other option would be to just simply run the action once per day

@tomjemmett an action has been added that creates an issue in this repository with the label airtable-update.
The issue includes the package name, the description added to the form and the use case.

I think that the action is now functioning - when an issue is labelled with "airtable-update" the action is triggered. It will render the Readme.Rmd then use the action peter-evans/create-pull-request to create a new PR.
The PR it creates is very minimal, and it doesn't like back to the issue that triggered it. We may be able to improve the action step that runs the PR to have better information like a better branch name, description, link to the previous issue (probably using the github.event object).
The idea for PR rather than direct commit is to prevent any thing that we do not approve being auto-pushed up.

Indiscernible from magic.
I don't know if this is pertinent to linking the PR back to the issue?
https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue
Alternatively It would be helpful to see the content of the issue in the PR message so the content being added through the merge can be screened. Even if it was a hyperlink to the issue.

Closed by https://github.com/nhs-r-community/awesome-nhsr/commit/d0367dc5036e906c2da0e04f8eabc17a483cab53


ignore html comments ?

looks like the loader will parse comments stuff. and the ignoreCustomFragments do not help either.
hmlLoader: {
  ignoreCustomFragments: [/<!--.*?-->/]
}

@michael-ciniawsky It works, thanks.

@michael-ciniawsky It works, thanks.

any docs about the options about htmlLoader I can use  ?


Added AI-BookRecommdation component

#343 The pull request adds the AI-BookRecommdation component to the repository. It includes the necessary files, such as main.py, requirements.txt, and sample.config.toml. The component utilizes Goodreads and OpenAI's ChatGPT to fetch book data and generate personalized book recommendations. The pull request also includes instructions for setting up the component and configuring the necessary dependencies.

#343 The pull request adds the AI-BookRecommdation component to the repository. It includes the necessary files, such as main.py, requirements.txt, and sample.config.toml. The component utilizes Goodreads and OpenAI's ChatGPT to fetch book data and generate personalized book recommendations. The pull request also includes instructions for setting up the component and configuring the necessary dependencies.

Kindly follow the template for creating pull request and add some screenshots or video showing your feature. @ArpitaAgrahari


from autovideo import extract_frames is nor working

when i ran
"from autovideo import extract_frames"
I get following error
"ImportError: cannot import name 'extract_frames' from 'autovideo' (/Volumes/Disk-Data/pose estimation/autovideo-main/autovideo/init.py)"

@amitvermanit Hi, thanks for the feedback. This interface is deprecated. Currently it is wrapped as a pipeline primitive. See https://github.com/datamllab/autovideo/blob/main/autovideo/utils/d3m_utils.py#L125-L132

After installing all the libraries. I ran
python3 examples/fit.py
as suggested in the github link provided.
I got following error
cInvalidPrimitiveCodeError: Method 'fit_multi_produce' has an argument 'produce_methods' with type 'collections.abc.Sequence[str]' and not an expected type: typing.Sequence[str]
Can you tell the autovideo version which I should install. I am using macbook. As you mentioned in the github link the code is only tested in linux, will it work in mac or not. Is there any other updated code. Please reply.

@amitvermanit Hi, thanks for the feedback. It should be able to work in Mac (at least for Python 3.6). Could you provide more details for me to reproduce the errors, like what Python version you are using, and which command you used to install (i.e., whether you used pip3 install autovideo or clone it and run pip3 instal -e .. Thansk!

python version 3.9.
I have installed with pip3 install autovideo.
the error usually comes at the 'config' part.

@amitvermanit I have just tried Mac with Python 3.8.2, torch==1.9.0, torchvision==0.10.0. It seems to work well in my side when setting the number of workers for the torch data loading to 0 here  https://github.com/datamllab/autovideo/blob/main/autovideo/recognition/tsn_primitive.py#L64 (two workers will cause some issues in pickle and we are looking into this) Could you try downgrading your python to see whether it works?
We didn't fully tested on Mac since most users will use Linux GPU servers. But thanks for the feedback. We will take more efforts to better support Mac system.

I have tested with Python 3.8.2, torch==1.9.0, torchvision==0.10.0 as suggested. one of the issue I was able to understand is there is a folder named 'autovideo' which is conflicting with python package 'autovideo'.
For example: when I ran "python3 examples/fit.py". I got following error "No module named 'autovideo.transformation'" (it pops inside 'pipeline = build_pipeline(config)' part.)
It has to call the file 'transformation' inside autovideo folder but it is looking in the 'autovideo' python package library. Is there any suggestion how it can be resolved. In windows and linux, it might not be causing the issue.

@amitvermanit Thanks for the feedback. This is bug. This is because we didn't put a __init__.py under the transformation folder. I have just fixed it and uploaded to PyPI. Please try again.

Thanks for the reply. As suggested I have installed the newer version of i.e. 1.2.2 with torch==1.9.0, torchvision==0.10.0.
Now I have getting  this error
ModuleNotFoundError: No module named 'autovideo.augmentation.meta'
I have seen that similar to previous code it should go to the folder 'autovideo'. Is the cause of this error is similar to previous one?

@amitvermanit Yes, this is a similar issue. I just fixed it. Please try again.

when i ran
"from autovideo import extract_frames"
I get following error
"ImportError: cannot import name 'extract_frames' from 'autovideo' (/Volumes/Disk-Data/pose estimation/autovideo-main/autovideo/init.py)"

I still get the error how to rectify it and make it run?

I am also getting same error. How to run it?


Added AI Code Validation

AI to validate code snippets. I had good results testing this. Text gets filtered out pretty much all the time.
Here are some test results:
testing: abc
false

testing: abbbbc
false

testing: a();
true

testing: hello bbs ((((
false

testing: this could be code()
found keyword: this
true

testing: <html stuff="blep">abc</html>
true

testing: for(var i = 0; i < 10; i++) { }
true

testing: while i make cheese, i eat it
found keyword: while
false

testing: if (i == 0) { }
found keyword: if
true

testing: if (i == 0) { } else { }
found keyword: if
found keyword: else
true

testing: lorem ipsum dolor sit amet
false

Code hasn't been integration tested. Unit tests succeeded with high confidence. Please run and see if everything works as expected.

Good idea @Kek5chen. Sorry I've been away for some time. I'll test it out this week!

Hey @Kek5chen, I tried to test the code. It works well with code that has a large amount. Though, not work in some cases, i.e simple code (which are unlikely to be a StackOverflow answer)
I used the following code to test (the link is actually long)
https://www.typescriptlang.org/play?#code/MYewdgzgLgBA1gUwJ4HcQCcAmEYF4YDaARJggGZEA0MRAbiAJaZU0NhQtEMXVEIA2EBJxQALBv2G90CKAFd0YTlHRypNMgENB6omDn9+y8RCIBdANwAoKwHpbMAMKiEwODDGyX6GFBcw2AAc5WAhREANMGAAjBBhQSCYEGSjNHFBSALBg2AwYdE0wTBAAW18EAA8oKzI5MGAoBnAAiEcQUgA1bSYACiCQgC4YaHQ2AHMASiHokBBJQpgAbysYVfjwaBgQQIQwAEEwMck8LJyAOghA-gYoHoByAB47ibPJQ78YAFoYAEZrNfWkFgwH4ICEmAORzi+H6UAuVxu9wAfM9Xrsxh9vn8bACEpttrsAEIFNyyHAw7IheHXW53Aiot4Y0RfX7-NZ44Gg8HEzSkqDk05Uy40+5mBnozGsnHsja5HZgRwKfhIE6w6mIu6LcXvZlYtmrDnxLkITCK9DK1WUuHCjUAX21TJZ2JWMqBW3lAAVNDIwJbzjbafcXozJdjcbKjWCTV6fX6hQjaRMHaHrC6DRGoCAoNoTgT9odjgBqSPgyFF91EkmIfkwYsgqOYHl8nDFvNmi1142mpUq1ue727Wsl6MDsCp8NutgIACOcm6UBV+E+AFlNH4zppohAenmy3FvvXSwWEBMvmmATBV+vN9u803qzgD1372TT59zwCr6IN1ud-L2yqT4NgBb4fmsX4-re-axkB4IxrsEzjq6mxCCUDCgKCkBxtaCb3BYya6lKE6bBhGzYeqtIDARTpIaskiwIgqAYNgABiERFCcAAMtGCjhIp3DAqIQKUCA9GgWB4EiSxgas3AwD0jHidgZxsCCcikNuSkTKeywXnp8DIEpEBsXUmCFoW+r6asMjyIoviqAglkAraMn5LICi+loOiWbaiHSnRsiAmQST1NCMDceeCTBaQoW1vgmbZvwMBSZxMAAPwwGcACsMBDJ82WWVFIXAHEhYUucIbMlJABs6WZTleVnGGrrRbsJVxVkM5zjSKoPOFdVTrO84qg4PypUMCU5rg+CpRl42ZbVABUvhZjmDgAEyFeArWxWVwwIGhpFYSldXZblXwFZF23FaV+BHTgJ0ZWc63nflm1XWAO3tXtinMcZ7FRI9mUAOznRF542R5QU3cl+DYi5ViGrCEBDAQIzjNQMxzAghRmAQZgnAQ-kwPYMAAEqFMUZTo4c54EAABoIIDJJg9PUF5QhmJQxOkwAUpotCaAAysAoyBNUAIM7U9SNM0PQ6TAtps-ZahczzDgAKpkwAMnT9NjLMUJnKAJTKxzCBq+efMC8LosMOLeuCz4FQnAALBYysqKr3NWw4-OCyLYsS2sDOGrAcMAJwe9QXsWz7AKkx6SB+OAeuaGMYVjZ7DmWwnfs2xAdsO5L9OBKM7A9EQLiGCAHgYPwzATNn3vqzA-tpEXsA9LUhjrKQEx69LDRNL6wDaMABhrqJsITMsaZOzAZAnIsMCaJgURDHchZ3DJ1AQHI0TnXcnwCbvMCYAwtBH7YO-6dQJT8LAm+Lbfen3+0R8AKSvxe1CVIER8AB6AlbS0VJo4GQU9V7oAKCqMgeQADyWBkhbCXggnYBRZa+kpjAMurgTRtQQGmMgZxZi13wAQAghASEPygATagBASEX1oPQ6hZwSjtFYYws4-8CZqyslQ7ha9MBcJIfvaIfDLD+VhORGQVxeSiVsAQQBnFPgR0-oAxaAAdWw8stGfELGcMwtgxiUDuM8CwVk1ikxBDjX0cgAF1DAPgiAEBvQqmAKIb0vIoDJAgP5BeEQoA5H1PA9APQF4MFwJxagYBcAkLIWiHUliGAPDHAEcyMBZ7SlJmTBAYxJ4+AAKIVDwa4keK0YCghAO4MJFZMEYBwLEKAKAECDjIKCNc4wYD6BKLEdAOA8hsF8RnAZLoF4yBOM4lA5N8klMCPcHoWitFmWWWcNKyzMCLQmD0Agyyt4JNmAQBgZgzgACtGBgHuPsiY28zA7M2YWNZGyVnbIsS6GQrw0hQAAJJFEqFxSxVjgUgr0qTWI8RNAhCaHIHAOCZBCFgJM6A3pYCBDBDkhw2tZgALEBIOIfhkhxAYDgaAEgkpj34BPfgXTmh1MkLQAQqDcEyBKjFEqLo8WSB6G5M4vjoB9CtKebJF57DFAniUXYcIUCjF8fcB4zCURDhkcWR4thFXvIvEEnIJxKXAAAPrDOSGAbQPQ8ljHmWcAAJD8Sg5rLVWvWnauZpTrUAGY-IXjkn0CAAA5TQvqdwhByKeAAPqGmAABCElLE2A3FEtqkI2k3K2V9ImqAQKSYOEqDcGAONzQqjkmALMq8elyD6ckD8MjZEIHkSVHoMhKDps9YrNMCM1iQzsum2iQ8sGQqpYa9gxrTWaCbYESg0QRWr1wJoRafwYi4GiHOtkEAUA3E8X+Kd7I0hxBIcIoYnbsGFmiJY6IkC4BOTHkIReFwD4HvcnZTQnwT0xHPZendN7mH3tTavWwL6z04wvR+K9u72GP2-VDWd-633AY-TQ9oEHH2f2g4B9916SH-0Q76CC6KUA9FHZO09MGLykC0AYKAQx9CGDZC5NYLklYxxzvHaxDgAASAAVZcusS4PGiEiVjAhQQwAAOoIJ1gAEQeH+pEzc46t0cELIWeszgzBdivFwDAxiiAo78TigQXYMdjrnFjbcbZ62CNEa4wBhjZkaNZ+gTAYAlE0GwHoQsVDjHxtAsYEAdJWAAJD+aFkgaAB1SFUjLsM-gVyiACZriJ+umBI1ED8v5hjKs5O+xgELAAitxkO9MhZFO1kUxw7GYDLRYmTBBy4nCwszJKgZ0cMvGdWLk+resRhyAaLMhohQoTSSsmuzAfghhyDdU61yGmtM6fG5N5ysmzA2CkYaQW1wojlRCBAdhmgFm7IyAgP+pTXC+JEaeXAUkSVtE6N0TAPQDvnemnm47DQTR+URhsbGxt5jhL8niL7oIxgRNu4hIAA

You can't really blame me for not detecting short codes though haha, your previous implementation didn't detect those either 😅. Anyways, I'll be adding some keywords and symbols to maybe increase the confidence for these smaller samples.

Absolutely, the old code was temporary. And as said, unlikely the result can be that short.
I'm hesitant to work on this part because (1) there are lots of cases to cover, and (2) I think it'd be safer to show text, rather than filter out code results. (tbh, I have no idea how to ensure how confident the filter is)
Though, I think this PR could cover common cases.

I can test the ability through collecting 100 cases from stackoverflow. That way i can calculate a fairly confident precision. Code that small is mostly of very low quality / very simple. So filtering it out wouldn't be that much of an issue. If people use captain stack to find out how to assign a javascript variable they should rather get autocompleted a youtube tutorial for javascript. haha

I'll just be cherry pick some samples from stackoverflow and paste them into my project. no real need for help, thanks though.

Alrighty then, I'm looking for the update! Feel free to tell me what to do too!

not tested in vscode, should be merge ready though.

https://pastebin.com/t00wi5SE here is the full code

Thanks for this work @Kek5chen. I think the new isCodeValid can be a NPM module itself.
By the way, is it really an AI code validation? AI is usually prefer to a program that process data. Not really important though

Am i not processing data and generating confidence values from it? I'm getting some input and passing it through some code to output a confidence of how certainly the input was code.
Input -> Hidden layer -> Output.

You did processing data and generated confidence values, but it was for testing the result. Whereas AI is used as the program to reproduce results.
So you used AI to test the results. The code itself isn't AI.

Oh wait, are we talking about the program to test the code, or the code?
We're might talking different things here

the code itself of course


[General Issue]: No Packets Reaching Raspberry Pi (Wireguard)

In raising this issue I confirm that

[X] I have read the documentation
[X] I have read and understood the PiVPN General Guidelines
[X] I have read and understood the PiVPN Troubleshooting Guidelines
[X] The issue I am reporting isn't a duplicate, see closed issues and open issues.
[X] I have searched for similar issues and solutions
[X] I can replicate the issue even after a clean OS installation

Describe the issue
TL;DR: Pfsense is not port forwarding correctly on UDP 51820. I need that for my VPN to work.I suspect my ISP could be overriding my port forwarding somehow.
Hello! I am trying to setup my VPN server with Wireguard and Pihole, both which are installed in a Raspberry Pi 400 of mine.
The Pihole is working just fine, when I use it as my DNS server on my pfsense router it works fine and my online activity has no issues.
I tried to connect Wireguard on my computer to see if it works and even though I got a connection, it basically killed all of my connections on my Windows 10 PC.
I ran some tests on my Raspberry Pi to see if there any issues with my connection, as per this link:
https://docs.pivpn.io/faq/#how-do-i-troubleshoot-connection-issues
Eventually, it led me to finding out I have no packets being received by my Raspberry Pi, which means no Wireguard VPN will work, either.
I am almost 100% convinced it is an issue with my router ports, which has pfsense installed on it.I opened the port 51820 on pfsense as it is the default used by Wireguard.
Results for the tcpdump on my Raspberry Pi -
"tcpdump: verbose output suppressed, use -v[v]... for full protocol decode
listening on eth0, link-type EN10MB (Ethernet), snapshot length 262144 bytes"
Results for the nmap on my Raspberry Pi (Port 51820 - UDP) -
"PORT STATE SERVICE
51820/udp open|filtered unknown"
Pivpn (Wireguard) Configuration File -
"PLAT=Debian
OSCN=bullseye
USING_UFW=0
pivpnforceipv6route=1
IPv4dev=eth0
IPv4addr=<IP_ADDRESS>/24
IPv4gw=<IP_ADDRESS>
install_user=REDACTED
install_home=/home/REDACTED
VPN=wireguard
pivpnPORT=51820
pivpnDNS1=<IP_ADDRESS>
pivpnHOST=REDACTED
INPUT_CHAIN_EDITED=1
FORWARD_CHAIN_EDITED=0
INPUT_CHAIN_EDITEDv6=
FORWARD_CHAIN_EDITEDv6=
pivpnPROTO=udp
pivpnMTU=1410
pivpnDEV=wg0
pivpnNET=<IP_ADDRESS>
subnetClass=24
pivpnenableipv6=0
ALLOWED_IPS="<IP_ADDRESS>/0, ::0/0"
UNATTUPG=1
INSTALLED_PACKAGES=(bsdmainutils unattended-upgrades)"
Results for my Windows 10 PC using tracert on my public ip -
Tracing route to REDACTED over a maximum of 30 hops 1 <1 ms <1 ms <1 ms REDACTED Trace complete.
Please help. I don't know what else to do.
Expected behavior
The expected behavior would be for Wireguard to work and grant me connection, not to mention also changing my IP.
A handshake and connection is formed with Wireguard, but there is no connection, not even with other devices on my LAN or home network.
I can't even connect to my router anymore when I connect to my Wireguard.
Please describe the steps to replicate the issue
I click on Wireguard and nothing happens.
Have you taken any steps towards solving your issue?
Yes, multiple. This is why I am posting here.
Screenshots
No response
Where did you run pivpn?
Raspberry Pi 400
Please provide your output from uname -a
Linux raspberrypi 6.1.21-v8+ #1642 SMP PREEMPT Mon Apr  3 17:24:16 BST 2023 aarch64 GNU/Linux
Details about Operative System
PRETTY_NAME="Debian GNU/Linux 11 (bullseye)"
NAME="Debian GNU/Linux"
VERSION_ID="11"
VERSION="11 (bullseye)"
VERSION_CODENAME=bullseye
ID=debian
HOME_URL="https://www.debian.org/"
SUPPORT_URL="https://www.debian.org/support"
BUG_REPORT_URL="https://bugs.debian.org/"
Installation
No response
Profile / Client creation
[Interface]
PrivateKey = =
Address = <IP_ADDRESS>/24
DNS = <IP_ADDRESS>
[Peer]
PublicKey = =
PresharedKey = =
Endpoint = REDACTED:51820
AllowedIPs = <IP_ADDRESS>/0, ::0/0
Debug output
::::            ^[[4mPiVPN debug^[[0m            ::::
::::            ^[[4mLatest commit^[[0m          ::::
Branch: master
Commit: 16189edc7b03d3ed930dbb3cc908d8e2275a7563
Author: 4s3ti
Date: Thu Aug 3 23:33:23 2023 +0200
Summary: fix(core): typo on distroCheck
::::        ^[[4mInstallation settings^[[0m      ::::
PLAT=Debian
OSCN=bullseye
USING_UFW=0
pivpnforceipv6route=1
IPv4dev=eth0
IPv4addr=<IP_ADDRESS>/24
IPv4gw=<IP_ADDRESS>
install_user=REDACTED
install_home=/home/REDACTED
VPN=wireguard
pivpnPORT=51820
pivpnDNS1=<IP_ADDRESS>
pivpnHOST=REDACTED
INPUT_CHAIN_EDITED=1
FORWARD_CHAIN_EDITED=0
INPUT_CHAIN_EDITEDv6=
FORWARD_CHAIN_EDITEDv6=
pivpnPROTO=udp
pivpnMTU=1420
pivpnDEV=wg0
pivpnNET=<IP_ADDRESS>
subnetClass=24
pivpnenableipv6=0
ALLOWED_IPS="<IP_ADDRESS>/0, ::0/0"
UNATTUPG=1
INSTALLED_PACKAGES=(bsdmainutils unattended-upgrades)
::::  ^[[4mServer configuration shown below^[[0m   ::::
[Interface]
PrivateKey = server_priv
Address = <IP_ADDRESS>/24
MTU = 1420
ListenPort = 51820
begin client-alpha
[Peer]
PublicKey = client-alpha_pub
PresharedKey = client-alpha_psk
AllowedIPs = <IP_ADDRESS>/32
end client-alpha
=============================================
::::  ^[[4mClient configuration shown below^[[0m   ::::
[Interface]
PrivateKey = client-alpha_priv
Address = <IP_ADDRESS>/24
DNS = <IP_ADDRESS>
[Peer]
PublicKey = server_pub
PresharedKey = client-alpha_psk
Endpoint = REDACTED:51820
AllowedIPs = <IP_ADDRESS>/0, ::0/0
::::    ^[[4mRecursive list of files in^[[0m     ::::
::::    ^[[4m/etc/wireguard shown below^[[0m     ::::
/etc/wireguard:
configs
keys
wg0.conf
/etc/wireguard/configs:
client-alpha.conf
clients.txt
/etc/wireguard/keys:
client-alpha_priv
client-alpha_psk
client-alpha_pub
server_priv
server_pub
::::            ^[[4mSelf check^[[0m             ::::
:: [OK] IP forwarding is enabled
:: [OK] Iptables MASQUERADE rule set
:: [OK] Iptables INPUT rule set
:: [OK] WireGuard is running
:: [OK] WireGuard is enabled
(it will automatically start on reboot)
:: [OK] WireGuard is listening on port 51820/udp
:::: Having trouble connecting? Take a look at the FAQ:
:::: ^[[1mhttps://docs.pivpn.io/faq^[[0m
:::: ^[[1mWARNING^[[0m: This script should have automatically masked sensitive       ::::
:::: information, however, still make sure that ^[[4mPrivateKey^[[0m, ^[[4mPublicKey^[[0m      ::::
:::: and ^[[4mPresharedKey^[[0m are masked before reporting an issue. An example key ::::
:::: that you should NOT see in this log looks like this:                  ::::
:::: YIAoJVsdIeyvXfGGDDadHh6AxsMRymZTnnzZoAb9cxRe                          ::::
::::            ^[[4mDebug complete^[[0m                 ::::

I had the same problem, and solved it by adding "ListenPort = xxxx" to the tunnel, and opening the same internal port in my router settings

Just installed latest version on bookworm, it now works (thanks). I had a similar issue, port forwarding and DDNS is enabled on the router and yet it it did not work.
I'm pretty convinced it's my new ISP, and not an issue with this script (I see other customers with this ISP seem to be complaining about it).
I was able to confirm it works at my end by installing a wireguard client on a computer connected to my local LAN. I edited the .conf file so that the endpoint so it goes to the static IP address of my pi server on the local LAN, rather than an external DNS address (supplied via a DDNS service). That worked. Investigations continue at my end, but so far no issues here.

Update. It was indeed my ISP blocking (using CGNAT). I'm now using Tailscale.

Pre-archive closing, more information here


Tennessee

TN scraper added

@lazd  this ones ready to go

Hmm, something is funky with the history, I still see that console.log and the other weirdness. I can pull it in manually if getting the branches in shape is a pain.


backoff_factor the pause between for each retry

With a large number requests of on a same URI on to a same unit performing leads to resetting connection back due to this affect we are loosing connection from unit the testcase are failing.
To avoid such scenario if we have a backoff_factor then it will wait some pause time to reset connection back again.

@bulkan any updates?

Thanks

@bulkan thanks, could you push into production this fix so that we could use this fix.


Spectral convergence loss, what does it measure?

Hi There!
I'd like to better understand spectral convergence loss. In the literature, these are the mentions I have found so far:

SC loss emphasizes highly on large spectral components, which helps especially in the early phases of training.
"Fast Spectrogram Inversion using Multi-head Convolutional Neural Networks" https://arxiv.org/pdf/1808.06719.pdf

Because the spectral convergence loss emphasizes spectral peaks and the log STFT magnitude loss accurately fits spectral valleys
"Probability Density Distillation with Generative Adversarial Networks for High-Quality Parallel Waveform Generation" https://arxiv.org/pdf/1904.04472.pdf

The above explanations for including the loss function are fairly vague and short. Furthermore, I am unable to find any mentions of a similar loss else were in literature. To better describe the loss, I searched for "relative spectral power" (f.y.i. since the spectrogram to the power of two is the "power spectrogram", and the sum of the power spectrogram is a "power spectral density").
Lastly, this paper from Google Brain "DDSP: DIFFERENTIABLE DIGITAL SIGNAL PROCESSING" https://arxiv.org/pdf/2001.04643.pdf just uses a spectrogram magnitude loss. It doesn't train with a spectral convergence loss and their results are pretty good.
Have you tried training without spectral convergence loss? Is there some more literature I am missing that validates it as a perceptual loss?

Related issue: https://github.com/magenta/ddsp/issues/12

Have you try it without a spectral convergence loss?

Am not sure if this answer is going to be helpful now. Am just sharing my understanding of SpectralConvergence Loss and it may or may not answer the question raised by you.
SpectralConvergence loss is more related to spectral loss. Spectral loss is just the L1/L2 norm over the difference in the log magnitude of the spectrogram of the signal. Whereas, SCL is Euclidean distance between magnitude of target and denoised signals normalized by euclidean length of target signal.
Since its euclidean it penalises largely, when even one of the frequency range is far apart from the target and forgiving for smaller differences.


Use dependency System: Part 2; Marking as unresolvable

Changes introduced with this PR
The way this works is by marking all outputs that failed as unresolvable.
A callback called OnStepStageFailure was created to allow the providers to self-report failure of specific stages within their lifecycle for a step. The callback is required because the DAG itself doesn't know everything.
Once all outputs are marked unresolvable, the entire workflow exits at that time. It will not keep things running.
The old way of determining failure is still implemented, but any activation of it is considered a bug, either due to a false activation, or due to a missing mark of unresolvability.
With the bugs fixed in the updated dependencies, the tests are very resilient based on my testing. This will need testing with real-world workflows.

By contributing to this repository, I agree to the contribution guidelines.

I need to test if this works with subworkflow/foreach. If it doesn't, I don't think that has to block these changes, but it should be an immediate follow-up PR.

Are we missing a commit here? I thought you were making a few minor changes after the meeting, and I don't see anything.

Yes. I will push the commit.


class_hash instead of implementation_address in proxy contracts

The proxy contract will work with a class rather than a contract, that is implementation_address would change to class_hash.
There is an implementation of ArgentProxyCheck and OpenZeppelinProxyCheck in our project, so we should check how they will manage this change.

Partially resolved in #190
Waiting for OZ, Argent to update their proxies so we can verify


erisdb.transactAndHold fails in rc3

curl -H "Content-Type: application/json" -X POST -d '{"jsonrpc":"2.0","method":"erisdb.transactAndHold","params":{"priv_key":"043133B125C6D574D52A0B6A1446A2BC35FEC48E146F9502323C8044FB8EF36BA3CD2B6DAE2C4C4129341EDFB1F391921A8E2C218233E9","address":"","data":"6060604052603b8060106000396000f3606060405260e060020a600035046360fe47b1811460245780636d4ce63c14602e575b005b6004356000556022565b6000546060908152602090f3","gas_limit":1000000,"fee":0},"id":"1"}' <IP_ADDRESS>:32911/rpc

yields:
{"result":null,"error":{"code":-32603,"message":"Error broadcasting transaction: invalid character 'x' in literal true (expecting 'r')"},"id":"1","jsonrpc":"2.0"}

I think this is fixed by https://github.com/eris-ltd/eris-db/pull/314, but we ought to have a js-libs test case in order to close it. @NodeGuy if you are able to add one that would be useful.

All of the existing tests use this function to create a contract so it's well-covered.


Implements #2734 “New API to configure BrowserWindow title bar on Mac”

New API supported on Yosemite 10.10 and newer.
Two new title bar styles:
window = new BrowserWindow({width: 600, height: 400, 'title-bar-style': 'hidden'});

window = new BrowserWindow({width: 600, height: 400, 'title-bar-style': 'hidden-inset'});

:dancer:

About the failing tests…
I don’t understand why the Linux tests fail, or rather, why are the two new tests run on Linux? I have a condition to bail out from those tests when the OS is not Mac—why is that not respected?
The Mac tests fail because I think Travis OSX runtime is 10.9. Should I make the tests conditional on 10.9 so they are not run (how?), or is there a way to make Travis use 10.10 runtime?

The Mac tests fail because I think Travis OSX runtime is 10.9. Should I make the tests conditional on 10.9 so they are not run (how?), or is there a way to make Travis use 10.10 runtime?

You can use os.release() to check the version of current operating system, I believe Yosemite is 14.x.x.

OK, tests don’t fail any more (they are not run on Travis but are run correctly if you run them locally on a Yosemite machine), I addressed all the comments, and made a squash commit.

Awesome, thanks! :sparkles:

Ooh, snazzy

Now we just have to wait for the release :)


[Request] Add ability to contribute destinations via Dagger multibindings

I'd like to be able to provide navigation generically from feature modules.
The app module could gather all provided navigation via dagger multibindings.
Perhaps something like this:

So for the above to work, you require presentations to not be internal? Or perhaps a way to provide the class itself like this in NavigationConfigBuilder:

That way you can call the above in your app module:

Or in case you have to use the Class<*>:

Otherwise, I haven't use Dagger in ages, so is there something more that needs to be added?

That looks like it would work :)

Alright! I'll add it ^^

Sweet thank you!  I'll close this :P


How to plot streamlines

Hello,
Sorry, if this is not the right place to post this.
I followed the different examples provided by vtkplotter to plot streamlines. Unfortunately I couldn't make it work.
Here is an example of a vtk file cavity.zip  that contains the velocity vector field U.
Here is my code:
"""
Draw streamlines for the cavity case from OpenFOAM tutorial
"""

from vtkplotter import loadUnStructuredGrid, load, show, interactive, Box, Grid, streamLines, Text

cavity = load('cavity.vtk')

domain = load('cavity.vtk')

comment = Text(__doc__, c='w')
probe = Grid(pos=[0.05,0.1,0.005], normal=[0,1,0], sx=0.01, sy=0.1, resx=30, resy=10)
probe.color('k')

stream = streamLines(domain, probe, direction='forward')
show(domain, stream, probe, comment, axes=8)

when I run the example above, I couldn't see the streamlines
I appreciate your help

Hi @Yoorthiziri
this is the right place to ask this type of question :)
please upgrade vtkplotter with the command:
pip install git+https://github.com/marcomusy/vtkplotter.git --upgrade
then you should be able to run:
"""Draw streamlines for the cavity case from OpenFOAM tutorial"""

from vtkplotter import *

# load file as type vtkUnStructuredGrid
ugrid = loadUnStructuredGrid('cavity.vtk')

# make a grid of points to probe as type vtkActor
probe = Grid(pos=(0.05,0.08,0.005), normal=(0,1,0),
             sx=0.01, sy=0.1, resx=4, resy=20, c='k')

# compute stream lines with Runge-Kutta4, return a vtkActor
stream = streamLines(ugrid, probe,
                     activeVectors='U', # name of the active array
                     #tubes={"radius":1e-04, "varyRadius":2},
                     lw=2, # line width
                     )

# make a cloud of points form the ugrid, in order to draw arrows
domain = pointCloudFrom(ugrid)
coords = domain.getPoints()
vects  = domain.getPointArray('U')/200
arrows = Arrows(coords-vects, coords+vects, c='jet_r') # use colormap
box    = domain.box().c('k') # build a box frame of the domain
comment= Text(__doc__)

show(stream, arrows, box, probe, comment, axes=5, bg='white')

Let me know if you encounter any problems running it.
PS; may I add this example to the gallery of examples? Thanks for your feedback.

Thank you very much for the detailed answer.
Of course you could add it to the examples. You could find many others in the tutorials directory of OpenFOAM, just use foamToVTK to convert the results to vtk files.

Cool! I guess that setting activeVectors='U', # name of the active array in the streamLines method should work?

Yes you are right about that for streamlines.
I don't think that this question belongs to this issue but just to clarify: I mean if I have a vtk file, example cavity.vtk that has multiple fields (p, U, ...etc) when I run it like this:
vtkplotter cavity.vtk

By default it show the pressure controus, hw can I switch to another field from within vtkplotter window? is there any shortcut, command line argument for that?
Thank you

Yes, press 4 to cycle through existing arrays.
If you press h, you will get the full list of key options.

Thank you very much.

If you press h, you will get the full list of key options.

Unfortunately the description in the help is not clear, here is the description:

4     use scalars as colors, if present
If I am not mistaken, this is not precise because vector fields are not scalar.
Thanks

You are right, although what is visualized should be the magnitude of the vector.

Hello again!
if the mesh is very dense, how can we plot the velocity vectors at only some locations (that is, by skipping some points)?
Another question, how could we plot the velocity vectors at thr cell centers? instead of using cell vertices?
Thank you

Hi, you could do something like this:

clean() imposes a minimum tolerance between close points (in percent of the bounding box)
cutWithPlane() removes one plane of points (but any numpy method here would also work fine), and z() shifts the whole set of arrows.

from vtkplotter import *

# load file as type vtkUnStructuredGrid
ugrid = loadUnStructuredGrid(datadir+'cavity.vtk')

# make a grid of points to probe as type Mesh(vtkActor)
probe = Grid(pos=(0.05,0.08,0.005), normal=(0,1,0),
             sx=0.1, sy=0.01, resx=20, resy=4, c='k')

# compute stream lines with Runge-Kutta4, return a Mesh(vtkActor)
stream = streamLines(ugrid, probe,
                     activeVectors='U', # name of the active array
                     #tubes={"radius":1e-04, "varyRadius":2},
                     lw=2, # line width
                     )

# make a cloud of points form the ugrid, in order to draw arrows
domain = pointCloudFrom(ugrid)
box = domain.box().c('k') # build a box frame of the domain

domain.clean(tol=0.07).cutWithPlane([0,0,0.0001], normal=(0,0,1)).z(-0.005)

coords = domain.points()
vects  = domain.getPointArray('U')/50
arrows = Arrows(coords-vects, coords+vects, c='jet_r') # use colormap

show(stream, arrows, box, probe, axes=5)

Thank you very much for your answer.
Regarding plotting vectors at cell centers, I am trying to use the following:
 # make a cloud of points form the ugrid, in order to draw arrows
domain = pointCloudFrom(ugrid).mapPointsToCells()
coords = domain.cellCenters()
vects  = domain.getCellArray('U')/200
arrows = Arrows(coords-vects, coords+vects, c='jet_r') # use colormap
box    = domain.box().c('k') # build a box frame of the domain
show(stream, arrows, box, probe, axes=5)

Is that the correct way to plot the vector at the cell centers? I am not able to show the mesh cells to see if the vectors are plotted at the centers because when I press 'w' to show the wireframe, it toggles the wireframes/surface for the vectors instead of the domain.
Thank you

Sorry if my question was not clear
I mean the velocity vectors at the cell centers. For example something like this:

..then you need to do something more complicated.. because the vector field is defined at the mesh vertices so you need to interpolate it to a different set of points (the cell centers).
This can be controlled in various ways, one possibility is to use RBF (radial basis functions) from scypi:
from vtkplotter import *

# load file as type vtkUnStructuredGrid
ugrid = loadUnStructuredGrid(datadir+'cavity.vtk')

#################################################
# use native vtk commands to get cell centers
from vtk import vtkCellCenters
from vtk.util.numpy_support import vtk_to_numpy
vcen = vtkCellCenters()
vcen.SetInputData(ugrid)
vcen.Update()
cns = vtk_to_numpy(vcen.GetOutput().GetPoints().GetData())

#################################################
# make a grid of points to probe as type Mesh(vtkActor)
probe = Grid(pos=(0.05,0.08,0.005), normal=(0,1,0),
             sx=0.1, sy=0.01, resx=20, resy=4, c='k')

# compute stream lines with Runge-Kutta4, return a Mesh(vtkActor)
stream = streamLines(ugrid, probe,
                     activeVectors='U', # name of the active array
                     #tubes={"radius":1e-04, "varyRadius":2},
                     lw=2, # line width
                     )

# make a cloud of points form the ugrid, in order to draw arrows
domain = pointCloudFrom(ugrid)
box = domain.box().c('k') # build a box frame of the domain

coords = domain.points()
vects  = domain.getPointArray('U')/50

################################################# scipy RBF
from scipy.interpolate import Rbf
import numpy as np

cns = Points(cns).clean(tol=0.07).points() # reduce the density

xr, yr, zr = cns[:, 0], cns[:, 1], cns[:, 2]
x,  y,  z  = coords[:, 0], coords[:, 1], coords[:, 2]
dx, dy, dz = vects[:, 0], vects[:, 1], vects[:, 2]

itrx = Rbf(x, y, z, dx)  # Radial Basis Function interpolator:
itry = Rbf(x, y, z, dy)  #  interoplate the deltas in each separate
itrz = Rbf(x, y, z, dz)  #  cartesian dimension

positions_x = itrx(xr, yr, zr) + xr
positions_y = itry(xr, yr, zr) + yr
positions_z = itrz(xr, yr, zr) + zr
positions_rbf = np.c_[positions_x, positions_y, positions_z]

arrows = Arrows(cns, positions_rbf, c='jet_r')

show(stream, arrows, box, probe, axes=5)

PS: the script you show cannot work because the cell positions corresponding to a point cloud is still the same point cloud..

Thank you very much. I appreciate your help.


Code cleanup

Hi @spco, I'll leave this for you to merge into devel. Straightforward PR from Vas which removed unnecessary model and library dependencies.

Thanks @vpnu!


Shares from different lots sold on same day not considered replacements

Thanks for this patch. If you can fix these I'll pull it in.
In true Google style, can you squash your changes into this commit?
Thanks,
-adlr

Done.

Thanks, it's in!


MS3: Peer Review Feedback -Harish

Simple, elegant and clean UI.
Nice architecture and design.
Wide variety of Software technologies (/integration).
Complete documentation.

Thanks for the feedback Harish.


The variation of g_i seems not to be used.

Hi, there may be a major problem that g_i  doesn't be used to modify pc_grad according to the code.
def _project_conflicting(self, grads, has_grads, shapes=None):
       shared = torch.stack(has_grads).prod(0).bool()
       pc_grad, num_task = copy.deepcopy(grads), len(grads)
       for g_i in pc_grad:
           random.shuffle(grads)
           for g_j in grads:
               g_i_g_j = torch.dot(g_i, g_j)
               if g_i_g_j < 0:
                   g_i -= (g_i_g_j) * g_j / (g_j.norm()**2)
       merged_grad = torch.zeros_like(grads[0]).to(grads[0].device)

       merged_grad[shared] = torch.stack([g[shared]
                                          for g in pc_grad]).mean(dim=0)
       merged_grad[~shared] = torch.stack([g[~shared]
                                           for g in pc_grad]).sum(dim=0)
       return merged_grad

Sorry, It's my fault. That is right!

Since the problem is solved, I close the issue.


fyi: Referential Integrity Problem

For your information - not sure if this can be solved:
For this code:
REPORT zevg.

CLASS lcl_app DEFINITION FINAL.
  PUBLIC SECTION.
    CLASS-METHODS:
      main.
  PRIVATE SECTION.
ENDCLASS.

CLASS lcl_app IMPLEMENTATION.
  METHOD main.
    DO 4 TIMES.
      CASE sy-index.
        WHEN 1 OR 2 OR 3. DATA(ext) = |AV{ sy-index - 1 }|.
        WHEN OTHERS. ext = 'MAX'.
      ENDCASE.
"      WRITE |now using { ext }|.
    ENDDO.
  ENDMETHOD.
ENDCLASS.

START-OF-SELECTION.
  lcl_app=>main( ).

I get the message:
Error executing rule 'Delete unused variables': Failed referential integrity test on command starting at source line 14!
When I add a usage of the variable, the problem disappears.

Hi Edo,
thanks for reporting this! This seems to be similar to #315: The issue is gone if you split this line into two:
        WHEN 1 OR 2 OR 3.
          DATA(ext) = |AV{ sy-index - 1 }|.

From the first look of it, this happens when ABAP cleaner tries to put a TODO comment ("variable is assigned but never used") before the DATA(ext) statement, and doesn't properly consider that the statement follows the previous statement on the same line.
Kind regards,
Jörg-Michael

Hi Jörg-Michael,
yes, looks similar.
Not a real problem.
Thanks,
Edo

Hi Edo,
this bug will be fixed with the next release. Since the comment cannot be inserted on the same line between the two statements, the statements are moved to an own line:

Kind regards,
Jörg-Michael

Hi Edo,
this bug should now be fixed with version 1.17.1, which was just released! Thanks again for reporting this.
Kind regards,
Jörg-Michael


Allow DropdownButton Items to be disabled

Feature Request
Description
We have a desire to have a dropdown button (essentially a Menu, but simpler, no nesting required) where some Items can be disabled.
Additional Context / Screenshots
Specifically, HealtheAnalytics is adding Data Set Versioning. A data set can now be in draft, published, or archived, and drafts should not be added to workflows, and published versions should not be edited (they're immutable). Here's the current component:

Here's the branch I've been working from, if you want to see the code: https://github.cerner.com/healtheintent/healtheanalytics/tree/20206-disable-actions
This request comes from UX (Hien Tran and Brenda Nolan) in response to my original suggestion that we just make unusable items disappear.
@ Mentions
@mjhenkes @dkasper-was-taken

In my opinion we could provide isDisabled prop as we have in Terra-Form-select options and Terra-Menu MenuItem.
@dkasper-was-taken , @StephenEsser  your thoughts on this ?.
we would need UX input for the disabled styles implementation.
cc. @neilpfeiffer , @jeremyfuksa

Adding Needs Design Input label until (1) request has been reviewed with Cerner Design Standards to see if disabling button options will be allowed and (2) what the UX styling will be needed to implement.


Missing props in MultilinkStoryblok

Hi, thanks for this very helpful package. I think there are missing and important props in the Multilink type, e.g. anchor or url
Two examples from the API:
 {
    "id": "bd897d30-ca59-4a5f-9577-3f18ca47dc74",
    "url": "",
    "anchor": "test",
    "linktype": "story",
    "fieldtype": "multilink",
    "cached_url": "start"
 }

 {
    "id": "",
    "url": "https://www.google.de/",
    "linktype": "url",
    "fieldtype": "multilink",
    "cached_url": "https://www.google.de/"
}

fixed in 1.10.2


chore: bdd-gh-1585674035 to 0.0.1

chore: Promote bdd-gh-1585674035 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

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


patch version bump v3.1.1

do you mind holding off on this until https://github.com/samvera-labs/bulkrax/pull/516 gets merged in too? hopefully it'll also be today and we won't need to turn around and do another version update in a few hours.
(might also be able to sneak in #524 if the specs act right.

@alishaevn is this good to merge now?

@alishaevn is this good to merge now?

good to go! thanks for waiting!


Enhance add lesson command to include AttendanceRecordList

Currently, upon adding a new lesson, there isn't a blank attendance list. Introduce a blank attendance list upon adding a lesson

Not required.


Move fastroute to suggestion

As it is not mandatory to use aerys

Suggested packages are horrible, because they don't allow defining compatible version constraints. I'd rather move the router out to a separate package than moving the dependency into "suggest".

Is there any win in not requiring FastRoute?
I think the Router is a pretty central feature, I'd dislike to move out. Sure, you may use your own router, but it'll probably be used by many.

IMO any current application get already a router. (this our case). So we want to use aerys but we will keep routing stuff in our side.

Using your own router is perfectly fine, why do we need to make FastRoute optional for that?

Because I like to install only what I really need.
But not a big deal, I was just thinking it was a small mistake so here is the PR. I will use a composer workaround to avoid to install fastroute.

@bwoebi What do you think about moving the router to a separate package?

wouldnt it be a single file repos?

@staabm Basically, yes. But only due to FastRoute containing everything that's needed already.

I'm personally in favor of splitting out the router into its own component library.

I personally consider it a deterrent when I have to first find a shitton of mircocomponents to even start at all…

Never heard of that; I personally never use these composer commands though, but directly edit the json file always.
In this case it may be acceptable though.

I personally consider it a deterrent when I have to first find a shitton of mircocomponents to even start at all…

Everything not having external dependencies is fine. I wouldn't split the document root handler out, as that doesn't have any external dependencies on its own.

But not a big deal, I was just thinking it was a small mistake so here is the PR. I will use a composer workaround to avoid to install fastroute.

@tyx Could you share your workaround? I actually want something similar to exclude all the polyfills in packages that require PHP 7 or even 7.1.

In your composer.json:
"replace": {
        "symfony/polyfill-php56": "*",
        "symfony/polyfill-php70": "*"
}

I don't know if it will continue to work in the next composer release, I hope so as it is very useful currently !

Didn't know about "replace", thanks! Works like a charm.
"replace": {
    "symfony/polyfill-util": "*",
    "symfony/polyfill-php56": "*",
    "paragonie/random_compat": "*"
}


MemoServ

Traditional IRC services packages have a MemoServ bot that lets users send messages to each other, which the other user will see once they connect to the network. It's basically a hack that allows asynchronous messages to be sent between users.
Obviously, the default config makes this sort of thing not useful at all, since y'know, asynchronous messages are baked into how that's designed.
MemoServ has been asked about a number of times, and for the traditional setup it would make sense to have an implementation of this. But whether or not we do add it depends a lot on whether we have time to, ehe, and if we had it, it'd only be enabled for the traditional config etcetc.
If I have time I might look into this, but yeah it's probably not that likely to happen otherwise.

How much of a priority is this because most bot frameworks come with an out of the box @tell command. Later plugin in Limnoria and BitBot also has it. It is possibly a lot of work to add this?

How much of a priority is this because most bot frameworks come with an out of the box @tell command. Later plugin in Limnoria and BitBot also has it. It is possibly a lot of work to add this?

It would be a bit of work to add this because it's a separate message-storing thing and we'd probably need to think about exactly where to store it. Not a priority at all, in my eyes, basically an 'eh nice to have'.

It would be a bit of work to add this because it's a separate message-storing thing and we'd probably need to think about exactly where to store it. Not a priority at all, in my eyes, basically an 'eh nice to have'.

If it's typically covered by a bot (as opposed to services), I think that's an argument against bringing functionality into core.

If it's typically covered by a bot (as opposed to services), I think that's an argument against bringing functionality into core.

I think having features baked into Oragono is an argument for it if someone considered moving from a very traditional IRCd or network such as IRCnet as that can remove the need of running a separate bot for the same thing.
There is also a privacy concern, I think bots mostly use their own user databases if they even check message recepients instead of just shouting their messages into channel upon seeing a matching nick, while the traditional MemoServ integrates with the network user database and sends the message in private with an option to email it to the recepient.

I think having features baked into Oragono is an argument for it if someone considered moving from a very traditional IRCd or network such as IRCnet as that can remove the need of running a separate bot for the same thing.
There is also a privacy concern, I think bots mostly use their own user databases if they even check message recepients instead of just shouting their messages into channel upon seeing a matching nick, while the traditional MemoServ integrates with the network user database and sends the message in private with an option to email it to the recepient.


Design and Cleanup iteration

Renamed Packet, which is an overloaded term, to MlspData
Added Send to MlspPackage
Added Borrow and AsRef to Mlsp
Removed "get" from Mlsp
Added Doc Tests and documentation
Added a multithreaded example

Note that multi threaded examples are in no way proof that it works. There are simply too many moving parts to test.
Other than that it looks good.

I understand that, it's more of a demonstration that it's not completely broken and can sometimes work. I'm not sure how to go about proving that it's robust.

Ok, just making sure. You can ask on user forum or discord for feddback on proving that it is safe.

I'll merge this, add some issues regarding some gaps and then request feedback


[5.0] Included of most used variables on model's stub file.

They would need to have sensible default values.


BUG: Do SetTransformIsStackTransform(false) in BeforeEachResolution()

Whenever BeforeEachResolution() detects that there is no stack transform as current transform, let it set TransformIsStackTransform = false, just like it sets TransformIsStackTransform = true when the current transform is a stack transform.

@stefanklein Do you agree that this would be a useful commit? Or would it be unnecessary? (TransformIsStackTransform is now already initialized to false during the construction of all four metrics, with pull request #1163. So I wonder if it's useful to then reset this property with each BeforeEachResolution() call.)

@stefanklein Do you happen to know, why is this Boolean property (metric->m_TransformIsStackTransform) set before each resolution? Do you agree that it would make (more) sense to only just set it in BeforeRegistration()?

Hi Niels,
I think you are right. I don't see a reason why it couldn't be done in the BeforeRegistration function
Cheers! Stefan
From: Niels Dekker @.>
Sent: Friday, June 28, 2024 11:50 AM
To: SuperElastix/elastix @.>
Cc: Stefan Klein @.>; Mention @.>
Subject: Re: [SuperElastix/elastix] BUG: Do SetTransformIsStackTransform(false) in BeforeEachResolution() (PR #1167)
Waarschuwing: Deze e-mail is afkomstig van buiten de organisatie. Klik niet op links en open geen bijlagen, tenzij u de afzender herkent en weet dat de inhoud veilig is.
Caution: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe.
@stefankleinhttps://github.com/stefanklein Do you happen to know, why is this Boolean property (metric->m_TransformIsStackTransform) set before each resolution? Do you agree that it would make (more) sense to only just set it in BeforeRegistration()?

Reply to this email directly, view it on GitHubhttps://github.com/SuperElastix/elastix/pull/1167#issuecomment-2196531927, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAF2LNIANJISJRCKLS7PLVTZJUWTJAVCNFSM6AAAAABJ6CRTISVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOJWGUZTCOJSG4.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>


Translating - missing string

Hi, I'm translating the last strings into catalan and cannot find the string: Including beta versions
Thanks !

It's no linger there. Please have a look at the latest EN version.

I send you a updated file with the latest phrases translated.
Thanks !
[20170414] ca.UTF-8.txt

Thanks.

Translate this please:
theme_xhred_filemanager_link_to_clipboard=Symbolic link's name %value has been copied to clipboard.

Hi !
theme_xhred_filemanager_link_to_clipboard=El nom de l'enllaç %value s'ha copiat correctament al porta-retalls.

Thanks.


Code lists for commodities

Code list for commodities is located both in the (often hidden) codeslist tabs of google spreadsheet and on PoolParty (requires login)
https://docs.google.com/spreadsheets/d/16vuQCT92gYsMGGNGgX-j12-giCmO3SrUak6cG8oulMM/edit#gid=1609374283
Fine to only display the ones that already have value
and originates from annex 3 of this document:
http://www-wds.worldbank.org/external/default/WDSContentServer/WDSP/CTR/2016/02/25/090224b0841b02d1/2_0/Rendered/PDF/Options0for0Ex00better0and0the0best.pdf
Fine to validate list in spreadsheet and then only display those that have already a value in loaded database.

Also note that list might need further small fine tuning in future

Pool party wont really work to replace the actual commodity collection. I do hope that we can use it in place of the alias model we are currently rolling.
Ive got the stuff from the spreadsheet in there now. Names are strings and theres an id filed that is '-' instead of spaces. We can use this for routing. so the aluminium commodity page would be http://resourceprojects.org/commodity/aluminum.

that works well!

These are all loaded. moved routing to #16

pool party to #17