VAF adjustment for duplications

I had a question on VAF adjustment for duplications. I think it makes sense for heterozygous variants where the real VAF is supposed to be 0.5. Let's say ploidy is 2, and tumor purity is 1.
In a real 0.5 VAF case, let's say the real support and normal counts are 10 and 20 respectively. The normal counts would be multiplied by 0.5 (which is 1 - (1/2), based on the eqn given in the paper) and the adjusted support and normal would become 10 and 10 respectively. This would give a VAF of 0.5.
But if it deviates from 0.5, say a subclonal variant of 0.2. The support and normal counted could be 4 and 20 respectively. The normal counts are multiplied again by 0.5, to give 4 and 10. This gives a VAF of 0.4.
Same with a homozygous duplication. The support and normal counted could be 20 and 20 respectively. The normal counts are multiplied again by 0.5, to give 20 and 10. This gives a VAF of 0.66, instead of 1.0.
This is what I observe with the data I provide too, the variants seem to center around 0.5. Am I understanding this wrong, and is there a way I can recapitulate expected VAFs?

The normal counts refers to reads unaffected by the variant only, but the VAF is calculated over the total reads (support + normal). There's some more detail about this in section 2.3.2 of my thesis.

I just read through the section you pointed out as well as a previous section but I am still a bit confused. Given the examples in my first post (I edited the subclonal final VAF, as as it was calculated wrong), the final VAF is different from the expected VAF for all the cases, save for the case where the true VAF is 0.5.
For example, in the homozygous example, the final VAF is 20 support / (20 support + 10 normal) = 0.66, not 1

You are correct in that the VAF adjustment calculation assumes that the variant is heterozygous, this is an over-simplification, but it is by design. This is because at this point of the pipeline, we do not know the multiplicity (i.e, we don't know if the variant is homozygous or heterozygous). The multiplicity model in ccube then adjusts for this possibility. The VAF adjustment is just a heuristic to bring amplification normal read counts in line with other variants.


Regenerate doesn't work on new page load

If you load new session with previous chat regenerate button does not work

fixed in a different issue


fix: made json_like_output inline to avoid collisions

I found that json_like_output function collides with itself when trying to include all.hpp into multiple source files.
So, I made it inline.

Sorry, If I made PR into wrong brunch, but I did not found any contribution guide...


Updated DOI in intro, readme to match Zenodo upload.

Coverage remained the same at 65.545% when pulling d077518b18cfe54f3f9217ce66dab432179ae7f5 on update-doi-1.0 into b6299dab59605ced739e10f39bf5b382c4295444 on master.


broken sign in

See here: https://learn.microsoft.com/en-us/answers/questions/1659638/weve-detected-a-microsoft-intune-powershell-script

Download latest 2.98 version of the script which now uses Microsoft Graph module for Graph API requests.
Install new module with command:
Install-Module -Name Microsoft.Graph.Authentication -Scope CurrentUser

thats what i did, but it didn´t work.
i fixed it like in the link i posted.
keep in mind, only if someone asks later


Fixed to use apierror.APIError instead of spanner.Error

Dear Developers.
I noticed that Yo is using DEPRECATED spanner.Error.
So I have modified it to use APIError.APIError, as exemplified in the spanner package.
ref. https://pkg.go.dev/cloud.google.com/go/spanner#Error
I also re-created various templates with the modified contents in the Docker image environment specified in .circleci/config.yml.
This is my first contribution to Yo. So I apologize if I skipped any necessary steps.
(I have already completed the signing of the Contributor License Agreement.)
Thank you in advance for your review.

Since you have included unnecessary files, I will revise them again and create a PR for you.
My apologies.


ci: add release and changelog generation

Pull Request Test Coverage Report for Build 459314668

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

Totals

Change from base Build 459313478:
0.0%

Covered Lines:
73

Relevant Lines:
77

💛  - Coveralls

Pull Request Test Coverage Report for Build 459314668

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

Totals

Change from base Build 459313478:
0.0%

Covered Lines:
73

Relevant Lines:
77

💛  - Coveralls

Pull Request Test Coverage Report for Build 459314668

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

Totals

Change from base Build 459313478:
0.0%

Covered Lines:
73

Relevant Lines:
77

💛  - Coveralls

Pull Request Test Coverage Report for Build 459314668

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

Totals

Change from base Build 459313478:
0.0%

Covered Lines:
73

Relevant Lines:
77

💛  - Coveralls


Add Contacts and Calendar Syncs

That’s our families main selfhosted issue
For example through Nextcloud + Owncloud
There are other solutions too: https://radicale.org
IOS: natively supported (Card/CalDAV)
Android: DAVx5 (formerly Davdroid)
Playstore: (paid) https://play.google.com/store/apps/details?id=at.bitfire.davdroid&hl=de
FDroid: https://f-droid.org/en/packages/at.bitfire.davdroid/

I can confirm that DAVx5 (It's open source and you can either pay for it in the Google Play Store - 3,99€ - or build it using Android Studio) is the best way to sync calendars and emails on Android. --> Source: https://gitlab.com/bitfireAT/davx5-ose
I'm using Synology Calendar and CardDav. Works like a charm ;)


Dreamcast translation to Russian

Hello! I finished translation of dreamcast article started by https://crowdin.com/profile/Spreit.
I'm not sure how I should put multiple authors to authors log and if it's fine to keep nicknames here instead of real names

Thanks! I'll give it a quick review tomorrow, it shouldn't take too long.
Ah, in that case just worry about your signature for now. If the other person wants to sign it, he/she can do it too. But it shouldn't deter you from authoring your work.

The translation is ready, only thing left is to sign it. Yes, you can use a nickname if you want.

#90

Translation published! https://www.copetti.org/ru/writings/consoles/dreamcast/
If you want, I can make a small announcement on Twitter and link your profile?

Sure, it would be nice

What's your Twitter?

Ah, sorry, I thought you mean github profile. I don't have one, github is probably only public profile I have.

Ok!


Fixed typo & improved the readability of the README file.

Notes

There is no .env.example but rather env.example.
Its a bit hard to follow the README.

Thanks @faizalzakaria for this improvement. I will merge this in after I fixed the travis failure. Seems like there is an update with travis latest image causing the build to fail.

@faizalzakaria ,
I've changed a bit the readme based on the below changes:
63a8d40bc320f74ada72c2495d4311917c8f4928
I'm still interested with the improvement you are doing here. Could you do it again based on the latest changes? I'm so sorry for the trouble.

That should be good now @taufek .
I still believe the doc could be a little bit better still. :) . Anyway, that is enough for me to bring this up.
This works well with simple project, need a better version for a more complex system/config.

Yes. If you use docker-sync, it requires the folders that it syncs to
exists. So if you use my sync.sh script to start it, it will create the
folder the folders for you.
If you don’t use docker-sync, docker-compose will create the folders
for you.
On Sun, 24 Dec 2017 at 6:22 AM, Faizal Zakaria<EMAIL_ADDRESS>wrote:

@faizalzakaria commented on this pull request.
In README.md
https://github.com/taufek/dockery/pull/1#discussion_r158590212:

@@ -57,6 +58,17 @@ because for Mac, it requires performance boost with Docker Sync. Refer to
docker-compose.linux.yml or docker-compose.mac.yml to docker-compose.override.yml
in order to get your ruby workspace volumes mapped to your host machine.

+
Actually the main problem was the .env file, the reason i created this.
But looks like you already fixed it.
So I guess we don't have to create the folder manually ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/taufek/dockery/pull/1#discussion_r158590212, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AKTYmkcwwSNGQPulkNErOqYHfDFPBaHEks5tDX0fgaJpZM4RI_qJ
.

I confirmed, I dont need that folder to be created manually.


DBATools - is it possible to export Summary Log File after using Update-DbaInstance

Good day,
After I run a patch using -Update-DbaInstance, is it possible with DBATools to export the Summary Log File to another location.
Please view the picture with the highlighted file:

Thanks and Regards

something among the lines of
$server = Connect-DbaInstance "localhost"
$commonLogDir = Get-ChildItem -Path "C:\Program Files\Microsoft SQL Server\$($server.Major)0\Setup Bootstrap\Log" -Directory

$lastSetupFolder = $commonLogDir | Sort-Object Name | Select-Object -Last 1

Get-ChildItem $lastSetupFolder.FullName | Where-Object Name -Like "Summary_*"

should work

Hi @niphlod , that worked perfectly, thank you!


Deprecated links for Microsoft.Azure.CognitiveServices.Language.TextAnalytics

And documentation

As part of this process, the package description needs to be updated to point to the new Azure.AI.TextAnalytics and its documentation

Also, updating docs.msft.com to primarily show docs for the new library, changing the moniker on reference docs on docs.msft.com for the old library as "legacy"


Added phantomjs to the dev deps

Not entirely sure how I feel about this, usually I just expect phantom to be installed. It's not the greatest but it means there isn't a huge download just to get started developing deku.
Thoughts?

This closes #138

I'm cool with this. I think the extra install time is worth it so we can prevent people getting discouraged.
We could solve it by first trying to use a global installation and then falling back to installing Deku's local phantom dep. Probably not worth the effort to get that working though.

+1


Move an application with attachments to talent pool does not work.

Trying to move an application with attachments to the talent pool results in exceptions stating that \Core\Entity\Collection does not have a ::getMapping() method or - even weirder - Doctrine expects a PersistentCollection but recievies an ArrayCollection.

I don't know if the following help you to solve it somehow, I found out commenting out this line prevent the exception to be thrown.
I am still searching for the root cause.


docs: update connector readme

Just an update with the readme of the connectors

bors r+

The connector docs should be updated in their hub packages, because we have automation to generate the changes to the website. I see that the Kafka docs are different than what is in this PR, and that's going to make it difficult to know where to look for the most up to date references


Client comms rework

Add dynamic redirect from passive server to client
Add diagnostic port to get basic state of a server not yet connected to the cluster
supporting interface changes and relaying

I went through all those changes, and basically you've "just" modified some of the plumbing to be able to send a redirect to a client.
I didn't see anything obviously wrong, but the comms layer's FSM is so complicated that nothing is obvious. These changes do make me worry despite the fact that I can't see anything wrong.

Ludovic.  Thanks for going through the PR.  Yes.  This change has me very concerned if for no other reason, the shear size of the diff.  I've tried to push the comms stack into a more manageable shape.  Time will tell if I succeeded.


Question: How to match a keyword (stored in a variable) in a specific column and color a range?

I would like to match the keyword "Baseballs" (stored in a variable) in column A and color columns A-C of the row...but I don't get it working. Here are my attempts:
$Data = ConvertFrom-Csv @'
Item,Quantity,Price,Total Cost
Footballs,9,21.95,197.55
Cones,36,7.99,287.64
Shin Guards,14,10.95,153.3
Turf Shoes,22,79.95,1758.9
Baseballs,68,7.99,543.32
Baseball Gloves,31,65.00,2015.00
Baseball Bats,38,159.00,6042.00
'@

# Attempt 01 - Matches the keyword in column A but doesn't color the range A-C

$Keyword = "Baseballs"

$Data | Export-Excel -Path "$ENV:USERPROFILE\Desktop\Data1.xlsx" -FreezeTopRow -BoldTopRow -AutoSize -AutoFilter -WorkSheetname "Attempt1" -CellStyleSB {
param($WorkSheet)
# BackgroundColor and FontColor for specific cells of TopRow
$BackgroundColor = [System.Drawing.Color]::FromArgb(255,220,0)
Set-Format -Address $WorkSheet.Cells["A1:D1"] -BackgroundColor $BackgroundColor -FontColor Black
# HorizontalAlignment "Center" of columns B-D
$WorkSheet.Cells["B:D"].Style.HorizontalAlignment="Center"
Add-ConditionalFormatting -Address $WorkSheet.Cells["A:C"] -WorkSheet $WorkSheet -RuleType ContainsText -ConditionValue "$Keyword" -BackgroundColor Red
}

# Attempt 2 - Matches the keyword in column A and colors the range A-C, but it doesn't support the keyword as variable

$Data | Export-Excel -Path "$ENV:USERPROFILE\Desktop\Data2.xlsx" -FreezeTopRow -BoldTopRow -AutoSize -AutoFilter -WorkSheetname "Attempt2" -CellStyleSB {
param($WorkSheet)
# BackgroundColor and FontColor for specific cells of TopRow
$BackgroundColor = [System.Drawing.Color]::FromArgb(255,220,0)
Set-Format -Address $WorkSheet.Cells["A1:D1"] -BackgroundColor $BackgroundColor -FontColor Black
# HorizontalAlignment "Center" of columns B-D
$WorkSheet.Cells["B:D"].Style.HorizontalAlignment="Center"
Add-ConditionalFormatting -Address $WorkSheet.Cells["A:C"] -WorkSheet $WorkSheet -RuleType 'Expression' 'NOT(ISERROR(FIND("Baseballs",$A1)))' -BackgroundColor Red
}

Single quotes with a variable in it do not get interpolated.
'$keyword' - no
"$keyword" - yes
I used -f to do replacement using the single quotes.
$Data = ConvertFrom-Csv @'
Item,Quantity,Price,Total Cost
Footballs,9,21.95,197.55
Cones,36,7.99,287.64
Shin Guards,14,10.95,153.3
Turf Shoes,22,79.95,1758.9
Baseballs,68,7.99,543.32
Baseball Gloves,31,65.00,2015.00
Baseball Bats,38,159.00,6042.00
'@

$xlfile = "$PSScriptRoot\Data.xlsx"
Remove-Item $xlfile -ErrorAction SilentlyContinue

$Keyword = "Baseballs"
# $Keyword = "Cones"

$Data | Export-Excel -Path $xlfile -FreezeTopRow -BoldTopRow -AutoSize -AutoFilter -WorkSheetname "Attempt2" -Show -CellStyleSB {
    param($WorkSheet)

    # BackgroundColor and FontColor for specific cells of TopRow
    $BackgroundColor = [System.Drawing.Color]::FromArgb(255, 220, 0)
    Set-Format -Address $WorkSheet.Cells["A1:D1"] -BackgroundColor $BackgroundColor -FontColor Black

    # HorizontalAlignment "Center" of columns B-D
    $WorkSheet.Cells["B:D"].Style.HorizontalAlignment = "Center"

    $cv = 'NOT(ISERROR(FIND("{0}",$A1)))' -f $Keyword

    Add-ConditionalFormatting -Address $WorkSheet.Cells["A:C"] -WorkSheet $WorkSheet -RuleType 'Expression' -ConditionValue $cv -BackgroundColor Red
}

Awesome! This will help me a lot!


Error: stdout maxBuffer exceeded.

This error message appears when running any navy command when a lot of services are running.

Closed by beb0340


Audio not working on Windows Phone 8.1 when changing position

Hello, I have faced a bug, when trying to play video file (avi, video codec: xvid, audio codec: ac3) on Windows Phone 8.1.
If I change position in MediaElement, audio is not working.
Code example (C#):

I get MediaStreamSouce:
IRandomAccessStream readStream = await file.OpenAsync(FileAccessMode.Read);
             // Instantiate FFmpeg object and pass the stream from opened file
             FFmpegMSS = FFmpegInteropMSS.CreateFFmpegInteropMSSFromStream(readStream, true, true);
             MediaStreamSource mss = FFmpegMSS.GetMediaStreamSource();

myMediaElement.SetMediaStreamSource(mss);

myMediaElement is MediaElement. It has MediaOpened event.
And if I want to play my file not from the start, I can set myMediaElement.Position in MediaOpened event.
So do I:
private void myMediaElement_MediaOpened(object sender, RoutedEventArgs e)
{
Media.Position = TimeSpan.FromMinutes(5);
}

The result: video starts playing from the time I wanted (in this example - 5 minutes from the start), FPS is OK, but there is no sound.
If I set myMediaElement.Pause and then myMediaElement.Play again several times, sound would appear. But new bug would appear too - FPS is dropping, so now I see not video, but a slideshow.

If I use Play and Pause again several times more, it would become OK (sound - ON, FPS - normal).
PS. If I start playing video from the very beggining (not setting myMediaElement.Positon), everything would be OK - video would play smooth and audio would work.

Hi,
I encountered seek issues too, using only the application seek bar, regardless of the platform.
Compressed or uncompressed samples appear to make a difference.
I experimented various tweaks but still encounter an issue where Media Foundation stops requesting samples after being able to seek to and display a single (software decoded) video frame.

Thanks for the information. There is a known issue with seeking uncompressed video that we are currently investigating in the Media Foundation side. As you saw, the sample request stops after the seek. Temporary workaround is to set the MediaStreamSource BufferTime to 0. I will keep you updated.

Thank you for remembering me to set the buffer time to 0, I used to set it in the past for other purpose but could not relate it to what I was experiencing regarding seek operations using the latest code.


ENH backport gcc and glib migrations to 3.6

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

These are needed to get a version of all of the r-base pinnings that works with the R package rebuilds.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin rerender

@conda-forge/r-base This PR is need to unblock the gcc930 (and possibly glib) migrations. It is ready for review and merge!


feat(explore): ColumnSelectControl with drag-and-drop

SUMMARY
POC of Explore drag and drop query panel. scope - Groupby control only.
Todo:

[x] styling, visual indicator for draggable columns/metrics and droppable field
[ ] glitch free when d&d quickly
[x]  add feature flag
[ ] metrics refractor + new feature(in-place saving) + d&d
[ ] ad hoc filter refractor + new feature(filter array value) + d&d
[ ] sort-by + d&d

This feature will be behind a FF until all drag & drop is implemented in all control fields.
added a new feature flage
The new FF name is ENABLE_EXPLORE_DRAG_AND_DROP,  and the default value is False.
DEFAULT_FEATURE_FLAGS = {
...
...
    "ENABLE_EXPLORE_DRAG_AND_DROP": False,
}

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
multi columns groupby
https://user-images.githubusercontent.com/2016594/108483127-508dc180-72d5-11eb-9530-f0c715b95e45.mp4
single column groupby
https://user-images.githubusercontent.com/2016594/108483173-5d121a00-72d5-11eb-9651-5666c2fcb2da.mp4
TEST PLAN

ADDITIONAL INFORMATION

[ ] Has associated issue:
[x] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[x] Introduces new feature or API
[ ] Removes existing feature or API

Codecov Report

Merging #13210 (58beef1) into master (9b5e66b) will decrease coverage by 13.72%.
The diff coverage is 16.19%.

@@             Coverage Diff             @@
##           master   #13210       +/-   ##
===========================================
- Coverage   72.29%   58.57%   -13.73%
===========================================
  Files         864      442      -422
  Lines       44883    13394    -31489
  Branches     5403     3548     -1855
===========================================
- Hits        32450     7845    -24605
+ Misses      12224     5549     -6675
+ Partials      209        0      -209

Flag
Coverage Δ

cypress
58.57% <16.19%> (?)

javascript
?

python
?

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

Impacted Files
Coverage Δ

...rontend/src/explore/components/DatasourcePanel.tsx
68.29% <ø> (-10.88%)
:arrow_down:

...ls/DndColumnSelectControl/DndColumnSelectLabel.tsx
0.00% <0.00%> (ø)

...trols/DndColumnSelectControl/components/Option.tsx
0.00% <0.00%> (ø)

...ndColumnSelectControl/components/OptionWrapper.tsx
0.00% <0.00%> (ø)

...ols/DndColumnSelectControl/utils/optionSelector.ts
0.00% <0.00%> (ø)

...-frontend/src/explore/components/controls/index.js
100.00% <ø> (ø)

...nts/DatasourcePanel/DatasourcePanelDragWrapper.tsx
100.00% <100.00%> (ø)

...nd/src/explore/components/DatasourcePanel/types.ts
100.00% <100.00%> (ø)

...frontend/src/explore/components/OptionControls.tsx
70.51% <100.00%> (+0.63%)
:arrow_up:

...omponents/controls/DndColumnSelectControl/types.ts
100.00% <100.00%> (ø)

... and 812 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 9b5e66b...58beef1. Read the comment docs.

@zhaoyongjie Yongjie, thanks for the PR! super excited about it. can you add manual text plan and instruction of turning on the feature flag in description? thanks!

@zhaoyongjie Yongjie, thanks for the POC! super excited about it. can you add manual text plan and instruction of turning on the feature flag in description? thanks!

I have added in PR description.

Addition thoughts about whether we should keep select or not.

SELECT has limitation in realizing complex hierarchy/grouping interaction. dimension/columns, metrics grouping and hierarchy are very common BI tool, I will send more example in Tableau and PowerBI.

When there are multiple DROP targets, dnd help set some drop restrictions. For example, If target A is dropped, target B will not accept drop values. (There are cases in PivotTable where  groupby and column restrict each other)

performance. do we need to render options in dimensions/columns and metrics twice?

Performance considerations: Now all targets have to repeat columns/metrics, which is completely unnecessary

@junlincc This PR was ready for review for only 2 days. I think we should not rush to merge this into master for something as fundamental as this.
If it's for convenience of reviewing and unblocking future PRs, let's make a feature branch in apache/superset.

🟢

Todo: for group-by that only accepts dimensions , add red indicator for metrics drop. @zhaoyongjie

Todo: for group-by that only accepts dimensions , add red indicator for metrics drop. @zhaoyongjie

copy that, this feature need more polish, thank you.

@yardz
issue found
in master, once you set this feature flag to True, new Echarts Graph is now working.
To reproduce
1.
DEFAULT_FEATURE_FLAGS = {
...
...
    "ENABLE_EXPLORE_DRAG_AND_DROP": True,
}

Create a Graph by

clicking the top right + button to create new chart
Select Graph(in the last row) from the gallery
Select any dataset
create, see error.
Expected behavior

https://user-images.githubusercontent.com/67837651/109529299-73a13980-7a6a-11eb-8d57-1b02713d3d82.mov
thanks!


SOLR-16204: Update lucene dependency version to 9.1.0

https://issues.apache.org/jira/browse/SOLR-16204
Description
Update the Lucene dependency version to 9.1.0
Solution
There are a lot of files changed due to the following changes in Lucene 9.1.0:
LUCENE-10301
This package splitting in Lucene caused the majority of the changes for this PR.
LUCENE-10377
Has been replaced the sortPos parameter with enableSkipping. enableSkipping should be set to true for the first sort field, and false for the others.
Tests
No tests were added as it is a dependency update
Checklist
Please review the following and check all that apply:

[x] I have reviewed the guidelines for How to Contribute and my code conforms to the standards described there to the best of my ability.
[x] I have created a Jira issue and added the issue ID to my pull request title.
[ ] I have given Solr maintainers access to contribute to my PR branch. (optional but recommended)
[x] I have developed this patch against the main branch.
[x] I have run ./gradlew check.
[x] I have added tests for my changes.
[x] I have added documentation for the Reference Guide

The tests look consistently green. Should we proceed with merging?

9.2.0 came out the day before. Maybe update to that already?

9.2.0 came out the day before. Maybe update to that already?

We discussed this today with @eliaporciani, our opinion is to:

merge this, monitor Jenkins and make sure everything is all right
open immediately another PR with the migration to Lucene 9.2

In this way, we can pinpoint issues separately and the transition may be easier.
Does it make sense?
We have the bandwidth to merge this one and open a new one in the week starting on the 6 of June.


Make the Prepublish Script Cross Platform Compatible and Stop Ignoring /lib.

Making the prepublish script cross platform compatible.
Reference:
https://docs.npmjs.com/misc/scripts#path
https://github.com/npm/npm/issues/4040#issuecomment-27159695
Removing the package.json consumable "main": "./lib" from gitignore.

We should ignore the generated files in git. Other it will be very difficult for me to review PR's

Fixed the cross compatibility of prepublish scripts through other PR.


Fixed #13883 -- Added optgroups for SelectBox in Admin

Modified SelectBox.js so that optgroups are honored. When an item is
moved from unselected to selected box, order is maintained by group
and then item name. Also added a test.
Thanks ᴙɘɘᴙgYmɘᴙɘj for the nice repro.

@mmclar Thanks for the patch :+1:
I found that this doesn't play nice with :heavy_plus_sign: , see
https://user-images.githubusercontent.com/2865885/165691060-84313333-39c3-452e-a546-4ba7cb786f54.mp4
I also cannot save chosen values anymore:
https://user-images.githubusercontent.com/2865885/165692282-9c56aa48-dddc-4a1c-bdd6-a06c344c3d2d.mp4

Thanks for the feedback!
For saving values: this is just an error with the demo app. In ProfileForm, the pairs added to the choices lists should be (pk, name), not (name, name) as seen here: https://github.com/reergymerej/ticket_13883/blob/master/demo/admin.py on lines 14 and 16.
For the + button: this is quite a tricky bit. Since the logic for partitioning the choices into optgroups lives in the admin object, I don't see a way to discern a new choice's optgroup without initializing ProfileForm again. To do that, the popup window needs to know the source form, so that needs to be passed to the add form. I've written out those changes and they are here: https://github.com/django/django/pull/15568/commits/e60d5f371e4985bae1658146cbf4d4965035d600

Are there any news on this?

Hello @silentbugs @mmclar
I see that this PR needs a new round of reviews, in order to progress with that, would you please rebase onto main, fix conflicts, and re-ping for a re-review?
Thank you!

@nessita I've resolved the conflicts and updated some new tests -- how do I get a re-review?

For saving values: this is just an error with the demo app. In ProfileForm, the pairs added to the choices lists should be (pk, name), not (name, name) as seen here: https://github.com/reergymerej/ticket_13883/blob/master/demo/admin.py on lines 14 and 16.

Yes, thanks it works now.

For the + button: this is quite a tricky bit. Since the logic for partitioning the choices into optgroups lives in the admin object, I don't see a way to discern a new choice's optgroup without initializing ProfileForm again. To do that, the popup window needs to know the source form, so that needs to be passed to the add form. I've written out those changes and they are here: e60d5f3

Unfortunately, I can still reproduce this issue.

Hm. I am not able to reproduce the issue using the same steps. I am not sure what the delta between my test system and yours is.
Screencast from 06-16-2023 06:57:37 PM.webm

Hm. I am not able to reproduce the issue using the same steps. I am not sure what the delta between my test system and yours is.

Probably because I'm checking on the "Add" view, not the "Change" view:
https://github.com/django/django/assets/2865885/b47b1b9a-cc8f-4ac7-bf03-980acc484a2a

I think I'm also using the "Add" view, and the plus icon to add a new sport item. Here's a screen recording with more context:
Screencast from 06-18-2023 03:51:12 PM.webm

I think I'm also using the "Add" view, and the plus icon to add a new sport item. Here's a screen recording with more context: Screencast from 06-18-2023 03:51:12 PM.webm

It still doesn't work for me. Can you share your test project? I think you've changed a structure of choices, from 2-tuples of (sport.pk, sport.name) to (sport.name, sport) :thinking: As far as I'm aware this, may break the way how ChoiceWidget.optgroups() works.

I used optgroup with select2 and chosen, those JS widgets searched also in group name.
I humbly suggest that search may also match otpgroup string.

I don't know much about JS performances, but as this  node_text  chain is also used  for sorting, I ask myself if somebody knowing something about JS perf would rather add an index field to options, or compute it in both cases.


fix: make ErrorKind visible outside the library

While the struct ErrorKind is defined as pub inside the module error, the module itself is private.
In contrast to the struct Error, ErrorKind is not exported by a pub use in lib.rs, effectively making it invisible outside the library and impeding fine-grained error handling.
This change fixes this by exporting ErrorKind alongside Error.

Thanks!


consider removing 'required' from protobuf fields

This might be important for versioning.
"Never ever ever have a required protobuf enum. SIGH Unknown enum values become unset, which violates the required attribute." from https://twitter.com/owen_omalley/status/563501178370027520
The protobuf docs seem to discourage it, and IIRC protobuf3 doesn't implement it.

Confirmed that if you send a message including a new enum value for a required enum field, an old recipient that does not know of this enum value will be unable to parse the message: the field will be missing.

Ok, so if we have an optional enum field, an unknown value will be equal to the first value listed in the enum, yet will serialize to the correct (unknown) value. That's workable.


[bug] Instructor when instrumented tries to instantiate the type that is given as a response_model

Describe the bug
When using instructor, one of the things you pass to the client.chat.completions.create is a Pydantic model type for the kwarg response_model.  When openinference instruments instructor, it takes all kwargs to the client.chat.completions.create and tries to save them as attributes.  Unfortunately, it doesn't expect to get anything that isn't a standard type (or sequence of standard types), and the mask operation for a span when saving attributes returns with:
return value() if callable(value) else value
which (in my case) tries to instantiate a pydantic type that can't be instantiated without arguments.
I would be happy to put together a pull request for this, however I'm not sure where the problem should be fixed.  Should it be fixed in the mask function? or in the instructor _wrapped.py patched_new_func where it creates the dictionary of attributes (by stringifying all inputs), or in _WrappedSpan.set_attributes where it can check if a value isn't a AttributeValue and stringily it or similar before it passes it to _WrappedSpan.set_attribute.
Let me know where you think this should be fixed.

Thanks for the note @spott - we'll take a look. If there's a minimum working example we can take a look at that would be amazing. Let us know if we can help in any way.

Let me try and show what I've found.  I don't think a minimum working example is necessary, though it shouldn't be that hard to make one.
When you patch instructor, the patched async function here is called (there is a similar function for sync functions as well).
here that function calls start_as_current_span.  However, notice that it takes the kwargs of the function, and flattens them and passes it as a dict as attributes for the span.
After a few more layers of indirection, we get to here, the _WrappedSpan class.  This class has two functions, set_attributes which accepts dicts of AttributeValues, and set_attribute which accepts callables that return AttributeValue or an AttributeValue.
AttributeValue looks like this:
AttributeValue = Union[
    str,
    bool,
    int,
    float,
    Sequence[str],
    Sequence[bool],
    Sequence[int],
    Sequence[float],
]

set_attribute calls a mask function that, at the end, checks if the value is callable, and if it is, calls it with no arguments.
And there is where the problem lies.  The function that this whole thing is wrapping (client.chat.completions.create in this case), takes as an argument a response_model... which is a Pydantic model.  This isn't a AttributeValue, and when mask tries to call it with no arguments, it tries to construct the Pydantic object, and if that object has required fields, it throws.
Ultimately the problem is that the types that set_attributes accepts aren't enforced, and the flattening of the kwargs here passes non-AttributeValue values along the chain.
So, I'm happy to submit a pull request... but I'm not sure where the contract for AttributeValue should be enforced.  Should it be enforced at the flattening of the kwargs? or should it be enforced at the set_attributes?  I can see good arguments for both.
I'm leaning towards enforcing it at the flattening of the kwargs.  In this case, I would stringify anything that wasn't an AttributeValue in the kwargs dict before passing it as an attribute.
It might also be worth adding some error catching so that an error in the tracing code doesn't result in the function not being called... but I'm not sure where that would go.


`open': No such file or directory - -w (Errno::ENOENT)

Sorry, another issue. I can connect fine with the settings, but it refuses to do anything related to uploading. I've tried multiple directory settings.
_config.yml:
markdown: discount
highlighter: pigments

ftp_host: 'ftp.128keaton.com'
ftp_dir: '/more/beta/'

ftp_passive: false

# optional
ftp_port: 21                  # default 21
ftp_username: ''     # default read from stdin
ftp_password: '' # default read from stdin
ftp_secure: false            # default false

The home directory of the user I am using has their path set properly.
Full error:
Sending site over FTP (host: ftp.128keaton.com, port: 21, ftps: false)
Connected to server. Sending site
/Library/Ruby/Gems/2.0.0/gems/glynn-1.2.3/lib/glynn/ftp.rb:46:in `open': No such file or directory - -w (Errno::ENOENT)
	from /Library/Ruby/Gems/2.0.0/gems/glynn-1.2.3/lib/glynn/ftp.rb:46:in `foreach'
	from /Library/Ruby/Gems/2.0.0/gems/glynn-1.2.3/lib/glynn/ftp.rb:46:in `send_dir'
	from /Library/Ruby/Gems/2.0.0/gems/glynn-1.2.3/lib/glynn/ftp.rb:17:in `block in sync'
	from /Library/Ruby/Gems/2.0.0/gems/glynn-1.2.3/lib/glynn/ftp.rb:27:in `block in connect'
	from /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/net/ftp.rb:135:in `open'
	from /Library/Ruby/Gems/2.0.0/gems/glynn-1.2.3/lib/glynn/ftp.rb:23:in `connect'
	from /Library/Ruby/Gems/2.0.0/gems/glynn-1.2.3/lib/glynn/ftp.rb:16:in `sync'
	from /Library/Ruby/Gems/2.0.0/gems/glynn-1.2.3/bin/glynn:80:in `<top (required)>'
	from /usr/local/bin/glynn:23:in `load'
	from /usr/local/bin/glynn:23:in `<main>'

-w breaks everything

I see you've closed the issue. Is your problem resolved?

@dmathieu yeah, I was trying to do --watch


New tests to check Codecov.io

@tisnik Your image is available in the registry: docker pull MISSING_REGISTRY_NAME/MISSING_IMAGE_NAME:SNAPSHOT-PR-14


fix to allow using only cognito authorizers

This is a 🐞 bug fix.

[ ] I've added tests (if it's a bug, feature or enhancement)
[ ] I've adjusted the documentation (if it's a feature or enhancement)
[x] The test suite passes (run bundle exec rspec to verify this)

Summary
Fix a bug when only cognito authorizers are being used w/o any lambda authorizers.
Context
https://community.rubyonjets.com/t/deploying-is-failing-with-cognito-authorizer/266
Version Changes
Patch

Released in v2.2.3


[v2] Bump up rk-entry version to v2.0.2

Codecov Report

Merging #60 (f9099c3) into master (5b916bb) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #60   +/-   ##
=======================================
  Coverage   92.92%   92.92%
=======================================
  Files          18       18
  Lines         763      763
=======================================
  Hits          709      709
  Misses         39       39
  Partials       15       15

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 5b916bb...f9099c3. Read the comment docs.


Cranelift: select.i128 is not implemented on x86_64

👋 Hey,
Fuzzgen found this when adding support for this instruction.
.clif Test Case
test interpret
test run
set enable_llvm_abi_extensions=true
target x86_64

function %a(i128, i128, i128) -> i128 {
block0(v0: i128, v1: i128, v2: i128):
    v3 = select.i128 v0, v1, v2
    return v3
}
; run: %a(1, 2, 3) == 2

Steps to Reproduce

clif-util test ./the-above.clif

Expected Results
The above tests to pass.
Actual Results
    Finished dev [unoptimized + debuginfo] target(s) in 0.14s
     Running `/home/afonso/git/wasmtime/target/debug/clif-util test ./lmao.clif`
thread 'worker #0' panicked at 'internal error: entered unreachable code: implemented in ISLE: inst = `v3 = select.i128 v0, v1, v2`, type = `Some(types::I128)`', cranelift/codegen/src/isa/x64/lower.rs:476:13
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
 ERROR cranelift_filetests::concurrent > FAIL: panicked in worker #0: internal error: entered unreachable code: implemented in ISLE: inst = `v3 = select.i128 v0, v1, v2`, type = `Some(types::I128)`
FAIL ./lmao.clif: panicked in worker #0: internal error: entered unreachable code: implemented in ISLE: inst = `v3 = select.i128 v0, v1, v2`, type = `Some(types::I128)`
1 tests
Error: 1 failure

Versions and Environment
Cranelift version or commit: main
Operating system: Linux
Architecture: x86_64

I see we already have select.i128 in some existing clif tests targeting x86_64, is there a reason why this is still erroring?

Right, I think this is incorrectly described, what's failing here is selecting on a i128 value, not using i128 as a control type (that works for i64 and smaller).
This test uses select.i128 with an i8 condition value, but switching that to i128 makes it fail to compile. I've updated the issue to make this a bit clearer.


question about C++ overloaded functions of vwadd/vwsub/vfwadd/vfwsub......

When I am implementing C++ overloaeded functions in GCC. I saw there is inconsistency between vfwadd and vfwmul. For C++ overloaded functions. vfwmul can be resolved into vfwmul.vv or vfwmul.vf instructions according to the arguments. So it make sense
that C++ overloaded functions define the intrinsic like this vfloat32mf2_t vfwmul (vfloat16mf4_t op1, vfloat16mf4_t op2, size_t vl);.
But for vfwadd, the intrinsic are not using this same rule. I saw there are definitions like
vfloat32mf2_t vfwadd_vv (vfloat16mf4_t op1, vfloat16mf4_t op2, size_t vl);
vfloat32mf2_t vfwadd_vf (vfloat16mf4_t op1, float16_t op2, size_t vl);
vfloat32mf2_t vfwadd_wv (vfloat32mf2_t op1, vfloat16mf4_t op2, size_t vl);
vfloat32mf2_t vfwadd_wf (vfloat32mf2_t op1, float16_t op2, size_t vl); . I think the suffix _vv _vf wv wf are redundant. It can be resolved to different instructions according to different argument too. Can anyone tell me the story about why the document define these C++ functions?

vfwmul have no wv or wf operation, so they do not have https://github.com/riscv-non-isa/rvv-intrinsic-doc/blob/master/rvv-intrinsic-rfc.md#widening-vector-scalar-arithmetic-instructions-for-vxwx-and-vfwf-cases
problem which vfwadd/vfadd have.

vfwmul have no wv or wf operation, so they do not have https://github.com/riscv-non-isa/rvv-intrinsic-doc/blob/master/rvv-intrinsic-rfc.md#widening-vector-scalar-arithmetic-instructions-for-vxwx-and-vfwf-cases problem which vfwadd/vfadd have.

OK, Thanks for your reply.
I have a another question about C++ overloaded functions in the new-policy api.
I saw vint8mf8_t vmv_s_ta (int8_t src, size_t vl); and vint8mf4_t vmv_s_ta (int8_t src, size_t vl); these 2 C++ functions.
I think in this case, it can not be overloaded. Am I wrong about this?

Yes, you are right. We are still working on implementation in clang and would fix all bugs in the policy PR later.
The another bug is reduction tam need to remove the merge operand.
int8m1_t vredsum_vs_i8mf8_i8m1_tam (vbool64_t mask, vint8m1_t merge, vint8mf8_t vector, vint8m1_t scalar, size_t vl); // incorrect
int8m1_t vredsum_vs_i8mf8_i8m1_tam (vbool64_t mask, vint8mf8_t vector, vint8m1_t scalar, size_t vl); // correct


Loosing parenthesis inside new

When compiling:
self.audio_context = __new__((window.AudioContext or window.webkitAudioContext)())

following JavaScript code was generated:
self.audio_context = new window.AudioContext || window.webkitAudioContext ();

loosing the paranthesis around those class names, thus leading to incorrect code.

Thanks, will be corrected in upcoming minor version!


Yamaha media player does not work anymore

Home Assistant release (hass --version):
0.38.1
Python release (python3 --version):
3.4.2
Component/platform:
yamaha
Description of problem:
My yamaha rx481 does not get discovery anymore, I see this error in the logs:
17-02-17 15:27:34 ERROR (MainThread) [homeassistant.components.media_player] Error while setting up platform yamaha
Traceback (most recent call last):
  File "/srv/homeassistant/homeassistant_venv/lib/python3.4/site-packages/homeassistant/helpers/entity_component.py", line 151, in _async_setup_platform
    entity_platform.add_entities, discovery_info
  File "/usr/lib/python3.4/asyncio/futures.py", line 388, in __iter__
    yield self  # This tells Task to wait for completion.
  File "/usr/lib/python3.4/asyncio/tasks.py", line 286, in _wakeup
    value = future.result()
  File "/usr/lib/python3.4/asyncio/futures.py", line 277, in result
    raise self._exception
  File "/usr/lib/python3.4/concurrent/futures/thread.py", line 54, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/srv/homeassistant/homeassistant_venv/lib/python3.4/site-packages/homeassistant/components/media_player/yamaha.py", line 89, in setup_platform
    YamahaDevice(name, receiver, source_ignore, source_names)])
  File "/srv/homeassistant/homeassistant_venv/lib/python3.4/site-packages/homeassistant/components/media_player/yamaha.py", line 109, in __init__
    self.update()
  File "/srv/homeassistant/homeassistant_venv/lib/python3.4/site-packages/homeassistant/components/media_player/yamaha.py", line 115, in update
    self._play_status = self._receiver.play_status()
  File "/home/homeassistant/.homeassistant/deps/rxv/rxv.py", line 336, in play_status
    playing = (res.find(".//Playback_Info").text == "Play")
AttributeError: 'NoneType' object has no attribute 'text'

Expected:
Auto discovery of my avr...
Problem-relevant configuration.yaml entries and steps to reproduce:
media_player:
- platform: yamaha

Additional info:
Maybe the problem is that my avr gets switched off during the night? I have seen similar problems in the past which could be solved with a hass restart but this time that does not seem to help.

I am getting similar error.
`17-02-22 11:42:45 ERROR (Thread-3) [rxv] Failed to fetch http://<IP_ADDRESS>:80/YamahaRemoteControl/desc.xml
Traceback (most recent call last):
File "/usr/local/lib/python3.5/site-packages/requests/packages/urllib3/connection.py", line 141, in _new_conn
(self.host, self.port), self.timeout, **extra_kw)
File "/usr/local/lib/python3.5/site-packages/requests/packages/urllib3/util/connection.py", line 83, in create_connection
raise err
File "/usr/local/lib/python3.5/site-packages/requests/packages/urllib3/util/connection.py", line 73, in create_connection
sock.connect(sa)
TimeoutError: [Errno 60] Operation timed out
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/local/lib/python3.5/site-packages/requests/packages/urllib3/connectionpool.py", line 600, in urlopen
chunked=chunked)
File "/usr/local/lib/python3.5/site-packages/requests/packages/urllib3/connectionpool.py", line 356, in _make_request
conn.request(method, url, **httplib_request_kw)
File "/usr/local/lib/python3.5/http/client.py", line 1106, in request
self._send_request(method, url, body, headers)
File "/usr/local/lib/python3.5/http/client.py", line 1151, in _send_request
self.endheaders(body)
File "/usr/local/lib/python3.5/http/client.py", line 1102, in endheaders
self._send_output(message_body)
File "/usr/local/lib/python3.5/http/client.py", line 934, in _send_output
self.send(msg)
File "/usr/local/lib/python3.5/http/client.py", line 877, in send
self.connect()
File "/usr/local/lib/python3.5/site-packages/requests/packages/urllib3/connection.py", line 166, in connect
conn = self._new_conn()
File "/usr/local/lib/python3.5/site-packages/requests/packages/urllib3/connection.py", line 150, in _new_conn
self, "Failed to establish a new connection: %s" % e)
requests.packages.urllib3.exceptions.NewConnectionError: <requests.packages.urllib3.connection.HTTPConnection object at 0x8103a8748>: Failed to establish a new connection: [Errno 60] Operation timed out
`

i get this error too:
17-03-15 21:16:02 ERROR (MainThread) [homeassistant.components.media_player] Error while setting up platform yamaha
Traceback (most recent call last):
File "/usr/local/lib/python3.4/dist-packages/homeassistant/helpers/entity_component.py", line 153, in _async_setup_platform
entity_platform.schedule_add_entities, discovery_info
File "/usr/lib/python3.4/asyncio/futures.py", line 388, in iter
yield self  # This tells Task to wait for completion.
File "/usr/lib/python3.4/asyncio/tasks.py", line 286, in _wakeup
value = future.result()
File "/usr/lib/python3.4/asyncio/futures.py", line 277, in result
raise self._exception
File "/usr/lib/python3.4/concurrent/futures/thread.py", line 54, in run
result = self.fn(*self.args, **self.kwargs)
File "/usr/local/lib/python3.4/dist-packages/homeassistant/components/media_player/yamaha.py", line 89, in setup_platform
YamahaDevice(name, receiver, source_ignore, source_names)])
File "/usr/local/lib/python3.4/dist-packages/homeassistant/components/media_player/yamaha.py", line 109, in init
self.update()
File "/usr/local/lib/python3.4/dist-packages/homeassistant/components/media_player/yamaha.py", line 115, in update
self._play_status = self._receiver.play_status()
File "/var/opt/homeassistant/deps/rxv/rxv.py", line 336, in play_status
playing = (res.find(".//Playback_Info").text == "Play")
AttributeError: 'NoneType' object has no attribute 'text'

same here:
17-03-16 17:39:57 ERROR (MainThread) [homeassistant.components.media_player] Error while setting up platform yamaha
Traceback (most recent call last):
  File "/srv/homeassistant/lib/python3.4/site-packages/homeassistant/helpers/entity_component.py", line 153, in _async_setup_platform
    entity_platform.schedule_add_entities, discovery_info
  File "/usr/lib/python3.4/asyncio/futures.py", line 388, in __iter__
    yield self  # This tells Task to wait for completion.
  File "/usr/lib/python3.4/asyncio/tasks.py", line 286, in _wakeup
    value = future.result()
  File "/usr/lib/python3.4/asyncio/futures.py", line 277, in result
    raise self._exception
  File "/usr/lib/python3.4/concurrent/futures/thread.py", line 54, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/srv/homeassistant/lib/python3.4/site-packages/homeassistant/components/media_player/yamaha.py", line 79, in setup_platform
    for recv in rxv.find():
  File "/home/homeassistant/.homeassistant/deps/rxv/__init__.py", line 26, in find
    for ri in ssdp.discover(timeout=timeout)
  File "/home/homeassistant/.homeassistant/deps/rxv/ssdp.py", line 64, in discover
    res = rxv_details(url)
  File "/home/homeassistant/.homeassistant/deps/rxv/ssdp.py", line 83, in rxv_details
    unit_desc_url_local = xml.find(UNITDESC_URL_QUERY).text
AttributeError: 'NoneType' object has no attribute 'text'

Are you all running the latest version of Homeassistant? I only ask because I am not having this issue anymore.

Yes 40.0. Been having problems for several versions.
On Mar 16, 2017 3:24 PM, "Cameron Llewellyn"<EMAIL_ADDRESS>wrote:

Are you all running the latest version of Homeassistant? I only ask
because I am not having this issue anymore.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/home-assistant/home-assistant/issues/6079#issuecomment-287180120,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ASYqgpfX6RmQzGWedgTvOy12cCFlvFR5ks5rmZptgaJpZM4MEyC_
.

Same error on version 0.44.1 with Yamaha RX-V479

I have a Yamaha RN-301 amp and get the following error.
HASS= 0.47.1
Traceback (most recent call last):   File "/usr/lib/python3.4/asyncio/tasks.py", line 233, in _step     result = coro.throw(exc)   File "/usr/local/lib/python3.4/dist-packages/homeassistant/components/discovery.py", line 118, in scan_devices     results = yield from hass.async_add_job(_discover, netdisco)   File "/usr/lib/python3.4/asyncio/futures.py", line 388, in __iter__     yield self  # This tells Task to wait for completion.   File "/usr/lib/python3.4/asyncio/tasks.py", line 286, in _wakeup     value = future.result()   File "/usr/lib/python3.4/asyncio/futures.py", line 277, in result     raise self._exception   File "/usr/lib/python3.4/concurrent/futures/thread.py", line 54, in run     result = self.fn(*self.args, **self.kwargs)   File "/usr/local/lib/python3.4/dist-packages/homeassistant/components/discovery.py", line 147, in _discover     for service in netdisco.get_info(disc):   File "/home/pi/homeassistant/deps/netdisco/discovery.py", line 97, in get_info     return self.discoverables[dis].get_info()   File "/home/pi/homeassistant/deps/netdisco/discoverables/__init__.py", line 23, in get_info     return [self.info_from_entry(entry) for entry in self.get_entries()]   File "/home/pi/homeassistant/deps/netdisco/discoverables/__init__.py", line 23, in <listcomp>     return [self.info_from_entry(entry) for entry in self.get_entries()]   File "/home/pi/homeassistant/deps/netdisco/discoverables/yamaha.py", line 27, in info_from_entry     service['X_unitDescURL'][1:]) KeyError: 'X_unitDescURL'

This seems to be working for me know.

I think in the future we should switch to yamaha_musiccast anyway


Add new elements to banks and cities

Hi!
I'm using this library, and some banks and cities are distinct from wikipedia, I'm getting the data from a development tools for a banking service.

What is the development tool that has the list of banks and cities?  By any chance does the tool vendor publish the list?

What is the development tool that has the list of banks and cities? By any chance does the tool vendor publish the list?

Is the bank STP. And you need user/pass to access that information, unfortunately

Without access to the data, making updates will be challenging.
To highlight the difficulty, this PR results in 88 banks but the current list has 102 banks.  That means at least 13 banks would be deleted, which could cause a lot of problems.

I cleaned up the banks codes and found the names for the banks with missing names.
The new and updated bank codes:
01f205ef5fad9f739cf552f41d7466e772f240dd

@medisoft I processed the new list of cities and formatted the list to work in the project.  Then I merged the list with the existing cities.  Attached is the combined formatted list of cities.
If the list looks good to you, feel free to replace the existing cities array on line 209 with this new combined array.  If you put it in a PR, I'll quickly merge it in and publish a new release.
   cities: [
      // Sources:
      //    https://en.wikipedia.org/wiki/Template:Mexico_State-Abbreviation_CodesMX
      //    https://es.wikipedia.org/wiki/CLABE#C.C3.B3digo_de_plaza
      [ 10, 'Aguascalientes MX-AGU'],
      [ 11, 'Asientos MX-AGU'],
      [ 12, 'Calvillo MX-AGU'],
      ...
      [960, 'Victor Rosales MX-ZAC'],
      [961, 'Villa Gonzalez Ortega MX-ZAC'],
      [962, 'Villanueva MX-ZAC'],
      ],

Full list of cities:
cities.txt

Without access to the data, making updates will be challenging.
To highlight the difficulty, this PR results in 88 banks but the current list has 102 banks. That means at least 13 banks would be deleted, which could cause a lot of problems.

About this, there are some banks that disappeared or merged, that could be the cause for the disappearing banks. We may keep the old banks in the list for backward compatibility, what do you think?

About this, there are some banks that disappeared or merged, that could be the cause for the disappearing banks. We may keep the old banks in the list for backward compatibility, what do you think?

Yep, and for the bank codes that had different names, I searched to verify why.  Sure enough, there was some kind of merger or takeover.  Last week I updated the banks array to include the new bank codes and updated bank names you provided plus the old banks (even if the old banks are out of business, the backward compatibility makes sense).
Next we should update the cities array.


Increase timeout to 10 seconds for ITHttp

Related: https://github.com/line/armeria/pull/2559
The tests could take longer because of instrumenting test coverage.
So I think it's better to increase it upto 10 seconds.

maybe just for consistency, hit the others?
$ ag 'Timeout.seconds'
instrumentation/dubbo-rpc/src/test/java/brave/dubbo/rpc/ITTracingFilter.java
39:  @Rule public TestRule globalTimeout = new DisableOnDebug(Timeout.seconds(5)); // max per method

instrumentation/http-tests/src/main/java/brave/test/http/ITHttp.java
102:  @Rule public TestRule globalTimeout = new DisableOnDebug(Timeout.seconds(5)); // max per method

instrumentation/dubbo/src/test/java/brave/dubbo/ITTracingFilter.java
40:  @Rule public TestRule globalTimeout = new DisableOnDebug(Timeout.seconds(5)); // max per method

Ah yes. Fixed. Thanks. 😉

thanks for pushing this upstream!

Hope to do it more. 😆


feat: add folding range provider

Closes #56
Adds a folding range provider. For now it only works on do end blocks.
Things to consider:

typespecs and docs and whether to fold them together with their respsective function - makes this PR a bit harder and maybe it's not necessary for starters?
folding on fn -> end blocks - I think this should be added as well in the first version

The folds I added to nvim-treesitter currently consist of

[
  (anonymous_function)
  (stab_clause)
  (arguments)
  (block)
  (do_block)
  (list)
  (map)
  (tuple)
] @fold

So should at least do those.
Eventually it would be nice to fold consecutive alias Foo.Bar, as we have a lot of them in the Next LS code base, would be cool to fold them up.

Thanks for the feedback, I'll go and implement those, will edit the original comment with a todo list for those.


docs(readme): Fix anchor link for example

Thanks @ahmadnassri
To be merged in the next release :)


Is there a way to gulpify running storybook or build-storybook without the npm command?

I love this tool, but for business reasons we need it running via gulp commands and we can't use gulp shell or any of those that allow us to simply run terminal commands. Any recommended way to make this work?

Can you run a webserver like express? If so that's an opening.

another opening would be a child_process in node?

Essentially, we have our own build tools at the company. I'm perfectly fine running whatever storybook has running when we start up storybook in the npm run storybook way. The dilemma is that I'm not allowed to run npm run with our build tools. I noticed that start-storybook runs the dist/server/index.js file for instance. If there is a way in gulp to latch onto that and get things started up... would be fantastic.

Is something blocking your technically from doing that?
Or are you prohibited from configuring something like that?
If you are prohibited from running npm run or gulp shell I'm assuming you're also prohibited from running a custom gulp pipe? or gulpFn?
https://www.npmjs.com/package/gulp-fn
https://stackoverflow.com/questions/27923523/how-can-i-write-a-simple-gulp-pipe-function
If you can run a pipe, you can run a node script.
I'm afraid there isn't much we can do here.. It sounds to me that what you want is perfectly possible to do, but you are just not allowed to do it..


Why Hybrid Connection is related to Biztalk services? can't my web app hosted in VM, which is hosted in Azure can't access onpremise SQL using Hybrid Connection?

Why Hybrid Connection is related to Biztalk services? can't my web app hosted in VM, which is hosted in Azure can't access onpremise SQL using Hybrid Connection?

Why Hybrid Connection is related to Biztalk services? can't my web app hosted in VM, which is hosted in Azure can't access onpremise SQL using Hybrid Connection?

@venkatrv please provide us with the document you are referring to so we can better assist

@venkatrv Just as an FYI, BizTalk Hybrid Connections is retired, and replaced by App Service Hybrid Connections. For more information, including how to manage your existing BizTalk Hybrid Connections, see Azure App Service Hybrid Connections.
I will close this out for now. If you have additional questions just let me know.


[Bug] Snap installation doesn't start

Insomnia Version: 6.2.0
Operating System: Ubuntu 18.04
uname: Linux darwin-xps-15 4.15.0-39-generic #42-Ubuntu SMP Tue Oct 23 15:48:01 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Details

The app doesn't start and just seems to fails.
I initially thought this might be a Wayland issue or maybe something with snap itself but this issue persist even after I switch to Xorg, and other apps I installed with snap starts fine. (Spotify, Postman)
I've also done some file search in this repo and couldn't fine those listed JS files in the logs, so I wasn't quite sure this is the right place to report, but I think you are likely to have better context of what's going on than I do.
Here are the likely related logs from the syslog.

It looks like the JSON file is not part of Insomnia, but of gnome.
You should be able to install older versions from the snap CLI, although I'm not familiar enough with it to know how. I can see all previous versions are still there on snapcraft.io.

where are you seeing the previous versions? :thinking:

I'm seeing them on the developer page. I don't know much about Snaps but I'm not sure why they wouldn't be available.
This is the command I use to push new versions:
snapcraft push insomnia_x.y.z_amd64.snap --release stable

An analogy for describing Snap would be Docker I guess, since Snaps are mostly encapsulated into a sandbox environment and usually don't poke their heads out of there unless required to.
This command will usually be able to list previous versions available but Insomnia doesn't seem to list them. What you see are likely only available to you.
~/workspace
❯ snap list --all | grep insomnia
insomnia              6.2.3                    21    stable    gschier1990   -

Oh well, it's just a nice to have personally in a sense that Insomnia's dependency will be encapsulated into a sandbox and I don't need to install all the GTK libraries.
The debian dpkg package seems to work fine so I'll just use that then.
Thank you for creating this project. Please keep up the good work. :smiley:

Hmm, well if you know a fairly easy way that I could change my publishing workflow to support older versions, please let me know. The small amount of Googling I did didn't seem to mention anything about managing past versions. I assumed that Snapcraft would keep previous versions available for download similar to how Bintray does it for Deb packages.

Looks like snap can make a couple past versions available to the user, but I'm unclear how you would do it as the app author.


Remove the deprecated fetchSignInMethodsForEmail function

The fetchSignInMethodsForEmail function was used to know if the email (user) exists.
Since this function is deprecated, it has been removed.
The error code now allows to do the same job.

It's no longer necessary to disable email address enumeration protection.
Issues related:

https://github.com/GogoVega/node-red-contrib-firebase-realtime-database/issues/64
https://discourse.nodered.org/t/error-unknown-email-when-using-firebase/89857


Global scss not working.

So I have styles.scss that import list of style from my scss folder.
The Issue happens when I try to use variables or mixins  that are defined and Imported inside my styles.scss.
`// GLOBALS
@import 'scss/index';`
I get this error

ERROR in ./src/app/navigation/navigation.component.scss
Module build failed:
@include shadow-1;
^
No mixin named shadow-1

I can solve this only If import the styles.scss directly to the component.scss.
Importing the global scss to all components every time is weird.
also tried to add the same scss to the app[0].styles array
please help.

Have same issue and that's uncomfortable. In Angular 1 I had set up only one scss file compiled which includes all the stuff in right order, but here every component's style compiled separately. That's possibly because of "style scoping".

The global style files are essentially in a different context and only the final output css is available for use by the components.  You'll need to import all of your variables and mixins into each of your components.

@clydin is correct.

The global style files are essentially in a different context and only the final output css is available for use by the components.
Does that mean global scss files will get compiled?
@clydin
I am using a global.scss and it is also not getting compiles. It's in an angular-cli project

I am new to sass, but @clydin wouldn't that produce duplicate css? if I include the same sass in different components, I get duplicate css of that @include right?

In my case (Angular 7):
angular.json
"projects": {
    "MyProject": {
      "sourceRoot": "src"
}}

styles.scss
@import "assets/css/variables.scss";


Fix test build integration test

This string has escape sequences in it.

On a broader note I'm unsure why this is suddenly the case, it seems to have correlated with the release of new base images, but its unclear to me why it would add escape sequences as part of its stderr output.

Some more observations here:

Turning off cog base images does nothing
This console output comes from go running the docker images, and is present in the go console, making me think this may not actually be related to the base images


Unable to specify an array as the value of a body parameter

I am trying to specify an array as a body parameter to a PUT request. The Swagger API seems to allow this, but when trying to make this work using swagger-blocks, Rails throws a 500.
For example, given the following API route:
api do
  key :path, '/contributions/bulk_submit'
  key :description, 'Efficiently submit a bunch of contributions by ID'
  operation do
    key :method, 'PUT'
    key :summary, 'Submits all the specified contributions'
    key :nickname, :Contributions_BulkSubmit
    parameter do
      key :paramType, :body
      key :name, :contribution_ids
      key :description, 'An array of the individual contribution IDs being submitted'
      key :required, true
      key :type, :ArrayContributionSubmitPayload
    end
    response_message do
      key :code, 200
      key :message, 'Success.'
    end
  end
end

I'd like the :ArrayContributionsSubmitPayload to look like this:
swagger_model :ArrayContributionSubmitPayload do
  key :id, :ArrayContributionSubmitPayload
  key :type, :array
  items do
    key :type, :integer
  end
end

or possibly this:
swagger_model :ArrayContributionSubmitPayload do
  key :id, :ArrayContributionSubmitPayload
  schema do
    key :type, :array
    items do
      key :type, :integer
    end
  end
end

However, neither option works. The only way I can specify an array in the body parameter is to further encapsulate it like so:
swagger_model :ArrayContributionSubmitPayload do
  key :id, :ArrayContributionSubmitPayload
  property :contribution_ids do
    key :type, :array
    items do
      key :type, :integer
    end
  end
end

But that results in documentation that leads the end user to believe that you have to name a contribution_ids parameter, and that parameter takes a JSON payload that specifies an object that looks like this:
{contribution_ids: [0, 1, ...]}

Which is WRONG.
Rather, the end users specifies a contribution_ids parameter, and that parameter is a JSON array, like this:
[0, 1, ...]

After much research, I believe that this should be possible, but that for some reason swagger-blocks does not support it, and thus must be a bug.

Check out my comment here from a while back, I think it might answer your question:
https://github.com/fotinakis/swagger-blocks/issues/4#issuecomment-70575007
Please re-open if I'm wrong and you're talking about something else. :)

That doesn't help. Let me post you links to 2 screenshots that illustrate the problem.

In that screenshot, you can see the "contributions" body parameter taking an object with a key of "contributions", which is associated with an array of objects. I.e., the documentation makes it look like this is what the body payload for the endpoint is:
{
  contributions: {
    contributions: [
      {id: 0, name: 'whatever', ...},
      {id: 1, name: 'blah', ...}
    ]
  }
}

Which is WRONG. What I want is to produce documentation that looks like this:

In that pic, you can see that the swagger docs make it look like the body payload should be as follows:
{
  contributions: [
    {id: 0, name: "something", ...},
    {id: 1, name: "blah", ...}
  ]
}

THAT is correct! And also what I want the swagger docs to generate. But I can't seem to make that happen.

Hi, Mike,
I posted an update three days ago to help clarify the issue I'm having.
Hoping you've had a chance to look at it. I allow that I could be missing
something very obvious, but otherwise I do believe there is a genuine bug
that needs to be fixed.
Best regards,
~Damon
On Mon, Jan 25, 2016 at 3:57 PM, Mike Fotinakis<EMAIL_ADDRESS>wrote:

Check out my comment here from a while back, I think it might answer your
question:
#4 (comment)
https://github.com/fotinakis/swagger-blocks/issues/4#issuecomment-70575007
Please re-open if I'm wrong and you're talking about something else. :)
—
Reply to this email directly or view it on GitHub
https://github.com/fotinakis/swagger-blocks/issues/38#issuecomment-174684677
.

Yes I saw, I've been too busy to dig in at the moment. I'll have a chance to look again tomorrow.

I think your problem is that you're not using the key :'$ref' feature to reference the model.
For example:
See here for the key :'$ref' usage to reference a model in a POST request. https://github.com/fotinakis/swagger-blocks/blob/v1.2.0/spec/lib/swagger_v2_blocks_spec.rb#L100
See here for how to use the $ref feature with an array:
https://github.com/fotinakis/swagger-blocks/blob/v1.2.0/spec/lib/swagger_v2_blocks_spec.rb#L86
        schema do
          key :type, :array
          items do
            key :'$ref', :Pet
          end
        end

I believe this is supported by swagger-blocks and by Swagger (see the createUsersWithArrayInput example). If you find that swagger-blocks cannot support the Swagger feature please re-open this issue, but I think this is just a matter of digging further in to figure out the correct Swagger format. You can try changing the underlying Swagger JSON manually until you get Swagger UI to be what you want, then see if you can replicate that inside swagger-blocks.

We're using both arrays and $ref in other parts of our documentation. And when employed as you point out in your examples, it does indeed work. With the exception of trying to specify an array as the value of a body parameter. That's why we find this so difficult to understand. In every other context, e.g., in responses, everything works as advertised. But the exact same syntax employed in those other contexts simply does not work when employed in association with a body parameter.

Interesting. Your original example shows this:
api do
  key :path, '/contributions/bulk_submit'
  key :description, 'Efficiently submit a bunch of contributions by ID'
  operation do
    key :method, 'PUT'
    key :summary, 'Submits all the specified contributions'
    key :nickname, :Contributions_BulkSubmit
    parameter do
      key :paramType, :body
      key :name, :contribution_ids
      key :description, 'An array of the individual contribution IDs being submitted'
      key :required, true
      key :type, :ArrayContributionSubmitPayload
    end
    response_message do
      key :code, 200
      key :message, 'Success.'
    end
  end
end

You're saying that this doesn't work?
api do
  key :path, '/contributions/bulk_submit'
  key :description, 'Efficiently submit a bunch of contributions by ID'
  operation do
    key :method, 'PUT'
    key :summary, 'Submits all the specified contributions'
    key :nickname, :Contributions_BulkSubmit
    parameter do
      key :paramType, :body
      key :description, 'An array of the individual contribution IDs being submitted'
      key :required, true
      key :type, :array
      items do
        key :'$ref', :ArrayContributionSubmitPayload
      end
    end
    response_message do
      key :code, 200
      key :message, 'Success.'
    end
  end
end

Removed key :name, :contribution_ids, and added:
      key :type, :array
      items do
        key :'$ref', :ArrayContributionSubmitPayload
      end


Feature/PRLC100-209

JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/PRLC100-209
Change description

Add content to c100-rebuild/contact-representative page

Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[x] No

@giorgi-hmcts Can we move 'contact-representative' folder inside src/main/steps/c100-rebuild/screening-questions as this is part of screening questions feature. Also please append /screening-questions in the URL

Moved 'contact-representative' inside inside 'src/main/steps/c100-rebuild/screening-questions' and updated the URL ✅


Implemented test matrices: pascal and toeplitz

For pascal, only the symmetric form of the matrix is implemented.
For toeplitz, the implemented functionality is as with Matlab, except for consideration for complex numbers

@Gaoooooo Could you help with this pull request? Thanks

Also, I added toeplitz, pascal to all.hhs, and pascal_test to test.hhs


Adding HelloBar script.

What's this PR do?
This PR adds a script for using HelloBar to add a persistent bar at the top of all Ashes pages to prompt visitors to register to vote. HelloBar will also let us do some customizations and A/B tests to the bar to see which variations perform better.
Any background context you want to provide?

Relevant tickets
Refs Pivotal #155590930

Before merging, @ashleybaldwin should confirm that the settings we want in HelloBar for the initial prompt are all set!


Share Panel Display Issues

When switching between share panel options, the display of the previous option remains visible.  Also, the email link now generates only text of the hyperlink, without generating an email...is this how it is intended? If so, we should consider a different name for this function.

We can see and will correct the persistence of the map extent information.  You are correct, the email link feature now just provides a link to email.  Many user's struggled to properly define their preferred email client in the past making the original email function non-responsive.

David - Would you check our instance to see whether it is functioning properly? The email isn't a link; it's just text & in order to make use of it, the user needs to manually copy & paste the url.  Is this the way that it is intended?

Sorry for the confusion.  Your understanding is correct. The "email link" feature is designed to provide a text URL that users can then copy and paste into a email.

See issue #58.  We will make this easier to use in the next update.

Fixed.


add GH action testing

What does this PR do?
Add Github action for CI testing
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

Codecov Report

Merging #29 into master will decrease coverage by 2.54%.
The diff coverage is 23.04%.

@@            Coverage Diff             @@
##           master      #29      +/-   ##
==========================================
- Coverage   42.58%   40.03%   -2.55%
==========================================
  Files          40       47       +7
  Lines        1975     2298     +323
==========================================
+ Hits          841      920      +79
- Misses       1134     1378     +244

Impacted Files
Coverage Δ

pl_bolts/datamodules/imagenet_dataset.py
17.50% <0.00%> (-0.12%)
:arrow_down:

pl_bolts/datamodules/ssl_imagenet_dataloaders.py
33.33% <0.00%> (+0.72%)
:arrow_up:

pl_bolts/models/self_supervised/moco/transforms.py
42.10% <0.00%> (-3.61%)
:arrow_down:

pl_bolts/models/self_supervised/cpc/cpc_module.py
19.86% <3.33%> (ø)

pl_bolts/metrics/aggregation.py
18.18% <8.33%> (-11.82%)
:arrow_down:

pl_bolts/losses/self_supervised_learning.py
12.50% <15.90%> (ø)

pl_bolts/datamodules/stl10_dataloaders.py
22.66% <16.66%> (-3.75%)
:arrow_down:

pl_bolts/optimizers/layer_adaptive_scaling.py
18.00% <18.00%> (ø)

..._bolts/models/self_supervised/moco/moco2_module.py
19.44% <19.44%> (ø)

pl_bolts/models/self_supervised/moco/callbacks.py
26.31% <26.31%> (ø)

... and 18 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 b41863b...bfa0100. Read the comment docs.


Add .dockerignore that only sends relevant files to the build process

Reduces the files that are sent to the build context by getting docker to ignore all files except for the files directory.

LGTM


Error when trying to simulate multiple bodies

When trying to simulate a scene with multiple bodies, e.g. one of them a free falling sphere, I receive the error: "TypeError: take requires ndarray or scalar arguments, got <class 'NoneType'> at position 1."
The simulation works in MuJoCo.
Below find a small snippet to reproduce the issue:
scene_xml = """
<mujoco>
	<option timestep="0.005"/>
	<worldbody>
		<body pos="0 0 3">
			<joint type="free"/>
			<geom size="0.5" type="sphere"/>
		</body>
		<geom size="40 40 40" type="plane"/>
		<body pos="1 0 0">
			<geom size="0.5" type="sphere"/>
		</body>
	</worldbody>
</mujoco>
"""

ball = mjcf.loads(scene_xml)
qd = jp.array([0, 0, 0, 0, 0, 0])
state = jax.jit(pipeline.init)(ball, ball.init_q, qd)
// Advance physics, error occurs here
jax.jit(pipeline.step)(ball, state, None)

Hi @hackertyper , I just ran the code in a colab, and can't reproduce the error.  Will close the issue for now

Hi @btaba, I also cannot reproduce it in the current version anymore, so consider it fixed. Thanks!


For Android ?

I loved your app for macOS. I enjoyed it a lot and I wonder if you can do this for Android ? I would like to have it on my Android TV Sony Bravia OLED because I tried other iptv androids and I don't liked them. I prefer your app. Please reconsider ? Thanks.

Hi, thank you for the feedback :+1:  There is no separate application for Android, but you can try the PWA: https://iptvnator.vercel.app/


Please remove read/write User Permission

Pls remove the read/write permission on the page
https://github.com/mininet/mininet/wiki/BGP-Path-Hijacking-Attack-Demo
It can be deleted by anyone else.
Or can be edit even can upload anything to your repo

Yes, that's how a wiki works.  If it's vandalized, anyone can revert it.


Unable to delete the first run in Aim UI

Here is an example of trying to delete the first run in Aim UI.
The run was created in v0.2.0 of FML.
https://github.com/G-Research/fasttrackml/assets/3692455/7daee554-4e6c-4162-a8b3-46897c151dbf

it looks like this is because the first run gets assigned row_num 0


fix: Placed app errors into the table of apps, instead of aborting

Fixes #155
When encountering issues with an app while listing all, it is better
to show the errors as part of the table, instead of blocking the
entire listing. This makes it easier to see if the cause is likely
more general, or more specific to some apps.

I pushed 2 apps, I manually deleted the ingress of the first one and listed apps with carrier:
~/workspace/carrier (issue-155-app-listing)*$ ./dist/carrier-linux-amd64 apps
W0223 07:14:18.296624  733959 warnings.go:70] extensions/v1beta1 Ingress is deprecated in v1.14+, unavailable in v1.22+; use networking.k8s.io/v1 Ingress

🚢 Listing applications
Organization: workspace

✔  Carrier Applications:
|   NAME    | STATUS |                 ROUTES                  |
|-----------|--------|-----------------------------------------|
| wordpress | 1/1    | failed to list ingresses:               |
|           |        | ingresses.networking.k8s.io             |
|           |        | "wordpress" not found                   |
| worpress2 | 1/1    | worpress<IP_ADDRESS>.38.omg.howdoi.website |
~/workspace/carrier (issue-155-app-listing)*$

not bad :) The output could become weird if there is a lot to print. But for now, it's better than before.

The output could become weird if there is a lot to print. But for now, it's better than before.

Yes. That is something in the underlying table implementation, I would think.
IOW the https://github.com/olekukonko/tablewriter
It might be as simple as adding code detecting terminal width to it, so that it can stretch the table.
Still, out of scope for now.


Possible memory leak when using linked processes

I'm doing some measurements on resource usage (memory and power mainly) and found a possible memory leak.
When spawning linked processes that end normally, the available Free Heap is lower than before starting the processes.
e.g.
f() ->
    [spawn_link(fun() ->  timer:sleep(500) end) || _ <- lists:seq(1, 50)].

In contrast, when spawning linked processes that terminate, the available Free Heap is more or less the same as before starting the processes.
e.g.
f() ->
    [spawn(fun() ->  ephemeral_start() end) || _ <- lists:seq(1, 50)].

ephemeral_start() ->
    spawn_link(fun() -> ephemeral() end).

ephemeral() ->
    timer:sleep(500),
    exit(normal).

I suspect, that memory related to the link remains allocated.
I'm doing the measurements on an ESP32 system.

Is this related to this comment?
https://github.com/atomvm/AtomVM/blob/fc824ad569648e1b75b9055b8ef819503196923c/src/libAtomVM/context.c#L392

It could possibly be if that code also relates to links.
However I can confirm that a similar leak also occurs when monitoring spawned processes:
f() ->
    Pids = [spawn(fun() -> timer:sleep(500) end) || _ <- lists:seq(1,N)],
    [erlang:monitor(process, P) || P <- Pids].

Links are handled internally there, this is the case where monitor->ref_ticks == 0.


Add Trans*Code to Schedule

Meta-bug for me to keep track of Trans*Code plans and add it to schedule when required.
@nceder commented 3 hours ago:
Presumably if we are doing TransCode, we'll want a space on Friday? My
thinking was that it would only be on the Friday, although the Django Girls
had suggested that having the TransCode hack day the after their workshop
would be nice - that way some of the newly trained people could
participate. Depends on the space/resources available.
@nceder commented 3 hours ago:
You're quite right about the agreement, the thing is we hadn't worked out a
ton of details. As I see it (and I would want to confirm this with Olivia
and Rina) what we talked about was:

Django Girls on Friday, with capacity X (I don't remember the exact
number), with some special outreach/accommodation to the trans* community.

Trans*Code Hackday for ~30 people with some special
outreach/accommodation to Django Girls/PyLadies (i.e., Django Girls who've
already completed the workshop)

Their (Django Girls orgs) suggestion was that having the hack the day
after their workshop would let those people choose to participate. My
assumption was that there wouldn't be space on the Saturday, since the kids
day really fills things up.

At this point I don't think we have anything more worked out than that.

Skeleton for this was added.


Investigate the use of enumerations instead of strings for Natural language classifier

Classifier status is a string instead of an enumeration. There are places in the code where we may want to check the classifiers state, e.g.
classifier.getStatus().equals(Status.AVAILABLE)

This prevents us from needing to hardcode the status strings, or define them as constants in our codebase.

Fixed


Tests not running for database backend

Hi!
I tried to run tests with html report from coverage -- and this is what I've got:

As I can understand, test_database.py not running correctly and run tests on redisd backend (I've put debug output into test_database.py).
This is not local problem, because travis coverage report also confirm that database/__init__.py doesn't have correctly testing:

Can you elaborate a fix?

@camilonova thanks for the answer! I'll try to provide fix.

@prokaktus Thank you.


feat: Replaced all mentions about multiple source file with Web IDE

@mtrunkat I think the API docs should be fine as it only explains that the "Actor source code is comprised of multiple files" which is still true and it stays like this. The Web IDE is only connected to the platform's user interface and I replaced all the mentioning of the Multiple source files type in this PR.
Closes apify-web #2597

Perhaps we could mention in the API that multiple source files as shown and editable in the Apify Console UI as web IDE. But the rest is fine.


Possible routing error: Preference of highway=footway over highway=residential

Important note: https://www.openstreetmap.org/changeset/73198089 changed the situation described in this issue by adding stairs on the footway.
From: 8FVC7HG8+7H
To: 8FVC7HG9+95
Routing looks like this:
https://www.openstreetmap.org/directions?engine=graphhopper_bicycle&route=47.27557%2C8.56648%3B47.27566%2C8.56822#map=19/47.27580/8.56732
(going over Holzgasse, a highway=footway)

but IMHO should look like this:
https://www.openstreetmap.org/directions?engine=fossgis_osrm_bike&route=47.27557%2C8.56648%3B47.27566%2C8.56822#map=18/47.27618/8.56703
(going the longer way over Hintere Bergstrasse, a highway=residential)

My understanding of https://wiki.openstreetmap.org/wiki/Tag:highway%3Dfootway is that bicycles should not be routed there unless there is also a bicycle=yes (which is not the case in this example).
The same behaviour occurs in OsmAnd: https://github.com/osmandapp/Osmand/issues/7372
Edit: Fix link

If you get off the bike and if you are still faster shouldn't we pick this route?
There might be still something wrong - highly likely something with the elevation as the other profiles seem to work fine. (especially now that there are stairs)

The average_speed is as expected only 6km/h

The alternative route is partly 10km/h (otherwise 18km/h)

The reverse directions is likely ok, because uphill the 6km/h is a 2km/h.
Of course bike is more than just speed, but in this example highly likely the problem. The data update will come soonish (we had an issue in our auto deployment pipeline)

If you get off the bike and if you are still faster shouldn't we pick this route?

Fair question. I don't know. Would this question warrant the creation of a corresponding configuration setting?

A configuration option that can be send per request will come (currently it is even required to code in Java), but it will likely not be available from the UI and probably take even more months (see this work). Still we need some default value :)

Can we close here?

To summarize: Graphhopper routes over the footway instead of the residential street, since it's significantly shorter and indicates to the user that he/she has to get off the bicycle?
Sounds fair to me... So this issue can be closed.


⚠ ClamAV has degraded performance

In 75e1f6f, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 200 ms

Resolved: ClamAV performance has improved in 1921d74.


Don't works in new CLion (error on the screenshot)

I currently don’t have much time to maintain this plugin, you can modify plugin version range here
https://github.com/xmake-io/xmake-idea/blob/b3340ced4096e9eb111adf8f27d6496fbc66b097/gradle.properties#L4-L12
Then compile a local jar package and load it directly, and welcome to open a pr to help modify the version number

I have uploaded the update, please wait for the official review time of one day


fix: pass down ADOPT_DEFAULT_JSON when create downstream job

- the content might be different if the generator job use a different branch than master

So the issue was:
when changed to use USE_ADOPT_SHELL_SCRIPT is true for the release pipeline,
the release-openjdk-pipeline was update with correct release branch in both DEFAULTJSON and ADPOTDEFAULTJSON
E.G https://ci.adoptopenjdk.net/job/build-scripts/job/utils/job/release_pipeline_jobs_generator_jdk17u/23/parameters/
but the downstream build job only got correct DEFAULTJSON , ADOPTDEFAULTJSON was still used the content from master branch. because it was not passed down when create object
e.g https://ci.adoptopenjdk.net/job/build-scripts/job/jobs/job/release/job/jobs/job/jdk17u/job/jdk17u-release-linux-x64-temurin/1/parameters/ instead of using repoHandler.getAdoptDefaultsJson()
so either we add a new support function in "helper" similar to setUserDefaultsJson() but to setAdoptDefaultJson()
and add repoHandler.setAdoptDefaultJson(context, ADOPT_DEFAULTS_JSON)
or remove Map ADOPT_DEFAULTS_JSON = repoHandler.getAdoptDefaultsJson()
no matter which was, we still need the ADOPT_DEFAULTS_JSON to be passed down

Class Regeneration will also need updated? https://github.com/adoptium/ci-jenkins-pipelines/blob/master/pipelines/build/common/config_regeneration.groovy#L59-L97?

run tests quick 19

run tests quick 19

run tests quick 19

I think to make this PR work  ADOPT_DEFAULTS_JSON need to be reset https://github.com/adoptium/ci-jenkins-pipelines/blob/v2023.01/pipelines/build/regeneration/build_job_generator.groovy#L37-L38 ( which will make the ADOPT_DEFAULTS_JSON to be the default master one) @zdtsw

I think to make this PR work ADOPT_DEFAULTS_JSON need to be reset https://github.com/adoptium/ci-jenkins-pipelines/blob/v2023.01/pipelines/build/regeneration/build_job_generator.groovy#L37-L38 ( which will make the ADOPT_DEFAULTS_JSON to be the default master one) @zdtsw

@sophia-guo @smlambert @zdtsw So this code in this PR has not been run as we haven't run the re-gen.., so I think we keep it that way, thereby we know the build jobs should work as they did for the dry-run.
Yes, I think that means we're building with build-scripts "master" branch not the v2023.01, but I think it's too late to try to fix..
[Pipeline] echo
17:28:50  [CHECKOUT] Checking out Adopt Build https://github.com/adoptium/temurin-build.git : master
[Pipeline] checkout

Correct me if there's something I might have missed?

The Pr is expected to set ADOPT_DEFAULT_JSON  same as DEFAULT_JSON for build jobs, which is related with https://github.com/adoptium/ci-jenkins-pipelines/pull/587.
Agreed it's not necessary to be in this release as current there is no issue with smoke tests. Can simply update to master https://github.com/adoptium/ci-jenkins-pipelines/pull/595


OXCGRT key map

Chris Grundy: "the other issue I have notices which I think we reported was that the oxford data is being put in wrong field for oxford data,  where the  compliance penality field if filled out with required which should be in the enforcement:"

For the OXCGRT records which have “required” in the non_compliance_penalty field the values we can assign to OXCGRT records in the non_compliance_penalty are “recommended” and “required” Should I remove both of those?

It seems like these are not appropriate non_compliance_penalty values so they should be removed from the coding and from the permitted values in thee dataset check - is that correct?

Closed in #107. These values will be moved to the enforcement column.


use static class methods to wrap calls to singleton

so that CmdLineOptions::instanceGet()->ParseOptions() is just CmdLineOptions::ParseOptions()
also what is the normal

aside:
This would probably fix the 87% coverage of:
https://app.codecov.io/gh/byronwatt/CmdLineOptions/blob/main/example/example_with_error_message.cpp
which looks kinda strange, but perhaps GetInstance is an inline function that has a branch for first/second time called ?


[Feature Request] Move table upload and installations progressable to jobs

Support a "move to background" action for progressables so that they end up in the jobs menu.

Duplicates https://github.com/syd711/vpin-studio/issues/561


Custom day range for boundaries

It would be great to have the ability to customize the fixed value of 10 days in the settings and/or have an option that aligns the time span with the current week (based on the global week start setting).
BTW Nice plugin! Thank you for your effort :+1:

I will release new version.
Add to plugin settings

The boundaries 2 week mode (only journals)

default: false

The boundaries custom day range: before today (Excludes 2 week mode)
The boundaries custom day range: after today (Excludes 2 week mode)

Regarding the upcoming two-week mode that I will release, there is still room for improvement in terms of design, but for now, I will include it in the release.

Thank you so much for implementing the feature!
I have a small suggestion: Would it be possible to allow setting the custom day range for 'before today' to 3 days instead of the minimum value of 5? This would help cover a full week in total. It would be really helpful!
Once again, thank you for your efforts!

OK!


Simplify code

This code is actually slower. You're always doing an assignment rather than conditionally, so thus this code is worse.

:+1:


Test issue please ignore

Where does this show up in the "Issue Triage" project?

Ah, issues only go to "needs triage" when I add them to the Issue Triage project.  That's neat I guess but I was really hoping for all new issues to show up there :disappointed:


feat: move InternalsVisibleTo in csproj

What does the pull request do?

move InternalsVisibleTo in csproj and add AvaloniaPublicKey.props
What is the current behavior?

What is the updated/expected behavior with this PR?

How was the solution implemented (if it's not obvious)?

Checklist

[ ] Added unit tests (if possible)?
[ ] Added XML documentation to any related classes?
[ ] Consider submitting a PR to https://github.com/AvaloniaUI/Documentation with user documentation

Breaking changes

Obsoletions / Deprecations

Fixed issues

You can test this PR using the following package version. 0.10.999-cibuild0020679-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]

Nice cleanup!
There is an incorrect spelling in the AvaloniaPrublicKey.props

You can test this PR using the following package version. 0.10.999-cibuild0020695-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]

You can test this PR using the following package version. 0.10.999-cibuild0020704-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]


distribution_scope: handle imported images

These have {'Labels': None} in their inspection data. They occur during creation of base images (pre_add_filesystem).

Coverage remained the same at 88.213% when pulling 45f882716db61957433196ed8844a2d5240c551b on distribution-scope-image-build into 9a5fba73b986e9f0f0c92750b2985e2e04e71221 on master.

Verified it fixes the issue.


Make data migration plan for Adventist Cutover

Blocked by https://github.com/scientist-softserv/adventist_knapsack/issues/755


Fix: add subnet to env

Description
Adds possibility to update docker network from .env file instead of editing docker compose.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[x] Refactor/improvement

How Has This Been Tested?
manual, integration
Checklist

[x] My commit message is conventional
[x] I have run pre-commit on all files before creating the PR
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I have checked my code and corrected any misspellings

:tada: This PR is included in version 1.12.1-beta.5 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Fix introduced bug + add linter action (chore)

Critical: fix broken onDrag callback from latest PR (startTime was undef, linter catches these errors)

Add linter.yml

Upgrade codeql from v1 to v2 https://github.blog/changelog/2022-04-27-code-scanning-deprecation-of-codeql-action-v1/

Thanks for the fix, LGTM.


feat: add implementation for array/base/mskbinary5d

Resolves #3180  .
Description

What is the purpose of this pull request?

This pull request:

implements the mskbinary4d package for masked binary operations in the array/base namespace.
and provides a 4D equivalent of the mskbinary4d package for binary callbacks using a mask array.

Related Issues

Does this pull request have any related issues?

This pull request:

resolves #3180

Questions

Any questions for reviewers of this pull request?

No.
Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

No.
Checklist

Please ensure the following tasks are completed before submitting this pull request.

[x] Read, understood, and followed the contributing guidelines.

@stdlib-js/reviewers

@Planeshifter Please review this PR.

/stdlib lint-autofix


Add Amuthan Sakthivel to the speaker list

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


Failed to setup DB with some ASCII characters in file path

For example this setup will fail:
DataMapper.setup :default, 'sqlite:///path/#/!!db.sqlite'

Log:
/usr/local/share/gems/gems/data_objects-0.10.16/lib/data_objects/connection.rb:79:in `initialize': unable to open database file (DataObjects::ConnectionError)
        from /usr/local/share/gems/gems/data_objects-0.10.16/lib/data_objects/connection.rb:79:in `new'
        from /usr/local/share/gems/gems/data_objects-0.10.16/lib/data_objects/pooling.rb:181:in `block in new'
        from /usr/local/share/gems/gems/data_objects-0.10.16/lib/data_objects/pooling.rb:176:in `synchronize'
        from /usr/local/share/gems/gems/data_objects-0.10.16/lib/data_objects/pooling.rb:176:in `new'
        from /usr/local/share/gems/gems/data_objects-0.10.16/lib/data_objects/pooling.rb:121:in `new'
        from /usr/local/share/gems/gems/data_objects-0.10.16/lib/data_objects/connection.rb:68:in `new'
        from /usr/local/share/gems/gems/dm-do-adapter-1.2.0/lib/dm-do-adapter/adapter.rb:251:in `open_connection'
        from /usr/local/share/gems/gems/dm-transactions-1.2.0/lib/dm-transactions/adapters/dm-do-adapter.rb:69:in `open_connection'
        from /usr/local/share/gems/gems/dm-do-adapter-1.2.0/lib/dm-do-adapter/adapter.rb:276:in `with_connection'
        from /usr/local/share/gems/gems/dm-do-adapter-1.2.0/lib/dm-do-adapter/adapter.rb:33:in `select'
        from /usr/local/share/gems/gems/dm-migrations-1.2.0/lib/dm-migrations/adapters/dm-sqlite-adapter.rb:43:in `table_info'
        from /usr/local/share/gems/gems/dm-migrations-1.2.0/lib/dm-migrations/adapters/dm-sqlite-adapter.rb:18:in `storage_exists?'
        from /usr/local/share/gems/gems/dm-migrations-1.2.0/lib/dm-migrations/auto_migration.rb:65:in `storage_exists?'
        from /usr/local/share/gems/gems/dm-migrations-1.2.0/lib/dm-migrations/auto_migration.rb:120:in `storage_exists?'
        from /usr/local/share/gems/gems/dm-constraints-1.2.0/lib/data_mapper/constraints/migrations/model.rb:22:in `auto_migrate_constraints_down'
        from /usr/local/share/gems/gems/dm-constraints-1.2.0/lib/data_mapper/constraints/migrations/singleton_methods.rb:39:in `block in auto_migrate_constraints_down'
        from /usr/local/share/gems/gems/dm-core-1.2.1/lib/dm-core/support/descendant_set.rb:64:in `block in each'
        from /usr/local/share/gems/gems/dm-core-1.2.1/lib/dm-core/support/subject_set.rb:211:in `block in each'
        from /usr/local/share/gems/gems/dm-core-1.2.1/lib/dm-core/support/ordered_set.rb:320:in `block in each'
        from /usr/local/share/gems/gems/dm-core-1.2.1/lib/dm-core/support/ordered_set.rb:320:in `each'
        from /usr/local/share/gems/gems/dm-core-1.2.1/lib/dm-core/support/ordered_set.rb:320:in `each'
        from /usr/local/share/gems/gems/dm-core-1.2.1/lib/dm-core/support/subject_set.rb:211:in `each'
        from /usr/local/share/gems/gems/dm-core-1.2.1/lib/dm-core/support/descendant_set.rb:63:in `each'
        from /usr/local/share/gems/gems/dm-constraints-1.2.0/lib/data_mapper/constraints/migrations/singleton_methods.rb:38:in `auto_migrate_constraints_down'
        from /usr/local/share/gems/gems/dm-constraints-1.2.0/lib/data_mapper/constraints/migrations/singleton_methods.rb:7:in `auto_migrate!'

Would you please fix this bug?
Thank you.

Moved to data_objects