Text Box value changing

hi
am working on a project where multiple sms will be sent at a time. Every thing is ok except the message delivered is getting changed to T0.message to all receivers. Attached the screen shot. where multiple mobile nos are selected and message from a popup dialog is send. attaching the code for reference.

The Hello message is converted into T0.Message and delivered to the mobile no.
find below the code and please let me know where am going wrong
BulkEndpoint.cs
[HttpPost]
    public MyActionResponse MyAction(IDbConnection connection, MyActionRequest request)
    {
        connection.Open();
        var r = new MyActionResponse();
      using (connection = SqlConnections.NewFor<sms.BULK.Entities.BulkRow>())
            {
                 var Myrow = connection.ById<sms.BULK.Entities.BulkRow>(request.MyId);
                string authKey = "26319ABsrQSypVZjw5b45f11e";
                String ContactNumber = sms.SMS.Entities.Sms1Row.Fields.MobileNumber.ToString();
                String Message = Myrow.Message;
                String message = HttpUtility.UrlEncode(Message);
            string senderId = "SIIIin";
            StringBuilder sbPostData = new StringBuilder();
            sbPostData.AppendFormat("authkey={0}", authKey);
            sbPostData.AppendFormat("&mobiles={0}", ContactNumber);
            sbPostData.AppendFormat("&message={0}", message);
            sbPostData.AppendFormat("&sender={0}", senderId);
            sbPostData.AppendFormat("&route={0}", "6");
            string sendSMSUri = "http://sms.fastsmsindia.com/api/sendhttp.php";

            HttpWebRequest httpWReq = (HttpWebRequest)WebRequest.Create(sendSMSUri);

            UTF8Encoding encoding = new UTF8Encoding();
            byte[] data = encoding.GetBytes(sbPostData.ToString());

            httpWReq.Method = "POST";
            httpWReq.ContentType = "application/x-www-form-urlencoded";
            httpWReq.ContentLength = data.Length;
            using (Stream stream = httpWReq.GetRequestStream())
            {
                stream.Write(data, 0, data.Length);
            }

            HttpWebResponse response = (HttpWebResponse)httpWReq.GetResponse();
            StreamReader reader = new StreamReader(response.GetResponseStream());
            string responseString = reader.ReadToEnd();
        reader.Close();
            response.Close();

        }

        return r;
    }

Bulk.grid
    protected getButtons() {
        var buttons = super.getButtons();

        buttons.push({
            title: 'Send',
            cssClass: 'Send-button ',

            onClick: e => {

            }
        });

        return buttons;

    }
    private myAction() {
        BulkService.MyAction(
            {
               // MyId: this.get_entityId(),

            },
            response => { Q.notifySuccess("Success! ", '', null) });

    }

Thank You

why don't u set breakpoint on server and have  a look?
what does Myrow.Message actually contain? That's where the message comes from, right?
Did u check in DB?
From the value it seem, like maybe u need to add .value when assigning value (as you are obviously assigning object (field name) rather than it's content...

Hi,
Tried breakpoint and found that Myrow.Message is getting empty after saving. and DB is returning T0 value to the message box. How can i get the last saved id from DB to the message box after saving the message to DB.
Thank You

Weird, you prolly doint it wrong.
I am saving user like this:
 var UserRequest = new SaveRequest<Users.Entities.UsersRow> { Entity = request.UserRow };
            new Users.Repositories.UsersRepository().Create(uow, UserRequest);

in my endpoint and it even contains ID or inserted user, once created/saved, so I dont see why it should be loosing existing values...
I am using uow passed into my endpoint...
maybe u should not open new connection, just use existing one...
OR better yet, just read entity back after save...

Closing due to inactivity.


How to specify a minimum selectable range for HRangeSlider?

Hi. First of all, thank you for an awesome library!
A quick question:
How do I specify a minimum selectable range for an HRangeSlider? Couldn't find it in the options.

Hi @skryshi
Sorry about answering so late.
There is no way to do it right now, but this is something I should definitely do.
I'll keep you posted here on the progress


Nested member access (dot notation) on a number-like property name fails

Note: This is a relatively minor issue with an easy workaround, but it would be good to understand if there are other ramifications and a fix would certainly be nice.
Steps to reproduce
([pscustomobject] @{ 1 = 'foo ' }).1 | Should -BeExactly 'foo '
([pscustomobject] @{ 1 = 'foo ' }).1.Trim() | Should -BeExactly 'foo'

Expected behavior
Both tests should succeed.
Actual behavior
The 2nd test fails, because the .1 is apparently no longer recognized as a property access in combination with the .Trim() call:
ParserError:
Line |
   1 |  ([pscustomobject] @{ 1 = 'foo ' }).1.Trim() | Should -BeExactly 'foo'
     |                                     ~
     | Missing property name after reference operator.

Note:

If the property name cannot be parsed as a number literal, the problem doesn't surface (e.g., .a1).

A related problem is trying to use a property name that starts with a digit; e.g.,
([pscustomobject] @{ '1a' = 'foo ' }).1a breaks, unless .'1a' is used instead.

Workarounds:

Quote the property name (thanks, @jantari): ([pscustomobject] @{ 1 = 'foo ' }).'1'.Trim()
Enclose the original property access in (...):  (([pscustomobject] @{ 1 = 'foo ' }).1).Trim()

This workaround also works when accessing  hashtables that have actual numeric keys via property syntax, such as with the automatic $Matches variable:

$null = 'foo bar' -match '^(foo\s)'; ($Matches.1).Trim()

As @BrucePay points out, enclosing just the property name / key in (...) also works in both scenarios:

([pscustomobject] @{ 1 = 'foo ' }).(1).Trim()
$null = 'foo bar' -match '^(foo\s)'; $Matches.(1).Trim()

Environment data
PowerShell Core 7.1.0-rc.2

Although you're probably aware of it I just want to mention for others who have this issue that this can be worked around for now by quoting the property:
([pscustomobject] @{ 1 = 'foo ' }).1.Trim() # FAILS
([pscustomobject] @{ 1 = 'foo ' }).'1'.Trim() # WORKS

This also works for other property-names that make PowerShell misbehave, such as properties starting with a '#' etc.

Thanks, @jantari - I've added the workaround to the OP. Note that (...) works in all cases, because quoting isn't an option when you access hashtables with truly numeric keys with property syntax, which can happen with the automatic $Matches variable - I've added that to the OP too.

There is actually a lexical ambiguity here. Consider data.1.0. Is this a nested index or indexing with the double 1.0?  In practice, we do a shift instead of a reduce so it becomes indexing with a float. I think that with data.1.b, shifting results in an invalid number ultimately resulting in the error you see (I haven't actually checked the code so I don't have the exact details.) Anyway, you can eliminate the ambiguity by putting the number in parens:
([pscustomobject] @{ 1 = 'foo ' }).(1).Trim()

Sort of? Property access tends to convert basically everything to a string in most cases, because most properties can only have strings for the property name. PSObject / PSCustomObject creates objects with string property names, not integer ones. So, yes, that works... incidentally only. $obj.PSObject.Properties.Name.GetType() on that object you create there will tell you the property name is still a string.
There's no reason not to assume a string value is being used for the property name IMO. The only exception is dictionaries/hashtables having non-string key types. Regular objects have strings for property names, I don't think there are really any exceptions to that. It's already true in some cases that you must use $table[$value] to index some kinds of keys, this wouldn't be a new thing.

Agreed, @vexx32:

For a non-IDictionary object, it is unequivocally unhelpful to try to parse the member name as a number, and fixing that should be considered a bug fix.

For an IDictionary, recognizing [double]s is a feature - even though it's hard to imagine that it would see much use:

PS>  @{ 1.5 = 'foo ' }.1.5.Trim()
foo

If we were to maintain strict backward compatibility, we can't take this feature away.
However, I've never seen a [double]-keyed dictionary in the wild, and the precision issues around binary floating-point representations alone make this an ill-advised thing to do (and the syntax does not work with [decimal]-typed keys).
I see two options:

Given the presumed rarity of [double]-keyed dictionaries  and the fact that users will most likely expect all . chars. to be member-access operators, my vote is to consider this a bucket 3 change and always stop parsing a  key name when a . is encountered.

If there really is a need to use a [double] key, use $dict.(1.5).Trim(), as @BrucePay suggests, or use $dict[1.5], as you suggest.

If we must maintain backward compatibility, at least fall back to interpreting $dict.1.b as $dict[1].b(though I can't speak to the implementation complexity).

It occurred to me that the parsing in question presumably happens during the lexical analysis phase, where distinguishing by data type being operated on isn't an option. I've revised the previous comment accordingly - again: I think simply not supporting [double] keys with (unquoted, non-expression) property syntax is the best solution.

Longer-term, if we ever get to break backward compatibility more substantially, my recommendation would be, for conceptual simplicity and for avoiding edge cases:

Only ever treat bareword property names such as .1 as strings.

For everything else, either require an explicit expression context (.(...), .$propName) or index-based access instead, in which case the intent to use an expression is unequivocally signaled.

(To soften the blow, the automatic $Matches variable could be converted to an [ordered] hashtable whose default keys are the stringified indices, so that something like $Matches.1 continues to work); the added benefit is that you both  indexed ($Matches[1]) and named-capture-group property then work $Matches.foo).

Not sure if it's directly related but in below example nested properties also doesn't work unless put in quotes
$l=@{I=1;V=5;X=10;L=50;C=100;D=500;M=1000}
$a='IVXLLD'

$l[$a[3]]
# Result:

$l["$($a[3])"]
# Result: 50

@catthehacker, no, this is a different problem: [hashtable] keys must be matched type-exactly during lookup.
Your keys are [string]-typed, despite comprising only a single character (e.g., L - quoting optional in hashtable literals; the quoting style makes no difference, unlike in C#): PowerShell has no literal [char] type.
By contrast, 'IVXLLD'[3] does return a [char] instance, and that's why the lookup fails, and only succeeds with stringification; $l[[string] 'IVXLLD'[3]] would work too.

@mklement0 Thanks a lot, that's awfully helpful.

This still fails in the latest preview so it's still relevant.


Auto intellisense for jquery doesn't work in 1.18.1

VSCode Version: Code 1.18.1 (929bacba01ef658b873545e26034d1a8067445e9, 2017-11-16T18:32:36.026Z)
OS Version: Windows_NT ia32 6.1.7601
Extensions:

Extension
Author (truncated)
Version

xml
Dot
1.9.2

rest-client
hum
0.16.1

vscode-JS-CSS-HTML-formatter
lon
0.2.3

Angular-BeastCode
Mik
5.0.17

csharp
ms-
1.13.1

angular2-inline
nat
0.0.17

vsc-angular-cli
web
1.0.0

html-css-class-completion
Zig
1.14.0

Steps to Reproduce:
1.open folder with node-modules which contains jquery
2.open a js file

Reproduces without extensions: Yes
In previous versions it can automatically show the intellisense of jquery once there is a node_modules folder in the project which contains jquery. But it doesn't work in version 1.18.1.

This issue was moved to Microsoft/TypeScript#20282


Typo on High limit actions?

Is this a typo?
https://docs.microsoft.com/en-us/power-automate/limits-and-config#:~:text=2%2C000 for Low%2C 5%2C000 for MediumLow1%2C 20%2C000 for MediumLow2%2C 25%2C000 for Medium%2C and 15%2C000 for High

Actions per 24 hours
2,000 for Low, 5,000 for MediumLow1, 20,000 for MediumLow2, 25,000 for Medium, and 15,000 for High

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 0a7b1f84-92bd-7188-a7d0-359a930525dd
Version Independent ID: dd7d2d01-bdee-b656-ca63-e19305963d28
Content: Limits and configuration - Power Automate
Content Source: articles/limits-and-config.md
Service: power-automate
Sub-service: cloud-flow
GitHub Login: @MSFTMan
Microsoft Alias: deonhe

@samadane - Thanks for your feedback. We are tracking this issue in https://github.com/MicrosoftDocs/power-automate-docs/issues/515. Closing this issue.


BoxConstraints has a negative minimum height.

Hello,
I'm trying to implement a small floating player so I set the width at 100.0 but it throws the following error :
════════ (1) Exception caught by widgets library ══════════
BoxConstraints has a negative minimum height.
User-created ancestor of the error-causing widget was:
  YoutubePlayer file:///C:/Dev/turn-up/frontend/lib/screens/player.dart:22:15
════════════════════════════════════════════

@baptisteArnaud Fixed in Version 5 and up.

I still have this issue when full screen is enabled.
youtube_player_flutter: ^9.0.3


duckling entities not generating

Rasa Core version: 0.8
Python version: 3.6
Operating system (windows, osx, ...): Windows 10 64
Issue:
@mouadino
nlu_config.json
{ "pipeline": [ "nlp_spacy", "spacy_sklearn", "tokenizer_spacy", "intent_featurizer_spacy", "intent_classifier_sklearn", "ner_crf", "ner_synonyms", "ner_duckling" ], "path" : "./models/nlu", "data" : "./data/training_data.json", "duckling_dimensions": ["time", "number", "amount-of-money", "distance" , "email" , "url", "phone-number", "duration"] }
Its creating ner_duckling.json file in model folder
{   "dimensions": [     "time",     "number",     "amount-of-money",     "distance",     "email",     "url",     "phone-number",     "duration"   ] }
but in training_data.json its not giving duckling entities
{         "intent": "info",         "entities": [],         "text": "the first Tuesday of October"   }

@mouadino

training_data.json won't contain any entities from duckling. That's fine though, this is training data you've written yourself. Ner_duckling will predict entities when you've actually got the model loaded.

It's working after I declared slot name as time and date

@praveenrengarajan We did that using interactive learning, with that the slot get generated automatically you can check our examples from the gist here https://gist.github.com/mouadino/78ab561e9164b4857528745e6668c0f1


Problem with ScalableVideoView in Recycler view

I noticed problem while checking out demo of your library on Google Play Store. I was using Nexus 5X with Android 6.0 on board. The problem I noticed happened with video list - RecyclerView in this case. While scrolling up and down for some row display black view instead of video view. Please refer to attached screenshots.

I found some relevant informations that this might be an Android 6.0 issue. some more info here: https://code.google.com/p/android/issues/detail?id=191218&q=label%3APriority-Medium&colspec=ID Type Status Owner Summary Stars so I guess issue can be closed

Same in android 4.4; sometimes video does not load (start) currectly, - scrolling video from viewport and back sometimes forces video to restart, but sometimes it does not happen.
At first glance it seems an issue either with recyclerView or sampleAdapter: scrolling videoView from viewport stops video from playing, but scrolling back not necessarily hits onBindViewHolder.

Did you found any workaround for this issue? Using RecyclerView support lib in version 23.1.1 problem still exists.

2depodefi: Unfortunately right now I don't have the time to play around with this one: I ended up using sprylab's textureVideoView (which I found more stable at the moment) and implementing scalability into that. I made my own adapter and it seems to work just fine.

@depodefi
Sorry for late replay.
I reproduced the problem and fixed it.
The problem is that bindViewHolder is not always called when scrolling although the item view is detached from the window.
So, I fixed the problem and published ver 1.0.2.
An app on google play should be updated soon, so please check if you are still interested in it.


Integrated with Yarn workspaces

Summary
Yarn 0.27 has workspaces feature that should be a better alternative to Lerna boostrapping.
Yarn workspaces still don't replace Lerna (and probably won't do it completely for a long time), so Lerna publishing is still used for Jest.
This PR replaces Lerna bootstrap phase with Yarn workspaces.
Things to discuss:

Unlike Lerna Yarn hoists workspaces to the root node_modules.

It means that integration/examples test don't run in isolation from the workspace root so I removed symlinking to babel-jest et al. at the tests boostrapping.
One test integration_tests/__tests__/transform.test.js required to be running in isolation, so I changed the test bootstrap to execute on a copy on a temp folder.

As of Yarn 0.27.2 workspaces are still fresh, I am working on making it snappy.
Known issues:

changes in workspaces (non root) package.json are not automatically invalidating integrity check
logs may have confusing lines when working with workspaces
yarn add/upgrade/remove is not fully tested from within workspaces

PR to support Yarn workspaces as first class citizen in Lerna https://github.com/lerna/lerna/pull/899

Test plan

yarn test-ci
lint/flow/build tasks work per workspaces correctly
ran yarn run publish and lerna seems to handle version bumping and npm publishing correctly

We need to bump Yarn 0.27 to stable to fix CI.

This is super sweet @bestander!
Mind rebasing and giving me an update on what the remaining problems are on this diff?

@cpojer will do.
What do you think of integration_tests/__tests__/transform.test.js change?
Yarn hoists all packages as symlinks to the top so all tests now have babel-jest available by default and we can't test integration/example tests in isolation anymore.

If I recall correctly we're already using os. tmpdir in some of integration tests so I'm not opposed to do it in this case.

Yep, I'm fine with it also.

Waiting for Yarn 0.27.2 release to be stable so that CI would run installation correctly

CI seems to be red these days and es5-eslint task fails locally with
yarn run lint-es5-build
yarn run v0.27.5
$ eslint --no-eslintrc --no-ignore --env=browser packages/*/build-es5

/Users/bestander/work/jest/packages/jest-matchers/build-es5/index.js
  4606:1  error  Parsing error: The keyword 'const' is reserved

✖ 1 problem (1 error, 0 warnings)

error Command failed with exit code 1.

I don't think my PR caused it

Also we might want to wait for Yarn to be installed on appveyor and circle.
They use older versions

@bestander is yarn stable now?

Yes
On Tue, Jul 4, 2017 at 10:47 AM Michał Pierzchała<EMAIL_ADDRESS>wrote:

@bestander https://github.com/bestander is yarn stable now?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/facebook/jest/pull/3906#issuecomment-312925713, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACBdWMiDz9h7tLKElwPrEsWAls1iychmks5sKnqwgaJpZM4OEsZ-
.

Would you mind rebasing this diff when you have a while? :)

Will do today.
Do you have any background on the es5 tests that are failing?
It seems like hoisting of workspace folders to the root affects the bundle
On Tue, Jul 4, 2017 at 11:22 AM Michał Pierzchała<EMAIL_ADDRESS>wrote:

Would you mind rebasing this diff when you have a while? :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/facebook/jest/pull/3906#issuecomment-312931354, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACBdWFpGJ-GlNi3157_dbOFXiaKpnUs_ks5sKoLdgaJpZM4OEsZ-
.

It passes locally, I've rebased and restarted the build, we'll see :)

Thanks, mate
On Tue, Jul 4, 2017 at 11:55 AM Michał Pierzchała<EMAIL_ADDRESS>wrote:

It passes locally, I've rebased and restarted the build, we'll see :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/facebook/jest/pull/3906#issuecomment-312934947, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACBdWMRFwBodTHAXgufq2GpW9RvPMhCAks5sKoqdgaJpZM4OEsZ-
.

Tried rebuilding without cache, but still the same. Not sure why it happens. cc @SimenB @skovhus

Sound weird that this will affect the rollup build, but now I'm super happy that I added the eslint check. So it seems some files are not transpired correctly.
Have you looked into the es5 build for each package and see if they all contains untranspiled code?

@bestander does it also fail locally?

Yes it fails locally if you clean repo from all node_modules in packages
On Tue, Jul 4, 2017 at 1:16 PM Kenneth Skovhus<EMAIL_ADDRESS>wrote:

@bestander https://github.com/bestander does it also fail locally?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/facebook/jest/pull/3906#issuecomment-312947345, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACBdWDd2TZN2UKeMAok5b8-BvtSqsTEhks5sKp2GgaJpZM4OEsZ-
.

This solves it, by transpiling node_modules (except for babel-runtime that gives some issues).
From b62040420421da46de054e7b9afc25649b45700d Mon Sep 17 00:00:00 2001
From: Kenneth Skovhus<EMAIL_ADDRESS>Date: Tue, 4 Jul 2017 22:39:04 +0200
Subject: [PATCH] Fix browser build (ansi-styles contains ES2015 code)

---
 scripts/browserBuild.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/scripts/browserBuild.js b/scripts/browserBuild.js
index ac43ca0..db1b4a2 100644
--- a/scripts/browserBuild.js
+++ b/scripts/browserBuild.js
@@ -20,7 +20,7 @@ const babelEs5Options = Object.assign(
   {},
   {
     babelrc: false,
-    exclude: 'node_modules/**',
+    exclude: 'node_modules/babel-runtime/**',
     plugins: [
       'syntax-trailing-function-commas',
       'transform-flow-strip-types',
--
2.10.2

Can we whitelist only the node_moduels that are not ES5 compliant and transform those? I'd much rather compile only one or two that need it, rather than all of them.

@cpojer a whitelist will be a moving target, but might be a better option.
This works:
-    exclude: 'node_modules/**',
+    exclude: 'node_modules/!(ansi-styles|remove-trailing-separator)/**',

Yeah let's do this. It's fine with me if we have to update this list from time to time.

Thanks a lot, @skovhus, this fixed the build.
Indeed it is great that we have es5 tests now.
The way rollup picks modules is a bit concerning though, it all goes down to what structure is inside the root node_modules and it is conforms with Node.js resolution, not for browser builds for workspaces in isolation
We might rethink how those jest-mock and jest-matchers are built.
They probably should do it in isolation with --flat mode.

Codecov Report

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

@@            Coverage Diff             @@
##           master    #3906      +/-   ##
==========================================
+ Coverage   59.85%   60.11%   +0.26%
==========================================
  Files         196      195       -1
  Lines        6760     6702      -58
  Branches        6        6
==========================================
- Hits         4046     4029      -17
+ Misses       2711     2670      -41
  Partials        3        3

Impacted Files
Coverage Δ

packages/jest-config/src/index.js
0% <0%> (-36.37%)
:arrow_down:

...ages/jest-config/src/reporter_validation_errors.js
16.66% <0%> (-16.67%)
:arrow_down:

packages/jest-config/src/find_config.js
0% <0%> (-15.16%)
:arrow_down:

packages/jest-config/src/defaults.js
85.71% <0%> (-14.29%)
:arrow_down:

packages/jest-config/src/vendor/jsonlint.js
0% <0%> (-5.73%)
:arrow_down:

packages/jest-config/src/utils.js
73.33% <0%> (-3.75%)
:arrow_down:

packages/jest-config/src/normalize.js
84.66% <0%> (-0.89%)
:arrow_down:

packages/jest-config/src/valid_config.js
100% <0%> (ø)
:arrow_up:

packages/jest-config/src/constants.js
100% <0%> (ø)
:arrow_up:

packages/jest-resolve-dependencies/src/index.js
100% <0%> (ø)
:arrow_up:

... and 9 more

Continue to review full report at Codecov.

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

Are we good to go on this one? Is the latest version stable and rolled out so that people can use this feature?

I was able to install yarn 0.27.5 through brew, so pretty sure it's rolled out 👍

we can put "engines" entry in package.json though

Are we good to go on this one? Is the latest version stable and rolled out so that people can use this feature?

A lot of dependencies might have been bumped in all packages as dependencies are resolved from outer package.json file now. But I guess this is fairly safe due to the test coverage we have.

@thymikee, "engines" probably won't do much, in latest npm it is not strict.

Updates Lerna to 2.0.0, it supports workspaces now

And I'm getting another error about const. If you fix CI, happy if you merge this. Love the change.

Should be green now again.

Woohoo, awesome work @bestander!


Question: How can I get access to block devices on host inside LXD container?

Hi!
My container now is do not privileged, but I also tried it switch to privileged, smartctl do not have access, but lsblk have it:
lxc exec jupyter -- /bin/bash

bash-5.0# lsblk -p
NAME        MAJ:MIN RM   SIZE RO TYPE MOUNTPOINT
/dev/loop0    7:0    0   2.8G  0 loop /
/dev/sda      8:0    0 931.5G  0 disk
├─/dev/sda1   8:1    0   499M  0 part
├─/dev/sda2   8:2    0   100M  0 part
├─/dev/sda3   8:3    0    16M  0 part
├─/dev/sda4   8:4    0   100G  0 part
├─/dev/sda5   8:5    0 780.9G  0 part
└─/dev/sda6   8:6    0    50G  0 part

smartctl do not see block device /dev/sda:
bash-5.0# smartctl -t short -a /dev/sda
smartctl 7.0 2018-12-30 r4883 [x86_64-linux-5.5.2-1-MANJARO] (local build)
Copyright (C) 2002-18, Bruce Allen, Christian Franke, www.smartmontools.org

Smartctl open device: /dev/sda failed: No such device

I also with ls do not see it:
bash-5.0# ls -l /dev/
total 0
crw--w----    1 root     root      136,   3 Feb 26 07:40 console
lrwxrwxrwx    1 root     root            13 Feb 26 07:40 fd -> /proc/self/fd
crw-rw-rw-    1 nobody   nobody      1,   7 Feb 16 18:35 full
crw-rw-rw-    1 nobody   nobody     10, 229 Feb 26 07:56 fuse
srw-rw-rw-    1 root     wheel            0 Feb 26 07:40 log
drwxr-xr-x    2 nobody   nobody          60 Feb 24 14:02 lxd
drwxrwxrwt    2 nobody   nobody          40 Feb 16 18:35 mqueue
drwxr-xr-x    2 root     root            60 Feb 26 07:40 net
crw-rw-rw-    1 nobody   nobody      1,   3 Feb 16 18:35 null
crw-rw-rw-    1 root     root        5,   2 Feb 26 07:40 ptmx
drwxr-xr-x    2 root     root             0 Feb 26 07:40 pts
crw-rw-rw-    1 nobody   nobody      1,   8 Feb 16 18:35 random
lrwxrwxrwx    1 root     root            15 Feb 26 07:40 stderr -> /proc/self/fd/2
lrwxrwxrwx    1 root     root            15 Feb 26 07:40 stdin -> /proc/self/fd/0
lrwxrwxrwx    1 root     root            15 Feb 26 07:40 stdout -> /proc/self/fd/1
crw-rw-rw-    1 nobody   nobody      5,   0 Feb 26 07:35 tty
crw-rw-rw-    1 nobody   nobody      1,   9 Feb 16 18:35 urandom
crw-rw-rw-    1 nobody   nobody      1,   5 Feb 16 18:35 zero

How to see host block devices in a container? How to do that Smartctl see block devices of the host?

You need to pass them using unix-block devices.
Note that doing that though will allow your container to cause serious damage to the host.


make local time message was written appear on timestamp hover

thanks to @ariganu for the idea!

We need to add timezones to the portal son for this, right?

Don't think so? As long as all message timestamps are written in UTC+0 we can pick up the local TZ from the browser at render time (unless I'm missing something)

"local time message was written" sounds like the author's local time, not the reader's local time, and that pretty much requires a new "timezone offset" field in the portal.json (AFAIK we've got no other means of assuming the author's timezone at the moment).

I'm 99% sure @aeonofdiscord is right. A JS date, when converted to a number or serialized as a string, is UTC+0. When you read that number/string into a JS date, it applies the local timezone.


WFLY-17322 Clustering subsystems create duplicate PersistentResourceXMLDescription instances

https://issues.redhat.com/browse/WFLY-17322
I recommend reviewing individual commits.

Thanks - I'll amend the javadoc fixes into the appropriate commit and merge when CI completes.


Add a new @Table annotation option to prevent the default table creation

@Table

Possible to define a table name?
eg I have a Model "Device" but the table name is "devices".

@Table(value = "Devices")
Source

@damson Thanks a lot.

Hey, for questions like these, I would suffice a quick look at the documentation before opening an issue. If its not explicitly clear in the docs or javadocs, of course ask a question.
Thank you and I appreciate your interest in this library.

@agrosner OK, I will spend more time to read the docu. I have to apologize.


build: set apache license in mvn artifacts

Fixes MODELIX-961

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


add instructions for generating models and components from CSV files

Notes for Reviewers
This PR fixes #
Signed commits

[X] Yes, I signed my commits.

🚀 Preview for commit 74c277e98d4282e4466fb4960dca576771bdfb42 at: https://67067a4ecb4e964612570581--meshery-docs-preview.netlify.app

@pranjalg1331 would you like to review?

@leecalcote The update to the command is helpful since using "-directory" causes an error. I’m assuming we’ll be adding sample CSV templates to the link mentioned in the command usage description, as they’re not available at the moment. This feels like a much-needed PR. If someone is unfamiliar with the terminal and flags, the current usage in the documentation could be confusing.

🚀 Preview for commit 04bbfcecd465848b71873e20a1ec1dbeacfae547 at: https://6708547fb3ea23984cebd388--meshery-docs-preview.netlify.app

@leecalcote The update to the command is helpful since using "-directory" causes an error. I’m assuming we’ll be adding sample CSV templates to the link mentioned in the command usage description, as they’re not available at the moment. This feels like a much-needed PR. If someone is unfamiliar with the terminal and flags, the current usage in the documentation could be confusing.

This PR addresses that by updating the command to --directory and I have also included a link to a sample template link to the docs as well

🚀 Preview for commit b07fa76dbb8032bb141740d089da34dc63ca1026 at: https://670859e56345d09a78459fc7--meshery-docs-preview.netlify.app

@leecalcote @Aijeyomah LGTM! The use of the sample template dir adds great value. Could we add a sample command with the template dir provided. It will make the process faster.

@leecalcote @Aijeyomah LGTM! The use of the sample template dir adds great value. Could we add a sample command with the template dir provided. It will make the process faster.

Done

🚀 Preview for commit ba33c236e960008a0cc0ae8bc1da6338e702394a at: https://670c55f12ae62507b55c83d5--meshery-docs-preview.netlify.app

🚀 Preview for commit f56a989d0c04c868b662bba7065b932e0cb3367d at: https://671f63da92866d6aa99a060b--meshery-docs-preview.netlify.app

🚀 Preview for commit a60e47ff489157d62f8c29f838213f329cc13628 at: https://672021a5cf79151c01b00af0--meshery-docs-preview.netlify.app

🚀 Preview for commit f323e7a82bca664eabbc04cc135ad6e11f86b545 at: https://67203dd1f587f43e31a5d5f7--meshery-docs-preview.netlify.app

@TraXIcoN, will you please review?

merge conflicts...


PRO-567: App crashed while trying to sign in.

Description
Crashes can be caused by accessing the database when it has been removed. Added additional safety checks so it will not crash.
What Was Changed
Added additional checking to db operations to verify that the database is open. In addition, some faults that were marked as fatal, now return instead of crashing the app.
How To Test

Change pod files to point to latest commit
Log in to app
Log out quickly
Perform normal database actions

@greglittlefield-wf @bryanhales-wf @brianblanchard-wf

@bryanhales-wf Shouldn't the bryanhales-wf:prevent-crashes branch be at https://github.com/bryanhales-wf/FCModel/commit/030a56edfb7a9f96acde8f79e93718346b2e14c5 (the last commit we pointed to in FCModel, and the commit these changes are branches off of), instead of master?

#facepalm
Yes. My apologies @jamesromo-wf - I'll fix that. You'll just need to re-PR your changes. I owe you a Coke.

I'm +1 on this

+1

Looks good. Merged. This will have a full round of testing with our other code in the PRO-567 branch of our main repo.


[decomp]  Unhandled branch delay in BranchElement::push_to_stack

Can be seen in:

collide-shape

(method 28 collide-shape-prim-mesh)

collide-mesh

I think I got these fixed now


How to alias React and ReactDOM as wp.element?

Continuing from https://github.com/parcel-bundler/parcel/issues/144#issuecomment-601017068
Maybe I'm not doing it just right or not understanding how things work, but here's what I've got so far:

6min video of how far I've gotten: https://share.getcloudapp.com/wbumLZyy
diff of before and after I add my "externals" rules: https://diffchecker.com/28RmeGL1 (expires in 1 month)

The issue here is that WordPress is already loading React and ReactDOM via the @wordpress/element package from yoursite-com/wp-includes/js/dist/element.js (WordPress core, not my own path, as you can see).
Here's my package.json:
{
	...
	"dependencies": {
		"react-notifications-component": "^2.3.0"
	},
	"devDependencies": {
		"@babel/core": "^7.8.7",
		"@wordpress/scripts": "^7.1.2",
		"npm-run-all": "^4.1.5",
		"parcel-bundler": "^1.12.4",
		"parcel-plugin-externals": "^0.3.3",
		"postcss-modules": "^1.5.0"
	},
	"babel": {
		"presets": [
			"@wordpress/default"
		]
	},
	"browserslist": [
		"extends @wordpress/browserslist-config"
	],
	"externals": "./.externals.js"
}

Here's the .externals.js that my package.json runs:
const rxWp = /node_modules\/@wordpress\/(.*?)\//;
const rxBabel = /node_modules\/@babel\/(.*?)\//;
const rxReact = /node_modules\/react-(.*?)\//;

module.exports = function( path ) {
	const wp = rxWp.exec( path );
	const babel = rxBabel.exec( path );
	const react = rxReact.exec( path );

	let wpSuffix;

	let babelSuffix;

	let reactSuffix;
	let reactName;

	if ( wp ) {
		wpSuffix = wp[ 1 ];
		return `@wordpress/${wpSuffix} => require('@wordpress/${wpSuffix}')`;
	}

	if ( react ) {
		reactSuffix = react[ 1 ];
		reactName = reactSuffix[ 0 ].toUpperCase() + reactSuffix.substr( 1 );
		return `react-${reactSuffix} => React${reactName}`;
	}

	if ( babel ) {
		babelSuffix = babel[ 1 ];
		return `@babel/${babelSuffix} => require('@babel/${babelSuffix}')`;
	}

	return undefined;
};

Hm can the rxReact catch React at all? It seems to be able to catch things like react-dom, but not react (it requires another part after a dash).
The other thing is that you may want to exclude @wordpress/element. If I understand this one brings React and React-DOM as globals already (but maybe I misunderstand).
Would it be possible to make a MWE of your solution? I'd like to better understand the current setup to give you a better advice. Maybe we can also enhance the API of the plugin to cover this case.
Hope that helps!

OK, I've made progress! :)
Reference: https://parceljs.org/module_resolution.html#aliases
"alias": {
	"react": "@wordpress/element",
	"react-dom": "@wordpress/element"
},
"externals": {
	"@wordpress/element": "wp.element"
}

Now the issue is my 3rd party component isn't satisfied :( https://www.npmjs.com/package/react-notifications-component
Any ideas why not???

FYI: your repo's (and this discussion's initial) code snippet includes "node_modules" in the regex match rule but path never starts with this. console.log( path ); worked a treat to see this.

your repo's (and this discussion's initial) code snippet includes "node_modules" in the regex match rule

That's a great catch. Do you want to contribute a PR for this? Would be awesome to improve the docs. Otherwise, I'll correct this.

Any ideas why not???

Hm since we take the React UMD I could assume it does not come with a default export. Not sure though. To see if this is indeed the case you can try two things:

"Hack" the react-notifications-component in your node_modules to use import * as React from 'react' instead of import React from 'react'
Place in some code before react-notifications-component is imported / running the following snippet: require('react').default = require('react');

I'll do PR after I get things straight here in case there are any other tips to add to the readme.
Man, nothing I tried worked... I'm not sure how I can edit what comes down from npm install
The github repo does not come down raw but instead already built by webpack so I don't know how it could be edited. And the 2nd idea you had didn't fix it either.

I'm facing a deadline and really appreciate your quick help :D
Do you think https://parceljs.org/cli.html#expose-modules-as-umd is relevant at all? A quick try didn't work quite as well as yours but maybe I'm not using it correctly for my use case.

Ok that it was prebuilt using Webpack may explain this. I would need to study the code, but unfortunately I am fully covered with meetings today - so I'll either catch a look later tonight or tomorrow ... sorry!
You can have a look at the code from the package how React is resolved there. Is it coming from the global? What name should be used there? Check the dependencies of the package.json of the external package, too - React should not be bundled in and marked as a peer dependency.

Tyvm. Yes it has react as a peer dependency:
https://github.com/teodosii/react-notifications-component/blob/master/package.json

Great news! I got it sorted out. It was as simple as this:
"externals": {
	"@wordpress/element": "wp.element"
}

No "alias" or anything else!
I think the issues I was experiencing that caused me to think something wasn't working correctly was that my API validation was incorrect so stuff wasn't working because of that.
THANK YOU!

I submitted https://github.com/FlorianRappl/parcel-plugin-externals/pull/14 -- thanks much for this package and your assistance!

Hi sorry for my late appearance. Wonderful that you made it work :rocket:!
Also thanks for your PR - much appreciated :beers:!

With your help:
https://github.com/cliffordp/cliff-wp-plugin-boilerplate#march-22-2020

Improve the JavaScript build for WordPress React, reducing the admin-settings.js file size:

Minified: from 265.91 KB to 35.16 KB (87% reduction)
Unminified: from 803.68 KB to 48.15 KB (94% reduction)


Expr formatting.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#889
#888  👈
#887
#879 : 1 other dependent PR (#883 )
#876
#875
dev

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Alon-Ti and the rest of your teammates on  Graphite

Codecov Report
Attention: Patch coverage is 97.72727% with 1 line in your changes missing coverage. Please review.

Project coverage is 91.87%. Comparing base (9037ea4) to head (89c469c).
Report is 1 commits behind head on graphite-base/888.

Files with missing lines
Patch %
Lines

crates/prover/src/constraint_framework/expr.rs
97.72%
1 Missing :warning:

Additional details and impacted files
@@                  Coverage Diff                  @@
##           graphite-base/888     #888      +/-   ##
=====================================================
+ Coverage              91.85%   91.87%   +0.01%
=====================================================
  Files                     93       93
  Lines                  13053    13097      +44
  Branches               13053    13097      +44
=====================================================
+ Hits                   11990    12033      +43
- Misses                   948      949       +1
  Partials                 115      115

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

🚨 Try these New Features:

Flaky Tests Detection - Detect and resolve failed and flaky tests


ReloadAfter doesn't work well in Lambda (2)

I'd like to reopen this issue: https://github.com/aws/aws-dotnet-extensions-configuration/issues/17
I've been testing with this again and it still doesn't work. I've downloaded the code from the SDK and put tracing in, but the function doesn't reload it's configuration after the indicated time. I've downloaded the code and put tracing in and could confirm that configuration isn't reloaded as expected. The freeze of the lambda function means the reload timer (cancellation token) freezes as well. Eventually the config is reloaded, but it isn't predictable when.
I wouldn't mind waiting synchronously for the configuration to reload, if that's easier to build.

I've reopened the original issue since it had a long chain of comments.  I'm going to close this one as a duplicate.


Distribute bounties for July & August

We're shifting to a new structure where all bounties are predefined.
This is thus the last bounty transfer following our old structure which has been running since the start of the org.

Oliver

$650 FOSS4Gc
$2000 Project Management
$1000 Maintenance on MapLibre GL Native

Harel:  $3500 for Maintenance of MapLibre GL JS

Birk: $3500 for Maintenance of MapLibre GL JS / MapLibre GL Native / MapLibre GL JS Docs

Oh, thanks for the generous thank-you! You might want to highlight this in slack...


Feat: add empty ScannerBuildListener, options and dependencies

Hi :), related issue ( #896). PR contains new options for project scanner and dependencies import in pom and an empty class to be implemented.

The small PR is good, thanks. But the three changes are unrelated to each other.
If you want to add a listening feature to the Scanner, you would first add one interface, and at least one call to the interface method somewhere.

Hi :) , it will be implementing an interface provided by ActiveMQ later called MessageListener. So I think we don't have to write our own for that :) .

#904 need to be closed first before this continues :) .

@monperrus close to open a new PR ? :) .

Resolved :) .

@gluckzhang @monperrus , thank you :) . Awaiting for new comments.

Just went through relevant classes and found that:
The procedure of scanning projects should be the same, the difference is:

print the list of builds to a file
print the list of builds to stdout
print (send) the list of builds to the queue

If this makes sense, I think we could just enhance the method private void processOutput(Map<ScannedBuildStatus, List<BuildToBeInspected>> listOfBuilds) as follows:
private void processOutput(Map<ScannedBuildStatus, List<BuildToBeInspected>> listOfBuilds) {
    if (this.config.getOutputPath() != null && !this.config.getOutputPath().isEmpty()) {
        this.printToFile(listOfBuilds);
+   } else if (this.config.getQueueURL() != null && !this.config.getQueueURL().isEmpty()) {
+       this.sendToQueue(listOfBuilds);
    } else {
        this.printToStdout(listOfBuilds);
    }
}

Then we implement another method which sends build ids to the queue. WDYT? @monperrus

I think we could just enhance the method private void processOutput(Map<ScannedBuildStatus, List> listOfBuilds) as follows:

Good idea.

Hi :) , sorry I failed to understand what to fix. Should I add the implementation now ? .

Hi :) , sorry I failed to understand what to fix. Should I add the implementation now ? .

Hi, we could implement a method which sends build ids to the queue in class Launcher and update the processOutput method. See my previous comment, in this way you don't need to design new interfaces and you directly follow the original workflow :)

Hi :) , sorry I failed to understand what to fix. Should I add the implementation now ? .

Hi, we could implement a method which sends build ids to the queue in class Launcher and update the processOutput method. See my previous comment, in this way you don't need to design new interfaces and you directly follow the original workflow :)

Hi :) @gluckzhang, I have already implemented another way locally which more or less is similar to your suggestion and it also works after testing it out. Should I implement it in a new PR and let you check , if there are some problems then we can compromise or go with your suggestion, then perhaps we also will have a better view of other hidden bugs ? . WDYT ? :).

Should I implement it in a new PR and let you check

No, the goal is to get this PR merged, though the design may need to be optimized :)
For the current design, I have some doubts:

The code does not show how you monitor the luncher in ScannerBuildListener. Probably in runAsConsumerServer()? But do you need to change the Luncher a lot?

If you would like to design an interface called BuildListener, then ActiveMQListener should be one of its implementation. I am not sure if javax.jms.MessageListener is specifically for ActiveMQ or it is a general one. If later we have another queue system supported by Redis, is the current design easily extended?

About polymorphism, this PR (add support for different runner with polymorphism) could be a good example for you. Let's discuss more tomorrow offline :)

@Tailp I just found that we are discussing two requirements :)

Fetch project slug from the queue (before the inspection logic in Launcher)
Send build ids to the queue (after the main logic in Launcher)

So let's combine our ideas and get this solved tomorrow.

@gluckzhang , actually not much changes :) . Only a new function called kubernetesProcess() will be used instead of the so called mainProcess() . KuberntesProcess(String slug) will reuse already-existed scanner functions and no unncessary steps like serializations and check a list. I will be a bit hard to read if we try to embed our codes in the current mainProcess, which I found out some weeks ago when I coded it :) .

@Tailp I just found that we are discussing two requirements :)
* Fetch project slug from the queue (before the inspection logic in `Launcher`)

* Send build ids to the queue (after the main logic in `Launcher`)

So let's combine our ideas and get this solved tomorrow.

Let's do that :) . I can just show you my implementation tomorrow :) . Yes the scanner works as both a server listener and a publisher which make it a bit more tricky.

Yes the scanner works as both a server listener and a publisher which make it a bit more tricky.

Sounds a bit weird, the relevant issue (#896) is about add support to push build ids into the queue. Not sure if you need to keep the scanner runs continuously like RTScanner.

Yes the scanner works as both a server listener and a publisher which make it a bit more tricky.

Sounds a bit weird, the relevant issue (#896) is about adding support to push build ids into the queue. Not sure if you need to keep the scanner runs continuously like RTScanner.

Our implementation in python works that way :) . It listens for a slug in a scanner queue and then submit build ids after scanning to pipeline queue.

Per discussion today with @gluckzhang , we should implements MessageListener as a separate interface than extending BuildListener with it, in case if we want to implement BuildListener but not using ActiveMQ (MessageListener) :)

Per discussion today with @gluckzhang , we should implements MessageListener as a separate interface than extending BuildListener with it, in case if we want to implement BuildListener but not using ActiveMQ (MessageListener) :)

Not exactly. I mean as an interface BuildListener designed by you, it should be loosely coupled, i.e. it should not depend on ActiveMQ or relevant interfaces. You could have one implementation: ScannerBuildListener (or maybe call it ActiveMQBuildListener), which communicates with a queue system and needs to implement MessageListener interface. If later you would like to have another implementation, for example, a FileBuildListener which reads information from a file, you only need to implement your own BuildListener interface.

@gluckzhang :) hi, please have a look at the latest commit.

@gluckzhang thanks for the comments :) . Resolved.

@gluckzhang thanks for the comments :) . Resolved.

Thanks, LGTM!

Thanks a lot!


Reproducible example of layerState is undefined in inView function issue

Describe the bug
I am trying to add vector layer when "moveend" event fires, getting  "Cannot read properties of undefined (reading 'visible')"
To Reproduce
Steps to reproduce the behavior:
Go to example
See the

every time you change the code this bug occurs
Expected behavior
keep getting layerState as undefined, probably have to check layerState exists in function

Glad if it was helpful


test with batch size 1

There is a problem. In testing, one can not use a batch representation for a certain image. Especially, on RAF-DB, you are doing batch recognition (all images belong to the same category in most cases) if shuffe=False.
So, the accuracy is convinced only bs = 1.

And I think if shuffe=True in testing, the performance will degrade seriously.

I have tried the Pre-trained model "epoch59_acc0.9205.pth" in RAFDB, results are as follow：test accuracy is 0.9192 with test batch-size=64,shuffle=False;

" each batch just contain one class" can obviously raise the acc. number. However, as I said, this is batch recognition since you use a batch of sample features for recognition in test which is not the case in practice.

Have you obtained the accuracy close to 65.2 when training on AffectNet?
The results I retraining on AffectNet is 5 points lower than the given.

You are right. with the addition of shuffle, there are fluctuations in accuracy, but they are not always degraded (trained on affectnet).
And I tried that, each batch just contain one class, then acc will improve to 85.0+.

Have you obtained the accuracy close to 65.2 when training on AffectNet?
The results I retraining on AffectNet is 5 points lower than the given.

You are right. with the addition of shuffle, there are fluctuations in accuracy, but they are not always degraded (trained on affectnet).
And I tried that, each batch just contain one class, then acc will improve to 85.0+.

No, 64.2 is the highest acc I got.

Same question here. I believe that we should not change the test pipeline and use information from other images when evaluating the performance of the trained model.

Same question here. I believe that we should not change the test pipeline and use information from other images when evaluating the performance of the trained model.

that's it.

Thank you for finding the defect. A solution to the problem of Sharing Affinity block will be proposed in the near future. Prior to that, the other two blocks (FA and DA) in the paper can be used, independent of the SA block.

any progress ?

The authors did not disrupt the order of data reading during the test, so the samples of a batch are from only one class, and the features of each sample plus the average features of the batch will naturally improve the accuracy.  The rest of the modules used in the paper did not work.

The authors did not disrupt the order of data reading during the test, so the samples of a batch are from only one class, and SA makes the features of each sample plus the average features of the batch can naturally improve the accuracy.  The accuracy improvement comes mainly from the SA module, and the rest of the modules used in the paper did not work. This is why the training process does not see the gain, and the test does not work when the batch size is 1.

And I think if shuffe=True in testing, the performance will degrade seriously.

Yes, exactly. The test results in the paper are not convincing. If the shuffle=True(it should be True), the performance will decrease dramatically.


use specific VPP commit ID 10e7a9f8d8c8572be16f9b3f0395da6f8eff22ec

Coverage decreased (-0.06%) to 53.045% when pulling ee4be2b9c11a234f09a1c6248603df51a001a5de on rastislavszabo:lukasmacko-master into dc77fe48e9b43623adf45025627599a27c2bb53d on contiv:master.

Coverage decreased (-0.06%) to 53.045% when pulling ee4be2b9c11a234f09a1c6248603df51a001a5de on rastislavszabo:lukasmacko-master into dc77fe48e9b43623adf45025627599a27c2bb53d on contiv:master.


[Event Request] - New event OnBeforeCheckProdOrdLineStatus inside codeunit 99000773 "Calculate Prod. Order"

Hello, please consider adding the following event to Codeunit 99000773 "Calculate Prod. Order"
    [IntegrationEvent(false, false)]
    local procedure OnBeforeCheckProdOrdLineStatus(var ProdOrderLine: Record "Prod. Order Line"; var CalcRouting: Boolean; var ItemLedgEntry: record "Item Ledger Entry"; var IsHandled: Boolean)
    begin
    end;

and call it inside the "Calculate" procedure:
    procedure Calculate(ProdOrderLine2: Record "Prod. Order Line"; Direction: Option Forward,Backward; CalcRouting: Boolean; CalcComponents: Boolean; DeleteRelations: Boolean; LetDueDateDecrease: Boolean): Boolean
    var
        CapLedgEntry: Record "Capacity Ledger Entry";
        ItemLedgEntry: Record "Item Ledger Entry";
        ProdOrderRoutingLine3: Record "Prod. Order Routing Line";
        ProdOrderRoutingLine4: Record "Prod. Order Routing Line";
        RoutingHeader: Record "Routing Header";
        ProdOrderRoutingLine: Record "Prod. Order Routing Line";
        ErrorOccured: Boolean;
        IsHandled: Boolean;
    begin
        ProdOrderLine := ProdOrderLine2;

        IsHandled := false;
        OnBeforeCalculate(ItemLedgEntry, CapLedgEntry, Direction, CalcRouting, CalcComponents, DeleteRelations, LetDueDateDecrease, IsHandled, ProdOrderLine, ErrorOccured);
        if IsHandled then
            exit(not ErrorOccured);

        **IsHandled := false;
        OnBeforeCheckProdOrdLineStatus(ProdOrderLine, CalcRouting, ItemLedgEntry, IsHandled);
        if not IsHandled then**

            if ProdOrderLine.Status = ProdOrderLine.Status::Released then begin
                ItemLedgEntry.SetCurrentKey("Order Type", "Order No.");
                ItemLedgEntry.SetRange("Order Type", ItemLedgEntry."Order Type"::Production);
                ItemLedgEntry.SetRange("Order No.", ProdOrderLine."Prod. Order No.");
                if not ItemLedgEntry.IsEmpty() then
                    Error(
                      Text001,
                      ProdOrderLine.Status, ProdOrderLine.TableCaption(), ProdOrderLine."Prod. Order No.",
                      ItemLedgEntry.TableCaption());
...
...

Thanks !

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 52717.


Detect touch device and go from hover to click

Great plugin, I also like the visual result a lot.
One thing though (see http://iamceege.github.io/tooltipster/ examples):
If you browse with a mobile device or any "touch" device, you do not have a mouse.
Thus none of those tooltips will ever work (there is no hover triggable).
Here it is highly advised to be auto-aware as library and use existing functionality to auto-detect "touch" functionality and switch the behavior based on this.
Many image libraries do that for the swiping for example, in this case it is needed to keep the tooltips working.

Some googling revealed http://www.codechewing.com/library/add-click-or-hover-tooltips-depending-on-touch-support-tooltipster/ by the way - but I think this would be very valuable as core functionality.

Thank you for the kind words :)
The short answer is that hover does exist on mobile devices and that Tooltipster lets you do whatever you want on whatever devices, using the custom triggers. For example click on desktop and hover on phones. It's up to you to detect where you are (using modernizr for example) and configure your tooltips according to your wishes. Thank you.

Really? I don't have any hover on my Nexus 6 for example. None of the examples work.
My point is: There could be an option "touchDevice true/false" which then does this on its own.
Sure, everyone can implement sth custom, but a build in quick switch would be nice none the less :)
But I can also understand if you want to keep your library "simple".

Hover the examples for a few hundred milliseconds and you should see them work.  Check out the custom triggers section of the documentation if you haven't. My previous answer may have been misleading as in most cases you don't need to detect yourself unless you want to force a specific behavior on hybrid devices or deal with problematic environments like a webview in Qt.
A "touchDevice true/false" option used to exist and was not a good idea so it got removed. It implied that the configuration on desktop could just be "translated" to touch devices, when there are many cases where it's not straightforward. A hover on mobile is not as natural as on desktop for example and you'd often rather use a "pressed" trigger. Tooltipster's code is actually pretty complex because it completely differentiates click and touch events to let you enable the triggers of your choice.

That's what you asked for in the first place lol, so I'm not sure why I'm arguing, I should just say "use custom triggers to achieve it" and ask you for more details if it doesn't work ;)

I have used http://zeno.github.io/mobile-detect-demo/ and then
var md = new MobileDetect(window.navigator.userAgent);
var triggerOperation = 'click';
if(!md.mobile()) {
    triggerOperation = 'hover';
}

$('.tooltip').tooltipster({
    trigger: triggerOperation,
});

This solution works for me event on notebooks with touch screen.

The solution isn't working.

Not sure why the use of navigator.maxTouchPoints. Use the following code for simple and expected behaviour on mobile and desktop:
trigger: ('ontouchstart' in window) ? 'click' : 'hover'


Local scripts won't run at all on Mac

I'm using the following code on both a Windows machine and a Macbook in Node.js environment (with no module loaders of any kind):
var jsdom = require('jsdom');

module.exports = function(url, inject, injectPath) {

  var jsdomConfig = {};

  jsdomConfig.url = url;
  jsdomConfig.scripts = [`${__root}lib/jquery.js`];

  if ( inject ) jsdomConfig.scripts.push(injectPath);

  jsdomConfig.done = function(error, window) {

    // Expecting all the scripts to be executed
    console.log(window.$ === undefined); // Prints true

  };

  jsdom.env(jsdomConfig);

};

_root resolves to the project path (example: C:/myproject/).
The code runs perfectly on my Windows machine and the console.log prints false. However, on my Macbook it prints true.
I've tried all the suggested fixes (waiting for window to finish loading, etc.) but as long as the scripts are local, they never run on my Macbook.
I tried using a URL to load jQuery and it works! When I change the 5th line to jsdomConfig.scripts = ['https://code.jquery.com/jquery-3.1.1.min.js']; the console.log prints false in both my Macbook and the Windows machine. So I came up with a workaround (since the code is part of an Express.js webservice) and added a route to serve my local files using res.download() method. So instead of adding the local path of jQuery, I used something like http://localhost:8000/jquery and got it loaded when done() is called. Used the same method for other scripts passed to the function and they all work too.
But this adds unnecessary time to the whole process and I assume the downloaded file should be saved temporarily somewhere, so it occupies unnecessary space.

window will be a fully-loaded window, with all external resources downloaded and <script>s executed.

...quoting from the documentation. Clearly that is not happening!
Any ideas why local scripts are not executed when done() is called as opposed to non-local scripts and why is it Mac specific?

You need to provide a full URL to scripts. That includes the protocol.

The exact same code works on Windows but it doesn't work on Mac. The file path when running on Mac resolves to something like this /Users/ramtin/myproject/lib/jquery.js. What am I doing wrong?

That includes the protocol.

scripts does not accept file paths. scripts only accepts URLs and you're not providing a URL to a file path, but a relative URL.

But why does it work on Windows when I provide a local path? If I add C:/myproject/lib/jquery.js to jsdomConfig.scripts it works.
Would file:///Users/ramtin/myproject/lib/jquery.js work on Mac?

But why does it work on Windows when I provide a local path?

That might actually be a bug. It seems like we're assuming that the protocol in that URL is "C:" and the rest is a data-segment and then at a lower level we just check if a file on that path exists (which will return false for all http:// URLs). Haven't checked if that's actually the case though.

Would file:///Users/ramtin/myproject/lib/jquery.js work on Mac?

Yes.

This part of the documentation is a little bit misleading:

scripts: a string or array of strings, containing file names or URLs that will be inserted as <script> tags
Since it says file names or URLs I assumed that I can just add local paths to it. If file:/// would be included as a hint in the documentation it surely saves a lot of confusion.

Thanks for your help. I'll be using file:/// to load local-hosted libraries/scripts.

Ah I see. There's some logic to try to make it work with file paths, but that seems to only work for windows, I guess.

Just tried the file:/// approach on both Windows and Mac and it works perfectly. Thanks for the help @Sebmaster .

Be sure to try the new v10 API documented in the readme, which should make things much clearer! There you can inject script tags the normal way, using document.createElement("script") for example.


Security Enhancements To HMNFD Subscription API

Abstract
HMNFD's subscription API is not secure.   Enhancements need to be made in order to do proper
OPA filtering.
Problem Statement
Currently, agents that wish to subscribe for State Change Notifications (SCNs) do so using HMNFD's
/subscribe API.  The information pertinent to the subscription, as well as the identity of the subscriber, is
currently inside of the JSON payload sent to POST, PATCH, and DELETE API endpoints.
This makes OPA filtering difficult, since it has to parse the JSON payload in order to do any sort of filtering.
Agents (such as ORCA) subscribe for SCNs using the /subscribe API.  Since it is not currently filtered, bad actors could
use this API for DOS type attacks by overwhelming HMNFD with lots of bogus subscriptions.  OPA filtering
must be efficient to be effective.
Internal References
HMNFD swagger:  https://github.com/Cray-HPE/hms-hmi-nfd/blob/master/api/swagger.yaml
External References
N/A
Proposed Solution(s)
The HMS team intends to enhance the HMNFD API to include XName information in the subscription URLs
in order to facilitate very efficient OPA filtering.
The new API set would deprecate the following:
POST /hmi/v1/subscribe
PATCH /hmi/v1/subscribe
DELETE /hmi/v1/subscribe

and add the following:
# Create  a new subscription or replace an existing one
PUT /hmi/v2/subscriptions/{xname}/agents/{agent}

# Delete all subscriptions on a given node
DELETE /hmi/v2/subscriptions/{xname}/agents

# Delete a specific subscription on a given node
DELETE /hmi/v2/subscriptions/{xname}/agents/{agent}

Note that the existing subscriptions API will carry over to the v2 API (as will all other API endpoints not relevant to this document):
# Fetch subscription information
GET /hmi/v1/subscriptions

Note that this will require that the HMNFD API be implemented as a v2 API, requiring the deprecation of
the v1 API.
Impact of Action/Inaction
If we don't implement this V2 API, we will leave HMNFD open to the DOS-like vulnerability described above.
It is possible to do payload-based filtering in OPA but there are performance issues with that, and it may be
that large payloads (which are expected on large systems) could bog OPA down even more or even fail.
Further Information
This EP will need to go to the Solution Interoperability Team.    https://github.com/Cray-HPE/community/wiki/Enhancement-Proposals#solution-interoperability-team-sit-engagement_
Suggested Reviewers
Specifically:

@deantroe
@nieuwsma

And, the tech-leads team members.
Comment Period
Comment period for this proposal shall close on Nov 5, 2021.

This is being handled by https://connect.us.cray.com/jira/browse/CASMHMS-5231 and https://connect.us.cray.com/jira/browse/CASMHMS-5230


rootDir warning even when set to './src'

Current Behavior
It complains that rootDir is set to ./ when it's not
Expected behavior
It should not complain about it since rootDir is correctly set to ./src
Additional context
[tsdx]: Your rootDir is currently set to "./". Please change your rootDir to "./src".
TSDX has deprecated setting tsconfig.compilerOptions.rootDir to "./" as it caused buggy output for declarationMaps and occassionally for type declarations themselves.
You may also need to change your include to remove "test", which also caused declarations to be unnecessarily created for test files.

tsconfig.json:
{
    "include": ["src", "types"],
    "compilerOptions": {
      "target": "esnext",
      "module": "esnext",
      "lib": ["esnext", "dom"],
      "importHelpers": true,
      "declaration": true,
      "sourceMap": true,
      "rootDir": "./src",
      "noImplicitAny": false,
      "noImplicitThis": true,
      "noUnusedLocals": false,
      "noUnusedParameters": false,
      "noFallthroughCasesInSwitch": true,
      "moduleResolution": "node",
      "baseUrl": "./",
      "esModuleInterop": true,
      "resolveJsonModule": true
    },
    "exclude": ["node_modules", "**/*.spec.ts"]
  }

Your environment

Software
Version(s)

TSDX
^0.13.1

TypeScript
^3.8.3

Browser
-

npm/Yarn
-/1.22.4

Node
v12.16.1

Operating System
WSL 2 Ubuntu 18.04

Can you provide a minimal reproduction? Can't say what's going on otherwise.
There is a different code-path where it could be hit, but it's highly unlikely a user has it as you'd need to have a src/src/ directory structure, which would be very confusing usage.

@agilgur5 Just happened to me:

Clone https://github.com/gustavopch/firebase-event-sourcing
Run yarn to install the dependencies
Run yarn build
See the error

@gustavopch thanks for the repro. Very weird, it is indeed creating a dist/src/ directory even though you don't have a src/src/ directory. There are no other directories created either, so doesn't seem like you're importing from another directory. 🤔
The check is here, which is inside a deprecated function that moves type declarations from dist/src/ to just dist/:
https://github.com/formium/tsdx/blob/f0963cb2d77f00bcd8606f8e4b99250972d81b02/src/deprecated.ts#L13-L21
I'm pretty confused as to why that's happening, maybe related to new TS version you're using. However, I was able to workaround the issue by using "files": ["./src/index.ts"] instead of the "include": ["./src"]

You can also use "include": ["./src/*"] which seems to work fine as well.

@agilgur5 Wont' that cause some undesired side-effect like ignoring any files that are not placed directly in the src folder?

Well you already had src, src/* isn't really any different except that it avoids this error. Using files means only things that are actually imported are compiled, which is even better.
It's also in your tsconfig.build.json and you only want to include src when building. If you include files that aren't in src, TS will type-check, compile, and create declarations for those files too. In the past that has meant erroneously adding tests' or examples' declarations into dist, which shouldn't be there as library users don't use those (and that causes bloating as well as potential side-effects due to how TS merges declarations)

@agilgur5 Unfortunately, dist/application/definitions is not generated when I apply that workaround.

so no fix for this issue yet rt, cause the work around is not working

Hi,
I am also getting this error when I start a project using pnpm run start

here is my tsconfig.json
{
"include": ["src"],
"compilerOptions": {
"module": "esnext",
"lib": ["dom", "esnext"],
"importHelpers": true,
"declaration": true,
"sourceMap": true,
"baseUrl": "src",
"strict": true,
"noImplicitReturns": true,
"noFallthroughCasesInSwitch": true,
"noUnusedLocals": true,
"noUnusedParameters": true,
"moduleResolution": "node",
"jsx": "react",
"esModuleInterop": true,
"skipLibCheck": true,
"forceConsistentCasingInFileNames": true,
"noEmit": true,
"resolveJsonModule": true,
"noImplicitAny": false
}
}
here is my tsdx.config.json
const copy = require('rollup-plugin-copy');
const replace = require('@rollup/plugin-replace');
module.exports = {
rollup(config, opts) {
config.plugins = config.plugins.map((p) =>
p.name === 'replace'
? replace({
'process.env.NODE_ENV': JSON.stringify(opts.env),
preventAssignment: true,
})
: p
);
config.plugins.push(
  copy({
    targets: [{ src: 'src/*.d.ts', dest: 'dist/' }],
  })
);

return config;

},
}


Supress UserCrypto Warning

Expected Behavior
I can run jobs on a Windows server without having the user profile loaded and without the job producing the error "UserCrypto is not supported.  This may be due to use of a non-Windows operating system or Windows Nano Server, or the current user account may not have its profile loaded. CryptProtectData failed. Error Code: -2146892987". Would like to be able to configure an App config setting to disable UserCrypto or suppress this error.
Current Behavior
When running automated jobs over WinRM on a server, the running environment does not have a user profile, so using the SDK we see the error "UserCrypto is not supported.  This may be due to use of a non-Windows operating system or Windows Nano Server, or the current user account may not have its profile loaded. CryptProtectData failed. Error Code: -2146892987". Credentials have been supplied using the App config so it is not necessary for UserCrypto to be used.
I would like a way to suppress the error or prevent the SDK from trying to use UserCrypto.
Steps to Reproduce (for bugs)
Tell me if you need a MRE, I can produce, but basically any CLI app that uploads a file to S3. Run this app remotely using WinRM or on a non-Windows operating system, or Window Server Nano.
Context
This issue caused concern that our app was broken by producing a misleading error message.
Your Environment

AWSSDK.Core version used: <IP_ADDRESS>
Service assembly and version used: AWSSDK.S3 version=<IP_ADDRESS>
Operating System and version: Windows Server 2016
Visual Studio version: 2017
Targeted .NET platform: 4.6.2

Also mentioned in https://github.com/aws/aws-sdk-net/issues/1072


Time series property

When opening the time series property from the unit testing package : grid interp -> two faulted sugar cubes (explicit geometry) ->Time Series Property :

No datetime appears in combobox
Nothing appears as well in the time step slider
Two properties are named "Time Series Property"

Could it be cleaned?
Either by putting each timestep in the treeview or by using the combobox and the slider please?

The unit testing package actually contains three properties named "Time Series Property", two show up on the Selection widget .


feat(Convex/Basic): add a @[simp] lemma convex_vadd

This is a simple corollary of the existing Convex.vadd.

Thanks!
bors merge


Error when trying in ReadGoodReceiptList flow

All the flows that use that step I added here in the SS fail with the same error:
"code": 502,
"message": "BadGateway",
"innerError": {
"error": {
"code": "RfcAbapRuntimeException",
"message": "An SAP ABAP Runtime exception occurred when calling an RFC. Exception message 'An error has occurred while parsing a dynamic entry.', RFC exception error code '204', ABAP exception key 'SAPSQL_PARSE_ERROR', ABAP message type 'X', ABAP message class '', ABAP message number '', ABAP message parameters ''.",
"target": "Microsoft.Integration.LobServiceBroker.Broker.d__12.MoveNext()\r"

I changed the filter and is now working used: ["LIFNR EQ '@{item()?['LIFNR']}'"]


Guide on memory/disk usage?

Hi, I'm trying to test meilisearch by ingesting existing dataset served by ElasticSearch to MeiliSearch.
Is there any guide on memory / disk space / disk IO usage of the search engine? I inserted 10M documents, 5GB JSON dataset with 18 fields, 4 text fields, 14 numeric/boolean fields.
MelliSearch is using

memory: RSS 17GB, virtual memory 250GB
79GB of disk space, 77GB main, 2GB update
reading 200~300MB/s from SSD while inserting documents (from LMDB memory-mapped file?)

Is there any guidelines for expected resources/machine size for the workload size?

Hi @yjh0502
You can use these calls to get system information:
https://docs.meilisearch.com/references/sys-info.html#get-pretty-system-information
https://docs.meilisearch.com/references/sys-info.html#get-system-information

Hey @yjh0502,
We are currently working on a document that we will put in the documentation website.
It will summarize how MeiliSearch and mostly LMDB, the key-value store we use internaly.
We will keep you up in this issue.

Hello @yjh0502, I just wrote a guide about MeiliSearch storage
Let me know if anything is still unclear or if you need any more precision


Fix meta's valuer when relation is pointer-based

@danialgood: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@danialgood: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@danialgood: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@danialgood: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@danialgood: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

no test


Change [Scrutinizer] branch coverage test target

Fixes #2629

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @calebcartwright!

  Generated by :no_entry_sign: dangerJS against 7eb515b7b4f1bf5ddee808000b599b8ddac50f2c


[BUG] Version 2.6.2 built with react-script build results in a runtime error

Description
After upgrading to<EMAIL_ADDRESS>(from 2.6.1) in a react project, the built solution (via react-scripts) produces a runtime error, resulting in a blank screen.
The following error is present in the console:
ReactPlugin.ts:103 Uncaught ReferenceError: __extends is not defined
    at ReactPlugin.ts:103
    at Object.<anonymous> (ReactPlugin.ts:17)
    at f ((index):1)
    at Object.53 (main.5d60f9e0.chunk.js:1)
    at f ((index):1)
    at Object.28 (main.5d60f9e0.chunk.js:1)
    at f ((index):1)
    at Module.345 (main.5d60f9e0.chunk.js:1)
    at f ((index):1)
    at r ((index):1)

Steps to Reproduce

Setup a create-react-app project with<EMAIL_ADDRESS>&<EMAIL_ADDRESS>Build the project
Serve the build folder
When accessing the webpage, the screen is blank with the error above in the console.

OS/Browser: Windows 10, Chrome
SDK Versions:

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

Additional info
This issue only occurs when built. Starting a development server works fine.

Same behavior as above.

This is a duplicate issue and is related to these issues (which include workarounds)
#1527, #1281, #1280, #1269
Additionally, v3.1.1 of the react plugin will address this as it will no longer be using the applicationinsights-shim v1.0.3 module
PR for version bumps #1552
Closing this specific issue due to duplications

React v3.1.1 and ReactNative v2.3.1 are now published to NPM

I've deprecated the react v3.1.1 and react-native v2.3.1 packages until I can publish an updated versions #1553

I am new to this NPM and I am facing the same error with the below dependencies in dev environment. The dev environment used to work fine till Friday.
"@microsoft/applicationinsights-react-js": "^3.1.0"
"@microsoft/applicationinsights-web": "^2.6.0"
"react-scripts": "3.4.0"
Can some one explain why this is happening all of a sudden?, Please let me know the possible work-around to overcome this issue.

The __extends issue has be "around" since May 2020 (because of #1269 and the workaround we put in place), as part of the workaround we created the application-shims module which exposed global versions of the __extends() and __assign() tslib helpers, and as these where imported in the index.ts of each module it was "generally" available (in devlopment mode), when using production mode for several packagers this caused the code that "exposes" the global versions of the tslib helpers to not get included and thus the exceptions __extends() not found and others.
The workaround for this for webpack IS to use the ProdiverPlugin and force these helpers to always be present in every created chunk and use either the tslib versions or the ai-shims version.
    config.plugins.push(
      new webpack.ProvidePlugin({
        '__assign': ['@microsoft/applicationinsights-shims', '__assignFn'],
        '__extends': ['@microsoft/applicationinsights-shims', '__extendsFn']
        // '__assign': ['tslib', '__assign'],
        // '__extends': ['tslib', '__extends']
      })

And as part of v2.6.2 we "fixed" (removed) the globals for the main Application Insights module (#1280) and all usages of these modules no longer need to use the workaround as during packaging these are now re-written to directly use the new v2.0.0 of the ai-shims components.
The missed / unexpected issue however, is that because we (currently) have to wait for the main modules to be published before we can publish the react and react-native plugins, the code directly in these plugins still require the global functions to be present (and they are defined in the main modules -- so dev / non-chunked versions work), and because the main v2.6.2 no longer import and expose the globals it now requires that either the ReactPlugin is included in totality or the above work-around is used.
I attempted to "fix" this eariler today by publishing v3.1.1, which "mostly" worked, however, the packaging step to re-write the usage got missed, thus making v3.1.1 a major breaking version (this is why I deprecated it).
I am planning to publish v3.1.2 to complete the fix and removed of the globals by the end of tomorrow (was trying for today, but validating the fix is a little more complex) the Issue I'm using for this is #1553.

I currently have the react plugin in a state that looks good and I'm finalizing the react-native changes
Our publish automation currently builds and deploys these 2 components together.


.net core zip+aes ReadAsync failed on one of files

Steps to reproduce
1.open archive with password "TestStorage"
2.try extract file "[www/autopilotandbridge/home.png]" using
var entry = _zf.GetEntry(path);
var zipStream = _zf.GetInputStream(entry);
var buffer = new byte[entry.Size];
var result = await zipStream.ReadAsync(buffer, 0, buffer.Length);

Expected behavior
file is extracted
Actual behavior
Got exception
TransformFinalBlock is not implemented and inputCount is greater than 0
   at ICSharpCode.SharpZipLib.Encryption.ZipAESTransform.TransformFinalBlock(Byte[] inputBuffer, Int32 inputOffset, Int32 inputCount)
   at System.Security.Cryptography.CryptoStream.<ReadAsyncCore>d__43.MoveNext()
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.ConfiguredTaskAwaitable`1.ConfiguredTaskAwaiter.GetResult()
   at System.Security.Cryptography.CryptoStream.<ReadAsyncInternal>d__38.MoveNext()
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
   at ResourceCollection.LazyZipDirectoryLoader.

Version of SharpZipLib
Obtained from (only keep the relevant lines)

Package installed using NuGet v1.3.1
ConsoleApp2.zip

It looks like calling ReadAsync on the CryptoStream in .NETCore 3.1 causes it to go through an internal code path that tries to do the crypto transform work itself, rather than using the Read() implementation in ZipAESStream (on .NET 4.6, it does seem to call through to Read to do the work).
That implementation calls ZipAESTransform.TransformFinalBlock(), which isn't really implemented (it doesn't need to do anything with the Read impl in ZipAesStream.)
TransformFinalBlock is called with a inputCount of 28, which is (presumably not cooincidentaly) the combined size of the AES header and auth block in the encrypted stream.
A few quick thoughts:

We could possibly give ZipAESStream an implementation of ReadAsync() so the version in CryptoStream isn't called.
We could maybe give the CryptoStream constructor a view of the base stream that only includes the actual encrypted data, and not the header and auth code parts (so they aren't included in any size calculations)?
Review the way the transforms are handled and maybe to the final block/auth code handling in way that better fits in with what CryptoStream wants.

Yeah, 3 is probably the "right" way, and 1 is probably the "safest" way.
If that wasn't clear @ShadowTeolog, your code should work if you use the non-async read:
var result = zipStream.Read(buffer, 0, buffer.Length);

Yes, sync read work flawless.
But this should be fixed somehow anyway or throw exception on any attempt to ReadAsync on ZIP+AES.  Too many people will be puzzled by an error like: (only this file out of a hundred cannot be read), a lot of projects will be moved to .net 5.0 from .net Framework

only this file out of a hundred cannot be read

I think it's only currently an issue with stored entries (because for deflated entries, the call to ReadAsync goes through InflatorInputStream, which itself only makes sync calls to the base stream, whereas for Stored entries, it does async reads on ZipAESStream).


Support compare functions with SortSlices and SortMaps

The SortSlices and SortMaps options predate generics and
accept an interface{}, so it is possible with reflection to support other function signatures than "func(T, T) bool".
In particular, the Go ecosystem is increasingly moving towards "func(T, T) int" as the signature for ordering as evidenced by the newer slices.SortFunc function in stdlib.
Thus, modernize cmpopts by supporting "func(T, T) int".
Fixes #365

CI are hosed and I don't have permissions to fix them, so you may need to merge on my behalf.


Xcode15.4 compilation error

Hello, I used Xcode15.4 to compile, and the error message was prompted: XPC module is repeatedly defined.

Sorry for not getting back to you sooner, but as of the next release of TrollInstallerX (1.0.3), it will only be compilable by default on Xcode 15.4+ (fixing your issue). Anyone on a lower version will need to add the XPC headers themselves.


Add scalastyle pre-commit hook

Add pre-commit hook for Scalastyle, a style checker for Scala.

Thanks! Merged in efc6c8141ea254ac39c99de8b6464ce99bd02ed8.


feat: 🎸 MSDK-759 Allow creating a token without ticker reservation

JIRA Link
https://polymath.atlassian.net/browse/MSDK-759
Changelog/Description
Added a createToken procedure in CurrentIdentity that allows creating Security Token even with an unreserved ticker.

:tada: This PR is included in version 13.0.0-alpha.6 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 14.0.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 14.0.0-beta.1 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


SA1028 code fix fails to fix all AssemblyInfo.cs

Repro: Create a new (console) project, all the analyzer reference. SA1028 is reported for a few places in the AssemblyInfo.cs file. The violations can be auto fixed one at a time, but when trying to fix all nothing will happen. No preview window will open.

Note that the code fix does work on any single item in the file, which means this file will pass our current test suite (which applies fixes one at a time). Only the batch fixer fails.
I have a suspicion that this was fixed by dotnet/roslyn#2286, but we won't know for sure until we can test this with a post-RC release.

I'm getting this on the comments and the fix only works one instance at a time. The batch fix doesn't even work for document,

And I'm getting the same for SA1518.

@paulomorgado The batch fixer was coded to treat AssemblyInfo.cs as generated code and disables all batch fixing in it.


Added ajeffrey's ssh key.

Added my ssh public key to common/ssh/ and added my name to common/map.jinja.

common/map.jinja, line 16 [r1] (raw file):
Grr, emacs tabs.

Comments from the review on Reviewable.io

@bors-servo r+

:pushpin: Commit c5fa192 has been approved by Manishearth

:hourglass: Testing commit c5fa192 with merge 1478233...

:broken_heart: Test failed - travis

@bors-servo retry

:hourglass: Testing commit c5fa192 with merge c9fbcc9...

:sunny: Test successful - travis


Can't satisfy dependency on UIImageViewModeScaleAspect (v 2.5.0)

[!] Unable to satisfy the following requirements:

- `UIImageViewModeScaleAspect (~> 1.5.0)` required by `BSImagePicker (2.5.0)`

Shouldn't it use UIImageViewModeScaleAspect v. 2.0?

Looks like you should require UIImageViewModeScaleAspect 1.5, not 1.5.0. Turns out, it makes difference )

I've created a fork w/ a tmp fix https://github.com/nikans/BSImagePicker

Great! Pull requests are welcome 🙂

Which version of cocoapods where you using? 1.1.0 managed to resolve 1.5.0 to 1.5
I've pushed 2.5.1 which has 1.5 in the podspec, hope that works better for you.


Update Lab_13_ConfigureAndDeploySelfServicePasswordReset.md

Module: 00
Lab/Demo: 00
Fixes # .
Changes proposed in this pull request:

Submitted by ParamRamki on behalf of MOC Labs Review


add AzureKeyCredentialPolicy.cs into project file

Description
Fix https://github.com/Azure/autorest.csharp/issues/3773
Fix https://github.com/Azure/autorest.csharp/issues/3569

Add AzureKeyCredentialPolicy.cs into project file(.csproj) when the sdk need azure key authentication.

Checklist
To ensure a quick review and merge, please ensure:

[ ] The PR has a understandable title and description explaining the why and what.
[ ] The PR is opened in draft if not ready for review yet.

If opened in draft, please allocate sufficient time (24 hours) after moving out of draft for review

[ ] The branch is recent enough to not have merge conflicts upon creation.

Ready to Land?

[ ] Build is completely green

Submissions with test failures require tracking issue and approval of a CODEOWNER

[ ] At least one +1 review by a CODEOWNER
[ ] All -1 reviews are confirmed resolved by the reviewer

Override/Marking reviews stale must be discussed with CODEOWNERS first

I thought the solution is supposed to be: only when azure key credential is used, we add it to project file.
This solution looks a workaround to me.

only when azure key credential is used, we add it to project file.
This solution looks a workaround to me.

Yes. @pshao25 we only add AzurekeyCredentialPolicy to project file when azure key credential is used. See  in the code we will check the actual authentication used, then decide to add this or not.

Can we have a test case which requires AzureKeyCredentialPolicy.cs to compile?

Actually there are some projects in autorest.csharp which need AzureKeyCredentialPolicy.cs, so in this PR, we update the project files for them. No need to add more.


Expose JsonSchema

Hello and thanks you for this amazing library.
I love the builder's ability to create JsonSchema. However, it seems that it's only possible to get a JsonValidator to validate a JSON. Is it possible to get the built JsonSchema?
My aim is to expose it via an API so that the frontend can automatically generate a dynamic form. The JsonSchema structure should be a valid Json schema according to the specification https://json-schema.org/specification.
Thanks !

Hi @L3Z4 Thanks for feedback.
Do you mean you want to be able to get that standard json schema "text" eventually after building JsonSchemaBuilder ?
Can I also understand more about your usecase about this? Previously this library focused on validation (from multiple possibilities to JsonValidator) but would happy to extend more functionalities.

Indeed I'd like to get the JsonSchema in "text" version to expose it via a REST Api.
Then I can use a frontend form generator like https://github.com/rjsf-team/react-jsonschema-form. Once the form is generated, the end-user should be able to fill it and I POST it to my REST Api.
Finally, I can use the initial JsonSchema to validate this model.
I cannot use C# classes to expose the model and deserialize it, because my model is dynamic.
I'm currently using https://github.com/gregsdennis/json-everything/tree/master/JsonSchema to do so, but it is quite challenging to extend the JsonSchema with custom attributes

@L3Z4 Ok, I would plan to implement this direction (from JsonSchema back to json schema text), thanks for idea.
BTW, this library targets json schema spec - 2020.12 version, hope you are able to adopt this version.

Thanks you !
Adapting to latest spec version is not an issue for me.

@L3Z4 the json schema output is supported in v1.1.4, document here, please try it, hope it can help.

Hi @L3Z4  any issue you found after adopting new one ?

Feel free to reopen if need, @L3Z4 .


Release 0.14.0

[x] Test in kubernetes cluster (it's already deployed in service-prod)
[x] Test scope in service mode (to avoid things such as https://github.com/weaveworks/scope/issues/1162 which we introduced in 0.13. If scope.weave.works is not backwards compatible, test against a separate local VM)
[ ] Test with ECS example, I've done some testing but probably wasn't enough (The ECS example is already deployed in standalone mode with this branch's version, @paulbellamy has access)
[x] Test the two example plugins and verify that documentation is friendly and clear (ideally done by somebody totally new to the scope plugins)

LGTM from me.

LGTM too

Three

LGTM


fix(store): normalize pattern ids across platforms

This fixes pattern detection on windows

@marionebl says split/join is not that wrong, and he convinced me.
The original paths are OS-specific, then joined OS-specific, and then we should split them OS-specific, and join again using the '/' character, as the join is a pattern ID, not a path.
Additionally, the RegEx construction is not perfectly correct. The string is a regex, but PathUtils.sep is not.
So we should split(PathUtils.sep) and join('/').

@TheReincarnator done


extractParameterValues checks for empty(), disabling value '0'

In Route->extractParameterValues, there is a check for if (!empty($value)), which means that the 0 in /api/users/123/access/0 is skipped. Then there is no route match and a 404 follows.
I think checking for an empty string is enough. 0 or '0' should be passed as values as well. Only empty strings are not valid URL parts. The URL is always a string, so NULL will never happen here. I therefore think checking for an empty string is enough to check for in extractParameterValues.

Fixed in #49


Adding an intellij project to local git gives code analysis errors on commit

Steps to Reproduce
Create a new Intellij flutter project
VCS/Import into version control/Create GIT repository
Commit changes on unversioned files
Browse and add unversioned files
Select all the files
Do the commit
Code analysis dialog box says there are errors

If you review the errors things are very mysterious and seem like there is a big problem.
If you just commit the project works fine
Flutter Doctor
[✓] Flutter (on Linux, channel master)
• Flutter at /home/darrell/flutter
• Framework revision ca2bf1efd0 (3 hours ago), 2017-04-20 11:37:10 -0700
• Engine revision a5b64899c9
• Tools Dart version 1.23.0-dev.11.7
[✓] Host Executable Compatibility
• Downloaded executables execute on host
[✓] Android toolchain - develop for Android devices (Android SDK 25.0.2)
• Android SDK at /home/darrell/Android/Sdk
• Platform android-25, build-tools 25.0.2
• Java binary at: /home/darrell/android-studio/jre/bin/java
• Java version: OpenJDK Runtime Environment (build 1.8.0_112-release-b736)
[✓] Android Studio (version 2.4)
• Android Studio at /home/darrell/android-studio
• Gradle version 3.4.1
• Java version: OpenJDK Runtime Environment (build 1.8.0_112-release-b736)
[✓] IntelliJ IDEA Community Edition (version 2017.1)
• Dart plugin version 171.4249.16
• Flutter plugin version 12.1
[✓] Connected devices
• Android SDK built for x86 • emulator-5554 • android-x86 • Android 7.1.1 (API 25) (emulator)

I can repro this. The errors and warnings are:
**/Users/mit/tmp/testapp/android/gradle.properties**
Warning:(1, 1) Unused property
**/Users/mit/tmp/testapp/android/app/src/main/AndroidManifest.xml**
Error:(1, 2) Cannot find the declaration of element 'manifest'.
Error:(1, 26) URI is not registered (Settings | Languages & Frameworks | Schemas and DTDs)
Error:(3, 5) Attribute android:versionCode is not allowed here
Error:(4, 5) Attribute android:versionName is not allowed here
Error:(19, 91) Attribute android:icon is not allowed here
Error:(21, 19) Attribute android:launchMode is not allowed here
Error:(22, 19) Attribute android:theme is not allowed here
Error:(23, 19) Attribute android:configChanges is not allowed here
Error:(24, 19) Attribute android:hardwareAccelerated is not allowed here
Error:(25, 19) Attribute android:windowSoftInputMode is not allowed here
Error:(32, 3) Cannot find the declaration of element 'manifest'.
 I believe it may have been fixed as a side effect of https://github.com/flutter/flutter/pull/10928.
@darrellpf can you try to flutter upgrade, create a new project, and see if things are working as expected for you?

First, when the Android framework detected message comes up it is necessary to click the dialog and then accept the Android manifest xml file. If you don't there are quite a few errors from the xml file.
Even with that there is still a few warnings and one error

[✓] Flutter (on Linux, locale en_US.UTF-8, channel master)
• Flutter at /home/darrell/flutter
• Framework revision 5f20f83415 (28 hours ago), 2017-07-02 09:34:31 -0700
• Engine revision 8ad1576f27
• Tools Dart version 1.24.2
[✓] Android toolchain - develop for Android devices (Android SDK 26.0.0)
• Android SDK at /home/darrell/Android/Sdk
• Platform android-26, build-tools 26.0.0
• ANDROID_HOME = /home/darrell/Android/Sdk
• Java binary at: /home/darrell/android-studio/jre/bin/java
• Java version OpenJDK Runtime Environment (build 1.8.0_112-release-b06)
[✓] Android Studio (version 2.4)
• Android Studio at /home/darrell/android-studio
• Java version OpenJDK Runtime Environment (build 1.8.0_112-release-b06)
[✓] IntelliJ IDEA Community Edition (version 2017.1)
• Flutter plugin version 15.0
• Dart plugin version 171.4694.29
[✓] Connected devices
• None

Did some digging. These errors are unrelated to source control; a smaller set of repro steps are:

Start IntelliJ IDEA
Create a new Flutter project
Configure Java support as per https://flutter.io/intellij-ide/#edit-android-code
Invoke the menu item 'Analyze > Inspect Code...'

Get a number of errors, warnings, and lint issues.

I'm leaving this in the current milestone since it's sort of a tool new user blocker (at least for users who are going to write OEM-side code), and it's sort of an IDE getting started blocker, and it's sort of engine maintenance...

Yes, let's leave. I have a hunch this is caused by the same root issue as https://github.com/flutter/flutter-intellij/issues/1482


☁ FNaF 3 --Cloud Saves!-- ☁

http://scratch.mit.edu/projects/55932084/
http://phosphorus.github.io/#55932084
Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/39.0.2171.71 Safari/537.36
Many of the sound effects that are supposed to play don't play.

Presumably relates to #260


DataReference, SimpleTableSelect & PromotedFilters component changes added…

Works ok for initial commit. One thing I noticed that can be fixed in future updates is that updating a search doesn't work as expected. For example, I can do an initial search for "pr" and get back 2 products with "pr" in the product name. But, if I update the text input and replace "pr" with "green" and then click the "Search" button, the search results don't update. However, if I "Clear" the search and then type in "green", I do get the search to that shows 1 record found (for "Green item"). This can be fixed in a later commit.

sure john, I will fix the above issue in next commit, Thankyou!


Update starknet-api version

Update the starknet-api version in all crates to 0.8.0

@juanbono please i will like to work on this issue