Web client "freezes" if caption class property is used in ListPart sub-pages

Describe the bug
Under certain conditions, the WebClient "freezes" when a sub-page (ListPart) of a CardPage uses the CaptionClass property. This "freezing" of the WebClient manifests as follows:

The current page can no longer be closed.
No more actions can be executed (including new record / edit record / change record / search / settings / ...)

To Reproduce
The bug occurs only under the following specific conditions:

Multiple instances of the same ListPart page are used as sub-pages in a CardPage
The ListPart sub-pages are child elements of a group element in the CardPage.
In the ListPart sub-pages, the CaptionClass property is filled for at least one field (with a value <> '').
The group in which the ListPart sub-pages are located is collapsed when the card is opened (important!).

To reproduce the bug I wrote a simple app: https://github.com/haweg/ALIssue001
In this app the behavior can be easily reproduced as follows:

Search / open "Test Document List"
Create a new record (CardPage "Test Document Card" opens)
If the "Details" tab is unfolded when the CardPage opens, the tab must be collapsed and the CardPage must be reopened to reproduce the bug.
Now you can observe the described behavior on the "Test Document Card".

Expected behavior
The client should not "freeze" under those conditions.
Screenshots
Let me know if you need any further documentation or screenshots.
5. Versions:

AL Language: 4.0.191880

Business Central: Platform 15.0.37492.0 + Application 15.0.36626.37543

Hi @haweg ! We appreciate that you took the time to write such a good bug report! Unfortunately, we can only handle issues related to the latest developer tools on this GitHub repository. Would you mind opening this issue on Collaborate and referencing this issue?

The fix for this issue has been checked in to the master branch. It will be available in the bcinsider.azurecr.io/bcsandbox-master Docker image starting from platform build number 37893 and VS Code Extension Version .
If you don’t have access to these images you need to become part of the Ready2Go program: aka.ms/readytogo
For more details on code branches and docker images please read:
https://blogs.msdn.microsoft.com/nav/2018/05/03/al-developer-previews-multiple-releases-and-github/
https://blogs.msdn.microsoft.com/freddyk/2018/04/16/which-docker-image-is-the-right-for-you/


Improve attribute error msg for unbounded modules

What does this PR do?
Fixes #1014. When  self.scope is None, AttributeError's message raised by __getattr__ for unknown fields now reminds the user it should call apply when looking for attributes that might be defined inside .setup().

Codecov Report

Merging #2440 (8fdc9f4) into main (6784fbd) will decrease coverage by 0.03%.
The diff coverage is 100.00%.

:exclamation: Current head 8fdc9f4 differs from pull request most recent head 94d1cfa. Consider uploading reports for the commit 94d1cfa to get more accurate results

@@            Coverage Diff             @@
##             main    #2440      +/-   ##
==========================================
- Coverage   79.72%   79.69%   -0.04%
==========================================
  Files          49       49
  Lines        4992     4984       -8
==========================================
- Hits         3980     3972       -8
  Misses       1012     1012

Impacted Files
Coverage Δ

flax/linen/module.py
92.75% <100.00%> (-0.11%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Cheatsheet/Transformation scenario

Propositions of display (reflecting exchanges with Arlo):
<orig>abc</>  --->  ¡abc!
<sic>abc</>    --->  ¿abc?   (NB: Arlo, you suggested ¡¿abc!?. I find ¿abc? lighter)
<reg>abc</>    --->  ⟨abc⟩
<corr>abc</>  --->  ⟨abc⟩
<supplied reason="omitted">abc</>   --->   ⟨abc⟩
<surplus>abc</>                                  --->   ⟨⟨abc⟩⟩
Thus:
<choice><sic>X</sic><corr>Y</corr></choice>     --->   ¿X?⟨Y⟩
<choice><orig>X</orig><reg>Y</reg></choice>   --->   ¡X!⟨Y⟩
((The idea behind ¿X?⟨Y⟩ and  ¡X!⟨Y⟩ is that the brackets for <corr> and <reg> should be the same as those we would use for <gap reason="omitted">))

What about pattern such as <reg><orig>u</orig><corr>i</corr></reg>?

All of the above looks fine to me, except that {abc} would be better for <surplus>. Are we planning to use {} for a different purpose? If not, then using it for suppression would be conformant with the Leiden conventions and in my opinion simpler to read than ⟨⟨abc⟩⟩.
Axelle: the combination <reg><orig>u</orig><corr>i</corr></reg>cannot occur, since neither <reg> nor <corr>  will ever be used outside a <choice>  that includes the original text, and neither <orig> nor <corr> will ever be used inside a <reg>.
Did your question concern cases where a normalisation is added on top of a correction, as in
<orig>mahār<choice><sic>a</sic><corr>ā</corr></choice>jñaḥ</orig> ?
I see no problem with displaying these by the above rules as
¿mahār¡a!⟨ā⟩jñaḥ?
or, with full normalisation instead of just flagging:
<choice><orig>mahār<choice><sic>a</sic><corr>ā</corr></choice>jñaḥ</orig><reg>mahārājasya</reg></choice> ?
displayable as
¿mahār¡a!⟨ā⟩jñaḥ?⟨mahārājasya⟩
The latter may be a bit ambiguous, with two different uses of ⟨⟩, but I think we can live with that given that correction within a normalised bit will be very rare.
Manu, Arlo?

I fully agree with Dániel's comment:
"All of the above looks fine to me, except that {abc} would be better for . Are we planning to use {} for a different purpose? If not, then using it for suppression would be conformant with the Leiden conventions and in my opinion simpler to read than ⟨⟨abc⟩⟩."
I am also very much in favour of the use of {}.

<reg><orig>u</orig><corr>i</corr></reg happens in several files...

To summarise:
<orig>abc</>     ---> ¡abc!
<sic>abc</>       ---> ¿abc?
<reg>abc</>      ---> ⟨abc⟩
<corr>abc</>     ---> ⟨abc⟩
<supplied reason="omitted">abc</>    ---> ⟨abc⟩
<surplus>abc</>                                    ---> {abc}
I was thinking of {...} for <g type="symbol" subtype="finalGomutra"/> ---> {finalGomutra}
Let thus duplicate it, thus
<g type="symbol" subtype="finalGomutra"/> ---> {{finalGomutra}}
As for
<orig>mahār<choice><sic>a</sic><corr>ā</corr></choice>jñaḥ</orig>
as per the rules above, it should be
¡mahār¿a?⟨ā⟩jñaḥ!
Not
¿mahār¡a!⟨ā⟩jñaḥ?
(I prefer this to full normalisation, which however we can allow)
(and in any case, using <orig> in this case seems to me redundant)
As for
<reg><orig>u</orig><corr>i</corr></reg>
could you, Axelle, identify the culprit and inform his/her PI?

The advantage of Manu's/my first proposal
<supplied reason="omitted">abc</> ---> ⟨abc⟩
<surplus>abc</> ---> ⟨⟨abc⟩⟩
would have been consistency with the conventions used in EIAD, which in turn were rather heavily influence by the (German) buddhologists/gandharologists who were initially involved in EIAD.
But I can live with Manu's summary and use of
<surplus>abc</> ---> {abc}
Instead of Manu's proposal
<g type="symbol" subtype="finalGomutra"/> ---> {{finalGomutra}}
I suggest something evocative of the fact that finalGomutra represents a symbol (σύμβολον), so I can imagine solutions like these:
<g type="symbol" subtype="finalGomutra"/> ---> «finalGomutra»
<g type="symbol" subtype="finalGomutra"/> ---> σfinalGomutraς
Or no enclosing signs at all, and simply a special color, or underlining, or other such special formating on the @subtype.

sorry, one more thing, we will also need conventions for abc and <add>abc</add>. this is what I would propose to displayed with {abc}.
So summary of my suggestions:
<unclear>abc</unclear> ---> (abc)
<supplied reason="lost">abc</unclear> ---> [abc]
<supplied reason="omitted">abc</unclear> ---> ⟨abc⟩
<add>abc</add> ---> ⟨⟨abc⟩⟩
<surplus>abc</surplus> ---> {abc}
abc ---> {{abc}}
The logic of the last two choices is that scribal invertention is marked doubly, while the corresponding editorial intervention is marked singly.
What brackets will we place around <gap>?

Axelle: let me know if I can help figure out what the encoder(s) intended by <reg><orig>u</orig><corr>i</corr></reg> but of course it's also fine if it stays between you, them and their PIs. My guess is they meant <choice><orig>u</orig><reg>i</reg></choice>.
Manu, indeed, I mixed up question marks and exclamation marks in my example; thanks for the correction.
Arlo: for symbol, I would propose either ((finalGomutra)) [following Krummrey-Panciera] or finalGomutra  [following Leiden+], but I'm also OK with «finalGomutra». I think using sigmas as markup is confusing.
I endorse your suggestions for displaying scribal alteration.
I very strongly suggest using [] for lacunae, which would be in perfect harmony with using the same for restoration.

So I summarize my proposals after the day's discussion so far:

<g type="symbol" subtype="abc"/> ---> abc

<unclear>abc</unclear> ---> (abc)

<supplied reason="lost">abc</supplied> ---> [abc]

<gap reason="lost"/> ---> [...]
(we still need to discuss what comes inside [...] depending on the case)

<supplied reason="omitted">abc</supplied> ---> ⟨abc⟩

<add>abc</add> ---> ⟨⟨abc⟩⟩

<surplus>abc</surplus> ---> {abc}

abc ---> {{abc}}
The logic of the last pair of choices is that scribal intervention is marked doubly, while the corresponding editorial intervention is marked singly.

<orig>abc<orig/> ---> ¡abc!

<sic>abc<orig/> ---> ¿abc?

<reg>abc<orig/> ---> ⟨abc⟩

<corr>abc<orig/> ---> ⟨abc⟩

<choice><sic>X</sic><corr>Y</corr></choice> ---> ¿X?⟨Y⟩

<choice><orig>X</orig><reg>Y</reg></choice> ---> ¡X!⟨Y⟩
The idea behind ¿X?⟨Y⟩ and ¡X!⟨Y⟩ is that the brackets for <corr> and <reg> should be the same as those we would use for <gap reason="omitted">.

@manufrancis : could you add your points about use of colors?

I see now that it was @danbalogh who proposed:
<orig> magenta
<sic> red
<reg> blue
<corr> green
Maybe @danbalogh or @manufrancis can propose a new (final?) summary that integrates the colors?

Would it not be better to revert to using the cheatsheet Gdoc for instead of creating ever new, longer and longer summaries? We can still do the discussion here if comments in the gdoc don't do the job well enough.
Nonetheless, here is Arlo's summary with colours.

<g type="symbol" subtype="abc"/> ---> abc (plus colour? maroon? teal?)
DB: are you sure italics are sufficient? what if someone copies and pastes from our display and loses the formatting? I urge adding something around the symbol subtype

<unclear>abc</unclear> ---> (abc) (how about adding colour? grey?)

<supplied reason="lost">abc</supplied> ---> [abc] (how about adding colour? grey too?)

<gap reason="lost"/> ---> [...] (add colour grey?)
we still need to discuss what comes inside [...] depending on the case

<supplied reason="omitted">abc</supplied> ---> ⟨abc⟩ (add colour green?)

<add>abc</add> ---> ⟨⟨abc⟩⟩ (no colour)

<surplus>abc</surplus> ---> {abc} (add colour red?)

abc ---> {{abc}} (no colour)
The logic of the last pair of choices is that scribal intervention is marked doubly, while the corresponding editorial intervention is marked singly.

<orig>abc<orig/> ---> ¡abc! (colour magenta)

<sic>abc<orig/> ---> ¿abc? (colour red)

<reg>abc<orig/> ---> ⟨abc⟩ (colour blue)

<corr>abc<orig/> ---> ⟨abc⟩ (colour green)

<choice><sic>X</sic><corr>Y</corr></choice> ---> ¿X?⟨Y⟩ (colours as above)

<choice><orig>X</orig><reg>Y</reg></choice> ---> ¡X!⟨Y⟩ (colours as above)
The idea behind ¿X?⟨Y⟩ and ¡X!⟨Y⟩ is that the brackets for <corr> and <reg> should be the same as those we would use for <gap reason="omitted">.

I close this issue, will update the cheatsheet googledoc with what is now stable in our transformation and will create further github issues with the points still to be discussed (which will, once settled, be added in cheatsheet googledoc).


Reproducing Global_Simple

It indeed a very clear and concise code that I was able to use it in Collaboratory very easily.
I am running the DenseNet161_emotiW.py to reproduce Global_Simple - finetune densnet161 on EmotiW 2018 .
I get better accuracy than the reported one in your paper - above the state of the art.
Paper's Table 1 : Val Acc  = 69.50%,  While running the code it outputs Best val Acc: 0.9359
Epoch 15/15
----------
0 Loss: 0.3900 Acc: 0.8424
1 Loss: 0.1979 Acc: 0.9359

Training complete in  482m 9.909503s

Are you aware of any bug that makes the result look too good?
What is your log output for DenseNet161_emotiW.py?

Hi there @limorki!
It seems like you trained the FullDataset_DenseNet161_EmotiW.py inside the Models_FullTrained folder.  The results mentioned in the paper are of models that are trained only on the Train Dataset, i.e. DenseNet161_emotiW.py inside the Models_TrainDataset folder.
However, I did retrain the FullDataset_DenseNet161_EmotiW.py model, just to check.  Turns out that you are getting a much better accuracy than ours.  The following is the log output:
Epoch 15/15
----------
0 Loss: 0.3801 Acc: 0.8486
1 Loss: 0.5872 Acc: 0.7730

Training complete in  301m 53.236140s
Best val Acc: 0.7760

I am not sure of any bug that makes the results look that good.
Could you mention the environment in which you are training the model? I am using 4 Nvidia Tesla K80 in parallel for training.

Hi @dakshitagrawal97
Thank you for putting the effort in running FullDataset_DenseNet161_EmotiW on your configuration, I really appreciate it.
I training on 1 gpu using Google's Collaboratory, this gist to reproduce the results:

To run it you only need to modify project_root_dir = ''
Can you please help me understand why you have used DenseNet161_emotiW.py and not the full dataset in your paper.
Thank you

Hi @limorki
The train dataset and the validation dataset were defined by the emotiW dataset, provided to us by the organizers.  The DenseNet161_emotiW.py was trained according to that split.
As a trick to increase the accuracy and for submitting to the challenge, we thought of training the model on the whole dataset as well.  But all the initial tests were done on the original split given in the emotiW dataset, hence we report those in the paper.


[BUG] Unexpected error when showing Pending events

Describe the bug
I get an error when trying to view Pending events.
To Reproduce
Steps to reproduce the behavior:

Go to User Events in Events Manager. Published events are checked by default
Click on Pending to also show events pending.
Uncheck Published and the screen updates to show "Unexpected error, please try again."

Expected behavior
The pending events are shown
Screenshots

Desktop (please complete the following information):

OS: Mac OS 11.5.1
Browser Chrome and Vivaldi

@isaac-galvan, this should be fixed now. Please check the latest DEV instance (https://api-dev.rokwire.illinois.edu/manager-events/). We also have included Groups authorization feature and a few other improvements and bug fixes there.

Closing this as it's fixed.


chore: don't remove optional dependencies in clean-shrinkwrap.js

If we include a platform specific optional dependency in the shrinkwrap
file, then npm will insist in installing it even if the platform doesn't
match. As a solution, we figured out we can avoid putting this platform
specific optional dependencies in the npm-shrinkwrap.json file.
In order to do this, we currently have a script called
clean-shrinkwrap.js that runs before any npm shrinkwrap file (its
a preshrinkwrap npm script) that deletes all the platform specific
modules we know about using npm rm.
The problem with this approach is that npm rm will remove the module's
code from node_modules, which means that if we run npm shrinkwrap,
we will lose certain optional dependencies, that may be needed at a
later stage.
The solution is to modify the clean-shrinkwrap.js script to parse
npm-shrinkwrap.json, and manually delete the entries that we want to
omit. Also, the script needs to be run after npm shrinkwrap, so we
change the npm script name to postshrinkwrap.
Signed-off-by: Juan Cruz Viotti<EMAIL_ADDRESS>

Let's see what the CI servers say. Fingers-crossed!

YESS, finally everything works \o/

What do you think @jhermsmeier ?

There's also a build failure on Windows:
[...]
mkdir dist/Etcher-cli-1.0.0+2a000ae-win32-x86-app
cp package.json dist/Etcher-cli-1.0.0+2a000ae-win32-x86-app
cp -RLf lib dist/Etcher-cli-1.0.0+2a000ae-win32-x86-app
cp -RLf dist/node-win32-x86-dependencies/* dist/Etcher-cli-1.0.0+2a000ae-win32-x86-app
cp: cannot copy cyclic symbolic link `dist/node-win32-x86-dependencies/node_modules/ajv/node_modules'
make: *** [dist/Etcher-cli-1.0.0+2a000ae-win32-x86-app] Error 1
Command exited with code 2

Is this a transient error, that only occasionally occurs?

cp: cannot copy cyclic symbolic link
Is this a transient error, that only occasionally occurs?

Yeah, it keeps popping up now and then, but only on Appveyor, and we've no idea why :-(  When you restart the build (which I've just done), it usually passes on the second attempt.
(and annoyingly, unlike on Travis CI where you can restart just one arch-specific build, on Appveyor you have to restart all arch builds)

Yeah, it keeps popping up now and then, but only on Appveyor, and we've no idea why :-( When you restart the build (which I've just done), it usually passes on the second attempt.
(and annoyingly, unlike on Travis CI where you can restart just one arch-specific build, on Appveyor you have to restart all arch builds)

It looks like a MinGW bug. Moving stuff using Node.js file-system works fine. We only use this for the CLI generation, which we will move to pkg, so it should be gone soon.

Fixed!

> node ./scripts/clean-shrinkwrap.js
Removing: 7zip-bin-mac, 7zip-bin-win, 7zip-bin-linux
Done
There are unstaged npm-shrinkwrap.json changes. Please commit the result of:
    npm shrinkwrap --dev
diff --git a/npm-shrinkwrap.json b/npm-shrinkwrap.json
index 2c2240b..44d9863 100644
--- a/npm-shrinkwrap.json
+++ b/npm-shrinkwrap.json
@@ -53,6 +53,13 @@
       "resolved": "https://registry.npmjs.org/7zip-bin/-/7zip-bin-2.1.0.tgz",
       "dev": true
     },
+    "7zip-bin-win": {
+      "version": "2.1.0",
+      "from"<EMAIL_ADDRESS><3.0.0",
+      "resolved": "https://registry.npmjs.org/7zip-bin-win/-/7zip-bin-win-2.1.0.tgz",
+      "dev": true,
+      "optional": true
+    },
     "abbrev": {
       "version": "1.1.0",
       "from"<EMAIL_ADDRESS><2.0.0",
warning: LF will be replaced by CRLF in npm-shrinkwrap.json.
The file will have its original line endings in your working directory.
make: *** [sanity-checks] Error 1
Command exited with code 2

> node ./scripts/clean-shrinkwrap.js
Removing: 7zip-bin-mac, 7zip-bin-win, 7zip-bin-linux
Done
There are unstaged npm-shrinkwrap.json changes. Please commit the result of:
    npm shrinkwrap --dev
diff --git a/npm-shrinkwrap.json b/npm-shrinkwrap.json
index 2c2240b..dc1282c 100644
--- a/npm-shrinkwrap.json
+++ b/npm-shrinkwrap.json
@@ -53,6 +53,13 @@
       "resolved": "https://registry.npmjs.org/7zip-bin/-/7zip-bin-2.1.0.tgz",
       "dev": true
     },
+    "7zip-bin-linux": {
+      "version": "1.1.0",
+      "from"<EMAIL_ADDRESS><2.0.0",
+      "resolved": "https://registry.npmjs.org/7zip-bin-linux/-/7zip-bin-linux-1.1.0.tgz",
+      "dev": true,
+      "optional": true
+    },
     "abbrev": {
       "version": "1.1.0",
       "from"<EMAIL_ADDRESS><2.0.0",
make: *** [sanity-checks] Error 1
The command "./scripts/ci/test.sh -o $HOST_OS -r $TARGET_ARCH" exited with 2.

:cry:

Looks like I made a couple of subtle issues while addressing the review comments. It should work now.

Let's merge this to check if the logs are reduced enough to make Travis CI builds complete.


Update Django-Flags to latest release, use the Flags Debug Toolbar panels, and update Wagtail-Flags

This PR does three things:

Updates Django-Flags to the latest release, which includes debug, logging, and state-checking related helpers.
Adds the new Django-Flags Debug Toolbar panels to Django Debug Toolbar (and, by necessity, sets the default panels explicitly)
Updates Wagtail-Flags to the latest beta release to include the admin overhaul. See below about testing the beta.

Testing

Check out this branch (bump-flags)
Rebuild your python Docker container to get the updated libraries (docker-compose build python2)
Run Docker (docker-compose up)

Once you've got the libraries updated, you can test the Wagtail-Flags beta release's admin overhaul and decide whether you prefer to keep the big-list-of-all-flags-and-conditions that we have no (with added features) or whether you prefer a drill-down approach into flags to see conditions. See https://github.com/cfpb/wagtail-flags/pull/35 for discussion and context.
To try the new drill-down conditions:

Visit http://localhost:8000/admin/flags/
Browse flags, clicking on the flag names to see conditions

To try one-big-list:

Visit http://localhost:8000/admin/flags/
Click "Add flag" in the top right
Add the flag named WAGTAILFLAGS_ADMIN_BIG_LIST and click "Save flag"
Visit the WAGTAILFLAGS_ADMIN_BIG_LIST flag and click "
Enable WAGTAILFLAGS_ADMIN_BIG_LIST for all requests"
View the great big list of all flags and conditions

This PR will be on hold pending feedback on whether the drill-down of flags or the great big list of flags and conditions is preferred. Once that decision is made, we'll strip the feature flag out of Wagtail-Flags, release 4.1.0 proper, and update this PR and merge it.
Checklist

[x] PR has an informative and human-readable title
[x] Changes are limited to a single goal (no scope creep)
[x] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the CFPB development guidelines
[x] Passes all existing automated tests
[ ] Any change in functionality is tested
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged / future todos are captured in comments
[ ] Visually tested in supported browsers and devices (see checklist below :point_down:)
[ ] Project documentation has been updated
[x] Reviewers requested with the Reviewers tool :arrow_right:

Hmm... we could let this be a permanent flag that users can self-select by adding a user condition :)

Honestly... no. I don't want to have to support two different admin views for Wagtail-Flags. We need to settle on one.

Works for me. I guess I slightly prefer the more compact (drill-down) version at this time.


Web component properties do not seem to accept objects or arrays

Example:
  static properties = {
    value1: {type: Object},
    value2: {type: Array},
  };

I'm not sure if I'm doing something wrong here, but when I try to pass in an Array or Object, I get undefined.

I think I made a bug here: https://github.com/google/mesop/blob/1fb6c0710a6f0db62ee8197f72b629a3b0e82905/mesop/web/src/component_renderer/component_renderer.ts#L213C23-L213C35
Will try to fix this soon


Consider traffic information in route computation

From @TimMcCauley on December 13, 2015 10:18
The new ORS backend allows specifying whether traffic information is considered in route computation or not. This can be done by providing useRealTimeTraffic attribute for xls:RoutePlan element.

<xls:XLS xmlns:xls="http://www.opengis.net/xls" xsi:schemaLocation="http://www.opengis.net/xls http://schemas.opengis.net/ols/1.1.0/RouteService.xsd" xmlns:sch="http://www.ascc.net/xml/schematron" xmlns:gml="http://www.opengis.net/gml" xmlns:xlink="http://www.w3.org/1999/xlink" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" version="1.1" xls:lang="en">
xls:RequestHeader
/xls:RequestHeader
<xls:Request methodName="RouteRequest" version="1.1" requestID="00" maximumResponses="15">
xls:DetermineRouteRequest
<xls:RoutePlan  useRealTimeTraffic="true">
xls:RoutePreferenceCar/xls:RoutePreference
xls:ExtendedRoutePreference
xls:WeightingMethodFastest/xls:WeightingMethod
/xls:ExtendedRoutePreference
xls:WayPointList
xls:StartPoint
xls:Position
<gml:Point xmlns:gml="http://www.opengis.net/gml">
<gml:pos srsName="EPSG:4326">8.20440472118994 49.36902403998967/gml:pos
/gml:Point
/xls:Position
/xls:StartPoint
xls:EndPoint
xls:Position
<gml:Point xmlns:gml="http://www.opengis.net/gml">
<gml:pos srsName="EPSG:4326">8.18045795910493 49.32350720550303/gml:pos
/gml:Point
/xls:Position
/xls:EndPoint
/xls:WayPointList
<xls:AvoidList />
/xls:RoutePlan
<xls:RouteInstructionsRequest provideGeometry="true" />
xls:RouteGeometryRequest
/xls:RouteGeometryRequest
/xls:DetermineRouteRequest
/xls:Request
/xls:XLS
This new feature useRealTraffic requires an additional UI element (see figure below), which will allow user to enable or disable this feature. Note, this feature must be visible only for car related profiles. Furthermore, as it can be seen from the figure, the ui is becoming a bit messy. Therefore, we need to think about rearranging/grouping the following elements: maximum speed, consider traffic (Germany only), avoid features.
Copied from original issue: GIScience/openrouteservice#234

Some traffic information for the city of Turin (Italy) are available as opendata from http://opendata.5t.torino.it/get_fdt
The same agency (5t) also provides information about parking areas and controlled traffic zones if you are interested to add some features that need these data for the route planning


stopped working

anyone else?

I think it's because it does not work for https. Only for http. But since most links to Wikipedia are https, it does not work anymore. I tried to copy the wikipedia link and explicitly typed http, then it worked. The plugin did its thing and inserted the .m for mobile, then wikipedia itself redirected to https afterwards and I got the mobile view.
I don't know if chrome plugins are allowed to alter https URLs...?

this works with https https://chrome.google.com/webstore/detail/m-wiki/ibnmikddaopgfbbngcgcfmanjfgbcopf


Unbale to access fieldGroup from fieldConfig.fieldArray after upgrading to formly v6

Solution using the array component itself. only downfall doing this is it is specifically programmed to an array of objects only.
https://stackblitz.com/edit/angular-ivy-vxvawr?file=src%2Fapp%2Fcustom-table%2Fcustom-table.type.ts
Originally posted by @kenisteward in https://github.com/ngx-formly/ngx-formly/issues/2988#issuecomment-926040055
After upgrading to formly v6, it fails at <th *ngFor="let tHead of fieldConfig.fieldArray.fieldGroup">
Unable to access the fieldGroup anymore. How to get the fieldGroup from fieldConfig.fieldArray with the new implementation?

It seems the 'fieldArray' property has changed to a function when I inspected the 'field' object.
Also this is the new type definition:
    fieldArray?: FormlyFieldConfig | ((field: FormlyFieldConfig) => FormlyFieldConfig);

When further examining I made it work by doing the following:
 fieldArray?: FormlyFieldConfig;

  ngOnInit() {
    if (typeof this.field.fieldArray === 'function') {
      this.fieldArray = this.field.fieldArray(this.field);
    } else {
      this.fieldArray = this.field.fieldArray;
    }
  }

And in the template replacing it by the following:
<th *ngFor="let headerField of fieldArray.fieldGroup">


Incorrect model loading

Hello,
I think you have 3 issues in mode loading:-

If someone loaded a model from a specific checkpoint, the model will be reloaded from best-model. It will simply override it.

The loading of best-model is written after the "train" function. This simply means it will never execute.

loading best-model assumes the checkpoint will exist in all nodes not only the first node.
I would recommend to take a look at:
https://pytorch.org/tutorials/intermediate/ddp_tutorial.html
How to map it from GPU0 at master node to all other nodes.

Thanks.

1&2.  Checkpointing in master branch is left over from original transformer. It's not used in the examples, and it actually doesn't even work because of interaction with FP16 optimizer . I've redone checkpointing in this branch -- https://github.com/cybertronai/transformer-xl/tree/checkpoint

The newer version only does checkpoint saving/loading in the 0 worker, and then broadcasts parameters to the rest of the workers

Great, waiting for the merge :)


Envoy returns status 200 while it's configured to return 503 immediately

Hi community,
I have a server that does nothing but just returns custom responses. I noticed in the below example, while I configure envoy to return 503, the status code in the response is 200 and the grpc-status code is 14.
The client then got confused when they look at the metric envoy_cluster_upstream_rq, because all response codes were 200.
Question:

is the server behavior expected? is there a way to ask it to return 503 in the response so that client can be aware?
on the client side, is there any other way to know the actual response code is 503 or 14? Do they have to use grpc_stats filter?
Could anyone please kindly give me some hints?

server configuration:
resources:
  # inbound listener
  - "@type": type.googleapis.com/envoy.config.listener.v3.Listener
    name: listener_inbound
    address:
      socket_address:
        address: <IP_ADDRESS>
        port_value: 16000
    traffic_direction: INBOUND
    filter_chains:
      - filters:
          - name: envoy.filters.network.http_connection_manager
            typed_config:
              "@type": type.googleapis.com/envoy.extensions.filters.network.http_connection_manager.v3.HttpConnectionManager
              codec_type: AUTO
              stat_prefix: ingress_http
              route_config:
                name: ingress
                virtual_hosts:
                  - name: ingress
                    domains:
                      - "*"
                    routes:
                      - match:
                          prefix: "/"
                        direct_response:
                          status: 503
                          body:
                            inline_string: "{'message': 'FAILURE INJECTOIN!'}"
              http_filters:
                - name: envoy.filters.http.router

When I enable the debug mode on the server, the response is
':status', '200'
'content-type', 'application/grpc'
'grpc-status', '14'
'grpc-message', '{'message': 'FAILURE INJECTOIN!'}'
'date', 'Sat, 30 Oct 2021 03:12:53 GMT'
'server', 'envoy'

client configuration:
  - "@type": type.googleapis.com/envoy.config.cluster.v3.Cluster
    name: egress_service
    type: STRICT_DNS
    typed_extension_protocol_options:
      envoy.extensions.upstreams.http.v3.HttpProtocolOptions:
        "@type": type.googleapis.com/envoy.extensions.upstreams.http.v3.HttpProtocolOptions
        explicit_http_config:
          http2_protocol_options: {}
    lb_policy: ROUND_ROBIN
    connect_timeout: 100s
    load_assignment:
      cluster_name: egress_service
      endpoints:
        - lb_endpoints:
            - load_balancing_weight: 100
              endpoint:
                address:
                  socket_address:
                    address: egress_service
                    port_value: 80

is there a grpc related http filter missing in your config?

@lambdai I can't think of anything. The grpc-status header in the response is actually correct. The status is wrong. I expect the direct response on the server be
':status', '503'
'content-type', 'application/grpc'
'grpc-status', '14'
'grpc-message', '{'message': 'FAILURE INJECTOIN!'}'
'date', 'Sat, 30 Oct 2021 03:12:53 GMT'
'server', 'envoy'

@lambdai
I tried to use the fault filter and the response looks the same.
I think this is the reason https://github.com/envoyproxy/envoy/blob/main/source/extensions/filters/http/fault/fault_filter.cc#L318
Only one of the statuses would be set. Is this the same for the direct_response?
If so, does that mean users have to use grpc_stats filter to check real http2 status codes? I was under the impression that envoy can translate grpc status code to HTTP status code.

@lambdai I can't think of anything. The grpc-status header in the response is actually correct. The status is wrong. I expect the direct response on the server be
':status', '503'
'content-type', 'application/grpc'
'grpc-status', '14'
'grpc-message', '{'message': 'FAILURE INJECTOIN!'}'
'date', 'Sat, 30 Oct 2021 03:12:53 GMT'
'server', 'envoy'

This is my confusing part: the server config only has router as http filter in http_connection_manager.  In the server log you shouldn't see grpc-status or grpc-message.


track n trace user/password

Hi.
In your documentation, you talk about having a track n tracker username&password.
Where would I get that? (exact link hopefully).
I have access to "Entwickler Portal/Developer Portal"
I tried using following values:
`
$cig_user = 'my_developer_user_id';
$cig_password = 'my_developer_password';
$tnt_user = 'dhl_entwicklerportal';
$tnt_password = 'Dhl_123';
$credentials = new DreiCredentials($cig_user, $cig_password,
DreiCredentials::ENDPOINT_SANDBOX, $tnt_user, $tnt_password);
$api = new ShipmentTracking($credentials);
$result = $api->getPiece('00340434161094042557', RequestBuilder::LANG_EN);
echo "just a test";
`
But as a result value I get
<?xml version="1.0" encoding="UTF-8"?> <data name="piece-shipment-list" request-id="e0728416-56c8-4f6a-b891-c1d6ee904728" code="5" error="Log-in failed">     <data name="piece-shipment" searched-piece-code="00340434161094042557" piece-code="00340434161094042557" piece-id="" recipient-name="" recipient-street="" recipient-city="" status="" short-status="" delivery-event-flag="" division="" recipient-id="" product-code="" product-key="" product-name="" international-flag="0" piece-status="5" piece-status-desc="Log-in failed"/> </data>

some more info: https://github.com/tabsl/dhl-shipment-tracking/blob/master/track.php#L6

Closing as this is in the current state of the project


No way to see RAISE notices in Postgres

Is your feature request related to a problem? Please describe.
In Postgres, you can raise notices, such as:
raise notice '%',now();

Currently there is no way to view this output like there is using the Microsoft Postgres plugin.
Describe the solution you'd like
A way to view notices.

You wanted to see it like this?

We also show the message...

Nvm, I see what you are looking for.
Feature Requirements:

list all notices and raises when running queries.

More info: https://node-postgres.com/api/client


0 vulnerabilities reported for iGoat-Swift

Describe the bug
No vulnerabilities were found when scanning the iGoat-Swift codebase
To Reproduce
Steps to reproduce the behavior:

mkdir vulnerable-ios
cd vulnerable-ios
git clone https://github.com/OWASP/iGoat-Swift/tree/master/iGoat-Swift/iGoat-Swift
cd ../scanner
./insider -tech ios -target ../vulnerable-ios/iGoat-Swift

Expected behavior
I was expecting to see at least a few vulnerabilities reported.
Screenshots
This is what the tool reported:
[INSIDER]: 2020/10/21 17:41:00 Starting analysis for iOS app #
[INSIDER]: 2020/10/21 17:41:00 Starting iOS Source code analysis
[INSIDER]: 2020/10/21 17:41:00 Extracting libraries
[INSIDER]: 2020/10/21 17:41:00 Removing: [ ] from file list from ios tech
[INSIDER]: 2020/10/21 17:41:00 Removing: [pods] from file list from ios tech
[INSIDER]: 2020/10/21 17:41:00 Found 0 files to analyze.
[INSIDER]: 2020/10/21 17:41:00 loading core rules
[INSIDER]: 2020/10/21 17:41:00 Rules 8
[INSIDER]: 2020/10/21 17:41:00 loading IOS rules
[INSIDER]: 2020/10/21 17:41:00 Rules 25
[INSIDER]: 2020/10/21 17:41:00 Rules 25
[INSIDER]: 2020/10/21 17:41:00 Starting extracting hardcoded information
[INSIDER]: 2020/10/21 17:41:28 Finished hardcoded information extraction
[INSIDER]: 2020/10/21 17:41:28 Scanned 0 lines
[INSIDER]: 2020/10/21 17:41:28 Finished code analysis
[INSIDER]: 2020/10/21 17:41:28 Json Report /Users/runner/work/1/s/report-20201021174128.json
[INSIDER]: 2020/10/21 17:41:28 Json Report 31.6 kB bytes written successfully
[INSIDER]: 2020/10/21 17:41:28 Html Report /Users/runner/work/1/s/report-20201021174128.html
[INSIDER]: 2020/10/21 17:41:28 Html Report 82.1 kB bytes written successfully
[INSIDER]: 2020/10/21 17:41:29 Found 0 warnings
[INSIDER]: 2020/10/21 17:41:29 -----------------------------------------------
[INSIDER]: 2020/10/21 17:41:29 Score Security 100/100
[INSIDER]: 2020/10/21 17:41:29 Vulnerability	Number
[INSIDER]: 2020/10/21 17:41:29 High		  0
[INSIDER]: 2020/10/21 17:41:29 Medium		  0
[INSIDER]: 2020/10/21 17:41:29 Low		  0
[INSIDER]: 2020/10/21 17:41:29 Total		  0
[INSIDER]: 2020/10/21 17:41:29 -----------------------------------------------------------------------------------------------------------------------
[INSIDER]: 2020/10/21 17:41:29 You are using the Insider open source version. If you like the product and want more features,
[INSIDER]: 2020/10/21 17:41:29 visit http://insidersec.io and get to know our enterprise version.
[INSIDER]: 2020/10/21 17:41:29 If you are a developer, then you can contribute to the improvement of the software while using an open source version
[INSIDER]: 2020/10/21 17:41:29 -----------------------------------------------------------------------------------------------------------------------
[INSIDER]: 2020/10/21 17:41:29 Finished analysis for iOS app #

Desktop (please complete the following information):

OS: Windows and macOS
Version 10 / 10-15

Additional context
Add any other context about the problem here.

Hello @robwillup, thank you very much for reporting, we are currently working on improvements for the ruleset and the way insider analyzes, meanwhile, we have the enterprise version with 2 free analysis if want to check it out: https://console.insidersec.io/en/register.

Hi @robwillup , sorry for delay, but what version are you running of insider? I tested here with latest 2.0.5, and found some vulnerabilities:
[INSIDER]: 2020/12/15 11:39:29 Starting analysis for iOS app #
[INSIDER]: 2020/12/15 11:39:29 Starting iOS Source code analysis
[INSIDER]: 2020/12/15 11:39:29 Extracting libraries
[INSIDER]: 2020/12/15 11:39:30 Removing: [] from file list from ios tech
[INSIDER]: 2020/12/15 11:39:30 Removing: [pods] from file list from ios tech
[INSIDER]: 2020/12/15 11:39:30 Found 278 files to analyze.
[INSIDER]: 2020/12/15 11:39:30 loading core rules
[INSIDER]: 2020/12/15 11:39:30 Rules 8
[INSIDER]: 2020/12/15 11:39:30 loading IOS rules
[INSIDER]: 2020/12/15 11:39:30 Rules 25
[INSIDER]: 2020/12/15 11:39:30 Rules 25
[INSIDER]: 2020/12/15 11:39:30 Starting extracting hardcoded information
[INSIDER]: 2020/12/15 11:39:59 Finished hardcoded information extraction
[INSIDER]: 2020/12/15 11:44:56 Scanned 81069 lines
[INSIDER]: 2020/12/15 11:44:56 Finished code analysis
[INSIDER]: 2020/12/15 11:44:56 Json Report /home/matheus/dev/github/insidersec/insider/report-20201215114456.json
[INSIDER]: 2020/12/15 11:44:56 Json Report 428.4 kB bytes written successfully
[INSIDER]: 2020/12/15 11:44:56 Html Report /home/matheus/dev/github/insidersec/insider/report-20201215114456.html
[INSIDER]: 2020/12/15 11:44:56 Html Report 805.5 kB bytes written successfully
[INSIDER]: 2020/12/15 11:44:57 Found 682 warnings
[INSIDER]: 2020/12/15 11:44:57 -----------------------------------------------
[INSIDER]: 2020/12/15 11:44:57 Score Security 12/100
[INSIDER]: 2020/12/15 11:44:57 Vulnerability    Number
[INSIDER]: 2020/12/15 11:44:57 High              10
[INSIDER]: 2020/12/15 11:44:57 Medium           660
[INSIDER]: 2020/12/15 11:44:57 Low               12
[INSIDER]: 2020/12/15 11:44:57 Total            682
[INSIDER]: 2020/12/15 11:44:57 -----------------------------------------------------------------------------------------------------------------------
[INSIDER]: 2020/12/15 11:44:57 You are using the Insider open source version. If you like the product and want more features,
[INSIDER]: 2020/12/15 11:44:57 visit http://insidersec.io and get to know our enterprise version.
[INSIDER]: 2020/12/15 11:44:57 If you are a developer, then you can contribute to the improvement of the software while using an open source version
[INSIDER]: 2020/12/15 11:44:57 -----------------------------------------------------------------------------------------------------------------------
[INSIDER]: 2020/12/15 11:44:57 Finished analysis for iOS app #

Hi @msAlcantara,
Thank you for the reply!
When I opened the issue I think I was running version 2.0.4.
However, today I downloaded the 2.0.5 version and got an error when trying to run it against iGoat-Swift on Windows 10. I'll add the error below. I also tried to run it in a macOS agent, there was no error, but I noticed that it found 0 files to analyze. I'm curious as to why it is not finding the .swift files.
Windows 10:
$ ./insider.exe -tech ios -target iGoat-Swift/ -no-banner -no-html -no-json -v
[INSIDER]: 2020/12/15 13:41:24 Starting analysis for iOS app #
[INSIDER]: 2020/12/15 13:41:24 Starting iOS Source code analysis
[INSIDER]: 2020/12/15 13:41:24 Extracting libraries
[INSIDER]: 2020/12/15 13:41:25 Removing: [ ] from file list from ios tech
[INSIDER]: 2020/12/15 13:41:25 Removing: [pods] from file list from ios tech
[INSIDER]: 2020/12/15 13:41:25 Found 0 files to analyze.
[INSIDER]: 2020/12/15 13:41:25 loading core rules
[INSIDER]: 2020/12/15 13:41:25 Rules 8
[INSIDER]: 2020/12/15 13:41:25 loading IOS rules
[INSIDER]: 2020/12/15 13:41:25 Rules 25
[INSIDER]: 2020/12/15 13:41:25 Rules 25
[INSIDER]: 2020/12/15 13:41:25 Starting extracting hardcoded information
panic: runtime error: index out of range [1] with length 1

goroutine 1 [running]:
insider/visitor.NewInputFile(0xc0000a00e0, 0xc, 0xc00038e340, 0x34, 0xc00010e000, 0x178, 0x378, 0x0, 0x0, 0x0, ...)
        /home/runner/work/insider/insider/visitor/file.go:237 +0x708
insider/visitor.FindFiles.func1(0xc00038e340, 0x34, 0xba8600, 0xc000499420, 0x0, 0x0, 0x34, 0x0)
        /home/runner/work/insider/insider/visitor/reader.go:146 +0x332
path/filepath.walk(0xc00038e340, 0x34, 0xba8600, 0xc000499420, 0xc000440fa8, 0x0, 0x0)
        /opt/hostedtoolcache/go/1.14.4/x64/src/path/filepath/path.go:360 +0x42c
path/filepath.walk(0xc00049ac40, 0x17, 0xba8600, 0xc0004991f0, 0xc000440fa8, 0x0, 0x0)
        /opt/hostedtoolcache/go/1.14.4/x64/src/path/filepath/path.go:384 +0x306
path/filepath.walk(0xc0000a00e0, 0xc, 0xba8600, 0xc00026df80, 0xc000440fa8, 0x0, 0x0)
        /opt/hostedtoolcache/go/1.14.4/x64/src/path/filepath/path.go:384 +0x306
path/filepath.Walk(0xc0000a00e0, 0xc, 0xc000440fa8, 0xefe4c0, 0x1)
        /opt/hostedtoolcache/go/1.14.4/x64/src/path/filepath/path.go:406 +0x106
insider/visitor.FindFiles(0xc0000a00e0, 0xc, 0x4cd200, 0x9ff0a8, 0x0, 0x49000cc500, 0xc0004410b0, 0x4d0ef4, 0xc0000cc500)
        /home/runner/work/insider/insider/visitor/reader.go:116 +0xc2
insider/lib.ExtractHardcodedInfo(0xc0000a00e0, 0xc, 0x0, 0x0, 0xba7b40, 0xc000282780, 0xc, 0x0)
        /home/runner/work/insider/insider/lib/analyzer.go:305 +0x68
insider/lib.AnalyzeIOSSource(0xc0000a00e0, 0xc, 0x0, 0x0, 0xc000282780, 0x9af06a, 0x2, 0x0, 0x0)
        /home/runner/work/insider/insider/lib/ios.go:197 +0x263
insider/supervisors.RunIOSCodeAnalysis(0x9af482, 0x3, 0xc0000a00a8, 0x3, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, ...)
        /home/runner/work/insider/insider/supervisors/ios.go:39 +0x83f
main.main()
        /home/runner/work/insider/insider/main.go:109 +0xb5c

macOS agent:
[INSIDER]: 2020/12/15 14:22:33 Starting analysis for iOS app #
[INSIDER]: 2020/12/15 14:22:33 Starting iOS Source code analysis
[INSIDER]: 2020/12/15 14:22:33 Extracting libraries
[INSIDER]: 2020/12/15 14:22:33 Removing: [ ] from file list from ios tech
[INSIDER]: 2020/12/15 14:22:33 Removing: [pods] from file list from ios tech
[INSIDER]: 2020/12/15 14:22:33 Found 0 files to analyze.
[INSIDER]: 2020/12/15 14:22:33 loading core rules
[INSIDER]: 2020/12/15 14:22:33 Rules 8
[INSIDER]: 2020/12/15 14:22:33 loading IOS rules
[INSIDER]: 2020/12/15 14:22:33 Rules 25
[INSIDER]: 2020/12/15 14:22:33 Rules 25
[INSIDER]: 2020/12/15 14:22:33 Starting extracting hardcoded information
[INSIDER]: 2020/12/15 14:22:33 Finished hardcoded information extraction
[INSIDER]: 2020/12/15 14:22:33 Scanned 0 lines
[INSIDER]: 2020/12/15 14:22:33 Finished code analysis
[INSIDER]: 2020/12/15 14:22:33 ---------------------------------------------------------------------
[INSIDER]: 2020/12/15 14:22:33 Score Security 100

[INSIDER]: 2020/12/15 14:22:33
[INSIDER]: 2020/12/15 14:22:33
[INSIDER]: 2020/12/15 14:22:33 ---------------------------------------------------------------------
[INSIDER]: 2020/12/15 14:22:33 No Json report
[INSIDER]: 2020/12/15 14:22:33 No Html report
[INSIDER]: 2020/12/15 14:22:33 Found 0 warnings
[INSIDER]: 2020/12/15 14:22:33 -----------------------------------------------
[INSIDER]: 2020/12/15 14:22:33 Score Security 100/100
[INSIDER]: 2020/12/15 14:22:33 Vulnerability	Number
[INSIDER]: 2020/12/15 14:22:33 High		  0
[INSIDER]: 2020/12/15 14:22:33 Medium		  0
[INSIDER]: 2020/12/15 14:22:33 Low		  0
[INSIDER]: 2020/12/15 14:22:33 Total		  0
[INSIDER]: 2020/12/15 14:22:33 -----------------------------------------------------------------------------------------------------------------------
[INSIDER]: 2020/12/15 14:22:33 You are using the Insider open source version. If you like the product and want more features,
[INSIDER]: 2020/12/15 14:22:33 visit http://insidersec.io and get to know our enterprise version.
[INSIDER]: 2020/12/15 14:22:33 If you are a developer, then you can contribute to the improvement of the software while using an open source version
[INSIDER]: 2020/12/15 14:22:33 -----------------------------------------------------------------------------------------------------------------------
[INSIDER]: 2020/12/15 14:22:33 Finished analysis for iOS app #

Thank you for the help!

Thanks for reply @robwillup , I think that this could be related with issue #44. Can you try to build from master and test please?  We are making some tests to generate next release with theses fixes.

Thanks @msAlcantara !
That did it! Running the version built from master worked really well! 👍🏼
Would you have an estimate of when this new version will be released?
Here's part of the output:
...
[INSIDER]: 2020/12/15 14:09:47 ---------------------------------------------------------------------
[INSIDER]: 2020/12/15 14:09:47 No Json report
[INSIDER]: 2020/12/15 14:09:47 No Html report
[INSIDER]: 2020/12/15 14:09:47 Found 694 warnings
[INSIDER]: 2020/12/15 14:09:47 -----------------------------------------------
[INSIDER]: 2020/12/15 14:09:47 Score Security 12/100
[INSIDER]: 2020/12/15 14:09:47 Vulnerability    Number
[INSIDER]: 2020/12/15 14:09:47 High              10
[INSIDER]: 2020/12/15 14:09:47 Medium           672
[INSIDER]: 2020/12/15 14:09:47 Low               12
[INSIDER]: 2020/12/15 14:09:47 Total            694
[INSIDER]: 2020/12/15 14:09:47 -----------------------------------------------------------------------------------------------------------------------
...

Thanks!

Great!! @robwillup
I think that with theses tests we are ready to publish. Thank very much for your help and patience.

Closing the issue because version 2.0.6 is released


description is blank in user public feed

i.e.: https://francetelevisions.cartodb.com/u/francetvinfo/me

please @brittanymicek @beloneys suggest a description for these public pages, this is a hotfix

Hi Matallo,
As @brittanymicek suggested for this specific profile:
Francetvinfo is a digital news platform, dedicated to providing news in real time with brief reports on timely topics.
Is that Ok for you?

Also for general profiles:
This is [Username] profile. CartoDB transforms location intelligence into
dynamic renderings enabling discovery of trends and patterns.
On Fri, Dec 11, 2015 at 9:32 AM, Carlos Matallín<EMAIL_ADDRESS>wrote:

please @brittanymicek https://github.com/brittanymicek @beloneys
https://github.com/beloneys suggest a description for these public
pages, this is a hotfix
—
Reply to this email directly or view it on GitHub
https://github.com/CartoDB/cartodb/issues/6245#issuecomment-163949373.

--
Brittany Micek
Content Writer
CartoDB—Map Your World's Data
201 Moore Street, Brooklyn, NY 11206
T: + <%2B34695497365>19174633232
W: www.cartodb.com
Live sessions to learn CartoDB http://cartodb.com/webinars/

we just need the generic description, thanks a lot!


Posibility to rearrange items

The backend currently has partial support for this. All items have a field called placement which is currently not used but may. This placement field could decide the items place in correlation to all others, then implement a UI in the frontend to let the user switch place of content.

Implemented in fb9990a


How do I create catalog for PDF?

RT

Hi, I am not sure what you want to ask. If you want to create a document, simply checkout the documentation.


feature: supports custom directories

{project}/.vscode/cspell.json
{project}/.idea/cspell.json

@guobin211  Can you give examples of the format of such files? Is it the same as other cspell.json formats?

No need, I've already found it

@BlackHole1 I mentioned a pr, please check it
https://github.com/BlackHole1/idea-spell-check/pull/16


Early activation

It is not activated until when any of the commands is calling.
However, it should be activated as soon as opening markdown file.

Add onLanguage:markdown in activationEvents.


Update extractive-summarization.md

Fix typo 'sortyby'

@JonathanGiles : Thanks for your contribution! The author(s) have been notified to review your proposed change.

The typo fix looks good; thanks @JonathanGiles.


句子长度超过256后发生的奇妙bug

# model/crf.py
#  def _viterbi_decode(self, feats, mask):
length_mask = torch.sum(mask, dim = 1).view(batch_size,1).long()

mask == [1, 1, ...,1]  # szie: 1 * 256
torch.sum(mask, dim=1)   # 当mask是ByteTensor时，结果是0

血崩 这个bug拖了我3天

Yes, this is a strange bug at PyTorch. I think I have reported this bug at https://discuss.pytorch.org/, but when I just check I found I didn't. : )
Thanks for your updates, will fix it soon.

Fixed, thanks again.

Have a nice dream


Go 640 Read request body

Add new endpoint - /sqlInjection/body/sqlite3Exec to handle usage of http.request.Body.Read method to get the user input.
Update endpoint - /sqlInjection/headers/json to /sqlInjection/headers-json/sqlite3Exec so it follows to common convention.
Refactored sqlite db setup and handling safety method for SQL Injection. Setting up the database, opening the connection, execution of the specific query (safe or unsafe) happens in one function which is used in different sources for that sink.
Testing
Build and run the go-test-bench app. You should see a new section Body where you can submit a form which uses the Read method of request.Body. You should see the vulnerability reported in Teamserver.

Hey @cixel, I've tested the problem from yesterday and here is the final result:
userInput = strings.TrimPrefix(userInput, "input=")

^ works and the vulnerability is reported to TS.
inputSplit := strings.Split(userInput, "input=")
userInput = inputSplit[1]

^ doesn't work and the vulnerability is not reported to TS.
In all cases we received userInput as input=<something> in the body.

Codecov Report

Merging #15 (6b08937) into main (4ff2879) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #15   +/-   ##
=======================================
  Coverage   95.23%   95.23%
=======================================
  Files           1        1
  Lines          21       21
=======================================
  Hits           20       20
  Misses          1        1

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 4ff2879...6b08937. Read the comment docs.

I think I may be doing something wrong; but, I don't see the vulnerability reported in TS 🤔
I'm wondering if, again, this is just me - or does this also happen for others on the team?


Support for armv7 (Rpi4b 32bits)

Hello,
I tried several times to recreate a container from Dockerfile, but I couldn't.
Do you think it is possible to make an image available that is compatible with Rpi4b, Raspberry Pi OS 32bits in armv7?
Thanks for everything ! \o/
Nikogaug.

32bit systems and support is generally being depreciated these days.
there's absolutely no reason at all to be running a 32bit OS especially on x64 hardware
reinstall with an OS that doesn't artificially limit your hardware and reduce compatibility with most things / that will only increasingly cause headaches

Yes you're right. I have to change my OS. Thanks again.


Suggestion System

Compress everything to a single Custom Command rather than having 4 of them for a suggestion system.

As suggestion v2 is out (and resolves this), will close this issue.


fix: remove clickhouse host and port from dictionary connection

Summary
This PR fixed an issue in which ClickHouse was using the configured host to load the dictionary, however, this would fail if it was running on Docker and would be slower if it was using a URL to connect to ClickHouse.
Before
SOURCE(CLICKHOUSE(USER 'ch_admin' PASSWORD '[HIDDEN]' HOST 'clickhouse' PORT '8123' QUERY ...)
After
SOURCE(CLICKHOUSE(USER 'ch_admin' PASSWORD '[HIDDEN]' QUERY ...)
Checklist
Delete items not relevant to your PR:

[ ] Unit and integration tests covering the common scenarios were added
[ ] A human-readable description of the changes was provided to include in CHANGELOG
[ ] For significant changes, documentation in https://github.com/ClickHouse/clickhouse-docs was updated with further explanations or tutorials

@BentsiLeviav This PR is ready for review

@BentsiLeviav, I was literally about to create an issue asking for very similar change. Only difference is I think it might be nice to support omitting credentials altogether for the table source? i.e.
SOURCE(CLICKHOUSE(TABLE 'table_name'))
Specifying the user in our case fails in our development environment because developers are not able to directly authenticate to ClickHouse from the server itself. As a result we get an auth error.

I think @rjoelnorgren brings up a good point. Would it be possible to update the credentials dict with connection_overrides and, if a given field is empty, omit that from the configuration?


Error during init [TypeError: Cannot read property 'apply' of undefined]

Please consider the following before submitting an issue:
Guidelines for contributing: https://github.com/chartjs/Chart.js/blob/master/CONTRIBUTING.md
Example of issue on an interactive website such as the following:

http://jsbin.com/
http://jsfiddle.net/
http://codepen.io/pen/
Premade template: http://codepen.io/pen?template=JXVYzq

Closing since there is no test case to reproduce. Will reopen if one becomes available


update pong source code links

via IRC user interestedinelm

Cool, thank you! Should be live on the site now.


treewide: remove unused variables

Rerun the bot which did https://github.com/NixOS/nixpkgs/pull/43857
It should have no effect besides the cleanup

Is the source code to the bot available anywhere?

@Profpatsch https://gist.github.com/volth/297b24d577f9868255210c31479fd56b

@volth FYI https://github.com/NixOS/ofborg#running-meta-checks-locally

OK, ofborg is happy.
Please do not hold it for long time, soon there will be merge conflicts

10.rebuild-linux: 1-10

Is that correct? What's being rebuilt?

10.rebuild-linux: 1-10

Is that correct? What's being rebuilt?

Maybe pkgs/applications/science/biology/sumatools/default.nix (there was let meta = ... in which I did not delete but added inherit meta below, apparently forgotten)

Just for information: «Show all checks» → eval — Details
There are sup, sumatra, terraform and some others.

So, another group are derivations with gemdir = ./.;
They will be rebuilt even on changes in comments or whitespacing.

So, another group are derivations with gemdir = ./.;
They will be rebuilt even on changes in comments or whitespacing.

I wonder if that might be a reason to consider deprecating gemdir... we
definitely shouldn't be copying the whole directory to the store.

So, another group are derivations with gemdir = ./.; They will be rebuilt even on changes in comments or whitespacing.

I wonder if that might be a reason to consider deprecating gemdir... we definitely shouldn't be copying the whole directory to the store.

It is not only gemdir, some python modules use src = ./.

It is not only gemdir, some python modules use src = ./.

Perhaps those should go too then. Feels like an antipattern.

@zimbatm introduced that. I think it should be possible to only add the files we actually need to the nix store.


Support Vector Machine

Is your feature request related to a problem? Please describe.
Support Vector Machine Algorithm in Python

Assigned!
@Soumya-Kushwaha : Assigned!


Add fix for webpack history merge bug

Fixes #28810, probably. I can't isolate a repro without webpack and a whole lot of other stuff (lodash, @type/histroy, react, more things that i'm not really sure about), but I've confirmed that this fixes @wanliyunyan's repro.

@weswigham Found the repro for this while investigation of  #29661 where ts-loader changes the order of files as the root files (history node typing before lib file) resulting in this error. The simplified repro is
//@filename: /other.d.ts
export as namespace SomeInterface;
export type Action = 'PUSH' | 'POP' | 'REPLACE';

//@filename: /main.ts
interface SomeInterface {
    readonly length: number;
}
const value: SomeInterface;

@sheetalkamat I've added your test to this PR.


The value serialization results of rmap and rmapcache are inconsistent

Expected behavior
Actual behavior
Steps to reproduce or test case
The value serialization results of rmap and rmapcache are inconsistent. Rmapcache will force the serialized value to undergo binary conversion again. This is not the result I want. I hope it can be fixed
Redis version
Redisson version
Redisson configuration

There are two serialization formats in a key hash

RMapCache keys has additional info about entry idle time. This can't be changed


UI redesign: add counter to compose box Drafts button

We should show a counter next to the compose box drafts button. This counter should only include drafts for the current compose context. No counter should be shown where there aren't any.
Note that the Drafts button is not shown in narrow windows.
The maximum displayed number is 99+
This issue is part of #24878.

Live demo with the source code on Repl.it. Here is the decomposed source code archive:
compose-decomposed.zip
The css code is split to decomposed areas but all the js code is in a single file. The file isn't big so it is easier to follow the logic of interactions between html and js.

hey @alya , if the issue is not resolved ,i can work on this


[unit test] use correct line separator instead of \n

Motivation
The unit test use \n, it can only paas in linux and OSX, can't run on windows.
Modifications
use System line separator instead of \n
Documentation
Check the box below and label this PR (if you have committer privilege).
Need to update docs?

[ ] no-need-doc

a little enhance, doesn't need doc

@sijie @codelipenghui @BewareMyPower PTAL

@merlimat @eolivelli PTAL


feat: don't add args with same value as default(#13089)

Fixes point 6 of #13089

I agree with @agilgur5 here. This is a lot of extra code for a minor shrinkage of the resulting pods. I'd prefer we didn't merge this as is.


Update phases.md

Modified details related to default value for the clean setting in UI.

@juliakm,
Can you review the proposed changes?
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


new numpy linking scheme

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


websocket server did not response to the client

hi ,
I am trying to modifying the websocket test scripts ,
var WebSocket = require('ws');
var ws = new WebSocket('ws://root:root@<IP_ADDRESS>:8898/web2');
ws.on('open', function open() {
ws.send('hello');
console.log('open');
});
ws.on('message', function (data) {
console.log(data);
process.exit(1);
ws.close();
});
but in the mongo client , I could not find the data was injected to the lan server ... can you please clarifying if the websocket is working ?
thanks

Hi @boonpin1975,
I'm so sorry to see this issue,is this issue still in here?

hi @phodal
this issue still happen ... mongo did not catch the data feeded by the client

Hi @boonpin1975 ,
Actually, this project was almost in deprecated stage, because it's user-less. And if you still use this, I will check it later.
At the beginning, the websocket part have a bad design, it need more time to fix it.

hi @phodal
it should be ok , I bought your IOT book and want to try out the IOT middleware


impl standard IO traits for hashing and encryption/decryption

See #227.
TODOs
Hashing

[x] implement from_reader for BLAKE2b
[ ] implement from_reader for sha2 (if we feel like we need to)

[ ] sha256
[ ] sha384
[ ] sha512

AEAD

[ ] implement seal_copy for XChaCha20Poly1305
[ ] implement open_copy for XChaCha20Poly1305
[ ] Other, non-high-level-interface methods (?)

General

[ ] decide whether to keep or change all these function names, like from_reader, seal_copy, open_copy

TODOs
Hashing

[ ] implement from_reader for Argon2i
[ ] implement from_reader for sha2 (if we feel like we need to)

[ ] sha256
[ ] sha384
[ ] sha512

AEAD

[ ] implement seal_copy for XChaCha20Poly1305
[ ] implement open_copy for XChaCha20Poly1305
[ ] Other, non-high-level-interface methods (?)

General

[ ] decide whether to keep or change all these function names, like from_reader, seal_copy, open_copy

@vlmutolo I think it'd be best to keep it behind safe_api for now. We can look into if it makes sense to provide Read/Write for hazardous-only things later.
ATM, I think safe_api makes most sense. Also, with the serde feature, I'd like to get the new wiki section written first, to see how much the new serde feature complicates the feature-history of Orion.

Is there something that blocks progression on this @vlmutolo? Should we discuss it more, maybe play around with more examples?

So I don't think anything is "blocking" this exactly, but there are some open design questions I have that would be good to discuss. I'll write a bit about that on the relevant issue.

I answered in the wrong place regarding some things. Apologies. Re-posting here:

decide whether to keep or change all these function names, like from_reader, seal_copy, open_copy

form_reader seems good to me. Perhaps the naming of the functions for non-streaming AEAD could be made a bit less generic. How about sealing_read()/opening_write() or seal_read()/seal_write()?

Do you think there should be a new io feature for this? It currently doesn't compile without default features.

I don't think another io feature is necessary. If we just gate it behind safe-api that should be fine, just like we do with the std:error:Error trait.

Edited your issue to having completed from_reader for BLAKE2b instead of Argon2i.

For when the implementation is more stable, I think it would be beneficial to try and integrate the *_from_reader() function into the streaming API consistency testing framework, to ensure equivalence in behavior. Possibly also Write impl if applicable. AEAD testing framework is an obvious candidate. But these could also be a different PR.

Maybe it would be a good idea to split this PR into two: one for hashing, one for AEAD.

Maybe it would be a good idea to split this PR into two: one for hashing, one for AEAD.

Good idea. I'd probably do so, especially to work on the tests and not grow the PRs too large with that alone.

I tried to use something like #[cfg_attr(docsrs, doc(cfg(feature = "safe_api")))] on the Blake2b Write impl, but couldn't figure get the little docs warning to show up. I also looked through how Orion already uses it and found that it was added for the serde impls here. However, when I render the docs with cargo doc --features serde, I still don't see the little bubble pop up locally.  Am I just rendering the docs wrong?

Did you build the docs with the parameters listed in the README?

Yep, that worked. Everything shows up as it should, including the feature bubble for Blake2b's Write impl.

Please rebase upon master

👍

do you plan to add Write impls for SHA2 primitives

That's a good question. I can definitely do the same write impl for SHA as I did for Blake. Wasn't thinking about that, but I should probably add it to this PR.


Subscribe to close event on connect and connectTo

I don't think #150 is completely fixed by v1.6.0, as it currently only rejects the promise on an error event. Closing the dialog by clicking elsewhere on the page does not reject the promise.
I have not tested these changes locally but believe they should work

Done! Published as v1.6.1. Thanks for the PR! 🙌

Awesome, just tested it out and it's working perfectly. Thanks for the quick release!


./new_dataset.npz

Hi,
May I know what is inside ./new_dataset.npz?
It seems that in https://www.kaggle.com/datasets/niloy333/kuhar, I can't find signals and labels that are specified in:
f = np.load('./new_dataset.npz')
signals = f['signals']
labels = f['labels']
Thanks in advance,

Hello,
May I know which version of Tensorflow you used?
Thank you,

Hello,
the version was 2.9.
new_dataset.npz contains samples after data augmentation of original kuHAR dataset. The data augmentation process logicaly combines the pairs of 2 different activities into one windows. It represents the transition from one activity to another one.
new_dataset.npz must contains signals and labels because is saving with
np.savez_compressed("new_dataset", signals=final_signals, labels=final_labels) !!!
Thanks.


Folder vector is missing from the installation with gem install

There's no subfolder vector in lib/pinecone in my installation. I tried ruby 2.7.0 and 3.2.1, versions 0.1.4 and several minutes old 0.1.5.
I guess it's because of files in a gemspec being formed wrong

On my side, I would wonder, how it doesn't appear in your environment :)
However, it is fixed for me now. Thanks for the fast reaction!


Stop using naive code

Describe the bug
We still use some naive code. Search the codebase for NAIVE to find functions that are know to be extremely slow but they are still used in non-optimized ways.
Expected behavior
Don't use known slow code.

All relevant naive code is gone now. Only some parts of weak subjectivity use naive code but that's tolerable since it's run once and under some usecases only


Backends and executors

Purpose/implementation Section
What changes are being implemented in this Pull Request?
What was your approach?
What GitHub issue does your pull request address?
Tell potential reviewers what kind of feedback you are soliciting.
New Content Checklist

[ ] New content/chapter is in an Rmd file with this kind of format and headers.

[ ] New content/chapter contains learning objectives.

[ ] Bookdown successfully re-renders and any new content files have been added to the _bookdown.yml.

[ ] Spell check runs successfully).

[ ] Any newly necessary packages that are needed have been added to the Dockerfile and image.

[ ] Images are in the correct format for rendering.

[ ] Every new image has alt text and is in a Google Slide.

[ ] Each slide is described in the notes of the slide so learners relying on a screen reader can access the content. See https://lastcallmedia.com/blog/accessible-comics for more guidance on this.

[ ] The color palette choices of the slide are contrasted in a way that is friendly to those with color vision deficiencies.
You can check this using Color Oracle.

consider making this the home of https://github.com/fhdsl/WDL-sandbox/commit/639c245bb8045d0962927e0a68ae3b3dae69943c

Not sure what happened to the sidebar for a few commits there, but it's working now!


Extract Variable on string literal

If I run Extract Variable on a string literal I get a confusing message, since I suppose you're really running the rename refactoring.
This also happens if there just happens to be a string literal with the guessed extracted name somewhere in the same scope.

What do you want me to do? That's the default behavior in IntelliJ...

I suppose I could use our own implementation of linked mode instead of calling the rename refactoring, that would fix the problem.

Or extend (or copy/paste) and hack their rename refactoring.

Fixed, thanks.


Fixed bug 14195: Added C++ mangling for passing function signatures

Added support for mangling the C++ equivalent type of:
template <typename T>
struct Delegate1 {};

template <typename R1>
struct Delegate1 < R1() > {};

template <typename T1, typename T2>
struct Delegate2 {};

template < typename R1, typename T1, typename T2, typename R2, typename T3, typename T4 >
struct Delegate2<R1(T1, T2), R2(T3, T4)> {};

void test14195a(Delegate1<void()> func){}

void test14195b(Delegate2<int(float, double), int(float, double)> func){}

The corresponding D code can be found in cppmangle.d. I hit this issue when trying to bind a C++ Delegate implementation to D.

How hard is it to do linux mangling as well?  I'd prefer the tests were in cppa/cppb so that the mangling is automatically verified.

I'd like to see a test for this in cppa/cppb as well for the same reason @yebblies stated. And such would provide motivation, and a test case, for doing the Linux mangling as well.

Juding from the source code this is already implemented for linux. I will move the tests over to cppa/cppb as soon as I find time.

I tried enabeling the test on non-windows systems and got the following error message from the build machines: undefined reference to `DelegateTest1(Delegate1<void ()>)'
While this tells me that it is almost correctly implemented (because the linker can demangle it), the fact that it does not link tells me its only 'almost' correct. I don't have a linux development environment setup at the moment and don't have time nor interrest to fix this on non-windows systems.

Sorry, forget what I said. It is already correctly implemented on linux, I just made a mistake transitioning the test to cppa/cppb.

In future pull requests please remember to add a bugzilla link to the description.

Auto-merge toggled on


Create an email distribution list for the sustainability governance team

Tom Tally, the leader of the sustainability initiative, wants to send emails to the rest of the sustainability governance team.  This task is to complete the sample that locates the sustainability governance roles and for each locate the people appointed to this role and for each, extract their work email and by this process, build up a table of roles, names and email addresses for the whole team.

The link to where the sample should be stored:
https://github.com/odpi/egeria-samples-api/tree/main/governance-program/sustainability-team-emails


New post is not appeared on the top of the New feed

Describe the bug
New post is not appeared on the top of the New feed
To Reproduce
Steps to reproduce the behavior:

Sign up 2 different account and open two incognito windows
post in one of the window
press on feed tabs to change and reload posts

Expected behavior
While click on new tab, the newest post should appear on top of the page
Screenshots

Won't fix due to sunsetting the project


Salvage chemicals

Description
In essence, salvage should be an additional source of chemicals.

Crates containing an assortment of raw materials should have a chance to show up in it's loot pool.
Salvage mobs should ideally have reagents useful for something (i.e. make carpotoxin harvestable)
Some wrecks should have unique chems, mostly only useful with equipment available in medbay/R&D (recipes need heating/catalysts?) so salv has reason to go talk with them.

As for what these additional chems should be, probably needs bikeshedded, but the idea's here.

i like the way afterlight does it with bottles and flasks of random reagents that can spawn on wrecks


bump terraform-equinix-fabric-connection module version to 0.4.0

Note: In fabric connection modules for cloud providers all the connection logic happens in submodule terraform-equinix-fabric-connection, therefore user-agent will include the terraform-equinix-fabric-connection metadata module name and not terraform-equinix-fabric-connection-azure in this case.

Please add the DRE team to this repo so that our approving reviews will make PRs mergeable.


Add Storybook to github pages

create script and actions to build storybook to the github pages on merge to master

Done


t-map(house-face)

@matthew-brett
We got a t-map for beta_house - beta_face for each run, and  I attached the plot and code.
could you please let me know if it is correct?
Code:
hf1 = np.loadtxt('house_betas_task001_run001.txt') - np.loadtxt('face_betas_task001_run001.txt')
hf2 = np.loadtxt('house_betas_task001_run002.txt') - np.loadtxt('face_betas_task001_run002.txt')
hf3 = np.loadtxt('house_betas_task001_run003.txt') - np.loadtxt('face_betas_task001_run003.txt')
hf4 = np.loadtxt('house_betas_task001_run004.txt') - np.loadtxt('face_betas_task001_run004.txt')
hf5 = np.loadtxt('house_betas_task001_run005.txt') - np.loadtxt('face_betas_task001_run005.txt')
hf6 = np.loadtxt('house_betas_task001_run006.txt') - np.loadtxt('face_betas_task001_run006.txt')
hf7 = np.loadtxt('house_betas_task001_run007.txt') - np.loadtxt('face_betas_task001_run007.txt')
hf8 = np.loadtxt('house_betas_task001_run008.txt') - np.loadtxt('face_betas_task001_run008.txt')
hf9 = np.loadtxt('house_betas_task001_run009.txt') - np.loadtxt('face_betas_task001_run009.txt')
hf10 = np.loadtxt('house_betas_task001_run010.txt') - np.loadtxt('face_betas_task001_run010.txt')
hf11 = np.loadtxt('house_betas_task001_run011.txt') - np.loadtxt('face_betas_task001_run011.txt')
hf12 = np.loadtxt('house_betas_task001_run012.txt') - np.loadtxt('face_betas_task001_run012.txt')
matrix =np.vstack((hf1, hf2, hf3, hf4, hf5, hf6, hf7, hf8, hf9, hf10, hf11, hf12))
t_hf = t_test(matrix)
t_hf_3d = np.reshape(t_hf, (91,109,91))
plt.imshow(t_hf_3d[:,:,45])
plot:

I can't say whether the code is correct - but the resulting map does look like it should, yes.
It would be good if you were able to avoid most of those very similar statements to make the 12 maps, maybe with a loop.

@matthew-brett I am still confused what the image is telling us. What is t-map telling us?

Those two red dots are areas where the signal (amount of the hemodyamic regressor shape in the BOLD signal) is high relative to the variance across runs.
In fact, you've just found the Parahippocampal Place area.  Have a look at the rather similar (but thresholded, displayed on anatomy) images in http://www.sciencedirect.com/science/article/pii/S0896627300807588

Thank you for the link. Will read tonight


Update stratified-sampling.md

🚀 Deployed on https://669fe8dc0b98060846a0b2a2--cozy-fox-0defec.netlify.app


Documents

Good afternoon I write to indicate that all xlsx documents can not be opened or give error when trying to open them, so please replace it with the files that if functions thank you very much

Working on getting this document reuploaded

I've tested and both documents are valid. They need to be downloaded to your computer first.


[Java] Binary Tree Traversal (Breadth First)

Added Solution for BFS traversal of a binary tree in java's folder

Thank you 😸


Gluster deployment for OCP

Optional playbook to deploy gluster. This PR include functionality to add facts to the cloudformation template to allow for subsequent templates to request variables from the original cluster.

 Deploy gluster server [3.6-OCP]

aos-ci-test


Should we document the requirements for the cloud backends?

Our CLI should suggest what is required to be set for the cloud backends. This would mean that https://www.pulumi.com/docs/reference/cli/pulumi_login/ would expose the Environment Variables that are supported for each of the cloud backends
We are using gocloud.dev for this work and not everything that the pulumi CLI supports is supported

Closing this as wont-fix. In brief, Pulumi has many ways of managing cloud credentials with environment variables being only one of those ways. The cloud-specific environment variables for credentials/configuration are generally documented in our various cloud-specific Get Started guides (e.g. this one for AWS.


Consider using extended format for logging

@dune73 suggested to use the extended log format as described here: https://www.netnea.com/cms/apache-tutorial-5_extending-access-log/#step_4_configuring_the_new,_extended_log_format.
Variables are described here: https://www.netnea.com/cms/apache-tutorial-6_embedding-modsecurity/#step_5_creating_the_base_configuration.
LogFormat "%h %{GEOIP_COUNTRY_CODE}e %u [%{%Y-%m-%d %H:%M:%S}t.%{usec_frac}t] \"%r\" %>s %b \
\"%{Referer}i\" \"%{User-Agent}i\" \"%{Content-Type}i\" %{remote}p %v %A %p %R \
%{BALANCER_WORKER_ROUTE}e %X \"%{cookie}n\" %{UNIQUE_ID}e %{SSL_PROTOCOL}x %{SSL_CIPHER}x \
%I %O %{ratio}n%% %D %{ModSecTimeIn}e %{ApplicationTime}e %{ModSecTimeOut}e \
%{ModSecAnomalyScoreInPLs}e %{ModSecAnomalyScoreOutPLs}e \
%{ModSecAnomalyScoreIn}e %{ModSecAnomalyScoreOut}e" extended

If we do this, I suggest also updating the nginx log format to match this.

Nginx does not allow you to display arbitrary env variables in the logfile. About half of the useful information above can not be displayed via nginx. A workaround is to display the info via ModSec alert messages. The reporting level introduced in CRS v4 allows that to a certain extent, but does not come close to the logfile above.
Example production log with the format above:
https://www.netnea.com/files/tutorial-5-example-access.log
Please notice that the %{ApplicationTime}e is empty in the example.
Aliases supporting extraction of arbitrary columns: https://raw.githubusercontent.com/Apache-Labor/labor/master/bin/.apache-modsec.alias


Revoke KYC

In TokenRevokeKycTransitionLogic.class:
Replaced TokenStore.class with TypedTokenStore.class.
Removed HederaLedger.class and added AccountStore.class.
The doStateTransition() method now instantiates TokenRelationship.class and invokes the newly created method : revokeKYC().
In TokenRelationship.class:
Added new method revokeKYC()
In the TokenRevokeKycTransitionLogicTest.class:
Refactoring to reflect the new dependencies in the class under test.
One of the tests was removed due to the fact, that the tested behavior is not responsibility of TokenRevokeKycTransitionLogic.class

Closing this as it's being resolved at upstream repo.


Another Connect4System

Hello, this is my own version of the connect4 custom command.
At first I was thinking to just leave this on my repo, but, I didn't enjoy the thought of someone else's code of the same thing being PRed xD.
Anyways, hope this doesn't go against the guidelines. Thanks for checking out my pull request.
Features:

a working gameTie/win/loss system
a turn based embed that'll help the player recognize what their color is
drops are highlighted in blue to help see the last player's move

Status

[x] Code changes have been tested on an instance of YAGPDB, or there are no code changes
[x] I have read and followed the contribution guide

Haven't had time to go over this in depth, but out of curiosity what was the issue referred to in this commit? https://github.com/yagpdb-cc/yagpdb-cc/pull/212/commits/7e314365c3b270c9d61763d58dcba9f0eec209ba

ahaha that was actually just a dummy mistake from my part as I was using 2 different codes while i was testing in 2 different servers 😅
I made a couple changes in this last commit, and I think these are the final adjustments.
Thanks again ^ you can take a look at it whenever you have the time.

I'm updating the ReadMe.md file with more information so if you see this, please wait for a minute.


Questions about specs

Can every element (or every node) have different UP axis? Or one UP axis for entire document?
Camera UP axis: https://forums.khronos.org/showthread.php/13262-COLLADA-1-5-Camera-Orientation
Spotlight: https://github.com/KhronosGroup/glTF/issues/824

I answered 3 in your link.
2 (I should check in at those forums) I would say that "always looks at -Z" depends on <up_axis> to define -Z. There's an implicit transform in the camera that must be dealt with regardless since -Z depends on it.
1 is pretty straightforward, but only elements with  in them and some elements like  I think it would not make sense to consider their <up_axis> because  is already very bad in so far as there is no single place that describes its contents and there is no "library_sources" and it's just very bad design IMO.

@mick-p1982 thanks for your comments and answers, I'll try to understand (1), I think spec should only allow 1 up_axis for entire document, this would make coord sys homogeneous

You have to think of it like a web of documents. And a webserver using a REST protocol could generate DAE resources on the fly drawing from many places, like webpages are created.  You have to think bigger. It's more like the WWW.

I got it, good example! But the part I don't understand is that should I add extra transforms (to node)  becaue of different up_axis-es ? Check this to see example: https://github.com/KhronosGroup/OpenCOLLADA/issues/517 .

The up_axis isn't technically a transform. See the Update_Matrix example in the COLLADA-DOM RT component's code for one way to do it (flipping the scalar components of <translate> for example) but there are different ways with different advantages. I think doing it at the source is best. If you try to do it based on if the parent and child have different <up_axis> you'll run into some troubles with instancing and controllers where there is no clear parent-and-child relationship.

It's been a while. I'm going to close this.
COLLADA seems as inactive standard: https://www.khronos.org/developers
So I'll change my direction about implementing all details in COLLADA spec. I may drop some features which are probably CG and GLSL profiles and some other platform (especially dead ones) dependent features to keep importer interface simple...
I'll work on my own file format: http://github.com/recp/asset-xchange in the future. Probably AssetKit will be the official importer of that format. It will be binary format like JPEG.

@recp Inactivity can be a good thing. I don't think it means abandoned.

@mick-p1982 hope it is a good thing.
My goal was implementing all details of both 1.4.1 and 1.5+. But since the spec is not clear enough and buggy, and since I can't get clarifications or answers, I think it is time to think on what I should do now.
I think all profiles except commonProfile must be dropped. Actually all platform dependent things must be dropped in importer side. Because;

CG is dead.
Apple deprecated OpenGL and I think Khronos may abandon it in favor of Vulkan in the future. I will switch to Metal on Apple platform, to DirectX/Vulkan on other platforms. OpenGL is dead for me.

Lets not maintain dead platforms and platform specific features in exchange format. I do not know why they are in the spec as standard in the first place. In the new format I'll consider all of these...
I will continue to implement remain parts of COLLADA. I learned a lot. There are lot of things to say but I prefer to not say here, in a closed issue :)

Cg is very close to HLSL. Nvidia restored the documentation last I checked, so perhaps it's not so dead, after backlash. GLSL is essential. Yes, it's very flawed, but unless implemented it's a fools errand to amend. GLES2 is WebGL.
My recommendation is to we develop soft standards for COLLADA in the noncommercial sectors, that will inform future editions, once competent implementations exist. The COMMON (geometry) profile is not useful for anything. I think it can be improved a little with some recommendations I developed that allows textures to be modulated with colors (with subscript based semantics) so that it looks more like standard basic material systems, and because using 8-bit textures as colors is too lossy, memory hungry, and artist unfriendly.
OpenVX and also OpenGL ES don't have desktop implementations. So these are bigger failures than COLLADA in my book. Khronos leaves implementations to others. COLLADA is pure software, and after the survey of what should be included in its description, there is nothing left to do with it. So it's inactive.

The COMMON profile[s] can be extended via extensions. If an engine want to embed platform specific shaders, it can embed whatever it want to extra element. So I think there is no need such platform specific profiles. But COLLADA could defined a new universal shader language similar to GLSL. Render engines could translate that shader language to GLSL/HLSL. This is what I'm planning for shaders in the future. Same for MathML, COLLADA could only use some equations and operators for formulas, instead it forces us to support all MathML elements, another XML scheme with its rules.
I liked the formula idea instead of pre-defined curve, I will try to extend this idea in the future.
COLLADA material system is too old. Khronos could update COLLADA after glTF 2.0, they did not :-S
After my render engine's  (https://github.com/recp/gk) initial version has been released then there are lot of things I will/want work on (including game engine and modeling/animation tool...), all be written in C99 ;).  Each one of these will help to improve others. This is why I think I can create a good exchange/runtime format in the future.

Khronos leaves implementations to others.  COLLADA is pure software, and after the survey of what should be included in its description, there is nothing left to do with it. So it's inactive.

I really do not understand how things work at Khronos side. Maybe there are too many members and maybe this is their common decision, I do not know. The obvious thing is that COLLADA appears to be  dead 😔

 isn't useful for a COLLADA ecosystem. It's an anti-pattern. It's a spot fix for private use. It defeats the purpose of having a "COMMON" pathway. Going your own way is counterproductive.  It's an anti-pattern. It's a spot fix for private use.

It can help where common profile does not enough to describe/define assets. Multiple tools/engines  can use same extension for same purpose.

It defeats the purpose of having a "COMMON" pathway.

I think the exchange format itself must be COMMON, not only some pieces called "common".

Going your own way is counterproductive.

We are already alone with abandoned spec, aren't we? I may try to create small community or use it only for myself, we will see it in the future. One way or another, it will be my tools' and engines'   exchange format (internal or public). So it will worth it, nothing to lose.
AssetKit still supports (and will support) COLLADA and glTF. In the future, it may support more common formats e.g. obj as extension to original library (not in core). My own format is a special case. Of course I will  update design of library in favor of new format...

Yes, everyone is alone (reinventing the wheel.) And that's what defines failure. Insert "Insanity is making the same mistake/not realizing it forever" quote here.


batch resource explods with RuntimeError

Description

When running batch resource, it is giving RuntimeError implicit argument passing of super from method defined by define_method() is not supported. Specify all arguments explicitly.

Chef Version
Chef Infra Client version: 15.7.32
Platform Version
Windows 2012r2
Replication Case
When running chef-apply with this recipe:
Chef::ReservedNames::Win32::Security.add_account_right("azure", 'SeAssignPrimaryTokenPrivilege')

hasReplaceTokenRight = Chef::ReservedNames::Win32::Security.get_account_right("azure").include?('SeAssignPrimaryTokenPrivilege')
log "hasReplaceTokenRight = #{hasReplaceTokenRight}" do
  level :warn
end

batch "Capture environment running as azure" do
  code "set > D:/env.txt"
end

it will give an error like this RuntimeError: implicit argument passing of super from method defined by define_method() is not supported. Specify all arguments explicitly.

Its Fixed: https://github.com/chef/chef/commit/cc792527ad9d8222b3a0990e70f2d59f86f074ae
now we can close this issue.


Timeout before end

Hi there, i've been trying to download an RTMP stream using librtmp, but it seems to only download the first ~5MB then pauses for a bit and returns success.
The same command using rtmpdump works just fine.
Reproduce: https://gist.github.com/mic159/b71a8efdad7f67916ff0
The rtmpdump command will be printed to test with.
OS: Ubuntu 14.10 24bit
Could you tell me if I'm doing something wrong?

You could try passing the timeout argument to librtmp.RTMP().
connection = librtmp.RTMP(host, playpath=file, swfurl=SWF_URL, timeout=60)
timeout – int, Timeout the session after num seconds without receiving any data from the server. The default is 30.

@norwack Thanks, increasing the timeout does help and it will now download the whole video.
But, it takes ages! running the rtmpdump command is much faster. What can I do to the script to make it just as fast?
I have updated the gist with the timeout increased.
Python:
real	22m13.594s
user	0m2.697s
sys	0m0.852s

rtmpdump:
real	1m49.613s
user	0m0.863s
sys	0m0.789s

The filename on the 'playpath' ends with 'mp4', but you are correct, the container is actually 'flv'.
The append option is for when opening the file multiple times, not for every write call.

@mic159 What did you set the timeout to? Try playing around with it a little.

@norwack I set it to 60 like you suggested (check the gist).
It seems like it gets a burst of 5MB at the start, then it pauses for 40 seconds then just starts drip feeding the rest of the content at a very slow speed.
But with rtmpdump it goes at the full speed the whole time.

@norwack I looked over rtmpdump and my theory is that i need to tell the server to send the whole video in the "buffer" using setBufferTimeMS just like rtmpdump does.

@mic159 Try setting the buffer parameter in librtmp.RTMP().
Something like this:
connection = librtmp.RTMP(host, playpath=file, swfurl=SWF_URL, buffer=30000)
Note that 30000 is the default value, try to play around with it.

@norwack Yes! That works!
But unfortunately it does not seem to let you change it after the stream has started, like what rtmpdump does.
I want to:
def download_video(file, host):
    connection = librtmp.RTMP(host, playpath=file)
    connection.connect()
    stream = connection.create_stream()

    # Grab first chunk
    data = stream.read(1024 * 2)
    yield data

    connection.set_option('buffer', int(stream.duration * 1000))

    # Read rest of file...

But it has no effect. It seems to have to be set before the create_stream method is called. like this:
def download_video(file, host):
    connection = librtmp.RTMP(host, playpath=file)
    connection.connect()
    connection.set_option('buffer', 1372137)
    stream = connection.create_stream()

    # Grab first chunk
    data = stream.read(1024 * 2)
    yield data

    # Read rest of file...

Is there a way to get the duration before reading the first chunk, or to be able to set the option after starting the stream?

I'm unsure, hopefully @chrippa can help out on this.

I've added a option (enabled by default) to RTMP.create_stream which toggles the same buffer hack by rtmpdump. You can also set the buffer manually with RTMPStream.update_buffer now.


Add line errors to polling matchers

After investigating this it doesn't appear to be possible with the current implementation, closing.


False positive: structure keys reported as variables without scope prefix

The following code code will trigger a false positive:
var testStructure = {
    testKey = "testValue"
};

reporting that testKey is "not declared within a var statement", where it's a structure key and hence doesn't need var.

What version are you using?
The latest in the dev branch, and release 0.7.3 don't flag for me.
Full example
 component {

public void function function1() {
	var testStructure = {
		testKey = "testValue"
	};
}
 }

I'm seeing the same issue with the latest master version and the latest dev version that built successfully. When prepping parameters for QueryExecute(), I get an error warning for each param, when it's structured this way:
    var params = {
      claimId = { value = claimDetail.getID(), cfsqltype = "CF_SQL_INTEGER" },
      serialNo = { value = claimDetail.getSerialNo(), cfsqltype = "CF_SQL_VARCHAR" }
    };

I.e.: "Variable claimId is not declared with a var statement. Use var or the local scope, or otherwise clarify the scope".
Happy to provide more information about version used, etc.

I didn't change any related code that I'm aware of, but it works for me on latest dev build
See:
/CFLint/src/test/resources/com/cflint/tests/VarScoper/structure_keys_197.cfc
/CFLint/src/test/resources/com/cflint/tests/VarScoper/structure_keys_197.expected.txt
If it's still broken for you (on dev branch), please provide a full source example.

I can build off the latest dev branch, but when I run cflint on a folder, I get:
Exception in thread "main" java.lang.NullPointerException
	at com.cflint.main.CFLintMain.mergeConfigFileInFilter(CFLintMain.java:439)
	at com.cflint.main.CFLintMain.execute(CFLintMain.java:364)
	at com.cflint.main.CFLintMain.main(CFLintMain.java:252)

Is there something else that I should be doing? Sorry for the bother and thanks for the help.

ok, sorry about that.   I null proofed that line - though I couldn't duplicate it with
java -jar target\CFLint-0.8.0-all.jar -folder c:\temp
try again?

Thanks Ryan! It builds and runs without issue now. I put together a repo with my failing example. It appears that it's actually related to a parsing error, which in turn generates the false positives.
https://github.com/mjclemente/cflintParamStruct

Ah a multiline quoted string.  Interesting.  Any idea with dialects and versions support that?

I actually don't think it's the multiline string that's throwing it off. I think it's the boolean !len( thing.getName() ) in the param:
name = { "value" = thing.getName(), "cfsqltype" = "CF_SQL_VARCHAR", null = !len( thing.getName() ) }

When I set it to be simply true or false, there are no issues with the parsing/validation.
And actually, I just tested including the pound signs for it, and that also works. That is, this syntax evaluates without issue: #!len( thing.getName() )#
I don't want to make more work for you. Including the pound signs is an acceptable workaround for my issue.

fixed in cfparser
https://github.com/cfparser/cfparser/issues/60
it will trickle out when the next cfparser and cflint releases happen.  Until then you can build it yourself.
Thanks for taking time to identify the issue.

@ryaneberly Thanks for all the work you put into cflint!


Fix fuse hardlinks

Adds multiple paths to the filesystem for each hardlink, maintaining the original inode number.

LGTM


Shapes lose connection to Material resource

Hi, nice plugin! Enjoying it so far.
I've noticed that if I create a Material for the MeshShapers of my shapes, and save that Material to the filesystem, after some time the material in the MeshShaper gets made unique and loses its connection to the saved material. This is a bit of a pain as it's nice to be able to have a central place in the filesystem to tweak and improve reused materials, and the unique materials add extra draw calls I think, too.
Cheers.

Thanks for the heads up. If you do submit a fix, I'm happy to do a PR and some testing, though I won't be updating for a few months.


Where has the module spring-cloud-stream-codec gone?

I'm trying to upgrade https://github.com/spring-cloud/spring-cloud-stream-binder-jms so that we can upgrade to spring-boot 2 for our projects. There is a dependency on org.springframework.cloud.stream.config.codec.kryo.KryoCodecAutoConfiguration. Has this been moved to another module/project or been deprecated?

@philiplourandos  Yes, the module is no longer used  and was removed in Spring Cloud Stream 2.0.0.RELEASE which has been released few days ago.
We've also deprecated type-based serialization such as Kryo and Java serialization.
That said, the module is still available in 1.3 so you can use it.
Also, keep in mind that only Kafka and Rabbit binders are currently supported and neither requires Kryo.
I am closing this issue as the question has been answered, but feel free to open another issue if you have more questions/concerns.


Re-use private subnets in subnet groups?

Rather than creating new subnets for each type of subnet group, we'd like to create the elasticache, redshift, and db subnet groups using the private subnets already created by the module.

I ended up making a separate resource to create the db subnet group.
resource "aws_db_subnet_group" "db-subnet-group" {
  name       = "${var.env}-db-subnet-group"
  subnet_ids = module.vpc.private_subnets
}

I also would like support for that in this module though.


chore(deps): update ci dependencies

This PR contains the following updates:

Package
Type
Update
Change

common-dev-assets

digest
b58632e -> 1a4e4e6

github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper
require
minor
v1.21.10 -> v1.22.0

Release Notes

terraform-ibm-modules/ibmcloud-terratest-wrapper (github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper)
v1.22.0
Compare Source
Features

cbr cleanup (#658) (7721cbe)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

/run pipeline

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


BUG: EntityNodes have incorrect texture offsets

A tiny error with a big impact. The entity node processing uses the wrong offset on the texture atlas, and so textures do not work correctly.

Will be fixed in 0.8.2.


sound not captured when stop it

Hello, it's a great job that you did, I have a small problem but I can not solve it, I hope you can help me ..
The problem is when I stop and re-start the playback of a sound, in the recording is not reflected, it seems as if I never stopped it, the sound is continuous with no pause
How can it be fixed?
I'm experimenting with your example on iOS 10 and air 25...
           .

Hey, give me couple more days to check this out, if you can't find a solution yourself, thanks.


apply settings to targeted platforms

@sjrd haha I know. That's why it's good to test in the wild before :).

@MasseGuillaume I've restarted the CI but it's still failing.

@densh caching issue, it's only failing on macosx. We should not cache ~/.ivy/cache for SNAPSHOT.

LGTM!