Upgrading yapf breaks our build

Issue Template
Title: Upgrading yapf in tools/requirements.txt breaks the build.
Description:
Most projects are moving quickly to python/black but we can stick with yapf if we want.  However, we have yapf==0.25.0 hardcoded in tools/requirements.txt and when we upgrade to the current version (0.27.0), our build fails.

https://github.com/google/yapf/blob/master/CHANGELOG

@cclauss we have no strong emotional attachment to yapf, if you want to switch to something else without too much churn, you're welcome to switch.


Bias-Free Communications Note

Added in a note recognizing that there is wording within this document, which some may feel is exclusionary.  However, this is done only to remain in alignment with the text in the software.

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

@ajlam, you can only sign off in the main PR conversation, not in review comments.

#sign-off


bump Golang to 1.20.8

Checklist

[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)

/run-e2e internal

/run-e2e internal


Add option capitalize first letter of each word

Also put the actual password generation into a closure because its gotten more complicated and is duplicated because if the interactive mode

Hey, thanks for the PR. The capitalize option has been suggested before e.g., https://github.com/redacted/XKCD-password-generator/pull/63. There are a lot of potentially useful transforms so the solution settled on in that issue - using the importable library functionality, or a secondary wrapper script, seems preferable to me.
I do like the closure fix though - would you consider opening a new PR with just that change?

Hmm, that would make my simple CLI usage of this otherwise perfect tool more complicated. I am a fan of the no spaces style HamsterBarbecueSubstanceShucking

$ sed -re 's#\b(.)#\U\1#g' -e 's#\s+##g' <<< "foo bar quux"
FooBarQuux

Updated the PR with just the closure change

@bonki Thanks, I have had to use awk version as sed on mac os is missing -r option.
xkcdpass | awk '{for (i=1;i<=NF;i++) $i=toupper(substr($i,1,1)) substr($i,2); gsub(/ /, "", $0); print}'

Thanks for this, merged and in the next release


Breaking dependencies between message_scroll and fetch_status.

This PR contains two commits:

Lifts up the formation of the Filter object from message_list_data to all_messages_data by removing the default parameter.
Extracts message_scroll_loading from message_scroll to remove dependency on message_scroll from fetch_status which allows us to easily convert fetch_status into typescript.

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[x] Calls out remaining decisions and concerns.
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[x] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

Looking at this, I think message_feed_loading.ts would be a better name than message_scroll_loading.ts; it's not really about scrolling, just about the top/bottom of page loading indicators.
As a small comment, I'd have titled this "Break dependencies ..." to use the imperative mood -- we prefer that for PR descriptions as well as commit messages.
Given how small a new file this is and how trivial the changes to the code is with the conversation to TypeScript, I think it's fine in this case to extract the code to a new file and migrate to TypeScript in the same commit, but in general it's better to have a pure-code-move commit and then a typescript-conversation commit; git show --color-moved makes that approach way easier to review if there are nontrivial changes as part of the TypeScript migration.

Thanks for the tips I will keep them in mind!

@timabbott Renamed the module name to message_feed_loading, also I will open a follow-up PR to migrate fetch_status to TypeScript after this gets merged.

Merged, thanks @Lalit3716!


WKWebView.takeSnapshot failing to generate an image on Xcode 14 RC

Describe the bug
All WKWebView tests that require generating an image are crashing while force unwrapping an optional image on the takSnapshot completion handler.
To Reproduce
Run the iOS tests on the swift-snapshot-testing repo's main branch with Xcode 14 RC.
Expected behavior
Tests to pass
Screenshots

Environment

swift-snapshot-testing version: latest main
Xcode 14 RC
Swift 5.7
OS: 12.5.1

More information
Console output logs "Invalid connection identifier" which makes me wonder if Xcode is expecting network capabilities to launch this.
Test Case '-[SnapshotTestingTests.SnapshotTestingTests testWebView]' started.
2022-09-02 16:45:53.832110-0700 xctest[26474:1131098] [ProcessSuspension] endowmentsForHandle: Process with PID 26474 is not running
2022-09-02 16:45:53.874338-0700 xctest[26474:1131098] [Process] 0x160064100 - [PID=0] WebProcessProxy::didFinishLaunching: Invalid connection identifier (web process failed to launch)
2022-09-02 16:45:53.874399-0700 xctest[26474:1131098] [Process] 0x160064100 - [PID=0] WebProcessProxy::processDidTerminateOrFailedToLaunch: reason=Crash
2022-09-02 16:45:53.874518-0700 xctest[26474:1131098] [Process] 0x13300ce18 - [pageProxyID=6, webPageID=7, PID=0] WebPageProxy::processDidTerminate: (pid 0), reason=Crash

Still happening with Xcode 14 RC.

Still happening with Xcode 14.0.1.
A crash inside WKWebView occurs that seems to make the wkWebView.takeSnapshot(...) method to return an error instead of an image.
xctest[42043:34329227] [Process] 0x110064100 - [PID=0] WebProcessProxy::didFinishLaunching: Invalid connection identifier (web process failed to launch)
xctest[42043:34329227] [Process] 0x110064100 - [PID=0] WebProcessProxy::processDidTerminateOrFailedToLaunch: reason=Crash
xctest[42043:34329227] [Process] 0x13a82da18 - [pageProxyID=6, webPageID=7, PID=0] WebPageProxy::processDidTerminate: (pid 0), reason=Crash
xctest[42043:34329227] [Loading] 0x13a82da18 - [pageProxyID=6, webPageID=7, PID=0] WebPageProxy::dispatchProcessDidTerminate: reason=Crash
xctest[42043:34329227] [Loading] 0x13a82da18 - [pageProxyID=6, webPageID=7, PID=0] WebPageProxy::dispatchProcessDidTerminate: Not eagerly reloading the view because it is not currently visible

I don't think this is caused by SnapshotTesting. Just using the takeSnapshot method without SnapshotTesting also fails. Weird that I can't find any other blog posts or issues about this.

@barrault01 I've opened a PR with your commit because it does improve things a little bit.

I believe #692 fixed this, so closing for now.


Application crashes on attempt to create the second instance of k4abt_tracker_t

If application is trying to create two instances of k4abt_tracker_t simultaneously (that is, the second instance is created before closing the first one), application silently crashes.

Ran into the same issue.
I believe I read somewhere that the body tracker doesn't support multiple instances, would be nice if it can support that in the future (for multi sensor scenarios) and at least for now gracefully fail with a return value and log item.

We do only support 1 instance at a time. We need to update documentation and attempt to generate an error.

Yes, there is still a problem. Shutdown and release can't release memory correctly.

@venscn Thanks for spotting the bug. I have a fix for the issue that shutdown/release can't fully release memory. You should see the fix in the next release. As for the issue @bibigone points out, it is by design. We don't allow user to create two instance of k4abt_tracker_t simultaneously. We are working on a fix that we will fail gracefully and log the error.
BTW, @bibigone and @Brekel , why do you want to create two k4abt_tracker_t simultaneously? What is the use case for this? If the use case is valid and important, we might consider adding this feature.

BTW, @bibigone and @Brekel , why do you want to create two k4abt_tracker_t simultaneously?

For multi-sensor configuration, for example. Skeletal data in conjunction with point clouds can be used to determine mutual sensors position.

BTW, @bibigone and @Brekel , why do you want to create two k4abt_tracker_t simultaneously? What is the use case for this? If the use case is valid and important, we might consider adding this feature.

@yijiew
Indeed, I'm also running multiple sensors on one machine (with a beefy GPU), each running in their own thread from a single application. So my thought was to run one tracker per sensor to keep things separate.

Since Kinect v1, I've created Kinect software for artists, researchers, and educators. These groups often provide the flashy and exciting demos, videos, and "buzz" articles that generate excitement and press. I have repeatedly been asked by my customers over all those years how they can use two sensors on a single PC to track humans in their solutions.

v1 new concept to everyone and USB2 days with computers not yet able to really process/render all the data.
v2 hungry customers wanted multiple sensors. MSFT devs in the v2 alpha started to implement multiple sensors but gave up giving a key reason of the rarity of two USB3 controllers in the same computer. I have the v2 alpha code that would have used these multi-sensor APIs.
K4A? Microsoft, devs like those on this thread, and all our customers all know this problem area well. My customers are still hungry. I had a customer this week ask me how to use multiple sensors for tracking with the v2 Kinect.

I want to tell my customers "a solution is coming...you can buy two K4A sensors and use my software". But, if the Microsoft team doesn't provide multiple sensor support, then the K4A story is only an small increment better than the already solid Kinect v2 which has well tested code and a solid aftermarket.

@yijiew

I have a fix for the issue that shutdown/release can't fully release memory. You should see the fix in the next release.

Does it fix issue#541 also?

@bibigone Yes that should fix #541. For this issue we are tracking a fix that would specifically give an error message (instead of crashing) when more than one body tracker instance is created in the same process.

BTW, @bibigone and @Brekel , why do you want to create two k4abt_tracker_t simultaneously? What is the use case for this? If the use case is valid and important, we might consider adding this feature.

I have a use case that I can't elaborate in detail on an open forum as some patents are still being filed. Multiple cameras are needed because redundancy is mandatory for the application, and because the tracked person's body may be occluded by other objects. Those other objects can be larger than the tracked person, so the only way to determine the person's location is through multiple cameras. The Azure Kinect is the cheapest part of the system (by an order of magnitude), so we expect the smallest installations to use a minimum of 3 Kinect's, and each customer will have multiple installations.
The market for this system is enterprise customers, and it is the exact same enterprise customers that Microsoft is targeting for Azure IoT and similar services, so this is an excellent way to pull them into the Azure ecosystem. I am surprised that Microsoft has not presented this particular application as the leading use-case for the technology.

Does body tracking sdk support multi kinects now? @cdedmonds

As of 0.9.5, body tracking with multiple devices is listed as supported, however in my tests a third device kinected will not track bodies, and there is no error message. Is this a known issue?

@richardeakin Did you pass frames from different sensors to the same body tracker? For multiple device to work, you will need to create different trackers for different devices.


Set up CI-CD and ci-cd.yml file Charles

Done with ci-cd.yml. The Github actions work well right now


[Netpol] Sleep between successive tests

Because the same Pods are re-used to validate NetworkPolicy
implementation, there is virtually no delay between successive
tests. This means that Antrea does not always have time to properly
clean-up stale NetworkPolicies and a test can fail because of the
NetworkPolicy from the previous test. By adding a short delay (2s) every
time we update or delete a NetworkPolicy, we can bypass the issue. This
change only increases testing time by < 20% (3min -> 3min30).
This commit also changes the verbosity of some log messages to "TRACE"
to make the logs easier to navigate.
Fixes #544

/skip-all


confluent_network.private-link.azure[0].private_link_service_aliases is a map of string, known only after apply

Below is the my terraform resource and trying to create private endpoint based on the private-link aliaces, but i am not able to do that since throwing error below error.
╷
│ Error: Invalid for_each argument
│
│   on privatelink.tf line 172, in resource "azurerm_private_endpoint" "endpoint":
│  172:   for_each = confluent_network.private-link.azure[0].private_link_service_aliases
│     ├────────────────
│     │ confluent_network.private-link.azure[0].private_link_service_aliases is a map of string, known only after apply
│
│ The "for_each" value depends on resource attributes that cannot be determined until apply, so Terraform cannot predict how many instances will be created. To work around this, use the -target argument to first apply only the resources
│ that the for_each depends on.
<
resource "azurerm_private_endpoint" "endpoint" {
for_each = confluent_network.private-link.azure[0].private_link_service_aliases
name                = "confluent-${local.network_id}-${each.key}"
location            = var.region
resource_group_name = data.azurerm_resource_group.rg.name
subnet_id = data.azurerm_subnet.subnet[each.key].id
private_service_connection {
name                              = "confluent-${local.network_id}-${each.key}"
is_manual_connection              = true
private_connection_resource_alias = each.value
request_message                   = "PL"
}
depends_on = [ confluent_network.private-link ]
}
``>>

Thanks for opening an issue @Maheshsuthari!
See step #2 from README:
Apply Terraform configuration in 2 steps:

# Creates an environment and a network
terraform apply -target=confluent_network.private-link

# Creates others resources (except already created environment and network) declared in main.tf
terraform apply

Going forward, we'll update the docs to make it more clear:


0.18.1 runs (seemingly) indefinitely on some files

I understand this is not an actionable bug report, but I am posting it so that the maintainers are aware this may be an issue.
I've got a file that took about 8 seconds to obfuscate in 0.16.0 which now took at least 10 minutes before I canceled it. The file I was attempting to obfuscate was an entry point for the main process of an<EMAIL_ADDRESS>application. My other files obfuscate in a comparable time as in 0.16.0, though my sample size is quite small.

Interesting, can you upload this file?

Hi, i'll look into this issue but currently i take a 1-2 months timeout with this project to doing renovations on my flat. So, until october i'll have no time to look into this.
If i'll have time - i'll fix it.

@Slapbox if you still have this problem, please share a code with me, I'll try to solve it.

@zamotkin I downgraded long ago, but I see some fixes have been made since then so I'll share problem code when I upgrade, if it still occurs. It could be a while before I have a chance to give 0.18.x a shot.

Any news?


Add test for discord and refactor

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area build

/area config

/area outputs

/area tests

/area helm

What this PR does / why we need it:
I want to refactor and add tests for it.
With the test, I was able to refactor safely.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

@KeisukeYamashita: The label(s) area/build cannot be applied, because the repository doesn't have them

In response to this:

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area build

/area config

/area outputs

/area tests

/area helm

What this PR does / why we need it:
I want to refactor and add tests for it.
With the test, I was able to refactor safely.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign leodido after the PR has been reviewed.
You can assign the PR to them by writing /assign @leodido in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 194cbf13a8375e9ec54453353a61289514a7e997

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Issif
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Issif]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


The profit balance changes to twice the value I click on BTC and then DOC on Lend window

Hi,
The profit balance changes to twice the value when I click on BTC and then DOC on Lend window

Obs. Today I had the same problem of profit balance changes to twice the value. But I don't know how to reproduce. It's not reproducible as before.
For now happened automatically, without any action.

Hm, ok, reopening.

I will close this issue. If I reproduce again, I reopen.

It's happening. So reopening.

Still not fixed.


Detail implementation of AES Data Masking is not documented.

After some research I find the AES masking algorithm (MySQL):
UPDATE _table_ SET _cipherColumn_ = to_base64(aes_encrypt(_plainColumn_, unhex(left(sha('_key_'), 32))))
It would be great if it is brought in document.

Hi @skyoxZ
Here is the implementation of AESShardingEncryptor.
Other than that, please read this article to learn the solution of Data masking of ShardingShpere.

This issue is closed, for there is no reply from author more than 7 days.


Air-gap Install Doesn't Work

When running the air-gap installation from the 0.1.5 release the following error is encountered.
ERROR: Could not find a version that satisfies the requirement markupsafe==2.1.1 (from versions: none) ERROR: No matching distribution found for markupsafe==2.1.1
To Replicate:

Install new Linux VM (same error occurs  with both Fedora and Ubuntu)
Verify python3 version
Install pip using the get-pip.py method
Unzip the "latest-ubuntu" AaC release file
Follow instructions for air-gap install in the README

Maybe not appropriate to address in a bug, but would using pyinstaller to create a self-contained aac executable be a simpler solution for air-gapped installation?

Maybe not appropriate to address in a bug, but would using pyinstaller to create a self-contained aac executable be a simpler solution for air-gapped installation?

It sounds like PyInstaller wouldn't be compatible with our plugin framework since pluggy leverages pip to automatically register/install AaC plugins.
I'm able to replicate this issue in a container-based installation. Pydantic and Markupsafe are both dependencies with compiled c code, as indicated by their names MarkupSafe-2.1.1-cp39-cp39-manylinux_2_17_x86_64.manylinux2014_x86_64.whl and pydantic-1.10.2-cp39-cp39-manylinux_2_17_x86_64.manylinux2014_x86_64.whl. It seems like these 'manylinux' dependencies are either out of date (2014?!) or not as generally compatible as their names seem to indicate. When I try to individually install pydantic, I get this error:
$pip install aac_secure_install_0.1.5/pydantic-1.10.2-cp39-cp39-manylinux_2_17_x86_64.manylinux2014_x86_64.whl
ERROR: pydantic-1.10.2-cp39-cp39-manylinux_2_17_x86_64.manylinux2014_x86_64.whl is not a supported wheel on this platform.

This is similar to some of the early hurdles I had with integrating python packages with pre-compiled c code when putting this together. I'll need to do some more research on how we can get around this.
In the meantime, are you able to install markupsafe manually via pip install aac_secure_install_0.1.5/MarkupSafe-2.1.1*.whl or do you also get the message MarkupSafe-2.1.1-cp39-cp39-manylinux_2_17_x86_64.manylinux2014_x86_64.whl is not a supported wheel on this platform.?

On the Ubuntu desktop VM the pip version is 22.3.1 (Python 3.10).  Installing the Markupsafe week from the aac release does result in the error message you have above.

On the Ubuntu VM, I was curious if the online install would work.  Running pip install aac works with no problem.
I then did a pip uninstall aac and attempted the air-gap install again.  It failed again, this time complaining about pydantic.  Seems like more evidence that the issue is with the pre-compiled c dependencies.

I attempted the same approach described for ubuntu on an brand new Fedora 37 Workstation VM.  In this instance I was unable to download the wheels and received the following error output:
ERROR: Cannot install aac==0.0.1, aac==0.0.3, aac==0.0.4, aac==0.0.5, aac==0.1.1, aac==0.1.2, aac==0.1.3, aac==0.1.4 and aac==0.1.5 because these package versions have conflicting dependencies.

The conflict is caused by:
    aac 0.1.5 depends on MarkupSafe>=2.1
    aac 0.1.4 depends on iteration-utilities>=0.11
    aac 0.1.3 depends on iteration-utilities>=0.11
    aac 0.1.2 depends on iteration-utilities>=0.11
    aac 0.1.1 depends on iteration-utilities>=0.11
    aac 0.0.5 depends on iteration-utilities>=0.11
    aac 0.0.4 depends on iteration-utilities>=0.11
    aac 0.0.3 depends on coverage==6.0
    aac 0.0.1 depends on coverage==6.0

To fix this you could try to:
1. loosen the range of package versions you've specified
2. remove package versions to allow pip attempt to solve the dependency conflict

ERROR: ResolutionImpossible: for help visit https://pip.pypa.io/en/latest/topics/dependency-resolution/#dealing-with-dependency-conflicts

After getting this error I did a standard pip install aac and it completed without any issues.

I attempted the same approach described for ubuntu on an brand new Fedora 37 Workstation VM. In this instance I was unable to download the wheels and received the following error output:
ERROR: Cannot install aac==0.0.1, aac==0.0.3, aac==0.0.4, aac==0.0.5, aac==0.1.1, aac==0.1.2, aac==0.1.3, aac==0.1.4 and aac==0.1.5 because these package versions have conflicting dependencies.

The conflict is caused by:
    aac 0.1.5 depends on MarkupSafe>=2.1
    aac 0.1.4 depends on iteration-utilities>=0.11
    aac 0.1.3 depends on iteration-utilities>=0.11
    aac 0.1.2 depends on iteration-utilities>=0.11
    aac 0.1.1 depends on iteration-utilities>=0.11
    aac 0.0.5 depends on iteration-utilities>=0.11
    aac 0.0.4 depends on iteration-utilities>=0.11
    aac 0.0.3 depends on coverage==6.0
    aac 0.0.1 depends on coverage==6.0

To fix this you could try to:
1. loosen the range of package versions you've specified
2. remove package versions to allow pip attempt to solve the dependency conflict

ERROR: ResolutionImpossible: for help visit https://pip.pypa.io/en/latest/topics/dependency-resolution/#dealing-with-dependency-conflicts

After getting this error I did a standard pip install aac and it completed without any issues.

I assume that output is what you got from running the secure install script? I'm a little confused since I wouldn't expect an index-less install to report on previous versions of AaC being incompatible...
Is this still the 3.11 pip environment? I've noticed some issues with the current project management setup in 3.11, so I'm also investigating a new build process that's 3.11 compatible.

Actually no, that's what I get when running python3 -m pip download --only-binary :all: --dest . --no-cache aac in an attempt to pull down wheels that I could use with the script.  I expected to be able to replicate what I did on Ubuntu within a Red Hat type environment, but I got that error instead.  That said, I agree that it seems strange to see references to the old versions.

All that said, I was able to successfully install from PyPI use NG pip in all cases.  It's just the air gap that seems to create issues.

I created a PR #515 with some changes to how we create the secure install archive. I'm able to leverage pip to pull down the python 3.9, 3.10, and 3.11 versions of AaC's dependencies into the secure installation, and it's now working in the ubuntu container image I linked above when I set the python version to 3.10 or 3.11.
Are you able to use this secure install package (secure_install_ubuntu-20.04.zip), or the most recent one from the PR pipeline to test in your environments to confirm this is the fix?

I'm on travel and don't have access to my laptop for a few days...maybe the week if the snow storm predicted for Denver is as bad as they say it may be.

I'm on travel and don't have access to my laptop for a few days...maybe the week if the snow storm predicted for Denver is as bad as they say it may be. But I may have an alternative...let's see.

Ok, no worries.

I was able to download and successfully install!!!  I've got an environment problem with my python ssh install I need to resolve before we claim complete success, but I think that's on me.   I'll do a full python reinstall tonight and try again.

I tested the new air-gap installer on a fresh install of Fedora Workstation 37 with python version  3.11.0 present by default.  I installed pip using the package manager.  I then followed the aac air-gap install using the package linked above.  I got no errors!
[jd@fedora aac_secure_install_0.1.5]$ aac version
0.1.5

I think we can call this complete.   Thank you!


Add comments in requirements

This PR adds comments for most of the packages listed in requirements directory.

Automated message from Dropbox CLA bot
@staranjeet, it looks like you've already signed the Dropbox CLA. Thanks!

Cool, posted a few comments on the first few commits.

@timabbott : Updated the PR

Cool, I merged the stuff that isn't the main common.txt file.  I'm a bit confused as to what your methodology was for resorting the order of the things in requirements.txt, though; previously, we had everything alphabetical, and I don't understand the new setup.
Since there are so few places in this file where we have multiple things where the same comment applies, maybe it'd be simpler to keep the sorted order and just duplicate the explanatory comments for the things where a comment applies to more than one?

Actually I changed the order to keep comments common for packages.
@timabbott

Cool, I did one edit pass and then merged this.  Thanks @staranjeet!


hwi 1.1.0 installation fails on arm

As i discovered on https://github.com/rootzoll/raspiblitz/issues/1171 one of the dependencies, the pyside2 package, is not available on arm:
https://pypi.org/project/PySide2/#files
As HWI is pretty useful in a command-line-only environment, imho the pyside2 dependency should somehow be marked optional. It seems that https://www.python.org/dev/peps/pep-0508/#extras has a recommendation on how to do that.
Is that the way to handle it? I might create a PR if someone can confirm?!

yes, the gui should be marked optional. This should be done in pyproject.toml.

@k9ert Can you test #332?

@k9ert Can you test #332?

Done. Thank you very much for the fast response!

#322 is now closed. How often do you create new releases? Can i help with that somehow?

@k9ert Can you test 1.1.1-rc.1 just to make sure that other build changes didn't affect this?

Yes, i will but won't be able until 03-04.June.

I've tested the new release and it works. Thanks for letting me know about the RC nevertheless.


Adapt test suite to use sh-py testing util for numpy stats

Codecov Report
Base: 83.68% // Head: 83.67% // Decreases project coverage by -0.01% :warning:

Coverage data is based on head (d18f1bb) compared to base (0c9d5cd).
Patch has no changes to coverable lines.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #506      +/-   ##
===========================================
- Coverage    83.68%   83.67%   -0.02%
===========================================
  Files           60       60
  Lines         5671     5671
===========================================
- Hits          4746     4745       -1
- Misses         925      926       +1

Impacted Files
Coverage Δ

core/eolearn/core/extra/ray.py
92.68% <0.00%> (-2.44%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Find a way to better deal with distances across different nodes

In general let the user choose the unit he wants to use. Available options are

meter
kilometer
miles
degree

If the input CRS is not projected and the unit is not degree we need to use a default projection. We can use the following code to define a good default projection:
https://github.com/gboeing/osmnx/blob/main/osmnx/projection.py#L104
-> Output a warning that the data was reprojected to the following CRS.
If the input CRS is projected and the unit is not degree we need to figure out the unit of the projection and potentially need to convert to the selected unit (meter, kilometer or miles). We get this information from PROJ.
If the selected unit is degree and the input CRS is projected we will always project to WSG84. -> Output a warning that the data was reprojected to WSG84.
If the input CRS is not projected we don't need to do anything.
In all cases where we need to reproject we need to memorize the input CRS in order to reproject back to it before returning the result.

We might want to add another option "Use unit from input CRS" to allow expert user to use what ever they want and to be backward compatible.

The new Isochrone Map also uses distance based on the input projection right @UrbanGISer ?

The new Voronoi node also uses distance: https://github.com/spatial-data-lab/knime-geospatial-extension/issues/133

The new Voronoi node also uses distance for the buffer


macro export (#900)

macro export (#900)
Commit to FRI first layer (#897)
Exported shared code to temp variables in ExprEvaluator. (#893)
relation tracker

This change is

#909  👈 (View in Graphite)
dev

This stack of pull requests is managed by Graphite. Learn more about stacking.

Merge activity

Dec 2, 4:45 AM EST: A user started a stack merge that includes this pull request via Graphite.


firebase_analytics  FirebaseAnalyticsObserver brought back as "part of" causing issues

9.0.0 removed FirebaseAnalyticsObserver. Before that time, we used an import for observer.dart
FirebaseAnalyticsObserver was brought back in 9.0.3 but it's not imported the same way, and this is causing issues.
The observer file is now "part of firebase_analytics", which requires a different import than before.

This is still a breaking change, even though the class was brought back. The imports in all code written before 9.0.0 are still going to be broken. Does the file absolutely need to be a part of now? Can it be changed back so the old import will still work?
Old import:
import 'package:firebase_analytics/observer.dart';
New import needed:
import 'package:firebase_analytics/firebase_analytics.dart';

Treating this as an enhancement request for proposed change.
/cc @Salakar


Fixes the upload keys function for a device. FLEX-1382

The fix assumes that this function is used for SSLD or PSLD devices (which is the case with the current code)

SonarQube analysis reported 4 issues:

 3 major
 1 info

Watch the comments in this conversation to review them.
Note: the following issues could not be reported as comments because they are located on lines that are not displayed in this pull request:

 Remove the unused method parameter(s) "messageType".
 Remove the unused method parameter(s) "messageType".
 Override this superclass' "equals" method.

All is well.


[Feature request] Exasol: add ability to edit scripts

Hi guys,
Please add ability to edit scripts, like:
create or replace lua script script_name ...
Now only in native ExaPlus you can do it.
Thanks

Hi,
I will implement this feature. This will basically be something like Create DDL. EXAPlus does this the same way as a new tab is opened with the "CREATE OR REPLACE" statement.
As I'm currently on the road this might take some time.
Kind Regards
Charly

Hi @Sargul
It is cool, but if script has parametres DBeaver can't save script and proposes fill parametres in addition window . Could you please fix it?

Hi @UkrZilla ,
I fear I can't do anything in the Exasol Plugin about this as ":name" is always interpreted as paramters on execution in DBeaver as long as SQL-Paramters is enabled.

@serge-rider is there a way to disable sql parameters for SQLDatabasePersistAction?
I'll open the issue again.
Regards
Charly

Please provide more details. What exactly should be changed in parameters handling?
As far as I remember DBeaver ignores parameters in SQLDatabasePersistAction (parameters are valuable only in SQL editor).

Sorry Serge,
wasn't able to verify the informations provided by the user at work. Check at home and you are right. SQLDatabasePersitAction doesn't replace named parameters. Sorry for that.
@UkrZilla you have 2 options:

directly edit the Scripts and function in the Source view like this:

disable SQL parameters or set the char for named parameters to a different char than ":"

I'm closing this issue again.
Kind Regards
Charly

@UkrZilla with the next version of DBeaver (4.0.7) @serge-rider  implemented a fix to not parse parameters in scripts (#1635).
So the workaround of setting the named parameter to something other than ":".

Hi @Sargul
According to directly edit scripts in the Source view: it is not good idea to make it as:
DROP SCRIPT 'scriptname';
CREATE LUA SCRIPT 'scriptname';
Let's assume we are editinig script in the Source view and made mistake, then we click on button 'Persist'. DBeaver will execute: DROP SCRIPT, it will really drop script from database and then won't create script due to mistake


HTTP Error 500: lost connection to cluster: failed to find Session for client stv-cluster

Rancher Server Setup

Rancher version:
Installation option (Docker install/Helm Chart):

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc):

Proxy/Cert Details:

Information about the Cluster

Kubernetes version:
Cluster Type (Local/Downstream):

If downstream, what type of cluster? (Custom/Imported or specify provider for Hosted/Infrastructure Provider):

User Information

What is the role of the user logged in? (Admin/Cluster Owner/Cluster Member/Project Owner/Project Member/Custom)

If custom, define the set of permissions:

Describe the bug
After upgrading to 2.9.1 . Rancher UI is unable to connect to down stream cluster and throws this error "HTTP Error 500: lost connection to cluster: failed to find Session for client stv-cluster-c-m-tbd98v62"
Couple of time it connects with out any visualization but get the download option to download down stream cluster kubeconfig
To Reproduce
Click on the Rancher UI down stream cluster and you get this error ""HTTP Error 500: lost connection to cluster: failed to find Session for client stv-cluster-c-m-rbd98v62""
Result

Expected Result
To get the visualization to navigate around the page to access workloads and pods.
Screenshots

Additional context
Rancher is installed via helm chart (https://github.com/rancher/rancher/blob/v2.9.1/chart/values.yaml) and this helm chart is deployed on EKS.
Downstream cluster is on-prem ubuntu servers.
values:
agentTLSMode: "system-store"
hostname: abc.xyz.com
ingress:
extraAnnotations:
#alb.ingress.kubernetes.io/conditions.rancher: >
#  [{"field":"path-pattern","pathPatternConfig":{"values":["/*"]}}]
#alb.ingress.kubernetes.io/target-type: ip
alb.ingress.kubernetes.io/certificate-arn: ${default_domain_cert_arn}
alb.ingress.kubernetes.io/listen-ports: '[{"HTTP": 80}, {"HTTPS": 443}]'
alb.ingress.kubernetes.io/scheme: internal
alb.ingress.kubernetes.io/subnets: ${subnet_ids}
alb.ingress.kubernetes.io/success-codes: 200,404,301,302
alb.ingress.kubernetes.io/healthcheck-path: "/healthz"
alb.ingress.kubernetes.io/healthcheck-interval-seconds : "60"
alb.ingress.kubernetes.io/load-balancer-attributes: idle_timeout.timeout_seconds=3600
ingressClassName: aws-alb
tls: external
useBundledSystemChart: true
Please let me know if any specific logs are needed. I can provide it based on the request. Thank you !

Hey there, I wasn't able to reproduce the issue so I wanted to get some more information.
Was this an RKE1 or RKE2/K3s downstream cluster?
In Rancher UI in your Local Cluster -> Apps -> Repositories are there are any errors for charts?
If you could please provide the rancher logs(from your local cluster)for more context that would be helpful as well.

Hello @joesims22
RKE2 down stream cluster.
No, errors under respositories.
Please let me know what are logs are needed because there could some sensitive information around our environment.
I can DM you the required details.
Right now I was able to overcome by doing following steps following from other issues posted in github.
I have logged into the downstream cluster node and changed the cattle-cluster-agent deployment from dnsPolicy: ClusterFirst to Default. After that it started working.
I think it might not be specific rancher version deployment but was suspecting some thing around downstream cluster created with Canal . I have even tested the Coredns with the dnstest specified here https://ranchermanager.docs.rancher.com/troubleshooting/other-troubleshooting-tips/dns and it passed all the tests.
Why cluster agent is not able to connect when dnspolicy is set to ClusterFirst . While creating the downstream cluster is it mandatory to specify "Cluster Domain" ? In my case I am not specifying it.
Please let me know if any more details are required from my side.
Thank you!

Even I have followed the above steps I am still experiencing the error "HTTP Error 500: lost connection to cluster: failed to find Session for client stv-cluster-c-m-rbd98v62"" while connecting to downstream cluster.
This time I have followed the step mentioned here "https://www.suse.com/support/kb/doc/?id=000020416" scaling the rancher pod from 3 to 1 and it started working flawlessly.
The moment I scale the rancher deployment to 3 . I see the below errors in rancher logs.
[INFO] Stopping cluster agent for c-m-tbd98v62
2024/09/09 04:13:00 [ERROR] failed to start cluster controllers c-m-tbd98v62: context canceled
2024/09/09 04:13:02 [ERROR] Failed to connect to peer wss://<IP_ADDRESS>/v3/connect [local ID=<IP_ADDRESS>]: dial tcp <IP_ADDRESS>:443: i/o timeout
2024/09/09 04:13:17 [ERROR] Failed to connect to peer wss://<IP_ADDRESS>/v3/connect [local ID=<IP_ADDRESS>]: dial tcp <IP_ADDRESS>:443: i/o timeout
2024/09/09 04:13:32 [ERROR] Failed to connect to peer wss://<IP_ADDRESS>/v3/connect [local ID=<IP_ADDRESS>]: dial tcp <IP_ADDRESS>:443: i/o timeout
Can you please give me some pointers on how to avoid this?
We can't scale rancher to 1 and need the HA .
Thank you !

@joesims22
Any guidance of this part why a single rancher pod is working and when scale to 3 it's failing of above messages.
Do I need to check any thing in EKS cluster ?
Thank you !

Did you perhaps find a resolution on this?  I have the same issue with 1 of ±12 clusters, all created identically with Terraform (same RKE version etc.) the only difference is the IP range used for the instances hosting the nodes.  Deleting the cluster entirely didn't resolve the issue.
Thanks

FYI renaming the Rancher cluster fixed this from central-tls-developmentv2-cluster to central-tls-dev2-cluster, changing the IP subnet range for the nodes made no difference 🤷

Glad to hear that. Mine was solved when i scale down the rancher from 3 to 1 replica.


Handle empty parent version

Intended to close issue 375.
The PR allows the Maven plugin to handle project parents with not version, but it causes a potential breaking change as the default goes from ${project.parent.version} to nothing, if people want a parent version they will now have to explicitly add it.
I am not sure how concerned you are about that change in behaviour.

@pmckeown are you okay with this change?

Sorry!  Got half way through testing it then life got in the way.  Will try to get it tested and released this weekend!


Suggestion: move test text to a separate folder

Currently, the license template and text files are in the same folder as the implementation code.  This make is hard to read.  Suggest moving to a separate folder such as testdata.

@goneall The template and text files that are in the same folder as the implementation one are just for testing. They will be removed once I complete the project. There is also a TemplateData folder which consists of all the templates in the source code which would mainly be used.

I thik we can close this issue now.


anatomy_snapshots.py using increasingly more RAM

I'm making snapshots of a huge number of Freesurfer subjects (~1000). Every 100 or so, anatomy_snapshots.py is using so much percent of RAM (over 60%) that the server I am letting it run on becomes very unresponsive (we only have 16GB of RAM), so other people using it cannot work properly anymore and I have to quit and restart the execution of the script.
From monitoring its process with top, it seems as if memory usage is increasing with every subject. Do you have an idea what could be the reason for the accumulating memory use? (I don't use the -noclose flag, if this is important)

Yeah, there's a memory leak in the surface visualization. It's not 100% clear whether it's a problem in PySurfer (see https://github.com/nipy/PySurfer/issues/91) or in Mayavi itself, but there's no way I know of to avoid it other than to batch running the script.
1000 is a lot of subjects!

Alright, thanks! I didn't see the -subjects flag before, now I split my subjects.txt in files with 30 lines/subjects. That should solve the problem...
Yeah, it's a multi-center structural imaging study...
Thanks!
Am 17.02.2015 um 07:43 schrieb Michael Waskom<EMAIL_ADDRESS>
Yeah, there's a memory leak in the surface visualization. It's not 100% clear whether it's a problem in PySurfer (see nipy/PySurfer#91) or in Mayavi itself, but there's no way I know of to avoid it other than to batch running the script.
1000 is a lot of subjects!
�
Reply to this email directly or view it on GitHub.


Version 1 interface.

The interface for themekit isn't the best and is ambiguous in some commands. We should make it better.
Design Doc:
https://docs.google.com/document/d/1QvkJP_2rnpVxcz5Lwqi6hPWnSsuEet-ngcxAfv-7eSw/edit?usp=sharing

Just took a look. I think sync is a good idea and it should definitely integrate with browsersync to --watch files on change with browser refreshing.
I would also check out the Firebase CLI as it has a lot of great conventions (google) which would be good to model.

browser sync is out of the scope of themekit and will be part of other tools that will soon be open sourced.

Would theme customize be something reasonable? Especially with the advent of sections it might be helpful to be able to pull up the theme customizer to be able to see and edit those sections.
I'd be willing to contribute this part of functionality.

@chryton I am not sure I fully understand. Would you just like a shortcut to opening the online store editor?

Correct, just like what theme open does.

Yeah that wouldn't be too hard.

Would it be possible to allow us to set the target browser to use for theme open? I use Chrome as my default browser, but it's heavily loaded with extensions, so when I'm developing I use Chrome Canary. Not a big deal since all I have to do it snag the url while chrome is loading the page (while it still contains url param for preview theme), and drop that into canary, but it would be more convenient if I could define the browser either as a flag e.g. theme open --canary or by adding a line to config.yml, perhaps pointing to the canary executable. This would also be convenient for cross browser testing.

@cameroncowden please open a new issue for this as it has no relation to version 1 planning.

@tanema To further clarify what I mean:
https://ecommerce.shopify.com/c/shopify-apis-and-technology/t/duplicate-theme-via-api-494136
While this can theoretically be done by copying assets one by one, for larger themes that will take a significant amount of time due to the API call limit. Shopify has obviously already implemented the duplication internally, and if that were exposed via the API it could turn minutes and hundreds of requests into a single request and a matter of seconds.


Predicting Student Dropout Rates Using Machine Learning Models

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
I will predict th student Dropout rates using Logistic Regression, K-Nearest Neighbors (KNN) and AutoML Tools (like, AutoKeras, and TPOT)
Use Case
Academic and Socio-Economic Factor
Benefits
No response
Add ScreenShots
No response
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[x] I want to work on this issue

@sanjay-kv, Sir i didn't get assigned yet, still go the labels added

@sanjay-kv I can work on this


Freeze on loading targets that need interpreter

Loading a file that requires the interpreter causes HIE to freeze and changing files does not help. "Loading done" is never logged (see https://github.com/alanz/ghc-mod/blob/66fc0980d2c731caf36215969a31bff8ef40f3e2/core/GhcMod/Target.hs#L555). Loading files that do not need HscInterpreted proceeds smoothly.
DEBUG: loadTargets: filterModSums: True
info: loadTargets:
      Target needs interpeter, switching to LinkInMemory/HscInterpreted. Perfectly normal if anything is using TemplateHaskell, QuasiQuotes or PatternSynonyms.

hie.log
Platform:
Windows 10
VSCode 1.25.1
HLS Extension 0.0.20
HIE <IP_ADDRESS> 17e1116
GHC 8.4.3

Just wanted to add that I switched to GHC-8.2.2 on LTS-11.20 and the problem went away.


Navbar current page highlight

Trello card
567
Context
background highlight on current navbar links
Changes proposed in this pull request
Guidance to review
This was reverted yesterday as I forgot to update the mobile links to reflect the changes in the content. I have added some tests to prevent that happening again

Looks fine other than the minor things I mentioned

Made those changes


Expose the prefixed storage mod

Any reason this was hidden from the users? (except of missing docs)
It actually breaks some of my tests where I change storage directly using this hack
But I also need it now for trying to build a custom module.

Hey @DariuszDepta
I got confused a little, I just noticed that im using use cosmwasm_storage::PrefixedStorage; which the whole crate is being deprecated, for some reason I was sure i was using the multi-test one.
So regarding 2 you are correct, seems like it was never public.
Regarding 3, I have 2 cases where I want to use the prefixed helper:

Modify storage of a contract directly, like in the "hack" above, there are some cases where its not easy to reach to a specific storage value for testing, the easiest way to do so would be to modify the storage directly, and make sure the contract behaves as expected in that case,
An example to this is to set a storage to an odd Uint128 number, and make sure rounding is being done correctly.

Use a namespace in my custom module, looking at how the bank module is using storage, I assumed I can do the same in my custom module, but the prefixed helper is not exposed.
Exposing the whole module is probably not the best way, we can expose only the helpers.

For my first use case above:

Modify storage of a contract directly, like in the "hack" above, there are some cases where its not easy to reach to a specific storage value for testing, the easiest way to do so would be to modify the storage directly, and make sure the contract behaves as expected in that case,
An example to this is to set a storage to an odd Uint128 number, and make sure rounding is being done correctly.

Seems like the wasmKeeper already have a useful helper which is private:
https://github.com/CosmWasm/cw-multi-test/blob/main/src/wasm.rs#L296-L306
Can we can expose it to the user to allow something like this?
let contract_addr = Addr::unchecked("some_addr");
let storage = app.storage_mut();
let contract_storage = WasmKeeper::contract_storage(storage, contract_addr);

SOME_MAP.save(contract_storage, key, value)?;

Thanks for your explanations, yes, there are such use cases already in MultiTest - that's why I asked for details. I am just looking now, how to make it public and support your second use-case and stay aligned with our plans... ;-)

Hi @Art3miX,
would you take a look at #153? This should address your case no.1 ;-)
Regards,
Darek

Hi @Art3miX, I guess use-case no. 2 is addressed in #154.

Looks good!
Thank you very much for addressing those use cases.

Changes are applied to version 2.0.0. Next step we will backport these into version 1.0.0. Please see #155 and #156. I am closing this PR as completed.


Improve TPS, part 1

Problem
TPS
Solution
Remove some duplicate verifications and tune things a bit. Solve #667 and #668 (though this one probably deserves a separate test case).

@fyrchik, @AnnaShaleva?


Handlebars helper for assets errors when input is SafeString

Issue Summary
Try to compute asset path with asset helper and passing (concat "some" "string") as input i.e. check {{asset (concat 'some' 'string'}}. It will error with [error.hbs] path.match is not a function.
Relevant lines of code:
https://github.com/TryGhost/Ghost/blob/c667620d8f2e32c96fe376ad0f3dabc79488532a/ghost/core/core/frontend/helpers/asset.js#L28
https://github.com/TryGhost/Ghost/blob/c667620d8f2e32c96fe376ad0f3dabc79488532a/ghost/core/core/frontend/meta/asset-url.js#L16
https://github.com/TryGhost/Ghost/blob/c667620d8f2e32c96fe376ad0f3dabc79488532a/ghost/core/core/frontend/helpers/concat.js#L7
Steps to Reproduce
{{asset (concat 'some' 'string'}} produces error regardless of the concatenated string provided. The reason is that asset helper expect plain strings as input but (concat …) produces SafeString object which results in the above error.
Ghost Version
5.35.1-alpine (https://hub.docker.com/_/ghost)
Node.js Version
Comes with https://hub.docker.com/_/ghost
How did you install Ghost?
https://hub.docker.com/_/ghost
Database type
MySQL 8
Browser & OS version
Does not matter
Relevant log / error output
TypeError: path.match is not a function
      at getAssetUrl (core/frontend/meta/asset-url.js:16:14)
      at Object.asset (core/frontend/helpers/asset.js:40:9)
      at Object.wrapper (/home/monkeysees/ghost-git/node_modules/handlebars/dist/cjs/handlebars/internal/wrapHelper.js:15:19)
      at Object.eval [as main] (eval at createFunctionContext (/home/monkeysees/ghost-git/node_modules/handlebars/dist/cjs/handlebars/compiler/javascript-compiler.js:262:23), <anonymous>:10:123)
      at main (/home/monkeysees/ghost-git/node_modules/handlebars/dist/cjs/handlebars/runtime.js:208:32)
      at ret (/home/monkeysees/ghost-git/node_modules/handlebars/dist/cjs/handlebars/runtime.js:212:12)
      at ret (/home/monkeysees/ghost-git/node_modules/handlebars/dist/cjs/handlebars/compiler/compiler.js:519:21)

Code of Conduct

[X] I agree to be friendly and polite to people in this repository

I’ll be happy to work on a PR.

@monkeysees We'd welcome a PR to fix this 🙂


NestArray default value

I just spent a couple hours debugging an issue loading UBJSON created via savebj() from Matlab in Node.js using the @shelacek/ubjson package.
It turns out I needed to enable NestArray in the savebj() call.
I think it would make sense to enable NestArray automatically if the user has specified UBJSON as the output format in order to keep people from dealing with this issue.
From a UX perspective, the note for the UBJSON option states it will use the Draft 12 style. I thought this covered me. I didn't realize I also had to search all other options to manually set them to match Draft 12.
You seem to do this already when the user selects MessagePack as the output format.

you should call saveubjson.m instead, see this help info
https://github.com/fangq/jsonlab/blob/master/saveubjson.m#L20-L27

Got it. Thanks.


Update LICENSE

This pull request adds license files for the core system (MIT), the content (CC), and third party licenses.
Fixes #22.
Also closed the 'Ready for Authors' milestone.

Thanks for the suggestion @ravenmaster001. I've clarified the coverage of licenses in the README, and made a note at the top of the LICENSE-CONTENT file. I've left the LICENSE file as is, because I want it easily identifiable by license bots (having a LICENSE file with extra content is placed as unknown).


Uashіeld for android

Is it possible to add support for sharing android (under termux)

its posible to compile for cordova. quasar dev -m cordova -T android . But it need further investigations


Add a modified property to project and IDE

Adds a modified property to <playground-project> and <playground-ide> which indicates whether the user has modified, added, or removed any project files. Resets whenever a new project is loaded.
We'll use this as part of fixing https://github.com/lit/lit.dev/issues/485.

Hmm, I think not without exposing or monkey patching some internals that I would prefer not to do. I did check it manually with a log statement, though.

Clarify: the test does confirm that the result is correct by checking the value twice after each change -- it just doesn't confirm that it's doing it in the optimal way.


TypeError: fit() got an unexpected keyword argument 'nb_epoch'

I'm training the model using fit with parameter nb_epoch. However, it shows the error:
`---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
 in
1 #fit model
----> 2 model.fit(X_train, Y_train, batch_size=32, nb_epoch=10, verbose=1)
/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/tensorflow/python/keras/engine/training.py in _method_wrapper(self, *args, **kwargs)
64   def _method_wrapper(self, *args, **kwargs):
65     if not self._in_multi_worker_mode():  # pylint: disable=protected-access
---> 66       return method(self, *args, **kwargs)
67
68     # Running inside run_distribute_coordinator already.
TypeError: fit() got an unexpected keyword argument 'nb_epoch'`
I wanna know how to fix it?

It's just epochs now. nb_epoch was deprecated years ago.

It's just epochs now. nb_epoch was deprecated years ago.

Thanks for your help. But when I used epochs , I got another error:
`Epoch 1/10
InvalidArgumentError                      Traceback (most recent call last)
 in
1 #fit model
----> 2 model.fit(X_train, Y_train, batch_size=32, epochs=10, verbose=1)
/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/tensorflow/python/keras/engine/training.py in _method_wrapper(self, *args, **kwargs)
64   def _method_wrapper(self, *args, **kwargs):
65     if not self._in_multi_worker_mode():  # pylint: disable=protected-access
---> 66       return method(self, *args, **kwargs)
67
68     # Running inside run_distribute_coordinator already.
/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/tensorflow/python/keras/engine/training.py in fit(self, x, y, batch_size, epochs, verbose, callbacks, validation_split, validation_data, shuffle, class_weight, sample_weight, initial_epoch, steps_per_epoch, validation_steps, validation_batch_size, validation_freq, max_queue_size, workers, use_multiprocessing)
846                 batch_size=batch_size):
847               callbacks.on_train_batch_begin(step)
--> 848               tmp_logs = train_function(iterator)
849               # Catch OutOfRangeError for Datasets of unknown size.
850               # This blocks until the batch has finished executing.
/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/tensorflow/python/eager/def_function.py in call(self, *args, **kwds)
578         xla_context.Exit()
579     else:
--> 580       result = self._call(*args, **kwds)
581
582     if tracing_count == self._get_tracing_count():
/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/tensorflow/python/eager/def_function.py in _call(self, *args, **kwds)
609       # In this case we have created variables on the first call, so we run the
610       # defunned version which is guaranteed to never create variables.
--> 611       return self._stateless_fn(*args, **kwds)  # pylint: disable=not-callable
612     elif self._stateful_fn is not None:
613       # Release the lock early so that multiple threads can perform the call
/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/tensorflow/python/eager/function.py in call(self, *args, **kwargs)
2418     with self._lock:
2419       graph_function, args, kwargs = self._maybe_define_function(args, kwargs)
-> 2420     return graph_function._filtered_call(args, kwargs)  # pylint: disable=protected-access
2421
2422   @property
/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/tensorflow/python/eager/function.py in _filtered_call(self, args, kwargs)
1663          if isinstance(t, (ops.Tensor,
1664                            resource_variable_ops.BaseResourceVariable))),
-> 1665         self.captured_inputs)
1666
1667   def _call_flat(self, args, captured_inputs, cancellation_manager=None):
/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/tensorflow/python/eager/function.py in _call_flat(self, args, captured_inputs, cancellation_manager)
1744       # No tape is watching; skip to running the function.
1745       return self._build_call_outputs(self._inference_function.call(
-> 1746           ctx, args, cancellation_manager=cancellation_manager))
1747     forward_backward = self._select_forward_and_backward_functions(
1748         args,
/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/tensorflow/python/eager/function.py in call(self, ctx, args, cancellation_manager)
596               inputs=args,
597               attrs=attrs,
--> 598               ctx=ctx)
599         else:
600           outputs = execute.execute_with_cancellation(
/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/tensorflow/python/eager/execute.py in quick_execute(op_name, num_outputs, inputs, attrs, ctx, name)
58     ctx.ensure_initialized()
59     tensors = pywrap_tfe.TFE_Py_Execute(ctx._handle, device_name, op_name,
---> 60                                         inputs, attrs, num_outputs)
61   except core._NotOkStatusException as e:
62     if name is not None:
InvalidArgumentError:  Conv2DCustomBackpropFilterOp only supports NHWC.
[[node gradient_tape/sequential_5/conv2d_8/Conv2DBackpropFilter (defined at :2) ]] [Op:__inference_train_function_2109]
Function call stack:
train_function`
Is this question related to the version of my laptop or python?
I'm using macOS 10.13.6, Python 3.6.4, tensorflow 2.2.0.

please check your code very well, 'epochs' work for me.

please check your code very well, 'epochs' work for me.

InvalidArgumentError:  Default MaxPoolingOp only supports NHWC on device type CPU
Gets this error when epochs was used .

Hi all,
TypeError: fit() got an unexpected keyword argument 'epochs'
I am getting the same error in spite of using epochs and my tensorflow is version 2+. I tried nb_epoch as well and same error.
What could be the reason? please let me know if you know how to fix this.
TypeError                                 Traceback (most recent call last)
Input In [66], in <cell line: 7>()
1 early_stopping = tf.keras.callbacks.EarlyStopping(
2     monitor='val_accuracy',
3     mode='max',
4     patience=6
5 )
----> 7 history = model_xgb.fit(X_for_training,y_train,epochs=25,batch_size=32,verbose=2,callbacks=[early_stopping])
File ~/miniconda3/lib/python3.8/site-packages/xgboost/core.py:620, in require_keyword_args..throw_if..inner_f(*args, **kwargs)
618 for k, arg in zip(sig.parameters, args):
619     kwargs[k] = arg
--> 620 return func(**kwargs)
TypeError: fit() got an unexpected keyword argument 'epochs'


refactor value_option in variables.py

carry over from #841. See inline comments in ec7206a797876e8b07f9e83646772a741757be34. The logic surrounding the loop variable value_option used again later in __new__ is confusing and possibly bug-prone.

@galbramc and @whoburg , by the construction of the code, value_option is guaranteed to have a value which makes sense if values is truthy. I do think it could use a bit more clarity there, but value_option is not the problem.

Yes it is, but the default value is actually "sp_init", and not None in case the 'if' statement inside the loop never evaluates to true. This is just a tad confusuing to parse out the logic that "sp_init" is the default only because its the last thing in the list in the for loop.


There are two items for Dragon Leg with Sauce

Dragon Leg With Sauce is the item which is craftable, but inedible. Plate of Dragon Leg With Sauce IS edible, but uncraftable. Not sure what happened here.

Ah, my bad. I've been testing stuff with my modpack before actually playing proper so I didn't experience it organically. Sorry to bother you.


Flexible Event Catcher

Currently, the event catchers for link, picture and video is not dynamic - meaning, you can't catch a specific picture, link, video or sticker.
We need to make sure that we can pass an incoming message to the event and use that as part of the validation process.

A comparator was introduce for this. Will be included on the next release.


OOB invitation not deleted when using use_existing_connection

When having an active connection between two agents and using the use_existing_connection flag when receiving another invitation from the inviter using the Out-Of-Band protocol, the invitation doesn't get deleted at the inviter. I would expect the invitation to be deleted in the database of the inviter.
Steps to reproduce (which will hopefully make my very long sentence above more understandable):

Create an OOB invitation at the inviter
Receive the invitation at invitee
Continue to establish the connection until it's active (I had auto_accept=true on both agents)
Create another OOB invitation at the inviter
Receive the invitation at invitee with flag use_existing_connection=true
invitee immediately get's back the previously created and still active connection
The invitation remains in the database of the inviter. There are two connections when querying the /connections endpoint: one is the still active connection, one is the invitation pending in state invitation

Is it expected that the invitation will remain at the inviter? In my opinion, this will pollute the database with already used invitations.

@ffendt PR #1521 (more details), deletes stale ConnRecords which were last updated at least 3 hours back. The trigger for this  is creating an OOB invitation, so whenever a new OOB invitation is created then this cleanup is performed. Does this strategy sound reasonable?

@shaangill025, thanks for the quick PR. I'll close this issue for now.


OOM Issues on prometheus instances

The problem
I am running prometheus operator and hoped for a simple prometheus setup which runs reliably. Unfortunately I am running into lots of prometheus pod restarts (all instances) which are caused due too OOM Kills by Kubernetes.
Right now I am running 3 prometheus instances each having 20GB Ram Limit, configured for 8d retention plus a thanos sidecar container which syncs data to our Google Cloud Storage Bucket. The Bucket has no lifecycle (I could enable 14days deletion lifecycle though?). This started happening just recently after having a stable prometheus environment for a couple weeks. Thus I wonder if it's caused by Thanos which may load the prometheus data on startup into RAM and then runs out of memory because of this?
I am trying to figure out what is causing our OOM issues and how we could get rid of them. I looked into limiting the number of max samples as this has helped us before we changed our setup to prometheus operator, but apparently I have no possibility to edit such a setting. What could we do to debug this?
Environment

Prometheus Operator version:

v0.28.0

Kubernetes version information:

Client Version: version.Info{Major:"1", Minor:"11", GitVersion:"v1.11.2", GitCommit:"bb9ffb1654d4a729bb4cec18ff088eacc153c239", GitTreeState:"clean", BuildDate:"2018-08-08T16:31:10Z", GoVersion:"go1.10.3", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"11+", GitVersion:"v1.11.6-gke.2", GitCommit:"04ad69a117f331df6272a343b5d8f9e2aee5ab0c", GitTreeState:"clean", BuildDate:"2019-01-04T16:19:46Z", GoVersion:"go1.10.3b4", Compiler:"gc", Platform:"linux/amd64"}

Kubernetes cluster kind:

Google Kubernetes Engine

Manifests:

My Prometheus CRD Manifest:
apiVersion: monitoring.coreos.com/v1
kind: Prometheus
metadata:
  creationTimestamp: 2019-01-23T14:39:17Z
  generation: 1
  labels:
    app: prometheus-operator-prometheus
    chart: prometheus-operator-1.8.0
    heritage: Tiller
    release: prometheus-operator
  name: prometheus-operator-prometheus
  namespace: prometheus
  resourceVersion: "227656940"
  selfLink: /apis/monitoring.coreos.com/v1/namespaces/prometheus/prometheuses/prometheus-operator-prometheus
  uid: a96b3732-1f1c-11e9-94aa-42010a8401e2
spec:
  additionalScrapeConfigs:
    key: additional-scrape-configs.yaml
    name: prometheus-operator-prometheus-scrape-confg
  alerting:
    alertmanagers:
    - name: prometheus-operator-alertmanager
      namespace: prometheus
      pathPrefix: /
      port: web
  baseImage: eu.gcr.io/redacted-registry-v002/prometheus
  externalUrl: http://prometheus-operator-prometheus.prometheus:9090
  listenLocal: false
  logLevel: info
  nodeSelector:
    cloud.google.com/gke-nodepool: default
  paused: false
  query:
    maxSamplesPerSend: 500000
  replicas: 3
  resources:
    limits:
      cpu: 4
      memory: 20Gi
    requests:
      cpu: 1
      memory: 15Gi
  retention: 8d
  routePrefix: /
  ruleSelector:
    matchLabels:
      app: prometheus-operator
      release: prometheus-operator
  securityContext:
    fsGroup: 2000
    runAsNonRoot: true
    runAsUser: 1000
  serviceAccountName: prometheus-operator-prometheus
  serviceMonitorNamespaceSelector: {}
  serviceMonitorSelector:
    matchLabels:
      release: prometheus-operator
  storage:
    volumeClaimTemplate:
      spec:
        accessModes:
        - ReadWriteOnce
        resources:
          requests:
            storage: 100Gi
        storageClassName: standard
  thanos:
    baseImage: eu.gcr.io/redacted-registry-v002/thanos
    gcs:
      credentials:
        key: credentials.json
        name: sa-thanos
    objectStorageConfig:
      key: objectStorageConfig.yaml
      name: thanos-object-storage-config
    tag: v0.2.1
  version: v2.7.1

Prometheus Operator Logs:

// Only lots of these logs
level=info ts=2019-03-06T12:15:01.679123998Z caller=operator.go:968 component=prometheusoperator msg="sync prometheus" key=prometheus/prometheus-operator-prometheus
level=info ts=2019-03-06T12:15:02.917486501Z caller=operator.go:968 component=prometheusoperator msg="sync prometheus" key=prometheus/prometheus-operator-prometheus
level=info ts=2019-03-06T12:15:06.046226412Z caller=operator.go:968 component=prometheusoperator msg="sync prometheus" key=prometheus/prometheus-operator-prometheus
level=info ts=2019-03-06T12:15:09.188421797Z caller=operator.go:968 component=prometheusoperator msg="sync prometheus" key=prometheus/prometheus-operator-prometheus
level=info ts=2019-03-06T12:15:09.775070723Z caller=operator.go:968 component=prometheusoperator msg="sync prometheus" key=prometheus/prometheus-operator-prometheus

Right now I am running 3 prometheus instances each having 20GB Ram Limit, configured for 8d retention

Personally, I'm running the kube-prometheus (along with a bunch of other workload) on a 3 node cluster on DigitalOcean that have 4GB each. So your setup should be sufficient!

plus a thanos sidecar container which syncs data to our Google Cloud Storage Bucket. The Bucket has no lifecycle (I could enable 14days deletion lifecycle though?). This started happening just recently after having a stable prometheus environment for a couple weeks. Thus I wonder if it's caused by Thanos which may load the prometheus data on startup into RAM and then runs out of memory because of this?

The sidecar simply watches the filesystem and once there's a new block written it will ship it to object storage. The sidecar itself doesn't read the data. If you run the Thanos compactor you don't really need to set a retention of the data in object storage. We plan to go with 13 months at some point.
By looking at the Pod description kubectl describe can you see more information on what container caused the OOM kill?

I checked the pod description and it is the prometheus container. I believe I can reproduce the issue by changing the timeframe of specific Grafana dashboards from Last 3 hours to Last 7 days for instance. It seems Prometheus tries to load too many samples into RAM then?

Sure, that exactly what happens if you query data for 7 days. Because you already run Thanos, I recommend running the compactor so it can start down sampling older metrics.

@metalmatze In an environment where multiple teams access the same backend it is not avoidable that one out of 50 developers or so uses Last 7 days timerange at least once in a week.
I am fine when prometheus does not respond to such queries, but I am not okay that my prometheus instances can be crashed due to this. I believe I prevented this with my own prometheus setup (before I started using prometheus operator) by limiting the number of max samples. As far as I know I can not change this with prometheus operator.

If the other developers mostly look at dashboards that show the last 7 days then maybe it's worth caching the data. Something like https://github.com/Comcast/trickster might help you then. Otherwise you need to shard Prometheus per team for example. Shouldn't be that hard either with the Prometheus Operator up and running.

Why isn't it possible to set the maximum number of allowed samples instead if this is exactly what could take down the prometheus server? Even if I create one prometheus server for each team the problem will still exist, unless there are enough resources available to handle such big requests.
Again I do not necessarily want to allow queries reaching back 7days if there is simply too much data to evaluate. It's just unacceptable that such queries bring down my prometheus server and therefore trigger alerts (because of execution errors / no data)

Starting with v2.5.0 Prometheus supports the --query.max-samples flag. As far as I can tell we don't support this flag yet. We should though.

@brancz Would you accept a PR for that change?  It doesn't seem terribly involved.  Rule eval is currently failing on our largest cluster for this exact reason and we have no way to mitigate today.

Closing this as we now have a way to limit the maximum samples thanks to #2505.
@weeco plz write back if you think this needs to be reopened.


can I provide array of imageURLs instead of images?

I have an array of image URLs which should get downloaded cached automatically. If I scroll fast it should cancel previous requests and provide current request.
is it supports ?

@santhoshs5, yes, this is supported


Parallel sweep

Weights and Biases version: 0.8.12
Python version: 3.6.9
Operating System: Debian 10

Description
I'm running multiple wandb agents in order to parallelize my grid search, and for each of them I have to set CUDA_VISIBLE_DEVICES to make them run on different GPUs. It would be really nice if there was a built-in feature to distribute agents over multiple GPUs.
There also seems to be an issue where starting two agents at the same time (e.g. in a script) makes them get the same parameters for the first run. A little delay fixes it though.

Thanks for reporting these sweeps issues!

Came with the same question in mind, any update or resolution for us to this?

any update?

any update?

Same request, this would be very useful

Any updates on this feature?

We isolate experiments at the agent level.  You should be able to achieve this by starting multiple agents on the same machine each with a different CUDA_VISIBLE_DEVICES environment variable.  i.e.
CUDA_VISIBLE_DEVICES=0,1 wandb agent XXXX/XXXX/XXX
CUDA_VISIBLE_DEVICES=2,3 wandb agent XXXX/XXXX/XXX

Then each agent would only have access to those 2 GPU's.  Another option would be to have your code specify the visible device as a hyper parameter and sweep over it, however this could cause issues where two agents tried to use the same device so may not be worth it.

Hey everyone, in the past 2 years we've majorly reworked the CLI and UI for Weights & Biases. We're closing issues older than 6 months. Please comment to reopen.

We isolate experiments at the agent level. You should be able to achieve this by starting multiple agents on the same machine each with a different CUDA_VISIBLE_DEVICES environment variable. i.e.
CUDA_VISIBLE_DEVICES=0,1 wandb agent XXXX/XXXX/XXX
CUDA_VISIBLE_DEVICES=2,3 wandb agent XXXX/XXXX/XXX

Then each agent would only have access to those 2 GPU's. Another option would be to have your code specify the visible device as a hyper parameter and sweep over it, however this could cause issues where two agents tried to use the same device so may not be worth it.

I created a config yaml including an argument like device_ids: 0,, which will be used to set the device.  I use the lightning fabric framework. I want to change the argument to parallelize different runs on diffferent GPUs on the single machine. I have four gpus. When the first four runs are done, the latter agents all try to use the last GPU. Is there any solution? Or is is still recommended to use CUDA_VISIBLE_DEVICES environment variable?


Facebook Pixel Tracking

Would it be possible for you to add an example for a Facebook Pixel code to the tracking documentation?

It looks like the Facebook Pixel Code works similar to Google Tag Manager, where it watches the History State. Can you confirm? I guess the documentation isn't necessary, as it does not require any customization.

Hello @bigskillet ;
We don't have really the time to check the behavior of Facebook Pixel Code.
Although, if you are able to test on your side and provide us the information of the working setup for you, we would appreciate if you could share so we could help other people :-)
Otherwise, we'll have to have a look later on, sorry.

Google Analytics
Tracking the initial page load
When using Google Analytics, you can use the following snippet to track the initial page load. Simply paste the snippet in your site's footer, and replace GA_TRACKING_ID (on both locations) with your Google Analytics Tracking ID.
HTML
<script async src="https://www.googletagmanager.com/gtag/js?id=GA_TRACKING_ID"></script>
<script>
  window.dataLayer = window.dataLayer || [];
  function gtag() {
    dataLayer.push(arguments);
  }
  gtag('js', new Date());
  gtag('config', 'GA_TRACKING_ID');
</script>

Tracking page-transitions
When it comes to page transitions, you can use the following snippet. Don't forget to replace GA_TRACKING_ID with your Google Analytics Tracking ID.
JavaScript
H.on('NAVIGATE_END', ({ to, location }) => {
  if (typeof gtag !== 'undefined') {
    gtag("config", "GA_TRACKING_ID", {
      page_path: location.pathname,
      page_title: to.page.title,
      page_location: location.href,
    });
  }
});

Facebook Pixel
Tracking the initial page load
Integrating FaceBook Pixel is quite similar to integrating Google Analytics. If you use the following snippet, you can simply replace FBP_TRACKING_ID (on both locations) with your Facebook Pixel Tracking ID.
HTML
<script>
  !function(f,b,e,v,n,t,s)
  {if(f.fbq)return;n=f.fbq=function(){n.callMethod?
  n.callMethod.apply(n,arguments):n.queue.push(arguments)};
  if(!f._fbq)f._fbq=n;n.push=n;n.loaded=!0;n.version='2.0';
  n.queue=[];t=b.createElement(e);t.async=!0;
  t.src=v;s=b.getElementsByTagName(e)[0];
  s.parentNode.insertBefore(t,s)}(window, document,'script',
  'https://connect.facebook.net/en_US/fbevents.js');
  fbq('init', 'FBP_TRACKING_ID');
  fbq('track', 'PageView');
</script>
<noscript>
  <img height="1" width="1" style="display:none" src="https://www.facebook.com/tr?id=FBP_TRACKING_ID&ev=PageView&noscript=1"/>
</noscript>

Tracking page-transitions
To track page transitions with Facebook Pixel, use the following snippet.
JavaScript
H.on('NAVIGATE_END', ({ to, location }) => {
  if (typeof fbq !== 'undefined')
    fbq('track', 'PageView');
  }
});


ci: check that locize placeholders are the same in all languages

Translators sometimes accidentally translate {{placeholder}} variables, like "name"=>"nombre", breaking the translation.
This script checks that for each key in English (reference lang), all translations contain the same placeholders.

CI fails with the expected errors in the locize files - we should probably fix all issues so make CI pass here, then merge to prevent future issues.

@thisconnect done

thank you. fixed

SL, made same change as it is in buy.info.disclaimer.protection.descriptionGeneric
FR, fixed placeholders
FA and AR google translated
IT fixed placeholder
ja used gpt to translate again (it's only some warning about bitsurance and new tx alert message)
es fixed placeholder
he deepled
zh deepled

Thanks. One remaining:

+ ./scripts/check-locize-placeholders.py
Issues found in fa:
  - Placeholder mismatch at 'notification.newTxs_one': '{{count}} معاملات جدید در: {{accountName}}' - found ['{{count}}', '{{accountName}}']; expected ['{{accountName}}']

Thanks. One remaining:

oups 🙈 sorry, should be fixed now.

Yeah it's fixed now, thanks.

Why was the scripts/check-locize-placeholders.py written in python? Why not Go, IMO it's not good to introduce another programming language.

Why not Go

or Node.js 😇

Indeed, because of frontends/web/src/locales/... i would see it within Node and "frontend".

Why was the scripts/check-locize-placeholders.py written in python? Why not Go, IMO it's not good to introduce another programming language. More complexity, more to maintain, etc.

Good point about another language, but it's available anyway in the Docker image.
Python felt like the best tool for the job - it's is kind of perfect for simple scripts like this imho. No strong opinion though, Go would have been fine too, but I don't think I'll re-do it now.

Yes, but like I said, then there is a Docker Python dependency. What if Python 4 is released and no more support for code fragments, etc. IMO the solution should be in Go, Node, or bash.

Fyi, this is not the first Python script. ./scripts/get_version has been there since the beginning and is literally untouched since 5 years :grin: as a measure of how much of a maintenance burden it is.

Yes, but like I said, then there is a Docker Python dependency. What if Python 4 is released and no more support for code fragments, etc. IMO the solution should be in Go, Node, or bash.

I see your point, but these risks are not realistic imho. Python is available (has always been, will likely always be), Python 4 is not coming for many years if ever, breaking changes are very rare (not sure I ever experiences a breaking change in Python except 2->3). I really don't expect much Python-related maintenance overhead for this.


Actually remove the nr1-alerts-los-migrator from the app catalog

relates to: https://github.com/newrelic/nr1-alerts-los-migrator/issues/22
We've found that this app is still in the catalog 🤷
Steps to reproduce
login newrelic one on production,
go to I/O
search for loss of signals
see the app

Acceptance Criteria

remove all subscribed accounts for nr1-alerts-los-migrator - UUID 6e48950e-bb5b-4c65-a3c3-045d83b4df46
Remove this application from the Catalog in the US / EU and staging
Confirm when complete.

I've asked for this to be done in #help-nr1-platform-services


refactor readonly mode check

is there a real case for testing that?

There are conflicting files..

is there a real case for testing that?

Qingxin will test that in a real scenario


[bug] zlib support required by widely used python modules

Hi!
Apparently cpython was compiled without zlib support, which is required by the requests library (probably the most used/famous http library in python).
% poetry run componentize-py --wit-path myapp/wit --world plugins componentize -p myapp main -o my-app.wasm
Traceback (most recent call last):
  File "/myapp/python/.venv/bin/componentize-py", line 8, in <module>
    sys.exit(script())
             ^^^^^^^^
AssertionError: Traceback (most recent call last):
  File "/0/main.py", line 2, in <module>
    import requests
  File "/1/requests/__init__.py", line 43, in <module>
    import urllib3
  File "/1/urllib3/__init__.py", line 18, in <module>
    from .connectionpool import HTTPConnectionPool, HTTPSConnectionPool, connection_from_url
  File "/1/urllib3/connectionpool.py", line 15, in <module>
    from ._request_methods import RequestMethods
  File "/1/urllib3/_request_methods.py", line 10, in <module>
    from .response import BaseHTTPResponse
  File "/1/urllib3/response.py", line 11, in <module>
    import zlib
ModuleNotFoundError: No module named 'zlib'

Caused by:
    ModuleNotFoundError: No module named 'zlib'

Issue probably linked to https://github.com/bytecodealliance/componentize-py/issues/64

I believe @brettcannon is planning to add zlib to the WASI build of CPython, along with other native modules such as bzip2, etc.
Regarding requests: I agree that this would be a great library to support, but it's going to take more than adding zlib to support it.  I assume it also uses the ssl module, which is going to be way tougher to support given that Wasm doesn't currently have constant time operations, which means OpenSSL is vulnerable to timing attacks when compiled to Wasm.  I think the most practical way to support requests is to port it to use wasi:http instead of sockets on WASIp2, which someone would need to volunteer to do.

I believe @brettcannon is planning to add zlib to the WASI build of CPython, along with other native modules such as bzip2, etc.

Correct, it's planned: https://opensource.snarky.ca/Python/WASI#Plans .

@brettcannon @dicej do you have any ETA/objective for this fix?

@c-x No ETA from me. The objective is to get zlib statically compiled into the binary I build for CPython via https://github.com/python/cpython/blob/main/Tools/wasm/wasi.py .


PHOENIX-1798 change Collections.empytList() to Lists.newArrayList()

This change related to PHOENIX-1798.
If the select SQL query a lot of data, the issue will occur.

This has been committed as PHOENIX-1798


Android - Update heartbeat model to use customer-provided unique ID

https://github.com/getditto/DittoAndroidTools/pull/48


feat: update color scheme of the notelist panel for the light theme

Description

Browser checklist
This PR has been tested in the following browsers:

[ ] Chrome
[ ] Firefox
[ ] Safari

Testing checklist

[ ] End-to-end tests have been created if necessary

#70  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @kleva-j and the rest of your teammates on  Graphite


Feature: New Tours Registration

As a new user of the New Tours website,
I want to be able to register,
so that I can store my personal information.
Scenario: 20 Successful Registration
Given I am on the New Tours Login Page
When I click the Register Menu Link
And I am redirected to the New Tours Registration Page
And I enter Bob in the First Name Field
And I enter Smith in the Last Name Field
And I enter bobsmith in the Username Field
And I enter passw0rd in the Password Field
And I enter passw0rd in the Confirm Password Field
And I click the Submit Button
Then I am redirected to the New Tours Registration Confirmation Page
And I verify the Registration Header exists

Child issue: #20

Part of an example project for showing how you should create stories and tasks to track your automated tests. Using Sentinel successfully is part of a larger process in which tests should be tied to tickets, which should be tied to code commits.


Response Related Models, Views, Serializers, URLs

Based on these tables from the schema:
Response, reflections taken, actions taken, conversations had, student times

Models and view sets are implemented for these tables.


[Rule] *** Tests / Tasks *** should not be defined in resource file

Right now we are recognizing file type in this code:
https://github.com/MarketSquare/robotframework-robocop/blob/af807515d7f160f6673f726f3b7cd33bca092a21/robocop/run.py#L93
(basically, we check if the file imports something, and if we're importing file that's being scanned by us - it's set as resource type).
This approach have some drawbacks, namely we cannot recognize resource files if we're not importing them in any scanned file. For example:
robocop file.resource

It will never be imported by other file (because we're only scanning one).
We have also proposed renaming files without *** Tests *** / *** Tasks ***  to .resource extension with newly added "can-be-resource-file".
I propose to add bit of reverse rule - if the file have .resource extension and is not recognized as resource file by our method, we should report if it contains  *** Tests *** / *** Tasks ***  sections.

@bhirsz I think this is a duplicate of #525. They were created on the same day, but I can't understand what's the difference between them. Maybe you remember?

I think the issue is that we already report this as parsing-error because Robot Framework detect it. If we want separate rule we can just catch it and raise as different rule. But we only need the other issue, not this one

But we only need the other issue, not this one

You mean that we need the issue from #525, and this can be closed?

Yes, exactly. Closing this issue


@numba.jit treats float as integer

The problem is best described by the following script.
from numba import jit
@jit(nopython=True)
def f(i):
    return 1 if i else 0
print(f(0))
print(f(0.5))
print(f(1))
@jit(nopython=True)
def f(i):
    return 1 if i else 0
print(f(0.5))
print(f(0))
print(f(1))
print(f(0.5))

The expected output is
0
1
1
1
0
1
1

but the actual output is
0
0
1
1
0
1
1

So it seems that when an integer argument is passed first, numba uses that version and never creates a new version for float input.
This behave correctly if nopython=True is made impossible (e.g. removing the nopython=True and add a print in the function f) or if the signature 'i8(f8)' is added to the decorator.
System info:
ArchLinux ([testing] enabled)
python: 3.4
llvm: 3.4
python-meta: (38 commits after 0.1.2) @ g6136057
python-llvm: (3 commits after 0.12.5 on llvm-3.4 branch) @ g7af2f71
python-llvmmath: (3 commits after 0.1.2) @ g0f4747c
python-numba: current master, 0.13.1

This bug seems to have been fixed in 0.20.0

Yes, it is fixed indeed.


Bump ASM

This should resolve https://github.com/architectury/architectury-transformer/issues/20
https://asm.ow2.io/versions.html

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


Balder/group commits to tls 2 remaining rebased 3

@toregge PR
As a base for discussion. Tests testing visibility delay are failing for now.

Part 1 in #14469


Fixed undefined of shouldUseStrictMode expo bared

When install the library to Expo Bared project, I got the error Cannot read properties of undefined (reading 'shouldUseStrictMode'). which the property shouldUseStrictMode is getting from app.json but it's not found on bared project.
Note: The project I'm working with is using Expo SDK 42

@amit-kremer93 when will this be merged into master? I'm getting the same error @xubmuajkub described when using AppsFlyer in a typescript project. Making the change @xubmuajkub suggested resolved the issue from what I can tell.

Update to the comment above:

the issue I was having was a result of how the plugin is expected to be added. See [here] (https://github.com/AppsFlyerSDK/appsflyer-react-native-plugin/blob/master/Docs/Expo.md#full-appjson-example).

...
"plugins": [
[
"react-native-appsflyer",{}
]
],
...

It expects an array with some objected.
I was able to get my project to work without the change mentioned here. Note, the issue might still be valid for Bared projects.

@barbarosaffan I'm no longer having this issue after I update the app.json as I mentioned in my comment above.
Are you getting an error message? If so, please share and I can try to see if I can help.

Im closing this MR. if you still facing this issue, please let us know.

I still have same issue with Expo 47 and react-native-appsflyer: 6.10.3
Fix from @xubmuajkub is working but not in eas build there I'm still getting error: "Cannot read properties of undefined (reading 'shouldUseStrictMode')"
Here is my app.json:
"plugins": [       [         "react-native-appsflyer",{}       ],   ]
Do you have any suggestions what I can do?


Update to UtiltiyNetwork ADE schema

The UtilityNetwork ADE schema has been updated, most of the changes appear to be to the "Network Components" package.
Giorgio, did you use any kind of software to generate the database schema, or did you do all this by hand?

The new UtilityNetwork ADE version should actually contain (for the major part) suggested improvements and minor bugfixes that I already implemented in the current version.
The database schema was derived manually.
As per 3DCityDB 4.0, you may want to give a try at the ADE manager plugin, which derives a database schema automatically from the xsd.
My implementation is still for version 3.3.1.


Cannot make sphinx-multiversion work when using nbsphinx

First of all, thanks for this awesome package! Works great!!
I have a few setups where I use the nbsphinx extension but when I call sphinx-multiversion the notebooks are ignored. Not sure if I'm doing something wrong or if sphinx-multiversion is not designed to work with other sphinx extensions.

Hi, can you provide us with a reprex (reproducing example) or if possible a MCVE?

Sorry, I moved to another setup 2 years ago and I don't even remember what I had before. Thank you for taking a look at this issue! I'll probably try multiversion if I have to set a new project up at some point.

Oh, no problem. Thanks for the quick response!


Increase ddev-webserver apache/nginx limit request field size for large cookie payloads

The Problem/Issue/Bug:
Some users have run into trouble with huge cookie payloads, both in nginx and apache. This doubles the relevant nginx default value is "large_client_header_buffers 4 8k;". This raises the size of the nginx buffers to "large_client_header_buffers 4 16k;"
Apache LimitRequestFieldSize  is doubled to 16380.

@TravisWhitehead would love it if you could review this, thanks.


Updated for use in SMF2.1

Here you go, it is all hooks.
There are two language files: english and dutch.
I dit not touch the 1.1 and 2.0 parts, just added the 2.1 stuff so it installs.
Note: I have added some text strings and also used slightly different names for the database settings compared to 1.1 and 2.0...

This is mega cool, BUT, on some files (and I can't understand the criteria here), it fails with undefined index (general one, without a variable) at line 165. Upon inspecting the browser console, the images that don't load tries to load like so: "https://domain.com/post/arrow.png", whereas it should be loading like so "https://domain.com/totallyatestforum/Themes/NameX_v1.0.5/images/post/arrow.png"
All the other ones do work with the correct path, been trying to figure out what is going on but haven't managed to do so, I might be missing something somewhere. Any ideas? Thanks!

No clue what issue that is: the code work prefectly on my site...

And I can just confirm it doesn't work for all images instead... Some work and some don't, the ones that were converted from gif to png don't work at all in particular.
Changing line 165 from:
$settings[$context['icon_sources'][$row['first_icon']]] . '/post/' . $row['first_icon'] . '.png',
to:
'icon_url' => $settings['theme_url'] . '/images/post/' . $row['first_icon'] . '.png',
seems to have fixed the issue for all images and removed the weird redirect.

@Bopske I now have time and the motivation for this. I'll be pushing my own commits to this/your branch to make this more similar to the existing mod. Okay by you?

@Bopske I now have time and the motivation for this. I'll be pushing my own commits to this/your branch to make this more similar to the existing mod. Okay by you?

Of course, I contributed my work, as far as I am involved, it is up to you to bring it forward...

And I can just confirm it doesn't work for all images instead... Some work and some don't, the ones that were converted from gif to png don't work at all in particular. Changing line 165 from: 'icon_url' => $settings[$context['icon_sources'][$row['first_icon']]] . '/post/' . $row['first_icon'] . '.png', to: 'icon_url' => $settings['theme_url'] . '/images/post/' . $row['first_icon'] . '.png', seems to have fixed the issue for all images and removed the weird redirect.
I can't understand why it happens, at all. The images that don't work are all custom ones but they're alongside the default ones and also present in the default theme. I even enabled the default theme image check in message icon settings, not a factor either.

I think I see the issue: the current code only works for SMF default icons, not for icons you have added manually. I do not have any custom topic icons.

Ok, just added a fix for the issue with custom post icons. Try the new Recent_top.php code: that should fix it

I see you added the check, whereas my approach was more brutal :P
It does indeed work as well, thank you!

IMO it is better to follow the approach that SMF code does. With this code also the functionality of checking theme vs default theme works as standard

Hello. How do I change the position of the Timeline button? I would like to put it before Members, although I have a Gallery button here, I would actually like to have a Timeline button placed in front of the Gallery.

finally got around to this today. Might be nice to eventually have some javascript to update relative times.

So will you be merging this?

Thank you!


Update prometheus and cortex

[ ] I added CHANGELOG entry for this change.
[X] Change is not relevant to the end user.

Changes
This updates the vendored Prometheus and Cortex versions. In the latest version of Prometheus the Querier's .LabelValues() call now takes a matchers parameter, so this also adds that parameter to the storepb.LabelValuesRequest type.

LGTM when fixed. Would you have time to address those comments? I can then release Thanos 0.19 with this

Great! yes I'll make the requested changes, thanks to both of you for the quick feedback.

I have addressed the comments, please take another look