Fix: update for pip and docker

Sadly the docker build has been broken because the systems started enforcing the "don't break system packages" for pip installs.
To stay closer to the actually build instructions, I've updated to install cmake from the repo as opposed to pip, this should really be problematic because nothing fancy is happening in cmake.
Additionally, I finally took the time to setup a docker hub account to host an image. I'm happy to create a buspirate org and leave it up there, but also, I didn't want to do that without asking because it wouldn't be owned by the real maintainers :)

Huh - that's neat. It looks like there's something up with the Windows build.
CMake Error at CMakeLists.txt:11 (include):
  include could not find requested file:

    C:\Users\runneradmin/.pico-sdk/cmake/pico-vscode.cmake

This was definitely something I saw at the top of the cmake file that I'm wondering if it was supposed to make it into main at this time?

I've got a change that looks like this will make WSL function correctly with picotool. Seems I've also been hitting this issue as well
https://github.com/raspberrypi/pico-sdk/pull/1820#issuecomment-2291371915
I'm going to tweak the container again and see if this resolves the build to make it operational on both.
Moving back to draft.

Thank you for the updates and attention to the docker scripts. I'll away your final pull request.

This should address the previous limitations on the docker container, also fixing the fact that the picotool wasn't actually being found correctly and doing the CMake things it was supposed to be doing.
Ref:

https://github.com/raspberrypi/pico-sdk/pull/1820#issuecomment-2291371915
https://github.com/raspberrypi/pico-sdk/pull/1820#issuecomment-2291611448

@DangerousPrototypes -- know any WSL users that can test out a windows .devcontainer in vscode? :-D

@DangerousPrototypes  -- for what it's worth, both of the build did complete successfully, but it looks like the GH actions cancelled them -- you can see it got to the linking step in the logs and did what it was supposed to.


Remove Jquery dependency

Why on earth would you pull in something like Jquery to do something jqLite can easily do? Talk about dependency bloat.
Using https://github.com/benmarch/angular-ui-tour instead

Pull requests 👍 snarky commentary 👎
If you have questions about making changes to the codebase or the pull request process, I'd be glad to help. And angular-ui-tour looks really nice. Thanks for bringing it to my attention 😄


spf plugin ran callback multiple times

Don't know why it's happening, but here's the log. No detailed info available.
Aug 21 10:24:48 dev3 haraka[5877]: [INFO] [2902D24F-8F83-4EAD-8A6C-8F4A8134AE6E.1] [spf] identity=mfrom ip=<IP_ADDRESS> domain="enbridgegas.com"<EMAIL_ADDRESS>result=TempError
Aug 21 10:24:48 dev3 haraka[5877]: [INFO] [2902D24F-8F83-4EAD-8A6C-8F4A8134AE6E.1] [spf] scope: mfrom, result: TempError, domain: enbridgegas.com
Aug 21 10:24:48 dev3 haraka[5877]: [INFO] [2902D24F-8F83-4EAD-8A6C-8F4A8134AE6E.1] [greylist] Requested to skip the GL because skip rule matched: dnswl.org(MED)
Aug 21 10:24:48 dev3 haraka[5877]: [INFO] [2902D24F-8F83-4EAD-8A6C-8F4A8134AE6E.1] [karma] applied results.spf.result:-2
Aug 21 10:24:48 dev3 haraka[5877]: [INFO] [2902D24F-8F83-4EAD-8A6C-8F4A8134AE6E.1] [karma] connect: -2, history: 23, total_connects: 77, fail:spf.result, skip:penalty
Aug 21 10:24:48 dev3 haraka[5877]: [INFO] [2902D24F-8F83-4EAD-8A6C-8F4A8134AE6E.1] [spf] identity=mfrom ip=<IP_ADDRESS> domain="enbridgegas.com"<EMAIL_ADDRESS>result=TempError
Aug 21 10:24:48 dev3 haraka[5877]: [INFO] [2902D24F-8F83-4EAD-8A6C-8F4A8134AE6E.1] [spf] scope: mfrom, result: TempError, domain: enbridgegas.com

Aug 21 10:24:48 dev3 haraka[5877]: [ERROR] [2902D24F-8F83-4EAD-8A6C-8F4A8134AE6E.1] [core] spf plugin ran callback multiple times - ignoring subsequent calls
Aug 21 10:24:48 dev3 haraka[5877]: [ERROR] [2902D24F-8F83-4EAD-8A6C-8F4A8134AE6E.1] [core] Error
Aug 21 10:24:48 dev3 at callback (/home/haraka/nodejs/node_modules/Haraka/plugins.js:285:34)
Aug 21 10:24:48 dev3 at Plugin.exports.return_results (/home/haraka/nodejs/node_modules/Haraka/plugins/spf.js:251:20)
Aug 21 10:24:48 dev3 at ch_cb (/home/haraka/nodejs/node_modules/Haraka/plugins/spf.js:187:23)
Aug 21 10:24:48 dev3 at mech_chain_caller (/home/haraka/nodejs/node_modules/Haraka/spf.js:311:24)
Aug 21 10:24:48 dev3 at /home/haraka/nodejs/node_modules/Haraka/spf.js:496:52
Aug 21 10:24:48 dev3 at Object.asyncCallback [as callback] (dns.js:81:16)
Aug 21 10:24:48 dev3 at Object.onresolve [as oncomplete] (dns.js:199:10)

Sorry - but you'll have to upgrade your spf plugin to match the version in git master as the line numbers do not correspond to it.


Target needs WDK installed

Before step 3 can be completed (run C:\Program Files (x86)\Windows Kits\10\Remote\x64\WDK Test Target Setup x64-x64_en-us.msi), you have to install the WDK. Otherwise, that path won't exist.
It is not necessary to install VS or to install the WDK as an extension to VS.

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

ID: ce050ea3-d61e-830c-3038-f725eb4c29e1
Version Independent ID: eb7fc65e-bcfe-dc86-3f8c-336d20eb0c86
Content: Provision a computer for driver deployment and testing (WDK 10) - Windows drivers
Content Source: windows-driver-docs-pr/gettingstarted/provision-a-target-computer-wdk-8-1.md
Product: windows-hardware
Technology: windows-devices
GitHub Login: @EliotSeattle
Microsoft Alias: eliotgra

Thanks for spotting this omission, @cubisttriangle
I've gone ahead and changed Step #2 to:

Install the WDK. You do not need to install Visual Studio, however, unless you plan on doing driver development on the target computer.

It's merged to master. Should be live shortly.
Eliot


master -> live

Docs Build status updates of commit d726681:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/reference/NuGet-Client-SDK.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 24fe1d2:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/api/rate-limits.md
:white_check_mark:Succeeded
View

docs/reference/NuGet-Client-SDK.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit d0e57c6:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/api/rate-limits.md
:white_check_mark:Succeeded
View

docs/create-packages/Creating-a-Package.md
:white_check_mark:Succeeded
View

docs/reference/NuGet-Client-SDK.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 60d203a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/api/rate-limits.md
:white_check_mark:Succeeded
View

docs/create-packages/Creating-a-Package.md
:white_check_mark:Succeeded
View

docs/reference/NuGet-Client-SDK.md
:white_check_mark:Succeeded
View

docs/reference/nuspec.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


[2.x] Correct import

Caught this when I ran vit --list-actions here on Fedora. I think these are the only instances, but worth a check in an installed environment (outside the source tree).

Nice catch, thanks!
I did a quick review and those seem to be the only places the change was needed.


Note that the doc page says to call response.reply() - but this function isn't defined. What is there is a replies array - which has zero elements.
My text is making it to the bot service (it shows up in the log feed), and a my response handler is getting called with a response object. response.toJSONString() returns:
"{\n  "replies" : [\n\n  ],\n  "status" : 200,\n  "source" : "Hello",\n  "next_actions" : [\n\n  ],\n  "language" : "en",\n  "entities" : {\n\n  },\n  "intents" : [\n    {\n      "slug" : "greetings",\n      "description" : "Says hello",\n      "confidence" : 0.98999999999999999\n    }\n  ],\n  "uuid" : "f6c9cadb-1eae-418a-867b-4d3036e12c68",\n  "timestamp" : "2018-04-16T03:18:45.762471+00:00",\n  "version" : "2.12.0",\n  "conversation_token" : "f33f96791e7139250b85bc8a7e4bab63",\n  "memory" : {\n\n  }\n}"
Notice the replies array is empty.
My bot returns a normal greeting reply when I run it in as a webchat or in the botbuilder.
Am I misunderstanding how to use the API here? Or is something wrong?
Thanks.

Hi Eric, Converse is the old Builder route, we are going to update the Swift SDK with the new route of the builder https://api.recast.ai/build/v1/dialog

Hi @plieb could you please push it to cocoapods? I cannot update it locally. Thanks


new exercise: change

implementation of change exercise for Haskell.

I guess you already discovered that, @abo64 , but Travis CI runs configlet, so it will refuse a new exercise if it is not added to config.yaml.

In what position do you think this exercise should be added?

not sure. the algorithm is perhaps not so trivial, so maybe somewhere in the middle?

I would say it is harder than simple-cipher for sure, but maybe connect and go-counting are harder.
I would put it before connect for now. After we classify all the exercises, we will reorder everything, so it is just temporary.

I'm still not used to this new review feature in GitHub, so this time I'll just write normally.
The PR seems great! Thank you @abo64.
You still need to fix the errors that prevents Travis CI to accept your solution.
Also, run hlint on your solution. It will give you a suggestion.

Now the only issue seems to be the warnings about shadowing variable names, that cause error in Travis CI because your new exercises is tests with stack test --pedantic.

about the eta-reductions of addCoin and dropCoin: compiler complains then about arity mismatch?
or am I doing something wrong here?

The compiler seems to enforce that the two definitions of dropCoin have the same number of arguments.
You can leave it the way it is, or change the last definition to allow the reduction:
    dropCoin _ _ = flip const

Anyway, the exercise seems good to merged as it is. :smile:
The one last thing you can do - if you want to practice your git skill - it to squash all of those commits in one or two. One is easier, with two you can separate the change in config.yaml if you want.
Just remember that the commit names and messages will be eternalized in this repository's history, and thy name shall never be forgotten, Oh, brave contributor! :smile:
If you don't want to do it, I can squash everything while merging. Your choice!

thanks a lot @rbasso
as always when having a discussion with you I have learnt a lot again.


Installation

Hi Guys,
I'm trying to add this to my codeignitor project. I'm not getting anything though.
Below is my code.

    <link rel="stylesheet" href="<?php echo base_url()?>bower_components/bootswatch/paper/bootstrap.min.css" />

    <link rel="stylesheet" href="<?php echo base_url()?>bower_components/angular-filemanager/dist/angular-filemanager.min.css">

    <angular-filemanager></angular-filemanager>

asddsaads

@durasj I tried your php bundle that didnt work either, am I missing something simple here, I'm fairly new to angular?

Hi @SeanFacer,
could you please say what error do you get, how you know it's not working?

Hi
@durasj thanks for your quick response!, theres no error, nothing displays
Thanks

Hi,
so I guess there has to be some JS error in the console or otherwise, JS isn't loaded at all.
Could you please check the console in the developer tools?


Update README.md

Updated with explanation.
Added website link.

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

Signed


Do you encrypt the passphrase client-side

It's actually more of a question than an issue... sorry about that. Do you encrypt the secret client-side (i.e. on the browser)?
I would really like the idea of knowing the server never knows about the secret.  I wonder if it's doable with the feature you offer though...

The encryption happens server-side. A little discussion on why here:
https://news.ycombinator.com/item?id=5141131
If you include a passphrase, that's included in the encryption key. That passphrase is sent to the server but it's not stored so it's not possible to decrypt that data once it's at rest.


A question about wrong pieces and right pieces

Hey @flbulgarelli,
Sorry for asking but what would be an easy way to validate the pieces in a basic scenario with an image and autogenerating the pieces. For example I have a DONE button and when clicked I would like to say : You connected 10 pieces right and 5 wrong.
I see there is a piecevalidator but i cannot grasp how to use it for such a scenario. I thought of going with a foreach through all the puzzle pieces and checking the connections but wanted to see if there is a better way. I tried to browse through the tests source code to figure out how to use refs but couldn't reach a conclusion.
Thank you in advance

@arius28
May i ask if you have found a solution to that ?

Hi! As @arius28 pointed out, you can use the PieceValidator, but in your case you will need a custom validator instead of for example the one you can attach using attachSolvedValidator.
I agree with @arius28 in that probably iterating over the pieces is the best way, since you seem to need to check for the count of properly connected instead of just whether they are fully properly connected. Actually the builtin validator does something similar to that:
https://github.com/flbulgarelli/headbreaker/blob/c5bd65720fdb091163058ff6b873a6091b7f780f/src/spatial-metadata.js#L30
https://github.com/flbulgarelli/headbreaker/blob/c5bd65720fdb091163058ff6b873a6091b7f780f/src/spatial-metadata.js#L35-L38
For instance, you could use filter + length instead of every in order to count the number of pieces that are in their - relative - place. If you also want to count the number of pieces connected in they proper place and / or the number of properly connected connection, that may be a bit more tricky - since you will have to check their up, left, etc connections, not just the relative position.


unable to import library

Please share the log as text. The last line seems to have a clear description of this issue.


Keep getting Bad credentials

I keep getting Bad credential again and again. I can't find what is wrong in the configuration.
The application already has a User security (/api path). I try to make an other one for the new api bundle (/apimobile path).

Config.yml

fos_user:
    db_driver:     orm
    firewall_name: api_secure_area
    user_class:    APIBundle\Entity\User
    from_email:
        address<EMAIL_ADDRESS>        sender_name:    Team

security.yml

security:

    role_hierarchy:
        ROLE_ANONYMOUS: ROLE_ANONYMOUS
        ROLE_USER:       [ROLE_ANONYMOUS]
        ROLE_ENTITY_M:  [ROLE_USER]
        ROLE_ENTITY_MANAGER: [ROLE_ENTITY_M]
        ROLE_ENTITY_SENIORS_MANAGER: [ROLE_ENTITY_M]
        ROLE_REZOPRO_MANAGER: [ROLE_USER]
        ROLE_MACIF: [ROLE_USER]
        ROLE_ADMIN: [ROLE_MACIF, ROLE_ENTITY_MANAGER, ROLE_ENTITY_SENIORS_MANAGER, ROLE_ENTITY_M, ROLE_REZOPRO_MANAGER]
        ROLE_SUPER_ADMIN: [ROLE_ADMIN, ROLE_ALLOWED_TO_SWITCH]

    # http://symfony.com/doc/current/book/security.html#where-do-users-come-from-user-providers
    providers:
        fos_userbundle:
            id: fos_user.user_provider.username
        auth_token_user_provider:
            id: auth_token_user_provider
        backend:
            entity:
                # class:    AppBundle\Entity\APIUser
                class:    AppBundle\Entity\APIUser
                property: email

    firewalls:
        # disables authentication for assets and the profiler, adapt it according to your needs
        api:
            pattern: ^/api/
            stateless: true
            simple_preauth:
                authenticator: auth_token_authenticator
            provider: auth_token_user_provider
            anonymous: ~

        api_secure_area:
            pattern: ^/apimobile/
            stateless: false
            anonymous: ~
            provider: fos_userbundle
            lexik_jwt: ~
            json_login:
                check_path: /apimobile/login
                success_handler: lexik_jwt_authentication.handler.authentication_success
                failure_handler: lexik_jwt_authentication.handler.authentication_failure
                require_previous_session: false

        app:
            pattern: ^/
            anonymous: true
            provider:  backend
            form_login:
                login_path: login
                check_path: login_check
            logout:
                path:       logout
                target:     homepage

        dev:
            pattern: ^/(_(profiler|wdt)|css|images|js)/
            security: false

    access_control:
        - { path: ^/api, roles: ROLE_ANONYMOUS }
        - { path: ^/admin, roles: ROLE_SUPER_ADMIN }
        - { path: ^/downloads, roles: ROLE_ANONYMOUS }
        - { path: ^/Resources, methods: [GET] }

    encoders:
        AppBundle\Entity\APIUser:
            algorithm: bcrypt
            cost: 12
        APIBundle\Entity\User: sha1

routing.yml

api_mobile_login_check:
    path: /apimobile/login

I'm wonderring if there isn't something tricky with the fact FOSUser need an username as identifier when then existing backend user provider uses email.
As a result, keep getting this:

curl -X POST -H "Content-Type: application/json" http://localhost:8000/apimobile/login -d '{"username":"UserTest","password":"test"}'

{"code":401,"message":"Bad credentials"}

Thanks for your help.
EDIT: UserTest created using the fos:user command in terminal.

You can define both username and email for authentication using FOS Userbundle
providers:
    fos_userbundle:
        id: fos_user.user_provider.username_email

In this way, FOS Userbundle knows that it should load users by username of email.
If you want to use the same users in different firewalls, you also have to define a contect in each firewall: https://symfony.com/doc/current/reference/configuration/security.html#firewall-context


Improved CI

Specific Demand
CI should be doing a lot more heavy lifting. bors might be too heavy for us right now, but we should at least configure things like

[ ] better test suite
[ ] visual CI
[ ] performance tracking
[ ] automatic app screenshots
[ ] automatic example deployments
[ ] automatic docsite deployments
[ ] run all examples

I believe we should distinguish between build and runtime tests. Build tests is easily catchable whereas runtime tests should be run in a browser environment.
In this doc we have some hints on how to perform these.
disclaimer: inspired by yew ci/di pipeline
I would then propose this example for testing runtime:
use wasm_bindgen_test::{wasm_bindgen_test as test, wasm_bindgen_test_configure};
// import necessary dioxus libs
use dioxus_core::prelude::*;
use dioxus_core_macro::*;
use serde::{Deserialize, Serialize};

wasm_bindgen_test_configure!(run_in_browser);

// for each test case
#[wasm_bindgen_test]
fn some_desc_of_the_test() {
    fn main() {
        console_error_panic_hook::set_once();
        wasm_logger::init(wasm_logger::Config::new(log::Level::Debug));
        dioxus_web::launch(APP);
    }

    static APP: Component = |cx| {
        cx.render(rsx! {
           // some code
        })
    };

    // defining some components

    main();
    // perform some tests with gloo_utils or other for instance in this section:
    // assert(...);
}

The tests should be located in a test folder in each packages along with a Makefile.toml with this for instance:
[tasks.test]
extend = "core::wasm-pack-base"
command = "wasm-pack"
args = [
    "test",
    "@@split(DIOXUS_TEST_FLAGS, )",
    "--",
    "--features",
    "${DIOXUS_TEST_FEATURES}",
]

In the Cargo.toml of the package, we should include at minima:
[dev-dependencies]
wasm-bindgen-test = "0.3"

In the root repo, we should have a Makefile.toml with for instance:
[config]
min_version = "0.32.4"
default_to_workspace = false

[env]
CARGO_MAKE_EXTEND_WORKSPACE_MAKEFILE = true
CARGO_MAKE_CLIPPY_ARGS = "-- --deny=warnings"

[config.modify_core_tasks]
private = true
namespace = "core"

[tasks.tests]
category = "Testing"
description = "Run all tests"
dependencies = ["tests-setup"]
env = { CARGO_MAKE_WORKSPACE_SKIP_MEMBERS = ["**/examples/*", "**/packages/changelog"] }
run_task = { name = ["test-flow", "doc-test-flow", "ssr-test", "website-test"], fork = true }

[tasks.tests-setup]
private = true
script_runner = "@duckscript"
script = [
    """
    test_flags = array --headless --firefox
    dioxus_test_features = set wasm_test
    dioxus_test_flags = array_join ${test_flags} " "
    echo "running tests with flags: ${dioxus_test_flags} and features: ${dioxus_test_features}"
    set_env DIOXUS_TEST_FLAGS ${dioxus_test_flags}
    set_env DIOXUS_TEST_FEATURES ${dioxus_test_features}
    """,
]

I would also note that examples should at least compile, and that runtime should be tested inside packages.
I'll try to do some PR regarding that.

#190


@alloy: Make height of marketing modal to 85% of viewport

@jessekedy mentioned that it would be ideal to have a full-screen modal, and I noticed that with variable copy length our fixed height would chop off content sometimes. This is somewhat of a compromise to address those issues that makes the modal height based on the viewport so that most of the screen is taken over with just a peak of content below. Related to: https://github.com/artsy/force/issues/365#issuecomment-262343858

Oooh, flex box 👌


Several Cmdlets from Microsoft.PowerShell.Utility are missing

The following Function and Cmdlets are missing from Microsoft.PowerShell.Utility
Function        ConvertFrom-SddlString                             <IP_ADDRESS>    Microsoft.PowerShell.Utility
Cmdlet          ConvertFrom-String                                 <IP_ADDRESS>    Microsoft.PowerShell.Utility
Cmdlet          Convert-String                                     <IP_ADDRESS>    Microsoft.PowerShell.Utility
Cmdlet          ConvertTo-Html                                     <IP_ADDRESS>    Microsoft.PowerShell.Utility
Cmdlet          Export-PSSession                                   <IP_ADDRESS>    Microsoft.PowerShell.Utility
Cmdlet          Import-PSSession                                   <IP_ADDRESS>    Microsoft.PowerShell.Utility
Cmdlet          Out-GridView                                       <IP_ADDRESS>    Microsoft.PowerShell.Utility
Cmdlet          Out-Printer                                        <IP_ADDRESS>    Microsoft.PowerShell.Utility
Cmdlet          Send-MailMessage                                   <IP_ADDRESS>    Microsoft.PowerShell.Utility
Cmdlet          Show-Command                                       <IP_ADDRESS>    Microsoft.PowerShell.Utility
Cmdlet          Unblock-File                                       <IP_ADDRESS>    Microsoft.PowerShell.Utility
Cmdlet          Update-List                                        <IP_ADDRESS>    Microsoft.PowerShell.Utility

It would be great if at least the Convert-FromString, Convert-String, ConvertTo-Html, Send-MailMessage and Update-List Cmdlets get implemented. The Out-GridViewand Show-Command Cmdlets may be difficult due to graphical interface requirements, similar for Out-Printer with external printer requirement. A removal notice (ideally including explanation) should be added to the release notes for the other Cmdlets if they will be permanantly removed.

Command availability table need to be updated.

I just coming up with these missing ones.  But specially I was need the ConvertTo-Html cmdlet.

+1 on the missing ConvertTo-Html

Missing Send-MailMessage

This example from Technet to solve my issue with missing the ConvertTo-Html cmdlet does not work either.
 $URL = Read-Host "Enter URL to Decode"  $Encode = [System.Web.HttpUtility]::UrlEncode($URL)  Write-Host "This is the Encoded URL" $Encode -ForegroundColor Green
`

$Encode = [System.Web.HttpUtility]::UrlEncode($URL)

        ~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : InvalidOperation: (System.Web.HttpUtility:TypeName) [], RuntimeException
FullyQualifiedErrorId : TypeNotFound`

+1 on the missing ConvertTo-Html

I successfully build the PS module in Ubuntu 16.04 as shown in https://github.com/PowerShell/PowerShell/blob/master/docs/building/linux.md
I am doing some tests and I have this error:
Unable to find type [System.Net.ServicePointManager].
Also I can't find [System.Web].

@PowerShell/powershell-committee reviewed this, show-command, unblock-file, out-gridview, out-printer won't be in PowerShell Core.  ConvertFrom-String and Convert-String is dependent on MSR.  The other ones should be in PS Core.

Is there any documentation I could read to get a better understanding of the planned structure? Is there a definition of PowerShell Core? Are there other PowerShell ??? kits defined?
Today I was looking for [System.Web.HttpUtility]::HtmlEncode(), but ended up using [uri]::EscapeDataString() mentioned in this issue. Not totally the same thing.

Are there plans to implement Send-MailMessage? Maybe this should be broken out into separate issues for specific cmdlet implementations.

Is ConvertFrom-String going to be added? I see in a comment above it is dependent on MSR.... what does this mean?

No plans at this time to port ConvertFrom-String.  MSR means Microsoft Research.  That cmdlet uses code not planned to be Open Source.

I was wondering what is the fundamental opposition against Unblock-File. The problem is, that without it on Windows, PS Core as it is, is very hard to use. #2657 shows that there is a major perf difference when it comes to downloaded content, may that be from the browser or an installed module.
(I suspect that the ~5 second startup time of PS6 versus ~0.7 second startup of PS5 is due to some similar difference, even though I unblocked the entire PS6 install dir and all downloaded modules already. Something must still remain unblocked I guess.)
I really want to advocate PS6 at my institute for daily use not just by sysadmins, but also end users. Truth is, discrepancies like Unblock-File, the total absence of NetAdapter cmdlets really take a toll on its view. I must say things like:
"Well, if you're on Windows, you can configure your network like this. If you're on Linux, you still gotta go with the same config files and/or config utilities. If you're on Windows, you gotta unblock the downloaded content you trust; to do that, pop-open another shell with slightly different language rules (& operator vs. operator &) and issue that oneliner there."
These stuff breaks the workflow and generally give the picture, that PS6 is a hack. I will probably demo PS5 on Windows and PS6 on Linux and just silently omit the differences (and also not show any of the differing parts which hurts usability on both ends), and also one has to make do with stuff like: don't use Workflow Foundation, because that stuff is not portable, also don't use DSC, beause proper PowerShell DSC resources don't exist yet (OMI+Linux vs. WMI is a strange duality, I'd rather mention DSC only when it's properly portable), even if it did, the number of Linux resources greatly lack landscape. (User, Group, Package and File. Really??? No NetAdapter, DNS, PackageMirror, PXEServer, PXEClient, ApacheServer or any of the interesting stuff??)
Apologies for the lengthy ranting, I understand that PS6 is the first step on a very long journey, but missing core cmdlets that are part of the basic workflow doesn't paint a good picture. Download a zip from web, extract/expand, deploy and run.

iwr recently caught up to browser/*nix wget speed (#2656)
Unzipping itself is still 6X slower than 7zip (issue)
PS6 launches 7X slower than PS5 (mentioned before)
PS6 cannot even unblock itself to omit triggering AMSE on every startup

How should I advocate PS6 with these issues? Any Linux sysadmin will think operning a terminal in 5 seconds is a joke already.

Just to be fair, when no $PROFILE file exists, PS6 launches just as fast as PS5 (with a simple profile). If I create an empty $PROFILE file, (totally empty), it adds 4 seconds to the startup time.

@MathiasMagnus We already have Unblock-File in PowerShell Core.  Microsoft.PowerShell.Archive is not in the repo.
Did you run startup tests on the latest version?

Current status:

[x] ConvertFrom-SddlString
[ ] ConvertFrom-String
[ ] Convert-String
[x] ConvertTo-Html
[x] Export-PSSession
[x] Import-PSSession
[ ] Out-GridView
[ ] Out-Printer
[x] Send-MailMessage
[ ] Show-Command
[x] Unblock-File
[ ] Update-List

Well, it exists, it just doesn't work.
PS C:\Users\Matty> Unblock-File $PROFILE
Unblock-File : A rendszer nem találja a megadott fájlt
At line:1 char:1
+ Unblock-File $PROFILE
+ ~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : PermissionDenied: (C:\Users\Matty\...ell_profile.ps1:String) [Unblock-File], Win32Except
   ion
    + FullyQualifiedErrorId : RemoveItemUnauthorizedAccessError,Microsoft.PowerShell.Commands.UnblockFileCommand

PS C:\Users\Matty> Test-Path $PROFILE
True

It throws an UnautorizedAccessError exception on my own profile, and any file I tested it with. How can my setup go wrong?

@MathiasMagnus working for me on beta.9 using win-x64.zip release:
unblock-file .\PowerShell-6.0.0-beta.9-win-x86.zip

However, if the file is not blocked, you will get that error message:
#running second time
PS C:\Users\slee\Downloads> unblock-file .\PowerShell-6.0.0-beta.9-win-x86.zip
unblock-file : The system cannot find the file specified
At line:1 char:1
+ unblock-file .\PowerShell-6.0.0-beta.9-win-x86.zip
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+ CategoryInfo          : PermissionDenied: (C:\Users\slee\D...a.9-win-x86.zip:String) [Unblock-File], Win32Exception
+ FullyQualifiedErrorId : RemoveItemUnauthorizedAccessError,Microsoft.PowerShell.Commands.UnblockFileCommand

We should probably quietly succeed if that stream isn't available to remove.  Created issue https://github.com/PowerShell/PowerShell/issues/5353

@SteveL-MSFT Thanks for looking into it, and thanks to @iSazonov for the fix.  I just use it as an interactive shell (and very little scripting) as a replacement for bash, but with an actual type system (something I feel home at).

@MathiasMagnus We welcome any feedback. Not only bugs but also the most hot features.

I'm prepping a PR to add Update-List the other cmdlets we don't have any plans to port

Just out of interest: What's the reason why Out-GridView  won't be part of PS Core?

The current implementation (in Windows PowerShell) uses WPF which isn't available on Linux or macOS.

@rkeithhill thanks for the explanation!

We want and have plans to implement Out-GridView on another platform.

When did Test-Connection disappear and why? It was in RC1 I believe...

Test-Connection was not ported (old Windows only code). Currently we have new ported version in #5328 but not jet merged.

I updated current status in https://github.com/PowerShell/PowerShell/issues/2123#issuecomment-341996659
I believe we can close the issue.


setting innerHTML does not create custom elements correctly in IE/Mozilla... maybe others

I'm not sure if this is a bug or just a limitation but I wanted to get any information I could about this.
If I have the following code:
var myDiv = document.createElement("div");
myDiv.innerHTML = "
<my-control></my-control>
";

I notice the behavior is different when comparing Chome to IE/Mozilla.  In Chrome the element gets created fine and ends up with the template inserted and everything working great.
In IE/Mozilla I am left with a dud element. I understand your documentation says you must create elements through document.createElement() or the constructor.  I was curious if this is supposed to be supported or if there is a workaround for this sort of scenario of loading HTML dynamically like this.
Thanks in advance!
Mike

Yes this should work, even under the polyfill. @sorvell @kevinpschaaf @sjmiles

The CustomElements polyfill can only upgrade Custom Elements that have been attached to the document (since it relies on a MutationObserver on document to find them).  If you create a detached node and innerHTML content containing Custom Elements, the document's MutationObserver will not see them yet, hence they will only be upgraded when attached.  This is a limitation of the polyfill.
See below, the inserted my-control will only be upgraded once its container is appended to the document.
http://jsbin.com/mirituyetu/edit?html,console,output

@kevinpschaaf @arthurevans is this documented anywhere? I may have known this at one point, but have since forgotten. Would be good to document it.

Not sure, maybe we should add doc that lists polyfill limitations/differences if we don't have something like that already...

We do not have it. I think we had a few notes on shadow DOM on the old site.
And there are polyfill notes on webcomponents.io. Those notes used to be on the Polymer site. Some of those may be relevant. I think we should come up with a list, provide a summary of polyfill limitations on p-p.org, and also make sure individual limitations are noted on webcomponents.io.
In this case, is it true that the element will be upgraded if you  use document.createElement or the constructor, and the issue only occurs if the parser creates but does not attach the docs -- for example, using innerHTML, cloning a template's contents manually, or copying contents from a <link rel="import">?

Let's  start a doc!

I am running into a similar problem on Firefox/EDGE (it works on Chrome). If I do innerHTML on an element with a slot, whatever tags I passed into the innerHTML don't get upgraded. Example
Is there a workaround for this?


Feat: Allow use of remote storage

I was attempting to use exiffield with S3 storage, but the storage backend doesn't support the path operation since the files are not local. I've implemented a workaround that allows exiffield to be used regardless of whether the files are remote.

Pull Request Test Coverage Report for Build 60

6 of 6 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.3%) to 97.973%

Totals

Change from base Build 57:
-0.3%

Covered Lines:
145

Relevant Lines:
148

💛  - Coveralls

Thanks for this great contribution. Will have a proper look at your changes soon, but at a first glance, they look good :)


Investigate GitHub Actions

GitHub Actions can do a lot of the work of Travis, without needing to use a separate platform. It would be nice to use GitHub Actions in place of Travis, as it is integrated directly into the repo.

Resolved in https://github.com/wayfair/awesome-learning/pull/48


how to decide a kmer model

Hi,
We're playing with the RNA reads of "cDNA Pass 12 15152101 1030.24 1072" at this page (https://github.com/nanopore-wgs-consortium/NA12878/blob/master/RNA.md).
I wonder if you know what can be a proper kmer model used (from https://github.com/jts/nanopolish/tree/master/etc/r9-models) for this dataset?
Usually how can we decide a proper kmer-model for a nanopore dataset?
Thanks!

What are you trying to do with the data? In general you don't need to choose the k-mer model, nanopolish will choose based on metadata in the fast5 file.
Jared

Hi Jared,
We try to do some analysis similar as the nanopolish eventalign, and need to convert reads back to current levels (instead of using original current file in fast5).
Just wonder if you know what can be a proper kmer model? Or if you can point us to how we could check this from the metadata in fast5 file?
Thanks a lot!

The main things we check are the pore version and molecule type. For the cDNA data from above you'll want r9.4_450bps.nucleotide.6mer.template.model since it is DNA going through the R9.4 pore.
Jared

I see. Thanks!


Update to reflect changes in Slate v0.22.0

Slate made some changes to its core in version 0.22.0, which breaks this plugin. A good summary of the things that'll need updating can be seen here:
https://github.com/ianstormtaylor/slate/pull/836.

#21

Fixed by #21


reveal.js plugin

Create reveal.js plugin, where animations are triggered by reveal.js events.

https://github.com/vizzuhq/vizzu-revealjs-plugin


WCS: creating instance fails with AttributeError

Hi,
I use owslib version 0.22.0.
When creating a new WCS instance
wcs = WebCoverageService(url='http://nrt.cmems-du.eu/motu-web/wcs',
                               version='2.0.1',
                               auth=auth)

the code fails with
AttributeError: 'WebCoverageService_2_0_1' object has no attribute 'timeout'
Error is thrown in line 67 in wcs201.py
 self._capabilities = reader.read(self.url, self.timeout)
This line was changed in this merge recently: https://github.com/geopython/OWSLib/pull/730

I can confirm this bug was introduced in 0.22.0, code was working fine on 0.21.0
I think the issue is that init sets a default but the var name is timeout, it never gets assigned as part of the self object.
Fix should be adding self.timeout = timeout in the _init_ of the class OR to use timeout not self.timeout
self._capabilities = reader.read(self.url, timeout)

I can confirm this bug was introduced in 0.22.0, code was working fine on 0.21.0
I think the issue is that init sets a default but the var name is timeout, it never gets assigned as part of the self object.
Fix should be adding self.timeout = timeout in the _init_ of the class OR to use timeout not self.timeout
self._capabilities = reader.read(self.url, timeout)

This issue is most probably fixed by PR https://github.com/geopython/OWSLib/pull/738

Yes, PR 738 fixes the bug. The new release 0.23.0 is working again.


Add non-blocking async waiting for rate limiters

Presently, waiting on a rate limiter permit involves blocking a thread, even for async executions. Async executions should be enhanced to not block when waiting on a permit.

RateLimiter waiting on async executions is now performed using the configured schduler/ExecutorService/etc.


Sarai threw exception in WorkGiver BuildRoofs: System.NullReferenceException:

Describe the bug
Another red error that caused the log to popup, no noticeable issues. I was about 20 minutes into a new game (same save as my last 2 posts), with a freshly installed Rimworld and config folder. I only have Harmony and RimThreaded installed, and I don't have Royalty DLC. I've reduced the total worker threads to 4 to see if it will reduce the number of errors I get, but it seemed to have no effect. Also I'll mention that I've restarted my game after each new unique error to submit a report.
EDIT: Version was RT 1.2.3
Error Log
https://gist.github.com/UndeadKitten/75a2e4612ec584011f756d39d3bbc0cd
Specs
Operating System: Windows 10 Home 64-bit
Processor: Intel(R) Core(TM) i5-4690K CPU @ 3.50GHz (4 CPUs), ~3.5GHz
Memory: 16384MB RAM
Card name: NVIDIA GeForce GTX 970

[Tested on RT 1.3.2]
I tested the save UndeadKitten had posted to another report here, which is about 5 minutes further into the game than the issue reported here.
Esora (Permadeath).zip
I ran the game for about a week in-game, but was unable to replicate this problem, despite building roofs several times during the test.
I did, however, run into two separate errors.
https://gist.github.com/HugsLibRecordKeeper/9d4252b2b1e5d427c903915b5901c965
The first one is a known problem that I have been tracking for a while now.
https://gist.github.com/Eldulac/effccd476da09cdf4da93f5489f1ec84
It has been consistently happening on every RT game I have played recently, and it now seems that it is a problem with vanilla RT, and not related to other mods.
The second error is a new one to me, and happened twice during my test:
https://gist.github.com/Eldulac/e62386a8bc36025ace5038253518a5ad
It seems easily reproducible. Just load the save and speed up the game. It should happen eventually.
Conclusion
It seems that whatever caused this error in the past, is no longer reproducible, and has likely been fixed since then.


Documentation style guide and cli support for .md file creation

Bug Report or Feature Request (mark with an x)
- [ ] bug report -> please search issues before submitting
- [x] feature request

Area
- [x] devkit
- [x] schematics

Versions
Angular CLI: 6.0.0
Node: 9.11.1
OS: linux x64
Angular:
...
Package                      Version
@ angular-devkit/architect    0.6.0
@ angular-devkit/core         0.6.0
@ angular-devkit/schematics   0.6.0
@ schematics/angular          0.6.0
@ schematics/update           0.6.0
rxjs                         6.1.0
typescript                   2.7.2
Repro steps
NA
The log given by the failure
NA
Desired functionality
What would like to see implemented?
Can we have a documentation format/style guide like jsdoc or a javadoc for angular code and style guide for cli to create .md (P1) or .rst (P2) or HTML (P3) files based on priority mentioned or as suggested by community? This is definitely a low priority task and good to have. If anyone is willing to take this up as a separate community project, I will be willing to pitch in with help.
What is the usecase?
Documentation creation for opensource or projects.
Mention any other details that might be useful
NA

Thanks for this feature request. However this is currently not in the Angular CLI roadmap.
Let's re-visit this in future.


Migration class name error

Downloaded template today, ran migration, got:
Class 'm141022_115912_create_rbac_tables' not found'
In console/migrations/m141022_115912_create_rbac_tables.php
class m1401022_115912_create_rbac_tables extends \yii\db\Migration
should be
class m141022_115912_create_rbac_tables extends \yii\db\Migration
Thanks

What version you have tried to install ? You shouldn't have this problem. I just tried full installation and everything is ok.
You can download the latest version with this command:
composer create-project nenad/yii2-advanced-template advanced

Hi
I forked from github this morning.
On 28 September 2015 at 16:59, Nenad Zivkovic<EMAIL_ADDRESS>wrote:

What version you have tried to install ? You shouldn't have this problem.
I just tried full installation and everything is ok.
You can download the latest version with this command:
composer create-project nenad/yii2-advanced-template advanced
—
Reply to this email directly or view it on GitHub
https://github.com/nenad-zivkovic/yii2-advanced-template/issues/21#issuecomment-143787953
.

--
Peter Dunham

Composer installation should work : http://www.freetuts.org/tutorial/view?id=6#advanced

It must work nicely in 2.2.3 version


Updated README.md to with env instructions

Included ENV instructions

@KRRISH96 please add .env.production too in .gitignore. That was missed.

Added .env.production to .gitignore

Added .env.production to .gitignore


CherryPick to 4.13: Update Adaptive Runtime to add telemetry client to the TurnState (#5509)

Update Adaptive Runtime to add telemetry client to the TurnState.  Also amend the AdaptiveDialogAction  telemetry event to use correct dialog Id and log dialog instance Id

Remove instanceId implementation and tracking for adaptive dialogs

Coverage decreased (-0.04%) to 78.604% when pulling 66c08e9c03dc25b77e2b8c64df0e538b8df35f69 on eric/cherryPickTelemetry into<PHONE_NUMBER>ae478d72e0f8ace64b06cc5467ad22 on 4.13.


Download JSON Schema button in pages doesn't work

The "Download JSON Schema" button in model pages doesn't work.
Example
https://models.accordproject.org/patents/patent.html

May I take this issue?

@vaibhav111tandon Here is an example of a page where the button leads somewhere.
<EMAIL_ADDRESS>The download JSON Schema button appears to work on the majority of model pages that you can navigate to from this page: https://models.accordproject.org/
I'm not sure  why the button leads somewhere on some pages and not others. I'm not very familiar with this part of our website.
@jeromesimeon may be the best person to direct questions to about this. Perhaps we need to remove the button on pages with no JSON Schema?

ok

The underlying cause is that some of the models fail to convert to JSON Schema (which happens statically in the build process).

Opening this back up after 16 days stale.

This issue is on the wrong repository. I'm moving it to models.


New-CsOnlineSession no longer supports the documented -OAuthAccessToken parameter

Using the MicrosoftTeams module 1.1.6 and 1.1.10 preview New-CsOnlineSession doesn't have the -OathAccessToken parameter, without this parameter we are unable to authenticate to SFBO using a Service Principal token.

Hello @happywino how are you?
The parameter has been removed from documentation in a recent update. as the parameter has been, as you said, removed from the cmdlet.
Thanks

Hello @happywino how are you?
The parameter has been removed from documentation in a recent update. as the parameter has been, as you said, removed from the cmdlet.
Thanks

@get-itips Thank you very much for the contribution and sharing this explanation. @happywino Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.

@get-itips Thank you very much for the contribution and sharing this explanation. @happywino Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Replaced freecodecamp.com with freecodecamp.org in the required files

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #56810

I have modified the desired url as specified.

Hi there,
Thanks for creating this pull request.
The linked issue has not been triaged yet, and a solution has not been agreed upon. Once the issue is open for contribution, you are welcome to update this pull request to reflect the issue consensus. Until the issue is open for contribution, we will not be able to review your pull request.


make fails on Ubuntu 18.04.5 LTS

Description
I was following instructions from the Wiki page for Ubuntu and I failed on step make.
The output follows:
[  1%] Linking CXX executable cincludeme
CMakeFiles/cincludeme.dir/cincludeme.cpp.o: In function `std::experimental::filesystem::v1::__cxx11::path::path<char const*, std::experimental::filesystem::v1::__cxx11::path>(char const* const&)':
/usr/include/c++/7/experimental/bits/fs_path.h:199: undefined reference to `std::experimental::filesystem::v1::__cxx11::path::_M_split_cmpts()'
CMakeFiles/cincludeme.dir/cincludeme.cpp.o: In function `main':
/home/chom/GitHub/contour/cmake/cincludeme/cincludeme.cpp:108: undefined reference to `std::experimental::filesystem::v1::last_write_time(std::experimental::filesystem::v1::__cxx11::path const&)'
/home/chom/GitHub/contour/cmake/cincludeme/cincludeme.cpp:108: undefined reference to `std::experimental::filesystem::v1::last_write_time(std::experimental::filesystem::v1::__cxx11::path const&, std::chrono::time_point<std::chrono::_V2::system_clock, std::chrono::duration<long, std::ratio<1l,<PHONE_NUMBER>l> > >)'
collect2: error: ld returned 1 exit status
cmake/cincludeme/CMakeFiles/cincludeme.dir/build.make:94: recipe for target 'cmake/cincludeme/cincludeme' failed
make[2]: *** [cmake/cincludeme/cincludeme] Error 1
CMakeFiles/Makefile2:85: recipe for target 'cmake/cincludeme/CMakeFiles/cincludeme.dir/all' failed
make[1]: *** [cmake/cincludeme/CMakeFiles/cincludeme.dir/all] Error 2
Makefile:129: recipe for target 'all' failed
make: *** [all] Error 2

Environment

Contour Version or Git commit hash: 2089d98a92e06661fab72650388c0bb88be457fe (current master)
Operating System (name / distribution / version): Ubuntu 18.04.5 LTS
Contour configuration: not relevant
TERM environment variable: not relevant
Compiler version: g++ (Ubuntu 7.5.0-3ubuntu1~18.04) 7.5.0 (Though I'm not sure how to confirm g++ is chosen and not e.g. clang)

Steps to Reproduce
Follow instructions from https://github.com/christianparpart/contour/wiki/HowToBuildOnLinuxUbuntu .
make step should fail.

Thanks for the report. Until we come up with an automated fix (in CMakeLists.txt) => https://stackoverflow.com/a/33159746/386670

After upgrading gcc (Ubuntu 9.3.0-11ubuntu0~18.04.1 9.3.0) the following error shows:
[  1%] Built target cincludeme
[ 28%] Built target yaml-cpp
[ 35%] Built target unicode
[ 37%] Built target uc-inspect
[ 43%] Built target crispy_test
[ 48%] Built target crispy-gui
[ 49%] Building CXX object src/terminal/CMakeFiles/terminal.dir/Screen.cpp.o
/home/chom/GitHub/contour/src/terminal/Screen.cpp: In member function ‘void terminal::Screen::operator()(const terminal::SendTerminalId&)’:
/home/chom/GitHub/contour/src/terminal/Screen.cpp:429:54: error: invalid type argument of unary ‘*’ (have ‘int’)
  429 |     auto constexpr Pv = (LIBTERMINAL_VERSION_MAJOR * 100 + LIBTERMINAL_VERSION_MINOR) * 100 + LIBTERMINAL_VERSION_PATCH;
      |                                                      ^~~
/home/chom/GitHub/contour/src/terminal/Screen.cpp:429:85: error: expected primary-expression before ‘)’ token
  429 |     auto constexpr Pv = (LIBTERMINAL_VERSION_MAJOR * 100 + LIBTERMINAL_VERSION_MINOR) * 100 + LIBTERMINAL_VERSION_PATCH;
      |                                                                                ^
/home/chom/GitHub/contour/src/terminal/Screen.cpp:429:120: error: expected primary-expression before ‘;’ token
  429 |     auto constexpr Pv = (LIBTERMINAL_VERSION_MAJOR * 100 + LIBTERMINAL_VERSION_MINOR) * 100 + LIBTERMINAL_VERSION_PATCH;
      |                                                                                ^
src/terminal/CMakeFiles/terminal.dir/build.make:326: recipe for target 'src/terminal/CMakeFiles/terminal.dir/Screen.cpp.o' failed
make[2]: *** [src/terminal/CMakeFiles/terminal.dir/Screen.cpp.o] Error 1
CMakeFiles/Makefile2:1484: recipe for target 'src/terminal/CMakeFiles/terminal.dir/all' failed
make[1]: *** [src/terminal/CMakeFiles/terminal.dir/all] Error 2
Makefile:129: recipe for target 'all' failed
make: *** [all] Error 2

It is really interesting to know why LIBTERMINAL_VERSION_MAJOR seems to be an empty preprocessor definition.
The quickfix might be for now:
auto const Pv = 0;

... to rewrite the Screen.cpp's line 429 to the above. (maybe you have this issue in a related line, too, then this can be fixed the same way).

Before trying your workaround I tried upgrading cmake to 3.18.x (from 3.10.2). Now make blows up in my face with the following log (I did clear build directory):
[  1%] Linking CXX executable cincludeme
cmake/cincludeme/CMakeFiles/cincludeme.dir/build.make:94: recipe for target 'cmake/cincludeme/cincludeme' failed
CMakeFiles/Makefile2:85: recipe for target 'cmake/cincludeme/CMakeFiles/cincludeme.dir/all' failed
Makefile:129: recipe for target 'all' failed

Can you try building with clang?

Build fails:
[  1%] Built target cincludeme
[ 28%] Built target yaml-cpp
[ 35%] Built target unicode
[ 36%] Building CXX object 3rdparty/libunicode/src/tools/CMakeFiles/uc-inspect.dir/uc-inspect.cpp.o
In file included from /home/chom/GitHub/contour/3rdparty/libunicode/src/tools/uc-inspect.cpp:4:
/home/chom/GitHub/contour/3rdparty/libunicode/src/unicode/utf8.h:160:25: error: constexpr function never produces a constant expression [-Winvalid-constexpr]
constexpr ConvertResult from_utf8(uint8_t const* _bytes, size_t* _size)
                        ^
/home/chom/GitHub/contour/3rdparty/libunicode/src/unicode/utf8.h:165:15: note: non-constexpr function 'operator=' cannot be used in a constant expression
    do result = from_utf8(state, *_bytes++);
              ^
/usr/bin/../lib/gcc/x86_64-linux-gnu/9/../../../../include/c++/9/variant:1291:16: note: declared here
      variant& operator=(variant&&) = default;
               ^
1 error generated.
3rdparty/libunicode/src/tools/CMakeFiles/uc-inspect.dir/build.make:62: recipe for target '3rdparty/libunicode/src/tools/CMakeFiles/uc-inspect.dir/uc-inspect.cpp.o' failed
make[2]: *** [3rdparty/libunicode/src/tools/CMakeFiles/uc-inspect.dir/uc-inspect.cpp.o] Error 1
CMakeFiles/Makefile2:1295: recipe for target '3rdparty/libunicode/src/tools/CMakeFiles/uc-inspect.dir/all' failed
make[1]: *** [3rdparty/libunicode/src/tools/CMakeFiles/uc-inspect.dir/all] Error 2
Makefile:129: recipe for target 'all' failed
make: *** [all] Error 2

@christianparpart can confirm, fails with clang-10 also

wrt libunicode, i just pushed an update to libunicode and updated the submodule reference in contour master branch.
The change basically consists of changing some constexpr to inline.
I hope that fixes it for you. :)

Still errors for me:
https://gist.github.com/milochadam/0395c4cc856a857b0e384eaed84f9926

I think I can close this now. GCC versions older than 8 are not supported (internal compiler errors), but you can easily install g++-8 to get it building. Also, build in release mode cmake -DCMAKE_BUILD_TYPE=Release in order to work around another problem of having a too old Qt version installed. (tested with Docker image ubuntu:18.04)


Injection of search data

Is your feature request related to a problem? Please describe.
I would like to have a way to inject a list of [ (keyword,link), ... ] pairs into the search_data.
The Anklang project now uses Poxy to generate its C++ and JS API reference documentation. Since C++ is supported directly by Poxy + Doxygen, search works here, but the JS docs are provided as generated html snippets (using JsDoc + pandoc under the hood) and included via @file f.js \n @htmlinclude[block] f.js.html.
You can take a peek here: https://tim-janik.github.io/docs/anklang/files.html
Due to the @htmlinclude, the JS symbols are not searchable.
Generating a list of interesting symbol names and anchors is doable with some effort, e.g.:
symbols = [  ( 'vue_mixins',  'https://tim-janik.github.io/docs/anklang/util_8js.html#vue_mixins-class' ), ]
But then i still need a way to inject those into the poxy or m.css search_data list.
Describe the solution you'd like
Ideally, I could configure poxy.toml to include a searchdata.json (or searchdata.py) list of symbols+links into searchdata-v2.js.
Additional context
I can start base85 decoding searchdata-v2.js to inject the search list myself, but I figured I'd ask before going through that effort. ;-)
BTW, another use case for injecting search data could be to add tokens from content included via the Doxygen \page command, but making the JS docs searchable is most pressing for me atm.

This is a great idea, but I should probably point out that all the search implementation is 100% m.css; poxy is mostly XML fixes and Doxygen post-processes. Consider suggesting this on the m.css repo?
I could reverse engineer the search implementation, but in general I'd like to keep a decent amount of parity with m.css, and contribute features (or ideas) there wherever possible.
cc @mosra


Catch StateError in sentry_dio

Platform:

[X]  Dart
[ ]  Flutter Android or iOS
[ ]  Flutter Web

IDE:

[X]  VSCode
[ ]  IntelliJ/AS
[ ]  XCode
[ ]  Other, which one?

split-debug-info and obfuscate (Flutter Android or iOS) or CanvasKit (Flutter Web):

[ ]  Enabled
[X]  Disabled

Platform installed with:

[X] pub.dev
[ ] GitHub

Output of the command flutter doctor -v below:
[√] Flutter (Channel stable, 2.10.1, on Microsoft Windows [Version 10.0.22000.556], locale ru-RU)
• Flutter version 2.10.1 at G:\Flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision db747aa133 (5 weeks ago), 2022-02-09 13:57:35 -0600
• Engine revision ab46186b24
• Dart version 2.16.1
• DevTools version 2.9.2
[√] Android toolchain - develop for Android devices (Android SDK version 31.0.0-rc1)
• Android SDK at G:\AndroidSdk
• Platform android-31, build-tools 31.0.0-rc1
• ANDROID_HOME = G:\AndroidSdk
• ANDROID_SDK_ROOT = G:\AndroidSdk
• Java binary at: G:\Program Files\Android\Android Studio\jre\bin\java
• Java version OpenJDK Runtime Environment (build 11.0.11+9-b60-7590822)
• All Android licenses accepted.
[√] Chrome - develop for the web
• Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe
[X] Visual Studio - develop for Windows
X Visual Studio not installed; this is necessary for Windows development.
Download at https://visualstudio.microsoft.com/downloads/.
Please install the "Desktop development with C++" workload, including all of its default components
[√] Android Studio (version 2021.1)
• Android Studio at G:\Program Files\Android\Android Studio
• Flutter plugin can be installed from:
https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 11.0.11+9-b60-7590822)
[√] VS Code, 64-bit edition (version 1.65.2)
• VS Code at C:\Program Files\Microsoft VS Code
• Flutter extension version 3.36.0
[√] Connected device (4 available)
• sdk gphone64 x86 64 (mobile) • emulator-5554 • android-x64    • Android 12 (API 31) (emulator)
• Windows (desktop)            • windows       • windows-x64    • Microsoft Windows [Version 10.0.22000.556]
• Chrome (web)                 • chrome        • web-javascript • Google Chrome 99.0.4844.51
• Edge (web)                   • edge          • web-javascript • Microsoft Edge 96.0.1054.62
[√] HTTP Host Availability
• All required HTTP hosts are available
The version of the SDK (See pubspec.lock):
6.4.0

I have the following issue:
Sentry fll
Steps to reproduce:
Init:
  await SentryFlutter.init(
    (options) {
      options
        ..dsn = 'someDSN'
        ..debug = true
        ..tracesSampleRate = 1.0;
    },
    appRunner: () => runApp(
     .....
    ),
  );
}

  dio.addSentry(
    maxRequestBodySize: MaxRequestBodySize.medium,
    captureFailedRequests: true, // <==== Enable
  );

final response = await dio.get<Map<String, Object?>>('/some_wrong_url'); // catch 404 status code

Actual result:
[sentry] [debug] Could not convert DioError to SentryException
.....
[sentry] StateError (Bad state: No element)
[sentry] #0      Iterable.first (dart:core/iterable.dart:624:7)
#1      DioEventProcessor._removeDioErrorStackTraceFromValue
package:sentry_dio/src/dio_event_processor.dart:78
#2      DioEventProcessor.apply
package:sentry_dio/src/dio_event_processor.dart:48
#3      SentryClient._processEvent
package:sentry/src/sentry_client.dart:289
           <asynchronous suspension>
#4      SentryClient.captureTransaction
package:sentry/src/sentry_client.dart:246
           <asynchronous suspension>
#5      Hub.captureTransaction
package:sentry/src/hub.dart:467
           <asynchronous suspension>
#6      SentryTracer.finish
package:sentry/src/sentry_tracer.dart:103
           <asynchronous suspension>
#7      new SentryTracer.<anonymous closure>
package:sentry/src/sentry_tracer.dart:50
           <asynchronous suspension>

Expected result:
There should be no StateError

Which sentry_dio version are you using?

Which sentry_dio version are you using?

6.4.0 from pub.dev

@Overman775 can you provide a minimal reproducible example on GH? I tried to reproduce it and it didn't happen.

Do you have obfuscation on?
element.type == dioError.runtimeType.toString() will probably fail there.

Do you have obfuscation on? element.type == dioError.runtimeType.toString() will probably fail here: https://github.com/getsentry/sentry-dart/blob/main/dio/lib/src/dio_event_processor.dart#L77

Hm, that should work even with obfuscation, because type is set from the runtime type?

Hmm, true.
Then the function is called with an empty list: List<SentryException>.from(event.exceptions ?? <SentryException>[]) so that  .first fails.
So the event has a throwable but not exceptions, not sure how this can happen, maybe a manually constructed event?

Yeah, strange, let's wait for the minimal example, but for the sake of defensive programming, we should check if the list isn't empty anyway before calling first.

Failed to reproduce the bug in the minimal example. I'll try again and let you know the result


Layer priority?

Is there an equivalent to z-index for layers? I noticed that when drawing some GeoJSON content with opacity, that markers from other layers are respected at outward zooms, but as the zoom comes in, the items appear more behind the lower opacity content.

Hi Kevin,
The layer order is determined by the child order in the MapGL component, like:
<MapGL>
  <Layer>A</Layer>
  <Layer>B</Layer>
</MapGL>

Renders to: Base Style -> Layer A -> Layer B -> Top
So Layer B will be on top of Layer A.
Markers and Popups will always be rendered on top of everything, except when you use a symbol layer as a marker.

Thank you. I will take a look closer and see if that can be mitigated using proper ordering there. Though, I'm wondering why this current scenario occurs only when zoomed in at some level.

The layer order should not be affected by the zoom level. If you looking for a ordering of features within a layer, there is a sort-key style property for that. See Mapbox style docs


Terminal title bar color does not change with dark/light mode switch in Windows Settings app

Environment
Windows build number: 18950
Windows Terminal version (if applicable): 0.3

Any other software? Python, Firefox, Visual Studio, etc. They shouldn't impact this problem.

Steps to reproduce

Open the Windows Settings app.
Change from dark mode to light mode, or vice versa.

Expected behavior
The Windows Terminal title bar color will change after a simple change in the Settings app and adapt to the rest of the Windows environment.
Actual behavior
The Windows Terminal title bar color stays the same after the Settings app dark/light mode change. I have to restart the app in order for the Terminal title bar to obey the Settings change.
(Not talking about accent color, by the way)

This'll be part of #1625.

(for linking purposes, this is also a dupe of #1230)


Advanced Vyazovkin Ea Buffer

Hi,
I think the advanced vyazovkin Ea Buffer is set incorrectly as in the advanced it is set to:
EaBuffer = 1000:1000:200000
which causes an error in a later step as the EA buffer is a different size to the temperature
In the Vyazovkin file the Ea buffer is set to
EaBuffer = 100:100:2000000
This works and if you change the Advanced Vyazovkin to this it then runs.
Thank you for this tool box it is phenomenal, I spent some time trying to build this type of thing myself and this is truly fantastic.

Hi, TownySmash. Thanks for your comment. I increased this range relative to the Vyazovkin method because with a smaller step, the calculation takes quite a long time due to direct integration. So on my data script worked fine with the range of 1000:1000:200000 except for the warnings that polynominal is badly conditioned. Its much faster than 100:100:200000.
Anyway feel free to use and change this toolset for your purposes.


Type of a local variable inferred from a PHPUnit class

We are in the process of adding Pest to an existing Symfony project. I've created a separate folder for Pest tests/Pest/Unit while the existing PHPUnit tests live in tests/Unit, tests/Functional etc. When I try to instantiate a local variable in the beforeEach() function it is somehow picking up the type from a PHPUnit test.
For example, a PHPUnit test in the folder tests/Functional/FooTest.php initialises a $user property:
class CrudTest extends BaseWebTestCase
{
    private User $user;
}

When I try to do the following in my Pest test tests/Unit/BarTest.php
beforeEach(function (): void {
    $this->user = 'test';
});

PhpStorm will complain about incompatible types because it's expecting a User object instead of a string. How do I prevent PhpStorm from inferring types from other totally unrelated PHPUnit tests?

Hmm this sounds pretty weird.
I tried a couple of times and haven't been able to reproduce yet. Would you be able to supply with a simple reproducible example?

@olivernybroe I've created a basic Laravel project to replicate the issue: https://github.com/kielabokkie/pest-issue.
As you can see in the screenshot below I have a separate tests/Pest folder and the default test/Unit folder that comes with Laravel. Somehow the public property $user of my ExampleTest.php class is picked up in the beforeEach function of my PestExampleTest.php class.

Awesome, thanks for providing an example, so we can reproduce it 🤩

@kielabokkie Thank you for the provided example! The real issue is that type of $this is inferred wrong by PhpStorm in context of Pest test. Due to Pest plugin as of now is maintained by PhpStorm team, we are going to gradually move all the reports from GitHub to YouTrack so that we have the single source of tickets. I've created a dedicated ticket for this particular problem in YouTrack WI-75119 so you can track the state of the issue and vote there. Also I've attached your example project to the ticket, thank you for it one more time. So for now I'm going to close this ticket and if you have any comments or other problems with Pest plugin, please report them to YouTrack


Added Response options whitelist

I recently did a PR in omniauth-saml gem to allow more options for a OneLogin::RubySaml::Response object (https://github.com/omniauth/omniauth-saml/pull/159).
After merging it, I was discussing with @md5 about the possibility to source the allowed options from ruby-saml gem instead of hardcode them in this gem.
As of today, omniauth-saml had ~2.3 million downloads and I believe that introducing a whitelist of response options will make the dependency on ruby-saml stronger and even more reliable.
Moreover, it will allow to developers who are using ruby-saml standalone to have a better overview on the options they can use.

hi @pitbulk
I saw in the travis build that jruby-1.7.21 and jruby-<IP_ADDRESS> are failing since a while. They might need a separated Gemfile (https://docs.travis-ci.com/user/languages/ruby/#JRuby:-C-extensions-are-not-supported).
In ree and ruby-1.8.7 an array of symbols is raising TypeError: Symbol as array index. How are you handling this data structure?
Thanks

My unique concern about merging it is:

What happens to projects using ruby-saml that extended the SAMLResponse options with more option? This PR will break their projects.

Related to jruby, I haven't got an environment to test it.

Thank you for your quick reply!
Actually the whitelist of options is a nice workaround to make sure that dependent gems such as omniauth-saml will be updated with the options offered by your gem. I haven't considered the case of someone extending the class and passing custom options.
This is not the only possible solution but it would prevent several prs on other gems which depends on yours.
Would it be suitable to you to rename the ALLOWED_OPTIONS to AVAILABLE_OPTIONS and remove the validation of the options provided? This would achieve the purpose of this PR.

Yes, I think adding ALLOWED_OPTIONS is not bad idea

Perfect! I changed then to AVAILABLE_OPTIONS and dropped the commit which was introducing the filter. I guess this could be ready for a review.

hi @pitbulk ! Could you please bump a minor version of the gem including this PR changes?
Thank you very much


bugfix for multiple freeze & restores

bugfix: adding a second freeze & restore removed items from the first freeze & restore
sort() is already stable -- there is no need to do it twice
fixed sort_by_list() to correctly return the relative order between a and b
simplified numerical_sort()
simplified xor()

Thanks!
Regarding double sorting you are indeed correct that I don't need to do it at this time. My original intention was to allow for a larger variety of sorts (it's a bit of a shame that scripts can't install sorts in WKOF, too), and double sorting does solve a potential problem a user could have if they had provided a less conventional sort (as illustrated in the example). I will accept the PR, but I think I will add the double sort function back (unused) in case it becomes relevant in the future.

Indeed. I was just trying to be thorough. Really if someone wants to do that a filter would be better


Distributed on shared clusters or non-homogenous machines

One of the issues I've faced when running xgboost is that if I have a shared cluster with 10 jobs running at any time... XGBoost behaves very badly.
In the sense, if I have 50 machines and 10 jobs (for map reduce or other xgboost jobs)  are running at any point of time... If I run a new xgboost job the job has the following issues:

the slowest machine in the allocated machines is bottle neck

If the slowest machine takes 1 minute to create a tree, and all other machines take only 10sec to create a tree, xgboost will wait for the slowest machine to decide the 1st split criteria.
Are there any thoughts on modifying this behaviour so that it runs on shared cluster setups or even for non-homogenous clusters in some way?

Any thoughts on this one ?

this is an open-ended question, we do  not have actionable items yet on this, so let us move it to https://discuss.xgboost.ai/.


update to a more recent version of transformers dependency

currently blip depends on transformers==4.15.0, which is from december 2021. making blip incompatible to run in the same venv as any current transformers-related project. updating to become compatible with 4.2x would be great.

Thanks for your comments. Our new LAVIS library supports BLIP with transformers<4.22.0:
https://github.com/salesforce/LAVIS/blob/main/requirements-dev.txt

thank you, i'll give it a try


Sandbox saied go118

This change is

Pull Request Test Coverage Report for Build 131

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

Totals

Change from base Build 125:
0.0%

Covered Lines:
261

Relevant Lines:
261

💛  - Coveralls


Improve "Copy from stream" UI

In the "Create stream" menu, there is a "Copy from stream" option in the "People to add" section. This can be an easy way to bulk-add users to a new stream.
We should improve the visual design of this option:

[ ] Add decorators (#, lock, etc.) based on the type of stream.
[ ] Make the member count part (icon+ number in parens) less dark in light mode / less light in dark mode. It's too bright at present.

These are independent improvements, so a PR for just one of these would be fine as well.

@zulipbot claim

This issue is pending further discussion; we may revamp how this UI works.

Okay i'll wait for the final UI

Closing in favor of #20499.


Enable manual reordering of group subquestions

Group subquestions are sometimes in the wrong order. This is mostly a problem for fan graphs:

The group edit ui should allow explicit reordering of subquestions, something like:

Duplicate of #1398


UI: Is it possible to have two Y axis scales on one graph (left and right)?

For showing two different datasets with very different maximum and minimum values. Weather people do this with precipitation and temperature, for example.

This was part of standing issue #91.  It is better as its own issue.
It is totally possible, it just hasn't been done yet.


ign -> gz Namespace migration (narrow) : sdformat

🦟 Bug fix
Part of https://github.com/gazebo-tooling/release-tools/issues/711, more narrow alternative to #1022
Summary
This is an alternative to #1022 (actually using some commits from that branch) that is narrower in scope. It avoids most macro changes in an attempt to simplify the review process.
Once these namespace changes start being merged, things will break, so we will try to merge them all relatively close together in time.
Checklist

[X] Signed all commits for DCO
[ ] Added tests
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] Consider updating Python bindings (if the library has them)
[X] codecheck passed (See contributing)
[X] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

#1022 got in


Add support for enabling SCIM testmode

The action will trigger a creation of a client and secret by the config job upon new installations. Also the persistence job is effected by this flag.

https://github.com/JanssenProject/jans-cloud-native/commit/c35eb38cd8e71c58e557a7f96962925d52b8d75b

https://github.com/JanssenProject/jans-cloud-native/commit/c35eb38cd8e71c58e557a7f96962925d52b8d75b


Add documentation templating

https://github.com/stackabletech/documentation/issues/237

We're not doing it, there's too much different stuff in there it can't be templated


Firejail itself probably not secure enough

There were several security issues found recently:
CVE-2017-5180: http://openwall.com/lists/oss-security/2017/01/04/1
tmpfs related (CVE-2016-10117, CVE-2016-10119), rewrite /etc/resolv.conf (CVE-2016-10118), full access to system files (CVE-2016-10120, CVE-2016-10121), root access (CVE-2016-10122, CVE-2016-10123): http://openwall.com/lists/oss-security/2017/01/05/4
And 2 more which are fixed by now: CVE-2017-5207, CVE-2017-5206.
It isn't about sandbox actually but if your user somehow vulnerable therefore (by means of firejail) your root is also vulnerable. That's If I got it right consequence of nature of firejail and of using suid mechanism.
Probably it is worth consideting to switch to some alternative. For example:

bubblewrap
Sandstorm.io
Other tools. Some of them mentioned (with some comparison) in bubblewrap README.

I think Firejail is still the best option for this use case. They have a decent community around it now and are quick to fix any reported problems, which makes me comfortable keeping it around. Tools like Sandstorm don't really address the same problem set. Bubblewrap is worth keeping an eye on for the future. I think the stuff coming out of Subgraph is probably the most promising.
In the meantime, if anyone is uncomfortable with Firejail they can --skip-tags=firejail.

@pigmonkey, what do you think on alternatives vs Firejail after a while?

I'm still happy with Firejail for daily use.


Fix a severe bug in the code when the settings change might run even in the PR (NOP) mode.

Fix a severe bug in the code when the settings change might run even in the Pull Request (NOP) mode.
When safe-settings runs in NOP mode, and there are changes to teams settings, it removes the existing teams even when run in the Pull request. This is caused by a bug in the code at https://github.com/github/safe-settings/blob/5dd8e7987f6bf685a44ad857d0c0cc3092cd7002/lib/plugins/teams.js#L75 where
This fixes #339

@decyjphr are you going to merge the fix soon?


Github templates

Github templates
Summary
Setup github templates for both issues and pull requests so that issues and pull requests follow a certain format including all the relevant information we'd need.
Unsure on what the templates should contain at the moment, but happy for others to propose ideas. Will also check similar repositories for inspiration.

@Jason2605 can you assign me ?
I want to work on it.

Done, thank you @MidouWebDev

@Jason2605 do you want these templates to be markdown based or the new github issues and PRs templates (.yml) ?

@MidouWebDev I'm open to either format really, just as long as they show up when creating issues and pull requests!

@Jason2605 Ok.

@Jason2605 I will be creating the feature template soon


codesign issue in macos

I have this issue on macos 12.3.1:
dyld[76184]: Library not loaded: @executable_path/../Frameworks/Library.dylib/librocksdb.6.26.0.dylib
  Referenced from: /private/tmp/cronosd/bin/cronosd
  Reason: tried: '/private/tmp/cronosd/bin/../Frameworks/Library.dylib/librocksdb.6.26.0.dylib' (code signature in <no uuid> '/private/tmp/cronosd/Frameworks/Library.dylib/librocksdb.6.26.0.dylib' not valid for use in process: library load disallowed by system policy), '/usr/local/lib/librocksdb.6.26.0.dylib' (no such file), '/usr/lib/librocksdb.6.26.0.dylib' (no such file)

The above error happens if I try to run a binary downloaded from web, but if I build it locally, it can run, even if the hash of binaries are identical.

One way to fix it is by remove the quarantine attribute on the downloaded file:
xattr -d com.apple.quarantine binary.tar.gz

I think it's because we modified the files after codesign: https://github.com/3noch/nix-bundle-exe/blob/main/default.nix#L36

Is this fixed by #9 ?

yes, closing now.


Fix process name for procfile worker

It should match what's defined in the Procfile in the CPM

 (3) [Tech debt] Use only publishing api to preload items


Update outdated dependencies

Describe your idea
Hi!
I noticed that Detox installs quite old versions of several dependencies (chalk, fs-extra, etc.)
I wanted to help updating these to a newer version, and was prompted by the CI check to create an issue for discussing changes to package.json
Here's my first PR that tackles updating chalk to a newer version (that still maintains node support with what Detox currently supports):
https://github.com/wix/Detox/pull/3987

Thanks @stianjensen! Sounds good 🙏🏼
I reviewed your PR. Generally, if there's a dependency that you think we should upgrade - you can open a PR for that, but please make sure to run our tests suite before. See our contributing guide for more guidance.

@stianjensen , do you want to contribute some pull requests on this regard?


Potential fix for race condition exposed by Node

I have run the test that was previously failing several hundred times locally with this change, and it has not yet failed. The previous failure rate was around 1/20.


Change in vscode breaks this extension

VS Code Version: 1.45.0-insider (user setup)
Commit: abb4a35cfc26102f93fd00df7b59ce1a19c2017a
Date: 2020-04-28T05:36:25.810Z
Electron: 7.2.2
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Windows_NT x64 10.0.18363
Extension Version: 0.11.0
What's the issue:
As mentioned in microsoft/vscode#96240, vscode command vscode.executeDefinitionProvider has been changed to return LocationLink[] instead Location[] in insider version.
Which breaks this extension.

Any idea how to fix it?

@cmoleka Currently this extension only calls that command here:
https://github.com/lannonbr/vscode-js-annotations/blob/f4d9659eeea483ff856695b21ab7883d5beba439/src/parser.ts#L241
The easiest way to fix it might be that if the command returns vscode.LocationLink (new version of vscode), convert them back to vscode.Location by renaming corresponding fields. So every thing else will continue to work.
As vscode.LocationLink was added in a newer version of vscode than this extension is targeting, we can't directly use this type. We can just use any here.

Still broken!
This extension was useful

I installed this extension last week and it doesn't work for javascript or typescript?
I assume it because of the above issue...


[runtime] Don't install libswiftRuntime.a.

libswiftRuntime.a is an internal object file. Its contents are incorporated
into libswiftCore.dylib and libswiftCore.a.
rdar://25353757

@swift-ci Please smoke test


Standard HDD Pricing Discrepancy

The pricing on this page specifies, "$0.0036 per 100,000 transactions for standard HDDs."  However, the Managed Disks Pricing page specifies, "$0.0005 per 10,000 transactions for Standard HDD Managed Disks." (https://azure.microsoft.com/en-us/pricing/details/managed-disks/)

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

ID: 2cd6eb6c-745c-b3eb-8ea8-7fb3be24b607
Version Independent ID: 93637a78-fc66-6c5d-7845-a4fad0045c7d
Content: Select a disk type for Azure IaaS Windows VMs - managed disks
Content Source: articles/virtual-machines/windows/disks-types.md
Service: virtual-machines-windows
GitHub Login: @roygara
Microsoft Alias: rogarana

Thanks for the feedback! We are currently investigating and will update you shortly.

@JGILLEAN , Both the values are correct, but $0.0036 is for Unmanaged Disk. I will assign this to author to get the change reflected.

@JGILLEAN thank you for taking the time to write us feedback, we appreciate it! We've decided to remove specific pricing information and instead refer readers to the ACOM pricing page. The article has been updated accordingly.
#please-close


Update Haystack glossary

@agnieszka-m I have an idea - can I try make the elements here H2 titles (##) and then we can link to one anohter? Like where you refer to 'Documents' from Files could have a link and then from other pages too?


Bugs in C# unit testing examples

File: themes/default/content/docs/using-pulumi/testing/unit.md
In the example Testing.cs in the Add Mocks section there are a couple of issues:

The line args.Id ??= $"{args.Name}_id"; in NewResourceAsync doesn't work as intended - args.Id is passed as an empty string. It should be replaced with:

if (string.IsNullOrWhiteSpace(args.Id))
{
    args.Id = $"{args.Name}_id";
}

The custom GetValueAsync<T> at the end of the file is broken, using it will result in a deadlock since the delegate in the Apply hasn't been executed when we await this version of GetValueAsync. So, the returned Task has no result and the program simply sits waiting for a result that will never arrive.
It would be better to use the SDK's OutputUtilities.GetValueAsync<T> as this correctly returns from the underlying DataTask.

Hello @nimro,
Thank you for submitting this issue! After further investigation, we were not able to reproduce the issues described. I have created a gist here that captures the outputs of both the failure and success cases as of yesterday.
We did however notice that there was a difference in the installed packages when creating a new Pulumi program using pulumi new aws csharp vs pulumi new csharp. I will be updating the guide to include those packages.


Connection to Azure redis failing.

Tried to follow the code sample to connect to Azure Cache For Redis, however the client (nodejs) won't connect. It throws ClientClosedError. I'm guessing it could be with the version of Redis npm package but not sure. I'm still debugging with ^4.0.1.
ClientClosedError: The client is closed
at Commander._RedisClient_sendCommand (C:\Users\krish\Work\Apps\redis\node_modules@node-redis\client\dist\lib\client\index.js:393:31)
at Commander.commandsExecutor (C:\Users\krish\Work\Apps\redis\node_modules@node-redis\client\dist\lib\client\index.js:160:154)
at Commander.BaseClass. [as set] (C:\Users\krish\Work\Apps\redis\node_modules@node-redis\client\dist\lib\commander.js:8:29)
at C:\Users\krish\Work\Apps\redis\index.js:29:18
at Object. (C:\Users\krish\Work\Apps\redis\index.js:37:3)
at Module._compile (node:internal/modules/cjs/loader:1095:14)
at Object.Module._extensions..js (node:internal/modules/cjs/loader:1147:10)
at Module.load (node:internal/modules/cjs/loader:975:32)
at Function.Module._load (node:internal/modules/cjs/loader:822:12)
at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:81:12)
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 8a8fa034-3574-1573-b79c-7bb2c8f411f3
Version Independent ID: 63f96866-471b-167b-7b48-4df3eec465a0
Content: Azure Redis Cache modules for JavaScript
Content Source: docs-ref-services/latest/redis-cache.md
Service: redis cache
Product: azure
Technology: azure
GitHub Login: @wesmc7777
Microsoft Alias: wesmc

With the new versions of redis (probably 3.1.0 and later), client connection should be established in the below format.
const client = redis.createClient({     socket: {         host: AzureRedisHostName,         port: 6380,         tls: true     },     password: azureRedisKey, });

@nidri Thanks for the report! Could you please share what NPM package you were using and its version?

@jeremymeng I was using redis package (https://www.npmjs.com/package/redis) and the version was 4.0.1

@nidri the repository for that package is at https://github.com/redis/node-redis. I'd suggest that you also check and log an issue there.

@jeremymeng will do. My suggestion is to update documentation to have examples using newer versions and formats of redis.
https://docs.microsoft.com/en-us/azure/azure-cache-for-redis/cache-nodejs-get-started

Thank you @nidri! I got it now. The feedback button links to this repo. We will find the proper owner to verify and update the docs.

The documentation points to https://github.com/Azure-Samples/azure-cache-redis-samples/tree/main/quickstart/nodejs which was updated 2 days ago in https://github.com/Azure-Samples/azure-cache-redis-samples/pull/17.
Updating the documentation is being tracked in https://github.com/MicrosoftDocs/azure-docs/issues/86087
Thanks for reporting @nidri!


Not in Scope - VAT Group. Really?

Why does this page state Group VAT in not in scope. This is not true, correct?

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

ID: 35bb67a1-ea3e-ae63-2b5d-c7f6e099459e
Version Independent ID: a6a3650f-2c0a-4d1e-b1a6-8472e0151df9
Content: Making Tax Digital - Submitting VAT Returns - Business Central
Content Source: business-central/LocalFunctionality/UnitedKingdom/making-tax-digital-submit-vat-return.md
Service: dynamics365-business-central
GitHub Login: @sorenfriisalexandersen
Microsoft Alias: soalex

Thanks for your feedback.
As per our instructions, it is correct.


mitxonline user models

Description
This adds staging and intermediate models for mitx user data
Motivation and Context
https://github.com/mitodl/hq/issues/266
How Has This Been Tested?
I spot checked the data. I also have automated tests to check that the int__mitxonline__users table 1) Has the same number of rows as stg__mitxpro__app__postgres__users_user 2) Has unique ids 3) Has no missing ids to make sure that no extra rows are accidentally dropped or created during data normalization
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Enhancement (improves on existing behavior)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project. (Did you install and run the pre-commit hooks?)
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

Ferdi and Peter asked me to break this into smaller pieces so closing it for now


More tweaks and updates

Codecov Report

Merging #21 (6a4213b) into develop (ed0b1fa) will increase coverage by 4.35%.
The diff coverage is 60.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff             @@
##           develop      #21      +/-   ##
===========================================
+ Coverage    59.23%   63.58%   +4.35%
===========================================
  Files           44       43       -1
  Lines          888      887       -1
===========================================
+ Hits           526      564      +38
+ Misses         362      323      -39

Flag
Coverage Δ

unittests
63.58% <60.00%> (+4.35%)
:arrow_up:

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

Impacted Files
Coverage Δ

backend/src/core/engine.py
0.00% <ø> (ø)

backend/src/interfaces/stream.py
0.00% <0.00%> (ø)

backend/src/services/stream/comment.py
0.00% <0.00%> (ø)

backend/src/services/stream/inbox.py
0.00% <0.00%> (ø)

backend/src/clients/reddit/inbox.py
63.63% <33.33%> (+63.63%)
:arrow_up:

backend/src/repositories/sqlalchemy.py
76.71% <33.33%> (+0.32%)
:arrow_up:

backend/src/api/v1/game.py
73.33% <50.00%> (-3.59%)
:arrow_down:

backend/src/api/v1/player.py
73.33% <50.00%> (+1.90%)
:arrow_up:

backend/src/api/v1/subreddit.py
73.33% <50.00%> (-3.59%)
:arrow_down:

backend/src/clients/reddit/base.py
78.57% <80.00%> (+78.57%)
:arrow_up:

... and 11 more

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


[MacOS] Hiden child widget of IndexedStack can be focused when using Tab shortcut key.

Steps to Reproduce

Execute flutter run on the code sample
There are two buttons, one is on the left side, another is on the right side but invisible.
Tap the 'Tab' key on keyboard to switch the focus and tap 'Enter' key to trigger the button.

Expected results: The hiden button should not gain the focus and be triggerred.
Actual results: The hiden button will gain focus and be triggerred.

Code sample

import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        // This is the theme of your application.
        //
        // Try running your application with "flutter run". You'll see the
        // application has a blue toolbar. Then, without quitting the app, try
        // changing the primarySwatch below to Colors.green and then invoke
        // "hot reload" (press "r" in the console where you ran "flutter run",
        // or simply save your changes to "hot reload" in a Flutter IDE).
        // Notice that the counter didn't
        // is not restarted.
        primarySwatch: Colors.blue,
      ),
      home: const MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({Key? key, required this.title}) : super(key: key);

  // This widget is the home page of your application. It is stateful, meaning
  // that it has a State object (defined below) that contains fields that affect
  // how it looks.

  // This class is the configuration for the state. It holds the values (in this
  // case the title) provided by the parent (in this case the App widget) and
  // used by the build method of the State. Fields in a Widget subclass are
  // always marked "final".

  final String title;

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {

  @override
  void initState() {
    super.initState();
  }

  @override
  Widget build(BuildContext context) {
    // This method is rerun every time setState is called, for instance as done
    // by the _incrementCounter method above.
    //
    // The Flutter framework has been optimized to make rerunning build methods
    // fast, so that you can just rebuild anything that needs updating rather
    // than having to individually change instances of widgets.
    return Scaffold(
      appBar: AppBar(
        // Here we take the value from the MyHomePage object that was created by
        // the App.build method, and use it to set our appbar title.
        title: Text(widget.title),
      ),
      body: IndexedStack(
        index: 0,
        children: [
          Container(
            // color: Colors.yellow,
            alignment: Alignment.centerLeft,
            child: InkWell(
              focusColor: Colors.red,
              onTap: () {
                print("Button 1 clicked");
              },
              child: const Text("Button 1"),
            ),
          ),
          Container(
            // color: Colors.blue,
            alignment: Alignment.centerRight,
            child: InkWell(
              focusColor: Colors.red,
              onTap: () {
                print("Button 2 clicked");
              },
              child: const Text("Button 2"),
            ),
          )
        ],
      ),
    );
  }
}

Logs

[✓] Flutter (Channel stable, 3.0.4, on macOS 12.1 21C52 darwin-x64, locale en-CN)
    • Flutter version 3.0.4 at /Users/megatronking/Github/flutter
    • Upstream repository https://gitee.com/mirrors/Flutter.git
    • Framework revision 85684f9300 (4 months ago), 2022-06-30 13:22:47 -0700
    • Engine revision 6ba2af10bb
    • Dart version 2.17.5
    • DevTools version 2.12.2

Checking Android licenses is taking an unexpectedly long time...[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/megatronking/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • ANDROID_HOME = /Users/megatronking/Library/Android/sdk
    • ANDROID_SDK_ROOT = /Users/megatronking/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2020.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)

[✓] VS Code (version 1.72.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.51.20221003

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 12.1 21C52 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 106.0.5249.119

https://user-images.githubusercontent.com/15059084/198515577-ab2a02d9-ddf0-4d42-a11b-90931933cb91.mov

Hello @MegatronKing. Your code sample provided has some errors. Can you fix them please?

@exaby73 I didn't see any error, can you provide me a screenshot?

Triage report
I can reproduce this issue. I fixed the error myself. Below is a simplified code sample.

Code Sample
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(primarySwatch: Colors.blue),
      home: const MyHomePage(),
    );
  }
}

class MyHomePage extends StatelessWidget {
  const MyHomePage({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: const Text('Demo page'),
      ),
      body: IndexedStack(
        index: 0,
        children: [
          Container(
            alignment: Alignment.centerLeft,
            child: InkWell(
              focusColor: Colors.red,
              onTap: () {
                print("Button 1 clicked");
              },
              child: const Text("Button 1"),
            ),
          ),
          Container(
            alignment: Alignment.centerRight,
            child: InkWell(
              focusColor: Colors.red,
              onTap: () {
                print("Button 2 clicked");
              },
              child: const Text("Button 2"),
            ),
          )
        ],
      ),
    );
  }
}

In a normal Stack, this is as designed, since the child might not take up the entire size of the widget, but in an IndexedStack, we should probably do something about that.
As a workaround, you can place the children inside of an ExcludeFocus and set it to exclude when it's not the selected index.


server/notifications: rewrite notification foundations

Simplify notification types, move all payloads, templates, etc to the same place
Remove deduplication key. If you want to send a notification, it should already be deduplicated
Support sending emails to anonymous pledgers
Very much a work in progress, pending new copy and design

@sultaniman Not at all! It was good to get some extra eyes on what’s happening, and it’s cool to see that you’re getting yourself this deep into Polar! ❤