knex with aws xray

aws xray sdk monkey patches the postgres module to capture postgress calls and capture metrics.
http://docs.aws.amazon.com/xray-sdk-for-nodejs/latest/reference/
I am trying to pass the aws sdk's monkey-patched postgres connection to knex using the .connection(dbConnection) feature as mentioned in the documentation. But it is not working.
Will there be support for passing in your own db connection? OR will there be support for xray?

Simply capturing Postgres within the context where you setup your knex object seems to work. See my code below and the X-Ray screen showing a few queries that are run in a Lambda function.
const knex = require("knex");
const AWSXRay = require("aws-xray-sdk");
const pg = AWSXRay.capturePostgres(require("pg"));

module.exports = () => {
  const connection = {
    host: process.env.PG_HOST,
    user: process.env.PG_USER,
    password: process.env.PG_PASSWORD,
    database: process.env.PG_DB
  };

  const db = knex({
    client: "pg",
    connection,
    pool: { min: 1, max: 1 }
  });

  return db;
};

@RickDT I'm following your example to get knex working with x-ray, but running into:
Unhandled rejection Error: Failed to get the current sub/segment from the context.
    at Object.contextMissing (/home/project/node_modules/aws-xray-sdk-core/lib/context_utils.js:34:13)
    at Object.getSegment (/home/project/node_modules/aws-xray-sdk-core/lib/context_utils.js:96:45)
    at Object.resolveSegment (/home/project/node_modules/aws-xray-sdk-core/lib/context_utils.js:77:19)
    at Client.captureQuery [as query] (/home/project/node_modules/aws-xray-sdk-postgres/lib/postgres_p.js:31:26)
    at /home/project/node_modules/knex/lib/dialects/postgres/index.js:180:18
    at Client_PG.checkVersion (/home/project/node_modules/knex/lib/dialects/postgres/index.js:179:12)
    at /home/project/node_modules/knex/lib/dialects/postgres/index.js:156:25
    at Connection.<anonymous> (/home/project/node_modules/pg/lib/client.js:159:7)
    at Connection.g (events.js:291:16)
    at emitOne (events.js:101:20)
    at Connection.emit (events.js:188:7)
    at Socket.<anonymous> (/home/project/node_modules/pg/lib/connection.js:136:12)
    at emitOne (events.js:96:13)
    at Socket.emit (events.js:188:7)
    at readableAddChunk (_stream_readable.js:176:18)
    at Socket.Readable.push (_stream_readable.js:134:10)
    at TCP.onread [as _originalOnread] (net.js:551:20)
    at TCP.onread (/home/project/node_modules/async-listener/glue.js:188:31)

Any thoughts? Also, in your example, doesn't look like you're passing the overridden const pg = AWSXRay.capturePostgres(require("pg")) into knex at all. Isn't that necessary? Thanks!

@justingshreve based on your error, I'm guessing you get that when you run locally? If so, you'll need to set an ENV var on your local machine: AWS_XRAY_CONTEXT_MISSING=LOG_ERROR. Obviously, you don't want to set that when running in Lambda.
Passing the overridden pg into knex is not necessary in my experience. I believe Knex pulls from the local context. (I may be wrong, but I am getting pg traces in Xray as seen in my screenshot).

Yes, AWS_XRAY_CONTEXT_MISSING=LOG_ERROR fixed the error running locally. I wanted to at least see tests run smooth before deploying. Thanks!

I'm still having trouble setting this up. Btw, I'm running this in an EC2 instance, not locally.
Unhandled rejection Error: Failed to get the current sub/segment from the context.     at Object.contextMissingRuntimeError [as contextMissing] (/home/ec2-user/outpatient/op-server-xray/node_modules/aws-xray-sdk-core/lib/context_utils.js:21:15)     at Object.getSegment (/home/ec2-user/outpatient/op-server-xray/node_modules/aws-xray-sdk-core/lib/context_utils.js:92:45)     at Object.resolveSegment (/home/ec2-user/outpatient/op-server-xray/node_modules/aws-xray-sdk-core/lib/context_utils.js:73:19)     at Client.captureQuery [as query] (/home/ec2-user/outpatient/op-server-xray/node_modules/aws-xray-sdk-postgres/lib/postgres_p.js:31:26)     at /home/ec2-user/outpatient/op-server-xray/node_modules/knex/lib/dialects/postgres/index.js:126:18     at Promise._execute (/home/ec2-user/outpatient/op-server-xray/node_modules/bluebird/js/release/debuggability.js:300:9)     at Promise._resolveFromExecutor (/home/ec2-user/outpatient/op-server-xray/node_modules/bluebird/js/release/promise.js:483:18)     at new Promise (/home/ec2-user/outpatient/op-server-xray/node_modules/bluebird/js/release/promise.js:79:10)     at Client.checkVersion (/home/ec2-user/outpatient/op-server-xray/node_modules/knex/lib/dialects/postgres/index.js:125:12)     at /home/ec2-user/outpatient/op-server-xray/node_modules/knex/lib/dialects/postgres/index.js:103:25     at Connection.<anonymous> (/home/ec2-user/outpatient/op-server-xray/node_modules/pg/lib/client.js:158:7)     at Connection.g (events.js:291:16)     at emitOne (events.js:101:20)     at Connection.emit (events.js:188:7)     at Socket.<anonymous> (/home/ec2-user/outpatient/op-server-xray/node_modules/pg/lib/connection.js:121:12)     at emitOne (events.js:96:13)
connection.js
`const environment = process.env.NODE_ENV;
const config = require('./../../../knexfile.js')[environment];
const knex = require('knex')(config);
const AWSXRay = require('aws-xray-sdk');
let pg;
if (process.env.NODE_ENV !== 'development' && process.env.NODE_ENV !== 'test') {
pg = AWSXRay.capturePostgres(require("pg"));
}
module.exports = knex;
`
For this to work is it required to open the segment somewhere else before this gets executed? I'm running out of ideas.

I found that patching the mysql client with the X-Ray SDK led to major issues when using it with knex transactions. So I've started writing a custom adapter. I just hacked it together last night as a prototype, but feel free to check it out. Seemed to work so far for me and since I need it for a current project, I'll very likely get it into proper shape soon:
https://github.com/marianzange/knex-aws-xray

I'm also having some issues getting @RickDT's suggestion working. My connection looks like this:
import AWSXRay from 'aws-xray-sdk'
import knex from 'knex'
//...
const pg = AWSXRay.capturePostgres(require('pg'))

const db = knex({
  connection: DB_ENDPOINT,
  dialect: 'pg'
})
//...

But my requests are all timing out, and the traces show as pending:

My Lambda is limited to 3 seconds, but increasing to 30 yields the same results.

I am seeing the same error: Cannot read property 'name' of undefined. Looking at the XRay code and the Postgres client code it would seem that the XRay code is meant to work with a very old version of the Postgres client. It assumes that the query method returns the created Query object, where as the pg client that supports Promises will either return a straight up Promise, or nothing at all.

hey guys! any update on this? I am facing the same problem.  I see this reply from AWS
https://forums.aws.amazon.com/thread.jspa?threadID=247556&tstart=0

Hey Gang, FYI I've opened a PR against @marianzange 's solution that adds postgres support..
https://github.com/marianzange/knex-aws-xray/pull/2 we've been using a modified version that only handles postgres and it's been working great!

Any update? the solution from marianzange/knex-aws-xray not work for me.

I don't think here is nothing that knex should do about this, maybe people working with marianzange/knex-aws-xray can help how to get it work. Closing.

@elhigu Well, I am trying to pass the aws sdk's monkey-patched postgres connection to knex using the .connection(dbConnection) feature as mentioned in the documentation. But it is not working. sounds like a potentially legitimate knex bug.

Actually IIRC that .connection is not implemented at all nor it has been properly specified what it should even do... https://knexjs.org/#Builder-connection
So feature request for that would be the way to go and good specification what it should really do... for example if it should take same kind of object that acquireConnection() returns or what.

https://github.com/tgriesser/knex/issues/2804 -> will improve on it later


InputNumber list re-render on splice

I have a list of values I'm using InputNumber to display, along with an up/down arrow next to each to move them up and down in the list using a splice operation. With a normal field the values move up and down but with InputNumber the fields don't refresh when the v-for list is mutated with splice.

Did you add key to the items in v-for?

No I didn't, thanks!


OCPCLOUD-2703: OWNERS: update subcomponent

Update subcomponent, our way of organizing them changed.

/lgtm

/approve


dependency scalatestjs 2.3.0-SNAP2 not found

starting from a clean install ( and after removing the Eclipse plugins ) if I try to run package I
get the errors below. I tried compiling scalatest-2.3x but that produced differents jars such as .ivy2/local/org.scalatest/scalatest_2.11/3.0.0-M1/bundles/scalatest_2.11.jar - eg no scalatestjs but just scalatest. Using those did not help.
> clean
[success] Total time: 0 s, completed 16-Jun-2015 18:44:21
> package
[info] Updating {file:/Volumes/Dev/Programming/ScalaJS/akka.js/}root...
[info] Updating {file:/Volumes/Dev/Programming/ScalaJS/akka.js/}akkaActorJS...
[info] Resolving org.scalatest#scalatestjs_sjs0.6_2.11;2.3.0-SNAP2 ...
[warn] 	module not found: org.scalatest#scalatestjs_sjs0.6_2.11;2.3.0-SNAP2
[warn] ==== local: tried
[warn]   /Users/hjs/.ivy2/local/org.scalatest/scalatestjs_sjs0.6_2.11/2.3.0-SNAP2/ivys/ivy.xml
[warn] ==== public: tried
[warn]   https://repo1.maven.org/maven2/org/scalatest/scalatestjs_sjs0.6_2.11/2.3.0-SNAP2/scalatestjs_sjs0.6_2.11-2.3.0-SNAP2.pom
[warn] ==== Typesafe repository: tried
[warn]   http://repo.typesafe.com/typesafe/releases/org/scalatest/scalatestjs_sjs0.6_2.11/2.3.0-SNAP2/scalatestjs_sjs0.6_2.11-2.3.0-SNAP2.pom
[warn] ==== sonatype-snapshots: tried
[warn]   https://oss.sonatype.org/content/repositories/snapshots/org/scalatest/scalatestjs_sjs0.6_2.11/2.3.0-SNAP2/scalatestjs_sjs0.6_2.11-2.3.0-SNAP2.pom
[info] Resolving org.eclipse.jetty#jetty-continuation;8.1.16.v20140903 ...
[warn] 	::::::::::::::::::::::::::::::::::::::::::::::
[warn] 	::          UNRESOLVED DEPENDENCIES         ::
[warn] 	::::::::::::::::::::::::::::::::::::::::::::::
[warn] 	:: org.scalatest#scalatestjs_sjs0.6_2.11;2.3.0-SNAP2: not found

yeah, I was using a custom built version locally, now that 3.0.0-SNAP5 is available, we can bump the version.
I'm writing my thesis ATM so I won't be able to take care of this before Monday, feel free to bump the version locally in your build.sbt tho :)

Will do that tomorrow if anybody will remind me ;-)
On 16 Jun 2015 21:20, "ヨーント"<EMAIL_ADDRESS>wrote:

yeah, I was using a custom built version locally, now that 3.0.0-SNAP5 is
available, we can bump the version.
I'm writing my thesis ATM so I won't be able to take care of this before
Monday, feel free to bump the version locally in your build.sbt tho :)
—
Reply to this email directly or view it on GitHub
https://github.com/unicredit/akka.js/issues/19#issuecomment-112534374.

FYI, The latest version is 3.0.0-M1

@ngbinh I found 3.0.0-SNAP5 on sonatype
On the other hand trying to move to 3.0.0-SNAP5 which does download and allows me to open sbt I get 141 errors starting with the following errors
[error] /Volumes/Dev/Programming/ScalaJS/akka.js/akka-js-actor/js/../../akka-js-testkit/js/src/test/scala/akka/testkit/AkkaSpec.scala:8: object scalatest is not a member of package org
[error] import org.scalatest.{ WordSpecLike, BeforeAndAfterAll }
[error]            ^
[error] /Volumes/Dev/Programming/ScalaJS/akka.js/akka-js-actor/js/../../akka-js-testkit/js/src/test/scala/akka/testkit/AkkaSpec.scala:9: object scalatest is not a member of package org
[error] import org.scalatest.Matchers
[error]            ^
[error] /Volumes/Dev/Programming/ScalaJS/akka.js/akka-js-actor/js/../../akka-js-testkit/js/src/test/scala/akka/testkit/AkkaSpec.scala:66: not found: type WordSpecLike
[error]   extends TestKit(_system) with WordSpecLike with Matchers with BeforeAndAfterAll { // @note IMPLEMENT IN SCALA.JS with WatchedByCoroner {

I am a bit surprised because when I open the project in IntelliJ those classes are not missing.

object scalatest is not a member of package org
this means you haven't actually downloaded the org.scalatest package.. have you tried cleaning and re-downloading?
otherwise check in your ~/.ivy2 if the correct package is there
┌[yawnt@xantico] [/dev/pts/0]
└[~]> ls .ivy2/local/org.scalatest/
....

it should show folders with an sjs postfix

First I remove everything in .ivy2
$ rm -rf ~/.ivy2/cache/org.scalatest/
$ rm -rf ~/.ivy2/local/org.w3/

$ sbt
Use of ~/.sbtconfig is deprecated, please migrate global settings to /usr/local/etc/sbtopts
Java HotSpot(TM) 64-Bit Server VM warning: ignoring option MaxPermSize=300M; support was removed in 8.0
[info] Loading global plugins from /Users/hjs/.sbt/0.13/plugins
[info] Loading project definition from /Volumes/Dev/Programming/ScalaJS/akka.js/project
[info] Set current project to root (in build file:/Volumes/Dev/Programming/ScalaJS/akka.js/)
> clean
[success] Total time: 0 s, completed 17-Jun-2015 09:38:19
> cleanFiles
[info] akkaActorTestJS/*:cleanFiles
[info] 	List(/Volumes/Dev/Programming/ScalaJS/akka.js/lib_managed, /Volumes/Dev/Programming/ScalaJS/akka.js/akka-js-actor-tests/js/target)
[info] akkaWorkerRaftJS/*:cleanFiles
[info] 	List(/Volumes/Dev/Programming/ScalaJS/akka.js/lib_managed, /Volumes/Dev/Programming/ScalaJS/akka.js/akka-js-worker/raft/target)
[info] akkaActorJS/*:cleanFiles
[info] 	List(/Volumes/Dev/Programming/ScalaJS/akka.js/lib_managed, /Volumes/Dev/Programming/ScalaJS/akka.js/akka-js-actor/js/target)
[info] akkaWorkerMainJS/*:cleanFiles
[info] 	List(/Volumes/Dev/Programming/ScalaJS/akka.js/lib_managed, /Volumes/Dev/Programming/ScalaJS/akka.js/akka-js-worker/main/target)
[info] akkaTestkitJS/*:cleanFiles
[info] 	List(/Volumes/Dev/Programming/ScalaJS/akka.js/lib_managed, /Volumes/Dev/Programming/ScalaJS/akka.js/akka-js-testkit/js/target)
[info] root/*:cleanFiles
[info] 	List(/Volumes/Dev/Programming/ScalaJS/akka.js/lib_managed, /Volumes/Dev/Programming/ScalaJS/akka.js/target)
> compile
[info] Updating {file:/Volumes/Dev/Programming/ScalaJS/akka.js/}akkaActorJS...
[info] Updating {file:/Volumes/Dev/Programming/ScalaJS/akka.js/}root...
[info] Resolving jline#jline;2.12.1 ...
[info] Done updating.
[info] Resolving org.eclipse.jetty#jetty-continuation;8.1.16.v20140903 ...
[info] downloading https://repo1.maven.org/maven2/org/scalatest/scalatest_sjs0.6_2.11/3.0.0-M1/scalatest_sjs0.6_2.11-3.0.0-M1.jar ...
[info] 	[SUCCESSFUL ] org.scalatest#scalatest_sjs0.6_2.11;3.0.0-M1!scalatest_sjs0.6_2.11.jar(bundle) (4205ms)
[info] Done updating.
[info] Updating {file:/Volumes/Dev/Programming/ScalaJS/akka.js/}akkaTestkitJS...
[info] Updating {file:/Volumes/Dev/Programming/ScalaJS/akka.js/}akkaWorkerRaftJS...
[info] Updating {file:/Volumes/Dev/Programming/ScalaJS/akka.js/}akkaWorkerMainJS...
[info] Resolving com.github.benhutchison#prickle_sjs0.6_2.11;1.1.5 ...
[info] Compiling 77 Scala sources to /Volumes/Dev/Programming/ScalaJS/akka.js/akka-js-actor/js/target/scala-2.11/classes...
[info] Resolving org.eclipse.jetty#jetty-continuation;8.1.16.v20140903 ...
[info] Done updating.
[info] Resolving org.scala-lang#scala-library;2.11.6 ...
[info] Updating {file:/Volumes/Dev/Programming/ScalaJS/akka.js/}akkaActorTestJS...
[info] Resolving org.eclipse.jetty#jetty-continuation;8.1.16.v20140903 ...
[info] Done updating.
[info] Resolving org.eclipse.jetty#jetty-continuation;8.1.16.v20140903 ...
[info] Done updating.
[info] Resolving org.eclipse.jetty#jetty-continuation;8.1.16.v20140903 ...
[info] Done updating.
[error] /Volumes/Dev/Programming/ScalaJS/akka.js/akka-js-actor/js/../../akka-js-testkit/js/src/test/scala/akka/testkit/AkkaSpec.scala:8: object scalatest is not a member of package org
[error] import org.scalatest.{

I cloned it with my changes https://github.com/bblfish/akka.js/commits/master

@InTheNow suggested the following hack
https://github.com/bblfish/akka.js/commit/660d6ad9c5a4426c5d119448064b86ed6ac65b58
With that it works. But apparently that reveals a deeper problem with the build file.

I am able to compile fine, but publishLocal has problems.
[error] /Volumes/Dev/Programming/ScalaJS/akka.js/akka-js-testkit/js/src/main/scala/akka/testkit/TestActorRef.scala:52: not found: value props
[error]     props,
[error]     ^
[error] /Volumes/Dev/Programming/ScalaJS/akka.js/akka-js-testkit/js/src/main/scala/akka/testkit/TestActorRef.scala:53: not found: value dispatcher
[error]     dispatcher,
[error]     ^
[error] /Volumes/Dev/Programming/ScalaJS/akka.js/akka-js-actor/js/../../akka-js-testkit/js/src/main/scala/akka/testkit/TestActorRef.scala:52: not found: value props
[error]     props,
[error]     ^
[error] /Volumes/Dev/Programming/ScalaJS/akka.js/akka-js-actor/js/../../akka-js-testkit/js/src/main/scala/akka/testkit/TestActorRef.scala:53: not found: value dispatcher
[error]     dispatcher,
[error]     ^

and it seems that as a result I don't have all the jars, so I can't yet run the examples.
 $ ls /Users/hjs/.ivy2/local/akka.js/
akkaactortest_sjs0.6_2.11    akkaworkermainjs_sjs0.6_2.11 akkaworkerraftjs_sjs0.6_2.11 root_2.11

Your problems target only testkit, that is not needed to run examples,
please try publishing local the other sub projects.
On 17 Jun 2015 15:09, "Henry Story"<EMAIL_ADDRESS>wrote:

I am able to compile fine, but publishLocal has problems.
[error] /Volumes/Dev/Programming/ScalaJS/akka.js/akka-js-testkit/js/src/main/scala/akka/testkit/TestActorRef.scala:52: not found: value props
[error]     props,
[error]     ^
[error] /Volumes/Dev/Programming/ScalaJS/akka.js/akka-js-testkit/js/src/main/scala/akka/testkit/TestActorRef.scala:53: not found: value dispatcher
[error]     dispatcher,
[error]     ^
[error] /Volumes/Dev/Programming/ScalaJS/akka.js/akka-js-actor/js/../../akka-js-testkit/js/src/main/scala/akka/testkit/TestActorRef.scala:52: not found: value props
[error]     props,
[error]     ^
[error] /Volumes/Dev/Programming/ScalaJS/akka.js/akka-js-actor/js/../../akka-js-testkit/js/src/main/scala/akka/testkit/TestActorRef.scala:53: not found: value dispatcher
[error]     dispatcher,
[error]     ^
and it seems that as a result I don't have all the jars, so I can't yet
run the examples.
$ ls /Users/hjs/.ivy2/local/akka.js/
akkaactortest_sjs0.6_2.11    akkaworkermainjs_sjs0.6_2.11 akkaworkerraftjs_sjs0.6_2.11 root_2.11
—
Reply to this email directly or view it on GitHub
https://github.com/unicredit/akka.js/issues/19#issuecomment-112793891.

@andreaTP I was in project root, which should publish all of them, no?
On looking at it closer it looks like a lot of files are generated locally during publishLocal
$ find . -name "*.jar" -exec ls -sh {} \;
392 ./akka-js-actor/js/target/scala-2.11/akkaactor_sjs0.6_2.11-0.2-SNAPSHOT-sources.jar
6432 ./akka-js-actor/js/target/scala-2.11/akkaactor_sjs0.6_2.11-0.2-SNAPSHOT.jar
8 ./akka-js-actor-tests/js/target/scala-2.11/akkaactortest_sjs0.6_2.11-0.2-SNAPSHOT-javadoc.jar
8 ./akka-js-actor-tests/js/target/scala-2.11/akkaactortest_sjs0.6_2.11-0.2-SNAPSHOT-sources.jar
8 ./akka-js-actor-tests/js/target/scala-2.11/akkaactortest_sjs0.6_2.11-0.2-SNAPSHOT.jar
56 ./akka-js-testkit/js/target/scala-2.11/akkatestkit_sjs0.6_2.11-0.2-SNAPSHOT-sources.jar
720 ./akka-js-testkit/js/target/scala-2.11/akkatestkit_sjs0.6_2.11-0.2-SNAPSHOT.jar
696 ./akka-js-worker/main/target/scala-2.11/akkaworkermainjs_sjs0.6_2.11-0.2-SNAPSHOT-javadoc.jar
8 ./akka-js-worker/main/target/scala-2.11/akkaworkermainjs_sjs0.6_2.11-0.2-SNAPSHOT-sources.jar
40 ./akka-js-worker/main/target/scala-2.11/akkaworkermainjs_sjs0.6_2.11-0.2-SNAPSHOT.jar
912 ./akka-js-worker/raft/target/scala-2.11/akkaworkerraftjs_sjs0.6_2.11-0.2-SNAPSHOT-javadoc.jar
24 ./akka-js-worker/raft/target/scala-2.11/akkaworkerraftjs_sjs0.6_2.11-0.2-SNAPSHOT-sources.jar
480 ./akka-js-worker/raft/target/scala-2.11/akkaworkerraftjs_sjs0.6_2.11-0.2-SNAPSHOT.jar
8 ./target/scala-2.11/root_2.11-0.1-SNAPSHOT-javadoc.jar
8 ./target/scala-2.11/root_2.11-0.1-SNAPSHOT-sources.jar
8 ./target/scala-2.11/root_2.11-0.1-SNAPSHOT.jar

but not all of them are copied to the .ivy2 directory
$ cd ~/.ivy2/local/akka.js/
$ find . -name "*.jar" -exec ls -sh {} \;
8 ./akkaactortest_sjs0.6_2.11/0.2-SNAPSHOT/docs/akkaactortest_sjs0.6_2.11-javadoc.jar
8 ./akkaactortest_sjs0.6_2.11/0.2-SNAPSHOT/jars/akkaactortest_sjs0.6_2.11.jar
8 ./akkaactortest_sjs0.6_2.11/0.2-SNAPSHOT/srcs/akkaactortest_sjs0.6_2.11-sources.jar
696 ./akkaworkermainjs_sjs0.6_2.11/0.2-SNAPSHOT/docs/akkaworkermainjs_sjs0.6_2.11-javadoc.jar
40 ./akkaworkermainjs_sjs0.6_2.11/0.2-SNAPSHOT/jars/akkaworkermainjs_sjs0.6_2.11.jar
8 ./akkaworkermainjs_sjs0.6_2.11/0.2-SNAPSHOT/srcs/akkaworkermainjs_sjs0.6_2.11-sources.jar
912 ./akkaworkerraftjs_sjs0.6_2.11/0.2-SNAPSHOT/docs/akkaworkerraftjs_sjs0.6_2.11-javadoc.jar
480 ./akkaworkerraftjs_sjs0.6_2.11/0.2-SNAPSHOT/jars/akkaworkerraftjs_sjs0.6_2.11.jar
24 ./akkaworkerraftjs_sjs0.6_2.11/0.2-SNAPSHOT/srcs/akkaworkerraftjs_sjs0.6_2.11-sources.jar
8 ./root_2.11/0.1-SNAPSHOT/docs/root_2.11-javadoc.jar
8 ./root_2.11/0.1-SNAPSHOT/jars/root_2.11.jar
8 ./root_2.11/0.1-SNAPSHOT/srcs/root_2.11-sources.jar

the 4 missing files are
akkaactor_sjs0.6_2.11-0.2-SNAPSHOT-sources.jar
akkaactor_sjs0.6_2.11-0.2-SNAPSHOT.jar
akkatestkit_sjs0.6_2.11-0.2-SNAPSHOT-sources.jar
akkatestkit_sjs0.6_2.11-0.2-SNAPSHOT.jar

updated dep and looks that everything works properly now,
at a first look the build has got problems, we will fix it as soon as @yawnt  get back focused on the project development.

yep. it works now. I was even able to open the calculator examples and get them to work. Thanks.


Implement a mapbox.js heatmap layer

Haven't found an existing implementation, might do so myself soon.

There's a mapbox heatmap plugin available on mapbox now


Invalid Saved Account Cookie (including bad json) causes uaa to throw 500

Coverage decreased (-20.8%) to 65.415% when pulling 2d4aabe22cbb640dc92d9cc65bc82acf4aeb9a7e on GESoftware-CF:fix/savedAccountCookieBadJson into 2a1103feca2f89319c065a3ccc91ae7fa1655b44 on cloudfoundry:develop.

Coverage increased (+0.002%) to 86.223% when pulling 6fd18d144f54678964df2f1df5752055e8fe4837 on GESoftware-CF:fix/savedAccountCookieBadJson into 2a1103feca2f89319c065a3ccc91ae7fa1655b44 on cloudfoundry:develop.

@6palace how was this invalid cookie produced? did UAA generate an invalid cookie, or was it manually crafted?

@cbguder Older versions of UAA produced cookies in a different format, a format that was invalid. So when that story was fixed, those cookies still existed in many user agents.
Merging this request, as it is good behavior to only read cookies of a format we can understand without causing a 500


System.IO.FileNotFoundException: The configuration file 'appsettings.json' was not found

When I set default project to Persistence and then Update-Database -Context AppDbContext in Package Manager Console and then show this error. How can I solve this problem?
System.Reflection.TargetInvocationException: Exception has been thrown by the target of an invocation.  ---> System.IO.FileNotFoundException: The configuration file 'appsettings.json' was not found and is not optional. The expected physical path was 'D:\sample\src\Infrastructure\Persistance\bin\Debug\net6.0\appsettings.json'.    at Microsoft.Extensions.Configuration.FileConfigurationProvider.HandleException(ExceptionDispatchInfo info)    at Microsoft.Extensions.Configuration.FileConfigurationProvider.Load(Boolean reload)    at Microsoft.Extensions.Configuration.FileConfigurationProvider.Load()    at Microsoft.Extensions.Configuration.ConfigurationRoot..ctor(IList1 providers)    at Microsoft.Extensions.Configuration.ConfigurationBuilder.Build()    at Sample.Persistance.DesignTimeDbContextFactoryBase1.Create(String basePath, String environmentName) in D:\sample\src\Infrastructure\Persistance\DesignTimeDbContextFactoryBase.cs:line 26    at Sample.Persistance.DesignTimeDbContextFactoryBase1.CreateDbContext(String[] args) in D:\sample\src\Infrastructure\Persistance\DesignTimeDbContextFactoryBase.cs:line 18

Write this command in powershell on Persistance directory can create database.
dotnet ef --startup-project ../../Web/Api database update --context  AppDbContext

Please make sure your project build successfully.
After that set the Api layer as your startup project.
If your problem still exist, feel free to contact me.

I ran into this issue also. I resolved it by ensuring that the 'Private Assets' value was cleared instead of being set to 'All' for each Microsoft.EntityFramework dependency package.

I tested again with a fresh solution.
The 3 packages that have 'Private Assets' set to 'All' are:
- Persistance Package: Microsoft.EntityFrameworkCore.Design (6.0.0)
- Persistance Package: Microsoft.EntityFrameworkCore.Tools (6.0.0)
- ApiFramework Package: Microsoft.EntityFrameworkCore.Design (6.0.0)
I cleared the 'Private Assets' value for ApiFramework Package: Microsoft.EntityFrameworkCore.Design (6.0.0) only.
I set the startup project to Api.
I was then able to successfully run the update command: Update-Database -Context AppDbContext.

Please change the Create method in DesignTimeDbContextFactoryBase.cs file as below:
    private TContext Create(string basePath, string environmentName)
    {

        var configuration = new ConfigurationBuilder()
            .SetBasePath(basePath)
            .AddJsonFile("appsettings.json")
            .AddJsonFile(Path.Combine(Path.GetDirectoryName(System.Reflection.Assembly.GetExecutingAssembly().Location),
            $"appsettings.{environmentName}.json"), optional: true, reloadOnChange: true)
            .AddEnvironmentVariables()
            .Build();

        var appOptions = configuration.GetSection("AppOptions:WriteDatabaseConnectionString");
        var connectionString = appOptions.Value;

        return Create(connectionString);
    }

@akazemie Thanks for your suggestion. Please fork the project and make changes, then I can merge it with the repository.

As there have been no further responses on this issue, it will now be closed.


Upgrade postgres for sandbox and production

Context
To upgrade the postgres database to version 14 for sadbox and production
Changes proposed in this pull request

Introduced variable to set the version number

Guidance to review

make sandbox_aks deploy-plan PASSCODE=1234
make production_aks deploy-plan PASSCODE=1234

Link to Trello card
https://trello.com/c/JH27e84R/275-upgrade-postgres-databases-to-14
Things to check

[ ] If the code removes any existing feature flags, a data migration has also been added to delete the entry from the database
[ ] This code does not rely on migrations in the same Pull Request
[ ] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[ ] If this code adds a column to the DB, decide whether it needs to be in analytics yml file or analytics blocklist
[ ] API release notes have been updated if necessary
[ ] Required environment variables have been updated added to the Azure KeyVault

 Upgrade postgres databases to 14


KolNovel(arabic) Source doesn't work

Steps to reproduce

Expected behavior
This should fixed
Actual behavior
Same thing
LNReader version
1.1.12
Android version
Android 11
Device
I don't know what's this
Other details

Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with an source, I should be opening an issue in the sources repository.
[X] I have updated the app to version 1.1.12.
[X] I will fill out all of the requested information in this form.

If this is an issue with an source, I should be opening an issue in the sources repository.


confuesed

V = ThinPlateSpline2(U, source, target, out_size)
can you tell me the meaning of source and target?what's the meaning of these points?

For example, assume that you want to make a warp that drags point 1 (p1) to point 2. Then, in this case, p1 is source and p2 is target. STN uses relative coordinates. As in the README files,
s_ = np.array([ # source position
  [-0.5, -0.5],
  [0.5, -0.5],
  [-0.5, 0.5],
  [0.5, 0.5]])

t_ = np.array([ # target position
  [-0.3, -0.3],
  [0.3, -0.3],
  [-0.3, 0.3],
  [0.3, 0.3]])

The square of scale 0.3 is dragged to 0.5. Therefore, the circle in 'transformed2.png' is swollen.

@iwyoo ,thanks,do you mean souce is an constant,target can change?If I want use TPS in cnn,I should regress the target rather than target？

 Source and target both can be constant or variable. There is more simpler implementation https://github.com/iwyoo/TPS_STN-tensorflow

@iwyoo thanks,in fact,I use this code in classfication task(rotated mnist),but it seems like the image didn't change after TPS,my code is like below:

mnist:

I save the trans data to image,but it's same as input

Is classification being trained well? You need to check values of 'loc' tensor. If training is not going well, you need to decrease number of control points (now it looks 20. However, even the original STN paper uses 4x4=16 control points.)

@iwyoo ss_ = np.array([ # source position
[-1, -1],
[1, -1],
[-1, 0],
[0, 1]])
t_ = np.array([ # target position
[-1, -1],
[1, -1],
[-1, 1],
[1, 1]])
Use this points to transform image,the output image  is the top half of the original image？

@iwyoo  Can I get your email? thanks for your help!

You can mail me<EMAIL_ADDRESS>For the question, no. To move top half, source and target should be,
source : [-1, -1], [-1, 1], [1, -1], [1, 1]
target : [-1, -1], [-1, 0], [1, -1], [1, 0]
It is easy to understand if you think about transformation that pulls target's points to the position of source.

can you please tell what is relative coordinate here ?


Crashing when taking screenshots?

Investigate the issue this user is having: https://stackoverflow.com/questions/38976711/python-quits-unexpectedly-running-pyautogui

They've solved their issue on stack overflow. This problem is caused by importing pyscreenshot along with pyautogui. I'm editing the title of this issue to reflect this.


Open to a new logo?

Is your feature request related to a problem? Please describe.
Often if I recommend Squoosh to a friend, they make a comment on the initial logo of the hand how it looks a bit strange as well as having slight graphical errors.
At least it is nicer logo than Tinypng's panda always looking at my soul.
Describe the solution you'd like
I can or someone else can make a logo that has a bit more polish to replace the hand logo.
Does other service/app have this feature?
Squoosh's logo stands out to me and people I have talked to as an uncomfortable logo out of apps I use regularly, though I realize this may not be the common experience.
Additional context

Thanks for bringing this up! We're actively looking into updating and rebranding our product, and will have an update for you in the coming months.


Quickfix lets trams ignore lane arrows

Quick-Fix to enable trams to ignore lane arrows when busses are allowed to ignore them
fixes #1053

can you please send us screenshot proving this change does fix #1053


Fix weird indentation in tpm(client|test) to keep GCC v6 happy.

Signed-off-by: Philip Tricca<EMAIL_ADDRESS>

This resolves #366 #226 and #229

resolved in e72e140fa7fc91405ef48fe188acefa4d3ce4271


EVM: download tokens at last processed round

a preview of what it will be like to do EVM queries for downloading token data + token balances at the ~latest round instead of the last mutated round
this is for sapphire where blocks far in the past aren't accessible
tested on emerald, where I ran the block scanner over a range of blocks with token and token balance analyzers disabled then ran with only token analyzer and token balance analyzer. checked that tokens and token balances are still downloaded successfully and recorded to have been downloaded at the latest scanned block round

testing description added :+1:


AttributeError: 'cv2.structured_light_GrayCodePattern' object has no attribute 'generate'

AttributeError: 'cv2.structured_light_GrayCodePattern' object has no attribute 'generate'
How to fix it?

@Xephen20
Unfortunately, however, the opencv-python and opencv-contrib-python packages on pip are sometimes released in a broken state.
Try to roll back your packages (for example 4.5.0 or 4.4.x).

Okey, thanks for fast response ;)


Azure App Service Deploy must support setting Runtime Stack to PHP 7.2

Environment
Hosted Linux Preview agent connected to VSTS
Issue Description
I am trying to deploy a PHP 7.2 web application to App Service on Linux using the task Azure App Service Deploy however there is no option for PHP 7.2:

@madhancock , Dynamic picklist is available in 4.* task version.
Please follow the steps for smooth transition from 3.* to 4.*

Copy the Package or folder path input. (Skip if the input is $(System.DefaultWorkingDirectory)/**/*.zip)

Change the task version to 4.*

Select Web App on Linux for App Service type.

Paste the Package or folder path input.
-Select your preferred Runtime stack.

@vincentdass This worked fine yesterday.
A release today failed with the below:
2018-04-27T13:18:28.4373008Z ##[section]Starting: Deploy Azure App Service 2018-04-27T13:18:28.4438422Z ============================================================================== 2018-04-27T13:18:28.4452650Z Task         : Azure App Service Deploy 2018-04-27T13:18:28.4469907Z Description  : Update Azure WebApp Services On Windows, Web App On Linux with built-in images or docker containers, ASP.NET, .NET Core, PHP, Python or Node based Web applications, Function Apps, Mobile Apps, Api applications, Web Jobs using Web Deploy / Kudu REST APIs 2018-04-27T13:18:28.4487871Z Version      : 4.1.1 2018-04-27T13:18:28.4502416Z Author       : Microsoft Corporation 2018-04-27T13:18:28.4516437Z Help         : [More Information](https://aka.ms/azurermwebdeployreadme) 2018-04-27T13:18:28.4531608Z ============================================================================== 2018-04-27T13:18:29.1194543Z ##[error]Unhandled: Cannot find module 'webdeployment-common/msDeployUtility.js' 2018-04-27T13:18:29.1950153Z ##[section]Finishing: Deploy Azure App Service

Hello Mike,
We have fixed this issue on Monday. Please try it out and let me know if you face any issues.
Thanks,
Vineet Mimrot


Auth Token in configuration file

I see there is a cli option for authorization via a token, however I cannot find anywhere how to put that into the actual configuration file.
Thanks,

Awesome, thank you!!


Add a basic README

The README should contain the following:

An overview of the project.
High level design.
A link to the image processing algorithm wiki.
Steps to run the application (without docker).
Steps to run the application (with docker).
Licensing information (with link).

The high level design will be omitted from this, it can be added in a wiki and linked here later.


Fix missing error output from timeout

The exits would prevent the printing code from running. Thus we would get a failed test but no error message.

Looks good. There is no WIP in the title, but I'll double-check anyway: ready for merge?

Yes it is ready. I wanted to isolate this little fix to get it in ASAP.


[Integration_test 2.5] tester.enterText is broken in release mode

Since the Flutter 2.5 upgrade, tester.enterText() seems broken in my web integration tests in --release mode.
When I run flutter drive with --no-headless, I can see that no text is added in text fields.
Steps to Reproduce
You can reproduce it with this repo https://github.com/rxlabz/text_drive
sh drive_web.sh
sh drive_web_release.sh

You can also see the result of 3 GH actions workflows in the project :

the first is running the integration_test without --release (https://github.com/rxlabz/text_drive/blob/master/.github/workflows/integration.yaml)  => it works ( all tests passed ) https://github.com/rxlabz/text_drive/actions/runs/1217290271

the other uses--release (https://github.com/rxlabz/text_drive/blob/master/.github/workflows/integration_release.yaml) and fails => https://github.com/rxlabz/text_drive/actions/runs/1217290268

the last one is running with flutter 2.2.3 in --release https://github.com/rxlabz/text_drive/actions/runs/1217434660/workflow => it works https://github.com/rxlabz/text_drive/runs/3556551737

App
import 'package:flutter/material.dart';

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

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

  @override
  Widget build(BuildContext context) => const MaterialApp(home: MainScreen());
}

class MainScreen extends StatefulWidget {
  const MainScreen({Key? key}) : super(key: key);

  @override
  State<MainScreen> createState() => _MainScreenState();
}

class _MainScreenState extends State<MainScreen> {
  final TextEditingController controller = TextEditingController();

  String value = 'test';

  @override
  void initState() {
    super.initState();
    controller.addListener(() => setState(() => value = controller.text));
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(),
      body: Center(
        child: Column(
          mainAxisSize: MainAxisSize.min,
          children: [
            TextFormField(key: const Key('field'), controller: controller),
            Text(value)
          ],
        ),
      ),
    );
  }
}

integration test
import 'package:flutter/widgets.dart';
import 'package:flutter_test/flutter_test.dart';
import 'package:integration_test/integration_test.dart';
import 'package:text_drive/main.dart';

void main() {
  IntegrationTestWidgetsFlutterBinding.ensureInitialized();

  testWidgets("failing test example", (WidgetTester tester) async {
    await tester.pumpWidget(const App());

    final input = find.byKey(const Key('field'));
    expect(input, findsOneWidget);

    expect(find.text('test'), findsOneWidget);

    await tester.tap(input);
    await tester.enterText(input, 'Hello');

    // same result with `updateEditingValue`
    // (cf.https://flutter.dev/docs/release/breaking-changes/enterText-trailing-caret)
    // await tester.showKeyboard(input);
    // await tester.pumpAndSettle();
    //tester.testTextInput.updateEditingValue(
    //  const TextEditingValue(text: 'Hello'),
    //);

    await tester.pump();

    expect(find.text('Hello'), findsNWidgets(2));
  });
}

@rxlabz
Thanks for reporting. This is strange, because I have a working integration_test login screen demo app which used to work until recently. After reading this issue, I tried to run it on web in debug and release mode which isn't working for me as well. On login screen, no action takes place:

But I verified this on stable 2.2.3, 2.5.0 and latest master, but on all of them, the test isn't entering any input.
Without --no-headless too, I don't think it's working.
I doubt if it's related to the chrome driver. Because it worked for Chrome 91, but now my Chrome version is 93, so I installed compatible chrome driver.
Does your code sample run locally in release mode ?

@darshankawar yes, the app runs normally with flutter run -d chrome --release

@rxlabz
I meant the integration_test in release mode.

@darshankawar :) sorry I wasn't sure.
Locally, it's the same than on GitHub actions : in release mode the integration test passes with Flutter 2.2.3, not with Flutter 2.5.

Thanks. Verified the same on stable 2.2.3 and on stable 2.5.0 in release mode and I see the same behavior.

 stable flutter doctor -v
[✓] Flutter (Channel stable, 2.2.3, on Mac OS X 10.15.4 19E2269 darwin-x64,
    locale en-GB)
    • Flutter version 2.2.3 at /Users/dhs/documents/fluttersdk/flutter
    • Framework revision f4abaa0735 (2 months ago), 2021-07-01 12:46:11 -0700
    • Engine revision 241c87ad80
    • Dart version 2.13.4

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.3, Build version 12C33
    • CocoaPods version 1.10.1

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

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

[✓] Connected device (3 available)
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729 • ios
      • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                               •
      web-javascript • Google Chrome 93.0.4577.63

• No issues found!

[✓] Flutter (Channel stable, 2.5.0, on Mac OS X 10.15.4 19E2269 darwin-x64,
    locale en-GB)
    • Flutter version 2.5.0 at /Users/dhs/documents/fluttersdk/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 4cc385b4b8 (5 days ago), 2021-09-07 23:01:49 -0700
    • Engine revision f0826da7ef
    • Dart version 2.14.0

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.3, Build version 12C33
    • CocoaPods version 1.10.1

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

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

[✓] Connected device (3 available)
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729 • ios
      • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                               •
      web-javascript • Google Chrome 93.0.4577.63

• No issues found!

It's not just in web mode. I have the same issue with running the integration tests on a real Android device. https://github.com/flutter/flutter/issues/89917

I am seeing this on both Android and iOS. When integration testing, tester.enterText() does nothing.

Please keep an eye on this PR : https://github.com/flutter/flutter/pull/89703

Removing the web label as this is a framework issue.

/cc: @dnfield

I'm not sure how this is wired up on the web side - it's more one for @yjbanov or @hterkelsen I think. Or maybe @mdebbar.

Please keep an eye on this PR : #89703

This PR does not change anything. It justs adds documentation.

Another workaround is to disable the semantics cf. https://github.com/flutter/flutter/issues/91492#issuecomment-973162329
testWidgets(
  'a test',
  (t){...},
  semanticsEnabled:false,
);

@lhimo yes it works in my projects. I just notice that tester.testTextInput should be registered to make it work. you can see it here https://github.com/rxlabz/text_drive/runs/5091722721?check_suite_focus=true

Data point for anyone having the same bug in ios, I can confirm simply adding tester.testTextInput.register(); at the start of the test fixes the text input for iOS on profile integration tests (previously would only work in debug builds). I did not need to disable semantics.


Get month json/html report from year database

I want to get HTML/JSON moth report from a persistent year database without storing month database, is posible?
Thanks in advance.

Are you looking to have a panel like this but for each month?

BTW, would this be a dup of this #1167?

Yes is it, currently Im doing 2 databases one year and one montly but by mistake I remove the montly one and want to resore the month html with yhe year database. Porbably yes is a duple. Thanks for reply!

Thanks for the remainder.  I will be working on this.

you are welcome!


Clean up HOOKS.md

Description
Cleaning up HOOKS.md in a separate PR to keep the hooks PR(s) active

[ ] Testing Added

Coverage remained the same at 63.902% when pulling fc3f456cf4d7e52d8fea94c845830d1bfd4bb642 on corinagum:HOOKS into a97fbb865982e0bccd3ca1e12d265628ab3441d9 on microsoft:master.


istioctl get all

Support istioctl get all, similar to kubectl get all.  The command would get and display all of the current types (routerule, egressrule, destinationpolicy, httpapispec, httpapispecbinding, quotaspec, quotaspecbinding, servicerole, servicerolebinding, virtualservice, gateway, serviceentry, destinationrule, policy) and support the -o yaml option but print tables by default.
In discussion on #networking Costin said "+1" and Lin Sun said "that seems useful.  today i do kubectl get crd to see which ones are istio".

It should also fetch all mixer Cards.
On Wed, May 9, 2018, 7:58 AM Ed Snible<EMAIL_ADDRESS>wrote:

Support istioctl get all, similar to kubectl get all. The command would
get and display all of the current types (routerule, egressrule,
destinationpolicy, httpapispec, httpapispecbinding, quotaspec,
quotaspecbinding, servicerole, servicerolebinding, virtualservice, gateway,
serviceentry, destinationrule, policy) and support the -o yaml option but
print tables by default.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/issues/5502, or mute the thread
https://github.com/notifications/unsubscribe-auth/AJGIxg4A1Mh3gmZ7pXlY9VxWH2vrakdgks5twwQtgaJpZM4T4eae
.

CRD can be assigned to categories (see https://kubernetes.io/docs/tasks/access-kubernetes-api/extend-api-custom-resource-definitions).
kind: CustomResourceDefinition
apiVersion: apiextensions.k8s.io/v1beta1
metadata:
  name: rules.config.istio.io
spec:
  group: config.istio.io
  names:
    kind: rule
    plural: rules
    singular: rule
    categories:
    - istio
  scope: Namespaced
  version: v1alpha2

You can then get by category, e.g.
$ kubectl get istio
NAME                                        AGE
routerule.config.istio.io/reviews-default   19m

NAME                                              AGE
servicerolebinding.config.istio.io/bind-ratings   19m

NAME                                                AGE
checknothing.config.istio.io/denyreviewsv3request   19m

NAME                                          AGE
denier.config.istio.io/denyreviewsv3handler   19m

NAME                                         AGE
servicerole.config.istio.io/ratings-viewer   19m

NAME                                 AGE
rule.config.istio.io/denyreviewsv3   19m

You could further categorize CRD by component type so things like kubectl get istio-mixer and kubectl get istio-pilot would work.


execution expired

[!] Couldn't determine repo type for URL: https://github.com/volcengine/volcengine-specs.git: execution expired


Fix heap-after-use bug in 'Concurrent'

ASAN found this.

This solution seems weird! Is there a possible "better" way to avoid it?

This solution seems weird! Is there a possible "better" way to avoid it?

Concurrent needs to get reworked now that we have a lot more abstractions to simplify it and not have this weird "dangling" continuation that gets type-erased into a Callback. Until then, any time we're executing code that could possibly be destructed we must ensure that we have everything that we might ever access on the stack.

ASAN found this.

I saw what looks like a very similar error: https://github.com/3rdparty/eventuals/runs/8028230226?check_suite_focus=true
In a run on https://github.com/3rdparty/eventuals/pull/507 which ran after this PR.
Is this issue still outstanding?


Table: don't show the sorting arrow when no sorting fn passed

Closes #128.
Sorting doesn't work if you don't pass onHeaderClick, so no need to show the arrow icon in this case.

Travis automatic deployment:
https://pr129--safereactcomponents.review.gnosisdev.com


Refactored the codebase into one monorepo

Refactored the codebase to maintain one monorepo rather than keeping 6 separate ones. Some advantages for this are removing the cyclic dependencies and decreasing the webpack size

@ryanwoconnor This needs more manual testing to ensure it all runs correctly, please consider it as a draft at the moment until full testing is completed.


Automatic resource deployment

Motivation
Automate the security resource deployment.

Feature
When a new security resource is contributed and merged, it requires a manual step by one of the SecurityHub owners, we want to make this automatic once the PR is merged.

Alternatives

Additional context

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle rotten

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

@poiana: Closing this issue.

In response to this:

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

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/test-infra repository.

@poiana: Closing this issue.

In response to this:

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

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/test-infra repository.


command line client

something that would use https://github.com/manifoldco/promptui and https://github.com/dairycart/dairyclient to view and manipulate data on a dairycart server

this should go in dairyclient


⚠ Bofrost.fr has degraded performance

In 0e6a7a6, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 961 ms

Resolved: Bofrost.fr performance has improved in b905eba after 7 minutes.


Examine issue with scenes/tri_area_light_with_sphere.pbrt

While I've recently improved correctness by a lot, I still see an issue with this scene. The sphere is too illuminated; I think it's possibly an issue with how we are sampling the triangular light?
As of 204524ed2

This is the issue. I would expect the shape of the light cast to match the random walk more closely; but we veer to the right. This seems like it could be due to how we're weighting samples on the triangular light?

This isn't really something I notice in other scenes with large triangular meshes; I think it's only apparent in some scenarios. Low-ish priority for right now.

Check if this is still an issue now that the most recent commit is in fixing uniform hemisphere PDF sampling


Misspelled "PowerShell" in defaults.json and settings.json files

Could PowerShell be properly capitalized in the defaults.json and settings.json file by GA release?
"Campbell Powershell" should be "Campbell PowerShell"
"Windows.Terminal.PowershellCore" should be "Windows.Terminal.PowerShellCore"
Actually, "Core" is now gone. In 0.11 Terminal team even removed 7 and is using just "PowerShell", so source should be "Windows.Terminal.PowerShell".

Environment
Windows build number:
Platform ServicePack Version      VersionString
-------- ----------- -------      -------------
 Win32NT             10.0.18363.0 Microsoft Windows NT 10.0.18363.0

Windows Terminal version:
Version: 0.11.1121.0

Steps to reproduce

Open auto-generated defaults.json and settings.json files.
Expected behavior

"name": "Campbell PowerShell"
"source": "Windows.Terminal.PowerShell"
Actual behavior

"name": "Campbell Powershell"
"source": "Windows.Terminal.PowershellCore"

有意思的是

Thanks for the report! Unfortunately, the source value is a contract between the user settings and Terminal, and it was created back when Core was part of powershell's name. It's not negotiable now that we've shipped it.
As for the color scheme name, I reckon we could fix it -- that color scheme is also deprecated, and only intended to be used by the inbuilt Windows PowerShell profile.
I don't disagree that these are wrong, but I'm going to mark this one as Won't fix. It's not worth the compatibility issue we'll probably cause with our future selves 😦

Windows Terminal is not shipped. It's still in preview. You broke my profiles.json in 0.11, and you haven't heard me complaining about that. Because it's a preview release. And I expect breaking changes. Not fixing wrong capitalization bugs before GA is just sloppy. Sorry.

If it were just you and just your opinion, I’d be sold on it, you know? But the number of duplicates on #5458 and the sheer volume of dissatisfaction even though we’re in preview has made us more than a little shy about making further breaking changes.. especially mere weeks before we do final signoffs. I’m happy to fix the scheme name, but the generator may just have to stay. Sorry.


Updated beta for 8.0

Checklist

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

The previous build reported the wrong version number.

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.

Looks like it built fine, but then died on the very last packaging step for only one of the osx_arm64 builds (looks like the rest are just fine)
2022-09-30T18:43:07.6103270Z [cctools-port]: generating fake signature for '/Users/runner/miniforge3/conda-bld/openmm_1664562128914/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_pl/lib/plugins/libOpenMMPME.dylib'
2022-09-30T18:43:07.6108130Z
2022-09-30T18:43:12.7859660Z Partially parsed some '.tbd' files in sysroot /Applications/Xcode_13.2.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.0.sdk, pretending .tbds are their install-names
2022-09-30T18:43:12.7861740Z Adding support to 'conda-build' for parsing these in 'liefldd.py' would be easy and useful:
2022-09-30T18:43:12.7863470Z ['System/iOSSupport/System/Library/Frameworks/_AuthenticationServices_SwiftUI.framework/_AuthenticationServices_SwiftUI.tbd', 'System/Library/PrivateFrameworks/AirPlaySupport.framework/AirPlaySupport.tbd'] ...
2022-09-30T18:43:12.7865660Z WARNING:conda_build.post:Partially parsed some '.tbd' files in sysroot /Applications/Xcode_13.2.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.0.sdk, pretending .tbds are their install-names
2022-09-30T18:43:12.7867190Z Adding support to 'conda-build' for parsing these in 'liefldd.py' would be easy and useful:
2022-09-30T18:43:12.7869100Z ['System/iOSSupport/System/Library/Frameworks/_AuthenticationServices_SwiftUI.framework/_AuthenticationServices_SwiftUI.tbd', 'System/Library/PrivateFrameworks/AirPlaySupport.framework/AirPlaySupport.tbd'] ...
2022-09-30T18:43:12.8699060Z /Users/runner/miniforge3/etc/profile.d/conda.sh: line 9:  3345 Segmentation fault: 11  "$CONDA_EXE" $_CE_M $_CE_CONDA "$@"
2022-09-30T18:43:12.8810370Z /Users/runner/miniforge3/lib/python3.10/multiprocessing/resource_tracker.py:224: UserWarning: resource_tracker: There appear to be 1 leaked semaphore objects to clean up at shutdown
2022-09-30T18:43:12.8813480Z   warnings.warn('resource_tracker: There appear to be %d '
2022-09-30T18:43:12.8938390Z ##[error]Bash exited with code '139'.
2022-09-30T18:43:12.8990220Z ##[section]Finishing: Run OSX build

I think that was probably just an intermittent failure.  They happen sometimes.  I'll watch the real builds and see if they succeed.

All builds succeeded.


Player Rendering

I forgot that we had to move the player render out of render_text. Should we render the player above the text? Or keep it above all level elements except for text (will be behind text, but ahead of everything else).

I'm thinking everything is fine, the way it is now. The player > text > draw > blocks, in terms of priority and viewing of pixels on screen.

So you want players to be on top of text?

Yes, that's fine.


rename "header" to "heading"

A header is a whole container of introductory content, such as site logo, navigation, and search field. A heading is the title of a section of a document — headings create the document outline. In HTML, headers are represented by the <header> element and level headings are represented by <h1>–<h6>.
It looks like this repo refers to the latter. It might be helpful to reinforce established terminology and use "heading" instead of "header".
see: https://www.google.com/search?q=header+vs+heading

Fully agree here.

+1

+1

+1


how to configure a two-line prompt with text on both sides

Prerequisites

[ ] I have read and understand the CONTRIBUTING guide
[x] I looked for duplicate issues before submitting this one

Description

I am trying to create a prompt that looks like the PNG embedded above, which is just a mockup. Notice that the prompt is made up of 4 blocks, labeled 1 through 4. And I'm sorry the two fields on the right are so hard to see - the contrast is better in real life.
I am having trouble getting all 4 blocks to be visible with the cursor landing after block 3. Can you help?
FYI, here are some of the things I have tried:

block 1, align=left
block 2, align=right
newline
block 3, align=left
block 4, align=right

This put all of the blocks in the right places, but the cursor lands on the screen's far-right end, after block 4.
Here's another experiment:

block 1, align=left
block 2, align=right
newline
block 4, align=right
block 3, align=left

This causes blocks 1, 2, and 4 to be displayed in their proper spots. However, block 4 begins to display immediately after block 3, on the screen's very far right. It then wraps to the next line, leaving a gap on the left side of the screen between blocks 1 and 3.
Another one:

block 1, align=left
block 2, align=right
newline
block 4, align=right
newline
block 3, align=left

This time everything is great, except there is a gap between blocks 1 and 3. Makes sense - that's obviously what I requested. But I thought this one was close, so I tried tweaking it:

block 1, align=left
block 2, align=right
newline
block 4, align=right
newline
block 3, align=left, vertical offset = -1

This caused the first 3 blocks to appear where they should, with the cursor after block 3, which is what I want. However, block 4 is not visible after the prompt renders, so it looks like block 3 deleted everything on the second line before being rendered.
I hope this makes sense. I may have made mistakes above as I'm trying to switch configurations in a single config file and then observe the effect and report it here in this little tiny box. :-) But the point is that I can't find a way to put all 4 blocks where I want them, with the cursor ending up after block 3. Is this possible?
Environment

Oh my Posh version: 3.44.1
Theme: a variation on jandedobbeleer. I will attach it.
Operating System: Windows 10 build 21277.1000
Shell: Powershell, both versions 7.1.0 and 5.1.21277.1000
Terminal: Windows Terminal 1.4.3243.0 and ConEmu 201011 Preview

Steps to Reproduce

Use the attached theme, or one of the variants described above
Observe and ponder

Expected behavior: [What you expected to happen]
Blocks 1, 2, 3, 4 all where they are in the mockup, with the cursor placed after block 3.
Actual behavior: [What actually happened]
It depends on how I order and configure the blocks (see above).
Here is my theme (this is the final layout discussed above which uses vertical offset):
{
  "blocks": [
    {
      "type": "newline"
    },
    {
      "type": "prompt",
      "alignment": "left",
      "segments": [
        {
          "type": "shell",
          "style": "diamond",
          "leading_diamond": "\uE0B6",
          "trailing_diamond": "\uE0B4",
          "foreground": "#ffffff",
          "background": "#444444",
          "properties": {
            "prefix": " \uFCB5 "
          }
        },
        {
          "type": "path",
          "style": "powerline",
          "powerline_symbol": "\uE0B4",
          "foreground": "#151515",
          "background": "#448844",
          "properties": {
            "prefix": " \uE5FF ",
            "styles": "agnoster, agnoster_full, agnoster_short, full, folder",
            "style": "agnoster"
          }
        },
        {
          "type": "git",
          "style": "powerline",
          "powerline_symbol": "\uE0B4",
          "foreground": "#111111",
          "background": "yellow",
          "properties": {
            "display_stash_count": true,
            "display_upstream_icon": true
          }
        }
      ]
    },
    {
      "type": "prompt",
      "alignment": "right",
      "segments": [
        {
          "type": "envvar",
          "style": "diamond",
          "leading_diamond": "\uE0B6",
          "trailing_diamond": "\uE0B4",
          "foreground": "#555555",
          "background": "#000000",
          "powerline_symbol": "\uE0B4",
          "properties": {
            "var_name": "POSH_lastCommandDuration"
          }
        }
      ]
    },
    {
      "type": "newline"
    },
    {
      "type": "prompt",
      "alignment": "right",
      "segments": [
        {
          "type": "time",
          "style": "diamond",
          "leading_diamond": "\uE0B6",
          "trailing_diamond": "\uE0B4",
          "foreground": "#555555",
          "background": "#000000",
          "powerline_symbol": "\uE0B4",
          "properties": {
            "time_format": "mon 02-jan-2006 15:04"
          }
        }
      ]
    },
    {
      "type": "newline"
    },
    {
      "type": "prompt",
      "alignment": "left",
      "vertical_offset": -1,
      "segments": [
        {
          "type": "session",
          "style": "diamond",
          "leading_diamond": "\uE0B6",
          "trailing_diamond": "\uE0B4",
          "foreground": "#ffffff",
          "background": "blue",
          "powerline_symbol": "\uE0B4"
        },
        {
          "type": "root",
          "style": "powerline",
          "powerline_symbol": "\uE0B4",
          "foreground": "#ffff66",
          "background": "#770000"
        },
        {
          "type": "exit",
          "style": "powerline",
          "powerline_symbol": "\uE0B4",
          "foreground": "red",
          "background": "white",
          "always_enabled": true,
          "always_numeric": true,
          "display_exit_code": true
        }
      ]
    }
  ],
  "final_space": true
}

Maybe something like this:

{
  "$schema": "https://raw.githubusercontent.com/JanDeDobbeleer/oh-my-posh3/main/themes/schema.json",
  "blocks": [
    {
      "type": "rprompt",
      "alignment": "left",
      "segments": [
        {
          "type": "text",
          "style": "plain",
          "foreground": "#ffffff",
          "properties": {
            "text": "block 4"
          }
        }
      ]
    },

    {
      "type": "prompt",
      "alignment": "left",
      "segments": [
        {
          "type": "text",
          "style": "plain",
          "foreground": "#ffffff",
          "properties": {
            "text": "block 1"
          }
        }
      ]
    },

    {
      "type": "prompt",
      "alignment": "right",
      "segments": [
        {
          "type": "text",
          "style": "plain",
          "foreground": "#ffffff",
          "properties": {
            "text": "block 2"
          }
        }
      ]
    },

    {
      "type": "newline"
    },

    {
      "type": "prompt",
      "alignment": "left",
      "segments": [
        {
          "type": "text",
          "style": "plain",
          "foreground": "#ffffff",
          "properties": {
            "text": "block 3"
          }
        }
      ]
    }
  ]
}

Maybe something like this:

{
  "$schema": "https://raw.githubusercontent.com/JanDeDobbeleer/oh-my-posh3/main/themes/schema.json",
  "blocks": [
    {
      "type": "rprompt",
      "alignment": "left",
      "segments": [
        {
          "type": "text",
          "style": "plain",
          "foreground": "#ffffff",
          "properties": {
            "text": "block 4"
          }
        }
      ]
    },

    {
      "type": "prompt",
      "alignment": "left",
      "segments": [
        {
          "type": "text",
          "style": "plain",
          "foreground": "#ffffff",
          "properties": {
            "text": "block 1"
          }
        }
      ]
    },

    {
      "type": "prompt",
      "alignment": "right",
      "segments": [
        {
          "type": "text",
          "style": "plain",
          "foreground": "#ffffff",
          "properties": {
            "text": "block 2"
          }
        }
      ]
    },

    {
      "type": "newline"
    },

    {
      "type": "prompt",
      "alignment": "left",
      "segments": [
        {
          "type": "text",
          "style": "plain",
          "foreground": "#ffffff",
          "properties": {
            "text": "block 3"
          }
        }
      ]
    }
  ]
}

Thank you, @TravisTX . I had forgotten about rprompt altogether, and it sounds like it may be meant for exactly what I'm trying to do.
However, I am having trouble reproducing your result. When I run your sample as-is I get this:

I've tried tweaking it, but this combination of things makes me uncertain of what to tweak:

block 4 is referenced first in the config
block 4 is referenced before the newline
block 4 has align=left (?)

Thanks again for the prompt response. Sorry I'm not able to sift an answer out of a good reply like that.

Thank you, @TravisTX . I had forgotten about rprompt altogether, and it sounds like it may be meant for exactly what I'm trying to do.
However, I am having trouble reproducing your result. When I run your sample as-is I get this:

I've tried tweaking it, but this combination of things makes me uncertain of what to tweak:

block 4 is referenced first in the config
block 4 is referenced before the newline
block 4 has align=left (?)

Thanks again for the prompt response. Sorry I'm not able to sift an answer out of a good reply like that.

@sql-sith it seems your version of oh-my-posh is rather old. Updating should do the trick as I don't think your version has rprompt support just yet.

@sql-sith it seems your version of oh-my-posh is rather old. Updating should do the trick as I don't think your version has rprompt support just yet.

@JanDeDobbeleer you are correct. I checked for newer versions tonight but obviously messed it up.
By the way, this is one of the most useful and comprehensive updates to a popular open-source project that I've seen for a while. Great job to all.

@JanDeDobbeleer you are correct. I checked for newer versions tonight but obviously messed it up.
By the way, this is one of the most useful and comprehensive updates to a popular open-source project that I've seen for a while. Great job to all.


Support pow of negative numbers with fractional powers

Support pow of negative numbers with fractional powers (note that I cheat in the example below since isclose is only in Python 3.5)
assert cmath.isclose(pow(-1, 0.5), 1j)
assert cmath.isclose((-1)**0.5, 1j)

Covered by test_pow in test_builtin.

Done in https://github.com/IronLanguages/ironpython3/pull/651.


Make cleanup pass on CheckModifiersDialog application

I think for a proper cleanup path you should probably lock in context.options's type earlier (like in the constructor?), that way you don't need to resolve set vs array in getData()

It's temporary until the next PR.

Then sure, a little regression is ok then. Was wondering because as is this isn't actually a cleanup.


Environment Variables

The only one coming to mind for this is the MAIL_URL at the moment, but it would be good to update the server startup with a loader for this like:
let startup = () => {
  _setBrowserPolicies();
  _generateAccounts();
  _setEnvironmentVariables();
};

let _setBrowserPolicies = () => {};

let _generateAccounts = () => Modules.server.generateAccounts();

let _setEnvironmentVariables = () => {
  process.env.MAIL_URL = Meteor.settings.private.MAIL_URL;
};

Modules.server.startup = startup;

@themeteorchef I actually did something like this recently: https://github.com/josephdburdick/vessel/commit/318bed1a39670e4867659ba6faf68a8e178710c6

@josephdburdick neat! Want to do a PR for Base? Only ask is that the pattern above is followed to support multiple environment variables.

@themeteorchef yeah that would be really cool. Given that in this context we're only talking about email and there could be many API token/keys/etc, should we leave blanks in the settings file like I have for Mailgun?

I would say just for MAIL_URL right now, like this:
{
  "public": {},
  "private": {
    "MAIL_URL": "" // leave empty
  }
}

I'm getting the following error when I try and start the app:
TypeError: Cannot read property 'MAIL_URL' of undefined
All I've done is clone the app and try and start it.

ignore my last comment. missed starting the app with --settings settings-development.json
(https://themeteorchef.com/base/settings/)
Thanks!

Issue #91 has been filed with a suggested fix @jasonbahl
Btw, a faster way to start the app without all that typing is by using $ npm start instead.

This will be patched and released as v3.2.1. Sorry for the confusion @jasonbahl!


Support AMD when loading splitChunks

Feature request

What is the expected behavior?
Webpack use JsonP to load other chunks. wondering whether we can load chunks by requireJs?
What is motivation or use case for adding/changing the behavior?
in my case, i want to share common chunks among different compilations, and assets from different compilations maybe used in the same page. since we use JsonP to load chunks, the output.chunkLoadingGlobal  from certain compilation will be set in common chunk files. so between different compilations , we wont get common chunks  exactly the same, and common share is failed.
How should this be implemented in your opinion?
maybe supply a option that splitChunks can be loaded by AMD, so the splitChunks  dont have to set some logic related to the main chunk. so we can share among compilations
Are you willing to work on this yourself?
sorry，i would like to but maybe not qualified.

Duplicate https://github.com/webpack/webpack/issues/5489


Anchor 0.30.0 IDL Support

closes https://github.com/metaplex-foundation/kinobi/issues/224

@lorisleiva I could use some guidance and maybe a jump start. What I have done is compared the IDLs and started working in the new or adjusted fields from the new Anchor spec. This makes a lot more fields optional and will require resolves to handle the cases where marked as optional.
Give it a pull, run specs, and let me know your thoughts.

Hey @kespinola, thanks for the PR. I was more thinking about separating the two IDL specs completely with a brand new set of types and helpers.
As I mentioned on the issue, I'm currently splitting up Kinobi into multiple modules inside a monorepo. This week, I've actually worked on the nodes-from-anchor package which contains all the logic of going from Anchor to Kinobi. With the Anchor IDL version 0.1.0 (the one used in Anchor 0.30) in mind, I marked all IDL types and helpers with V00 to represent Anchor IDL version 0.0.0 (as it was not versioned before). I've also grouped them all in a v00 folder so it is easier to create a new v01 next to it.
https://github.com/kinobi-so/kinobi/blob/main/packages/nodes-from-anchor/src/v00/idl.ts
It might be easier for you to work on the v01 spec there directly by just copy/pasting the new IDL spec, adding V01 markers to the types, and start doing the wiring with Kinobi. There are even lots of tests for v00 that you can copy/paste/edit to make sure it creates the right tree.
Just keep in mind that I've still got more work to do on the Kinobi monorepo before I can publish it so if you go down that route, it might take a bit longer to ship.

Thanks @lorisleiva, that's great. I too would prefer them split so will start working with the new version.
I also like that the project is being pulled into its on org.
As long as I can build a client by referencing it locally I'm happy.
Moving convo and PR to the new repository.


v-img directive can be used with any element

Changed in e03bbcd

Anybody resolve conflicts? :)

@dev-blinov Will do in few days! :) Thanks!

@fahrenq +1 would be great to have 😄!


How to add optional queries?

I have a REST API that is defined as follows:
/transactions?account_id=1234
i.e. get me all the transactions with account_id = 1234. The query parameter account_id is optional.
What is the most elegant way (code-wise) to structure this query? This is what I have right now:
function getTransactions(account_id) {
    return Transaction
        .where({account_id: account_id})
        .fetchAll()
        .then(function(transactions) {
            return transactions.load([
                'account',
                'category'
            ]);
        });
}

However when account_id is undefined, I am obviously getting an empty collection. So I am trying to figure out what's the best way to put the where clause optionally. Also is there a good pattern whereby a number of optional parameters can be accommodated?

How about:
function getTransactions(account_id) {
  var options = account_id ? {where: {account_id: account_id}} : {}

  return Transaction
    .query(options)
    .fetchAll()
    .then(function(transactions) {
      /* ... */
    });
}

Good one! Thanks for your help, @ricardograca.


Changing the default directory in the "Open folder..."  doesn't work

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:  1.86.2
OS Version: Winodws 11, wsl ubuntu 22.04

Steps to Reproduce:
I tried to use this new feature, but it didn't work :((  https://github.com/microsoft/vscode/pull/182908 https://github.com/microsoft/vscode/pull/186156
Has this setting been removed? I tried different options but none of them work ..,

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.89.1. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

Also I updated vscode 1.86.2 -> 1.89.1 and it didn't help :(

files.dialog.homePath isn't a setting. files.dialog.defaultPath is a setting. Please see the description of the setting to learn when it applies.

files.dialog.homePath isn't a setting. files.dialog.defaultPath is a setting. Please see the description of the setting to learn when it applies.

vscode don't have this setting


[ALLUXIO-3243] Optimize concatPath

https://alluxio.atlassian.net/browse/ALLUXIO-3243
This creates a StringBuilder of the correct size, so it does not have to be constantly resized.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/20524/
Test PASSed.

Jenkins, test this please

shall we also provide the version of concatPath with 3, 4,... args?

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/20535/
Test PASSed.

Jenkins, test this please

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/20541/
Test FAILed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/20543/
Test PASSed.

Merged build finished. Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/20546/
Test PASSed.


Fix rounding error in reserved token split

What does this PR do and why?
Original bug: Can't submit form when sum of percentages is 100%
Identified as being caused by a rounding error where the number would appear as:<PHONE_NUMBER>0000001.
The fix instead makes the calculation before converting to a percentage (in its permyriad) and converts after the fact, avoiding the rounding error.
Closes https://github.com/jbx-protocol/juice-interface/issues/1243.
Screenshots or screen recordings
At 99.99%

At 100.00%

At 100.01%

Acceptance checklist

[ ] I have evaluated the
Approval Guidelines
for this PR.
[ ] I have tested this PR in
all supported browsers.
[ ] I have tested this PR in dark mode and light mode (if applicable).

@wraeth-eth awesome! Thanks for a nice pr description too. lezzzz merge


add/verify support for schema import

SchemaImpl.kt includes some logic to support this.  At a minimum, tests are needed in the (likely) event that imports are not implemented fully/correctly.

The current implementation of the import has a mismatch with the schema. As seen here, the id of the import is being interpreted as a symbol, but the spec defines it as a string.
Where spec defines as string
Example that should work

Thanks @dgkimamzn.  Yes that's a bug, schema id should be a string as indicated by the spec.  You're welcome to submit a pull request to fix that if you'd like.

Bug described above (import schema id incorrectly being interpreted as a symbol) has been fixed in #41.


New package: FuzzyClusteringSimilarity v1.0.0-DEV

Registering package: FuzzyClusteringSimilarity
Repository: https://github.com/ryandewolfe33/FuzzyClusteringSimilarity.jl
Created by: @ryandewolfe33
Version: v1.0.0-DEV
Commit: 6eaec16c43d427b77ee12a3953157947de2f7287
Git reference: HEAD

This can be ignored/removed. I did not "simple retrigger Registrator" and made a new PR instead. https://github.com/JuliaRegistries/General/pull/93455.


npm updates badly needed

npm audit shows multiple dependencies out of date, with security issues
I am publishing an update from my fork with changes from #429 to resolve the security issues for an upstream project, would like to see this fixed on npmjs ASAP.
FYI I sent a request for owner access on NPM according to https://www.npmjs.com/policies/disputes, should be able to resolve this within the next month or so.

Thanks @mhevery for adding me, taking assignment now.

Just published 1.14.6 which resolves the npm audit issues.


Expandable/ExpandableGroup prototypes focus issue

The entire Expandable isn't highlighted when the element has the focus.
Depicted in the following image :

Technically the clickable area of the expandable is the area highlighted with the border, not the whole expandable. So when tabbing to the expandable the highlighted area shown is what would receive keyboard input, since that's the only area that responds to mouse clicks. I do agree it looks weird though. So does it matter if the border is not truly the active area, and instead is the whole expandable? Also, it seems like this highlighting should only appear on keyboard input, not on mouse clicks, right? cc @KimberlyMunoz

The issue is more that it looks off. It can be fixed by either adding focus to the entire expandable or fixing the bottom focus.

Moved to https://github.com/cfpb/capital-framework/issues/760


Implement Emscripten 3.1.47 additions

Implement emval iterator
Implement emval_new_array_from_memory_view
Fix bug in emval_get_global

Codecov Report

Merging #15 (85fec1a) into main (586bd31) will decrease coverage by 0.10%.
The diff coverage is 76.47%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@            Coverage Diff             @@
##             main      #15      +/-   ##
==========================================
- Coverage   72.46%   72.36%   -0.10%
==========================================
  Files          21       21
  Lines        4667     4712      +45
==========================================
+ Hits         3382     3410      +28
- Misses        905      922      +17
  Partials      380      380

Files
Coverage Δ

internal/emval.go
60.60% <76.47%> (+0.09%)
:arrow_up:


Update Opera configurations for ES6 requirements

This PR updates the configuration for some of the ES6 requirements that are still tagged as incompatible with Opera, although they have been available several Opera versions back.
The versions are very conservative version choices based on the compatibility table here.

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

@dlebech Thanks for the PR! Are you able to run our test suite against the minimum Opera version each polyfill is now no longer targeting and confirm the native features work as expected? You can do this by visiting https://polyfill.io/v2/docs/features/ and click the "Live test" radio button at the top of the table.

@JakeChampion Neat feature :smiley: I ran Opera 34 in a Windows 7 VM and I actually get "Native" on all the included features. I then installed version 32 which has feature parity with Chrome 45 that seems to have introduced most of Array features. Indeed, still native -- across the board.
I have updated the PR so all the minimum versions are 32. I think perhaps some of the other features were introduced earlier, but if I can avoid going hunting for exact versions, I would appreciate it :smiley:
What do you think?

@dlebech Thanks for doing that!


404 for css/js files after Installation

Gitea version (or commit ref): v1.12.2
Git version:
Operating system: Ubuntu 18.04
Database (use [x]):

[ ] PostgreSQL
[ x] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[ ] Not relevant

Log gist:

Description
I have used the installation of your documentation for the binary https://docs.gitea.io/en-us/install-from-binary/, but after this, each asset give me a 404.
Screenshots

How do you access the Gitea instance?

http://gitea.domain.tld:3000?
http://gitea.domain.tld with Nginx reverse proxy on http://localhost:3000 for example?
http://gitea.domain.tld with Nginx as reverse proxy on a Unix socket (HTTP_ADDR = /var/run/gitea/gitea.sock in app.ini)?

With a nginx reverse proxy. Try already some variants in the nginx configuration, but nothing changes.

With a nginx reverse proxy.

Did you follow either the official reverse proxy(es) guide (https://docs.gitea.io/en-us/reverse-proxies/) or a well-known good and working config example? I can drop a gist with mine if you need, I have both localhost:3000 config and Unix path, as I'm migrating the server from the first to the latter.

Both and i have gitea installed not the first time. I have build all assets from the source and serve them from another subdomain, that works. A gist would be great, maybe there is a hint what went wrong.

There you have it, both work fine in their deployment, feel free to ask for any snippet I included if it isn't clear what it does: https://gist.github.com/bryanpedini/0ffa556abd742078c27e813b4fc9f6f2

Check your STATIC_URL_PREFIX configuration, it's what most assets use.

I have, but the best solution was to build all assets and place them in a sub directory. I tried the configuration from @bryanpedini but, the problem was the same, 404 errors for all assets. Thanks for your help!

I tried the configuration from @bryanpedini but, the problem was the same, 404 errors for all assets.

Both of the provided configurations work fine for me in both of my completely different servers...
Tho, I don't either serve files from a different subdomain nor a different directory than the standard /static/ (I think) from the Gitea server itself, so I can't help in that way. Maybe just use the Gitea instance static serving from the Golang server, no?

Ah, no Problem :) I only wondering why it didn't work for me in "basic" Version. But now its even faster with the static files ;-)

I told ya'... use the Go server builtin static file serving... 😉


BUG: #10565 Series.name lost in rolling_* funcions

Addresses #10565

Looks good to me... anyone else?

Sure. I think @sinhrks has been closing a bunch of these losing names issues. Is there a master list somewhere?

@chris-b1 thanks!

@TomAugspurger There is no master tracker for Series.name, but only for Index.name (#9862). I'll change this to list both Series.name and Index.name.


Ruby based linters?

Can Ruby based linters be packaged and called?

@bobrocke I haven't even thought of this...  I am not much of a Ruby guys, so I can't tell you how well it would fit.  Any interest in a PR?

I'm not a Ruby guy either. Knowing the path to a gem, is there any reason this code base wouldn't be able to launch a ruby gem top be run by an installed Ruby interpreter??

Looks like it's been done before for sass-lint.


The results of Mean detector are consistent

ynthid_text_huggingface_integration.ipynb ,in part of Option 1: Mean detector,i got sample result.From the results, it cannot be distinguished whether there is a watermark or not.
Mean scores for watermarked responses:  [0.5001013  0.49999997 0.49628338 0.49961364]
Mean scores for unwatermarked responses:  [0.5001013  0.49999997 0.49628338 0.49961364]
Weighted Mean scores for watermarked responses:  [0.49701414 0.5016432  0.4982002  0.5014231 ]
Weighted Mean scores for unwatermarked responses:  [0.49701414 0.5016432  0.4982002  0.5014231 ]

When I add watermarks to the text or not, I set the same random number conditions and use different models, and the output is the same. From my understanding, I use synthid_mixin.SynthIDGemmaForCausalLM and transformers.GemmaForCausalLM. The results of the model should not be consistent
SynthIDGemmaForCausalLM
# Initialize a SynthID Text-enabled model.
model = synthid_mixin.SynthIDGemmaForCausalLM.from_pretrained(
    MODEL_NAME,
    device_map='auto',
    torch_dtype=torch.bfloat16,
)
# Prepare your inputs in the usual way.
inputs = tokenizer(
    INPUTS,
    return_tensors='pt',
    padding=True,
).to(DEVICE)
# Generate watermarked text.
outputs = model.generate(
    **inputs,
    do_sample=True,
    max_length=1024,
    temperature=TEMPERATURE,
    top_k=TOP_K,
    top_p=TOP_P,
)

GemmaForCausalLM
# Initialize a standard tokenizer from Transformers.
tokenizer = transformers.AutoTokenizer.from_pretrained(MODEL_NAME)
tokenizer.pad_token = tokenizer.eos_token
tokenizer.padding_side = "left"
# Initialize a GemmaForCausalLM model.
model = transformers. GemmaForCausalLM.from_pretrained(
    MODEL_NAME,
    device_map='auto',
    torch_dtype=torch.bfloat16,
)

inputs = tokenizer(
    INPUTS,
    return_tensors='pt',
    padding=True,
).to(DEVICE)
outputs = model.generate(
    **inputs,
    do_sample=True,
    max_length=1024,
    temperature=TEMPERATURE,
    top_k=TOP_K,
    top_p=TOP_P,
)

What I understand is that after adding the watermark, the output results should be slightly different, but judging from the results, the two are currently completely consistent.Can you provide a sample of the notebook's result set? Thank you.

@wrx1990 Couple of questions
Are you using this model
model = synthid_mixin.SynthIDGemmaForCausalLM.from_pretrained(
    MODEL_NAME,
    device_map='auto',
    torch_dtype=torch.bfloat16,
)

as watermarked model? The watermarked model needs to set up differently which you can walk through the colab implementation.

@sumedhghaisas2
first.
I run colab's code in the local environment, and the results in the Mean detector part are exactly the same. In theory, the results should be different between adding watermarks and not adding watermarks.
Mean scores for watermarked responses:  [0.5001013  0.49999997 0.49628338 0.49961364]
Mean scores for unwatermarked responses:  [0.5001013  0.49999997 0.49628338 0.49961364]
Weighted Mean scores for watermarked responses:  [0.49701414 0.5016432  0.4982002  0.5014231 ]
Weighted Mean scores for unwatermarked responses:  [0.49701414 0.5016432  0.4982002  0.5014231 ]

second,
In the Generate watermarked output  section, enable_watermarking is true or false, the output results are consistent.Can't see any difference in results with watermark。
Can you tell me how to run the program and get different results on the output with and without adding watermark?

I met the similar problem. I run the colab's  code in the local, and use the same instance and settings. However, when I run the experiments on the Gemma-2b-it, the results are as fellow.
Mean scores for watermarked responses:  [0.5109244  0.5097022  0.51533335 0.52014655]
Mean scores for unwatermarked responses:  [0.4982544  0.49496853 0.511985   0.48817205]
Weighted Mean scores for watermarked responses:  [0.5145333  0.51516587 0.5331452  0.5292408 ]
Weighted Mean scores for unwatermarked responses:  [0.49958172 0.49456924 0.5091847  0.4849563 ]

It seems that I can't classify the watermarked and unwatermarked responses.  Can you tell me if this is a normal result?
And how can I classify the watermarked or unwatermarked responses base on the scores?

@Kegard the scores look as intended. The (weighted) mean scores for unwatermarked text should be close to 0.5 and for watermarked text, slightly higher than 0.5. Here, looks like  weighted mean watermarked scores are between 0.514-0.533 and watermarked scores are between 0.484 - 0.509, so they're fully separated... longer lengths & higher temperatures will make the scores more separable. If you want more interpretable values, I would recommend turning them into p-values or z-scores as discussed in the paper.

@sdathath Thanks. It seems that I can set 0.5 as the thereshold to classify the watermarked text and the unwatermarked.  What's more, I find that outputs is used as tensor to compute the g_values. If I use a different tokenizer to convert the watermarked text to a tensor, will that affect the results?

I would recommend against using 0.5 directly as the threshold since you
would expect 50% of scores for unwatermarked samples to cross that
threshold. Would it be possible to maybe use z-scores?
On Fri, Dec 6, 2024, 1:13 PM Kegard @.***> wrote:

@sdathath https://github.com/sdathath Thanks. It seems that I can set
0.5 as the thereshold to classify the watermarked text and the
unwatermarked. What's more, I find that outputs is used as tensor to
compute the g_values. If I use a different tokenizer to convert the
watermarked text to a tensor, will that affect the results?
—
Reply to this email directly, view it on GitHub
https://github.com/google-deepmind/synthid-text/issues/7#issuecomment-2522391499,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BMJM3UZNZ4M7BRLJVAP6VYD2EFIQZAVCNFSM6AAAAABRKP4SSSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMRSGM4TCNBZHE
.
You are receiving this because you were mentioned.Message ID:
@.***>

"If I use a different tokenizer to convert the watermarked text to a tensor, will that affect the results?" -- with the current implementation, you would need to use the same tokenizer used to generate the text. There maybe ways around this requirement, but that would require redesigning a few components to look the contents of the tokens instead of the tokenizer ids.


New

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Browser console output
Console logs from your browser's developer tool console.
Terminal commands & output
Commands you used to start the app and the terminal output.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Browser console output
Console logs from your browser's developer tool console.
Terminal commands & output
Commands you used to start the app and the terminal output.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.


WANDB integration causes failures to import on an unsupporting venv

🐛 Bug
I get the following exception when importing  pytorch_lightning (python 3.9, latest version of PL):
Traceback (most recent call last):
  File "<frozen importlib._bootstrap>", line 1007, in _find_and_load
  File "<frozen importlib._bootstrap>", line 986, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 680, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 850, in exec_module
  File "<frozen importlib._bootstrap>", line 228, in _call_with_frames_removed
  File "/home/razor/anaconda3/envs/dev/lib/python3.9/site-packages/pytorch_lightning/__init__.py", line 30, in <module>
    from pytorch_lightning.callbacks import Callback  # noqa: E402
  File "/home/razor/anaconda3/envs/dev/lib/python3.9/site-packages/pytorch_lightning/callbacks/__init__.py", line 14, in <module>
    from pytorch_lightning.callbacks.base import Callback
  File "/home/razor/anaconda3/envs/dev/lib/python3.9/site-packages/pytorch_lightning/callbacks/base.py", line 25, in <module>
    from pytorch_lightning.utilities.types import STEP_OUTPUT
  File "/home/razor/anaconda3/envs/dev/lib/python3.9/site-packages/pytorch_lightning/utilities/__init__.py", line 18, in <module>
    from pytorch_lightning.utilities.apply_func import move_data_to_device  # noqa: F401
  File "/home/razor/anaconda3/envs/dev/lib/python3.9/site-packages/pytorch_lightning/utilities/apply_func.py", line 29, in <module>
    from pytorch_lightning.utilities.imports import _compare_version, _TORCHTEXT_LEGACY
  File "/home/razor/anaconda3/envs/dev/lib/python3.9/site-packages/pytorch_lightning/utilities/imports.py", line 121, in <module>
    _WANDB_GREATER_EQUAL_0_10_22 = _WANDB_AVAILABLE and _compare_version("wandb", operator.ge, "0.10.22")
  File "/home/razor/anaconda3/envs/dev/lib/python3.9/site-packages/pytorch_lightning/utilities/imports.py", line 79, in _compare_version
    pkg_version = Version(pkg_resources.get_distribution(package).version)
  File "/home/razor/anaconda3/envs/dev/lib/python3.9/site-packages/pkg_resources/__init__.py", line 477, in get_distribution
    dist = get_provider(dist)
  File "/home/razor/anaconda3/envs/dev/lib/python3.9/site-packages/pkg_resources/__init__.py", line 353, in get_provider
    return working_set.find(moduleOrReq) or require(str(moduleOrReq))[0]
  File "/home/razor/anaconda3/envs/dev/lib/python3.9/site-packages/pkg_resources/__init__.py", line 897, in require
    needed = self.resolve(parse_requirements(requirements))
  File "/home/razor/anaconda3/envs/dev/lib/python3.9/site-packages/pkg_resources/__init__.py", line 783, in resolve
    raise DistributionNotFound(req, requirers)
pkg_resources.DistributionNotFound: The 'wandb' distribution was not found and is required by the application

I can not tell you how to reproduce this error, but I can describe it however I'll manage. What happens is that both importlib.util.find_spec and importlib.import_module return a not none spec/module. When I try to load wandb in the console, it succeeds but the module is empty (has no attributes other than the most basic dunder ones). I don't have the package installed in my conda environment, or even on the python distribution I'm using. I did use it once in some projects with python 3.6 (I currently use a venv of 3.9). When I use importlib.util.find_spec it points to an auto-generated empty folder named wandb somewhere in the old project I mentioned. When I erased that folder the problem ceased.
To make a long story short, PLEASE, in the imports module, either catch
pkg_resources.DistributionNotFound to return False in the _compare_versions, or - better yet - group these availability and version checks in try: .. except: ... blocks according to the checked import. For example:
try:
     _WANDB_AVAILABLE = _package_available("wandb")
     _WANDB_GREATER_EQUAL_0_10_22 = _WANDB_AVAILABLE and _compare_version("wandb", operator.ge, "0.10.22")
     _WANDB_GREATER_EQUAL_0_12_10 = _WANDB_AVAILABLE and _compare_version("wandb", operator.ge, "0.12.10")
except:
     _WANDB_AVAILABLE = _WANDB_GREATER_EQUAL_0_10_22 = _WANDB_GREATER_EQUAL_0_12_10 = False

Environment

CUDA:
- GPU:
- NVIDIA GeForce GTX 1080 Ti
- NVIDIA GeForce GTX 1080 Ti
- NVIDIA GeForce GTX 1080 Ti
- available:         True
- version:           10.2
Packages:
- numpy:             1.23.1
- pyTorch_debug:     False
- pyTorch_version:   1.12.0+cu102
- pytorch-lightning: 1.6.5
- tqdm:              4.64.0
System:
- OS:                Linux
- architecture:
- 64bit
- ELF
- processor:         x86_64
- python:            3.9.12
- version:           #136~18.04.1-Ubuntu SMP Fri Jun 10 18:00:44 UTC 2022

Hello @erezinman
We are currently refactoring these imports to become lazy so that wandb and other optional dependencies only get imported when actually needed (#12786).
For wandb, we recently applied this in #13474. Could you try to install from master and try again?

@awaelchli Alas, I cannot because I deleted and reinstalled a bunch of stuff. However, that sounds like the right way to go, so you can surely close this issue. Kudos!

Alright, that's great. Ping us again if this problem surfaces again.