BULK VALIDATION title missing fixes

Docs Build status updates of commit fe69fd7:
:warning: Validation status: warnings

File
Status
Preview URL
Details

Skype/UCWA/contactLocation_ref.md
:warning:Warning

Details

Skype/UCWA/conversation_ref.md
:warning:Warning

Details

Skype/UCWA/delegatesGroup_ref.md
:warning:Warning

Details

Skype/UCWA/participantDataCollaboration_ref.md
:warning:Warning

Details

Skype/UCWA/replacesPhoneAudio_ref.md
:warning:Warning

Details

Skype/WebSDK/docs/ManageDevices.md
:warning:Warning

Details

Skype/AppSDK/GetOnlineMeetingJoinResources.md
:white_check_mark:Succeeded

Skype/UCWA/RequirementPolicy_ref.md
:white_check_mark:Succeeded

Skype/UCWA/Status_ref.md
:white_check_mark:Succeeded

Skype/UCWA/invitations_ref.md
:white_check_mark:Succeeded

Skype/UCWA/onBehalfOf_ref.md
:white_check_mark:Succeeded

Skype/UCWA/onlineMeetingEligibleValues_ref.md
:white_check_mark:Succeeded

Skype/WebSDK/docs/DevelopForSkypeforBusiness.md
:white_check_mark:Succeeded

Skype/WebSDK/docs/PTChat.md
:white_check_mark:Succeeded

Skype/WebSDK/docs/PTChatOutgoingP2P.md
:white_check_mark:Succeeded

Skype/WebSDK/docs/troubleshooting/auth/AADAuth-ClientID.md
:white_check_mark:Succeeded

Skype/UCWA/contactLocation_ref.md

Line 52, Column 1: [Warning: table-syntax-invalid - See documentation] Table syntax is invalid. Ensure your table includes a header and is surrounded by empty lines.
Line 64, Column 1: [Warning: table-syntax-invalid - See documentation] Table syntax is invalid. Ensure your table includes a header and is surrounded by empty lines.

Skype/UCWA/conversation_ref.md

Line 80, Column 1: [Warning: table-syntax-invalid - See documentation] Table syntax is invalid. Ensure your table includes a header and is surrounded by empty lines.
Line 92, Column 1: [Warning: table-syntax-invalid - See documentation] Table syntax is invalid. Ensure your table includes a header and is surrounded by empty lines.
Line 128, Column 1: [Warning: table-syntax-invalid - See documentation] Table syntax is invalid. Ensure your table includes a header and is surrounded by empty lines.
Line 164, Column 1: [Warning: table-syntax-invalid - See documentation] Table syntax is invalid. Ensure your table includes a header and is surrounded by empty lines.

Skype/UCWA/delegatesGroup_ref.md

Line 55, Column 1: [Warning: table-syntax-invalid - See documentation] Table syntax is invalid. Ensure your table includes a header and is surrounded by empty lines.
Line 67, Column 1: [Warning: table-syntax-invalid - See documentation] Table syntax is invalid. Ensure your table includes a header and is surrounded by empty lines.
Line 107, Column 1: [Warning: table-syntax-invalid - See documentation] Table syntax is invalid. Ensure your table includes a header and is surrounded by empty lines.

Skype/UCWA/participantDataCollaboration_ref.md

Line 51, Column 1: [Warning: table-syntax-invalid - See documentation] Table syntax is invalid. Ensure your table includes a header and is surrounded by empty lines.
Line 64, Column 1: [Warning: table-syntax-invalid - See documentation] Table syntax is invalid. Ensure your table includes a header and is surrounded by empty lines.
Line 105, Column 1: [Warning: table-syntax-invalid - See documentation] Table syntax is invalid. Ensure your table includes a header and is surrounded by empty lines.
Line 146, Column 1: [Warning: table-syntax-invalid - See documentation] Table syntax is invalid. Ensure your table includes a header and is surrounded by empty lines.

Skype/UCWA/replacesPhoneAudio_ref.md

Line 45, Column 1: [Warning: table-syntax-invalid - See documentation] Table syntax is invalid. Ensure your table includes a header and is surrounded by empty lines.

Skype/WebSDK/docs/ManageDevices.md

Line 5, Column 1: [Warning: duplicate-h1s - See documentation] H1 'Manage devices' is duplicated with other articles: 'WebSDK/docs/ManageDevices.md(5,1), WebSDK/docs/PTDevicesManagerManageDevices.md(4,1)'. First level headings must be unique within a docset.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Global effort to fix build validation errors
@lindalu-MSFT NOTE: The validation warnings and suggestions in this PR will be fixed by another author in another PR.
The Content & Learning team is fixing build validation errors on docs.microsoft.com for the rest of H2. This will eliminate potential accessibility, security, and usability issues. This PR includes only build validation fixes and does not change other content. Please review within five business days and merge or comment in the PR with any changes you’d like to see. Thank you!

Docs Build status updates of commit de589fc:
:warning: Validation status: warnings

File
Status
Preview URL
Details

Skype/UCWA/contactLocation_ref.md
:warning:Warning

Details

Skype/UCWA/conversation_ref.md
:warning:Warning

Details

Skype/UCWA/delegatesGroup_ref.md
:warning:Warning

Details

Skype/UCWA/participantDataCollaboration_ref.md
:warning:Warning

Details

Skype/UCWA/replacesPhoneAudio_ref.md
:warning:Warning

Details

Skype/WebSDK/docs/ManageDevices.md
:warning:Warning

Details

Skype/AppSDK/GetOnlineMeetingJoinResources.md
:white_check_mark:Succeeded

Skype/UCWA/RequirementPolicy_ref.md
:white_check_mark:Succeeded

Skype/UCWA/Status_ref.md
:white_check_mark:Succeeded

Skype/UCWA/invitations_ref.md
:white_check_mark:Succeeded

Skype/UCWA/onBehalfOf_ref.md
:white_check_mark:Succeeded

Skype/UCWA/onlineMeetingEligibleValues_ref.md
:white_check_mark:Succeeded

Skype/WebSDK/docs/DevelopForSkypeforBusiness.md
:white_check_mark:Succeeded

Skype/WebSDK/docs/PTChat.md
:white_check_mark:Succeeded

Skype/WebSDK/docs/PTChatOutgoingP2P.md
:white_check_mark:Succeeded

Skype/WebSDK/docs/troubleshooting/auth/AADAuth-ClientID.md
:white_check_mark:Succeeded

Skype/UCWA/contactLocation_ref.md

Line 52, Column 1: [Warning: table-syntax-invalid - See documentation] Table syntax is invalid. Ensure your table includes a header and is surrounded by empty lines.
Line 64, Column 1: [Warning: table-syntax-invalid - See documentation] Table syntax is invalid. Ensure your table includes a header and is surrounded by empty lines.

Skype/UCWA/conversation_ref.md

Line 80, Column 1: [Warning: table-syntax-invalid - See documentation] Table syntax is invalid. Ensure your table includes a header and is surrounded by empty lines.
Line 92, Column 1: [Warning: table-syntax-invalid - See documentation] Table syntax is invalid. Ensure your table includes a header and is surrounded by empty lines.
Line 128, Column 1: [Warning: table-syntax-invalid - See documentation] Table syntax is invalid. Ensure your table includes a header and is surrounded by empty lines.
Line 164, Column 1: [Warning: table-syntax-invalid - See documentation] Table syntax is invalid. Ensure your table includes a header and is surrounded by empty lines.

Skype/UCWA/delegatesGroup_ref.md

Line 55, Column 1: [Warning: table-syntax-invalid - See documentation] Table syntax is invalid. Ensure your table includes a header and is surrounded by empty lines.
Line 67, Column 1: [Warning: table-syntax-invalid - See documentation] Table syntax is invalid. Ensure your table includes a header and is surrounded by empty lines.
Line 107, Column 1: [Warning: table-syntax-invalid - See documentation] Table syntax is invalid. Ensure your table includes a header and is surrounded by empty lines.

Skype/UCWA/participantDataCollaboration_ref.md

Line 51, Column 1: [Warning: table-syntax-invalid - See documentation] Table syntax is invalid. Ensure your table includes a header and is surrounded by empty lines.
Line 64, Column 1: [Warning: table-syntax-invalid - See documentation] Table syntax is invalid. Ensure your table includes a header and is surrounded by empty lines.
Line 105, Column 1: [Warning: table-syntax-invalid - See documentation] Table syntax is invalid. Ensure your table includes a header and is surrounded by empty lines.
Line 146, Column 1: [Warning: table-syntax-invalid - See documentation] Table syntax is invalid. Ensure your table includes a header and is surrounded by empty lines.

Skype/UCWA/replacesPhoneAudio_ref.md

Line 45, Column 1: [Warning: table-syntax-invalid - See documentation] Table syntax is invalid. Ensure your table includes a header and is surrounded by empty lines.

Skype/WebSDK/docs/ManageDevices.md

Line 5, Column 1: [Warning: duplicate-h1s - See documentation] H1 'Manage devices' is duplicated with other articles: 'WebSDK/docs/ManageDevices.md(5,1), WebSDK/docs/PTDevicesManagerManageDevices.md(4,1)'. First level headings must be unique within a docset.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


MD051 fails with cjk characters

This issue seems to start with 0.30.0

Can you please paste some example characters here in this issue?

(The "Copy link" link at the bottom of that demo page will create a sample directly from your demo.)

标题

Looks related: #955

Please see my comments in issue #955 for an explanation. As in that case, using the URI-encoded form for the link works fine:
https://dlaa.me/markdownlint/#%25m%23 标题

[标题](%23标题)

[标题](%23%25E6%25A0%2587%25E9%25A2%2598)

I would not expect this to have worked in a previous version, but the intent was always to require the use of the GitHub canonical version as is currently being done and observed in the demo page.

You're right that this was not reported in v0.29.0 of the library. I debugged and that's because the markdown-it parser automatically applied a URI encoding to the link destination (likely because it's tokenized as an href) and so it matched the canonical GitHub form. In v0.30.0 of the library, the micromark parser is used for this rule and it does not do that (being a lower-level parser) and so there is a mismatch which leads to the reported issue. The current/updated behavior is what is intended and I consider it an improvement that the rule catches something it didn't before.

I hope you can have an option for this because many editors like vscode,  And a lot of markdown renderers like markdown-it,  do not requird users to.encode anchor hash.
I would be disappointing  If I can only disable this rule,  Instead of having their non-url-encoded version being checked.

@DavidAnson I know this issue is already “Closed” and I’m probably missing something here but,

Please see my comments in issue #955 for an explanation. As in that case, using the URI-encoded form for the link works fine:
https://dlaa.me/markdownlint/#%25m%23 标题

[标题](%23标题)

[标题](%23%25E6%25A0%2587%25E9%25A2%2598)

both the URI-encoded and the non-URI-encoded form works there. One of them would generate an error in the CLI for example.

The current/updated behavior is what is intended and I consider it an improvement that the rule catches something it didn't before.

True. But as already mentioned, URI-encoding is not required in many tools. Both encoded and non-encoded should be allowed. Besides, URI-encoding makes the (raw) Markdown looks hideous.
I hope there’s a way to make it work like your demo site above.

I changed the behavior of this library to allow both kinds in v0.31.0 and that's what you can see in the demo page now. This version of the library is not yet part of a release of CLI or CLI2, so you will not see the change if using either. A new release for both should be forthcoming.

Awesome! Thanks for the heads-up.

We also got hit by this. Is it the same issue:
Heading:
### 设置HTTP重定向

Link:
[HTTP 重定向](#设置http重定向)

It worked in 0.29.0, fails in 0.30.0.

@silverwind Your example produces no errors in the latest version: https://dlaa.me/markdownlint/#%25m%23 Issue 954

%23%23 设置HTTP重定向

[HTTP 重定向](%23设置http重定向)


Rails api mode config

This PR adds notes to the README for necessary rails API configurations.

From my tests it seems this is not necessary and the controller additions are added automatically as expected.


Wrong Inbound Governor State Transitions

The transitions to and from Hot in the inbound governor is wrong. By requiring all hot protocols to have started peers that never fetches a block from the node will be classified as warm.
Current behaviour:

the transition from warm to hot requires all hot protocols to have started.
the transition from hot to warm requires any hot protocol to have terminated

I propose the opposite use of qualifiers, so:

the transition from warm to hot requires any hot protocol to have started.
the transition from hot to warm requires all hot protocols to have terminated

Indeed, we should use Any for all promotions, and All for all demotions - this compose  well while a connection is going from Hot to Cold.


Multi-Key Overseer

First of all, many thanks for this feature. As a pathfinder's lover, I always tought that manage flasks can be too difficult if you try to not "cheat" the system (macro pot spam) and your feature is just amazing since I can really track it much better than GGG's interface.
BUT... I'm used to play holding shift and many times spam flasks holding ability buttons. In this case I won't press only the a hotkey, but a multikey and overseer doesn't reconize that I just pressed a flask button. Eg. Running maps, I hold "W" to use an ability but press "4" to use a flask. Overseer won't reconize that and do not start the count.
I I don't know if it's possible to do such thing, but would be great to have a feature like this: http://i.imgur.com/DCxxr61.png

for now just setup 2 identical buttons with "shift+4" and "4"


Factor out bintray credentials to a setting key

Also allow to provide bintray credentials via environment variable or system property. Resolves #25.
This PR also adds sbt-git plugin, which allows automatic version management depending on the latest tag in git. In addition to clean version management this is also a one step forward to tag based releases (only step required for a release is to create a tag on some commit).

In the meantime I have released bintray-sbt with the following PR merged under
addSbtPlugin("me.lessis" % "bintray-sbt" % "0.3.0-8-g6d0c3f8")
resolvers += Resolver.url("2m-sbt-plugin-releases", url("https://dl.bintray.com/2m/sbt-plugin-releases/"))(Resolver.ivyStylePatterns)

so I can use tag based automatic releases from Travis CI.

@softprops what do you think could be improved, so this PR gets merged? After looking at your past discussions and at some other sbt plugins, I think I should rework this PR to leverage sbt support for credentials. WDYT?

I'd also curious about this PR, I'd like to make a small change to reduce the warnings for missing bintray credentials, and I'd like to base it off of these changes if they are going to be merged soon.

Hey @2m, what's happening with this PR?

This PR should not be merged, but instead changed to use the credentials key provided by sbt. I am not currently working on this, so a PR inspired by this one is very welcome.

Understood! Thanks for the update @2m

Here is my work in progress PR: https://github.com/2m/bintray-sbt/commits/wip-%2324-%2325-sbt-credentials-2m
There is quite a bit of progress there. But I got stuck when making publishTo depend on credentials key. Basically we need this: https://github.com/sbt/sbt/issues/2059

My workaround to use secure env vars on Travis CI is to run a script that includes:
if [[ "$TRAVIS_PULL_REQUEST" == "false"
   && "$TRAVIS_BRANCH" == "master"
   && "$TRAVIS_SECURE_ENV_VARS" == "true"
]]; then
  mkdir ~/.bintray
  cat > ~/.bintray/.credentials <<EOF
realm = Bintray API Realm
host = api.bintray.com
user = dwijnand
password = $BINTRAY_API_KEY
EOF
fi

Why not use credentials += Credentials("Bintray API Realm", "api.bintray.com", "dwijnand", sys.env.getOrElse("BINTRAY_API_KEY", ""))?

bintray-sbt doesn't use the credentials key
On Wed, 23 Nov 2016, 18:36 Schmitt Christian<EMAIL_ADDRESS>wrote:

Why not use credentials += Credentials("Bintray API Realm", "
api.bintray.com", "dwijnand", sys.env.getOrElse("BINTRAY_API_KEY", ""))?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/softprops/bintray-sbt/pull/69#issuecomment-262597035,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAVCIjArE-l_NQHVJEL14UUtUO0VUIDaks5rBIeggaJpZM4FgAdt
.


Collapsible clickable area added

I added this clickable area to allow insert actions (e.g buttons) in collapsible-header and now trigger expand event.
E.g

              ....

              ....

Lorem ipsum dolor sit amet.

Do you have any idea about when this solution will be in official version?

I used a different solution to prevent the collapsible, i'm using a button and setting the class "not-collapsible-clickable" and checked if the button doesn't have this class.
$this.on('click.collapse', '> li > .collapsible-header', function (e) {
if (!$(e.target).hasClass('not-collapsible-clickable')) {
//continue the event
}
});

Hey there 👋
We are happy to see, that you have created a pull request to fix a problem or add a feature or did something else.
Unfortunately does your PR include compiled files. Please do not edit materialize.css, materialize.js or any .html file directly! Those files are automatically generated. You should edit the source files in the locations below instead. Also make sure to check out our CONTRIBUTING GUIDE!
/materialize/sass/
/materialize/js/
/materialize/jade/page-contents/

We won't look into this, unless you modify your PR, so that it matches with our CONTRIBUTING guidelines.

Your PR does not fulfill the requirements outlined by CONTRIBUTING.md


Fix test_delegate_subscribe_unsubscribe flaking test

What is changed and how it works?

Issue Number: Close https://github.com/tikv/tikv/issues/12278
What's Changed:

Fix test_delegate_subscribe_unsubscribe flaking test.
Use the corresponding id, otherwise there will be failures due to concurrent runs of the test. (We will assign downstream ids concurrently).

Related changes

Need to cherry-pick to the release branch

Check List
Tests

Unit test

Side effects
None
Release note
None

Just curious, it seems like DOWNSTREAM_ID_ALLOC is an atomic var, why this problem happened?


Enchantment for issue #277 (ComparisonConfig.CompareBackingFields)

added new config flag for backing fields comparison; added support for backing fields comparison parameter in Cache.GetFieldInfo; added test for new backing field comparison parameter

Approved


ng-add doesn't install the latest version correctly

🐞 Bug report
Command (mark with an x)

[ ] new
[ ] build
[ ] serve
[ ] test
[ ] e2e
[ ] generate
[x] add
[ ] update
[ ] lint
[ ] xi18n
[ ] run
[ ] config
[ ] help
[ ] version
[ ] doc

Is this a regression?
Yes. With @angular/cli@8 doesn't cause that error.
Description
In a project created by @angular/cli@latest, ng add command seems not working correctly.
Executing ng add @ngrx/store, it throws an error.
$ ng add @ngrx/store
Installing packages for tooling via yarn.
Installed packages for tooling via yarn.
The package that you are trying to add does not support schematics. You can try using a different version of the package or contact the package author to add ng-add support.
error Command failed with exit code 1.

But adding @latest to the @ngrx/store, it installs and runs schematics of @ngrx/store.
ng add @ngrx/store@latest
Package has unmet peer dependencies. Adding the package may not succeed.
Installing packages for tooling via yarn.
Installed packages for tooling via yarn.
CREATE src/app/reducers/index.ts (359 bytes)
UPDATE src/app/app.module.ts (715 bytes)
UPDATE package.json (1349 bytes)
✔ Packages installed successfully.
✨  Done in 6.67s.

It seems Angular CLI's bug in ng add command.
🔬 Minimal Reproduction

Create new project with the latest CLI (npx @angular/cli@latest new my-app)
Execute ng add @ngrx/store in the project.

🔥 Exception or Error

$ ng add @ngrx/store
Installing packages for tooling via yarn.
Installed packages for tooling via yarn.
The package that you are trying to add does not support schematics. You can try using a different version of the package or contact the package author to add ng-add support.
error Command failed with exit code 1.

🌍 Your Environment

Angular CLI: 9.0.3
Node: 12.13.0
OS: darwin x64

Angular: 9.0.2
... animations, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... router
Ivy Workspace: Yes

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.900.3
@angular-devkit/build-angular     0.900.3
@angular-devkit/build-optimizer   0.900.3
@angular-devkit/build-webpack     0.900.3
@angular-devkit/core              9.0.3
@angular-devkit/schematics        9.0.3
@angular/cli                      9.0.3
@ngtools/webpack                  9.0.3
@schematics/angular               9.0.3
@schematics/update                0.900.3
rxjs                              6.5.4
typescript                        3.7.5
webpack                           4.41.2

Anything else relevant?

Hi @lacolaco, the problem seems to be that @ngrx/store version 8 is not compatible with Angular 9. This is due that the peerDependencies for this package are the following;
"peerDependencies": {
   @angular/core": "^8.0.0",
  "rxjs": "^6.4.0"
},

ng-add will try to find a compatible version and version 1.2.1 has no peerDependencies which is making it the package valid to be used.
//cc @brandonroberts

@alan-agius4 Thank you for clarifying. IMO, peerDependencies is not an absolute restriction always. In fact Angular CLI allows TypeScript version mismatch with a warning.
I think the CLI always should try installing the @latest version because developers perhaps expect ng add is a wrapper of npm install or yarn add. Following their default version resolution is intuitive.
If it is not compatible with peerDeps, the CLI can warn.

Facing a similar issue with a current project on v9. Any update?

@Brian611 This is a problem with the @ngrx/store package.  It does not currently support Angular version 9.

For others trying to use Angular v9 with NgRx v8, use the @latest tag when using ng add to get the latest released version according to npm. The Angular CLI tries to figure out the version to install, taking into account the peer dependencies. This is why NgRx 1.2.1 is being chosen without using the @latest tag. This issue is not present if you are installing NgRx v9 after it goes final.
ng add @ngrx/store@latest

@brandonroberts  Yes i noticed and upgraded but would've hoped this would be a fix rather than waiting on ngrx v9 cause now we running prod project on an rc version of ngrx which is not ideal.

@Brian611 there's nothing preventing you from running NgRx v8 with Angular v9. There's only the peer dependency warning. NgRx v9 will be out soon

@brandonroberts  ohk cool.

Closing as this is working as expected.


Test CI

As title

https://github.com/dask/dask-ml/pull/856 got this partway fixed.

Thanks @TomAugspurger


STYLE: Fix file without end line

Picked up by pre-commit.

Note that this file was created by calling pandas DataFrame's to_json method in  src\histopathology\scripts\aggregate_metrics_crossvalidation.py and future JSON files created by this script will face the same problem. If this is out of scope of this PR, we can create a separate issue

Note that this file was created by calling pandas DataFrame's to_json method in src\histopathology\scripts\aggregate_metrics_crossvalidation.py and future JSON files created by this script will face the same problem. If this is out of scope of this PR, we can create a separate issue

I think it's not out of scope. I'm looking into saving a data frame as JSON including the last line. If I figure it out, I'll add it here and re-request your review.

I shouldn't have enabled auto-merging for this. I'll have to bother you with a new PR in a minute. Sorry about that.


driver: Use shard-aware driver by default

Classpath contained Datastax driver and used it by default, meanwhile maven downloaded shard-aware driver but didn't use it.
It might have been misleading, so this commit replaces
./lib/cassandra-driver-core-3.5.1-shaded.jar
with
./lib/scylla-driver-core-3.7.1-scylla-2-shaded.jar
(Note: both files are similar, e.g. my git client incorrectly considers this change as a "rename")
Refs scylladb/scylla#5406

@penberg @avikivity Could you merge this, please?


Relationship lines not showing on ER diagram

Description
Multiple team members are unable to open the ER diagram and observe the relationship lines.  The table entities are shown but fact tables are now highlighted until one goes to the individual table ER diagram.  I am attaching screenshots for both the schema and individual table ER diagrams.

DBeaver Version
Community Edition Version <IP_ADDRESS>402181738
Operating System
macOS Sonoma Version 14.2.1
Database and driver
DB2 for LUW
Steps to reproduce

I select the schema or table.
I select the ER diagram tab.

Additional context
On initial download of DBeaver in December, the ER diagram correctly showed the relationship lines.  Multiple updates have since been applied, but it is unclear at what point the current behavior began.

Hello @trdinger
Could you please provide the log files? You can find instructions here: https://dbeaver.com/docs/wiki/Log-files/. Thank you.

dbeaver-debug-1708435893565.log
dbeaver-debug.log

Thanks for the report!
I can reproduce it.


Passing of Lists and Arrays

Hi Jacques, framework working well for me thanks - Small update for the DefaultValueConverter to handle Lists & Arrays :
            if (origValue is JArray)
            {
                destValue = ((JArray)origValue).ToObject(destType);
                return true;
            }

@mrwoodo Thanks for suggestion I'll integrate this.


Allow the database connection protocol to be overridden

We're trying to use this chart to connect our Confluence installation to a MS SQL database, and encountered the following issue :

the official Atlassian image expects ATL_DB_TYPE to be set to mssql (see /opt/atlassian/etc/confluence.cfg.xml.j2)
but the ATL_JDBC_URL should start with jdbc:sqlserver to be JDBC-compliant

This PR solves this issue by allowing an override of the protocol used in the JDBC URL.

Hello @LoicAG ,
First of all, thanks for the PR, collaboration is always appreciated.
Before doing a merge, could you try replacing the value of databaseConnection.type from postgresql to sqlserver?
That should already provide you a JDBC-compliant string because the jdbc prefix is hard-coded.
ATL_JDBC_URL: "jdbc:{{ .Values.databaseConnection.type }}://{{ .Values.databaseConnection.host }}:{{ .Values.databaseConnection.port }}/{{ .Values.databaseConnection.database }}"
Cheers,

Hey @javimox, thanks for the quick reply :)
Actually, setting databaseConnection.type to sqlserver was the first thing I tried; but it fails because the configuration template (/opt/atlassian/etc/confluence.cfg.xml.j2) expects it to be mssql  in order to set the driver class and dialect.
See the last 2 lines in this excerpt:
...
   {% if atl_jdbc_url is defined %}
      {% set databases =  {
           "mysql": ["com.mysql.jdbc.Driver", "MySQLDialect"],
           "postgresql": ["org.postgresql.Driver", "PostgreSQLDialect"],
           "mssql": ["com.microsoft.sqlserver.jdbc.SQLServerDriver", "SQLServerDialect"],
           "oracle12c": ["oracle.jdbc.driver.OracleDriver", "OracleDialect"]
          } %}
      <property name="confluence.database.choice">{{ atl_db_type }}</property>

      <property name="hibernate.connection.url">{{ atl_jdbc_url }}</property>
      <property name="hibernate.connection.username">{{ atl_jdbc_user }}</property>
      <property name="hibernate.connection.password">{{ atl_jdbc_password }}</property>
      <property name="hibernate.connection.driver_class">{{ databases[atl_db_type][0] }}</property>
      <property name="hibernate.dialect">com.atlassian.confluence.impl.hibernate.dialect.{{ databases[atl_db_type][1] }}</property>
...

So using the same value to set the type and to build the JDBC URL doesn't work.
After further testing, I've also established that the JDBC URL format for SQL Server is actually quite different than the one for Postgres, and even setting the type to mssql and the protocol to jdbc:serversql wasn't enough; what did work was re-building the URL entirely, in this format:
dbc:sqlserver://sql.server.host:1234;database=mydb;user=myuser;password=mypassword

If it's alright with you, I'll close this PR, and open a new one that allows over-riding the entire JDBC URL, how does that sound?

Here's the link to the official documentation regarding the format of the JDBC URL for SQL Server: https://docs.microsoft.com/en-us/sql/connect/jdbc/building-the-connection-url

Hey @javimox, thanks for the quick reply :)
Actually, setting databaseConnection.type to sqlserver was the first thing I tried; but it fails because the configuration template (/opt/atlassian/etc/confluence.cfg.xml.j2) expects it to be mssql in order to set the driver class and dialect.
See the last 2 lines in this excerpt:
...
   {% if atl_jdbc_url is defined %}
      {% set databases =  {
           "mysql": ["com.mysql.jdbc.Driver", "MySQLDialect"],
           "postgresql": ["org.postgresql.Driver", "PostgreSQLDialect"],
           "mssql": ["com.microsoft.sqlserver.jdbc.SQLServerDriver", "SQLServerDialect"],
           "oracle12c": ["oracle.jdbc.driver.OracleDriver", "OracleDialect"]
          } %}
      <property name="confluence.database.choice">{{ atl_db_type }}</property>

      <property name="hibernate.connection.url">{{ atl_jdbc_url }}</property>
      <property name="hibernate.connection.username">{{ atl_jdbc_user }}</property>
      <property name="hibernate.connection.password">{{ atl_jdbc_password }}</property>
      <property name="hibernate.connection.driver_class">{{ databases[atl_db_type][0] }}</property>
      <property name="hibernate.dialect">com.atlassian.confluence.impl.hibernate.dialect.{{ databases[atl_db_type][1] }}</property>
...

So using the same value to set the type and to build the JDBC URL doesn't work.
After further testing, I've also established that the JDBC URL format for SQL Server is actually quite different than the one for Postgres, and even setting the type to mssql and the protocol to jdbc:serversql wasn't enough; what did work was re-building the URL entirely, in this format:
jdbc:sqlserver://sql.server.host:1234;database=mydb;user=myuser;password=mypassword

If it's alright with you, I'll close this PR, and open a new one that allows over-riding the entire JDBC URL, how does that sound?

I see what you mean. Closing this PR and opening a new one sounds good.
Thanks @LoicAG


Fixes github #1572

The InChI parser has a number of special cases to try and get the molecule more "RDKit like". This makes some small modifications to one of them in order to fix the handling of the substructure CN(=O)=N(=O)C. (this is something that the InChI code produces)

This looks innocuous, but I'm always hesitant when making parsing changes, should we run it on a larger test or is what we have complete?
btw - do we have an example of a proper zwitterion with the new changes?
Old version:
>>> m = Chem.MolFromSmiles("C(C(C(=O)[O-])[NH3+])O")
>>> Chem.MolToSmiles(MolFromInchi(MolToInchi(m)))
[11:55:42] WARNING: Proton(s) added/removed; Omitted undefined stereo
'NC(CO)C(=O)O'

These changes (should) only apply to the very specific case of the substructure N(=O)=N and the change is to avoid converting that to N(=O)-[N-] (which was something added a long time ago to handle a particular InChI pathology). There are a good number of tests that are run and this change actually helped with round-tripping.
Given that this is parsing InChI, which one really shouldn't do anyway, I'm not particularly concerned about it.


Support multiple service names

In application servers, it is possible to deploy multiple applications. Currently, the agent only supports one service name per JVM.
This should add the ability to configure one service name per javax.servlet.ServletContext#getContextPath.

Blocked by https://github.com/elastic/apm-server/issues/1175

This is not blocked anymore https://github.com/elastic/apm-agent-java/issues/335 is done.

Is there an ETA for this? We would appreciate this as it makes our task a lot easier.

@wolframhaussig I'm currently working on this so it won't be too long probably 🙂
I assume you want this so that you can have different service names for different applications deployed to the same application server?
The way I have implemented this is that it takes the display-name from the web.xml or the context path. If the display name is not set and the application is mapped to the root context, the service name will be the name of the application server (for example tomcat-application).

@felixbarny
Partially: We have multiple tomcat installations with n applications on each. This would be resolved if I understand you right.
The other thing is that we have webservices in an Axis2 container. As I understand you these webservices would all show as "Apache-Axis2" as this is defined in the Axis2 web.xml, right? In this case I would be happy if it was possible to manully match the application name to the Url...

we have webservices in an Axis2 container

Could you specify what you mean by that? Do you mean the Axis2 standalone server?

As I understand you these webservices would all show as "Apache-Axis2" as this is defined in the Axis2 web.xml, right?

Are all of these webservices packaged in the same war file? How would you differ between them? Does it contain multiple Axis2 servlets which are mapped to different URLs?
Another thing to note here is that general JVM metrics like GC statistics will use the service name configured in service.name or the default name.

I meant the Axis2 webApp. Our structure is this:
Tomcat

WebApp 1
WebApp 2
WebApp N
Axis2

In the Axis2 webApp there is a directory WEB-INF/services- In this directory are over 60 .aar files.
If the url mappings were possible I could do something like this:
/axis2/services/ws1 => ws1
/axis2/services/ws2 => ws2
Currently I always see AxisServlet#doPost as name. Of course it would be cool if the apm agent would recognise the service.name from the Axis2 services...

Would it be an option for you to enable use_path_as_transaction_name so that the transactions have different names?

That would be fine for me.
In the documentation is a note:

If set to true, transaction names of unsupported Servlet API-based frameworks will be in the form of $method $path instead of just $method.

So this would currently not work as the Axis2 controller is already recognised, right?

So this would currently not work as the Axis2 controller is already recognised, right?

Hmm, yeah, that's true. Forgot about that.
Another approach could be to set the transaction name in a custom servlet filter manually: https://www.elastic.co/guide/en/apm/agent/java/current/public-api.html#api-set-name

I will leave it for now as I am not happy to modify hundreds of web applications/web services. As a fallback I might reconfigure the apm server to logstash output and do the mapping from url to transaction there.


Update lsmod to 1.0.0

This is for issue #152 to handle a null exception if no node modules are loaded (required).

Thanks. :)


Fix bug in setCurrentPageInBounds

After the change in #161 the setCurrentPageInBounds needed to be updated not to change the currentPage to the last one on slide mode.

Fix the index.js on play to display different images on the samples.

Fix index.js on in play, bug on the navigateTo examples.

Hey @kulaone! Thanks for fixing this I didn't even notice until you made the PR. Everything's good to go but we've got another 1.5K line diff in the package-lock. Weird that it keeps happening, I wonder if something is up with the package-lock in mainline? If you don't mind, lets fix it in the PR again but I'll investigate and see if something is up on my end

@quinnlangille , I think the problem is in my NPM configuration. I need to work on it. sorry.

Hmm ok no worries! I'll merge this in now, thanks again - this PR and your last have been a huge help 👍


Adapt Milvus2.2 Channel checkpoint

Resolves #64
/kind enhancement
Signed-off-by: Congqi Xia<EMAIL_ADDRESS>

/lgtm


When VW password is changed the program gets your account locked

When VW password is changed program gets your account locked for too many failed login attempts.
Steps to reproduce:

Stop docker instance
Change the password on VW site
Restart docker
VWsFriend notices there is no token
and logins
gets login failed
moves back to step 4.

Desired action:

Stop docker instance
Change the password on VW site
Restart docker
VWsFriend notices there is no token
and logins
gets login failed
prints out error message and ends retries

The problem is that the account will still be temporarily locked in the "desired" case because the docker ends and is automatically restarted. It would be good to find out how long the "allowed" time between two wrong passwords attempts can be. Then I can just make the retry time longer in that case.

Mine was locked for over 12 hours without eve trying to login during that time. I had to use reset password to get VW login working again.

I actually run into the same issue and didn't suspect Home Assistant or the We Connect ID integration to be the problem first (but then found this issue). The ID 3 is register to my wife and she had requested to create a new password at volkswagen.de (which after the new password is set, logs you in directly, so one does not notice any issue right away). However, afterwards when she tried to log in with the new password, the account got locked because of too many wrong passwords and she could not log in again. This was caused by the We Connect ID integration trying to log in with the old password in the background. So even with the new and correct password a login on the website actually fails because of too many failed login attempts with the password used by the We Connect ID integration.

One can then request a new password again and will then be logged in again, but after a while resetting passwords, the account locks further because of too many password reset attempts:

To solve this, I temporarily removed the We Connect ID integration from HA, restarted HA, waited approx. 12h reinstalled the integration and was then able to login again with the most resent password via the integration and the website. A bit of a dance, but I fully get that this is not easy to circumvent this so nothing to complain or fix here really, just wanted to document as well what can happen when you change the password on volkswagen.de and what you then need to do to get back into a stable state.
Thanks for the awesome work @tillsteinbach!


Carthage can't create Realm.framework

I'm using Carthage 0.9.1 and am trying to update to Realm 0.96.0 using sudo carthage update. However after cloning my dependencies Realm results in a crash.
It's randomly either
Failed to write to /Users/christophermuehl/Code/MSUapp/Carthage/Build/iOS/Realm.framework: Error Domain=NSCocoaErrorDomain Code=516 "“strip-frameworks.sh” couldn’t be copied to “Realm.framework” because an item with the same name already exists." UserInfo={NSSourceFilePathErrorKey=/private/var/folders/zz/zyxvpxvq6csfxvn_n0000000000000/T/carthage-archive.4vDLQN/Carthage/Build/watchOS/Realm.framework/strip-frameworks.sh, NSUserStringVariant=(
    Copy
), NSDestinationFilePath=/Users/christophermuehl/Code/MSUapp/Carthage/Build/iOS/Realm.framework/strip-frameworks.sh, NSFilePath=/private/var/folders/zz/zyxvpxvq6csfxvn_n0000000000000/T/carthage-archive.4vDLQN/Carthage/Build/watchOS/Realm.framework/strip-frameworks.sh, NSUnderlyingError=0x7ff5214b2970 {Error Domain=NSPOSIXErrorDomain Code=17 "File exists"}}

or
Failed to write to /Users/christophermuehl/Code/MSUapp/Carthage/Build/iOS/Realm.framework: Error Domain=NSCocoaErrorDomain Code=513 "“Realm.framework” couldn’t be removed because you don’t have permission to access it." UserInfo={NSFilePath=/Users/christophermuehl/Code/MSUapp/Carthage/Build/iOS/Realm.framework, NSUserStringVariant=(
    Remove
), NSUnderlyingError=0x7f8b8866d7b0 {Error Domain=NSPOSIXErrorDomain Code=66 "Directory not empty"}}

@Padarom Carthage 0.9.2 or 0.9.3 will probably fix this issue. Please update your Carthage to the latest version. Carthage 0.9.2 and 0.9.3 cannot be installed via Homebrew. So you build Carthage from source or download and run the Carthage.pkg from latest release https://github.com/Carthage/Carthage/releases

Thanks @kishikawakatsumi! Carthage 0.9.3 worked! :)


[CSE-204] Add maybe tx inputs

See https://issues.serokell.io/issue/CSE-204
This PR includes few updates of README of Explorers frontend, which are not part of CSE-204, but just nice to have.
How to test?

Build and run Explorer's frontend as described here: https://github.com/input-output-hk/cardano-sl/tree/master/explorer/frontend#build-in-development-mode

Run Explorer's mock server

$ cardano-sl » stack exec cardano-explorer-mock

Open http://localhost:3100/tx/b29fa17156275a8589857376bfaeeef47f1846f82ea492a808e5c6155b450e02 and you should see something like this (note that we have "--" for Nothing values of tx inputs now)

Thanks for fixing the README.
I run it, seems there were some issues with CSS (look at the image), otherwise code looks good.
Linux Mint Google Chrome Version 60.0.3112.90 (Official Build) (64-bit)
System:    Host: ksaric-H61M-S2V-B3 Kernel: 4.4.0-53-generic x86_64 (64 bit gcc: 5.4.0)
Desktop: Cinnamon 3.4.4 (Gtk 2.24.30) dm: mdm Distro: Linux Mint 18.2 Sonya
Graphics:  Card: NVIDIA Device 128b bus-ID: 01:00.0 chip-ID: 10de:128b
Display Server: X.Org 1.18.4 drivers: nvidia (unloaded: fbdev,vesa,nouveau)
Resolution<EMAIL_ADDRESS>GLX Renderer: GeForce GT 710/PCIe/SSE2 GLX Version: 4.5.0 NVIDIA 375.39 Direct Rendering: Yes
Also, seems there were some issues with tests, restarted it - Test suite cardano-test: FAIL.

@ksaric What's your screen resolution? Does your browser zoom this page? I can't reproduce the CSS issue you mentioned. Could you try it again or test it with other browser?

@sectore The resolution is written above the image, with the additional details. It's not zoomed in, I'll try it out soon and report.

@sectore Works ok now. Weird. There is some strange behavior there, tried zooming and it's responsive (as opposed to the image above). 1 in a 1000? :confused:


use ParsedLoci in loadIndexedBam

I recently factored a small library for parsing string-representations of (possibly open-ended, e.g. chr1, chr1:100-) loci ranges out of guacamole and over to https://github.com/hammerlab/genomic-loci, because I was preparing to use it across a few of my projects, and in particular wanted it plus adam-core's loadIndexedBam functionality, which this PR adds.
@fnothaft you mentioned there is some similar loci-parsing code laying around somewhere in BDG, lmk if I should find/consider it instead / in addition to this.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/1620/Build result: FAILURE[...truncated 38 lines...]Triggering ADAM-prb ? 2.6.0,2.11,1.3.1,centosTriggering ADAM-prb ? 2.3.0,2.11,1.4.1,centosTriggering ADAM-prb ? 2.6.0,2.11,2.0.0,centosTriggering ADAM-prb ? 2.3.0,2.10,1.4.1,centosTriggering ADAM-prb ? 2.6.0,2.11,1.6.1,centosADAM-prb ? 2.3.0,2.11,1.5.2,centos completed with result SUCCESSADAM-prb ? 2.3.0,2.10,1.5.2,centos completed with result SUCCESSADAM-prb ? 2.3.0,2.10,2.0.0,centos completed with result SUCCESSADAM-prb ? 2.6.0,2.10,1.4.1,centos completed with result SUCCESSADAM-prb ? 2.6.0,2.10,1.6.1,centos completed with result SUCCESSADAM-prb ? 2.6.0,2.10,2.0.0,centos completed with result SUCCESSADAM-prb ? 2.6.0,2.11,1.4.1,centos completed with result SUCCESSADAM-prb ? 2.3.0,2.10,1.3.1,centos completed with result SUCCESSADAM-prb ? 2.6.0,2.10,1.3.1,centos completed with result SUCCESSADAM-prb ? 2.3.0,2.11,1.6.1,centos completed with result SUCCESSADAM-prb ? 2.3.0,2.11,1.3.1,centos completed with result SUCCESSADAM-prb ? 2.3.0,2.11,2.0.0,centos completed with result SUCCESSADAM-prb ? 2.3.0,2.10,1.6.1,centos completed with result SUCCESSADAM-prb ? 2.6.0,2.11,1.3.1,centos completed with result SUCCESSADAM-prb ? 2.3.0,2.11,1.4.1,centos completed with result SUCCESSADAM-prb ? 2.6.0,2.11,2.0.0,centos completed with result FAILUREADAM-prb ? 2.3.0,2.10,1.4.1,centos completed with result SUCCESSADAM-prb ? 2.6.0,2.11,1.6.1,centos completed with result SUCCESSNotifying endpoint 'HTTP:https://webhooks.gitter.im/e/ac8bb6e9f53357bc8aa8'
Test FAILed.

Jenkins, retest this please.
SPARK_VERSION=2.0.0 SCALAVER=2.11 HADOOP_VERSION=2.6.0 ./scripts/jenkins-test

passes for me locally. In particular, the error in the logs:
[ERROR] Failed to execute goal on project adam-core-spark2_2.11: Could not resolve dependencies for project org.bdgenomics.adam:adam-core-spark2_2.11:jar:0.20.1-SNAPSHOT: The following artifacts could not be resolved: org.spire-math:spire-macros_2.11:jar:0.11.0, org.typelevel:machinist_2.11:jar:0.4.1: Could not find artifact org.spire-math:spire-macros_2.11:jar:0.11.0 -> [Help 1]

seems erroneous, as org.spire-math:spire-macros_2.11:jar:0.11.0 exists on Maven Central.

Open ended range support in was added in https://github.com/bigdatagenomics/adam/pull/1252.

Cool, gtk; the ParsedLoci class I'm using here comes with some facilities for parsing to/from strings/files that I think are still useful for what I am after here, fwiw.

While I am supportive of the functionality proposed here, I would have to -1 it as is with the new dependency.  For better or worse, we need to cross-build for Spark 1.x/2.x and Scala 2.10/2.11 and so need upstream dependencies to do the same.
Would there be much of a performance hit to transform
ParsedLoci → Iterable[ReferenceRegion] → Iterable[LocatableReferenceRegion]
as compared to
ParsedLoci → LociSet → lociSet.toHtsJDKIntervals?

The genomic-loci library is published for Scala 2.1[01] and I'm not sure uses anything that would differ between Spark 1 and 2 but I'll dual-publish on that axis as well if I find that that's the case.
As it currently stands it needs some guava-shading that I haven't worked out yet to run happily inside a Spark app so this PR is not ready to merge, closing for now.


Add radius attribute on progress bar

This PR add a new attribute to set radius for progress bar on drawing it.

Was just looking for this! Any chance this will be implemented in the future?


Bssayla/issue18

add demo video for resume data extraction
add demo video for job fitting
add instruction to install models using ollama

Done


Fields explanation

Can you briefly explain the fields meaning?
[t=1619610289] [BBs=1567] [seed=795] [fc=1591] [fcps=13] [cur_loop=105957] [mut_avg=2] [cov_avg=59723] [corpus=2]
Also how a "healthy" fuzzing output should look like?
I assume corpus increasing etc?
Thanks,

Ah, good question. It's something I could put into the Readme...

t: the current timestamp
BBs: basic blocks that were covered in total
seed: the current seed
fc: total fuzz cases
fcps: fuzz cases per second
cur_loop: time it took to execute the whole fuzzing loop in µs (includes mutation of input, sending data to target, collecting coverage, and evaluating whether new coverage was discovered)
mut_avg: average time it took to mutate the inputs (in µs)
cov_avg: average time it took to obtain coverage (in µs)
corpus: amount of files currently in the corpus

In general, this really depends on the target, but yeah, fcps should ideally be fairly constant. cur_loop, mut_avg, and cov_avg should be as low as possible. BBs should be increasing over time, but only if the corpus also increases.

Thank you


Droppable issue and unable to prevent the default insert on "droppable:over"

Please use this template to help contributors get a detailed description of the issue or feature.

For support questions, please use stackoverflow. This repository's issues are reserved for feature requests and bug reports.
Describe the bug or feature
Bug: When the droppable dropped outside the drop-zone we see error in the console ERROR DOMException: Failed to execute 'insertBefore' on 'Node': The new child element contains the parent.
Feature / How to: As you can see the attached GIF the draggable item get auto insert into the drop-zone but I don't want it to auto insert (and remove on out). Because I should have full control on what to insert or whether I should allow to insert or not. How do I prevent the default insert behavior?

3. If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem
Please see the attached GIF for better understanding and also the code I have written: (Angular 5 directive)
import { Directive, ElementRef, AfterViewInit, Input, OnDestroy, NgZone, ChangeDetectorRef } from '@angular/core';
import { Droppable } from '@shopify/draggable';
import { LayoutManager } from '../charts/layout.manager';

@Directive({
    selector: '[dnd]',
})
export class DndDirective implements AfterViewInit {
    private _droppable: Droppable;

    @Input() dndDraggable: any;
    @Input() dndSelector: string;

    constructor(private elementRef: ElementRef,
        private _zone: NgZone,
        private _cdr: ChangeDetectorRef,
        private _lm: LayoutManager) { }

    ngAfterViewInit() {
        const root: HTMLElement = this.elementRef.nativeElement.querySelector('.portlets-container');
        const droppable = this._droppable = new Droppable(this.elementRef.nativeElement, {
            droppable: () => {
                const ret = root.firstElementChild ? root.querySelectorAll('.drop-zone') : [root]; //We must return array of elements otherwise it will not work.
                return ret;
            },
            appendTo: document.body,
            mirror: {
                constrainDimensions: true
            },
            scrollable: {
                speed: 6,
                sensitivity: 12
            }
        });

        droppable.on('droppable:over', () => console.log('droppable:over'));
        droppable.on('droppable:out', () => console.log('droppable:out'));
        droppable.on('drag:start', () => console.log('drag:start'));
        droppable.on('drag:stop', () => console.log('drag:stop'));
    }

    ngOnDestroy() {
        this._droppable.destroy();
    }
}

4. Please tell us about your environment:

Library version: 1.0.0-beta.5
Browsers: [all | Chrome 64
**Tech stack:**TypeScript 2.7 | ES6 | Angular 5

I'm looking for this functionality in the project I'm currently working on as well.  I am not sure if you are envisioning when
droppable.on('droppable:dropped', (event) => {   if (/* some logic */) {     event.cancel(); // prevents drop   } })
executes that the dragged object gets reverted to the original container or not.
For example:
Object 1 in container A gets dropped to container B.
Some custom logic is executed
Object 1 gets put back in container A.
Furthermore, would you roll back the animation process in such a scenario?  Would ordering be maintained if there were other draggable elements that were in that original source container as well?

@tsov Honestly this lib is going in right direction and also I recommended to Angular Material team.
#146 solves one use-case (drop cancel)
2nd use-case: Should provide option to prevent the droppable getting inserted (by default now see in the gif) into dropzone while the drag is in progress and the droppable is over the dropzone.
The 2nd use-case (see the gif in this comment), when the droppable is over on the dropzone it should get highlight, css classes will be added to all dropzone when the drag start (to indicate where all can the droppable item can be dropped/accepted) and another class will be added when drag is over the dropzone to highlight the active target. If you see the droppable is just tell what kind of chart but after dropped we should be able to control what to insert and where.

Today I'm using interactjs but I have to deal many major performance issues and unexpected behaviors. I have patched it to make it work for my needs but I don't want to go in that path. To my Luck I found this lib but I have to test and request (or contribute as plugin) if any missing feature

@tsov @manju-reddys +1 for adding option to prevent default or customize the insert behavior for droppable:over event, when you are building some designer or editor applications, user drag and drop a toolbar icon or button, obviously you don't want that icon or button moved into the edit area, instead you just want to know which icon or button is dropped and insert corresponding content for that icon or button.
Another issue about the droppable:over insert is that: it changes the order of the original items. See example here: https://codepen.io/pachecoder/pen/xPKNmg, two list, first one with 5 items in it, second one is empty, drag the first one item from the first list over the second list(but don't drop it yet), the dragged item is still in the first list, but it just become the last one!!! This is really strange! Even drag and move the item within the first list will change the order of items randomly(obviously this is not a sortable list).

@tsov Any plans to address this soon?


Different images resulting into same hash

I'm having a unusual problem in my project where I'm getting the same hash to two different images. Those are the mentioned images:

The project is simply using the hash(BufferedImage image) method from the API and I'm not sure if there are different approaches to do that, but is there any possible solution to this problem? Thanks in advance!

Hash collisions happen by design due to the fact that you map arbitrarily many images to a fixed length hash.
For example using java's default hashcode implementation converts the string Hash to a numeric value of 2241838. Just looking at 4 character strings, any of the following word have the exact same hashcode

Hc5h
ID5h
IBsh
HbTh
ICTh
Hc6I
ID6I
HatI
IBtI
HbUI
ICUI

This is also true for images. A solution would be to use a secondary hash function looking at different features of the image (e.g. average hash and perceptive hash) to confirm your classification. If they agree the image is most likely a duplicate.
The chain algorithm example show how you might approach this:
https://github.com/KilianB/JImageHash/blob/master/src/main/java/com/github/kilianB/examples/ChainAlgorithms.java

Thanks!!! But sadly I might say I'm having a bad time at trying to find a solution to this. Oddly enough, I've tried to use multiple of the available hashing methods, but always kept ending up with the same digits for both of 'em images. Even changing the "bit resolution" didn't worked; What intrigues me the most is the fact that, as you can see from the images uploaded in my prior comment, they are indeed different images. I would love to find a way to show that for the hash methods as well, hahaha. There is any other possibilities in the game? Anyway, thank you very much for the help!

I'm also going to leave here the actual piece of code I'm using to test the similarity between the images! Maybe I'm doing something I can't see right now! Would appreciate some insights!
`@PostMapping(value = "/v1.0/hashTest")
public ResponseEntity getImageHashTest(@RequestParam(name="file") MultipartFile file,
@RequestParam(name="file2") MultipartFile file2){
SingleImageMatcher matcher = new SingleImageMatcher();
Map<String, String> hashMap = new HashMap<>();
    try {
        BufferedImage image = ImageIO.read(file.getInputStream());
        BufferedImage image2 = ImageIO.read(file2.getInputStream());

        matcher.addHashingAlgorithm(new AverageHash(8), 0.4);
        matcher.addHashingAlgorithm(new AverageHash(32), 0.4);
        matcher.addHashingAlgorithm(new AverageHash(64), 0.4);

        matcher.addHashingAlgorithm(new PerceptiveHash(32), 0.4);
        matcher.addHashingAlgorithm(new PerceptiveHash(64), 0.4);

        matcher.addHashingAlgorithm(new MedianHash(32), 0.4);
        matcher.addHashingAlgorithm(new MedianHash(64), 0.4);

        matcher.addHashingAlgorithm(new DifferenceHash(64, DifferenceHash.Precision.Simple), 0.4);
        matcher.addHashingAlgorithm(new DifferenceHash(32, DifferenceHash.Precision.Triple), 0.4);

        if(matcher.checkSimilarity(image, image2))
            hashMap.put("similarity", "yes");
        else
            hashMap.put("similarity", "no");

        return ResponseEntity.ok(hashMap);
    } catch (IOException e) {
        e.printStackTrace();
    }

    return ResponseEntity.noContent().build();
}`

I did some testing and indeed those images will result in the same hash no matter what you try. Upon further investigation the issue arises due to the alpha channel. The black parts of the image are solid black, the white parts simply have an opacity of 0.
As far as the program is aware, computing the luminosity values only takes the rgb values into account which are the same for each pixel.
Are there any guidelines how transparency should be regarded when calculating Y in the YCbCr color model? I assume that for this trivial case an alpha of 0 can
be assumed as white, but this isn't entirely correct for every single use case.
For now an ugly work around would be to replace the opaque pixels with a white color until I can figure out how to correctly compute luminosity. (Is there a formula how to handle alpha? Always assume white?)

Yes, choosing a different hash method will not make a difference since the issue resides at the hash precalculation step.
I see where you are coming from and this indeed is an issue. Semantically there isn't a valid solution I am afraid. We never know what color a missing pixel (invisible) will be.
More often than not those pixels are displayed as white, as seen in the above images. I will add an option to let people choose how to handle transparency. This will fix your current issue.

That's awesome! I was actually going to come back with this exactly answer! Going through a little search, I've noticed how much the transparency affected the hash calculation by the API and ended up with the idea of simply modifying the original BufferedImage with an white background and then generating an hash over it. The only problem I see is when the actual image is an white png icon. Maybe we could iterate over this solution to get to a good place.

Everything is pretty much implemented I just need to do some unit tests in order to ensure I didn't mess up anything else.
The heavy lifting is done at the utility code repository.
From now on you can define:
   HashingAlgorithm aHasher = new AverageHash(64);
   //Define how to handle opaque pixel
   double alphaThreshold = 0;
   aHasher.setOpaqueHandling(Color.white,alphaThreshold);

   //Proceed as normal`

Will this suit you or do you have any other ideas? By default I will retain the old behavior to not break backwards compatibility.

Ok, if I got it right, the hasher will by default treat the image with a white background while also letting me choose another color/threshold if demanded, right? If so, it is amazing! It solves the problem as we can get even more preciser hashes! About the black and white w/ no background: what if you calculate the bg color over the luminance of the predominant image color? Maybe so we can avoid as much as possible making the user set the color manually!

While refactoring the utility code I changed a few design decisions which takes a while longer than expected. I really wanted to get the new version released this night but sadly it will take a tiny bit.

Cool! Man, I would like to report a new ununsual case after the solution of adding a white bg to transparent backgrounds... For some reason my code resulted to generate equal codes (once again) when trying to hash those two following images using new PerceptiveHash(32) (respectively, one being the transparent png and the other being just a regular jpeg):

Would you please try to hash 'em so we can test if the anomaly isn't only at my side?


Bug: next's 'use client' leads to 'Fix objects must not be overlapped in a report'

Describe the bug
Plugin got stuck if it finds 'use client' on top line
Code example
"use client";

import { Input } from "@/components/ui/input";
import React from "react";
import { useEffect, useState } from "react";

const EmailNameChanging = () => {...}

ESLint version
8.42.0
ESLint Plugin Perfectionist version
1.2.0
Additional comments
No response
Validations

[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.

I tried to reproduce the problem with your code, but everything works correctly for me.
Are you sure that this part of the code is the problem?
Can you run the eslint command with the --debug flag to find the problem file? Or perhaps the problem is in another part of that file?

For convenience, I created a project on Stackblitz:
https://stackblitz.com/edit/node-nth3q2?file=.eslintrc.js,src%2Findex.js
You can try to open this page, write some code and run the command pnpm run lint.
I have not been able to reproduce your problem at this time.

It looks like it doesn't connected to 'use client'
I thought it was exactly because of --debug output. Line 1 was stated as a problem.
I've found that two empty lines in import statements creates the problem
import Image from "next/image";
import { useState } from "react";

import React from "react";

 eslint --debug output
  eslint:cli-engine Lint /home/mark/p/gg_frontend/app/profile/settings/password-changed/page.tsx +12ms
  eslint:linter Linting code for /home/mark/p/gg_frontend/app/profile/settings/password-changed/page.tsx (pass 1) +4ms
  eslint:linter Verify +0ms
  eslint:linter With ConfigArray: /home/mark/p/gg_frontend/app/profile/settings/password-changed/page.tsx +0ms
  eslint:linter Parsing: /home/mark/p/gg_frontend/app/profile/settings/password-changed/page.tsx +1ms
  eslint:linter Parsing successful: /home/mark/p/gg_frontend/app/profile/settings/password-changed/page.tsx +2ms
  eslint:linter Scope analysis: /home/mark/p/gg_frontend/app/profile/settings/password-changed/page.tsx +0ms
  eslint:linter Scope analysis successful: /home/mark/p/gg_frontend/app/profile/settings/password-changed/page.tsx +0ms
  eslint:linter An error occurred while traversing +4ms
  eslint:linter Filename: /home/mark/p/gg_frontend/app/profile/settings/password-changed/page.tsx +0ms
  eslint:linter Line: 1 +0ms
  eslint:linter Parser Options: {
  ecmaFeatures: { globalReturn: false, jsx: true },
  sourceType: 'module',
  warnOnUnsupportedTypeScriptVersion: true,
  requireConfigFile: false,
  allowImportExportEverywhere: true,
  babelOptions: {
    presets: [ 'next/babel' ],
    caller: { supportsTopLevelAwait: true }
  },
  ecmaVersion: undefined
} +0ms
  eslint:linter Parser Path<EMAIL_ADDRESS>+0ms
  eslint:linter Settings: {
  react: { version: 'detect' },
  'import/parsers': {
<EMAIL_ADDRESS>[ '.ts', '.mts', '.cts', '.tsx', '.d.ts' ]
  },
  'import/resolver': {
    '/home/mark/p/gg_frontend/node_modules/eslint-import-resolver-node/index.js': { extensions: [Array] },
    '/home/mark/p/gg_frontend/node_modules/eslint-import-resolver-typescript/lib/index.cjs': { alwaysTryTypes: true }
  }
} +0ms

Oops! Something went wrong! :(

ESLint: 8.42.0

AssertionError [ERR_ASSERTION]: Fix objects must not be overlapped in a report.
    at mergeFixes (/home/mark/p/gg_frontend/node_modules/eslint/lib/linter/report-translator.js:152:9)
    at normalizeFixes (/home/mark/p/gg_frontend/node_modules/eslint/lib/linter/report-translator.js:182:16)
    at /home/mark/p/gg_frontend/node_modules/eslint/lib/linter/report-translator.js:349:49
    at Object.report (/home/mark/p/gg_frontend/node_modules/eslint/lib/linter/linter.js:1021:41)
    at /home/mark/p/gg_frontend/node_modules/eslint-plugin-perfectionist/dist/index.js:3:644
    at D (/home/mark/p/gg_frontend/node_modules/eslint-plugin-perfectionist/dist/index.js:1:2730)
    at /home/mark/p/gg_frontend/node_modules/eslint-plugin-perfectionist/dist/index.js:3:143
    at Array.forEach (<anonymous>)
    at Program:exit (/home/mark/p/gg_frontend/node_modules/eslint-plugin-perfectionist/dist/index.js:3:131)
    at ruleErrorHandler (/home/mark/p/gg_frontend/node_modules/eslint/lib/linter/linter.js:1050:28)

Can you show me content of /home/mark/p/gg_frontend/app/profile/settings/password-changed/page.tsx file?
Or you can use Stackblitz to figure out the problem. Perhaps the problem is not with the imports.

Here's an example of a code with a empty lines in imports: link

Nice. Thank you. I'll fix it soon.

Fixed in 21bc7a8ef5ea21e308447aba7c8474a1368fb686
Released in v1.2.1


Add Query Editor null checks

It looks like some of the changes I merged for extensibility are triggering these nullrefs.  I'll investigate why the code paths have changed, but these should go in first to avoid triggering exceptions.

The null checks fix a break in master so I'll push them now.


is there a way to clear all the fields if we enter the wrong code?

You could try remove state from component @Sourabhd1164

Is this still in the works as a feature update?


Alignment Search Confuses "Any Alignment" with "any"

When you set no filter for Alignment (any alignment option) all alignments appear, as expected.
When you change the combo to "any" I expected to show only creatures with "any", but it shows all alignments.

This is a bit confusing, but it's accurate.
The idea for the alignment search is to find all monsters which could be that alignment. So if you're searching for "chaotic evil", it will find all monsters which are chaotic evil, as well as all monsters that are "any chaotic" or "any evil", monsters with "any", etc.
This behavior makes more sense when you consider what happens if you search for "any chaotic". In this case, it shows monsters like the Cambion which have "any evil", because a Cambion could be chaotic evil, which matches the search for "any chaotic" monster.
So bringing it back, searching for "any" matches any monster that can have an alignment. In other words, everything except for unaligned monsters.


[MINOR]Simplify boolean expression

What changes were proposed in this pull request?
The comparison between Boolean expression and Boolean literal is redundant.
How was this patch tested?
N/A

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8264/
Test PASSed.

Test build #102732 has started for PR 23884 at commit 312d2c9.

How did you check this? I rungrep -e "== true" -e "== false" -e "!= true" -e "!= false" in all the source code, and I quickly looked over the result and I found at least one more file (SecurityManagerSuite) having this comparison, too.

How did you check this? I rungrep -e "== true" -e "== false" -e "!= true" -e "!= false" in all the source code, and I quickly looked over the result and I found at least one more file (SecurityManagerSuite) having this comparison, too.

I checked it using IntelliJ IDEA---Analyze.
It  seems that === has not been detected

Test build #102732 has finished for PR 23884 at commit 312d2c9.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102732/
Test FAILed.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8269/
Test PASSed.

Test build #102742 has started for PR 23884 at commit 312d2c9.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8273/
Test PASSed.

Merged build finished. Test PASSed.

Test build #102749 has started for PR 23884 at commit 3e9dc53.

Test build #102742 has finished for PR 23884 at commit 312d2c9.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102742/
Test PASSed.

Test build #102749 has finished for PR 23884 at commit 3e9dc53.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102749/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8289/
Test PASSed.

Test build #102775 has started for PR 23884 at commit e278b58.

Test build #102775 has finished for PR 23884 at commit e278b58.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102775/
Test PASSed.

The comparison between Boolean expression and Boolean literal is redundant.

The PR description can be improved.

The comparison between Boolean expression and Boolean literal is redundant.

The PR description can be improved.

Thanks, updated.

Merged to master


Fix/DYN 5219 dismiss notification icon should mark as read

This PR implements a fix for the Notifications-Panel v 0.0.7 regarding to DYN-5219.
Anytime we dismiss a notification it will be mark as read and will be removed from the list.

FYI
@QilongTang
@reddyashish
@RobertGlobant20

LGTM with two comments

@RobertGlobant20 I've removed both comments

Thank you @Enzo707


Extract tool generating invalid parameters file to deploy

After extracting my APIM using the following parameters (real values removed):
{
  "sourceApimName": "removed",
  "destinationApimName": "removed",
  "resourceGroup": "removed",
  "fileFolder": "C:\\Source",
  "paramNamedValue": true,
  "paramApiLoggerId": true,
  "paramLogResourceId": true,
  "baseFileName": "removed"
}

I got this parameters.json file:
{
  "$schema": "https://schema.management.azure.com/schemas/2015-01-01/deploymentTemplate.json#",
  "contentVersion": "<IP_ADDRESS>",
  "parameters": {
    "ApimServiceName": {
      "value": "removed"
    },
    "NamedValues": {
      "value": {
        "removed-prop-name": "removed-value"
      }
    },
    "ApiLoggerId": {
      "value": {
        "my-api-name": {
          "applicationinsights": "resource-address"
        }
      }
    },
    "LoggerResourceId": {
      "value": {
        "azuremonitor": null,
        "PROPNAME": "resource-address"
      }
    }
  },
  "resources": []
}

Later when running my release pipeline on Azure DevOps I've got this error:
Deployment template validation failed: 'The template parameters 'ApiLoggerId, LoggerResourceId' in the parameters file are not valid; they are not present in the original template and can therefore not be provided at deployment time. The only supported parameters for this template are 'ApimServiceName, NamedValues'. Please see https://aka.ms/arm-deploy/#parameter-file for usage details.'.
Does anyone have any idea how to fix it? I've tried changing the schema to 2019-04-01, but got the same error.

What do the parameters sections of your template look like?  For example, here is one without the optional parameters:
"$schema": "https://schema.management.azure.com/schemas/2015-01-01/deploymentTemplate.json#",
 "contentVersion": "<IP_ADDRESS>",
 "parameters": {
   "ApimServiceName": {
     "type": "string"
   }
 }

Yours should have the parameter definitions for NamedValue, ApiLoggerId, and LoggerResourceId defined as well.

I've discovered the issue, although my parameters.json file has 4 params, not all templates references those for 4 params.
So, if my template only references 2 parameters, using a parameters.json file with 4 parameters will rise an error.

This is a general ARM issue, not specifically this toolkit. However, the issues is that the parameter file contains more parameters than the individual template needs.  Very annoying... I am working on a temp solution but it should be resolved in ARM itself but since that may never happen, the Toolkit should ensure that every template it generates has the same parameters as defined in the parameters file.

I have this issue resolved in Powershell as part of the pipeline  sample I have created.
In the Powershell script I have created for deployment, it uses the parameter template as a resource to retrieve the values from but the deployment is done using the -TemplateParameterObject option on the New-AzResourceGroupDeployment function. In my script I create a parameter object based on the defined parameters in the arm template with the values retrieved from the parameter file.

Is it possible to have the Extractor tool generate a separate parameter file per template file?
so:

apim-backends.template.json
apim-backends.params.json
apim-tags.template.json
apim-tags.params.json
etc

The extractor is meant to have a single master template with linked templates for ease of editing.  As such, it's only meant to be deployed as a single entity - not as a split entity.  However, I'll get the engineer to think about this.  It may be that we can't do anything and we simply close this as "by design".

Is it possible to have the Extractor tool generate a separate parameter file per template file?
so:

apim-backends.template.json
apim-backends.params.json
apim-tags.template.json
apim-tags.params.json
etc

It would be very convenient for use in the yaml file:
- task: AzureResourceGroupDeployment@2
      displayName: 'Deploying API Backend'
      inputs:
        azureSubscription: $(azureSubscription)
        location: $(location)
        resourceGroupName: $(resourceGroupName)
        csmFile: 'apim-backends.template.json'
        csmParametersFile: 'apim-backends.params.json'

hey , are you able to deploy apis throgh yaml (Azure Devops / Github) to Azure APIM ? because I am struggling with template errors

You have to copy the ARM templates and parameter files to an Azure Storage location and then use that.  This is normal for a pipeline deployment.

Hello @rodrigoburim.
Please be informed that the ticket was closed, since the changes regarding this issue were merged into the main branch and will be available in future releases.
Changes include:

Generation of parameters file per each resource template
Possibility to redefine the output parameters directory name
If previously the parameter directory were already existing, it will be saved with same name plus datetime of creation for future references and comparison (if applies)


Very high memory usage and crash

TypeScript Version:  2.6.0-dev.20170902 (also happens on earlier versions)
I work at AmCharts, we have a fairly large code base written in TypeScript (~29,315 lines of code), but when compiling with tsc it consumes a few gigabytes of RAM and finally we get this error:
<--- Last few GCs --->

[24391:0x2c63610]    70596 ms: Mark-sweep 1401.0 (1535.3) -> 1401.0 (1535.3) MB, 1770.2 / 0.0 ms  allocation failure GC in old space requested
[24391:0x2c63610]    72459 ms: Mark-sweep 1401.0 (1535.3) -> 1401.0 (1505.3) MB, 1862.6 / 0.0 ms  (+ 0.0 ms in 0 steps since start of marking, biggest step 0.0 ms, walltime since start of marking 1863 ms) last resort
[24391:0x2c63610]    74312 ms: Mark-sweep 1401.0 (1505.3) -> 1401.0 (1505.3) MB, 1852.1 / 0.0 ms  last resort

<--- JS stacktrace --->

==== JS stack trace =========================================

Security context: 0x28731ee29891 <JS Object>
    2: findAncestor [amcharts4dev-0effa4a915dcea363716ff67c2bc2798eeb1d717/node_modules/typescript/lib/tsc.js:~284] [pc=0x38642953cd5f](this=0x24541b5fdfc1 <an Object with map 0x2b2aae5a411>,node=0x272f911b6331 <a Node with map 0x2b2aae4d891>,callback=0x1bd9ab9f80a1 <JS Function (SharedFunctionInfo 0x31ac7bec9ab1)>)
    3: isSymbolInScopeOfMappedTypeParameter(aka isSym...

FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed - JavaScript heap out of memory
 1: node::Abort() [node]
 2: 0x13647ec [node]
 3: v8::Utils::ReportOOMFailure(char const*, bool) [node]
 4: v8::internal::V8::FatalProcessOutOfMemory(char const*, bool) [node]
 5: v8::internal::Factory::NewFixedArray(int, v8::internal::PretenureFlag) [node]
 6: v8::internal::DeoptimizationInputData::New(v8::internal::Isolate*, int, v8::internal::PretenureFlag) [node]
 7: v8::internal::LCodeGenBase::PopulateDeoptimizationData(v8::internal::Handle<v8::internal::Code>) [node]
 8: v8::internal::LChunk::Codegen() [node]
 9: v8::internal::HCompilationJob::FinalizeJobImpl() [node]
10: v8::internal::CompilationJob::FinalizeJob() [node]
11: 0xd9c4fa [node]
12: 0xd9cd42 [node]
13: v8::internal::Compiler::Compile(v8::internal::Handle<v8::internal::JSFunction>, v8::internal::Compiler::ClearExceptionFlag) [node]
14: v8::internal::Runtime_CompileLazy(int, v8::internal::Object**, v8::internal::Isolate*) [node]
15: 0x386428a8437d

This means we don't get any type checking at all, because it crashes with the above error before it finishes compiling. This is seriously hurting our development.
Our code doesn't do anything super crazy with the type system, just a lot of classes and interfaces. However, we do use one particular code pattern which seems to be particularly slow:
interface IFoo {
  foo: number;
}

class Foo {
  _interface: IFoo;

  foo(): this["_interface"] {
    ...
  }
}

interface IBar extends IFoo {
  bar: string;
}

class Bar extends Foo {
  _interface: IBar;
}

It's hard to create a self-contained example because the crash seems to only happen on large projects.
In addition, our code is private. How can I get our code to you so that you can figure out why it's crashing?

It might be related to https://github.com/Microsoft/TypeScript/issues/17968, but I'm not sure.

@Pauan We just merged #18231 which I suspect will fix your issue. Can you give it a try with the next nightly build?

@ahejlsberg Yes, version 2.6.0-dev.20170906 fixes the crash, thank you very much!


field macros

turn field group into macro
turn field description into macro
turn field errors into macro
turn label into macro

Pull Request Test Coverage Report for Build 781915e911ecf43ca7714479e8a55c62a5c392cf-PR-319
Details

27 of 30 (90.0%)  changed or added relevant lines in 6 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.2%) to 85.031%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/doggo/storybook/field_group_builder.ex
6
9
66.67%

Files with Coverage Reduction
New Missed Lines
%

lib/doggo.ex
3
96.5%

Totals

Change from base Build 8aa8a224617d773b691608c5a83fb44a495f0f2a:
-0.2%

Covered Lines:
818

Relevant Lines:
962

💛  - Coveralls


ngtcp2-gnutls: 0.19.1 -> 1.0.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/ngtcp2/ngtcp2/releases.
meta.description for ngtcp2-gnutls is: an effort to implement RFC9000 QUIC protocol.
meta.homepage for ngtcp2-gnutls is: https://github.com/ngtcp2/ngtcp2
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.0.0 with grep in /nix/store/ypgr5k2x4496ap068ziaqlljjwhq1ckx-ngtcp2-1.0.0
found 1.0.0 in filename of file in /nix/store/ypgr5k2x4496ap068ziaqlljjwhq1ckx-ngtcp2-1.0.0

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

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath
knot-dns
knot-resolver
ngtcp2-gnutls

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/fa5e615286e236dee20921e02a1808dcbb45a44d#ngtcp2-gnutls

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

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

3 packages built:

knot-dns
knot-resolver
ngtcp2-gnutls

Maintainer pings
cc @vcunat for testing.

Looking forward to better API stability from now on.


implement response discriminator checker

Codecov Report

Merging #347 (380f597) into main (3762745) will increase coverage by 0.23%.
Report is 1 commits behind head on main.
The diff coverage is 85.24%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main     #347      +/-   ##
==========================================
+ Coverage   79.36%   79.59%   +0.23%
==========================================
  Files         188      191       +3
  Lines        9768    10008     +240
==========================================
+ Hits         7752     7966     +214
- Misses       1725     1731       +6
- Partials      291      311      +20

Flag
Coverage Δ

unittests
79.59% <85.24%> (+0.23%)
:arrow_up:

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

Files Changed
Coverage Δ

checker/check-response-discriminator-updated.go
85.24% <85.24%> (ø)

... and 14 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


[nightly] No event received in iOS when bluetooth disconnected

Describe the Bug
When music is playing through a bluetooth device, and suddenly the device is disconnected (powered-off or similar), Android receives a "paused" state as PlaybackState event, but iOS is not receiving an event at all, even though the music/player is actually paused.
Steps To Reproduce

Use the hook useTrackPlayerEvents including Event.PlaybackState, and log events.
Play music with bluetooth device, then power-off the device.
Music will stop and button will be paused at notification center, Android device will show the 'paused' state as PlaybackState event, but iOS will show nothing.

Code To Reproduce
 const events = [
    Event.PlaybackState,
    ...
  ];

  useTrackPlayerEvents(events, async data => {
    console.log(data);
    ...
  }

Environment Info:
react-native: 0.70.2
react-native-track-player: 3.2.0-4b83020b465d3e14f418e61e9f6043937d9410d5
Android 9.0 real device (API 28)
iPhone real device iOS 15.6.1

I wonder if this might be an underlying iOS bug.. Does it also occur in 3.2.0?

What does calling await TrackPlayer.getPlaybackState() give after the audio paused due to the bluetooth disconnecting?

I wonder if this might be an underlying iOS bug.. Does it also occur in 3.2.0?

I don't remember having the issue before, but to confirm, I just downgraded to 3.2.0 and tested, bug is not present there, both devices receive the "paused" state.

What kind of bluetooth device are you using? Would I be able to replicate using a pair of airpods?

What kind of bluetooth device are you using? Would I be able to replicate using a pair of airpods?

I've seen this with a car bluetooth, and an LG speaker: no event received at all on iOS.
Now I just tested with Airpods (2nd gen) and it's different: when I remove them from my ears, two events are triggered:
(1) RemotePause
(2) PlaybackState is "stopped"

What does calling await TrackPlayer.getPlaybackState() give after the audio paused due to the bluetooth disconnecting?

Just tested this, I'm still getting {"state": "playing"} in this situation (bluetooth device powered-off, music stops, notification button turned to 'play' icon, but no event received in the app)

I am able to replicate this issue using a pair of bluetooth headphones.

The following pr should fix this issue: https://github.com/doublesymmetry/SwiftAudioEx/pull/40
Thanks for reporting it!

Looks like the pr broke some tests – will need to fix those first before landing this.

Fixed in nightly


[Enhancement] Added Testimonial Component for Home Page

Describe your changes
With this PR merged, the Home page will have a Testimonial section that proves the credibility of the website.
Working Demonstration
Video-Tue-May-23-2023-20-21-43.webm
Issue ticket number and link - If Any
This PR closes #172
Checklist before requesting a review

[x] I have performed a self-review of my code.
[x] Starred the repository.
[x] Added My Name to the Contributor's List.
[x] Followed the repository's Contributing Guidelines.
[x] I ran the app and tested it locally to verify that it works as expected.

Thanks for allowing me to contribute to Amupedia. Would be glad if you'd merge this PR. Thanks! @anmode

Hey, @anmode I've made the changes requested. Here is the working demonstration. Please review, Thanks 😄
Video-Wed-May-24-2023-09-58-02.webm

@tusharbansal22 what do you think? :)

Cool... Idk I am getting feeling code is AI generated... isn't @pooranjoyb? haha

Yup, I did worked out with the JSON integration with AI, also referring to Official Docs of SwiperJS. @anmode

@tusharbansal22 what do you think? :)

Aa, gpt models are surely involved. But cant do anything about that.
But instead of testimonials section would prefer to have a similar swiper section with the dev programs the project is associated with

@tusharbansal22 what do you think? :)

Aa, gpt models are surely involved. But cant do anything about that.
But instead of testimonials section would prefer to have a similar swiper section with the dev programs the project is associated with

It's nice idea.... I should have think before. My bad!

Congrats @pooranjoyb

Congrats @pooranjoyb

Most welcome @anmode , would be up for more productive Issues. Thanks.


Need documentation for deploying ingress controller for Rancher v2.5 in hosted Kubernetes (AKS/EKS/GKE)

Need documentation for deploying ingress controller for Rancher v2.5 in hosted Kubernetes (AKS/EKS/GKE) gz#13872

Updated documentation


API for Arena

Are you guys planning on building the same toolset (the Rust bindings, the cargo deploy tools and an example project) for Screeps Arena as well?
If so, and if you would appreciate help with that; what would be some more easy tasks I could take on? I'm not that proficient in Rust yet, but I'm willing to learn and I do have time to work on something like this.

Yup, that's the plan! We've been able to get the library adapted to the new smaller API surface, and have successfully gotten test wasm modules loaded in the arena environment (albeit in a hacky way, encoded in base64 due to the new ES6 wasm import syntax needing extra work to integrate).
The last step for having a 'hello arena world' setup that I can share (and get help getting the API coverage completed) is getting the build tooling working, which I was able to get started on this morning - should have something usable soon!

Ooohh that is really good to read! It sounds as though you are a lot further than I had expected, but still; don't hesitate to poke me if I can help with something.
Thanks for maintaining all of this.

how's progress going on arena?

We're still pending getting the crate released, but closing this out as we've got dedicated repos for arena bindings that are mostly feature complete, and should have a crate published before too long 🎊 : https://github.com/rustyscreeps/screeps-arena-starter-rust and https://github.com/rustyscreeps/screeps-arena-game-api


[Container] - property disableGutters doesn't work if theme styleOverrides have been applied

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
When the Container padding style is overridden, the disableGutters property stops working.
Expected behavior 🤔
The disableGutters property should work even with the style override.
Steps to reproduce 🕹
If the container style have been overridden, the disableGutters property won't work.
// theme.tsx
    ...
    components: {
      MuiContainer: {
        styleOverrides: {
          root: {
            [customBreakpoints.between('xs', 'sm')]: {
              padding: '0 20px',
            },
            [customBreakpoints.up('md')]: {
              padding: '0 90px',
            },
          },
        },
      },
    },
    ...

      <Container disableGutters> // won't remove gutters from 'xs' to 'sm' and above 'md'

As a workaround (must have @mui/material v5.3.0+), you can use callback and do this:
components: {
  MuiContainer: {
    styleOverrides: {
      root: ({ ownerState }) => ({
        ...!ownerState.disableGutters && {
          [customBreakpoints.between('xs', 'sm')]: {
            padding: '0 20px',
          },
          [customBreakpoints.up('md')]: {
            padding: '0 90px',
          },
        },
      }),
    },
  },
},

I mention this as a workaround because this makes customization very confusing. The real fix in my opinion is disableGutters should have higher specificity similar to Mui-disable, ...states. This applies to other components as well (not only disableGutters but any props that overrides existing CSS properties).
cc @oliviertassinari @mnajdova for more opinions.

I am not sure that this is unexpected behavior to be honest. The override should win, that is why it is an override. It is your responsibility to make sure that if you add a override the other props combination still work as expected. I wouldn't be bumping t he specificity to be honest.
It is similar to as if you change the color of a button in a style override. It would have that color even if the color prop is used, because that is the override you've added. You can use the props combination to specify when the style override should be applied.

Interesting. Let's assume that anything specified in styleOverrides can break the props, then what is the proper way to specify a different gutter size for the Container in the theme while still being able to use disableGutter?

@RemyMachado is what's proposed in https://github.com/mui/material-ui/issues/30825#issuecomment-1028793865 not working for you?
Note that we are working on adding CSS variables for customization like this, that would allow us to specify these per component in the long term.
Till then, I think what's proposed in the above comment is the best option. You could also add a CSS selector that would apply the styles if the class .MuiContainer-disableGutters is not applied, if you are on older version that does not support props in the styleOverrides - https://stackblitz.com/edit/react-xajkta?file=demo.tsx
Does this makes sense?


Update go modules

Fixes stolostron/backlog#21596
Signed-off-by: Robin Y Bobbitt<EMAIL_ADDRESS>

/lgtm
I will be doing the part to pull this into idp-mgmt-operator for the next version, 0.3.3


[video_player] Implement caching

Description
This PR adds a way to cache videos played from an Uri. Two new parameters (maxCacheSize and maxFileSize) have been added to the platform method call and the Dart side has been changed to pass these parameters as well.
Related Issues
Resolves flutter/flutter#28094.
Checklist

[X] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[X] All existing and new tests are passing.
[X] I updated/added relevant documentation (doc comments with ///).
[X] The analyzer (flutter analyze) does not report any problems on my PR.
[X] I read and followed the Flutter Style Guide.
[X] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[X] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[X] I updated CHANGELOG.md to add a description of the change.
[X] I signed the CLA.
[X] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[X] No, this is not a breaking change.

@lukepighetti can you sign the google cla please to be able to merge this?

I believe I have signed it in the past

@googlebot I consent.

@999eagle did you end up getting eyes on the uri stuff on iOS to make sure it was setup appropriately? I kept having issues with certain URLs but I never was able to determine if that was a local issue or not.

@999eagle Thank you for your solution. Could you update this branch due master updates please?

@999eagle thank you for your contribution! Unfortunately the structure of the plugin has changed lately to support implementations on more platforms. Mainly the plugin was moved to a new subdirectory and now uses a common platform interface. Adapting your PR to the new structure should be straight forward. If you have questions please ask!
Please also make sure to include tests for the new functionality.

Hello.
We created new PR which fix this PR to speed up the process of merging of this feature

@999eagle Hello.
Can you confirm CLA in new PR?

Hi Would like to ask when would this be merged? really want to have this feature

What's blocking this from being merged?

It appears this PR should be closed for being stale, and the tracking PR for this feature is located here: https://github.com/flutter/plugins/pull/2429#issuecomment-642653144

Closing in favor of #2429

Closing in favor of #2429


Letsencrypt changes broke dehydrated

Dehydrated no longer works. The execution failed with the following problem.
ERROR: Problem connecting to server (get for https://acme-v02.api.letsencrypt.org/directory; curl returned with 23)
EXPECTED value GOT EOF

According to Kerio, changes were made by Let's Encrypt : https://support.kerioconnect.gfi.com/hc/en-us/articles/19547534733202-Troubleshooting-Let-s-Encrypt-Failure-contacting-https-acme-v02-api-letsencrypt-org-directory-rel-index-to-read-header

Have you tried using the latest version from the github repo? As I am not experiencing this issue with the latest version myself. And to confirm that I have had a certificate renew today where as the Keno article was posted 24 days ago.

Yes, I did try the latest version of the master branch (last modified in December 2023).

I tried again today and everything works now, maybe a network problem with my ISP.
Thanks for your reply.


How to use union types with graphql-yoga?

Hi, is there any support for union types yet in graphql-yoga? It seems to need a resolveType function to determine the actual type of an object at runtime. I couldn't find any place refering to supporting this so far.
Taking this opportunity to express my deep admiration for the work you guys at graphcool have been doing. Amazing developer experience across the board since the release of the graphcool framework, and really excited at what the stack is becoming with native gql db + yoga as api gateway 👍.

Is there any example tackling basic union types?

@maticzav is this helpful?:
https://github.com/eliperelman/taskcluster-graphql-server/blob/679237debc15bd055e8448d71fac78c5b2df8a88/src/graphql/Tasks.graphql#L325-L331
https://github.com/eliperelman/taskcluster-graphql-server/blob/679237debc15bd055e8448d71fac78c5b2df8a88/src/resolvers/Tasks.js#L17-L36

@eliperelman this is super useful! Thanks!

@eliperelman would you be open to adding an example via PR demonstrating how to use interfaces + unions?

@schickling I've written a medium post covering Interfaces / Unions with Prisma & Yoga. Let me know which is the best way, if any, for me to help take parts of that and add to the docs. Also if there are any additions, clarifications, errors in my post, happy to correct things.

Thanks a lot for this great blog post @tibotiber! Would be great if you could create a PR adding an example demonstrating how to use interfaces + unions. 🙏