Tagged template literals compiled incorrectly, TemplateStringsArray not cached

TypeScript Version:  2.4.1
Code
let previousStrings;

function test(strings, ...value) {
  if (previousStrings !== undefined && previousStrings !== strings) {
    window.alert('failure');
  }
  previousStrings = strings;
}

function go(n) {
  return test`literal ${n}`;
}

Expected behavior:
According to the ECMAScript spec, every time a template tag is evaluated, the same strings object should be passed as the first argument:

Each TemplateLiteral in the program code of a realm is associated with a unique template object that is used in the evaluation of tagged Templates (<IP_ADDRESS>). The template objects are frozen and the same template object is used each time a specific tagged Template is evaluated.

https://tc39.github.io/ecma262/#sec-gettemplateobject
Actual behavior:
The above snippet compiles to:
var previousStrings;
function test(strings) {
    var value = [];
    for (var _i = 1; _i < arguments.length; _i++) {
        value[_i - 1] = arguments[_i];
    }
    if (previousStrings !== undefined && previousStrings !== strings) {
        window.alert('failure');
    }
    previousStrings = strings;
}
function go(n) {
    return (_a = ["literal ", ""], _a.raw = ["literal ", ""], test(_a, n));
    var _a;
}

You can see that _a is going to have a new identity for each call of go().
FWIW, Babel compiles this correctly:
'use strict';

var _templateObject = _taggedTemplateLiteral(['literal ', ''], ['literal ', '']);

function _taggedTemplateLiteral(strings, raw) { return Object.freeze(Object.defineProperties(strings, { raw: { value: Object.freeze(raw) } })); }

var previousStrings = void 0;

function test(strings) {
  if (previousStrings !== undefined && previousStrings !== strings) {
    window.alert('failure');
  }
  previousStrings = strings;
}

function go(n) {
  return test(_templateObject, n);
}

This is breaking lit-html when used in TypeScript: https://github.com/PolymerLabs/lit-html/issues/58

Is there a workaround for typescript to do this correctly or do we have to wait for a patch? For future facing JavaScript frameworks moving away from virtual-dom this is a bit of a problem :/

For now I'd rather wait on discussion on https://github.com/rwaldron/tc39-notes/blob/master/es8/2017-07/jul-27.md to get settled out first before we revisit the implementation.

@DanielRosenwasser whether or not equal literals from separate source locations remain identical might be up in the air, but this should be true:
let t = () => id``;

t() === t(); // true

That it isn't is a pretty critical bug that prevents using TemplateStringsArray as map key. Until this is fixed I need to tell my TypeScript users to target esnext, then compile with Babel.
The incremental fix is to hoist the TemplateStringsArray declaration to the top-level:
var _a = ["literal ", ""];
_a.raw = ["literal ", ""];

function go(n) {
    return test(_a, n);
}

for anyone using template literals or polymer you can use this feature detection
const t = () => ((s) => s)``;
const shouldCacheTemplates = t() === t();
const cacheTemplates = new Map<string, TemplateStringsArray>();

function tagged(strings: TemplateStringsArray, ...values: any[]) {
  if (shouldCacheTemplates) {
    let key = strings.join('{{typescriptProblems}}');
    let _strings = cacheTemplates.get(key)
    if (_strings === undefined) {
      cacheTemplates.set(key, strings)
    } else {
      strings = _strings
    }
  }

}

do you think a typescript plugin could be used as a workaround?
https://github.com/Microsoft/TypeScript/wiki/Writing-a-Language-Service-Plugin

@gdi2290 A plugin would need to change its behavior depending on the target. When the target is es2015+, there's no bug. Only when the target is <= ES5 is there a problem. That may be possible though, I haven't written a plugin yet.

@DanielRosenwasser is there an ETA for this bug to be fixed?
If not, is there a way from JS side to feature-detect TypeScript env is running?
I already fallback for old versions of FireFox that were incompatible with ES2015 standard, but being template literals feature-undetectable I need to use some ugly hack to know it's TypeScript and fallback with the solution I know it works already (per realm).
Thanks in advance for any advice.

@WebReflection Technically you can check whether the raw property is enumerable. TypeScript's is enumerable, whereas the one in ES2015 runtimes should be non-enumerable. But I hesitate from recommending you take a dependency on that.
I already have a prototype working. We're thinking about 2.6, but are concerned with trying to do the more standards-compliant thing but still coming short with respect to having globally cached template literal objects.

Thanks @DanielRosenwasser, but it's still undetectable to me, unless I put that check (even once) in the entry point of my logic.
It's a great hint, but it's too specific. What I really would like to know, is if there's any (hidden) global TypeScript leak I can detect upfront, like a typeof _classCreator === 'function' within my code, in case it get's "compiled" with TypeScript projects.

In lit-html we check for this bug by comparing the results of two calls to a tag:
const envCachesTemplates = ((t) => t() === t())(() => ((s) => s)``);

The expanded version is a little easier to read:
const t = () => ((s) => s)``;
const envCachesTemplates = t() === t();

@justinfagnani but that's more of a thing that client code needs to opt into, isn't it? In other words, wouldn't users of lit-html need to inform lit-html of the current behavior rather than lit-html discovering it on its own?

@justinfagnani you are using code that's going to be transpiled to address a bug. I cannot a single backtick string on my code because it runs on every browser already.

@DanielRosenwasser lit-html uses TypeScript already so it's inevitably trusting the transpiling env it's running itself. My case is different. I don't need/want/use transpilers at all, and even if the surrounding code uses transpilers, my library would be transpiler resistant and it will work there no matter what.
However, I need to figure out non-standard behaviors upfront, and using a transpiled feature to do that is not really an option because you can include my library as <script> resource right away, and it should be compatible with whatever env is there.
If I understand correctly, lit-html, will fallback with hacks for every browser once it's used transpiled because that check can be resolved only after TypeScript transpilation.

To add more content, hyperHTML can be embedded on any website now through unpkg.com/hyperhtml@latest. You cannot do the same with lit-html unless you are targeting already 100% compliant ES2015 browser (only WebKit these days?).
So lit-html trusts transpilation by default, hence it can do feature detections through the resulting code, but that's not possible for my library.
As summary, is there really nothing TypeScript leaks beside template literals?
I might go and find out myself :smile:

so, typeof __extends looks safe enough as assumption, isn't it?

so, typeof __extends looks safe enough as assumption, isn't it?

If you believe all of your users will be creating subclasses, yes, but that's probably not always the case.

is there anything that always leak ?

Found it. Please let me know when this bug is solved. Until then, I'll slightly penalize TypeScript transpiled code, degrading them like Firefox 54 or lower.
function isTypeScript() {
  var k, v;
  for (k in window) {
    if (k.charAt(0) === '_') {
      v = window[k];
      if (Array.isArray(v) && 'raw' in v) {
        return true;
      }
    }
  }
  return false;
}

actually I have doubts that would happen on the global scope if there's something like
(function(){}``);

I think the pkayground is ling to me ... thoughts? Thanks

@WebReflection
I think it's better to have a feature detection for your usecase rather than banning TypeScript and having to patch it in the future when it's fixed. This means detecting if the reference is the same.
const shouldCachesTemplates = ((t) => t() === t())(() => ((s) => s)``);

TypeScript might take 2+ years to fix this because their original template codegen wasn't up to spec (at the time). It's at a point where introducing a "fix" might be worse if the spec changed.
Here's an example of a long lived issue dependent on TC39 https://github.com/Microsoft/TypeScript/issues/16
The thing that's blocking the TC39 is something that hyperHTML and lit-html can benefit from. The edge case is when a WeakMap is used, rather than a Map, in order for the template string to be garbage collected. Right now if you use a WeakMap to store the template string it's kept in-memory and is never collected (kind of defeating the purpose of using WeakMap). For hyperHTML, this isn't an issue since the design is meant use the templates as a key. If hyperHTML were to allow certain templates to only wire once (they never rerender for example a footer render etc) then it would be better to release the template string from memory.
(I guess with that example you can just never store it given another codepath)
htmlOnce`yolo ${ }`

hyperHTML can benefit by using the same codepath html rather than creating a new one via htmlOnce

@gdi2290 which part of I cannot use template literals in my code wasn't clear?

@gdi2290 just in case it's stil not clear: I cannot feature detect template literals. This breaks my browsers:
//                                                      see this ? ↓↓
const shouldCachesTemplates = ((t) => t() === t())(() => ((s) => s)``);

@gdi2290 moreover

The thing that's blocking the TC39 is something that hyperHTML and lit-html can benefit from.

I'm looking forward to that, meanwhile I have production code to offer so I need to make it work.
I'm implementing right now the one-off hack suggested by @DanielRosenwasser which seems to be reliable and Babel proof.
Once again, hyperHTML can be used as <script src="hyperHTML"> without needing tooling or anything else and it's going to workdown to IE9. It's code cannot possibly feature detect backticks, but thanks for the rest of the thoughts.

@DanielRosenwasser since lit-html is distributed, for the moment, only as standard ES2015, the presumption with this part of the fix, is that if the app is running in ES5, the consumer code and lit-html were compiled with the same compiler. On the Polymer project we distribute all our code as ES2015 and let applications compile as needed. Internal to Google, all TypeScript code is compiled at app build time with the same version. This detection should also work for uncompiled code running on browsers with buggy template literal implementations.
It's true that this particular detection won't work if lit-html is either uncompiled or compiled with a spec-compliant compiler, and the client code is compiled with a buggy compiler. There are other checks we can do, like you suggested.
@WebReflection lit-html's use of TypeScript internally has absolutely nothing to with how most users will consume it. lit-html is distributed as standard ES2015, and we do not "trust" TypeScript's downlevel compilation because we don't use it.

@justinfagnani my point is what you wrote already to Daniel. lit-html trusts inevitably transpilations at its core, because it's written in TypeScript which is not a standard, and it's distributed as ES2015, which is 80% of the time transpiled these days.
hyperHTML is written in ES5, and that's about the end of the story. Any consumer, every browser on the planet.
This is a huge different in terms of runtime, transpiler potentially aware, features detection.
hyperHTML can be a script tag on top of any new and old project, it doesn't have tooling as core dependency for today web.
On top of that, I don't care much, really, I just want to make my library transpiler proof, hence I need to know what these "beasts" do with the JavaScript code they are targeting.

@justinfagnani when you get the chance, check out whether the branch at #18300 works well with lit-html.

Hi @DanielRosenwasser
FYI the detection hack you proposed has been implemented and published.
If TypeScript intend to set raw as not enumerable without fixing issues with template literals please do let me know. Thank You!


Build does not yield installable headers or linkable library

Ubuntu 18.04 / GCC 7.4 / Cmake 3.15.0
Perhaps I'm missing something, but I'm unable to find any installable headers or linkable modules from this SDK.
Is this purely a referential implementation? How does one integrate this SDK into an existing project?

Hi @snarkeystarkey ,
After running min-install-script/install-script successfully, you should be able to find dynamic libraries (libcproducer.so, libproducer.so, libgstkvssink.so). You can use those depending on what function you need.

I have found those libraries, yes - however I'm unable to locate the corresponding headers. Attempting to include headers in my project yields a lot of errors and path issues. I'm able to make this work if I combine most of the "com" includes in my system include path, as well as pull a few from various locations in this project. Is there something I'm not understanding regarding the headers?

Hi @snarkeystarkey ,
We will try to improve the install path so header files and libraries would be easier to use. For now you need to find the header files. What you were doing is correct right now.
If you are using libcproducer then you will just need header files from https://github.com/awslabs/amazon-kinesis-video-streams-producer-sdk-cpp/tree/master/kinesis-video-c-producer/src/include
If you are using libproducer then you will need header files from https://github.com/awslabs/amazon-kinesis-video-streams-producer-sdk-cpp/tree/master/kinesis-video-producer/src
If you are using libgstkvssink then you will need header files from https://github.com/awslabs/amazon-kinesis-video-streams-producer-sdk-cpp/tree/master/kinesis-video-gstreamer-plugin/src

OK! Now we're talking --- THANK YOU!

Closing this issue.


Viewer: viewing controls

Who it's for
Viewer users - to control my viewing area
Problem to be solved
Need to provide ways of controlling the focussed view of a work by

zooming in and out to see detail
switching into a 2-page view so researcher can see any marginalia that bridges both pages
rotating a single page to make it easier to read content printed in different orientations

This functionality is feasible with OpenSeadragon
CAVEAT about 2-page view: May not be wanted with non-book-shaped materials, e.g archives, loose manuscripts, anything printed but not-bound. Consider how configurable the viewer needs to be based on material type.
Components

[x] Zoom in/out
[x] Rotate 90°
[ ] Single/double page toggle

TODO

[ ] Event tracking on components
[x] Check page view event being sent
[ ] Check download tracking being sent
[ ] Add tracking to detail view/view all toggle
[ ] Put tracking on all viewer controls (need for A/B testing on arrow controls)
[x] Agree on visual design

Acceptance criteria

User can initiate zoom in/out
User can rotate an image in 90° increments
User can switch between a single page and a double-page view
Double-page view must reflect book opening (Left and Right page, not Front and Back)

Success?
TBD

#4325

@jennpb During the viewer pop-up research, the majority of users prefer to see 2-page views for books or manuscripts.

provide much-needed context e.g. how it relates, discover any additional info
clear understanding of the material type
less click and visually less overwhelming

Based on those findings, we recommended displaying 2-page view in both list and detail views by default for relevant material types.
The current implementation display a single page view. Should I create a separate ticket as an enhancement?

@Heesoomoon @pollecuttn Yes please, make the 2-page view a new ticket with label:improvement
There is some back-end logic we need to sort out to determine what gets displayed as a 2-page view


Bitcoin pri key to address problem?

i dont think bitcoin prikey to address dont true work
Because
i import in blockchain wallet private key
i see another address
But this code generate another addresd

If you'll have problems with correct generation of wallet, feel free to share your initial private key, I'll look into it.
Also, you can cross validate wallet by using https://www.bitaddress.org ("Wallet Details" tab).


Virtual concierge page

Description
VCON-225
Description of the task you completed and/or features added. If it is a visual component or feature,
add a screenshot(s) of it.
Added a folder for the projects
added a folder for virtual concierge
added an index page for the virtual concierge
added tests for the virtual concierge page and component
added images for the VC
added strings and placeholders for the translations.
Acceptance Criteria
Get the acceptance criteria for the story and/or task you are working on and put it all here.
Test Instructions
Help Requested
There seems to be an accessibility issue on the page but it appears that it is with other items on the page and not what I added and I am not sure where it is coming from exactly.
Checklist

[ ] Strings use placeholders for translation (No hard coded strings)
[ ] Unit tests have been added/updated

Product and Sprint Backlog
Trello

I didn't have push access so had to create a separate PR #216 with the images for the VA


Model name 'nboost/pt-tinybert-msmarco' was not found in model name list

I'm trying to follow the example but following either the pip (or docker routes) I get:
nboost                                      --uhost localhost
--uport 9200
 --search_route "/<index>/_search"
 --query_path url.query.q
 --topk_path url.query.size
  --default_topk 10
 --choices_path body.hits.hits
--cvalues_path _source.passage

Model name 'nboost/pt-tinybert-msmarco' was not found in model name list.  This looks similar in some ways to #58 .  Any suggestions?

When installing nboost using pip install nboost[pt], for some reason it installs an older version of transformers package (2.2.1). Upgrading that package using the following command solved the issue for me:

pip install --upgrade transformers

Thanks @kaykanloo, that worked perfectly


Is not a valid endorsement system chaincode

I'm getting the following error when running node main.js -c config-fabric.json -n fabric.json in /benchmark/simple. I want to test my Hyperledger chaincode, currently, I've not changed anything. I've run npm install and installed the required packages for Fabric: npm install<EMAIL_ADDRESS>fabric-ca-client fabric-client.
# create mychannel......
ok 1 created mychannel successfully
# Sleep 5s......
# Join channel......
# join mychannel
ok 2 Successfully joined mychannel
# install all chaincodes......
ok 3 Installed chaincode simple successfully in all peers
# Instantiate chaincode......
error: [client-utils.js]: sendPeersProposal - Promise is rejected: Error: 2 UNKNOWN: chaincode error (status: 500, message:  is not a valid endorsement system chaincode)
    at new createStatusError (/home/michiel/learning/hlf/caliper-benchmark/node_modules/fabric-client/node_modules/grpc/src/client.js:64:15)
    at /home/michiel/learning/hlf/caliper-benchmark/node_modules/fabric-client/node_modules/grpc/src/client.js:583:15
error: [client-utils.js]: sendPeersProposal - Promise is rejected: Error: 2 UNKNOWN: chaincode error (status: 500, message:  is not a valid endorsement system chaincode)
    at new createStatusError (/home/michiel/learning/hlf/caliper-benchmark/node_modules/fabric-client/node_modules/grpc/src/client.js:64:15)
    at /home/michiel/learning/hlf/caliper-benchmark/node_modules/fabric-client/node_modules/grpc/src/client.js:583:15
error: [client-utils.js]: sendPeersProposal - Promise is rejected: Error: 2 UNKNOWN: chaincode error (status: 500, message:  is not a valid endorsement system chaincode)
    at new createStatusError (/home/michiel/learning/hlf/caliper-benchmark/node_modules/fabric-client/node_modules/grpc/src/client.js:64:15)
    at /home/michiel/learning/hlf/caliper-benchmark/node_modules/fabric-client/node_modules/grpc/src/client.js:583:15
error: [client-utils.js]: sendPeersProposal - Promise is rejected: Error: 2 UNKNOWN: chaincode error (status: 500, message:  is not a valid endorsement system chaincode)
    at new createStatusError (/home/michiel/learning/hlf/caliper-benchmark/node_modules/fabric-client/node_modules/grpc/src/client.js:64:15)
    at /home/michiel/learning/hlf/caliper-benchmark/node_modules/fabric-client/node_modules/grpc/src/client.js:583:15
not ok 4 Failed to instantiate chaincodes, Error: Failed to send instantiate due to error: Error: Failed to send instantiate Proposal or receive valid response. Response null or status is not 200. exiting... at Client.newDefaultKeyValueStore.then.then.then.then (/home/michiel/learning/hlf/caliper-benchmark/src/fabric/e2eUtils.js:372:19) at <anonymous> at Client.newDefaultKeyValueStore.then.then.then.then.then (/home/michiel/learning/hlf/caliper-benchmark/src/fabric/e2eUtils.js:385:15) at <anonymous>
  ---
    operator: fail
    at: chaincodes.reduce.then.catch (/home/michiel/learning/hlf/caliper-benchmark/src/fabric/instantiate-chaincode.js:60:19)
    stack: |-
      Error: Failed to instantiate chaincodes, Error: Failed to send instantiate due to error: Error: Failed to send instantiate Proposal or receive valid response. Response null or status is not 200. exiting...
          at Client.newDefaultKeyValueStore.then.then.then.then (/home/michiel/learning/hlf/caliper-benchmark/src/fabric/e2eUtils.js:372:19)
          at <anonymous>
          at Client.newDefaultKeyValueStore.then.then.then.then.then (/home/michiel/learning/hlf/caliper-benchmark/src/fabric/e2eUtils.js:385:15)
          at <anonymous>
          at Test.assert [as _assert] (/home/michiel/learning/hlf/caliper-benchmark/node_modules/tape/lib/test.js:224:54)
          at Test.bound [as _assert] (/home/michiel/learning/hlf/caliper-benchmark/node_modules/tape/lib/test.js:76:32)
          at Test.fail (/home/michiel/learning/hlf/caliper-benchmark/node_modules/tape/lib/test.js:317:10)
          at Test.bound [as fail] (/home/michiel/learning/hlf/caliper-benchmark/node_modules/tape/lib/test.js:76:32)
          at chaincodes.reduce.then.catch (/home/michiel/learning/hlf/caliper-benchmark/src/fabric/instantiate-chaincode.js:60:19)
          at <anonymous>

Yes, have the same problem. Spend a lot of time, but not resolve this issue

It may be a sdk compatibility issue. Please install fabric sdk v1.1.0 and retry the test. For example, npm install<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

@haojun Yes that solves the issue, accurate answer!

Another solution is to simply update the network to use containers matching the client/sdk version you are trying to test.
If you're running 1.2.0 for example, just go into network/fabric/simplenetwork/docker-compose.yml, and change all the image references from hyperledger/fabric-peer:x86_64-1.1.0 to hyperledger/fabric-peer:1.2.0.

create mychannel......
ok 1 created mychannel successfully
Sleep 5s......
Join channel......
join mychannel
ok 2 Successfully joined mychannel
install all chaincodes......
info: [packager/Golang.js]: packaging GOLANG from contract/fabric/simple/go
info: [packager/Golang.js]: packaging GOLANG from contract/fabric/simple/go
ok 3 Installed chaincode simple successfully in all peers
Instantiate chaincode......
error: [Peer.js]: sendProposal - timed out after:120000
error: [Peer.js]: sendProposal - timed out after:120000
error: [Peer.js]: sendProposal - timed out after:120000
error: [client-utils.js]: sendPeersProposal - Promise is rejected: Error: REQUEST_TIMEOUT
at Timeout._onTimeout (/root/node_modules/fabric-client/lib/Peer.js:124:19)
at ontimeout (timers.js:498:11)
at tryOnTimeout (timers.js:323:5)
at Timer.listOnTimeout (timers.js:290:5)
error: [client-utils.js]: sendPeersProposal - Promise is rejected: Error: REQUEST_TIMEOUT
at Timeout._onTimeout (/root/node_modules/fabric-client/lib/Peer.js:124:19)
at ontimeout (timers.js:498:11)
at tryOnTimeout (timers.js:323:5)
at Timer.listOnTimeout (timers.js:290:5)
error: [client-utils.js]: sendPeersProposal - Promise is rejected: Error: REQUEST_TIMEOUT
at Timeout._onTimeout (/root/node_modules/fabric-client/lib/Peer.js:124:19)
at ontimeout (timers.js:498:11)
at tryOnTimeout (timers.js:323:5)
at Timer.listOnTimeout (timers.js:290:5)
not ok 4 Failed to instantiate chaincodes, Error: Failed to send instantiate due to error: Error: Failed to send instantiate Proposal or receive valid response. Response null or status is not 200. exiting... at Client.newDefaultKeyValueStore.then.then.then.then (/root/caliper/src/fabric/e2eUtils.js:372:19) at  at process._tickCallback (internal/process/next_tick.js:188:7) at Client.newDefaultKeyValueStore.then.then.then.then.then (/root/caliper/src/fabric/e2eUtils.js:385:15) at  at process._tickCallback (internal/process/next_tick.js:188:7)
operator: fail
at: chaincodes.reduce.then.catch (/root/caliper/src/fabric/instantiate-chaincode.js:60:19)
stack: |-
  Error: Failed to instantiate chaincodes, Error: Failed to send instantiate due to error: Error: Failed to send instantiate Proposal or receive valid response. Response null or status is not 200. exiting...
      at Client.newDefaultKeyValueStore.then.then.then.then (/root/caliper/src/fabric/e2eUtils.js:372:19)
      at <anonymous>
      at process._tickCallback (internal/process/next_tick.js:188:7)
      at Client.newDefaultKeyValueStore.then.then.then.then.then (/root/caliper/src/fabric/e2eUtils.js:385:15)
      at <anonymous>
      at process._tickCallback (internal/process/next_tick.js:188:7)
      at Test.assert [as _assert] (/root/caliper/node_modules/tape/lib/test.js:224:54)
      at Test.bound [as _assert] (/root/caliper/node_modules/tape/lib/test.js:76:32)
      at Test.fail (/root/caliper/node_modules/tape/lib/test.js:317:10)
      at Test.bound [as fail] (/root/caliper/node_modules/tape/lib/test.js:76:32)
      at chaincodes.reduce.then.catch (/root/caliper/src/fabric/instantiate-chaincode.js:60:19)
      at <anonymous>
      at process._tickCallback (internal/process/next_tick.js:188:7)

...
fabric.installSmartContract() failed, Error: Fabric: instantiate chaincodes failed
at chaincodes.reduce.then.catch (/root/caliper/src/fabric/instantiate-chaincode.js:61:31)
at
at process._tickCallback (internal/process/next_tick.js:188:7)
[Transaction Info] - Submitted: 0 Succ: 0 Fail:0 Unfinished:0
unexpected error, Error: Fabric: instantiate chaincodes failed
at chaincodes.reduce.then.catch (/root/caliper/src/fabric/instantiate-chaincode.js:61:31)
at
at process._tickCallback (internal/process/next_tick.js:188:7)
docker-compose -f network/fabric/simplenetwork/docker-compose.yaml down;docker rm $(docker ps -aq);docker rmi $(docker images dev* -q)
Stopping peer1.org2.example.com ... done
Stopping peer0.org1.example.com ... done
Stopping peer1.org1.example.com ... done
Stopping peer0.org2.example.com ... done
Stopping orderer.example.com ... done
Stopping ca_peerOrg1 ... done
Stopping ca_peerOrg2 ... done
Stopping simplenetwork_ca_1 ... done
Removing peer1.org2.example.com ... done
Removing peer0.org1.example.com ... done
Removing peer1.org1.example.com ... done
Removing peer0.org2.example.com ... done
Removing simplenetwork_peer_1 ... done
Removing orderer.example.com ... done
Removing ca_peerOrg1 ... done
Removing ca_peerOrg2 ... done
Removing simplenetwork_ca_1 ... done
Removing network simplenetwork_default
daf7fbeb1870
a72b28b6aa82
89feb2493ff3
d7da8028f71c
b7982b562bc5
4d85531298cd
Untagged: dev-peer1.org1.example.com-simple-v0-2d56163f24dc127bcb48cf2dcdc1ac529d9164d868f9bbc2ce823e3a047dbc31:latest
Deleted: sha256:b77c47b1bdfbb8b19a315527fea24b02bd582e1e089e0280f874a3402fb1a6c2
Deleted: sha256:4abed7d2d900933b343859cbc3f41fb84857b3ecef5fd4aadef831b32d3caf7a
Deleted: sha256:b63255851f5b3687721c048c9f09feedcc8eef4a59170f8547b3842300afceb7
Deleted: sha256:e82bf183da6029666b5640329819e851a56270dbb3e021faa0d511a501832ae5
Untagged: dev-peer1.org2.example.com-simple-v0-7d068bf14ab99f7f01c1d580a3480868301e570c8b524a32918f99e2e241aa0a:latest
Deleted: sha256:65251cf8ff83b03094a66b8d8b85fb7aa72e076398f3cf58d9cf3c515849d6c7
Deleted: sha256:98663d2c17192cab196faa854d57111777247050d96a4f0e4084170cf7d4e6a8
Deleted: sha256:d2d17d15c46dde65f6f76b64cb75c4ba4de4762620c7277256e2d717b585a627
Deleted: sha256:3e5892ddce3d9b0fb519244e86aed61b34ec2dd51ececd40540d857a0d4f71dc
Untagged: dev-peer0.org2.example.com-simple-v0-7ebccf7a7fd05d8ac0a55ea61f1f0397ca89de629cef5f537c0950ecf302d423:latest
Deleted: sha256:0e3e42d273f1c55c6130f5c20d3fd2a78b68c39afa6ad5f4dd99f52c435a624a
Deleted: sha256:4afa17b1ace190886771fc30b3dec306eba251c812f9f9d96a5c1211da07c5e0
Deleted: sha256:b85a485702b6557cc8cbd2c7189f7bddb00d4448452f00a8d4f109b2c165d50d
Deleted: sha256:7405b6be5a383d766ab714dfc3d375fb822055c5bb88b40972bb81f5af52301f
1..4
tests 4
pass  3
fail  1
Benchmark return code:  1
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>e2e: node ./scripts/test.js "simple" "./benchmark/simple/config.json" "./benchmark/simple/fabric.json"
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>e2e script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!  Test failed.  See above for more details.

i have installed golang,but still reported this mistake, which one step wrong ? please help solve i

Is there any workaround for this issue when using IBM images instead of hyperledger's?
I need to use fabric sdk 1.2.x and it was working fine with hyperledger images, now I'm trying the same with IBM's and can't get past this error.

I received the following error while trying to run sample benchmark in hyperledger caliper
cse@system:~$ sudo npx caliper benchmark run --caliper-workspace ./ --caliper-benchconfig ~/caliper/packages/caliper-samples/benchmark/drm/config.yaml --caliper-networkconfig ~/caliper/packages/caliper-samples/network/fabric-v1.4.1/2org1peergoleveldb/fabric-go.yaml
Benchmark for target Blockchain type fabric about to start
2019.11.29-16:28:51.888 info  [caliper] [caliper-flow] 	####### Caliper Test #######
2019.11.29-16:28:51.890 info  [caliper] [local-observer] 	Observer interval set to 1 seconds
2019.11.29-16:28:51.891 info  [caliper] [caliper-utils] 	Executing command: cd /home/cse;docker-compose -f caliper/packages/caliper-samples/network/fabric-v1.4.1/2org1peergoleveldb/docker-compose.yCreating orderer.example.com ... done
Creating network "2org1peergoleveldb_default" with the default driver
Creating peer0.org1.example.com ... done
Creating ca.org2.example.com ...
Creating ca.org1.example.com ...
Creating peer0.org2.example.com ...
 This might take some time...
2019.11.29-16:29:10.463 info  [caliper] [adapters/fabric] 	Successfully instantiated marbles@v0 in mychannel
2019.11.29-16:29:10.466 info  [caliper] [adapters/fabric] 	Instantiating drm@v0 in mychannel. This might take some time...
2019.11.29-16:29:16.470 error [caliper] [caliper-flow] 	Error: Error: Invalid endorsement for drm@v0 in mychannel from peer0.org2.example.com: chaincode registration failed: container exited with 0
at proposalResponses.forEach (/home/cse/node_modules/@hyperledger/caliper-fabric/lib/fabric.js:1358:31)
at Array.forEach ()
at Fabric._instantiateChaincodes (/home/cse/node_modules/@hyperledger/caliper-fabric/lib/fabric.js:1354:35)
at
2019.11.29-16:29:17.473 info  [caliper] [local-observer] 	[undefined Round undefined Transaction Info] - Submitted: 0 Succ: 0 Fail:0 Unfinished:0
2019.11.29-16:29:17.474 info  [caliper] [caliper-utils] 	Executing command: cd /home/cse;docker-compose -f caliper/packages/caliper-samples/network/fabric-v1.4.1/2org1peergoleveldb/docker-compose.yaml down;(test -z "$(docker ps -aq)") || docker rm $(docker ps -aq);(test -z "$(docker images dev* -q)") || docker rmi $(docker images dev* -q);rm -rf /tmp/hfc-*
Stopping peer0.org1.example.com ... done
Stopping peer0.org2.example.com ... done
Stopping ca.org1.example.com    ... done
Stopping orderer.example.com    ... done
Stopping ca.org2.example.com    ... done
Removing peer0.org1.example.com ... done
Removing peer0.org2.example.com ... done
Removing ca.org1.example.com    ... done
Removing orderer.example.com    ... done
Removing ca.org2.example.com    ... done
Removing network 2org1peergoleveldb_default
/bin/sh: 1: test: "4a8ff37c4fd1: unexpected operator
4a8ff37c4fd1
6aa8f6f7d17c
5d78f3f841f6
docker: "images" requires a maximum of 1 argument.
See 'docker images --help'.
Usage:	docker images [OPTIONS] [REPOSITORY[:TAG]]
List images
docker: "images" requires a maximum of 1 argument.
See 'docker images --help'.
Usage:	docker images [OPTIONS] [REPOSITORY[:TAG]]
List images
docker: "rmi" requires a minimum of 1 argument.
See 'docker rmi --help'.
Usage:	docker rmi [OPTIONS] IMAGE [IMAGE...]
Remove one or more images
Benchmark failure
Error: Benchmark failure
at Function.handler (/home/cse/node_modules/@hyperledger/caliper-cli/lib/benchmark/lib/runBenchmark.js:70:23)
at
at process._tickCallback (internal/process/next_tick.js:189:7)
Command failed

@PADMAARUN Check the peer logs to see why the chaincode registration has failed. Caliper (as a Fabric client) only receives a high-level error message


Replace sponsors design on front page

Feature
Currently, the sponsors section looks like this;

What should your feature do?
This takes up a lot of space. I'd like to open this ticket to consider either decreasing the white space between the sponsor logos and the text by 50%, or to look into a carousel-type design.

Currently, the sponsors are jpgs with a white background. I think a good way to accomplish this task - and be able to use the images anywhere on the website - would be to convert the images to .pngs and then decreasing the padding between the flex container's items. BONUS: This makes the links clickable only where the logo exists, whereas I can currently click on white space.
I also wanted to implement a method of highlighting images like in the "Proudly Sponsored By" section of Vue.js's website.
I do think this is a relatively low priority item.
If anybody from team design like @Wimsy113 or @dmarchante has the time to convert the .jpgs to .pngs - that would be a good start.

Those are all transparent .png's, the ones with filenames like *_border.png are 600x600 like the existing ones. They can be cleaned up a little so they have standard sizes even with the border but these ones should fit for now.

Also, I'm just jumping back in after an unexpected break from OC, but once I update my repo I will create the hover effect that @kylemh mentioned - will attempt with a hover state that toggles css filter: grayscale(100%); and a transition.

I think doing non-border PNGs will make the logo section look unkempt, but now that you've made them all into PNGs, you can just crop off like half of the white space (in a uniform manner) and I believe you will achieve what Conrad wants.

what about using space-between instead of space-around? that will set each of them along the edges when they're 2x2 and then they can all just have a uniform height instead of having to create each one with the same amount of whitespace

I think at the time that I used justify-content: space-between; as a way to deal with the pain that used to be the <Section> component. Feel free to change up any local stylesheets to get the desired affect!

Ok will do, also I updated my comment as you were typing - check it out :P

IMO it's better to do layout stuff with all CSS instead of doing it in the image, because the sponsor list will be changing and it's a lot easier to drop in a cropped logo than to create a new image with a required amount of padding in px.

I actually disagree with this. We could end up with logos of varying widths and heights that don't line up properly on the y- or the x-axis or both, with a responsive website. Imo, it's better to make sure that every image has the same view-size to prevent hacky CSS compensations for an expanding list. It's also really easy to add a set amount of padding - just crop the image to the border, and then expand the canvas size.

I think I have a way to solve both of our concerns, I'll build it out and update. It's flexbox stuff ive never used before so im not sure but if it doesn't work out I can always just crop the ones I uploaded :P

If it's flexbox you fear, have fun with flexboxfroggy.com 💃  🐸

Is that a little more space efficient? Its not as tight as the vuejs one but still takes up less space. I think a carousel or different approach might be the way to go if more and more sponsors get added, but I can make a PR for this one if it works in the meantime.

This is good! I think eventually we will want to go to a carousel, and probably add segmentation to the sponsors displayed based on origin - @mattmacneil, this should be good to submit!

Coming in late on this, but I just wanted to say good job @mattmacneil  - that's exactly what I was imagining/hoping for.


SQL error [1305] [42000]

Description
I can't save and use it when I connect to mysql and create procedure and function ( show SQL error [1305] [42000]). It works fine when I use other tools and Command line client.
DBeaver Version
DBeaver Community 22.2.2
Operating System
windows
Database and driver
MySQL8.0
Steps to reproduce
No response
Additional context
No response

Please, describe steps to reproduce. How exactly are you trying to create procedure and function?

I can't save and use it when I connect to MySQL and create procedure and function ( show SQL error [1305] [42000]). It works fine when I use other tools like navicat and Command line client. The attached pictures show the situation I encountered.
------------------ 原始邮件 ------------------
发件人:                                                                                "dbeaver/dbeaver"                                                                                @.>;
发送时间: 2022年10月22日(星期六) 上午6:00
@.>;
<EMAIL_ADDRESS>主题: Re: [dbeaver/dbeaver] SQL error [1305] [42000] (Issue #17994)
Please, describe steps to reproduce. How exactly are you trying to create procedure and function?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

I don't see attached images. Please, send them via github, because attached to reply email files do not appear here.

OK.
------------------ 原始邮件 ------------------
发件人:                                                                                "dbeaver/dbeaver"                                                                                @.>;
发送时间: 2022年10月22日(星期六) 下午2:38
@.>;
<EMAIL_ADDRESS>主题: Re: [dbeaver/dbeaver] SQL error [1305] [42000] (Issue #17994)
I don't see attached images. Please, send them via github, because attached to reply email files do not appear here.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

The procedure was not created successful, because DBeaver consider blank line as a statement delimiter by default. You can turn off this setting here (see Blank line as a statement delimiter)

After I turn this off, it still can't work.

How do you execute your script? Which button or hotkey do you use?

This one. When I execute the SQL, it shows 'Updated Rows -1'.

Where is your cursor at the moment when you run this button? Maybe you accidentally run another query instead of create procedure, because your script works for me.
Please, try using "Execute script" button (Alt + X hotkey) or highlight the whole statement and press the button which you are trying to use. I hope it should help.

Mine is the community version.

This still doesn't work.

Thank you for bug report.
As a workaround, please, select the query part this way and press Execute. Than DBeaver will send the selected part of text to the database.

DELIMITER command works only with Execute script button. We need to make it work with Execute statement too.

DELIMITER command works only with Execute script button. We need to make it work with Execute statement too.

Hello,
has this been implemented yet?
I'm having the same issue.

Hi, @MattAnctil . It hasn't been implemented yet.
You can vote for the issue with 👍 under the issue description to promote it to a higher priority.


Fix Structure Service for properties

This fixes https://github.com/Microsoft/visualfsharp/issues/2190

nice one

:)
Thanks @vasily-kirichenko.

I've also fixed https://github.com/Microsoft/visualfsharp/issues/2191

Thank you thank you - I don't think there are any more issues with block structure/guidelines anymore. Hopefully these can be merged into RTM.

Hm, I think we do guidelines fundamentally in a wrong way. They should mark indentations, not syntactic blocks. I mean it should look like this:

(where red rectangles are collapsing [+])
It's completely different way to do outlining and guidelines and it'd basically require rewriting all the AST traversing code.

@vasily-kirichenko That's an interesting point.
My take is that we should ultimately offer two ways - indentation-based and syntax-based, and then offer the ability to toggle one or the other (or have all structure service turned off).  The default is something we could think about some more, and base off of feedback from what we're shipping at release.  Thoughts?

@cartermp I agree.

Personally I've never seen the appeal of indentation guidelines - you already know how far you're indented by looking at the current line. I certainly don't think we should be disabling what we've got for release - I've found being able to collapse blocks really useful and it's a feature I've really missed until VS 2017.

I think it has good educational value for new users who aren't used to whitespace significant scoping. Also - VFPT has had collapse blocks functionality for ages now.

Thank you


Use flowview in pager

Can we use flowview in view pager, i want same dragging effect in view pager page slide, but cannot achieve that. Can you please help.

it's not support view pager page slide . Sorry about that .


Treat invokeinterface bridges similarly to invokespecial

In both cases we need to avoid using invokesuper. For interface bridges,
using invokesuper will fail since the method being bridged to is in a
superinterface, not a superclass.
Starting in Java 8, javac emits default bridge methods in interfaces,
which use invokeinterface to bridge to the target method.
Without this fix, the included test case fails with:
Caused by: java.lang.NoSuchMethodError: java.lang.Object.f(Ljava/lang/Object;)V
    at B$$EnhancerByCGLIB$$7aa3e8e0.f(<generated>)
    ... 28 more

can you clarify this to explain why we need to find invokeinterface (basically the description from the PR)?

done

could you maybe add a variant of this that's based off the classes as compiled from source

The tests are compiled with -source 6 -target 6, and the test case relies on Java 8 features. I'm not sure how to add a variant that uses a newer language level?

and optionally keep this but bail if < java8?

done

any idea how/why this only became a problem after 942ab9e?

I'm not sure it was cauesd by 942ab9e. I ran into this trying to use a newer version of cglib with mockito 1.x, which is currently using cglib 2.2.: https://github.com/mockito/mockito/commit/b6b13e8b28d163f989e4f95c941a3da5c619d770

I think force pushing confused it. I started a new PR: #142


Add support for scripts-dev

Composer plugin: https://github.com/neronmoon/scriptsd
Result:

I am not convinced to support for schema provided by external plugins. According to official composer json schema, package definition does not allow extra properties. When you run composer validate, you get an error.

So, wont fix?

I think about new "quick fix" (alt+enter) called "Allow x property to be here" or something like that, so errors caused by custom properties would be able to ignore. What do you think about this solution?

Of course I would like full support of composer's plugins on the API level. But in any case, it's better than nothing. Thanks

It is hard to add support for schemas provided by custom plugins, because this is not standardised by composer api (probably even more - composer intentionally does not support this in clean way, because this approach is deprecated). If the plugins defined additional schema that they provide, it would be possible to fetch extra schema file and in runtime merge it with standard composer schema. Adding support for plugins one by one is not solution for me.
I can see also maybe better solution, define custom additional schema in project settings, something like "Type provider" in symfony 2 plugin (http://symfony2-plugin.espend.de/extension/signature_type.html#settings), so everyone would be able to add custom thing, ignore additional properties etc.


appJsPublicFile value suggestion for Laravel 9 \ Vite

This appears to be working with Laravel 9, but do you have any suggestions what that value should for appjsPublicFile and appcssPublicFile since we should be using Laravel Vite instead of Laravel Mix going forward?
My CSS and JS files are bundled here now:
/public/build/assets/app-238453fd.css
/public/build/assets/app-d35ea772.js
    'enableAppCss'                  => true,
    'appCssPublicFile'              => 'css/app.css',

    'enableBootstrapJsCdn'          => true,
    'bootstrapJsCdn'                => 'https://maxcdn.bootstrapcdn.com/bootstrap/4.0.0/js/bootstrap.min.js',

    'enableAppJs'                   => true,
    'appJsPublicFile'               => 'js/app.js',

Also, composer says this dependency laravelcollective/html is abandoned and should avoid using it. They suggest using spatie/laravel-html instead.
Can this be updated too?

I accept PR's.
Thank you


Written a docopt grammar, interested ?

Not seeing any references to an actual docopt formal grammar yet, I am writing a PEG grammar for use in Python with Tatsu (https://tatsu.readthedocs.io).  I am hopeful that it will be usable with other PEG parser generators but am not well enough versed in the area to know by how much they differ in syntax.
I would like to work with others interested in developing a formal grammar for docopt.  I'm not fixed on Tatsu or even PEG though I suspect PEG grammars will provide more accurate results for the corner cases.
I appended a sketch of the steps I believe necessary to develop the grammar.  I've written a grammar for the usage examples section.  Now I need to take a step back and write the frame work to snag each major section.  Divide and conquer seemed like the best approach.  In a day or so, I could post the grammar and test cases to github.
So, any information in this area or would anyone like to collaborate ?
Philip

Unix Utility Usage Grammar
Or, a formal grammar for the docopy language.
First identify which elements must be parsed and which (if any) may be discarded (or perhaps remain free form text).
Characterize the delineations of each section sufficiently to write regular expression matchers for their start and end, or span.
Write a grammar to parse to just the sections out as blocks of free form text.  This forms the framework within which the section parsers will operate.
For each section independently write a grammar to parse the it.
One by one, incorporate the section parsers into the framework.

docopt-parser includes PEG grammar for the docopt language completed and working with Arpeggio's clean peg parser.
The grammar is docopt.peg.  The python test code is parser.py.

Oohh Exciting! This was a big dream of mine when I was working with the codebase last, thanks!

This is so cool! I am extremely into this train of thought, the current parser is a nightmare - adding type hints and my 2019ish refactoring was an effort at adding sanity to a fundamentally insane-ish design. It's better than it was, but deserves more love - and a PEG sounds like just the thing.

Hi @philip-h-dye , @itdaniher , I have a bit of time since I'm currently "available for employment" and new to jazzband.
Is this issue stalled at some stage? What is missing for it to move forward?
Is it a matter of including the grammar files from docopt-parser into a PR?
RTFMs gratefully accepted....

@blaisep this is really late, but perhaps you or someone else still is motivated. I would totally merge this if it comes together fairly cleanly. I took a look at docopt-parser and it looks like it is a bit messy, so it won't be a simple matter of just copying stuff over.
I think steps would be:

create a new branch
add the .peg grammar files
add scripts/whatever to make it easy/reproducible to generate the .py parser from the .peg files. The generated parser should also be checked into git. It will need to stay in sync with the grammar files.
modify the existing docopt/__init__.py to use this generated parser instead of the custom parsing logic in there now.
ensure all existing tests pass. Maybe add some more tests if you think you are finding a new edge case.
submit a PR for this branch

I think you're going to have to be mostly self-directed for those stages, I can provide some help but don't really want to sink a ton of time into it.

See EBNF grammar from docopts port: https://github.com/docopt/docopts/wiki/Home/443b839150f8aec2fc581f5a7a70b8bb7648d5c2


wl-gammarelay-applet: init at 0.1.4

Description of changes

wl-gammarelay-applet is a small desktop applet for controlling wl-gammarelay-rs via DBus. This applet is written in Rust and provides a Slint UI.
The original repo is at https://github.com/junelva/wl-gammarelay-applet
This package derivation uses the fork https://github.com/lgbishop/wl-gammarelay-applet which contains a few UI changes/fixes.
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review run on x86_64-linux 1

1 package built:

wl-gammarelay-applet

I believe I have done everything correctly:

Can confirm the output from nix-build -A wl-gammarelay-applet runs. Including the package as a derivation in my system flake also seems to work well on my personal machine.
I have squashed the commits as required.

@Aleksanaa thank you for your time and attention!


gnome-shell crashes on login

Describe the bug
On my system with Gnome, I enter my password in GDM and then I see a blank screen and end up at GDM again. In the background gnome-session has crashed.
I would say there is a 80% chance this happens directly after boot. The second login attempt is usually fine.
My system is a Lenovo L14 (AMD) with integrated Radeon graphics running NixOS 23.05, but this has happened a long time already. It's not a Gnome 44.1 regression.
Steps To Reproduce
Steps to reproduce the behavior:

Select user and type password in GDM.
See GDM re-appear

Expected behavior
Logging in to the session works reliably.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
There is a complete system log with
  services.xserver.displayManager.gdm.debug = true;
  services.xserver.desktopManager.gnome.debug = true;

here:
gnome-session-issue.log
Look for "segfault" to find the interesting part. The coredump produced this backtrace:
Core was generated by `/nix/store/z3f6qcbkk6px6x6hnr8mxkkpv47xr06n-gnome-shell-44.1/bin/gnome-shell'.
Program terminated with signal SIGSEGV, Segmentation fault.
#0  0x00007fddfa60d7dc in JS::HeapObjectPostWriteBarrier(JSObject**, JSObject*, JSObject*) ()
   from /nix/store/19v08nmkidbzg1adzpycnd30xbkyhf8c-spidermonkey-102.8.0/lib/libmozjs-102.so
[Current thread is 1 (Thread 0x7fddf8351e00 (LWP 1694))]
(gdb) bt full
#0  0x00007fddfa60d7dc in JS::HeapObjectPostWriteBarrier(JSObject**, JSObject*, JSObject*) ()
   from /nix/store/19v08nmkidbzg1adzpycnd30xbkyhf8c-spidermonkey-102.8.0/lib/libmozjs-102.so
No symbol table info available.
#1  0x00007fddfbf51541 in GjsContextPrivate::~GjsContextPrivate() [clone .localalias] ()
   from /nix/store/y7p15pygbi1npvj7rb9hq2f3kd2id9cg-gjs-1.76.0/lib/libgjs.so.0
No symbol table info available.
#2  0x00007fddfbf4ffbf in gjs_context_finalize(_GObject*) () from /nix/store/y7p15pygbi1npvj7rb9hq2f3kd2id9cg-gjs-1.76.0/lib/libgjs.so.0
No symbol table info available.
#3  0x00007fddfc1d0530 in g_object_unref () from /nix/store/n0bf4ddl69nk0lm6awh834syxqh0d3ss-glib-2.76.2/lib/libgobject-2.0.so.0
No symbol table info available.
#4  0x00000000004039fd in main ()

The backtrace is pretty similar in this bug: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=973740
Notify maintainers
@NixOS/gnome
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.3.3, NixOS, 23.05 (Stoat), 23.05.20230525.3e01645`
 - multi-user?: `no`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.13.3`
 - channels(root): `""`
 - channels(julian): `""`
 - nixpkgs: `not found`

Could you please try to install debug symbols for GNOME Shell?
Could you please get the output of bt full in coredumpctl gdb $pid with the following installed (where pid is from the output of coredumpctl)?
environment.enableDebugInfo = true;
environment.systemPackages = [
  # Explicitly install to get debug symbols.
  pkgs.gnome.gnome-shell
  pkgs.gnome.mutter
  pkgs.gnome.gnome-session
  pkgs.glib
  pkgs.gjs
  pkgs.spidermonkey_102
];

Do not forget to re-login after switching to that configuration or run coredumpctl with NIX_DEBUG_INFO_DIRS=/run/current-system/sw/lib/debug environment variable, since nixos-rebuild switch cannot update environment variables for running programs.

This yielded the following:
(gdb) bt full
#0  0x00007f227740d7dc in JS::HeapObjectPostWriteBarrier(JSObject**, JSObject*, JSObject*) ()
   from /nix/store/19v08nmkidbzg1adzpycnd30xbkyhf8c-spidermonkey-102.8.0/lib/libmozjs-102.so
No symbol table info available.
#1  0x00007f2278d51541 in js::BarrierMethods<JSObject*, void>::postWriteBarrier (next=0x0,
    prev=<optimized out>, vp=0x22b8fb0)
    at /nix/store/kq4468pg4jrwg5vzsvr4kg7bhmknvwfp-spidermonkey-102.8.0-dev/include/mozjs-102/js/RootingAPI.h:795
No locals.
#2  JS::Heap<JSObject*>::postWriteBarrier (next=<optimized out>,
    prev=@0x22b8fb0: 0x2685adb724c0, this=0x22b8fb0)
    at /nix/store/kq4468pg4jrwg5vzsvr4kg7bhmknvwfp-spidermonkey-102.8.0-dev/include/mozjs-102/js/RootingAPI.h:376
No locals.
#3  JS::Heap<JSObject*>::~Heap (this=0x22b8fb0, __in_chrg=<optimized out>)
    at /nix/store/kq4468pg4jrwg5vzsvr4kg7bhmknvwfp-spidermonkey-102.8.0-dev/include/mozjs-102/js/RootingAPI.h:338
No locals.
#4  mozilla::detail::VectorImpl<JS::Heap<JSObject*>, 0ul, js::SystemAllocPolicy, false>::destroy (aEnd=0x22b9010, aBegin=<optimized out>)
    at /nix/store/kq4468pg4jrwg5vzsvr4kg7bhmknvwfp-spidermonkey-102.8.0-dev/include/mozjs-102/mozilla/Vector.h:65
        p = 0x22b8fb0
#5  mozilla::Vector<JS::Heap<JSObject*>, 0ul, js::SystemAllocPolicy>::~Vector (
    this=0xd932a8, __in_chrg=<optimized out>)
    at /nix/store/kq4468pg4jrwg5vzsvr4kg7bhmknvwfp-spidermonkey-102.8.0-dev/include/mozjs-102/mozilla/Vector.h:901
        g = <optimized out>
#6  JS::GCVector<JS::Heap<JSObject*>, 0ul, js::SystemAllocPolicy>::~GCVector (this=0xd932a8,
    __in_chrg=<optimized out>)
    at /nix/store/kq4468pg4jrwg5vzsvr4kg7bhmknvwfp-spidermonkey-102.8.0-dev/include/mozjs-102/js/GCVector.h:43
No locals.
#7  GjsContextPrivate::~GjsContextPrivate (this=0xd93230, __in_chrg=<optimized out>)
    at ../gjs/context.cpp:487
        _pp = <optimized out>
        _ptr = <optimized out>
        _pp = <optimized out>
        _ptr = <optimized out>
        _pp = <optimized out>
        _ptr = <optimized out>
#8  0x00007f2278d4ffbf in gjs_context_finalize (object=0xd933b0) at ../gjs/context.cpp:500
        gjs = <optimized out>
#9  0x00007f2278fd0530 in g_object_unref (_object=0xd933b0) at ../gobject/gobject.c:3938
        weak_locations = <optimized out>
        nqueue = 0x2261900
        object = 0xd933b0
--Type <RET> for more, q to quit, c to continue without paging--c
        old_ref = <optimized out>
        __func__ = "g_object_unref"
        retry_atomic_decrement1 = <optimized out>
#10 0x00007f2279228ba9 in _shell_global_destroy_gjs_context (self=<optimized out>)
    at ../src/shell-global.c:750
        _pp = <optimized out>
        _ptr = <optimized out>
#11 0x00000000004039fd in main (argc=<optimized out>, argv=<optimized out>)
    at ../src/main.c:674
        context = 0x8d9ef0
        error = 0x0
        ecode = 0

Could it help to run this in valgrind? If so, is there a straightforward way to do that?

No straightforward way, will require something like https://github.com/NixOS/nixpkgs/issues/226355#issuecomment-1510488966 (including the valgrind patch for debug symbols).
Weird thing that the backtrace is different from the previous one. And that the crash is only triggered by cleanup code, which means the shell termination must have been caused by something that happened earlier.

the shell termination must have been caused by something that happened earlier.

Yes.
Regarding valgrind, I'll try that later today.

With valgrind enabled it doesn't crash. I've tried many times and the login just worked. When I rolled back the changes it crashed again on the first try. Might be a race condition of sorts.

So waiting at the login prompt for a while also seems to workaround the problem. So my laptop is too fast? :smile: Any debug ideas are welcome. Otherwise, I'm going to dig into the libmozjs code that actually crashes, but I give this a low chance of success.

So waiting at the login prompt for a while also seems to workaround the problem.

I have the same issue, for over a year. I think I can make Gnome crash indefinitely by typing my password quickly all the time -- it seems to require a few seconds between showing the password input field and me pressing enter.
I wonder if this is related to https://github.com/NixOS/nixpkgs/issues/103746. That issue can also be worked around with a 5s sleep.

same issue here

i also have this problem first login after unplugging my charger

same here, multiple failed logins, sometimes blank grey screen with cursor working after login ( i presume that the main thing isnt working because the mouse doesnt have the flat profile applied ) -> when that happens,

i press ctrl+alt+f2 to go to tty2(you can go to any one),
there, systemctl status gdm outputs that the process

then, i run pkill gdm ( sometimes works without sudo and sometimes doesn't)

gdm starts up properly, working as expected, logs in in the first try

if you press ctrl+alt+f1(for tty1), you can see the gnome desktop ( with your wallpaper and all customization )

you cannot see your mouse, you cannot give any input, by every means of input, it stays unresponsive
but, the time in the panel updates, if you have vitals extension, you can see it update too. but you cannot do anything
and once you get here, you cannot go to another tty by pressing any amounts of ctrl+alt+F-num
and the only means of recovery is hard reboot by long pressing the power button,

i hope this helps in the diagnosis, would love to help in any way.


Create index.html file on VS CODE

Create the index.html file on VS CODE to be able to house all of the HTML coding

Completed by Andrey


Clarify how to create a CNAME

Clarify that users should create a CNAME in repo settings, rather than creating it directly.

Why:
Close issue #2184.

What's being changed:
See the diff above

Check off the following:

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

Hmm. Don't know how that happened. It's fixed now!

Hmm. Don't know how that happened. It's fixed now!

Thanks @ajs256! I'll get this triaged for review ✨

Thanks @ajs256! I'll get this triaged for review ✨

Hey there @duc811997. I would strongly advise changing your recovery codes now, since you seem to have accidentally uploaded them. @janiceilene or someone else with permissions, could you please edit their comment to remove the file?

Hey there @duc811997. I would strongly advise changing your recovery codes now, since you seem to have accidentally uploaded them. @janiceilene or someone else with permissions, could you please edit their comment to remove the file?

Wait a minute, did I close this? I don't seem to be able to re-open it.

Wait a minute, did I close this? I don't seem to be able to re-open it.

Re-triaged!

Re-triaged!

Looks like your stalebot fixisn't working...

I can't re-open it, but I will not delete the branch so someone can re-open it and hopefully merge it.

It looks like our new action inadvertently closed this :upside_down_face: Sorry about that! Reopening and triaging this for review :yellow_heart: Thanks for not deleting the branch!


Question - is using *FunctionApp with existing resources possible?

My question is specifically about ArchiveFunctionApp and cousins, but it can apply almost anywhere where a component's args takes another component as input with typescript.
Given something like the following:
  const externalResourceGroup = azure.core.getResourceGroup({
    name: "some-existing-rg"
  })

  const azfn = new azure.appservice.ArchiveFunctionApp("appazfn", {
    resourceGroup: externalResourceGroup,
    archive: new pulumi.asset.FileArchive("../dist/app/FunctionApp/0.1.0-alpha.0.zip"),
  })

TypeScript will fail this with the message: Type 'Promise<GetResourceGroupResult> & GetResourceGroupResult' is not assignable to type 'ResourceGroup | undefined'.   Type 'Promise<GetResourceGroupResult> & GetResourceGroupResult' is missing the following properties from type 'ResourceGroup': urn, getProviderts(2322)
Is it possible to use the higher level components with existing resources or to somehow get the same type as the created component in a downstream stack? Thanks

In place of azure.core.getResourceGroup, I think you can use azure.core.ResourceGroup.get. It's unfortunate, but the former returns different information than the latter due to differences in the underlying Terraform APIs.
  const externalResourceGroup = azure.core.ResourceGroup.get("some-existing-rg", "some-existing-rg")

  const azfn = new azure.appservice.ArchiveFunctionApp("appazfn", {
    resourceGroup: externalResourceGroup,
    archive: new pulumi.asset.FileArchive("../dist/app/FunctionApp/0.1.0-alpha.0.zip"),
  })

Oh I had not come across these resource based gets, thanks. This worked with the 2nd parameter being the azure resource id.
Thanks again for the quick response!


Replace C# Developer Center broken link with new one

The C# Developer Center link is broken. This commit replaces the link with the interactive tutorial.
I suggest we use the "Getting Started with C#" interactive tutorial instead which is aimed at people who are learning C# for the first time.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot


Reorder some operation symbols

Reordering some of the operation symbols to make them work.

It seems like the order is important.  I reorded #9 (division sign) before - (minus). I also added #8 (star operator) and changed #7 (asterisk operator) to ∗ (from ⋆) and ordered them ***, ** and *.

Thanks for the fix. In copying symbols from the original parser's numeric form to named form, I must have used some of the wrong names.
Order definitely matters. I found these were some of the trickier bugs when writing the grammar. I guess I should write more tests :smile:


messages with role 'tool' must be a response to a preceeding message with 'tool_calls'

in researcher.tsx
{"error":{"message":"Invalid parameter: messages with role 'tool' must be a response to a preceeding message with 'tool_calls'. (request id:<PHONE_NUMBER>100984850589317176109) (request id:<PHONE_NUMBER>100973706982837582869) (request id:<PHONE_NUMBER>10097110056956WoKcYSs) (request id:<PHONE_NUMBER>1006903012458ab9obA9U)","type":"invalid_request_error","param":"messages.[3].role","code":null}}

This error occurs when it’s not properly called by the tool. Are you experiencing this without making any changes to the code? Please describe the steps to reproduce.

use model gpt-4 is ok. gpt-4-turbo sometimes error.
just add images and answers in research


blazor (server side) javascript interop: Javascript fails because code must run as user-initiated

I am running into a problem writing data into the clipboard using server-side blazor. The library https://github.com/lgarron/clipboard-polyfill fails to write a text to clipboard after a button press using server-side blazor under firefox V69, but is works under chrome V77.
With a plain old javascript html page without blazor the clipboard write works in firefox and chrome.
I assume that firefox has the security requirement that clipboard manipulation must happen in code flagged as "user-initiated".
Because server-side blazor goes from user-initiated javascript to server side c# to websocket-event javascript, the "copy to clipboard" javascript code is not flagged as "user-initiated" anymore.
Is there a official way or hack in blazor to execute javascript interop code with the "user-initiated" flag on?
Repo with example: https://github.com/Henk7/BlazorJSUserInitiatedPermission

Thanks for contacting us, @Henk7.
@SteveSandersonMS can you please look into this? Thanks!

Is there a official way or hack in blazor to execute javascript interop code with the "user-initiated" flag on?

This is a tricky area and is unfortunately beyond the control of Blazor. As you've observed, this is a browser-implemented deliberate limitation.
To satisfy the browser's requirements, you may need to trigger the clipboard JS call from JS code rather than .NET code, so that it can run synchronously.
If you wanted to have a textbox plus a button that copies the contents of some .NET-generated version of that text to the clipboard, then you might need to implement it like this:

Have an @oninput event handler on the textbox. In your .NET event handler, take whatever value is being typed and output the .NET-generated version of that text onto some data-something attribute on that same textbox. So, the DOM will always contain the text that you might copy to the clipboard.
On the button that copies to the clipboard, put a JavaScript onclick event handler that takes the data-something value and copies it to the clipboard. This way, the actual clipboard API call will run synchronously on click.

I know it's inconvenient. There are very few scenarios where this limitation applies, but I'm afraid that clipboard is one of them.


1770 la funded places

Context
Changes proposed in this pull request
Guidance to review

Can you please fill in the description of the pull request to provide the reviewer with some guidance. See https://github.com/alphagov/styleguides/blob/master/pull-requests.md for some reference and other pull requests raised in this repository

can we also squash the commits. the only merge commits in a repository should be those of pull requests


Connection Refused to Mosquitto

So I'm running mosquitto mqtt in docker. Other containers such as zigbee2mqtt connect fine but I cannot get this container to connect.
Log below. The container stops instantly so I can't even troubleshoot from the container.
Traceback (most recent call last):
  File "/usr/src/app/./mqtt_gateway.py", line 941, in <module>
    asyncio.run(mqtt_gateway.run(), debug=debug_log_enabled())
  File "/usr/local/lib/python3.12/asyncio/runners.py", line 194, in run
    return runner.run(main)
           ^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/asyncio/runners.py", line 118, in run
    return self._loop.run_until_complete(task)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/asyncio/base_events.py", line 687, in run_until_complete
    return future.result()
           ^^^^^^^^^^^^^^^
  File "/usr/src/app/./mqtt_gateway.py", line 506, in run
    await self.publisher.connect()
  File "/usr/src/app/publisher/mqtt_publisher.py", line 86, in connect
    await self.client.connect(host=self.host, port=self.port, version=gmqtt.constants.MQTTv311, ssl=ssl_context)
  File "/usr/local/lib/python3.12/site-packages/gmqtt/client.py", line 225, in connect
    self._connection = await self._create_connection(
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/gmqtt/client.py", line 241, in _create_connection
    connection = await MQTTConnection.create_connection(host, port, ssl, clean_session, keepalive, logger=self._logger)
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/gmqtt/mqtt/connection.py", line 27, in create_connection
    transport, protocol = await loop.create_connection(MQTTProtocol, host, port, ssl=ssl)
                          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/asyncio/base_events.py", line 1122, in create_connection
    raise exceptions[0]
  File "/usr/local/lib/python3.12/asyncio/base_events.py", line 1104, in create_connection
    sock = await self._connect_sock(
           ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/asyncio/base_events.py", line 1007, in _connect_sock
    await self.sock_connect(sock, address)
  File "/usr/local/lib/python3.12/asyncio/selector_events.py", line 651, in sock_connect
    return await fut
           ^^^^^^^^^
  File "/usr/local/lib/python3.12/asyncio/selector_events.py", line 691, in _sock_connect_cb
    raise OSError(err, f'Connect call failed {address}')
ConnectionRefusedError: [Errno 111] Connect call failed ('<IP_ADDRESS>', 1833)
2024-06-15 11:59:42,942 [ ERROR  ] Task was destroyed but it is pending!
task: <Task pending name='Task-1' coro=<Client._resend_qos_messages() running at /usr/local/lib/python3.12/site-packages/gmqtt/client.py:176>> - asyncio
sys:1: RuntimeWarning: coroutine 'Client._resend_qos_messages' was never awaited

2024-06-15 18:50:57,961 [  INFO  ] [CONNECTION MADE] - gmqtt.mqtt.protocol

2024-06-15 18:50:58,062 [  INFO  ] [CONN CLOSE NORMALLY] - gmqtt.mqtt.protocol

2024-06-15 18:50:58,063 [WARNING ] [TRYING WRITE TO CLOSED SOCKET] - gmqtt.mqtt.protocol

So if i switch to websockets i get this.  The mosquitto log doesnt show any attempt for connection from the SAIC gateway container :/

So managed to get connected. Seems I had to use a username and password despite anonymous being enabled on my broker.


Fix update_graph counting logic in test_default_scheduler_on_worker

Closes https://github.com/dask/dask/issues/10144. I'm certainly open to other suggestions.
cc @fjetter @gjoseph92

Thanks @gjoseph92! I'll plan to merge this in after CI finishes
Note the Windows + Python 3.11 failures are unrelated to the changes here https://github.com/dask/dask/issues/10146


Story um 868 add umpire prefix

Note to self: error involves cuda builds and not being able to find -lgmock

Note to self: error involves cuda builds and not being able to find -lgmock

You can remove gmock from the depends list here: https://github.com/LLNL/Umpire/blob/develop/tests/unit/op/CMakeLists.txt#L11

Making notes about errors...
PGI+cuda:

error: unrecognized preprocessing directive
#warning "crt/host_defines.h is an internal header file and must not be used directly.  Please use cuda_runtime_api.h or
cuda_runtime.h instead."
CMake Error at /usr/tce/packages/cmake/cmake-3.18.0/share/cmake/Modules/CMakeTestCUDACompiler.cmake:52
(message):
The CUDA compiler
"/usr/tce/packages/cuda/cuda-10.1.168/bin/nvcc"
is not able to compile a simple test program.
PGI on ruby:
"/g/g92/ukbeck/.gitlab-runner/builds/UrMJDUmU/2/gitlab/radiuss/Umpire/blt/third
party_builtin/googletest-master-2020-01-07/googletest/include/gtest/i
nternal/gtest-port.h", line 2079: error: function declared with
"noreturn" does return
[[noreturn]] inline void Abort() { abort(); }
XL+omp:
runs out of time
clang shmem:
/g/g92/ukbeck/.gitlab-runner/builds/T5vctq-D/1/gitlab/radiuss/Umpire/examples/cookbook/recipe_shared_memory.cpp:16:10: fatal error: 'mpi.h' file not found

Decided to start from scratch because this PR was going down rabbit hole of errors - see PR #588 instead


New Task: Replaced packer module with uglify due to lack of NPM support (deprecation)

Issue
To put it simply.. :fire:Dependency hell :fire:
The Tidy-Table dependency packer which is used by this script for minification relies on a dependency chain of the following unsupported packages:

jsdom
nomnom
contextify

Due to this, when installing this package using modern versions of NPM you will likely receive the following error:
npm ERR<EMAIL_ADDRESS>install: `node-gyp rebuild`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.

Solution
This package, including minifier, should be replaced with uglify since it can handle both JavaScript/CSS minification (and they are fully supported).
If there is anyone that wants to take the lead on this, be my guest, or I will address this when time becomes available.

Replaced with the minify since it solves both CSS/JavaScript operations in a single package.


Read ExposedPorts from Config, not ContainerConfig

.ContainerConfig.ExposedPorts does not exist in docker inspect output of a recent docker tool. .Config.ExposedPorts seems to work in all currently used versions.

lgtm


Object.require is deprecated.

Use remote.BrowserWindow instead of remote.require("browser-window")
Object.require (/Applications/Atom.app/Contents/Resources/app.asar/src/electron-shims.js:54:12)
Object.<anonymous> (/Users/andrzejdubaj/.atom/packages/integrated-learn-environment/lib/models/browser-window-wrapper.coffee:9:26)
Object.<anonymous> (/Users/andrzejdubaj/.atom/packages/integrated-learn-environment/lib/models/browser-window-wrapper.coffee:71:4)
Object.<anonymous> (/Users/andrzejdubaj/.atom/packages/integrated-learn-environment/lib/models/browser-window-wrapper.coffee:77:3)
Module._compile (<embedded>:11544:36)
Object..coffee (<embedded>:11795:29)

Closed as no longer occur.


build cli command does not take root dir as an argument.

Steps to reproduce
Take a look at https://www.farmfe.org/docs/cli/cli-api/#build
In vite start and build commands both take root dir.
What is expected?
vite build works with only import.meta.env tuning.
What is actually happening?
It crashes:
><EMAIL_ADDRESS>my-build-client
> cross-env APP=d farm build ./my -c ./my/farm.config.ts

[ Farm ] Using config file at C:\Users\houd1ni\front\my\farm.config.ts
[ Farm ] Build failed due to errors: Can not resolve index.html  from undefined.
Error: ENOENT: no such file or directory, stat 'C:\Users\houd1ni\front\index.html'
    at Object.statSync (node:fs:1660:25)
    at checkCompilationInputValue (file:///C:/Users/houd1ni/front/node_modules/@farmfe/core/dist/config/index.js:645:24)
    at normalizeUserCompilationConfig (file:///C:/Users/houd1ni/front/node_modules/@farmfe/core/dist/config/index.js:116:30)
    at resolveConfig (file:///C:/Users/houd1ni/front/node_modules/@farmfe/core/dist/config/index.js:88:44)
    at async build (file:///C:/Users/houd1ni/front/node_modules/@farmfe/core/dist/index.js:41:32)
    at async handleAsyncOperationErrors (file:///C:/Users/houd1ni/front/node_modules/@farmfe/cli/dist/utils.js:111:9)
[ building ] ⠂ transform (110) node_modules/core-js/internals/object-get-prototype-of.js.farm-runtime
[ Farm ] Error: Can not resolve `` from C:\Users\houd1ni\front.
Original error: None.

Potential Causes:
1.The file that `` points to does not exist.
2.Install it first if `` is an dependency from node_modules, if you are using pnpm refer to [https://pnpm.io/faq#pnpm-does-not-work-with-your-project-here] for solutions.
3. If `` is a alias, make sure your alias config is correct.

System Info
Win10x64 Pro latest, Intel 14600kf, 32Gb ram, almost empty.

Any additional comments?
No response

i will try it

fixed with 1.1.0


"The test framework adapter has not been registered" for simple Testing.Platform example

I am trying out the Microsoft.Testing.Platform functionality. From the Note in the overview docs, it sounds like for simple cases I should not need to explicitly create a Main() function or do any setup in it. Everything compiles OK, but when I run the test executable I get
Unhandled exception. System.InvalidOperationException: The test framework adapter has not been registered. Use 'ITestApplicationBuilder.RegisterTestFramework' to register it
   at Microsoft.Testing.Platform.Builder.TestApplicationBuilder.BuildAsync() in /_/src/Platform/Microsoft.Testing.Platform/Builder/TestApplicationBuilder.cs:line 102
   at TestingPlatformEntryPoint.Main(String[] args) in C:\onebranch\plattest\plattest\obj\Debug\net9.0\TestPlatformEntryPoint.cs:line 14
   at TestingPlatformEntryPoint.<Main>(String[] args)

I think there is something I need to add to my .csproj but I'm not clear what it is. It might be helpful to have a sample project that doesn't do any explicit setup.
Plattest.csproj:
<Project Sdk="Microsoft.NET.Sdk">

  <PropertyGroup>
    <OutputType>Exe</OutputType>
    <TargetFramework>net9.0</TargetFramework>
    <ImplicitUsings>enable</ImplicitUsings>
    <Nullable>enable</Nullable>
  </PropertyGroup>

 <ItemGroup>
   <PackageReference Include="Microsoft.Testing.Platform.MSBuild" />
   <PackageReference Include="MSTest.TestFramework" />
 </ItemGroup>

</Project>

Directory.Packages.props:
<Project>
  <PropertyGroup>
    <ManagePackageVersionsCentrally>true</ManagePackageVersionsCentrally>
  </PropertyGroup>
  <ItemGroup>
    <PackageVersion Include="Microsoft.Testing.Platform" Version="1.4.3" />
    <PackageVersion Include="Microsoft.Testing.Platform.MSBuild" Version="1.4.3" />
    <PackageVersion Include="MSTest.TestFramework" Version="3.6.4" />
  </ItemGroup>
</Project>

Tests.cs:
using Microsoft.VisualStudio.TestTools.UnitTesting;

[TestClass]
public class Tests
{
    [TestMethod]
    public void Test1()
    {
        Assert.AreEqual(1, 1);
    }

    [TestMethod]
    public void Test2()
    {
        Assert.AreEqual(2, 2);
    }
}

Please have a look here https://learn.microsoft.com/dotnet/core/testing/unit-testing-mstest-runner-intro#enable-mstest-runner-in-an-mstest-project. You are missing the MSTest.TestAdapter project and also the EnableMSTestRunner property:
Plattest.csproj:
<Project Sdk="Microsoft.NET.Sdk">

  <PropertyGroup>
    <OutputType>Exe</OutputType>
    <TargetFramework>net9.0</TargetFramework>
    <ImplicitUsings>enable</ImplicitUsings>
    <Nullable>enable</Nullable>
    <EnableMSTestRunner>true</EnableMSTestRunner>
  </PropertyGroup>

 <ItemGroup>
   <PackageReference Include="MSTest" />
 </ItemGroup>

</Project>

Directory.Packages.props:
<Project>
  <PropertyGroup>
    <ManagePackageVersionsCentrally>true</ManagePackageVersionsCentrally>
  </PropertyGroup>
  <ItemGroup>
    <PackageVersion Include="MSTest" Version="3.6.4" />
  </ItemGroup>
</Project>

Thank you! That works. I think I didn't notice that because it was in the MSTest rather than the Testing Platform part of the content hierarchy.
One follow up question: is this mechanism compatible with AOT? If I add <PublishAot>true</PublishAot> to my .csproj and run dotnet publish I get a number of warnings about reflection and the published .exe doesn't work.
I'm partly interested in this because we have a lot of code we would like to move to compile as AOT. I would like to unit test that everything works as expected in that mode.
Thanks again.

For native aot, you will need to switch the nuget packages to use, see here https://github.com/microsoft/testfx/tree/main/samples%2Fpublic%2Fmstest-runner%2FNativeAotRunner.
Note that at the moment, we have only really partial support (test methods and parametrized tests but no fixture or other attribute and feature). If you could +1 on the issue for native aot support that would help us with convincing management on prioritizing this work.
Note 2, I'd strongly recommend the usage of MSTest.Sdk https://learn.microsoft.com/dotnet/core/testing/unit-testing-mstest-sdk as it would simplify your project setup and would automatically switch the packages based on PublishAot being true or false.


Future of Http Clients and Feed

Hi all,
We have PSR-7. What are the plans of integrating different http clients ?
Do we need something like https://github.com/beberlei/http-client-middleware ?
Is there any plans about this?

I am closing for this is been done on https://github.com/zendframework/zend-feed/pull/14


Consolidate test sample documents into one subdir

This gets rid of the extra testdata directory for functional tests and collapses those into a single one, using the existing sample_documents subdir of the unit tests.

This addresses some review comments from @khaledsulayman on #349 that I wasn't able to get in before that one auto-merged. Not urgent, and just some cleanup of folder structures for the new functional test sample docs.


Feature/editable data points

Looks good overall.
One issue that should probably be fixed before merging is when clicking on the "X" in the new data point line it dissapears and there are no way to get the line back.

Agreed, thanks. Disappearing row now fixed and test added that covers the scenario.


Same name of Group and same name of Assignments should not be their

Describe the bug
When we create a group of name eg : yash15601 the one group is created now create the another group of same name i.e yash15601 ,it will create the group of same group
Same in Assignments :,When we create a Assignments of name eg : myfistassignments the one Assignment is created now create the another Assignments of same name i.e myfistassignment ,it will create Assignments of same name
Expected behavior
Their should not exists the same name of Assignments  and group
Screenshots
Group:

Assignments:

Are you working on this issue? (Yes/No)
Yes

Didn't understand what is the expected behaviour?

Group,Project,Assignment names should be unique

@nitin10s  As said by  @Ananthavijay

Great observation! :)

@nitin10s  thanks 😂

This is not really much of an issue. Any fix should consider the following things

It should be efficient
It should not break current assignments/groups etc

@satu0king  like having same name of group will create problem for user to identify the in which he/she added related  members  a part from this about efficiency  and breaking assignment /group  will fix this soon

I am working on this issue.

What I was thinking is if a user is creating an assignment/group with the same name instead of showing the message Same name group/assignment already exist we can just confirm that if he/she wanted to continue. We can popup something like ${name} assignment/group already exist, are you sure want to create a new one?. In this way, it is just in the user's hands.
WDYT @gr455 @tachyons @ayan-biswas0412 @ryuk156 ?

yah I think this check should be there.

What I was thinking is if a user is creating an assignment/group with the same name instead of showing the message Same name group/assignment already exist we can just confirm that if he/she wanted to continue. We can popup something like ${name} assignment/group already exist, are you sure want to create a new one?. In this way, it is just in the user's hands.
WDYT @gr455 @tachyons @ayan-biswas0412 @ryuk156 ?

I also agree this check should be there. I am working on this issue and will raise a pr soon.

Hello ! Is this issue assigned to someone.
I can fix this issue can you please assign me, if its not assigned!

@chaitanyahere Go ahead 👍


[HUDI-975] Add unit tests in TestHoodieTableFileSystemView to test view for non-partitioned table

What is the purpose of the pull request
Add unit tests in TestHoodieTableFileSystemView to test view for non-partitioned table
Brief change log
Add unit tests in TestHoodieTableFileSystemView to test view for non-partitioned table
Verify this pull request
This change added tests.
Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Codecov Report

Merging #1692 into master will increase coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #1692   +/-   ##
=========================================
  Coverage     18.19%   18.20%
- Complexity      856      857    +1
=========================================
  Files           348      348
  Lines         15344    15344
  Branches       1523     1523
=========================================
+ Hits           2792     2793    +1
+ Misses        12195    12193    -2
- Partials        357      358    +1

Impacted Files
Coverage Δ
Complexity Δ

...che/hudi/common/table/timeline/TimelineLayout.java
85.71% <0.00%> (-7.15%)
3.00% <0.00%> (ø%)

...apache/hudi/common/fs/HoodieWrapperFileSystem.java
22.69% <0.00%> (+0.70%)
29.00% <0.00%> (+1.00%)

Continue to review full report at Codecov.

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


Made improvement on UI of complex numbers

Issue Reference: #386
PS: Moved polar function as it was part of complexlogic.js .

Hi! Kindly tell how you reviewed the PR#489


Add Header Authentication for SSO

When the env variable REMOTE_USER_HEADER is set, it will use the username provided in will skip both websocket and basic auth authentication.
This allows for login through something like Authelia.
How to test:

run server with REMOTE_USER_HEADER=Remote-User npm run start-server-dev or add REMOTE_USER_HEADER env variable to your docker container.
Make sure the user set in your header exists in the database (Mikhail in this case).
Open up a websocket connection with the following header included Remote-User:Mikhail.
You should now be logged in as Mikhail through the same mechanics as autoLogin uses.

I believe it is duplicate of no auth feature.

I don't believe its the same, compleetly disabling authentication vs letting a reverse proxy handle the logged in user (and thus still allow different user accounts) is different.

Then security will be a big problem, because I believe strangers can login without password using http header.
I probably will implement access token for API features in the future.
I suggest this feature should use that access token at that time.

Http headers should only be used with a correctly configured reverse proxy. (thus disallowing end-user to set the REMOTE_USER_HEADER themselves). If you want to know some more about this I would advice you to read something like https://www.authelia.com/docs/community/using-remote-user-header-for-sso-with-jira.html

If you want I can even demo it how I personally use it with SSO for multiple services (so you can see why I like it so much)

If you want I can even demo it how I personally use it with SSO for multiple services (so you can see why I like it so much)

After read the authelia's doc, I now understand how it works.
But honestly I am personally not familiar with sso. I need more time to study the code changes and the workflow of sso. And make sure there is no vulnerabilities that hate current users.
So it may take a longer time to get it merged. Thank you for the pr btw.

No problems, if you have any questions feel free to ask.

Btw, just a general question, is there a reason why the authentication is done over web-socket and not through a normal post request? (and then session stored in cookie instead of local storage token?) (Sorry if this is the incorrect place to ask this)

Btw, just a general question, is there a reason why the authentication is done over web-socket and not through a normal post request? (and then session stored in cookie instead of local storage token?) (Sorry if this is the incorrect place to ask this)

Just because the whole application is based on websocket and it did using local storage token.

OK that's a clear design choice.
I personally prefer to open the websocket connection after authenticating (unless you also have some public information over the websocket).
It makes it slightly easier since you only need to verify the jwt token once on connection, and after that all you can assume for each message that the user is authenticated.


Adopt existing pull-request

almende/vis

[ ] https://github.com/almende/vis/pull/4262
[ ] https://github.com/almende/vis/pull/4142
[ ] https://github.com/almende/vis/pull/4123

almende/vis#4123

Graph3d rotating axis labels (heatmap labels)

:tada: done!


EDDB.IO is offline

Might want to consider Inara or EDSM lookups for systems, since EDDB.IO went offline.

You mean from the UI itself? Unfortunately, I believe that the only way forward is to remove the link altogether. This is because I was able to have a link to EDDB only because I was also importing its dumps and was able to cross verify systems names (they can be duplicates), something I don't think I can do over an API call to Inara or EDSM.
But yes, there are some long due maintenance activities pending in EliteBGS.


Silverbullet broken on firefox 102.14.0esr (64-bit)?

Looks like latest SilverBullet does not work on Firefox.
Error: SyntaxError: export declarations may only appear at top level of a module..
This is the stack:
Booting SilverBullet client in Sync Mode [boot.ts:8:10](https://silverbullet.md/web/boot.ts)
Running in read-only mode [boot.ts:14:12](https://silverbullet.md/web/boot.ts)
Service worker registered... [boot.ts:32:14](https://silverbullet.md/web/boot.ts)
Rebuilding editor state [client.ts:830:12](https://silverbullet.md/web/client.ts)
Reading file content from fallback for SETTINGS.md 2 [fallback_space_primitives.ts:26:16](https://silverbullet.md/common/spaces/fallback_space_primitives.ts)
Loading plugs [client_system.ts:210:12](https://silverbullet.md/web/client_system.ts)
(Re)loading plugs [client_system.ts:213:12](https://silverbullet.md/web/client_system.ts)
Booting up worker for emoji [worker_sandbox.ts:33:12](https://silverbullet.md/lib/plugos/sandboxes/worker_sandbox.ts)
SyntaxError: export declarations may only appear at top level of a module

this on Firefox 102.14.0esr (64-bit) (linux), on Chrome it works.

Ok, this could be. But from what I can tell this is a release from June 2022 https://www.mozilla.org/en-US/firefox/102.0/releasenotes/ I’m not sure it’s worth investing a lot of time in attempting to support almost 2 year old browsers. Upgrading your firefox is not an option?

You are right, upgraded to 106 and it works! Didn't notice the browser was so old.


Create multisim script - Subnational

Script to create and run a multisim with Ethiopia's regional models

Example of uncalibrated results for multiple regions to provide example default output

Issue Closed: PR #319


Pre-trained models

Hi,
Could you share the pre-trained models? I am particularly interested in the 3D brain MRI ones for Pytorch.
Thanks!

Hi and thanks for your comment,
I do my best to update the 3D part


feat: migrate commonjs to esm

:tada: This PR is included in version 1.6.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Allow folder location to be current directory and not root

would be nice if a config can be set to use the current directory of the active file rather than the root.

you can try set vim.opt.autochdir = true

yeah autochdir does the trick. would add value for others maybe being a note in the readme.
coming from francoiscabrol/ranger.vim it just worked.

gonna re open because i noticed when using this method, it culls my fuzzy file search list as i go deep inside a folder tree.
essentially i like to fuzzy find files within the tree and open by name if i know what i am looking for then lean on ranger to find nearby files or handle directory management, moving and creating files / folders.

ok, i understand

solved

yeah been using it for about an hour flawlessly. thank you very much.
all my little issues now fully resolved can move onto this full time <3


Segmentation fault (core dumped) when training on TIMIT

Thank you for sharing the code. When I ran it on TIMIT dataset, it occurs Segmentation fault here. I would like to ask how to solve it.

i have met the same status, have you solved this problem??


Remove close_on_error store.py

Remove close_on_error in store.py. This change involves only apiv2.
Currently, api_v2.open_dataset can take in input a store instead of a file. In case of error, xarray closes the store.
Xarray should manage the closure of a store that has been instantiated externally.
This PR correct this behaviour in apiv2

[x] Related https://github.com/pydata/xarray/pull/4673
[x] Passes isort . && black . && mypy . && flake8

Hello @aurghs! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file xarray/backends/store.py:

Line 3:1: F401 '..core.utils.close_on_error' imported but unused


Update to clang-format 12.0.1

#no-changelog

I'm going to merge this here so the repo is in sync with the clang-format on the GHA machines.  Run brew upgrade clang-format locally to update.
Feel free to continue the discussion here if there are any concerns about the formatting changes.

Yep. No need to do an elaborate review. The review notice was mostly FYI and to confirm that the formatter didn't do anything really crazy - which has occasionally happened on past updates.


containers/image: vendor

Vendor in latest containers/image
Signed-off-by: Giuseppe Scrivano<EMAIL_ADDRESS>

Ref: https://bugzilla.redhat.com/show_bug.cgi?id=1499274

LGTM (assuming Travis is ok)

@ashcrow good catch.  I've updated my vndr and now vendor.conf is not deleted.

LGTM

LGTM assuming happy tests.

tests are passing


Circular View not happening

Team, Thanks for Lib.
But am not able to swipe last item to first item and first item to last item, Did i have to include any more
Thanks in advance.

Sorry, this lib did not support this feature.


Fixes #56. Do not skip voice packages if user is not talking.

We cannot skip incoming packages when the user is not currently talking. Otherwise the first packet of each stream will get lost.

I'm still a bit unsure of this change.
If it's the only way to fix the issue, I'll be fine merging it (or something like it) - but I'm mainly worried about the needless jitterbuffer processing. Also with this change the "missedFrames" value will increase needlessly while the users aren't talking.
Could you try whether replacing the continue; with user.buffer.tick(); continue; would fix the issue? This way the jitter buffer would know that time is passing at least.

With the proposed change, do you know if the packets.push is being called repeatedly even if voiceActive == false? Or does the jitterbuffer return something that takes the execution to the else-block which doesn't append any frames to the packets array.

frame.data is undefined in that case and the else-branch is used, so the packets.push is not called repeatedly.
if( !user.voiceActive ) {
    user.buffer.tick();
    continue;
}

does not solve the problem either.
I am not sure whether there are any other implications or problems with this fix.

Okay. Will go with this one then!


How is nvdApiKey set in gradle version?

How do we set the nvdApiKey in the gradle version. I only see the option in cli

Disregard. I located the config: it's apiKey


implement ctr --connect-timeout

the connect-timeout global flag was already defined, but not used.

Hi, @lalyos thanks for contribution. could you mind to repush it to trigger the CI again? since we have flaky case here.
ok  	github.com/containerd/containerd/archive	1.069s	coverage: 19.6% of statements
--- FAIL: TestImagesCreateUpdateDelete (0.06s)
    --- FAIL: TestImagesCreateUpdateDelete/Touch (0.01s)
        images_test.go:531: timestamp for updatedat not after createdat: 2019-09-16 10:38:49.5748944 +0000 UTC <= 2019-09-16 10:38:49.5748944 +0000 UTC

Thanks

Codecov Report

Merging #3653 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #3653   +/-   ##
=======================================
  Coverage   42.39%   42.39%
=======================================
  Files         127      127
  Lines       14048    14048
=======================================
  Hits         5955     5955
  Misses       7193     7193
  Partials      900      900

Flag
Coverage Δ

#linux
45.9% <ø> (ø)
:arrow_up:

#windows
37.27% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

LGTM


docs: update image files of mm scoring paper

Description

This PR updates image files in marketmaker document

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Appropriate labels applied
[ ] Targeted PR against correct branch
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards.
[ ] Wrote unit and integration
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[ ] Added relevant godoc comments.
[ ] Re-reviewed Files changed in the Github PR explorer
[ ] Review Codecov Report in the comment section below once CI passes

@seth100kim Can you review this PR to see if the images are correctly updated?