Update own-or-env to 1.0.1

This PR updates the own-or-env dependency to version 1.0.1. This fixes an issue running tap when projects using tap have had their dependencies installed by a package manager that does not create a flat node_modules directory, e.g. pnpm.

I don't see any reason why AppVeyor would be failing.

This doesn't really seem necessary?  ^1.0.0 will already pull in 1.0.1 in a fresh install, and 1.0.1 only fixed a bug that didn't affect tap in the first place (since it also has own-or as a dep.)


Restart-condition flag should have an option "on-update"

Description
When creating/updating services, I believe a valid restart-condition should be "on-update". "none", "on-failure", and "any" do not work when I only want the service to restart when it is updated.
Usefulness:
To achieve global volumes, I create a global "config" service with a volume attached to it with a restart condition of "none". I then create a replicated service that mounts the volume created by the global "config" service. When I update both services, the new volume name is reflected when I inspect both services, but because the global "config" service was never restarted on update, the changes in the volume are not reflected on the replicated service.
My current workaround is to remove and re-create the global "config" service with the new volume and update the replicated service.
Steps to reproduce the issue:

`docker service create --restart-condition "on-update"

Describe the results you received:
Error response from daemon: invalid RestartCondition: "on-update"
Describe the results you expected:
The service is created with a RestartCondition of "on-update"
Output of docker version:
-bash-4.2$ docker version
Client:
Version:      1.12.1
API version:  1.24
Go version:   go1.6.3
Git commit:   23cf638
Built:
OS/Arch:      linux/amd64
Server:
Version:      1.12.1
API version:  1.24
Go version:   go1.6.3
Git commit:   23cf638
Built:
OS/Arch:      linux/amd64
Output of docker info:
-bash-4.2$ docker info
Containers: 5
Running: 1
Paused: 0
Stopped: 4
Images: 12
Server Version: 1.12.1
Storage Driver: devicemapper
Pool Name: docker-8:3-134351990-pool
Pool Blocksize: 65.54 kB
Base Device Size: 10.74 GB
Backing Filesystem: xfs
Data file: /dev/loop0
Metadata file: /dev/loop1
Data Space Used: 1.332 GB
Data Space Total: 107.4 GB
Data Space Available: 45.24 GB
Metadata Space Used: 2.154 MB
Metadata Space Total: 2.147 GB
Metadata Space Available: 2.145 GB
Thin Pool Minimum Free Space: 10.74 GB
Udev Sync Supported: true
Deferred Removal Enabled: false
Deferred Deletion Enabled: false
Deferred Deleted Device Count: 0
Data loop file: /var/lib/docker/devicemapper/devicemapper/data
WARNING: Usage of loopback devices is strongly discouraged for production use. Use --storage-opt dm.thinpooldev to specify a custom block storage device.
Metadata loop file: /var/lib/docker/devicemapper/devicemapper/metadata
Library Version: 1.02.107-RHEL7 (2016-06-09)
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
Volume: local
Network: host overlay bridge null
Swarm: active
NodeID: 09cnqvf34rrf8k0bzdt6mg6qw
Is Manager: true
ClusterID: 7b17sczyypucbhj8e4q2zr8m3
Managers: 3
Nodes: 5
Orchestration:
Task History Retention Limit: 5
Raft:
Snapshot Interval: 10000
Heartbeat Tick: 1
Election Tick: 3
Dispatcher:
Heartbeat Period: 5 seconds
CA Configuration:
Expiry Duration: 3 months
Node Address: <IP_ADDRESS>
Runtimes: runc
Default Runtime: runc
Security Options: seccomp
Kernel Version: 3.10.0-327.22.2.el7.x86_64
Operating System: Red Hat Enterprise Linux Server 7.2 (Maipo)
OSType: linux
Architecture: x86_64
CPUs: 1
Total Memory: 3.703 GiB
Name: aglqpsswm01
ID: LNPV:D26H:5SFY:SWNH:WYS2:SSXO:7XPW:PKOV:NAJB:LI3P:YRIA:OHQC
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): false
Registry: https://index.docker.io/v1/
Insecure Registries:
<IP_ADDRESS>/8

Services are always restarted when they are updated, so I think on-update would be confusing. I don't quite understand your use case.

the changes in the volume are not reflected on the replicated service.

Any changes to the data in a volume should always be reflected immediately to all consumers of the volume. Are you changing the name of the volume or the contents? Which volume driver are you using?

Sounds a bit like you're looking for a "batch mode" service; https://github.com/docker/docker/issues/23880

@dnephin I am changing the name and the contents of the volume. My volume driver is local and my storage driver is the loop devicemapper. It seems services are only restarted when they are updated if they are already running, no?

@thaJeztah Yes, I think this issue is a dupe. This comment in particular convinced me: https://github.com/docker/docker/issues/23880#issuecomment-236184953

Thanks @nylon22, let me close this one then, so that we can have the discussion in a single place


Understanding migration from one sharded cluster to another

Hi,
I am trying to use mongo connector to migrate from one sharded database to another. A few clarifications needed

Does the migration process track the oplog from each shard and do a diff migration from each shard?
If the target cluster is empty initially will it work or does the user need to go in and manually recreate the whole sharding architecture so that the oplog syncing can work on a shard level?

It will also be good to have this documented in the wiki.

mongo-connector tails the oplog of each shard separately. I'm not sure what you mean by a "diff migration," though.
mongo-connector will work if the target is empty. When you provide the "target URL" (-t), provide the URL for one or more mongos, and the router will take care of routing the documents to the right shards. You'll need to set up sharding on the target yourself manually.

Just to be clear: mongo-connector doesn't create a "mirror image" of a sharded cluster. In other words, mongo-connector just listens to one sharded cluster, then does the same operations on the destination mongos router and has no knowledge that the destination is sharded.

Hi Luke,
Thanks. This helps a lot. How does the mongo-connector have credentials to tail each of the oplog? Also does the connector automatically figure out how many shards are present? In our case the shard credentials (used to connect to mongos) are different from the replica set credentials.
I would like to us a technique similar to the replica set migration you provided earlier in a previous bug. Please let me know if this will work

Start the mongo connector with "--nodump" and record the latest oplog entry
MongoDump from the source
Mongorestore on the destination
Configure sharding on the destination
Restart the mongo connector

The only question I have now is how to do 1. for each shard.

Mongo Connector uses the same credentials to authenticate to the shards as
to the mongos. I'll open another ticket as a feature request to add the
capability to specify shard credentials separately. The number and URIs of
the shards are determined automatically from the config server.
The strategy you outline seems fine to me, though you might want to
configure sharding before running "mongorestore", so that the restored
documents can be balanced by mongos during the process.
On Mon, Jul 20, 2015 at 10:19 AM, Dharshan Rangegowda <
<EMAIL_ADDRESS>wrote:

Hi Luke,
Thanks. This helps a lot. How does the mongo-connector have credentials to
tail each of the oplog? Also does the connector automatically figure out
how many shards are present? In our case the shard credentials (used to
connect to mongos) are different from the replica set credentials.
I would like to us a technique similar to the replica set migration you
provided earlier in a previous bug. Please let me know if this will work

Start the mongo connector with "--nodump" and record the latest
oplog entry
MongoDump from the source
Mongorestore on the destination
Configure sharding on the destination
Restart the mongo connector

The only question I have now is how to do 1. for each shard.
—
Reply to this email directly or view it on GitHub
https://github.com/10gen-labs/mongo-connector/issues/289#issuecomment-122957417
.

Hello @dharshanr, has this answered your questions?
I'm going to close this issue due to inactivity but if you still have more questions or are experiencing issues feel free to reopen or file a new issue.
Thank you!


Feat: add transactional db for units of work

Description
The decorator prevents CRUDS from committing changes in the database and commit everything when everything was resolved. It allows endpoints that need either all or none CRUDS to be done when committing
Checklist

[X] Created tests which fail without the change (if possible)
[x] All tests passing
[x] Extended the documentation, if necessary

Superseded by #498


Add user_filter & gitlab_default_theme configuration option

The travis build does not seem to be failing due to this patch but more because of an installation issue.

Can this be re-tested somehow?
The test failed due to:

"WARNING: The following packages cannot be authenticated!", "  gitlab-ce"

Since the merging of #105, these extra settings, which are slightly less common, could be set via that PR's new variable.

Thanks Jeff, that makes sense. Probably best to always handle uncommon options that way. :)


[openssl] Build error

Host Environment

OS: Linux
Compiler: GNU 9.3.0
vcpkg-tool version: 2022-06-15-b0f08e40f8e8752dd0b260138ee078bd9599559c
vcpkg-scripts version: 91393faf1 2022-06-16 (11 hours ago)

To Reproduce
Steps to reproduce the behavior:
./vcpkg install sleepy-discord
Failure logs

issue_body.md

The error message says
CMake Error at scripts/cmake/vcpkg_find_acquire_program.cmake:592 (message):
  Could not find pkg-config.  Please install it via your package manager:

      sudo apt-get install pkg-config

Have you done it?

@KiNa738 , Thanks for postinf this issue, please run command "sudo apt-get install pkg-config" and try again


Firefox Color: A Firefox Test Pilot Experiment

Firefox just launched a Test Pilot experiment called Color allowing much more customizable recoloring than stock Firefox. It's only being tested right now, but I figured I would let you know about it. It would eliminate some basic code.

Thanks for the heads up @lhindir! I'll check out.
It would definitely help to remove some of the code base - my only initial concern would be adding on an additional step to setting up ShadowFox, but definitely worth looking into :)

I like the idea behind Color and think it's a great way to go for some people, but since users would still have to install ShadowFox, I would rather keep everything together in one install instead of adding another step.
Thanks for letting me know about it though!


CB-16053 Change the Environment entity to EnvironmnetView during the …

…env deletion. The Query updated to fetch the network as well because it caused n+1 problem in every fetching.
See detailed description in the commit message.

retest this please


Restore-DbaDatabase - Add new parameter "-DbOwner" to switch login context before restore

Summary of new feature
Add a new parameter to Restore-DbaDatabase that would switch context to another login before the Restore.
Scenario:
I have hundreds of databases in Availability Groups.

The version is 2014, so there's no Automatic Seeding.

Plus, there are no shared folders that can be used between instances.

The requirement is that all databases must have "sa" as the database owner

So my procedure to automate adding a database (like a new SharePoint "ContentDB") to an existing availability group is basically the following in T-SQL:

Backups on the primary replica
Copy backups files to the secondary replica
On secondary: EXECUTE AS LOGIN = 'sa' + RESTORE.... WITH Norecovery.
Add database to AG

The "EXECUTE AS LOGIN" makes sure the database on the secondary replica is not restored using my domain account.
I can't afford to do a failover to the secondary after a database is added to the AG.
Using dbatools it doesn't seem possible to perform a restore using another login context like it is in T-SQL.
New parameter?
I would have loved to have a "-DbOwner" parameter on Restore-DbaDatabase to do this.
Would it be feasible to add such a feature?
Obviously, Set-DbaDbOwner can't alter a database in Restoring state.
PS C:\> Set-DbaDbOwner -SqlInstance $Instance -Database $Db -TargetLogin 'sa' -Verbose

WARNING: [16:39:40][Set-DbaDbOwner] Database [$Db] is not accessible. Skipping.

I hope that makes sense. Thanks.
Latest version of dbatools as of writing
1.0.133

is sa on the destination disabled ? because if it isn't, you could just connect to the dest with sa credentials instead of your own.
That being said, I totally get the usecase, although I'm not so sure on how to achieve that with SMO restore.
pinging @Stuart-Moore , aka the god of restore, as I feel there's nothing native in microsoft.sqlserver.management.smo.restore for this and we may have to tinkle with the generated script within Invoke-DbaAdvancedRestore, then surface the new parameter up to all functions using it.

is sa on the destination disabled ? because if it isn't, you could just connect to the dest with sa credentials instead of your own.
Most of the time sa is enabled, but what do you mean ? I can't run a PowerShell script with sa credentials right ?
I understand it's not an easy feature to add. For this usecase I can either stay with my PowerShell script that generate T-SQL (execute with login = 'sa'... restore) so without dbatools. Or I could have an Agent Job to change the owner after a Failover occurs.

Is there any reason you cannot simply pass in the SqlCredential when you call Restore-DbaDatabase? That would be the proper way of doing this from PowerShell's perspective.
Restore-DbaDatabase -SqlInstance YourReplica -SqlCredential $yourSaAccount ...

We use SMO Microsoft.SqlServer.Management.Smo.Restore class for restore processing and there is no viable option for us to add in an EXECUTE AS LOGIN statement to that object.
If the use of the -SqlCredential is not an option and you still need to use the EXECUTE AS statement I would recommend using this process:

Backups on the primary replica
Copy backups files to the secondary replica
Call Restore-DbaDatabase using the -OutputScriptOnly parameter.
Insert the EXECUTE AS LOGIN = 'sa' statement into the script output (at the start)
Call the T-SQL code on replica

Something like this should work:
$script= Restore-DbaDatabase ... -OutputScriptOnly
$script = "EXECUTE AS LOGIN = 'sa';`n" + $script
Invoke-DbaQuery ... -Query $script

The use-case is absolutely valid, there is no other way to change the DB owner on a secondary AG database. There is also a workaround:
$server = Connect-DbaInstance -SqlInstance ...
Invoke-DbaQuery -SqlInstance $server -Query "EXECUTE AS LOGIN = 'sa'"
Restore-DbaDatabase -SqlInstance $server ...

But it would be nice to add this to the command so that this connection dance won't be necessary.

Doesn't work like that. The SMO connector appears to issue a default REVERT after each query:

As you can see, the only way it works when the EXECUTE AS is in the same query. This can't be done with an SMO restore as there is no way of issuing the EXECUTE AS as part of the restores.
The only way to get around this would be to run the first part of the restore as a script rather than as the current SMO.restore call, so that's what I did. Pester  test included to ensure it doesn't get undone in future

Perhaps I'm missing a crucial part: the WITH NO REVERT statement.
https://docs.microsoft.com/en-us/sql/t-sql/statements/execute-as-transact-sql?view=sql-server-ver15#using-with-no-revert

Thank you all for your feedback, I didn't try the workarounds you guys proposed yet but I will do for sure.
Thank you @Stuart-Moore for working on the issue.
Would it be possible to have the "sa" account name as a parameter?
In some places where I do consulting they rename all their sa accounts to something like "SA_NOT_USED" for example. In this case, the login is also disabled but it still works fine with EXECUTE AS LOGIN.
Another way to do it would be to retrieve the name querying sys.syslogins as the SA account always has the same SID. I don't if it's better like this.

To allow passing in a login name @Stuart-Moore I'd think it best to validate the login exists. Not sure if the error returned if it did not would be clear?

@nvarscar that doesn't work:

Appears SMO does an implicit revert after each query, so combining them into a single query statement makes sure it works

@Stuart-Moore Okay, so this one works. I'm not sure if there were changes to invoke/query methods (those are custom methods provided by dbatools), but it seems they use their own connection now, at least I can't explain it any other way.
[220.0ms] Users/kkravtsov> Invoke-DbaQuery -SqlInstance $c -Query 'EXECUTE AS login = ''sa'' WITH NO REVERT;'
[104.0ms] Users/kkravtsov> Invoke-DbaQuery -SqlInstance $c -Query 'SELECT SUSER_NAME()'

Column1
-------
sa

[ 49.0ms] Users/kkravtsov> Invoke-DbaQuery -SqlInstance $c -Query 'EXECUTE AS login = ''testlogin'' WITH NO REVERT;'
[ 33.0ms] Users/kkravtsov> Invoke-DbaQuery -SqlInstance $c -Query 'SELECT SUSER_NAME()'

Column1
-------
testlogin

[ 30.0ms] Users/kkravtsov> Invoke-DbaQuery -SqlInstance $c -Query 'EXECUTE AS login = ''sysadmin'' WITH NO REVERT;'
WARNING: [15:36:34][Invoke-DbaQuery] [localhost] Failed during execution | Cannot execute as the server principal because the principal "sysadmin" does not exist, this type of principal cannot be impersonated, or you do not have permission.


Add a shared inbox

Sounds like this could significantly reduce the amount of AP traffic we get, notably Delete activities: https://www.w3.org/TR/activitypub/#shared-inbox-delivery

Nothing yet. 😠

Argh, I got the actor AS2 wrong, sharedInbox needs to be inside an endpoints mapping.

It's working!


Support real time message time

It would be nice if the message time in the chat were real-time, you can archive that using a library like timeago.js, they support real-time render.

great idea @DannyFeliz, we'll check it out :)


Get-FolderAccessList, Get-Item, and the root share of UNC paths

$Permissions = Get-FolderAccessList @AclParams -FolderTargets $FolderTargets
    # If $ThisTargetPath was on a local disk such as C:\
    # The Get-FolderTarget cmdlet has replaced that local disk path with the corresponding UNC path \\$(hostname)\C$
    # If $ThisTargetPath is the root of that local disk, Get-FolderAccessList's dependency Get-Item cannot retrieve a DirectoryInfo object for the root of the share
    # (error: "Could not find item")
    # As a workaround here we will instead get the folder ACL for the original $ThisTargetPath
    # But I don't think this solves it since it won't work for actual remote paths at the root of the share: \\server\share
    if ($null -eq $Permissions) {
        Write-LogMsg @LogParams -Text "Get-FolderAccessList -FolderTargets '$ThisTargetPath' -LevelsOfSubfolders $SubfolderLevels"
        $Permissions = Get-FolderAccessList @AclParams -FolderTargets $ThisTargetPath
    }

Cannot reproduce.  Get-FolderAccessList dependency on Get-Item appears to have been removed


.AddGitHubWebHooks(); custom secret value

.AddGitHubWebHooks(); seems to internally use appsettings.json.
It would be nice to get the value directly myself and pass it in as a parameter because I don't want to use appsettings.json.
Thanks!

This issue was moved to aspnet/AspLabs#36


Make text cleaning optional.

Is your feature request related to a problem? Please describe.
The cleaning of the text makes it impossible to link annotated spans to the character indices of the original text. This in turn makes it impossible to compare the performance of this model to other ner models.
Describe the solution you'd like
Make the text cleaning step optional. When the cleaning step is omitted, then abv_text == immutable_text.
Describe alternatives you've considered
Provide additional metadata containing the start and end character indices of each annotated span linked to the original text rather in addition to the boundaries linked to the cleaned text

You could instantiate your own empty skillNer.cleaner.Cleaner to bypass text cleaning. However you also want to protect abv_text from later processing, which would require some changes in the code.


Execution Reverted in AddEnode from NodeRules.sol

Here is the error
{
    "code": -32000,
    "message": "Execution reverted",
    "data": "0x08c379a00000000000000000000000000000000000000000000000000000000000000020000000000000000000000000000000000000000000000000000000000000002a6f6e6c7920746865206c617465737456657273696f6e2063616e206d6f6469667920746865206c69737400000000000000000000000000000000000000000000"
}

@fahadh4ilyas Can you provide further details eg what was the enode you were trying to add, any further info?

I already fix it in #260

This PR replaces #260 https://github.com/ConsenSys/permissioning-smart-contracts/pull/265

fixed!


TB config window removes  from the end of a removal reason.

http://gfycat.com/VainLegitimateBassethound

Fixed it. Just had to escape the removal reason when going in to the template


[Bug]: Airtable connections reset not handled correctly

Is there an existing issue for this?

[X] I have searched the existing issues

Description
When I use an Airtable datasource it seems that Appsmith doesn't handle connections to their API correctly. Randomly when Queries are run I get
recvAddress(..) failed: Connection reset by peer

as a response. Complete query object after query is:
{
  "isLoading": false,
  "data": "recvaddress(..) failed: connection reset by peer"
  "responseMeta":{
    "isExecutionSuccess": false
    "statusCode": "5000"
  }
}

Steps To Reproduce

create an Airtable base
add it as a datasource
run a lot of list queries to produce the error

Public Sample App
No response
Environment
Production
Issue video log
No response
Version
Cloud

I'm having the exact same problem using the Airtable datasource.

I found the same issue recorded in video in Appsmith own YouTube channel (2m19s): https://www.youtube.com/watch?v=9aQe18DPYXI
I'm an Airtable heavy user and the only application that faces these instabilities is Apssmith. Airtable API never fails when executed in Google Colab, for example.

@rohan-arthur do we know the cause for this yet? Is there a workaround with the API, instead of our native integration?

update:

we are not able to replicate this error
had a call with one of our users, and we are waiting for them to share the server logs

meanwhile, if anyone reading this is facing the problem, please extract the server logs and share it with<EMAIL_ADDRESS>quoting this issue link.
how to extract server logs: https://docs.appsmith.com/getting-started/setup/instance-management/how-to-get-container-logs#docker

down-grading this to High, we will continue to watch this issue for updates about reproduction/user logs


feat(RingTheory/Congruence): split RingCon construction and add some constructions

added results are mainly about RingCon on the opposite ring

bors r+


mungegithub: Create an options structure instead of individual variables

Makes adding new command line args which get passed to mungers much more sane.

should be rebased after #36


refactor(AnonCred): refactor get getSchema method in PolluxImpl

Overview
This PR updates a piece of the logic to getSchema to support the latest changes to ZKP verification.
Fixes #
Checklist
My PR contains...

[ ] No code changes (changes to documentation, CI, metadata, etc.)
[ ] Bug fixes (non-breaking change which fixes an issue)
[X] Improvements (misc. changes to existing features)
[] Features (non-breaking change which adds functionality)

My changes...

[X] are breaking changes
[ ] are not breaking changes
[ ] If yes to above: I have updated the documentation accordingly

Documentation

[X] My changes do not require a change to the project documentation
[ ] My changes require a change to the project documentation
[ ] If yes to above: I have updated the documentation accordingly

Tests

[X] My changes can not or do not need to be tested
[ ] My changes can and should be tested by unit and/or integration tests
[ ] If yes to above: I have added tests to cover my changes
[ ] If yes to above: I have taken care to cover edge cases in my tests

Code Coverage

File
Coverage [43.33%]

atala-prism-sdk/src/commonMain/kotlin/io/iohk/atala/prism/walletsdk/pollux/PolluxImpl.kt
43.33%

Total Project Coverage
25.26%


Can't Mount Virtual Drive (Vhdx)

Hello Everyone,
Today I mount my vhdx. When I click Files prompt for admin(filesfulltrust.exe) I give permission but after that It gives me error :There was a error mounting the file (See Screenshot). I tried many times even reboot but error remains same. The mounting works from file explorer. I think same problem with vhd also I had not tested it

I also noticed one more issue there is no eject option for virtual drives(e.g. vhdx) like file explorer.
So please fix issue and add option for virtual drives.

Thanks for this report. Atm Files does not support operations that require admin access. We'll check if there's a workaround for mounting vhds (e.g launching an elevated powershell)

Still happens on 1.5.32. Any update on this?

This issue should be resolved with #5206.


Obscure bug: Only get warning when using chart as a subchart

I only get this warning:

coalesce.go:199: warning: cannot overwrite table with non table for myfield (map[])

when I use mychart as a subchart of another chart. If I use mychart directly, I don't get the warning.
I just wanted to report it so it is out there. Feel free to close it if it is too obscure :-)
Details
The warning occurs because in the chart's values.yaml, I have myfield: {} that I override with an array. But I only get the warning when used as a subchart.
Say I have a chart that contains mychart as a subchart.
In mychart's values.yaml, I have:
myfield: {}

Because I made a mistake, I'm using it like this overriding the object/hash {} with an array:
$ cat custom-direct.yaml
myfield:
  - 1
  - 2
$ helm template foo -f custom-direct.yaml charts/mychart > /dev/null

That is of course wrong, but the interesting thing is that there is no STDERR warning output here.
If I now do it in the parent chart:
$ cat custom-parent.yaml
mychart:
  myfield:
    - 1
    - 2
$ helm template foo -f custom-parent.yaml . > /dev/null
coalesce.go:199: warning: cannot overwrite table with non table for myfield (map[])

notice that now I do get a warning, when asking the parent chart to render mychart as a subchart.
I expect to always get the warning. Not only when I use mychart as a subchart. The annoying thing is that no amount of unit testing of mychart will catch this warning until it is used in a larger system.
I've attached coalesce-bug.tar.gz containing the charts and files used above.
$ helm version
version.BuildInfo{Version:"v3.7.1", GitCommit:"1d11fcb5d3f3bf00dbe6fe31b8412839a96b3dc4", GitTreeState:"clean", GoVersion:"go1.16.9"}

$ kubectl version
Client Version: version.Info{Major:"1", Minor:"21", GitVersion:"v1.21.4", GitCommit:"3cce4a82b44f032d0cd1a1790e6d2f5a55d20aae", GitTreeState:"clean", BuildDate:"2021-08-11T18:16:05Z", GoVersion:"go1.16.7", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"19", GitVersion:"v1.19.8", GitCommit:"fd5d41537aee486160ad9b5356a9d82363273721", GitTreeState:"clean", BuildDate:"2021-02-17T12:33:08Z", GoVersion:"go1.15.8", Compiler:"gc", Platform:"linux/amd64"}
WARNING: version difference between client (1.21) and server (1.19) exceeds the supported minor version skew of +/-1

K8s provided by Nutanix Karbon
Running in Ubuntu-20.04 under WSL2 on Windows 10.

@pmorch I will try to solve it. Thanks very much.

@pmorch I will try to solve it. Thanks very much.

Hey, any updates?

I don't think this has been resolved, so bump to remove stale label.

@pmorch Oh. Sorry. I will update it.

Ping to avoid going stale.
On Mon, Nov 21, 2022 at 1:15 AM github-actions[bot] <
@.***> wrote:

This issue has been marked as stale because it has been open for 90 days
with no activity. This thread will be automatically closed in 30 days if no
further activity occurs.
—
Reply to this email directly, view it on GitHub
https://github.com/helm/helm/issues/10265#issuecomment-1321287743, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AABG4PAIO7G47XG6O3SLAZLWJK5LNANCNFSM5GPD3MNA
.
You are receiving this because you were mentioned.Message ID:
@.***>

It looks like this happens only with subcharts because this bit of the dependencies merge and upgrades are the only actions that use the CoalesceTables function.
To get this in template (or install) some additional check would need to be made that isn't being made today. I'm not sure where that would live, or what that would look like.


Update to new build API

Recent (yesterday) changes to the build.zig API require swapping artifact.install with b.installArtifact(artifact) etc.

Thank you!


keypunch: 4.0 -> 5.0

Changelog: https://github.com/bragefuglseth/keypunch/releases/tag/v5.0
Diff: https://github.com/bragefuglseth/keypunch/compare/v4.0...v5.0

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@ofborg build keypunch keypunch.passthru.tests


[AutoPR sql/resource-manager] Add version 2018-06-preview version to readme.go in sql

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/7150

This PR has been merged into https://github.com/Azure/azure-sdk-for-go/pull/5720


[WIP] Estimating Memory Overhead Before Domain Creation

This work-in-progress PR introduces a new approach to enhance memory management during the creation of new domains in the ZedManager component. The key idea is to estimate the memory overhead required by a new domain before actually initiating its creation. This ensures that sufficient memory is available on the device to accommodate the domain, taking into account both its memory requirements and overhead.
Key Changes

Memory Overhead Calculation: Implemented in updatestatus.go using the CountMemOverhead function from the hypervisor package.
Memory Usage Accounting: Modified getRemainingMemory in memorysizemgmt.go to include memory overhead in the computation of used memory size.
Hypervisor Package Extensions: Adapted various hypervisor files to support memory overhead estimation for different types of hypervisors.
Assignable Adapters Integration: Significantly, this PR adds information about assignable adapters into the zedmanagerContext. A new subscription for these adapters has been created and integrated within zedmanager.go. Handling their updates is essential for accurately estimating the memory overhead.

Purpose of this PR
The primary purpose of this PR is to gather feedback on the implemented approach. Given the complexity and impact of these changes on memory management, it’s crucial to ensure that the methodology is sound and aligns well with the existing architecture.
Request for Comments

Approach Validity: Does this approach effectively address the issue of estimating memory overhead? Are there any potential pitfalls or scenarios that haven't been considered?
Code Implementation: Any suggestions for improving the code changes, in terms of efficiency, readability, or adherence to project standards?
Integration with Existing Systems: How well does this new feature integrate with the existing components and functionalities of the ZedManager and hypervisor packages?
Testing and Validation: Recommendations for thorough testing strategies to validate this feature under various scenarios and workloads.

Note
This is a work-in-progress and should not be merged until it has been thoroughly reviewed and all comments have been addressed.

I have not tested it yet, just made the code buildable

I have an overall behaviorial question: does the accounting of memory in getRemainingMemory() for an app instance change as the app instance is running, or is it fully defined by the assigned adapters, type of hypervisors, and the amount of memory allocated to the VM/container?
Reason I'm asking is that we want to make sure that if a user can deploy three app instances in order A, B, C without getRemainingMemory showing that there isn't enough for C, this should be stable independent of order that the those app instances are deployed and/or booted.

does the accounting of memory in getRemainingMemory() for an app instance change as the app instance is running, or is it fully defined by the assigned adapters, type of hypervisors, and the amount of memory allocated to the VM/container?

My current understanding is that the MemOverhead values calculated are static. The MemOverhead function estimates the potential memory overhead for a VM based on the configuration and list of assignable adapters. This estimation is done once for each app instance and does not change dynamically during the app's runtime. @rucoder can provide some more explanation here.
While the MemOverhead itself is static, the way getRemainingMemory() calculates available memory does change with this update. getRemainingMemory() is dynamic, assessing the system's memory availability in real-time. The PR's changes affect this function by adding the static MemOverhead value to the total memory requirement calculation for an app instance. This means that while the overhead value is a constant once calculated, its inclusion is a new factor in the dynamic calculation performed by getRemainingMemory(). However, the "variation" part is not changed within this PR.
In summary, the PR introduces a static memory overhead estimation into the dynamic memory availability calculation performed by getRemainingMemory()

This should be a very good work, which fixes incorrect free app memory estimation. @OhmSpectator I added a few comments requesting go deeper with the fix (not just memory overhead calculations). Feel free to address these request in the following PRs.

In the CountMemOverhead, use explicit params instead of DomainConfig

Fixed typos in log messages and the Yetus errors

LGTM. It might make sense to add your explanation e.g., in pkg/pillar/docs/zedmanager.md or some other place.
Also, do we know whether these changes will cause us to fail to boot something which currently boots (but might result in an oom later.) It would be good to understand that so that we know whether some users might see an immediate failure with these changes where they today can run things but be at risk of oom.

This PR looks very nice @OhmSpectator, thanks. I agree with @eriknordmark regarding add more documentation about it. Also, make sure to test it on ARM devices to avoid any surprises.... I can help with that, just let me know...

It might make sense to add your explanation e.g., in pkg/pillar/docs/zedmanager.md or some other place.

@eriknordmark, do you mean the description of the memory types taken into consideration when making a decision on Domain activation, from https://github.com/lf-edge/eve/pull/3588#discussion_r1396008557 and https://github.com/lf-edge/eve/pull/3588#discussion_r1396086943 ? I think it would make sense to update documentation separately after we agree on the way we handle "latent" memory.

Also, do we know whether these changes will cause us to fail to boot something which currently boots (but might result in an oom later.)

Based on my understanding, this issue may only impact situations where users allocate nearly all of the device's memory to running applications. To my knowledge, there is only one such case, and this PR is a direct response to that situation.

I started to port the changes to 10.4 and realized that that version does not contain the changes done by @rucoder for better overhead estimation. Neither contains them 9.4.
So, I'm going to backport only the fix now.


[h264enc] Make max motion search range adjustable to improve encoding speed

My tests showed that the motion search range makes the biggest impact on the H.264 encoder speed. For example: 16 (default) - 7.7 fps, 8 - 12.1 fps, 4 - 16.3 fps, 2 - 19.7 fps, 1 - 21.6 fps.
Unfortunately this parameter is currently hardcoded. This pull request adds a getter and setter for it.

@JohnA2 thanks


pub sub

closed via https://github.com/hstreamdb/hstream/pull/59

closed via https://github.com/hstreamdb/hstream/pull/59


pull request for e2e test at 12/04/2016 15:26:00

:white_check_mark: Validation status: passed
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.


Typed version of the library

Is there a plan for the typed version of this library?  If not, I will be glad to contribute

it'd be nice if you sent in a PR for this, it'll help a lot 🤝

Sure, but I can't do that alone. This will require something like a roadmap to make the contribution seamless

is this issue still open?

Yes
On Fri, Jan 5, 2024, 13:11 Mwelwa @.***> wrote:

is this issue still open?
—
Reply to this email directly, view it on GitHub
https://github.com/Flutterwave/Node-v3/issues/139#issuecomment-1878570526,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMSQ2BCOVIRZNQNB3J4CXQLYM7UXBAVCNFSM6AAAAAA6NX5POSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZYGU3TANJSGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

it's counter-intuitive to me that a company like FW would have no typed version

Hello everyone👋🏾, Thank you all for volunteering to contribute.
The team is working on a roadmap and contribution guidelines to better manage this library. We would absolutely love everyone to contribute here and would communicate the roadmap soon.

Any progress on the roadmap and contribution guidelines

Hello Everyone 👋  (including the Flutterwave Node Team)
I was planning on integrating Flutterwave into one of my projects as well and was a bit sad that there wasn't typescript support or type sensing.
In order to address this, as well as your concerns, I was able to fork the repository and begin working on adding types to the library.
The library is a big one, so I am starting with making use of JSDoc and generating .d.ts files from them. That way, adding types to the library can be done incrementally, and eventually (with the help of the Flutterwave team), it can be implemented completely in TypeScript
The fork is on my repo: https://github.com/nikeokoronkwo/Node-v3
Let me know what you guys think, and maybe if the Flutterwave team is willing I can finish it up and make a pull request.
Contributions are very welcome, and I would be glad to get any support if possible for this.
@corneliusyaovi


Fairlead  Tension Output

Dear @jjonkman ,
Running the simulation will show the tension in the command line, but I would like to know how to output the  Fairlead  tension out with a scope inside Simulink, because I don't find a string about the tension in OutlistParameters.xlsx. I'm trying to output it out, but it reports an error.

Dear @WS991106,
FairTen1 is an output of MoorDyn if you've selected it in the OUTPUTS section of the MoorDyn input file.
The MoorDyn outputs are not documented in the OutListParameters.xlsx spreadsheet, but are documented in the online MoorDyn documentation: https://moordyn.readthedocs.io/en/latest/inputs.html#the-v2-input-file.
Best regards,

Dear @jjonkman ，
If I want to output the Fairlead tension inside simulink to a scope, do I need to modify the source code? If I need to modify it, what part of it should I modify? Is there any information to introduce it?
Best regards,

Dear @jjonkman，

I tried what you said, but I still get the error as below, I don't know what the problem is.

Dear @WS991106,
I'm not sure what this error says in English.  Regardless, is FairTen1 included in the OutData array?
Best regards,

Dear @jjonkman ,
Thank you very much for your help, my problem has been solved by changing the string all caps to FAIRTEN1 and it won't report an error!
Best regards,

@WS991106 -- I'm glad you solved the problem.
@RyanDavies19 @andrew-platt -- Sounds like a simple issue to fix in MoorDyn by making use a call to Conv2UC().
Best regards,

@jjonkman @andrew-platt I added a comment to the documentation to address this. For now I think its best to leave it as is, so that users who have scripts set up already don't have to make changes

@RyanDavies19 -- I'm not sure what scripts you are referring to, but using Conv2UC() would simply open up the possibility for users to select MoorDyn output FAIRTEN1 using any combination of upper or lower case letters, e.g., FairTen1.  This is how most other output parameters of other OpenFAST modules work and provides user some flexibility.

@jjonkman from the MoorDyn input file side, the output channels are not case sensitive. Users could give fairten1 and it would be correctly read. MoorDyn uses Conv2UC() before processing the output channel name and assigning it to the output parameter structure as all caps, so that the output file has headers in caps.
I discussed this with Matt and the idea is that any users who have set up their own scripts to process moordyn output files will have done so based on the all caps method, so in order to not break that convention its better to leave it as is. My understanding of the OutList variable is that it contains a list of all the output channels as written by MoorDyn, so I'm not sure if there is a way to allow for searching while still having the MoorDyn output file headers be caps.

I'm not sure we're all talking about the same issue here. I don't know how MoorDyn deals with the case of channel output names, so I won't comment on that. However, the issue here is likely that the output channel from MoorDyn (possibly using whatever case the user entered in the input file) does not have the same case as the Simulink block is expecting. The best fix is to change the Simulink block to ignore case:
The Simulink model should change u(strmatch('FairTen1',OutList)) to u(strcmpi('FairTen1',OutList)). Matlab documentation also says strmatch() should not be used anymore, so this fixes both issues.

Thanks for clarifying, @bjonkman.  That makes sense.  I'll go ahead and close this issue.

Updated the Simulink example cases with PR #2254.


[HP] missing focus state when tabbing through mega menu

Detailed description

Describe in detail the issue you're having.

There is no focus state appearing when tabbing through the mega menu content. Keyboard navigation works as expected in Storybook.
Recording of keyboard tabbing through mega menu on homepage staging:

Is this a feature request (new component, new icon), a bug, or a general
issue?

Bug

Is this issue related to a specific component?

Mega menu

What did you expect to happen? What happened instead? What would you like to
see changed?

What browser are you working in?

Chrome
Version 84.0.4147.125 (Official Build) (64-bit)
MacBook pro, osCatalina 10.15.5

What version of the IBM.com Library are you using?

Latest canary, seen on homepage staging site

What offering/product do you work on? Any pressing ship or release dates we
should be aware of?

Steps to reproduce the issue

Open homepage staging
Tab to select Products & Solutions
Hit enter key to open
Tab again to tab into mega menu content

Additional information

@oliviaflory This bug is ready to test.  Use the staging link.

Looks good @rsebade !

Also retested by Ram.  Fixed.  Closing.


User's Name is not displayed in conversation list for new conversations

When I initiate a new conversation via yakyak (or if I simply receive a message from someone that causes a new conversation), the name of the contact is not displayed.  Once I close and re-open the application, then the name is displayed.

+1'd

Bump.  👎 probot-stale ^ 👎

Bump. 👎 stalebot 👎


WLS2 UBUNTU 22.04

Hi
Describe the bug
make fails with message
Compiling pwg-driver.c...
Linking testmainloop...
Compiling testpappl.c...
testpappl.c: In function ‘test_client’:
testpappl.c:3426:3: warning: implicit declaration of function ‘httpReconnect’; did you mean ‘httpConnect’? [-Wimplicit-function-declaration]
3426 |   httpReconnect(http, 30000, NULL);
|   ^~~~~~~~~~~~~
|   httpConnect
Linking testpappl...
/usr/bin/ld: testpappl.o: in function test_client': /home/lelan/pappl/testsuite/testpappl.c:3426: undefined reference to httpReconnect'
collect2: error: ld returned 1 exit status
make[1]: *** [Makefile:66: testpappl] Error 1
make[1]: Leaving directory '/home/lelan/pappl/testsuite'
make: *** [Makefile:28: all] Error 1
Best regards

Would help to know what version of PAPPL you are trying to compile.
Note that PAPPL master requires the current libcups master (not the old beta release) to build.

Hi 1.4.6
i don't understand "the current libcups master (not the old beta release)" How to see what i use?

PAPPL is a library, not a command.
There is a test suite program you can use to play with PAPPL's capabilities (it is in the testsuite directory and is called testpappl) but all it does is convert print jobs to PWG raster and stores the resulting files in a directory. It normally is used to test the various functions of the PAPPL library and isn't meant as a program for users.

ah sorry! With snap, is it possible to access PAPPL logs?  I see device Epson AL C4100 but it doesn't work. Only PS3... lines written


Features request: Seedbox manager

Is there any chance you to integrate this script with seedbox manager? So user can restart seedbox using panel and have some additional info on panel.

Looks interesting. To be honest though I doubt I will have time for much new development in the foreseeable future. If some one else wanted to have a go would be great.


Calling details page increments invocation counter for flow

Every time the flow detail page is reloaded the invocation counter is increased, whereas it should only report real invocations

We will use the requests count, to get there requests
requests, err := listFlowRequests(flowName)
	if err != nil {
		log.Printf("failed to get requests, error: %v", err)
		requests = make(map[string]string)
	}

On success it returns a map, we can just use the Len
For error case we can just use 0

Fixed in: https://github.com/faasflow/faas-flow-tower/commit/8640e33b3013a8e681dbc129190ccb861159289d


Pass ResponseInterface and parsed body to checkResponse

Attempt to fix #338.
The problem that this is trying to solve:
It is not possible to detect an error when a service uses status codes to indicate failure rather than the response body. Currently, checkResponse only accepts the parsed body, so this PR proposes to also use the ResponseInterface alongside the parsed body. This allows a provider to utilise the status code or other metadata on the response to determine whether an error has occurred.
Note: I've also added checkResponse to getResponse - not sure if that's what we want though? Wasn't sure if checkResponse is an access token request thing only.

It looks like checkResponse() is a generic way for the provider to implement its own error-checking capabilities, so I think this works out well.


CI

We need to setup CI. Maybe try GitHub Actions for this?

I setup GH actions for this. Hopefully will also use it for builds and deploying to PyPI. I am leaving this open for now as I mentioned in #9 that after 21.3 is released we should also add CI testing against the released version of Sanic.

I setup GH actions for this. Hopefully will also use it for builds and deploying to PyPI. I am leaving this open for now as I mentioned in #9 that after 21.3 is released we should also add CI testing against the released version of Sanic.

:muscle: Version 0.2.1 was deployed using GH Actions.

:muscle: Version 0.2.1 was deployed using GH Actions.


Proposal: code snippet testing for docs

Our docs are full of code snippets, which would ideally be kept up to date with the code. One way to encourage this is with a CI check that extracts the code snippets, runs them, and checks for errors and/or compares with expected output. There are many ways to approach this, and factors to consider, including:

Multiple language support, potentially including our "request/response" blocks, pql, and toml
Multi-block sessions, maintaining state over an example that spans multiple code blocks
Tool development effort
Docs maintenance effort
Docs backward compatibility

After a brief discussion, the best option seems to be add a small amount of unobtrusive metadata to our existing markdown files, and write a custom tool to run the tests. I'll summarize the proposed process here.
Prerequisites
Several website tasks are soft blockers for this work, in that it would be beneficial to understand the constraints they force before starting on the work for this project. One is enabling syntax highlighting, which is minor and probably insignificant. Others are larger and more relevant:

Expanding the docs file and nav structure to allow multiple levels
Language tabs
General site redesign

These first two may are subject to Hugo constraints that we don't fully control. Therefore it is preferable to implement them first before starting work on the docs testing tool. Since the docs structure and language tabs work are also dependent on site redesign, we can not finish them in a reasonable time. Instead, I suggest we implement both of them as MVP prototypes without any (graphical) design effort - or at least revisit our research on them - to understand any constraints they may impose on the docs testing tool.
Docs changes
The markdown files will only need minor updates.
Metadata about the test session for each code block will be added above, as toml within an html comment. This will not affect the rendered output at all, nor will it cause any backward compatibility issues with older docs versions. metadata will include sessionid:<int>, with the value starting at 0 and incrementing within each file. Unique IDs might be more practical (see prerequisites section). Any other toml syntax could be supported, but prefer minimal metadata.
Example:
<!-- sessionid:0 -->
```request
curl foo
```
```response
bar
```

<!--
sessionid:1
-->
```bash
curl localhost:10101/index/foo -XPOST
```

<!--
sessionid:1
-->
```go
response, err = client.Query(myindex.BatchQuery(
		myfield.Row(5),
		myfield.Row(10)))
```

Any sessionid:<int> line corresponds to the code block that follows. All code blocks that share a sessionid are run in sequential order, with state maintained between them. Ideally this would include any variables, changes made in a shell, and changes made to a Pilosa server. Any output code block corresponds to the preceding code block, similar to the way request/response currently work. request/response could potentially be replaced with shell/output, in case that simplifies anything.
Some code blocks currently have no language specified, while some have request/response. This may be a minor abuse of the info string, normally used in Hugo (and other tools) to determine the language for syntax highlighting. As long as the info string is set to the correct language whenever necessary, this doesn't seem to be a problem, practically.
I suggest all code blocks be tagged, either with a proper language, or some other identifier. Looking through the existing untagged blocks, they could all be handled with these four identifiers:

"bash" (19x, for testing pilosa subcommands, environment setup, etc)
"output" (4x, for comparing results of previous block)
"spec" (16x, for query-language.md (this could just as well be "ignore" for now))
"ignore" (55x, code that can't be reasonably tested, e.g. installation, docker commands)

Alternatively, the "spec" and "ignore" blocks can be left blank, which will indicate the blocks should be ignored.
Docs build process
No change for this approach. Additional steps might be necessary, if we moved docs to org files or code files. Note that Hugo does support org directly (as does github), but it may not be as mature as markdown.
Test runner tool
The tool, written in go, will read each markdown file, naively parse html comments and markdown code fences, and create sessions and "blocks" (one per markdown code block). A session consists of multiple blocks, all of the same type. Each session may require some setup, such as a fresh Pilosa server, or a clean working directory. For a language session, variables should be maintained across code blocks (low priority).
The tool will be designed to handle new types of code blocks in a modular way, so it can be built to handle only one type (request/response or bash/output) at first, then expanded as necessary.
As this tool is dependent on the rest of the process, I'd like to see some comments on that before developing it much further.
Numbers
Currently, the docs have 321 total code blocks, split up like this:

tag
#

request
95

response
94

bash*
19

go
5

ignore*
55

java
1

json
1

output*
4

pql
3

python
2

spec*
16

sql
1

toml
27

yaml
1

* = proposed, currently untagged

file
#

administration.md
11

api-reference.md
23

client-libraries.md
6

configuration.md
24

console.md
2

data-model.md
10

examples.md
20

getting-started.md
27

installation.md
32

pdk.md
1

query-language.md
73

tutorials.md
92

I made an attempt at this, and unsurprisingly found that there were too many special cases to be able to support this properly. The point was to automatically check that code snippets run as expected, but the amount of effort to maintain this test runner defeats the purpose.


Better dashboard

Dashboard that knows the sites, their domains/aliases, current state, and can run common commands against it.

Looking at Laravel as possible system for this.


Update Free Credit Program Terms of Service [SATURN-1284]

Broad Legal has asked us to make the updates contained in this google doc immediately to the modal notification that pops up when a Terra user clicks “Sign up for free credits” from the Terra UI hamburger menu.
I tested this by comparing screenshot of the desired language in the ticket and they look identical to me.
Ticket here.

Ready for re-review per getting access to doc with edits.

In testing, I notice that the text in the modal scrolls now because it's longer. It's not much extra height. Looks like it's currently 500px and bumping it to 600px should do the trick.


Write a post about the obsessive statelessness in Peril

lol, not even artsy work, but Artsy ideas have played an important part in it

Love the concept!

label states


Uncaught Error: error: 126/?~\AppData\Local\atom\app-1.30.0\resources\app.asar.unpacked\node_mo...

[Enter steps to reproduce:]

...
...

Atom: 1.30.0 x64
Electron: 2.0.5
OS: Unknown Windows version
Thrown From: Atom Core
Stack Trace
Uncaught Error: error: 126\?\C:\Users\Hector De los Santos\AppData\Local\atom\app-1.30.0\resources\app.asar.unpacked\node_modules\keyboard-layout\build\Release\keyboard-layout-manager.node
At ELECTRON_ASAR.js:172

Error: error: 126\\?\C:\Users\Hector De los Santos\AppData\Local\atom\app-1.30.0\resources\app.asar.unpacked\node_modules\keyboard-layout\build\Release\keyboard-layout-manager.node
    at process.module.(anonymous function) [as dlopen] (ELECTRON_ASAR.js:172:20)
    at Object.Module._extensions..node (module.js:671:18)
    at Object.module.(anonymous function) [as .node] (ELECTRON_ASAR.js:186:18)
    at Module.load (module.js:561:32)
    at tryModuleLoad (module.js:504:12)
    at Function.Module._load (module.js:496:3)
    at Module.require (/app.asar/static/index.js:47:45)
    at require (internal/module.js:11:18)
    at customRequire (~/AppData/Local/atom/app-1.30.0/resources/app/static/<embedded>:96:26)
    at get_KeyboardLayoutManager (~/AppData/Local/atom/app-1.30.0/resources/app/node_modules/keyboard-layout/lib/keyboard-layout.js:7:65)
    at get_manager (~/AppData/Local/atom/app-1.30.0/resources/app/node_modules/keyboard-layout/lib/keyboard-layout.js:16:42)
    at Object.getCurrentKeyboardLayout (~/AppData/Local/atom/app-1.30.0/resources/app/node_modules/keyboard-layout/lib/keyboard-layout.js:32:17)
    at exports.keystrokeForKeyboardEvent (~/AppData/Local/atom/app-1.30.0/resources/app/node_modules/atom-keymap/lib/helpers.js:201:42)
    at KeymapManager.module.exports.KeymapManager.keystrokeForKeyboardEvent (~/AppData/Local/atom/app-1.30.0/resources/app/node_modules/atom-keymap/lib/keymap-manager.js:467:20)
    at KeymapManager.module.exports.KeymapManager.handleKeyboardEvent (~/AppData/Local/atom/app-1.30.0/resources/app/node_modules/atom-keymap/lib/keymap-manager.js:347:30)
    at WindowEventHandler.handleDocumentKeyEvent (~/AppData/Local/atom/app-1.30.0/resources/app/src/window-event-handler.js:110:40)

Commands

Non-Core Packages

Thanks for contributing!
We noticed that this looks like a duplicate of #14461 so you can subscribe there for updates if you'd like.  Also, if you have an anti-virus program running you can try temporarily disabling it and then reinstall Atom to see if that helps.

Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.


Export of entityToJSONSchema broken

Link to reproduction
none
To Reproduce
Just look closely at the src/utilities/configToJSONSchema.ts file to see that the entityToJSONSchema is not exported.
Describe the Bug
entityToJSONSchema is explicitely exported from payload/utilities. In a recent change due to an internal refactor, configToJSONSchema was moved to the same module and entityToJSONSchema is no longer exported from the module.
Payload Version
1.10

@teunmooij I see what you mean, I'm on it!

@jacobsfletch , thanks. I've already created a PR

Aha, very good! I'll get that merged up and it'll go out in the next release 🙌

Awesome stuff, thank you both @teunmooij and @jacobsfletch for the rapid response ❤


fix helm value pullPolicy mismatch bug

change all imagePullPolicy to pullPolicy for consistency

Agree on the version bump, but this is an incompatible change, so at least a minor version bump, not patch please.

Agree on the version bump, but this is an incompatible change, so at least a minor version bump, not patch please.

Interesting, I need to learn more about incompatible changes. @kfox1111 poking you for a quick explanation of why this is an incompatible change?

The helm folks convention likes to stick closish to semver but seem to have relaxed it a little bit in practice...
They usually do:
Major is large change.
Minor is a technically breaking change, though very minor. So, like this, changing the pull policy variable is a user facing breaking change, but its tiny and won't effect most people.
Patch, totally safe to go from one patch release to another without the user noticing.
Proper semver interpretation would be to bump the major version for this change as it really is an api change. That is technically the right thing to do.

pullPolicy is the only key used in current two values.yaml file
Helm template engine will produce an empty imagePullPolicy:  for those *.yaml files.
Then k8s automatically set it to IfNotPresent, because it is the default pull policy in k8s.
No matter what value user sets for pullPolicy, no effect.

pullPolicy is the only key used in current two values.yaml file
Helm template engine will produce an empty imagePullPolicy:  for those *.yaml files.
Then k8s automatically set it to IfNotPresent, because it is the default pull policy in k8s.
No matter what value user sets for pullPolicy, no effect.

Considering this we would need to update the patch number at least? @MiaoZhou @kfox1111
Interestingly I hit the same issue yesterday, and yes it is defaulted to IfNotPresent when the variable is missing from the template.

Oh, I didn't realize that the issue was the api was actually broken (values not matching whats used in the templates). I was thinking it was a rename of the values. As it never worked, there shouldn't be users of it, so its not a breaking change to actually fix it. So bumping the patch version might be considered ok as its a pure bug fix. Or you could look at it as adding a new feature that didn't exist before (because it was broken) and then it would be a minor version bump. I'm good with either.

@kfox1111 thanks @MiaoZhou let's go with a minor version bump (so it can be pushed to helm hub)

@MiaoZhou request a port of the changes to master branch as well or, let me know and I can ensure the same. Thanks!

hmm.... what is the difference  between master and csi-v1.0 at this point?

master is the forward looking branch, and csi-v1.0 is the current 1.0.0 branch. So master needs to have all fixes, for future releases to not miss out any.
At this point afaik functionally the both should be equal, but 2 PRs are missing against master.
Issue to change the default branch to master is still open #308

@ShyamsundarR got it, i'll create another PR.

@ShyamsundarR got it, i'll create another PR.
@MiaoZhou this is taken care by https://github.com/ceph/ceph-csi/pull/363


Extractor error: Sequence contains no elements

When running the Extractor in the CLI I get the following error:
Extracting products from my-api API:
Error occured: Sequence contains no elements

I've seen it fail on Extracting tags as well, when exporting other API's.
Last part of the log
 - Operation policy found for xxxxxx operation
'xxxxxx' Operation found
 - Operation policy found for xxxxxx operation
'xxxxxx' Operation found
API policy found
------------------------------------------
Extracting API version sets from service
------------------------------------------
Extracting authorization servers from service
------------------------------------------
Extracting loggers from service
------------------------------------------
Extracting products from service
'my-other-api-product' Product found
'starter' Product found
 - Product policy found for starter product
'unlimited' Product found
1 APIs found ...
------------------------------------------
Extracting products from my-api API:
Error occured: Sequence contains no elements

APIM configuration
In my APIM I have a few API's with policies and one defined a product, but this product is not used for all API's. Some API's are accessible without a product.
First hunch
Perhaps something in the hierarchy when a product is used but not for all API's? But I also get it at tags when exporting two API's:
------------------------------------------
Extracting products from my-api API:
2 APIs found ...
------------------------------------------
Extracting tags from my-other-api API:
'xxxxxx' Operation found
Error occured: Sequence contains no elements
Sequence contains no elements

I think this is a duplicate of #466 (PR #467 is pending to resolve it)

I have the same issue - do you know the latest stable branch I could go back to?

Not an ideal solution but until the PR is actioned you could try my patched fork (https://github.com/fortunkam/azure-api-management-devops-resource-kit)

@fortunkam I've tried your branch and get the same error.

Closed via #467


[AMBARI-24610] [Log Search UI] Show user friendly component name in query input

(cherry picked from commit 7f4d05bf5ee89dd771637dbca5164ace95383ac7)
What changes were proposed in this pull request?
I added the componentLabel pipe to the parameter rendering template.
How was this patch tested?
It was tested manually and by unit tests:
PhantomJS 2.1.1 (Mac OS X 0.0.0): Executed 271 of 271 SUCCESS (9.72 secs / 9.625 secs)
✨  Done in 37.85s.

Please review Ambari Contributing Guide before opening a pull request.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/3895/
Test PASSed.


ditto issue with smb drive

Hello,
First let me start by saying osxphotos is just what I was looking for to replace the old ExportPhotosLibrary I was using before. Great work !
I noticed an issue linked to the usage of ditto and remote samba shares. As it can't copy extended attributes it fails without the norsrc option in utils.py. Would you consider adding this as an export option ?
subprocess.run(
            ["/usr/bin/ditto", "--norsrc", src, dest], check=True, stderr=subprocess.PIPE
        )

also added import sys for the below to work
f"ditto returned error: {e.returncode} {e.stderr.decode(sys.getfilesystemencoding()).rstrip()}"
Thanks again :-)
PS: I will also look in parallel if there is a way for samba share to accept extended attributes

Thanks for alerting me of this issue.  I hadn't considered exporting to non-HFS drives but that's obviously a good use case.  The reason for using ditto was to preserve all metadata and adding --norsrc by default would strip the metadata.  It might be better to to check if destination directory supports metadata and if it does, preserve it, otherwise copy without metadata?  I'll take a look at this and work on a fix.

oh yes, without norsrc should definitely remain the default, but perhaps there could be a copy_metadata export parameter. Would be true by default and in case it's false then use the --norsrc or an alternative method to copy.
But you are right ideally the destination share should support xattr. In my case, I have a docker samba exposing exposing a couple of ext4 mounted volumed on a NAS that is compatible with both mac and windows. I think the mount should be done with xattr but still need to investigate. I have the same behaviour with home assistant samba addon. So it might be default for several file share solutions.
Thanks again :-)

Updated the docker image with latest version that implements those guidelines:
https://wiki.samba.org/index.php/Configure_Samba_to_Work_Better_with_Mac_OS_X
ditto works great now :-) in that case what I suggested might not be required. You were right, better fix the root cause.

Glad you found a solution! I'll keep the issue open while I work on a solution for filesystems that don't support extended attributes.  Some sort of flag to export, as you suggested, would work.

@floco I've added --no-extended-attributes option to export to address this issue

Perfect 👍 thanks !

@floco FYI, I've removed the --no-extended-attribute option in v0.38.0 because I think it's no longer needed.  I implemented a new copy method that takes advantage of copy-on-write on APFS volumes which makes export much faster.   As a result, I don't think the --no-extended-attribute methods is needed any more.  If you run into any issues though, please post a note here or open an issue.

thanks a lot for keeping me informed 👍


[BUG] Cannot Properly Deploy Pods

What happened:
Trying to deploy the pods but they are in CrashLoopBackOff state. The pods are running in g5.xlarge instances.
dynamicsdruntime-sd-on-eks-sd-webui-inference-api-567d869748bls   2/3     CrashLoopBackOff   3 (1s ago)   4m29s
sdruntime1-sd-on-eks-sd-webui-inference-api-5b5d67c95f-blvbg      2/3     CrashLoopBackOff   3 (3s ago)   4m29s
Warning  FailedScheduling  5m32s                 default-scheduler  0/2 nodes are available: 2 Insufficient cpu, 2 Insufficient memory, 2 Insufficient nvidia.com/gpu. preemption: 0/2 nodes are available: 2 No preemption victims found for incoming pod..
I tried to increase the instance type but it didn't work out.
Environment:
Following the same config.yaml in stable-diffusion-on-eks
/config.yaml

Hi, thank you for your response. Please attach log of SD Web UI in the issue so we can investigate into the issue.
Also, could you please check if the model is properly synced to EFS? You can check it via AWS Console or mount EFS filesystem to a EC2 instance. If not, you may need to run DataSync task manually.

Close stale issue. Feel free to reopen it if the issue persists.


Copy

Why do I want to push these changes to my development branch? I wouldn't have an example page.


Automated cherry pick of #1860: 添加同步安全组日志

Cherry pick of #1860 on release/2.11.
#1860: 添加同步安全组日志

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.11@2c786c0). Click here to learn what that means.
The diff coverage is 0%.

@@              Coverage Diff               @@
##             release/2.11   #1863   +/-   ##
==============================================
  Coverage                ?   7.59%
==============================================
  Files                   ?     412
  Lines                   ?   63634
  Branches                ?       0
==============================================
  Hits                    ?    4835
  Misses                  ?   58389
  Partials                ?     410

Flag
Coverage Δ

#aFlag
7.59% <0%> (?)

Impacted Files
Coverage Δ

pkg/compute/models/cloudsync.go
0% <0%> (ø)

pkg/compute/models/secgroupcache.go
6.38% <0%> (ø)

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 2c786c0...e5ef25b. Read the comment docs.

/approve
/lgtm


[Web] User profile tag component

Tasks

[ ] Component of user profile tags
[ ] Management dialog for managing tags

Design

figma

Documentation

doc

Issue requires review and rearrangement. Close it.


Multi select with chips has styling issue in material ui example

I'm trying to add chips in multi select component, I already did using material ui sample example, so came to material ui page to check it and I see that it has styling issue. Here: https://material-ui.com/components/selects/

[x ] The issue is present in the latest release.
[ x

] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯

Expected Behavior 🤔

Steps to Reproduce 🕹

Steps:

Context 🔦

Your Environment 🌎

Tech
Version

Material-UI
v4.?.?

React

Browser

TypeScript

etc.

If I select multiple chips, all I see is only top 2 components, I don't see all selected chips. As far as I remember last time when I checked the linked, it was working fine but not now.
Here I selected all chips but I could see only 2. I saw issue in my deployed project, I checked in local it was fine then I deleted node_modules and did re installed then I saw this issue in my local too.

The problem is solved here https://github.com/mui-org/material-ui/pull/20822

@ravi586941 As @marcosvega91 said, this will be fixed in v4.9.13 - https://material-ui.netlify.app/components/selects/

Duplicate of #20811

I'm trying to use 4.9.13 version but I'm getting the error
npm ERR! notarget No matching version found for<EMAIL_ADDRESS>I can only use till v4.9.12. I see 4.9.13 is published. When should I be able to use the latest 4.9.13 version? @joshwooding

@ravi586941 Caching issue


Seperate behavior for . and []

I'd like to use jsonpath for matching python objects structure. So the way I see is to create Child override in parser. But I don't want to overload entire function https://github.com/kennknowles/python-jsonpath-rw/blob/master/jsonpath_rw/parser.py#L71-L88 Maybe It is good idea to split it into separate methods for each rule?

It is JSONPath spec that these are the same, as far as JSON goes.
But in fact, preliminary versions of this library supported object fields as well as dictionary entries, since there is no rule that JSON has to be deserialized into dicts and lists, etc. The trouble is that there are ambiguous cases where it is not clearly right to do one or the other.
I am open to ideas. A couple that come to mind:

Global configuration as to whether you want to query into objects or dicts+lists
Add a new backtick operator like `field` (or even aliased to f) that either gets one field, or switches behavior for a whole sub-JSONpath expression.

I hope that would not abuse you. I'm trying to get such functionality from any of mature query-libraries. I've described use case here https://github.com/adriank/ObjectPath/issues/50


fix the dependancies in the Cargo.toml.

Signed-off-by: Daogang Tang<EMAIL_ADDRESS>

Thanks,


Fix Fernet no_encryption_at_rest ignored for encryptOrDecrypt

Describe your changes :
Disabled Fernet encryption causes exceptions. This skips encryption/decryption when the key is not set.

Type of change :

[x] Bug fix

Frontend Preview (Screenshots) :
For frontend related change, please link screenshots of your changes preview! Optional for backend related changes.

Checklist:

[x] I have read the CONTRIBUTING document.
[x] I have commented on my code, particularly in hard-to-understand areas.
[x] I have added tests that prove my fix is effective or that my feature works.
[x] All new and existing tests passed.

Reviewers

Backend: @open-metadata/backend

PR not relevant anymore


Makefile changes for bundle generation

conditionally install operator-sdk locally
ability to supply REPLACES and OPERATOR_NAMESPACE vars for
bundle target

ref: #110, #111
Signed-off-by: Leela Venkaiah G<EMAIL_ADDRESS>

not sure why label changed, ptal

# ./bin/operator-sdk version
operator-sdk version: "v1.8.0", commit: "d3bd87c6900f70b7df618340e1d63329c7cd651e", kubernetes version: "1.20.2", go version: "go1.16.4", GOOS: "linux", GOARCH: "amd64"

broken into #204 and #205


enterers, burbolt (logical-conflict)

This pull request was generated by the 'mq' tool

/trunk merge


bug fix number of k points in each direction

This fixes a bug affecting for example the madelung constant evaluation and exxdiv=vcut_ws, etc. It was noticed for shifted k point meshes.
The unique function might consider two k point coordinates as distinct even though they represent the same k point coordinate and only differ very slightly due to floating point noise. Consider 0.055000000001 and 0.0549999999999 (made-up example), which might get rounded to 0.06 and 0.05 respectively and therefore considered different, even though they belong to the same k point. This fix looks at the differences between coordinates directly and is hopefully a little more robust.
How to set the minimum tolerance for the difference is a question. If it is too low, floating point deviations might cause two coordinates being considered distinct even though they are not. If it is too high, we limit the size of k point meshes. Currently, it is set at 1e-5.
Credits for code and discussion goes to @tberkel and @hongzhouye .

Meshes like this:
kpts = array([[ 0.26682728,  0.62113383,  0.52914209],
       [-0.26682728, -0.62113383, -0.52914209]])

were not considered by us when checking that prod(Nk) = length of kpts array. Here, Nk = [2,2,2], giving the unique k points in each direction. There are only two k points though, not eight.
A solution is to delete the check (not ideal but can't see any solution at this moment).

I'm slightly confused about the intial error - your examples should both round to 0.055000 = 0.055 when using round(decimals=6).
Of course if you have enough leading zeros, at some point you would get different rounding. k-points can become small in big unit cells, but cell.get_scaled_kpts returns them dimensionless, so they shouldn't be that small?

Good point! This is an actual example that I had commented somewhere else: due to numerical noise, two scaled k point coordinates that should be the same number, were 0.05167749999999998 and 0.05167750000000004, being rounded to 0.051677 and 0.051678 respectively and therefore found not to be the same.

Right! And I guess just reducing decimals to say 10 wouldn't be a robust solution either, since for another grid you might have a 5 in the 10th decimal position!

Indeed. Assuming they come from a uniform mesh (as the function name
suggests), then you only need to compare k-points to the order of
1/nkpts to decide if they are the same kpt or not.
On Tue, Sep 27, 2022 at 11:35 AM Max @.***> wrote:

Right! And I guess just reducing decimals to say 10 wouldn't be a robust
solution either, since for another grid you might have a 5 in the 10th
decimal position!
—
Reply to this email directly, view it on GitHub
https://github.com/pyscf/pyscf/pull/1443#issuecomment-1259683317, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ADGAJC6YEUH7INZ4JEPYU23WAMH2LANCNFSM6AAAAAAQWBWNJU
.
You are receiving this because you were mentioned.Message ID:
@.***>

The tests currently fail due to two k points like this:
kpts = array([[ 0.26682728,  0.62113383,  0.52914209],
       [-0.26682728, -0.62113383, -0.52914209]])

However, they don't represent a uniform monkhorst mesh, so maybe the function get_monkhorst_pack_size should not be called? It happens in multigrid DFT. I'm having a closer look.

@sunqm is it correct that multigrid DFT evaluates the madelung constant with Nk = [2,2,2], even though only two k points are present?

@sunqm is it correct that multigrid DFT evaluates the madelung constant with Nk = [2,2,2], even though only two k points are present?

madelung constant is evaluated against [2,2,2] . The code searches in each direction the number of different k points. Multigrid DFT uses the same code as the regular DFT code

Investigating test failures. Meshes like this:
kpts = array([[ 0.26682728,  0.62113383,  0.52914209],
       [-0.26682728, -0.62113383, -0.52914209]])

were not considered by us when checking that prod(Nk) = length of kpts array. Here, Nk = [2,2,2], giving the unique k points in each direction. There are only two k points though, not eight. A solution is to delete the check (not ideal but can't see any solution at this moment).

The check prod(Nk) == len(kpts) may be skipped here. When k point symmetry is considered, it may also happen that the prod(Nk) != len(kpts)

It’s a bit dangerous to infer the Monkhorst Pack mesh from only a limited
set of k-points, but I suppose as long as it’s always used carefully, the
assert can be removed.
On Thu, Oct 6, 2022 at 6:02 PM Qiming Sun @.***> wrote:

Investigating test failures. Meshes like this:
kpts = array([[ 0.26682728,  0.62113383,  0.52914209],
[-0.26682728, -0.62113383, -0.52914209]])
were not considered by us when checking that prod(Nk) = length of kpts
array. Here, Nk = [2,2,2], giving the unique k points in each direction.
There are only two k points though, not eight. A solution is to delete the
check (not ideal but can't see any solution at this moment).
The check prod(Nk) == len(kpts) may be skipped here. When k point symmetry
is considered, it may also happen that the prod(Nk) != len(kpts)
—
Reply to this email directly, view it on GitHub
https://github.com/pyscf/pyscf/pull/1443#issuecomment-1270743756, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ADGAJC2UAF5ISQETF57K3DLWB5D65ANCNFSM6AAAAAAQWBWNJU
.
You are receiving this because you were mentioned.Message ID:
@.***>

Thanks for the comments. I've removed the assert.


Some small fixes to tax_functions.md

This PR adds some small fixes (e.g., broken references and equations) to @jdebacker's PR #681 to OG-USA master.

Thanks for these fixes, @rickecon.


Dynamic lookup types

It'd be useful to allow client to specify the lookup_type for the field. Example: ?qty=1&qty_loookup_type=gte
Ref: https://twitter.com/jpadilla_/status/583326079114461184
I could probably tackle this with a bit of guidance.
Update: Ideally this would render a form with a field for foo and a choice field for foo_lookup_type.

I was going to write, "There's no reason why this couldn't be supported...".
I was then looking up MultiValueField and thinking it needs to be done with this and making sure the selected lookup type is valid. (If the form is not valid an empty QuerySet is returned — which is a fair behaviour here.)
I then cracked open fields.py and saw this added in 4eb281c8bdba4b99f6b3dcde4c7442ca4cc58731.
The docs are a bit skimpy — really top of the things I need to do here — but what happens if you just supply an iterable to the lookup_type parameter (as discussed here?

@carltongibson hmmm that does seem to work. Shouldn't we be seeing the exact same behavior when doing this:
class ProductFilter(django_filters.FilterSet):
    class Meta:
        model = Product
        fields = {'price': ['lt', 'gt']}

which results in:

Quantity:	<input id="id_quantity__lt" name="quantity__lt" step="any" type="number" />

Quantity:	<input id="id_quantity__gt" name="quantity__gt" step="any" type="number" />

and this:
class ProductFilter(django_filters.FilterSet):
    price = django_filters.NumberFilter(lookup_type=['lt', 'gt'])
    class Meta:
        model = Product
        fields = ['price']

which results in:

Quantity:	<input id="id_quantity_0" name="quantity_0" step="any" type="number" /><select id="id_quantity_1" name="quantity_1">
<option value="gt">gt</option>
<option value="lt">lt</option>
</select>

That's a nice example.
My first pass at the story goes like this:

the One True Way to define a FilterSet is to do it explicitly, defining each filter as per your second price example;
the fields shortcut is a convenience for auto generating filters;
the dictionary syntax there (allowing multiple filters per field) is an extra convenience added onto that;
I'm not sure about making that any more complicated in order to switch between multiple filters vs dynamic lookups — it strikes me the syntactic gain over just defining the filters explicitly is limited.

I'm happy to see any of that questioned — but I'm wary of adding too much here.

I'm going to close this as Out of Scope for the moment. Happy to consider documented, tested pull requests. We may have capacity to reconsider in the future.


Clean up blog post

TELCODOCS-2000 documenting the AGOF 2
Trying to fix build
Trying to fix build 2
adding more content
Adding more info
Adding new section on running default install
Trying to fix table not displaying
Adding more content
Add some information how to develop in git
Adding content about collections
Adding content about collections 2
Adding content about collections 3
Adding docs on overview of install
Reordering content
Trying to fix display issues
Trying to fix display issues 2
Trying to fix display issues 3
Updating based on SME review
fix: Make it clear you need ascii doctor
fix: Support make serve on mac os
fix: correct port passthrough on a mac
fix: cleanup
Adding peer review feedback
Add blog post
Fix a bunch of spellcheck errors
Added overview of validated patterns and removed the old content (#469)
Moved patterns tier info under about validated patterns section (#470)
Adding links to see page history and report doc bugs (#363)
Content updated on peer review
Content updated on peer review3
Content updated on SME review
Content updated on SME review 2
Tweak slimming post
Initial pattern docs for ansible-edge-gitops-kasten
New blog post about how to sequence subscriptions
Minor formatting and remove a stray sentence
Clarify support for resource hooks
Drop some unneeded metadata from example yaml

This is an automated message:
You can preview this docs PR at http://492.docs-pr.validatedpatterns.io
Note that they get generated every five minutes, so please wait a bit.


Add interface to MiMiC

MiMiC is a framework for multiscale modeling. Currently capable of coupling GROMACS and CPMD for QM/MM MD simulations. See https://mimic-project.org/ for more information.
Implemented by Viacheslav Bolnykh and Jógvan Magnus Haugaard Olsen.

I know that no procedure for testing and code review has been set up yet so it cannot be merged at the moment. However, we may as well get started sooner rather than later, and here is some work that we would like to get into CPMD.

Other people involved are @ippoliti and @dmandelli

@foeroyingur is it possible to split it in several commits?

@foeroyingur is it possible to split it in several commits?

It could be split into two commits without too much effort. This work started some years ago and unfortunately it is not trivial to split it into many convenient parts.


Changing directory does not always work and sometimes breaks Quickgui

While moving around my system looking for somewhere to drop another test VM I notice that attempting to chose /home/<user>  and clock on OK left the working directory where it was.
Also, trying to find some reason/pattern to this (in the past the presence of symlinks has had surprising impacts and of course permissions/acls were a likely candidate ) I found I could consistently break quickgui by selecting /home/<user>/snap/flutter  (window goes blank/white and loses possibility to do anything if in "Manage Existing" mode, or if in "New" mode bottom half goes white but top half logo is still there and you can "Close" and on return the previous directory will be current ) .
This may disappear with a reboot as that directory is left over from removing the flutter snap, and is actually a directory  from a zfs dataset bind mounted in to try and cope with the demands of the flutter snap so probably somewhat spurious edge-case.  However, the error message suggests it might be worth "handling"  this  "it cannot happen" event more graciously (i.e. the file picker allows selection of a directory that flutter believes does not exist )
[ERROR:flutter/lib/ui/ui_dart_state.cc(209)] Unhandled Exception: FileSystemException: Getting current working directory failed, path = '' (OS Error: No such file or directory, errno = 2)
#0      _Directory.current (dart:io/directory_impl.dart:42)
#1      Directory.current (dart:io/directory.dart:136)
#2      _ManagerState._getVms (package:quickgui/src/pages/manager.dart:133)
#3      _ManagerState.initState.<anonymous closure> (package:quickgui/src/pages/manager.dart:67)
#4      _rootRunUnary (dart:async/zone.dart:1436)
#5      _CustomZone.runUnary (dart:async/zone.dart:1335)
#6      _CustomZone.runUnaryGuarded (dart:async/zone.dart:1244)
#7      _CustomZone.bindUnaryCallbackGuarded.<anonymous closure> (dart:async/zone.dart:1281)
#8      _rootRunUnary (dart:async/zone.dart:1444)
#9      _CustomZone.runUnary (dart:async/zone.dart:1335)
#10     _CustomZone.bindUnaryCallback.<anonymous closure> (dart:async/zone.dart:1265)
#11     _Timer._runTimers (dart:isolate-patch/timer_impl.dart:395)
#12     _Timer._handleMessage (dart:isolate-patch/timer_impl.dart:426)
#13     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:192)

Further investigation suggests the directory that did not exists was in some kind of snap-remove-related limbo: (because snapd tried to remove it but it was a bind mount so unlink failed on resource busy)
drwxr-xr-x 4 phil phil  4.0K Dec 13 10:25 ubuntu-accomplishments
drwxr-xr-x 0 phil phil     2 Dec 13 10:31 flutter    <---
drwxr-xr-x 5 phil phil  4.0K Dec 13 13:27 btop
drwxr-xr-x 5 phil phil  4.0K Dec 14 19:02 openscad
drwxr-xr-x 5 phil phil  4.0K Dec 15 10:33 cura-slicer

rmdir returns device or resource busy and fuser -v   indicated kernel mount  ... so unmounted it and then removed it.  Problem 2 resolved (and probably not worth the time to try handling something so odd that should not happen ever)  Should probably let the snapd folks know though.

And the other problem is between seat and keyboard:  slightly counter-intuitive (but consistent) behaviour of the file picker confuses old person who prefers the command line.
Nothing is real.
Strawberry Fields Forever!


how to test projection via slots

we are dealing with question how to test projected content via slots
https://github.com/wc-catalogue/blaze-elements/issues/238
Just wanted to ask how do you do it within atlassian. Any suggestions are more than welcome!

on thing that comes to mind is assignedSlot or assignedNodes() but that dosn't work in browsers which are using polyfill for shadow dom https://developers.google.com/web/fundamentals/getting-started/primers/shadowdom#workwithslots

As discussed IRL, the best thing to do here is probably to only go as far as to test to ensure the slots you expect to exist, are there. If you test slotting, then you're testing things that should already be tested by the polyfills and browsers.
Will close this, but if there's anything else to action here, let's address it!


[Event Request] Codeunit 6620 "Copy Document Mgt." - OnCopySalesLineOnBeforeSalesLineAutoReserve

Hi team,
In Codeunit 6620 "Copy Document Mgt." procedure CopySalesLine we would like to perform some additional checks before running ToSalesLine.AutoReserve, for that a handler event would be a good solution:

[IntegrationEvent(false, false)]
local procedure OnCopySalesLineOnBeforeSalesLineAutoReserve(var ToSalesLine: Record "Sales Line"; var ToSalesHeader: Record "Sales Header"; var isHandled: Boolean)

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


New git aliases 'grm' and 'grmc'

alias grm='git rm'.
alias grmc='git rm --cached'.
No other git aliases will be overridden.
grm and grmc aren't commands within oh-my-zsh yet.

I would be interesting in having these aliases in the main branch too. Thank you.


Implement log module

Should hook to the stick driver (driver chain?) and log all written and read packets for posterior analysis.

Decided against instrumenting Node. I see no real use case, much less for a 0.1 release.


add_node script fails

Add node fails
On cmwell environment with missing node, add node operation fails.
Steps taken:

Stop all cmwell processes and wait 15 minutes
Verify that health-detailed shows n-1 nodes.
In folder with installation package have ready a custom json file with this content:

[
{"ip":"<IP_ADDRESS>", "mappings":["<IP_ADDRESS>","<IP_ADDRESS>","<IP_ADDRESS>","<IP_ADDRESS>"]}
]

Execute the add_node cmw.env_<IP_ADDRESS> command.

Expected: all processes up and running. Health-detailed.md shows N nodes.

Actual: NPE in script output. No cmwell processes up.

The command line output is:
WARNING: <IP_ADDRESS> Will be added to CM-Well grid!!!
Continue?
If YES, write: YeS and press Enter
YeS
current cm-well environment is /clfmnt/data/cmwell
ping: Warning: source address might be selected on device other than lo.
Info: Connecting to grid with ip: <IP_ADDRESS>
This is a production cluster. Are you sure you want to do this operation: (yes/no)
yes
Info: purging cm-well
Info:   stopping processes
Info:   clearing application data
Info:   clearing data
Info: finished purging cm-well
This is a production cluster. Are you sure you want to do this operation: (yes/no)
yes
Info: deploying application
Info:   creating application directories
Info:   deploying components
Info:   creating symbolic links
Info:   linking cassandra data directories
Info:   linking elasticsearch data directories
Info:   linking zookeeper data directories
Info:   linking kafka data directories
Info:   linking cmwell-batch data directories
Info:   creating cassandra log directories
Info:   creating elasticsearch log directories
Info:   creating zookeeper log directories
Info:   creating kafka log directories
Info:   creating cmwell-bg log directories
Info:   creating cmwell-batch log directories
Info:   creating cmwell-ws log directories
Info:   creating cmwell-controller log directories
Info:   creating cmwell-dc log directories
Info:   creating cassandra configuration directories
Info:   creating elasticsearch configuration directories
Info:   creating zookeeper configuration directories
Info:   creating kafka configuration directories
Info:   creating cmwell-bg configuration directories
Info:   creating cmwell-ws configuration directories
Info:   creating cmwell-controller configuration directories
Info:   creating links in app directory
Info:   creating scripts
Info:   deploying plugins
Info:   linking libs
Info: finished deploying application
java.lang.NullPointerException
      at cmwell.ctrl.client.CtrlClient$.addNode(CtrlClient.scala:165)
      at Host.$anonfun$addNodes$3(ctl.scala:1253) ```

This is a Grid issue:
from logs in TRACE:
2018-03-25 13:47:33,473 ell-mc-ppe-akka.actor.default-dispatcher-19] WARN  a.r.transport.netty.NettyTransport   - Remote connection to [null] failed with java.net.ConnectException: Connection refused: /<IP_ADDRESS>:0
2018-03-25 13:47:33,473 ell-mc-ppe-akka.actor.default-dispatcher-19] WARN  a.remote.ReliableDeliverySupervisor  - Association with remote system [akka.tcp://cm-well-mc-ppe@<IP_ADDRESS>:0] has failed, address is now gated for [5000] ms. Reason: [Association failed with [akka.tcp://cm-well-mc-ppe@<IP_ADDRESS>:0]] Caused by: [Connection refused: /<IP_ADDRESS>:0]
2018-03-25 13:47:33,479 [scala-execution-context-global-61]          TRACE k.grid.GridImp                       - Selecting actor at: akka.tcp://cm-well-mc-ppe@<IP_ADDRESS>:0/user/HealthActor-router
The console cannot connect to the HealthActor and thus gets null pointer exception.


AdaptiveCard rendering issue in Virtual Assistant : Error reading JArray from JsonReader. Current JsonReader item is not an array

Platform

.NET
C#

Author or host
NA
Version of SDK
Microsoft.Bot.Solutions  :  Version=0.8.0-preview1
AdaptiveCards : Version 1.2.4
Microsoft.AspNetCore.Mvc.NewtonsoftJson : Version 3.1.0
Details
We have migrated the VA to 0.8. Json rendering is fine but while clicking the adaptive card button I am getting the above error.
For rendering the card I have used the below code
var card = JsonConvert.DeserializeObject(ItemFinal.ToString());
card = AdaptiveCard.FromJson(ItemFinal.ToString()).Card;
var attachment = new Attachment(AdaptiveCard.ContentType, content: card);
return attachment;
In above code ItemFinal is an Json object.

@Jashmi-1402 can you please include your card json please? Also what is the virtual assistant that you are referring to here? (@ckkashyap ideas what they are referring to here?) Thanks!

Hi Shalini,
I have used the below json.
{"type":"AdaptiveCard","version":"1.0","body":[{"type":"Container","items":[{"type":"TextBlock","text":"","wrap":true}]},{"type":"Container","items":[{"type":"ColumnSet","columns":[{"type":"Column","width":"stretch","items":[{"type":"TextBlock","text":"","horizontalAlignment":"Center","weight":"Light","separator":true,"color":"Accent"}],"id":"1","selectAction":{"type":"Action.Submit","title":"actionButton","data":"{3}"},"horizontalAlignment":"Center","style":"emphasis"},{"type":"Column","width":"stretch","separator":true,"items":[{"type":"TextBlock","text":"","horizontalAlignment":"Center","weight":"Light","separator":true,"color":"Accent"}],"id":"2","selectAction":{"type":"Action.Submit","title":"actionButton","data":"{4}"},"horizontalAlignment":"Center","style":"emphasis"}]}]}],"$schema":"http://adaptivecards.io/schemas/adaptive-card.json\"}
for migrating the Virtual Assistant I have followed the below link.
https://microsoft.github.io/botframework-solutions/overview/whats-new/0.8-beta/migrate-existing-va-to-0.8/

Closing as this appears to be a BotFramework issue. If you're still seeing a problem, please file an issue in their GitHub repo.


How to get started

I expected the README to have basic, no Java background needed, instructions on how to e.g. run tests on the project, and otherwise get started. Could such be added?
I tried:
$ mvn test
...
Tests run: 56, Failures: 0, Errors: 0, Skipped: 0

[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 48.992 s
[INFO] Finished at: 2016-11-05T23:49:52+02:00
[INFO] Final Memory: 24M/306M
[INFO] ------------------------------------------------------------------------

While that eventually provides the above message, it does show multiple call stacks during the execution, which made me think the code did not run well on my clone.
Is this normal in Java?
I'd like:

mvn test to be mentioned in the README
to see no stack traces when tests pass

Thanks for considering.

It's not absolutely unusual for tests to print stack traces even in case they pass.
Regarding the mvn test part on the README, could you consider opening up a PR with what you consider would make sense?

Hi, checking in on this to see what the next step might be. We could probably get one of any number of Zalando's using this project to help with these very essential onboarding details. Sound good?

Sounds definitely like a plan.
@akauppi I'm not 100% sure which type of getting started you would like to have: Getting started as a user or getting started as a developer?

+10 to both @duergner @akauppi, we can split these so no single person has to do it all