empty-line-between-blocks issues

Sass-lint 1.11.1 (used by gulp-sass-lint) does not seem to validate properly.
The empty-line-between-blocks throws warnings about what I believe to be valid according to how the docs say the config option is suppose to work.
  empty-line-between-blocks:
    - 1
    - include: true
    - ignore-single-line-rulesets: true
===========================
src/scss/options.scss
   31:1  warning  ID selectors not allowed                                no-ids
   41:1  warning  Space expected between blocks                           empty-line-between-blocks
   44:1  warning  Space expected between blocks                           empty-line-between-blocks
   48:1  warning  Space expected between blocks                           empty-line-between-blocks
   80:1  warning  Space expected between blocks                           empty-line-between-blocks
   85:1  warning  Space expected between blocks                           empty-line-between-blocks
   93:1  warning  Space expected between blocks                           empty-line-between-blocks
   97:1  warning  Space expected between blocks                           empty-line-between-blocks
  101:1  warning  Space expected between blocks                           empty-line-between-blocks
  111:1  warning  Space expected between blocks                           empty-line-between-blocks
  123:1  warning  Space expected between blocks                           empty-line-between-blocks
  123:5  warning  Nesting depth 4 greater than max of 3                   nesting-depth
  127:1  warning  Space expected between blocks                           empty-line-between-blocks
  127:5  warning  Nesting depth 4 greater than max of 3                   nesting-depth
  132:1  warning  Space expected between blocks                           empty-line-between-blocks
  132:4  warning  Type-selector should be nested within its parent Class  force-element-nesting
  137:1  warning  Space expected between blocks                           empty-line-between-blocks
  137:5  warning  Nesting depth 4 greater than max of 3                   nesting-depth
  143:1  warning  Space expected between blocks                           empty-line-between-blocks
  144:5  warning  Expected indentation of 3 tabs but found 4              indentation
  145:5  warning  Expected indentation of 3 tabs but found 4              indentation
  146:5  warning  Expected indentation of 3 tabs but found 4              indentation
  149:1  warning  Space expected between blocks                           empty-line-between-blocks
  156:1  warning  Space expected between blocks                           empty-line-between-blocks
  160:1  warning  Space expected between blocks                           empty-line-between-blocks
  187:1  warning  Space expected between blocks                           empty-line-between-blocks
  191:1  warning  Space expected between blocks                           empty-line-between-blocks
  199:1  warning  Space expected between blocks                           empty-line-between-blocks
  215:1  warning  Space expected between blocks                           empty-line-between-blocks
  264:5  warning  Nesting depth 4 greater than max of 3                   nesting-depth
  272:5  warning  Nesting depth 4 greater than max of 3                   nesting-depth
  288:1  warning  Space expected between blocks                           empty-line-between-blocks
  294:1  warning  Space expected between blocks                           empty-line-between-blocks
  298:1  warning  Space expected between blocks                           empty-line-between-blocks

✖ 34 problems (0 errors, 34 warnings)

Here's line 32 through 52
&.collapsed {
	.menu-text {
		.expand {
			display: inline-block;
		}

		.text, .collapse {display: none;}
	}

	.sidebar {
		width: rem-calc($sidebar-collapsed);

		.name {
			display: none;
		}
	}
	.main-content, .header-content {
		left: rem-calc($sidebar-collapsed);
	}
}

And here's 149 through 162
.name {
		@include breakpoint(800px down) {
			display: none;
		}
	}
}

&-content {
	border: 0;
}

&-panel {
	padding: 0;
}

So, for example. If I remove the empty line between the closing .menu-text block and the .sidebar block it doesn't throw a warning about it. Also if I set include to false it also doesn't warn about the empty line between them.

Thanks for this and the detailed report, will look into it.

The line between the blocks has a tab in it, and therefore isn't empty...
I wish there was a way to allow tabs - maybe there is.

I'm also having this issue. When I hit enter to insert a new line, a tab is also inserted by my code editor to place the cursor at the same indentation level as the previous line. I guess the current behaviour is correct in the sense that whitespace is not the same as empty in empty-line-between-blocks, but it's inconvenient to the extent that I prefer to have the rule disabled.

So we could add an option to ignore whitespace on the new lines if that's what is required.

@DanPurdy will this be implemented any time soon?  My editor inserts the cursor at the same indentation when I make a new line, but I would love it if the linter would not treat this as white space, but instead a new line (I rather keep my soft tabs instead of hard tabs).

here's another false positive for empty-line-between-blocks:
@for $i from 1 through 6
  h#{$i}
    color: $color-text-default

p // empty-line-between-blocks here
  margin-bottom: 0


Nether Portals Launch people

Whenever Non-OP people come into close proximity with a netherportal, they are hurt and are launch some 5-20 blocks away.

Closing issue because it's an old post. If you are still running a Minecraft server and this persists, try updating all the Multiverse plugins and try again, otherwise please don't hesitate to ask for support on our Discord (quickest support) or submit a new issue.


[Macros] Mangle the names of macro expansion buffers

Introduce a name mangling scheme for the names of macro expansion buffers, so macro expansion buffer names have a few desirable properties:

They are uniquely-determined, so different expansions will have different buffer names
They benefit from being demangleable to figure out where the macro expansion actually was
They are resilient to changes in the source file that are independent of the macro expansion itself, e.g., they don't use line/column information
They are recognizable via the new prefix @__swiftmacro_, which is sufficiently ugly/recognizable that other files won't step on them, and that prefix can be replaced with $s for demangling purposes.

This is the majority of the infrastructure for rdar://104038303. However, it needs to be expanded out to attached macros.

@swift-ci please smoke test


create and use shorthands/aliases

similiar to aliases in bash there are recurring tasks. example prompt: 'sgpt "Generate git commit message, my changes: $(git diff)"'
make it possible to add shorthands or aliases to reference and reuse existing prompts.

one could use bash aliases to mimic this behaviour

examples are collected in .bash_aliases

use bash aliases (e.g. https://github.com/tbckr/sgpt/blob/c688f5d10c764f2d9aa0a1f8152685bc2d79d6e1/.bash_aliases)


Update to json data format, add filesystem

Q
A

Is bugfix?
❌

New feature?
✔

Breaks BC?
❌

Tests pass?
✔

Merge https://github.com/yiisoft/var-dumper/pull/6 first.


Updates swiftlint builtins to check exit code

Swiftlint returns a 2 exit code when it detects errors in your code, I've added a check to account for that. Currently when trying to use it for diagnostics, none-ls assumes it has failed.

LGTM. Thanks!


Fixes avatar cache bug

@RocketChat/android

Closes #2040
Changes: Replaces username with corresponding imageUri

@divyanshub024 PTAL!

Is this fixing #1217 as well?

This PR is all about updating locally cached avatar whenever it is updated by the device itself @rockneverdies55 .
Add I have already started working on #1217 and will soon come up with a PR or a conclusive statement on it.

@dev-ritik thank you very much!! I can confirm that this solved #2040 for me. This does not solve #1217 though.


Support right-to-left and other orientations

Currently we do support following directions of packing

"rightDown" - find place on right, if none go down /
horizontal layers left-to-right
"downRight" - find place bellow, if none go right /
horizontal layers left-to-right

It would be nice to support at least left-to-right direction: for RTL languages, though layouts, for compatibility with bootstrap/flexbox/flexgrid demos (Juicy/juicy-tile-grid#13) ;)
Bottom up seems awkward, but why don't we support it as well.

After brief investigation it seems, that even if we could add new sorters might be cool, and gives opportunity to provide very weird orders. It only chooses slot to pack, and then place a rectangle in  same x,y as slot, so conceptually in top left corner.
Then I think it makes no point of changing packing algorithm, as from it's point of view positions, angle, direction, and interpretation of X  & Y axes does not matter, and we should not mess the code and increase performance costs.
The think we would like to achieve is completely visual issue, so we can rotate, mirror, flip the axes as we would like.
Therefore I would

[x] 1. leave current "downRight" and "rightDown" sorters separated in Packer as they are, as choosing between them is cheap, however I would rename them to

"horizontal", "vertical",
"xTy", "yTx", or
"x_then_y","y_then_x" (Package, and Rectangle uses only abstract x,y

[ ] 2. implement additional property:

flag "leftToRight": true,
string property "horizontalDirection": "RTL"||"LTR"

[ ] 3. implement additional property:

flag "bottomUp": true,
string property "verticalDirection": "BU"||"UTB"

WDYT @miyconst? Any opinion on name to choose?

I've added experimental support for .leftToRight and .bottomUp to the gh-pages branch

I like the first variant of all points.

"direction": String ("horizontal", "vertical").
"leftToRight": Boolean.
"bottomToTop": Boolean.

With "horizontal", "vertical" the question is which is which ;),
I would say, that current default is "horizontal" as it lays horizontal layers, and grow in horizontal direction first.

Agree.

A bug found:
This is how page looks with "rightToLeft": false:

And this is how the same page and setup looks with "rightToLeft": true:

It seems to be a bug in juicy-tile-grid only, so I created an issue there https://github.com/Juicy/juicy-tile-grid/issues/14


Add blockingProcesses to garminexpress

Apparently Garmin Express also has a Service process, which I forgot in min initial PR.
Installing the pkg with Installomator doesn't seem to quit the service process, so it needs to be quit before installation.

Thank you!


Adding new category and CityStrides

A web platform that encourages the user to run every street in their city. Based on your Strava data it shows where you have and have not run, provides challenges, a leaderboard, and a forum.

Failed lint-check because I did not add entry in TOC. Will try again.


Does it support the following GPU: Sun XT [Radeon HD 8670A/8670M/8690M / R5 M330 / M430 / R7 M520]

I followed all the steps mentioned in the description.
But while running "rocminfo" command, I got the following error
hsa api call failure at line 900, file: /home/jenkins/jenkins-root/workspace/compute-rocm-rel-1.8/rocminfo/rocminfo.cc. Call returned 4104
Any suggestions what to do further?
My target is to use tensorflow with my GPU.
Thanks

Sun XT (Hainan) is GFX6, which is not supported in ROCm, We have experimental support for GFX7, but GFX8+ is the only official support for GPUs (see https://rocm.github.io/ROCmInstall.html for more info). The kernel will work, but the runtime (and rocminfo) doesn't have gfx6 support.

From what I can see your chip (Sun XT) is first Gen GCN (gfx 6xx) and not supported. Unfortunately there is very little info around on it, but it might be second gen GCN (gfx 7xx) if it is then ROCm can be used but Tensorflow cannot.
I am sure that sadly you can NOT use ROCm Tensorflow. To use Tensorflow you need a gfx 803, 900 or 906 GPU. No others and supported and if you use something else then Tensorflow will use the CPU instead.
The supported GPU list is meant to be updated at sometime but nobody has done it yet. Also more GPU support looks like it will or was going to be added to HC but it's very unlikely this will ever mean Tensorflow working on gfx7xx Chips.


Rename "quadratic voting" vote type as sybil-protection isn't implemented

"Quadratic voting" as it's being defined by academic papers and e.g. on Wikipedia has a particular quality to it as it's supposed to address the tyranny of the majority. The voting mechanism described in Radical Markets and elsewhere makes an implicit assumption that each identity gets to vote once with a fixed number of voting credits.
However, from what I'm understanding when testing snapshot's implementation (this one: https://github.com/snapshot-labs/snapshot/blob/25907c5da4b77462eb65814bd960e525640da3df/src/helpers/voting/quadratic.ts), then the snapshot system itself isn't capable of protecting against identity fraud during votes.
E.g. if a snapshot vote is created with the vote type being quadratic, but e.g. arbitrarily distributed ERC20 token balances are used as a strategy, then the vote shouldn't indeed be called a "quadratic vote".
It's because then, any ERC20 holder is free to distribute their tokens to as many Ethereum addresses that they want before the "snapshot", meaning that big ERC20 holders can have an even higher impact on the vote through creating "Sybils" ("cheating").
I think it is hence important to signal to all users that indeed the snapshot vote type "quadratic" isn't a "quadratic vote", but rather a vote tally algorithm that - for each voting issue, takes the square root of the voting result and assumes it as the outcome.
The similarity to a "quadratic vote" is that the same mathematical function is used in the tally. The name "quadratic vote" is already taken by the idea proposed by Weyl and Lalley in Radical Markets (e.g. see here: https://en.wikipedia.org/w/index.php?title=Quadratic_voting&oldid=1049808681). A quadratic vote makes no fragile assumptions about identities. It guarantees that each person can only vote once and hence is a solution to the tyranny of the majority. Square rooting the vote tally isn't a solution to the tyranny of the majority.
Square rooting the vote tally can also have benefits in voting as e.g. it limits the impact of honest whales. But without Sybil protection and the assumption that 1 address = 1 identity, it's fragile and "unsafe" according to the safety property in distributed systems.
I suggest renaming the vote type to e.g. "vote tally square root".

@TimDaub Agree on this, the quadratic voting on Snapshot should not be used with just token holding, there should be some sybil resistance with it. This is something you can implement with voting strategies, sybil resistance mechanisms are very opinionated, we have strategies using NFT, BrightID, Colony reputation, we also working with Orange protocol with others mechanism learn more here https://twitter.com/OrangeProtocol/status/1453298869077430274

There are a growing number of solutions POH, BrightID, WorldCoin, soulbound NFT's. It would be nice to see this as a more dominant feature on Snapshot.

Hey @samuveth would you be up for scoping the solution for this issue?

This is basically solved by voting validation.


Added the template tutorial

Harness Developer Pull Request
The PR adds a tutorial on creating and using Harness Pipeline Templates for CD
What Type of PR is This?

[ ] Issue
[X] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[x] Tested Locally
[ ] Optional Screen Shoot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app


Allow to toggle RPM repo configuration on openldap_install resource

Description
On RHEL >= 8 the resource is configuring the OSUSOL repository to support openldap-server install.
However in some cases you may not need this specific repository.
Now by setting install_repository false on the resource, the repo won't be installed.
Also the openldap.package_install_repository attributes allow to control the value set to this property in the openldap::default recipe.
Default behaviour is preserved, and RHEL >= 8 servers will continue to install the repository.
Issues Resolved
None created.
Check List

[x] A summary of changes made is included in the CHANGELOG under ## Unreleased
[ ] New functionality includes testing.
[x] New functionality has been documented in the README if applicable.

Released as: 6.1.0

So efficient @xorima , thanks!


IP Access Check for client_credentials login

Repro
Create the following access policy
{
  "resourceType": "AccessPolicy",
  "name": "My Access Policy",
  "ipAccessRule": [
    {
      "value": "<IP_ADDRESS>", // not a real IP
      "name": "Allowed IP",
      "action": "allow"
    },
    {
      "value": "*",
      "name": "Default Block",
      "action": "block"
    }
  ],
  "resource": [
    {
      "resourceType": "Practitioner"
    }
  ]
}

Create the following ClientApplication
{
  "resourceType": "ClientApplication",
  "name": "My Client",
  "pkceOptional": false,
}

Set the access policy of this ClientApplication to the access policy above.
Use the CLI to login
export MEDPLUM_CLIENT_ID=<CLIENT_ID>
export MEDPLUM_CLIENT_SECRET=<CLIENT_SECRET>
medplum get Practitioner

Expected Behavior
Reject login based on incorrect IP
Actual Bevahior
Successful response

Closed via #4137


Bump NAN to 2.2.0

Hello,
Can NAN be bumped to 2.2.0? On newer node versions the build fails with:
gyp info spawn args [ 'BUILDTYPE=Release', '-C', 'build' ]
make: Entering directory '/Users/fritz/PP/brand-activation/interface-activity/node_modules/zmq/build'
  CXX(target) Release/obj.target/zmq/binding.o
In file included from ../binding.cc:38:
../../nan/nan.h:590:20: error: no type named 'GCEpilogueCallback' in 'v8::Isolate'
      v8::Isolate::GCEpilogueCallback callback
      ~~~~~~~~~~~~~^
../../nan/nan.h:596:20: error: no type named 'GCEpilogueCallback' in 'v8::Isolate'
      v8::Isolate::GCEpilogueCallback callback) {
      ~~~~~~~~~~~~~^
../../nan/nan.h:601:20: error: no type named 'GCPrologueCallback' in 'v8::Isolate'
      v8::Isolate::GCPrologueCallback callback
      ~~~~~~~~~~~~~^
../../nan/nan.h:607:20: error: no type named 'GCPrologueCallback' in 'v8::Isolate'
      v8::Isolate::GCPrologueCallback callback) {
      ~~~~~~~~~~~~~^
4 errors generated.

Using node 5.9.1 currently
Thanks!

I'll have a look at it, thanks for reporting.

cc @kkoopa

Any update? Currently I just alter the package.json file of zmq to use nan 2.2.0

I'm not really near a PC for 3 weeks starting today, so if you could make a PR I could happily merge it even from a phone, but otherwise don't expect much from me for the next 20 days, sorry :(

https://github.com/JustinTulloss/zeromq.node/pull/520

When will this change been pushed to npm? It's broken with Node v6 if you installed from npm.


Fix visual jitter when card is dropped into a new column

Example of jitter:

Fixed in https://github.com/joekrump/retro-board/pull/44


Allows you to add customizable fields to the event

Codecov Report

Merging #7 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##             master     #7   +/-   ##
=======================================
  Coverage       100%   100%
- Complexity      288    292    +4
=======================================
  Files            45     46    +1
  Lines           817    827   +10
=======================================
+ Hits            817    827   +10

Impacted Files
Coverage Δ
Complexity Δ

src/Transformers/Events/OrderTransformer.php
100% <100%> (ø)
1 <0> (ø)
:arrow_down:

src/Transformers/Events/ChargebackTransformer.php
100% <100%> (ø)
1 <0> (ø)
:arrow_down:

src/Events/BaseEvent.php
100% <100%> (ø)
13 <2> (+2)
:arrow_up:

src/Transformers/AbstractTransformer.php
100% <100%> (ø)
2 <2> (?)

src/Transformers/Events/AccountTransformer.php
100% <100%> (ø)
1 <0> (ø)
:arrow_down:

src/Transformers/Events/TransactionTransformer.php
100% <100%> (ø)
1 <0> (ø)
:arrow_down:

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 1ae257d...3d30a50. Read the comment docs.


<EMAIL_ADDRESS>not in NPM

I see in the release notes for the repo that fixes were provided in<EMAIL_ADDRESS>yet when I check npm it only has 5.15.9. I'm assuming there was a release failure?
Search keywords:

The change mentioned in @mui/system has been wrongly tagged. It belongs to the Docs section of changelog. There weren't any changes in @mui/system in this release.


feat(python): initial setup for new API client

🧭 What and Why
🎟 JIRA Ticket: https://algolia.atlassian.net/browse/DI-1667
Changes included:
Successor of https://github.com/algolia/api-clients-automation/pull/2221
This PR initializes the Python API client with the:

most basic code generation (no cleanup or anything done yet)
CI support
scripts (generate, build, format)

CTS is skipped for now
In the next PR I will:

Add a playground for each client
Do some cleanup on the templates
Add the Algolia transporter for the retry strategy

huge ! I didn't review the mustache, I'm not familiar with api clients in python, but I wasn't expecting the generated code to be so verbose !

there's a loooot of cleanup to do so I'm iterating on this part right now in a subsequent PR, hopefully it will make things easier to understand


Warn the user if the minimum coverage is set greater than 100

I was trying something out, and I thought that it would be funny/useful to point this out to the user

Ugh just saw the JRuby failure, I'd guess/hope that it's not related I'll rerun that and otherwise will have to do some testing.

Yeah, I don't know what could've gone wrong with JRuby

failure in JRuby is persistent... I'll open a mock PR without changes to see if that also fails and then try to fix it. Sorry for the hassle :(

It seems that rvm is not identifying <IP_ADDRESS>?

<IP_ADDRESS> yeah, my bad as charlie commented it's fixed there I just tried it but it's not released. I set the JDK version instead. Works, I'll be brave and emrge this :)

Ah, sorry, hehe. Thank you very much!


added screenshots of examples

this essentially closes #93.. Actually I added screenshots but did not change the readme.

I signed, I used the github username feature.

Thanks for the pull-request @waynenilsen. I'm looking to put these screenshots on a GXUI webpage somewhere as I doubt anyone would actually want to grab these screenshots when they checkout the source.
I'll let you know when that's sorted out.

Its possible to just make another repo called gxuiscreenshots then cross reference them in the readme.

Or a branch called screenshots.

@ben-clayton you can just add them to the README (like this) using Markdown:
![Alt text](rel/path/to/image.png)

Then, they appear in the README

@ben-clayton you can just add them to the README (like this) using
Markdown:
![Alt text](rel/path/to/image.png)

Then, they appear in the README

Reply to this email directly or view it on GitHub:
https://github.com/google/gxui/pull/134#issuecomment-149044262

Did you read what he said? People probably don't want to clone the
screenshot files.

How about uploading these to imgur and change the readme to include external links to the images?

I generally create an orphan "images" branch for images in Readme,s and what not.


Feature/profile page

First draft of private profile page
none of the buttons are clickable yet. Profile page will get revised as app continues to get developed.

remove packages/js/app/assets/user.jpeg

Why are the assets needed?

Same here, please avoid raster graphics


Use ConnectionAttempts instead of while loop in bootstrap

This would avoid printing a lot of lines when the script is waiting for bootstrap node to come up

Build SUCCESS, see build http://<IP_ADDRESS>:8080/job/dev-tools/148/

Build SUCCESS, see build http://<IP_ADDRESS>:8080/job/dev-tools/150/


Update feature gate info for Service and Node exclusion labels

These features are beta in 1.19 and the legacy gate will be disabled
by default starting in 1.20.
Moved from #21921 due to GitHub not updating the PR text on push.

/assign
/milestone 1.19

Page preview:
https://deploy-preview-22535--kubernetes-io-vnext-staging.netlify.app/docs/reference/command-line-tools-reference/feature-gates/

Hi @smarterclayton 👋 Can you address @kbhawkey's review comments and get this one ready to be merged? Thanks

Fixed, sorry.

Thank you, @smarterclayton and @savitharaghunathan!
/lgtm


Jcaccamo/surface placement rel to scene

@ldanzinger Ready for review. Thanks!
Updated surface placement sample with new surface placement mode, relative to scene.

@jared-esri since this is being reworked, should we close out the PR for now?

closing PR as the sample is being reworked.


Nginx?

What would be nginx equivalent of .htaccess rewrite rules for the wiki to run properly?

@Furgas : https://github.com/victorstanciu/Wikitten/wiki/nginx-configuration
Here there are a configuration.
But it's only if you're in the root. it's not my case, and i don't found the good conf when i'm into the /wiki folder

Thanks. It works.


[REQUEST] Plugin with DL model

https://github.com/hongzimao/decima-sim
would you like to instruct me how to integrate this above deep learning model into your pipeline. Moreover, how can I integrate or plug it with your frameworks?

Closing as this looks to be a stale issue.  The above doesn't appear to be a deep leaning model, but we recommend taking a look at the Getting started section or tutorials in our repo if you are curious to run some models with DeepSpeed.


Remove the Reverse Proxy and host old content

This relates to https://github.com/pythonsd/pythonsd.org/issues/111 and I'm not 100% sold on this but I wanted to discuss it.
We could simply import the content from the old site into a Django app and serve it rather than proxying to GitHub pages. The amount of content on the old site isn't overwhelming. There are:

~70 total HTML pages on the old site

~40 blog posts on the old site
~5 are static content pages like the code of conduct, the home page, and job posting guidelines
The rest are for navigating blog posts by tag, author, category, etc.

~20 images on the old site most of which are sponsor logos

If we don't think we will be producing blogs anymore (no one has blogged for PythonSD in over 3 years) then I think this is fine. Another idea is to have a Django-based blogging engine as part of this repo and move the old posts into it.

I looked at this and it's possible we don't need everything. A few of the static pages are very outdated like the getting started page and the workshops page.
The blog posts are also probably of limited value. Perhaps we just take what's still useful (CoC, sponsors, etc.) and get rid of the rest. We'll try to have meaningful redirects where possible.


The North and South db performance is low by single thread

I  have a problem when I use 3 nodes north-db and 3 nodes southdb and I have 2000 nodes in my clusster, I made 2000 chassis and 100 subnet in my cluster and at least 200,000 lsp. When I update some config or list some config, It has a large delay.
I try to understand the source code, I think ovsdb uses an event-driven approach to single-threading events, in this ways, how can I enhance performance.

Is there any optimize parameters for nb and sb or some optimize in the future
Thanks, alin

Where is the Bottleneck? is it on OVN south bound or on OVN north bound? Did you tried to use ovsdb-relay for large scale?


JWT openid-config v2.0 URL?

On the page...
This openid-config URL corresponds to the v1 endpoint. For the v2 openid-configendpoint, use https://login.microsoftonline.com/common/v2.0/.well-known/openid-configuration.
Should this read as...
This openid-config URL corresponds to the v1 endpoint. For the v2 openid-configendpoint, use https://login.microsoftonline.com/{aad-tenant}/v2.0/.well-known/openid-configuration.
?

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

ID: 823eb96f-e5f1-0832-b175-0b31009be92f
Version Independent ID: 330085ae-dcf6-e930-86a8-0e4a60ab1314
Content: Protect an API by using OAuth 2.0 with AAD and API Management - Azure API Management
Content Source: articles/api-management/api-management-howto-protect-backend-with-aad.md
Service: api-management
GitHub Login: @miaojiang
Microsoft Alias: apimpm

Hi @mcalnd70 Thanks for your feedback. We will review and update as appropriate.

@mcalnd70 It is already mentioned in the doc under note section :
https://docs.microsoft.com/en-us/azure/api-management/api-management-howto-protect-backend-with-aad#configure-a-jwt-validation-policy-to-pre-authorize-requests

At this moment, I don't think document change is required. For now I will proceed with closure of this and If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

@DixitArora I was referring to the Note section. The v2 endpoint /common/ in it, should it not be {aad-tenant} ?

@DixitArora-MSFT I was referring to the Note section. The v2 endpoint /common/ in it, should it not be {aad-tenant} ?

@mcalnd70 I inserted the tenant into my openid-config (endpoint) and it works for me.
url="https://{tenant_name}.b2clogin.com/{tenant_name}.onmicrosoft.com/v2.0/.well-known/openid-configuration?p=B2C_1_{user-flow}"
`

@fabienso sorry I have no idea what you mean


AsyncEventPublisherInterface doesn't have $storeId property

I was confused as to how the store filtering worked since I could not seem to find a property for setting it, but it seems that the AsyncEventPublisherInterface (which I assume is the correct class for DI) doesn't have the storeId property while AsyncEventPublisher does.

Also it seems like in https://github.com/mage-os/mageos-async-events/pull/26/files the type of the property was changed to string, which seems inconsistent / wrong. I'm not sure why it was necessary but if typing is applied it should be int, shouldn't it?

Hi @Swahjak, it looks like it was missed on the interface and should be present there. thanks for picking this up :slightly_smiling_face:
regarding the string instead of int, this will have to be a string simply because we define everything we publish to a topic is a string. please see https://github.com/mage-os/mageos-async-events/blob/4.x/etc/communication.xml#L4
as to why it's changed recently to a string, please see https://github.com/mage-os/mageos-async-events/pull/26#issuecomment-2194237079

I see your argument, but isn't that exposing / enforcing logic required by the publisher to the client? Wouldn't it make sense to i.e. make sure conversion happens in the publisher? A simple (string) cast on https://github.com/mage-os/mageos-async-events/blob/4.x/Model/AsyncEventPublisher.php#L37 would do?
Awaiting you comment before updating the pull (including the feat/ branch name)

true, the current approach is a bit of a poor design choice. my main concern was introducing a breaking change with this. however, if you can make that change and update the PR and all tests pass, we can package it up in a beta release and see how it goes.

I've made the adjustment and opened a new merge request with the feature/ branch name.


Invoices subclient and methods

Dependencies

#2
#5

Example
import billit

client = billit.Client(base_url=billit.SANDBOX_BASE_URL, api_key="sandbox_api_key")

taxes = [billit.Tax(tax_id=514), billit.Tax(tax_id=524, tax_vat_show=1)]
invoice = client.invoices.create(customer_id="abc", send_mail=False, taxes=taxes)

Closed in #7


use warning instead of deprecated warn

logging.warn has been deprecated since Python 3.3 and one should use logging.warning
See also https://github.com/bitcoin-core/HWI/pull/696

Great point.  Many thanks for the PR!

ack

Rebased, slight change to line length/formatting in a couple of places to pass our clang-format check.
Slight tweaks to commit message.
Merged to master (as part of #78).
Many thanks.


Migrate away from react-test-renderer

The following files currently still use react-test-renderer:
HexGrid.test.tsx
Layout.test.tsx
Path.test.tsx
Pattern.test.tsx
All of these need to be migrated to @testing-library/react

#17


[WIP] Docker backend service

This PR adds two new Docker images, the intention is to create a backend_bin service container, an older version of this backend container is being used by a new experimental js client for web apps, running the backend next to sitl and a proxy translating http1 to grpc as shown here docker-compose.yml.
New images

Image
Description
Size

dronecode-sdk-base
based on alpine, minimal deps for SDK build, small image size
287MB

dronecode-sdk-backend
based on sdk-base builds and launches the backend
???

TODO

[ ] Build grpc on alpine
[ ] Build DronecodeSDK on alpine
[ ] Publish on docker hub

Note: building the dronecode-sdk-backend image needs to happen on the root of the project
docker build -f docker/Dockerfile-backend -t mrpollo/dronecode-sdk-backend:latest .

There is ENV BUILD_BACKEND 1 above, but I would also add it to this command instead of globally.

Ah, I missed that.

Closed in favor of #673


Update std::math::u64 according to the new u32 refactoring.

std::math::u64 module of the stdlib should be updated to be consistent with new u32 refactoring.

Closed by #1142.


Polygons consisting of several long lines are not always filled properly

See an example screenshot attached below.
It is taken from this place
http://www.openstreetmap.org/edit#map=16/53.1133/30.1565
and is about this polygon
http://www.openstreetmap.org/relation/7745393
The issue is not constanct. It happens only sometimes with one and the same object.
I have not managed to figure out the exact steps yet but decided to submit it case this can be understood from the code.
The only thing which I noticed it happens quite often after submitting edits.

Yep, this is #1201.  iD can't render multipolygons correctly if they are only partially downloaded.  Fully downloading them might take a while, but might be something reasonable to do in a background task.
Most multipolygons are small, but it's the outliers with thousands of member ways that make this issue scary.


Component Not found

Hi, I am using daisyui for my laravel app and it works fine or that's what I thought. Whenever I use something with components as below chrome console throws an error. I tried to use a Carousal for my app.
1
Saying 404 it's not found. So I believe these components should be in my components folder. Right?
I followed the documentation clearly. Is there anyway to publish these components to my reources\Views folder

@SupunSam that's just demo URL. If you're using that carousel example, you just need the anchor link like this:
1

Let me know if you have any questions

Oh, how stupid of me. Yes of course. Thanks. Got it.


Only 50 Addresses Shown

Firstly, let me thank you for a very useful tool and your continued support.
The issue I am having curently is that the local network I scan has 100+ devices online at any one time, but the WYL home page only ever show a maximum of 50 IP's, in my case <IP_ADDRESS> - <IP_ADDRESS>.  Anyting higher than 50 is now ever shown as online.
Hope there is a fix for this.

Hi! Thanks for the feedback!
I'm afraid that's the problem of arp-scan tool used by WYL. I'll look into it in the next release.

I'm not sure this will solve the problem, but there is now an option to tweak arp-scan timeout. May be setting longer timeout will help (like 2000 ms)

I'm not sure this will solve the problem, but there is now an option to tweak arp-scan timeout. May be setting longer timeout will help (like 2000 ms)

Are you able to point me in the right direction as to where I would tweak the arp-scan setting?

arp-scan timeout

Found it in the Config page, trying it out now.

Feedback...
I have set the Arp Timeout value to 2000, but this seems to have made no difference, I can still only see 50 [up] addresses on my IP range.

Feedback...

Ok. Can you please execute arp-scan -glNx -I YOUR_IFACE on the same host as WYL?
It will help determine whether the problem is within WYL or arp-scan.
I don't need full scan results, just the fact that bug keeps showing in arp-scan or not.

So I have run the following:
sudo docker exec WatchYourLAN arp-scan -glNx -I ovs_eth2
Scan results show nothing higher than xxx.xxx.xxx.50

Hi aceberg,
Have you any idea when this issue is likely to be sorted.  If I list my offline devices, I see devices from 192.168.xxx.02 through to 254, but if I look at the online devices, the list stops at device 192.168.xxx.50 an never displays any devices higher than the 1st 50.  Is there a limit set on the amount of only devices that can be displayed, have a missed something?


Is it compatible with ObjectiveDDP

I'm using objectiveDDP in my iOS app, is this package compatible with any ddp connection or it requieres client side Javascript to work?

Hi @bitomule, probably it's compatible, but you need to implement calls to set user as away and online because I do this in client side code of package.
You can read the code here https://github.com/Konecty/meteor-user-presence/blob/master/client/client.js
Basically you need to call methods UserPresence:away and UserPresence:online.

It works. Right now I'm only using online, not away. Just called the method on login in iOS app. I'll even add away when app goes to background. Great work!

Nice, will be great if you can create a PR adding how to do this in the README of this project.


Initial magnetometer alignment is never performed by GPS CoG

Issue was first raised here but I thought it would be best to create a separate issue not to lose track of it.
The code that enables planes to recover from bad mag data at start of flight that was added here is never triggered anymore since the yaw reset by magnetometer that was added here always runs before.
https://github.com/PX4/ecl/blob/ad9414e4dadc6a7a25d33b7edc219775d8a88086/EKF/control.cpp#L1455-L1470

Fixed on master.


[DOC] Fixed conf,py

Description
Changed registered signed in the title to (R) -- output is the same.

/intelci: run


GitHub Actions - added PHP 8.0 and 8.1

Added testing in GitHub Actions agains latest stable PHP versions

Added testing in GitHub Actions agains latest stable PHP versions

$httpClient = new \Http\Adapter\Guzzle6\Client(
new \GuzzleHttp\Client()
);
$httpStack = new \SocialConnect\Provider\HttpStack(
$httpClient,
// @todo Where is factories?
new \SocialConnect\HttpClient\RequestFactory(),
new \SocialConnect\HttpClient\StreamFactory()
);


Extra parameters on click of bar

Is it possible to be able to click on a part of a stacked bar chart and then be create a table/open a new page where you can see all the  For example:
This is a chart with the conditions (very good,good,...) of various buildings. I would like for the user to click on that part of the chart and then see a table with all the objects. For this I think we would need custom parameters we can give with JSON.
This table is custom generated with options that the user can choose using JSON. For example, they want to see all the conditions of the churches in Sittart and Geleen and compare them to each other. After the graph is generated, the user wants to see which churches in Sittart have a bad state. So he would go to the 'goed' (which means good in English) bar of Sittart , click (or double click) on it and get a table with all the bad churches.

I hope I have explained my request good enough. I've read the documentation already and couldn't find anything.

Try to use that https://apexcharts.com/docs/options/chart/events/

Try to use that https://apexcharts.com/docs/options/chart/events/

But how do I add hidden parameters to a bar? So I can call those parameters when a user clicks on a bar.

Try to use that https://apexcharts.com/docs/options/chart/events/

But how do I add hidden parameters to a bar? So I can call those parameters when a user clicks on a bar.

What is hidden parameters you want add?

Try to use that https://apexcharts.com/docs/options/chart/events/

But how do I add hidden parameters to a bar? So I can call those parameters when a user clicks on a bar.

What is hidden parameters you want add?

An sql so I can get all the items of that bar in a table when a user clicks on the bar. So he knows what specific items that bar contains you know?

Try to use that https://apexcharts.com/docs/options/chart/events/

But how do I add hidden parameters to a bar? So I can call those parameters when a user clicks on a bar.

What is hidden parameters you want add?

An sql so I can get all the items of that bar in a table when a user clicks on the bar. So he knows what specific items that bar contains you know?

There are 491 good objects in the city 'Geleen' as you can see on the screenshot. But my user wants to know which objects those are. He would simply click on the bar then a table would popup with all the good objects in 'Geleen' .

Well, sorry for digging this up from it's grave, but I had the exact same problem and couldn't find anything anywhere to help. In my case I had a donut chart of categories, and when I clicked in a section, I wanted to be redirected to a list filtered by that same category.
Here is what I did, if anyone else happens to stumble upon this issue.
Here is my options object:
{
      series: [200, 99, 30],
      labels: ['Label 1', 'Label 2', 'Label 3'],
      xaxis: {
        categories: [55, 77, 99]
      },
      chart: {
        type: 'donut',
        events: {
          dataPointSelection: (event: any, chartContext: any, config: any) => {
            console.log(config);
            console.log(config.w.config.series[config.dataPointIndex])
            console.log(config.w.config.labels[config.dataPointIndex])
            console.log(config.w.config.xaxis.categories[config.dataPointIndex])
          }
        }
      }
 }

I ended up using the xaxis.categories property, that's not used on a donut chart, to send data through the dataPointSelection event, and then use it however I want inside my funcion.
If the xaxis.categories property it's actually used in the chart, then I would look for another property that's not being used.
If there's no intended way of doing this, creating a custom property would be a more decent approach, but I don't know how to do it.


Updated Dojah.js Added config?.pages?.some(

An issue was raise on React Native Library repository by Payshiga Developer
https://github.com/cjayprime/react-native-dojah/issues/7
The fix has been made in Dojah.js file by changing config.pages.some to config?.pages?.some(

An issue was raise on React Native Library repository by Payshiga Developer
https://github.com/cjayprime/react-native-dojah/issues/7
The fix has been made in Dojah.js file by changing config.pages.some to config?.pages?.some(


knn neighbors name list?

In face_recognition_knn.py
I want name list of 5 neighbors. So I change n_neighbors=5.
closest_distances = knn_clf.kneighbors(faces_encodings, n_neighbors=5)
And it returned just 5 values of distance_threshold from trained .clf file
I found that knn_clf.predict(faces_encodings) return only 1 best match name.
How can I get the name list of all that 5 people?

I have found my own way. Look stupid but it's working lol
`   probe = knn_clf.predict_proba(faces_encodings) # find 5 most similar faces
name_list = knn_clf.classes_ # list all name that have trained
ordered_index = probe[0].argsort()[-5:][::-1] # reorder 'probe' array from most to less similar

name_order = []
global perc

for l in ordered_index:
    name_order.append(name_list[l]) # order name list most to less similar

for x in range(0,len(name_order)):
    print (name_order[x],closest_distances[0][i][x]) # got name with distance_threshold `


xmr: move functions from ge25519.c

This PR moves

bignum25519 related functions (curve25519_*) from ge25519.c to curve25519-donna-32bit.c and
ge25519 related functions (ge25519_*) from ge25519.c to ed25519-donn-impl-base.c.

It follows this commit.

If merged, trezor-core may need some changes, @ph4r05.


remove a book from the start screen

i made the mistake of trying to find a book and open it's .json file
now i have two books listed on the start screen:

one of which looks like this:

sorry for the noise._.

% cd ~/.Raven/
% rm -rf -- stuff-i-want-gone

sorry for the noise._.

No worries.
There isn't a way of removing things from the interface yet, it should be added eventually.


Arquillian persistance

Uses weld arquillian container and dbunit so we can use injection / reduce setup code

fixes issue AGPUSH-1243

rebased

:+1: Works here, the build is just a bit longer, but just a few seconds. Can we merge this ?

I think this needs a rebase

rebased @matzew

@edewit I wonder why Travis is not happy
I don't see Derby as an explicit dependency here. Perhaps that's related ?

@kpiwko can you/your team check this too ? (e.g. on RHEL)

FYI - I just tested in a VM (Fedora 21), and the build worked fine there

@matzew I think @kpiwko is on PTO, maybe @TadeasKriz could give it a try?

@abstractj I'll give it a try.

I've run the command mvn clean test and mvn clean install. Both succeeded with no failing tests. I also asked @smiklosovic to test in on his setup as well (I believe he has Fedora 20 or something).

@TadeasKriz @edewit @matzew @abstractj
I am not able to test it, it gives me this error:

I wonder why Travis is not happy. Worked for me on Fedora21 as well :-/
On Friday, February 13, 2015, Tadeas Kriz<EMAIL_ADDRESS>wrote:

I've run the command mvn clean test and mvn clean install. Both succeeded
with no failing tests. I also asked @smiklosovic
https://github.com/smiklosovic to test in on his setup as well (I
believe he has Fedora 20 or something).
—
Reply to this email directly or view it on GitHub
https://github.com/aerogear/aerogear-unifiedpush-server/pull/473#issuecomment-74246416
.

--
Sent from Gmail Mobile

@matzew seems that it fails on cleaning the db, but 3 days ago

@matzew Travis is not happy most likely due to 10k lines log limit.
Tests are failing on RHEL7 - my machine - but because of some npm related issue, I need to dig deeper into what's going on.

Ok, passed on RHEL7. My issues was fixed by this PR: https://github.com/aerogear/aerogear-unifiedpush-server/pull/492

@smiklosovic could you try on your machine with @kpiwko's fix?

@TadeasKriz @smiklosovic right, that's the same issue I was having :-)

@kpiwko @TadeasKriz yes with Karel's PR it works

@matzew @edewit this should make Travis happy, based on this PR  - https://github.com/aerogear/aerogear-unifiedpush-server/pull/495 - however, now it fails with different error.

@kpiwko did you send a PR against @edewit's branch ?
@edewit or you do a cherry-pick and rebase this branch ?

@kpiwko did you send a PR against @edewit's branch ?

@edewit or you do a cherry-pick and rebase this branch ?

@kpiwko @edewit any comment?

I also have a model/jpa/derby.log file now in my repo, looks like it comes from arquillian, we could probably add this to the gitignore

+1
but Travis needs to be happy before we can actually merge this
On Fri, Feb 20, 2015 at 9:38 AM, Sebastien Blanc<EMAIL_ADDRESS>wrote:

I also have a model/jpa/derby.log file now in my repo, looks like it
comes from arquillian, we could probably add this to the gitignore
—
Reply to this email directly or view it on GitHub
https://github.com/aerogear/aerogear-unifiedpush-server/pull/473#issuecomment-75205527
.

--
Matthias Wessendorf
blog: http://matthiaswessendorf.wordpress.com/
sessions: http://www.slideshare.net/mwessendorf
twitter: http://twitter.com/mwessendorf

@kpiwko @edewit any updates?

@matzew don't know why only Travis is not happy there is nothing in the logs, how can we find out?

@edewit Cool ! I will retry @TadeasKriz test case, if it's okay, we can merge, there will be no conflicts

@edewit I can see some failures in the log around deleting installations (oh the irony :) ) orare that test cases that are supposed to fail ?

I reopened https://issues.jboss.org/browse/AGPUSH-1295 because the issue is still there even with the update to orm.xml , but this is unrelated to this PR

So despite the travis failure , this can be merged for me.

@sebastienblanc on https://issues.jboss.org/browse/AGPUSH-1295 you say the failure is present on master - and the travis build is having test failures, on the devices... I wonder what's going on there.
I am -1 on merging this if Travis keeps having test failures

Closing it until we figure out what's happening with Travis. Please,  feel free to reopen when fixed.
Thank you

Sorry for maybe my stupid question but can we trigger manually a new build on Travis ? Since we can not reproduce that locally, curious what happens in Travis if it builds again

@sebastienblanc http://stackoverflow.com/questions/17606874/trigger-a-travis-ci-rebuild-without-pushing-a-commit

seems we need this, for JMS let's see if we can find out why travis is failing

@edewit please rebase

see also https://issues.jboss.org/browse/AGPUSH-1356

could be a problem with xml parsers, as the test data is specified in xml 'dbunit' seems that on oracle-jdk it fails but succeeds on openjdk

@sebastienblanc @kpiwko can you review ?

hasn't this been reviewed enough? Last thing that changed make it work on
travis and squashed commits
On Mon, Apr 13, 2015 at 10:37 AM Matthias Wessendorf <
<EMAIL_ADDRESS>wrote:

@sebastienblanc https://github.com/sebastienblanc @kpiwko
https://github.com/kpiwko can you review ?
—
Reply to this email directly or view it on GitHub
https://github.com/aerogear/aerogear-unifiedpush-server/pull/473#issuecomment-92271640
.

merging now, was reviewed as long time ago, did just a rebuild Friday to check that things worked after the rebase and :+1:

alright
On Mon, Apr 13, 2015 at 1:34 PM, Sebastien Blanc<EMAIL_ADDRESS>wrote:

merging now, was reviewed as long time ago, did just a rebuild Friday to
check that things worked after the rebase and [image: :+1:]
—
Reply to this email directly or view it on GitHub
https://github.com/aerogear/aerogear-unifiedpush-server/pull/473#issuecomment-92321670
.

--
Matthias Wessendorf
blog: http://matthiaswessendorf.wordpress.com/
sessions: http://www.slideshare.net/mwessendorf
twitter: http://twitter.com/mwessendorf

:tada: yay

:-)


helm search repo naming lowercase

Output of helm version:
3.7.2
Output of kubectl version:
v1.18.3
Cloud Provider/Platform (AKS, GKE, Minikube etc.):
I upload a chart (HelmTest-0.1.0.tgz) to the repo, then use command "helm search repo " to show charts, but it convert to lowercase.

helm install cc myrepo/helmtest
Error: INSTALLATION FAILED: failed to download "myrepo/helmtest"
helm install ccc myrepo/HelmTest
NAME: ccc
LAST DEPLOYED: Thu Jan  6 17:49:39 2022
NAMESPACE: default
STATUS: deployed
REVISION: 1
TEST SUITE: None

I was able to duplicate this behavior using chartmuseum and the servecm and cm-push plugins:
helm servecm --port=8879 --storage local --storage-local-rootdir ./local --context-path=/charts

$ helm create MyChart
Creating MyChart
$ helm cm-push MyChart local
Pushing MyChart-0.1.0.tgz to local...
Done.
$ helm repo add local http://<IP_ADDRESS>:8879/charts
"local" has been added to your repositories
$ helm search repo MyChart
NAME         	CHART VERSION	APP VERSION	DESCRIPTION
local/mychart	0.1.0        	1.16.0     	A Helm chart for Kubernetes
$  helm fetch local/mychart
Error: chart "mychart" matching  not found in local index. (try 'helm repo update'): no chart name found
$ helm fetch local/MyChart
$

@joejulian verified this behavior and changing label to bug.

Hi @joejulian I'd like to take this issue.


Recommend source-build compatible dependabot pattern

Some .NET repos (e.g. templating and msbuild) have taken a dependency on utilizing dependabot to update dependencies.  The standard pattern is not compatible w/source-build because the dependent package versions gets hard-coded and not overridable by source-build.
Investigation should be done to see if version properties can be utilized that source-build can override.  Regardless on the finding repo follow-up needs to be done to update/recommend against the existing patterns being used.  Source-build documentation should also be created that captures the recommendation.

@jeffkl you worked with dependabot folks to get the central package version pattern supported, right? Do you have contacts that could help with this?

Unfortunately no, they never contacted me and implemented it entirely on their own.
https://github.com/dependabot/dependabot-core/pull/1311
It would be cool to extend dependabot to know about more files with the configuration, I don't see an existing issue for that.
Existing list of files it searches:
https://github.com/dependabot/dependabot-core/blob/8315e7b213870959158c1c2bcfb482728a7e801a/nuget/lib/dependabot/nuget/file_fetcher.rb#L117
And items it looks for:
https://github.com/dependabot/dependabot-core/blob/8315e7b213870959158c1c2bcfb482728a7e801a/nuget/lib/dependabot/nuget/file_parser/project_file_parser.rb#L17-L21


Bugfix/yousong owrt

这个 PR 实现什么功能/修复什么问题:
hostman: openwrt: deploy public key for root
hostman: split DeployAuthorizedKeys()
hostman: openwrt: support deploy networking scripts
hostman: openwrt: deploy hostname with board config
hostman: distro name OpenWrt instead of OpenWRT

是否需要 backport 到之前的 release 分支:

release/2.10.0

/area host
/cc @wanyaoqi @swordqiu

Codecov Report

Merging #4683 into master will decrease coverage by 0.01%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master   #4683      +/-   ##
=========================================
- Coverage    6.79%   6.78%   -0.02%
=========================================
  Files         524     524
  Lines       89593   89660      +67
=========================================
- Hits         6088    6079       -9
- Misses      82998   83076      +78
+ Partials      507     505       -2

Flag
Coverage Δ

#aFlag
6.78% <0%> (-0.02%)
:arrow_down:

Impacted Files
Coverage Δ

pkg/hostman/guestfs/fsdriver/linux.go
0% <0%> (ø)
:arrow_up:

pkg/hostman/guestfs/fsdriver/base.go
13.08% <0%> (-0.38%)
:arrow_down:

pkg/cloudcommon/cronman/cronman.go
49.51% <0%> (-5.34%)
:arrow_down:

pkg/appsrv/response.go
69.23% <0%> (+3.07%)
: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 96c48ca...b90a1e8. Read the comment docs.

/lgtm

/lgtm

/approve


FIX: skip t1w check if anat-derivatives provided.

Changes proposed in this pull request
Fixes #2197 .
In the case where a precomputed anatomical derivatives are provided with --anat-derivatives, skip the check for an existing T1w.
For now the side-tracking of smriprep will produce a warning when the anat-derivatives does not match the necessary input.  I think that this option will be mainly used by skilled users, and they would rather know if fast-track didn't work.

Documentation that should be reviewed

There is also a check for an existing T1w in niworkflows.interfaces.bids.BIDSDataGrabber.
Before I move forward with a PR to niworkflows, do you agree with the changes?

With the current changes, I still have two problems with the following nodes that requires path to raw T1w file:
fmriprep_wf.single_subject_01_wf.ds_report_about
fmriprep_wf.single_subject_01_wf.bids_info
What would be the best approach with these nodes when using anat fast-track?
Skip them? (might not be possible for BidsInfo which apparently convert the grabbed T1w file to BIDS entities).
Replace the raw filename with the preprocessed filenames for T1w,T2w?

Sorry, left this behind for a while.
Tests are failing because it requires the changes in niworkflows#545.

@bpinsard Can you install your niworkflows branch by updating this line?
https://github.com/poldracklab/fmriprep/blob/658c0ad603aafaa247b75819deffdec35c7a2ca3/setup.cfg#L30

Cool. Tests passing, so merged in niworkflows. Reverting to the dependency on master, and I think this can go in.

Cool, thanks @bpinsard!

Thanks for your help and feedback!


Apply new licence to all active GitHub repos in the Open Targets organisation

Once the new licence text is ready - see #1583 - can we please update the LICENCE files of all of our active repositories?
@cmalangone has recommended that we update the licence text and then open a pull request so that the technical team can merge the changes into the most appropriate branch. She also mentioned that GWAS Catalog has developed a process for updating their codebase licences yearly. Please speak with her before beginning this project and consider meeting with GWAS Catalog to learn more about their approach.

All Open Targets GitHub repositories can be updated with the Apache 2.0 licence provided by GitHub.
https://docs.github.com/en/communities/setting-up-your-project-for-healthy-contributions/adding-a-license-to-a-repository

all repos have the Apache 2.0 license now.


Slice bounds out of range

Hi guys,
i'm trying to run mezzanine inside a docker container and I get always that error. Any idea about what is going wrong?
Thanks.
goroutine 12 [running]:
bufio.(*Reader).fill(0xc082142360)
        /usr/local/go/src/bufio/bufio.go:86 +0x2e2
bufio.(*Reader).WriteTo(0xc082142360, 0x2624c8, 0xc08200a4b0, 0x18c1, 0x0, 0x0)
        /usr/local/go/src/bufio/bufio.go:449 +0x285
io.Copy(0x2624c8, 0xc08200a4b0, 0x264920, 0xc082142360, 0x0, 0x0, 0x0)
        /usr/local/go/src/io/io.go:354 +0xb9
github.com/docker/docker/api/client.func┬À022(0x0, 0x0)
        /go/src/github.com/docker/docker/api/client/hijack.go:210 +0x1e0
github.com/docker/docker/pkg/promise.func┬À001()
        /go/src/github.com/docker/docker/pkg/promise/promise.go:8 +0x36
created by github.com/docker/docker/pkg/promise.Go
        /go/src/github.com/docker/docker/pkg/promise/promise.go:9 +0x102

goroutine 1 [chan receive]:
github.com/docker/docker/api/client.(*DockerCli).CmdRun(0xc08205f220, 0xc082012090, 0x5, 0x5, 0x0, 0x0)
        /go/src/github.com/docker/docker/api/client/commands.go:2469 +0xfcd
reflect.callMethod(0xc0820b51d0, 0xc082045be8)
        /usr/local/go/src/reflect/value.go:605 +0x180
reflect.methodValueCall(0xc082012090, 0x5, 0x5, 0x1, 0xc0820b51d0, 0xc08205f201, 0xc082072240, 0x5, 0xc08200a401, 0x255790, ...)
        /usr/local/go/src/reflect/asm_amd64.s:29 +0x3d
github.com/docker/docker/api/client.(*DockerCli).Cmd(0xc08205f220, 0xc082012080, 0x6, 0x6, 0x0, 0x0)
        /go/src/github.com/docker/docker/api/client/cli.go:81 +0x385
main.main()
        /go/src/github.com/docker/docker/docker/docker.go:138 +0xb90

goroutine 5 [syscall]:
os/signal.loop()
        /usr/local/go/src/os/signal/signal_unix.go:21 +0x26
created by os/signal.init┬À1
        /usr/local/go/src/os/signal/signal_unix.go:27 +0x3c

goroutine 11 [chan receive]:
github.com/docker/docker/api/client.(*DockerCli).hijack(0xc08205f220, 0x898b90, 0x4, 0xc082140500, 0x76, 0x1, 0x262498, 0xc08200aaf0, 0x2624c8, 0xc08200a4b0, ...)
        /go/src/github.com/docker/docker/api/client/hijack.go:237 +0xd38
github.com/docker/docker/api/client.func┬À013(0x0, 0x0)
        /go/src/github.com/docker/docker/api/client/commands.go:2437 +0x1c9
github.com/docker/docker/pkg/promise.func┬À001()
        /go/src/github.com/docker/docker/pkg/promise/promise.go:8 +0x36
created by github.com/docker/docker/pkg/promise.Go
        /go/src/github.com/docker/docker/pkg/promise/promise.go:9 +0x102

goroutine 9 [IO wait]:
net.(*pollDesc).Wait(0xc0820b6170, 0x72, 0x0, 0x0)
        /usr/local/go/src/net/fd_poll_runtime.go:84 +0x4e
net.(*ioSrv).ExecIO(0xc082042380, 0xc0820b6060, 0x89b3f0, 0x7, 0x97d8c0, 0x91, 0x0, 0x0)
        /usr/local/go/src/net/fd_windows.go:188 +0x305
net.(*netFD).Read(0xc0820b6000, 0xc0820ba000, 0x400, 0x400, 0x0, 0x0, 0x0)
        /usr/local/go/src/net/fd_windows.go:470 +0x180
net.(*conn).Read(0xc082042398, 0xc0820ba000, 0x400, 0x400, 0x0, 0x0, 0x0)
        /usr/local/go/src/net/net.go:121 +0xe3
crypto/tls.(*block).readFromUntil(0xc0820ca960, 0x263e68, 0xc082042398, 0x5, 0x0, 0x0)
        /usr/local/go/src/crypto/tls/conn.go:454 +0xed
crypto/tls.(*Conn).readRecord(0xc08207c580, 0x17, 0x0, 0x0)
        /usr/local/go/src/crypto/tls/conn.go:539 +0x2e1
crypto/tls.(*Conn).Read(0xc08207c580, 0xc082011000, 0x1000, 0x1000, 0x0, 0x0, 0x0)
        /usr/local/go/src/crypto/tls/conn.go:904 +0x16d
net/http.noteEOFReader.Read(0x2646e0, 0xc08207c580, 0xc0820fc3c8, 0xc082011000, 0x1000, 0x1000, 0xc08206ea80, 0x0, 0x0)
        /usr/local/go/src/net/http/transport.go:1270 +0x75
net/http.(*noteEOFReader).Read(0xc08202e400, 0xc082011000, 0x1000, 0x1000, 0xc082014000, 0x0, 0x0)
        <autogenerated>:125 +0xdb
bufio.(*Reader).fill(0xc082142300)
        /usr/local/go/src/bufio/bufio.go:97 +0x1d5
bufio.(*Reader).Peek(0xc082142300, 0x1, 0x0, 0x0, 0x0, 0x0, 0x0)
        /usr/local/go/src/bufio/bufio.go:132 +0xf7
net/http.(*persistConn).readLoop(0xc0820fc370)
        /usr/local/go/src/net/http/transport.go:842 +0xab
created by net/http.(*Transport).dialConn
        /usr/local/go/src/net/http/transport.go:660 +0xca6

goroutine 10 [select]:
net/http.(*persistConn).writeLoop(0xc0820fc370)
        /usr/local/go/src/net/http/transport.go:945 +0x424
created by net/http.(*Transport).dialConn
        /usr/local/go/src/net/http/transport.go:661 +0xcc3

goroutine 14 [sleep]:
github.com/docker/docker/api/client.func┬À025()
        /go/src/github.com/docker/docker/api/client/utils.go:286 +0x69
created by github.com/docker/docker/api/client.(*DockerCli).monitorTtySize
        /go/src/github.com/docker/docker/api/client/utils.go:295 +0x163

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

Docker version:
Client version: 1.6.2
Client API version: 1.18
Go version (client): go1.4.2
Git commit (client): 7c8fca2
OS/Arch (client): windows/amd64
Server version: 1.6.2
Server API version: 1.18
Go version (server): go1.4.2
Git commit (server): 7c8fca2
OS/Arch (server): linux/amd64
Docker info:
Containers: 144
Images: 123
Storage Driver: aufs
Root Dir: /mnt/sda1/var/lib/docker/aufs
Backing Filesystem: extfs
Dirs: 411
Dirperm1 Supported: true
Execution Driver: native-0.2
Kernel Version: 4.0.3-boot2docker
Operating System: Boot2Docker 1.6.2 (TCL 5.4); master : 4534e65 - Wed May 13 21:24:28 UTC 2015
CPUs: 2
Total Memory: 1.957 GiB
Name: boot2docker
ID: OPAM:ZVLV:SFU3:7Q26:ULV5:THYB:XLSY:7FRS:HYIK:LODB:QKA5:KVPI
Debug mode (server): true
Debug mode (client): false
Fds: 19
Goroutines: 32
System Time: Thu Jun 18 11:20:19 UTC 2015
EventsListeners: 0
Init SHA1: 7f9c6798b022e64f04d2aff8c75cbf38a2779493
Init Path: /usr/local/bin/docker
Docker Root Dir: /mnt/sda1/var/lib/docker
Uname -a :
$ uname -a
MINGW32_NT-6.1 PC-084707 1.0.12(0.46/3/2) 2012-07-05 14:56 i686 unknown
Steps to reproduce:
###############################################################
Dockerfile to build mezzanine environment and run python app.
Based on python2.7
################################################################
Set the base image to python2.7
FROM python:2.7
Update the repository sources list
RUN apt-get update
#instalar dependecias mezzanine
#RUN apt-get install -y libjpeg8 libjpeg8-dev
#RUN apt-get build-dep -y python-imaging
#instalar mezzanine
RUN pip install mezzanine
EXPOSE 8000
####fin Dockerfile ########

After, that we build the image and launch it:
docker run -i -t python/mezz bash

mezzanine-project proy

python manage.db createdb

python manage.db runserver

Expected result: a python app running on localhost:8000
Received result: the error posted before.
Adittional info:
First time I run the server it works, but if I close it and stop the cointainer, and run it againt it does'nt

Plase close, same error as #13299


Added alt to footer social icons #26

Hi @nexus0212 ,
Great work with this issue, the code looks great. There is one problem though, can you please have it merge into staging rather than straight into master?
For future contributions, please branch from staging and have your PRs merge into staging. Thanks for taking the time to contribute!
If you run into any issues or have any questions, please let me know.


chore: use node v18.14.2 in .nvmrc

Ensures the same Node.js version is used locally as on CI.

Released: v0.25.9 🎉
This has been released in v0.25.9!

📄 Release notes
📦 npm package

Make sure to always update to the latest version (npm i @mswjs/interceptors@latest) to get the newest features and bug fixes.

Predictable release automation by @ossjs/release.


Update documentation to reflect new branching strategy

With the release of 4.0.0, the default branch has been renamed to 4.1 and will be renamed on every release to ensure pull requests are changed to target the new default branch. This PR updates the documentation to reflect this and removes unnecessary steps from the release instructions.

Codecov Report

Merging #2632 (fe90324) into 4.1 (7880990) will not change coverage.
Report is 1 commits behind head on 4.1.
The diff coverage is n/a.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@            Coverage Diff            @@
##                4.1    #2632   +/-   ##
=========================================
  Coverage     91.04%   91.04%
  Complexity      756      756
=========================================
  Files            35       35
  Lines          1931     1931
=========================================
  Hits           1758     1758
  Misses          173      173


feat: support cross-linking synthetics & apm

fix #265

Relies on the properties being injected from the Heartbeat side to make it work seamless.

Adds support for crosslinking the Synthetics Waterfall information with the APM data to be able to understand the performance monitoring data across Elastic observability solutions. The key fields that are added to make the transition easier are

synthetics.monitor.id - monitor id of the synthetics monitor
syntheitcs.monitor.type -  type of the synthetics monitor, can be browser or http
synthetics.trace.id - checkgroup/exec id that denotes execution of a journey at any point
synthetics.monitor.location - location id where the monitor was run from
tracestate: es=s:1 - Sampling decision is propagated via tracestate header which is understood by APM agents

Adds the relevant headers only for the routes/urls that are configured via the apm.traceUrls. Sampling decision can also be propagated via apm.sampleRate. These options can be modified via CLI or Synthetics config file.

journey("test", () => {
   step("trace check", async() => {
      await page.goto("https://example.com")
   });
});

// config
{
  "apm": { "traceUrls":  ["**/*"] } // add to all requests
}

// Headers
baggage: `synthetics.trace.id=<test>;synthetics.monitor.id=<test>;`
tracestate: `es=s:1`

Env variables will be added by the Heartbeat to make this complete - https://github.com/elastic/beats/issues/36249

:broken_heart: Build Failed

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2022-09-14T06:01:55.758+0000

Duration: 5 min 2 sec

Steps errors
Expand to view the steps failures

Checks linting

Took 0 min 4 sec . View more details here
Description: npm run-script lint

:robot: GitHub comments
To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.

+1


Generate single image per pdf to avoid using to much memory

I have added first_page and last_page arguments to convert_from_bytes so the function uses less memory

Thanks - I approved and merged


Updated page listings header to new designs

Fixes #8868.
Before:

After:

did not update changelog as we already had an entry about slim header on page listings - this work is just a bit of polish on top of that.


[NewTxType] for multiple currency chains

Multiple currency chains usually pay fee and send with multiple currency

Append TxMultiCurrency & TxMultiCurrencyAnyAction type

Remove MultiCurrencyAnyAction

Remove MultiCurrencyAnyAction

Done, but later the staking operations on multi-currency chain will require new tx type to describe withdrawn rewards & fees.
If new chain is connected to cosmoshub and the fee is paid with new denom, the withdrawn rewards can be multiple currency.


Add a minimal GlobalSettings accessor

Expose a GlobalSettings struct to access document-wide settings
information, such as scale factor or axis direction.
The API as defined in this PR is EXTREMELY BAREBONE, requiring
the user to manually specify the name of the properties they want to
access.
This is in order to avoid blocking on any particular details. Favoring
further discussions about unit precision or how to expose a proper
axis direction API for later.
I'd really like to see this go through, it's a blocker for some features on https://github.com/HeavyRain266/bevy_fbx/
See also:

https://github.com/lo48576/fbxcel-dom/pull/7

Thank you!
Released as v0.0.9.

Thank you too!


$global-margin declared twice in settings.scss

$global-margin is declared twice in the settings.scss file, quick fix I am sure!
$body-antialiased: true;
$global-margin: 1rem;
$global-padding: 1rem;
$global-margin: 1rem;
$global-weight-normal: normal;

v6.0.4 fixed that.

In 6.0.4 it's not fixed !

This was previously fixed but it looks like it got added back in by accident in the 6.0.4 commit: https://github.com/zurb/foundation-sites/commit/8ff8ba68f1d1b0b4c6299e0e1e6fee2b5999fcbe#diff-35a517bca933d90868763d8450d1da1e


adjusting awsx name+title

Yah I closed it; let's talk sometime about the names of these things that appear in the registry.


Modal still appearing on desktop with showErrorMessageForUnsupportedBrowsers set to false

Hi,
On both Chrome and Safari on a Mac, when I set showErrorMessageForUnsupportedBrowsers to false, the black background which would show up behind the modal window still appears. The modal and message itself isn't there, just a blank background. Is this a known issue?

hi, let me try to repro this

hi @dankingsley , this has been fixed in https://github.com/philfung/add-to-homescreen/commit/2e7119d3a47f14468a15e95043bb37fbac2bb8f7 .  Use the latest release 1.6 .  Thanks for reporting!


Update callback plugin

Update custom_logger to generate summary file called "test_run_result.out" with the number of failures and success and then generate a detailed report for each host in separate file called "{host}_results.log"

Depends-On: #90

Depends-On: #90

The Depends-On needs to be in the PR description -- I've updated it.
I don't know whether the GHA tests will run or not, since it's Zuul that acts on Depends-On.


Unable to remove alternate airport

Describe the bug
When creating a route, if an alternate airport is added, it cannot be removed - no option to return that field to blank is available.
Version
7.0.0-beta.4
To Reproduce
Add a flight
Select an alternate airport
Attempt to change it to blank
Expected behavior
A blank airport should be selectable to remove the current alternate and set the flight back to being without alternate.

Describe the bug
When creating a route, if an alternate airport is added, it cannot be removed - no option to return that field to blank is available.
Version
7.0.0-beta.4
To Reproduce
Add a flight
Select an alternate airport
Attempt to change it to blank
Expected behavior
A blank airport should be selectable to remove the current alternate and set the flight back to being without alternate.

If you see correctly, there is a little blank space above the first airport in that list. Just click on it.

This should be closed too, selecting the blank was possible back then and still possible (with latest dev of course, not sure about the beta4)


[Snyk] Upgrade debug from 4.1.1 to 4.2.0

Snyk has created this PR to upgrade debug from 4.1.1 to 4.2.0.
As this is a private repository, Snyk-bot does not have access. Therefore, this PR has been created automatically, but appears to have been created by a real user.

:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released a month ago, on 2020-05-19.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

snyk removed from project dependencies


Feature/fix file structure

Description
https://www.loom.com/share/802390482d044ecc8d0d0ce8a8a419bc

placed document cards in proper file
fixed key placements
added mock data to proper folder
adjusted tests for implementation of redux

Fixes # (issue)
Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have removed unnecessary comments/console logs from my code
[x] I have made corresponding changes to the documentation if necessary (optional)
[x] My changes generate no new warnings
[x] I have checked my code and corrected any misspellings
[x] No duplicate code left within changed files
[x] Size of pull request kept to a minimum
[x] Pull request description clearly describes changes made & motivations for said changes

by:

Jacob Shomali
Cara Locke
Juliane Vo Nguyen

This PR is not granular and I see multiple tasks for multiple Trello cards, as well as tasks that are not reflected in cards nor discussed with the team. UX and FE met today and got on the same page on file structure as well as component names and route tree. Bailey is drafting an update for the team as I type this review. Please align with the team before making a bunch of changes around the repo. Remember if you need help "finding" PRs talk to your TPMs. Do not go rogue please. It causes merge conflicts if we're not on the same page.


WIP Mp/dev and test cleanup with travis

This builds atop on @smith's added .travis.yml and gets for the tests running in both travis and docker.
This also cleans up a few things:

minor unit test changes to work correctly in docker and travis env
include rebar 3.2
remove override build settings for erlzmq since chef/erlzmq is (or will shortly be) updated with rebar3 support.

http://wilson.ci.chef.co/job/opscode-push-jobs-server-trigger-ad_hoc/24/


frequency band selection based on class distinctiveness

Hello!
I created a new function for frequency band selection on the manifold using the class distinctiveness measure.
This function finds the frequency band with the large class distinctiveness for training data from a broad frequency band that the user input. For instance, if the user inputs 5-35Hz,  the function adjusts the band and returns 8-12Hz as the most class-distinctive frequency band.
I added its example code using a motor imagery dataset, so please take a look.
Using an optimized frequency band is one of the important aspects for enhancing oscillatory activity classification, so hopefully, this new function is helpful for the user:-)

Ah… I did not expect but so many old commits were included in this new pull request...
This is not the proper way to do it, is it?

You can merge master into this branch to fix it
Next time when you create a new PR create the new branch from the most up to date master

I see! I should have created a new branch from the latest state master.
I am sorry I did not know that...
I will close this pull request and create the new one.
Thank you so much for your help!


useEffect cleanup not being called

Don't have a simple example to share at the moment, but curious if this library supports calling the cleanup of the useEffect hook when there are no more instances of the hook in the React DOM.

Hi @ebramanti, many thanks to your issue!
This is not a bug, it's a subtle feature 😊
Now instances exist regardless of the state of the DOM and always exist after the first call. This gives an advantages in testing, makes it possible to work with shared logic before building a DOM (for example, to start loading the current user, even before the first render occurs).
But... I feel there is a need for the logic you describe.
The "free" function is now implemented. This allows you to destroy the shared state whenever you want.
‘’’javascript
import { free } from “use-between”;
free(useCounter); // Any shared hook
‘’’
But I feel that this is not quite what is needed. Maybe we can add something like that to the library API:
‘’’javascript
const App = () => {
const { count } = useBetween(useCounter, { arc: true });
return {count};
}
‘’’
“arc” means - automatic reference counter.
What do you think about?

Hmm.
Hi @ebramanti again!
In the previous "arc" option, I don't like the confusion that arises when using the same shared hook with and without the "arc" option. It is also not clear how to deal with the “get” operation.
get(useCounter).inc()
With this in mind, I propose the following solution:
const App = () => {
const { count } = useBetween(useCounter, { freeOnUnmount: true });
return {count};
}
In fact, it will be an alias to the following code:
const App = () => {
const { count } = useBetween(useCounter);
useEffect(() => () => free(useCounter), []);
return {count};
}
What do you think about?

looks good to me.


Sending transactional template discards subject specified via data object

Issue Summary
When using the sendgridMail.send function to send a Transactional Template, any subject line added to the data object which is passed sendgridMail.send (via data.subject or data.personalizations[someIndex].subject) is discarded and the subject line that is specified on the actual Transactional Template itself is used.
The behaviour that I expect is that the fields specified on the data object would override any existing template fields, but this does not seem to be the case. Looking at the documentation for the v3 Mail Send API, the template_id field's description states:

The id of a template that you would like to use. If you use a template that contains a subject and content (either text or html), you do not need to specify those at the personalizations nor message level.

To me, "you do not need to specify those at the personalizations nor message level" implies that you can override the template subject. If that is not the case, the description should be changed to "you cannot specify those at the personalizations nor message level".
Example

I created a transactional template with the subject "My Email Subject"
I tried to send an email using sendgridMail.send where I specified data.subject to be "[TEST] My Email Subject"
The email I received had the subject line "My Email Subject"

Steps to Reproduce

Create a transactional template on https://sendgrid.com that has a subject specified
Use the sendgridMail.send function to send the template. Also make sure to specify either data.subject or data.personalizations[someIndex].subject.
Confirm that email is received with the original subject line, not the one passed to the API

Technical details:

sendgrid-nodejs Version: 6.3.1
Node.js Version: v8.11.3

Hello @10kc-zack,
I believe in the transactional template you would set the subject to be {{ subject }}, then in your code you set data.subject to be "[TEST] My Email Subject".
Does that work for you?
With Best Regards,
Elmer

Thanks for the reply @thinkingserious and your suggestion!
While this does get the job done (and is the work-around we are currently using), I would much prefer being able to set the subject as part of our API call body. Are you able to confirm that it is in fact the case that the subject provided to the API is discarded and that that information is missing from the API documentation?

Hello @10kc-zack,
I believe in the transactional template you would set the subject to be {{ subject }}, then in your code you set data.subject to be "[TEST] My Email Subject".
Does that work for you?
With Best Regards,
Elmer

As @10kc-zack stated this workaround does work but I've also noticed the template is converting some characters such as '  apostrophy.. ie "Someone's" to html entities.  My email arrives with the subject Someone's

Doh! I found where the {{subject}} needs to be set. In case anyone else has this issue, go in to edit your template, then click "Settings" on the left. There will be a space to enter your Subject. Enter {{subject}} here.

@10kc-zack,
I have a request open to make that change the documentation. I'm assuming you are referring to this documentation, correct?
With Best Regards,
Elmer

@thinkingserious That is indeed the documentation I was referring to :+1:

are there any plans to be able to set the subject via the API without using dynamic_template_data?

@mou01 Simple example here for how to set the subject: https://github.com/sendgrid/sendgrid-nodejs/blob/master/docs/use-cases/single-email-single-recipient.md

Yes, it's rather confusing that you can't pass the subject via personalizations when using dynamic template data, you have to pass it in the data it seems.

@mou01 Simple example here for how to set the subject: https://github.com/sendgrid/sendgrid-nodejs/blob/master/docs/use-cases/single-email-single-recipient.md

does this work with dynamic templates?

I think not, I think the idea behind the dynamic templates is that you set the subject in the template set up, and you get to use dynamic data in the subject. The trade off is that you have to pass it in dynamic data if you want it to be fully flexible.

if anyone having this issue:
go to your template and click the "Settings" with a gear icon and put this on the subject field {{subject}}
and finally when sending your payload "dynamic_template_data" should have the subject value.
 "dynamic_template_data":{             "subject": "My Subject Value here" }

Closing this issue as the docs change is not something that can be fixed in this repo. I've submitted a request to the docs team (reference title: Updates to mail-send template_id description) and will circle back here with any updates.

Docs have been updated.

Thank you @childish-sambino

The triple brackets is still working, is that still the official solution?
Thanks!

@tcukanoot Yes.


AtomicFile::new with a relative path like foo will panic

If you do this:
fn main() {
    AtomicFile::new("foo", OverwriteBehavior::AllowOverwrite).write(|_| {
        Ok::<(),()>(())
    }).unwrap();
}

then you get an I/O error:
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: Internal(Os { code: 2, kind: NotFound, message: "No such file or directory" })', libcore/result.rs:945:5

I believe the cause is that new invokes parent() and then supplies (if there is no parent) the value p:
https://github.com/untitaker/rust-atomicwrites/blob/3ad97ac63fc6dc92fc0887d66502e99403dda7e4/src/lib.rs#L84-L87
Probably it should supply the current directory? Or perhaps just None?
(My workaround was to invoke canonicalize on the path.)

This is really bad, thanks for reporting. The intent is to put the temporary file in the same directory as the target file to ensure they are stored on the same physical volume. Otherwise rename will not be atomic

Well, today I learned how parent() actually works. Thanks, released 0.1.5.


development-workspace image: integrate all auto-tools dependencies

Hi,
the development-workspace Image is one of the most important images for us, as it allows us to port and compile different packages.
Unfortunately, we can't port everything with the development-workspace image, as it hasn't included all auto-tools packages. It includes e.g. autoconf, but not automake, neither the m4 stuff required by autoconf.
Instead of only using development-workspace image, I used the node:17 image to build the last dependencies, which worked fine.
Would it be possible to include the other auto-tools components into the development-workspace image? With this, we wouldn't need another image as intermediate image for our build process.
Thanks in advance & regards
Michael

We solved this otherwise.


[API] Message filters

Hi,
For testing purpose, i need to filter messages returned by GET /messages endpoint.
Actually it seems not possible.
Do you think you can provide a such feature asap ?
If i provide a PR, do you think it can be merged asap ?
Thanks !

Can you not filter in your consumer?

I'd consider adding this, but would prefer to do so carefully. If you'd like something which works already maybe try MailHog — they seem to have a search endpoint.

This is my use case,
I just want to know if a known mail gots new mail.
If i can filter by recipient, i can easily count the result.
For the moment, i need to iterate over each messages and put in a list for each recipient, and then count the a specific recipient. Not really friendly ...
Do you see the use case ?

Ah, I see the use case. Have you tried something like:
HTTParty.get("http://localhost:1080/messages").any? { |message| message["sender"].include<EMAIL_ADDRESS>}
# => true

So, i did like this in groovy, and the solution is OK ;)
Thanks !


Make dataframe object similar to seqlogo Ppm

I don't know if seqlogo Ppm is a generalized class or not, but effectively it achieves what I originally intended to. Have a general purpose class/object with properties (such as an alphabet), but when it is called it is effectively a pandas dataframe. Then maybe I could have functions attached to it (such as calculate_nEffective from #8 ) that could add columns to itself (ideally, only if they don't already exist, of course with the option to overwrite). This would make #10 much easier to deal with too!

We have this now!
https://github.com/tijeco/berteome/blob/9cf5a72af7e17cdb705fdad20153d9cbaf247487/berteome/berteome.py#L13


add aspen mesh to general members

Pre-submission checklist:
Please check each of these after submitting your pull request:

[ ] Are you only including a repo_url if your project is 100% open source? If so, pick the single best GitHub repository for your project, not a GitHub organization.
[ ] Does it follow the guidelines for new entries?
[ ] Have you added an SVG to hosted_logos and referenced it there?
[ ] Does your logo clearly state the name of the project/product and follow logo guidelines?
[ ] Have you verified that the Crunchbase data for your organization is correct (including headquarters and LinkedIn)?
[ ] ~5 minutes after opening the pull request, the CNCF-Bot will post a link for you to preview your changes. Have you confirmed that they are correct and then added a comment to the PR saying "LGTM"?

Build failed because of:
Aspen Mesh  has a crunchbase https://crunchbase.com/organization/aspen-mesh which does not start with 'https://www.crunchbase.com/organization'

Build failed because of:
Aspen Mesh (member)  has a crunchbase https://crunchbase.com/organization/aspen-mesh which does not start with 'https://www.crunchbase.com/organization'

LGTM @tracymiranda 🎉


Reuse strndup(), simplify code

The original code would contain strndup() reimplemented, a bunch of unneeded casts and variables with too wide scope which made code harder to analyze.

minor nit, but +1 from me and I'll merge after that's fixed and another review.

Yup

pushed with commit 1caaea1,  thanks!

pushed in 1caaea1 thanks!


Atwho: shows users who are not in a room

The user list that pops up when you type @ includes users who are not in the room. This is misleading since we don't have the ability yet to send messages to users who aren't in a room,

@hhaidar - Simon tells me this is expected behavior?

Yea, it's nice to at people even if they're offline. We have another ticket open about showing online/offline status in the atwho plugin.

We now use @ for users currently in room, and @@ for all registered users.


Generated approval test file names can become too long

Is the feature request related to a problem
On Windows we unfortunately still need to live with a maximum path length when creating files. This can quickly become a problem when using various build agents who create folders in folders to perform the builds, combined with long nested namespaces.
I am using this library because it works perfectly when combining xUnit with xBehave and ApprovalTests as xBehave's [Scenario] attribute uses the xUnit [Theory] attribute.
Given the following test class:
namespace Long.Namespace.Example.Tests
{
  public class Examples : XunitApprovalBase
  {
    public Examples(ITestOutputHelper output)
      : base(output) {}

    [Scenario]
    public void TestMethod()
    {
        "When I verify the contents"
          .x(() => Approvals.Verify("foo"));
    }
  }
}

When generating a unique test name, the following snippet is used:
var name = $"{method.TestClass.Class.Name}.{method.Method.Name}";

The Name property on the class returns the fully qualified type name (for non-generic parameters), or the simple type name (for generic parameters). (According to the xUnit.TestClass.Name documentation.)
This means that in my example the generated file name will be Long.Namespace.Example.Tests.Examples.TestMethod.approved.json.
When using xUnit's [Fact] attribute the correct class name is used for the file, but I loose the ability to use xBehave's scenario's or xUnit's [Theory] attributes.
If my file names get too long, I can't get them checked into git anymore because they are too long for git's path index, and even with a shorter name my build server freaks out because it can created folders like D:\agent-1\working-dir\project\s\... were I already lose a lot of characters before the file is even generated.
Describe the solution
When generating the unique test name, only the class name should be used because the files will be in the correct folder anyway. Or allow the use of the test display name or another override.

as an aside, i dont hit this since namespaces are usually redundant for tests and hence i dont use them.
as for your proposal, how would u handle a conflict? ie multiple files in the same dir with the same name but diff namespaces?

You could always allow us to override the default behavior (the current behavior) to counter this.
In my case, I always have a physical folder per namespace too, so I won't have multiple files with the same name in the same folder.
Even then, since the file name contains both the class name and the method name (with optional parameters), I think the changes are low enough for this to be an edge-case which can be bypassed by overriding the generated class name for a single test.

This is now deployed. NuGet may take some time to make it available for download.


Libra - singlecell giving opposite results

Hi Jordan,
Thanks for creating & maintaining Libra
I've noticed that when using de_family = "singlecell" & de_method = "wilcox" , for some of my comparisons, the results are exact opposite to what I get when I run FindMarkers(..., test.use = "wilcox")  directly on my Seurat object. I've also corroborated this by looking at gene expression differences between the groups in my dataset and the Libra result seems to be opposite to that.
I am wondering what might be causing this reversal ? Does the presence / absence of factors on the label_col affect the design of the comparison?  I also noticed in the below referenced code, it is mentioned, to avoid logFC conflicts ? Could you clarify this a bit more?
https://github.com/neurorestore/Libra/blob/3598007b70232f294872bfa60ef0212324b728a3/R/singlecell_de.R#L56-L64
Have you come across any such cases in your testing?
Thanks,
Somesh

Seurat flips the factor order differently compared to all other DE methods so we flip it to match everything else. If you want it to be identical to Seurat FindMarkers you can just set your factor levels.


Em 3700

JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/EM-3700
Change description
EM-CCD-Orchestrator: Add functional tests for NewBundleController
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[X] No

some issue with this PR dont see changed files, will raise another PR


Problem with voyager avatar and folder path.

Version information

Laravel: v5.7.17
Voyager: v1.1
PHP: 7.2.12
Database: MySQL

Description
The image for the avatar is broken, and the path for the default.png is not working.
Screenshots

Additional context
Also I would like to ask if the Voyager 1.1 has different path for the /storage?

Run the command "php artisan storage:link". Here you have more info.

Run the command "php artisan storage:link". Here you have more info.

I already tried it, and I also tried deleting the file and running it again.

Try with php artisan config:cache. If still fails, check #722 and this..

@emptynick some ppl said clearing cache solved this issue after creating the symlink to storage

Clearing config cache would only work if you changed the setup in config/filesystems.php and had the config cached before.
Please make sure public/storage is actually a symlink to storage/app/public, and not just a directory.  I see you're in windows, but if you had a bash terminal (cygwin, git-bash, etc), you could run la -al public/ to see what it shows as.  Or view properties in windows explorer. Should show as a "junction" or something I think

Make sure you have a correct APP_UR in .env

I tried and it called default.png but in admin panel, still broken !!!


Create A Responsive Calculator Design

Things to keep in mind

Make sure it is responsive.
Layout design should look good
You can use any library you want
Read README.md for instructions

Note - Only create designs using HTML, CSS or JavaScript do not need to code it's functionality.

I can help out with this

Could i help you on this?

Let me help with this...

Yes @RamanaVem you can help by developing a better version of computer.
Create Calculator Designs Here
Strict instructions

Please make a new folder
Put all your files inside that folder
And do not make changes in README.md

Note

Create a readme.md file inside folder you have created.
Put your Email address or name inside readme.md.
You can also provide other information about you in readme.md.

@adarshbalu, Yes you can help by developing a better version of computer.
Create Calculator Designs Here
Strict instructions

Please make a new folder
Put all your files inside that folder
And do not make changes in README.md

Note

Create a readme.md file inside folder you have created.
Put your Email address or name inside readme.md.
You can also provide other information about you in readme.md.
Yes @RamanaVem you can help by developing a better version of computer.

Create Calculator Designs Here
Strict instructions

Please make a new folder
Put all your files inside that folder
And do not make changes in README.md

Note

Create a readme.md file inside folder you have created.
Put your Email address or name inside readme.md.
You can also provide other information about you in readme.md.

Will a working tip calculator work in here?


Add battery actuator temperature

Hi @weltenwort I consider this entity useful. I use it together with the heat sink temp for controlling our external fan. So I wanted to be available also for others.

Hi @gorge511, could you clarify which temperature measurement that is? Is it different from the battery_temperature that is already exposed?

Ah, I guess you mean db.temp2 (0x4F735D10)? Sure, that should be easy to add.

Would you mind adding an entry to the entity table in the README?

Sure, no problem. Added in https://github.com/weltenwort/home-assistant-rct-power-integration/pull/352/commits/256f2d47936e0fea2608e93d1e287cc9563245e0

:information_source: released in https://github.com/weltenwort/home-assistant-rct-power-integration/releases/tag/v0.13.1


Collaborative Development Using Github and SAP Web IDE - parts 1 to 6

PR to merge tutorial for Collaborative Development Using Github and SAP Web IDE - parts 1 to 6.
cla-assistant.io API is down, so I can't sign a CLA just yet.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


[FLINK-15723] Update AbstractKeyedOperatorRestoreTestBase to restore from 1.10 savepoint

What is the purpose of the change
This pull request updates AbstractKeyedOperatorRestoreTestBase to restore from 1.10 savepoint
Brief change log

Update AbstractKeyedOperatorRestoreTestBase to restore from 1.10 savepoint

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@flinkbot run travis