DB import fails on 'ASYNC_STATS_UPDATE_WAIT_AT_LOW_PRIORITY'

SqlPackage or DacFx Version:
.NET Framework (Windows-only) or .NET Core: .NET core
Environment (local platform and source/target platforms): local platform

Steps to Reproduce:

Have azure sql Db with the feature on ASYNC_STATS_UPDATE_WAIT_AT_LOW_PRIORITY
Export the DB using sql export into a bacpac
Locally import into on-prem/stand alone SQL

Expected behavior - if there are any SQL azure features not applicable for on-prem sqlpackage should handle and let the DB gets imported successfully
Actual behavior - Failing wit the following exception
The import fails with the below error -
Error output was: \r\n *** Error importing database:Could not import package. Error SQL72014: Core Microsoft SqlClient Data Provider: Msg 102, Level 15, State 1, Line 5 Incorrect syntax near 'ASYNC_STATS_UPDATE_WAIT_AT_LOW_PRIORITY'. Error SQL72045: Script execution error. The executed script: IF EXISTS (SELECT 1 FROM [master].[dbo].[sysdatabases] WHERE [name] = N'$(DatabaseName)') BEGIN ALTER DATABASE SCOPED CONFIGURATION SET ASYNC_STATS_UPDATE_WAIT_AT_LOW_PRIORITY = ON; ALTER DATABASE SCOPED CONFIGURATION SET GLOBAL_TEMPORARY_TABLE_AUTO_DROP = OFF; END
Suggested fix:
IF ( EXISTS  (select * from sys.database_scoped_configurations where name = 'ASYNC_STATS_UPDATE_WAIT_AT_LOW_PRIORITY'))
BEGIN

End

Did this occur in prior versions?  If not - which version(s) did it work in?
(DacFx/SqlPackage/SSMS/Azure Data Studio) - it fails with all the sqlpackage versions and it never worked

see ICM - 258531280 for more details

Notes for later:
Had a similar the problem for with OPTIMIZE_FOR_ADHOC_WORKKLOADS. This was fixed in https://sqlbuvsts01/Main/Data Tools/_versionControl/changeset/1379071, so those files are a starting point for investigation.
That was a one-off fix, but since this is a scenario that will be hit by all the new database scoped configurations that get added that aren't supported on previous versions of SQL Server, we need to have a different solution to handle all of these.


CSV Buffer Readlines

Can you please update the ReadLine function in CSV Buffer table to the following, because I have many issues right now with quotes is being ignored. For instance if I want to import a file with mail and the customers have written "xx<EMAIL_ADDRESS>i the mail field my import will fail.
This should solve the issue.
IF StreamReader.EndOfStream THEN
EXIT(FALSE);
REPEAT
String := StreamReader.ReadLine;
CurrentLineNo += 1;
CurrentIndex := 0;
REPEAT
IF String.Substring(CurrentIndex,1) = '"' THEN BEGIN
InQuotes := TRUE;
CurrentIndex := CurrentIndex + 1;
END ELSE
InQuotes := FALSE;
CurrentFieldNo += 1;
INIT;
"Line No." := CurrentLineNo;
"Field No." := CurrentFieldNo;
IF InQuotes THEN
  NextIndex := String.IndexOf('"',CurrentIndex)
ELSE
  NextIndex := String.IndexOf(Separator,CurrentIndex);

IF NextIndex = -1 THEN
  Length := String.Length - CurrentIndex
ELSE
  Length := NextIndex - CurrentIndex;
IF Length > 250 THEN
  Length := 250;

IF InQuotes THEN BEGIN
  Value := String.Substring(CurrentIndex+1,Length-2);
  NextIndex := NextIndex + 1;
END ELSE
  Value := String.Substring(CurrentIndex,Length);

CurrentIndex := NextIndex + 1;
INSERT;

UNTIL NextIndex = -1;
CurrentFieldNo := 0;
UNTIL StreamReader.EndOfStream OR (CurrentLineNo = NumberOfLines);
EXIT(TRUE);

We're going to fix this in the upcoming minor release. Thanks for reporting.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.

Availability update: We will publish a fix for this issue in the next update for releasee 17.

Here is an updated code.. maybe you can use it.
IF StreamReader.EndOfStream THEN
EXIT(FALSE);
REPEAT
String := StreamReader.ReadLine;
CurrentLineNo += 1;
CurrentIndex := 0;
REPEAT
//JMA-01:Start
IF String.Length > CurrentIndex THEN BEGIN
IF String.Substring(CurrentIndex,1) = '"' THEN BEGIN
InQuotes := TRUE;
CurrentIndex := CurrentIndex + 1;ReadLines(0);
END ELSE
InQuotes := FALSE;
END;
//JMA-01:End
CurrentFieldNo += 1;
INIT;
"Line No." := CurrentLineNo;
"Field No." := CurrentFieldNo;
//JMA-01:Start
//NextIndex := String.IndexOf(Separator,CurrentIndex);
IF InQuotes THEN
  NextIndex := String.IndexOf('"',CurrentIndex)
ELSE
  NextIndex := String.IndexOf(Separator,CurrentIndex);
//JMA-01:End

IF NextIndex = -1 THEN
  Length := String.Length - CurrentIndex
ELSE
  Length := NextIndex - CurrentIndex;
IF Length > 250 THEN
  Length := 250;

//JMA-01:Start
IF InQuotes THEN BEGIN
  //Value := String.Substring(CurrentIndex+1,Length-2);
  IF String.IndexOf(Separator,CurrentIndex) <> -1 THEN
    NextIndex := NextIndex + 1
  ELSE
    NextIndex := -1;
END;
//JMA-01:End

Value := String.Substring(CurrentIndex,Length);
CurrentIndex := NextIndex + 1;
INSERT;

UNTIL NextIndex = -1;
CurrentFieldNo := 0;
UNTIL StreamReader.EndOfStream OR (CurrentLineNo = NumberOfLines);
EXIT(TRUE);


update header with react training content

Working with @mjackson to update a few minor details on the React Router docs. 🎉

Going to pass on this for now. Thanks, @ladyleet :)


Handle spinup errors in blue/green deployments

As has been demonstrated, sometimes AWS cannot provide enough instances
for an ASG and this causes the spinup function to fail and throw an
exception. Now, we should catch any exceptions from the spinup command
and try to teardown the newly created testing ASG (and ELB if necessary)
so that the ASGs are not left in a bad state.
Added a number of unit tests to try to cover the different combinations
of already existing old ASGs and newly created ASGs to make sure that we
don't accidently destroy an ASG that is already in use can cause
downtime.

+1

👍


FC009 chef metadata update

adds latest versions to Appraisals
expunges Appraisals of version that no longer build
updates metadata of older versions for ruby 2.2.x
adds metadata for all the new versions
adds a README so next time the process should be easier to remember

For reference this was a subtask off of #344


OKTA_BROWSER_AUTH doesn't work with multiple roles

Describe the bug
I have two AWS accounts - one where I have one role I can assume and one where I have multiple roles I can assume. Using OKTA_BROWSER_AUTH=true, I get prompted for my credentials, then the following happens:

In the account with one role, i get the one role, my credentials get written to ~/.aws/credentials, and life is good.
In the account with multiple roles, I go to a second screen which prompts me to pick my role. I select a role and then end up in AWS in the browser. I quit the browser and nothing is written to ~/.aws/credentials.

Additional context
If, instead, I set OKTA_BROWSER_AUTH=false, I provide my password as I always have, then get a prompt in the commandline to select my role, and life is good.

@robkinyon-roivant I tried this with a single account and multiple roles and multiple accounts with multiple roles and was not able to reproduce it. In both cases I had a single Okta org with a single Amazon Web Services app set up with multiple AWS accounts.
That the embedded browser is proceeding to the AWS SAML sign-on page graphically suggests that the SAMLResponse can’t be found in the web page returned by the Amazon Web Services Okta app.
That web page should have a form with the SAMLResponse, RelayState, and JavaScript to automatically POST it to AWS’ SAML sign-on page.
The code is meant to intercept the web page document before the WebView renders it. It looks for the SAMLResponse and, if it finds it, it closes the WebView and proceeds with the console role selection flow.
Are you using the v1.0.3 release? There are some serious bugs with browser auth in v1.0.2.

Upgrading fixed the problem. Thanks!


whenever in new project deletes existing cron tasks

I have already operated cron tasks created by whenever.
However, existing cron tasks which whenever had created was deleted
when I updated[whenever -w] cron tasks in my new project.
Is this bug?
Or, I cannot use whenever in multiple projects on the same server?

you should use whenever --update-crontab

Great! thaks.


#370 Session timeout updated to 24hrs for international exchange

License update

Coverage remained the same at 95.138% when pulling 525f916f7a9f40f858fa45916d65ae871ee266f1 on release/2.12.0 into 52b5732c0704294ee162f11853612cb5100352b1 on master.


Add OhMyArtifacts

Codecov Report

Merging #21 (81e516d) into master (fd48f2b) will increase coverage by 0.03%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #21      +/-   ##
==========================================
+ Coverage   99.34%   99.37%   +0.03%
==========================================
  Files           5        6       +1
  Lines         153      161       +8
==========================================
+ Hits          152      160       +8
  Misses          1        1

Impacted Files
Coverage Δ

src/QuantumStateBase.jl
100.00% <100.00%> (ø)

src/wigner.jl
100.00% <100.00%> (ø)

src/wigner_util.jl
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


SchemaModel with nullable Config property fails to validate correctly

Describe the bug
I'm trying to validate a data frame that contains null values in an integer column. However, I'm encountering a SchemaError when validating the data, even after specifying nullable=True in the Config class

[x] I have checked that this issue has not already been reported.
[x] I have confirmed this bug exists on the latest version of pandas.
[ ] (optional) I have confirmed this bug exists on the master branch of pandas.

Note: Please read this guide detailing how to provide the necessary information for us to reproduce your bug.
Code Sample, a copy-pastable example
import numpy as np
import pandas as pd
import pandera as pa

from pandera import Check, Column, DataFrameSchema

df = pd.DataFrame({"column1": [5, 1, np.nan]})

null_schema = DataFrameSchema({
	"column1": Column(pa.Int, Check(lambda x: x > 0), nullable=True)
	})

print(null_schema.validate(df)) #succeeds

class InputDataModel(pa.SchemaModel):
	column1: pa.typing.Series[pa.typing.Int]

	class Config:
		nullable = True

InputDataModel.validate(df) #raises SchemaError

Expected behavior
InputDataModel should correctly validate the given data frame

hi @jstammers, currently there isn't a nullable option at the schema model Config (which maps onto the DataFrameSchema keyword options), but specifying additional constraints on Field should work:
class InputDataModel(pa.SchemaModel):
    column1: pa.typing.Series[pa.typing.Int] = pa.Field(nullable=True)

My mistake! I was following the docs here and assumed that because coerce was specified, other properties would map onto keyword arguments for Field.

no probs! closing this issue


Fixed Tty and Stdin save

Previous TTY/Stdin settings weren't kept if General/Volumes/Ports were modified and save - This fixes the issue by reading the Container data before the update.
Signed-off-by: TeckniX<EMAIL_ADDRESS>

LGTM


Consider standard and mocha-standard

http://standardjs.com/
your eslint style is almost exactly like standard, except spaces after the function keyword is enforced in standard (function () vs function()).

We already enforce spaces before anonymous function parens and don't require them for named functions.
I agree that my style is similar to standard, but I prefer to use ESLint.


[ENH] this new interface is trash

i refuse to use this new verson breaks all my macros if there is no revert change i will be modifying  it and reverting it myself

Duplicate of #890


Large loss socialisation in console before closing a position

Description
A user reported a bug where it gets info that they have a big loss socialisation, the number is huge as you can see they have not closed any positions nor have they have had anything near those values.

This is a Core bug which is bein addressed here
https://github.com/vegaprotocol/vega/pull/10573


[Fix] In Helm type of component, is InstallTimeout parameter optional or required?

This issue is already fixed in 1.3.


[Feature] Select templates without having modelled them

[x] Description created
[x] Reviewed by peer: @lmoesle
[ ] Prioritised

Description

As modeling I would like to be able to model a template directly when modeling elements to be able to use templates faster.

It should be possible to directly select a custom task (backed by an element template) instead of modelling a service task and then select the element template. e.g. If I want to model a send message task I would like to select a send message task that uses the send message element template.
Sub-Tasks
### Tasks
- [x] Test [extension](https://github.com/bpmn-io/bpmn-js-create-append-anything) for C7
- [ ] If changes need to be made, create a fork

Acceptance criteria

Append pattern can be used for modeling
Camunda 7 Element Templates work (Without Icons

The repository https://github.com/bpmn-io/bpmn-js-connectors-extension is now archived. We should check if bpmn-io moved development to a different repository. If not we should maintain our own fork.

The repository https://github.com/bpmn-io/bpmn-js-connectors-extension is now archived. We should check if bpmn-io moved development to a different repository. If not we should maintain our own fork.

I am not aware of any other repository. I think it is valid to maintain an own fork of this repo 🤔

It is not possible to create custom tasks in C7 with this extension. The reason is the different modules used (bpmn-js-element-templates).

C8 uses CloudElementTemplatesPropertiesProviderModule. This module uses ElementTemplates with a createElement()-Method. This method is used by the Append-Extension for creating custom tasks.
C7 uses the ElementTemplateProviderModule. Here the createElement()-Method is missing. Which leads to the following error:

@dominikhorn93 I'm not familiar with the development of bpmn-js plugins. But is it possible to extend the ElementTemplates from the ElementTemplateProviderModule with the createElement functionality? And would that be a better approach than forking the Append-Extension?


Refactor decrypt and encrypt NEP-2 account

Added ScryptParameters as a parameter instead of just using hardcoded values for N, R, and P.

👍

@luc10921 note that I changed how the default ScryptParameters are set because passing a mutable object to a function can lead to nasty bugs.
See this commit: f909e1f
And here's some background on the issue: https://docs.python-guide.org/writing/gotchas/#mutable-default-arguments


S3 storage doesn't store the contents of the files for Namespace Files

Issue description
https://share.descript.com/view/32Y9giWvidr
files stored on S3 are blank

Fixed by https://github.com/kestra-io/kestra/commit/9c49e5da3d3affb7ae752a38d750fafef2de8916


Support bridge errors in responses

Implement #8
Add decoding unit tests
Update example to show example error destructuring

Opened this into a feature branch as this has some large changes, and we might we want to keep refining things.
Further considerations

It might be a good idea to move errors into a separate exposed module

Some of the method definitions are a little long.

Task Hue.BridgeReferenceError (Result (List Hue.GenericError) (List Hue.LightDetails))
There might be a nice way to shorten this with type aliases

This is great! Thanks for putting in all the work. I think you came up with a solution that is both useful and elegant. I'd love to have your thoughts documented somewhere, e.g. at the end of the readme. For first time users, it would be really helpful to get a a brief walkthrough.
There's one thing which I've found really confusing:
|> (flip T.andThen) failResponseIfAuthError

Does (A (flip andThen) B) mean that B is executed first? I think this is a bit misleading. It's the only thing I'd like to see changed before merging this.
Last but not least I love your suggestion to pull out an errors module!
Thanks again.

Good idea adding some explanation in the readme. I think the sample could be improved too.
Ah yeah the flip is confusing. It returns a Task.andThen where the 2 argument orders are switched. So we can pass in the operation function and the task from the pipe can then be added to the end.
I'm not sure how to get around that. I first discovered that trick here https://github.com/NoRedInk/elm-style-guide#use-flip-andor-pipes-over-backticks.
I asked if there was a way to get around it in the elm slack channel and was told in a future version of Elm Task.andThen will be optimized better for pipes so you won't need the flip.
One alternative would be to do something like
taskAndThen = flip T.andThen
Then in pipes use
|> taskAndThen failResponseIfAuthError

I've had a look at the styleguide and couldn't find a reason for avoiding the back ticks. Too me the flip solution looks more complicated then using back ticks.
I'd either keep the back ticks or pull out the named taskAndThen function with a comment - what's your take on this?

Actually we could do this.
checkResponseForAuthError : T.Task BridgeReferenceError (LD.Response a) -> T.Task BridgeReferenceError (LD.Response a)
checkResponseForAuthError task =
    task `T.andThen` failResponseIfAuthError

 |> checkResponseForAuthError
That gives us the task from the pipe and then we can use back ticks.

I like that.

Cool, I made that refactor. Is it ok if the separate errors module and updated readme are in a new pull request?

Sure thing :)


Flutter web on download without file dialog freezes in chromium based browsers

Steps to reproduce

Open a flutter web app that offers downloading a file from a remote
Make sure your browser directly downloads to your download folder and does not open a file dialog
Trigger the download

Used versions:
flutter 3.22.2 on dart 3.4.3
chrome 127.0.6533.72 (but was also reproducable with 126 and 125)
Expected results
The download starts and you can continue to use the app while the download runs in the background.
Actual results
The webapp freezes for quite some time. The time seems not related to the time the download takes. The freeze can be unblocked by removing focus from the current browser window (e.g. click into dev tools) or resizing the view.
Without unblocking it might take a few minutes until the app unfreezes.
Code sample
https://dartpad.dev/?id=3d77b2fb211cfec9dca72766404aa480
Screenshots or Video
No response
Logs
No response
Flutter Doctor output
$ flutter doctor -v
[✓] Flutter (Channel stable, 3.22.2, on Ubuntu 22.04.4 LTS 6.5.0-1025-oem, locale en_US.UTF-8)
• Flutter version 3.22.2 on channel stable at ~/fvm/versions/3.22.2
• Upstream repository git@github.com:flutter/flutter.git
• Framework revision 761747bfc5 (8 weeks ago), 2024-06-05 22:15:13 +0200
• Engine revision edd8546116
• Dart version 3.4.3
• DevTools version 2.34.3
[✗] Android toolchain - develop for Android devices
✗ Unable to locate Android SDK.
Install Android Studio from: https://developer.android.com/studio/index.html
On first launch it will assist you in installing the Android SDK components.
(or visit https://flutter.dev/docs/get-started/install/linux#android-setup for detailed instructions).
If the Android SDK has been installed to a custom location, please use
flutter config --android-sdk to update to that location.
[✓] Chrome - develop for the web
• CHROME_EXECUTABLE = /snap/bin/chromium
[✗] Linux toolchain - develop for Linux desktop
✗ clang++ is required for Linux development.
It is likely available from your distribution (e.g.: apt install clang), or can be downloaded from https://releases.llvm.org/
✗ CMake is required for Linux development.
It is likely available from your distribution (e.g.: apt install cmake), or can be downloaded from https://cmake.org/download/
✗ ninja is required for Linux development.
It is likely available from your distribution (e.g.: apt install ninja-build), or can be downloaded from https://github.com/ninja-build/ninja/releases
✗ pkg-config is required for Linux development.
It is likely available from your distribution (e.g.: apt install pkg-config), or can be downloaded from https://www.freedesktop.org/wiki/Software/pkg-config/
[!] Android Studio (not installed)
• Android Studio not found; download from https://developer.android.com/studio/index.html
(or visit https://flutter.dev/docs/get-started/install/linux#android-setup for detailed instructions).
[✓] Connected device (2 available)
• Linux (desktop) • linux  • linux-x64      • Ubuntu 22.04.4 LTS 6.5.0-1025-oem
• Chrome (web)    • chrome • web-javascript • Chromium 126.0.6478.182 snap
[✓] Network resources
• All expected network resources are available.
! Doctor found issues in 3 categories.

This was initially reported in https://github.com/flutter/flutter/issues/149417 and has since been fixed by https://github.com/flutter/flutter/issues/150636. A cherrypick request has been filed in https://github.com/flutter/flutter/issues/152303, hopefully it gets added to the current stable.
Closing as this has already been fixed.
Thank you


Can not  open loader.bat successfully on win11

Hi,
When I updated the system to Win11（version number : 22H2）recently , I found the loader.bat on jatos_win_java  crashed and did't show the address as expected . I don't know  what causing this probrom. FirstIy, I tried to pause the loader.bat, but though the pause was succeed, it didn't show the address. Then, I downloaded the ducument again, but it didn't work. Jatos_win_java's version is Jan16. Could you help me ? Thanks a lot !!

Hi, please have a look at this post in the forum.
If you're using JATOS 3.8.1, please try it with Java 11.
And let us know if this solves your issue

Hi, please have a look at this post in the forum. If you're using JATOS 3.8.1, please try it with Java 11. And let us know if this solves your issue

Thanks! It works!


Is it possible to run individual test case/test class?

          Original issue by @kinwahlai - Imported from fastlane/scan#90

Is it possible to run individual test case/test class? like -only in xctool
Darren

now that xcodebuild supports the flag -only-testing:TestBundle/TestSuite/TestCase for test, is it more likely that this feature will be supported by scan in the near future?

@lucianomarisi Are you interested in implementing this in a PR? I think we'd be really happy to see a PR? I'm marking this open to contribution. Thanks!

@TKBurner unfortunately my ruby skills are not existent (barely know the syntax and have no idea on how unit testing works on ruby), so it's unlikely that I'll be able to contribute

@lucianomarisi No problem! I'll leave this open in case someone from the community wants to take it on. 👍


I run gradio for inference, it starts with an error, but no error code is given.

(real3dportrait) D:\Program\Real3DPortrait>python inference/app_real3dportrait.py
| WARN: checkpoints\240210_real3dportrait_orig\th1kh_512_audio2motion\base.yaml not exist.
| load 'model' from 'checkpoints/240210_real3dportrait_orig/audio2secc_vae/model_ckpt_steps_400000.ckpt', strict=True
| WARN: checkpoints\240210_real3dportrait_orig\secc2plane_torso_orig\secc_img2plane_orig.yaml not exist.
Error in sys.excepthook:
Original exception was:
(real3dportrait) D:\Program\Real3DPortrait>
The above is the information printed by the terminal, the environment is created normally, and the dependencies are all installed correctly.Here are the relevant dependency version numbers:pytorch3d   0.7.7+pt2.1.0cu121,  torch  2.1.0+cu121,  torchaudio 2.1.0+cu121,  torchvision    0.16.0+cu121

I got the same problem!


DeleteInfo.RecordInfo is not assigned when calling ConcurrentDeleter

The RecordInfo field of the DeleteInfo struct that is passed into ConcurrentDeleter is not being assigned before the call.
This causes the cache size tracking to be off. For example, in the samples, the class MemonlyCache.CacheFunctions needs the record info to be properly set:
public override bool ConcurrentDeleter(ref CacheKey key, ref CacheValue value, ref DeleteInfo deleteInfo)
        {
            sizeTracker.AddTrackedSize(-value.GetSize);
            // Record is marked invalid (failed to insert), dispose key as well
            if (deleteInfo.RecordInfo.Invalid)
                sizeTracker.AddTrackedSize(-key.GetSize);
            return true;
        }

This appears to be the root cause of the Netherite issues https://github.com/microsoft/durabletask-netherite/issues/132 and https://github.com/microsoft/durabletask-netherite/issues/160.

Thanks for the report, we should have a fix and new release tomorrow.


exclude windows or unix specific specs

this depends on merging https://github.com/elasticsearch/logstash/pull/2507 into master to access LogStash::Environment.windows?

LGTM this require to have the method
LogStash::Environment.windows? present in logstash, I only merged it in the wip/bundler-move branch.

you could cherry-pick that commit?

elasticsearch/logstash#2507 has been merged to 1.5 and master, so merging this


⚠ Bofrost.at has degraded performance

In 4ae0457, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1257 ms

Resolved: Bofrost.at performance has improved in 48b5046.


make 0.8 tag so versions are set to 0.8...

Done and pushed. We still have to figure out the best way to sync our two repositories :)
Out of curiosity, are you guys using this repo or the docker images we provide?

mostly using the repo, though we'll use the latest docker image for our docker builds.


propagate exporter logger to WSGIServer

Adding verbosity control on exporter server side.

Hi @ahuret, thanks a lot for the patch. Can you add a bit more details about what is your use case and why this change is needed? More than happy to merge but I'd like to understand it a bit more.
We use github as mirror for https://gerrit.wikimedia.org/r/admin/repos/operations/software/druid_exporter, so there are two options:

I commit your changes adding credits to the commit msg.
You register to Gerrit, checkout the repo and create a code review (should be as simple as git clone etc.. and git push origin HEAD:refs/for/master).

Let me know :)

Hi @elukey :)
For now exporter allow to decide if logs are debug or warn, but only at exporter level, and since exporter use external services (as WSGISERVER) logs verbosity for these external services is not handle. This change allows to use the same logger and then use the same verbosity for the entire application (exporter + external services).
With this change, I can disable debug mode and then don't have 'INFO' WSGISERVER logs.
Next step would be to allow user to decide which verbosity he wants (debug/info/warn/error/fatal) (by adding a verbosity/log-level argument to cli). But imo better to keep it for another iteration since it requires much more work.
Gonna push directly to Gerrit. Thx :)

https://gerrit.wikimedia.org/r/c/operations/software/druid_exporter/+/608607

Ahhh got it, for example the access logs from WSGI are not logged anymore if not using --debug (from what I can see from a quick test). It seems that adding the custom verbosity shouldn't be very hard, would you be interested in attempting a patch? Always nice when people from the community send patches, otherwise I'll try to work on it :)

Yep it shouldn't be much complicated :

Adding log-level argument to cli
Parse log-level (ensure it's valid)
Add a switch case to set level to logging library

I don't have time for contribution now but I'll be happy to review :)

closing it while it was merged on gerrit: commit 93eaecda5d27f9f105ab8b78eb0ab77e4903cb1e


Exported Driver Support Routines

Will PCI Controller Drivers are part of export drivers,like SCSI port drivers???

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

ID: a00526dc-0f04-4927-7e18-c07fd69ab9d5
Version Independent ID: d87a7326-2a07-1eff-64c4-afcd8446c791
Content: User mode and kernel mode
Content Source: windows-driver-docs-pr/gettingstarted/user-mode-and-kernel-mode.md
Product: windows-hardware
GitHub Login: @nabazan-microsoft
Microsoft Alias: windowsdriverdev

Hi @abhijeet085
Thanks for the question, but as of right now we do not have any plans to create a PCI export driver.
All the best.
Eliot


Refresh project

Add NonNull and Nullable annotations
Add CI
Publish via MavenPublisher
Bump dependencies
Move to AndroidX

@natario1 any plan to release the library with those change to mavencentral? See #51. Thanks!

I don't have time to do this, sorry!


how to train my own model?

Version
branch v2. latest
Description
if I write a new model (for APC task), how should I train and eval it?
sorry, badly, i dont have much time to read your source code. In examples-v2/aspect_polarity_classification, i can't find relevant code example. thanks.

另外，您论文中LSA、LSA-X分别对应code中的哪几个模型呢？


Add prettier/eslint config

Overview
Files are formatted with prettier and eslint, and I fixed all the complaints that eslint had.
Checking this PR
You'll probably want to look through the commit diffs individually, because the prettier formatting adds a lot of noise. You can check that the formatting seems reasonable, and that the manual fixes for the eslint warnings/errors were correct.

@calebeby Question for ya'. Should tests be passing if I run npm run test? They pass on master but not on this branch. 🤔

Yeah, they should be passing (at least they did on my computer and Travis CI)

Yeah, they should be passing (at least they did on my computer and Travis CI)

@calebeby Hmm...I do see the CI checks passing, am I doing something wrong? LOL
I ran a git pull on this branch, then ran npm run test. There are more errors, but here's an example:
92 passing (2s)
  4 pending
  15 failing

  1) drizzle should return data used to build the drizzle:
     TypeError: Cannot read property 'contents' of undefined
      at walkPages (dist/render/pages.js:64:20)
      at renderPages (dist/render/pages.js:80:10)
      at render (dist/render/index.js:28:23)
      at tryCatcher (node_modules/bluebird/js/release/util.js:16:23)
      at Promise._settlePromiseFromHandler (node_modules/bluebird/js/release/promise.js:512:31)
      at Promise._settlePromise (node_modules/bluebird/js/release/promise.js:569:18)
      at Promise._settlePromise0 (node_modules/bluebird/js/release/promise.js:614:10)
      at Promise._settlePromises (node_modules/bluebird/js/release/promise.js:693:18)
      at Async._drainQueue (node_modules/bluebird/js/release/async.js:133:16)
      at Async._drainQueues (node_modules/bluebird/js/release/async.js:143:10)
      at Immediate.Async.drainQueues (node_modules/bluebird/js/release/async.js:17:14)

🤔

Are you sure the git pull ran successfully? Yesterday I did some history-mangling (I separated some of the commits) and force-pushed.
Also did you try to run npm install on this branch?

@calebeby Something wonky was going on. I deleted the repo and recloned it. Tests pass now on the eslint branch. 😉

@tylersticka Wanted to make sure you got a look at this PR. I think the linting changes are looking fine. The only one that caught my attention was the removal of the space after function names.
// Space after function name
function foo (baz) {
  // ...
}

vs
// No space after function name
function foo(baz) {
  // ...
}

The project I'm on at the moment is set up to require a space after the function name so I got used to that more recently. But I'm okay either way. I just wanted to make sure you didn't see anything that you had a strong feeling against.

@gerardo-rodriguez This is not a deal-breaker for me, but I'll admit I'm definitely more of a fan of keeping a space in function declarations to subtly distinguish them from function calls. I've found this especially helpful in ES6 classes:
function example () { // New function

}

class Foobar {
  example () { // New method
    example(); // Calling the function
    this.example(); // Calling the method
  }
}

But again, not a deal-breaker for me.

@tylersticka Here is the discussion on prettier to support this: https://github.com/prettier/prettier/issues/3847
I expect that they eventually will add an option for this, and at that point we can switch the option on (when prettier was first released, it didn't have the option to switch between single/double quotes and semi/no semi)

We could configure eslint to do this, but the main problem with that is that prettier could no longer output code that conforms to the eslint config. All code would have to run through prettier, then eslint. Each time you ran it, prettier would remove the space, and then eslint would add it back. In order to avoid this kind of thing, the eslint config currently doesn't have any formatting/stylistic rules. We are only using eslint to catch probable bugs and bad practices. We use Prettier to handle all the formatting. But prettier doesn't support space after function.
https://github.com/prettier/prettier/issues/3845 You can give it an upvote to increase the likelihood that they will add an option for that.

I'm going to merge this, if we want to continue this discussion we can open an issue in the eslint config.


Add batch resources to UNDOCUMENTED_NAME_TYPE_MAP in cloudformation/parsing.py

Resolves #3127, at least partially.
However, as noted in #3127, other services could be added to the map.

Coverage decreased (-0.05%) to 94.356% when pulling 7792e8aef0e6036a8bef991fb1780b6667685d48 on richford:add-undocumented-name-types into<PHONE_NUMBER>c9e4afbe271ab398a4c9fa7ec9aacd on spulec:master.

Closing in favor of the recommended approach discussed in #3127


Polynomial multiplication is often inaccurate

Sometimes multiplying a polynomial p with another one, p_, may not yield the expected result. Here is a printout of a loop where a polynomial Pj is updated in every iteration. The new Pj does not always reflect the update:
2 -4.047588495693385e-21
2 -154.8971429 + x
2 6.269598936178143e-19
3 6.269598936178143e-19
3 -211.0 + x
3 -1.3228853755335882e-16
4 -1.3228853755335882e-16
4 -230.1282051 + x
4 3.044332370245841e-14
5 3.044332370245841e-14
5 -242.552795 + x
5 -7.384113253121036e-12 + 3.044332370245841e-14x
6 -7.384113253121036e-12 + 3.044332370245841e-14x
6 -257.96875 + x
6 1.904870465766067e-9 - 1.5237539414489605e-11x + 3.044332370245841e-14x^2

Here is the source code for this (c_ = some scalar value, x is a vector of data):
Pj = Poly([c_])

for i in not_j
    println(i, " ", Pj)

    Pj *= Poly([-x[i], 1])

    println(i, " ", Poly([-x[i], 1]))
    println(i, " ", Pj)
end

Can you point out which value isn't computed correctly? They all look correct to me at first glance.

This is due to the somewhat arbitrary truncation in the constructor in 0.0.5 version.  It is already fixed in master:  the constructor no longer truncates the result of multiplication and there is an explicit truncate function which you can use if you need to.  See #38 for the discussion.  If you want to use the master branch run
julia> Pkg.checkout("Polynomials","master")

and restart julia.

This branch just got tagged and published. Closing now.


Fix the token refresh in the Python client (+ associated flaky tests)

We sometimes see the "deploy stage" task fail in CI, because it takes over an hour to send a bag through the staging storage service, e.g. https://buildkite.com/wellcomecollection/storage-service-deploy-stage/builds/46
The exact error we see is:
Traceback (most recent call last):
  | File ".buildkite/scripts/send_and_wait_for_test_bag.py", line 59, in <module>
  | current_ingest = client.get_ingest_from_location(ingest_url)
  | File "/var/lib/buildkite-agent/.local/lib/python3.7/site-packages/wellcome_storage_service/__init__.py", line 54, in get_ingest_from_location
  | status_code, body = self._http_get(ingest_url)
  | File "/var/lib/buildkite-agent/.local/lib/python3.7/site-packages/wellcome_storage_service/__init__.py", line 174, in wrapper
  | return f(self, *args, **kwargs)
  | File "/var/lib/buildkite-agent/.local/lib/python3.7/site-packages/wellcome_storage_service/__init__.py", line 205, in _http_get
  | return super(RequestsOAuthStorageServiceClient, self)._http_get(url)
  | File "/var/lib/buildkite-agent/.local/lib/python3.7/site-packages/wellcome_storage_service/__init__.py", line 153, in _http_get
  | resp = self.sess.get(url)
  | File "/var/lib/buildkite-agent/.local/lib/python3.7/site-packages/requests/sessions.py", line 542, in get
  | return self.request('GET', url, **kwargs)
  | File "/var/lib/buildkite-agent/.local/lib/python3.7/site-packages/requests_oauthlib/oauth2_session.py", line 478, in request
  | url, http_method=method, body=data, headers=headers
  | File "/var/lib/buildkite-agent/.local/lib/python3.7/site-packages/oauthlib/oauth2/rfc6749/clients/base.py", line 198, in add_token
  | raise TokenExpiredError()
  | oauthlib.oauth2.rfc6749.errors.TokenExpiredError: (token_expired)

The Python client should be refreshing the token when it expires, but apparently it isn't.
Fixing this would make these tests more reliable.

Nooooo still broken: https://buildkite.com/wellcomecollection/storage-service-deploy-stage/builds/82#0183403b-b754-4ae7-a85b-dc7444f4c846

Yeah so it turns out this was nothing to do with token refresh, and everything to do with a line in the tests that said 'fail this after an hour". Should be fixed now. 🤦♀


Feature Request: HTTP Interceptor / TransferState Support [Angular Universal]

The does not currently appear to be any hook for transferring data between the server and client using a package like HttpStateTransfer. Are there any plans to use HttpInterceptors? Thanks to Angular 5 TransferState module, we can accomplish this, but this could be better abstract in the AngularFire library itself.

@patrickmichalina We plan on doing this when we implement SSR support. Follow #1073 until the official proposal is listed.


EasyCLA UI Redesign Part 3

Summary
Some more changes to the UI redesign task related to #361
##1. In the WebForm which is generated after clicking the "Request CLA Manager Access" button do the following.
1.1 Put Company at the top and make it searchable (i.e. you type a few characters and company name should appear in a drop down list that the user can click and pick). User can only select one company name.
1.2 Once the company name is selected in top most row, projects should show up in a dropdown list where we can also type in. Allow user to choose single project. If the user wants to choose multiple projects they can submit form multiple times. See the flow chart

1.3 Detect if this project and company already has a primary CLA Manager on the form itself and provide a message to the user that "There is already an initial CLA Manager for this project from  company. Your request has been sent to initial CLA Manager to add you as a CLA Manager". Then the user can close the web form modal and go back to landing page.
Send an email to primary CLA manager with all the information provided in the form. Email format should be as follows:
Hello LFCLA Manager for ,
The following user is requesting access to your organization {company name}.
{user email id}
{user's lfid}
Project : {project(s)}
Reason for request: {reason field in the form}
Please navigate to the Corporate Console using the link below, where you can approve this user's request.
corporate.lfcla.com

Linux Foundation CLA System
1.4 If the project doesn't have a primary cla manager. Display a message  "Your request has been sent to Linux Foundation CLA Support to add you as an Initial CLA Manager".
Then the user can close the web form modal and go back to landing page.
Send an email to<EMAIL_ADDRESS>with all the information provided in the form. Email format should be as follows:
Hello Lfcla Admin,
The following user is requesting access to your organization {company name}.
{user email id}
{user's lfid}
Project : {project(s)}
Reason for request: {reason field in the form}
Please navigate to the Corporate Console using the link below, where you can approve this user's request.

corporate.lfcla.com

Linux Foundation CLA System

1.5 When filling out the form if the Company name is not found, the user can type in their company name. Display a message  "Your request has been sent to Linux Foundation CLA Support to add your company to CLA Database".
An email will be sent to<EMAIL_ADDRESS>to add the company.
Hello Lfcla Admin,
Please add the following company to CLA database
{ Company Name }

Linux Foundation CLA System

Acceptance Criteria
Demo of the pages that satisfied all the scenarios under https://confluence.linuxfoundation.org/display/PROD/EasyCLA+Corporate+CLA+UI+Redesign
References
(Optional) Provide any code or specification references that would be helpful for the developer implementing this feature.

During review with the business owners (Steve Winslow and Mike Dolan) it was decided that these UI changes were not needed. Filing a new issue for the UI changes and closing this issue.


Bug: Fix cors issue with spotify sdk

Potential fix: move calls server side

I can't see this issue on my end so someone else is gonna have to work on it

Potentially try running in chrome see here


Injecting Additional Responses

Background
Long story short I built a static site generator on top of iso-relay/router. The generator generates static html pages with embedded iso-relay preload data, which works wonderfully. It also generates a client.js and a "static JSON-file API" from the relay responses when generating the html pages. client.js uses the preload data on a given html page to initialize iso-relay, which also works wonderfully. client.js then implements a custom Network Layer that turns Relay requests into requests to the previously mentioned hashed JSON files.
client.js
import IsomorphicRelay from 'isomorphic-relay';
import Relay from 'react-relay';
import IsomorphicRouter from 'isomorphic-relay-router';
import {
  Router,
  match,
  browserHistory
} from 'react-router';
import React from 'react';
import ReactDOM, { render } from 'react-dom';
import md5 from 'md5';

import routes from './load-routes';

var myNetworkLayer = {
  sendMutation(mutationRequest) {
    // ...
  },
  sendQueries(queryRequests) {
    console.log('queryRequests', queryRequests);
    return Promise.all(queryRequests.map(
      queryRequest => fetch(`/api/${md5(window.location.pathname)}.json`)
                        .then(response => response.json())
                        .then(result => {
                          console.log('result',result);
                          if(result.errors) {
                                      queryRequest.reject(new Error('Failed to fetch JSON file'));
                                    } else {
                                      IsomorphicRelay.injectPreparedData(environment, result);
//                                      queryRequest.resolve(result[0]);
                                    }
                                  })
    ));
  },
  supports(...options) {
    // ...
  },
};

// Set up Relay Environment
const environment = new Relay.Environment();
environment.injectNetworkLayer(myNetworkLayer);

// Inject Data from DOM (and network?)
const data = JSON.parse(document.getElementById('preloadedData').textContent);
IsomorphicRelay.injectPreparedData(environment, data);

// Client render (optional):
if (typeof document !== 'undefined') {
  console.log('using client render');
  const outlet = document.getElementById('react-mount');

  // use the same routes object as on the server
  match({routes, history: browserHistory}, (error, redirectLocation, renderProps) => {
    IsomorphicRouter.prepareInitialRender(environment, renderProps).then(props => {
      console.log('matched and props', props)
      ReactDOM.render(<Router {...props} />, outlet);
    });
  });
}

The compiled static site showing the usage of client.js is on GitHub, as well as live at http://www.christopherbiscardi.com/.
Problem
To reproduce the issue, navigate to http://www.christopherbiscardi.com/, wait for client.js to log using client render, then click on the posts item in the navigation twice. The first click with successfully make the request for the JSON file and navigate the URL to /posts/, but will re-render the homepage instead of the Posts page. The second click will render the Posts page correctly without making an additional request (since Relay already has the appropriate data).
gif showing the reproduction:

I'm not sure why this is happening and have tried multiple combinations of the following two functions in sendQueries to varying success.
   IsomorphicRelay.injectPreparedData(environment, result);
// queryRequest.resolve(result[0]);

The question I have then is how should I inject additional responses to the Relay store from the Network Layer so that iso-relay and iso-router behave appropriately and the proper page is rendered.

The problem with your network layer is that Relay waits for query requests to resolve before rendering new route, and your network layer never resolves them (you commented queryRequest.resolve)...
And generally, abusing the network layer like this feels very wrong. Instead I recommend you to use the onEnter hook to fetch and inject data for the new route, this way Relay won't even send queries because the data will be already there.

The problem with your network layer...

Indeed. The current version was the "least error inducing". Resolving with data was causing different issues.
@denvned

And generally, abusing the network layer like this feels very wrong. Instead I recommend you to use the onEnter hook to fetch and inject data for the new route, this way Relay won't even send queries because the data will be already there.

This sounds like a much better approach, except that iso-/relay-router seems to send the request in the middleware, which fires before onEnter.
react-router-relay exports the middleware and expects the user to apply it
...
  <Router
    history={history}
    render={applyRouterMiddleware(useRelay)}
    environment={Relay.Store}
  >
...

; whereas iso-relay-router applies the middleware itself and exports the result. I'm going to experiment in this area (specifically writing middleware that runs before iso-relay's) to see if it yields a solution.

Thanks for pointing out that the network-layer was a poor solution. I was too heads-down to see it.

which fires before onEnter

No, I believe you are mistaken, it should fire after onEnter. Just don't forget to use callback, the third argument of onEnter, call it after the data is fetched and injected.

You are absolutely correct, it was my own mistake.
I ended up using onChange with a no-path Route that wraps all of the application's routes, which works quite nicely as long as one is comfortable blocking rendering instead of rendering loading states.


Head-check on moving web.config back to wwwroot

@JunTaoLuo @Tracher @moozzyk
[Opened outside of the discussion on the announcement so that I can link/find it easier.]
Coming off of Publish for IIS changes to web.config location (Announcements/#173) and concern over accidental renaming or moving of web.config that would leak sensitive files, may I get a head-check for those wishing to put web.config back into wwwroot?

Use .UseContentRoot("../") on WebHostBuilder
Set processPath="..\my_app.exe" (self-contained) or arguments="..\my_app.dll" (portable)
Set stdoutLogFile="..\logs\stdout" (presuming logs is in the content root)
Move web.config to wwwroot
Point the IIS website physical path to wwwroot

Seems ok in testing. Is that correct or would that break something?

I second this. I too will be moving it back to be similar to RC1.

Just to clarify one more point. I recall the conversations with @moozzyk on paths. I'm not asking about dev/test, where debugging might not like that .UseContentRoot("../") ... One could easily program around that. I'm asking about on the IIS server in production.

@GuardRex did you get a chance to test it on Azure Websites?

@muratg No. I only ran an IIS Local test. It would be good to know if this will break in Azure Apps.

Use .UseContentRoot("../") Seems non portable. It'll only ever work if always hosting on IIS, self host will fail.

Very interesting ... It works self-host if you double-click the .exe, but it fails if you run the .exe through the command line???? What's up with that?

Spoke too soon. I was in the wrong dir. I wasn't in the publish directory. Kestrel runs it from the command line or double-clicking the .exe ...

It fails in VS Code debug ...
App Started: 5/6/2016 4:20:39 AM
dbug: Microsoft.AspNetCore.Hosting.Internal.WebHost[3]
Hosting starting
Exception thrown: 'System.AggregateException' in Microsoft.AspNetCore.Server.Kestrel.dll

... but if you simply change the ../ to Directory.GetCurrentDirectory(), of course, it runs in debug.
@Bartmax might be testing this config in Azure Apps soon.

@GuardRex - I actually think it could be possible to contain all the changes to web.config. Instead of changing the content root in the application you could try setting it as an environment variable (ASPNETCORE_CONTENTROOTKEY) in web.config. You would need to ensure that configuration is reading environment variables.

@moozzyk There would be no effect on the app for debugging that way :+1:, so that sounds very good. That would also be easy to roll into my GuardRex.AspNetCore.Server.IISIntegration.Tools <--- malware :smile:. I'll test this approach on Saturday. Thx. /cc @Aequoris

@moozzyk @Aequoris
Env Var Approach
I didn't fully implement it b/c I could see along the way that it was something I wasn't interested in using, but it was shaping up along the lines of the following. Please correct the pseudo-code for others. I actually stopped cold with what you see here ...
"Microsoft.Extensions.Configuration.EnvironmentVariables": "1.0.0-*"

using Microsoft.Extensions.Configuration;

var config = new ConfigurationBuilder()
    .AddEnvironmentVariables(prefix: "ASPNETCORE_")
    .Build();
var contentRootPath = Environment.GetEnvironmentVariable("ASPNETCORE_CONTENTROOT") ??
        Directory.GetCurrentDirectory();
var host = new WebHostBuilder()
    .UseContentRoot(contentRootPath) ...;

web.config to be placed in wwwroot ...
<?xml version="1.0" encoding="utf-8"?>
<configuration>
  <system.webServer>
    <handlers>
      <add
          name="aspNetCore"
          path="*"
          verb="*"
          modules="AspNetCoreModule"
          resourceType="Unspecified" />
    </handlers>
    <aspNetCore
          processPath="..\teststandalone.exe"
          stdoutLogEnabled="true"
          stdoutLogFile="..\logs\teststandalone">
      <environmentVariables>
        <environmentVariable name="ASPNETCORE_CONTENTROOT" value="../" />
      </environmentVariables>
    </aspNetCore>
  </system.webServer>
</configuration>

Nah ..... It's not called the "CRAPJOB" pattern ... it's the "NUTJOB" pattern. :smile: This approach seems to do a lot of engineering for something that should be a simple workaround.
Another option ... Hidden Segments
Going back to my earlier idea (and the one spelled out in the IIS doc): Leave web.config in the content root but leverage Request Filtering to protect known sensitive files. You know, a big aspect of "NUTJOB" is to keep server config at the absolute bare minimum, so I would want to implement this in web.config.
web.config in its normal spot in the content root ...
<?xml version="1.0" encoding="utf-8"?>
<configuration>
  <system.webServer>
    <handlers>
      <add
      <add
          name="aspNetCore"
          path="*"
          verb="*"
          modules="AspNetCoreModule"
          resourceType="Unspecified" />
    </handlers>
    <aspNetCore
          processPath=".\teststandalone.exe"
          stdoutLogEnabled="true"
          stdoutLogFile=".\logs\teststandalone">
    <security>
      <requestFiltering>
        <hiddenSegments>
          <add segment="teststandalone.xml" />
          <add segment="teststandalone.deps.json" />
          <add segment="teststandalone.runtimeconfig.json" />
        </hiddenSegments>
      </requestFiltering>
    </security>
  </system.webServer>
</configuration>

I confirmed that this works. The Hidden Segments entries shown below were loaded from the web.config when IIS hit the file ...

Note: teststandalone.xml is not a normal (cli-generated) file. This file is here because I set "xmlDoc": true, on buildOptions in project.json to get an XML Documentation comments file out that I would leave in the content root.
This is lean, clean, and cool :sunglasses: as far as the app is concerned. It would very easy to roll the logic for this into GuardRex.AspNetCore.Server.IISIntegration.Tools. I'll do that because I want the tool to entirely create the file (with logging enabled and with a custom log filename) when I publish and not need to have a web.config file in my projects ... one less trivial file lying around the office! :+1:

@Bartmax @Aequoris Were either/both going to test for web.config in wwwroot for Azure Apps? I'm getting ready to close this, because I'm going to go with the Hidden Segments approach and just leave web.config in my content root (full explanation is below).
Modding dotnet-publish-iis tooling to include Hidden Segments
I have a tooling version that will do this now :sparkles: auto-magically :sparkles:. Now if @moozzyk thought GuardRex.AspNetCore.Server.IISIntegration.Tools was "malware" before, he'll think this is the largest abomination on Earth now. I'll be looking over my shoulder for the next few weeks! :smile:

Enables application logging by default
Changes the name of the log file to be the application name (sans the file extension)
Adds the bits for Hidden Segments to web.config, including XML Doc comments file

I'm using the tool as a self-contained .exe program in my projects, but there shouldn't be any problem leaving it as a tooling dependency and portable app and running it in the usual way. I trigger it ...
"scripts": {
    "postpublish": "<PATH_TO_MY_EXE>\\dotnet-publish-iis --publish-folder %publish:OutputPath% --framework %publish:FullTargetFramework%"
},

... and there is no tools section in my project.json; however, you can clone and mod the tooling and use it in the ordinary way ...
"scripts": {
  "postpublish": "dotnet publish-iis --publish-folder %publish:OutputPath% --framework %publish:FullTargetFramework%"
},
"tools": {
  "Microsoft.AspNetCore.Server.IISIntegration.Tools": {
    "version": "1.0.0-*",
    "imports": "portable-net45+wp80+win8+wpa81+dnxcore50"
  }
}

I'm going to show the full WebConfigTransform.cs, which contains all of the changes that I made to pull this off.
using System;
using System.IO;
using System.Linq;
using System.Xml.Linq;

namespace GuardRex.AspNetCore.Server.IISIntegration.Tools
{
    public static class WebConfigTransform
    {
        public static XDocument Transform(XDocument webConfig, string appName, bool configureForAzure, bool isPortable)
        {
            const string HandlersElementName = "handlers";
            const string aspNetCoreElementName = "aspNetCore";
            const string SecurityElementName = "security";
            const string RequestFilteringElementName = "requestFiltering";
            const string HiddenSegmentsElementName = "hiddenSegments";

            webConfig = webConfig == null || webConfig.Root.Name.LocalName != "configuration"
                ? XDocument.Parse("<configuration />")
                : webConfig;

            var webServerSection = GetOrCreateChild(webConfig.Root, "system.webServer");

            TransformHandlers(GetOrCreateChild(webServerSection, HandlersElementName));
            TransformAspNetCore(GetOrCreateChild(webServerSection, aspNetCoreElementName), appName, configureForAzure, isPortable);

            // Add the Hidden Segments to secure sensitive files
            var securitySection = GetOrCreateChild(webServerSection, SecurityElementName);
            var requestFilteringSection = GetOrCreateChild(securitySection, RequestFilteringElementName);
            var hiddenSegmentsSection = GetOrCreateChild(requestFilteringSection, HiddenSegmentsElementName);
            var appNameWithoutExtension = System.IO.Path.ChangeExtension(appName, null);
            TransformHiddenSegments(hiddenSegmentsSection, appNameWithoutExtension, ".xml");
            TransformHiddenSegments(hiddenSegmentsSection, appNameWithoutExtension, ".deps.json");
            TransformHiddenSegments(hiddenSegmentsSection, appNameWithoutExtension, ".runtimeconfig.json");

            return webConfig;
        }

        private static void TransformHiddenSegments(XElement hiddenSegmentsElement, string appName, string fileNamePart)
        {
            var addSegmentElement =
                hiddenSegmentsElement.Elements("add")
                    .FirstOrDefault(e => string.Equals((string)e.Attribute("segment"), appName + fileNamePart, StringComparison.OrdinalIgnoreCase));

            if (addSegmentElement == null)
            {
                addSegmentElement = new XElement("add");
                hiddenSegmentsElement.Add(addSegmentElement);
            }

            addSegmentElement.SetAttributeValue("segment", appName + fileNamePart);
        }

        private static void TransformHandlers(XElement handlersElement)
        {
            var aspNetCoreElement =
                handlersElement.Elements("add")
                    .FirstOrDefault(e => string.Equals((string)e.Attribute("name"), "aspnetcore", StringComparison.OrdinalIgnoreCase));

            if (aspNetCoreElement == null)
            {
                aspNetCoreElement = new XElement("add");
                handlersElement.Add(aspNetCoreElement);
            }

            aspNetCoreElement.SetAttributeValue("name", "aspNetCore");
            SetAttributeValueIfEmpty(aspNetCoreElement, "path", "*");
            SetAttributeValueIfEmpty(aspNetCoreElement, "verb", "*");
            SetAttributeValueIfEmpty(aspNetCoreElement, "modules", "AspNetCoreModule");
            SetAttributeValueIfEmpty(aspNetCoreElement, "resourceType", "Unspecified");
        }

        private static void TransformAspNetCore(XElement aspNetCoreElement, string appName, bool configureForAzure, bool isPortable)
        {
            // Forward slashes currently work neither in AspNetCoreModule nor in dotnet so they need to be
            // replaced with backwards slashes when the application is published on a non-Windows machine
            var appPath = Path.Combine(configureForAzure ? @"%home%\site" : ".", appName).Replace("/", "\\");
            var logPath = Path.Combine(configureForAzure ? @"\\?\%home%\LogFiles" : @".\logs", System.IO.Path.ChangeExtension(appName, null)).Replace("/", "\\");

            if (!isPortable)
            {
                aspNetCoreElement.SetAttributeValue("processPath", appPath);
            }
            else
            {
                aspNetCoreElement.SetAttributeValue("processPath", "dotnet");

                // In Xml the order of attributes does not matter but it is nice to have
                // the `arguments` attribute next to the `processPath` attribute
                aspNetCoreElement.Attribute("arguments")?.Remove();
                var attributes = aspNetCoreElement.Attributes().ToList();
                var processPathIndex = attributes.FindIndex(a => a.Name.LocalName == "processPath");
                attributes.Insert(processPathIndex + 1, new XAttribute("arguments", appPath));

                aspNetCoreElement.Attributes().Remove();
                aspNetCoreElement.Add(attributes);
            }

            SetAttributeValueIfEmpty(aspNetCoreElement, "stdoutLogEnabled", "true");
            SetAttributeValueIfEmpty(aspNetCoreElement, "stdoutLogFile", logPath);
        }

        private static XElement GetOrCreateChild(XElement parent, string childName)
        {
            var childElement = parent.Element(childName);
            if (childElement == null)
            {
                childElement = new XElement(childName);
                parent.Add(childElement);
            }
            return childElement;
        }

        private static void SetAttributeValueIfEmpty(XElement element, string attributeName, string value)
        {
            element.SetAttributeValue(attributeName, (string)element.Attribute(attributeName) ?? value);
        }
    }
}

This issue is probably related, I perhaps should have created it here: https://github.com/aspnet/Home/issues/1645#issuecomment-230870188


unwind dejan's papi changes

make PAPI work on CPU
eliminate ROCM support that never worked
roll back changes to CUDA support which are problematic because they don't support forking

NOTE: papi CUDA support probably shouldn't be compiled in, though it is

@laksono I believe that this commit fixes all papi CPU issues


Uglify cannot handle the "impure helper"

Hello,
Me again.
I'm using Laravel-Elixir to transpile and compress/minify my app. Looks like Uglify is not able to parse your code. I could successfully replace the helper function by a method, by copy/pasting the helper into a methods: {} section (same level as mounted() {}) and replacing the call to replaceElementWith (line 20) by this.replaceElementWith, and it did the trick.
Uglify still can't parse the code, but it seems to be related to medium-editor and not to your package.
I'll be glad to submit a PR if you wish :)

PRs are very welcome :) Thank you very much.

@dsampaolo did you try excluding medium-editor from uglify?
I would like to reproduce this issue. Can you give me the specific steps / an example? :) thanks!

Closing this for now.

After debugging this (as I'm having the same exact error) turns that Uglify doesn't support some ES6 syntax, so, the recommended solution to me was to add "uglify-js": "github:mishoo/UglifyJS2#harmony" to my package.json which supports es6+
Hope this helps somebody.

Thank you for updating us on this issue :) Appreciated.


Gestures don't work well when using gestures over inappwebview.

So there is a problem when using multiple gesture recognizers on top of a webview in iOS (have not tested with android).  This stackoverflow issue explains the problem and has a solution for the webview_flutter package. https://stackoverflow.com/questions/57069716/scrolling-priority-when-combining-horizontal-scrolling-with-webview/57150906#57150906 .  Basically when your gesture matches a custom gesture it always wins.  Ultimately this prevents the webview from being able to scroll unless it receives a gesture that does not overlap with a custom gesture I setup in a stack.
I have created a mimimum reproduction so you can see this issue.  This project will allow you to swipe left and right between three pageviews.  When you try to scroll vertically it only works if you do a perfectly vertical swipe so as not to also activate the pageview gesture detector.
If you comment out your package import and uncomment out the webview_flutter one.
Then replace this line runApp(InappWebViewUI());  with this one runApp(WebViewExample());
you can see how scrolling works ok with the stackoverflow fix and the gesture is passed to the webview even when it is not a perfect vertical scroll.
The only solution I know of is if you can add gesture recognizers similar to webview_flutter package.  gestureRecognizers: [             Factory(() => PlatformViewVerticalGestureRecognizer()),           ].toSet(),
I don't know if this is doable or not on your end but it would allow me to activate flutter widgets when performing gestures over inappwebview.  I am using gestures in my own app so I can swipe left and right on the webview to open and close a custom drawer.  Maybe you know of a different way to fix this issue?
import 'package:flutter/material.dart';
import 'package:flutter/gestures.dart';
import 'package:flutter_inappwebview/flutter_inappwebview.dart';
import 'package:flutter/foundation.dart';
//import 'package:webview_flutter/webview_flutter.dart';

void main() async {
  WidgetsFlutterBinding.ensureInitialized();
  debugPrintGestureArenaDiagnostics = true;
  runApp(InappWebViewUI()); //WebViewExample()
}

class InappWebViewUI extends StatelessWidget {
  final GlobalKey webViewKey = GlobalKey();

  InAppWebViewController? webView;
  late ContextMenu contextMenu;
  String url = "";
  double progress = 0;
  var options = InAppWebViewGroupOptions(
      crossPlatform: InAppWebViewOptions(
        useShouldOverrideUrlLoading: false,
        mediaPlaybackRequiresUserGesture: false,
      ),
      android: AndroidInAppWebViewOptions(
        useHybridComposition: true,
      ),
      ios: IOSInAppWebViewOptions(
        allowsInlineMediaPlayback: true,
      ));

  @override
  Widget build(BuildContext context) {
    //print('updated');
    return PageView.builder(
      itemCount: 3,
      itemBuilder: (context, index) {
        return InAppWebView(
          key: webViewKey,
          // contextMenu: contextMenu,
          initialUrlRequest:
              URLRequest(url: Uri.parse("https://flutter.dev/docs")),
          // initialFile: "assets/index.html",
          //initialUserScripts: UnmodifiableListView<UserScript>([]),
          initialOptions: options,
          onWebViewCreated: (controller) {
            webView = controller;
            print("onWebViewCreated");
          },
          onLoadStart: (controller, url) {
            print("onLoadStart $url");
          },
          androidOnPermissionRequest: (InAppWebViewController controller,
              String origin, List<String> resources) async {
            return PermissionRequestResponse(
                resources: resources,
                action: PermissionRequestResponseAction.GRANT);
          },
          shouldOverrideUrlLoading: (controller, navigationAction) async {
            var uri = navigationAction.request.url!;

            if (![
              "http",
              "https",
              "file",
              "chrome",
              "data",
              "javascript",
              "about"
            ].contains(uri.scheme)) {}

            return NavigationActionPolicy.ALLOW;
          },
          onLoadStop: (controller, url) async {
            print("onLoadStop $url");

            webView = controller;
          },
          onProgressChanged: (controller, progress) {},
          onUpdateVisitedHistory: (controller, url, androidIsReload) {
            print("onUpdateVisitedHistory $url");
          },
          onConsoleMessage: (controller, consoleMessage) {
            print(consoleMessage);
          },
        );
      },
    );
  }
}

class WebViewExample extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    //print('updated');
    return PageView.builder(
      itemCount: 3,
      itemBuilder: (context, index) {
        return WebView(
          initialUrl: 'https://flutter.dev/docs',
          gestureRecognizers: [
            Factory(() => PlatformViewVerticalGestureRecognizer()),
          ].toSet(),
        );
      },
    );
  }
}

class PlatformViewVerticalGestureRecognizer
    extends VerticalDragGestureRecognizer {
  PlatformViewVerticalGestureRecognizer({PointerDeviceKind? kind})
      : super(kind: kind);

  Offset _dragDistance = Offset.zero;

  @override
  void addPointer(PointerEvent event) {
    startTrackingPointer(event.pointer);
  }

  @override
  void handleEvent(PointerEvent event) {
    _dragDistance = _dragDistance + event.delta;
    if (event is PointerMoveEvent) {
      final double dy = _dragDistance.dy.abs();
      final double dx = _dragDistance.dx.abs();

      if (dy > dx && dy > kTouchSlop) {
        // vertical drag - accept
        resolve(GestureDisposition.accepted);
        _dragDistance = Offset.zero;
      } else if (dx > kTouchSlop && dx > dy) {
        // horizontal drag - stop tracking
        stopTrackingPointer(event.pointer);
        _dragDistance = Offset.zero;
      }
    }
  }

  @override
  String get debugDescription => 'horizontal drag (platform view)';

  @override
  void didStopTrackingLastPointer(int pointer) {}
}

Environment

Technology
Version

Flutter version
2.0.0

Plugin version
^5.0.4-nullsafety.1

Android version

iOS version

Xcode version

You can set the same gestureRecognizers also in InAppWebView.
The problem here is that you are setting the same GlobalKey webViewKey for all 3 InAppWebViews.
Simple working example:
class InappWebViewUI extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Directionality(
        textDirection: TextDirection.ltr,
        child: PageView.builder(
          itemCount: 3,
          itemBuilder: (context, index) {
            return InAppWebView(
              initialUrlRequest: URLRequest(url: Uri.parse('https://flutter.dev/docs')),
              gestureRecognizers: [
                Factory(() => PlatformViewVerticalGestureRecognizer()),
              ].toSet(),
            );
            return Container();
          },
        )
    );
  }
}

Ok thanks for the info! Somehow missed that you could set them in inappwebview.  Also thanks for the key info.  I will double check that I am using unique keys in my own project.


Banner overlaps properties when "Properties in document" is set to "Visible"

As the title says.
To reproduce simply add properties and banner to a note.
I don't think other plugins I use are the issue but I also use: Advanced Tables, Completr, Emoji Toolbar, Excalidraw, Iconize, Image in Editor, Image Layouts, Latex Suite, Remotely Save and Text Format.

Same problem here...

same problem :(

Same here. I created an empty vault to make sure this issue is not caused by other plugins I use and I can confirm – it occurs also with the most default of Obsidian settings.

How to fix
Step 1: Install this plugin

Step 2:
Create a directory/folder with exact name css-snippets at root of your vault.
Create CSS file inside this folder, name doesn't matter. Follow the instructions given in the plugin above.
Note: You can't adit CSS file inside Obsidian. You need to open this file in some other text editor.
Step 3:
Copy & paste the following snippet in the CSS file that you have created:
.obsidian-banner-wrapper {
    margin-top: 0;
    padding-top: 0;
}

.mod-header {
    margin-top: calc(var(--banner-height) - 30px);
}

Note: You can customize how much space you want between image and front matter by updating the 30px in the CSS snippet above
Step 4: Reload Obsidian

This is how fix looks like:

@tejas-hosamani, Obsidian supports snippet management out of the box in Settings > Appearance > CSS Snippets. Once a snippet is active, Obsidian re-renders its view every time the snippet is modified. Since the plugin you submitted was last updated 4 years ago, I infer it must've been created when snippets management was not as easy as it is now.
I tested your snippet in a clean, default vault and it worked fine. In my customized vault, however, it created excessive top margins in note embeds and previews.


Dense dimension sparse input

Proposed changes:
Do not set the output dimension of the sparse-to-dense layers to the same dimension as the dense features.
closes https://github.com/RasaHQ/rasa/issues/6555
Status (please check what you already did):

[ ] added some tests for the functionality
[ ] updated the documentation
[x] updated the changelog (please check changelog for instructions)
[x] reformat files using black (please check Readme for instructions)

/modeltest

include:
 - dataset: "Carbon Bot", "Sara","Hermit"]
   config:
      - "Sparse + DIET(bow) + ResponseSelector(bow)"
      - "Sparse + DIET(seq) + ResponseSelector(bow)"
      - "Sparse + ConveRT + DIET(bow) + ResponseSelector(bow)"
      - "Sparse + ConveRT + DIET(seq) + ResponseSelector(bow)"
      - "Sparse + Spacy + DIET(bow) + ResponseSelector(bow)"
      - "Sparse + Spacy + DIET(seq) + ResponseSelector(bow)"

/modeltest

include:
 - dataset:
      - "Carbon Bot"
      - "Sara"
   config:
      - "Sparse + DIET(bow) + ResponseSelector(bow)"
      - "Sparse + DIET(seq) + ResponseSelector(bow)"
      - "Sparse + ConveRT + DIET(bow) + ResponseSelector(bow)"
      - "Sparse + ConveRT + DIET(seq) + ResponseSelector(bow)"
      - "Sparse + Spacy + DIET(bow) + ResponseSelector(bow)"
      - "Sparse + Spacy + DIET(seq) + ResponseSelector(bow)"

Results match those from the master branch (see https://metabase.rasa.com/dashboard/166)

The train time for Sparse + ConveRT + DIET(bow) + ResponseSelector(bow) increased for Sara and Carbon Bot by a couple of seconds. Without ConveRT it stayed the same.


Latex not working after download external resources

I used academic admin to download all external resources.
academic import --assets
Downloading jquery.min.js from https://cdnjs.cloudflare.com/ajax/libs/jquery/3.3.1/jquery.min.js...
Downloading bootstrap.min.js from https://cdnjs.cloudflare.com/ajax/libs/twitter-bootstrap/4.1.3/js/bootstrap.min.js...
Downloading highlight.min.js from https://cdnjs.cloudflare.com/ajax/libs/highlight.js/9.12.0/highlight.min.js...
Downloading MathJax.js from https://cdnjs.cloudflare.com/ajax/libs/mathjax/2.7.4/MathJax.js?config=TeX-AMS_CHTML-full...
Downloading isotope.pkgd.min.js from https://cdnjs.cloudflare.com/ajax/libs/jquery.isotope/3.0.4/isotope.pkgd.min.js...
Downloading imagesloaded.pkgd.min.js from https://cdnjs.cloudflare.com/ajax/libs/jquery.imagesloaded/4.1.3/imagesloaded.pkgd.min.js...
Downloading autotrack.js from https://cdnjs.cloudflare.com/ajax/libs/autotrack/2.4.1/autotrack.js...
Downloading gmaps.min.js from https://cdnjs.cloudflare.com/ajax/libs/gmaps.js/0.4.25/gmaps.min.js...
Downloading leaflet.js from https://cdnjs.cloudflare.com/ajax/libs/leaflet/1.2.0/leaflet.js...
Downloading jquery.fancybox.min.js from https://cdnjs.cloudflare.com/ajax/libs/fancybox/3.2.5/jquery.fancybox.min.js...
Downloading fuse.min.js from https://cdnjs.cloudflare.com/ajax/libs/fuse.js/3.2.1/fuse.min.js...
Downloading jquery.mark.min.js from https://cdnjs.cloudflare.com/ajax/libs/mark.js/8.11.1/jquery.mark.min.js...
Downloading instantsearch.min.js from https://cdnjs.cloudflare.com/ajax/libs/instantsearch.js/2.10.2/instantsearch.min.js...
Downloading anchor.min.js from https://cdnjs.cloudflare.com/ajax/libs/anchor-js/4.1.1/anchor.min.js...
Merging JS assets into static/js/vendor/main.min.js
Downloading bootstrap.min.css from https://cdnjs.cloudflare.com/ajax/libs/twitter-bootstrap/4.1.3/css/bootstrap.min.css...
Downloading all.css from https://use.fontawesome.com/releases/v5.6.0/css/all.css...
Downloading academicons.min.css from https://cdnjs.cloudflare.com/ajax/libs/academicons/1.8.6/css/academicons.min.css...
Downloading leaflet.css from https://cdnjs.cloudflare.com/ajax/libs/leaflet/1.2.0/leaflet.css...
Downloading jquery.fancybox.min.css from https://cdnjs.cloudflare.com/ajax/libs/fancybox/3.2.5/jquery.fancybox.min.css...
Downloading instantsearch.min.css from https://cdnjs.cloudflare.com/ajax/libs/instantsearch.js/2.9.0/instantsearch.min.css...
Downloading instantsearch-theme-algolia.min.css from https://cdnjs.cloudflare.com/ajax/libs/instantsearch.js/2.10.2/instantsearch-theme-algolia.min.css...
Downloading github.min.css from https://cdnjs.cloudflare.com/ajax/libs/highlight.js/9.12.0/styles/github.min.css...
Merging CSS assets into static/css/vendor/main.min.css

As you can see, MathJax.js was downloaded.
I came across https://github.com/sourcethemes/academic-admin/issues/15 and solved it for me by downloading the missing files and add them to static/extensions.
Beside I expecting this should be done by academic-admin, I found out that LaTeX support is not working anymore as it shows the LaTeX sourcecode.

This is unrelated to this repo. Academic Admin's function to import assets is incomplete and there are a number of issues open awaiting contributions to finish it. Feel free to contribute fixes to that repo.


TSet sink should return a value so we can add an input

Can you put an example API usage here?

We can use lazySink to achieve this. We cannot use sink because it makes the graph evaluated


Enhancing Itinerary page with date selection and Sorting based on date and time

Letting them plan their multiple trips by letting them schedule the itinerary will keep the trip organized. So adding date and time field and automatically sorting the locations based on the date saved using event listeners.


Issue in post by clout

In dark theme there is no visibility for post by clout drop down

closed with #151


fix: setting tags for serialization

ref: https://github.com/cectc/dbpack/issues/
Ⅰ. Describe what this PR did
Ⅱ. Does this pull request fix one issue?
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #245 (defdbb9) into dev (6f496d3) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev     #245   +/-   ##
=======================================
  Coverage   43.20%   43.20%
=======================================
  Files          82       82
  Lines        9810     9810
=======================================
  Hits         4238     4238
  Misses       5139     5139
  Partials      433      433

Impacted Files
Coverage Δ

pkg/filter/breaker/breaker.go
61.53% <ø> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


NTR: interphase MTOC

There is an additional type of non centrosomal MTOC in
fission yeast
can we have a new term
interphase MTOC
poss Def: A non centrosomal microtubule organizing
center which nucleates microtubules from multiple sites
near the cell nucleus during interphase
This def is adapted from
Sawin et. al Current biology in press
no pubmed yet
Reported by: ValWood
Original Ticket: "geneontology/ontology-requests/1642":https://sourceforge.net/p/geneontology/ontology-requests/1642

thanks www.movied.org


fix: liked songs icon on artist profile

Fixes incorrect color of liked songs icon from the artist profile.

Done with ❤ @MGPlayzYT


Criteo bid adapter. Faulty description.

Page - http://prebid.org/dev-docs/bidders.html#criteo
Bidder code say that:
// either one of zoneId or networkId should be set
if (!(bid && bid.params && (bid.params.zoneId || bid.params.networkId))) {
return false;
}
zoneId parameter is not listed among other parameters while it is actually required one.

@leonardlabat - can you take a look at this?

Hello,
Sorry for the late answer. zoneId is actually a deprecated param. I've just updated the doc to make it appearing in the parameters list (while specifying that it is deprecated) : https://github.com/prebid/prebid.github.io/pull/2275
Thanks for reporting this


unsafe DOM request

https://github.com/bamdadsabbagh/youtube-blacklist--extension/blob/64451df40b8c085d545404eb0c2090ce0397dbd1/src/utils/create-close-button.js#L39
can fail

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

v1.0.5
GitHub release

Your semantic-release bot :package::rocket:

seem fixed, closing


add support of property validation message

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

Proposed changes

As discussed in this  forum thread, the purpose of this PR is to present to the user the cause of the failure of node property verification in an easy-to-understand manner.

The property validation function of the property definition can return a string instead of a boolean value. This string is used as an explanation for the validation error.

For compatibility, this function takes optional second argument which is currently empty object.  So, newly developed node can check this argument and change its behavior.
// example:
defaults: {
    property_name: {
         value: ...,
         validate: function (v, opt) {
              if (check is ok) {
                  return true;
              }
              if (opt) {
                  return "error description";
              }
             return false;
         }
    }, ...
}

Property definition may have label property.  The value s used as a display name in the validation of the required field and configuration node error.

// example:
defaults: {
    property_name: {
         value: ..., required: true, label: "meaningful name"
    }, ...
}

Standard validation function generation functions provided as RED.validators.* is modified to accept additional  option object.  This object can have a property property. property The property is used as the display name when generating an error message.

// example:
defaults: {
    property_name: {
         value: ...,
         validate: RED.validators.number(false, {
             property: "meaningful property name"
         })
    }, ...
}

Show an error message using tooltip for a field that failed validation.

For this, added delete function to  RED.popover.tooltip.
This feature does not work for some fields (e.g. typedInput), probably because of a conflict.

Checklist

[x] I have read the contribution guidelines
[x] For non-bugfix PRs, I have discussed this change on the forum/slack team.
[x] I have run grunt to verify the unit tests pass
[ ] I have added suitable unit tests to cover the new/changed functionality

Coverage remained the same at 67.313% when pulling 9f98b4b082db0a56740941072cda02eb65eb3d5d on node-red-hitachi:property-validation-tooltip-message into 5293563a6a5903106a84920043f5948ab205c1c3 on node-red:dev.

Are these "meaningful labels" easily translated ? by the standard this._("mynode.meaningful.label") style ?

Are these "meaningful labels" easily translated ? by the standard this._("mynode.meaningful.label") style ?

Yes. You can define it in property definition as:
prop: {
    value: "...",
    label: RED._("mynode.meaningful.label")
}


An improved asm command

I was doing some benchmarks on assembling larger problems. I noticed that there is some unnecessary overhead in the approach used here.
The overhead is not visible in smaller problems but starts to be significant in 3D after 2M DOF's or so.
Some remarks:

Assembling similar matrices in MATLAB using sparse is faster
It seems that this was solved in SfePy by writing a separate matrix assembler in Cython

Could you share the benchmark?  I've also been doing some profiling here and it might help if our measures are commensurate.
For mine so far (2D, 1.5M DOFs), cProfile revealed that almost half of the time was spent in meshio.read, so I was going to look into that before assembly or solution.
Before the profiling, I had thought that assembly might be the culprit and had been thinking about Cython or Numba.  After reading The case for Numba in community code I thought that an advantage of the latter in the context of scikit-fem might be that in happy cases it only involves a wrapping a @jit decorator around native Python which can be ignored if Numba isn't available.  As my immediate problem lay in I/O, I didn't measure this yet.

I am not sure how to make a good benchmark as you would probably need to compare to MATLAB or SfePy.
You can see the effect in the following snippet. Save this to benchmark.py.
from skfem import *

@bilinear_form
def mass(u, du, v, dv, w):
    return u*v

m = MeshTet()
m.refine(6)
ib = InteriorBasis(m, ElementTetP1(), MappingAffine(m), 2)

def run1():
    return asm(mass, ib)

def run2():
    return asm(mass, ib, assemble=False)

Then in iPython do
%run benchmark.py
%time run1()
%time run2()

and compare the timings. I suppose the difference is larger than it should be.
I think one could do something similar to what is done in SfePy using Numba.

I forgot to add my timings
In [1]: %run benchmark.py

In [2]: %time run1()
CPU times: user 28.2 s, sys: 1.56 s, total: 29.8 s
Wall time: 29.9 s
Out[2]:
<230945x230945 sparse matrix of type '<class 'numpy.float64'>'
	with 3363425 stored elements in Compressed Sparse Row format>

In [3]: %time run2()
CPU times: user 3.61 s, sys: 868 ms, total: 4.48 s
Wall time: 4.49 s

I suggest a modification to this part so that the user can provide a function which inputs the variables data (flattened), rows and cols, and outputs a sparse matrix in any suitable format. A default implementation could be the current one, and we could then implement a faster alternative using Numba.
I can foresee a third alternative implementation to be handy in, e.g., GPU computations where the output could be a sparse array allocated on the GPU.

Thank you.
Yes, after reading 'The case for Numba in community code' I was hoping Numba might do; compared to Cython, it seemed closer to the spirit of scikit-fem being native Python.
I don't have MATLAB but do have GNU Octave and some in-house finite element code which uses sparse to assemble finite-element bilinear forms.  I don't think I've run the latter since GNU Octave acquired a JIT compiler.  I also haven't installed SfePy since I switched to Python 3 - it's not nearly as easy  pip install scikit-fem .

Wow, I see now why I hadn't noticed this issue before.  I'd been working on planar problems and it really is a three-dimensional issue, as you say.  Much bigger two-dimensional operators are much quicker; e.g.
m = MeshTri()
m.refine(10)

gives
%time run1()
CPU times: user 3.98 s, sys: 1.26 s, total: 5.24 s
Wall time: 5.25 s
Out[3]:
<1050625x1050625 sparse matrix of type '<class 'numpy.float64'>'
	with 7346177 stored elements in Compressed Sparse Row format>

(This is on a slightly slower machine; taking about 40 s for the three-dimensional run1 cf. the 28 quoted above.)

Hmm, hang on.  I'm getting quite different behaviour in comparing run1 and run2.  I'm not that familiar with ipython so am running with your benchmark.py as copied and pasted into a file, then two separate other scripts, benchmark1.py:
from benchmark import run1

run1()

and benchmark2.py:
from benchmark import run2

run2()

Then running these with cProfile; e.g.
python -m cProfile -s cumtime benchmark2.py

The results are almost the same: 46 and 44 s.  Two thirds of this in each case is 30 s in refine.  The two calls to asm are 7 s and 4 s; so assemble=False makes a little bit of difference but not much.

Actually no, I'm getting the same 7 and 4 s results in ipython too…; viz.
In [1]: %run benchmark.py

In [2]: %time run1()
CPU times: user 5.54 s, sys: 1.52 s, total: 7.06 s
Wall time: 7.08 s
Out[2]:
<230945x230945 sparse matrix of type '<class 'numpy.float64'>'
	with 3363425 stored elements in Compressed Sparse Row format>

In [3]: %time run2()
CPU times: user 3.09 s, sys: 942 ms, total: 4.04 s
Wall time: 4.05 s

So I can't reproduce your 30 second run1 v. 4 second run2.  I'm running ipython 6.4.0 and Python 3.6.6 under Debian GNU/Linux testing (buster).

It seems that this was fixed already in SciPy 1.0.0. For some reason I had an ancient version on my personal laptop.
I'm leaving the issue open since I still wish to make the modifications discussed earlier.
Thank you @gdmcbain for your input.

I'm postponing this idea of modifying asm so that a practical use case appears.


chore(contributing): change lit-html to lit

Just change lit-html to lit in CONTRIBUTING.md

Thanks!

Thanks!

You're welcome

👍🏻


Some CLI commands require the following common options:

running on travis,
$ ./jfrog bt pc --key=$BTKEY --user=$GH_USER --licenses=MIT --vcs-url=https://github.com/$GH_USER/$GH_APP $GH_USER/$GH_APP/$GH_APP || echo "package already exists"

Some CLI commands require the following common options:
- User
- API Key
- Default Package Licenses
Configuring JFrog CLI with these parameters now will save you having to include them as command options.
You can also configure these parameters later using the 'config' command.
Configure now? (y/n):

if there would be -y option that s cool
if there is no such question becasue i provided al required data, that s coo too
if detects its not a tty, thus does not ask user input, that s cool also

@mh-cbon, if you set the value of the JFROG_CLI_OFFER_CONFIG environment variable to true, the CLI will not prompt the offer config message.
Please let us know if this helps.

I will not say this is obvious, but yeah that helps. thanks

Why don't you support the --interactive false? That would avoid later errors on the same topic...

How to set JFROG_CLI_OFFER_CONFIG to false ?

We pushed https://github.com/jfrog/jfrog-cli-go/commit/93818f536d84451d064841ca8bdc52e438340bcd, so that prompts message also suggests the option of setting the JFROG_CLI_OFFER_CONFIG to false.
@Kuanlin-Chen - JFROG_CLI_OFFER_CONFIG is an environment variable.
Set it to false as follows:
export JFROG_CLI_OFFER_CONFIG=false
or on Windows:
set JFROG_CLI_OFFER_CONFIG=false


feat: 主题中增加图例样式的配置，便于用户了解怎么修改

Checklist

[x] npm test passes
[ ] tests and/or benchmarks are included
[x] commit message follows commit guidelines

Description of change

legend 的状态样式写死在 component 中，其实是可以修改的；参考下面两个文件：

https://github.com/antvis/component/blob/7d74c4753ce4cc5bcb54f8f278f9ea62d6a0c6bf/src/util/theme.ts
https://github.com/antvis/component/blob/912fb582894be39307b508edbd0468ea7fca16d1/src/legend/category.ts#L109

closed: #3375

Pull Request Test Coverage Report for Build 750911505

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 93.509%

Totals

Change from base Build 743742278:
0.0%

Covered Lines:
10916

Relevant Lines:
11321

💛  - Coveralls

你好，我想问一下，图例的每一项内容太长超过之后会隐藏，那么鼠标hover上去的tooltip样式可以通过什么方式修改呢？


KuBeanClusterOps explain info is not accurately

Describe the version
version about:

kubean: v0.0.0
kubespray: master
kubenetes: not related
what CNI and itsversion: not related

Describe the bug
A clear and concise description of what the bug is.

The explanation is incorrect.
How To Reproduce
Steps to reproduce the issue:
kubectl explain KuBeanClusterOps
Expected behavior
A clear and concise description of what you expected to happen.
KuBeanClusterOps should be explained correctly.
Screenshots and log
If applicable, add screenshots and log to help explain your problem.
Additional context
Add any other context about the problem here.

KuBeanCluster explanation should also be confirmed.


Can't deploy lines starting with dash with eyaml

When trying to deploy a eyaml file and a line in my encrypted file starts with a dash "-", I can't deploy the file.
Example:
private-file: >
DEC::PKCS7[---TEST---]!
If I add a single quote or other character (i.e. '---TEST---), it gets deployed without issues (but of course with the single quote)
Please help

Anybody know more about this or a workaround? Causing some headache for me when trying to encrypt SSL certificates.


Modify jcloudsLocation.releaseNode

Workaround waiting for a fix for https://issues.apache.org/jira/browse/JCLOUDS-1332
Uses destroyNodesMatchingId instead of destroyNode as it consistently invokes cleanUpIncidentalResourcesOfDeadNodes

Should the same change be applied to killMachine( as well?

@andreaturli code LGTM - thanks! Nice and simple fix for an annoying problem.
I've not tested it myself. If you or someone else has done so on appropriate cloud(s), then very happy for you to do so.

retest this please

This PR swaps the mechanism Brooklyn is using to destroy a node to one which is less tested within Brooklyn. The problem it addresses has been fixed upstream in jclouds so assuming we can incorporate that in before the next release we should not need this PR.
I think however we should leave this PR open and unmerged in the meantime in case the jclouds release is not available and this fix has not been incorporated before the next release.


Make timeout configurable for DNS MX record lookup

There are 2 obscure extra properties that control the initial timeout and retries in JNDI.

com.sun.jndi.dns.timeout.initial - the default is 1000 (ms)
com.sun.jndi.dns.timeout.retries - the default is 4

These go in the env Hashtable passed to InitialDirContext.
Currently, checking if an MX record exists for a domain like coolio.com is super bad for performance. Since user input cannot be trusted, perhaps this could be configurable.
Awesome lib, btw. 💯

Hi @dotneutron, thank you for opening this issue and bringing this to my attention, I didn't know that these configuration options existed!
I'm actually now considering setting a lower default within JMail, along with allowing for configurable timeout and retries. Maybe a default of 100ms for the initial timeout and 2 retries.
What do you think?

I went ahead and made these changes and released version 1.5.1 which contains the new feature. Thanks again for the suggestions!


How to repack .unity3d ? HELP

After doing the desunity of a file, how can I do repack for .unity3d?

You can't pack files without a property file. Please close.

dump the raw file, modify it, then rewrite it to replace the old raw file, you should make a tool yourself to do this


[feature request] Make use of the node description field

Thanks for this extension.
Since #1438 we can add a DESCRIPTION field in python that can be leveraged in js (I'm implementing a side documentation panel based on that here)
It would be great to optionally display DESCRIPTION in the sidebar.

Genial!!!

I second this request. This sidebar with the node preview (thanks again for listening to my request on Reddit and implementing it) + the node description would be exceptional.

Ok, Description implemented! Even though I don't know any node that really uses it xD, but it supports HTML so it's pretty cool!


index page does not display header

[Bug]
David, thanks for your beautiful web design. I have an exact copy of your repo and set it as a github page here:
https://lillianpetersen.github.io/elements/
but the home page does not display the header, it only shows the content of index.md:
https://lillianpetersen.github.io/
Have you ever seen that before? It might be a github pages bug, rather than related to alembic. This started when I copied a few simple html pages into the top directory, but after I deleted them, and then even reverted to the head of your repo, it did not fix.
I'm on your last commit 174733d, but removed the CNAME file. But it happened the first time with an alteration of a 2017 commit, aaa56f0, so it is not any problem that was recently introduced.

Well, I found a work-around. I copied index.md to home.md and put the line:
<meta http-equiv="Refresh" content="0; url=//lillianpetersen.github.io/home" />

at the top of index.md. The problem was mysterious, but probably not worth investigating.

Hey @mark-petersen, I think that's a configuration bug at your end. Not sure why the nav wouldn't be appearing, it could be the formatting in your index.md or that you need to set the layout to 'page'


Create custom-made logo using image creator

You get the idea!
We can generate custom-made logo
I did these with OpenAI and Blazor logo!

This way the logo of the project itself is created using AI

I was thinking if we come up with good generated picture, It's good replacement for main image of App
this way beside that we have AI generated logo, we show the power of Bing.
One for two. 😉😈


Update PPP

Codecov Report

Merging #274 (3057fa8) into master (66c74cf) will increase coverage by 0.13%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #274      +/-   ##
============================================
+ Coverage     70.60%   70.74%   +0.13%
- Complexity      811      814       +3
============================================
  Files           111      111
  Lines          2732     2738       +6
  Branches        366      365       -1
============================================
+ Hits           1929     1937       +8
+ Misses          693      691       -2
  Partials        110      110

Impacted Files
Coverage Δ

...ddress/logic/commands/HideAppointmentsCommand.java
31.57% <0.00%> (-1.76%)
:arrow_down:

...ddress/logic/parser/HidePatientsCommandParser.java
65.00% <0.00%> (-1.67%)
:arrow_down:

...el/person/predicates/HideAppointmentPredicate.java
23.07% <0.00%> (-0.93%)
:arrow_down:

...java/seedu/address/logic/commands/FindCommand.java
96.87% <0.00%> (ø)

...du/address/logic/commands/HidePatientsCommand.java
95.45% <0.00%> (ø)

.../address/logic/commands/UnhidePatientsCommand.java
0.00% <0.00%> (ø)

...ress/logic/commands/UnhideAppointmentsCommand.java
0.00% <0.00%> (ø)

.../logic/parser/UnhideAppointmentsCommandParser.java
0.00% <0.00%> (ø)

...rc/main/java/seedu/address/model/ModelManager.java
98.63% <0.00%> (+0.03%)
:arrow_up:

...main/java/seedu/address/model/AppointmentList.java
95.12% <0.00%> (+0.52%)
:arrow_up:

... and 2 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more