[superset-client] replace misc ajax calls

This PR is one of several that implement the final step 4) discussed in #5772, to refactor misc remaining ajax calls in the following components

AsyncSelect
DashboardTable
TableLoader
utils.js
common.js
Deck.gl multi layer
dashboard template css

@kristw @mistercrunch @graceguo-supercat @michellethomas @conglei
Other ajax PRs

dashboard #5854
explore #5869
charts #5875
sql lab #5896
logging #6133
datasource editor #6134
misc calls (this PR)

Codecov Report

Merging #6135 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #6135   +/-   ##
=======================================
  Coverage   76.91%   76.91%
=======================================
  Files          47       47
  Lines        9362     9362
=======================================
  Hits         7201     7201
  Misses       2161     2161

Continue to review full report at Codecov.

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

Good


French Settings Added To Seeder #3889

BUGS:
Link: https://github.com/bagisto/bagisto/issues/3889

working fine


protoutil an pb packages moved to aserto-protobuf

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.9%) to 14.203%

Totals

Change from base Build<PHONE_NUMBER>:
0.9%

Covered Lines:
271

Relevant Lines:
1908

💛  - Coveralls


Update About.tsx

Changed "about" header title to be more descriptive

Already added in latest commit


Avoid polluting Pigweed environment namespace

Issue Being Resolved
Sourcing with custom Pigweed environment currently fails:
$ PW_CONFIG_FILE=scripts/environment_no_cipd.json source scripts/bootstrap.sh
...
Traceback (most recent call last):
  File "/home/sag/projects/project-chip/connectedhomeip3/.environment/gn_out/../../third_party/pigweed/repo/pw_protobuf_compiler/py/pw_protobuf_compiler/generate_protos.py", line 211, in <module>
    setup_logging()
  File "/home/sag/projects/project-chip/connectedhomeip3/third_party/pigweed/repo/pw_cli/py/pw_cli/log.py", line 118, in install
    colors = pw_cli.color.colors(use_color)
  File "/home/sag/projects/project-chip/connectedhomeip3/third_party/pigweed/repo/pw_cli/py/pw_cli/color.py", line 66, in colors
    env = pw_cli.env.pigweed_environment()
  File "/home/sag/projects/project-chip/connectedhomeip3/third_party/pigweed/repo/pw_cli/py/pw_cli/env.py", line 86, in pigweed_environment
    _memoized_environment = pigweed_environment_parser().parse_env()
  File "/home/sag/projects/project-chip/connectedhomeip3/third_party/pigweed/repo/pw_cli/py/pw_cli/envparse.py", line 153, in parse_env
    raise ValueError(
ValueError: Unrecognized environment variable PW_CONFIG_FILE

This presumably broke due to a Pigweed update.
Change overview
Clear the relevant environment variable to avoid Pigweed complaining about unknown environment variable.

@agners could you apply restyle changes to this?


Children content flickering on grid resize

Has this every happened to anyone? I'm having trouble replicating this on codesandbox.
https://streamable.com/kdg4k
This is all my code is essentially:
<ResponsiveGridLayout
	className="layout"
	breakpoints={{ lg: 1200, md: 996, sm: 768, xs: 480, xxs: 0 }}
	cols={{ lg: 2, md: 2, sm: 2, xs: 1, xxs: 1 }}
>

		sdsdsd

		test

</ResponsiveGridLayout>

This happened to me too. It turned out to be the css ::selection pseudo-element that was changing to the background color while resizing.

Hello guys, I have same problem.
@bgold0, as @MilesMorel  mentioned it is because elements becoming ::selection.
as i see threre are two lines of code are doing that:
styleEl.innerHTML = '.react-draggable-transparent-selection *::-moz-selection {background: transparent;}\n';     styleEl.innerHTML += '.react-draggable-transparent-selection *::selection {background: transparent;}\n';
By default selected elements have white color. With own transparent background,  selected element 'has' parent's background color - in our case it is white - thats why it flickers.
For now i use this adhock solution:
.react-draggable-transparent-selection {   span, div, a {     &::selection {       color: inherit;     }   } }
But would be greate if somebody from @STRML team, will fix this problem, because it looks like a bug, moreover it is hard to find reason of this blinking.

Google Chrome - OK

I'll close it for now

Seems like it can't be reproduces right now. I'll close this bug (at least for now)

this happened for me too, the simplest fix in my case was to add user-select: none on the grid items


USHIFT-4868: Remove centos9 image and tests from ostree

We have enough CentOS 9 tests in bootc testing procedure.

/lgtm

/cherry-pick release-4.18


Make Footer look good on al devices.

Any of you who have time and wants can make the footer look good on all devices.

Done.


Missing standard preamble for abstract operations

The new abstract operations are missing preambles. There should be at least one of:

When the OP_NAME abstract operation is called with arguments ARG1, ARG2, and ARG3, it performs the following steps:

or:

When the OP_NAME abstract operation is called with arguments ARG1, ARG2, and ARG3, the following steps are taken:

Fixed by #42


0599 - Minimum Index Sum of Two Lists

Description of the Problem
Suppose Andy and Doris want to choose a restaurant for dinner, and they both have a list of favorite restaurants represented by strings.
You need to help them find out their common interest with the least list index sum. If there is a choice tie between answers, output all of them with no order requirement. You could assume there always exists an answer.
Code
class Solution(object):
    def findRestaurant(self, list1, list2):
        """
        :type list1: List[str]
        :type list2: List[str]
        :rtype: List[str]
        """

Link To The LeetCode Problem
LeetCode

Please assign me to this issue

@chw9 Waiting for your PR 😸


CI/CD: failed to initialize rtls instance: sgx_ecdsa

Fixes: #1523
Signed-off-by: HuZhiming<EMAIL_ADDRESS>

@HuangYijuan Please change the commit title. It is detected by CICD but it is not the root cause.


[New Port Request]

Library name
QSkinny
Library description
A lightweight framework on top of the Qt scene graph and only few classes from Qt/Quick. It is usable from C++ and/or QML.  Resources
Source repository URL
https://github.com/uwerat/qskinny
Project homepage (if different from the source repository)
https://qskinny.github.io/
Anything else that is useful to know when adding (such as optional features the library may have that should be included)
The (Q)Skinny library is a framework built on top of the Qt scene graph and very few core classes from Qt/Quick. It offers a set of lightweight controls, that can be used from C++ and/or QML.

Hope it can be added

Hope it can be added


Logging: standardise log line format and fix debug logging

Currently some loglines end with a full stop while others do not. We should standardise this.
Also, there is at least one log that logs PaymentHashes in an unfriendly way (https://github.com/bitcoin-dev-project/sim-ln/blob/main/sim-lib/src/lib.rs#L527):
TRACE [sim_lib] Resolved payment received: (DispatchedPayment { source: PublicKey(62af0a7a2b86804657a09f03a183cef60a5b06cab96c9ce73f3a855d70123bbe1fd69edc33ee9d9ad485180faddcfeb0305a659219ebac346a0cc91985a9a01e), destination: PublicKey(501c3bd8dc0927d4af194cb052e97f6fb80d0600ff93af151d5b480c433447ab60762f3b6a5e07b7161e8e25b7557c4c381d38cbbfe180ebd219d6b5acd4d14d), hash: PaymentHash([61, 231, 192, 34, 122, 55, 46, 29, 250, 81, 215, 110, 252, 47, 98, 151, 93, 27, 176, 27, 91, 15, 88, 150, 87, 232, 19, 173, 253, 218, 50, 113]), amount_msat: 100000, dispatch_time: SystemTime { tv_sec:<PHONE_NUMBER>, tv_nsec: 957666000 } }, PaymentResult { htlc_count: 1, payment_outcome: Success })

I believe this was fixed by #86, we can reopen if we run into anything else!


[Constraint system] Compute and use a common type among overloads.

Given an overload set, attempt to compute a "common type" that
abstracts over all entries in the overload set, providing more
structure for the constraint solver.

@swift-ci please smoke test

@swift-ci please test source compatibility

@swift-ci please smoke test

@swift-ci please test source compatibility

@swift-ci please test source compatibility

@swift-ci please test compiler performance


[Doc-Improvement][Acro-Score-Improvement][Teams-Store-ADO-2438788]

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/messaging-extensions/how-to/add-authentication.md
88
10
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit 573e85d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/messaging-extensions/how-to/add-authentication.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
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.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/messaging-extensions/how-to/add-authentication.md
88
9
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/apps-in-teams-meetings/teams-live-share-quick-start.md
89
7
link
:white_check_mark:

msteams-platform/messaging-extensions/how-to/add-authentication.md
88
9
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit f5ec8c5:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/apps-in-teams-meetings/teams-live-share-quick-start.md
:white_check_mark:Succeeded
View

msteams-platform/messaging-extensions/how-to/add-authentication.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
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.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/apps-in-teams-meetings/teams-live-share-quick-start.md
89
7
link
:white_check_mark:

msteams-platform/messaging-extensions/how-to/add-authentication.md
83
12
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit aad4c70:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/apps-in-teams-meetings/teams-live-share-quick-start.md
:white_check_mark:Succeeded
View

msteams-platform/messaging-extensions/how-to/add-authentication.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
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.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Fix /dev/kvm permissions error.

This fixes #894

This change is

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

Test in ubuntu 16.04

We probably shouldn't remove chown here.


Prtest-nometavalidation to staging-nometavalidation

Version(s) of document impacted

[ ] Impacts 7 document
[ ] Impacts 6 document
[ ] Impacts 5.1 document
[ ] Impacts 5.0 document
[ ] Impacts 4.0 document
[ ] Impacts 3.0 document

Reason(s) for not updating all version of documents

[ ] The documented feature was introduced in version (list version here) of PowerShell
[ ] This issue only shows up in version (list version(s) here) of the document
[ ] This PR partially fixes the issue, and issue # tracks the remaining work

Docs Build status updates of commit 90c8f38:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details
Dependent Files

.openpublishing.publish.config.json
:x:Error

Details

.openpublishing.publish.config.json

[Error]

Publish service's config file: .openpublishing.publish.config.json is not valid. Details: System.ArgumentException: Invalid json string for type Microsoft.OpenPublishing.Build.DataContracts.GitModel.PublishConfig. ---> Newtonsoft.Json.JsonSerializationException: Error setting value in extension data for type 'Microsoft.OpenPublishing.Build.DataContracts.GitModel.PublishConfig'. Path 'enable_custom_metadata_validation', line 110, position 46. ---> System.ArgumentException: An item with the same key has already been added.
   at System.ThrowHelper.ThrowArgumentException(ExceptionResource resource)
   at System.Collections.Generic.Dictionary`2.Insert(TKey key, TValue value, Boolean add)
   at Void Add(System.String, System.Object)(Object , Object[] )
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.SetExtensionData(JsonObjectContract contract, JsonProperty member, JsonReader reader, String memberName, Object o)
   --- End of inner exception stack trace ---
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.SetExtensionData(JsonObjectContract contract, JsonProperty member, JsonReader reader, String memberName, Object o)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.PopulateObject(Object newObject, JsonReader reader, JsonObjectContract contract, JsonProperty member, String id)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.CreateObject(JsonReader reader, Type objectType, JsonContract contract, JsonProperty member, JsonContainerContract containerContract, JsonProperty containerMember, Object existingValue)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.Deserialize(JsonReader reader, Type objectType, Boolean checkAdditionalContent)
   at Newtonsoft.Json.JsonSerializer.DeserializeInternal(JsonReader reader, Type objectType)
   at Newtonsoft.Json.JsonConvert.DeserializeObject(String value, Type type, JsonSerializerSettings settings)
   at Newtonsoft.Json.JsonConvert.DeserializeObject[T](String value, JsonSerializerSettings settings)
   at Microsoft.OpenPublishing.Build.Common.JsonUtility.FromJsonString[T](String value) in d:\a\1\s\Libraries\Common\JsonUtility.cs:line 93
   --- End of inner exception stack trace ---
   at Microsoft.OpenPublishing.Build.Common.JsonUtility.FromJsonString[T](String value) in d:\a\1\s\Libraries\Common\JsonUtility.cs:line 101
   at Microsoft.OpenPublishing.Build.WorkflowTasks.SyncGitSourceWithDownloadingPackage.CheckPublishConfig() in d:\a\1\s\Libraries\WorkflowTasks\SyncGitSourceWithDownloadingPackage.cs:line 68

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 90c8f38:
:clock10: Pending: waiting for processors (20 builds ahead of you)
:warning: Docs Build is busy, currently there are 20 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 90c8f38:
:clock10: Pending: waiting for processors (27 builds ahead of you)
:warning: Docs Build is busy, currently there are 27 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 90c8f38:
:clock10: Pending: waiting for processors (27 builds ahead of you)
:warning: Docs Build is busy, currently there are 27 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 90c8f38:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details
Dependent Files

.openpublishing.publish.config.json
:x:Error

Details

.openpublishing.publish.config.json

[Error]

Publish service's config file: .openpublishing.publish.config.json is not valid. Details: System.ArgumentException: Invalid json string for type Microsoft.OpenPublishing.Build.DataContracts.GitModel.PublishConfig. ---> Newtonsoft.Json.JsonSerializationException: Error setting value in extension data for type 'Microsoft.OpenPublishing.Build.DataContracts.GitModel.PublishConfig'. Path 'enable_custom_metadata_validation', line 110, position 46. ---> System.ArgumentException: An item with the same key has already been added.
   at System.ThrowHelper.ThrowArgumentException(ExceptionResource resource)
   at System.Collections.Generic.Dictionary`2.Insert(TKey key, TValue value, Boolean add)
   at Void Add(System.String, System.Object)(Object , Object[] )
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.SetExtensionData(JsonObjectContract contract, JsonProperty member, JsonReader reader, String memberName, Object o)
   --- End of inner exception stack trace ---
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.SetExtensionData(JsonObjectContract contract, JsonProperty member, JsonReader reader, String memberName, Object o)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.PopulateObject(Object newObject, JsonReader reader, JsonObjectContract contract, JsonProperty member, String id)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.CreateObject(JsonReader reader, Type objectType, JsonContract contract, JsonProperty member, JsonContainerContract containerContract, JsonProperty containerMember, Object existingValue)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.Deserialize(JsonReader reader, Type objectType, Boolean checkAdditionalContent)
   at Newtonsoft.Json.JsonSerializer.DeserializeInternal(JsonReader reader, Type objectType)
   at Newtonsoft.Json.JsonConvert.DeserializeObject(String value, Type type, JsonSerializerSettings settings)
   at Newtonsoft.Json.JsonConvert.DeserializeObject[T](String value, JsonSerializerSettings settings)
   at Microsoft.OpenPublishing.Build.Common.JsonUtility.FromJsonString[T](String value) in d:\a\1\s\Libraries\Common\JsonUtility.cs:line 93
   --- End of inner exception stack trace ---
   at Microsoft.OpenPublishing.Build.Common.JsonUtility.FromJsonString[T](String value) in d:\a\1\s\Libraries\Common\JsonUtility.cs:line 101
   at Microsoft.OpenPublishing.Build.WorkflowTasks.SyncGitSourceWithDownloadingPackage.CheckPublishConfig() in d:\a\1\s\Libraries\WorkflowTasks\SyncGitSourceWithDownloadingPackage.cs:line 68

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 90c8f38:
:clock10: Pending: waiting for processors (8 builds ahead of you)
:warning: Docs Build is busy, currently there are 8 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 90c8f38:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details
Dependent Files

.openpublishing.publish.config.json
:x:Error

Details

.openpublishing.publish.config.json

[Error]

Publish service's config file: .openpublishing.publish.config.json is not valid. Details: System.ArgumentException: Invalid json string for type Microsoft.OpenPublishing.Build.DataContracts.GitModel.PublishConfig. ---> Newtonsoft.Json.JsonSerializationException: Error setting value in extension data for type 'Microsoft.OpenPublishing.Build.DataContracts.GitModel.PublishConfig'. Path 'enable_custom_metadata_validation', line 110, position 46. ---> System.ArgumentException: An item with the same key has already been added.
   at System.ThrowHelper.ThrowArgumentException(ExceptionResource resource)
   at System.Collections.Generic.Dictionary`2.Insert(TKey key, TValue value, Boolean add)
   at Void Add(System.String, System.Object)(Object , Object[] )
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.SetExtensionData(JsonObjectContract contract, JsonProperty member, JsonReader reader, String memberName, Object o)
   --- End of inner exception stack trace ---
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.SetExtensionData(JsonObjectContract contract, JsonProperty member, JsonReader reader, String memberName, Object o)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.PopulateObject(Object newObject, JsonReader reader, JsonObjectContract contract, JsonProperty member, String id)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.CreateObject(JsonReader reader, Type objectType, JsonContract contract, JsonProperty member, JsonContainerContract containerContract, JsonProperty containerMember, Object existingValue)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.Deserialize(JsonReader reader, Type objectType, Boolean checkAdditionalContent)
   at Newtonsoft.Json.JsonSerializer.DeserializeInternal(JsonReader reader, Type objectType)
   at Newtonsoft.Json.JsonConvert.DeserializeObject(String value, Type type, JsonSerializerSettings settings)
   at Newtonsoft.Json.JsonConvert.DeserializeObject[T](String value, JsonSerializerSettings settings)
   at Microsoft.OpenPublishing.Build.Common.JsonUtility.FromJsonString[T](String value) in d:\a\1\s\Libraries\Common\JsonUtility.cs:line 93
   --- End of inner exception stack trace ---
   at Microsoft.OpenPublishing.Build.Common.JsonUtility.FromJsonString[T](String value) in d:\a\1\s\Libraries\Common\JsonUtility.cs:line 101
   at Microsoft.OpenPublishing.Build.WorkflowTasks.SyncGitSourceWithDownloadingPackage.CheckPublishConfig() in d:\a\1\s\Libraries\WorkflowTasks\SyncGitSourceWithDownloadingPackage.cs:line 68

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit fbcaf3d:
:clock10: Pending: waiting for processors (5 builds ahead of you)
:warning: Docs Build is busy, currently there are 5 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit fbcaf3d:
:clock10: Pending: waiting for processors (6 builds ahead of you)
:warning: Docs Build is busy, currently there are 6 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit fbcaf3d:
:clock10: Pending: waiting for processors (6 builds ahead of you)
:warning: Docs Build is busy, currently there are 6 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 282cf8e:
:clock10: Pending: waiting for processors (10 builds ahead of you)
:warning: Docs Build is busy, currently there are 10 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit fbcaf3d:
:clock10: Full build: the average full build time is 10 min(s) 21 sec(s), this is based on the last 10 build(s) for this repository.

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

File
Status
Preview URL
Details
Dependent Files

developer/TestingPullRequest.md
:warning:Warning
View
Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

developer/TestingPullRequest.md

[Warning] Metadata with following name(s) are not allowed to be set in Yaml header, or as file level metadata in docfx.json, or as global metadata in docfx.json: content_type. They are generated by Docs platform, so the values set in these 3 places will be ignored. Please remove them from all 3 places to resolve the warning.
[Warning] the value invalid_type is undefined for metadata content_type
Line 8: [Warning] Invalid file link:(~/../nonexisted1.md).
Line 9: [Warning] Invalid file link:(~/../nonexisted2.md).
Line 10: [Warning] Invalid file link:(~/../nonexisted3.md).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 282cf8e:
:clock10: Full build: the average full build time is 10 min(s) 27 sec(s), this is based on the last 13 build(s) for this repository.

Docs Build status updates of commit 282cf8e:
:warning: Validation status: warnings

File
Status
Preview URL
Details
Dependent Files

developer/TestingPullRequest.md
:warning:Warning
View
Details

reference/6/Microsoft.PowerShell.Core/About/about_Functions_Advanced_Parameters.md
:warning:Warning
View (powershell-6)
Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

README.md
:white_check_mark:Succeeded

reference/5.0/PowershellGet/Get-InstalledModule.md
:white_check_mark:Succeeded
View (powershell-5.0)

reference/5.1/PowershellGet/Get-InstalledModule.md
:white_check_mark:Succeeded
View (powershell-5.1)

reference/6/Microsoft.PowerShell.Core/Register-ArgumentCompleter.md
:white_check_mark:Succeeded
View (powershell-6)

reference/6/PowerShellGet/Get-InstalledModule.md
:white_check_mark:Succeeded
View (powershell-6)

reference/docs-conceptual/learn/Writing-Portable-Modules.md
:white_check_mark:Succeeded
View (powershell-3.0) View (powershell-4.0) View (powershell-5.0) View (powershell-5.1) View (powershell-6)

developer/TestingPullRequest.md

[Warning] Metadata with following name(s) are not allowed to be set in Yaml header, or as file level metadata in docfx.json, or as global metadata in docfx.json: content_type. They are generated by Docs platform, so the values set in these 3 places will be ignored. Please remove them from all 3 places to resolve the warning.
[Warning] the value invalid_type is undefined for metadata content_type
Line 8: [Warning] Invalid file link:(~/../nonexisted1.md).
Line 9: [Warning] Invalid file link:(~/../nonexisted2.md).
Line 10: [Warning] Invalid file link:(~/../nonexisted3.md).

reference/6/Microsoft.PowerShell.Core/About/about_Functions_Advanced_Parameters.md

[Warning] Metadata with following name(s) are not allowed to be set in Yaml header, or as file level metadata in docfx.json, or as global metadata in docfx.json: locale. They are generated by Docs platform, so the values set in these 3 places will be ignored. Please remove them from all 3 places to resolve the warning.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 282cf8e:
:clock10: Full build: the average full build time is 10 min(s) 22 sec(s), this is based on the last 14 build(s) for this repository.

Docs Build status updates of commit 282cf8e:
:warning: Validation status: warnings

File
Status
Preview URL
Details
Dependent Files

developer/TestingPullRequest.md
:warning:Warning
View
Details

reference/6/Microsoft.PowerShell.Core/About/about_Functions_Advanced_Parameters.md
:warning:Warning
View (powershell-6)
Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

README.md
:white_check_mark:Succeeded

reference/5.0/PowershellGet/Get-InstalledModule.md
:white_check_mark:Succeeded
View (powershell-5.0)

reference/5.1/PowershellGet/Get-InstalledModule.md
:white_check_mark:Succeeded
View (powershell-5.1)

reference/6/Microsoft.PowerShell.Core/Register-ArgumentCompleter.md
:white_check_mark:Succeeded
View (powershell-6)

reference/6/PowerShellGet/Get-InstalledModule.md
:white_check_mark:Succeeded
View (powershell-6)

reference/docs-conceptual/learn/Writing-Portable-Modules.md
:white_check_mark:Succeeded
View (powershell-3.0) View (powershell-4.0) View (powershell-5.0) View (powershell-5.1) View (powershell-6)

developer/TestingPullRequest.md

[Warning] Metadata with following name(s) are not allowed to be set in Yaml header, or as file level metadata in docfx.json, or as global metadata in docfx.json: content_type. They are generated by Docs platform, so the values set in these 3 places will be ignored. Please remove them from all 3 places to resolve the warning.
[Warning] the value invalid_type is undefined for metadata content_type
Line 8: [Warning] Invalid file link:(~/../nonexisted1.md).
Line 9: [Warning] Invalid file link:(~/../nonexisted2.md).
Line 10: [Warning] Invalid file link:(~/../nonexisted3.md).

reference/6/Microsoft.PowerShell.Core/About/about_Functions_Advanced_Parameters.md

[Warning] Metadata with following name(s) are not allowed to be set in Yaml header, or as file level metadata in docfx.json, or as global metadata in docfx.json: locale. They are generated by Docs platform, so the values set in these 3 places will be ignored. Please remove them from all 3 places to resolve the warning.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


App failing and generating tons of log

@DanNBullock
I have 2 issues to report..

The app is failing with the following error message.

hayashis@carb(h1):~/git/app-wmaSeg(disabled) $ cat 947749.s1.err
IU-Quota version 1.0 loaded.
VIM text editor version 8.0 loaded.
singularity version 2.6.1 loaded.
nodejs version 8.12.0 loaded.
WARNING: Non existent bind point (file) in container: '/etc/localtime'
WARNING: Non existent bind point (directory) in container: '/N/dcwan'

real	2m0.860s
user	1m26.411s
sys	0m13.533s
ERROR  : Could not write to /N/soft/rhel7/singularity/2.6.1/var/singularity/mnt/final/etc/localtime: Too many levels of symbolic links
File is empty
File is empty
File is empty
The file
   "/opt/mcr/v92/mcr/toolbox/matlab/lang/varargin.m"
cannot be executed.

Error in wma_segMajTracks_BL (line 58)

MATLAB:err_parse_cannot_run_m_file

I am feeding this config.

    "freesurfer": "/N/dc2/scratch/hayashis/testdata/100206.freesurfer/output",
    "track": "/N/dc2/scratch/hayashis/testdata/100206.track-tck/track.tck"
}

The app is still outputting enormous amount of stdout log. Can you trim the output?

I just rerun the app and now it's failing with a different error message.
Subscripted assignment dimension mismatch.

Error in bsc_makePlanarROI (line 66)

Error in bsc_planeFromROI_v2 (line 56)

Error in bsc_segmentCorpusCallosum_v3 (line 85)

Error in wma_segMajTracks_BL (line 40)

MATLAB:subsassigndimmismatch

Hi, I also experienced issue number 1).
In another case instead I got this:
Reference to non-existent field 'wbfg'.

Error in main (line 21)

MATLAB:nonExistentField

Has this been dealt with?

I don't think so.., but I haven't been running wmaSeg a lot myself.

This has been handled yes?  I don't think it is erroring any more.  Was this a specific branch?

I haven't seen this issue lately and nobody is complaining as far as I know. I am going to close this one.


Is there a way to check if a given List was constructed using List.unmodifiable?

I am interested to know if I can check if a given List was constructed using List.unmodifiable but I couldn't find a way.
My use case is that I am copying an object extensively (i.e. using Object.copyWith(...) method), and in the constructor, I would like to make sure if the members of type List are unmodifiable.
For example, this is what I was looking to achieve (i.e. values.isUnmodifiable):
class Example {
  final String id;
  // ....
  final List values;

  Example(this.id, List values) : values = values.isUnmodifiable ? values : List.unmodifiable(values);

  Example copyWith({String? id, List? values}) => Example(id ?? this.id, values ?? this.values);
}

I ended up creating the following interface just for type checking as workaround so I can do values is UnmodifiableList ? values : UnmodifiableList(values):
class UnmodifiableList<E> extends ListBase<E> {
  final List<E> _source;

  UnmodifiableList(Iterable<E> source) : _source = List.unmodifiable(source);

  @override
  int get length => _source.length;

  @override
  E operator [](int index) => _source.elementAt(index);

  // redirect to `List.unmodifiable` which will throw
  @override
  set length(int newLength) => _source.length = newLength;

  // redirect to `List.unmodifiable` which will throw
  @override
  void operator []=(int index, E value) => _source[index] = value;
}

I did look at UnmodifiableListView from package:collection but it allows the source List to be modified which is undesirable.

There is no way to check whether a list was created using List.unmodifiable.
How unmodifiable lists are implemented is up to the individual back-ends, and it differs. It might be using a separate class, which may or may not be the same used for constant lists, or it can use a flag on the object to make it unmodifiable.
We could probably add a way to ask whether a list is unmodifiable, which would be true for this and constant lists, but that's counter productive if it doesn't include other ways to create unmodifiable lists. Those are not the only unmodifiable lists.

There is no way to check whether a list was created using List.unmodifiable.
How unmodifiable lists are implemented is up to the individual back-ends, and it differs. It might be using a separate class, which may or may not be the same used for constant lists, or it can use a flag on the object to make it unmodifiable.
We could probably add a way to ask whether a list is unmodifiable, which would be true for this and constant lists, but that's counter productive if it doesn't include other ways to create unmodifiable lists. Those two are not the only unmodifiable lists.


Bug: Plugin constantly tries to use cordial even when still on cooldown

When using the plugin and enabling the "Use Cordials" function it will use the cordials when up but also when they are still on cooldown. It will keep pressing the cordial and the game will display the "Not ready yet" error until you turn off the "Use Cordials" function.

Yeah thanks for spotting that, I forgor to check the pot cooldown. It should be fine now


@craigspaeth - [WIP] Forced login inquiry QA

Closes #404
Kind of hard to test this locally, but there is an actual CORS issue happening. Microgravity is proxy-ing Force but then requesting to log_in using the actual Force url and getting a No 'Access-Control-Allow-Origin' error. Moving the cors middleware to before artsy-passport should fix this issue.
But..... now getting the dreaded asset is not a function when trying to hit that route from MG, I can't tell if this is just my local setup or not. When you get a second (ha!), mind pulling this down and seeing if you can replicate locally?

Start mg in staging
Start force in staging
Visit an artwork inquiry page (I used http://localhost:3333/inquiry/jon-cattapan-september-session ) in the mobile mode of your browser
Try to place an inquiry with a new email and create an account (hitting /log_in is what I couldn't get past)

Ok, so I dug into it and I'm seeing a 302 response when POSTing to /sign_up... which seems to be due to how we try and redirect back after signing up. Changing req.xhr to req.accepts 'application/json' seems to get past this issue, but then it runs into a CSRF issue.
The asset is not a function issue could be stifling these errors—I also saw a User already exists surfaced from the API. I went ahead and made some small changes to not stifle those errors in our error handler.
However, given I'm working towards merging MG + Force and this seems to be a bit of a rabbit hole b/c of the  proxying approach—maybe we should go ahead and pair on refactoring this to not use the proxying and do something more sane. Feel free to put a screen hero session on the calendar.

Thanks for looking @craigspaeth. I'm gonna give it one more little shot with the updates you made to the error handler. But you're right, we should refactor auth regardless.

OK @craigspaeth , you're correct, the CSRF issue is a blocker. It's probably worth merging this anyway, even though it doesn't fix the main issue it fixes a few minor ones (and updates artsy-error-handler).
It is strange though, the  csrf cookie value is identical on both local force and local microgravity, I'm sure I'm missing something but I don't see how the token would be invalid.

Sounds good. Feel free to merge when you remove that stray log and it's 💚


Stop appengineRun without killing gradle daemon

To stop the appengine devserver, I've been using CTRL-C. This stops the devserver, but also kills the gradle daemon, which means that everything takes a while to start back up again.
Is there a keyboard shortcut or method to stop the appengine devserver without killing gradle?

You can run the appengineStop task from another terminal?
If that doesn't work (I don't know why it wouldn't) and you're on a linux machine, you can find the process that's using the port the dev appserver was started on, for instance on port 8080
$ lsof -i:8080
COMMAND   PID USER   FD   TYPE   DEVICE SIZE/OFF NODE NAME
java    25708 bob   35u  IPv6 264550      0t0  TCP localhost:http-alt (LISTEN)                    │functionalTest  main

$ kill 25708

This looks like it kills the server without killing the daemon.

For the sake of convenience, is it possible to make the appengineRun task swallow CTRL-C and exit "sucessfully", or have a different keyboard combo like CTRL-D to do that?
appengineStop works great though, thanks! It's just a little bit more "remember" overhead, which would be great to not have to worry about.

The keyboard shortcuts is something we don't control unfortunately.

Can the plugin/task control the handling of signals? I think that, in Linux, CTRL-C is SIGINT, and CTRL-D is some sort of end-of-file delimiter. I want to believe :wink:

Actually it's worse than that now.


pip install fails if IDF is shallow cloned (IDFGH-2203)

Environment

Operating System: Arch Linux

Problem Description
pip install -r requirements.txt fails with a shallow clone.
Expected Behavior
The installation should complete.
Actual Behavior
pip install fails with the following stack trace:
Extracting /home/hack/.espressif/dist/openocd-esp32-linux64-0.10.0-esp32-20190313.tar.gz to /home/hack/.espressif/tools/openocd-esp32/v0.10.0-esp32-20190313
Installing Python environment and packages
fatal: No names found, cannot describe anything.
Traceback (most recent call last):
  File "/mnt/test/projects/esp32/esp/idf/tools/idf_tools.py", line 1344, in <module>
    main(sys.argv[1:])
  File "/mnt/test/projects/esp32/esp/idf/tools/idf_tools.py", line 1340, in main
    action_func(args)
  File "/mnt/test/projects/esp32/esp/idf/tools/idf_tools.py", line 1139, in action_install_python_env
    idf_python_env_path, _, virtualenv_python = get_python_env_path()
  File "/mnt/test/projects/esp32/esp/idf/tools/idf_tools.py", line 840, in get_python_env_path
    idf_version_str = subprocess.check_output(['git', '--work-tree=' + global_idf_path, 'describe', '--tags'], cwd=global_idf_path, env=os.environ).decode()
  File "/usr/lib/python3.7/subprocess.py", line 395, in check_output
    **kwargs).stdout
  File "/usr/lib/python3.7/subprocess.py", line 487, in run
    output=stdout, stderr=stderr)
subprocess.CalledProcessError: Command '['git', '--work-tree=/mnt/test/projects/esp32/esp/idf', 'describe', '--tags']' returned non-zero exit status 128.

Steps to reproduce
I'm running the following script to setup IDF on my machine:
#!/usr/bin/sh

export ESP_ROOT=$(pwd)/esp
export ESP_VIRTUALENV=$ESP_ROOT/venv
export IDF_PATH=$ESP_ROOT/idf  # constant used by the IDF

export ESP_IDF_VERSION="v4.0"
export ESP_IDF_GIT_URL="https://github.com/espressif/esp-idf.git"

export PATH="$IDF_PATH/tool:$PATH"

[ ! -e $IDF_PATH ] && {
    git clone --depth 1 --recursive $ESP_IDF_GIT_URL -b release/$ESP_IDF_VERSION $IDF_PATH
}

[ ! -e $ESP_VIRTUALENV ] && {
    python -m venv $ESP_VIRTUALENV
    source $ESP_VIRTUALENV/bin/activate
    pip install -r $IDF_PATH/requirements.txt
}

I just noticed the venv was created with python3, however the git command seems to be unrelated to that.
Ideally the setup would work with a shallow copy, otherwise documenting this would be useful.

Hi @hackaugusto. I can reproduce this error message but just to note that it is not related to pip install. It fails for tools/idf_tool.py probably while you are invoking it directly or through export.sh.


[frameworks] Fix Dojo logo

Use the existing PNG for Dojo, instead of the SVG, which does not work.

Because the SVG doesn't work

@leo this specific SVG does not work


Support Python 3.6

As noted.

Consider adding the recently-released Python 3.7 version to this list, too.


Adds bolt and tests from serial-ghost-tables PR

Not updating NEWS.md because it's an unused tool atm.
Please CR @juanignaciosl

:ok_hand:


The vote button rounds up the vote percentages.

https://github.com/turbo-src/chrome-extension/pull/77
Percentages now round down

Merged to master


Fixed up README a bit

Improved definitions, fixed broken variable names, use OpenAPI > OAS.
I find people are getting used to the term OpenAPI, but OAS does not get as much recognition. For the sake of humans and SEO lets stick with OpenAPI. Also calling the YAML that users are writing a "OAS Specification" is double confusing. Technically they are both different types of specification but lets avoid using that word to death.
https://blog.apisyouwonthate.com/resolving-overloaded-terms-for-api-specifications-descriptions-contracts-c31a6d44d269
I think OpenAPI Description Document or OpenAPI Document covers things pretty well, and then OpenAPI Specification is the 3.0.x.md spec itself. It's backed up roughly in 3.0.2.md.

This change is less important, but something to consider. This PR improves things regardless. :)
Thanks for the great work!

Yeah, I feel that I am not good with all the terms regarding OpenAPI. Thank you for corrections!


Benchmarking Framework

Description
Design and implement an out of the box benchmarking framework for stencil based micro-services.
Tasks
1. Research

[ ] Design the framework - Research

[ ] wrk for HTTP requests
[ ] what other tech can be used?

2. PoC

[ ] Setup the required files in a sample app

3. Incorporate in Stencil

[ ] 1. Add suitable schematics
[ ] 2. Implement suitable components in packages/common
[ ] 3. Update suitable stencil-cli commands

https://docs.google.com/document/d/19PqClLN2Tp_gYZeWOFUkx3SbarfDSsdriW3UFf0RdBI/edit


Saving new df as xpt file

Hi, can you explain how we can save the dataframe (df) to new xpt file?
Thanks,
Sagar

I do it using this line to convert it to a list of dictionaries:
Turns dataframe to list of dictionaries for writing xpt
data = df.T.to_dict().values()
And then these two lines to write the .xpt file:
Writing the xpt file.
with open('file.xpt', 'wb') as f:
xport.from_rows(data, f)

New API. #34
In [1]: import pandas as pd

In [2]: import xport.v56

In [3]: df = pd.DataFrame()

In [4]: with open('example.xpt', 'wb') as f:
   ...:     xport.v56.dump(df, f)
   ...:

However, it'll be best if you use the xport.Dataset to set metadata for your dataset.


Slow loading

I'm stuck on loading for a long time when starting up, sometimes it takes a long time to load (or freezes) when switching to a new song.
What can I do to make it load faster?

Yeah, this makes sense, since it buffers entire songs rather than individual parts. This makes more sense with music files since they're relatively tiny. What's your download speed?

Well, the speedtest results show a download speed of around 92 Mbps.

You could try updating, since 1.5.0 overhauled a lot of networking & buffering code, but if that also doesn't help then try to send a video.

i am having the same issue, it is stuck at loading for me.

Did you try the manual method? It could be related to mpris perhaps?

I can't seem to reproduce this issue. I'll reopen it if this issue is still prevalent on 1.5.0+.

Sorry for the delayed response, it is working perfectly for me after i installed it using cargo install with mpris.


KYLIN-4485 Fix NoSuchMethodError in kylin-it

Proposed changes
Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request. If it fixes a bug or resolves a feature request, be sure to link to that issue.
Types of changes
What types of changes does your code introduce to Kylin?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[ ] I have create an issue on Kylin's jira, and have described the bug/feature there in detail
[ ] Commit messages in my PR start with the related jira ID, like "KYLIN-0000 Make Kylin project open-source"
[ ] Compiling and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] If this change need a document change, I will prepare another pr against the document branch
[ ] Any dependent changes have been merged

Further comments
If this is a relatively large or complex change, kick off the discussion at user@kylin or dev@kylin by explaining why you chose the solution you did and what alternatives you considered, etc...

Codecov Report

Merging #1257 into master will decrease coverage by 0.28%.
The diff coverage is 14.29%.

@@             Coverage Diff              @@
##             master    #1257      +/-   ##
============================================
- Coverage     25.71%   25.43%   -0.29%
- Complexity     6626     6731     +105
============================================
  Files          1484     1505      +21
  Lines         91185    93308    +2123
  Branches      12726    13082     +356
============================================
+ Hits          23452    23730     +278
- Misses        65431    67241    +1810
- Partials       2302     2337      +35

Impacted Files
Coverage Δ
Complexity Δ

...main/java/org/apache/kylin/common/KylinConfig.java
38.29% <0.00%> (-2.08%)
29.00 <0.00> (+1.00)
:arrow_down:

.../java/org/apache/kylin/common/KylinConfigBase.java
12.70% <0.00%> (+0.99%)
51.00 <0.00> (+7.00)

...org/apache/kylin/common/restclient/RestClient.java
15.25% <0.00%> (-3.11%)
6.00 <1.00> (ø)

...n/java/org/apache/kylin/common/util/CheckUtil.java
0.00% <0.00%> (ø)
0.00 <0.00> (ø)

...c/main/java/org/apache/kylin/cube/CubeSegment.java
42.27% <0.00%> (-6.25%)
65.00 <0.00> (ø)

...rg/apache/kylin/cube/gridtable/CubeCodeSystem.java
22.07% <0.00%> (-3.30%)
8.00 <0.00> (ø)

.../org/apache/kylin/cube/model/AggregationGroup.java
77.15% <ø> (ø)
85.00 <0.00> (ø)

...ain/java/org/apache/kylin/cube/model/CubeDesc.java
58.15% <0.00%> (-4.14%)
182.00 <0.00> (ø)

...va/org/apache/kylin/cube/model/DictionaryDesc.java
48.57% <0.00%> (-16.82%)
8.00 <0.00> (ø)

...apache/kylin/cube/model/HBaseColumnFamilyDesc.java
23.80% <0.00%> (-14.66%)
4.00 <0.00> (ø)

... and 100 more

Continue to review full report at Codecov.

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


Config Created with arr and --config passed to a non-existent file

bakerboy448@hetz-baker-2004:~/.config/omegabrr$ rm config.yaml
bakerboy448@hetz-baker-2004:~/.config/omegabrr$ omegabrr arr --config ~/.config/omegabrr/config.yaml
2022-11-28T13:53:21-06:00 DBG starting filter processing...
2022-11-28T13:53:21-06:00 DBG gathering titles... client=sonarr type=sonarr
2022-11-28T13:53:21-06:00 DBG gathering titles... client=radarr4k type=radarr
2022-11-28T13:53:21-06:00 DBG gathering titles... client=radarr type=radarr
2022-11-28T13:53:21-06:00 ERR Something went wrong trying to update filters! Total time: 2.647045ms error="api.Get(series): failed:  (status: 401 Unauthorized): invalid status code, <200||>299: "

Instead of generating the config and error should be thrown that the config is missing
bakerboy448@hetz-baker-2004:~/.config/omegabrr$ omegabrr version
Version: 1.0.1
Commit: 62573067a39e79af98a8feb86daead94ce9f0075

I'm torn if this is an issue or not. Creating the config for the user if it doesn't exist is a good thing imo :thinking:
But having the defaults be what they currently are is not ideal. I'll update default it generates to have the services commented out.


add Notional-gentx

Description and Motivation

Related Issues

Checklist:

[ ] I have read Migaloo's contribution guidelines.
[ ] My pull request has a sound title and description (not something vague like Update index.md)
[ ] All existing and new tests are passing.
[ ] I updated/added relevant documentation.
[ ] The code is formatted properly golangci-lint run ./... --fix.

Hey could you have the PR target the release/v1.0.x branch?

consilidated in #97


Intercom script get inserted in a wrong place

Version info

intercom-rails version: 0.2.31
Rails version: 5.0.0.rc1

Expected behavior
I've used a widget like a cities map from Intercom, which I've inserted to my html page.
It looked something like that:

Actual behavior
It did not appear on the page.
I've spent some time trying to fix this issue, looked through and there is what I got:
In this code here
https://github.com/intercom/intercom-rails/blob/master/lib/intercom-rails/auto_include_filter.rb#L34
def include_javascript!
  response.body = response.body.gsub(CLOSING_BODY_TAG, intercom_script_tag.to_s + '\\0')
end

It finds any <\/body> symbols on the page and inserts intercom script there. So, after that script for adding map has these symbols it tried to insert script tag there which was breaking the html markup.

One suggested fix could be to make sure we only match the last occurrence of the closing body tag, with something like this:
-CLOSING_BODY_TAG = %r{}
+CLOSING_BODY_TAG = %r{.*}

(I don't this would work with the way gsub is used, but hopefully the idea is clear enough)
Or we could use something like nokogiri to parse the document and add it before the end of the outmost body tag way, but maybe performance would be a concern.

Hi,
For cases such as this, or situations where there is no closing body tag we do allow for manually inserting the Intercom Javascript. We describe this in the  trouble shooting sections here. That might be a better workaround then trying to capture the different body locations? Were you able to get a workaround here or did you try manually inserting the javascript?


Respect  element

I currently use the  element in my html.  Is there a way to instruct restangular to prefix this value to all relative urls?

https://github.com/mgonto/restangular#setbaseurl


[maven-central] Maven Central badge based on repo1.maven.org rather than search.maven.org

According to this:

Upon release, your component will be published to Central: this typically occurs within 10 minutes, though updates to search can take up to two hours.

So, besides the delay, if the indexing jobs for search.maven.org are paused, it may take quite some time to see the last artifact version even though it has been actually released on Maven Central.
Fixes #846.

Hi! Thank you for tackling this!
Would you be willing to add tests using the project's new testing framework?

I will give it a try, I'm sure it will be instructive! :nerd_face:

Feedback would be really appreciated! Don't hesitate to reach out if you have questions.

First question @paulmelnikow: since the new testing framework has been introduced only after my pull request, how do you want me to proceed? Rebase and (force) push the branch associated to this pull request? Merge master into my feature breanch? Other option?
I'm asking because I know every team has its own "rules" when it comes to not messing up with the history! :sweat_smile:

Hehe, indeed, every project is different. My preference is that you merge master into your branch. With all the commits in place, the discussion is easier to join midstream and after the fact, and means reviewers can just read the new commits. Think I read this in ESLint's guidelines, and it matches my experience with code review. Then it's on the maintainer to summarize the commit message at squash time. Though, it's helpful if feel you update the top comment and/or title to explain the "why" behind the changes.

@paulmelnikow I may be wrong but this should be complete now.
Please find below a couple of things I've found:

Following the code coverage section I wasn't able to get the code coverage because of this:

$ npm run coverage:test:services -- -- --only=maven-central

><EMAIL_ADDRESS>coverage:test:services /Users/fabriziocucci/git/shields
> istanbul cover _mocha --delay service-tests/runner/cli.js --dir coverage/services "--" "--only=maven-central"

=============================================================================
Writing coverage object [/Users/fabriziocucci/git/shields/coverage/services/coverage.json]
Writing coverage reports at [/Users/fabriziocucci/git/shields/coverage/services]
=============================================================================

=============================== Coverage summary ===============================
Statements   : 60.47% ( 78/129 )
Branches     : 26.92% ( 7/26 )
Functions    : 50% ( 9/18 )
Lines        : 60% ( 72/120 )
================================================================================
internal/process/next_tick.js:142
      throw new TypeError('callback is not a function');
      ^

TypeError: callback is not a function
    at process.nextTick (internal/process/next_tick.js:142:13)
    at Object.<anonymous> (/Users/fabriziocucci/git/shields/service-tests/runner/cli.js:9:4713)
    at Module._compile (module.js:556:32)
    at Object.Module._extensions.(anonymous function) [as .js] (/Users/fabriziocucci/git/shields/node_modules/istanbul/lib/hook.js:107:24)
    at Module.load (module.js:473:32)
    at tryModuleLoad (module.js:432:12)
    at Function.Module._load (module.js:424:3)
    at Module.require (module.js:483:17)
    at require (internal/module.js:20:19)
    at /Users/fabriziocucci/git/shields/node_modules/mocha/lib/mocha.js:222:27
    at Array.forEach (native)
    at Mocha.loadFiles (/Users/fabriziocucci/git/shields/node_modules/mocha/lib/mocha.js:219:14)
    at Mocha.run (/Users/fabriziocucci/git/shields/node_modules/mocha/lib/mocha.js:487:10)
    at Object.<anonymous> (/Users/fabriziocucci/git/shields/node_modules/mocha/bin/_mocha:459:18)
    at Module._compile (module.js:556:32)
    at Object.Module._extensions..js (module.js:565:10)
    at Object.Module._extensions.(anonymous function) [as .js] (/Users/fabriziocucci/git/shields/node_modules/istanbul/lib/hook.js:109:37)
    at Module.load (module.js:473:32)
    at tryModuleLoad (module.js:432:12)
    at Function.Module._load (module.js:424:3)
    at Function.Module.runMain (module.js:590:10)
    at runFn (/Users/fabriziocucci/git/shields/node_modules/istanbul/lib/command/common/run-with-cover.js:122:16)
    at /Users/fabriziocucci/git/shields/node_modules/istanbul/lib/command/common/run-with-cover.js:251:17
    at /Users/fabriziocucci/git/shields/node_modules/istanbul/lib/util/file-matcher.js:68:16
    at /Users/fabriziocucci/git/shields/node_modules/async/lib/async.js:52:16
    at /Users/fabriziocucci/git/shields/node_modules/async/lib/async.js:361:13
    at /Users/fabriziocucci/git/shields/node_modules/async/lib/async.js:52:16
    at done (/Users/fabriziocucci/git/shields/node_modules/async/lib/async.js:246:17)
    at /Users/fabriziocucci/git/shields/node_modules/async/lib/async.js:44:16
    at /Users/fabriziocucci/git/shields/node_modules/async/lib/async.js:358:17
    at LOOP (fs.js:1739:14)
    at _combinedTickCallback (internal/process/next_tick.js:67:7)
    at process._tickCallback (internal/process/next_tick.js:98:9)

npm ERR! Darwin 15.6.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "run" "coverage:test:services" "--" "--" "--only=maven-central"
npm ERR! node v6.6.0
npm ERR! npm  v3.10.3
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>coverage:test:services: `istanbul cover _mocha --delay service-tests/runner/cli.js --dir coverage/services "--" "--only=maven-central"`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>coverage:test:services script 'istanbul cover _mocha --delay service-tests/runner/cli.js --dir coverage/services "--" "--only=maven-central"'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the gh-badges package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     istanbul cover _mocha --delay service-tests/runner/cli.js --dir coverage/services "--" "--only=maven-central"
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs gh-badges
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls gh-badges
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/fabriziocucci/git/shields/npm-debug.log

regardless by the previous error, after executing that command the coverage folder is created and any subsequent execution of npm test will fail because that folder is scanned by eslint (i.e. eslint '**/*.js').

if we want to be really strict, the tests don't cover the different color that the badge has if the latest maven version is a snapshot dependency or has version zero, i.e.

...
      if (version === '0' || /SNAPSHOT/.test(version)) {
        badgeData.colorscheme = 'orange';
      } else {
        badgeData.colorscheme = 'blue';
      }
...

I look forward to receiving your feedback.

This is off-topic here but I'll write it down just in case I forget! :smile:
If this PR goes through, as potential customization of this badge, it could be worth considering to allow the user to specify latest (either release or snapshot), release (only release) or snapshot (only snapshot) as parameter. Since this PR relies on the maven-metadata.xml, it would be really easy to extract that information. The common use case I'm thinking is the possibility of having two badges, one for the latest (stable) release version and one for the latest (unstable) snapshot.

Thanks for the detailed feedback, which is very helpful. These tests look great! Will leave you a couple comments and then respond to your feedback.

I opened #975 with a fix for the two issues you encountered. As a temporary workaround until that is merged, you can:

Run ./node_modules/.bin/istanbul cover node_modules/.bin/_mocha  service-tests/runner/cli.js --dir coverage/services "--" "--delay" "--only=maven-central"
Add the .eslintignore from that PR

if we want to be really strict, the tests don't cover the different color that the badge has if the latest maven version is a snapshot dependency or has version zero

It's a good point you make. Testing colors would be a nice improvement. It's important for some badges, less important for others. Seems like it would be good here, since the color depends on a custom test.
To facilitate this, we could add the colors to the json badges. That seems a fine addition to the API.

If this PR goes through, as potential customization of this badge, it could be worth considering to allow the user to specify latest (either release or snapshot), release (only release) or snapshot (only snapshot) as parameter. Since this PR relies on the maven-metadata.xml, it would be really easy to extract that information. The common use case I'm thinking is the possibility of having two badges, one for the latest (stable) release version and one for the latest (unstable) snapshot.

That's a nice idea. 😄  It could be done with an optional parameter in the badge, as in #959.

Replaced regexp with xml2js as per comment and added test to cover a potential parsing error.
Waiting for reply to the last pending comment!

Tangentially related: something's wrong with the code that is supposed to run the badge tests in Travis.
$ if [ "$TRAVIS_EVENT_TYPE" == pull_request ]; then npm run test:services -- --pr; fi
><EMAIL_ADDRESS>test:services /home/travis/build/badges/shields
> mocha --delay service-tests/runner/cli.js "--pr"
PR: badges/shields#957
Title: [maven-central] Maven Central badge based on repo1.maven.org rather than search.maven.org
No services found. Nothing to do.

[maven-central] seems fine as a service id. Maybe the regex needs to be modified.

Added comment with the link to the project on Maven Central used in tests for the maven-central badge and opened pull request to fix the regxexp used for matching services in pull requests.

Merged #978 and re-triggered the build, and the Maven Central tests ran and passed on Travis. :+1:
The CRAN tests ran too though, so something's not quite right, though that needn't hold this up.


Support for clustering mode

I can add multiple worker but it expose only one worker metrics at a time.
How can I support the cluster mode using this plugin?

I checked out the code, and find out that this library doesn't support AggregatorRegistry which is for clustering.
Who looking for the way can follow the steps below. It works.

Use createPrometheusExporterPlugin with each new registry and registry to AggregatorRegistry

In worker,

const registry = new Registry();

createPrometheusExporterPlugin({ ..., register: registry})
AggregatorRegistry.setRegistry(registry)

In master, I opened a new metrics server to expose aggregatorRegistry.

const registry = new AggregatorRegistry();
const metricsApp = express()
metricsApp.get('/metrics', async (req, res) => {
  try {
	  const metrics = await aggregatorRegistry.clusterMetrics();
	  res.set('Content-Type', aggregatorRegistry.contentType);
	  res.send(metrics);
  } catch (ex) {
	  res.statusCode = 500;
	  res.send(ex.message);
  }
});

metricsApp.listen({another port})
`
``

https://github.com/siimon/prom-client/tree/master#usage-with-nodejss-cluster-module


ValueError: No op named NonMaxSuppressionV3 in defined operations.

Hey Alex, thanks so much for a wonderful writeup.
I downloaded the model from tensorflow/models repo and trained ssd_inception_v2_coco on your simulator dataset.
I had to train on TF1.10 because otherwise i would get some error about axis dimension not in [0,0) in the hard negative miner class. Someone filed this issue on tensorflow/models repo and the fix was to train on v1.8 and above.
After training, i exported it out on TF1.4, now i am running the imported graph on TF1.3 on the simulator and this is what I get:
Traceback (most recent call last):
  File "/capstone/ros/src/tl_detector/tl_detector.py", line 179, in <module>
    TLDetector()
  File "/capstone/ros/src/tl_detector/tl_detector.py", line 46, in __init__
    self.light_classifier = TLClassifier(True)
  File "/capstone/ros/src/tl_detector/light_classification/tl_classifier.py", line 25, in __init__
    tf.import_graph_def(od_graph_def, name='')
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/importer.py", line 285, in import_graph_def
    raise ValueError('No op named %s in defined operations.' % node.op)
ValueError: No op named NonMaxSuppressionV3 in defined operations.

Wondering if u know the fix for this?

My experience.
I am using a library named POSENET.
This library automatically download some model files like model-mobilenet_v1_101.pb and model-mobilenet_v1_101.ckpt.meta into a directory named _models.
I just clean the 2 directories: _models and pycache.
So Posenet downlod them again.
My environment: ubuntu18 (host), docker; ubuntu18 (guest); python3.6.9; tensorflow1.4.0


Unabe to download any images

I have tried to download images with a simple seach but no images are found so no downloads occur.
Starting Download...

Unfortunately all 100 could not be downloaded because some images were not downloadable. 0 is all we got for this search filter!

Starting Download...

Unfortunately all 100 could not be downloaded because some images were not downloadable. 0 is all we got for this search filter!

Errors: 0

Everything downloaded!
Total errors: 0
Total time taken: 1.5911729335784912 Seconds

My Python version is 3.7.6 (Anaconda):
$ python --version
Python 3.7.6

I have google-images-download version 2.8 installed:
$ conda list | grep google
google-images-download    2.8.0                    pypi_0    pypi

What might be the issue? If I can provide any further info to help troubleshoot then please advise. I have used this package in the past in this manner and it has worked very well, so I think the error is not with the input arguments. In any event, let me take this opportunity to thank the developers of this package, it is quite useful. Thanks in advance for any help with this issue.

This looks to be a duplicate of #280 and #285, sorry I didn't check those first...


Referencing a missing levelSetId doesn't give a useful error (complains about thresholds)

I think in the case of the hackathon, we might have left out levelSetId entirely, but not sure.
FWIW- I think the error also doesn't say which metric had the issue, which would be very helpful for debugging.


Clicking the dots instead of swiping to the next page does not work

The page doesn't change when the page view dots are clicked, even though the dots respond to being tapped.

This was discussed here https://github.com/mamaral/Onboard/issues/15. It was determined that for now the page control's user interaction was to be disabled.


Patch script failed, had to adapt to proceed with build process

Was in the process of updating to Hyprland 0.40.0, build was crashing with:
meson.build:83:11: ERROR: Command `/builddir/hyprland-0.40.0/subprojects/wlroots-hyprland/./patches/apply.sh` failed with status 1.

Had to open up meson-log.txt to find out what's wrong (so you might want to do something about it, because users probably wont go that far and you'll have to ask..)
--- stderr ---
fatal: not a git repository: /builddir/hyprland-0.40.0/subprojects/wlroots-hyprland/../../.git/modules/subprojects/wlroots-hyprland
fatal: not a git repository: /builddir/hyprland-0.40.0/subprojects/wlroots-hyprland/../../.git/modules/subprojects/wlroots-hyprland

Fixed by replacing git apply $PATCH with patch -p1 < $PATCH in a build patch.
I'm building from source tar ball of a release (source-v0.40.0.tar.gz) trough cmake, nothing complicated really (package template), although Void builds packages in a chroot jail.
Just a heads up, not sure if you want to do anything about it, just felt wrong not to report.
PS: Only now noticed, probably related: (https://github.com/hyprwm/Hyprland/issues/5898)

probably how it should've been anyways.

The check applied function is needed if compiling from the same source multiple times or else it will fail to apply every time after the first time any patch is applied.


Traceback with PySide 1.0.7/1.0.8 when Outline item is clicked

From<EMAIL_ADDRESS>on 2011-10-16T04:33:29Z
Spyder Version:  2.1.0beta3
Python Version:  2.7.1
Qt Version:      4.7.4, PySide 1.0.7 on Linux
What steps will reproduce the problem?
It is hard to reproduce 100%. Seems like the problem is caused by destroyed scrollbar object. Backtrace is at http://pastebin.com/pmNHYtKm I'll attach more info if it repeats
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=800

From<EMAIL_ADDRESS>on 2011-10-16T02:57:25Z
Here is the traceback after which Spyder starts quitting with SegFault. Seems like it is caused by interaction with Outline.
Traceback (most recent call last):
File "/home/user07/spyderlib/spyderlib/widgets/sourcecode/codeeditor.py", line 345, in paintEvent
self.code_editor.scrollflagarea_paint_event(event)
File "/home/user07/spyderlib/spyderlib/widgets/sourcecode/codeeditor.py", line 1223, in scrollflagarea_paint_event
position = self.scrollflagarea.value_to_position(line_number)
File "/home/user07/spyderlib/spyderlib/widgets/sourcecode/codeeditor.py", line 364, in value_to_position
return (y-self.vsb.minimum())*self.get_scale_factor(slider)+offset
RuntimeError: Internal C++ object (PySide.QtGui.QScrollBar) already deleted.

From ccordoba12 on 2011-10-20T19:26:04Z
Labels: Component-PySide Cat-SpyderGUI

From pierre.raybaut on 2011-10-16T04:04:23Z
This is still too vague for me to understand exactly what's going on.
Keep me posted if you are able to reproduce it.

From<EMAIL_ADDRESS>on 2011-10-21T07:43:21Z
Filed to PySide, because with PyQt everything is ok. http://bugs.pyside.org/show_bug.cgi?id=1030 (round number too) =)

From pierre.raybaut on 2011-10-22T09:46:33Z
I don't know why exactly, but with PySide, the editor's (QPlainTextEdit) vertical scroll bar is at some point deleted and re-created. As a consequence, the first reference pointing to it (stored in the editor widget's constructor) is later pointing to a Python object associated with a deleted underlying C++ object...
So calling editor.verticalScrollBar() each time (instead of using the old reference) solved the problem.
See the fix in next comment.
I'm not so sure that it's a PySide bug -- all I can tell is that it just behaves differently from PyQt, but anyway informing PySide developers was a good idea as this may be a bug or a misbehavior of PySide.
Status: Started

From<EMAIL_ADDRESS>on 2011-10-21T02:47:11Z
It appeared that it is really easy to reproduce. Just start Spyder RC1 with PySide 1.0.8 and click on any Outline item. In less than a second a traceback appears in internal console.
Summary: Traceback with PySide 1.0.8 when Outline item is clicked

From pierre.raybaut on 2011-10-22T09:47:01Z
This issue was closed by revision 9e20bc8ccc63 .
Status: Fixed

From<EMAIL_ADDRESS>on 2011-10-22T14:44:16Z
The bad side that with this fix the problem is not reproducible anymore as described in http://bugs.pyside.org/show_bug.cgi?id=1030

From pierre.raybaut on 2011-10-23T00:41:57Z
If they take a look at my comments (the fact that the vertical scroll bar is deleted and recreated...) maybe they will be able to conclude on this anyway.
Note that updating to the revision just before this fix will allow to reproduce the bug.


Corrections in yaml examples for request body and response errors

What type of PR is this?
Add one of the following kinds:

bug
correction

What this PR does / why we need it:
Fixes:

Missing response error examples for Device object related errors
Incorrect request body example due to typo in tabulation
Small format corrections

Which issue(s) this PR fixes:

Fixes #10

Hi @FabrizioMoggio ! please take a look to this small fixes so we can proceed and merge. Thanks!!


dissent: 0.0.24 -> 0.0.25

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/diamondburned/dissent/releases.
meta.description for dissent is: GTK4 Discord client in Go, attempt #4 (formerly gtkcord4)
meta.homepage for dissent is: https://github.com/diamondburned/dissent
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.0.25 in filename of file in /nix/store/5yhwm059wkkds4ynkv5imab8ml3zaiqf-dissent-0.0.25

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

dissent

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/5yhwm059wkkds4ynkv5imab8ml3zaiqf-dissent-0.0.25 \
  --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 dissent https://github.com/r-ryantm/nixpkgs/archive/d59fae3afac889c006df05a9c0052792332a24de.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/d59fae3afac889c006df05a9c0052792332a24de#dissent

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/5yhwm059wkkds4ynkv5imab8ml3zaiqf-dissent-0.0.25
ls -la /nix/store/5yhwm059wkkds4ynkv5imab8ml3zaiqf-dissent-0.0.25/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 --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

1 package built:

dissent

Maintainer pings
cc @hmenke @urandom2 @Aleksanaa for testing.

lemme do it


GetWorkflowHistory with unknown workflowID need improvements

Currently, GetWorkflowHistory with unknown workflowID will return context timeout to user, and causing panic and errors in server side.
We may need some improvements on this API for graceful error return and handle.

https://github.com/uber/cadence/pull/3188 panic is fixed.
return error is wontfix, and keep it like today's WorkflowRun.Get() to return


Linking to sample app suzanne

I am attempting to run the sample app suzanne by linking to the precompiled libraries. When the sample apps are compiled by compiling the whole source code, everything works fine, however when I attempt to instead link to the precompiled libraries, I get the following error:
-- Configuring done
-- Generating done
-- Build files have been written to: /home/joel/Documents/KiwiTest/filament-suzanne/out/cmake-debug
[1/347] Compiling material materials/sandboxLitThinRefractionSsr.mat to /home/joel/Documents/KiwiTest/filament-suzanne/out/cmake-debug/generated/material/sandboxLitThinRefractionSsr.filamat
FAILED: generated/material/sandboxLitThinRefractionSsr.filamat
cd /home/joel/Documents/KiwiTest/filament-suzanne/out/cmake-debug && /home/joel/Documents/KiwiTest/filament-suzanne/bin/matc -o /home/joel/Documents/KiwiTest/filament-suzanne/out/cmake-debug/generated/material/sandboxLitThinRefractionSsr.filamat /home/joel/Documents/KiwiTest/filament-suzanne/materials/sandboxLitThinRefractionSsr.mat
ERROR: 0:1198: '*' :  wrong operand types: no operation '*' exists that takes a left-hand operand of type 'layout( column_major std140 offset=256) uniform highp 4X4 matrix of float' and a right operand of type ' global mediump 3-component vector of float' (or there is no acceptable conversion)
ERROR: 0:1198: '' : missing #endif
ERROR: 0:1198: '' : compilation terminated
ERROR: 3 compilation errors.  No code generated.

ERROR: Unable to parse fragment shader:
ERROR: 0:69: 'absorption' : no such field in structure
ERROR: 0:69: '' : compilation terminated
ERROR: 2 compilation errors.  No code generated.

Could not compile material /home/joel/Documents/KiwiTest/filament-suzanne/materials/sandboxLitThinRefractionSsr.mat```

This error has nothing to do with linking. It looks like you are using a different matc? Which version of matc are you compiling with? Did you modify the materials?

Thank you, yes I was using a slightly different version of matc.
If I want to take one of sample apps (for example, suzanne), and decouple it so I can compile it without having to recompile the entire source code, how would I do that? I have been really struggling to achieve this...
Thanks

You should be able to download one of the binary releases from the GitHub page and compile/link against the static libraries that are part of the binary release. The sample apps rely on SDL2 so you'll need to link against that as well. That said you'll also have to choose versions of the samples that match whatever release you downloaded: we often update the samples as we add new APIs, etc. so they may not compile with older versions of Filament.


#34 Support Spring Boot 3

This MR adds support for Spring Boot 3.
There's one open question: Should we also update the spring-boot.version  to 3.0.5? This would also require updating to Java 17.

Hi @csviri, does new version of starter supports building as spring boot native image?

Hi @OlegHudyma , I just basically make sanity check on pull requests as part of code review (because of the currently have limited time for this component of JOSDK),  want to make a release soon, just have to also adjust it to use the newest JOSDK.
It should come this week.
maybe @LucaMertens can tell use more if he tested with native. But my guess is it not works out of the box. But certainly something that we need to take a look in the future. (Also adjust the CI/CD to test that)
It definitely works with Quarkus: https://github.com/quarkiverse/quarkus-operator-sdk

I'm looking at building a few operators with Spring Boot 3, and I think I can test compatibility of this starter with native image and give you feedback.
I can work with the snapshot and report any issues if I found.
Which repo should I use for accessing snapshot builds?

I'm looking at building a few operators with Spring Boot 3, and I think I can test compatibility of this starter with native image and give you feedback.

that would be awesome! thx!

Which repo should I use for accessing snapshot builds?

the one from he pom, if you mean that:
 <distributionManagement>
        <snapshotRepository>
            <id>ossrh</id>
            <url>https://oss.sonatype.org/content/repositories/snapshots</url>
        </snapshotRepository>
    </distributionManagement>

Ah, sorry missed that. Thx!


musescore: add darwin-specific package

Motivation for this change
The existing package has: platforms = platforms.linux and I wanted a version that works on macOS. The added musescore-darwin package is basically a clone of the Homebrew cask version and uses the .dmg distributed by MuseScore.
Things done

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

[ ] NixOS
[x] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

/cc @vandenoever

Did you try to build it from source?

I only tried building the existing package (with allowBroken = true), which failed.

I just tried briefly by adding -f Makefile.osx to makeFlags but ran into some Qt-related errors.

@LnL7, will do.

@LnL7, I've made the change you suggested and rebased.


Future developments

Interpolator (Combine field, dofbasis, geomap)
FE Space (global numbering, Dirichlet boundary conditions, multiphysics)
Mesh (Structured mesh, Dirichlet boundary, queries, ... GMesh.jl)
FE Operator
Nonlinear operators with Automatic Differentiation
Assembler
VTK IO
Linear algebra (direct solver, CG, PETSc.jl)

Other things also worth considering:

Automatic code style revision (i.e. look for trailing white space, max line width violations, etc. ). This can be integrated in the test suite.

Some missing things:

[ ] Use abstract interface getters in FESpaces module instead of access to concrete data
[ ] Create test FE space from FE Space (w/ homogeneous Dirichlet data)
[ ] Create trial FE space from FE Space (w/ non-homogeneous Dirichlet data)
[ ] Interpolators for FE Space without Dirichlet data and the FE Spaces above with Dirichlet data, where the boundary values are respected in the first case but not in the other ones
[ ] Create multi-field Assembler with an array of FESpaces
[ ] Integrate a multi-field bilinear form with arrays of FEFunctions, etc
[ ] Create const for FEFunction in CellMaps, and fixed/free DOF values getters
[ ] Define FEOperator, which provides the residual for an array of FEFunctions (Vararg type)
[ ] Implement an evaluate and solve method for FEOperator
[ ] Consider problems with physical values that are also FEFunctions
[ ] Use automatic differentiation to obtain the system matrix from the residual value (cell-wise or global?)
[ ] Put in FESpace an array of PhysicalTags that define the Dirichlet boundary, waiting for @fverdugo work
[ ] Determine a Dirichlet data interpolator (for a FEFunction) in which for every PhysicalTag that is in the FESpace Dirichlet boundary array, we provide a vector of analytical fields
[ ] Put an assert when a function to be interpolated provides an Int
[ ] Extract the fixed dofs info from PhysicalTag in the FESpace constructor
[ ] Create strucs NonConformingFESpace with ConstrainingDOFs and ConstrainingValues arrays
[ ] Create Constrained FESpace that composes an abstract FESpace plus ConstrainingDOFs and ConstrainingValues arrays

After @fverdugo re-factorization of CellMaps

[ ] Fix the problems with evaluate of cell vectors/matrices
[ ] Create a MultiValuedCellValue that generalizes ConstantCellValue, and use it in FESpace, etc

Other misc (longer-term) issues:

[ ] Make all tet grids oriented at the very beginning
[ ] Implement change of basis for the DOF equivalence class for non-oriented meshes (e.g., unstructured hex meshes)

[ ] Put in FESpace an array of PhysicalTags that define the Dirichlet boundary, waiting for @fverdugo work

Done!

After @fverdugo re-factorization of CellMaps

Refactoring of CellValues and CellMaps will take place in branch  cell_values_and_cell_maps_refactoring

@fverdugo : I would like to create a wrapper of FESpace that includes the Dirichlet Data. However, I don't want to replicate all the methods already defined for a concrete FESpace in the wrapper. I would like to have something like this:
struct FESpaceWithDirichletData{D,Z,T,E,V<:FESpace{D,Z,T,E}} <: V
  fesp::V
  dir_data::Vector{Float64}
end

This way FESpace + Dirichlet data would always sub-type the original FESpace concretization, and it would not require any method implementation for FESpaceWithDirichletData but dir_data related things.
Unsurprisingly, it does not work, wrong subtyping. I guess it cannot check what V is and whether it allows sub-typing, etc (?)
Any other solution?

Probably, I can just do:
for op in (all FESpace interface methods)
  @eval begin
      $op(this::FESpaceWithDirichletData) = $op(this.fesp)
    end
end

since all current methods only have one FESpace as input arguments...

It works!
We could probably use it to automatically generate getters for all structs by iterating over their attributes
It is not going to work for more complicated interfaces

In julia, it is only allowed to subtype abstract types, and surely there is a good reason.
In consequence, I would not try to emulate extension of concrete types...
Any way, even if it would be allowed to extend concrete types, I would not do it here since we want to extend with Dirichlet data ANY FE space (and not only a concrete implementation)...
I would do a concrete struct like this
struct FESpaceWithDirichletData{F<:AbstractFESpace} <: AbstractFESpace
  fespace::F
  diridata::TypeThatHoldsDiriData
end

Then FESpaceWithDirichletData will delegate all queries that are required to the internal fespace

I finally did
struct FESpaceWithDirichletData{D,Z,T,E,V<:FESpace{D,Z,T,E}} <: FESpace{D,Z,T,E}
	fesp::V
	dir_data::Vector{Float64}
end

for op in (:reffes, :triangulation, :gridgraph, :nf_eqclass, :cell_eqclass,
	:num_free_dofs, :num_fixed_dofs)
	@eval begin
		$op(this::FESpaceWithDirichletData) = $op(this.fesp)
	end
end

My only concern was re-implementing all the getters, but it is not so bad

Yes! this is what I meant.
The way you avoid code repetition with the @eval is totally fine.

@fverdugo
I want to create a wrapper for FESpaces that adds dirichlet data to it, as we have commented many times. However, if my FESpace is of type X, I would like that the FESpaceWithDirichletData could be re-used in the functions defined for X type, i.e., the interpolation function for a ConformingFESpace should be also used for the FESpaceWithDirichletData that is generated with a ConformingFESpace.
I was thinking on something like this
´´´julia
abstract type str{T} end
struct str1{T} <: str{T}
a::T
end
struct str2{T,S<:str{T}}
a::T
s::S
end
strX{T} = Union{str1{T},str2{T,str1{T}}}
´´´
What do you think? str should be replace by FESpace, etc...

This is the typical situation where "traits" are useful.
For instance:

# we define here the trait
abstract type ConformityStyle end
struct ConformingStyle <: ConformityStyle end
struct NonConformingStyle <: ConformityStyle end

abstract type AbstractFESpace end

# Each FE space has to define which is its conformity style
ConformityStyle(::AbstractFESpace)::ConformityStyle = @abstractmethod

# insted of an abstract method one can also use non-conforming by default:
ConformityStyle(::AbstractFESpace) = NonConformingStyle()

then imagine that there is a query (e.g., enum_dofs) that can be optimized depending on the conformity, one can implement two variants:
function enum_dofs(f::AbstractFESpace,::NonConformingStyle)
# Implementation for **ANY** FEspace that is NonConforming
...
end

function enum_dofs(f::AbstractFESpace,::ConformingStyle)
# Implementation for **ANY** FEspace that is Conforming
...
end

# These functions are used as follows

fespace = MyFESpaceConcreteType()

enum_dofs( fespace, ConformityStyle(fespace) )

This is very usefull also if we want to dispatch on multiple traits, e.g., one could also have  other traints like OrderStyle, (FirstOrderStyle, HighOrderStyle), HomogeneityStyle (SingleRefFEStyle, MultipleRefFEStyle), which avoids ending with types like ConfonformgAndFirstOrderFESpace, ConformingAndHighOrderFESpace ...
Julia Base is full of this pattern. e.g. IndexStyle for arrays... If you think it is useful in your situation, use use traits.

I think that it makes sense to have this pattern in FESpace, since in the future we want to optimize for  different situations. The other good thing is that you do not have to anticipate for with traits you want to optimize (adding a new trait in the future is straight forward)

At the end, it is just to add a new element in the type hiearchy.
I can do it this way, it is simple.

Actually, traits are also useful since you cannot extend from concrete types. They allows to "inherit" the optimizations when you wrap a type.

Also useful since there is no multiple inheritance

I will look at this now. I have read some discussions on it.
At the end of the day, it defines a subset of types independent of the type hierarchy.
It is pretty cool.

I am not sure traits work here, since it seems that one type must have one trait, and in my case, I want it to be different, depending on the values of the type. In your previous example, I think that it should be
ConformityStyle(::Type{AbstractFESpace})::ConformityStyle = @abstractmethod

instead of
ConformityStyle(::AbstractFESpace)::ConformityStyle = @abstractmethod

So, it seems it does not solve my problem... I will do more research

Ok, using templatization I can fix it

abstract type mystyle end
struct style1 <: mystyle end
struct style2 <: mystyle end

abstract type str{T} end
mystyle(::str)::mystyle = error("Not defined")

struct str1{T} <: str{T}
	a::T
end
mystyle(::Type{str1{T}}) where {T}= style1

struct str2{T} <: str{T}
	a::T
end
mystyle(::Type{str2{T}}) where {T} = style2

struct strc{T,S<:str{T}}
	a::T
	s::S
end
mystyle(::Type{strc{T,S}}) where {T,S<:str1{T}} = style1
mystyle(::Type{strc{T,S}}) where {T,S<:str2{T}} = style2

s1 = str1(10)
s2 = str2(10)
s1c = strc(20,s1)
s2c = strc(20,s2)

mystyle(typeof(s1)) # style1
mystyle(typeof(s2)) # style2
mystyle(typeof(s1c)) #style1
mystyle(typeof(s2c)) #style2

Yes, the style getter applyes in the type instead of the value.
If you want to get the style of the wrapped object type, it can also be done. Actually we are already doing it in some places. Eg:
https://github.com/lssc-team/Numa.jl/blob/83bfd3f532430437548494670763f3a2e8950c8f/src/CellValues/Wrappers.jl#L37

Yes, it is an obvious wrapper for the type-based function.
What I meant before was that I do not think you want to implement something like
mystyle(this::strc) = mystyle(strc.s)

because we would not be able to implement the type-based function (it is not well defined).

OK, I understand now what you ment. It gives the same result, but it is better.

Implemented... I am using a wrapper function to not to show it to users.
Any syntatic sugar in Julia? I will take a look...
function interpolate(fun::Function, fesp::ConformingFESpaces{D}) where {D}
	@assert MeshConformity(fesp) == ConformingMesh() # Not implemented
	_interpolate(fun,fesp, MeshConformity(fesp))
end

function _interpolate(fun::Function, fesp::ConformingFESpaces{D}, ::ConformingMesh) where {D}
...

Target driver single field
model = GMeshModel("model.msh")
g1(x) = 2*x[1]
g2(x) = 3*x[2]
f1(x) = 5*x[3]
h1(x,u) = 3*x[3]+0.5*u
ufun(x) = 3*x[2]

trian = Triangulation(model)
quad = CellQuadrature(trian,order=2)

fespace = FESpace(model,order=1,diritags=[3,34])
V = TestSpace(fespace)
U = TrialSpace(fespace,[g1,g2])

f = cellfield(f1,trian)
nu(u) = cellfield(h1,trian,u)

dr(u,v,du) = a(v, nu(u)*gradient(du) )

r(u,v) = inner(v,f) - dr(u,v,u)

A = FEOperator(r,dr,V,U,quad)

uh = solve(A,tol=10e-3)

uexa = cellfield(ufun,model)

e = uexa - uh

a(u,u) = dr(u,u,u)

err = integrate(a(e,e),trian,quad)
@test err < 10e-10

writevtk(model,cellfields=["uh"=>uh])

@fverdugo
As suggested, I have added two new methods to Polytope
One is function generate_admissible_permutations(p::Polytope), which provides the list of possible enumerations of a polytope. An admissible enumeration is uniquely determined by the first D+1 nodes.
I have also included a new method
function _dimfrom_fs_dimto_fs(p::Polytope, dim_from::Int, dim_to::Int)

which provides the n-faces of dim_to of all n-faces of dim_from in a given polytope. Example of usage:
p = Polytope(1,1,1)
nf_vs = Gridap.Polytopes._dimfrom_fs_dimto_fs(p,2,0)
@test length(nf_vs) == 6


Any plans for a Vi mode?

Just tried out the editor and it looks great!  Sorry if this is the wrong place for this, but do you have any plans or feelings about supporting a vi mode?

I think the intention behind an editor like micro is to be more accessible and easier to use than vi. A vi-mode would go against this in my opinion.
(I am a long time vi user - both professionally and at home)

I am second to say that mental models are incompatible here, but looking at the code, it might be possible to maintain a fork.

I think what @naddeoa means is an optional vi editing mode, not vi key bindings by default.
I too think it would be nice to have that.

Whenever I use an editor with a Vi mode, I often find that I'd rather just be using Vim, so while Vim keybindings might be nice to have, I'm not sure it's worth the effort of implementing them.
I'm going to close this issue because even if we do get a Vi mode, there are many things that have to get done before that. But btw, this was the right place to post this. You can ask questions in the Gitter chat as well.

I would +1 for this, I like this idea and for me micro feels more future-robust than Vim.
For example, in vim I cannot easily remove default mappings. What needs to be done in order to get mode support?

@Anachron make a side by side table with comparison how Vim mode in micro would be better (and worse) than neovim in native mode. And note that "remove default mapping" is not an argument, because it doesn't explain why and for what anyone would need it.

I think wherever Micro sites, Vi/Vim is most definitely going to be installed (maybe not Windows). I use Vim/VscodeVim and don't really think Micro needs this. It's intended to be more feature-rich and powerful than Nano and other simple line-based text editors in the command line. Not necessary.

Is this something that could technically be implemented as a plugin in Lua?

@zyedidia
indeed, for me, the only acceptable vi-mode is "evil" from Emacs. Others are just horrible to use.

Just tried out micro editor. Is it possible to fro it to be a replacement for something like vscode in future? Thanx.

You can come a long way with a few tactical keybindings.  I think the cursor movement hjkl is most essential after that I am less bothered.
    "Alt-h": "CursorLeft",
    "Alt-j": "CursorDown",
    "Alt-k": "CursorUp",
    "Alt-l": "CursorRight",
    "Alt-H": "SelectLeft",
    "Alt-J": "SelectDown",
    "Alt-K": "SelectUp",
    "Alt-L": "SelectRight",
    "Alt-o": "EndOfLine,InsertNewline",
    "Alt-v": "SelectLine",

https://github.com/zyedidia/micro/blob/master/runtime/help/keybindings.md


Add pjrt_c_api_unimplemented pytest marker to skip unsupported tests.

Also adds test_util.pytest_mark_if_available helper function.

Looks good - I assume you're doing this so you can run things like
$ pytest tests/ -m "not pjrt_c_api_unimplemented"
If so, should we document this pattern in the developer guide?

Probably :) I filed https://github.com/google/jax/issues/13989 for now, since this is just for with the PJRT C API enabled, which isn't on by default yet and I don't think worth documenting until it is.


fix: update version, and fix DecoratedPlatformTextfield

update version of flutter_platform_widgets
fix DecoratedPlatformTextfield

Hi @robert-virkus, can you take a look please.
Thanks,

Can you bundle a new version please @robert-virkus ?

Thanks again - I released the Flutter 3.16+ version 1.0.0 now!


Accept cases where the title is: :

Accept cases where the the title is:
<JIRA-ISSUE>:<Description>
<JIRA-ISSUE>_<Description>
<JIRA-ISSUE>-<Description>
<JIRA-ISSUE>/<Description>
In other words, when there isn't space between either the jira issue and the separator and separator and the description.

Added tests
Also ran the npm formatter too.

you can use this action:
github.com/asafhalely/enforce-pr-title-style-action

Hello @wllmtrng  thank you for your pull request. I have implemented this functionality as part of #20, please upgrade to V2 and use the separator property.


Exporter crash when server is powered off

Hello,
it seems there is an issue with exporter when server is powered off (shutdown) but iLo is still reachable.
Using:
iLo version 2.98
Exporter version: latest as of today
Python version: 3.9
OS: RHEL 9.4
The error seen in the logs:
Exception occurred during processing of request from ('<IP_ADDRESS>', 53232) Traceback (most recent call last):   File "/usr/lib64/python3.9/socketserver.py", line 683, in process_request_thread     self.finish_request(request, client_address)   File "/usr/lib64/python3.9/socketserver.py", line 360, in finish_request     self.RequestHandlerClass(request, client_address, self)   File "/tmp/tmp_container_build/hpilo-exporter/src/hpilo_exporter/exporter.py", line 109, in __init__     BaseHTTPRequestHandler.__init__(self, request, client_address, server)   File "/usr/lib64/python3.9/socketserver.py", line 747, in __init__     self.handle()   File "/usr/lib64/python3.9/http/server.py", line 433, in handle     self.handle_one_request()   File "/usr/lib64/python3.9/http/server.py", line 421, in handle_one_request     method()   File "/tmp/tmp_container_build/hpilo-exporter/src/hpilo_exporter/exporter.py", line 337, in do_GET     self.watch_memory()   File "/tmp/tmp_container_build/hpilo-exporter/src/hpilo_exporter/exporter.py", line 142, in watch_memory     for cpu_idx, cpu in memory_values['memory_details_summary'].items(): KeyError: 'memory_details_summary'
As soon as server is powered on (doesnt have to be in the OS) - the iLo exporter works fine.

fyi, I fixed it in my fork with a bit more thorough check whether memory returns any values (I guess when server is off, memory stats are empty)