Getting Started Documentation

When I was getting started with this project I had no idea how to include and access the library.
I ended up creating my own browser bundle with browserify, which worked for a while until I ran into a problem because my bundle contained a second copy of React.
I only found out from this closed issue that there is already a browser bundle, ReactMDL.js. It's just only on npm and not in the repo, even though there's lots of other stuff in the repo for distribution purposes.
(I've since updated my usage, getting rid of webpack.)
The README should explain how to install and include the library.
You don't need to import it (which isn't supported in major browsers), or use browserify or webpack, you can just copy node_modules/react-mdl/out/ReactMDL.js to a lib or vendor folder and include it as a script (unless you have a setup where you can rely on the existence of node_modules)
I could create a PR with some improvements.

Hi @1j01,
I understand the way you do to use the library will also work, but most users will use webpack or browserify to bundle their project, and therefore include react-mdl using the same way.
Either way, I also believe a "getting started" document might help people to better understand how to use the lib. Would you want to help on this?

Yes, it should explain how to use it with webpack or browserify. I'd be using one of those if I could get a live-reloading working with either.
So, were you thinking this should be a separate document? I was thinking of just having all the information on the README, except for the external examples (which benefit from the interactivity). In other words, READMEs are generally the getting started documents, no?

Having it in the readme is fine, unless it becomes too big. Either way, we can start with that and move it in another file later if necessary.

Sounds good, I'll send a PR soon.


CS:GO Round Winner Classification

Predict who wins individual snapshots of rounds. Using 3-4 algorithms to implement the models and compare all the algorithms to find out the best fitting algorithm for the model by checking the accuracy scores.
@Niketkumardheeryan Please assign this issue to me.

@Niketkumardheeryan sir please assign this to me

Hi I want to do this and I can do this very efficiently, please assign me this repository!

@Niketkumardheeryan please unassign tandrima and assign me this!

@SoumyoNathTripathy Go for it. But don't forget to make a PR within 5 days.


[WIP] rx.match

nested rx.cond can get very intricate and messy. This PR introduces rx.match to fix this problem.
rx.match takes in a condition which should be a var, match case tuples and a non tuple argument that will be treated as the default or catchall case.
import reflex as rx

class State(rx.State):
    value: int = 0

def index():
    return rx.vstack(
        rx.match(
            State.value,
            (1, rx.vstack(rx.text('first value'))),
            (2, rx.text("second value")),
            rx.text("default value")
        )
    )

This renders as(formatted manually) :

/** @jsxImportSource @emotion/react */import { Fragment, useContext } from "react"
import { Fragment_fd0e7cb8f9fb4669a6805377d925fba0 } from "/utils/stateful_components"
import { Text, VStack } from "@chakra-ui/react"
import { StateContexts } from "/utils/context"
import "focus-visible/dist/focus-visible"
import NextHead from "next/head"

export default function Component() {
  const state__state = useContext(StateContexts.state__state)

  return (
    <Fragment>
      <Fragment_fd0e7cb8f9fb4669a6805377d925fba0 />
      <VStack>
        <Fragment>
          {
            (() => {
              switch (state__state.value) {
                case 1:
                  return <VStack>
                    <Text>
                      {`first value`}
                    </Text>
                  </VStack>;
                  break;
                case 2:
                  return <Text>
                    {`second value`}
                  </Text>;
                  break;
                default:
                  return <Text>
                    {`default value`}
                  </Text>;
                  break;
              }
            })()
          }
        </Fragment>
      </VStack>
      <NextHead>
        <title>
          {`Reflex App`}
        </title>
        <meta content={`A Reflex app.`} name={`description`} />
        <meta content={`favicon.ico`} property={`og:image`} />
      </NextHead>
    </Fragment>
  )
}

Match cases
These are tuples containing cases to be matched as well as their return values. The first n-1 values are treated as the cases to be matched and the last value in the tuple is treated as the return value.
Note that, a match case tuple MUST have a minimum of 2 elements (the condition and the return value).
Default/catchall case
A default case is a non-tuple argument that is matched when none of the match cases are matched. Below are the caveats to know about the default case:

The default case MUST be the last argument(the app will throw a ValueError if not)
(not accepted) ❌

 rx.match(
            State.value,
            (1, rx.vstack(rx.text('first value'))),
            rx.text("default value"),
            (2, rx.text("second value")),
        )

(not accepted) ❌
 rx.match(
            State.value,
            (1, 'first value'),
            "default value",
            (2, "second value"),
        )

There can only be one default case.
(not accepted) ❌

 rx.match(
            State.value,
            (1, rx.vstack(rx.text('first value'))),
            (2, rx.text("second value")),
            rx.text("default value"),
            rx.text("Another default value")
        )

(not accepted) ❌
 rx.match(
            State.value,
            (1, 'first value'),
            (2, "second value"),
            "default value",
            "another default value",
        )

The default case is optional when the type of the return value of the cases is a component (a default case Fragment.create() is implicitly created), however, when the return type is a Var (or when rx.match is used as a prop) a default case MUST be provided.

(accepted)✅
rx.match(
            State.value,
            (1, rx.vstack(rx.text('first value'))),
            (2, rx.text("second value")),
        )

(not accepted) ❌
 rx.match(
            State.value,
            (1, 'first value'),
            (2, "second value"),
        )

rx.match as prop
...
class State(rx.State):
    value: int = 0
    num: int = 1
...
rx.text(
            "Random text",
            color=rx.match(
                State.value,
                (1, "red"),
                (2, "blue"),
                (State.num + 1, "orange"),
                "yellow"
            ),
        ),

support for multiple cases
...
rx.match(
            State.value,
            (1, rx.text("first case")),
            (2, rx.text("second case")),
            (3, 4, 5, rx.text("third case")),
	    ((1,2,3), 6, 7, rx.text("fourth case")),
            ({"foo": "bar"}, [10, 11, 12], "random_case" ,rx.text("fifth case")),
             rx.text("default case"),
        ),

As mentioned above, the first n-1 elements of the tuple are treated as cases to be matched and the last as the reurn value.
Note : Match cases must not have different return types:
(not accepted) ❌
...
rx.match(
            State.value,
            (1, rx.text("first case")),
            (2, rx.text("second case")),
            (1, "red"),
            (2, "blue"),
             rx.text("default case"),
        ),

Looks really cool.  What types of values are supported for case values?  (examples above show "int").

Looks really cool. What types of values are supported for case values? (examples above show "int").

Looks really cool. What types of values are supported for case values? (examples above show "int").

Should work with any regular python type
https://github.com/reflex-dev/reflex/blob/1adfeafb0e1910ec6329c5e3b97f608e8017b04f/tests/components/layout/test_match.py#L216-L237

"""Welcome to Reflex! This file outlines the steps to create a basic app."""
from rxconfig import config

import reflex as rx

docs_url = "https://reflex.dev/docs/getting-started/introduction"
filename = f"{config.app_name}/{config.app_name}.py"

class State(rx.State):
    value: str = 'a'

    def increment(self):
        self.value += 'a'

def index():
    return rx.vstack(
        rx.text("Hello, world!"),
        rx.text(State.value),
        rx.match(
            State.value,
            ('a', rx.vstack(rx.text('first value'))),
            ('aa', rx.text("fourth value")),
            ('aaa', rx.text("fifth value")),
            rx.text("default value")
        ),
        rx.button("Increment", on_click=State.increment),
    )

# Add state and page to the app.
app = rx.App()
app.add_page(index)

"""Welcome to Reflex! This file outlines the steps to create a basic app."""
from rxconfig import config

import reflex as rx

docs_url = "https://reflex.dev/docs/getting-started/introduction"
filename = f"{config.app_name}/{config.app_name}.py"

class State(rx.State):
    value: str = 'a'

    def increment(self):
        self.value += 'a'

def index():
    return rx.vstack(
        rx.text("Hello, world!"),
        rx.text(State.value),
        rx.match(
            State.value,
            ('a', rx.vstack(rx.text('first value'))),
            ('aa', rx.text("fourth value")),
            ('aaa', rx.text("fifth value")),
            rx.text("default value")
        ),
        rx.button("Increment", on_click=State.increment),
    )

# Add state and page to the app.
app = rx.App()
app.add_page(index)

This is fixed now

```python
"""Welcome to Reflex! This file outlines the steps to create a basic app."""
from rxconfig import config

import reflex as rx

docs_url = "https://reflex.dev/docs/getting-started/introduction"
filename = f"{config.app_name}/{config.app_name}.py"

class State(rx.State):
    value: str = 'a'

    def increment(self):
        self.value += 'a'

def index():
    return rx.vstack(
        rx.text("Hello, world!"),
        rx.text(State.value),
        rx.match(
            State.value,
            ('a', rx.vstack(rx.text('first value'))),
            ('aa', rx.text("fourth value")),
            ('aaa', rx.text("fifth value")),
            rx.text("default value")
        ),
        rx.button("Increment", on_click=State.increment),
    )

# Add state and page to the app.
app = rx.App()
app.add_page(index)

This is fixed now

Ok nice testing

Seems to be working alright for me when I test it


Infinite loop when using grunt watch with babel require hook

Im trying to use the Babel require hook in my Gruntfile. Everything works fine until I try to use grunt-watch.
Here is an example.
module.exports = function(grunt) {

  require('babel/register')({
    experimental: true,
    extensions: ['.es6'],
  });

  require('load-grunt-tasks')(grunt);

  grunt.initConfig({
    mochaTest: {
      options: {
        reporter: 'spec'
      },
      src: ['test/**/*.es6'],
    },
    watch: {
      files: ['lib/**/*.es6'],
      tasks: ['test']
    }
  });

  grunt.registerTask('test', ['mochaTest']);

};

First a get a bunch of these messages
Running "watch" task
Waiting...
 This will break in the next version of node. Please use setImmediate for recursive deferral.

and then
RangeError: Maximum call stack size exceeded

@shama example project https://github.com/vladikoff/grunt-watch-issue-babel425
Not seeing what causing this yet...

Adding
  require('babel/register')({
    experimental: true,
    extensions: ['.es6'],
  });

breaks things...

What happens when you enable stack? grunt watch --stack and remove mochaTest from the equation.
node-inspector is a great tool for debugging these things.

My guess is it's from tiny-lr and specifically this: https://github.com/jshttp/on-finished/blob/master/index.js#L65
FWIW, this kind of bad stuff happens when you globally apply an es6 library. Why not just use babel when you need it instead of applying it everywhere?

the babel require hook above actually will only touch .es6 files
Thats what extensions: ['.es6'] is setting.
So babel shouldn't be touching anything from tiny-lr

FWIW moving the require('babel/register') to the end of the file seems to work.
module.exports = function(grunt) {

  require('load-grunt-tasks')(grunt);

  grunt.initConfig({
    mochaTest: {
      options: {
        reporter: 'spec'
      },
      src: ['test/**/*.es6'],
    },
    watch: {
      files: ['lib/**/*.es6'],
      tasks: ['test']
    }
  });

  grunt.registerTask('test', ['mochaTest']);

  require('babel/register')({
    experimental: true,
    extensions: ['.es6']
  });

};

Nice! Thanks!

Thanks @tikotzky. That is very strange though, I'm curious to know where it's getting tripped up.


[ecalrec]: "/" converts to "" on Linux

Describe the bug
using forward slashes in the Meas dir field leads to creation of folders in your home directory which have the same name as the full path including the measurement folders name. The real measurement with it's compartment's is correctly stored at the desired location in ~\home\meas
To Reproduce
start a measurement with the configuration as shown in the first sceenshot.
$TARGET{OSSELECT WIN "C:\meas" LINUX "/home/continental/meas/"}
Expected behavior
don't create not relevant home-dir flooding empty folders
Screenshots

Desktop (please complete the following information):

OS: Ubuntu
Version 18.04

Additional context
Add any other context about the problem here.

Hi nilshamacher89,
I checked with Windows + Linux VM Setup, where Windows is running eCAL 5.9.1 and Linux (Ubuntu 20.04) eCAL 5.9.2. Everything worked as expected in that setup:

Which eCAL Version are you using? Can you upload an .ecalrec config file that produces that error?
Regards
Florian

Closing, as aparently the directories are not created by eCAL Rec, but by a custom addon that seems to convert / to \ for some reason


custom GraphQLScalarType not used on mergedSchemas.

Example using a custom scalar type without mergeSchemas is working correctly : https://codesandbox.io/s/custom-scalar-type-jewe1
Example using a custom scalar type with mergeSchemas is not working: https://codesandbox.io/s/custom-scalar-type-p831z
Any idea how I can address  this issue?
Thank you.

In graphql tools version of mergeSchemas, custom scalars are not parsed, just passed through, a problem for gateway logic.
https://github.com/yaacovCR/graphql-tools-fork/commit/2ec9ac210aa376e2739180e6d587006464f0a360
Fixed in graphql tools fork

Why just not taking over graphql-tools-fork fixes to graphql-tools? Really hard for me to understand, just to say its deprecated. Seems to me, like a sad story.

We recently released an alpha version of GraphQL Tools (#1308) that should fix the issue.
Please update graphql-tools to next or run:
npx match-version graphql-tools next

Let us know if it solves the problem, we're counting for your feedback! :)

Rolled into #1306


Extend simulator to account for warmth ladder

Control loop behaviour is deeply affected by delays and lags. The principal lag in our system is startup time for functions.
We currently model this as a single delay for all startups. However, startup time is not constant. As systems become progressively "warmer", the time to startup changes. Worse, this warming curve is affected by the number of available worker nodes.
This issue introduces two new requirements for the autoscaler.

Distinguish between levels of the warmth ladder: cold start, disk-warm, memory-warm and CPU-warm.
Model how instances "move" from one warmth level to another over the lifetime of the simulation.

Out of scope but ideal for followup issues:

Allowing the simulation to show behaviour on small and large worker clusters (more nodes means average temperature will be lower)
Show behaviour of chained functions

This issue has gone stale, we're on Knative now! Please reopen if still relevant.


enh: add select to InterchangeFrame

We should allow for selecting columns by name in InterchangeFrame, falling back to select_columns_by_name for generic dataframe interchange protocol implementers

Started to take a quick look at this. For DuckDB and Ibis it should be doable with native methods.
For InterchangeFrame, since we both store _native_frame and _interchange_frame, the latter can be slices with select_columns_by_name, yet, how do we return the former object?
From a talk of yours, is it by accessing .dataframe attribute? From the docs this is not very clear to me

That was the "dataframe standard" 😄 The interchange protocol has no standardised way of getting the original dataframe object back
I think we can implement this for duckdb and ibis for now
I've opened an issue about this in the dataframe-api repo, but I'm not holding my breath regarding a response/decision: https://github.com/data-apis/dataframe-api/issues/360


Build Target: Server-Rendered planed?

What problem does this feature solve?
Any plans on Server-Rendered-Apps as build target?
What does the proposed API look like?
vue-cli-service build --target ssr

An SSR build target doesn't make much sense because you can't "build" JUST any Vue app into an SSR app.
The app has to be written in a certain way in the entry file , which is notably different to the code that the cli currently generates for main.js
So it would make more sense to implement it as a cli plugin.
And no, we are currently not planning to write such a plugin, since we feel that nuxt and ream cover this use case pretty great right now.
We would welcome community contributions though.


MIC-111B remediation on Create CSIP and Referral

hardcoding isReferralAffected=true for relevant audit event / domain event
annotating CsipRecordService class to be Transactional
enhance error message when multiple Contributory Factor Type codes are inactive or not found

Codecov Report
Attention: Patch coverage is 94.23077% with 3 lines in your changes missing coverage. Please review.

Project coverage is 87.43%. Comparing base (432935b) to head (0768c62).

Files
Patch %
Lines

...api/exception/InvalidReferenceDataCodeException.kt
95.83%
1 Missing :warning:

...rtinterventionplanapi/service/CsipRecordService.kt
93.75%
0 Missing and 1 partial :warning:

...nterventionplanapi/service/InvestigationService.kt
87.50%
0 Missing and 1 partial :warning:

Additional details and impacted files
@@             Coverage Diff              @@
##               main      #38      +/-   ##
============================================
+ Coverage     87.11%   87.43%   +0.32%
- Complexity      472      477       +5
============================================
  Files            98       99       +1
  Lines          1645     1679      +34
  Branches         36       35       -1
============================================
+ Hits           1433     1468      +35
- Misses          191      192       +1
+ Partials         21       19       -2

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


[WIP] Add adhoc playbook docker_storage_to_overlay2.yml

Wraps the similarly named role in openshift-ansible.
Heavily based on the os_update.yml adhoc playbook.
@dak1n1 : I'm not sure if I'm calling the vendored in roles correctly.  Can't seem to find any other examples.  Please check.

Validation test are going to fail until https://github.com/openshift/openshift-ansible/pull/5216 gets merged and the new roles are vendored in.

[test]

Scrapping this PR since I didn't follow the SOP correctly.


add some style

the current version has no css at all: make your proposition :)

Added some styling via PR #5


[CI] Bump pyOpenSSL version used in testing

This will allow us to test with the latest version of cryptography.
Also some cryptography upper bound pins were removed in order to be compatible with the latest pyopenssl. They don't appear to be needed anymore.

/azp run python - identity

Cancelled builds are from me manually cancelling manually triggered builds at the release/approval step. All tests passed.

/check-enforcer override


Buttons keep skin reference

Fixes #4387
These few constructors (out of all the Button and Button subclass constructors) were missing this pass-through of the Skin reference, causing getSkin() to return null when it shouldn't (that is, when a skin was passed to the constructor).

Thanks!


[$250] Company cards - “Assign card” and “Settings” buttons are below intro modal

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.28-0
Reproducible in staging?: Y
Reproducible in production?: N
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause Internal Team
Action Performed:

Launch New Expensify app.
Go to workspace settings > Company Cards.

Expected Result:
“Assign card” and “Settings” buttons should be at the top of the of the page and above the intro modal.
Actual Result:
“Assign card” and “Settings” buttons are below the intro modal.
Workaround:
Unknown
Platforms:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/25f38cf0-4afc-4ee1-8e6c-6df02f9e399f

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01fcc55f0c4c0d495b
Upwork Job ID:<PHONE_NUMBER>821009051
Last Price Increase: 2024-09-03

Issue OwnerCurrent Issue Owner: @eVoloshchak

This issue will be fixed in this PR. This functionality is still under active development.
cc @joekaufmanexpensify @koko57

Ah awesome. Perhaps we can close this one then?

@joekaufmanexpensify Yeah, I think it makes sense

Great!


bazel: extend oss-fuzz set up.

Split CFLAGS and CXXFLAGS into language-specific bazel constructs. This is needed to make Tensorflow work.
Related: https://github.com/google/oss-fuzz/pull/8992
Signed-off-by: David Korczynski<EMAIL_ADDRESS>

This one is good to from my side. Have tested with the OSS-Fuzz projects tensorflow, fuzztest-example and fuzztest-raksha, using both ASAN, UBSAN and coverage visualization.


F0750 hide chromosomes

Coverage remained the same at 100.0% when pulling e3cc6378231977f8d07d47140e885a918126ad2a on F0750_hideChromosomes into 78598387b6b7ec466cc1194b708d6ec68d719862 on master.

Coverage remained the same at 100.0% when pulling 32b0d4ea79bbe81b9d93df79fb90aa835d2d38c1 on F0750_hideChromosomes into 78598387b6b7ec466cc1194b708d6ec68d719862 on master.


Feature request: add option to hide unread labels

Feature request: add option to hide unread labels
This would help with people with OCD as seeing an unread label triggers compulsion to read in order to clear the labels. It can get very annoying when opening the app and seeing all this unread labels in Folders, feeds, today section etc. I think it should be easy to implement

I can see how this would help people. The downside is that the user would have to click through all their folders to know if there are any unread items in them.
Maybe it would be useful to have some kind of indicator to show that there are unread items in a folder just so that users who need this feature don't have to do a bunch of hunting to find their unread articles. Then they could see the total of unread in that folder as displayed at the top of the Timeline.
Maybe all 3 options could be made available? I'm just dubious that completely removing the ability to know if there are unread articles in a folder is a good idea.

You can always go to the unread section and there you find them. No need to search through folders
El 20 sept 2022, a las 18:42, Maurice Parker @.***> escribió:

I can see how this would help people. The downside is that the user would have to click through all their folders to know if there are any unread items in them.
Maybe it would be useful to have some kind of indicator to show that there are unread items in a folder just so that users who need this feature don't have to do a bunch of hunting to find their unread articles. Then they could see the total of unread in that folder as displayed at the top of the Timeline.
Maybe all 3 options could be made available? I'm just dubious that completely removing the ability to know if there are unread articles in a folder is a good idea.
—
Reply to this email directly, view it on GitHubhttps://github.com/Ranchero-Software/NetNewsWire/issues/3679#issuecomment-1252546357, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAGJAYXYGWBAQSA6PQF5TQDV7HLPZANCNFSM6AAAAAAQREHP3Q.
You are receiving this because you authored the thread.Message ID: @.***>

Then you lose all the utility of organizing your feeds into folders. I personally never use the All Unread Smart Feed and I'm sure others do the same.

I can tell you that having unread indicators even without numbers it would drive me crazy. But that is my own problem. But different options would be good, like nothing, indicators only and unread numbers
El 20 sept 2022, a las 19:09, Maurice Parker @.***> escribió:

Then you lose all the utility of organizing your feeds into folders. I personally never use the All Unread Smart Feed and I'm sure others do the same.
—
Reply to this email directly, view it on GitHubhttps://github.com/Ranchero-Software/NetNewsWire/issues/3679#issuecomment-1252580816, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAGJAYQTB2KR5FRL4L3S4MLV7HOTHANCNFSM6AAAAAAQREHP3Q.
You are receiving this because you authored the thread.Message ID: @.***>

I came here to second this. For a few years I’ve used sfeed, which just generates a static HTML page, no unread count. Trust me, removing any sense of read/unread is an infinitely better experience. No indicators, no secret read flag for smart folders or whatever. Just off. NNW would benefit greatly from this feature!

I would love this feature also, especially if we could enable/disable it on a feed level.
Some feeds I feel are important enough (to me) to see the unread count, others are occasional skim through the list to see if something catches my attention.


Provide context and activation subscription for macros

Hello. Congratulations on the awesome extension. I have a use case where I want to use extension to listen to a context menu event and execute custom command.
Everything is working except one thing. To be able to subscribe to an event I'd need to have access to the context object.
The same you use to register macro commands in the menu.
So far I've figured out it may be possible to create a kind of global variable and put the context there to be accessible from macros.
It would expand the use cases of the extention because we'd be able to write macros that tap into vscode.
Building on the idea it would be even greater to be able to define next to a macro, an activation even listener.
That way I could subscribe to interesting events at the start of the extension. It would only require a check if the context can be stored in a way macro can use it.
Cheers! Have a nice day and again congrats on the amazing extension!

Thanks for updating the ticket. I've explained the motivations for this enhancement in an article here: https://tomaszs2.medium.com/vscode-macros-saved-me-1000s-of-seconds-of-misery-f1946696ec94?sk=2704992e68c5bf0bb79aae8151dd3627

@tomaszs
Thank you for your excellent suggestion and the great Medium article.
I would also like to implement the feature you suggested at some point.
Unfortunately, I am currently involved in several projects with tight schedules and am unable to find time for OSS development, so I'm not sure when I can start working on your proposal.
If possible, I would appreciate your contribution.
Best regards,

@exceedsystem Hear you. I'm in the same spot 😅 I don't know if and when I could add it. But for the future reference it would involve two things in short:

adding a way to add activation handlers. I'd assume on the macro level as an additional property next to the func
exposing context as a func argument

What do you think about that approach ?


Standardize volume

Pulled all the ATR code except the propagating of volumes into the data_standardization section, as a standardize_volume script. Renamed ATR_utils to boston_volume because it's all boston-specific volume stuff
Created a volume schema, storing the ATRs there instead of geocoded_atrs
Changed the geocoding slightly so we're storing all addresses, from ATRs and TMCs in the geocoded_addresses.csv file. If the address exists, we read from that geocoded cache, otherwise we geocode, add to the cache and write out the cache to geocoded_addresses at the end
Pulled the propagate volume functionality into its own script, called propagate_volume, and hooked that up into the pipeline
Made this compatible with existing TMC functionality. Pulling parsing TMCs into the new schema in the data_standardization step will be a later task
Added some tests so code coverage at least doesn't go down. Having code coverage actually go up will also be a later task

@j-t-t these are only minor suggestions and functionally I can't see any reason why this can't be merged.
@andhint do you have any feedback regarding the changes to ATR handling?


Adds new command sw:tinker based on psy/psysh

1. Why is this change necessary?
We used to have an interactive shell while Shopware included Symfony 2. Now this interactive shell is gone because Symfony does no longer have one. With PsySH we can have a new interactive shell that is even more powerful than what we used to have. PsySH is a REPL shell and integrates great with Symfony based applications. For more information about it visit https://psysh.org/
2. What does this change do, exactly?

Adds a new dependency: psy/psysh
Adds a new command: sw:tinker

3. Describe each step to reproduce the issue or behaviour.
Execute sw:tinker and have fun in an interactive REPL shell that can execute every present php console command while also being able to evaluate php code live and have full access to the dependency injection container and everything else you only have at runtime.
4. Please link to the relevant issues (if any).
None.
5. Which documentation changes (if any) need to be made because of this PR?
Maybe the developer documentation should mention this as a way to tinker around in a running environment. Maybe not.
6. Checklist

[ ] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

I want to point out the best command psysh has. wtf

Looks like the composer.lock is messed up. I'll take a look at it tomorrow.

I fixed the composer.lock file.

psy/psysh uses a library from good ol' @dnoegel :open_mouth: :+1:

This PR was closed because your change implements a feature which does not fit to our road map and therefore won't be implemented in the near future.
For more information about our contribution guidelines, see https://developers.shopware.com/contributing/contribution-guideline/.


error

Challenge http://www.freecodecamp.com/challenges/waypoint-use-conditional-logic-with-ifelse-statements has an issue. Please describe how to reproduce it, and include links to screenshots if possible.

@vishdha thanks for posting this issue!
Take a look at this.
And next time, please, provide a description of your issue, thanks.

Please review the Guidelines for Contributing, thank you!.


[ISSUE-148] Remove the existing TestCase and create a new one

Related Issues
<!-Please enter the relevant issue number (# 000). ->
https://github.com/LeeKyoungIl/illuminati/issues/148
Changes and Reasons
As the Source code kept changing, I didn't care about the Test cases
As a result, the number of test cases that did not work properly increased.
Delete all existing tests and create new ones.
And it solves the issues in the sonarcube.
PR Point
<!-Please write what you would like the reviewers to focus on->
Note
<!-Please write down any references. ->
none

build please

build please

build please

build please

build please

build please

build please

build please

build please

build please

build please

build please


Made test failures not fail the package build.

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[X] The toolchain has been rebuilt successfully (or no changes were made to it)
[X] The toolchain/worker package manifests are up-to-date
[X] Any updated packages successfully build (or no packages were changed)
[X] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[X] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[X] All package sources are available
[X] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[X] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[X] All source files have up-to-date hashes in the *.signatures.json files
[X] sudo make go-tidy-all and sudo make go-test-coverage pass
[X] Documentation has been updated to match any changes to the build system
[X] Ready to merge

Summary

My changes in #7764 introduced a bug, which caused ptest failures to fail the whole build. The intended behaviour is for the test failures to be reported in the summary but the package build itself succeed.
The change keeps failing ptests reported as failures but the test nodes themselves are still marked as available, so the top goal node depending on them can unblocked and thus mark the whole build as finished.
Change Log

Made test failures not fail the package build.

Does this affect the toolchain?

No.
Associated issues

#7764

Test Methodology

Local test build.
Buddy build of a package with a known failing test.

Overriding pipeline build check - the pipeline is currently down due to infrastructure issues. Relying on the linked buddy build instead.


Bigger QR code

The QR-code on the pass should be nearly the full width, as it currently is very hard to scan at shops, etc. where most people use old phones with bad cameras.

Duplicate of #75


Error while training

The process stops while running the evaluation step for the model.

Does the retrieval of your chunks have the same dimensions?


Version Packages

This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated.
Releases
<EMAIL_ADDRESS>Patch Changes

#834 f15d9e2 Thanks @threepointone! - fix: default import from node: builtins should work

 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.


Textfield selection can scroll outside the container in iOS

https://user-images.githubusercontent.com/74231510/180356622-2cccbb54-94d5-456f-ad60-c665ad26853e.mov
https://user-images.githubusercontent.com/74231510/180356622-2cccbb54-94d5-456f-ad60-c665ad26853e.mov

Code sample

import 'package:flutter/material.dart';

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      home: MyHome(),
    );
  }
}

class MyHome extends StatelessWidget {
  var titleCtr = TextEditingController();
  var contentCtr = TextEditingController();

  @override
  Widget build(BuildContext context) {
    return Container(
      color: Colors.yellow,
      child: SafeArea(
        child: Column(
          children: [
            // A top bar.
            Container(
              height: 40,
              color: Colors.green,
              child: Row(
                children: [
                  Spacer(),
                  GestureDetector(
                    onTap: (){
                      FocusScope.of(context).requestFocus(FocusNode());
                    },
                    child: Icon(Icons.arrow_downward),
                  ),
                ],
              ),
            ),
            // Adjust the container height according to the keyboard so that the cursor will not be blocked.
            Container(
              height: MediaQuery.of(context).size.height -
                  MediaQuery.of(context).padding.top -
                  MediaQuery.of(context).padding.bottom -
                  MediaQuery.of(context).viewInsets.bottom -
                  40,
              color: Colors.white,
              // Wrap two TextFields in a scrollable widget so that they can scroll simultaneously.
              child: SingleChildScrollView(
                child: Column(
                  children: [
                    // This is for entering the title of the article.
                    Material(
                      color: Colors.white,
                      child: TextField(
                        controller: titleCtr,
                        maxLines: 1,
                        decoration: const InputDecoration(
                          filled: true,
                          fillColor: Colors.transparent,
                          border: InputBorder.none,
                          hoverColor: Colors.transparent,
                          hintText: "input title",
                          counterText: "",
                          hintStyle: TextStyle(
                            fontWeight: FontWeight.normal,
                            color: Colors.grey,
                          ),
                        ),
                      ),
                    ),
                    // This is for entering the content of the article.
                    Material(
                      color: Colors.white,
                      child: TextField(
                        controller: contentCtr,
                        maxLines: null,
                        decoration: const InputDecoration(
                          filled: true,
                          fillColor: Colors.transparent,
                          border: InputBorder.none,
                          hoverColor: Colors.transparent,
                          hintText: "input content",
                          counterText: "",
                          hintStyle: TextStyle(
                            fontWeight: FontWeight.normal,
                            color: Colors.grey,
                          ),
                        ),
                      ),
                    ),
                  ],
                ),
              )
            ),
          ],
        ),
      ),
    );
  }
}

flutter doctor -v
[✓] Flutter (Channel stable, 3.0.1, on macOS 12.2.1 21D62 darwin-arm, locale en-US)
    • Flutter version 3.0.1 at /Users/zhonghanyu/Library/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision fb57da5f94 (9 weeks ago), 2022-05-19 15:50:29 -0700
    • Engine revision caaafc5604
    • Dart version 2.17.1
    • DevTools version 2.12.2

[✓] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
    • Android SDK at /Users/zhonghanyu/Library/Android/sdk
    • Platform android-32, build-tools 32.1.0-rc1
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

[!] Proxy Configuration
    • HTTP_PROXY is set
    ! NO_PROXY is not set

[✓] Connected device (3 available)
    • iPhone 13 Pro (mobile) • 0F7287C2-03B7-4EC4-9D5C-96C2D560B563 • ios            •
      com.apple.CoreSimulator.SimRuntime.iOS-15-2 (simulator)
    • macOS (desktop)        • macos                                • darwin-arm64   • macOS 12.2.1 21D62
      darwin-arm
    • Chrome (web)           • chrome                               • web-javascript • Google Chrome
      103.0.5060.134

[✓] HTTP Host Availability
    • All required HTTP hosts are available

Hi @Huahua2021
For the issue to be workable, it needs to be reproducible with a completed and minimal reproducible code sample that doesn’t include 3rd party plugins or complex production code. Please provide such a sample.
And please provide the output of flutter doctor -v, iOS device information as well.
Also, I can not reproduce the issue with below sample code on the latest stable and master channels:

Sample code
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
        visualDensity: VisualDensity.adaptivePlatformDensity,
      ),
      home: const MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({Key? key, required this.title}) : super(key: key);

  final String title;

  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  int _counter = 0;

  void _incrementCounter() {
    setState(() {
      _counter++;
    });
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text(widget.title),
      ),
      body: Center(
        child: Column(
          mainAxisAlignment: MainAxisAlignment.center,
          mainAxisSize: MainAxisSize.min,
          children: <Widget>[
            const Text(
              'An example title',
              style: TextStyle(color: Colors.black),
            ),
            Container(
              color: Colors.amber,
              width: 400,
              height: 200,
              child: const TextField(
                maxLines: 10,
              ),
            ),
          ],
        ),
      ),
      floatingActionButton: FloatingActionButton(
        onPressed: _incrementCounter,
        tooltip: 'Increment',
        child: const Icon(Icons.add),
      ),
    );
  }
}

Demo
https://user-images.githubusercontent.com/104349824/180189222-46022236-e07a-4430-b872-766f59039586.mov

flutter doctor -v
[✓] Flutter (Channel stable, 3.0.5, on macOS 12.2.1 21D62 darwin-x64, locale en-VN)
    • Flutter version 3.0.5 at /Users/huynq/Documents/GitHub/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision f1875d570e (9 hours ago), 2022-07-13 11:24:16 -0700
    • Engine revision e85ea0e79c
    • Dart version 2.17.6
    • DevTools version 2.12.2

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/huynq/Library/Android/sdk
    • Platform android-33, build-tools 31.0.0
    • ANDROID_HOME = /Users/huynq/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

[✓] IntelliJ IDEA Community Edition (version 2020.3.3)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] IntelliJ IDEA Community Edition (version 2022.1.1)
    • IntelliJ at /Users/huynq/Library/Application Support/JetBrains/Toolbox/apps/IDEA-C/ch-0/221.5591.52/IntelliJ IDEA CE.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.69.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.44.0

[✓] Connected device (3 available)
    • SM T225 (mobile) • R9JT3004VRJ • android-arm64  • Android 11 (API 30)
    • macOS (desktop)  • macos       • darwin-x64     • macOS 12.2.1 21D62 darwin-x64
    • Chrome (web)     • chrome      • web-javascript • Google Chrome 103.0.5060.114

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

[✓] Flutter (Channel master, 3.1.0-0.0.pre.1775, on macOS 12.2.1 21D62 darwin-x64, locale en-VN)
    • Flutter version 3.1.0-0.0.pre.1775 on channel master at /Users/huynq/Documents/GitHub/flutter_master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision a400d132c1 (2 hours ago), 2022-07-20 21:04:06 -0400
    • Engine revision 7892b36d1e
    • Dart version 2.19.0 (build 2.19.0-16.0.dev)
    • DevTools version 2.15.0

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/huynq/Library/Android/sdk
    • Platform android-33, build-tools 31.0.0
    • ANDROID_HOME = /Users/huynq/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 13E113
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

[✓] IntelliJ IDEA Community Edition (version 2020.3.3)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] IntelliJ IDEA Community Edition (version 2022.1.1)
    • IntelliJ at /Users/huynq/Library/Application Support/JetBrains/Toolbox/apps/IDEA-C/ch-0/221.5591.52/IntelliJ IDEA CE.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.69.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.44.0

[✓] Connected device (3 available)
    • SM T225 (mobile) • R9JT3004VRJ • android-arm64  • Android 11 (API 30)
    • macOS (desktop)  • macos       • darwin-x64     • macOS 12.2.1 21D62 darwin-x64
    • Chrome (web)     • chrome      • web-javascript • Google Chrome 103.0.5060.134

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

Hi @Huahua2021 For the issue to be workable, it needs to be reproducible with a completed and minimal reproducible code sample that doesn’t include 3rd party plugins or complex production code. Please provide such a sample. And please provide the output of flutter doctor -v, iOS device information as well.
Also, I can not reproduce the issue with below sample code on the latest stable and master channels:
Sample code
Demo
flutter doctor -v

Thanks for your directions. I have modified my issue. I don't understand why my video isn't playing correctly, so I attached the address of the demo video.

Hi @Huahua2021
I see there are some issues addressing the case you described #59229, #13182
Closing as a duplication, so please follow up on them for further updates.


Vectrino averaging interval needs to be later than performance

Should have a Run.t1_vec variable, though it looks like right now the wake measurements were used to pick the steady interval.

Note that results compared with https://github.com/UNH-CORE/RVAT-baseline are very similar, so this likely is not an issue.


Document the variant parameter types

Many methods in MS Office COM API define parameter types as VT_VARIANT (Variant).
This is set in the Data Type column in the documentation. As a developer I must read the Description to know what is the data type of an actual parameter.
Can you create a new column, Expected Data Type which will containt an actual type name (bool, string, enum name, etc.) so the documentation provides more structured description of method parameter types?
I understand not all parameters have data type documented in the Description - you can leave the column empty, or include the Variant again. But even having just a few more parameters documented in more structured way will help.
PS: This will greatly help in development of the NetOffice library, which provides .NET wrapper for MS Office applications COM API. Many wrapper methods contain parameters with object type, which is not very helpful. If we can gather these expected data types we can create better wrapper methods.

Hi Joze,
That's a good observation!  I like the idea and will look at the feasibility of the fix. It isn't a quick fix because of the sheer size of the SDK with tens of thousands of reference topics. However, this is something that may be automated.  Stand by for further news :-)
John


Link to bash example in documentation not working

at https://github.com/google/kctf/blob/beta/docs/vrp.md the link to  kCTF bash example challenge https://github.com/google/kctf/tree/master/samples/bash is not working because neither the path nor the file exists

thanks, fixed :)


EIP-1014: Skinny CREATE2

https://eips.ethereum.org/EIPS/eip-1014

@Bhargavasomu is going to handle this one.


Accept yo my man

Sorry, you can't remove the releative and absolute points.
Feel free to reopen the PR once the issue is fixed.


Fix bug with _.last(a, n) where n > a.length and add test

This is also a problem with _.initial(), but I didn't fix that.
_.initial([1,2,3], 4)
[1, 2]

As expected initial is also broken for n > a.length. Are you going to fix that too? Should be trivial.

I wasn't planning on fixing that too. last() was the one that was giving me grief. :)


Added zensen-router to router docs list

Just adding my router to the router list.

Our list of routers is growing, which isnt really helping developers. We are having some discussions internally what we want to, probably set some criteria to be listed. We will get back to you on this.

Hi @travistrue2008 , would you mind filling the following template?
- [my-router](https://github.com)
  - **Features:**
    - (max 4)
  - **When to use this router:**
    - example
  - **When not to use this router:**
    - example
  - **Used by:**
    - [Link to demo](), [link to app](), etc.


Ignore other bots (+B mode)

If +B is set on a user, Cardinal should ignore their messages

Started looking at this one - Do you have an example of what Cardinal sees when the decoder is running inbot.py?
I'm assuming we'd need to store the name from the message line (assuming it's even held in there) and if the name includes +B then do not send the self.event_manager.fire("irc.raw", command, line) and instead break?

@GuyT-DevOps You can setup the logger to show raw messages from the server - change this line to include the "file" and or "console" handler (Docker logs use the console handler): https://github.com/JohnMaguire/Cardinal/blob/d3262193e2a50610144d55f78a43b2ef4e2d5c74/config/config.example.json#L75
Alternatively you can listen for irc.raw in a plug-in and log the values.
I'm not positive if the IRC spec includes anything about showing +B mode in the messages from the server.
It may be necessary to make a NAMES request, and maybe even WHO requests or similar to determine if bot modes are set.

https://ircv3.net/specs/extensions/multi-prefix-3.1
There is this which will show all prefixes in NAMES. I'm not sure if this is only channel membership or includes user modes. Probably just channel membership.
It may be necessary to make a NAMES/WHO call for the current channel, followed by WHOIS for users, to determine who is a bot. It may be the case that this is too slow/spammy. I am also not sure that we currently have WHOIS support built in. If we were to build it, it should probably work very similarly to the WHO command.

OK, did some more digging -
There's quite a few different ways servers support this mode. They are mostly summed up here: https://github.com/ircv3/ircv3-ideas/issues/43#issuecomment-640027381
It looks like Unreal is considering adding a message tag similar to InspIRCd.
I'm not sure there's really a good, full solution to this today.

I think this is blocked on coming up with some method for actually detecting bots.

I think this is blocked on coming up with some method for actually detecting bots.

In the meantime, is there any present way to configure an ignore_users style list to prevent Cardinal (globally) from responding to known bots or integrations?


core-metadata  v2   /device/name/Camera001 with trailing slash got 404

🐞 Bug Report
Affected Services [REQUIRED]

 The issue is located in:
core-metadata
Is this a regression?

 Yes, the previous version in which this bug was not present was: ....
Description and Minimal Reproduction [REQUIRED]
🔥 Exception or Error

http://localhost:59882/api/v2/device/name/Camera001/

🌍  Your Environment
Deployment Environment:
docker desktop
EdgeX Version [REQUIRED]:
jakarta
Anything else relevant?
http://localhost:59882/api/v2/device/name/Camera001 + /   report 404 ,
http://localhost:59882/api/v2/device/name/Camera001 works

this behavior meets the API design

This is indeed following the API specs; see https://app.swaggerhub.com/apis-docs/EdgeXFoundry1/core-command/2.2.0
As discussed in the core call; this is NOT a regression and is consistent across other native components and also via the proxy:
$ curl http://localhost:59880/api/v2/event/device/name/Random-Integer-Device
{"apiVersion":"v2","statusCode":200,"totalCount":0,"events":[]}
$ curl http://localhost:59880/api/v2/event/device/name/Random-Integer-Device/
404 page not found
#
# proxy:
$ curl --insecure -H 'Authorization: Bearer '$JWT https://localhost:8443/core-data/api/v2/event/device/name/Random-Integer-Device
{"apiVersion":"v2","statusCode":200,"totalCount":0,"events":[]}
$ curl --insecure -H 'Authorization: Bearer '$JWT https://localhost:8443/core-data/api/v2/event/device/name/Random-Integer-Device/
404 page not found

If needed, this can be changed by either:

Add additional proxy routes with forward slash
Change proxy paths to be regex with optional trailing forward slash
Enabling strict slash on the router (see https://pkg.go.dev/github.com/gorilla/mux#Router.StrictSlash). However, the redirection will not work as expected behind the proxy if the router is unaware of the context paths.

Closing ticket. Thanks for the analysis @farshidtz


LibWeb/Crypto: Fix sizes being passed into generate_aes_key()

Previously, callers were passing the size in bytes, but the method expected bits. This caused a crash in LibCrypto when verifying the key size later on.
Also make the naming of local variables and parameters a little more clear between the different AES algorithms :^)
cc: @BenWiederhake

These WPT tests are now all green :)
https://wpt.live/WebCryptoAPI/generateKey/successes_AES-CBC.https.any.worker.html
https://wpt.live/WebCryptoAPI/generateKey/successes_AES-CBC.https.any.html


⚠ ClamAV has degraded performance

In afa5ba6, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 4290 ms

Resolved: ClamAV performance has improved in 4e7916a.


Various accessibility problems in the admin screens

Issue Summary
There are various problems in the Admin UI of Ghost(Pro) that make it difficult for screen reader users to operate Ghost. These include missing labels for graphical buttons, missing roles for actionable items, and other semantic problems with custom markup.
To Reproduce
With a screen reader such as VoiceOver on the Mac and a compatible browser such as Safari or Chrome, or on Windows, using NVDA and Firefox, navigate to the main admin screen and the post screens.
Examples:

The button to expand and collapse post types has no label.
On mobile, the Nav bar More item doesn't tell VoiceOver that it's a button.
When creating or editing a post, the button to add a card has no label.
When viewing the post settings side bar, the Close button has no label.
The side bar is visible to screen readers even when it is hidden visually.

These make it more difficult for blind users to operate the Ghost admin UI.
Technical details:

Ghost Version: 3.16.1, but also happens in 3.17.1.
Node Version: 12 LTS
Browser/OS: Firefox 76 on Windows 10, Safari 13 on MacOS Catalina, Safari 13 on iOS 13.5.
Database: Default.

I have created some pull requests that deal with these issues except the side bar that is only visually hidden. Someone who is not blind should deal with that one: The side bar needs to be either display:none; or visibility:hidden; via CSS, or if that's not possible, aria-hidden="true" if it is hidden, otherwise aria-hidden should be removed to make it show up for screen readers.
Pull requests:
https://github.com/TryGhost/Ghost-Admin/pull/1583
https://github.com/TryGhost/Ghost-Admin/pull/1584

Hey @MarcoZehe thanks for this, really appreciate it.
Will get someone to figure out what the best solution for the sidebar is, thanks for the advice.

Thank you @ErisDS ! I have one more that deals with various Close buttons throughout the UI.
https://github.com/TryGhost/Ghost-Admin/pull/1585

This is still present in current Ghost.


Basic Integration with Datafusion

I had some fun, working with Datafusion and I want to share the (rough) design with you.
As outlined in #242 I went ahead and:

created a new crate integrations
a subfolder datafusion with its respective modules (catalog.rs, schema.rs, etc.)
implemented some of the traits e.g. CatalogProvider
basic test infra for integration test with HiveMetastore

The overall structure should be extensible, meaning - if we want to support an integration with e.g. polars we simply create a new subfolder and implement the traits.
The test infra is supposed to be used by every integration.
The IcebergCatalogProvider, for example, should be usable by any Catalog (since it Arc for the client (dynamic dispatch))
I really would appreciate your thoughts on this approach - so we can discuss how to proceed, whats missing upstream (like #277), etc.

@liurenjie1024 @ZENOTME @Fokko
PTAL and let me know what you think.

Thanks @marvinlanhenke This is amazing, I'll take a review later!

Thanks! Sorry for replying late. I think this is a good start for the integration work. And I have completed https://github.com/apache/iceberg-rust/pull/277. Maybe we can convert this PR to ready for review now. @marvinlanhenke

Thanks! Sorry for replying late. I think this is a good start for the integration work. And I have completed #277. Maybe we can convert this PR to ready for review now. @marvinlanhenke

thanks for the feedback - i made some minor changes based on the suggestions; and converted this PR into ready for review. If we agree on the basic design here, we could merge and split the actual impl into multiple issues.
@liurenjie1024 @ZENOTME @Fokko @Xuanwo @sdd @tshauck PTAL - especially, the part about "feature-flags" I have no idea whats best practice in 🦀

I've left some comment to improve, but it looks great! I'll invite datafusion community to help review.

@liurenjie1024
Thanks for the review.
I fixed most of the basic issues regarding structure, naming, async-trait etc.
I'll work on the missing impls over the next couple of days - and see how far we can get with the current state of iceberg-rust.

@liurenjie1024 @ZENOTME @viirya @simonvandel
...I just went ahead and pushed the recent updates; PTAL
unresolved / todo:

[ ] proper integration tests for table scan (requires us to setup a table with actual snapshots / data )
[ ] a cache for IcebergCatalogProvider and IcebergSchemaProvider / so data does not become stale
[ ] improve impl ExecutionPlan for IcebergTableScan (once we support filter pushdown, etc.)

@liurenjie1024 @ZENOTME @viirya @simonvandel ...I just went ahead and pushed the recent updates; PTAL
unresolved / todo:

[ ]  proper integration tests for table scan (requires us to setup a table with actual snapshots / data )
[ ]  a cache for IcebergCatalogProvider and IcebergSchemaProvider / so data does not become stale
[ ]  improve impl ExecutionPlan for IcebergTableScan (once we support filter pushdown, etc.)

Hi, @marvinlanhenke How about opening a tracking issue to track the issues related to datafusion integration? I guess we will have more things to do, and we can edit on that tracking issue.

Let's wait a moment to see what others think about the catalog snapshot problem. cc @Xuanwo @Fokko @viirya @sdd PTAL

Let's wait a moment to see what others think about the catalog snapshot problem. cc @Xuanwo @Fokko @viirya @sdd PTAL

If it turns out, that the PR is okay (for now) - let me update the missing docs; before we merge - should make the follow up work a lot easier in a couple of days/weeks.

I've addressed those issues raised by @Xuanwo - thanks for the review.
I think we can merge this PR for now - and continue working on #357;
especially the caching issues; and proper integration testing with data.
@Fokko @liurenjie1024
I haven't checked now, but from memory I believe py-iceberg uses pyspark to setup tables with actual data for proper integration testing? Would this make sense to tackle such an issue in the next weeks in order to provide proper integration testing? Or should we wait once #349 lands and we have a dedicated crate for e2e testing?

Thanks for working on this @marvinlanhenke . Looks good to me. There are some more integrations with DataFusion I'm interested to work with like improving ExecutionPlan for IcebergTableScan. Let's move this forward.


Type T[] is not assignable to type Draft<T[]>

🐛 Bug Report
I am trying to write a generic reducer function, but running into some errors.
My reducer is to update a list of some type with functions like get the list, update an item, delete an item etc. I am getting a static typing error on the line ...
 draft.data = action.listPayload || [] as T[]
The error is Type T[] is not assignable to type Draft<T[]>
Is what i am trying to achieve even possible?
To Reproduce
interface CrudState<T> extends AsyncSource {
    data: T[]

}

  export enum CrudActions {
    GET_LIST_SUCCESS,
    GET_LIST_ERROR,
    GET_LIST_INIT,
    DELETE_ITEM_SUCCESS,
    DELETE_ITEM_ERROR,
    DELETE_ITEM_INIT,
    GET_ITEM_SUCCESS,
    GET_ITEM_ERROR,
    GET_ITEM_INIT,
    PUT_ITEM_INIT,
    PUT_ITEM_SUCCESS,
    PUT_ITEM_ERROR,
    UPDATE_ITEM_INIT,
    UPDATE_ITEM_ERROR,
    UPDATE_ITEM_SUCCESS
  }

      export interface CrudItemAction<T> {
        type: CrudActions
        itemPayload?: T
        listPayload?: T[]

      }

export const crudReducer = <T> (state: CrudState<T>,
    action: CrudItemAction<T>
  ): CrudState<T> => {

    const result =  produce(state, draft => {

        switch (action.type) {
            case CrudActions.GET_LIST_INIT:
                draft.loading = true
                return draft
            case CrudActions.GET_LIST_ERROR:
                    draft.loading = false
                    draft.error = true
                    return draft
            case CrudActions.GET_LIST_SUCCESS:
                    // this line errors with  Type T[] is not assignable to type Draft<T[]>
                    draft.data = action.listPayload || [] as T[]
                    draft.loading = false
                    return draft
            default:
                return draft
        }
    }) as CrudState<T>
    return result
  }

Expected behavior
Be able to update the draft
Environment

Immer version: 2.1.4
[ ] Occurs with setUseProxies(true)
[ ] Occurs with setUseProxies(false) (ES5 only)

Issues with generics have been brought up before. Try searching a little.

Closing this now. Let me know if you couldn't find a solution.

Hi, i did find a solution that worked with 5.0.0 but on upgrading to 5.3.4 i now get type issues again
Below is my reducer code.
The two lines that are not reporting errors are ...
return new Map([...draft, ...toAddNew])
This reports - 'Draft' is assignable to the constraint of type 'T', but 'T' could be instantiated with a different subtype of constraint 'RemovableResource'.
and
const replaceWith : Draft<T> = {...action.payload[0]}
This reports - Type 'RemovableResource' is not assignable to type 'Draft'
Any help would be massively appreciated.
export const projectResourceReducer = <T extends RemovableResource>() => (
  state: Map<string, T>,
  action: ProjectAssetAction<T>
): Map<string, T> => {
  return produce<Map<string, T>>(state, draft => {
    switch (action.type) {
      case REFRESH_ACTION: {
        return new Map(
          (action.payload || []).map(x => [action.idFunction(x), x])
        )
      }
      case ADD_OR_UPDATE_ACTION: {
        const toAddNew = new Map(
          (action.payload || []).map(x => [action.idFunction(x), x])
        )
        return new Map([...draft, ...toAddNew])
      }
      case REPLACE_ITEM: {
        console.log(action)

        if (action.payload) {
          const replaceWith : Draft<T> = {...action.payload[0]}
          draft.set(action.idFunction(action.payload[0]), replaceWith)
        }
        return draft
      }
      case REMOVE_START_ACTION: {
        if (action.deleteId) {
          const existing = draft.get(action.deleteId)
          if (existing) {
            const updated = {
              ...existing,
              deleting: true,
              deleteFailed: false
            }
            draft.set(action.deleteId, updated)
          }

          return draft
        }
        return draft
      }
      case REMOVE_ERROR: {
        if (action.deleteId) {
          const existing = draft.get(action.deleteId)
          if (existing) {
            const updated = {
              ...existing,
              deleting: false,
              deleteFailed: true
            }

            draft.set(action.deleteId, updated)
          }
        }
        return draft
      }
      case REMOVE_COMPLETE_ACTION: {
        if (action.deleteId) {
          draft.delete(action.deleteId)
        }
        return draft
      }
      default:
        return draft
    }
  })
}

@ed-sparkes please don't respond on closed issues, nobody will see them except those that took part on the original questions.
Check https://immerjs.github.io/immer/docs/typescript#cast-utilities.
If that doesn't help open a new issue including a sandbox reproduction


Build custom device for 2021

[x] This contribution adheres to CONTRIBUTING.md.

What does this Pull Request accomplish?
Enables building the custom device for LabVIEW/VeriStand 2021.
Why should this Pull Request be merged?
VeriStand 2021 needs a 64-bit build of the custom device with LabVIEW 2021 64-bit.
What testing has been done?
Waiting on PR build.

Approved pending a successful build.


Showing All Issues The package product 'LanguageServerProtocol' requires minimum platform version 10.15 for the macOS platform, but this target supports 10.13

Showing All Issues
The package product 'LanguageServerProtocol' requires minimum platform version 10.15 for the macOS platform, but this target supports 10.13

What is "this target" here? Are you trying to this package with something that needs to support 10.13?

update the latest version look good，thanks
Matt Massicotte @.***> 于2024年8月1日周四 19:44写道：

What is "this target" here? Are you trying to this package with something
that needs to support 10.13?
—
Reply to this email directly, view it on GitHub
https://github.com/ChimeHQ/LanguageServerProtocol/issues/21#issuecomment-2262834783,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BAEK32KDOA33YE6VYUVFXMDZPINTLAVCNFSM6AAAAABL2ILN4WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENRSHAZTINZYGM
.
You are receiving this because you authored the thread.Message ID:
@.***>

ok great! sorry for any confusion. There was some churn here a while ago.


DRILL-8251: Upgrade hadoop 2 due to CVE

DRILL-8251: Upgrade hadoop 2 due to CVE
Description
Relates to https://github.com/apache/drill/security/dependabot/21
Documentation
Unknown
Testing
CI build

LGTM +1
@jnturton Is there anything special we need to do for this before we merge?

When we update our Hadoop 3 libs we must also update the winutils binaries we ship for Windows. But in the case of Hadoop 2 we do not support Windows, and I can't imagine anyone out there who wants that, so LGTM.

@pjfanning It looks like you have two approvals.  If you want to mark this as ready for review, we can merge.   Thanks for this!

@cgivre thanks - marked as ready to review


Need to fix formatting and IntelliJ plugin grammar for the new function changes

PR #20770 modifies the grammar related to functions and methods. This breaks the formatter in the tooling side. The formatting logic needs to be updated accordingly.

@pubudu91 Kindly note that the ballerina grammar definition changes affects IDEA plugin as it requires to update the plugin grammar definitions accordingly. Therefore shall we keep this issue opened till the plugin side fixes are done?

+1

Closing since the required changes for IntelliJ plugin and formatting are added with the referred PRs.


IRGen: fix get/store enum tag functions for big-endian platforms

Use sized loads and stores, rather than memcpy calls, to get and
set tag values in the payload and extra tag bytes. The code works
the same regardless of the endianness of the target platform.

@jckarter Can you take a look at this? I'd also appreciate it if you could trigger the benchmark regression CI tests.
It's a similar fix to https://github.com/apple/swift/pull/19832 however I've approached the problem slightly differently. Instead of fixing up addresses on big endian platforms I've rewritten the code generator so that it no longer depends on the endianness of the target platform at all.
Since this code handles enums dynamically I'm not sure it makes sense to move it under the EnumPayload umbrella just yet.  We could perhaps come up with another abstraction (EnumPayloadDynamic maybe?).
Thanks.

@swift-ci Please benchmark

What does this achieve? It seems like a pretty severe code size cost, assuming LLVM doesn't just fold this back to a memcpy.

Performance: -O

TEST
OLD
NEW
DELTA
RATIO

Regression

EqualStringSubstring
38
43
+13.2%
0.88x

EqualSubstringString
38
43
+13.2%
0.88x

LessSubstringSubstring
39
44
+12.8%
0.89x

EqualSubstringSubstring
38
42
+10.5%
0.90x (?)

EqualSubstringSubstringGenericEquatable
38
42
+10.5%
0.90x (?)

LessSubstringSubstringGenericComparable
39
43
+10.3%
0.91x

StringAdder
414
452
+9.2%
0.92x (?)

StringBuilderSmallReservingCapacity
330
355
+7.6%
0.93x (?)

Performance: -Osize

TEST
OLD
NEW
DELTA
RATIO

Regression

EqualSubstringSubstring
38
43
+13.2%
0.88x

EqualStringSubstring
38
43
+13.2%
0.88x

LessSubstringSubstring
39
44
+12.8%
0.89x (?)

EqualSubstringSubstringGenericEquatable
39
44
+12.8%
0.89x (?)

EqualSubstringString
39
44
+12.8%
0.89x (?)

LessSubstringSubstringGenericComparable
39
44
+12.8%
0.89x

Improvement

StringBuilderLong
1330
1210
-9.0%
1.10x (?)

Performance: -Onone

TEST
OLD
NEW
DELTA
RATIO

Regression

EqualSubstringSubstringGenericEquatable
44
48
+9.1%
0.92x (?)

LessSubstringSubstringGenericComparable
44
48
+9.1%
0.92x

Improvement

ArrayOfPOD
855
777
-9.1%
1.10x (?)

CharIndexing_tweet_unicodeScalars
1189480
1100320
-7.5%
1.08x (?)

How to read the data
The tables contain differences in performance which are larger than 8% and
differences in code size which are larger than 1%.
If you see any unexpected regressions, you should consider fixing the
regressions before you merge the PR.
Noise: Sometimes the performance results (not code size!) contain false
alarms. Unexpected regressions which are marked with '(?)' are probably noise.
If you see regressions which you cannot explain you can try to run the
benchmarks again. If regressions still show up, please consult with the
performance team (@eeckstein).

Hardware Overview
  Model Name: Mac Pro
  Model Identifier: MacPro6,1
  Processor Name: 12-Core Intel Xeon E5
  Processor Speed: 2.7 GHz
  Number of Processors: 1
  Total Number of Cores: 12
  L2 Cache (per Core): 256 KB
  L3 Cache: 30 MB
  Memory: 64 GB

Thanks for taking a look.

What does this achieve?

We need to change this code to make it work on big-endian platforms. Without patching this code we can't run the tests to completion (with this PR we fail 11 validation tests at this point). memcpy isn't a portable way to copy data between values of different sizes. We can fix up the addresses to make it work but this way is much cleaner and more robust IMO.

It seems like a pretty severe code size cost

It shouldn't cost anything really, and the benchmarks appear to back that up (except perhaps for the SubstringString comparison functions that I'll investigate). The new code doesn't cost any more because the old code called the emitMemCpy function which was manually unrolled to emit fixed size memcpy calls.

There's already a runtime function that does this in the runtime implementation; maybe we could expose that and emit runtime calls instead of open-coding it.

We could replace the new emitGetTag and emitSetTag functions with a runtime call. For the code that sets/gets the payload value however I think the code in this PR is optimal (one fixed sized store followed by a memset to zero the rest of the payload).

It shouldn't cost anything really, and the benchmarks appear to back that up (except perhaps for the SubstringString comparison functions that I'll investigate). The new code doesn't cost any more because the old code called the emitMemCpy function which was manually unrolled to emit fixed size memcpy calls.

I didn't realize we were already open-coding this. It'd be nice to fix that, since small_memcpy already exists in the runtime for this purpose. I think it's used exclusively for enum tag manipulation, so we could adapt it to be the get/set tag entry point that does the right thing for big-endian platforms.

I didn't realize we were already open-coding this. It'd be nice to fix that, since small_memcpy already exists in the runtime for this purpose. I think it's used exclusively for enum tag manipulation, so we could adapt it to be the get/set tag entry point that does the right thing for big-endian platforms.

I've thought about this a bit and I'm not sure it makes sense to call the runtime version of small_memcpy from the generated code. The open coded version included here should compile down to just over 10 instructions on x86_64 (roughly: https://godbolt.org/z/kcmcUl). Though it does take more instructions on machines without fast unaligned accesses. A function call is likely to require (at least) a few instructions anyway and the open coded version probably benefits from very predictable branches.
So if we care about the performance of this generated code then I think we should continue to open code the get/set tag functionality. If on the other hand we don't really care about the performance of this generated code then I think it would make more sense to call the higher level runtime functions instead as that would allow us to simplify IRGen a lot more.
Regardless, to me it makes sense to separate that change from this one to minimize the impact of this change on performance (in case, for example, it causes a regression and needs to be rolled back).

I suspect that code size is more important than performance here, since unspecialized code is generally undesirable on hot paths, so the higher-level we can go with runtime entry points, the better. I'm fine with splitting that off as a different change.

Thanks. I'll take a look at calling out to the runtime but I'd like to merge this PR (or something similar to it) if possible anyway to reduce the number of patches we are maintaining. Are there any changes you'd like me to make?
I can't recreate the performance regressions on my laptop (Intel(R) Core(TM) i5-6267U CPU @ 2.90GHz) so I think the benchmark changes are just (alignment?) noise. I don't think they are hitting the modified code paths anyway.

@swift-ci Please test

Is this OK to be merged now? (I don't have commit access.) Thanks!

Done. Thanks!


Folder Creation

Enable users to create folders within their account for better organization of study decks. Each folder should have a unique name and display the number of decks it contains.
Requirements:
Folder Creation Interface:

Add an input field and a button to create new folders on the dashboard.
Display folder name and deck count once a folder is created.

Feedback for Folder Creation:

Show a confirmation message upon successful folder creation.
Prevent creation of folders with duplicate names.

Acceptance Criteria:

Users can successfully create folders, which appear on the dashboard immediately.
A confirmation message appears upon successful creation.
Attempting to create a folder with a duplicate name prompts an error message.
Folder displays the correct deck count, initially set to zero upon creation.

Issues handled


image allignment

For situations where the origin is in a different place for different designs, would it be better to try to align images for minimum diff?
Plan to add user control's to move layers of RHS around if they are mis-aligned. (manually and with numeric input boxes

implemented by checkbox and arrow keys


Not getting full 4 cores usage on Pi 4

I am trying to do some power measures on with a Yocto build based on the dunfell branch, on a Raspberry Pi 4. Running a bunch of yes > /dev/null & to stress the four cores and using top to monitor them, I expected to see the usage go near 400%, instead I only managed to get about 100%. From the /sys/devices/system/cpu/online file all 4 cores seems to be online.
Is this a known issue?
Under Raspbian running the same commands I get the foreseen 400% usage.

Seems like busybox top shows 100% as the maximum; using htop show that all four cores are at 100% usage.

Seems like busybox top shows 100% as the maximum; using htop show that all four cores are at 100% usage.

Right use top from procps package

Right use top from procps package


Improve Stocator init method

Consider Spark with many workers. When Spark job is submitted, each worker will try to initialize Stocator. This can be easily seen. Just make Spark to access an object with many parts or access a container with multiple objects. Then see the logs of the tasks. You will notice that each task on the worker node will perform Stocator init.
Stocator init method is costly, since it contains token authentication and configuration parsing. This means that each Spark task on each worker will perform the same Stocator init which affects the performance.
We should improve Stocator init method and make it per worker JVM and not per task, Perhaps by using Singelton approach will do the trick, since it's per JVM. This way the init will happen per JVM and not every time per task. Token creation and configuration parsing should be in the Singlelton.

@djalova Can you check it please?


How to merge equinox pretty-printing and jax debug printing?

So I'm trying to do two things here:

Raise a error through the jit boundary using eqx.error_if.
Print a pytree using the jax.debug.print function.

I'm looking to do this because some Nans are arising during my training that are difficult to isolate. I can't use the usual jax debug_nans flag as some of data naturally has Nans present.
Presently I am creating a boolean pytree that checks for any nans on the leaves, and ideally I would be able to print the actually boolean value as opposed to the usual Traced<ShapedArray(bool[])>, which would be done using the jax debug print.
Extending your error_if example:

@eqx.filter_jit
def f(x):
    bool_tree = jax.tree_map(lambda x: np.isnan(x).any(), x)
    vals = np.array(jax.tree_util.tree_flatten(bool_tree)[0])
    msg = "Nan found in tree:\n" + eqx.tree_pformat(bool_tree, short_arrays=False)
    x = eqx.error_if(x, vals.sum() > 0, msg)
    return x

pytree = (np.zeros(3), np.zeros(5))
_ = f(pytree)

nan_pytree = eqx.tree_at(lambda pytree: pytree[0], pytree, np.zeros(3).at[2].set(np.nan))
_ = f(nan_pytree)

However this doesn't print the actual array values, hence the desire for some interface with jax.debug.print. Is there some simple way to achieve this?
Thanks in advance!

So I think we have a couple options.
First of all, if you just want to debug something and want to use jax.debug.print with pretty printing, then this can be done using the following trick:
class PrettyFormat:
    def __init__(self, obj):
        self.obj = obj

    def __repr__(self):
        return eqx.tree_pformat(self.obj, short_arrays=False)

jax.debug.print("{}", PrettyFormat(bool_tree))

This defers the pretty-formatting until the repr is called inside jax.debug.print.
If you actually want to go ahead and include this information inside an eqx.error_if call, however, then right now Equinox doesn support that. It should be possible to add, though: we could imagine adding an eqx.error_if(..., format_with=...) argument that is used to format the string before printing. I'd be happy to take a pull request on this.

Ah that is a very nice solution, awesome! Yeah I think having that extra functionality would be great. I'm happy to build this in when I can find some time in the next few weeks.


[Linux] Autocompletion doesn't seem to obey caps:swapescape xkbmap option

System information:

VSCode Version: 1.12.2 (19222cdc84ce72202478ba1cec5cb557b71163de)
OS Version: Arch Linux 64-bit, updated somewhen last month
Desktop environment: none
Window manager: bspwm
Current keyboard layout: pl

Steps to Reproduce:

do setxkbmap -option caps:swapescape
run vscode
open a file
input some text
try to input a word from the previously input text
autocompletion dropdown shows up to suggest the word
try to dismiss the autocompletion dropdown

Expected behaviour:
The autocompletion dropdown closes when CapsLock is pressed (which due to xkbmap option functions as Escape now).
Actual behaviour:
The autocompletion dropdown stays open when CapsLock is pressed, hides when Escape is pressed (which due to xkbmap option should function as CapsLock now).

@jaen Could you please try our keybinding troubleshooting guide -- https://github.com/Microsoft/vscode/wiki/Keybindings#troubleshoot-linux-keybindings -- and let us know if anything in there helps in this case?

@alexandrudima Sorry, for answering this late, but I've checked it just now and setting "keyboard.dispatch": "keyCode" seems to fix this issue. Does it have any downsides though?

When using "keyboard.dispatch": "keyCode", and using a non-US standard keyboard layout, the rendered keybindings in the UI will not 100% accurately reflect what is printed on the keyboard.

I see. Polish keyboard has the same basic layout as the US one so this shouldn't be an issue for me. Thanks!


[Cypress Automation] - Add data-testid to HTML tags.

Changes:

Added data-testid attributes for various elements to ensure they are more resilient to changes in the UI for cypress automation.

Reason for Changes:
The previous locators were high maintenance in Cypress due to frequent changes in the UI structure. By adding the data-testid attributes, we aim to:

Reduce test failures caused by dynamic changes in the UI.
Enhance the maintainability of the test scripts.
Ensure that our test suite accurately reflects user interactions and behaviors.

ClickUp Card -> [For Cypress] Update locators for Automation
Google Sheet for Locators

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

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

Details

7 of 7 (100.0%)  changed or added relevant lines in 4 files are covered.
3 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.2%) to 85.61%

Files with Coverage Reduction
New Missed Lines
%

src/components/AppHeader/MobileMenu/MobileMenuConfig.tsx
1
85.71%

src/components/AppHeader/AppHeader.tsx
2
93.62%

Totals

Change from base Build<PHONE_NUMBER>9:
0.2%

Covered Lines:
5625

Relevant Lines:
6113

💛  - Coveralls


tqdm needs to be added to requirements.txt

It is a requirement in the conda build recipe, but it should also be in the requirements.txt so those that install with a git clone and pip install -e . also get tqdm.

The alternative would be to make tqdm optional, and remove it from meta.yaml . Which is better? Mostly we've erred on the side of asking for the kitchen sink.


add circle ci config

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@9a2e921). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #54   +/-   ##
=========================================
  Coverage          ?   84.14%
=========================================
  Files             ?       30
  Lines             ?      656
  Branches          ?      108
=========================================
  Hits              ?      552
  Misses            ?       96
  Partials          ?        8

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 9a2e921...6abaf5e. Read the comment docs.


MessageEvent is not of type Event

Describe the bug
For a normal event this test runs fine:
fireEvent(window, new Event('message'));

However this:
fireEvent(window, new MessageEvent('message'));

throws:
TypeError: Failed to execute 'dispatchEvent' on 'EventTarget': parameter 1 is not of type 'Event'.
This test worked previously when using jest
Reproduction

This is when using the jsdom envrionment

System Info
System:
    OS: macOS 12.3.1
    CPU: (8) x64 Intel(R) Core(TM) i7-8569U CPU @ 2.80GHz
    Memory: 186.50 MB / 16.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 16.15.1 - ~/.nvm/versions/node/v16.15.1/bin/node
    npm: 8.11.0 - ~/.nvm/versions/node/v16.15.1/bin/npm
    Watchman: 2<IP_ADDRESS> - /usr/local/bin/watchman
  Browsers:
    Chrome: 103.0.5060.53
    Firefox: 95.0.2
    Safari: 15.4
  npmPackages:
    @vitejs/plugin-react: 1.3.2 => 1.3.2
    vite: 2.9.13 => 2.9.13
    vitest: 0.17.0 => 0.17.0

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.

For any others having this issue, this is a workaround:
const event = new Event('message');
(event as any).data = { eventType: 'test' }
fireEvent(window, event);


Override temporary file write directory

Love the library, but couldn't be used inside of AWS Lambda, because on the /tmp is available for write access. I made it so you can pass in a new temporary directory path and tested it on lambda. Works perfect.
Let me know if something else needs to be done for it, or if you have a better way.

Hi @individual11 @jjwilly16 ,
Will this be implemented anytime soon? Currently using this tool in AWS and would like to have this ability in production. Thanks in advance!

What needs to happen to get this merged?  I'm in the same situation with AWS Lambda, happy to help.
https://github.com/Buddy-Technology/node-pdftk/pull/1

It's just a time thing. I'll try to knock it out this weekend with a new pull request. The code still works when you update manually. I deploy it to lambda via the serverless framework and haven't had a single issue.

@individual11 If you merge my pull request in your forked repository, you shouldn't have to resubmit. Those changes should be reflected here and I can merge this request, if I'm not mistaken.

Ok, did the merge on my fork (sorry for the delay on that, not sure how that slipped through).

@individual11 no worries. Thanks for your help.
I published 2.1.0 on npm with this feature. I kind of rushed it out since so many people are waiting for it. Just let me know if there are problems.

Thank you both for making this happen!


TraceFilter doesn't close span if service return error responce in ResponseEntity object

Hi,
we've faced with the following issue:
There is some Rest controller, and it returns responses with ResponseEntity class. But if request returns error response, TraceFilter won't close it's span.
Close look to TraceFilter showed, that it will close span on error response only if it returns through throwing an exception.
There is an example of the controller:
 @RestController
 public class TestController {
   @NewSpan("TestController.testRequest")
   @RequestMapping(
         method = RequestMethod.POST,
         value = "/test_failure"
   )
   public ResponseEntity processRequestFailRe(){
         return ResponseEntity.badRequest().build();
   }
 }

Which version of Sleuth are you using?

Can you paste a stacktrace with what's going on when you currently do this?

Hi,
I'm not quite sure stack trace from which method could help you, but I've created test application that reproduce this issue:
https://github.com/wirelessMousetail/SpringSleuthWebApplicationSample

Thanks for providing the test and for double checking if our work is done properly :)
Have you actually checked if the span is there in Zipkin? Cause actually it is... The flow looks like this that we go from TraceFilter to TraceHandlerInterceptor. Next, we go back to TraceFilter where we have a method recordParentSpan. Since parent (which is our root span) is remote (it's the remote side of the RPC call) we are sending it to Zipkin. If you turn on the logs to DEBUG you'll see this
2017-10-23 14:05:58.767 DEBUG [bootstrap,,,] 54328 --- [o-auto-1-exec-1] o.s.c.sleuth.instrument.web.TraceFilter  : Received a request to uri [/test_bad_request] that should not be sampled [false]
2017-10-23 14:06:00.852 DEBUG [bootstrap,,,] 54328 --- [o-auto-1-exec-1] o.s.c.sleuth.instrument.web.TraceFilter  : Found a parent span [Trace: e3d46ce28ba263d6, Span: e3d46ce28ba263d6, Parent: null, exportable:true] in the request
2017-10-23 14:06:00.855 DEBUG [bootstrap,e3d46ce28ba263d6,e3d46ce28ba263d6,true] 54328 --- [o-auto-1-exec-1] o.s.c.sleuth.instrument.web.TraceFilter  : Parent span is [Trace: e3d46ce28ba263d6, Span: e3d46ce28ba263d6, Parent: null, exportable:true]
2017-10-23 14:06:03.982 DEBUG [bootstrap,e3d46ce28ba263d6,e3d46ce28ba263d6,true] 54328 --- [o-auto-1-exec-1] o.s.c.s.i.web.TraceHandlerInterceptor    : Handling span [Trace: e3d46ce28ba263d6, Span: e3d46ce28ba263d6, Parent: null, exportable:true]
2017-10-23 14:06:04.502 DEBUG [bootstrap,e3d46ce28ba263d6,e3d46ce28ba263d6,true] 54328 --- [o-auto-1-exec-1] o.s.c.s.i.web.TraceHandlerInterceptor    : Adding a method tag with value [processFail] to a span [Trace: e3d46ce28ba263d6, Span: e3d46ce28ba263d6, Parent: null, exportable:true]
2017-10-23 14:06:04.642 DEBUG [bootstrap,e3d46ce28ba263d6,e3d46ce28ba263d6,true] 54328 --- [o-auto-1-exec-1] o.s.c.s.i.web.TraceHandlerInterceptor    : Adding a class tag with value [ExceptionThrowingController] to a span [Trace: e3d46ce28ba263d6, Span: e3d46ce28ba263d6, Parent: null, exportable:true]
2017-10-23 14:06:28.406 DEBUG [bootstrap,e3d46ce28ba263d6,e3d46ce28ba263d6,true] 54328 --- [o-auto-1-exec-1] o.s.c.sleuth.instrument.web.TraceFilter  : Trying to send the parent span [Trace: e3d46ce28ba263d6, Span: e3d46ce28ba263d6, Parent: null, exportable:true] to Zipkin
2017-10-23 14:06:29.270 TRACE [bootstrap,e3d46ce28ba263d6,e3d46ce28ba263d6,true] 54328 --- [o-auto-1-exec-1] o.s.c.sleuth.instrument.web.SsLogSetter  : Will set SS on the span
2017-10-23 14:08:29.252 DEBUG [bootstrap,e3d46ce28ba263d6,e3d46ce28ba263d6,true] 54328 --- [o-auto-1-exec-1] o.s.c.sleuth.instrument.web.TraceFilter  : Detaching the span [Trace: e3d46ce28ba263d6, Span: e3d46ce28ba263d6, Parent: null, exportable:true] since the response was unsuccessful

and this line is important
2017-10-23 14:06:28.406 DEBUG [bootstrap,e3d46ce28ba263d6,e3d46ce28ba263d6,true] 54328 --- [o-auto-1-exec-1] o.s.c.sleuth.instrument.web.TraceFilter  : Trying to send the parent span [Trace: e3d46ce28ba263d6, Span: e3d46ce28ba263d6, Parent: null, exportable:true] to Zipkin

So we don't explictly call close but we do report the span manually and detach it from the current thread.

Yes, We've checked span in Zipkin, and it isn't in it for /test_bad_request (It's in it for /test_ok request).
And why do you suppose that parent span is remote?
Looks like you've run some modified version of application. But in the shared application parent span is not remote. Could you, please, check it (I've added spring-cloud-starter-zipkin to dependencies, so it can be connected).

I've ran your sample as a test in sleuth. I'll run your example again later and double check. Maybe in my tests i had some conditions you don't face in your app.

Thanks, this fix works fine.
Could you please tell me, when release with this fix is expected?

Here you have the calendar of our releases - https://github.com/spring-cloud/spring-cloud-release/milestones .


Making this gem independently operable

Hi @danchoi, I wanted to ask before I proceed because this change would add a new dependency to the gem: What do you think about using the kindlegen gem (https://github.com/tdtds/kindlegen) to download kindlegen program and execute regardless of the system specific PATH variable? Making this gem independently operable would help a ton if anyone would use this gem when deploying to a PaaS such as Heroku. Also maybe adding a few more methods instead of just making it a one-run app would be good improvement as well. If you agree, I'll fork and make some changes accordingly.

@emiraydin Thank you for the suggestion. I like the idea & added you as a collaborator to this repo. Please feel free to fork and then merge your changes. You can also add your name to the author section of the README.

I checked the kindlegen gem and it is a bit problematic. It doesn't work on Ruby versions below 2.0.0 and has some bugs. I think it would be smarter to write the download script from scratch instead. I'll get back to this later.

OK
On Mon, Jan 12, 2015 at 02:07 AM, Emir Aydin<EMAIL_ADDRESS>wrote:

from: Emir Aydin<EMAIL_ADDRESS>date: Sun, Jan 11 11:07 PM -08:00 2015
to: danchoi/kindlerb<EMAIL_ADDRESS>cc: Daniel Choi<EMAIL_ADDRESS>reply-to: danchoi/kindlerb<EMAIL_ADDRESS>subject: Re: [kindlerb] Making this gem independently operable (#3)
I checked the kindlegen gem and it is a bit problematic. It doesn't work on Ruby versions below 2.0.0 and has some bugs. I think it would be smarter to write the download script from scratch instead. I'll get back to this later.

Reply to this email directly or view it on GitHub:
https://github.com/danchoi/kindlerb/issues/3#issuecomment-69535232

Sent from Vmail
http://danielchoi.com/software/vmail.html


Include exported DSC resources in the module manifest file

Describe the bug
On push to private repo
WARNING: This module 'C:\Users\fladmin\AppData\Local\Temp\1886670852\cChoco\cChoco.psd1' has exported DscResources. As
a best practice, include exported DSC resources in the module manifest file(.psd1). If your PowerShell version is
higher than 5.0, run Update-ModuleManifest -DscResourcesToExport to update the manifest with ExportedDscResources
field.

To Reproduce
Steps to reproduce the behavior:
Register-PSRepository -Name pwsh -InstallationPolicy Trusted -SourceLocation https://nexus.local/repository/pwsh/ -PackageManagementProvider nuget -PublishLocation https://nexus.local/repository/pwsh/

Publish-Module -Name ".\cchoco" -Repository pwsh -NuGetApiKey "123"

Expected behavior
Clean push without warnings
Software (please complete the following information):

Server 2022
PowerShell Version 5.1
cChoco <IP_ADDRESS> / 2.5.0

I suspect this might be causing the error I'm having right now:
ERROR: The PowerShell DSC resource cChocoInstaller from module <cChoco,<IP_ADDRESS>> does not exist at the PowerShell module path nor is it registered as a WMI DSC resource.

Will try updating my local manifest copy with the resources and submit a fix if that helps.


Implemented Various Flags

This PR implements some flags.

Closing due to commit
https://github.com/MinecraftPortCentral/GriefPrevention/commit/624fa61e165e903768e5288e613207dad28451be