Change the Accordian Props Structure.

The current Implementation of Accordion takes props in the following way:
accordionList: [
    {
      labelBaseProps: {
        label: 'Accordion 1',
      },
      content: ExpandedText,
      contentProps: {
        text: text,
      },
      variant: 'dark',
    },
    {
      labelBaseProps: {
        label: 'Accordion 2',
      },
      content: ExpandedText,
      contentProps: {
        text: text,
      },
      variant: 'dark',
    },
    {
      labelBaseProps: {
        label: 'Accordion 3',
      },
      content: ExpandedText,
      contentProps: {
        text: text,
      },
      variant: 'dark',
      accordionSuffix: <LabelBase label='Suffix Text' />,
    },
  ],

Here Label props are of type string label?: string; but we need to pass components in it. For example an icon and a text.
This is being allowed, but VS code gives an error.
To resolve this, In LabelBaseProps we'll need to make label?: string; => label?: any;
Or look for other solutions.
Also, the Above payload looks complex and I am unable to understand the purpose of content and content props intuitively. Maybe we should relook into that too.

@ShubhamJoshi-1994
    <label
      htmlFor={props.htmlFor}
      className={finalClassNames}
      style={props.labelStyles}
    >
      {props.label}
    </label>

If props.label is going to be a component, not just a "string", is it actually a label or a button?
But, we can make label?: any not an issue.

@ShubhamJoshi-1994
label has been changed to ReactNode which can accept string or a component, and this is better way than just using any

May be you can use something like this if your problem is not solved?
Else can this issue be closed?


[Communication] - Auth - Using BearerTokenCredentialPolicy instead of CommunicationAuthenticationPolicy

POC to remove the policy that is duplicated by BearerTokenCredentialPolicy

why this PR includes empty files like sdk/communication/AzureCommunicationChat/Tests/Util/Recordings/sendTypingNotification.json

why this PR includes empty files like sdk/communication/AzureCommunicationChat/Tests/Util/Recordings/sendTypingNotification.json

Looks like there was an issue with my rebase. I will open up another PR for this

Looks like there was an issue with my rebase. I will open up another PR for this


http -> https

Description
Updated some links from http to https.
Notably, I also fixed a broken link (http://developer.apple.com/library/mac/#qa/qa2004/qa1361.html).

Thanks!


Add manager_email and OMIS flag to countries

This:

adds manager_email to Country. This is the email address of the overseas manager overseeing that market.
adds omis_disabled_on to Country. This is None if the market offers OMIS, manages legacy records and makes sure that a market can always be disabled in the future.
omis_disabled_on is exposed via API but not manager_email as the endpoint does not require authentication and that piece of info is not really needed.
the fixtures have been updated to include the correct value for omis_disabled_on. All manager_emails are currently empty.

Codecov Report

Merging #406 into develop will increase coverage by 0.01%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop     #406      +/-   ##
===========================================
+ Coverage    95.94%   95.95%   +0.01%
===========================================
  Files           79       79
  Lines         2835     2846      +11
  Branches       244      245       +1
===========================================
+ Hits          2720     2731      +11
  Misses          81       81
  Partials        34       34

Impacted Files
Coverage Δ

datahub/metadata/metadata.py
100% <100%> (ø)
:arrow_up:

datahub/metadata/admin.py
100% <100%> (ø)
:arrow_up:

datahub/metadata/serializers.py
100% <100%> (ø)
:arrow_up:

datahub/company/serializers.py
97.9% <0%> (ø)
:arrow_up:

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 36d717b...b603ac1. Read the comment docs.

Closing as we decided to use a different approach instead (see #410)


abnormal chars appears in some places in the Powertoys interface

Microsoft PowerToys version
0.62.1
Installation method
PowerToys auto-update
Running as admin
No response
Area(s) with issue?
Always on Top, TextExtractor
Steps to reproduce
Open the Powertoys and check the related pages
✔ Expected Behavior
Shoud not have the abnormal chars
❌ Actual Behavior
Now the chars is very strange.

Other Software
No response

Just found v0.64 has fixed this issue.

Thanks for the updated!


All possible subset problem using backtracking| c++

All possible subset Problem using backtracking in C++ -
Adding code to implement N Queens problem as a classic back tracking example -
Type of Issue -
Please add/delete options that are not relevant.
Adding New Code
Programming Language
Please add/delete options that are not relevant.
C++
Self Check
Ask for issue assignment before making Pull Request.
Add your file in the proper folder
Clean Code and Documentation for better readability
Add Title and Description of the program in the file
Happy Coding,

Please assign it to me.


Typo "Jupyter notebook"→"Jupyter Notebook"

https://docs.microsoft.com/en-us/azure/machine-learning/how-to-machine-learning-fairness-aml

@hyoshioka0128 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@hyoshioka0128 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Hi @hyoshioka0128
Our internal guidance for this terminology has been clarified:

Jupyter Notebook: Open-source web application to create and share documents that contain code, equations, visualizations and text. Two words, title case
Jupyter notebook, notebook: A file or “output” of the Jupyter Notebook app. “notebook” is lowercase
JupyterLab: Web-based development environment for Jupyter notebooks, code, and data. One word, camel case

In this instance, because the brand name "Jupyter Notebook" hasn't been used yet in the article, I'll revise the updated sentence to Train a sample model in Jupyter Notebook. But the original is otherwise correct and would not be updated. Thanks!

Hi @hyoshioka0128
Our internal guidance for this terminology has been clarified:

Jupyter Notebook: Open-source web application to create and share documents that contain code, equations, visualizations and text. Two words, title case
Jupyter notebook, notebook: A file or “output” of the Jupyter Notebook app. “notebook” is lowercase
JupyterLab: Web-based development environment for Jupyter notebooks, code, and data. One word, camel case

In this instance, because the brand name "Jupyter Notebook" hasn't been used yet in the article, I'll revise the updated sentence to Train a sample model in Jupyter Notebook. But the original is otherwise correct and would not be updated. Thanks!


New-branding update 22-08

Targeting issues #505 #506 #507 #508 and #511

Updated responsive behavior of the right sidebar
Left sidebar is collapsable
Tuned up the code snippet colors to be less contrast

Merging for testing.


raw_output return 0

I tried the latest version of this Actions.
Here is my GitHub Action yaml:
name: "~Sandbox"

on:
  workflow_dispatch:

jobs:
  sandbox:
    runs-on: ubuntu-latest
    name: List the display name and shape of the instances in my compartment
    env:
      OCI_CLI_USER: ${{ secrets.OCI_CLI_USER }}
      OCI_CLI_TENANCY: ${{ secrets.OCI_CLI_TENANCY }}
      OCI_CLI_FINGERPRINT: ${{ secrets.OCI_CLI_FINGERPRINT }}
      OCI_CLI_KEY_CONTENT: ${{ secrets.OCI_CLI_KEY_CONTENT }}
      OCI_CLI_REGION: ${{ secrets.OCI_CLI_REGION }}
    steps:
      - name: Retrieve the OCID of a named compartment in tenancy
        uses<EMAIL_ADDRESS>        id: find-compartment-id
        with:
          command: 'iam compartment list --compartment-id-in-subtree=true'
          query: "data[?name=='jax'].id"
          silent: false
      - name: List the display name and shape of the instances in my compartment
        run: |
          echo output=${{ steps.find-compartment-id.outputs.output }}
          echo raw_output=${{ steps.find-compartment-id.outputs.raw_output }}

For some reason raw_output is 0:
1. Run<EMAIL_ADDRESS>Installing Oracle Cloud Infrastructure CLI
Executing Oracle Cloud Infrastructure CLI command
/home/runner/.local/bin/oci --query data[?name=='jax'].id iam compartment list --compartment-id-in-subtree=true
[
  <reasonable_output>
]

2. Run second step
  shell: /usr/bin/bash -e {0}
  env:
    OCI_CLI_USER: ***
    OCI_CLI_TENANCY: ***
    OCI_CLI_FINGERPRINT: ***
    OCI_CLI_KEY_CONTENT: ***
    OCI_CLI_REGION: ***
output=<reasonable_output>
raw_output=0

Is there more than one result? If so, this is to be expected. The raw_output is only available if there is a single result from the command.

Is there more than one result? If so, this is to be expected. The raw_output is only available if there is a single result from the command.

It is only one result:
/home/runner/.local/bin/oci --query data[?name=='jax'].id iam compartment list --compartment-id-in-subtree=true
[
  "<one result only>"
]


Enabled processing for TexturePreview on AnimatedTexture

Fixes #77415
Problem
The issue was, that the TexturePreview would only get redrawn when the redraw was issued from another place (like the viewport calling queue_redraw().
My solution
In the contructor of TexturePreview, I check if the "previewed" Texture is an AnimatedTexture, and if so, I set_process(true) and then handle the NOTIFICATION_PROCESS by queuing a redraw there.
Here are two videos showing the end result (second one is to show, that the preview updates even when the mouse isnt moving).
https://github.com/godotengine/godot/assets/75789249/a006bcda-5bd7-4c6e-99a9-7df28730fcd7
https://github.com/godotengine/godot/assets/75789249/84197618-f184-4025-a5d4-a8c6119d6e0e

Closing because this seems to be a deeper issue. Not only the TexturePreview is not redrawing, but also a normal Sprite2D with an AnimatedTexture in the Viewport.


NSPC: Add warning if namespace does not exist

Part 1 of #835
If a repo contains objects in a namespace and the namespace does not exist in the local system, then a warning is shown pointing to SE03.
Test https://github.com/mbtools/TEST_MISSING_NSPC

checks split into separate methods

checks split into separate methods


Can save Excel

SQL Operations Studio Version: sqlops 0.26.7 (723c7cb3bfd80d87fc8a1141632089cd4808495c, 2018-02-16T21:34:47.034Z)
OS Version: Windows_NT x64 6.1.7601
Extensions: none

Steps to Reproduce:

Make a SQL with Select
Run
press Save as Excel
No error, but I haven't excel file

It only happens when I use Work Area. Without working area it is working.
Sorry

@joni4444 are you unblocked with this scenario?  Or is there still something you'd like us to look at?

No, I can work without solve it. Don't worry. Thanks to your reply

De: Karl Burtram<EMAIL_ADDRESS>Enviado: jueves, 8 de marzo de 2018 23:58:46
Para: Microsoft/sqlopsstudio
Cc: joni4444; Mention
Asunto: Re: [Microsoft/sqlopsstudio] Can save Excel (#839)
@joni4444https://github.com/joni4444 are you unblocked with this scenario? Or is there still something you'd like us to look at?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/sqlopsstudio/issues/839#issuecomment-371666756, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AStNIXEExIjHkelnU3CwSTKavBLqU1L7ks5tccW2gaJpZM4ScHlr.


[SPARK-36846][PYTHON] Inline most of type hint files under pyspark/sql/pandas folder

What changes were proposed in this pull request?
Inlines type hint files under pyspark/sql/pandas folder, except for pyspark/sql/pandas/functions.pyi and files under pyspark/sql/pandas/_typing.

Since the file contains a lot of overloads, we should revisit and manage it separately.
We can't inline files under pyspark/sql/pandas/_typing because it includes new syntax for type hints.

Why are the changes needed?
Currently there are type hint stub files (*.pyi) to show the expected types for functions, but we can also take advantage of static type checking within the functions by inlining the type hints.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Existing tests.

Test build #143613 has started for PR 34101 at commit 0a43396.

Test build #143613 has finished for PR 34101 at commit 0a43396.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/48125/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/48125/

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

LGTM, thanks!

Merged to master.


Camera not connecting after update

Description
Lost connection after the update.
I have tried reconfiguring. I can able to stream rtsp in the VLC player. But HA integration says below error.
"Invalid authentication data. Make sure you have created your 3rd party account via Tapo app. You can also test if these credentials work via rtsp stream, for example VLC using link rtsp://username:password@IP Address:554/stream1"

Reproduction Steps

Added Tapo camera integration via HACS
Tried configuring it in HA
Provided the ip address
It asked for credentials
But credentials did not work (but same credentials worked in VLC)

Expected behavior
Lost connection after the update.
I have tried reconfiguring. I can able to stream rtsp in the VLC player. But HA integration says below error.
"Invalid authentication data. Make sure you have created your 3rd party account via Tapo app. You can also test if these credentials work via rtsp stream, for example VLC using link rtsp://username:password@IP Address:554/stream1"

If applicable, add error logs.
No response
Device Firmware
latest
Using stream component
Yes
Does camera work via official integrations?
No
Camera has all attributes filled out in developer tools
Yes
HASS Environment
Laptop hard disk
Search for similar issues
Yes
Additional information
No response

What was the solution for this problem?

Update was not an issue. Restart camera, restart hass, make sure camera is accessible and you are entering correct information etc.

I am able to connect the camera after restarting the camera and reinstalling Tapo camera in HA


Error with Build Appx Package

Overview
When building Appx package with build window, get the error: Win32Exception. No change with Unity Editor and VS, could work well last day but raise this error when trying to build now.
Expected Behavior
get the Appx package
Actual Behavior
Win32Exception: ApplicationName='cmd.exe', CommandLine='/C vswhere -version 15.0 -products * -requires Microsoft.Component.MSBuild -property installationPath', CurrentDirectory='C:\Program Files (x86)\Microsoft Visual Studio\Installer', Native error= The system cannot find the file specified.
Steps to reproduce
(Links to sample github project preferred)

Unity Editor Version
Unity -- 2017.3.1f1
VS -- 15.6.2
Mixed Reality Toolkit Release Version
2<IP_ADDRESS> Hot Fix

Add your build settings in the repro steps please.

Chances are you don't have your dev environment setup correctly.

have correctly setup dev environment, and could build two days ago, maybe it has some relation with my windows insider update, my system version is 1803

Nah, I've got that insider preview installed as well. No issue here.

That looks like a VS error from the error output, are you able to build other projects in it? Maybe your VS installation is a bit screwed and needs a clean install.


Add source_guild and source_channel to Webhook

Summary
Add source_guild and source_channel to Webhook
Checklist

[x] I have run nox and all the pipelines have passed.
[x] I have made unittests according to the code I have added/modified/deleted.

Related issues

https://github.com/discord/discord-api-docs/pull/1837


wildcard can have different parameter name based on the handler

Hi,
I've been dealing with a case where I'd want to have different parameter name for the wildcard, depending on the handler.
Right now, it's impossible to do:
/get/:doc
/get/:user/:doc
This patch aims to make this possible, without adding any complexity when getting the route.

It's against the design and makes it less efficient.

In my implementation, the tree produced by
/get/:doc/
/get/:user/:doc

is the exactly same as:
/get/:user/
/get/:user/:doc

Inside getValue, the only extra work is go inside a for loop and loop (nbrElement) times. I'm not sure this is going to have any impacts on the performances.
The only downside I see is a slightly bigger memory footprint, as the parameter name are stored inside an array for each node that have handlers.

I appreciate your pull request (and that you are brave enough to dig into the tree code), but currently
I am very reluctant to add anything that bloats the nodes further. Also I actually somehow intended this limitation to enforce sensible API design.
Please don't delete the fork / branch, I will reconsider it for v2.

Thank you for answering / taking a second look!
The current limitation is a bit frustrating when implementing an api like:
/get/view
/get/doc/rev

We can either do:
/get/doc
/get/doc/rev

or
/get/:p1
/get/:p1/:p2

Neither solution feel quite right, in the first, we would have  view_id := router.Get("doc") in our code, and in the second solution, technically the most correct, would make the code really difficult.
Not to mention we are generating an API documentation based on the same route we are passing to this router.


add Test_imageToCriImage

Ⅰ. Describe what this PR did
fixes #1914
Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how you did it
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews
gourp 7

 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.


Proxy collisions with Database class instances

mongoist version: 1.7.2 but I believe this is problem affects older versions.
The problem is that the Database object instance created with mongoist has quite a few instance properties whose name can be somewhat commonly used as collection names, this causes the proxy functionality to break. For example:
this.features - means you cannot reference to a features collection in your database via Proxy.
Besides this one, options, client, connection and domain (this one introduced by extending EventEmitter can probably be considered very plausible collection names.

@saintedlama would you take a PR to fix this? This is causing bugs in our codebase.

@bradvogel would appreciate a PR for this issue. Thought about circumventing that problem by using prefix $ for private fields (See https://docs.mongodb.com/manual/reference/limits/#naming-restrictions)


Merge Intake/Shooter implementation to main

After testing on the target robot,
Merge feature branch to main

Fixed before BSU Weekend


Bug: Lualine does not load after recent updates

Self Checks

[x] I'm using the latest lualine and neovim nightly.
[x] I didn't find the issue in exsisting issues or prs.

How to reproduce the problem
Loading the plugin with these configs on the latest commit does not work, as in lualine just doesn't load.
require('lualine').setup{
  theme = 'gruvbox',
  extensions = { 'fzf', 'fugitive' },
}

Everything works fine however if I rollback to this commit https://github.com/hoob3rt/lualine.nvim/commit/6ba2b80b594c3ead11ab9bd1dbc94c0b4ea46c33
Actual behaviour
Lualine doesn't load.
Expected behaviour
Lualine loads :)
Minimal config to reproduce the issue
I don't have one but let me know if you can't reproduce and I can try and provide one.

Config

Lualine loads but you don't have any components in any of the sections so it's not displaying anything . It's a intended behavior. Previously the components defined in user config would extend the default . That causes conflicts and extra boilarplate when you want to omit some  sections. So the behavior was changed in f08e8514ddbe73cb02644cbd7b4b98e3681a7f9d .Now default sections don't get applied if you provide configuration (by passing a table to setup function or by g:lualine) . In short require'lualine'.setup() would load the default sections but require'lualine'.setup({}) won't . If you want the default components just copy paste the default config . If you think your issue is resolved then close the issue :)

i suppose if you want the defaults to be managed for you you could write
sections = require'lualine.defaults'.sections,
inactive_sections = require'lualine.defaults'.inactive_sections

for as long as that works :d

Might I suggest that README.md be updated to reflect this change?

I have also been trying to figure this out for a bit. This is what I have following @shadmansaleh's advice of copying in the default config.
  use { 'hoob3rt/lualine.nvim',
        config = function()
          require('lualine').setup {
            options = {
              theme = 'auto',
              icons_enabled = false,
              section_separators = '',
              component_separators = '',
              disabled_filetypes = {}
            },
            sections = {
              lualine_a = {'mode'},
              lualine_b = {'branch'},
              lualine_c = {'filename'},
              lualine_x = {'encoding', 'fileformat', 'filetype'},
              lualine_y = {'progress'},
              lualine_z = {'location'}
            },
            inactive_sections = {
              lualine_a = {},
              lualine_b = {},
              lualine_c = {'filename'},
              lualine_x = {'location'},
              lualine_y = {},
              lualine_z = {}
            },
            tabline = {},
            extensions = {}
          }
        end
  }

However, lualine still does not load. The only custom configuration I really want is no separator icons. Am I handling the table wrong? Thank you

@michzappa your config works fine for me. have you run :PackerCompile?

Wow, I have done that before but completely neglected doing so now. Thank you for reminding me.

i suppose if you want the section defaults to be managed for you you could write

sections = require'lualine.defaults'.sections,
inactive_sections = require'lualine.defaults'.inactive_sections
for as long as that works :d

That will work for now but I won't recommend it as it's a internal file and can be moved/removed/changed without notice . That will break the config :)

Might I suggest that README.md be updated to reflect this change?

Yes you might . But It's not a advertised use case in my opinion . There's actually config example in both lua and vimL so users can copy them and make changes as they like , The defaults in defaults.lua is there to make it easy to try lualine for the first time I think . Can you suggest where that should go in README ? Since it was never recomanded to rely on defaults . We could probably have a section in README listing breaking behavioral changes like packer.

Wow, I have done that before but completely neglected doing so now. Thank you for reminding me

Yeah that gets everyone time to time :D
At least wbthomason working on making PackerCompile unnecessary .

Since it was never recommended to rely on defaults

I think this conflicts with the way most people interact with configuring vim plugins. Most are probably familiar with the workflow of incrementally changing only the options which are relevant to them. This way changes to the plugin affect users minimally, and there's little cognitive overhead. It might be unexpected that changing 1 or few options in setup changes the defaults for every option. Just my 2 cents

I think this conflicts with the way most people interact with configuring vim plugins.
Most are probably familiar with the workflow of incrementally changing only the options which are relevant to them. This way changes to the plugin affect users minimally, and there's little cognitive overhead.

This makes me think I use plugins weirdly :D .As I always  copy paste configs and tweak them later . But I can see your point .
To be honest my main annoyence was having to do this to disable inactive status
inactive_sections = {
              lualine_a = {},
              lualine_b = {},
              lualine_c = {},
              lualine_x = {},
              lualine_y = {},
              lualine_z = {}
            },

And this issue with evil_lualine got my attention :)

i think i've seen a pattern like x.enable_defaults = false in the wild

At least would be nice to have smth like lsp have for customizing handlers -> vim.lsp.with s o you can edit single option without copy&paste all defaults which IMHO is bad practice because if you someday touch defaults - none will figure out or it may break.
so smth like: lualine.setup(lualine.defaults({sections = { lualine_c = {'xxx'}))
Or somehow make defaults properly accessible from code and not from readme.
Thanks for plugin btw, really enjoying so far!

I think I'm pretty convinced to revert the changes :]

Now you guys can go back to enjoying default config like before :D


Tests for CASSANDRA-7423

Thanks for explaining the change and adding a new method. Could you make this a parameterized test to remove the duplication here? Other than that, looks good.

Looks like this has fallen out of sync with master and has to be rebased, fyi.

Once it's been rebased, should we hold merge on CASSANDRA-7423?

Right, these shouldn't pass without 7423 being committed

I think this breaks non trunk branches. missing a @since() tag?

The first test has a gate inside the test based on version, but it does appear to be failing still. We'll check it out

I found the problem. I'll open a PR


Target root is empty by default on batch import

First, thanks for this amazing plugin.
Problem
When I do a Batch Drawable Import, the target root is empty and I need to set it for each files.
Demo

Is it as expected? Is there something I can do to fix it?

This is a duplicate of #135


Working manual scrolling credits menu

I made the credits menu with animated texts and the manual scrolling along with the back button on the bottom of the scroll container.

What do you by manual scrolling?

I couldn’t get auto scrolling to work properly just yet so for now, you can manually scroll through the credits until I get the auto scroll script working

I will approve the the pull request once you the auto scrolling to work


spell errors: misusing a and an

Signed-off-by: tim-zju<EMAIL_ADDRESS>
Proposed changes

Unreleased project version (optional)

Related issues (optional)

Thanks for contributing 🐳


explicit user configuration is not loaded in 20.0.5

Current behavior
Since version 20.0.5 no explicit user configuration is loaded from package.json is loaded.
When I place the configuration in  .cypress-cucumber-preprocessorrc.json within the cypress directory it works.
Debug Log with Version 20.0.5:
cypress-cucumber-preprocessor resolved no explicit user configuration +1ms
Debug Log with Version 20.0.4
cypress-cucumber-preprocessor resolved explicit user configuration { stepDefinitions: [ './integration/common/*.ts' ] } +2ms

Desired behavior
User configuration in package.json is recognized.
Test code to reproduce
Structure of my project

project

package.json
projects/

project-e2e/

integration/

common/

Common Step Definitions.ts

My configuration in package.json:
  "cypress-cucumber-preprocessor": {
    "nonGlobalStepDefinitions": true,
    "stepDefinitions": [
      "./integration/common/*.ts"
    ]
  },

Run DEBUG=cypress:electron,cypress-configuration,cypress-cucumber-preprocessor cypress open, the explicit user configuration is not loaded.
Versions

Cypress version: 13.6.6
Preprocessor version: 20.0.5
Node version: 20.11.1

Checklist

[x] I've read the FAQ.
[x] I've read instructions for logging issues.
[x] I'm not using<EMAIL_ADDRESS>(package name has changed and it is no longer the most recent version, see #689).

Works seemingly fine as far as I can see. You need to provide a minimal, reproducible example.

I'm experiencing a similar issue (common step definitions not found), although our custom configuration is placed in cypress-cucumber-preprocessor.config.js, and not in package.json 🙂. Downgrading to 20.0.4 fixes it.

Here's an example illustrating configuration using package.json and here's a similar example using cypress-cucumber-preprocessor.config.js. They both work, so I'm guessing you're doing something that you're not telling me about.
The purpose of a reproducible example is that it's unambiguous and you need to provide it.

Hello,
I think this issue is linked to cosmicconfig package version 8.3.6 to 9.0.0 and the structure of your project.
searchStrategy is none by default and your package.json with configuration is not in the same directory(root).
Can you try creating a package.json file located in project-e2e with your configuration :
 "cypress-cucumber-preprocessor": {     "nonGlobalStepDefinitions": true,     "stepDefinitions": [       "./integration/common/*.ts"     ]   }
More details here : https://github.com/cosmiconfig/cosmiconfig/blob/main/CHANGELOG.md#900

@badeball based on what @florianbnd said, are you planning on adding the option to set searchStrategy to project?
Creating separate package.json files for all my e2e projects feels like a lot off overkill right now.
Thank you!

@florianbnd, I'm aware of the changelog, but I don't make any changes based on a hunch. Instead, I'd really like for someone to just provide some information indicating that that is indeed the issue being observed. @warri93 has done so now, hence the change. This has been added with v20.0.6.


Add search by labels.

@dastultz  Could you try to add some integration tests please ?
See RealSmokeTest.java
Just tell us what to configure in our sandbox to have them passing

Sorry for the delay, I plan to do this, please don't close the issue. Thanks.

Added unit tests. I did not run them! Documented article contents needed in sandbox.


Create specific data array

All data treatments I have ever worked with is done by exporting/treating a single array of data and then conduct specific analysis (dI/dV or Int I-I0 dV). And array function with interp MXX data for specific conditions, time_span, Ns, cycle number would be really helpful for user doing specialised analysis this function only handles snippets of an experiments and thus it should not be an issue mem.-wise).

Hey Kim,
If I understand right here, the question is how to get an array of data out of an ixdat object?
There's a few ways.
The canonical way is to use Measurement.grab(). This will give you two arrays - the second is the value that you requested, the first is time. Check the function's documentation - you can do a few things with it including selecting a specific timespan.
For example:
from ixdat import Measurement

meas = Measurement.read("my_ec_data.mpt", reader="biologic")
t, v = meas.grab("potential", tspan=[100, 200])

from matplotlib import pyplot as plt
fig, ax = plt.subplots()
ax.plot(t, v)          #  plots potential vs time in the time range 100 s to 200 s

If you already know the time vector you want to use, you can use Measurement.grab_for_t() and get a single vector out.
For example,  continuing from above:
j = meas.grab_for_t("current", t=t)
dQ = np.trapz(j, t)     # integrates the current vector vs the time vector.

The most direct way to access a vector with the exact raw data that was in the file is by indexing a measurement to get a DataSeries  and taking the data attribute (the t attribute is the corresponding time):
raw_I = meas["I/mA"].data     # a single numpy array with the raw current data
raw_t = meas["I/mA"].t           # the corresponding time data

You can check which raw data series are available as arrays by any of the above methods with the value_names attribute:
print(meas.value_names)

This list is not exclusive, as you can also grab calibrated or otherwise derived data series.
If you want to select according to a cycle number, you can use the select function to make a smaller measurement, and then get vectors by any of the measurements above:
meas_subset = meas.select(Ns=1)    # selects the data where Ns=1
t, dQ = meas.grab("(Q-Q0)/C")         # grabs the cumulative charge passed, and corresponding time as numpy vectors

An especially elegant way to select according to cycles is to convert the measurement first to a CyclicVoltammogram. Then you can index it with an integer to select a particular cycle.
cv = meas.as_cv()
cycle_1 = cv[1]       # selects the first cycle
next_cycles = cv[2:5]    # selects the next five cycles
t, v = next_cycles.grab("potential")
j = next_cycles.grab_for_t("current", j)
fig, ax = plt.subplots()
ax.plot(v, j)          # plot of J vs V for cycles 2 to 5

ixdat is especially nice with cyclic voltammograbs because it will let you redefine the cycle to select exactly what you want. This is done with the CyclicVoltammogram.redefine_cycle() function:
cv.plot_measurement(j_name="cycle")     # a plot to see where the cycle presently increments
#  ^ note that j_name is called J_str in ixdat v0.1.x
cv.redefine_cycle(start_potential=0.5, redox=True)    # redefines the `cycle` variable to increment at 0.5 V in anodic direction
cv[2].plot()    # plots J vs V for the second cycle as defined above. You can of course grab data, etc from cv[2]

Note that this (and everything here) also works for EC-MS measurements, regardless of whether the EC and MS data were originally read from one file or two file. I like to use the EC program to select cycles, etc, and MS data comes along for the ride.
Finally, you mentioned exporting. You can at any time export a Measurement as a .csv, and specify the column names to export:
meas.export("meas_100s_to_200s.csv", tspan=[100, 200])
cv[3:6].export("cycles_3_to_6.csv", colums=["potential", "current", "(Q-Q0)/C", "cycle"])
# note, in ixdat v0.1.x, columns is instead called v_list.

In the first export, columns for time, potential, and current are included by default. In the second, the desired columns are specified and ixdat automatically makes sure all needed time variables are included.
Keep an eye out for ixdat v0.2.0, as there will be a few minor changes.
I hope that helps! Let me know if there's anything in your question not covered. (If I got everything, please resolve this issue.)

@KimDegnJensen , does this answer you question?

Hi Søren good to hear from you, I think this is exactly what i wanted to
do, I was actually told that my use of grab was ill advised. But now that i
understand a bit more concerning the attributes of the different functions
it all makes a bit more sense, thanks. NB i have a question regarding some
of your old thesis data that we wanted to include in a tech sheet, I would
would prefer to mail it to directly and not through github. could i ask you
to mail me @.*** It concerns soem old CVs of Cu, I believe
these CVs were never published in peer reviewd (i would have expected them
to be in yours and Eijlers paper) and possibly only appears in your thesis.
Hence, i would like to ask if we could use the plots in a tech. sheet,
which ill again prefer to mail to you ensuring that we dont use soemthing
we shouldnt:-)
KR Kim
On Mon, Mar 14, 2022 at 11:46 PM Søren Bertelsen Scott <
@.***> wrote:

Hey Kim,
If I understand right here, the question is how to get an array of data
out of an ixdat object?
There's a few ways.
The canonical way is to use Measurement.grab(). This will give you two
arrays - the second is the value that you requested, the first is time.
Check the function's documentation - you can do a few things with it
including selecting a specific timespan.
For example:
from ixdat import Measurement
meas = Measurement.read("my_ec_data.mpt", reader="biologic")
t, v = meas.grab("potential", tspan=[100, 200])
from matplotlib import pyplot as plt
fig, ax = plt.subplots()
ax.plot(t, v)          #  plots potential vs time in the time range 100 s to 200 s
If you already know the time vector you want to use, you can use
Measurement.grab_for_t() and get a single vector out.
For example, continuing from above:
j = meas.grab_for_t("current", t=t)
dQ = np.trapz(j, t)     # integrates the current vector vs the time vector.
The most direct way to access a vector with the exact raw data that was in
the file is by indexing a measurement to get a DataSeries and taking the
data attribute (the t attribute is the corresponding time):
raw_I = meas["I/mA"].data     # a single numpy array with the raw current data
raw_t = meas["I/mA"].t           # the corresponding time data
You can check which raw data series are available as arrays by any of the
above methods with the value_names attribute:
print(meas.value_names)
This list is not exclusive, as you can also grab calibrated or otherwise
derived data series.
If you want to select according to a cycle number, you can use the select
function to make a smaller measurement, and then get vectors by any of the
measurements above:
meas_subset = meas.select(Ns=1)    # selects the data where Ns=1
t, dQ = meas.grab("(Q-Q0)/C")         # grabs the cumulative charge passed, and corresponding time as numpy vectors
An especially elegant way to select according to cycles is to convert the
measurement first to a CyclicVoltammogram. Then you can index it with an
integer to select a particular cycle.
cv = meas.as_cv()
cycle_1 = cv[1]       # selects the first cycle
next_cycles = cv[2:5]    # selects the next five cycles
t, v = next_cycles.grab("potential")
j = next_cycles.grab_for_t("current", j)
fig, ax = plt.subplots()
ax.plot(v, j)          # plot of J vs V for cycles 2 to 5
ixdat is especially nice with cyclic voltammograbs because it will let
you redefine the cycle to select exactly what you want. This is done with
the CyclicVoltammogram.redefine_cycle() function:
cv.plot_measurement(j_name="cycle")     # a plot to see where the cycle presently increments
^ note that j_name is called J_str in ixdat v0.1.x
cv.redefine_cycle(start_potential=0.5, redox=True)    # redefines the cycle variable to increment at 0.5 V in anodic direction
cv[2].plot()    # plots J vs V for the second cycle as defined above. You can of course grab data, etc from cv[2]
Note that this (and everything here) also works for EC-MS measurements,
regardless of whether the EC and MS data were originally read from one file
or two file. I like to use the EC program to select cycles, etc, and MS
data comes along for the ride.
Finally, you mentioned exporting. You can at any time export a Measurement
as a .csv, and specify the column names to export:
meas.export("meas_100s_to_200s.csv", tspan=[100, 200])
cv[3:6].export("cycles_3_to_6.csv", colums=["potential", "current", "(Q-Q0)/C", "cycle"])
note, in ixdat v0.1.x, columns is instead called v_list.
In the first export, columns for time, potential, and current are included
by default. In the second, the desired columns are specified and ixdat
automatically makes sure all needed time variables are included.
Keep an eye out for ixdat v0.2.0, as there will be a few minor changes.
I hope that helps! Let me know if there's anything in your question not
covered. (If I got everything, please resolve this issue.)
—
Reply to this email directly, view it on GitHub
https://github.com/ixdat/ixdat/issues/37#issuecomment-1067375628, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AXR3S544SR4J6WZNANASNY3U766THANCNFSM5QMJL7BA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

Glad that answered your question. Yeah, grab is the cannonical way to get numpy arrays out of ixdat Measurements.
Closing this and replying to you by email :)


Fix pydantic tests again

I don't know why the last two commits in https://github.com/pydantic/logfire/commits/pydantic-main-again/ didn't make it into https://github.com/pydantic/logfire/pull/572

thanks @alexmojaki for moving this across the line, have a good evening!


Bump Dataflow worker container to beam-master-20181018

This change upgrades the Dataflow worker container.

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

It will help us expedite review of your Pull Request if you tag someone (e.g. @username) to look at it.
Post-Commit Tests Status (on master branch)

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

---
---
---
---
---
---

Java

Python

---

---
---
---

R: @udim @kennknowles
CC: @tvalentyn @aaltay

Run Python PostCommit

LGTM when green

Thanks, merging now as tests pass.


clicking on the logo should take you to the home page

I almost thought this would be easy... cc @Wilto

fixed by d040984


Go 1.15: conversion from OAuth2Provider (int) to string yields a string of one rune

New vet check causes multiservice/providers.go to fail to build:
./providers.go:46:50: conversion from OAuth2Provider (int) to string yields a string of one rune, not a string of digits (did you mean fmt.Sprint(x)?)
See:

Travis CI: https://travis-ci.org/github/grokify/oauth2more/builds/727681688

See also:

https://github.com/lightningnetwork/lnd/pull/4554
https://github.com/vmware/govmomi/issues/2075
https://github.com/prometheus/prometheus/issues/7706
https://issues.apache.org/jira/browse/THRIFT-5270

Fixed in 6189047fcffd9252c41a9ff0b83cff6ba53ebbc8


Socket connection throws "EPIPE" error

I get this error every time I start the client "The local end has been shut down on a connection oriented socket"
I already tried using different node versions, twitch-irc versions, restarting my network but no luck at all. I don't know what else to do.
This is all the messages printed in my console

I'm not sure if this error is cause of my server or there is a problem with the client.
Any help is very appreciated. Thanks in advance

Hello,
Please include your OS, Node.JS and module versions. Also, try adding this listener to your application too see if it fixes your issue:
process.stdout.on('error', function() {
    //
});

Hey Schmoopiie,
I forgot to mention that I only get this error on my production server, when I use it in my local, everything works just fine.
My production server specifications:

OS: Ubuntu 14.04.2 LTS (GNU/Linux 3.13.0-53-generic x86_64)

Kernel: PV-GRUB 64 --- I changed it from Cloud Kernel to this, because the guys who host my server suggested it, they thought it could fix the problem but nope :(

Firewall disabled and no iptables

It is a self managed server not shared.

Hosted by www.hostvirtual.com

node.js version: 0.12.3

twitch-irc version: 1.2.7
( I tried with nodejs 12.2 and twitch 1.2.6 just like my local server is)

I just tried adding that listener but nothing changed.
If you need some more information just ask.
Thanks for your time bro!

Thank you, I will get in touch with your provider and try to replicate your issue. It might also be a Node.JS issue, so I will get someone from the team too.

Thanks! I really appreciate your support, if there is anything that I can help, dont hesitate to contact me

Hello,
I just discovered something interesting, while on my local server (which never throws that error) I made the bot to talk into the channel (with say method) every 2 seconds, and after 30 messages more or less, the error was thrown "error  - The local end has been shut down on a connection oriented socket."
I did 3 times just to confirm and after that I tried increasing the timer to 15 seconds, never got the error again.

Strange, and I did test with hostvirtual and did not get this error, however I made some changes to the module that could fix your issue, if you can, give it a try and tell me if it is still happening. However, I'm not sure about the error you had when sending a message every 2 seconds.

Ohh, I see, hehehe ... Thanks for your interest

Hey there,
Good news! I just fix this error (I believe) I updated my nodejs version to 0.12.4 and specified a prefferedPort (443) and prefferedServer (<IP_ADDRESS>) in the client options
After doing this I haven't had this error again, my client has been live for like 4 hours. I hope it doesn't break tomorrow
Cheers

Hi again,
It's crashing every 5-6 hours, couldn't see the error message but, I just installed PM2 in order to log messages when node crashes, so I'm waiting for it to crash and see the error
I'll keep you posted

I could log the error and it was caused by my application code, so everything is fine now
The question I ask is, why setting a prefferedServer fixed it ?

Could be because Twitch is still working on some servers.. irc.twitch.tv returns some WebSocket servers which are not compatible with the module and it cause the application to crash. I have been working on the module for a few days to make it compatible with WebSockets. The good thing about it is that the module will work on node.js / io.js and as a standalone javascript (as long as the browser supports WebSockets). I'm doing my best to keep most of the commands / events the same so the developpers using this module won't have to change too much things. I forgot to mention that it does not use "pipe", which might fix your issue as well.
I don't have an ETA but I'm on vacation next week, so I will put a lot of my free time on it :smile:


bolt died because the read_tuple() has TypeError: 'int' object has no attribute 'getitem'

I have a bolt A that continuously emits a namedtuple that contains a list that could have thousands of ids in it.
Fields = namedtuple("Fields", "table action ids"), ids is a list that contains thousands of items in it.
The bolt B in the downstream have the error below. My guess is that the namedtuple is overflowing the sys.stdin. Is this possible? What I should do in this case ?
  4 topology:
  5     - spout:
  6         name: spoutA
  7         module: folder.spoutA
  8
  9     - bolt:
 10         name: boltA
 11         module: folder.boltA
 12         groupings:
 13             - shuffle_grouping:
 14                 component: spoutA
 15
 16     - bolt:
 17         name: boltB
 18         module: folder.boltB
 19         groupings:
 20             - shuffle_grouping:
 21                 component: boltA

I further checked what it is returned in read_tuple() to cmd. It is 56.
03/31/2015 03:55:13 PM - pyleus.storm.component - read_tuple - INFO: 56
30 03/31/2015 02:56:29 PM - pyleus.storm.component - run - ERROR: Exception in bolt.run
 31 Traceback (most recent call last):
 32   File "/usr/lib/python2.7/site-packages/pyleus/storm/component.py", line 233, in run
 33     self.run_component()
 34   File "/usr/lib/python2.7/site-packages/pyleus/storm/bolt.py", line 45, in run_component
 35     tup = self.read_tuple()
 36   File "/usr/lib/python2.7/site-packages/pyleus/storm/component.py", line 291, in read_tuple
 37     cmd['id'], cmd['comp'], cmd['stream'], cmd['task'], cmd['tuple'])
 38 TypeError: 'int' object has no attribute '__getitem__'

Can you try to run it locally and to debug the issue? I run a topology with some tuples in the order of the MB some time ago and it seemed to work fine (with some Storm tweaks).

The issues is because the msgs are too long for msgpack to handle. After I switch to json, it worked.

Ah, so this is actually a bug of our msgpack serializer. Good to know, thanks


Facebook Messenger attachments defined in Studio that do not have buttons result in an API error

Botkit Studio allows construction of generic (carousel) attachments.
However, even if no buttons are specified, the button element is left in as a blank array in the message object.
Facebook rejects this as invalid!
The button element needs to be pruned off of the attachment object if it is empty.

This is fixed in 0.5.6!


Add optional share url configuration

I've added a config option for a share url rather than the authenticated url. I also made the hostname filter optional.

Hi Team, any chances this could be reviewed and merged. I'd really appreciate it!

Hi, just want to ping this...

Hello, I'll close this in favor of #3.
Thanks for contributing! 🔥


bug: CI testing succeeded for PR that broke some tests

Describe the bug
The merge of PR #814 made some of the docs-ecutable tests start failing.  Indeed, the branch in that PR --- the new-issue-559-560 branch in https://github.com/fileppb/edge-mc, which pointed to commit 0e6e3f90c12020bcba9cc409c2278f8a4e8c6108 at the time --- also fails those tests.  And yet the testing in CI succeeded!  See https://github.com/kubestellar/kubestellar/actions/runs/5626329356 for an example.  See https://github.com/kubestellar/kubestellar/pull/814#issuecomment-1646894692 and the later comments for an explanation of the problem introduced and what is mysterious about the CI testing.
Steps To Reproduce
See above.
Expected Behavior
For a PR that breaks some tests, running those tests on the merge of that PR does not succeed.
Additional Context
No response

will work on this tomorrow

@MikeSpreitzer @clubanderson I also noticed that there are references in the .md's to SyncTarget.workload.kcp.io and Location.scheduling.kcp.io that need to be replaced with references to the edge.kcp.io api group. Will take a look at all the .md's.

@MikeSpreitzer @fileppb
I would like to get the CI to fail for 814.  It appears that the simple checkout that the workflow (for all docs-ecutable files) is referencing the base branch and not the the latest commit in any given PR.  To fix this I would like to test by

unmerging PR 814
add (at line 50 of docs-ecutable-example1.yml)
“with:
ref: ${{ github.event.pull_request.head.sha }}”
push a commit.

what do you think about this approach?  Or do you see another way to test the fix?

I am recommending (for all docs-ecutables)
 - uses<EMAIL_ADDRESS>        with:
          ref: ${{ github.event.pull_request.head.sha }}
        if: github.event_name == 'pull_request'

      - uses<EMAIL_ADDRESS>        if: github.event_name != 'pull_request'

@clubanderson , the workflow is already checking out and testing the relevant merge.  If you look at https://github.com/kubestellar/kubestellar/actions/runs/5626329356/job/15246915748?pr=814 , in the output of the actions/checkout, you will see that it checked out https://github.com/kubestellar/kubestellar/commit/bb51749176bcdfbd36f36249ab599b9917b1ab95 , which is a merge of @fileppb 's branch with what I surmise was in main at the time.  This is what we want tested.

I have re-created https://github.com/kubestellar/kubestellar/commit/bb51749176bcdfbd36f36249ab599b9917b1ab95 by the following commands.
(venv) mspreitz@mjs12 kubestellar % git checkout -b 817-test1 3281ffe2bb684bd738a7803e002c415568f71eca
Switched to a new branch '817-test1'
(venv) mspreitz@mjs12 kubestellar %
(venv) mspreitz@mjs12 kubestellar % git merge 0e6e3f90c12020bcba9cc409c2278f8a4e8c6108
hint: Waiting for your editor to close the file... 2023-07-24 13:31:02.324 Emacs[25786:3206670] It's not legal to call -layoutSubtreeIfNeeded on a view which is already being laid out.  If you are implementing the view's -layout method, you can call -[super layout] instead. Break on void _NSDetectedLayoutRecursion(void) to debug.  This will be logged only once.  This may break in the future.
Merge made by the 'ort' strategy.
...
(venv) mspreitz@mjs12 kubestellar % git status
On branch 817-test1
nothing to commit, working tree clean
(venv) mspreitz@mjs12 kubestellar % git log
commit 3092ac2827190d96183ee116ef487e56e8972b98 (HEAD -> 817-test1)
Merge: 3281ffe2 0e6e3f90
Author: Mike Spreitzer<EMAIL_ADDRESS>Date:   Mon Jul 24 13:31:01 2023 -0400

    Merge commit '0e6e3f90c12020bcba9cc409c2278f8a4e8c6108' into 817-test1

commit 0e6e3f90c12020bcba9cc409c2278f8a4e8c6108 (fileppb/new-issue-559-560, new-issue-559-560)
Author: Robert Filepp<EMAIL_ADDRESS>Date:   Fri Jul 21 16:26:18 2023 -0400
...

I then tested it, and it failed as it should:
(venv) mspreitz@mjs12 kubestellar % make MANIFEST="'docs/content/Coding Milestones/PoC2023q1/placement-translator.md'" docs-ecutable
...
++ kubectl kubestellar ensure location florin loc-name=florin env=prod
--- current directory is /Users/mspreitz/go/src/github.com/kubestellar/kubestellar/kubestellar
current ws is root:imw-1
imw is .
current cwsi is root:imw-1
error: apibindings.apis.kcp.io "edge.kcp.io" is forbidden: unable to create APIBinding: no permission to bind to export root:espw:edge.kcp.io
make: *** [docs-ecutable] Error 1

So I am still mystified about why https://github.com/kubestellar/kubestellar/actions/runs/5626329356/job/15246915748?pr=814 --- which tested the same contents --- succeeded.

I am out of ideas for how to investigate why https://github.com/kubestellar/kubestellar/actions/runs/5626329356/job/15246915748?pr=814 succeeded.
We need to make some testing gate merging a PR.  Right now the GitHub Actions for docs-ecutables are only FYI and not connected to the merge that updates the base branch of the PR.  I do not know whether or how GitHub Actions could be used to make these tests gate the merge that updates the base branch.  I think this sort of thing is part of what Prow does.

lets try a different PR (a simple and non consequential one) that we know breaks the docs.  Then we can fast-forward and rewind as much as we like.

Yes, that sounds like a good idea.

should we continue to chase this or regard it as a flake and re-open if we find this behavior again?  @MikeSpreitzer ?

If you're asking me, I would regard it as a weird combination of events that might be difficult to recreate, and shouldn't recur now that we're pointing to the correct (or expected) source for our build.

I am going to close this for now as a flake.


[question] Monitoring of the ecosystem

The more I look into this project the more I am considering it as good fit for my project. What tools do you use in production for monitoring the platform. With SpringBoot I like a lot springboot admin opensource project, zipkin for distributed tracing (https://github.com/openzipkin/zipkin), hawtio.
What is your support here?

We have ELK for the centralized logging and Influxdb/Grafana for metrics. There is built in traceablityId and correlationId in the middleware handler.

@stevehu - when you get into tracing using ELK, can you show me how it looks like when you trace chain of events and what all info are available? Dummy screenshot is enough, I would like to compare it with zipkin as some friends from IBM suggested its usage:
I am interested in both seing the detailed trace as zipkin provides:

But also if possible the full system map:

Fast capability in resolving issue is core.
Regarding metrics, I think you might be interested in this high preformance metrics system:
https://github.com/firehol/netdata
Here is available life instance:
http://london.my-netdata.io/default.html
As you suggested performance is one of your main concerns, this could be a playing factor.

We have something built on top of https://github.com/Netflix/vizceral and it is a bank IP. I cannot post it here.

For the response time per service, it is not automatically done by the framework but by the service builder to put into the log. Normally, it won't be enabled due to performance impact. Our main collections of metrics are statistic info only every 1 minutes or 5 minutes. Thanks for these links which are very inspiring.

For the response time per service, I might be thinking about overkill maybe :-). Foolowing scenario definitely is must for long running event of chains, but probably not for miliseconds scale :-)) haha.
So here we have the scenario:

This is common setup for long running tasks, but probably anything higher than 5-15 seconds in whole system.

So I finally found solution I will be happy with -> AOP, any kind of generic logging will be handled by AspectJ http://aspects.jcabi.com/  , to support flow fine-grined monitoring I will:

automatically log entry and exit from business methods - this I can use for automatical checking on method execution timeout alerts
automatically log errors
And more

This will as well significantly reduce size of code and therefore burden on developers...
Streaming of logs from services will be done via log4j2-elastic appender. I don't want to build any kind of kafka and logstash infrastructure. So just async log4j2 (non-blocking) elastic.
Additionally streaming of other components like message broker, event store database logs I will use syslog-ng which can as well stream directly via syslog-ng server to elasticsearch.
Thank you guys again for comments here. Zipkin is also choice.
And I will go with SpringBoot for speedup the development (even reusing some of your code base), I will use as well spring boot admin with jmx extensions.

If you are not dealing with huge volume, you solution will work. For high volume system, AOP is very slow and I would consider it is an deprecated technology. Also logging to elastic directly is no scaling and you have to put something like Kafka or Redis in between. If you go with spring-boot, take a look at eventuate-local which is our eventuate-4j based on.

@stevehu - totally agree Steve with both concerns, but this project consists of about 5-7 services and number of messages per DAY is in hundreds :-))), delivery time is 4 months, so any kind of shortcuts make sense. Also I worked on high scaled Elastic with Kafka and Logstash and it is really expensive to maintain it in big cluster... so in the end you have 2 projects:

your new microservice layers
your devops frameworks (CI/CD including central logging cluster for issue resolution)

Regarding AOP I meant AspectJ, not pure AOP and it is really not slow, there are differences why/how/where it is slow:

The Spring AOP performance penalty is considerable because dynamic proxies are being used through which each call will be routed.
AspectJ performance is much better, the penalty is really small because no dynamic proxies are needed. You can use native AspectJ from within Spring as described in the Spring manual, chapter "Using AspectJ with Spring applications". http://docs.spring.io/spring/docs/current/spring-framework-reference/html/aop.html#aop-using-aspectj
As described in my comments https://stackoverflow.com/a/27268153/1082681, you need to be careful what you measure. For instance, it makes no sense to measure the additional time it takes to do the actual timekeeping and logging compared to the original application without any timekeeping/logging because if you would add those manually via scattered timekeeping/logging statements, the penalty
would also exist. I am also reasoning about profiling aspects https://stackoverflow.com/a/32421622/1082681

As I said, it is always depends on your use case. CI/CD is a must in microservices architecture and it is part of the tool chain along with infrastructure services like security, messaging, logging and metrics. If you look at the middleware handlers we have, it is replacing AOP but plugable  and configurable.

@stevehu - can you please put here some link of middleware handlers you are referring to?


Issue 1905

Ⅰ. Describe what this PR did
add unit test for  apis/opts/portbindings.go
Ⅱ. Does this pull request fix one issue?

fix #1905
Ⅲ. Describe how you did it
add unit test for  apis/opts/portbindings.go
Ⅳ. Describe how to verify it
run go test -v under package opts
Ⅴ. Special notes for reviews

 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.zhuxiaoyuan seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


upstream, api(ticdc): support topology and config

What problem does this PR solve?

Issue Number: ref #10375, ref https://github.com/pingcap/tidb/issues/50723
What is changed and how it works?

Use /topology/ticdc/{clusterID}/{ip:port} to store topology info.

Support querying server config via /config api.

Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
 `None`.

/retest

/retest


feat: noir -> u32

Please read contributing guidelines and remove this line.

#6886  👈
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Maddiaa0 and the rest of your teammates on  Graphite


Update tests for Windows \r\n newline and \ path delimeter

All 28 Mocha tests were failing, and now pass.
Resolves Issue #89, and updates the stale PR #90.

Awesome, thanks!


Benchmark int decoding methods.

There are few approaches:

bitshift + bitor
bitshift + add
add bitshifted
zero pad and nativecast

Nativecast wins hands down. However it comes at a cost of checking endianess, zero-padding bytes (not all bytes of given type must be given) and there are no 182bits pointers to map to.
Built in unpack is the slowest due to pattern recognition and subbuffer cloning.
For other methods there is no significant difference. Currently used byteshift+byteor is good enough. I'll keep it until Perl 6 gets new pack/unpack API.

Also there is almost no penalty for first byteshift. In fact $out +<= 8 unless $out == 0; performs 9% worse.

Just to add to results - P5pack module:
P5pack: 1.030 wallclock secs (1.077 usr 0.010 sys 1.087 cpu) @ 9713.029/s (n=10000)


[WARN] NVD API request failures are occurring; retrying request for the 6 time

getting this error log in console

Why do you create numerous issues without first utilizing the search function? Similar problems have been noticed with others as well.


feat(raw-table): add headers option to raw-table

To Reviewers

[ ] Skipping review is not a problem (style, chore ONLY)
[ ] Not that difficult

Type of Change

[x] New feature
[ ] Bug fixes
[ ] Feature improvement

[ ] These changes require a usage change

[ ] Others (performance improvement, refactoring, CI/CD, etc.)

Checklist

[x] Updated Storybook documents for component
[x] Wrote test codes for composable, utils and etc.
[ ] Tested with console(if usages are changed)

Description

Things to Talk About

:tada: This PR is included in version 2.0.0-beta.33.6 :tada:
The release is available on:

npm package (@beta.33 dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Use actual unicode for the unicode example

é and à are in the ASCII tables, so they don't really demonstrate
unicode. Used the arabic word for "schema" instead.

That's not quite accurate.  é and à are not in ASCII, but just certain extended "ASCII" (which is not ASCII) code pages.  There are others that don't use those characters at all.  In a practical sense it does demonstrate unicode since we're using UTF-8 which encodes the unicode code points for characters (which in this case happens to correspond with a ISO 8859-1 which is an extended ASCII scheme.

All that said maybe instead of removing the existing example you could add this one in addition--that would be nice.

This is a good suggestion.  I will add some non-Latin-1 Unicode to the examples.


Avoid loading full data in memory?

As a long-term improvement, it would be great to be able to construct and run the pipeline without loading the whole dataset in memory, but processing it iteratively (hard to achieve when operations are performed both on rows and columns).

Hi Loan,
I just noticed BioProfiling.jl which looks great, thanks for your efforts! In regards to this issue, I was wondering whether you are aware of HDF5.jl and more generally DiskArrays.jl. Given that BioProfiling.jl at the moment expects a DataFrame to be passed this would obviously need some rewriting and I am not sure what the easiest way for this could be. Just wanted to give you a heads-up in any case.
Cheers,
Jesko

Hi Jesko,
Thanks, that sounds like a good start! I feel that supporting to select a subset of the dataset with lazy loading shouldn't be hard thanks to what DiskArrays.jl offers. That would already be an improvement, although the transformation steps might be a bit trickier to implement without loading everything in memory.
Cheers.

I see two ways to address transformations:

If two features with all observations fit into memory: one could just loop over the features for correlations, MAD etc. Please correct me if I am missing something there.
If that is not guaranteed
2.1 Approximation by reading data sequentially and computing intermediary statistics. See e.g. approximation of MAD here: https://stats.stackexchange.com/a/507132 streaming datasets
2.2 Random subsampling, i.e. selecting random cells. I am not sure how efficiently this would run with DiskArray.jl and would need to further look into it.

Not sure whether I will have time to create a PR soon but let me know what you think.
Cheers, Jesko


column type INT is not recognized properly in some situations

Like mentioned in this comment, INT column types are not recognized properly in some situations.
If you use INTEGER, everything will be fine. Additional type checks need to be implemented, since SQLite does not seem to normalize everything to INTEGER obivously.
At the same time, it makes sense to check for things like VARCHAR instead of  TEXT, and so on as well.

The type matching now works in the following way:

Check if a proper, correct type for SQLite has been used. The only "correct" types that exist in SQLite are:

INTEGER
REAL
TEXT
BLOB
see https://www.sqlite.org/datatype3.html

If no "correct" type has been used, apply SQLite type-affinity rules to check for other matches. These checks have a lot more overhead and you should always use proper SQLite types for best performance. The rules from 3.1. Determination Of Column Affinity have been applied. Not 100% exactly though because of the other checks I am doing beforehand already and I don't need to check some things twice.


SVN repository location?

Hi,
I know you no longer maintain this git repo, but I was wondering if you could point me to the SVN repository that Cyrus is using.  svn.mulberry.com no longer seems to work.
Thanks!

I don't know. I haven't even tried to access it for ages. If it's dead then
probably Cyrus has stopped maintaining it altogether.
Cheers,
Martin
On Wed, 10 Jul 2019 at 18:28, Quanah Gibson-Mount<EMAIL_ADDRESS>wrote:

Hi,
I know you no longer maintain this git repo, but I was wondering if you
could point me to the SVN repository that Cyrus is using. svn.mulberry.com
no longer seems to work.
Thanks!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mbert/mulberry-main/issues/3?email_source=notifications&email_token=AAC5SJ4BQAUJLNJ3DMCSRODP6YEZFA5CNFSM4H7REJXKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4G6MZMVQ,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAC5SJ6TPTQ6WGCGRS3YDHDP6YEZFANCNFSM4H7REJXA
.

--
Dr. Martin Dietze

Vorsitzender
Deutsch-Ukrainischer Kulturverein e.V.
http://www.deutsch-ukrainischer-kulturverein.de/

Ok, fair enough.  I've tried contacting him directly on linkedin, will see if that gets anywhere.  I need to rebuild mulberry with your repo at the least, my version has all sorts of crazy errors with modern email, like it'll send out chinese characters on a reply (???)


Pure Java sentencepiece implementation

As requested in https://github.com/vespa-engine/vespa/issues/18881
I'm going to integrate it into IL and YQL next so you can embed with no code.

Found Banned Dependency: com.google.protobuf:protobuf-java:jar:3.11.4
19:38:31 Use 'mvn dependency:tree' to locate the source of the banned dependencies.

Build failure is unrelated.

I'll add a few more model to test, looks great so far!


package port_for installs each update command

Each time I update, this package installs again.
poetry master and dev
(b35) [jim@PortableCricri]: ~/trash/apistar-ponyorm>$ poetry add port_for
Using version ^0.4.0 for port_for

Updating dependencies
Resolving dependencies

Package operations: 1 install, 0 updates, 0 removals

Writing lock file

  - Installing port_for (0.4)
(b35) [jim@PortableCricri]: ~/trash/apistar-ponyorm>$ poetry update
Updating dependencies
Resolving dependencies

Package operations: 1 install, 0 updates, 0 removals

Writing lock file

  - Installing port_for (0.4)
(b35) [jim@PortableCricri]: ~/trash/apistar-ponyorm>$ poetry update
Updating dependencies
Resolving dependencies

Package operations: 1 install, 0 updates, 0 removals

Writing lock file

  - Installing port_for (0.4)

 [tool.poetry]
name = "aa"
version = "0.2.8"
description = ""
authors = ["jgirardet<EMAIL_ADDRESS>
[tool.poetry.dependencies]
python = "^3.6"
port_for = "^0.4.0"

[tool.poetry.dev-dependencies]

@jgirardet The comes from the fact that the true package name is port-for but poetry never normalizes it which leads to the package not being detected as installed.
I will fix it.

@jgirardet Commit 3716ea9 on the develop branch fixes the issue. It will be available in release 0.9.0

I test it with next bump

Release 0.9.0 is out!


Issue 1575: PyPi controller

Pull Request Description
This pull request closes #1575 .
Acceptance Test

[x] Building the code with mvn clean install -Dintegration.tests still works.
[ ] Running mvn spring-boot:run in the strongbox-web-core still starts up the application correctly.
[ ] Building the code and running the strongbox-distribution from a zip or tar.gz still works.
[ ] The tests in the strongbox-web-integration-tests still run properly.

Questions

Does this pull request break backward compatibility?

[ ] Yes
[x] No

Does this pull request require other pull requests to be merged first?

[ ] Yes, please see #...
[x] No

Does this require an update of the documentation?

[x] Yes, please see strongbox/strongbox-docs#{PR_NUMBER}
[ ] No

Thanks for this contribution!
Would you mind creating a task list in the pull request's description where you outline what tasks you've done so far and what needs to still be done...? Thanks! :)
Also, could you please make sure you've formatted the code according to our convention? Thanks! :)


picojson does not support -fno-exceptions

Repro

Put file as foo.cc with below content
Run clang++ -fno-exceptions foo.cc

#include "/path/to/picojson.h"
int main() { return 0; }
// foo.cc

Expected
It compiles successfully
Actual
Many compilation errors for example:
picojson.h:1142:3: error: cannot use 'throw' with exceptions disabled
  PICOJSON_ASSERT(0);
  ^

As long as PICO JSON uses std:: it will not be usable in "no exceptions" projects.
Since PICO JSON is "pico" it will be not a big job (I assume) to make it usable with alternatives to std:: containers.  One example is EASTL.
The first step might be a slight redesign so that PICO JSON itself does not throw exceptions.

at least libc++ supports -fno-exceptions
https://reviews.llvm.org/D20677

Go ahead: fork and pull :)


Thumbnails in ftp connection

I'm building CMS with multiple websites where each one can be on a different serve, so I'm using the ftp storage with the filemanager.
The thing is that the thumbnais don't load because they are not in the CMS domain...
On the file script.js i needed to have the website url.
Any ideia how i could do this?
var image = $('<div>').css('background-image', 'url(URL + item.thumb_url + '?timestamp=' + item.time + '")');

Hello!
Firstly, I need to see your lfm.php config file.
Secondly, I guess you should define the URL of your FTP storage in filesystems.php config file
Something like this
'disks' => [
    ..........
    ..........

    'myFtpDisk' => [

        ...........

        'url' => 'my-ftp.example.com',
    ],
],

This worked! I didn't know about this option...
Thank you!


Height is not changing until I click the frame to activate it.

Describe the bug
Height is not changing until we click the frame to activate it. I'm using Interact.js for resizing
To Reproduce
Steps to reproduce the behavior:

Go to https://headwindui.vercel.app/
Drag the grey bar to resize the frame
You can see the height is not resizing
Now click on the frame and then it works

Expected behavior
Height should increase by default without needing to click first.
Screenshots

Desktop (please complete the following information):

OS: Windows 10
Browser: chrome
Version: latest

Okay. I have fixed it. I have found two reasons.

The plugin I have used (interact.js) was adding a height. I removed it.

Then added the following code for iFrameResize.

  iFrameResize({
    log: true,
    scrolling: "omit",
    checkOrigin: !1,
    resizeFrom: "child",
  });

Also, for future googlers  If your iframe resize is stopping while resizing, you will need to add a position: absolute element with pointer-events: none while resizing.
Closing this as its resolved.
Thanks to the makers for this awesome plugin, at first I didn't understood the power of this.
~Surjith


refactor: namespace of AsTool attribute

Needs https://github.com/php-llm/llm-chain/pull/89

Needs a version bump then


Fix launch.py for ios xrv >7.1

IOS XRv 24.2.2 does not show call-home configuration.
waiting for the linecard to give status IOS XR RUN works instead.
Tested with 24.2.2, 7.11.2 and 7.9.2

thanks @AndersLaastad
@kaelemc is this something you can check when you run your cisco lab?

@hellt Will test on 7.x but I don't have 24.x

Tested and working on 7.11.1 and 7.2.2. Should be good to go!


Question about the setting

Hi, thanks for your interesting work.
I am confused about the setting. Does the setting use the same training data with HCMoCo for down-stream tasks?  I mean are there any difference between the pre-training modalities and down-stream modality? Maybe, I miss something in the paper. But I do not find an apparent introduction. I might not understand this description well.

To evaluate HCMoCo, we transfer our pre-train model to four human-centric downstream tasks using different modalities,

Thank you for your interest in our work.
HCMoCo uses RGB, depth and 2d keypoints for pre-train. And we transfer the pre-trained RGB backbone to DensePose prediction and RGB human parsing.
The modalities are the same. But for some down-stream tasks like DensePose estimation and RGB human parsing, we use different datasets (MPII and NTURGBD for pre-train while COCO or Human3.6M for down-stream evaluation) which brings domain gap.
I hope the above explanation clarify your confusion.

Thanks for your reply. I get it.


Add post on managing tunnels with Argo CD

Signed-off-by: Han Verstraete (OpenFaaS Ltd<EMAIL_ADDRESS>Description

Add blog post on managing tunnels with Argo CD
Types of changes

[x] New blog post
[ ] Updating an existing blog post
[ ] Updating part of an existing page
[ ] Adding a new page

Does the background image render properly in the roll-up when you go to http://localhost:4000/blog/ ?

Does the background image render properly in the roll-up when you go to http://localhost:4000/blog/ ?

Made sure the logos fit.

You're missing an ending to the article! :slightly_smiling_face:

It renders like this for me


ui#1455 - pax2pay provider module

just a side note, i made this before in preparation for the assuming bit
assumeNew(orgCode: string) {
		const newClient = Client.create(this.connection.url, this.connection.token)
		newClient.connection.assumedOrg = orgCode
		return newClient
	}

the way this does it with sending the header looks fine and maybe even is easier but then i've at least mentioned that other part


Improve docs and error handling of Label::new

If you like these changes please merge the commits without squashing (e.g. via rebasing).

This seems sensible to me, thanks!


REQUEST: Add option to FILTER what it downloads

Hi and first of all thank you for this great peace of software. Awesome job.
Do you think would be viable to add options to filter what it downloads from the playlists?
Ex.: I don't want to download DJ Sets and stuff. I would like to download only single songs. Would be nice to have an option to filter. Like don't download anything above 20min or something like that.
Thank you in advance ;-) Keep up the great work!

That's a bit vague, I was planning to add an option to individually pick videos you don't want to download, but this feature didn't play nice with large playlists so I didn't add it in the end. I might find a solution to that problem since I have some time now.
Would that be what you're looking for?

"option to individually pick videos" This is also nice. Similar to what I've requested.
I use your app to download playlists from ytube so I can make mixtapes djing. Sometimes those playlists have unwanted stuff. Specially these large music videos over 1hr. Woulb be nice to filter these out. If you could implement the YouTube's filers itself would do it.

Another workaround I found but is very time consuming is to create a Playlist of my own and download that one.
But definitely would be very handy to just exclude by filtering. ;-)
Thanks again man and be safe ;-)

when downloading music from ytube. What is the maximum kbps it download it before it compress?

Usually youtube doesn't store anything over 138kbps in opus format, there are some cases they might decide to go beyond 138kbps, and sometimes they even go below that
I'll start working on the filtering soon

I added the filtering you requested, let me know if you're still missing some features

I added the filtering you requested, let me know if you're still missing some features

Wow! That was fast! Very very sweet. One more thing! Remember to also add this filter option on the subscription settings;

Cheers mate.

I completely forgot about that, it has been added


Some code questions

Thanks for the sharing code.

I only saw the repvgg module in the code, but you mentioned it in your paper without actually using it, right?
It seems like the inference model only outputs [out] instead of [out, res_2, res_3, res_4] in the code. It looks like I could potentially trim off the extra three output heads when converting it to trt.
Are the first three convolutional layers used for upsampling XYZIR, similar to the upsampling MLP for XYZ in PointNet?

btw
The CENet with 64x512 input model has been deployed on Jetson AGX Xavier and it worked successfully.
fp32 ≈ 6HZ
fp16 ≈10HZ

Hi. Thx for your attention. For ques:

Yes. I did not use any repvgg modules for this job. I'm just quoting their experimental description to show the efficiency comparison between 1x1 conv and 3x3 conv. However, I think it might be interesting to explore re-parameterization architectures for LiDAR segmentation in a range view perspective.
Yes, you are right, this part should be discarded when inferring.
The first three convolutional layers are only used to extract features and increase dimensions.

Glad to hear that 👍, network design on constrained devices is also a good topic.
In my personal opinion, changing the first downsampling to 2x downsampling gives better inference speed (i.e. 16x downsampling overall instead of the original 8x), however such a choice would require retraining or fine-tuning the model, and the final performance would be reduced.
Moreover, MINet (Code and Paper) also provides speed tests on the Jetson AGX, which might be a good reference.

你好！
关于1：该问题确实是球面投影的局限所致，对于同一方位角和顶角，只能保留一个点。关于这个问题，你可以看下PRNet、CPGNet和GFNet都有相关的讨论。理论上，多视角BEV RV PV融合会好点。当然，也可以尝试下多重投影的方法，可以尝试不只保留最前面的点，参考ThickSeg，不过这篇没开源。RangeFormer也讨论了这个Many-to-One的问题，理论上Transformer架构也会提升模型的鲁棒性。增大分辨率我不确定能不能解决这种问题， 看上去只能投影完后做插值resize范围图像，这样做应该是会涨点的。(参考TORNADO-NET（128x2048）和KPRNet（289x4097）的实验部分)
关于2：是的，我也觉得可以这么做 但是性能应该还是会降的。参考我们Readme之前更新的Robo3D Table 10里Sensor那一排的效果，miou下降的还是比较明显的。不过看上去CPGNet的效果会很好（我猜是BEV下beams数的影响会降低），可以参考下当时复现的CPGNet，Robo3D应该是用我之前版本训练的模型测试的。
关于3：这一块的话我不太确定，应为没有接触过其他真实数据，感觉可以尝试下。或者统计下分布Normalized一下hhh
个人感觉也可以做SSL半监督生成pseudo label 然后fixed权重 finetune self.conv1 = BasicConv2d(5, 64, kernel_size=3, padding=1) 这块。当然，如果是类似MINet那样x、y、z、d、r 5通道分开finetune会更容易点。
工程经验有限hhh 可能一些建议没啥参考价值:joy:

太强了


Drop the sample extension

We have a sample extension which is meant to demonstrate the basic scaffolding to get a minimalistic extension to work.
This is not super useful since we already have quite a lot other extensions in the repository, which are actually maintained since they expose functionality that people use. There is a risk is that this extension does not get updated (because breaking it does not actually break anything for users).
Also, as an extension developer, I generally want to develop my extension in its own repository, with its own build system. The Maven set up for extensions included in this repository is not easy to translate to extensions developed outside the repository, since our own set up relies on the main OpenRefine classes being present in the reactor at build time.
Once #2254 is done, it should be easier to propose a clean Maven setup for external extensions. We could move the sample extension to a dedicated repository, which was created a long time ago but remained empty:
https://github.com/OpenRefine/refine-sample-extension

@wetneb: Hi, I would like to work on this issue if it's not already taken. Thanks.

Concerning rearranging the source into new modules and namespaces, I have done that in 4.x and was thinking about keeping the 3.x release series with the current architecture. But I guess we could already publish the main module on Maven to make this issue easier. What do you think?

Sounds good to me. It will be easier after the release is published to Maven central.

@wetneb Are we still waiting for the release to start working on this?

Unless you have ideas about how to do this without releasing to Maven, I would wait for this, yes.

@wetneb: I suggest for a nightly release for this to proceed. Otherwise, I can release a snapshot that can be replaced later.

Sure, go for it then :)

Changing the title of this issue to reflect the opinion I voiced in https://github.com/OpenRefine/OpenRefine/issues/6426#issuecomment-1999565189

I think it would still be useful to have a basic scaffold that people could use to start their own extensions, so I copied the sample extension to a dedicated repository here:
https://github.com/OpenRefine/sample-extension


subnet qol

Replaces Org DevaddrConstraint with DevaddrSubnet to print CIDR subnet.
{
  "org": {
    "oui": 1,
    "owner": "13tyMLKRFYURNBQqLSqNJg9k41maP1A7Bh8QYxR13oWv7EnFooc",
    "payer": "112qB3YaH5bZkCnKA5uRH7tBtGNv2Y5B4smv1jsmvGUzgKT71QpE",
    "delegate_keys": [
      "112qB3YaH5bZkCnKA5uRH7tBtGNv2Y5B4smv1jsmvGUzgKT71QpE"
    ],
    "locked": false
  },
  "net_id": "000024",
  "devaddr_constraints": [
    {
      "range": {
        "start_addr": "48000000",
        "end_addr": "480003FF"
      },
      "subnets": [
        "48000000/22"
      ]
    }
  ]
}

Does the description need updated? It looks like the output is going to include a subnet cidr notation address within the constraints object but the code looks like it matches the title in that the entire constraints field is being replaced by a subnet object


[AutoDiff] Implement a preview of generalized differential operators

This patch implements a preview of generalized differential operators as higher-order functions using a mandatory inlining hack. This patch includes:

Serialize/deserialize the gradient instruction.
Allow gradient in canonical SIL. Emit a runtime error message from IRGen for now.
Make floating point types conform to VectorNumeric and Differentiable so that they can be differentiated.
Add transparent gradient(of:) and gradient(at:in:) operators.

@swift-ci please test tensorflow


Changing credential providers icon

Solved Jira Issue
New UI looks like this

Old UI looked like this

I changed the key symbol to the id-card symbol as it was closest to credential providers ( please suggest a better icon if you disagree )
The build was successful in my system with all tests. Please tell me where can I find the test results and how do I attach it here

[x] Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
[x] Ensure that the pull request title represents the desired changelog entry
[x] Please describe what you did
[x] Link to relevant issues in GitHub or Jira
[x] Link to relevant pull requests, esp. upstream and downstream changes
[x] Ensure you have provided tests - that demonstrates feature works or fixes the issue

Thanks @raunak111 I think this design has been incorporated in #417


Update http4s-circe, http4s-dsl, ... to 0.23.27

About this PR
📦 Updates

org.http4s:http4s-circe
org.http4s:http4s-dsl
org.http4s:http4s-ember-server

from 0.23.18 to 0.23.27
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.http4s" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.http4s" }
}]

labels: library-update, early-semver-minor, semver-spec-patch, version-scheme:early-semver, commit-count:1

Superseded by #182.


Correct includes value in library.properties

The includes field in library.properties is used to specify which #include directives should be added to the sketch via Sketch > Include Library > ShiftRegister74HC595. This field should specify the primary header file(s) of the library. Therefore, the value should be changed from Arduino.h to ShiftRegister74HC595.h. There is no need to #include Arduino.h in the sketch because the Arduino IDE automatically does this.

Thank you very much, didn't know about that.


Back cursor to default state in infowindows

STR

Create a map
Go to Popup tab
Add a infowindow.
Remove infowindow (select none)
If i hover in some point cursor hand appears.

Now it's different  because we can change the geometry. Closing then.


[Bug]  line_item_history can have duplicate IDs

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the issue
After reaching out to Fivetran support, I was directed to make a ticket here.
Currently this package has a test expecting  line_item_history to have unique ids. According to fivetran support, that is not the case, attaching the response below.

I understand that you are seeing duplicate ids in the recurly.line_item_history table. This is expected behaviour as history tables will capture versions of the objects as they are updated. Whenever a line item is updated, instead of updating the existing record in the table, the connector will add a new record for the line item id with the updated values. So the table will have the history of changes to the line item.

Relevant error log or model output
17:19:43  Finished running 51 tests in 0 hours 0 minutes and 15.34 seconds (15.34s).
17:19:43
17:19:43  Completed with 2 errors and 0 warnings:
17:19:43
17:19:43  Failure in test unique_recurly__balance_transactions_balance_transaction_id (models/recurly.yml)
17:19:43    Got 717 results, configured to fail if != 0
17:19:43
17:19:43    compiled Code at target/compiled/recurly/models/recurly.yml/unique_recurly__balance_transactions_balance_transaction_id.sql
17:19:43
17:19:43  Failure in test dbt_utils_unique_combination_of_columns_stg_recurly__invoice_coupon_redemption_history_coupon_redemption_id__invoice_updated_at (models/stg_recurly.yml)
17:19:43    Got 35 results, configured to fail if != 0
17:19:43
17:19:43    compiled Code at target/compiled/recurly_source/models/stg_recurly.yml/dbt_utils_unique_combination_o_0668cff24a63b94c64f5d150d5aeedd2.sql
17:19:43

Expected behavior
I don't know exactly. Either the correct behavior for that model is it has duplicates, and the test should be removed, OR upstream the latest version of the row per id is chosen. I'm not clear enough on what the final models are intended to represent to say for sure, but I expect is the latter.
dbt Project configurations
No special config here
Package versions

package: fivetran/recurly
version: 0.2.0

What database are you using dbt with?
snowflake
dbt Version
1.3.1
Additional Context
I might be able to help with fixing this upon, but I'm unclear on what the final models are intended to represent.
Are you willing to open a PR to help address this issue?

[ ] Yes.
[X] Yes, but I will need assistance and will schedule time during our office hours for guidance
[ ] No.

Hi @DylanNWatt thanks for opening this issue and sorry you are seeing these failures.
A few things I wanted to clarify and confirm before moving forward:
We do include a test on the line_item_history staging model; however, we take into account that the source table has historical records and therefore test the uniqueness on the combination of line_item_id and updated_at. This should show us unique historical records.
Expanding on the above, it looks like your models are not failing on the above test based on your error messages. Instead it looks like your tests are failing on the recurly__balance_transactions test and the stg_recurly__invoice_coupon_redemption_history test. To best understand what may be going on here I would recommend querying these tables on the respective fields that are being tested for uniqueness:

For the stg_recurly__invoice_coupon_redemption_history test failure

    select
        coupon_redemption_id, updated_at
    from stg_recurly__coupon_redemption_history
    group by coupon_redemption_id, updated_at
    having count(*) > 1

Once you run this, you should see 35 results. I would take one of those id and updated_at records and do a query on the staging table to determine if this truly is a duplicate. If all the fields are the same, then I would assume this to be a true duplicate and needs to be resolved within the source. If not, then this may be something we need to adjust in the package.

For the recurly__balance_transactions test failure

with dbt_test__target as (

  select balance_transaction_id
  from recurly__balance_transactions
  where balance_transaction_id is not null

)

select
    balance_transaction_id,
    count(*) as n_records

from dbt_test__target
group by unique_field
having count(*) > 1

Similar to the above, I would then expect to see 717 results from this query. I would then take one of these balance_transaction_id and inspect this id in the stg_recurly__line_item_history staging model. If you you see that this field has true duplicates then I would assume this to be something that needs to be sorted out in the source. Otherwise, we can explore any modifications needed to the package.
I do want to call out that in the recurly__balance_transactions end model, we do filter for only active records. This is making me a bit suspicious that these may be true duplicates. Regardless, running the operations above will definitely help us get to the bottom of the issue!
Let me know if you have any questions!

Hi, for the sql for  stg_recurly__invoice_coupon_redemption_history I assume the SQL should reference that table and not stg_recurly__coupon_redemption_history?
If so, this looks like legit data, because coupon updated IDs are 1:many with invoice ids. I don't think these IDs are particularly sensitive data, so here:

Similarly for balance_transaction_ids:

So for both of these, it appears the source intentionally has these IDs duplicated, one for 1:many, the other maybe for history logging?

Hi @DylanNWatt thanks for sharing this additional information. Unfortunately, I am still having a hard time determining what may be causing the test failures for you. Even though you are showing the duplicate IDs, our tests should take into account the is_most_recent_record boolean and not flag them as dupes since they are historical.
I feel the only way we will be able to get to the bottom of this is if you are able to schedule some time during our office hours for us to debug together live.

Done, set time for next wednesdau.

Hey @DylanNWatt! Thanks again for taking the time out today to debug these test failures with us live. It's super critical for us to see what customers are seeing and this should hopefully help us.
For creating a PR on dbt_recurly_source, Instructions on how to contribute to the package  are here.  You'll need to fork the repo and open a PR off the forked changes you make!
Please let us know if you have any questions! We're happy to assist.

Hi @DylanNWatt @fivetran-avinash 👋🏼
I also got a test failure for dbt_utils_unique_combination_of_columns_stg_recurly__invoice_coupon_redemption_history_coupon_redemption_id__invoice_updated_at, did you manage to find the root cause?

HI @benigls yep we believe we have found the issue for both of these test failures.
For dbt_utils_unique_combination_of_columns_stg_recurly__invoice_coupon_redemption_history_coupon_redemption_id__invoice_updated_at, we will need to add invoice_id into the combination of columns.
For unique_recurly__balance_transactions_balance_transaction_id (I see you opened up an issue there too), we plan on adding a unique_combination_of_columns for transaction_id, which appears to provide uniqueness.
@DylanNWatt and I are working to create a PR to hopefully fix these issues shortly.

Hey @DylanNWatt, just wanted to check in and see if you'll have time to create a PR for this issue. Otherwise we will fold this task into our next sprint next week. Let us know if you need any assistance!

Hi @DylanNWatt and @benigls, we've deployed these latest test fixes to our packages in our latest releases to both packages (dbt_recurly & dbt_recurly_source). Try it out and let us know what you think!

Thanks @fivetran-avinash 🙏🏼 I'll check it out


Inconsistent textboxes

https://github.com/Microsoft/vscode/issues/57301
One is monospace, the other isn't.

IIRC this was by design with the reasoning being that when leaving a comment in a file, you're more likely to want to write code and have a monospace font.

Shouldn't we align with github.com here? If we have 2 different fonts, not only are we inconsistent with github but we're inconsistent in our own product.

Duplicate of https://github.com/microsoft/vscode-pull-request-github/issues/167


[yasm] Build error on x64-windows

how can i solve this issue?
error: building yasm:x64-windows failed with: BUILD_FAILED
Elapsed time to handle yasm:x64-windows: 3 s
Please ensure you're using the latest port files with git pull and vcpkg update.
Then check for known issues at:
https://github.com/microsoft/vcpkg/issues?q=is%3Aissue+is%3Aopen+in%3Atitle+yasm
You can submit a new issue at:
https://github.com/microsoft/vcpkg/issues/new?title=[yasm]+Build+error+on+x64-windows&body=Copy+issue+body+from+D%3A%2Fvcpkg%2Fvcpkg%2Finstalled%2Fvcpkg%2Fissue_body.md

Please provide specific steps to reproduce the problem and error log.

Thanks for posting this issue. Please reopen this issue if this is still a problem for you.


module mock does not work in some cases in browser mode

Describe the bug
When mocking the contentful package in browser mode, the mock doesn't seem to apply the first time tests are run and the mocked named export is undefined. Rerunning the tests causes the named export to be a spy, but it does not register being called.
Changing to the jsdom environment causes tests to pass.
import { createClient as contentfulCreateClient } from "contentful";
import { createClient as localCreateClient } from "../src/contentful.js";
import { vi, expect, describe, it } from "vitest";

const mocks = vi.hoisted(() => ({
  contentfulMock: vi.fn(),
  localMock: vi.fn(),
}));
vi.mock("contentful", () => ({
  createClient: mocks.contentfulMock,
}));
vi.mock("../src/contentful.js", () => ({
  createClient: mocks.localMock,
}));

describe("mocking", () => {
  describe("from contentful", () => {
    it("can make a client", () => {
      // this fails, it's undefined?
      contentfulCreateClient({
        accessToken: "accessToken",
        space: "space",
      });
      expect(mocks.contentfulMock).toHaveBeenCalled();
    });
  });
  describe("from local", () => {
    it("can make a client", () => {
      localCreateClient({
        accessToken: "accessToken",
        space: "space",
      });
      expect(mocks.localMock).toHaveBeenCalled();
    });
  });
});

First test run: TypeError: createClient is not a function
2nd test run: AssertionError: expected "spy" to be called at least once
Reproduction
https://github.com/kwojcik/vitestsourcemapbug/tree/contentfulMock
git checkout contentfulMock
npm install
npm test

Rerunning the tests gives a different error
System Info
System:
    OS: macOS 13.6.7
    CPU: (12) arm64 Apple M2 Max
    Memory: 6.03 GB / 64.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 18.18.2 - ~/.asdf/installs/nodejs/18.18.2/bin/node
    Yarn: 1.22.21 - /opt/homebrew/bin/yarn
    npm: 9.8.1 - ~/.asdf/plugins/nodejs/shims/npm
    pnpm: 9.4.0 - /opt/homebrew/bin/pnpm
    Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
  Browsers:
    Chrome: 126.0.6478.127
    Safari: 16.6
  npmPackages:
    @vitest/browser: v2.0.0-beta.12 => 2.0.0-beta.12
    @vitest/ui: latest => 2.0.0-beta.12
    vite: latest => 5.3.2
    vitest: v2.0.0-beta.12 => 2.0.0-beta.12

Used Package Manager
npm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Looks like it's an issue with ES interoperability - the contentful only has a default export in the browser, and then Vite adds a helper wrapping it into a function:
const interopHelper = (m: any) => m?.__esModule ? m : { ...(typeof m === 'object' && !Array.isArray(m) || typeof m === 'function' ? m : {}), default: m }