Initialization error

I am getting the following init error from main():
flutter: SettingsModel init error: type 'Null' is not a subtype of type 'Object'
[ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: Error initializing settings model. Model: PrintSettings. null

It seems as though the issue is coming from having nullable properties, is this not allowed?

Thank you for trying out the library and sharing your experience. Yes, you are right about nullable values in properties. This is not intended. Perhaps it should have been mentioned in the description. All classes inheriting from BasedProperty should use actual values for settings and should not have null values. If a value is null, it means the absence of such setting. For example, as for default values, you can use things like empty strings "" or 0


KmlDataSource coordinates converted using the WGS84 / earth ellipsoid

I am loading some KML to render on my moon ellipsoid.  When I look at the coordinate conversion in KmlDataSource, I see that it is not using any ellipsoid to calculate the coordinates.
For example, with:
longitude: 24.070617695061806
latitude:87.90173269295278
I get coordinates:
Cartesian3 {x:<PHONE_NUMBER>247161, y: 95566.36983235707, z: 6352461.425213023}
instead of lunar coordinates:
Cartesian3 {x: 58080.7702560248, y: 25945.04756005268, z: 1736235.0758562544}
I suggest the constructor for a KmlDataSource takes an option for an ellipsoid, which is then used for every single Cartesian3.fromDegrees call.

Thanks @tamarmot!  While we do support non-earth ellipsoids, it's one of the areas that is least tested because most of our users are using WGS84.
If you find any more issues, let us know!
As a workaround, setting Cesium.Ellipsoid.WGS84 = new Cesium.Ellipsoid(x, y, z) should fix most problems

Now that is a clever hack ... er ... workaround!  Thanks!
On Thu, Feb 1, 2018 at 12:18 PM, Hannah<EMAIL_ADDRESS>wrote:

Thanks @tamarmot https://github.com/tamarmot! While we do support
non-earth ellipsoids, it's one of the areas that is least tested because
most of our users are using WGS84.
If you find any more issues, let us know!
As a workaround, setting Cesium.Ellipsoid.WGS84 = new Cesium.Ellipsoid(x,
y, z) should fix most problems
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/AnalyticalGraphicsInc/cesium/issues/6176#issuecomment-362389115,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFgK8Kg8rcuSqfdWJw2IDTY2EAPnKxQGks5tQhv7gaJpZM4R2MwY
.

For the record, making that change does not correct this issue ...

... because it is hardcoded in Cartesian3:
var wgs84RadiiSquared = new Cartesian3(6378137.0 * 6378137.0, 6378137.0 * 6378137.0, 6356752.3142451793 * 6356752.3142451793);
maybe not so flexible ...

Ah yeah.  We had to do that because the Ellipsoid module requires Cartesian3, so we can't also have the Cartesian module require Ellipsoid because of the circular dependency.
If you have time, we'd be happy to review a PR with the KML fix!

Just wanted to stop in and say hi, @tamarmot!
But now that I'm here, that's definitely a bug (KmlDataSource should not be using Cartesian3.fromDegrees, or any of our internal code for that matter).  We have those hardcoded values purely because it lets us provide that helper function for end users, but internally it's a no go.

Thanks guys!  I just pushed some code in my fork to fix it -- I will do a
little testing and then a pull request.
WHILE YOU ARE LOOKING AT THIS, @mramato, what I'm really working on is
polar stereographic projection on the moon:
https://groups.google.com/forum/#!topic/cesium-dev/e_osjMgHD4E
I know this is off topic but if you have a chance to make suggestions that
would be very helpful.  Once I solve this I would love to contribute it.
I've done what I thought was the "hard part", namely the projecting
to/from, but the bounds of my tiles are drawing in the wrong place.
Thanks!  I'm having so much fun with Cesium.
Tamar
p.s. and my fix does not refactor your dependency on
Cartesian3.fromDegrees, just passes along the ellipsoid.
On Thu, Feb 1, 2018 at 12:37 PM, Matthew Amato<EMAIL_ADDRESS>wrote:

Just wanted to stop in and say hi, @tamarmot https://github.com/tamarmot
!
But now that I'm here, that's definitely a bug (KmlDataSource should not
be using Cartesian3.fromDegrees, or any of our internal code for that
matter). We have those hardcoded values purely because it lets us provide
that helper function for end users, but internally it's a no go.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/AnalyticalGraphicsInc/cesium/issues/6176#issuecomment-362394242,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFgK8JFTCzN142o-TYKzu9bGTqVBcerDks5tQiCigaJpZM4R2MwY
.

Are there instructions somewhere for doing a development npm build?
Doing this (from my fork):
npm install<EMAIL_ADDRESS>completely broke my webpack process, which works when I use the regularly
released npm cesium.
Thanks
Tamar
On Thu, Feb 1, 2018 at 12:51 PM, Tamar Cohen<EMAIL_ADDRESS>wrote:

Thanks guys!  I just pushed some code in my fork to fix it -- I will do a
little testing and then a pull request.
WHILE YOU ARE LOOKING AT THIS, @mramato, what I'm really working on is
polar stereographic projection on the moon:
https://groups.google.com/forum/#!topic/cesium-dev/e_osjMgHD4E
I know this is off topic but if you have a chance to make suggestions that
would be very helpful.  Once I solve this I would love to contribute it.
I've done what I thought was the "hard part", namely the projecting
to/from, but the bounds of my tiles are drawing in the wrong place.
Thanks!  I'm having so much fun with Cesium.
Tamar
p.s. and my fix does not refactor your dependency on
Cartesian3.fromDegrees, just passes along the ellipsoid.
On Thu, Feb 1, 2018 at 12:37 PM, Matthew Amato<EMAIL_ADDRESS>wrote:

Just wanted to stop in and say hi, @tamarmot
https://github.com/tamarmot!
But now that I'm here, that's definitely a bug (KmlDataSource should not
be using Cartesian3.fromDegrees, or any of our internal code for that
matter). We have those hardcoded values purely because it lets us provide
that helper function for end users, but internally it's a no go.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/AnalyticalGraphicsInc/cesium/issues/6176#issuecomment-362394242,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFgK8JFTCzN142o-TYKzu9bGTqVBcerDks5tQiCigaJpZM4R2MwY
.

Hi @tamarmot, we just pushed an npm fix about half an hour ago - https://www.npmjs.com/package/cesium
I'm running the build right now with our webpack configuration to see if that is working.

Thanks Gabby!
On Thu, Feb 1, 2018 at 1:31 PM, Gabby Getz<EMAIL_ADDRESS>wrote:

Hi @tamarmot https://github.com/tamarmot, we just pushed an npm fix
about half an hour ago - https://www.npmjs.com/package/cesium
I'm running the build right now with our webpack configuration
https://github.com/AnalyticalGraphicsInc/cesium-webpack-example to see
if that is working.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/AnalyticalGraphicsInc/cesium/issues/6176#issuecomment-362408524,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFgK8JVkquWvrCoQ6M3H0haxqJXSrAflks5tQi1OgaJpZM4R2MwY
.

Sorry -- that didn't answer my question -- is there documentation for how I
can take my fork and npm-ify it so I can test it in my environment before I
do a pull request?
Thanks
Tamar
On Thu, Feb 1, 2018 at 1:32 PM, Tamar Cohen<EMAIL_ADDRESS>wrote:

Thanks Gabby!
On Thu, Feb 1, 2018 at 1:31 PM, Gabby Getz<EMAIL_ADDRESS>wrote:

Hi @tamarmot https://github.com/tamarmot, we just pushed an npm fix
about half an hour ago - https://www.npmjs.com/package/cesium
I'm running the build right now with our webpack configuration
https://github.com/AnalyticalGraphicsInc/cesium-webpack-example to see
if that is working.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/AnalyticalGraphicsInc/cesium/issues/6176#issuecomment-362408524,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFgK8JVkquWvrCoQ6M3H0haxqJXSrAflks5tQi1OgaJpZM4R2MwY
.

Looking forward to your PR @tamarmot!
You can find a bunch of contributor documentation here: https://github.com/AnalyticalGraphicsInc/cesium/tree/master/Documentation/Contributors
Here is some info about opening a PR: https://github.com/AnalyticalGraphicsInc/cesium/blob/master/CONTRIBUTING.md#opening-a-pull-request
To run the test, run npm start to start a local server, and navigate to localhost:8080.  On that page, there is a link to run the unit tests
You don't need to npm-ify your fork.  The problem we had with our npm package was a special case.

fixed by my pull request ...

Congratulations on closing the issue! I found these Cesium forum links in the comments above:
https://groups.google.com/forum/#!topic/cesium-dev/e_osjMgHD4E
If this issue affects any of these threads, please post a comment like the following:

The issue at https://github.com/AnalyticalGraphicsInc/cesium/issues/6176 has just been closed and may resolve your issue. Look for the change in the next stable release of Cesium or get it now in the master branch on GitHub https://github.com/AnalyticalGraphicsInc/cesium.

I am a bot who helps you make Cesium awesome! Contributions to my configuration are welcome.
:earth_africa: :earth_americas: :earth_asia:


Fix typo.

Because it’s a small Craft CMS plugin. :)

Thanks! And its nice to see someone else besides me uses this plugin :)

My pleasure, thanks for sharing it! Went from "I wonder what this does..." to being annoyed when it's not installed.


Rename input_xy_quick_pressed() to input_quick_pressed_2d()

https://github.com/JujuAdams/Input/commit/d65e2114d3bcf51a06503a00cfc48cc48b70765c
https://github.com/JujuAdams/Input/commit/8c45787fb72e054727bbc9bbf2115cb01b3d6edd


Allow for service templates to have multiple VM's

https://bugzilla.redhat.com/show_bug.cgi?id=1331867
Expose the number_of_vms when building the provision request for
a service.
The number_of_vms can be set it multiple places
(1) In the Provision Request
(2) In the Single Service Item Dialog
(3) In the Service Bundle Dialog
(4) Default, allow a single vm under a service

@gmcculloug @tinaafitz
Please review

@mkanoor @gmcculloug I'm concerned about how the approval would work for multiple VMs. I'll run some tests and report back here.

@mkanoor @gmcculloug Tested service changes for multiple VMs and it works properly.
Cloud and Infra VM provisioning approval checks for the max_vms, max_cpus, max_memory and max_retirement_days.
Service approval does not have any max checking, just auto approve or not.
We should change the service provisioning approval class to have the same max values.

@mkanoor @gmcculloug what is the status of this PR?

@tinaafitz Any additional information on the approval research you mentioned here https://github.com/ManageIQ/manageiq/pull/8515#issuecomment-220394416.

@gmcculloug @mkanoor Bill Fitz and I started working on the service approval state machine.
It will need some design.

@mkanoor Please rebase

@gmcculloug
Please review

Opened issue #10145 to track quota work.

Darga Backport conflict:
$ git cherry-pick -x -m 1  47a2d33
error: could not apply 47a2d33... Merge pull request #8515 from mkanoor/multi_vm_service_template
hint: after resolving the conflicts, mark the corrected paths
hint: with 'git add <paths>' or 'git rm <paths>'
hint: and commit the result with 'git commit'

$ git status
On branch darga
Your branch is up-to-date with 'upstream/darga'.
 as appropriate to mark resolution)

	deleted by us:   app/services/service_template_fields_visibility_service.rb
	deleted by us:   spec/services/service_template_fields_visibility_service_spec.rb

$ git diff

* Unmerged path app/services/service_template_fields_visibility_service.rb
* Unmerged path spec/services/service_template_fields_visibility_service_spec.rb

Backported to Darga via #11108
$ git log
commit 1fc7a8ef5a48448c44b8d27319eb7319da7d7b6d
Merge: 41ee681 ecfc6f7
Author: Oleg Barenboim<EMAIL_ADDRESS>Date:   Thu Sep 8 12:05:19 2016 -0400

    Merge pull request #11108 from lfu/darga-pr_8515

    [DARGA] Allow for service templates to have multiple VM's.


Reduce intel targets using a variety of methods

Increase recheckInterval from 10k to 50k ticks.
Exclude rooms which can’t be reached from being targeted.
Remove oldest targets if queue gets too large.
Make sure spook clears the target lists when it switches targets.
Update pathfinding to exclude rooms that are unreachable (which should reduce requests for room intel).

This Pull Request has been merged by GitConsensus.
Vote Totals

Yes
No
Abstain
Voters

3
0
0
3

Vote Breakdown

User
Yes
No
Abstain

dragoonreas
✔

ags131
✔

tedivm
✔

Vote Results

Criteria
Result

Has Quorum
True

Has Votes
True


Allow load to read from another worker and force to local worker

The earlier change can potentially cause undesired load balance among workers.
Because it always respects the alluxio configuration instead of LocalFirst.
Also it always loads from the ufs instead of potentially from another worker when
it is already in alluxio memory. (e.g. When replication checker runs load)

Codecov Report

Merging #13466 (88a3306) into master (2ebe171) will decrease coverage by 5.04%.
The diff coverage is 74.34%.

@@             Coverage Diff              @@
##             master   #13466      +/-   ##
============================================
- Coverage     43.55%   38.51%   -5.05%
+ Complexity     8951     7784    -1167
============================================
  Files          1366     1366
  Lines         79214    79332     +118
  Branches       9628     9634       +6
============================================
- Hits          34505    30556    -3949
- Misses        41781    46219    +4438
+ Partials       2928     2557     -371

Impacted Files
Coverage Δ
Complexity Δ

.../java/alluxio/client/file/AlluxioFileInStream.java
75.34% <ø> (ø)
45.00 <0.00> (ø)

...rc/main/java/alluxio/client/file/FileInStream.java
7.14% <0.00%> (-92.86%)
1.00 <0.00> (ø)

.../alluxio/master/journal/AbstractJournalSystem.java
51.21% <0.00%> (-10.55%)
7.00 <0.00> (ø)

...in/java/alluxio/fuse/AlluxioJniFuseFileSystem.java
0.00% <0.00%> (ø)
0.00 <0.00> (ø)

...erver/src/main/java/alluxio/job/util/JobUtils.java
0.00% <0.00%> (-87.84%)
0.00 <0.00> (-13.00)

...luxio/master/journal/raft/JournalStateMachine.java
40.45% <45.45%> (+0.31%)
38.00 <0.00> (ø)

.../server/common/src/main/java/alluxio/RpcUtils.java
30.76% <62.50%> (+2.90%)
7.00 <1.00> (+1.00)

...in/java/alluxio/master/journal/ufs/UfsJournal.java
33.91% <72.72%> (+1.94%)
21.00 <2.00> (+2.00)

...a/alluxio/master/file/DefaultFileSystemMaster.java
64.63% <72.72%> (-0.03%)
290.00 <0.00> (-2.00)

...ommon/src/main/java/alluxio/metrics/MetricKey.java
97.47% <100.00%> (+0.10%)
15.00 <0.00> (ø)

... and 149 more

Continue to review full report at Codecov.

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

alluxio-bot, merge this please


Fix file path in the lodash template on windows os

Thank you for your pull request, I'll look into it today/tomorrow.

Fix in https://github.com/nuxt/nuxt.js/releases/tag/v0.5.1
Thank you!


[Bug]: Error installing the to_pdf helper app

What happened?
When I try to install the to_pdf helper app using the command in the documentation, i.e. go install github.com/danielmiessler/fabric/to_pdf/to_pdf@latest, I get the following error.
Version check

[X] Yes I was.

Relevant log output
go: github.com/danielmiessler/fabric/to_pdf/to_pdf@latest: module github.com/danielmiessler/fabric@latest found (v1.4.12), but does not contain package github.com/danielmiessler/fabric/to_pdf/to_pdf

Relevant screenshots (optional)
No response

Same!

There is small error in path
Try this -> go install github.com/danielmiessler/fabric/to_pdf@latest

@akashkankariya thank you, that was the problem 😄


how can I use this module to deploy jira with servicedesk?

I tried defining product => 'servicedesk' but it looks like the download fails.
I can see a file "atlassian-servicedesk-3.1.2.tar.gz_..." under /tmp. The correct filename seems to be "atlassian-servicedesk-3.1.2-jira-7.1.2.tar.gz".
Best regards,
Denny

Closed. I simply added ServiceDesk within the deployed Jira instance.

Hi,
I'm planning on using this module to deploy Jira Software and Service Desk. Are you still using it this way?  Any problems up to now? How did the upgrades went?
Thanks,

Hi,
Thanks for your input.  Are the upgrades of both Jira Service Desk and Jira Software done via puppet?

software gets updated via puppet and servicedesk via web-ui of jira


HACS not HCAS

Just did a PR for your readme, but you'll also have to change the filename of hcas.json to hacs.json
All the best :)

Fixed


Is there a way to record without an api key?

The api is needed to get the stream title, simply recording is also possible with the following command:
streamlink --twitch-disable-ads twitch.tv/STREAMER -o MyRecordedStream.mp4


[Refactor-696]editing imports to absolute imports in services + store+ utils files

approved, don't forget assign yourself and link issue


Your website doesn't work.

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

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

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

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

Smartphone (please complete the following information):

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

Additional context
Add any other context about the problem here.

@13554599576 I don't know what's wrong... I'll close it.


Alignment was not proper in angular

You must import "suneditor.css" and "suneditor-contents.css" or "suneditor.min.css".
Please refer this link.


nginx: preserve host information on redirect

Tested on AWS with bosh-lite.

I fixed this independently.  Thanks for the PR!


Add redirect to main page

Add a new route to the Flask application that redirects the root URL to the "apidocs/" endpoint. This will provide a more user-friendly experience when accessing the application.
The redirect is implemented using the redirect function from the Flask library, and the HTTP status code HTTPStatus.OK is returned to indicate a successful redirection.
This commit addresses the need to improve the user experience by providing a clear entry point to the application.
Refs: #N/A

All the test pass with 149-guaranteed-analysis-title-in-fertiscan


Find & Replace new-line characters does not work

VSCode Version: 1.3.0
OS Version: Window 10

Steps to Reproduce:

Open sample file with multiple lines:
a.txt

"CTRL+SHIFT+H" to find and replace "\n" with any non new-line character. Ensure that 'Use regular expression' is selected.

(The new line characters are not being replaced, rather being appended to)

@metafractal When I openend the attached file, I have noticed that it uses \r\n as a line separator:

Would searching for \r\n fix your problem in this case?

Didn't notice line ending was CRLF. This was a false alarm.
Thanks!


help for the generateMask.py

Hi ! Thank you for your great project!
Can you tell me the detaill of how to use the generateMask.py to gengeate the mask files?

You can run the code by default setting to see the generated results, ingore the "--loadSize" (seems not used here). for "--imgSize", it defineds the resolution of the generated masks, "--numbers" is the total running numbers of masks to generate, "--density" controls the percentage of the masked area, and "--res" defines the continuity of the mask areas, the lower the thicker, the higher the sparser.
Just randomly run some settings, and you will see the results :).

You can run the code by default setting to see the generated results, ingore the "--loadSize" (seems not used here). for "--imgSize", it defineds the resolution of the generated masks, "--numbers" is the total running numbers of masks to generate, "--density" controls the percentage of the masked area, and "--res" defines the continuity of the mask areas, the lower the thicker, the higher the sparser.
Just randomly run some settings, and you will see the results :).

Hi ! Thank you for your reply!
but I can't understand how to choose data to generate the mask files;
Also: Generate the file with generatemask.py first, and then run the train.py,is right?


The attachSwaggerArtifact parameter is ignored if no outputFormat is specified explicitly.

If no outputFormat is specified in the configuration, swagger-maven-plugin doesn't attach
generated swagger spec file.
ApiDocumentMojo doesn't handle this case correctly.
                if (apiSource.isAttachSwaggerArtifact() && apiSource.getSwaggerDirectory() != null && project != null) {
                    String outputFormats = apiSource.getOutputFormats();
                    if (outputFormats != null) {
...
                    }
                }

It should use default outputFormat (json) if outputFormats is null.

Thanks for this report I was looking why my artifact were not deployed and that was the reason.


Merge latest main

Please check if the PR fulfills these requirements

[ ] The commit message follows our guidelines
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
What is the current behavior? (You can also link to an open issue here)
What is the new behavior (if this is a feature change)?
Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
Other information:

Codecov Report

Merging #885 (648484a) into dev/0.8 (6f0e38c) will increase coverage by 33.37%.
The diff coverage is 50.30%.

:exclamation: Current head 648484a differs from pull request most recent head bb8ab6b. Consider uploading reports for the commit bb8ab6b to get more accurate results

@@             Coverage Diff              @@
##           dev/0.8     #885       +/-   ##
============================================
+ Coverage    10.33%   43.70%   +33.37%
============================================
  Files          242      420      +178
  Lines        11432    23096    +11664
  Branches      1707     3185     +1478
============================================
+ Hits          1181    10095     +8914
- Misses       10049    13001     +2952
+ Partials       202        0      -202

Impacted Files
Coverage Δ

packages/core/src/2d/sprite/Sprite.ts
20.65% <ø> (+19.25%)
:arrow_up:

packages/core/src/Component.ts
91.22% <ø> (+62.65%)
:arrow_up:

packages/core/src/Scene.ts
31.81% <0.00%> (+3.86%)
:arrow_up:

packages/core/src/animation/AnimationClip.ts
10.52% <0.00%> (+8.48%)
:arrow_up:

packages/core/src/animation/AnimationCurve.ts
2.66% <0.00%> (+2.10%)
:arrow_up:

packages/core/src/animation/enums/LayerState.ts
100.00% <ø> (ø)

...core/src/animation/internal/AnimationCurveOwner.ts
8.10% <0.00%> (+5.88%)
:arrow_up:

...s/core/src/animation/internal/AnimatorStateData.ts
50.00% <ø> (-50.00%)
:arrow_down:

...re/src/animation/internal/AnimatorStatePlayData.ts
15.15% <0.00%> (+10.15%)
:arrow_up:

packages/core/src/fog/Fog.ts
35.00% <ø> (+23.88%)
:arrow_up:

... and 449 more

Continue to review full report at Codecov.

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


make axios script import corrently

hmm I think we should implement something like this instead only updating particular package.


refactor: update lnd build script

Description:
enable lnd tags signrpc, walletrpc, chainrpc, autopilotrpc
remove arm from lnd build script
Motivation and Context:
Incorporate recent updates to lnd build process
Checklist:

[x] My code follows the code style of this project.
[x] I have reviewed and updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes where needed.
[ ] All new and existing tests passed.
[x] My commits have been squashed into a concise set of changes.

Coverage remained the same at 21.016% when pulling 35ec51c687b9f1070e68c682bfdd1379901290d3 on mrfelton:refactor/lnd-build-script into 41e29a1bc4faa4ab7608a934487f3311156412c4 on LN-Zap:master.


(#1201) Support newline character for text rendering

fix #1201
Can be used in #449 & #450

@ozy looks good to me! :+1: Thank you for your contribution!


Base element

Would you consider adding some documentation on the best practices concerning the order of the base element? I feel it's good to put it somewhat early because it may apply to subsequent elements but the script makes me feel like I'm doing something wrong (I might be, but would like to know why).

Good catch! Yes we should add support for base, probably somewhere as high as meta/title.


Create mapping info page placeholder

One thing that is critical is that every single page is identifiable by a URL.


BUG:vs2022 build error  yaramod-project

plain_string.cpp
rule.cpp
sections_summary.cpp
token.cpp
token_stream.cpp
yara_file.cpp
json.cpp
filesystem_operations.cpp
utils.cpp
yaramod.cpp
D:\code\hack\retdec\build\external\src\yaramod-project\deps\pog\include\pog/html_report.h(100,81): error C2001: newline in constant [D:\code\hack\retdec\build\external\src\yaramod-project-build\src\yaramod.vcxproj]
D:\code\hack\retdec\build\external\src\yaramod-project\deps\pog\include\pog/html_report.h(101,82): error C2001: newline in constant [D:\code\hack\retdec\build\external\src\yaramod-project-build\src\yaramod.vcxproj]
D:\code\hack\retdec\build\external\src\yaramod-project\deps\pog\include\pog/html_report.h(162,42): error C2001: newline in constant [D:\code\hack\retdec\build\external\src\yaramod-project-build\src\yaramod.vcxproj]
D:\code\hack\retdec\build\external\src\yaramod-project\deps\pog\include\pog/html_report.h(151,18): fatal  error C1075: '{': no matching token found [D:\code\hack\retdec\build\external\src\yaramod-project-build\src\yaramod.vcxproj]
Generating Code...

After careful investigation, here's the problem
In the wide character version of the windows operating system, the above error occurs
After my manual experiments, I manually changed the character set of html_report.h header file to utf-8,The build result compiles is successfully
Or add the following but the project structure is a bit complicated and I don't know where to add it
add_compile_options("$<$<C_COMPILER_ID:MSVC>:/utf-8>")
add_compile_options("$<$<CXX_COMPILER_ID:MSVC>:/utf-8>")
This problem affects the wide character operating system for project compilation


Allow repost of the SAMLResponse on consent

This provides a proof of concept of the possibility to repost the SAMLResponse and thus enabling us to reload the consent screen without yielding an error. See the discussion in Pivotal for more info on how I got to this POC.
@pablothedude, do you agree with the chosen solution and how this was implemented in this PR? And what could/should be improved on?
https://www.pivotaltracker.com/story/show/164244128

Great work, the only thing I'm wondering is why the completeCurrentProcedure call seems to be removed from the SPController and added to the IDPController. For me it's not clear why that should be done. Could you please explain?

The only thing I'm wondering is why the completeCurrentProcedure call seems to be removed from the SPController and added to the IDPController. For me it's not clear why that should be done. Could you please explain?

The commit message of 22eb97a should further explain this.


Add Typhoon to Kubernetes GCE installation docs

Description
Add Typhoon to GCE installation docs. Typhoon provides minimal and free Kubernetes clusters via Terraform module. It supports bare-metal, GCE, and Digital Ocean.
Clusters currently default to flannel for networking, but Calico v2.5.1 is an option on bare-metal, GCE, and Digital Ocean (DO w caveats) and runs in production. The default will be switched to "calico" soon.
Example
module "google-cloud-yavin" {
  source = "git::https://github.com/poseidon/typhoon//google-cloud/container-linux/kubernetes"

  # Google Cloud
  zone          = "us-central1-c"
  dns_zone      = "example.com"
  dns_zone_name = "example-zone"
  os_image      = "coreos-stable-1465-6-0-v20170817"

  cluster_name       = "yavin"
  ssh_authorized_key = "ssh-rsa AAAAB3Nz..."

  asset_dir = "/home/user/.secrets/clusters/yavin"
  networking = "calico"
}

Reviewer
Should this change be made in the v2.5 folder or do those not get republished often?

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

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

@dghubble thanks! This LGTM.  You can add to v2.5 if you like and it will be published once the PR is merged.
Over to @emanic for docs review.

Added to both master and v2.5


bug(rule): rule KMS Key With Full Permissions is creating false alarms

I believe there is a bug in the KMS Key With Full Permissions rule.
A previous issue brought this up, but I don't think the right conclusion was reached.
KMS keys frequently have a policy that allows IAM to manage access to keys.   This policy would generally look something like this:
{
    "Version" : "2012-10-17",
    "Statement" : [
      {
        "Sid" : "Enable IAM User Permissions",
        "Effect" : "Allow",
        "Principal" : {
          "AWS" : "arn:aws:iam::ACCOUNTID:root"
        },
        "Action" : "kms:*",
        "Resource" : "*"
      }
   ]
}

This policy statement includes the action kms:*, which is triggering this rule.  Despite the fact that it has the string kms:* in the policy, it is not actually granting any KMS permissions to any IAM identities (because the only principal listed is the account principal - the :root principal), and hence, does not represent a security vulernability.
From the documentation:

When the principal in a key policy statement is the account principal, the policy statement doesn't give any IAM principal permission to use the KMS key. Instead, it allows the account to use IAM policies to delegate the permissions specified in the policy statement. This default key policy statement allows the account to use IAM policies to delegate permission for all actions (kms:*) on the KMS key.

Crafting these kinds of policies to avoid a KICS error is not practical, as you have to enumerate all KMS actions in the policy.
Can this rule be modified so that it only triggers on kms:* if there are principals other than the account principal?
In addition, it seems that if the key is defined without a policy, KICS will trigger this error.  In the case of a key with no policy, AWS creates a default policy that looks much like the one above.  So a key with no policy is not a security risk, either.

We have disabled this rule internally as the way the check is handled it will always fail if the default statement is included. It's unfortunate it's handled this way as it has the potential to hide actual overly permissive statements further down in the policy.


Notification not working when the App is Closed in Android

Hi,
Right Now, when the app is closed, we cannot receive the notification.
Please note that:

The app is not in battery saving mode
The app is not force closed, i just swipe out from the active application list.
the app is connected to the internet
I have used your TEST NOTIFICATION tool, too, and not working during the app is closed (and we able to receive when the app is opened)
My account currently has many pending notifications.

Hi @robert589,
Nice to connect with you. My name is John and I'd be glad to help you investigate this issue further.
If you could please provide the following information about the device experiencing this issue, that would be great:

Device manufacturer
Device model name
Android OS version

Also, please consider whether the device is connecting to the Internet through a restrictive firewall which may be blocking the device from connecting to our servers.

We have tried to use in different device, internet, and location. so we are quite sure not because of the firewall
You can try in this phone but actually it happens in all android phone we try

XiaoMi
Redmi 5 plus (MEG7)
8.1.0

Hi @robert589,
Chinese-based devices such as Huawei and Xiaomi may not receive notifications after your app is swiped away from the recent apps menu, goes to background, or after a device reboot due to custom power saving features which affect Firebase Cloud Messaging as well.
Most notably, Chinese devices from the following manufacturers include custom power-saving restrictions which may hinder background notification delivery for your app:
Xiaomi
Huawei
OnePlus
Samsung
Oppo
Vivo
Asus
Sony
Honor
Unfortunately, since these power saving mechanisms also affect FCM, it is not a Pushy-specific issue, and the only solution is to instruct the user to whitelist your app from the respective manufacturer's power saving mechanism in the Settings. The instructions for doing this vary from one manufacturer to another.
Read more about this issue:
https://dontkillmyapp.com/problem
We apologize for this inconvenience introduced by Chinese device manufacturers.

but we can receive other app notifications, btw, i did whitelist the app from the battery and performance setting.
Could you try the app the I have sent from the email and u try to whitelist your self?

Hi @robert589,
This behavior is most likely caused by Android's new power-saving optimizations (Doze mode / Background Execution Limits) which aim to conserve battery life by restricting background services for your app on Android O and newer.
Apart from the whitelisting your app from the respective Chinese manufacturer-introduced power saving optimizations, you will need to whitelist your app from the built-in Android battery optimizations to receive notifications when your app goes to background.
You can manually verify that this is indeed the cause of the issue you are experiencing by exempting your app from Android battery optimizations as follows:
Find your app in the app launcher -> tap & hold and go to App Info -> Battery -> Battery Optimization -> Don't Optimize.
Then, try to reproduce the behavior you have been experiencing.
To send notifications to devices with Doze / Background Execution Limits, your app can programmatically check whether it is currently exempt from battery optimizations using the PowerManager.isIgnoringBatteryOptimizations() API, and if not, display a dialog asking the user to exempt your app from battery optimizations if they wish to receive notifications while your app is in the background.
Please let me know if you have any questions.

sorry, it also does not work in IOS

i think we need to think why the notification only receives when we open the app. what is the scenario that this can happen.

I have tested using your demo https://github.com/pushy-me/pushy-demo-react-native
it is also not working when using your demo, you could investigate on that

i think we need to think why the notification only receives when we open the app. what is the scenario that this can happen.
I also have said previously that the android has been whitelisted from battery optimization

Please read our response again regarding Android. There are two different power saving optimizations on Chinese devices that require whitelisting your app. Please follow the instructions in this comment and try again:
https://github.com/pushy-me/pushy-react-native/issues/24#issuecomment-515215073
As per iOS, maybe you are trying to send a notification to the wrong device token. What is the device token you are trying to send to?

OK, I just found a solution
I install this library https://github.com/stepsetgo/react-native-autostart
and put it into the App.js
    if (AutoStart.isCustomAndroid()) {
        AutoStart.startAutostartSettings();
    }

the problem is the app's autostart is not enabled

Hi @robert589,
This is indeed the Chinese manufacturer power saving feature we were referring to in this comment:
https://github.com/pushy-me/pushy-react-native/issues/24#issuecomment-514872317
For every manufacturer, the name is different, therefore we did not list every single name. But they all require your app to be whitelisted.


Karma test fails to initialize when using reexported types

Hi,
In my project I have a structure like
/models/Foo.ts
/models/Bar.ts
/models/models.ts

In models.ts I do something like
import {Foo} from './Foo';
import {Bar} from './Bar;
export * from './Foo';
export * from './Bar';

to give a single import point for the rest of the code to all model types.
Now, I am writing a test for a service, that includes a model, like so:
`
import {Foo} from '../../models/models';

@Injectable()
export class TestService {
...
}
`

And the associated test:
`
import {TestService} from './TestService';

describe('TestService', () => {

	beforeEachProviders(() => [
		BaseRequestOptions,
		MockBackend,
		TestService,
	]);

	it('should be injectable', inject([ TestService ], (x) => {
		expect(true).toEqual(true);
	}));
});
`

Running karma on this gives an error with:

Chrome 48.0.2564 (Mac OS X 10.11.2) LOG: 'TypeError: Cannot read property 'toString' of undefined'
Chrome 48.0.2564 (Mac OS X 10.11.2) ERROR
Uncaught TypeError: Cannot read property 'getOptional' of undefined
at /Users/akj/Documents/angular2/angular2-webpack-starter/spec-bundle.js:22991 <- webpack:///angular2/src/core/application_ref.ts:292:2

(same for PhantomJS)
However, if I change my include in TestService.ts to read
import {Foo} from '../../models/Foo';
then the test runs.
What could cause this behaviour?
During normal runtime, everything works with the /models/models import. It only fails during test.

Seemed to be an errant binding, in the app scope.
I had declared an empty binding array in models.ts, which was included in the global bootstrap bindings.
When it (the array) is present, the testsuite fails to start. If absent (removed from bootstrap.ts), the test-suite works.
It seems to not matter, if the array contains anything or not.
I am very confused.


Fix Typescript definition of constructor arguments

TS uses type[] to denote arrays rather than Rusts [type]

Oops, thanks!


Chrome: encrypted Widevine VoD stream fails, clear is good

Hello,
Using shaka player to steram VoD video content. So, playback of clear stream without encryption is good. The same stream encrypted by DASH-CENC(Widevine) fails with the next errors (log from shaka player):
...
"message": "Shaka Error MEDIA.VIDEO_ERROR (3,)"
...
"message": "Shaka Error STREAMING.INCONSISTENT_BUFFER_STATE (video)"
...
"message": "Shaka Error MEDIA.MEDIA_SOURCE_OPERATION_THREW (InvalidStateError: Failed to execute 'appendBuffer' on 'SourceBuffer': The HTMLMediaElement.error attribute is not null.)"
Full log: https://drive.google.com/file/d/0B1bdskC7PG1WNzZONi1nNWJDYWs/view
Logs from chrome://media-internals:
Player Properties
audio_decoder	FFmpegAudioDecoder
debug	DecryptingDemuxerStream: no key
duration	1799.56
error	media::MediaSourceState::Append: stream parsing failed. Data size=131072 append_window_start=0 append_window_end=1799.63
event	WEBMEDIAPLAYER_CREATED
found_audio_stream	true
found_video_stream	true
info	Video codec: avc1.42E01E
pipeline_error	pipeline: decode error
pipeline_state	kStopped
player_id	9
render_id	727
url	blob:http%3A//shaka-player-demo.appspot.com/a736cb8b-4f96-4454-9bfe-1e85240ca912
video_codec_name	h264
video_dds	false
video_decoder	DecryptingVideoDecoder
Log
Timestamp	Property	Value
00:00:00 00	pipeline_state	kCreated
00:00:00 00	event	WEBMEDIAPLAYER_CREATED
00:00:00 00	url	blob:http%3A//shaka-player-demo.appspot.com/a736cb8b-4f96-4454-9bfe-1e85240ca912
00:00:00 00	pipeline_state	kInitDemuxer
00:00:01 122	duration	1799.56
00:00:02 48	info	Audio codec: mp4a.40.2. Sampling frequency: 44100Hz. Sampling frequency(Extension): 0Hz. Channel layout: 3.
00:00:02 48	found_audio_stream	true
00:00:02 48	audio_codec_name	aac
00:00:02 113	info	Video codec: avc1.42E01E
00:00:02 113	found_video_stream	true
00:00:02 113	video_codec_name	h264
00:00:02 114	pipeline_state	kInitRenderer
00:00:02 307	debug	DecryptingAudioDecoder: failed to init decoder on decryptor
00:00:02 310	audio_dds	true
00:00:02 310	audio_decoder	FFmpegAudioDecoder
00:00:02 312	debug	Video rendering in low delay mode.
00:00:02 336	video_dds	false
00:00:02 336	video_decoder	DecryptingVideoDecoder
00:00:02 336	pipeline_state	kPlaying
00:00:02 338	debug	DecryptingDemuxerStream: no key
00:00:02 995	error	Failed to prepare video sample for decode
00:00:02 995	error	media::MediaSourceState::Append: stream parsing failed. Data size=131072 append_window_start=0 append_window_end=1799.63
00:00:02 995	pipeline_state	kStopping
00:00:02 995	pipeline_state	kStopped
00:00:02 995	pipeline_error	pipeline: decode error
Full log: https://drive.google.com/file/d/0B1bdskC7PG1WVHF2RTRpVEhVTlE/view
Also I can share privately link to the manifest & license server.
Thank you!

This looks similar to a DRM filtering bug we fixed incidentally in a644900.  Can you try again with the latest from master and let us know if that fixed your issue as well?

@karlovich Is this issue still affecting you?  We believe it should be fixed a while ago.  Please let us know.

@joeyparrish Seems that problem was on Content provider side. After some changes on configuration of CENC encrypted MPEG-DASH VOD streams shaka 2.0,0-beta2 was able to stream without this error. Thanks for your response, please close the issue.

Thanks!


visit http://demo.simplcommerce.com/ error

visit http://demo.simplcommerce.com/ error

@freemsly the site is up. The issue happened because someone create a brand with a special name "หอยไทย", the system try to create a url for that name which return an empty string. It relate to #155


zeroctl copy vdisk does not complain when storage types do not match

version
zeroctl version
Version: 1.1.0-beta-1
Build: ef49d8a 2017-10-22T13:14:48-0500
Runtime: go1.9.1 darwin

(master)

how to reproduce
config:
storageClusters:
  a:
    servers:
      - address: localhost:16380
      - address: localhost:16381
      - address: localhost:16382
  b:
    servers:
      - address: localhost:16383
      - address: localhost:16384
      - address: localhost:16385
vdisks:
  foo:
    type: boot
    blockSize: 4096
    size: 10
    nbd:
      storageClusterID: a
  bar:
    type: db
    blockSize: 4096
    size: 10
    nbd:
      storageClusterID: b
      templateStorageClusterID: a

create foo vdisk:
$ nbdserver --address=":6666" --protocol="tcp"
# ...
$ qemu-img convert -p -n myimg.img -O nbd nbd+tcp://localhost:6666/foo
    (100.00/100%)

copy foo vdisk as bar
$ zeroctl copy vdisk foo bar b -v
INFO[10-22|17:05:18] parsing positional arguments...          module=global caller=vdisk.go:49
INFO[10-22|17:05:18] collecting all metadata from source vdisk "foo"... module=global caller=deduped.go:437
INFO[10-22|17:05:18] collected 10 meta indices from source vdisk "foo" module=global caller=deduped.go:447
INFO[10-22|17:05:18] buffering 10 meta indices for target vdisk "bar"... module=global caller=deduped.go:461
INFO[10-22|17:05:18] flushing buffered metadata for target vdisk "bar"... module=global caller=deduped.go:468
INFO[10-22|17:05:18] collecting all metadata from source vdisk "foo"... module=global caller=deduped.go:437
INFO[10-22|17:05:18] collected 10 meta indices from source vdisk "foo" module=global caller=deduped.go:447
INFO[10-22|17:05:18] buffering 10 meta indices for target vdisk "bar"... module=global caller=deduped.go:461
INFO[10-22|17:05:18] flushing buffered metadata for target vdisk "bar"... module=global caller=deduped.go:468
INFO[10-22|17:05:18] collecting all metadata from source vdisk "foo"... module=global caller=deduped.go:437
INFO[10-22|17:05:18] collected 9 meta indices from source vdisk "foo" module=global caller=deduped.go:447
INFO[10-22|17:05:18] buffering 9 meta indices for target vdisk "bar"... module=global caller=deduped.go:461
INFO[10-22|17:05:18] flushing buffered metadata for target vdisk "bar"... module=global caller=deduped.go:468

expected result
This is wrong however, as even though the copying went successful, the bar vdisk will never be able to boot up as it is of storage type non-deduped, while the original (foo) vdisk is of storage type deduped.
It would be great to have the command instead fail, with as error that the source- and target- storage types do not match.

The reason the current behavior is like that, is because the target vdisk's config is not checked. This is also why the third positional argument is there (target_clusterid). Therefore I propose that we'll always load the target vdisk's config such that we both can validate that the storage type compatibility, as well as eliminate the third positional argument. This would mean the command would now have the following signature instead:
$ zeroctl copy vdisk source_vdiskid target_vdiskid


Please add recipes for "Diphenhydramine" and "Histamine"

Hello, I kindly ask you to add recipes or a method of extraction for histamine and Diphenhydramine, this will greatly help doctors.

Just for sure,  Diphenhydramine is a posion medicine healing that heals 2 poison damage more than dyloven.
Histamine is just allergic thing, you can get it from uh emagges booze dispenser. You can also make a medicine from it, Ultravasculine, that heals toxic damage, instead of specifec poison. But it deals brute damage when used, overdose just make damage bigger.

Наверняка, Дифенгидрамин — это лекарство от яда, которое лечит на 2 урона ядом больше, чем диловен. Гистамин вызывает аллергию, его можно получить из дозатора выпивки. Вы также можете сделать из него лекарство, Ультраваскулин, которое лечит токсические повреждения вместо конкретного яда. Но при использовании он наносит грубый урон, передозировка только увеличивает урон.

Can you tell us a little more about histamine production? Thank you

We added anti-histamine on DV. Histamine would probably be something you extract from plants.

Here, I had to get some coffee first and wake up a bit but I copied it over. #23006
You can extract histamine from nettles already.  Not sure what else would potentially produce it but there is currently at least the one source.

You can also get them from various science things.

I agree a recipe should be available, additionally I think histamine would be very interesting if it were implemented in more thorough way throughout. Just one example; histamine can be released from injury/damage which also induces and insinuates that pain is present.  It may be an interesting mechanic, if pain is ever a thing along with morphine etc, to keep histamine's OD levels in addition to having it as a recipe and generating from pain as well as other sources? Just a thought, as it pertains to balance..


vertica-python on Windows

it seems that the package imports crypt which is not supported on Windows.
Any way around this ?

Try to check #46 and #44

pulled the version from pypi - which is 3 month old.

just pushed master as v 0.3.6 to pypi


Support for float type is missing

Support is required for:

0x3E FLT8TYPE
0x6D FLTNTYPE

Support added for:

0x3B FLT4TYPE
0x3E FLT8TYPE
0x6D FLTNTYPE


Cant drag sounds or music onto scene

https://gyazo.com/bf12a7fd2f2fcd7b7740beda3f65f495
occurs:

0.3.0
0.3.1
0.3.2

This is intentional, you need to mark directory as sounds root first. See https://github.com/kotcrab/vis-editor/wiki/Marking-Assets-Directories


chore(release): create new release

Coverage remained the same at 89.787% when pulling 139ff326088b2f89644190c15b30ffa088b81f09 on chore/release-3.1.0 into 80e21c8cd9210e10e2479b18c16bf528856192c7 on master.


a link in tag is broken

🆖 https://krymtkts.github.io/pages/2020-03-31_oh-my-posh-theme.html
🆗 https://krymtkts.github.io/posts/2020-03-31_oh-my-posh-theme.html

2020-03-31_oh-my-posh-theme.html is incorrect post file name.
2020-03-31-oh-my-posh-theme.html is expected.

> ll -Recurse .\contents\posts\**\*.md | ? -Property Name -Like *_* | select -Property Name

Name
----
2019-02-24_lein-ring-server-error.md
2019-03-21_want-to-run-qunit-in-cli.md
2019-03-24_coverage-of-qunt-puppeteer.md
2019-04-20_disable-touch-screen.md
2019-06-02_install-shfmt-to-win-using-cli.md
2020-03-31_oh-my-posh-theme.md


Package "react-native-sqlite-storage" is using deprecated "rnpm" config that will stop working from next release. Please notify its maintainers about it.

After upgrading ReactNative to 0.60, the prompt can not load rpm, the current database can not be used

Yeah, I'm experiencing the same Issue with React Native 0.60.

Experiencing the same issue, can't start my project when using react-native-sqlite-storage. Is there a work around ?

PR fixing the issue is welcome.

@andpor I did created a PR for solve it, please test for me haha

@feedcast-me we are discuss it on #363


Reservation Modal - Cart Contains Old Items

Some bug is present that is causing a new cart or new reservation to appear with old lineitems in it.    This is really bad.

Closed as duplicate


About the design of Cannon

One promise of L2 rollup is that one can recover L2 state purely from L1, if you only submit header to L1, it will break this promise IMO.

Hey zhiqiangxu! Thanks for your input!
Currently Cannon can only challenge L1 blocks on the L1 blockchain. The next step in its evolution is to make it challenge L2 block transitions instead. This work is scheduled to start very soon :)


Tests under Yotta

I'm currently working through writing unit tests for a few Yotta modules and I'm trying to figure out the best way to stub or mock out dependencies under Yotta. Has anyone else wrestled with this and found a good solution? The json configuration files do not appear to support replacing dependencies when building tests. I also don't see any flags set (by default) which indicate that the current module being built is a library with tests present.

we use greentea for all microcontroller testing. github.com/armmbed/greentea

@BlackstoneEngineering I'll have to take a look at that. We aren't using yotta under mbed but it would be interesting to see how the tool works. Do you use it for unit testing and integration testing?

Yes, and its not tied to mbed. Greentea is based on Utest and really just adds some extra bits for automatically loading programs onto microcontrollers and returning the results across the serial channel.

@plauche, is this issue resolved?

@thegecko, yes, we were able to come to a satisfactory solution using additional .cmake files under the test folders to get the linker to create function stubs.


Show source of privilege

Let's say this tool has told you that a user has some unused privilege. The next thing you'll want to know is why the user has that privilege in the first place, especially if there is potentially a condition or just if this user is a member of many groups and this is due to an attached policy within that group, or if the privilege was granted with wildcards which will make it tougher to grep for. So we should know the source of these privileges.

Samesies, I'd like to help


Detect connection state changes?

In the previous SignalR client, I could subscribe to connection state changes using:
$.connection.hub.stateChanged
Is there something similar in the new (preview2) client? Or do I have to do some kind of polling to get that information?

The state diagram for the client right now is much simpler. The start Promise lets you know when the connection starts and the closed event lets you know it's stopped. We don't have automatic reconnects, so those are the only state transitions.

Thanks for that. Been testing with preview2 in a .NET Core 2.0 application and am very pleased with the performance. Looking forward to the real-deal release. :)

How can i get close connection event?
var connection = new signalR.HubConnection('/chat', { logging: signalR.LogLevel.Trace });
connection.onclose = function(){
   console.log('connecition closed');
}
connection.start().then(function(){
   console.log('connection started');
})

im having difficult to understand it.

 This is so that you can register multiple callbacks. So this modification to your code should work:
var connection = new signalR.HubConnection('/chat', { logging: signalR.LogLevel.Trace });
connection.onclose(function(){
   console.log('connecition closed');
});
connection.start().then(function(){
   console.log('connection started');
})


feat: checkPluginVersion.sh bump plugins for 1.4.0 release

feat: checkPluginVersion.sh bump packages/cli to 1.19.0 in main
Signed-off-by: RHDH Build (rhdh-bot<EMAIL_ADDRESS>

feat: checkPluginVersion.sh bump plugins/aap-backend to 2.3.0 in main
Signed-off-by: RHDH Build (rhdh-bot<EMAIL_ADDRESS>

feat: checkPluginVersion.sh bump plugins/audit-log-node to 1.8.0 in main
Signed-off-by: RHDH Build (rhdh-bot<EMAIL_ADDRESS>

feat: checkPluginVersion.sh bump plugins/kiali to 1.35.0 in main
Signed-off-by: RHDH Build (rhdh-bot<EMAIL_ADDRESS>

feat: checkPluginVersion.sh bump plugins/kiali-backend to 1.21.0 in main
Signed-off-by: RHDH Build (rhdh-bot<EMAIL_ADDRESS>

feat: checkPluginVersion.sh bump plugins/kubernetes-actions to 2.3.0 in main
Signed-off-by: RHDH Build (rhdh-bot<EMAIL_ADDRESS>

feat: checkPluginVersion.sh bump plugins/scaffolder-annotator-action to 2.3.0 in main
Signed-off-by: RHDH Build (rhdh-bot<EMAIL_ADDRESS>

feat: checkPluginVersion.sh bump plugins/shared-react to 2.14.0 in main
Signed-off-by: RHDH Build (rhdh-bot<EMAIL_ADDRESS>

feat: checkPluginVersion.sh bump plugins/web-terminal to 1.12.0 in main
Signed-off-by: RHDH Build (rhdh-bot<EMAIL_ADDRESS>

closed, trying to regenerate with actual package.json updates...


[Refactor] Remove radix toast dependencies and related components

#3808  👈
main

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

Join @jnsdls and the rest of your teammates on  Graphite


Consumer logging tweaks

Include the partition number when logging offsets, as the offset isn't meaningful otherwise.
Clean up the Consumer repr so that it has fewer sigils. Remove the processor function as it isn't usually useful information but tends to be quite long. I picked the convention of {topic}/{partition} as / isn't valid in a topic name. For example, mymessages/0 for partition 0 of topic "mymessages". This will extend nicely once we have a multi-partition consumer to something like mymessages/0,1,2.

Merging per https://github.com/ciena/afkak/pull/104#pullrequestreview-290330567


Refactor: quick clean-up of iteration idioms in the vfs crate

This PR cleans up some of the iteration idioms used in the vfs crate. Most of the changes simply converted for loops into their std::iter::Iterator-method counterpart. Other changes required some inversion of logic to accommodate for better short-circuiting. Overall, there should be no behavioral changes. If there are any stylistic issues, I will gladly adhere to them and adjust the PR accordingly. Thanks!

I'd say most of these were written that way intentionally. The .any is a good catch though.

Thanks for the constructive feedback! I must agree that the usage of ? does seem like I've used it just because. I will definitely revert it back to the original.
Regarding the filter_map, I did not realize (until now) how powerful that separation was from a readability standpoint. It had never occurred to me that "using less powerful constructs" expressed the intent of the code more clearly and effectively. Previously, my intention behind the filter_map was that it seemed really intuitive—and dare I say, tempting—to collapse all filter + map operations into one big filter_map. As you have pointed out, this is not always the best course of action.

I've tried to spell out rationale for our code style in https://github.com/rust-analyzer/rust-analyzer/blob/master/docs/dev/style.md#functional-combinators

This is definitely a helpful document. Thanks for taking the time to write it up! The part about "bending the code to fit into some combinator" is definitely most relevant in this PR. 😅

Hello there @matklad! Thanks again for your helpful feedback. I have addressed the stylistic issues you raised. I have also added the code comments you wished were present in Directories::contains_file. Please do let me know if there are any other changes that must be made.

bors r+


rename md-card "title" property

If it's used with string interpolation, the element's native tooltip becomes: ${expression}.

I'm running into this very problem with 1.2.11.   and ${something} is appearing on hover of title.


Use d3 paths to prevent vertical edges from overlapping?

If there are multiple vertical edges corresponding to the same sample collection date, then we can apply some curvature to prevent them from overlapping.
Maybe we can have this be animated, so if the user selects the derived variant bead (child node) then the vertical edge curves to the left.

I think this is mostly resolved with rescaling by window size #132 and filtering edges by genomic distance #255


kylin#4280  segmenPruner support or

Can one of the admins verify this patch?

Can one of the admins verify this patch?

retest this please

Codecov Report

Merging #975 into master will decrease coverage by <.01%.
The diff coverage is 79.41%.

@@             Coverage Diff              @@
##             master     #975      +/-   ##
============================================
- Coverage     25.43%   25.43%   -0.01%
- Complexity     6148     6158      +10
============================================
  Files          1414     1415       +1
  Lines         85698    85714      +16
  Branches      12021    12026       +5
============================================
  Hits          21801    21801
- Misses        61776    61787      +11
- Partials       2121     2126       +5

Impacted Files
Coverage Δ
Complexity Δ

...va/org/apache/kylin/cube/common/SegmentPruner.java
61.29% <100%> (+0.76%)
24 <0> (-1)
:arrow_down:

.../org/apache/kylin/cube/common/TupleFilterNode.java
76.66% <76.66%> (ø)
11 <11> (?)

...rg/apache/kylin/cube/inmemcubing/MemDiskStore.java
66.56% <0%> (-4.26%)
7% <0%> (ø)

...che/kylin/stream/core/storage/CheckPointStore.java
63.63% <0%> (-3.04%)
21% <0%> (-1%)

...c/main/java/org/apache/kylin/cube/CubeSegment.java
50.44% <0%> (+0.89%)
64% <0%> (+1%)
:arrow_up:

Continue to review full report at Codecov.

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

Pull Request Test Coverage Report for Build 5334

31 of 34 (91.18%)  changed or added relevant lines in 2 files are covered.
11 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.009%) to 27.918%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

core-cube/src/main/java/org/apache/kylin/cube/common/TupleFilterNode.java
27
30
90.0%

Files with Coverage Reduction
New Missed Lines
%

stream-core/src/main/java/org/apache/kylin/stream/core/storage/CheckPointStore.java
2
73.74%

core-cube/src/main/java/org/apache/kylin/cube/inmemcubing/MemDiskStore.java
9
76.29%

Totals

Change from base Build 5320:
0.009%

Covered Lines:
23930

Relevant Lines:
85714

💛  - Coveralls


Establish format for metadata associated with each scan

A scan, which will contain amplitude and phase data has to have some metadata indicating the configuration state around it. I wanted to get people's thoughts on what has to be included. Here's a starting point from our own data:
avg_power
bandwidth
fit_theta
freq
min_avgs
points
power
size
span
span_factor
stitch_power
time_out
@cary-earnest @crmcrae @MartinWeides suggestions for other parameters to include? Things like temperature would be great to have, but perhaps as optional fields?

This seems useful. Since VNA sweeps are (always?) upward in frequency the direction of scan may not need to be included. Maybe incluede temperature?

@joshmutus can you explain more what each of those things are? e.g what is stitch_power? wouldn't the actual number of averages be more useful than the min number of averages? The number of points taken are self explanatory from the data length, is it useful to record that separately?  What is the difference between size/span/spanfactor?

@cary-earnest all good questions. I just pasted something from a dataset. I'll figure out what's going on here...

From Jeremy's email yesterday, it's clear that we need to establish a way to report the data format and units, as well as any preprocessing that may have been performed. We should add this as soon as possible to ensure that the community survey of resonator fits is accurate.

Yes. I sent Cary and Jeremy an averaged dataset to see if that is more suitable. This is a good exercise to see all the assumptions we have about the data

Hi all, it would make sense to have all of the input data in the same format, to avoid having everyone replicate the data input routines for all the different datasets.
It seems simplest to have the data be in the format that you end up feeding into the fitting algorithm, post averaging & any normalization/calibration. For us, that would be a single "trace", with the line attenuation calibrated away. Whether the data is complex or mag/phase is not an issue since it is trivial to convert.
The issue of normalization/calibration is important though, and maybe can be explored separately from the actual fitting. I also realize that it is possible that some groups might not pre-calibrate, and instead include the line attenuation directly into the fit. This would throw a wrench into the above.

Just chiming in to say that Leo Ranzani at BBN has also contacted me with issues fitting the IQC data due to the frequency shift. I'm copying Leo's comment below:
"We couldn't fit the Waterloo resonator, because it looks like the frequency axis has been shifted to center the resonator at 0Hz. If Matteo can provide the center frequency we could try again though (otherwise the answer is easy: Q factor is zero :)"

Hi all,
Due to community feedback about issues with fitting a variety of data formats, I went through the datasets we have so far and compiled guidelines for a standard data format. Going forward, we hope that everyone can share their data in the following format:
Three column .csv file starting in row 1, with no empty rows between data.
Data should be the raw data with no normalization or frequency shift.
Column 1:
Monotonically increasing frequency in GHz
Column 2:
Magnitude of S21 in dB (log mag)
Column 3:
Phase of S21 in radians
Other info to be included in a .txt accompanying the data (encouraged but not required):
Input power (dBm)
Number of averages
IFBW
Fridge temperature (mK)
Type of device (materials, design geometry, etc.)
Please let me know if you have any questions or feedback about the guidelines. We're still in the early stages of this effort so any comments would be very helpful.
The NIST and Google data should be in the correct format already, and the GitHub will be updated when other datasets are reformatted.
Thanks!

Oh and I'll close this thread and make a new thread to discuss changes and steps going forward with the standard data format.

The Keysight Touchstone data format doesn't include any metadata associated with the scans. How should we go about including this? What do our metadata files look like now @mullinska ?

Two reasonable options:

Use a format like netcdf4 that allows metadata and then provide methods
to read and export touchstone files. The xarray package would help you do
this seamlessly. This would also let you store sweeps of temperature,
power, magnetic field, etc in a native highly compact binary format.

Put a JSON string with metadata as the first comment line in the
touchstone file. Then just read the first line in to a dict on load prior
to parsing the rest of the file with scikit-rf (no need to reinvent the
wheel).

Some combination of these is probably the best. Allow for import of
touchstone, but natively store in netcdf4. If someone wants to export to
touchstone, include metadata as a comment.
On Fri, Feb 7, 2020, 19:23 Corey Rae McRae<EMAIL_ADDRESS>wrote:

The Keysight Touchstone data format doesn't include any metadata
associated with the scans. How should we go about including this? What do
our metadata files look like now @mullinska https://github.com/mullinska
?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Boulder-Cryogenic-Quantum-Testbed/measurement/issues/12?email_source=notifications&email_token=ACTQRRDQRPCHLX2JWEHWACLRBYQRTA5CNFSM4GXQ2FM2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELFIJYY#issuecomment-583697635,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACTQRRB4X62J4YJUEKERG2LRBYQRTANCNFSM4GXQ2FMQ
.

@crmcrae The current metadata files (and by that I assume you mean the file that prints the conditions for the sweep ran), looks like this:
STARTPOWER: X dB
ENDPOWER: X dB
NUMSWEEPS: X
CENTERF: X GHz
SPAN: X MHz
TEMP: X mK
STARTING AVERAGES: X
EDELAY: X ns
IFBAND: X kHz
POINTS: X


Write port for libretro

This looks like some awesome work, thanks! Doesn't seem like it breaks any existing functionality so I'll go ahead and merge it, though I'm not familiar with libretro myself. If you ever get a chance, I'd appreciate it if you update the README with some brief instructions or other details pertaining to libretro.

This looks like some awesome work, thanks! Doesn't seem like it breaks any existing functionality so I'll go ahead and merge it, though I'm not familiar with libretro myself. If you ever get a chance, I'd appreciate it if you update the README with some brief instructions or other details pertaining to libretro.


Why parallelize?

@:parallel(FULL) // | @:p(FULL) - Valid values FULL | DOUBLE | HALF | # - Will create threads to call this function in parallel according to the number specifed in the parameters.  Will collect all threads before continuing.
@:bucket(5) // | @:b(5) Specifies the parallel bucketing size valid values MAX | # - Max will take total / threads
@:fork(SPRITE_UPDATE) // - Named synchronization - Will split off this update in another thread and continue processing other updates, can be combined with @:parallel
@:u inline function updateSpritePosition(spr:Sprite, pos:Position) {
  spr.x = pos.x;
  spr.y = pos.y;
}

// PARALLEL API NOT IMPLEMENTED YET
@:join(SPRITE_UPDATE) // - Will wait until the corresponding fork is completed before running this function
@:u inline function afterSpritePositionsUpdated() {
  // rendering, etc
}

In my opinion, threads are best used for large, self-contained tasks. Things like loading a large file, or solving an equation. Something that doesn't involve updating memory that's also in use by the rest of the program. An ECS is pretty much the opposite of that. Everything is broken up into miniature tasks, and these tasks constantly access and modify memory.
Just look at updateSpritePosition: in the span of two lines it reads two values and writes two more. What if something else also wants to update those values, and thanks to @:fork, they happen at the same time?
Plus, I'm concerned it'll be too many fork/join operations. Once in a while is fine, but every single frame? Potentially multiple per frame? I wouldn't trust that to perform well across devices and targets. Maybe on good computers in pure C++, but in HashLink, message-passing seems to incur a more significant penalty. Disclaimer: I was testing thread performance in general, not this specifically. But it did seem that there was a fixed delay at the start of each job, regardless of whether I used Deque or Thread.readMessage().

That's a great question. I designed the API before looking at HashLink's thread overhead. The use case is primarily for large pools of high computations like ray tracing, collision or physics.
I hope to find a thread model that has the low overhead I'm used to in C++.  There's some haxe libraries for concurrency but I wonder if they are built on the questionable fundamentals.
The examples below are just toy examples to demonstrate the api.
Based on your comments, I will definitely do some profiling before implementing the final API.
The fork/join was a thought experiment of trying to define an API that relates the thread containment to the core program flow. Since the systems are already implicitly defining program flow, it seemed natural to embed the parallelism within it. Keeping them more obvious and a little more defined felt like these constraints would ultimately allow you to build more sophisticated systems without running into a lot of the regular synchronization issues. Ie you are effectively claiming ownership of the entities / components you are operating on so it's best to weave it into the system declaration you avoid conflicts.
The joins also provide a simple way of doing dependency chains where multiple systems can join on the same identifier, which may be a non-standard way to use join tbh.
Great question. Thanks for the insight.

I'm not sure what else to say besides, good luck with all that. I've personally had enough threads for a long while.

I've been looking into the threading issues and I've just found a bunch of discussion about how bad it is.  Do you have any hard data anywhere?

No, sorry. You can check out my two side-by-side HTML5 implementations (top is single-threaded, bottom is multi-threaded, hover to see how long a job took). However, those come with a lot of caveats, including that they were designed to measure the performance of large jobs, not message-passing. And certainly not passing several messages per frame.

I think we've hashed this through.


Memory leaks

Opening this issue because I dug into it a bit, but don't have more time this week.
iD leaks memory pretty bad.  This relates to #2743 - performance as a user makes many edits - but the causes and solutions are different.
We expect iD's memory usage to grow in certain ways as a user edits:

Scrolling the map downloads more entities from OSM which get added to the base graph.
Each edit makes a new immutable Graph, and they get stacked up in history.
Changing an Entity makes a new immutable Entity.
Caches (Graph's parentWays/parentRels/childNodes, History's rbush, Features' internal cache)

But we expect memory to be garbage collected at other times.  D3 should remove() DOM elements from exit selections as things are no longer needed. This applies to:

SVG elements on the map
Everything in the inspector / entity editor / sidebar
Modals

We are sometimes lax about calling that remove, and other times remove can't do its job properly because of references elsewhere in the code.

In that screenshot from the Chrome heap profiler, red things are DOM elements that are detached but can not be garbage collected, yellow things are DOM elements that we still refer to somewhere.
The screenshot shows that the event handler which toggles the raw tag editor is still holding a reference to a D3 selection of the raw tag editor itself.  So D3 remove() detached it from the DOM, but it stuck around in memory anyway.
A good summary of what's happening can be found on the Chrome dev tools page, especially this graphic:

This is an easy problem to study, because all you need to do is take a heap snapshot, select/deselect a thing on the map so that the entity inspector shows and then clears, then take another heap snapshot and compare the two.  But that's just one memory leak in one UI element, and the bigger problem is that this is a widespread "death by a thousand cuts" antipattern everywhere in iD.
We need to be smarter about removing stale references to things in:

D3 selections
Event handlers
DOM element __data__
closures that mash all of the above together

Also related:  https://github.com/mbostock/d3/issues/1041
mbostock's take on this:  "well, duh"

The presence of detached DOM nodes in memory is not in and of itself a bug. There are cases where for performance reasons, retention is intended, so that the DOM nodes can be re-attached when the UI component becomes visible again, rather than rebuilt from scratch. I believe this is the case for some of the things in the "inspector / entity editor / sidebar".

Unfortunately this is still an issue (using Safari 17.3). iD uses 4+ GB of memory after opening the editor, making & saving edits and closing the editor (going back to OSM Carto) several times without closing the tab itself.

This makes iD unresponsive for several seconds after a single click on an element.


ReferenceError: 'Spinner is not defined' Using RequireJS

I use RequireJS to load spin.js:
require.config({
paths: {
jquery: 'jquery-1.9.0',
json2: 'json2',
handlebars: 'handlebars',
spin: 'spin',
login: 'login'
}
});
And than load it like this:
require(['jquery','spin', 'json2', 'handlebars'], function() {
//do some stuff
require(['login'], function() {
   //no scripts
});

});
when in the login.js I set the spinner:
var spinTarget = document.getElementById('login_spin');
var spinThis = new Spinner(spinnerOpts).spin();
I will see in this in the console:
ReferenceError: 'Spinner is not defined'
I tried this solution you offered:
https://github.com/hull/spin.js/commit/d2e39d1015cfbe2e6d51dbc194a8d0fcada9b7f7
But this did not make a difference.

I would like to stop this spinner in a different module. How can I do this ?


Compile error

Hi ,
I'm trying to use this library and followed the steps mentioned in /app/build.gradle

In the latest Android Studio , there is "runtime" dependencies. So I modified it as
dependencies {
// ...
compile 'su.litvak.chromecast:api-v2:0.9.3'
// ...
}

and added the "api-v2-0.9.3.jar" to the Projects/libs/ folder.
When I try to run the app , it gives me the following error:
"
Error:Execution failed for task ':app:transformClassesWithJarMergingForDebug'.

com.android.build.api.transform.TransformException: java.util.zip.ZipException: duplicate entry: su/litvak/chromecast/api/v2/AppEvent.class
"

Has anyone faced this issue. Any suggestions ? Am I missing something ?

and added the "api-v2-0.9.3.jar" to the Projects/libs/ folder.

Why do you add it there? It should be enough to put entry to build.gradle file. From my point of view you have included library into your project two times. That's why you are getting a duplicate entry error during some build step.

Hi Vitalidze,
Thanks  a lot . it worked.


Update disable-private-endpoint-network-policy.md

Mentioning prefix length clearly to avoid confusion.

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

Hello @sm947, please review @ivapplyr recommendations. We appreciate your contributions to the docs!

#please-close


Adding support of telegram parse_mode argument

Description of Problem:

Today there is no native support of parse_mode argument for telegram channel.
So if we need markdown in utters (e.g. for hyperlinks), we must use some workarounds with custom output payload(https://github.com/RasaHQ/rasa/issues/3750#issuecomment-506579280)
This workaround solves the problem but makes harder to use another rasa features.
As an example, buttons:
We can't define parse_mode in custom section and define buttons near  (because of this logic):
  utter_test:
  - custom:
      text: sample
      parse_mode: markdown
    buttons:
    - title: 'yes'
      payload: /yes_btn
    - title: 'no'
      payload: /no_btn

Instead, we need to define buttons in custom section  via reply_markup - special telegram field that contains json :) :
  utter_test:
  -  custom:
       text: sample
       parse_mode: markdown
       reply_markup: '{{"inline_keyboard": [[{{"text": "yes", "callback_data": "/yes_btn"}},
        {{"text": "no", "callback_data": "/no_btn"}}]]}}'

As a result we have complex and fragile utter definition.
Also, this workaround complicates utters usage in custom actions.
We need to detect current input channel to invoke utter_message with correct arguments:
input_channel = tracker.get_latest_input_channel()
if input_channel == "telegram":
    dispatcher.utter_message(json_message=create_custom_telegram_message(response_text, buttons))
else:
    dispatcher.utter_message(text=response_text,
                             buttons=[{"title": button.title, "payload": button.payload} for button in buttons])

So, without native support there is no convient way to use parse_mode and buttons at the same time.
Overview of the Solution:

parse_mode  in utter definition.
(e.g. near variable button_type: https://rasa.com/docs/rasa/connectors/telegram#supported-response-attachments)
Examples (if relevant):

Blockers (if relevant):

Definition of Done:

Thanks for submitting this feature request 🚀 @alopez will get back to you about it soon! ✨


ENH: add actions to filter MAGs

Closes #160.
Some artifacts useful for testing can be found here.

Hey @VinzentRisch, thanks, good catch! Actually, it does not make much sense for the metadata to be optional - otherwise, there's nothing to filter on... I just turned it into a required argument - hope that addresses your comment ;)

Ah yeah that makes sense.
Now I'm thinking that where should also be mandatory.
I saw that if where is None then an artifact is returned with all the original Mags.
But this does not have any application right? It just copies the artifact. So i dont think that where should be allowed to be None.

No, it should not be mandatory. You can also provide a metadata file which contains the IDs you want to filter on - there is no need to provide the "where" condition in such a case (you may already have a metadata table which is pre-filtered). This is explained in the help description of metadata and where parameters.

Ah okay that makes sense.


Warning with every video in console

Whenever it tries to unfavorite a video it says [Starling ICU Warning] The interpolation parameter "{number}" is missing in the translation text "Like video
{number} likes" for the key "accessibilityLabels_forYou_btn_like"
Suggestions: check whether the value is null or undefined. Try using empty string '' as fallback! e.g I18n.t('key', { apple: count || '' })
I have had it running multiple times and it sometimes just skips unfavoriting videos even though I have done the steps as needed. Had it running for about 12 hours total already and it only unfavorited about 400 videos even tough if all went well it should delete 1 per 3 seconds  which would be 14400 videos.
Any fix for this?

The error you mentioned (and in the screenshot below) occurs when opening any TikTok video through the web version. I uninstalled my browser extension and tested it to check if it was related, but it was not. It seems to be an internal UI error on TikTok's front end.

I tested this with a list of 10 TikTok videos in my favorites. It took about 50 seconds to start the process, delete the videos, and close the tab. I set this 3-second interval because sometimes it takes a while for the video to load, depending on the user's internet.
If the video doesn't load properly, the process won't proceed with removing it from favorites. Instead, it will move to the next video and continue the process.
I've created a 1-minute video that demonstrates the persistent occurrence of the error even when the browser extension isn't in use. Additionally, I've provided a walk-through of the extension's functionality. You can observe the extension's messages in the browser's console, specifically within the "Info" section of the "Console" tab.
This ensures that only the extension's messages are showcased, even if errors arise. Toward the end of the video, I intentionally induced this behavior to display the messages in a formatted manner.
This will help you ascertain that the error is indeed originating from the extension itself and not from any other section of the page.
https://github.com/gabireze/tiktok-all-favorite-videos-remover/assets/31194373/093bc0ab-a711-4f01-bb45-d64f755ac304
If the extension doesn't work as expected on Google Chrome, you might want to try using it on Mozilla Firefox or Microsoft Edge. Both browsers support the extension. I came across another user who mentioned that the extension wasn't working on Chrome, but they were able to use it seamlessly on Edge. I always make sure to test the extension on these three browsers before releasing it. I test both the local version and the version from the Chrome extensions store to ensure they're both correct.

Alright, I tried on Chrome first but after I saw the comment of the other person that said that Edge worked better I started using Edge. All videos do load in as I have the browser open just for best results and to verify they are loading but if I check back after some time I see that not all were unfavorited. Good that the error is not because of the extension though.
I will just try again and see what it does and otherwise I just have to do it a lot of times.
Thanks for the quick response!


Fix runbook with params and start conditions..

Description
Please include a summary of the change, motivation and context.

Testing
Please describe the tests that you ran to verify your changes. Please summarize what did you test and what needs to be tested e.g. deployed and tested helm chart locally.
Checklist:

[ ] My changes generate no new warnings.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] Any dependent changes have been merged and published.

Documentation
Make sure that you have documented corresponding changes in this repository.

This is not a useful contribution.


doc: write improved documentation for nixosOptionsDoc

Description of changes
Improves the previous documentation by extending explaining concepts and restructuring it.
This already makes use of doc-comments, though there is no way (yet) to display it in the manual.

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

CC @DanielSidhion @pennae can i ask you for some review?

Ok. Agreed i'll give it another revise. What major pain points do you see as unresolved?

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/2024-03-14-documentation-team-meeting-notes-113/41462/1

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/2024-03-21-documentation-team-meeting-notes-114/41957/1


 This is why we need tests for this part.

Pushed 3.5.1


Handle post duplicates in stream

If multiple monitored hashtags are present in a post - the post is also streamed multiple times.
For example, assuming a post:

Our company x is hiring for a position y
#Hiring #FediHiring

and that hashtags #Hiring and #FediHiring are monitored by the bot - they are streamed twice.
Results in either:

re-posting the content multiple times
errors while trying to write 'reposted' to the database

Gotchas

requires to monitor hashtags carefully, because otherwise the program can be flooded with (re-...)-re-streamed posts

Workaround: Make sure that multiple hashtags wouldn't be present in a single post - difficult, limits functionality
Solution: Implement a caching mechanism


Update Features.md

删除markdown语法加粗的多余标记
Why I'm doing:
介绍中误加了不必要的markdown加粗语法。
What I'm doing:
删除markdown语法加粗的多余标记
Fixes #issue
What type of PR is this:

[ ] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[x] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[x] This pr needs user documentation (for new or modified features or behaviors)
[ ] I have added documentation for my new feature or new function
[ ] This is a backport pr

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.3
[x] 3.2
[x] 3.1
[x] 3.0
[x] 2.5

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


[Hub Generated] Review request for Microsoft.Security to add version stable/2020-01-01

This is a PR generated at OpenAPI Hub. You can view your work branch via this link.
Changelog
Please ensure to add changelog with this PR by answering the following questions.

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[ ] update existing version for new feature
[x] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When you are targeting to deploy new service/feature to public regions? Please provide date, or month to public if date is not available yet.
When you expect to publish swagger? Please provide date, or month to public if date is not available yet.
If it's an update to existing version,  please select SDKs of specific language and CLIs that require refresh after swagger is published.

[x] SDK of .NET (need service team to ensure code readiness)
[x] SDK of Python
[ ] SDK of Java
[x] SDK of Js
[x] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[ ] No, no need to refresh for updates in this PR

Contribution checklist:

[ ] I commit to follow the Breaking Change Policy of “no breaking changes
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[ ] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
Adding a new service

[x] Please ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from API Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable version with Breaking Change Validation errors
[ ] Updating API(s) in preview over 1 year

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

Fixing the following issue: https://github.com/Azure/azure-rest-api-specs/issues/12308
The swagger up until now declared the the responses for the serverVulnerabilityAssessments delete operation are synchronous only, but in reality the service performs an asynchronous operation and therefore returns 202 accepted (which is not declared).
Also, setting the operation with x-ms-long-running-operation: true, and adding the relevant path to poll on the serverVA operations (that will be returned in the 'location' header of the delete response).

The changes in the jitNetworkAccessPolicies and the externalSecuritySolutions specs and examples were done in order to pass the "SDK Track2 Validation".
These changes were not initially planned, and technically are unconnected to this PR.
This is also the case with the commented out spec versions in the readme (which were duplicates of the same specs under the same tag) - but in this case these specs are under the responsibility of an entirely different team.

/azp run