Add internal attribute to service template

Add internal attribute to service template table.
Depend on PR: ManageIQ/manageiq-schema#238
https://bugzilla.redhat.com/show_bug.cgi?id=1594408

@bzwei @gmcculloug Please review.

@hsong-rh Since this is upstream, can you remove the upstream wording from your commit and PR title as they make no sense.

@Fryguy Words removed.

@gmcculloug Schema PR was merged, build passed. Please merge. Thanks.

@hsong-rh @gmcculloug @bzwei I think I found a bug.
which boolean value means ServiceTemplate#internal = nil ?
objects.second.class =>
ServiceTemplateCatalog

(byebug) objects.second.service_templates.map {|x| [x.service_template_catalog_id, x.internal] } =>
[[75000000000001, nil], [75000000000001, nil], [75000000000001, nil], [75000000000001, nil]]

objects.second.service_templates.public_service_templates =>
[]

objects.second.service_templates.where.not(:internal => true) =>
[]

it is not considering NULL values because so-called three value logic.
https://stackoverflow.com/questions/17679721/why-does-postgresql-not-return-null-values-when-the-condition-is-true
thanks!

@lpichler We shouldn't have the case of internal = nil. In the base class, its default value is set to false, and only be true if the type is ServiceTemplateTransformationPlan. Are objects.second.service_templates the type of ServiceTemplateTransformationPlan?

@hsong-rh No, type is nil. And it is possible because of internal column has been added
https://github.com/ManageIQ/manageiq-schema/pull/238/files#diff-1d061c844b9d792d54472130a429e360R10 without default value false - which is ok according to our recommendations.
but in this case you have to set false value in migration to cover also other ServiceTemplate.

@lpichler I were aware of this three value logic issue when I was testing this PR. I could not remember what was the code that passed all manual tests nor can I understand how the PR finally ended up at this state. Nevertheless the following fix seems working:
scope :public_service_templates,                  ->         { where(:internal => [false nil]) }

@lpichler the default value is set using default_value_for on the model instead of at the db level.
https://github.com/ManageIQ/manageiq/blob/master/app/models/service_template.rb#L73
https://github.com/ManageIQ/manageiq/blob/master/app/models/service_template_transformation_plan.rb#L10


[FEATURE] Task Categorization and Priority Tagging

Feature description:
This feature would allow users to categorize their tasks and assign priority levels, making it easier to organize and focus on the most important tasks. Here's how we could implement this:
Task List:

Add category and priority fields to the task input form.
Modify the todoList structure to include these new fields.
Update the display of tasks to show categories and priorities.
Add sorting and filtering options based on categories and priorities.

can i be assigned this issue ??

@SSShogunn assigning to you.
Also please star ⭐ the repository before starting work


Allow skipping hot reload dn validation

Description
This change adds two new boolean settings for the Security plugin:

plugins.security.ssl.http.enforce_cert_reload_dn_verification
plugins.security.ssl.transport.enforce_cert_reload_dn_verification

Both options will default to True, which will maintain the current behaviour where performing a hot reload requires the Distinguished Names (IssuerDN, SubjectDN, and SAN) in the new certificate to match the current certificate.
When set to False, the Distinguished Names validation will be skipped when hot reloading the respective certificate (HTTP or Transport).
The current behaviour is preventing us from hot-reloading new Let's Encrypt certificates that were signed by a different intermediate CA than the original certificate, meaning we need to perform a rolling restart of each of our clusters in order to rotate the certificates. According to Let's Encrypt, one of their root CAs is expected to expire as soon as 2030 (https://letsencrypt.org/certificates/), so an approach that allowed different intermediate CAs but still rejected changed root CAs will still require us to perform a rolling restart for our clusters when the root CAs are inevitably rotated.
We do not believe adding the ability to disable this validation for hot-reloads creates any added security risk, as long as it is properly considered considering organisational context. There is no similar validation performed when changing certificates by restarting OpenSearch, and an actor that could trigger a hot-reload (requiring them to modify the certificate files on the OpenSearch node) would also likely be able to bypass the validation by restarting OpenSearch.
For example, in our operational context only the super admin user can trigger a hot-reload. The certificate for the super admin user is stored only on the OpenSearch nodes, so any malicious actor who had the super admin certificate and ability to modify the OpenSearch certificate files would also have the ability to restart OpenSearch. This makes the validation performed when triggering a hot-reload irrelevant, as any malicious actor could simply restart OpenSearch instead.

Category: New feature

Issues Resolved

Resolves https://github.com/opensearch-project/security/issues/4509, although this approach covers a slightly larger scope by allowing skipping the validation entirely, and may not be appropriate for all deployments.

Testing
Unit tests have been added that cover reloading certificates to a certificate signed by a different Certificate Authority (different root and signing CA), and validating that this is rejected when the new settings are set to true, and accepted when the settings are set to false.
Check List

[X] New functionality includes testing
[ ] New functionality has been documented - draft: https://github.com/parislarkins/documentation-website/pull/1
[ ] New Roles/Permissions have a corresponding security dashboards plugin PR
[ ] API changes companion pull request created
[X] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Thank you @parislarkins , this change looks reasonable to me.
What happens if you disable dn validation and change the dn of a node, but the dn is not in the nodes_dn list?

@cwperks I have not tested this specifically, but I expect the node would no longer be able to connect to the remote cluster. As I understand it, this is the same behaviour that would happen if the cert was changed by restarting OpenSearch as well.


Show API and debug API status in the left navigation

Rationale
Pretty much all the pages require connection to the API of the bee node and many to the debug API. However, except for the status page, it is unclear whether the connection is working.
Task
Display the connection to bee API and debug API in the bottom left corner or some other appropriate place

https://github.com/ethersphere/bee-status/pull/1/commits/8bb43ddeded3033a22db79fa210a13093361c6d1

resolved in #1


Build numbers

Hello
Why are there two build numbers for the SQL RP? Both supporting the same Azure Stack build. Reference pages are also misaligned. PowerShell compatibility was fixed with <IP_ADDRESS> however, <IP_ADDRESS> is a later build. Which version do we use? Both installs fail with the latest ASDK environment.

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

ID: dabc86bd-3770-5832-7fa5-d28856998ae6
Version Independent ID: cbdb8c6f-ad4d-9e3d-a490-f1348b5ccdfa
Content: Using SQL databases on Azure Stack
Content Source: azure-stack/operator/azure-stack-sql-resource-provider-deploy.md
Service: azure-stack
GitHub Login: @jeffgilb
Microsoft Alias: jeffgilb

@rfsonders Thank you for your feedback, could you please share the document you are referring to for this issue?

This page, with an older update date, is not in sync
https://docs.microsoft.com/en-us/azure-stack/operator/azure-stack-sql-resource-provider-deploy#prerequisites
This page, has been updated.
https://docs.microsoft.com/en-us/azure-stack/operator/azure-stack-sql-resource-provider-11300

@rfsonders Thanks for the information! I have assigned the issue to the content author to investigate further and update the document as appropriate.

@mattbriggs please re-assign as I'm off the docs team now.

@rfsonders I've passed the following comments along to the technical team.

Why are there two build numbers for the SQL RP? Which version do we use?
Both installs fail with the latest ASDK environment.

I updated this issue:

Reference pages are also misaligned. (Updated.)

#in-progress

Thanks for sharing your feedback, @rfsonders !
Actually SQL/MySQL RPs have separated release process, instead of releasing along with Azure Stack regular updates. <IP_ADDRESS> is the current latest version, and <IP_ADDRESS> is the one before that. Azure Stack 1808 update is the minimum Azure Stack version the RP versions could support. That's why you see two SQL RP versions are both corresponded to Azure Stack 1808 update.
About the deployment failure that you mentioned, could you please share a little more information about the versions of ASDK, PowerShell profile, AzureStack module. And what's the exact error that you got during deployment?
For PowerShell profile and AzureStack module, current <IP_ADDRESS> version of RP requires specific version of PowerShell modules.
https://docs.microsoft.com/en-us/azure-stack/operator/azure-stack-sql-resource-provider-deploy#deploy-the-sql-resource-provider-using-a-custom-script
Install the AzureRM.Bootstrapper module, set the profile and install the AzureStack module
Note that this might not be the most currently available version of Azure Stack PowerShell
Install-Module -Name AzureRm.BootStrapper -Force
Use-AzureRmProfile -Profile 2018-03-01-hybrid -Force
Install-Module -Name AzureStack -RequiredVersion 1.6.0

@xiaofmao Thanks! @rfsonders let me know if you have any additional questions. #please-close

Team.
Documentation is still not correct.
Please take a look at this page.
https://docs.microsoft.com/en-us/azure-stack/operator/azure-stack-sql-resource-provider-deploy
This table is confusing.
Which Azure Stack version aligns with which SQL RP. I see 2 options for 1808.
Minimum Azure Stack version | SQL RP version
Version 1808 (1.1808.0.97) | SQL RP version <IP_ADDRESS>
Version 1808 (1.1808.0.97) | SQL RP version <IP_ADDRESS>
Version 1804 (1.0.180513.1) | SQL RP version <IP_ADDRESS>

@rfsonders The Azure Stack Version # corresponds with the SQL RP versions. Both <IP_ADDRESS> and <IP_ADDRESS> require Azure Stack Version 1808 or later. Thanks for your feedback, and sorry to have confused you. Thanks, Matt

#please-close


[KYUUBI #3239] [Subtask] DorisSQLEngine - Add integration tests

Why are the changes needed?

close https://github.com/apache/incubator-kyuubi/issues/3239#issue-1338893480
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

Thanks, merging to master


[Help] TexMobject matrix - Manipulate matrix's elements

I have this matrix:
        ShiftRows_example_matrix_3 = TexMobject(r"""\begin{bmatrix}
                                                    20 & 3b & 83 & 64 \\
                                                    8f & 72 & 6f & 45 \\
                                                    61 & 6d & f9 & 2f \\
                                                    21 & 8f & ed & 6e
                                                    \end{bmatrix}""")

Is there a way to split all the values it have and for example move the first element ('20') somewhere else without changing the rest matrix's position?
Something like:
self.play( ApplyMethod( ShiftRows_example_matrix_3[0].shift , RIGHT ) )

Why not use Matrix in manim.

Thank you!
I didn't knew there was this method!
ShiftRows_example_matrix_3 = Matrix([["20","3b","83","64"],["8f","72","6f","45"],["61","6d","f9","2f"],["21","8f","ed","6e"]])
self.play(ShiftRows_example_matrix_3[0][0].shift, RIGHT)


Add two relevant papers

Hi, thanks for your excellent survey.
Please consider adding two relevant papers to your repository and paper:
[1] Title: Making Large Language Models Better Reasoners with Alignment
Link: https://arxiv.org/pdf/2309.02144.pdf
This paper proposes a constrained preference alignment method to improve the reasoning ability of LLMs.
[2] Math-Shepherd: A Label-Free Step-by-Step Verifier for LLMs in Mathematical Reasoning
Link: https://arxiv.org/pdf/2312.08935.pdf
This paper proposes a framework to automatical construct the training dataset of process reward models.
Thank you for your consideration. 😊

Thanks!

The update version is here. https://arxiv.org/abs/2312.11562
If there are any questions, please let us know.
Again, thank you very much for your attention to our work, and thank you for your notice.


Feature: Add the new Microsoft JDK distribution

[ x] I have read the CONTRIBUTING guidelines

Feature request

Microsoft moves away Azul Zulu as the Azure JDK because the contract with Azul expires on the 30th of June 2021.
Instead they decided to package their own OpenJDK distribution as announced here: https://devblogs.microsoft.com/java/announcing-preview-of-microsoft-build-of-openjdk/#frequently-asked-questions
The packages can be found here: https://www.microsoft.com/openjdk

These were already supported shortly after their release. Running sdk ls java will show you all available options.
Please do follow the contributing guidelines before opening an issue here.


space-cli should be able to apply a folder as well

The space-cli apply command currently takes a filename and applies the config in that. We need to make it capable to apply all the files present in a folder as well.
It should first look out for a project.yaml file since that would contain the config to create a project.
Also, it is possible that there can be multiple configs present in the same file

fixed in #884


e2e tests fail because of missing package.json

If you are reporting a bug, please fill in below. Otherwise feel free to remove this template entirely.
Can you reproduce the problem with latest npm?
Many errors, especially related to "missing modules", are due to npm bugs.
If you're using Windows, follow these instructions to update npm.
If you're using OS X or Linux, run this to update npm:
npm install -g npm@latest

cd your_project_directory
rm -rf node_modules
npm install

Then try to reproduce the issue again.
Can you still reproduce it?
Yes
Description
I was trying to fork react-scripts and extending it to have a more extended setup for my projects. While running the e2e tests they failed because create-react-app was not properly installed under $temp_cli_path. No node_modules folder was created. This also happened on all my CI nodes. Creating an empty package.json before running npm install $cli_path solved the issue.
I think more people could run into this issue and creating an empty package.json would make the e2e tests more stable.
Expected behavior
e2e tests should not fail on a fork with no changes.
Actual behavior
e2e tests failed.
Environment
Run these commands in the project folder and fill in their results:

npm ls react-scripts (if you haven’t ejected):
node -v: v7.4.0, v6.9.4
npm -v: 4.0.5, 3.10.10

Then, specify:

Operating system: OS X El Capitan
Browser and version: not browser related

Could you explain why the problem occurred in more detail and how the fix solves it?

When the e2e test script ran npm install $cli_path npm would print a warning that it could not find a package.json file and exit with code 0. At the same time, it did not create a node_modules folder nor did it install the react-scripts package from the local *.tgz archive. The following call create_react_app --scripts-version=$scripts_path test-app would then fail.
After adding an empty package.json a few days back the tests on our CI nodes started running through. After removing it again today to retest that this is what breaks the build, the test started to fail again in all environments.

Tagging as 0.9.1 since, whatever it is (I haven't dived in yet), it should be an easy fix.
Maybe https://github.com/facebookincubator/create-react-app/pull/1402 or something else.

Fixed by #1402.


个人信息修改功能无法保存的问题

Bug Report or Feature Request (mark with an x)

[X] Bug report -> please search issues before submitting
[ ] Feature request
[ ] Documentation issue or request

What is expected?

登入一个已存在的用户，修改个人信息时如不修改手机号码出现无法保存。

What is actually happening?

功能待修复

系统初始化时数据有问题，不存在此问题


Is it possible to add a sting var to a array?

Soled the issue right now to concat(array1, array2) but array2 is only one value.
Is there a way to do concat(array, varString)

I've solved it with concat two arrays so no more assistant needed.


Update travis with python 3.6 support

Should probably use:
matrix:
  allow_failures:
    - python: '3.6'
  include:
    - python: '2.7'

@rhafer I addressed your comments


feat(planning-unit-grid): derive planning area from custom grid

(the original, as well as expected planning area geojsons are added to this PR and can be viewed on github rich diff)
Please note that using non-grid geojson (like previous one, with "random" polygons) won't create a proper planning area - with the same query, geometry won't change at all (thus will remain as collection of polygons)
Uploading grid:

Produced planning area:

BBox:

Run result:

Manual testing steps

create new project database row (keep in mind to set NULLs for Gadm and area)
(also add relevant entry to users_projects to make it visible on frontend)
please note that, if you are using restore_dump, it can crash if you upload the same geometry more than once (missing restriction on project_id in planning_units_geometry)
send post:

curl --request POST \
  --url http://localhost:3030/api/v1/projects/PROJECT_ID/grid \
  --header 'Authorization: Bearer TOKEN' \
  --header 'content-type: multipart/form-data; boundary=---011000010111000001101001' \
  --form file=@/SHAPE_FILE_PATH_FROM_TESTS

Please note that using non-grid geojson (like previous one, with "random" polygons) won't create a proper planning area - with the same query, geometry won't change at all (thus will remain as collection of polygons)

which is the previous non-grid geojson? (I see only the new NAM grid and "unionized" PA in the repo folder in this PR)
in any case, I guess you mean non-always-touching polygons... in which case, what happens? I would expect that polygons that touch will be joined in a new merged polygon while isolated ones will stay as they are - if so, this sounds ok to me for now, until we confirm and implement (if at all) validation of contiguity of polygons for PU grid uploads.


Feat : Get users name in profile page

Overview

Get user's name in Profile page.
Changes

Add API client to get current user's id
Add API client to get user's info by userId
Display user's name in UI for testing purpose. (*UI will be implemented in the next task)

Review points

Functionality of API client. This PR doesn't include any UI change to review.
Readability of the code.

Notes

Both of (auth/me and users/{userId} has an issue, it's the same one with #255
I resolved it with temporally solutions.
Assignee Checklist:

[x] The naming convention of the PR title is correct (See the comment at the top of this template)
[x] The base branch is correct (See: Types of Branches)
[x] The branch name follows the Branch Naming Conventions
[x] The correct assignees and reviewers have been designated for this PR
[x] The coding style follows the Coding Style Guide
[x] All the related issues are associated with this PR
[x] All criteria in the associated issues are met (please tick the checkboxes)
[x] My changes do not generate new warnings or errors (especially in the console)

Reviewer Checklist:

[x] Double-check the "Assignee Checklist"
[ ] The code follows the generic best practices and our Coding Style Guide
[ ] The code is well-commented and easy to understand
[x] The UI changes accurately reflect the provided design

@nick-y-ito
Thank you for the review!
I addressed the issues you mentioned! Please have a look when you have a moment

@nick-y-ito
Thank you for the review, and sorry for missing some points!
I re-pushed my changes, please kindly take a look at them when you have a moment


Add support to allow images to span the width of the page

Opening a new issue after discussion in issue [https://github.com/asciidoctor/asciidoctor-pdf/issues/418]
It is possible to achieve this by setting the pdfwidth option to 120% but this possibility either needs to be documented or another, more obvious, way to do it needs to be implemented.
To me, the most obvious way for the percentage to work would be to let 100% actually span the page, not taking any margins into account.

I have an idea that fits more with the spirit of CSS, which we're attempting to imitate.
In CSS, percentage is always relative to the parent element. That's consistent with how % in Asciidoctor PDF works today. However, CSS encounters the same limitation we're observing with this issue. What if you want the image to span the width of the viewport?
What CSS has done is introduce an additional percentage unit that communicates the parent should be ignored and the percentage should be based on the width of the viewport. That unit is vw. I think that unit solves our problem.
To get an image to span the width of the entire page, I propose using the following:
image::image01.png[pdfwidth=100vw]

This would be easy for Asciidoctor PDF to calculate. I think we should also assume that the image should be placed relative to the edge of the page instead of aligned with the content.

Sounds good!
What about the cases where you want the image aligned with the content, would you then  use the normal percentage unit?

Correct.

Related to this issue, we've also discussed having a "cover" role for the image block. This role would instruct the converter to fit the image to the width of the page (clipping if it overflows). This would be similar to the background-size property in CSS. However, I believe that differs from the vw units.

I designed it such that the image stretches to the width of the page, but the caption for the image stays within the content width.


2.6.1

Coverage remained the same at 97.959% when pulling bea7d19931e5d87786c6c792e377a98da8bb28c6 on renan/2.6.1 into 1e93cfe902b27de959dc8f34acdab4ea31a80977 on master.


host: Migrate preflight computations from soroban-rpc

What
Migrate preflight computations from https://github.com/stellar/soroban-tools
Why
https://github.com/stellar/soroban-tools/issues/1098
Known limitations
N/A

@dmkozh PTAL

Just a reminder, if you have any unit tests for any components here, I would really appreciate moving them here as well.

Unfortunately, there aren't


Sprites made by TextSprite extension are not moving with velocity

From: https://youtu.be/6nFs_JoNXE0?t=2103
Simpler repro:
https://makecode.com/_JkH6oiJ4fUuq
With these blocks the sprite does not move:

this extension:
https://github.com/microsoft/arcade-text
This is most likely a PXT problem not an extension problem, or at a minimum we need to update documentation on how sprites can be used/made in extensions, because this scenario should work.

I wouldn't necessarily say it's a bug in pxt (https://github.com/microsoft/arcade-text/pull/3), could use docs but there are a lot of things in the game engine that could use more docs as well / this only really comes up when subclassing sprite and that's definitely on the advanced side

Fixed by: https://github.com/microsoft/arcade-text/pull/3


Router is not responding after gc

nothing suspicious in logs:
Oct 30 00:00:52 surfer nix-gc-start[4380]: deleting '/nix/store/h4z9agb794j3ygldz094kj9ic8l3qxf2-nftables-rules'
Oct 30 00:00:52 surfer nix-gc-start[4380]: deleting '/nix/store/biyrikrsg0pcdni5i4hpaa815jlnvfvv-unit-nftables.service.drv'
Oct 30 00:00:52 surfer nix-gc-start[4380]: deleting '/nix/store/bw2ikwjqv2xgf52j7bk62p2ckw1g20li-nftables-rules.drv'
Oct 30 00:00:52 surfer nix-gc-start[4380]: deleting unused links...
Oct 30 00:00:52 surfer nix-gc-start[4380]: note: currently hard linking saves -0.00 MiB
Oct 30 00:00:55 surfer nix-gc-start[4380]: 180 store paths deleted, 200.71 MiB freed
Oct 30 00:00:55 surfer systemd[1]: nix-gc.service: Deactivated successfully.
Oct 30 00:00:55 surfer systemd[1]: nix-gc.service: Consumed 12.136s CPU time, no IP traffic.
Oct 30 01:00:19 surfer systemd[1]: Started Logrotate Service.
Oct 30 01:00:20 surfer systemd[1]: logrotate.service: Deactivated successfully.
Oct 30 01:13:46 surfer hostapd[836]: wlan0-1: AP-STA-DISCONNECTED ea:c6:81:11:7e:2b
Oct 30 02:00:29 surfer systemd[1]: Started Logrotate Service.
Oct 30 02:00:29 surfer systemd[1]: logrotate.service: Deactivated successfully.
Oct 30 03:00:24 surfer systemd[1]: Started Logrotate Service.
Oct 30 03:00:25 surfer systemd[1]: logrotate.service: Deactivated successfully.
Oct 30 04:00:03 surfer systemd[1]: Started Logrotate Service.
Oct 30 04:00:03 surfer systemd[1]: logrotate.service: Deactivated successfully.
Oct 30 04:29:17 surfer dnsmasq-dhcp[914]: DHCPREQUEST(br-lan) <IP_ADDRESS> 68:3e:26:90:27:61
Oct 30 04:29:17 surfer dnsmasq-dhcp[914]: DHCPACK(br-lan) <IP_ADDRESS> 68:3e:26:90:27:61 focus
Oct 30 05:00:07 surfer systemd[1]: Started Logrotate Service.
Oct 30 05:00:07 surfer systemd[1]: logrotate.service: Deactivated successfully.
Oct 30 05:53:05 surfer hostapd[836]: wlan0-1: STA ea:c6:81:11:7e:2b IEEE 802.11: authenticated
Oct 30 05:53:05 surfer hostapd[836]: wlan0-1: STA ea:c6:81:11:7e:2b IEEE 802.11: associated (aid 2)
Oct 30 05:53:05 surfer hostapd[836]: wlan0-1: STA ea:c6:81:11:7e:2b IEEE 802.11: authenticated
Oct 30 05:53:05 surfer hostapd[836]: wlan0-1: STA ea:c6:81:11:7e:2b IEEE 802.11: associated (aid 2)
Oct 30 05:53:05 surfer hostapd[836]: wlan0-1: AP-STA-CONNECTED ea:c6:81:11:7e:2b
Oct 30 05:53:05 surfer hostapd[836]: wlan0-1: STA ea:c6:81:11:7e:2b RADIUS: starting accounting session D01FCA3DCCC7C8AD
Oct 30 05:53:05 surfer hostapd[836]: wlan0-1: STA ea:c6:81:11:7e:2b WPA: pairwise key handshake completed (RSN)
Oct 30 05:53:05 surfer hostapd[836]: wlan0-1: EAPOL-4WAY-HS-COMPLETED ea:c6:81:11:7e:2b
Oct 30 05:53:05 surfer hostapd[836]: wlan0-1: STA ea:c6:81:11:7e:2b RADIUS: starting accounting session D01FCA3DCCC7C8AD
Oct 30 05:53:05 surfer hostapd[836]: wlan0-1: STA ea:c6:81:11:7e:2b WPA: pairwise key handshake completed (RSN)
Oct 30 05:53:05 surfer dnsmasq-dhcp[914]: DHCPREQUEST(br-lan) <IP_ADDRESS> ea:c6:81:11:7e:2b
Oct 30 05:53:05 surfer dnsmasq-dhcp[914]: DHCPACK(br-lan) <IP_ADDRESS> ea:c6:81:11:7e:2b OnePlus-7T
Oct 30 06:00:22 surfer systemd[1]: Started Logrotate Service.
Oct 30 06:00:22 surfer systemd[1]: logrotate.service: Deactivated successfully.
Oct 30 07:00:26 surfer systemd[1]: Started Logrotate Service.
Oct 30 07:00:26 surfer systemd[1]: logrotate.service: Deactivated successfully.

TODO: run gc manually and check the results

no longer happens


six.moves.urllib.request added to install.py but no requirements.txt

Some of my deployment automation failed because six.moves.urllib.request was added in this commit. Before you could run install.py with just the base Python and no dependencies. Could you publish a requirements.txt in case more dependencies are added?
(this is not urgent for me anyways right now because I am able to see the error and add the library manually myself).

Ah, I am pretty green with python and thought that was a basic library.


Vue2 support

I just can't run the example code in my vue2 project. Does this plugin support vue2?

And I think the registering part and creating part should be put before the using part, cause I need to created the models before I use.
Just my opinion.

Done!


feat: Add "startWeekOn" to the DatePicker

Fixes: https://github.com/tremorlabs/tremor/issues/552
Demo
https://www.loom.com/share/922bba28526a4423bb64d73f8a31029a
What
Add the option to choose the day the week will start on in the DatePicker calendar
Why
Because previously it was always defaulted to Monday and users had no control over it
How
Adding a new prop to the component called weekStartsOn  which takes in a day of the week
Testing
npm run storybook
and go to InputElement > DatePicker > Uncontrolled with weekStartsOn

Hello @mitrotasios and @christopherkindl !
Could you review this PR, please?

Closed with #643


sort_handle_column is not working

That's because column with fun renders content outside of table like it was in https://github.com/smpallen99/ex_admin/issues/41
table_for(category.category_properties) do
  column "", [], fn(_) ->
    i "", class: "fa fa-bars handle", "aria-hidden": "true"
  end
  column :displyable
end

Looks like it is Xain related issue, b/c Xain.i returns :ok instead of content.

Well, I found. It was broken by https://github.com/smpallen99/ex_admin/pull/116/commits/cae5749ad19b3a7259976f1df59005a1f1b6768c

Yes, I was a little worried about that change, but seemed like the right thing to do. It was a but that was introduced with the theme support I think. I worked around it by forcing them to something like row “Name”, &(text &1.full_name) instead of the documented row “Name”, &(&1.full_name).
Is this casing big issues for you?

On May 22, 2016, at 5:39 AM, Roman Smirnov<EMAIL_ADDRESS>wrote:
Well, I found. It was broken by cae5749 https://github.com/smpallen99/ex_admin/pull/116/commits/cae5749ad19b3a7259976f1df59005a1f1b6768c
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub https://github.com/smpallen99/ex_admin/issues/128#issuecomment-220823166

Yes, Xain.tag does not return the contents, only the rendered html. I spent a few hours the other day trying to fix that without any success and gave up.

On May 22, 2016, at 5:31 AM, Roman Smirnov<EMAIL_ADDRESS>wrote:
Looks like it is Xain related issue, b/c Xain.i returns :ok instead of content.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub https://github.com/smpallen99/ex_admin/issues/128#issuecomment-220822868

@smpallen99 Atm it renders like:

I guess, correct solution would be add case clause instead of replacing
for field <- columns do
  case field do
    {f_name, %{fun: fun}} when is_function(fun) ->
      td ".td-#{parameterize f_name} #{fun.(resource)}"
    {f_name, fun} when is_function(fun) ->
      td ".td-#{parameterize f_name} #{fun.(resource)}"
    {f_name, opts} ->
      build_field(resource, conn, {f_name, Enum.into(opts, %{})}, fn(contents, f_name) ->
        td ".td-#{parameterize f_name} #{contents}"
      end)
  end
end

@smpallen99 Ignore previous message.. adding clause is not working as well.
I'm going to revert #116 and fix it in other way.

Fixed.


Maturity Assessment (Apache)

ASF Maturity model:
http://community.apache.org/apache-way/apache-project-maturity-model.html
Example of a project using it:
https://cwiki.apache.org/confluence/display/GEODE/Geode+Podling+Maturity+Assessment

The Apache Maturity Model appears to be similar to the CII-Census.
I did not make a 1-to-1 comparison, but the items seem similar and the way they are answered is comparable. (met or unmet with explanation).
We already identified metrics to address most items.
A detailed analysis of the maturity model might inform new metrics.


[Bug Fix] fix bug in label encoding of exp_store

What do these changes do?

As titled.
Related issue number

#2013

This would cause unexpected results for bi-11. Here, we fix label encoding in GraphProxy. In the future, it would be fixed in ExpStore.

Codecov Report

Merging #2014 (284f96b) into main (819ea4a) will decrease coverage by 3.75%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2014      +/-   ##
==========================================
- Coverage   44.04%   40.29%   -3.76%
==========================================
  Files          89       89
  Lines        9838     9838
==========================================
- Hits         4333     3964     -369
- Misses       5505     5874     +369

Impacted Files
Coverage Δ

python/graphscope/tests/unittest/test_java_app.py
0.00% <0.00%> (-100.00%)
:arrow_down:

python/graphscope/deploy/hosts/cluster.py
29.41% <0.00%> (-58.83%)
:arrow_down:

python/graphscope/analytical/app/java_app.py
24.24% <0.00%> (-44.95%)
:arrow_down:

python/graphscope/framework/utils.py
40.10% <0.00%> (-14.85%)
:arrow_down:

python/graphscope/framework/app.py
69.95% <0.00%> (-10.33%)
:arrow_down:

python/graphscope/framework/graph_schema.py
55.28% <0.00%> (-6.98%)
:arrow_down:

python/graphscope/tests/conftest.py
48.21% <0.00%> (-4.65%)
:arrow_down:

python/graphscope/client/rpc.py
86.11% <0.00%> (-3.71%)
:arrow_down:

python/graphscope/client/session.py
63.65% <0.00%> (-3.20%)
:arrow_down:

python/graphscope/analytical/udf/utils.py
36.17% <0.00%> (-3.20%)
:arrow_down:

... and 3 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 819ea4a...284f96b. Read the comment docs.


AttributeError: module 'igraph' has no attribute 'VertexClustering'

Hi there, I get this error when trying to run Compass, tried uninstalling and reinstalling igraph but this did not help. Any idea how to fix this? Thanks!

What version of igraph is installed?

igraph-0.9.8 python-igraph-0.9.8

I have not been able to replicate this, Compass microclustering works on my windows boot with igraph-0.9.8 and leidenalg 0.8.8. I would guess leidenalg is worth trying to reinstall, as Compass only uses igraph through the leiden clustering algorithm (unless scikit-learn's nearest neighbors is using it under the hood). The igraph module also has VertexClustering as an attiribute on my machine.
If you can import igraph in a python console and enter igraph.VertexClustering and get <class 'igraph.clustering.VertexClustering'>  then it indicates igraph is correctly installed, but Compass is likely installed incorrectly in some way. I would try pip uninstall compass-sc and reinstalling it to the same python environment as igraph is installed to (for instance a conda environment if that is where igraph is installed correctly). If igraph does not have a VertexClustering attribute in the python console, it's most likely an installation issue with igraph which I have no particular expertise with beyond their documentation.

Hi, Compass was correctly installed so it was the igraph installation. I solved the problem by uninstalling and reinstalling igraph pip uninstall igraph and python -m pip install python-igraph after the Compass install. Thanks for your help.

I'm glad the issue was resolved.


Need a unity-test JPL C-name Endpoint for Unity system.

need to hear back from @jonathansmolenski or @galenatjpl on an API gateway being set up so that we can point the CName at it.


Cookie question

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm asking a question and not reporting a bug or requesting a feature
[X] I've looked through the README
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've searched known issues and the bugtracker for similar questions including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Please make sure the question is worded well enough to be understood
Can you answer at simple question.
When I'm running yt-dlp like this "yt-dlp https://www.youtube.com/watch?v=g9o12X6sG_4 --dump-json --cookies "www.youtube.com_cookies.txt" --verbose" cookie file works correctly
But, when I pass path to this file in python script (in Docker) like this yt_dlp.YoutubeDL({"cookiefile": 'www.youtube.com_cookies.txt'}) it didnt work (but file exists in container FS)
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
this is part of output when running like app

[debug] Command-line config: ['-vU', 'https://www.youtube.com/watch?v=g9o12X6sG_4', '--dump-json', '--cookies', 'www.youtube.com_cookies.txt', '--verbose']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp-nightly-builds [96da95250] (pip)
[debug] Python 3.10.12 (CPython x86_64 64bit) - Linux-5.15.0-105-generic-x86_64-with-glibc2.35 (OpenSSL 3.0.2 15 Mar 2022, glibc 2.35)
[debug] exe versions: none
[debug] Optional libraries: Cryptodome-3.20.0, brotli-1.1.0, certifi-2020.06.20, mutagen-1.47.0, requests-2.31.0, secretstorage-3.3.1, sqlite3-3.37.2, urllib3-2.2.1, websockets-12.0
[debug] Proxy map: {}
[debug] Request Handlers: urllib, requests, websockets
[debug] Loaded 1810 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp-nightly-builds/releases/latest
Latest version<EMAIL_ADDRESS>from yt-dlp/yt-dlp-nightly-builds
yt-dlp is up to date<EMAIL_ADDRESS>from yt-dlp/yt-dlp-nightly-builds)
[youtube] Extracting URL: https://www.youtube.com/watch?v=g9o12X6sG_4
[youtube] g9o12X6sG_4: Downloading webpage
[debug] [youtube] Extracted SAPISID cookie
[youtube] g9o12X6sG_4: Downloading ios player API JSON
[youtube] g9o12X6sG_4: Downloading android player API JSON
WARNING: [youtube] Skipping player responses from android clients (got player responses for video "aQvGIIdgFDM" instead of "g9o12X6sG_4")
[debug] Loading youtube-nsig.7d1f7724 from cache
[debug] [youtube] Decrypted nsig FHrc8X1S5rN-xu6awm => odQwLywhhVT5Tg
[debug] Loading youtube-nsig.7d1f7724 from cache
[debug] [youtube] Decrypted nsig oz6ePvZCBqfljhsF72 => yihB3aNpg0VXDA
[youtube] g9o12X6sG_4: Downloading m3u8 information
[debug] Sort order given by extractor: quality, res, fps, hdr:12, source, vcodec:vp9.2, channels, acodec, lang, proto
[debug] Formats sorted by: hasvid, ie_pref, quality, res, fps, hdr:12(7), source, vcodec:vp9.2(10), channels, acodec, lang, proto, size, br, asr, vext, aext, hasaud, id
[debug] Default format spec: bestvideo*+bestaudio/best
[info] g9o12X6sG_4: Downloading 1 format(s): 616+251
*here is big json*

this output in script

[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8 (No ANSI), error utf-8 (No ANSI), screen utf-8 (No ANSI)
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp [ff0779267] (pip) API
[debug] params: {'audio-quality': 0, 'format': 'bestaudio/best', 'restrictfilenames': True, 'noplaylist': True, 'nocheckcertificate': True, 'ignoreerrors': False, 'logtostderr': False, 'quiet': True, 'no_warnings': True, 'default_search': 'auto', 'verbose': True, 'cookiefile': '/app/www.youtube.com_cookies.txt', 'compat_opts': set(), 'http_headers': {'User-Agent': 'Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/95.0.4638.17 Safari/537.36', 'Accept': 'text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8', 'Accept-Language': 'en-us,en;q=0.5', 'Sec-Fetch-Mode': 'navigate'}}
[debug] Python 3.8.19 (CPython x86_64 64bit) - Linux-5.15.0-105-generic-x86_64-with-glibc2.34 (OpenSSL 3.0.11 19 Sep 2023, glibc 2.36)
[debug] exe versions: ffmpeg 5.1.4-0 (setts), ffprobe 5.1.4-0
[debug] Optional libraries: Cryptodome-3.20.0, brotli-1.1.0, certifi-2024.02.02, mutagen-1.47.0, requests-2.31.0, sqlite3-3.40.1, urllib3-2.2.1, websockets-12.0
[debug] Proxy map: {}
[debug] Request Handlers: urllib, requests, websockets
[debug] Loaded 1810 extractors
[youtube] Extracting URL: https://www.youtube.com/watch?v=ZE5zXLOyEOQ3
[youtube] ZE5zXLOyEOQ: Downloading webpage
[youtube] ZE5zXLOyEOQ: Downloading ios player API JSON
[youtube] ZE5zXLOyEOQ: Downloading android player API JSON
ERROR: [youtube] ZE5zXLOyEOQ: Get the best YouTube experience. Sign in to save videos, subscribe, and more

maybe i doing something wrong?

Looks like you could be running into #8227
see this comment for workarounds: https://github.com/yt-dlp/yt-dlp/issues/8227#issuecomment-1867184804

It's not like this. I doing this steps.

Open browser in private mode
Go to youtube
Login
Export cookies
Copy file with cookies to VPS
Run yt-dlp as app -> Works fine
Copy file again to vps
Run docker build (it copying file with cookies to container and run it)
Try to remote call YoutubeDL.py with cookiefile param -> error in description
Making step 6 -> Works fine
So I think cookies not expired at this moment

If passing cookies works outside of the container before and after, then it seems like a problem with your docker container rather than yt-dlp.


Support for seelie.me

Hello.
First up great work. Was looking for an achievement scanner and finally found one that worked perfectly.
While I personally use paimon.moe for achievements there are also other sites like seelie.me that support achievements.
Seelie.me is also offering an data import/export feature which should allow you to add the data like with paimon.moe

Will add this soon. Thanks for the suggestion!


[BugFix] add dropped flag to avoid db dropped after db got from catalog

What type of PR is this：

[x] bug
[ ] feature
[ ] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：

Fixes #10381
Fixes #10346
Problem Summary(Required) ：

Many places call db.writelock after getting db from catalog, and then write bdb log. But after getting db from catalog, the db may still be dropped, which will cause the NPE error when replay bdb log. So add a dropped flag to cover this case. The set and check of dropped flag should be in the db lock.
Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] I have added user document for my new feature or new function

[FE PR Coverage Check]
:disappointed: fail : 27 / 63 (42.86%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/catalog/HiveTable.java
0
2
00.00%
[287, 288]

:large_blue_circle:
com/starrocks/leader/ReportHandler.java
0
1
00.00%
[937]

:large_blue_circle:
com/starrocks/load/InsertOverwriteJobRunner.java
0
3
00.00%
[203, 204, 205]

:large_blue_circle:
com/starrocks/consistency/CheckConsistencyJob.java
0
3
00.00%
[256, 257, 258]

:large_blue_circle:
com/starrocks/alter/MaterializedViewHandler.java
0
2
00.00%
[723, 724]

:large_blue_circle:
com/starrocks/server/LocalMetastore.java
6
16
37.50%
[537, 538, 566, 1269, 4176, 4391, 4392, 4432, 4528, 4529]

:large_blue_circle:
com/starrocks/alter/Alter.java
6
12
50.00%
[136, 172, 240, 419, 518, 629]

:large_blue_circle:
com/starrocks/scheduler/PartitionBasedMaterializedViewRefreshProcessor.java
2
4
50.00%
[156, 549]

:large_blue_circle:
com/starrocks/clone/TabletScheduler.java
1
2
50.00%
[582]

:large_blue_circle:
com/starrocks/load/PartitionUtils.java
1
2
50.00%
[33]

:large_blue_circle:
com/starrocks/alter/AlterJobV2.java
1
2
50.00%
[220]

:large_blue_circle:
com/starrocks/load/loadv2/BrokerLoadJob.java
1
2
50.00%
[287]

:large_blue_circle:
com/starrocks/clone/DynamicPartitionScheduler.java
1
2
50.00%
[346]

:large_blue_circle:
com/starrocks/alter/SchemaChangeHandler.java
1
2
50.00%
[1206]

:large_blue_circle:
com/starrocks/clone/TabletSchedCtx.java
1
2
50.00%
[911]

:large_blue_circle:
com/starrocks/catalog/Database.java
4
4
100.00%
[]

:large_blue_circle:
com/starrocks/task/AlterReplicaTask.java
2
2
100.00%
[]

[FE PR Coverage Check]
:disappointed: fail : 30 / 61 (49.18%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/catalog/HiveTable.java
0
2
00.00%
[287, 288]

:large_blue_circle:
com/starrocks/consistency/CheckConsistencyJob.java
0
3
00.00%
[256, 257, 258]

:large_blue_circle:
com/starrocks/alter/MaterializedViewHandler.java
0
2
00.00%
[723, 724]

:large_blue_circle:
com/starrocks/server/LocalMetastore.java
6
16
37.50%
[537, 538, 566, 1269, 4176, 4391, 4392, 4432, 4528, 4529]

:large_blue_circle:
com/starrocks/alter/Alter.java
6
12
50.00%
[136, 172, 240, 419, 518, 629]

:large_blue_circle:
com/starrocks/scheduler/PartitionBasedMaterializedViewRefreshProcessor.java
2
4
50.00%
[156, 549]

:large_blue_circle:
com/starrocks/clone/TabletScheduler.java
1
2
50.00%
[582]

:large_blue_circle:
com/starrocks/load/loadv2/BrokerLoadJob.java
1
2
50.00%
[287]

:large_blue_circle:
com/starrocks/task/AlterReplicaTask.java
1
2
50.00%
[169]

:large_blue_circle:
com/starrocks/clone/DynamicPartitionScheduler.java
1
2
50.00%
[346]

:large_blue_circle:
com/starrocks/alter/SchemaChangeHandler.java
1
2
50.00%
[1206]

:large_blue_circle:
com/starrocks/clone/TabletSchedCtx.java
1
2
50.00%
[911]

:large_blue_circle:
com/starrocks/catalog/Database.java
4
4
100.00%
[]

:large_blue_circle:
com/starrocks/load/PartitionUtils.java
2
2
100.00%
[]

:large_blue_circle:
com/starrocks/leader/ReportHandler.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/load/InsertOverwriteJobRunner.java
3
3
100.00%
[]

[FE PR Coverage Check]
:disappointed: fail : 27 / 61 (44.26%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/catalog/HiveTable.java
0
2
00.00%
[287, 288]

:large_blue_circle:
com/starrocks/load/InsertOverwriteJobRunner.java
0
3
00.00%
[203, 204, 205]

:large_blue_circle:
com/starrocks/consistency/CheckConsistencyJob.java
0
3
00.00%
[256, 257, 258]

:large_blue_circle:
com/starrocks/alter/MaterializedViewHandler.java
0
2
00.00%
[723, 724]

:large_blue_circle:
com/starrocks/server/LocalMetastore.java
6
16
37.50%
[537, 538, 566, 1269, 4176, 4391, 4392, 4432, 4528, 4529]

:large_blue_circle:
com/starrocks/alter/Alter.java
6
12
50.00%
[136, 172, 240, 419, 518, 629]

:large_blue_circle:
com/starrocks/scheduler/PartitionBasedMaterializedViewRefreshProcessor.java
2
4
50.00%
[156, 549]

:large_blue_circle:
com/starrocks/clone/TabletScheduler.java
1
2
50.00%
[582]

:large_blue_circle:
com/starrocks/load/PartitionUtils.java
1
2
50.00%
[33]

:large_blue_circle:
com/starrocks/load/loadv2/BrokerLoadJob.java
1
2
50.00%
[287]

:large_blue_circle:
com/starrocks/clone/DynamicPartitionScheduler.java
1
2
50.00%
[346]

:large_blue_circle:
com/starrocks/alter/SchemaChangeHandler.java
1
2
50.00%
[1206]

:large_blue_circle:
com/starrocks/clone/TabletSchedCtx.java
1
2
50.00%
[911]

:large_blue_circle:
com/starrocks/catalog/Database.java
4
4
100.00%
[]

:large_blue_circle:
com/starrocks/leader/ReportHandler.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/task/AlterReplicaTask.java
2
2
100.00%
[]

run all


Connection pool is full, discarding connection

I'm getting a massive amount of these exceptions while trying to retrieve S3 logs, as I'm retrieving each object in separate threads.
WARNING:botocore.vendored.requests.packages.urllib3.connectionpool:Connection pool is full, discarding connection: bucket.s3.amazonaws.com
Should I be closing connections after using them? Or is this in boto's hands?
thanks

Should not have to manually close any connections, botocore should be handling this for you.  If you have sample code the demonstrates the issue I'd be happy to investigate further.

Hi James, thanks for helping
I'm trying to retrieve (and then delete) all logs present in an S3 bucket, in the fastest possible way. I'm retrieving them all in separate threads, so that it doesn't wait for one request to complete before starting another. I wrote the code in a bit of a rush, and just now noticed that all threads are using the same client object, which might be the reason for the above exceptions.
I'm not aware of how botocore internals work. Does a client object reuse connections? / Avoid repeat TLS setup?
What is the most efficient way to fetch multiple objects?
thanks,
Jon
def fetch_logs(bucket):
    s3 = boto3.client('s3')

    # Retrieve list of object keys for logs
    objects = s3.list_objects(Bucket=bucket, Prefix='logs/')
    if 'Contents' not in objects:
        return
    objects = [obj['Key'] for obj in objects['Contents']]

    # Start processing objects
    threads = []
    for key in objects:
        threads.append(Thread(target=_fetch_log, args=[s3, bucket, key]))
        threads[-1].start()

    # Wait till all objects processed
    for thread in threads:
        thread.join()

def _fetch_log(s3, bucket, key):
    body = s3.get_object(Bucket=bucket, Key=key)['Body'].read().decode('utf-8')
    process...
    s3.delete_object(Bucket=bucket, Key=key)

I'm having trouble reproducing this issue. Using the code as you have it, I don't run into it. I modified it to run on more than 1000 objects at a time and ran into a RuntimeError rather than a connection pool. I'm gonna try with bigger objects to see if that does anything, but in the meantime could tell me approximately the size and number of objects you're looking at?
Also, just to note, list_objects returns a max of 1000 objects at a time.

This issue is being closed due to inactivity

I'm having the same issue when writing to DynamoDB using 20 connections/threads. Any suggestions on debugging/fixing it?

Not sure, sorry, haven't had the time to investigate further. Though it doesn't seem to break anything for my app. I wonder if it is affecting network efficiency though.

Yes the network calls complete eventually but I'm experiencing delays of up to 10 minutes for a program that should take ~1 minute.

Got this on my experiments with Celery + eventlet + DynamoDB.
Celery worker:

@JordonPhillips this happens for me as well with boto3, I can share the code that's causing it while trying to download several hundred of files from S3 - or is this a known issue now?

Please also see related discussion: https://github.com/boto/boto3/issues/268#issuecomment-199759246

This one can happen if the number of threads that create a http connection is higher than the default connection pool size of botocore. which is 10.
https://github.com/boto/botocore/issues/766

Wondering the same thing, as @farshidz is this connection pool a process singleton?

I'm having the same issue when trying to send bulk emails out with SES

Same with Lambda.

Not sure what your use case is @JustinTArthur, but I was able to fix this problem in my situation. I was attempting to send requests to SES with a Lambda and after about 1000ish emails sent on different threads I'd get the connection pool error. For some reason, new threads kept being made and weren't being discarded.
What fixed it was changing how I instantiated my ThreadPoolExecutor from this:
with concurrent.futures.ThreadPoolExecutor(max_workers=4) as executor:
to this:
with concurrent.futures.ThreadPoolExecutor(4) as executor:
And also making sure that I wasn't going over my maximum send rate for AWS.

Same with Lambda.

Same with SQS

This warning is fine. If you dig into the urllib3 connection pool code, it's basically the pool of persistent connections and not the maximum number of concurrent connections you can have. The connections in the pool are re-used if they haven't been closed by the endpoint.
If you'd like to boost the size of this pool, it can be done per-endpoint using the low level botocore config. boto3 example:
client_config = botocore.config.Config(
    max_pool_connections=25,
)
boto3.client('lambda', region_name=region, config=client_config)

Botocore Config Documentation:
https://botocore.amazonaws.com/v1/documentation/api/latest/reference/config.html

Solution here .. use new feature max_pool_connections
s3_client = boto3.client('s3', config=botocore.client.Config(max_pool_connections=50))

I am having the same issue with a package that use boto3 for reading parquets when they are in S3.
Why doesn't the max pool connections parameter appear in the docs so I can set it from a config file or environment variable?

The max_pool_connection config doesn't work for me, I keep seeying the message:
"[WARNING]	{...} Connection pool is full, discarding connection: archive.org. Connection pool size: 10"
My config:
`import boto3
from botocore.config import Config
client_config = Config(
max_pool_connections=20,
)
dynamodb = boto3.resource('dynamodb',
region_name='us-west-2',
config=client_config)`

The max_pool_connection config doesn't work for me, I keep seeying the message:

[WARNING]	{...} Connection pool is full, discarding connection: {...}. Connection pool size: 10

My config:
import boto3
from botocore.config import Config

client_config = Config(
    max_pool_connections=20,
)
dynamodb = boto3.resource('dynamodb',
                          region_name='us-west-2',
                          config=client_config)

+1 - I am experiencing the same issue

The max_pool_connection config doesn't work for me, I keep seeying the message:

[WARNING]	{...} Connection pool is full, discarding connection: {...}. Connection pool size: 10

My config:
import boto3
from botocore.config import Config

client_config = Config(
    max_pool_connections=20,
)
dynamodb = boto3.resource('dynamodb',
                          region_name='us-west-2',
                          config=client_config)

I have the same issue

The max_pool_connection config doesn't work for me, I keep seeying the message:

[WARNING]	{...} Connection pool is full, discarding connection: {...}. Connection pool size: 10

My config:
import boto3
from botocore.config import Config

client_config = Config(
    max_pool_connections=20,
)
dynamodb = boto3.resource('dynamodb',
                          region_name='us-west-2',
                          config=client_config)

I have the same issue

You might need to use boto3.client instead.

@crojasqnextroll is correct.
For the record, here are the official docs:

Create a configuration object (param max_pool_connections): https://botocore.amazonaws.com/v1/documentation/api/latest/reference/config.html
Get a resource (suply configuration object in param config): https://boto3.amazonaws.com/v1/documentation/api/latest/reference/core/session.html#boto3.session.Session.resource


Sage Docs Browser R1

@paulmasson
Have a look at:
Sage Docs Browser R1
If you are interested I can do a pull request.

I'm still considering this idea. The intention of the project is really to have individual interlinked pages for each function or operation. I'm not really sure I like seeing so much information at one time in that context.

I understand - and agree to the sense you describe.
And it's also nice to have multiple and alternative ways of
viewing/slicing/dicing that content.
So I could see a separate folder with varies viewing scripts. One script
might highlight only files updated in the last three months.  Another might
show where each function has been used in other functions.
The primary access would be through the vehicle you describe. The existence
of other viewing types would help with specialty uses as well as being a
demo of how flexible your data is when hosted on GitHub and accessible via
the GitHub API.
On Sat, Feb 6, 2016 at 1:56 PM paulmasson<EMAIL_ADDRESS>wrote:

I'm still considering this idea. The intention of the project is really to
have individual interlinked pages for each function or operation. I'm not
really sure I like seeing so much information at one time in that context.
—
Reply to this email directly or view it on GitHub
https://github.com/paulmasson/sagemath-docs/issues/1#issuecomment-180871919
.


Allow multiple provider contexts to be added at the same time

Should this be an RFC?

[X] This is not a substantial change

Which package is this a feature request for?
Context (@lit-labs/context)
Description
It would be useful to be able to provide multiple contexts at the same time; allowing a provider component to be instantiated the same way but with a dynamic array of contexts
  private provider = new MultiContextProvider(this, [
    [loggerContext, {
      log: (msg) => {
        console.log(`[my-app] ${msg}`);
      },
    }],
    [themeContext, myTheme],
    [apiContext, myAPI],
  ]);

Alternatives and Workarounds
It's possible to create a separate provider for every injected service.

I had a conversation with @justinfagnani about this same thing in the Lit discord channel:
https://discord.com/channels/1012791295170859069/1053423934453854289


METRON-246 metron_streaming role needs to explicitly install defaults

Tested on quick-dev-platform and bare metal.

+1 Ran it up in AWS

+1, looks good

+1


[Feature Request]: Improve the UI of the Frequent Asked Questions

Is there an existing feature request for this?

[X] I have searched the existing feature requests

Describe the feature you would like to request
I would like to improve the UI of the frequently asked question section under GSSOC"24 and Hacktofest'24.
thank you

What problem does this feature solve?
This is improve the UI of the frequently asked question section which make   user experience more engaging
Alternative solutions
No response
Confirmation

[X] I have read the Bitbox Contributing Guidelines
[X] I am a GSSOC-EXT contributor
[X] I am a HACKTOBERFEST contributor
[X] I have starred the Bitbox repository
[X] I would like to help implement this feature

Please share any screenshots of the current FAQ section layout or any previous work you have done on similar features. This will help us better understand the changes you propose. and  we assign you this issue with labels

I work on navbar last time

 I work on navbar last time

Sorry @Himanshu-kumar025, We can't assign you because last time of navbar is not looking good and not fit to the website color pallate. If you give design of FAQ then we will think about this.

That kind of color plat I wanna  to apply .

Hyy @Himanshu-kumar025, I'm assigning this issue to you.

Can you give the star to the repo?


Why not use JSON login?

I can see that the login function in session.py uses the login API to get the session ID by requesting the entire page.
I was wondering why not just get the JSON response from the login API, which includes the x-token already, the same way the browser does when you login using the box on the top-right?
Also I was curious why you did not not use the "json" and "cookies" parameters for the requests functions instead of setting the data to a json.dumps result and adding cookies to the header?
This is a (working) example of how I implemented this request for one of my personal projects:
request = requests.post(
    "https://scratch.mit.edu/accounts/login/",
    json={
        "username": username,
        "password": password
    },
    headers={
        "x-requested-with": "XMLHttpRequest",
        "referer": "https://scratch.mit.edu/",
        "x-csrftoken": token
    },
    cookies={
        "scratchcsrftoken": token
    }
)

and request.json()[0]["token"] to get the x-token. The session ID can still be obtained the same way through "Set-Cookie".

I didn't use scratch.mit.edu/accounts/login because I couldn't get it to work for some reason. I always get [{'username': '', 'num_tries': 0, 'success': 0, 'msg': '', 'messages': [], 'id': None}] from the API.

@TimMcCool Try adding the trailing slash at the end of url like this -> https://scratch.mit.edu/accounts/login/. I learned this myself the hard way and this small thing was the entire cause of this problem

Thanks, I'll try that


Angelic_search

Added angelic_search in planning.py, tests in test_planning.py  and notebook planning_angelic_search.ipynb,
-- Refinements function supports high level plans with multiple implementations.
-- Angelic search doesn't work correctly for all instances
-- Making progress is not implemented yet

The Travis CI build fails because of some problematic tests in mdp.py. These have been fixed in #942 and the build will pass once that is merged.

@delaray the unit test functions have been temporarily fixed in #942
The Travis build will pass once that is merged. There are no changes required to this pull request (atleast for this issue)

@ad71 Awesome. Thank you.

Angelic search code merged into master.


docs: Correct company name

https://github.com/treely/openyieldtables/blob/main/CONTRIBUTING.md
Please check the commit message format. I think something like docs: Correct company name is good.

please recheck


Update Helm Chart to easily supply pullSecret

We have a requirement to mirror images to a private authenticated registry that we pull from. We need this helm chart to update the defined pullSecret for each service.

Hi @tomwganem !
Do you need separate secrets for each service or is it enough to set global secrets for your private registry that then apply to all keptn services?


a

[ ] Changelog.
[ ] Tests added.
[ ] Security.
[ ] Docs.

[ ] Changelog.
[ ] Tests added.
[ ] Security.
[ ] Docs.

Can one of the admins verify this patch?


ci(Mergify): configuration update

This change has been made by @LudovicoPiero from Mergify config editor.

@Mergifyio rebase


feat(pingcap/tidb): fail the CI build when found long time cost test cases

Fixes #2441

no need the report step anymore, also remove the container, we will send the data by cloud events in global CI server.
currently hardcode the threshold value.
fail in build when long time case found, contributors should fix it, make it run fast.

Signed-off-by: wuhuizuo<EMAIL_ADDRESS>

/hold
Should finish the tests manually.

testing in:

[ ] https://do.pingcap.net/jenkins/job/pingcap/job/tidb/job/ghpr_unit_test/33146/

Please note, please make sure that all ut cases in the current tidb master branch can be completed within the threshold time. Otherwise, after it takes effect, all PR merges will be blocked.

In addition, please note that there may be a certain fluctuation range in the duration of test execution. If there are currently some test cases whose execution time is very close to the threshold, the execution of these test cases may also exceed the threshold.

/unhold


Invalid csrf redirect

This PR updates Nginx Auth Proxy for 403 errors to:

read the X-Status-Message HTTP header and check if the 403 error is because of an invalid CSRF token
if yes, it redirects to the same page. This will reload the page with a new CSRF token.
if no, shows access denied page

Possibly also worth adding a test to the Nginx tests for this too?


fix(Toggle): do not manually set height since Container uses this.h and style sets default h

Description

Manually setting the Toggle's height causes the "hSetByUser" flag to be set to true in withThemeStyles, preventing a theme update from updating to the new height set in styles. Because the Container references this.h, which will pull from the style mappings, this isn't necessary anyway. In the future, we may want to consider automatically emitting "itemChanged" events from withThemeStyles when the theme updates h/w anyway, but TBD.
References

Testing

Should resolve the failing Toggle test here: https://lui-tests.devplat.comcast.com/prValidateSanity/Aug-14-2023-Mon-11-44-46-EDT/index.html
Automation

Checklist

[ ] all commented code has been removed
[ ] any new console issues have been resolved
[ ] code linter and formatter has been run
[ ] test coverage meets repo requirements
[ ] PR name matches the expected semantic-commit syntax

Test Execution Passed.

:tada: This PR is included in version @lightningjs/ui-components-v2.9.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Example for Tensorflow 2.x and smdebug

Issue #, if available:
Description of changes:
Adding examples for using TF 2.x fit API with SageMaker Debugger v0.7.1+.
Example 1: Modifying the training script to add sagemaker debugger hook, to be used with a custom container
Example 2: Using sagemaker debugger without modifying training script but configuring the debugger hook through SageMaker Estimator.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@Vikas-kum for review

What is custom container vs default container?

What is custom container vs default container?

BYOC vs SM DLC


Plexdrive playing the first file that is opened

With Plexdrive 5.2.1, I will open a file on computer startup from the mount. Then, when I close the file and try to play a different file, it still plays the same file, I guess saved in cache. If I try to forward the video, it hangs on one frame. I don't see anything in the logs.
This is on Ubuntu 20.04
Any idea why this might be happening?

If it matters, I deleted the cache.bolt file from .plexdrive but a new one wasn't built.

This problem gets fixed when I restart the computer, leading me to think that somehow the caches are getting polluted. Maybe because I'm starting a file, stopping it in the middle, then immediately opening a new one.

So it seems to work when I run it from command line but not when I use the systemd script. Maybe systemd is making a separate cache?

Nvm seems to be working now. Guess I had to run it from command line once to let the cache.bolt file build properly...

Never mind the actual problem was that plexdrive was being run as root when I ran it as a service, so the cache.bolt file was populated in root/.plexdrive instead of $HOME/.plexdrive . There are 2 ways to resolve this:

Set the user in your service file as you User=____
Delete the cache.bolt file in root/.plexdrive


registry v2 schema1 support will be removed in an upcoming release

docker info
Client:
 Debug Mode: false

Server:
 Containers: 38
  Running: 3
  Paused: 0
  Stopped: 35
 Images: 9
 Server Version: 19.03.1
 Storage Driver: overlay2
  Backing Filesystem: xfs
  Supports d_type: true
  Native Overlay Diff: true
 Logging Driver: json-file
 Cgroup Driver: cgroupfs
 Plugins:
  Volume: local
  Network: bridge host ipvlan macvlan null overlay
  Log: awslogs fluentd gcplogs gelf journald json-file local logentries splunk syslog
 Swarm: inactive
 Runtimes: runc
 Default Runtime: runc
 Init Binary: docker-init
 containerd version: 894b81a4b802e4eb2a91d1ce216b8817763c29fb
 runc version: 425e105d5a03fabd737a126ad93d62a9eeede87f
 init version: fec3683
 Security Options:
  seccomp
   Profile: default
 Kernel Version: 3.10.0-862.11.6.el7.x86_64
 Operating System: CentOS Linux 7 (Core)
 OSType: linux
 Architecture: x86_64
 CPUs: 2
 Total Memory: 7.639GiB
 Name: craig-dev-1
 ID: 4YKQ:SVQZ:MPSW:UXHC:JLFM:2LSO:KGNR:HNL6:DBI7:4OLZ:DGFA:23XR
 Docker Root Dir: /var/lib/docker
 Debug Mode: false
 Username: pwxbuild
 Registry: https://index.docker.io/v1/
 Labels:
 Experimental: false
 Insecure Registries:
  <IP_ADDRESS>/8
 Registry Mirrors:
  https://docker-passthrough.portworx.co/
 Live Restore Enabled: false

WARNING: bridge-nf-call-iptables is disabled
WARNING: bridge-nf-call-ip6tables is disabled

If I do:
docker rmi library/mariadb:10.1.11 && docker pull library/mariadb:10.1.11

I get:
10.1.11: Pulling from library/mariadb
[DEPRECATION NOTICE] registry v2 schema1 support will be removed in an upcoming release.   That registry is running:
registry --version
registry github.com/docker/distribution v2.7.1

with a config:
version: 0.1
log:
  level: info
proxy:
  remoteurl: https://registry-1.docker.io
storage:
  filesystem:
    rootdirectory: /var/lib/registry
    maxthreads: 100
http:
  addr: <IP_ADDRESS>:5000
  net: tcp
delete:
  enabled: true/

This deprecation warning was introduced in https://github.com/dweomer/docker-ce/commit/34ff5a01d37671c7a5ddb58638ab629a5a5e71ad by @tiborvass , but it is not clear to me what I should fix.

on my client, if I do:
docker rmi library/mariadb:10.1.11 && docker pull library/mariadb:10.1.11
if I look in the log of my passthrough I see:
time="2019-08-08T17:46:18.220246276Z" level=error msg="response completed with error" err.code=unknown err.detail="manifest schema v1 unsupported" err.message="unknown error" go.version=go1.11.2 http.request.host=docker-passthrough.portworx.co http.request.id=5e67adc5-9d82-49be-8321-16b33da66c5e http.request.method=GET http.request.remoteaddr=<IP_ADDRESS> http.request.uri="/v2/library/mariadb/manifests/10.1.11" http.request.useragent="docker/19.03.1 go/go1.12.5 git-commit/74b1e89 kernel/3.10.0-862.11.6.el7.x86_64 os/linux arch/amd64 UpstreamClient(Docker-Client/19.03.1 \(linux\))" http.response.contenttype="application/json; charset=utf-8" http.response.duration=666.386244ms http.response.status=500 http.response.written=70 vars.name="library/mariadb" vars.reference=10.1.11

Nothing to fix on your end, Docker mistakenly added the message on pull. As for the passthrough, it is possible the passthrough isn't supporting schema1 images. The pull clients will support it just fine. If you want to file an issue related to pull through caching of schema1 not working, that would be separate. I am going to close this issue, I filed something on moby/moby to remove the incorrect deprecation message. Keeping your images up to date is always a good idea, any image still pulled as schema 1 from the hub was built a few years ago at this point and does not have security updates.

https://github.com/moby/moby/issues/39701

A simple one line Dockerfile will break Kaniko. See gist


fontDisplay: "swap" Typescript error

Bug report
Describe the bug
Typescript complains about invalid styles object when a @font-face uses fontDisplay: "swap"
The swap of the font works correctly in the browser, but the error is present in the IDE and Jest tests.
To Reproduce

See example at https://codesandbox.io/s/stitches-fontdsiplay-error-7bh0l4?file=/src/App.tsx
Comment/uncomment fontDisplay: "swap" on line 12 to see the error on line 23.

Expected behavior
No Typescript error.
System information

OS: macOS
Version of Stitches: 1.2.7 (localhost) and 1.2.8 (codesandbox)
Version of Node.js: 16.14.0 (localhost) and 14.18.1 (codesandbox)

That was quick!
Thanks a lot :)

@hdennison I believe #1011 was reverted, and the error still persists. Can you please re-open this issue?


feat: New title style

New feature: designed a new title style can help visitor locate the position of the model switching function:
Rendering:

Would it be better to use icons instead of text, such as drop-down triangles?🤔

Add a svg arrow icon to remind users this is a drop-down menu

Add a svg arrow icon to remind users this is a drop-down menu

LGTM


Support containerization of client-side webassembly applications

Currently users that containerize a standard blazorwasm .NET SDK template end up with something unrunnable - because this style of application is compiled to WASM and then intended to be run from a browser, our defaults are all wrong.
We should detect (or allow the user to tell us) that they want to serve static content and then specify or infer

an image to act as an http server
the root to serve data from

I was able to make this work today with the busybox image, which includes an http server in it, using the following configuration:
<PropertyGroup>
  <ContainerBaseImage>localhost:5010/library/busybox:1.35</ContainerBaseImage>
</PropertyGroup>

<ItemGroup>
    <ContainerEntrypoint Include="busybox;httpd;-f;-v;-p;80;-h;./wwwroot" />
</ItemGroup>

The main annoyance is that you can't change the working directory directly to wwwroot because we copy the publish contents to the working directory by default, so the relative host directory of the app had to be changed via CLI argument.

because we copy the publish contents to the working directory by default

I think this behavior stems from ASP.NET which looks for some files in the working directory.
You could add an additional property that determines where the published output goes, that can be set different from the ContainerWorkingDirectory.
Probably it makes sense to use that property as the default value of ContainerWorkingDirectory to match the ASP.NET behavior.

In addition to blazor wasm, we should support wasi-wasm targets. This would be a very simple container, coping the built .wasm module and invoking it with wasmtime's container as the base image.

Mismatched blazorwasm RIDs is currently the second-most cause of error using the SDK containers feature:

So if we wanted to reduce that error entirely we should implement this feature.

Hi @baronfel, is the workaround with the custom base image still working? I can't seem to get through the container build using:
dotnet publish -p:PublishProfile="DefaultContainer"

C:\Program Files\dotnet\sdk\8.0.401\Containers\build\Microsoft.NET.Build.Containers.targets(242,5): error MSB4018: The "CreateNewImage" task failed unexpectedly. [D:\Git\selfhosted\root\WasmTest\WasmTest\WasmTest.csproj]
C:\Program Files\dotnet\sdk\8.0.401\Containers\build\Microsoft.NET.Build.Containers.targets(242,5): error MSB4018: System.NotImplementedException: CONTAINER2003: The manifest for library/busybox:1.35 from registry https://registry-1.docker.io/ was an unknown type: application/vnd.oci.image.index.v1+json. Please raise an issue at https://github.com/dotnet/sdk-container-builds/issues with this message. [D:\Git\selfhosted\root\WasmTest\WasmTest\WasmTest.csproj]
C:\Program Files\dotnet\sdk\8.0.401\Containers\build\Microsoft.NET.Build.Containers.targets(242,5): error MSB4018:    at Microsoft.NET.Build.Containers.Registry.GetImageManifestAsync(String repositoryName, String reference, String runtimeIdentifier, IManifestPicker manifestPicker, CancellationToken cancellationToken) in /_/src/Containers/Microsoft.NET.Build.Containers/Registry/Registry.cs:line 194 [D:\Git\selfhosted\root\WasmTest\WasmTest\WasmTest.csproj]
C:\Program Files\dotnet\sdk\8.0.401\Containers\build\Microsoft.NET.Build.Containers.targets(242,5): error MSB4018:    at Microsoft.NET.Build.Containers.Tasks.CreateNewImage.ExecuteAsync(CancellationToken cancellationToken) in /_/src/Containers/Microsoft.NET.Build.Containers/Tasks/CreateNewImage.cs:line 83 [D:\Git\selfhosted\root\WasmTest\WasmTest\WasmTest.csproj]
C:\Program Files\dotnet\sdk\8.0.401\Containers\build\Microsoft.NET.Build.Containers.targets(242,5): error MSB4018:    at Microsoft.NET.Build.Containers.Tasks.CreateNewImage.Execute() in /_/src/Containers/Microsoft.NET.Build.Containers/Tasks/CreateNewImage.cs:line 36 [D:\Git\selfhosted\root\WasmTest\WasmTest\WasmTest.csproj]
C:\Program Files\dotnet\sdk\8.0.401\Containers\build\Microsoft.NET.Build.Containers.targets(242,5): error MSB4018:    at Microsoft.Build.BackEnd.TaskExecutionHost.Execute() [D:\Git\selfhosted\root\WasmTest\WasmTest\WasmTest.csproj]
C:\Program Files\dotnet\sdk\8.0.401\Containers\build\Microsoft.NET.Build.Containers.targets(242,5): error MSB4018:    at Microsoft.Build.BackEnd.TaskBuilder.ExecuteInstantiatedTask(TaskExecutionHost taskExecutionHost, TaskLoggingContext taskLoggingContext, TaskHost taskHost, ItemBucket bucket, TaskExecutionMode howToExecuteTask) [D:\Git\selfhosted\root\WasmTest\WasmTest\WasmTest.csproj]

with
<PropertyGroup>
    <ContainerBaseImage>docker.io/library/busybox:1.35</ContainerBaseImage>
</PropertyGroup>

<ItemGroup>
    <ContainerEntrypoint Include="busybox;httpd;-f;-v;-p;80;-h;./wwwroot" />
</ItemGroup>

This is slightly different - it looks like we don't understand how to read image indexes, regardless of registry. I think this may be coming with https://github.com/dotnet/sdk/pull/43085, but will need to check.

https://github.com/dotnet/sdk/pull/43631 will fix at least part of this. However, we still need to teach the tooling what to do with browser-wasm as a RID. There's no golang architecture that directly maps here. In the meantime, when this fix releases, you'll need to specify <ContainerRuntimeIdentifier>linux-x64</ContainerRuntimeIdentifier> (or linux-arm64, or whichever is relevant for your situation) until we have full browser-wasm support.

Related: https://github.com/dotnet/dotnet-docker/issues/5070


Sass map formatting

Hello!,
At the moment the plugin's formatting of Sass maps isn't ideal. It would be nice if the default formatting could be changed, or perhaps introduce it as config option.
Currently it's...
$sass-map: ("key1":"value1", "key2":"value2", "key3":"value3");

...but it should be...
$sass-map: (
    "key1" : "value1",
    "key2" : "value3",
    "key2" : "value3"
);

Cheers,
Dave

I've also just run into this problem.

Adding my hat to the ring on this one. Would love an option.

+1

+1

+1
This is seriously needed

I frequently bump into the problem of all my maps arranging themselves onto a single line.

+1 NEED !


test: add expect for performance claim

Interestingly enough the comment claims that the test should run within 50 to 100ms. However, adding an expect shows that this test fails.

We cant test this in CI, as CI is very unstable when it comes to performance. Also performance differs when you run a single test vs running everything.

Hmm... That is right yeah, but it is hard to have this as a test then, since you won't notice it if the performance is off by a lot, right?
By the way, on my machine with an intel i7 (9th gen I believe) I got around 160ms.

I run the tests locally very often, the time gets logged, so I can see the changes. Also if its significant, it will be visible on the total test run time too. It would be nice to automate this, but I am not interested in adding flaky tests (already got some, lol). Maybe if the perf tests would be executed outside of jest, it could be more predictable.
And well, Intel being slow, that's not my problem :P Didn't expected such a difference, do you run just the single test?

Hehe ofc that is not your problem. Neither did I expect such differences. Ah well, maybe someone could create a benchmarking suite and run that on every pr


revendor github.com/docker/swarmkit to ef3c57a

To get the changes:

docker/swarmkit#2234 [ca] Validate the Root CA certificate before updating the security config
docker/swarmkit#2237 allocator: Avoid assigning duplicate IPs during initialization
docker/swarmkit#2238 [ca/node] Maybe update the root CA when renewing the TLS cert

The changes between the two commit hashes on docker/swarmkit repo:
https://github.com/docker/swarmkit/compare/758b59114f7eef55faca7007af773d4097540ed2...6083c7689ed00a6f2d67941443603df69c2ff6ba
Signed-off-by: Andrew Hsu<EMAIL_ADDRESS>

@andrewhsu @aaronlehmann shall we also get https://github.com/docker/swarmkit/pull/2244 in ?
This will fix a P0 and it will be great to have all known P0s in RC3. Which can help avoid RC4 if there are no more P0s found.

This looks like a duplicate of #67.

Although, to be fair, this one got the full set of changes before #67 was updated to have them all.

LGTM
integration and unit tests are green


Application.Run() Alternative

I noticed in the stdCallback code there are comments indicating that an alternative to Application.Run() might exist. Is this possible? Not all VBA host applications provide an Application.Run() method, and it can be very limiting to what you can do and which libraries/code you can use.

Added, there are some nuances to using pointers though.
https://github.com/sancarn/stdVBA/blob/master/tests/stdCallbackTests.bas#L50-L86 gives a good idea of what you need to be careful of. You can either supply the argument types or they can be predicted but they can't be wrong. Same with the return type. Similarly if variables are declared byref (or not declared at all) then you need to pass pointers in instead of the variable itself...
Ultimately, calling by pointer becomes very tricky. Hope it makes sense, let me know if it works for you in your other vba environments.


[1.x] Emit SourceInfos when incremental compilation fails

This PR implements https://github.com/sbt/zinc/issues/932 via building SourceInfos from analysis callback and pass them to handleErrors in CompilerBridge.scala.
c.c. @adpi2

@adpi2 Reopened this PR and added a unit test.
@eed3si9n c.c. https://github.com/sbt/zinc/issues/932#issuecomment-2376210214 for reason of revisiting this PR. I am thinking if this can be merged alongside #1309 as both involves adding AnalysisCallback3.

LGTM. could you squash the commits and add problem and solution descriptions in the commit message plz?

@eed3si9n Squashed!


Documentation on how to submit a Teams App to the Marketplace though Developer Portal

There is no detailed documentation on how to submit a Teams App to the Marketplace though Developer Portal
[Enter feedback here]

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

ID: 9f071593-9440-5115-242d-488cf25e9256
Version Independent ID: 746b1c6e-1bcc-21fe-e2be-149e7c10f1d5
Content: Overview - Teams app store publishing process - Teams
Content Source: msteams-platform/concepts/deploy-and-publish/appsource/publish.md
Product: msteams
GitHub Login: @heath-hamilton
Microsoft Alias: surbhigupta

@maritzace - We are looking into this I will get back to you soon.

@maritzace -Please have a look at this doc- https://docs.microsoft.com/en-us/microsoftteams/platform/concepts/build-and-test/teams-developer-portal#distribute-your-app
Let us know if you have any query.

Thanks - however, it doesn´t explain how to do the process in Developer Portal, it redirects the user on how to do it in Partner Center
Best,
Maritza
From: sayali-MSFT @.>
Sent: Wednesday, January 5, 2022 6:53 AM
To: MicrosoftDocs/msteams-docs @.>
Cc: Maritza Cartaya Escalona @.>; Mention @.>
Subject: Re: [MicrosoftDocs/msteams-docs] Documentation on how to submit a Teams App to the Marketplace though Developer Portal (Issue #4957)
@maritzacehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmaritzace&data=04|01|maritzac%40microsoft.com|57df15431b964c3425f808d9d04a483b|72f988bf86f141af91ab2d7cd011db47|0|0|637769839782946274|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=IrhXybyfiyflNUbZByyiKqWnU3kxucf68tYsudtlxfw%3D&reserved=0 -Please have a look at this doc- https://docs.microsoft.com/en-us/microsoftteams/platform/concepts/build-and-test/teams-developer-portal#distribute-your-apphttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdocs.microsoft.com%2Fen-us%2Fmicrosoftteams%2Fplatform%2Fconcepts%2Fbuild-and-test%2Fteams-developer-portal%23distribute-your-app&data=04|01|maritzac%40microsoft.com|57df15431b964c3425f808d9d04a483b|72f988bf86f141af91ab2d7cd011db47|0|0|637769839782946274|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=SH%2BeNM1r5RbLLoVPfOKKjmacPCLRxH1SB3%2FqkrVClHI%3D&reserved=0
Let us know if you have any query.

Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fmsteams-docs%2Fissues%2F4957%23issuecomment-1005660513&data=04|01|maritzac%40microsoft.com|57df15431b964c3425f808d9d04a483b|72f988bf86f141af91ab2d7cd011db47|0|0|637769839782946274|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=vPdUt1nkSqOwdNffAPEpQJshZg4cN%2F6nIRLby68%2FtGQ%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FATL34DYIBDFMIDBWR5QVXODUUQ5KDANCNFSM5LILLHKQ&data=04|01|maritzac%40microsoft.com|57df15431b964c3425f808d9d04a483b|72f988bf86f141af91ab2d7cd011db47|0|0|637769839782946274|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=Esr4XkHkwj9RvdQucuI66II%2F1l9mXTv0DbLFFY%2F39I4%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01|maritzac%40microsoft.com|57df15431b964c3425f808d9d04a483b|72f988bf86f141af91ab2d7cd011db47|0|0|637769839782946274|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=4T%2BOxrtysWJ9O5MEhUFFBns1yQXan3Ok7c6dEd2FpFs%3D&reserved=0 or Androidhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04|01|maritzac%40microsoft.com|57df15431b964c3425f808d9d04a483b|72f988bf86f141af91ab2d7cd011db47|0|0|637769839782946274|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=1RZ2VLeQcr5EFrbjCtnIjlPt38yBIdNlJFzfl95CmqQ%3D&reserved=0.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

@maritzace - Currently developer portal not supported to submit a Teams App to the Marketplace. So it redirects the user on how to do it in Partner Center.
Ref doc:-https://docs.microsoft.com/en-us/microsoftteams/platform/concepts/deploy-and-publish/appsource/publish#understand-the-publishing-process
Let us know if you have any query.

Thanks.
The button "Publish" in the right upper corner is the one causing the confusion among partners then.


Fix start_with method call

start_with rather than starts_with

Thanks. I'm going to release 1.2.1 today so it will be included in that.

Introduced in 53c0e4d9ed2abc6fffc5bbe6d13189a1b928dc62 cc @trishume

So this is an ActiveSupport alias:
https://github.com/rails/rails/blob/08754f12e65a9ec79633a605e986d0f1ffa4b251/activesupport/lib/active_support/core_ext/string/starts_ends_with.rb#L2
Was this a bug for you? We have ActiveSupport as an explicit dependency.

@kmcphillips Yes, I was getting no method errors last night after upgrading from a very old version of active_shipping
It was failing on a very simple test file
require 'json'
require 'active_shipping'
require 'yaml'
include ActiveShipping

loginConfig = YAML::load(File.open('<config file>'))
usps_api = USPS.new(:login => loginConfig['usps']['login_name'])
response = usps_api.find_tracking_info('9574211132094339557534')

Oh this is very interesting.
So we depend on active support but we don't actually require it anywhere. Since you're working outside of a Gemfile in the above example it's not being loaded automatically.
We'll change it so that we explicitly require "active_support". @trishume has offered to PR that.

https://github.com/Shopify/active_shipping/blob/master/lib/active_shipping.rb#L24 :neutral_face:
Now I'm even more confused. I have no idea why you were getting those errors.

Ok at least I can reproduce it locally. Still no idea why this is happening though.

require 'active_support' doesn't actually load much. If you want to load all of AS, do require 'active_support/all'

If you include ActiveShipping in a Rails app, you won't likely run into this because Rails will load the necessary part of ActiveSupport.

Thanks @wvanbergen that explains a lot. Now I know why it didn't also crash from the calls to try. I implemented this fix in #274.


Dreamcast games freeze when using save state to save game

On newest release of ark os Dreamcast games freeze when saving game on save state, I’m using a rg353v I tried shenmue and phantasy star online bith gave same issue

I'm able to save state and load state just fine using retroarch32 and flycast32_rumble (default emulator and core) with Power Stone, Power Stone 2, Crazy Taxi, Dynamite Cop, and ChuChu Rocket.  What emulator and core are you using?

Any update?

Closing as there's been no updates from the OP on this.


Only the first line is done

http://<IP_ADDRESS>:8085/deliver/360000_900/1000/420/LINE2:tempaC00aa00_LINE2:tempaiC0000ff_LINE1:tempiCaaaaff_LINE1:tempaxCff0000_LINE2:tempxCffaa00

http://<IP_ADDRESS>:8085/deliver/360000_900/1000/420/LINE2:tempaiC0000ff_LINE2:tempaC00aa00_LINE1:tempiCaaaaff_LINE1:tempaxCff0000_LINE2:tempxCffaa00

Fixed by 4fa7d3cc486d007c34e1a0fcf3aea0f942ddefae. The fix has introduced #2.


vulkan support

hi @Gekkio
Do you have a plan to add vulkan support?

any comments?

gfx already supports Vulkan, so it shouldn't be too difficult to use it with imgui-rs. However, right now imgui-gfx-renderer doesn't include Vulkan-compatible shaders, so some manual work is needed. Once the big changes in gfx are finally released, I plan to make imgui-gfx-renderer easier to use with alternative backends (Vulkan, DX, possibly Metal...).
I've also considered writing a vulkano-based renderer, but I don't really know Vulkan at all so it's not going to happen very soon.

Hello @Gekkio
Do you plan to add Vulkano  backends soon ?

Sorry, no plans right now because I still haven't found time to learn Vulkan, and no pull requests have appeared. Once the new LL redesign of gfx gets a proper crates.io release, the gfx renderer will be updated to use it and you should be able to at least target Vulkan as a backend with it.

I wrote a backend for gfx-hal a while ago, if anybody is interested in taking a look. I am not super experienced with vulkan though, so I'm pretty sure it's not optimal.

hello,
i've updated bejamins code to work with current dependencies. you can add textures too. but it's just a hack for now.
https://github.com/kollapsderwellenfunktion/imgui-backend-gfx-hal

https://www.reddit.com/r/rust/comments/crb7q7/success_i_have_a_working_setup_of_imgui_on_vulkan/
I think this can be closed?

Closing, can be handled by a thirdparty crate like https://github.com/adrien-ben/imgui-rs-vulkan-renderer


separate parameter settings for acceleration and deceleration

Entering a new value for a acceleration as deceleration after the acceleration is done works as described in Issue 118. Setting a separate value for acceleration and deceleration would be nicer though.

Agree, that this would be nicer for some applications. Unfortunately based on current structure of the ramp generator, this is a quite complex endeavor. So there is no plan to start implementation in the coming weeks and more likely even months.
PRs are surely welcome.


mamba run throws away exit code

This is similar to the corresponding bug in conda (https://github.com/conda/conda/issues/8385).
The conda cli no longer returns an int but instead a Response class with stdout, stderr and rc fields.
The exit code is the rc field.
>>> from conda.cli.main import generate_parser
>>> from conda.cli.main_run import execute as run
>>> parser = generate_parser()
>>> result = run(parser.parse_args(['run','false'], parser)
ERROR conda.cli.main_run:execute(32): Subprocess for 'conda run ['false']' command failed.  (See above for error)
>>> result
Response(stdout='', stderr='', rc=1)
>>> result.fc
1

mamba doesn't check for this return type so it ends up throwing away the return type:
https://github.com/TheSnakePit/mamba/blob/66fb193293f13bd8028ed02e86f1439ff66a45b8/mamba/mamba.py#L769-L771
that code should be extended to check for Response object and return result code.

do you want to open a PR? Would be wonderful

Sure, when I have the time. The fix itself should be easy.

fixed with #395


Migrate Controller Specs to Request Specs

We began the process of converting our Controller specs to Request specs. The current attitudes are to move away from Controller specs in favor of Request specs. We have some Request specs started for our API. If any of these have been completed, please delete the Controller spec as part of your pull request.
See here for some more discussion: https://github.com/rspec/rspec-rails/issues/1838
This is also an opportunity to review our specs and ensure:

we have good coverage for our controller actions
we are not testing things unnecessarily

Controller Specs Needing Conversion:

[x] barcode_items_controller_spec (admin)

[ ] users_controller_spec (admin)

[ ] admins_controller_spec

[ ] donation_sites_controller_spec

[ ] donations_controller_spec

[ ] items_controller_spec

[ ] organizations_controller_spec

[ ] transfers_controller_spec

Reference

https://relishapp.com/rspec/rspec-rails/docs/request-specs/request-spec
https://everydayrails.com/2016/08/29/replace-rspec-controller-tests.html

Helping with users_controller_spec (admin)

It looks like transfers_controller_spec it's already been migrated.


Partition-based regression verification

http://dl.acm.org/citation.cfm?id=2486829

public code from apache CLI used for analysis


chore(main): release 0.1.2

:robot: I have created a release beep boop
0.1.2 (2024-09-11)
Refactoring

make API entities strings (#370) (11cf62f)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v0.1.2
:sunflower:


Fixed #warning preprocessor directive

Added condition to use of #warning preprocessor directive

@alexbaden Is there a header file in HDK that is included in all other places in the code where we could define such macro?

No, usually we would create a new header file in Shared and define the macro there. It can then be included where needed. See, for example, Shared/checked_alloc.h (not a macro but a free function that is used in a lot of different places).

@alexbaden There is BOOST_PRAGMA_MESSAG. What do you think about using it instead?


'Other' greenhousegas emissions

Currently the 'Emissions' side bar item is somewhat confusing as users think they can change actual energy emissions here.
Furthermore, other emission types are scattered throughout the ETM (such as emissions on electricity import or heat).
Proposal:
Rename 'Greenhouse gases' into 'Other greenhouse gases' & place all other emission sliders under this sidebar item (e.g. by creating a new sidebar item 'Emission factors').
What do you think @redekok @mabijkerk ?

I'm in favour of clarifying this, as it is currently confusing for our users. However, I think discussion is needed what the best solution is.
As for the Emission factors, this may be something that can be picked up within the Transit flows project, as it will restructure import flows.

Sounds good, shall we have a (live) discussion about this during that project @mabijkerk ?

I agree with both of you! This definitely needs clarification. Discussing this during the transit flows project sounds like a great idea.


ci: added github actions workflow

Since building docker image takes a very long time (example: https://github.com/audrium/reana-env-root6/runs/1351920523) it was decided not to build the docker image, but use a Dockerfile linter for validation.
Here is an example of GH actions workflow run: https://github.com/audrium/reana-env-root6/runs/1352926699

As discussed elsewhere, we can:

Introduce a Dockerfile verifier/checher/linter for MVP. This could actually be useful for other repositories as well.

Think about moving from Ubuntu to CentOS where ROOT packages are provided natively. This will (i) reduce docker image build times and (ii) move to HEP-standard OS to run ROOT analyses on. However, we would have to test all reana-demo-* examples that use ROOT, to make sure everything runs well. We can do this at a later stage.

Found a nice Dockerfile linter called hadolint. Seems like a most widely used tool for this case. There are even dedicated wiki pages for each linting error with examples: example here


Double check science cost for Acetylene

Mod source
PyAE Beta
Which mod are you having an issue with?

[ ] pyalienlife
[ ] pyalternativeenergy
[ ] pycoalprocessing
[X] pyfusionenergy
[ ] pyhightech
[ ] pyindustry
[ ] pypetroleumhandling
[ ] pypostprocessing
[ ] pyrawores

Operating system

=Windows 10

What kind of issue is this?

[ ] Compatibility
[ ] Locale (names, descriptions, unknown keys)
[ ] Graphical
[ ] Crash
[ ] Progression
[X] Balance
[ ] Pypostprocessing failure
[ ] Other

What is the problem?
Tech is 100x(3x automation science) = 300 packs whereas downstream is 100 packs.
Note: it was interesting to debug why my ratios didn't work but this is probably not the best place to put a troll tech
Steps to reproduce

Look at tech
Look at downstream and upstream techs xD

Additional context
No response
Log file
No response

Fixed in latest


Fix some inheritance issues with renames and expr refs

Preserve computed/inherited types for computed pointers
Avoid mangling inherited attributes with the expression fixer

Fixes #3620

Cherry-picked for 1.3


Make downcasting safer

Per discussion with @Sh3Rm4n at https://github.com/rust-embedded/embedded-hal/commit/d5f9747a060ca4109d215f512aa80a89ccb90df0#r133323441

the cast should never overflow as long as the user gives a valid fraction (witih num <= denom). IMO we shouldn't take the performance hit.

@Dirbaio is there any mechanism to catch user errors like assert! or debug_assert! that rust embedded tends to use?  As the saying goes, if there is a way to make a mistake, someone will make it, so it might be good to notify user of an error in some way?
Comparing assembly, on x64 it does generate two additional assembly instructions:  https://rust.godbolt.org/z/oPs59nEnY - but I wouldn't know how significant it is.  If you think this is fine, should i add an #[allow(...)] to it?  BTW, it might make sense to add a bunch of [lints.clippy] to Cargo.toml (even though it will produce a warning on older Rust compilers, it shouldn't affect anything, and it will auto-work on the newer ones)

If you think this is fine, should i add an #[allow(...)] to it

I'd propose to use #[allow(...)] on it and adding a comment, why it is no problem to cast here (pointing out the API contract, that num is less than or equal to denom

Updated the code, now uses debug_assert.  There are a few things that keep bothering me (somewhat unrelated):

Would it make sense to use thiserror crate?
The code uses Error everywhere - would it make sense to rename it to HalError?  See clippy lint
Would it make sense to add a justfile in the root - so that developers can run things like just test without thinking of all the various cargo flags?  This is very similar to a Makefile, but far simpler and frequently used, especially in the Rust projects

these are indeed unrelated, can you open separate issues?

Would it make sense to use thiserror crate?

we have an error trait, the HAL prvides the actual type, so I don't think there's much we gain from it. Also std::error::Error is not available in core.


allow usage of hidapi shared libs

This makes it possible to use libhidapi-libusb.so or libhidapi-hidraw.so on Linux rather than statically linking hidapi. This will save some space and make it possible to upgrade the hidapi library independently of the app. I'm using this on a small embedded system, so the space is actually relevant for me.
I don't have any macOS or Windows machines to test it on, so it's limited to Linux for now. The feature can be enabled by building with cargo build --features=shared-hidraw or --features=shared-libusb. If no feature flags are specified, you get the old behaviour. If both are specified, libhidapi-hidraw is used (this is primarily to allow usage of --all-features).

Closed in favor of #3


parse benchmark results

moved to lance-benchmark-results repo to keep it with result parsing and visualization scripts


Create openttd-nightly.json

OpenTTD Nightly
Closes #334

As with OpenTTD, the name of bin has been modified.

6a85a79
Restored the original name.
4c61105
Notes has been modified.