Find route sample

Simple routing sample: two stops, no barriers. Click to solve route and get directions.

Hey @nCastle1 ... would you mind taking a look at this if you have time?

@ThadT I noticed that the screenshots were out of date for some of the samples. As I had to open and run the samples to review them, I decided to save some time and take screenshots simultaneously. With those changes having been made, this looks good and ready to merge.

@ThadT Fully tested on all platforms post-merge, ready to merge into master.

Thanks @nCastle1 !


Angular templates to devon4j 3.0.0

Changing Angular templates to fit the new devon4j/oasp4j 3.0.0 version.
Implements

Added pageable, searchCriteria and sort interfaces.
Fixed issue on the server side when doing pagination.
Fixed a bug when sorting.
Fixed a bug when searching for elements.
Renamed the Angular application template to devon4ng.

@devonfw/cobigen

I will also push here the changes related to crud_openapi_angular templates

Ready to be reviewed and merged @maybeec


Allow use non-unique indexed fields in templates

Allows generation functions like "getBy{{field}}" for non-unique indexed fileds using custom templates

Thank you very much.

This message was created automatically by mail delivery software.
A message that you sent could not be delivered to one or more of its
recipients. This is a temporary error. The following address(es) deferred:
<EMAIL_ADDRESS>Domain reulen.nl has exceeded the max emails per hour (5/5 (100%)) allowed.  Message will be reattempted later
------- This is a copy of the message, including all the headers. ------
Received: from mail-ob0-f177.google.com ([<IP_ADDRESS>]:34542)
by fyvie.footholds.net with esmtps (TLSv1.2:ECDHE-RSA-AES128-GCM-SHA256:128)
(Exim 4.86_1)
(envelope-from<EMAIL_ADDRESS>id 1altn9-0000IF-Ny
for<EMAIL_ADDRESS>Fri, 01 Apr 2016 08:46:51 +0100
Received: by mail-ob0-f177.google.com with SMTP id kf9so116260400obc.1
for<EMAIL_ADDRESS>Fri, 01 Apr 2016 00:46:31 -0700 (PDT)
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
d=1e100.net; s=20130820;
h=x-gm-message-state:delivered-to:date:dkim-signature:from:reply-to
:to:message-id:in-reply-to:references:subject:mime-version
:content-transfer-encoding:precedence:list-id:list-archive:list-post
:list-unsubscribe;
bh=FQh2mvv4p2KGDTAd6Nb0l8R/iD0BmY+iWDzNkDfL2jw=;
b=ZO4wyTHFF/NsfYlta1MPVcw0UZ6Bxv+lU9WVwy8KUok4Uz+gnOekJXl5IbBDJVzi2+
JPydmzRsYdm31wSlZAu2dbd1tSh//KT4UTQVMUrEQfqD2QJz3LFDL26tNEnazo3oum1z
uHWgxLIXiS0J0w0afK44QLJo3l8unzLitbto6CczW1uidHoaVaLG3Tcvm2UfIa7TgydG
XHid/AYaTltnSQABnmfpvJbTEEZgX0tvVUhGW8cKsFnJgiJTZ82tSLMtvdFnMf40Cmwu
AzVybF1/2FJIGCGwDUD/Y4cPhIIxnMTIbO52s8N/Vdvch4JQlRNeMlmvLXYcqtbyVgDg
Iukw==
X-Gm-Message-State: AD7BkJKuL0XKqyTWZWrnDT5h009UPPHqasjBFWXUcULMZkH9XV1SURYnpoHsHK3OpNvmK5sweAfhGIlfbw2PGQAgn0Vc3Fo=
X-Received: by <IP_ADDRESS> with SMTP id v8mr1892257obn.48.1459496770800;
Fri, 01 Apr 2016 00:46:10 -0700 (PDT)
X-Forwarded-To<EMAIL_ADDRESS>X-Forwarded-For<EMAIL_ADDRESS><EMAIL_ADDRESS>Delivered-To<EMAIL_ADDRESS>Received: by <IP_ADDRESS> with SMTP id 94csp530612ots;
Fri, 1 Apr 2016 00:46:10 -0700 (PDT)
X-Received: by <IP_ADDRESS> with SMTP id k63mr24524882qhc.11.1459496770259;
Fri, 01 Apr 2016 00:46:10 -0700 (PDT)
Received: from github-smtp2a-ext-cp1-prd.iad.github.net (github-smtp2-ext8.iad.github.net. [<IP_ADDRESS>])
by mx.google.com with ESMTPS id z192si11079181qka.1.2<IP_ADDRESS>.46.10
for<EMAIL_ADDRESS>(version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128);
Fri, 01 Apr 2016 00:46:10 -0700 (PDT)
Received-SPF: pass (google.com: domain of<EMAIL_ADDRESS>designates <IP_ADDRESS> as permitted sender) client-ip=<IP_ADDRESS>;
Authentication-Results: mx.google.com;
dkim=pass (test mode<EMAIL_ADDRESS>spf=pass (google.com: domain of<EMAIL_ADDRESS>designates <IP_ADDRESS> as permitted sender<EMAIL_ADDRESS>dmarc=pass (p=NONE dis=NONE) header.from=github.com
Date: Fri, 01 Apr 2016 00:46:09 -0700
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com;
s=pf2014; t=1459496769;
bh=FQh2mvv4p2KGDTAd6Nb0l8R/iD0BmY+iWDzNkDfL2jw=;
h=From:Reply-To:To:In-Reply-To:References:Subject:List-ID:
List-Archive:List-Post:List-Unsubscribe:From;
b=H4mDHQV2fMfEY2f5hNP1wIJg4eXbS6j5c3MonrUhuT8MwdFzdSWlDSdhdHrVTmgV9
mmMVztfoPL1Va5QcT604JFXQ1aHSVefPP1STYoh1FPAGJOuty1pZkZ3A6Sva582xCk
q4+Zkds5y/Bjui8ISm5Ij8i4538X6l8/UsBRKWQQ=
From: Jia Mi<EMAIL_ADDRESS>Reply-To: mijia/modelq<EMAIL_ADDRESS>To: mijia/modelq<EMAIL_ADDRESS>Message-ID<EMAIL_ADDRESS>In-Reply-To<EMAIL_ADDRESS>References<EMAIL_ADDRESS>Subject: Re: [mijia/modelq] Allow use non-unique indexed fields in templates
(#20)
Mime-Version: 1.0
Content-Type: multipart/alternative;
boundary="--==_mimepart_56fe2741e09e9_34863fdb29de92b832728c";
charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: mijia
X-GitHub-Recipient: Gys
List-ID: mijia/modelq <modelq.mijia.github.com>
List-Archive: https://github.com/mijia/modelq
List-Post<EMAIL_ADDRESS>List-Unsubscribe<EMAIL_ADDRESS>https://github.com/notifications/unsubscribe/AA5kk8J4477oZy08XxZxKi-bRs-K4E12ks5pzM1BgaJpZM4H9Si0
X-Auto-Response-Suppress: All
X-GitHub-Recipient-Address<EMAIL_ADDRESS>----==_mimepart_56fe2741e09e9_34863fdb29de92b832728c
Content-Type: text/plain;
charset=UTF-8
Content-Transfer-Encoding: 7bit
Thank you very much.

You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/mijia/modelq/pull/20#issuecomment-204296347
----==_mimepart_56fe2741e09e9_34863fdb29de92b832728c
Content-Type: text/html;
charset=UTF-8
Content-Transfer-Encoding: 7bit
Thank you very much.
—You are receiving this because you are subscribed to this thread.Reply to this email directly or view it on GitHub

----==_mimepart_56fe2741e09e9_34863fdb29de92b832728c--


when condition to see if element is present on the page by id

is there any way to use
when:
visible
not by text but by id

Yep. Anything Selector-ish is allowed
- runFlow:
    when:
      visible:
        id: my_id


Permission model to use for KayVault access

Hi,
You should add in the documentation that it also possible to use the Permission model "Azure role-based access control" for the KeyVault.
I.e. for the example (using Permission model “Vault access policy” for the KeyVault)
"# set policy to access secrets in your key vault
az keyvault set-policy -n  --secret-permissions get --spn "
You should also provide the info that it's possible to use Permission model "Azure role-based access control"
Example something like this:
"az role assignment create --role "Key Vault Secrets User" --assignee  --scope /subscriptions//resourcegroups/"
BR
Johan Stenberg

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

ID: ec9ff712-aa4c-4e95-ba96-2405010d9f83
Version Independent ID: f553a5a4-a20f-8efd-3712-e64112c35676
Content: Provide an access identity to the Azure Key Vault Provider for Secrets Store CSI Driver for Azure Kubernetes Service (AKS) secrets - Azure Kubernetes Service
Content Source: articles/aks/csi-secrets-store-identity-access.md
Service: container-service
GitHub Login: @Nickomang
Microsoft Alias: nickoman

@johanstenberg-sj Thank you for your feedback!  We've assigned this to the document author for further review.

Hi, @johanstenberg-sj, thanks for your feedback! This doc covers using Azure Key Vault Secrets Store CSI Driver in an AKS cluster. When you refer to using the permission model, are you suggesting this is added to the doc in question or in a separate doc? Closing the issue for now. Thanks again!
#please-close


Parses and renders release notes

What does this PR do?

Parses the release notes gathered from GitHub
Finds the desired version in the release notes
Replaces any relative links in the markdown
Renders the markdown on the command line
Adds a flag to suppress errors, this will be used in the post update/install scripts
Adds a footer with more information about the release notes (requires --hook flag)

What issues does this PR fix or reference?
@W-10120079@
@W-10120109@
Testing Instructions
This plugin will get configuration from the cli's pjson, you will need to add this config locally to test this PR. You can find these pjsons by running which sfdx or which sf.
For sfdx

sfdx plugins:link .
Add the following to pjson.oclif:

"info": {
  "releasenotes": {
    "distTagUrl": "https://registry.npmjs.org/-/package/sfdx-cli/dist-tags",
    "releaseNotesPath": "https://raw.githubusercontent.com/forcedotcom/cli/main/releasenotes/sfdx",
    "releaseNotesFilename": "README.md"
  }
}

For sf

sf plugins:link .
yarn build important
Add the following to pjson.oclif:

"info": {
  "releasenotes": {
    "distTagUrl": "https://registry.npmjs.org/-/package/@salesforce/cli/dist-tags",
    "releaseNotesPath": "https://github.com/forcedotcom/cli/tree/main/releasenotes/sf",
    "releaseNotesFilename": "README.md"
  }
}

Then try out some commands

sfdx info:releasenotes:display
Also try the alias: sfdx whatsnew
sfdx whatsnew -h
sfdx whatsnew -v 50.0.0

Note that this returns the release notes from v50.md

sfdx whatsnew -v latest (gets README.md)
sfdx whatsnew -v stable-rc (gets README.md)
sfdx whatsnew -v rc (gets README.md)
sfdx whatsnew (uses installed version)
sfdx whatsnew --hook (renders footer)
PLUGIN_INFO_HIDE_FOOTER=true sfdx whatsnew --hook (hides footer)
PLUGIN_INFO_HIDE_RELEASE_NOTES=true sfdx whatsnew (hides notes)
sfdx whatsnew -v foo (throw error)
sfdx whatsnew -v foo --hook (only shows warning, not error)

@jshackell-sfdc Here's what the footer and help docs look like with the substitutions:

Looks good on Windows:

@jshackell-sfdc here is an update screenshot. Thank you!

!

QA Notes:
❓ we don't want to return anything for json?  Maybe even a url would be nice?
✅ displays correct RN on stable-rc
✅ displays correct RN on stable
✅ handle rc as a version
❌ sfdx whatsnew --version 7.121.0 results in an error.  And the error has a url that's a broken link.
ERROR running info:releasenotes:display:  Didn't find version '7.121.0'. View release notes online at: https://raw.githubusercontent.com/forcedotcom/cli/main/releasenotes/sfdx

✅ sfdx whatsnew --version 7.121.8 works (because that's a real version).  I think it was supposed to ignore the patch digit.
👎🏻 that footer has a lot of space in it.  Maybe single spaced, with bullets?  For bigger release notes on a smaller-capacity terminal, it's taking up most of the screen
👎🏻 sfdx whatsnew --version 7.121.8 --hook returned a link with https://raw.githubusercontent.com/forcedotcom/cli/main/releasenotes/sfdx/./README.md#5140-march-18-2021---cli-7920 rather than a nice-output link
✅ envs work fine
✅ errors are thrown without --hook, warning if --hook
✅ I screwed up all the values in the package.json and --hook still only gave warnings (I tried this lots of ways attempting to break it)

Hey @mshanemc, sorry the info.releasenotes.releaseNotesPath url changed since you tested it last and I had the wrong url in this PR description. It should be https://github.com/forcedotcom/cli/tree/main/releasenotes/sfdx now (for sfdx). That should correct some of the link errors you mentioned.

I need to think about JSON output, will add that to another ticket.
Good point on the footer. It was originally 3 lines. Now that it's 4, it is too tall. Will change to a bulleted list
I wonder if the "expanded" link has something to do with the url being incorrect? Try it again after you update your package.json. It looks fine for me.

Worth noting that the marked-terminal library looks to see if your terminal supports hyperlinks and will print them out if it doesn't. You can see this in some of the CommandPrompt testing Rod did above ^

One final note, you had a comment about it not being able to find 7.121.0. That's because it actually doesn't exist in the Release notes. 7.121.8 is the closest. I had originally thought we would look for major.minor in the release notes, but sf is handling their versioning differently than sfdx. They are doing true semver where some weeks just the patch will bump. That makes the major.minor lookup more complicated. Especially when you consider a scenario where we look for major.minor.patch and if it's not found we find the closest. Do we look forward to the next patch or backward? I feel it's better to error out rather than making an assumption on what the user wants and potentially presenting them with incorrect information.
We have talked about having a info:releasenotes:list command that could be used in this situation. "Version 1.2.3 was not found, run sfdx info:releasenotes:list to see available versions or view them online here [...]"

@iowillhoit ,  I think that sf is meant to follow the same versioning rules as sfdx.  But I'm not sure if the process is still set in stone.  Enforcing that is above my pay grade :). Anyway, see this Slack thread for more info: https://salesforce-internal.slack.com/archives/C02J2118Z47/p1635440935079000
FYI @mdonnalley

Thanks for sharing the thread @jshackell-sfdc. I do remember seeing the beginning of that thread and assumed that semver was being followed because there are two version in the RN with the same minor (1.1.0 and 1.1.5). I still don't think we want to be just searching on major.minor though.
Another example would be a week we do a hotfix. Say someone has 1.0.0 installed and we do a hotfix a few days after release of 1.0.1. If they run sfdx whatsnew, I think it would be best for the script to say "i couldn't find 1.0.0" instead of showing them notes for 1.0.1 (when they still have 1.0.0 installed). Ideally the script would offer the versions that were available, but that won't be included at launch.


Prep Support/wagtail 63

Includes: #10

The branch here #10, merged into this didn't work with wagtail 6.3. The AutocompletePanel doesn't show up.
However the release package at release 0.11.0 does work OK with Wagtail 6.3 so I will close this.


Trap zone mechanism is broken if Dropdown is used inside of a popup with a trap zone or in Dialog

Bug Report
Trap zone mechanism is broken if Dropdown is used inside of a popup with a trap zone or in Dialog
Steps
Create an example with Dropdown inside of a focus trap popup.
Select an item
In the input box, press shift+tab
Expected Result
Focus should be trapped inside of the popup
Actual Result
Focus goes outside of the popup

fixed by #917


feat: List component to use styles

Types of Changes
What types of changes does your code introduce? Put an x in all the boxes that apply:

[ ] Bug fix (non-breaking change which fixes an issue, please reference the issue id)
[ ] New feature (non-breaking change which adds functionality, make sure to open an associated issue first)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] My change requires a change to the documentation
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] All new and existing tests passed

Description
[Place a meaningful description here.]

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Enable use of redis.conf in start_servers.sh

Apparently I never set up start_servers.sh to let you specify a redis.conf file. So, in development, it wasn't possible to change how Redis persists data, and the built-in defaults only save periodically. That explains why running Redis for a short time and then shutting it down caused data loss. Commit 418aba3 lets you use a REDIS_CONF_PATH environment variable to tell start_servers.sh what configuration file you want to use for Redis. Doing this and setting the appendonly yes option in redis.conf fixes the problem described in #21. If you don't set the env var, then Redis will run as before using the built-in default configuration.
This also updates the README file based on this change and adds appendonly.aof to the .gitignore, which is needed if you turn on AOF persistence in Redis as the README now recommends.

@vphill @damonkelley @somexpert @jason-ellis @ldko Please review as you have time. Thank you!

This looks okay to me. :+1:

@somexpert You are correct! Fixed in d034c0d.

:thumbsup:


Implement Shadow Dom for the inspector pane

Implementing Shadow Dom might solve some possible issues and limits:

Page CSS affecting the UI
Make it possible to implement richer third-party libraries to handle the UI rendering and interaction (I heard AngularJS is cool)

For reference - Browser support:
http://caniuse.com/#feat=shadowdom
Firefox:
Available only if the user enables flag dom.webcomponents.enabled
Safari:
Not supported :(
Chrome:
Full support.
There is a polyfill for Shadow DOM (I believe Polymer uses it). It works by wrapping/patching every DOM API that would "break" shadow DOM otherwise. Last time I heard about it (> 1 year ago), I heard lots of talk that there were big performance implications. It would definitely be worth checking back in on that, though.


Is it possible to update a model with new data without retraining the model from scratch?

I need to update a model with new data without retraining the model from scratch. That is, incremental training for the cases when not all the data available right away.
This problem is similar to the "can't fit data in memory" problem, which was raised before in #56, #163, #244. But it's been 2-3 years ago and I see some changes in available parameters process_type and updater. The FAQ suggests using external memory via cacheprefix. But this assumes I have all the data ready.
The solution in #1686 uses several iterations over the entire data.
Another related issue is #2970, in particular https://github.com/dmlc/xgboost/issues/2970#issuecomment-354684604. I tried 'process_type': 'update' but it throws the initial error mentioned in that issue. Without it, the model gives inconsistent results.
I tried various combinations of parameters for train in Python. And train keeps making the model from scratch or something else. Here're the examples.
In a nutshell, this is what works (sometimes) and needs a feedback from more experienced members of the community:
print('Full')
bst_full = xgb.train(dtrain=train, params=params)
print(mean_squared_error(y_true=y_test, y_pred=bst_full.predict(test)))

print('Subset 1')
bst_1 = xgb.train(dtrain=train_1, params=params)
print(mean_squared_error(y_true=y_test, y_pred=bst_1.predict(test)))

print('Subset 2')
bst_2 = xgb.train(dtrain=train_2, params=params)
print(mean_squared_error(y_true=y_test, y_pred=bst_2.predict(test)))

print('Subset 1 updated with subset 2')
bst_1u2 = xgb.train(dtrain=train_1, params=params)
bst_1u2 = xgb.train(dtrain=train_2, params=params, xgb_model=bst_1u2)
print(mean_squared_error(y_true=y_test, y_pred=bst_1u2.predict(test)))

Here I'm looking to minimize the difference between the first and the fourth models. But it's keep jumping up and down. Even with equalling total boosting rounds in both methods.
Is there a canonical way to update models with newly arriving data alone?
Environment

Python: 3.6
xgboost: 0.7.post3

Similar issues

Continue training in a dynamic training data environment #1225
incremental learning, partial_fit like sklearn? #1686
[xgboost4j-spark] Incremental training #1859
Retrain Xgboost Model #2495
"xgb_model" parameter in train() doesn't increment learning correctly #2707

Contributors saying new-data training was impossible at the time of writing:

https://github.com/dmlc/xgboost/issues/2495#issuecomment-317240566
https://github.com/dmlc/xgboost/issues/56#issuecomment-53962722

Firstly, there is a paper about using a random forest to initialise you Gbm model to get better final results than just rf or Gbm and in few rounds. I cannot find it however :( This seems like a similar consept except you are using a different Gbm to initialise. I guess the other main difference is that it is on another set of data...
Secondly, sometimes it is more important to train a model quickly. I have been working on some time series problems where I have been doing transfer learning with LSTMs. I train the base model on generic historical data and then use transfer learning for the fine tuning on specific live data. It would take too long to train a full new model on live data when ideally I would. I think the same could be true of using xgboost. I. E. 95% of model optimal prediction is better than no prediction.

@hcho3  While my mention of the "concept drift" was in a broad sense, the boosting continuation would likely do better with "concept shifts" (when new data has some differences but is expected to remain stable). Picking slow continuous data "drifts" would be harder. But for strong trending drifts, even that random forest method might not work well, and some forecasting elements would have to be utilized. A lot would depend on a situation.
Also, a weak spot for boosted trees learners is that they are greedy feature-by-feature partitioners, so they might not pick well on such kinds of changes where the univariate effects were not so significant, e.g., when only interactions change. It might be rather useful if we could add some sort of limited look-ahead functionality to xgboost. E.g., in https://www.microsoft.com/en-us/research/wp-content/uploads/2017/06/kdd13.pdf they have a bivariate scan step that I think might work well as a spin of the histogram algorithm.
As for why the predictive performance in future data that is similar to "new data" is sometimes worse for a model trained over a combined "old data" + "new data" dataset, when comparing to a training continuation in "new data", this is because the former model would be optimized over the whole combined dataset, and that might happen at the expense of "new data" when that "new data" is somewhat different and relatively small.

I thought incremental training with minibatches of data (just like SGD) is kind of equivalent to subsampling the rows at each iteration. Is the subsampling in XGBoost only performed once for the whole training lifecycle or once every iteration?

I also need to use incremental learning. I've read all links that have been mentioned above. However, I'm confused.
Finally, is there any version of XGBoost to retrain a trained xgb model based on new received data point or batch of data?
I've found below links that addressed this issue before the date of this post. Don't they work? Can't we do incremental learning with them? What's the problem with them?
https://github.com/dmlc/xgboost/issues/1686
https://github.com/dmlc/xgboost/issues/484
https://stackoverflow.com/questions/38079853/how-can-i-implement-incremental-training-for-xgboost?utm_medium=organic&utm_source=google_rich_qa&utm_campaign=google_rich_qa
https://github.com/dmlc/xgboost/issues/2495

@benyaminelc90
This particular answer got close to proper incremental learning:

https://stackoverflow.com/a/47900435/911945

Still, as I learned from hcho3, GBM has limited capacity for updates without seeing all the data from the start. For example, it can easily update leaves but has difficulties with altering splits.

I came into this issue while searching for a way to train an xgboost model in an online manner. The primary use case for me is in reinforcement learning: we need to update the models after each time step.
Most people use neural networks as function approximators in RL. However, my experience has been that NN is very finicky and its performance in RL is very sensitive to hyper parameters. So, I'm looking for a solution that is more robust and less sensitive to hyper parameters. Boosted trees and random forests come to mind.


Revert "Change the package entry point to preprocessor.js itself"

Reverts kulshekhar/ts-jest#363
Some tests are failing on my local machine after this merge. Temporarily reverting this

Can someone please approve this?
/cc @GeeWee @bcruddy @morajabi

nvm. False alarm.
Sorry for bothering you all!


Support existing standards packages with subfolders

Proposed Changes

Search package sub-folders to a limited depth (< 4), to find PHP CodeSniffer standards
Only add installed paths where a ruleset.xml exists in a standards sub-folder
Update README to reflect requirements change
Fix some spelling mistakes in comments

Motivation
I want to use your excellent installer plugin with the Drupal PHP CodeSniffer standards. Their package uses a subfolder, one deeper than the repository root, which they're unable to change. I am sure there are other examples where Standards may be one or two folders deeper than the current installer allows.
Notify

@frenck

If this change or similar is accepted, allowing a slightly deeper directory tree for standards, I'll be submitting a change to the Drupal Coder module, asking them to adjust the package type from "library" to "phpcodesniffer-standard".
See this fork

https://github.com/christopher-hopper/coder

Changes look good, motivation is solid.

I've been using my changes with the following composer.json and it seems to be working very well:
{
  "type": "metapackage",
  "minimum-stability": "dev",
  "prefer-stable": true,
  "config": {
    "preferred-install": "dist",
    "discard-changes": true
  },
  "repositories": [
    {
      "type": "vcs",
      "url": "https://github.com/christopher-hopper/coder"
    },
    {
      "type": "vcs",
      "url": "https://github.com/christopher-hopper/phpcodesniffer-composer-installer"
    }
  ],
  "require": {
    "php": ">=5.4",
    "mouf/nodejs-installer": "~1.0"
  },
  "require-dev": {
    "dealerdirect/phpcodesniffer-composer-installer": "dev-master",
    "frenck/php-compatibility": "*",
    "drupal/coder": "dev-8.x-2.x"
  }
}

Once merged I will be submitting a pull request to the drupal/coder project so their Composer package type is changed to "type" : "phpcodesniffer-standard"


Update Composer dependencies (2022-04-03-00-11)

Visual regression test passed!
View the visual regression test report


add country field in profiles table #396

I think that pr will solve the issue #396

Hey @n3k00n3, great work!!!
Could you please check, why the Travis build is failing and make it unfailing ;)
We also have to add the country field to the detailed search but this could be made in another PR.
@tyranja could you also please have a look?

@zaziemo Done!

Great work :heart: We totally forgot in the issue to mention, that the country field should be searchable as well.

@n3k00n3 some test are failing - apparently because no profiles are shown on the search page (https://speakerinnen.org/en/profiles) which would be the expected behaviour. Could you please check why this doesn't work in your version?


[Bug Report] dae更新后daed不正常

Checks

[X] I have searched the existing issues
[X] I have read the documentation
[X] Is it your first time sumbitting an issue

Current Behavior
使用最新版本dae通过daed面板添加订阅节点会先报panic occurred: runtime error: invalid memory address or nil pointer dereference ，然后报database is locked (5) (SQLITE_BUSY)
Expected Behavior
No response
Steps to Reproduce
No response
Environment

Daed version:0.7.0
OS (e.g cat /etc/os-release):OpenWrt 23.05.4 r24012-d8dd03c46f
Kernel (e.g. uname -a):Linux OpenWrt 5.15.162 #0 SMP Mon Jul 15 22:14:18 2024 x86_64 GNU/Linux

Anything else?
No response

请尝试 0.8.0 或者 0.7.4 问题修复在 https://github.com/daeuniverse/dae-wing/commit/25b949bc9a204d78abd12faade9b43aa8f4c8fb8 ，这两个版本包含这个修复

已修复了


/benchmark/sql_regressions/DoltRegressionsJenkinsfile: Add sql watcher 4

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


Add API to configure database types for DbFunction's parameters

See https://github.com/aspnet/EntityFrameworkCore/issues/13736#issuecomment-432871876 for the setup and the conversion.
I attempted to map my own function like this:
    public static class GeoExtensions
    {
        public static double Distance(this GeoPoint x, GeoPoint y)
            => throw new NotImplementedException();
    }

which is based on the GeoPoint mapping in the comment linked above.
My attempt was this:
modelBuilder.HasDbFunction(
    typeof(GeoExtensions).GetMethod(nameof(GeoExtensions.Distance)),
    b => b.HasTranslation(
        e => new SqlFunctionExpression(
            e.First(),
            "STDistance",
            typeof(double),
            e.Skip(1))));

This failed here because there is no mapping registered for GeoPoint.
This would likely be fixed by #10784, but even then it probably should be possible to make work with a custom mapping.

Triage: this is too risky to try to fix for 2.1 and probably requires better type inference and/or the ability to explicitly associate type mappings with parameters in translation--the current code only allows type mappings to be applied for SQL generation if translation can succeed without them.
A workaround is below, but it won't work with preview3 due to other bugs to be fixed for RTM. Also, the workaround requires using internal code, which make break in subsequent releases.
public static class GeoExtensions
{
    public static double Distance(this GeoPoint x, GeoPoint y)
        => throw new NotImplementedException();
}

public struct GeoPoint
{
    public GeoPoint(double lat, double lon)
    {
        Lat = lat;
        Lon = lon;
    }
    public double Lat { get; }
    public double Lon { get; }
}

public class House
{
    public int Id { get; set; }
    public GeoPoint  Location  { get; set; }
}

public class GeoPointConverter : ValueConverter<GeoPoint, IPoint>
{
    public static readonly GeoPointConverter Instance = new GeoPointConverter();

    private static readonly IGeometryFactory _geoFactory
        = NtsGeometryServices.Instance.CreateGeometryFactory(srid: 4326);

    public GeoPointConverter()
        : base(
            v => _geoFactory.CreatePoint(new Coordinate(v.Lon, v.Lat)),
            v => new GeoPoint(v.Y, v.X))
    {
    }
}

public class BloggingContext : DbContext
{
    protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder)
        => optionsBuilder
            .ReplaceService<IRelationalTypeMappingSource, ReplacementTypeMappingSource>()
            .UseSqlServer(
                @"Server=(localdb)\mssqllocaldb;Database=Test;ConnectRetryCount=0",
                b => b.UseNetTopologySuite());

    protected override void OnModelCreating(ModelBuilder modelBuilder)
    {
        modelBuilder
            .Entity<House>()
            .Property(e => e.Location)
            .HasConversion(GeoPointConverter.Instance);

        modelBuilder.HasDbFunction(
            typeof(GeoExtensions).GetMethod(nameof(GeoExtensions.Distance)),
            b => b.HasTranslation(
                e => new SqlFunctionExpression(
                    e.First(),
                    "STDistance",
                    typeof(double),
                    e.Skip(1))));
    }
}

public class ReplacementTypeMappingSource : SqlServerTypeMappingSource
{
    public ReplacementTypeMappingSource(
        TypeMappingSourceDependencies dependencies,
        RelationalTypeMappingSourceDependencies relationalDependencies)
        : base(dependencies, relationalDependencies)
    {
    }

    protected override RelationalTypeMapping FindMapping(in RelationalTypeMappingInfo mappingInfo)
        => mappingInfo.ClrType == typeof(GeoPoint)
           || "geography".Equals(mappingInfo.StoreTypeName, StringComparison.OrdinalIgnoreCase)
            ? (RelationalTypeMapping)base.FindMapping(typeof(IGeometry))
                .Clone(GeoPointConverter.Instance)
            : base.FindMapping(mappingInfo);
}

public class Program
{
    public static void Main()
    {
        using (var context = new BloggingContext())
        {
            context.Database.EnsureDeleted();
            context.Database.EnsureCreated();

            context.Add(new House { Location = new GeoPoint(-122.34877, 47.6233355) });
            context.Add(new House { Location = new GeoPoint(-122.3308366, 47.5978429) });

            context.SaveChanges();
        }

        using (var context = new BloggingContext())
        {
            var distances = context
                .Set<House>().Select(e => e.Location.Distance(new GeoPoint(-121.3308366, 46.5978429)))
                .ToList();
        }
    }
}

this is too risky to try to fix for 2.1
Did you mean 2.2?

@ajcvickers Since 2.2 is out I tried your workaround. Unfortunately, I get the following exception:
Unhandled Exception: Microsoft.EntityFrameworkCore.DbUpdateException: An error occurred while updating the entries. See the inner exception for details. ---> System.Data.SqlClient.SqlException: A .NET Framework error occurred during execution of user-defined routine or aggregate "geography":
System.FormatException: One of the identified items was in an invalid format.
System.FormatException:
.
The statement has been terminated.
   at System.Data.SqlClient.SqlConnection.OnError(SqlException exception, Boolean breakConnection, Action`1 wrapCloseInAction)
   at System.Data.SqlClient.TdsParser.ThrowExceptionAndWarning(TdsParserStateObject stateObj, Boolean callerHasConnectionLock, Boolean asyncClose)
   at System.Data.SqlClient.TdsParser.TryRun(RunBehavior runBehavior, SqlCommand cmdHandler, SqlDataReader dataStream, BulkCopySimpleResultSet bulkCopyHandler, TdsParserStateObject stateObj, Boolean& dataReady)
   at System.Data.SqlClient.SqlDataReader.TryConsumeMetaData()
   at System.Data.SqlClient.SqlDataReader.get_MetaData()
   at System.Data.SqlClient.SqlCommand.FinishExecuteReader(SqlDataReader ds, RunBehavior runBehavior, String resetOptionsString)
   at System.Data.SqlClient.SqlCommand.RunExecuteReaderTds(CommandBehavior cmdBehavior, RunBehavior runBehavior, Boolean returnStream, Boolean async, Int32 timeout, Task& task, Boolean asyncWrite, SqlDataReader ds)
   at System.Data.SqlClient.SqlCommand.ExecuteReader(CommandBehavior behavior)
   at Microsoft.EntityFrameworkCore.Storage.Internal.RelationalCommand.Execute(IRelationalConnection connection, DbCommandMethod executeMethod, IReadOnlyDictionary`2 parameterValues)
   at Microsoft.EntityFrameworkCore.Storage.Internal.RelationalCommand.ExecuteReader(IRelationalConnection connection, IReadOnlyDictionary`2 parameterValues)
   at Microsoft.EntityFrameworkCore.Update.ReaderModificationCommandBatch.Execute(IRelationalConnection connection)
   --- End of inner exception stack trace ---
   at Microsoft.EntityFrameworkCore.Update.ReaderModificationCommandBatch.Execute(IRelationalConnection connection)
   at Microsoft.EntityFrameworkCore.Update.Internal.BatchExecutor.Execute(DbContext _, ValueTuple`2 parameters)
   at Microsoft.EntityFrameworkCore.SqlServer.Storage.Internal.SqlServerExecutionStrategy.Execute[TState,TResult](TState state, Func`3 operation, Func`3 verifySucceeded)
   at Microsoft.EntityFrameworkCore.Update.Internal.BatchExecutor.Execute(IEnumerable`1 commandBatches, IRelationalConnection connection)
   at Microsoft.EntityFrameworkCore.ChangeTracking.Internal.StateManager.SaveChanges(IReadOnlyList`1 entriesToSave)
   at Microsoft.EntityFrameworkCore.ChangeTracking.Internal.StateManager.SaveChanges(Boolean acceptAllChangesOnSuccess)
   at Microsoft.EntityFrameworkCore.DbContext.SaveChanges(Boolean acceptAllChangesOnSuccess)
   at TestSQLSpatialQueries.Program.Main() in C:\Users\andsav\Code\Projects\Temp\TestSQLSpatialQueries\TestSQLSpatialQueries\Program.cs:line 107

@ajcvickers You mean even the workaround?

@smitpatel - I originally did have the ability to add parameters.  I can't find the version that was part of that PR.  I best I have been able to find is an older version when the code was still in the main EFCore assembly and
You can see that here.
https://github.com/pmiddleton/EntityFramework/blob/udfSlim2/src/EFCore/Metadata/Internal/DbFunctionParameter.cs
I think the version that got converted to be in EFCore.Relational might be lost.
Is there something I can help you with?

That should be fine. Basically, for this issue, we need ability to configure parameter's column type on server side. Something like, modelBuilder.HasDbFunction(...).HasParameter(...).HasColumnType("geography") so that when translating we can use appropriate type mapping.

@smitpatel @ajcvickers, @divega
I just pulled latest and it appears the build system has changed a bit for 3.0 and I can't get the build script to work.
When I try to run the build script I just get a "build failure" error with no reason why.  I tried adding the -v flag but that just spit out the command line that failed.  KoreBuild is installing the 3.0.100-preview-010010 sdk to a .dotnet directory local to the project (not in userprofile as it use to).
I also tried building in VS and get the error "Unable to locate the .NET Core SDK. Check that it is installed and that the version specified in global.json (if any) matches the installed version."
I've also tried making a fresh clone but that has the same issue.
Is there something else that I need to setup, or a way to get a better error message to know what is failing?

@pmiddleton Try running git clean -xdf before running build.cmd.
Afterwards you can open the solution using startvs.cmd EFCore.Runtime.sln

@andriysavin -  Still no go on the build script.  However opening EFCore.Runtime.sln did get me a different error.
Version 3.0.100-preview-010010 of the .NET Core SDK requires at least version 16.0.0 of MSBuild. The current available version of MSBuild is <IP_ADDRESS>4.
The version of msbuild.dll included in the 3.0 framework is 16.  However it seems VS isn't wanting to load that version up.
Maybe everything is just too frozen still from the polar vortex :)

Ok I got it.  This was strange - I was missing %USERPROFILE%/.dotnet/NuGetFallbackFolder  Once I added that empty folder the build script started working and then VS would build after that.

VS 2019 is preferred, but VS 2017 should still work for a little while.

A very little while :)

@smitpatel, @ajcvickers  -
modelBuilder.HasDbFunction(...).HasParameter(...).HasColumnType("geography")
For this example we are assuming there is already a mapping setup from the method parameter type to geography correct?
We just want to do a lookup into IRelationalTypeMappingSource based on the column type mapping specified correct?

API shape looks correct. @ajcvickers can tell if we want ability to define custom value converter for parameter.

Also configure nullability propagation facet. i.e. function is null if parameter value is null.

@smitpatel  - I don't follow what the purpose of that feature.  Are you thinking that if a parameter passed in at runtime can be detected as null that we can remove the function call?

@pmiddleton - @maumar has optimization in SQL as if
FUNCTION(Arg1, Arg2) IS NULL => Arg1 IS NULL OR Arg2 IS NULL
In order to do that, parameter needs to say that.

@smitpatel
I was working on this yesterday and am getting close to done.  I originally had an attribute to set the parameter values.  Do we still want this?  I'm thinking that since this is an advanced scenario the fluent api is enough.  Thoughts?

@divega can answer that.

Hi folks. I'm wondering if this was finally fixed in preview 7? Because I tried this sample project (after fixing NTS API breaking changes in latest version) without that ReplacementTypeMappingSource and I'm still getting this error:

System.InvalidOperationException: 'The parameter 'x' for the DbFunction 'GeoExtensions.Distance' has an invalid type 'GeoPoint'. Ensure the parameter type can be mapped by the current provider.'

This is with preview 7. BTW, the workaround still doesn't work as well, returning me some SQL Server-side error on saving changes:

Microsoft.Data.SqlClient.SqlException (0x80131904): A .NET Framework error occurred during execution of user-defined routine or aggregate "geography":
System.FormatException: One of the identified items was in an invalid format.
System.FormatException:
.
The statement has been terminated.

@smitpatel any comments on https://github.com/aspnet/EntityFrameworkCore/issues/13752#issuecomment-514377214 is this something that should be addressed by the change or does it require additional calls, e.g. to configure the store type.

HasDbFunction(..).HasParameter("name").HasStoreType("geography or whatever") and it will work.

@smitpatel Wow, that function mapping works (doesn't fail during configuration), thanks! I have one usability question, though.
As far as I remember, EF implies that GeoPoint has store type of geography by default. So I'm wondering why do I have to configure store type for each function parameter while they are of GeoPoint type and store type could be inferred automatically? Not that I'm too lazy to specify store type, but I have to reference function parameters by name, and this is what makes me uncomfortable. This is my code:
modelBuilder.HasDbFunction(
                typeof(GeoExtensions).GetMethod(nameof(GeoExtensions.Distance)),
                builder =>
                {
                    builder.HasName("STDistance");
                    builder.HasParameter("x").HasStoreType("geography");
                    builder.HasParameter("y").HasStoreType("geography");
                });

Now, the bad part. When I execute the following simple query (from the sample code above)
var distances = context
                    .Houses.Select(e => e.Location.Distance(new GeoPoint(-121.3308366, 46.5978429)))
                    .ToList();

on a DB without any rows I get this error:

System.InvalidOperationException: 'Null TypeMapping in Sql Tree'

The place where it happens:

The stack trace:

at Microsoft.EntityFrameworkCore.Relational.Query.Pipeline.RelationalSqlTranslatingExpressionVisitor.SqlTypeMappingVerifyingExpressionVisitor.VisitExtension(Expression node)
at System.Linq.Expressions.Expression.Accept(ExpressionVisitor visitor)
at System.Linq.Expressions.ExpressionVisitor.Visit(Expression node)
at Microsoft.EntityFrameworkCore.Relational.Query.Pipeline.SqlExpressions.SqlFunctionExpression.VisitChildren(ExpressionVisitor visitor)
at System.Linq.Expressions.ExpressionVisitor.VisitExtension(Expression node)
at Microsoft.EntityFrameworkCore.Relational.Query.Pipeline.RelationalSqlTranslatingExpressionVisitor.SqlTypeMappingVerifyingExpressionVisitor.VisitExtension(Expression node)
at System.Linq.Expressions.Expression.Accept(ExpressionVisitor visitor)
at System.Linq.Expressions.ExpressionVisitor.Visit(Expression node)
at Microsoft.EntityFrameworkCore.Relational.Query.Pipeline.RelationalSqlTranslatingExpressionVisitor.Translate(Expression expression)
at Microsoft.EntityFrameworkCore.Relational.Query.Pipeline.RelationalProjectionBindingExpressionVisitor.Visit(Expression expression)
at Microsoft.EntityFrameworkCore.Relational.Query.Pipeline.RelationalProjectionBindingExpressionVisitor.Translate(SelectExpression selectExpression, Expression expression)
at Microsoft.EntityFrameworkCore.Relational.Query.Pipeline.RelationalQueryableMethodTranslatingExpressionVisitor.TranslateSelect(ShapedQueryExpression source, LambdaExpression selector)
at Microsoft.EntityFrameworkCore.Query.Pipeline.QueryableMethodTranslatingExpressionVisitor.VisitMethodCall(MethodCallExpression methodCallExpression)
at Microsoft.EntityFrameworkCore.Relational.Query.Pipeline.RelationalQueryableMethodTranslatingExpressionVisitor.VisitMethodCall(MethodCallExpression methodCallExpression)
at System.Linq.Expressions.MethodCallExpression.Accept(ExpressionVisitor visitor)
at System.Linq.Expressions.ExpressionVisitor.Visit(Expression node)
at Microsoft.EntityFrameworkCore.Query.Pipeline.QueryCompilationContext.CreateQueryExecutor[TResult](Expression query)
at Microsoft.EntityFrameworkCore.Storage.Database.CompileQuery[TResult](Expression query, Boolean async)
at Microsoft.EntityFrameworkCore.Query.Internal.QueryCompiler.CompileQueryCore[TResult](IDatabase database, Expression query, IModel model, Boolean async)
at Microsoft.EntityFrameworkCore.Query.Internal.QueryCompiler.<>c__DisplayClass9_01.<Execute>b__0()    at Microsoft.EntityFrameworkCore.Query.Internal.CompiledQueryCache.GetOrAddQueryCore[TFunc](Object cacheKey, Func1 compiler)
at Microsoft.EntityFrameworkCore.Query.Internal.CompiledQueryCache.GetOrAddQuery[TResult](Object cacheKey, Func1 compiler)    at Microsoft.EntityFrameworkCore.Query.Internal.QueryCompiler.Execute[TResult](Expression query)    at Microsoft.EntityFrameworkCore.Query.Internal.EntityQueryProvider.Execute[TResult](Expression expression)    at Microsoft.EntityFrameworkCore.Query.Internal.EntityQueryable1.GetEnumerator()
at System.Collections.Generic.List1..ctor(IEnumerable1 collection)
at System.Linq.Enumerable.ToList[TSource](IEnumerable`1 source)

@smitpatel Wow, that function mapping works (doesn't fail during configuration), thanks! I have one usability question, though.
First of all I'm curious how does EF know how to convert my GeoPoint type to NTS Point to then convert to geoprgahy when we call that function? We're not setting up any conversion for the parameters. But if it's able to do so, then why do I have to configure store type for each function parameter while EF already knows that GeoPoint=>Point=>geography and store type could be inferred automatically? Not that I'm too lazy to specify store type, but I have to reference function parameters by name, and this is what makes me uncomfortable. This is my code:
modelBuilder.HasDbFunction(
                typeof(GeoExtensions).GetMethod(nameof(GeoExtensions.Distance)),
                builder =>
                {
                    builder.HasName("STDistance");
                    builder.HasParameter("x").HasStoreType("geography");
                    builder.HasParameter("y").HasStoreType("geography");
                });

Now, the bad part. When I execute the following simple query (from the sample code above)
var distances = context
                    .Houses.Select(e => e.Location.Distance(new GeoPoint(-121.3308366, 46.5978429)))
                    .ToList();

on a DB without any rows I get this error:

System.InvalidOperationException: 'Null TypeMapping in Sql Tree'

I feel that this is related to the GeopPoint=>Point=>geography conversion I mentioned earlier.
The place where it happens:

The stack trace:
   at Microsoft.EntityFrameworkCore.Relational.Query.Pipeline.RelationalSqlTranslatingExpressionVisitor.SqlTypeMappingVerifyingExpressionVisitor.VisitExtension(Expression node)
   at System.Linq.Expressions.Expression.Accept(ExpressionVisitor visitor)
   at System.Linq.Expressions.ExpressionVisitor.Visit(Expression node)
   at Microsoft.EntityFrameworkCore.Relational.Query.Pipeline.SqlExpressions.SqlFunctionExpression.VisitChildren(ExpressionVisitor visitor)
   at System.Linq.Expressions.ExpressionVisitor.VisitExtension(Expression node)
   at Microsoft.EntityFrameworkCore.Relational.Query.Pipeline.RelationalSqlTranslatingExpressionVisitor.SqlTypeMappingVerifyingExpressionVisitor.VisitExtension(Expression node)
   at System.Linq.Expressions.Expression.Accept(ExpressionVisitor visitor)
   at System.Linq.Expressions.ExpressionVisitor.Visit(Expression node)
   at Microsoft.EntityFrameworkCore.Relational.Query.Pipeline.RelationalSqlTranslatingExpressionVisitor.Translate(Expression expression)
   at Microsoft.EntityFrameworkCore.Relational.Query.Pipeline.RelationalProjectionBindingExpressionVisitor.Visit(Expression expression)
   at Microsoft.EntityFrameworkCore.Relational.Query.Pipeline.RelationalProjectionBindingExpressionVisitor.Translate(SelectExpression selectExpression, Expression expression)
   at Microsoft.EntityFrameworkCore.Relational.Query.Pipeline.RelationalQueryableMethodTranslatingExpressionVisitor.TranslateSelect(ShapedQueryExpression source, LambdaExpression selector)
   at Microsoft.EntityFrameworkCore.Query.Pipeline.QueryableMethodTranslatingExpressionVisitor.VisitMethodCall(MethodCallExpression methodCallExpression)
   at Microsoft.EntityFrameworkCore.Relational.Query.Pipeline.RelationalQueryableMethodTranslatingExpressionVisitor.VisitMethodCall(MethodCallExpression methodCallExpression)
   at System.Linq.Expressions.MethodCallExpression.Accept(ExpressionVisitor visitor)
   at System.Linq.Expressions.ExpressionVisitor.Visit(Expression node)
   at Microsoft.EntityFrameworkCore.Query.Pipeline.QueryCompilationContext.CreateQueryExecutor[TResult](Expression query)
   at Microsoft.EntityFrameworkCore.Storage.Database.CompileQuery[TResult](Expression query, Boolean async)
   at Microsoft.EntityFrameworkCore.Query.Internal.QueryCompiler.CompileQueryCore[TResult](IDatabase database, Expression query, IModel model, Boolean async)
   at Microsoft.EntityFrameworkCore.Query.Internal.QueryCompiler.<>c__DisplayClass9_0`1.<Execute>b__0()
   at Microsoft.EntityFrameworkCore.Query.Internal.CompiledQueryCache.GetOrAddQueryCore[TFunc](Object cacheKey, Func`1 compiler)
   at Microsoft.EntityFrameworkCore.Query.Internal.CompiledQueryCache.GetOrAddQuery[TResult](Object cacheKey, Func`1 compiler)
   at Microsoft.EntityFrameworkCore.Query.Internal.QueryCompiler.Execute[TResult](Expression query)
   at Microsoft.EntityFrameworkCore.Query.Internal.EntityQueryProvider.Execute[TResult](Expression expression)
   at Microsoft.EntityFrameworkCore.Query.Internal.EntityQueryable`1.GetEnumerator()
   at System.Collections.Generic.List`1..ctor(IEnumerable`1 collection)
   at System.Linq.Enumerable.ToList[TSource](IEnumerable`1 source)

The thing is EF Core does not know how to do the conversion.
The original issue was, we did not know how to map parameter of any type unless mapped by provider. There are 2 ways to resolve it fully.

Add global level additional type mapping to typemapper used by provider. See #10784
Allow all facets of type mapping (including valueconverter) to be configured on any functions parameter. Essentially mapping the parameter same way mapping any property.

As you noted, 2nd option is not great experience compared to the first.
Now, EF Core needs to know everything about how to map given parameter's ClrType if it has to send constant/parameter to server for such function. As an escape hatch or to unblock certain users, if you are using your DbFunction with one of your column as parameter (and never passing constant/parameter), then your column already has been already converted and stored on server side and we don't need any additional info while querying. So we added API to configure store type, which you specify when you are going to use a column only and you tell us, you will find the column to be this particular database type on server side. This would pass validation check and we will happily pass your column in the DbFunction and run query for you. Though since it does not work for parameter/constant, you hit the second issue.

@smitpatel Thanks for explaining! I think that both ways you mentioned would be helpful depending on situation. Any hope to see at least one of them implemented in 3.0?

Regarding the second error I was mentioning, SQL Server-side error on saving changes or on a query (with using the workaround):

Microsoft.Data.SqlClient.SqlException (0x80131904): A .NET Framework error occurred during execution of user-defined routine or aggregate "geography":
System.FormatException: One of the identified items was in an invalid format.
System.FormatException:
.
The statement has been terminated.

While this looks to be a candidate for new issue, I'll describe what I found out since we have context here (the sample code). So the sample code queries with filtering by location distance, and for point new GeoPoint(-121.3308366, 46.5978429) the resulting query looks like this:
DECLARE @__myLocation_0 geography = 0xE6100000010C7854466D2C555EC0AAD2BB1D864C4740;

SELECT [h].[Location].STDistance(@__myLocation_0)
FROM [Houses] AS [h]

Executing this query directly against the database gives the same error. Actually, it's enough to execute just the declare statement. As far as I understand, the @__myLocation_0 parameter is sent in the SQL-Server binary format (not in WKT Binary format). So I can try to guess that this format is not directly convertible to geography, but I don't believe that such a severe bug wasn't noticed so far. Any ideas?

The referenced issue (#10784) has actually been expanded to include any type being used in property or anywhere else.
The later has been partially implemented. And we are not planning to expand it further for 3.0
HasName configures name of the function as UDF. Hence it will try to add dbo.STDistance. If you are mapping function to built-in function, then you would need to use HasTranslation API to construct SqlFunctionExpression

Thank you once more :)
BTW, regarding configuring method parameters, to avoid referencing parameter names a good alternative could be using an approach similar to what different test mock frameworks do, e.g. with Moq:
mock.Setup(foo => foo.Add(It.Is<int>(i => i % 2 == 0))).Returns(true);

E.g:
modelBuilder.HasDbFunction(() => GeoExtensions.Distance(
     It.Is<GeoPoint>().HasStoreType("geography"),
     It.Is<GeoPoint>().HasStoreType("geography")));

or
modelBuilder.HasDbFunction(() => GeoExtensions.Distance(
     It.Is<GeoPoint>(b => b.HasStoreType("geography")),
     It.Is<GeoPoint>(b => b.HasStoreType("geography"))));


Update ismobilejs

mobilejs has a vulnerability on versions <0.5.0.
https://app.snyk.io/vuln/SNYK-JS-ISMOBILEJS-72624
Can we move this forward? I realize this pull request is incomplete. Just want to see if there are any objections first.

Seems like I don't have permission to merge code in. Would one of the project maintainers take over this request?

Thank you very much @bigtimebuddy !


Unable to pair Apple TV HDs

I have an assortment of Apple TV models and was able to immediately pair the 4K. However, I have been unable to pair the HDs (model A1625). Most of the time, the output looks like this (running with -l -d all=sdebug):
pick an AppleTV or type "exit" to leave pairing mode

[IP1]      => [ID]@Apple TV 4K._raop._tcp.local
[IP2]      =>  [ID]@Apple TV HD2._raop._tcp.local
[IP3]      =>  [ID]@Apple TV HD1._raop._tcp.local

IP address: [IP3]
[17:02:12.098712] http_parse_simple:961 sock: 15, received Date: Sat, 29 Jul 2023 21:02:12 GMT
[17:02:12.099253] http_parse_simple:961 sock: 15, received Content-Length: 0
[17:02:12.099882] http_parse_simple:961 sock: 15, received Server: AirTunes/695.5.1
enter PIN code displayed on AppleTV: [PIN]
step1 ... verifying pin
please wait 5 seconds...

and then it loops back to "pick an AppleTV…"
Occasionally (maybe 1 out of 10 times?) it will say step2 ... verifying M1 but even then it never comes back with a credential string.
Please let me know what I can do to help figure out what's happening. Thank you as always!

 Can you share the ID of one of your ATV, or at least tell me is there is anything much different compared to the 4K one?

I've found and older ATV A1625 which had the pairing issue as well. Version 0.1.3 fixes it for these, let me know how it works for you


[BUG] Account following infinite of the same account

Please make sure you've tried the latest version of Fritter first, as the F-Droid version is sometimes out of date!

Describe the bug
A twitter account is seemingly following infinite of the same account
To Reproduce
Steps to reproduce the behavior:

Go to https://twitter.com/SenseClient on fritter
Tap on following
See error

Screenshots

Device
Redmi Note 9 pro, Android 11, Fritter 2.15.1

The fix for this will go out in the next version (v2.16.0)!


Build Error

I am using Xcode 6.4 (6E35b). And I use AudioKit as dynamic framework according to the installation steps and screencasts.
Please help me. Thanks a lot.

We'd like to help you but you haven't given us much information to go on.  There's nothing in the screenshot that helps either.  Provide more information and we'll look again.

This does look like a problem with the compiler itself though (clang).

But there is no more information. What should I do? This issue occurs after I run the playground's install.sh and run.sh.   The problem is with The dynamic code injection, not necessarily AudioKit, but necessary for the playgrounds.  Sometimes, and I am not sure when or how, it seems to mess up Xcode badly, and reinstalling Xcode is the only option.  Other times, it works perfectly.

Yeah unfortunately you should try to reinstall Xcode from the App Store from scratch and see if it helps.

OK. I will reinstall Xcode and try again later. Thanks for help.


Small typo fix

According to their website, it's listed first 50,000 participants can earn a T-shirt. Source: https://hacktoberfest.digitalocean.com/resources/participation

@chandrikadeb7 plz check I open this pr way back #215 :-(


Speaker audio recording not working

Steps to reproduce:
0. Install RecordRtc chrome extension

Launch youtube video
Start recording Screen + Speakers
Wait 10 seconds
Stop recording

Expected behavior: resulting video file contains audio
Actual behavior: resulting video file contains no audio
OS: Ubuntu 20.10
Browser: Chromium Version 87.0.4280.88 (Official Build) snap (64-bit)
Extension version: 9.0

Apparently same issue on my side, I will try manually first.
A useful use-case would be as well camera + screen + mic + desktop sound in the extension.


Deprecate python-based FMCS code.

There has been a C++ replacement for quite a while.

giant +1 on this - we've been confused by FMCS and rdFMCS far too many times :)


[CWS] Use containerutils.ContainerID in user and group resolver

What does this PR do?
Switch to containerutils.ContainerID type instead of string for user/group resolver.
Motivation
This allows not mixing cgroup and container ids.
Describe how to test/QA your changes
Possible Drawbacks / Trade-offs
Additional Notes

/merge


Introspection dashboard should capture display name of services in mono repos

In the screenshot above there are two separate services in the "order-store" mono repo. It would be great is the display name in bedrock.yaml for these services was actually seen in the service column.

[ ] prototype and implement a command spk project get-display-name which gets the display name of a service from its folder name, and repository name can stay as is

A first pass idea on a way to accomplish this

The SRC to ACR/ACR to HLD pipeline is based in the application repo codebase. Regardless of mono-repo or not the bedrock.yaml file contains the metadata needed.
During the SRC to ACR/ACR to HLD pipeline we clone the application repo and can read the bedrock.yaml and map the directory that the Dockerfile is in to a displayName in the bedrock.yaml (we can build this logic into spk, e.g. spk project get-display-name?)
We can then extend spk deployment create to accept an optional parameter for displayName and pass through the bedrock.yaml value.
The HLD to Manifest pipeline would link to the SRC to ACR/ACR to HLD via the existing "latest commit" method

At this point we would have additional metadata in the Azure Storage table and can give the user to the option to show displayName of a service or the repository name
@samiyaakhtar

@andrebriggs i support the idea of building this logic into spk, to have a command that extracts the display name. However I'm thinking if we need a new field for this, perhaps we can reuse the ring/env column in storage for this? In that case we can pass in this ring name when spk deployment create is called for --env. Thoughts? Particularly, is there a reason to still keep the ring name we show in this?

@samiyaakhtar that could work if it is consistent for single ring (default) scenarios. For instance below is a screenshot from a version of the dashboard from early Feb. I'm assuming your HLD only has the master (default) ring. Where is DEV coming from?

@andrebriggs yes this is a dashboard that's not really making the use of rings, so it has only one default environment (DEV). This was set up a long time ago before spk configured rings, so it's not a good reference for this.

Depending on https://github.com/microsoft/bedrock/issues/1117 for changes to bedrock.yaml schema.

@samiyaakhtar is this still a small task?

@andrebriggs changed it to medium. The work for this is complete but will PR after 1117 is merged, since I worked off of that.


526 mybabb get ogimage tag to work

Testing with Lorem Picsum url.

Yes, Indeed this configuration works, need to change the inputs and find out why the url for the images only work on an external url,   It needs the https corrected for the website,  Look into the inspect to find it maybe.   Leave this configuration until modified,   Now to check about adding meta with    Front-Matter in Markdown Pages.


Service virtualbox-tools keeps restarting.

RancherOS Version: (ros os version)
v1.5.3
Where are you running RancherOS? (docker-machine, AWS, GCE, baremetal, etc.)
Baremetal
Just enabled and uped virtualbox-tools.

Did not realize this is guest addition, not host modules.


How to improve recognition speed on CPU or increase concurrency

How to improve recognition speed on CPU or increase concurrency？
Whether it's Model Large2 or Model Large3, recognition is very slow. I run it on the CPU, which has 16 cores
My CPU usage is only 12% and memory usage is only 18% in the entire running state. May I ask if there is a way to increase speed

It will use multiple threads, but the maximum concurrency is the one that the underlying library will use (for both encoder and decoders). They are doing performance improvements all the time so you might check v1.7.0 for updated version. However, there is nothing we can do in this wrapper to improve the efficiency of CPU inference.


fix: get current year

Right now the default year will be 2022 since it is hard coded to it. It would be nice to change the default year according to the year right now since most people will use current year for creating new license.
I added chrono crate for getting current year. I also changed function name get_current_year to get_year since it will not return current_year which is a variable I added.

Hi! Sorry I missed this PR.
Thank you for the contribution, happy to merge this, is there any reason you closed it?

I assumed that there were some particular reason you chose 2022 as default.
Anyway, I will reopen it.

I'll make a release in 2-3 days when I find free time.

@nexxeln bump to make a release for this and subsequent versions when you get time!


Repeating sections fixes

Adds a default [] for repeating sections fields, instead of having it return nil.

This is ready to be merged and released as 1.2.8, yeah?

@kariannemah It is indeed. Merging and releasing now.


:memo: docs: Add getting started docs for CONTRIBUTING.md

Description
Add CONTRIBUTING.md to root
Testing
Explain the quality checks that have been done on the code changes
Additional Information

[ ] I read the contributing docs (if this is your first contribution)

Your ENS/address:

Current dependencies on/for this PR:

main

PR #596

PR #597

PR #599
PR #598

PR #601   👈

This comment was auto-generated by Graphite.

Current dependencies on/for this PR:

main

PR #596

PR #597

PR #599
PR #598

PR #601   👈

This comment was auto-generated by Graphite.


feat(default-AP-seeding): Add default date for AP on seeding in case it doesn't exists

What?
TAP-261 TX dates should read from manifest during seeding, create new date otherwise.
Why?
To have the current date as default in case there is no date provided on the manifest.
Screenshots (optional)

No longer relevant after cancelling TXs seeding on chain


Wiring guide

Hi @philpownall,
Thanks for your great work! I'm planning on using it for an old roomba and I was wondering if you had any schematics or simple wiring guide to help get started?
I have looked at the other/previous repos and they have some sort of wiring guide, but I was not sure if you had to change anything to make the new components work.
Looking forward to your reply!

Hi Bruno,
No particular wiring guide - I used the diagrams in:
https://community.home-assistant.io/t/add-wifi-to-an-older-roomba/23282/19
Basically, you take TX from the Roomba to the level shifter to change it to 3V, and from there to whatever pin you use for RX on the ESP32.  Be sure to match the pin number in the ESP32 code.
I used a Level converter (rather than a single transistor), and an ESP32 on an Expansion board, so that I had both 3V and 5V supplies for the level shifter.  I used Standard 2.54mm Pitch jumper wires to connect the DIN socket to the level shifter and from there to the ESP32.
I really enjoy driving the Roomba around using the Home Assistant app on my phone, I hope you do to!

Great! Thanks a lot for your help! I can't wait to make it work 😄


[Documentation] More ways to deploy poktrolld

Objective
Document how to deploy and use poktrolld in different environments.
Origin Document
Currently we only provide an example of how to deploy in poktroll-docker-compose-example repo, but we also need to add documentation how to run in other environments.
Goals

Document how to run fullnode/supplier/appgate without docker (systemd?)
Document helm charts (v1 helm charts are a good example - they were documented) and provide a simple example on how to push to k8s cluster

General deliverables

[ ] Comments: Add/update TODOs and comments alongside the source code so it is easier to follow.
[ ] Testing: Add new tests (unit and/or E2E) to the test suite.
[ ] Makefile: Add new targets to the Makefile to make the new functionality easier to use.
[ ] Documentation: Update architectural or development READMEs; use mermaid diagrams where appropriate.

Creator: @okdas

cc @Olshansk

@Olshansk we can separate k8s helm chart docs into the separate ticket

@Olshansk we can separate k8s helm chart docs into the separate ticket

Let's do it and mark them as deliverables in that ticket.

We must cover https://docs.cosmos.network/main/build/tooling/cosmovisor as this is the sidecar process that is used with x/upgrade to upgrade the binary on particular heights.

Moving this to the next iteration because I recently realized I want to also add Cosmovisor to the docker-compose setup. It should be quick and easy, but will only become usable after the next release.
After that — I'll create a separate issue to deploy off-chain actors on fly.io as this is a great platform for our use case.

@okdas Curious why fly.io over render.com?
Only asking because I know the founding team in the latter.

@Olshansk fly.io offers an entirely different approach to how the service is deployed across regions.
TLDR is their public network IP addresses are anycast - basically what GCP MCI offers but natively (meaning not a half-baked solution, but something that fly.io is built on from the beginning).
From what I understand, render.com is basically a PaaS.

@okdas This is really great to know!
I think fly.io might be a good one click deployment partner in the future then :)

Too general. When needed, it'll come back up.


Publishing quality attack flow diagrams

Related to https://github.com/center-for-threat-informed-defense/attack-flow/issues/40, how does the community use the attack flow JSON files to publish quality diagrams, such as this one?
Are you able to share what tooling was used [to generate this]?
Edit (opening question is still valid): Found the answer. They were manually(?) generated drawio diagrams (e.g. https://github.com/center-for-threat-informed-defense/attack-flow/blob/main/data/action-object-tesla.drawio).

For reference, here's what the attack-flow-example looks like via the current graphviz script:
cd ~/Downloads
curl https://raw.githubusercontent.com/center-for-threat-informed-defense/attack-flow/main/schema/attack-flow-example.json -O -L
cd ~/Downloads/attack-flow/src
python3 -m attack_flow.scripts.graphviz ~/Downloads/attack-flow-example.json ~/Downloads/attack-flow-example.dot
dot ~/Downloads/attack-flow-example.dot -Tpng -O

So here's the "Agent Tesla" sample output from the original graphviz.py, included in the 1.0.0 release

Then after I changed the Graphviz.py (in post #40) to use the Asset State for the label, I get this output, which is arguably more usable...

Finally, I see no value in the "http://flow-1" Oval at the top, so I manually edit the DOT file and comment out all of the flows from that oval.

Here's my current version of graphviz.py
graphviz.py.txt


molotov: init at 4.2.2

Motivation for this change
Package online tv application molotov which lets French users to access a range of TV channels
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/223

integrated your variations, last week end I tried to only extract the binary from the appimage, but was stuck with a SIGTRAP error (I think due to chrome-sandbox requiring suid, but I am not sure).
I added me and @apeyroux  as maintainer also if you are ok ?

integrated your variations, last week end I tried to only extract the binary from the appimage, but was stuck with a SIGTRAP error (I think due to chrome-sandbox requiring suid, but I am not sure).

You can try with https://github.com/apeyroux/molotov.nix
I didn't have any problem during the execution (authentication and watching TV)

I added me and @apeyroux  as maintainer also if you are ok ?

Yes

integrated your variations, last week end I tried to only extract the binary from the appimage, but was stuck with a SIGTRAP error (I think due to chrome-sandbox requiring suid, but I am not sure).

You can try with https://github.com/apeyroux/molotov.nix

Well using wrapType2 it is also ok on my side, it doesn't work if I try to extract just the app from the appimage + buildFHSEnv machinery

@ofborg build molotov

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/202


Rubocop

I've created a basic .rubocop.yml file, automatically run it, then manually completed some of it's recommendations. Each are three separate commits.
We can use this across other projects, once we confirm that the standard file is inline with our style standards.

Ooops. I commented on the individual commits, not this PR.

I wish it moved "#{variable}" to "#{ variable }" but maybe that's something we can add?

Either I'm missing something, or this isn't an option in Rubocop. See https://gist.github.com/glasnt/ee9aff3d62ddeb306194
/me grabs snorkel and goes code diving.

So rubocop PR#1836 introduces string interpolation detection and fixing. As soon as rubocop is bumped past 0.31, it'll be too simple to run it over this code base, with the .rubocop.yml file in this PR, and fix all the code.
I'll create a new branch with the yml file and it's autofixing once a) rubocop is bumped, b) index-page branch things are fixed (current major pending change)

Rubocop v0.32 is now a thing. This should be fixable robotically

$ gem list rubocop

*** LOCAL GEMS ***

rubocop (0.25.0)
sk@diffie:~/bp/playground/assemblage/instance-shutdown-scheduling (master) $ gem update rubocop
Updating installed gems
Updating rubocop
Fetching: parser-<IP_ADDRESS>.gem (100%)
Fetching: powerpack-0.1.1.gem (100%)
Fetching: astrolabe-1.3.0.gem (100%)
Fetching: rubocop-0.32.0.gem (100%)
Successfully installed parser-<IP_ADDRESS>
Successfully installed powerpack-0.1.1
Successfully installed astrolabe-1.3.0
Successfully installed rubocop-0.32.0
Gems updated: parser, powerpack, astrolabe, rubocop
sk@diffie:~/bp/playground/assemblage/instance-shutdown-scheduling (master) $ rbenv rehash
sk@diffie:~/bp/playground/assemblage/instance-shutdown-scheduling (master) $ rubocop --version
0.32.0

:laughing:
Let the games begin :+1:

Seen #32 since this branch is well out of date now.


Linux distribution not detected on Alpine Docker image

It seems none of the methods used to find the Linux OS/distribution works in an Alpine Docker image.
This issue prevents Rez installation (or at least the rez-bind --quickstart command tu run).
I was trying to add a rez-test call in a GitLab pipeline job, and using a container seemed like a good starting point.
After some research I found that OS specific data was registered in a /etc/os-release file:
https://www.freedesktop.org/software/systemd/man/os-release.html
I managed to make it work by parsing this file in case the lsb_release command fails, but I was wondering if adding another resolution method was the way to go.

I wonder if we should replace finding the distribution code on Linux with distro instead of trying to find it ourselves. It handles a lot of distributions, it's pretty much self contained and we don't need to maintain that code ourselves.

Great find, that looks promising. It'd be ideal to offload this code from
rez.
A
On Mon, Aug 20, 2018 at 5:04 PM, Sebastian Kral<EMAIL_ADDRESS>wrote:

I wonder if we should replace finding the distribution code on Linux with
distro https://github.com/nir0s/distro instead of trying to find it
ourselves. It handles a lot of distributions, it's pretty much self
contained and we don't need to maintain that code ourselves.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/nerdvegas/rez/issues/517#issuecomment-414503593, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABjqSlaDuQJavftVhLweXC4IfUBzAz9Pks5uS06LgaJpZM4VNVqp
.


[feature request] edgejump

edgejump would be really nice to have aswell as a jumpbug feature and edgebug but I think thoose ones are harder to add

add auto\on key\after jump edgejump


Failed without error message

The following test code fails without an error:
I have the same issue. Here is my test code.
<?php

require_once __DIR__.'/../vendor/autoload.php';

use mikehaertl\wkhtmlto\Pdf;

// You can pass a filename, a HTML string or an URL to the constructor
$pdf = new Pdf('Hello pdf');

// On some systems you may have to set the binary path.
 $pdf->binary = 'C:\Program Files\wkhtmltopdf\bin\wkhtmltopdf.exe';

if (!$pdf->saveAs('user/page.pdf')) {
    echo $pdf->getError();
}
?>

I have the "write" access to the "/user" directory and I can write a test file to it in the same script.
I'm on Windows 8.1 32 bit
PHP 5.5.12
Apache 2.4.9
What could be the reason?

Your problem here seems to be related to the Windows OS. I'm sorry, that I can't really help much with this: I don't use Windows. You could try to set the 'useExec' => true option in the commandOptions. This will use a different method to execute wkhtmltopdf. For more options, check: https://github.com/mikehaertl/php-shellcommand
If you can locate the problem and supply a fix, I'm happy to include it.

Does wkhtmltopdf have write access to that directory? When using wkhtmltopdf from the command line and trying to save the PDF it appeared to not exist - but it was saving to Uses/MyUser/AppData/Roaming/.... Perhaps it failed silently because it succeeded in writing to there instead?

Also notice, that during PDF generation a temporary file is written to disk (see https://github.com/mikehaertl/php-tmpfile). By default this will use the return value of sys_get_temp_dir. So make sure you also have write access there.

I solved this problem uninstalling wkHtmlToPdf MSVC 2013 and using the version MinGW-w64 of the wkHtmlToPdf. I recommend to edit the README.md to include this information for Windows users. Btw, congratulations for the project!

I had same issue. Switch 'useExec' => true helped me to track down bad URL for CSS file.

I was able to make this work on Windows by setting the full path of the binary
$pdf = new Pdf('http://www.google.com');   $pdf->setOptions(['binary' => 'C:\wk\bin\wkhtmltopdf.exe' ]);


clairvoyant: 3.1.2 -> 3.1.3

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/clairvoyant/versions.
meta.description for clairvoyant is: Ask questions and get psychic answers
meta.homepage for clairvoyant is: https://github.com/cassidyjames/clairvoyant
meta.changelog for clairvoyant is: https://github.com/cassidyjames/clairvoyant/releases/tag/3.1.3
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.1.3 with grep in /nix/store/ahf0jqd4xhn2jg5b4szqj1lvd02w1qrd-clairvoyant-3.1.3
found 3.1.3 in filename of file in /nix/store/ahf0jqd4xhn2jg5b4szqj1lvd02w1qrd-clairvoyant-3.1.3

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
clairvoyant

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/ahf0jqd4xhn2jg5b4szqj1lvd02w1qrd-clairvoyant-3.1.3 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A clairvoyant https://github.com/r-ryantm/nixpkgs/archive/543c6ce4f8d0553f312dfa186de1e2f03e05bf28.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/543c6ce4f8d0553f312dfa186de1e2f03e05bf28#clairvoyant

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/ahf0jqd4xhn2jg5b4szqj1lvd02w1qrd-clairvoyant-3.1.3
ls -la /nix/store/ahf0jqd4xhn2jg5b4szqj1lvd02w1qrd-clairvoyant-3.1.3/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

clairvoyant

Maintainer pings
cc @michaelgrahamevans for testing.

I would like to make changes to the nix file, adding wrapGAppsHook4 to fix the icon not showing in the about window, adding an option to hide the Flatpak banner and switching to the finalAttrs pattern. Do I make these changes separately in a new pull request or do I close this one and do the package update in the new pull request as well?

Since it's related to the update, it might make more sense to close this PR and make a new one


Port improvements from Anatine

This ports over hidePromotedTweets, and CSS changes for links and the profile avatar image.
https://github.com/sindresorhus/anatine/commit/ceb42dcaa3a748f4623f603bfbbce88fa3b2f084
https://github.com/sindresorhus/anatine/commit/8da0bfd3e2d838a77a874412b28d78ef7b871fb3
https://github.com/sindresorhus/anatine/commit/a5d328170e795cd10ac498774b060d6ba98be249

@sindresorhus I broke hidePromotedTweets and waitFor out into separate files.
The first one I did per your suggestion here: https://github.com/sindresorhus/refined-twitter/pull/40#issuecomment-225941782
The second one I did because waitFor was required to be accessible for hidePromotedTweets, and I still wait it accessible to future enhancements.
Whether or not this is all easier to work with, I don't know. Comments welcome.

@sindresorhus I made the same changes as requested in anatine

Great!


Official Mojang launcher

mclogs should have separate instructions for the official Mojang launcher. While it may be uncommon in the heavily modded Minecraft scene, a lot of vanilla players use this launcher.
From time to time, these players may also want to install a mod (e.g. SimpleVoiceChat for use on bukkit-ish servers). Often times players who have no experience in installing mods will install the modloader directly into the mojang launcher.
This way, they'll be using the Mojang launcher on their first experience with modded Minecraft and possibly on their first crash. In order to make it as easy as possible for users new to modded Minecraft to get help with their problems, mclogs should include separate instructions for people using the Mojang launcher.


HaritoraX 1.1 Usage

I saw the video of the HaritoraX wireless trackers working with this software and i was wondering if it was possible to use the HaritoraX1.1 with it too, the X1.1 connects through bluetooth similarly to the Wireless trackers

Thank you for creating issue!
But I only have Haritora X Wireless, So I can't test except for HaritoraX Wireless. If you can provide a packet capture about tracking data or the product itself, I may be able to support it.

Just want to clarify Im not the best when it comes to this, I used Wireshark to capture the bluetooth packets, if its not this please elaborate
HaritoraXBluetoothPacket.zip

Heyo, this is a couple months old but I am working on HaritoraX 1.1b/1.1/1.0 support via SlimeTora which you may be interested in (the program's backend is my NPM package, haritorax-interpreter, which I'm working on v2.0.0 for which will include support for those).
From what I have found, we do not need to listen directly to the Bluetooth packets, rather the COM port that it assigns, as it is a full Bluetooth device and not a BLE device like the HaritoraX Wireless (which uses services/characteristics and such).


Feature/update badge

@k2n Sorry for the back in forth :pray:
I gave up on lein-release due to some ssh related issues that I did not want to dig further at the moment.
Now a stable version is deployed on clojars


Setting Security Token does nto accoutn for expiration time

Below code does not set expiration of cached token which means at certain point, there will not be any space left for new token to be cached ?
public bool TryAdd(string securityToken, DateTime expiresOn)
{
if (this.cache.Get<string>(securityToken) == null)
{
this.cache.Set(securityToken, securityToken);
return true;
}
return false;
}

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

ID: 004c9f09-ba18-c5ff-43bc-b44d9e0ee809
Version Independent ID: b3c686a6-8ab3-8480-293b-a6f0100e06d0
Content: Authentication - Microsoft Threat Modeling Tool - Azure
Content Source: articles/security/azure-security-threat-modeling-tool-authentication.md
Service: security
GitHub Login: @jegeib
Microsoft Alias: jegeib

@KaishM Thanks for the feedback! We are currently investigating and will update you shortly.

@KaishM Thanks for reporting this. The sample code provided is just an example of implementation and needs to be customized based on your environment. You can implement cache clean up in the following ways.

Clean up the cache on a timer.
Schedule an asynchronous task to remove the cache entry on expiry.
Check the cache size when TryAdd is called and trigger an asynchronous cleanup job.

I agree that this should be included in the sample provided for better customer experience and I am assigning this to the author to consider adding this to the sample code provided in the doc.


GUACAMOLE-5: Provide most recent Credentials to updateUserContext()

The updateUserContext() function of AuthenticationProvider has existed for a while, but it was written under the assumption that AuthenticatedUser will always receive an updated set of credentials, thus the possibly-updated UserContext will be able to access this information.
This is false. The AuthenticatedUser will only be updated if updateAuthenticatedUser() does so, and relying on that is bad. There's not much for updateUserContext() to update with if it can't be guaranteed access to fresh information.
Updating UserContext may require peeking at the most recent HTTP request / credentials, so the most recent Credentials object should be passed to updateUserContext() as well.

So none of the auth providers are actually using the new Credentials as of yet?

Correct, though I will have to do so when adding support for multiple share keys within the same session.
In the case of screen sharing via the database auth backend, there will be at least two or three auth providers involved:

The auth provider which authenticated the user (not necessarily part of the database backend, but likely so)
The auth provider which stores the connection data and tracks the shared connections (either the PostgreSQL or MySQL auth providers).
The secondary auth provider used by the database backend to track strictly shared connections.

The secondary auth provider will only be the one that authenticates the user if they authenticated using a share key in the first place. If they authenticated via the database, or even via LDAP, then the AuthenticatedUser comes from elsewhere and cannot be guaranteed to have the new Credentials (ie: any new share keys).


SyncJob does not increment attempts counter

Laravel Version: 5.4
PHP Version: 7.1

Description:
SyncJob always returns 1 for attempts. Hence, it does not increment when release method is called.
Steps To Reproduce:
In any test class:
$job = new ExampleJob();

dispatch($job);
$job->attempts(); // returns 1

dispatch($job); // or $job->release()

$job->attempts(); // returns 1, but it it’s not correct

Why do I need it
In my job there is some logic that checks that job attempts === 1, then code x is run, if attempts > 1 it is required to run another code. You should get the point.
Tests are being executed using sync queue driver and the only solution now to make it work is to create class in tests that extends required Job and has public attempts property which in tests is overridden to desired value.
Also, release method should execute SyncJob again.
Conclusion
If this feature can be accepted I will make a PR. Waiting for a feedback.

You can easily simulate this with database jobs. In your tests, insert a raw job directly into the database with attempts = 2 or something - then process the job in your test.
I do this all the time to simulate different "states" of queues during tests.
I dont like the idea of sync jobs trying to re-run themselves. It's likely to lead to all sorts of unintended side effects in current code - I can just see it breaking things...

Either way - this is probably a better discussion for https://github.com/laravel/internals - open a ticket there for further discussion...

@laurencei If I use RabbitMQ for my queue, why do I need a database driver and migration itself?

You said it was for testing. For testing it is common to use mock/alternative drivers to simplify testing.
Anyway - as mentioned - lets move the discussion to  https://github.com/laravel/internals

Moved to https://github.com/laravel/internals/issues/754.


Android: Remove redundant pattern for matching build directories

build/ already matches all directories named 'build' in the repository,
regardless of level. Gradle can have more than two levels of project nesting.
However, /*/build/ matches only the 'build' directories of 2nd-level projects.
Thus, the first pattern is more appropriate than the second.

Thanks, but are you sure about that? I haven't checked, but I seem to recall that if there is a slash anywhere in the pattern, git starts anchoring it to the top of the project. It would be nice to double check this first.

Yes I'm sure. /build would match a file or directory at the root, while
build/ would match a directory at any level. /*/build/ would match a
directory at the second level while /**/build/ would match a directory at
any level, which is essentially the same as build/.

Pertinent excerpts from the gitignore manpage:

If the pattern ends with a slash, it is removed for the purpose of the following description, but it would only find a match with a directory. In other words, foo/ will match a directory foo and paths underneath it, but will not match a regular file or a symbolic link foo (this is consistent with the way how pathspec works in general in Git).

...

A leading slash matches the beginning of the pathname. For example, "/*.c" matches "cat-file.c" but not "mozilla-sha1/sha1.c".

If you're still in doubt, you could check it yourself. :)

@arcresu any updates on this? I've already quoted the pertinent excerpts from the gitignore manpage; all info needed is there. And as I said before, this is really easy to test. Just create a new Git repo, add a bunch of "build" directories on different levels, and observe how each pattern works.

Sorry for the delay - I haven't had much time to process PRs here lately. Yes, you're right about this rule. Thanks for confirming, I just couldn't quite remember how it worked. :tada:


Send a numeric direction attribute on a 'create' command creates an orphan location with no associated surface.

This was uncovered through a bug in ocmud-fe which has since been fixed.

This points to a need for more validation on the back-end.


Add option to select cell types in histograms

Similar to the Heatmap panel (Cell subtypes search bar option), in the Gene Expression histogram panel of Wormcells-viz, an option to select specific cell subtypes will be useful.

New menu looks good, but I noticed on the histograms that the "deselect all" button doesnt seem to be working:
http://cervino.caltech.edu:3000/ridge_line/WBGene00000149

I've added a 'refresh' button to avoid fetching data every time a cell is checked or unchecked. This also fixes the 'deselect all' button issue.

refresh button is awesome


MD - Ast - MD Roundtripping ?

Is it possible to parse markdown to an ast and then output that back to markdown ? And if so, can I expect it to be identical to the initial markdown text? I figure there might be a lossy process somewhere

@spand, the AST is loss-less with offsets into original source available for every part of a parsed element, not only the element itself.
The parser can be setup to include blank line nodes in the AST, which by default are not included. With this option the AST is complete with all elements in your source. Set Parser.BLANK_LINES_IN_AST to true to have blank line nodes in AST.
As for outputting identical markdown is concerned, it is up to you. I don't see why you would want identical as opposed to syntactically identical but cleaned up, as done by the flexmark-formatter module.
You can look at the flexmark-formatter module implementation for details on how it is used.
It does a lot more than just output the same markdown. It can re-arrange non-rendered elements: references, footnotes, abbreviations; renumber ordered items, sort list items, change format for different parser rules, etc.
It will give you a good start for idiosyncrasies of rendering AST to Markdown.

I intend to rewrite certain custom nodes while a user is in the process of writing his document so the lossless aspect is to minimize disruption of her writing process.
Is there anything else I should be aware of while altering the ast ?
Thanks for the quick response!

Since all strings in the AST are fragments of the original string there is a bit of overhead of modifying strings in the AST.
All strings are an instances of BasedSequence whose base string is the original full parse text.
An example of inserting an element into the AST with a string you can find in SyntheticLinkSample

@spand, you are probably better off using the AST for the user text but then building your replacement text the way formatter does it. That way you will not need to modify the AST and can generate whatever replacement text you want.
Unless you are using the AST afterwards for something other than generating the replacement markdown, you are better off with the second approach. If you need the AST with modified text then you are still better off building the "modified" markdown text (formatter style) and just re-parsing it to get the AST with modifications.


Add ability to include / exclude incidents based on package / modules / assembly

Add ability to include / exclude incidents based on which package they come from. The business logic of inclusion / exclusion will change from provider to provider.
In general, here are the four scenarios describing how it will work:
Scenario 1: Only Include Set
Include:
- org.apache.*
Exclude: null

Filter-in incidents in packages matching org.apache.* pattern.
Filter-out everything else.
Scenario 2: Only Exclude Set
Include: null
Exclude:
- org.apache.*

Filter-out incidents in packages matching org.apache.* pattern.
Filter-in everything else.
Scenario 3: Include / Exclude Set with non-exclusive packages
Case 1: Exclude is a subset of Include
Include:
- org.apache.*
- io.konveyor.*
Exclude:
- org.apache.example.*
- io.konveyor.demo.*

Filter-in incidents in packages matching org.apache.* OR io.konveyor.*.
Among the filtered-in packages, filter-out everything that further matches org.apache.example.* OR io.konveyor.demo.*.
Case 2: Include is a subset of Exclude
Include:
- org.apache.example.*
- io.konveyor.demo.*
Exclude:
- org.apache.*
- io.konveyor.*

Filter-in incidents in packages matching org.apache.example.* OR io.konveyor.demo.*.
Among the filtered-in packages, filter-out everything that further matches org.apache.* OR io.konveyor.*.

The result of this specific operation will be 0 incidents.

Scenario 4: Include / Exclude Set with exclusive packages
Include:
- org.apache.*
Exclude:
- io.konveyor.*

Filter-in incidents in packages matching org.apache.*.
Among the filtered-in packages, filter-out everything that further matches io.konveyor.*

Result of this operation will be equivalent to setting Exclude to null

I agree with the logic described, specifically, that if a user does do case 2 in scenario 3 and scenario 4 there will be no results.
@rromannissen @jmle can we get an agreement that this logic makes sense.
I would also suggest that we move to talking about the field as "scopes" or something similar to what @jortel mentioned.

I intentionally kept the language include / exclude here just to be clear about the logic. The scope thing I am addressing in the enhancement

Looks good, I think it is the most logical and intuitive way of implementing it

Looks great!!!

@pranavgaikwad can I work on this?

@Parthiba-Hazra apologies, most of our team was on vacation last week, we missed your ping here. This issue is already solved. I am closing it right now. Thank you for your interest! Please let us know if you want to take a look at any other issue.


Is this github actively curated for new contributions?

The issue title is the question.

Not sure we have a clear owner and process, but I would think the answer should be 'yes'.  I'll bring this up at the Khronos F2F meeting next week.

We're asking the Vulkan Ecosystem Forum to look at it and decide how to handle. Assigning to @nsubtil as EF chair.


[ClickAwayListener] Fix mounting behavior in Portals in React 17

We actually rely on the useEffect semantics i.e. the click handler should not be attached synchronously. Otherwise, we might handle the event that mounted the component. Unfortunately, react 17 has a bug where effects are flushed synchronously in portals so we have to find a userland workaround.
Benefits over #23313: Since we have many listeners we would need to repeat it whenever we add the event listeners. Touches might behave the same as clicks so we're fixing the same problem for touch events as well. There's an argument to be made to get rid of mountedRef/activatedRef entirely since that's no different from just removing the handler but the "mounted"-pattern helps here since we don't have to repeat this quirky logic everywhere. If the bug is fixed we should be able to get rid of the ref entirely. That IE11 comments is curious. React 17 removes the listener asynchronously anyway so I wonder if there's a realistic chance that we trigger a click-away between unmount and removal of the listener.
I was not able to write a test for it since the synchronous flush only happens on a user-click and not programmatic (element.click) or dispatched (element.dispatchevent) click.
https://github.com/facebook/react/issues/20074 tracks the React bug.
Closes #23215
Closes #23313
Test plan

[x] Added test fails in React 17 before fix: https://app.circleci.com/pipelines/github/mui-org/material-ui/25651/workflows/979393e5-e3fc-4b60-904a-c8c80ffcc262/jobs/191785
[x] Added test passes in React 17 after fix: https://app.circleci.com/pipelines/github/mui-org/material-ui/25654/workflows/046392bb-f599-4f9a-a8fb-46090bbb1106/jobs/191798
[x] Verify fix in codesandbox using this PR: https://codesandbox.io/s/material-ui-issue-forked-ptpxo?file=/src/Demo.js
[x] existing test pass in React 16 and 17

Should have a test for it now.

@eps1lon Great job!


Ambiguous infix expression

Importify is unable to process some modules when running importify cache due to ambiguous infix operators. This results in too many imports being removed from sources.
According to https://github.com/haskell-suite/haskell-src-exts/issues/72 we can provide fixities to haskell-src-exts. I believe operator fixities are not important in terms of importify. I have hardcoded fixities = Just [] and this helped to remove those warnings (see PR).

@kfigiela I agree. Problems with fixities really decreased usefulness of importify... I'm glad it can be solved in a such simple way!


fix a couple of strings

The UPnP change looks good, but I don't especially like the Wi-Fi change.
Wifi or wifi is used so much in Openwrt (e.g. "wifi up" command etc.) that using Wi-Fi looks a bit strange. I know that Wi-Fi is the "official" term defined by Wi-Fi Alliance, but in everyday use "wifi" is currently more common.

Maybe we could compromise and use WiFi as that also seems to an acceptable way of writing it. Although it looks like LuCI just uses the term Wireless everywhere else so we could switch to that instead.

I would maybe prefer "Wireless" as the item in the Network menu, as the settings page itself is.titled "Wireless Overview"
@jow-
Any comments on changing Wifi to WiFi, Wi-Fi or Wireless ???

Could you drop the Wi-Fi change from the pull request and leave just the UPnP changes?
I could then merge the UPnP part, which makes sense.