Issue In Card App

 ══╡ EXCEPTION CAUGHT BY RENDERING LIBRARY ╞═════════════════════════════════════════════════════════
I/flutter ( 1705): The following message was thrown during layout:
I/flutter ( 1705): A RenderFlex overflowed by 4668 pixels on the bottom.
I/flutter ( 1705): The overflowing RenderFlex has an orientation of Axis.vertical.
I/flutter ( 1705): The edge of the RenderFlex that is overflowing has been marked in the rendering with a yellow and
I/flutter ( 1705): black striped pattern. This is usually caused by the contents being too big for the RenderFlex.
I/flutter ( 1705): Consider applying a flex factor (e.g. using an Expanded widget) to force the children of the
I/flutter ( 1705): RenderFlex to fit within the available space instead of being sized to their natural size.
I/flutter ( 1705): This is considered an error condition because it indicates that there is content that cannot be
I/flutter ( 1705): seen. If the content is legitimately bigger than the available space, consider clipping it with a
I/flutter ( 1705): ClipRect widget before putting it in the flex, or using a scrollable container rather than a Flex,
I/flutter ( 1705): like a ListView.
I/flutter ( 1705): The specific RenderFlex in question is:
I/flutter ( 1705):   RenderFlex#2e63d OVERFLOWING
I/flutter ( 1705):   creator: Column ← RepaintBoundary-[<0>] ← NotificationListener<KeepAliveNotification> ← KeepAlive
I/flutter ( 1705):   ← AutomaticKeepAlive ← SliverFillViewport ← Viewport ← _ScrollableScope ←
I/flutter ( 1705):   IgnorePointer-[GlobalKey#1b5c0] ← Semantics ← Listener ← _GestureSemantics ← ⋯
I/flutter ( 1705):   parentData: <none> (can use size)
I/flutter ( 1705):   constraints: BoxConstraints(w=360.0, h=512.0)
I/flutter ( 1705):   size: Size(360.0, 512.0)
I/flutter ( 1705):   direction: vertical
I/flutter ( 1705):   mainAxisAlignment: start
I/flutter ( 1705):   mainAxisSize: max
I/flutter ( 1705):   crossAxisAlignment: center
I/flutter ( 1705):   verticalDirection: down

Unfortunately this is not enough information to diagnose the problem.
Please add the output of flutter doctor -v
and a minimal reproduction.

[✓] Flutter (Channel dev, v0.9.7-pre.25, on Mac OS X 10.13.6 17G65, locale en-IN)
• Flutter version 0.9.7-pre.25 at /Users/Mac/Documents/flutter
• Framework revision a68b03e9ce (2 weeks ago), 2018-10-03 17:37:00 -0700
• Engine revision 74662ab695
• Dart version 2.1.0-dev.6.0.flutter-d2c5a24fd9
[!] Android toolchain - develop for Android devices (Android SDK 28.0.3)
• Android SDK at /Users/Mac/Library/Android/sdk
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-28, build-tools 28.0.3
• Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b06)
! Some Android licenses not accepted.  To resolve this, run: flutter doctor --android-licenses
[!] iOS toolchain - develop for iOS devices
✗ Xcode installation is incomplete; a full installation is necessary for iOS development.
Download at: https://developer.apple.com/xcode/download/
Or install Xcode via the App Store.
Once installed, run:
sudo xcode-select --switch /Applications/Xcode.app/Contents/Developer
✗ libimobiledevice and ideviceinstaller are not installed. To install, run:
brew install --HEAD libimobiledevice
brew install ideviceinstaller
✗ ios-deploy not installed. To install:
brew install ios-deploy
✗ CocoaPods not installed.
CocoaPods is used to retrieve the iOS platform side's plugin code that responds to your plugin usage on the Dart side.
Without resolving iOS dependencies with CocoaPods, plugins will not work on iOS.
For more info, see https://flutter.io/platform-plugins
To install:
brew install cocoapods
pod setup
[✓] Android Studio (version 3.2)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin version 29.0.2
• Dart plugin version 181.5540.11
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b06)
[✓] Connected device (1 available)
• Google Nexus 5, 6 0 0, API 23, 1080x1920 • <IP_ADDRESS>:5555 • android-x86 • Android 6.0 (API 23)
! Doctor found issues in 2 categories.

Fixed


The system variable PATH is not parsed by the agent if the length is >= 1024

Agent Version and Platform
Agent.OS | Windows_NT
Agent.OSArchitecture | X64
Agent.OSVersion | 6.3.9600
Agent.Version | 2.144.2
OS of the machine running the agent?
Windows 2012R2 Standard
Azure DevOps Type and Version
On-premises Azure DevOps Server 2019
What's not working?
The system variable PATH is not parsed by the agent if the length is >= 1024
The PATH variable may therefor not be available after installing new software that modifies the PATH variable (such as dotnet core). Leading to unexpected results in builds.
See: https://github.com/microsoft/azure-pipelines-agent/blob/c113d89607308107d663e3ff288be88faa66ec6b/src/Microsoft.VisualStudio.Services.Agent/Capabilities/EnvironmentCapabilitiesProvider.cs#L69
Agent and Worker's Diagnostic Logs
INFO EnvironmentCapabilitiesProvider] Skipping: 'Path'
Expected behavior
At a minimum the INFO trace should mention the 1024 limit on the length of the environment variable.
Whatever the length is, I would expect the system PATH environment variable to be used by the agent.
Steps to reproduce
Make sure your PATH variable is < 1024
Place an executable file in the path (for example echo echo testmessage > test.bat)
Create a new build
Add a commandline script task executing test.bat
Notice the build completes correctly
Make your PATH variable >= 1024
Restart the agent
Notice that PATH is no longer listed under 'capabilities'
Run the build and notice that it can no longer find the test.bat executable
'test.bat' is not recognized as an internal or external command, operable program or batch file.
This behavior is not limited to the commandline script task.

It appears a "PATH" variable was set in a variable group, during the build this is translated to an environment variable which overrides the system variable. Possibly overriding a system variable could have raised a warning.
The agent logging the "PATH" as skipped appears to have been no more than a distraction.


Interactive snapshot update issue with nested sibling describe blocks

Following on from #6611, #6485 and the fix for that issue in #6579. Probably not directly related to that issue.
🐛 Bug Report
Minimal example structure:

describe

describe

test containing .toMatchSnapshot()

describe

test containing .toMatchSnapshot()

When entering interactive snapshot update mode in Jest 23.3.0, the prompt for the first failed snapshot says that all the other snapshots (for other tests) are "obsolete" and that they will be removed. Example output from my actual project:
Snapshot Summary
 › 1 snapshot failed from 1 test suite. Inspect your code changes or press `u` to update them.
 › 6 snapshots obsolete from 1 test suite. To remove them all, press `u`.

...

Interactive Snapshot Progress
 › 4 snapshots remaining

Upon pressing u that's what happens: the one snapshot gets updated but the rest get deleted (including two that passed). Then when it moves onto the next failing snapshot, it says that no previous snapshot exists (since it was just deleted) and that the the one that was just updated is obsolete. Pressing u again does more or less the same thing: the current snapshot gets written, and the one that was updated just before is deleted. In effect it's only allowing allowing the storage of a single snapshot when going through interactive mode.
Doing a standard snapshot update with u/-u works as expected, it writes all the snapshots.
Link to repl or repo (highly encouraged)
https://github.com/noinkling/jest-bug-repro
With Jest 23.3.0 (can npm install), run npm test or jest --watchAll directly, then press i to enter interactive mode.
Run npx envinfo --preset jest
  System:
    OS: Linux 4.4 Ubuntu 16.04.4 LTS (Xenial Xerus)
    CPU: x64 Intel(R) Core(TM)2 Quad CPU    Q9400  @ 2.66GHz
  Binaries:
    Node: 10.5.0 - /usr/bin/node
    Yarn: 1.7.0 - /usr/bin/yarn
    npm: 6.1.0 - /usr/bin/npm

I'm using WSL.

This behaviour was introduced in 23.0.0
Before that (v22.4.2) all 3 snapshots in the linked example repo were grouped together (and therefore updated together).

Probably broken by https://github.com/facebook/jest/pull/6579

@rickhanlonii Sorry I should have made it more clear: this issue also exists in 23.0.0, so I don't think it's directly related to #6579.

Ah, ok thanks 👍

This issue seems to be fixed in the latest version (23.4.0), no issues with "obsolete" snapshots any more.
However the overall process is a little slow if there are more than a couple of snapshots to go through, which seems to be due to reinitializing(?) the suite for each snapshot. Is this intended?

I'll close this as I think the last point about the slowness is just a consequence of design, and it's not really related to the original issue.


HADOOP-17426. Upgrade to hadoop-thirdparty-1.1.0.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 57s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
36m 41s

trunk passed

+1 :green_heart:
compile
0m 17s

trunk passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
compile
0m 17s

trunk passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
mvnsite
0m 21s

trunk passed

+1 :green_heart:
javadoc
0m 20s

trunk passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javadoc
0m 18s

trunk passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
shadedclient
54m 12s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 11s

the patch passed

+1 :green_heart:
compile
0m 11s

the patch passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javac
0m 11s

the patch passed

+1 :green_heart:
compile
0m 10s

the patch passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
javac
0m 10s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
mvnsite
0m 13s

the patch passed

+1 :green_heart:
xml
0m  1s

The patch has no ill-formed XML file.

+1 :green_heart:
javadoc
0m 11s

the patch passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javadoc
0m 11s

the patch passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
shadedclient
16m 59s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
0m 14s

hadoop-project in the patch passed.

+1 :green_heart:
asflicense
0m 29s

The patch does not generate ASF License warnings.

75m 27s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3024/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/3024

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient codespell xml

uname
Linux 2eb679373115 4.15.0-101-generic #102-Ubuntu SMP Mon May 11 10:07:26 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / fa72b8a654a5eafce1164e2b09cfd953f272e6e9

Default Java
Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3024/1/testReport/

Max. process+thread count
601 (vs. ulimit of 5500)

modules
C: hadoop-project U: hadoop-project

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3024/1/console

versions
git=2.25.1 maven=3.6.3

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

Merged. Thank you @jojochuang


Settings table cleanup

Our settings table needs a little bit of cleanup.
The "blog" settings type was meant to be for data that can safely be exposed, however it has been misused. Slack and unsplash shouldn't use this type, instead they should be set to something like secret or credentials.
The name "blog" is also now out of date, we should instead use site.
The old force_i18n & permalinks fields should also be removed.
The setting active_timezone should be just timezone.
The setting default_locale should be just lang.
Some of these changes are already being managed through internal code. As part of work to add a new Settings API endpoint to the Content API, I need to make these changes more official, hence this issue.
Many of these changes can't be made easily, and will require us to do it as part of a migration, possibly in Ghost 3.0.  However, the changes can be affected through the v2 APIs, so that we are already working with the correct data.
The new Settings API Endpoint will expose "public settings" which are a few fields that are whitelisted - roughly the same settings already exposed as @blog in the theme layer (to be renamed {{@site}}).
The plan is to centralise this concept, expose these settings via the API and via the theme middleware.

[ ] slack and unsplash need moving to a new type, e.g. credentials
[ ] switch  the "type" blog to be called site in settings
[ ] get rid of force_i18n and permalinks
[ ] rename default_locale to lang and active_timezone to timezone

This also affects the theme layer slightly

[ ] deprecate {{@blog}} in favour of {{@site}}
[ ] deprecate the {{lang}} helper in favour of<EMAIL_ADDRESS>

Not stale

slack and unsplash need moving to a new type, e.g. credentials

Does credentials make sense for Unsplash? There are no credentials stored for Unsplash, the only setting used is a toggle to enable/disable the Unsplash image selector in Ghost-Admin.


Login Through OSU SSO

See https://github.com/osumb/challenges/pull/31 as to why we're not doing this anymore


Add e-connect elmo alarm integration

Breaking change

Proposed change

Type of change

[ ] Dependency upgrade
[ ] Bugfix (non-breaking change which fixes an issue)
[X] New integration (thank you!)
[ ] New feature (which adds functionality to an existing integration)
[ ] Breaking change (fix/feature causing existing functionality to break)
[ ] Code quality improvements to existing code or addition of tests

Example entry for configuration.yaml:

# Example configuration.yaml
### Sample configuration:
```yaml

elmo_alarm:
  host: 'https://connect.elmospa.com'
  vendor: '<vendor>'
  username: '<username>'
  password: '<password>'
  scan_interval: 5
  states:
    - name: 'arm_home'
      zones: [1]
    - name: 'arm_away'
      zones: [1, 2, 3, 4]
    - name: 'arm_night'
      zones: [1, 2, 4]
    - name: 'arm_custom_bypass'
      zones: [3]

Additional information

This PR fixes or closes issue: fixes #
This PR is related to issue:
Link to documentation pull request:

Checklist

[X] The code change is tested and works locally.
[X] Local tests pass. Your PR cannot be merged unless tests pass
[X] There is no commented out code in this PR.
[X] I have followed the development checklist
[X] The code has been formatted using Black (black --fast homeassistant tests)
[X] Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated for www.home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[X] The manifest file has all fields filled out correctly.
Updated and included derived files by running: python3 -m script.hassfest.
[X] New or updated dependencies have been added to requirements_all.txt.
Updated by running python3 -m script.gen_requirements_all.
[X] Untested files have been added to .coveragerc.

The integration reached or maintains the following Integration Quality Scale:

[ ] No score or internal
[ ] 🥈 Silver
[ ] 🥇 Gold
[ ] 🏆 Platinum

Hi @markin,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

Hi @markin,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

@markin please add tests to ensure that your code works :)

@markin please add tests to ensure that your code works :)

There is documentation on how to add tests?

Unfortunately, our documentation does not include that. However, I would advise you to take a look at other integrations and mimic their tests and test structure.
Here is the Home Assistant Dev documentation on how to run your tests with tox. Hopefully this article on getting started with Testing in python can help you.


Use gcr.io/google_containers/pause:3.0.

Now we support schema 1 image, we could use official pause image now.
Signed-off-by: Lantao Liu<EMAIL_ADDRESS>

CRI Validation test result:
Summarizing 5 Failures:

[Fail] [k8s.io] Streaming runtime should support streaming interfaces [It] runtime should support attach [Conformance]
/home/lantaol/workspace/src/github.com/kubernetes-incubator/cri-tools/pkg/validate/streaming.go:183

[Fail] [k8s.io] Streaming runtime should support streaming interfaces [It] runtime should support portforward [Conformance]
/home/lantaol/workspace/src/github.com/kubernetes-incubator/cri-tools/pkg/validate/streaming.go:231

[Fail] [k8s.io] Security Context runtime should support container with security context [It] runtime should support RunAsUser [security context]
/home/lantaol/workspace/src/github.com/kubernetes-incubator/cri-tools/pkg/validate/container.go:326

[Fail] [k8s.io] Streaming runtime should support streaming interfaces [It] runtime should support exec [Conformance]
/home/lantaol/workspace/src/github.com/kubernetes-incubator/cri-tools/pkg/validate/streaming.go:127

[Fail] [k8s.io] Security Context runtime should support container with security context [It] runtime should support RunAsUserName [security context]
/home/lantaol/workspace/src/github.com/kubernetes-incubator/cri-tools/pkg/validate/container.go:326

Ran 35 of 36 Specs in 29.342 seconds
FAIL! -- 30 Passed | 5 Failed | 0 Pending | 1 Skipped --- FAIL: TestE2ECRI (29.34s)
FAIL

Ginkgo ran 1 suite in 29.409289447s
Test Suite Failed
exit status 1


DOCS: correct changes in syntax, remove out-of-date information

This PR is not meant to add anything new or to make the docs more complete; Instead it brings everything up to date, corrects the docs in regards to changes in onyo (e.g. onyo new --path gets a directory, not an asset name) or removes information that is simply out of date (e.g. "_" is not a reserved character anymore and can be used in keys of asset names).
This should allow other people to focus later on adding new information and help during the hackathon to either use the documentation or see that information is just not given yet, instead of wondering permanently if the user does something wrong or if the docs might just be incorrect.

Thanks for the quick feedback!
I saw it on Friday and it seemed to be because of a header/link. I will check that today, and then let you immediatly know :)

I updated this PR to fix the build-problem, so feel free to review when you have time.

Thanks!


Use prefixed node: APIs

fixes #113

:+1: Also, one thing we could do is to add a Biome rule to avoid mistakenly omitting the node: prefix. WDYT?

Very nice.
Running $ biome lint --write . doesn't fix the imports that are missing the node: prefix, can we make Biome autofix this as well? It says Unsafe fix so maybe that's why, and maybe we need to tell Biome to apply its autofix despite being "unsafe".
Biome doesn't complain about dynamic imports such as import('path'), is there a way to make it complain about dynamic imports as well?

The arg --unsafe does the trick.

Closing in favor of https://github.com/brillout/telefunc/pull/115.


cephadm adopt - broken on EL7

Hi,
trying to migrate my ceph pacific cluster (containerized, EL7) to cephadm via adopt playbook. ceph-ansible tries to install cephadm via yum but there are no pacific builds for EL7 (only EL8). ceph-ansible should install cephadm directly from github via curl:
https://chedaogou.site/cephadm/install/index.html#cephadm-install-curl
TASK [install cephadm requirements] ****************************************************************************************************************************************************************************************************************************************************************************************************************************************
Monday 02 May 2022  12:37:29 +0200 (0:00:00.190)       0:00:26.041 ************
FAILED - RETRYING: install cephadm requirements (3 retries left).
FAILED - RETRYING: install cephadm requirements (3 retries left).
FAILED - RETRYING: install cephadm requirements (3 retries left).
FAILED - RETRYING: install cephadm requirements (3 retries left).
FAILED - RETRYING: install cephadm requirements (3 retries left).
FAILED - RETRYING: install cephadm requirements (3 retries left).
FAILED - RETRYING: install cephadm requirements (2 retries left).
FAILED - RETRYING: install cephadm requirements (2 retries left).
FAILED - RETRYING: install cephadm requirements (2 retries left).
FAILED - RETRYING: install cephadm requirements (2 retries left).
FAILED - RETRYING: install cephadm requirements (2 retries left).
FAILED - RETRYING: install cephadm requirements (2 retries left).
FAILED - RETRYING: install cephadm requirements (1 retries left).
FAILED - RETRYING: install cephadm requirements (1 retries left).
FAILED - RETRYING: install cephadm requirements (1 retries left).
FAILED - RETRYING: install cephadm requirements (1 retries left).
FAILED - RETRYING: install cephadm requirements (1 retries left).
FAILED - RETRYING: install cephadm requirements (1 retries left).
fatal: [ceph-mon02]: FAILED! => changed=false
  attempts: 3
  msg: |-
    Failure talking to yum: failure: repodata/repomd.xml from ceph_stable: [Errno 256] No more mirrors to try.
    http://download.ceph.com/rpm-pacific/el7/x86_64/repodata/repomd.xml: [Errno 14] HTTP Error 404 - Not Found

Regards,
Kilian

yeh, we could probably make the playbook download cephadm directly from github when the running OS is el7


Investigation on how to run: git cz

Goal
Be able to run git cz instead of cz c.
Description
I haven't researched much but I know it's possible to add some hook or plugin to git in order to add a new command. It's a nice to have. Maybe it could be part of cz init.

This only requires an executable named git-cz on the user's PATH.
$ cat git-cz
#!/usr/bin/env python
from __future__ import print_function

if __name__ == '__main__':
  print("hello world cz")

$ chmod +x git-cz
$ git cz
hello world cz

Awesome! Do you think it would be possible to add a command to generate this script?
I think your example can be added as documentation in the meanwhile.
Thanks!

If the existing commitizen.cli:main is appropriate to run, this is a one-line change to pyproject.toml. Added PR.


Progress Indicator for Input-Related Messages

Problem
The dismissing of messages related to health measurements and questionnaire responses is dependent on cloud functions that might take a few seconds to complete. Until then, the messages on the Home Screen are not dismissed but also don't show any indication that calculations are happening in the background.
Solution
Add some custom logic around the messages related to questionnaires & health measurements and display a progress indicator for about 1 minute after new values have been submitted.
Additional context
The implementation could be very simple, using a variable in the relevant messages manager that stores the last recorded health measurement & questionnaire. If the date is less than a minute away we show a progress indicator on the relevant messages & possible a small message indicating that the measurements are processed.
Code of Conduct

[X] I agree to follow this project's Code of Conduct and Contributing Guidelines

I'll take a look into this issue by tuesday and let you know any updates asap.

I took a look at this issue - I'd propose to introduce a new processing state to track submitted measurements and questionnaire responses, implement a simple one-minute display, and extend our Messages implementation to show a nice loading indicator. This solution aligns well with the design pattern using the nice capsulestyle elements we already have in MessageRow, and also integrates well with our existing NotificationManager for handling edge cases.
As I see right now it, the changes would have effects on three areas: extending Message model for processing state tracking, showing a loading UI in MessageRow, and adding proper progress states to our submission functions like addMeasurement and questionnaire responses.
Any comments on this approach or should I start building @PSchmiedmayer and we take a look at the result?


Improvements to the daemon

Fixes #949
Fixes #951

@cfergeau I think i addressed all your review comments, PTALA, Thanks!

With that said, it seems the worst that can happen is bad error messages (which may be handled in the tray already to show something better?)

Yes, if we get an error from the daemon that is handled in tray to show the status as unknown.

We should also document what we consider is acceptable to call concurrently, and what is not to make it easier to decide between "this is a bug we need to fix" and "this is not supported"
Which info do you need to get in your regular "status" calls?

For now we show the status in a window, so all the info that is part of crc status command is needed, later when we poll regularly only the VM status would be enough to change the color of the tray icon to indicate that a VM is running/stopped.

@cfergeau Updated the PR, PTALA


[COOK-3418] Don't store non temporary files in /tmp

https://tickets.opscode.com/browse/COOK-3418

Actually fixes COOK-1447
Doesn't store non temporary files inside of /tmp

No longer needed, the new python_package provider in poise-python relies on the pip to DTRT.


LR schedule for the fine-tune phase

Hi,
I have noticed that you use different lr schedules in the fine-tune phase.
For the COCO experiments, when using 30 shots, the total iterations is up to 240000, which is larger than the base phase(110000).
I wonder does the iterations matter a lot for the fine-tune stage, 240000 iterations are too expensive for the fine-tuning.

Hello,
The large number of fine-tuning iterations does not matter that much. For the 30 shot case, we observed the same level of performance using half as many (and potentially even fewer) iterations. I just evaluated the checkpoint at 48k iterations, and the model was able to achieve nAP 13.1 and nAP75 13.0. Also, we tried to make the learning schedule consistent with other experiments, so we did not tune the hyper parameters that much. A better learning schedule might require less fine-tuning iterations.
Hope that helps!


Add TE-2.1: gRIBI IPv4 Entry test

Resolves #98.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 72.677%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1314

Relevant Lines:
1808

💛  - Coveralls


Resolve (assigned) issues mentioned by PE Testers

List of assigned issues to fix (To be reviewed and pushed after all issues are fixed)

[x] #179 (Solution: Edit UG to include notice when adding additional tags)
[x] #182 (Solution: Edited text from "Address Book" to "BookFace")
[x] #188 (Solution: Add check in JsonSerializableBookFace.java to ensure unloaned books are not loaned and do not have return dates
[x] #190 (Solution: Fixed typo)
[x] #192 (Solution: Edit UG to include the change of list ordering in FAQ and Loan feature details)
[ ] #194 / #197
[x] #202 (Solution: Edit UG to include notice when entering an Author. This PR will also fix #174)

Codecov Report

Merging #207 (247fa03) into master (8ee7044) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master     #207      +/-   ##
============================================
- Coverage     53.85%   53.84%   -0.01%
- Complexity      484      485       +1
============================================
  Files           115      115
  Lines          2000     2002       +2
  Branches        202      203       +1
============================================
+ Hits           1077     1078       +1
+ Misses          850      849       -1
- Partials         73       75       +2

Impacted Files
Coverage Δ

src/main/java/bookface/MainApp.java
0.00% <ø> (ø)

src/main/java/bookface/model/book/Author.java
60.00% <ø> (ø)

...ava/bookface/storage/JsonSerializableBookFace.java
68.57% <0.00%> (-4.16%)
:arrow_down:

src/main/java/bookface/model/book/Book.java
31.11% <0.00%> (+2.22%)
:arrow_up:

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

For bug #194/#197, I have been trying to modify the label under a method which sets a wrapText and fixed maxWidth. However, this does not seem to completely fix the tag size; the width does not scale accordingly, and it does not wrap to the width of PersonCard


Better Mobile: Fixed Pkg

Better Mobile: Fixed Pkg

Azure Sentinel to Microsoft Sentinel
Required items, please complete
Change(s):

Azure Sentinel to Microsoft Sentinel
Reason for Change(s):

Azure Sentinel to Microsoft Sentinel
Version Updated:

N/A

Testing Completed:

N/A

Checked that the validations are passing and have addressed any issues that are present:

N/A

@v-dvedak
Fixed Better Mobile pkg 2.0.1 from "Azure Sentinel" to "Microsoft Sentinel"

@msft-sm, I see changes in 2.0.1. Please accommodate the changes for text Microsoft Sentinel in data connector here as well.

@v-dvedak
Fixed Better Mobile Data Connector from "Azure Sentinel" to "Microsoft Sentinel".


[Event Hubs Processor] June Release Prep

Summary
The focus of these changes is to prepare the Event Hubs processor and Functions extension packages for the June 2023 release.
NOTE
This PR cannot be merged until the June release of the core package is complete.

API change check
API changes are not detected in this pull request.

/check-enforcer override

Failure is unrelated to these changes.  Merging.


Accessibility of select lists with images

Images in the select lists complain about not having alt info because none is set. This is an easy one, it the alt info is set to null (as they are purely decorative -- http://webaim.org/techniques/alttext/#decorative ) then it'll be skipped by screen readers / be happy.
alt="" will fix this issue

Associated pull request #3493

PR was accepted so I'll close out :)


SonarCloud stopped working

SonarCloud has been reporting 0 coverage for all PRs in the past few months. It was working for a PR on 10/18/2023, but not for a PR on 12/7/2023. We need to investigate what changed in between.
In addition, the total coverage for the repo has dropped from 90% to 20% since the mono repo migration. Which is not true, we may need to tweak some settings to make it display the correct coverage for multi-module project.

I believe this should be resolved with this: https://github.com/googleapis/sdk-platform-java/pull/1872#issuecomment-2164243448 and the java.time PR being merged in.


:bug: Fix: bug with using buffer

Description
Concise description of proposed changes
Testing
Explain the quality checks that have been done on the code changes
Additional Information

[ ] I read the contributing docs (if this is your first contribution)

Your ENS/address:

#1227
#1226  👈
main

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

Join @roninjin10 and the rest of your teammates on  Graphite


fix: add missing brackets

Description
Add missing brackets causing an error when disabling a library in Settings > Jellyfin.
Screenshot (if UI-related)
To-Dos

[ ] Successful build pnpm build
[ ] Translation keys pnpm i18n:extract
[ ] Database migration (if required)

Issues Fixed or Closed

Fixes #XXXX

:tada: This PR is included in version 2.0.0 :tada:
The release is available on:

v2.0.0
GitHub release

Your semantic-release bot :package::rocket:


getting an association is returning Her::Model::Associations:: object rather than collection

class Company
  include Her::Model

  primary_key :company_number
  has_many :announcements
end

class MAP::Announcement
  include Her::Model

  primary_key :announcement_id
  belongs_to :company
end

company = Company.find('ABC')
company.announcements
=> #<Her::Model::Associations::HasManyAssociation:0x00000103e3f018 ...>

Gonna close this as a duplicate of #135. Thanks @chewi!


Change the way args are passed to modules

Fixes spring-cloud/spring-cloud-dataflow-admin-cloudfoundry#56
Might fix spring-cloud/spring-cloud-stream-modules#195 :)

LGTM, merging.

Rebased and merged as 5a5be0b

Also verified the changes by running a CF deployment successfully.


Fix mage tool and did some workflow improvements

Hi @Haishi2016, I am not able to merge the PR without write access. Can you merge?


Restorebits

Type of Change

[x] Bug fix (non-breaking change, fixes #2144, #2119, #2126 )
[ ] New feature (non-breaking change, adds functionality)
[ ] Breaking change (effects multiple commands or functionality)
[ ] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Pester test is included

Purpose
Fixing a couple of oddball occurences that weren't envisioned
Approach
Handles the situations correctly
Commands to test
See individual Issues

Thanks @Stuart-Moore ! Merging because failures were 100% DAC sucking.


Morning session | peer to peer code review

Requirements of contact form

contact form is in the section tag
Changes needed : need to change the section tag to footer tag of the contact form.

The social media links are not working. You should have to make sure you used a working link.

The site is well done, it is very hard to find an issue. But, see
. I think it would be better semantically to use a section tag on the div inside the li tag that contains h3 header.

.language class
consider screeshot below

Since the class .languages does not have responsibilities, it would be better if it was removed I think.

Thanks guys i've noted all of your review feedback and would definitely work on them )))


refactor: immutabledb -> ipfs

Description
Really minor, to increase code readability and resolve small point of confusion, opened pr instead of issue. Changes immutabledb to ipfs as the inline docs already specify and as it used elsewhere. If there was an intention for using  immutabledb, could open pr to change inline docs instead.

Thank you @zachferland! test/log.spec.js has been split into a few files. Do you mind rebasing on master? Lmk if I can help

thanks for the review @shamb0t, rebased it and fixed comment above

Thank you @zachferland!


Add Mailspring application

Hey folks! Just adding Mailspring (a fork of Nylas Mail with a new C++ sync engine) to the electron-apps list. Here's the metadata in link form:

URL: http://getmailspring.com
Screenshot<EMAIL_ADDRESS>(same as hero region on the site)

Cheers!

Nice. Thanks, @bengotow


Add @DataObject for generated POJOs

It is perfect to generate POJO with fomJson and toJson, but is it possible to add @DataObject to generated POJOs also? if so , we can use it in service proxies. refer to Service proxies doc, the asynchronous result can only be primitive/json/enum, and class annotated with @DataObject

This totally makes sense. Will see how fast I can add it.

On the other hand, I noticed that the generated DAO classes have methods like findManyByxxx/findOneByxxx which accept Condition parameter. But I don't know how to add order by segment and execute complex queries like join, could you please advide ?

I'd just add @DateObject-support for the POJOs for now.
Using DAOs via service proxies would be the next thing and out of the scope of this issue. What you can do for converting the Condition is to convert it into a String on one side and convert it back afterwards using DSL.condition(String) method.

you mean this ?
contentDao.findManyByCondition(DSL.condition(CONTENT.ID.gt(begin).toString() + "order by id desc limit 10"))

What I mean is, if you need to remotely call a function that takes a Condition as an argument you have to convert it into a string before. So let's assume there is a wrapper around the dao that replaces the methods that take a Condition, like findOneByCondition(Condition) with findOneByCondition(String). The wrapper would do something like this
public Future<P> findOneByCondition(String condition){
  return dao.findOneByCondition(DSL.condition(condition));
}

It was more work to add this annotation as expected. As described in https://github.com/jOOQ/jOOQ/issues/10355 I had to cook a lot copy pasta to achieve this functionality. I will remove the duplicate code once it is implemented.


Possible bug in alpha_bar

Thanks for all the work you shared.
I noticed the recent update in ddpm code and found possible bug:
https://github.com/lucidrains/denoising-diffusion-pytorch/blob/0b8cdb4c8bf96384ebf31a4fdc4fcb59a1242cb9/denoising_diffusion_pytorch/denoising_diffusion_pytorch.py#L321
The line was previously:
https://github.com/lucidrains/denoising-diffusion-pytorch/blob/f4615599bcbc884dcb7efa0dd70840f011a0fbb9/denoising_diffusion_pytorch/denoising_diffusion_pytorch.py#L337
From my understanding, padding should be applied to the left instead of right, which may require modification:
alphas_cumprod_prev = F.pad(alphas_cumprod[:-1], (1, 0), value = 1.)

to make them equivalent.

@jihoonerd oh yes :man_facepalming: thank you so much for catching this!


Is there a way to have TextField only insert the value of the 'data-value' attribute into the Input field?

I kind of want to search for stuff, display the id + some other data, but the only place an id in a text field
TextField seems put the entire text contents of my choice template into the text field.
I tried modifying my ModelForm to use the ChoiceWidget instead which almost works, except when editing a 'bound' form as the single value doesn't map to the ChoiceWidget correctly.

Duplicate of #338


registerReceiver only for the running app?

Is it possible to register the receiver to only get it access to SMS intended for the app specified in the SMS (hash)?

not sure i understand, maybe explain with a use-case? but if you are using the app hash the you should be using the SMS Retriever API which won't require any "Allow <my_app> to read message" permission so the os will route the SMS directly to the app so no other messages will be accessed.
consent api due to the nature of not having a hash there's a probability the SMS coming in isn't the otp that's why user is asked to give consent after verifying its the OTP message.

I was referring to the RECEIVER_NOT_EXORTED flag instead of RECEIVER_EXPORTED as discussed here: https://stackoverflow.com/a/77276774/299209
But now sure how relevant it is?

From the docs, they show case the receiver being exported (although registered statically). I assume they would want us to use the RECEIVER_EXPORTED flag when registering dynamically.

Understood. I have tested that when the SMS does not contain the hash, the app will not get access to it. So I'll close this as its working as designed. Thanks!


0.1.0-rc2 - AntiJank "issues"

Tested AntiJank from 0.1.0-rc2.
While this might be expected just wanted to share this nevertheless, using AntiJank breaks "Bannerlord Mod Launcher" with the following error/stacktrace:
System.Reflection.ReflectionTypeLoadException
HResult=0x80131602
Nachricht = Unable to load one or more of the requested types. Retrieve the LoaderExceptions property for more information.
Quelle = mscorlib
Stapelüberwachung:
at System.Reflection.RuntimeModule.GetTypes(RuntimeModule module)
at System.Reflection.Assembly.GetTypes()
at TaleWorlds.MountAndBlade.Module.CollectModuleAssemblyTypes(Assembly moduleAssembly)
at TaleWorlds.MountAndBlade.Module.LoadSubModules()
at TaleWorlds.MountAndBlade.Module.Initialize()
Native Launcher with AntiJank works fine ... problem is, that Bannerlord Mod Launcher is the only launcher at the moment which works perfectly with the mod "Party Screen Enhancements", especially the sorting functionality stops to work frequently.
This sorting functionality of PSE does not work consistently with the native launcher, native launcher+antijank, fixed Launcher. All with the same mod load order and PSE loaded last.
I have no clue what Bannerlord Mod Launcher does different than all the other ones.
https://github.com/tstavrianos/BannerLordLauncher
https://github.com/Hirtol/PartyScreenEnhancements

Yeah - got a fix for it sent to a few testers, testing for next RC is ongoing.

Yeah - got a fix for it sent to a few testers, testing for next RC is ongoing.


Jupyter Bug: causing Travis Python 3.6 build failure

prompt-toolkit requirement conflict is causing failure in Travis build for Python 3.6 due to recent update with Jupyter.  More info: https://github.com/jupyter/jupyter/issues/370
Potential solution: specify specific versions of Jupyter and prompt-toolkit?

Appears to be fixed with update of mapbox gl version.


Tech anomaly bugs

[x] It uses a return in the loop and multiple tech anomalies break one another.
[x] The timer has rounding problems and eats the remainder every time.
[ ] It's using the obsolete lookup versions and allocating out the ass.

it can link things to themselves. this causes an incredibly loud noise if it happens to doors.

Here to reiterate veritius's comment - causes lights to strobe extremely fast also.


Issue while running the simulator docker image

ISSUE TYPE

Other

COMPONENT NAME

Docker image - simulator

CLOUDSTACK VERSION

Master branch

CONFIGURATION

N/A
OS / ENVIRONMENT

N/A
SUMMARY

After cloning the repo, building the docker image from scratch and running it, the UI doesn't show up and throws the following error:
"You're in developer mode, please build and run UI using npm."
STEPS TO REPRODUCE

cd tools/docker
docker build -f Dockerfile -t cloudstack/simulator ../..
docker run --name simulator -p 8080:8080 -d cloudstack/simulator
docker exec -ti simulator python \
/root/tools/marvin/marvin/deployDataCenter.py -i /root/setup/dev/basic.cfg

EXPECTED RESULTS

Cloudstack UI

ACTUAL RESULTS

On the web browser:

"You're in developer mode, please build and run UI using npm."

Hello @srisgajare, can share some screenshots and show where exactly you're facing the issue?

@srisgajare After running the docker container.
Open another terminal and follow the steps:

git clone https://github.com/apache/cloudstack.git.
cd cloudstack\ui (If using windows).
then npm install.
and finally, npm serve.
and also make sure to set CS_URL=http://localhost:8080/client on .env.local file on ui.
Hopefully, you'll be able to run the management server on http://localhost:5050

Hi @srisgajare can you confirm if @apurv-1's workaround fixes the issue?

This is not an issue @srisgajare you're supposed to run the UI separately as developer. How are you building the docker image, one solution is to also build the UI using npm and copy and replace the src as Apurv mentioned above.

@nvazquez @sureshanaparti @davidjumani @shwstppr @Pearl1594 @DaanHoogland @weizhouapache - any update on this, who's assigned/working on this? This however, may not be a high-priority task - but let's discuss and conclude on a resolution, thanks.


"Control RADIUS clients that require MFA"

Is my assumption correct that if the extension is installed on a NPS server all the configured RADIUS clients on that server will require MFA. "Once you enable MFA for a RADIUS client using the NPS Extension" that is mentioned in the documentation  indicates that MFA can be enabled explicit for one RADIUS client.

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

ID: a61a7e9a-44bf-7bdc-f810-87e2fd6792e1
Version Independent ID: 71d11105-b83e-952e-d107-1ff9a7c8d107
Content: Use existing NPS servers to provide Azure MFA capabilities
Content Source: articles/active-directory/authentication/howto-mfa-nps-extension.md
Service: active-directory
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

@matthias-scharl
Thanks for your feedback! We will investigate and update as appropriate.

No that's not the case. When you are authenticating through the RADIUS client the authentication still happens in the cloud, and you enable or disable MFA for individual users in the Azure portal. You can do by going to the Users section in your Azure Active Directory, or by going to aka.ms/mfasetup

Here is a workflow diagram that shows how this authentication works. This is the same whether you are using RDP or VPN:

I think we do not have the same understanding of the term "RADIUS client". The "RADIUS client" is not the end user client device. It´s the device, typically a switch, a Wireless Controller or a VPN gateway that forwards the RADIUS request from the client device to the RADIUS server (NPS). Based on the above diagram the RADIUS client is the NAS / VPN server. This matches also the terminology on NPS.

So let´s assume we have several RADIUS clients defined. Is it possible to enable the MFA extension for one RADIUS client only or is all traffic that is sent to the RADIUS server redirected to Azure MFA ?

Hi @matthias-scharl ,
Sorry, I misunderstood the question initially. I'm looking into this with an engineer on our team.

All traffic is redirected to Azure MFA, so you cannot set it up per client, unfortunately.

However, if you would like to configure some clients for normal AD and some for MFA, this article provides a workaround that will allow you to tweak the NPS so it will allow authentication for either: http://azuredummies.com/2018/01/01/secure-azure-gateway-radius-authentication-with-azure-mfa-nps-extension/ . The tweak is a registry edit towards the end of the article “Prepare for users that aren’t enrolled for MFA”.

There is a registry key "IP_WHITELIST", I think it could be used in your situation.
https://docs.microsoft.com/en-us/azure/active-directory/authentication/howto-mfa-nps-extension-advanced#ip-exceptions

Many of the suggestions here are just bad.
@MarileeTurscak-MSFT proposes registry to change so this is not required for users, and totally ignoring that the question was per RADIUS client.
As an example we have the same user Adam, which is enrolled in MFA, From home, he connects to VPN, and should need to perform MFA auth as well.
However at the office, he connects to the wifi and SHOULD NOT need to do this.
This should be seen as a critical bug in the extension.
@MoHelal Suggests the whitelist, which is a hack at best.
Where is the "only use MFA for this specific auth" I should not need to whitelist a few hundred IP addresses

Having the same issue here. I even tried using the "IP_WHITELIST". This would have worked perfectly for me, but MICROSOFT's own RRAS service doesn't submit the required "NasIpAddress" RADIUS attribute, so the whole setting is ignored. Horrible oversight.

@caspChristian agree 100%
Unfortunately, until now it seems like that there is no solution for this problem. The MFA extension looks half-baked and Microsoft doesn't really care...


Easy SCImageView Implementation resulting in black screen Swift

Really simple implementation using your tutorial results in a black screen.
self.moviePlayer = SCPlayer()
self.moviePlayer.setItemByAsset(recordSession.assetRepresentingSegments())
var playerLayer = AVPlayerLayer(player: moviePlayer)
playerLayer.frame.origin = CGPoint(x: 0,y: 0)
playerLayer.frame.size = self.mainview.frame.size
self.mainview.layer.addSublayer(playerLayer)
moviePlayer.loopEnabled = true
moviePlayer.play()

This works perfectly fine but once I add
var a = SCImageView(frame: CGRect(origin: CGPoint(x: 0, y: 0), size: self.mainview.frame.size))
a.filter = SCFilter(CIFilter: CIFilter(name: "CIPhotoEffectInstant"))
moviePlayer.CIImageRenderer = a
self.mainview.addSubview(a)

the mainview turns black.
Any Idea?

Also, the AVVideoPlayerView way just plainly doesn't work.

Did you try with SCVideoPlayerView to render the filters through SCImageView as the tutorial suggest?

Yes, SCVideoPlayerView isn't working for me even without the filter. I meant SCVideoPlayerView in my comment that says AVVideoPlayerView

I used this code for the SCVideoPlayerView path
var playerView = SCVideoPlayerView(frame: CGRect(x: 0, y: 50, width: UIScreen.mainScreen().bounds.width, height: UIScreen.mainScreen().bounds.width))
        self.view.addSubview(playerView)
        self.moviePlayer = playerView.player
        playerView.player?.setItemByAsset(recordSession.assetRepresentingSegments())
        playerView.player?.loopEnabled = true
        playerView.player?.play()

This results in the playerView not being seen.
In the view hierarchy debugger it shows the playerView as an empty SCVideoPlayerView

Im relative new to coding, but since the code is working perfectly well, I think the mess up is either in your views set up or , as I can think of now (and like I did before), there is no input to render,
it's hard to figure out since I cant see the viewcontrollers, but -->
with the small information I have, did you prepare the recordSession  in prepareforsegue in the RecordViewcontroller before going into the PlayerViewcontroller after recording finished?
Because if you didnt, this will result in black screen with nothign to render since there are no sessions loaded. :)

the sessions are definitely loaded because the scplayer + scplayer layer path shows the video perfectly. It's just that when I try to add the filter ( the 4 lines of code in my initial question) nothing is shown.

Yes, but if you try to export when nothing is shown, with a filter line on,
and you can see if the filter is processed in the export or not, or if you even get any video at all.
then you can localize where the error occurs easier.

Yep it is exported nicely. However I'm seeing that the moviePlayer is nil. I thought the mySCPlayerView.player is already initialized, judging from the tutorial. Do I have to initialize a SCPlayer myself and set it to mySCPlayerView.player?

Cool, then it is only the matter of views setup. My experience/knowledge has reached its limits now sorry :P
but  if you check the example project I think you can get the hints, I just followed the example project and had no problem setting up anything, but I use storyboard and set up the "renderer/player" VIEW and set its class in storyboard to SCSwipeableFilterView and connect the view with property:
@property (weak, nonatomic) IBOutlet SCSwipeableFilterView *filterSwitcherView;
then just in my .m I do _player.CIImageRenderer = self.(myrendererview)
I do the exact same thing with SCImageView, and it should work for you if you set up the views correctly.

Alright. I'll look more into it. Thanks a lot!!


python3Packages.wordfreq: fix build

update source
use poetry-core as build-system

Description of changes
ZHF: #230712
https://hydra.nixos.org/build/220628964

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 233412 run on x86_64-linux 1

4 packages built:

python310Packages.wordfreq
python310Packages.wordfreq.dist
python311Packages.wordfreq
python311Packages.wordfreq.dist

@ofborg build python3Packages.wordfreq python3Packages.wordfreq.passthru.tests


[BUG] Can't restore from backup

Describe the bug

You can't restore from backup with replace as an option
To Reproduce
Steps to reproduce the behavior:

Create a backup target with S3
Create a new VM from live image
Make some modifications to the filesystem, like create a file
Make a backup of the VM
Wait for the backup to complete
Turn off the VM
Restore the backup with replace as the option

Expected behavior

The restore should complete and the VM should work

Support bundle

harvester-supportbundle_046bfe97-0e0e-45e2-b9e2-6ed767e0aa1d_2021-05-28T23-20-14Z.zip
Environment:

Harvester ISO version: master-4997d634-head
Underlying Infrastructure (e.g. Baremetal with Dell PowerEdge R630): baremetal DL160G9

Additional context
It looks like the volume fails to attach. This happens both with create new, and with replace. This was working on v0.2.0.

according to the log the restoring is still in progress and the time will be dependens on the volume size and network performance, can u test using smaller volume size and images first, and if u like validate image that is over 10G maybe check the lognhorn UI as it will inlcues the progress status(this is something we would like to add in the harvester side in the future).
  spec:
    size:<PHONE_NUMBER>0
    frontend: blockdev
    frombackup: s3://ngildersleeve-harvester@us-west-2/?backup=backup-6eee31ad90bf4d5d&volume=pvc-d5d5a0a8-3bc6-4d9f-a93b-8730b69f4292
    numberofreplicas: 3
    datalocality: disabled
    stalereplicatimeout: 30
    nodeid: ""
    migrationnodeid: ""
    engineimage: longhornio/longhorn-engine:v1.1.1
    recurringjobs: []
    backingimage: image-czjfv
    standby: false
    diskselector: []
    nodeselector: []
    disablefrontend: false
    revisioncounterdisabled: false
    lastattachedby: ""
    accessmode: rwx
    migratable: true
    baseimage: ""
  status:
    ownerid: hpd11s19
    state: attached
    robustness: degraded
    currentnodeid: hpd11s19
    currentimage: longhornio/longhorn-engine:v1.1.1
    kubernetesstatus:
      pvname: pvc-8d5dcb14-7757-43fe-8c85-15e721c0ae9f
      pvstatus: Bound
      namespace: default
      pvcname: restore-test-backup2-1-be13afd6-07f1-43fa-a630-adaff02bef66-rootdisk
      lastpvcrefat: ""
      workloadsstatus:
      - podname: virt-launcher-test-backup2-82d4k
        podstatus: Pending
        workloadname: test-backup2
        workloadtype: VirtualMachineInstance
      lastpodrefat: ""
    conditions:
      restore:
        type: restore
        status: "True"
        lastprobetime: ""
        lasttransitiontime: "2021-05-28T23:14:39Z"
        reason: RestoreInProgress
        message: ""

I retested this on the same machines, but I had wiped and re-installed since then. That cluster was exhibiting strange behavior around how it was upgraded. I tested with a 10GB Backup, and a 5GB backup. They both worked on master-c13c2a35-head. I'm closing this for now, but will keep an eye on it during testing, and will re-open it if needed.


5.12.1

Pull Request Type

[ ] Feature addition
[x] Bugfix
[ ] Documentation update
[ ] Code refactor
[ ] Tests improvement
[ ] CI/CD pipeline enhancement
[ ] Other: [Replace with a description]

Description
Hotfix: Bump minimum version of discord-typings to match the things we're importing
Changes

fix: bump minimum discord-typings version to 0.7.0 (#1675)

Related Issues

Test Scenarios

Python Compatibility

[ ] I've ensured my code works on Python 3.10.x
[ ] I've ensured my code works on Python 3.11.x

Checklist

[ ] I've run the pre-commit code linter over all edited files
[ ] I've tested my changes on supported Python versions
[ ] I've added tests for my code, if applicable
[ ] I've updated / added  documentation, where applicable

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 46.22%. Comparing base (1a7de56) to head (37e73c7).
Report is 49 commits behind head on stable.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           stable    #1677      +/-   ##
==========================================
+ Coverage   45.95%   46.22%   +0.26%
==========================================
  Files         144      148       +4
  Lines       15492    15772     +280
==========================================
+ Hits         7120     7291     +171
- Misses       8372     8481     +109

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Is there a way to add a maker to the existing enabled makers?

I'm writing an internal plugin for my company, and I'd like to enable some custom makers for specific filetypes. However, I don't want to clobber any existing makers that people might have configured.
I tried overriding b:neomake_<ft>_enabled_makers in hopes that both the global and buffer-local makers would run, but this doesn't seem to be the case.
Is there a better solution than the following (for each filetype, in an ftplugin)?
if executable('custom_maker') && exists(':Neomake')
  let g:neomake_<ft>_custom_maker_maker = ...
  if exists('g:neomake_<ft>_enabled_makers')
    call add(g:neomake_<ft>_enabled_makers, 'custom_maker')
  else
    let g:neomake_<ft>_enabled_makers = add(neomake#makers#<ft>#EnabledMakers(), 'custom_maker')
  endif
endif

Interesting use case.
Maybe let b:neomake_<ft>_enabled_makers = neomake#GetEnabledMakers(1, '<ft>') + ['custom_maker'] works?
Haven't tested it, but I agree that there should be an easy way to achieve this.

Hm, neomake#GetEnabledMakers(1, '<ft>') seems to always return empty list for me.

Sorry, args were wrong.
To get names you can use e.g. map(neomake#GetEnabledMakers('python'), 'v:val.name').

For me that function is always returning the default makers, regardless of having g:neomake_<ft>_enabled_makers set. Maybe it's just easiest to provide the maker args and have the users enable it if they want it?

I ended up setting up per-filetype autocmds that set b:neomake_<ft>_enabled_makers, and it worked! Since this is a pretty unique use-case, I'm going to go ahead and close this. Thanks for the help.


[🐴] Error recovery

This PR started out as specifically block handling, but turned out to be easier to refactor everything at once.
So included here is:
Handling for when the user on the other end of the line blocks me and I don't know yet, so I send a message. It checks for block errors, and handles those as unrecoverable pending errors, whereas a network error (which I now check for) is recoverable. It also invalidates profile queries used elsewhere for shadowed optimistic block state using an emitter. Not super proud of this but it works and I think it's clear.
Better handling for network errors. Now if combinations of errors occur, the errors won't shift around, and resuming from one error will attempt to resume from other applicable errors.
It now attempts to recover when returning from background.
I also cleaned up the error components to be a little less invasive, since hopefully we'll clean stuff up before the user has a change to interact with them.
Not in this PR
The case of a user blocking me, I get into a blocked state, and then they unblock me, is not handled here. I expect that to be a rare occurrence, and it's only partially recoverable via the firehose e.g. if the blocker messages me again. Otherwise we'd need to poll for block state, and I don't think that's necessary atm. A simple refresh or pop/push back to the chat will reset it back to zero and the users can message again.
Testing:

turn off network and send message, load history, etc
turn network back on and recover

via interaction
via background->foreground

block yourself from test account and then try to send a message, see block state in header

Slightly weird state where we have two error messages, but I think we can tweak these separately.


[comp:popover] body 进行 scale 缩放之后popover如果处在页面的右下角，浮沉不会正常展示到左上角

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Reproduction link
https://github.com/clfeng/idux-scale-bug-demo
Steps to reproduce

拉取项目
clone https://github.com/clfeng/idux-scale-bug-demo 项目

安装依赖和运行项目
yarn install
yarn dev;

提供了两个页面
http://localhost:5173/#/origin 页面为没有进行body缩放的表现行为

http://localhost:5173/#/scale 页面为进行了缩放后的页面的表现行为
What is expected?
body 进行 scale 缩放之后popover如果处在页面的右下角，其表现行为和未进行缩放的表现一致，可计算得到底部不够空间进而展示在顶部
What is actually happening?
body 进行 scale 缩放之后popover如果处在页面的右下角，浮沉不会正常展示到左上角，而是继续在下部显示，从而撑开了页面
Environment Info
Browsers:
    Chrome: 107.0.5304.121
    Firefox: 77.0.1
    Safari: 14.1.2
  npmPackages:
    @idux/cdk: ^1.2.1 => 1.2.1
    @idux/components: ^1.2.1 => 1.2.1
    @idux/pro: ^1.2.1 => 1.2.1
    vue: ^3.2.41 => 3.2.45

可参考 https://github.com/clfeng/idux-scale-bug-demo 项目READEM.md 的相关截图；
另外window下也可以直接复现问题

初步定位跟 @floating-ui 库的实现有关系

Hi @clfeng can you make an issue at Floating UI's repo?
I think it may be a repro for this issue no one has managed to make yet: https://github.com/floating-ui/floating-ui/pull/1763

Hi @clfeng can you make an issue at Floating UI's repo?
I think it may be a repro for this issue no one has managed to make yet: floating-ui/floating-ui#1763

I provide a demo based on float-ui to reproduce the bug and make an issue at Floating UI's too;
https://github.com/floating-ui/floating-ui/issues/2012

closed #1362


text updates for kunal

[ ] add discussion about compression, how good it is, that we do it in parallel, that makes it go fast, that even though we store up to 7 downsampled resolution, by compressing, # of GB < # GV at native resolution
[ ] ndstore machinve vision should be subsubsection
[ ] OCAR each section

@kunallillaney:  reproducibility code currently lives here nddocs/ndpaper2016
@alexbaden and @MrAE may revise this tomorrow (with my blessing, if that means anything).  So goes in nddocs now, but I'm guessing we migrate to the repo as originally intended.  Stand by for the hackathon tomorrow.

@willgray can you provide an actual link.  i don't know what that means. thanks.

https://github.com/neurodata/nddocs/tree/gh-pages/ndpaper2016

I'm fully briefed. We will handle at lunch.
On Thu, Jun 16, 2016 at 09:10 William Gray<EMAIL_ADDRESS>wrote:

https://github.com/neurodata/nddocs/tree/gh-pages/ndpaper2016
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/neurodata/ndpaper/issues/64#issuecomment-226479152,
or mute the thread
https://github.com/notifications/unsubscribe/AFkzUhMLc7JKfM8wnyUGZHnLiUlqwR6Kks5qMUpDgaJpZM4I29-2
.


[8pt] Integrate the ACTIVE_SERVICES approach into ansible

Replaced in new sprint by https://github.com/gchq/stroom/issues/1239


MSConvertGUI.exe

I tried to run MSConvertGUI.exe

Launched MSConvertGUI.exe under [ pwiz-master\build-nt-x86\msvc-release];
Selected example_data and clicked [Start] button.
An error message box showed up;
I clicked [Continue] button, hangs up...
I have tried diff. sample files, same results.
Please help. thanks; - Iris Li

-) Screen shot:

-) Details
See the end of this message for details on invoking
just-in-time (JIT) debugging instead of this dialog box.
************** Exception Text **************
System.ArgumentException: Illegal characters in path.
at System.IO.Path.CheckInvalidPathChars(String path, Boolean checkAdditional)
at System.IO.Path.GetFileName(String path)
at System.IO.StreamReader..ctor(String path, Encoding encoding, Boolean detectEncodingFromByteOrderMarks, Int32 bufferSize, Boolean checkHost)
at System.IO.StreamReader..ctor(String path)
at MSConvertGUI.MainLogic.ParseCommandLine(String outputFolder, String argv) in C:\SoftwareDevelopment\LC-MS\ProteoWizard\pwiz-master\pwiz_tools\MSConvertGUI\MainLogic.cs:line 337
at MSConvertGUI.ProgressForm.ProgressForm_Load(Object sender, EventArgs e) in C:\SoftwareDevelopment\LC-MS\ProteoWizard\pwiz-master\pwiz_tools\MSConvertGUI\ProgressForm.cs:line 167
at System.Windows.Forms.Form.OnLoad(EventArgs e)
at System.Windows.Forms.Form.OnCreateControl()
at System.Windows.Forms.Control.CreateControl(Boolean fIgnoreVisible)
at System.Windows.Forms.Control.CreateControl()
at System.Windows.Forms.Control.WmShowWindow(Message& m)
at System.Windows.Forms.Control.WndProc(Message& m)
at System.Windows.Forms.ScrollableControl.WndProc(Message& m)
at System.Windows.Forms.Form.WmShowWindow(Message& m)
at System.Windows.Forms.Form.WndProc(Message& m)
at System.Windows.Forms.Control.ControlNativeWindow.OnMessage(Message& m)
at System.Windows.Forms.Control.ControlNativeWindow.WndProc(Message& m)
at System.Windows.Forms.NativeWindow.Callback(IntPtr hWnd, Int32 msg, IntPtr wparam, IntPtr lparam)
************** Loaded Assemblies **************
mscorlib
Assembly Version: <IP_ADDRESS>
Win32 Version: 4.8.4075.0 built by: NET48REL1LAST
CodeBase: file:///C:/Windows/Microsoft.NET/Framework/v4.0.30319/mscorlib.dll
MSConvertGUI
Assembly Version: 3.0.20006.0
Win32 Version: 3.0.20006
CodeBase: file:///C:/pwiz-master/build-nt-x86/msvc-release/MSConvertGUI.exe
System.Windows.Forms
Assembly Version: <IP_ADDRESS>
Win32 Version: 4.8.4042.0 built by: NET48REL1LAST_C
CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/System.Windows.Forms/v4.0_<IP_ADDRESS>__b77a5c561934e089/System.Windows.Forms.dll
System
Assembly Version: <IP_ADDRESS>
Win32 Version: 4.8.4001.0 built by: NET48REL1LAST_C
CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/System/v4.0_<IP_ADDRESS>__b77a5c561934e089/System.dll
System.Drawing
Assembly Version: <IP_ADDRESS>
Win32 Version: 4.8.3752.0 built by: NET48REL1
CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/System.Drawing/v4.0_<IP_ADDRESS>__b03f5f7f11d50a3a/System.Drawing.dll
System.Core
Assembly Version: <IP_ADDRESS>
Win32 Version: 4.8.4110.0 built by: NET48REL1LAST_B
CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/System.Core/v4.0_<IP_ADDRESS>__b77a5c561934e089/System.Core.dll
STL_Containers
Assembly Version: <IP_ADDRESS>
Win32 Version: <IP_ADDRESS>
CodeBase: file:///C:/pwiz-master/build-nt-x86/msvc-release/STL_Containers.DLL
System.Configuration
Assembly Version: <IP_ADDRESS>
Win32 Version: 4.8.3752.0 built by: NET48REL1
CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/System.Configuration/v4.0_<IP_ADDRESS>__b03f5f7f11d50a3a/System.Configuration.dll
System.Xml
Assembly Version: <IP_ADDRESS>
Win32 Version: 4.8.3752.0 built by: NET48REL1
CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/System.Xml/v4.0_<IP_ADDRESS>__b77a5c561934e089/System.Xml.dll
CustomDataSourceDialog
Assembly Version: <IP_ADDRESS>
Win32 Version: <IP_ADDRESS>
CodeBase: file:///C:/pwiz-master/build-nt-x86/msvc-release/CustomDataSourceDialog.DLL
pwiz_bindings_cli
Assembly Version: <IP_ADDRESS>
Win32 Version:
CodeBase: file:///C:/pwiz-master/build-nt-x86/msvc-release/pwiz_bindings_cli.DLL
Accessibility
Assembly Version: <IP_ADDRESS>
Win32 Version: 4.8.3752.0 built by: NET48REL1
CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/Accessibility/v4.0_<IP_ADDRESS>__b03f5f7f11d50a3a/Accessibility.dll
************** JIT Debugging **************
To enable just-in-time (JIT) debugging, the .config file for this
application or computer (machine.config) must have the
jitDebugging value set in the system.windows.forms section.
The application must also be compiled with debugging
enabled.
For example:

<system.windows.forms jitDebugging="true" />

When JIT debugging is enabled, any unhandled exception
will be sent to the JIT debugger registered on the computer
rather than be handled by this dialog box.

Yes, I built it using quickbuild.bat --i-agree-to-the-vendor-licenses with no other options.

Chambm, thanks for your reply.
See attached  MSConvertGUI_Build_ReleaseX86.docx

Thanks for the thorough screenshots. Have you tried the 64-bit build? That's a strange error. I don't see anything obviously wrong with your settings.

Nevermind I see the problem. You have "File of filenames" selected. You want "List of files". "File of filenames" is like if you give it a text file where each line is the path to a file you want converted.

Granted we could definitely be doing a better job of validating that instead of that totally unhelpful error. Thanks for the report. :)

build [debug, x64]
MSConvertGUI_Build_DebugX64.docx

Chambm: You want "List of files".
Thanks, it works, see the last page.
MSConvertGUI_Build_ReleaseX86.docx


[CARE-1683] Exclude main branch from running E2E tests in themes

Source

Ahhh... That's what I thought when I made the change. Good to know, thanks!
I'll close this and apply the same change in public themes 😄


Add Todd Arbagast's elements

https://www.oden.utexas.edu/media/reports/2017/1728.pdf
Also (more recent work): http://users.oden.utexas.edu/~arbogast/PaperArchive/Arbogast_Tao_Wang_2021X.pdf

Also #59

I think #92 did this?

I'm currently unsure about how correct the elements in #92 are, so leaving this open until I've reread both links

Ah, I see.

See also
https://arxiv.org/pdf/1807.01416.pdf (Hexahedral div elements)
and
https://www.doi.org//10.1137/15M1013705 (AC div elements)
and
https://arxiv.org/pdf/1809.02192.pdf (direct serendipity)


Suggestion: Add docs on $ref resolving

As seen by issues #132  and #98 , there are some people who have been confused about how to use jsonschema in order to resolve references between schema files. I myself couldn't figure it out until I saw issue #98. I suggest adding some documentation here or perhaps at http://python-jsonschema.readthedocs.org/en/latest/ if it is affiliated.

Revealing implementation a bit there I think ?

It's explicit in the API that RefResolver takes a (mutable) mapping as a cache, it's not particularly its concern what mapping that is as long as it satisfies that interface.

Although it looks like RefResolver does some more setup these days, which won't break things but which might actually signal that it's time for some more thought there.


bump nix version requirement to 1.10, fixes #10337

Since 2d9885d we encourage to use 1.10 features (and we do use them),
but we only check for 1.8. This goes for both master and 15.09 nixpkgs versions.
@edolstra: I assume it was meant that 15.09 requires nix-1.10.

Updated that wiki article as well. Are there no version-agnostic one-click links? I suppose we should review such critical stuff and move to nix manual (maybe rewrite to be sure of licensing).

merge conflicts, otherwise +1.

Sorry, I didn't see this. I just commited something similar (2e5b6362ae299bf699e5b12f20e8952878b43d75).

OK. What about 15.09? It's the same, isn't it?

Done, cherry-picked to 15.09.


1000ms delay when only IPv4 address is used when specifying ListenOptions for Kestrel.

I've noticed that http requests to Kestrel take at least 1000ms if only a IPv4 endpoint is bound to. If a request normally takes 15ms, it will take 1015ms instead.
If you use UseKestrel() on its own, it automatically listens to the localhost IPv4 and IPv6 addresses. If you manually specify addresses using KestrelServerOptions but only specify a single Listen with an IPv4, then you see the bug. If you add a second endpoint that uses IPv6, the bug goes away.
Using a default ASP.NET Core 2 template from VS 2017:
No bug
.UseKestrel((options) =>
            {
                options.Listen(IPAddress.Loopback, 5000);
                options.Listen(IPAddress.IPv6Loopback, 5000);
            })

Bug
.UseKestrel((options) =>
            {
                options.Listen(IPAddress.Loopback, 5000);
            })

It would be because you have IPv6 and IPv4 enabled for localhost on your machine, so it will try the IPv6 address first, wait for timeout (1 sec) then try the IPv4 address.
You can easily bind to both with ListenLocalhost(int port)
.UseKestrel((options) =>
    {
        options.ListenLocalhost(5000);
    })

Note ListenLocalhost is new in 2.1.

This is a known issue as described by Ben. It's more of an issue with the HttpClient than Kestrel

Any client would have this problem unless they opened two connections in parallel.

It's been that way since Vista.

We only saw it when rolling out windows 10. But you are right about vista. Our desktops probably only had IPV4 enabled on 7

Can always untick this box on the network adapter :)

👀 Doesn't work for loopback though.

Presuming you don't work for a big company and can tick said boxes ;)

Closing as this is not a Kestrel issue.


Special types'  __doc__ property should be readonly

Feature
Special types' __doc__ property should be readonly (i.e. getter) except for user-defined type. But __doc__ properties of RustPython's special types is writable.
CPython
>>> SystemExit.__doc__
'Request to exit from the interpreter.'
>>> SystemExit.__doc__ = ""
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: can't set attributes of built-in/extension type 'SystemExit'
>>> class A:
...   ...
...
>>> A.__doc__ = "123"
>>> A.__doc__
'123'

RustPython
>>>>> SystemExit.__doc__
'object()\n--\n\nThe base class of the class hierarchy.\n\nWhen called, it accepts no arguments and returns a new featureless\ninstance that has no instance attributes and cannot be given any.'
>>>>> SystemExit.__doc__ = ""
>>>>> class A:
.....   ...
.....
>>>>> A.__doc__
>>>>> A.__doc__ = "123"
>>>>> A.__doc__
'123'

Python Documentation

type_getsets definitions
type_set_doc(PyTypeObject *type, PyObject *value, void *context)

check_set_special_type_attr(PyTypeObject *type, PyObject *value, const char *name)

related:  https://github.com/RustPython/RustPython/pull/3821
__doc__ is also obeying the setattro rule of the type. not specialized one.
If the type is IMMUTABLE, any attributes are non-writeable.
If the given type is not IMMUTABLE, we can also modify its __doc__
>>> import contextlib
>>> contextlib.redirect_stderr.__doc__
'Context manager for temporarily redirecting stderr to another file.'
>>> contextlib.redirect_stderr.__doc__ = 10
>>> contextlib.redirect_stderr.__doc__
10

#2766 looks similar


Injection Error when Including the module.

I started using this module today, but when I include it it gives me an injection error and i'm stuck on it. I've followed the steps:

bower install mdDataTable
Included scripts as folows:

script(defer, src="/bower_components/mdDataTable/dist/md-data-table.js")
script(defer, src="/bower_components/mdDataTable/dist/md-data-table-templates.js")

Included css as follows:
link(rel="stylesheet" href="/bower_components/mdDataTable/dist/md-data-table-style.css")

Added dependency:

var app = angular.module('app', ['ngRoute', 'ngCookies', 'ngResource', 'ngMaterial', 'mdDataTable', 'ui-notification', 'ngFileUpload'])
ERROR: Uncaught Error: [$injector:modulerr]
Unfortunately i can't make it work. Can you help me please?
Thanks,
Leonardo

Hello, can you please try loading the templates first and then the core library?
e.g.:
script(defer, src="/bower_components/mdDataTable/dist/md-data-table-templates.js") script(defer, src="/bower_components/mdDataTable/dist/md-data-table.js")

Unfortunately it does not solve the problem 😞

@iamisti I solved the problem adding script(defer, src="//cdnjs.cloudflare.com/ajax/libs/angular-material-icons/0.5.0/angular-material-icons.min.js") to my scripts include. Hope it can help to fix the problem on the module. Keep on and congratulations, fantastic job here! 👍  😄


homework1.ipynb

Hi Theo,
I am running the homework1.ipynb notebook locally and have received the following error. Before troubleshooting too much further on my own, I wanted to double check with you if you had any recommendations. All my imports worked fine.

Best,
Ralph

Hi Ralph,
Thanks for bringing that up.
The error is related to the underlying NEURON simulation package. NEURON basically offers a high-level Python interface but under the hood it is running compiled code to speed up simulations. Hence, the differential equations governing the axon's nodal membrane dynamics (defined in the MRGnode.mod file) must be compiled in order to be useable by NEURON.
Here's how to do it:

If you're on Mac/Linux, just go to the repository's folder and type in the command nrnivmodl.
If you're in Windows, open the folder named NEURON xxx (xxx being the NEURON version number) located on your Desktop, and run the mknrndll executable. In the window that appears, select the move to the repository's folder and then click `make nrnmech.dll.

Once the MOD file is compiled, restart the notebook and everything should work normally.
Let me know if that solves it.
Best,
Theo

Hi Theo,
All good now, that fixed it! Thanks for the quick response.
-Ralph


multi-network: fix eastwest gateway endpoint filtering

I think this fixes/helps with https://github.com/istio/istio/issues/38704
Gateways care about server settings only (authn, and tlsMode labels) when doing this filtering.
We will filter out servers that we don't think accept mTLS (DISABLED or tlsMode: none). If a client somehow sent non-mTLS traffic, they would have to spoof SNI and then the server would deny them anyway.

@stevenctl: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
5767260e6395e19e9a9b87e10a0e8ca5b8a93f57
link
true
/test release-notes_istio

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. I understand the commands that are listed here.

@stevenctl can you also update this test to include a Count > 1? This test passing in multi-network actually relies on Count==1, which is incredibly fragile.

Adjusted a test per @nmittler comment. His PR https://github.com/istio/istio/pull/37914/ improves coverage much more.

@stevenctl: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
1c2ee0124b71b6362db9683ef14878763fd8b339
link
true
/test release-notes_istio

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. I understand the commands that are listed here.

@stevenctl: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-istiodremote_istio
b8f7e78d1efbcc6238d5e60088d1da8bfb3d0335
link
true
/test integ-security-istiodremote_istio

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. I understand the commands that are listed here.

@stevenctl: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-istiodremote_istio
b8f7e78d1efbcc6238d5e60088d1da8bfb3d0335
link
true
/test integ-security-istiodremote_istio

integ-security_istio
b8f7e78d1efbcc6238d5e60088d1da8bfb3d0335
link
true
/test integ-security_istio

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. I understand the commands that are listed here.

@stevenctl: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-istiodremote_istio
b8f7e78d1efbcc6238d5e60088d1da8bfb3d0335
link
true
/test integ-security-istiodremote_istio

integ-security_istio
b8f7e78d1efbcc6238d5e60088d1da8bfb3d0335
link
true
/test integ-security_istio

integ-security-multicluster_istio
b8f7e78d1efbcc6238d5e60088d1da8bfb3d0335
link
true
/test integ-security-multicluster_istio

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. I understand the commands that are listed here.

@stevenctl: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-multicluster_istio
d8f1313b33279e070c8c2c69443a21b89c5f51e1
link
true
/test integ-security-multicluster_istio

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. I understand the commands that are listed here.

@stevenctl: PR needs rebase.

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.

@stevenctl: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-multicluster_istio
d8f1313b33279e070c8c2c69443a21b89c5f51e1
link
true
/test integ-security-multicluster_istio

lint_istio
348d517ae95e1e6fe251688ed37eb9c6584cc238
link
true
/test lint_istio

gencheck_istio
348d517ae95e1e6fe251688ed37eb9c6584cc238
link
true
/test gencheck_istio

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. I understand the commands that are listed here.

@stevenctl: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-multicluster_istio
d8f1313b33279e070c8c2c69443a21b89c5f51e1
link
true
/test integ-security-multicluster_istio

lint_istio
348d517ae95e1e6fe251688ed37eb9c6584cc238
link
true
/test lint_istio

gencheck_istio
348d517ae95e1e6fe251688ed37eb9c6584cc238
link
true
/test gencheck_istio

integ-ipv6_istio
348d517ae95e1e6fe251688ed37eb9c6584cc238
link
true
/test integ-ipv6_istio

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. I understand the commands that are listed here.

It needs a test

Nathan's PR added tests. This PR removed the need for the global DestinationRule

What;'s the behavior with tls disabled before and after? IIUC, for east-west gateway the cluster will include cluster local endpoints.

This makes it so TLS disabled in DR doesn't affect passthrough gateways (the endpoint will remain). That DR based filtering makes it so clients can't send plaintext that will inevitably fail since we require TLS/SNI.
If a server is plaintext, we still remove the endpoint from the gateway.

will include cluster local endpoints.

Network local, non plaintext endpoints.

Network local, non plaintext endpoints.

Looks good

In response to a cherrypick label: new issue created for failed cherrypick: #39226

In response to a cherrypick label: #38762 failed to apply on top of branch "release-1.14":
Applying: multi-network: prevent eastwest gateway endpoint filtering
Applying: rel note
Applying: skip check for 500s
Using index info to reconstruct a base tree...
Falling back to patching base and 3-way merge...
CONFLICT (add/add): Merge conflict in tests/integration/security/authorization_test.go
Auto-merging tests/integration/security/authorization_test.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0003 skip check for 500s
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to a cherrypick label: new issue created for failed cherrypick: #39229

I opened https://github.com/istio/istio/issues/39330


Piping

Brief description:
The 'Piping' feature in REDCap allows you to inject previously collected data into text on a data collection form or survey, thus providing greater precision and control over question wording.
User spec:
https://github.com/CCTC-team/redcap_cypress/blob/TA_v13.8.1/user_requirement_specification/non_core/09_piping.spec
Feature link:
https://github.com/CCTC-team/redcap_cypress/blob/TA_v13.8.1/manual_scripts/09_Piping.feature
Pre review:

[ ] Script(s) targets the correct feature sufficiently
[ ] The user requirements are met

Post review:

[ ] Script(s) targets the correct feature sufficiently
[ ] Script(s) appropriately tests the feature

@MintooXavier  - Please review script.

https://github.com/CCTC-team/redcap_cypress/blob/45b170c23775e1be2bcd486cba7b7161ea4cfdbd/manual_scripts/D.109.100_Piping.feature#L16 - Check name
https://github.com/CCTC-team/redcap_cypress/blob/45b170c23775e1be2bcd486cba7b7161ea4cfdbd/manual_scripts/D.109.100_Piping.feature#L19 - add First button as there are many buttons. change similar step below
https://github.com/CCTC-team/redcap_cypress/blob/45b170c23775e1be2bcd486cba7b7161ea4cfdbd/manual_scripts/D.109.100_Piping.feature#L20 - Specify exact words of select option
https://github.com/CCTC-team/redcap_cypress/blob/45b170c23775e1be2bcd486cba7b7161ea4cfdbd/manual_scripts/D.109.100_Piping.feature#L21 - "Field Label) " no ) required
https://github.com/CCTC-team/redcap_cypress/blob/45b170c23775e1be2bcd486cba7b7161ea4cfdbd/manual_scripts/D.109.100_Piping.feature#L54 - verify you can see 'How much do you love ______?' before selecting chocolate
Verify similarly for I love [ice_cream]
Select another option and verify the value changes in both places

@TanDBPRO : Pre-review completed

@MintooXavier  -  Please review when you can. I made the changes you have advised above.

Remove the extra space in field labels.  Similarly changes in lots of steps.
Use https://aldefouw.github.io/redcap_cypress/#i-should-see-the-dropdowntype-field-labeled-string-with-the-options-below instead of https://github.com/CCTC-team/redcap_cypress/blob/3b90a2e165026c3339c6a1f5f768f522cd6696d8/manual_scripts/Piping_109/D.109.100 - Piping.feature#L31
Check the option name. https://github.com/CCTC-team/redcap_cypress/blob/3b90a2e165026c3339c6a1f5f768f522cd6696d8/manual_scripts/Piping_109/D.109.100 - Piping.feature#L35
Just Min: is ewnough. https://github.com/CCTC-team/redcap_cypress/blob/3b90a2e165026c3339c6a1f5f768f522cd6696d8/manual_scripts/Piping_109/D.109.100 - Piping.feature#L42
Same as above. https://github.com/CCTC-team/redcap_cypress/blob/3b90a2e165026c3339c6a1f5f768f522cd6696d8/manual_scripts/Piping_109/D.109.100 - Piping.feature#L43
Maybe take this out. There is no step to check options for slider. Not worth adding a new step definition to check this. https://github.com/CCTC-team/redcap_cypress/blob/3b90a2e165026c3339c6a1f5f768f522cd6696d8/manual_scripts/Piping_109/D.109.100 - Piping.feature#L47
Event name is 'Event 1'. https://github.com/CCTC-team/redcap_cypress/blob/3b90a2e165026c3339c6a1f5f768f522cd6696d8/manual_scripts/Piping_109/D.109.100 - Piping.feature#L60
Steps 65 till 67 can be deleted. https://github.com/CCTC-team/redcap_cypress/blob/3b90a2e165026c3339c6a1f5f768f522cd6696d8/manual_scripts/Piping_109/D.109.100 - Piping.feature#L65
It is before not after.  But no need to specify position. 'Then I should see a field named "How much do you love Chocolate?"' is enough. https://github.com/CCTC-team/redcap_cypress/blob/3b90a2e165026c3339c6a1f5f768f522cd6696d8/manual_scripts/Piping_109/D.109.100 - Piping.feature#L70
This step is not required. https://github.com/CCTC-team/redcap_cypress/blob/3b90a2e165026c3339c6a1f5f768f522cd6696d8/manual_scripts/Piping_109/D.109.100 - Piping.feature#L71
No need this step. https://github.com/CCTC-team/redcap_cypress/blob/3b90a2e165026c3339c6a1f5f768f522cd6696d8/manual_scripts/Piping_109/D.109.100 - Piping.feature#L77
Verify you can see 'I love Strawberry'. https://github.com/CCTC-team/redcap_cypress/blob/3b90a2e165026c3339c6a1f5f768f522cd6696d8/manual_scripts/Piping_109/D.109.100 - Piping.feature#L78

Hi Mintoo,  I have reviewed your comments and made the necessary changes. Please review when you have time.

Remove the extra space in field labels. Similarly changes in lots of steps.
Extra spaces removed.

Use https://aldefouw.github.io/redcap_cypress/#i-should-see-the-dropdowntype-field-labeled-string-with-the-options-below instead of
Use the correct language as above

Check the option name.
Option name corrected

Just Min: is enough.
Min: is used now in the step

Just Max: is enough.
**Max: is used now in the step

Maybe take this out. There is no step to check options for slider. Not worth adding a new step definition to check this.
Step has been removed

Event name is 'Event 1'
Event 1 is the used now

Steps 65 till 67 can be deleted
Steps have been deleted

It is before not after. But no need to specify position. 'Then I should see a field named "How much do you love Chocolate?"' is enough
Removed reference to specify position

This step is not required
Removed step

No need this step.
Removed step

Verify you can see 'I love Strawberry'
Step now verifies 'I love Strawberry

Automated the script..


Ability to authenticate in the HTTP endpoint (for WS or HTTP) before executing router

It would be great to be able to, for example, block unauthenticated users or rate limit WebSockets connections since these act as Dos vectors.

We should deal with the semantics of the context function with subscriptions cause right now it's problematic.
I think we need to enable the Clone bound on the TCtx type + document this behavior.


Unable to use Domain heuristic in parallel mode with more than 14 competing threads.

In the newest version of clingo, using the following options simultaneously: --heuristic=Domain and --parallel-mode=n,compete, with n > 14, throws an error *** ERROR: (clingo): <config>.[solver.14] : Heuristic requires lookback strategy!.
This is true on the command line and by using the python API. For example:
(ASH) PS C:\Users\OllieKampo\OneDrive\Code\ASH-Planner> python
Python 3.10.6 | packaged by conda-forge | (main, Oct 24 2022, 16:02:16) [MSC v.1916 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> import clingo
>>> clingo.control.Control(['--heuristic=Domain', '--parallel-mode=20,compete'])
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\Users\OllieKampo\anaconda3\envs\ASH\lib\site-packages\clingo\control.py", line 172, in __init__
    self._rep = _c_call('clingo_control_t *', _lib.clingo_control_new,
  File "C:\Users\OllieKampo\anaconda3\envs\ASH\lib\site-packages\clingo\_internal.py", line 41, in _c_call
    _handle_error(c_fun(*args, p_ret), handler)
  File "C:\Users\OllieKampo\anaconda3\envs\ASH\lib\site-packages\clingo\_internal.py", line 65, in _handle_error
    raise RuntimeError(msg)
RuntimeError: <config>.[solver.14] : Heuristic requires lookback strategy!

Enabling parallel mode with more than 14 threads, compete enabled, but no domain heuristic with --parallel-mode=20,compete for example does work fine though.
If you use --heuristic=Domain and --parallel-mode=n,split, with n > 14 it also works fine. It seems to be the compete that causes this. For example:
(ASH) PS C:\Users\OllieKampo\OneDrive\Code\ASH-Planner> python
Python 3.10.6 | packaged by conda-forge | (main, Oct 24 2022, 16:02:16) [MSC v.1916 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> import clingo
>>> clingo.control.Control(['--heuristic=Domain', '--parallel-mode=20,split'])
<clingo.control.Control object at 0x000001C0B2EEB6A0>

I couldn't find anything in the documentation that mentioned this, but perhaps I was searching for the wrong thing.

Not all portfolios are compatible with all command line options. You can easily print and modify it. For example:
➜ clingo --print-portfolio | grep -v solver.14 > portfolio.txt
➜ echo | clingo --heuristic=Domain --parallel-mode=20 --configuration=portfolio.txt
clingo version 5.6.2 (e759a62)
Reading from stdin
Solving...
Answer: 1

SATISFIABLE

Models       : 1
Calls        : 1
Time         : 0.001s (Solving: 0.00s 1st Model: 0.00s Unsat: 0.00s)
CPU Time     : 0.003s
Threads      : 20       (Winner: 3)

Thanks for the quick response. I understand, I will take a look at this, thanks!
I think the issue is though, that the error it reports with the lookback strategy isn't particularly helpful, since it is not obvious why changing the number of threads from 14 to 15 would suddenly cause an issue with the lookback strategy, which isn't mentioned on the command line.
Would I be correct in assuming that the problem happens because clingo changes its default configuration when the number of threads becomes large, therefore causing a conflict with other command line arguments (in this case the domain heuristic)?
I'm not sure how you'd make this error more helpful necessarily, but I hope you see what I mean.

Command line options overwrite options in the portfolio configurations. This can lead to conflicts. The error might be a bit hard to read but but  <config>.[solver.14] : Heuristic requires lookback strategy! indicates this conflict.

Looks like I need to do a bit more experimenting and reading on this then.
Thanks for the help Roland.


[BUG] CloudBlockBlob getURI Returns a URL with an Extra Space at the End Occassionally

Describe the bug
On some occassions the CloudBlockBlob .getURI() method returns an extra space at the end of the blob url
Exception or Stack Trace
example https://storagedocsdev.blob.core.windows.net/filled-forms/49f4735f-d168-4277-9938-b9705f357973/4ed84478-666a-4fd9-b659-d43f66694762.pdf .  - after encoding the url
To Reproduce
The behaviour is intermittent
Code Snippet
CloudBlockBlob blob = container.getBlockBlobReference(blobName);
blob.upload(file.getInputStream(), file.getSize());
LOG.info("file uploded ");
URI documentUri = blob.getUri();
Expected behavior
There should not be a blank space returned at the end of the blob url.
Screenshots
If applicable, add screenshots to help explain your problem.
Setup (please complete the following information):

OS: Linux
IDE: NA
Library/Libraries: Azure Blob Java SDK

Additional context
Add any other context about the problem here.
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[*] Bug Description Added
[*] Repro Steps Added
[*] Setup information Added

Hi, @nilavghosh. Unfortunately we are no longer doing bug fixes in track 1. If you would like more support, we request that you move to track 2 (v12+).


Scan for servers on LAN

The server browser should be able to find servers on the local area network, without said servers being port forwarded and available on the internet.

Figured out how to parse the ARP table. Now I just need to actually have the server list fetch the servers alongside the regular internet ones.


Better handling of variable creation with tensor as initializer.

when the variable dtype is not specified, the dtype of the tensor/array passed as initializer is used instead of defaulting to backend.floatx().
with the JAX backend, don't needlessly create a copy of the initializer value, reuse it if possible

Codecov Report
Attention: Patch coverage is 66.66667% with 1 line in your changes missing coverage. Please review.

Project coverage is 60.15%. Comparing base (e0f61ee) to head (b634edf).

Files with missing lines
Patch %
Lines

keras/src/backend/jax/core.py
0.00%
1 Missing :warning:

:exclamation:  There is a different number of reports uploaded between BASE (e0f61ee) and HEAD (b634edf). Click for more details.
HEAD has 6 uploads less than BASE

Flag
BASE (e0f61ee)
HEAD (b634edf)

keras
4
1

keras-torch
1
0

keras-tensorflow
1
0

keras-jax
1
0

Additional details and impacted files
@@             Coverage Diff             @@
##           master   #20557       +/-   ##
===========================================
- Coverage   82.20%   60.15%   -22.05%
===========================================
  Files         515      515
  Lines       48085    48087        +2
  Branches     7526     7527        +1
===========================================
- Hits        39527    28926    -10601
- Misses       6740    17403    +10663
+ Partials     1818     1758       -60

Flag
Coverage Δ

keras
60.15% <66.66%> (-21.90%)
:arrow_down:

keras-jax
?

keras-numpy
60.15% <66.66%> (+<0.01%)
:arrow_up:

keras-tensorflow
?

keras-torch
?

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

I think both behaviors are acceptable to be honest -- it's a bit weird to create a Variable from a np array and have it be float64 by default.

What was also weird is that you would often get 3 copies in memory: the original array, the cast array and then the copied array.

Perhaps we can compromise by first converting the value to a tensor (which might cast the dtype) and then using the resulting dtype?

Oh yeah, that's a good idea. This will take care of the int vs float Python array case.

Perhaps we can compromise by first converting the value to a tensor (which might cast the dtype) and then using the resulting dtype?

Oh yeah, that's a good idea. This will take care of the int vs float Python array case.

@fchollet this is now ready.