feat(explain_manifest): add ability to test docker images as well

Summary
In this PR, the static analysis part in "Legacy Smoke Test" is moved to "Verify Manifest - Images".
We will move the e2e test part of "Legacy Smoke Test" in sepearte workflow (possibly e2e api tests) and remove
smoke tests.
Not required by 3.4 release.
Checklist

[x] The Pull Request has tests
[na] There's an entry in the CHANGELOG
[na] There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

Full changelog

[Implement ...]

Issue reference
https://github.com/Kong/kong-ee/pull/6024
KAG-1388

good packaging run https://github.com/Kong/kong/actions/runs/5759618530/job/15614121821?pr=11349


Add IMEX functionality to advection-diffusion example (ex23)

Thanks to @mlstowell for the nice DG advection-diffusion example code. This PR implements IMEX-RK time integration (1st and 2nd order for now) in the parallel version. The current solver for the fully-implicit (non-symmetric) system has some issues with large time steps, e.g.
./ex23p -m ../data/periodic-square.mesh -o 4 -dt 1e-1 -d 1e-6

results in GMRES: No convergence!. This may make IMEX time-stepping more attractive.
The IMEX method only solves the symmetric diffusion problem implicitly, so we can use CG instead of GMRES. Also I expect the AMG preconditioner to be more effective for diffusion problems than advection problems.
Of course the (explicit) advective CFL still must be satisfied for the IMEX solvers. However, the time step can be chosen to be independent of the diffusion coefficient.
Another option would be to improve the solvers for advection-dominated problems, but this is probably beyond the scope of a simple example.

This is great, @pazner!  This is precisely what I had wanted to experiment with in this example.
After our discussion last week I also reimplemented this example to handle advection fully implicitly, as you suggested.  Interestingly, the fully implicit version seems to perform better than the IMEX version, at least when the diffusion constant is much larger.  I'll try to modify the example to support both options.  As long as the example remains simple this may be the best choice.
Also, when I try the command line you mention the IMEX version seems to go unstable somewhere between the 45th and 50th time steps.

Yes, the IMEX version is unstable for the example I gave because the time step does not satisfy the advective CFL. There is probably not too much benefit to the IMEX method for this linear problem, especially if we have a good solver for the non-symmetric problem (Cf. @ben-s-southworth). If we want to add a nonlinear advection term things become more interesting.

Good, that means my version of your updates where working correctly.  I'm glad to know that.
I'm actually very pleased to have these IMEX solvers even if they aren't absolutely necessary for this particular example.  Aside from having the IMEX solvers themselves implemented, which is valuable in itself, I also like how you use TimeDependentOperator::ExplicitMult.  I think this is our only example of how this method was intended to be used.
I'd like to move these two IMEX solvers into lining/ode.?pp so that I can also use them in miniapps.  I'd also like to generalize FE_Evolution so that it can be used with fully implicit or IMEX solvers.  Any objections?

I think these are all great ideas

Maybe a bit more comments to follow what is happening as the add and Add functions are unnatural to read.

Good point. I will add some comments/documentation.

I'd like to move these two IMEX solvers into lining/ode.?pp so that I can also use them in miniapps. I'd also like to generalize FE_Evolution so that it can be used with fully implicit or IMEX solvers. Any objections?

The one potential pitfall here is the design of the TimeDependentOperator class. A new interface for something like IMEXSolve would have to be added, so that ImplicitSolve would perform a fully-implicit solve. There is the possibility of some slight confusion because of the naming, but it should be ok.

You make a good point.  I'm often confused when trying to understand the TimeDependentOperator class.  I suspect we could clarify things by using a deeper inheritance hierarchy for different types of TimeDependentOperators tailored to the different flavors of ODE solvers.  Perhaps we should try to put together a proposal independent form this PR.

I added our AIR nonsymmetric AMG method to this and it is very fast on the implicit solves. I will push ex23p with hypre support to the AIR branch soon. However, AIR cannot be pushed to master until we merge it to hypre master, but this will hopefully happen within a few weeks. Question on this - has it been tested in parallel? When I run in parallel I get errors with the add routine,
Verification failed: (internal::hypre_ParCSRMatrixSum(A, beta, B.A) == 0) is false:
--> error in hypre_ParCSRMatrixSum
... in function: mfem::HypreParMatrix &mfem::HypreParMatrix::Add(double, const mfem::HypreParMatrix &)
... in file: /g/g19/bs/quartz/mfem/linalg/hypre.hpp:489

@ben-s-southworth , yes, it fails in parallel.  I created a small PR #885 earlier today to address the issue.

@mlstowell Great, thanks! Pulled that in and it all looks good in parallel. Running a refined version of the problem @pazner posted,
srun -n 216 ./ex23p -m data/periodic-square.mesh -o 4 -dt 1e-1 -d 1e-6 -rs 3 -rp 5
I'm getting 16 GMRES iterations/time step to get 1e-12 relative residual per time step. Am experimenting with when to transition from AIR to AMG depending on h and the diffusion coefficient.

@ben-s-southworth , that sounds very impressive!  I look forward to trying it.

I'm going to pull these improvements into adv-diff-ex-dev and then make changes there to support explicit, implicit, or IMEX ODE solvers. The goal is to show, in one simple example, how the different ODE solver types can be used and provide an easy way to compare them.


Broken styles in build-docs output when migrating from redoc-cli

Describe the bug
When attempting to migrate from redoc-cli to @redocly/cli, I could not obtain a proper standalone HTML file of the documentation. Possibly due to the lack of suitable CSS stylesheets, everything gets jumbled up.
To Reproduce
Given a single OpenAPI yaml file named api.yaml:
openapi: 3.0.0
info:
  description: |

    # Introduction
    This is an introduction.

    <!-- ReDoc-Inject: <security-definitions> -->

  version: 1.0.0
  title: Example title
paths:
  "/api/manage/user":
    post:
      tags: [ Management ]
      summary: "Add a new user"
      description: Creates a new user
      requestBody:
        content:
          application/json:
            schema:
              type: object
              required:
                - "email"
                - "password"
              properties:
                email:
                  type: string
                  description: "A unique e-mail address for the user"
                password:
                  type: string
                  minLength: 8
                  description: "The pass phrase for the user to authenticate"
                name:
                  type: string
                  description: "The display name for the user"
      responses:
        '200':
          description: OK
          content:
            application/json:
              schema:
                type: object
                properties:
                  status:
                    type: string
                    enum: ["success"]
                  message:
                    type: string
components: {}

And this npm package:
{
  "name": "use-redoc",
  "version": "0.1.0",
  "private": true,
  "scripts": {
    "build-html": "redocly build-docs api.yaml -o dist/API.html",
    "build-html-legacy": "redoc-cli bundle api.yaml -o dist/API.html",
    "preview": "redocly preview-docs api.yaml"
  },
  "dependencies": {
    "@redocly/cli": "^1.0.0-beta.126",
    "redoc": "^2.0.0",
    "redoc-cli": "^0.13.21"
  }
}

I ran the build-html command above:
npx @redocly/cli build-docs api.yaml -o dist/API.html

To see this when opening the HTML file:

I originally did not have a redocly.yaml file, so I created one based on the starter project as an attempt to fix these issues:
apis:
  sample@v1:
    root: api.yaml
extends:
  - recommended
rules:
  no-unused-components: error
theme:
  openapi:
    htmlTemplate: ./index.template.html
    theme:
      colors:
        primary:
          main: "#32329f"
    generateCodeSamples:
      languages:
        - lang: curl

Along with this template HTML file named index.template.html:
<!DOCTYPE html>
<html>
  <head>
    <meta charset="utf-8">
    <title>API Reference | ReDoc</title>
    <meta name="viewport" content="width=device-width, initial-scale=1">

    <style>
      body {
        margin: 0;
        padding: 0;
      }
    </style>
    {{{redocHead}}}
  </head>
  <body>
    {{{redocHTML}}}
  </body>
</html>

But the outcome was the same as the screenshot above.
Expected behavior
The command which I used in redoc-cli provides the expected result.
redoc-cli bundle api.yaml -o dist/API.html

It is more bizarre that the preview obtained through redocly preview-docs also shows the document correctly. The problem occurs specifically with build-docs.
Logs
There doesn't seem to be much going on at the docs.
λ npm run build-html

><EMAIL_ADDRESS>build-html
> redocly build-docs api.yaml -o dist/API.html

Found redocly.yaml and using theme.openapi options
Prerendering docs
🎉 bundled successfully in: dist/API.html (15 KiB) [⏱ 21ms].

Redocly Version(s): 1.0.0-beta.126
Node.js Version(s): Reproduced both with 16.20.0 and 18.16.0
Additional context
This is the same effect as the one reported in Redocly/redoc#2212, although it only happens when using the non-deprecated tools.

Thanks for reporting the issue!

Something interesting is going on here, I have no explanation yet but I'll add my observations, because I can replicate, and it's strange.
Using the api.yaml and package.json from above:
Running npm run build-html produces the following:
$ npm run build-html

><EMAIL_ADDRESS>build-html
> redocly build-docs api.yaml -o dist/API.html

Found undefined and using theme.openapi options
Prerendering docs
🎉 bundled successfully in: dist/API.html (15 KiB) [⏱ 1829ms].

The result is unstyled, and I'd like you to note the filesize.
Using npx gives the same result:
$ npx @redocly/cli build-docs api.yaml -o dist/API.html
Found undefined and using theme.openapi options
Prerendering docs
🎉 bundled successfully in: dist/API.html (15 KiB) [⏱ 1620ms].

However if I use the redocly command directly:
$ redocly build-docs api.yaml -o dist/API.html
Found undefined and using theme.openapi options
Prerendering docs
🎉 bundled successfully in: dist/API.html (50 KiB) [⏱ 1708ms].

The generated HTML file has a large additional <style> tag in the source, which explains the larger file size, and the output looks exactly as I would expect it to.
I'm unclear why running the command by itself is different from when it is wrapped by npm though, hoping some of the JS experts will have some suggestions. It might also be that I have different global dependencies in place, or I'm doing something else differently.

Hi @Enet4,
I was poking around a bit and I figured out the issue was some inconsistency with styled-components versions when installing the packages. Installing a specific version helped me: npm i<EMAIL_ADDRESS>Could you check if that resolves the issue on your end?
Another solution was to remove package-lock.json + node_modules and installing packages using node 18 / npm 9.

Hi @Enet4,
Could you check if the issue has gone with the v1.0.0-beta.130?

@tatomyr Yes, I confirm that the problem is gone! Much appreciated!


New versions output error when adjusting TC output

On updating to the latest version of Modtweaker many of my Tinkers Construct casting adjustments are returning errors.  For example the line
mods.tconstruct.Casting.removeTableRecipe(Forestry:ingotBronze);
has traditionally worked but no longer does so.

Well, please paste your script and the error ,_,

Script: mods.tconstruct.Casting.removeTableRecipe(Forestry:ingotBronze);
Output: WARNING: [ModTweaker2] No TConstruct Casting Recipe found for output Forestry:ingotBronze, material <> and cast <>. Command ignored!

... ... ... ... ... ... ... ... ... ... no, in the past it has just not thrown errors, this is a user error, it is basiclly saying, there is no recipe for that item, in the past, the code work go through, but wouldn't do anything, this has changed.


max_connections is missing from type CreateOfferRequest

If I give duffel.offerRequests.create a max_connections value it works but I get a type error:
const { data } = await duffel.offerRequests.create({
    ...
    max_connections: 0 // <-- this shows a type error
});

I'd make a PR for this but given the problems we had with #653 I thought it would just be easier to make this issue.

Hey – thanks for this, we're very happy you're helping us find and fix issues! We've just released v1.19.0 which contains this fix.


Documentation Redesign

We want to refresh the look of the documentation.
Design: https://www.figma.com/file/JTJerpdO1zzihdVBympX6c/Documentation?node-id=124%3A0
Must-dos

[ ] Header

[x] Update to new design
[ ] Add active state for "Docs"

[ ] Layout

[x] Implement new layout
[x] Make layout responsive

[ ] Navigation

[x] Update to design system
[x] Active states
[ ] Responsive behaviour?

[x] Search bar

[x] Update to match new design

[ ] Content

[x] Add tailwind typography
[ ] Adjust design of text components
[ ] Update design of hr

[x] Footer

[x] Convert to design system

[x] "On this page" section

[x] Create mockup of section

[ ] General

[x] Upgrade FontAwesome
[ ] Remove unused CSS
[x] Check if it's possible to remove the whole design language?

[ ] For Jeroen

[ ] Check On this Page section
[ ] Check active system

Nice-to-haves

[ ] Content

[ ] Add title to code block
[ ] Add link box
[ ] Figure out what to do with tables

[ ] "On this page" section

[ ] Write script to automatically create table of contents

[ ] Footer

[ ] Update design

@jonasbrinkhoff can this be closed now that #344, #341 are merged?

@jonasbrinkhoff can this be closed now that #344, #341 are merged?

Yes :)


sensord: Avoid extra copies on sensor init

Description:
Use emplace_back instead of push_back on sensor init.
This avoids an extra copy by constructing the std::pair in place.
If it were a primitive both would optimize to the same instructions, but std::pair makes it very unlikely to be optimized.
See https://godbolt.org/z/dheqYj611
Verification:
Currently, I don't have the hardware to test the changes myself.

Thanks for the PR, but we tend to not merge PRs like this as it effectively makes no difference. Nice job with the linked example, though!


Help translate Mutify.

Although, translations to any language are welcomed. Here are the translations required as per user-base ordered by priority -

[ ] German (Germany)
[ ] Spanish (United States)
[ ] French (France)
[ ] Vietnamese (Vietnam)
[ ] German (Austria)
[ ] Spanish (Spain)
[ ] Spanish (Latin America)

Recommend tool for translation - Android Studio with translation helper plugin.

Hi, Teekam Congratulations on building this helpful application. I would like to work on this.

Thanks for helping out with the current translations in Mutify @gupta-shrinath, I will be pushing out a new commit with updated strings soon, so please let me know if you will be able to help with translation of those as well. :)

The dialog_positive_btn and open_spotifystring key have the same value Open Spotify should I change code to use only open_spotify?

Yes, please keep all the extra duplicate values, even if they're same. The id's are different so that these can be identified easily in code, also it helps with code readability. Thanks :)

The dialog_positive_btn and open_spotifystring key have the same value Open Spotify should I change code to use only open_spotify?

Yes, please keep all the extra duplicate values, even if they're same. The id's are different so that these can be identified easily in code, also it helps with code readability. Thanks :)

Make sense. I'll have to revert the last commit in the PR.

Thanks for helping out with the current translations in Mutify @gupta-shrinath, I will be pushing out a new commit with updated strings soon, so please let me know if you will be able to help with translation of those as well. :)

Sure.

Hi, @teekamsuthar I have reverted the commit which removes the key dialog_positive_btn.


FloatingPanel is hard to dismiss if isRemovalInteractionEnabled is enabled

Description
If isRemovalInteractionEnabled enabled, the panel is actually "hard" to dismiss compared to panel v1.
In v1 the panel was also dismissed if it was dragged no near bottom.
In v2 the panel is only dismissed if the panel is dragged with a very high velocity.
Expected behavior
The panel is easier to dismiss if isRemovalInteractionEnabled is enabled.
Actual behavior
Steps to reproduce
Open a example app panel where isRemovalInteractionEnabled is enabled.
Code example that reproduces the issue
How do you display panel(s)?

Add as child view controllers

How many panels do you displays?

1

Environment
Library version
2.0.0
Installation method

CocoaPods

iOS version(s)
14.0
Xcode version
Version 12.0.1 (12A7300)

@aclassen Thank you for your feedback. You can make it easy to remove a panel using floatingPanel(_:shouldRemoveAt location:with) delegate method which is called [here].(https://github.com/SCENEE/FloatingPanel/blob/master/Sources/Core.swift#L717).
But now I'm thinking the API should be reconsidered because the logic, which the velocity vector is calculated from a distance to a point of the hidden state and the pan gesture's velocity, is not appropriate for a gesture. So I have an idea that the core will use just a velocity of the panel pan gesture to determine whether a pane is removed or not, and then the delegate method will pass it to the argument.

I was able to do a workaround and now is working nicely.
@aclassen you just need to add state "tip" and when you are going to that step he will automatically dismiss
    func floatingPanelWillBeginAttracting(_ fpc: FloatingPanelController, to state: FloatingPanelState) {

        if state == FloatingPanelState.tip {
            fpc.dismiss(animated: true, completion: nil)
        }
    }

class MyFloatingPanelLayout: FloatingPanelLayout {
    let position: FloatingPanelPosition = .bottom
    let initialState: FloatingPanelState = .half
    var anchors: [FloatingPanelState: FloatingPanelLayoutAnchoring] {
        return [
            .full: FloatingPanelLayoutAnchor(absoluteInset: 60.0, edge: .top, referenceGuide: .safeArea),
            .half: FloatingPanelLayoutAnchor(fractionalInset: 0.5, edge: .bottom, referenceGuide: .safeArea),
            .tip: FloatingPanelLayoutAnchor(fractionalInset: 0.25, edge: .bottom, referenceGuide: .safeArea),
        ]
    }
    func backdropAlpha(for state: FloatingPanelState) -> CGFloat {
        return 0.5
    }
}

@SCENEE I also find it very hard to remove the panel after upgrading to 2.0.0 today. Do you have an example of how to use the "ShouldRemoveAt" method to determine if the panel should be removed or not? I am not sure how to use location and velocity to calculate if the shouldRemoveAt method should return true or false here.

@sipersso
Here is an example to ease removing a panel in 'shoudlRemoveAt' delegate method.
    func floatingPanel(_ fpc: FloatingPanelController, shouldRemoveAt location: CGPoint, with velocity: CGVector) -> Bool {
        let threshold: CGFloat = 5.0
        switch fpc.layout.position {
        case .top:
            return (velocity.dy <= -threshold)
        case .left:
            return (velocity.dx <= -threshold)
        case .bottom:
            return (velocity.dy >= threshold)
        case .right:
            return (velocity.dx >= threshold)
        }
    }

The default value of threshold is 10.0 in 2.0.0. It causes that it's hard to remove a panel. So I'm thinking it will be between 5.0 and 6.0.

@SCENEE I just tried this with 5.0 and it works perfectly. Thank you!

the velocity vector is calculated from a distance to a point of the hidden state and the pan gesture's velocity, is not appropriate for a gesture. So I have an idea that the core will use just a velocity of the panel pan gesture to determine whether a pane is removed or not, and then the delegate method will pass it to the argument.

After I try the above new logic, I confirmed the current logic is better because a swipe speed to dismiss a panel and ease of the acceleration can changes depending on a distance from a screen edge to hide it.
As a result, I'm going to ease a threshold to remove a panel in PR #395.


Kong and Instaclustr link broken

Instaclustr link seems to be broken and gives a 404 error
Summary
Kong and Instaclustr link broken
Full changelog
Instaclustr links seams to be broken and gives a 404 error

@suizman Thank you!


Removing JS components

This may not be an issue but me missing something.
I'm trying to remove some of the js components via the gruntfile on lines 73-88.
If I comment out or delete a file it seems to make no difference to the foundation.js build.
The build seems to run with no errors etc.. but the foundation.js stays the same.
Am I missing something?

Are you running grunt build after you comment them out?

@webthread Did you get a chance to try running "Grunt Build" and seeing if that resolved your issue?

Yes, I new I was missing something. Thanks for your quick response!

:+1:


feat: ConstrainedBox

Added ConstrainedBox attributes, model, widget variants

https://github.com/Duit-Foundation/duit_js/pull/51

https://github.com/Duit-Foundation/duit_go/pull/40


FIX for issue #285: Allow number and boolean as well as strings.

Very very simple one-line change that should fix #285.

The issue is already fixed, so the merge request is obsolete => closed.

Wooooo!  Only took three years to realize this!


On segment info popups, the "width" buttons don't line up with the "variant" buttons

Here's what it looks like on my monitor. Enlarged 200% with a red line for comparison:

I'm on Chrome 28. Also seeing it in Firefox 22. Looks lined-up and great on Safari, though.

I any one working on this? I'd like to take a stab at it.

@comoss Go for it!

Resolved https://github.com/codeforamerica/streetmix/pull/603


Create Kids With the Greatest Number of Candies.cpp

this is solution to Kids With the Greatest Number of Candies question on leetcode.

Great work @Pragya0115 please give this repo a star if it really helped you to complete your PR.
Also follow me for follow back if you want to get connected with me for any such open source contribution program. Thank you 😊


[Doc]Fix path to get comments.css

See https://github.com/FriendsOfSymfony/FOSCommentBundle/commit/342cd638147798125ee98c84c95ade794456399d

LGTM :+1:

Thanks.


Fixes list().Join()

Fixes #808
Not sure if this is the cleanest way to do it, but it works

We have DirectConvertable() for this.

@wixoaGit Done, forgot about that


[BrokerPool] Failed to connect to seed broker, trying another broker from the list: Failed to connect: Port should be >= 0 and < 65536. Received NaN.

DESCRIPTION -
When tried to consume Kafka using Broker URL, Topic and GroupID I'm getting the below mentioned error.
[BrokerPool] Failed to connect to seed broker, trying another broker from the list: Failed to connect: Port should be >= 0 and < 65536. Received NaN.
Steps to reproduce the behavior:

Create a AWS Lambda function that is triggered from CloudWatch events.
Run a consumer that subscribes to the topic with given topic name, broker url and group Id.

Expected behavior
The topic should return a success message.
Observed behavior
Getting below mentioned error.
{
"level": "ERROR",
"timestamp": "2022-07-08T16:38:21.511Z",
"logger": "kafkajs",
"message": "[BrokerPool] Failed to connect to seed broker, trying another broker from the list: Failed to connect: Port should be >= 0 and < 65536. Received NaN.",
"retryCount": 3,
"retryTime": 3326
}
Environment:

AWS Lambda Function
OS: [Amazon Linux]
KafkaJS version [2.0.2]
Kafka version [2.1.3]
NodeJS version [14.17.1]

I have the same issue, please help  @tulios  @Nevon  @meister  @Dieken  @pihentagy

@RamCTS56

Handle the promises correctly
Allow the IAM role to access to the VPC; attach relevant policies.

To piggy back off this issue and file a more legitimate issue; I get this when I prefix my broker with any protocol.
kafka://kafka-server-url:1234

Where as the following works
kafka-server-url:1234

I didn't dive into the code, but from the variable assignment the broker is parsed as kafka:NaN so I assume the code splits on the first : rather than checking for :// beforehand.
This would be a nice QoL improvement

Issue is really old, but as I encountered this problem today I'll leave a comment anyway: make sure that your brokers value is an array. I had the same error but had only defined one broker, as a string.


Data Not Saving in Local participant.db File on PsiTurk 3.3.1 with Heroku #566

Context
PsiTurk Version: 3.3.1
Hosting Platform: Heroku
Experiment Repository: fredcallaway/heroku-experiment
Problem Description
The experiment runs perfectly fine, and participants can complete and submit the experiment. However, the data is not being saved in the local participant.db file. Consequently, when attempting to approve workers, I receive the following error:
Response(Operation: approve_assignment_by_assignment_id | Status: error | Exception: AssignmentIdNotFoundInLocalDBError: )
Steps to Reproduce

Deploy the experiment to Heroku following the steps provided in the repository fredcallaway/heroku-experiment. changes the config file with the ad url
Run the experiment to completion.
Submit the experiment.
Check the local participant.db file for saved data.
Attempt to approve a worker using the worker approve command.

Observed Behavior

Experiment runs and completes without issues.
Data submission appears to be successful on the frontend.
No data is saved in the participant.db file.
Attempting to approve workers results in AssignmentIdNotFoundInLocalDBError.

Expected Behavior

Data should be saved in the participant.db file upon experiment completion and submission.
Worker approval should proceed without errors.

Any help or suggestions on resolving this issue would be greatly appreciated. Thank you!

participants.db is only used for local testing. Data will be saved there if you run the experiment locally (I think make dev?)
The data generated online should be saved in the heroku psql server. You can get the db location with heroku config:get DATABASE_URL. You can also view it with heorku psql.
But if you just want to download the data, you should be able to just run bin/fetch_data.py
As for using the psiturk command line to approve people (if that's what you're doing), that won't work directly because the database location is not set in config.txt (for security reasons). @cgc might have advice here.
I've been running on Prolific exclusively, so I don't really remember how I was approving/bonusing MTurk participants. I think I used this script at one point—maybe it will be of help. https://github.com/fredcallaway/webofcash/blob/retention/bin/compensation.py
If you are using Prolific, then you can try bin/prolific.py pay (there are other useful commands, see the CLI methods)

Hi there,
Thank you for getting back to me. I have a couple of follow-up questions:

Broken Link: The link you provided (https://github.com/fredcallaway/webofcash/blob/retention/bin/compensation.py) does not seem to be working. Could you please check the link or provide the correct one?
Specifying Database Path in Config File: To ensure the data is saved correctly to the Heroku PostgreSQL database, I would like to confirm if specifying the database URL in the config.txt file as follows is the correct approach:
[Database Parameters]
database_url = postgres://username:password@hostname:port/dbname

Or can you give an example.
Thankyou again for all your help.

Ah, didn't realize that was a private repo! Here you go https://gist.github.com/fredcallaway/9b8eb28f3ac7539950fee10eb7d53bbb @./0?redirect=https%3A%2F%2Fgist.github.com%2Ffredcallaway%2F9b8eb28f3ac7539950fee10eb7d53bbb&recipient=cmVwbHkrQUI0RVRGR04zNVJaNFNBUVozSVZRRTZFVk9HUk5FVkJOSEhKQTdHR0o0QHJlcGx5LmdpdGh1Yi5jb20%3D)
The data will be saved correctly even if you don't update the config file. It gets set on the server by herokuapp.py. Carlos set it up this way so that the config file could be included in a public repo without exposing the data. If you want to use the local psiturk command line, you will to edit the config.txt. In that case make sure not to push the config file to github!
On Jul 23 2024, at 3:11 pm, Rohit.K @.> wrote:

Hi there,
Thank you for getting back to me. I have a couple of follow-up questions:
Broken Link: The link you provided (https://github.com/fredcallaway/webofcash/blob/retention/bin/compensation.py @./1?redirect=https%3A%2F%2Fgithub.com%2Ffredcallaway%2Fwebofcash%2Fblob%2Fretention%2Fbin%2Fcompensation.py&recipient=cmVwbHkrQUI0RVRGR04zNVJaNFNBUVozSVZRRTZFVk9HUk5FVkJOSEhKQTdHR0o0QHJlcGx5LmdpdGh1Yi5jb20%3D)) does not seem to be working. Could you please check the link or provide the correct one?
Specifying Database Path in Config File: To ensure the data is saved correctly to the Heroku PostgreSQL database, I would like to confirm if specifying the database URL in the config.txt file as follows is the correct approach:
[Database Parameters]
database_url = @.:port/dbname
Or can you give an example.
Thankyou again for all your help.
—
Reply to this email directly, view it on GitHub @./2?redirect=https%3A%2F%2Fgithub.com%2Ffredcallaway%2Fheroku-experiment%2Fissues%2F19%23issuecomment-2245212381&recipient=cmVwbHkrQUI0RVRGR04zNVJaNFNBUVozSVZRRTZFVk9HUk5FVkJOSEhKQTdHR0o0QHJlcGx5LmdpdGh1Yi5jb20%3D), or unsubscribe @./3?redirect=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAB4ETFACWSYR5GIXV7KRLBTZNZJBNAVCNFSM6AAAAABLJNKIIOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENBVGIYTEMZYGE&recipient=cmVwbHkrQUI0RVRGR04zNVJaNFNBUVozSVZRRTZFVk9HUk5FVkJOSEhKQTdHR0o0QHJlcGx5LmdpdGh1Yi5jb20%3D).
You are receiving this because you commented.

Hi Fred. Thankyou for your message. But using the above code is giving a lots of problem. Especially because how fetch data.py code is written and how it creates the file in data folder.

I'm not sure how to help without knowing exactly what issues you're encountering with fetch_data.py. That script is the only way I ever download data. You should be able to pull out the key code to access the database if you don't like the way it preprocess and saves the data.


Snapshot failures

moving snapshot logic from reporters to the core of jest.
Decision of whether the test run is successful or not is based on three factors

Are there any test failures?
Were any snapshots updated/removed/unused?
Did any of the reporters throw? (the only case right now is coverage threshold)

1 and 2 have to be captured in jest itself and not anywhere else, because it's the core functionality of jest. Because we had 2 spread across multiple files, things kept breaking and it especially hard to integrate it with the internal runner.
after this change:

Test results are generated only in test runner (initially in jasmine though)
Reporters only consume results and don't modify anything (there is still one exception in the old code, but i'll fix it)

Nice!


Add MyXQL support for lateral joins

Lateral joins were added to MySQL in v8.0.14: https://dev.mysql.com/doc/relnotes/mysql/8.0/en/news-8-0-14.html#mysqld-8-0-14-sql-syntax

:green_heart: :blue_heart: :purple_heart: :yellow_heart: :heart:


Delay after combat.

After combat there is a long delay before I can hit my keybind and reengage the addon.

unable to replicate in latest build 1.9.4


FIX Force line endings to LF on sake file

Git (by default) will change line endings to the environment standard during checkout.
This change forces the sake bash script to keep lf line endings which is needed for better compatibility with windows hosts/shared files/VMs/whatever

No more 3.1 pull requests. XD

X


Tz fold support

Testing code coverage commenting, etc.

[Current coverage][cc-pull] is 90.31%

Merging [#1][cc-pull] into [master][cc-base-branch] will increase coverage by +0.70%

@@             master         #1   diff @@
==========================================
  Files            12         12
  Lines          2883       2909    +26
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           2583       2627    +44
+ Misses          300        282    -18
  Partials          0          0

File dateutil/tz/tz.py was modified. more

Misses -11
Partials 0
Hits +11

File dateutil/parser.py (not in diff) was modified. more

Misses -1
Partials 0
Hits +1

Powered by Codecov. Last updated by [bb31e49...66d95db][cc-compare]
[cc-base-branch]: https://codecov.io/gh/pganssle/dateutil-test-codecov/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/pganssle/dateutil-test-codecov/compare/bb31e49e006217ac4ca5a846679438c76cd12759...66d95db3a2363f307565b54166f52f980f030f9e
[cc-pull]: https://codecov.io/gh/pganssle/dateutil-test-codecov/pull/1?src=pr

[Current coverage][cc-pull] is 90.31%

Merging [#1][cc-pull] into [master][cc-base-branch] will increase coverage by +0.70%

@@             master         #1   diff @@
==========================================
  Files            12         12
  Lines          2883       2909    +26
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           2583       2627    +44
+ Misses          300        282    -18
  Partials          0          0

File dateutil/tz/tz.py was modified. more

Misses -11
Partials 0
Hits +11

File dateutil/parser.py (not in diff) was modified. more

Misses -1
Partials 0
Hits +1

Powered by Codecov. Last updated by [bb31e49...66d95db][cc-compare]
[cc-base-branch]: https://codecov.io/gh/pganssle/dateutil-test-codecov/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/pganssle/dateutil-test-codecov/compare/bb31e49e006217ac4ca5a846679438c76cd12759...66d95db3a2363f307565b54166f52f980f030f9e
[cc-pull]: https://codecov.io/gh/pganssle/dateutil-test-codecov/pull/1?src=pr


YJAF and YJSM Repo Access for Drew Maughan

Use this issue template if you do not feel comfortable raising a PR with the required changes to files in the terraform/ directory.
Please supply all the requested information, so that your request can be processed as quickly as possible.
Which MoJ repository are you requesting access to (e.g. money-to-prisoners)?
yjsm-ui
yjaf-app-template/
yjaf-assets/
yjaf-auth/
yjaf-bands/
yjaf-bed-unlock/
yjaf-case/
yjaf-cmm/
yjaf-conversions/
yjaf-csppi/
yjaf-dal/
yjaf-documents/
yjaf-gateway/
yjaf-mule-hub-connector/
yjaf-mule-mis-processor/
yjaf-nginx-gateway/
yjaf-nginx-pentaho/
yjaf-placements/
yjaf-refdata/
yjaf-returns/
yjaf-sentences/
yjaf-transfers/
yjaf-utilities/
yjaf-views/
yjaf-workflow/
yjaf-yp/
yjsm-hub/
yjsm-hubadmin/
yjaf-ui/
yjsm-hub-svc/
What is the GitHub username of the person who needs access?
turtle-nec
What is their full name?
Drew Maughan
What is their email address?
<EMAIL_ADDRESS>What company/organisation do they belong to (e.g. "MadeTech", "Department for Education")?
NEC SWS
What level of access should they have?

[x] Read
[ ] Triage
[x] Write
[ ] Maintain
[ ] Admin

Why is this access being requested?
New NEC Developer
Which MoJ team/person is responsible for this request? This must be someone with a "*.justice.gov.uk" email address.
Jon Dent<EMAIL_ADDRESS>Review date
Date after which this collaborator's access should be reviewed. This should be a date in the format: 2021-01-15, no more than one year into the future.
11/02/2023

I've added Drew, all done.


Overseerr

Which addon?
Overseerr
https://docs.overseerr.dev/getting-started/installation
Is your feature request related to a problem? Please describe.
no
**If a new addon, have you checked on Google that such as addon doesn't already exists?
yes
Describe alternatives you've considered
none
Additional context
none

Hi, if you're still interested I've created the addon ;-)

Hi, if you're still interested I've created the addon ;-)

Damn dude that was quick. I appreciate it. I ended up closing this shortly after posting because I found the requestrr service and it's pretty cool. If I find need for it still I'll be back here to get it.
RDTclient would be another cool one to have it's a downloader that works with real debrid.
Thanks again dude your support is top notch.


ZeroClipboard doesn't work in JsFiddle

We've been using JsFiddle for a long time and recently it seems to have stopped working when embedded in a JsFiddle iframe.
We're naturally curious to see why this has regressed, so we made a test case.

JsFiddle test case (doesn't work) - http://jsfiddle.net/loganvolkers/koa04mju/
Without an iframe (works) - http://fiddle.jshell.net/loganvolkers/koa04mju/show/

In an iframe it appears that flash has been disabled by JsFiddle:
{"type":"error",
"name":"flash-disabled",
"target":null,
"relatedTarget":null,
"currentTarget":null,
"timeStamp":1415318935611,
"message":"Flash is disabled or not installed",
"minimumVersion":"11.0.0"}

Directly visiting the page seems to work just fine
{"type":"ready",
"target":null,
"relatedTarget":null,
"timeStamp":1415319109675,
"message":"Flash communication is established",
"version":"15.0.0"}

I've done all I can for this issue now. The sandboxing is a huge PITA but it's nothing that we can expect to go away as it adds very legitimate value for the various playground sites that employ it.

I've done all I can for this issue now. The sandboxing is a huge PITA but it's nothing that we can expect to go away as it adds very legitimate value for the various playground sites that employ it.

P.S. Link to documentation on "starter snippets": docs/instructions.md#starter-snippets-for-playground-sites

P.S. Link to documentation on "starter snippets": docs/instructions.md#starter-snippets-for-playground-sites


Add sqs error handling

I had to rewrite sqs.send_message_batch a bit to be able to handle most errors.
Also fixes #1799
Over time I will also check other functions in the SQS implementation and update the coverage file while doing so.

Coverage decreased (-0.03%) to 93.518% when pulling 05dc97b468cdfbb05686966444e2ed9c4dff4e3d on edekadigital:add-sqs-error-handling into 36a8a4ad00a6983ec6afc8bb6f77afa2a035389b on spulec:master.


xrdp root user AuthN

There is a proven bug that login xrdp as root user, you cannot unlock the screen with proper root user password.

Does it happen only on Ubuntu?


Feature: add the state's last modified date/time in the fuzzy search dialog

The modified code adds a new date/time column within the label field passed to the fuzzy dialog. After the glob is retrieved by wezterm, the last change of each file is retrieved. The maximum length is derived from the list of filenames such that a proper column is formatted with a padding of '.' so it is easy to read. The format functions have been modified to display the date/time in white, leaving the color focus on the name of the state.
I've noticed that when changing the state_save_dir, the proper subfolders (tab, workspace, window) are expected to be created as well, which is non-intuitive, in my opinion. Thus, I have modified the code such that when the folder is changed, the plugin ensures that the necessary folders are present and, if needed, creates them.

I've attempted to write the changes for Windows and Mac/Linux, but I wasn't able to test thoroughly for Windows.
This PR closes issue #80

To answer your question: I have not noticed a difference, but I don't have that many states stored so the approach might be suboptimal, e.g. to be fast we could try to retrieve everything in one go rather than picking up the glob and then enriching it with the date/time for each file. But I wasn't sure how to achieve it.
Second, yes we can make it an option same as the date format. But I will need some time to achieve both.
Btw I am giving a try at Ghostty, and thus back with tmux 😉. Let's see how it shapes up.

I've refactored the whole code. It should now be faster as all the data is fetched by the OS, instead of relying on wezterm API and returning fetching the rest one by one. The cost is that there is now one command for Mac, one for Linux, and one for Windows. Unfortunately, I did not have time to check them besides the Mac one, which works well.
There are now two new key-values in the opts for the fuzzy_load

show_state_with_date, default false, which can be used to add the date (works for both read and delete)
date_format, default "%d-%m-%Y %H:%M:%S", a strftime-string to format the date and can be selected by the user.

The documentation has also been updated to factor in the changes in this PR.
Do you have a chance to test the Linux and Windows versions yourself? Unfortunately, I won't have the time in the coming weeks. I can only handle your guidance, feedback, and change requests. I'm very sorry about that.


Boolean is converted to empty string instead of 0

A boolean value is converted to an empty string instead of 0
$client = new GuzzleHttp\Client;
$client->get('http://docs.guzzlephp.org/en/latest/', [
        'query' => [
                'foo' => false
        ]
]);

Expected result: 'foo=0' (same as http_build_query())
Actual result: 'foo='

Hello,
I'm using guzzle to post data and I need to have 0 for false boolean instead of "". You said we can subclass Query but my code looks like:
$request = $this->client->createRequest('POST', $this->path, ['body' => $data]);
$response = $this->client->send($request);

How can I inject my custom Query class?
Is there another more elegant method to deal with this issue?
Thanks a lot for your help.

@mtdowling, please, can you point me some documentation to help me modify default Query implementation?
Thanks in advance

Hello @mtdowling,
thank you so much for your help!
Finally, I wrote a custom aggregator function to replace the default phpAggregator.
If someone gets the same issue, my code is here:
$request = $this->client->createRequest('POST', $this->path, ['body' => $data]);
$request->getQuery()->setAggregator(BooleanQueryAggregator::phpAggregator());
$response = $this->client->send($request);

and my aggregator looks like this:
<?php

namespace AppBundle\Http\Guzzle;

class BooleanQueryAggregator
{
    public static function walkQuery(array $query, $keyPrefix, callable $prefixer)
    {
        $result = [];
        foreach ($query as $key => $value) {
            if (false === $value) {
                $value = 0;
            }
            if ($keyPrefix) {
                $key = $prefixer($key, $keyPrefix);
            }
            if (is_array($value)) {
                $result += self::walkQuery($value, $key, $prefixer);
            } elseif (isset($result[$key])) {
                $result[$key][] = $value;
            } else {
                $result[$key] = array($value);
            }
        }

        return $result;
    }

    public static function phpAggregator($numericIndices = true)
    {
        return function (array $data) use ($numericIndices) {
            return self::walkQuery(
                $data,
                '',
                function ($key, $prefix) use ($numericIndices) {
                    return !$numericIndices && is_int($key)
                        ? "{$prefix}[]"
                        : "{$prefix}[{$key}]";
                }
            );
        };
    }
}

Nobody answered to me so I don't know if it is the best solution or at least an acceptable solution but it works.

Does this fix the problem?
http://stackoverflow.com/a/39278703/3456488

Thank you for your reply but this issue is too old for me now.
I don't use guzzle anymore.
regards


issues on docker container

Hi,
just installed the container and hopefully i'm doing something wrong. Server is up and i can connect at port 3000 but no device is showed.
When i look and the container log i see:
_currentUrl: 'https://mysecurity.eufylife.com/api/v1/passport/login',
[Symbol(kCapture)]: false
},
response: undefined,
isAxiosError: true,
toJSON: [Function: toJSON]
}
error stack:
• dns.js:71 onlookup
dns.js:71:26
2021-11-08 11:37:45.088  ERROR Status return code not 200
{
status: undefined,
statusText: undefined
}
2021-11-08 11:37:45.089  ERROR Token error
Can you please help?
Thank you!

This is a container network problem. In the error above you can see a DNS resolution problem.


Include Bayesian sampling in Hyperband implementation

The paper "BOHB: Robust and Efficient Hyperparameter Optimization at Scale" includes an interesting parallelization technique for Bayesian sampling in a Hyperband implementation. In Section 4.2 the describe a scheme that does the following:

Has a global hyperparameter space for Bayesian sampling. This hyperparameter space will be refined over time according to the Bayesian sampling principle.
Initializes models in a particular order:

At first, initialize num_workers models. Train them as the most aggressive bracket of Hyperband specifies.
When a model is stopped, initialize a new model with parameters sampled from the current hyperparameter space estimate. This model is from the most aggressive bracket if that bracket is not complete; otherwise it's from the next most aggressive bracket.

The number of workers will definitely influence performance: if there are infinite workers, the Bayesian sampling algorithm will not have time to run any inference on the best set of parameters. Likewise, if there's one worker Bayesian sampling can do as much inference as possible.
They show this performance:

Similar to Dask-ML's benchmark, they start saturating between 16 and 32 workers.

Sounds fun
On Sun, Jul 12, 2020 at 3:32 PM Scott Sievert<EMAIL_ADDRESS>wrote:

The paper "BOHB: Robust and Efficient Hyperparameter Optimization at Scale
http://proceedings.mlr.press/v80/falkner18a/falkner18a.pdf" includes an
interesting parallelization technique for Bayesian sampling in a Hyperband
implementation. In Section 4.2 the describe a scheme that does the
following:

Has a global hyperparameter space for Bayesian sampling. This
hyperparameter space will be refined over time according to the Bayesian
sampling principle.
Initializes models in a particular order:

At first, initialize num_workers models. Train them as the most
aggressive bracket of Hyperband specifies.
When a model is stopped, initialize a new model with parameters
sampled from the current hyperparameter space estimate. This model is from
the most aggressive bracket if that bracket is not complete; otherwise it's
from the next most aggressive bracket.

The number of workers will definitely influence performance: if there are
infinite workers, the Bayesian sampling algorithm will not have time to run
any inference on the best set of parameters. Likewise, if there's one
worker Bayesian sampling can do as much inference as possible.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask-ml/issues/697, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACKZTEXCMC3EY5KUWT34LLR3I2XNANCNFSM4OYAGQ3Q
.

Any updates here ?

Probably not. All development happens on GitHub.
On Tue, Aug 25, 2020 at 3:04 PM Unex<EMAIL_ADDRESS>wrote:

Any updates here ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask-ml/issues/697#issuecomment-680241692, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKAOITLJAAWQMKZ7WVUKBLSCQKLZANCNFSM4OYAGQ3Q
.

Sorry, I mean, do you have the plan to implement the BOHB in the near future? I just hope I can use it when available:)

I don't believe anyone is working on it at the moment, though @stsievert
might have a better idea.
On Tue, Aug 25, 2020 at 3:14 PM Unex<EMAIL_ADDRESS>wrote:

Sorry, I mean, do you have the plan to implement the BOHB in the near
future? I just hope I can use it when available:)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask-ml/issues/697#issuecomment-680246244, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKAOIXAGDVPC756XRRKFYTSCQLSBANCNFSM4OYAGQ3Q
.

do you have the plan to implement the BOHB in the near future?

I don't know of anyone that has a plan to implement BOHB. I have some ideas on how to implement it but that's about it.


Add support for Strapi v4

This plugin looks great by the way. I was wondering if you have any plans to make this plugin compatible with Strapi v4?

https://github.com/Baboo7/strapi-plugin-import-export-entries I think it does the same, have you tried it? with compatibility to V4


When "calcdeps.py" is executed, the exception "WindowsError: [Error 87]" is caused by the "proc = subprocess.Popen(args, stdout=subprocess.PIPE)" parameter "args" of the function "Compile".

When "calcdeps.py" is executed, the exception "WindowsError: [Error 87]" is caused by the "proc = subprocess.Popen(args, stdout=subprocess.PIPE)" parameter "args" of the function "Compile".
Currently, the temporary processing scheme optimizes the parameter "args" more often.
Such as:
The "testA" folder contains files: a.js, b.js, c.js
The "testB" folder contains files: a2.js, b2.js, c2.js
The files that need to be compiled are: testA\a.js, testA\b.js, testA\c.js, testB\a2.js, testB\b2.js
Not optimized:
args = ['--js', 'testA\a.js', '--js', 'testA\b.js', '--js', 'testA\c.js', '--js ', 'testB\a2.js', '--js', 'testB\b2.js']
Optimized:
args = ['--js', 'testA\**.js', 'testB\a2.js', '--js', 'testB\b2.js']
Code:
Add function:
def GetFileExtension(path):
  arr = os.path.splitext(path)
  return arr[len(arr) - 1]

def GetKeyPathFile(file):
  dir = os.path.dirname(file)
  ext = GetFileExtension(file)
  if len(ext) == 0:
    ext = '.'
  else:
    ext = ext.lower()
  key = dir + '\\**' + ext
  return key

def TotalFile(path, ex):
  count = 0
  for name in os.listdir(path):
    # logging.info(name)
    if os.path.isfile(os.path.join(path, name)):
      if name.lower().endswith(ex):
        count += 1
  return count

def Simplify(source_paths):
  dic = {}
  for path in source_paths:
    key = GetKeyPathFile(path)
    if not dic.has_key(key):
      dir = os.path.dirname(path)
      ext = GetFileExtension(path)
      value = {'t': TotalFile(dir, ext), 'n':0}
      dic[key] = value
      # logging.info(key)
    if dic.has_key(key):
      dic[key]['n'] += 1
  array = []
  for key,value in dic.items():
    # logging.info(key)
    if value['t'] == value['n']:
      array.append(key)
    # logging.info(str(value['t']) + ' = ' + str(value['n']))
  for one in array:
    logging.info(one)
  return array

Modified in the function "Compile":
  args = ['java', '-jar', compiler_jar_path]
  for path in source_paths:
    args += ['--js', path]

Replace with:
  array = Simplify(source_paths)
  args = ['java', '-jar', compiler_jar_path]
  for path in source_paths:
    if GetKeyPathFile(path) not in array:
      args += ['--js', path]
  for one in array:
    args += ['--js', one]

Created internal Google issue http://b/143616292

Hi, I think I understand what you're suggesting, but wanted to make sure.
Your problem is that calcdeps.py is causing an error when the number of dependencies is too large. You found that this is because the script passes too many arguments to a subprocess.
Your suggestion is to reduce the number of arguments by globbing individual files together into one argument, in the case that all js files from a given directory are dependencies.
Is that correct?

Are you able to use the closure-deps package for this?  calcdeps.py is generally unmaintained.  Alternatively if you want to make a PR, that would also be a possibility.


PATH to SQLITE file

if I copied the file into the project, how can access to the file??
i don't understand the path
let db = try Connection("path/to/db.sqlite3")
thank you

Same answer as in https://github.com/groue/GRDB.swift/issues/17

Please ask usage questions on Stack Overflow :smile: There's also an answer in the documentation.
(Thanks, @groue!)

We share the same users ;-)


empty_feed_notice: Style empty message feed similar to other views.

after
before

The screenshots and manual testing look good to me! (the labels for the screenshot columns are swapped)

Merged, thanks @amanagr!


🐛 BUG: wrangler init produce project with warning

Which Cloudflare product(s) does this pertain to?
Wrangler
What version of Wrangler are you using?
3.0.1
What operating system are you using?
Linux
Describe the Bug
npm create cloudflare@2 # answer -> Enter -> Hello world script -> Y (typescript) ->  N (deploy)
cd <folder>
npm start

See warning getting generated
Failed to get worker definitions TypeError: fetch failed

      at fetch
  (/home/wighawag/dev/local-only/wrangler-worker-test/young-sky-dae6/node_modules/wrangler/wrangler-dist/cli.js:16584:17)
      at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
      at async getRegisteredWorkers
  (/home/wighawag/dev/local-only/wrangler-worker-test/young-sky-dae6/node_modules/wrangler/wrangler-dist/cli.js:124280:22)
      at async getBoundRegisteredWorkers
  (/home/wighawag/dev/local-only/wrangler-worker-test/young-sky-dae6/node_modules/wrangler/wrangler-dist/cli.js:124299:29)
  {
    cause: ConnectTimeoutError: Connect Timeout Error
        at onConnectTimeout
  (/home/wighawag/dev/local-only/wrangler-worker-test/young-sky-dae6/node_modules/wrangler/wrangler-dist/cli.js:7185:29)
        at
  /home/wighawag/dev/local-only/wrangler-worker-test/young-sky-dae6/node_modules/wrangler/wrangler-dist/cli.js:7141:51
        at Immediate._onImmediate
  (/home/wighawag/dev/local-only/wrangler-worker-test/young-sky-dae6/node_modules/wrangler/wrangler-dist/cli.js:7173:13)
        at process.processImmediate (node:internal/timers:476:21) {
      code: 'UND_ERR_CONNECT_TIMEOUT'
    }
  }

@wighawag thanks for filing this. Could you try running npm create cloudflare@latest and see if you experience the same?

I'm going to close this due to inactivity. If it continues to be a problem, please re-open. Thanks!

thanks @admah for the reply
I tried with @latest as you suggested and it seems to work, Thanks!

I'm really glad to hear that. I'm going to close this as resolved. Thank you.


Trigger setAttributes on radio inputs with shared wrapper

This change allows radio input events (for conditional reveals) to be scoped higher to parent .govuk-form-group wrappers, allowing groups with multiple lists to behave correctly.
Originally proposed by @frankieroberto in https://github.com/alphagov/govuk-frontend/pull/1163 and has been separated from https://github.com/alphagov/govuk-frontend/pull/1266.

Hey Colin,
We are currently prioritising work to makeGOV.UK Frontend pass WCAG 2.1 requirements and we have an external audit soon.
This work looks really well done, we want to discuss it as a team and make sure it's the right approach, and then give some proper feedback.
We're not going to be able to follow up on this until we've done the audit so thanks a lot for your patience. :)
(Sorry for the copy paste :))
Nick

Hey Colin,
We've discussed this as a team, thanks for waiting.
Have you considered instead of looking at a parent class, to rely on inputs with the same name attribute? This would allow us to keep the logic within the component itself without relying on something that might not exist.
Nick

Hi @nickcolley, yeah I did but noticed the original didn't do that?
Trying to also apply this feedback: https://github.com/alphagov/govuk-frontend/pull/1163#issuecomment-463223512
Can you suggest an approach the team would be happiest with?

Colin and I spoke about this and thought if we could instead look for inputs with the same name that were scoped within a radios component.
Because there's no way (yet https://github.com/alphagov/govuk-frontend/issues/1127) of knowing what elements are initialised (data-module is optional), I think Colin's current approach might be the best way to do it without drastically changing how this feature works. (For example, decoupling conditional hide/show logic from the component).

Hey @colinrotherham !
Firstly, thanks for your patience with us taking so long on this pull request.
I sat down with Ollie and we think we have an alternative that will fix this behaviour without relying on the .govuk-form-group parent.
https://github.com/alphagov/govuk-frontend/compare/radio-trigger-setattributes
If you think this approach is good, we intend on optimising it to only run code on inputs with the same name, and within the same form owner.
Let me know what you think.

@nickcolley sounds good to me. Querying on the same name attribute and the same form parent sounds like a reasonable thing to do too.

@nickcolley Yep this is great, using document.querySelector() was @frankieroberto's plan originally. I'm happy if you start with this and narrow it down a little. Glad we found a way, thanks.

@frankieroberto @colinrotherham yeah, we've gone round in circles a bit trying to avoid making this a more global change but it seems like the right approach in the end, thanks a lot for your patience again.


Why not Apache License?

Please explain

I can't speak for @gembin, but for us, this crate sticks out like a sore thumb:  it is the only of our dependencies (among thousands) that is LGPL. That is itself okay, but it means that after every license audit, we are asking ourselves why we are using this. The prevailing license in the Rust crate ecosystem is overwhelmingly MIT/Apache; having this be available under MIT or Apache (or MPLv2 or BSD-3) would allow us to stop asking this question on every scan. Hope that helps!

I see, what do you think @NULLx76? I think switching to MIT would be fine. To be quite honest I forgot what the original thought process was to do GPL and I actually thought it already was MIT. If @NULLx76 is okay with it, I'm also fine with changing it.

Just merge #63 if you're okay

Exactly as @bcantrill explained.

I can't speak for @gembin, but for us, this crate sticks out like a sore thumb:  it is the only of our dependencies (among thousands) that is LGPL. That is itself okay, but it means that after every license audit, we are asking ourselves why we are using this. The prevailing license in the Rust crate ecosystem is overwhelmingly MIT/Apache; having this be available under MIT or Apache (or MPLv2 or BSD-3) would allow us to stop asking this question on every scan. Hope that helps!

Yeah this is totally fair, to be honest not much thought went into the initial license. So I'm totally okay changing it to MIT


Add sensorCenterOffset, sensorOverallDimensions, sensorPhotositePitch…

… and sensorAcquisitionRectangle optional standard attributes
Also removed a redundant and non-convention-following IMATH_NAMESPACE:: from the data type of the cameraColorBalance optional standard attribute.

Cary asked me the other day to update the documentation for the optional standard attributes. That really should be done in this pull request for the sensor* additions and I might as well get the doc in for the others as well. Can we hold off on actually merging this for 2-3 days?

@cary-ilm , does that address your problem? It's saying "all checks have passed", presumably after having built everything, including documentation. And is there a way for me to see build artifacts so that, e.g., I can actually look at the text on the now-modified page?


Client configurations are invalid if user does not specify port in public endpoint

To reproduce:

Enter Wireguard server port under "Listen port" under "Server's interface configuration"
Enter Wireguard server IP under "Public endpoint" under "Client's global configuration" without port
Create a new client profile
Try adding client profile via configuration file or QR code
WireGuard client reports invalid configuration as listen port from server configuration is not appended to public endpoint address

Listen port is the wireguard server's bind port.
Public endpoint port is for clients to connect to port, in case you have to NAT.
You have to specify both.


Show and sort by commit counts on all-time page - fixes #9

Display authors by commit count in descending order on the all-time page.

@killercup thanks!
I've updated the code to use the Diesel code suggested.

Thanks so much @ThatOtherPerson ! (and to @killercup for review) Let's :shipit:


add FromAddres dimension & take it from cli parameter, filter txs by from_address & to_address

Motivation
Partially solves #97. for erc*_transfers and trace-related calls, comments needed.
Solution

Take from_address from cli parameter to query & request params

Request params in collect() may need separation of filters that are supported natively by RPC calls vs manually supported by cryo.
Some non-exhaustive branches makes adding new cli param into filter query & debugging difficult. Noted by comments, and may be improved using macros.

Implement txs dataset to filter result in extract() layer

Request for comments

Is using from_address and to_address dimension to represent contract interaction appropriate? I think this is not a big problem because these datasets are derived from logs but there may be a case that causes name collision between tx's from/to. Alternatives like sender/recipient may be considered?
trace related datasets only carry tx hash and will require additional transaction by hash / full block call to fetch from / to address. This may be implemented just like txs dataset that optionally calls receipt only if exclude_failed option is present or gas_used parameter is required.
Any tests to add to assert filtering behavior?

PR Checklist

[ ] Added Tests
[ ] Added Documentation
[ ] Breaking changes

hi. this is looking good. left 2 comments
to your Q's:

I think collisions are actually good as long as they don't restrict functionality. they allow a simpler interface and reuse of underlying machinery. but we cannot use too few parameters either. as they would make the interface harder to understand and it would be problematic for datasets that need multiple dimensions of control. check out the dataset list here...there are examples where we need multiple parameters per dataset, so we cannot collapse those parameters into one
for the traces dataset, I was thinking it would not be filtering on the to_address of the transaction, but the to_address of what contract is being called by each trace. in the trace objects there are values analogous to to/from on the 4 action types
the testing framework is still a major WIP as automating data collection is an extra level of complexity. the right way to do this is with a mockprovider that returns predefined outputs and we test that things get filtered appropriately. but examples of this in the codebase are limited. instead, you can post some example cli commands that demonstrate the functionality. I have been gathering these types of commands and will eventually automate them once a better testing story is in place

I think this works as a self contained pr or if you wanted to add the other datasets that would also work

oh I see the issue with the ordering now. thats a bit tricky
maybe only use eth_getBlockReceipts if filters are NOT being used. because filters probably mean a tiny subset of the data is relevant, and the benefits of eth_getBlockReceipts disappear

basically use get_tx_receipts_in_block if no filters
and if there are filters, use the fallback code inside of get_tx_receipts_in_block:
        // fallback to `eth_getTransactionReceipt`
        let mut tasks = Vec::new();
        for tx in &block.transactions {
            let tx_hash = tx.hash;
            let fetcher = self.fetcher.clone();
            let task = task::spawn(async move {
                match fetcher.get_transaction_receipt(tx_hash).await? {
                    Some(receipt) => Ok(receipt),
                    None => {
                        Err(CollectError::CollectError("could not find tx receipt".to_string()))
                    }
                }
            });
            tasks.push(task);
        }
        let mut receipts = Vec::new();
        for task in tasks {
            match task.await {
                Ok(receipt) => receipts.push(receipt?),
                Err(e) => return Err(CollectError::TaskFailed(e)),
            }
        }

        Ok(receipts)

(could pull this out of get_tx_receipts_in_block into a separate new function like get_tx_receipts())
everything else looks good. and keeping it self contained sgtm

awesome. great feature to have


Text Mining Guide 404 error?

It seems like there are some links that are dead, which I'm assuming is from the move to blogdown? Below is what I have found:
text mining guide is not found:
https://spark.rstudio.com/articles/guides-textmining.html
dplyr link does not work:
https://spark.rstudio.com/extensions/

Found this through this nav bar on the left on the new site:
https://spark.rstudio.com/guides/textmining/
https://spark.rstudio.com/dplyr/


tBTC stale block page links to BTC mainnet explorers

The 'Root' and 'Tip' links on the testnet stale block page link to mainnet explorers. This results in 'block not found' errors.
See e.g. https://forkmonitor.info/stale/tbtc/1830472

Good catch, will fix soon.


remove scare quotes and so-called

So-called is defined as "commonly named" or "falsely or improperly so named". The scare quotes are definitely unnecessary. It makes it sound like things aren't actually called that, or it hints at sarcasm: "He's the so-called 'mayor', but he never does anything!"
http://en.wikipedia.org/wiki/Scare_quotes

I took care of the CLA.


(Sentinel) Automate inspector vrf into sentinel

will be closed after observation
https://github.com/Bisonai/orakl/pull/2227


doc:ad717x_iio:Fixing Typos

Hi all, I have added ad7124 documentation to the same PR. Are there any comments on the same?

Hi all, are there any more comments on this PR?


Update picture

Codecov Report

Merging #85 (1d6bbb0) into master (03b3b53) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #85   +/-   ##
=========================================
  Coverage     41.74%   41.74%
  Complexity      413      413
=========================================
  Files           116      116
  Lines          2247     2247
  Branches        287      287
=========================================
  Hits            938      938
  Misses         1254     1254
  Partials         55       55

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Row Span compatibility added

You can recreate this by setting default value for allowWrapping in drawTextCell as true

This is great, I was just looking for rowSpan or I a way to merge cells in a column. Thanks!


Update contour-utils.js normalizeSeries to do filtering

Filter data to a desired num pts (configurable with new data object ({filter:true, filterNumPts:1000}))
Always keeps first/last -- and then for all the ones in the middle it grabs the high/low per interval

Anything you want me to do to make it easier to merge this?  I can revert the /dist folder if you need.

Hi billy, I'm thinking of what would be the correct way to incorporate the point simplification into the data workflow. I'm not sure about 'loosing' the data points on normalization, because the next visualization may actually use the full data set do display other characteristics of the data. That's why I liked the idea of filtering the data that you will be rendering not not the original data set.
Also, I think we should provide a way to specify or pass in different filtering/data simplification functions so the user decides what's the best way to simplify the data.
What do you think?

I definitely agree with the ability to pass a custom filtering function.
As for where to 'loose' the data it depends on if we want all
visualizations to have the 'same' data or not.  Think of a visualization
that displays the mean, median, mode, min max etc.  Do we want those stats
to be based on the full data set or the filtered set?  If you answer the
full set then we need to filter per visualization -- if you answered the
filtered set we should do it in normalization.
My answer after thinking about it is per visualization.  Do you agree?
On Tue, Feb 10, 2015 at 4:30 PM, Jaime del Palacio<EMAIL_ADDRESS>
wrote:

Hi billy, I'm thinking of what would be the correct way to incorporate the
point simplification into the data workflow. I'm not sure about 'loosing'
the data points on normalization, because the next visualization may
actually use the full data set do display other characteristics of the
data. That's why I liked the idea of filtering the data that you will be
rendering not not the original data set.
Also, I think we should provide a way to specify or pass in different
filtering/data simplification functions so the user decides what's the best
way to simplify the data.
What do you think?
—
Reply to this email directly or view it on GitHub
https://github.com/forio/contour/pull/202#issuecomment-73787758.

Yes, I think the simplification should be per visualization without modifying the original data set. How about something like this:

We add a preprocess function to each visualization's config object (by default is a noop, returning the same dataset)
We provide a set (we can start with only one) simplification functions on _.nw namespace.

Then you would use it something like:
new Contour(..)
  .cartesian()
  .line(myBidDataSet, { preprocess: _.nw.simplifications.minMaxFilter(1000) })
  .statsVisualization(myBidDataSet) // <- this gets original data set
  .render()

the minMaxFilter(1000) would return a simplification function configured for 1000 points.
What do you think? (not crazy about the simplifications name for the namespace)

I like the idea of a preprocess function per viz
How about we call it a dataFilter?
Then the filter I coded could be bound to _.nw.dataFilters.minMaxFilter
On Wed, Feb 11, 2015 at 12:35 PM, Jaime del Palacio <
<EMAIL_ADDRESS>wrote:

Yes, I think the simplification should be per visualization without
modifying the original data set. How about something like this:

We add a preprocess function to each visualization's config object
(by default is a noop, returning the same dataset)
We provide a set (we can start with only one) simplification
functions on _.nw namespace.

Then you would use it something like:
new Contour(..)
.cartesian()
.line(myBidDataSet, { preprocess: _.nw.simplifications.minMaxFilter(1000) })
.statsVisualization(myBidDataSet) // <- this gets original data set
.render()
the minMaxFilter(1000) would return a simplification function configured
for 1000 points.
What do you think? (not crazy about the simplifications name for the
namespace)
—
Reply to this email directly or view it on GitHub
https://github.com/forio/contour/pull/202#issuecomment-73925312.

Sorry for the delay,
dataFilters sounds good

Closing in favor of optimization2 pull request


Feature name fix: use WGPUFeatureName_DepthClipControl

Feature Depth Clamp is removed in dawn. Require Depth Clip Control instead.
https://dawn.googlesource.com/dawn/+/8d73198aca5f1a83283068298c9bfd79640ca36b
This unblocks the build failure on my local linux machine. (with dawn header at f8c07d4753a32b62b52ba86e8ae0dba4830063b3)
Update Readme.md for the running examples cmd line as well (with -s now)

Hi shrekshao,
thanks for the build failure fix and update in the Readme.md :)
With best regards,
Sam

Hi shrekshao,
when testing the examples you will probably notice I ported your deferred rendering example to native:
./wgpu_sample_launcher -s deferred_rendering
The result does not completely match the JavaScript version from the WebGPU Samples: de gray background hides the ground plane.

the gBuffer view looks fine:

I could not find the reason of the difference with the JavaScript version. Do you have any ideas or suggestions to try out to resolve the issue ?
Thanks and best regards,
Sam

I believe the reason is at https://github.com/samdauwe/webgpu-native-examples/blob/master/src/examples/deferred_rendering.c#L770-L772
The clearValue for position GBuffer should be something like
.r = FLT_MAX,
.g = FLT_MAX,
.b = FLT_MAX,
.a = 1.0f,

cuz for areas with no rasterized fragments, we should have a way to differentiate them in the Gbuffers. In the fragmentDeferredRendering.wgsl there's
if (position.z > 10000.0) {
    discard;
  }

to early return.
But I just tried locally that still didn't fix the issue. Maybe I should find another time to further look at it.


Only focus output area if necessary

This prevents unnecessary focusing of the external output area.
Previously when receiving results in rapid succession (for example with a progress bar) using the command palette wasn't possibly since it was unfocused once new outputs arrive.
We could make the focusing behavior customizable with the same setting introduced by @BenRussert in #893.

This keeps the command palette usable when executing code similar to
for i in range(1000):
    time.sleep(0.05)
    sys.stdout.write('\rProgress: {}%'.format(i))


Retrieving Token ID in onToken section

The value isn't available to me, any ideas where I'm going wrong?

Hi @BrianSHenderson, it looks like you're running the code in the sample section of the README; the native fetch function resolves a Response object. (See https://developer.mozilla.org/en-US/docs/Web/API/Response). You'll have to do something like call .json() on that object to get the value in the body.
Try running a console.log() before the fetch to see the data coming back from StripeCheckout.

Thanks for the response, my fetch line says: fetch('/save-stripe-token' but I see no way of passing a destination of where to post the data to.  I seem to be reading the data from a blank page, how do I set this?

That's just an example of passing the data in token to your own server at the endpoint /save-stripe-token. You'll have to change what is done with the token object based on how your sever is setup.


Feature Request: 'minuteInterval' support in 'time' picker

/**

enum(1, 2, 3, 4, 5, 6, 10, 12, 15, 20, 30)
The interval at which minutes can be selected.

@extends from DatePickerIOSProperties
*/
minuteInterval?: 1 | 2 | 3 | 4 | 5 | 6 | 10 | 12 | 15 | 20 | 30;

Besides, if the "minuteInterval" can support 60 (which means only hour is editable), that would be better.


Trying to build Win32 DLL

Hi,
just found this library today, and I have been trying to build a shared library for Win32 to use it in a Delphi project (I will port the header files manually).
I have VS C++ 2008 Express installed. Is VC the preferred build environment under Windows? Or should I go for MinGW? I must admit that I'm not an expert for either toolchain.
CMake generated several VS project files, but I can't get them to compile. First problem was, VC did not find sys/queue.h, so I had to add the "queue" directory manually to the includes in the project config.
Now the compiler is complaining:
1>.\ebur128.c(413) : fatal error C1021: invalid preprocessor command 'warning'

Which points to the following line:
#warning "manual FTZ is being used, please enable SSE2 (-msse2 -mfpmath=sse)"

So it seems that SSE2 was not turned on. After a Google search, I found that I have to set "Enable Enhanced Instruction Set" to "SSE2", but I still get the same error. Looks like the SSE2 defines are different on Windows?
Has anyone has any luck compiling a Windows DLL from this? Are there possibly any binaries available somewhere? Or can you give me some hints about the above problems?
Thanks,
Torben

Does it compile if you delete the "warning" line?
Also, for a shared library under Windows, all symbols must be exported in the header file with a special syntax: https://msdn.microsoft.com/en-us/library/a90k134d.aspx.
I'll try to fix those issues once I get access to VS next week.

Thanks for the feedback. I commented out the warning line, and also added __declspec(dllexport) (through a LIBRARY_API define) to all functions in ebur128.h. Now I managed to build ebur128.dll. Also checked it with dllexp.exe, all function exports seem to be there. So far so good.
I'm not a C expert, so I don't feel like providing a proper, clean pull request for this.
And also, this is a build without Speex (and maximum peak) though. Can you provide build instructions for that?
Thank you so much for your efforts. This is the library I have been looking for for years.
I will contribute the Delphi headers once I have them ready.

Any progress here? Would really love to use this lib in my Delphi project.

I have fixed some of those issues locally. I'll commit/push them ASAP. I haven't tried to build speexdsp with Visual Studio yet, but it should be possible with their VS solutions (https://github.com/xiph/speexdsp/tree/master/win32/VS2008/libspeexdsp).

Sounds great, thanks. If you were able to produce a DLL during your tests, could you post it please?

Hello,
I sent in a pull request earlier today (#52) that should help. The commit should resolve the SSE2 compile issues, and ensures a MSVC-compiled shared library exports symbols (via a def file).
With that applied, you should be able to compile on Windows - just be sure to set the ENABLE_INTERNAL_QUEUE_H to ON in the cmake options.

As a side note, using cmake cross compile I get:
-- queue.h:        no      using system copy of queue.h
but then
/Users/packrd/dev/ruby/2ffmpeg-windows-bh/sandbox/win32/lib_ebur128_git/ebur128/ebur128.c:12:23: fatal error: sys/queue.h: No such file or directory
perhaps it could check for system copy existence as well? (fix is to use the ENABLE... flag SZ-MSFT mentioned). Thanks!

There appears to be a bug in the CMake script where, even when you don't have a valid system queue.h, configuration still succeeds. I've updated my pull request with a fix. In the main CMakeLists file, just change:
if(NOT SUMMARY_HAS_QUEUE AND NOT ENABLE_INTERNAL_QUEUE_H)
to
if(NOT ${SUMMARY_HAS_QUEUE} AND NOT ${ENABLE_INTERNAL_QUEUE_H})
This should cause the configuration step to fail, and prompt the user to use the internal queue. I'm not sure if this will resolve the issue when cross compiling, though.

I did the above RE: queue.h. Now I have (on visual studio 2015):
C:\Users\gnewell\Downloads\libebur128-master (1)\libebur128-master\ebur128\ebur128.c(27): error C2061: syntax error: identifier 'SLIST_ENTRY'
2>C:\Users\gnewell\Downloads\libebur128-master (1)\libebur128-master\ebur128\ebur128.c(28): error C2059: syntax error: '}'
2>C:\Users\gnewell\Downloads\libebur128-master (1)\libebur128-master\ebur128\ebur128.c(52): error C2079: 'block_list' uses undefined struct 'ebur128_double_queue'
2>C:\Users\gnewell\Downloads\libebur128-master (1)\libebur128-master\ebur128\ebur128.c(54): error C2079: 'short_term_block_list' uses undefined struct 'ebur128_double_queue'
2>C:\Users\gnewell\Downloads\libebur128-master (1)\libebur128-master\ebur128\ebur128.c(293): warning C4013: 'SLIST_INIT' undefined; assuming extern returning int
2>C:\Users\gnewell\Downloads\libebur128-master (1)\libebur128-master\ebur128\ebur128.c(350): warning C4013: 'SLIST_EMPTY' undefined; assuming extern returning int
2>C:\Users\gnewell\Downloads\libebur128-master (1)\libebur128-master\ebur128\ebur128.c(351): warning C4013: 'SLIST_FIRST' undefined; assuming extern returning int
2>C:\Users\gnewell\Downloads\libebur128-master (1)\libebur128-master\ebur128\ebur128.c(351): warning C4047: '=': 'ebur128_dq_entry *' differs in levels of indirection from 'int'
2>C:\Users\gnewell\Downloads\libebur128-master (1)\libebur128-master\ebur128\ebur128.c(352): warning C4013: 'SLIST_REMOVE_HEAD' undefined; assuming extern returning int
2>C:\Users\gnewell\Downloads\libebur128-master (1)\libebur128-master\ebur128\ebur128.c(352): error C2065: 'entries': undeclared identifier
2>C:\Users\gnewell\Downloads\libebur128-master (1)\libebur128-master\ebur128\ebur128.c(356): warning C4047: '=': 'ebur128_dq_entry *' differs in levels of indirection from 'int'
2>C:\Users\gnewell\Downloads\libebur128-master (1)\libebur128-master\ebur128\ebur128.c(357): error C2065: 'entries': undeclared identifier
2>C:\Users\gnewell\Downloads\libebur128-master (1)\libebur128-master\ebur128\ebur128.c(413): fatal error C1021: invalid preprocessor command 'warning'

Progress.
I needed to:
#include ".\queue\sys\queue.h"
and comment out:
//#warning "manual FTZ is being used, please enable SSE2 (-msse2 -mfpmath=sse)"

Is that issue using the patch I submitted for Win32 compilation, or the main branch? The patch should instruct VS to build with SSE2 enabled and set flags to prevent SSE2 warnings.
With the patch applied, it is building without error on my system (VS2015).

Pretty sure I just downloaded the main branch.
It wasn't clear to me what the other branch was.

The issues should be fixed in pull request #52 - it should be the same as the main branch, but with a few CMake changes applied to ensure smooth compilation on windows (as well as a def file to ensure symbol exports).

I've merged PR #52 and added some new library symbols to the ebur128.def. Hopefully building/using the library should "just work" now in VS.

I'm trying to build it but I get an error:
D:\3rdparty\libebur128\build>cmake ..
-- Building for: Visual Studio 14 2015
-- The C compiler identification is MSVC 19.0.23506.0
-- Check for working C compiler using: Visual Studio 14 2015
-- Check for working C compiler using: Visual Studio 14 2015 -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Status          found / disabled --
-- queue.h:        no      using system copy of queue.h
-- build static library and shared library!
CMake Error at CMakeLists.txt:33 (message):
  queue.h not found, please set ENABLE_INTERNAL_QUEUE_H to ON

-- Configuring incomplete, errors occurred!
See also "D:/3rdparty/libebur128/build/CMakeFiles/CMakeOutput.log".

cmake .. -DENABLE_INTERNAL_QUEUE_H=ON should do the trick.

Builds both static and dynamic now. Thanks!

Great! Closing.


WIP chat stuff

way not ready , just uploading this to github

replaced by: https://github.com/keybase/client/pull/4743


Remove version: MonkeysAudio.MonkeysAudio version 1.029

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://monkeysaudio.com/files/MAC_1029.exe
Response status code does not indicate success: 404 (Not Found).
(Automated message - build 694)


removing the 'pin' item from connection_kwargs in serial mode

In serial mode I got a ValueError exception, raised by pyserial because of the pin item in connection_kwargs:
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/dist-packages/samsung_mdc/cli.py", line 542, in call
    await mdc_call(connection, display_id)
  File "/usr/local/lib/python3.10/dist-packages/samsung_mdc/cli.py", line 388, in mdc_call
    _repr(await self.mdc_command(connection, display_id,
  File "/usr/local/lib/python3.10/dist-packages/samsung_mdc/command.py", line 51, in __call__
    await connection.send(
  File "/usr/local/lib/python3.10/dist-packages/samsung_mdc/connection.py", line 165, in send
    await self.open()
  File "/usr/local/lib/python3.10/dist-packages/samsung_mdc/connection.py", line 82, in open
    await wait_for(
  File "/usr/local/lib/python3.10/dist-packages/samsung_mdc/connection.py", line 19, in wait_for
    return await asyncio.wait_for(aw, timeout)
  File "/usr/lib/python3.10/asyncio/tasks.py", line 445, in wait_for
    return fut.result()
  File "/usr/local/lib/python3.10/dist-packages/serial_asyncio/__init__.py", line 504, in open_serial_connection
    transport, _ = await create_serial_connection(
  File "/usr/local/lib/python3.10/dist-packages/serial_asyncio/__init__.py", line 448, in create_serial_connection
    serial_instance = serial.serial_for_url(*args, **kwargs)
  File "/usr/lib/python3/dist-packages/serial/__init__.py", line 87, in serial_for_url
    instance = klass(None, *args, **kwargs)
  File "/usr/lib/python3/dist-packages/serial/serialutil.py", line 241, in __init__
    raise ValueError('unexpected keyword arguments: {!r}'.format(kwargs))
ValueError: unexpected keyword arguments: {'pin': None}

Thanks for PR.
I believe PIN with TLS should be used also for pyserial, if provided, not just removed, while i didn't had chance to actually test it.
I've fixed with https://github.com/vgavro/samsung-mdc/commit/5c2b8f9ddb1a7c9d8d09f248a299335ee578b02d and  released 1.12.1 - plz confirm it's working for you.

Ok, I thought PIN is only a feature when TCP connection is used. I tested it and it is working for me. Thank you :)

Thanks, @imolein, maybe you're right.
I'm not sure if PIN/TLS is actually may be used with serial connection for Samsung devices, while TLS (without client's cert validation) doesn't make sense for serial, PIN authentication MAY make sense, and while PIN/TLS is coupled together with MDC protocol as far as I've seen, it's better to leave this decision to end user.
Still, it's interesting - if you set PIN in device settings, can you connect with serial MDC without using this PIN? Or maybe there is other kind of auth flow, when you provide PIN without switching to TLS?
This PIN/TLS functionality was developed by reverse engineering of reference application, and I've never found any docs on it, can't test it with actual Samsung device (because I don't have any anymore), so leaving this comment for Google.


migrate tests in detekt-core to junit

This is part of #4450 and migrates the tests in detekt-core to junit

Do you mind rebasing? Hopefully that will trigger the codecov report. It's a nice sanity check to make sure coverage didn't change.


Backbport 3326 logging yaml

This PR backports #3326 :
Added logging.yaml support (#3326)

Added logging.yaml support to generate a yaml file if the logging.yaml file is specified as output. Tested with CIAB and it appears to work

(cherry picked from commit 8fcde36019e37965ad5d1dd137032f26e3e4113d)

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/4171/
Test PASSed.


Differentiate between missing data and untranslatable codon

From the tool documentation:

For amino acids, X denotes missing data/an untranslatable codon

Can the tool output a different character for missing data (e.g. "N"?) and untranslatable codon?
Thank you in advance!

Hello,
I don't want to use an N for missing data because it is also the IUPAC amino acid code for Asparagine.
Could you give me an example of two codons, one of which is untranslatable, and the other of which has missing data, so that I can think about how the two cases are different and how we should handle them?
Thanks,
Ben

Hello Ben,
I see your rational for not using N.
You mean all untranslatable codons will result from missing data in the input consensus FASTA, right?
Thank you!
Javier

CCN is a translatable codon because it is always a P irrespective of what the N is, whereas CAN could either be a Q or an H and is thus untranslatable (and is denoted X).

@jaquol sorry, I probably confused things initially more than I helped.
I believe that the current behaviour is:
    CCN will aways be translated to P by type_variants and CAN or NNN will be X
    CC- (or any other codon which is partially or completely gaps in the input alignment) will also return X.

All I was really trying to understand with my first response was whether there was a specific case that you had in mind which you could give as an example with the following things included:

the input
the current output
what you would prefer the output to be

just to help me understand exactly what behaviour you are requesting (apologies if I'm being slow on the uptake).

Thank you @benjamincjackson for the additional explanation, which makes sense! I guess that X for CAN, NNN or CC- is informative enough.


Update the Twitter SVG to X SVG

Here's the official SVG provided by the X brand toolkit:
<svg xmlns="http://www.w3.org/2000/svg" width="1200" height="1227" viewBox="0 0 1200 1227" fill="none">
<path d="M714.163 519.284L1160.89 0H1055.03L667.137 450.887L357.328 0H0L468.492 681.821L0 1226.37H105.866L515.491 750.218L842.672 1226.37H1200L714.137 519.284H714.163ZM569.165 687.828L521.697 619.934L144.011 79.6944H306.615L611.412 515.685L658.88 583.579L1055.08 1150.3H892.476L569.165 687.854V687.828Z" fill="white"/>
</svg>

See https://github.com/carbon-design-system/carbon/issues/14997

For future readers:
The X logo was added last year to the upstream library, also available in this library.
<script>
  import LogoX from "carbon-icons-svelte/lib/LogoX.svelte";
</script>

<LogoX />


fix: Move card image api route

Description
I think we're running into this netlify caching issue for the static image endpoint. https://answers.netlify.com/t/netlify-function-with-query-strings-ignores-custom-cache-control-header/15390/1
I can reproduce it if i go to one of these URLs and then the other one.

https://insights.opensauced.pizza/api/card.png?username=foxyblocks
https://insights.opensauced.pizza/api/card.png?username=bdougie

This change moves the endpoint so that the username parameter is part of the path so the cache is correctly busted for different users.
I've yet to figure out how to access these parameters from the request other than doing string parsing. request.nextUrl.searchParams.get('username') works locally but doesn't work when deploying to netlify, probably due to some difference in runtime environment.
What type of PR is this? (check all applicable)

[x] 🐛 Bug Fix

Related Tickets & Documents

Mobile & Desktop Screenshots/Recordings

Added tests?

[ ] 👍 yes
[ ] 🙅 no, because they aren't needed
[x] 🙋 no, because I need help

Added to documentation?

[ ] 📜 README.md
[ ] 📓 docs.opensauced.pizza
[ ] 🍕 dev.to/opensauced
[ ] 📕 storybook
[x] 🙅 no documentation needed

Should we merge this today or wait til monday?


Updated MIMIC ETL and transforms

Adding custom transform and ETL for MIMIC-IV to handle data errors present in source data.
A small number of visits in the original MIMIC-IV data have a end date that occurs before the start date, which is causing the labeling functions to break.
The MIMIC ETL takes visits where the end date occurs before the start date and replaces the erroneous end date with the start date. This fixes the issues that were causing the labeling functions to break.

Started from main and re-added @joshualemmon 's MIMIC transforms and ETL.


Correc the Yarn site property resource.percentage-physical-cpu-limit

The supported yarn-site property name in is yarn.nodemanager.resource.percentage-physical-cpu-limit.

Looks good to me.

Added breaking as it it causes kernel panics on stock 12.04

BREAKING: Until a newer kernel is installed (e.g. as per PR #377 ) this can cause your NM machines to kernel panic the mechanism is not fully understood.
It looks like YARN may trigger the kernel panics per Canonical due to walking /proc. Past examples of YARN and kernel interactions are (but seem unrelated):

YARN-2809 - Implement workaround for linux kernel panic when removing cgroup
YARN-4048 - Linux kernel panic under strict CPU limits
Further, it was found the following was not the issue per a hotfix attempt:
LP #1534413


Add more filters to identities

:+1:

@KaitCrawford updated again, looks like the dates needed declaring. Now supports /api/v1/identities/?created_from=2016-03-17T14:00:39.168705Z&created_to=2016-04-21T09:16:00.952922Z

@imsickofmaps ah ok! :+1:


Build Error

I'm trying to install fbcunn and I tried to run the build script in Ubuntu 14.04 and got the following error:
/tmp/fblualib-build.VwAHdm/fblualib/fblualib/util/Util.cpp: In function ‘bool eventBaseRunAfterDelay(folly::EventBase*, int, void (*)())’:
/tmp/fblualib-build.VwAHdm/fblualib/fblualib/util/Util.cpp:217:46: error: void value not ignored as it ought to be
     return eb->runAfterDelay(fn, milliseconds);
                                              ^

I've tried to use this https://github.com/soumith/fblualib/blob/master/install_all.sh and got the same error too.
Any idea? Thanks.

I believe the error is because of a recent change in folly.
https://github.com/facebook/folly/commit/1a61493fec7ca4d8d64e8325ba7bbe82fa78116a
I guess the code here needs to be updated.

Had same issue; needed to build and make install folly release 0.22.0 separately

fixed now in master!


Refactoring support: Reorder parameters

I would really like something like reorder parameters as a refactoring option, like C# already has: http://msdn.microsoft.com/en-us/library/5ss5z206.aspx
edit:
This might be a bit hard when the info comes from a .d.ts file, though.

Please reopen this issue, #13080 marked duplicated and closed and this one marked closed because linked to the duplicated item.

@wongchichong, this issue is still open ;)

You seem to be confusing the posting in this issues timeline of the other issue.

Yaya.... LOL..

This refactoring would also be useful on function types.
type MyFunction = (foo: string, bar: number) => void;

function doSomething (fn: MyFunction): void {
    fn('foo', 4);
}

doSomething((foo, bar) => foo + (2*bar));

// A reordering refactor on MyFunction would change all this to:

type MyFunction = (bar: number, foo: string) => void;

function doSomething (fn: MyFunction): void {
    fn(4, 'foo');
}

doSomething((bar, foo) => foo + (2*bar));

Especially useful with generic type parameters

Is there any way I can vote for this issue? This would be a really convenient refactor feature.

I miss this feature often: Reordering parameters (of functions, methods, constructors) manually is painful and error-prone. Alas, it’s an important part of improving APIs.


Multiple Units create multiple identical lines in .csv

Hi,
Here is an example.

Is this necessary?
What, when 3 of 4 identical lines get deleted before back-importing?
Greets,
Karl

I'll try to make an test file next week, to try to make an elegant solution for this issue.

I had a look into this again.
I think, this could be a first approach:

On loading the schematic: collect all references in a map (key, value) references, where the key is the components Reference and the value is another map unitInstances, where the key is the filename and the value is a link to the Component object.
On export, we export for each entry in the map references the first entry of the sub-map unitInstances. Optionally we can do a "merge" of the fields of the different units. It could be, that the value is only set in one unit, and was not copied to the other units. Same could be for the footprint, datasheet and custom fields (properties) and even the component name (Part). I don't know, if there are easy to use merge-libraries available, which can merge different strings into one common string.
this happens then for a random file name, if the units are distributed over multiple schematic files.
On import, we update all matching fields of all entries of the corresponding sub-map unitInstances.
On save, there shouldn't be any difference, since the sub-map contains only links to the Component objects.


Reduce size of toolbar in the panels

closes : https://github.com/zakodium-oss/react-science/issues/679

I'm not in favour of this. The icons look blurry.
The icons are made to be displayed 16x16 and there is not much we can do about it because smaller versions of the icons do not exist.
@lpatiny

16x16 seems to be a standard size for small icons, we should not try to make it smaller or people will have trouble to see them.

@targos Do I close this PR?

I think so, yes


Needed: variant of assertMacroExpansion() that does not use XCTest

Description
Test targets that use swift-testing instead of XCTest need some way to work with this function; let's add a throwing equivalent that can be used without any dependency on a specific testing library.

Tracked in Apple’s issue tracker as rdar://119519713

Implemented in https://github.com/apple/swift-syntax/pull/2647


[Chips] BIg chip broke layout inside Autocomplete inside dialog

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯

Pop a Dialog
Put a multiple autocomplete with an option with very long name in the dialog
Set renderTags into a map of Chip components
Select the option with super long name
See the dialog layout broken with the chip taking more than 100% of the dialog width

First screenshot is when you select short / normal option, second after you add a very long named option.
Expected behavior 🤔

Pop a Dialog
Put a multiple autocomplete with an option with very long name in the dialog
Set renderTags into a map of Chip components
Select the option with super long name
See the Chip with an elipsis or whatever needed to crop it to max width of the autocomplete

Steps to reproduce 🕹

Pop a Dialog
Put a multiple autocomplete with an option with very long name in the dialog
Set renderTags into a map of Chip components
Select the option with super long name
See the dialog layout broken with the chip taking more than 100% of the dialog width

Context 🔦
I guess what I'm trying to accomplish is easily understandable, wanna do a list choose in a dialog, rendered as Chips.
Your environment 🌎

npx @mui/envinfo

System:
OS: Linux 5.13 Ubuntu 20.04.3 LTS (Focal Fossa)
Binaries:
Node: 16.13.0 - ~/.nvm/versions/node/v16.13.0/bin/node
Yarn: Not Found
npm: 8.1.0 - ~/.nvm/versions/node/v16.13.0/bin/npm
Browsers:
Chrome: 98.0.4758.80
Firefox: 96.0
Using Chrome atm.

Found the fix and PR is created
https://github.com/mui/material-ui/pull/31087

Please provide a minimal reproduction test case with the latest version. This would help a lot 👷 .
A live example would be perfect. This codesandbox.io template may be a good starting point. Thank you!
I have tried in https://codesandbox.io/s/tags-material-demo-forked-dret2?file=/demo.js, I can't.

@oliviertassinari
Issue is coming when we use Select with Chip inside dialog, and with lengthy options ( try using lengthy options without dashes )
You can use below link to check ( Reproduced issue )
https://codesandbox.io/s/formdialog-material-demo-forked-18sqb?file=/demo.js

@PunitSoniME Thanks, I have simplified the reproduction down in https://codesandbox.io/s/formdialog-material-demo-forked-ef9sw2?file=/demo.js. It seems that the Autocomplete solve the problem like this:
https://github.com/mui/material-ui/blob/4d1938c6f970bb930235ed3340fdbc77eaea09d3/packages/mui-material/src/Autocomplete/Autocomplete.js#L101
Hence why I couldn't reproduce with this component.


It's not possible to delete a stack

I've tried this twice now, but every time I try to delete a stack it gives the following message:

Simply stopping the containers and deleting the compose.yml file results in a dangling stack that cannot be removed, as it is 'not managed by Dockge'.

Just released 1.0.4, it should be fixed.