dev: add tests and CI workflows

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 6
- Added 17

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 6
- Added 27

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 6
- Added 27

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 203
- Added 77

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 203
- Added 57

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 203
- Added 62

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 203
- Added 56

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 203
- Added 27

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 204
- Added 25

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 204
- Added 25

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 204
- Added 22

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 204
- Added 19

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 204
- Added 20

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 204
- Added 12

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 204
- Added 10

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 204
- Added 8

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 204
- Added 8

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 204
- Added 5

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 204
- Added 5

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 204
- Added 1

See the complete overview on Codacy


How to select from the UIAPickerWheel

I have no success using the following code, it gives me the tag name exception.
WebElement picker = webDriver.findElement(By.xpath("//UIAPickerWheel[contains(@value,'Month')]"));
Select select = new Select(picker);
select.selectByValue("1 - January");
And/Or please let me know if there is a way to scroll/swipe the UIAPickerWheel...
Any help is really appreciated!!!
Thank you :)

@federicoviscomi , Please help for below Issue:
I try to select a random month with year, I m unable to select it as scroll/swipe is not working.
PS: It always try to select a current value only as I understand UIAPickerWheel isn't having list of values, so I try to scroll it, but could luck, any valuable suggestion will be appreciated.
Below is my code snippet with image:
//Click on select from year dropdown		driver.findElement(By.xpath("//UIAApplication[1]/UIAWindow[1]/UIAScrollView[1]/UIATextField[7]")).click();
//Get number of UIAPickerWheel's
List DatePicker = driver.findElements(By.xpath("//UIAApplication[1]/UIAWindow[1]/UIAPicker[1]/UIAPickerWheel"));
for (int i = 1; i <=DatePicker.size(); i++) {
IOSElement we = (IOSElement) driver.findElementByXPath("//UIAApplication[1]/UIAWindow[1]/UIAPicker[1]/UIAPickerWheel["+(i)+"]");
driver.scrollTo("June");
we.sendKeys("June");
}

any input will be appreciated.

@shabana-parveen
You can use Horizontal and Vertical swipe functionality as scroll method is deprecated update your Java client jar file.


When visiting an app, ensure the user is connected on the Pod provider frontend with the same user ?

This is not necessary at the moment but it sometimes creates confusion, since you can be logged as Alice on an app, and as Bob on the Pod provider.
For example, if Alice's app need to be upgraded, she will be redirected to the Pod provider.
If she is logged there as Bob, it will upgrade Bob's application registration but not Alice's.
If Alice is not logged at all on the Pod provider, it will require her to login first before the upgrade, which adds confusion.
This won't happen when we implement global logout but maybe there is something quick to do in the meantime.

Do you think this is urgent though? I can imagine this is annoying during testing but for end-users this is not the most common pitfall.

Indeed I wanted to document this problem. If there is no quick fix, we can wait for the global logout.


Game crashes on shearing sheep with kama

the game throws a ClassCastException. the shearing appears to have happened after loading the game after the crash. the xp doesn't go through, however.
to reproduce, just use a kama to shear a sheep.
logs:
crash-2023-06-13_16.15.27-client.txt
crash-2023-06-13_16.26.25-client.txt
crash-2023-06-13_16.31.15-client.txt

Can confirm this bug. It occurs both in single-player and multiplayer. In MP, the server is unaffected and other players can see the wool drop after shearing.


Transient textures.

I'm probably missing something, but there doesn't seem to be a way to hint that a texture is transient. This would probably be a Bind flag.

@quadrupleslap in what sense do you want a texture to be transient?
Frostbite uses transient targets in this presentation. For them to be supported, we need to separate memory allocation from resource assignment to memory. This is addressed in the actively developed CoreLL logic, see https://github.com/gfx-rs/gfx/issues/1102


Fix testing and linting

Now we build with every python version which we support on approval and lint on every push (pull_request was redundant)
Note: As I tested a bit this probably should get squashed merged

Tests are failing because the few tests I created in tests/test_smb_database.py are quite old. Might want to just disable running those tests for now, until we can actually get a proper (non-e2e) functional testing.

Ah yeah i just wanted to have something that checks that installations are working on every platform and with every supported python version. Maybe just disable the poetry run pytest for now? If we want to test installations i will add caching as well

Yeah, I would get rid of that part, and just do the install. Maybe run the binary with -h after, to make sure stuff loads?

Sounds good!


Operation Timed Out

Platform : OPenshift 3.11
Vmware Host Version
[vmware exporter container logs.txt]
: 6.5.0
I used kubernetes configuration file to  deploy vmware exporter on Openshift 3.11 .  I am able to successfully deploy it on Openshift 3.11 . I am having a vmware export container running smoothly  without any resource issues.
I have exposed service on port 9272 & also created route.  I am unable to access metrics over route and getting attached error ((https://github.com/pryorda/vmware_exporter/files/4613585/vmware.exporter.container.logs.txt))in running container .
Note: I am successfully able to extract metrics in standalone docker container on my local vm.

Any updates on this issue ?

I have resolved this issue. Firewall was not opened to allow connection between openshift nodes and vmware host.
So opened firewall.


29 move consts to use rsmath constants

now using constants from rsmath.
Some tests are broken and cant tell if related to changes I made.
One that may be affected by my changes:
tests/beam_plot_test.py - AssertionError: assert 75365 < 45000

Multiple RadiaSoft repositories depend on rsbeams.
What is your protocol for identifying the relevant dependencies and the subsequent plan for correcting any breakage?

@gurhar1133 please fix the code that used rsbeams.physics.rsconst.

I just checked out master to confirm and the tests that are broken on this branch are the same that are broken on master.
So it appears my branch did not break things in this repo.
I am down to help with fixing them, but I may not have enough context to do so on my own.
I will fix the other repos that import rsphysics.rsconst though


Import styled components as "styled" rather than "$"

Description :sparkles:

In VSCode, importing styled-components in a different name other than styled prevents the extension from working properly:

There's no syntax highlighting?
Syntax Highlighting is specifically made to work with styled so make sure your default import is styled and nothing else.
See: https://github.com/styled-components/vscode-styled-components/issues/118#issuecomment-833007295

Therefore, I renamed the default import from $ to styledbut kept the components name as they are. e.g:
export const $SecondaryButton = $(Button)
export const $SecondaryButton = styled(Button)
Testing :alembic:

Everything should works as expected

Codecov Report

Merging #179 (d434af5) into develop (1fc42e5) will decrease coverage by 6.21%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop     #179      +/-   ##
===========================================
- Coverage    67.81%   61.59%   -6.22%
===========================================
  Files          273       50     -223
  Lines         5741      651    -5090
  Branches       433       87     -346
===========================================
- Hits          3893      401    -3492
+ Misses        1811      232    -1579
+ Partials        37       18      -19

Impacted Files
Coverage Δ

...loyer/src/components/companyInfo/CompanyInfo.sc.ts
100.00% <100.00%> (ø)

backend/benefit/common/tests/conftest.py

backend/shared/shared/common/tests/factories.py

frontend/benefit/applicant/src/utils/forms.ts

frontend/shared/src/styles/main.ts

frontend/shared/src/i18n/i18n.ts

...kesaseteli/applications/migrations/0001_initial.py

frontend/shared/src/components/content/Content.tsx

frontend/shared/src/components/stepper/Step.tsx

...react-query/get-default-react-query-test-client.ts

... and 200 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 1fc42e5...d434af5. Read the comment docs.


Board Review: namespace for AppComplianceAutomation management plane SDK

Thank you for starting the process for approval of the client library for your Azure service.  Thorough review of your client library ensures that your APIs are consistent with the guidelines and the consumers of your client library have a consistently good experience when using Azure.
The Architecture Board reviews Track 2 libraries only. If your library does not meet this requirement, please reach out to Architecture Board before creating the issue.
Please reference our review process guidelines to understand what is being asked for in the issue template.
Before submitting, ensure you adjust the title of the issue appropriately.
Note that the required material must be included before a meeting can be scheduled.
Contacts and Timeline

Service team responsible for the client library:
Main contacts:
Expected stable release date for this library:

About the Service

Link to documentation introducing/describing the service: https://servicetree.msftcloudes.com/#/ServiceModel/Home/b58f0a95-9bd2-403f-81d9-fe7dec477f6e, https://learn.microsoft.com/en-us/microsoft-365-app-certification/docs/acat-overview
Link to the service REST APIs: https://github.com/Azure/azure-rest-api-specs-pr/tree/RPSaaSMaster/specification/appcomplianceautomation/resource-manager
Is the goal to release a Public Preview, Private Preview, or GA? for Public Preview

About the client library

Name of client library:
.Net: Azure.ResourceManager.AppComplianceAutomation
Java: com.azure.resourcemanager.appcomplianceautomation
Python: azure.mgmt.appcomplianceautomation
Go: sdk/resourcemanager/appcomplianceautomation/armappcomplianceautomation
JS: @azure/arm-appcomplianceautomation

@kyle-patterson @ronniegeraghty since we have 3 services asked for namespace review can we schedule a combined meeting to review these namespaces together asap?

@kyle-patterson @ronniegeraghty AzureLoadTest also need a namespace review, we now have 4 I think

@kyle-patterson @ronniegeraghty since we have 3 services asked for namespace review can we schedule a combined meeting to review these namespaces together asap?

We approved:

.Net: Azure.ResourceManager.AppComplianceAutomation
Java: com.azure.resourcemanager.appcomplianceautomation
Python: azure.mgmt.appcomplianceautomation
Go: sdk/resourcemanager/appcomplianceautomation/armappcomplianceautomation
JS: @azure/arm-appcomplianceautomation

Recording (MS INTERNAL ONLY)


UCBasicUsage3 = not set up in correct order

move receivedContext up too above the listener (line 56) in fdc3.user-channels.ts

Also in UCBasicUsage4.


Add the e2e tests so that we can cover the integration test scenarios

This PR adds integration test to verify whether knativeserving is ready
and the deployment is able to be recreated.
Closes: #43

/test pull-knative-serving-operator-integration-tests

/lint

/retest

/test pull-knative-serving-operator-integration-tests

seems like a rebase/vendor update is due.

@jcrossley3 https://github.com/knative/serving-operator/issues/104 I opened an issue to track that feature. We need to change that in future.

/lgtm


Failed to bypass cloudflare

Steps to reproduce

Select PhoenixScan or Mangas-Origines (not the NSFW one) as sources
You get the error (and a text with already saved ones)

Expected behavior
No error, maybe put a popup to manually confirm the cloudflare box, like the web page open so you can confirm it
Actual behavior
Error 403 : Failed to bypass cloudflare
Mangayomi version
0.2.9
Device
iPhone 13 Pro
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with an extension, I should be opening an issue in the extensions repository.
[X] I have updated all installed extensions.

same


Compatfix/collection self

bug: osdi service was not returning self links on collections, nor top level curies.  This also includes a better fix for returning 200 on empty collections based on @shaisachs factoring concerns in #7

Definitely thought you were misspelling "query" until I looked at the code.  What is a curie?

(If you can fix the url thing the rest looks good.)

Curie is a hypermedia thang, it's the mechanism for prefixing like osdi:people or van:shift.  The curie definitions identify the prefix and the URL for documentation on what it means.  These docs links are exposed in the HAL browser too.
you'll see that at https://vangogh.splitatom.com


data argumentation

Hi, thanks for your code, it's elegant, and I learned a lot from it,
I have some questions when I read your paper,

I noticed that you do a lot of data argumentation when training, and I wonder how much this impacts the performance in semi-supervised learning?
In my research field, I can not do data argument for samples, and I just have a few like one or five samples per class, I wonder the keys and values define in memory could learn the semi-supervised, and how could we guarantee the memory updated with just very few labeled samples?  think about this, in extra situation, we just have one sample, and I update the keys and values with this only sample, please asking your advice may this work?

Thank you.
Best wishes.

Hi,
Here are my understanding regarding your questions:

Data augmentation is beneficial and improves SSL in classification - avoid overfitting.
A very few samples (e.g. one sample) are unlikely to learn good representations for keys (i.e. class-level feature representations).


Use minimum access profile in UTIL_Profile when available

Critical Changes
Changes
Issues Closed
Community Ideas Delivered
New Metadata
Deleted Metadata

GUS Work Item: W-8543149

GUS Work Item: W-8543149


AssetsController #partial_tree is broken

I'm not sure we still use this route
Probably the solution is @json      = account.partial_trees.last.to_json.html_safe

@yssavchuk If it is not used anywhere we can remove this action and route.

Or if it is used we need to make it work.

Closed by #741.


Update metrics handler in SMA to return RAM and/or CPU usage.

Update SMA to return CPU in addition to memory metric on getMetrics function in https://github.com/edgexfoundry/edgex-go/blob/master/internal/system/agent/services.go.
Subtask of #832. This service should be written so that the metrics service can serve requests for:

RAM only
CPU only
RAM and CPU

Additionally, this should be implemented with the abstraction layer defined in #1076 and should be implemented with an eye toward extensibility for the future.

Closing. It was determined that CPU and RAM do not need be separated. If that decision changes, this issue can be reopened.


Build script

Build script contains absolute paths to go binary.
Use $GOROOT or $GOBIN env variable to determine location of go installation.

Done


Support SwiftPM using precompiled binary target

It does not seem like SwiftPM is capable of compiling libgit2 by itself, so I am using the new binary target functionality in Swift 5.3 beta. It is working, but with some major caveats:

It requires Swift 5.3, currently in beta.
You cannot currently use Xcode (not even 11.5 beta 2) to open or import SwiftGit2 as a Swift package , because it uses Swift 5.2 for this even if the 5.3 toolchain is selected. This will be fixed in a future version of the Xcode beta. (Update May 19: it seems like Xcode 11.5 will ship with Swift 5.2.something. It’s looks like there won’t be an Xcode with Swift 5.3 package support until WWDC)
It is currently only working for macOS. I will look into iOS support when Xcode starts using Swift 5.3 internally.
I tried to make as few changes as possible to the Xcode project, but I had to replace the test dependency ZipArchive with Zip because the former does not work from a Swift package. This also meant I had to set the iOS deployment target to 9.2 as required by Zip. If this is a problem I can try to find another solution.
When using SwiftGit2 as a dependency in my test package the first build took several minutes, because SwiftPM clones all the submodules. It doesn't need any of those, but there doesn't seem to be any way of preventing SwiftPM from doing this.

And I had to change the way git_libgit2_init() is automatically called, because SwiftPM can't use Objective-C code, and module initialisers are not available in Swift. I am using a lazy static property on Repository instead, and calling it from all its class methods. I am assuming any user will have to call one of these methods first before doing anything else with this library.

Has anyone had a chance to look at this?

Has anyone had a chance to look at this?

As this is relying on Swift 5.3 I'd say we should wait before merging.

I agree. Let’s hope Swift 5.3 is released before the fall.

@kareman Did you try to wrap libgit2 in another Package, like the objc.io folks did with cmark for their CommonMark Swift package? https://github.com/objcio/commonmark-swift/ It's depending on a system library installed via homebrew instead of a git submodule, though.

Yes that would work too for macOS and other desktop OSes. But not for iOS/iPadOS/watchOS/AppleTVOS. Because brew will only install libgit2 compiled for Intel/macOS.
I haven’t tested my solution for iOS yes because Xcode will need to be able to import Swift 5.3 packages first to test it.

@kareman Hello, thanks for writing this! I was interested in trying your fork to see if it worked on a macOS app I'm working on and when I try to build I get No such module 'Clibgit2' error in CheckoutStrategy.swift. I'm running XCode 12 Beta on 10.15. Would you happen to know why I would get that?

I got the same error from within Xcode beta 2. I didn't have time to look into it closer before my vacation. It worked fine when running swift test from the command line so I think it is Xcode-related. It would be interesting to know if other projects have similar errors when importing precompiled frameworks when opening projects as swift packages in Xcode 12.
Also note that this fork did not work in Xcode 11 either when opened as a swift package, though that produced different errors.

Opening this as a Swift Package in Xcode 12 beta 4 now works (for macOS, see below), presumably because of this:

Resolved
Fixed an issue where Xcode didn’t properly copy headers while processing an XCFramework. (64754387) (FB7786076)

It works for both platforms when opening the Xcode workspace.
When opened as a swift package in Xcode, it only works for macOS. The precompiled framework at External/Clibgit2.xcframework does not include iOS or simulator binaries of libgit2. When I try to include the simulator with
xcodebuild -create-xcframework -library External/libgit2.a -headers External/libgit2/include/ -library External/libgit2-ios/iphonesimulator14.0-x86_64.sdk/lib/libgit2.a  -headers External/libgit2-ios/iphonesimulator14.0-x86_64.sdk/include/  -output External/Clibgit2.xcframework

, Xcode says "No such module 'Clibgit2'" for both macOS and iOS. This also happens if I only include the iPhone simulator build in the precompiled framework.

@kareman Thank you so much! I was just about to go through the effort of pulling it in as a manual dependency when I saw the commit. 🙏

When opening as a Swift Package in Xcode 12 beta 4, you can now build for macOS, iOS and iOS simulator.
Turns out you have to build the libgit2 binaries with Xcode, even though it uses the same ./script/update_libgit2_ios script, to be able to combine them all in an XCFramework. I have no idea why.
The binaries seem a bit large though, the entire precompiled and checked-in framework is now 33.6 MB. And the iOS binaries are twice the size of the macOS one.

I still have the Clibgit2 issue when Archiving. It's fine when debugging, but when I want to build my app for production, then I get an error. I'm using Xcode 12 GM.

I get a code signing issue with Xcode 12 GM during a debug build:
CodeSign /Users/thrashr888/Library/Developer/Xcode/DerivedData/Scour2-gtfsnrlizgmwboeqjigigxvfagub/Build/Products/Debug/Scour2.app (in target 'Scour2' from project 'Scour2')
    cd /Users/thrashr888/workspace/hikari-passport/Scour2
    export CODESIGN_ALLOCATE\=/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/codesign_allocate

Signing Identity:     "Apple Development: XXXXXXXXXX (XXXXXXXXXX)"

    /usr/bin/codesign --force --sign 27123B160A4834A8837500DE2188512C910B645C --entitlements /Users/thrashr888/Library/Developer/Xcode/DerivedData/Scour2-gtfsnrlizgmwboeqjigigxvfagub/Build/Intermediates.noindex/Scour2.build/Debug/Scour2.build/Scour2.app.xcent --timestamp\=none /Users/thrashr888/Library/Developer/Xcode/DerivedData/Scour2-gtfsnrlizgmwboeqjigigxvfagub/Build/Products/Debug/Scour2.app

/Users/thrashr888/Library/Developer/Xcode/DerivedData/Scour2-gtfsnrlizgmwboeqjigigxvfagub/Build/Products/Debug/Scour2.app: code object is not signed at all
In subcomponent: /Users/thrashr888/Library/Developer/Xcode/DerivedData/Scour2-gtfsnrlizgmwboeqjigigxvfagub/Build/Products/Debug/Scour2.app/Contents/Frameworks/libgit2.a
Command CodeSign failed with a nonzero exit code

I'll take a look when I get back to my computer on Monday.

Try adding "--deep" to the "Other Code Signing Flags" build setting in Xcode.

@arguiot I also get an error about not being able to import Clibgit2 when archiving, even in Xcode 12.2 beta 1. I haven’t been able to find much about this elsewhere. Everything works fine when running the app, including in release mode.

@arguiot I also get an error about not being able to import Clibgit2 when archiving, even in Xcode 12.2 beta 1. I haven’t been able to find much about this elsewhere. Everything works fine when running the app, including in release mode.

Hi @kareman , did you resolve this issue? I've got the same error when I'm trying to Archive the project with SPM dependency.

Alrighty, if you need to archive and distribute your app with Xcode 12, a solution I found was to install ObjectiveGit, then copy all the source of SwiftGit2 (from this PR) into your code. A few changes are required to make it work, but Xcode warnings will help you.

@palyvoda I haven't had much time to look into it (looking for a new job takes up most of my time), but I tried again today and it seems like an Xcode 12 bug when archiving applications that import Swift Packages with binary targets. There are others with the same problem. I tried creating a new application and importing this Swift package with a binary target, and it had the exact same problem.

@arguiot if you just need this project to compile in Swift 5.3 you can use PR https://github.com/SwiftGit2/SwiftGit2/pull/176

Hello @kareman thanks for the mention on Apple's developer forum :)
For info we didn't find a fix for this issue yet, but I've submitted a bug to Apple with a sample code to reproduce the issue. Hope we'll receive an answer with a fix 🙏

@kareman For the record here is the link to the sample project if it can help: https://github.com/zocario/swiftpm_archive_binary_target

Nice. To get around the code signing issue, did you try adding "--deep" to the "Other Code Signing Flags" build setting in Xcode? That worked for me.

@zocario Nice. To get around the code signing issue, did you try adding "--deep" to the "Other Code Signing Flags" build setting in Xcode? That worked for me.

Yes it works with --deep thanks for the advice :)
Just need to find out a way for archive now!


closed game at random times (error c2-12828-1) sometimes in consecutive battles

The game works really well at first, it even arrives in the first city, then the closings start happening at random moments, sometimes when we get into some fights or go through doors

Fix config.txt
*!FF5000000
*ALL

I extracted from my psvita some dumps of the random crashes, hope it helps
Dumps.zip

Started playing this over the last few months myself. Also have some coredumps if needed. I've hit a wall at "The Guardian Tree" wherein you enter, are brought into a cutscene, then a battle, then a following cutscene, followed by another battle. The second battle so far has triggered a crash 100% of the time before actually entering.
Attempted overclock with psvshell on a PSTV, as well as running the game at 720p. PSVshell is unable to set the processor to 500mhz setting for this game, and 720p video output resulted in no change. Based on my experiences so far, it appears the frequent crashes occur around cutscenes, primarily when transitioning from a cutscene to a battle. I mitigated this usually by using a quicksave (or the natively generated ones) and closing the application and re-opening it after cutscenes, and continuing from there.
I was able to get the psvshell monitor up during this part of the game where the crash was occurring, and have no idea how relevant or useful the info is, but the fourth core would spike upward of 80% usage at the very moment the crash occurred.
I am well aware there is little likelihood this project will be returned to after so long, but wanted to share findings on the off chance this helps at all.

The game crashes in some places few times in a row after starting the battle,I ‘ve been playing the game for 10 hours and during that period of time my game crashed about 20-25 times with error c2-12828-1 error,the game is playable and save states saved me a lot of time,but it irritates me a bit


(ITTN-071) rke Upgrade Playbook

The following document details step by step the procedure to upgrade the current version of k8s on a cluster with the rke tool.

Hi Carlos, ITTN looks good if I had one suggestion to make. Is to add a Topic Overview of all the contents covered.


Add content_sets.yml

Please rebase.
Are both versions (1 and 16) deprecated? I can't find it anywhere...

Ah right, I forgot thermostat is deprecated. Closing.


Moving over to a single top level package evergreen or evergreen-ui

Read the complete background on this issue here: #103.
After having tried a Lerna mono-repo, we found that the benefits don't outweigh the downsides. The main issues are:

A lot of boilerplate to import components
Keeping dependencies in sync is a pain for the consumer
Publishing packages gets confusing
Harder to setup code splitting

That's why we are in the progress of moving towards a single evergreen-ui package.
I have made PoC codemod to move over current projects that use Evergreen. Which mainly internal projects at Segment.

We'll rename the current package folders from evergeen-colors into colors.
Test and run a codemod script on the current projects using Evergreen

#125


Problem about plotting policy tree

Please make sure that this is a bug.
System information

OS Platform and Distribution (e.g., CentOS 7.6):Win10
Python version:3.9.12
YLearn version:0.1.2
Other Python packages(run pip list):

Describe the current behavior

I wanna use API why.plot_policy_tree(), and I specify control, like:
why.plot_policy_tree(whatif_data,  max_depth=2, treatment = 'CRIM',control = 3)
Then I  get bug which show as below:
invalid index to scalar variable

Describe the expected behavior

It should output the different leaf nodes and causal effect value compared to control value I specify.(like I specify control = 3)

Standalone code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate
the problem. If possible, please share a link to Jupyter notebook.
Are you willing to submit PR?(Yes/No)

Yes

Other info / logs
Include any logs or source code that would be helpful to diagnose the problem.
If including tracebacks, please include the full traceback. Large logs and files
should be attached.

Can you "Provide a reproducible test case that is the bare minimum necessary to generate
the problem. If possible, please share a link to Jupyter notebook."?

OK, I share a notebook use open source dataset like "Boston House Price", please check~
@.***
From: LiXuefeng
Date: 2022-08-03 13:19
To: DataCanvasIO/YLearn
CC: Vihagle; Author
Subject: Re: [DataCanvasIO/YLearn] Problem about plotting policy tree (Issue #32)
Can you "Provide a reproducible test case that is the bare minimum necessary to generate
the problem. If possible, please share a link to Jupyter notebook."?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

But, I can not find the notebook you shared here.  Upload it as a attachment from github website please.

Sry, I just send a email with attachment, maybe I can send notebook to your email<EMAIL_ADDRESS>

Thank you for your notebook, I reproduced the issue.

I fix this issue in the main branch.
At current timestamp, call plot_policy_tree with control=None please.

OK, looking forward to seeing the next version of ylearn update~


Python3 markdown

Can you please link to the package database entries for the new keys? That makes it easier to review. Thanks.


Build a browser testfarm

This is just a note for me to do on a local computer:
I just introduced a browser compatiblity table in the readme. It would be much easier to maintain if I could run all the tests in a bunch of browsers in a single run. Should be done on Windows (perhaps a vm) to have IE and Edge in the suite.

Tests now run in node (server). There should be tests with selenium or browserstack but for now a browser farm is pretty useless.


Check that IV length is not less than zero

As EVP_CIPHER_CTX_get_iv_length indicates failure with -1, this error
should be processed. Also the result of this function shouldn't be
assigned to an unsigned variable.
Reviewed-by: Paul Dale<EMAIL_ADDRESS>Reviewed-by: Tomas Mraz<EMAIL_ADDRESS>Reviewed-by: Hugo Landau<EMAIL_ADDRESS>(Merged from https://github.com/openssl/openssl/pull/18891)
Signed-off-by: Dmitry Belyavskiy<EMAIL_ADDRESS>
Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

Backport of #18891 to openssl 3.0

@t8m please reconfirm

This pull request is ready to merge

Merged. Thanks!


missing word in feature-workshop-prep instructions

should be "the service may still be..."

Thanks, Chris. Fixed.


Use URL of repository

This PR uses the modified the action (once badge #20 PR is merged) and passes the REPOSITORY input.

Closing this PR to avoid confusion with PR #4


Server doesn't listen on IPv4 on Raspberry Pi

I installed HA Bridge on a Raspberry Pi 2 today (latest Raspbian Jessie Lite) and installed the java 8 SDK. In troubleshooting the startup, I realized that while it was listening on all interfaces, it was only listening on IPv6. I found a reference on the Vera forums that I need to add:
-Djava.net.preferIPv4Stack=true
On the line that starts the bridge for Raspberry Pi. It might be useful to add this in the documentation.
ExecStart=/usr/bin/java -jar -Dconfig.file=/home/pi/habridge/data/habridge.config -Djava.net.preferIPv4Stack=true /home/pi/habridge/ha-bridge.jar

Yes, I see that now as well. Will need to see what sparkjava is doing as of some version around 2, this code is called:
        String ipV6Stack = System.getProperty("ipV6Stack");
        if(ipV6Stack == null || !ipV6Stack.equalsIgnoreCase("true")) {
        	System.setProperty("java.net.preferIPv4Stack" , "true");

Well, looking into the internal set and it is not valid after the vm starts, it must be passed on the command line. Thanks, will update the documentation.


ENH: adapt antsai paramaters from cli

(Hopefully) takes a series of arguments on the command line to adapt antsai parameters on the fly

Hello @eilidhmacnicol! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file nirodents/workflows/brainextraction.py:

Line 39:12: E231 missing whitespace after ','
Line 39:14: E231 missing whitespace after ','
Line 290:17: E265 block comment should start with '# '

Can confirm this change doesn't break the CLI or the workflow. Haven't used it to complete a workflow, but quietly confident this does the job.

Deleting branch now I can confirm that it does work in docker container 👍🏼


sql generation of count_values is too large

the amount of code generated by count_values seems questionable
In [16]: print(compute(result.device_type.count_values()))
SELECT alias_3.device_type, count(alias_4.device_type) AS count
FROM (SELECT click.device_type AS device_type
FROM click GROUP BY click.device_type) AS alias_3, (SELECT alias_3.device_type AS device_type
FROM (SELECT click.device_type AS device_type
FROM click GROUP BY click.device_type) AS alias_3) AS alias_4 GROUP BY alias_3.device_type ORDER BY count DESC

whoops my result was something else than what i expected


form-control-label class was removed from bootstrap

https://getbootstrap.com/docs/4.1/migration/#breaking says:

Removed the unused .form-control-label class. If you did make use of this class, it was duplicate of the .col-form-label class that vertically centered a  with it’s associated input in horizontal form layouts.

So we should remove or replace .form-control-label from simple_form_bootstrap.rb where needed

Sure, if it wasn't used we can just remove it. Do you mind to open a pull request for that?
Thanks for the issue ❤

#1617

closes https://github.com/plataformatec/simple_form/pull/1617


Is there a non-terrible way to access the HitCounter?

I have some code that's annoying to exercise all the multi-threaded code paths in reliably, and the desire to run my test suite for as short a time as possible (that is, running a stress test for a minute will exercise these paths but it might also exercise them if you only run them for a few seconds).
I think accessing the HitCounter would be really ideal here, as that would allow my code to check the value() and then short-circuit to the test's end when it's been hit the appropriate number of times.
Is that a thing you can see this library supporting?

It feels like this could be a separate crate perhaps?

Extracting the hit counter to a separate crate would work for me, too! Any way to make it accessible really (:

The impl of cov-mark is tiny, feel free to create cov-mark-with-hit-counts fork!


[WIP] Reorg

See the example: https://osc.github.io/ood-documentation-test/reorg/
Enhancements include:

Lighter navbar text is easier to read
New sections Install, Configure, Customize, Extend

TODO:

Add in documentation for missing sections
Add missing sections (quotas and other features)

Actually, I think we should flatten the left customization menu:

Announcements
MOTD
Branding
Default SSH Host
Files Shortcuts
Custom Job Composer Templates

Move to the Configure section

Whitelist directories

And maybe the difference between Configure and Customize (that is in my head) is not pronounced enough to make sense to other users.
My intention was Configure would deal with cluster config, nginx_stage.yml, ood-portal.yml, and things like handling first user login flow when home directory doesn't exist with custom error pages. The custom error pages seems more like a "customization" though.
Maybe a list of 20 items is not overwhelming...

Well the hierarchy does help if you want to know "I want to customize Active Jobs. What are all my options?"


Implement functionality to specify interfaces to be connected in the workflow in the parameter file

The idea would be to be able the specify which interface is enabled in the parameter file such that the workflow is build accordingly. This will allow to be more flexible and skip processing steps.

What do you mean by "specify which interface is enabled in the parameter file"?
Isn't the workflow built accordingly, for instance when we chose to use manual/automatic masks?

Also which parameter file exactly?

@pdedumast @hamzake The current workflow consists of number of interfaces that one might want to skip especially in the preprocessing phase.
For instance Hélène might want to skip histogram normalization for quantitative imaging (not sure thought which steps were exactly skipped by her, please @HeleneMIAL could you clarify this point?) or in the case of simulated data free of motion to skip the slice-to-volume registration step in mialsrtkImageReconstruction.
Or any other interface that you might think it's good to have such a control.
The parameter file in question is the following:
https://github.com/Medical-Image-Analysis-Laboratory/mialsuperresolutiontoolkit/blob/88f726d64c71aaedc17da401af3e97de7f51ac6f/data/code/participants_params.json#L1-L9
We could think of being able to process such a file:
{
   "01": [
     { "sr-id":1,
       "stacksOrder": [1, 3, 5],
       "paramTV": {
         "lambdaTV": 0.75,
         "deltatTV": 0.01 } ,
       "steps": {
         "HistogramNormalization": False,
         "S2VRegistration": False}
     }]
 }

Any step that would not be listed in the parameter file are considered to be True and performed in the workflow.

This would be great indeed!!
As @sebastientourbier suggested:

For in vitro acquisitions without motion, I usually skip the slice-to-volume registration.
As the histogram normalization step is based on the stacks used for SR reconstruction, it is tough to quantitatively compare SR volumes reconstructed from different LR stacks. I am still not sure how to handle this problem to enable accurate quantitative evaluation, but having a more modular implementation of the pipeline would indeed help in such specific cases.

This would be great indeed!!
As @sebastientourbierhttps://github.com/sebastientourbier suggested:

For in vitro acquisitions without motion, I usually skip the slice-to-volume registration.
As the histogram normalization step is based on the stacks used for SR reconstruction, it is tough to quantitatively compare SR volumes reconstructed from different LR stacks. I am still not sure how to handle this problem to enable accurate quantitative evaluation, but having a more modular implementation of the pipeline would indeed help in such specific cases.

De : Priscille de Dumast<EMAIL_ADDRESS>Envoyé : mardi 10 novembre 2020 15:46:29
À : Medical-Image-Analysis-Laboratory/mialsuperresolutiontoolkit
Cc : Hélène Lajous; Mention
Objet : Re: [Medical-Image-Analysis-Laboratory/mialsuperresolutiontoolkit] Implement functionality to specify interfaces to be connected in the workflow in the parameter file (#27)
@sebastientourbierhttps://github.com/sebastientourbier Ok, I understand the point now, thanks for clarifying!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Medical-Image-Analysis-Laboratory/mialsuperresolutiontoolkit/issues/27#issuecomment-724747892, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AOB62IF2M336LP2FXVZAB6LSPFG4LANCNFSM4TLMMYWA.

@hamzake This would requires to:

To extract the steps from the json file as we do for paramTV and pass them to the main() function in the docker/bidsapp/run.py:
https://github.com/Medical-Image-Analysis-Laboratory/mialsuperresolutiontoolkit/blob/fc711a3c53f4163dcbb176dd250bfd0b63d451c4/docker/bidsapp/run.py#L112-L133
with the addition of a new if 'steps' in sr_params.keys(): condition and addition of a new parameter steps for the main() function (to stay consistent with the example json).
Modify accordingly the main() function and add a new parameter steps= for the constructor of AnatomicalPipeline as we do with paramTV
Modify the constructor in pymialsrtk/pipelines/anatomical/srr.py accordingly. I would suggest to add new attributes such as skip_histogram_normalization and then in the constructor you can add:

if 'HistogramNormalization' in steps.keys():
    self.skip_histogram_normalization = not steps['HistogramNormalization']

and so on for each similar step/attribute.

Finally, add in the create_workflow() function conditions with these new attributes to connect a node or not as we do with the use_manual_masks. If a node is not connected, make sure that the inputs of the following interface are well connected to the one before the one skipped. You might need to redefine connections. For the registration option, it would be add an option that run miasrtkImageReconstruction with the flag '--no-reg' (or similar).


feat: add authMethod to kratix chart

This PR will add the possibility to define the authMethod for the Kratix helm chart. If no value is defined, the value is automatically set to basicAuth. With my change, this will also be explicit visible.

Sry, it seems to be a bit messy on my site. Please see https://github.com/syntasso/helm-charts/pull/8


No new releases since 2017

Hi,
I was wondering what's the status on new releases? Is 1.0.0 still to be considered the only "stable" release? No bug fixes?
The commits since the last release show multiple memory leaks fixed and ideally those would require a bugfix release without waiting a year or more.
Thanks

Hi @mtorromeo, we would love to work on a new release but we are tied up to do another one right now. Ideally I would like to have a release before end of year. Fingers crossed :)

That's good to hear, thanks! Any thoughts on releasing small bugfix releases more frequently in the future?

Hi @mtorromeo,
We are cooking a release and we are expecting to have it out of the door till the end of the year.


fix/ssl_error

Hello, very good job, we wanted to propose you a patch for your python library
There was an exception which is raised, on each API call:
Exception: https://api.insee.fr/token
We print it the exception details, and it shows :
<urlopen error [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:852)>
Maybe the insee API need to have a specific ssl certificate, so we pass it at line 39
Hope it can help ;)

Hello !
Thanks you a much for your contribution :)
Cordialement,


Add Benchmark library to performance tests

Can we integrate the Benchmark library (.ipsa) for our current performance tests?
Look at #906 results to guide changes
Consider moving the logic behind creating our performance tests into a separate gem

This will be evaluated as part of #2157


vue-slider the value of this slider can not be less than the minimum value

I try to execute this code by  changing dynamically the max and min of the second slide but I have the error that happens all the name

Do you mean this error occurs when the value is not less than the minimum value?
If not, it is a normal prompt because the value cannot be less than the minimum.

is there anyway to try catch this error ? any solution would be great. because sometime user can change value manually on other input and vue-slider follow other input's value that it was right way in their project.

@theara72 Can use the error callback event @error.
Can also refer to this example:
https://nightcatsama.github.io/vue-slider-component/#/advanced/input

thank that nice.


Cookbook upload missing subdirectories in libraries

If I try to upload a cookbook with subdirectories in libraries (such as https://supermarket.chef.io/cookbooks/f5-bigip, https://github.com/target/f5-bigip-cookbook/tree/master/libraries/load_balancer) it does not seem to include all the subdirectory files on the Chef Server.
Our Chef server is 2.1.2 I think (based on it's /changelog)
So if I knife cookbook upload f5-bigip it does create the cookbook in my org on the Chef Server but does not contain /libraries/load_balancer/**/*.
I would expect it to contain the subdirectories contained in libraries.
I'm assuming this is supposed to work based on https://github.com/chef/chef/pull/2129 and the fact it uploads everything fine to supermarket and that the cookbook runs fine as well in a chef zero environment.
Thanks!

How is this this cookbook being used internal to Target? Is it running in Chef Zero?


MA-325 make downloaded video file not showing up in downloadUI

@BenjiLee would you please review it
cc  @aleffert
for MA-325,  downloaded video should not shown in downloadUI.   Tested
for renaming of filename,  it is in the code already.

:+1:

:+1:

@hanningni Any reason why this isn't being merged?


Installation problem: cannot import name from 'paperqa'

Sorry for the (hopefully) simple question - I'm trying to install paper-qa using mamba and it's giving me an error of not being able to import names. Trying to run the example:
from paperqa import Settings, ask

answer = ask(
    "What manufacturing challenges are unique to bispecific antibodies?",
    settings=Settings(temperature=0.5, paper_directory="my_papers"),
)

Returns:
ImportError: cannot import name 'Settings' from 'paperqa' (C:\Users\b\mambaforge\envs\py311\Lib\site-packages\paperqa\__init__.py)
What could cause this error? Thank you for any help in advance.

Hi @bsvisser , can you please give us the output of pip list? I am thinking you don't have paper-qa version 5, likely you have v4. The Settings object is new in that version.
So you'll want to do something like pip install paper-qa>=5. Please let us know how this turns out


[CT-378] Create tests for vars in packages.yml and selectors.yml

We fixed a bug in $4878 with the use of vars in package.yml and selectors.yml. For ease of backporting and because we're in the middle of a test overhaul, new tests were not added in that ticket/pull request. This ticket is to add additional tests.

Also convert the tests.


test: set and values cannot be used simultaneously

I have a reproducer here:
helm-unittest-reproducer-global-double-setting.zip
It boils down to having a required value global.app that's not set by default:
values.yaml
global:
  app:

template:
metadata:
  name: {{ .Values.global.app | required "global.app is mandatory" }}

Hence to help out in the writing of tests a requiredValues.yaml is created on the side to set it:
tests/values/requiredValues.yaml
global:
  app: my-name

But then in a test we write the following:
templates:
  - templates/deployment.yaml
tests:
  - it: reproducer test
    values:
      - values/requiredValues.yaml
    set:
      global:
        containerName: blibli
    asserts:
      - equal:
          path: spec.template.spec.containers[0].name
          value: blibli

This test fails with:
 FAIL   helm-unittest-reproducer-global-double-setting/tests/deployment_test.yaml
        - reproducer test
                Error: execution error at (helm-unittest-reproducer-global-double-setting/templates/deployment.yaml:4:32): global.app is mandatory

It looks like the set global.containerName has overridden entirely the requiredValues.yaml setting.
Important: happens since helm unittest 0.3.0

Hello @mederel,
Thanks for the feedback.
Due to the upgrade of yaml.v3 the merging of the values was failing when the same yaml object is used.
Have added your testcase as an regression-test and fixed the bug.
Will create a new version in a few days (still want to add some other improvements)
Greetings,
@quintush

A-MA-ZING, thanks for the quick feedback. Greetings back, @mederel

Thank you for fixing this issue! Looking forward for the the new version release.
Bumped into this migrating from another fork https://github.com/vbehar/helm3-unittest back to this repo.
Also, thank you @quintush for mantaining this as a whole.

Very same scenario here.
Migrating from helm3-unittest I also faced this issue. Thank you for reviving this repo!

can you please make a new release? I'd like to be able to use this ability.

The update is released in 0.3.1.


Safe Mode and Configurable Cache Size

Change
Added two parameters:

SAFE_MODE to enable safe mode
MAX_CACHE_SIZE to specify max cached blocks

Will print warning when safe mode is on, or cache size too big.
fix #273 #274
Details
Since all unfinalized info is managed by subscribing to latest heads, we have an 3-line elegant solution for safe mode, which makes it only subscribe to finalized head when safe mode is on. As a result:

cache behaves like subql (only cache finalized blocks), so only finalized tx can be found.
subscription callbacks are only triggered by new finalized heads, so only finalized logs will be notified.

Test
It will take ~50 hours to test it with code, so I just manually tested:

cache only cached MAX_CACHE_SIZE number of blocks
when safe mode enabled, eth_subscribe only returns logs from finalized blocks, and a transaction hash can only be found after it's block is finalized.
when safe mode disabled, eth_subscribe returns logs from latest blocks, and a transaction hash can be found as soon as it's in latest block.

also I am not sure if subscribe to finalized head is enough.
e.g if people query a unfinalized block directly, it should be treated as block not found, instead of return the unfinalized block.
basically we want a helper to check if a tx hash / block hash is finalized, and every time we want to return a block / tx, it needs to ensure the data is finalized or treated as non existent.

closes this for #280


Enable TLS peer verification and add CA root certificates to Docker images

Enables TLS peer verification for connection to upstream IDP services, to ensure the connections are secure. Also adds standard ca_certificates package to the Docker images which is required for verification of the certificate chains.

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

Needs approval from an approver in each of these files:

OWNERS

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

/lgtm

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

Needs approval from an approver in each of these files:

OWNERS

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

@cfryanr do you want to wait for #54 before merging this?

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

Needs approval from an approver in each of these files:

OWNERS

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

@cfryanr do you want to wait for #54 before merging this?

We’ve been testing against an IDP which has TLS certs signed by a trusted CA, so I don’t think it will effect us to merge this before #54 is implemented.


Addition in Issue template

Description:
I noticed that the repository does not have an appropriate issue template.
Issue Template: I want to add  some more points to your issue template like styling changes etc
Please assign this issue to me under Gssoc'23 @apoorvaron

@Anshika-Jain02 Assigned!

This issue has been inactive for a long time
I would like to work on this issue
Please assign

@sagnik-p i am currently working on this and just going to create the PR. You can look out for other issues.


Demonstration for using pre-computed LD matrix

Dear developer,
Congratulation on the SuSiEx method, it certainly fills the great needs of the fine-mapping community. I was trying to use SuSiEx based on precomputed LD matrix corresponding to the summary statistics. However, I received the error message such as,
"* Please provide precomputed LD and frequency files or a reference panel for each GWAS using --ref-file"
The command line of SuSiEx is the following:
SuSiEx/bin/SuSiEx
--sst_file=data/Susiex/AMR_sumstats.txt,data/Susiex/EAS_sumstats.txt
--n_gwas=3000,3000
--ld_file=data/Susiex/AMR_LD.txt,data/Susiex/EAS_LD.txt
--out_dir=Output/Susiex
--out_name=SuSiEx_AMR_EAS
--level=0.95
--pval_thresh=1
--chr=20
--bp=$top,$bottom
--snp_col=2,2
--chr_col=1,1
--bp_col=3,3
--a1_col=4,4
--a2_col=5,5
--eff_col=6,6
--se_col=7,7
--pval_col=9,9
--mult-step=True
--keep-ambig=True
--threads=16
The original SuSiE simply requires the Z-scores and the LD matrix for the computation. I wonder if SuSiEx can do the same. I also noticed that there is no demonstration for using precomputed LD matrix with SuSiEx. It would be great if there is a clear example for using the precomputed LD matrix for SuSiEx.
Thank you!

Hi ZikunY,
Thank you for choosing our method.
We sincerely apologize for the inconvenience caused by this update during your analysis.
In this C++ version, we have implemented the use of a binary format LD file instead of a readable text file. The speed of software reading the binary format file is much faster than reading a readable text file.
Could you use the Python script,
SuSiEx/utilities/SuSiEx_LD.py
to re-calculate your LD again? This code will generate a binary LD file, a ref file, and an allele frequency file. SuSiEx requires these three files as input.
To use this Python script, please find a bash script example here,
SuSiEx/examples/cal_ld.sh
In our example, you can use the following commands to get the fine-mapping results.
#Calulate LD for EUR
python2 ../utilities/SuSiEx_LD.py
--ref_file=EUR
--ld_file=EUR
--chr=1
--bp=7314654,8314677
--plink=../utilities/plink
--maf=0.005
#Calulate LD for AFR
python2 ../utilities/SuSiEx_LD.py
--ref_file=AFR
--ld_file=AFR
--chr=1
--bp=7314654,8314677
--plink=../utilities/plink
--maf=0.005
#run SuSiEx
../bin/SuSiEx
--sst_file=EUR.sumstats.txt,AFR.sumstats.txt
--n_gwas=50000,50000
--ref_file=EUR,AFR
--ld_file=EUR,AFR
--out_dir=./
--out_name=SuSiEx.EUR.AFR.output.cs95
--level=0.95
--pval_thresh=1
--chr=1
--bp=7314654,8314677
--snp_col=2,2
--chr_col=1,1
--bp_col=3,3
--a1_col=4,4
--a2_col=5,5
--eff_col=6,6
--se_col=7,7
--pval_col=9,9
--plink=../utilities/plink
--mult-step=True
--keep-ambig=True
--threads=16
I hope this example has helped you solve your problem. If it has not resolved the issue or if you have any additional questions, please don't hesitate to reach out.
My best
Kai Yuan

Hi Kai,
Thank you so much for this detailed answer, appreciate it!
Yes, I actually used SuSiEx_LD.py to compute the LD based on the ref files and run SuSiEx after. Thanks for the demonstration here.
I totally understand the need for improving the loading speed of LD matrix and it could be quite burdensome if the region was really large or there were many regions. However, it would be flexible for users if there was an alternative option that the users can directly provide the pre-computed LD matrix for SuSiEx along with the Z-scores. It would be easier for users to use and improve the popularity of the method.
Thanks again for the answer!
Best,
Zikun

Hi Zikun,
Thank you for your suggestion. We will make an effort to incorporate the LD + z-scores format into future updates of SuSiEx.
My best
Kai Yuan

Hi Kai,
I have a similar question regarding to the precomputed LD and frequency.
I used the datasets in your examples directory and supplied ld files to --ld_file parameter via --ld_file=tests/EUR.ld,tests/AFR.ld.
Unfortunately, the SuSiEx program errored out with "* Please provide precomputed LD and frequency files or a reference panel for each GWAS using --ref-file".
However, I cannot find the arguments that accept the precomputed frequency file. I wonder if there is a more flexible way to specify the precomputed LD and frequency files? Besides, is the .bin suffix of parameters passed to --ld_file argument required (--ld_file=tests/EUR.ld,tests/AFR.ld or --ld_file=tests/EUR.ld.bin,tests/AFR.ld.bin)?
Best,
Jie Wang

Hi Kai,
I checked the source code of SuSiEx and found that SuSiEx will automatically add ".ld.bin", "_frq.frq" and "_ref.bim" to ld_file parameters. It might be helpful to include this additional step in the README so that other users can know how to prepare the precomputed LD and frequency files without needing to check the source code. It might be clearer to have three separate arguments that accept the absolute paths for each of the three different file types, though I understand this could be a bit cumbersome.
Besides, I wonder why the "_ref.bim" file are required, because sometimes the bim file are not easy to accuire.
Finally, after I specified --ld_file=tests/EUR,tests/AFR arguments and prepared those files, SuSiEx still errored out with "* Please provide the directory and filename of PLINK using --plink". It seems that plink are still required even after I already provided the precomputed LD and frequency files. I'm a bit unclear about the specific role of plink in this case.
Best,
Jie Wang

Hi @Jesson-mark ,
I'm sorry for the confusion.
SuSiEx requires pre-calculated binary format LD information as input (*ld.bin). We strongly recommend using the script located in the "utilities" folder, utilities/SuSiEx_LD.py, to prepare your LD input data.
The reason we ask for the "_ref.bim" file is that it helps us identify which allele corresponds to 0 and which corresponds to 1 in the LD matrix, ensuring the LD data aligns with the summary statistics files.
Please note that the current version of SuSiEx does not support plain text format LD matrices (i.e., ".ld" files). We require the binary version, which is optimized for faster data loading and can be easily generated using the utilities/SuSiEx_LD.py script.
If you already have the LD matrix in plain text format, you can convert it to binary format using the script utilities/convert2bin.cpp.

Compile the script
g++ utilities/convert2bin.cpp -o utilities/convert2bin
Convert your LD matrix
cat $path_to_ld_file | ./utilities/convert2bin $path_to_binary_ld_file

If you would like to use your pre-calculated LD matrix, please make sure the "_ref.bim" file consists of the LD matrix.
My best
Kai Yuan

Hi Kai,
thank you so much for your detailed explanation! Now I understood the importance of "_ref.bim" file.
As to my project, I already calculated covariates-adjusted LD matrix and saved into a ".bin" file with single-precision. I'm curious about the preparation of "_ref.bim" and "_frq.frq" files associated with LD matrix.
I processed my dataset using plink2 that will preserve reference and alternative alleles, which is different from plink1.9 that will use minor allele as A1 thus losing reference and alternative alleles.
So what I need to do is to ensure that alleles (A1 and A2) in "_ref.bim" and "_frq.frq" files are consistent with those used in my LD calculation, is that correct?
Best,
Jie Wang

Hi Jie,
I apologize for the delayed response. Yes, the "_ref.bim" and "_frq.frq" files should align with your LD calculations.
My best
Kai Yuan

Hi Kai,
thanks for your explanation! I'll have a try!
Best,
Jie Wang


Microsoft Team needs to Clean Up The VS-Code Extensions Store Please...!!

When either on the website or within VS-Code looking for extensions, a person finds themselves immersed within
thousands of deprecated rotting extensions that are either outdated, no longer maintained or completely deprecated by the author.
Or if an author stopped development/maintenance because the extension conflicts with VS-Code itself or other extensions etc etc.
The store tends to build up a massive amounts of junk that hinders users from getting actual extensions that would benefit them.
In particularly with new users along with everyone else in the ecosystem.
Why do these old outdated, deprecated and useless extensions still exist in the extensions store ?
Its time for the entire Microsoft Team to go through all the extensions and start cleaning out the closet.
And also setting new rules for the extensions store, forcing authors of extensions to completely remove their extension,
if its been posted that their extension is either no longer maintained, deprecated, outdated or the feature has been implemented within vs-code by Microsoft.
Hundreds and hundreds of extensions should no longer remain in the store as it massively clogs the store with useless outdated extensions
that either are deprecated, maintained, outdated, feature been implemented within vs-code or no longer in use in 2021 going on 2022.
I also find that a lot of extensions features & ideas where already implemented within the official release of VS-Code by Microsoft,
and therefore the extension itself should no longer remain within the store clogging up the store with useless items that is rotting away within the store.
Thanks for reading...   :)

We clean up deprecated / outdated extensions on restart of VS Code. Can you please restart VS Code and check?

Hi sandy,   I am talking about the store itself. Not what's installed.
even though it is nice :) to know that what's installed you clean up...
But I am talking about the store itself.

I am guessing by Store you mean the Marketplace that hosts extensions? If so please file issue against - https://github.com/microsoft/vsmarketplace


"invalid escape sequence" error for regex

Example of Weave parsing error inside function's help string leading to
Error: Weaving failed.
│   error = Base.Meta.ParseError("invalid escape sequence")
└ @ Main none:1
test.jmd:
example of error
```julia
"""
  calculate_snr(regex, label)
regex similar to `(r"SNR_ETD_(?<Ip>\d+)pA_sig_noise_*"` capture probe current Ip in pA
"""
function calculate_snr(data_dir, reg::Regex)
  data_dir
end

Is it a Julia problem or weave parsing problem?

If you paste your code into the Julia REPL you'll get the same error message, so it's not a Weave parsing problem.
As mentioned in the documentation of Documenter.jl, some characters such as $ and \ are treated differently in doc strings. Instead of your example you should write
"""
    calculate_snr(regex, label)

regex similar to `(r"SNR_ETD_(?<Ip>\\d+)pA_sig_noise_*"` capture probe current Ip in pA
"""
function calculate_snr(data_dir, reg::Regex)
    data_dir
end

or
@doc raw"""
    calculate_snr(regex, label)

regex similar to `(r"SNR_ETD_(?<Ip>\d+)pA_sig_noise_*"` capture probe current Ip in pA
"""
function calculate_snr(data_dir, reg::Regex)
    data_dir
end

Note that I used a four-space indent in the line with the function signature at the top of the documentation to ensure that it is printed as Julia code. You can find more conventions in the documentation.

Thanks @devmotion for your answer, please open a new bug if you run into any other issues

Sorry for such long delay in my response. Thank you for your quick and precise answer.


use new crypto and add devDependencies in package.json

@runfan  I've tested the pull request and it works fine with the supported versions. Thanks for the efforts!


Setup Kanban

There are two people in this Project

@AnsarIbrahim
@YoutLone

This is the Project link:
https://github.com/YoutLone/Capstone-Js/projects/1

STATUS: APPROVED ✔ ✔
Hi @AnsarIbrahim and @YoutLone,
✔ Tasks are complete and well assigned
Your project is complete!  There is nothing else to say other than... it's time to go ahead with the project :shipit:
Congratulations! 🎉
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the Issue thread if something is not 100% clear. And please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


replace "interface{}" with "any"

This PR replaces interface{} with any by running the command:
gofmt -w -r 'interface{} -> any' .

any appears in Go 1.18.

@alexandear I do appreciate what you're doing here, but this change makes it non-backwards compatible in general without really gaining anything other than a syntax change. Myself, I'm not a fan of backwards compatibility, but this seems like a superficial change that create hassles for users who may be stuck using a older version of Go. I know there are a number of different orgs, who for a number of different reasons, are stuck on Go 1.16 (or on even older versions of Go). This would force them to upgrade their entire Go toolchain to at least Go 1.18, which suffice to say, may not be possible due to compatibility with other packages/libs, or for any number of other reasons (political or otherwise).
That said, I'd be happy to consider integrating this kind of a change later this year after 1.21 or 1.22 next year, after Go 1.18+ has been out for a bit longer. It's worth noting that Go 1.17 received an update as recently as a few months ago, and is still being supported by the Go team.

@kenshaw thank you for the detailed explanation. chromedp already can't be compiled with go < 1.18. This explicitly states in the release notes.
Also, I tried to compile chromedp with go 1.17:
go1.17 build
# github.com/chromedp/chromedp
./eval.go:81:19: undefined: reflect.Pointer
./eval.go:84:9: undefined: reflect.Pointer

So, that's why I made the PR.

@alexandear I must have missed when this change happened. I've pushed a commit, and tagged it, reenabling support for Go 1.16.

My bad. I used reflect.Pointer without realizing that it is only available since go 1.18.

@ZekeLu not your fault. They changed the name from reflect.Ptr to reflect.Pointer in go1.18. I have no idea why. To be honest, looking at that code, I'm pretty sure I can rewrite it without relying on a specific reference to reflect.Ptr, but I don't have the time at the moment to bother. The fix I pushed earlier today is more than adequate for now.

reflect.Ptr still exists in Go 1.20. It's defined as const Ptr = Pointer


use warnings module as opposed to logging warnings.

We could use the warnings module to raise either Warning or RuntimeWarning as opposed to the custom warnings in:
https://github.com/BienfaitKI/polus-dev/blob/2c010f74548d0d2621289e7c71779fab8d014ff3/src/polus/utils/logging.py#L8-L28

Sounds interesting, @SauravMaheshkar .  I've not used that module before.  Maybe you could give a quick summary of what the benefits of using it, instead of logging, would be?


Allow HTTP address in API server

For now, any HTTP address will be rewrite into HTTPS, which makes even the local API server (using the copilot to gpt4 service) I deployed in docker needs a certificate. I think local API server doesn't need HTTPS support.

Typo


[BUG] trackException() doesn't send custom properties

Description/Screenshot
When using exception tracking with custom properties, we can't see that trackException is collecting them and sending with along with the exception's baseData to Application Insights.
Looking at https://github.com/microsoft/ApplicationInsights-JS/blob/a7f41a2f76a5143643b9195975eb84b50740aafb/channels/applicationinsights-channel-js/src/EnvelopeCreator.ts#L289-L303 I think that this is why it isn't working. We're missing a line similar to EnvelopeCreator.extractPropsAndMeasurements(telemetryItem.data, customProperties, customMeasurements); in the function above.
Steps to Reproduce

OS/Browser: Any
SDK Version [e.g. 22]: 2.2.4
How you initialized the SDK:

appInsights = new ApplicationInsights({
    config: {
      instrumentationKey: instrumentationKey,
      enableCorsCorrelation: false,
      disableAjaxTracking: false,
      disableCorrelationHeaders: false,
      enableDebug: true
    }

Expected behavior
When calling:
trackException(new Error('myerror'), { custom1: '1', custom2: '2'})

I expect to a properties array on baseData when the SDK calls POST https://dc.services.visualstudio.com/v2/track with the exception, where the array contains the two objects sent to the trackException-function.

Related issue for the same problem but for Trace and Metrics: https://github.com/microsoft/ApplicationInsights-JS/pull/1065

Hi @HenrikWM , please use the new API:
appInsights.trackException({
    exception: new Error('some error'),
    properties: {'custom': '1'}
});

Related issue: https://github.com/microsoft/ApplicationInsights-JS/issues/1079


keeps crashing, not allowing me to delete anything

Description
so my file's app keeps crashing and when i try to delete something in file explorer by keeping active in the file.
Steps To Reproduce

go on files
press any buttton
it will crash

Requirements

eg. Fix crash that occurs when...
eg. Correct action happens when clicking...
eg the x button

Files Version
<IP_ADDRESS>
Windows Version
51621011
Log file
debug.log

sorry the windows version is actually:Windows Feature Experience Pack 1000.22639.1000.0 22621.1413 Windows 11 Home:2H22

Can you clarify what you are doing to cause the crash. You said it crashed when you deleted something the File Explorer, but also when pressing any button (What button?) in Files.

@sawm90 I haven't been able to reproduce this issue locally but there are some improvements in v2.4.64 that might have helped. Can you give it another try and see what happens?

@sawm90 this appears to be resolved but please let us know if it's still an issue after updating to v2.4.66.


Add Linxura smart controller button quirk

Proposed change

This PR adds a new quirk for the Linxura smart controller's Zigbee mode, enabling Linxura smart controller proper communication and control within the ZHA framework. Linxura smart controller's Zigbee mode support four Zigbee buttons. Each button support click, double click, long press actions.
Additional information

Test file is in tests/test_linxura.py
Checklist

[x] The changes are tested and work correctly
[x] pre-commit checks pass / the code has been formatted using Black
[x] Tests have been added to verify that the new code works

Thanks for the PR. Please remove the commented out code.
Thank you for reviewing the code, the comment code has been removed!

@TheJulianJES Can you please merge this pull request? Thanks!

Thanks for the PR. Please remove the commented out code.

Hi Julian, I've uploaded the code that removed commented part, is there anything I need to do before merge?

Have a bit of patience please. The PR needs to be reviewed properly. That'll be done when one of us has some free time.

Have a bit of patience please. The PR needs to be reviewed properly. That'll be done when one of us has some free time.

Sure！Thank you for your quick feedback.

Linxura smart controller zigbee button, in order to meet persist feature, modify 4 endpoints to 1 endpoint. Still 4 buttons are shown on Home Assistant with press, double press, long press actions.
Please review, Thank you!

All suggestions modifications has been made. Please review if you spare time. Many thanks!

Thanks for making the changes!
I've made some improvements to the quirk + tests and made sure the code doesn't break when values 0, 6, 12, or 18 are passed in. Those values shouldn't generate events, right?
The COMMAND_ID is also removed from the event and the device triggers. I don't think we actually need it here.
Please have a look at the new code, let me know what you think, and try to see if everything works still.


Changed the input data format for consistency.

After starting to document the input data format, I realised that it was very confusing and inconsistent. This PR makes datasets and referencesets symmetrical and treated in the same way.
The input data directory must now be in the following format:
dataDir/
    datasets/
        dataset1
            /variants
            /reads
        dataset2
            #etc
    referenceSets
        referenceSet1/
        referenceSet2/

Datasets and ReferenceSets are now treated symmetrically (other than their annoying spelling difference...).
This is a breaking change, and will affect all users. It is better to get this change out of the way before we start the release process though.

+1

(after the tests are fixed, of course!)

Thanks @dcolligan, fixed the issue.

Hi @jeromekelleher - don't forget to @-mention us (@macieksmuga and @hjellinek) on changes that can affect the compliance data, so we can prepare.
This one caught us with our trousers descended. :-)  Thanks.

Oops, sorry @hjellinek, my bad.

@jeromekelleher I updated #656 to comply with this new format.


#175494073 Integrate the requests table

What does this PR do?

this PR help users to get the trip request she/he have requested.

Description of Task to be completed?

user or a requester will be able to get the whole information she have commanded with the status of his/her trip. by using the route of http://localhost:3000/api/requests in post man with the authorization to get her/his own request

How should this be manually tested?

clone the RP by using clone https://github.com/Soma-Technologies-Inc/visitAfrica-backend.git
make sure you know where you cloned your RP and enter in it and run code .
run npm install  to install some packages you do not have
run npm run startDev
Loging and copy the token of yours to use in the following route
got to post man on the router of http://localhost:3000/api/requests
get your request

Any background context you want to provide?
N/A
Screenshots (if appropriate)

Questions:

@Hyacinthe396  can you change your response message when no request founded.

@Hyacinthe396  can you change your response message when no request founded.

I think you can squash your commits into one commit.

I think you can squash your commits into one commit.


Add Node Discovery

At the moment, Happynodes uses the mainnet.json file from neo-mon project and looks for valid JSON-RPC, P2P or WS connections. However, there are more nodes than the ones in that file.
Since we are running a node through our neo-node sub-project, we should be able to run node discovery and update the list of known addresses and connections.

Protocode:

Start with list of known addresses
Start a P2P node or use JSON-RPC to get IPs of other nodes
Do Reverse lookup to get hostname from IP
If hostname exists add it to address table (basically a ID->hostname mapping)
If not hostname exists, add the IP address to the address table - that's the default hostname essentially

fixed by #64 #63


'cpu-clock' event do not have CPU attribute set

Some configurations result in the CPU attribute not being set on cpu-clock events.  We get an explicit failure on calls to 'perf script' when this happens because we ask for the CPU number.
Consider removing this or looking for a way to optionally get the CPU number.

Example issue: https://github.com/dotnet/coreclr/issues/18452

FYI , apart from kubernetes , same behavior is observed on Ubuntu VM on azure.
when running that command manually  :
 /usr/bin/perf script -i perf.data.merged -f comm,pid,tid,cpu,time,event,ip,sym,dso,trace > perf.data.txt :
output : Samples for 'cpu-clock' event do not have CPU attribute set
OS :
NAME="Ubuntu"
VERSION="18.04.1 LTS (Bionic Beaver)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 18.04.1 LTS"
VERSION_ID="18.04"
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
VERSION_CODENAME=bionic
UBUNTU_CODENAME=bionic
uname -r : 4.18.0-1011-azure

I found this issue on Google when researching a similar error but in a different software suite.. sharing a few notes here in case someone comes back to this ticket:
I believe the reason this happens is that if you are on metal with hardware performance counters, perf uses the 'cycles' event that has the CPU number. If you are on a VM without hardware performance counters (which is most of the time, particularly in public clouds due to security reasons, I think some AWS instance types where you have dedicated hardware access might give you some performance counters even in a VM), then it uses the software 'cpu-clock' event which does not have this field.
I didn't get to figuring out why in some configurations using all-CPU versus per-process mode seems to result in the CPU field appearing or disappearing.
When using "perf script"'s "-F" field specifier you can specify different fields for "software", "hardware" and "trace" events. Unfortunately if you specify a format for hardware events and there are no such in the file then it errors out. So it seems we can't ask "perf script" to handle this for us and likely some kind of software detection is going to be required.
Hope it helps someone...


Turbinia not picking up zone (-z) from turbiniactl

I want to process a disk that is in zone us-central1-a:
$ turbinia/bin/turbiniactl googleclouddisk -p ramses-test2 -z us-central1-a -d test-disk-small
[INFO] Turbinia version: 20190819.9
[INFO] Creating request e15e80c8811c4a8ba35cce542a3956cf with evidence test-disk-small
[INFO] Run command "turbiniactl status -r e15e80c8811c4a8ba35cce542a3956cf" to see the status of this request and associated tasks
[INFO] Published message<PHONE_NUMBER>706214 to topic turbinia-e254c28a83874edb
[INFO] Done.

Turbinia errors out as it tries to look for the disk in the project default zone us-central1-f
$ turbinia/bin/turbiniactl status -r e15e80c8811c4a8ba35cce542a3956cf
[INFO] Turbinia version: 20190819.9

# Turbinia report e15e80c8811c4a8ba35cce542a3956cf
* Processed 12 Tasks for user xyz123
<snip>
# Failed Tasks
* HadoopAnalysisTask: Task Result was auto-closed from task executor on turbinia-worker-e254c28a83874edb-0 likely due to previous failures.  Previous status: [HadoopAnalysisTask Task failed with exception: [<HttpError 404 when requesting https://compute.googleapis.com/compute/v1/projects/ramses-test2/zones/us-central1-f/disks/test-disk-small?alt=json returned "The resource 'projects/ramses-test2/zones/us-central1-f/disks/test-disk-small' was not found">]]

Turbinia will copy the disk from the source zone into the zone/project where Turbinia exists (on the client side before it starts processing), so my guess is that this was a failure in the copying or somewhere else rather than it not picking up the zone.  It uses cloud-forensics-utils to do the copy, and there was recently a new release, so off the top of my head I'm guessing that it is related to that change.   @giovannt0 also made some changes on the Turbinia side, but they were non-trivial to backport to the release branch, so for now I think I'll probably just pin the dependency to the previous release and can push a new Turbinia point release with that change.

Actually, I just noticed that the project is the same for the source disk and where Turbinia is running, and I double checked the logic for copying the disk, and it only checks to see that the project is different and not the zone, so the copy likely never actually happened:
https://github.com/google/turbinia/blob/6733eea42ba3a2442c49aaf933656ace45bd20e1/turbinia/turbiniactl.py#L496
I'll create a quick fix to check if either zone or project changes to do the copy.


Multi-reference editor 'Create new entry and link' dropdown won't work in an app

To reproduce this:

Clone https://github.com/kamsar/ctfl-multi-reference-embed-bug (specific repro code location)
npm run start
In Contentful, add a private app pointed at http://localhost:1338, with a Entry field editor for Entry reference, list
Assign the app's field editor to any Entry reference, list field. Ensure that more than one content type is allowed to be linked.
Edit an entry with the custom editor. It will render correctly and work as a duplicate of the default multiple reference editor until you click Create new entry and link. Then the dropdown for content type selection becomes hidden and can't be clicked.

The same view, but with sdk.window.startAutoResizer() replaced with sdk.window.updateHeight(800) to force a very tall iframe:

There are two issues here (and they're probably closely related):

The extension SDK's autoResizer is not picking up the opening of the dropdown and resizing the iframe
This seems to be because the technique used to open the dropdown is also not updating any of the usual browser height detection properties, such as document.body.scrollHeight or document.documentElement.getBoundingClientRect() all of which remain constant within the iframe whether the dropdown is open or not.

@kamsar, I encountered this same issue. It is because the dropdown containers use fixed positioning. Subsequently the iframe height calculations do not take the dropdown dimensions into account.

It is related to #188 and also impacts the action menu dropdown, which leverages the same container and positioning. The bottom dropdown, (or otherwise long actions menus), will always get cut off by the iframe.

Specifically regarding the Create new entry and link action, I was able to produce a quick and dirty workaround by overriding the styles to force the container to be static positioned:
.DropdownContainer__DropdownContainer___3WlJM[data-test-id="add-entry-menu-container"] {
  position: static !important;
  display: inline-block !important;
  margin-top: 5px !important;
}

This results in:

Unfortunately it isn't an applicable workaround for the actions menu dropdown on the individual entry cards.

Experiencing the same issue.

@kamsar @knupska @matt-koevort We're working on the fix for this issue. We're about to release a new version of field-editors with the latest Forma36 under the hook that should resolve the issue.

This fix makes the dropdown at least usable without a hack but it's not particularly nice of an experience. The popper dropdown is absolutely positioned using transform: translate(0px, iframeHeightpx), which still does not affect the document height calculations.
Before click:

After click - now you get scrollbars, but the menu is not visible without scrolling - to a non-technical user, that's not very obvious especially since the text does not indicate a menu is open via icon change, etc:

After scrolling down within the iframe, you can use the menu:

User-experience wise the hack we were using with the old menu was actually a lot better, as autoresizer worked with it.
For anyone looking for an updated hack that works with the new version, this does the trick when we negate the transform:
.DropdownContainer__DropdownContainer___3WlJM[data-test-id="add-entry-menu-container"] {
  position: static !important;
  display: inline-block !important;
  margin-top: 5px !important;
  transform: none !important;
}

The way I worked around this on one of our apps was as follows. However, it's only useful in certain situations (see caveats):

Create a ref on the absolute-positioned element which overflows the iframe (caveat: this is only possible if the component lets you pass a ref obviously, such as using a Forma 36 <DropdownList>)
Create an event handler for any event which causes the overflowing element to appear (caveat: again, this is only possible if you have control over when the element is displayed - for example, you can't add a callback to get notified when the user opens the dropdown on an AssetCard as far as I know)
In the event handler above, grab the element reference (the ref added in point 1) and call getBoundingClientRect() on it - you now have the dimensions of the absolute-positioned element
From there it's simply a case of turning off the window auto-resizer and calling updateHeight() with the bounding box height, plus any offsets / padding / whatever you need to make it fit
When the element is hidden again, make sure to restart the window auto-resizer

https://user-images.githubusercontent.com/273311/144236000-d9085360-4948-464c-8983-d77de861e22f.mov
The results still aren't perfect - obviously the dropdown is causing the whole iframe to grow, rather than just appearing over the field below it, but there are limits to what HTML can do 😄  This was good enough for our use case - to be able to dynamically adjust the height of the iframe to encompass the (variable-length) dropdown items without clipping/scrolling.


Adding list items to receipes

I am trying to add several words in a list to the ingredients sections of the front matter and i keep getting errors. What format does your design expect for lists!?

Try wrapping the ingredients in double quotes.


Add Mopidy-Mobile webclient

Development of mopidy-http-kuechenradio has ceased but Mopidy-Mobile is here! Update the system to include this great new webclient.

Fixed in https://github.com/woutervanwijk/Pi-MusicBox/commit/3c679bf11d58610dc8f3b1bfd9411c786264ce91


GLTF2 Import Issue - Visual Overlap

Godot version:
Godot 3.0 RC1
OS/device including version:
Ubuntu 17.10 x64, AMD Radeon R9 Fury
Issue description:
I'm trying to import the GLTF2 download of phoenix bird into Godot.
Once the import completes, Godot displays a red sick face as an icon for the scene.gltf file.  Hovering over it reports:
Status: Import of file failed.  Please fix file and reimport manually.
The console shows no messages that would indicate an error occurred.  Indeed, the scene can be opened.  I can no longer recreate this.
That said, the scene has some obvious issues.

Note: the Scene Root has been scaled to 0.05 for the screenshot.
I tested the same model with the another viewer and it works successfully.
Just to confuse things further, if you Reimport the scene, the red sick face goes away and no errors are reported.
Steps to reproduce:
Download GLTF2 version of phoenix bird and import into Godot.
Minimal reproduction project:
PhoenixGLTFImport.zip

Seems to work. I scaled to 0.01 in the import settings. Then, used Depth Draw Mode Opaque Pre-Pass and enable transparency. 5b70ad9d34d2777e3abd0a13118442eb132aff70


Feature Request: Progress Bar for Scanning the Blockchain

It would be great if there was a progress indication when scanning the blockchain.  I'm new to SIA and the new user experience is important for adoption.  I'm not sure how long the scanning will take and then the program on Mac hide and I unlocked my wallet and it started scanning the blockchain again.  I get why it's important to scan, but for a new wallet and new users, progress indication would be a nice feature.

This issue belongs in the Sia-UI repo at https://github.com/NebulousLabs/Sia-UI.


visualize the ColorMatchTabs at the bottom.

I would like to know if you can place the component at the bottom of a viewcontroller, because I think it would be even better (and an important alternative). If yes, how? Or, you must change the component?

Hi, if you don't use circle menu you can just replace some functions in private extension of MenuView class with this:
`    func layoutNavigationBar() {
navigationBar.translatesAutoresizingMaskIntoConstraints = false
navigationBar.leadingAnchor.constraint(equalTo: leadingAnchor).isActive = true
navigationBar.bottomAnchor.constraint(equalTo: bottomAnchor, constant: 6).isActive = true
navigationBar.trailingAnchor.constraint(equalTo: trailingAnchor).isActive = true
navigationBar.heightAnchor.constraint(equalToConstant: 50).isActive = true
}

    func layoutTabs() {
        tabs.translatesAutoresizingMaskIntoConstraints = false
        tabs.leadingAnchor.constraint(equalTo: navigationBar.leadingAnchor).isActive = true
        tabs.topAnchor.constraint(equalTo: navigationBar.topAnchor).isActive = true
        tabs.trailingAnchor.constraint(equalTo: navigationBar.trailingAnchor).isActive = true
        tabs.heightAnchor.constraint(equalToConstant: 44).isActive = true
    }
    func layoutScrollMenu() {
        scrollMenu.translatesAutoresizingMaskIntoConstraints = false
        scrollMenu.leadingAnchor.constraint(equalTo: leadingAnchor).isActive = true
        scrollMenu.topAnchor.constraint(equalTo: topAnchor).isActive = true
        scrollMenu.trailingAnchor.constraint(equalTo: trailingAnchor).isActive = true
        scrollMenu.bottomAnchor.constraint(equalTo: navigationBar.topAnchor).isActive = true
    }

    func layoutShadowView() {
        shadowView.translatesAutoresizingMaskIntoConstraints = false
        shadowView.leadingAnchor.constraint(equalTo: leadingAnchor).isActive = true
        shadowView.trailingAnchor.constraint(equalTo: trailingAnchor).isActive = true
        shadowView.bottomAnchor.constraint(equalTo: bottomAnchor).isActive = true
        shadowView.heightAnchor.constraint(equalToConstant: 80).isActive = true
    }

    func layoutCircleMenu() {
        circleMenuButton.translatesAutoresizingMaskIntoConstraints = false
        circleMenuButton.bottomAnchor.constraint(equalTo: bottomAnchor, constant: PlusButtonButtonOffset).isActive = true
        circleMenuButton.centerXAnchor.constraint(equalTo: centerXAnchor).isActive = true
    }`

and if you want bottom line just decrease the value of navigationBar.bottomAnchor.constraint(equalTo: bottomAnchor, constant: 6).isActive = true
I hope it will help)

Hi @j7ios. You can compound views as you want and create your own component based on this

@bars395 I have replaced code in demo app and its not working.

@sidlowanshi This code is 3 years old ;), I think guys already remade this library differently

@sidlowanshi This code is 3 years old ;), I think guys already remade this library differently

Yeah but still not found any solutions.


how to express GET alias_xxx/_alias?filter_path=index_name.aliases.alias_xxx

Please use the following questions as a guideline to help me answer
your issue/question without further inquiry. Thank you.
Which version of Elastic are you using?
[x ] elastic.v7 (for Elasticsearch 7.x)
[ ] elastic.v6 (for Elasticsearch 6.x)
[ ] elastic.v5 (for Elasticsearch 5.x)
[ ] elastic.v3 (for Elasticsearch 2.x)
[ ] elastic.v2 (for Elasticsearch 1.x)
Please describe the expected behavior
GET alias_xxx/_alias?filter_path=index_name.aliases.alias_xxx
but may cant use olivere implement
Please describe the actual behavior
i want  result this filter
{
  "indedx_name" : {
    "aliases" : {
      "alias_name" : {
        "filter" : {
          "bool" : {
            "filter" : [
              {

              }
            ]
          }
        }
      }
    }
  }
}

Any steps to reproduce the behavior?

The API you need is IndicesGetAliases. Here's a lot of code on how to work with that API. The FilterPath method is on the service here.
HTH.

but i think IndicesGetAliases only return
indexMap := make(map[string]struct {
		Aliases map[string]struct {
			IsWriteIndex bool `json:"is_write_index"`
		} `json:"aliases"`
	})

dont have my want filter body

That's what IndicesGetAliases returns IIRC. What API is that GET alias_xxx/_alias endpoint? Can you refer me to the documentation page on elastic.co?

https://www.elastic.co/guide/en/elasticsearch/reference/7.17/indices-get-alias.html

Maybe you confused 'indices_get_aliases' and 'indices_get_alias'

So the API is correct (the indices_get_aliases.go implements the endpoint you want). But you're missing part of the response?

yes

Unfortunately, as the response is dynamic, all you can do is to use e.g. the PerformRequest method on the Client then and parse the body on your side.

ok,thx


Revert "Ensure Exploration Center is enabled by default"

Finally, we have fixed the way the extension is installed and
managed via Group Policies and this hack can be now exterminated.
This reverts commit 31d89e7bcf9bb283c4d08b46abdb2c47331996ee.
https://phabricator.endlessm.com/T16331

Yay!


signTransaction Doesn't Always Handle Smart Contract Errors Correctly

Describe the bug
When a smart contract throws an error, the handling of the error appears to make false assumptions about the structure of the error object. Notice, specifically the json property, which is not read in the current code base. This results in an

@kylanhurt I'm wondering if this would not be a duplicate of #130 , but describe the root cause of the problem.

@karynemayer Yes you could argue that this could possibly be considered a duplicate. I will take my comments over to that issue.


feat(removeSelected): Implement removeSelected property for multiple selects

Expose and implement remove-selected="false" for multiple selects. This will disable a choice
in the dropdown of a multiple-select element instead of removing it.
Sample behavior:

In the existing version, id would not have appeared in the dropdown because it is currently selected. This pull request adds the option removeSelected so that, if set to false, selected items still appear in the dropdown but are disabled (like in the screenshot).

@dondi did you want to update the wiki with some docs? (add a note saying available v0.17.2+)
Thanks again :+1:

Great, thanks! I've added a section on this to the wiki. I made it an example section instead of adding it to the attribute table because it really only applies for multiple. (I noticed that multiple itself is not in that attribute table at the beginning, so I took my cue from there.) Let me know if the writeup needs any changes. Thank you again!

@user378230 Is there any indication on when this feature will be released?

Is it possible that this pr introduced a new bug in the normal dropdown? Wrong options get removed in my dropdown. After a reload they appear again. Also in older versions you didn't remove the option on select.

Yes, a bug was indeed caught and the fix has since been merged, presumably to emerge in the next release. Please see #1672 and #1681 to see if these behaviors match what you are seeing. You can try a build from master to see if the latest code addresses what you’re seeing.


Hide undeployed servers in the servers page in day2 production (SCRD-6682)

…6682)

@jeremy-moffitt you should not see any changes in dev, hide is only for production.  Just need to leave some convenient for dev testing.

If force production in dev, when just drag a server in compute without run deploy...that server will NOT show up in server summary page day2.  however in day0, it should show up.
export function isProduction() {
if (PRODUCTION) { // eslint-disable-line no-undef
return true;
}
return true;
}
If NOT force production, then it will show up in the server summary in day2, no restriction there.

@guglo sorry for the delay, trying it out now in production mode

I see the filter applying in production mode (the server does appear, but is quickly filtered out), thanks!


Use "black" profile when running isort (commit hooks)

There are some incompatibilities between the standard configurations for isort and black (ref.
To avoid issues of black and isort continuously reformatting the same file, add the --profile black option to isort

@leothomas I think we got this covered in https://github.com/developmentseed/eoAPI/blob/master/setup.cfg#L3