WIP: run bare rust CLI binary, no node

Probably can't merge this until some of the missing flags are added to SWC's rust CLI. Then we'll have to force users to upgrade to newest SWC version that is compatible.

Okay this has been hard and I haven't found much focus time for it.
As far as I know, https://github.com/swc-project/swc/issues/5171 was the last bug preventing Bazel from using the pure-rust CLI.
We will have some rules_swc API changes:

[ ] just mirror the rust CLI, no npm packages
[ ] allow integrity_hashes={"darwin-arm64": "sha512-DuB... to be passed along with version number in WORKSPACE so users don't have to rely on our mirroring
[ ] look for .swcrc in the package and use it if present, like we do for tsconfig.json in ts_project
[ ] maybe the swcrc file becomes required to make the rust CLI work. Accept a dictionary to dynamically generate it, just like we do with tsconfig.json
[ ] get all the tests passing again, and document the breaking changes that were required

Update: it's still necessary to provide a fake tty to make the swcx binary accept a folder, now trying to make a tty_binary rule in https://github.com/aspect-build/bazel-lib/pull/251
This keeps getting stuck because it's a side-project for me. We need some sponsorship for this work. Filed #88 so that the PR has an associated issue.

I'm working on this

Fixed an upstream bug, see https://github.com/aspect-build/rules_swc/issues/88

awesome, LMK if you'd like some help getting this rebased or anything

Yes, I already made a rebase in https://github.com/realtimetodie/rules_swc/tree/swc_rust and got bazel build //...  running with a custom swc cli and the changes from https://github.com/swc-project/swc/pull/6708. In addition, I found new bugs in the swc cli and will create subsequent PRs upstream.
bazel test //... 13 tests pass and 2 fail locally. All failed tests are related to the --output-dir option of the swc cli compile command. It is due to the cwd. The cwd is set to the execroot directory and the swc cli can not find the auto-generated mock test files that are written into the bazel-out directory. But I think this might be related to my custom binary location, see
https://github.com/realtimetodie/rules_swc/commit/ac516ac2256f9a273c862a966d8485a19649b802#diff-1444361579758fc6be4458b5105b2ee3ee623d72154094c3fa1fd9056845903eR154
Test summary
//docs:update_test                                              (cached) PASSED in 0.0s
//examples/custom_outs:tests_0_test                             (cached) PASSED in 0.1s
//examples/custom_outs:tests_1_test                             (cached) PASSED in 0.0s
//examples/macro:test_test                                      (cached) PASSED in 0.0s
//examples/opaque_src:test_test                                 (cached) PASSED in 0.1s
//examples/out_dir:check_outputs                                (cached) PASSED in 0.1s
//examples/paths:test_test                                      (cached) PASSED in 0.1s
//examples/rc/src:tests_0_test                                  (cached) PASSED in 0.1s
//examples/rc/src:tests_1_test                                  (cached) PASSED in 0.0s
//examples/root_dir:check_outputs                               (cached) PASSED in 0.1s
//examples/simple:test_test                                     (cached) PASSED in 0.1s
//examples/transitive:assertion_test                            (cached) PASSED in 0.0s
//swc/tests:versions_test_test_0                                (cached) PASSED in 0.1s
//examples/directory:minify_test                                         FAILED in 0.0s
//examples/directory:out_dir_test                                        FAILED in 0.0s
//examples/filegroup:check_outputs                                       FAILED in 0.0s

merry christmas

Fixed an upstream bug, see https://github.com/aspect-build/rules_swc/issues/88

The tests are now passing 🎉

https://github.com/aspect-build/rules_swc/issues/61

hey @realtimetodie this is fantastic, I'm watching those upstream PRs.
In theory we could use rules_rust to allow a from-source swc toolchain here, then users could point to an unreleased branch of swc (or they could vendor swc sources into their monorepo so they can make local modifications, though that's essentially a fork)
That's probably not worth it though, the upstream maintainers are active and responsive so I bet you'll be able to get an official swc release that works 100% with Bazel.

Yes, the swc team is very responsive. The changes for symbolic links were merged upstream.

I updated the mirror_release.sh script. The binaries are now downloaded from GitHub releases instead from npm
Next, I will refactor repositories.bzl and do the same there.
There is one remaining PR to make the cli work in non tty environments. This is blocking. I'm also working to accept a dictionary to dynamically generate the swrc configuration file, just like we do with tsconfig.json.

I refactored repositories.bzl. The binaries are now downloaded from GitHub releases instead from npm.

Node.js: All remaining rules_nodejs and npm artifacts were removed. Now, Node.js is only required to run some of the examples.

In addition, the rule accepts a dictionary to dynamically generate the swrc configuration file.
I added tests to validate that the generated configuration works
https://github.com/realtimetodie/rules_swc/tree/swc_rust/examples/rcdict

While doing all this, I oriented myself on the existing aspect-build rules and followed the general coding style.

Upgrade: The swc macro is unchanged 🎉 and users do need to upgrade their targets.

As this is a breaking change, I used the opportunity to do some house cleaning (please bear with me).
I renamed the underlying rule from swc_transpile -> swc_compile. This matches the wording of the SWC project (I think the word "transpile" was originally inspired by the Node-based TypeScript compiler, but this is different in the SWC world).
Checklist

[x]  just mirror the rust CLI, no npm packages
[x]  allow integrity_hashes={"darwin-arm64": "sha512-DuB... to be passed along with version number in WORKSPACE so users don't have to rely on our mirroring
[x]  look for .swcrc in the package and use it if present, like we do for tsconfig.json in ts_project
[x]  maybe the swcrc file becomes required to make the rust CLI work. Accept a dictionary to dynamically generate it, just like we do with tsconfig.json
[x]  get all the tests passing again, and document the breaking changes that were required

Breaking changes will be documented in the final PR.
Blocked by https://github.com/swc-project/swc/pull/6714

@realtimetodie SWC just cut a release that includes your last fix, so I think it's time to merge this!

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

@realtimetodie do you mind signing that CLA so I can include your commits without any fear of licensing issues?

signed

wow nice, the build is green let's goo

I'm doing another review/cleanup pass on it. The k8-fastbuild is going to be a portability problem
https://github.com/aspect-build/rules_swc/pull/57/files#diff-d1df27564968f40aec96caf8b7e3f176263984b5c8f12dc418a9ea61a19f5073R18

The solution to obtain the bazel runfiles directory as an environment variable is very elegant, I like it

look for .swcrc in the package and use it if present, like we do for tsconfig.json in ts_project

This doesn't actually seem to work, if I drop the swcrc = ".swcrc" attribute from an swc_compile then it stops using the config file. I'll fix in a follow-up, doesn't really have to be in-scope here as it wasn't a feature before either.


Better control of database read replicas

Add the ability to specify read database regions for failover or performance.
I want to be able to set the order that Flagsmith uses for reads. Right now the code randomly picks a replica which isn’t super helpful.

I did a review online of common approaches and here are some of my findings:

Cross regional database reads are rare. Most of the documentation focusses on local replicas.
There are two main approaches for failover from one replica to another. The more prevalent way of approaching it is using a heartbeat connection to the replica in question (either before handing off to Django or by storing it locally as a cache that's refreshed once a second) and if the replica is still online, hand off the replica to the reader otherwise hand it off to the next available replica (or even the default database if there is none). The secondary approach uses middleware to intercept database queries, this approach looked inferior to the first solution.
Falling back from the primary database to a promoted read replica is not widely covered by online sources. I think it should be possible but we would be venturing into the unknown and would come with unknowns like WAL log non-replication between the primary and the secondary databases.

Given what I've read online so far, I think we can pretty easily handle this ticket the following way:

Implement a heartbeat connection to our replicas using a Django cache to avoid querying every cycle.
Create a secondary set of CROSS_REGION_REPLICA_DATABASE_URLS in settings.py. These would not be used unless the local regional replication databases have fallen over.
Keep the current REPLICA_DATABASE_URLS in settings.py which are the first line of querying for reads.
In order to support setting orders instead of randomly distributed the reads I suggest a new settings.py variable called REPLICA_READ_STRATEGY set to either DISTRIBUTED which is the current approach of spreading reads across replicas, or set to SEQUENTIAL which would try, sequentially, the REPLICA_DATABASE_URLS in order and then fallback to CROSS_REGION_REPLICA_DATABASE_URLS and follow them in order once they're all exhausted.

I'm not sure if we should complicate it more than that, but one of the downsides of this approach is that if the REPLICA_DATABASE_URLS have suffered loss to the point where only a single replica is remaining, the load may be high on it even though the CROSS_REGION_REPLICA_DATABASE_URLS may be on standby. We could consider introducing another new variable that specifies a minimum distributed replica pool, which could mix the two pools if necessary, but I doubt this strategy is really necessary.

Solved in https://github.com/Flagsmith/flagsmith/pull/3300


Need information on where to get the IActivityLogger interface

I tried to implement step one in my current bot, which was to make the class that extends the IActivityLogger class. But when I started coding, I received the error message that it cannot find the IActivityLogger interface. Can you please insert it into the documentation?

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

ID: d8de1be0-19df-c8e9-9a80-e8f6df9dc362
Version Independent ID: 92ae3a1e-10a1-1911-70b2-b8fd6142ba4f
Content: Intercept messages - Bot Service
Content Source: articles/dotnet/bot-builder-dotnet-middleware.md
Service: bot-service
Sub-service: sdk
GitHub Login: @RobStand
Microsoft Alias: kamrani

The document you referenced is for the older version of the SDK (v3). Are you using SDK v3 or v4?

Ah ok. I'm using v4. Is there documentation for this for v4 I can read?

I'm unfamiliar with v3's IActivityLogger, but you have a number of options in v4:

Use the built-in logging features in ASP.NET
Add telemetry to your bot
Debug a bot with inspection middleware

Please reopen if you need more information.


Investigate old links to mta

[ ] 0,0
[ ] imc
[x] j:pvark
[ ] j:iskv

j:pvark just leads to the mta redline.

/dest kallos at j:iskv actually leads to j:hits_spqr

0,0 goes to the MTA East system.

IMC goes to the mta redline

If it wasn't obvious, the goal of this issue was to make sure there weren't any rails that went to MTA that weren't added to the new station.


Adding shell script to automate the creation of the Terraform's service principal and remote state

611 & 612
PR Checklist

[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.
[ ] My code follows the code style of this project.
[ ] I ran lint checks locally prior to submission.
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Description
Adding shell script to automate the creation of the service principal and remote state resources, this script can be used to provision an environment using Terraform
These scripts are based on the ones discussed here https://github.com/retaildevcrews/ServicePrincipal/tree/main/infra
This is related to tasks #612 and #611
Does this introduce a breaking change

[ ] YES
[x] NO

Testing
Ran the script successfully from my local machine. It created resource group rg-jafartest4-cse-dev-tf to hold the Terraform's remote state. It also successfully created the jafartest4-sp-dev  service principal (no roles assigned at the moment)

I ran the script myself and ran into an issue with TF_CLIENT_SECRET, when the terraform.tfvars file is created that variable has "<<svc_ppl_CLIENT_SECRET>>" in the middle of the secret string

Did the run succeed for you? I tried it a couple of times and everything worked as expected


Add ability to pass extra arguments into board-specific workflows.

This PR adds an EXTRA_LITEX_ARGS command line argument to our build system. These extra arguments can be parsed in board-specific workflows to allow for a more flexible building process; ie. make prog EXTRA_LITEX_ARGS="--variant a7-100".
The Arty workflow has been modified to use these extra args to specify the variant of the Arty board (a7-35 or a7-100).

This is the command that I have confirmed to work, to target the Arty A7-100T board:
make TARGET=digilent_arty EXTRA_LITEX_ARGS="--variant=a7-100" clean prog load

Note that the build directory is the same as for the Arty a7-35t variant, so you must do a clean if you switch from one to the other.


Add feedback form

@eghale a link to the feedback form?

Yeah I wondered if there were any widgets that we could use so we don't have to take someone away from the page?
They only need to be able to write a line or two of feedback, and the text could just read something simple like 'This is currently in beta but we hope it's helpful. Please tell us what you think - all feedback and comments welcome!' @andrewMacmurray

@eghale maybe we could try the one we use for the cast website, I don't think I can sign up for it though as you can only have one free one, do you recon you could make one on your account?
https://getsitecontrol.com/

Hmm have we had any responses to the one on the CAST website whatsoever? I think if not, it might not be the best option... It's also quite a big, intrusive pop-up, especially on mobile. Perhaps leave this one for now. Hard to know what best practice around collecting feedback on a beta site is!!


Local images in Image tags only appear after a hotfix

Describe the bug
When putting a relative path into the uri field of an Image tag's source property, the image will not display, however it will take the correct amount of "space" given in the width and height of its style property.
Such an issue doesn't occur when loading from an external URL, the image appears immediately
To Reproduce
Minimal example:
import React, {Component} from 'react';
import {
  App,
  Window,
  Image,
} from 'proton-native';

export default class Example extends Component {
  render() {
    return (
      <App>
        <Window>
          <Image
            style={{
              width: 64,
              height: 64,
            }}
            source={{uri: "./img/next.png"}}>
          </Image>
        </Window>
      </App>
    );
  }
}

Run with npm run dev and then modify the source uri to something like img/next.png in the case of this example. When saving that change, it should automatically apply a hotfix which makes the image actually appear
Expected behavior
The image loads from the relative path immediately
Screenshots

Versions:

OS: Windows 10
Version 2.0.4
Node version 6.13.4

Additional context
Possibly related to #262

Seeing this also on 2.0.4

I have a similar issue, I am unable to use local image assets without receiving the following error:
Warning: Failed prop type: Invalid prop sourceof typestringsupplied toImage, expected object.
I have tried importing my assets at the top like so:
import Test from '../assets/images/testing.png'; const Test2 = require('../assets/images/secondTest.png');
I have used these implementations:
<Image source={Test} style={{height: 50, width: 50}}/>
<Image source={{uri: Test2}} style={{height: 50, width: 50}}/>
<Image source={{uri: "../assets/images/testing.png" style={{height:50, width:50}}/>
None of the above work. They all throw the exact same issue.
My current version of proton is: 2.0.4
I installed it through the proton-cli npx package.
I would really like to use this for my companies embedded solution in our factories, so I am hoping to get some basic issues resolved. Is there any better documentation on the native layer? I will need to use native libraries in SPI and other lower languages to build better high level components for our uses.
Thanks


Convert all powerplatform references to power-platform wherever possible

Describe the bug
Documentation and HCL samples variously reference powerplatform in some places and power-platform in others, so a user trying to piece together multiple examples to try to fit their specific scenario may run into confusing issues if they don't understand how naming is handled in HCL files.
To Reproduce
Steps to reproduce the behavior:

Open the repo
Search for references to "powerplatform", note nonzero hits
Search for references to "power-platform", note nonzero hits

Sample Terraform Code
If applicable, add terraform code to help explain your problem.
# Wrong:
provider "powerplatform" {
  use_cli = true
}
# Right:
provider "power-platform" {
  use_cli = true
}

Expected behavior
Any examples should be combinable without causing Power Platform provider reference issues.
System Information

Provider Version: 0.2.2-preview
OS & Version: Linux

Additional context
Add any other context about the problem here.
Contribution
Contribution
Do you plan to raise a PR to address this issue?  YES

I've checked and the only correct setup is as follows:
terraform {
  required_providers {
    **powerplatform** = {
      source = "microsoft/power-platform"
      version = "2.0.2-preview"
    }
  }
}

provider "**powerplatform**" {
  # Configuration options
}

also we need to fix some typos where resource names where where changed with - in the name


global leak error detected in mocha

While doing unit test to upgrade browserify module, I keep to get global leak error, which is introduced due to the following line. I am sorry I see no reason why the replacements variable should be defined in global object. Should it be fixed??
https://github.com/defunctzombie/node-browser-resolve/blob/master/index.js#L37

seems like a bug/oversight


Run Avalonia on Raspberry Pi OS Lite

Hi, I would like to run my very simple application on Raspbian buster Lite (the one without desktop, only terminal). The app is just window with one button and one text box. I followed all steps in this guide. The project also uses NuGet package Avalonia.Skia.Linux.Natives. But when I copied files to raspberry and tried to run with command sudo dotnet publish/MyApp.dll, I got following error:
Unhandled Exception: System.Exception: XOpenDisplay failed
   at Avalonia.X11.AvaloniaX11Platform.Initialize(X11PlatformOptions options)
   at Avalonia.AvaloniaX11PlatformExtensions.<>c__0`1.<UseX11>b__0_0()
   at Avalonia.Controls.AppBuilderBase`1.Setup()
   at Avalonia.ClassicDesktopStyleApplicationLifetimeExtensions.StartWithClassicDesktopLifetime[T](T builder, String[] args, ShutdownMode shutdownMode)
   at SmartMirror.Program.Main(String[] args) in C:\Users\User\Documents\Visual Studio 2019\Sources\MyApp\MyApp\Program.cs:line 13
Aborted

My app lib versions:
Avalonia 0.9.12
Avalonia Desktop 0.9.12
Avalonia.Skia.Linux.Natives <IP_ADDRESS>
netcoreapp2.0
I tried to install several libs to get X working:
xserver-xorg xinit libx11-dev libgtk2.0-dev libgtk-3-dev gir1.2-gtk-2.0 gir1.2-harfbuzz-0.0 libcairo2-dev libcairo-perl libcairomm-1.0-dev libpango-perl libsdl2-dev libnss3 x11-xserver-utils
I even tried set DISPLAY environment variable and run xhost + but it says xhost: unable to open display "" (or :0.0). This error occurs on every attempt to run some X app. Nothing worked to get at least one X app running even directly on Raspberry without using SSH. Does anybody have any idea how to solve this issue?  Hopefully it will be just some missing library. Thank you for any help

Try 0.10.0-rc1

Try 0.10.0-rc1

Thank you for your reply @danwalmsley

Try 0.10.0-rc1

Did you mean Avalonia.Diagnostics -Version 0.10.0-rc1 or Avalonia.Headless -Version 0.10.0-rc1? Or something else? Sorry I am not such experienced

Thank you for your reply @danwalmsley

Try 0.10.0-rc1

Did you mean Avalonia.Diagnostics -Version 0.10.0-rc1 or Avalonia.Headless -Version 0.10.0-rc1? Or something else? Sorry I am not such experienced

Thank you for your reply @danwalmsley
I updated all Avalonia libs to 0.10.0-rc1 but now I'm getting Segmentation fault error.. On the recommended version for Raspbery Pi with desktop it works flawless and also on Windows but on the Raspbian Lite version without GUI doesn't. Any other ideas?

Thank you for your reply @danwalmsley
I updated all Avalonia libs to 0.10.0-rc1 but now I'm getting Segmentation fault error.. On the recommended version for Raspbery Pi with desktop it works flawless and also on Windows but on the Raspbian Lite version without GUI doesn't. Any other ideas?

In order to render your application in Terminal you need to use FrameBufferor or DRM (i think).
Here is a sample code:
https://github.com/AvaloniaUI/Avalonia/blob/master/samples/ControlCatalog.NetCore/Program.cs
Check how --fbdev and --drm args are handled.
I would recommended cloning the repo to explore sample, since there are multiple gotchas.

In order to render your application in Terminal you need to use FrameBufferor or DRM (i think).
Here is a sample code:
https://github.com/AvaloniaUI/Avalonia/blob/master/samples/ControlCatalog.NetCore/Program.cs
Check how --fbdev and --drm args are handled.
I would recommended cloning the repo to explore sample, since there are multiple gotchas.

Thank you for help I'll definitely take a look on that sample code and try to make my code work. Later I'll post my findings

Thank you for help I'll definitely take a look on that sample code and try to make my code work. Later I'll post my findings


Fix dwFlags on CreateProcessW calls

Command prompt is still being shown when calling os.execute on Windows. The previous flags always results in 0 because flags should be merged with an OR operator.
Bugsquad edit: May help with https://github.com/godotengine/godot/issues/52242.

This is the right fix according to the official docs. Will make sure and test.

Thanks!

Built & tested, seems to work fine. Funnily enough, I don't actually seem to get a console window with current master though? Not sure why.
Anyway, this is correct, the current code doesn't work.
Edit: does the additional console window without this only show up in release exports? May be the case the otherwise existing default console window gets inherited. Will check.

It's deeper than that though. This doesn't fix all the problems related to command prompts showing up (for me at least), it fixes only the flag passing that was wrong.
The main problem when trying to launch an executable from godot (I'm building a game launcher) with godot is that OS.execute will always pass a non-null r_pipe value (https://github.com/WoLfulus/godot/blob/3c9d7203ea4360a521569a0142b029ba87c09448/platform/windows/os_windows.cpp#L416-L442), which makes the process execute with _wpopen (https://github.com/WoLfulus/godot/blob/3c9d7203ea4360a521569a0142b029ba87c09448/platform/windows/os_windows.cpp#L424) instead of CreateProcessW + Win32 pipe functions.


Openstack domain is required for V3

#105696468

@voelzmo we're closing this PR as we found it simpler to redo it in the right directory here: https://github.com/cloudfoundry-incubator/bosh-openstack-cpi-release/commit/673ea645a830fc228ea215cb1f5239eba4a94080 which is already merged into master. For some reason you had jobs/cpi/spec instead of jobs/openstack_cpi/spec.


Add process to create Billing Profiles

Page should include process to create new Billing Profiles for customers.

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

ID: 5ccc6ede-f1df-1c54-5b14-8dcfdd65011a
Version Independent ID: 0ba01e76-4b60-def0-9dd6-14069e1a7647
Content: Get started with your Microsoft Partner Agreement billing account - Azure CSP
Content Source: articles/cost-management-billing/understand/mpa-overview.md
Service: cost-management-billing
GitHub Login: @bandersmsft
Microsoft Alias: banders

@bunkrur Thanks for the feedback. This issue is being assigned to the content owner to review further and update as required.
@bandersmsft FYI!

Is there an update on this?

@bunkrur Hi there. This article is specifically for CSP partners with a Microsoft Partner Agreement. Partners can't create new billing profiles.
#please-close


ParameterOffset Type

A parameter offset is when you have a prepared statement with placeholder variables that are filled in at run time:
PREPARE my_stmt AS SELECT * FROM xxx WHERE id = $1;

For this query, the WHERE clause contains a ParameterValueExpression with the value idx of 0. This is then used at runtime to get the value to fill in from the query's parameter array in its execution context.
In Peloton, there was an explicit type for keeping track of parameter offsets:
https://github.com/cmu-db/peloton/blob/master/src/include/type/type_id.h#L21
But right now the ParameterValueExpression represents the value idx field as a uint32_t:
https://github.com/cmu-db/terrier/blob/master/src/include/parser/expression/parameter_value_expression.h#L59
I first think that we should have an explicit type to represent parameter offsets. I think we can all agree on that.
But the question is what kind of type should this be. Should it be a C++ strong type (like catalog::db_oid_t) or should it be part of the value type system?
The problem with making it a C++ type is that there is code in the optimizer that treats the parameter value type as any other value in the DB's type system (e.g., like a ConstantValueExpression value):
https://github.com/cmu-db/terrier/blob/optimizer/src/optimizer/rule_impls.cpp#L349-L352
So if we make it a C++ type, then this code won't work without wrapping it in TransientValue. Truth be told I don't even know whether that particular snippet of code is correct.
I propose that we make it a C++ type and then see whether @17zhangw can fix things in the optimizer.

@apavlo Is there any update on this issue?
Here is a stable link to the referenced L349-352 of rule_impls.cpp as referenced above: https://github.com/cmu-db/terrier/blob/151ca26e06fbf710da0d48ec299d38150274059d/src/optimizer/rule_impls.cpp#L349-L352
I would say that we still need a type::TypeId::PARAMETER_OFFSET for TransientValue to use so the execution engine would be able to identify that it is actually a parameter offset.


issue

Steps to Reproduce

Run flutter create bug.
Update the files as follows: ...
...

Expected results:
Actual results:

Logs

This should be closed since it does not provide any information related to the issue.
Cc: @TahaTesser


[KeePassXC] Add Yubikey support

Extension
https://github.com/raycast/extensions/tree/9fb1ff46e2ce4185a0c7ff8d769e4fab0677065a/extensions/keepassxc/
Description
Please add support for Yubikeys:
https://www.mankier.com/1/keepassxc-cli#Options-General_options
Example:
keepassxc-cli open -y 1 Documents/keepass.kdbx

The slot needs to be configurable (should be mostly 1 or 2).
Who will benefit from this feature?
Every KeePaasXC user that has their database secured with a Yubikey.
Anything else?
No response

Any update? Before the PR goes stale ...

Any news? It is again going stale.

The PR got closed due to inactivity, is there anyone who can work on it?

This issue is again going stale. Any updates?

Any news? Or is the extension just dead?

Ping

Ping

ping

Ping


Show api doesn't work in vue 3

Version
vue-final-modal: 1.8.0
vue: 3.0.4
Reproduction Link
using api show doesn't work -> https://jsfiddle.net/05ougr6L/24/
What is Expected?
Should open the modal
What is actually happening?
Modal isn't showing

Hi @manuelmazzuola,
every modal component must have own v-model.
Here is the example https://jsfiddle.net/9uLx4r7s/

should be specified in the documentation
thanks!

I agree, I will update the docs.
Thanks for your feedback!

The documentation was updated


update community resources

Wouldn't it be easier to simply link to the main scope README or the 3rd party plugins section of https://github.com/weaveworks/scope/pull/3192 (once merged). Otherwise it's going to be a nightmare to update the documentation when/if the number of plugins grows.

Updated. Let me know if it's what you had in mind and I'll update the other PRs as well.


[Pokemon TCG - GBC] black screen

Black screens when starting the game. works fine in DMG mode...
This game used to work in an earlier commit, i tried #4fc0099736 and its works.
i'll probably just keep trying later commits until it breaks, then i'll know what it is

found the commit, #ebe12f7 broke it


Scanline?

How to disable scanline?

This week end I will release a version where we can choose between multiple shaders, or no shaders at all.
The scanline was a test that I forgot to disable. Sorry for the inconvenience.

I quickly released a version without scanlines.


Allow folding of entire YAML structures i.e. documents (---)

When a file is set to use the YAML language, YAML can be folded. What I find to be missing however is the ability to fold from a document start (---) up to the next start or end (--- / ..., if present).
L1  ---
L2  key1: val
L3  key2: val
L4  ---
L5  keyA: val
L6  keyB: val
L7  ...

I don't have much experience of using pure YAML, but in the above scenario, I'd like to be able to fold lines 1 to 3.
L1  --- …
L4  ---
L5  keyA: val
L6  keyB: val
L7  ...

Hopefully pure YAML users would find this practical. I on the other hand would find this to be particularly useful within Markdown (see FR #180849).
Thanks.
VS Code 1.77.3

Please file the request against the YAML extension that you use.
By default, for YAML, VS Code creates folding ranges based on indentation

I do not use a YAML extension. I suppose you're saying that VS Code wouldn't not like to extend folding beyond indentations and that an extension should implement folding on --- instead.

That's correct. Is --- coming from front matter? Then actually a front matter extension should add such folding ranges.

That's correct. Is --- coming from front matter? Then actually a front matter extension should add such folding ranges.

--- can exist in both YAML or Markdown front matter; I was hoping that one underlying feature could address both cases, but for me, folding the --- in Markdown is my wish. I will look into which Markdown extension I have that could possibly implement this.

Your files are FrontMatter files, from which markdown/yaml is generated. FrontMatter is a preproccessor.
The Markdown and Yaml language extensions should not have to know about FrontMatter syntax.
The folding ranges should come from a FrontMatter extension


Clarify 'Code to copy'

Additional context
It's hard for a beginner to understand where to put Code to copy. Perhaps refactoring to express where exactly that code needs to be copied to would help.

Yes, do you have any suggestions? You can submit a PR directly

Well, where exactly do you put that code? Is it necessary somewhere in the users project? If so, my suggestion is to update the README to express where it's necessary.


docs-bug(button): Missing documentation for MatButtonHarness with() static function

Documentation Feedback
https://material.angular.io/components/button/api#MatButtonHarness
There is a static with() function that lets you find the right instance of mat-button on the page, yet this documentation doesn't support it.
https://material.angular.io/guide/using-component-harnesses states
You can also use the static with method implemented on all Angular Material component harnesses. This method creates a HarnessPredicate, an object that filters loaded harnesses based on the provided constraints. The particular constraint options vary depending on the harness class, but all harnesses support at least:
The docs for the with() function should exist on the Harness docs page as well, and not on this random guide.
Here is the function I am talking about:
https://github.com/angular/components/blob/master/src/material/button/testing/button-harness.ts#L34

Lumping this in but can file a new bug if needed.
https://material.angular.io/components/button/api#ButtonHarnessFilters states that there is a text filter on the filter interface. But, there is also the selector and ancestor filter from BaseHarnessFilters that isn't in that documentation.
I guess superclasses/inheritance isn't parsed when generating this doc?

@devversion do we extract API docs for static methods?


Support polymorphism and collections with @Nested annotation

There are two often repeated issues with how @Nested works currently:

it doesn't support collections – IIRC, we revert back automatically to using the serialized version of anything that is not understood by @Nested
it doesn't support polymorphism – when you declare a field to be of type MinimalJavadocOptions, only the annotations on that type are processed by @Nested, ignoring anything declared in the actual runtime type of the value. Again, there's no warning to the user, and this leads to unexpected results.

We could rework this, and use the runtime type of the values instead of the declared type to walk the data. There are a few things to pay attention to:

when validating the task type as part of the build (see ValidateTaskProperties task), we can only use the declared type information, so it won't be the same as the runtime validation
we need to handle cyclic references
processing annotated inputs needs to be moved from the AnnotationProcessingTaskFactory to ValidatingTaskExecuter (or a new executer before that).
calling getProperties() before task execution won't return the values for annotated properties

We may want to split the "handle collections" from the "handle runtime types/polymorphism" parts.
Other things to keep in mind...

What's the names of properties that are in a collection? or is the property's value the collection?

If the property's value is a collection, then what kind of error do you get when the 3rd element in a list has a null value for a non-@Optional property?

Should we allow a Map property?

I think we currently consider the value of the property to be the (serialized form of the) collection, and we simply report that the value has changed (need to check the details, but I think this is what happens with @Input at least). To me the only logical way to improve the situation is to map collection indexes to sub-properties, like elements.3.name etc. Mapping Maps to sub-properties by key is also easy as long as the keys are Strings containing valid Java identifiers.

I'd make the types we use to represent snapshots richer and polymorphic so they can deal with these use cases naturally, rather than trying to coerce every path through the graph into a string. Though we certainly could start with the use cases where we can do the coercion though.

@lptr Hasn't this issue been effectively resolved? What's remaining to be done here?

@mark-vieira You are right, we already support Iterables and Map as well as polymorphism. Validation could still need some improvement, but that should be tracked separately.
Let's close this.


Use voluptuous

This is for tracking the PRs for the migration to voluptuous (story).
 They is in progress at the moment. The hm component have allready vol. Thanks

Dito mysensors.

I completed qwikswitch and icloud a while ago, and got stuck with device_tracker, will submit PR

#3000 #2998 #2908

Vol for Modbus is merged

What is the current state?

Universal media player is the last one. I will close this because I spent enough time on this already.

Shouldn't this be kept open, just to track that the universal media player still isn't moved over to vol?

It doesn't look like that somebody wants to step up thus there is nothing to track. My PR for the Universal media player was open for 2 months and the only thing I got were questions about the state.

I would like to take a look at this, but I don't know the workings of voluptuous so I'm not sure if I can be of any help...


updated mouse locater to correct sha256 and version number

updated mouse locater to correct sha256 and version number

https://github.com/caskroom/homebrew-cask/pull/20109
Your PR had errors - try using cask-repair (check CONTRIBUTING)


RTSP-Simple-Server stops sending RTP packets to Janus Gateway

Which version are you using?
v0.17.6
Which operating system are you using?

OS

[x] Linux
[ ] Windows
[ ] macOS

Architecture

[x] amd64
[ ] arm64
[ ] arm7
[ ] arm6

Describe the issue
I am currently using rtsp-simple-server to ingest a RTSP stream (VP8)  and I am using Janus-gateway to read the stream from rtsp-simple server to send it to WebRTC clients. Everything works right, but after 1 or 2 minutes rtsp-simple-server stops sending RTP packets (over UDP) to Janus. After 5 seconds without media, Janus sends a FIN, ACK and restarts the connection.
I don't understand why rtsp-simple-server stops sending RTP packets to Janus, but will continue to send them to VLC
Describe how to replicate the issue

Start RTSP-Simple-Server
publish an RTSP(VP8) to it
clone https://github.com/meetecho/janus-gateway
Follow the installation guide to be able to run the streaming demo on localhost
add this field in janus.plugin.streaming.jcfg:
rtsp-test: {      type = "rtsp"      id = "test"      description = "test"      audio = false      video = true      rtsp_failcheck = "false"      url = "<rtsp url of the rtsp-simple-server stream eg: rtsp://localhost:8554/mystream>" }
Janus and RTSP-Simple-Server must be on the same LAN because of UDP
Start Janus and watch the video under http://localhost/janusDemo/streamingtest.html. Press start and watch video
Video should play and after a minute or so, the stream will stop and restart.

Did you attach a network dump?
pcap.tar.gz

<IP_ADDRESS> is Janus
<IP_ADDRESS> is RTSP-Simple-Server

You can see after packet no. 8322 that rtsp-simple-server (<IP_ADDRESS>) stops sending RTPs  before Janus(<IP_ADDRESS>) restarts the connection at packet no. 8327
In this pcap, a colleague was watching the WebRTC steram on another machine so it was easier to isolate the communication between janus and rtsp-simple-server
Note: there is no VLC in this capture, but when rtsp-server stops sending to janus, it still sends to VLC

In case someone find this ticket with the same issue, the answer is here:
https://janus.conf.meetecho.com/docs/streaming.html
rtsp_session_timeout = by default the streaming plugin will check the RTSP connection with an OPTIONS query,
        the value of the timeout comes from the RTSP session initializer and by default
        this session timeout is the half of this value In some cases this value can be too high (for example more than one minute)
        because of the media server. In that case this plugin will calculate the timeout with this
        formula: timeout = min(session_timeout, rtsp_session_timeout / 2). (default=0s)

So I'm not sure how it works exactly but this option seems to override delay between two OPTIONS calls in case default value computed (from what ?) is to high.
From my understanding, it should be set to 60 to ensure a maximum delay of 30s between two OPTIONS calls but it did not work.
However setting rtsp_session_timeout = 30 fixed the issue with RTSP Simple Server.


add numpy to the installation instructions

The test suite depends on numpy since we import
https://github.com/data-apis/array-api-tests/blob/b47e667af1f216572862bb23c8b1d5ed2e9f0a3a/array_api_tests/hypothesis_helpers.py#L9
which in turn imports numpy unconditionally. Thus, we should add it to the installation instructions.

NumPy isn't supposed to be a dependency. We need to update hypothesis so that the array strategies can be array module independent. We need to do this anyway so that we can use the more powerful array generation strategies without depending on numpy. For now, though I guess we do depend on NumPy. It didn't occur to me that this would happen when I used the mutually_broadcastable_shapes strategy, but it's hard to go without that strategy.


[BUG] use of set as class attribute type causes exception

Describe the bug
If you define a class attribute's type as set pytkdocs crashes.
To Reproduce
class Example:
    """The example class is a stand-in class for illustration"""

    my_legal_attribute: List[str]

    my_illegal_attribute: set[str]

Expected behavior
pytkdocs would parse the attributes without breaking.
Error text
ERROR    -  mkdocstrings: 'type' object is not subscriptable
            Traceback (most recent call last):
              File "/home/user/repo/avenv/lib/python3.8/site-packages/pytkdocs/cli.py", line 205, in main
                output = json.dumps(process_json(line))
              File "/home/user/repo/avenv/lib/python3.8/site-packages/pytkdocs/cli.py", line 114, in process_json
                return process_config(json.loads(json_input))
              File "/home/user/repo/avenv/lib/python3.8/site-packages/pytkdocs/cli.py", line 91, in process_config
                obj = loader.get_object_documentation(path, members)
              File "/home/user/repo/avenv/lib/python3.8/site-packages/pytkdocs/loader.py", line 358, in get_object_documentation
                root_object = self.get_module_documentation(leaf, members)
              File "/home/user/repo/avenv/lib/python3.8/site-packages/pytkdocs/loader.py", line 426, in get_module_documentation
                root_object.add_child(self.get_class_documentation(child_node))
              File "/home/user/repo/avenv/lib/python3.8/site-packages/pytkdocs/loader.py", line 483, in get_class_documentation
                merge(attributes_data, get_class_attributes(parent_class))
              File "/home/user/repo/avenv/lib/python3.8/site-packages/pytkdocs/parsers/attributes.py", line 115, in get_class_attributes
                type_hints = get_type_hints(cls)
              File "/usr/lib/python3.8/typing.py", line 1232, in get_type_hints
                value = _eval_type(value, base_globals, localns)
              File "/usr/lib/python3.8/typing.py", line 270, in _eval_type
                return t._evaluate(globalns, localns)
              File "/usr/lib/python3.8/typing.py", line 518, in _evaluate
                eval(self.__forward_code__, globalns, localns),
              File "<string>", line 1, in <module>
            TypeError: 'type' object is not subscriptable
ERROR    -  Error reading page 'reference/api/Example.md':
ERROR    -  Could not collect 'repo.Example'

System (please complete the following information):

pytkdocs version: 0.16.1
Python version: 3.8
OS: Linux

Workaround:
class Example:
    """The example class is a stand-in class for illustration"""

    my_legal_attribute: List[str]

    # my_illegal_attribute: set[str]
    my_illegal_attribute = set()

A few questions:

I believe you're using Python 3.8, can you confirm?
do you actually from __future__ import annotations which allows using types as subscriptable objects?
can you see if #113 fixes it locally? it's a single line to add to pytkdocs/loader.py :slightly_smiling_face:

I can confirm I am using Python 3.8.
I am using from __future__ import annotations.
I tried your fix, which, added the line to my loader.py file like so:

And tried again, unfortunately I got the same error. ☹
I've had vs-code not save when I thought it did sometimes so I verified that the change was there via head  ./avenv/lib/python3.8/site-packages/pytkdocs/loader.py and it was in fact there, just not helping.
Thanks though 🙂

I can confirm that get_type_hints simply does not support future annotations on Python less than 3.9. So I'm inclined to say that the pytkdocs + future annotations combo is not supported on Python less than 3.9 :confused:
If you feel like something could be improved in the code (to bring support or at fail gracefully), please do open a PR, I will review it.

Thanks for looking into this @pawamoy! I don't have any recommended changes.


Added the opposite of slurp: spurt

Coverage increased (+0.006%) to 81.955% when pulling c1b7a481ecaf8e53ef2ddadb902ac6cbb6fc32ba on muffato:spurt into 29c98e98273070d29f356d32c038c9dd287f11c8 on Ensembl:master.

Looks reasonable and it comes with test cases, nice job :-)

And a 0.006% coverage increase ;)

Super, thank you ! I'll start using it in Compara :)


Reference to freed node equates to true in conditional

Godot version
v4.1.1.stable.official [bd6af8e0e]
System information
Godot v4.1.1.stable - Windows 10.0.19045 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 3080 (NVIDIA; <IP_ADDRESS>23) - AMD Ryzen 7 5800X 8-Core Processor (16 Threads)
Issue description
I have a reference to a node that is then deleted with queue_free() which causes the reference to be set to null.  The debugger tells me this reference is now  however it still passes an if condition as if it were not null.  Directly comparing it to null has the expected result (ie. freed_reference == null -> true) however just testing the reference itself does not (ie. if freed_reference: -> passes).
Steps to reproduce
Add a node to the scene and attach the following script.
extends Node

var freed_node: Node = null

func _ready():
	freed_node = Node.new()

func _process(delta):
	if freed_node != null:
		freed_node.queue_free() #Breakpoint here will be hit one time only, as expected
		return
	if freed_node:
		pass #Breakpoint here should never be hit but is

Minimal reproduction project
N/A

Use is_instance_valid(freed_node) to check for validity instead of a direct null check.

See also #59816.

Thanks for the report! Consolidating in https://github.com/godotengine/godot/issues/59816.


Documentation about the ajax callback is confusing

The README says that the ajax parameter takes a callback that should be called as follows:
  // 'callback' is a function that takes two parameters, 'data' and 'xhr'.
  //            'data' should be the key:value translation pairs for the
  //            requested language and namespace, or null in case of an error.
  //            'xhr' should be a status object, e.g. { status: 200 }

However, that's not what the included implementation does. When there is an error, it logs to the console and never calls the callback.
I tried to actually call with null on error and that did not work for me (at least not with the local storage backend in the next layer of a chained backend.) I could provide a minimal example if that helps here.
So, what is the right thing to do here? Just not call the callback as the default implementation does? Then the documentation should be updated.

more important is status as this is the trigger for sending back error with option for retry or fail permanently: https://github.com/i18next/i18next-xhr-backend/blob/master/src/index.js#L51

But the provided implementation does not actually report such errors as it does not seem to be calling the callback when an error happens. Or am I missing something?

like said...status is all that counts...read the code

When an exception happens, the callback is not called. It would probably be better to call it than to just log to the console. I'll try to create a PR for this and also improve the documentation a bit. It confused me at least. We can then discuss whether that's really an improvement :)

and what you like todo with the error? -> which only can be a network error (offline)...all other cases are handled with a retry or permanent fail which gets handled in the i18next events / callbacks

Reporting it to the caller with "no retry" seems most natural to me.

PR welcome


Update logback-classic to 1.4.1

Updates ch.qos.logback:logback-classic from 1.4.0 to 1.4.1.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "ch.qos.logback", artifactId = "logback-classic" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "ch.qos.logback", artifactId = "logback-classic" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Codecov Report
Base: 88.76% // Head: 88.76% // No change to project coverage :thumbsup:

Coverage data is based on head (77053e0) compared to base (91a6830).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #349   +/-   ##
=======================================
  Coverage   88.76%   88.76%
=======================================
  Files          91       91
  Lines        2812     2812
  Branches      157      157
=======================================
  Hits         2496     2496
  Misses        316      316

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

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


Enhancement/move token generate to user service

the point of having auth service was to be able to scale it separately if needed

I have added oauth2 here for feature, in case of authenticating other external services agains our auth service, seems for me natural to use it for user authentication as well then. our services are authenticated agains auth service using static client as well.

Yeps as discussed in the thread before we keep it as a feature for external services when such are needed and used but we have none so far. Thus we can save the hassle of calling that service when we can generate the token without going through the auth service and the whole oauth2 process. After all internal registration is not oauth2 and in the social auth registration we are the client so no need to use a server on our side.


mappings: change text to keyword type of PID and role fields

Closes https://github.com/inveniosoftware/cookiecutter-invenio-datamodel/issues/43
Tested with a cookiecutter instance (v3.1). Added 13 documents and checked that queries such as ?q=id:1 gave as a result only the document with id 1 and not 1,10,11,11, etc. Same for 2 and 12, 3 and 13.

This one has to go inveniosoftware/cookiecutter-invenio-instance after inveniosoftware/cookiecutter-invenio-instance#92 is merged.

Closed in favor of https://github.com/inveniosoftware/cookiecutter-invenio-instance/pull/95


path3 primitive in modeling

I know that you guys are working on v3 but if I create a pull request for a path3 primitive for v2 would it be considered? Of course I would follow the style guide but i just wanted to check before i spend time creating it.

@clupnyluke sure. But I'm wondering what other shapes are planned after path3 becomes available...

I mostly want it to be able to display paths in 3d space like using a bezier curve for a support or pipe through 3d space. I'm not sure what other people will create with it but I hope other people find use

There's another related issue #488

@clupnyluke supporting 3d lines in jscad requires also changes in entitiesToSolids function and renderer due to z values being ignored during conversion to webgl data.
I have work in progress that supports lines in 3d space on the rendering side, and is par of my long running effort to prepare friendlier reusable UI components for jscad.
firstly I did it inside a jscad branch, #944
Right no I am temporarily doing it outside jscad to be more free to maybe release less polished versions before bringing in it into official jscad.
https://github.com/hrgdavor/jscadui/issues/15
there is also a discord channel if you use discord. https://discord.com/invite/AaqGskur93
Part of this effort will be making vanilla js UI parts for jscad: gizmo, orbit controls, 3 render engines for jscad (regl, Three.js, Babylon.js) these will be trivial to wrap in Svelte or React. A developer that prefers react has joined there to provide React based examples.
Regardless of my work mentioned above, a contribution of 3d lines for jscad, would be very welcome :) if you are willing to make it.

@hrgdavor I defintely don't mind helping out and I will join discord thanks for the heads up

@clupnyluke lets continue the discussion here. Feel free to create a pull request, and let us know if you have any questions.


Minor fix to map scroll-jacking behaviour to make it easier to scroll the auckland hub page

merged


Fix #61 - Migrate to new runtime client, take II

Pull Request (PR) description
Based on PR #62 I changed a few accessors which are different with the new runtime.
This Pull Request (PR) fixes the following issues
Fixes #61

LGTM though I don't have a way to test

Is there anything missing for merging?

Any objection to getting this merged?


More guidance / notice on roadmap and breaking changes

At the moment, the roadmap and plans for Nuget server is difficult to know. Keeping up with changes to the server API if you're working on a client that interfaces with it is hard e.g. Paket, Rider, MyGet, AppVeyor etc. etc.
As an example, there's a change coming up to the server API for SemVer2 support which we on the Paket team only found out about purely by chance. It would really help if the same sort of information flow that goes to the NuGet client team would go out publicly so all products that depend on the server API could know in advance what's upcoming, rather than just having to find out further down the road etc..
Even better would be to allow others to join in the discussion for features when they're just at the design stage - perhaps we'll have some great ideas that can make them even better. As an tiny example, Paket already found an issue in the SemVer2 support - which has now been fixed, so a win for everyone.
Cheers

@isaacabraham We did have a spec  out for discussion quite some time before we started to execute on it. But I guess we can do better than that. I am discussing with the team on how to make these specs and future work more visible.

Great, that would be a good start. I notice (unless I've misunderstood) that there was a breaking change pushed out today / yesterday that broke Paket as well :( Thanks to Nuget team to helping to help resolve the issue but again forward notice / discussions would really help :)

@anangaur, your team could use the same approach as dotnet team - create a repo for announcement, where in issues you could share proposals which could be discussed by community.

@dtretyakov An announcements repo would be a fantastic addition, although it would also have to go beyond a reactive announcements repo ("after the fact" announcements). Ideally it would also announce specs that are still in progress. This allows better community participation in the design/implementation process if anyone out there wants to get involved.

@isaacabraham @dtretyakov @maartenba
Here is the https://github.com/NuGet/Announcements repo. The intention is not just "after the fact" announcements but also to notify new & important features before the implementation. Especially the ones with breaking changes.
We will soon be announcing the new features once they are ready for reviewing

Thumbs up! Can existing ones announced in the latest blog post be added there as well?

Yes ! As they become ready for review.


cpPivotJoint jitter?

I'm trying to move dynamic body with mouse input and still keep body response to simulation.
the dynamic body is connect with a kinematic body by pivot joint. after read the souce code, I know set kinematic body to new position will move dynamic body to the position gradually with velocity keep in sync and position 'drift' biased. so I set kinematic body to the position every time user mouse click to the new position.
Then I found if I set maxFoce of pivot joint to a high value like Infinity, and click with reverse position to current dynamic body's move direction, the dynamic body will jitter (It may due to a suddenly velocity change) and turn move direction then. if I set maxForce to some low value the jitter gone, but the acceleration was too slow like a damping spring effect, that was not I want.
Is there a proper way to move dynamic body and turn it's direction quickly, but still keep simulation stable?

Sorry for the late reply. Unfortunately I've never found an ideal way to handle this. It's pretty necessary to set some non-infinite limit to the correction force when applying using input. Otherwise you end up with a lot of sudden starts and stops every couple frames. (Mouse input is not always as smooth as you think)
There's probably some smart way to do it by multiplying some filter coefficient against the apparent mass at the joint's anchor, but I've never gotten around to trying it. Until then, multiplying against the object's mass might be the best you can do.


stub-zone could be specified with either ip or hostname

This enables specifying stub zones with either or both of hostname(s) and ip address(s). We may want to create stub-zone settings in unbound.conf using stub-host: option in some case.
e.g. When using external nameserver services for some ip segment, it's likely that we would prefer specifying multiple name servers' hostnames to doing so with those ip addresses.

I like the idea, though the define needs some work.  Perhaps setting the default values to undef and requiring that at most one is set.

Thank you for the review and comment. I intended that adding some flexibility might help in some case so that stubbing targets for one stub-zone can be specified with both of ip addresses and hostnames. Anyway the commit was incorrect in the sense that any stub-addr config can only be specified before any stub-hosts. So I will post another commit later. I would appreciate your reviewing.

Thanks for the comments. I made another commit.
https://github.com/K-DEN/puppet-unbound/commit/987cf0362cb6eed2df30564b0165b82f2f2c0b0e

separate defined resource type for address(es) validation added.
for compatibility, unbound::stub args not changed.
made it accomplish validation and to retain addresses order user provided, like in all the following cases,

address => ["{domain name 1}", "{ip address 1}", "{domain name 2}", ...]
e.g.

stub-addr: {domain name 1}
stub-host: {ip address 1}
stub-addr: {domain name 2}
:

* stub.erb template still contains is_domain_name. if you would come up with a nice way to remove it, I'd appreciate your help.

Thank you for the comment. I made another commit. (one test case just for port validation)
https://github.com/K-DEN/puppet-unbound/commit/b1488bf039bbb0149c0e0773723132eda66bac50

This is great, thank you.


insufficient system memory in resource pool to run this query

Running Docker in Fedora 26, and trying to restore a database from backup, I get the error:

SQL Error [701] [S00013]: There is insufficient system memory in resource pool 'default' to run this query.

Some more information:
The command that I'm using to run Docker:
# docker run --name mssql -e 'ACCEPT_EULA=Y' -e 'SA_PASSWORD=secret' \
  -p 1433:1433 -v /home/igal/Documents/docker-mssql-fs/:/docker-mssql-fs \
  --memory=16g -d microsoft/mssql-server-linux

SELECT @@VERSION:
Microsoft SQL Server 2017 (RC2) - 14.0.900.75 (X64)
Jul 27 2017 08:53:49
Copyright (C) 2017 Microsoft Corporation
Developer Edition (64-bit) on Linux (Ubuntu 16.04.2 LTS)

master.sys.configurations WHERE name LIKE '%memory%'
configuration_id |name                      |value      |value_in_use |description                            |is_dynamic |is_advanced |
-----------------|--------------------------|-----------|-------------|---------------------------------------|-----------|------------|
1505             |index create memory (KB)  |0          |0            |Memory for index create sorts (kBytes) |1          |1           |
1544             |max server memory (MB)    |2147483647 |2147483647   |Maximum size of server memory (MB)     |1          |1           |
1540             |min memory per query (KB) |1024       |1024         |minimum memory per query (kBytes)      |1          |1           |
1543             |min server memory (MB)    |0          |16           |Minimum size of server memory (MB)     |1          |1           |

RESTORE FILELISTONLY
FROM DISK = '/docker-mssql-fs/mydatabase.bak'
LogicalName    |PhysicalName                 |Type |FileGroupName |Size       |MaxSize        |FileId |CreateLSN |DropLSN |UniqueId                             |ReadOnlyLSN |ReadWrit
---------------|-----------------------------|-----|--------------|-----------|---------------|-------|----------|--------|-------------------------------------|------------|--------
mydatabase_dat |C:\SQLData\mydatabase.mdf    |D    |PRIMARY       |5411700736 |35184372080640 |1      |0         |0       |00000000-0000-0000-0000-000000000000 |0           |0
mydatabase_log |C:\SQLData\mydatabase_1.ldf  |L    |              |2011627520 |35184372080640 |2      |0         |0       |00000000-0000-0000-0000-000000000000 |0           |0

And the command that throws that error:
RESTORE DATABASE mydatabase
FROM DISK = '/docker-mssql-fs/mydatabase.bak'
WITH 	MOVE 'mydatabase_dat' TO '/docer-mssql-fs/mydatabase_dat',
	MOVE 'mydatabase_log' TO '/docer-mssql-fs/mydatabase_log',
	STATS = 1

How can I resolve that?
Thanks!

Looks like there is a typo in your RESTORE command:
'/docer-mssql-fs/mydatabase_dat'
should be:
'/docker-mssql-fs/mydatabase_dat'
Could that be it?

@twright-msft Good eye!  I did, indeed, had a typo there, but after fixing it I still get the same error.  I think that the process doesn't even get to the point where it tries to validate the path.
Any ideas on how to increase memory size etc. to the Docker MSSQL instance?  I tried some DBCC commands but kept getting the same error no matter what I tried.
Thanks!

I am having the same issue as @isapir.

@rekahsoft Can you share some information about the database size and/or memory settings?

Sure @isapir. I am restoring a 6GB full backup. I was using AWS and used a couple different instances, with 8GB, 16GB and 32GB of RAM. I found that no matter what, mssql would only use 8GB, and crash once it reached the 8GB threshold. I used the following command to run the mssql docker image:
docker run -e 'ACCEPT_EULA=Y' -e 'SA_PASSWORD=e3EZJSnd0fR39AH01AFCg06OP' -p 1433:1433 -v $(pwd)/ge-db:/tmp/db -d microsoft/mssql-server-linux

And then restored the backup using:
docker exec -it $(docker ps -aq | head -n1) /opt/mssql-tools/bin/sqlcmd -S localhost -U sa -P 'e3EZJSnd0fR39AH01AFCg06OP' -Q "RESTORE DATABASE pmps FROM DISK='/tmp/db/pmps.bak' WITH MOVE 'PM_Log' TO '/var/opt/mssql/data/PM_Log.ldf', MOVE 'PM_DB' TO '/var/opt/mssql/data/PM_DB.mdf'"

After a while, I receive the error: There is insufficient system memory in resource pool 'default' to run this query..
I also tried setting the maximum memory mssql setting to 32GB prior to attempting to restore the backup (based on https://docs.microsoft.com/en-us/sql/database-engine/configure-windows/server-memory-server-configuration-options) like so:
sp_configure 'show advanced options', 1;
GO
RECONFIGURE;
GO
sp_configure 'max server memory', 32768;
GO
RECONFIGURE;
GO

However, I still received the same error, and mssql does not use all RAM. As far as I'm aware, the express version of mssql for windows has a 8GB memory limit. Perhaps this is the case for the mssql docker image as well?

We're experiencing the same issue with the :latest, :2017-latest and :GA-latest images.
We tried to restore a 90Mb .bak file. The host is a Linux so the mssql-server-linux container has full access to the host ram space which is 24Gb.
We tried running from within the container while making sure all the files and path had the right user access levels and nothing changed on that end.

Same issue here. I wanted to restore a 10MB backup and got the same error There is insufficient system memory in resource pool 'default' to run this query. (Microsoft SQL Server, Error: 701). The Docker-Machine has 10GB memory overall.
Version: Microsoft SQL Server 2017 (RTM-CU1) (KB4038634) - 14.0.3006.16 (X64)   Oct 19 2017 02:42:29   Copyright (C) 2017 Microsoft Corporation  Developer Edition (64-bit) on Linux (Ubuntu 16.04.3 LTS)

I wanted to restore a 10MB backup and got the same error There is insufficient system memory

Then I guess it's even worse than I thought.  I'm surprised that this issue is not given any priority at all.

Then I guess it's even worse than I thought. I'm surprised that this issue is not given any priority at all.

Yea. I got an workaround for this. I installed a Windows Server 2016 VM with MSSQL Server 2016 and the MSSQL Server Manager, after that I attached the database, restored it with the backup and created a new backup, then I was able to use the new backup on the Docker MSSQL server.

So a backup from MSSQL Server 2016 is restored in the Docker MSSQL Server but a backup by an older server does not?  The backups I was testing were generated by MSSQL 2008.

@isapir yes, looks exactly like that.

I have the same problem with a backup coming from MSSQL 2008R2. Another backup coming from MSSQL 2008R2 is restoring just fine however.

@rvdginste Can you think of some of the differences between the two backups?

@isapir I tried to restore 8 databases (coming from MSSQL 2008R2), 5 succeeded and 3 failed. The only thing I can tell is that the biggest ones (mdf size) are failing.  The smallest of those 3 is ~290MB, the biggest of those 5 is in size just below that.

Have the same issue, haven't tried the VM option yet, but seems like the thread may have gone quiet?

I'm having the same issue while I'm trying to install the mssql-server-linux through docker-compose up.
database:
image: microsoft/mssql-server-linux:2017-GA
environment:
- ACCEPT_EULA=Y
- SA_PASSWORD=AceSqlDB123
- USER=sa
ports:
- "1433:1433"
when it try with docker-compose up.
database_1        | 2018-03-28 15:55:29.41 Server      Setup step is copying system data file 'C:\templatedata\master.mdf' to '/var/opt/mssql/data/master.mdf'.
2018-03-28 15:55:29.52 Server      Did not find an existing master data file /var/opt/mssql/data/master.mdf, copying the missing default master and other system database files. If you have moved the database location, but not moved the database files, startup may fail. To repair: shutdown SQL Server, move the master database to configured location, and restart.
2018-03-28 15:55:29.52 Server      Setup step is copying system data file 'C:\templatedata\mastlog.ldf' to '/var/opt/mssql/data/mastlog.ldf'.
2018-03-28 15:55:29.53 Server      Setup step is copying system data file 'C:\templatedata\model.mdf' to '/var/opt/mssql/data/model.mdf'.
2018-03-28 15:55:29.56 Server      Setup step is copying system data file 'C:\templatedata\modellog.ldf' to '/var/opt/mssql/data/modellog.ldf'.
2018-03-28 15:55:29.59 Server      Setup step is copying system data file 'C:\templatedata\msdbdata.mdf' to '/var/opt/mssql/data/msdbdata.mdf'.
2018-03-28 15:55:29.63 Server      Setup step is copying system data file 'C:\templatedata\msdblog.ldf' to '/var/opt/mssql/data/msdblog.ldf'.
2018-03-28 15:55:29.74 Server      Microsoft SQL Server 2017 (RTM-CU5) (KB4092643) - 14.0.3023.8 (X64)
databaseMar  2 2018 18:24:44
databaseCopyright (C) 2017 Microsoft Corporation
databaseDeveloper Edition (64-bit) on Linux (Ubuntu 16.04.4 LTS)
2018-03-28 15:55:29.74 Server      UTC adjustment: 0:00
2018-03-28 15:55:29.74 Server      (c) Microsoft Corporation.
2018-03-28 15:55:29.74 Server      All rights reserved.
2018-03-28 15:55:29.74 Server      Server process ID is 4124.
2018-03-28 15:55:29.74 Server      Logging SQL Server messages in file '/var/opt/mssql/log/errorlog'.
2018-03-28 15:55:29.74 Server      Registry startup parameters:
database -d /var/opt/mssql/data/master.mdf
database -l /var/opt/mssql/data/mastlog.ldf
database -e /var/opt/mssql/log/errorlog
2018-03-28 15:55:29.75 Server      SQL Server detected 6 sockets with 1 cores per socket and 1 logical processors per socket, 6 total logical processors; using 6 logical processors based on SQL Server licensing. This is an informational message; no user action is required.
2018-03-28 15:55:29.75 Server      SQL Server is starting at normal priority base (=7). This is an informational message only. No user action is required.
2018-03-28 15:55:29.75 Server      Detected 9595 MB of RAM. This is an informational message; no user action is required.
2018-03-28 15:55:29.75 Server      Using conventional memory in the memory manager.
2018-03-28 15:55:29.81 Server      Large Page Allocated: 32MB
2018-03-28 15:55:30.25 Server      Buffer pool extension is already disabled. No action is necessary.
2018-03-28 15:55:30.46 Server      InitializeExternalUserGroupSid failed. Implied authentication will be disabled.
2018-03-28 15:55:30.47 Server      Implied authentication manager initialization failed. Implied authentication will be disabled.
2018-03-28 15:55:30.48 Server      Successfully initialized the TLS configuration. Allowed TLS protocol versions are ['1.0 1.1 1.2']. Allowed TLS ciphers are ['ECDHE-ECDSA-AES128-GCM-SHA256:ECDHE-ECDSA-AES256-GCM-SHA384:ECDHE-RSA-AES128-GCM-SHA256:ECDHE-RSA-AES256-GCM-SHA384:ECDHE-ECDSA-AES128-SHA256:ECDHE-ECDSA-AES256-SHA384:ECDHE-ECDSA-AES256-SHA:ECDHE-ECDSA-AES128-SHA:AES256-GCM-SHA384:AES128-GCM-SHA256:AES256-SHA256:AES128-SHA256:AES256-SHA:AES128-SHA:!DHE-RSA-AES256-GCM-SHA384:!DHE-RSA-AES128-GCM-SHA256:!DHE-RSA-AES256-SHA:!DHE-RSA-AES128-SHA'].
2018-03-28 15:55:30.57 Server      Node configuration: node 0: CPU mask: 0x000000000000003f:0 Active CPU mask: 0x000000000000003f:0. This message provides a description of the NUMA configuration for this computer. This is an informational message only. No user action is required.
2018-03-28 15:55:30.57 Server      The maximum number of dedicated administrator connections for this instance is '1'
2018-03-28 15:55:30.67 Server      Using dynamic lock allocation.  Initial allocation of 2500 Lock blocks and 5000 Lock Owner blocks per node.  This is an informational message only.  No user action is required.
2018-03-28 15:55:30.70 Server      In-Memory OLTP initialized on standard machine.
2018-03-28 15:55:30.80 Server      Database Instant File Initialization: enabled. For security and performance considerations see the topic 'Database Instant File Initialization' in SQL Server Books Online. This is an informational message only. No user action is required.
2018-03-28 15:55:30.83 Server      Query Store settings initialized with enabled = 1,
2018-03-28 15:55:30.84 spid5s      Starting up database 'master'.
2018-03-28 15:55:30.87 Server      Software Usage Metrics is disabled.
2018-03-28 15:55:31.16 spid5s      The tail of the log for database master is being rewritten to match the new sector size of 4096 bytes.  3072 bytes at offset 418816 in file /var/opt/mssql/data/mastlog.ldf will be written.
2018-03-28 15:55:31.47 spid5s      Converting database 'master' from version 862 to the current version 869.
2018-03-28 15:55:31.48 spid5s      Database 'master' running the upgrade step from version 862 to version 863.
2018-03-28 15:55:31.50 spid5s      Database 'master' running the upgrade step from version 863 to version 864.
2018-03-28 15:55:31.54 spid5s      Database 'master' running the upgrade step from version 864 to version 865.
2018-03-28 15:55:31.55 spid5s      Database 'master' running the upgrade step from version 865 to version 866.
2018-03-28 15:55:31.57 spid5s      Database 'master' running the upgrade step from version 866 to version 867.
2018-03-28 15:55:31.59 spid5s      Database 'master' running the upgrade step from version 867 to version 868.
2018-03-28 15:55:31.61 spid5s      Database 'master' running the upgrade step from version 868 to version 869.
2018-03-28 15:55:31.88 spid5s      Resource governor reconfiguration succeeded.
2018-03-28 15:55:31.90 spid5s      SQL Server Audit is starting the audits. This is an informational message. No user action is required.
2018-03-28 15:55:31.92 spid5s      SQL Server Audit has started the audits. This is an informational message. No user action is required.
2018-03-28 15:55:32.05 spid5s      SQL Trace ID 1 was started by login "sa".
2018-03-28 15:55:32.07 spid5s      Server name is '02a6db196fd6'. This is an informational message only. No user action is required.
2018-03-28 15:55:32.08 spid18s     Password policy update was successful.
2018-03-28 15:55:32.18 spid20s     Always On: The availability replica manager is starting. This is an informational message only. No user action is required.
2018-03-28 15:55:32.19 spid5s      Starting up database 'msdb'.
2018-03-28 15:55:32.19 spid9s      Starting up database 'mssqlsystemresource'.
2018-03-28 15:55:32.20 spid20s     Always On: The availability replica manager is waiting for the instance of SQL Server to allow client connections. This is an informational message only. No user action is required.
2018-03-28 15:55:32.24 spid9s      The resource database build version is 14.00.3023. This is an informational message only. No user action is required.
2018-03-28 15:55:32.30 spid9s      Starting up database 'model'.
2018-03-28 15:55:32.57 spid18s     A self-generated certificate was successfully loaded for encryption.
2018-03-28 15:55:32.58 spid18s     Server is listening on [ 'any' <ipv4> 1433].
2018-03-28 15:55:32.58 Server      Server is listening on [ <IP_ADDRESS> <ipv4> 1434].
2018-03-28 15:55:32.58 Server      Dedicated admin connection support was established for listening locally on port 1434.
2018-03-28 15:55:32.60 spid5s      The tail of the log for database msdb is being rewritten to match the new sector size of 4096 bytes.  512 bytes at offset 52736 in file /var/opt/mssql/data/MSDBLog.ldf will be written.
2018-03-28 15:55:32.61 spid18s     SQL Server is now ready for client connections. This is an informational message; no user action is required.
2018-03-28 15:55:32.64 spid5s      Converting database 'msdb' from version 862 to the current version 869.
2018-03-28 15:55:32.64 spid5s      Database 'msdb' running the upgrade step from version 862 to version 863.
2018-03-28 15:55:32.67 spid5s      Database 'msdb' running the upgrade step from version 863 to version 864.
2018-03-28 15:55:32.69 spid5s      Database 'msdb' running the upgrade step from version 864 to version 865.
2018-03-28 15:55:32.70 spid5s      Database 'msdb' running the upgrade step from version 865 to version 866.
2018-03-28 15:55:32.71 spid5s      Database 'msdb' running the upgrade step from version 866 to version 867.
2018-03-28 15:55:32.72 spid9s      The tail of the log for database model is being rewritten to match the new sector size of 4096 bytes.  2048 bytes at offset 75776 in file /var/opt/mssql/data/modellog.ldf will be written.
2018-03-28 15:55:32.72 spid5s      Database 'msdb' running the upgrade step from version 867 to version 868.
2018-03-28 15:55:32.73 spid5s      Database 'msdb' running the upgrade step from version 868 to version 869.
2018-03-28 15:55:32.74 spid9s      Converting database 'model' from version 862 to the current version 869.
2018-03-28 15:55:32.74 spid9s      Database 'model' running the upgrade step from version 862 to version 863.
2018-03-28 15:55:32.76 spid9s      Database 'model' running the upgrade step from version 863 to version 864.
2018-03-28 15:55:32.78 spid9s      Database 'model' running the upgrade step from version 864 to version 865.
2018-03-28 15:55:32.78 spid9s      Database 'model' running the upgrade step from version 865 to version 866.
2018-03-28 15:55:32.79 spid9s      Database 'model' running the upgrade step from version 866 to version 867.
2018-03-28 15:55:32.80 spid9s      Database 'model' running the upgrade step from version 867 to version 868.
2018-03-28 15:55:32.81 spid9s      Database 'model' running the upgrade step from version 868 to version 869.
2018-03-28 15:55:32.92 spid9s      Polybase feature disabled.
2018-03-28 15:55:32.92 spid9s      Clearing tempdb database.
2018-03-28 15:55:33.40 spid9s      Starting up database 'tempdb'.
2018-03-28 15:55:33.71 spid9s      The tempdb database has 1 data file(s).
2018-03-28 15:55:33.71 spid24s     The Service Broker endpoint is in disabled or stopped state.
2018-03-28 15:55:33.71 spid24s     The Database Mirroring endpoint is in disabled or stopped state.
2018-03-28 15:55:33.73 spid24s     Service Broker manager has started.
2018-03-28 15:55:33.84 spid5s      Recovery is complete. This is an informational message only. No user action is required.
2018-03-28 15:55:33.85 spid20s     The default language (LCID 0) has been set for engine and full-text services.
2018-03-28 15:55:34.41 spid51      Configuration option 'show advanced options' changed from 0 to 1. Run the RECONFIGURE statement to install.
2018-03-28 15:55:34.46 spid51      Configuration option 'Agent XPs' changed from 0 to 1. Run the RECONFIGURE statement to install.
2018-03-28 15:55:34.48 spid51      Configuration option 'show advanced options' changed from 1 to 0. Run the RECONFIGURE statement to install.
2018-03-28 15:55:34.68 spid51      Attempting to load library 'xpsqlbot.dll' into memory. This is an informational message only. No user action is required.
2018-03-28 15:55:34.70 spid51      Using 'xpsqlbot.dll' version '2017.140.3023' to execute extended stored procedure 'xp_qv'. This is an informational message only; no user action is required.
2018-03-28 15:55:34.79 spid51      Attempting to load library 'xpstar.dll' into memory. This is an informational message only. No user action is required.
2018-03-28 15:55:34.83 spid51      Using 'xpstar.dll' version '2017.140.3023' to execute extended stored procedure 'xp_sqlagent_notify'. This is an informational message only; no user action is required.
2018-03-28 15:55:37.60 spid54 Using 'dbghelp.dll' version '4.0.5'`

@siddthota - it looks like your container came up fine.  That's just the end of the initial output.  Have you tried connecting to it?  I dont think it is the same issue that was originally raised by @isapir with restoring database backups.  Or is there something about your situation that you didnt explain which ties it to the original issue?

status?

I received this error as well..  The database itself has been up and working.. I ran it on my system (Win 10 Pro) and left it up for a couple days without issue..  Then I simply tried to query a table (which turned out to be empty) and received this error.
docker stop and docker start appears to have resolved the error for now.

It looks like this is fixed in SQL Server 2019, using the mcr.microsoft.com/mssql/server:vNext-CTP2.0-ubuntu image I am able to restore a bak file that previously gave me the insufficient system memory in resource pool  error

It looks like this is fixed in SQL Server 2019, using the mcr.microsoft.com/mssql/server:vNext-CTP2.0-ubuntu image I am able to restore a bak file that previously gave me the insufficient system memory in resource pool  error

same image also resolved the issue for me.

I gave a try with mcr.microsoft.com/mssql/server:vNext-CTP2.0-ubuntu but still got the same issue.
Is there any other options which I might look into or any debug-info which I could share?

I was getting this same error on Docker for Mac. I went into the docker preferences, increased the memory allocated to the VM, then restarted docker and started back up the container. This allowed me to import a larger sql file without the insufficient system memory error.

I was getting this same error on Docker for Mac. I went into the docker preferences, increased the memory allocated to the VM, then restarted docker and started back up the container. This allowed me to import a larger sql file without the insufficient system memory error.

Works for me!

I'm seeing the same error when running certain queries.  However when I watch the memory usage on the container, it never even gets close to maximum.

I'm seeing the same error when running certain queries. However when I watch the memory usage on the container, it never even gets close to maximum.
I'm using the following image mcr.microsoft.com/mssql/rhel/server:2019-latest

Try increase memory on docker preferences, maybe 6GB

I have increased the memory of the container to 6GB.  It appears to resolve the issue.  I suspect the container and SQL Server are not communicating with each other correctly about available memory.

Running Docker in Fedora 26, and trying to restore a database from backup, I get the error:

SQL Error [701] [S00013]: There is insufficient system memory in resource pool 'default' to run this query.

Some more information:
The command that I'm using to run Docker:
# docker run --name mssql -e 'ACCEPT_EULA=Y' -e 'SA_PASSWORD=secret' \
  -p 1433:1433 -v /home/igal/Documents/docker-mssql-fs/:/docker-mssql-fs \
  --memory=16g -d microsoft/mssql-server-linux

This error happens in some situation where the configured max memory parameter is over the operational system RAM or over SQL Edition limitation.
You can see each SQL 2017 Edition limitations here:
https://docs.microsoft.com/en-us/sql/sql-server/editions-and-components-of-sql-server-2017?view=sql-server-ver15
For example, if you are runnig the SQL 2017 Express Edition in a VM with 4GB RAM, you must limit SQL max memory param with less than 1410MB (1.4GB), because the 2017 Express edition crashs if you exceed these memory limit with error 701. I recommend 1300MB in this scenario.
Another example: If you are running the 2017 Web edition, that supports max memory 64GB, in a VM with 8GB RAM, I recommend you limit the SQL Max memory param in 6GB, leaving 2GB to OS (Ubuntu). If you don't limit it, you can get the same error 701.
To set this max server memory parameter, you can use this script:
sp_configure 'show advanced options', 1;
GO
RECONFIGURE;
GO
sp_configure 'max server memory', 1300;
GO
RECONFIGURE;
GO

You also can check how to pass this parameter to Docker run instruction to prevent the crash on container load process.
Try -e MSSQL_MEMORYLIMIT_MB=1300 in the Docker run command

I had a similar issue while running mysql in docker. Few things to take into consideration:
1. Increase the memory allocation of docker.
   You can do it easily using docker desktop dashboard.
2. Run the docker-compose with an admin access.
       For an example: sudo docker-compose -f mysql.docker-compose.yml up -d

Point 2 above will resolve inaccessible file issue.
The above I have done in my MacBook pro 11.5.1


Fix clustermap for sparse X

Fixes #356 for me
It's a pretty simple change. It's a little hard for me to add a test at the moment, but I'm pretty sure this works.

Fine, thank you! :smile:


⚠ Bofrost.de has degraded performance

In eeb4f4d, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 6069 ms

Resolved: Bofrost.de performance has improved in cd154f5.


Review format of Annotation.annotationSource

The current design is based on the format adopted by Google Healthcare API.

Creating and managing annotation records
Google AnnotationSource schema (beta)

First, Google defined the object CloudHealthcareSource that includes a single property: name. This name can be used to build a URL to get the resource. This approach allows to easily reference different objects such as our Note schema or any (FHIR) schema that could be added in the future. Simplify storing the DB record ID is not possible and advised because: 1) ObjectId in MongoDB are not unique across multiple collections, 2) does not implement ID opacity (good practice for security).
We could have:
properties:
    annotationSource:
        type: string
        example: "name/of/a/resource"

This implementation is the most straightforward. However it requires the user of the API to read the doc to figure out what the value of annotationSource must be the name of a resource. This design also limit future extension and is more likely to introduce breaking change than the approach adopted by Google with
properties:
    annotationSource:
        schema:
            $ref: CloudHealthcareSource.yaml

where CloudHealthcareSource.yaml would be
properties:
    name:
        type: string
        example: "name/of/a/resource"

One assumption we make if annotationSource is a string and not a schema that we can extend in the future is that the source of an annotation must be accessible through the same API service instance as where the annotation objects are saved. However, there could be benefit to storing annotation to a data node instance that is different from the data node that includes the source information, for example for scalability and performance reason.
Proposal
Defining CloudHealthcareSource.yaml limit the risk of introducing breaking changes in the future and provide more flexibility. Thus, I would recommend adopting this design.

Related to #89


Plan Payments

EASI-2056
Changes and Description

SQL and store methods for Plan Payments
Resolvers for Plan Payments
GraphQL definition and endpoints for Plan Payments

How to test this change

PR Author Review Checklist

[ ] Met the ticket's acceptance criteria, or will meet them in a subsequent PR.
[ ] Added or updated tests for backend resolvers or other functions as needed.
[ ] Added or updated client tests for new components, parent components, functions, or e2e tests as necessary.
[ ] Tested user-facing changes with voice-over and the rotor menu

PR Reviewer Guidelines

It's best to pull the branch locally and test it, rather than just looking at the code online!
Check that all code is adequately covered by tests - if it isn't feel free to suggest the addition of tests.
Always make comments, even if it's minor, or if you don't understand why something was done.

@OddTomBrooks, wanted to point out there is a pre-commit check error that needs to be resolved. (I'm not sure how this got by pre-commit locally, perhaps Go-Land bypassed it?)

The pre-commit checks should be agnostic to IDE. How fascinating. I’ll look
into it.
On Wed, Jun 29, 2022 at 4:35 PM Steven Wade @.***>
wrote:

@OddTomBrooks https://github.com/OddTomBrooks, wanted to point out
there is a pre-commit check error
https://github.com/CMSgov/mint-app/runs/7104375535?check_suite_focus=true
that needs to be resolved. (I'm not sure how this got by pre-commit
locally, perhaps Go-Land bypassed it?)
—
Reply to this email directly, view it on GitHub
https://github.com/CMSgov/mint-app/pull/109#issuecomment-1170468524, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AX277I77UA77NJVYJBROA5DVRSXRJANCNFSM5YINAWTQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

The pre-commit checks should be agnostic to IDE. How fascinating. I’ll look into it.
…
On Wed, Jun 29, 2022 at 4:35 PM Steven Wade @.> wrote: @OddTomBrooks https://github.com/OddTomBrooks, wanted to point out there is a pre-commit check error https://github.com/CMSgov/mint-app/runs/7104375535?check_suite_focus=true that needs to be resolved. (I'm not sure how this got by pre-commit locally, perhaps Go-Land bypassed it?) — Reply to this email directly, view it on GitHub <#109 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AX277I77UA77NJVYJBROA5DVRSXRJANCNFSM5YINAWTQ . You are receiving this because you were mentioned.Message ID: @.>

Yeah it's interesting, just a random thought. VsCode can give me trouble when I try to commit through the UI, so I just command line. Made me wonder if go-land is causing an issue too? IDK


add sklearn-crfsuite install instructions

fixing rst
test
Proposed changes:

add explicit install of sklearn-crfsuite ( see issue #442 )

Status:

[ ] ready for code review
[ ] there are tests for the functionality
[ ] documentation updated
[ ] changelog updated

Thanks a lot, should fix installation troubles before they occur!


Feature/download survey user tester optin

This PR adds an option for users to sign to be tester of future platform developments
Pivotal
Basecamp

Was hard for me to understand why I couldn't activate the checkbox until you write an email so I propose:

Short version: at least change the cursor when it is disabled
Longer version: Include an error warning prop when trying to activate it to describe why it is disabled

@j8seangel, @Bluesmile82 ready to review again.
Maybe the errorText logic inside the checkbox component is quite ad hoc for this case but could be improved in future use cases.


Fixes Issue2110: Set transaction type at output level

To enable a spendable refund output in a freeze transaction we set the Payment, Freeze or Coinbase enum on each output.

The last commit is to enable the Integration test to run as an updated Faucet is required. When green and approved for merging I will drop it and once merged we will need to update Faucet PR: https://github.com/bosagora/faucet/pull/87.

Green again.


Bug in bugscanner

When i type bugscanner bug.txt i get
Traceback (most recent call last):
File "/data/data/com.termux/files/usr/bin/bugscanner", line 33, in
sys.exit(load_entry_point('bugscanner==0.1.8', 'console_scripts', 'bugscanner')())
File "/data/data/com.termux/files/usr/bin/bugscanner", line 25, in importlib_load_entry_point
return next(matches).load()
File "/data/data/com.termux/files/usr/lib/python3.10/importlib/metadata/init.py", line 162, in load
module = import_module(match.group('module'))
File "/data/data/com.termux/files/usr/lib/python3.10/importlib/init.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 1050, in _gcd_import
File "", line 1027, in _find_and_load
File "", line 992, in _find_and_load_unlocked
File "", line 241, in _call_with_frames_removed
File "", line 1050, in _gcd_import
File "", line 1027, in _find_and_load
File "", line 1006, in _find_and_load_unlocked
File "", line 688, in _load_unlocked
File "", line 883, in exec_module
File "", line 241, in _call_with_frames_removed
File "/data/data/com.termux/files/usr/lib/python3.10/site-packages/bugscanner/init.py", line 1, in
from .bug_scanner import BugScanner
File "/data/data/com.termux/files/usr/lib/python3.10/site-packages/bugscanner/bug_scanner.py", line 1, in
import multithreading
File "/data/data/com.termux/files/usr/lib/python3.10/site-packages/multithreading/init.py", line 3, in
from multithreading.multithread_request import MultiThreadRequest
File "/data/data/com.termux/files/usr/lib/python3.10/site-packages/multithreading/multithread_request.py", line 4, in
import requests
ModuleNotFoundError: No module named 'requests'

Run this command
python3 -m pip install requests

Yes, execute command like @hungnm1518 said. Or if you want bugscanner new version take a look at this repo https://github/aztecrabbit/bugscanner-go


Feature request: Sponge removes fish from sink

My wife was upset that the sink was dirty.
Maybe you could use a sponge to clean it  :)
Great mod!  keep up the amazing work.

The fish is leaving to find a new home with the upcoming 1.14 port.


⚠ jaswdr.dev has degraded performance

In d7eda79, jaswdr.dev (https://jaswdr.dev/) experienced degraded performance:

HTTP code: 200
Response time: 610 ms

Resolved: jaswdr.dev performance has improved in a74f731.


feat(intro): update signup description.

Issue: none
What I did
Update introduction to have admin signup instruction.
How to test
Pure text. Not test needed.

Deploy preview ready!
Built with commit 502ee00026ba237120b4b8fd3dcd8e9fa5d4b7bc
https://deploy-preview-425--mcs-lite-ui.netlify.com

Deploy preview ready!
Built with commit 502ee00026ba237120b4b8fd3dcd8e9fa5d4b7bc
https://deploy-preview-425--mcslite.netlify.com

Deploy preview ready!
Built with commit 502ee00026ba237120b4b8fd3dcd8e9fa5d4b7bc
https://deploy-preview-425--mcs-lite-introduction.netlify.com