Issue 1015

Add HSCAN, BITFIELD, KEYS, RANDOMKEY, SCAN, SORT, TOUCH, and TYPE.
Description

Motivation and Context

How Has This Been Tested?

Checklist:

[ ] I have submitted a CLA form
[ ] If applicable, I have updated the documentation accordingly.
[x] If applicable, I have added tests to cover my changes.

Codecov Report

Merging #50 into master will decrease coverage by -0.21%.
The diff coverage is 84.63%.

@@            Coverage Diff             @@
##           master      #50      +/-   ##
==========================================
- Coverage   86.99%   86.79%   -0.21%
==========================================
  Files          36       40       +4
  Lines        6052     6869     +817
==========================================
+ Hits         5265     5962     +697
- Misses        787      907     +120

Impacted Files
Coverage Δ

Tests/SwiftRedisTests/TestSetCommandsPart2.swift
95.35% <ø> (-0.01%)
:x:

Tests/SwiftRedisTests/TestTransactionsPart7.swift
93.77% <ø> (ø)
:white_check_mark:

Sources/RedisMulti+Basic.swift
90.47% <ø> (ø)
:white_check_mark:

Sources/RedisMulti+List.swift
100% <ø> (ø)
:white_check_mark:

Sources/RedisMulti.swift
76.92% <100%> (+3.39%)
:white_check_mark:

Sources/Redis.swift
72.01% <56.75%> (-2.75%)
:x:

Sources/Redis+Geo.swift
66.66% <66.66%> (ø)

Sources/RedisMulti+Geo.swift
66.66% <66.66%> (ø)

Sources/Redis+Hash.swift
84.82% <76.92%> (-1.04%)
:x:

Sources/Redis+Basic.swift
91.58% <84.37%> (-1.27%)
:x:

... and 8 more

Continue to review full report at Codecov.

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

@shmuelk This includes the GEO commands I added

LGTM


format to standard output on JSON blob in image-layout

This should format the output as standard JSON, as above JSON objects.
Signed-off-by: xiekeyang<EMAIL_ADDRESS>

LGTM

LGTM


Add link exam feature

Added LinkExamCommand class and LinkExamCommandParser to allow linking of task to the exam. Added ability for GUI to display the exam that the task is linked.

Codecov Report

Merging #104 (b3702bb) into master (163a496) will decrease coverage by 1.41%.
The diff coverage is 1.60%.

@@             Coverage Diff              @@
##             master     #104      +/-   ##
============================================
- Coverage     37.48%   36.06%   -1.42%
  Complexity      374      374
============================================
  Files           135      137       +2
  Lines          2297     2390      +93
  Branches        267      289      +22
============================================
+ Hits            861      862       +1
- Misses         1393     1484      +91
- Partials         43       44       +1

Impacted Files
Coverage Δ

...main/java/seedu/address/commons/core/Messages.java
0.00% <ø> (ø)

.../seedu/address/logic/commands/LinkExamCommand.java
0.00% <0.00%> (ø)

.../seedu/address/logic/parser/AddressBookParser.java
45.71% <0.00%> (-1.35%)
:arrow_down:

...du/address/logic/parser/LinkExamCommandParser.java
0.00% <0.00%> (ø)

src/main/java/seedu/address/model/AddressBook.java
43.05% <0.00%> (+0.58%)
:arrow_up:

src/main/java/seedu/address/model/Model.java
100.00% <ø> (ø)

...rc/main/java/seedu/address/model/ModelManager.java
40.22% <0.00%> (+1.34%)
:arrow_up:

...ava/seedu/address/model/exam/DistinctExamList.java
16.27% <0.00%> (-3.73%)
:arrow_down:

src/main/java/seedu/address/model/exam/Exam.java
0.00% <0.00%> (ø)

...c/main/java/seedu/address/model/exam/ExamDate.java
0.00% <0.00%> (ø)

... and 11 more

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


Git town command not found

Since I am using Linux, I always install using the manual installation: http://www.git-town.com/install.html
But now I installed as always I did, and always worked fine, but now when I type "git town" I have this error:
git: 'town' is not a git command. See 'git --help'
When I saw the src directory, there is no executables there anymore... What do I need to do??

The latest release was a rewrite in go and we still need to update the install instructions. You can grab the appropriate binary from https://github.com/Originate/git-town/releases/tag/v4.0.0 rename it to git-town and put it somewhere in your PATH.

Created #910 to track updating the install instructions


avoid scrolling body of Scaffold

when i add body to Scaffold, after opening bottom bar with TextField inside expandable-bottom-bar, and click on text field, that cause of scrolling body content, how can i resolve that?
for example
child: Scaffold(
  resizeToAvoidBottomPadding: true,
  backgroundColor: Theme.of(context).canvasColor,
  extendBody: true,
  body: Stack(children: <Widget>[Center(child: Text('aaaaa'))]),

How did you solve this?


Avoid warnings when purging an empty database

Description:
This hopefully addresses the SQLAlchemy warning mentioned by @balloob in https://github.com/home-assistant/home-assistant/pull/12448#pullrequestreview-97392619 though I am unable to verify because I do not know how to get the warning.
Anyway, to avoid the warning about an empty IN clause, we now only add the filter for protected states when we actually have some states to protect.
Checklist:

[X] The code change is tested and works locally.

If the code does not interact with devices:

[X] Local tests with tox run successfully.
[ ] Tests have been added to verify that the new code works.

py.test -W error testfile.py


affected/package: os.Getenv() don't work when it has more than 1 env inside

Hello, i'm using go v1.20.4 and os.Getenv function don't work when it has more than 1 variable inside it.
.env file looks like this:

LOCALADDR="http://localhost:2000"
WEBSITEADDR="http://www.postgres-with-go.com"
ENVIRONMENT="dev"

When i find that bug, i make some tests like this:

	_, exist := os.LookupEnv("LOCALADDR")
	_, exist1 := os.LookupEnv("WEBSITEADDR")
	_, exist2 := os.LookupEnv("ENVIRONMENT")

	fmt.Printf("localAddr: %v \nwebsiteAddr: %v \nenvironment: %v \n", exist, exist1, exist2)

And i take this Promt in the terminal:
localAddr: false
websiteAddr: false
environment: false
Normally, i should use environment variables but i don't know why i cant if they are more than 1. This also happened when i try to write the values without quotes. I'm using Windows 11 and Visual Studio Code.
Update: It's Solved
First, I'm solved it with giving env variables to empty string on .env file:

LOCALADDR=""
WEBSITEADDR=""
ENVIRONMENT=""

Later than that i use os.Setenv() functions for each variable:

	os.Setenv("LOCALADDR", "http://localhost:2000")
	os.Setenv("WEBSITEADDR", "http://postgres-with-go.com")
	os.Setenv("ENVIRONMENT", "dev")

	_, exist := os.LookupEnv("LOCALADDR")
	_, exist1 := os.LookupEnv("WEBSITEADDR")
	_, exist2 := os.LookupEnv("ENVIRONMENT")

	fmt.Printf("localAddr: %v \nwebsiteAddr: %v \nenvironment: %v \n", exist, exist1, exist2)

And i take this prompt on terminal:
localAddr: true
websiteAddr: true
environment: true
So, why this weird thing happened from beginning? Why i have to set this variables manually?

CC @golang/windows

have you checked that your shell has set the environment variables correctly?


Update Mozilla Platform Status URLs

The name of the repo has been changed to platform-status and the new home is https://platform-status.mozilla.org/.

Thanks!


allow pass an array of colors to text

Description
see #2019
remove the indexing of color array in order to pass different colors for text
Type of change

[ ] Bug-fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
although not 100% sure about breaking
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

References
address #2019 (but does not close it)
How has this been tested?
some tests pass but did not checked all
EDIT: napari/layers/points/_tests/test_points.py fails.
Final checklist:

[x] My PR is the minimum possible work for the desired functionality
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works

Tests fail as it looks like the assumption that this property is now 1D is not true and so should be changed possibly with something like an np.expand_dims if needed.
E           (shapes (1, 4), (4,) mismatch)
E            x: array([[0., 0., 0., 1.]], dtype=float32)
E            y: array([0, 0, 0, 1])

Would be great if @kevinyamauchi can weigh in with a review as I havn't followed convo in #2019, but overall this seems very reasonable, and thanks @giovp for the contribution and thanks @kevinyamauchi for helping out!!

Hi all,
thanks a lot for the very prompt reply and detailed explanation! Was very useful.

This was great additional context @giovp and I think this is something we don't want to support!

So I'd be totally happy to close this and wait for the new release with Color Manager and get back to you then! Looking forward to it!

Thanks for letting us know this. I'm fine leaving this open so we don't forget about it, but I think waiting till after #1940 makes the most sense!

No worries and thank you for the additional information, @giovp ! I will ping you once we open a PR to add ColorManager  to the TextManager in case you'd like to follow along.

No worries and thank you for the additional information, @giovp ! I will ping you once we open a PR to add ColorManager to the TextManager in case you'd like to follow along.

yes, that'd be great, thank you!

@giovp thanks for you patience here! We've added the ColorManger in #2204 and we're very close to adding the text manager in #2855. Now might be a good time to revisit this and the underlying issue described in #2019.
Maybe @andy-sweet when you have time you can read through the issue in #2019 and #2020 and we can think about the best way to address that request in light of the progress we've made. Thanks!!


CI Test - Don't merge

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.4 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/110341&drop=dogfoodAlpha

Close with reason: null;


first draft of splitting NWSS signals

Description
This splits the dataset based on the provider and normalization (not every pair is actually present), and adds the metric signals. The resulting signals are called:

pcr_conc_smoothed_CDC_VERILY_flow-population
detect_prop_15d_CDC_VERILY_flow-population
percentile_CDC_VERILY_flow-population
ptc_15d_CDC_VERILY_flow-population
pcr_conc_smoothed_CDC_VERILY_microbial
detect_prop_15d_CDC_VERILY_microbial
percentile_CDC_VERILY_microbial
ptc_15d_CDC_VERILY_microbial
pcr_conc_smoothed_NWSS_flow-population
detect_prop_15d_NWSS_flow-population
percentile_NWSS_flow-population
ptc_15d_NWSS_flow-population
pcr_conc_smoothed_NWSS_microbial
detect_prop_15d_NWSS_microbial
percentile_NWSS_microbial
ptc_15d_NWSS_microbial
pcr_conc_smoothed_WWS_microbial
detect_prop_15d_WWS_microbial
percentile_WWS_microbial
ptc_15d_WWS_microbial

Some of these can have negative values; for e.g. ptc_15d, the values are small enough that I expect these may actually be exponents. Still looking into why the concentration data has negative values, which are too large to make sense as exponents.
Fixes

Fix generate_weights in the case where some weights are negative, and added a test for it

Minh confirmed that this runs on her machine and that the output looks reasonable. A couple of things to do before we merge this:

[ ] figure out the right cut-off for sig digits so that the noise is removed
[ ] seeing what values the formerly negative concentrations became
[ ] write docs similar to this PR

Currently not passing because of the same update that made nssp tests fail. Once that's merged and this is rebased it will pass.

This hasn't been released at all yet, right? You previously did statistical review on this; is there anything else we want to do for these new signals?

It has not. There's the corresponding docs, which I think Will read through at one point.

closing as this is migrating to a new endpoint


can't run .sh or .py file directly from bashoperater.

I am trying to run python script from bash command in airflow, I even tried running the .sh file from the bash command which may trigger the python file. But it keeps giving me error file not found. I just can't figure out the path to be given to the file.

Is there anybody else facing the same problem?

@CodeVINCI The BashOperator will execute a bash located in the container. It doesn't run anything from the host.
What you should look into:

can it be done by a simple python script with a PythonOperator?
share a volume with the worker so it can see the script that's on the host, but then make sure all paths are compatible


Test Prometheus integration in CI

Verifies that Prometheus integration works properly after changes.
Performs the following checks:

Test controller metrics on both controller and Prometheus endpoints
Test agent metrics on each agent, and on Prometheus

+1 for Antonin's comment on adding test in e2e (while in general I'm OK to put the validation in either Jenkins jobs or e2e).

@antoninbas @edwardbadboy
I agree with both - thought that this could be a stopgap while I figure out how to do this in e2e.
Can you show me something which I could use as an example?

There are a lot of examples under https://github.com/vmware-tanzu/antrea/tree/master/test/e2e
I don't want to merge this as it is. Even if Jenkins reported a success right now, there would actually be no guarantee that the test is correct or passing.

@antoninbas no sense in merging this, I agree. I'll redo this as e2e.

Skipped the Prometheus tests on Kind. @antoninbas @edwardbadboy  do you think that running Prometheus within Kind is doable? Or will create capacity issues in the present or in the future?

/test-all

@antoninbas testing coverage isn't complete in TestPrometheusMetricsOn* as it only checks one metric (basically a ping on the Prometheus client listener).
TestMetricsOnPrometheusServer actually verifies that all the metrics actually reach the Prometheus server and are parsable - to avoid registration errors etc.
I could transfer this functionality to TestPrometheusMetricsOn - that would allow execution without Prometheus deployment too.
So I could spawn an rbac yaml out of antrea-prometheus.yml, and skip Prometheus server deployment.
LMK if that make any sense (even though I believe that at some point we will need testing with Prometheus server as well).

@ksamoray my preference would have been to do most of the testing without actually relying on a Prometheus server - if that makes any sense, given that we are testing Prometheus support... Is there any Prometheus testing library that can programmatically connect to our /metrics endpoint, verify all the available metrics for a small window of time and return the number of metrics and possible errors? If the only way to do that is to deploy a Prometheus server, then I think your approach is the right one.
Regardless I think you know best here, so if you think this is the best way to have comprehensive testing of the metrics we expose, then that's what we should to. In any case, it seems reasonable to at least validate that the antrea-prometheus.yml manifest that we provide does indeed work.

@antoninbas I've utilized the parsing capabilities from Prometheus on the queried data from the controller, agent.
So from testing capabilities it covers everything except the antrea-prometheus.yml file. As it deploys Prometheus, I don't see how we can validate it without Prometheus deployment :)
I've added a --prometheus flag to push_antrea.sh and to go test as you advised.
I could run TestPrometheusMetricsOn* always as they do not require the server and make only Test*MetricsOnPrometheusServer dependent on the -prometheus flag - that would require splitting the yamls into deployment and rbac which is consistent with the other yamls as well.
Does that make sense?

/test-all

@ksamoray

I could run TestPrometheusMetricsOn* always as they do not require the server and make only Test*MetricsOnPrometheusServer dependent on the -prometheus flag - that would require splitting the yamls into deployment and rbac which is consistent with the other yamls as well.
Does that make sense?

I think it's ok as it is now, and that we can punt this to a future PR if needed. What do you think?
If we went with this approach, I would favor creating the ServiceAccount programmatically for these tests as part of the framework instead of using a yaml file.

I think it's ok as it is now, and that we can punt this to a future PR if needed. What do you think?
If we went with this approach, I would favor creating the ServiceAccount programmatically for these tests as part of the framework instead of using a yaml file.
@antoninbas yeah makes sense

/test-e2e

For some reason the Prometheus tests are skipped although I added the -prometheus flag to go test in macros.yml.

/test-e2e

@edwardbadboy when I make changes to macros.yml, do they apply to CI while testing my change? Or only after the PR is merged?

@antoninbas @srikartati tried to get rid of this, also getting rid of using prometheus.MustNewConstMetric() as the metrics wrapper offers its own implementation.
Both @tnqn and myself tried this...

@edwardbadboy when I make changes to macros.yml, do they apply to CI while testing my change? Or only after the PR is merged?

I think this is expected unfortunately. This is the downside of hardcoding the test logic in the yml file (I'm hoping we can move it to scripts like we did for EKS / GKE testing). Please run locally after rebasing master and make sure it runs on your cluster. After it's merged, this job (https://jenkins.antrea-ci.rocks/job/antrea-job-updater-for-period/) should run and update the e2e CI jobs.

@done. Tests do pass on my vagrant env but that doesn't validate macros.yml

/test-all

/test-networkpolicy

/test-networkpolicy

/test-networkpolicy


fix typos

I assume the html in pages are generated by these files, so there no need to edit them

Good catch, thanks!


Ensure Identity.Views.{V3, V4}.dll are the AllItemsFullPathWithTarget…

…Path group so they get copied across ProjectReferences
Fix https://github.com/aspnet/AspNetCore-Internal/issues/1612.

Lots of test failures still. I'm going to try and rebase this on #6350 to reduce the noise and see if this actually worked.

Waiting on https://github.com/aspnet/AspNetCore/pull/6390


copy root fs into tmpfs

@roymarantz Looks like this works as expected. This should hopefully solve the issues we have been seeing where we fill up the overlay FS causing IO errors. This change moves the rootfs onto tmpfs when booting up.
cc @maddalab @Primer42 @defect

Hmm... actually this may not be happy just yet. It appears the cleanup step isnt properly detaching the old loop devices. It looks like the script lazy unmounts them, but if whatever is holding them open never closes them, the mount will stick around forever. Or, its not actually booting with the toram option. Either way ;)
Still debugging

Got it sorted out :)
Successfully running root out of tmpfs

:100: my testing of the boot image with this works perfectly!

Sweet. @roymarantz mind if i merge?

Please hold on the merge till possible legal issue gets settled.

We are good to merge now.
From a comment on the blog post:
espebra Mod  Roy • an hour ago
Hi Roy,
There are no restrictions, so use it as needed. Hope it will provide value for you :)

Ok, great! Thanks @roymarantz!


Add files via upload

Purpose

Fixes: https://github.com/wso2/docs-mi/issues/534

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

Please send the fixes to the 4.3.0 branch.


Support PHPUnit version 5 and 6

I've added PHPUnit versions 5 and 6 support.
I just need to bump PHPUnit version to ^4.8.35, that support the PHPUnit\Framework\TestCase namespace.

Thanks! 😃


Move UI code out of http.js

Moves all the UI state stuff into lib/ui.js. the live reload connection counter state.sse is still managed by http.js, it emits sse-connect and sse-disconnect when the value updates so the UI can rerender. Maybe the counting should also happen in the UI code, with http.js only emitting those events?
Also added in @Flet's simple log output UI, just to make sure that the UI is indeed separate from the HTTP server now :)

Nice thanks!


Merge main into production

Maybe it’s because of squashing???

Looks like this production push removed the feature images on blog cards and the header background on author/tag templates @backspace


Improvements to DMA

Hi @9names I figured it would be easier to get this integrated into your PR rather than open a new one on rp-rs/rp-hal. I've used your branch to successfully get DMA working with PIO. You can see a working example here: https://github.com/Nashenas88/servo-pio/blob/160bf2f3e6f438e773f670fe20f4b20f43266f3c/src/pwm_cluster.rs#L25
I needed access to some structs that were already pub, but not exported from mod.rs. I also fixed a typo I found in a doc comment.

I'm not sure yet what the API is going to look like, so I'm going to take these changes into the branch as-is (since they enable functionality), but they might get updated to match how we want them to be used.
Thanks for contributing, if you find anything else you need out of DMA please do this again.


NImage的底部工具栏可扩展

This function solves the problem (这个功能解决的问题)

底部工具栏可以自行增加按钮，并配置图标、提示、行为
底部工具栏可选择展示哪些按钮（不重要

Expected API (期望的 API)
toolbarOptions: {
    icon: DefineComponent
    tooltip: string
    action: (...args) => void
}[]

tks.
i have tried to search some issue has existed, but i can't read en.
so i missed it that u tell me.
hhhhhh


Adds CSS for displaying codediff tables correctly

Codecov Report

Merging #990 (fb0995d) into master (947923e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #990   +/-   ##
=======================================
  Coverage   81.22%   81.22%
=======================================
  Files          57       57
  Lines        4533     4533
=======================================
  Hits         3682     3682
  Misses        851      851

Continue to review full report at Codecov.

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


Configurable http proxy per host

Fixes #388

I think I fixed most complaints (except restarting of docker/cri-o).


Alternate servo command across PWM controllers

The current implementation does the servo movements in each leg, in leg order: L1H, L1K, L1A, L2H, L2K...
This means that the first 9 servo movements all go to the same PWM controller. If we alternated the controllers, each would have some recovery time while the other one was being used.
Two possible orders come to mind:
L1H, L1K, L1A, L4H, L4K, L4A, L2H, L2K ... This would be easy to implement, because the calculations for movements are done "one leg at a time". The change would just be a simple re-ordering of existing procedures. There's also an opportunity to put a delay at the end of each leg, to let I2C bus activity settle.
L1H, L4H, L1K, L4K, L1A, L4A, L2H ... This would require restructuring the existing code. Could use existing code to build a table of new positioning commands, and then perform the actual commands in whatever order is desirable. Since only the command execution has critical timing, the calculations could be lowered in priority, and it might even be possible to do the actual commands at interrupt level.
You might be able to detect when the I2C bus goes idle, and wait until that point to issue the next command
If the 2 PWM controllers were on separate I2C busses, alternating them might further reduce I2C contention.

This is implemented in the flow processing command via a nested for loop:
         for(int l_base=1;l_base<=3;l_base++)   // use alternate sides for leg movements, resting PWM drivers
         {  for(L=l_base;L<=l_base+3;L=L+3)     // i.e. 1, 4, 2, 5, 3, 6
            {                                   // use frame count to figure next frame position

It seems to have resolved the problems with the last servo command being lost occasionally. There are no DELAY commands between servo commands in flow processing.


Modify initial data to populate Measurement model

As a result, the old individual measurement models are no longer populated with initial data. The dash apps will therefore not work as expected until #170 is implemented.

Looks good. I'll build #170 off this, but maybe we should wait until that and #178 are done done before merging this.
@Sahil590 This modifies the data so that each measurement is associated with a station and a variable. This is relevant for #106 and #107, but you'll have to wait until I've done #170 which links the validation app with the new model

@tsmbland , agree. Let's built on top of this one the relevant issues and then merge when we are ready.


[Feature Request] DNSCrypt - Blacklist Support

DNSCrypt-Proxy has built-in ability to use domain blacklists:
https://github.com/jedisct1/dnscrypt-proxy/wiki/Public-blacklists
Are there any plans to support this (and possible logging of blocked requests) from the within the existing plug-in?

I'm thinking about integrating it the same way like with Bind plugin, yes

That would be awesome, would definitely be interested in it.   Also, thank you for creating the DNSCrypt plug-in!!

@mimugmail @fichtner Very exciting to see this merged.  Thank you both!

@unipacket1 thank you for the request ❤


Could not find package sky_engine at "../../bin/cache/pkg/sky_engine"

Steps to Reproduce
Download ZIP from repo
Please tell us what you were doing and what went wrong.
cd stocks
flutter run
> examples $ cd stocks/
> stocks $ flutter run
Running 'pub get' in stocks...                   \
Could not find package sky_engine at "../../bin/cache/pkg/sky_engine".
Depended on by:
- flutter 0.0.21

Flutter Doctor
[✓] Flutter (on Mac OS, channel alpha)
    • Flutter at /usr/local/Dart/flutter
    • Framework revision e456d86 (11 days ago), engine revision a299d69

[✓] Android toolchain - develop for Android devices (Android SDK 24.0.0-preview)
    • Android SDK at /Users/mikemitterer/Library/Android/sdk
    • Platform android-N, build-tools 24.0.0-preview
    • Java(TM) SE Runtime Environment (build 1.8.0_60-b27)

[✓] iOS toolchain - develop for iOS devices (Xcode 6.2)
    • XCode at /Applications/Xcode.app/Contents/Developer
    • Xcode 6.2, Build version 6C131e
    x Flutter requires a minimum XCode version of 7.0.0.
      Download the latest version or update via the Mac App Store.
    x ideviceinstaller not available; this is used to discover connected iOS devices.
      Install via 'brew install ideviceinstaller'.
    x ios-deploy not available; this is used to deploy to connected iOS devices.
      Install via 'brew install ios-deploy'.

[✓] Atom - a lightweight development environment for Flutter
    • Atom installed; Flutter plugin version 0.2.2

flutter create <myapp> + flutter run works

For more information about diagnosing and reporting Flutter bugs, please see https://flutter.io/bug-reports/.

I would check which flutter, flutter when you run it will populate the bin/cache directory adjacent to the file, examples/stocks is depending on the existence of this bin/cache directory.  If you're running flutter from a separate flutter checkout on disk, it's likely confused.
This issue has come up before:
https://groups.google.com/forum/#!topic/flutter-dev/5wTABjmwhNs
And I suspect we may need to teach the flutter tool or flutter doctor to be smart about this error.

Filed https://github.com/flutter/flutter/issues/3404 for making flutter catch this.


PWA Checklist: Icons for Polaris App

Requirement
favicon and icon is needed for the demo app. This is a subtask required to make the app PWA.
Tasks

Create icons in client folder.
Icons should be clean and should not infringe on copyright. Even a clear 'P' with a background is enough for demo purposes.
in client folder, create a public/images folder and create icons of these sizes required for PWA.

48x48
72x72
96x96
128x128
256x256

Create a 16x16 favicon.ico as well to be used as favicon.

Please Note
This issue is being tagged as hacktoberfest and thus seeing a lot of traffic. If you're going to work on it and raise a PR, please comment. First preference to merge the project will be given to the first comment. However, the person has 24 hours of time to raise the PR. After that, the issue will be open for contribution again.

@tanaypratap I'd love to take this one, too. PWAs are something I've been practicing lately, and I love design work. I'll try to have a mock up at the end of the day.

@tanaypratap Does this meet your approval?

If you want 2018 in there somewhere, I can work that in, too.


Solution: rewrite setup section

Normally, I would not merge this by myself, but after 2 days, that is ok according to C4.


Use import type where needed

This allows consumers to set the typescript option
importsNotUsedAsValues to "error", without getting errors from our
types. To us it should not make a difference at all.

are we sure that these are all the places where this is needed?
also, we need a way to enforce this because otherwise, problematic imports might be introduced later on and we won’t notice any problem until users complain to us.

For one, you are right I forgot to set the option in our own repo. And by doing that, I discovered what was making it show up in my own system: It was because I imported the helperTypes file directly. It is actually a lot more places than that. I will go over those and update the PR.

OK, this is ready now and the correct imports will be enforced by tsc. This only affects .ts files. That means it mainly affects the tests and the helperTypes file.

@kmoschcau shouldn't we just rename the helper types file to helperTypes.d.ts? It should only contain declarations

That is not a bad idea and it would remove the need for import type in the file. I will do that tomorrow.


[WIP] feat(pipelines): add support for collapsible groups

What
Will close #153.
WIP:

Recursive selection of subgroups when parent group is selected
Draw location and edge anchor points of collapsed pipeline group

Description

Adds support for collapsible pipelines groups and related styling as discussed with @andrew-ronaldson and @jeff-phillips-18
Type of change

[x] Feature
[ ] Bugfix
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

I'll pull this down and take a look. Thanks for tagging me.
We'll need a v6 branch update to take in this new functionality. I have a branch open now where I am applying Penta tokens.


Remove extra space

Fixes https://github.com/MicrosoftDocs/feedback/issues/792

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


Data in the reporting cluster does not match data elsewhere

Slack
When investigating https://github.com/wellcomecollection/catalogue-pipeline/issues/2536, I noticed that one of the problematic records has an invalid MARC 650 field.  It declares that it is a Library of Congress id (ind2=0), but contains a MeSH id (subfield 0 starts with D)
The record in question is b1269556, but this is not unique to that record. I have also seen this error occur with D009524Q000266 and D010297 and plenty of other incorrectly marked MeSH ids.
650  0 Newspapers|xEnglish.|0D009524.

When this record last went through the pipeline, it logged an error:
Could not determine LoC scheme from id 'D009524'

When I looked in VHS for it, the field in question is incorrect as expected (ind2=0, subfield 0= D009524)
{
      "fieldTag": "d",
      "marcTag": "650",
      "ind1": " ",
      "ind2": "0",
      "subfields": [
        {
          "tag": "a",
          "content": "Newspapers"
        },
        {
          "tag": "x",
          "content": "English."
        },
        {
          "tag": "0",
          "content": "D009524."
        }
      ]
    },

I decided I should make a report on this, to see how widespread the issue is, and facilitate its resolution.
Imagine my surprise when there appear to be no 6xx varfields with ind2=0 and a MeSH id.  I know this to be incorrect, as I am currently looking at one. I searched for a few other known offenders, and they all have the correct ind2 value (2).
I cannot work out where this is coming from.  How does the reporting cluster end up with different content to everywhere else?

I did fix a problem in the adapters recently, that could cause reporting to have missed some changes that the pipeline will have picked up.
However, that's the wrong way round.

The source data error is the kind of error one makes when originally writing the content.  I can't imagine someone coming along and changing ind2 to the wrong value.
I have looked back through some of the history of this record on VHS. It last changed before that problem arose, this field seems to always have contained ind2=0

I have just run a window covering the last known change on that document {"start": "2023-08-18T09:35:00.000000+00:00", "end": "2023-08-18T09:35:36+00:00"}, and ind2 in reporting is now 0.
This is now the only ind2=0 varfield whose code does not start with sh.
Perhaps someone did correct them all and then a batch process on Sierra put them all back again?

Right. Very odd.
b1269556 was originally correct, then at 2023-08-10T18:08:07Z it changed ind2 from 2 to 0.
So, we have (had) two problems here

The source data problem.  It's wrong.
The reporting hole, perhaps I was wrong when I estimated it had been broken since September.  The data is now going into reporting as expected, but there is still a gap of some kind.

The existence of this source data problem could have been easier to spot and deal with if we had a better way to report dodgy content to collections staff.

the same is true of b30489313. August last year -  2023-08-17 15:53:15Z - it changed from the correct ind2=2 to the incorrect ind2=0


After removing the project the audit log doesn't look consistent with the record of creating the project.

Fixed by #956 .


OWL-ViT playground imports

This is using T4 GPU runtime. All the installs ran successfully, but the second cell failed:
AttributeError                            Traceback (most recent call last)
[<ipython-input-4-2d9aa1f916a4>](https://localhost:8080/#) in <cell line: 10>()
      8 import numpy as np
      9 from scenic.projects.owl_vit import configs
---> 10 from scenic.projects.owl_vit.models import TextZeroShotDetectionModule
     11
     12 from scenic.projects.owl_vit.notebooks import inference

14 frames
[/content/scenic/projects/owl_vit/models.py](https://localhost:8080/#) in <module>
     23 import jax.numpy as jnp
     24 import ml_collections
---> 25 from scenic.projects.owl_vit import layers
     26 from scenic.projects.owl_vit import matching_base_models
     27 from scenic.projects.owl_vit import utils

[/content/scenic/projects/owl_vit/layers.py](https://localhost:8080/#) in <module>
     29 import numpy as np
     30 from scenic.model_lib.base_models import box_utils
---> 31 from scenic.projects.owl_vit import utils
     32 from scenic.projects.owl_vit.clip import layers as clip_layers
     33 from scenic.projects.owl_vit.clip import model as clip_model

[/content/scenic/projects/owl_vit/utils.py](https://localhost:8080/#) in <module>
     22 import jax.numpy as jnp
     23 import numpy as np
---> 24 from scenic.train_lib import train_utils
     25 import scipy
     26

[/content/scenic/train_lib/train_utils.py](https://localhost:8080/#) in <module>
     34 import ml_collections
     35 import numpy as np
---> 36 import optax
     37 from scenic.common_lib import debug_utils
     38 from scenic.dataset_lib import dataset_utils

[/usr/local/lib/python3.10/dist-packages/optax/__init__.py](https://localhost:8080/#) in <module>
     15 """Optax: composable gradient processing and optimization, in JAX."""
     16
---> 17 from optax import contrib
     18 from optax import losses
     19 from optax import monte_carlo

[/usr/local/lib/python3.10/dist-packages/optax/contrib/__init__.py](https://localhost:8080/#) in <module>
     19 from optax.contrib._complex_valued import split_real_and_imaginary
     20 from optax.contrib._complex_valued import SplitRealAndImaginaryState
---> 21 from optax.contrib._dadapt_adamw import dadapt_adamw
     22 from optax.contrib._dadapt_adamw import DAdaptAdamWState
     23 from optax.contrib._mechanic import MechanicState

[/usr/local/lib/python3.10/dist-packages/optax/contrib/_dadapt_adamw.py](https://localhost:8080/#) in <module>
     25 from optax import tree_utils
     26 from optax._src import base
---> 27 from optax._src import utils
     28
     29

[/usr/local/lib/python3.10/dist-packages/optax/_src/utils.py](https://localhost:8080/#) in <module>
     23 import jax
     24 import jax.numpy as jnp
---> 25 import jax.scipy.stats.norm as multivariate_normal
     26
     27 from optax import tree_utils as otu

[/usr/local/lib/python3.10/dist-packages/jax/scipy/stats/__init__.py](https://localhost:8080/#) in <module>
     16 # See PEP 484 & https://github.com/google/jax/issues/7570
     17
---> 18 from jax.scipy.stats import bernoulli as bernoulli
     19 from jax.scipy.stats import beta as beta
     20 from jax.scipy.stats import binom as binom

[/usr/local/lib/python3.10/dist-packages/jax/scipy/stats/bernoulli.py](https://localhost:8080/#) in <module>
     16 # See PEP 484 & https://github.com/google/jax/issues/7570
     17
---> 18 from jax._src.scipy.stats.bernoulli import (
     19   logpmf as logpmf,
     20   pmf as pmf,

[/usr/local/lib/python3.10/dist-packages/jax/_src/scipy/stats/bernoulli.py](https://localhost:8080/#) in <module>
     14
     15
---> 16 import scipy.stats as osp_stats
     17
     18 from jax import lax

[/usr/local/lib/python3.10/dist-packages/scipy/stats/__init__.py](https://localhost:8080/#) in <module>
    606 from ._warnings_errors import (ConstantInputWarning, NearConstantInputWarning,
    607                                DegenerateDataWarning, FitError)
--> 608 from ._stats_py import *
    609 from ._variation import variation
    610 from .distributions import *

[/usr/local/lib/python3.10/dist-packages/scipy/stats/_stats_py.py](https://localhost:8080/#) in <module>
     35 from numpy import array, asarray, ma
     36 from numpy.lib import NumpyVersion
---> 37 from numpy.testing import suppress_warnings
     38
     39 from scipy.spatial.distance import cdist

[/usr/local/lib/python3.10/dist-packages/numpy/testing/__init__.py](https://localhost:8080/#) in <module>
      9
     10 from . import _private
---> 11 from ._private.utils import *
     12 from ._private.utils import (_assert_valid_refcount, _gen_alignment_data)
     13 from ._private import extbuild

[/usr/local/lib/python3.10/dist-packages/numpy/testing/_private/utils.py](https://localhost:8080/#) in <module>
     55 IS_PYSTON = hasattr(sys, "pyston_version_info")
     56 HAS_REFCOUNT = getattr(sys, 'getrefcount', None) is not None and not IS_PYSTON
---> 57 HAS_LAPACK64 = numpy.linalg._umath_linalg._ilp64
     58
     59 _OLD_PROMOTION = lambda: np._get_promotion_state() == 'legacy'

AttributeError: module 'numpy.linalg._umath_linalg' has no attribute '_ilp64'

What's your version of numpy?

Getting the same error. I used to restart the kernel after installing dependencies on colab and it used to work. But now after running the first cell and restarting the kernel. Getting the below error -
AttributeError: module 'jax.random' has no attribute 'PRNGKeyArray'
Numpy version: Version: 1.26.4
Can someone please make the docker file or something? I have never been successful in installing it on my local machine.

Getting the same error. I used to restart the kernel after installing dependencies on colab and it used to work. But now after running the first cell and restarting the kernel. Getting the below error -
AttributeError: module 'jax.random' has no attribute 'PRNGKeyArray'
Numpy version: Version: 1.26.4
Can someone please make the docker file or something? I have never been successful in installing it on my local machine.

You can try to change PRNGKeyArray to PRNGKey. It worked for me.

@Chi-chicken Can you please clarify how to do that? Thank you.

@Chi-chicken Can you please clarify how to do that? Thank you.
I checked my jax.random file on local machine. It doesn't contain PRNGKeyArray. Only PRNGKey.
Attached is my jax.random python file in conda directory.
random.txt
And the full error while importing - from scenic.projects.owl_vit import models
WARNING:absl:Type handler registry overriding type "<class 'float'>" collision on scalar
WARNING:absl:Type handler registry overriding type "<class 'bytes'>" collision on scalar
WARNING:absl:Type handler registry overriding type "<class 'numpy.number'>" collision on scalar

AttributeError                            Traceback (most recent call last)
Cell In[1], line 1
----> 1 from scenic.projects.owl_vit import models
File ~/Downloads/owl_vit/scenic/scenic/projects/owl_vit/models.py:26
 24 import ml_collections

 25 from scenic.projects.owl_vit import layers

---> 26 from scenic.projects.owl_vit import matching_base_models
 27 from scenic.projects.owl_vit import utils

 28 from scenic.projects.owl_vit.clip import model as clip_model

File ~/Downloads/owl_vit/scenic/scenic/projects/owl_vit/matching_base_models.py:23
 21 import jax.numpy as jnp

 22 import ml_collections

---> 23 from scenic.model_lib import matchers
 24 from scenic.model_lib.base_models import base_model

 25 from scenic.model_lib.base_models import box_utils

File ~/Downloads/owl_vit/scenic/scenic/model_lib/matchers/init.py:25
 23 from scenic.model_lib.matchers.hungarian_jax import hungarian_tpu_matcher

 24 from scenic.model_lib.matchers.lazy import lazy_matcher

---> 25 from scenic.model_lib.matchers.sinkhorn import sinkhorn_matcher
File ~/Downloads/owl_vit/scenic/scenic/model_lib/matchers/sinkhorn.py:24
 22 import jax.numpy as jnp

 23 import numpy as np

---> 24 from ott.geometry import geometry
 25 from ott.tools import transport

 28 def idx2permutation(row_ind, col_ind):

File ~/anaconda3/envs/owl_vit/lib/python3.10/site-packages/ott/init.py:15
  1 # Copyright 2022 Google LLC.

  2 #

  3 # Licensed under the Apache License, Version 2.0 (the "License");

(...)
 12 # See the License for the specific language governing permissions and

 13 # limitations under the License.

 14 """OTT library."""

---> 15 from . import geometry, initializers, math, problems, solvers, tools, utils
 16 from ._version import __version__

File ~/anaconda3/envs/owl_vit/lib/python3.10/site-packages/ott/initializers/init.py:1
----> 1 from . import linear, nn, quadratic
File ~/anaconda3/envs/owl_vit/lib/python3.10/site-packages/ott/initializers/nn/init.py:1
----> 1 from . import initializers
File ~/anaconda3/envs/owl_vit/lib/python3.10/site-packages/ott/initializers/nn/initializers.py:21
 16 # TODO(michalk8): add initializer for NeuralDual?

 17 __all__ = ["MetaInitializer", "MetaMLP"]

 20 @jax.tree_util.register_pytree_node_class

---> 21 class MetaInitializer(initializers.DefaultInitializer):
 22   """Meta OT Initializer with a fixed geometry :cite:`amos:22`.

 23

 24   This initializer consists of a predictive model that outputs the

(...)
 56         )

 57   """

 59   def __init__(

 60       self,

 61       geom: geometry.Geometry,

(...)
 65       state: Optional[train_state.TrainState] = None

 66   ):

File ~/anaconda3/envs/owl_vit/lib/python3.10/site-packages/ott/initializers/nn/initializers.py:64, in MetaInitializer()
 20 @jax.tree_util.register_pytree_node_class

 21 class MetaInitializer(initializers.DefaultInitializer):

 22   """Meta OT Initializer with a fixed geometry :cite:`amos:22`.

 23

 24   This initializer consists of a predictive model that outputs the

(...)
 56         )

 57   """

 59   def __init__(

 60       self,

 61       geom: geometry.Geometry,

 62       meta_model: Optional[nn.Module] = None,

 63       opt: optax.GradientTransformation = optax.adam(learning_rate=1e-3),

---> 64       rng: jax.random.PRNGKeyArray = jax.random.PRNGKey(0),
 65       state: Optional[train_state.TrainState] = None

 66   ):

 67     self.geom = geom

 68     self.dtype = geom.x.dtype

File ~/anaconda3/envs/owl_vit/lib/python3.10/site-packages/jax/_src/deprecations.py:54, in deprecation_getattr..getattr(name)
 52   warnings.warn(message, DeprecationWarning, stacklevel=2)

 53   return fn

---> 54 raise AttributeError(f"module {module!r} has no attribute {name!r}")
AttributeError: module 'jax.random' has no attribute 'PRNGKeyArray'

You can try to change PRNGKeyArray to PRNGKey in the initializers.py file.

@Chi-chicken Thank you for the reply. Can you please tell where is the file and what did you change in it?

@DishantMewada The file path:  ~/anaconda3/envs/owl_vit/lib/python3.10/site-packages/ott/initializers/nn/initializers.py
You can use vim ~/anaconda3/envs/owl_vit/lib/python3.10/site-packages/ott/initializers/nn/initializers.py to edit initializers.py:64 (rng: jax.random.PRNGKeyArray = jax.random.PRNGKey(0) -> rng: jax.random.PRNGKey = jax.random.PRNGKey(0))

You are such a saviour @Chi-chicken. Thank you so much. It is working on my local machine. I know the original question from @kaninaba94 was related to Google Colab. But still, I confirm that this method works for the local machine if you have the same dependencies as me.


[BUG]: Impossible to launch twice a test on Keithley 2450 SMU

Description of the bug
I am not able to launch more than one time a script with my Keithley2450 connected through LAN.
Steps To Reproduce

Launch any examples scripts (for example: the solar cell one)
It works
Try again with the same scripts
I get the following message everytime

 in __clear_visa_output_buffer_and_get_idn
    raise SystemError(error_msg)
      visa_resource = <'TCPIPInstrument'('TCPIP0::192.168.XXX.YYY::inst0::INSTR')>
      idn_response = ''
      error_msg = "VI_ERROR_ABORT (-1073807312): User abort occurred during transfer.\n\tUnable to read data after 2s.\n\t\n\tPlease reboot or read/clear the VISA output buffer on your device and try again."
      old_timeout = 2000
builtins.SystemError: VI_ERROR_ABORT (-1073807312): User abort occurred during transfer.
        Unable to read data after 2s.
        Please reboot or read/clear the VISA output buffer on your device and try again.

Environment Information

Keithley 2450 connected through LAN

Additional Information
No response

@nfelt14 A way to avoid the problem was to add a visa_resource.write("*CLS?") between the two following lines to enforce the clearing.
https://github.com/tektronix/tm_devices/blob/093f7a905182088302f8682df8cc8cc918447999/src/tm_devices/device_manager.py#L1242-L1243

@lucasgrjn,If you have access to other series SMU, can you let us know whether this issue is observed on other series SMU as well?
This issue seems to be independent of the device being connected, will soon look into this and find a possible fix.

When system log massages gets full in the log table, it throws this kind of Visa communication Error. Either clear log messages manually if you have SMU device:  MENU ->System ->System Events-> Clear Log.
If you are running any scripts use *visa_resource.write("CLS") to clear the event log table.

@lucasgrjn, let us know whether clearing log messages manually is allowing the user to launch twice a test on the SMU 2450.


S3File Checksum

The current checksum method can return an incorrect MD5 checksum if the file was created using a multi part upload. I did a local fix on my own looking like that:
def get_s3_checksum(self, file_path: str) -> int:
    with self.open(file_path, "rb") as f:
        md5 = hashlib.md5()
        while byte_var := f.read(4096):
            md5.update(byte_var)
        readable_hash = md5.hexdigest()
    return int(readable_hash, 16)

Maybe someone has a better alternative.

  File "/usr/local/lib/python3.8/site-packages/s3fs/core.py", line 573, in checksum
    return int(info["ETag"].strip('"'), 16)
ValueError: invalid literal for int() with base 16: 'b6e64f35b2cb730d93770942a59b2efa-2'

The ETAG from multi-part files will break int(readable_hash, 16). This is a critical problem since multi-part is enabled by default for high-level APIs like s3 cp.  Can someone take a look the issue?

Can you please point to where in the code you found this line? The current implementation looks like:
    async def _checksum(self, path, refresh=False):
        """
        Unique value for current version of file
        """

        info = await self._info(path, refresh=refresh)

        if info["type"] != "directory":
            return int(info["ETag"].strip('"').split("-")[0], 16)
        else:
            return int(tokenize(info), 16)

You can see that the hyphen from multi-part-uploads is accounted for, and the gitlog says this hasn't changed in two years. Note that checksum gives you a unique value for your file, and is not necessarily the value you expect from any particular checksumming algorithm.

version:
pip3 list | grep s3fs
s3fs                      0.4.2

code
class S3FileSystem(AbstractFileSystem):
    """
    Access S3 as if it were a file system.

    This exposes a filesystem-like API (ls, cp, open, etc.) on top of S3
    storage.
   .....

    def checksum(self, path, refresh=False):
        """
        Unique value for current version of file

        If the checksum is the same from one moment to another, the contents
        are guaranteed to be the same. If the checksum changes, the contents
        *might* have changed.

        Parameters
        ----------
        path : string/bytes
            path of file to get checksum for
        refresh : bool (=False)
            if False, look in local cache for file details first

        """

        info = self.info(path, refresh=refresh)

        if info["type"] != 'directory':
            return int(info["ETag"].strip('"'), 16)
        else:
            return int(tokenize(info), 16)

When I build my dependencies, I used pip install mlflow boto3 s3fs without specifying the version. I think due to  pip3 choose to use an older version. After I manually change to newer version, it works. Thanks for the pointer

OK, so we close. I have no idea why pip would do that - I suggest pinning/specifying versions where possible, or using conda/mamba.


Missing docs on ping

Documentation on what the ping operation looks like seems to be missing from the docs. Would be nice to have it.

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

ID: 6e0efd5e-72d6-5124-f225-dd6426c9e004
Version Independent ID: 8ca96d3d-d9bc-fcd4-fe19-f10cd6fa3370
Content: Azure Relay Hybrid Connections protocol guide
Content Source: articles/service-bus-relay/relay-hybrid-connections-protocol.md
Service: service-bus-relay
GitHub Login: @clemensv
Microsoft Alias: clemensv

Hi @tomasr
Thank you for reporting this issue! We will investigate and make any necessary changes.

What ping are you referring to? WebSocket level ping/pong? Aside from that I don't think there is any other ping at this time.

I'm referring to this:

Ping operation
If the control channel stays idle for a long time, intermediaries on the way, such as load balancers or NATs may drop the TCP connection. The "ping" operation avoids that by sending a small amount of data on the channel that reminds everyone on the network route that the connection is meant to be alive, and it also serves as a "live" test for the listener. If the ping fails, the control channel should be considered unusable and the listener should reconnect.

There doesn't seem to be any actual description of what the Ping operation looks like on the wire.

@clemensv are the details for a "ping" operation missing from the Protocol Reference section?

@dlstucki
I'm creating an AzureListener, the connection is established sucessfully, but when I send Websocket Ping to azure relay server,  the control connection is disconnected by the server.
Am I doing something wrong ?
I also debugged with the official azure relay sample:
https://github.com/Azure/azure-relay/tree/master/samples/hybrid-connections/dotnet/simple-websocket/Server
After the connection is established,  it seems that the listener never sends a Ping to the azure relay server,   but the server just send Pong to the listener every 210 seconds(from wireshark).
Any comments ?

#please-close

#please-close


Floating image issue 10

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix #
Test URLs:

Before: https://floating-image-issue-10--takeda-ihs--hlxsites.hlx.page/drafts/floating-images
After: https://floating-image-issue-10--takeda-ihs--hlxsites.hlx.page/drafts/floating-images

Go ahead and add the necessary global styles to get the header, font, and page width management to this PR. That way the draft page for this auto-block looks like the live page.


cxk should move faster or slower when the mode changes.

yes, this is a good idea. However, it would make the harder mode become easier


Breaking sidekiq v6.5.4

The latest version of sidekiq is breaking the gem - https://github.com/mperham/sidekiq/issues/5462#issuecomment-1209985257
I am not sure what the fix is for this Gem.
Additional details on what is breaking can be found in the link above.

MAybe add in the file lib/sidekiq-status/server_middleware.rb, the line require 'sidekiq/component' above require 'sidekiq/job_retry'?

See https://github.com/mperham/sidekiq/issues/5462#issuecomment-1209985257

It seems that sidekiq have fixed it too https://github.com/mperham/sidekiq/commit/9d5a6d7fa7edba006db86fc520fbcd353005cf9c

I'm going to close this issue out as it has been addressed upstream.


Create UniversalWindowApp test application

Since they should be fully supported, should we add such an application?

What about this topic? Is it possible to write tests for UWP apps? Would be very nice :)

Yes it should be possible. This issue is just a note that there is no test UWP app in FlaUI itself.

Ah ok.
I will try to write some UI tests for UWP.
But anyway it would be very nice if there were an UWP testapplication.

Feel free to create a small application (even when it only has a few simple controls) and create a PR for it.

@Roemer
How do I launch a UWP App using the FlaUI library?
What path should I provide to Application.Launch method?

See https://github.com/Roemer/FlaUI/blob/master/src/FlaUI.Core.UITests/CalculatorTests.cs
 Application.LaunchStoreApp("Microsoft.WindowsCalculator_8wekyb3d8bbwe!App");

@Roemer Will check. Thanks!

@Roemer What about an application being developed?
What is the proper way to start it as opposed to .Launch(exePath) in WPF ?

@Roemer Excellent job.
If I might recommend that you should add a short How-To as to the correct app Id to be supplied to the LaunchStoreApp method. It took me a while to figure out what exactly I should put there

Probably won't happen anytime soon and there are now plenty of UWP apps in Windows that can be used for testing.


Add tools/miqversions.rb

Does the same thing as https://github.com/chrisarcand/miqversions with a few perks over using that repo:

We own it
Not written in Go
Extensible
No dependencies (portable ruby script)
In an easy place to find it

Cons (not really)

Requires ruby runtime (though, everyone should have it if they care about this info anyway)
About 4x slower to execute than the compiled Go program...

...but we are talking about 0.020sec (Ruby) versus 0.005sec (Go), so I don't think anyone really will notice.
We could turn this into a rake task as well (something like rake miq:versions), but I figured I would get a feel for changing directions instead of implementing things more than was needed.
Links

https://github.com/chrisarcand/miqversions

@miq-bot... go home, you are drunk...
$ git ls-files -s tools/miqversions.rb
100755 495828f27fc9a5f7f41dd6cd623c3fab6c25072e 0       tools/miqversions.rb

Also, the github UI shows that file as a "Executable File":
https://github.com/NickLaMuro/manageiq/blob/a86a6615aa620f4fc6c8be1110e5aed576474f85/tools/miqversions.rb
So not sure what you are on about...

@miq-bot... go home, you are drunk...
$ git ls-files -s tools/miqversions.rb
100755 495828f27fc9a5f7f41dd6cd623c3fab6c25072e 0       tools/miqversions.rb

Also, the github UI shows that file as a "Executable File":
https://github.com/NickLaMuro/manageiq/blob/a86a6615aa620f4fc6c8be1110e5aed576474f85/tools/miqversions.rb
So not sure what you are on about...

https://github.com/ManageIQ/miq_bot/issues/445

I like the idea! As Nick explained, it's easier if we own the code in our repo, and makes it easier for us to update.
I think hosting it somewhere under the MangeIQ GitHub in a repo would solve the no CFME in ManageIQ codebase right? But that would just mean it's an additional install, however it could be as simple as gem install manageiqversions right?!

I am :-1: on this change in the ManageIQ org, as we want to keep ManageIQ free of commercial versions.  However I would be very :+1: on this living the https://github.com/RedHatCloudForms org.  Let me know the repo name and I can create.

I am in the process of pushing a initial commit at the repo @Fryguy provided (thanks!) and taking into account some of the feedback given here.  At that point, we can create pull requests to add or remove things as desired.


normalize aws and gcp scripts

Description
With this commit, both the AWS and GCP scripts ...
Use similar output formatting
Pass shellcheck testing
Implement shellspec testing
Document (in comments) spec testing and script usage
Renames the spec test files to eliminate a redundancy
Motivation and Context
In addition, the AWS script ...
Correctly skips a member account when assume role fails
(caused by a return in a function not resulting in a continue in a calling loop)
How Has This Been Tested?
Tested with ...
shellspec spec/resource-count-aws.sh
shellspec spec/resource-count-gcp.sh
Types of changes

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)

Checklist

[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes if appropriate.
[x] All new and existing tests passed.

Sorry for the knock on pull request: holiday project.


Add a delegate_to_remote_methods macro

delegate_to_remote_methods
This PR adds delegate_to_remote_methods:
use ambassador::{delegate_to_remote_methods, delegatable_trait};

#[delegatable_trait]
pub trait Shout {
    fn shout(&self, input: &str) -> String;
}

pub struct Cat;

impl Shout for Cat {
    fn shout(&self, input: &str) -> String {
        format!("{} - meow!", input)
    }
}

pub struct BoxedCat(Box<Cat>);

impl BoxedCat {
fn inner(&self) -> &Cat { &self.0 }
}

#[delegate_to_remote_methods]
#[delegate(Shout, target_ref = "inner")]
impl BoxedCat {
    // `inner` can be defined anywhere: trait method or inherent method, in this crate or elsewhere
    fn inner(&self) -> &Cat;
}

It's identical to delegate_to_methods except:

it does not preserve the impl block it is applied to in its output
it checks that there are no method bodies or unused items in the impl block it is applied to

This allows it to be used in scenarios involving foreign Self types (without needing to create intermediary local traits to house the target methods) like the one described in #36:
use ambassador::{delegatable_trait, delegate_to_remote_methods};
use std::ops::{Deref, DerefMut};
use std::{sync::Arc, rc::Rc};

#[delegatable_trait]
pub trait Shout { fn shout(&self, input: &str) -> String; }

pub struct Cat;
impl Shout for Cat { fn shout(&self, input: &str) -> String { format!("{} - meow!", input) } }

pub struct Dog;
impl Shout for Dog { fn shout(&self, input: &str) -> String { format!("{} - wuff!", input) } }

#[delegate_to_remote_methods]
#[delegate(Shout, target_ref = "deref")]
impl<S: ?Sized + Send + Shout, T: Deref<Target = S>> T {
    fn deref(&self) -> &S;
}

pub fn shout(pet: &impl Shout) { println!("{}", pet.shout("hi")); }

pub fn main() {
    shout(&Cat);
    shout(&Dog);

    let a: Box<dyn Shout> = Box::new(Cat);
    let b: Arc<dyn Shout + Send> = Arc::new(Dog);
    let c: Rc<dyn Shout + Send + Sync + 'static> = Rc::new(Cat);
    // shout(&a); // Will error since we required `Send` above.
    shout(&b);
    shout(&c);

    let d: Box<Cat> = Box::new(Cat);
    shout(&d);
}

Other Stuff
There are a couple of other things in this PR:

tests and updates to the docs clarifying that #[delegate_to_methods] impl Trait for Type { ... } is permitted
some miscellaneous fixes and tests for the delegate(...) attr parsing logic

I can split these into their own PRs if required.
Open Questions
Is the name okay? delegate_to_remote_methods mirrors delegate_remote (which makes sense since, like delegate_remote does not require the type the impls are on to be local, this macro relaxes the requirement that the methods used to do the delegation be local) but it's perhaps misleading since delegate_to_remote_methods will also accept a remote type (as will delegate_to_methods if you place it on a block that implements a trait).
Currently the macro is pretty aggressive about checking that there aren't extra/unused items in the impl it is applied to; my thinking was that silently dropping these items could be a source of confusion. Do we want to relax these checks?

@dewert99 I've rebased and updated this PR now that #34 has been merged.

Thanks I just merged this


1 meg test

Closes #1332.

This change is

PR is added to the queue for testing as 1. task. (74424c9)

Test<PHONE_NUMBER>424c9(74424c9) has failed
See https://github.com/ThaliTester/TestResults/tree/9015740074424c9_1_meg_test_andrew-aladev/ for the fail logs

PR is added to the queue for testing as 1. task. (28b93f6)

Test (Success) 90157400 build is completed (28b93f6)
See https://github.com/ThaliTester/TestResults/tree/9015740028b93f6_1_meg_test_andrew-aladev/ for the logs

Test<PHONE_NUMBER>b93f6(28b93f6) has failed
See https://github.com/ThaliTester/TestResults/tree/9015740028b93f6_1_meg_test_andrew-aladev/ for the fail logs

PR is added to the queue for testing as 4. task. (84a7855)

PR is added to the queue for testing as 4. task. (a66ed30)

Test (Fail) 90157400 build is completed (84a7855)
See https://github.com/ThaliTester/TestResults/tree/9015740084a7855_1_meg_test_andrew-aladev/ for the logs

Test (Success) 90157400 build is completed (a66ed30)
See https://github.com/ThaliTester/TestResults/tree/90157400a66ed30_1_meg_test_andrew-aladev/ for the logs

PR is added to the queue for testing as 2. task. (9516b37)

Test 90157400a66ed30(a66ed30) has failed
See https://github.com/ThaliTester/TestResults/tree/90157400a66ed30_1_meg_test_andrew-aladev/ for the fail logs

Test (Success) 90157400 build is completed (9516b37)
See https://github.com/ThaliTester/TestResults/tree/901574009516b37_1_meg_test_andrew-aladev/ for the logs

Test<PHONE_NUMBER>16b37(9516b37) has failed
See https://github.com/ThaliTester/TestResults/tree/901574009516b37_1_meg_test_andrew-aladev/ for the fail logs

We need to work further with this test.


Decide whether we call it displayname or display_name

Different bits of API seem to use or omit the underscore. We ought to standardise on one or the other.

once we figure it out, we should write it down in https://github.com/matrix-org/matrix-doc/blob/master/meta/documentation_style.rst.

I think we're tending towards displayname.


Increasing Code Coverage on UI Components

Pull Request Description
What issue does this change request address? (Use "#" before the issue to link it, i.e., #42.)
Closes #161
What are the significant changes in functionality due to this change request?
This adds the capacity to programmatically test UI elements by using a keyword argument when running raven similar to the interfaceCheck keyword.

For Change Control Board: Change Request Review
The following review must be completed by an authorized member of the Change Control Board.

[ ] 1. Review all computer code.
[ ] 2. If any changes occur to the input syntax, there must be an accompanying change to the user manual and xsd schema. If the input syntax change deprecates existing input files, a conversion script needs to be added (see Conversion Scripts).
[ ] 3. Make sure the Python code and commenting standards are respected (camelBack, etc.) - See on the wiki for details.
[ ] 4. Automated Tests should pass, including run_tests, pylint, manual building and xsd tests. If there are changes to Simulation.py or JobHandler.py the qsub tests must pass.
[ ] 5. If significant functionality is added, there must  be tests added to check this. Tests should cover all possible options.  Multiple short tests are preferred over one large test. If new development on the internal JobHandler parallel system is performed, a cluster test must be added setting, in  XML block, the node <internalParallel> to True.
[ ] 6. If the change modifies or adds a requirement or a requirement based test case, the Change Control Board's Chair or designee also needs to approve the change.  The requirements and the requirements test shall be in sync.
[ ] 7. The merge request must reference an issue.  If the issue is closed, the issue close checklist shall be done.
[ ] 8. If an analytic test is changed/added is the the analytic documentation updated/added?

Note that on my local machine with the appropriate library installed (PySide), each file in the UI directory is now well above the 80% threshold. In total, there are 157 missed lines out of 2727 lines in these files. This gives a coverage of 94% for the UI folder of RAVEN.

@joshua-cogliati-inl There is now the issue of ensuring that wherever the code coverage is run, we should ensure that Pyside is installed and working as these tests do depend on PySide in order to get the coverage mentioned above.

Job Test qsubs:linux-hpc on 118bc85 : invalidated by @maljovec
Changes should not have affected the qsub tests. Invalidating to see if error is consistent.

Hm, what happens when PySide is missing?  (Getting on the coverage machines is probably just an email to Brian or Jason, and might already be there because of peacock)

If PySide is missing then these tests will try to run, realize they don't have the necessary libraries, and either skip in the test harness, or run without generating the UIs.
In both cases, the PostProcessors do not need the UI, but it injects a point where the user can modify the end results by interactively setting a scale parameter that basically dictates how many clusters exist in each case. So, if the UI is unavailable, then a warning is spit out to the user and the test cases will continue in "non-interactive" mode using whatever scale parameter is set in the input file.

Hm, this seems to have caused a failure in the coverage script:
https://www.moosebuild.org/job/88433/
Does it modify where the html files are located?


Updated Arduino BSP links on Quickstart for WisBlock core

Updated BSP Links on Quickstart.
Corrected the battery example of RAK11200.

@carlrakwireless
Thank you, I missed these mistakes before.


So you're just abandoning all your Azure DevOps customers now?

Yet another documentation section that assumes all your customers only use GitHub and the Azure CLI.
Once upon a time, Microsoft realized that catering only to Unix fatbeards was a losing strategy and so all the Azure documentation covered how to do things in the Azure Portal, with the Azure PowerShell module, with ARM/Bicep templates, and the Azure CLI.  CI/CD integration was assumed to be with Azure DevOps because of its tight integration into Microsoft developer tools.
I'm not going to retool all of our existing CI/CD and infrastructure-as-code ecosystem to use technically inferior tools.  That means features like Azure Web App for Containers and Azure Container Apps aren't of any value to me because I can't use them with Microsoft's first-class tools or the tools aren't documented at all.
That means I'm not going to give you any money for those features.
If I can't use your features, why am I using Azure at all?

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

ID: cd35737d-566a-1a5c-6b0b-c9fbc2cd4ed8
Version Independent ID: fae96436-9730-460b-8c8e-9064a02cd970
Content: Publish revisions with GitHub Actions in Azure Container Apps Preview
Content Source: articles/container-apps/github-actions-cli.md
Service: container-apps
GitHub Login: @craigshoemaker
Microsoft Alias: cshoe

Hello @cpdohert - Thank you for your feedback. We have shared this feedback internally with our product team.

Hello @cpdohert - Thanks again for your feedback! I have communicated your concerns internally to the Container Apps team.
Since this repo is more for driving improvements towards MS Docs and we didn't determine any immediate changes to this particular documentation upon reviewing your feedback, we will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


[Bug] [Android] Android deploy could not load .so files.

TVM version = 0.0.0
Android-platform = 17
Hi, I am trying to use demo android_deploy to embed mobilenetv2 into my android phone.
But it can not load .so file when the app runs.
Here is the code that generates the .so file
model = models.MobileNetV2(width_mult=1)
model.eval()
input_name = "data"
shape_list = [(input_name,(1,3,224,224))]
img = torch.rand([1,3,224,224])
model = torch.jit.trace(model,img)
shape_list = [(input_name,(1,3,224,224))]
mod, params = relay.frontend.from_pytorch(model, shape_list)
print("dumping lib...")
lib.export_library(output_path_str + "/" + "deploy_lib_cpu.so", ndk.create_shared)
print("dumping graph...")
with open(output_path_str + "/" + "deploy_graph.json", "w") as f:
   f.write(graph)
print("dumping params...")
with open(output_path_str + "/" + "deploy_param.params", "wb") as f:
   f.write(tvm.runtime.save_param_dict(params))

But once I put these files into asset folder and insert the app into android device. It shows that I could not load the .so file(the other two could be loaded).

@ kparzysz-quic
@argrento
@areusch
Could you please take a look at this?
I also find some reports on tvm discuss which may infer this problem. But they have no answer.

@HaoKang-Timmy do you have a little more detail about the problem? could you copy some terminal/LogCat output or a stack trace? We'll need to make sure we can understand whether we've reproduced your problem.

@HaoKang-Timmy do you have a little more detail about the problem? could you copy some terminal/LogCat output or a stack trace? We'll need to make sure we can understand whether we've reproduced your problem.

Hi, thanks for the reply. I solve this problem by changing jni folder to 'android_camera' jni folder. For 'android deploy, the obj folder generated by jni folder does not contain file 'libc++_shared.so'. Do you know why this happens?

Also, I use logcat to check the log. And it shows that there is no 'libc++_shared.so'. After I changed the jni files. It suddenly works. And obj folders in 'android_deploy' does not contain this file at all.


Crash when no correspondences can be found

I am trying to solve this problem,  As when there is a sudden move near the object tge program crashs.

Having same problem. Please, let me know if you can fix this issu. Thanks!


Social media icons have no hover effect

Feature idea.
In footer section, social media icons should have some hover effect it makes the page more dynamic and also redirection links need to be added.
pleas allow me to make these minor changes under Gssoc'23.

Hey @Guli93 you can work on this issue.

Hey @Guli93 you can work on this issue.

Thanks for assigning me this issue


Profiling

[ ] Is unit motion costly?
[ ] Is laggyness over time in my vscode caused by this or other extensions?
[ ] does it make sense to move away from generators and move to stepper functions

So far, when seeing slow performance speeds in VSCode, this has yet to be due to the any problems with selection utilities. Rather there are other extensions (e.g. for typescript or markdown language support) that are creating issues.

Profiling while specifically doing a lot of unit motions does not seem to cause any noticeable expenditure of time in selection utilities.


Include index_name in Weviate example.

A small improvement to the docs for the Weaviate example is to include index_name
client = Langchain::Vectorsearch::Weaviate.new(
    url: ENV["WEAVIATE_URL"],
    api_key: ENV["WEAVIATE_API_KEY"],
    index_name: "Document", # add this
    llm: :openai, # or :cohere
    llm_api_key: ENV["OPENAI_API_KEY"]
)

Since this is required by the API it makes sense to include
Reference: https://weaviate.io/developers/weaviate/quickstart/custom-vectors#schema

@jackpaulcollins Good catch!


Environment variables used in variable values are not evaluated

When an environment variable is used as part of a variable value, it is not expanded.
Example:
commands:
  demo:
    variables:
      path: "$HOME/demo"

    action:
      sh: echo $path

This results in the following being used verbatim in subsequent actions.
$HOME/demo

Potentially related, but this also applies to the ~ character.
demo:
  action: mkdir -p ~/demo

This results in a ~/demo directory being created from the current directory, and not the home directory.


Rust nightly removed the lifetime from Pattern

Closes https://github.com/rust-lang/regex/issues/1216

This fix is on crates.io in regex 1.10.6.

Great, thank you for the fast release!


Fix failing test

Conn converts param values to string, so assert should expect stringified values.

I think the test fail you're seeing is unrelated, and I'll try to dig into that at some point. That said, master is currently green. Can you help me understand how this test is failing for you?

$ mix -v
Erlang/OTP 22 [erts-10.6.4] [source] [64-bit] [smp:12:12] [ds:12:12:10] [async-threads:1] [hipe]

Mix 1.9.4 (compiled with Erlang/OTP 22)

plug 1.9.0 in mix.lock

Sorry, plug version is 1.10.1


Quick start guide does not work

Quick start guide references stark_private_key without initialising it.

This is working, Please make sure you are using the correct ETH ADDRESS, PRIVATE KEY and STARK PRIVATE KEY of the option (mainnet or testnet) you are trying to use.


rviz: 1.11.19-0 in 'indigo/distribution.yaml' [bloom]

Increasing version of package(s) in repository rviz to 1.11.19-0:

upstream repository: https://github.com/ros-visualization/rviz.git
release repository: https://github.com/ros-gbp/rviz-release.git
distro file: indigo/distribution.yaml
bloom version: 0.6.2
previous version for package: 1.11.18-0

rviz
* Indigo backports from kinetic-devel
  * Fixed bug where help.html wasn't being installed (#1218 <https://github.com/ros-visualization/rviz/issues/1218>)
  * Fixed crash when unchecking options of "triangle list" markers (#1164 <https://github.com/ros-visualization/rviz/issues/1164>)
  * Fixed crash when robot model not loaded before processing JointState msg (#1229 <https://github.com/ros-visualization/rviz/issues/1229>)
* Fixed BOOST_JOIN moc error with Qt 4 and Boost 1.58 (#1149 <https://github.com/ros-visualization/rviz/issues/1149>)
* Contributors: Antoine Hoarau, Johannes Meyer, Lucas Walter, dhood

Jade uses the same branch as Indigo for rviz. Even though Jade is EOL, from looking at jade/distribution.yaml it seems we still do a bloom release for it, is that correct @wjwwood ? (1.11.16 was the release it went EOL and it's at 1.11.18 now).

I would default to no, do not release into Jade anymore, despite the shared branch with indigo. We may have done that in the past on accident or for a specific reason, but I'd default to no.

The rosdistro checks now fail when a PR tries to modify an EOL ROS distro file.
We recommend not blooming for EOL distros so that the control files match the state of the last debs built  by the buildfarm.

ok, thanks!


Note taking solutions

Perhaps, more specifically, Obsidian?

Yes, I will assign @HallJoseph to this and schedule it in for next week.

Closed by mistake, forgot which issue I was looking at!


Handling exceptions in loop for application run

In the running application we want to keep the system running on best effort, so the loop should not raise a fatal exception due to conditions of the hardware or controllers. The proposed method reads/executes/commits as much as possible while logging issues, but another method would be to fail on first exception and capture that in the loop. There is config option to control behavior.
resolves #152

Sorry for prying whilst this is a draft, however, I really don't think we want to do this, at least not as proposed.
Normally, you add one outermost exception handler for logging, we can add two due to the app/SDK nature. So we'd need exception handling in the app (already handled for us) and then I think the other best place would be in evolver. device.Evolver.loop_once. That would then only need to look like the following:
def loop_once():
  try:
    ...
  except Exception:
    self.logger.exception(f"Error in loop:")
    raise

I don't think we ever want the need to keep limping on if an error occured, this would be very misleading, a warning would at least need to be raised, and even then it could be detrimental to experimental results as the logs would always need to be checked to determine whether results are valid.
Again, sorry for commenting whilst this is only a draft.

I thought about wrapping at the higher level, but I decided to do it this way so that if there is some transient issue with say the serial line on a single hardware we can log the error(which would be made available as warning to user) and keep trying others. After an exception in any of these we don't know necessarily that the system is all bad, next loop could get back to good functioning.
For the commit, that is true, what we could do is allow the user to decide if they want to stop committing actions (pause) or even abort on errors in the previous actions, which would require a manual action to recover. They already have the option to just raise errors and not skip (which would presently cause the system to do cease to function). Alternatively or in addition we could try all reads but have the option to exit the loop if any of them raised an exception.

After an exception in any of these we don't know necessarily that the system is all bad, next loop could get back to good functioning.

This is true, it could, however, the difficulty is how would you know that your experimental results weren't impacted by this?

Since this is arleady being reviewed, I will open it up.
How about this latest change? The user has the option to skip the rest of control loop on read error, which is probably the safest thing (so defaults to true). If things come back then the controllers can implement there logic based on latest state.
One reason to handle individually is that some transient error in a controller should not necessarily prevent another from keeping its experiment going just cause it was after in the list. The experiment conditions can possibly be recovered in subsequent runs by experiment logic - the real danger would be things like accidentally pumping too much and overflowing -  but not sure there is a good handle on that since no direct volume measurement, if this is a risk, user could choose to abort on failure (we can handle this when we handle stopping conditions).

Note:  We are going to ask the BU folks what are the desired behaviors in case of failure at read/logic/commit phases, before proceeding further with this PR.

Spoke with Ezira. They currently ignore transient errors in the serial comms from rPI to hardware (analogous to our read step), and have exception handling for experiment code where the evolver will stop (analogous to our control step and abort). They agreed that matching this behavior with the various options like to abort on control exception is desired as an escape hatch, but that generally an experiment can go on with low risk.
I've updated this PR with the option abort_on_control_errors - together with other options here I believe captures the desired behaviors.


Can't View Entire Circle at Full Size

I'm thankful that it's now possible to "Show Partial Circle" and "Show Entire Circle". Unfortunately, that leaves me without a way to see the details of each pony except for the ones that the website picks for "Show Partial Circle".
Please make it possible to zoom in and pan on the circle.
I've attached 2 screenshots; one showing the entire circle and the other showing the partial circle. Ignore the smile counters; I fiddled with the variables to quickly reproduce the problem.

If you click and hold (or swipe if you are on a tablet), you can rotate the wheel. You are intended to use this on the partial circle to get to the pony you want.


add amend options to amendment

Add support for AmendOptions.
Implementation taken from https://github.com/jmoline/zuora/commit/54cdde65a5de761162cbc6bbdd930082349582fb#diff-06f64771896a832e5ff4b574cdc40eef

lgtm


Make the commands' list in Ascending order

Resolve the conflicts and submit another PR

If you wish to continue working on this issue, submit another PR without conflicts


Dropdown arrow

Hello everybody!
Anyone knows how to use the "drop down arrow" in vue stripe menu? I checked up this small guide of variable list. link
but still does not work. May somebody can help and know more than me. Would be thankful!

Hi, I don't understand the question. You mean:

How to change (or replace the styles) in the dropdown arrow?
How to use dropdown arrow in other components?
Or is the dropdown arrow not displayed on the site?

Hi, thank you for fast respond. Yes i need to show the arrow for the dropdown navigation.  Like this one,

But not found how to do that. If you can help would be tahnkful!

You can try through the title slot:
<vsm-menu>
    <template #title="{ item }">

        {{ item.title }}
        \/

    </template>
</vsm-menu />

And rotate with css, like:
    transition: transform .2s;
}
.vsm-has-dropdown:hover .dropdown-arrow {
    transform: rotate(
90deg);
    display: inline-block;
}

Wow amazing thank you very much! Love it!


[$1000] There is a noticeable lag before chat turns from Unread to Read in the LHN for IOU reports/threads

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Send a money request from account A to account B
Open the newly created IOU thread from account B and notice the Unread status change to read on LHN

Expected Result:
The change from the Unread state to Read should happen instantly just as user clicks on the report on LHN
Actual Result:
There is a noticeable lag happening before the report changes from Unread to read in the LHN
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: Needs reproduction
Reproducible in production?: Needs reproduction
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/43996225/fc7c9361-3777-4525-9567-2706ea961122
Expensify/Expensify Issue URL:
Issue reported by: @nathan-mulugeta
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1684481982104969
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01aeb277d858be7b5c
Upwork Job ID:<PHONE_NUMBER>252942336
Last Price Increase: 2023-05-24

Hi team, can I resolve this task?

ok Thank @Nathan-Mulugeta

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/namletrung

Thanks @namhihi237 for the proposal! Could you attach a video result?

Proposal
Updated

Thanks @namhihi237 for the proposal! Could you attach a video result?

@mollfpr I had added video results on the proposal.

Going OOO and this is pretty active so needs management by bug0 team member, as per the new process to stay assigned but assign another team member, please can you keep a watch of this. I am OOO until 8th June. Thanks!

ugh sorry i unassigned @lschurr accidentally.

@mollfpr could you review the updated proposal?

Thanks for the ping @lschurr

However, this process, including the re-rendering time, may cause a slight delay. As a result, updates might not be instantaneous and the application could appear slower.

@namhihi237 I feel like the root cause is not complete. Why it’s working fine with regular report?

Proposal
Please re-state the problem that we are trying to solve in this issue.
There is a noticeable lag for chat to change from unread to read in LHN for IOU reports
What is the root cause of that problem?
In App/src/pages/home/sidebar/SidebarLinks.js
Lines 224 to 226
 if (ReportUtils.isIOUReport(report)) {
        return null;
    }

returns null for IOUReports and prevents updating lastReadTime. lastReadTime is empty. Since isUnread returns lastReadTime < lastVisibleActionCreated,  it does not change from false to true
What changes do you think we should make in order to solve the problem?
Remove this from Lines 224 to 226 of SidebarLinks.js
 if (ReportUtils.isIOUReport(report)) {
        return null;
    }

Then it IOU reports do not return null right away and the lastReadTime is updated and isUnread changes to false.
What alternative solutions did you explore? (Optional)
Retain the conditional but change the statement to
if (lodashGet(SidebarUtils.getOptionData(lodashGet(report, 'reportID')), 'lastReadTime') && ReportUtils.isIOUReport(report)) {
            return null;
        }

or
if (!lodashGet(SidebarUtils.getOptionData(lodashGet(report, 'reportID')), 'isUnread') && ReportUtils.isIOUReport(report)) {
            return null;
        }

When isUnread is true or lastReadTime is empty, the conditional is falsy then lastReadTime is updated from the following return statement.

Thanks @c3024! I still have some questions.

returns null for IOUReports and prevents updating lastReadTime. lastReadTime is empty. Since isUnread returns lastReadTime < lastVisibleActionCreated, it does not change from true to false

I don't think this is true. lastReadTime is not empty, because we set the value optimistically on the OpenReport API. If the chatReportSelector prevents the iou report from re-render, why does the IOU report re-render eventually?

Also, the text style unread on request money and send money has different behavior. In the request money report, the unread style is not disappearing even after opening the report.
https://github.com/Expensify/App/assets/25520267/f690053c-c2be-42f7-b740-b9158934a544

Thanks @c3024! I still have some questions.

returns null for IOUReports and prevents updating lastReadTime. lastReadTime is empty. Since isUnread returns lastReadTime < lastVisibleActionCreated, it does not change from true to false

I don't think this is true. lastReadTime is not empty, because we set the value optimistically on the OpenReport API. If the chatReportSelector prevents the iou report from re-render, why does the IOU report re-render eventually?

Thanks @mollfpr .
You are right. If the lastReadTime field exists it would have a value.
I think a new OptionRowLHN initially has no field of lastReadTime. It can get the lastReadTime from chatReportSelector or from the openReport call from Report in componentDidMount or componentDidUpdate in ReportActionsView. When chatReportSelector returns null, it gets the value from the openReport.
Returning null from chatReportSelector for IOU reports appears to be the reason for not updating status from unRead to read because just removing the line selector: chatReportSelector from
Line 291 - 300 from src/pages/home/sidebar/SidebarLinks.js
withOnyx({
        // Note: It is very important that the keys subscribed to here are the same
        // keys that are subscribed to at the top of SidebarUtils.js. If there was a key missing from here and data was updated
        // for that key, then there would be no re-render and the options wouldn't reflect the new data because SidebarUtils.getOrderedReportIDs() wouldn't be triggered.
        // This could be changed if each OptionRowLHN used withOnyx() to connect to the Onyx keys, but if you had 10,000 reports
        // with 10,000 withOnyx() connections, it would have unknown performance implications.
        chatReports: {
            key: ONYXKEYS.COLLECTION.REPORT,
            selector: chatReportSelector,
        },

fixes this issue.

@mollfpr could you review this last post?

Thanks @c3024; you're right this was fixed there.
@lschurr We can complete this, I just test it with the latest main.

If #19644 fixes this issue I think I am eligible for reporting bonus, according to the guidlines since this issue came first before the PR, am I right?

If the bug is fixed by a PR that is not associated with your bug report, then you will not be eligible for the corresponding compensation unless you can find the PR that fixed it and prove your bug report came first.

Hey guys, what do you think of this?

@Nathan-Mulugeta I believe you will be compensated. @lschurr will handle that.

Cool, so the bug is fixed and we just need to compensate @Nathan-Mulugeta for reporting, correct?

Just applied @lschurr

Great, payment issued. I think we can close this one out.


How to balance the cooler if multiple regions from same chromosome were captured?

Thank you very much for this great tool. We have a captured Hi-C dataset and probe contains multiple regions from the same chromosome, and several other chromosomes also have captured regions. How should we proceed in this situation?
For examples:
Chr1 100000	200000
Chr1 500000	600000
Chr1 800000	950000
Chr2 100000	250000
Chr3 200000	350000
Chr4 400000	550000

Because I noted that  if capturing multiple regions (not more than 1 per chromosome!).

Hi Linhua! Sorry for the relaid reply. Are you interested in the heatmaps outside of the captured regions? Because from what I see (e.g. in RCMC: PMID 37157000) many people just select individual continuous captured regions and do ICE correction for each of them separately (with standard cooler balance function). This also simplifies comparison with Hi-C data: you can also carve Hi-C submatrices corresponding to the individual captured regions and then ICE-normalize them which will give you values comparable to your C-TALE experiments.


bug: Can't use NavPush on MyApp class (app.ts)

Type: bug
Ionic Version: 2.x
Platform: all
I am injecting NavController:
import { Platform, ionicBootstrap, NavController } from 'ionic-angular';
And put it inside the class:
constructor(private platform: Platform, private navCtrl: NavController) { }
I have buttons on the menu, with (click)="goToPage(page)", this is the function:
goToPage(page): void {
	this.navCtrl.push(this.pages[page]);
}

This is the errors i get:
ORIGINAL EXCEPTION: No provider for NavController!
https://s21.postimg.io/ga91h3uc7/Screenshot_from_2016_09_13_22_24_19.png
So I added NavController to providers array:
@Component({
	templateUrl: 'build/app.html',
	providers: [NavController]
})

Now I don't have any errors when i enter app, but if I click on any button (that using goToPage function), this is the error I get:
ORIGINAL EXCEPTION: TypeError: this.navCtrl.push is not a function
Any suggestions? It's know issue?

You should not have to provide NavController.
On Parent Component (its likely your menu will be sitting here), you need to use
import { Nav } from 'ionic-angular';
and
@ViewChild(Nav) nav: Nav;
To control navigation.
A good example is the Ionic Conference App.
https://github.com/driftyco/ionic-conference-app

Hello! Thanks for opening an issue with us! As this seems like more of a support question i will urge that you ask this question on our forum. Thanks for using Ionic!

@cojj90: thank you for providing the right clue. I can confirm the this.navCtrl.push is not a function error is specifically caused by including NavController in the providers list. Removing it fixes the error.


Change whitelist URI handling - change URI to string in JSON

Situation
Currently (0.19.1) when a user defines a URI in a wrong way, e.g. something like
"https://myserver.example.com:8081/"
We got the situation, that URIValdiation will not return a correct message for the resulting 406..
Reason
We have defined URI as datatype for whitelist entries in JSON. So Jackson tries to create a URI object which is not possible for the wrong defined URI and will fail. This fail is done before the validation can happen and user got ne message...
Solution
We change the type to string in JSON, provide a utility method which returns URI, let the validation be done by validator and refactor usage.

Albert Tregnaghi<EMAIL_ADDRESS>Daimler TSS GmbH, imprint

@de-jcup is this still relevant?


Default showDetails in JsonProcessingExceptionMapper to true

After using Dropwizard for 5 months at work now, I discovered this option recently and it's saved hours of debugging and digging through stack traces since then.  It doesn't look like anything is gained by not providing the error field and what the validation issue is, so I think showDetails should be set to true by default.
Here is the class:
https://github.com/dropwizard/dropwizard/blob/ae9844a80a1ab273b7ccdb35d86fa624ee915d7a/dropwizard-jersey/src/main/java/io/dropwizard/jersey/jackson/JsonProcessingExceptionMapper.java
The same theme might also might apply to LoggingExceptionMapper, but I'm less sure that exposing e.getMessage() or something similar to the client would be the desired behavior.

Looking at the history of the file in question, the showDetails guard was introduced in 2014. @joschi, do you remember what the motivator was for this?
If I were to guess, I'd assume it was to prevent cases where a pathological client could induce large response payloads if a developer was longwinded when writing exception messages. Debatable whether this is a realistic concern, but this file's history shows a similar tack being taken with regard to logging: https://github.com/dropwizard/dropwizard/commit/9f2df976b2b1fda7dfb19fbdfd0c670914791f8a

@evnm The idea was to prevent information leakage to malicious clients.

@joschi How about default to showing the error message without the classpath and have the showDetails flag append the offending class?  I don't think an error message like this wouldn't be useful for anything malicious.
{
  "code": 400,
  "message": "Unable to process JSON",
  "details": "Unrecognized field \"foo\" not marked as ignorable"
}

I agree with @jeremygehrshg that providing some information regarding the failure wouldn't be "leaking potentially insecure implementation details" to a malicious actor. Having this available to be configured would greatly ease my integration efforts internally (e.g. having showDetail=true in a development and integration environment while leaving off in a staging and production environment). Instead I'm currently just re-registering this JsonExceptionMapper with jersey a second time and ignoring the warning.
I'm not sure if defaulting to "true" is really the correct approach, but definitely exposing a way for downstream users to set it up in their configuration files would be ideal.

Addressed in #1705

hopefully this configuration value will help you out @jeremygehrshg once 1.1.0 is upon us

It will, thanks @tmack8001


Out of Space banners

Add banner components for running out of space, for in the widget and in the file rows.
Issue: KBFS-4013

TODO:

[ ]  fix height issue
[ ]  get the out-of-space status from an RPC and put it in the state

Decided to wait for KBFS implementation of sending remaining space before parsing it into state.

Okay, this PR now contains both out-of-space and the modified offline banner, and mostly connects them [except for connecting the out-of-space data which we don't have yet anyway].
Issue: KBFS-3968
Issue: KBFS-4013


add support for object keys in queries.

Recreating this pull request w/o cruft from my old clone of the repo. Pulled the keys delete out of its if block.

This replaces PR #166
We've been using the Query with keys POSTed change (thanks for that one), but now we need support for object keys. I had to get the keys from the original opts since query is a map[string]string, not map[string]interface{}.

Thank you for the contribution!