Enabling jsonProtocol preview feature breaks Prisma Data Platform

After enabling the jsonProtocol feature on the new 4.11 release, the data browser and query console both are broken since it fails to properly parse the prisma.schema. I also noticed the the data platform is reportedly still using client version 4.10.1, perhaps it just needs an upgrade to the new release?
Raw error message I am receiving when trying to access the data browser now:
Error starting Prisma Client: {
  "type": "PrismaClientKnownRequestError",
  "code": "P1012",
  "message": "Prisma schema validation - (get-dmmf wasm)\nError code: P1012\n\u001b[1;91merror\u001b[0m: \u001b[1mThe preview feature \"jsonProtocol\" is not known. Expected one of: fullTextSearch, fullTextIndex, tracing, metrics, orderByNulls, multiSchema, filteredRelationCount, fieldReference, postgresqlExtensions, clientExtensions, deno, extendedWhereUnique, views\u001b[0m\n  \u001b[1;94m-->\u001b[0m  \u001b[4mschema.prisma:6\u001b[0m\n\u001b[1;94m   | \u001b[0m\n\u001b[1;94m 5 | \u001b[0m  provider = \"prisma-client-js\"\n\u001b[1;94m 6 | \u001b[0m  previewFeatures = \u001b[1;91m[\"jsonProtocol\"]\u001b[0m\n\u001b[1;94m   | \u001b[0m\n\nValidation Error Count: 1\n[Context: getDmmf]\n\nPrisma CLI Version : 4.10.1",
  "stack": "Error: Prisma schema validation - (get-dmmf wasm)\nError code: P1012\n\u001b[1;91merror\u001b[0m: \u001b[1mThe preview feature \"jsonProtocol\" is not known. Expected one of: fullTextSearch, fullTextIndex, tracing, metrics, orderByNulls, multiSchema, filteredRelationCount, fieldReference, postgresqlExtensions, clientExtensions, deno, extendedWhereUnique, views\u001b[0m\n  \u001b[1;94m-->\u001b[0m  \u001b[4mschema.prisma:6\u001b[0m\n\u001b[1;94m   | \u001b[0m\n\u001b[1;94m 5 | \u001b[0m  provider = \"prisma-client-js\"\n\u001b[1;94m 6 | \u001b[0m  previewFeatures = \u001b[1;91m[\"jsonProtocol\"]\u001b[0m\n\u001b[1;94m   | \u001b[0m\n\nValidation Error Count: 1\n[Context: getDmmf]\n\nPrisma CLI Version : 4.10.1\n    at /var/task/node_modules/.pnpm/@prisma+internals@4.10.1/node_modules/@prisma/internals/dist/engine-commands/getDmmf.js:149:12\n    at n.e.run (/var/task/node_modules/.pnpm/ts-pattern@4.1.3/node_modules/ts-pattern/dist/index.cjs:1:4927)\n    at n.e.exhaustive (/var/task/node_modules/.pnpm/ts-pattern@4.1.3/node_modules/ts-pattern/dist/index.cjs:1:4613)\n    at getDMMF (/var/task/node_modules/.pnpm/@prisma+internals@4.10.1/node_modules/@prisma/internals/dist/engine-commands/getDmmf.js:153:6)\n    at processTicksAndRejections (node:internal/process/task_queues:96:5)\n    at async Object.getDmmf (/var/task/apps/fox/.next/server/chunks/9033.js:8250:25)\n    at async /var/task/apps/fox/.next/server/pages/api/databrowser.js:882:30\n    at async /var/task/apps/fox/.next/server/chunks/9033.js:10194:13\n    at async runMiddlewares (/var/task/apps/fox/.next/server/chunks/9033.js:10200:5)\n    at async /var/task/apps/fox/.next/server/chunks/9033.js:10202:23"
}
init@https://cloud.prisma.io/studio/assets/index.js:1:82961

Thanks, that is an aspect we did indeed not consider. The team is on it. Let's see if the update makes it work or if the old protocol is somehow hard coded there.

We can confirm this problem, both specifically in Data Browser and Query Console but also with Data Proxy in general.

I get a similar error but on  npx prisma generate
❯ npx prisma generate
Environment variables loaded from .env
Prisma schema loaded from prisma/schema.prisma
Error:
✔ Generated Prisma Client (4.11.0 | library) to ./node_modules/@prisma/client in 227ms

Prisma schema validation - (get-dmmf wasm)
Error code: P1012
error: The preview feature "jsonProtocol" is not known. Expected one of: fullTextSearch, fullTextIndex, tracing, metrics, orderByNulls, multiSchema, filteredRelationCount, fieldReference, postgresqlExtensions, clientExtensions, deno, extendedWhereUnique, views
  -->  schema.prisma:6
   |
 5 |     provider        = "prisma-client-js"
 6 |     previewFeatures = ["jsonProtocol"]
   |

Validation Error Count: 1
[Context: getDmmf]

Prisma CLI Version : 4.10.1❯ npx prisma generate
Environment variables loaded from .env
Prisma schema loaded from prisma/schema.prisma
Error:
✔ Generated Prisma Client (4.11.0 | library) to ./node_modules/@prisma/client in 227ms

Prisma schema validation - (get-dmmf wasm)
Error code: P1012
error: The preview feature "jsonProtocol" is not known. Expected one of: fullTextSearch, fullTextIndex, tracing, metrics, orderByNulls, multiSchema, filteredRelationCount, fieldReference, postgresqlExtensions, clientExtensions, deno, extendedWhereUnique, views
  -->  schema.prisma:6
   |
 5 |     provider        = "prisma-client-js"
 6 |     previewFeatures = ["jsonProtocol"]
   |

Validation Error Count: 1
[Context: getDmmf]

Prisma CLI Version : 4.10.1

Note sure why it says prisma CLI version is 4.10.1 even though I upgraded prisma with npm i prisma@latest and package.json shows 4.11.0

What you seen in the output is truth @arnm - so somehow your local node_modules folder still contains the previous version @arnm. I suggest you double check this, and make sure that prisma -v actually shows the correct version. (Often nuking the node_modules folder and re-running npm install fixes these problems.)

Could you figure out what was wrong in your project @arnm?
@craigsc We now understand the problem in Data Browser and Query Console better: Although you upload or use a Prisma schema with previewFeatures = ["jsonProtocol"] the Prisma Client of these two components uses an empty previewFeatures array. When it then talks to the Data Proxy under the hood, that one parses the array correctly from the schema file though and expects to receive JSON - but then gets GraphQL and does not like that and crashes. For now you unfortunately can not use jsonProtocol with Data Browser or Query Console.

This should be fixed now @craigsc.
Thanks again for reporting it originally.


RFP - flameshot

Checklist

[x] The package I am requesting does not already exist on https://chocolatey.org/packages;
[x] There is no open issue for this package;
[x] The issue title starts with 'RFP - ';
[x] The download URL is public and not locked behind a paywall / login;

Package Details
Software project URL : https://github.com/flameshot-org/flameshot/
Direct download URL for the software / installer : https://github.com/flameshot-org/flameshot/releases/download/v0.9.0/Flameshot-0.9.0-win64.msi
Software summary / short description: Powerful yet simple to use screenshot software
Additional Details
Already available on:

scoop
winget

I will pick this up

Actually, it appears like this package has been created since this issue was opened:
https://community.chocolatey.org/packages/flameshot#versionhistory


feat: types auto import

Resolve https://github.com/nuxt/nuxt/issues/12808#issuecomment-1420406745
Resolve https://github.com/antfu/unplugin-auto-import/issues/61
It seems to work with my TypeScript v4.9.5, not even requiring 5.0.

🤯 This is phenomenal. Tested and working locally in Nuxt in IDE and in nuxi typecheck with:

Also working on IntelliJ Ultimate Edition:

But auto imports seems to be broken:


Unable to validate coupon codes that are longer than 3 parts

Hi there,
Firstly - thank you for your gem - it's really appreciated. I"m having an issue with getting validation working with codes that are longer than three parts.
Here's how to reproduce:
working = CouponCode.generate # => "HRVE-2BAU-P2JD"
CouponCode.validate(working) # => "HRVE-2BAU-P2JD"

not_working = CouponCode.generate(parts: 4) # => "XTVE-A8KF-6PVL-EH6M"
CouponCode.validate(not_working) # => nil

Any ideas? I hope you can help :)

Hey, I issued a PR for this, see #5


added column "Lane" to amplicon sample sheet

To address issue #239 . @AmandaBirmingham @tanaes
Currently, the "Lane" column has been successfully added to the amplicon sample sheet. This was achieved by modifying the _generate_amplicon_sample_sheet function.
However I felt that this function was originally flawed. It only generates one row of the whole sample sheet, rather than the entire sample list, as the counterpart _generate_shotgun_sample_sheet does.

Hi @AmandaBirmingham I think I have tackled all the requirements. Thanks in advance for reviewing! A separate email with example result files will be sent out.

@AmandaBirmingham Thanks for reviewing! I will take care asap.

Hi @AmandaBirmingham I addressed your first two comments. But for the third one I have difficulties. I can't figure out how to hard code a SequencingProcess object, as this line does. Do you have some ideas?

@qiyunzhu Uggh.  We shall have to modify populate_test_db.sql.  I suspect it will go fastest if we do it together; email me to let me know when you're available today to pair-program it.

Hi @AmandaBirmingham I figured out a workaround. Everything is okay now!

Hi @wasade Thanks for reviewing! I have made edits and comments.

Ahh!  Now I gotcha!  Thank you for clarifying for me :)
On Jul 27, 2018, at 12:21 PM, Qiyun Zhu <notifications<EMAIL_ADDRESS>wrote:
@qiyunzhu commented on this pull request.

In labman/db/process.pyhttps://github.com/jdereus/labman/pull/268#discussion_r205875461:

@@ -2968,7 +2968,7 @@ def _generate_amplicon_sample_sheet(self):
% ('Lane,' if self.include_lane else '')]
for pool, lane in self.pools:
data.append('%s%s,,,,,NNNNNNNNNNNN,,%s,,,'

                   % (('%s,' % (lane if self.include_lane else '')),

                   % (('%s,' % lane) if self.include_lane else '',
                      self._bcl_scrub_name(pool.container.external_id),

The correct logic is:
if self.include_lane:
return '%s,' % lane
else:
return ''
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/jdereus/labman/pull/268#discussion_r205875461, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AKLur2_EUH5iJuUVwd3giyDuoBoulZV6ks5uK2gvgaJpZM4Vg2qQ.


[spark][pyspark][sql] Correcting code for Documentation Strings(PEP8)

What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
Just correcting an unnecessary interval on  [column.py] line 133.  It does not fit Documentation Strings(PEP8).
How was this patch tested?
Change is not affecting any other code.
Please review http://spark.apache.org/contributing.html before opening a pull request.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?


[Authentication] Flash a message if the user's entered password is incorrect.

When the user entered a wrong password, a message will flash on the login page.
You may need to communicate with #77 about the HTML file.

I will be working on that issue. @Omeramsc


Add batch normalization to TF models

Notes: https://github.com/tensorflow/tensorflow/blob/master/tensorflow/contrib/layers/python/layers/layers.py
Will add to reLSTM, not simple linear models.


Calling toast.promise() in componentDidMount()

Hi, I'm having an issue using toast.promise() and I can't figure out what's wrong with my code. The loading toast doesn't get displayed at all. The success/error toasts appear after the fetch is complete, all good here.
Weirdly, if I uncomment the toast("Hey") (see code below) in the render function, then the loading toast is displayed too (there's also a number that appears on top of the page).
Steps to reproduce on a fresh React install:

npx create-react-app toaster-example
cd toaster-example
npm install react-hot-toast
Use the following code in src/App.js:

import React from "react";
import logo from "./logo.svg";
import "./App.css";
import toast, { Toaster } from "react-hot-toast";

const notify = () => toast("Here is your toast.");

class App extends React.Component {
  componentDidMount() {
    console.log("mounted");

    // Random API from https://github.com/public-apis/public-apis
    const fetchPromise = fetch(`https://randomfox.ca/floof/`);
    toast.promise(fetchPromise, {
      loading: "Loading",
      success: "Got the data",
      error: "Error when fetching",
    });
  }
  render() {
    return (

        {/* {toast("Hey")} */}
        <Toaster />

          <button onClick={notify}>Make me a toast</button>

            Edit src/App.js and save to reload.

            Learn React

    );
  }
}

export default App;

Thanks

I have same issue.

I ended up having a acceptable workaround for this.
I put  at ReactDOM.render level:
ReactDOM.render(   <React.StrictMode>     <App />     <Toaster />   </React.StrictMode>,   document.getElementById('root') );

The toast won't show on the first page load in useEffect when I'm using toast(), until I add setTimeout:
  useEffect(() => {
    const timeout = setTimeout(() => {
      toast(My toast);
    });
    return () => clearTimeout(timeout);
  }, []);


Is there a Slack channel or anything?

Description:
A/A.  Where does one chat, or just ask questions and what not, without raising a bunch of 'issues' which are more likely issues I have, rather than the ractive code has ;)

Twitter is more or less it, but it's not so great for detailed conversations. I've created a slack channel at ractivejs.slack.com to see how that works out. There's also a gitter, but it seems harder to monitor.

Thanks... how do I get an invite? :)

I suppose send an email to my username without the hyphen at gmail and I can add you. I've not used slack before, so it's a bit experimental. @fskreuz can I make you an admin as well?

Done.

@evs-chris  I sent for invite too :)

To be clear, at gmail, I'm evschris 😃. It seems that any member can invite others by default, which seems reasonable.
Just gonna ping a few more ractive regulars off the top of my head in case they're interested @JonDum @aharpervc @dagnelies @Jeff17Robbins @andreujuanc @brakmic @TehShrike

That's where I sent it. :)
When it's set up, perhaps it can be added onto the homepage markdown asap, so other people will know it's there in future after this is closed.

@paulie4  might be interested (and hopefully will explain to me what I need to do?!  😃)

What about this:
https://gitter.im/
I think it's quite neat for github based communities.

@dagnelies Whatever has a consensus, I don't mind... there's millions of these chat things.

I can see people already googlin' "slack vs gitter" xD

I was probably doing something wrong, but my issue with gitter was lack of notifications. So far, slack seems to be quite good at making monitoring easy.

I recommend asking questions on Github tho. That way, it's sort of "written in stone" and anyone stumbling across the same problem can just ask [insert your favorite search engine] and be pointed back to the same issue.
For lengthy discussions, a chat room wouldn't be that bad.

+1 for Gitter : https://gitter.im/ractivejs/ractive

So it turns out that I hadn't actually managed to "join" the Ractive gitter 🤣. Since its barrier to entry is much lower, I think I'll point a chat link to the gitter, but I'll still watch the slack channel for those who prefer it.


Feature request. Fresh SOLT benchmarks

Benchmarks for SOLT are a bit outdated.

fixed in https://github.com/albu/albumentations/pull/227


pull request for e2e test at 01/14/2017 22:15:13

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


Update resultList format & images

Done

Upload images
Update resultList format

    [{
        "Level": 1,
        "Name": "Amoeba",
        "Top": "100",
        "Description": "No brain",
        "ReferenceTitle": "",
        "ReferenceLink": ""
    }],

Please split commits for the next time.
LGTM with a minor comment.


Create an intro

Add an intro about yourself: who are you, what is your strength, why are you here.

Done - https://github.com/ticial/homepage/pull/32/commits/e5a5886145a7086d8c5080a18c7c4b5f0ea0af90


About the structure of the DDRNet

Dera Yuanduo
Thanks very much for your perfect Network.
I have a question about the structure of the DDRNet.
In your program code, the layer of the bottleneck block in the low-resolution branch was made with stride=2, I think after this bottleneck block, the output size will become 1/2.
And in your paper, as the image below shows, in the conv5_1 of the low-resolution branch,  one Residual basic block (stride=2) and one bottleneck block(stride=2) have been used. Why does the output size change from14x14 to 7x7 after two blocks whose stride is both equal to 2?

I would appreciate it if you would answer my question.

Dera Yuanduo
Thanks very much for your perfect Network. I have a question about the structure of the DDRNet.
In your program code, the layer of the bottleneck block in the low-resolution branch was made with stride=2, I think after this bottleneck block, the output size will become 1/2. And in your paper, as the image below shows, in the conv5_1 of the low-resolution branch, one Residual basic block (stride=2) and one bottleneck block(stride=2) have been used. Why does the output size change from14x14 to 7x7 after two blocks whose stride is both equal to 2?
I would appreciate it if you would answer my question.

Stride = 2 is for cityscapes. For imagenet, stride should be 1. The stride can be adjusted flexibly according to the image resolution.


Add Free

Adds an instance for the Free group on a given set of generators.
Does require a couple of new dependencies (lists and foldable-traversable)
Not sure if there are any other instances that Free should have.

Some Slack conversation exposed another construction we might consider for implementation of free groups:
data FreeGrp a = GEmpty | GConsPlus a (FreeGrp a) | GConsMinus a (FreeGrp a)
I'm not sure which would be better from an API design perspective and want to solicit feedback :)

Good idea! This looks right to me from a math perspective.

In my opinion the current approach is nicer than the one suggested on Slack. But is there a reason to prefer the Slack suggestion?

Yes, I like the Signed approach better as well (and the Haskell library that exposes free groups does the same thing, for whatever that's worth.)
The upside of the GConsPlus/GConsMinus construction was that it might make the parallels to other free constructions even clearer, in that it "lifts the laws of the algebraic structure to data constructors" or something along those lines. I'm sorry if that's not too clear; I'm probably not a great advocate for something I don't really want to reimplement.
By the way, I went ahead and published this as its own library on Pursuit under the name purescript-free-group. (What can I say, I was excited to publish a library.) I still think it probably fits better as a part of purescript-group, and I'll deprecate the package if this PR gets accepted, but I would like to port over the changes I've made to this branch.

@hrb90 Let me know when(/if) you have something you feel is ready to be merged. Thank you!

Updated the code, think it's ready to merge :smile:

Merged, changed names as described, bumped version, published to Pursuit!


phpExtensions: skip performance sensitive tests

Motivation for this change
This time #150770 should be done properly by skipping performance sensitive tests for extensions. I built phpExtensions.fileinfo, and according to the logs affected tests are properly skipped.
SKIP Bug #66731: file: extensive backtracking [tests/cve-2014-3538-mb.phpt] reason: performance sensitive test
SKIP Bug #66731: file: extensive backtracking, with pcre.jit=0 [tests/cve-2014-3538-nojit.phpt] reason: performance sensitive test
SKIP Bug #66731: file: extensive backtracking [tests/cve-2014-3538.phpt] reason: performance sensitive test

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

@ofborg build phpExtensions.fileinfo

SKIP Bug #66731: file: extensive backtracking [tests/cve-2014-3538-mb.phpt] reason: performance sensitive test
SKIP Bug #66731: file: extensive backtracking, with pcre.jit=0 [tests/cve-2014-3538-nojit.phpt] reason: performance sensitive test
SKIP Bug #66731: file: extensive backtracking [tests/cve-2014-3538.phpt] reason: performance sensitive test

Cool, this actually skips affected tests according to ofborg logs (https://logs.nix.ci/?attempt_id=4ccd72b3-a9eb-40d4-8b8e-3c9ce03e9b53&key=nixos%2Fnixpkgs.150885).

And on Hydra: https://hydra.nixos.org/build/161511358


How to write tests in a scalable way

The current way of writing end to end tests involves writing build process of the app in the before hook. If someone wants to write tests in multiple files, the app needs to be built multiple times. Is it possible to

write tests in multiple files but just build the app once? or
specify only one page to build to reduce the build time while makes it possible to split the tests in multiple files?

#987


Document pgbouncer=true connection URL option

Add it to the following pages:

https://www.prisma.io/docs/reference/database-connectors/connection-urls
https://www.prisma.io/docs/reference/tools-and-interfaces/prisma-client/deployment

Duplicate of https://github.com/prisma/docs/issues/443


No way to configure MTU, for AWS, for example

Our docs currently say that you need to configure the MTU to make AWS perform but they instructions they give are involved and require nsentering the container.
That's not acceptable!  We need a better way of configuring the MTU.
A potential customer just hit this in their initial testing, giving a bad impression.

@robbrockbank I think this is with you

@TrimBiggs Tim - what happened with this one?

Closing as duplicate of #488.


Attempt to include Python 3.6 and 3.7 via conda_build_config.yaml

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

Ah, I see python 3.6/3.7 was dropped, that's why it doesn't render :)


[Appium helper] Improvements on wdiov5

This PR aims to fix/improve the following issues with appium-wdiov5:

https://github.com/Codeception/CodeceptJS/issues/1538 -> due to those such methods like swipeUp, swipeDown, swipeLeft, swipeRight are replaced by touchAction
https://github.com/Codeception/CodeceptJS/issues/1535 -> call the appium api endpoint to remove app.
https://github.com/Codeception/CodeceptJS/issues/1528 -> due to those such methods like swipeUp, swipeDown, swipeLeft, swipeRight are replaced by touchAction
https://github.com/Codeception/CodeceptJS/issues/1527 -> call the appium api endpoint to set the orientation.

Thank you!


Font-family issue

Challenge Set the Font Family of an Element has an issue.
User Agent is: Mozilla/5.0 (Linux; Android 5.0.2; SM-T350 Build/LRX22G) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/52.0.2743.98 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.

Still bug is pending

Again, it's not a bug, the issue is your code

@aman1486
Thanks for reporting this. But your code is incorrect.
GitHub Issues are for reporting bugs on the website only.
Please use the Help chat room for getting challenge related help.
Happy Coding!


Add travis script to run lint on pull requests

OMG. It actually works! It's a Christmas miracle!

@concavelenz Did you want to take a look at this before merging?

Usually, we ask folks to squash there commits before merging.  Can you do that?

Commits squashed.

merge conflict?


lub_constrain boundary behaviour

Description
The parameter constraining transform lub_constrain() has an code path with a comment
// Prevent x from reaching zero unless it really really should.

The limiting value 1e-15 chosen is so large as to make the transform non-monotone. The branch also cannot prevent rounding when the lower bound is not zero. e.g. <lower=100,upper=101> does not see any effect, other than it stopping derivative propagation.
Similar code is not present in lb_constrain() which is allowed to round to the boundary value.
A related question is what to do about the unconstraining transform lub_free(). The check_bounded() is inclusive so values at boundaries are accepted but the logit function maps boundary to infinity so it will fail later. A reasonable choice might be to map the boundary values to some finite number beyond the rounding limit. However, the rounding limit depends on the bounds in a complex fashion.
Current Version:
v3.0.0

Thanks for filing.  Is there a proposed behavior change?  I'd be OK with it throwing exceptions or just rounding.

Stan manual says declared bounds are inclusive. "Just rounding" is the expected behavior. It's also effectively the current behavior when bounds are far from zero. The only weird thing about just rounding is that lub_free(lub_constrain(x)) might not be finite even if x is.

On Nov 8, 2019, at 1:21 AM, Niko Huurre<EMAIL_ADDRESS>wrote:
Stan manual says declared bounds are inclusive.

Right, that was the original intention.  It also makes the real and int cases equivalent
in that they are inclusive boundaries.
But it wasn't particularly well thought out for reals and what would happen downstream
when plugged into something like a beta or a binomial.

"Just rounding" is the expected behavior.

That's how we coded it.

It's also effectively the current behavior when bounds are far from zero. The only weird thing about just rounding is that lub_free(lub_constrain(x)) might not be finite even if x is.

Right.  We also tried to get the constraints the other way around to make sense, so that exp(-inf) = 0 and you could round trip from constrained (0) to unconstrained (-inf) and back.  But we never got the derivatives right for any of this;  they often only exist one-sided.
I'm just saying we should probably think through again what the right behavior is.

Currently the code in lub_constrain is
  inv_logit_x = inv_logit(x);
  if ((x > NEGATIVE_INFTY) && (inv_logit_x == 0))
    inv_logit_x = 1e-15;
  return fma((ub - lb), inv_logit_x, lb);

The inverse logit does not underflow even when the result is very small.
  inv_logit(-35) = 6e-16
  inv_logit(-700) = 1e-305
  inv_logit(-750) = 0.0

That means the transform goes to very small values before jumping back up to 1e-15. When the lower bound is not zero that fma() will round these tiny numbers to the boundary.
A better option would be
  t_x = fma((ub - lb), inv_logit(x), lb);
  if ((x > NEGATIVE_INFTY) && (t_x == lb))
    t_x = boost::math::float_next(lb);
  return t_x;

A single ulp difference isn't very robust. I believe the best choice is to simply remove the branch entirely.

Yes, the branch is definitely bad news for precision.  I'm embarrassed it's in there---we were just learning floating point when writing v1 of Stan.
My question was mostly about whether we should be throwing an exception if we hit the round-to-0 or round-to-1 point.  We can get to within 1e-300 of 0 but only to within about 1e-16 of 1 before rounding.  The derivative is inv_logit'(x) = inv_logit(x) * inv_logit(-x), which rounds as the best of inv_logit(x) and inv_logit(-x) because the other rounds to 1.

Some thoughts, which I expect others have been thinking, too, but couldn't find summarized.
There is a general problem that real values are not exactly continuous, but discrete with floating point representation. In the idealized world the bounded variables often have non-inclusive bounds (e.g. strictly positive values). If Stan lower and upper would define non-inclusive bounds, it would depend on the bound value how close the closest floating point value is. In that sense, it seems more clear if Stan lower and upper would define inclusive bounds (which would match also with int type). Of course, even then if the numeric string used to present the bound can not be represented exactly with floating point represented, the stored fp value for the bound can be smaller or larger than the value presented by the numeric string.
It seems difficult to check when the fp presentation of the bound is different than the numeric string in Stan code. Bounds can be also computed on the fly, e.g. <lower=alpha>, where the accuracy of computation for alpha is usually unknown for the user.  Recently, in combination with ode, we saw negative values for parameter with constraint <lower=0>, and then got error in constraint checking. I didn't know before that this could happen.
For <lower=0>, most of the time the accuracy near 0 is sufficient and users don't see 0's which could cause problems in other parts of the code. Sometimes users write something like <lower=0.0000001>, and this has been openly criticized as bad practice, but it seems that floating point presentation is causing more headache than previously thought. Maybe we should say in Stan reference manual that e.g. <lower=0.0000001> is fine, if you want have strictly positive values.

The only weird thing about just rounding is that lub_free(lub_constrain(x)) might not be finite even if x is.

This holds in general for floating point math, e.g. log(exp(x)+exp(y)) might not be finite even if x and y are. I think it means that there should not be premature "limiting values", but instead let numerical issues to show and then require thinking numerically more stable computation, e.g. log_sum_exp.
I think it would be good to extend the discussion in Stan reference manual
section "Univariate Data Types and Variable Declarations" about the floating point problems (I think, the current version says this bit too briefly). Or add pointer to Stan User's guide chapter "14 floating point arithmetic" and more discussion there. The current version has only:

Real variables may be bounded using the same syntax as integers. In theory (that is, with arbitrary-precision arithmetic), the bounds on real values would be exclusive. Unfortunately, finite-precision arithmetic rounding errors will often lead to values on the boundaries, so they are allowed in Stan.
The variable sigma may be declared to be non-negative as follows.
real<lower=0> sigma;

Recently, in combination with ode, we saw negative values for parameter with constraint <lower=0>, and then got error in constraint checking. I didn't know before that this could happen.

The inverse tansform is exp(x) from unconstrained x.  Does that mean exp(x) can be negative?  It should just round to 0 for values below about -400 or so.  I assume we're still using std::exp---anything could happen if we use someone else's approximation.

If Stan lower and upper would define non-inclusive bounds,

Mathematically, the inverse transforms make the continuous bounds non-inclusive.  As you indicate, the problem is that rounding will lead to boundary values.  For example if the unconstrained vlaue is -500, the constrained value will be 0 for lower = 0.
The question I was raising is whether we should just throw an exception when we get non-inclusive bounds.

Maybe we should say in Stan reference manual that e.g. <lower=0.0000001> is fine, if you want have strictly positive values.

Then if the unconstrained value is -500, then the constrianed value is 1e-7 and the gradient is zero, becuase exp(-500) = 0 and exp'(-500) = 0 with double-precision floating point.  This seems like it'll introduce the same problem as that 1e-15 lower bound in  lub_constrain.

I think it means that there should not be premature "limiting values", but instead let numerical issues to show and then require thinking numerically more stable computation, e.g. log_sum_exp.

This sounds like the right approach to me.  That's why I'm wondering if we should just be throwing exceptions when we get catastrophic rounding to 1 (very common given limited dynamic range around 1), underflow to 0 (less likely), or overflow to +/- infinity (again less likely).

I think it would be good to extend the discussion in Stan reference manual
section "Univariate Data Types and Variable Declarations" about the floating point problems

Makes sense.  I created an issue in stan-dev/docs to do just that.

Then if the unconstrained value is -500, then the constrianed value is 1e-7 and the gradient is zero, becuase exp(-500) = 0 and exp'(-500) = 0 with double-precision floating point. This seems like it'll introduce the same problem as that 1e-15 lower bound in lub_constrain.

I momentarily forgot that we need to think gradients, too. It seems that the relevant discussion should mention that the gradient computation can have different floating point accuracy than the constrained value, and the gradient can be zero or fluctuating even if the constrained value is not yet at the boundary value.

That's why I'm wondering if we should just be throwing exceptions when we get catastrophic rounding to 1 (very common given limited dynamic range around 1), underflow to 0 (less likely), or overflow to +/- infinity (again less likely).

I guess there exists counterexamples where rounding to 1 or underflow to 0 is not catastrophic. At least it should not cause sampling to stop completely, but even a reject might be too much?

A mildly tangential comment — the issues with the current lub_constrain
behavior is not so much poorly handling of the overflow but rather that it
makes the output non-monotonic and hence hard to understand.
Boundary behavior is a lot easier to reason about when the floating point
values are at least monotonic

Recently, in combination with ode, we saw negative values for parameter with constraint <lower=0>, and then got error in constraint checking. I didn't know before that this could happen.

This was with a parameter that was being explored by the Markov chain or
the states defined by the ODE?  The latter definitely can run into problems
because of floating point in the ODE solver unless you solve the log states
themselves and then exponentiate afterwards.
If the former then what was the constraint?

For <lower=0>, most of the time the accuracy near 0 is sufficient and users don't see 0's which could cause problems in other parts of the code. Sometimes users write something like <lower=0.0000001>, and this has been openly criticized as bad practice, but it seems that floating point presentation is causing more headache than previously thought. Maybe we should say in Stan reference manual that e.g. <lower=0.0000001> is fine, if you want have strictly positive values.

To be honest I don’t think that there have actually been many headaches
with floating point.  Most of the examples being discussed are somewhat
pathological from a modeling perspective and so the stress on the floating
point is somewhat artificial.  It may be more productive to figure out models
motivated by actual applications that run into problems and then reduce
them to a suite of minimally failing examples.

I think it means that there should not be premature "limiting values", but instead let numerical issues to show and then require thinking numerically more stable computation, e.g. log_sum_exp.
This sounds like the right approach to me. That's why I'm wondering if we should just be throwing exceptions when we get catastrophic rounding to 1 (very common given limited dynamic range around 1), underflow to 0 (less likely), or overflow to +/- infinity (again less likely).
I guess there exists counterexamples where rounding to 1 or underflow to 0 is not catastrophic. At least it should not cause sampling to stop completely, but even a reject might be too much?

It depends on how pedantic we want to be.  If the floating point calculations
are questionable then the validity of the entire calculation can’t be guaranteed
and so if we wanted to be really, really careful we’d terminate instead of just
giving a warning or rejecting the current transition.  In fact rejecting would
change the target distribution a bit so we’d have to be careful to not do that
without providing some context to the user.
In general I think the most robust framework is to have each function responsible
for handling inputs at boundary values and reacting appropriately (especially
with regard to the gradients).  The only thing that we wouldn’t be able to
handle would be catastrophic cancellations, especially in terms that get automatically
added like the constraint Jacobians.  But maybe there the best approach is to
build functions desired for better boundary behavior — for example a beta
density that takes in an unconstrained value and handles the constraints
internally to handle the boundary behavior better.

I momentarily forgot that we need to think gradients, too. It seems that the relevant discussion should mention that the gradient computation can have different floating point accuracy than the constrained value, and the gradient can be zero or fluctuating even if the constrained value is not yet at the boundary value.

Yes!  For example the density functions are somewhat robust to boundary values
but the gradients are not nearly as examined.

On Dec 7, 2019, at 2:32 PM, Michael Betancourt<EMAIL_ADDRESS>wrote:
A mildly tangential comment — the issues with the current lub_constrain
behavior is not so much poorly handling of the overflow but rather that it
makes the output non-monotonic and hence hard to understand.
Boundary behavior is a lot easier to reason about when the floating point
values are at least monotonic

Right---the issue is only addressing the non-monotonicity by removing the conditional.

This sounds like the right approach to me. That's why I'm wondering if we should just be throwing exceptions when we get catastrophic rounding to 1 (very common given limited dynamic range around 1), underflow to 0 (less likely), or overflow to +/- infinity (again less likely).
I guess there exists counterexamples where rounding to 1 or underflow to 0 is not catastrophic. At least it should not cause sampling to stop completely, but even a reject might be too much?

It depends on how pedantic we want to be. If the floating point calculations
are questionable then the validity of the entire calculation can’t be guaranteed
and so if we wanted to be really, really careful we’d terminate instead of just
giving a warning or rejecting the current transition.

After warmup, as issues like this during warmup may be recoverable. Otherwise, it might make sense to treat like a divergence---keep going, but provide a stern warning.

In fact rejecting would
change the target distribution a bit so we’d have to be careful to not do that
without providing some context to the user.
In general I think the most robust framework is to have each function responsible
for handling inputs at boundary values and reacting appropriately (especially
with regard to the gradients).

That was how we were looking at it, but we haven't been consistent. I'm not even sure it's possible to be consistent in this.

That was how we were looking at it, but we haven't been consistent. I'm not even sure it's possible to be consistent in this.

Right — all we can do is maintain consistency within the context of
a given function.  That leaves us unable to handle cancellations between
Jacobians and densities when dealing with constrained parameters.
One possibility is to create the push forward unconstrained densities
explicitly as new functions and maybe let the compiler dispatch to those
instead of just aggregating the Jacobians and densities separately.


Dataset hashes can change with package upgrades

The current way we handle data hashing doesn't survive package upgrades. For example, with pandas, we have been dumping dataframes and the hashes change (even if the data itself doesn't) with upgrades to pandas.

Possible references:

https://stackoverflow.com/questions/31567401/get-the-same-hash-value-for-a-pandas-dataframe-each-time#47800021
http://drorata.github.io/posts/2017/May/26/when-trying-to-hash-a-data-frame/index.html

Example: In https://github.com/acwooding/ReproAllTheThings, we get different hashes with pandas==1.0.5 and pandas 1.3.2 on MacOS.

Another potential culprit: https://github.com/joblib/joblib/pull/1136

The risk, (which is the reason, I assume, it was not done this way already) is that the pickle memoization process will interfere will hashing and create spurious changes in pickle string of dtypes with the final consequence of assigning different hash values for seemingly identical objects

I think there's a really deep issue here, and that's that in order to be truly reproducible here, we need a hash that's more aware of the data, as certain data formats will change version-to-version even through the underlying raw data is identical.


Extract Chat layout configuration and make it open.

Codecov Report

Merging #446 into master will increase coverage by 0.04%.
The diff coverage is 87.5%.

@@            Coverage Diff             @@
##           master     #446      +/-   ##
==========================================
+ Coverage   64.47%   64.51%   +0.04%
==========================================
  Files          74       75       +1
  Lines        3851     3861      +10
==========================================
+ Hits         2483     2491       +8
- Misses       1368     1370       +2

Impacted Files
Coverage Δ

...ource/ChatController/ChatLayoutConfiguration.swift
100% <100%> (ø)

...Source/ChatController/BaseChatViewController.swift
81.77% <75%> (-0.87%)
:arrow_down:

Continue to review full report at Codecov.

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


Missing template.html file

Hi,
I'm trying to build the project on my computer, but it can't find the template.html file on startup and I couldn't find it in the repo files. Am I missing something?

I tried to copy the missing html files from the installation directory to the build output folder and it seems working but throws many script errors as shown below.

Thanks in advance.

It's working normally now after including the jquery scripts file in addition to the missing HTML files.

The best way is to install the latest Calcpad release in c:\Program Files\Calcpad, then select and copy all installed files into your bin/Debug and bin/Release folders in the project. Then, build and run the application. In this way, you will be sure that nothing will be missing.


docs(contributing): Update CONTRIBUTING.md

Pull Request Test Coverage Report for Build 3287
Details

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

Totals

Change from base Build 3286:
0.0%

Covered Lines:
4082

Relevant Lines:
4907

💛  - Coveralls


Allow network localhost durning graph init

When I try to initialize a subgraph, I'm forced to select a public chain from the registry. It's a pretty common use-case to start developing a subgraph against a local hardhat/anvil node.
$ graph init
 ›   Warning: In next major version, this flag will be removed. By default we will
 ›   stop initializing a Git repository.
? Network … localhost
Other · other network not on the list
< 112 more - type to filter >

Selecting other exits the wizard:
graph init
 ›   Warning: In next major version, this flag will be removed. By default we will
 ›   stop initializing a Git repository.
✔ Network · N/A

  The chain list is populated from the Networks Registry:

  https://github.com/graphprotocol/networks-registry

  To add a chain to the registry you can create an issue or submit a PR

It should continue the flow but forcing to specify a path to an ABI file instead of trying to download the ABI from a blockexplorer in my opinion.

Good point, missed that option.
I like having "Other" option that gives you a way out if you can't find the chain you are looking for. From UX perspective.
Perhaps we can add "Local Node" as an option here.

Just checked earlier versions of graph cli and not seeing "localhost" option. How was it handled before?

It wasn't. It's a new feature request.


Subsequent import of the same media files makes them not displayable

Kentico.Xperience.UMT package version - 1.0.1
Reproduction steps:

Import one or more media files into Xperience by Kentico instance
Load media library interface and make sure you can see these media files
Import the same media files again
In the media library interface these files will become grey rectangles, trying to open media file by URL /getmedia returns 404 error

What I've discovered, during the subsequent imports FilePath attribute removes media file extension.
I.e. if after the first import media file FilePath was "Images/Main Images/Homepage/solutions-1-638x380.jpg", then after the second import it changes to "Images/Main Images/Homepage/solutions-1-638x380" which breaks the interface to display it.

Hi @tkrch
Apologies, apparently there was a bug in my export code that led to this behavior. I fixed it now and close this issue. Thanks for your comment it pushed me in the right direction.


Meta node hang DDL processing when connection setup timeout of SASL connection

Describe the bug
The Meta node hangs again which blocked all DDLs. And there are many lines of WARN log of connection timeout of librdkafka:
{"timestamp":"2024-02-24T06:09:17.35828774Z","level":"WARN","fields":{"message":"librdkafka: FAIL [thrd:sasl_ssl://b0-xxx.aws.confluent.cloud:9092/boot]: sasl_ssl://b0-xxx.aws.confluent.cloud:9092/0: Connection setup timed out in state CONNECT (after 30034ms in state CONNECT, 1 identical error(s) suppressed)","log.target":"librdkafka","log.module_path":"madsim_rdkafka::std_::client","log.file":"/root/.cargo/registry/src/index.crates.io-6f17d22bba15001f/madsim-rdkafka-0.3.0+0.34.0/src/std/client.rs","log.line":78},"target":"librdkafka"}

Meta node cannot process DDL commands and it seems due to the connection timeout of librdkafka. (output of show processlist)

Error message/log
"message":"librdkafka: FAIL [thrd:sasl_ssl://b0-xxx.aws.confluent.cloud:9092/boot]: sasl_ssl://b0-xxx.aws.confluent.cloud:9092/0: Connection setup timed out in state CONNECT (after 30032ms in state CONNECT, 1 identical error(s) suppressed

https://grafana.prod.risingwave.cloud/explore?panes={"rF-":{"datasource":"P5EC303186A5DB006","queries":[{"refId":"A","expr":"{app%3D\"risingwave-meta-default-0\", namespace%3D\"rwc-g1hmdvc3u9f88otor7j1kbpin2-thumbtack-prod-poc\"} |~ `(WARN|ERROR)` | json","queryType":"range","datasource":{"type":"loki","uid":"P5EC303186A5DB006"},"editorMode":"builder"}],"range":{"from":"1708750800000","to":"1708756259000"}}}&schemaVersion=1&orgId=1
To Reproduce
No response
Expected behavior
No response
How did you deploy RisingWave?
tenant: https://grafana.prod.risingwave.cloud/d/AdminDashboard_Tenant/tenant?var-datasource=PE662C12516FAE815&var-id=3&orgId=1
The version of RisingWave
PostgreSQL 9.5-RisingWave-1.6.1 (02ee186211e44001c645027bf5aca3db5f076d29)
Additional context
No response

cc @tabVersion @yezizp2012

Caused by https://github.com/confluentinc/librdkafka/pull/4460
@wangrunji0408 please help to update madsim-rdkafka and patch to release branch of v1.6 and v1.7, thanks.

#15313 for main
#15314 for release-1.7
#15315 for release-1.6


Answers'filters box

After trying, I suggest reducing the number of fields to avoid scrolling in the window by replacing the radio buttons with checkboxes for the first two filters.

Should we remove "uniquement"?

Yes, good point!

Still relevent?

It is fixed


Fix tutorials not stored persistently into data file

Tutorials should be edited through use of "mark" command
Hence, removing from EditCommand Class and its tests


websites without protokoll should be redirected to http:// protokol

so something like that www.youtube.com
should go to http://www.youtube.com

Currently not possible because it would think every sindle word search would be a website


[Feature][Config UI/BE] Need to be able to search boards for adding data scope for Jira

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Use case
When selecting boards for Jira, sometime the users have too many boards and it would be convenient for them if they can search boards by name.

Description
No response
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

related to #5640


Replace SED command to update templates with fill-jinja python script

Description

Replace the SED command with a fill-jinja python script used in main_aqm of SRW and RRFS-workflow.

Anticipated changes to regression tests:

[ ] Is baseline change expected ?

Subcomponents involved:

[ ] apply_incr.fd (NOAA-PSL/land-apply_jedi_incr)
[ ] ufs_model.fd (ufs-community/ufs-weather-model)
[x] none

Linked PR's and Issues:

Resolves Issue #156
Testing (for CM's):

RDHPCS

[x] Hera
[ ] Orion
[ ] Hercules

CI

[ ] Completed

WE2E

[x] Completed

PW-Clouds

[ ] AWS
[ ] AZURE
[ ] GCP

@EdwardSnyder-NOAA We prefer to merge this PR to use jinja.


DeltaTextInputClient does not create a TextEditingDeltaInsertion for space key press in web

Steps to Reproduce

Checkout https://github.com/Renzo-Olivares/basic_text_field
Run the application in web or windows
Press space key

Expected results:
updateEditingValueWithDeltas method receives a TextEditingDeltaInsertion with a space as insertedText.
Actual results: Nothing
Tried on both

Channel master, 3.1.0-0.0.pre.727 (chrome)
Channel stable, 3.0.0 (chrome)

Code sample

@override
void updateEditingValueWithDeltas(List<TextEditingDelta> textEditingDeltas) {
  for (final TextEditingDelta delta in textEditingDeltas) {
    print(delta.runtimeType.toString());

    if (delta is TextEditingDeltaInsertion) {
      print('delta: ${delta.textInserted}');
    }
  }
}

Logs

❯ flutter doctor -v
[√] Flutter (Channel master, 3.1.0-0.0.pre.727, on Microsoft Windows [Version 10.0.19044.1706], locale en-BE)
    • Flutter version 3.1.0-0.0.pre.727 at C:\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 36f9a186bf (13 hours ago), 2022-05-15 14:34:06 -0400
    • Engine revision 1b064fd1a3
    • Dart version 2.18.0 (build 2.18.0-116.0.dev)
    • DevTools version 2.13.1

[X] Android toolchain - develop for Android devices
    X Unable to locate Android SDK.
      Install Android Studio from: https://developer.android.com/studio/index.html
      On first launch it will assist you in installing the Android SDK components.
      (or visit https://flutter.dev/docs/get-started/install/windows#android-setup for detailed instructions).
      If the Android SDK has been installed to a custom location, please use
      `flutter config --android-sdk` to update to that location.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop for Windows (Visual Studio Professional 2019 16.11.8)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Professional
    • Visual Studio Professional 2019 version 16.11.32002.261
    • Windows 10 SDK version 10.0.19041.0

[!] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/docs/get-started/install/windows#android-setup for detailed instructions).

[√] VS Code, 64-bit edition (version 1.60.1)
    • VS Code at C:\Program Files\Microsoft VS Code
    • Flutter extension version 3.27.0

[√] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.19044.1706]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 101.0.4951.64
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 100.0.1185.29

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

[√] Flutter (Channel stable, 3.0.0, on Microsoft Windows [Version 10.0.19044.1706], locale en-BE)
    • Flutter version 3.0.0 at C:\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision ee4e09cce0 (6 days ago), 2022-05-09 16:45:18 -0700
    • Engine revision d1b9a6938a
    • Dart version 2.17.0
    • DevTools version 2.12.2

[X] Android toolchain - develop for Android devices
    X Unable to locate Android SDK.
      Install Android Studio from: https://developer.android.com/studio/index.html
      On first launch it will assist you in installing the Android SDK components.
      (or visit https://flutter.dev/docs/get-started/install/windows#android-setup for detailed instructions).
      If the Android SDK has been installed to a custom location, please use
      `flutter config --android-sdk` to update to that location.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop for Windows (Visual Studio Professional 2019 16.11.8)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Professional
    • Visual Studio Professional 2019 version 16.11.32002.261
    • Windows 10 SDK version 10.0.19041.0

[!] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/docs/get-started/install/windows#android-setup for detailed instructions).

[√] VS Code, 64-bit edition (version 1.60.1)
    • VS Code at C:\Program Files\Microsoft VS Code
    • Flutter extension version 3.27.0

[√] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.19044.1706]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 101.0.4951.64
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 100.0.1185.29

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

Hi @maheshmnj I'm also doing a print('delta: ${delta.textInserted}');.
I just tried your code sample and it doesn't work in Chrome but it does work in Windows.

Thanks for the info @Matthiee, with the code sample I shared above I see space empty string getting logged to console
on hitting space on web. See the below screenshot it prints "delta: " 10 times.

flutter doctor -v (mac)
[✓] Flutter (Channel stable, 3.0.0, on macOS 12.3 21E230 darwin-arm, locale en-IN)
    • Flutter version 3.0.0 at /Users/mahesh/Documents/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision ee4e09cce0 (6 days ago), 2022-05-09 16:45:18 -0700
    • Engine revision d1b9a6938a
    • Dart version 2.17.0
    • DevTools version 2.12.2

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

[!] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! CocoaPods 1.10.2 out of date (1.11.0 is recommended).
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin code that responds to your
        plugin usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To upgrade see https://guides.cocoapods.org/using/getting-started.html#installation for instructions.

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

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

[✓] IntelliJ IDEA Community Edition (version 2021.2.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 61.2.4
    • Dart plugin version 212.5080.8

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

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-arm64   • macOS 12.3 21E230 darwin-arm
    • Chrome (web)    • chrome • web-javascript • Google Chrome 101.0.4951.64

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

! Doctor found issues in 1 category.

Hi @maheshmnj thank you for confirming it is working on web in mac.
The issues seems to only reproduce on a Windows device in Web.
Device: Flutter (Channel stable, 3.0.0, on Microsoft Windows [Version 10.0.19044.1706], locale en-BE)
https://user-images.githubusercontent.com/3512339/168779850-227b4bdf-2517-487d-b479-0978b8d55754.mp4

Can reproduce on Flutter (Channel stable, 3.0.0, on Microsoft Windows [Version 10.0.19044.1706], locale en-BE)
Flutter 3.0.0 • channel stable • https://github.com/flutter/flutter.git
Framework • revision ee4e09cce0 (7 days ago) • 2022-05-09 16:45:18 -0700
Engine • revision d1b9a6938a
Tools • Dart 2.17.0 • DevTools 2.12.2

Hello @Matthiee, the main branch of that demo is outdated. Try https://github.com/flutter/samples/tree/master/simplistic_editor for a more up to date demo. The root problem here is that the text field is wrapped with a ListView, and the Shortcuts defined at the top of the tree in WidgetsApp is triggering the space bar shortcut which is set to scroll the ListView, so instead of doing a whitespace we get no delta.

Thank you @Renzo-Olivares! That makes sense.
I can confirm that adding a Shortcuts with space bar shortcut override indeed fixes my problem.


⚠ Bofrost.be has degraded performance

In 0646c8b, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1174 ms

Resolved: Bofrost.be performance has improved in 05def16.


[InternalEvent] CS0117 'NavEventType' does not contain a definition for Internal

Describe the bug
Hi,
Susbscribing to an InternalEvent give this error when publishing:
error CS0117: 'NavEventType' does not contain a definition for 'Internal'
If I only publish my InternalEvent it's ok. But if i'm subscribing to it (from the same module) the error occurs.
Documentation doesn't specify to do anything else:
https://docs.microsoft.com/en-us/dynamics365/business-central/dev-itpro/developer/methods/devenv-internal-attribute
Thanks for your help
To Reproduce
Copy/Paste bellow code
AL Code to reproduce the issue
codeunit 54000 MyCodeunit
{
    var

    [InternalEvent(false)]
    procedure MyPublisher()
    begin
    end;

    [EventSubscriber(ObjectType::Codeunit, Codeunit::MyCodeunit, 'MyPublisher', '', false, false)]
    local procedure MySubcriber()
    begin

    end;
}

Expected behavior
App should publish
Screenshots
The error is located in a csharp file at line 49:
Here is a screenshot of the copy

5. Versions:

AL Language: 4.0.191880

Business Central: Platform 15.0.36985.0 + Application 15.0.36626.37063

Thank you for reporting this issue! It has been fixed in builds newer than 37404 and it is now available through Collaborate. It will be available in CU1 of the OnPrem release.

Ok thanks for the quick answer!


problems with rivets.bundled.min and gulp

Hi,
I'm trying to use this library on a project where the build is done using gulp, but even though I'm using the bundle version I'm getting the following error:
[15:06:05] gulp-notify: [[browserify] Compile Error] Cannot find module 'sightgl
ass' from 'C:\Users\Johnny Carmona\Documents\projects\UIG\dls-framework\src\scri
pts\mck-ui\lib'
Regards,
Johnny

Hi Johnny,
Did you install with bower or npm?
Rivets has a dependency on a library called sightglass.
In the rivets dist directory there are 3 versions. You will probably want to use the bundled version, OR, you need to ensure that rivets is shimmed with sightglass.
Looks like you are using browserify, so I recommend using the NPM version. I have never had problems with that version.
Let me know where you get.

Hi,
I downloaded the bundled version, also tried with bower install and still
had the same issue.
I managed to make it work after changing a few lines in the code.
btw I'm trying to use this so I can have static templates on handle bars
and just use doble side binding, is there a way I can set a class on an
element like this:
{ item.text } or like this { item.text }
Regards,
Johnny
On Wed, Apr 6, 2016 at 10:49 AM, Greg Larrenaga<EMAIL_ADDRESS>wrote:

Hi Johnny,
Did you install with bower or npm?
Rivets has a dependency on a library called sightglass
https://github.com/mikeric/sightglass.
In the rivets dist directory there are 3 versions. You will probably want
to use the bundled version, OR, you need to ensure that rivets is shimmed
with sightglass https://github.com/mikeric/sightglass.
Looks like you are using browserify, so I recommend using the NPM version.
I have never had problems with that version.
Let me know where you get.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/mikeric/rivets/issues/609#issuecomment-206461208

Hi Johnny,
You will def want a custom binder.
This one is a classic. I think everyone has this in their rivets projects.
rv-add-class='item.icon'
// make sure you do this before rivets.bind
rivets.binders['add-class'] = function(el, value) {
    el.classList.add(value);
}

That should do it for you.

Closing, please comment if your issue is not fixed.


Strip out mo/mi elements when rendering MathML if they only contain non-breaking space characters or other whitespace

There's a bug in MathJax's experimental [mml]/mml3 package that can cause equations to fail to render if extra spaces are added to equations via <mo> and <mi> elements containing non-breaking spaces. We don't really want this behaviour anyways, since adding extra spaces just messes up MathJax's rendering, so the simplest option here is to just remove these elements if we find them.
It does look like there's a fix for this, but only in MathJax 4, which is still in beta, so not something we'll be updating quite yet.

Damn, I forgot to update the commit message on merge... Might be a bit of confusion when looking at the commit history now.


Program gets stuck forever if tcp connect error happens

The code is as followings. stream.binance.com is blocked in China. The thing is that there is no timeout by default, and the program just stay here forever trying to reconnect.
use tracing_log::LogTracer;
use tracing_subscriber::fmt;
use tracing_subscriber::fmt::format::FmtSpan;
use tracing::Level;

pub fn setup_logs(log_level: Level) -> std::io::Result<()> {
    LogTracer::init().expect("Cannot setup_logs");
    let subscriber = fmt().with_max_level(log_level).with_thread_names(true).with_span_events(FmtSpan::CLOSE).finish();
    tracing::subscriber::set_global_default(subscriber).expect("Cannot setup_logs");
    Ok(())
}

#[tokio::main]
async fn main() -> anyhow::Result<()> {
    setup_logs(Level::TRACE);
    let x = reqwest::get("https://stream.binance.com").await?.text().await?;
    println!("Result {}", x);
    Ok(())
}

Apr 25 21:54:26.492 TRACE main hyper::client::pool: checkout waiting for idle connection: ("https", stream.binance.com)
Apr 25 21:54:26.492 DEBUG main reqwest::connect: starting new connection: https://stream.binance.com/
Apr 25 21:54:26.492 TRACE main hyper::client::connect::http: Http::connect; scheme=Some("https"), host=Some("stream.binance.com"), port=None
Apr 25 21:54:26.492 DEBUG tokio-runtime-worker hyper::client::connect::dns: resolving host="stream.binance.com"
Apr 25 21:54:26.498 DEBUG                 main hyper::client::connect::http: connecting to <IP_ADDRESS>:443
Apr 25 21:54:26.498 TRACE                 main mio::poll: registering event source with poller: token=Token(1), interests=READABLE | WRITABLE
Apr 25 21:54:26.800 DEBUG                 main hyper::client::connect::http: connecting to [2001::1f0d:5036]:443
Apr 25 21:54:26.800 TRACE                 main hyper::client::connect::http: connect error for [2001::1f0d:5036]:443: ConnectError("tcp connect error", Os { code: 101, kind: Other, message: "Network is unreachable" })

My debugger can only trace to hyper-0.14.5/src/client/client.rs:227.

    async fn send_request(
        &self,
        mut req: Request<B>,
        pool_key: PoolKey,
    ) -> Result<Response<Body>, ClientError<B>> {
 227:       let mut pooled = self.connection_for(pool_key).await?;

Maybe it's because retryably_send_request does not have a retry limit

@qiujiangkun possibly running into the same problem. Did you ever find the root cause and/or find a way to fix it? Was it the actually retry limit?

I tried to use proxies. It's a one time issue


Proof of concept: minify HTML

This addresses GHE/CFGOV/platform/issues/1824 by using the django-htmlmin middleware to minify HTML. This shouldn't be merged without, however, without further testing and discussion. In particular, I notice our own @rosskarchner has an open bug report around aggressive minification.
Additions

adds the django-htmlmin middleware

To test
To test this locally or in debug mode add the following setting:
HTML_MINIFY = True

Added myself as a reviewer so I can remember to test the performance impact of this change by deploying to a dev server and measuring the HTML size! I'll post results here (likely will get to this next week).

I'm going to close this since we don't have plans to merge/implement anytime soon, but in the long-term I think that this is an idea worth further exploration.


Update links (from wiki PR 2020)

From Wiki PR #2020 (https://github.com/ArduPilot/ardupilot_wiki/pull/2020) discussions:

Remove old link
Insert a contact us option
Insert a getting involved option

Bruno, thanks....could you put the additions as a if hasdoc conditional, rather than a direct site link....it will help maintainability....so if the site showing this banner has that doc, it will appear...never gets broken then...thanks

Yes, sure. I inserted the tests and built them with the new file. LGTM now.


Is it theoretically possible to pass ChatGPT credentials to the wrapper without relying on playwright, a planned future feature?

From what I gather, there probably isn't a truly accessible API for chatgpt -- probably unless you partner with them like Microsoft, but I wonder if its just a matter of time before you all figure out how to pass a token or credentials (forgive me my knowledge in the subject is limited) to the wrapper instead of having to rely on GUI-automation tools like playwright.
I apologize for posting a question on the issues section if this is not the appropriate place, not sure where else to ask.  I'm working on a personal side project using this wrapper, and have become more eager to share it as a service.  The only thing keeping me from doing so is the limitation of how you have to log in to ChatGPT at this time.

Somebody please correct me if I'm wrong, but there have been other ChatGPT API projects that were calling the API directly, and these broke after some recent changes by OpenAI to the unofficial ChatGPT API endpoints. Several of those projects simply gave up and archived their code.
One of the reasons that this project still works is that it's using a real browser in the background, and making these API calls (from OpenAI's point of view) from the official ChatGPT website.
I did see another project that supported sending API requests to a proxy server, which I'm guessing ran a browser on its end, and that's probably faster. But it's also probably a lot harder to set up and maintain.
IMO, until/unless there's an official ChatGPT API, the approach that this project uses is the simplest and most stable available, despite being slower.
If an official API is published/supported by OpenAI, then using Playwright would be obviated, we'd need to rip it out and make some adjustments (most importantly converting to whatever their supported auth scheme would be) to keep it viable.

Please excuse me if what I'm mentioning doesn't help here, but I know a project that calls ChatGPT through a given API key - and since I used it just today I don't beleive it broke so far. Here's the link, maybe it helps:
https://github.com/yonashailug/chatgpt-cli

My understanding is that ChatGPT runs GPT 3.5 (whatever that means), and the published OpenAI endpoints use GPT 3.
I'm not 100% sure if that means ChatGPT is more capable, but in my brief tests, it sure looks to be.

That is correct, the public OpenAI callable model from the API (latest one) is the text-davinci-003 which is DIFFERENT from
ChatGPT.(If I'm not mistaken it is missing the RLHF part and the initial prompt that ChatGPT is given) I've read somewhere that they intend to publish ChatGPT as an API as well in the future.

My understanding is that ChatGPT runs GPT 3.5 (whatever that means), and the published OpenAI endpoints use GPT 3.
I'm not 100% sure if that means ChatGPT is more capable, but in my brief tests, it sure looks to be.

That is correct, the public OpenAI callable model from the API (latest one) is the text-davinci-003 which is DIFFERENT from
ChatGPT.(If I'm not mistaken it is missing the RLHF part and the initial prompt that ChatGPT is given) I've read somewhere that they intend to publish ChatGPT as an API as well in the future.

Hello,
thanks for pointing me in the right direction here, I didn't mind the difference between GPT-3 and 3.5 previosly. Now I learned a bit more about it.
Anyway, it looks like now the API is released:
https://techcrunch.com/2023/03/01/openai-launches-an-api-for-chatgpt-plus-dedicated-capacity-for-enterprise-customers/

Hello,
thanks for pointing me in the right direction here, I didn't mind the difference between GPT-3 and 3.5 previosly. Now I learned a bit more about it.
Anyway, it looks like now the API is released:
https://techcrunch.com/2023/03/01/openai-launches-an-api-for-chatgpt-plus-dedicated-capacity-for-enterprise-customers/

Nice find!  Something I think would be a valuable feature to add for this wrapper!

Browser backend is deprecated, no new features specific to the backed will be implemented.


[DOCUMENTATION] Add New YouTube channel

Is your feature request related to a problem? Please describe.
I want to add one YouTube Channel in README.md file
Describe the solution you'd like
Channel Link - Chainlink
Describe alternatives you've considered
null
Additional context
null

PR has been closed so will close this issue


Is there any need for implementing Monorepo's ??

As the code of backend python code and frontend typescript code residing in same repo  I think it is important to do so
here are more docs for what is monorepo/Turborepo?
https://turbo.build/repo/docs

Interesting idea looks like it could be a nice speed up for performance/dev workflow. Can you elaborate more on how you'd approach migrating/setting up our repo for Turbo?
Feel free to provide a rough implementation plan and take into consideration the current AWS build/deploy CI/CD flow.

yeah working on it will release a sample soon

Feel free to open it up again if you will see some progress.


no clean way to use different keys for snipe commands

hi, first of all, evil-snipe is fantastic. thanks.
that said, i am having some trouble making it work with different keys. i use the colemak keyboard layout, and use evil-colemak-basics to use different keys for various evil commands.
recently i decided to also move the functionality for f and t to different keys for improved ergonomics and preserved muscle memory. in short, i want t to jump forwards, and j to jump backwards. with colemak, keys are at a different location, and t and j fit in my hybrid qwerty/colemak keybinding scheme for non-insert states); details here.
this works fine for evil's built-in evil-find-char-* commands. however, i want to use evil-snipe to spice up f and t. at first i thought all that was needed would be binding evil-snipe-f and friends to the appropriate keys. i do not use evil-snipe-override-mode since that does not use the keys i want, but with a few key mappings the result will be the same.
it turns out this works fine for triggering evil-snipe, but repeating a snipe fails. the reason is that the transient keymap hardcodes the key bindings "f" (forward) and "t" (backward) in the evil-snipe-def macro.
i managed to get it working using this code:
(evil-snipe-def 1 inclusive "t" "T")
(evil-snipe-def 1 exclusive "j" "J")
(evil-define-key '(motion normal visual) keymap
  "t" 'evil-snipe-t
  "T" 'evil-snipe-T
  "j" 'evil-snipe-j
  "J" 'evil-snipe-J)

...but i am not very happy about it:

it uses a private evil-snipe macro
it replaces the standard evil-snipe-t command

perhaps there is a better way to get this working? perhaps (this-command-keys) can be used instead of hardcoding the keys?
hopefully i made the issue clear. ideas and discussion more than welcome. :tulip:

Hey there! You're right, this isn't ideal and I've certainly neglected non-QWERTY layouts. I'll look into this (and this-command-keys) this week and see if I can cook something up that's more flexible but still backward compatible.

It's been almost a year. I've looked into this a number of times and couldn't come up with anything clever, so I think I'll go with the simple solution: a evil-snipe-style switch variable that defaults to 'qwerty and accepts 'colemak.
That said, I'd like to include a reasonable default for colemak users, but I'm not familiar with the layout, so I'm hoping you can offer some advice: are t, T, j and J good colemak alternatives, or are they simply your personal preference? Could you also offer alternatives for s and S as well?

`vim layouts on colemak are plenty, my own evil-colemak-basics obviously the best among them :wink:.
using evil-snipe-def is the most flexible way i would say. this is what evil-colemak-basics already does:
        (evil-snipe-def 1 inclusive "t" "T")
        (evil-snipe-def 1 exclusive "j" "J")

(https://github.com/wbolster/evil-colemak-basics/blob/master/evil-colemak-basics.el#L111-L114)
no opinion on s and S since i have those disabled (i use avy / evil-easy-motion).

using evil-snipe-def overwrites the old commands/functions though, perhaps the macro could take an argument so that it can create unique names, but i do not have a strong opinion on that since i only use colemak and never switch between layouts (my brain would melt if i were to try).

Hi, dvorak user here.  I've been getting into evil-snipe more and the ; is tuff to hit for me many times a day.
I'll post back if i come up with something better for myself.

@joefromct have you tried my evil-snipe-def approach? it works fine actually. see above or consult my dotfiles

I actually realized i had a different ailment regarding my evil-snipe-needs as i dug into it... i think this does the trick for me but curious if this would be the correct way.. i haven't tinkered with emacs lisp too much.
Also, I know realize that my issue was different than yours?  Let me know if thats the case and i'll make myself a new issue, and apologies for hi-jacking the thread.


Feature: add other fuzzing platforms to the Fuzzing check

Our current check only looks for OSS-Fuzz integration. We want to expand our list of supported platforms.
Which fuzzing platforms to support remains an open question. Suggestions welcome!

Another one to add would be clusterfuzzlite which is launching soon: https://github.com/google/clusterfuzzlite.

On a different note should we add this check to scorecard?

Thanks Oliver. To detect clusterfuzzlite, we need to check for a workflow which uses google/clusterfuzzlite/actions/run_fuzzers action, is this correct? Do you expect users to enable it on PRs or push to main or something else?

Yep! Just check for the google/clusterfuzzlite/actions/run_fuzzers action. I think we can be flexible about where it's enabled (PR or push or otherwise) so we don't need to check that.

What about results of this check from this Fuzz?
On a different note should we add this check to scorecard?

Could you clarify what you mean here? Are you suggesting a more detailed check beyond just checking if Fuzzing is enabled?

Thanks Oliver. To detect clusterfuzzlite, we need to check for a workflow which uses google/clusterfuzzlite/actions/run_fuzzers action, is this correct? Do you expect users to enable it on PRs or push to main or something else?

Yep! Just check for the google/clusterfuzzlite/actions/run_fuzzers action. I think we can be flexible about where it's enabled (PR or push or otherwise) so we don't need to check that.

What about results of this check from this Fuzz?
On a different note should we add this check to scorecard?

@naveensrinivasan Could you clarify what you mean here? Are you suggesting a more detailed check beyond just checking if Fuzzing is enabled?

Yes , how do we report the results of these fuzz runs?

Thanks Oliver. To detect clusterfuzzlite, we need to check for a workflow which uses google/clusterfuzzlite/actions/run_fuzzers action, is this correct? Do you expect users to enable it on PRs or push to main or something else?

Yep! Just check for the google/clusterfuzzlite/actions/run_fuzzers action. I think we can be flexible about where it's enabled (PR or push or otherwise) so we don't need to check that.

great! Do you have cycles to add it before you officially release clusterfuzzlite? Or shall we add this to v4 milestone (~EOY)?

@naveensrinivasan is there a special command/keyword we can use to say "this issue should not be closed automatically by the bot"?

I wanted to also not that go1.18 has fuzzing out of the box. didn't know if it should be included in this issue or a different one

I think it fits in this issue. If you know how to check for its use (there's a command for it?) please let us know. Feel free to send. aPR if you have time for it. Thanks!

I think oss-fuzz integrates with the native fizzing, but not sure how exactly.
Worth checking unrelated to my investigation

We need to look for the --fuzz command in the workflow https://go.dev/doc/fuzz/. The assumption is that the command is used directly in the workflow, and not in a script that's invoked via the workflow. We can start simple and then iterate, though

@georgettica @laurentsimon could we start with checking for *_test.go files in the repository which contain functions matching the regex func Fuzz* (* \*testing.F) ? Would that be a good start for adding this support?

I think go test -fuzz takes any string and it's convention to use the prefix Fuzz
Not that I am opposed to the ideas, but letting you know it's based on convention rather than validation

https://go.dev/doc/fuzz/ mentions this as a requirement. But maybe this is not a hard requirement?

yeah, same (verified aswell)
expected the og test -fuzz=Bar to at least. warn me

I think it's fair to start with the regex @azeemshaikh38 provided. The warning is something Go team should fix, rather than us, no? Note that a while back, we were contemplating having a Unit-Test checks (or part of an existing check) so the work you're doing could be useful in this context too.

Yup, it's a go team thing. Noting it here as I didn't start the conversation there

If we are extending the Fuzzing check to include looking for fuzzer definitions for Go (which I think is a good idea!), we should also extend this to other languages, which all have a consistent way to do this that we can detect.
e.g. for C/C++, we can look for instances of LLVMFuzzerTestOneInput in c/cc/cxx/cpp files.

@oliverchang - can someone from fuzzing team pick this, it will be nice to have this scorecard check comprehensive.

@navidem -- this (https://github.com/ossf/scorecard/issues/897#issuecomment-1089849918) might be a interesting side project to tackle!

@Navidem feel free to ask questions if you need some pointers

Fuzzing patterns to add:

C/C++: https://llvm.org/docs/LibFuzzer.html#fuzz-target
Other languages: See the language subguides on the canonical ways to write fuzzers in other languages on https://google.github.io/oss-fuzz/getting-started/new-project-guide/

/cc @aidenwang9867

Thanks @oliverchang !
/cc @aidenwang9867

Thanks Laurent & Oliver, I'll take the Go side. @laurentsimon @oliverchang

@oliverchang @Navidem
In the merged PR#1979, I added support for checking Go built-in fuzzers as a part of scorecard's fuzzing check. Most of the important changes have been made in checks/raw/fuzzing.go, please refer to the changes in this PR to see them.
Built-in fuzzer patterns for other programming languages can be added here. I'll work on another feature for a while but I might implement some fuzzing supports for other languages if I have time. I'll ping you here if I start to do that. :)
cc @laurentsimon

what about One-Fuzz? Any chance to support that by Scorecard?

Absolutely. What does the integration in a repository look like?
For example, this is how we check for clusterfuzzlite: https://github.com/ossf/scorecard/blob/69eb1ccf1d0cf8c5b291044479f18672bf250325/checks/raw/fuzzing.go#L120
If you tell us what we should be looking for in thee repo, we can add a function to check for it in the file linked above.

@laurentsimon, sorry for the delay, how about something like that: https://github.com/ossf/scorecard/pull/2141 :)

Added support for C, CPP, Python, Java and Rust fuzzing in https://github.com/ossf/scorecard/pull/3473
Also opened up a discussion for splitting fuzzing checks in two categories: (1) a check for the presence of source code for a fuzzer and (2) a check for whether a project is continuously fuzzed. Issue is here: https://github.com/ossf/scorecard/issues/3475


Reverse direction CD?

As a HA developer, it would be neat for script/config/etc changes made through the remote HA dashboard to be reflected in my repository because then remote changes (new scripts, dashboard layouts) could be easily stored and verified on other installations.
The CD pipeline of repo -> instance seems pretty straightforward, but what about the other way around? If I make a change to a script in HA in the cloud, I want it to pull down into my repo. Configuration files seem to be much simpler, but what about dashboard layouts/etc that (I think) are stored in the docker image?
Not critical by any means, but an interesting topic to research.
Post #111

Thought about it, and this just doesn't make sense.


chore(package): update config-expander to version 10.0.2

:tada: This PR is included in version 5.0.50 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Pass mypy and link issues

Ref: https://github.com/jaraco/skeleton/issues/143
Upstream issues:

https://github.com/lxml/lxml-stubs/issues/30

Python 3.14 install failing because prebuilt lxml wheels are (unsurprisingly) not available and manual build fails due to missing system dependencies.
I think that can be ignored (or excluding 3.14 in a different PR)


Design client library

Identify what functionality from existing updater/download modules should exist in a TUF client on top of a metadata model as sketched in #1112.
Start with API functionality.
Coordinate with #1134.

allowing a package manager to plug-in their own network/download stack

Filed 1142 for this

supporting metadata and targets on separate hosts, and different targets on different hosts

and  #1143 for this

See #1158 for "client/updater: Parallel Downloads"

We're seeing multiple areas where the current Updater interface/API is brittle or confusing (#1143, #1174, possibly others).
I had originally expected that we would try to maintain the current updater API, but we should review that and even if we do stay close to the current API plan to fix the confusing/outdated/inflexible aspects.

I had originally expected that we would try to maintain the current updater API, but we should review that and even if we do stay close to the current API plan to fix the confusing/outdated/inflexible aspects.

Same same.
Now that we've kicked out mirrors (so are not directly compatible anyway) and now that I've had time to think about it, I think we should improve the API as much as we can. It'll still be quite close to the old one (so not hard to port) but hopefully an improvement.
Latest ideas in https://github.com/theupdateframework/tuf/issues/1317#issuecomment-843872644

IMO the final missing piece(s) here might be documented in #1580.

I'm sure there are features that are still wanted by someone, but I think the client design is now done and implemented: anything still missing is a new feature, closing this one


file startup significantly slower in RCs post 0.19.0

The Issue
Post 0.19.0 release candidates result in an extremely slow file startup process. A PC with 500 files went from 7 seconds to 7 minutes.
System Configuration

LBRY Daemon version: 0.19.1RCx
LBRY App version:
LBRY Installation ID:
Operating system:

Anything Else

Screenshots

fixed


npm run docs tasks prints a lot of output tothe terminal

The npm run docs prints a lot of output to the terminal, so much so that it overflows the default history buffer.
It looks like it is printing it's full JSON output to stdout. This should not be done, as it severly hinders debugging and reading of error messages

Since that scripts seems to be the one run, we should make it respect the following UNIX convention:

print message that are human-readable and for the user to STDERR
print a computer-parsable output, in structured format (e.g. JSON) to STDOUT
If a problem occurs, something fails or is abnormal etc..., exit code must be non-zero

This should no longer be happening.


Show empty tab when tabs are overflowing

When tabs are overflowing the empty tab is not shown. It should always be shown.

Hint: Due to the way scroll-tabs works it's difficult to make this work.

Reproduced it

This is not a bug but a possible enhancement. In the current version of the Camunda Modeler the empty tab is not visible whenever the tabs are overflowing.

What is the benefit of having the + button if you already got 40 tabs open? If you are a heavy user of the application (you opened many tabs already) you are probably aware of how to open / create diagrams without using the + button.
I'd be hapy to close this issue as won't fix.

This was listed as a requirement in the original scroll and drag ticket.

From: Nico Rehwaldt<EMAIL_ADDRESS>Sent: Monday, November 19, 2018 4:10:06 PM
To: camunda/camunda-modeler
Cc: Philipp Fromme; Author
Subject: Re: [camunda/camunda-modeler] Show empty tab when tabs are overflowing (#968)
What is the benefit of having the + button if you already got 40 tabs open? If you are a heavy user of the application (you opened many tabs already) you are probably aware of how to open / create diagrams without using the + button.
I'd be hapy to close this issue as won't fix.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/camunda/camunda-modeler/issues/968#issuecomment-439925271, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AHR6pNIrXTPWH6AGY5v2aiWOYr4lNWVLks5uwsnNgaJpZM4YDxRT.

Let us relax the original requirement(s) and close this issue then.

Relaxing sounds good! ??

From: Nico Rehwaldt<EMAIL_ADDRESS>Sent: Monday, November 19, 2018 4:14:34 PM
To: camunda/camunda-modeler
Cc: Philipp Fromme; Author
Subject: Re: [camunda/camunda-modeler] Show empty tab when tabs are overflowing (#968)
Let us relax the original requirement(s) and close this issue then.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/camunda/camunda-modeler/issues/968#issuecomment-439926662, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AHR6pJ9zMG_wV6ijKu8ldJhgDSehZjUoks5uwsrBgaJpZM4YDxRT.


Remove non existent option from README.md

Remove the readContent option as is not used anymore

hello, it is used in windows so I'd keep it

It'd make sense to add a label there so it's clear it's windows only, maybe you can do that :)


Provide more detail about how Rancher's role promotion logic is

https://rancher.com/docs/rancher/v2.x/en/admin-settings/rbac/default-custom-roles/
@catherineluse you will need to speak to @cjellick to get details

Duplicate of #366


Soil Dataset

Since it is not specified in Assignment RDD, one shouldn't use the soil metrics provided in the dataset of this Github, right ?
(soil texture, depth, carbon)

Hey, the assignment does not specifically state this dataset and hence you can choose what you would like to work with.


Tensor-LLM for BLIP-2 (ViT and Q-Former)

I notice in here that the ViT and Q-Former part do not leverage any new Tensor-LLM plugins (e.g., use_gemm_plugin and use_gpt_attention_plugin). Does it mean that the new Tensor-LLM plugins cannot be used to further speedup ViT further compared with Tensor-RT plugins?

Hi @HaisongDing , ViT and Q-Former performance with native TensorRT should be decent already. TensorRT itself has many optimizations already, for example, BERT, native TensorRT performance is better than --use_bert_plugin in most cases.
In BLIP2, although ViT and Q-Former are on native TensorRT, the LLM part is on TRT-LLM.

Got it, thanks.


FIX KIT reader in Maint/0.13

Maintenance release to fix KIT reader for old files (#3721)
Thanks @agramfort, should I update the version number? Add a note on 0.13.1 to whats_new?

Current coverage is 87.36% (diff: 71.42%)

Merging #3777 into maint/0.13 will decrease coverage by <.01%

@@           maint/0.13      #3777   diff @@
============================================
  Files             343        343
  Lines           60843      60827    -16
  Methods             0          0
  Messages            0          0
  Branches         9317       9318     +1
============================================
- Hits            53161      53142    -19
- Misses           4923       4925     +2
- Partials         2759       2760     +1

Powered by Codecov. Last update cb8707f...25cdf94

thx @christianbrodbeck
shall we do a maint release now?

From my part yes, this was the issue that I want to fix.
On Saturday, November 19, 2016, Alexandre Gramfort<EMAIL_ADDRESS>wrote:

thx @christianbrodbeck https://github.com/christianbrodbeck
shall we do a maint release now?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mne-tools/mne-python/pull/3777#issuecomment-261700647,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAI5a3f6HF4GEOXe4vzQjhYeNNV1TiOqks5q_q_IgaJpZM4K2sun
.

I just pushed a 0.13.1 version on pypi
https://pypi.python.org/pypi/mne/0.13.1

Great, thanks!!


istioctl: support parsing out v1 CRDs

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-ambient_istio
1507dd8de8a26e95f6366fa995316bd7cf1f2f1a
link
true
/test integ-ambient

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

/retest


Readme edits

Hello word

Why:

What's being changed:

Check off the following:

[ ] I have reviewed my changes in staging. (look for the deploy-to-heroku link in your pull request, then click View deployment)
[ ] For content changes, I have reviewed the localization checklist
[ ] For content changes, I have reviewed the Content style guide for GitHub Docs.

Readme edits

Readme edits


Upgrade urllib3 to 490d3a227fadb626cd54a240b9d0922f849914b4

This is the release tagged at 85dfc16817df1e3604c238ad5d64f3b229e0598b

Tested on OS X 10.10:
[tw-mbp-jsmith requests (urllib_upgrade)]$ make test
# This runs all of the tests. To run an individual test, run py.test with
# the -k flag, like "py.test -k test_path_is_not_double_encoded"
py.test test_requests.py
============================================================================== test session starts ===============================================================================
platform darwin -- Python 2.7.9 -- pytest-2.3.4
plugins: cov
collected 154 items

test_requests.py ..........................................................................................................................................................

========================================================================== 154 passed in 26.34 seconds ===========================================================================

This is to get https://github.com/shazow/urllib3/pull/526 merged in https://github.com/pypa/pip/issues/2395 can take advantage of the fix for older SSL library versions. This was previously tracked at https://github.com/kennethreitz/requests/issues/2435.

We actually will pull in tip when we do a release. Speaking of which @Lukasa when do you think we should do a release? There's extra header awesomeness that's unreleased in urllib3 (merged last night) that I'd like to grab before doing a release.

@sigmavirus24 sounds good- I updated to HEAD, but feel free to drop this PR in favor of another one if this is in flight elsewhere.

It'd be great to get a bug-fix release soon (for issue #2435), so that we can use requests on python 2.7.9...

Howdy all, anything I can do to help get this over the finish line? Happy to pitch in on testing or updating docs for 'upgrading urllib3' if I missed something too.
Thanks!!

This is a pretty big blocker for me right now. Any chance we can get this pull request merged soon?

Thanks @sigmavirus24 !


Track down forwarded services

Hello.
Can you add some mechanism to track down forwarded services in kubefwd? Event dispatch or something.
My team could access the port-forwarded services, but now it's impossible due to unexported variables. We use your project as a go module.
Or can i make a PR to implement such mechanism?

This request is due to the following - some environments have a kubelet parameter --streaming-connection-idle-timeout duration equal to a small value. My team suffers from the fact that on some environments, connections are dropped.
At the moment we are using our extension, in which we reopen connections after dropping, but this behavior is possible only in version 1.13 and below.

@flupec interesting, I did not know about --streaming-connection-idle-timeout, this could account for a few complaints about constant connection dropping. I don't have that arg set on any of my kubelets.
There have been attempts to add re-connection code but it has always caused more problems than it solved. It's very difficult to determine if a connection goes down without actively polling it, and if the loss is temporary then an attempt to automatically re-connect may actually be worse.
Re: "due to unexported variables", while kubefwd was never meant to be used as a module, I am not at all against it. Kubefwd is very stable in the way my team and I use it. I am open to changes that don't disrupt that. What would you like exposed?
This project is also very immature in terms of tests and proper documentation. It was originally a weekend project to solve one specific development problems.

@cjimti I made a PR. This approach works as expected But i wonder if we need to add an option to change stream connection period creation.
PR


Eloquent virtual attribute is wrongly added to the update query. [L4]

I have an Eloquent / Ardent model with an 'opt-in' virtual attribute (a form checkbox that must be validated but not saved).
class Foobar extends \LaravelBook\Ardent\Ardent {
	protected $fillable = [..., 'optin'];
	protected $appends = ['optin'];
	public static $rules = ['optin'  => 'required'];
}

When the object is updated in the controller, MySQL complains that optin is not a column:
SQLSTATE[42S22]: Column not found: 1054 Unknown column 'optin' in 'field list' (SQL: update `foobar` set `updated_at` = 2015-01-16 16:03:46, `optin` = 1 where `id` = 14)

The virtual attribute, when declared in the $appends array, shouldn't be added to the query.

Does the same issue occur when you extend Eloquent instead of Ardent? Could be an issue on their end.

@otagi You got it wrong. $appends is not for virtual attributes literally. It's for appending mutated vaues to the toArray/toJson output of the model:
public function getOptinAttribute() { ... }

So it has nothing to do with not adding to the query, and it's on your head to handle it, with current Model setup.

You're both right. The issue is with Ardent rather than Eloquent. I should have tested it more thoroughly before posting. Sorry about the noise.
PS: In case someone reading this has the same problem: I fixed it by adding a 'purgeFilter' in the model's constructor.


Feat/51

This has the code required to import configlets into a blueprint.
Two methods for importing exist.
One uses the ID of a catalog configlet as a model. The other allows a user to build their own configlet for their blueprint.

Please change the PR title to something informative. It's going to wind up in the end-user-facing release notes.

Please change the PR title to something informative. It's going to wind up in the end-user-facing release notes.

Done