Problems when cycling through tabs

Description
When switching between tabs using Ctrl + Tab, while it does show the new tab (tab 2) for the first time,  Files is still in the previous tab (tab 1).
If you happen to have more than 2 tabs only it also only works a single time before going back to tab 1.
Steps To Reproduce
Simply use Ctrl + Tab to cycle through tabs.
Expected behavior
That it should work like when cycling through tabs in a browser.
Files Version
<IP_ADDRESS>
Windows Version
21H2 19044.1526
Relevant Assets/Logs
debug_fulltrust.log
debug.log

Hi, I can confirm strange behavior of Ctrl + Tab keystroke.
Description

Tab no.1 + [Ctrl + Tab] = switched to Tab no.2 ✔
Tab no.2 + [Ctrl + Tab] = nothing happened, nor screen (folder content) changed ❌
Still Tab no.2 + [Ctrl + Tab] = screen changed to Tab no.1 ✔ ; folder path inputbox still showing path from Tab no.2 ❌ ;  Back and Forward buttons affecting Tab no.2 ❌

Files Version
Version: <IP_ADDRESS>
Windows Version
OS Version 10.0.19042.1466
Edition        Windows 10 Enterprise
Version       20H2
OS build     19042.1466
Relevant Assets/Logs
debug_fulltrust.log

Idk why but after commenting out:
<KeyboardAccelerator Key="Tab" Modifiers="Control">
    <i:Interaction.Behaviors>
        <icore:EventTriggerBehavior EventName="Invoked">
            <icore:InvokeCommandAction Command="{x:Bind ViewModel.NavigateToNumberedTabKeyboardAcceleratorCommand}" />
        </icore:EventTriggerBehavior>
    </i:Interaction.Behaviors>
</KeyboardAccelerator>

Ctrl+Tab still changes tabs (although not by MainPageViewModel.NavigateToNumberedTabKeyboardAccelerator() as it should normally) , even though this is the only place that defines Ctrl+Tab action.

Merging with #6111


Enhance description

Summary
#20236 Enhance description on Enhance the app

Closing this PR, as the changes here were superseded by #20240


fix: Use a writeable cache directory

Temporary workaround - see
https://github.com/aws/aws-cdk/issues/21867#issuecomment-1234233411.

Reference
Code review checklist

Can this be closed @l0b0 ?


addIndex and removeIndex do not behave as expected (MySQL)

I used addIndex to, I thought, setup a multi-column index:
$table->addIndex(array('dealership_id', 'dealership_name_id'), array('unique' => true))

Later I wanted to remove it, so I tried:
 $table->removeIndex(array('dealership_id', 'dealership_name_id'))

But because the dealership_id column had a foreign-key restraint it caused an error as it tried to delete the dealership_id index for the foreign key, not the multi-column index (I thought) I'd setup.
Didn't seem to be a way to get Phinx to remove this index.
Maybe I've misunderstood the syntax for addIndex and I wasn't setting up a multi-column index? Or maybe the removeIndex method has a bug.

+1 at this date, using Phinx 0.4.6

I've tried deleting a single-column index but Phinx deleted a multi-column index where the column was used instead. I think it's the same error.

I had the same issue with 0.4.6 (mysql), upgraded to 0.6.4 (which must have the changes from #671 in it) and Phinx removes the correct index on a rollback now.
Can this ticket be closed now?

I haven't tested it, but -- FWIW -- I'm also now on 0.6.4. I'm happy for the ticket to be closed.

I am still having the same issue, I am using v0.8.0

@zorji which db adapter are you using?

Closing due to lack of activity or out of scope. If there is new interest, please open a PR.


Add an MCB command to backfill Sites with the TTWA or London Borough

Context
Changes proposed in this pull request
Guidance to review
Checklist

[ ] Make sure all information from the Trello card is in here
[ ] Attach to Trello card
[ ] Rebased master
[ ] Cleaned commit history
[ ] Tested by running locally

Yea no worries. If possible it would be great if we could chat on Monday. I was going to do the rest of the work to get the bulk of this done today as it's blocking a couple of our cards.
I'll ping you tomorrow morning @gpeng

Discussed this offline with @davidgisbey We've agreed on adding TTWA and London Borough to v3 of the API for now but will explore moving Find (and other app) specific functionality into the apps themselves and out of the the API in the future as part of the work to deprecate and remove v3.

closing this for now as it needs more work. We've got the data we need on prod and it's been exposed on the api


docker cp with windows containers crash with  hcsshim::ActivateLayer failed in Win32

After a fresh installation of docker and pulling the container I cannot copy any file into the container
I get the following:
docker run -it microsoft/dotnet-framwork cmd
docker cp D:\x\1.txt <container_name>:c:\1.txt

Error response from daemon: hcsshim::ActivateLayer failed in Win32:
The process cannot access the file because it is being used by another process. (0x20) id=d1ad2f812e3b7a37da79538611d7fb4496b0a89369b2ba430e08a28d73f1ab50 flavour=1

Output of docker version:
Client:
 Version:      17.03.0-ce
 API version:  1.26
 Go version:   go1.7.5
 Git commit:   60ccb22
 Built:        Thu Feb 23 10:40:59 2017
 OS/Arch:      windows/amd64

Server:
 Version:      17.03.0-ce
 API version:  1.26 (minimum version 1.24)
 Go version:   go1.7.5
 Git commit:   60ccb22
 Built:        Thu Feb 23 10:40:59 2017
 OS/Arch:      windows/amd64
 Experimental: true

Output of docker info:
Containers: 5
 Running: 1
 Paused: 0
 Stopped: 4
Images: 2
Server Version: 17.03.0-ce
Storage Driver: windowsfilter
 Windows:
Logging Driver: json-file
Plugins:
 Volume: local
 Network: l2bridge l2tunnel nat null overlay transparent
Swarm: inactive
Default Isolation: hyperv
Kernel Version: 10.0 14393 (14393.206.amd64fre.rs1_release.160915-0644)
Operating System: Windows 10 Pro
OSType: windows
Architecture: x86_64
CPUs: 8
Total Memory: 31.91 GiB
Name: wk-ofir
ID: PKF6:LKFA:ROVD:X2LP:I2NL:HJZF:NSGN:BLOM:JDBH:RXA7:F3HQ:B3TK
Docker Root Dir: C:\ProgramData\Docker
Debug Mode (client): false
Debug Mode (server): true
 File Descriptors: -1
 Goroutines: 33
 System Time: 2017-03-07T15:19:34.6735797+02:00
 EventsListeners: 1
Registry: https://index.docker.io/v1/
Experimental: true
Insecure Registries:
 <IP_ADDRESS>/8
Live Restore Enabled: false

ping @jhowardmsft @johnstep PTAL

I see a similar error message in https://github.com/docker/docker/issues/29832, not sure if there's a relation

I will say that I found a way around it using a Dockerfile and a COPY command.

This is expected - online filesystem operations cannot be performed on a running Hyper-V container. They can on a Windows Server container. Windows 10 only supports Hyper-V containers.

@jhowardmsft Can we improve the error message, and return a message that it's not supported? The current error is not useful.

@jhowardmsft  I must agree with @thaJeztah, this error message is completely uninformative that lead me (and possibly others) on a pursuit for something the doesn't / cannot be fixed and wasted a few hours of work.

Yes, let me reopen; we should block this action if it's not supported by the platform

Just to clarify.  cp does NOT work on Windows 10?
Hmmm...  With a docker version from a few months ago I was able to.

It works if the container is not running.

Sweet thanks!!

Just to clarify - the running/online vs stopped/offline thing is a limitation of Hyper-V containers. Windows 10 only supports Hyper-V containers. Windows Server 2016 supports both Windows Server and Hyper-V containers, and defaults to Windows Server containers. Windows Server containers support both online and offline file-system operations.


Reevaluate try/catch in input formatters

Right now we have a try/catch around input formatters to allow them to throw and have us treat it as a model state error. I think we're possibly swallowing more errors than we should here, and turning them into 400s instead of 500s.
See https://github.com/aspnet/Home/issues/1605#issuecomment-229158767
The root cause of this issue was a package mismatch, but it became a model state error instead.

We think the best way to fix this would be a breaking change, so putting this in the 2.x milestone.

This failed to happen during 2.0 😢

Then maybe it's reasonable to just close this issue?

@Eilon how is it reasonable if this change has not happen/the issue was not resolved?

@another-guy this issue has not had much attention so it seems as a relatively low priority item, especially considering that it's a breaking change, so the next time we could fix this is in the 3.0 release (we're nearly shut down in the 2.0 release). I talked to @rynowak earlier and we think we might be able to incorporate this into some future "Better Model Binding" implementation.

@Eilon makes sense. Thank you for clarification!

@Eilon - if this is for 3.0 can we close this in favor of tracking it with https://github.com/aspnet/Mvc/issues/6549?

https://github.com/aspnet/Mvc/commit/83c3ac62fb9187c75689a26e9749046825cdd04b


test om issue - please delete

Just want to see if Gitbot integration is configured correctly and creating stories in the PPE Home Team icebox.

Looks good.  closing issue.


Create a leaflet map

Goals

Need to get familiar with leaflet and relevant packages like folium.
Need to get better understanding of backend needs for leaflet.

Acceptance criteria:
Create a basic map in leaflet that allows us to:

Zoom in/zoom out
Search for places on the map
Select a layer

Resolved by a lot of other issues and PRs already


react-dev-utils/FileSizeReporter fails when stats of multi-config of webpack is passed to printFileSizesAfterBuild

Hi,
I am using multi-config setup in my webpack.config like below.
module.exports = [
{
entry: 'path/to/indexjs',
output: 'build/folder'
},
{
entry: 'path/to/indexjs',
output: 'build/folder'
}
];
I am trying to print file sizes after build using printFileSizesAfterBuild by passing webpack stats.
It keeps failing because webpack send a different stats object for multi config.
This is the error I get -
TypeError: Cannot read property 'filter' of undefined
at printFileSizesAfterBuild (/projects/chainwalker-ui/node_modules/react-dev-utils/FileSizeReporter.js:32:12)
at measureFileSizesBeforeBuild.then.then (/projects/chainwalker-ui/devscripts/build.js:69:7)
Thanks,
Mat

Hi! We don't have enough time to support ejected setups. If you'd like to send us a PR fixing this, feel free.
Thanks!

@matheenpasha Have you found a solution to this? I am facing the same issue.

@juanparadox  I am working on this. Will post a PR soon.


The project is dead or archived?

Hi,
I wanted to make sure whether this project is dead or not?

Hasn't been updated in 6 months.
Maven repo does not even contain the latest version (and there's no ETA on it).

Is this going to be maintained or is this something that we should not use?

Hi, I am also interested in this topic and in case the project is dead do anybody knows ay similar alternative to this one?

Hi. I`m already looking for driver for Spark 3.5? Any ETA?

Is this going to be maintained or is this something that we should not use?

Looks like something that we should not use anymore. No updates for 6+ months, no plans, no response in issues, no anything. Fair to assume this project is abandoned and dead.

Microsoft is working on Fabric.  In the meantime you are welcome to add your feedback in their ideas portal:
https://ideas.fabric.microsoft.com/


[Doc] Update version from 0.4.0 to 0.5.0 on remaining kuberay docs files

Some files (ray service doc, kuberay operator doc, kind doc) still were using 0.4.0 in their examples. This PR updates them to 0.5.0.
Why are these changes needed?

Related issue number

Checks

[ ] I've made sure the tests are passing.
Testing Strategy

[ ] Unit tests
[ ] Manual tests
[ ] This PR is not tested :(

Maybe you can use 0.4.0, v0.4.0, and 0.4 in your VSCode search to make sure all versions are replaced?

Maybe you can use 0.4.0, v0.4.0, and 0.4 in your VSCode search to make sure all versions are replaced?

Yup that's what I did, the rest of them did not make sense to change (e.g. it was referring specifically to 0.4.0 not 0.5.0, or it was using 0.4.0 as an example in some release process doc)

There seem to be some documents that need to be updated. For example,

https://github.com/ray-project/kuberay/blob/master/clients/python-client/python_client/__init__.py
https://github.com/ray-project/kuberay/tree/master/helm-chart/kuberay-apiserver

Do I miss anything?

Ah I wasn't sure if I was supposed to update those.  I've updated them now, that should be everything

Updated the rest of the files.


[Modules] storage account module : conditional nfsv3 parameter

Azure Storage account : Only the NFSv3 parameter when set to true as this can only be added during storage account creation

Thank you for your contribution !

Please include a summary of the change and which issue is fixed.
Please also include the context.
List any dependencies that are required for this change.

Pipeline references

For module/pipeline changes, please create and attach the status badge of your successful run.

Pipeline

Type of Change
Please delete options that are not relevant.

[ ] Bugfix (non-breaking change which fixes an issue)

Checklist

[x] I'm sure there are no other open Pull Requests for the same update/change
[ ] My corresponding pipelines / checks run clean and green without any errors or warnings
[x] My code follows the style guidelines of this project
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation (readme)
[x] I did format my code

tested this
and with an exiting storage account with nfsv3 disabled this continues to work
also a new storage account deployment works

@robbertwortell I've done initial investigations on this, and would like to add the following:
I've created a small ARM template with the following:
Template
{
    "$schema": "http://schema.management.azure.com/schemas/2015-01-01/deploymentTemplate.json#",
    "contentVersion": "<IP_ADDRESS>",
    "parameters": {
        "location": {
            "type": "String"
        },
        "storageAccountName": {
            "type": "String"
        },
        "accountType": {
            "type": "String"
        },
        "kind": {
            "type": "String"
        },
        "accessTier": {
            "type": "String"
        },
        "publicNetworkAccess": {
            "type": "String"
        },
        "networkAclsBypass": {
            "type": "String"
        },
        "networkAclsDefaultAction": {
            "type": "String"
        },
        "isHnsEnabled": {
            "type": "Bool"
        },
        "isNfsV3Enabled": {
            "type": "Bool"
        },
        "isSftpEnabled": {
            "type": "Bool"
        }
    },
    "variables": {},
    "resources": [
        {
            "type": "Microsoft.Storage/storageAccounts",
            "apiVersion": "2022-05-01",
            "name": "[parameters('storageAccountName')]",
            "location": "[parameters('location')]",
            "dependsOn": [],
            "tags": {},
            "sku": {
                "name": "[parameters('accountType')]"
            },
            "kind": "[parameters('kind')]",
            "properties": {
                "accessTier": "[parameters('accessTier')]",
                "publicNetworkAccess": "[parameters('publicNetworkAccess')]",
                "networkAcls": {
                    "bypass": "[parameters('networkAclsBypass')]",
                    "defaultAction": "[parameters('networkAclsDefaultAction')]",
                    "ipRules": []
                },
                "isHnsEnabled": "[parameters('isHnsEnabled')]",
                "isNfsV3Enabled": "[parameters('isNfsV3Enabled')]",
                "isSftpEnabled": "[parameters('isSftpEnabled')]"
            }
        }
    ],
    "outputs": {}
}

Parameters
{
    "$schema": "https://schema.management.azure.com/schemas/2015-01-01/deploymentParameters.json#",
    "contentVersion": "<IP_ADDRESS>",
    "parameters": {
        "location": {
            "value": "westeurope"
        },
        "storageAccountName": {
            "value": "teststoragecarmls"
        },
        "accountType": {
            "value": "Standard_LRS"
        },
        "kind": {
            "value": "StorageV2"
        },
        "accessTier": {
            "value": "Hot"
        },
        "publicNetworkAccess": {
            "value": "Enabled"
        },
        "networkAclsBypass": {
            "value": "AzureServices"
        },
        "networkAclsDefaultAction": {
            "value": "Deny"
        },
        "isHnsEnabled": {
            "value": true
        },
        "isNfsV3Enabled": {
            "value": true
        },
        "isSftpEnabled": {
            "value": false
        }
    }
}

I can deploy this template over and over without issues, meaning that the isNfsV3Enabled property is not impacting the template deployment.
Is there something in our module that may cause this issue? Can you please help share more context on the error you are getting please?

@ahmadabdalla
The issue seems to be with an existing storage account created without sending the NFSv3 parameter.
And then trying to send the NFSv3 paramter (either true or false)
so the path is :
scenario 1:
tag 0.9:
SA is created without param NFSv3 (if you export you don't see this)
scenario 2:
tag 0.10:
SA will be created with NFSv3 (true or false)
scenario 3:
SA created with 0.9 and there after the modules are upgraded from 0.9>0.10
this fails as MS does not allow to add this param.
Current solotion is todo an union, to only add this parameter condional,
But maybe the same solution as this following example could also work for NFSv3?
isHnsEnabled: enableHierarchicalNamespace ? enableHierarchicalNamespace : null
so:
isNFSv3Enabled: isNfsV3Enabled? isNfsV3Enabled: null

OK now I see where this is happening and my updated testing criteria
Test Scenarios
v1
I've used the same template above but with NO NFS properties, and then stored it as v1 template spec.
v2
Updated that template to include NFS properties like our CARML pattern to set it to false if left default, then stored it as v2 template spec.
    isHnsEnabled: enableHierarchicalNamespace ? enableHierarchicalNamespace : null
    isSftpEnabled: enableSftp
    isNfsV3Enabled: enableNfsV3

v3
Updated that template to now set the NFS properties as null if not set to true then stored it as v3:
    isHnsEnabled: enableHierarchicalNamespace ? enableHierarchicalNamespace : null
    isSftpEnabled: enableSftp ? enableSftp : null
    isNfsV3Enabled: enableNfsV3 ? enableNfsV3 : null

Observations:

Deploying a storage account using V1 is SUCCESS. Then querying the Storage account using REST, the NFS properties are not even returned.
Deploying the same storage account again using V2 will FAIL the deployment.

{
      "code": "AccountPropertyCannotBeUpdated",
      "message": "The property 'isNfsV3Enabled' was specified in the input, but it cannot be updated as it is read-only. For more information, see - https://aka.ms/storageaccountupdate"
}

Deploying the same storage account where NFS properties are default to false but are now set to null as per the template logic is SUCCESS

This indicates we do not need to perform the union function to maintain backwards compatibility between 0.9 and 0.10.
What are your thoughts here?

@ahmadabdalla  i assumed (wrongly) that your V3 didn't work as the original was added differently then all the others.
But i never tested this.
But for sure V3 would be the best. (uses the same logic and layout as the rest and is most readable.
Ill update the code

@robbertwortell how did you go with the tests?

@robbertwortell there are new breaking changes to the CARML CI and module folder structure that has recently been merged. Can you please sync from main and re-apply your changes please. If you'd like me to support you please let me know :) :)

@ahmadabdalla i moved to a personal account
this pull request can be ignored (will remove later and checkout the new PR)
https://github.com/Azure/ResourceModules/pull/3290


Simple fix for stdio: 'inherit'

Hi mhluska. I am having an issue with stdio: 'inherit' causing "Warning: Implement me. Unknown stdin file type!"
Does your fix solve this?

@CThuleHansen I don't think so. I don't remember fixing that error. Though it was a long time ago so I may have forgotten.


For loop syntax enhancements

Two highly related enhancement ideas to for loops:

Don't require a colon as a marker like :FOR but instead require both FOR and IN to be all upper case.

Support the pipe character as a block marker instead of a backslash.

Together these enhancements would allow writing current
*** Keywords ***
Example
    :FOR    ${x}    IN    @{STUFF}
    \    Keyword    ${x}
    \    Another keyword

like
*** Keywords ***
Example
    FOR    ${x}    IN    @{STUFF}
    |    Keyword    ${x}
    |    Another keyword

This is especially nice if we later add support for other programmatic structures like IF/ELSE. Compare
*** Keywords ***
Example
    :FOR    ${x}    IN    @{STUFF}
    \    Keyword    ${x}
    \    :IF    ${x} > 42
    \    \    Another keyword
    \    \    Yet another keyword
    \    :ELSE
    \    \    Keyword here too
    \    One keyword more    ${x}

with
*** Keywords ***
Example
    FOR    ${x}    IN    @{STUFF}
    |    Keyword    ${x}
    |    IF    ${x} > 42
    |    |    Another keyword
    |    |    Yet another keyword
    |    ELSE
    |    |    Keyword here too
    |    One keyword more    ${x}

Is it will be only supported in TSV and TXT/ROBOT with no | pipe separated format?

You don't get this same problem when using the pipe separate format as you can just use
| FOR | ${x}            | IN   | @{STUFF} |
|     | Keyword         | ${x} |          |
|     | Another keyword |      |          |

and don't need such escaping at all.

Notice that at the moment this issue is just in idea, not a concrete proposal. Removal of : from :FOR sounds like a good idea, but how to handle blocks when using the normal plain text format (i.e. not the pipe separated format) is still totally open.
One possibility would be using significant whitespace like in Python:
*** Keywords ***
Example
    FOR    ${x}    IN    @{STUFF}
        Keyword    ${x}
        IF    ${x} > 42
            Another keyword
            Yet another keyword
        ELSE
            Keyword here too
        One keyword more    ${x}

but that's likely to cause backwards incompatibility problems.
Another solution would be closing blocks with END like in Ruby:
*** Keywords ***
Example
    FOR    ${x}    IN    @{STUFF}
        Keyword    ${x}
        IF    ${x} > 42
            Another keyword
            Yet another keyword
        ELSE
            Keyword here too
        END
        One keyword more    ${x}
    END

Thanks @pekkaklarck for clarification, so this syntax is only for not pipe separated format.

I'm really liking the END idea more than pipe separation.

#2990 covers using just FOR (case-sensitive) as a marker and #2991 deprecates using IN, IN RANGE, and friends case- and space-insensitively. Both of them are targeted for RF 3.1.
How to handle blocks is still open, but the pipe idea doesn't sound too good anymore. A separate and more concrete issue will be submitted about that later.


Miri reports Race condition on Collector Test

Miri currently reports a race condition in the Collector test. I narrowed it down to for _ in 0..100 { collector.collect() } which races with the threads created before that. I have a branch where i changed two Atomic Accesses and miri didn't complain anymore, but i honestly know too little about the linked list used to determine if that really fixes it or  just doesn't trigger in that test anymore.

I orginally wanted to build T: ?Sized Support for the Library and just got sidetracked. That PR sadly isn't going happen, as AtomicPtr can't deal with T: ?Sized, due to platform limitations and i can't see a way around that. This (as far as i can see) also makes #3 completely impossible, which i hoped would be able to happen soon (with the Rust for Linux push for custom Smart Pointers).

Miri Output:

Finished test profile [unoptimized + debuginfo] target(s) in 0.00s
Running unittests src/lib.rs (target/miri/aarch64-apple-darwin/debug/deps/basedrop-43f3ef052c3bee08)
running 1 test
test collector::tests::collector ... error: Undefined Behavior: Data race detected between (1) non-atomic write on thread unnamed-97 and (2) atomic store on thread collector::tests::collector at alloc1585770. (2) just happened here
--> src/collector.rs:281:21
|
281 |                     (*tail).link.next.store(head, Ordering::Relaxed);
|                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Data race detected between (1) non-atomic write on thread unnamed-97 and (2) atomic store on thread collector::tests::collector at alloc1585770. (2) just happened here
|
help: and (1) occurred earlier here
--> src/collector.rs:99:9
|
99  |         (*node).header.link.next = ManuallyDrop::new(AtomicPtr::new(core::ptr::null_mut()));
|         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
= help: this indicates a bug in the program: it performed an invalid operation, and caused Undefined Behavior
= help: see https://doc.rust-lang.org/nightly/reference/behavior-considered-undefined.html for further information
= note: BACKTRACE (of the first span) on thread collector::tests::collector:
= note: inside collector::Collector::collect_one at src/collector.rs:281:21: 281:69
note: inside collector::Collector::collect
--> src/collector.rs:238:15
|
238 |         while self.collect_one() {}
|               ^^^^^^^^^^^^^^^^^^
note: inside collector::tests::collector
--> src/collector.rs:391:13
|
391 |             collector.collect();
|             ^^^^^^^^^^^^^^^^^^^
note: inside closure
--> src/collector.rs:362:19
|
361 |     #[test]
|     ------- in this procedural macro expansion
362 |     fn collector() {
|                   ^
= note: this error originates in the attribute macro test (in Nightly builds, run with -Z macro-backtrace for more info)
note: some details are omitted, run with MIRIFLAGS=-Zmiri-backtrace=full for a verbose backtrace
error: aborting due to 1 previous error
error: test failed, to rerun pass --lib
Caused by:
process didn't exit successfully: /Users/lucasbaumann/.rustup/toolchains/nightly-aarch64-apple-darwin/bin/cargo-miri runner /Users/lucasbaumann/Documents/GitHub/basedrop/target/miri/aarch64-apple-darwin/debug/deps/basedrop-43f3ef052c3bee08 collector (exit status: 1)
note: test exited abnormally; to see the full output pass --nocapture to the harness.

Interesting, thanks for reporting this! I'll look into it.


how can i get the Composition percent of gm

within GLV? or the tokens that make up a given GM?

The tokens that make up a given GM

https://github.com/snipermonke01/gmx_python_sdk/blob/main/gmx_python_sdk/scripts/v2/get/get_pool_tvl.py
i believe this script is what you are looking for


fix for error with matrix input (also issue #19)

Testing if logcounts in assayNames (line 282) throws error if input is not SpatialExperiment. Separated ifs in line 282 to test if SpatialExperiment before it tests if logcounts in assayNames.

Thanks for this fix, and apologies for the delay. Yes, looks like this was a bug due to combining the two if statements on line 282, and the fix looks great. I'll merge this and add a version bump, and push to Bioc.

On further reflection I have updated this to solve it in a slightly simpler way by using && instead of & inside the if statement on line 282: if (is(input, "SpatialExperiment") && ("logcounts" %in% assayNames(spe))) {
This also solves the issue since && immediately stops / returns FALSE if the first condition is FALSE, vs. & which keeps going and only returns FALSE at the end (and hence returns an error if spe does not exist or assayNames(spe) cannot be found).
I think this option is slightly safer than separating the if statements, since it avoids potential complications if is(input, "SpatialExperiment") is TRUE and ("logcounts" %in% assayNames(spe)) is FALSE.
Thanks again for noticing this and sending the pull request!

Ok this is now merged, checked, and pushed to Bioc (both devel and RELEASE_3_18 branches). The updated versions should be installable from Bioc in 1-2 days after going through the Bioc build system, and are also available here from GitHub. Thank you!


Update VIPER.podspec, lower deployment target

This set of protocols don't really seem to require target ios 11+. Built with ios 9, works fine

yeah, why not, thx for the contribution.  😄


Users should know who eachother is

People are complaining. When they get invitations they do not know who the user is. This is because usernames can be anything "kittydog" is not recognizable.
So we should make the following changes:

Real name fields. First / last name as part of profile. These are optional fields.

When you register, you can fill out these fields. There should be a note "Entering your real name will help other people identify you if you invite them to be friends."

When you look at a user's profile you see these fields.

When a user sends you an invitation, you will see (a) their avatar and (b) their real name.

If the user has a real name, the title of the email will change to "real name" has invited you, rather than "username" has invited you.

@falconandy @dimakrsna here you go.

So @dimakrsna and @falconandy please show full names (when available) INSTEAD of usernames for:

Messages
Comments
Profiles
Notifications
Emails (invites etc)


docs: move cloud support to integrations

Signed-off-by: Sami Jaghouar<EMAIL_ADDRESS>Context
Moving Cloud support part of documentation to Integration

Codecov Report
Base: 88.13% // Head: 50.39% // Decreases project coverage by -37.74% :warning:

Coverage data is based on head (26d075a) compared to base (97374ec).
Patch has no changes to coverable lines.

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #798       +/-   ##
===========================================
- Coverage   88.13%   50.39%   -37.75%
===========================================
  Files         133      133
  Lines        6703     6703
===========================================
- Hits         5908     3378     -2530
- Misses        795     3325     +2530

Flag
Coverage Δ

docarray
50.39% <ø> (-37.75%)
:arrow_down:

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

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

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

lets wait for the documentation to be generated


Empty Elements cannot be loaded in table

Steps to reproduce
I'm trying to migrate the notebook gallery's Reddit notebook. It polls data from Reddit and stores in a dataframe.

With v0.6, I have cast the columns with data type object to string:
df[
    [
        "id",
        "author",
        "title",
        "post",
        "lemmatized post",
        "lemmatized title",
        "created time",
        "subreddit",
    ]
] = df[
    [
        "id",
        "author",
        "title",
        "post",
        "lemmatized post",
        "lemmatized title",
        "created time",
        "subreddit",
    ]
].astype(
    str
)

df["created date"] = pd.to_datetime(df["created date"])

I was able to create the table after that.
However, when I try to start the cube, I encountered the exception:
AtotiJavaException: State transition INIT => STARTED failed because of an internal exception.

If I store the panda df into csv, I'm able to create the cube normally.
main.zip
Environment

atoti: 0.6.0
Python: 3.9.6
Operating System: win32

Logs (if relevant)
server.log

I fixed it by casting the strings:
types={"lemmatized post": tt.type.NULLABLE_STRING, "post": tt.type.NULLABLE_STRING}

Although I can create the cube after setting the columns to nullable, they don't get created into hierarchies.
Trying to manually create the hierarchy will end up with the same exception:

Caused by: java.lang.IllegalArgumentException: The name of a member cannot be empty(level post, hierarchy post, dimension Reddit_Posts)

The problem seems to be that empty values are not being replaced with the default value when reading the parquet file.
You can work around this by replacing empty strings in the problematic columns with a different value, like "N/A". By doing so I was able to start the cube without issue. df["post"] = df["post"].replace({"": "N/A"}).
I'm looking into this to see how to make atoti do this automatically for non nullable columns.

I'm closing this as I cannot reproduce with the following notebook and atoti 0.6.4.
empty values.zip

Hi, @bloussou
I think this should be reopened.
From the original issue https://github.com/atoti/atoti/issues/324#issue-953482195, @HuifangYeo notes this occurs when creating the cube (quoted below)

I was able to create the table after that.
However, when I try to start the cube, I encountered the exception:

The attached example in empty values.zip provided in https://github.com/atoti/atoti/issues/324#issuecomment-1023217620 goes as far as table creation, but doesn’t create the cube. Adding the line
cube = tt.create_cube(table)
will reproduce the error originally reported... or at least, did for me :)
reproduction.zip

Hello,
Indeed the issue is present in atoti 0.6.5 but will be fixed in the next release:

A minimum reproducible example:
import pandas as pd
import atoti as tt

dataframe = pd.DataFrame({"id": [" ", "text"]})

dataframe["id"] =  dataframe["id"].apply(lambda x: x.replace(" ", ""))

session = tt.Session()

table = session.read_pandas(dataframe, table_name="Empty")

cube = session.create_cube(table)


Files got deleted

Type: Bug
Hello. Today I started VS Code and a lot of files got deleted out of nowhere including lyberies and packages like react, axios and other. My Code is completely destroyed and I didn't deleted those files. I simply started VS Code and it was gone. I found 2-4 files in the bin but the rest is just not there
VS Code version: Code 1.74.2 (e8a3071ea4344d9d48ef8a4df2c097372b0c5161, 2022-12-20T10:29:14.590Z)
OS version: Windows_NT x64 10.0.22000
Modes:
Sandboxed: No

System Info

Item
Value

CPUs
11th Gen Intel(R) Core(TM) i7-1165G7 @ 2.80GHz (8 x 2803)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
undefined

Memory (System)
15.70GB (6.26GB free)

Process Argv
--crash-reporter-id 16198857-d85d-4ddf-88e8-2cde0ccf04b0

Screen Reader
no

VM
0%

Extensions (4)

Extension
Author (truncated)
Version

code-runner
for
0.11.8

LiveServer
rit
5.7.9

open-in-browser
tec
2.0.0

vscode-pets
ton
1.21.0

(1 theme extensions excluded)

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vsdfh931:30280409
vshan820:30294714
vstes263:30335439
vscorecescf:30445987
pythondataviewer:30285071
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593:30376534
pythonvs932:30410667
cppdebug:30492333
vsclangdc:30486549
c4g48928:30535728
dsvsc012cf:30540253
azure-dev_surveyone:30548225
vsccc:30610678
pyindex848cf:30577861
nodejswelcome1:30587005
3biah626:30602489
gswce1:30612156
iaj6b796:30613358
89544117:30613380
fim-prod:30623723

Do you have any reproducible steps? There hasn't been an update to VS Code or anything like that

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


WIP: v0.1.1, LAPACK, and Windows support

Let's see how this works...

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


MemoryError on computing checksums for large files

When creating a command which depends on a large file (which cannot be fitted into memory), weasel still tries to load the whole file which results in a MemoryError.
The traceback for such a run:

  File "/home/gorosz/workspace/ascend/invoice-ai-models/.venv/bin/weasel", line 8, in <module>                                                                                sys.exit(app())

  File "/home/gorosz/workspace/ascend/invoice-ai-models/.venv/lib/python3.10/site-packages/weasel/cli/run.py", line 42, in project_run_cli                                                                                project_run(

  File "/home/gorosz/workspace/ascend/invoice-ai-models/.venv/lib/python3.10/site-packages/weasel/cli/run.py", line 88, in project_run
    project_run(
  File "/home/gorosz/workspace/ascend/invoice-ai-models/.venv/lib/python3.10/site-packages/weasel/cli/run.py", line 113, in project_run
    update_lockfile(current_dir, cmd)
  File "/home/gorosz/workspace/ascend/invoice-ai-models/.venv/lib/python3.10/site-packages/weasel/cli/run.py", line 270, in update_lockfile
    data[command["name"]] = get_lock_entry(project_dir, command)
                                                                                File "/home/gorosz/workspace/ascend/invoice-ai-models/.venv/lib/python3.10/site-packages/weasel/cli/run.py", line 286, in get_lock_entry
    deps = get_fileinfo(project_dir, command.get("deps", []))

  File "/home/gorosz/workspace/ascend/invoice-ai-models/.venv/lib/python3.10/site-packages/weasel/cli/run.py", line 308, in get_fileinfo
    md5 = get_checksum(file_path) if file_path.exists() else None

  File "/home/gorosz/workspace/ascend/invoice-ai-models/.venv/lib/python3.10/site-packages/weasel/util/hashing.py", line 33, in get_checksum
    return hashlib.md5(Path(path).read_bytes()).hexdigest()

  File "/home/gorosz/Applications/miniconda3/lib/python3.10/pathlib.py", line 1127, in read_bytes
    return f.read()

MemoryError

This happens because Weasel needs to check the dependencies of a command and whether they've changed or not. To prevent this from happening, the large file should simply not be listed as output or input to a given command.

Thanks. I just discovered this workaround for myself as well. Do you think it's feasible to use the last modification date instead of hashes? Alternatively, would it be a solution to compute hashes for file chunks to address this issue (refer to https://stackoverflow.com/questions/1131220/get-the-md5-hash-of-big-files-in-python)?


New ecms tutorials

This branch contains the EC-MS tutorial currently published on the ixdat documentation page (i.e. relying only on ixdat functions). Can be merged ASAP.

Made corrections and responded to comments.
I generally think it's a good idea to split the tutorial in three (or more?) parts, however, as discussed elsewhere, I propose following steps on how to continue with this tutorial:
1)Merge as soon as the small issues are fixed to get a functioning version online ASAP.
2)Separate the tutorial into several smaller ones on "importing", "calibration" and "plotting" as suggested, with two different tutorials for calibration - one with ixdat and one with siq
3)Once the new way of doing native ixdat calibrations is merged into main, update the calibration part in the tutorial.


Maybe adjust install instructions

The current git page suggests starting using the command "pip install fontmake" This failed for me because it didn't set up the python support or the virtual environment that seems to be the preferred method of using fontmake as of now.
I was fortunate to get some help from Stephen Nixon and this is what worked for me instead. Some of this is just checking and isn't essential but I include it here especially to help people like me that are still a bit new to terminal commands etc.
Note: If I get some of the explanations mixed up I hope Stephen will help me correct it! :

First we moved the place the terminal pointed to to my user dir using a series of cd .. commands
Then we used the command mkdir virtualenvs Note: this is just a dir to put instances of virtual environments in - hence the plural name.
Then ls so we could see that the dir was there
Then into the new folder or dir cd virtualenvs
Then we checked the version of virtualenv I had installed already virtualenv --version
Then we installed the python we wanted virtualenv -p python3
Then we made new environment to test that we could make new standalone project environments  virtualenv -p python3 TestEnv
We checked that the new dir was made ls
We then went into that dir cd TestEnv
Checked contents of the dir  ls
Opened a finder window showing the contents open .
Moved back out to the "virtualenvs" directory cd ..
Entered and Activated the virtual environment TestEnv/bin/activate
The terminal now lists the bash with the text (TestEnv) to start indicating we are in and using this Virtual Environment
Installed fonttools pip install fonttools
The success is reported "Successfully installed fonttools" ( Normally silence or no message indicates success )
We used TTX to convert a TTF file which was done by typing ttx a space and then dragging the file to the terminal to indicate the path to the file. This happened to look like this afterwards` ttx /Users/ebensorkin/Library/Fonts/Merriweather-Regular.ttf
With that done we deactivated the virtual env deactivate
Then we made a new Virtual environment meant to be used specifically for the Merriweather Serif project and to remind me that you can do this. virtualenv -p python3 TestMerriweather
Then we activated the new virtual env TestMerriweather/bin/activate
Then we installed fontmake pip install fontmake
A great deal of activity followed including Installing collected packages: lxml, appdirs, pytz, six, fs, fonttools, defcon, glyphsLib, compreffor, pyclipper, booleanOperations, cu2qu, ufo2ft, fontMath, MutatorMath, fontmake after which we listed the commands for this fontmake --help
Then we tested making a Variable Font with Fontmake from a Glyphs file source fontmake -o variable -g

There were problems we found and we fixed some of them today but this post is about how to get running not the problems of my Glyphs file!
I suspect that having a contemporary copy of virtualenv and pip running are probably key. I am not able to advise about this and hope Stephen or others will chime in on this subject!

Glad you got the VF building! A couple of small edits:
Step 6 is unnecessary / won't do anything on its own.
13 and 18 need source before the path, like source nameOfVenv/bin/activate.
Generally, I've moved toward having a virtual environment inside each project folder, called 'venv,' and including this in the .gitignore file. This is a bit more to manage, but seems to be the best practice, as it does a better job keeping dependencies exactly as each given project needs to successfully run builds, checks, etc. I do keep a high-level venv for easy access to ttx when I need it, though.
One very useful addition to this workflow is using bash aliases to give yourself shortcuts to activate different venvs.

Do you know who maintains the install instructions? In is Nicholas W?
-e.

Eben, this might be an approach that avoids the need for virtual environments: https://github.com/googlei18n/fontmake/issues/284#issuecomment-418338172
Cosimo has been adding these builds to the repository releases for the last several months and they only require a Py3.6 or Py3.7 interpreter install (no dependencies, no venv's).  Download by platform and your installed Python interpreter version under the Assets link in any release.

Should we look into something like https://github.com/chriskiehl/Gooey?

How about https://github.com/googlei18n/fontmake/blob/793f1febaa4e8af04322c3cd5db9a5cb14212ec8/README.rst?


How do you handle multiple media drives?

Nice project. Thanks for sharing it.
I have a few of these running in separate containers but as I'd like to add a few of the *arr services I'm considering replacing all of them with your setup here.
The one point I'm a bit unsure of is how you handle multiple media disks? At the moment I have containers running on a small ssd and then multiple hdds which store the media. I assume most people with home servers have a similar setup. So unless you have one enourmous ssd I'm wondering:

Do container performance not suffer on a hdd?
How do you add and structure other disks for media storage?

The current setup I have looks something like this:
Datadrive/docker/containers
Mediadrive1/film
Mediadrive2/tv
Mediadrive3/film
Mediadrive3/music
Since the current containers do not operate with one another all I had to do was include their drive paths in the dockercompose. With your setup I assume this would not work. Would you mind sharing an example of how I could handle this?
Thanks in advance.

Note: this is a lot more involved than what I thought when I started typing this answer so please forgive any inconsistencies :)
Thanks for your interest! First off, regarding your questions:

In principle, container performance will not be impacted by what you mount to it. Of course, the access speed to the media will be slower than if you had put that on your SSD, but like you said, that's not feasible.
I assume you mean how you'd do this in this specific project, as you seem to understand how mounting drives to the host system works, so let's get into that.

This is a neat feature. In the future, I'd like to implement a CLI which could support multiple drives, but doing it now indeed requires some manual labour. What I would advise doing, for now, is removing the variables from docker-compose.yml and setting them to the specific paths you'd like. I think it should work with the torrent client as well. Using only Sonarr and Radarr, that could look something like this:
---
version: "3.1"
services:
  jellyfin:
    image: lscr.io/linuxserver/jellyfin:latest
    container_name: jellyfin
    environment:
      - PUID=${UID}
      - PGID=13000
      - UMASK=002
      - TZ=${TIMEZONE}
    volumes:
      - Datadrive/docker/jellyfin-config:/config
      - Mediadrive1/film:/media/film
      - Mediadrive2/tv:/media/tv
    ports:
      - 8096:8096
    restart: unless-stopped

  radarr:
    image: lscr.io/linuxserver/radarr:latest
    container_name: radarr
    environment:
      - PUID=13002
      - PGID=13000
      - UMASK=002
      - TZ=${TIMEZONE}
    volumes:
      - Datadrive/docker/radarr-config:/config
      - Mediadrive1/film:/data
    ports:
      - 7878:7878
    restart: unless-stopped

  sonarr:
    image: lscr.io/linuxserver/sonarr:latest
    container_name: sonarr
    environment:
      - PUID=13001
      - PGID=13000
      - UMASK=002
      - TZ=${TIMEZONE}
    volumes:
      - Datadrive/docker/sonarr-config:/config
      - Mediadrive2/tv:/data
    ports:
      - 8989:8989
    restart: unless-stopped

  qbittorrent:
    image: lscr.io/linuxserver/qbittorrent:latest
    container_name: qbittorrent
    environment:
      - PUID=13007
      - PGID=13000
      - UMASK=002
      - TZ=${TIMEZONE}
      - WEBUI_PORT=8080
    volumes:
      - Datadrive/docker/qbittorrent-config:/config
      - Datadrive/data/torrents:/data/torrents
    ports:
      - 8080:8080
      - 6881:6881
      - 6881:6881/udp
    restart: unless-stopped

Now. I think that should work and that it's not breaking anything that they don't share the same root directory, although it does look less neat. Along with this docker-compose comes a long list of permission hassle, but most of it is comparable to what is already in setup.sh
Depending on OS, you could also choose to mount a certain drive in a directory of your choosing. This might be the simplest way with the least modification to this repo. For example, mounting Media1 as movies in data/media/.
I'm not sure how ANY of this works in practice, I haven't tested it, but those are my thoughts :) Let me know what you discover!

Thanks a lot for the detailed response. Think I'll be able to sort it out now.


Fixed typo (funciton)

Docs Build status updates of commit cbf00bf:
:warning: Validation status: warnings

File
Status
Preview URL
Details

cordovadocs/2015/samples/cordova-samples.md
:warning:Warning

Details

cordovadocs/2015/first-steps/ios-guide.md
:warning:Warning

Details

cordovadocs/2015/known-issues/known-issues-general.md
:warning:Warning

Details

cordovadocs/2015/first-steps/get-started-with-ionic2.md
:warning:Warning

Details

cordovadocs/2015/access-device-capabilities/manage-plugins.md
:warning:Warning

Details

cordovadocs/2017/first-steps/ios-guide.md
:warning:Warning

Details

cordovadocs/2017/tips-workarounds/measuring-performance.md
:white_check_mark:Succeeded

cordovadocs/2015/samples/cordova-samples.md

Line 14: [Warning] Illegal link: `<a href="#examples-we-like/">Examples We Like</a>` -- missing bookmark. The file 2015/samples/cordova-samples.md doesn't contain a bookmark named examples-we-like/.

cordovadocs/2015/first-steps/ios-guide.md

Line 125: [Warning] Illegal link: `<a href="#stack">Are you experiencing a symptom that does not appear in this list?</a>` -- missing bookmark. The file 2015/first-steps/ios-guide.md doesn't contain a bookmark named stack.
Line 157: [Warning] Illegal link: `<a href="#cli">Did you receive any other type of build error</a>` -- missing bookmark. The file 2015/first-steps/ios-guide.md doesn't contain a bookmark named cli.
Line 159: [Warning] Illegal link: `<a href="#stack">Are you experiencing a symptom that does not appear in this list?</a>` -- missing bookmark. The file 2015/first-steps/ios-guide.md doesn't contain a bookmark named stack.
Line 165: [Warning] Illegal link: `<a href="#team-profile">Option 3: Use your team's provisioning profile</a>` -- missing bookmark. The file 2015/first-steps/ios-guide.md doesn't contain a bookmark named team-profile.
Line 477: [Warning] Illegal link: `<a href="#no-provision">You've been told that no provisioning profiles were found</a>` -- missing bookmark. The file 2015/first-steps/ios-guide.md doesn't contain a bookmark named no-provision.
Line 487: [Warning] Illegal link: `<a href="#cli">Did you receive any other type of build error</a>` -- missing bookmark. The file 2015/first-steps/ios-guide.md doesn't contain a bookmark named cli.
Line 489: [Warning] Illegal link: `<a href="#stack">Are you experiencing a symptom that does not appear in this list?</a>` -- missing bookmark. The file 2015/first-steps/ios-guide.md doesn't contain a bookmark named stack.
Line 574: [Warning] Illegal link: `<a href="#no-provision">You've been told that no provisioning profiles were found</a>` -- missing bookmark. The file 2015/first-steps/ios-guide.md doesn't contain a bookmark named no-provision.
Line 576: [Warning] Illegal link: `<a href="#cli">Did you receive any other type of build error</a>` -- missing bookmark. The file 2015/first-steps/ios-guide.md doesn't contain a bookmark named cli.
Line 584: [Warning] Illegal link: `<a href="#stack">Are you experiencing a symptom that does not appear in this list?</a>` -- missing bookmark. The file 2015/first-steps/ios-guide.md doesn't contain a bookmark named stack.

cordovadocs/2015/known-issues/known-issues-general.md

Line 31: [Warning] Illegal link: `<a href="#change-cli-version.md">Change the CLI version of your Visual Studio Tools for Apache Cordova project</a>` -- missing bookmark. The file 2015/known-issues/known-issues-general.md doesn't contain a bookmark named change-cli-version.md.

cordovadocs/2015/first-steps/get-started-with-ionic2.md

Line 919: [Warning] Illegal link: `<a href="#scp">Did you receive an error message saying that the Content Security Policy is missing?</a>` -- missing bookmark. The file 2015/first-steps/get-started-with-ionic2.md doesn't contain a bookmark named scp.
Line 927: [Warning] Illegal link: `<a href="#other">Other issues?</a>` -- missing bookmark. The file 2015/first-steps/get-started-with-ionic2.md doesn't contain a bookmark named other.
Line 1009: [Warning] Illegal link: `<a href="#configWindows">earlier steps</a>` -- missing bookmark. The file 2015/first-steps/get-started-with-ionic2.md doesn't contain a bookmark named configWindows.

cordovadocs/2015/access-device-capabilities/manage-plugins.md

Line 16: [Warning] Illegal link: `<a href="#List">List of available plugins</a>` -- missing bookmark. The file 2015/access-device-capabilities/manage-plugins.md doesn't contain a bookmark named List.
Line 76: [Warning] Illegal link: `<a href="#remove">Remove them</a>` -- missing bookmark. The file 2015/access-device-capabilities/manage-plugins.md doesn't contain a bookmark named remove.

cordovadocs/2017/first-steps/ios-guide.md

Line 225: [Warning] Illegal link: `<a href="#stack">Are you experiencing a symptom that does not appear in this list?</a>` -- missing bookmark. The file 2017/first-steps/ios-guide.md doesn't contain a bookmark named stack.
Line 261: [Warning] Illegal link: `<a href="#stack">Are you experiencing a symptom that does not appear in this list?</a>` -- missing bookmark. The file 2017/first-steps/ios-guide.md doesn't contain a bookmark named stack.
Line 315: [Warning] Illegal link: `<a href="#stack">Are you experiencing a symptom that does not appear in this list?</a>` -- missing bookmark. The file 2017/first-steps/ios-guide.md doesn't contain a bookmark named stack.
Line 406: [Warning] Illegal link: `<a href="#stack">Are you experiencing a symptom that does not appear in this list?</a>` -- missing bookmark. The file 2017/first-steps/ios-guide.md doesn't contain a bookmark named stack.

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.


WIP: Bugfix AccessToken was never refreshed

bump depencendies
add logging
add error handling
try to refresh before every request

TODO: implement more intelligent refreshing

Coverage decreased (-0.2%) to 66.485% when pulling 26bbe2005724cb01928b9ed71bb91ee406bb12e1 on refresh-accesstoken into c5a8e8ec782e541ba0e5d925a36a93c94ade66d2 on master.


Web view support

We should add support for web view. Here's Facebook documentation on this: https://developers.facebook.com/docs/messenger-platform/messenger-extension. I haven't looked into details yet.

I already implemented some of the components for this. I'm working on the white listing of domains now. Once I got that working, I'll be able to test this feature.

Completed this with the latest commit.


validation of input

The code doesn't check for the validation of input and tries to execute directly giving wrong in case of unexpected inputs.

@harshraj22 good you picked up brother lets grind github together :)


Compiled BeanShell more strict with semi colons

BeanShell's engine (bsh.engine.BshScriptEngine) reports to implement Compileable, still, compiled code is strict about single statements requiring semicolons. This makes it hard to transition from interpreted to compiled code without forcing 3:rd party script developers to append semicolons.
Please allow single statements to not have to be semicolon separated, even for compiled BeanShell code.

The compile functionally is intended for use as complete scripts, not scriptlets. If the script to be compiled is not valid and contains parser errors, like a missing line terminator ;, it will fail.
Closed: works as implemented

The ONLY relaxation I asked for was the same relaxation that holds true for interpreted BeanShell and that is that single statement scriptlets may have the semicolon omitted.

@davidekholm I understand what you asked for but you are not hearing me.
Compile is not for scriptlets.

Who defines that? I'm sorry, but it's naturally a tremendous advantage to not textually parse the same scriptlet hundreds of times, when it can be parsed once and then executed multiple times.

@davidekholm I will do this for you...
Did an impact analysis and some benchmarks and the cost of checking line endings is marginal however the price of additional empty lines does not come for free.
Stumbled on a different disaster that relates to the compiled scripts implementation which puts everything in a bit of a mess at the moment. It is going to take a little longer...
Thank you for your patience!
Open: will implement

Happy to hear that you're willing to allow this. From a user perspective, It's ideal if one only needs to handle the caching of the compiled scritplets but can otherwise enjoy the benefits of compiled code without the need to rewrite the end user code.

@davidekholm All done!
You should use the serial garbage collector with java command line argument -XX:+UseSerialGC because the parallel garbage collector chokes trying to allocate new eden capacity.
Managed some major optimisations with test time to completion reduced by about 8 seconds overall.
The following jmh benchmark:
import org.openjdk.jmh.annotations.*;
import bsh.Interpreter;
import bsh.PreparsedScript;
import java.util.Collections;

@State(Scope.Benchmark)
public class MyBenchmark {

    Interpreter bsh = new Interpreter();
    PreparsedScript pscr;
    Integer val = 9;

    public MyBenchmark() {
        try {
            pscr = new PreparsedScript("4+5;");
        } catch(Throwable t) {
            throw new RuntimeException(t);
        }
    }

    @Benchmark
    public void test_eval() throws Throwable  {
       if ( !val.equals(bsh.eval("4+5;")) )
           throw new Exception("Incorrect value");
    }

    @Benchmark
    public void test_prepared() throws Throwable  {
        if ( !val.equals(pscr.invoke(Collections.EMPTY_MAP)) )
           throw new Exception("Incorrect value");
    }
}

Originally could not be completed by prepared/compiled scripts with application stalling to a halt.
Benchmark                  Mode  Cnt    Score     Error  Units
MyBenchmark.test_eval      avgt    5   23.783 ±   0.767  us/op
MyBenchmark.test_prepared  avgt    5  417.903 ± 494.772  us/op

Then managed to get things squared up again to produce satisfactory results over the weekend.
Benchmark                  Mode  Cnt   Score   Error  Units
MyBenchmark.test_eval      avgt    5  23.618 ± 0.659  us/op
MyBenchmark.test_prepared  avgt    5  16.676 ± 1.839  us/op

Now after all the additional optimisations, you can hardly believe it is the same test.
Benchmark                  Mode  Cnt  Score   Error  Units
MyBenchmark.test_eval      avgt    5  9.279 ± 1.083  us/op
MyBenchmark.test_prepared  avgt    5  3.316 ± 0.301  us/op

The score is read in microseconds per operation, error shows the deviation between slowest and fastest over 5 iterations.

Awesome Nick! Thanks for these fixes.


bump dependencies

Applied mkpr on package.json
/tmp/mkpr.sh

:tada: This PR is included in version 1.32.2 :tada:
The release is available on:

npm package (@latest dist-tag)
v1.32.2
GitHub release

Your semantic-release bot :package::rocket:


Design v1

Do you think it would be better if you organise all the old design files in a separate folder?.

So far I don't see any problems with the autocomplete functions. The only possible explanation for why it doesn't work is because you are using secure http (which is https) in the AJAX and the subdomain is http. Therefore it will not work.

I'm thinking of getting rid of the home.blade.php file in resources\views\home.blade.php . So far it seems pointless for me. If you don't mind checking all the routes and see if one points to the file. If there is, just change it to other route.


Add onRedirect callback

Why it is needed?
From the docs :

Once the redirect URL is visited (a GET hits your server), handle the result and perform a server redirect to com.company.myapp://close?success=true to close the browser (and pass any data back to your app using query params ✨).

This works well when we manage the server behind the webpage being opened, but not when it's a third party website.
For example, in an "OAuth dance", the third party will redirect to the redirect URL we defined when declaring the application on their side. Some of them do not allow to put a deeplink there, forcing an HTTP URL. Which makes it impossible to have the onClose callback called with the deeplink. I can link my mobile app present in the stores to illustrate if need be.
Possible implementation
One of the solutions I imagined is to have an onRedirect callback, having the same signature as onClose. This way, on the mobile app side, we could listen to the URL changes and mimick the onClose callback when needed.
Code sample
import { openBrowser } from "@swan-io/react-native-browser";

openBrowser("https://swan.io", {
  dismissButtonStyle: "close", // "cancel" | "close" | "done" (default to "close")
  barTintColor: "#FFF", // in-app browser UI background color
  controlTintColor: "#000", // in-app browser buttons color
  onOpen: () => {
    // fired on browser opened
    // useful to switch the StatusBar color, for example
  },
  onClose: (url) => {
    // fired on browser closed
    // url will be defined if the browser has been closed via deeplink
  },
  onRedirect: (url) => {
    // fired on each page redirect within the browser
    // url corresponds to the new URL of the browser (i.e. window.location.href)
  },
}).catch((error) => {
  console.error(error);
});

@c100k You can't do that without a dedicated deeplink (at least in in-app browsers)

Ah ! Indeed : https://stackoverflow.com/questions/42192701/detect-url-change-in-sfsafariviewcontroller
Closing then. Thanks for your quick reply.


Add note on required import for ES2015 usage

I was briefly confused by this until I realized I was missing this import. It's probably since I'm new to node, but so are a lot of other people.
Thanks for a great project!

That's not needed. We handle that for you.

Just write your tests in ES2015. No extra setup needed.

This is actually true, you don't need to import babel-core/register to write tests in ES2015. I assume your code was crashing, because you wrote other code (outside of tests) in ES2015, but using Node.js' version that does not support it.

Oh interesting. Thanks for the clarification @vdemedes.


Operator recognition problem.

With opMap:
       OppMap_t& opp = calculator::Default().opPrecedence;
        opp.add("[]", 2); opp.add("()", 2); opp.add(".", 2);
        opp.add("**", 3);
        opp.add("*",  5); opp.add("/", 5); opp.add("%", 5);
        opp.add("+",  6); opp.add("-", 6);
        opp.add("<<", 7); opp.add(">>", 7);
        opp.add("<",  8); opp.add("<=", 8); opp.add(">=", 8); opp.add(">", 8);
        opp.add("==", 9); opp.add("!=", 9); opp.add("=%", 9);
        opp.add("&&", 13);
        opp.add("||", 14);
        opp.add("=", 15); opp.add(":", 15);
        opp.add(",", 16);

Parsing expression 'function() == [1,2,3]' - ok, but 'function()==[1,2,3]' gives 'Invalid operator: ==[' exception.

That's a true problem I have to figure out a good solution.
Currently, there is a hard-coded solution that solved this for "(", ")" and "_" and string delimiting characters, but it did not include "[" and "{".
I just fixed that in this commit: 235da7034c35f72af6c031d425f78ab02df679be
So your code should work now if it doesn't please let me know.
This is a hard-coded solution so I will not consider this issue closed until a plan something cleaner.

Thanks, it works now.


typeahead-editable binding

When trying to replace old directive names with the uib- prefixed versions, I run into $parse errors with uib-typeahead. Previously, with the un-prefixed directive, I could use typeahead-editable="{{editable}}"  where editable is defined on the controller scope, but now this causes a $parse error. See http://plnkr.co/edit/vmF98QeXxj5E5pDRYOSQ?p=preview

This is still a bug in master.

@DavidBanksNZ, thanks for catching this.

In this case, this is easy to fix by not using {{ }}, as seen here

Going to close this as not a bug - did some git history tracing, it doesn't look like this was officially supported with interpolation for quite a long time, if ever.

Yes it's an easy workaround thanks. Perhaps it used to be 'unintentionally supported'


[es] Deep learning

Hope that helps

Great work! Now, let's wait for the opinion of another native speaker on these translations.
Please also add yourself to the list of contributors (CONTRIBUTORS located in the root folder).

I forked and pull-requested my own version of this translation. I think you should review it. I saw yours but there are many inconsistencies and lacks information. I seriously don't want to sound rude but please compare both of them and let me know If i can help.

Thank you both for your hard work!
I realized that the translation was already done in this pull request. I propose we continue this discussion from there.

Thanks kolt. You're right. Me falta aprender mucho del tema y familiarizarme con los términos correctos. Let's see the another translation

This PR should be closed


Implement try_new for non-panicking version of new

Panicking function might not be best for production usage. Therefore, I propose a non-panicking version of the function with Result<Client, String> as the return value.
I also propose to mark the new function as deprecated to discourage people to use it on production.
Other changes:

simplify the new function implementation

Please let me know if there's anything I need to address. Thank you

panic on Client creation is for security reasons. the AES should not run/created without any environment.
reasons should panic and crash the program on creation :

Client should not be triggered on runtime after boot-time
Clients should not run without any environment

to enforce usage of environment and keys, panic and stop the program should be the most secure options.

An example/use-case of runtime key change would be helpful.

An example/use-case of runtime key change would be helpful.

Sure things

Deprecated macro for new is removed ea39fcc
Example code is updated to demonstrate try_new as well b8af7a6


Fix nav_file column being off by 1

What issue does it try to solve
In create_mark and store_offset vim.api.nvim_win_get_cursor returns (1,0)-based cursor position, but :call cursor() in nav_file expects (1,1)-based one.

@w1ck3dg0ph3r ?


⚠ Bofrost.lu has degraded performance

In 943c4a2, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2479 ms

Resolved: Bofrost.lu performance has improved in d96e3c6.


Major llama.cpp Change Notes

llama.cpp 3328 just got merged. This contains a major change to the API which will require some big changes to LLamaSharp to support.

[ ] Make a release before this change, with current API
[ ] Make basic changes to low level API
[ ] Adapt current executors to work with new API (minimal changes)
[ ] Work out how to expose new capabilities (entirely new executor?)

I ran into this issue when I wanted to try the new mistral model.
I updated the llama.cpp dlls and it failed the token generation with current main branch llamasharp branch.
It's tough to keep up with Llama.cpp breaking changes and all the new models coming in daily :-)

That's odd, I actually tried out Mistral-7B-Instruct earlier today using 0.5.1 and it worked perfectly.

Thanks Martin. I used the current Llamasharp main branch with the Cuda 11 dll in runtime folder and it worked!
I thought Mistral required the new llama dlls and when I had tried it with yesterday's Llama.cpp dlls, it threw the tokenization error.
Thanks again,
Ash


Updates not working

Copy of #102040 which is locked, but still an issue. See https://github.com/microsoft/vscode/issues/102040#issuecomment-663160453 for --verbose logs.

VSCode Version: 1.46.1 and others
OS Version: macOS 10.15.5

When I try to apply an update, VS Code doesn't restart and after a manual restart the update hasn't been applied and VS Code offers the update again.
Steps to Reproduce:

Choose "Restart to update"
VS Code doesn't restart
Restart manually
Update isn't applied

Does this issue occur when all extensions are disabled?: Yes

Seems like a duplicate of https://github.com/microsoft/vscode/issues/100990
Can you check the permissions of ShipIt_stderr.log, ShipIt_stdout.log under Library/Caches/com.microsoft.VSCode.ShipIt/ and make sure they are owned by the current user.

Hi @deepak1556 ,
I don't have that com.microsoft.VSCode.ShipIt directory path neither in my /Library/Caches/ nor `~/Library/Caches.
Where else can I look for the logs?
P.S. I am on Catalina and you can see the full info on this related issue I created? https://github.com/microsoft/vscode/issues/106742


"Error in subgraph loop" for failed provider call

The subgraph loop should not cause a full error for a failed RPC call. Users still need to see whatever txs they can.

@jakekidd is this still a thing? Please update or close.

unsure. going to keep an eye out for it for now, will tack this on to my todo list in the meantime

We can close this when we check health through index node.


LUCI bots Building flutter tools is failing Because flutter_tools depends on collection >=1.15.0-nnbd <1.15.0-nullsafety.2 which requires SDK version >=2.9.0-18.0 <=2.9.10, version solving failed.

The current Dart SDK version is 2.10.0-0.0.dev.flutter-a3815b6590.
Because flutter_tools depends on collection >=1.15.0-nnbd <1.15.0-nullsafety.2 which requires SDK version >=2.9.0-18.0 <=2.9.10, version solving failed.
Error: Unable to 'pub upgrade' flutter tool. Retrying in five seconds... (9 tries left)
Resolving dependencies...
The current Dart SDK version is 2.10.0-0.0.dev.flutter-a3815b6590.
https://cirrus-ci.com/task/5090607431417856?command=setup#L444

I think there are some problems with our CI stability, we often need to rerun to pass, especially the flutter-build tree, which is often closed.

After synchronizing the forked branch and the upstream master, the problem is solved

A quick git pull and then any flutter command such as flutter -h does downloads a valid dart SDK (2.9.0-edge.a3815b659047cfc6b63d41d180aa1ca1b781ee68).

A quick git pull and then any flutter command such as flutter -h does downloads a valid dart SDK (2.9.0-edge.a3815b659047cfc6b63d41d180aa1ca1b781ee68).

Thank you @Vickycp


timerange constraints support for TimeEntity in duckling from template side

solves #127

Codecov Report

Merging #128 (9bb1254) into master (e5b3c11) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master      #128   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           49        49
  Lines         2059      2079   +20
=========================================
+ Hits          2059      2079   +20

Flag
Coverage Δ

unittests
100.00% <100.00%> (ø)

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

Impacted Files
Coverage Δ

dialogy/constants/__init__.py
100.00% <100.00%> (ø)

dialogy/plugins/text/duckling_plugin/__init__.py
100.00% <100.00%> (ø)

dialogy/types/entity/deserialize/__init__.py
100.00% <100.00%> (ø)

dialogy/types/entity/time/__init__.py
100.00% <100.00%> (ø)

dialogy/types/entity/time_interval/__init__.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Changefeed cursors must be in quotes

Fixes #5502.

This change is

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/851982f1e8e8fcf685875e14a123c40270201ba4/
Edited pages:

v19.1/create-changefeed.md
v19.2/create-changefeed.md
v2.1/create-changefeed.md

@rolandcrosby, such a minor change, I'm just going to merge.


Panic in get_search_list when the SearchList registry key is missing

On two different computers running Windows 10 version 1803 (OS build 17134.191), I encountered the following panic while running cargo test:
thread 'no_error' panicked at 'called `Result::unwrap()` on an `Err` value: Error(Io(Os { code: 2, kind: NotFound, message: "The system cannot find the file specified." }), State { next_error: None, backtrace: Some(stack backtrace:
   0:     0x7ff743abc920 - backtrace::backtrace::trace<closure>
                        at C:\Users\tyn\.cargo\registry\src\github.com-1ecc6299db9ec823\backtrace-0.3.9\src\backtrace\mod.rs:42
   1:     0x7ff743ab3f94 - backtrace::capture::Backtrace::new_unresolved
                        at C:\Users\tyn\.cargo\registry\src\github.com-1ecc6299db9ec823\backtrace-0.3.9\src\capture.rs:88
   2:     0x7ff743ab3efd - backtrace::capture::Backtrace::new
                        at C:\Users\tyn\.cargo\registry\src\github.com-1ecc6299db9ec823\backtrace-0.3.9\src\capture.rs:64
   3:     0x7ff743ab1ccc - error_chain::make_backtrace
                        at C:\Users\tyn\.cargo\registry\src\github.com-1ecc6299db9ec823\error-chain-0.8.1\src\lib.rs:413
   4:     0x7ff743ab1d8a - error_chain::{{impl}}::default
                        at C:\Users\tyn\.cargo\registry\src\github.com-1ecc6299db9ec823\error-chain-0.8.1\src\lib.rs:494
   5:     0x7ff743a8fdb8 - ipconfig::error::Error::from_kind
                        at E:\dev\ipconfig\<error_chain_processed macros>:49
   6:     0x7ff743a8fee3 - ipconfig::error::{{impl}}::from
                        at E:\dev\ipconfig\<error_chain_processed macros>:77
   7:     0x7ff743a9580a - ipconfig::computer::get_search_list
                        at E:\dev\ipconfig\src\computer.rs:52
   8:     0x7ff743a42fe3 - test::no_error
                        at E:\dev\ipconfig\tests\test.rs:7
   9:     0x7ff743a4244e - test::__test::TESTS::{{closure}}
                        at E:\dev\ipconfig\tests\test.rs:5
  10:     0x7ff743a419fb - core::ops::function::FnOnce::call_once<closure,()>
                        at C:\projects\rust\src\libcore\ops\function.rs:223
  11:     0x7ff743a4569e - alloc::boxed::{{impl}}::call_box<(),closure>
                        at C:\projects\rust\src\liballoc\boxed.rs:643
  12:     0x7ff743ae7fe2 - panic_unwind::__rust_maybe_catch_panic
                        at C:\projects\rust\src\libpanic_unwind\lib.rs:105
  13:     0x7ff743a5097b - std::sys_common::backtrace::__rust_begin_short_backtrace<closure,()>
                        at C:\projects\rust\src\libstd\sys_common\backtrace.rs:136
  14:     0x7ff743a5186e - std::panicking::try::do_call<std::panic::AssertUnwindSafe<closure>,()>
                        at C:\projects\rust\src\libstd\panicking.rs:312
  15:     0x7ff743ae7fe2 - panic_unwind::__rust_maybe_catch_panic
                        at C:\projects\rust\src\libpanic_unwind\lib.rs:105
  16:     0x7ff743a58a15 - alloc::boxed::{{impl}}::call_box<(),closure>
                        at C:\projects\rust\src\liballoc\boxed.rs:642
  17:     0x7ff743ad04a2 - std::sys_common::thread::start_thread
                        at C:\projects\rust\src\libstd\sys_common\thread.rs:25
  18:     0x7ff743ad29e9 - std::sys::windows::thread::{{impl}}::new::thread_start
                        at C:\projects\rust\src\libstd\sys\windows\thread.rs:58
  19:     0x7ff8975e3034 - BaseThreadInitThunk) })', libcore\result.rs:983:5

On both computers, I do not have a SearchList subkey in SYSTEM\CurrentControlSet\Services\Tcpip\Parameters. I assume that it can be a valid state, since I didn't remove it manually.
I can work around this issue by creating an empty SearchList subkey at that location, but it's obviously not great.

Closed by #16


checkTypes.assert(false, ...) throwing wrong exception

I found a regression in 7.1.3:
const checkTypes = require('check-types');

checkTypes.assert(false, 'this is false');

I have expected this to throw 'this is false', but in fact it's throwing 'TypeError: ErrorType is not a constructor'. this had been working as expected in 7.0.1.

Sigh, sorry yet again!
I thought I had them all covered now, but no. I'll add tests and fix this later on.

@stefanheine, I've published a fix in 7.1.4. Let me know if you get any (more) problems! 😄

thanks!


Fix Issue 16642  - byCodeUnit doesn't work properly with alias this.

Basically, byCodeUnit works properly with strings, with structs that are ranges of characters (but do not implicitly convert to any string type), and with structs that use alias this with a member variable to convert to a string type. It does not work properly with enums (which pass the template constraint but fail to compile, because they're not actually ranges), it does not work with structs that use a member function with alias this, and it does not work with structs that are already ranges of characters but also use alias this with a string type. This PR fixes all that.
The main thing is that it now forces the conversion to the string type when the type needs an implicit conversion to be a range of characters. In general, it's bad practice to have a template test for an implicit conversion and then not force the conversion, because it can result in some fun, subtle problems. A prime example being that enums are not actually ranges even if their base type is a string type.
Given that enums aren't really ranges and that they didn't compile properly with byCodeUnit before, it might make more sense to just disallow them just like static arrays are disallowed, but since at heart, this is basically a function for taking a string and converting it to a non-autodecoding range, and functions that take strings work with enums with a base type of string, it makes sense to me that byCodeUnit would work with enums.
Probably the ugliest bit with this is making it so that a struct that's already a range of characters but also implicitly converts to a string doesn't do the implicit conversion in byCodeUnit. But since normally, implicit conversions do not occur unless the type needs to be converted to work, it doesn't make sense to do the implicit conversion on a type that's already a range of characters on its own.
The second commit is for some formatting clean-up for byCodeUnit, so you may want to look at the first commit separately.

Params needs to be updated

There seem to be bugs in the codecov coverage marking. Pinging @wilzbach

Docs updated.

There seem to be bugs in the codecov coverage marking. Pinging @wilzbach

Hmm - it seems like CodeCov shifts the resulting coverage for a couple of lines. Compare an excerpt from their coverage report:

with running the coverage result locally (e.g. make -f posix.mak std/utf.test):

Unfortunately I have no idea what could cause this shift (I tried to restart CircleCi).

@jmdavis you would need to remove the # from the git message in order for the bot (and changelog gen) to detect. For more details:
https://github.com/dlang-bots/dlang-bot#automated-references

Unfortunately I have no idea what could cause this shift (I tried to restart CircleCi).

It could be due to the fact that we merge the commit into the master branch before running the coverage tests and thus it's possible that CodeCov receives a different .lst file from CircleCi as the underlying sources are different.

Okay, I updated the tests to use more characters that won't fit in a char or wchar, and I redid the changes on top of master so that they're mergeable again.

Okay. I made the fix to the template constraint and the 2nd static if condition as well as beefing up the tests so that they actually catch what I missed before.

Man, I hate this maze of traits involving strings, and implicit conversions are hell for generice code. :|

Yup, the alias this and enums make templates very hard to get 100% correct. I bet there are tons of hidden bugs in most D code in the wild dealing with strings/string-like-ranges

Auto-merge toggled on

@jmdavis @JackStouffer Looks like this broke CircleCI tests on master. To avoid this in the future you can use the "auto-merge" label to use @dlang-bot instead of enabling auto-merge on Brad's CI, which does not take into account other CI services.

https://github.com/dlang/phobos/pull/5266


ERROR: Manifest merger failed

ERROR: Manifest merger failed : Attribute application@appComponentFactory value=(android.support.v4.app.CoreComponentFactory) from [com.android.support:support-compat:28.0.0] AndroidManifest.xml:22:18-91 	is also present at [androidx.core:core:1.0.0] AndroidManifest.xml:22:18-86 value=(androidx.core.app.CoreComponentFactory). 	Suggestion: add 'tools:replace="android:appComponentFactory"' to <application> element at manifestMerger6286653408933565996.xml:7:5-9:19 to override.

pull最新代码


Add BBM redirect view and use it to detect BBM

This is easier to maintain than separate domain names for BBM. My hope is that we can direct users to the /bbm/ endpoint which will redirect them to the homepage but mark them as BBM users.

Updated to address one of Codie's comments on Slack that the middleware needs to be updated to send to Analytics based on the cookie, I think this is ready for review on Monday. See you then!

@alexmuller just had a thought - users need to be able to share articles via bbm, so there are share buttons for both the home page and article-specific shares.
In this case, we'll need to update molo-gem/gem/templates/springster/patterns/basics/social-media/social-media_bbm.html so that we include the redirect link in the shared content and the redirect will need to handle both home page and article page redirects.

Gaaaah ✊
But yeah thanks, good point

Updated to use /bbm as a prefix for redirecting. Took a little bit longer than I would have liked because it has some interesting repercussions for not being an open redirect that other people can abuse.


Standard way of sharing architectures, e.g., to PyPi.org

There are a huge number of new statistical, machine-learning and artificial intelligence solutions being released every month.
Most are open-source and written in a popular Python framework like TensorFlow, JAX, or PyTorch.
In order to 'guarantee' you are using the best [for given metric(s)] solution for your dataset, some way of automatically adding these new statistical, machine-learning and artificial intelligence solutions to your automated pipeline needs to be created.
(additionally: useful for testing your new optimiser, loss function, &etc. across a zoo of datasets)
Ditto for transfer learning models. A related problem is automatically putting ensemble networks together. Something like:
import some_broke_arch  # pip install some_broke_arch
import other_neat_arch  # pip install other_neat_arch
import horrible_v_arch  # builtin to keras

model   = some_broke_arch.get_arch(   **standard_arch_params  )
metrics = other_neat_arch.get_metrics(**standard_metric_params)
loss    = horrible_v_arch.get_loss(   **standard_loss_params  )

model.compile(loss=loss, optimizer=keras.optimizers.RMSprop, metrics=metrics)
print(model.summary())
# &etc.

In summary, I am petitioning for standard ways of:

exposing algorithms for consumption;
combining algorithms;
comparing algorithms.

To that end, I would recommend encouraging the PyPi folk to add a few new classifiers, and a bunch of us trawl through GitHub every month sending PRs to random repositories—associated with academic papers—linking up with CI/CD so that they are now installable with pip install and searchable by classifier on PyPi.
Related:

my open-source multi-ML meta-framework;

uses builtin ast and inspect modules to traverse the module, class, and function hierarchy for 10 popular open-source ML/AI frameworks;
will enable experimentation with entire 'search-space' of all these ML frameworks (every transfer learning model, optimiser, loss function, &etc.)
with a standard way of sharing architectures will be able to expand the 'search-space' with community contributed solutions.

this issue from Jul 20, 2019: https://github.com/tensorflow/tensorflow/issues/30896

Triage notes:
In the short-term, we don't have enough bandwidth for supporting this feature.
but we will keep it open for consideration.

@haifeng-jin I can add this feature and maintain it on my own org. The issue is no one knows who I am or have heard of my org. So sending PRs around the place and requested PyPi add new classifiers is likely to fall on deaf ears.
As for exposing the intricacies of Keras, TensorFlow, JAX, PyTorch, &etc… that's what multi-ML meta-framework—written with my compiler—is for. So I won't need major changes made to any of these frameworks (only minor changes to conform types to expected inputs & outputs).
What I do need is enough reputation to push the community from a hodge-podge collect of adhoc Jupyter Notebooks, scripts, and python packages to conformance to a new specification (PyPi I think is still best… RFC welcome).

@SamuelMarks You are welcome to build your own package to do it, but, unfortunately, we are not able to officially support it right now.

@SamuelMarks try to contact https://github.com/mlcommons/ck/blob/master/ck2/docs/motivation.md
They could be interested in something similar.


out of preview

Azure Database Migration Service MySql is out of preview

@cbek : Thanks for your contribution! The author, @HJToland3, has been notified to review your proposed change.

@cbek  :  Azure DMS support for online migrations of MySQL to Azure Database for MySQL is in Public Preview, but customers no longer need to sign-up to participate via the preview site. Also, for the next steps section, I would point to this tutorial, which is specific to the MySQL to Azure DB for MySQL scenario: https://docs.microsoft.com/azure/dms/tutorial-mysql-azure-mysql-online

@cbek : Thanks for your contribution! The author, @HJToland3, has been notified to review your proposed change.

@PRMerger13, do I need to do anything more that approve these changes, such as sign-off?


Helm Integration?

Hi,
I cant see any way to put HelmChart or HelmChartConfig manifests in /var/lib/rancher/rke2/server/manifests with this module?

If we want to install another CNI like Cilium Enterprise we need to set cni: none and pass a custom HelmChart and values ...

Thank you so much for sharing!
I have added https://github.com/rancher/terraform-null-rke2-install/commit/d9352d3da2ec4956e11d90dea095f2b819f81b3d to enable this use case.

I also added an example of using the new feature to deploy a custom cilium chart, hopefully this gives you a jump up on the process. Our examples are used directly in our tests, so this will also serve as an acceptance test for your use case.
Please open a new issue if there is anything else I can do!
https://github.com/rancher/terraform-null-rke2-install/tree/main/examples/manifest


feat: add aws connection support

This PR was generated using Autosynth. :rainbow:
Synth log will be available here:
https://source.cloud.google.com/results/invocations/7491c866-49e9-410f-818d-dc23eb0e17a2/targets

[ ] To automatically regenerate this PR, check this box.

PiperOrigin-RevId: 334870265
Source-Link: https://github.com/googleapis/googleapis/commit/aa72330a8d16d281df131e8d37bb1a4bde53401f

@googlebot I consent.


Add dropzap world (iOS and macOS)

https://testflight.apple.com/join/CdEXgjst - dropzap world - lots ofmirrors, zappers, color matching and lots of levels

done


Subclasses of "Data about an ontology part" do not have labels, are not longer in GO

The following classes do not have labels in IAO. Also, they are not longer in GO. Can you please either give these labels or deprecate them?
http://www.geneontology.org/formats/oboInOwl#DbXref
http://www.geneontology.org/formats/oboInOwl#Definition
http://www.geneontology.org/formats/oboInOwl#Subset
http://www.geneontology.org/formats/oboInOwl#Synonym
http://www.geneontology.org/formats/oboInOwl#SynonymType

All terms are defined as classes and not used in IAO except http://www.geneontology.org/formats/oboInOwl#Subset, which is type of individual 'core'.
So, all except oboInOwl#Subset should be fine to be deprecated.

Discussed on OBO operation committee call on Feb 11th, 2020.  The terms are used in OBO to OWL format conversion. All these terms will not be used any more including the individual 'core'. So, all of these terms will be deprecated.

Deprecated following classes:
http://www.geneontology.org/formats/oboInOwl#DbXref
http://www.geneontology.org/formats/oboInOwl#Definition
http://www.geneontology.org/formats/oboInOwl#Subset
http://www.geneontology.org/formats/oboInOwl#Synonym
http://www.geneontology.org/formats/oboInOwl#SynonymType
Deprecated individual:
http://purl.obolibrary.org/obo/IAO_0000224 core


Fix/tls

Issues associated with this PR
Your PR should present related issues you want to solve.
Solutions
You should show your solutions about the issues in your PR, including the overall solutions, details and the changes. At this time, Chinese is allowed to describe these.
UT result
Unit Test is needed if the code is changed, your unit test should cover boundary cases, corner cases, and some exceptional cases. And you need to show the UT result.
Benchmark
If your code involves the processing of every request, you should give the Benchmark Result.
Code Style

Make sure Goimports has run
Show Golint result

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@562b50c). Click here to learn what that means.
The diff coverage is 40%.

@@            Coverage Diff            @@
##             master    #1037   +/-   ##
=========================================
  Coverage          ?   55.59%
=========================================
  Files             ?      247
  Lines             ?    23905
  Branches          ?        0
=========================================
  Hits              ?    13290
  Misses            ?     9395
  Partials          ?     1220

Impacted Files
Coverage Δ

pkg/mtls/secret_manager.go
68.53% <0%> (ø)

pkg/mtls/sds/sdsclient.go
78.57% <100%> (ø)

pkg/mtls/sds/subscriber.go
54.28% <33.33%> (ø)

pkg/mtls/conn.go
27.58% <60%> (ø)

Continue to review full report at Codecov.

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


tools installed with rokit having issues with stdio

Ive been having troubles with using the stdio library and the repl command in lune recently, and Ive figured out that this only happens when I install lune using rokit, Ive tested this on other releases of lune aswell so Im sure this is related to rokit
✅ repl command with manual install:

❌ repl command with rokit install:

Ugh. Looks like this is (once again) an issue with the command-group crate and how it wraps spawned programs. Will need some further investigation...


Update version and configs

Checklist

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

Just updating to have python 3.7

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


Start building on Ethereum section and Batches page

Add staging changes from #41 to main

Thanks for the fast review @portdeveloper ! ♥

I have found two places we might want to fix, just added them as comments.

Ooooof I clearly did not check it properly on mobile! Both problems should be fixed, and conflicts resolved 🙏

the last td in currentItems tr becomes:

Love your change ♥ TYSM @portdeveloper


Unrelayed Sequences for Unordered channels

A first approach to implementing the UnrelayedSequences for unordered channels. In my understanding, this is the biggest barrier to #34.
This approach uses the acknowledgments stored on the destination chain, compared against the SeqSend on the source chain to determine which packets haven't been relayed.
This approach is used because NextSeqRecv isn't incremented on Unordered channels, so we need a different approach that accounts for the semantics specific to unordered channels.
Few things Id like to fix up, some naming things here and there. In general, it works though.
I've only done some basic testing but wanted to get some feedback on the approach before polishing things up.

This approach may be too resource intensive since it uses the store subspace query across all ackknowledged packets over the channel. At the moment there is no pagination or anything to prevent massive query results. There's several ways that could be remedied though.

Approach seems reasonable to me (given the current constraints), though I think it is worth following up on the cosmos-sdk see if it is possible to get some optimizations added to unordered channel handling. Maybe unordered channels can keep a queue of unack'd seqs.
I will look into this on the sdk side later this week

Yeah, this was my best idea short of updating the SDK to make it more efficient. I'm open to investigating how that might look.

Ack approach here. Great work @jsimnz. Agree with @colin-axner that it would be ideal if we could directly query unacked. Barring that, pagination added to querier is a must.

One possible solution would make this feature a lot more efficient

unordered channels are supported now


[Docs] Add explainer for gotchas with NSView based native components

Please select one of the following

[ ] I am removing an existing difference between facebook/react-native and microsoft/react-native-macos :thumbsup:
[ ] I am cherry-picking a change from Facebook's react-native into microsoft/react-native-macos :thumbsup:
[x] I am making a fix / change for the macOS implementation of react-native
[ ] I am making a change required for Microsoft usage of react-native

Summary
While working on https://github.com/microsoft/fluentui-react-native/pull/3026, I found a couple of "gotchas" while writing a native component for NSVisualEffectView. This change attempts to documents those gotchas so that others can benefit. Special thanks to @lenaic who  figured out the hit testing issue.
Changelog
[GENERAL] [ADDED] - Add documentation for gotchas with NSView based native components
Test Plan
N/A

Marking as draft to see if I can update RCTUIView hitTest instead


Unable to Download SQL Server Management Studio 17 from the Doc

🛑 IMPORTANT: You can get your feedback addressed faster if you use the comment section for the article in which you encountered a problem.
Link to article:
https://docs.microsoft.com/en-us/sql/ssms/download-sql-server-management-studio-ssms?view=sql-server-2017#ssms-180-preview-6
https://download.microsoft.com/download/D/D/4/DD495084-ADA7-4827-ADD3-FC566EC05B90/SSMS-Setup-ENU.exe
Problem:
Unable to download Download SQL Server Management Studio 17.9.1

Hi @vigarnyalvarez , since we haven't heard back from you, we will assume your Issue has been resolved. As such, I will proceed with closing out this Git Issue. However, please feel free to reach out to us if you need anything further. Thanks again for contacting Microsoft! #please-close

Hi!
Thanks for contacting back. I determined that it was a firewall issue with
the network that I was connected to at that time. Please, close the ticket.
Regards,
Vigarny A.
El mar., 5 feb. 2019 a las 18:48, Masha at MSFT<EMAIL_ADDRESS>escribió:

Hi @vigarnyalvarez https://github.com/vigarnyalvarez , since we haven't
heard back from you, we will assume your Issue has been resolved. As such,
I will proceed with closing out this Git Issue. However, please feel free
to reach out to us if you need anything further. Thanks again for
contacting Microsoft! #please-close
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/feedback/issues/1098#issuecomment-460862674,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Afcq6L8AKSFHd-wT_gQPMak4eSEwB5fNks5vKiZNgaJpZM4aZQLh
.

Hi @vigarnyalvarez , awesome, I'm thrilled to hear you were able to get SSMS downloaded :) thanks for letting us know, and thanks for reaching out to us!

#please-close

@tysonn - not sure how to get this closed.  Thanks!

#please-close

#please-close

#please-close


Reserved word "type" used as a parameter

The function LNetwork:SendToPeersExcept() can't work as is.
This works:
function LNetwork:SendToPeersExcept(peer, type_prm, data)
[followed by according replacements]

I added a pull request to fix this, when it's simple stuff like this it'd probably take less time to write an issue than it would to fix it and PR.

Pull Request submitted and merged. Issue resolved.


Refresh results can become inconsistent as quay.io builds age

Currently, we look through a projects builds in order to determine which git branches particular tagged images in quay.io are associated with. This can go awry when quay.io "ages out" these builds. Currently, valid tags can eventually become invalid when they lose their git references.
We need one of two possible solutions:
a) Fix refresh to look at more data sources than the list of builds, there should be a way to ask quay.io to elaborate on a specific tag if no build is available
b) never replace a valid git reference with a blank git reference, this would at least allow us to cache build information before it ages out

Crummy solution 1)
The call /api/v1/repository/{repository}/build/ with a higher limit (say, the maximum integer). This endpoint has been tested to exceed the default number of builds returned (looks like 5).

Note: b) looks like its the current behaviour already

This is in the current hotfix branch, close when ready


update OSX no app patch

Checklist

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

This refreshes the OSX "no app" patch against qgis 3.16.3 and incorporates https://github.com/qgis/QGIS/pull/41207 - apparently the original patch broke the GRASS plugin on OSX, but it would be good to by in sync with upstream anyway.

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

@conda-forge-admin, please rerender


Feature request: A paste button above the input

Or some other way of easily converting code from the clipboard.

Closing as clipboard access isn't practically possible.