this is just a question (not any issue or so)

can the below steps be written in any other file than package.json ?
"cypress-cucumber-preprocessor": {
    "step_definitions": "cypress/support/step_definitions/"
  }

As per documentation:

Step definition files are by default in: cypress/support/step_definitions. If you want to put them somewhere please use [cosmiconfig](https://github.com/davidtheclark/cosmiconfig) format. For example, add to your package.json :

```javascript
  "cypress-cucumber-preprocessor": {
    "step_definitions": "cypress/support/step_definitions/"
  }

Please take a look at the cosmiconfig


Added GS SIGNAL register test

As discussed in #31, this is a first shot of what GIF packet construction could look like.
Not quite sure about the register value generation in gsregs.h that uses defines. The alternative was to use functions, but I was thinking that might be a bit heavy.
Anyways, I'm open to all your suggestions, so, just tell me what you think!

Hmm.  I definitely feel like it'd be nicer to specify the regs in a different way on the tag - ideally more typesafe.  But it's tricky, since you're building a structure which will have nloop rows.
It might make sense for it to have methods which "build" the active tag, and have a method to init a new tag.  I'm not sure though, been busy lately so haven't thought about this properly yet.
An idea, some of which I'm not liking:
// GIFtag(u16 nloop, DataFormat fmt, bool last, RegDesc r0 = REG_INVALID, RegDesc r1 = REG_INVALID, ... RegDesc r15 = REG_INVALID);
packet.GIFtag(1, GIF::FMT_PACKED, true, GIF::REG_AD);
packet.SetPrim(GS::PRIM_LINE);

It needs a way to add raw data too, so we can test other bits of the tag.  But yeah, I think AD makes sense just taking a 64 bit value.  Though, I think ideally they can be quick funcs under GS:: rather than defines, like u64 SIGNAL(u32 id, u32 msk) { return (u64)id | ((u64)idmsk << 32); }.
packet.PackedAD(GS::REG_SIGNAL, GS::SIGNAL(1234, 5768));

Hmm.  It'd be awesome if it were somehow typesafe, but there are a ton of registers, so that seems like a pain.  Could be something like GS::RegData SIGNAL(u32 id, u32 msk) { return RegData(GS::REG_SIGNAL, (u64)id | ((u64)idmsk << 32)); } and then packet.PackedAD(GS::SIGNAL(1234, 5678));.  That seems cleanest but likely not worth the hassle, and would still want the alternative of a reg and data to add raw.
-[Unknown]

Thanks for your comments. Following your input, I've made some changes to the branch:

Changed the GS register value generators to proper functions.
Added a Tag class in the GIF namespace with setters for the tag's fields. The data is kept in an union to allow easy access to the fields or to the raw data (if we want to fiddle with the unused bits).
Created a new PackedPacket which is a sub-class of Packet which has functions that generates writes to GS registers in the way this format allows. A RegListPacket and ImagePacket could be added later.
Made the Emit function of Packet public to allow writing arbitrary data in the packet.

Well, tell me what you think! There's surely a way to make this even better.

Cool.  I think this is a pretty good starting point, we may find ways to make it better as we work with it more but the important thing is that I think tests written this way will be clear enough for someone to read and have an idea what's being tested.
To be fair, some of the CPU tests are maybe not beautiful (RRI_, etc.) but it's at least pretty clear what's being tested.
Thanks.
-[Unknown]


fix compiler error caused by pull request #6, caused by incorrect dep…

…endencies (NB! looks like after pull request #6 everything from sample-app is copeid to sample-app-server or sample-app-client)

note, if pull request #18 is merged, then this pull request can be closed
(as #18 contains everything here + other two issues, including resolved merge conflicts and fixed compiler errors)


crash when use tesseract 4 from cocoapod

actual_tessdata_num_entries_ <= TESSDATA_NUM_ENTRIES:Error:Assert failed:in file tessdatamanager.cpp, line 53

https://github.com/tesseract-ocr/tesseract/wiki/FAQ#actual_tessdata_num_entries_-tessdata_num_entrieserrorassert-failedin-file-ccutiltessdatamanagercpp-line-55_

Are you sure you have right version of traineddata file and are using
correct --oem
On 06-Oct-2017 2:19 PM, "saly2310"<EMAIL_ADDRESS>wrote:

actual_tessdata_num_entries_ <= TESSDATA_NUM_ENTRIES:Error:Assert
failed:in file tessdatamanager.cpp, line 53
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tesseract-ocr/tessdata_best/issues/8#issuecomment-334696805,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE2_o5-0PpL2Ux-zbOYifWQhqfa9tTNdks5spemkgaJpZM4PwLUx
.

@zdenop  Please close this issue.


.into() is very slow

rustc 1.23.0-nightly (246a6d19c 2017-11-23)
CPU: Intel i7-6700HQ (8)

I am trying to use coresimd in gimli-permutation, but it very slow.

I found a lot of Into is not optimized.
            x = _mm_shuffle_epi8(
                x.into(),
                _mm_set_epi8(12, 15, 14, 13, 8, 11, 10, 9, 4, 7, 6, 5, 0, 3, 2, 1).into()
            ).into();
            y = (shift(y.into(), 9) | _mm_srli_epi32(y.into(), 32 - 9)).into();
            let newz = x.as_i32x4() ^ shift(z.into(), 1)    ^ shift((y & z).into(), 2);
            let newy = y            ^ x                     ^ shift((x | z).into(), 1).into();
            x = z ^ y ^ shift((x & y).into(), 3).into();

I don't know if this should be a rust issue, so I report it here.

Please provide a reproducible example that we can easily compile and test. Please also tell us how you're compiling your code. For example, are you using codegen units or incremental compilation? Are you compiling in release mode?

Since those are cargo bench results I assume that is release mode but it would be good to know if you have a custom bench profile in your Cargo.toml.
@BurntSushi I will try to send a PR with the bench to stdsimd so that we can either add it or easily look at it, but I am fully booked till sunday.
@quininer how urgent is this?

@BurntSushi Yes, I use incremental compilation.
@gnzlbg It is less urgent, I currently use transmute instead of Into.

@quininer have you tested with the .as_ methods?

The bug here is incremental compilation which is known to decrease performance in release mode. The problem here is that function calls are being made, which means that these are happening in different codegen units as inlining is prevented. This is probably because Into::into (the blanket impl) is just #[inline] instead of #[inline(always)] whereas the From impls generated in the crate are indeed tagged with #[inline(always)]
Your two possible courses of action are:

Switch to using iNNxYY::from(...) instead of .into() (as terribly unergonomic as that is...)
Enable ThinLTO or wait for ThinLTO to stabilize

Other than that though there's not much we can do about this, it's just how incremental compilation in release mode works right now (performance gets worse in some cases)

@gnzlbg .as_* works fine.

That's suspicious, .into() should IMO work fine as well. @alexcrichton we should re-open this. I won't be able to look into this today, but I will check it out on sunday.

No as I mentioned, into is not tagged #[inline(always)], that's why it's not inlined when using incremental compilation.

We are implementing From in this crate for the types, could we maybe work around that by implementing Into as well (and adding #[inline(always)] to our implementation) ? Into is implemented already for everything that implements From but maybe we could use specialization here to override that.

Unfortunately coherence won't allow that

Shouldn't we make that impl from Into #[inline(always)] in general then? And leave it to From whether things should be inlined or not?

Unfortunately coherence won't allow that

If the Into trait impl for From would mark into as default then coherence would allow that.
We define the SIMD types here, so we can specialize blanket impls of traits we don't define for those types here as long as they are specializable.

Sorry but I think this is bring approached from the wrong angle. I believe it is not our job to fix this at all costs. This has been diagnosed and there are a number of reasonable workarounds. This will be fixed in the future with ThinLTO. I'd rather not spend more time debating various solutions for this.

Makes sense. I tried a couple of things this morning to address this but I
couldn’t really find anything we can reasonably do that always works.
IMO the best thing we could do is add a small benchmark for this once it is
fixed upstream to prevent upstream from breaking this in the future (but
arguably we should just check that this is the case in rustc once LTO is
merged).
Sorry for the noise.
On Sat 25. Nov 2017 at 15:39, Alex Crichton<EMAIL_ADDRESS>wrote:

Sorry but I think this is bring approached from the wrong angle. I believe
it is not our job to fix this at all costs. This has been diagnosed and
there are a number of reasonable workarounds. This will be fixed in the
future with ThinLTO. I'd rather not spend more time debating various
solutions for this.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/rust-lang-nursery/stdsimd/issues/215#issuecomment-346944336,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA3NpgcII-pJW8mqBSsg-NmEPwtNR5Voks5s6CaLgaJpZM4Qpe0p
.


CORS policy error

When running full setup (ui + proxy + database) on real URLs and trying to log in, fetching from database is blocked by CORS policy.
CORS error:
Access to fetch at '[Database URL]' from origin '[Web UI URL]' has been blocked by CORS policy: Response to preflight request doesn't pass access control check: No 'Access-Control-Allow-Origin' header is present on the requested resource. If an opaque response serves your needs, set the request's mode to 'no-cors' to fetch the resource with CORS disabled.
Blazor error:
fail: BlazorApp.AuthService[0]       Login failed System.Net.Http.HttpRequestException: TypeError: Failed to fetch  ---> System.Runtime.InteropServices.JavaScript.JSException: TypeError: Failed to fetch    at System.Net.Http.BrowserHttpHandler.SendAsync(HttpRequestMessage , CancellationToken )    Exception_EndOfInnerExceptionStack    at System.Net.Http.BrowserHttpHandler.SendAsync(HttpRequestMessage , CancellationToken )    at Microsoft.Extensions.Http.Logging.LoggingHttpMessageHandler.SendAsync(HttpRequestMessage , CancellationToken )    at Microsoft.Extensions.Http.Logging.LoggingScopeHttpMessageHandler.SendAsync(HttpRequestMessage , CancellationToken )    at System.Net.Http.HttpClient.<SendAsync>g__Core|83_0(HttpRequestMessage , HttpCompletionOption , CancellationTokenSource , Boolean , CancellationTokenSource , CancellationToken )    at Testar.ChangeDetection.Core.ChangeDetectionHttpClient.LoginAsync(Uri serverUrl, LoginModel loginModel)    at BlazorApp.AuthService.LoginAsync(User user)
This issue blocks ability to log into the database and therefore the whole service is unusable.

This was mistake on my side. Database URL should be proxy URL instead, user error.


Question: Global variable replacement possible?

Subject of the issue
Is it possible to use "global variables" in the translate directive without passing it into the directive via translate-values?
We are using variable replacement with the translate directive using the translate-values attribute. By doing so, we pass a scope value into the directive which gets replaced.
What we are trying to do now, however, is to replace "global variables" which are static across our app so that we don't need to pass in the same variable to the translate directive in a multitude of places. We couldn't find anything like this in the docs and therefore would like to ask if there is anything available to accomplish this? If not, maybe you guys have thought about something like this before already and can share your thoughts?
Example:
"userprofile.popup.title.email_to_admin": "Send an email to {{ adminemail }}"

where {{ adminemail }} is globally defined across the app. You you might wonder why we don't just hard-code the email. The reason is that the value is different depending on the logged in user and the app area the user is authorized to use.
Your environment

angular-translate version 2.11.0
angular version 1.4.9
Ionic version 1.3.1

You should be able to use any scope value as a translate value. However, this will not work if any scope is isolated of course.
Said this, this will not work in filters as filters are not having access to scope by default since some AJS  versions (I think it was 1.3).

@knalli Thanks for your really fast reply! Let's say we'd call the above translation id 20 times from different places in the code like this:
userprofile.popup.title.email_to_admin

What we'd like to achieve is that we do not have to pass in the variable into the directive 20 times so that we could use it just like this:
userprofile.popup.title.email_to_admin

However, the key-value pair in our translation json file would still look like this:
"userprofile.popup.title.email_to_admin": "Send an email to {{ adminemail }}"

and {{ adminemail }} would be replaced by a global variable somewhere. In other words, we would like to get rid of padding in the same scope variable into the directive again and again.

My personal advice is not doing such "optimizations". It will help you some seconds of writing, but will require more time in the future for you and others to understand what is happening.
As I said already: You can use a reference to the scope, but this is still an explicit way.
I don't think anything more "global" (as you preferred) will come, because there another limitation:
If any global key has changed (and how should that be triggered?), every translation has to be refreshed. There is no information which translation uses which (global) variables or expressions with (global) variables.

Hello,
I'm closing this issue due inactivity. We have asked for additional details, but you could not helped us. Therefore we close this issue as not reproducible.
If you still have an issue, please create a new issue because we cannot be sure to track closed issues.


Optimizations

Mostly in the size of the fq files.

Nice, did this have much effect on the run times?

On Apr 16, 2015, at 2:45 AM, Panagiotis Vekris<EMAIL_ADDRESS>wrote:
Merged #124.
—
Reply to this email directly or view it on GitHub.

It speeds up most tests, including navier-stokes but the bottleneck were the expensive qualifiers to begin with, so not a dramatic change.
BTW, the latest merge in FP has broken RefScript. I temporarily use:
https://github.com/panagosg7/liquid-fixpoint
instead.

Can you tell me what is broken/breaks (in FP) ? I am working on a separate
branch so there shouldn't have been breaks...
On Thu, Apr 16, 2015 at 7:09 AM, Panagiotis Vekris<EMAIL_ADDRESS>
wrote:

It speeds up most tests, including navier-stokes but the bottleneck were
the expensive qualifiers to begin with, so not a dramatic change.
BTW, the latest merge in FP has broken RefScript. I temporarily use:
https://github.com/panagosg7/liquid-fixpoint
instead.
--
Reply to this email directly or view it on GitHub
https://github.com/UCSD-PL/RefScript/pull/124#issuecomment-93613893.

--
Ranjit.

Just send a snippet of the compile error?
On Thu, Apr 16, 2015 at 7:17 AM, Ranjit Jhala<EMAIL_ADDRESS>wrote:

Can you tell me what is broken/breaks (in FP) ? I am working on a separate
branch so there shouldn't have been breaks...
On Thu, Apr 16, 2015 at 7:09 AM, Panagiotis Vekris <
<EMAIL_ADDRESS>wrote:

It speeds up most tests, including navier-stokes but the bottleneck were
the expensive qualifiers to begin with, so not a dramatic change.
BTW, the latest merge in FP has broken RefScript. I temporarily use:
https://github.com/panagosg7/liquid-fixpoint
instead.
--
Reply to this email directly or view it on GitHub
https://github.com/UCSD-PL/RefScript/pull/124#issuecomment-93613893.

--
Ranjit.

--
Ranjit.

There were some commits in parallel to my last PR which might be relevant.
It's not a compile error. FP gives a bad constraint to almost all tests.
I haven't tried to fix it yet -- it might be sth trivial. Here's the fq file for negate (for example):
https://gist.github.com/panagosg7/f459b9612d33b71f0495

ok, i'll take a look!
On Thu, Apr 16, 2015 at 7:31 AM, Panagiotis Vekris<EMAIL_ADDRESS>
wrote:

There were some commits in parallel to my last PR which might be relevant.
It's not a compile error. FP gives a bad constraint to almost all tests.
I haven't tried to fix it yet -- it might be sth trivial. Here's the fq
file for negate (for example):
https://gist.github.com/panagosg7/f459b9612d33b71f0495
--
Reply to this email directly or view it on GitHub
https://github.com/UCSD-PL/RefScript/pull/124#issuecomment-93616705.

--
Ranjit.

hmm. odd, I think this may have to do with your changes to fixpoint.native
-- I have not touched it in a long time. Can you take a look?
On Thu, Apr 16, 2015 at 7:33 AM, Ranjit Jhala<EMAIL_ADDRESS>wrote:

ok, i'll take a look!
On Thu, Apr 16, 2015 at 7:31 AM, Panagiotis Vekris <
<EMAIL_ADDRESS>wrote:

There were some commits in parallel to my last PR which might be relevant.
It's not a compile error. FP gives a bad constraint to almost all tests.
I haven't tried to fix it yet -- it might be sth trivial. Here's the fq
file for negate (for example):
https://gist.github.com/panagosg7/f459b9612d33b71f0495
--
Reply to this email directly or view it on GitHub
https://github.com/UCSD-PL/RefScript/pull/124#issuecomment-93616705.

--
Ranjit.

--
Ranjit.


Missing License file in npm package

Hello, is it possible to incorporate the LICENSE file inside the npm package ? Now it's missing. It's implied by MIT license that if licensed under the MIT the source code must be accompanied by full license text. If the license file is not in the npm package a lot of people cannot use it.
Thank you

@floatdrop any chance you would get to this?

@honzajavorek published version 1.0.1 - now license should be in place.

Thank you so much!
On Fri, 31 Aug 2018 at 17:52, Vsevolod Strukchinsky <
<EMAIL_ADDRESS>wrote:

Closed #3 https://github.com/floatdrop/capture-stack-trace/issues/3.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/floatdrop/capture-stack-trace/issues/3#event-1820931200,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AARTMbHa3IOjjNTpD_-TijrFVY_jsxasks5uWVu_gaJpZM4Vp_Ci
.


OData functions substring() and replace() don't work with 3 parameters.

The following $filter parameters would not be parsed correctly because calls to substring() or replace() with 3 arguments don't work:
"$filter=replace('abcde', 'a', 'b') eq 'bbcde'"
"$filter=substring('abcde', 1, 2) eq 'bc'"

This issue is addressed here: https://github.com/auth0/node-odata-parser/pull/26

#26 was merged, so this is now fixed.


Input Format

Hi Mandar,
Thanks for this repo.
I am using the pretrained model. If I have a set of sentences in txt format how should I predict the coref. Is there any method to directly convert my sentences in txt format into the required format that model need as an input?

Hi Abhinandan,
I have't written a script for this, but it should be fairly simple. The Batched Prediction Instructions  section in the README has details about the format. Please let me know if you have questions.

On second thought, this might be a useful script that you might be able to use as a starting point. I'm closing this for now. Please feel free to re-open if you have more questions.
https://github.com/mandarjoshi90/coref/blob/master/gap_to_jsonlines.py

On second thought, this might be a useful script that you might be able to use as a starting point. I'm closing this for now. Please feel free to re-open if you have more questions.
https://github.com/mandarjoshi90/coref/blob/master/gap_to_jsonlines.py

Hi @mandarjoshi90
First of all thanks for this script . As @abhinandansrivastava  said even i had a text file where each row having text in tsv file .when i run this script i get the following key error
a_start, a_end =  datum['A-offset'],  datum['A-offset'] + len(datum['A']) KeyError: 'A-offset'
I would like to know how can I get the column A  and **A-offset **.
my input tsv file looks like this
index	Text
1	because where you need you just set the time to it . what you do . bell et one good . were you to censorious ? you think it again , to take nice for nifa a bit waiting to with bush you actually think you rhinoceros antigonish , which will never happen where you love .
2	the shires of toth bush today outcome with me . yes , that would be great . how about sunshade ? that trophies ? i know .

'

It would be really thankful if you can help me with this problem

Hi @mandarjoshi90,
could you provide some more info on that?
I would like to perform coreference resolution over custom data. So all I have are plain sentences.
As far as I can see, gap_to_jsonlines.py takes an input tsv and a vocab file. The tsv file already contains additional information (A-offset, B-offset, Pronoun-offset, etc) but it does not seem to be documented anywhere how to get this information or what information exactly that is. It is also unclear what format the vocab file should have.
I understand, this is not the main purpose of this code, but I think providing some documentation on how to get from plain custom text to the required input format for predict.py would be a great help for anyone who would like to apply your system on their own datasets. It certainly would be a great help for me :)

Hi Natalie,
I'll try to put something together. A quick answer is that you need to create a jsonlines file with the input tokenized using the BERT tokenizer and other information such as the (natural language) sentence and subword maps. See the example here for the format:
https://github.com/mandarjoshi90/coref#batched-prediction-instructions
The BERT-tokenized input text goes into the sentences key. Each element of "sentences" is a piece of tokenized text (possibly with multiple natural language sentences) less than 512 subwords. If your document has more than 512 subwords, you'll need to add more elements to sentences. You don't really need the pronoun offsets.
The model will add a key called predicted_clusters to the output jsonlines file where each element of the value is a cluster and each element of the cluster is a span in the input.
Please let me know if you have questions.


Show Serious adverse events using a different mark.

Use an "X" or an "S" instead of a circle.
See https://github.com/RhoInc/aeexplorer/issues/3 for original request.

I've added a second set of lines and marks for just SAEs.  The lines are thinner and black while the circles have a larger radius and a black stroke.

I like the direction this is going, but might tweak a few details.
How about using the fill instead of the stroke to indicated seriousness. fill-opacity:0 for non serious. fill-opacity:1 for serious? Maybe change the stroke-width to be a bit smaller as well? Then add a footnote that say "Filled circles indicate serious adverse events"

Also, the "end date" circles should probably just be hidden. I just created issue #20 for that.

I dig it.  Will test out on branch SAEs.

Implemented in v1.3.0.  Went with the black outline and added a legend item.


JSX: Error - Unexpected token <

espree 2.0.4
eslint 0.24.0
I am getting this error in SublimeLinter-contrib-eslint. In the console it seems to be running but I'm not 100% sure that my config is being applied and that is resulting in this error, so I'm not sure if it's my linter config, SublimeLinter config, or what. The console logs the following (the [...] are my replacing of long paths):
SublimeLinter: eslint: MyView.jsx ['[...]\\eslint.cmd', '--format', 'compact', '--stdin', '--stdin-filename', '__RELATIVE_TO_FOLDER__']
SublimeLinter: eslint output: [...]MyView.jsx: line 3, col 11, Error - Unexpected token <

Code:
var MyView = React.createClass({
	render: function() {
		return (
Coming Soon!
);
	}
});

My SublimeLinter configuration:
"default": {
    [...]
    "linters": {
        "eslint": {
            "env": {
                "browser": true,
                "jquery": true
            },
            "ecmaFeatures": {
                "jsx": true
            }
        }
    },
}

I guess the first question is: should this eslint configuration work, assuming SublimeLinter is working properly? Any other hints as to what might be wrong?

Running eslint on the command-line with a .eslintrc file in that folder works, so there seems to be a problem with my configuration and getting that passed to eslint.

You should use only .eslintrc configuration with plugin, SublimeLinter configuration does not allow add .eslintrc config there.

Okay. Is there a way to specify a path to a .eslintrc file on the command-line from the SublimeLinter config? Or use some sort of default path

Ah it looks like args might be what I need.

I am getting this same error in Sublime Text. What is the fix?

@cleechtech You should configure eslint to understand JSX http://eslint.org/docs/user-guide/configuring#specifying-language-options (jsx: true)

Ok thank you @roadhump


Create alumnosuls.txt

@abelorian I am sorry for a delay with processing your request. It is now merged. Thank you!


Install Script - htpasswd not installed on base system

I've tried installing using the script but it fails the step of using htpasswd to create an authfile.  How can I fix this?

This should install it:
apt-get install -y apache2-utils
It is, however, in the install script. So it should be available.


[keymgr, flash] question about finding flash seeds for keymgr

@tjaychen
I'm creating a top-level test for keymgr, which needs to get some secret seeds from flash.
I wait until flash init is completed, but seeds get all 1s. Should these seed be set to a constant random value?

BTW,  looks like hw2reg.status.init_wip.d has some glitches (de-assert in some cycles) before the init is done. Should we fix those glitches?
cc: @nikola-miladinovic @TimEwins @sriyerg

o that's not good on the init, thanks for catching that.
For the seed value, we don't directly backdoor load it right now in the environment. So the test I had worked on awhile back actually wrote the flash values in the test, rebooted and continued that way.  Do you want to try that? Of course backdoor loading in system verilog works too.

yep that's right! it does both flash and otp manipulation.

@weicaiyang are you going to try and re-create the keymgr output in software? If you do, it might make sense to probe from the SV side... because there are some constant values that are never exposed to software.

yes, that's the plan. Just discussed with Michael and Cindy, we need to probe OTP to get the root key. I will probably probe some internal keymgr key_state that aren't exposed to outside, and compare them as well.

why not probe directly on the keymgr boundary? or are you trying to pull it out of the otp image?
For the root key, assuming you follow the example test i gave, you actually know what it is, because you program it into the otp.
I was actually thinking more the internal constants, things you can't program that would make more sense to probe.

why not probe directly on the keymgr boundary? or are you trying to pull it out of the otp image?

I feel getting it from the otp image may be slightly better, as it's the source, in case it's not propagated to the boundary correctly.

For the root key, assuming you follow the example test i gave, you actually know what it is, because you program it into the otp.

I only see you lock the partition, where did you program root key? In dv test, we load the image, so we don't need to program it.
https://github.com/lowRISC/opentitan/blob/161f2c1f43b0f575b95da2c69b3022bf6f335e91/sw/device/silicon_creator/lib/drivers/keymgr_functest.c#L191

o oops, i kind of forgot, i lock it and just rely on some random value from
the otp image.
uhh..let me correct myself ..you COULD program it :)
but are you going to generate the expected value in software? or in sv land?
if it's the latter, then i guess reading backdoor via otp is probably way
easier.
On Thu, Mar 3, 2022 at 5:12 PM weicaiyang @.***> wrote:

why not probe directly on the keymgr boundary? or are you trying to pull
it out of the otp image?
I feel getting it from the otp image may be slightly better, as it's the
source, in case it's not propagated to the boundary correctly.
For the root key, assuming you follow the example test i gave, you
actually know what it is, because you program it into the otp.
I only see you lock the partition, where did you program root key? In dv
test, we load the image, so we don't need to program it.
https://github.com/lowRISC/opentitan/blob/161f2c1f43b0f575b95da2c69b3022bf6f335e91/sw/device/silicon_creator/lib/drivers/keymgr_functest.c#L191
—
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/issues/11235#issuecomment-1058727178,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2RSXTL5JBYHYKKXPTJILU6FPP3ANCNFSM5P3Z3SPQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were assigned.Message ID:
@.***>

o oops, i kind of forgot, i lock it and just rely on some random value from the otp image. uhh..let me correct myself ..you COULD program it :) but are you going to generate the expected value in software? or in sv land? if it's the latter, then i guess reading backdoor via otp is probably way easier.

Thanks for the clarification. I will probably just backdoor read the root key. I may try to call the C function to compute the kmac digest value in C code.

btw, i'd recommend invoking the c function from system verilog land if you can.
since you have to probe / backdoor read, this test is already not portable to other platforms. If that's the case, there's probably no reason to force yourself to do computation in C only on ibex.  That could take quite awhile.

btw, i'd recommend invoking the c function from system verilog land if you can. since you have to probe / backdoor read, this test is already not portable to other platforms. If that's the case, there's probably no reason to force yourself to do computation in C only on ibex. That could take quite awhile.

Make sense. Thanks for the suggestion.

closing this via #11238.


ImportError

We have written the needed data into your clipboard because it was too large to send. Please paste.

Problem with Python imports.


ci(github): fix type exports in packages/cacti-plugin-copm-fabric

Commit to be reviewed

ci(github): fix type exports in packages/cacti-plugin-copm-fabric
Primary Changes
---------------
1. Removed packages/cacti-plugin-copm-fabric/hyperledger-cacti-cacti-plugin-
copm-fabric-*.tgz in ignore paths in get-all-tgz-path.ts file
2. Added main, module, types, and files in package.json

Fixes: #3629
Pull Request Requirements

[x] Rebased onto upstream/main branch and squashed into single commit to help maintainers review it more efficient and to avoid spaghetti git commit graphs that obfuscate which commit did exactly what change, when and, why.
[x] Have git sign off at the end of commit message to avoid being marked red. You can add -s flag when using git commit command. You may refer to this link for more information.
[x] Follow the Commit Linting specification. You may refer to this link for more information.

Character Limit

[x] Pull Request Title and Commit Subject must not exceed 72 characters (including spaces and special characters).
[x] Commit Message per line must not exceed 80 characters (including spaces and special characters).

A Must Read for Beginners
For rebasing and squashing, here's a must read guide for beginners.

With the code changes, attw gives all green passing result

Apologies for the delay on pushing, I had issue with my device and the yarn run configure with fork in the past few days but I have updated my fork now to resolve merge conflict, thanks @petermetz


First stab at  mappings

Using a non-existent key stroke is great, but ambiguous
plug mappings aren't supported out of the box. Imagine you
have two mappings, both of which take a motion:
More
MoreMagic
You've mapped gmm to More and gmc to MoreMagic.
When you type gmc, the recursive mapping is available, and the
else if (mappingInfo != null) branch can feed the mapped keys
back into the system. Because these are fed as recursive, when
the keys come back through, they can get the mapping to MoreMagic
(which is no longer a prefix) and execute.
For gmm, when it feeds the keys for More, it is still
a prefix; when a motion is inputted, it is no longer a prefix,
and so it would hit the else block, feeding the keys WITHOUT
recursion. So, we have to catch the case where we're no longer
a prefix, but can find a  mapping within the input. This
seems to work pretty well, but causes "recursive records found"
warnings in the console, and complaints about local history being
broken. I'm not entirely sure whether this is just an idiosyncrasy
of the unit test environment, or whether this would cause
significant problems in the live environment. I'm hoping you have
more experience here about whether that warning can be ignored,
and am open to suggestions otherwise.
Also included is some prep work for -based repeat handling.
Extra input sequences past the  mapping are stored in a
PlugInputModel which is read from in preference to the
ModalInputDialog.

It seems like the "recursive records found" is also printed when I run the test on master, so either there's also a problem there... or it's no big deal.

Also, it seems the ambiguous mappings is not just an issue for <Plug> mappings. See my branch https://github.com/dhleong/ideavim/tree/dhleong/CommentaryExtension. That provides two mappings; gc and gcc. gc expects an operator, while gcc operates on the current line. Because extensions are handled as mappings, the same situation as described above happens—gce, though expected to be valid, is not a prefix and does not have a mapping, so it is ignored in handleKeyMapping. It would be handled if it were built on top of this branch and mapped everything to <Plug> mappings first, but I'm not sure if we should force that just to get extensions to work.
@vlasovskikh thoughts?

I'll look at this PR in more detail on the weekend. The general idea should be to emulate ambiguous mappings as close to Vim as possible.
At the moment I'm very concerned with VIM-1086.

I just added some more tests. The techniques here actually do handle ambiguous, direct (IE: not <Plug>) mappings, mostly because the KeyMapping#getPlug() method just looks for any extension handler.
There is still a bug if you do those sort of mappings with the <Plug> indirection, however. I'll look into that today.

Okay just pushed some changes that fix the test case I had.

I'm not sure about this weekend, but this is the next PR I'll look at.

@dhleong
I haven't read your code changes in detail but maybe consider enforcing that all plug names must start and end with '(' and ')' (and thus forbid them in the names themselves). It can at least remove ambiguity in plugs because you can't accidentally match some other plug which is a prefix of the .
Fixing prefix remappings is then a different problem and at least won't suprpise extension authors.

That's certainly one approach. I'm hesitant to break from Vim's behavior
here, but I'm not opposed to enforcing this best practice. Either way,
though, we have to handle the case where regular keymaps to
ExtensionHandlers are ambiguous, as well. This solution handles both cases
with the same mechanism, I believe.
On Thu, Feb 25, 2016 at 11:11 AM Bert Jacobs<EMAIL_ADDRESS>wrote:

@dhleong https://github.com/dhleong
I haven't read your code changes in detail but maybe consider enforcing
that all plug names must start and end with '(' and ')' (and thus forbid
them in the names themselves). It can at least remove ambiguity in plugs
because you can't accidentally match some other plug which is a prefix of
the http://stackoverflow.com/a/13698530/983949.
Fixing prefix remappings is then a different problem and at least won't
suprpise extension authors.
—
Reply to this email directly or view it on GitHub
https://github.com/JetBrains/ideavim/pull/105#issuecomment-188857310.

I'm going to restore the original Vim behavior in IdeaVim regarding ambiguous mappings with timeout / notimeout and then use only the part of your PR about <Plug> pseudo-character. Extension authors should not use ambiguous mappings, and if they do, they are responsible for the correct behavior of their extensions, similar to the original Vim.

As for getchar() and input() in the case of ambiguous mappings, I'm going to investigate options here. Maybe it's not worth fixing at all, at least until there is real demand from the extensions where their emulation requires ambiguous mappings and user input.

The comments extension requires ambiguous mappings. Doesn't use
getchar(), but it uses gcc to comment a line, and gc to comment a
text object. vim-exchange similarly allows cxx to select a line, and cx
to select a text object. Again, neither of these use getchar(), but
because the g@ stuff doesn't use the regular "argument" handling, this
common mapping strategy doesn't work without extra attention.
On Sun, Feb 28, 2016 at 6:38 PM Andrey Vlasovskikh<EMAIL_ADDRESS>wrote:

As for getchar() and input() in the case of ambiguous mappings, I'm going
to investigate options here. Maybe it's not worth fixing at all, at least
until there is real demand from the extensions where their emulation
requires ambiguous mappings and user input.
—
Reply to this email directly or view it on GitHub
https://github.com/JetBrains/ideavim/pull/105#issuecomment-189939453.

I still haven't managed to spend some time working on this PR yet. Hope things will get better the next weekend.

I've (mostly) restored the original Vim behavior for ambiguous mappings in 530846bc7ac03c1f533fc6317499d29214d021ca and added <Plug> for mapping commands in f280f8138d435100168a10007c9b7bf16231e412.
@dhleong I hope this is enough for your examples with gc and gcc. See the unit tests in 530846bc7ac03c1f533fc6317499d29214d021ca. Thank you for your ideas on handling <Plug>.


MSID: 90051 Version: 3 DOI: 10.1101/2<IP_ADDRESS>0570

MSID: 90051
Version: 3
Preprint DOI: https://doi.org/10.1101/2<IP_ADDRESS>0570
Step 1. Awaiting reviews
Editorial to post reviews via hypothesis
Useful links:

DocMap: https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=90051
New model tracking: https://docs.google.com/spreadsheets/d/1_fHaoOy7hjyocptKtVJRijeNpUY4hBS7Ck_aVmx6ZJk/
Reviews on sciety: https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>0570

For trouble shooting (e.g. no Docmaps available):

DocMap issue addressing: https://miro.com/app/board/uXjVNCwK6EI=/
Explore DataHub DocMaps API: https://lookerstudio.google.com/reporting/4c2f0368-babb-4beb-b5b3-497e7e7b0f08/page/ejphD
Unmatched submissions and preprints: https://lookerstudio.google.com/u/0/reporting/9f86204f-3bf7-477c-9b18-5c5ef141bf69/page/p_gxi57ha93c
Unmatched manuscripts spreadsheet: https://docs.google.com/spreadsheets/d/15QcK8w-ssB7109RQEDtFpJPZ0J5HTGxoHa_2TtpMBbg/edit#gid=1336081641

Step 2. Preview reviewed preprint
Production QC content ahead of publication
Instructions:

QC preview: https://prod--epp.elifesciences.org/previews/90051v3
Update ticket with any problems (add blocked label)
When QC OK, add QC OK label to ticket and add publication date and time to https://docs.google.com/spreadsheets/d/1amAlKvdLcaDp5W8Z8g77NmkwbMF5n_u89ArSqPMO8jg
Move card to next column
(At end of the day post link in #enhanced-preprint and ask for PDF to be generated)

Useful links:

Preprint DOI :  https://doi.org/10.1101/2<IP_ADDRESS>0570
Confirm reviews returned by EPP: https://prod--epp.elifesciences.org/api/reviewed-preprints/90051/v3/reviews
To update the MECA path in the docmap: https://docs.google.com/spreadsheets/d/1mctCQuNFBjSn97Lihy7_vBO6z7-N-oqyLv4clyi6zHg

Step 3: Awaiting search reindex
This step adds the reviewed preprint to the homepage: https://elifesciences.org
The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.
Useful links:

Jenkins pipeline to reindex search can be triggered sooner or monitored here: https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 4: Published! PDF requested
Waiting for PDF to be generated
Useful links:

PDF tracking: https://docs.google.com/spreadsheets/d/106_XeDjmuBae7gexOTNzg60lapeqjl2aRn9DzupGyS8/

Step 5: Introduce PDF to data folder and git repo
Upload PDF to relevent folder in git repo https://github.com/elifesciences/enhanced-preprints-data/
Step 6: Done!

[ ] Kettle is on!

elocations missing from references


flowtype support

Does this support Flow? Could this be used as a replacement for Flow's editor integration?

No. If you write Flow code, you're much better off using Flow's tools.

Thanks! Appreciate the quick response!

I would just like to add something to this for anyone who stumbled here through google...
While tern does not support flow, it might benefit you to still use it alongside the flow tools because the flow tools only work on package that were written in flow or converted to have an accompanying .js.flow type file with it. Valid Javascript is valid FlowType, but valid FlowType is not valid Javascript.
If you only use flow tools, you will run into a case where you want to import regular javascript and there are no flowtyped files for it which means you lose all of the tooling. Run them side by side and tern makes a nice fallback for when the flow tools cannot help you.

It would be good if tern could understand as extension a flow type of
import type {$Foo} from 'pkg';

and then use that info as well, my usecase is auto completion of tern in emacs and flow auto completion support there is terrible, tern works much better.


add event to emit logs for new anchor selection

In the first reveal, we emit an event that shows a new anchor hash.


Update 3/28 state action on homepage with readability edit

[x] Check readability on homepage state action blurb recently posted by Kalin Kipling (CalEPA)
[x] Edit
[x] Get approval for edits from Kalin
[x] Stage
[x] Check
[x] Publish
[x] Check

Published 10AM 3/29/22.


Add feature to have multiple endpoints

Since the MAX-Object-Detector now supports multiple models. I updated the web application to support multiple endpoints for the MAX model.
The app can be started with multiple endpoints using the command:
node app --model=[endpoint1] --model=[endpoint2]
The application now has a dropdown menu to switch between the model endpoints configured.

After discussion with @djalova and @xuhdev I will be closing this as "Will Not Fix" since it breaks the design of the application, it's extremely "hacky", and is only wanted for a customer POC, not as feature of the production instance.


Display geolocalized points in itowns

exported as txt with PSQL, converted to las with PDAL and loaded into PostgreSQL with lopocs:
it sounds good !

@mbredif @elemoine i think we can add this for demo

That's great! I'll add a screenshot to the report.


Deploy on ENS+IPFS so that callthis.eth.link works

Will eventually be on https://callthis.eth.link

This is done, though callthis.eth.limo is more reliable.


chore: update dependencies

Codecov Report
Attention: 10 lines in your changes are missing coverage. Please review.

Comparison is base (015afa4) 46.00% compared to head (abaf165) 24.49%.

Files
Patch %
Lines

stories/components/panel-header.stories.tsx
0.00%
3 Missing :warning:

src/app-data/loaders/biologicLoader.ts
50.00%
2 Missing :warning:

src/app-data/loaders/spcLoader.ts
0.00%
2 Missing :warning:

playwright-base.config.mts
0.00%
1 Missing :warning:

playwright-ct.config.mts
0.00%
1 Missing :warning:

playwright-e2e.config.mts
0.00%
0 Missing and 1 partial :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##             main     #601       +/-   ##
===========================================
- Coverage   46.00%   24.49%   -21.52%
===========================================
  Files         135      223       +88
  Lines        6871    12954     +6083
  Branches      146      234       +88
===========================================
+ Hits         3161     3173       +12
- Misses       3710     9693     +5983
- Partials        0       88       +88

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


Reduce package size

My surprise was immeasurable when I saw that:

Uh, a package of 175MB????? Maybe ffmpeg is included? Nope, it says in the README that it should be made available by ourselves...
And when investigating, I found that Yarn's cache was included mistakenly in the final bundle. This is why I propose the following change:

Remove the .npmignore: this will allow NPM to fallback to the default behavior of using the .gitignore file for ignoring stuff. This has the advantage of being easier to maintain, and prevent mistakes in the future
Add a "files" key in the package.json, that will only include those files (and a few others) and ignoring the result of the first bullet point

In the end, I'm not 100% sure everything is good with it, but the whole cjs and es folders are included, with the package.json, the README.md, the CHANGELOG.md and the LICENSE, which should be enough for releasing a package on NPM.
NB: I also added the types entry under exports, for better automatic import management using VSCode and others in some situations.

Good stuff! Thanks and sorry for taking so long to get this in.


UnhandledFutureError is hard to debug

Currently, we don't offer good / any debugging tools to debug and fix unhandled future errors. This makes them annoying and very hard to find and fix.

See https://github.com/ScriptFUSION/Async-Throttle/commit/83c311a747d63c39a788f7e6f3da7ce66eeee8d7.

Closed via #405.


Create a lookahead method for the camera

Some players like to see where they're going.
Others prefer the precision of a fixed camera.
When players are on a platform, they probably want to look down before they jump.

The built-in camera2d in godot doesn't have a lookahead method, but it has nice drag margins.
I've parented the camera to a node2d which will serve as the lookahead camera target.
Now I just need to get that target to go places that make sense to players.
Bad lookahead is worse than no lookahead, so I might just leave a fixed camera.

https://www.youtube.com/watch?v=TdWFzpgnljs&list=PLc38fcMFcV_t66OnpNFFXKIqsQRPVHq6U

works ok without camera lookahead, after we reduced the size of the player onscreen.


TFLite -- Converting the trained pb file into tflite failed predictions on android

Question description：
training a pb file, and then convert to tflite file, i use python testing pb file is correct ,but after into tflite file on android testing is wrong, I think this problem should be related to BN(tf.nn.batch_normalization),because when i remove BN get results of android and python are all the same, but with BN is different, the BN is tflite support,and the input data are the same, I don't know why?

Test Demo:
def save_to_pb(sess):
    constant_graph = graph_util.convert_variables_to_constants(sess, sess.graph_def, ['out'])
    with tf.gfile.FastGFile(MODEL_DIR + 'expert-graph.pb', mode='wb') as f:
        f.write(constant_graph.SerializeToString())

def read_data(session):
    image_name = '182133.jpg'
    TEST_IMAGINE_PATH = '/home/leve/lcr/ClelebA/Celebra_crop_20w+_w128_dataset/images/test_100/'
    image = cv.imread(TEST_IMAGINE_PATH + image_name)
    image = tf.reshape(image, [128, 128, 3])
    image = image.eval(session=session)
    image = image[np.newaxis, :]
    return image

def batch_norm_lite(x, train=True, bn_decay=0.5,epsilon = 0.001,name='bn'):
    is_training = tf.convert_to_tensor(train,dtype='bool',name='is_training')
    x_shape = x.get_shape()
    params_shape = x_shape[-1:]
    axis = list(range(len(x_shape) - 1))
    beta = tf.get_variable(name+'_beta', params_shape, initializer=tf.zeros_initializer())
    gamma = tf.get_variable(name+'_gamma', params_shape, initializer=tf.ones_initializer())
    moving_mean = tf.get_variable(name+'_moving_mean', params_shape, initializer=tf.zeros_initializer(), trainable=False)
    moving_variance = tf.get_variable(name+'_moving_variance', params_shape, initializer=tf.ones_initializer(), trainable=False)
    mean, variance = tf.nn.moments(x, axis)
    update_moving_mean = moving_averages.assign_moving_average(moving_mean, mean, bn_decay)
    update_moving_variance = moving_averages.assign_moving_average(moving_variance, variance, bn_decay)
    tf.add_to_collection(name+'_update_moving_mean', update_moving_mean)
    tf.add_to_collection(name+'_update_moving_variance', update_moving_variance)
    mean, variance = control_flow_ops.cond(
        is_training, lambda: (mean, variance),
        lambda: (moving_mean, moving_variance))

    return tf.nn.batch_normalization(x, mean, variance, beta, gamma, epsilon,name=name)

def train_model():
    img = tf.placeholder(name="img", dtype=tf.float32, shape=(1, 128, 128, 3))
    b = tf.Variable(tf.truncated_normal((1, 128, 128, 3), seed=1),name='w1')
    y_real = img + tf.constant([1., 2., 3.]) + tf.constant([1., 4., 4.])
    val = tf.add(img, b)
    val = batch_norm_lite(val)
    out = tf.identity(val, name="out")
    MSE = tf.reduce_mean(tf.square(y_real - out), name='mse')
    train_step = tf.train.GradientDescentOptimizer(0.9).minimize(MSE)
    saver = tf.train.Saver(max_to_keep=10)
    os.environ["CUDA_VISIBLE_DEVICES"] = ''
    config = tf.ConfigProto()
    config.gpu_options.per_process_gpu_memory_fraction = 0.1
    config.gpu_options.allow_growth = True
    with tf.Session() as sess:
        sess.run(tf.initialize_all_variables())
        for epoch in range(100):
            sess.run(train_step,feed_dict={img:read_data(sess)})
            if epoch % 20 == 0:
                save_path = saver.save(sess, MODEL_DIR + MODEL_NAME, global_step=epoch + 1)
            print('step = ' + str(epoch))
        save_to_pb(sess)
train_model()

run upper  code and get pb file, when you test pb file with python you maybe get a result like this:
39.09046 45.927864 84.797905 45.952957 62.701195 68.00796 41.789146 80.99372 81.67459 81.2203....
then run this command to create tflite file:
bazel run --config=opt tensorflow/contrib/lite/toco:toco -- --input_file=/path/expert-graph.pb --output_file=/path/expert-graph.tflite --input_format=TENSORFLOW_GRAPHDEF --output_format=TFLITE --input_shape=1,128,128,3 --input_array=img --output_array=out --inference_type=FLOAT --input_data_type=FLOAT --allow_custom_ops

you will get expert-graph.tflite,then use tflite file to android,will get result like this:
49.392387， 42.290344，135.30707， 7.7764254 ， 125.98051，79.317825 ，157.29184 ，107.58522 ， 283.46997 ，36.103508....
Android and python have the same input data,but the result is different,if i remove BN and train, that's ok,but BN is very important,i can remove it ,i don't know how to solve it,this question has been around me for a long time,please help solve it, if don't use TF Lite, what should i do?Thanks

System information
Linux Ubuntu 16.04:
TensorFlow installed from source:
TensorFlow version 1.8.0:
Python version:3.6:
Bazel version 0.11.1:
GCC/Compiler version 5.4.0 20160609 (Ubuntu 5.4.0-6ubuntu1~16.04.10) :
CUDA/cuDNN 9.0/7.0.5:
GPU 1060-6G:

@longchr123  Can you include the test data and your implementation of custom op: SquaredDifference ?

@shashishekhar  there is no test data for training, If i don't use --allow_custom_ops in bazel command,there will be errors,like this:
Running command line: bazel-bin/tensorflow/contrib/lite/toco/toco '--input_file=/home/leve/lcr/model_result/tflite_test/expert-graph.pb' '--output_file=/home/leve/lcr/model_result/tflite_test/expert-graph.tflite' '--input_format=TENSORFLOW_GRAPHDEF' '--output_format=TFLITE' '--input_shape=1,128,128,3' '--input_array=img' '--output_array=out' '--inference_type=FLOAT' '--input_data_type=FLOAT'
2018-07-18 10:50:45.979461: I tensorflow/contrib/lite/toco/import_tensorflow.cc:1265] Converting unsupported operation: SquaredDifference
2018-07-18 10:50:45.979688: I tensorflow/contrib/lite/toco/graph_transformations/graph_transformations.cc:39] Before Removing unused ops: 27 operators, 41 arrays (0 quantized)
2018-07-18 10:50:45.979826: I tensorflow/contrib/lite/toco/graph_transformations/graph_transformations.cc:39] Before general graph transformations: 27 operators, 41 arrays (0 quantized)
2018-07-18 10:50:45.980017: I tensorflow/contrib/lite/toco/graph_transformations/graph_transformations.cc:39] After general graph transformations pass 1: 13 operators, 21 arrays (0 quantized)
2018-07-18 10:50:45.980086: I tensorflow/contrib/lite/toco/graph_transformations/graph_transformations.cc:39] Before dequantization graph transformations: 13 operators, 21 arrays (0 quantized)
2018-07-18 10:50:45.980144: I tensorflow/contrib/lite/toco/allocate_transient_arrays.cc:329] Total transient array allocated size: 196736 bytes, theoretical optimal value: 196672 bytes.
2018-07-18 10:50:45.980271: F tensorflow/contrib/lite/toco/tflite/export.cc:315] Some of the operators in the model are not supported by the standard TensorFlow Lite runtime. If you have a custom implementation for them you can disable this error with --allow_custom_ops. Here is a list of operators for which you will need custom implementations: RSQRT, SquaredDifference.

how to sovle it?

Ah, I am going to explain what the error means:
First some background, TFLite doesn't have all the operators available in Tensorflow, we are working on improving and adding more operator support. When you convert your model the tool tells you if there are operators which don't have implementations. Here the tool is mentioning that RSQRT, SquaredDifference may need custom implementations.
I am surprised the model didn't crash, I will take a look and see if these ops were used only for training, usually toco strips out the ops used for training.

Nagging Assignee @shashishekhar: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.


Reducer unit tests

The reducers in the Redux section of the app should be unit tested to ensure they are functioning correctly.

https://github.com/reactjs/redux/blob/master/docs/recipes/WritingTests.md

@dan1elhughes  would you say this has been completed by #95 ?

As recommended by @ShaunBulbrook I'm currently using the coverage tool by Jest to fill in any gaps I feel are required to complete this task.

I've tackled this one as much as I can (see #136 for a PR). Now developers should be able to do npm test-watch -- --coverage to see where there are gaps for writing unit testing.
The biggest gaps are for more complicated actions/reducers and the API calls, which may need the original author to look into writing unit tests... @dan1elhughes where do you think we should take it from here?


Restart drum

I need to restart the plugin for some cases depend on something, is there any way for this ?

when click Reset button, i need to restart default value

If I understand your requirement (like my own)... you need to reset this:
element.data('drum', null)
Have a look at drum.js file and find line 331
if (!element.data('drum')) {
This line is getting in your (our way).  In my case I have one drum options filter the list shown in another drum.


List: Pagination

The ListPart doesn't seem to have a pager when it is rendered.
Also, it would be interesting to have a definition setting to be able to specify the number of items per page (additionally to the 'PageSize' general setting).

See PR #428


Automatically generate proto files for Python

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[ ] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[ ] Make sure tests pass via mvn clean verify.
[ ] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

R: @chamikaramj

Changes Unknown when pulling 7760577df8df6be4ae294188ed85e999def9f1f2 on robertwb:gen-proto into ** on apache:master**.

apache_beam/runners/portability is a runner implementation like any other. I don't think users should be importing anything from apache_beam.core. We could have made apache_beam.sdk, but it's too late for that.


Memory error in compute_dist_mat.py

The size of embedding_matrix in 30050001 due to which there is a memory error while doing -2np.dot(embedding_matrix.T , embedding_matrix).
How do I fix this?

How much memory you have in your machine ? The computed matrix is expected to be large and require a lot of memory. The size of output file size is approximately 20 GB. Also, I would recommend using python 3.6.

I have 64GB memory available. Still I get the following error:
Traceback (most recent call last):
File "compute_dist_mat.py", line 37, in
c_ = -2*np.dot(embedding_matrix.T , embedding_matrix)
MemoryError

Is it 32 bits or 64 bits python ? I would recommend using 64 bits to handle  such a huge array.

It is 64 bit. The error is resolved though.
I changed the python version from 3.6.0 to 3.6.8 and it worked.
Thanks!

I have similar problem and I have python 3.6.9 (which is a 64 bit system). I have a 25gb RAM and 107gb of disk memory. Does the post above refer to 64gb of RAM ? Any idea of how to fix this?
2020-06-04 00:45:38.006929: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcudart.so.10.1
Using TensorFlow backend.
tcmalloc: large alloc<PHONE_NUMBER>8 bytes == 0xca0a000 @  0x7f1b981671e7 0x7f1b95d0d5e1 0x7f1b95d71c78 0x7f1b95d71e24 0x7f1b95d6874c 0x7f1b95fa528c 0x7f1b95e04c10 0x7f1b95e0955a 0x566d63 0x59fc4e 0x7f1b95d5d4ed 0x50a2bf 0x50bfb4 0x507d64 0x509a90 0x50a48d 0x50bfb4 0x507d64 0x50ae13 0x634c82 0x634d37 0x6384ef 0x639091 0x4b0d00 0x7f1b97d64b97 0x5b250a


Type of childs of query

As of now, the type of query childs is mutable Queue of the standard library. Should I change it to immutable one?

We changed to use Map, instead of Queue, for efficient lookup.


[Question]: will you add another argument(dest_path, etc...) as the path that the complete.html stored

when complete.html stored in the allure-report folder, allure web will not worked as well.
Great works, thanks.

What do you mean on "allure web will not worked as well"?
Adding parameters allowing to save output file to other location is one of planned future enhancements.
If you have some specific problem right bow, you can change path to "complete.html" right in the code, it's easy.

Line to change the output path is here https://github.com/MihanEntalpo/allure-single-html-file/blob/main/combine.py#L241

Hi, @MihanEntalpo
I made a pull request, thanks.
https://github.com/MihanEntalpo/allure-single-html-file/pull/9

closed


first attempt at cmake build system

review? @lukewagner
this builds the two c++ executables, but doesn't build the emscriptenized load-wasm-worker.js file or remove anything from the Makefile.

Do you expect you'll add the emscriptenized load-wasm-worker.js feature eventually? (In other words, is this the first step toward implementing the comment in the first line of https://github.com/WebAssembly/polyfill-prototype-1/blob/master/Makefile ?

Not sure about the js exectubale regarding best practices of having compiler specific build targets.  I'll have to do some research, but I think it's doable.  @juj probably knows.

Sure, it's possible to have CMake target .js or .html output files. See the CMake tests in Emscripten tests/ directory. Note that it is not a project-specific property, but Emscripten is a platform to target, just like Windows, Linux or OS X. That means that one CMakeLists.txt file can't contain two projects that would target different platforms, e.g. one that would target native, and another that would target JS, but the choice of which one to compile for is done at command line by choosing the appropriate target toolchain file (for Emscripten, that's running with emcmake cmake xxx instead of just cmake xxx)

so basically, the cmake build script would have to be run twice; once to build the C++ code, and once to build the JS code, possibly removing the cmake cache in between?  Does that sound correct @juj?

Yes, or set it to build to two different directories, so they can each have their own cache files and directory structures.

or set it to build to two different directories
So I would have to move the source files into their own directories each with their own CMakeLists.txt?

No, there's no need to move anything. CMake is not restricted to making builds to the root directory of the project, but good CMake practices employ what they call "out-of-source" builds, see http://www.cmake.org/Wiki/CMake_FAQ#What_is_an_.22out-of-source.22_build.3F . This is intended to exactly cater to the case when one wants to simultaneously keep builds of the same project with multiple different configurations (e.g. different target platforms, different build options, like debug vs release, or simd enabled vs not, or anything else custom).

On vacation atm, but when everything works, @sunfishcode feel free to do the merge.


Docs typo tracking 2

Doc typos aren't urgent to fix and it's more efficient to batch the fixes.
We track the fixes here and when there's enough we can fix them in a single PR.
Just mention this issue number in other issue to add typo or add a comment with exact file & line.
Replaces #828

/// This is an integer newtype representing weigth in wu. It provides protection against mixing
in weight.rs


[ISSUE] Add pg_hba.conf override in timescaledb-multinode chart

Describe the bug
No clear way to override pg_hba.conf
Expected behavior
A yaml node in values.yaml for setting the pg_hba.conf contents
Additional context
I would like to be able to enable ldap authentication for timescaledb-multinode

Up.
I also need this configuration.

Samething for me! Did anyone find the solution?

Also please.

We are sorry to say that multinode helm chart is deprecated and not supported anymore. If you want to see this chart supported then we are looking for community maintainers - https://github.com/timescale/helm-charts/tree/master/charts/timescaledb-multinode#call-for-maintainers

I am closing this issue as TimescaleDB-Multinode helm chart is no longer maintained and deprecated. We are looking for potential community maintainers who could help us in getting that helm chart working again. If you wish to become a maintainer, please contact us on slack.
Keep in mind that TimescaleDB is not and will not be dropping multinode setup. Deprecation is only about helm chart.


Image deprecations and optimisation

Consolidates thumbnail methods to use the common image manipulation API, to prevent unnecessary creation of identical images with different filenames. Generate methods also tagged for removal in 4.0.

Have reduced this a bit - AssetLibraryPreview and AssetLibraryThumbnail don't seem to be used anywhere in Framework or CMS and are not present in 4.0 so no sense introducing new API. I notice the thumbnail config variables are a bit of a mess and used in unintended ways but I see you've addressed that in 4.0 (Thanks!). So it's really just StripThumbnail getting it's own method now, and a few deprecation notices.


Add no-ignored-test-files rule (fixes #51)

Add no-ignored-test-files rule (fixes #51)
I would like to add more checks to see whether the test is in the AVA glob (with an option to de-/activate or set a custom glob), but I haven't found (yet, haven't searched much for now) a library that simply tells me whether a file matches a glob, and I don't know if we can reliably get the cwd from ESLint (or the editor running ESLint).
This can be added later to the rule though. I might take a look at it later, or someone could take a look too if interested.

I would like to add more checks to see whether the test is in the AVA glob (with an option to de-/activate or set a custom glob), but I haven't found (yet, haven't searched much for now) a library that simply tells me whether a file matches a glob

The watcher does this, see https://github.com/sindresorhus/ava/blob/005b656b9950ab3fa1e2bd9af326d5b39540e71b/lib/watcher.js#L333:L379. It uses https://www.npmjs.com/package/multimatch.

From what I could test, in Atom's ESLint plugin (the Atom plugin that runs ESLint I mean), process.cwd() returns the current file's folder everytime, not the root of the project, and ESLint's context.getFilename() returns the file's absolute path. Meaning it becomes pretty hard to figure out the project's root folder.
So it means two things:

Globbing becomes pretty hard, because even the default glob is relative to the folder Ava is run from.
Anybody who uses eslint-plugin-ava (at least in Atom) will have this rule ringing loudly for every file if his project, dev folder, or any higher-up folder is named fixtures/helpers.

The second is obviously pretty rare (I'd think at least), but it can be problematic for some users. Let me know what you guys think about this.

@jfmengels could use https://www.npmjs.com/package/pkg-up until you find a package that depends on AVA?

@novemberborn Sounded good, so I tried it. I disabled the rule altogether progammatically when no package.json is found (debatable, but probably pretty rare anyway).
The result is pretty rad IMO!

Do people put test factories in their helpers? That might be a legitimate ava import. Though you could disable the rule in that case or customize the pattern.

Very cool @jfmengels!

@novemberborn @jamestalmage had the same comment https://github.com/sindresorhus/eslint-plugin-ava/issues/51#issuecomment-201146646, and I agree that you should then simply disable the rule. There was an interesting proposition https://github.com/sindresorhus/ava/issues/695 for test macros which would be better.

Nothing further from me :dancers:

LGTM

Cool, I'll squash and merge then.

@jfmengels - FYI - GH has a new "Squash and Merge" feature built in. Just click the merge button.
Ideally you would use that, even for 1 commit PR's. It avoids creating an extra merge commit.
If there are multiple commits that you want to keep separate, the old merge method is fine.

Aah. Yeah I knew about it, but I thought it was something that you needed to activate as I didn't see it. But it only shows when you click the merge button once apparently ^^'
(Hoping for a rebase button to come soon)

You usually only need to rebase if there is a merge conflict (in which case you probably want to do some edits / testing locally).  But that is a rare enough need I'm not clamoring for it. I would much rather see improvements to issue management first.

I mean rebase in the sense of no merge commits. I like straight lines.

I mean merge rebase in the sense of no merge commits. I like straight lines.

That's exactly what the squash button does, but you seems you used the old merge button.


CLI does not work with Bun

Issue description
CLI does not work with Bun
Expected Behavior
bunx --no node-llama-cpp chat --model ./models/llama-2-7b.Q4_K_M.gguf
model should load correctly and enter a chat
Actual Behavior
bunx --no node-llama-cpp chat --model ./models/llama-2-7b.Q4_K_M.gguf
Returns: Not enough non-option arguments: got 0, need at least 1
Steps to reproduce
bun install node-llama-cpp
bunx --no node-llama-cpp chat --model ./models/llama-2-7b.Q4_K_M.gguf
My Environment
Ubuntu 22                  |
"node-llama-cpp": "^2.8.5",
Additional Context
It would be great if this module works with bun
bun is super fast and let's us also use commonjs modules in the same script alongside esm import-only modules like node-llama-cpp
Relevant Features Used

[ ] Metal support
[ ] CUDA support
[ ] Grammar

Are you willing to resolve this issue by submitting a Pull Request?
Yes, I have the time, and I know how to start.

@drschwabe node-llama-cpp CLI works with Bun.
bunx doesn't support the --no parameter, so you should omit it when using bunx.
You should use the CLI like this:
bunx --bun node-llama-cpp chat --model ./models/llama-2-7b.Q4_K_M.gguf


Fix NuttX build after dev/socket merge

i dropped the ci bits for now as it was more complex than i expected.
i will retry sooner or later. (probably in a separate PR)


Add implicit type conversions for JdbcResultSet when possible

For example, if we have a column of type int, and then we call getLong, then currently we will get a ClassCastException but in fact, it should safely convert it to long

Perhaps it's implemented in IMDG under com.hazelcast.sql.impl.type.converter package


Add mobile login url

** Please provide as much detail as possible, but feel free to remove irrelevant sections **
Fix Jobstreet SG Header not clickable

duplicated with #461


deps: use peft==0.6.0

0.6.0 was finally releases a few days ago: https://github.com/huggingface/peft/releases/tag/v0.6.0
This will make it possible to publish caikit-nlp on pypi :tada:

Closing in favor of #265


Add dependency object

implement feature https://dev.icinga.org/issues/12373

implement feature https://dev.icinga.org/issues/12373

Merged, Thanks 👍


Revolvers don't unload.

Revolvers only unload a single round from the chamber, instead of fully unloading.
Bug could probably exist with shotguns too, but confirmed with revolvers.

Unable to reproduce.


Use AWS region from AWS config by default

Read from ~/.aws/config to get user's default region rather than hard-coding aws_region to us-west-2.

This will require making AMI IDs dynamic based on region.

This is an example of a dynamic AMI data object:
data "aws_ami" "ubuntu" {
  most_recent = true

  filter {
    name   = "name"
    values = ["ubuntu/images/hvm-ssd/ubuntu-focal-20.04-amd64-server-*"]
  }

  filter {
    name   = "virtualization-type"
    values = ["hvm"]
  }

  owners = ["099720109477"] # Canonical
}

More details here: https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/instance#example-usage

Many users do not use ~/.aws/config, and others do not want their infrastructure built in their default region. For instance my default region in ~/.aws/config is us-east-2, but I only build in us-west-2 as that is where most people in the org are.


Connect the warehouse database to metabase

Metabase should connect to the warehouse database using a readonly user.
Connect metabase to the data warehouse database using that user (and perhaps a service account should be created in openshift to do this?)

Please add your planning poker estimate with Zenhub @dleard

This network policy was created manually due to apparent Argo / Helm lookup problems. The database has been connected to Metabase, but it's 17:37 on a Friday and I don't see a schema yet. I'll let the jobs run and see if that solves it before marking complete.

@mikevespi is this complete?


Working on the page skeleton

Added the background, list components and logo.

Added the background, list components and logo.

@godsakani I need a review from you here, check the output. A running version can be seen here 🤌


SourceSpec Type default to empty

This is a follow up of this PR comment. As reported in camel-k-runtime, we may experience some issue because the value of SourceSpec type is default to empty in several parts of Camel K. We assume that empty value to be a source type on the runtime.
Should we enforce the SourceType to a value instead? I think it would be cleaner.
cc @lburgazzoli @astefanutti @nicolaferraro

can we set the default value on runtime ?

can we set the default value on runtime ?

I think we can. Does that mean that we assume that a default values is always a source type?

I'm fine with that


Hot Reloading and System.Runtime.InteropServices.NativeLibrary

Describe the bug
Not sure if this is a bug or something this library cannot support. I was trying to reproduce #146 using the hot-reload sample. There we use Microsoft.Data.Sqlite to test shadow copying and loading of sni.dll. When expandig the sample by creating an instance of SqliteConnection, there is a lot more happening under the hood. Particulary in SQLitePCL.raw which uses system.runtime.interopservices.nativelibrary.load to load native libraries. This API does not invoke AssemblyLoadContext.LoadUnmanagedDll. Hence, this library does not control how those native libraries are loaded, and cannot perform any shadow copying etc. That is, if my understanding is correct.
To Reproduce
Steps to reproduce the behavior:

Add using var dbConnection = new SqliteConnection("Data Source=db.sqlite"); to the InfoDisplayer in the hot-reload sample
Run samples/hot-reload/run.ps1|sh
See error System.DllNotFoundException: Unable to load DLL 'e_sqlite3' or one of its dependencies: The specified module could not be found. (0x8007007E)

Expected behavior
Would be nice if there is a solution to this issue, but I can understand that there might not be.
Additional context
Full stack trace:
Unhandled exception. System.Reflection.TargetInvocationException: Exception has been thrown by the target of an invocation.
 ---> System.TypeInitializationException: The type initializer for 'Microsoft.Data.Sqlite.SqliteConnection' threw an exception.
 ---> System.Reflection.TargetInvocationException: Exception has been thrown by the target of an invocation.
 ---> System.DllNotFoundException: Unable to load DLL 'e_sqlite3' or one of its dependencies: The specified module could not be found. (0x8007007E)
   at System.Runtime.InteropServices.NativeLibrary.LoadByName(String libraryName, QCallAssembly callingAssembly, Boolean hasDllImportSearchPathFlag, UInt32 dllImportSearchPathFlag, Boolean throwOnError)
   at System.Runtime.InteropServices.NativeLibrary.LoadLibraryByName(String libraryName, Assembly assembly, Nullable`1 searchPath, Boolean throwOnError)
   at System.Runtime.InteropServices.NativeLibrary.Load(String libraryName, Assembly assembly, Nullable`1 searchPath)
   at SQLitePCL.Batteries_V2.MakeDynamic(String name, Int32 flags)
   at SQLitePCL.Batteries_V2.DoDynamic_cdecl(String name, Int32 flags)
   --- End of inner exception stack trace ---
   at System.RuntimeMethodHandle.InvokeMethod(Object target, Object[] arguments, Signature sig, Boolean constructor, Boolean wrapExceptions)
   at System.Reflection.RuntimeMethodInfo.Invoke(Object obj, BindingFlags invokeAttr, Binder binder, Object[] parameters, CultureInfo culture)
   at Microsoft.Data.Sqlite.Utilities.BundleInitializer.Initialize()
   at Microsoft.Data.Sqlite.SqliteConnection..cctor()
   --- End of inner exception stack trace ---
   at Microsoft.Data.Sqlite.SqliteConnection..ctor(String connectionString)
   at TimestampedPlugin.InfoDisplayer.Print()
   --- End of inner exception stack trace ---
   at System.RuntimeMethodHandle.InvokeMethod(Object target, Object[] arguments, Signature sig, Boolean constructor, Boolean wrapExceptions)
   at System.Reflection.RuntimeMethodInfo.Invoke(Object obj, BindingFlags invokeAttr, Binder binder, Object[] parameters, CultureInfo culture)
   at System.Reflection.MethodBase.Invoke(Object obj, Object[] parameters)
   at HostApp.Program.InvokePlugin(PluginLoader loader) in C:\Source\private\DotNetCorePlugins\samples\hot-reload\HotReloadApp\Program.cs:line 39
   at HostApp.Program.Main(String[] args) in C:\Source\private\DotNetCorePlugins\samples\hot-reload\HotReloadApp\Program.cs:line 22
   at HostApp.Program.<Main>(String[] args)

Well, what did work in this specific scenario was to explicitly load e_sqlite3.dll by calling LoadUnmanagedDll("e_sqlite3") during initialization of ManagedLoadContext. Not exactly sure why that worked. Probably due to my limited knowledge of how loading of managed/unmanaged DLLs actually works.
My first thought was that e_sqlite3.dll could be found if it existed within the shadow copy directory. However, I've been unable to confirm it.
Anyways. It seems like this would be solvable with an API to "pre-load" specific unmanaged DLLs. ManagedLoadContext would then probably have to include some clean-up logic to free the pre-loaded handles.

The public API suggestion above was abandoned. Looking for solutions where the host does not have to care about the dependencies of plugins, instead.
NativeLibrary.SetDllImportResolver(Assembly, DllImportResolver) looks promising. However, havn't been able to solve the SQLite scenario using it yet.
Also, seems like the behavior of NativeLibrary.Load(String, Assembly, DllImportSearchPath?) is changed in .NET 5 (dotnet/runtime#13819), and will call AssemblyLoadContext.LoadUnmanagedDll(String) going forward.

Closing this. Not possible to fix for .NET Core.
Also, this issue is a duplicate of #84 (sorry about that).


Assertion is not within its valid time range.

Using code based on your TodoListController.cs where the only differences are no token cache and the way I am retrieving the first access token since Azure itself does the authentication (configured in the Azure portal) I am sometimes receiving the following error: AADSTS50013: Assertion is not within its valid time range. I think always after leaving the session open for a while. If a new session is started, for example with incognito mode, then it will work again.
These differences in of themselves should to my knowledge not cause an error like this. I've also found very little information online about this. I am using Microsoft.IdentityModel.Clients.ActiveDirectory version 3.19.4.
Can you please nudge me in the right direction for solving this?
This is my current code to get an on behalf token based on your TodoListController.cs:
        protected async Task<string> GetOnBehalfOfToken(string tenant, string clientId, string appKey, string resourceId)
        {
            // Get access token from header.
            string accessToken = Request.Headers.FirstOrDefault(fod => fod.Key == "X-MS-TOKEN-AAD-ID-TOKEN").Value?.FirstOrDefault();
            if (accessToken == null)
            {
                throw new Exception("No access token found.");
            }

            // Get username from claims.
            string userName = ClaimsPrincipal.Current.FindFirst(ClaimTypes.Upn) != null ? ClaimsPrincipal.Current.FindFirst(ClaimTypes.Upn).Value : ClaimsPrincipal.Current.FindFirst(ClaimTypes.Email).Value;
            if (userName == null)
            {
                throw new Exception("No username found.");
            }

            // Get on behalf of token.
            UserAssertion userAssertion = new UserAssertion(accessToken, "urn:ietf:params:oauth:grant-type:jwt-bearer", userName);
            const string aadInstance = "https://login.microsoftonline.com/";
            string authority = aadInstance.UrlCombine(tenant);
            AuthenticationContext authenticationContext = new AuthenticationContext(authority);
            ClientCredential clientCredential = new ClientCredential(clientId, appKey);
            AuthenticationResult authenticationResult = await
                authenticationContext.AcquireTokenAsync(resourceId, clientCredential, userAssertion);

            return authenticationResult.AccessToken;
        }

The access token is valid for an hour and the ADAL library tucks away a refresh_token in the cache to renew it automatically every time it expires. You can read about the various tokens and their expiration times here (https://docs.microsoft.com/en-us/azure/active-directory/develop/active-directory-v2-tokens#token-lifetimes).
I believe the access token you are trying to use to construct an UserAssertion is an expired one. And without the cache, the library has no means to get a fresh one for you.

@kalyankrishna1 Thank you for your response. I will look into adding the cache to the code and see if that resolves the issue.

I'm still having this issue. Even with a cache. Do note that I'm retrieving the X-MS-TOKEN-AAD-ID-TOKEN header to build the on behalf of token and not the X-MS-TOKEN-AAD-ACCESS-TOKEN because the latter header is not present in the request.

@bdebaere : would there be a dis-synchronization between the clock of the computer running the sample and the internet time?

Email us the fiddler trace if possible.

@GeoK in case you have an idea

@jmprieur I am able to reproduce this with the following code:
Use this to get an access token:
        public static async Task<string> GetAccessToken(
            string userName,
            string password)
        {
            var formContent = new FormUrlEncodedContent(new[]
            {
                new KeyValuePair<string, string>("resource", "hidden"),
                new KeyValuePair<string, string>("client_secret", "hidden"),
                new KeyValuePair<string, string>("username", userName),
                new KeyValuePair<string, string>("password", password),
                new KeyValuePair<string, string>("client_id", "hidden"),
                new KeyValuePair<string, string>("grant_type", "password")
            });

            var httpClient = new HttpClient();
            var result = await httpClient.PostAsync(new Uri("https://login.microsoftonline.com/hidden.onmicrosoft.com/oauth2/token"), formContent);
            var content = await result.Content.ReadAsStringAsync();
            var accessToken = JObject.Parse(content)["access_token"].ToString();

            return accessToken;
        }

Use the token you just got from the function above with this function below:
        protected async Task<string> GetOnBehalfOfToken(string tenant, string clientId, string appKey, string resourceId)
        {
            const string aadInstance = "https://login.microsoftonline.com/";
            var accessToken = "tokenYouGotFromFunction";
            var userName = "hidden";

            // Get on behalf of token.
            var userAssertion = new UserAssertion(accessToken, "urn:ietf:params:oauth:grant-type:jwt-bearer", userName);
            var authority = aadInstance.UrlCombine(tenant);
            var authenticationContext = new AuthenticationContext(authority, new FileCache());
            var clientCredential = new ClientCredential(clientId, appKey);
            var authenticationResult =
                await authenticationContext.AcquireTokenAsync(resourceId, clientCredential, userAssertion);

            return authenticationResult.AccessToken;
        }

If you keep using the token as pasted string it will work for about an hour. Then it will throw the error: "Error validating credentials. AADSTS50013: Assertion is not within its valid time range.".
This is my crude FileCache:
    public class FileCache : TokenCache
    {
        public string CacheFilePath;
        private static readonly object FileLock = new object();

        // Initializes the cache against a local file.
        // If the file is already present, it loads its content in the ADAL cache
        public FileCache()
        {
            string filePath =<EMAIL_ADDRESS>            CacheFilePath = filePath;
            this.AfterAccess = AfterAccessNotification;
            this.BeforeAccess = BeforeAccessNotification;
            lock (FileLock)
            {
                //this.Deserialize(File.Exists(CacheFilePath) ? ProtectedData.Unprotect(File.ReadAllBytes(CacheFilePath), null, DataProtectionScope.CurrentUser) : null);
                //this.Deserialize(File.Exists(CacheFilePath) ? ProtectedData.Unprotect(File.ReadAllBytes(CacheFilePath), null, DataProtectionScope.LocalMachine) : null);
                this.Deserialize(File.Exists(CacheFilePath) ? File.ReadAllBytes(CacheFilePath) : null);
            }
        }

        // Empties the persistent store.
        public override void Clear()
        {
            base.Clear();
            File.Delete(CacheFilePath);
        }

        // Triggered right before ADAL needs to access the cache.
        // Reload the cache from the persistent store in case it changed since the last access.
        void BeforeAccessNotification(TokenCacheNotificationArgs args)
        {
            lock (FileLock)
            {
                //this.Deserialize(File.Exists(CacheFilePath) ? ProtectedData.Unprotect(File.ReadAllBytes(CacheFilePath), null, DataProtectionScope.CurrentUser) : null);
                //this.Deserialize(File.Exists(CacheFilePath) ? ProtectedData.Unprotect(File.ReadAllBytes(CacheFilePath), null, DataProtectionScope.LocalMachine) : null);
                this.Deserialize(File.Exists(CacheFilePath) ? File.ReadAllBytes(CacheFilePath) : null);
            }
        }

        // Triggered right after ADAL accessed the cache.
        void AfterAccessNotification(TokenCacheNotificationArgs args)
        {
            // if the access operation resulted in a cache update
            if (this.HasStateChanged)
            {
                lock (FileLock)
                {
                    // reflect changes in the persistent store
                    //File.WriteAllBytes(CacheFilePath, ProtectedData.Protect(this.Serialize(), null, DataProtectionScope.CurrentUser));
                    //File.WriteAllBytes(CacheFilePath, ProtectedData.Protect(this.Serialize(), null, DataProtectionScope.LocalMachine));
                    File.WriteAllBytes(CacheFilePath, this.Serialize());
                    // once the write operation took place, restore the HasStateChanged bit to false
                    this.HasStateChanged = false;
                }
            }
        }
    }

@bdebaere : indeed, @kalyankrishna1 is right: the authentication libraries (ADAL.NET and MSAL.NET) refresh the token by themselves.
Which is your scenario ? Are you writing a Web API? or a Web App? or a desktop application?

If you are writing a Web App, you might want to read acquiring tokens by authorization code,
If you are writing a Web API, you'd want to read on behalf of (Service to service calls)

Finally if you are writing a web app or a Web API you should not use Username password as this should be only called for Desktop/Mobile applications. BTW ADAL.NET can do it by itself (See Using Username/password )
Did you have a look at ADAL.NET conceptual documentation where we explain all the ways to acquire a token?

@jmprieur The scenario is a web API with authentication configured in the Azure portal. The access token is sent around by the Azure wrapper around my API. I'm never calling AcquireTokenAsync() to get the first access token. It is given to me by Azure.
Are you telling me this scenario is not supported?

The bit I don't understand @bdebaere is your GetAccessToken(string userName, string password) method in your comment above. Also I see that you are using a file cache. where is the file written? Usually Web apis use other kind of caches? See Token cache serialization in Web APIs

@jmprieur Allow me to resketch the situation

I created a web API from scratch without any authentication configured. So no Startup.Auth.cs.
I created a web API resource in the Azure portal with nothing special configured.
In the created web API resource in the Azure portal I configured authentication to express.

In the express configured app registration I added the required permissions for the resource I'm trying to access on behalf of

In my web API I get the accesstoken with the following code: string accessToken = Request.Headers.FirstOrDefault(fod => fod.Key == "X-MS-TOKEN-AAD-ID-TOKEN").Value?.FirstOrDefault();. I do this because BootstrapContext is empty.
The rest of the code is the same as the sample. I fill in my web API id as clientId and generate a key in the app registration to use as appKey.

            string userName = ClaimsPrincipal.Current.FindFirst(ClaimTypes.Upn) != null ? ClaimsPrincipal.Current.FindFirst(ClaimTypes.Upn).Value : ClaimsPrincipal.Current.FindFirst(ClaimTypes.Email).Value;
            UserAssertion userAssertion = new UserAssertion(accessToken, "urn:ietf:params:oauth:grant-type:jwt-bearer", userName);
            const string aadInstance = "https://login.microsoftonline.com/";
            string authority = aadInstance.UrlCombine(tenant);
            AuthenticationContext authenticationContext = new AuthenticationContext(authority, new FileCache());
            ClientCredential clientCredential = new ClientCredential(clientId, appKey);
            AuthenticationResult authenticationResult = await
                authenticationContext.AcquireTokenAsync(resourceId, clientCredential, userAssertion);

Where exactly in this process am I going wrong?
The file cache I'm using is just a test. I realize that a db cache is more fitting, but this is just a proof of concept at this moment. I can see that the file cache is being written to, but as soon as it comes to refreshing the token, it no longer works and I get the error I mentioned before.
If I access my web API through http://myapi.azurewebsites.com/FunctionThatRequiresOnBehalfOf then it works for an hour because the token does not need refreshing, but as soon as it does it doesn't seem to realize that it needs to do so when asking the on behalf of token.
I'm really trying hard here to understand where the issue lies and what I can do to solve it. Please help me understand.

Thanks for your detailed explanation. @bdebaere. This definitively clarifies.
The issue you have is that the token used to call your Web API has expired (the token in the user assertion), and therefore Azure AD does not exchange this token against another token to call your downstream API.
Did you think of securing your Web API directly with an [Authorize] attribute? instead of using the app service authentication? like in https://github.com/Azure-Samples/active-directory-dotnet-webapi-onbehalfof

@jmprieur Yes, this was the obvious next route to try, but I wanted to confirm if there wasn't an option anyway. This way I can also relay this information to my colleagues.


CJS / ESM selection case - styled-components

Styled components broke recently at https://generator.jspm.io/#U2NhYGBkDM0rySzJSU1hKC6pBFK6yfm5Bfl5qXklxQ6mesZ6xgCnze/3JgA.
This is due to the CJS variant being used from https://ga.jspm.io/npm<EMAIL_ADDRESS>instead of the .esm.js variant.
With the dominance change in https://github.com/jspm/generator/pull/86, issues like this might be more likely.
Steps:

[ ] Test the latest version of this generator source against it to see if it's still doing that.
[ ] Try changing the priority in the PR at https://github.com/jspm/generator/pull/86/files to instead do a !wasCJS variant on all the conditionals to check if the ESM priority can be made
[ ] Possibly consider allowing priority to be user-defined to resolve cases like this in future or alternative customization approaches.

@JayaKrishnaNamburu finally tracked this one down and it wasn't due to the determinism issue, but a previously resolved CDN bug around "module" and "browser" interaction.

Thanks for looking into this @guybedford.
https://codesandbox.io/s/serene-neumann-07flq?file=/module.js
Looks like we have similar issue around @emotion/is-prop-valid which doesn't have export maps 😓

We are trying to introduce import-map workflows for users and so trying to put together a template with jspm cdn and our esm components which is using styled-components

@JayaKrishnaNamburu if you look at the package.json for is-prop-valid - https://ga.jspm.io/npm<EMAIL_ADDRESS>you can see it has the correct branching for the browser both for browser + module and browser + !module condition paths.
Looking at your sandbox it's because you're on an older cached build of is-prop-valid - https://ga.jspm.io/npm<EMAIL_ADDRESS>that doesn't have that in the package.json.
I can rebuild this 0.8.8 version for you.

Ok, the cache clear should complete shortly, then it should be working. Just let me know if there are any other package + versions you need. I trust you can tell from the package.json what to look for by now.

That would be pretty helpful 😄

And i believe this map is generated by jspm itself right. Because i don't see any on the emotion package itself.
https://unpkg.com/browse/<EMAIL_ADDRESS>Would it still e a good thing having on the package itself, because i am trying to make a PR if so, to add exports to emotion packages 😄

@JayaKrishnaNamburu yes JSPM automatically generates the "exports" field for all packages. It's obviously better for packages to define their own of course, but it's perfectly fine to support "module" and "browser" fields, there was just a bug only fixed recently in the conversion that wasn't supporting the combination of "module" and "browser" together, a pattern emotion uses extensively.

Ah got it, got it 👍

Here is a working example, needed a little bit of manual correction of import-map
https://codepen.io/jkrishna/pen/VwMaWaN?editors=1010
Import map -> https://generator.jspm.io/#U2NjYGBkDM0rySzJSU1hKC6pBFK6yfm5Bfl5qXklxQ6mesZ6xgDY5X4tJgA
@emotion/stylis is pointing to esm but the other two emotion packages @emotion/is-prop-valid and @emotion/memonize still pointing to cjs paths. A manual change of it, and it started working. Thanks for helping with this one 😄

@JayaKrishnaNamburu rebuilt and flushed those caches. Should be working now with a cache clear!


[ServiceBus] Investigate whether mgmt calls should disallow object-based passing or not.

See work within  the following branch
This was ostensibly for consistency work to change e.g. "topic -> topic_name" for all non-update calls.
It raised concerns, namely:

Multiple call patterns for a user between different flows, both by param name and type (name, topic_name, topic, for instance) and in the update (takes type) vs non update (takes string only) paths.
It violates the "be generous when accepting" guideline and gives less useful hints than strongly typed languages that the above occurs.

Discussion with python crew today came to consensus that we will make this change.  Re-enabling old PR and writing unit tests.
Primary reasons:

ARM uses names, not objects.
Want to remove ambiguity that, e.g. in delete, ONLY the target is being deleted, not the intermediary things.
Make more explicitly clear to users what is expected in a given field.

The one pitfall here is the update_ flow, but this is acknowledged and for the explicit purposes of guardrails.


Dev

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Was this intentional, I don't see any related to the solution?


Strange Compile Error

When using the latest release of the library, I get an error message when I try to compile.

This code worked fine previously however it seems to not work, although no significant changes have been made to the source.
A pastebin with my code can be viewed here, although it's basically just the example in samples, with a few small changes

On the line 25 in your code you should add DiscordRichPresence.create();, mostly these are issues from hxcpp, not from the library itself really.

I tweaked the code a little bit, and that seemed to fix the issue, so ig I'll close this.

I tweaked the code a little bit, and that seemed to fix the issue, so ig I'll close this.

Sometimes hxcpp can do weird stuff and that's kinda it.


Osmumtens fang incorrect

For some reason the previous fix to the osmumtens fang passive has been undone and the calculator no longer takes into account the osmumtens fang passive

Would like to add that this is especially apparent as the dps calculator currently thinks the zombie axe is better than the fang here


More robust release.sh script

Just a small pull request which makes the build script a bit more modular and robust. In case of an issue in the future, the script will scream earlier.

Use more general /usr/bin/env instead of hardcoded /bin/bash with bash options for better scripts. Break early and break loud.
Use cd … || exit in case cd fails and cause harm
Double quote variables to prevent globbing and word splitting

Thanks for your pull requesti!


is it possible to add support to decorators?

I added the plugin but i encountered some issues,
i use mobx so need to use of decorators:
[plugin:vite:inferno-jsx] /Users/liturribarria/Sites/evolve/temp/render/src/components/main/Main.js: Support for the experimental syntax 'decorators' isn't currently enabled (15:1):

Add @babel/plugin-proposal-decorators (https://github.com/babel/babel/tree/main/packages/babel-plugin-proposal-decorators) to the 'plugins' section of your Babel config to enable transformation.
If you want to leave it as-is, add @babel/plugin-syntax-decorators (https://github.com/babel/babel/tree/main/packages/babel-plugin-syntax-decorators) to the 'plugins' section to enable parsing.
/Users/liturribarria/Sites/evolve/temp/render/src/components/main/Main.js:15:0
13 |  import { cookie as Cookie } from '../../services/Cookie';
14 |
15 |  @inject('AD')
   |  ^
16 |  @observer
17 |  class Main extends Component {

it will be great to have this enabled
thanks

Hello,
To enable support for decorators in your project while using vite-plugin-inferno, simply adjust your Vite config as follows. This will enable Babel to process decorator syntax properly:
import { defineConfig } from 'vite';
import inferno from 'vite-plugin-inferno';

export default defineConfig({
  plugins: [
    inferno({
      // Pass custom Babel options to the Inferno plugin
      babel: {
        // Include the decorators plugin with legacy syntax support
        plugins: [['@babel/plugin-proposal-decorators', { legacy: true }]]
      }
    }),
  ],
});

Ensure you have @babel/plugin-proposal-decorators installed in your project.
Best regards and happy coding!

amazing! thank you so much @Sofiane-77


Binning should never create only 1 bin

I'm opening this issue at Graham's request.
When a binning operation is performed, the result should never consist of a single bin.

Should be fixed in 2.4_defects and will push to master and later to the spark jar