[az containerapp env certificate upload] Container Apps Environment - InternalServerError on adding a certificate via KeyVault and Managed Identity

Describe the bug
When trying to add a new certificate to a Container Apps Environment using the KeyVault option with an assigned Managed Identity, the process fails with (InternalServerError) Internal server error occurred.
The Managed Identity was both assigned to the Env using the Settings / Identity tab in the Azure Portal, and providing it via the --identity option in the cli.
It also has both Key Vault Secrets Officer and Key Vault Certificates Officer roles, as indicated in this article.
This happens in all cases:

Azure Portal
Azure Preview Portal
az CLI using containerapp extension 0.3.52

Thank you for your assistance.
Related command
az containerapp env certificate upload -g <ResourceGroup> --name <Environment> --akv-url <Certificate's Secret url> --identity <Resource ID for Managed Identity>
Errors
(InternalServerError) Internal server error occurred. correlation ID: 87b5614d-1a2f-4a5e-a3ae-2da7c67db809
Issue script & Debug output
urllib3.connectionpool: Starting new HTTPS connection (1): management.azure.com:443
urllib3.connectionpool: https://management.azure.com:443 "PUT /subscriptions/SUB/resourceGroups/RG/providers/Microsoft.App/managedEnvironments/CAE/certificates/<autogenerated_name>?api-version=2024-02-02-preview HTTP/1.1" 500 203
cli.azure.cli.core.util: Response status: 500
cli.azure.cli.core.util: Response headers:
cli.azure.cli.core.util:     'Cache-Control': 'no-cache'
cli.azure.cli.core.util:     'Pragma': 'no-cache'
cli.azure.cli.core.util:     'Content-Length': '203'
cli.azure.cli.core.util:     'Content-Type': 'application/json; charset=utf-8'
cli.azure.cli.core.util:     'Expires': '-1'
cli.azure.cli.core.util:     'x-ms-ratelimit-remaining-subscription-resource-requests': '99'
cli.azure.cli.core.util:     'api-supported-versions': '2022-03-01, 2022-06-01-preview, 2022-10-01, 2022-11-01-preview, 2023-04-01-preview, 2023-05-01, 2023-05-02-preview, 2023-08-01-preview, 2023-11-02-preview, 2024-02-02-preview, 2024-03-01'
cli.azure.cli.core.util:     'X-Powered-By': 'ASP.NET'
cli.azure.cli.core.util:     'x-ms-failure-cause': 'service'
cli.azure.cli.core.util:     'x-ms-request-id': '87b5614d-1a2f-4a5e-a3ae-2da7c67db809'
cli.azure.cli.core.util:     'x-ms-correlation-request-id': '87b5614d-1a2f-4a5e-a3ae-2da7c67db809'
cli.azure.cli.core.util:     'x-ms-routing-request-id': REDACTED
cli.azure.cli.core.util:     'Strict-Transport-Security': 'max-age=31536000; includeSubDomains'
cli.azure.cli.core.util:     'X-Content-Type-Options': 'nosniff'
cli.azure.cli.core.util:     'X-Cache': 'CONFIG_NOCACHE'
cli.azure.cli.core.util:     'X-MSEdge-Ref': REDACTED
cli.azure.cli.core.util:     'Date': 'Wed, 05 Jun 2024 13:36:12 GMT'
cli.azure.cli.core.util: Response content:
cli.azure.cli.core.util: {"error":{"code":"InternalServerError","message":"Internal server error occurred. correlation ID: 87b5614d-1a2f-4a5e-a3ae-2da7c67db809","details":null,"target":null,"additionalInfo":null,"traceId":null}}
cli.azure.cli.core.azclierror: Traceback (most recent call last):
File "C:\Users\USER.azure\cliextensions\containerapp\azext_containerapp\containerapp_env_certificate_decorator.py", line 160, in create_or_update
r = self.client.create_or_update_certificate(self.cmd, self.get_argument_resource_group_name(), self.get_argument_name(), self.cert_name, self.certificate)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "D:\a_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/command_modules/containerapp/_clients.py", line 692, in create_or_update_certificate
File "D:\a_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/util.py", line 1007, in send_raw_request
azure.cli.core.azclierror.HTTPError: Internal Server Error({"error":{"code":"InternalServerError","message":"Internal server error occurred. correlation ID: 87b5614d-1a2f-4a5e-a3ae-2da7c67db809","details":null,"target":null,"additionalInfo":null,"traceId":null}})
Expected behavior
A new certificate entity being created for the CAE that is associated to the indicated KeyVault Certificate through the Managed Identity.
Environment Summary
azure-cli                         2.61.0
core                              2.61.0
telemetry                          1.1.0
Extensions:
containerapp                      0.3.52
cosmosdb-preview                   1.0.0
interactive                        0.5.3
Dependencies:
msal                              1.28.0
azure-mgmt-resource               23.1.1
Python location 'C:\Program Files\Microsoft SDKs\Azure\CLI2\python.exe'
Extensions directory 'C:\Users\USER.azure\cliextensions'
Python (Windows) 3.11.8 (tags/v3.11.8:db85d51, Feb  6 2024, 22:03:32) [MSC v.1937 64 bit (AMD64)]
Additional context
Curiously, it seems that even though the CLI is using the "latest" containerapp extension, it is using the 2024-02-02-preview API Version, when apparently a stable 2024-03-01 is already available?
Thank you!

Thank you for opening this issue, we will look into it.

Hi @omni-htg
I checked the error log, error code is ForbiddenByFirewall.
error message:
Client address is not authorized and caller is not a trusted service
You can refer this to authorize the client address  IP to the key vault:
https://stackoverflow.com/questions/52985252/client-address-is-not-authorized-and-caller-is-not-a-trusted-service-in-azure

Thank you for looking into this, @Greedygre .
When you mention

The managed environment's OutBoundIP

do you mean the Static Ip ?
I currently have my Container Apps (and environment) in a subnet on the same VNet as the KeyVault.
What I did not do was:

Add that Static Ip to the Firewall IP list.
Add the Azure.KeyVault service endpoint to the Container Apps subnet.

What is in place is:

The "Allow trusted Microsoft services to bypass this firewall" exception.
The Container Apps themselves seem to be able to access the KeyVault when adding a secret (or at least, they get added and don't report any issues).

I will do some trial and error on my side, but any confirmation on yours would be greatly appreaciated!
If possible, I'd love to avoid having to add the CAE IP to the Firewall.

On another note, is it intended that I don't recieve the error message ForbiddenByFirewall on the CLI, and that the "latest" extension is not using the latest API version?
Thanks a ton!

ForbiddenByFirewall
Hi @omni-htg

The ForbiddenByFirewall is not shown in CLI now.
The OutBoundIP, you can get it from your containerapp with command:
 az containerapp show -n {} -g {} --query "properties.outboundIpAddresses"

In my environment, the CAE is an Internal one, only available to the VNet where all the other Azure services (including the KeyVault) have been set up -- I believe this is why properties.outboundIpAddresses is null.
I have tried to add the staticIp property from the CAE into the Firewall IP list in the KV, but was disallowed for it being a "private IP".
Please advise.

@omni-htg
Can I know what you mean “I currently have my Container Apps (and environment) in a subnet on the same VNet as the KeyVault.”？
Azure keyvault supports add vnet/private endpoint, can you add Container apps env subnet to the allow list or create private endpoint for AKV, then container apps can access AKV via private endpoint.
thanks
Vincent

I apologize, tried to explain in a quick fashion and made it more convoluted.

Azure keyvault supports add vnet/private endpoint, can you add Container apps env subnet to the allow list

This is exactly what was needed.
Initially I had the KV as part of the VNet by "allowing" a KeyVault-specific Subnet, completely missing the point.
Once I added the Container Apps subnet it worked flawlessly.
So now I can close this, thank you!
I'll leave with the suggestion that, if possible, add a more verbose message on Azure Portal and CLI when it gets blocked by the Firewall like in my case.


Optimization: batch update cancelledOrFilled amount

Inside the loop in the settleRing method,  this function is called multiple times,
     // Update fill records
            if (state.buyNoMoreThanAmountB) {
                delegate.addCancelledOrFilled(state.orderHash, nextFillAmountS);
            } else {
                delegate.addCancelledOrFilled(state.orderHash, state.fillAmountS);
            }

We can construct an array first then pass it to batchTransferToken (probably rename it to batchTransferTokenAndUpdateTradeHistory.

That's something I did in #309

Oh, will take a look.  We credit it to you if the changes are similar.
Sent from my iPhone

On Apr 23, 2018, at 3:49 AM, Brechtpd<EMAIL_ADDRESS>wrote:
That's something I did in #309
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

fixed


Invalid code generation for HashMap keys

In YAML, map keys are allowed to be complex types, such as arrays or maps. For example, a Rust HashMap<Vec<u32>, String> could be written in YAML as [1, 2, 3]: foo.
However, serializing HashMap<Vec<u32>, String> with serde-yaml 0.7 gives back this string: [+   1  , 2  , 3]: foo. The + sign is a problem, because +     1 parses as a string, while +1 parses as a number, which means the YAML can't be deserialized correctly.

After looking through the code some, it looks like this is a bug on rust-yaml's part. I'll take this over there.


Exception instance returned from resolver is raised within executor instead of being passed down for further resolution.

Today when experimenting with 3rd party library for data validation (https://github.com/samuelcolvin/pydantic) I've noticed that when I take map of validation errors from it (which is dict of lists of ValueError and TypeError subclasses instances), I need to implement extra step to convert those errors to something else because query executor includes check for isinstance(result, Exception). This check makes it raise returned exception instance, effectively  short-circuiting further resolution:
https://github.com/graphql-python/graphql-core-next/blob/master/src/graphql/execution/execute.py#L731
The fix for issue was considerably simple to come up with: just write util that converts those errors to a dict as they are included in my result's validation_errors key, but I feel such boilerplate should be unnecessary:
try:
    ... run pydantic validation here
except (PydanticTypeError, PydanticValueError) as error:
    return {"validation_errors": flatten_validation_error(error)}

Is this implementation a result of something in spec, or mechanism used to keep other feature's (eg. error propagation) code simple? I think we should consider supporting this use-case. Considerable number of libraries use exceptions for messaging (eg. Django with its ValidationError and bunch of core.exceptions.*)

This is exactly what GraphQL.js does as well:
https://github.com/graphql/graphql-js/blob/fbed2c2356eeb83d84280ae3c0635eb54e04a419/src/execution/execute.js#L809
However, the raised error is always caught and wrapped into a located error in the complete_value_catching_error method.


Support php 8.1

From what I have tested the tests doesn't pass with php 8.1 version, we should correct this
Here is log output :
There were 15 failures:

1) Yasumi\tests\Base\HolidayBetweenFilterTest::testHolidaysBetweenDateRange
Failed asserting that an array has the key 'summerTime'.

yasumi/tests/Base/HolidayBetweenFilterTest.php:56
yasumi/vendor/phpunit/phpunit/phpunit:92

2) Yasumi\tests\Base\HolidayBetweenFilterTest::testHolidaysBetweenDateRangeWithDateTimeImmutable
Failed asserting that an array has the key 'summerTime'.

yasumi/tests/Base/HolidayBetweenFilterTest.php:111
yasumi/vendor/phpunit/phpunit/phpunit:92

3) Yasumi\tests\Base\HolidayBetweenFilterTest::testHolidaysBetweenDateRangeExclusiveStartEndDate
Failed asserting that an array has the key 'summerTime'.

yasumi/tests/Base/HolidayBetweenFilterTest.php:200
yasumi/vendor/phpunit/phpunit/phpunit:92

4) Yasumi\tests\Base\HolidayFiltersTest::testSeasonalHolidaysFilter
Failed asserting that an array has the key 'summerTime'.

yasumi/tests/Base/HolidayFiltersTest.php:154
yasumi/vendor/phpunit/phpunit/phpunit:92

5) Yasumi\tests\Base\HolidayOnFilterTest::testHolidaysOnDate
Failed asserting that an array has the key 'summerTime'.

yasumi/tests/Base/HolidayOnFilterTest.php:56
yasumi/vendor/phpunit/phpunit/phpunit:92

6) Yasumi\tests\Base\HolidayOnFilterTest::testCorrectNumberOfHolidaysOnDate
Failed asserting that 1 is greater than 1.

yasumi/tests/Base/HolidayOnFilterTest.php:103
yasumi/vendor/phpunit/phpunit/phpunit:92

7) Yasumi\tests\Denmark\DenmarkTest::testSeasonalHolidays
Failed asserting that an array has the key 'summerTime'.

yasumi/tests/YasumiBase.php:89
yasumi/tests/Denmark/DenmarkTest.php:84
yasumi/vendor/phpunit/phpunit/phpunit:92

8) Yasumi\tests\Denmark\SummerTimeTest::testSummerTime
Failed asserting that null is an instance of class "Yasumi\Holiday".

yasumi/tests/YasumiBase.php:117
yasumi/tests/Denmark/SummerTimeTest.php:53
yasumi/vendor/phpunit/phpunit/phpunit:92

9) Yasumi\tests\Denmark\SummerTimeTest::testTranslation
Failed asserting that null is an instance of class "Yasumi\Holiday".

yasumi/tests/YasumiBase.php:225
yasumi/tests/Denmark/SummerTimeTest.php:71
yasumi/vendor/phpunit/phpunit/phpunit:92

10) Yasumi\tests\Denmark\SummerTimeTest::testHolidayType
Failed asserting that null is an instance of class "Yasumi\Holiday".

yasumi/tests/YasumiBase.php:273
yasumi/tests/Denmark/SummerTimeTest.php:82
yasumi/vendor/phpunit/phpunit/phpunit:92

11) Yasumi\tests\Denmark\WinterTimeTest::testWinterTime
Failed asserting that null is an instance of class "Yasumi\Holiday".

yasumi/tests/YasumiBase.php:117
yasumi/tests/Denmark/WinterTimeTest.php:57
yasumi/vendor/phpunit/phpunit/phpunit:92

12) Yasumi\tests\Denmark\WinterTimeTest::testHolidayType
Failed asserting that null is an instance of class "Yasumi\Holiday".

yasumi/tests/YasumiBase.php:273
yasumi/tests/Denmark/WinterTimeTest.php:83
yasumi/vendor/phpunit/phpunit/phpunit:92

13) Yasumi\tests\Netherlands\NetherlandsTest::testSeasonalHolidays
Failed asserting that an array has the key 'summerTime'.

yasumi/tests/YasumiBase.php:89
yasumi/tests/Netherlands/NetherlandsTest.php:88
yasumi/vendor/phpunit/phpunit/phpunit:92

14) Yasumi\tests\Netherlands\SummertimeTest::testTranslation
Failed asserting that null is an instance of class "Yasumi\Holiday".

yasumi/tests/YasumiBase.php:225
yasumi/tests/Netherlands/SummertimeTest.php:72
yasumi/vendor/phpunit/phpunit/phpunit:92

15) Yasumi\tests\Netherlands\SummertimeTest::testHolidayType
Failed asserting that null is an instance of class "Yasumi\Holiday".

yasumi/tests/YasumiBase.php:273
yasumi/tests/Netherlands/SummertimeTest.php:83
yasumi/vendor/phpunit/phpunit/phpunit:92

FAILURES!
Tests: 16347, Assertions: 50486, Failures: 15.
Script vendor/bin/phpunit handling the test event returned with error code 1

@sylvaincombes Thanks for spotting this. From a first look, it is related to the summer/wintertime calculation. Seems some transitions are missing. I will investigate further.

Issue with PHP8.1: https://github.com/php/php-src/issues/7752

Based on the comments of the issue (https://github.com/php/php-src/issues/7752) this issue has been fixed and looks to be set to be included in PHP 8.1.6

Tested it locally with 8.1.6 and the tests run correctly.
PHP 8.1.6 (cli) (built: May 15 2022 06:10:08) (NTS)
Copyright (c) The PHP Group
Zend Engine v4.1.6, Copyright (c) Zend Technologies
    with Xdebug v3.1.4, Copyright (c) 2002-2022, by Derick Rethans

................                                            16418 / 16418 (100%)

Time: 00:06.068, Memory: 200.00 MB

OK (16418 tests, 50891 assertions)

@sylvaincombes Could you try with the updated 8.1.6 version as well and let me know if the tests run successfully? Thanks!

Hello @stelgenhof I tested it locally with php 8.1.6 branch master and develop, all is green

@sylvaincombes Thanks for verifying! I'll close this issue and mention that only PHP >= 8.1.6 is supported.


Copyright should only contain year of file creation

We're currently enforcing every file to contain a copyright header, which is mandatory for Adobe OpenSource projects.
Using the https://github.com/mycila/license-maven-plugin/, we're enforcing every header to contain a date in the format of
<year-of-project-inception> - <current-year>
even though that's completely unnecessary.
We should enforce the file to contain a date in the format of
<year-of-file-creation>
this will still comply to the mandatory copyright header format.

Looking at the https://github.com/mycila/license-maven-plugin/ documentation, this may be hard to configure.
Even though this would be an easy check using e.g. a regular expression, the license plugin is built not only to enforce the existence of a header, but it can also add the header.
Because of this feature, the plugin reads the template and replaces any properties found, in our case ${license.git.copyrightYears} which translates to <year-of-project-inception> - <current-year>.
It seems that other people are having the same problem:
https://github.com/mycila/license-maven-plugin/issues/124
In version 4.0-rc1, a feature was added that allows tracking the file creation date using Git:
https://github.com/mycila/license-maven-plugin/milestone/14?closed=1
https://github.com/mycila/license-maven-plugin/pull/145
I'll have a look if that works for us without breaking this project and changing all existing copyright headers...

also see discussion in #231

PoC:
adding <license.git.copyrightLastYearMaxCommitsLookup>100000</license.git.copyrightLastYearMaxCommitsLookup> to the configuration to enable finding a file's creation date even in large GIT histories and changing the line in the copyright template to  Copyright ${license.git.copyrightCreationYear} Adobe. works with version `4.0.rc2.
Caveat:
we need to fix all headers in all files if we do this change.
Annoying:

if a header is in the wrong format (e.g. wrong year) the plugin will complain:
[WARNING] Missing header in: /Users/franken/github/adobe/S3Mock/pom.xml
The error message couldn't be less helpful - the header is not missing...

I haven't found a way to change this to useful output.
Only the whole message could be changed, it will never change based on the problem found by the plugin.

When you locally change a file, for some reason the plugin thinks it should use the current year instead of the creation year.
So if we always run the check on every maven build, the build will always fail if you change a file. (I see this right now - I changed the plugin's configuration and now it complains about the main pom.xml header to contain the year 2020. But once I commit, it must contain the year 2017.

We could mitigate this by checking the license in a profile.
This would make things less understandable though - the build would only fail when you know which profiles to activate, so a contributor would see the build passing locally, and it would fail with an obscure error message in the PRB.
I could also open an issue against the plugin and provide a fix. It seems that this year lookup is supposed to work around the fact that previously uncommitted files do not have commits to look up the year. So they're checking to see if it's currently unstaged and return the current year before even checking whether commits exist:
https://github.com/mycila/license-maven-plugin/blob/master/license-maven-plugin-git/src/main/java/com/mycila/maven/plugin/license/git/GitLookup.java#L148
This should be done as a fallback after we know that there are no commits for a file.
=========
We could let the plugin automatically change/add the headers.
I'm personally not a big fan of having third party plugins change our code, but it would get rid of this problem from a contributor's point of view.
Vladimir even suggested to auto-format our code instead of just checking with the Checkstyle plugin (again a plugin with sometimes unhelpful error messages).

I looked into this two years ago, the option was not available in the plugin and just learned to live with it.
Updating the copyright header is a bit tedious but not too hard to do.
The license-maven-plugin has since added license.git.copyrightCreationYear which means "file creation date in GIT" to the copyright instead of license.git.copyrightYears which lists inceptionYear and currentYear, but it requires lookups in the GIT history.
This works for some files, but does not (really) work for others - or, rather, it probably depends on interpretation.
If I create a file src/main/java/com/test/Test.java in 2018, this year will be enforced in the copyright header.
If I later refactor this file in 2022 and move it to a different package src/main/java/com/test/newpackage/Test.java, the year 2022 will be enforced since that is the creation date of the file at that location.
Not sure is this would be correct since the file itself was created in 2018, and the copyright started there...
I'll close this issue and maybe try out the new version with file creation date.

In case I (or someone else) picks this up again:
I change the copyright header from
 Copyright ${license.git.copyrightYears} Adobe.
to
 Copyright ${license.git.copyrightCreationYear} Adobe.
and ran the command
mvn license:format
to re-format all files in the repo.


[FEAT] Add support for live imaging

Add support for live imaging

should be threaded, and use an image queue to read the buffer

CHECK WITH THE BOSS

Need to disable taking images after movement when live imaging. Should return to previous setting when finished

[ ] enable click to move


Not able to upload both a pit map and event schedule

When going to upload a pit map and event schedule the system will only let me do one or the other and it always labels as the "Pit Map" no matter if I hit upload schedule or upload pit map.
I uploaded the pit map and it filled both upload boxes:

Did the same as above but uploaded the schedule in the schedule part and it filled both boxes:

In both instances above it always uploads, but I can only upload 1 thing, and it always labels at "Pit Map"

Fixed


Return with world height instead of local height

Fix for @antzGames's comment.
So we should scale the height with y scale value because currently the height value is the local height value, not the height in the world.

Tested on the runtime.  Looks good.  Used:
api "com.github.DGZT:Mundus:world-height-instead-of-local-height-SNAPSHOT"
Thank you for fixing this.  So cool I can test PRs on the runtime very easy now.


Did you try to add SE Block?

If adding se layer, will there be any prompt?

Yes, the SE block can bring additional improvement.

Thanks!


Release/1.8.1

Summary

Concisely describe the issue and your solution:
Link to existing issue (type "#" to check):
Specify the tests done so far:

Solutions (Optional)
Please describe the details of your solution if it might be too complicated for reviewers.

Codecov Report

Merging #297 (ddb71db) into master (6c28457) will increase coverage by 0.32%.
The diff coverage is 49.50%.

@@             Coverage Diff              @@
##             master     #297      +/-   ##
============================================
+ Coverage     43.04%   43.37%   +0.32%
- Complexity     1587     1608      +21
============================================
  Files           236      237       +1
  Lines         12752    12783      +31
  Branches       1662     1663       +1
============================================
+ Hits           5489     5544      +55
+ Misses         6579     6540      -39
- Partials        684      699      +15

Impacted Files
Coverage Δ
Complexity Δ

...main/java/com/webank/weid/protocol/cpt/Cpt109.java
0.00% <ø> (ø)
0.00 <0.00> (ø)

...a/com/webank/weid/service/impl/CptServiceImpl.java
86.07% <0.00%> (-2.24%)
39.00 <0.00> (ø)

...in/java/com/webank/weid/util/OffLineBatchTask.java
0.00% <0.00%> (ø)
0.00 <0.00> (ø)

...ervice/impl/engine/fiscov2/CptServiceEngineV2.java
50.26% <23.52%> (-1.51%)
29.00 <2.00> (ø)

.../main/java/com/webank/weid/util/DataToolUtils.java
65.02% <32.60%> (-4.33%)
144.00 <6.00> (-1.00)

...a/com/webank/weid/service/fisco/WeServerUtils.java
80.16% <40.00%> (-8.62%)
13.00 <0.00> (ø)

.../main/java/com/webank/weid/config/FiscoConfig.java
70.37% <50.00%> (-0.06%)
24.00 <0.00> (+2.00)
:arrow_down:

...om/webank/weid/service/impl/PolicyServiceImpl.java
57.77% <50.00%> (-0.65%)
12.00 <0.00> (ø)

.../com/webank/weid/service/impl/WeIdServiceImpl.java
59.86% <56.52%> (-0.88%)
70.00 <2.00> (+2.00)
:arrow_down:

...e/impl/engine/fiscov2/AuthorityIssuerEngineV2.java
71.38% <62.74%> (-1.80%)
44.00 <7.00> (+7.00)
:arrow_down:

... and 15 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 5fef93f...ddb71db. Read the comment docs.


[SPARK-28527][FOLLOW-UP][SQL][TEST] Add guides for ThriftServerQueryTestSuite

What changes were proposed in this pull request?
This PR add guides for ThriftServerQueryTestSuite.
Why are the changes needed?
Add guides
Does this PR introduce any user-facing change?
No.
How was this patch tested?
N/A

cc @HyukjinKwon @MaxGekk @beliefer

Test build #114039 has started for PR 26587 at commit 4fb30fe.

Merged build finished. Test PASSed.

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

Test build #114039 has finished for PR 26587 at commit 4fb30fe.

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

Merged build finished. Test PASSed.

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

LGTM. I learned it.


subset of reads during conversion

Hey @Psy-Fer,
This enhancement request [https://github.com/hasindu2008/slow5tools/issues/69] from @mbhall88 for slow5tools could not be honoured because of the amount of effort required to implement such a feature due to the complexity of the FAST5 handling code. Could you please evaluate how feasible this is in blue-crab when doing POD5->BLOW5 conversion?

Hey,
Yea so is the input just a flat file of read_ids
Then they are the only reads you want converted?
So kind of a get->convert in one step?

Correct 👍


[MM-20456] Typescript Conversion of integrations/edit_outgoing_webhook

Summary
In this PR I have converted the integrations/edit_outgoing_webhook module to typescript as stated in the issue #15484 on github. This PR fixes any loosely typed variables and has implemented strict types to the areas required.
Ticket Link
Fixes https://mattermost.atlassian.net/browse/MM-20456

Hello @Ayanrocks,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

Any review updates? @hanzei @nevyangelova @devinbinnie

Hey @Ayanrocks LGTM! Can you please update the snapshots?

Thanks @Ayanrocks , you just have a lint error /root/mattermost/mattermost-webapp/components/integrations/edit_outgoing_webhook/edit_outgoing_webhook.tsx   9:9  error  'ServerError' is defined but never used  @typescript-eslint/no-unused-vars

@devinbinnie

/update-branch

/update-branch


Documenting AWS Lambdas / FaaS

I was wondering how would you approach the documentation of an AWS Lambda function (or anything similar).
The handler would be:
// ./src/createRecord.js
module.exports = (event, context, callback) => {
  // do your thing here using `event` and `context`

  callback({}, { success: true})
  // or `callback(new Error())` for errors
}

The above function has multiple ways of being called. Through a REST API, directly invoking it using the AWS SDK, listening to an SNS topic, etc. I have nothing enabled so only direct invoking will work and only a member of the team can invoke it. I want to describe that to ESDoc and include it in the generated documentation, to let other team members know that this function can be invoked directly.
Also, I want to let them know that they don't have to wait for a response since the function will return nothing important.
Also, I want to let them know that they don't have to include a context or a callback param cause those are being handled internally through AWS. I want to define a custom input that isn't necessarily the same as the function's signature.
I wrote the above as a question and also as a proposal in case ESDoc wants to take the route to document the uprising FaaS technology such as AWS Lambdas.
Thank you

Why not just provide a description above the lambda (you can use a @desc tag), and then describe the parameters with @param tags? If you are consistent in you documentation, by not setting a @return tag, you can imply that there is no return. In specifying the context and callback parameters you can point out they are nullable.
I'm not sure what you mean by a "custom input that isn't necessarily the same as the function's signature."
I don't see what additional features ESDoc could or need to provide to document any AWS Lambdas. Aren't they just functions?

Aren't they just functions?

No, they are not just functions, meaning they have a different purpose and cannot be treated as regular functions.
One cannot directly call them from a JS module. She can but won't be much useful.
I am thinking of having a way to specify that a function:

Must be called by other means, direct invocation through AWS-SDK, listens to SNS topic, etc.
Cannot be called directly
The function's signature have no meaning to the caller, I have to specify a different input schema

I could, of course, use the regular ESDocs tags to define that but the generated HTML wouldn't be that descriptive. That's why I asked how are you using the regular tags to define that.

@kbariotis
Did you work it out finally?
I have similar request to run esdoc on a serverless project (nodejs), but there is nothing generated.
Any sample for me?


remove gregor from vendor

Goal is to remove Gregor from vendor, and to file the keybase1 protocol in its own directory.

can you change the name of the PR?

ok @maxtaco, you can check this out now, although I wouldn't be surprised if Flow doesn't work.

In general this looks great.
We have an issue now that our protocol/ directory hierarchy is now inconsistent.  What do you think about fixing it?
Also, what are your thoughts about go/gregor and go/gregor/storage?

What do mean exactly about the hierarchy? What should we try to do? I just want to make sure I am thinking about it right.
I can make that storage change no problem.

@maxtaco This passes CI now! You can check it out and make sure it looks ok to you. Also, @chrisnojima might want to look at this as well. The major thing to look out for is the changes to the protocol directory.

👍

👍


V2Wizard: Improve validators

Follow up to https://github.com/RedHatInsights/image-builder-frontend/pull/1548
This cleans up implementation for the AWS account ID ad GCP e-mail validators.

Also linking Thomas' original proposal on how to clean up the validators a bit: https://github.com/RedHatInsights/image-builder-frontend/pull/1548#discussion_r1450619968


Remove hide useless comments From Pr's

Closes #1543
As per https://github.com/sindresorhus/refined-github/issues/1543#issuecomment-423378536

I think @hkdobrev mentioned that PRs get useless comments too so it shouldn’t be disabled that.

There was not a conflict with this base

I see, I am good with that – merge this one.   Thanks!
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10

From: Federico Brigante<EMAIL_ADDRESS>Sent: Wednesday, September 26, 2018 1:21:10 AM
To: sindresorhus/refined-github
Cc: Annastacia007; Comment
Subject: Re: [sindresorhus/refined-github] Remove hide useless comments From Pr's (#1544)
Let’s merge this one for now, the feature doesn’t work well on PRs
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://nam01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fsindresorhus%2Frefined-github%2Fpull%2F1544%23issuecomment-424598521&data=02|01||73a464fefa7f484a377508d623784073|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636735396707763374&sdata=Usrpvs%2FPHY0bkokBVdOxKSuG6KDVkXLyZHe3b5uarps%3D&reserved=0, or mute the threadhttps://nam01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAULBFCj33dc3JRqhAtr-oltE-NpfQBOHks5uexzWgaJpZM4W0PyA&data=02|01||73a464fefa7f484a377508d623784073|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636735396707763374&sdata=RHOUOve3vaOwjLGnPJry1HAVlOvj8LcQANOflmFDqJ8%3D&reserved=0.

@sindreshorhus @bushes

Yes, please merge this. It also hides PR approval comments with only +1 or similar ...
Thanks for the awesome plugin!!!

@hkdobrev can you merge this?

Thanks for the PR, there was a lot of feedback on this!


B #1826 Improve error messages in vCenter driver

Apply to 5.6 and also to 5.4.9

Please avoid merging, use pull --rebase instead. I'd appreciate it if you could close this issue and create another PR without the various merges.

I see additional commits not related with this. Can you please close this PR and open a new one?


Create a Riot application request for the bot

Currently, every request requires an API key which has to be refreshed every day. This is not sustainable. We can apply for a Riot token which doesn't expire here: https://developer.riotgames.com/docs/portal#product-registration_application-process
Apply the bot for a permanent API token on that link.
If it's not possible to do so, discuss it with the team.

Please notices that:

We can't use this API for MMR or ELO calculation.
This API its just for this project, if we want to make another project we must have another API.
Publish a project that doesn't properly secure your API key.
We can't use any of their official logos(Riot logos) but can use their art.

The complete "Terms of use" link for reference: https://developer.riotgames.com/terms
The complete "Legal Jibber Jabber" link for refence: https://www.riotgames.com/en/legal


Bug where noqa or unimport:skip is ignored

I've found an issue with # noqa and # unimport:skip handling with v0.6.6. I don't get this issue with v0.3.0.
The snippet below works as expected:
import math
import collections  # noqa

--- foo.py

+++

@@ -1,2 +1 @@

-import math
 import collections  # noqa

As soon as a newline is introduced between those two imports then the diff changes incorrectly:
import math

import collections  # noqa

--- foo.py

+++

@@ -1,3 +1 @@

-import math

-import collections  # noqa

Hi, thank you!
This bug only exists in python3.8, I'll fix it soon.


Langchain

Hi @doomgrave , thanks for the PR. Happy to help facilitate langChain interop...but can you give a description of which scenarios this helps support? Thanks!

Hi @doomgrave , thanks for the PR. Happy to help facilitate langChain interop...but can you give a description of which scenarios this helps support? Thanks!

Sorry Scott, i pushed to main for error, not really skilled with git!
Anyway i made a simple interface to use the Llama-cpp model loded with Guidance and also be able to use Langchain Embeddings and Langchain generations/chains.
The advantage its simply that you load the model once without loading/unloading it from memory. I've put an use description in the class. Feel free to inspect the idea.
    #llama.cpp embedding models using Guidance
    #To use, you should have the llama-cpp-python and langchain library installed.
    #LlamaCpp istance must have embedding = true.
    #USAGE EXAMPLE (using Chroma database):

        llama2 = guidance.models.LlamaCpp(model=modelPath,n_gpu_layers=-1,n_ctx=4096,embedding = true)
        embeddings = GuidanceLlamaCppEmbeddings(client=llama2)
        vectordb = Chroma(persist_directory={path_to_chromadb}, embedding_function=embeddings)


Driver.js Invisble Element on Driver Start

Hi,
I am facing two issues as can be seen in the below video

The main element to be highlighted gets hidden
Scrollbar appears on highlighting elements to the far right

You may want o see a codepen but I can probably share the dev URL of our production site for your reference

We just released v1.0 of driver.js which has been rewritten from the ground up. This issue should no longer exist. Please have a look at the docs and do let me know if you face any issues.


Determine Test Naming Conventions

Tests conventions

Class name ends with postfix 'Test'
Test-Data-Generator-Classes have prefix 'Test'
Test-Method-Names should look like:

testMethod_State_Happens()
testMethod_Given_Then()
testToString_DefaultObject_DoesNotThrowException()
testFindById_ExistingImpact_ReturnImpact()

Test-Methods have (nearly always) three parts: '// given, // when, // then'
For Asserts we use AssertJ - fluent assertions java library

assertThat( actual ).isEqualTo( expected )
assertThat( actualList).hasSize( expectedSize )
assertThat( actualList).contains( expectedValue ...)

For general SetUp in one Test-Class we Use @Before and all other useful JUnit Annotations

needs a review.

needs a review.


gl generator

Why do you need to use it?

I’m using the GL bindings that it exposes. I’m using SDL only to get a window and for file operations. Everything graphics is direct calls to opengl

I see, thanks!


modules/k8s: Tunnel can have a custom logger

With #510, there is now an ability to provide your own custom logger, which is awesome! However, the Tunnel struct used for port-forwading in the k8s module doesn't allow its logger to be customized. This PR proposes to use a custom logger in the for Tunnel.

I'm not familiar with this code, so assigning to @yorinasub17 to take a look when he's back.

Looks like the test failed, but the failures are unrelated to this change so we can get this merged once you address the style issue and update to include the latest changes!

Thanks for the contribution! This is good to go, so will merge and release now.


feat: public properties can have a leadingUnderscore

There could be the need of having a trailUnderscore in public property name.
In my company pagination is defined like this:
{ "items": [], "_metadata": {...} }

Please target the develop branch

Done


Added Tomo, Pomodoro app for MacOS

https://github.com/fluttergems/awesome-open-source-flutter-apps/issues/675

Thanks for sending the PR. 🚀


Closes: #1

Updated validations.py
Fixed the behaviour
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[x] Tests pass
[x] Appropriate changes to README are included in PR

@googlebot I signed it!


json example on drill site doesn't work

Using drill 1.21
The example on page https://drill.apache.org/docs/json-data-model/ of querying the citylots.json does not work:
SELECT features[0].properties.MAPBLKLOT, FROM dfs.`/home/mbeckerle/Downloads/citylots.json.gz`;
Error: PARSE ERROR: Encountered "." at line 1, column 19.

SQL Query: SELECT features[0].properties.MAPBLKLOT, FROM dfs.`/home/mbeckerle/Downloads/citylots.json.gz`
                             ^

[Error Id: bfb6226d-273f-4a9f-a58b-81c9cd32c18c ] (state=,code=0)

In addition, I've tried numerous variants of this query to see if I can find out what is wrong, but I can't get anything to work.
This much works:
SELECT flatten(features) as feature FROM dfs.`/tmp/citydata.json`;
+----------------------------------------------------------------------------------+
|                                     feature                                      |
+----------------------------------------------------------------------------------+
| {"type":"Feature","properties":{"MAPBLKLOT":"0001001","BLKLOT":"0001001","BLOCK_NUM":"0001","LOT_NUM":"001","FROM_ST":"0","TO_ST":"0","STREET":"UNKNOWN","ODD_EVEN":"E"},"geometry":{"type":"Polygon","coordinates":[[[-122.42200352825247,37.80848009696725,0.0],[-122.42207601332528,37.808835019815085,0.0],[-122.42110217434863,37.808803534992904,0.0],[-122.42106256906727,37.80860105681815,0.0],[-122.42200352825247,37.80848009696725,0.0]]]}} |
| {"type":"Feature","properties":{"MAPBLKLOT":"0002001","BLKLOT":"0002001","BLOCK_NUM":"0002","LOT_NUM":"001","FROM_ST":"0","TO_ST":"0","STREET":"UNKNOWN","ODD_EVEN":"E"},"geometry":{"type":"Polygon","coordinates":[[[-122.42082593937107,37.80863147414603,0.0],[-122.4208580496797,37.80879564136959,0.0],[-122.4198119587043,37.80876180971401,0.0],[-122.42082593937107,37.80863147414603,0.0]]]}} |
| {"type":"Feature","properties":{"MAPBLKLOT":"0004002","BLKLOT":"0004002","BLOCK_NUM":"0004","LOT_NUM":"002","FROM_ST":"0","TO_ST":"0","STREET":"UNKNOWN","ODD_EVEN":"E"},"geometry":{"type":"Polygon","coordinates":[[[-122.41570120460688,37.80832725267146,0.0],[-122.4157607435932,37.808630700240904,0.0],[-122.4137878913324,37.80856680131984,0.0],[-122.41570120460688,37.80832725267146,0.0]]]}} |
+----------------------------------------------------------------------------------+

But if I then try this in a sub-select to try to dig out the MAPBLKLOT....
 select t.properties.MAPBLKLOT from (SELECT flatten(features) as feature FROM dfs.`/tmp/citydata.json`) t;

select t.properties.MAPBLKLOT from (SELECT flatten(features) as feature FROM dfs.`/tmp/citydata.json`) t;
Error: PARSE ERROR: Encountered "." at line 1, column 9.

SQL Query: select t.properties.MAPBLKLOT from (SELECT flatten(features) as feature FROM dfs.`/tmp/citydata.json`) t
                   ^

[Error Id: 49670771-e7d5-41c7-9ede-9fa42e3ff261 ] (state=,code=0)

I found this works, but I don't know why the "['properties']" notation is needed vs. just "."
select t.feature['properties']['MAPBLKLOT'] as MAPBLKLOT from (SELECT flatten(features) as feature FROM dfs.`/tmp/citydata.json`) t;
select t.feature['properties']['MAPBLKLOT'] as MAPBLKLOT from (SELECT flatten(features) as feature FROM dfs.`/tmp/citydata.json`) t;
+-----------+
| MAPBLKLOT |
+-----------+
| 0001001   |
| 0002001   |
| 0004002   |
+-----------+
3 rows selected (0.128 seconds)
apache drill>

It appears 'properties' is a keyword. This works, note the backticks around properties
select t.feature.`properties` as property from (SELECT flatten(features) as feature FROM dfs.`/tmp/citydata.json`) t;
select t.feature.`properties` as property from (SELECT flatten(features) as feature FROM dfs.`/tmp/citydata.json`) t;
+----------------------------------------------------------------------------------+
|                                     property                                     |
+----------------------------------------------------------------------------------+
| {"MAPBLKLOT":"0001001","BLKLOT":"0001001","BLOCK_NUM":"0001","LOT_NUM":"001","FROM_ST":"0","TO_ST":"0","STREET":"UNKNOWN","ODD_EVEN":"E"} |
| {"MAPBLKLOT":"0002001","BLKLOT":"0002001","BLOCK_NUM":"0002","LOT_NUM":"001","FROM_ST":"0","TO_ST":"0","STREET":"UNKNOWN","ODD_EVEN":"E"} |
| {"MAPBLKLOT":"0004002","BLKLOT":"0004002","BLOCK_NUM":"0004","LOT_NUM":"002","FROM_ST":"0","TO_ST":"0","STREET":"UNKNOWN","ODD_EVEN":"E"} |
+----------------------------------------------------------------------------------+
3 rows selected (0.157 seconds)
apache drill (dfs.tmp)>

@mbeckerle thanks for this!


AV-1670: Move ckan to data path in container

Tests probably fail until they are updated for 2.7 image.

CKAN logging changed to level Info, as some logging was translated which caused a failure.


mylg.config now in .config. and many small non-code changes.

@jungle-boogie  Thanks for the commits!

If we change the config per your change , the current users lose their configuration, let's postpone this for later.
I can not see the binary file under github

@mehrdadrad,
yes, the config would be lost. How can we make it copy it to the new location? Alternatively, in release notes you can mention the new location.
The binary file was a mistake as I built and added mylg on my branch and inadvertently committed it.

A solution would be to run a check if ~/.mylg.config exists, move it to ~/.config/mylg.config just before the configuration is loaded.
However, and this is just my $0.02, I dont think mylg should create/use a generically named directory in the users ~ for just one file. I kinda feel like mylg's configuration file should be just a dotfile in ~. If mylg should create a directory to store its configuration I think it should use ~/.mylg/config or smething like that.

The reason I like it in .config is because that's where lots of other configuration files are saved. In my own .config, I have: htop, hub, mutt, ntfy, profanity, syncthing.
If I want to automate backups of my home directory, I could do everything in my home directory or simply important things I want to keep, which includes .config. I think there's a stronger likelihood of someone forgetting mylg saves files to ~/.mylg.config and not ~/.config/mylg.config
You're right, some the items in my .config are directories that contain only single files within the directory.
% ls -l .config/ntfy
total 4
-rw-r--r--  1 user user  92 Apr 19  2016 ntfy.yml

Ah, I have missed the fact that many applications nowdays put their config in .config. Then my point about that might be a bit useless. :D

no problem!
Would you be happy with ~/.config/mylg/mylg.config?
If so, do you have any suggestions on check/move code?

@jungle-boogie I think we need to have an issue for this subject (change configuration path) then we can discuss and track it.

@jungle-boogie Thanks!


Distinct?

How do I use distinct?

Very badly documented, but after a bit of tinkering I got 	db.Table("points").Select("DISTINCT(field_name)").Where("created_at >= ?", time.Now().Local().AddDate(0, 0, -1)).Scan(&points) working.


Added forking terminology to DBaaS Create description

Per the conversation in this thread, I updated the database create description to equate the backup restore functionality to the forking functionality in the UI. I've also updated the version fields in the create examples to something more current.

!deploy

!deploy


7.c.i Statistics # Builds (Succeeded/Failed)

7.c Integrating Travis CI is useful for integrating continuously (which is another agile principle) to avoid a big bang integration and the end of the project, we mainly want to get build statistics from Travis CI if it is used by the team.
Statistics # Builds (Succeeded/Failed) As a user I want to see statistics about how many builds failed and how many builds succeeded.
Priority: LOW
Estimation: 3

Note that this may only be done with the most recent builds if we don't save the data ourselves (as TravisCI doesn't save ALL the builds).
We could save them, but ideally that would be done by some background job processing every day or so...


social icon doesn’t appear in one row

social icon doesn’t appear in one row
https://wordpress.org/support/topic/bugs-no-featured-image-and-broken-social-icon-in-design-1/

https://github.com/ahmedkaludi/Accelerated-Mobile-Pages/commit/22a86ddf6ef250603f8300fea752b8e6a863e94d

It was corrrect, but Had issue in design two, so fixed


gadgets/snapshot_process: Port test to use new testing framework

This PR contains two commits:

gadgets/snapshot_process: Port test to use new testing framework
testing: Consolidate match logic in pkg/testing/match

The first commit is a rework / cleanup of the testing:

Move logic from helpers.go to match.go in order to have all the matching logic in the same place.
Consolidate the testing functions into two: MatchEntries() and MatchAllEntries() by creating a
new OutputMode type that defines how to parse the output. It (hopefully) makes the code more clear
as there aren't a lot of different functions around. This commit also adds some documentation explaning what those different modes are for.
Update all tests to use the new functions.

I thought more about this. I think the main source of confusion is that we were trying to differentiate between the output being in a single line or multiple lines. Actually we don't need to differentiate those cases as it's possible to decode both cases with a json decoder. I updated the PR to use this new idea, the OutputMode thing is still present, but now it's about the kind of object to decode, and not how it's formatted.
What do you think?


Update disassembly.md

T10 screws

Thanks! I don't even have a T15, so I'm not sure where that came from.


Error calling LogReceiverService from asp.net core project

Hi! Thanks for reporting this feature/bug/question!
Please keep / fill in the relevant info from this template so that we can help you as best as possible.
For .NET Core users, please check the platform support: https://github.com/NLog/NLog/wiki/platform-support
Type (choose one): Bug

Bug
Feature request
Question

NLog version: 5.0.0-beta05
Platform: ASP.NET CORE 1.1 with Full .Net Framework 4.6.1
Current NLog config (xml or C#, if relevant)
 <target name="RemoteWcfLogger"
          xsi:type="LogReceiverWcfService"
          useBinaryEncoding="false"
          endpointAddress="http://xxxx/LogReceiverServer.svc"

          clientId="xxx"
          includeEventProperties="true">
      <parameter layout="Layout" name="String" type="String"/>
      <!-- repeated -->
    </target>

In case of a BUG:

What is the current result? No wcf call to the server
What is the expected result? There should be wcf call to to server/ or should display the error in the internal file log.
Did you checked the Internal log?Yes
Please post full exception details (message, stacktrace, inner exceptions).
Are there any work arrounds? yes/no - NO
Is there a version in which it did worked? - Don't know
Can you help us by writing an unit test? Sure

link to the failure when debugging - https://pasteboard.co/fj5OHm1Rt.jpg
in case of a FEATURE REQUEST:

Why do we need it?
An example of the XML config, if relevant.

it's type LogReceiverService?
target xsi:type="LogReceiverService"

I did see that and i coded based on it because i am using asp.net core made it's not working.

Without a clear error (stacktrace, exception etc), I don't think I could help you further :(

Closing this due to inactivity. Please let us know if this still an issue and please provide the requested info.


Added Dockerfile for building Ubuntu based Docker images

Dockerfile for librealsense containers

Ubuntu base system (Ubuntu 20.04 by default)
librealsense-builder stage - builds the binaries and has all the build dependencies
librealsense stage -  contain only the built binaries and the required runtime dependencies (~60MB)
Support for Python bindings (python runtime not included in image)
Binaries are stripped during build stage to minimize image size
Support scripts for building and running the image are also included

resolves IntelRealSense/librealsense#3849

Does it solve the udev setup issue? I didn't see related commands.

Does it solve the udev setup issue? I didn't see related commands.

Not sure what the udev setup issue is. Can you please provide more information?

Does it solve the udev setup issue? I didn't see related commands.

Not sure what the udev setup issue is. Can you please provide more information?

You can see issues like this:
https://github.com/CII-BIRDS/girder-detection-ros/issues/22#issuecomment-828086879

Does it solve the udev setup issue? I didn't see related commands.

Not sure what the udev setup issue is. Can you please provide more information?

You can see issues like this:
CII-BIRDS/girder-detection-ros#22 (comment)

I’ve seen no issues with dynamically adding or removing devices multiple times with the Docker image we provided.

Superseded by #9268


Defined names support

How about supporting specific names (so-called named ranges)?

Hi. Sorry for delay with response. About your question, I'm 99% it will be quite easy to add that feature, just need to look specifications. You are the first one who ever asked. I will try to look in next week.


Adding method for returning a decoded JWT token

Robert and I spoke in slack and decided to move the decode logic into discovery and this will just be a method to get and return a token. I will update the PR soon


/usr/bin/textql: Argument list too long

Hi all,
I got this error "/usr/bin/textql: Argument list too long" when executing many  SQL statement(s) (more than 100K) to run on the data.
Have the limit length of sql query ?
Do you have any ideas to solve this problem?
Thank you for your support.
Riven.

Hi all,
I found  that this is linux command line character limit.
So, I will try it in my side.


Icon for Diaspora

Hi,
Could you add the asterik icon of the social network Diaspora, please ?
https://wiki.diasporafoundation.org/Branding
Kind regards,
P3ter

Duplication : https://github.com/danleech/simple-icons/issues/191


Table doesn't display on macOS terminal

Table does not display neither the ampltiude bars, nor the probability bars when running on macOS terminal.
The to_table utility function produces a tabular format visualization of the state. One can see an example of this by running:
cargo run --release --example h -- -q 3 -t 1 -p
To reproduce this issue, re-run the aforementioned command using macOS terminal.
Tested on the following system config:

OS
macOS Sonoma 14.3.1

terminal
Version 2.14

Hi @inmzhang,
Thank you for tackling this issue! The issue with the table exists on the macOS native terminal. I just pulled your changes, and the lack-of-color issue still exists. Please see the attached screenshot.

The issue seems to be the colors not being supported, or something to that effect. This may also be terminal config issue. Personally, I use wezterm, so I haven't run into this issue otherwise.
Best,
Saveliy

I only tested it on Linux with the Kitty terminal and assumed it would work. I'll look into it and test it on my Mac to see if I can resolve the issue.

Hi Saveliy, the comfy_table::Color::Rgb used "RGB color model" that requires the terminal has Truecolor support. And the default terminal on Mac does not satisfy^1.
A possible solution is to use comfy_table::Color::AnsiValue instead, which only requires 256-color support. However, I think it's fine to keep this Truecolor-teminal-only constraint and note it in the documentation.
Best,
Yiming

Hi @inmzhang,
Thank you for your hard work! I agree that we can specify the limitation to Truecolor-terminal-only. I'll create a PR for this.
Best,
Saveliy


Global hotkeys for Spotify

I assume this is such an obvious idea that if it were possible, it would've been suggested/added already, but in case it is an oversight.
I'd personally love to be able to set hotkeys, preferably my media keys, to interact with Spotify. It's one of my main annoyances with the app, but the web app's audio quality is horrendous and having my browser open swallows a bunch of resources anyway.
Not just to be able to play, skip and go back, having a keybind for "like" would be godly.
Spicetify looking great so far, thanks for it <3

Found out about the one on the main Spicetify page!


Add PuneetPunamiya to OWNERS

Relates to https://github.com/tektoncd/community/pull/1175

@chitrangpatel, @wlynch ptal :pray:

/lgtm


Character Experience Issue

If you play the game and level up it should go
Level 1 - 2000 experience needed for next level
Level 2 - 4000 cap
Level 3 - 6000 cap
Each time you defeat a dungeon it should knock off
500 experience points from the total experience points needed to earn to level up, but instead each time you level up you need to defeat the same amount of dungeons instead it should be like this.
Level 1 4 dungeons to level up
Level 2 8 dungeons
Level 3 12 dungeons

Implemented this in the character.py file


Add possibility to export events as iCal

This PR adds the possibility to export events from the share screen as an .ical-file to import them into a calendar application.
It additionally links the events in the "mine"-table to the share view for simplified testing/sharing for organizers.
Possible issues to discuss:

duration = parts * 45 min or longer in order to incorporate pauses?
add more meta-info to the calendar event, e.g. the organizer?
the design (i.e. choose a better icon, replace with text, ...)

Thanks, really useful!
I am sorry about fontawesome breaking the build due to the licensing issues. Its advantages (small build size, abundance of icons) currently outweight this nuisance.
To the design questions:

I'd skip modelling the pauses, as its up to the speaker how to structure the time.
I would include the organizer, as its also shown on the webpage.
fa-calendar-check looks good to me as an icon

I'll test & publish the update in the evening.

Awesome, thanks. I just realized that with the library I am using, adding the organizer will only be possible if the php requirement is bumped up to >= 7.4|8.0 (see here). I can either
a) do that, bump the PHP version & use a newer version of the library,
b) switch library,
c) dump the library and do it manually,
d) just skip outputting organizer for now
What would be most suitable for your setups/& philosophy?

I did a quick refactor for consistency, and fully tested the implementation.
For your references, the changes I have made:

Moved the iCal action & translation inside the EventController
Linked the registered events also to the shared view (below the "own" events in the my events view)
Loaded the required FontAwesome icon in the JavaScript

I did not include the organizer in the iCal:

It requires the E-Mail of the organizer; which would need the explicit approval of the speaker to publish
Just publishing<EMAIL_ADDRESS>arguably does not make much sense.
The Organizer feature is not yet published in the dependency

Don't get fooled by the number of commits; I have not changed much with your contribution. It was well done. :) Also, nice usage of the a title attribute; I use this not often enough.
I will quickly draft a release and publish the changes.

Ah, concerning the php version: To bump to php 7.4 makes total sense; php 7.2 is already EOL. I generally do not have a strong opinion which php version to aim for.


Use hapi to implement the authentication proxy

Hi I recently got my hands on kibana 5. Want to put front end authentication with the help of node.js and SQL. But I am unable to understand where should I start developing the authentication part?

Kibana uses hapi11 to serve its pages. You could create a Kibana plugin that utilizes hapi's request/response lifecycle hooks to build an authentication plugin for Kibana. You can learn more about creating Kibana plugins here: https://github.com/elastic/kibana/wiki/Plugin-Resources28

https://discuss.elastic.co/t/kibana-5-front-end-authentication/52024
See http://hapijs.com/

on hold.


Generic function for obtaining the privilege stack bounds for x86

Describe the bug
arch.priv_stack_start is implemented only for ARM and ARC. Necessary to have a generic function for obtaining the privilege stack bounds for x86 too.
In other words, necessary to have an arch-neutral API for getting a pointer to the privilege mode stack, then there would not need to be any arch-specific code in this test case and it could be entirely cross-platform.
To Reproduce
Steps to reproduce the behavior:

Try to use thread->arch.priv_stack_start somewhere in the code
Build it for x86
It can't build for x86, because that feature not implemented for x86

Expected behavior
Obtaining the privilege stack bounds must be implemented for all architectures.
Impact
For x86 no generic function for obtaining the privilege stack bounds
Environment (please complete the following information):

OS: Linux
Toolchain Zephyr SDK 0.11.4
Commit SHA d1f4a91183ac77cd3e38ad897ac2cc191b23661a

@andrewboie Could you take a look at that issue? So after solving it, I can finish my test cases PR #25323

yes will spend some time on this, thanks

@andrewboie Hello, any news from you regards that issue?

Hi @nashif @andyross , do we indeed need a function/interface like this?  or we can implement this as a test util? thank you!

It doesn't seem like a bad API to have, but clearly we don't need it generically right now.  In some architectures, there's no MMU state behind kernel stacks, so it's not necessarily something that matters to the hardware.
I guess if nothing else is should be part of CONFIG_THREAD_STACK_INFO, which is where we put this information (optionally) for apps that require it.

Hi @andyross , thanks for your guidance!
So my understanding is to implement this kind of API within CONFIG_THREAD_STACK_INFO=y, we might not need to implement it for all of the ARCH/platform at this moment.


Addressing review comments for industrial_ci#319

Addressing review comments in the upstream PR https://github.com/ros-industrial/industrial_ci/pull/319 except for https://github.com/ros-industrial/industrial_ci/pull/319/files#r230591386 that I couldn't figure out and need your @AustinDeric input.

@AustinDeric I think all review comments from https://github.com/ros-industrial/industrial_ci/pull/319 are addressed in this PR.


Bump Jetty 9 Ring adapter to 0.8.2 to address CVE-2015-2080

See https://github.com/sunng87/ring-jetty9-adapter/pull/20.

It seems like for some reason this commit was cherry-picked (33132b65171a10864624762722ea8ac905476adb) rather than merged in. I have no idea why as I was away last week, but I'm closing this and deleting the branch to clean up.


Blog Post - Micro:bit 2024 update

Hi Galen, can you please prep this blog post?  Would like to go out next Tuesday.  Files are all here:  https://1drv.ms/f/s!AmMIW5Hxi0Rtgac1yTqJ9Q7XABQJCQ?e=eo8Zk7
There may be some changes/edits coming back from the micro:bit Foundation...
Thanks!
Jaq.

Hey @ganicke - not sure if you've started working on this, but hold tight.  There are a few changes coming down the pipe.  Will let you know when we've finalized changes.

Ok, standing by...

Hi Galen.  Okay, should be good to start getting it prepped - the only thing that's missing is an updated gif of the Version History feature.  Should have that later today.  Let's plan on posting tomorrow.  Thanks!

@ganicke - the Version History demo is ready to go.  There's a .gif and .mp4 version of it in the OneDrive share.  Will plan to publish tomorrow (Friday).  Thanks!


Switch etcd mode to use an init contianer for CNI

Description

We did this for KDD a while back and it seems to be working well. Let's do it for etcd in the same release.
https://github.com/projectcalico/calico/pull/2231
Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

None required

 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.

Deploy preview for calico ready!
Built with commit bbe923a23daf64273ac653dcce68f93e6ff9d965
https://deploy-preview-2300--calico.netlify.com

ISTR that we used to keep the container running so it monitored the etcd certs, or something?  What are we losing here?


Fix capitalization of D3DRTYPE_CUBETEXTURE

@ccawley2011 : Thanks for your contribution! The author(s) have been notified to review your proposed change.


fix(firebase/rules): only allow system users to write into common lab executions

Fixes the issue that authed users were able to write into their own common/lab executions, this should only be allowed for system users.
Fixes: #791

Here's a rebased version of this PR https://github.com/machinelabs/machinelabs/pull/798
We had to rebase it on https://github.com/machinelabs/machinelabs/commit/d3a8933a644f4053d433b506eb7fc706abde0fd2 to ensure the travis is going to be green.

We can close this one then, no?


Cannot update dependencies

Even after a fresh install, I get an error when trying to update dependencies. It appears that it can't find the requirements.txt file even though I've left it alone since I installed it.
Upon running update_deps.bat, I get this error message: http://i.imgur.com/6jSpOcu.png
Even when I tried to do it manually, same thing: http://i.imgur.com/95UkcfF.png
For posterity, here's the file, right where it should be: http://i.imgur.com/UPcdcOe.png

is your file in the root directory of your D: drive?
try shift + right click in the folder -> open command window here -> start update_deps.bat

Try doing it manually, but have your command prompt navigated to the right folder. The command won't work if you aren't navigated to the right folder.

I tried out your solution and it works fine now. I guess it's just because my default command prompt location is the root of the D: drive. Thanks so much.

I tried out your solution and it works fine now. I guess it's just because my default command prompt location is the root of the D: drive. Thanks so much.


deb: fix td-agent tmp files handling in postinst

It is false posivive because of /tmp/td-agent is
accessed by only td-agent:td-agent user/group.

Could you fix a typo in the commit message?
-posivive
+positive

@ashie Thanks, I've fixed push -f.

Thanks!


HOSTEDCP-1063: Account for guest webhook URLs without a port

What this PR does / why we need it:
snapshot.storage.k8s.io webhook was not detected because of this, although the service didn't have the label yet.
PR to add the label to the csi-snapshot-webhook service:
https://github.com/openshift/cluster-csi-snapshot-controller-operator/pull/155
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #
Checklist

[ ] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/test e2e-aws

/test e2e-aws

/approve

/ok-to-test


fix(Subject): Limit Subject interface public accessibility

Description:
This PR updates interface of Subject. I came to notice current Subject interface expost its properties too wide to public scope, any instance of Subject exposes state variables like observers, isStopped can be mutated. This PR limits those variables to inherited scope only, and expose it as readonly property.
It is certainly breaking change since anyone have used those property to change Subject's state now it won't work - but not sure how popular usecase it'll be. (and is it recommended pattern?) For now, I've targeted this into master until gets reviewed and approved.
BREAKING CHANGE: if there's code change state of Subject via publicly exposed properties, it won't work anymore
Related issue (if exists):
Related note, is it worth #2232 to add Subject:hasObservers() interface or should recommend to use Subject::observers.length and update migration guide?

Coverage decreased (-0.09%) to 97.601% when pulling 9357d8ae8b926d12b62e62d15bb16355f82dc96e on kwonoj:refactor-subject into 6922b16f4a297a4acbe8ad2b086c947b7c0d8315 on ReactiveX:master.

Coverage decreased (-0.09%) to 97.601% when pulling 9357d8ae8b926d12b62e62d15bb16355f82dc96e on kwonoj:refactor-subject into 6922b16f4a297a4acbe8ad2b086c947b7c0d8315 on ReactiveX:master.

Coverage increased (+0.002%) to 97.692% when pulling a0f2764b57c2524ac366563f722bad22876f6966 on kwonoj:refactor-subject into 6922b16f4a297a4acbe8ad2b086c947b7c0d8315 on ReactiveX:master.

Coverage increased (+0.002%) to 97.692% when pulling a0f2764b57c2524ac366563f722bad22876f6966 on kwonoj:refactor-subject into 6922b16f4a297a4acbe8ad2b086c947b7c0d8315 on ReactiveX:master.

This is akin to my comments on #2234... I'm not sure what this is buying us.

I'm closing this for now as same reason of https://github.com/ReactiveX/rxjs/pull/2234.


Uncrustify the docs

Fixes #344

Pull Request Test Coverage Report for Build 1245

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

Totals

Change from base Build 1238:
0.04%

Covered Lines:
2225

Relevant Lines:
2760

💛  - Coveralls


Import estimates/items list derivation improvements

Import item list (also used for deriving estimate counts) now stored as Map of normalized + encoded URLs -> import items. Previously was Array of import items.

custom Pouch.find queries moved to standard Pouch.allDocs queries
normalization on import item keys means a lot of semantically-identical (in our context) URLs will now be properly filtered out
bookmarks and history counts should make more sense for users (proper distinction made). Old ext imports is still a bit of an odd-case as each old-ext import will increase either the history or bookmark AND old-ext "already saved" count
put concurrency setting back into imports processor (currently set at 3 items at any time). Concurrency queue implemented a bit back should guarantee safety at DB-level for indexing, allowing us to do stuff like the XHRs concurrently (still won't move onto the next import item until all DB ops are finished). Need to see if any weird behaviour, but initial play with this seems fine

@poltak Btw that was my commit had it previously on a different branch but decided to add it here. I must have my git set up on the wrong email address as my name isn't popping up 😞

Alright done a bit of a review and just cleaned up some minor things. All looks good.
Thanks a lot for the file ext check in the loggability checker @swissums!
RE the name/email thing: I think you must have<EMAIL_ADDRESS>set as your email in your immediate git config, but not in your GH @swissums account settings (maybe?)


feat: 🎸 add Instance Metadata Tags access option

Fixes #3650
Description
InstanceMetadataTags MetadataOption enables access to tags in the instance metadata. It allows local processes that are running on an instance to view the instance's tag information directly from the instance metadata. That means you do not need to execute DescribeInstances or DescribeTags API calls to retrieve tag information, which reduces amount of your AWS API calls.
Does this change impact docs?

[x] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[ ] No

Release Note
Adds `InstanceMetadataTags` `MetadataOption`  in provisioner launch template to allow access to tags from Instance Metadata. Default `disabled`.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Closing for now due to a merge conflict, feel free to re-base, re-test and re-submit.

This is a pretty annoying blocker for us for some work but as indicated the illegal tag names are causing issues.


Server is not starting

I have tried to start server putting FCNPC, but the server is not starting even if i did upgrade the server version to r3 @ziggi

Why?

Why?

i dont know but when i tried to start the server without FCNPC it works fine, and when i read the other issues i tried the latest version but still it didnt changed the same issue occurs

Try to read logs.

Try to read logs.

=========================== FCNPC Exception Handler ============================
Signal 11 (Segmentation fault), address is f74e4040 from f77af591

this is FCNPC crash log
[08:46:32]    Error: Function not registered: 'FCNPC_GetPluginVersion'
[08:46:32]    Error: Function not registered: 'FCNPC_IsValid'
[08:46:32]    Error: Function not registered: 'FCNPC_Respawn'
[08:46:32]    Error: Function not registered: 'FCNPC_Destroy'
[08:46:32]    Error: Function not registered: 'FCNPC_Create'
[08:46:32]    Error: Function not registered: 'FCNPC_SetInvulnerable'
[08:46:32]    Error: Function not registered: 'FCNPC_Spawn'
[08:46:32]    Error: Function not registered: 'FCNPC_ExitVehicle'
[08:46:32]    Error: Function not registered: 'FCNPC_GetPosition'
[08:46:32]    Error: Function not registered: 'FCNPC_EnterVehicle'
[08:46:32] Script[gamemodes/nexalv8.amx]: Run time error 19: "File or function is not found""

and this is the error from serverlog

this is FCNPC crash log

No it's first two lines of the crash log.

`Loaded log file: "server_log.txt".
SA-MP Dedicated Server
v0.3.7-R3, (C)2005-2019 SA-MP Team
[08:45:28] password = ""  (string)
[08:45:28]
[08:45:28] Server Plugins
[08:45:28] --------------
[08:45:28]  Loading plugin: crashdetect.so
[08:45:28]   CrashDetect plugin 4.19.4
[08:45:28]   Loaded.
[08:45:28]  Loading plugin: whirlpool.so
[08:45:28]
[08:45:28]  ==================
[08:45:28]
[08:45:28]   Whirlpool loaded
[08:45:28]
[08:45:28]  ==================
[08:45:28]
[08:45:28]   Loaded.
[08:45:28]  Loading plugin: pawncmd.so
[08:45:28] Pawn.CMD plugin v3.2.0 by urShadow has been loaded
[08:45:28]   Loaded.
[08:45:28]  Loading plugin: sampvoice.so
[08:45:28] [sv:dbg:network:init] : module initializing...
[08:45:28] [dbg:raknet:init] : module initializing...
[08:45:28] [dbg:raknet:init] : module initialized
[08:45:28] [sv:dbg:network:init] : module initialized
[08:45:28] [sv:dbg:main:Load] : creating 4 work threads...
[08:45:28]  -------------------------------------------
[08:45:28]    ___                __   __    _
[08:45:28]   / | __ _ _ __  _ \ \ / / () __ ___
[08:45:28]   _ / ` | '  | ' \   / _ | |/ / -)
[08:45:28]   |/_,|||| ./_/_/||____|
[08:45:28]                   ||
[08:45:28]  -------------------------------------------
[08:45:28]            SampVoice by MOR loaded
[08:45:28]  -------------------------------------------
[08:45:28]   Loaded.
[08:45:28]  Loading plugin: sscanf.so
[08:45:28]
[08:45:28]  ===============================
[08:45:28]       sscanf plugin loaded.
[08:45:28]          Version:  2.8.3
[08:45:28]   (c) 2018 Alex "Y_Less" Cole
[08:45:28]  ===============================
[08:45:28]   Loaded.
[08:45:28]  Loading plugin: streamer.so
[08:45:28]
*** Streamer Plugin v2.9.5 by Incognito loaded ***
[08:45:28]   Loaded.
[08:45:28]  Loading plugin: textdraw-streamer.so
[08:45:28]
[08:45:28]  =================================
[08:45:28]  |                               |
[08:45:28]  |    textdraw-streamer v2.0.3   |
[08:45:28]  |            Loaded             |
[08:45:28]  |                               |
[08:45:28]  |  Coding:                      |
[08:45:28]  |                               |
[08:45:28]  |  Burak (Nexor)                |
[08:45:28]  |                               |
[08:45:28]  |  Compiled:                    |
[08:45:28]  |                               |
[08:45:28]  |  01.12.2023, 07:11:19         |
[08:45:28]  |                               |
[08:45:28]  |  Github:                      |
[08:45:28]  |                               |
[08:45:28]  |  github.com/nexquery          |
[08:45:28]  |                               |
[08:45:28]  |  Discord:                     |
[08:45:28]  |                               |
[08:45:28]  |  benburakya - Nexor#4730      |
[08:45:28]  |                               |
[08:45:28]  =================================
[08:45:28]
[08:45:28]   Loaded.
[08:45:28]  Loading plugin: mysql_static.so
[08:45:28]  >> plugin.mysql: R39-6 successfully loaded.
[08:45:28]   Loaded.
[08:45:28]  Loading plugin: discord-connector.so
[08:46:30]  >> plugin.dc-connector: timeout while initializing data.
[08:46:30]                          plugin will proceed to retry connecting in the background.
[08:46:30]   Loaded.
[08:46:30]  Loading plugin: nativechecker.so
[08:46:30]   Loaded.
[08:46:30]  Loaded 10 plugins.
[08:46:30]
[08:46:30] Ban list
[08:46:30] --------
[08:46:30]  Loaded: samp.ban
[08:46:30]
[08:46:31]
[08:46:31] Filterscripts
[08:46:31] ---------------
[08:46:31]   Loading filterscript 'android-check.amx'...
[08:46:31] [sv:dbg:main:AmxLoad] : net game pointer (value:0x849cca0) received
[08:46:31] [sv:dbg:network:bind] : voice server running on port 44803
[08:46:31] Android check has been successfully loaded.
[08:46:31]   Loading filterscript 'mapping.amx'...
[08:46:31]   Loading filterscript 'nexalmap.amx'...
[08:46:31]   Loading filterscript 'login.amx'...
[08:46:31]   Loading filterscript 'hold.amx'...
[08:46:31] [FS]HoldingWeapons Loaded!
[08:46:31]   Loaded 5 filterscripts.
[08:46:32] [Pawn.CMD] AmxLoad: mismatch between the plugin and include versions
[08:46:32]    Error: Function not registered: 'FreezeSyncData'
[08:46:32]    Error: Function not registered: 'SetFakeFacingAngle'
[08:46:32]    Error: Function not registered: 'SendLastSyncData'
[08:46:32]    Error: Function not registered: 'ClearAnimationsForPlayer'
[08:46:32]    Error: Function not registered: 'SetKnifeSync'
[08:46:32]    Error: Function not registered: 'SetFakeHealth'
[08:46:32]    Error: Function not registered: 'SetFakeArmour'
[08:46:32]    Error: Function not registered: 'PlayerTextDrawSetPosition'
[08:46:32]    Error: Function not registered: 'SpawnPlayerForWorld'
[08:46:32]    Error: Function not registered: 'FCNPC_GetPluginVersion'
[08:46:32]    Error: Function not registered: 'FCNPC_IsValid'
[08:46:32]    Error: Function not registered: 'FCNPC_Respawn'
[08:46:32]    Error: Function not registered: 'FCNPC_Destroy'
[08:46:32]    Error: Function not registered: 'FCNPC_Create'
[08:46:32]    Error: Function not registered: 'FCNPC_SetInvulnerable'
[08:46:32]    Error: Function not registered: 'FCNPC_Spawn'
[08:46:32]    Error: Function not registered: 'FCNPC_ExitVehicle'
[08:46:32]    Error: Function not registered: 'FCNPC_GetPosition'
[08:46:32]    Error: Function not registered: 'FCNPC_EnterVehicle'
[08:46:32] Script[gamemodes/nexalv8.amx]: Run time error 19: "File or function is not found"
[08:46:32] Number of vehicle models: 0

Loaded log file: "server_log.txt".
SA-MP Dedicated Server
v0.3.7-R3, (C)2005-2019 SA-MP Team
[08:48:46] password = ""  (string)
[08:48:46]
[08:48:46] Server Plugins
[08:48:46] --------------
[08:48:46]  Loading plugin: crashdetect.so
[08:48:46]   CrashDetect plugin 4.19.4
[08:48:46]   Loaded.
[08:48:46]  Loading plugin: whirlpool.so
[08:48:46]
[08:48:46]  ==================
[08:48:46]
[08:48:46]   Whirlpool loaded
[08:48:46]
[08:48:46]  ==================
[08:48:46]
[08:48:46]   Loaded.
[08:48:46]  Loading plugin: pawncmd.so
[08:48:46] Pawn.CMD plugin v3.2.0 by urShadow has been loaded
[08:48:46]   Loaded.
[08:48:46]  Loading plugin: sampvoice.so
[08:48:46] [sv:dbg:network:init] : module initializing...
[08:48:46] [dbg:raknet:init] : module initializing...
[08:48:46] [dbg:raknet:init] : module initialized
[08:48:46] [sv:dbg:network:init] : module initialized
[08:48:46] [sv:dbg:main:Load] : creating 4 work threads...
[08:48:46]  -------------------------------------------
[08:48:46]    ___                __   __    _
[08:48:46]   / | __ _ _ __  _ \ \ / / () __ ___
[08:48:46]   _ / ` | '  | ' \   / _ | |/ / -)
[08:48:46]   |/_,|||| ./_/_/||____|
[08:48:46]                   ||
[08:48:46]  -------------------------------------------
[08:48:46]            SampVoice by MOR loaded
[08:48:46]  -------------------------------------------
[08:48:46]   Loaded.
[08:48:46]  Loading plugin: sscanf.so
[08:48:46]
[08:48:46]  ===============================
[08:48:46]       sscanf plugin loaded.
[08:48:46]          Version:  2.8.3
[08:48:46]   (c) 2018 Alex "Y_Less" Cole
[08:48:46]  ===============================
[08:48:46]   Loaded.
[08:48:46]  Loading plugin: streamer.so
[08:48:46]
*** Streamer Plugin v2.9.5 by Incognito loaded ***
[08:48:46]   Loaded.
[08:48:46]  Loading plugin: textdraw-streamer.so
[08:48:46]
[08:48:46]  =================================
[08:48:46]  |                               |
[08:48:46]  |    textdraw-streamer v2.0.3   |
[08:48:46]  |            Loaded             |
[08:48:46]  |                               |
[08:48:46]  |  Coding:                      |
[08:48:46]  |                               |
[08:48:46]  |  Burak (Nexor)                |
[08:48:46]  |                               |
[08:48:46]  |  Compiled:                    |
[08:48:46]  |                               |
[08:48:46]  |  01.12.2023, 07:11:19         |
[08:48:46]  |                               |
[08:48:46]  |  Github:                      |
[08:48:46]  |                               |
[08:48:46]  |  github.com/nexquery          |
[08:48:46]  |                               |
[08:48:46]  |  Discord:                     |
[08:48:46]  |                               |
[08:48:46]  |  benburakya - Nexor#4730      |
[08:48:46]  |                               |
[08:48:46]  =================================
[08:48:46]
[08:48:46]   Loaded.
[08:48:46]  Loading plugin: mysql_static.so
[08:48:47]  >> plugin.mysql: R39-6 successfully loaded.
[08:48:47]   Loaded.
[08:48:47]  Loading plugin: discord-connector.so
[08:49:42]  >> plugin.dc-connector: timeout while initializing data.
[08:49:42]                          plugin will proceed to retry connecting in the background.
[08:49:42]   Loaded.
[08:49:42]  Loading plugin: FCNPC.so
[08:49:42]   Failed (plugins/FCNPC.so: cannot open shared object file: No such file or directory)
[08:49:42]  Loading plugin: nativechecker.so
[08:49:42]   Loaded.
[08:49:42]  Loaded 10 plugins.
[08:49:43]
[08:49:43] Ban list
[08:49:43] --------
[08:49:43]  Loaded: samp.ban
[08:49:43]
[08:49:44]
[08:49:44] Filterscripts
[08:49:44] ---------------
[08:49:44]   Loading filterscript 'android-check.amx'...
[08:49:44] [sv:dbg:main:AmxLoad] : net game pointer (value:0x91c9cc0) received
[08:49:44] [sv:dbg:network:bind] : voice server running on port 49975
[08:49:44] Android check has been successfully loaded.
[08:49:44]   Loading filterscript 'mapping.amx'...
[08:49:44]   Loading filterscript 'nexalmap.amx'...
[08:49:44]   Loading filterscript 'login.amx'...
[08:49:44]   Loading filterscript 'hold.amx'...
[08:49:44] [FS]HoldingWeapons Loaded!
[08:49:44]   Loaded 5 filterscripts.
[08:49:45] [Pawn.CMD] AmxLoad: mismatch between the plugin and include versions
[08:49:45]    Error: Function not registered: 'FreezeSyncData'
[08:49:45]    Error: Function not registered: 'SetFakeFacingAngle'
[08:49:45]    Error: Function not registered: 'SendLastSyncData'
[08:49:45]    Error: Function not registered: 'ClearAnimationsForPlayer'
[08:49:45]    Error: Function not registered: 'SetKnifeSync'
[08:49:45]    Error: Function not registered: 'SetFakeHealth'
[08:49:45]    Error: Function not registered: 'SetFakeArmour'
[08:49:45]    Error: Function not registered: 'PlayerTextDrawSetPosition'
[08:49:45]    Error: Function not registered: 'SpawnPlayerForWorld'
[08:49:45]    Error: Function not registered: 'FCNPC_GetPluginVersion'
[08:49:45]    Error: Function not registered: 'FCNPC_IsValid'
[08:49:45]    Error: Function not registered: 'FCNPC_Respawn'
[08:49:45]    Error: Function not registered: 'FCNPC_Destroy'
[08:49:45]    Error: Function not registered: 'FCNPC_Create'
[08:49:45]    Error: Function not registered: 'FCNPC_SetInvulnerable'
[08:49:45]    Error: Function not registered: 'FCNPC_Spawn'
[08:49:45]    Error: Function not registered: 'FCNPC_ExitVehicle'
[08:49:45]    Error: Function not registered: 'FCNPC_GetPosition'
[08:49:45]    Error: Function not registered: 'FCNPC_EnterVehicle'
[08:49:45] Script[gamemodes/nexalv8.amx]: Run time error 19: "File or function is not found"
[08:49:45] Number of vehicle models: 0
`

Failed (plugins/FCNPC.so: cannot open shared object file: No such file or directory)

Copy FCNPC.so to plugins/ directory.

server_log.txt
svlog.txt
crash_log (2).txt
these are the fresh logs on the last crash

It looks like you are trolling me. There is no crash, FCNPC is not even trying to load. Add FCNPC to plugins in server.cfg file.

It looks like you are trolling me. There is no crash, FCNPC is not even trying to load. Add FCNPC to plugins in server.cfg file.

ouch sorry i did not added the plugin in cfg forgotten i did removed due to the crash issues i will sent you the log after adding it sorry for wasting your time

@ziggi  The problem seems fixed now i think the problem was i put the name of the npc like Firstname Secondname i changed it to FIRSTNAME_SECONDNAME the problem is now fixed


Lane detection ROS2 node

Adds a ROS2 node for lane detection using ultra-fast-lane-detection_v2 + TensorRT .
Source repo: https://github.com/iwatake2222/play_with_tensorrt/tree/master/pj_tensorrt_lane_ultra-fast-lane-detection_v2
Day demo:
https://github.com/WATonomous/wato_monorepo/assets/28817028/9fafa078-0e51-4cc6-9c6a-6d7787ab58fb
Night demo:
https://github.com/WATonomous/wato_monorepo/assets/28817028/4c0021a1-e390-4141-81f9-2feb707f2d19

oh dont mind tese comments for now, let me know when u need a review :)

oh dont mind tese comments for now, let me know when u need a review :)

sure feel free to re-review


Increase test coverage of internal package

Check List:

Fixes: NA
Unit tests: YES
Javadoc with a code example (on API only) : NA
PR meets the contributing guidelines

Following the contributing guidelines will make it easier for us to review and accept your PR.

@nstruharova could you please rebase the PR? It should fix the CI issues

I just pushed the rebase, everything is up to date :)

@nstruharova let us know if you have time to address these comments, otherwise I might do it in the next few days

@joel-costigliola No worries, I will finish and push first thing Tuesday!

apologies for the delay, thank you for the feedback @joel-costigliola! All suggestions should be applied now. The branch is also rebased on main so there are no conflicts.

Integrated thanks @nstruharova!


Crash on App Start: NoSuchMethodError for acquireEnvironmentalHdrCubeMap

Hello,
I'm encountering a runtime crash in my AR app that uses Sceneform. The crash occurs immediately after granting camera and AR permissions, and it seems related to the ARCore LightEstimate functionality, although my application does not explicitly use the LightEstimate.acquireEnvironmentalHdrCubeMap() method.
Error Message:
FATAL EXCEPTION: main
Process: com.example.draw, PID: 17087
java.lang.NoSuchMethodError: No virtual method acquireEnvironmentalHdrCubeMap()[Lcom/google/ar/core/ArImage; in class Lcom/google/ar/core/LightEstimate; or its super classes (declaration of 'com.google.ar.core.LightEstimate' appears in /data/app/~~HqDIVrudygaS9gw55gIagg==/com.example.draw-CJtrcjl5EEp1CNnlpVOyNA==/base.apk) at com.gorisse.thomas.sceneform.light.LightEstimationKt.environmentalHdrEnvironmentLights(LightEstimation.kt:394)
Steps to Reproduce:

Build and run the app on the device.
Grant camera and AR permissions as requested.
The app crashes immediately after permissions are granted.

Environment:

ARCore SDK version: [1.41.1]
Sceneform version: [1.23.0]
Android device and OS version: [Samsung Galaxy A14 5G, Android v13]

I've included all relevant code in a gist for review: Gist Link
Here's the screenshot of the run time error I am getting in logcat

Could you please help identify the cause of this issue or suggest a workaround?
Thank you for your support.

The error is correct, ARCore version 1.41.1 does not have acquireEnvironmentalHdrCubeMap(). it's best to use the same ARCore version the Sceneform-Maintained release does. In this case, com.gorisse.thomas.sceneform:sceneform:1.23.0 is using ARCore version 1.31.0


public datasets: fix license icons

compromise solution for https://github.com/CartoDB/cartodb/issues/6413
I'll leave the issue open, to create the new icons or replace them with a more general one
CR @xavijam @juanignaciosl

Screenshot?

Strictly speaking license_info can be nil, so it should be checked, but since old code hadn't the check I guess it can be skipped :+1:

sure, ded01

:+1:


How to get public key

Hi,
I want to know how to get and print public key with KeysetHandle for show log cat.
I tried to use keysetHandle.getPublicKeysetHandle() but log cat not show.
Thank
--
Gee

I hope that I answered your question. Please reopen if you have any further questions.


added images and credits to readme

I added mock up image and wireframes to readme and also put the colour pallete credit into the readme.  I know someone else is working on the readme later on? so havent added anything else in.  Still lots to add later on, presumably by the other team member.

There was no conflict, so I've merged it.


unable to access Vue instance on Custom Methods

Version
2.6.10
Reproduction link
https://stackoverflow.com/questions/59171904/vue-optionmergestrategies-methods-has-no-access-to-this-keyword
I don't know if its a bug, feature or eventually I am doing it wrong.
please check the StackOverflow link I have given.
What is expected?
I want to access current Vue instance by 'this' keyword under the customMethod
What is actually happening?
returning 'undefined'

Hi, thanks for your interest but Github issues are for bug reports and feature requests only. You can ask questions on the forum, the Discord server or StackOverflow.


Protractor with For Loop not able to understand logic:

what i need to do is that i have to click some images and with that the text is changing in another element. I am using For Loop with promises , so i am able to successfully click on images 1 by 1 but the text is not getting displayed . its displaying the last text 5 times . please help
element.all(by.xpath(".//*[@ng-repeat='num in facesGrid[$index]']")).then(function(faces){
    for(var i=0;i<faces.length;i++)
    {
        element(by.model('currentTestimonialIndex')).getText().then(function(Index){
        console.log(Index);
        });
        faces[i].click();

}

    });

Hi there!
Your question is better suited for StackOverflow. Please ask a question there with the 'protractor' tag to get help.
From the the getting help section of the README:

Please ask usage and debugging questions on StackOverflow (use the "protractor" tag) or in the Angular discussion group. (Please do not ask support questions here on Github.)

Maybe you need a chain of then().


keyv.iterator is not a function when using KeyvEtcd

keyv.iterator is not a function when using KeyvEtcd.
const keyv = new Keyv(new KeyvEtcd('etcd://localhost:2379'));
  await keyv.set('key1', 'value1');
  await keyv.set('key2', 'value2');

  console.log(keyv.iterator); // TypeError: keyv.iterator(...) is not a function
  for await (const [key, value] of keyv.iterator()) {
    console.log(key, value);
  }

In fact, KeyvEtcd has an iterator.
https://github.com/jaredwray/keyv/blob/27c49a00c7c0b12875eec48b071cf297152644bf/packages/etcd/src/index.ts#L126-L136

Moving this to an enhancement as Keyv adapters do not need to provide iteration


GEOMESA-919 Adding support for linestrings in BIN output

Signed-off-by: Emilio Lahr-Vivaz<EMAIL_ADDRESS>

Merged


titleTemplate should take a function

Ideally I'd like to be able to determine whether or not to place dividers in the title (eg. pipes or hyphens).
titleTemplate: (titleChunk) => {
  // If undefined or blank then we don't need the hyphen
  return titleChunk ? `${titleChunk} - Site Title` : 'Site Title';
}

Hi @homerjam
This should be backward compatible as well, any PR is welcome :)


[WIP]K8s keyring for docker pull

Using the keyring should allow us to check only the relevant credentials.

dependent on https://github.com/openshift/image-inspector/pull/42

@pweil- @csrwng any suggestion here to minimize the dependencies for implementing the keyring?


URI::InvalidComponentError: bad component(expected host component): http://3b49977a.ngrok.io

Hi @matiaskorhonen ,
When i try to run the following line i was getting the error
status = Monit::Status.new({ :host => "http://3b49977a.ngrok.io",:auth => true, :username => "username",:password => "password" })
status.get
URI::InvalidComponentError: bad component(expected host component): http://3b49977a.ngrok.io

I believe this is because of the http:// in the host option. The host should be just the host (with no scheme).


not work on latest browers

cant open the demo, I am using chrome 61, and firefox 56.
but note:
Your browser does not support camera access.
We recommend Chrome — modern, secure, fast browser from Google.
It's free.

first check webcam drivers. sometimes multiple webcam drivers would create some issue. try running it in mozilla firefox, it will ask for a selection box wherein you can select the webcam driver if you have multiple drivers installed, i recently installed this and checked on all browsers, seems to be working fine.

I opened the demo on Chrome on my iPhone and it tells me:

Your browser does not support camera access.
We recommend Chrome
😂 😂 🤣


[BUGFIX] Added padding to unordered lists in custom pages

Description
Styling looks bugged on custom pages when unordered lists are used. Cause is, that ul styling receives padding: 0.

Questions
Answers

BC breaks?
no

Tests pass?
yes

Related tickets?

How to test?
Add list to static page.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/SW-18287.
Please use this issue to track the state of your pull request.

Thanks for your contribution @wolv-dev I added sorted list to your commit, so we're having the same behavior across both list types. The commit should be pushed in a few.


Plenary: Open Science Discussion

Lead: Andy Barrett
Date: 23/08/2024
Start Time: 0900
Duration: 45
Description: Open science best practices
Details

Learning Outcomes

outcome1
outcome2
outcome3

People Developing the Tutorial (content creation, helpers, teachers)
Summary Description
Dependencies (things people should know in advance of the tutorial)
Not dependencies, just adding a few relevant links to previous content

https://icesat-2-2023.hackweek.io/reference/open_science.html
https://book.cryointhecloud.com/content/about.html
https://www.earthdata.nasa.gov/technology/open-science

Technical Needs (GPUs? Large file storage? Unique libraries?)

Happy to lead development of this session.
Does it make sense to have this as a panel discussion, or quick pop-ups, and maybe a Q&A.

Does it make sense to have this as a panel discussion, or quick pop-ups, and maybe a Q&A.

Those are all fun ideas!

Here's another thought I had.
My initial idea was to have a discussion on producing reusable and interpretable data.  I was thinking about providing groups with various data with missing attributes etc and asking them to recreate a plot.
The best-case scenario would be a CF-Compliant NetCDF.  This would hopefully seed a discussion on how to create data that can be reused out-of-the box.

My initial idea was to have a discussion on producing reusable and interpretable data. I was thinking about providing groups with various data with missing attributes etc and asking them to recreate a plot.

I love this idea! It's such a good way to illustrate why this is so important.

That's a super cool idea 💡

As a reference to add, at NOAA/NCAR/NREL's hackathon earlier this year, we hosted a speaker from AGU Open Science Leadership. They gave a great talk to our participants about Open Science. Feel free to use this as a resource to inspire this session or panel discussion questions: https://zenodo.org/records/10720248

Adding @mstudinger to this thread who I've been speaking with regarding his role as a NASA TOPS Champion. Michael has been teaching the Open Science 101 curriculum to summer interns and offered to join in this session.
Suggest all of us hop on a call sometime soon?

Apologies for not getting back to this sooner.  I've been moving house from Denver to Taos, NM.
@aaarendt and @mstudinger Happy to jump on a call Thursday 8 August (tomorrow) or Friday 9 August.  Could also do early next week.

Happy to talk/brainstorm about this session. I really like the idea of asking participants to create a reusable and interpretable data set or science result. I am not sure if we can do that in 45 minutes though, but let’s definitely think about it.
Here are few random thoughts (mostly based on my experience teaching OS/advocating for it in the community)
I think the title “open science best practices” is a great focus of what this session could be. Given the focus of the hackweeks on open science, open collaboration, etc I don’t think we need to convince anyone at the hackweek that open science is a good thing. But in my experience people have still a lot of anxieties with something new, just like there was when NASA switched to the Dual-Anonymous Peer Review (DAPR) a few years ago. When I talk about open science folks are typically very keen to ask questions and share their concerns. We should allocate a good amount of time for questions/discussions if possible.
The monstrous 12.5 hour NASA curriculum covers mostly how to use, make and share open data, open code, and open results and could be a structure for this session as well. Things like: when it is ok to be open and when it is not ok to be open? Why should you give your data/code/result a license? After taking the training earlier this year I assigned licenses to all my git repos….  I hadn’t bothered doing that before because I didn’t know…. How do you search for open data/code/results and evaluate the quality? How do you make you data/code/result findable (Zenodo DOI vs git repo/web address). Things like that.
As already mentioned by others there is a lot of material available on the web. Not that it would be needed, but if people are interested here is the NASA OS 101 curriculum on github:
https://github.com/nasa/Transform-to-Open-Science
The 5 modules with 5 lessons each are here: https://github.com/nasa/Transform-to-Open-Science/tree/open-science-101

I agree 45 minutes is not much time and also that we do not need to duplicate the TOPS curriculum, or evangelize open science.  Hopefully, we would be preaching to the choir.
meeting of how
Maybe we should title the session "Putting open science into practice" and try to be practical rather than prescriptive.
I think it is also worth emphasizing that the open science best practices not only benefit the community and satisfy funding agency or publisher requirements, but also benefit our "future selves" when we come back to a project.  I've been involved with Openscapes for a while.  I am very drawn to emphasizing that open science is about improving collaboration (Anthony showed a great graphic of how improving interactions can be transformative in yesterdays meeting) within research groups, between groups and for the community in general.  None of this can happen without openness and more importantly without creating psychologically safe spaces where everyone can be vulnerable and adopt a learning mindset.
I think topics that would be of benefit may fall along what are the next steps for projects.  So things like

Using Github and Googledocs to document work and discussion.
Creating dynamic lab guidebooks - the books from the hackweeks are a great example of this.
Making data interoperable and reusable

data formats
conventions
metadata
cloud optimized formats - I think Aimee will cover some of that

Licenses
PIDS
Archives

@mstudinger just pushed to https://github.com/andypbarrett/website-2024/tree/open-science-plenary and created draft PR https://github.com/ICESAT-2HackWeek/website-2024/pull/31