Won't cast spell on @mouseover target

This macro doesnt cast Taunt on my mouseover target:
/cast [stance:2 nocooldown:Taunt @mouseover] Taunt
This works as intended (taunts my mouseover target if I'm next to it - even if I have another target chosen in the UI)
/cast [stance:2 nocooldown:Taunt @mouseover]  {"/run CastSpellByName("Taunt","mouseover")"}
My guess is the add-on tries to target the normal target no matter if @mouseover is chosen? (Maybe it checks mouseover for the conditional, but then reverts to the normal target before actually casting the spell?)

Thanks for the report.  It's a known issue that happened while trying to deal with HealComm support.  I already have it resolved, just haven't released an update before I work out which way I want to apply the fix to try and not break HealComm or make things more complex than they need to be.  I'll shoot you a reply as soon as I push an update.

As a follow-up, I did push the update I mentioned.  If you use HealComm, please see the README for information.  That aside, I just now more closely read your macro -- unitsids, spell and item names, and most things are case sensitive.  Does it work with @mouseover instead of @mouseOver  ?

Can confirm that it works as intended now. :-)
(And yeah, github changes how '@'mouseover is displayed, as you noticed.)


ERROR: [SSL: CERTIFICATE_VERIFY_FAILED]

Hello,
After I downland installed the GAM and trying to configure it I got this error:
ERROR: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1129)
Project creation failed. Trying again. Say N to skip project creation.
Need help here, please
I am using Mac Big Sur version 11.6

Please post on:
git.io/gam-group


Paper for JOSS

This is currently in a very WIP state, but I wanted to at least get it started so we can build up some momentum before Vizcon.

This is what the paper looks like now, BTW:
paper.pdf
CCing people from https://github.com/JuliaPlots/Makie.jl/issues/428 in here:
@kescobo @mkborregaard @SimonDanisch @jkrumbiegel @Datseris @Kevin-Mattheus-Moerman

I need to be detached from this if you ask for me to be a reviewer, so I'll go ahead and unsubscribe from this post :)

Fair enough, I just blanket pinged everyone in the thread anyway :D.  There's no content in this as of now.

Please feel free to use my figure - I'd only ask that you cite https://doi.org/10.1101/2<IP_ADDRESS>4181 (if it gets peer reviewed and published elsewhere, I'll add that citation).
One thought though - @mkborregaard was suggesting possibly multiple papers, including a separate one for MakieLayout, in which case my figure fit better there.
I'd be happy to help with writing etc too, though would want to talk about being included as an author before getting too heavily involved. That said, I actually don't think I'd be of much use on that front, as all I did was ask @jkrumbiegel for help and do what he told me :laugh:. I'm perfectly happy to do some rounds of review / comment once there's more content without expecting author credit.


/etc/init.d/phpbrew-fpm is not writable.

Output
*WARNING* php-fpm --pid option requires php >= 5.6. You need to update your php-fpm.conf for the pid file location.
/etc/init.d/phpbrew-fpm is not writable.

Command
phpbrew fpm setup --initd
Platform
OS:
Debian 8
Running PHP: 7.3.10

I don't have the Debian 8, but I use the Debian 10 to try to reproduce the issue. Here are my steps to build/compile PHP 7.3 in Debian 10:

Prepare fresh and clean Debian 10.
Install some required packages with apt install command.
Install the PHPBrew 1.27.0 version.
Running the phpbrew install 7.3 +default +fpm to build and compile latest PHP 7.3 version.
After compiling latest PHP 7.3 version, running the phpbrew switch php-7.3.33 command to switch the this PHP 7.3 version.
Running the phpbrew fpm setup --initd command to setup the init FPM service. And the outputs are as follows:

$ phpbrew switch php-7.3.33
$ phpbrew fpm setup --initd
*WARNING* php-fpm --pid option requires php >= 5.6. You need to update your php-fpm.conf for the pid file location.
Writing init.d script: /etc/init.d/phpbrew-fpm
To setup the startup item, remember to run update-rc.d to link the init script:
    sudo update-rc.d phpbrew-fpm defaults

I don't have the issue in Debian 10 and I think the issue is going to be closed.


COOK-107 Use Java cookbook jdk.sh before BigTop tools

This will source the Java cookbook's jdk.sh prior to running the BigTop tools for detecting JAVA_HOME, which works for Java set outside the java cookbook.

If I understand correctly, if java cookbook is used, then bigtop will be a no-op unless they explicitly set JAVA_HOME via /etc/default/bigtop-utils.  if java cookbook is not used, then bigtop will still autodetect.  I suggest adding a comment to that effect... current comment implies it will autodetect
travis failures, but otherwise 👍 lgtm


Tap not detected on SINGLE_MANUAL

Hi, on SINGLE_MANUAL mode, when multiple barcodes are detected, they are highlighted on screen, but when tapping on one of them, nothing happens. Sometimes the selection action is triggered if tapping randomly near the highlighted areas and one of the codes is selected.
Do you know something about this issue? I need help as quick as possible.
Thanks.

@samurake You need to tap inside the highlighted box to select that barcode. The barcode detection is a continuous process and the highlighted boxes are updated after processing each frame from camera. The tap might do nothing in case if the box just moved as soon as you tapped and the point of tap is no longer inside the box. This should be a rare case though.

I am tapping inside the highlighted box, but nothing happens. I cloned your sample app code to see if the bug is in my app or the library itself and I made a screen recording so you can see it's behavior.
https://drive.google.com/open?id=0B-ZbHNmWeq_QVThVME5ldmJKUWM
Can you help me with a fix as soon as possible?

As I said before, sometimes it randomly works, but that rarely happens. Maybe there's a memory leak on the touch listeners for the highlighted areas...

@samurake the scan with dialog is a test setup for testing #15, that setup is not recommended.

So you can't fix it ?

@samurake pull the develop branch and test. The issue should be resolved.


[docs] Fix regex for ReactNative transformIgnorePatterns

The current example for using transformIgnorePatterns to test ReactNative apps doesn't matches anything because the / is at the end.
new RegExp("node_modules/(?!react-native|my-project|react-native-button)/").test('node_modules/')
false
new RegExp("node_modules/(?!react-native|my-project|react-native-button)/").test('node_modules/react-native/')
false
new RegExp("node_modules/(?!react-native|my-project|react-native-button)/").test('node_modules/my-project/')
false
new RegExp("node_modules/(?!react-native|my-project|react-native-button)/").test('node_modules/my-project-foo/')
false

When we put the slash inside the parens, it does what I believe we expect.
new RegExp("node_modules/(?!(react-native|my-project|react-native-button)/)").test('node_modules/')
true
new RegExp("node_modules/(?!(react-native|my-project|react-native-button)/)").test('node_modules/react-native/')
false
new RegExp("node_modules/(?!(react-native|my-project|react-native-button)/)").test('node_modules/my-project/')
false
new RegExp("node_modules/(?!(react-native|my-project|react-native-button)/)").test('node_modules/my-project-foo/')
true

Codecov Report

Merging #3596 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #3596   +/-   ##
======================================
  Coverage    62.4%   62.4%
======================================
  Files         181     181
  Lines        6648    6648
  Branches        6       6
======================================
  Hits         4149    4149
  Misses       2496    2496
  Partials        3       3

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 709db1e...e9a130f. Read the comment docs.

Thanks Eli!


Fix #16543 (Add ability to copy errors from debugger)

Fixes #16543.
Adds button to copy errors (warnings and successes too) from debugger.
Looks like this:

The button doesn't look obvious to me...

Copy icon is usually like 2 sheets of paper, one over the other (there is no copy icon in the editor?).

Oh, yes, duplicate looks more like a copy than the node path one.

I was searching for copy editor icons and that's why I didn't notice Duplicate icon exists. I amended the commit.

Thanks, looks good.


Spyder does not recognize rtree 0.9.4

Issue Report Checklist

[x] Searched the issues page for similar reports
[x] Read the relevant sections of the Spyder Troubleshooting Guide and followed its advice
[x] Tried basic troubleshooting (if a bug/error)

[x] Restarted Spyder

[x] Completed the Problem Description, Steps to Reproduce and Version sections below

Problem Description
I did a fresh install of Miniconda in Python 3.8, created a new environment, and installed Spyder. Nothing has been yet configured or changed. On startup Spyder always complains about rtree >=0.8.3 being a missing dependency. Conda list shows that rtree 0.9.4 is installed which satisfies the version requirement.
What steps reproduce the problem?

Create fresh conda environment with a fresh Spyder install on Python 3.8.
Start Spyder

What is the expected output? What do you see instead?
Spyder should acknowledge that rtree is installed.

Hi @Khris777 thanks for reporting this. It seems this problem is not related to Spyder but to miniconda. Several people have reported it before and it seems like installing from conda-forge is a good solution https://stackoverflow.com/questions/59981914/missing-dependancies-of-rtree.
I'll close this one since you managed to solve it but let me know if there is anything else we can help you with.


FINERACT-1791: Remove Travis CI from documentation and syncronize the MariaDB version to version 10.9 in all the documents

Description

Travis CI text has been replaced to reflect the current use of GitHub Actions
MariaDB version has been updated to version 10.9

Checklist
Please make sure these boxes are checked before submitting your pull request - thanks!

[X ] Write the commit message as per https://github.com/apache/fineract/#pull-requests

[X] Acknowledge that we will not review PRs that are not passing the build ("green") - it is your responsibility to get a proposed PR to pass the build, not primarily the project's maintainers.

[X ] Create/update unit or integration tests for verifying the changes made.

[X] Follow coding conventions at https://cwiki.apache.org/confluence/display/FINERACT/Coding+Conventions.

[X] Add required Swagger annotation and update API documentation at fineract-provider/src/main/resources/static/legacy-docs/apiLive.htm with details of any API changes

[X] Submission is not a "code dump".  (Large changes can be made "in repository" via a branch.  Ask on the developer mailing list for guidance, if required.)

FYI our guidelines for code reviews are at https://cwiki.apache.org/confluence/display/FINERACT/Code+Review+Guide.

Now the tests have been completed and passed.


ingest: deduplicate sequences using strain names

Context
Once we've completed #32, we can use strain names to deduplicate sequences.
This is necessary in case different groups sequence the same virus or if sequences are generated from different protocols.
(NOTE: This is separate from the versioning in GenBank, we already pull in the latest version of GenBank sequences).
Description
The duplicate sequences should probably be filtered out in a new script (e.g. ingest/bin/deduplicate-records) OR potentially use the augur deduplicate command (see https://github.com/nextstrain/augur/issues/919).
We probably want to keep a file with all sequences in case people want the duplicate sequences for any reason.
The deduplicated files will be the main ones used for LAPIS and/or our monkeypox builds.

Update: We currently have a duplicate in the hMPX build (MPXV-M5312_HM12_Rivers from accessions MT903340 and NC_063383). It’s not a huge problem as it's not in the current outbreak.


Moving Exception Handling to be done at the end of the Download/CommitChunkHandler

There was a live flakey test bug introduced where the DownloadChunkHandler/CommitChunkHandler would have an exception thrown when invoking an event however nothing there was catching it. Putting the transfer job in a state of endless "InProgress".
Also the exceptions were getting logged, however disposing the *ChunkHandlers too early. We want to log the exception and then dispose the handler as well. So that way things are disposed correctly.
This PR also updates the Download and Copy type jobs to allow in one place for the exceptions to be caught and not just whenever it can in the helper methods.

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

/azp run net - storage - tests

Update from latest commit:
In order to add tests for small chunking on download, upload, and copy on directory transfers. It exposed a bug where the failure on exists, when uploading/copying a chunked transfer, wasn't being respected on the commit block list call which affects upload copy. The latest commit contains changes that

Fix for bug where failure on exists, on a chunked upload or copy was not being respected
- API change required on StorageResource.CompleteTransferAsync to pass overwrite parameter
Returns the Event Args to CommitBlockHandler and DownloadChunkHandler when there's a failure to propagate failures in the right order

/azp run net - storage - tests

/azp run net - storage - tests

/azp run net - storage - tests

/azp run net - storage - tests

/azp run net - storage - tests

/azp run net - storage - tests

Live test pipeline failures are unrelated to the changes (E.g. aes gcm mac os tests, immutability policy tests)

/check-enforcer override


Streaming audio input format

Hello,
I would like to know if there is a default input audio format and if so what it was.
I am trying to implement this in AWS, but I am wondering how it can be integrated with other audio/video streaming services.
Thank you

yes. Here: https://github.com/ufal/whisper_streaming?tab=readme-ov-file#server----real-time-from-mic


Pie Chart Label Fails to Update

Description

Steps to Reproduce

Setting config
Re-initilaize config options labels
Not working

Expected Behavior
Actual Behavior
type: "pie",
height: 250,
series: [],
options: {
    chart: {
        width: 250,
        type: "pie",
    },
    colors: [
        "#62B2FD",
        "#9BDFC4",
        "#F99BAB",
        "#FFB44F",
        "#a3e635",
        "#6ee7b7",
        "#67e8f9",
        "#93c5fd",
        "#a78bfa",
        "#f472b6",
    ],
    labels: [],
    legend: {
        show: false,
    },
    responsive: [
        {
            breakpoint: 480,
            options: {
                chart: {
                    width: 200,
                },
            },
        },
    ],
},

Screenshots

Reproduction Link

Reproduction link not provided


[BUG] SignUp - United States is displayed in country dropdown when selecting Canada

Describe the bug
Choosing Canada in the Country dropdown on the Phone/Entry screen during sign up makes United States appear instead.

Canadians have qualified this as a blocker.

To Reproduce
Steps to reproduce the behavior:

Go to https://reach4help-staging-web-client.firebaseapp.com/phone/entry
Choose Canada from the country dropdown
Notice the error

Screenshots
Video here: https://drive.google.com/open?id=11lyj1tw99XcO3U4qcl1F3M1nrLNFsBIt
Desktop (please complete the following information):
Windows - Chrome
Build Version 2.0

ZenHub Information
This information is updated automatically. To modify it, please use ZenHub.

Poor Canadians, we can't let this slide. Also +1 for the TWD reference :)

@CorporateInvesther lol.  this is fixed on development  @sabind

This fix should be pushed to https://reach4help-webclient.web.app/phone/entry as well, as outreach is going to be relaying this to canadian contacts who were previously blocked by this screen.

Verified Fixed, v2.1.0.
Canadians forgive you this time. :)


SeatAndEat :  Add Footer To All Pages

Add Footer to every page, which you would find in the landing page.
Make it responsive.

What would the footer look like? Can you help me understand.

The footer is on the landing page (index.html). You have to add that to other pages too. If you want you can make enhancements to make it look better.

The footer is on the landing page (index.html). You have to add that to other pages too. If you want you can make enhancements to make it look better.

I would like to work on this

Cool.
You may checkout the  KSOC website to register yourself to take part in it and learn about the important dates.
Follow GDSC KGEC linked page to learn about the future updates and events.

Cool. You may checkout the KSOC website to register yourself to take part in it and learn about the important dates. Follow GDSC KGEC linked page to learn about the future updates and events.

I have registered already. I want to clarify something. I have to add the footer which is in home page to : about,menu and contact us .
These are the changes I need to make?

Cool. You may checkout the KSOC website to register yourself to take part in it and learn about the important dates. Follow GDSC KGEC linked page to learn about the future updates and events.

How to get assigned ?

Assign me this issue , I wanna work on it

I have assigned you this. All the best.


SCB-735 Add admin dump api

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Format the pull request title like [SCB-XXX] Fixes bug in ApproximateQuantiles, where you replace SCB-XXX with the appropriate JIRA issue.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Run go build go test go fmt go vet to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Codecov Report

Merging #396 into master will decrease coverage by 5.32%.
The diff coverage is 83.89%.

@@            Coverage Diff             @@
##           master     #396      +/-   ##
==========================================
- Coverage   71.69%   66.37%   -5.33%
==========================================
  Files          39       73      +34
  Lines        4130     5909    +1779
==========================================
+ Hits         2961     3922     +961
- Misses       1002     1773     +771
- Partials      167      214      +47

Impacted Files
Coverage Δ

server/service/notification/common.go
100% <ø> (ø)
:arrow_up:

pkg/async/service.go
72.8% <100%> (ø)

pkg/util/goroutines.go
100% <100%> (ø)

...erver/service/notification/notification_service.go
91.54% <100%> (ø)
:arrow_up:

server/service/watch.go
70.96% <100%> (ø)
:arrow_up:

server/service/notification/stream.go
65.21% <33.33%> (ø)
:arrow_up:

server/service/util/domain_util.go
61.19% <50%> (ø)
:arrow_up:

pkg/etcdsync/mutex.go
66.66% <79.54%> (ø)

server/service/notification/websocket.go
83.87% <82.05%> (+3.58%)
:arrow_up:

pkg/util/log.go
48.76% <0%> (ø)

... and 34 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 7590fec...19ed886. Read the comment docs.

Coverage decreased (-5.8%) to 68.681% when pulling 19ed886ec0e8167a0b4dc564a8cb6fb56ca2d2ca on little-cui:admin into 7590fec4e4447fdd3b2977970d544e88ae3762cf on apache:master.

Please update the swagger.yaml for dump api's


Dockerfile not up to date on Dockerhub

see: https://hub.docker.com/r/prom/prometheus/dockerfile/
This fix has not been applied on dockerhub: https://github.com/prometheus/prometheus/pull/2942/files
ref: https://github.com/prometheus/prometheus/issues/2878

This is a long-standing issue... We got the Dockefile tab removed with https://github.com/docker/hub-feedback/issues/1792 but unfortunately the page itself can't be deleted.

Ok i see. May i suggest to add an example of a docker-compose deployement here : https://prometheus.io/docs/prometheus/latest/installation/ ?
Just explaining the case. I wanted to use the default command and arguments in my deployement but just to enable the admin api in addition. Problem is i've found the ressource on dockerhub and tried to reproduce. I think it has its place in the doc.

Dupe of https://github.com/prometheus/prometheus/issues/5348 as discussed in our bugscrub.


Export as CVAT video format

Summary

Support for exporting as CVAT video format
Add 'use_track' option to export DatasetItem into  tag
Implement CVS-93496
e.g.)
<track id="0" label="label2">
<box frame="4" occluded="1" outside="0" keyframe="1" xtl="3.00" ytl="4.00" xbr="10.00" ybr="5.00" z_order="0">
</box>
.....
<box frame="14" occluded="1" outside="0" keyframe="1" xtl="3.00" ytl="4.00" xbr="10.00" ybr="5.00" z_order="0">
</box>
</track>

How to test

Checklist

[x] I submit my changes into the develop branch
[x] I have added description of my changes into CHANGELOG
[ ] I have updated the documentation accordingly
[x] I have added tests to cover my changes
[x] I have linked related issues

License

[x] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.
[x] I have updated the license header for each file (see an example below)

# Copyright (C) 2021 Intel Corporation
#
# SPDX-License-Identifier: MIT

I resolve a conflict of CHANGELOG via 8ea3be2 after the review approval. @vinnamkim, would you please give an approval again?

Codecov Report
Base: 84.20% // Head: 84.17% // Decreases project coverage by -0.02% :warning:

Coverage data is based on head (8ea3be2) compared to base (1909d85).
Patch coverage: 79.54% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #757      +/-   ##
===========================================
- Coverage    84.20%   84.17%   -0.03%
===========================================
  Files          157      157
  Lines        21855    21925      +70
  Branches      4779     4804      +25
===========================================
+ Hits         18402    18456      +54
- Misses        2365     2374       +9
- Partials      1088     1095       +7

Flag
Coverage Δ

macos-10.15_Python-3.10
?

macos-10.15_Python-3.7
?

macos-10.15_Python-3.8
?

macos-10.15_Python-3.9
?

ubuntu-20.04_Python-3.10
84.12% <79.54%> (+<0.01%)
:arrow_up:

ubuntu-20.04_Python-3.7
83.81% <79.54%> (+<0.01%)
:arrow_up:

ubuntu-20.04_Python-3.8
?

ubuntu-20.04_Python-3.9
84.06% <79.54%> (+<0.01%)
:arrow_up:

windows-2019_Python-3.10
?

windows-2019_Python-3.7
?

windows-2019_Python-3.8
?

windows-2019_Python-3.9
?

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

Impacted Files
Coverage Δ

datumaro/plugins/cvat_format/converter.py
87.53% <79.54%> (-1.23%)
:arrow_down:

datumaro/plugins/mars_format.py
86.15% <0.00%> (-3.08%)
:arrow_down:

datumaro/util/os_util.py
75.75% <0.00%> (-1.52%)
:arrow_down:

datumaro/util/test_utils.py
88.39% <0.00%> (-1.11%)
:arrow_down:

datumaro/plugins/transforms.py
81.43% <0.00%> (+0.35%)
:arrow_up:

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

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


Stopped working - no longer populates the screen

After working perfectly for a few weeks (thank you for a great module, @daniel-windsor ) MMM-eswordoftheday has suddenly stopped working. To my VERY amateur eye it appears to not be getting the correct response from the website.
If I use Chromium on the Pi, it renders the https://www.spanishdict.com/wordoftheday page perfectly.
As you can see on the images, I have tried to debug but have so little prior understanding of MM and css, I'm fumbling in the dark! ;-) so any help would be VERY much appreciated!
Thanks.

Thanks for letting me know!  I would guess the web scraper isn't working because they changed something about their site. I'll check it out.

Thank you so much!

Hi Daniel. Is there anything I can do to help? I am by no means a programmer but can 'dabble' and can test if that helps?

I was really looking forward to use this module for my wife's mirror but I'm having the same issue with no data.

Same here

I did this changes (node_helper.js):
//const container = $(".entry--1qTEOKKt")[0];
 container = $("._1qTEOKKt")[0];
and it works
`

THANK YOU!
I edited the node_helper.js file inline, then rebooted and it worked! I did have to make a slight change to your text above and ended up with:
       //const container = $(".entry--1qTEOKKt")[0];         const container = $("._1qTEOKKt")[0];
Thank you so much - my wife is learning Spanish and this will make her day!

I did this changes (node_helper.js):
//const container = $(".entry--1qTEOKKt")[0];
 container = $("._1qTEOKKt")[0];
and it works
`

You should do a pull request

THANK YOU!
I edited the node_helper.js file inline, then rebooted and it worked! I did have to make a slight change to your text above and ended up with (I did try formatting as code but couldn't force the new line on here):
//const container = $(".entry--1qTEOKKt")[0];
const container = $("._1qTEOKKt")[0];
Thank you so much - my wife is learning Spanish and this will make her day!

De nada ;-) (You'are welcome)

I did this changes (node_helper.js):
//const container = $(".entry--1qTEOKKt")[0];
 container = $("._1qTEOKKt")[0];
and it works
`

You should do a pull request

Yes, I'll try this week

Fixed.  Only took nearly a year :sweat_smile:


Business cases sends many requests to the server?

Assigning @antw

That's expected. The business case is being calculated and JS is polling to find out if it has finished. Once the server returns the calculated business case, the requests will stop.


🐛 Can't create Message with MessageComponent inside of MessageComponent

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Try to create this object
Message(
    content="1234",
    components=[
        MessageComponent(
            type=1,
            components=[
                MessageComponent(
                    type=2,
                    label="click me!",
                    style=2,
                    custom_id="1234"
                )
            ]
        )
    ]
)

Theres 2 errors.

options should be optional. A few other APINullable properties are not correctly marked as missing.
This error is raised if the other problem is fixed.

This might be an easy fix now that the APIObject system isn't broken

I just had to change Options to MISSING so I just pushed to upstream. Its a pretty safe change so nothing can break.


Sort on many_many relation table doesn't work

Affected Version
5.0.0
Description
I have a BlogPost DataObject that has many_many relation with TaxonomyTerm DataObject through BlogPostTaxonomyTerm as below:
private static array $many_many = [
    'Terms' => [
        'through' => BlogPostTaxonomyTerm::class,
        'from' => 'Parent',
        'to' => 'TaxonomyTerm',
    ],
];

There is a Sort field on the TaxonomyTerm DataObject and a separate Sort field on the many_many through relation DataObject BlogPostTaxonomyTerm.
I want to sort the linked TaxonomyTerms on the BlogPost based on the Sort value on the BlogPostTaxonomyTerm table. However, I am not able to do that. The Sorting result is always determined by the Sort value on the TaxonomyTerm DataObject instead.
Here is an example, I have the following fixture file for unit test:
SilverStripe\Taxonomy\TaxonomyTerm:
  term1:
    Name: term1
    Sort: 1
  term2:
    Name: term2
    Sort: 2
  term3:
    Name: term3
    Sort: 3

App\Pages\BlogPost:
  page1:
    Title: 'Blog Post 1'
    PublishDate: '2023-06-20 15:55:50'

App\Taxonomy\ThroughRelations\BlogPostTaxonomyTerm:
  blogPostTerm1:
    Parent: =>App\Pages\BlogPost.page1
    TaxonomyTerm: =>SilverStripe\Taxonomy\TaxonomyTerm.term3
    Sort: 1
  blogPostTerm2:
    Parent: =>App\Pages\BlogPost.page1
    TaxonomyTerm: =>SilverStripe\Taxonomy\TaxonomyTerm.term2
    Sort: 2
  blogPostTerm3:
    Parent: =>App\Pages\BlogPost.page1
    TaxonomyTerm: =>SilverStripe\Taxonomy\TaxonomyTerm.term1
    Sort: 3

And I want to test the first term of the App\Pages\BlogPost.page1.
The expected result is the first term should be "term3", however the actual first term is "term1".
After I add the following config to the BlogPostTaxonomyTerm many_many relation, the Sorting works as expected.
 private static string $default_sort = '"BlogPostTaxonomyTerm"."Sort"';

Note that the problem here seems to be that private static string $default_sort = 'Sort'; on the join class does not sort by the Sort field on the join class, but rather by the Sort field on the taxonomy term. To sort by the field on the join class, you need to add the table name as part of the sort config.
It will be worth checking whether adding the table name is always required, or if it's only required when there is a conflict of field names.
Most likely this is caused by the way the query is declared, where you're more-or-less doing SELECT "Sort" FROM "TaxonomyTerm" LEFT JOIN "BlogPostTaxonomyTerm" ORDER BY "Sort" - i.e. the Sort field with no alias is from the taxonomy term class, so that's what it sorts by - the ManyManyThroughList isn't taking the join's alias into account when using it to sort by.
The problem then becomes - how can we determine what the developer wants to do - they might want to sort by the Sort field on the taxonomy term itself. So... maybe this is already behaving as expected and just needs to be much more clearly communicated through documentation and PHPDoc comments.

In fact I think I've just convinced myself that _assuming private static string $default_sort = 'Sort'; on the join record does sort by the sort field on taxonomy term, this is purely a documentation problem. I'll move it to the docs repo.


Can not apply different CSS classes for custom-markers

I am trying to apply different CSS classes for different custom markers, but looks like that custom markers only understand "custom-marker" class name and you can not assign a class name with different name for a custom marker.

I am pretty sure that you can add your own class according to the code
CustomMarker.prototype.addClass = function(className) {
  var classNames = this.el.className.split(' ');
  (classNames.indexOf(className) == -1) && classNames.push(className);
  this.el.className = classNames.join(' ');
};

What should I do to pass my own class?

http://plnkr.co/edit/6cMEyk?p=preview
You need to add your own class inside of container. This can be improved though.

This is fixed with 1.13.6 release, Example: https://rawgit.com/allenhwkim/angularjs-google-maps/master/testapp/custom-marker-custom-style.html

I see. Thanks much for the examples.


E197: Cannot set language to "en_US.utf8"

I don't enough about this locale business, but, basically this doesn't work:
:language messages en_US.utf8
E197: Cannot set language to "en_US.utf8"

But this works (note the capital "UTF-8"):
:language messages en_US.UTF-8

I guess its related to https://github.com/benekastah/neomake/issues/96 and/or https://github.com/benekastah/neomake/pull/98.
OS X 10.10.3

$ locale
LANG="en_US.UTF-8"
LC_COLLATE="en_US.UTF-8"
LC_CTYPE="en_US.UTF-8"
LC_MESSAGES="en_US.UTF-8"
LC_MONETARY="en_US.UTF-8"
LC_NUMERIC="en_US.UTF-8"
LC_TIME="en_US.UTF-8"
LC_ALL="en_US.UTF-8"

I don't have much knowledge about locales, but I think maybe some systems accept one as an alias while some don't. Just looked into vim runtime files and those folders are using capital UTF-8, so I think it may be the standard. Better just use the capital one in code I think.

@renstrom Please let me know if this fixes the issue for you.

@benekastah Unfortunately no. I only now noticed that there was also missing a dash - between UTF and 8:
-:language messages en_US.UTF8
+:language messages en_US.UTF-8

Seems like OS X/iTerm2/zsh is rather pick about this.


RFP - Search Deflector

Checklist

[X] The package I am requesting does not already exist on https://chocolatey.org/packages;
[X] There is no open issue for this package;
[X] The issue title starts 'RFP - ';
[X] The download URL is public and not locked behind a paywall / login;

Package Details
Software project URL :
https://github.com/spikespaz/search-deflector
Direct download URL for the software / installer :
https://github.com/spikespaz/search-deflector/releases/download/1.1.5/SearchDeflector-Installer.exe
Software summary / short description:
A small program that forwards searches from Cortana to your preferred browser and search engine.

You are not permitted to transfer the file "SearchDeflector-Installer.exe" because its signature "c0caae0193d0f8ecd81f64ecb900082bf5be1f1a" has been identified by the Virus Outbreak Prevention service.
URL: https://github-production-release-asset-2e65be.s3.amazonaws.com/138102350/11349d00-e38f-11e9-8954-0c5089c0a629?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIAIWNJYAX4CSVEH53A/20191120/us-east-1/s3/aws4_request&X-Amz-Date=20191120T160235Z&X-Amz-Expires=300&X-Amz-Signature=03aafd587d1d7cfd7649c70f3a49890f1960e54bc10988a786632563fe8fcb3e&X-Amz-SignedHeaders=host&actor_id=10491809&response-content-disposition=attachment%3B filename%3DSearchDeflector-Installer.exe&response-content-type=application/octet-stream
File quarantined as: [disabled].
Hummm... Strange...

I would like to be the maintainer.

@boubou please hold off on creating a package for this, it got quite alot of hits on different anti-virus scanners on virustotal.
It is rare that a reliable software gets the amount of positive hits on virustotal as this software gets (17 positive matches).
https://www.virustotal.com/gui/file/7d0e210917a8005815868a9a34e15157a60bfd3560e8fca04832811da0aabeb3/detection
I have posted in an internal channel to ask the other moderators if this is something we would allow or not.

@AdmiringWorm do you have an update for this?

Yes, I do have an update.
I have been looking around to find reasons of accepting this RFP despite the high positive virus count number, and have found none.
Due to the high positive count, and the only reason explained is due to an unsigned binary (which would only explain a small number of the hits, not that many).
Due to this, I will be closing this request.
Feel free to ask us to re-open the issue in the future should anything change regarding the positive virustotal hits.


OIPA-2143

https://app.zenhub.com/workspaces/iaticloud-5b8e865bb2cc7b6c78ece322/issues/zimmerman-zimmerman/iati.cloud/2143

:tada: This PR is included in version 5.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Parser ignores AlternativSet

When parsing a rule with a Literal followed by an AlternativeSet and again a Literal, the AlternativeSet is not parsed as such. If there is no Literal either on the rigtht or on the left side of the AlternativeSet everything works fine.
Example:
Rule: public <greet> = i (go | run) to school;
Parser Output: Rule(name='greet', visible=True, expansion=Sequence(Literal('i'), Literal('go'), Literal('run'), Literal('to school')))

I did some debugging and I think I found the mistake.
The function flatten_chain is called if there is a Sequence as a child of another Sequence. So if there is something like Sequence(Literal('i'), Sequence(AlternativeSet(Literal('go'), Literal('run')), Literal('to school'))) it will be called.
But the function does not only remove children that are sequences but also AlternativeSets. So the result is Sequence(Literal('i'), Literal('go'), Literal('run'), Literal('to school') instead of Sequence(Literal('i'), AlternativeSet(Literal('go'), Literal('run')), Literal('to school')).

Worked out a fix for this issue in pull request #10.

Nice catch! :+1:
Thanks for the issue and fix. I'll merge your PR and add a test case into test_parser.py. I'm happy to release this along with the other changes I made a few days ago as version 1.4.1.


Update device-plugins.md

add link to Xilinx FPGA device plugin

Please sign the CLA

Our company(Xilinx) has e-signed the CLA.
Will the CLA check be passed automatically or anything else from my side required?

/assign

@xuhz Thank you for your contribution. Please sign the CLA so we can accept your PR for review.

/check-cla

@xuhz Thanks for your PR. Please see: https://github.com/kubernetes/community/blob/master/CLA.md#how-do-i-sign for instructions in case your company has already signed a CLA.

I assigned it (actually Xilinx signed it) already
Username | First Name | Last Name | E-mail | Roles | Member since |
-- | -- | -- | -- | -- | -- | --
songc-xil | Chuck | Song |<EMAIL_ADDRESS>| organization manager | 2 weeks 5 days ago | edit remove
xuhz | Brian | Xu |<EMAIL_ADDRESS>| organization manager | 2 weeks 5 days ago | edit remove

I assigned it (actually Xilinx signed it) already
Username	First Name	Last Name	E-mail	Roles	Member since
songc-xil	Chuck	Song<EMAIL_ADDRESS>organization manager	2 weeks 5 days ago	edit remove
xuhz	Brian	Xu<EMAIL_ADDRESS>organization manager	2 weeks 5 days ago	edit remove

Can you please check if you received an email https://github.com/kubernetes/community/blob/master/CLA.md#5-look-for-an-email-indicating-successful-signup if not is suggest you check: https://github.com/kubernetes/community/blob/master/CLA.md#troubleshooting

I think I know what the issue is.
I tried to sign the CLA twice -- In the first one, I assigned myself as the signer on behalf of our company, then I figured out I couldn't that. Then in the second one, our VP e-signed on behalf of the company and I was assigned as one of the manager to manage the group.
I can't delete the first one -- which is still pending and should be deleted. That's why you think I still need to e-sign the CLA.
How to delete the first one?
First Name:
Brian
Last Name:
Xu
Full Name:
Brian Xu
Groups:
CNCF - Xilinx Inc.
CNCF - Xilinx Inc
History
Site user for
3 weeks 21 min

I think I know what the issue is.
I tried to sign the CLA twice -- In the first one, I assigned myself as the signer on behalf of our company, then I figured out I couldn't that. Then in the second one, our VP e-signed on behalf of the company and I was assigned as one of the manager to manage the group.
I can't delete the first one -- which is still pending and should be deleted. That's why you think I still need to e-sign the CLA.
How to delete the first one?
First Name:
Brian
Last Name:
Xu
Full Name:
Brian Xu
Groups:
CNCF - Xilinx Inc.
CNCF - Xilinx Inc
History
Site user for
3 weeks 21 min

Thank you @xuhz !!!
Could you please log an issue here: https://sso.linuxfoundation.org/login?state=g6Fo2SBpeFNJYUlLcHp0X3NMOURkWG5WSWROWGlCRDFrd2Z4aaN0aWTZIDBaVk1IRVpZR2RiVjdlakEyY2J3X1F3Q2xINkNFM25to2NpZNkgNE1iTWxDcFhvb0RvTlEwMlRvUXY5eXB3Tk96aWY1MTY&client=4MbMlCpXooDoNQ02ToQv9ypwNOzif516&protocol=samlp
If you encounter any problems signing the CLA, log a ticket with the Linux Foundation ID group through the Linux Foundation Support Site.
Should you have any issues using the LF Support Site, send a message to the backup e-mail support address<EMAIL_ADDRESS>Someone from the CNCF will respond to your ticket to help.

The e-sign takes effect now.
Please recheck it,
thanks!

/lgtm

Thanks for your PR @xuhz!!!!
/approve


Cleanup function context creation and disposal

Clean up context usage in InvocationHandler
Relevant comments:
https://github.com/Azure/azure-functions-dotnet-worker/pull/972#discussion_r958910198
https://github.com/Azure/azure-functions-dotnet-worker/pull/972#discussion_r958913155

@liliankasem to add more details to the issue.


backup2go widget like timemachine on the Dashboard

I'd like to see an enhancement:
Adding a backup2go widget on the dashboard like the 'timemachine' one.
Thanks in advance.

Fixed in the wip branch


Security issue: allowFetch is insecure, using it with untrusted code is a security vulnerability

This library exposes the ability for the guest (untrusted) code to fetch with the same cookies as the host fetch function. You must not run untrusted code if enabling fetch. Library should come with a big blinking warning about what is safe and unsafe to enable when running untrusted code. It’s not a “sandbox” if the sandboxed code can call arbitrary HTTP APIs authenticated as the host context!
I recommend designing the allowFetch API to require an allow-list of URL prefixes, disable following redirects to URLs not in the allow list, and force a default rate-limit on the guest.
Exposing fetch to untrusted code would be one of these vulnerabilities:

server-side: https://owasp.org/Top10/A10_2021-Server-Side_Request_Forgery_(SSRF)/
client-side CSRF: https://cheatsheetseries.owasp.org/cheatsheets/Cross-Site_Request_Forgery_Prevention_Cheat_Sheet.html#dealing-with-client-side-csrf-attacks-important

Yes, that is valid and the reason why it is disabled by default.
Currently, fetch is support is on a "technically" possible level.
My plan is, to abstract the fetch lib a bit more in the direction of module/plugin.
The allow-list is a good idea for a lot of use cases, but won't fit for any cases, where you do not know the URLs upfront.
The abstract idea is, to have the option, to provide the fetch functionality similar to console, and to only provide some basic adapter as default. The default one will stay basic and unrestricted to cover each use case.
This way, it can be adapted and aligned for specific needs, while keeping this lib focused on the core functionallity.

created as small proposal #3

fetch has been refactored and a bunch of options and security settings are available now. The functionality itself is a configurable adapter, which can be replaced by custom adapter
https://sebastianwessel.github.io/quickjs/fetch.html


Implement Default trait for Version

Slippi replay base version is 0.1.0
A user might want to use Version to encode some sort of compatibility information in a type (eg. certain stats only work for certain replay versions). It's better to use Default::default than Version(0,1,0) because

You don't have to remember which version is the earliest
It more clearly communicates the intent to someone reading the code. For example, min_version: Default::default() more clearly communicates that all versions should work.
Allows user types that include a Version to also derive default if it makes sense

Could you describe why we want this in the PR description?

You don't have to remember which version is the earliest

That assumes the default is the earliest version of Slippi, which doesn't make sense to me. If there even is a context where it makes sense to have a "default Slippi version", IMO the latest supported version of Slippi would be a more sensible and intuitive choice than the earliest version. Besides, I don't think remembering "0.1.0" is particularly hard, and we could even provide a const like MIN_VERSION if there's demand.

It more clearly communicates the intent to someone reading the code. For example, min_version: Default::default() more clearly communicates that all versions should work.

Very much disagree. In general, I would not assume a "default version" of something automatically means the earliest supported version. To me, min_version: Version(0, 1, 0) is clearer and less ambiguous.

Allows user types that include a Version to also derive default if it makes sense

When would that make sense, though? I can only think of cases where you'd want the latest version.

Closing for lack of activity. Feel free to reopen for further discussion.


Incomming view does not correctly show diffs.

Currently diffs are working, but
When we show commits, which actions should be available?

For a newer commit in my branch
for an incomming commit from upstream
for an older commit, just load an old version? reset head to that commit?

When we load a commit, we should update the reference/updation commits.


Using different methods to implement non-embedded child windows will trigger the always_on_top breaks non-embedded popups problem

Tested versions
4.3 Until now
System information
Windows 10 x64
Issue description
We can get non-embedded child windows in the following two ways:

disabling embed_subwindows.
Change force_native of a subwindow

When using the second method, it seems that the parent window of the subwindow is not handled correctly. This causes all pop-ups on the subwindow to use the main window as its parent window(This is a guess). This further causes the position of the pop-ups of the subwindow to be wrong, and after always_on_top is enabled on the subwindow, it will cause the window sorting to be wrong and cover all the pop-ups of the subwindow.
Steps to reproduce

Create default project
Add Window node, enable always_on_top and force_native, disable visible
Add any node with popup menu (e.g. TextEdit) to window node
Run
Make the window node visible
Open the popup menu on window node

The PopopMenu is not only covered but also displayed in the wrong position.
Minimal reproduction project (MRP)
N/A

Please provide an MRP to help us understand the issue more clearly; thanks.


Devel

As discussed on #1606 , i did try to implement the addRoute passing extra argument for parentRoute name. I was able to create the subRoute now
I'm not sure if the code works as your expectation and guidelines as your 3rd item from https://github.com/TelescopeJS/Telescope/blob/devel/CONTRIBUTING.md says it should be on their own repo, so please let me know what are your thoughts

I didn't have time to test the feature, but the code looks ok to me so I went ahead and merged it. Thanks for the PR :)


👋 Hey there spelunker. It looks like you've modified some files that we can't accept as contributions:

          👋 Hey there spelunker. It looks like you've modified some files that we can't accept as contributions:

.devcontainer/devcontainer.json
.github/workflows/node.js.yml
.github/workflows/notify-about-deployment.yml

You'll need to revert all of the files you changed that match that list using GitHub Desktop or git checkout origin/main <file name>. Once you get those files reverted, we can continue with the review process. :octocat:
The complete list of files we can't accept are:

.devcontainer/**
.github/**
data/reusables/rai/**
Dockerfile*
src/**
package*.json
content/actions/deployment/security-hardening-your-deployments/**

We also can't accept contributions to files in the content directory with frontmatter type: rai.
Originally posted by @github-actions[bot] in https://github.com/github/docs/issues/35580#issuecomment-2525968532

gh pr checkout 35580


How to display the visergui results of remote machines on Mac？

──────────────── viser ────────────────╮
│             ╷                         │
│   HTTP      │ http://<IP_ADDRESS>:6789   │
│   Websocket │ ws://<IP_ADDRESS>:6789     │
│             ╵                         │
╰───────────────────────────────────────╯
This is viser result log, but i want to display on mac

the host setting seems to be configured with viser
https://viser.studio/server/

A little bit late, but for this kind of thing I usually just use SSH port forwarding:
ssh user@server -L 6789:<IP_ADDRESS>:6789

<IP_ADDRESS>:6789 on the local machine will then point to to <IP_ADDRESS>:6789 on the remote machine.

A little bit late, but for this kind of thing I usually just use SSH port forwarding:
ssh user@server -L 6789:<IP_ADDRESS>:6789

<IP_ADDRESS>:6789 on the local machine will then point to <IP_ADDRESS>:6789 on the remote machine.

Just tried it but still nothing is being displayed on the local mac at <IP_ADDRESS>:6789. When tried with localhost:6789 I got connection refused error.

A little bit late, but for this kind of thing I usually just use SSH port forwarding:
ssh user@server -L 6789:<IP_ADDRESS>:6789

<IP_ADDRESS>:6789 on the local machine will then point to <IP_ADDRESS>:6789 on the remote machine.

Just tried it but still nothing is being displayed on the local mac at <IP_ADDRESS>:6789. When tried with localhost:6789 I got connection refused error.


Implementing string arrays in resource file

I believe I have a working solution, but am unable to set the gps on the emulator to create an account to login and test it. Can someone tell me a account that's already set up?

If you look at the esclient page there is a url to a website where you should be able to see everything that's saved.

On Apr 5, 2015, at 6:27 PM, IchigoKIl<EMAIL_ADDRESS>wrote:
I believe I have a working solution, but am unable to set the gps on the emulator to create an account and test it. Can someone tell me a account that's already set up?
—
Reply to this email directly or view it on GitHub.

I have just tested it and it is working correctly. Is AddItemActivity the only activity that uses the category and currencies spinner?

Yes

On Apr 6, 2015, at 5:37 PM, IchigoKIl<EMAIL_ADDRESS>wrote:
I have just tested it and it is working correctly. Is AddItemActivity the only activity that uses the category and currencies spinner?
—
Reply to this email directly or view it on GitHub.


What are the minimum requirements for pinokio?

I tried it on my win 10 laptop and failed at the first hurdle
conda download aborted
Error: aborted at connResetException (node:internal/errors:) at TLSSocket.socketCloseListener (node:_http_client:) at TLSSocket.emit (node:events:) at node:net: at TCP.done (node:_tls_wrap:)
downloaded the file it points to, manually and installed miniconda. Next run does not detect the install and fails again...
Point is, before I spend anymore time reading up on issues and so on, I looked for min requirements, and basically cannot find a clear answer.
Can I request the link I'm failing to find please?
edit: sorry I was installing e2-f5-tts. Not pinokio itself. Question still stands I suppose.
edit: results are swamped with youtubers cashing in on videos about installing it or showing it off. Non of which I tried were helpful.

I get the same error too, no matter what I try to install:
Error: aborted at connResetException (node:internal/errors:711:14) at TLSSocket.socketCloseListener (node:_http_client:454:19) at TLSSocket.emit (node:events:525:35) at node:net:313:12 at TCP.done (node:_tls_wrap:587:7)


go vet errors

Basically,
$ go vet ./...
app_test.go:46: cli.StringFlag composite literal uses unkeyed fields
cli_test.go:50: cli.StringFlag composite literal uses unkeyed fields
cli_test.go:60: cli.StringFlag composite literal uses unkeyed fields
cli_test.go:70: cli.StringFlag composite literal uses unkeyed fields
exit status 1

I am still getting this error, am I missing something?
/vagrant/src/code.softlayer.local/softlayer/nidp on master* make
==> Checking sanity
go fix ./...
go vet ./...
main.go:13: github.com/codegangsta/cli.StringFlag composite literal uses unkeyed fields
exit status 1
make: *** [sane] Error 1
{code}
app := cli.NewApp()
app.Name = "Data Processor"
app.Usage = "Insert data from a queue data stream into Cassandra"
app.Flags = []cli.Flag {
cli.StringFlag{"server, s", "localhost", "RabbitMQ server hostname"},
}
app.Action = func(c *cli.Context) {
    runApp(c)
}
app.Run(os.Args)

{/code}

@binodluitel that indicates that your code is not passing go vet, not codegangsta/cli. In this case, the error message is accurate that you aren't keying your struct fields.

Gotcha, thanks
Incase anyone will anyone needs it, this is what @jszwedko meant
    app.Flags = []cli.Flag {
        cli.StringFlag{
            Name: "server, s",
            Value: "localhost",
            Usage: "RabbitMQ server hostname",
        },
    }


Add the ability to pass token/password to hub-tool using CLI switches or environment vars

Description
Add the ability to pass token/password to hub-tool using CLI switches or environment vars. Other CLI's like docker do this for comparison.
$ hub-tool/hub-tool login <user> --password-stdin <<<$(echo "password")

Do something similar with LastPass's CLI
$ docker login -u $(lpass show --username 'dockeruser') --password-stdin <<<$(lpass show --password 'dockerpass')

References

https://docs.docker.com/engine/reference/commandline/login/

I tried this (very hacky) workaround (I'm using ubuntu-latest on my GitHub Actions runner):
          /usr/bin/expect <(cat << EOF
          spawn /tmp/hub-tool login ${{ secrets.DOCKERHUB_USERNAME }}
          expect "\033\[0;34mPassword: \033\[0m"
          send "${{ secrets.DOCKERHUB_TOKEN }}\n"
          interact
          EOF
          )

But it's failing, most likely due to "Password: " being printed with colors. I tested locally and this WORKS on my machine even without the ANSI colors codes, but not on GitHub Actions' runner. :-(

What I ended up doing, for now (but sucessfully): built the ~/.docker/config.json by hand. For the JWT token, I get it with a cURL request directly to the API. 🤷🏻♂

I have a patch for this in https://github.com/docker/hub-tool/pull/198 which enables providing the Docker Hub password using the DOCKER_PASSWORD environment variable.


Tests for Partner.createPartner

Overview
Our partner service is currently untested. This is not great. Ideally we would have at least one pass and fail scenario for each endpoint.
Action Items

[ ] You might need to create a new test file if it has not already been created
[ ] Create a test scenario where partnerService.createPartner passes
[ ] Create a test scenario where partnerService.createPartner fails

Resources
See other service tests such as bloom-backend/src/partner-feature/partner-feature.service.spec.ts to see how tests can be structured.

Absolutely, thank you for your interest. Let me know if you have any questions!


adapt 3 swift

👍

I am no mantainer, but I'd definitely merge this =D

Hello, I am not familiar with the Zewo project but I'm currently using ZeroMQ in swift with goloveychuk's fork.
I am no expert but I feel some changes should not be made, mainly to avoid introducing external dependencies. In my case I am using ZeroMQ for connecting a desktop and mobile app, so I wouldn't use other Zewo services and would prefer if I didn't have to include them at all.
I don't know what the Axis API does, but if you suggest  that it should be used, I think that the Axis repo would be a better place to implement this with an extension, or in ZeroMQ with the canImport conditional flag, so there is always a way to use ZeroMQ without depending on Axis

Yea, I agree with @BoteRock . If we can easily  avoid dependence, why we shouldn't

oops, I don't know why it's merged :)

@BoteRock @goloveychuk What's the issue with using Axis? It's the common code used by all zewo modules. It's not really external dependency, it doesn't bring in anything opinionated (ie libmill) - just a bunch of protocols and utilities. Without it, it's difficult to use ZeroMQ with, for example, HTTPServer (you can't do things like stream the 0mq socket to the client without a bunch of glue if you don't conform to the Stream protocol). The only thing we need out of there is buffer and stream.
That said, I'm not entirely against having ZeroMQ be an outlier. What are your reasons for not wanting to include Axis?
@goloveychuk that's alright, this pr is pretty much ready anyway. Can you update the readme as well please?

@Danappelxx for this 0mq implementation Axis is not really needed. This is just a simple wrapper.

Axis is not foundation - its not a toolkit. It's just the glue for compatibility with the rest of the ecosystem. The fact that JSON and such are included in Axis is something I disagree heavily with and will be changed by the next release (0.15). Perhaps once that happens we can add it as a dependency here.

@Danappelxx I understand what you want, but zeromq is not simple socket.
You cannot just use it as transport. It have many types of sockets, support many patterns.
There are many limitations. E.g. you will error if will try to read from req/rep socket, while you should write to it.
If somebody want it combine with axis - it will require writing a wrapper.

you, guys, forgot about this one https://github.com/Zewo/Canale
this is specifically developed for Zewo & friends.


[question]: Can HTML previews fetch/XHR content from virtual document URIs?

I was very delighted to find out that HTML previews could run JavaScript, which is the whole reason my vscode-map-preview extension (https://github.com/jumpinjackie/vscode-map-preview) exists.
One of the questions I have, is that the HTML preview I generate with the extension is not so much content, but a combination of an interactive viewer "container" into the actual content that is also embedded into the same preview HTML.
One of the small annoyances is that I have to sanitize the document content for any unruly characters before embedding into the preview HTML so that the content doesn't break out of the ES6 template string that I have defined in the preview HTML script block for this content.
What would be nicer and cleaner for me is if instead of embedding the document content into the preview HTML, is if I could define a 2 virtual documents (with their own URI schemes):

For the viewer "container" HTML
For the actual document content

Then HTML preview I generate from the 1st virtual document can fetch/XHR the content from, allowing for the graceful ability to handle failures in setting up the preview. Currently any failures in previewing content are mostly silent, and I would have to physically dump the preview HTML content out to a separate file in my extension code and open that up in Chrome to see what's wrong with it.
Is this possible to make fetch/XHR inside a HTML preview work with virtual documents in vscode?

No, that's currently not possible and I wouldn't know how to do that... The implementation uses loads the document in an iframe that runs out of process.
What you can do (and other do that) is starting a server on local host which serves the actual document content.


[ready] perf: constant in while in for in busy func

This is a surprisingly big one. Constant definition in while loop in for loop in busy function.
Also uses is instead of == for singletons.
Before

Total time: 1.51153 s
File: /home/rvd/src/roelofvandijk/tinygrad/tinygrad/lazy.py
Function: _push_movement_ops at line 306

Line #      Hits         Time  Per Hit   % Time  Line Contents
==============================================================
   306                                           @profile
   307                                           def _push_movement_ops(srcs:Tuple[LazyBuffer, ...]) -> Tuple[LazyBuffer, ...]:
   308     80483      22235.8      0.3      1.5    new_srcs = []
   309    144813      40322.9      0.3      2.7    for x in srcs:
   310    144813      29607.2      0.2      2.0      mops: List[Tuple[MovementOps, Any]] = []
   311    144813      22430.4      0.2      1.5      bx = x
   312                                               # backwalk all the movement ops. don't push PAD or EXPAND
   313    144813      94432.1      0.7      6.2      while not bx.realized and bx.optype == MovementOps and bx.op.op != MovementOps.EXPAND and (SHUFFLE_PAD_OPS or bx.op.op != MovementOps.PAD) and len(bx.children) <= 1:
   314     71302      39646.5      0.6      2.6        assert isinstance(bx.op.op, MovementOps)
   315     71302      47581.9      0.7      3.1        mops.append((bx.op.op, bx.op.arg))
   316     71302      59826.3      0.8      4.0        bx = cast(LazyBuffer, bx.op.src[0])
   317                                               # NOTE: can't push pads past anything where f(0, 0) != 0 or f(0) != 0
   318    144813     545717.9      3.8     36.1      unsafe_pad_ops = {BinaryOps.DIV, BinaryOps.CMPLT, UnaryOps.LOG2, UnaryOps.EXP2, UnaryOps.RECIP}
   319    140049      33643.6      0.2      2.2      if mops and not bx.realized and bx.optype == BinaryOps and len(bx.children) <= 1 and (all(x[0] != MovementOps.PAD for x in mops) or all(x.op not in unsafe_pad_ops for x in bx.op.get_lazyops())):
   320      4764     477063.6    100.1     31.6        new_srcs.append(bx.op.replace_with_movement_ops(mops[::-1]))
   321                                               else:
   322    140049      69685.7      0.5      4.6        new_srcs.append(x)
   323     80483      29334.0      0.4      1.9    return tuple(new_srcs)

After

Total time: 0.969821 s
File: /home/rvd/src/roelofvandijk/tinygrad/tinygrad/lazy.py
Function: _push_movement_ops at line 308

Line #      Hits         Time  Per Hit   % Time  Line Contents
==============================================================
   308                                           @profile
   309                                           def _push_movement_ops(srcs:Tuple[LazyBuffer, ...]) -> Tuple[LazyBuffer, ...]:
   310     80483      23872.4      0.3      2.5    new_srcs = []
   311    144813      38574.1      0.3      4.0    for x in srcs:
   312    144813      27994.1      0.2      2.9      mops: List[Tuple[MovementOps, Any]] = []
   313    144813      20276.5      0.1      2.1      bx = x
   314                                               # backwalk all the movement ops. don't push PAD or EXPAND
   315    144813      89660.5      0.6      9.2      while not bx.realized and bx.optype is MovementOps and bx.op.op is not MovementOps.EXPAND and (SHUFFLE_PAD_OPS or bx.op.op is not MovementOps.PAD) and len(bx.children) <= 1:
   316     71302      38714.8      0.5      4.0        assert isinstance(bx.op.op, MovementOps)
   317     71302      46368.5      0.7      4.8        mops.append((bx.op.op, bx.op.arg))
   318     71302      58354.1      0.8      6.0        bx = cast(LazyBuffer, bx.op.src[0])
   319                                               # NOTE: can't push pads past anything where f(0, 0) != 0 or f(0) != 0
   320    140049      30260.9      0.2      3.1      if mops and not bx.realized and bx.optype is BinaryOps and len(bx.children) <= 1 and (all(x[0] is not MovementOps.PAD for x in mops) or all(x.op not in UNSAFE_PAD_OPS for x in bx.op.get_lazyops())):
   321      4764     498316.2    104.6     51.4        new_srcs.append(bx.op.replace_with_movement_ops(mops[::-1]))
   322                                               else:
   323    140049      70097.0      0.5      7.2        new_srcs.append(x)
   324     80483      27332.3      0.3      2.8    return tuple(new_srcs)

Changes made in tinygrad/:
------------------------------------------------------------
files                             insertions       deletions
------------------------------------------------------------
tinygrad/lazy.py                           4               3
------------------------------------------------------------
lines added in the tinygrad folder: 1


Fix/vaccination statistics (#1432)

Fix for keyFigureCard -> return all the cards

delete unnecesary space

Modify test to have the correct number of KeyCards

Checklist

[x] Test your changes as thoroughly as possible before you commit them. Preferably, automate your test by unit/integration tests.
[x] Make sure mvn -P integration-test clean verify runs for the whole project and, if you touched any code for a service in the services folder, ensure it can be run with spring-boot:run

Description
Cherry-picked the commits for Vaccination Statistics Cards (FIX) into release/2.4.

not required for 2.4, will come with 2.5


Support a number of SPIR-V capability related features.

Whitespace
CHANGELOG_VULKANO.md:4: trailing whitespace.
+- Added additional device feature flags for enabling SPIR-V related capabilities.
The command "git diff-tree --check $(git hash-object -t tree /dev/null) HEAD" exited with 2.

Fixed.


Update executeTakeFirstOrThrow to allow throwing custom error objects

Currently, executeTakeFirstOrThrow only accepts a custom error class. This would allow users to pass a fully formed error object to throw if they don't care about the node.

[x] implement
[x] document
[x] test

Hey 👋
Thanks! 💪
I'm not sure about this one..
Currently you can provide a custom error class that extends NoResultError.
You receive the query syntax tree at runtime, which enables more accurate error messages.
You can also define a hard-coded a hard coded error message.
I don't see any benefit to passing an instance of an error instead of the class itself. What am I missing?

I agree. Functionally, this offers very little benefit, though if the user doesn't care about the underlying error, it would be nice to reduce the amount of lines of code required.
Here's a real example that I'm currently facing:
const building = await req.db
    .selectFrom("building")
    .where("building_id", "=", building_id)
    .executeTakeFirst()

  if (!building) {
    throw new NotFoundException({
      type: "not_found",
      message: "Building not found",
    })
  }

After this PR:
const building = await req.db
    .selectFrom("building")
    .where("building_id", "=", building_id)
    .executeTakeFirstOrThrow(
      new NotFoundException({
        type: "not_found",
        message: "Building not found",
      })
    )

The project above avoids introducing new types where possible to reduce complexity. Creating a new error class would violate this.

If a user doesn't care about the error, she can simply call .executeTakeFirstOrThrow() without any arguments. This will throw the basic NoResultError.

If a user doesn't care about the error, she can simply call .executeTakeFirstOrThrow() without any arguments. This will throw the basic NoResultError.

The project above avoids introducing new types where possible to reduce complexity. Creating a new error class would violate this.

But it instantiates an error prematurely for every invocation.

But it instantiates an error prematurely for every invocation.

Fair. Will this be better?
const building = await req.db
    .selectFrom("building")
    .where("building_id", "=", building_id)
    .executeTakeFirstOrThrow(
    () =>  new NotFoundException({
        type: "not_found",
        message: "Building not found",
      })
    )

Yeah, that makes sense.

Thanks @mikewuu and @igalklebanov!
Would it make sense for the error creation function to also receive OperationNode as an argument? This way we could have a custom error class hierarchy and still get the same context as NoResultError and its extensions.
const building = await req.db
    .selectFrom("building")
    .where("building_id", "=", building_id)
    .executeTakeFirstOrThrow(
    // 👇 Added argument
    (node: OperationNode) =>  new NotFoundException({
        type: "not_found",
        message: "Building not found",
      })
    )

I was thinking the same thing literally minutes ago. Will push an update later today, thanks!

@igalklebanov , @bjornlll , just updated this.

No reason to be sorry @koskimas ! I'm just excited we're getting this in. It's one of those things that won't fundamentally change life but will make me feel a tad bit happier when looking at my code.
Thanks again @mikewuu for spearheading this!


What do those operations mean?

Dear,
Thank you for open-sourcing the codebase! I am enjoying working on this great codebase.
Earlier this week, I've encountered some part which I couldn't fully understand what they meant to do. So, if possible could you briefly correct my take on these parts?
Code

https://github.com/google-research/planet/blob/c04226b6db136f5269625378cd6a0aa875a92842/planet/training/utility.py#L230-L231

I think this is to calculate the log probs of the deterministic state components other than images.

https://github.com/google-research/planet/blob/c04226b6db136f5269625378cd6a0aa875a92842/planet/training/utility.py#L257

this aggregates the defined objectives above, including KL div for image observation model, right?

https://github.com/google-research/planet/blob/c04226b6db136f5269625378cd6a0aa875a92842/planet/training/utility.py#L261-L275

if my run went correctly, at this point objectives only contains main objective which subsequently is minised by the optimiser.

The reason why I'd like to understand this part is to know better about like " Does collecting the deterministic state components, e.g., velocity/position, really makes the model robust? "
Edit:
Sorry, I've pondered the content of this issue a lot and totally forgot to modify the title....

Sorry, I've pondered the content of this issue a lot and totally forgot to modify the title....

Apologies for the late reply.
Predicting the true state components is decoupled from the rest of the model and purely used for visualization. This is why the diagnostic predictors receive a stop-gradient version of the latent states and are optimized using different optimizers.
The actual agent is trained only from images, actions, and rewards.

Dear @danijar
Thank you for your update!!
I got it, it makes sense!!
Best,


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 445 ms

Resolved: ClamAV performance has improved in 95e6a1a.


Eval.Util: Define FlagSpec type synonym

Apparently this wasn't exported pre-7.10.
Properly fixes breakage reported in #380 due to #375.

Still waiting for the test builds to finish.

This should not work; TurnOnFlag and DynP are not imported. (I tried the same thing when trying to fix this and ended up getting rid of the imports instead of tracking down all the types, sorry)

@gibiansky right, indeed the build failed. I have the type tracked down but just omitting them may be the least bad solution.

@gibiansky indeed, even the types that I tracked down aren't sufficient. Given that this is just to handle the older GHC releases, I say we just remove the type signatures for now.

Sounds good to me. Leaving things as they are now.


adding searchbar.jsx

@meltedspork @rgoodie 1st pass for searchbar jsx. Review when you're able.

Do we still need this? Or is this already taken care of in different branch?


Gnosis Documentation Out of Date

Describe the bug
Safe Transaction Service documentation is out of date
To Reproduce
The following is broken:

Running the provided example code for proposing a transaction returns a 422 due to incorrect contract hash. I assume EIP-712 validation has changed.
The link to how EIP-712 structuring happens on the safe side is broken
Not as urgent for my particular needs but worth noting this applies to the SDK documentation too. The import structure has a breaking change that is not reflected in the documentation. Which says

import { EthersAdapter } from<EMAIL_ADDRESS>
But that structure is no longer correct
Expected behavior
Documentation should provide working examples and up-to-date usage guidelines. A documentation for proposing a batch of multiple tx's to the safe contract would be highly appreciated as well

Hi @Anteloper, regarding the SDK documentation are you refering to https://docs.gnosis-safe.io/ or anywhere in the repo? In case you are referring to the docs in the website, it's a known issue that will be soon updated. The repos should be updated.
Thank you!

I'm referring to the docs on the website, but I submitted this issue here because I believe there has been a behavior change either in this repo or in the safe repo. I'm not sure if that change is intended or not, but I can be reasonably sure it happened because the example implementation (which the docs link to) doesn't work
There is currently no accurate documentation for how to submit proposed transactions to gnosis anywhere on the internet as far as I can tell. Is it possible to prioritize the documentation update?

@Anteloper you may want to check this one: https://github.com/gnosis/safe-core-sdk/blob/main/packages/guides/integrating-the-safe-core-sdk.md
It shows how to use the Safe Core SDK to create and sign a Safe transaction and propose it to the Safe Transaction Service using the Safe Service Client.

Excellent will try that thank you

Thank you this is great. One other question - can you point me to documentation on proposing a transaction for a token other than ETH on a safe?


CPU environment . How to install your package and test?

@yangxue0827 How can I compile and run the repo on CPU only machine ? Thanks

@yangxue0827 How can I compile and run the repo on CPU only machine ? Thanks
In cfg.py make change as follow:
ROTATE_NMS_USE_GPU = False
and  run ./tools/inference.py
But  I am getting error
File "./libs/detection_oprations/anchor_target_layer_without_boxweight.py", line 16, in
from libs.box_utils.cython_utils.cython_bbox import bbox_overlaps
ModuleNotFoundError: No module named 'libs.box_utils.cython_utils.cython_bbox' and when I checked there is no folder cython_bbox.


Only release lock when lock.isHeld() == true

Prevent java.lang.RuntimeException: WakeLock under-locked exceptions.

We need to determine why the WakeLock is underlocked in the first place. If you have a reproducible test case, or even a general instinct, for scenarios when the WakeLock is underlocked, I'd be interested in hearing them. I thought they were all solved in the current release, but there may be a scenario that I am missing.

Perhaps is has something todo with the amount of time we wait.. In your example in the PDF you are using 1 minute delay for the first run and then run every 15 minutes.
I'm running both in 6 seconds.. So that could be an issue.
However.. with the pull request changes my app seems to be running fine.
(Android 2.2 on a Samsung GIO)
On Jan 19, 2012(w3), at 3:41 PM, Mark Murphy wrote:

We need to determine why the WakeLock is underlocked in the first place. If you have a reproducible test case, or even a general instinct, for scenarios when the WakeLock is underlocked, I'd be interested in hearing them. I thought they were all solved in the current release, but there may be a scenario that I am missing.

Reply to this email directly or view it on GitHub:
https://github.com/commonsguy/cwac-wakeful/pull/8#issuecomment-3564077

Firing off a WakefulIntentService every 6 seconds is insane. A minute is about the over/under for where using something like AlarmManager and WakefulIntentService makes sense. Less than that, you should be doing something else.

Ok. Thanks for the heads up.. I'd love to hear your opinion in what to use instead? It's a polling service to sync data from an api server..
On Jan 19, 2012(w3), at 5:01 PM, Mark Murphy wrote:

Firing off a WakefulIntentService every 6 seconds is insane. A minute is about the over/under for where using something like AlarmManager and WakefulIntentService makes sense. Less than that, you should be doing something else.

Reply to this email directly or view it on GitHub:
https://github.com/commonsguy/cwac-wakeful/pull/8#issuecomment-3568077

Well, if you are polling every 6 seconds, it damn well better only be happening when the application has an activity in the foreground. In which case, the whole WakeLock thing should become a non-issue, as the user is actively using the app. Have a regular Service use a TimerTask for the polling/background thread, where you keep the service running so long as you have an activity in the foreground, shutting down the service and TimerTask once it's clear the user has left and won't be coming back for a while.

Thanks!, Will be using the Wakeup on 1 minute now..
On Jan 19, 2012(w3), at 5:16 PM, Mark Murphy wrote:

Well, if you are polling every 6 seconds, it damn well better only be happening when the application has an activity in the foreground. In which case, the whole WakeLock thing should become a non-issue, as the user is actively using the app. Have a regular Service use a TimerTask for the polling/background thread, where you keep the service running so long as you have an activity in the foreground, shutting down the service and TimerTask once it's clear the user has left and won't be coming back for a while.

Reply to this email directly or view it on GitHub:
https://github.com/commonsguy/cwac-wakeful/pull/8#issuecomment-3569089


Only generate Javadocs for latest Spark runner version (Spark 3)

The process of aggregating Javadocs is a bit sketchy in cases where source sets are shared. Particularly that’s the case for Spark and Flink runners.
Considering that everything is thrown into one bucket, we could easily end up with an inconsistent and misleading mix of Javadocs from multiple versions due to version specific overrides.
This disables Javadocs for the Spark2 runner so that aggregateJavadoc doesn't fail.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

@y1chi sorry for breaking that... here's what I'd suggest to fix the problem.


install tot error : can not find README.md

Problem:
pip install -e .
shown error message "can not find README.md"
Solution:
rename file readme.md to README.md

thanks! done!

let me know if there's any other issues when installing!


title bar button effects

Is there a plan to add hover and pressed effect for buttons, then with little changes themes from xfce could be used in jwm

At one point we could specify xbm images to use for the title bar (removed?)
It looks like each title "button" has a significant amount of code to be "drawn" whereas it looks like using a specified image may take less code
... I know it wouldn't work if icons were disabled which brings me to:
Now it also looks like the xpm-create-image-from-data function and its associated helpers are only used in 1 place and it looks like you cannot have icons without --enable-xpm (though I haven't tried)
Perhaps generating a static const char[] array for these is an option (it appears to just be RGBA format) ... title bar images, the "X" icon, etc...
P.S. would it be possible to fallback to the "X" image or something if a specified image is not found/supported?   Useful for instance as a "placeholder" if a theme is created with svg images, but jwm is compiled without svg support... at least in trays where a missing icon can result in a tray button of 0 width/height if label="" is not set

It's still possible to specify xbm images (or any image format that JWM supports) for title bar icons (see http://joewing.net/projects/jwm/config.shtml#other).  There is a good amount of code to draw the non-image icons, but I prefer those since they don't require an external file (or data blob that JWM may be unable to interpret), they scale correctly, and they use the correct colors (which xbm images would as well, of course).
You should also be able to have images without enable-xpm, just not xpm images.  If not, that's a bug.
In any case, I think there are 2 additional icons that JWM would need to support for each button ideally.  The first for mouse hover and the second for mouse click.  The internal icons could just change color (or stay the same).  For external icons, I was thinking there could be an additional property when specifying icons.  For example:
<ButtonClose>the-default.png</ButtonClose>
<ButtonClose on="hover">icon-to-display-on-hover.xpm</ButtonClose>  <!-- optional -->
<ButtonClose on="click">icon-to-display-on-click.jpg</ButtonClose> <!-- optional -->

Somewhere along the way xbm support was broken (it worked in 2.2.2, but since 3.0 it just frees the data and does nothing with it)
something like this should work:

ImageNode *LoadXBMImage(const char *fileName)
{
   ImageNode *result = CreateImage(0,0,1);
   int dummy1, dummy2, rc;
   rc = XReadBitmapFileData(fileName,&result->width,&result->height,
                            &(result->data),&dummy1,&dummy2);
   return (rc == BitmapSuccess)?result:NULL;
}

also CreateImage() should have

//image->bitmap = 0;
image->bitmap = bitmap;

@technosaurus, I'm not sure what code you're looking at.  There is no 3.0 and xbms seem to work in master.  I think I might have had a horribly broken, experimental 3.0 branch at one point, but it's long gone.
LoadXBMImage is here: https://github.com/joewing/jwm/blob/master/src/image.c#L527

I meant 2.3.0... but yeah, master is broken too.
It broke @ https://github.com/joewing/jwm/commit/4c62a4796910d93702736ae1829d7a5547acdb80
image->data never gets filled
image->bitmap is set to 0 instead of 1
The result is an image of random data, but the above changes fix it.

Thanks for the fix!  I was loading an XPM instead of an XBM, so I thought it was working when it wasn't.  It seems to actually work now.

I got to thinking about this and it seems that the bitmap color on title bar buttons (close, maximize...) is set to black, regardless of the foreground color... Am I missing a setting?  It used to be the same as font color.

@joewing I try to use your example for on hover and on click button event but it seems to not work.  The last instruction is the only icon that appears. I use window style flat. Is this feature removed from last version?

I don't believe this was ever implemented.

@joewing oh sorry, I saw the thread closed and I supposed it was implemented. Do you have any plan for this enhancment?


fix: #11 parse metadata and valid optional

This pull request fixes parsing metadata and allows empty string on optional values.
All groups and proposals are currently not passing as valid metadata because we are stringifying the metadata before parsing, which produces another string rather than an object. The solution is to parse twice.
When using valid, we are not respecting the optional values. A group with a valid name but no description (i.e. empty string) was not passing validation. A group with a valid name but no forum link (i.e. empty string) was not passing validation.
Also, a group created outside the UI with metadata that does not include updatedAt but has a valid name was not passing validation. Making updatedAt optional allows the name to be displayed and "Last Edited" to be empty.

cc @haveanicedavid can you take a look at this? The current deployment is not showing group names and making validation less strict at the metadata parsing level will allow group names to be displayed when other metadata is missing.


Deal with Albertsons switching to different booking API

(Noticed this while looking into #1459.)
Sometime in the last 7 months (last time I dealt with Albertsons’s systems was in August), Albertsons moved all their COVID vaccine booking to their normal pharmacy scheduling system instead of the MHealth system they were originally using and that our system used. Store data is still in the MHealth, but it’s now stale.
The new booking site is at https://www.albertsons.com/vaccinations/home. Data is loaded by POSTing JSON to https://rxie.albertsons.com/abs/prod/rxie/appointment/service-availability. It searches based on zip code, so will more like our other scrapers to use, and much more intense to run. OTOH, the response data is formatted much better, and does not have all the nutty parsing issues we dealt with from MHealth.
We should probably disable the old MHealth source.
A new scraper needs to be written, though we may consider only running it for NJ and AK.

Example POST payload:
{
    "radius": 15,
    "dateOfBirth": "04/20/1980",
    "serviceTypes": [
        {
            "serviceTypeName": "COVID-19 Pfizer",
            "scientificName": ""
        }
    ],
    "zip": "60190"
}

Whole request as cURL:
curl 'https://rxie.albertsons.com/abs/prod/rxie/appointment/service-availability' -X POST -H 'User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:109.0) Gecko/20100101 Firefox/113.0' -H 'Accept: application/json, text/plain, */*' -H 'Accept-Language: en-US,en;q=0.5' -H 'Accept-Encoding: gzip, deflate, br' -H 'correlationId: 7e0e336a-3dc8-45fe-a6c2-280cab92f349' -H 'version: 2' -H 'ocp-apim-subscription-key: b7bda12ac31a47b0a2bb3cad633d60c4' -H 'Content-Type: application/json' -H 'Origin: https://www.albertsons.com' -H 'DNT: 1' -H 'Connection: keep-alive' -H 'Referer: https://www.albertsons.com/' -H 'Sec-Fetch-Dest: empty' -H 'Sec-Fetch-Mode: cors' -H 'Sec-Fetch-Site: same-site' -H 'Pragma: no-cache' -H 'Cache-Control: no-cache' -H 'TE: trailers' --data-raw '{"radius":15,"dateOfBirth":"04/20/1980","serviceTypes":[{"serviceTypeName":"COVID-19 Pfizer","scientificName":""}],"zip":"60190"}'

Need to catalog various serviceTypeName values and how many we can include together.
Need to figure if other data needs to go along with it (e.g. for boosters, a last vaccination date).
Search radii available in the UI is 15, 30, or 50 miles. Can we query other values?

Looks like you can get a list of services from a POST to https://rxie.albertsons.com/abs/prod/rxie/appointment/services. It returns data like:
{
    "services": [
        {
            "type": "Vaccine",
            "name": "COVID-19 Booster Dose Janssen",
            "label": "COVID-19",
            "scientificname": "",
            "epsDisplayName": "COVID-19 Booster Dose Janssen",
            "eligible": true,
            "description": "Minimum required age for vaccine is 12 years",
            "restrictionText": "",
            "additionalText": "Ages 3+",
            "sortIndex": 1,
            "category": "COVID-19"
        },
        // ...etc....
    ],
    "correlationId": "abc123"
}

The name field appears to match the serviceTypeName in other requests, and it looks like we can filter COVID vaccines based on label == "COVID-19" (COVID non-vaccine stuff seems to have other labels).
Here’s a full output I’m getting right now: https://gist.github.com/Mr0grog/f1cc7f89e3b27dc1ad567b0f23c14692
cURL example:
curl 'https://rxie.albertsons.com/abs/prod/rxie/appointment/services' \
  -X POST \
  -H 'Accept: application/json' \
  -H 'Accept-Encoding: gzip, deflate, br' \
  -H 'correlationId: 7e0e336a-3dc8-45fe-a6c2-280cab92f349' \
  -H 'version: 2' \
  -H 'ocp-apim-subscription-key: b7bda12ac31a47b0a2bb3cad633d60c4' \
  -H 'Content-Type: application/json' \
  --data-raw '{"dateOfBirth":"04/20/1980","sendOnlyEligible":true}' \
  --compressed

Not sure if the correlationId header is some kind of session token or a more static API key sort of thing, or what, but it is required.

First vaccine serviceTypeName:

COVID-19 Pfizer
COVID-19 Moderna
COVID-19 Novavax

Second dose does not include previous vaccination date in the payload. serviceTypeNames:

COVID-19 Unknown (If you select second dose but not of Pfizer/Moderna/Novavax)
COVID-19 Dose 2 Pfizer (You also get this if you select dose 2 of Moderna, although the services JSON from above includes a COVID-19 Dose 2 Moderna listing)
COVID-19 Dose 2 Novavax
COVID-19 Booster Dose Novavax
COVID-19 Updated Booster Pfizer
COVID-19 Updated Booster Moderna
COVID-19 Additional Dose Pfizer
COVID-19 Additional Dose Moderna

If I set a young age:

COVID-19 Dose 1 Pfizer Child
COVID-19 Dose 1 Moderna Child
COVID-19 Dose 2 Pfizer Child (You also get this if you select dose 2 of Moderna, although the services JSON from above includes a COVID-19 Dose 2 Moderna Child listing)
COVID-19 Dose 2 Pfizer Child
COVID-19 Updated Booster Pfizer Child
COVID-19 Updated Booster Moderna Child

It appears that if you specify multiple serviceTypeNames, you get locations that support all of those types, not locations that support any. So we need to make multiple queries per zip code to get a full accounting.

Finally, the availableAppointments field always seems to be null, BUT it appears that a location is only returned if there are available appointments. So we can use listings as a YES signal, then find all other Albertsons locations (since we have a complete list) and mark them as NO.

Search radius can go at least to 76 miles, but it does get capped at some level. It's unclear what number that is, so maybe best to stick with 50, which is the max in the UI.


Implement network configuration

Do you think It may be possible to network from a module?
On dokku_network.
for example
services:
  web:
     build: .
      links:
         - "db:database"
  db:
      image: postgres

I sometimes coded with python actually, but maybe I could have a look if this feature may interest you and I found sometime later.

@ltalirz Just a note, we support specifying an associated app network, but not necessarily specifying --link except through docker options.

hi @Guerlielton - that certainly sounds doable.
As @josegonzalez mentions you would need to call docker directly but that is not a problem.
Writing an ansible module is really quite straightforward, I suggest you have a look at the dokku_proxy for example, and see the contributing guide for local setup.
Feel free to get started!
On a more general note, this issue touches on one feature that is currently missing in dokku, namely support for docker-compose.
@josegonzalez were just discussing potential formats for writing down a configuration of an app - what are your thoughts on the docker-compose.yaml?

I think we have support for dokku_docker_options which may work for arbitrary flags (though in this case, a dedicated library that interacts with the network plugin is best).
I don't think docker-compose support made a ton of sense in the past - PR here - but maybe make more sense now? The biggest issues I see are:

lack of zero downtime support
we'd have to figure out a way to properly inject the Docker images instead of having docker-compose rebuild
how do we - and do we? - stop compose from starting certain containers?

Probably worth a separate discussion, but this is a bit more possible with custom builders and schedulers. Let me know if you want to talk about this on slack :)

@Guerlielton Does https://github.com/dokku/ansible-dokku/pull/108 address your use case?

I think yes, I will test in my case thanks for feedback  @ltalirz  and  @lemoony  congrats for good job.
I will close this issue


Determine what to do on IBC channel closure

Probably nothing.

On ChanCloseInit it seems like we can safely error. We should test this to verify, but my understanding is that this will cancel the action.
ChanCloseConfirm is a different story though.

By the time we're receiving that message, our partner module has closed its connection. At that point, maybe we should delegate the ability to an admin (likely a community pool) to release all the tokens in escrow for that chain. The admin would only be able to do this if the channel had closed, locking those NFTs.

From the spec:

Once closed, channels cannot be reopened and identifiers cannot be reused.

This means that any NFTs that have IBCd away on that channel will never be recoverable.


Binding Option unavailable

The Default quick move button on controller is "B", and I would like it to be "Y" but there is no option within the bindings menu to change that binding.

You'll need to upgrade to 0.10.1

You'll need to upgrade to 0.10.1


Request to change Root domain to main domain.

[Enter feedback here]
Change root domain to main domain:
Map a subdomain (for example, www.contoso.com) by using a CNAME record.
Map a main domain (for example, contoso.com) by using an A record.
Root domain is the .(root) 13 root domain servers (a-m.root-servers.net) and contains the resource records for the authoritative servers of all TLDS. Subdomains are to the left of the 1st level domain and maps to a directory in the 1st level domain. A subdomain is a subset of the main domain.
Example:
www.contoso.com.
www.contoso.com.(root)
Sub-domain.firstleveldomains.topleveldomains.rootdomains
Sub-domain.maindomain.topleveldomain.rootdomain

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

ID: 224b73dd-c59d-a833-0d4d-a778af6086a9
Version Independent ID: c6b167c9-9589-45d0-ee6a-7a47a4aec41e
Content: Tutorial: Map existing custom DNS name - Azure App Service
Content Source: articles/app-service/app-service-web-tutorial-custom-domain.md
Service: app-service
GitHub Login: @cephalin
Microsoft Alias: cephalin

Thank you for your feedback! We will review and update as appropriate.

@nelroosevelt Thanks for the feedback! I have assigned the issue to the content author to review further and update the document as appropriate.

@nelroosevelt Thank you for the feedback. While we agree that "main domain" avoids the accuracy issue, it's meaning is nebulous in the context of this doc. I've brought your feedback to the App Service product team and we decided that in the interest of clarity in the steps, we're going to stick with "root domain" at this point, accepting that it's less than a completely accurate use of the term. Thank you for your understanding. #please-close


Run CI with --frozen

Some distributions run their builds with --frozen. If the Cargo.lock committed into the tree is not up to date, this build will fail.
Now that cargo auditable is packaged by an increasing number of distributions, it's important to keep the package maintainer effort low. We should verify on CI that the package is buildable with --frozen to keep the checked-in Cargo.lock up to date.
See #85 for more details on the issue.

running with --locked is probably good enough for this purpose


High DPI Scaling Not Working

Bitwig 5.1.8 Clap and VST3 menu doesn't seem to observe High DPI Scaling. Easily resolved by setting Bitwig to not use High DPI Scaling and sandboxing Airwindows Consolidated or by running Bitwig's pluginhost.exe in HDPI Compatibility mode to force correct scaling on High DPI monitors, but would be nice if the menus could be High DPI aware so they are not so small? Great work on this though, thank you!

This is on windows yes? Thanks for the report.

OK I'm not sure why the HDPI isn't scaling but I see why the menu isn't following the parent. I have a fix for that that I just tested.
I'm going to leave this open to figure out why it doesn't come up as an HDPI plugin but will push the menu fix this morning.

Fantastic, thank you and yes, was Windows. Will have a look tomorrow.

Hi! Just curious how the absolute latest version does for you? Lots of changes this week and it would be great if I could know issue status of the open issues over the weekend
thanks!

Hi Paul, sorry work got busy to test again. Looks good. Bitwig 5.1.8 CLAP.

Awesome thanks for confirming
so only thing left is: why doesn’t bitwig launch as hdpi and set the scale right? Or is that working also now?

Bitwig itself scales fine, probably the best I've seen in any DAW in fact. It's a little different though because it uses a JAVA JRE for the GUI and then C++ for the audio engine I believe. This way they decouple all graphics processing from the audio processing for stability and offload it to the GPU instead of CPU which many plugins still seem to use for some reason.
The plugins themselves can either be set to run in Bitwig's audio engine or sandboxed by developer or individually within their own .exe host so if they crash, Bitwig carries on playing as if nothing happened. This is BitwigPluginHost-X64-SSE41.exe for 64-bit plugins.

What I do for problematic plugins is set that .exe hosting them to a compatibility mode for HDMI overriding the default Windows behaviour and that usually fixes the issue if Bitwigs own plugin scaling didn't work.
From the plugins I've reported scaling issues, it nearly always turns out to be because the development was done on MacOS, but it's reactively common issue in plugins and applications. As I said it took Blackmagic 15 years just to add Windows HDPI support as it's' been standard part of Windows for 15 years or something now.

I thought on windows bitwig had two host processes an hdpi one and not? I was more asking if somehow bitwig is choosing the non hdpi one here
last I checked all claps run in the hdpi host but I’m not sure what the vst3 semantic is
(and windows has had hdpi for 15 years but it’s still a mess of an api. The reason it works well on Mac is because they broke old apps and windows not doing that makes plugins hard)

I suppose you could say Bitwig itself has two scaling modes. e.g. if I set Windows Display to 175% scaling, Bitwig set to 'auto' mode will simply inherit the 175% dpi scaling from Windows OS.

But if I disable 'auto' Bitwig can then scale itself to whatever independently of Windows scaling.
In the plugins settings, the default scaling mode is to enable high dpi scaling for them. I assume this is essentially running the host in high dpi awareness mode and then plugins it hosts should inherit the scale as Bitwig inherits from Windows? But this doesn't always work for some plugins (if I were to guess, maybe Juice plugins don't observe it reliably).

The other issue is some plugins simpy don't scale whatever you do. If Windows isn't set to 100% 1:1 scaling, you simply can't use the plugin as it gets cropped in its own window.
Here's Clap plugin:

and here's the VST3 plugin.

Scaling is working fine too.
200% Bitwig Scaling, Plugin Scaling On

150% Bitwig Scaling, Plugin Scaling On

imo that is how it should be when Bitwig isn't set to adopt Windows scaling. The Plugin shouldn't be affected by the DAW scaling, the plugins should adopt what Windows is set to.

And with plugin scaling off, nothing should change if the plugin observes scaling from the OS, which is what happened for Airwindows Consolidated.

OK so it sounds like we are all good then! Thank you for the detailed explanation. I'll tell my bitwig friends that we got it sorted and close this, then, since it seems we are all good now we've fixed the font rendering and menu scaling.
Appreciate the help. Enjoy the plugin!!