Model simulation fails during synthesis

Hello,
I tried running https://github.com/Xilinx/FINN/blob/master/FINN/test/test_hwgq_cnv_w1a1.py to simulate my model. During synthesis part I get the following error:
$ python FINN/test/test_hwgq_cnv_w1a1.py
test_fpgabackend_rawhls (main.TestHWGQCNVw1a1) ... [ConvolutionLayer, LinearLayer, PoolingLayer, LinearLayer, LinearLayer, BipolarThresholdingLayer, ConvolutionLayer, LinearLayer, PoolingLayer, LinearLayer, LinearLayer, BipolarThresholdingLayer, ConvolutionLayer, LinearLayer, LinearLayer, LinearLayer, BipolarThresholdingLayer, FullyConnectedLayer, LinearLayer, LinearLayer, LinearLayer, BipolarThresholdingLayer, FullyConnectedLayer, LinearLayer, LinearLayer, LinearLayer, BipolarThresholdingLayer, FullyConnectedLayer, LinearLayer, LinearLayer, LinearLayer]
[ConvolutionLayer, BipolarThresholdingLayer, PoolingLayer, ConvolutionLayer, BipolarThresholdingLayer, PoolingLayer, ConvolutionLayer, BipolarThresholdingLayer, FullyConnectedLayer, BipolarThresholdingLayer, FullyConnectedLayer, BipolarThresholdingLayer, FullyConnectedLayer, LinearLayer]
Synthesising
ERROR
======================================================================
ERROR: test_fpgabackend_rawhls (main.TestHWGQCNVw1a1)
Traceback (most recent call last):
File "FINN/test/test_hwgq_cnv_w1a1.py", line 90, in test_fpgabackend_rawhls
hlslayers, self.net, self.dev, res_alloc_predetermined, dirpath, "sfcall-")
File "/home/steven/FINN/FINN/backend/fpga/backend_fpga.py", line 363, in synthesize
pipeline = convert(pipeline_in, net, dev, res_alloc, pipeline_ibits)
File "/home/steven/FINN/FINN/backend/fpga/backend_fpga.py", line 340, in convert
pipeline = prepare(pipeline, pipeline_ibits)
File "/home/steven/FINN/FINN/backend/fpga/backend_fpga.py", line 163, in prepare
pipeline = trns.apply_repeated(pipeline, passConvertToFPGALayers)
File "/home/steven/FINN/FINN/transforms/transformations.py", line 48, in apply_repeated
(ret, numChanges) = pass_to_apply(ret)
File "/home/steven/FINN/FINN/backend/fpga/backend_fpga.py", line 76, in passConvertToFPGALayers
raise Exception("Unsupported layer type in FPGA backend: %s" % L.get_type())
Exception: Unsupported layer type in FPGA backend: BipolarThresholdingLayer

The same error occurs if I try to simulate cnv-w1a1 model. However both models can synthesize hardware and create .bit files (when FINN is called with --mode=synth).

I managed to fix this issue by replacing layer restructuring in test_hwgq_cnv_w1a1.py with the one commented in lines 163-170 from https://github.com/Xilinx/FINN/blob/master/FINN/bin/finn.
However now it fails to compile the solution. The error message:
/usr/lib/gcc/x86_64-linux-gnu/5/../../../x86_64-linux-gnu/crt1.o: In function _start': (.text+0x20): undefined reference to main'
collect2: error: ld returned 1 exit status
Traceback (most recent call last):
File "FINN/bin/finn", line 192, in
process_args(args)
File "FINN/bin/finn", line 94, in process_args
generate_hardware(net, dev, gen_bitfile=False)
File "FINN/bin/finn", line 164, in generate_hardware
hlspipeline = ret.getSimLayer()
File "/home/steven/FINN/FINN/backend/fpga/backend_util.py", line 48, in getSimLayer
subprocess.check_call(["sh", self.getSimCompileScriptPath()])
File "/usr/lib/python2.7/subprocess.py", line 541, in check_call
raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['sh', '/tmp/finn-build-R6F4eC/simcompile.sh']' returned non-zero exit status 1
As I understand it is missing description of main() function. I'm not sure what is supposed to be in the main(). Should it be something like in BNN-PYNQ (https://github.com/Xilinx/BNN-PYNQ/blob/master/bnn/src/network/cnv-pynq/sw/main_python.cpp)?
Thank you in advance.

Please provide more details. It looks like it's going wrong at synthesis. Are you sure you have Vivado installed and in your path?


Update sbt version

My system sbt is 1.0.3, but running sbt new scala/hello-world.g8 sets the build.properties sbt version at 1.0.0 which is buggy. If there's a reason to leave this in the 0.13.x let me know.

Related: https://github.com/scala/scala-seed.g8/pull/14, which leaves the g8 version at 0.13.6

Thank you!! ❤


[CT-1987] [Feature] utility to show manifest differences

Is this your first time submitting a feature request?

[X] I have read the expectations for open source contributors
[X] I have searched the existing issues, and I could not find an existing issue for this feature
[X] I am requesting a straightforward extension of existing dbt functionality, rather than a Big Idea better suited to a discussion

Describe the feature
When slim ci does not work as expected it is difficult to troubleshoot what exactly is different between two manifests.
It would be great if there was a way to output the exact key in the manifest which is different.  This would make debugging simpler.
Describe alternatives you've considered
dbt ls -s state:modified.
This process at least narrows down the location of the difference, but you still need to check at key differences between the two manifests
Who will this benefit?
Anyone using slim ci.
Are you interested in contributing this feature?
No response
Anything else?
No response

Thanks for this suggestion @noel !
It makes sense to figure out ways to simplify debugging when two manifests differ in some unknown way.
I wonder if something like jq could help out here?
Example
Let's say that you have two manifests manifest-1.json and manifest-2.json and jq is installed. Then run the following:
diff -I "created_at" <(jq . manifest-1.json) <(jq . manifest-2.json) > my_diff

Here's my_diff for a project similar to this one. Can you tell what I changed between each invocation?

Click here to reveal diff
5,6c5,6
<     "generated_at": "2023-02-01T00:55:39.494863Z",
<     "invocation_id": "0e612d69-17aa-4331-aa77-559def7bf77b",
---
>     "generated_at": "2023-02-01T00:56:09.170195Z",
>     "invocation_id": "3649e233-054e-49f9-8f39-111212939828",
848c848,854
<         "post-hook": [],
---
>         "post-hook": [
>           {
>             "sql": "select 1",
>             "transaction": true,
>             "index": null
>           }
>         ],
862,863c868,873
<       "unrendered_config": {},
<       "created_at":<PHONE_NUMBER>.54368,
---
>       "unrendered_config": {
>         "post-hook": [
>           "select 1"
>         ]
>       },
>       "created_at":<PHONE_NUMBER>.255466,
908c918,924
<         "post-hook": [],
---
>         "post-hook": [
>           {
>             "sql": "select 1",
>             "transaction": true,
>             "index": null
>           }
>         ],
922,923c938,943
<       "unrendered_config": {},
<       "created_at":<PHONE_NUMBER>.545138,
---
>       "unrendered_config": {
>         "post-hook": [
>           "select 1"
>         ]
>       },
>       "created_at":<PHONE_NUMBER>.258373,
968c988,994
<         "post-hook": [],
---
>         "post-hook": [
>           {
>             "sql": "select 1",
>             "transaction": true,
>             "index": null
>           }
>         ],
982,983c1008,1013
<       "unrendered_config": {},
<       "created_at":<PHONE_NUMBER>.546604,
---
>       "unrendered_config": {
>         "post-hook": [
>           "select 1"
>         ]
>       },
>       "created_at":<PHONE_NUMBER>.261195,
1028c1058,1064
<         "post-hook": [],
---
>         "post-hook": [
>           {
>             "sql": "select 1",
>             "transaction": true,
>             "index": null
>           }
>         ],
1042,1043c1078,1083
<       "unrendered_config": {},
<       "created_at":<PHONE_NUMBER>.547963,
---
>       "unrendered_config": {
>         "post-hook": [
>           "select 1"
>         ]
>       },
>       "created_at":<PHONE_NUMBER>.263995,

Explanation of the command

This gist gist demonstrates the diff
This post to ignore all the different created_at timestamps

More ergonomic, using a shell alias
The command above is a bit of a bear. There's no way my fingers could possibly remember it.
Different shells have different ways of creating parametrized aliases, but I'll demonstrate using zsh (as described here):

Open ~/.zshrc
Add something like this (replacing diffman with your preferred alias):diffman() {
    diff -I "created_at" <(jq . $1) <(jq . $2)
}

Save and close
Run source ~/.zshrc to re-load the shell
diffman manifest-1.json manifest-2.json

Admittedly, I haven't done a ton of manifest diff'ing myself, so I'll be interested to hear your feedback on how useful (or not!) an approach like this might be.

Still seems valuable and maybe even more so with the new features in 1.5+

I meant that things like model versions and cross project dependencies may make this even harder. I still have to play around with 1.5, but I suspect there can be scenarios that should/shouldnt trigger during a state:modified run and when they do it could be difficult to pin point the reason why a model did/didnt run.
Regarding the manifest diffing, I guess I forgot to report back when I initially looked at this, but the output was too verbose and difficult to interpret.  I can show you if you are interested as I just reran it on a branch that has a single change yet I got a lot of output from diff

@noel Providing that example of how to generate a diff between manifests is probably the best we can offer here.
You're right that those diffs can be pretty verbose! Performing some kind of diff between manifests is a decent place to start troubleshooting, but it will be more art than science to sort the signal from the noise and interpret the differences.
Troubleshooting / debugging problems with state:modified will always have some inherent complexity to it that we can't make go away -- it's necessarily hard.
For those reasons, this seems like something we'd choose not to take on. What do you think @jtcohen6 ?


Final changes for notebook support

Some notes:

Added sphinx building to the testing-and-coverage workflow. Not against taking this out of the unit test workflow and creating a doc-building workflow, if people prefer that.
Made a number of changes to the Makefile. I'm not terribly up to date on best practices wrt those, so this is an area where I'm especially open to hear feedback.
This PR didn't end up troubleshooting the jupyter-nb-clear-output pre-commit hook after all (but it solves the sphinx pre-commit check failing with notebooks, though). I had trouble getting its included command to even work from the command line, and was already way past the time I'd budgeted to this. It would definitely be great to resolve that at some point.

This looks great @OliviaLynn, nice work!


Powertoys Awake keeps my laptop awake even if it is set to off

Microsoft PowerToys version
0.60.1
Running as admin

[ ] Yes

Area(s) with issue?
Awake
Steps to reproduce
1.boot my laptop
2.set powertoys awake to "keep awake indefinitely"
3.set it to "off"
4.my laptop cannot be put to sleep
Even if I exit PowerToys Awake, this keeps happening
✔ Expected Behavior
No response
❌ Actual Behavior
No response
Other Software
No response

/dup https://github.com/microsoft/PowerToys/issues/15052


[Docker Desktop Extension] Make theming consistent in the UI

hey, can i get assigned, i can make theming consistent for the desktop extension
@leecalcote @sudhanshutech

@humblenginr what are the other things required regarding theme?

This issue just outlines the requirement to use Sistent in Docker Desktop Extension. Nothing else should be needed for now since the migration should ensure consistency in theming.


Support for Neon Serverless Driver for Postgres

our serverless PostgreSQL driver, @neondatabase/serverless, connects to Neon databases from Cloudflare Workers and other environments supporting WebSockets, such as web browsers.

https://neon.tech/blog/serverless-driver-for-postgres
https://www.npmjs.com/package/@neondatabase/serverless

Hello, it's been a month, is there anything new yet? also hoping to bump through this comment

No. If there were news, there would be a comment about news - we wouldn't hide it. Sorry.

Hey, we at Prisma have been working on this for the past few weeks. Soon, we'll have something to share and test!
If you are interested in helping us, please respond to this super short survey and leave your email address so we can get in touch with instructions: https://pris.ly/survey/driver-adapter-neon Thanks.

I'm wondering if using the Neon or Plantescale drivers will allow us to run Prisma on edge functions.
I did a little experiment and it's not working. Neon driver is working great though, on regular serverless functions.

Not quite yet, but is indeed a building block. We are actively working on it though, so expect news soon.
(If you have not yet, respond to our survey I linked above and select the Edge Function providers you are interested in - we'll send out an Early Access version when we have something useful.)

I have good news:
We released support for using Neon's serverless driver with Prisma Client with our latest release 5.4.0 and just published a blog post that introduces and explains how it works and can be used: https://www.prisma.io/blog/serverless-database-drivers-KML1ehXORxZV!

If you try it, as it's behind the preview feature flag driverAdapters you can leave feedback in this discussion: https://github.com/prisma/prisma/discussions/21346

Are Serverless Database Drivers also coming to Prisma's edge client?

Yes @andrelandgraf: https://github.com/prisma/prisma/issues/21394 And more, depending on the edge function platform you are using.


mandatory code formatting complicates code reviews

The code formatting script makes the author's exact changes ambiguous, leading to more complicated code reviews. Reviews would be easier if PRs came in without additional formatting changes.
Options:

Don't require the formatting for the build.
Have the person doing the merge run the formatting tool as part of that step.
Rely upon code review only to catch strange formatting.

We don't currently run code formatting tools on Windows. In addition, the code formatting tools on OSX have proven to have some compatibility differences. Testing and developing on OSX can also be problematic if no one on the team is actively using that OS.

In this model, It's up to the PR author to run formatting to ensure it passes the diff check before issuing it.  Maybe that isn't the right answer, but that's what the current checking is trying to enforce.
If we're not going to use this model, I'd like to see (after the upstream merge) some kind of automated formatter that just runs clang-format nightly across the entire code base and commits changes if any.
The advantage to how it is now is that it guarantees reviewers don't have to page through useless style changes (made manually or automatically by some editors).  I suspect removing this will further complicate the review process.

If we're not going to use this model, I'd like to see (after the upstream merge) some kind of automated formatter that just runs clang-format nightly across the entire code base and commits changes if any.

It looks good to me.

The advantage to how it is now is that it guarantees reviewers don't have to page through useless style changes (made manually or automatically by some editors).

This is exactly how it should be, about me.

Don't require the formatting for the build.

I think this is fine. Because as we've discussed , many areas of the code do not conform to standard formatting and it creates very inconsistent looking code if the PR is a small change. So maybe it is best to ignore formatting issues returned by the clang auto format tool in some cases. And yes its also annoying to have to review formatting changes together with the actual change.
But I still think we should run it anyways individually, and have it show up on travis so that we have some way of enforcing formatting where we need to. Just that we can choose to ignore it in certain situations.

Have the person doing the merge run the formatting tool as part of that step.

I think this could be fine, or there could be seperate PR's just to correct the formatting. Or maybe it can be something automated? If its automated, I think it should not run on the entire code base, maybe just for the claimtrie code or just for any new commits.

It's no longer part of the build.


updated alexnet ecoset model weight path

Codecov Report

Merging #43 (75416a3) into master (fd91e30) will increase coverage by 1.03%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #43      +/-   ##
==========================================
+ Coverage   73.68%   74.72%   +1.03%
==========================================
  Files          13       13
  Lines        1258     1258
  Branches      240      240
==========================================
+ Hits          927      940      +13
+ Misses        290      276      -14
- Partials       41       42       +1

Flag
Coverage Δ

unittests
74.72% <100.00%> (+1.03%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

thingsvision/custom_models/alexnet_ecoset.py
92.85% <100.00%> (+92.85%)
: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 fd91e30...75416a3. Read the comment docs.


contracts: add seal_origin(), seal_code_hash and seal_own_code_hash to API

This is a follow-up to #10789 to add the following functions to contracts API:
fn origin() -> &AccountId;
fn code_hash(&AccountId) -> Option<CodeHash>;
fn own_code_hash() -> &CodeHash;

Sorry for taking so long with my review. Kind of swamped here. Most of this is very minor stuff except the benchmarks.

/benchmark runtime pallet pallet_contracts

/benchmark runtime pallet pallet_contracts

/benchmark runtime pallet pallet_contracts

Yeah I tend to agree. @agryaznov can you remove seal_origin from this PR?

CI not happy. Try to merge master again.

bot merge

Hey, do you have an idea why i do not have access to the "code_hash" function inside  "ink_env"?
I updated the ink! version to the latest 3.0.1 and still don't have access to it..

It was just merged to ink! and is not part of 3.0.1. You need to wait for the next release or use master.


Difference between documentation and code

Hi Ego4d Team,
Thanks for your great work.
In the cli/README.md, it states the dataset options are [annotations, full_scale, clips]. But in the documentation here https://ego4d-data.org/docs/data/videos/ , it states there are 3 types of videos, Canonical videos, Canonical Clips, and Video Components.
May I ask how they correspond to each other and how to download them?

Thanks for the feedback on the docs. It does seem not very clear. We will iterate on them.

Canonical videos are full_scale
Canonical clips are clips
annotations are annotations over the videos. Please see this page for their schemas: https://ego4d-data.org/docs/data/annotations/, this page for the higher level overview: https://ego4d-data.org/docs/benchmarks/overview/ and this page for how the annotation process was performed: https://ego4d-data.org/docs/data/annotation-guidelines/

Finally, for video components. Please refer to the docs here:  https://ego4d-data.org/docs/data/unprocessed_data/


feat(checkout): CHECKOUT-6643 add checkoutBillingSameAsShipping check for checkout

What?
Allows store admin to enable/disable "Billing Address same as Shipping Address" check when a shopper is using checkout process. This can be checked within the admin panel of the store under the Checkout Settings.
Why?
Store admin suggestion to BC as the customer would sometimes skip or ignore the check where it could be incorrect
Testing / Proof
![Screen Shot 2022-05-13 at 9 54 48 am](https://user-images.githubusercontent.com/9646373/168513934-b01a25d1-0652-45fa-8e4

8-7316e97b9f82.png)
ping @bigcommerce/checkout

On page refresh, it's not preserving the checkbox state of Billing Address

Hey @rajguru827 that is intended, we have moving to make that particular value merchant's config driven. So the API will populate that particular value going forward.


PRJNA705464 - ImmuneLandscapeccRCC

Project short name:
ImmuneLandscapeccRCC
Primary Wrangler:
Ida
Secondary Wrangler:

Link to Ingest

https://contribute.data.humancellatlas.org/projects/detail?uuid=12f32054-8f18-4dae-8959-bfce7e3108e7&tab=project

Associated files

Google Drive: folder

Published study links

Paper: Single-cell sequencing links multiregional immune landscapes and tissue-resident T cells in ccRCC to tumor topology and therapy efficacy

Accessioned data: PRJNA705464, SRZ190804

Key Events

[x] Convert published metadata to HCA spreadsheet
[x] Manually curate dataset to meet HCA metadata standard
[x] Collect any matrix and cell-type annotation files
[x] Upload sheet to validate metadata
[x] Transfer raw files to ingest to validate data files
[x] Check linking using ingest graph validator
[x] Ask the Secondary Wrangler for an end-to-end review of the project. Ask the Expertise Wrangler to review specific tabs if needed
[x] Submit dataset to Production
[x] Complete the Export SOP
[ ] Convert project data to SCEA format following the SCEA conversion SOP if appropriate

Review:
Donor:

Medications should be separated with commas

Sequence files:

Processes should be grouped by SRR accession, and lane indexes (which are indicated) are there to distinguish between replicates

Other than that, LGTM!!

I applied the comments and I will write to the authors to check the sequencing machine. Also asking about cellxgene

@arschat to chat with @idazucchi to start wrangling this dataset for cellxgene

Accessed files needed from 20+gb rds object, through codon-cluster.
Attaching a small markdown with guidance on how to do this, for future reference.
Proceeding with load all into an h5ad for the jupyter notebook.
https://app.zenhub.com/files/261790554/b0bc0844-b9f8-4ed1-9e2e-7c83b999f7b6/download

Converted both matrices to csr and size reduced to 5.6gb. Uploading h5ad file in drive folder.

collection has been published on CxG
https://cellxgene.cziscience.com/collections/3f50314f-bdc9-40c6-8e4a-b0901ebfbe4c


Date-only deserialization is not correct when local time uses negative UTC offset

When operationSpec for an API response specifies an object property with format "Date" its value is not deserialized to Javascript Date object as expected.
Swagger / Open API example:
"dateOfIssue": {
          "format": "date",
          "description": "Date of issue",
          "type": "string"
        },

For example when value "2021-03-31" is supplied by API in aforementioned property dateOfIssue the resulting Date object is actually deserialized as "2021-03-30" because the local timezone offset is applied.
Code in question is: https://github.com/Azure/ms-rest-js/blob/f6f0d92d79a1dfa8d92ff0891b88bc6b7a349e69/lib/serializer.ts#L214-L215
This is due to a rather strange behaviour of Javascript Date object when local time is set to negative UTC offset and only a date value is passed to its contructor:
// Timezone set to -6
new Date("2021-03-31")  // is in fact interpreted as "Tue Mar 30 2021 18:00:00 GMT-0600 (Central Standard Time)"

The above issue does not apply to positive UTC offsets.
However, when time fraction is supplied (with midnight) it yields expected results:
// Timezone set to -6
// when provided with time fraction (and without explicit time zone) it is constructed properly
new Date("2021-03-31T00:00:00")  // is in fact interpreted as "Wed Mar 31 2021 00:00:00 GMT-0600 (Central Standard Time)"

I believe that Date only values should be interpreted strictly as local date regardless of currently used time zone offset. The following fix could be used to provide such a behaviour:
else if (mapperType.match(/^(Date)$/ig) !== null) {
      payload = new Date(responseBody + "T00:00:00");
}
else if (mapperType.match(/^(DateTime|DateTimeRfc1123)$/ig) !== null) {
      payload = new Date(responseBody);
}

The very same behaviour is actually also present in the newer version of azure-sdk-for-js.

@skippone Thanks for the feedback!  This is an interesting problem (as always for any Date related issues).  We will investigate a bit deeper then get back to you.

The counter-example is a date-only string that is something like a validity period of a certificate, but I suppose most reasonable start boundaries might have time/timezone info rather than just a date?

yes for things that are important I'd expect time instead of just a date.


Text-only dataset

Hi, does your code supports fine-tuning with text-only dataset?

yes we do, tutorials are coming soon


SD Card fixes

Split out the SD card fixes from #204 as it would be a good idea to merge these first.

Oh, the #pragma fix ended up in here too...

Ah, reverting all of my "improvements" to fatfs_sd :D Thank you!
I'll check this out when I can and make sure things still work as expected. Might be interesting to see if this happens to fix any of the numerous other SD card woes that have befallen us, too.

#204 should rebase nicely on top of this. (Okay, it will... I've already done it in an attempt to get my SD working: https://github.com/Daft-Freak/32blit-beta/tree/MassStoragePR)

Seems the revert is causing issues in music-player.

Attempt to get both music-player and storage to work at the same time resulted in:
diff --git a/32blit-stm32/Src/fatfs_sd.c b/32blit-stm32/Src/fatfs_sd.c
index eebbf83..a6b8571 100644
--- a/32blit-stm32/Src/fatfs_sd.c
+++ b/32blit-stm32/Src/fatfs_sd.c
@@ -146,9 +146,21 @@ static bool SD_RxDataBlock(BYTE *buff, UINT len)
 	if(token != 0xFE) return FALSE;

 	/* receive data */
-	do {
-		SPI_RxBytePtr(buff++);
-	} while(len--);
+	if(len > 16)
+	{
+		// SD card expects 0xff dummy bytes for reads
+		for(unsigned int x = 0; x < len; x++){
+			buff[x] = 0xff;
+		}
+
+		HAL_SPI_TransmitReceive(HSPI_SDCARD, buff, buff, len, SPI_TIMEOUT);
+	}
+	else
+	{
+		do {
+			SPI_RxBytePtr(buff++);
+		} while(len--);
+	}

 	/* discard CRC */
 	SPI_RxByte();

(Or half-reverting the revert, but only for big transfers)

Okay, the latest fix from #204 fixes music-player as well. (Pulled in here as well.)

Thank you- sorry I was confused about the nature of these fixes (it's been a busy few weeks both at work and home (more work, haha) and my brain is tied in knots). Thank you!


Usage breaks import rule for Webfonts

converts to an empty

Sorry for the late answer, @Webling . This happens, because of the call to $inliner->setStripOriginalStyleTags();. Did you try placing the font rule outside of the {% cssinline %}...{% endcssinline %} tag?

Hi and thanks for your reply. Of course I tried to place the rule outside of the cssinline, but that lead to this https://github.com/eschmar/css-inliner-bundle/issues/2#issuecomment-398826722
I will update CSSToInline now and see what happens.


gui: RenameDialog: Warn user if deobfuscation settings are invalid.

:exclamation: Please review the guidelines for contributing
Done.
Description
The rename functionality relies on deobfuscation for now - so let the user know
this and ask the user to enable deobfuscation to get rename working.
The "Force rewrite deobfuscation map file" option effectively disables
renaming, because renaming relies on deobfuscation map modification for now,
but Force rewrite rewrites the map on each file reload, destroying changes.
So. let the user know this issue instead of silent failure.
TODO: Split rename map and deobfuscation map (for example, load the rename map
file after deobfuscation map file to override auto-generated aliases).

TODO: Split rename map and deobfuscation map (for example, load the rename map
file after deobfuscation map file to override auto-generated aliases).

I was planning to save renames in the project file (this class serialized to json), so it can be restored after rewrite.
And thanks for the fix!


Fix compilation error in BasicAuthSecurityFilterTest

This variables need to be final when compiled for Java < 8.

+1.

0.8.x: b1915bbc5700f82aced196055bff6bf4b7fee009


set config.php to stop working after config is done, refs #14

moved the check for file to the front than then only show setup steps if not configured.
Check for existance of cryptographically secure functions and then warn if not present.

@Zegnat updated the PR to merge master and fix these


Build fixes for tip

Please apply these changes. They fix the build for Go1. Additionally, the Makefile has been replaced by build annotations for the platform-specific code.

Thanks!
I've been meaning to make those changes when I had the time!
-Tarm
On Mon, Feb 20, 2012 at 10:44 AM, Benny Siegert
<EMAIL_ADDRESS>wrote:

Please apply these changes. They fix the build for Go1. Additionnally, the Makefile has been replaced by build annotations for the platform-specific code.
You can merge this Pull Request by running:
git pull https://github.com/bsiegert/goserial master
Or you can view, comment on it, or merge it online at:
https://github.com/tarm/goserial/pull/3
-- Commit Summary --

Remove now-obsolete Makefile.
Rename darwin to posix, add build constraint comments.
Use O_foo constants from syscall, they are no longer in package os.
Skip the connection test if test.short is used.
Use uintptr for os.NewFile.
gofmt

-- File Changes --
D Makefile (12)
M basic_test.go (3)
D serial_linux.go (1)
R serial_posix.go (5)
M serial_windows.go (4)
-- Patch Links --
https://github.com/tarm/goserial/pull/3.patch
 https://github.com/tarm/goserial/pull/3.diff

Reply to this email directly or view it on GitHub:
https://github.com/tarm/goserial/pull/3


Hotfix/whats new

This change is

Reviewed 4 of 4 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


ability for python skillet to parse type=checkbox with cbx list

The python skillet appears to only work with single value inputs as '--name value', name from the var section of the metadata file. Want to use a checkbox list model and var section parsing should be using each key/value pair in the list as '--key value' as an argument and ignore the name attribute.
cnc tasks.py likely should include: if type == 'checkbox' then:  and parses the variables values differently than the text and other var types. Although not sure if the key (displayed value) is sent in as a value and value can be used as a variable key - no reversed?
https://github.com/PaloAltoNetworks/pan-cnc/blob/f3278ebbafe0ec312290568eab6173013b5a54f5/pan_cnc/tasks.py#L265
from def python3_execute_script:
    for k, v in args.items():
        if input_type == 'env':
            env[k] = v
        else:
            cmd_seq.append(f'--{k}={v}')

Related comment: should the checkbox list be updated to show the value and not the key? Then the key/value relationship makes sense for these true/false checks.
use case is to select a list of security rule names, rule name = value and key is rule position (eg. rule1, rule2, rule3). Checkbox list should show the rule names (values) and the python args would be --key value. Eg: --rule1="allow outbound" --rule2="gsuite app rules" etc. Only the checked rules are pass in as key/value args and the unchecked ignored

Testing more the python parsing is now showing the value but only a single value, last in the list, so perhaps a checkbox list issue. This could be useful if the code filters based on checkbox=True for each item in the list and then creates the arg --name=value where the name is the var name.
Looks like in the python code you can use:
    parser.add_argument("--security_rules", action='append')

and this will create a list to send into python code with the values from the checked items in the list.
Use Case: I have a script to get all of the security rules from the firewall and would be able to feed this into a second .meta-cnc.yaml file var section to generate the checkbox list for second skillet that adds data (eg. zone, profile-group, tag...) to the set of rules.

@scotchoaf  example python script to handle list type input args is here. let me know if this works for your use case: https://github.com/nembery/Skillets/tree/experimental/skillets_by_type/python/input_args

this was implemented using csv delimited env or cli args


Rename spago bundle and spago make-module

These two commands do basically the same thing but have different enough names to be confusing.
So we should rename them to something clearer. Proposals:

spago bundle-app, spago bundle-module
spago bundle app spago bundle module
spago bundle defaults to "app" mode, and the module functionality is behind a flag spago bundle --module

My vote would be 2. 👍 👍

--main it to specify the main module that gets exposed (either to run or to export)

So I understand you're pointing to the --main module but it seems like a strange mnemonic to me for bundling a module. Does the module have to be a module Main?

@FintanH it doesn't have to be, but I hear you that this flag might be misnamed. If you do spago bundle --help, it gets a bit cleared as it tells you what it is for:
-m,--main MAIN           The main module to bundle

However, this same flag is --module in purs, so I'd support renaming it to --module here too

The main module to bundle

I think that's still ambiguous because it says "main" 😄
--module is good, my only gripe is that it's repetitive if we go down the route of spago bundle module. But I have no better alternative 👍


Phase 4: Make Connectors to talk to data sources

Phase 4 in the epic
Make Connectors that use this interface to talk to a WS or Web worker based backend for our UI:
{ subscribe, unsubscribe, send, connect, disconnect }

The connectors should

abstract the communication method so the UI doesn't have to know details after instantiation
be performant
allow a mock connector, for testing, to be easy to construct

Note
This is only for the request response parts of the connector. The subscriptions stuff is upcoming.

ref #1786


Connects OK but no stream

Hi,
I am trying to use the mjpeg-relay to relay a IP camera mjpeg stream. When launching the streamer, everything is looking fine. The terminal looks like this:
andre@rs209835:~/mjpeg-relay$ python relay.py -d -w 54018 http://admin:****@dyndns-host:8091/mjpeg/stream.cgi?chn=1
Starting new HTTP connection (1): dyndns-host
send: 'GET /mjpeg/stream.cgi?chn=1 HTTP/1.1\r\nHost: dyndns-host:8091\r\nAccept: /\r\nAccept-Encoding: gzip, deflate\r\nAuthorization: Basic YWRtaW46MTIzNDU2\r\nUser-Agent: python-requests/2.3.0 CPython/2.7.3 Linux/3.2.0-102-generic\r\n\r\n'
reply: 'HTTP/1.1 401 Unauthorized\r\n'
header: WWW-Authenticate: Digest realm="IPCamera Login", nonce="d2196ed82d421beea792716429f434d0", qop="auth"
header: Content-Type: text/html
header: Content-Length: 351
header: Connection: close
header: Date: Tue, 07 Mar 2017 21:09:09 GMT
header: Server: lighttpd/1.4.35
"GET /mjpeg/stream.cgi?chn=1 HTTP/1.1" 401 351
send: 'GET /mjpeg/stream.cgi?chn=1 HTTP/1.1\r\nHost: dyndns-host:8091\r\nAuthorization: Digest username="admin", realm="IPCamera Login", nonce="d2196ed82d421beea792716429f434d0", uri="/mjpeg/stream.cgi?chn=1", response="caf37f39406629b9aee0eba02ded2e75", qop="auth", nc=00000001, cnonce="bb118369fb119c46"\r\nAccept-Encoding: gzip, deflate\r\nAccept: /\r\nUser-Agent: python-requests/2.3.0 CPython/2.7.3 Linux/3.2.0-102-generic\r\n\r\n'
reply: 'HTTP/1.1 200 OK\r\n'
header: Cache-Control: public,max-age=0
header: Accept-Ranges: bytes
header: Content-Type: multipart/x-mixed-replace;boundary=ipcamera
header: Connection: close
header: Transfer-Encoding: chunked
header: Date: Tue, 07 Mar 2017 21:09:10 GMT
header: Server: lighttpd/1.4.35
"GET /mjpeg/stream.cgi?chn=1 HTTP/1.1" 200 None
Connected to stream source, boundary separator: --ipcamera
I can see the to-be-relayed stream OK from the internet and on my local network. When I run the mjpeg-relay, I can see the status page and I do see a incoming bandwidth. When I connect to the /stream URL, I do see that the outbound bandwidth goes up. But 1. I do not see the mjpeg in the browser/-Tag. 2. Also VLC does not play the outgoing stream. I tried running the relay on a different server but makes no difference. Anything else I can look at?

Does the /snapshot url displays a snapshot?

No. The returning image is empty (also double checked wget - resulting file is empty)

curl --raw -v http://localhost:54321/stream

Hostname was NOT found in DNS cache
Trying <IP_ADDRESS>...
Connected to timemelter.com (<IP_ADDRESS>) port 54321 (#0)

GET /stream HTTP/1.1
User-Agent: curl/7.37.0
Host: timemelter.com:54321
Accept: /

HTTP 1.0, assume close after body
< HTTP/1.0 200 OK
< Connection: close
< Server: MJPEG-Relay
< Cache-Control: no-store, no-cache, must-revalidate, pre-check=0, post-check=0, max-age=0
< Pragma: no-cache
< Expires: -1
< Content-Type: multipart/x-mixed-replace;boundary=--ipcamera
<

Give https://github.com/Silex/mjpeg-relay/tree/chunked-encoding-support a try and report.
Btw, in your above paste after the headers there's nothing?

andre@rs209835:~$ git clone
https://github.com/Silex/mjpeg-relay/tree/chunked-encoding-support
Cloning into 'chunked-encoding-support'...
error: The requested URL returned error: 403 while accessing
https://github.com/Silex/mjpeg-relay/tree/chunked-encoding-support/info/refs
fatal: HTTP request failed
To your question: No. I assume it will run into a timeout eventually
On 07.03.2017 22:47, Philippe Vaucher wrote:

Give
https://github.com/Silex/mjpeg-relay/tree/chunked-encoding-support a
try and report.
Btw, in your above paste after the headers there's nothing?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/OliverF/mjpeg-relay/issues/16#issuecomment-284870086,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AZC178CqdwRT5z0hNc7joa0EaFhktsfLks5rjdBggaJpZM4MWCGv.

--
André Stricker
Stricker Consulting
Lindenstrasse 45
D-35398 Gießen
Germany
Tel: +49 (0)641-33055205-0
Fax: +49 (0)641-33055205-9
Mob: +49 (0)179 29 43 130
e-Mail<EMAIL_ADDRESS>www.scw-services.com

git clone git://github.com/Silex/mjpeg-relay

Ah thanks.
Done that. When I run the relay now, it looks ok again, but the /status page show no incoming bandwidth..
python relay.py -d -w 54018 http://admin:****@dyn-url:8091/mjpeg/stream.cgi?chn=1 Starting new HTTP connection (1): dyn-url send: 'GET /mjpeg/stream.cgi?chn=1 HTTP/1.1\r\nHost: dyn-url:8091\r\nAccept: */*\r\nAccept-Encoding: gzip, deflate\r\nAuthorization: Basic YWRtaW46MTIzNDU2\r\nUser-Agent: python-requests/2.3.0 CPython/2.7.3 Linux/3.2.0-102-generic\r\n\r\n' reply: 'HTTP/1.1 401 Unauthorized\r\n' header: WWW-Authenticate: Digest realm="IPCamera Login", nonce="0c84df3082134992e588ef7d1dfe739f", qop="auth" header: Content-Type: text/html header: Content-Length: 351 header: Connection: close header: Date: Tue, 07 Mar 2017 22:04:09 GMT header: Server: lighttpd/1.4.35 "GET /mjpeg/stream.cgi?chn=1 HTTP/1.1" 401 351 send: 'GET /mjpeg/stream.cgi?chn=1 HTTP/1.1\r\nHost: dyn-url:8091\r\nAuthorization: Digest username="admin", realm="IPCamera Login", nonce="0c84df3082134992e588ef7d1dfe739f", uri="/mjpeg/stream.cgi?chn=1", response="c7d421e513fa43b1e48079761fcf6896", qop="auth", nc=00000001, cnonce="10cbc87611e2c11f"\r\nAccept-Encoding: gzip, deflate\r\nAccept: */*\r\nUser-Agent: python-requests/2.3.0 CPython/2.7.3 Linux/3.2.0-102-generic\r\n\r\n' reply: 'HTTP/1.1 200 OK\r\n' header: Cache-Control: public,max-age=0 header: Accept-Ranges: bytes header: Content-Type: multipart/x-mixed-replace;boundary=ipcamera header: Connection: close header: Transfer-Encoding: chunked header: Date: Tue, 07 Mar 2017 22:04:09 GMT header: Server: lighttpd/1.4.35 "GET /mjpeg/stream.cgi?chn=1 HTTP/1.1" 200 None Connected to stream source, boundary separator: --ipcamera
mjpeg-relay status Summary Parameter	Value Clients	0 Incoming bandwidth	0.0000Mb/s Outgoing bandwidth	0.0000Mb/s

Please use a pastebin or use github's code blocks.
Anyway, you are sure http://admin:****@dyndns-host:8091/mjpeg/stream.cgi?chn=1 works properly? Is it possible to setup some temporary viewer user for us to use?

Hm. I just rebooted the camera and it didn't come up again. Camera is on
rooftop. I will have to double check tomorrow. I will get back. Sorry
for the delay and thanks for the support
On 07.03.2017 23:11, Philippe Vaucher wrote:

Please use a pastebin or use github's code blocks.
Anyway, you are sure
|http://admin:****@dyndns-host:8091/mjpeg/stream.cgi?chn=1| works
properly? Is it possible to setup some temporary viewer user for us to
use?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/OliverF/mjpeg-relay/issues/16#issuecomment-284876647,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AZC1724bEduve3V84rJKFwwnN88XEZgCks5rjdX-gaJpZM4MWCGv.

--
André Stricker
Stricker Consulting
Lindenstrasse 45
D-35398 Gießen
Germany
Tel: +49 (0)641-33055205-0
Fax: +49 (0)641-33055205-9
Mob: +49 (0)179 29 43 130
e-Mail<EMAIL_ADDRESS>www.scw-services.com

Here u go.
http://test:test@dev.scw-services.com:8091/mjpeg/stream.cgi?chn=1
I tried relaying this stream as well but no luck.
On 07.03.2017 23:11, Philippe Vaucher wrote:

Please use a pastebin or use github's code blocks.
Anyway, you are sure
|http://admin:****@dyndns-host:8091/mjpeg/stream.cgi?chn=1| works
properly? Is it possible to setup some temporary viewer user for us to
use?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/OliverF/mjpeg-relay/issues/16#issuecomment-284876647,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AZC1724bEduve3V84rJKFwwnN88XEZgCks5rjdX-gaJpZM4MWCGv.

--
André Stricker
Stricker Consulting
Lindenstrasse 45
D-35398 Gießen
Germany
Tel: +49 (0)641-33055205-0
Fax: +49 (0)641-33055205-9
Mob: +49 (0)179 29 43 130
e-Mail<EMAIL_ADDRESS>www.scw-services.com

Hello,
Just had a quick attempt, looks like that gives a 401 even when connecting directly without using mjpeg-relay. Please check the credentials

Camera is down again. Will climb up there tomorrow morning and get back.
Thanks already!
On 08.03.2017 00:08, OliverF wrote:

Hello,
Just had a quick attempt, looks like that gives a 401 even when
connecting directly without using mjpeg-relay. Please check the
credentials
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/OliverF/mjpeg-relay/issues/16#issuecomment-284890455,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AZC1758cc5Ai4l_KlSY4flH0vuIrn_9Fks5rjeNTgaJpZM4MWCGv.

--
André Stricker
Stricker Consulting
Lindenstrasse 45
D-35398 Gießen
Germany
Tel: +49 (0)641-33055205-0
Fax: +49 (0)641-33055205-9
Mob: +49 (0)179 29 43 130
e-Mail<EMAIL_ADDRESS>www.scw-services.com

The credentials are fine, you need to use digest auth.
The problem is more like the camera only accepts very few connections... curl -v --raw --anyauth "http://test:test@dev.scw-services.com:8091/mjpeg/stream.cgi?chn=1" timeout before sending the stream.
In all my attempts it only sent the MJPG once.


napari sphinx theme

Summary
The napari.org website features a custom theme to support a consistent look-and-feel with the napari hub, however, the current process for building the napari.org site is too complex for documentation contributors to easily preview their changes locally. To simplify the build process, we are going to release the napari.org theme as a pip-installable sphinx theme and rely on the sphinx build process to render the napari.org website.
Intended outcome
The napari.org docs will be refactored to have a more conventional build process via make docs which works the same locally and in production.
How will it work?
We will implement the current napari.org theme as a pip-installable sphinx theme.
Assets

Product Spec
Designs
Tech Spec

Tasks

[ ] #31
[x] #32
[ ] #33
[ ] #34
[ ] #35

@neuromusic all the open dev / design issues are complete, so I'm going to close this and the following issues:

https://github.com/napari/napari-sphinx-theme/issues/31
https://github.com/napari/napari-sphinx-theme/issues/33
https://github.com/napari/napari-sphinx-theme/issues/34
https://github.com/napari/napari-sphinx-theme/issues/35

After I'll create a release for v0.1.0 😄 🎉


should i need to give attribution or any thing in my commercial project?

Issue type:

[ ] Feature request
[ ] Bug report
[ ] Documentation

Environment:

AdminLTE Version:
Operating System:
Browser (Version):

Description:

I am working on a commercial project and using adminlte. In AdminLTE license its written that include the copyright in "all copies or substantial portions of the Software". What does it mean?

The License Conditions are License and copyright notice.
A copy of the license and copyright notice must be included with the software.


feat: improve display of clear filters button

only show the button when there is/are filter(s)

Related issue(s)

Fixes: #816

@magicmatatjahu 😂😂😂 I amended it but I still saw a lot of changes. It looked scary.

/rtm


Question: Is it possible and how to specify width: 100% for picker?

I see that width is computed here:
https://github.com/TypeCtrl/ngx-emoji-mart/blob/0ec517cb0b12a61540bc0b63fa592cca52d11f53/src/lib/picker/picker.component.html#L2
Need to have a picker that fits a re-sizeable chat container <div>.
Tried many things:
Editing the div .emoji-mart 's width in console to 100% behaves as desired.
The following does not work.
<emoji-mart title="Pick your emoji…" emoji="point_up" [style]="{ width: '100%' }"></emoji-mart>

the width decides how many emoji will be in each row and how far to scroll when using the categories

I am able to use this:
TS:
@ViewChild('emojiPicker', { read: ElementRef }) emojiPicker: ElementRef;
constructor( private renderer: Renderer2) { }

   public ngAfterViewInit() {
      const el = this.emojiPicker.nativeElement.querySelector('.emoji-mart');
      this.renderer.setStyle(el, 'width', '100%');
}

Template:
 <emoji-mart #emojiPicker title="Pick your emoji…" emoji="point_up"></emoji-mart>

:tada: This issue has been resolved in version 0.14.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fix the C lowering for RecordLift to be compatible with C++.

C++ prohibits non-constant struct literal initializers from being implicitly converted to narrower types, so insert explicit casts. This allows the generated code to be compiled as both C and C++.
Specifically, this fixes the following error in C++:
test.c:40:5: error: non-constant-expression cannot be narrowed from type 'int32_t' (aka 'int') to 'enum_t' (aka 'unsigned char') in initializer list [-Wc++11-narrowing]

While debugging this, I also tidied up some redundant parens in load_ext.

Do you know if it would be possible to add a test for this by compiling with -Werror or similar?

I tried testing the old generated code with -Weverything in C mode and clang has no warning for this.

I think right now the test framework supports wasm.c as C source files, but could wasm.cpp also be supported and an existing test translated to C++? (and/or just copied over)

I've now modified the test runner to also compile all the tests as C++. That uncovered several more compilation errors, which I've now also fixed.


Bump frameworks version to v0.13.0

Includes changes to UK regions.

:+1:

Closing as another version will be coming soon.


Revise About Us Section

[x] develop new content
[x] get translated
[ ] Publish on site

Yesterday did first review of About us content draft and @brockhigg10 and I met to discuss.
Today revised and added content based on our discussion and comments in the doc.

Id id review and let my comments

Reviews complete. Ready for translation.

To send to translation today and then will need bilingual review

Pull requests sent for new About Us content in both french and english


Prod function

Nice work! I did check all your work and it definitely helped me as well. I used the checklist and it seems like you followed everything.
Also, i'm confused where we should put our code. You did put it in a separate file and lovisa used int.java. We'll see more after the sprint review.


Add api to manage docker daemon labels

Managing docker daemon labels is good for doing some specific scheduling with Swarm.
I am interesting in posting a PR to add api to manage docker daemon labels with @feiyang21687 help.
Docker was able to modify daemon label with docker label command, like this:

Signed-off-by: hacpai<EMAIL_ADDRESS>

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "16203-manage-daemon-labels" git@github.com:hacpai/docker.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

I think this lands more in the daemon configuration conversations that we've had with other maintainers, like @duglin. He was proposing to allow changing certain configuration options with a command like docker config and labels seem like a good candidate to be included in that set.
I'd like to see something in a more general idea of configuration than adding specific commands for specific options, like this one.
So, I'm going to say that we shouldn't do this. But, maybe other maintainers think otherwise.

I do agree with @calavera that it would make more sense to have a more general commands if any.
I'm still having hard time to see use cases to change dynamically the daemon options (but I might be blind :sweat:). I would think it is more the job of a configuration management system to add / maintain this daemon options / labels.

I'm gonna close this for now, but feel free to discuss on it and argue for it :wink:.

Changing daemon labels is really something that is needed for swarm or you have to restart the docker daemons that have attributes and characteristics changing dynamically (changing a node that was handling dev workloads to now handle prod workloads for example), as the scheduling policies rely heavily on those labels.
Now I totally agree with the comments above, we need a more general command to deal with the daemon configuration and allow to dynamically change attributes at runtime.

@calavera @vdemeester @abronan Thank you for your suggestion. I think docker config is a good design. But maybe it is implemented in a time. I want to perfect docker label to make it more general. So I plan to add the feature of managing container label to help more people use docker easily to do something. Referencing https://github.com/docker/docker/issues/15496
Is it a good idea?

@vdemeester Does docker team have any progress on this issue now?  We are using swarm and facing the exact same problem.  It's too heavyweight to restart daemon just to set a label on it.
At which version would we see the label api?  If it is still not in the plan, we'd have to use other workaround, e.g. powerstrip to hijack the api, sadly.

@hongqn see my other answer; https://github.com/docker/docker/pull/16978#issuecomment-175983014 this will be possible in docker 1.10; you can try the release candidates now


Add ActiveMQ Artemis recommended monitors

What does this PR do?
Add recommended monitors for ActiveMQ Artemis integration.
Motivation

Additional Notes

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached

activemq_artemis_high_disk_store.json is of unsupported type: "metric alert". Only the following types are allowed: ['query alert', 'event alert', 'service check']


LENS 6.0 Incompatible

Thanks for this lens extension , I really enjoy it.
The extension is not compatible with LENS 6.0

thanks

Version 1.1.0 (found on npmjs) is now compatible with Lens 6.0

Thanks @nevalla
the link -><EMAIL_ADDRESS>

Thanks @nevalla
the link -><EMAIL_ADDRESS>
Thank you! Worked like a charm :)


Linux Mint instalation - missing Release a InRelease files

Describe the bug
Ign:14 https://pipewire-debian.github.io/pipewire-debian ./ InRelease
Err:15 https://pipewire-debian.github.io/pipewire-debian ./ Release
  404  Not Found [IP: <IP_ADDRESS> 443]

To Reproduce
curl -SsL https://pipewire-debian.github.io/pipewire-debian/ubuntu/KEY.gpg | sudo apt-key add -
sudo curl -SsL -o /etc/apt/sources.list.d/pipewire.list https://pipewire-debian.github.io/pipewire-debian/ubuntu/pipewire.list
sudo apt update

I tries also this Github repo as path in sources list, but it failed too - for another error (sorry for Czech language in console output, but you will definitely understand)
Stahuje se:13 https://github.com/pipewire-debian/pipewire-debian/tree/master/ubuntu ./ InRelease
Err:13 https://github.com/pipewire-debian/pipewire-debian/tree/master/ubuntu ./ InRelease
  Podepsaný soubor není platný, obdrženo „NOSPLIT“ (vyžaduje přístup na síť ověření?)
Načítají se seznamy balíků… Hotovo
E: Selhalo stažení https://github.com/pipewire-debian/pipewire-debian/tree/master/ubuntu/./InRelease  Podepsaný soubor není platný, obdrženo „NOSPLIT“ (vyžaduje přístup na síť ověření?)
E: Repositář „https://github.com/pipewire-debian/pipewire-debian/tree/master/ubuntu ./ InRelease“ není podepsán.
N: Aktualizaci z takového repositáře nelze provést zabezpečeně a je proto implicitně zakázána.
N: Podrobnosti o vytváření a nastavení repositářů naleznete v manuálové stránce apt-secure(8).

Expected behavior
APT will update repository without error :-)
Screenshots
-> cat /etc/apt/sources.list.d/pipewire.list
deb https://pipewire-debian.github.io/pipewire-debian ./

Desktop (please complete the following information):

Distro: Linux Mint
Desktop Environment Gnome
Version 20.1 (ulyssa)

sorry for the issue. edit /etc/apt/sources.list.d/pipewire.list and append /ubuntu at the end of the repo url
or issue : echo "deb https://pipewire-debian.github.io/pipewire-debian/ubuntu ./" | sudo tee /etc/apt/sources.list.d/pipewire.list
problem should be solved.
Actually I recetnly cleanup this repo, and moved all packages to ubuntu/ this is why the problem occurred.
You may fail to install pipewire from launchpad for now, because PipeWire 0.3.27 released, and is in building queue under launchpad.  It may take 2~4 hours to fix for launchpad.

Thank you for your quick reply, now I have installed pipewire and all dependencies successfully.
I want to try Pipewire, because when I'm using bluetooth headset with mic, the sound quality is very poor. But... so...  current configuration with Pipewire didn't solve my problem :-(
Now I can choose one of profiles:

High Fidelity Playback (A2DP Sink)
High Fidelity Playback (A2DP Sink, codec SBC)
High Fidelity Playback (A2DP Sink, codec AAC)
Headset Head Unit (HSP/HFP)
Headset Head Unit (HSP/HFP, codec CVSD)

When I choose any profile with AD2P, sound quality is great, but system select internal mic. When I switch to headset mic, the profile is automaticaly switched to HSP/HFP and sound quality is not so good :-(
Can you please guide me, what am I doing wrong?

I think, Its normal, I'm using PW since 0.3.22 on my native Arch linux, It behaves the same way you described. please read wiki section and Troubleshooting page and consult with this page also. If you have any problem regarding on pipewire but this ppa, create a issue here. Search your topic before creating any new issue.

Once more, thank you! :-)


Fix live mode

Resolves the issue where the video’s current time kept changing at the start of live mode, causing it to freeze for a few seconds.

:tada: This PR is included in version 1.0.6-next.1 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


ble client example, gatt_rsp_timeout and disconnect after a few seconds (IDFGH-14144)

Hello,
I run the gatt_client example with ESP32 and use it to connect a ble server running on an Android device.
The connection can be established and Ble notification can be received.
But after 20-30 sec, "BT_GATT: gatt_rsp_timeout" will occur and disconnection will be made.
I found that this may be related to the void gatt_rsp_timeout(TIMER_LIST_ENT *p_tle) function in gatt_utils.c.
What is the cause of the problem and how to fix it?
Thank you!
The program's log is shown below.
I (31693) GATTC_DEMO: 01 00 62 22 37 84
I (31873) GATTC_DEMO: ESP_GATTC_NOTIFY_EVT, receive notify value:
I (31873) GATTC_DEMO: 01 00 62 22 37 84
I (32063) GATTC_DEMO: ESP_GATTC_NOTIFY_EVT, receive notify value:
I (32063) GATTC_DEMO: 01 00 61 22 37 84
I (32243) GATTC_DEMO: ESP_GATTC_NOTIFY_EVT, receive notify value:
I (32243) GATTC_DEMO: 01 00 61 22 37 84
W (32263) BT_GATT: gatt_rsp_timeout disconnecting...
W (32263) BT_HCI: hci cmd send: disconnect: hdl 0x0, rsn:0x13
W (32263) BT_APPL: gattc_conn_cb: if=3 st=0 id=3 rsn=0x16
E (32263) GATTC_DEMO: write descr failed, error status = 85
W (32273) BT_HCI: hcif disc complete: hdl 0x0, rsn 0x16
I (32283) GATTC_DEMO: ESP_GATTC_DISCONNECT_EVT, reason = 22
===========================

@Astdg9826 have you fixed this issue?

Thank you for asking! I found the problem, which is on the Android program side. It failed to respond a ble call from esp32.


FindElements does not work, but FindElement does.

I have found this issue randomly where at some point FindElements just stops working. Below is one such example I have run into without a solution. The below code finds all E-Mail elements on the Mail page of Outlook 365.
Target Application: Outlook 365
Target App Version: 16.0.11929.20606
WinAppDriver Version: 1.2
Selenium Version: 3.141
var test = Driver.FindElement(By.XPath("//*[@ClassName='SuperGrid']//*[@ClassName='ThreadItem']"));
will deliver an object.
var test2 = Driver.FindElements(By.XPath("//*[@ClassName='SuperGrid']//*[@ClassName='ThreadItem']"));
Will deliver 0 results.
Output from WinAppDriver:
POST /session/4EAD86D6-0952-4548-B1D9-26942D614AE9/elements HTTP/1.1
Accept: application/json, image/png
Content-Length: 83
Content-Type: application/json;charset=utf-8
Host: <IP_ADDRESS>:4723
User-Agent: selenium/3.141.0 (.net windows)

{"using":"xpath","value":"//*[@ClassName='SuperGrid']//*[@ClassName='ThreadItem']"}
HTTP/1.1 200 OK
Content-Length: 74
Content-Type: application/json

{"sessionId":"4EAD86D6-0952-4548-B1D9-26942D614AE9","status":0,"value":[]}

==========================================
POST /session/4EAD86D6-0952-4548-B1D9-26942D614AE9/element HTTP/1.1
Accept: application/json, image/png
Content-Length: 83
Content-Type: application/json;charset=utf-8
Host: <IP_ADDRESS>:4723
User-Agent: selenium/3.141.0 (.net windows)

{"using":"xpath","value":"//*[@ClassName='SuperGrid']//*[@ClassName='ThreadItem']"}
HTTP/1.1 200 OK
Content-Length: 102
Content-Type: application/json

{"sessionId":"4EAD86D6-0952-4548-B1D9-26942D614AE9","status":0,"value":{"ELEMENT":"42.854534.4.-384"}}

I found a work around, seems I need to explicitly set the experimental mode to false:
if (typeof(T) == typeof(AppiumOptions))     (options as AppiumOptions).AddAdditionalCapability("ms:experimental-webdriver", false);
I don't know why this would have to be explicitly set, but it seems to have address the issue.


Http monitor to also check website performance metrics

⚠ Please verify that this feature request has NOT been suggested before.

[X] I checked and didn't find similar feature request

🏷 Feature Request Type
UI Feature
🔖 Feature description
When doing an http monitor it would be pretty nice to have a feature allowing us to also track the websites lighthouse scores and also plot them over time.
Similar to https://github.com/zachleat/performance-leaderboard in https://www.11ty.dev/speedlify/
A workflow would maybe be

check uptime
if up (and option selected), check full performance - but on a different schedule.. daily is more useful in this case than 60 seconds

It would be great to list sub-urls as well.
So if my main uptime check is example.com, I could add a list of 5 sub-pages that I want to check the full performance on.
✔ Solution
Add an option to http monitors to "check full page performance" on a daily schedule.
When the uptime check passes, check the full performance.
There will probably be an additional db table added to store the performance data. The output from performance-leaderboards looks pretty flat and could probably be kept in one table.
New charts would be added to the monitor preview page and also the status pages.
❓ Alternatives
No response
📝 Additional Context
Often times we don't want to know just if the site is "on" but how the performance is going over time. Others have request checking page size, or response time, and I think this would provide a way to add more strict "on" checks for certain sites.

Hello
I find that the solution is very specialized and can be too complete (SEO, ranking)
Given the objective of the project, I think that adding only the variables essential to monitoring the performance of the website is sufficient:
Example of recoverable data with CURL
              time_appconnect
                             The time, in seconds, it took from the start until the SSL/SSH/etc connect/handshake to the remote host was completed. (Added in 7.19.0)

              time_connect   The time, in seconds, it took from the start until the TCP connect to the remote host (or proxy) was completed.

              time_namelookup
                             The time, in seconds, it took from the start until the name resolving was completed.

              time_pretransfer
                             The time, in seconds, it took from the start until the file transfer was just about to begin. This includes all pre-transfer commands and negotiations that
                             are specific to the particular protocol(s) involved.

              time_redirect  The  time, in seconds, it took for all redirection steps including name lookup, connect, pretransfer and transfer before the final transaction was started.
                             time_redirect shows the complete execution time for multiple redirections. (Added in 7.12.3)

              time_starttransfer
                             The time, in seconds, it took from the start until the first byte was just about to be transferred. This includes time_pretransfer and also  the  time  the
                             server needed to calculate the result.

I think they are two complementary tools


Does not install correctly with yarn

For some reason, the node_modules folder is being included when downloading the package. The node_modules folder contains rxjs and @angular. I removed the rxjs dependency in #57 but Angular being in node_modules is the culprit.

Fixed in #3a01d604707c70393d70dcc73b55cbdaab39e820


make deb reader handle new control.tar options introduced in dpkg 1.17.6

newly supported is uncompressed control.tar and xz compressed
control.tar.xz. latter is used by ubuntu for dbgsym ddebs.
Fixes #655
Requirements
All new code should be covered with tests, documentation should be updated. CI should pass.
Description of the Change
Currently Ubuntu 18.04 builds with ddebs (debug symbols) cannot be uploaded to aptly as they get rejected with
    Unable to read file /home/me/.aptly/upload/Aptly__Repository-smith-20171031-18307-yjnsbr/test.ddeb: unable to find control.tar.gz part in package /home/me/.aptly/upload/Aptly__Repository-smith-20171031-18307-yjnsbr/test.ddeb

which is caused by the packages making use of control.tar.xz members which are fairly newly supported in dpkg. The proposed change adds support for new control.tar variants as per deb format specification manpage.
Checklist

[x] unit-test added (if change is algorithm)

Travis seems to fall over gometalinter, not sure why or what to do about that.

Linting fixed by #659

rebased & travis green now


Add support for AWS DynamoDB storage backend

Adds support for configuring Vault to use the DynamoDB storage engine.
Documentation for all of the backend-specific variables and their defaults have been added to the README. The defaults are based on the AWS KMS unseal backend defaults, allowing for
overriding of most of the defaults via environment variables.
Note: This PR also includes some changes to the AWS KMS template as well, as I ran into issues
with that template while testing a full deployment with my changes.
I was having issues with values not appearing in the generated configuration file due to the existing if var | bool checks.
If you would prefer to deal with those changes separately, I can back them out of this PR and submit them in a separate PR.

Thanks!


Updating SW version and its dependencies

updating libs
updating to latest weaver version

In #105, I updated the weaver version. This PR also updates the go version that leads to some github workflow failures. Let's do this later.


Not found: Could not find session factory for DIRECT_SESSION

Operating System:  Android
because my android system is below Android 5.0, so I rebuild the tensorflow librarys for android with NDK android-11.  the building is success.
then,  with these librarys, I write a android demo.  But when I run, it output the message ...
  **Not found: Could not find session factory for DIRECT_SESSION**

I don't know, what's wrong with my works.

There are problems with Bazel and NDK 11 currently; it's suggested that you use NDK r10e until this is resolved. See #1468 for details and links to relevant r10e downloads.
If the problem persists after trying r10e please let me know the bazel version and command you're using to build.

@andrewharp
thanks for your replay.
"Not found: Could not find session factory for DIRECT_SESSION"
my ndk is always the NDK r10e, and i rebuild it,  the build is ok, but the problem still persists.

libandroid_tensorflow_lib.a
libprotobuf.a
libprotobuf_lite.a
libprotos_all_cc.a
libre2.a

I extract these static librarys, and write my jni code, just like 'android:tensorflow_demo'.
tensorflow::SessionOptions options;
tensorflow::ConfigProto& config = options.config;
LOG(INFO) << "Got config, " << config.device_count_size() << " devices";
tensorflow::Session* tempSession = tensorflow::NewSession(options);

here, config.device_count_size() is 0,  and tempSession is nullptr, internal it print
    "Not found: Could not find session factory for DIRECT_SESSION"

the device count is importance, but it's zero.

android-ndk-r10e-linux-x86_64.bin
bazel-0.2.1-installer-linux-x86_64.sh
git clone --recurse-submodules https://github.com/tensorflow/tensorflow -b r0.7
android_sdk_repository(
name = "androidsdk",
api_level = 23,
build_tools_version = "23.0.1",
path = "/home/gzzhang/android-sdk",
)
android_ndk_repository(
name="androidndk",
path="/opt/android-ndk",
api_level=11,                                        <<==  this is diff, i want to down the api_level.
)
bazel build //tensorflow/examples/android:tensorflow_demo --verbose_failures
Target //tensorflow/examples/android:tensorflow_demo up-to-date:
bazel-bin/tensorflow/examples/android/tensorflow_demo_deploy.jar
bazel-bin/tensorflow/examples/android/tensorflow_demo_unsigned.apk
bazel-bin/tensorflow/examples/android/tensorflow_demo.apk
INFO: Elapsed time: 2363.175s, Critical Path: 2126.57s

I just tried with api_level=11, but was unable to reproduce the issue.
If you update tensorflow/examples/android:libtensorflow_demo.so to have the following linkopts:
    linkopts = [
        "-landroid",
        "-ljnigraphics",
        "-llog",
        "-lm",
        "-z defs",
    ],

And then run:
bazel build //tensorflow/examples/android:tensorflow_demo --verbose_failures
unzip bazel-bin/tensorflow/examples/android/tensorflow_demo.apk -d apk
nm --radix=d --print-size -C --size-sort apk/lib/armeabi-v7a/libtensorflow_demo.so  | grep DirectSession

Do you see anything?
As a potential workaround, you can try adding alwayslink=1 to tensorflow/core:android_tensorflow_lib and see if that does anything.
How are you installing it on your device?

@andrewharp
"Not found: Could not find session factory for DIRECT_SESSION"
It seems that the DirectSessionFactory is not been regist.
So I browse the codes, and in the end of direct_session.cc, i find:
//////////
class DirectSessionFactory : public SessionFactory {
public:
DirectSessionFactory() {}
Session* NewSession(const SessionOptions& options) override {
std::vector<Device*> devices;
DeviceFactory::AddDevices(options, "/job:localhost/replica:0/task:0",
&devices);
return new DirectSession(options, new DeviceMgr(devices));
}
};
class DirectSessionRegistrar {
public:
DirectSessionRegistrar() {
SessionFactory::Register("DIRECT_SESSION", new DirectSessionFactory());
}
};
static DirectSessionRegistrar registrar;
//////////
today, i rebuild it with ndk api_level=9 and api_level=21
libandroid_tensorflow_lib.a
direct_session.o
nm libandroid_tensorflow_lib.a | grep DirectSessionFactory    OK
nm libandroid_tensorflow_lib.a | grep DirectSessionRegistrar  Found Nothing
nm libandroid_tensorflow_lib.a | grep registrar  Found Nothing
direct_session.o is same.  api_level=9 and api_level=21 is some.
so, you know, DIRECT_SESSION is not registed.
and then, "Not found: Could not find session factory for DIRECT_SESSION"
================================================
It really occur.  very very strange.

@andrewharp
It my fault, now it's ok.
I develop under Windows, and I use Android.mk to build.
I must use LOCAL_WHOLE_STATIC_LIBRARIES instead of LOCAL_STATIC_LIBRARIES to link these libraries. And it also depend on android-ndk-r10e, just like bazel build.
LOCAL_WHOLE_STATIC_LIBRARIES :=
android_tensorflow_lib
protobuf re2
protobuf_lite
protos_all_cc
Now I can develop APP under Windows with Eclipse and Android.mk.
thanks.


[support request/code review] Efficient use of Bolt

Hi,
I'm using Bolt in simplebolt. However, being new to Bolt, I am not fully confident that Bolt is being used efficiently. If someone could just look over the code and check if the use is reasonably efficient, that would be great.
Best regards,
Alexander Rødseth

Overall the usage looks good. Two suggestions (after taking a quick glance):

Be sure to check errors from View() and Update(). For example, there's a missing error check in NewList().

It's more efficient to use binary.BigEndian.PutUint64() instead of fmt.Sprintf() to represent ids. For one thing, it's much faster to encode & decode. Second of all, when you use a stringified representation then it won't insert sequentially. For example, the numbers 1,2,10 get sorted as 1,10,2 when you use fmt.Sprintf().

I hope that helps!

Thank you! I really appreciate it. Additional error handling is now in place and I'll be looking into your suggestion to use binary.BigEndian.PutUint64() instead.


Support viewing commits on github.com

As a non-sublime user, when clicking the external-link-symbolic icon, I can only see a "Unable to find a tool for viewing Git commits." dialog and I have to copy the commit id, and paste it in a browser or in git show.
I wonder if it can be possible to just fallback to open a github.com page in a browser when possible, I actually have https://github.com/bobby285271/nonemast/commit/24f671d9794946ad0438a766e97cde9eadead715 in my fork and it fits my needs quite well though I am always not sure how the proper UX should be :joy:

I think this is reasonable fallback but I want to make it not specific to Nixpkgs. Maybe we could use gh browse $commit command to make it more generic (but only when the remote is GitHub).

Or add support for custom actions in vein of #15


Remove expired instance metrics

What
We want to remove app instance metrics when apps were scaled down as these would cause additional noise which didn't make sense for teams.
In order to make the tests pass we had to consume appEventChan even though we were not expecting any messages on this channel.
Reference
https://trello.com/c/a9dNxa8u/684-paas-metric-exporter-doesnt-expire-metrics-when-scaling-an-app-down
Worked with: @philandstuff @JonathanHallam @issyl0

I think the fact that the channel should receive messages only if the app attributes we are interested in have changed shows that this would be a leaky abstraction

that is why maybe we need a new type: application info, which includes stuff that the senders might care about.

App guid.
Number of instances
Org, space, name and guid

We only send a message for updates on any of those attributes.
To construct a string with guid + instance is somehow encoding this new type in a string, so we are already leaking the abstraction. Also in the long run it is difficult to maintain.
A good example would be the concept of processes in applications that is in the v3 of the CF api. One application can have multiple different processes: web, api, etc. What would we do in that case? Encode strings as <app_guid>:<processname>:<instance_id>.
If, instead, we have a new app type, we can model that in the new type.

BTW, I found other potential bug in your code, a race condition:
Loggregator does not gurantee realtime metrics. Metrics can arrive with some delay, so you might get a metrics of CPU of an instance several seconds later.
That means that it is possible that you detect that an app is downscaled (e.g. instances 3 => 1) , you delete the map, but then you get a delayed metric of an instance  that should not be there anymore!
A possible solution is that you "keep track" of the number of instances for each application in the prometheus sender, and ignore any metric of an instance ID > than the expected number of instances.
If you do that, maybe you actually want to replace that <guid>:<instance> string with an struct with the app info including number of instances as suggested in https://github.com/alphagov/paas-metric-exporter/pull/35#discussion_r230683220, it would make it easier to track them.

@keymon We think we've addressed all of your earlier comments and would be good for a second look at this PR please.
Good spot in terms of the potential race condition.
For context the next story we are working on after this is the following:
## Why

The PaaS metric exporter does not instantiate a metric for the number of 5xx status code responses when the paas metric exporter is started. This means thats if you run an `increase` function on the number of 5xx status codes, you will not get the increase from `0` to `1` new 5xx status code but only from `1` upwards. This means 5xx is underreported.

## Goal

Metrics are initialised with a 0 value for `requests` and `response_time_bucket` metrics. This will mean that we don't miss the first count for any status code, not just 5xx.

We think that that story would be a good one that can solve the problem of the race condition at the same time by only creating new metrics when a new instance appears rather than on receiving from loggregator.
We think we can do this by changing getOrCreateAppMetrics to only get app metrics and not create them. Instead app metrics can only be created when a new instance is created in PrometheusSender.AppInstanceCreated.
I think our suggestion is therefore to let this bug get merged for the moment but tackle it in the next ticket which we will start working on immediately after.
What are your thoughts? Happy to discuss in person which might be easier

I think our suggestion is therefore to let this bug get merged for the moment but tackle it in the next ticket which we will start working on immediately after.

Happy with it meanwhile it is captured (as you did) :+1:

Regarding this https://github.com/alphagov/paas-metric-exporter/pull/35#issuecomment-435875966

that is why maybe we need a new type: application info, which includes stuff that the senders might care about.

@LeePorte and @richardTowers are now checking how to expose also services with this application. If this new feature is added, maybe we should update this channels to provide events about "updates" (create, update, delete) on "resources" (apps, services, routes).
I think it is OK in this PR to keep using a string with <guid>:<instance_id> but we might want to refactor later.


Disable Twitter and Imgur toggles

Todo

[x] Disable Twitter and Imgur toggles
[x] Add tooltip to disabled toggles

Motivation
The Twitter and Imgur post detection has not been implemented (and may not be for a while).
Disabling them is a better solution than removing them entirely because:

Removing them makes the extension UI look less complete. This is important for presentation purposes.
There would be whitespace where the Twitter/Imgur toggles are currently.
Some of the UI components would not make much sense e.g. the global toggle, "Enabled Sites", "Add Other Sites".

Summary of changes

Disable Twitter and Imgur toggles
Add tooltip to disabled toggles

Attached GitHub issue
Closes #27
Checklist
Documentation

[x] Updated the readme documents (README.md etc.)
[x] New functions and methods have additional comments added to document their usage

Local Build

[x] Extension has been tested to build correctly cd extension && yarn && yarn build

GitHub

[x] Target branch has been set correctly
[x] PR has been rebased onto target branch
[x] PR has been assigned an owner
[x] Author has performed a self-review of the code
[x] Removed the WIP message from the top of this PR

Screenshots
Hovering over the Imgur (and Twitter) toggles will show a tooltip

The Reddit toggle still works

Codecov Report

Merging #65 (6dd3dfa) into Release-1.0 (8f94ff8) will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff              @@
##           Release-1.0      #65   +/-   ##
============================================
  Coverage        75.19%   75.19%
============================================
  Files                8        8
  Lines              262      262
  Branches            41       41
============================================
  Hits               197      197
  Misses              42       42
  Partials            23       23

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


Initial stab at issue/1

Odds are this needs some work, but I think I got the basics working

Changes Unknown when pulling 8910f03d32a2d6fd00ba0b3070e66d7085d1d2b7 on jcpunk:issue-1 into ** on hercules-team:master**.

Changes Unknown when pulling 8910f03d32a2d6fd00ba0b3070e66d7085d1d2b7 on jcpunk:issue-1 into ** on hercules-team:master**.

Changes Unknown when pulling 8910f03d32a2d6fd00ba0b3070e66d7085d1d2b7 on jcpunk:issue-1 into ** on hercules-team:master**.

Changes Unknown when pulling 3f15a6c7362672112780e87d563c1b739ecc614d on jcpunk:issue-1 into ** on hercules-team:master**.

Changes Unknown when pulling db10bd74a27efe566655d6f53347b19ad87f4bcf on jcpunk:issue-1 into ** on hercules-team:master**.

Changes Unknown when pulling 3008981f9d8dcebc3e4586f1cf4b1242c969ac3b on jcpunk:issue-1 into ** on hercules-team:master**.

Changes Unknown when pulling ac2fda20185eed17f52e09a3b0ddbc0ddae82ca2 on jcpunk:issue-1 into ** on hercules-team:master**.


*: support roll update

What type of PR is this?

/enhancement
Which issue(s) this PR fixes?

Fixes #121
What this PR does?
Summary:
Upgrade the follower nodes first, and then upgrade the leader node last.
Special notes for your reviewer?

Fix :make run failed #149
ConfMapVolumeMountPath

LGTM


fix invalid *requirements*/*.{txt,in} discovery

Now it adds files from the root when tries to discover subfolders with requirements in name

Codecov Report

Merging #1016 into develop will not change coverage.
The diff coverage is 0%.

@@           Coverage Diff            @@
##           develop    #1016   +/-   ##
========================================
  Coverage    89.75%   89.75%
========================================
  Files           12       12
  Lines         2693     2693
  Branches       461      461
========================================
  Hits          2417     2417
  Misses         187      187
  Partials        89       89

Codecov isn't ok with zeros. I guess, because of floating-point comparation :upside_down_face:

Looks great, thank you!


Fix/issue 3542

@sivasu37 I fixed the merging conflicts.

@maxcapodi78 ,  Sorry, Yes I noticed the duplication. I will fix and comment on the PR later. Then we can merge.

I will create a new PR for the changes


Congestion cpp hashmap2

Improved approach to #7. This rebases the congestion branch against master, to bring in the improvements from master.

Codecov Report

Merging #8 into master will decrease coverage by 16.53%.
The diff coverage is 0%.

@@             Coverage Diff             @@
##           master       #8       +/-   ##
===========================================
- Coverage   60.73%   44.19%   -16.54%
===========================================
  Files           7       11        +4
  Lines         326      448      +122
  Branches        0        5        +5
===========================================
  Hits          198      198
- Misses        128      250      +122

Impacted Files
Coverage Δ

third_party/sparsepp/sparsepp/spp.h
0% <ø> (ø)

src/types.hpp
0% <ø> (ø)
:arrow_up:

src/hashmap.cpp
0% <0%> (ø)

src/hashmap.hpp
0% <0%> (ø)

third_party/sparsepp/sparsepp/spp_utils.h
0% <0%> (ø)

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 03785e8...02a3217. Read the comment docs.

closing, now a part of #5


Schedule for Monthly Community Meetings

The first community meeting (https://bluejeans.com/s/8vFc) was a tremendous success!  In the last meeting we decided to start an issue to get a majority vote on the date / time moving forward.
I will throw out an initial day to start and then we can iterate on the time.  How does the 4th Thurs. of each month sound?
Let's give times in UTC.

thanks....  i am flexible., but sure 4th Thur @ 3pm (EST) ==> 7pm (UTC)

+1 works for me too

@ehazlett , hi Evan, are we set then?

@ehazlett any updates on this?  any news on the docker-machine<-->shim<--->mypluggin?

Schedule will be here: https://github.com/docker/machine/wiki


avoid invoking sudo when polling

catch PermissionError as process exists
closes https://github.com/jupyterhub/jupyterhub/issues/1128

@parente, can we do a release with this PR?..
the other potential issue i found is that when random_port generated already occupied, the process would get into error loop

@nfx just pushed 0.4.1 with this fix


./bin/extract-qt-installer not finish

In ubuntu20.04 WSL | Windows 10
After run command:
sudo ./bin/extract-qt-installer /root/qtci/qt-opensource-linux-x64-5.12.0.run /root/Qt

The extract-qt-installer keeps on [18] QTCI:  Welcome Page.
Is this an error ? How can i finish de extracting and finally use Qt ?

for me, stuck in  Retrieving meta information from remote repository
greetings


[Improvement][Master] Master overload will delay scheduling of high priority process

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
We have the conception Process priority:

Process priority: The priority of process operation, divide into five levels: highest (HIGHEST), high (HIGH), medium (MEDIUM), low (LOW), and lowest (LOWEST). When the number of master threads is insufficient, high priority processes will execute first in the execution queue, and processes with the same priority will execute in the order of first in, first out.

However, in cluster mode, when we have multi-master nodes, it's possible that one master node becomes overload and do not consume new command (including some process with the highest priority). Some processes with low priority handled by other Master may run earlier. Thus, the actual behavior may not be same with priority order.
Three ideas about this:

The first one is directly declare in doc about this scenario (not recommend).
The second one is when we judge overload for five(configurable) consecutive rounds, current master should fetch commands and send those commands with higher priority to other Master nodes. In a word, after introducing this mechanism, we combine push-based with pull-based way to consume commands in Master.
The third one is to introduce a coordinator, each master can fetch commands from coordinator, and replace current slot-based consuming logic.

And there's an existed solution proposed in https://github.com/apache/dolphinscheduler/issues/14271
However, on my side, scale-down is a little inappropriate, slot changes is a relatively heavy step, in DS current design, getting aware of slot changes can only guarantee eventual consistency, which means that several master may consume same commands, although we declare some transaction to make sure command will not be re-consumed, it still cause a waste of resource.
Some other related work
In addition, maybe auto-scaling Master nodes can also help, I think we can do it in later PR cuz it's a cloud-native feature. Although scale-up will also cause slot changes, we have more Master nodes and increase the computing capacity to share the pressure, which means it will bring more advantage than negative effect.
Anyway, we do have to figure out some way for current design flaws (for those deploy in cluster mode and do not have any scalable ability). And I think the solution for this issue and feature of auto-scaling Master nodes can CO-EXIST.
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

cc @ruanwenjun

After some discussion, I will add overload states in master heartbeat first (slot change solution) which makes the minimal change.

After some discussion, I will add overload states in master heartbeat first (slot change solution) which makes the minimal change.

Yes, we need to redesign the master heartbeat info, and add server status in master heartbeat.


Commits to master bypassing a PR

Please, not commit to master branch bypassing PR, because it can lead to not valid master branch and also skip all code quality tests and potential code review.

This is currently not possible. But we will do the work resolving PR merge conflict. Please stay tuned regrading your pending merge request - we work on it and they are scheduled for the next upcoming release (planned for end of this month)

The problem with not a merge conflict. The problem with code quality. You can not guarantee quality of your code without code review, tests and so on. All of this points not work if you commit to master branch by passing a PR.


Fix typo in developer documentation.

What this PR does / why we need it:
This PR fixes a typo in the documentation that might potentially discourage some contributors.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

/lgtm
/ok-to-test
/approve

Thanks very much for picking up these issues in our development guide! This, plus your other PR must lead to a frustrating start!

/ok-to-test
/release-note-none


Create Least Number of Unique Integers after K Removals.java

Copied from Leetcode

That's my solution that I have written
On Sat, 3 Oct, 2020, 1:24 pm Archit<EMAIL_ADDRESS>wrote:

Copied from Leetcode
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/gargarchit/Data_Structure_n_Algorithms/pull/30#issuecomment-703064076,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJJQ637HSVHMUXDNWFXKK2TSI3KBRANCNFSM4SCOSNAQ
.


druid最新代码解析clickhouse array join 有误

druid 最新代码 1.2.24 snap
SQLStatementParser parser = SQLParserUtils.createSQLStatementParser(sql, DbType.clickhouse);
List sqlStatements = parser.parseStatementList();
for(SQLStatement sqlStatement:sqlStatements){
if(sqlStatement instanceof SQLSelectStatement){
SQLSelectStatement sqlSelectStatement = (SQLSelectStatement) sqlStatement;
System.out.println(sqlSelectStatement);
}
}
解析前：
SELECT
capture_time,
resource_uid,
service_uid,
s_cost,
cost
FROM base_customized_cost
LEFT ARRAY JOIN
split.service_uid AS service_uid,split.cost AS s_cost
WHERE
capture_time >= DATE('2024-03-19')
AND capture_time < DATE('2024-04-01')
AND ler_3 = '1'
AND middleware_type = ''
AND resource_uid = 'b3b1f4a7a7ce4c7ab9ff7619c9f0eac0'
AND service_uid = 'f874cc2a3e934e488769eb9e2d82479e'
ORDER BY capture_time ASC
解析后：
SELECT capture_time, resource_uid, service_uid, s_cost, cost
FROM base_customized_cost
LEFT ARRAY JOIN split.service_uid service_uid, split.cost s_cost
WHERE capture_time >= DATE('2024-03-19')
AND capture_time < DATE('2024-04-01')
AND ler_3 = '1'
AND middleware_type = ''
AND resource_uid = 'b3b1f4a7a7ce4c7ab9ff7619c9f0eac0'
AND service_uid = 'f874cc2a3e934e488769eb9e2d82479e'
ORDER BY capture_time ASC
去除了 array join 的 as 导致sql执行报错
报错信息：
Syntax error: failed at position 122 ('service_uid') (line 3, col 35):
SELECT capture_time, resource_uid, service_uid, s_cost, cost
FROM base_customized_cost
LEFT ARRAY JOIN split.service_uid service_uid, split.cost s_cost
WHERE capture_time >= DATE('2024-03-19')
AND capture_time < DATE('2024-04-01')
AND ler_3 = '1'
AND middleware_type = ''
AND resource_uid = 'b3b1f4a7a7ce4c7ab9ff7619c9f0eac0'
AND service_uid = 'f874cc2a3e934e488769eb9e2d82479e'
ORDER BY capture_time ASC;
Expected one of: UNION, LIMIT, WHERE, WINDOW, DoubleColon, INNER, LIKE, GLOBAL NOT IN, end of query, HAVING, AS, DIV, IS, UUID, GROUP BY, INTO OUTFILE, OR, QuestionMark, BETWEEN, OFFSET, NOT LIKE, MOD, LEFT ARRAY JOIN, PREWHERE, AND, JOIN, Comma, alias, ORDER BY, array join, SETTINGS, IN, ARRAY JOIN, ILIKE, FORMAT, Dot, NOT ILIKE, WITH, NOT, table, table function, subquery or list of joined tables, token, NOT IN, GLOBAL IN

优化逻辑增强了解析。 #5955


[BUG] Starting App in Parallels Running Windows 11

The app won't start up.  Uncaught rejection - quitting
Caught rejection: Error: no such file or directory, watch 'C:\Users{USERNAME}\log.txt'
Origin: [object Promise]
To Reproduce
Steps to reproduce the behavior:

Start Parallels Version 18.1.1 (53328)
Open FSModAssistant-..any version of 2.0
See error.

Desktop (please complete the following information):

OS: Windows 11 running in Parallels v.18.1.1 (53328)

Added to list. Probably a bug fix mid-day will be released

This should be corrected in 2.0.1, if it's not, please let me know.

That was quick!  Nice work.  No errors on startup.

Excellent, thanks!


Feature/adding wac module

This is a module that wraps the WAC api. This module could evolve to be a our official WAC SDK for android.  Call<SendCodeResponse>
}

This is a sample app that uses this WAC module.

@RioVic the wrapper of the api is completed now.
It is implemented in a separated module with his own demo app. It is already in a shape where it can be announced as our official android WacSDK to allow any wallet to use our ATMs. It might need some minor documentation.


redux-logic-test

react-boilerplate-logic
Would it be possible to add redux-logic-test unit test as a demo in this boilerplate ?
Issue Type

[ ] Bug (https://github.com/jeffbski/react-boilerplate-logic/blob/master/.github/CONTRIBUTING.md#bug-reports)
[x] Feature (https://github.com/jeffbski/react-boilerplate-logic/blob/master/.github/CONTRIBUTING.md#feature-requests)

yes, that would be a great idea.


coreos / bash

is... this only meant to be used with something that comes with docker and bash pre-installed?
what about using it with coreos?

I've not tried this with coreos. I will take a look and see if it works ok or if it can easily be adapted to work with it.

It works fine with CoreOS. I didn't realise when I wrote my earlier reply that the default image actually is coreos-alpha (It's been a little while since I last looked at it).
But yes, it does require that docker and bash are pre-installed. It also requires that the Swarm communication ports are opened on the machine's firewall, e.i. iptables, firewalld or ufw.
Otherwise, you can install Docker using user-data - the provision script will wait until the Docker daemon is up and running. This is an example using centOS 7 and a bash script as user data to install Docker CE.
Or prepare a snapshot on DigitalOcean which comes with everything pre-installed and pass that as the image (how I run the module myself).

<3

installed on the dev machine, not the cluster? I know docker would need to be installed on the cluster. - Thank you for answering my question :D

Docker does not need to be installed on the dev machine, unless you want to use the Docker CLI to connect to a remote docker API.


Fix infura mainnet URL in README

Noticing this URL is gmainnet in other places as well. Intentional? Should be fixed?

the "g" tells infura to request from a geth node as opposed to parity (which might have a slightly different list of RPC requests it supports)


[MBL-1854] Fix reward filtering

📲 What
Fix reward filtering by location by a) fetching expanded shipping rules for each reward and b) using the expanded shipping rules to filter.
👀 See
Jira

Before 🐛
After 🦋

todo
todo

✅ Acceptance criteria

[x] Tested filtering for a reward with restricted shipping to the EU, and it shows up when an eu country is selected with these changes
[x] Performance is a little slower ("back this project" button takes longer to load) but it's not terrible

⏰ TODO

[ ] The rewards carousel with shipping location has no test coverage :( Either I need to add comprehensive test coverage in this pr, or file a pledge redemption tech debt ticket and make sure that ticket is prioritized.

          1 Warning

:warning:
Big PR

  Generated by :no_entry_sign: Danger


#91 issue and other minor fixes

Replace null with friendly text in home page
Removed window.location.origin to support localhost
Added randomcolor.js to fix color issue
#91  Apache 2.0 license added

@shalithasuranga Thanks for the PR. Few changes required.

Make the title bit descriptive
We are adhering to MIT Licence. So replace APACHE with MIT.

I will add the rest as inline comments

Changes
Added missing language labels C# and PHP
Fixed preloader gif animation
CONTRIBUTING.md ws added
Thanks

I uploaded my version of this repo to a server please check
http://pclmarketing.com/software/hacktitude/repos/
compare with your version http://opensource.99xtechnology.com/ with PR commits
and advice me about committing this PR (3 days old)
Thanks
@rehrumesh  @lakindu95

@shalthasuranga please be mindfulll that we are getting large number of pull requests these days. Team is putting lot of effort to review everything. So be patient. None of your contributions will be disrigarded


Support scheduler-port, worker-port and nanny-port args

@kmpaul
Does this seem like the right approach?
Note: I was only able to test with no-nanny, as the Nannies don't seem to be forking Workers in my tests. I believe our installation of MPI allows this. But perhaps this is for another issue.

Codecov Report

Merging #27 into master will increase coverage by 0.49%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #27      +/-   ##
==========================================
+ Coverage   93.44%   93.93%   +0.49%
==========================================
  Files           4        4
  Lines         122      132      +10
==========================================
+ Hits          114      124      +10
  Misses          8        8

Impacted Files
Coverage Δ

dask_mpi/common.py
87.93% <100%> (+1.65%)
:arrow_up:

dask_mpi/cli.py
97.05% <100%> (+0.28%)
: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 f96716f...a4de593. Read the comment docs.

This seems like the right approach.  It's not clear to me why the tests are failing at this point.

I am realizing now that the worker-port and nanny-port command-line args will only work correctly when one launches exactly one worker per node on the cluster. While I'm guessing this is the most common configuration, should this be documented explicitly?
Ideally both options would support a range of ports (https://github.com/dask/distributed/issues/899).

@shwina Nice job!  I think this looks good.  Considering your last comment about documentation, though, would you like to take a crack at updating the documentation, too?  As part of this PR?  Everything should be in the docs/source directory of the repo.

@shwina  ...Incidentally, comments specifically related to the command-line arguments, though, should probably go in the help strings in the click.option arguments of the command-line tool.

Thanks! Will do

Yes.  That would be helpful.  Thank you.
On Tue, Feb 26, 2019 at 5:19 PM Ashwin Srinath<EMAIL_ADDRESS>wrote:

@shwina commented on this pull request.
In dask_mpi/tests/test_cli.py
https://github.com/dask/dask-mpi/pull/27#discussion_r260558975:

with tmpfile(extension="json") as fn:

   cmd = mpirun + ["-np", "2", "dask-mpi", "--scheduler-file", fn, nanny,

                   "--scheduler-port", "56723",

                   "--worker-port", "58464",

                   "--nanny-port", "50164"]

   with popen(cmd):

       with Client(scheduler_file=fn) as c:

           start = time()

           while len(c.scheduler_info()["workers"]) != 3:

               assert time() < start + 10

               sleep(0.2)

           assert c.submit(lambda x: x + 1, 10, workers="mpi-rank-1").result() == 11

Thanks! I can confirm that the ports are indeed used - should I go ahead
and add this to the test?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask-mpi/pull/27#discussion_r260558975, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AASszHh8HoOmhzc9V8JxZd72qYhGYhn0ks5vRd0WgaJpZM4bSozr
.

@kmpaul Would it be OK to submit the documentation addition in a separate PR? More generally, I'd like to add a section on how to request resources from the cluster when using Dask-MPI

@shwina That's fine.  I think this is working, now, so thank you very much for this PR!


Add config for multi language

What should happen ❓
Need a config file to set content there. This file must be a JSON to translate to other languages.

This section done after release 1